*** luksky has quit IRC | 00:22 | |
*** spatel has joined #openstack-lbaas | 00:43 | |
*** spatel has quit IRC | 00:48 | |
johnsom | So, somehow we broke stable/stein, octavia-v2-dsvm-scenario-ubuntu-xenial, jobs. It is trying to use python3.6 but obviously xenial only had python2.7..... | 01:17 |
---|---|---|
johnsom | https://review.opendev.org/#/c/698597/ | 01:17 |
openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Fix octavia-v2-dsvm-scenario-ubuntu-xenial https://review.opendev.org/705590 | 01:22 |
johnsom | rm_work Would you please take a look at https://review.opendev.org/705590 ? stable/stein is broken. | 01:22 |
rm_work | sure | 01:23 |
johnsom | We got a little over excited about the python3 transition.... lol | 01:23 |
johnsom | It goes boom: https://zuul.opendev.org/t/openstack/build/3516c4d67f3447f69792b97aecbb9589/log/job-output.txt#13202 | 01:25 |
johnsom | ERROR: InterpreterNotFound: python3.6 | 01:25 |
*** mithilarun has quit IRC | 01:54 | |
*** hongbin has joined #openstack-lbaas | 02:29 | |
haleyb | johnsom: i saw that failure in a stable/stein backport, i'd be happy if your change fixes it :) | 03:39 |
*** vishalmanchanda has joined #openstack-lbaas | 03:52 | |
*** hongbin has quit IRC | 04:33 | |
*** larsks has quit IRC | 05:05 | |
*** squarebracket has quit IRC | 05:05 | |
*** larsks has joined #openstack-lbaas | 05:12 | |
*** squarebracket has joined #openstack-lbaas | 05:13 | |
*** gcheresh has joined #openstack-lbaas | 05:53 | |
openstackgerrit | Merged openstack/octavia-tempest-plugin master: Fix octavia-v2-dsvm-scenario-ubuntu-xenial https://review.opendev.org/705590 | 06:03 |
*** gcheresh has quit IRC | 06:08 | |
*** gcheresh has joined #openstack-lbaas | 06:08 | |
*** ltomasbo has joined #openstack-lbaas | 07:13 | |
*** ccamposr has joined #openstack-lbaas | 07:29 | |
*** mugsie has quit IRC | 07:37 | |
*** mugsie has joined #openstack-lbaas | 07:41 | |
*** yboaron_ has joined #openstack-lbaas | 07:48 | |
*** AlexStaf has joined #openstack-lbaas | 07:49 | |
*** yboaron_ has quit IRC | 07:55 | |
*** yboaron_ has joined #openstack-lbaas | 07:55 | |
*** yboaron_ has quit IRC | 08:03 | |
*** yboaron_ has joined #openstack-lbaas | 08:04 | |
*** yboaron_ has quit IRC | 08:04 | |
*** vesper11 has quit IRC | 08:06 | |
*** vesper11 has joined #openstack-lbaas | 08:07 | |
*** maciejjozefczyk has joined #openstack-lbaas | 08:14 | |
*** tkajinam has quit IRC | 08:17 | |
*** luksky has joined #openstack-lbaas | 08:21 | |
*** tesseract has joined #openstack-lbaas | 08:27 | |
*** rpittau|afk is now known as rpittau | 08:33 | |
*** gcheresh has quit IRC | 09:10 | |
*** gcheresh has joined #openstack-lbaas | 09:10 | |
brtknr | rm_work: thanks | 09:39 |
dulek | johnsom: The Amp build issue I mentioned before happens consistently on all of our stable/train gates. It's not a hiccup. | 10:39 |
*** gcheresh has quit IRC | 10:50 | |
*** gcheresh has joined #openstack-lbaas | 11:00 | |
*** andy_ has quit IRC | 11:14 | |
*** AlexStaf has quit IRC | 11:14 | |
*** mugsie has quit IRC | 11:14 | |
*** vishalmanchanda has quit IRC | 11:14 | |
*** ltomasbo has quit IRC | 11:14 | |
*** luketollefson has quit IRC | 11:14 | |
*** gcheresh has quit IRC | 11:14 | |
*** maciejjozefczyk has quit IRC | 11:14 | |
*** squarebracket has quit IRC | 11:14 | |
*** nmickus has quit IRC | 11:14 | |
*** sapd1 has quit IRC | 11:14 | |
*** strigazi has quit IRC | 11:14 | |
*** canori01 has quit IRC | 11:14 | |
*** logan- has quit IRC | 11:14 | |
*** fyx has quit IRC | 11:14 | |
*** dougwig has quit IRC | 11:14 | |
*** sorrison has quit IRC | 11:14 | |
*** irclogbot_2 has quit IRC | 11:15 | |
*** openstackstatus has quit IRC | 11:16 | |
*** irclogbot_3 has joined #openstack-lbaas | 11:16 | |
*** luketollefson has joined #openstack-lbaas | 11:17 | |
*** andy_ has joined #openstack-lbaas | 11:18 | |
*** vesper11 has quit IRC | 11:18 | |
*** ccamposr has quit IRC | 11:18 | |
*** gthiemonge has quit IRC | 11:18 | |
*** TMM has quit IRC | 11:18 | |
*** dmellado has quit IRC | 11:18 | |
*** TMM_ has joined #openstack-lbaas | 11:18 | |
*** AlexStaf has joined #openstack-lbaas | 11:18 | |
*** mugsie has joined #openstack-lbaas | 11:18 | |
*** ltomasbo has joined #openstack-lbaas | 11:18 | |
*** vishalmanchanda has joined #openstack-lbaas | 11:18 | |
*** gcheresh has joined #openstack-lbaas | 11:18 | |
*** maciejjozefczyk has joined #openstack-lbaas | 11:18 | |
*** squarebracket has joined #openstack-lbaas | 11:18 | |
*** nmickus has joined #openstack-lbaas | 11:18 | |
*** sapd1 has joined #openstack-lbaas | 11:18 | |
*** strigazi has joined #openstack-lbaas | 11:18 | |
*** canori01 has joined #openstack-lbaas | 11:18 | |
*** logan- has joined #openstack-lbaas | 11:18 | |
*** fyx has joined #openstack-lbaas | 11:18 | |
*** dougwig has joined #openstack-lbaas | 11:18 | |
*** sorrison has joined #openstack-lbaas | 11:18 | |
*** rm_work has quit IRC | 11:21 | |
*** dmellado has joined #openstack-lbaas | 11:21 | |
*** ccamposr has joined #openstack-lbaas | 11:21 | |
*** mnaser has quit IRC | 11:21 | |
*** rm_work has joined #openstack-lbaas | 11:22 | |
*** gthiemonge has joined #openstack-lbaas | 11:22 | |
*** vesper11 has joined #openstack-lbaas | 11:22 | |
*** mnaser has joined #openstack-lbaas | 11:22 | |
*** tesseract has quit IRC | 11:22 | |
*** tesseract has joined #openstack-lbaas | 11:24 | |
*** tesseract has quit IRC | 11:56 | |
brtknr | does neutron lbaas work on Train? | 12:09 |
cgoncalves | brtknr, neutron-lbaas was not released in Train. last release was Stein | 12:10 |
brtknr | cgoncalves: ah thanks | 12:15 |
*** tesseract has joined #openstack-lbaas | 12:26 | |
*** vishalmanchanda has quit IRC | 12:29 | |
*** tesseract has quit IRC | 12:29 | |
*** tesseract has joined #openstack-lbaas | 12:31 | |
*** tesseract has quit IRC | 12:37 | |
*** tesseract has joined #openstack-lbaas | 12:38 | |
*** tesseract has quit IRC | 12:40 | |
*** tesseract has joined #openstack-lbaas | 12:40 | |
*** tesseract has quit IRC | 12:50 | |
*** tesseract has joined #openstack-lbaas | 13:03 | |
*** tesseract has quit IRC | 13:07 | |
*** tesseract has joined #openstack-lbaas | 13:15 | |
*** luksky has quit IRC | 13:19 | |
*** gcheresh has quit IRC | 13:22 | |
*** tkajinam has joined #openstack-lbaas | 13:34 | |
*** maciejjozefczyk has quit IRC | 13:36 | |
*** gcheresh has joined #openstack-lbaas | 13:39 | |
*** pcaruana has quit IRC | 13:43 | |
*** maciejjozefczyk has joined #openstack-lbaas | 13:46 | |
*** sapd1_x has joined #openstack-lbaas | 14:17 | |
*** spatel has joined #openstack-lbaas | 14:19 | |
*** spatel has quit IRC | 14:24 | |
*** TrevorV has joined #openstack-lbaas | 14:32 | |
*** tkajinam has quit IRC | 14:32 | |
*** luksky has joined #openstack-lbaas | 14:44 | |
*** vishalmanchanda has joined #openstack-lbaas | 14:49 | |
*** gcheresh has quit IRC | 14:53 | |
*** pcaruana has joined #openstack-lbaas | 14:56 | |
*** sapd1_x has quit IRC | 15:10 | |
*** gcheresh has joined #openstack-lbaas | 15:32 | |
openstackgerrit | Carlos Goncalves proposed openstack/octavia stable/stein: DNM: testing fix for amphora image builds https://review.opendev.org/705713 | 15:32 |
*** Trevor_V has joined #openstack-lbaas | 15:43 | |
johnsom | dulek I just built a stable/train image locally, so the diskimage-builder project didn't break something. Can you provide a link the patch that is experiencing this? | 15:44 |
*** TrevorV has quit IRC | 15:44 | |
*** Trevor_V is now known as TrevorV | 15:44 | |
johnsom | 2020-02-04 15:43:42.840 | Build completed successfully | 15:44 |
johnsom | Successfully built the amphora using the stable/train amphora-agent. | 15:44 |
dulek | johnsom: Any of those: https://review.opendev.org/#/q/project:openstack/kuryr-kubernetes+status:open+branch:stable/train. | 15:45 |
dulek | johnsom: I'll go over our Zuul settings… | 15:45 |
dulek | johnsom: Do you see anything weird or missing here: https://github.com/openstack/kuryr-kubernetes/blob/28a871a036492e8fd1487744243714bc137df595/.zuul.d/octavia.yaml#L15-L47 ? | 15:47 |
johnsom | Yeah, these are all odd: | 15:48 |
johnsom | https://www.irccloud.com/pastebin/VDlP2b0O/ | 15:48 |
johnsom | Image size of 3 is only needed for centos/rhel images, but you aren't setting the centos variable. | 15:49 |
johnsom | No idea why you want to wait 9999 on nova to start the qemu process, but.... | 15:49 |
dulek | johnsom: The first one is a leftover from us using prebuilt CentOS image. | 15:50 |
johnsom | That usually is pretty fast, seconds on a healthy cloud. It's the nova boot time variable that can take too long and poor clouds. | 15:50 |
dulek | johnsom: The second one is there for a looong time, probably fighting gate slowness with nested virt. | 15:50 |
johnsom | dulek Yeah, that isn't even the nested virt timeout variable | 15:50 |
johnsom | It's just nova scheduling and qemu process start | 15:51 |
dulek | johnsom: Anyway that shouldn't matter too much as we've merged patches on Jan 15 with this. | 15:51 |
dulek | And as I only recently noticed this I'd bet it's either a59e7235560ad045b24c58ed722ab62a451174d3 or db26a3c96b98d13d7da7cc5575c81308e0b7a09b on openstack/octavia? | 15:52 |
dulek | Both touch DIB? | 15:52 |
dulek | johnsom: If that doesn't ring a bell I can try reverting them and testing? | 15:54 |
johnsom | Yeah, we merged stable/train patches Friday, so a bit puzzled too. Let me dig into your job a bit more. I see you have an older DIB for some reason. | 15:54 |
johnsom | dulek This isn't an Octavia bug for sure. The error is diskimage-builder before it hits our code. | 15:54 |
dulek | johnsom: Ah, interesting. So maybe it's db26a3c96b98d13d7da7cc5575c81308e0b7a09b as it seems to be reverted because a new version got released? | 15:55 |
dulek | And we don't have it? | 15:55 |
*** ltomasbo has quit IRC | 15:57 | |
johnsom | Yeah, your job has 2.27.0 from Sep 8, 2019. Not sure why as DIB is branchless | 15:57 |
dulek | johnsom: You have DIB on required-projects, so probably you're using git version. We don't and we use pip releases. | 15:58 |
johnsom | dulek I didn't use the git release for my local test. See pypi: https://pypi.org/project/diskimage-builder/#history | 15:58 |
dulek | johnsom: Ah, but latest is 2.33.0… | 15:59 |
johnsom | It should have used 2.33.0 | 15:59 |
dulek | johnsom: https://github.com/openstack/requirements/blob/stable/train/upper-constraints.txt#L612 | 16:00 |
dulek | johnsom: It's upper-constraints that block it. | 16:00 |
dulek | And the idea is to not bump it lightly in stable branches. | 16:01 |
johnsom | Yeah, that is a problem as DIB has had breaking issues | 16:01 |
johnsom | Maybe we should inquire in the DIB channel | 16:02 |
dulek | So there are 3 fixes - reverting db26a3c96b98d13d7da7cc5575c81308e0b7a09b, bumping upper-constraint or making kuryr-kubernetes use git version of DIB. | 16:02 |
dulek | I'd say either of the 2 first is a correct one. Probably best to bump the constraint. | 16:02 |
dulek | johnsom: I just joined #openstack-dib. | 16:03 |
*** ltomasbo has joined #openstack-lbaas | 16:19 | |
*** ltomasbo has quit IRC | 16:24 | |
*** gcheresh has quit IRC | 16:30 | |
*** maciejjozefczyk has quit IRC | 16:32 | |
*** jamesdenton has joined #openstack-lbaas | 16:39 | |
*** mithilarun has joined #openstack-lbaas | 16:42 | |
*** mithilarun has quit IRC | 16:55 | |
*** Trevor_V has joined #openstack-lbaas | 16:56 | |
*** TrevorV has quit IRC | 16:59 | |
*** tesseract has quit IRC | 17:04 | |
*** luksky has quit IRC | 17:07 | |
*** ccamposr has quit IRC | 17:22 | |
*** rpittau is now known as rpittau|afk | 17:27 | |
*** armax has quit IRC | 17:30 | |
*** mithilarun has joined #openstack-lbaas | 17:31 | |
*** mithilarun has quit IRC | 17:35 | |
*** luksky has joined #openstack-lbaas | 17:50 | |
*** TrevorV has joined #openstack-lbaas | 18:13 | |
*** Trevor_V has quit IRC | 18:16 | |
*** armax has joined #openstack-lbaas | 18:17 | |
*** mithilarun has joined #openstack-lbaas | 18:26 | |
*** mithilar_ has joined #openstack-lbaas | 18:27 | |
*** mithilarun has quit IRC | 18:31 | |
cgoncalves | can requirements be bumped on stable branches? | 18:36 |
cgoncalves | does kuryr assert for the stable policy tag? | 18:36 |
cgoncalves | never mind. you said upper-constraints (requirements project). that would be a "no" I guess | 18:37 |
*** mithilar_ has quit IRC | 18:46 | |
*** mithilarun has joined #openstack-lbaas | 18:46 | |
*** amuller has joined #openstack-lbaas | 18:56 | |
*** andy_ has quit IRC | 18:59 | |
*** luketollefson has quit IRC | 19:00 | |
*** andy_ has joined #openstack-lbaas | 19:01 | |
*** luketollefson has joined #openstack-lbaas | 19:02 | |
johnsom | Yeah, so I had a chat in the requirements channel. Since DIB is branch-less, we need to bump it in all of the stable UC files. | 19:09 |
*** mithilarun has quit IRC | 19:11 | |
*** mithilarun has joined #openstack-lbaas | 19:12 | |
*** mithilar_ has joined #openstack-lbaas | 19:17 | |
*** mithilarun has quit IRC | 19:21 | |
johnsom | haleyb For some reason your comment about the stable/train branch gives me the feeling you know more about this issue... lol. FYI, there is a conversation about it in the qa channel and they are working on a fix for the tempest tox env. | 19:37 |
*** vishalmanchanda has quit IRC | 19:53 | |
haleyb | johnsom: there was an issue with tempest on rocky, and part of the fix was to make things run in a 3.6 env, this might have been collateral damage. i haven't been able to keep up with it all | 20:05 |
*** born2bake has joined #openstack-lbaas | 20:09 | |
*** jamesdenton has quit IRC | 20:48 | |
*** TrevorV has quit IRC | 20:57 | |
johnsom | Yeah, me either | 21:34 |
*** armax has quit IRC | 21:40 | |
*** luksky has quit IRC | 21:41 | |
*** amuller has quit IRC | 21:46 | |
*** spatel has joined #openstack-lbaas | 22:02 | |
*** armax has joined #openstack-lbaas | 22:03 | |
*** spatel has quit IRC | 22:06 | |
johnsom | dulek FYI, talking with the requirements team, the train issue is a requirements UC issue with DIB. I have proposed UC updates based on the feedback. | 22:23 |
*** mithilar_ has quit IRC | 22:32 | |
haleyb | johnsom: question for you. i was trying to rebase by "remove six" patch today and don't know if something snuck-in in the recent jobboard patches | 22:53 |
haleyb | https://opendev.org/openstack/octavia/src/branch/master/octavia/controller/worker/v1/tasks/database_tasks.py#L934 and https://opendev.org/openstack/octavia/src/branch/master/octavia/controller/worker/v2/tasks/database_tasks.py#L1004 are slightly different | 22:53 |
haleyb | the v2 one has an encode("utf-8") but not the v1, and looking through the tree it's not consistent | 22:54 |
haleyb | the encode change was introduced in dad38f61f823f5121f32b24ebcbd8703128dd32e | 22:55 |
johnsom | I haven't been following the jobboard patches super closely, I think Adam and Carlos have been reviewing those. | 22:57 |
haleyb | johnsom: ack, thanks | 22:58 |
johnsom | I'm looking to see if I can see why that changed... | 22:58 |
haleyb | rm_work, cgoncalves ^^^ question above, i don't know if something snuck in that patch? | 22:58 |
johnsom | haleyb I guess this is a clue: https://review.opendev.org/#/c/668898/47/octavia/controller/worker/v2/tasks/cert_task.py | 22:59 |
johnsom | I'm still not really following the *why* part. The DB should be setup to store utf-8 | 23:01 |
haleyb | johnsom: oh, that would be it, i'll have to work backwards too see how removing six is breaking the unit tests for me. but it doesn't explain why the v1 code doesn't do it | 23:01 |
johnsom | Oh, it must be storing that in the new jobboard db... hmmm | 23:01 |
johnsom | haleyb Yeah, I think she is talking about the jobboard DB as in the legacy code, we don't store that data in mysql | 23:03 |
haleyb | legacy == v1 ? | 23:03 |
johnsom | haleyb Yeah, sorry, making up my own terms | 23:04 |
haleyb | as long as what's there is right, i can fix my rebase nightmare to deal with it :) | 23:05 |
johnsom | fernet usually takes bytes | 23:07 |
*** tkajinam has joined #openstack-lbaas | 23:09 | |
haleyb | the failure i am seeing is AttributeError: 'bytes' object has no attribute 'encode' :-/ | 23:10 |
haleyb | clear as mud | 23:10 |
haleyb | unless it's something i did in the six removal changing something to the wrong type | 23:11 |
johnsom | Yeah, so whatever "server_pem" was passed in is already in bytes | 23:11 |
johnsom | She probably added that encode as it was 'str' type | 23:13 |
johnsom | for python3 | 23:13 |
haleyb | johnsom: i think you've got something there, the unit tests were using get_six_compatible_value() which I removed, so this it probably my fault | 23:15 |
haleyb | johnsom: thanks for the help... when i initially rebased i removed two decode() calls by accident :-o | 23:31 |
johnsom | +1 | 23:31 |
*** mithilarun has joined #openstack-lbaas | 23:31 | |
openstackgerrit | Brian Haley proposed openstack/octavia master: Remove all usage of six library https://review.opendev.org/701290 | 23:31 |
*** mithilarun has quit IRC | 23:31 | |
*** mithilarun has joined #openstack-lbaas | 23:32 | |
*** born2bake has quit IRC | 23:50 | |
rm_work | would be good to get these in, as multi-AZ is very broken without them: https://review.opendev.org/#/q/topic:az-tweaks | 23:57 |
rm_work | you can hopefully understand why, just by looking at what they touch | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!