opendevreview | Takashi Kajinami proposed openstack/octavia master: Regenerate octavia.conf https://review.opendev.org/c/openstack/octavia/+/804175 | 00:13 |
---|---|---|
opendevreview | Takashi Kajinami proposed openstack/octavia master: Add conf file for oslo-config-generator https://review.opendev.org/c/openstack/octavia/+/804179 | 00:32 |
opendevreview | Takashi Kajinami proposed openstack/octavia master: Add conf file for oslo-config-generator https://review.opendev.org/c/openstack/octavia/+/804179 | 00:35 |
opendevreview | Takashi Kajinami proposed openstack/octavia master: Add conf file for oslo-config-generator https://review.opendev.org/c/openstack/octavia/+/804179 | 00:36 |
opendevreview | Takashi Kajinami proposed openstack/octavia master: Add missing [cinder] parameters https://review.opendev.org/c/openstack/octavia/+/804180 | 00:43 |
opendevreview | Takashi Kajinami proposed openstack/octavia master: Add conf file for oslo-config-generator https://review.opendev.org/c/openstack/octavia/+/804179 | 00:53 |
opendevreview | Takashi Kajinami proposed openstack/octavia master: Add conf file for oslo-config-generator https://review.opendev.org/c/openstack/octavia/+/804179 | 05:52 |
opendevreview | Takashi Kajinami proposed openstack/octavia master: Regenerate octavia.conf https://review.opendev.org/c/openstack/octavia/+/804175 | 05:56 |
opendevreview | Takashi Kajinami proposed openstack/octavia master: Regenerate octavia.conf https://review.opendev.org/c/openstack/octavia/+/804175 | 06:12 |
opendevreview | Merged openstack/octavia master: Add caps for pip-extra-reqs/pip-missing-reqs https://review.opendev.org/c/openstack/octavia/+/803461 | 09:47 |
opendevreview | Gregory Thiemonge proposed openstack/octavia stable/wallaby: Add caps for pip-extra-reqs/pip-missing-reqs https://review.opendev.org/c/openstack/octavia/+/804159 | 09:49 |
opendevreview | Gregory Thiemonge proposed openstack/octavia stable/victoria: Add caps for pip-extra-reqs/pip-missing-reqs https://review.opendev.org/c/openstack/octavia/+/804160 | 09:50 |
opendevreview | Takashi Kajinami proposed openstack/octavia master: Fix callback method interface to remove warning by tenacity https://review.opendev.org/c/openstack/octavia/+/803342 | 09:59 |
opendevreview | Takashi Kajinami proposed openstack/octavia master: Remove redundant override of project_id in RequestContext https://review.opendev.org/c/openstack/octavia/+/801860 | 10:00 |
opendevreview | Takashi Kajinami proposed openstack/octavia master: Exclude invalid TLS version https://review.opendev.org/c/openstack/octavia/+/804270 | 15:52 |
gthiemonge | #startmeeting Octavia | 16:00 |
opendevmeet | Meeting started Wed Aug 11 16:00:20 2021 UTC and is due to finish in 60 minutes. The chair is gthiemonge. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
opendevmeet | The meeting name has been set to 'octavia' | 16:00 |
gthiemonge | Hi! | 16:00 |
johnsom | o/ | 16:00 |
gthiemonge | #topic Announcements | 16:02 |
gthiemonge | Gates are/were broken after a pip-check-reqs update | 16:02 |
gthiemonge | master is now fixed | 16:02 |
gthiemonge | but wallaby and victoria are still affect | 16:03 |
johnsom | I just reviewed some of the stable branch patches this morning | 16:03 |
gthiemonge | there's a fix to cap pip and pip-check-reqs: | 16:03 |
gthiemonge | #link https://review.opendev.org/q/I6ae2740557f0e62b6805e5dadd025e7bf44ec57d | 16:03 |
gthiemonge | perhaps another core reviewer can take a look | 16:04 |
gthiemonge | cgoncalves: rm_work: ^ | 16:04 |
gthiemonge | Next week is the deadline for the final release for non-client libraries for Xena | 16:05 |
gthiemonge | johnsom: i guess this concerns octavia-lib, right? | 16:06 |
johnsom | Correct | 16:06 |
gthiemonge | ok, there are 3 reviews that are waiting for a CR+2, I'll review them | 16:07 |
gthiemonge | Any other announcements? | 16:07 |
johnsom | There was an e-mail about elections coming up | 16:08 |
rm_work | Kk | 16:08 |
gthiemonge | elections?? oh | 16:08 |
johnsom | #link http://lists.openstack.org/pipermail/openstack-discuss/2021-August/024093.html | 16:08 |
gthiemonge | thanks johnsom | 16:10 |
gthiemonge | #topic Brief progress reports / bugs needing review | 16:11 |
gthiemonge | I was on PTO but I'd like to highlight some reviews that are still open: | 16:11 |
gthiemonge | Add generic network interface management in the amphora | 16:11 |
gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/761195 | 16:11 |
gthiemonge | Switch centos-8 jobs to centos-8-stream | 16:12 |
gthiemonge | #link https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/698450 | 16:12 |
gthiemonge | ^ it fixes centos-8 that has been broken for ... years? | 16:12 |
gthiemonge | Alias change amphorav2 -> amphora | 16:12 |
gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/740432 | 16:12 |
gthiemonge | this one is important, we discussed it 3 weeks ago | 16:13 |
gthiemonge | we agreed that amphorav2 is still a goal for octavia in xena | 16:13 |
johnsom | Yes, this needs to happen. | 16:14 |
johnsom | Looks like I lost track of that one | 16:14 |
gthiemonge | this is amphorav2 _without_ persistence | 16:14 |
gthiemonge | #topic Open Discussion | 16:19 |
gthiemonge | tkajinam has been working on the octavia.conf generation (instead of maintaining it manually) | 16:19 |
gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/804175 | 16:19 |
johnsom | Yeah, I have concerns about that. We didn't do this on purpose as the generated conf file was inaccurate and didn't allow for better comments, etc. | 16:20 |
gthiemonge | I'm not against this change if we manage to not lose information with this commit | 16:21 |
gthiemonge | so I think we need to carefully review the patch to find if anything is missing | 16:22 |
gthiemonge | Michael already pointed an error in a generated field | 16:23 |
gthiemonge | any other opinions? | 16:24 |
johnsom | Yeah, I have not yet gone through it line-by-line, but that will be what is required. | 16:24 |
gthiemonge | ok | 16:26 |
gthiemonge | of course I can help if we need to fix/extend some strings | 16:27 |
johnsom | It's changes like this that... concern me | 16:28 |
johnsom | https://review.opendev.org/c/openstack/octavia/+/804175/3/octavia/common/config.py#531 | 16:28 |
johnsom | It will just take time to walk through. | 16:30 |
gthiemonge | ack | 16:33 |
gthiemonge | Any other topics today? | 16:33 |
gthiemonge | Ok guys! Thanks everyone! | 16:35 |
gthiemonge | #endmeeting | 16:35 |
opendevmeet | Meeting ended Wed Aug 11 16:35:31 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:35 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2021/octavia.2021-08-11-16.00.html | 16:35 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2021/octavia.2021-08-11-16.00.txt | 16:35 |
opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2021/octavia.2021-08-11-16.00.log.html | 16:35 |
johnsom | Thanks Greg | 16:35 |
lxkong | Hi all, has anyone seen this error before, `2021-08-12 10:34:16.707 16 ERROR octavia.controller.healthmanager.health_drivers.update_db [-] Health update for amphora 7c002dc5-ab9a-4c09-ae05-b36bdaef21d7 encountered error (pymysql.err.InternalError) (1205, 'Lock wait timeout exceeded; try restarting transaction')` | 23:54 |
lxkong | octavia: stable/victoria | 23:54 |
johnsom | Hi lxkong, yes, your DB has turned read only most likely | 23:55 |
lxkong | thanks johnsom. But none of our opeartors were touching db at that moment :-( | 23:58 |
johnsom | Things like backups being run or load balancers pointing to the wrong galera cluster instance can cause a read-only situation. | 23:59 |
johnsom | Is it rare or a lot of log messages? | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!