| opendevreview | Boxiang Zhu proposed openstack/kolla-ansible master: skyline-apiserver support masakari and designate https://review.opendev.org/c/openstack/kolla-ansible/+/898310 | 03:09 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: octavia-certificates: fix missing collection for ansible-core https://review.opendev.org/c/openstack/kolla-ansible/+/900749 | 09:07 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: octavia-certificates: fix missing collection for ansible-core https://review.opendev.org/c/openstack/kolla-ansible/+/900749 | 09:25 |
| opendevreview | Grzegorz Koper proposed openstack/kayobe master: Follow up to: Change-Id: Ia3550644e5fc3e83f792e46d42b4c35d5eec4705 https://review.opendev.org/c/openstack/kayobe/+/900753 | 09:37 |
| opendevreview | Grzegorz Koper proposed openstack/kayobe master: Following up: Change-Id: Ia3550644e5fc3e83f792e46d42b4c35d5eec4705 https://review.opendev.org/c/openstack/kayobe/+/900753 | 09:37 |
| opendevreview | Grzegorz Koper proposed openstack/kayobe master: New openstack.cloud changed return output https://review.opendev.org/c/openstack/kayobe/+/900753 | 10:04 |
| opendevreview | Grzegorz Koper proposed openstack/kayobe master: ipa-images: use changed openstack.cloud output https://review.opendev.org/c/openstack/kayobe/+/900753 | 10:07 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: octavia-certificates: fix missing collection for ansible-core https://review.opendev.org/c/openstack/kolla-ansible/+/900749 | 10:11 |
| kevko | \o/ | 11:25 |
| kevko | anyone ? https://review.opendev.org/c/openstack/kolla-ansible/+/899895 | 11:25 |
| SvenKieske | you mean anyone with +2? ;) | 11:42 |
| opendevreview | Pierre Riteau proposed openstack/kolla master: Add missing newline at end of file https://review.opendev.org/c/openstack/kolla/+/900792 | 12:42 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: octavia-certificates: fix missing collection for ansible-core https://review.opendev.org/c/openstack/kolla-ansible/+/900749 | 13:00 |
| mnasiadka | frickler, kevko: https://review.opendev.org/c/openstack/kolla-ansible/+/900749 - seems I didn't check octavia CI job in ansible-core patch ;-) | 13:02 |
| mnasiadka | (the last patchset is just commit message change) | 13:02 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Fix tenks for newer libvirt https://review.opendev.org/c/openstack/kolla-ansible/+/900794 | 13:18 |
| mnasiadka | jangutter: ^^ this might fix the Ironic CI | 13:18 |
| jangutter | mnasiadka: and there might be another unrelated issue with the container startup sequence. (restart policy = no means ironic-inspector races with ironic scheduler) | 13:20 |
| opendevreview | Martin Hiner proposed openstack/kolla-ansible master: Rename kolla_docker to kolla_container https://review.opendev.org/c/openstack/kolla-ansible/+/900412 | 13:32 |
| hrw | SvenKieske: I think that those without +2 are also welcomed to reviewing patches | 14:02 |
| SvenKieske | hrw: sure, but this specific one already had some +1 at least :) | 14:03 |
| mnasiadka | jangutter: that's another thing that might be solved with some patches I raised long time ago and we didn't merge them :) | 14:31 |
| mnasiadka | jangutter: but one thing at a time | 14:31 |
| jangutter | agreed | 14:31 |
| kevko | hmm, have anyone upgrade octavia from wallaby to xena ? :D | 14:36 |
| *** ozzzo1 is now known as ozzzo | 14:37 | |
| kevko | this is definitely bug :( | 14:39 |
| kevko | https://paste.openstack.org/show/biNjNmNznd1BUYYnKBb7/ | 14:39 |
| kevko | how is it possible that nobody was affected ? | 14:40 |
| kevko | k8s openstack provider :/ ? | 14:40 |
| kevko | provider want's to deploy LB with octavia provider ...but they changed default from v1 to v2 (and someone bluntly wrote that it was due to backward compatibility) | 14:42 |
| mnasiadka | kevko: welcome to hell, but isn't kubernetes ephemeral? recreate the LBs :D | 14:52 |
| opendevreview | Merged openstack/kolla-ansible master: Add fluent config for letsencrypt https://review.opendev.org/c/openstack/kolla-ansible/+/899895 | 15:24 |
| mnasiadka | jangutter: you're right, we're back to the ordering issue - but I'll refresh my old patches once we get this one merged | 15:37 |
| jangutter | as it's non-voting the patches don't _need_ to be squashed. | 15:37 |
| kevko | mnasiadka: hmm, 5 hours i am debugging the issue ... | 15:38 |
| kevko | mnasiadka: some advice ? :D | 15:38 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 15:38 |
| kevko | mnasiadka: 1000 LBs ? :P | 15:38 |
| mnasiadka | kevko: damn k8s :) | 15:38 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 15:40 |
| kevko | mnasiadka: ^^ thanks for LE fluent merge | 15:41 |
| mnasiadka | np | 15:41 |
| kevko | frickler: also thanks | 15:41 |
| kevko | have you tried already :) ? | 15:41 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 15:42 |
| mnasiadka | kevko: https://review.opendev.org/c/openstack/kolla-ansible/+/900794 - if you have a minute ;) | 15:45 |
| kevko | done | 15:45 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Introduce oneshot docker_restart_policy https://review.opendev.org/c/openstack/kolla-ansible/+/881707 | 15:46 |
| kevko | mnasiadka: and can you check this trivial patch please ? https://review.opendev.org/c/openstack/kolla-ansible/+/896362 | 15:47 |
| mnasiadka | kevko: I think me and frickler commented on this - and it hasn't been updated since? | 15:48 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Make designate bind9 cmdline configurable https://review.opendev.org/c/openstack/kolla-ansible/+/896362 | 16:21 |
| kevko | mnasiadka: done | 16:21 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: redis: Use dedicated repository https://review.opendev.org/c/openstack/kolla/+/900809 | 16:52 |
| opendevreview | Merged openstack/kolla-ansible master: octavia-certificates: fix missing collection for ansible-core https://review.opendev.org/c/openstack/kolla-ansible/+/900749 | 17:15 |
| opendevreview | Merged openstack/kolla-ansible master: CI: Fix tenks for newer libvirt https://review.opendev.org/c/openstack/kolla-ansible/+/900794 | 17:15 |
| kevko | mnasiadka: for some reason octavia is not working after upgrade | 17:27 |
| kevko | pinging ..but after while ssh and 9443 just broken | 17:27 |
| kevko | any advice ? :/ | 17:27 |
| opendevreview | Merged openstack/kolla-ansible master: CI: Add redis to octavia scenario in setup_gate https://review.opendev.org/c/openstack/kolla-ansible/+/900536 | 18:04 |
| simondodsley | can i get reviews of https://review.opendev.org/c/openstack/kolla-ansible/+/879844 and https://review.opendev.org/c/openstack/kolla-ansible/+/879846. They have been hanging around for quite a few months now. | 22:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!