Tuesday, 2018-03-13

*** udesale has joined #storyboard04:03
*** jamesmcarthur has joined #storyboard04:17
*** jamesmcarthur has quit IRC04:22
*** jtomasek has joined #storyboard08:30
*** tosky has joined #storyboard08:54
*** udesale_ has joined #storyboard11:39
*** udesale has quit IRC11:42
*** udesale_ has quit IRC11:44
*** udesale has joined #storyboard12:57
*** jamesmcarthur has joined #storyboard13:20
corvusi'm still running into the problem that i can only be logged into one tab with the storyboard web client.  i don't have this problem with other applications (eg, gerrit)14:25
corvusdoes anyone have suggestions about how to debug further?14:26
fungiweird. i don't _think_ that's happened to me. seeing if i can reproduce16:00
fungiyeah, i just refreshed two sb tabs in my browser and both are still showing me authenticated with the same account16:01
fungii wonder if you could be running into a similar dueling-sessions issue to the one gerrit exhibits where authentication expires, you log back into that tab, switch to a different tab which was also using the now-expired session and it says you've been logged out, so you log in there and that invalidates the login from the other tab, you switch back to that tab to find it's no longer logged in, log back16:05
fungiinto it, lather, rinse, repeat16:05
*** udesale has quit IRC16:21
*** jamesmcarthur has quit IRC16:27
*** jamesmcarthur has joined #storyboard16:27
fungitosky: diablo_rojo: i've finished test imports of sahara team bugs to the projects in https://storyboard-dev.openstack.org/#!/project_group/22 using the mapping from http://paste.openstack.org/show/698760/ coupled with patch https://review.openstack.org/549222 and got no errors16:38
toskyyay!16:41
toskythanks16:41
fungi1739 stories across active, merged and invalid states16:41
diablo_rojoYay fungi !16:43
diablo_rojoI think we are good to go ahead and merge then if you wanna +2 +W16:44
fungiyup16:48
toskytime to increase the time for py35 unit tests? I rechecked for now17:31
persiaThe problem is really that the tests are not necessarily testing the right things: increasing the timeout might make them fail less, but they ought be updated to actually be useful.17:49
persiaMy memory of the last discussion of this issue was "yeah, let's fix the tests, and keep them failing as a reminder that we need to do that".17:49
toskyack17:50
toskyok, apart from waiting for the patch to be merged, what should we do now? The guide suggests a test migration with the production data, which is probably something that only infra can do17:50
toskyunless diablo_rojo's or fungi's tests were executed against production data17:51
fungiyes, they were17:51
diablo_rojofungi's tests were, mine were not.17:52
toskyoh, good to hear; now, do you need an official request from the PTL? And who can we nag from infra for the real migration?17:52
persiaI think anyone can test against LP production data (assuming they have permission to read LP bugs for the project in question)17:52
diablo_rojofungi, is usually our infra friend to run it :)17:52
fungithe next step really is to decide when you want to pull the trigger and have me (or some other infra-root person) import into production so you can start locking down your trackers in lp17:52
* fungi is sometimes a bit _too_ friendly about running things, but willing to volunteer17:53
diablo_rojowe appreciate it :)17:54
persiaDo we already have the project-config change to add use-storyboard: true submitted?17:54
toskyshould it be submitted before or after the migration?17:54
*** tellesnobrega has joined #storyboard17:54
toskyand here is the PTL17:54
persiaI believe it needs to land before the migration.17:54
persiaAnd I believe the process strongly encourages the PTL to have given a positive review to the project-config change17:55
* persia may be mistaken17:55
tellesnobregahey guys, tosky told me the tests went fine and we can fully migrate to storyboard, so here goes my thanks to all of you on the hard work, and related to when we can migrate I say ASAP, the fewer stuff we have on lp for rocky the better17:56
fungioh, right, submit that patch first. when it merges our manage-projects automation will precreate the relevant projects/groups in sb (which need to be there before i run the migration script)17:56
toskyok, going to submit the patch, and then tellesnobrega can +1 it17:56
diablo_rojoPerfect :)17:57
fungibasically just add a "use-storyboard: true" and "groups:\n  - sahara" to each of the 7 project entries to the gerrit/projects.yaml file in our openstack-infra/project-config repository17:57
fungiyou'll see plenty of other examples in there if you get lost17:58
toskyI wonder why some projects add groups:\n - sahara already and others did not17:59
toskys/projects/sahara repositories/17:59
toskysahara-tests has two groups: sahara and sahara-tests; should I keep it as it is?18:00
persiaUp to you, really.  Depends on whether you want one or two groups in Storyboard.  Since you already have groups without Storyboard, I suspect it may also affect something else.18:04
toskyI don't remember the historical reasons - can the second group be removed later if we want? (I guess yes, but just to be sure)18:05
persiaI suspect that removing a group requires a migration for all the databases of all the tools that consume groups.18:05
toskyok, let's leave it as it is for now18:06
persiaSomeone who knows more about the manage-projects automation may be able to give better guidance, but I suspect the longer any group exists, the more complicated any removal becomes.18:06
toskyin the review for project-config should I also add Story: and Task: pointing to the Sahara task in the "migrate all projects" story?18:06
toskyor not, given that it's not really going to close the task?18:07
toskynamely Story: 2000876, Task: 630718:08
diablo_rojoI dont think its necessary tosky18:24
diablo_rojoStill have to do the actual migration and then the disabling/updates to launchpad18:25
toskyack, thanks18:25
persiaThere is the argument that 2000876 should have at least two tasks per-project (one project-config, one for the migration, and maybe more for testing, outreach, etc.), but that just makes the list huge.18:25
toskyyeah, I was thinking about that18:26
toskytellesnobrega: https://review.openstack.org/55265118:26
diablo_rojoIts already huge, and I havent even added all the projects yet18:26
diablo_rojoLol18:26
persiaPersonally, I think that for cases where it is possible to have a comprehensive list of <30 things, having one task per commit is good.  When trying to track much larger things, sometimes the level of detail the automation requires is frustrating.18:26
toskyI should probably rtfm, but can stories be "nested"?18:27
persiaAt some point, we'll probably need proper support for epics (I think the current recommendation is: go write a wiki page that links to a bunch of stories), but I also think more folk need to use Storyboard for different things for us to have enough input to do a good job with epics.18:28
persiatosky: Stories cannot be nested.18:28
toskyor add epics, yes18:28
toskythis migration is really an epic effort18:28
toskyfor all possible meanings18:28
persiaYes :)18:28
persiaI think there are a few epics that have been done with Storyboard, and hope that by the time we get to the T PTG, there will be enough input that a good design can be made.  (Yes, S would be better, but that's fairly optimistic, as some projects have already said they don't expect to migrate in Rocky).18:31
tellesnobregatosky, done19:31
openstackgerritMerged openstack-infra/storyboard master: Migration script: optional parameters to filter by tags  https://review.openstack.org/54922219:35
diablo_rojoSo, since I have finished migrating all the ironic lp projects locally and its all happy it looks like Ironic can migrate :)20:07
diablo_rojofungi, SotK Zara ^^20:07
diablo_rojoTheJulia and I were discussing a timeline.20:08
diablo_rojoMaybe the week of the 26th? So like 2 weeks?20:08
TheJulia\o/ I'll bring it up with the team on monday, and we should be able to migrate the following week.20:08
diablo_rojoWOOHOO!!20:09
diablo_rojoI am so excited :) :)20:09
fungitosky: historical reasons are that "groups" is used to mean different things for launchpad (look for this repo's bugs in the indicated lp project) or storyboard (put this repo in a project group with the indicated name)20:11
fungifor lp we used it to be able to maintain a many-to-one mapping of gerrit repos to lp projects. for sb we use it to specify what project groups the repo is a member of20:13
toskyfungi: uhm, so maybe in this case I could drop the sahara-tests group, or is anything going to fail miserably if I do it?20:17
fungitosky: yeah, you don't need the other old groups, i would just give them all a groups entry for sahara and nothing else20:18
fungithe ability to have multiple groups is something we might want to take advantage of in the future, for example adding a tempest-plugins group which groups together all tempest plugins even though they'll probably also be members of some team-specific groups (nova, neutron, et cetera)20:19
fungibut as i said the reason for groups entries with lp was completely different from the reason to add groups entries for sb, so no need for them to be kept the same as what they were20:19
toskyfungi: so the sahara-tests group will continue to live in gerrit independently?20:20
toskyI guess so, oki, patch updated20:23
toskytellesnobrega: ^^20:27
tellesnobregatosky, thanks20:28
openstackgerritKendall Nelson proposed openstack-infra/storyboard-webclient master: Update Private Label  https://review.openstack.org/55057220:41
corvushere's another piece of information about my authn problem: if open 2 storyboard tabs *before* logging in, both of those tabs will be logged in.  any additional tabs after logging in aren't.21:48
corvusi've opened up the firefox storage inspector and verified that both of the logged-in tabs have the same token information in local storage.  nothing appears in local storage for the un-logged-in tab.21:49
corvusi'm very confused because local storage is supposed to be shared across all tabs from the same origin.21:49
*** jamesmcarthur has quit IRC21:58
corvusi'm also confused because local storage isn't supposed to be cleared at the end of a session, but when i restart firefox, i'm not logged in (i suspect it's a second manifestation of the same issue...)21:59
corvusalso -- is that actually the way storyboard behaves for everyone else?  if you log in to storyboard, then quit your browser, then restart your browser, are you still logged into storyboard?21:59
*** jamesmcarthur has joined #storyboard22:04
diablo_rojocorvus, not sure, would have to check at a point where I dont have like 10 tabs open and being doing things in them simultaneously :)22:28
diablo_rojoI will try to remember to check though when I tie up some of these loose ends22:29
*** tosky has quit IRC22:47
fungii tend to have firefox resume all my tabs when i close and reopen, so not sure if that has any impact on the session persistence23:11
*** jamesmcarthur has quit IRC23:22
*** jamesmcarthur has joined #storyboard23:50
openstackgerritJames E. Blair proposed openstack/boartty master: Remove archived lanes and worklist items  https://review.openstack.org/55271423:52
corvusfungi: localstorage should generally persist across browser restarts, so i expect that to happen.  i can't tell, of course.23:53
openstackgerritJames E. Blair proposed openstack/boartty master: Display story access level  https://review.openstack.org/55271523:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!