opendevreview | Iury Gregory Melo Ferreira proposed openstack/sushy stable/2023.1: Handle non-default language for registries https://review.opendev.org/c/openstack/sushy/+/876912 | 01:42 |
---|---|---|
rpittau | good morning ironic! o/ | 07:53 |
jssfr | good morning ironic! o/ | 07:54 |
jssfr | we're back in installing on-prem racks with Ironic, and there's lots of praise from those who first touch it :) | 07:54 |
jssfr | (coming from a "attach Ubuntu ISO to server using BMC web ui and run through the installer" world) | 07:55 |
dtantsur | nice! :) | 08:13 |
iurygregory | good morning Ironic | 11:50 |
opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic-inspector/+/876968 | 13:37 |
opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic-inspector/+/876969 | 13:37 |
opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/ironic-inspector/+/876970 | 13:37 |
iurygregory | spam time, bot release | 13:45 |
dtantsur | that's all good spam :) | 13:52 |
iurygregory | yeah | 13:53 |
dtantsur | iurygregory: if you have a minute: https://review.opendev.org/c/openstack/bifrost/+/875929 | 13:54 |
iurygregory | I do have, but it will be late for 2023.1 =X | 13:54 |
iurygregory | Merged openstack/releases master: Release bifrost 16.1.0 and branch stable/2023.1 https://review.opendev.org/c/openstack/releases/+/876567 | 13:55 |
dtantsur | well, fine, then we backport it | 13:55 |
opendevreview | OpenStack Release Bot proposed openstack/bifrost stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/bifrost/+/876975 | 13:58 |
opendevreview | OpenStack Release Bot proposed openstack/bifrost stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/bifrost/+/876976 | 13:58 |
opendevreview | OpenStack Release Bot proposed openstack/bifrost master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/bifrost/+/876977 | 13:58 |
opendevreview | Verification of a change to openstack/ironic-inspector stable/2023.1 failed: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic-inspector/+/876969 | 13:59 |
dtantsur | this ^^^ is really weird | 14:00 |
iurygregory | yeah, the change is merged... the one zuul was complaining | 14:00 |
dtantsur | It's not merged, it's stuck with verified +2 | 14:01 |
iurygregory | ohh | 14:01 |
iurygregory | right, I saw verified +2 so I just assumed it was merged | 14:01 |
iurygregory | maybe the promoting job is running | 14:01 |
dtantsur | I assumed it runs after the merge.. | 14:01 |
* iurygregory is looking at status | 14:01 | |
opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/876978 | 14:09 |
opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/876979 | 14:09 |
opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/876980 | 14:09 |
iurygregory | looks like is stuck O.o | 14:12 |
dtantsur | iurygregory: #opendev? | 14:14 |
iurygregory | yeah, will raise this there | 14:17 |
iurygregory | let me check how it looks for our other projects | 14:17 |
iurygregory | seems like we are not the only ones with this problem | 14:20 |
iurygregory | "it's something to do with the switch to submit requirements in preparation for upgrading to gerrit 3.7, i've just about exhausted my troubleshooting options so am asking the broader gerrit community for suggestions while waiting for people here who have a better grasp of the feature in order to discuss how to roll back the changes" | 14:25 |
iurygregory | ok, we should probably hold trying to merge things it seems | 14:25 |
* TheJulia awaits the coffee to appear and thinks “maybe it was too early to get up” | 14:34 | |
ebbex | could some cores help push this over the finish line? https://review.opendev.org/c/openstack/bifrost/+/855806 | 14:36 |
TheJulia | Coffee first :) https://usercontent.irccloud-cdn.com/file/nTk5WQ8D/1678372693.JPG | 14:38 |
iurygregory | coffee ++ | 14:40 |
iurygregory | seems like zuul is a bit weird today, not sure if it will affect all patches or not (release patches are getting merged without problems) | 14:41 |
*** sfinucan is now known as stephenfin | 14:44 | |
opendevreview | Lon Hohberger proposed openstack/ironic-inspector master: tests: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/ironic-inspector/+/876988 | 15:23 |
opendevreview | Merged openstack/bifrost master: Fix enabling epel repo for rpm distributions https://review.opendev.org/c/openstack/bifrost/+/875929 | 15:30 |
kubajj | TheJulia: do you think you'll manage to look at the email before Monday? | 15:46 |
kubajj | sorry to bother you again, but I might need to start writing up the evaluation rather soon | 15:47 |
opendevreview | Merged openstack/ironic-inspector master: tests: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/ironic-inspector/+/876988 | 15:56 |
dtantsur | rpittau: I assume we better backport this ^^? | 16:05 |
rpittau | yep | 16:05 |
rpittau | to infinity and beyond | 16:05 |
dtantsur | \o/ | 16:05 |
rpittau | dtantsur: lon is actually backporting that downstream, but better safe than sorry (it's been a while I haven't said that, feel nice) | 16:06 |
rpittau | let me start the backports chain | 16:06 |
opendevreview | Riccardo Pittau proposed openstack/ironic-inspector stable/2023.1: tests: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/ironic-inspector/+/877010 | 16:07 |
opendevreview | Riccardo Pittau proposed openstack/ironic-inspector stable/zed: tests: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/ironic-inspector/+/877011 | 16:07 |
rpittau | mmm | 16:07 |
rpittau | looks like we don't need a lot of backports, the test hass been added recently | 16:07 |
dking | What setting in Ironic tells it where to pull the IPA images from? on an older system I have it in pxelinux.cfg/default, but in my newer one, I don't see it being set anywhere. Is there a default? | 16:08 |
opendevreview | Riccardo Pittau proposed openstack/ironic-inspector bugfix/11.3: tests: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/ironic-inspector/+/877012 | 16:08 |
rpittau | yep, should be all | 16:09 |
dtantsur | dking: it's deploy_kernel/ramdisk either in ironic.conf or on the node. pxelinux.cfg/default is a consequence, not the cause | 16:16 |
dking | dtantsur: Thank you. I'm still hunting down where that is set in my current config. We're using Metal3, so I'm trying to see how they transfer info from configMaps, etc. | 16:17 |
Sandzwerg[m] | <dking> "What setting in Ironic tells..." <- deploy_kernel and deploy_ramdisk on node or conductor level. Btw can we make it more clear in the documentation that it's also possible to set this in the conductor? If one only looks at the install/enrollment page https://docs.openstack.org/ironic/zed/install/enrollment.html#enrollment-process in step 4 it only mentions the node level settings without a reference to the conductor. | 16:19 |
dtantsur | dking: in case of metal3, it's an environment variable you provide to baremetal-operator | 16:20 |
dtantsur | the default metal3 (its upstream variation) uses CS9 images from RDO master | 16:20 |
dking | BTW, I'm looking more for the inspection/cleaning image. What I have set on the node seems to work, but not during cleaning | 16:21 |
dtantsur | dking: I'd expect baremetal-operator to override whatever you set manually | 16:21 |
dtantsur | the variables: https://github.com/metal3-io/baremetal-operator/blob/main/docs/configuration.md | 16:22 |
dking | So, I'm setting `DEPLOY_KERNEL_URL` in the configMap for baremetal-operator, which does seem to make it into the node. However, for new nodes, at least, they seem to download an image from some other URL. | 16:23 |
dtantsur | dking: can you see the old links on the nodes? | 16:24 |
dking | In the web server, I see `${uuid}/deploy_kernel ..}' with a different md5sum from what I can download from the `deploy_kernel` URL | 16:27 |
dtantsur | dking: does your server provide correct Last Modified header (or none at all)? | 16:27 |
dking | Yes, the 'Last-Modified' HTTP header looks correct to me. | 16:28 |
dtantsur | dking: then you need to grep for image_cache logs in ironic. Check its decisions on whether to download/update things. | 16:29 |
dtantsur | (I assume you or whatever tooling you use did restart baremetal-operator with the new variables) | 16:31 |
dking | I see "Starting clean up for master image cache /shared/html/master_images", which seems fine as /master_images is empty. | 16:34 |
opendevreview | OpenStack Release Bot proposed openstack/ironic stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic/+/876998 | 16:41 |
opendevreview | OpenStack Release Bot proposed openstack/ironic stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic/+/876999 | 16:41 |
opendevreview | OpenStack Release Bot proposed openstack/ironic master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/ironic/+/877000 | 16:41 |
opendevreview | Merged openstack/bifrost master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/bifrost/+/876977 | 17:14 |
rpittau | good night! o/ | 17:18 |
opendevreview | Merged openstack/ironic-inspector master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/ironic-inspector/+/876970 | 17:18 |
-opendevstatus- NOTICE: Yesterday's change to Gerrit configs to use submit-requirements had a boolean logic bug. This has now been corrected and any changes that did not merge as a result can be rechecked. We have reenqueued the changes we identified as being affected. | 17:23 | |
opendevreview | Merged openstack/ironic-inspector stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic-inspector/+/876968 | 17:31 |
opendevreview | Merged openstack/sushy stable/2023.1: Handle non-default language for registries https://review.opendev.org/c/openstack/sushy/+/876912 | 17:41 |
opendevreview | Merged openstack/metalsmith stable/yoga: list_instances - cache allocations https://review.opendev.org/c/openstack/metalsmith/+/876671 | 18:51 |
opendevreview | Merged openstack/metalsmith stable/xena: list_instances - cache allocations https://review.opendev.org/c/openstack/metalsmith/+/876672 | 18:51 |
opendevreview | Verification of a change to openstack/metalsmith stable/wallaby failed: list_instances - cache allocations https://review.opendev.org/c/openstack/metalsmith/+/876673 | 19:01 |
opendevreview | Merged openstack/ironic stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic/+/876998 | 20:28 |
opendevreview | Merged openstack/ironic stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic/+/876999 | 20:33 |
opendevreview | Steve Baker proposed openstack/metalsmith stable/zed: test_provision: don't assert provision_node call order https://review.opendev.org/c/openstack/metalsmith/+/877039 | 20:35 |
opendevreview | Steve Baker proposed openstack/metalsmith stable/yoga: test_provision: don't assert provision_node call order https://review.opendev.org/c/openstack/metalsmith/+/877040 | 20:36 |
opendevreview | Steve Baker proposed openstack/metalsmith stable/xena: test_provision: don't assert provision_node call order https://review.opendev.org/c/openstack/metalsmith/+/877041 | 20:38 |
opendevreview | Steve Baker proposed openstack/metalsmith stable/wallaby: test_provision: don't assert provision_node call order https://review.opendev.org/c/openstack/metalsmith/+/877042 | 20:39 |
stevebaker[m] | Here are some metalsmith stable CI reliability backports ^^ | 20:40 |
opendevreview | Merged openstack/ironic-inspector stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/ironic-inspector/+/876969 | 22:23 |
opendevreview | Merged openstack/ironic-python-agent stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/876978 | 22:35 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!