| *** artom has joined #openstack-qa | 00:08 | |
| *** rfolco has quit IRC | 00:29 | |
| *** ricolin has joined #openstack-qa | 01:12 | |
| openstackgerrit | zhufl proposed openstack/tempest master: Adding description for testcases - compute part4 https://review.opendev.org/730565 | 01:28 |
|---|---|---|
| *** brinzhang_ has quit IRC | 01:43 | |
| *** rh-jlabarre has quit IRC | 01:55 | |
| openstackgerrit | zhufl proposed openstack/tempest master: Adding description for testcases - compute part5 https://review.opendev.org/731067 | 02:11 |
| openstackgerrit | Colleen Murphy proposed openstack/tempest master: Add system scope for admin auth https://review.opendev.org/686073 | 02:38 |
| openstackgerrit | Colleen Murphy proposed openstack/tempest master: Add default RBAC personas to dynamic credentials https://review.opendev.org/686306 | 02:38 |
| openstackgerrit | Colleen Murphy proposed openstack/tempest master: Add default RBAC personas to pre-provisioned creds https://review.opendev.org/698397 | 02:38 |
| openstackgerrit | Colleen Murphy proposed openstack/tempest master: Add client methods and tests for system grants https://review.opendev.org/743865 | 02:38 |
| *** brinzhang has joined #openstack-qa | 02:44 | |
| *** brinzhang_ has joined #openstack-qa | 02:54 | |
| *** brinzhang has quit IRC | 02:57 | |
| *** ramishra has joined #openstack-qa | 03:52 | |
| *** paras333 has quit IRC | 03:59 | |
| *** paras333 has joined #openstack-qa | 03:59 | |
| *** paras333 has quit IRC | 04:05 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #openstack-qa | 04:33 | |
| *** vishakha has joined #openstack-qa | 04:55 | |
| *** ramishra has quit IRC | 04:59 | |
| *** ramishra has joined #openstack-qa | 05:04 | |
| *** udesale has joined #openstack-qa | 05:13 | |
| *** ramishra has quit IRC | 05:14 | |
| *** ramishra has joined #openstack-qa | 05:15 | |
| *** gcheresh has joined #openstack-qa | 05:29 | |
| *** chandankumar is now known as chkumar|ruck | 05:37 | |
| *** ralonsoh has joined #openstack-qa | 05:43 | |
| *** brinzhang0 has joined #openstack-qa | 05:46 | |
| *** vishalmanchanda has joined #openstack-qa | 05:49 | |
| *** brinzhang_ has quit IRC | 05:50 | |
| openstackgerrit | Colleen Murphy proposed openstack/tempest master: Add default RBAC personas to pre-provisioned creds https://review.opendev.org/698397 | 06:09 |
| *** vhari has quit IRC | 06:12 | |
| *** lpiwowar has joined #openstack-qa | 06:12 | |
| *** chkumar|ruck is now known as chkumar|rover | 06:13 | |
| *** Luzi has joined #openstack-qa | 06:27 | |
| *** akahat is now known as akahat|PTO | 06:30 | |
| *** tosky has joined #openstack-qa | 06:32 | |
| *** ricolin has quit IRC | 06:43 | |
| *** ricolin has joined #openstack-qa | 06:43 | |
| *** ricolin has quit IRC | 06:44 | |
| *** ricolin has joined #openstack-qa | 06:46 | |
| *** slaweq has joined #openstack-qa | 07:10 | |
| *** bhagyashris is now known as bhagyashris|lunc | 07:28 | |
| *** rcernin has quit IRC | 07:32 | |
| *** jpena|off is now known as jpena | 07:56 | |
| *** nightmare_unreal has joined #openstack-qa | 07:58 | |
| *** lucasagomes has joined #openstack-qa | 08:08 | |
| *** markvoelker has joined #openstack-qa | 08:20 | |
| *** markvoelker has quit IRC | 08:25 | |
| *** rcernin has joined #openstack-qa | 08:28 | |
| *** rcernin has quit IRC | 08:34 | |
| *** ccamposr has joined #openstack-qa | 08:40 | |
| openstackgerrit | Ignacy Osetek proposed openstack/devstack stable/queens: Fix for downloading uwsgi on centos in lib/apache https://review.opendev.org/736189 | 08:47 |
| openstackgerrit | zhufl proposed openstack/tempest master: Adding description for testcases - identity part5 https://review.opendev.org/724249 | 09:02 |
| *** dtantsur|afk is now known as dtantsur | 09:03 | |
| *** markvoelker has joined #openstack-qa | 09:18 | |
| *** markvoelker has quit IRC | 09:22 | |
| openstackgerrit | Alexandre Arents proposed openstack/tempest master: Add related test to Bug #1732428 https://review.opendev.org/743708 | 09:26 |
| openstack | bug 1732428 in OpenStack Compute (nova) "Unshelving a VM breaks instance metadata when using qcow2 backed images" [Medium,In progress] https://launchpad.net/bugs/1732428 - Assigned to Alexandre arents (aarents) | 09:26 |
| openstackgerrit | Merged openstack/devstack master: Install bindep packages when installing lib from src https://review.opendev.org/743548 | 09:27 |
| *** bhagyashris|lunc is now known as bhagyashris | 09:37 | |
| kopecmartin | afazekas: when you have a moment, https://review.opendev.org/#/c/737545/ , https://review.opendev.org/#/c/742836/ thanks | 10:04 |
| kopecmartin | gmann: masayukig can you please have a look https://review.opendev.org/#/c/701352/ - it's blocking manila efforts | 10:04 |
| masayukig | kopecmartin: looking, is it urgent one? | 10:05 |
| kopecmartin | masayukig: i wouldn't say urgent urgent but it would be great if could move it forward in the upcoming days | 10:07 |
| masayukig | kopecmartin: ok, I'm not sure it's good to approve with a single company author and reviewers. | 10:09 |
| kopecmartin | masayukig: no rushing, therefore i'm trying to socialize the change a little, so that we can get approval also from other non redhat cores :) | 10:12 |
| masayukig | kopecmartin: thanks, +2, anyway :) | 10:13 |
| kopecmartin | thanks | 10:15 |
| *** dtantsur is now known as dtantsur|brb | 10:23 | |
| *** psahoo has joined #openstack-qa | 10:28 | |
| *** rcernin has joined #openstack-qa | 10:37 | |
| *** rcernin has quit IRC | 10:54 | |
| openstackgerrit | Merged openstack/tempest master: Adding description for testcases - identity part3 https://review.opendev.org/723365 | 10:58 |
| openstackgerrit | Merged openstack/tempest master: Adding description for testcases - network part3 https://review.opendev.org/723166 | 10:58 |
| *** nightmare_unreal has quit IRC | 10:59 | |
| *** markvoelker has joined #openstack-qa | 11:19 | |
| *** vishakha has quit IRC | 11:24 | |
| *** raildo has joined #openstack-qa | 11:25 | |
| *** markvoelker has quit IRC | 11:27 | |
| *** jpena is now known as jpena|lunch | 11:29 | |
| *** elod is now known as elod_afk | 11:42 | |
| openstackgerrit | Merged openstack/tempest master: Adding description for testcases - compute part4 https://review.opendev.org/730565 | 11:47 |
| *** udesale_ has joined #openstack-qa | 11:54 | |
| *** udesale has quit IRC | 11:57 | |
| *** brinzhang_ has joined #openstack-qa | 12:00 | |
| *** rh-jelabarre has joined #openstack-qa | 12:01 | |
| *** rfolco has joined #openstack-qa | 12:02 | |
| *** brinzhang0 has quit IRC | 12:02 | |
| *** brinzhang0 has joined #openstack-qa | 12:09 | |
| *** brinzhang_ has quit IRC | 12:11 | |
| *** psahoo has quit IRC | 12:18 | |
| *** jpena|lunch is now known as jpena | 12:32 | |
| *** psahoo has joined #openstack-qa | 12:32 | |
| *** nweinber has joined #openstack-qa | 12:37 | |
| openstackgerrit | Lukas Piwowarski proposed openstack/tempest master: Make create_user domain-aware for V3CredsClient https://review.opendev.org/642237 | 12:42 |
| *** tkajinam has quit IRC | 12:47 | |
| *** dtantsur|brb is now known as dtantsur | 12:53 | |
| *** paras333 has joined #openstack-qa | 13:07 | |
| openstackgerrit | Soniya Murlidhar Vyas proposed openstack/tempest master: [WIP] Need to have stable tempest scenario manager https://review.opendev.org/724631 | 13:27 |
| *** yoctozepto has quit IRC | 13:54 | |
| *** yoctozepto has joined #openstack-qa | 13:55 | |
| *** psahoo has quit IRC | 13:55 | |
| openstackgerrit | Merged openstack/tempest master: Fix KeyError exc in volumes_client https://review.opendev.org/742836 | 13:56 |
| *** Luzi_ has joined #openstack-qa | 14:10 | |
| *** Luzi has quit IRC | 14:11 | |
| gmann | masayukig: afazekas please check this - https://review.opendev.org/#/c/742295/ | 14:12 |
| gmann | kopecmartin: looking | 14:12 |
| kopecmartin | gmann: thanks .. btw, is this still in progress? i see bunch of merged reviews, isn't it done already? https://bugs.launchpad.net/tempest/+bug/1844568 | 14:22 |
| openstack | Launchpad bug 1844568 in tempest "[compute] "create_test_server" if networks is undefined and more than one network is present" [Medium,In progress] - Assigned to Ghanshyam Mann (ghanshyammann) | 14:22 |
| *** elod_afk is now known as elod | 14:27 | |
| *** vishakha has joined #openstack-qa | 14:31 | |
| gmann | kopecmartin: we covered most of the tests and we kept this to find and fix more tests but i have not seen network failure recently. | 14:36 |
| gmann | kopecmartin: we can close this and if something comes up then just fix it. | 14:36 |
| gmann | without this bug link or just with ref to bug | 14:36 |
| kopecmartin | ok, makes sense | 14:38 |
| *** bhagyashris is now known as bhagyashris|away | 14:47 | |
| *** andrebeltrami has joined #openstack-qa | 14:55 | |
| *** lpiwowar has quit IRC | 15:32 | |
| *** gcheresh has quit IRC | 15:35 | |
| *** Luzi_ has quit IRC | 15:36 | |
| gmann | kopecmartin: i can not reproduce this - https://review.opendev.org/#/c/701352/9 | 15:56 |
| gmann | decorator import is added at the right location | 15:56 |
| kopecmartin | gmann: manila tempest plugin is perfect repo for testing that | 16:01 |
| gmann | kopecmartin: humm i am testing in tempest and working fine | 16:02 |
| kopecmartin | gmann: really? without the patch? hmm, let me try | 16:04 |
| gmann | kopecmartin: yeah, i removed all id and decorator import from one test class and then run check-uuid --fix | 16:06 |
| kopecmartin | gmann: it wasn't causing problems in all files, just under a certain conditions like number of imports | 16:08 |
| *** gcheresh has joined #openstack-qa | 16:08 | |
| *** lucasagomes has quit IRC | 16:09 | |
| gmann | kopecmartin: number of import ? why. i tried with where import were after decorator also. | 16:11 |
| gmann | do you know exact cases where is it failing? | 16:11 |
| tosky | iirc manila-tempest-plugin | 16:12 |
| kopecmartin | gmann: i don't remember, it's been a few months already , trying to reproduce now, give me a few minutes | 16:12 |
| *** udesale_ has quit IRC | 16:13 | |
| kopecmartin | gmann: http://paste.openstack.org/show/796469/ | 16:15 |
| kopecmartin | f.e. manila_tempest_tests/tests/api/admin/test_share_networks.py | 16:15 |
| kopecmartin | in that file the import line (from tempest.lib import decorators) was added at the end of the file | 16:16 |
| *** dtantsur is now known as dtantsur|afk | 16:25 | |
| *** gcheresh has quit IRC | 16:26 | |
| gmann | kopecmartin: strange, i ran and it says fixed but no file changed | 16:30 |
| kopecmartin | gmann: because it modifies the installed plugin which is a copy somewhere in site-packages in your venv .. when you check my paste, you'll see I did 'cp -r ..' to copy the changes to the repo in order to run pep8 | 16:34 |
| gmann | kopecmartin: humm that should not be the case, for cinder tempest plugin it worked fine and changed cloned repo | 16:35 |
| kopecmartin | really? :D that's maybe another bug then | 16:35 |
| *** slaweq has quit IRC | 16:37 | |
| gmann | kopecmartin: it work fine for tempest and cinder tempest plugin but i could reproduce the error for manila tempest plugin where it fix all files in installed path | 16:54 |
| *** jpena is now known as jpena|off | 17:04 | |
| gouthamr | gmann: i'm only guessing here, but it looks like the behavior occurs when there is no tempest import: https://review.opendev.org/#/c/701108/1/manila_tempest_tests/tests/api/admin/test_security_services.py | 17:04 |
| gouthamr | when there's a tempest import, it works fine: https://review.opendev.org/#/c/701108/1/manila_tempest_tests/tests/api/admin/test_services_negative.py | 17:05 |
| gouthamr | so its possible cinder_tempest_plugin has tempest imports in each test file | 17:05 |
| gmann | gouthamr: ah right, tried it locally and yes this is the case, thanks | 17:07 |
| gmann | gouthamr: all these are now manually updated for import right? - https://review.opendev.org/#/c/701108/ | 17:09 |
| gouthamr | gmann: the problem seems to be on this line: https://review.opendev.org/#/c/701352/9/tempest/lib/cmd/check_uuid.py,unified@189 | 17:09 |
| gmann | not with check-uuid fix | 17:09 |
| gmann | yeah checking tempest_imports is the issue | 17:10 |
| gouthamr | gmann: nope, they were generated with check-uuid, after liron's tempest change - there's a depends-on in the commit message | 17:10 |
| gouthamr | to indicate that we need this change for the future :) | 17:11 |
| gmann | ohk in PS3 not in latest - https://review.opendev.org/#/c/701108/3..7//COMMIT_MSG | 17:11 |
| gouthamr | oh, didn't notice that.. | 17:12 |
| gouthamr | gmann: i'll suggest including it if he needs to make another patch | 17:13 |
| gmann | ok, i will test locally for tempest fix which should be enough | 17:14 |
| gouthamr | okay, i just noticed that the import order wasn't really maintained, as well - so will add comments on lkuchlan's patch | 17:52 |
| gouthamr | gmann: we had a generic question we asked around a bit, but not on this channel | 17:53 |
| gouthamr | gmann: what's the guidance on test case UUIDs when using ddt? | 17:53 |
| gouthamr | gmann: i'm wondering if there's been a discussion prior, here ... | 17:54 |
| gmann | gouthamr: yeah, the best we can do there is to add a uuid (single uuid) and treat all the ddt generated tests as single test | 17:54 |
| gouthamr | gmann: ack, that's what liron's proposed against manila-tempest-plugin, but, it doesn't solve the problem of test case uniqueness, no? | 17:55 |
| gouthamr | i mean, you could drop a ddt input, or change it substantially | 17:55 |
| gmann | gouthamr: in case of ddt, no. | 17:55 |
| gouthamr | and there is this issue where when hash randomization is enabled, ddt generated test names could be different on each run | 17:56 |
| gmann | yeah, even we add uuid map with ddt input that also does not solve the issue as regex would not find those tests as separate tests | 17:57 |
| *** ralonsoh has quit IRC | 18:00 | |
| gmann | that is why we said consider the ddt tests as single test when using uuid things | 18:00 |
| *** gcheresh has joined #openstack-qa | 18:02 | |
| gouthamr | hmmm, makes sense, a tough problem, ddt also insists on putting a test case number even when you're adding an annotation and a friendly name/uuid - and that test case number will change if hash randomization is enabled | 18:04 |
| *** andrebeltrami has quit IRC | 18:04 | |
| gouthamr | thanks for talking through this gmann, ddt is really useful, and i don't want to discourage folks from using it - but, it makes things tough for folks that use these test case names for bookkeeping and analyzing regressions over time | 18:07 |
| openstackgerrit | Merged openstack/tempest master: Adding description for testcases - compute part5 https://review.opendev.org/731067 | 18:08 |
| afazekas | gmann, Line 89 still sounds unsafe to me.. | 18:16 |
| *** chaconpiza has quit IRC | 18:21 | |
| openstackgerrit | Merged openstack/tempest master: Adding description for testcases - identity part5 https://review.opendev.org/724249 | 18:28 |
| gmann | afazekas: humm, ok. i am not sure if that goes to active before import is executed | 18:28 |
| gmann | abhishekk: can image status move to other than 'uploading' once image staging is completed and before import is executed? | 18:28 |
| *** gcheresh has quit IRC | 18:29 | |
| *** vishakha has quit IRC | 19:04 | |
| *** dtroyer has joined #openstack-qa | 19:15 | |
| afazekas | gmann, '-ing' named status sounds like transient state name .. | 19:41 |
| gmann | afazekas: yeah, i can remove that check if glance changing the states asynchronously. | 19:43 |
| gmann | dansmith: do you know after image stage and before import operation, image status goes to active or any other status than 'uploading' ? | 19:45 |
| dansmith | gmann: I think it's queued -> uploading -> active | 19:46 |
| dansmith | uploading means you have staged it, AFAIK | 19:46 |
| dansmith | import won't run unless it's in 'uploading' state, unless it's copy-image, and then 'active' in that case | 19:46 |
| gmann | yeah that is my understanding and it stay 'uploading ' until import operation is run and complete | 19:47 |
| openstackgerrit | Lee Yarwood proposed openstack/tempest master: config: Introduce [compute-feature-enabled]/cross_az_attach https://review.opendev.org/744039 | 19:47 |
| openstackgerrit | Lee Yarwood proposed openstack/tempest master: compute: Skip AttachVolumeShelveTestJSON when cross_az_attach unavailable https://review.opendev.org/744040 | 19:47 |
| dansmith | gmann: looks like there is also an "importing" state for glance-direct and web-download | 19:48 |
| gmann | that is after import operation is started | 19:48 |
| dansmith | so stage puts it to 'uploading', | 19:48 |
| dansmith | import puts it to 'importing' until complete, then 'active', except for copy-image | 19:48 |
| dansmith | I didn't know about the importing state until I just looked | 19:48 |
| gmann | afazekas: this is pre condition for import operation, you can see in '' - https://docs.openstack.org/api-ref/image/v2/?expanded=import-an-image-detail#import-an-image | 19:49 |
| gmann | 'Additional Preconditions' part ^^ | 19:49 |
| gmann | dansmith: yeah. copy is the only exception to have image in active state | 19:50 |
| dansmith | yeah | 19:50 |
| gmann | afazekas: so checking 'uploading' state before glance-direct import is right thing | 19:50 |
| dansmith | and that is done before we return from the api, so you should be able to assume it right away | 19:53 |
| gmann | afazekas: it could be named as 'staged' which might be more clear name than '..ing' | 19:53 |
| gmann | dansmith: yeah, confusion is about its name 'uploading' which sounds like in background it will be complete uploading and change state to something else. | 19:54 |
| dansmith | yeah | 19:54 |
| dansmith | it's very confusing | 19:54 |
| dansmith | I'm also not sure about the states for web-download, | 19:57 |
| dansmith | because there's no stage step, which means never uploading, but then importing includes the download and the upload to the store | 19:58 |
| dansmith | so in glance-direct, $time for stage and $time for import, but web-download will be 2x$time for importing | 19:58 |
| gmann | yeah, in current proposed test we are asserting on same timeout as glance-direct import but we can see if that is less for web-download. | 20:00 |
| dansmith | it'll be more for web-download, but it shouldn't be long anyway at least for cirros | 20:01 |
| gmann | yeah, most of the time issue are reported from 3rd party CI not with upstream cirros | 20:01 |
| gmann | next step can be to move these tests to scenario tests and boot VM(or snapshot operation ) etc from these images after import | 20:02 |
| dansmith | ack | 20:03 |
| gmann | afazekas: i can add note about 'uploading' state in followup if that work for you? | 20:04 |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: Add import line unit tests for check_uuid command https://review.opendev.org/744048 | 20:34 |
| gmann | kopecmartin: adding coverage for current issue where line_no was 0 and end up adding import at the end of file ^^ | 20:36 |
| gmann | we can see result on that with and without fix | 20:37 |
| gmann | and one fix needed is for no import case where it can end up adding import before licence paragraph. | 20:38 |
| *** vishalmanchanda has quit IRC | 20:39 | |
| *** nweinber has quit IRC | 21:20 | |
| *** paras333 has quit IRC | 21:23 | |
| *** raildo has quit IRC | 21:29 | |
| *** tkajinam has joined #openstack-qa | 22:05 | |
| *** sucho has joined #openstack-qa | 22:36 | |
| sucho | hi i have a question regarding | 22:36 |
| sucho | installing devstack on my ubuntu. I've been trying many other things to get devstack on my device and each time i'm getting losetup: /dev/loop0: set direct io failed: Invalid argument | 22:37 |
| sucho | i'm pretty much new to programming world and i have no idea how to solve out the issue | 22:37 |
| clarkb | sucho: can you use a service like http://paste.openstack.org or https://gist.github.com to paste some log context then share the resulting link here? Also information about the version of ubuntu you are running on would be helpful. If it is focal I'm not sure if devstack is fully working there yet and bionic may be a better choice for now | 22:38 |
| sucho | of course, its morning time here my buddy is fixing breakfast now after i finish them i'll share them asap | 22:43 |
| *** paras333 has joined #openstack-qa | 23:08 | |
| openstackgerrit | Merged openstack/openstack-health master: Remove .testr.conf https://review.opendev.org/729138 | 23:13 |
| *** rcernin has joined #openstack-qa | 23:13 | |
| *** rcernin has quit IRC | 23:14 | |
| *** rcernin has joined #openstack-qa | 23:14 | |
| *** tosky has quit IRC | 23:14 | |
| *** sucho has quit IRC | 23:25 | |
| *** artom has quit IRC | 23:32 | |
| *** artom has joined #openstack-qa | 23:40 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!