*** ramishra has quit IRC | 00:04 | |
*** nicolasbock has quit IRC | 00:28 | |
*** vishalmanchanda has joined #openstack-lbaas | 01:33 | |
*** ianychoi_ has joined #openstack-lbaas | 01:38 | |
*** ianychoi has quit IRC | 01:40 | |
*** yamamoto has joined #openstack-lbaas | 01:43 | |
*** sapd1 has joined #openstack-lbaas | 01:50 | |
*** sapd1_x has joined #openstack-lbaas | 01:52 | |
*** servagem has quit IRC | 01:58 | |
*** yamamoto has quit IRC | 02:16 | |
*** yamamoto has joined #openstack-lbaas | 02:17 | |
*** psachin has joined #openstack-lbaas | 02:31 | |
*** yamamoto has quit IRC | 03:00 | |
*** yamamoto has joined #openstack-lbaas | 03:01 | |
*** ramishra has joined #openstack-lbaas | 03:29 | |
rm_work | sorrison: it was on my list, but very far down it. If you can, that'd be great. Possibly also tho, still need to get the AZ tempest testing in better shape and merged... Need to actually have a different network set up, so we can prove the fixes I have up are necessary (rather than just a second AZ on all the same networks) | 04:06 |
---|---|---|
sorrison | OK cool, we can take care of the dashboard. We're just about to hire an ex horizon core :-) | 04:08 |
sorrison | and yeah forgot about tempest changes, will try tackle them next week | 04:09 |
*** yamamoto has quit IRC | 04:17 | |
*** yamamoto has joined #openstack-lbaas | 04:28 | |
dawzon | I'm having some trouble updating a listener over REST with curl. Getting 405 method not allowed back, is there something I'm missing? Here's my command: curl -X PUT -H "Content-Type: application/json" -H "X-Auth-Token: <token>" -d {"listener": {"tls_ciphers": "SHA384:ECDHE-RSA-AES128-GCM-SHA256"}} localhost/v2/lbaas/listeners/17a23f66-282b-4fe1-aeaa-0a779e2a3644 | 04:38 |
*** ramishra has quit IRC | 04:54 | |
*** ramishra has joined #openstack-lbaas | 05:01 | |
johnsom | Did your token expire? | 05:24 |
*** gcheresh has joined #openstack-lbaas | 05:51 | |
*** gcheresh has quit IRC | 06:00 | |
*** gcheresh has joined #openstack-lbaas | 06:19 | |
*** gcheresh has quit IRC | 06:25 | |
*** gcheresh has joined #openstack-lbaas | 06:46 | |
*** gcheresh has quit IRC | 07:31 | |
*** gcheresh has joined #openstack-lbaas | 07:33 | |
*** tesseract has joined #openstack-lbaas | 07:51 | |
*** maciejjozefczyk has joined #openstack-lbaas | 07:58 | |
*** threestrands has quit IRC | 08:01 | |
*** dawzon has quit IRC | 08:07 | |
*** lxkong has quit IRC | 08:08 | |
*** kevinz has quit IRC | 08:08 | |
*** luketollefson has quit IRC | 08:09 | |
*** guilhermesp has quit IRC | 08:09 | |
*** fyx has quit IRC | 08:09 | |
*** luketollefson has joined #openstack-lbaas | 08:09 | |
*** dawzon has joined #openstack-lbaas | 08:09 | |
*** lxkong has joined #openstack-lbaas | 08:10 | |
*** mnaser has quit IRC | 08:10 | |
*** kevinz has joined #openstack-lbaas | 08:11 | |
*** fyx has joined #openstack-lbaas | 08:11 | |
*** mnaser has joined #openstack-lbaas | 08:11 | |
*** rpittau|afk is now known as rpittau | 08:11 | |
*** guilhermesp has joined #openstack-lbaas | 08:11 | |
*** tkajinam has quit IRC | 08:12 | |
*** yamamoto has quit IRC | 08:16 | |
*** yamamoto has joined #openstack-lbaas | 08:30 | |
*** psachin has quit IRC | 09:01 | |
*** psachin has joined #openstack-lbaas | 09:02 | |
*** yamamoto has quit IRC | 09:04 | |
*** dawzon has quit IRC | 09:08 | |
*** dawzon has joined #openstack-lbaas | 09:09 | |
*** vishalmanchanda has quit IRC | 10:03 | |
*** vishalmanchanda has joined #openstack-lbaas | 10:22 | |
*** yamamoto has joined #openstack-lbaas | 10:32 | |
*** tesseract has quit IRC | 10:36 | |
openstackgerrit | Maciej Józefczyk proposed openstack/octavia-tempest-plugin master: Do not use requests.session in _wait_for_lb_functional() https://review.opendev.org/713826 | 10:38 |
openstackgerrit | Maciej Józefczyk proposed openstack/octavia-tempest-plugin master: Send HTTP request while testing L4 TCP Load Balancer https://review.opendev.org/713827 | 10:38 |
openstackgerrit | Maciej Józefczyk proposed openstack/octavia-tempest-plugin master: Do not test reuse connections for UDP https://review.opendev.org/713828 | 10:38 |
*** tesseract has joined #openstack-lbaas | 10:42 | |
*** tesseract has quit IRC | 10:46 | |
*** tesseract has joined #openstack-lbaas | 10:48 | |
*** yamamoto has quit IRC | 10:52 | |
*** ivve has joined #openstack-lbaas | 10:55 | |
openstackgerrit | Maciej Józefczyk proposed openstack/octavia-tempest-plugin master: Do not test reuse connections for UDP https://review.opendev.org/713828 | 10:56 |
*** tesseract-RH has joined #openstack-lbaas | 11:00 | |
*** tesseract-RH has left #openstack-lbaas | 11:02 | |
*** tesseract has quit IRC | 11:03 | |
*** sapd1_x has quit IRC | 11:04 | |
*** nicolasbock has joined #openstack-lbaas | 11:05 | |
*** sapd1 has quit IRC | 11:05 | |
*** pck has joined #openstack-lbaas | 11:16 | |
*** pck is now known as pckizer | 11:16 | |
*** rpittau is now known as rpittau|bbl | 11:30 | |
*** ccamposr__ has joined #openstack-lbaas | 11:31 | |
*** ccamposr has quit IRC | 11:33 | |
*** yamamoto has joined #openstack-lbaas | 11:56 | |
*** gcheresh has quit IRC | 12:18 | |
*** ccamposr__ has quit IRC | 12:26 | |
*** ccamposr__ has joined #openstack-lbaas | 12:26 | |
*** gcheresh has joined #openstack-lbaas | 12:33 | |
*** yamamoto has quit IRC | 12:47 | |
*** rpittau|bbl is now known as rpittau | 12:59 | |
*** gcheresh has quit IRC | 13:16 | |
*** nicolasbock has quit IRC | 13:18 | |
*** yamamoto has joined #openstack-lbaas | 13:19 | |
*** nicolasbock has joined #openstack-lbaas | 13:20 | |
*** yamamoto has quit IRC | 13:20 | |
*** psachin has quit IRC | 13:21 | |
openstackgerrit | Carlos Goncalves proposed openstack/octavia-tempest-plugin master: Fix skip check in active-standby test https://review.opendev.org/713874 | 13:25 |
openstackgerrit | Adam Harwell proposed openstack/octavia-tempest-plugin master: Remove old api_v1_enabled cruft from job defs https://review.opendev.org/713879 | 13:35 |
rm_work | not sure if someone had already done that, if so then whatev, just saw it and was faster to do it than look lol | 13:35 |
cgoncalves | may still be applicable to stable branches where v1 was supported | 13:37 |
rm_work | ahh hmm, let me check how those run | 13:40 |
rm_work | yeah you may be right, we may have to wait one more release | 13:41 |
rm_work | I think rocky was the last to contain v1 at all? | 13:41 |
cgoncalves | deprecated in rocky, removed in train | 13:43 |
cgoncalves | https://docs.openstack.org/releasenotes/octavia/rocky.html#deprecation-notes | 13:43 |
cgoncalves | https://docs.openstack.org/releasenotes/octavia/train.html#deprecation-notes | 13:43 |
rm_work | wait yeah seriously gah, just noticed it existed in stein | 13:43 |
rm_work | so yeah, gotta wait for stein to fall off too, ffff | 13:43 |
rm_work | oh wel | 13:44 |
cgoncalves | or move the setting to the -$branch jobs | 13:44 |
rm_work | ah, time for today's plug for https://review.opendev.org/#/q/topic:az-tweaks+status:open | 13:44 |
rm_work | ah yeah could do that | 13:44 |
*** yamamoto has joined #openstack-lbaas | 13:55 | |
openstackgerrit | Adam Harwell proposed openstack/octavia-tempest-plugin master: Remove old api_v1_enabled cruft from job defs https://review.opendev.org/713879 | 13:56 |
rm_work | fuck, rocky gates are broken for us because of MarkupSafe | 14:16 |
rm_work | >_< | 14:16 |
rm_work | 2020-03-19 13:55:05.431232 | controller | 2020-03-19 13:55:05.431 | Downloading MarkupSafe-1.0.tar.gz (14 kB) | 14:16 |
rm_work | 2020-03-19 13:55:05.680071 | controller | 2020-03-19 13:55:05.679 | ERROR: Command errored out with exit status 1: | 14:16 |
rm_work | 2020-03-19 13:55:05.680140 | controller | 2020-03-19 13:55:05.679 | command: /opt/amphora-agent-venv/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-feio8mx5/MarkupSafe/setup.py'"'"'; __file__='"'"'/tmp/pip-install-feio8mx5/MarkupSafe/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, | 14:16 |
rm_work | __file__, | 14:16 |
rm_work | '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-install-feio8mx5/MarkupSafe/pip-egg-info | 14:16 |
rm_work | been dealing with that elsewhere for like a week internally | 14:17 |
rm_work | didn't realize it would hit us too T_T | 14:17 |
rm_work | I honestly don't know how to fix this besides updating to MarkupSafe==1.1, which is capped at 1.0 in requirements upper-constraints in rocky | 14:17 |
rm_work | downstream we just patch it and ignore the u-c... but we can't do that in the gates... so... ??? | 14:18 |
rm_work | will ask infra | 14:18 |
*** yamamoto has quit IRC | 14:24 | |
openstackgerrit | Merged openstack/octavia master: Remove all usage of six library https://review.opendev.org/701290 | 14:27 |
openstackgerrit | Merged openstack/octavia master: Remove the barbican "Grant access" from cookbook https://review.opendev.org/713533 | 14:27 |
*** TrevorV has joined #openstack-lbaas | 14:30 | |
rm_work | see https://zuul.opendev.org/t/openstack/build/b75e42523e6c41e5a6e97c722fbb0ecd for an example of what's broken | 14:31 |
rm_work | there's only two options AFAIK: upgrade to MarkupSafe>=1.1, or somehow pin setuptools to an older version... I don't honestly know if it is possible to do either... | 14:32 |
johnsom | Can't we pin in our requirements.txt file? | 14:34 |
johnsom | I would ask in requirements myself | 14:34 |
rm_work | i did previously and posted a patch | 14:37 |
rm_work | but discussion was not very deep | 14:37 |
rm_work | we can't pin setuptools easily as it's pre-installed in the virtualenv | 14:38 |
rm_work | it's not something we pip-install | 14:38 |
rm_work | that's why it suddenly broke as far back as rocky | 14:38 |
johnsom | Ah, yeah, that is a problem | 14:38 |
rm_work | i'm honestly a little stumped here, without resorting to some EVIL trickery | 14:38 |
rm_work | like bumping the version and just... dropping the requirements job | 14:39 |
johnsom | What did the requirements folks say? | 14:40 |
rm_work | "no" | 14:40 |
rm_work | lemmie see if i can find the exact scrollback | 14:40 |
rm_work | oh huh | 14:41 |
rm_work | maybe they changed their minds | 14:41 |
* rm_work looks | 14:41 | |
rm_work | see reqs channel | 14:43 |
*** gcheresh has joined #openstack-lbaas | 14:48 | |
rm_work | johnsom: I may propose dropping support for queens/rocky going forward as the CI is essentially no longer fixable | 15:09 |
rm_work | cgoncalves: ^^ | 15:09 |
*** gcheresh has quit IRC | 15:20 | |
*** gcheresh has joined #openstack-lbaas | 15:28 | |
*** gcheresh has quit IRC | 15:35 | |
cgoncalves | rm_work, you mean dropping the queens and rocky jobs in octavia-tempest-plugin CI? | 15:37 |
*** Trevor_V has joined #openstack-lbaas | 15:37 | |
rm_work | yes, and declaring queens/rocky unmaintained | 15:37 |
rm_work | I am making a patch to test one alternative option tho | 15:37 |
johnsom | Personally I am itching to drop queens. It's still on xenial, etc.... It is just getting vary dated at this point. | 15:39 |
johnsom | I think babysitting FIVE releases is..... a bit much | 15:39 |
rm_work | yes | 15:39 |
rm_work | we can apparently just... decide to drop them whenever | 15:39 |
rm_work | there's no requirement to continue supporting them | 15:40 |
rm_work | and it especially makes sense in case of unfixable CI, like... this | 15:40 |
johnsom | That said, I am still on the hook for queens downstream, but that is a different issue/different hat to wear | 15:40 |
*** TrevorV has quit IRC | 15:40 | |
*** vishalmanchanda has quit IRC | 15:43 | |
johnsom | Yeah, so how do we declare the "Unmaintained" or EOL status for that? This was a question I raised when they made these new categories, but got no clear answer. | 15:43 |
johnsom | Do we just drop the gate jobs and *ignore* them? | 15:43 |
johnsom | I guess EOL gets a tag | 15:43 |
cgoncalves | not sure I fully follow. the changes would only impact the octavia-tempest-plugin CI, yes? | 15:44 |
cgoncalves | if so, I agree with stop gating on queens and rocky jobs | 15:44 |
rm_work | cgoncalves: err, also the gate jobs would be unrunable on the stable branches | 15:45 |
rm_work | so no changes could be merged anyway | 15:45 |
rm_work | sorry, "would be" -> "are already" | 15:45 |
johnsom | I think we are discussing a bigger change, where patches to queens would not trigger any job any longer. | 15:45 |
cgoncalves | yeah but if someone wants to get something proposed and merged can still do | 15:45 |
rm_work | no, they can't, because if you propose a change to rocky now it won't merge | 15:45 |
cgoncalves | if CI is broken, there is no guarantees from the octavia project team we would fix them | 15:45 |
rm_work | err, won't pass gates | 15:45 |
rm_work | so how would we merge anything? | 15:45 |
rm_work | not going to merge changes especially to a stable release without any testing | 15:46 |
johnsom | Right, the gate jobs would block everything, so it is the same as EOL | 15:46 |
rm_work | ah damn, looks like requirements decided it's fine to merge the bump | 15:47 |
rm_work | err i mean, ah, good | 15:47 |
johnsom | lol | 15:47 |
cgoncalves | merging queens/rocky backport patches without any CI isn't that good either. I wish we could override Zuul's verification vote in such cases | 15:47 |
johnsom | Well, I still propose we EOL queens | 15:47 |
rm_work | yeah | 15:47 |
rm_work | I second that | 15:47 |
johnsom | cgoncalves we *could* but, does that make sense or bring value? | 15:48 |
*** spatel has joined #openstack-lbaas | 15:48 | |
rm_work | https://review.opendev.org/#/q/status:open+project:openstack/octavia+branch:stable/queens | 15:48 |
rm_work | so these are the only things open on queens | 15:49 |
rm_work | and I don't see us merging any of them | 15:49 |
johnsom | Well, one less now, abandoned the fix for queens that I could not get working | 15:49 |
rm_work | I'm going to make an ML thread for ya'll to vote on? or can we just decide it here | 15:49 |
rm_work | we have core quorum? | 15:49 |
cgoncalves | johnsom, job results could be first checked and/or patches tested manually and if looks good we override verification | 15:49 |
johnsom | We do, but I think a ML thread is the best answer for this. It impacts others beyond the core team | 15:50 |
rm_work | k | 15:50 |
cgoncalves | having just said this, I don't want to say I'd manually verify every/any queens patch | 15:50 |
johnsom | cgoncalves With all the spare time the core team has.... | 15:50 |
johnsom | Right | 15:50 |
johnsom | lol | 15:50 |
cgoncalves | right | 15:50 |
rm_work | yeah I am just *not* going to do that, so | 15:50 |
johnsom | <see above nag from rm_work about patch reviews> | 15:50 |
*** spatel has quit IRC | 15:51 | |
* johnsom thanks rm_work for reminding us | 15:51 | |
rm_work | Yeah, I will continue to helpfully remind you every day until those merge :D | 15:51 |
johnsom | As you should | 15:51 |
rm_work | just sent a ML email | 15:55 |
rm_work | should show up shortly | 15:56 |
cgoncalves | IIUC, many of the issues we see in queens and rocky are due to python3 and other flaky stuff where typically we are the first ones to notice (canary!) because blocks us primarily in octavia-tempest-plugin but other projects should also be impacted by. I would say let's start with removing queens and rocky jobs in tempest plugin first | 15:56 |
cgoncalves | whenever someone sees a problem with a queens/rocky backport, it might be that we just need to wait (and pray) a bit for someone to fix tempest/dib/devstack/whatever | 15:57 |
cgoncalves | if we see the issue persists for an unreasonable time, we could consider disabling queens and rocky CI | 15:58 |
cgoncalves | my 2 cents | 15:58 |
rm_work | well, i am not in favor of dropping testing on tempest repo and still having those tests run on the stable branches | 15:59 |
rm_work | that's just asking for trouble | 15:59 |
johnsom | +1 | 16:00 |
johnsom | I don't like "half-EOL" things. I just leads to confusion and trouble IMO. | 16:01 |
*** ccamposr has joined #openstack-lbaas | 16:01 | |
johnsom | I would rather lean towards we do or we don't. | 16:01 |
cgoncalves | code changes in octavia-tempest-plugin does not impact queens/rocky octavia code. it might uncover issues but not negatively impact | 16:01 |
cgoncalves | I'm just worried not having CI coverage would lead people to merge whatever without any checking | 16:02 |
johnsom | But code changes in tempest-plugin impact the ability to test queens/rocky at all. | 16:02 |
cgoncalves | or cores being blamed for holding backport patches | 16:02 |
*** ccamposr__ has quit IRC | 16:03 | |
johnsom | Yeah, so, if we declare EOL, we don't have that problem. | 16:03 |
cgoncalves | if EOL we rather just stop merging any code in those branches | 16:04 |
johnsom | I mean really, in a way we are already through the 6 month "gates don't work" window for queens. That interface issue has not been fixed in that time window | 16:04 |
cgoncalves | doesn't that go against the branches in extended maintenance (queens, rocky)? | 16:04 |
johnsom | To quote the EOL description: "After a project/branch exceeds the time allocation as Unmaintained, or a team decides to explicitly end support for a branch, it will become End of Life." | 16:04 |
cgoncalves | ok. I was looking for that now. thanks | 16:05 |
cgoncalves | https://docs.openstack.org/project-team-guide/stable-branches.html#end-of-life | 16:05 |
johnsom | https://docs.openstack.org/project-team-guide/stable-branches.html#end-of-life | 16:05 |
cgoncalves | "and the branch deleted." | 16:06 |
johnsom | I just feel we give people a false sense of "support" when the reality is that it probably should be EOL. | 16:06 |
johnsom | Yep | 16:06 |
cgoncalves | ok. upstream hat on, I am supportive of this | 16:06 |
johnsom | I get that downstream it makes things... trickier/harder. But it's going to happen at some point anyway. | 16:07 |
cgoncalves | agreed. I was also considering the pros/cons upstream | 16:09 |
*** maciejjozefczyk has quit IRC | 16:10 | |
cgoncalves | rm_work, are you expecting an email reply to the email you sent to -discuss? | 16:10 |
rm_work | yes | 16:15 |
rm_work | +1's from cores please to show quorum | 16:15 |
rm_work | and I'll wait a week to see if we get any other feedback i guess | 16:15 |
rm_work | I suppose I could mandate but trying to be a nice PTL :D | 16:16 |
johnsom | rm_work FYI, there was a discussion of the state of releases in the release team meeting just now. They were wondering why the Octavia ocata jobs were not working. lol. I think we really need to move forward with a strong EOL statement and getting the branches dropped. | 17:00 |
*** gcheresh has joined #openstack-lbaas | 17:19 | |
*** gcheresh has quit IRC | 17:58 | |
rm_work | ROFL wat | 17:59 |
rm_work | i thought we did EOL everything prior to queens | 17:59 |
*** rpittau is now known as rpittau|afk | 18:06 | |
openstackgerrit | Michael Johnson proposed openstack/octavia master: Add oslo middleware healthcheck to Octavia API https://review.opendev.org/711127 | 18:24 |
johnsom | Me too | 18:24 |
johnsom | Rebased, but had conflicts, so we get to vote again.... | 18:24 |
dawzon | johnsom Still getting the 405 it with a brand new token (taken from the "id" field from openstack token issue, right?). The command I posted yesterday was missing single quotes around the JSON but that wasn't the problem. Is the REST API hosted on a different port? | 18:41 |
johnsom | dawzon Use "openstack endpoint list" to see the URL to use | 18:42 |
dawzon | Ah, that was it. The URL that gave for octavia is different than the one in the curl example file. Is that just because I'm running DevStack instead of a regular deployment? | 18:45 |
johnsom | Yeah, the docs try to be "generic". There is a discussion of endpoints at the top of the API doc to clarify. | 18:46 |
johnsom | Some deploy with ports, some with just URL paths, etc. | 18:46 |
*** maciejjozefczyk has joined #openstack-lbaas | 19:02 | |
*** maciejjozefczyk has quit IRC | 19:40 | |
*** maciejjozefczyk has joined #openstack-lbaas | 19:41 | |
*** Trevor_V has quit IRC | 19:41 | |
*** gcheresh has joined #openstack-lbaas | 19:41 | |
*** maciejjozefczyk has quit IRC | 20:06 | |
*** TrevorV has joined #openstack-lbaas | 20:12 | |
*** TrevorV has quit IRC | 20:18 | |
*** gcheresh has quit IRC | 20:23 | |
*** ivve has quit IRC | 21:08 | |
*** zigo has quit IRC | 22:22 | |
*** zigo has joined #openstack-lbaas | 22:31 | |
openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: Add skip_if_not_implemented to the service client https://review.opendev.org/714003 | 22:42 |
openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: WIP: Adjust scenario tests for NotImplemented skip https://review.opendev.org/714004 | 23:00 |
openstackgerrit | Michael Johnson proposed openstack/octavia-tempest-plugin master: WIP: Adjust scenario tests for NotImplemented skip https://review.opendev.org/714004 | 23:02 |
*** spatel has joined #openstack-lbaas | 23:42 | |
*** spatel has quit IRC | 23:46 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!