| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Fix wrong parameter mentioned by a deprecation warning https://review.opendev.org/c/openstack/puppet-octavia/+/905404 | 03:48 |
|---|---|---|
| opendevreview | Merged openstack/puppet-openstack-integration stable/2023.1: nova: Capture cell details https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905328 | 04:02 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Support (api_)db_commit_retry parameters https://review.opendev.org/c/openstack/puppet-octavia/+/905405 | 04:16 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Support (api_)db_commit_retry parameters https://review.opendev.org/c/openstack/puppet-octavia/+/905405 | 05:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Fix wrong audit class tested in unit tests https://review.opendev.org/c/openstack/puppet-octavia/+/905408 | 05:57 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Enforce appropriate depth for cert file https://review.opendev.org/c/openstack/puppet-octavia/+/905409 | 07:11 |
| opendevreview | Alfredo Moralejo proposed openstack/puppet-openstack-integration stable/yoga: Undefine ssl certificate parameters for mariadb https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905336 | 07:15 |
| opendevreview | Alfredo Moralejo proposed openstack/puppet-openstack-integration stable/xena: Undefine ssl certificate parameters for mariadb https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905337 | 07:16 |
| opendevreview | Alfredo Moralejo proposed openstack/puppet-openstack-integration stable/wallaby: Undefine ssl certificate parameters for mariadb https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905338 | 07:17 |
| opendevreview | Alfredo Moralejo proposed openstack/puppet-openstack-integration master: DNM test with ssl disabled with current conf https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905413 | 08:00 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Enforce appropriate depth for cert file https://review.opendev.org/c/openstack/puppet-octavia/+/905409 | 08:04 |
| amoralej | tkajinam, fyi, actually setting ssl = false should be enough since https://github.com/puppetlabs/puppetlabs-mysql/commit/6cfcdbdbc60222b928bef9f483e145f69572257a 13.1.0 which ignores ssl-* if ssl = false so i will restore your approach, anyway i'll test it with a dnm job setting ssl = false | 08:05 |
| tkajinam | amoralej, ahhh | 08:06 |
| tkajinam | ok | 08:06 |
| tkajinam | amoralej, does it mean we don't need any change if that change is present, right ? | 08:06 |
| amoralej | we probably need only the fix from zed backwards | 08:06 |
| amoralej | yes | 08:06 |
| tkajinam | ah, ok. | 08:07 |
| amoralej | tkajinam, https://review.rdoproject.org/zuul/builds?project=rdoinfo&pipeline=periodic&skip=0 | 08:08 |
| tkajinam | yeah stable/zed has puppetlabs-mysql 13.0.0 and stable/2023.1 has 13.2.0. That fix was merged in 13.1.0 | 08:08 |
| amoralej | those are periodic jobs we have | 08:08 |
| amoralej | exactly | 08:08 |
| amoralej | see failing since zed | 08:08 |
| amoralej | or https://prometheus.monitoring.softwarefactory-project.io/grafana/d/1cbR4SISz/rdo-puppet-promotions-pipeline?orgId=1&from=now-7d&to=now | 08:09 |
| tkajinam | ahh yes | 08:09 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration stable/zed: Undefine ssl certificate parameters for mariadb https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905311 | 08:13 |
| tkajinam | amoralej, I've updated the commit message to explain that fix in puppetlabs-mysql ^^^ thanks for these all inputs ! | 08:13 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration stable/yoga: Undefine ssl certificate parameters for mariadb https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905336 | 08:14 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration stable/xena: Undefine ssl certificate parameters for mariadb https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905337 | 08:14 |
| amoralej | thanks tkajinam++ | 08:16 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration stable/wallaby: Undefine ssl certificate parameters for mariadb https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905338 | 08:16 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Add support for [libvirt] migration_inbound_addr https://review.opendev.org/c/openstack/puppet-nova/+/905424 | 09:18 |
| amoralej | tkajinam, confirmed, it works fine without patch in master https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905413?tab=change-view-tab-header-zuul-results-summary failure in scenario005 is unrelated | 09:20 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Remove deprecated use_forwarded_for parameter https://review.opendev.org/c/openstack/puppet-nova/+/905425 | 09:21 |
| tkajinam | amoralej, thanks. so let's merge only the stable only patch | 09:21 |
| amoralej | yes | 09:21 |
| amoralej | tkajinam, btw, i'm using a different approach in packstack and it is also working https://review.opendev.org/c/x/packstack/+/905415/1/packstack/puppet/modules/packstack/manifests/mariadb.pp | 09:29 |
| tkajinam | amoralej, yeah, it would work unless you attempt to enable ssl. | 09:34 |
| amoralej | we don't support ssl in mysql in packstack, so good enough :) | 09:34 |
| tkajinam | yup | 09:35 |
| opendevreview | Merged openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905169 | 09:43 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Remove deprecated use_forwarded_for parameter https://review.opendev.org/c/openstack/puppet-nova/+/905425 | 11:22 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Enforce appropriate depth for cert file https://review.opendev.org/c/openstack/puppet-octavia/+/905409 | 13:26 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Restrict access to certificate files https://review.opendev.org/c/openstack/puppet-octavia/+/905439 | 13:33 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Restrict access to certificate files https://review.opendev.org/c/openstack/puppet-octavia/+/905439 | 13:34 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Restrict access to certificate files https://review.opendev.org/c/openstack/puppet-octavia/+/905439 | 13:36 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova stable/zed: Avoid restarting dbus service https://review.opendev.org/c/openstack/puppet-nova/+/905479 | 15:51 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova stable/yoga: Avoid restarting dbus service https://review.opendev.org/c/openstack/puppet-nova/+/905454 | 15:52 |
| opendevreview | Merged openstack/puppet-octavia master: Fix wrong parameter mentioned by a deprecation warning https://review.opendev.org/c/openstack/puppet-octavia/+/905404 | 17:37 |
| opendevreview | Merged openstack/puppet-swift master: Use ensure_resource to declare rsync::server https://review.opendev.org/c/openstack/puppet-swift/+/905185 | 17:42 |
| opendevreview | Merged openstack/puppet-swift master: Fix constraint related to dnspython https://review.opendev.org/c/openstack/puppet-swift/+/905186 | 17:42 |
| opendevreview | Merged openstack/puppet-octavia master: Support (api_)db_commit_retry parameters https://review.opendev.org/c/openstack/puppet-octavia/+/905405 | 17:43 |
| opendevreview | Merged openstack/puppet-octavia master: Fix wrong audit class tested in unit tests https://review.opendev.org/c/openstack/puppet-octavia/+/905408 | 17:43 |
| opendevreview | Merged openstack/puppet-octavia master: Restrict access to certificate files https://review.opendev.org/c/openstack/puppet-octavia/+/905439 | 17:43 |
| opendevreview | Merged openstack/puppet-openstack-integration stable/zed: Undefine ssl certificate parameters for mariadb https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905311 | 17:52 |
| opendevreview | Merged openstack/puppet-horizon master: Restart httpd after config change https://review.opendev.org/c/openstack/puppet-horizon/+/905123 | 17:53 |
| opendevreview | Merged openstack/puppet-swift master: Fix handling of cors options https://review.opendev.org/c/openstack/puppet-swift/+/905187 | 18:00 |
| opendevreview | Merged openstack/puppet-swift master: proxy: Fix typo in the dummy resource for pipeline validation https://review.opendev.org/c/openstack/puppet-swift/+/905188 | 18:01 |
| opendevreview | Merged openstack/puppet-openstack-integration stable/yoga: Undefine ssl certificate parameters for mariadb https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905336 | 18:06 |
| opendevreview | Merged openstack/puppet-openstack-integration stable/xena: Undefine ssl certificate parameters for mariadb https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905337 | 18:06 |
| opendevreview | Merged openstack/puppet-openstack-integration stable/wallaby: Undefine ssl certificate parameters for mariadb https://review.opendev.org/c/openstack/puppet-openstack-integration/+/905338 | 18:06 |
| opendevreview | Merged openstack/puppet-nova master: Add support for [libvirt] migration_inbound_addr https://review.opendev.org/c/openstack/puppet-nova/+/905424 | 18:06 |
| opendevreview | Merged openstack/puppet-nova master: Remove deprecated use_forwarded_for parameter https://review.opendev.org/c/openstack/puppet-nova/+/905425 | 18:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!