| opendevreview | Yamato Tanaka proposed openstack/tripleo-ansible master: fix to be able to edit ns_records of /etc/designate/pools.yaml https://review.opendev.org/c/openstack/tripleo-ansible/+/866978 | 00:27 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Cinder: Fix wrong internal parameter to enable Dell/EMC SC backend https://review.opendev.org/c/openstack/tripleo-heat-templates/+/867189 | 01:42 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove environment file to load removed template https://review.opendev.org/c/openstack/tripleo-heat-templates/+/867190 | 01:46 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Cinder: Remove implementation for Dell SC driver https://review.opendev.org/c/openstack/puppet-tripleo/+/867191 | 01:49 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo stable/wallaby: Enable policy rule management in nova-compute https://review.opendev.org/c/openstack/puppet-tripleo/+/824690 | 01:53 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/zed: Fix list_concat for Nfs mounts https://review.opendev.org/c/openstack/tripleo-heat-templates/+/867155 | 03:36 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/wallaby: Fix list_concat for Nfs mounts https://review.opendev.org/c/openstack/tripleo-heat-templates/+/867156 | 03:40 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible stable/wallaby: Hide output which can contain password strings https://review.opendev.org/c/openstack/tripleo-ansible/+/867157 | 03:42 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible stable/train: Hide output which can contain password strings https://review.opendev.org/c/openstack/tripleo-ansible/+/867192 | 03:43 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/train: DNM: Trigger Ceph job https://review.opendev.org/c/openstack/tripleo-heat-templates/+/867193 | 03:49 |
| *** yadnesh|away is now known as yadnesh | 04:34 | |
| chkumar|ruck | tkajinam: hello | 05:46 |
| chkumar|ruck | tkajinam: please have a look at this file https://bugs.launchpad.net/tripleo/+bug/1996933 | 05:46 |
| chkumar|ruck | when free, It is a promotion and gate blocker | 05:46 |
| tkajinam | that's not a code problem but CI problem, IMO | 05:47 |
| tkajinam | https://review.opendev.org/c/openstack/python-tripleoclient/+/861548 | 05:48 |
| tkajinam | we did remove usage of that file. as long as CI pulls this we should not face that error | 05:48 |
| chkumar|ruck | tkajinam: let me fix the CI | 05:50 |
| opendevreview | chandan kumar proposed openstack/python-tripleoclient master: [DNM] Debug undercloud-upgrade https://review.opendev.org/c/openstack/python-tripleoclient/+/867195 | 06:43 |
| tkajinam | chkumar|ruck, we probably need to check the rpm used in the job | 06:59 |
| chkumar|ruck | tkajinam: https://review.opendev.org/c/openstack/python-tripleoclient/+/867195 might give a clue | 06:59 |
| tkajinam | chkumar|ruck, I suspect somehow the package is built with wrong content but I have no idea how it can happen | 06:59 |
| tkajinam | yeah, though the package installed in that node looks correct, according to the package list in extra log | 06:59 |
| tkajinam | unless the job override the codes installed by the t-c package | 07:00 |
| tkajinam | Hear I mean tripleoclient , not tripleo-common | 07:00 |
| opendevreview | chandan kumar proposed openstack/python-tripleoclient master: [DNM] Debug undercloud-upgrade https://review.opendev.org/c/openstack/python-tripleoclient/+/867195 | 07:02 |
| opendevreview | chandan kumar proposed openstack/python-tripleoclient master: [DNM] Debug undercloud-upgrade https://review.opendev.org/c/openstack/python-tripleoclient/+/867195 | 07:15 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-pacemaker stable/1.1.x: Stable-only: Run stable/wallaby job https://review.opendev.org/c/openstack/puppet-pacemaker/+/866954 | 07:21 |
| ykarel | tkajinam, just to be sure if you saw my message last week about the upgrade issue? and ruled out the the problem that i pointed out? | 07:21 |
| opendevreview | Merged openstack/tripleo-ansible stable/zed: Hide output which can contain password strings https://review.opendev.org/c/openstack/tripleo-ansible/+/865903 | 07:22 |
| ykarel | https://meetings.opendev.org/irclogs/%23tripleo/%23tripleo.2022-12-08.log.html#t2022-12-08T12:17:51 | 07:22 |
| ykarel | reading above seems you talking the same issue so wondering if you checked the versioning part already | 07:23 |
| tkajinam | ykarel, ahh. I saw the conversation a bit later but I now understand what you mean | 07:31 |
| tkajinam | https://bugs.launchpad.net/tripleo/+bug/1996933/comments/11 | 07:32 |
| tkajinam | we probably have to bump subminor version for master until we get the next release ? | 07:33 |
| tkajinam | chkumar|ruck, ^^^ | 07:33 |
| tkajinam | so the problem is we use the same version for zed and master now. and if zed is promoted AFTER master is promoted then the packages are not updated because zed packages have "newer" date | 07:34 |
| ykarel | tkajinam, yes that's one way | 07:34 |
| ykarel | tkajinam, also tripleo packages are not part of promoter cycle, they just gets build and available in CI | 07:34 |
| ykarel | so as soon as new zed package get's build issue should be seen until new master package get's build | 07:35 |
| tkajinam | makes perfect sense | 07:35 |
| tkajinam | I misread the dnf log earlier but now I understand it does not contain the log record for update | 07:36 |
| ykarel | one way in past was used to have Semver api-break in commit message but that used to happen as soon as a realease used to happen | 07:36 |
| ykarel | not sure if it's fine to do now too after we are couple of weeks after the release | 07:36 |
| chkumar|ruck | tkajinam: on it | 07:41 |
| chkumar|ruck | tkajinam: https://review.opendev.org/c/openstack/releases/+/867196 19.1.0 version | 07:49 |
| chkumar|ruck | or 19.0.1 creating from last commit | 07:49 |
| chkumar|ruck | ? | 07:49 |
| tkajinam | we also need the tht fix https://review.opendev.org/c/openstack/tripleo-heat-templates/+/861547/ | 07:51 |
| tkajinam | which that tripleoclient change depends on | 07:51 |
| tkajinam | if we can bump release in spec files for all tripleo packages in master then that would be better. we can revert it back before the final Antelope release | 07:53 |
| *** amoralej|off is now known as amoralej | 07:54 | |
| chkumar|ruck | tkajinam: Does this will help https://review.opendev.org/c/openstack/releases/+/867196 ? | 07:57 |
| tkajinam | technically but we have to consider a proper version | 07:59 |
| tkajinam | we probably want major version bump here because minor version will be used for zed minor release. though I'm not sure the current release model allows us to create a release in stable branches | 08:00 |
| tkajinam | if that's not possible then that minor version bump would be good. though I tend to prefer more static solution instead of creating releases every time, if possible | 08:00 |
| tkajinam | we have seen a few problems caused by t-a and tht as well | 08:01 |
| chkumar|ruck | marios: please have a look at this one https://review.opendev.org/c/openstack/releases/+/867196 since you are our release expert | 08:02 |
| chkumar|ruck | context: in order to fix undercloud upgrade job https://bugs.launchpad.net/tripleo/+bug/1996933, we need a version bump of tht and tripleoclient so that undercloud upgrade can pass | 08:03 |
| tkajinam | amoralej, do you have an idea to prevent newer package version in zed ? | 08:04 |
| tkajinam | amoralej, currently we use the same version for zed and master and when zed is promoted after master then the package gets "newer" data and is not updated properly during upgrade job. wondering whether we can add the release -1 for all tripleo packages until we get an Antelope release | 08:05 |
| tkajinam | newer dat*e* I mean | 08:05 |
| marios | chkumar|ruck: yes thank you i saw that bug (and spoke briefly with ykarel last week who reminded me we had similar issue in the past) i wanted to dig on that on friday but family emergency so was out as u know | 08:05 |
| marios | chkumar|ruck: i searched for the older bug for context but couldn't find it | 08:06 |
| marios | tkajinam: so we can just bump the version for the new series (this is what we usually do) so we can cut releases during the A cycle if we need/want | 08:06 |
| marios | chkumar|ruck: will comment but i think we need to bump major version instaed (really this breaks semver but this is how we have always released and we use it to signal the new series) | 08:07 |
| chkumar|ruck | marios: thank you, let me udpate the patch with major version bump for all tripleo projects | 08:10 |
| tkajinam | ook | 08:12 |
| tkajinam | https://review.opendev.org/c/openstack/releases/+/771830 | 08:12 |
| tkajinam | so I see we did it for wallaby | 08:12 |
| tkajinam | and GA was all .1.0 https://review.opendev.org/c/openstack/releases/+/789558 | 08:13 |
| marios | chkumar|ruck: ack also include tripleo-common/validations i think (commented there ) | 08:19 |
| chkumar|ruck | marios: tkajinam included all the package https://review.opendev.org/c/openstack/releases/+/867196 | 08:26 |
| chkumar|ruck | for puppet-image-elements there is no new commit | 08:26 |
| chkumar|ruck | Do I need to create a new tag from previous commit? | 08:26 |
| marios | chkumar|ruck: may not want to do that since this is cix issue (include all packages i mean) | 08:30 |
| marios | chkumar|ruck: e.g. for puppet-tripleo, it means you also have to bump https://opendev.org/openstack/puppet-tripleo/src/commit/1461e26d7c20c98e4d6dcd09c25788fcfd89ce3b/metadata.json#L3 | 08:31 |
| marios | chkumar|ruck: we probably *should* have a new release for the A series for all the things, but it can also be a separate patch e.g. from ramishra | 08:31 |
| marios | chkumar|ruck: no don't create tag from previous commit (probably that would fail tox validation job anyway) | 08:31 |
| marios | chkumar|ruck: (i.e like that for puppet-tripleo) https://opendev.org/openstack/puppet-tripleo/commit/e40b7c4a230c72c5f0f43eaf174bfe82238c7f77 | 08:33 |
| marios | chkumar|ruck: so ... anyway ok to go with what you posted ... as long as we post puppet-tripleo first. | 08:36 |
| *** jpena|off is now known as jpena | 08:43 | |
| chkumar|ruck | marios: make sense to keep only cix related projects and I have removed rest of them | 08:44 |
| chkumar|ruck | marios: tkajinam thank you :-) | 08:45 |
| chkumar|ruck | marios: tox-validate job is failing with deliverables/_independent/python-tripleoclient.yaml: validate_branch_points: stable/zed branch exists in openstack/python-tripleoclient and does not seem to have been created from b5e59b562329aaa0975e406e4669d3dbbef68d41 | 09:06 |
| chkumar|ruck | https://1e2c404f763bd2116554-8408acfc1ca35a1df36c3c89e90ce648.ssl.cf1.rackcdn.com/867196/5/check/openstack-tox-validate/dac859a/job-output.txt | 09:06 |
| chkumar|ruck | so I need to take the commits from stable/zed | 09:07 |
| chkumar|ruck | not from master | 09:07 |
| marios | chkumar|ruck: no it needs to be from master ... worth asking in openstack-release channel about this not sure how we can get around it or perhaps validation needs to be altered to accomodate | 09:13 |
| chkumar|ruck | let me remove validations and see | 09:17 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-pacemaker stable/1.1.x: Stable-only: Run stable/wallaby job https://review.opendev.org/c/openstack/puppet-pacemaker/+/866954 | 09:23 |
| opendevreview | Roberto Alfieri proposed openstack/tripleo-ansible master: Changed the way how we gather interface and mtu info and fixed indentation inside block https://review.opendev.org/c/openstack/tripleo-ansible/+/867201 | 09:36 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: VF container security and stability adjustment https://review.opendev.org/c/openstack/validations-libs/+/854782 | 09:43 |
| opendevreview | Jesse Pretorius proposed openstack/tripleo-heat-templates stable/zed: DNM test bug/1998501 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/867026 | 09:55 |
| opendevreview | Jesse Pretorius proposed openstack/tripleo-upgrade stable/zed: [workload_launch] Update µversions for Wallaby & Zed https://review.opendev.org/c/openstack/tripleo-upgrade/+/867158 | 09:56 |
| opendevreview | Jesse Pretorius proposed openstack/tripleo-upgrade stable/zed: [workload_launch] Update µversions for Wallaby & Yoga https://review.opendev.org/c/openstack/tripleo-upgrade/+/867158 | 09:59 |
| opendevreview | Damien Ciabrini proposed openstack/tripleo-heat-templates master: HA: enable new container image name by default https://review.opendev.org/c/openstack/tripleo-heat-templates/+/862147 | 10:16 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: VF container security and stability adjustment https://review.opendev.org/c/openstack/validations-libs/+/854782 | 10:39 |
| lmiccini | hey folks is there anyone that can help me figure out why https://review.opendev.org/c/openstack/puppet-pacemaker/+/863375/ didn't trigger gate jobs? how can I fix this? | 10:46 |
| lmiccini | chkumar|ruck: marios ^ maybe you guys know? | 10:51 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-quickstart-extras master: Refactor a bit the script called to deploy external compute https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/866864 | 10:54 |
| marios | lmiccini: looks liek depends-on? | 10:56 |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Fix compatibility with Ansible 2.14 https://review.opendev.org/c/openstack/tripleo-ansible/+/865089 | 11:13 |
| *** rlandy|out is now known as rlandy | 11:16 | |
| *** dviroel_ is now known as dviroel|rover | 11:20 | |
| lmiccini | marios: ouch right | 11:24 |
| lmiccini | marios: do you think https://review.opendev.org/c/openstack/tripleo-ci/+/866722/ is ok as is or shall we keep train around? (not sure how) | 11:25 |
| *** anbanerj is now known as frenzy_friday | 11:35 | |
| *** pojadhav is now known as pojadhav|afk | 11:39 | |
| marios | ack will have a closer look on next reviews run | 11:47 |
| *** amoralej is now known as amoralej|lunch | 13:11 | |
| *** rlandy is now known as rlandy|brb | 13:17 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure firewall for standalone deploy https://review.opendev.org/c/openstack/tripleo-ansible/+/863146 | 13:21 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-pacemaker stable/1.1.x: Stable-only: Run stable/wallaby job https://review.opendev.org/c/openstack/puppet-pacemaker/+/866954 | 13:24 |
| *** rlandy|brb is now known as rlandy | 13:27 | |
| opendevreview | Jesse Pretorius proposed openstack/tripleo-heat-templates stable/zed: DNM test bug/1998501 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/867026 | 13:38 |
| *** pojadhav|afk is now known as pojadhav | 13:40 | |
| opendevreview | Soniya Murlidhar Vyas proposed openstack/openstack-tempest-skiplist master: Add command to delete old releases https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/865123 | 13:42 |
| jm1 | Tengu: hello :) did you have luck with ansible galaxy mirroring? | 13:45 |
| Tengu | jm1: not mirroring - but proxying | 13:45 |
| Tengu | jm1: there's apparently something I need to check with https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 13:45 |
| opendevreview | Martin Magr proposed openstack/tripleo-heat-templates master: Switch to Ansible for script download https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866799 | 13:46 |
| Tengu | but I had to put it aside for now, having some headaches with another patch. | 13:46 |
| jm1 | Tengu: i tried to do the same for aoc: https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867202/ | 13:46 |
| Tengu | jm1: you're missing the port. | 13:46 |
| Tengu | either 8085 for http, or 4448 for httpS | 13:47 |
| opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: Set tripleo-ci-centos-9-undercloud-upgrade to non-voting https://review.opendev.org/c/openstack/tripleo-ci/+/867224 | 13:57 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/wallaby: [Wallaby-only] Role-spec NovaComputeStartupDelay https://review.opendev.org/c/openstack/tripleo-heat-templates/+/867226 | 13:58 |
| opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: Set tripleo-ci-centos-9-undercloud-upgrade to non-voting https://review.opendev.org/c/openstack/tripleo-ci/+/867224 | 13:58 |
| *** Guest1665 is now known as dasm | 14:00 | |
| *** dasm is now known as Guest1892 | 14:01 | |
| *** Guest1892 is now known as dasm | 14:01 | |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/wallaby: [Wallaby-only] Role-spec NovaComputeStartupDelay https://review.opendev.org/c/openstack/tripleo-heat-templates/+/867226 | 14:05 |
| opendevreview | Merged openstack/tripleo-ansible stable/zed: Fix issues with connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/867012 | 14:08 |
| *** amoralej|lunch is now known as amoralej | 14:10 | |
| rlandy | slagle: hi - sent an email - wanted to know if you could join our community call tomorrow to talk about opendev and the zed/mater check/gate testing | 14:14 |
| slagle | rlandy: i can. at 1500utc? | 14:26 |
| rlandy | slagle: yes - thanks | 14:27 |
| rlandy | we'd just like your thoughts on which jobs are valuable and should be kept and run | 14:27 |
| slagle | can you send me an invite? | 14:27 |
| rlandy | slagle: invite sent | 14:29 |
| slagle | thanks | 14:31 |
| *** yadnesh is now known as yadnesh|away | 14:52 | |
| lmiccini | o/ marios: https://review.opendev.org/c/openstack/tripleo-ci/+/866722 zuul seems to be happy. thoughts on what to do with train? | 14:55 |
| fultonj | slagle: would you mind reviewing https://review.opendev.org/c/openstack/tripleo-ansible/+/865197 ? | 15:04 |
| slagle | fultonj: lgtm | 15:14 |
| fultonj | thanks slagle | 15:14 |
| *** dviroel|rover is now known as dviroel|rover|lunch | 15:22 | |
| opendevreview | Roberto Alfieri proposed openstack/tripleo-ansible master: Changed the way how we gather interface and mtu info and fixed indentation inside block https://review.opendev.org/c/openstack/tripleo-ansible/+/867201 | 15:25 |
| *** marios is now known as marios|out | 15:59 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure firewall for standalone deploy https://review.opendev.org/c/openstack/tripleo-ansible/+/863146 | 16:10 |
| *** dviroel|rover|lunch is now known as dviroel|rover | 16:10 | |
| *** hjensas is now known as hjensas|out | 16:11 | |
| cloudnull | ☕ happy Monday! 👋 | 16:20 |
| Tengu | hey cloudnull :) | 16:20 |
| cloudnull | 👋 Tengu | 16:21 |
| cloudnull | how's life? | 16:21 |
| Tengu | like a Monday with snow, cold and no heater | 16:21 |
| Tengu | what about you? | 16:21 |
| cloudnull | its Monday, raining. | 16:29 |
| cloudnull | there's probably not enough coffee in my house. so a minor crisis | 16:30 |
| Tengu | ^^ | 16:30 |
| Tengu | at least we do have enough coffee - else I'd be frozen on my chair. | 16:30 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-common stable/wallaby: Remove sysfsutils from nova container https://review.opendev.org/c/openstack/tripleo-common/+/867163 | 16:33 |
| cloudnull | > at least we do have enough coffee - else I'd be frozen on my chair. | 16:34 |
| cloudnull | ROFL | 16:34 |
| lmiccini | o/ rlandy any chance someone from your team could review https://review.opendev.org/c/openstack/tripleo-ci/+/866722? seems like we need this to unblock puppet-pacemaker for wallaby (https://review.opendev.org/c/openstack/puppet-pacemaker/+/863375/ and the likes) | 16:35 |
| lmiccini | jcapitao[m]: ^ fyi | 16:35 |
| rlandy | lmiccini: sure - will take a look after meeting | 16:35 |
| lmiccini | thanks! | 16:35 |
| rlandy | dviroel|rover: ^^ can you give you eyes to lmiccini review? | 16:35 |
| rlandy | thanks | 16:35 |
| dviroel|rover | i will take a look | 16:38 |
| rlandy | lmiccini: does this failure have any relevance to your change: https://logserver.rdoproject.org/22/866722/1/openstack-check/tripleo-ci-centos-9-ovb-3ctlr_1comp-featureset001-wallaby/e972f14/logs/undercloud/home/zuul/undercloud_install.log.txt.gz? | 16:48 |
| lmiccini | rlandy: FATAL | make sure certmonger is installed -> "Failed to download packages: certmonger-0.79.14-5.el9.x86_64 | 16:48 |
| lmiccini | I wouldn't say so | 16:48 |
| rlandy | ok - then you're ok | 16:49 |
| dviroel|rover | lmiccini: https://review.opendev.org/c/openstack/puppet-pacemaker/+/866954/3..5#message-edd04b210e9a4ddf6d5585bf417c6713168aba73 | 16:49 |
| lmiccini | dviroel|rover: thanks, that's a different review tho.. I am happy to fix it but I'd need to figure out how :) | 16:54 |
| dviroel|rover | lmiccini: just wondering if we merge tripleo-ci one, it will work for stable/1.1.x | 16:56 |
| dviroel|rover | lmiccini: i see that there is a depends-on there | 16:56 |
| lmiccini | I'd do https://review.opendev.org/c/openstack/tripleo-ci/+/866722 -> https://review.opendev.org/c/openstack/puppet-pacemaker/+/863375/ and then figure out 1.1 after that. unless I am missing something? | 16:58 |
| dviroel|rover | lmiccini: ok, np, i was considering that you had both working already, 1.5 and 1.1 | 17:02 |
| rlandy | https://review.opendev.org/c/openstack/tripleo-ci/+/866722/1/roles/run-test/templates/common_vars.bash.j2 really only impacts what tripleo-quickstart will run | 17:03 |
| rlandy | lmiccini: you have +2s from dviroel|rover and me | 17:03 |
| rlandy | you can w+ when ready | 17:03 |
| rlandy | lunch - brb | 17:03 |
| lmiccini | I don't have +w powers :) just trying to sort out this branching madness, any help will be appreciated :D | 17:06 |
| opendevreview | Luca Miccini proposed openstack/puppet-pacemaker stable/1.1.x: Stable-only: Run stable/wallaby job https://review.opendev.org/c/openstack/puppet-pacemaker/+/866954 | 17:06 |
| lmiccini | ok this should hopefully do, at least zuul is not complaining anymore | 17:08 |
| rlandy | lmiccini: ok w+'ed for you | 17:15 |
| rlandy | https://review.opendev.org/c/openstack/tripleo-ci/+/866722 | 17:15 |
| lmiccini | \o/ | 17:15 |
| *** amoralej is now known as amoralej|off | 17:29 | |
| *** jpena is now known as jpena|off | 17:32 | |
| opendevreview | Manojkatari proposed openstack/tripleo-ansible master: Rotate cephadm_command.log https://review.opendev.org/c/openstack/tripleo-ansible/+/867232 | 18:21 |
| opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: Remove reference to IBM BM2 provider https://review.opendev.org/c/openstack/tripleo-ci/+/867276 | 18:32 |
| csibbitt | Hi all, I would appreciate some help in getting this patch merged. It has been up and passing verification since Nov 24th. I recently corrected a minor paperwork concern with the commit message, but I'm fairly certain there should be no further objection so I'd really like to get this to the next step before the end of the year. https://review.opendev.org/c/openstack/tripleo-ansible/+/865601 | 19:05 |
| fultonj | csibbitt: I gave it a +2 . It will need a second +2 since it's not a clean cherry pick | 19:14 |
| csibbitt | Thanks! | 19:15 |
| csibbitt | As for "clean cherry-pick". Yeah, sorry. It's really a 1-character fix that happened to get mixed in with molecule fixes to get it through the gate back when it was first discovered. The confidence and safety on the 1-character fix is very high, and the molecule fixes speak for themselves since they pass CI. I'll try to keep the concerns separated in the future | 19:20 |
| fultonj | "clean cherry-pick" just means that there were no conflicts | 19:21 |
| fultonj | it's ok if there are conflicts and you resolve them | 19:22 |
| fultonj | they look resolved | 19:23 |
| csibbitt | Gotcha, okay thanks. :) | 19:29 |
| -opendevstatus- NOTICE: Gerrit will be unavailable for a short time as it is upgraded to the 3.6 release | 20:00 | |
| opendevreview | Merged openstack/tripleo-puppet-elements stable/train: Move tuned_profiles_cpu_partitioning dependency in overcloud-base https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/863156 | 20:28 |
| opendevreview | Julia Kreger proposed openstack/tripleo-common stable/train: Remove tftp healthcheck from ironic-pxe image https://review.opendev.org/c/openstack/tripleo-common/+/867290 | 21:05 |
| *** dviroel|rover is now known as dviroel|rover|out | 21:28 | |
| opendevreview | Merged openstack/tripleo-ci master: Match puppet-pacemaker branches to respective releases https://review.opendev.org/c/openstack/tripleo-ci/+/866722 | 21:31 |
| *** dasm is now known as dasm|off | 21:34 | |
| opendevreview | Merged openstack/tripleo-ci master: Set tripleo-ci-centos-9-undercloud-upgrade to non-voting https://review.opendev.org/c/openstack/tripleo-ci/+/867224 | 21:35 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!