Monday, 2020-07-13

*** hongbin has quit IRC00:20
*** hongbin has joined #openstack-qa00:22
*** yamamoto has joined #openstack-qa00:36
*** yamamoto has quit IRC00:42
*** paras333 has quit IRC00:54
*** markvoelker has joined #openstack-qa01:15
*** markvoelker has quit IRC01:19
*** paras333 has joined #openstack-qa01:20
*** paras333 has quit IRC01:24
*** yamamoto has joined #openstack-qa01:38
*** paras333 has joined #openstack-qa01:52
*** paras333 has quit IRC01:56
*** markvoelker has joined #openstack-qa02:21
*** markvoelker has quit IRC02:25
*** paras333 has joined #openstack-qa02:28
*** paras333 has quit IRC02:32
*** ricolin has joined #openstack-qa02:33
*** markvoelker has joined #openstack-qa02:34
*** markvoelker has quit IRC02:39
*** paras333 has joined #openstack-qa02:52
*** psahoo has joined #openstack-qa02:55
*** paras333 has quit IRC02:57
*** paras333 has joined #openstack-qa03:25
*** hongbin has quit IRC03:26
*** paras333 has quit IRC03:30
*** markvoelker has joined #openstack-qa03:31
*** markvoelker has quit IRC03:36
*** paras333 has joined #openstack-qa03:39
*** paras333 has quit IRC03:43
*** ramishra has joined #openstack-qa03:44
*** paras333 has joined #openstack-qa03:45
*** paras333 has quit IRC03:50
*** gcheresh has joined #openstack-qa04:12
*** gcheresh has quit IRC04:32
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-qa04:33
*** psahoo has quit IRC04:35
*** soniya29 has joined #openstack-qa04:38
*** psahoo has joined #openstack-qa04:40
*** bhagyashris|away is now known as bhagyashris04:41
*** gcheresh has joined #openstack-qa04:53
*** ricolin_ has joined #openstack-qa04:56
*** ricolin has quit IRC04:57
*** abdysn has joined #openstack-qa05:02
*** Abdallahyas has joined #openstack-qa05:07
*** Abdallahyas has quit IRC05:09
*** psahoo has quit IRC05:10
*** Abdallahyas has joined #openstack-qa05:10
*** abdysn has quit IRC05:11
*** paras333 has joined #openstack-qa05:15
*** paras333 has quit IRC05:19
*** psahoo has joined #openstack-qa05:22
*** psahoo has quit IRC05:28
*** markvoelker has joined #openstack-qa05:32
*** markvoelker has quit IRC05:36
*** Luzi has joined #openstack-qa05:37
*** udesale has joined #openstack-qa05:38
*** psahoo has joined #openstack-qa05:44
*** rcernin has quit IRC06:02
*** rcernin has joined #openstack-qa06:11
*** tosky has joined #openstack-qa06:39
*** psahoo has quit IRC06:49
*** vishalmanchanda has joined #openstack-qa06:55
*** psahoo has joined #openstack-qa07:04
*** nightmare_unreal has joined #openstack-qa07:09
*** ccamposr__ has joined #openstack-qa07:09
*** ccamposr has quit IRC07:11
*** ralonsoh has joined #openstack-qa07:19
*** abdysn has joined #openstack-qa07:34
*** Abdallahyas has quit IRC07:37
kopecmartinafazekas: hi, when you have a moment, check this please: https://review.opendev.org/#/c/737453/07:47
*** lucasagomes has joined #openstack-qa07:56
*** rcernin has quit IRC08:06
*** ramishra has quit IRC08:28
*** ramishra has joined #openstack-qa08:29
openstackgerritLucas Alvares Gomes proposed openstack/devstack master: [OVN] Move OVN module from Neutron to DevStack  https://review.opendev.org/73462108:32
openstackgerritLucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW: Change Neutron's default ML2 driver to OVN  https://review.opendev.org/73509708:33
*** dtantsur|afk is now known as dtantsur08:34
*** abdysn has quit IRC08:55
*** abdysn has joined #openstack-qa08:55
*** Abdallahyas has joined #openstack-qa08:57
*** abdysn has quit IRC09:00
*** yamamoto has quit IRC09:03
*** yamamoto has joined #openstack-qa09:11
*** Abdallahyas has quit IRC09:25
*** abdysn has joined #openstack-qa09:26
*** yamamoto has quit IRC09:33
*** yamamoto has joined #openstack-qa09:38
*** rcernin has joined #openstack-qa09:53
*** ricolin_ has quit IRC10:01
*** rcernin has quit IRC10:03
*** rcernin has joined #openstack-qa10:08
*** rcernin has quit IRC10:40
openstackgerritLucas Alvares Gomes proposed openstack/devstack master: [OVN] Move OVN module from Neutron to DevStack  https://review.opendev.org/73462110:52
*** yamamoto has quit IRC10:54
*** finucannot is now known as stephenfin10:57
*** markvoelker has joined #openstack-qa11:06
*** markvoelker has quit IRC11:16
*** ricolin_ has joined #openstack-qa11:23
*** yamamoto has joined #openstack-qa11:26
*** bhagyashris is now known as bhagyashris|afk11:33
*** tkajinam has quit IRC11:37
*** yamamoto has quit IRC11:42
*** psahoo has quit IRC11:43
*** yamamoto has joined #openstack-qa11:43
*** yamamoto has quit IRC11:44
*** yamamoto has joined #openstack-qa11:49
*** yamamoto has quit IRC11:52
*** raildo has joined #openstack-qa11:56
*** psahoo has joined #openstack-qa11:57
*** adriant has quit IRC11:58
*** dpaterson has quit IRC11:58
*** masayukig has quit IRC11:58
*** dpaterson has joined #openstack-qa11:58
*** masayukig has joined #openstack-qa11:58
*** adriant has joined #openstack-qa11:59
*** markvoelker has joined #openstack-qa12:01
*** rfolco has joined #openstack-qa12:04
*** markvoelker has quit IRC12:04
*** yamamoto has joined #openstack-qa12:04
*** udesale_ has joined #openstack-qa12:09
*** dave-mccowan has joined #openstack-qa12:11
*** rh-jelabarre has joined #openstack-qa12:12
*** udesale has quit IRC12:12
*** rh-jelabarre has quit IRC12:12
*** rh-jelabarre has joined #openstack-qa12:12
*** dave-mccowan has quit IRC12:15
*** ccamposr has joined #openstack-qa12:24
*** ccamposr__ has quit IRC12:26
*** bhagyashris|afk is now known as bhagyashris12:35
*** ralonsoh has quit IRC12:43
*** ralonsoh has joined #openstack-qa12:44
*** yamamoto has quit IRC12:51
*** lbragstad_ is now known as lbragstad12:51
*** paras333 has joined #openstack-qa12:56
*** zombieJulia is now known as TheJulia13:06
*** ccamposr__ has joined #openstack-qa13:07
*** jparker has quit IRC13:09
*** ccamposr has quit IRC13:09
*** jparker has joined #openstack-qa13:15
*** ramishra has quit IRC13:22
*** soniya29 has quit IRC13:22
*** ramishra has joined #openstack-qa13:26
*** paras333 has quit IRC13:27
*** udesale_ has quit IRC13:27
*** rfolco has quit IRC13:27
*** sean-k-mooney has quit IRC13:27
*** irclogbot_2 has quit IRC13:27
*** paras333 has joined #openstack-qa13:28
*** udesale_ has joined #openstack-qa13:28
*** rfolco has joined #openstack-qa13:28
*** sean-k-mooney has joined #openstack-qa13:28
*** irclogbot_2 has joined #openstack-qa13:28
*** yamamoto has joined #openstack-qa13:28
*** abdysn has quit IRC13:35
*** yamamoto has quit IRC13:36
*** soniya29 has joined #openstack-qa13:44
*** bhagyashris is now known as bhagyashris|afk13:56
*** artom has joined #openstack-qa13:56
*** Luzi has quit IRC14:00
*** dviroel has joined #openstack-qa14:05
openstackgerritLukas Piwowarski proposed openstack/tempest master: Fix import line location  https://review.opendev.org/70135214:16
*** vishakha has joined #openstack-qa14:22
*** amodi has quit IRC14:22
openstackgerritLucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW: Change Neutron's default ML2 driver to OVN  https://review.opendev.org/73509714:41
*** udesale_ has quit IRC15:07
*** soniya29 has quit IRC15:17
*** paras333 has quit IRC15:23
openstackgerritGayathri Devi proposed openstack/tempest master: [DNM] Test commit for volume versions  https://review.opendev.org/74071415:28
*** yamamoto has joined #openstack-qa15:36
dansmithgmann: friendly reminder that we're looking for your feedback on this: https://review.opendev.org/#/c/73596615:39
dansmithgmann: also, I put up an attempt to refactor the glance module to allow standalone-ness independent of WSGI_MODE, if you can look at that too.. it's stacked above that ^15:39
gmanndansmith: sorry about that, checking15:40
dansmithgmann: thanks15:41
*** amodi has joined #openstack-qa15:50
*** yamamoto has quit IRC15:53
*** lucasagomes has quit IRC16:02
*** paras333 has joined #openstack-qa16:03
*** ccamposr has joined #openstack-qa16:04
*** ccamposr__ has quit IRC16:06
*** artom has quit IRC16:07
openstackgerritLukas Piwowarski proposed openstack/tempest master: Create tests for check-uuid CLI  https://review.opendev.org/74072316:10
openstackgerritLukas Piwowarski proposed openstack/tempest master: WIP: Create tests for check-uuid CLI  https://review.opendev.org/74072316:11
*** artom has joined #openstack-qa16:18
*** sean-k-mooney has quit IRC17:01
*** dtantsur is now known as dtantsur|afk17:15
*** ralonsoh has quit IRC17:23
*** ricolin_ has quit IRC17:40
*** yamamoto has joined #openstack-qa17:53
*** yamamoto has quit IRC18:08
gmanndansmith: frickler abhishekk commented on glance import patch, I am ok with that waiting for nova job results(latest) - https://review.opendev.org/#/c/735966/18:12
gmanndansmith: one comment on echo statement if you want to fix in that or followup.18:12
*** ccamposr__ has joined #openstack-qa18:13
dansmithgmann: okay the changes in this patch are only tested in the setup phase fwiw, which has worked many times, but I think the job is going to pass this time too18:13
dansmithit has been failing for nova/glance/policy reasons as I've worked through other issues, but the devstack part has been solid for a while18:13
dansmithgmann: yep, I will fix that echo once the job finishes18:13
gmanndansmith: btw do we need create-image testing also in https://review.opendev.org/#/c/734184/29/playbooks/ceph/glance-copy-policy.yaml18:13
gmannyeah i saw the extra check_str error in log18:14
dansmithgmann: my nova glance multistore job will keep that policy tweak in it, as the default policy is to disallow images if they're not owned by the user18:14
dansmithso I think my job will provide that coverage18:15
dansmithand I added testing for this in glance itself too18:15
*** ccamposr has quit IRC18:15
gmannok.18:15
dansmithgmann: so when I update the devstack patch to remove that echo,18:16
dansmithgmann: I can make the nova job dependent on the second standalone refactor patch, and make the job set that instead of WSGI_MODE18:16
dansmithwhich will get some coverage on that and have the rest of the stack running in uwsgi mode18:17
dansmithI just didn't want to do that until you had looked over that patch and given it a tentative blessing18:17
dansmithgmann: job just finished, failed for other reasons: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_8a7/734184/29/check/nova-ceph-multistore/8a722be/testr_results.html18:17
*** psahoo has quit IRC18:18
dansmithI might have to fix that test if it's uploading a non-raw image or something18:18
gmanndansmith: yeah i was writing the same either modifying the job with new standalone flag or merge the current way and modify job with standalone with 740322 set. anything is ok for me18:19
dansmithokay I think using the new flag makes the most sense.. my dependency tree for this is insanely deep right now, but hopefully you guys will help get those two merged sooner than later and it won't be much larger for too much longer :)18:20
gmannok.18:21
gmanndansmith: i need to go for my driving test now, will re-review patch and results in my evening.18:22
dansmithgmann: okay I'm confused about the echo comment actually18:22
dansmithbut don't let me hold you18:23
gmanndansmith: i mean if GLANCE_USE_IMPORT_WORKFLOW is false we still echo that 'unable to import...'18:23
dansmithgmann: I don't think it will18:23
dansmithgmann: only if WSGI_MODE==uwsgi *and* import is requested18:24
dansmithif [[ "$GLANCE_USE_IMPORT_WORKFLOW" == "True" && "$WSGI_MODE" == "uwsgi" ]]; then18:24
dansmithif import is false, then it won't echo18:24
gmannah you are right, i overead that.18:24
gmannlgtm, canimport local you can remove if re-writing in simple nested if condition  of GLANCE_USE_IMPORT_WORKFLOW  and WSGI_MODE18:25
dansmithyep, I could, I just don't like nested conditionals, but if you'd prefer I will change it18:25
gmannthat is ready to read :).18:26
*** nightmare_unreal has quit IRC18:26
dansmiths/ready/easier/ ?18:27
dansmithI totally disagree, but I will change it just for you :)18:27
gmann:) its both way but not something you have to change now but if you respin.18:28
dansmithI will respin the top one so I'll just change it18:28
gmannok, thanks and sorry for late review, i missed that because of focal-migration things last weke18:30
gmannweek18:30
dansmithno probs, thanks for looking18:30
dansmithgo to your driving test :)18:30
gmann:)18:30
*** andrebeltrami has joined #openstack-qa18:32
openstackgerritDan Smith proposed openstack/devstack master: Use glance import workflow for creating image  https://review.opendev.org/73596618:37
openstackgerritDan Smith proposed openstack/devstack master: Make Glance standalone-ness a separate flag  https://review.opendev.org/74032218:37
*** paras333 has quit IRC18:47
*** paras333 has joined #openstack-qa19:05
*** vishakha has quit IRC19:11
openstackgerritDan Smith proposed openstack/devstack master: Make Glance standalone-ness a separate flag  https://review.opendev.org/74032220:07
*** yamamoto has joined #openstack-qa20:10
*** yamamoto has quit IRC20:23
openstackgerritDan Smith proposed openstack/devstack master: Make Glance standalone-ness a separate flag  https://review.opendev.org/74032220:41
gmanndansmith: this condition should be '!='   https://review.opendev.org/#/c/735966/8/functions@10620:57
dansmithdoh, yep20:58
dansmiththe next patch  flips it to an == so I got mixed up20:59
openstackgerritDan Smith proposed openstack/devstack master: Use glance import workflow for creating image  https://review.opendev.org/73596620:59
openstackgerritDan Smith proposed openstack/devstack master: Make Glance standalone-ness a separate flag  https://review.opendev.org/74032220:59
gmannoh i was about to write to wait for update until nova  job finish as job testing with standalone flag which is with correct condition  - https://review.opendev.org/#/c/734184/21:03
gmanndansmith: also do we need to run on all tests in that job? left comment in nova patch.21:04
dansmithgmann: sure, I was just being careful by getting a bunch of them, but then I hit that cinder test which clearly does something else that I don't want21:07
dansmithgmann: you're suggesting I just parent to that existing ceph job to get its regex?21:07
*** gcheresh has quit IRC21:08
*** vishalmanchanda has quit IRC21:08
*** raildo has quit IRC21:10
*** raildo has joined #openstack-qa21:11
gmanndansmith:  if your end goal is to replace devstack-plugin-ceph-tempest-py3 with new job then it is fine to run all but you have to skip encrypted vol test so i am thinking whether we need both job running on nova gate?21:12
gmannif both job then we can run a very subset of tests in new job.21:13
dansmithgmann: well, it shouldn't really be any different between the two.. that encrypted vol test is creating an image that isn't allowed on ceph, but the default nova config will just ignore that21:14
dansmithso if that test is included in the current ceph job, then we should make nova detect/reject that case and fix the test21:14
dansmithor just say "meh" and skip that test21:15
dansmithmaking my job inherit from that one and use the smaller set of tests is probably fine21:15
dansmithin the end21:15
gmannyeah none of ceph job set never_download_image_if_on_rbd=True21:16
dansmithright21:16
*** raildo has quit IRC21:17
*** paras333 has quit IRC21:20
*** ccamposr__ has quit IRC21:30
*** paras333 has joined #openstack-qa21:40
*** paras333 has quit IRC21:43
openstackgerritDan Smith proposed openstack/devstack master: Make Glance standalone-ness a separate flag  https://review.opendev.org/74032221:58
*** yamamoto has joined #openstack-qa22:28
*** yamamoto has quit IRC22:38
*** tosky has quit IRC22:42
openstackgerritColleen Murphy proposed openstack/tempest master: Add system scope for admin auth  https://review.opendev.org/68607322:48
openstackgerritColleen Murphy proposed openstack/tempest master: Add default RBAC personas to dynamic credentials  https://review.opendev.org/68630622:48
openstackgerritColleen Murphy proposed openstack/tempest master: Add default RBAC personas to pre-provisioned creds  https://review.opendev.org/69839722:48
*** tkajinam has joined #openstack-qa22:54
*** rcernin has joined #openstack-qa23:05
*** rcernin has quit IRC23:05
*** rcernin has joined #openstack-qa23:08
openstackgerritColleen Murphy proposed openstack/tempest master: Add default RBAC personas to dynamic credentials  https://review.opendev.org/68630623:28
openstackgerritColleen Murphy proposed openstack/tempest master: Add default RBAC personas to pre-provisioned creds  https://review.opendev.org/69839723:28
*** adriant has quit IRC23:32
*** adriant has joined #openstack-qa23:33
openstackgerritGoutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: [manila][ci] Replace legacy jobs  https://review.opendev.org/74083023:49
openstackgerritGoutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: [manila][ci] Replace legacy jobs  https://review.opendev.org/74083023:53

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!