| *** tosky has quit IRC | 00:05 | |
| *** pmannidi has quit IRC | 00:11 | |
| *** pmannidi has joined #tripleo | 00:17 | |
| *** zzzeek has quit IRC | 00:30 | |
| *** zzzeek has joined #tripleo | 00:32 | |
| openstackgerrit | Merged openstack/tripleo-ansible master: Tune-up plan create when running swift-less https://review.opendev.org/c/openstack/tripleo-ansible/+/757648 | 01:10 |
|---|---|---|
| *** rlandy has quit IRC | 01:16 | |
| *** rcernin has joined #tripleo | 01:17 | |
| *** holser has quit IRC | 01:27 | |
| *** holser has joined #tripleo | 01:35 | |
| openstackgerrit | Merged openstack/paunch stable/ussuri: Actually fail if a stop or remove call fails https://review.opendev.org/c/openstack/paunch/+/766779 | 01:54 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Cinder: Do not manage default volume type by puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/768799 | 02:01 |
| openstackgerrit | Takashi Kajinami proposed openstack/paunch stable/train: Actually fail if a stop or remove call fails https://review.opendev.org/c/openstack/paunch/+/769186 | 02:04 |
| *** pleimer has joined #tripleo | 02:19 | |
| *** holser has quit IRC | 02:25 | |
| *** rcernin has quit IRC | 02:26 | |
| *** zzzeek has quit IRC | 02:30 | |
| *** zzzeek has joined #tripleo | 02:31 | |
| *** skramaja_ has joined #tripleo | 02:50 | |
| *** skramaja has quit IRC | 02:50 | |
| *** baha has quit IRC | 02:52 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/victoria: Remove Luna HSM clients on scaledown https://review.opendev.org/c/openstack/tripleo-heat-templates/+/767930 | 02:52 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/ussuri: Remove Luna HSM clients on scaledown https://review.opendev.org/c/openstack/tripleo-heat-templates/+/767931 | 02:52 |
| openstackgerrit | Merged openstack/puppet-tripleo master: Adding key_size option on the certmonger_certificate function https://review.opendev.org/c/openstack/puppet-tripleo/+/757142 | 02:52 |
| *** tbarron has quit IRC | 02:52 | |
| openstackgerrit | Merged openstack/python-tripleoclient stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/c/openstack/python-tripleoclient/+/759681 | 02:52 |
| *** artom has quit IRC | 02:52 | |
| *** artom has joined #tripleo | 02:53 | |
| *** baha has joined #tripleo | 02:53 | |
| *** leifmadsen has quit IRC | 02:53 | |
| *** leifmadsen has joined #tripleo | 02:57 | |
| *** rcernin has joined #tripleo | 02:58 | |
| *** skramaja_ has quit IRC | 02:59 | |
| *** rcernin has quit IRC | 03:10 | |
| *** rh-jelabarre has quit IRC | 03:15 | |
| openstackgerrit | Alan Bishop proposed openstack/tripleo-heat-templates stable/train: Ensure cinder LVM volumes work after system restart https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769239 | 03:16 |
| openstackgerrit | Alan Bishop proposed openstack/tripleo-heat-templates stable/train: Define a new CinderVolumeEdge service https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769240 | 03:19 |
| *** pleimer has quit IRC | 03:24 | |
| openstackgerrit | Merged openstack/python-tripleoclient stable/ussuri: Lower ansible poll interval time https://review.opendev.org/c/openstack/python-tripleoclient/+/751877 | 03:29 |
| *** psachin has joined #tripleo | 03:33 | |
| *** skramaja has joined #tripleo | 03:33 | |
| *** zzzeek has quit IRC | 03:43 | |
| *** zzzeek has joined #tripleo | 03:45 | |
| *** zzzeek has quit IRC | 03:53 | |
| *** zzzeek has joined #tripleo | 03:55 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-tripleo master: Remove useless parameters for keystone resource management https://review.opendev.org/c/openstack/puppet-tripleo/+/768380 | 03:56 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-tripleo master: Remove useless parameters for keystone resource management https://review.opendev.org/c/openstack/puppet-tripleo/+/768380 | 03:57 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove useless hieradata for keystone resource management by puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/768378 | 04:03 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Cinder: Clean up volume type management by puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/768799 | 04:08 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Cinder: Clean up volume type management by puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/768799 | 04:09 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Cinder: Remove useless container puppet task https://review.opendev.org/c/openstack/tripleo-heat-templates/+/768799 | 04:14 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove useless hieradata for keystone resource management by puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/768378 | 04:14 |
| *** zzzeek has quit IRC | 04:23 | |
| *** pmannidi has quit IRC | 04:24 | |
| *** zzzeek has joined #tripleo | 04:25 | |
| *** ykarel has joined #tripleo | 04:26 | |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove useless hieradata for keystone resource management by puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/768378 | 04:27 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Cinder: Clean up volume type management by puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/768799 | 04:29 |
| *** jmasud has quit IRC | 04:50 | |
| openstackgerrit | Rabi Mishra proposed openstack/python-tripleoclient master: Set DeployIndentifier early in tripleoclient-parameters.yaml https://review.opendev.org/c/openstack/python-tripleoclient/+/766897 | 04:55 |
| openstackgerrit | Rabi Mishra proposed openstack/python-tripleoclient master: Use heat server side env merging https://review.opendev.org/c/openstack/python-tripleoclient/+/765808 | 04:55 |
| *** ratailor has joined #tripleo | 04:57 | |
| openstackgerrit | Rabi Mishra proposed openstack/python-tripleoclient master: Use heat server side env merging https://review.opendev.org/c/openstack/python-tripleoclient/+/765808 | 05:12 |
| openstackgerrit | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: [DNM] Disable swift from undercloud https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/769245 | 05:23 |
| *** jmasud has joined #tripleo | 05:23 | |
| *** udesale has joined #tripleo | 05:25 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #tripleo | 05:33 | |
| *** zzzeek has quit IRC | 06:31 | |
| *** zzzeek has joined #tripleo | 06:33 | |
| *** marios has joined #tripleo | 06:35 | |
| *** hakhande has joined #tripleo | 06:35 | |
| *** pmannidi has joined #tripleo | 06:39 | |
| *** pmannidi has quit IRC | 06:40 | |
| *** pmannidi has joined #tripleo | 06:42 | |
| *** pojadhav is now known as pojadhav|rover | 06:44 | |
| *** zzzeek has quit IRC | 06:46 | |
| *** zzzeek has joined #tripleo | 06:47 | |
| *** jmasud has quit IRC | 06:51 | |
| *** lmiccini has joined #tripleo | 07:00 | |
| openstackgerrit | Jaganathan Palanisamy proposed openstack/tripleo-ansible master: NFV derive parameters molecule tests https://review.opendev.org/c/openstack/tripleo-ansible/+/754630 | 07:04 |
| *** zzzeek has quit IRC | 07:17 | |
| *** zzzeek has joined #tripleo | 07:19 | |
| *** sanjayu_ has joined #tripleo | 07:24 | |
| *** abdysn has joined #tripleo | 07:26 | |
| *** udesale_ has joined #tripleo | 07:33 | |
| *** rpittau|afk is now known as rpittau | 07:33 | |
| *** udesale has quit IRC | 07:33 | |
| *** Abdallahyas has joined #tripleo | 07:36 | |
| *** abdysn has quit IRC | 07:40 | |
| *** slaweq has joined #tripleo | 07:42 | |
| *** zzzeek has quit IRC | 07:45 | |
| *** zzzeek has joined #tripleo | 07:45 | |
| openstackgerrit | MaAoyu proposed openstack/python-tripleoclient master: Use unittest.mock instead of mock https://review.opendev.org/c/openstack/python-tripleoclient/+/768857 | 07:47 |
| *** jcapitao has joined #tripleo | 07:47 | |
| *** zzzeek has quit IRC | 07:50 | |
| *** zzzeek has joined #tripleo | 07:53 | |
| openstackgerrit | daniel.pawlik proposed openstack/diskimage-builder master: Replace Fedora 31 functional tests with Fedora 33 https://review.opendev.org/c/openstack/diskimage-builder/+/768092 | 07:57 |
| *** abdysn has joined #tripleo | 08:03 | |
| Tengu | oh. I think I have a topic for the mtg today \o/ | 08:06 |
| *** Abdallahyas has quit IRC | 08:06 | |
| Tengu | marios: you'll be happy :D | 08:06 |
| marios | o/ Tengu :) | 08:11 |
| *** zzzeek has quit IRC | 08:11 | |
| *** zzzeek has joined #tripleo | 08:11 | |
| Tengu | marios: hey :). (though the topic will probably burn everything down to ashes for some reason :]) | 08:11 |
| *** bogdando has joined #tripleo | 08:12 | |
| marios | Tengu: i saw you mail to list yeah its a tough one but worth bringing up for some discussion for sure at least point folks to the thread | 08:13 |
| *** akahat has joined #tripleo | 08:13 | |
| Tengu | yup :) | 08:15 |
| *** cylopez has joined #tripleo | 08:16 | |
| *** zzzeek has quit IRC | 08:16 | |
| Tengu | the best thing would be to get support for module in RPM dependencies.... | 08:16 |
| Tengu | but there's apparently no way to get it for obscure reasons :(. | 08:16 |
| *** zzzeek has joined #tripleo | 08:18 | |
| openstackgerrit | Jaganathan Palanisamy proposed openstack/tripleo-ansible master: NFV derive parameters molecule tests https://review.opendev.org/c/openstack/tripleo-ansible/+/754630 | 08:21 |
| openstackgerrit | Hervé Beraud proposed openstack/tripleo-common master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-common/+/769259 | 08:32 |
| *** Abdallahyas has joined #tripleo | 08:35 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/tripleo-validations stable/victoria: Use the UBI8 image for testing https://review.opendev.org/c/openstack/tripleo-validations/+/769108 | 08:36 |
| *** tosky has joined #tripleo | 08:39 | |
| *** abdysn has quit IRC | 08:39 | |
| *** zzzeek has quit IRC | 08:39 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient stable/ussuri: Raise if no validation has been executed https://review.opendev.org/c/openstack/python-tripleoclient/+/759695 | 08:40 |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient master: [Validator Run] Add return code management https://review.opendev.org/c/openstack/python-tripleoclient/+/757820 | 08:41 |
| *** zzzeek has joined #tripleo | 08:41 | |
| openstackgerrit | Hervé Beraud proposed openstack/tripleo-heat-templates master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769263 | 08:51 |
| *** pcaruana has quit IRC | 08:59 | |
| *** pcaruana has joined #tripleo | 09:01 | |
| *** xek has joined #tripleo | 09:02 | |
| *** pcaruana has quit IRC | 09:04 | |
| *** lucasagomes has joined #tripleo | 09:09 | |
| *** gfidente has joined #tripleo | 09:10 | |
| *** zzzeek has quit IRC | 09:11 | |
| *** zzzeek has joined #tripleo | 09:13 | |
| *** pcaruana has joined #tripleo | 09:22 | |
| openstackgerrit | zhangtongjian proposed openstack/tripleo-specs master: remove unicode from code https://review.opendev.org/c/openstack/tripleo-specs/+/769286 | 09:24 |
| *** zzzeek has quit IRC | 09:26 | |
| *** zzzeek has joined #tripleo | 09:27 | |
| *** jmasud has joined #tripleo | 09:33 | |
| openstackgerrit | Hervé Beraud proposed openstack/tripleo-image-elements master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-image-elements/+/769298 | 09:34 |
| *** jbadiapa has joined #tripleo | 09:35 | |
| openstackgerrit | Merged openstack/diskimage-builder master: Add Python3 Wallaby unit tests https://review.opendev.org/c/openstack/diskimage-builder/+/757247 | 09:37 |
| *** sanjayu_ has quit IRC | 09:39 | |
| *** tkajinam has quit IRC | 09:42 | |
| *** zzzeek has quit IRC | 09:45 | |
| *** zzzeek has joined #tripleo | 09:47 | |
| openstackgerrit | Hervé Beraud proposed openstack/tripleo-puppet-elements master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/769302 | 09:53 |
| *** ratailor_ has joined #tripleo | 09:54 | |
| *** derekh has joined #tripleo | 09:54 | |
| *** ratailor_ has quit IRC | 09:55 | |
| *** ratailor_ has joined #tripleo | 09:57 | |
| *** ratailor has quit IRC | 09:57 | |
| *** ratailor_ has quit IRC | 09:57 | |
| *** ratailor has joined #tripleo | 09:58 | |
| openstackgerrit | Hervé Beraud proposed openstack/tripleo-quickstart-extras master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/769303 | 09:59 |
| *** ratailor has quit IRC | 09:59 | |
| *** ratailor has joined #tripleo | 10:02 | |
| *** paramite has quit IRC | 10:02 | |
| *** paramite has joined #tripleo | 10:03 | |
| openstackgerrit | Hervé Beraud proposed openstack/tripleo-repos master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-repos/+/769304 | 10:03 |
| *** abdysn has joined #tripleo | 10:06 | |
| openstackgerrit | Hervé Beraud proposed openstack/tripleo-upgrade master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-upgrade/+/769305 | 10:06 |
| openstackgerrit | Martin Magr proposed openstack/tripleo-heat-templates master: Fluentd cleanup in rsyslog implementation https://review.opendev.org/c/openstack/tripleo-heat-templates/+/757579 | 10:07 |
| *** cylopez has quit IRC | 10:08 | |
| *** Abdallahyas has quit IRC | 10:09 | |
| *** ysandeep is now known as ysandeep|afk | 10:14 | |
| *** rpittau is now known as rpittau|bbl | 10:32 | |
| *** sanjayu_ has joined #tripleo | 10:32 | |
| *** cylopez has joined #tripleo | 10:34 | |
| marios | http://lists.openstack.org/pipermail/openstack-discuss/2021-January/019610.html [tripleo] next irc meeting Tuesday Jan 05 @ 1400 UTC in #tripleo Please consider adding your news/blockers/plans in https://etherpad.opendev.org/p/tripleo-meeting-items even if you cannot attend | 10:41 |
| *** udesale__ has joined #tripleo | 10:45 | |
| *** udesale_ has quit IRC | 10:48 | |
| Tengu | marios: yeah. that dependency topic will be interesting :] | 10:50 |
| *** ratailor has quit IRC | 10:54 | |
| Tengu | oh.... in fact, we might... hmmm. we might be able to override the installed podman via tripleo-ansible directly. IF we set the "_tripleo_podman_packages" to "@container-tools:2.0/podman", it COULD work. ugly as hell. buuut.. | 10:55 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-tripleo master: Add ability to specify memcached port for all services. https://review.opendev.org/c/openstack/puppet-tripleo/+/769319 | 10:55 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-tripleo master: Add ability to specify memcached port https://review.opendev.org/c/openstack/puppet-tripleo/+/769319 | 10:56 |
| Tengu | ah. no. we can't pass the package name. wokay. | 10:59 |
| * Tengu goes back to his thoughts and tests | 10:59 | |
| *** efoley has joined #tripleo | 11:00 | |
| openstackgerrit | Merged openstack/python-tripleoclient master: [UX] Align to left the table for the validations run results https://review.opendev.org/c/openstack/python-tripleoclient/+/764642 | 11:00 |
| marios | ack thanks for filing the LP Tengu | 11:01 |
| openstackgerrit | Rabi Mishra proposed openstack/python-tripleoclient master: Use heat server side env merging https://review.opendev.org/c/openstack/python-tripleoclient/+/765808 | 11:03 |
| openstackgerrit | Lukas Bezdicka proposed openstack/python-tripleoclient stable/train: [train-only] Extend list of DEPRECATED_SERVICES https://review.opendev.org/c/openstack/python-tripleoclient/+/769320 | 11:04 |
| openstackgerrit | Lukas Bezdicka proposed openstack/tripleo-heat-templates stable/rocky: [Rocky&QueensOnly] Update ovs to 2.11 without network loss https://review.opendev.org/c/openstack/tripleo-heat-templates/+/696482 | 11:09 |
| *** chem has quit IRC | 11:14 | |
| *** ratailor has joined #tripleo | 11:16 | |
| *** chem has joined #tripleo | 11:17 | |
| *** ysandeep|afk is now known as ysandeep | 11:26 | |
| *** zzzeek has quit IRC | 11:26 | |
| *** zzzeek has joined #tripleo | 11:28 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Fix unreachable handling https://review.opendev.org/c/openstack/tripleo-heat-templates/+/767760 | 11:32 |
| *** jcapitao is now known as jcapitao_lunch | 11:38 | |
| *** zzzeek has quit IRC | 11:43 | |
| openstackgerrit | Martin Schuppert proposed openstack/tripleo-heat-templates stable/queens: [FFU] Use output parameter instead of hiera lookup for cell creation https://review.opendev.org/c/openstack/tripleo-heat-templates/+/734341 | 11:43 |
| *** zzzeek has joined #tripleo | 11:43 | |
| openstackgerrit | Jaganathan Palanisamy proposed openstack/tripleo-ansible master: NFV derive parameters molecule tests https://review.opendev.org/c/openstack/tripleo-ansible/+/754630 | 12:02 |
| *** abdysn has quit IRC | 12:03 | |
| *** Abdallahyas has joined #tripleo | 12:03 | |
| fultonj | /away | 12:13 |
| *** ratailor has quit IRC | 12:15 | |
| openstackgerrit | Marios Andreou proposed openstack/python-tripleoclient master: Wire up new tripleo upgrades jobs template https://review.opendev.org/c/openstack/python-tripleoclient/+/769334 | 12:15 |
| openstackgerrit | Marios Andreou proposed openstack/python-tripleoclient stable/victoria: Wire up new tripleo upgrades jobs template https://review.opendev.org/c/openstack/python-tripleoclient/+/769335 | 12:16 |
| openstackgerrit | Marios Andreou proposed openstack/python-tripleoclient stable/ussuri: Wire up new tripleo upgrades jobs template https://review.opendev.org/c/openstack/python-tripleoclient/+/769336 | 12:16 |
| openstackgerrit | Merged openstack/tripleo-common stable/victoria: Fix ImportWarning during importing a module https://review.opendev.org/c/openstack/tripleo-common/+/768406 | 12:17 |
| openstackgerrit | Marios Andreou proposed openstack/python-tripleoclient stable/train: Wire up new tripleo upgrades jobs template https://review.opendev.org/c/openstack/python-tripleoclient/+/769337 | 12:17 |
| *** jmasud has quit IRC | 12:18 | |
| *** zzzeek has quit IRC | 12:20 | |
| openstackgerrit | Marios Andreou proposed openstack/puppet-tripleo master: Wire up new tripleo upgrades jobs template https://review.opendev.org/c/openstack/puppet-tripleo/+/769338 | 12:22 |
| openstackgerrit | Marios Andreou proposed openstack/puppet-tripleo stable/victoria: Wire up new tripleo upgrades jobs template https://review.opendev.org/c/openstack/puppet-tripleo/+/769339 | 12:23 |
| *** xek has quit IRC | 12:24 | |
| *** abdysn has joined #tripleo | 12:24 | |
| *** xek has joined #tripleo | 12:24 | |
| *** zzzeek has joined #tripleo | 12:24 | |
| openstackgerrit | Marios Andreou proposed openstack/puppet-tripleo stable/ussuri: Wire up new tripleo upgrades jobs template https://review.opendev.org/c/openstack/puppet-tripleo/+/769340 | 12:25 |
| *** Abdallahyas has quit IRC | 12:25 | |
| openstackgerrit | Marios Andreou proposed openstack/puppet-tripleo stable/train: Wire up new tripleo upgrades jobs template https://review.opendev.org/c/openstack/puppet-tripleo/+/769341 | 12:26 |
| *** tbarron has joined #tripleo | 12:32 | |
| hberaud | marios: o/ | 12:33 |
| hberaud | marios: First, thanks for reviews. | 12:33 |
| marios | hberaud: o/ | 12:34 |
| hberaud | marios: I think that some projects need further work but this is outside of the scope of my changes, indeed some of you project have deps incompatibilities, example => https://review.opendev.org/c/openstack/tripleo-image-elements/+/769298 | 12:35 |
| hberaud | s/you project/your projects/ | 12:35 |
| marios | hberaud: ack thanks for posting them | 12:36 |
| marios | hberaud: i will dig into the errors or at least see if other folks know more about it zbr comes to mind | 12:36 |
| marios | hberaud: no problem we will handle it | 12:36 |
| marios | conflicting basepython version (set 3.8, should be 3.7) for | 12:37 |
| marios | hmmm | 12:37 |
| hberaud | marios: it could be painful to fix them... I already tried to fix this kind of issues on oslo and finally we have chose to drop LC jobs | 12:37 |
| marios | hberaud: thanks. so I'll add them into my list of stuff to check. I will bring them up in the irc meeting today anyway but i'll digg into the ones that failed and see how we make progress there | 12:38 |
| hberaud | but here this is different, and I think it's due to the pip -c <contrainsts> used in your tox targets | 12:38 |
| hberaud | awesome thanks | 12:39 |
| marios | hberaud: we already had some pain from the new pip resolver in an internal repo | 12:39 |
| marios | hberaud: thank you for pushing this | 12:39 |
| hberaud | the new resolver broken everything :) | 12:39 |
| marios | hberaud: yeah we had to post much stricter requirements file but before that fix it was trying >1 hour to fetch all the available versions for all the things | 12:40 |
| hberaud | marios: do not hesitate to ping me if you need some help | 12:40 |
| marios | hberaud: thank you | 12:40 |
| openstackgerrit | Merged openstack/tripleo-docs master: Add new Component/Integration pipeline docs https://review.opendev.org/c/openstack/tripleo-docs/+/767375 | 12:43 |
| *** rpittau|bbl is now known as rpittau | 12:44 | |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-tripleo master: Add ability to specify memcached port https://review.opendev.org/c/openstack/puppet-tripleo/+/769319 | 12:48 |
| zbr | hberaud: fyi: the new resolver was fixed, that is what "broke everything". that time we cannot blame pip. | 12:53 |
| zbr | mainly we failed to install compatible deps with old pip resolver, it was just giving you the impression that it worked. | 12:53 |
| *** rh-jelabarre has joined #tripleo | 12:53 | |
| hberaud | zbr: yes I know it was a joke | 12:53 |
| hberaud | zbr: the problem is on our side and not on the pip side | 12:54 |
| zbr | when it comes to pip, is hard to get a joke right ;) | 12:54 |
| *** gfidente has quit IRC | 12:55 | |
| hberaud | zbr: I surely need to improve my comic skills | 12:55 |
| hberaud | :) | 12:55 |
| *** rlandy has joined #tripleo | 13:04 | |
| *** jcapitao_lunch is now known as jcapitao | 13:19 | |
| *** rbrady has quit IRC | 13:20 | |
| *** pleimer has joined #tripleo | 13:38 | |
| cloudnull | mornings | 13:56 |
| marios | o/ | 13:59 |
| marios | tripleo meet starting in few seconds | 13:59 |
| marios | #startmeeting tripleo | 14:00 |
| openstack | Meeting started Tue Jan 5 14:00:03 2021 UTC and is due to finish in 60 minutes. The chair is marios. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| *** openstack changes topic to " (Meeting topic: tripleo)" | 14:00 | |
| openstack | The meeting name has been set to 'tripleo' | 14:00 |
| marios | #topic agenda | 14:00 |
| marios | * Review last minutes & action items | 14:00 |
| marios | * One off agenda items | 14:00 |
| marios | * Bugs & Blueprints | 14:00 |
| marios | * Projects releases or stable backports | 14:00 |
| *** openstack changes topic to "agenda (Meeting topic: tripleo)" | 14:00 | |
| marios | * Specs | 14:00 |
| marios | * open discussion | 14:00 |
| marios | Anyone can use the #link, #action and #info commands, not just the moderatorǃ | 14:00 |
| marios | Hello folks, who is around today? o/ | 14:00 |
| slagle | o/ | 14:00 |
| Tengu | «o/ | 14:00 |
| weshay|ruck | 0/ | 14:00 |
| ramishra | o/ | 14:00 |
| rlandy | o/ | 14:01 |
| marios | great hello folks, let's get going | 14:01 |
| marios | #topic review last meeting logs & action items | 14:01 |
| *** openstack changes topic to "review last meeting logs & action items (Meeting topic: tripleo)" | 14:01 | |
| marios | #link http://eavesdrop.openstack.org/meetings/tripleo/2020/tripleo.2020-12-22-14.00.html | 14:01 |
| marios | last logs there ^^^ | 14:02 |
| rfolco | o> | 14:02 |
| marios | checking for any action items | 14:02 |
| marios | we had a request for victoria release i already have that on agenda | 14:02 |
| chandankumar | \o/ | 14:02 |
| marios | anything else from last week folks want to mention/raise/revisit here? | 14:02 |
| marios | k then moving on ... | 14:03 |
| marios | #topic one off agenda items | 14:03 |
| *** openstack changes topic to "one off agenda items (Meeting topic: tripleo)" | 14:03 | |
| marios | #link https://etherpad.openstack.org/p/tripleo-meeting-items | 14:03 |
| marios | we have a few there... lets take them in order | 14:03 |
| marios | #info [slagle] Ephemeral Heat spec for review: https://review.opendev.org/c/openstack/tripleo-specs/+/765000 | 14:04 |
| slagle | just wanted to point out the spec | 14:04 |
| marios | slagle: o/ you want to say something about that? besides the spec is there a topic branch with something to look at? | 14:04 |
| slagle | we're moving forward with a prototype, so it's last call for spec comments | 14:04 |
| fultonj | o/ | 14:04 |
| slagle | should have a branch up soon | 14:04 |
| marios | slagle: ack. thanks | 14:05 |
| marios | anything else on this topic? | 14:05 |
| marios | #info [tengu] Tripleo dependency: the podman case http://lists.openstack.org/pipermail/openstack-discuss/2021-January/019625.html | 14:05 |
| Tengu | ah, that's me :) | 14:05 |
| marios | Tengu: sent a message to the list earlier today about this ... Tengu want to hilight something in particular? | 14:05 |
| Tengu | well, just poking around in order to get inputs.... that case is pretty annoying and reccurring | 14:06 |
| Tengu | as said in my last mail in the thread, we'd have the same case for the "virt" module on OSP, apparently. | 14:06 |
| Tengu | soooo yeah.... something ensuring we're using the right stream, in a generic way, is kind of mandatory at some point. People don't read doc anyway. | 14:06 |
| marios | Tengu: is there support for the tripleo-ansible case in the deployment framework team or haven't discussed yet? | 14:07 |
| slagle | i'd say drop the REquires in tripleoclient, and more generally, we should do that for a lot of things | 14:07 |
| Tengu | (or are using automated tools and they don't really update them... /me hides) | 14:07 |
| Tengu | marios: I've mostly discussed with Delivery at this point | 14:07 |
| slagle | there are too many things there that are required for the deployment, but not required by the actual python code | 14:07 |
| marios | slagle: yah i hilighted that in the thread, i mean it impacts the ephemeral heat case since afaics right now the heat leanucher is (one of ) the main users right now for tha trequirement | 14:07 |
| marios | slagle: so can we? remove that dependency? | 14:08 |
| marios | slagle: how? use somehting not podman? | 14:08 |
| Tengu | slagle: though a container is launched for heat from within the tripleoclient directly. So we'll need to do some ordering at some point | 14:08 |
| Tengu | that's the beauty of this thing :) | 14:08 |
| Tengu | anyway... please jump in the thread (or LP https://bugs.launchpad.net/tripleo/+bug/1910217) and let's find a clean solution | 14:08 |
| openstack | Launchpad bug 1910217 in tripleo "Package dependencies, podman, dnf modules" [Medium,Triaged] | 14:08 |
| slagle | the heat container isn't used by default | 14:08 |
| slagle | not sure it even works tbh | 14:09 |
| Tengu | guess that's a nice 2021 topic: dependency cleanup and consolidation :) | 14:09 |
| marios | slagle: k i just did a quick grep on the code when replying to the thread so maybe it isn't as hard a requirement as i thought | 14:09 |
| Tengu | slagle: oh? really? I'd think it was used by default when deploying the undercloud/standalone in order to get the heat service. | 14:09 |
| Tengu | if it's not used/working..... let's drop it for the sake of sanity :) | 14:10 |
| marios | Tengu: thanks for bringing that topic and hilighting here and thanks slagle for jumping in | 14:10 |
| slagle | don't think so. pretty sure heat-all is still run uncontainerized | 14:10 |
| slagle | i'll reply on the thread | 14:10 |
| mwhahaha | yes it's not containerized | 14:10 |
| Tengu | slagle: since it's linked to your spec... :) | 14:10 |
| mwhahaha | it's an option | 14:10 |
| mwhahaha | but not the default | 14:10 |
| mwhahaha | as discussed @ptg | 14:10 |
| marios | mwhahaha: thanks for confirming that's what slagle is saying too | 14:11 |
| marios | i thought it was default but i only did a quick grep to see what was calling podman | 14:11 |
| * Tengu prepars the kill-shot on the Requires podman | 14:11 | |
| marios | great, lets carry on in the list | 14:11 |
| marios | moving on ... . | 14:11 |
| mwhahaha | so we also have modules configurable in THT | 14:11 |
| mwhahaha | there's some other options | 14:11 |
| *** zbitter is now known as zaneb | 14:12 | |
| marios | #info [fultonj] Adding an SSH private key on overcloud node for cephadm https://review.opendev.org/c/openstack/tripleo-ansible/+/768365 https://specs.openstack.org/openstack/tripleo-specs/specs/wallaby/tripleo-ceph.html | 14:12 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-tripleo master: Add authtoken security options https://review.opendev.org/c/openstack/puppet-tripleo/+/769349 | 14:12 |
| marios | fultonj: o/ ^^ are you around for that one? | 14:12 |
| fultonj | yes | 14:12 |
| fultonj | basically it's a change for the cephadm work | 14:13 |
| bogdando | Tengu: I didn't get the point in ML, why not to use Conflicts: podman < lower? | 14:13 |
| fultonj | just asking for reviews | 14:13 |
| marios | ack thanks fultonj | 14:13 |
| bogdando | sorry, keep it for the open discussion | 14:13 |
| marios | bogdando: Tengu: we can pickup again in the open session at the end? | 14:13 |
| marios | bogdando: thanks | 14:13 |
| Tengu | :) | 14:13 |
| *** skramaja has quit IRC | 14:14 | |
| marios | #info [fultonj] Just need one more patch to make new ceph client migration complete https://review.opendev.org/c/openstack/tripleo-heat-templates/+/760915 https://specs.openstack.org/openstack/tripleo-specs/specs/wallaby/tripleo-ceph-client.html | 14:14 |
| Tengu | brb | 14:14 |
| marios | fultonj: o/ ^^ that one too | 14:14 |
| fultonj | ack, one more reivew and new ceph client config work is complete | 14:14 |
| marios | great work thanks fultonj | 14:14 |
| marios | any comments or questions on those? | 14:14 |
| marios | #info [fultonj] https://review.opendev.org/c/openstack/tripleo-specs/+/760536 merged | 14:15 |
| marios | cool looks like the ports v2 spec merged zo/ | 14:15 |
| marios | thanks hjensas|PTO | 14:15 |
| marios | anything else on that one? | 14:15 |
| marios | i know a few things depend on it like ephemeral heat spec | 14:15 |
| *** tosky has quit IRC | 14:15 | |
| *** tosky_ has joined #tripleo | 14:16 | |
| *** tosky_ is now known as tosky | 14:16 | |
| marios | k moving on | 14:16 |
| marios | #info (marios): promotion CI docs @ https://review.opendev.org/c/openstack/tripleo-docs/+/767375 (build@ https://f267574198a517c003d1-a40b9478ae1bd073dc32f331038fe6d7.ssl.cf5.rackcdn.com/767375/7/check/openstack-tox-docs/8252ffe/docs/ci/stages-overview.html ) | 14:16 |
| fultonj | ports v2 spec depends on ephemeral heat? | 14:16 |
| marios | fultonj: no other way around i believe | 14:16 |
| fultonj | ok, that makes more sense | 14:16 |
| marios | so we can get the network provision done before the heat stack create | 14:16 |
| slagle | marios: ++ | 14:16 |
| marios | k so promotion docs you may have heard the ci team talk about a 'victoria promotion' or a master promotion etc that doc explains that if you are interested in our process | 14:17 |
| marios | #info (marios): reduce content providers (upgrade jobs) https://review.opendev.org/q/topic:reduce-content-providers | 14:17 |
| marios | this is some ongoign work that will reduce the number of content providers we have to run for upgrade jobs | 14:17 |
| marios | the current situation is pretty bad, like standalone-upgrade-ussuri is running content providers for master, victoria, ussuri and train right now | 14:18 |
| marios | #info (rfolco): https://review.opendev.org/c/openstack/tripleo-ci/+/767591 Enable html report for content provider jobs https://fa41a0ce81614565dae6-7ca08d7dc339532aeab9467cc90f03e8.ssl.cf1.rackcdn.com/767591/15/check/tripleo-ci-centos-8-content-provider/3f24737/logs/report.html | 14:18 |
| marios | i added that for rfolco ^ anything you want to add there? | 14:18 |
| rfolco | thats it | 14:18 |
| marios | thanks rfolco that adds a nice report for all the container builds | 14:18 |
| rfolco | self-explanatory | 14:18 |
| rfolco | thx | 14:19 |
| marios | will make our lives much easier debugging | 14:19 |
| marios | rfolco:++ | 14:19 |
| marios | #info made a stable/victoria release for repos at https://review.opendev.org/c/openstack/releases/+/768237 | 14:19 |
| marios | that was on request from ykarel last meeting it merged yesterday i believe ^^^ fyi | 14:19 |
| *** TrevorV has joined #tripleo | 14:19 | |
| marios | #info [release] Status: RED - pip resolver issue with publish-openstack-releasenotes-python3 http://lists.openstack.org/pipermail/openstack-discuss/2021-January/019611.html ** please help merge ** https://review.opendev.org/q/topic:%22fix-relmgt-pip-doc%22+(status:open%20OR%20status:merged | 14:19 |
| marios | that came up yesterday related to the release - will need some help merging and fixing the ones that aren't working from the topic branch (tripleo repos) | 14:19 |
| weshay|ruck | that does look nice rfolco | 14:20 |
| marios | sorry for rush trying to get through quickly | 14:20 |
| marios | fiunally | 14:20 |
| marios | #info reminder re [TripleO] moving stable/rocky for tripleo repos to unmaintained (+ then EOL) OK? http://lists.openstack.org/pipermail/openstack-discuss/2020-December/019338.html - we'll start moving that forward - possibly bump the thread a bit first to hilight. | 14:20 |
| marios | so just bringing that up again, probably i should bump the thread but we will start making that so now since there were no objections so far | 14:20 |
| marios | please see and reply if you have comments | 14:20 |
| *** artom has quit IRC | 14:20 | |
| marios | we can revisit any of those topics in the open session at the end i am going to move on | 14:21 |
| marios | thanks all for those topics | 14:21 |
| marios | #topic Bugs and blueprints | 14:21 |
| *** openstack changes topic to "Bugs and blueprints (Meeting topic: tripleo)" | 14:21 | |
| marios | #link https://bugs.launchpad.net/tripleo/ | 14:21 |
| marios | #link https://storyboard.openstack.org/#!/project/openstack/tripleo-ansible | 14:21 |
| marios | #link https://launchpad.net/tripleo/+milestone/wallaby-2 | 14:21 |
| marios | #link https://launchpad.net/tripleo/wallaby | 14:21 |
| marios | as always a reminder to please check the wallaby-2 bugs and close out anything stale that belongs to you | 14:21 |
| marios | any bugs folks want to hilight here? are the ruck/rover around anything worth mentioning weshay|ruck ? | 14:22 |
| weshay|ruck | just FYI.. to folks doing local deployments w/ centos ens3 is back as ens3-1 | 14:23 |
| weshay|ruck | https://bugs.launchpad.net/tripleo/+bug/1910107 | 14:23 |
| openstack | Launchpad bug 1910107 in tripleo "master/wallaby undercloud install failure unable to start network service" [Critical,Triaged] | 14:23 |
| weshay|ruck | https://review.opendev.org/c/openstack/tripleo-image-elements/+/769209 | 14:23 |
| marios | thank you weshay|ruck | 14:23 |
| marios | moving on if there is nothing else from anyone | 14:24 |
| marios | #topic Project releases or stable backports | 14:24 |
| *** openstack changes topic to "Project releases or stable backports (Meeting topic: tripleo)" | 14:24 | |
| marios | #info tripleo wallaby repos https://releases.openstack.org/teams/tripleo.html#wallaby | 14:24 |
| marios | anything here? any release requests? we should probably do a 'final' rocky release soon before we nuke it... | 14:24 |
| marios | wallaby-3 is a way off apparently, mar 08-12 https://releases.openstack.org/wallaby/schedule.html | 14:25 |
| marios | k if there is nothing here i will move on to specs | 14:25 |
| marios | (I am using the agenda at https://wiki.openstack.org/wiki/Meetings/TripleO for these things by the way ) | 14:25 |
| marios | for copy/pasta | 14:25 |
| marios | #topic specs | 14:26 |
| *** openstack changes topic to "specs (Meeting topic: tripleo)" | 14:26 | |
| marios | #info https://review.opendev.org/q/project:openstack/tripleo-specs | 14:26 |
| marios | #info https://opendev.org/openstack/tripleo-specs/src/branch/master/specs/wallaby | 14:26 |
| *** artom has joined #tripleo | 14:26 | |
| marios | k so anything else here? we had a mention on the ephemeral heat already | 14:26 |
| marios | the ports v2 merged | 14:26 |
| marios | i think the frrouter can merge lemme fetch link | 14:26 |
| marios | #link Spec for installing and configuring FRRouter for BGP https://review.opendev.org/c/openstack/tripleo-specs/+/758249 | 14:27 |
| marios | reviews please? ^^^ | 14:27 |
| marios | anything else on this topic for today? | 14:27 |
| marios | k... moving on | 14:28 |
| marios | #topic open discussion | 14:28 |
| *** openstack changes topic to "open discussion (Meeting topic: tripleo)" | 14:28 | |
| marios | Anything else that folks want to bring up to the meeting? | 14:28 |
| Tengu | bogdando: want to talk about the conflict thing? | 14:28 |
| marios | Tengu: bogdando: mwhahaha: was there further discussion around the podman version issue | 14:28 |
| mwhahaha | we're trying to solve it in tripleo-repos as well | 14:29 |
| mwhahaha | i don't think we any rpm dep is really going to help | 14:29 |
| mwhahaha | as we've already tried that and it's caused problems | 14:29 |
| Tengu | so... while it seems nice to set upper/lower constraints for a given package, it might happen it's not that easy, and, as written in the ML, it will lead in a ton of LP/BZ about "unable to install, package conflict". I'm not the one wanting to sort them out just because ppl don't read doc, don't understand and other reasons ;). | 14:30 |
| mwhahaha | without module deps in rpm specs we just have to document it and try and fixup the initial setup requirements | 14:30 |
| Tengu | mwhahaha: right. I've broken teeth on this issue last year, and now... well, if we can get rid of it once for all and use a proper ansible thing in order to get the right version directly, I'm all for it. | 14:30 |
| Tengu | though switching module stream when packages are installed might fail -.-' | 14:31 |
| mwhahaha | without creating a meta package that'll do all the things (setup repos, install stuff) it's not something really solvable | 14:31 |
| mwhahaha | we already have issues with versions in ci, which is why we were going to try and express all the versions in tripleo-repos | 14:31 |
| Tengu | yeah..... maybe such a "meta-thing" is a better way than dropping the Requires and relying on tripleo-ansible roles? | 14:31 |
| mwhahaha | https://review.opendev.org/c/openstack/tripleo-repos/+/767214 | 14:32 |
| mwhahaha | the other issue is the lack of an equivalent downstream for customers | 14:32 |
| Tengu | yup | 14:32 |
| mwhahaha | like there needs to be an initial package that does something like that | 14:32 |
| Tengu | we tried to add Requirements on the osp-release (not sure about the name though) | 14:32 |
| mwhahaha | that would be my recommendation is that the repo setup is automated via a cli tool | 14:32 |
| ade_lee | cloudnull, Tengu , https://review.opendev.org/c/openstack/tripleo-heat-templates/+/764988 please | 14:32 |
| bogdando | so I was thinking about a simple way to fail the client package install if podman looks old... | 14:32 |
| mwhahaha | and that be the 'official' way of doing thigns | 14:33 |
| mwhahaha | bogdando: the issue isnt' too old, it's too new | 14:33 |
| mwhahaha | bogdando: so we don't want container-tools:rhel8 for train | 14:33 |
| bogdando | and conflicts: alongside requires: looked like a good fit. apparently not | 14:33 |
| mwhahaha | cause it causes problems | 14:33 |
| Tengu | bogdando: it's not a matter of "old" in some cases... container-tools:rhel8 provides podman 2.x, while we need 1.6.4 (at a critical sub-version) | 14:33 |
| mwhahaha | we already tried version constraints and it leads to poor UX due to conflict messaging | 14:33 |
| Tengu | also that -^ thanks mwhahaha | 14:33 |
| mwhahaha | reducing the user's ability to miss a setup step is an improvement | 14:34 |
| mwhahaha | having to specify module container-tools:2.0 is too much it appears | 14:34 |
| Tengu | mwhahaha: having tripleo-repos setting everything fine is better, and ensuring via tripleo-ansible everything is actually GOOD is a nice to have | 14:34 |
| mwhahaha | there's a THT change recent to ensure module versions | 14:34 |
| Tengu | for upgrades iirc | 14:35 |
| mwhahaha | for upgrades but could be used to ensure overcloud | 14:35 |
| mwhahaha | and undercloud | 14:35 |
| Tengu | Jose did it last year | 14:35 |
| mwhahaha | it just landed | 14:35 |
| Tengu | nice | 14:35 |
| Tengu | (and there are validations.... I know, you don't like them, still....) | 14:35 |
| mwhahaha | https://review.opendev.org/c/openstack/tripleo-heat-templates/+/767994+ | 14:35 |
| mwhahaha | validations is too late in the game | 14:35 |
| Tengu | not with the prep ones, but one has to run them. if they don't read the doc.... no way they run anything | 14:36 |
| openstackgerrit | Ade Lee proposed openstack/puppet-tripleo stable/victoria: Adding key_size option on the certmonger_certificate function https://review.opendev.org/c/openstack/puppet-tripleo/+/769195 | 14:36 |
| mwhahaha | the other goal is not to have to define these requirements in multiple palces | 14:36 |
| Tengu | +42 | 14:36 |
| mwhahaha | so validations will require a duplicate update to something that may set it up | 14:36 |
| mwhahaha | i would prefer we settle on one thing | 14:37 |
| Tengu | I thought about the tripleo-ansible/tripleo_podman since it's, well, linked to podman.. | 14:37 |
| mwhahaha | it's not what installs on the undercloud unfortunately which is where we traditional hit this issue | 14:37 |
| ade_lee | mwhahaha, Tengu , cloudnull https://review.opendev.org/c/openstack/tripleo-heat-templates/+/767933 please | 14:37 |
| Tengu | but if we're going for a tripleo-repos modification, tripleo-ansible change might be useless. That is, as long as ppl are using this tripleo-repos, of course (they should be.) | 14:37 |
| mwhahaha | i think tripleo-ansible is also too late | 14:37 |
| *** tmazur has joined #tripleo | 14:37 | |
| Tengu | mwhahaha: depends - if we drop the Requirement from the spec... ? | 14:38 |
| bogdando | mwhahaha, Tengu: ok, then that could be conflicts: podman > 1.9 | 14:38 |
| openstackgerrit | Ade Lee proposed openstack/puppet-tripleo stable/ussuri: Adding key_size option on the certmonger_certificate function https://review.opendev.org/c/openstack/puppet-tripleo/+/769356 | 14:38 |
| mwhahaha | bogdando: conflicts not going to help, it'll just be the same confusing error | 14:38 |
| bogdando | a fixed fence to never accept 2.0 for that release, and no need to poke it with newer 1.6.x | 14:39 |
| Tengu | confusion leading to massive spam on BZ -.-' | 14:39 |
| bogdando | ...or >= 2.0 | 14:39 |
| mwhahaha | 1.6 and 1.9 are not compatible either | 14:39 |
| mwhahaha | (seen that one) | 14:39 |
| marios | what do you think folks? are we done on this one for now/here? | 14:40 |
| mwhahaha | this is not something that can be solved simply, we'll need to talk more about the overal process | 14:40 |
| marios | sounds like we are leaning towards the tripleo-repos review currently | 14:40 |
| marios | but there can be more discussion upstream on the list too | 14:40 |
| Tengu | guess so. Would love seeing your thoughts in the thread, mwhahaha :) | 14:40 |
| Tengu | (or LP) | 14:41 |
| mwhahaha | closed: invalid? got it | 14:41 |
| marios | thank you for bringing that today Tengu | 14:41 |
| Tengu | marios: yep - I'm all for the tripleo-repos change | 14:41 |
| * Tengu tortures mwhahaha with a scary Windows© ME user guide | 14:41 | |
| marios | any other topics someone wants to hilight here? | 14:41 |
| bogdando | mwhahaha: I don't think installing with 2.0 podman instead of 1.6.x would improvie UX | 14:41 |
| *** abdysn has quit IRC | 14:41 | |
| bogdando | so that would fix nothing from the UX pov... | 14:41 |
| Tengu | bogdando: it will most likely crash at some point. | 14:42 |
| mwhahaha | bogdando: container-tools:2.0 not podman 2.0 | 14:42 |
| Tengu | bogdando: also, beware, container-tools:2.0 doesn't provide podman-2.x -.- (yeah.... don't ask) | 14:42 |
| mwhahaha | anwyay | 14:42 |
| marios | thank you to everybody for participating and following along today | 14:43 |
| marios | #info next meeting tuesday 19th Jan (2 weeks) | 14:44 |
| marios | #endmeeting tripleo | 14:44 |
| *** openstack changes topic to "CI status: ¯\_(ツ)_/¯ | tripleo-ci-community meeting Tues@1330 UTC | https://docs.openstack.org/tripleo-docs" | 14:44 | |
| openstack | Meeting ended Tue Jan 5 14:44:13 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:44 |
| marios | o/ | 14:44 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/tripleo/2021/tripleo.2021-01-05-14.00.html | 14:44 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/tripleo/2021/tripleo.2021-01-05-14.00.txt | 14:44 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/tripleo/2021/tripleo.2021-01-05-14.00.log.html | 14:44 |
| sshnaidm | cloudnull, hi, this was by mistake opened for openstack collection, I moved it to tripleo-ansible, just fyi: https://bugzilla.redhat.com/show_bug.cgi?id=1911891 | 15:02 |
| openstack | bugzilla.redhat.com bug 1911891 in tripleo-ansible "[RFE] Improvement in ansible-playbook" [Medium,New] - Assigned to rhos-maint | 15:02 |
| *** sanjayu_ has quit IRC | 15:03 | |
| cloudnull | sshnaidm ++ | 15:03 |
| cloudnull | o/ | 15:03 |
| *** sanjayu_ has joined #tripleo | 15:03 | |
| mwhahaha | that'll likely get solved in 16.2 | 15:03 |
| openstackgerrit | David Peacock proposed openstack/tripleo-operator-ansible master: add role to list available tripleo validations https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/755365 | 15:03 |
| * mwhahaha digs up bz | 15:03 | |
| mwhahaha | actually the fact thing isnt' something we can solve | 15:04 |
| mwhahaha | large number of network interfaces is a problem in ansible | 15:04 |
| mwhahaha | (like it was in facter but facter improved it) | 15:04 |
| *** dciabrin has quit IRC | 15:07 | |
| cloudnull | that's also something that "should"tm be improved with the introduction of fact caching, though that initial gather is likely takes forever. | 15:08 |
| mwhahaha | fact caching was in place | 15:09 |
| mwhahaha | but i bet it's also an ansible.cfg tuning that might also be missing | 15:09 |
| openstackgerrit | David Peacock proposed openstack/tripleo-operator-ansible master: WIP - add role to show tripleo validation https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/755375 | 15:09 |
| mwhahaha | will need to look at the case later to see if it has the full deployment log and we can see what's actually taking too long | 15:10 |
| cloudnull | ++ | 15:13 |
| *** ysandeep is now known as ysandeep|away | 15:13 | |
| mwhahaha | i bet they have a bunch of roles | 15:13 |
| cloudnull | you think? | 15:13 |
| mwhahaha | they have 3 sets of computes with the same names (??) | 15:14 |
| mwhahaha | if they do have a bunch of roles, the issue is https://bugzilla.redhat.com/show_bug.cgi?id=1897890 | 15:14 |
| openstack | bugzilla.redhat.com bug 1897890 in openstack-tripleo-heat-templates "OSP16.1 config-download does not scale with increasing number of roles" [High,Post] - Assigned to aschultz | 15:14 |
| mwhahaha | i think it's forks or something. the longest only two set of task exceeded 1 minute | 15:21 |
| *** rbrady has joined #tripleo | 15:29 | |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-tripleo master: Add ability to specify memcached port https://review.opendev.org/c/openstack/puppet-tripleo/+/769319 | 15:32 |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-ci master: WIP: enabling smoke tests for octavia scenario https://review.opendev.org/c/openstack/tripleo-ci/+/766278 | 15:34 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-tripleo master: Add authtoken security options https://review.opendev.org/c/openstack/puppet-tripleo/+/769349 | 15:37 |
| *** soniya29 has quit IRC | 15:45 | |
| *** ramishra has quit IRC | 15:45 | |
| openstackgerrit | Ade Lee proposed openstack/puppet-tripleo stable/ussuri: Adding key_size option on the certmonger_certificate function https://review.opendev.org/c/openstack/puppet-tripleo/+/769356 | 15:45 |
| *** soniya29 has joined #tripleo | 15:46 | |
| weshay|ruck | anybody know what's up w/ the retries on https://zuul.openstack.org/builds?job_name=tripleo-ansible-centos-8-molecule-tripleo-modules+ | 15:47 |
| openstackgerrit | Ade Lee proposed openstack/puppet-tripleo stable/train: Adding key_size option on the certmonger_certificate function https://review.opendev.org/c/openstack/puppet-tripleo/+/769360 | 15:47 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-ci master: Adds new per-branch zuul-templates for upgrades jobs https://review.opendev.org/c/openstack/tripleo-ci/+/761188 | 15:49 |
| marios | sshnaidm: thank you updated ^^^ | 15:49 |
| sshnaidm | marios, thanks, +2 | 15:51 |
| *** ramishra has joined #tripleo | 15:52 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/openstack-tempest-skiplist master: un skip neutron tests for queens https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/766252 | 15:56 |
| *** udesale__ has quit IRC | 15:59 | |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-image-elements master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-image-elements/+/769298 | 16:10 |
| openstackgerrit | Sofer Athlan-Guyot proposed openstack/tripleo-heat-templates stable/queens: DNM: Testing restart of ceph service. https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769393 | 16:10 |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-image-elements master: ensure any ens3 interface is nuked during image build https://review.opendev.org/c/openstack/tripleo-image-elements/+/769209 | 16:11 |
| openstackgerrit | Jaganathan Palanisamy proposed openstack/tripleo-ansible master: NFV derive parameters molecule tests https://review.opendev.org/c/openstack/tripleo-ansible/+/754630 | 16:13 |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-image-elements master: Add Victoria release info https://review.opendev.org/c/openstack/tripleo-image-elements/+/768899 | 16:14 |
| openstackgerrit | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/queens: Add new role parameters for cpu/ram/disk allocation ratio https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769394 | 16:15 |
| openstackgerrit | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/queens: Add new role parameters for cpu/ram/disk allocation ratio https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769394 | 16:16 |
| openstackgerrit | Yadnesh Kulkarni proposed openstack/python-tripleoclient master: Overcloud image has missing kernel_id and ramdisk_id properties https://review.opendev.org/c/openstack/python-tripleoclient/+/769395 | 16:23 |
| *** lmiccini has quit IRC | 16:24 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/tripleo-validations stable/train: Add tripleo-validation.py as script in setup.cfg https://review.opendev.org/c/openstack/tripleo-validations/+/756493 | 16:24 |
| openstackgerrit | Gael Chamoulaud proposed openstack/tripleo-validations stable/train: Ensures script is executable https://review.opendev.org/c/openstack/tripleo-validations/+/756494 | 16:24 |
| openstackgerrit | Gael Chamoulaud proposed openstack/tripleo-validations stable/train: Change path for validation Ansible files https://review.opendev.org/c/openstack/tripleo-validations/+/756495 | 16:24 |
| openstackgerrit | Gael Chamoulaud proposed openstack/tripleo-validations stable/train: Restore layout.yaml file https://review.opendev.org/c/openstack/tripleo-validations/+/763969 | 16:24 |
| *** jonmasters has quit IRC | 16:26 | |
| *** jonmasters has joined #tripleo | 16:26 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient master: [Validator Run] Add return code management https://review.opendev.org/c/openstack/python-tripleoclient/+/757820 | 16:31 |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient stable/victoria: [UX] Align to left the table for the validations run results https://review.opendev.org/c/openstack/python-tripleoclient/+/769361 | 16:33 |
| xek | mwhahaha, hey, please take a look at https://review.opendev.org/c/openstack/tripleo-heat-templates/+/768400 | 16:33 |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-repos master: tripleo version data https://review.opendev.org/c/openstack/tripleo-repos/+/767214 | 16:38 |
| openstackgerrit | Merged openstack/tripleo-upgrade master: [ffwd] Don't run healthcheck on redis if it's not present https://review.opendev.org/c/openstack/tripleo-upgrade/+/767712 | 16:47 |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient stable/victoria: [Validator Show Run] Display simplified results by default https://review.opendev.org/c/openstack/python-tripleoclient/+/760935 | 16:50 |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient stable/victoria: Dumping task key instead of tasks from validation_output https://review.opendev.org/c/openstack/python-tripleoclient/+/769399 | 16:50 |
| *** lucasagomes has quit IRC | 16:51 | |
| *** marios is now known as marios|out | 16:58 | |
| *** psachin has quit IRC | 17:00 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient stable/victoria: Run validation without authentication in particular context https://review.opendev.org/c/openstack/python-tripleoclient/+/769402 | 17:02 |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient stable/victoria: Get deployment status to check if plan and stack exists on the env https://review.opendev.org/c/openstack/python-tripleoclient/+/769403 | 17:02 |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient stable/ussuri: [Validator Show] Properly throwing an exception for unknown validation https://review.opendev.org/c/openstack/python-tripleoclient/+/760519 | 17:07 |
| *** rpittau is now known as rpittau|afk | 17:08 | |
| *** marios|out has quit IRC | 17:12 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient stable/train: [Validator Run] Ensure the plan really exists https://review.opendev.org/c/openstack/python-tripleoclient/+/769407 | 17:13 |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient stable/train: [Validator Run] Don't rely on parameter file extension https://review.opendev.org/c/openstack/python-tripleoclient/+/769408 | 17:13 |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient stable/train: [Validator Show] Properly throwing an exception for unknown validation https://review.opendev.org/c/openstack/python-tripleoclient/+/769409 | 17:13 |
| *** jcapitao has quit IRC | 17:18 | |
| *** bogdando has quit IRC | 17:24 | |
| *** cylopez has quit IRC | 17:29 | |
| openstackgerrit | Merged openstack/tripleo-repos master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-repos/+/769304 | 17:32 |
| openstackgerrit | Merged openstack/tripleo-repos master: Update TOX_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/tripleo-repos/+/767682 | 17:33 |
| *** ykarel has quit IRC | 17:36 | |
| weshay|ruck | cloudnull, 0/ qq re: cinder and tcib | 17:50 |
| weshay|ruck | https://opendev.org/openstack/tripleo-common/src/branch/master/container-images/kolla/glance-api/extend_start.sh | 17:50 |
| * cloudnull looking | 17:50 | |
| weshay|ruck | https://bugs.launchpad.net/tripleo/+bug/1908750 | 17:50 |
| openstack | Launchpad bug 1908750 in tripleo "stable/train volume failed to build and is in ERROR status, Permission denied: '/var/lib/cinder/groups'" [Critical,Triaged] | 17:50 |
| weshay|ruck | do the db_sync commands need to be in the container files? | 17:50 |
| cloudnull | I don't know if they need to be ? | 17:51 |
| cloudnull | I do know that they were in the old setup | 17:51 |
| weshay|ruck | aye.. abishop ^ | 17:51 |
| mwhahaha | i don't know if we use kolla bootpstrap for that | 17:51 |
| cloudnull | so a lot of that carried over | 17:51 |
| mwhahaha | yes it's invoked | 17:51 |
| mwhahaha | KOLLA_BOOTSTRAP is in glance-api | 17:51 |
| weshay|ruck | in tcib? | 17:51 |
| mwhahaha | https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/glance/glance-api-container-puppet.yaml#L674 | 17:52 |
| mwhahaha | that code is only invoked int he glance_api_db_sync container | 17:52 |
| weshay|ruck | https://opendev.org/openstack/tripleo-common/src/branch/master/container-images/tcib/base/os/glance-api/glance-api.yaml#L4 | 17:52 |
| weshay|ruck | ah thanks | 17:52 |
| weshay|ruck | :) | 17:53 |
| mwhahaha | and is not related to the LP you pasted | 17:53 |
| * weshay|ruck goes away | 17:53 | |
| mwhahaha | probably missing a chown or something | 17:53 |
| weshay|ruck | c/p fail | 17:53 |
| abishop | cloudnull, mwhahaha: per my comment in the LP, I was surprised to find no extend_start for cinder-volume. kolla provides one that sets ownership of /var/lib/cinder, which seems to be the source of the lp failure | 17:54 |
| weshay|ruck | https://opendev.org/openstack/tripleo-common/src/branch/master/container-images/tcib/base/os/glance-api/glance-api.yaml#L4 | 17:54 |
| openstackgerrit | Sofer Athlan-Guyot proposed openstack/tripleo-heat-templates stable/queens: DNM: Testing restart of ceph service. https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769393 | 17:54 |
| openstackgerrit | Sofer Athlan-Guyot proposed openstack/tripleo-heat-templates stable/queens: Ensure no rebalance during ceph update if docker is updated. https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769169 | 17:54 |
| weshay|ruck | glance has it.. is that enough? or cinder-volume needs it too? | 17:55 |
| cloudnull | yea that extend start is copied over, is it not being invoked ? | 17:55 |
| cloudnull | or do we need to modify it ? | 17:55 |
| mwhahaha | extended start is not invoked when run under pacemaker i believe | 17:55 |
| mwhahaha | because bundles | 17:55 |
| cloudnull | ah that ^ | 17:56 |
| mwhahaha | actually it should be invoked | 17:56 |
| mwhahaha | the config specifies stuff | 17:56 |
| mwhahaha | though it seems there is a permissions difference between non-pacemaker and pacemaker | 17:56 |
| mwhahaha | https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/cinder/cinder-volume-container-puppet.yaml#L303-L339 | 17:56 |
| *** d34dh0r53 has quit IRC | 17:57 | |
| openstackgerrit | David Peacock proposed openstack/tripleo-operator-ansible master: add role to show tripleo validation https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/755375 | 17:57 |
| mwhahaha | https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/cinder/cinder-volume-pacemaker-puppet.yaml#L154-L177 | 17:57 |
| mwhahaha | though a permission denied by tooz seems like there's an inconsistency between users of containers maybe? | 17:57 |
| abishop | mwhahaha: issue is with /var/lib/cinder, and note that under pcmk cinder runs as root | 17:57 |
| mwhahaha | what creates /var/lib/cinder? | 17:58 |
| abishop | so standalone, which runs as 'cinder' (no pmck in standalone train) is missing kolla's extend_start that fixes ownership of /var/lib/cinder | 17:58 |
| mwhahaha | i don't think so? | 17:59 |
| abishop | mwhahaha: https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/cinder/cinder-common-container-puppet.yaml#L109 | 17:59 |
| abishop | https://opendev.org/openstack/kolla/src/branch/master/docker/cinder/cinder-volume/extend_start.sh#L4 | 18:00 |
| mwhahaha | so permissions needs to be set in teh kolla_config | 18:00 |
| mwhahaha | in order for that to do it | 18:00 |
| abishop | ownership | 18:00 |
| mwhahaha | wtf why is that there | 18:00 |
| abishop | before tcib, I think kolla handled that | 18:00 |
| mwhahaha | that should be in the kolla_config in the service | 18:00 |
| abishop | seems reasonable, but I still can't figure out how this worked before (why are we hitting this now?) | 18:02 |
| *** derekh has quit IRC | 18:02 | |
| openstackgerrit | Merged openstack/paunch stable/train: Fix option to not cleanup containers not in config https://review.opendev.org/c/openstack/paunch/+/763145 | 18:02 |
| mwhahaha | were we not testing it? | 18:03 |
| *** hakhande has quit IRC | 18:03 | |
| mwhahaha | that would be my guess | 18:03 |
| mwhahaha | because we should be hitting it in master | 18:03 |
| abishop | I think in master it works because master runs standalone c-vol under pcmk (train does not) | 18:04 |
| mwhahaha | shouldn't matter | 18:04 |
| weshay|ruck | mwhahaha, comes out in tempest results.. those failing tests are now skipped | 18:04 |
| mwhahaha | we don't mount in /var/lib/cinder do we? | 18:04 |
| mwhahaha | that might also be an issue | 18:04 |
| mwhahaha | oh there it is | 18:05 |
| mwhahaha | it's in cinder-common | 18:05 |
| mwhahaha | so yea, i would throw a permissions block into the services for /var/lib/cinder and /etc/iscsi | 18:05 |
| mwhahaha | thought is /etc/iscsi shared with ironic? | 18:05 |
| abishop | I'll try permissions block for cinder. I think iscsi may be ok because only writer is iscsid which runs as root | 18:07 |
| mwhahaha | except https://opendev.org/openstack/kolla/src/branch/master/docker/cinder/cinder-volume/extend_start.sh#L9-L14 | 18:07 |
| * mwhahaha shrugs | 18:07 | |
| *** d34dh0r53 has joined #tripleo | 18:08 | |
| openstackgerrit | Merged openstack/tripleo-ci master: Enable html report for content provider jobs https://review.opendev.org/c/openstack/tripleo-ci/+/767591 | 18:09 |
| openstackgerrit | Merged openstack/tripleo-common stable/train: Move away from "ss" and drop default verbose mode https://review.opendev.org/c/openstack/tripleo-common/+/766706 | 18:09 |
| openstackgerrit | Merged openstack/python-tripleoclient stable/victoria: Add Python3 victoria unit tests https://review.opendev.org/c/openstack/python-tripleoclient/+/764036 | 18:10 |
| *** sanjayu_ has quit IRC | 18:18 | |
| *** rlandy is now known as rlandy|brb | 18:19 | |
| slagle | cloudnull: is there a way to see our log messages from tripleo-common when called by the tripleo playbooks? | 18:45 |
| slagle | cloudnull: cli-update-deployment-plan.yaml uses the tripleo_passwords_rotate module, which imports tripleo_common, which logs a lot of messages | 18:46 |
| slagle | (for example) | 18:46 |
| mwhahaha | use --debug | 18:46 |
| mwhahaha | it should turn on the ansible debug | 18:46 |
| cloudnull | via module, it should be logged to its respective file from oslo, so I think there's a tripleo log in /var/log | 18:47 |
| cloudnull | what mwhahaha said it should go to stdout in debug | 18:47 |
| slagle | --debug does not do it. our custom callback plugin must not output stdout/stderr | 18:47 |
| *** jmasud has joined #tripleo | 18:50 | |
| *** jmasud has quit IRC | 18:54 | |
| *** jmasud has joined #tripleo | 18:55 | |
| *** jmasud has quit IRC | 18:55 | |
| *** rlandy|brb is now known as rlandy | 18:55 | |
| *** ayoung has joined #tripleo | 18:58 | |
| openstackgerrit | Merged openstack/tripleo-ansible master: Add role_net_map to expand roles output https://review.opendev.org/c/openstack/tripleo-ansible/+/764639 | 19:05 |
| openstackgerrit | Merged openstack/tripleo-ansible stable/victoria: [kernel] Updating facts before disabling NICs https://review.opendev.org/c/openstack/tripleo-ansible/+/768396 | 19:07 |
| mwhahaha | slagle: try --verbose and --debug | 19:24 |
| mwhahaha | it might be ansible-runner eating it | 19:25 |
| slagle | debug isn't getting passed to ansible runner | 19:34 |
| slagle | not sure that would fix it anyway, i'm trying a few things | 19:34 |
| mwhahaha | slagle: i think you can also ANSIBLE_DEBUG=True | 19:44 |
| mwhahaha | as another work around | 19:44 |
| mwhahaha | though there's a bug around that | 19:44 |
| mwhahaha | should be fixed in master tho | 19:44 |
| mwhahaha | https://review.opendev.org/q/Ib67b6bd69ed1c3f9fcd32552b6599cbe4ff478ce | 19:45 |
| *** jmasud has joined #tripleo | 19:51 | |
| *** jmasud has quit IRC | 19:57 | |
| *** efoley has quit IRC | 20:26 | |
| *** slaweq has quit IRC | 20:33 | |
| *** chem has quit IRC | 20:35 | |
| openstackgerrit | Merged openstack/python-tripleoclient stable/ussuri: Raise if no validation has been executed https://review.opendev.org/c/openstack/python-tripleoclient/+/759695 | 20:42 |
| slagle | mwhahaha: cloudnull : to fix the logging, it looks like we need to configure logging when we use tripleo-common as a library (previously this was done by mistral -or- tripleoclient) | 21:02 |
| mwhahaha | fun | 21:02 |
| slagle | and we also need to capture stdout/stderr in the ansible modules and return it as part of result | 21:02 |
| slagle | or, we can configure a log file | 21:02 |
| slagle | although that would be pretty disjointed have a tripleo.log and ansible.log | 21:02 |
| slagle | *to have | 21:02 |
| mwhahaha | ansible-runner has a custom call back that it uses, is it not capturing teh stdout/err? | 21:03 |
| slagle | it's not in the callback, we're losing it earlier in the module | 21:03 |
| slagle | when we import tripleo-common and call a function, it's the module's responsibility to set stdout/stderr in the result dict for the callback to do something with | 21:04 |
| mwhahaha | not certain if we can mix the log configs | 21:04 |
| slagle | i don't think we want to. i'd say we configure the python logging to go to stdout/stderr, and then just have ansible log and/or print that | 21:06 |
| mwhahaha | i wouldn't do that because that usually ends up being unreadable | 21:08 |
| mwhahaha | what specific logging where you looking for? | 21:08 |
| mwhahaha | maybe we should just better error handle? | 21:08 |
| slagle | it is readable if it's in stdout_lines, ansible will at least display that nicely | 21:10 |
| slagle | what i'm looking for is all the logging we have in tripleo-common :) | 21:10 |
| mwhahaha | but why? | 21:11 |
| mwhahaha | :D | 21:11 |
| mwhahaha | serious tho, what were you looking into anyway | 21:11 |
| slagle | it was one of the debug lines in generate_passwords | 21:11 |
| slagle | all this stuff has been added over the years for $reasons | 21:12 |
| slagle | not to mention debugging during development | 21:12 |
| slagle | you can't print or log | 21:13 |
| mwhahaha | there is no LOG calls in generate_passwords? | 21:13 |
| mwhahaha | that's why i'm asking because it seems like logging/debugging is lacking overall | 21:13 |
| mwhahaha | or were you adding calls in and could see what was being outputted | 21:14 |
| slagle | it is lacking | 21:14 |
| slagle | but there is still some | 21:14 |
| mwhahaha | right so i would actually propose that we nuke that code and put it in the module w/ actual logging than trying to fixup tripleo-common unless it's used elsewhere | 21:14 |
| mwhahaha | i don't like the fact that we use tripleo-common the way we do with the cli-* stuff | 21:15 |
| mwhahaha | it's such a hack | 21:15 |
| slagle | well, that's fine, it's another issue | 21:15 |
| slagle | for now, we can fix the logging we do have | 21:15 |
| slagle | it's a bad regression, and an easy fix | 21:15 |
| mwhahaha | famous last words | 21:17 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-tripleo master: Add authtoken security options https://review.opendev.org/c/openstack/puppet-tripleo/+/769349 | 21:17 |
| slagle | put it on my tombstone :) | 21:19 |
| mwhahaha | Also it appers you were talking about tripleo_common.utils.plan.generate_passwords, not tripleo_common.utils.generate_passwords | 21:20 |
| mwhahaha | the latter has 0 logging | 21:20 |
| openstackgerrit | Grzegorz Grasza proposed openstack/puppet-tripleo master: Add authtoken security options https://review.opendev.org/c/openstack/puppet-tripleo/+/769349 | 21:21 |
| slagle | that's right | 21:21 |
| slagle | plan.generate_passwords calls passwords.generate_passwords | 21:22 |
| mwhahaha | i wonder if you can just substitute the regular logger for the ansible one | 21:23 |
| slagle | the latter is used by both overcloud/tripleo deploy | 21:23 |
| mwhahaha | and have it magically do things correctly | 21:23 |
| slagle | if we get rid of tripleo deploy and have everything go through overcloud deploy, then we can move all the code into the module, if that's what we want to do long term | 21:24 |
| mwhahaha | i'd personally prefer the opposite but i know we spent a bunch of time on the ansibling of stuff | 21:24 |
| mwhahaha | i don't like the ansible cli bits specifically because of things like this but yea | 21:24 |
| mwhahaha | this isn't the first time that troubleshooting got harder | 21:25 |
| mwhahaha | introspection and stuff also is now hidden in ansible magic | 21:25 |
| slagle | what do you mean by "the opposite" | 21:25 |
| mwhahaha | tripleo deploy becomes the standard | 21:25 |
| *** chem has joined #tripleo | 21:26 | |
| slagle | my thinking was overcloud deploy uses heat-all, tripleo deploy can then be deprecated | 21:27 |
| slagle | reason being is that there's lots of "production" logic in overcloud deploy that tripleo deploy just ignores as not existing | 21:27 |
| mwhahaha | like what | 21:28 |
| slagle | well, plan.generate_passwords | 21:28 |
| mwhahaha | it doesnt' ignore it | 21:28 |
| mwhahaha | it expects there's a file that provides it | 21:28 |
| mwhahaha | we have logic that has that | 21:29 |
| slagle | there's a file for overcloud deploy as well, we just get it from the plan instead of reading it from a directory | 21:29 |
| slagle | but there's additional logic in plan.generate_passwords | 21:29 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/train: Remove Luna HSM clients on scaledown https://review.opendev.org/c/openstack/tripleo-heat-templates/+/767933 | 21:31 |
| openstackgerrit | Merged openstack/python-tripleoclient stable/train: Do not require authentication for the validator CLI https://review.opendev.org/c/openstack/python-tripleoclient/+/760905 | 21:31 |
| openstackgerrit | Merged openstack/tripleo-common master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-common/+/769259 | 21:31 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/victoria: Move cell_v2 discovery off compute hosts https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769138 | 21:31 |
| mwhahaha | slagle: ok well it seems like that's a small thing to adjust in the tripleo deploy to make it more configurable (e.g. plan dir instead of output dir) | 21:32 |
| slagle | since overcloud deploy is what has always been used, i'd prefer to start there, and whittle that down, instead of trying to build up parity in tripleo deploy | 21:32 |
| mwhahaha | so i think you can turn overcloud deploy into a wrapper around tripleo deploy | 21:33 |
| mwhahaha | similar to what we did with the undercloud install/upgrades | 21:33 |
| mwhahaha | there's logic that has to be handed in an 'overcloud' that's more than tripleo deploy so you can do that logic in that call and then just invoke tripleo deploy when you've built up the plan items correctly | 21:34 |
| mwhahaha | passwords are just parameters on a deployment so all that logic could just be ignored in tripleo-deploy if you're providing the passwords straight in from the overcloud command | 21:35 |
| mwhahaha | also adding a --skip-password-update to tripleo-deploy is pretty light weight | 21:36 |
| *** xek has quit IRC | 21:40 | |
| *** xek has joined #tripleo | 21:40 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769263 | 21:43 |
| cloudnull | slagle mwhahaha sorry was off eating. I wonder if we could just instantiate a log handler in our module-utils , maybe somewhere here - https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/ansible_plugins/module_utils/tripleo_common_utils.py#L16 - that way we have tripleo-common logs and ansible logs in separate files? | 21:43 |
| mwhahaha | you could but separate files isn't great | 21:43 |
| cloudnull | within the module until it'd ensure we've an appropriate log handler whenever we call tripleo-common things | 21:44 |
| openstackgerrit | Merged openstack/tripleo-operator-ansible master: Bump molecule https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/759413 | 21:44 |
| mwhahaha | makes it hard to correlate what's going on | 21:44 |
| cloudnull | I think it'd be worse mixing openstack and ansible logs in the same file? | 21:44 |
| cloudnull | that's kinda what we had before albeit from bash magic | 21:44 |
| mwhahaha | it's no different than the awfulness from ansible-playbook --debug | 21:45 |
| cloudnull | that's fair | 21:45 |
| slagle | i think we just need to configure logging for tripleo-common to a stream, then have ansible display that | 21:48 |
| cloudnull | if we are going to keep it all in the same ansible log file, couldn't we pull the ansible log handler and plumb it into the oslo tomfoolery? | 21:48 |
| mwhahaha | it would be nice if we could get the tripleo-common logging outputted as the module is running but I don't think it works like that | 21:49 |
| cloudnull | the problem with modules is that we wont capture the stdout/stderr | 21:49 |
| mwhahaha | that's why container image prepare goes to a file | 21:49 |
| mwhahaha | the executor in ansible eats that stuff i think | 21:50 |
| cloudnull | with action plugins we can trap stdout/error but modules are executed on the remote node and the stdout is almost all dropped, unless ANSIBLE_DEBUG is enabled. | 21:50 |
| mwhahaha | aren't these modules tho | 21:50 |
| mwhahaha | not action plugins | 21:51 |
| mwhahaha | ? | 21:51 |
| * mwhahaha checks | 21:51 | |
| mwhahaha | yea these are modules so it gets weird | 21:51 |
| cloudnull | yea its all modules | 21:52 |
| cloudnull | https://github.com/ansible/proposals/issues/33 this seems to have never been implemented | 21:54 |
| mwhahaha | i think the proposal was to write some code as a wrapper to collect the logging stream and just add it to the return values | 21:59 |
| *** rbrady has quit IRC | 21:59 | |
| mwhahaha | that'll work (tm) | 21:59 |
| mwhahaha | though not ideal, it does solve it for now | 22:00 |
| slagle | we can get the log messages from tripleo if we set ANSIBLE_DEBUG based on --debug, and configure a basic stream logger within tripleo-common | 22:00 |
| *** TrevorV has quit IRC | 22:00 | |
| slagle | then we don't have to capture stdout/stderr in the module, but the messages only go to the console as part of the ansbile debug output | 22:01 |
| slagle | b/c ansible doesn't send it's own debug to ansible.log (go figure) | 22:01 |
| mwhahaha | --debug should flip the verbosity on the ansible-runner invocation tho | 22:01 |
| mwhahaha | we have logic for that | 22:01 |
| slagle | --verbose sets ansible-runner verbosity, --debug may as well. but nothing sets ANSIBLE_DEBUG | 22:02 |
| slagle | which produces a different set of output | 22:02 |
| mwhahaha | ansible_debug is supposed to be that verbosity | 22:02 |
| mwhahaha | like it has to be a certain number or something | 22:02 |
| mwhahaha | trying to find the logic | 22:02 |
| mwhahaha | we really don't want --debug except in very specific instances and ANSIBLE_DEBUG should get inherited from the openstackcli command | 22:03 |
| mwhahaha | https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/utils.py#L221 | 22:03 |
| mwhahaha | if i recall verbose_level 5 is debug? | 22:04 |
| slagle | i'll try it with 5 v's :) | 22:05 |
| slagle | i'll just keep adding one and will report back | 22:05 |
| mwhahaha | :D | 22:05 |
| openstackgerrit | Shnaidman Sagi proposed openstack/tripleo-ansible stable/ussuri: Update for a podman_container from collection https://review.opendev.org/c/openstack/tripleo-ansible/+/769363 | 22:11 |
| openstackgerrit | Sofer Athlan-Guyot proposed openstack/tripleo-heat-templates stable/queens: DNM: Testing restart of ceph service. https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769393 | 22:11 |
| openstackgerrit | Sofer Athlan-Guyot proposed openstack/tripleo-heat-templates stable/queens: Ensure no rebalance during ceph update if docker is updated. https://review.opendev.org/c/openstack/tripleo-heat-templates/+/769169 | 22:11 |
| slagle | no amount of -v's produces the debug output that ANSIBLE_DEBUG=true produces | 22:11 |
| openstackgerrit | Shnaidman Sagi proposed openstack/tripleo-ansible stable/train: Update for a podman_container from collection https://review.opendev.org/c/openstack/tripleo-ansible/+/769364 | 22:11 |
| slagle | i mean, i could keep trying | 22:11 |
| mwhahaha | so if you do "ANSIBLE_DEBUG=True openstack foo" it doesn't work? | 22:12 |
| *** tosky_ has joined #tripleo | 22:12 | |
| slagle | you can see the logging with ANSIBLE_DEBUG=true | 22:12 |
| mwhahaha | i know i was having issues with the callback that ansible-runner uses to collect stuff which is why i said maybe that's where it's being eaten | 22:12 |
| mwhahaha | i guess --debug can set ANSIBLE_DEBUG since --debug is pretty unreadable as it is | 22:13 |
| *** tosky has quit IRC | 22:13 | |
| *** tosky_ is now known as tosky | 22:17 | |
| *** jamesdenton has quit IRC | 22:19 | |
| *** jamesden_ has joined #tripleo | 22:19 | |
| *** rfolco has quit IRC | 22:21 | |
| openstackgerrit | Shnaidman Sagi proposed openstack/tripleo-ansible stable/train: Update for a podman_container from collection https://review.opendev.org/c/openstack/tripleo-ansible/+/769364 | 22:24 |
| openstackgerrit | Merged openstack/tripleo-puppet-elements master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/769302 | 22:26 |
| openstackgerrit | Merged openstack/tripleo-upgrade master: Add doc/requirements https://review.opendev.org/c/openstack/tripleo-upgrade/+/769305 | 22:26 |
| openstackgerrit | Shnaidman Sagi proposed openstack/tripleo-ansible stable/train: Update for a podman_container from collection https://review.opendev.org/c/openstack/tripleo-ansible/+/769364 | 22:28 |
| openstackgerrit | Shnaidman Sagi proposed openstack/tripleo-ansible stable/ussuri: Update for a podman_container from collection https://review.opendev.org/c/openstack/tripleo-ansible/+/769363 | 22:33 |
| openstackgerrit | Jaganathan Palanisamy proposed openstack/tripleo-ansible master: NFV derive parameters molecule tests https://review.opendev.org/c/openstack/tripleo-ansible/+/754630 | 22:39 |
| ade_lee | cloudnull, mwhahaha https://review.opendev.org/c/openstack/tripleo-heat-templates/+/765007 and https://review.opendev.org/c/openstack/tripleo-heat-templates/+/765006 please | 22:54 |
| ade_lee | cloudnull, mwhahaha also https://review.opendev.org/c/openstack/puppet-tripleo/+/769356 and https://review.opendev.org/c/openstack/puppet-tripleo/+/769360 please | 22:55 |
| *** tkajinam has joined #tripleo | 23:01 | |
| *** tmazur has quit IRC | 23:05 | |
| mwhahaha | ade_lee: you'll get faster reviews if you add us as reviewers | 23:07 |
| mwhahaha | i check my email and generally approve stuff once CI has passed | 23:08 |
| ade_lee | mwhahaha, ack - I'll do that in future and suggest others in our group do so too | 23:08 |
| cloudnull | ++ | 23:11 |
| *** zzzeek has quit IRC | 23:25 | |
| *** zzzeek has joined #tripleo | 23:27 | |
| *** rcernin has joined #tripleo | 23:33 | |
| *** jmasud has joined #tripleo | 23:43 | |
| *** jmasud has quit IRC | 23:51 | |
| *** rcernin has quit IRC | 23:53 | |
| *** jmasud has joined #tripleo | 23:53 | |
| *** rcernin has joined #tripleo | 23:56 | |
| *** rcernin has quit IRC | 23:59 | |
| *** jmasud has quit IRC | 23:59 | |
| *** cloudnull has quit IRC | 23:59 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!