| *** paras333 has joined #openstack-qa | 01:05 | |
| *** paras333 has quit IRC | 01:09 | |
| gmann | AJaeger: yeah we need those along with project side change. | 01:22 |
|---|---|---|
| openstackgerrit | Ghanshyam Mann proposed openstack/patrole master: Add Patrole gate job for stable/train https://review.opendev.org/701271 | 01:27 |
| *** gregwork has quit IRC | 01:33 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/patrole master: Add Patrole gate job for stable/train https://review.opendev.org/701271 | 01:33 |
| *** ramishra has joined #openstack-qa | 02:20 | |
| *** yamamoto has joined #openstack-qa | 03:35 | |
| *** yamamoto has quit IRC | 03:38 | |
| *** ricolin_ has joined #openstack-qa | 03:49 | |
| *** evrardjp has quit IRC | 04:35 | |
| *** evrardjp has joined #openstack-qa | 04:36 | |
| *** udesale has joined #openstack-qa | 04:42 | |
| *** gcheresh has joined #openstack-qa | 04:51 | |
| *** vishalmanchanda has joined #openstack-qa | 05:43 | |
| *** slaweq has joined #openstack-qa | 06:13 | |
| openstackgerrit | Andreas Jaeger proposed openstack/devstack master: Switch to new grenade job name https://review.opendev.org/725179 | 06:21 |
| AJaeger | gmann: pushed https://review.opendev.org/725176 and https://review.opendev.org/725179 for the grenade-py3 rename ^ | 06:23 |
| *** rpittau|afk is now known as rpittau | 06:33 | |
| *** ccamposr has joined #openstack-qa | 07:06 | |
| *** ccamposr__ has joined #openstack-qa | 07:08 | |
| *** ccamposr has quit IRC | 07:11 | |
| *** gcheresh has quit IRC | 07:30 | |
| *** tosky has joined #openstack-qa | 07:32 | |
| *** vishakha has joined #openstack-qa | 07:37 | |
| *** gcheresh has joined #openstack-qa | 07:45 | |
| *** jpena|off is now known as jpena | 07:49 | |
| *** nightmare_unreal has joined #openstack-qa | 08:00 | |
| *** chaconpiza has joined #openstack-qa | 08:03 | |
| iurygregory | cgoncalves, if you want to run another set of tests you can use tempest_test_regex in your job definition, by default it will run smoke tests after grenade, maybe you can also set BASE_RUN_SMOKE to False (you can set under grenade_devstack_localrc: shared: | 08:05 |
| iurygregory | something like http://paste.openstack.org/show/793028/ | 08:07 |
| tosky | BASE_RUN_SMOKE controls the smoke tests before grenade (the ones after are disabled) | 08:07 |
| tosky | everything else is correct | 08:07 |
| iurygregory | oh =) | 08:07 |
| tosky | thanks iurygregory :) | 08:07 |
| cgoncalves | iurygregory, hi. we have that already today: https://github.com/openstack/octavia/blob/master/devstack/upgrade/settings#L10-L11 | 08:07 |
| tosky | cgoncalves: I answered on the list too | 08:07 |
| iurygregory | tks tosky o/ | 08:07 |
| tosky | cgoncalves: the settings file is not used anymore | 08:08 |
| cgoncalves | tosky, oh, good to know. thanks | 08:08 |
| tosky | cgoncalves: just like for devstack the settings are set in one place only, the zuul definition | 08:08 |
| cgoncalves | tosky, hum? what if one wants to run grenade locally? | 08:09 |
| iurygregory | yeah you need to set in the job definition | 08:09 |
| tosky | uhm, grenade still support the old way (running also devstack), but that will may go away at some point; when it does, I guess we need an easy way to do what zuul does by setting the configuration | 08:11 |
| tosky | small correction: the devstack/upgrade/settins files are still read, just the function devstack_localrc is a noop | 08:13 |
| tosky | so yeah, the idea is that you run devstack, then you set GRENADE_USE_EXTERNAL_DEVSTACK=True and you execute grenade | 08:15 |
| cgoncalves | tosky, are you sure it is a noop? per https://opendev.org/openstack/grenade/src/branch/master/PLUGINS.rst#user-content-changing-devstack-localrc, it does not look like so | 08:32 |
| tosky | cgoncalves: it is a noop | 08:33 |
| tosky | cgoncalves: https://opendev.org/openstack/grenade/src/branch/master/inc/plugin#L167 | 08:33 |
| cgoncalves | ok, it is noop *if* GRENADE_USE_EXTERNAL_DEVSTACK=True | 08:34 |
| tosky | which is the new way, and the non-external devstack will probably go away | 08:34 |
| cgoncalves | if that is so, please consider adding a deprecation note in the doc | 08:35 |
| tosky | sure, that makes sense | 08:35 |
| cgoncalves | thanks | 08:36 |
| tosky | cgoncalves: but it's written there | 08:36 |
| tosky | " When GRENADE_USE_EXTERNAL_DEVSTACK is set to True, as it happens on the Zuul grenade jobs where devstack is configured and executed before grenade, the function has no effect." | 08:36 |
| iurygregory | tosky, do you have any ideas on smoke tests failing?.-. https://5da7fa74180cdcfe2917-eb0b1bb80d6d8b85b62f24c50591cd92.ssl.cf1.rackcdn.com/723646/6/check/ironic-inspector-grenade/960703c/controller/logs/grenade.sh_log.txt | 08:37 |
| cgoncalves | tosky, yes but there's no mention of the *possibility* of deprecating those | 08:37 |
| tosky | right | 08:38 |
| cgoncalves | tosky, users will read that sentence as: "look, there are 2 supported ways: X and Y" | 08:38 |
| tosky | that point about the deprecation came during the review process, and in fact I haven't realized that when we switch everything to native, the path where grenade itself runs devstack won't be tested anymore anywhere | 08:39 |
| tosky | just need to find the right spot | 08:40 |
| tosky | iurygregory: uh, any news with the ironic grenade job? I remember we discussed about specific tests taking too much time | 08:42 |
| iurygregory | tosky, it's for the inspector, I've try adding new configs but still facing timeouts | 08:43 |
| *** dtantsur|afk is now known as dtantsur | 08:43 | |
| iurygregory | now it's happenign in the smoke during the grenade run | 08:43 |
| iurygregory | =( | 08:43 |
| tosky | uhm, but would those tests run successfully (in an acceptable time) after the upgrade? | 08:44 |
| iurygregory | we have scenarios where it would run in the acceptable time | 08:45 |
| iurygregory | https://zuul.openstack.org/builds?job_name=ironic-inspector-grenade | 08:45 |
| iurygregory | but we have a lot of FAILURE / TIMED_OUT =( | 08:45 |
| tosky | I see, and it started failing in a more consistent way only recently? | 08:47 |
| iurygregory | kinda | 08:48 |
| iurygregory | when we get FAILURE there are 2 scenarios I've found | 08:49 |
| iurygregory | smoke tests during the upgrade or after | 08:49 |
| *** udesale has quit IRC | 08:49 | |
| iurygregory | seems most of the times it happens in rax - ORD | 08:52 |
| cgoncalves | tosky, updated the octavia grenade patch. if you have 2 minutes, I'd appreciate a review https://review.opendev.org/#/c/725098 | 08:54 |
| *** gfidente has joined #openstack-qa | 08:57 | |
| tosky | sure! | 08:57 |
| *** udesale has joined #openstack-qa | 09:27 | |
| cgoncalves | thanks | 09:38 |
| *** yamamoto has joined #openstack-qa | 09:50 | |
| *** yamamoto has quit IRC | 09:58 | |
| *** ralonsoh has joined #openstack-qa | 10:03 | |
| *** gcheresh has quit IRC | 10:04 | |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/devstack master: WIP: Make devstack run on focal (Ubuntu LTS 20.04) https://review.opendev.org/704831 | 10:10 |
| *** rpittau is now known as rpittau|bbl | 10:14 | |
| *** gcheresh has joined #openstack-qa | 10:53 | |
| *** slaweq_ has joined #openstack-qa | 11:18 | |
| *** slaweq has quit IRC | 11:21 | |
| *** jpena is now known as jpena|lunch | 11:30 | |
| *** yamamoto has joined #openstack-qa | 11:37 | |
| *** yamamoto has quit IRC | 11:37 | |
| *** nweinber has joined #openstack-qa | 11:45 | |
| *** raildo has joined #openstack-qa | 11:58 | |
| *** rfolco has joined #openstack-qa | 12:04 | |
| openstackgerrit | Vishakha Agarwal proposed openstack/patrole master: Add Patrole gate job for stable/ussuri https://review.opendev.org/725023 | 12:10 |
| *** dmellado has quit IRC | 12:10 | |
| openstackgerrit | Merged openstack/devstack master: Switch to new grenade job name https://review.opendev.org/725179 | 12:15 |
| *** dmellado has joined #openstack-qa | 12:16 | |
| *** rpittau|bbl is now known as rpittau | 12:22 | |
| *** paras333 has joined #openstack-qa | 12:23 | |
| *** paras333 has quit IRC | 12:27 | |
| *** jpena|lunch is now known as jpena | 12:31 | |
| *** udesale_ has joined #openstack-qa | 12:38 | |
| *** paras333 has joined #openstack-qa | 12:38 | |
| *** paras333 has left #openstack-qa | 12:38 | |
| *** paras333 has joined #openstack-qa | 12:39 | |
| *** udesale has quit IRC | 12:41 | |
| *** yamamoto has joined #openstack-qa | 13:11 | |
| *** yamamoto has quit IRC | 13:16 | |
| *** ricolin_ has quit IRC | 13:17 | |
| *** ralonsoh has quit IRC | 13:37 | |
| *** artom has joined #openstack-qa | 13:37 | |
| *** ralonsoh has joined #openstack-qa | 13:49 | |
| openstackgerrit | Jens Harbott (frickler) proposed openstack/devstack master: WIP: Make devstack run on focal (Ubuntu LTS 20.04) https://review.opendev.org/704831 | 14:04 |
| iurygregory | tosky, now I'm trying to figure out why ironic-grenade works fine but ironic-inspector-grenade has problems with timeouts XD | 14:26 |
| gmann | AJaeger: there is already for keystone -https://review.opendev.org/#/c/725149/1 | 14:37 |
| iurygregory | tosky, it's possible to disable smoke in pre-upgrade and run a set of tests from ironic only? | 14:46 |
| iurygregory | I know that for post-upgrade I can just set the tempest_test_regex | 14:46 |
| tosky | iurygregory: yes, just set vars.grenade_localrc.BASE_RUN_SMOKE: False | 14:48 |
| gmann | iurygregory: you can disable base smoke test run by BASE_RUN_SMOKE=False but running other tests are not configurable for pre-upgrade testing | 14:48 |
| gmann | the only things it can include the other tests if those are from tempest plugin which is installed and smoke tests. but it will run those tests with tempest smoke | 14:49 |
| iurygregory | tosky, tks | 14:49 |
| iurygregory | dtantsur, ^ pre-upgrade tests are not configurable but we can disable smoke | 14:50 |
| gmann | iurygregory: idea behind running smoke tests in pre-upgrade is just to verify the basic installation is fine or not otherwise there is not much benefits of running pre-upgrade tests | 14:50 |
| tosky | iurygregory: that said, I mean, smoke tests should work; it's just a normal devstack deployment up to that point | 14:50 |
| dtantsur | gmann: well, for us smoke tests are super heavy | 14:50 |
| tosky | I see | 14:50 |
| dtantsur | we'd be happy to verify the basic installation, but not through something that can take 1.5 hrs... | 14:51 |
| gmann | dtantsur: ok, i will disable them does not harm much. | 14:51 |
| dtantsur | I'd rather be able to have our own sanity tests run | 14:51 |
| gmann | * i will say | 14:51 |
| dtantsur | but if we need to disable them, so be it | 14:51 |
| AJaeger | gmann: my change has already +2 and passed tests - do you want to abandon yours? Normally I would abandon mine since yours is older... | 14:51 |
| tosky | dtantsur: if they are failing now, maybe they were disabled even in the legacy job? | 14:51 |
| gmann | if installation is wrong or something , it can be caught during upgrade tests | 14:51 |
| gmann | AJaeger: i am ok to abandon. | 14:52 |
| dtantsur | tosky: they're not failing, they just take long and are so extensive that they catch transient problems quite often | 14:52 |
| AJaeger | gmann: thx | 14:52 |
| AJaeger | gmann: and sorry for missing that - I checked but missed that one ;( | 14:52 |
| gmann | yeah, if those are taking long then disable is ok. I was thinking to remove those at all initially but as those are smoke and does not take time i kept it to run. | 14:53 |
| tosky | dtantsur: right, sorry, it's timeout: my point is that even with the legacy job you couldn't configure them, so they were probably disabled | 14:53 |
| gmann | AJaeger: its ok. that happen :) | 14:53 |
| tosky | so disabling them now would just bring back the old behavior | 14:53 |
| dtantsur | tosky: they were not disabled, we were just suffering :) | 14:54 |
| tosky | oh :) | 14:54 |
| dtantsur | and now I'd like to use this refactoring as a chance to mitigate this problem | 14:54 |
| dtantsur | if that means disabling pre-upgrade tests - fine with me | 14:54 |
| tosky | ok, so nothing new - it's not great, but from the point of view of the goal (migration from legacy to native zuulv3) is fine | 14:55 |
| gmann | dtantsur: +1 | 14:55 |
| dtantsur | yeah, it's not exactly blocking the goal. it just happened to coincide with another case where timeouts in a grenade job brought us to standstill :) | 14:56 |
| openstackgerrit | Felipe Monteiro proposed openstack/patrole master: Undo renaming member to Member role in devstack plugin install https://review.opendev.org/725297 | 15:06 |
| openstackgerrit | Felipe Monteiro proposed openstack/patrole master: Undo renaming member to Member role in devstack plugin install https://review.opendev.org/725297 | 15:07 |
| *** rh-jelabarre has joined #openstack-qa | 15:40 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/patrole master: Add Patrole gate job for stable/train https://review.opendev.org/701271 | 15:57 |
| *** rpittau is now known as rpittau|afk | 16:08 | |
| *** gcheresh has quit IRC | 16:22 | |
| openstackgerrit | Merged openstack/tempest master: Update supported version doc tempest https://review.opendev.org/725029 | 16:26 |
| *** evrardjp has quit IRC | 16:35 | |
| *** evrardjp has joined #openstack-qa | 16:36 | |
| *** dtantsur is now known as dtantsur|afk | 16:57 | |
| *** jpena is now known as jpena|off | 17:04 | |
| *** udesale_ has quit IRC | 17:06 | |
| *** ralonsoh has quit IRC | 17:07 | |
| *** vishalmanchanda has quit IRC | 17:15 | |
| openstackgerrit | Merged openstack/patrole master: Remove the stable rocky job from Patrole master gate https://review.opendev.org/722184 | 17:25 |
| openstackgerrit | Vishakha Agarwal proposed openstack/patrole master: Add Patrole gate job for stable/ussuri https://review.opendev.org/725023 | 17:40 |
| *** nightmare_unreal has quit IRC | 17:52 | |
| *** redrobot has joined #openstack-qa | 18:33 | |
| gmann | masayukig: frickler: https://review.opendev.org/#/c/724716/ https://review.opendev.org/#/c/724718/1 | 19:02 |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: Moving tetsing to Victoria tetsing runtime https://review.opendev.org/725373 | 19:17 |
| openstackgerrit | Merged openstack/devstack-plugin-ceph master: Removes upstart support https://review.opendev.org/722371 | 19:18 |
| openstackgerrit | Merged openstack/devstack-plugin-ceph master: Check that ceph-mon service has started to get its version https://review.opendev.org/722427 | 19:18 |
| openstackgerrit | Ghanshyam Mann proposed openstack/bashate master: Moving tetsing to Victoria tetsing runtime https://review.opendev.org/725376 | 19:23 |
| openstackgerrit | Ghanshyam Mann proposed openstack/os-testr master: Moving tetsing to Victoria tetsing runtime https://review.opendev.org/725377 | 19:29 |
| AJaeger | gmann: want to fix your script to use "testing" instead of "tetsing"? | 19:30 |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: Moving tetsing to Victoria testing runtime https://review.opendev.org/725373 | 19:31 |
| openstackgerrit | Ghanshyam Mann proposed openstack/os-testr master: Moving tetsing to Victoria testing runtime https://review.opendev.org/725377 | 19:31 |
| openstackgerrit | Ghanshyam Mann proposed openstack/bashate master: Moving tetsing to Victoria testing runtime https://review.opendev.org/725376 | 19:31 |
| gmann | thanks, done | 19:32 |
| gmann | felipe: patrole train job is passing now. i think ussuri too | 19:32 |
| AJaeger | gmann: you changed one place but I see now it was broken twice "moving testing..." | 19:36 |
| gmann | :), i am too lazy for commit msg. | 19:39 |
| openstackgerrit | Ghanshyam Mann proposed openstack/bashate master: Switch testing to Victoria testing runtime https://review.opendev.org/725376 | 19:40 |
| AJaeger | nothing to really worry about... | 19:40 |
| openstackgerrit | Ghanshyam Mann proposed openstack/os-testr master: Switch testing to Victoria testing runtime https://review.opendev.org/725377 | 19:40 |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest master: Switch testing to Victoria testing runtime https://review.opendev.org/725373 | 19:40 |
| *** gcheresh has joined #openstack-qa | 19:46 | |
| *** lseki has joined #openstack-qa | 19:51 | |
| *** vishakha has quit IRC | 20:13 | |
| *** nweinber has quit IRC | 20:14 | |
| *** slaweq_ has quit IRC | 21:08 | |
| *** slaweq_ has joined #openstack-qa | 21:14 | |
| *** gcheresh has quit IRC | 21:15 | |
| *** slaweq_ has quit IRC | 21:19 | |
| *** rfolco has quit IRC | 21:27 | |
| openstackgerrit | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Updates script to use latest version of Ceph https://review.opendev.org/676722 | 21:49 |
| *** paras333 has quit IRC | 21:49 | |
| *** ysirndjuro has joined #openstack-qa | 21:50 | |
| *** rfolco has joined #openstack-qa | 22:03 | |
| *** raildo has quit IRC | 22:08 | |
| *** rcernin has joined #openstack-qa | 22:22 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/patrole master: Add Patrole gate job for stable/ussuri https://review.opendev.org/725023 | 23:08 |
| *** ccamposr has joined #openstack-qa | 23:17 | |
| *** ccamposr__ has quit IRC | 23:20 | |
| *** tosky has quit IRC | 23:28 | |
| openstackgerrit | Merged openstack/devstack-plugin-ceph master: [manila] Disable firewall and bgp setups https://review.opendev.org/724229 | 23:30 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!