*** amoralej|off is now known as amoralej | 06:06 | |
opendevreview | Dr. Jens Harbott proposed openstack/kolla master: drop infra_image_prefix https://review.opendev.org/c/openstack/kolla/+/837114 | 06:22 |
---|---|---|
mnasiadka | morning | 06:50 |
mnasiadka | mgoddard, hrw, frickler: I can't do the Wed weekly meeting this time - does any of you want to run it, or should I cancel? | 06:52 |
frickler | mnasiadka: I'll also be on PTO from this wed to next, so fine with cancelling | 06:53 |
*** ChanServ changes topic to "IRC meetings on Wednesdays @ 14:00 UTC - agenda @ https://goo.gl/OXB0DL | Whiteboard: https://bit.ly/2MM7mWF | IRC channel is *LOGGED* @ http://goo.gl/3mzZ7b" | 06:55 | |
frickler | updated the meeting time | 06:56 |
mnasiadka | thanks | 07:00 |
hrw | https://review.opendev.org/c/openstack/kolla/+/836577 waits for brave cores | 07:08 |
hrw | +552 -3160 one so takes some time | 07:08 |
yoctozepto | morning | 07:37 |
opendevreview | Merged openstack/kolla master: drop infra-rename support https://review.opendev.org/c/openstack/kolla/+/836576 | 07:37 |
yoctozepto | frickler: I wonder if it's the new zuul but it seems dependent patches are enqueued in the gate only after the parents actually merge: https://review.opendev.org/c/openstack/kolla/+/837114 | 07:50 |
frickler | yoctozepto: no, in that case at least the reason is that the patch had to be rebased and then it had to collect a V+1 before entering gate. there was some discussion regarding this clean-check policy recently, but afaict it is configured per tenant, so not something we as a project could change | 08:12 |
yoctozepto | frickler: argh, I hate gerrit hiding comment entries :D thanks for the explanation, I simply missed the fact it was rebased (because gerrit hid it from me) | 08:13 |
frickler | yoctozepto: yeah, I'd also be grateful if there were a way to make the "show all comments" the default in my settings | 08:22 |
opendevreview | Michal Nasiadka proposed openstack/kayobe master: nclu-switch: always use become https://review.opendev.org/c/openstack/kayobe/+/837264 | 08:40 |
opendevreview | Verification of a change to openstack/kayobe master failed: ironic: default to ipxe booting https://review.opendev.org/c/openstack/kayobe/+/836203 | 09:03 |
hrw | wow. just a week and we are out of binary images | 09:07 |
hrw | morning | 09:07 |
opendevreview | Maksim Malchuk proposed openstack/kayobe master: Fix variable name for stackhpc.os-networks upper constraints https://review.opendev.org/c/openstack/kayobe/+/837201 | 09:14 |
opendevreview | Merged openstack/kolla master: drop infra_image_prefix https://review.opendev.org/c/openstack/kolla/+/837114 | 09:21 |
yoctozepto | hrw: let's hope we handle other priorities the same ;D | 09:31 |
yoctozepto | hrw: we need your input on https://review.opendev.org/c/openstack/kolla-ansible/+/834512 | 09:35 |
opendevreview | Merged openstack/kolla master: docker: drop binary parts https://review.opendev.org/c/openstack/kolla/+/836577 | 09:39 |
yoctozepto | yay! | 09:40 |
mgoddard | so long binary | 09:41 |
hrw | yoctozepto: will do | 09:42 |
hrw | sent to Kevin | 09:45 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: horizon: binary images are gone https://review.opendev.org/c/openstack/kolla/+/837277 | 09:48 |
hrw | ok, replied to question about project which I do not use since 2013 (answer should help) and can go back to kolla | 09:56 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: Drop qdrouterd image https://review.opendev.org/c/openstack/kolla/+/837141 | 10:05 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: drop add_binary_source_envs() macro https://review.opendev.org/c/openstack/kolla/+/836575 | 10:09 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: [WIP] Move to CentOS Stream 9 https://review.opendev.org/c/openstack/kolla/+/836664 | 10:16 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: for Zed cycle minimal Python version is 3.8 https://review.opendev.org/c/openstack/kolla/+/837282 | 10:17 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: watcher: drop binary parts https://review.opendev.org/c/openstack/kolla/+/837283 | 10:28 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: cyborg-agent: no need to check for install_type https://review.opendev.org/c/openstack/kolla/+/837284 | 10:29 |
opendevreview | Merged openstack/kayobe master: [release] Use OpenStack Yoga release https://review.opendev.org/c/openstack/kayobe/+/836040 | 10:29 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: drop install_type from image names https://review.opendev.org/c/openstack/kolla/+/837285 | 10:31 |
hrw | kolla/debian-source-nova-compute -> kolla/debian-nova-compute | 10:31 |
yoctozepto | hrw: tbh, I was thinking about following the majority of projects out there and moving the distro dimensionality to the version tag, i.e. kolla/nova-compute:zed-debian | 10:35 |
yoctozepto | this way we also have fewer repos in the image registry and it looks more manageable this way | 10:35 |
hrw | yoctozepto: free to send patch. looks good | 10:36 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: drop install_type almost everywhere https://review.opendev.org/c/openstack/kolla/+/837287 | 10:36 |
hrw | after that patch: | 10:36 |
hrw | --type INSTALL_TYPE, -t INSTALL_TYPE | 10:36 |
hrw | Ignored, kept for script compatibility. Allowed values: source | 10:36 |
yoctozepto | ++ | 10:37 |
hrw | ok, next cleanup is merging RUN lines | 10:39 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: CI: no need to set install_type https://review.opendev.org/c/openstack/kolla-ansible/+/837291 | 10:42 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: drop install_type from image names https://review.opendev.org/c/openstack/kolla-ansible/+/837293 | 10:54 |
Fl1nt | Hi there! | 10:55 |
opendevreview | Merged openstack/kayobe stable/xena: Restore forgotten linuxbridge-agent container https://review.opendev.org/c/openstack/kayobe/+/837197 | 10:55 |
opendevreview | Merged openstack/kayobe stable/wallaby: Restore forgotten linuxbridge-agent container https://review.opendev.org/c/openstack/kayobe/+/837198 | 10:56 |
opendevreview | Merged openstack/kayobe stable/victoria: Restore forgotten linuxbridge-agent container https://review.opendev.org/c/openstack/kayobe/+/837199 | 10:56 |
Fl1nt | I've an issue with gerrit, when I do a git review, it complain about the publickey missing, BUT when I just do a ssh <user>@review.openstack.org it works fine... I'm on Windows 11, does it matter? | 10:57 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: drop list of images unbuildable for source type https://review.opendev.org/c/openstack/kolla/+/837294 | 10:57 |
Fl1nt | my ssh .config file is working fine when I do a ssh command | 10:57 |
Fl1nt | it just looks like if git review (git remote update gerrit) isn't actually using the ssh config file. | 10:58 |
hrw | no idea how ssh work on windows | 11:02 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: drop install_type almost everywhere https://review.opendev.org/c/openstack/kolla/+/837287 | 11:02 |
hrw | yoctozepto: deprecation note added | 11:03 |
yoctozepto | ack | 11:04 |
opendevreview | Merged openstack/kolla-ansible master: Ironic: Support both plain PXE and iPXE https://review.opendev.org/c/openstack/kolla-ansible/+/832159 | 11:06 |
opendevreview | Merged openstack/kolla-ansible master: Ironic: rename containers https://review.opendev.org/c/openstack/kolla-ansible/+/832134 | 11:06 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: CI: drop 'source' from job names https://review.opendev.org/c/openstack/kolla/+/837295 | 11:07 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible master: [release] Switch to stable/yoga branch of openstack.yoga collection https://review.opendev.org/c/openstack/kolla-ansible/+/837296 | 11:08 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/yoga: [release] Switch to stable/yoga branch of openstack.yoga collection https://review.opendev.org/c/openstack/kolla-ansible/+/837298 | 11:09 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/yoga: Ironic: Support both plain PXE and iPXE https://review.opendev.org/c/openstack/kolla-ansible/+/837068 | 11:09 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: drop install_type almost everywhere https://review.opendev.org/c/openstack/kolla/+/837287 | 11:09 |
hrw | yoctozepto: fixed | 11:09 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: drop install_type from image names https://review.opendev.org/c/openstack/kolla-ansible/+/837293 | 11:10 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/yoga: Ironic: rename containers https://review.opendev.org/c/openstack/kolla-ansible/+/837069 | 11:11 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/yoga: [release] Deploy yoga images https://review.opendev.org/c/openstack/kolla-ansible/+/837299 | 11:11 |
yoctozepto | mnasiadka: https://review.opendev.org/c/openstack/kolla-ansible/+/834511 | 11:12 |
opendevreview | Mark Goddard proposed openstack/kolla stable/yoga: [release] centos: switch to CentOS cloud SIG packages https://review.opendev.org/c/openstack/kolla/+/837300 | 11:15 |
mnasiadka | yoctozepto, mgoddard: are we backporting the Ironic work to Y? | 11:15 |
yoctozepto | mnasiadka: yes | 11:15 |
mgoddard | It's fine by me | 11:15 |
mnasiadka | ok, just asking | 11:15 |
mnasiadka | (to stop asking in Gerrit) :D | 11:15 |
mnasiadka | yoctozepto: done | 11:16 |
yoctozepto | mnasiadka: you got your answer in gerrit :D | 11:16 |
hrw | ok, https://review.opendev.org/c/openstack/kolla-ansible/+/837293 + https://review.opendev.org/c/openstack/kolla/+/837285 need proper ordering | 11:20 |
hrw | one drops install_type from image name in k-a, other in k | 11:20 |
yoctozepto | and probably something inbetween to actually allow us to merge at all | 11:20 |
yoctozepto | I think this and job renames are less urgent | 11:21 |
hrw | yep | 11:22 |
mgoddard | you would think it was a critical bug, the way binary drop has been reviewed :) | 11:22 |
mnasiadka | yoctozepto: got lost in all the threads in my mailbox :) | 11:23 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: CI: drop 'source' from build job names https://review.opendev.org/c/openstack/kolla/+/837295 | 11:26 |
hrw | only kolla ones | 11:26 |
hrw | shit. wrong | 11:26 |
yoctozepto | mnasiadka: :-) | 11:27 |
yoctozepto | mgoddard: indeed :-) | 11:27 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: CI: drop 'source' from build job names https://review.opendev.org/c/openstack/kolla/+/837295 | 11:29 |
hrw | now proper | 11:29 |
frickler | Fl1nt: might be a known issue with gerrit and newer openssh. workaround is to use ed25519 instead of rsa. did you add -p 29418 when testing? | 11:30 |
opendevreview | Mark Goddard proposed openstack/kolla stable/yoga: [release] centos: switch to CentOS cloud SIG packages https://review.opendev.org/c/openstack/kolla/+/837300 | 11:38 |
opendevreview | Mark Goddard proposed openstack/kayobe-config master: [release] Synchronise with kayobe changes for Yoga https://review.opendev.org/c/openstack/kayobe-config/+/836041 | 11:40 |
hrw | frickler, Fl1nt: I have 4 rsa and 1 ed25519 key | 11:41 |
opendevreview | Mark Goddard proposed openstack/kayobe-config-dev master: [release] Sync with kayobe changes for Yoga https://review.opendev.org/c/openstack/kayobe-config-dev/+/836042 | 11:41 |
hrw | debug1: Server accepts key: hrw@puchatek.local ED25519 SHA256:fK40l1Ul6e6TVTjW/+5rhzl6uJeq963WATHgZ5q4ir4 agent | 11:42 |
hrw | Authenticated to review.opendev.org ([199.204.45.33]:29418) using "publickey". | 11:42 |
hrw | debug1: Offering public key: /home/marcin/.ssh/id_rsa RSA SHA256:tehRPSRjGOy7/Pfq06+sljOc+VhVa/eqEMK4A0ZlL6g agent | 11:42 |
hrw | debug1: send_pubkey_test: no mutual signature algorithm | 11:42 |
hrw | time to drop rsa ones | 11:42 |
frickler | hrw: it will be fixed with the next gerrit release hopefully | 11:42 |
hrw | frickler: for me it is fine - forcing me to recheck where I hve old key | 11:43 |
opendevreview | Merged openstack/kolla-ansible master: drop qdrouterd support https://review.opendev.org/c/openstack/kolla-ansible/+/837142 | 11:52 |
Fl1nt | mgoddard, hrw ok, thanks a lot, I'll generate an ed based one so :D | 11:58 |
hrw | Fl1nt: ssh -v helps | 11:59 |
Fl1nt | Yep, bug when I did it nothing special happened except a close bye bye 127 error code :( Not really anything explicit. | 11:59 |
Fl1nt | s/bug/but | 12:00 |
hrw | Fl1nt: paste output? | 12:02 |
Fl1nt | don't bother, I've just generated an ED based key and it works. | 12:03 |
hrw | https://review.opendev.org/c/openstack/kolla/+/837282 - who adds +W? | 12:04 |
hrw | (I prefer not to +W own patches) | 12:04 |
yoctozepto | hrw: added | 12:04 |
yoctozepto | but I would not mind if you did after 2CR+2 | 12:05 |
Fl1nt | done | 12:05 |
hrw | https://review.opendev.org/c/openstack/kolla/+/837287 https://review.opendev.org/c/openstack/kolla/+/837294 wait for !yoctozepto | 12:06 |
yoctozepto | hrw: and for zuul | 12:07 |
Fl1nt | Are you rather using git rebase --preserve-merge or --rebase-merges ? | 12:09 |
opendevreview | Gaël THEROND proposed openstack/kolla-ansible master: Add Grafana anonymous access: * Disable by default. * Introduce two new vars. https://review.opendev.org/c/openstack/kolla-ansible/+/837302 | 12:12 |
opendevreview | Gaël THEROND proposed openstack/kolla-ansible master: Add Grafana anonymous access: * Disable by default. * Introduce two new vars. https://review.opendev.org/c/openstack/kolla-ansible/+/837302 | 12:13 |
hrw | Fl1nt: I use 'git up' == 'git pull --rebase --autostash' | 12:15 |
hrw | no special setup for rebase options | 12:15 |
Fl1nt | ok, thx | 12:16 |
hrw | Fl1nt: but when I have patch series to rebase then it is always a list of patches not list of merges | 12:16 |
Fl1nt | I keep forgeting gerrit rules all the time... So many different rules all over the internet with repo management :D | 12:16 |
hrw | https://review.opendev.org/c/openstack/kolla-ansible/+/837302 is -1 for commit message itself | 12:17 |
hrw | without even looking inside | 12:17 |
Fl1nt | Ok, what do I need to fix on it ? | 12:18 |
hrw | one line \n\ndescription of change | 12:18 |
Fl1nt | We use only one line description now ? | 12:19 |
hrw | https://cbea.ms/git-commit/ | 12:19 |
Fl1nt | Hum... let me argue that this rule is grammatically shitty: https://cbea.ms/git-commit/#end developers that don't give a shit about langage rules is kinda funny... | 12:22 |
yoctozepto | language | 12:22 |
yoctozepto | and also - the explanation is there - it is better to avoid the period in that one case | 12:23 |
Fl1nt | I'll edit the commit MSG no worry. | 12:23 |
opendevreview | Gaël THEROND proposed openstack/kolla-ansible master: Add Grafana anonymous access https://review.opendev.org/c/openstack/kolla-ansible/+/837302 | 12:24 |
hrw | Add Grafana anonymous access | 12:25 |
hrw | \n | 12:25 |
Fl1nt | there is a return, I'm using review UI. | 12:25 |
hrw | okok | 12:26 |
Fl1nt | ^^ I mean, if even our tools aren't doing what we expect it's kinda weird ^^ | 12:26 |
hrw | commit message has two parts: oneline info + 'why' | 12:26 |
hrw | Fl1nt: 'git commit .' + editor does the job here | 12:27 |
*** amoralej is now known as amoralej|lunch | 12:28 | |
Fl1nt | Doing the patch within the editor of course, but editing/republishing is easier on the UI, which is already open and doesn't need new command and branch switch as I'm working on another one :D | 12:28 |
opendevreview | Gaël THEROND proposed openstack/kolla-ansible master: Add Grafana anonymous access https://review.opendev.org/c/openstack/kolla-ansible/+/837302 | 12:30 |
Fl1nt | Is that a clearer topic message? | 12:30 |
hrw | much better | 12:33 |
Fl1nt | ok, perfect. | 12:33 |
hrw | thanks | 12:33 |
Fl1nt | Sure, you're welcome | 12:33 |
opendevreview | Merged openstack/kayobe master: Fix variable name for stackhpc.os-networks upper constraints https://review.opendev.org/c/openstack/kayobe/+/837201 | 12:45 |
opendevreview | Maksim Malchuk proposed openstack/kayobe stable/xena: Fix variable name for stackhpc.os-networks upper constraints https://review.opendev.org/c/openstack/kayobe/+/837305 | 12:46 |
opendevreview | Maksim Malchuk proposed openstack/kayobe stable/wallaby: Fix variable name for stackhpc.os-networks upper constraints https://review.opendev.org/c/openstack/kayobe/+/837306 | 12:46 |
opendevreview | Maksim Malchuk proposed openstack/kayobe stable/victoria: Fix variable name for stackhpc.os-networks upper constraints https://review.opendev.org/c/openstack/kayobe/+/837307 | 12:47 |
opendevreview | Verification of a change to openstack/kolla-ansible master failed: Deprecate enable_ironic_ipxe https://review.opendev.org/c/openstack/kolla-ansible/+/834511 | 12:51 |
*** amoralej|lunch is now known as amoralej | 13:14 | |
opendevreview | Pierre Riteau proposed openstack/kayobe master: Update documentation link for NCLU https://review.opendev.org/c/openstack/kayobe/+/837335 | 13:20 |
opendevreview | Merged openstack/kayobe-config master: [release] Synchronise with kayobe changes for Yoga https://review.opendev.org/c/openstack/kayobe-config/+/836041 | 13:23 |
opendevreview | Mark Goddard proposed openstack/kayobe master: proxy: add ansible issue 8743 workaround https://review.opendev.org/c/openstack/kayobe/+/837340 | 13:38 |
opendevreview | Mark Goddard proposed openstack/kayobe-config master: proxy: add ansible issue 8743 workaround https://review.opendev.org/c/openstack/kayobe-config/+/837342 | 13:39 |
opendevreview | Mark Goddard proposed openstack/kayobe-config-dev master: [release] Sync with kayobe changes for Yoga https://review.opendev.org/c/openstack/kayobe-config-dev/+/836042 | 13:39 |
Fl1nt | hrw, let say I want to add a link for doc reference on a releasenote for a new feature of a service, do I just put it on the yaml file or is there a specific syntax? | 13:44 |
hrw | to have a link in a releasenote which will point to doc reference? or reverse? | 13:45 |
Fl1nt | A release note that point to doc | 13:46 |
opendevreview | Gaël THEROND proposed openstack/kolla-ansible master: Add Grafana SMTP support https://review.opendev.org/c/openstack/kolla-ansible/+/837345 | 13:48 |
Fl1nt | like this one for instance | 13:49 |
hrw | tbh I do not remember much of reno markup | 13:56 |
hrw | usually check other notes to find hints | 13:56 |
Fl1nt | Let's wait for others review then ^^ | 13:57 |
hrw | Fl1nt: you can add comment on link to point where you have problem | 13:57 |
Fl1nt | yep | 13:57 |
hrw | `Link description <link>` | 13:59 |
Fl1nt | ok will fix | 13:59 |
hrw | releasenotes/notes/network-mode-host-2aca256e2a1fbf68.yaml: since `Wallaby <https://review.opendev.org/c/openstack/kolla-ansible/+/751795>`__ | 14:00 |
Fl1nt | even if it's longer than 80chars ? | 14:00 |
hrw | grep -r http releasenotes | 14:00 |
hrw | ;D | 14:00 |
yoctozepto | Fl1nt: before you add other changes - the proposals are invalid since they can already be achieved with current code - in a cleaner and more flexible way | 14:00 |
hrw | even | 14:00 |
opendevreview | Gaël THEROND proposed openstack/kolla-ansible master: Add Grafana SMTP support https://review.opendev.org/c/openstack/kolla-ansible/+/837345 | 14:00 |
yoctozepto | grafana.ini provided by you will be merged | 14:00 |
Fl1nt | ok, fixed then :D | 14:01 |
Fl1nt | yeah but having to only add new vars is easier for operators | 14:01 |
yoctozepto | Fl1nt: docs are for such examples | 14:02 |
Fl1nt | the whole merging mechanism on kolla is kinda obscur for beginners we have OPS that constantly complain about that. | 14:02 |
yoctozepto | we have the principle of not owning config if we don't have to | 14:02 |
yoctozepto | Fl1nt: well, that's the way of kolla, the thing we love about it and thus care for | 14:02 |
Fl1nt | AUTH and SMTP feature enablement can be argued as mostly used everywhere. | 14:03 |
Fl1nt | for something else sure | 14:03 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: drop install_type from image names https://review.opendev.org/c/openstack/kolla-ansible/+/837293 | 14:04 |
hrw | rebased | 14:04 |
Fl1nt | isn't part of the kolla purpose to ease ops life? I mean, who's interested in knowing ini/yaml syntax and specifics of a tool anymore? Sure people like us but L1/L2 that are basic OPS from enterprise don't. | 14:05 |
Fl1nt | and company aren't interested to get them trained extensively. | 14:06 |
Fl1nt | so that a chicken and egg issue. | 14:06 |
hrw | Fl1nt: then they can hire highly paid consultants from Initech | 14:06 |
Fl1nt | Most company will hire trained profil like us to specify/design/architect the thing, then they want all the push button works to be offloaded to less skilled people. | 14:07 |
Fl1nt | basically, you design the thing, automate it through CICD jobs and then let the less technically skilled guys to push the button and enable feature they want/need with as few constraints and complexity as possible. | 14:09 |
hrw | yeah. | 14:09 |
Fl1nt | I'm just pushing patch that go this way tbh. | 14:09 |
hrw | I maintain ~20 jenkins jobs at Linaro | 14:09 |
Fl1nt | he he he :D | 14:09 |
Fl1nt | We do pretty much everything through Jenkins+Ansible because of that | 14:10 |
hrw | most of time do nothing with them or +2/merge someones patches now | 14:10 |
hrw | Fl1nt: jenkins job builder plugin rocks | 14:10 |
Fl1nt | I even worked on a company wide adopted ansible framework. | 14:10 |
hrw | jobs as yaml files | 14:10 |
Fl1nt | We use pipeline with Groovy/Json but yeah, pretty much the same in the end :D | 14:11 |
hrw | the good part - jobs are in git, go through gerrit. no one break them in jenkins and take no responsibility | 14:13 |
Fl1nt | yep all our repo got a .ci file normalized with .ci/build .ci/deploy .ci/run and that's it. | 14:14 |
hrw | and I have a deal with jenkins overlords that I can +2/submit anything related to my jobs without waiting for admins. my jobs == my problem when break something | 14:14 |
Fl1nt | men, what's the point of kolla if it's to just tell people, do whatever you want throw your config file and we just take care of orchestration? | 14:20 |
Fl1nt | I mean, Jenkins jobs and plain ETL tools do just that too perfectly. | 14:21 |
Fl1nt | Kolla brings value to the OPS | 14:21 |
opendevreview | Merged openstack/kayobe-config master: proxy: add ansible issue 8743 workaround https://review.opendev.org/c/openstack/kayobe-config/+/837342 | 14:34 |
opendevreview | Mark Goddard proposed openstack/kayobe master: Rework network interface to host mapping https://review.opendev.org/c/openstack/kayobe/+/825832 | 14:35 |
hrw | Fl1nt: for me kolla allowed to drop in-house nasty solution | 14:38 |
Fl1nt | Yeah, I like the way you can just drop downstream patch your own way etc I'm not against INI, but if we can bring less and less things to take care to the end user by just adding default vars and not even enable by default service, it's easier. | 14:39 |
Fl1nt | @yoctozepto, so no patch for SMTP Grafana, what about the RO/Anonymous AUTH part? | 14:44 |
hrw | imho it is a question 'where to stop' | 14:45 |
hrw | adding 'enable anon grafana access' is one flag. smtp setup probably few more etc. | 14:45 |
Fl1nt | the same apply with Openstack services tho, why do we bother with SAML/OpenID then as it's ok to just drop it on downstream :D | 14:45 |
Fl1nt | and let basic keystone auth do the job | 14:46 |
Fl1nt | but I actually I'm on your side about the, what's the limit. | 14:46 |
hrw | maybe it is good to have one place to give smtp data and then reuse it in all services which need it | 14:50 |
Fl1nt | aaaaah that! That is a solution that I like/ | 14:50 |
opendevreview | Merged openstack/kolla master: horizon: binary images are gone https://review.opendev.org/c/openstack/kolla/+/837277 | 14:50 |
opendevreview | Merged openstack/kolla master: drop add_binary_source_envs() macro https://review.opendev.org/c/openstack/kolla/+/836575 | 14:50 |
opendevreview | Merged openstack/kolla master: Drop qdrouterd image https://review.opendev.org/c/openstack/kolla/+/837141 | 14:51 |
opendevreview | Merged openstack/kolla master: watcher: drop binary parts https://review.opendev.org/c/openstack/kolla/+/837283 | 14:51 |
opendevreview | Merged openstack/kolla master: cyborg-agent: no need to check for install_type https://review.opendev.org/c/openstack/kolla/+/837284 | 14:51 |
opendevreview | Merged openstack/kolla master: for Zed cycle minimal Python version is 3.8 https://review.opendev.org/c/openstack/kolla/+/837282 | 14:51 |
opendevreview | Verification of a change to openstack/kolla master failed: Adjust permissions on _extend_start files. https://review.opendev.org/c/openstack/kolla/+/836212 | 14:51 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: [WIP] Move to CentOS Stream 9 https://review.opendev.org/c/openstack/kolla/+/836664 | 14:59 |
hrw | rebased to drop conflicts | 14:59 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: [WIP] Move to CentOS Stream 9 https://review.opendev.org/c/openstack/kolla/+/836664 | 15:00 |
yoctozepto | Fl1nt: also out; as long as something does not require kolla ansible to handle it specifically - we avoid adding support for that in k-a; SAML/IdP required hooks inside of the k-a - your example is just about new sections in the ini file ;-) | 15:10 |
* hrw out | 15:13 | |
Fl1nt | ok ok | 15:14 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: Adjust permissions on _extend_start files. https://review.opendev.org/c/openstack/kolla/+/836212 | 15:22 |
opendevreview | Merged openstack/kolla-ansible master: Deprecate enable_ironic_ipxe https://review.opendev.org/c/openstack/kolla-ansible/+/834511 | 15:57 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/yoga: Deprecate enable_ironic_ipxe https://review.opendev.org/c/openstack/kolla-ansible/+/837312 | 16:06 |
opendevreview | Juan Pablo Suazo proposed openstack/kolla-ansible master: Configure Prometheus Pushgateway Options https://review.opendev.org/c/openstack/kolla-ansible/+/837371 | 16:13 |
opendevreview | Juan Pablo Suazo proposed openstack/kolla-ansible master: Configure Prometheus Pushgateway Options https://review.opendev.org/c/openstack/kolla-ansible/+/837371 | 16:19 |
opendevreview | Juan Pablo Suazo proposed openstack/kolla-ansible master: Configure Prometheus Pushgateway Options https://review.opendev.org/c/openstack/kolla-ansible/+/837371 | 16:20 |
opendevreview | Juan Pablo Suazo proposed openstack/kolla-ansible master: Configure Prometheus Pushgateway Options https://review.opendev.org/c/openstack/kolla-ansible/+/837371 | 16:22 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: [WIP] Move to CentOS Stream 9 https://review.opendev.org/c/openstack/kolla/+/836664 | 16:42 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: [WIP] Move to CentOS Stream 9 https://review.opendev.org/c/openstack/kolla/+/836664 | 16:57 |
opendevreview | James Kirsch proposed openstack/kolla master: Replace Certbot with Lego for Let's Encrypt container https://review.opendev.org/c/openstack/kolla/+/834211 | 17:04 |
opendevreview | Verification of a change to openstack/kayobe master failed: proxy: add ansible issue 8743 workaround https://review.opendev.org/c/openstack/kayobe/+/837340 | 17:08 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: drop install_type from image names https://review.opendev.org/c/openstack/kolla-ansible/+/837293 | 17:09 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible master: doc: do not mention install_type https://review.opendev.org/c/openstack/kolla-ansible/+/837382 | 17:10 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: Remove duplicate entry in UNBUILDABLE_IMAGES https://review.opendev.org/c/openstack/kolla/+/837383 | 17:12 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: [WIP] Move to CentOS Stream 9 https://review.opendev.org/c/openstack/kolla/+/836664 | 17:12 |
opendevreview | Radosław Piliszek proposed openstack/kolla master: Deprecate hacluster-pcs https://review.opendev.org/c/openstack/kolla/+/837384 | 17:14 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: [WIP] Move to CentOS Stream 9 https://review.opendev.org/c/openstack/kolla/+/836664 | 17:28 |
*** amoralej is now known as amoralej|off | 17:30 | |
opendevreview | Merged openstack/kayobe-config-dev master: [release] Sync with kayobe changes for Yoga https://review.opendev.org/c/openstack/kayobe-config-dev/+/836042 | 17:39 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: CI: drop install_type from build jobs https://review.opendev.org/c/openstack/kolla/+/837388 | 17:39 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: CI: drop install_type from build jobs https://review.opendev.org/c/openstack/kolla/+/837388 | 17:40 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: [WIP] Move to CentOS Stream 9 https://review.opendev.org/c/openstack/kolla/+/836664 | 17:40 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: [WIP] Move to CentOS Stream 9 https://review.opendev.org/c/openstack/kolla/+/836664 | 19:03 |
opendevreview | James Kirsch proposed openstack/kolla master: Replace Certbot with Lego for Let's Encrypt container https://review.opendev.org/c/openstack/kolla/+/834211 | 19:20 |
opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: Bump ansible-lint version to 6.* https://review.opendev.org/c/openstack/kolla-ansible/+/836913 | 19:29 |
opendevreview | Merged openstack/kolla master: Remove duplicate entry in UNBUILDABLE_IMAGES https://review.opendev.org/c/openstack/kolla/+/837383 | 20:03 |
opendevreview | Merged openstack/kayobe master: proxy: add ansible issue 8743 workaround https://review.opendev.org/c/openstack/kayobe/+/837340 | 23:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!