*** hongbin has quit IRC | 00:20 | |
*** hongbin has joined #openstack-qa | 00:22 | |
*** yamamoto has joined #openstack-qa | 00:36 | |
*** yamamoto has quit IRC | 00:42 | |
*** paras333 has quit IRC | 00:54 | |
*** markvoelker has joined #openstack-qa | 01:15 | |
*** markvoelker has quit IRC | 01:19 | |
*** paras333 has joined #openstack-qa | 01:20 | |
*** paras333 has quit IRC | 01:24 | |
*** yamamoto has joined #openstack-qa | 01:38 | |
*** paras333 has joined #openstack-qa | 01:52 | |
*** paras333 has quit IRC | 01:56 | |
*** markvoelker has joined #openstack-qa | 02:21 | |
*** markvoelker has quit IRC | 02:25 | |
*** paras333 has joined #openstack-qa | 02:28 | |
*** paras333 has quit IRC | 02:32 | |
*** ricolin has joined #openstack-qa | 02:33 | |
*** markvoelker has joined #openstack-qa | 02:34 | |
*** markvoelker has quit IRC | 02:39 | |
*** paras333 has joined #openstack-qa | 02:52 | |
*** psahoo has joined #openstack-qa | 02:55 | |
*** paras333 has quit IRC | 02:57 | |
*** paras333 has joined #openstack-qa | 03:25 | |
*** hongbin has quit IRC | 03:26 | |
*** paras333 has quit IRC | 03:30 | |
*** markvoelker has joined #openstack-qa | 03:31 | |
*** markvoelker has quit IRC | 03:36 | |
*** paras333 has joined #openstack-qa | 03:39 | |
*** paras333 has quit IRC | 03:43 | |
*** ramishra has joined #openstack-qa | 03:44 | |
*** paras333 has joined #openstack-qa | 03:45 | |
*** paras333 has quit IRC | 03:50 | |
*** gcheresh has joined #openstack-qa | 04:12 | |
*** gcheresh has quit IRC | 04:32 | |
*** evrardjp has quit IRC | 04:33 | |
*** evrardjp has joined #openstack-qa | 04:33 | |
*** psahoo has quit IRC | 04:35 | |
*** soniya29 has joined #openstack-qa | 04:38 | |
*** psahoo has joined #openstack-qa | 04:40 | |
*** bhagyashris|away is now known as bhagyashris | 04:41 | |
*** gcheresh has joined #openstack-qa | 04:53 | |
*** ricolin_ has joined #openstack-qa | 04:56 | |
*** ricolin has quit IRC | 04:57 | |
*** abdysn has joined #openstack-qa | 05:02 | |
*** Abdallahyas has joined #openstack-qa | 05:07 | |
*** Abdallahyas has quit IRC | 05:09 | |
*** psahoo has quit IRC | 05:10 | |
*** Abdallahyas has joined #openstack-qa | 05:10 | |
*** abdysn has quit IRC | 05:11 | |
*** paras333 has joined #openstack-qa | 05:15 | |
*** paras333 has quit IRC | 05:19 | |
*** psahoo has joined #openstack-qa | 05:22 | |
*** psahoo has quit IRC | 05:28 | |
*** markvoelker has joined #openstack-qa | 05:32 | |
*** markvoelker has quit IRC | 05:36 | |
*** Luzi has joined #openstack-qa | 05:37 | |
*** udesale has joined #openstack-qa | 05:38 | |
*** psahoo has joined #openstack-qa | 05:44 | |
*** rcernin has quit IRC | 06:02 | |
*** rcernin has joined #openstack-qa | 06:11 | |
*** tosky has joined #openstack-qa | 06:39 | |
*** psahoo has quit IRC | 06:49 | |
*** vishalmanchanda has joined #openstack-qa | 06:55 | |
*** psahoo has joined #openstack-qa | 07:04 | |
*** nightmare_unreal has joined #openstack-qa | 07:09 | |
*** ccamposr__ has joined #openstack-qa | 07:09 | |
*** ccamposr has quit IRC | 07:11 | |
*** ralonsoh has joined #openstack-qa | 07:19 | |
*** abdysn has joined #openstack-qa | 07:34 | |
*** Abdallahyas has quit IRC | 07:37 | |
kopecmartin | afazekas: hi, when you have a moment, check this please: https://review.opendev.org/#/c/737453/ | 07:47 |
---|---|---|
*** lucasagomes has joined #openstack-qa | 07:56 | |
*** rcernin has quit IRC | 08:06 | |
*** ramishra has quit IRC | 08:28 | |
*** ramishra has joined #openstack-qa | 08:29 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/devstack master: [OVN] Move OVN module from Neutron to DevStack https://review.opendev.org/734621 | 08:32 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW: Change Neutron's default ML2 driver to OVN https://review.opendev.org/735097 | 08:33 |
*** dtantsur|afk is now known as dtantsur | 08:34 | |
*** abdysn has quit IRC | 08:55 | |
*** abdysn has joined #openstack-qa | 08:55 | |
*** Abdallahyas has joined #openstack-qa | 08:57 | |
*** abdysn has quit IRC | 09:00 | |
*** yamamoto has quit IRC | 09:03 | |
*** yamamoto has joined #openstack-qa | 09:11 | |
*** Abdallahyas has quit IRC | 09:25 | |
*** abdysn has joined #openstack-qa | 09:26 | |
*** yamamoto has quit IRC | 09:33 | |
*** yamamoto has joined #openstack-qa | 09:38 | |
*** rcernin has joined #openstack-qa | 09:53 | |
*** ricolin_ has quit IRC | 10:01 | |
*** rcernin has quit IRC | 10:03 | |
*** rcernin has joined #openstack-qa | 10:08 | |
*** rcernin has quit IRC | 10:40 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/devstack master: [OVN] Move OVN module from Neutron to DevStack https://review.opendev.org/734621 | 10:52 |
*** yamamoto has quit IRC | 10:54 | |
*** finucannot is now known as stephenfin | 10:57 | |
*** markvoelker has joined #openstack-qa | 11:06 | |
*** markvoelker has quit IRC | 11:16 | |
*** ricolin_ has joined #openstack-qa | 11:23 | |
*** yamamoto has joined #openstack-qa | 11:26 | |
*** bhagyashris is now known as bhagyashris|afk | 11:33 | |
*** tkajinam has quit IRC | 11:37 | |
*** yamamoto has quit IRC | 11:42 | |
*** psahoo has quit IRC | 11:43 | |
*** yamamoto has joined #openstack-qa | 11:43 | |
*** yamamoto has quit IRC | 11:44 | |
*** yamamoto has joined #openstack-qa | 11:49 | |
*** yamamoto has quit IRC | 11:52 | |
*** raildo has joined #openstack-qa | 11:56 | |
*** psahoo has joined #openstack-qa | 11:57 | |
*** adriant has quit IRC | 11:58 | |
*** dpaterson has quit IRC | 11:58 | |
*** masayukig has quit IRC | 11:58 | |
*** dpaterson has joined #openstack-qa | 11:58 | |
*** masayukig has joined #openstack-qa | 11:58 | |
*** adriant has joined #openstack-qa | 11:59 | |
*** markvoelker has joined #openstack-qa | 12:01 | |
*** rfolco has joined #openstack-qa | 12:04 | |
*** markvoelker has quit IRC | 12:04 | |
*** yamamoto has joined #openstack-qa | 12:04 | |
*** udesale_ has joined #openstack-qa | 12:09 | |
*** dave-mccowan has joined #openstack-qa | 12:11 | |
*** rh-jelabarre has joined #openstack-qa | 12:12 | |
*** udesale has quit IRC | 12:12 | |
*** rh-jelabarre has quit IRC | 12:12 | |
*** rh-jelabarre has joined #openstack-qa | 12:12 | |
*** dave-mccowan has quit IRC | 12:15 | |
*** ccamposr has joined #openstack-qa | 12:24 | |
*** ccamposr__ has quit IRC | 12:26 | |
*** bhagyashris|afk is now known as bhagyashris | 12:35 | |
*** ralonsoh has quit IRC | 12:43 | |
*** ralonsoh has joined #openstack-qa | 12:44 | |
*** yamamoto has quit IRC | 12:51 | |
*** lbragstad_ is now known as lbragstad | 12:51 | |
*** paras333 has joined #openstack-qa | 12:56 | |
*** zombieJulia is now known as TheJulia | 13:06 | |
*** ccamposr__ has joined #openstack-qa | 13:07 | |
*** jparker has quit IRC | 13:09 | |
*** ccamposr has quit IRC | 13:09 | |
*** jparker has joined #openstack-qa | 13:15 | |
*** ramishra has quit IRC | 13:22 | |
*** soniya29 has quit IRC | 13:22 | |
*** ramishra has joined #openstack-qa | 13:26 | |
*** paras333 has quit IRC | 13:27 | |
*** udesale_ has quit IRC | 13:27 | |
*** rfolco has quit IRC | 13:27 | |
*** sean-k-mooney has quit IRC | 13:27 | |
*** irclogbot_2 has quit IRC | 13:27 | |
*** paras333 has joined #openstack-qa | 13:28 | |
*** udesale_ has joined #openstack-qa | 13:28 | |
*** rfolco has joined #openstack-qa | 13:28 | |
*** sean-k-mooney has joined #openstack-qa | 13:28 | |
*** irclogbot_2 has joined #openstack-qa | 13:28 | |
*** yamamoto has joined #openstack-qa | 13:28 | |
*** abdysn has quit IRC | 13:35 | |
*** yamamoto has quit IRC | 13:36 | |
*** soniya29 has joined #openstack-qa | 13:44 | |
*** bhagyashris is now known as bhagyashris|afk | 13:56 | |
*** artom has joined #openstack-qa | 13:56 | |
*** Luzi has quit IRC | 14:00 | |
*** dviroel has joined #openstack-qa | 14:05 | |
openstackgerrit | Lukas Piwowarski proposed openstack/tempest master: Fix import line location https://review.opendev.org/701352 | 14:16 |
*** vishakha has joined #openstack-qa | 14:22 | |
*** amodi has quit IRC | 14:22 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/devstack master: DO NOT REVIEW: Change Neutron's default ML2 driver to OVN https://review.opendev.org/735097 | 14:41 |
*** udesale_ has quit IRC | 15:07 | |
*** soniya29 has quit IRC | 15:17 | |
*** paras333 has quit IRC | 15:23 | |
openstackgerrit | Gayathri Devi proposed openstack/tempest master: [DNM] Test commit for volume versions https://review.opendev.org/740714 | 15:28 |
*** yamamoto has joined #openstack-qa | 15:36 | |
dansmith | gmann: friendly reminder that we're looking for your feedback on this: https://review.opendev.org/#/c/735966 | 15:39 |
dansmith | gmann: 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 |
gmann | dansmith: sorry about that, checking | 15:40 |
dansmith | gmann: thanks | 15:41 |
*** amodi has joined #openstack-qa | 15:50 | |
*** yamamoto has quit IRC | 15:53 | |
*** lucasagomes has quit IRC | 16:02 | |
*** paras333 has joined #openstack-qa | 16:03 | |
*** ccamposr has joined #openstack-qa | 16:04 | |
*** ccamposr__ has quit IRC | 16:06 | |
*** artom has quit IRC | 16:07 | |
openstackgerrit | Lukas Piwowarski proposed openstack/tempest master: Create tests for check-uuid CLI https://review.opendev.org/740723 | 16:10 |
openstackgerrit | Lukas Piwowarski proposed openstack/tempest master: WIP: Create tests for check-uuid CLI https://review.opendev.org/740723 | 16:11 |
*** artom has joined #openstack-qa | 16:18 | |
*** sean-k-mooney has quit IRC | 17:01 | |
*** dtantsur is now known as dtantsur|afk | 17:15 | |
*** ralonsoh has quit IRC | 17:23 | |
*** ricolin_ has quit IRC | 17:40 | |
*** yamamoto has joined #openstack-qa | 17:53 | |
*** yamamoto has quit IRC | 18:08 | |
gmann | dansmith: 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 |
gmann | dansmith: one comment on echo statement if you want to fix in that or followup. | 18:12 |
*** ccamposr__ has joined #openstack-qa | 18:13 | |
dansmith | gmann: 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 too | 18:13 |
dansmith | it has been failing for nova/glance/policy reasons as I've worked through other issues, but the devstack part has been solid for a while | 18:13 |
dansmith | gmann: yep, I will fix that echo once the job finishes | 18:13 |
gmann | dansmith: btw do we need create-image testing also in https://review.opendev.org/#/c/734184/29/playbooks/ceph/glance-copy-policy.yaml | 18:13 |
gmann | yeah i saw the extra check_str error in log | 18:14 |
dansmith | gmann: 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 user | 18:14 |
dansmith | so I think my job will provide that coverage | 18:15 |
dansmith | and I added testing for this in glance itself too | 18:15 |
*** ccamposr has quit IRC | 18:15 | |
gmann | ok. | 18:15 |
dansmith | gmann: so when I update the devstack patch to remove that echo, | 18:16 |
dansmith | gmann: I can make the nova job dependent on the second standalone refactor patch, and make the job set that instead of WSGI_MODE | 18:16 |
dansmith | which will get some coverage on that and have the rest of the stack running in uwsgi mode | 18:17 |
dansmith | I just didn't want to do that until you had looked over that patch and given it a tentative blessing | 18:17 |
dansmith | gmann: 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.html | 18:17 |
*** psahoo has quit IRC | 18:18 | |
dansmith | I might have to fix that test if it's uploading a non-raw image or something | 18:18 |
gmann | dansmith: 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 me | 18:19 |
dansmith | okay 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 |
gmann | ok. | 18:21 |
gmann | dansmith: i need to go for my driving test now, will re-review patch and results in my evening. | 18:22 |
dansmith | gmann: okay I'm confused about the echo comment actually | 18:22 |
dansmith | but don't let me hold you | 18:23 |
gmann | dansmith: i mean if GLANCE_USE_IMPORT_WORKFLOW is false we still echo that 'unable to import...' | 18:23 |
dansmith | gmann: I don't think it will | 18:23 |
dansmith | gmann: only if WSGI_MODE==uwsgi *and* import is requested | 18:24 |
dansmith | if [[ "$GLANCE_USE_IMPORT_WORKFLOW" == "True" && "$WSGI_MODE" == "uwsgi" ]]; then | 18:24 |
dansmith | if import is false, then it won't echo | 18:24 |
gmann | ah you are right, i overead that. | 18:24 |
gmann | lgtm, canimport local you can remove if re-writing in simple nested if condition of GLANCE_USE_IMPORT_WORKFLOW and WSGI_MODE | 18:25 |
dansmith | yep, I could, I just don't like nested conditionals, but if you'd prefer I will change it | 18:25 |
gmann | that is ready to read :). | 18:26 |
*** nightmare_unreal has quit IRC | 18:26 | |
dansmith | s/ready/easier/ ? | 18:27 |
dansmith | I 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 |
dansmith | I will respin the top one so I'll just change it | 18:28 |
gmann | ok, thanks and sorry for late review, i missed that because of focal-migration things last weke | 18:30 |
gmann | week | 18:30 |
dansmith | no probs, thanks for looking | 18:30 |
dansmith | go to your driving test :) | 18:30 |
gmann | :) | 18:30 |
*** andrebeltrami has joined #openstack-qa | 18:32 | |
openstackgerrit | Dan Smith proposed openstack/devstack master: Use glance import workflow for creating image https://review.opendev.org/735966 | 18:37 |
openstackgerrit | Dan Smith proposed openstack/devstack master: Make Glance standalone-ness a separate flag https://review.opendev.org/740322 | 18:37 |
*** paras333 has quit IRC | 18:47 | |
*** paras333 has joined #openstack-qa | 19:05 | |
*** vishakha has quit IRC | 19:11 | |
openstackgerrit | Dan Smith proposed openstack/devstack master: Make Glance standalone-ness a separate flag https://review.opendev.org/740322 | 20:07 |
*** yamamoto has joined #openstack-qa | 20:10 | |
*** yamamoto has quit IRC | 20:23 | |
openstackgerrit | Dan Smith proposed openstack/devstack master: Make Glance standalone-ness a separate flag https://review.opendev.org/740322 | 20:41 |
gmann | dansmith: this condition should be '!=' https://review.opendev.org/#/c/735966/8/functions@106 | 20:57 |
dansmith | doh, yep | 20:58 |
dansmith | the next patch flips it to an == so I got mixed up | 20:59 |
openstackgerrit | Dan Smith proposed openstack/devstack master: Use glance import workflow for creating image https://review.opendev.org/735966 | 20:59 |
openstackgerrit | Dan Smith proposed openstack/devstack master: Make Glance standalone-ness a separate flag https://review.opendev.org/740322 | 20:59 |
gmann | oh 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 |
gmann | dansmith: also do we need to run on all tests in that job? left comment in nova patch. | 21:04 |
dansmith | gmann: 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 want | 21:07 |
dansmith | gmann: you're suggesting I just parent to that existing ceph job to get its regex? | 21:07 |
*** gcheresh has quit IRC | 21:08 | |
*** vishalmanchanda has quit IRC | 21:08 | |
*** raildo has quit IRC | 21:10 | |
*** raildo has joined #openstack-qa | 21:11 | |
gmann | dansmith: 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 |
gmann | if both job then we can run a very subset of tests in new job. | 21:13 |
dansmith | gmann: 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 that | 21:14 |
dansmith | so if that test is included in the current ceph job, then we should make nova detect/reject that case and fix the test | 21:14 |
dansmith | or just say "meh" and skip that test | 21:15 |
dansmith | making my job inherit from that one and use the smaller set of tests is probably fine | 21:15 |
dansmith | in the end | 21:15 |
gmann | yeah none of ceph job set never_download_image_if_on_rbd=True | 21:16 |
dansmith | right | 21:16 |
*** raildo has quit IRC | 21:17 | |
*** paras333 has quit IRC | 21:20 | |
*** ccamposr__ has quit IRC | 21:30 | |
*** paras333 has joined #openstack-qa | 21:40 | |
*** paras333 has quit IRC | 21:43 | |
openstackgerrit | Dan Smith proposed openstack/devstack master: Make Glance standalone-ness a separate flag https://review.opendev.org/740322 | 21:58 |
*** yamamoto has joined #openstack-qa | 22:28 | |
*** yamamoto has quit IRC | 22:38 | |
*** tosky has quit IRC | 22:42 | |
openstackgerrit | Colleen Murphy proposed openstack/tempest master: Add system scope for admin auth https://review.opendev.org/686073 | 22:48 |
openstackgerrit | Colleen Murphy proposed openstack/tempest master: Add default RBAC personas to dynamic credentials https://review.opendev.org/686306 | 22:48 |
openstackgerrit | Colleen Murphy proposed openstack/tempest master: Add default RBAC personas to pre-provisioned creds https://review.opendev.org/698397 | 22:48 |
*** tkajinam has joined #openstack-qa | 22:54 | |
*** rcernin has joined #openstack-qa | 23:05 | |
*** rcernin has quit IRC | 23:05 | |
*** rcernin has joined #openstack-qa | 23:08 | |
openstackgerrit | Colleen Murphy proposed openstack/tempest master: Add default RBAC personas to dynamic credentials https://review.opendev.org/686306 | 23:28 |
openstackgerrit | Colleen Murphy proposed openstack/tempest master: Add default RBAC personas to pre-provisioned creds https://review.opendev.org/698397 | 23:28 |
*** adriant has quit IRC | 23:32 | |
*** adriant has joined #openstack-qa | 23:33 | |
openstackgerrit | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: [manila][ci] Replace legacy jobs https://review.opendev.org/740830 | 23:49 |
openstackgerrit | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: [manila][ci] Replace legacy jobs https://review.opendev.org/740830 | 23:53 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!