Tuesday, 2020-06-23

*** rfolco has quit IRC00:32
brinzhang_Why there is an error in stable/ussuri branch of the CI lack of pip3?00:35
brinzhang_the error is https://zuul.opendev.org/t/openstack/build/58ebb827638a43aa90cfcff269403138/log/job-output.txt#412-41700:35
brinzhang_msg": "[Errno 2] No such file or directory: 'pip3': 'pip3'",00:35
*** artom has quit IRC00:39
*** artom has joined #openstack-qa00:40
*** artom has quit IRC00:45
*** Liang__ has joined #openstack-qa01:18
*** lseki has quit IRC01:31
*** paras333 has quit IRC01:42
*** ricolin has joined #openstack-qa02:08
*** ricolin has quit IRC02:20
*** ricolin has joined #openstack-qa02:30
*** rcernin has quit IRC02:41
*** rcernin has joined #openstack-qa02:43
*** ricolin has quit IRC02:51
*** paras333 has joined #openstack-qa02:52
*** paras333 has quit IRC02:56
*** ricolin has joined #openstack-qa03:05
*** markvoelker has joined #openstack-qa03:06
*** markvoelker has quit IRC03:11
*** vishalmanchanda has joined #openstack-qa03:15
*** ricolin has quit IRC03:45
*** ricolin has joined #openstack-qa03:45
*** dmellado has quit IRC03:59
openstackgerritFederico Ressi proposed openstack/devstack master: Fix the cornercase when $PYTHON is /bin/python3*  https://review.opendev.org/73730504:12
*** rh-jelabarre has quit IRC04:16
openstackgerritFederico Ressi proposed openstack/devstack master: Fix the cornercase when $PYTHON is /bin/python3*  https://review.opendev.org/73730504:23
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-qa04:33
*** ricolin has quit IRC04:33
*** udesale has joined #openstack-qa04:41
*** markvoelker has joined #openstack-qa04:44
*** markvoelker has quit IRC04:49
openstackgerritFederico Ressi proposed openstack/devstack master: Fix the cornercase when $PYTHON is /bin/python3*  https://review.opendev.org/73730504:56
*** ricolin has joined #openstack-qa04:56
openstackgerritFederico Ressi proposed openstack/devstack master: Fix the cornercase when $PYTHON is /bin/python3*  https://review.opendev.org/73730504:59
*** markvoelker has joined #openstack-qa05:04
*** markvoelker has quit IRC05:09
*** abdysn has joined #openstack-qa05:13
*** ricolin has quit IRC05:14
*** dtantsur|afk has quit IRC05:14
*** ricolin has joined #openstack-qa05:14
*** dtantsur has joined #openstack-qa05:15
*** Liang__ has quit IRC05:17
*** Liang__ has joined #openstack-qa05:18
*** gcheresh has joined #openstack-qa05:21
*** ricolin has quit IRC05:26
*** soniya29|afk is now known as soniya29|rover05:26
*** ricolin has joined #openstack-qa05:28
*** soniya29|rover is now known as soniya29|afk05:40
openstackgerritJens Harbott (frickler) proposed openstack/devstack master: Use python3 pip module instead of pip binary  https://review.opendev.org/73505806:12
openstackgerritJens Harbott (frickler) proposed openstack/devstack master: Make devstack run on focal (Ubuntu LTS 20.04)  https://review.opendev.org/70483106:12
openstackgerritJens Harbott (frickler) proposed openstack/devstack master: Stop claiming to support Debian distros  https://review.opendev.org/73744306:12
*** rpittau|afk is now known as rpittau06:34
*** ralonsoh has joined #openstack-qa06:37
*** nightmare_unreal has joined #openstack-qa06:38
openstackgerritJens Harbott (frickler) proposed openstack/devstack master: Switch devstack base job to Ubuntu Focal  https://review.opendev.org/73120706:49
*** markvoelker has joined #openstack-qa07:05
*** markvoelker has quit IRC07:11
*** vishakha has joined #openstack-qa07:24
*** soniya29|afk is now known as soniya29|rover07:35
*** tosky has joined #openstack-qa07:35
openstackgerritMartin Kopec proposed openstack/tempest master: Unskip test_container_synchronization  https://review.opendev.org/73745207:47
*** markvoelker has joined #openstack-qa07:49
openstackgerritMartin Kopec proposed openstack/tempest master: Unskip test_server_connectivity_cold_migration_revert  https://review.opendev.org/73745307:51
*** markvoelker has quit IRC07:54
*** jpena|off is now known as jpena07:54
*** lucasagomes has joined #openstack-qa08:07
*** rcernin has quit IRC08:18
*** gfidente|afk is now known as gfidente08:25
openstackgerritJens Harbott (frickler) proposed openstack/devstack master: Drop support for python2  https://review.opendev.org/73745508:26
openstackgerritMerged openstack/devstack master: Remove neutron-fwaas from the jobs' required project  https://review.opendev.org/73585308:38
*** ricolin has quit IRC08:39
*** gcheresh has quit IRC09:31
*** gcheresh has joined #openstack-qa09:32
*** udesale_ has joined #openstack-qa09:46
*** ricolin has joined #openstack-qa09:47
*** udesale has quit IRC09:49
*** markvoelker has joined #openstack-qa09:50
*** markvoelker has quit IRC09:55
*** gfidente has quit IRC10:04
*** ccamposr has quit IRC10:10
*** rpittau is now known as rpittau|bbl10:12
*** ccamposr has joined #openstack-qa10:16
*** tkajinam has quit IRC10:17
*** Liang__ has quit IRC10:23
*** jhesketh has quit IRC10:36
*** jhesketh has joined #openstack-qa10:37
*** gcheresh has quit IRC10:55
*** ricolin has quit IRC10:55
openstackgerritFederico Ressi proposed openstack/devstack master: Look for python executables in /usr prefix first than others  https://review.opendev.org/73749811:14
*** raildo has joined #openstack-qa11:18
*** gfidente has joined #openstack-qa11:28
openstackgerritFederico Ressi proposed openstack/devstack master: Look for python executables in /usr prefix before than other places  https://review.opendev.org/73749811:28
*** jpena is now known as jpena|lunch11:31
*** gcheresh has joined #openstack-qa11:32
openstackgerritLukas Piwowarski proposed openstack/tempest master: WIP: Create documentation for tests  https://review.opendev.org/73561911:45
openstackgerritJens Harbott (frickler) proposed openstack/devstack master: Drop support for python2  https://review.opendev.org/73745511:46
*** markvoelker has joined #openstack-qa11:52
*** vishakha has quit IRC11:53
*** markvoelker has quit IRC11:57
*** rfolco has joined #openstack-qa12:01
*** rh-jelabarre has joined #openstack-qa12:04
*** rpittau|bbl is now known as rpittau12:14
*** jpena|lunch is now known as jpena12:35
*** ricolin has joined #openstack-qa12:36
openstackgerritIgnacy Osetek proposed openstack/devstack master: Fix for downloading uwsgi on centos in lib/apache  https://review.opendev.org/73619412:45
openstackgerritIgnacy Osetek proposed openstack/devstack stable/queens: Fix for downloading uwsgi on centos in lib/apache  https://review.opendev.org/73618912:46
*** lpiwowar has joined #openstack-qa12:48
*** nweinber has joined #openstack-qa12:54
*** chaconpiza has joined #openstack-qa12:57
masayukig#startmeeting qa13:01
openstackMeeting started Tue Jun 23 13:01:13 2020 UTC and is due to finish in 60 minutes.  The chair is masayukig. Information about MeetBot at http://wiki.debian.org/MeetBot.13:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.13:01
*** openstack changes topic to " (Meeting topic: qa)"13:01
openstackThe meeting name has been set to 'qa'13:01
masayukighi, who all are here today?13:01
kopecmartini'm here, but only in a read mode, i have another meeting at the same time this week :/13:01
lpiwowarHi13:02
masayukigkopecmartin: thanks, sure13:02
masayukignp13:02
masayukiglpiwowar: Hi :)13:02
*** paras333 has joined #openstack-qa13:02
masayukigLet's wait for a couple of minutes so that other can join just in case13:03
paras333Hi13:03
masayukigparas333: hi \o13:03
toskyhi13:05
masayukigtosky: hi :D13:06
masayukigok, let's start.13:06
masayukig#link https://wiki.openstack.org/wiki/Meetings/QATeamMeeting13:07
masayukig^^^^^ today's agenda13:07
masayukig#topic Announcement and Action Item (Optional)13:07
*** openstack changes topic to "Announcement and Action Item (Optional) (Meeting topic: qa)"13:07
masayukigThere's no announcement from my side.13:07
masayukigso, let's skip if nothing13:07
masayukig#topic Victoria Priority Items progress13:08
*** openstack changes topic to "Victoria Priority Items progress (Meeting topic: qa)"13:08
masayukig#link https://etherpad.opendev.org/p/qa-victoria-priority13:08
masayukigany update about the priority things?13:10
lpiwowarI looked today at the generating of the documentation for the tests but there is some kind of problem with generating documentation using tox -e pdf-docs.13:10
masayukiglpiwowar: yeah, #link https://review.opendev.org/#/c/735619/ this pathc?13:11
masayukigpatch13:11
*** ccamposr has quit IRC13:12
lpiwowarI am sorry for not sending the link. Yes, this patch.13:12
masayukiglpiwowar: ++ np :) thanks for working on that13:12
lpiwowarmasayukig: No problem:)13:13
masayukigI'll look at it, maybe next week since I've been busy last week and this week too :/13:13
lpiwowarOk:)13:14
masayukigcool13:15
masayukigSo, tosky paras333 anything from you?13:15
paras333masayukig:  nothing much this week, but yes I would like you to comment on the hacking etherpad that is still pending13:16
paras333https://etherpad.opendev.org/p/hacking13:16
paras333specially on the most common one13:16
masayukig#link https://etherpad.opendev.org/p/hacking13:16
masayukigparas333: ++ cool13:16
masayukigyeah, I'll do it definitely13:17
toskymasayukig: nothing for me; watching the fires and, when possible, trying to fix them13:18
kopecmartintosky: yeah, lots of things got broken last week :D13:18
masayukigtosky: thank you so much, that's super important for us13:19
masayukigok, let's move on if nothing else13:20
masayukig#topic OpenStack Events Updates and Planning13:20
*** openstack changes topic to "OpenStack Events Updates and Planning (Meeting topic: qa)"13:20
masayukigIt looks there'll be Berlin Summit on Berlin, Germany OCTOBER 19-23. But I don't know that much and I'm not sure I'll be able to attend or not.13:21
masayukig#link https://www.openstack.org/summit/2020/13:21
masayukigMaybe I can't in this circumstance so far.13:22
kopecmartinyeah, hard to predict what the situation will be then :/13:22
masayukigyup :(13:23
masayukiganyway, I'll share if I know something about it13:23
masayukigok, let's move on13:23
masayukig#topic Sub Teams highlights (Sub Teams means individual projects under QA program)13:24
*** openstack changes topic to "Sub Teams highlights (Sub Teams means individual projects under QA program) (Meeting topic: qa)"13:24
masayukigDo you have any updates/highlights on the sub-projects such as Tempest, Patrole and Hacking?13:24
masayukig#link https://review.openstack.org/#/q/project:openstack/tempest+status:open13:24
masayukig#link https://review.openstack.org/#/q/project:openstack/patrole+status:open13:24
masayukig#link https://review.opendev.org/#/q/project:openstack/hacking+status:open13:24
masayukigIf nothing, let's move on.13:25
masayukig#topic Community goal tracking13:25
*** openstack changes topic to "Community goal tracking (Meeting topic: qa)"13:25
masayukig#link https://etherpad.opendev.org/p/qa-community-wide-goals-tracking13:26
masayukigIt looks no updates on the etherpad, let's skip atm13:26
masayukig#topic Bug Triage13:27
*** openstack changes topic to "Bug Triage (Meeting topic: qa)"13:27
masayukig#link https://etherpad.openstack.org/p/qa-bug-triage-victoria13:27
masayukigAny bugs we would like to discuss?13:27
kopecmartinthis week mainly the reviews there13:28
kopecmartinit be great if you could take a look at some of them13:28
kopecmartinsome of them are really close being merged and fix a bug13:29
masayukigkopecmartin: ++ thanks! Line 30-36 patches on the etherpad?13:29
kopecmartinmasayukig: yes13:30
masayukigthanks, will do, next week hopefully..13:30
masayukigOK, let's move on13:30
masayukig#topic Critical Reviews13:30
*** openstack changes topic to "Critical Reviews (Meeting topic: qa)"13:30
masayukigAny patches you'd like to discuss?13:31
masayukigOK, if nothing, open discussion13:32
masayukig#topic Open Discussion13:32
*** openstack changes topic to "Open Discussion (Meeting topic: qa)"13:32
masayukigToday's office hour is super quick one :)13:33
masayukigDo you have anything to discuss?13:33
masayukigOK, I think we have nothing this time. Let's close the office hour.13:35
masayukigthank you all!!13:35
masayukig#endmeeting13:36
*** openstack changes topic to "Discussion of OpenStack Quality Assurance Projects (tempest, devstack, grenade, and friends) | docs https://wiki.openstack.org/wiki/QA | specs https://review.openstack.org/#/q/project:openstack/qa-specs | channel logs http://eavesdrop.openstack.org/irclogs/%23openstack-qa/"13:36
openstackMeeting ended Tue Jun 23 13:36:00 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)13:36
openstackMinutes:        http://eavesdrop.openstack.org/meetings/qa/2020/qa.2020-06-23-13.01.html13:36
lpiwowarBye13:36
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/qa/2020/qa.2020-06-23-13.01.txt13:36
openstackLog:            http://eavesdrop.openstack.org/meetings/qa/2020/qa.2020-06-23-13.01.log.html13:36
*** markvoelker has joined #openstack-qa13:53
*** markvoelker has quit IRC13:57
openstackgerritMartin Kopec proposed openstack/tempest master: Fix and unskip test_list_servers_filtered_by_ip_regex  https://review.opendev.org/73754514:02
*** ccamposr has joined #openstack-qa14:13
*** abdysn has quit IRC14:22
*** dave-mccowan has joined #openstack-qa14:30
*** markvoelker has joined #openstack-qa14:39
*** ricolin has quit IRC14:42
*** markvoelker has quit IRC14:44
*** yamamoto has quit IRC14:45
*** yamamoto has joined #openstack-qa14:50
*** jparker_ has joined #openstack-qa14:50
*** jparker_ has quit IRC14:50
*** lpiwowar has quit IRC14:58
*** artom has joined #openstack-qa15:12
*** yamamoto has quit IRC15:18
openstackgerritSean McGinnis proposed openstack/devstack master: Remove dragonflow reference  https://review.opendev.org/73756315:33
*** yamamoto has joined #openstack-qa15:57
*** rpittau is now known as rpittau|afk16:00
*** lucasagomes has quit IRC16:03
*** yamamoto has quit IRC16:07
kopecmartingmann: hi, i'm trying to unskip a test which should be fixed by now https://review.opendev.org/#/c/737452/ .. but i just wonder where has it been executed last 3 months when it was skipped that time: status.openstack.org/openstack-health/#/test/tempest.api.compute.servers.test_servers_negative.ServersNegativeTestJSON.test_rebuild_deleted_server?duration=P3M16:16
gmannfrickler: yoctozepto documenting the greande job process for stable -https://review.opendev.org/#/c/736866/16:16
*** dave-mccowan has quit IRC16:17
gmannkopecmartin: you mean you are not able to see that in o-h?16:18
AJaegercould I get some review love for a bashate cleanup https://review.opendev.org/#/c/734436/ and a os-performance-tools cleanup https://review.opendev.org/#/c/734508/, please?16:20
kopecmartingmann: hmm, should i? i just see statistics , not a job name running it :/16:21
gmannkopecmartin: yeah, o-h would not help that. if you are 100% sure then we can unskip ortherwise we have another decorator unstable_test which will keep recording its pass state but would fail jobs if it fail and after some time we can check the test stability from o-h and make it unskip16:22
gmannkopecmartin: https://github.com/openstack/tempest/blob/206a9abec5e317b0644072d1308f74ce4f6a326d/tempest/lib/decorators.py#L15916:23
gmannAJaeger: done16:23
kopecmartingmann: oh, the decorator sounds better , i'll edit the review16:24
gmanncool16:24
gmannthat way we can keep job also not failing if that fail16:25
*** dave-mccowan has joined #openstack-qa16:27
AJaegerthanks, gmann !16:28
*** dtantsur is now known as dtantsur|afk16:29
openstackgerritMerged openstack/bashate master: Remove obsolete files  https://review.opendev.org/73443616:30
openstackgerritMerged openstack/os-performance-tools master: Cleanup py27 support  https://review.opendev.org/73450816:33
*** gcheresh has quit IRC16:37
*** vishakha has joined #openstack-qa16:38
*** markvoelker has joined #openstack-qa16:40
openstackgerritMartin Kopec proposed openstack/tempest master: Unskip test_container_synchronization  https://review.opendev.org/73745216:43
*** markvoelker has quit IRC16:45
openstackgerritMartin Kopec proposed openstack/tempest master: Unstable test_server_connectivity_cold_migration_revert  https://review.opendev.org/73745316:46
openstackgerritMerged openstack/devstack master: Enable certificate check for glance_store+swift  https://review.opendev.org/64455816:47
openstackgerritMartin Kopec proposed openstack/tempest master: Unstable test_container_synchronization  https://review.opendev.org/73745216:48
*** udesale_ has quit IRC17:05
*** jpena is now known as jpena|off17:11
*** ralonsoh has quit IRC17:28
*** ccamposr has quit IRC17:42
*** gcheresh has joined #openstack-qa17:42
*** markvoelker has joined #openstack-qa17:54
*** markvoelker has quit IRC17:59
*** markvoelker has joined #openstack-qa18:11
*** gcheresh has quit IRC18:12
*** markvoelker has quit IRC18:15
*** markvoelker has joined #openstack-qa18:21
*** markvoelker has quit IRC18:31
*** nightmare_unreal has quit IRC18:44
*** lseki has joined #openstack-qa19:07
*** gfidente is now known as gfidente|afk19:15
openstackgerritGhanshyam Mann proposed openstack/devstack master: Switch devstack base job to Ubuntu Focal  https://review.opendev.org/73120719:35
*** vishakha has quit IRC19:37
openstackgerritGhanshyam Mann proposed openstack/tempest master: Migrate tempest jobs to focal  https://review.opendev.org/73470019:39
openstackgerritGhanshyam Mann proposed openstack/devstack master: Switch devstack base job to Ubuntu Focal  https://review.opendev.org/73120719:41
*** gcheresh has joined #openstack-qa19:47
*** yamamoto has joined #openstack-qa20:07
*** yamamoto has quit IRC20:11
*** markvoelker has joined #openstack-qa20:25
*** markvoelker has quit IRC20:29
*** nweinber has quit IRC20:30
*** Open10K8S has joined #openstack-qa20:43
openstackgerritJens Harbott (frickler) proposed openstack/devstack master: Drop support for python2  https://review.opendev.org/73745520:46
Open10K8SHi team20:49
Open10K8SI am trying to use tempest plugins in zuul gate job20:49
Open10K8Shttps://opendev.org/vexxhost/openstack-operator/src/branch/master/zuul.d/functional-jobs.yaml#L26-L3020:49
Open10K8Sthen tempest plugins are listed when "tempest list-plugins"20:51
Open10K8SBut i cannot find in the Unit Test Report20:51
gmannOpen10K8S: you need to use either one tempest_plugins or TEMPEST_PLUGINS, former one is latest and recommended variable20:51
Open10K8Sgmann: Can you see the link?20:52
Open10K8SI used them20:52
Open10K8S2020-06-22 14:21:51.262747 | controller | venv-tempest run-test: commands[0] | tempest list-plugins20:54
Open10K8S2020-06-22 14:21:52.293332 | controller | +--------------+--------------------------------------------------+20:54
Open10K8S2020-06-22 14:21:52.293391 | controller | |     Name     |                    EntryPoint                    |20:54
Open10K8S2020-06-22 14:21:52.293421 | controller | +--------------+--------------------------------------------------+20:54
Open10K8S2020-06-22 14:21:52.293437 | controller | | magnum_tests | magnum_tempest_plugin.plugin:MagnumTempestPlugin |20:54
Open10K8S2020-06-22 14:21:52.293456 | controller | |     heat     |   heat_tempest_plugin.plugin:HeatTempestPlugin   |20:54
Open10K8S2020-06-22 14:21:52.293470 | controller | +--------------+--------------------------------------------------+20:54
Open10K8Shttps://fe62bacc72140c45c9b8-1d41175f1f59c744683b1a918bffa4f9.ssl.cf2.rackcdn.com/737088/8/check/openstack-operator:functional/7a1a430/job-output.txt20:54
*** vishalmanchanda has quit IRC20:54
Open10K8Sand as you can see in this zuul job result, the tempest plugins are listed20:54
Open10K8Sgmann: but not executed the test scenarios surely20:55
Open10K8Shttps://fe62bacc72140c45c9b8-1d41175f1f59c744683b1a918bffa4f9.ssl.cf2.rackcdn.com/737088/8/check/openstack-operator:functional/7a1a430/testr_results.html20:55
Open10K8Sgmann: how about your idea?20:56
gmannOpen10K8S: yeah you are using both var so plugins defined in tempest_plugins are ignored which is same as defined in TEMPEST_PLUGINS so you can see both plugins but you need to use either one.20:57
Open10K8Sgmann: so you mean I need to use only one var?20:58
Open10K8Stempest_plugins or TEMPEST_PLUGINS?20:58
gmannyes, tempest_plugins but that is not the issue for test not run but may be later if we remove the support of TEMPEST_PLUGINS20:58
gmannOpen10K8S: to run the test from plugin you need to define the tempest_test_regex and tox_envlist21:00
gmannotherwise default test running are tempest smoke test21:00
gmannexample: https://opendev.org/openstack/magnum-tempest-plugin/src/branch/master/.zuul.yaml#L1721:00
gmanntempest_test_regex can be combination of tempest or plugins tests depends on what all you want to run in that job21:01
Open10K8Show about several plugins?21:02
Open10K8Sif i want to run 3 plugins in tempest, how can i pass that param?21:02
gmannsame way, via tempest_test_regex21:02
gmanntempest will run the tests matching the tempest_test_regex and what all plugins are installed in your system.21:03
gmannso 1. you have to install all plugins you want to run the test 2. use tox_envlist: all 3. tempest_test_regex21:04
gmannOpen10K8S: example:  tempest_test_regex: '(^tempest\.(api|scenario)|(^magnum_tempest_plugin)|(^heat_tempest_plugin)|(^designate_tempest_plugin))'21:05
gmannthis should run tempest + all three mentioned plugins tests21:06
Open10K8Sgmann: Thank you21:06
Open10K8SI got your point21:06
gmannyou can define the regex as per your need and also we support tempest_black_regex if you want to skip some test21:08
gmannOpen10K8S: this doc can give more detail on what all you can do with the tempest run - https://github.com/openstack/tempest/tree/206a9abec5e317b0644072d1308f74ce4f6a326d/roles/run-tempest21:10
Open10K8Sgmann: ok21:11
openstackgerritParas Babbar proposed openstack/tempest master: Add skip flags for block level live migration  https://review.opendev.org/73763421:12
openstackgerritMerged openstack/tempest master: Fix the argument check for account-generator  https://review.opendev.org/73312721:13
paras333gmann:  kopecmartin : Can you take a look on this patch https://review.opendev.org/#/c/737634/, pretty trivial just added two skip condition for block migration which uses max_microversion:2.2421:14
gmannparas333: does not seems trivial :). 2.24 is pretty old microversion and tests were passing till now?21:17
openstackgerritMerged openstack/tempest master: Prevent raising exception if IP address is set in _hotplug_server  https://review.opendev.org/71635521:17
*** gcheresh has quit IRC21:17
gmannbut added it for review21:17
paras333gmann:  yeah this test pass if the we set this conf block_migration_for_live_migration: true21:19
paras333but my suggestion is it should not be even run with this value yes, for  micro_version: 2.25 it works fine as that api bu default add "—block-migrate"21:20
paras333gmann:  anyways you can review it and then we can discuss later, no rush!21:21
openstackgerritDan Smith proposed openstack/devstack master: Use glance import workflow for creating image  https://review.opendev.org/73596621:22
gmannparas333: is this test not run on any other job than tempest-slow-py3 ? tempest-slow-py3 run it with block_migration_for_live_migration: true21:23
paras333gmann:   I actually tested this locally downstream21:25
paras333let me find out if there is any job upstream running it without the flags, but I suspect there is none21:26
paras333:/21:26
gmannparas333: commented, that test is already capped with 2.24 and 2.25 test class run those test with block_migration=auto21:29
gmannparas333: i am curios why this test run with <2.2421:30
gmanni mean >2.24, test will be making request to nova with <2.2421:31
paras333gmann:  LiveAutoBlockMigrationV225Test works fine with block level migration, it's just through the class lass LiveMigrationTest(LiveMigrationTestBase):21:32
paras333    max_microversion = '2.24'21:32
paras333    block_migration = None21:32
paras333where block_migration=None that fails21:32
openstackgerritMerged openstack/devstack master: Use python3 pip module instead of pip binary  https://review.opendev.org/73505821:33
gmannparas333: with max_microversion = '2.24' means, if configured min microversion in tempest conf is >2.24 then this test will be skipped21:33
gmannand it it is None, which is default then requested microversion from tempest will be <2.2.421:34
gmannparas333: this is how we handle the microvesion tests - https://opendev.org/openstack/tempest/src/branch/master/tempest/lib/common/api_version_utils.py#L4121:36
paras333gmann:  ack, so tempest.conf we have set max_microversion=latest and the test were not skipped21:37
paras333and failed by adding block_migration=none in the api21:38
gmannparas333: it will not be skipped with max_microversion=latest, it mean run all, but what is min_microversion ?21:40
paras333gmann:  2.121:40
paras333for compute and for volume it is 3.021:41
gmannparas333: ok so then this test should run with 2.1 microversion and nova should pass as request is less than 2.24 microversion21:41
paras333gmann:  So don't you think we should have skip flags for this case?21:42
gmannparas333: no, because with that config you are saying run all the tests falling int these microversion and tempest make API call with max-of-min-config-or-text-min version21:52
paras333paras333:  correct but this still should be skipped if the block_migration_for_live_migration is not enabled it should not be performing the block level migration and end up performing the shared storage live migration21:56
gmannparas333: but this test passing block_migration=None22:00
paras333gmann:  as this is getting the value from here conf file https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_live_migration.py#L63-%23L6722:02
paras333in my case if I add this block_migration_for_live_migration : true in conf this will works22:03
*** markvoelker has joined #openstack-qa22:25
*** markvoelker has quit IRC22:30
*** artom has quit IRC22:41
*** tosky has quit IRC22:42
*** clarkb has quit IRC22:44
*** tkajinam has joined #openstack-qa22:53
*** clarkb has joined #openstack-qa22:58
*** rcernin has joined #openstack-qa23:02
gmannparas333: yeah but that will not be >2.24 as is_requested_microversion_compatible() checks the selected microversion only https://github.com/openstack/tempest/blob/f9ba3e6625d2ef5704a569d9a107996e58061273/tempest/api/compute/base.py#L19323:06
gmannwith max_microversion=2.24 in test we are well sure that if test run it will run with microversion <2.24 else skip.23:07
gmannif it end up running with >2.24 and block_migration other than None then it is issue.23:08
*** rcernin has quit IRC23:08
*** rcernin has joined #openstack-qa23:08
*** yamamoto has joined #openstack-qa23:10
gmannparas333: may be we can run this in DNM patch to reproduce this .23:15
paras333gmann:  DNM?23:16
gmannparas333: ah, we run that in multinode job - https://321f9535d79beb0392d4-53a7512c858a5878113edb75a034ec0e.ssl.cf5.rackcdn.com/737453/2/check/tempest-multinode-full-py3/1c88c95/testr_results.html23:18
paras333gmann:  but we are setting this https://321f9535d79beb0392d4-53a7512c858a5878113edb75a034ec0e.ssl.cf5.rackcdn.com/737453/2/check/tempest-multinode-full-py3/1c88c95/controller/logs/tempest_conf.txt block_migration_for_live_migration = True23:20
paras333which is fetching the value from here: block_migration = (CONF.compute_feature_enabled.23:21
paras333https://review.opendev.org/gitweb?p=openstack/tempest.git;a=blob;f=tempest/api/compute/admin/test_live_migration.py;h=a845c7231145e372459e65a848a9c0b64c6e7508;hb=HEAD#l66                                block_migration_for_live_migration and23:21
paras333https://review.opendev.org/gitweb?p=openstack/tempest.git;a=blob;f=tempest/api/compute/admin/test_live_migration.py;h=a845c7231145e372459e65a848a9c0b64c6e7508;hb=HEAD#l67                                not volume_backed)23:21
paras333sorry this :block_migration = (CONF.compute_feature_enabled.23:22
paras333                               block_migration_for_live_migration and not volume_backed)23:22
paras333this will fail only when we don't mention this and this test will add block_migration=none and run the test as shared storage live migration which is what the error says23:23
paras333so may be before this condition we can add the skip flag like if this value is none CONF.compute_feature_enabled. block_migration_for_live_migration then skip the test23:24
gmannparas333: here it is passing - https://zuul.opendev.org/t/openstack/build/1c88c95b42b8429e8c45594635742ca6/log/controller/logs/tempest_log.txt#1336823:25
gmannwhat is different in your env?23:25
gmannblock_migration is passed as true in this23:25
paras333gmann:  this flag was missing block_migration_for_live_migration = True23:28
gmannparas333: in ^^ job?23:28
paras333no in my env23:28
paras333paras333:  in the job you mentioned it's there if i add that test will work perfectly fine23:29
*** markvoelker has joined #openstack-qa23:29
paras333in my env too hence i suggested to skip the test rather than giving error23:29
gmannok, i think we do not have multinode job with that23:30
*** freerunner has quit IRC23:30
paras333yeah because if you see in the code it will be taking this value from the conf file only at <2.2423:30
openstackgerritGhanshyam Mann proposed openstack/tempest master: DNM: testing block live migration test  https://review.opendev.org/73765823:30
gmannparas333: ^^23:30
*** freerunner has joined #openstack-qa23:31
paras333gmann:  ack23:31
gmannparas333: yeah in that case block_migration will be false right so should pass but let's see the request and resposne in testing patch23:32
paras333gmann:  sure , thanks!23:32
gmannparas333: we do not test multinode things in different way, may be we can add some experimental jobs with live migration config. let's see how it behave and then we can add to capture the issues if any23:33
paras333gmann:  yeah good idea23:34
*** markvoelker has quit IRC23:34
*** raildo has quit IRC23:35
paras333gmann:  do we have to remove this conf from group_vars too?23:38
paras333group-vars:23:38
paras333      peers:23:38
paras333        devstack_localrc:23:38
paras333          NOVA_ALLOW_MOVE_TO_SAME_HOST: false23:38
paras333          LIVE_MIGRATION_AVAILABLE: true23:38
paras333          USE_BLOCK_MIGRATION_FOR_LIVE_MIGRATION: true23:38
paras333for the DNM job you added23:38
gmannah yes23:38
openstackgerritGhanshyam Mann proposed openstack/tempest master: DNM: testing block live migration test  https://review.opendev.org/73765823:39
paras333gmann:  cool, thanks!23:39
*** yamamoto has quit IRC23:42
*** gfidente|afk has quit IRC23:52

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