*** MentalSiege has quit IRC | 00:09 | |
janders | TheJulia thank you ( regarding https://review.opendev.org/c/openstack/bifrost/+/762998 ). I forgot to re-set workflow-1 after re-basing and then the second +2 came in from Shivanand, hence it merged before you could respond - apologies :) | 00:09 |
---|---|---|
TheJulia | lbotell: if you are willing, we have a troubleshooting.rst file in the admin docs | 00:09 |
TheJulia | janders: it happens, no worries | 00:10 |
* TheJulia is having all sorts of fun with policy testing | 00:10 | |
lbotell | TheJulia: I will take a look ASAP :) | 00:11 |
TheJulia | lbotell: thanks | 00:12 |
*** mushrushu has quit IRC | 00:19 | |
*** tosky has quit IRC | 00:23 | |
*** mushrushu has joined #openstack-ironic | 00:25 | |
*** anuradha1904 has quit IRC | 00:35 | |
janders | TheJulia ML heads-up regarding inspector/auto-discovery is out | 00:37 |
*** lifeless has quit IRC | 00:53 | |
*** lifeless has joined #openstack-ironic | 00:55 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP Write stub ACL test for every existing API call https://review.opendev.org/c/openstack/ironic/+/767445 | 00:59 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP Start populating existing policy tests https://review.opendev.org/c/openstack/ironic/+/768136 | 00:59 |
*** gyee has quit IRC | 01:25 | |
*** rh-jelabarre has quit IRC | 02:29 | |
*** mkrai has joined #openstack-ironic | 03:40 | |
*** mgoddard has joined #openstack-ironic | 04:08 | |
*** lbotell has quit IRC | 04:30 | |
*** ociuhandu has joined #openstack-ironic | 05:08 | |
*** ociuhandu has quit IRC | 05:13 | |
janders | rpittau|afk Do you know if there is any reason why https://review.opendev.org/c/openstack/sushy-tools/+/764341 hasn't merged yet, or does it just need reviewer attention? I remembered testing this around late November (and it worked) so I took it for granted it's already merged... and than wasted a day figuring out why my MAC-OOB patch stopped | 05:16 |
janders | working after reinstalling bifrost. I compared latest sushy code with your patch suspecting regression and realised I just need to drop the patch in... :) | 05:16 |
*** zzzeek has quit IRC | 05:42 | |
*** zzzeek has joined #openstack-ironic | 05:43 | |
*** zzzeek has quit IRC | 05:50 | |
*** zzzeek has joined #openstack-ironic | 05:52 | |
*** zzzeek has quit IRC | 05:59 | |
*** zzzeek has joined #openstack-ironic | 06:00 | |
*** zzzeek has quit IRC | 06:05 | |
*** zzzeek has joined #openstack-ironic | 06:11 | |
*** zzzeek has quit IRC | 06:18 | |
*** zzzeek has joined #openstack-ironic | 06:19 | |
*** zzzeek has quit IRC | 06:46 | |
*** zzzeek has joined #openstack-ironic | 06:49 | |
*** mkrai has quit IRC | 06:56 | |
*** abdysn has joined #openstack-ironic | 07:03 | |
*** zzzeek has quit IRC | 07:09 | |
*** zzzeek has joined #openstack-ironic | 07:12 | |
*** zzzeek has quit IRC | 07:21 | |
*** zzzeek has joined #openstack-ironic | 07:22 | |
*** tzumainn has quit IRC | 07:25 | |
*** tosin has joined #openstack-ironic | 07:27 | |
openstackgerrit | Jacob Anders proposed openstack/ironic master: Use OOB inspection to fetch MACs for IB inspection https://review.opendev.org/c/openstack/ironic/+/750943 | 07:38 |
openstackgerrit | Jacob Anders proposed openstack/ironic-inspector master: Use OOB inspection to fetch MACs for IB inspection https://review.opendev.org/c/openstack/ironic-inspector/+/758994 | 07:39 |
*** zzzeek has quit IRC | 07:50 | |
*** zzzeek has joined #openstack-ironic | 07:54 | |
*** mkrai has joined #openstack-ironic | 07:59 | |
*** rpittau|afk is now known as rpittau | 08:01 | |
rpittau | good morning ironic! o/ | 08:02 |
janders | good morning rpittau o/ | 08:02 |
arne_wiebalck | Good morning janders rpittau and Ironic! | 08:02 |
rpittau | janders arne_wiebalck hey! :) | 08:02 |
janders | good morning arne_wiebalck o/ | 08:03 |
rpittau | janders: unfortunately that patch was forgotten and neglected because of the lower-constraints singularity | 08:03 |
rpittau | That and the processors one | 08:04 |
janders | thank you rpittau. Should I lobby for reviews for these two? :) | 08:07 |
rpittau | yeah :) | 08:09 |
janders | will do :) | 08:09 |
*** Abdallahyas has joined #openstack-ironic | 08:09 | |
*** abdysn has quit IRC | 08:13 | |
*** zzzeek has quit IRC | 08:16 | |
*** zzzeek has joined #openstack-ironic | 08:19 | |
rpittau | there are actually a few sushy-tools patches that can be quickly reviewed if anyone has a minute https://review.opendev.org/q/project:openstack/sushy-tools+status:open | 08:26 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/victoria: Simplify injecting network data into an ISO image https://review.opendev.org/c/openstack/ironic/+/769194 | 08:30 |
rpittau | ^ this should be the fix for ironic-tempest-partition-uefi-redfish-vmedia in stable/victoria | 08:31 |
openstackgerrit | Merged openstack/ironic-inspector master: Update version of doc8 https://review.opendev.org/c/openstack/ironic-inspector/+/766896 | 08:37 |
*** dougsz has joined #openstack-ironic | 08:46 | |
*** zzzeek has quit IRC | 09:03 | |
*** zzzeek has joined #openstack-ironic | 09:04 | |
*** lucasagomes has joined #openstack-ironic | 09:10 | |
*** mkrai has quit IRC | 09:27 | |
*** ociuhandu has joined #openstack-ironic | 09:34 | |
*** ricolin has joined #openstack-ironic | 09:41 | |
*** zzzeek has quit IRC | 09:42 | |
*** zzzeek has joined #openstack-ironic | 09:43 | |
*** mkrai has joined #openstack-ironic | 09:46 | |
*** derekh has joined #openstack-ironic | 09:48 | |
*** zzzeek has quit IRC | 09:51 | |
*** zzzeek has joined #openstack-ironic | 09:53 | |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost bugfix/8.3: Remove lower-constraints job https://review.opendev.org/c/openstack/bifrost/+/769498 | 09:56 |
*** ociuhandu has quit IRC | 09:57 | |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost bugfix/8.3: Remove lower-constraints job https://review.opendev.org/c/openstack/bifrost/+/769498 | 09:58 |
*** zzzeek has quit IRC | 10:00 | |
*** zzzeek has joined #openstack-ironic | 10:00 | |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost bugfix/8.3: Remove lower-constraints job https://review.opendev.org/c/openstack/bifrost/+/769498 | 10:01 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost bugfix/8.3: Remove lower-constraints job https://review.opendev.org/c/openstack/bifrost/+/769498 | 10:02 |
rpittau | yay confusing ussuri & victoria | 10:02 |
*** ociuhandu has joined #openstack-ironic | 10:02 | |
*** mkrai has quit IRC | 10:06 | |
*** mkrai_ has joined #openstack-ironic | 10:06 | |
*** anuradha1904 has joined #openstack-ironic | 10:07 | |
openstackgerrit | Merged openstack/sushy-tools master: Adding status to ethernet interface template https://review.opendev.org/c/openstack/sushy-tools/+/764341 | 10:12 |
*** zzzeek has quit IRC | 10:14 | |
*** zzzeek has joined #openstack-ironic | 10:17 | |
openstackgerrit | Merged openstack/sushy-tools master: Add doc/requirements https://review.opendev.org/c/openstack/sushy-tools/+/769175 | 10:23 |
openstackgerrit | Merged openstack/sushy-tools master: Provide correct libvirt name/bus for virtual floppy https://review.opendev.org/c/openstack/sushy-tools/+/768361 | 10:23 |
openstackgerrit | Merged openstack/sushy-tools master: Adding basic support for processors schema https://review.opendev.org/c/openstack/sushy-tools/+/761909 | 10:29 |
*** dtantsur|afk is now known as dtantsur | 10:35 | |
*** zzzeek has quit IRC | 10:35 | |
dtantsur | morning ironic, happy new year! | 10:35 |
* dtantsur declares scrollback bankruptcy | 10:36 | |
dtantsur | iurygregory, rpittau, have my later comments clarified the usedevelop vs requirements situation? | 10:37 |
rpittau | hey dtantsur, happy new year :) | 10:37 |
rpittau | dtantsur: yep, thanks | 10:37 |
*** zzzeek has joined #openstack-ironic | 10:38 | |
dtantsur | cool | 10:38 |
dtantsur | it's super confusing, I've discovered it by trial-and-error | 10:38 |
rpittau | yeah, I didn't really get it until I saw your last comment and tried myself | 10:38 |
dtantsur | it's super confusing, I've discovered it by trial-and-error | 10:40 |
dtantsur | eeehmm, wrong window | 10:40 |
rpittau | deja vu | 10:41 |
dtantsur | I need to re-learn to use my tooling :D | 10:42 |
rpittau | :) | 10:47 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost bugfix/10.0: Remove lower-constraints job https://review.opendev.org/c/openstack/bifrost/+/769502 | 10:50 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost bugfix/10.0: Remove lower-constraints job https://review.opendev.org/c/openstack/bifrost/+/769502 | 10:51 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic bugfix/15.2: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic/+/769503 | 10:54 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic bugfix/15.2: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic/+/769503 | 10:55 |
openstackgerrit | Merged openstack/ironic-inspector-specs master: remove unicode from code https://review.opendev.org/c/openstack/ironic-inspector-specs/+/769293 | 10:55 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.3: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-inspector/+/769504 | 10:56 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.3: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-inspector/+/769504 | 10:56 |
openstackgerrit | Tosin Farai proposed openstack/bifrost master: Test new features added to baremetal_node https://review.opendev.org/c/openstack/bifrost/+/769495 | 10:57 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.5: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-inspector/+/769505 | 10:57 |
*** Abdallahyas has quit IRC | 10:58 | |
*** Abdallahyas has joined #openstack-ironic | 10:58 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.5: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-inspector/+/769505 | 10:58 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/6.3: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-python-agent/+/769506 | 11:00 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/6.3: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-python-agent/+/769506 | 11:00 |
janders | dtantsur welcome back, Happy New Year 2021! :) | 11:01 |
dtantsur | o/ | 11:01 |
*** mkrai_ has quit IRC | 11:03 | |
openstackgerrit | Merged openstack/python-ironicclient master: Move pep8 dependencies from test-requirements to tox.ini https://review.opendev.org/c/openstack/python-ironicclient/+/766908 | 11:10 |
*** Abdallahyas has quit IRC | 11:11 | |
*** ricolin has quit IRC | 11:17 | |
*** abdysn has joined #openstack-ironic | 11:18 | |
*** mkowalski has quit IRC | 11:20 | |
iurygregory | good morning Ironic o/ | 11:22 |
iurygregory | hey dtantsur yeah, I will update the patches | 11:22 |
*** mkowalski has joined #openstack-ironic | 11:30 | |
dtantsur | I'm going to request a sushy-tools release if nobody objects | 11:35 |
iurygregory | ++ | 11:37 |
* iurygregory is wondering if "nobody objects" equals to "someone else will request the release" | 11:43 | |
*** zaneb has quit IRC | 11:44 | |
viks____ | hi, i'm facing issues in logging in to the baremetal node. Below are the steps i followed. Can someone point me out what is going wrong? | 11:47 |
viks____ | https://www.irccloud.com/pastebin/UKx1caJa/ | 11:47 |
*** zaneb has joined #openstack-ironic | 11:48 | |
rpittau | dtantsur: talking about sushy-tools, when you have a minute the rdo patch is green | 11:53 |
dtantsur | ack | 11:57 |
*** ociuhandu has quit IRC | 11:59 | |
*** ociuhandu has joined #openstack-ironic | 12:00 | |
ajya | viks____: try this - pxe_append_params = rootpwd="$$1$$LTZOXtgv$$AcW90ATdayFZWX.XpjTc91" - no quotes around | 12:04 |
*** ociuhandu has quit IRC | 12:05 | |
priteau | viks____: pxe_append_params is only used when PXE booting, for example when booting the IPA kernel & ramdisk images. I assume your ubuntu image boots from disk, the kernel parameters will come from the on-disk grub config. | 12:05 |
priteau | Oh, you're using the baremetal element, not vm, so I guess this is partition images. | 12:07 |
priteau | viks____: if ajya's suggestion doesn't work, try using the devuser element first? | 12:08 |
viks____ | ajya: ok.. but are the above steps proper ? | 12:08 |
viks____ | priteau: sure.. | 12:08 |
ajya | viks____: looks ok, but this stood out. You can check what's actually passed in conductor logs and also in IPA logs (if get access to those) to see if that's expected. | 12:09 |
viks____ | i just wonder how deploy image know abt user image... i.e. i uploaded centos deploy image and ubuntu as user image... how it knows what user image to pick when i run `provide` command | 12:10 |
*** tosky has joined #openstack-ironic | 12:15 | |
dtantsur | folks, looking for a 2nd +2 on https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/767158 please | 12:23 |
dtantsur | then we can likely make a release | 12:23 |
*** ociuhandu has joined #openstack-ironic | 12:26 | |
*** ociuhandu has quit IRC | 12:26 | |
*** ociuhandu has joined #openstack-ironic | 12:27 | |
iurygregory | looking now | 12:27 |
*** ociuhandu has quit IRC | 12:28 | |
*** ociuhandu has joined #openstack-ironic | 12:29 | |
*** ociuhandu has quit IRC | 12:35 | |
*** zzzeek has quit IRC | 12:46 | |
janders | see you tomorrow Ironic o/ | 12:48 |
*** zzzeek has joined #openstack-ironic | 12:49 | |
dtantsur | ajya: hi! are there plans to implement counterparts for known_good_state/clear_job_queue clean steps for idrac+redfish? | 12:49 |
*** ociuhandu has joined #openstack-ironic | 12:51 | |
iurygregory | bye janders o/ | 12:52 |
ajya | dtantsur: yes - https://storyboard.openstack.org/#!/story/2007617 | 12:52 |
dtantsur | ajya: it's blocked on https://review.opendev.org/c/x/sushy-oem-idrac/+/721596/ right? | 12:53 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: CI: Remove l-c job, set ipa-b to 2.2.0 https://review.opendev.org/c/openstack/ironic-python-agent/+/767622 | 12:55 |
ajya | dtantsur: yes, working on it, should be finalized "soon" | 12:56 |
dtantsur | cool! | 13:00 |
dtantsur | I'd like to make metal3 clear the job queue after enrolling a Dell node | 13:01 |
*** ociuhandu has quit IRC | 13:02 | |
*** ociuhandu has joined #openstack-ironic | 13:07 | |
*** ociuhandu has quit IRC | 13:12 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/victoria: Document using ramdisks with the ramdisk deploy interface https://review.opendev.org/c/openstack/ironic/+/769508 | 13:14 |
*** paras333 has quit IRC | 13:18 | |
*** ociuhandu has joined #openstack-ironic | 13:33 | |
*** ociuhandu has quit IRC | 13:37 | |
*** ricolin has joined #openstack-ironic | 13:41 | |
*** tosin has quit IRC | 13:45 | |
viks____ | ajya: i could not find from where the image being picked from... it seems to be picking up some other images.. any idea what's going wrong? | 13:48 |
*** rloo has joined #openstack-ironic | 13:49 | |
ajya | viks____: IPA image or user image? IPA comes from `deploy_ramdisk` and user image from `image_source`. What's happening now? | 13:51 |
rpittau | great, this is passing now, just need a kind heart to review it :) https://review.opendev.org/c/openstack/python-ironicclient/+/767776 | 13:51 |
*** rloo has quit IRC | 13:51 | |
*** rloo has joined #openstack-ironic | 13:52 | |
rpittau | and one more vote needed to fix ironic CI in stable/victoria https://review.opendev.org/c/openstack/ironic/+/769194 | 13:52 |
*** ociuhandu has joined #openstack-ironic | 13:55 | |
*** rh-jelabarre has joined #openstack-ironic | 14:00 | |
viks____ | i'm running it along with openstack services, not as a standalone... i have uploaded the images as per https://docs.openstack.org/ironic/train/install/configure-glance-images.html where do i set `image_source` ? | 14:00 |
viks____ | i could not find to do that in : https://docs.openstack.org/ironic/train/install/enrollment.html | 14:02 |
*** rloo has quit IRC | 14:03 | |
*** rloo has joined #openstack-ironic | 14:03 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Make the debian job voting and start publishing debian images https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/767158 | 14:10 |
dtantsur | TheJulia: https://storyboard.openstack.org/#!/story/2008491 something we've discussed so many times | 14:12 |
ajya | viks____: not familiar with that use case. Is there smth set in `instance_info` of the node? | 14:15 |
arne_wiebalck | viks____: you set deploy_{kernel,ramdisk} when enrolling the node, image_source is set when the node is instantiated (and not set explicitly set by the operator/admin) | 14:15 |
arne_wiebalck | viks____: deploy* is the "deploy image" (i.e. with the IPA), "image_source" is the user image (i.e. passed via nova) | 14:18 |
dtantsur | iurygregory: do you plan on updating https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/769196/ ? we could request a release after that | 14:19 |
iurygregory | dtantsur, will do in a few minutes (finishing some status here) | 14:20 |
viks____ | arne_wiebalck: im stuck at changing node from `manageable` state to `available` i.e. as per https://docs.openstack.org/ironic/train/install/enrollment.html#making-node-available-for-deployment | 14:24 |
viks____ | and when doing so, it's going to `clean wait` state. So when i check via kvm console, it some how boots up in to ubuntu image. I think i'm missing something here | 14:24 |
arne_wiebalck | viks____: right, the question is if that image is the deploy image which has an IPA inside which will call back to the conductor to get cleaning instructions | 14:25 |
arne_wiebalck | viks____: if there is no IPA, Ironic will wait (clean_wait) until it times out | 14:25 |
arne_wiebalck | viks____: all of this is Ironic internal (plus Glance), meaning Nova is not involved in what you are trying to achieve atm | 14:26 |
viks____ | arne_wiebalck: ok.. i downloaded the prebuilt images with IPA from here: https://tarballs.opendev.org/openstack/ironic-python-agent/dib/files/ | 14:27 |
arne_wiebalck | viks____: ok (we build our own, so I have no experience with the pre-uilt ones, but they should work I would think) | 14:28 |
arne_wiebalck | viks____: you uploaded these to Glance? | 14:28 |
viks____ | yes.. right | 14:28 |
arne_wiebalck | viks____: with deploy_kernel, deploy_ramdisk set in the driver_info of the node? | 14:29 |
viks____ | also i have set: | 14:29 |
viks____ | https://www.irccloud.com/pastebin/Td4gpLrJ/ | 14:29 |
arne_wiebalck | viks____: looks good, assuming these are the correct UUIDs | 14:30 |
viks____ | yes.. they are correct.. | 14:31 |
arne_wiebalck | viks____: now, when you provide the node and ironic switches the node's state to clean_wait, the node starts booting shortly after? | 14:32 |
viks____ | yes.. right | 14:32 |
arne_wiebalck | viks____: ok | 14:32 |
arne_wiebalck | viks____: next thing to verify is that the node boots into the deploy image you set up | 14:33 |
arne_wiebalck | viks____: you said it is booting Ubuntu, no? | 14:33 |
viks____ | arne_wiebalck: nope.. it boots in to some ubuntu image... i used centos 7 image... | 14:34 |
viks____ | sorry.. yes.. it is not bootng in to centos .. | 14:34 |
arne_wiebalck | viks____: the deploy image is centos, but the node boots ubuntu? | 14:34 |
arne_wiebalck | viks____: :) | 14:34 |
viks____ | right.. thats where i'm confused.. from where it gets the ubuntu image from? | 14:35 |
arne_wiebalck | viks____: ok, so the node does not boot into the deploy image, so there is no IPA calling back into Ironic, and therefore the node stays in clean_wait: ironic never hears back from your node | 14:35 |
arne_wiebalck | viks____: the pxe config is wrong I would think: it either gives the wrong image (unlikely) or does not answer properly at all, and the node boots from the disk | 14:36 |
viks____ | arne_wiebalck: what could be the reason for this? where it could be picking the image from? is some pxe config giving problem | 14:36 |
arne_wiebalck | viks____: :) | 14:36 |
arne_wiebalck | viks____: when you follow the node booting, you can usually see if the node boots from the network or the disk | 14:38 |
arne_wiebalck | viks____: but, yeah, I would think the pxe config is the place to look next | 14:38 |
viks____ | arne_wiebalck: it should be booting from network right? i see some logs in kvm console that it downloads some images... | 14:42 |
arne_wiebalck | viks____: yes, it should pxe boot and get the centos deploy image | 14:43 |
arne_wiebalck | viks____: in our deployment we do not use the standard setup for this, i.e. we do not rely on Ironic/Neutron for this, so my experience is limited, but for sure you need to make sure the node gets the deploy image at this stage :) | 14:44 |
viks____ | arne_wiebalck: is something configured in bios for pxe? i do not know much how exactly it works | 14:44 |
viks____ | arne_wiebalck: ok | 14:44 |
arne_wiebalck | viks____: yes, the physical node needs to be configured explictly to boot from the network | 14:45 |
arne_wiebalck | viks____: in the BIOS | 14:46 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent-builder master: Add doc/requirements https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/769196 | 14:46 |
iurygregory | dtantsur, done ^ | 14:46 |
iurygregory | docs, pdf-docs, releasenotes all green locally with usedevelop = False | 14:46 |
dtantsur | thx | 14:47 |
arne_wiebalck | viks____: there is also legacy vs UEFI booting (in the BIOS), and a boot_mode property for each node in Ironic | 14:47 |
*** tzumainn has joined #openstack-ironic | 14:47 | |
viks____ | arne_wiebalck: ok.. it's legacy bios and i'm using appropriate option while setting up | 14:48 |
iurygregory | np, I will update the other patches after testing | 14:49 |
arne_wiebalck | viks____: the node will typically tell you during booting when/if it tries the network or the disk | 14:50 |
viks____ | arne_wiebalck: is there any chance that dhcp/tftp etc configured in BIOS already and it's picking up that image by any chance? | 14:50 |
arne_wiebalck | viks____: could be, like dhcp replies with the wrong tftpserver which has a default image | 14:54 |
viks____ | arne_wiebalck: ok.. i'll investigate in that direction further.. also one more question... when deploy image first gets installed, user image will run on top of them? hw exactly does it work? | 14:55 |
arne_wiebalck | viks____: the deploy image is not installed, it is a RAM disk | 14:56 |
arne_wiebalck | viks____: it is used to get the user image (which goes to the disk) | 14:57 |
viks____ | arne_wiebalck: ok.. it's kind of temporary mount ? | 14:57 |
arne_wiebalck | viks____: it is in RAM only: the node is booted into this RAM disk to execute commands from Ironic | 14:58 |
arne_wiebalck | viks____: atm, for you, it will clean the node | 14:58 |
viks____ | arne_wiebalck: ok... thnx | 14:58 |
arne_wiebalck | viks____: later on, it will deploy the user image | 14:58 |
viks____ | i'll investigate on BIOS config... | 14:59 |
viks____ | thanks to all of you for your help | 14:59 |
arne_wiebalck | viks____: in addition, I would watch what the node is doing | 15:00 |
arne_wiebalck | viks____: via the console, I mean | 15:00 |
arne_wiebalck | viks____: np, good luck :) | 15:00 |
viks____ | arne_wiebalck: ok.. | 15:00 |
arne_wiebalck | viks____: (and don't worry, it is not that easy to set up :-D) | 15:00 |
viks____ | :P | 15:01 |
*** MentalSiege has joined #openstack-ironic | 15:07 | |
*** MentalSiege has quit IRC | 15:14 | |
*** MentalSiege has joined #openstack-ironic | 15:15 | |
openstackgerrit | Merged openstack/python-ironicclient stable/victoria: Remove lower-constraints job https://review.opendev.org/c/openstack/python-ironicclient/+/767776 | 15:19 |
*** derekh has quit IRC | 15:27 | |
dtantsur | ajya, rpioso, is there any downside to resetting iDRAC on each enrollment (asking in the context of metal3)? | 15:29 |
*** abdysn has quit IRC | 15:33 | |
*** ociuhandu has quit IRC | 15:34 | |
*** mgoddard has quit IRC | 15:41 | |
*** mgoddard has joined #openstack-ironic | 15:41 | |
*** spotz has quit IRC | 15:44 | |
*** ociuhandu has joined #openstack-ironic | 15:46 | |
*** mgoddard has quit IRC | 15:47 | |
TheJulia | dtantsur: w/r/t story, lgtm. It is exactly what has been dsicussed like ten times | 15:52 |
TheJulia | discussed | 15:52 |
* TheJulia is heads down on writing policy testing | 15:52 | |
dtantsur | yeah, I think I'll finally sit down and do it | 15:53 |
TheJulia | sounds good to me | 15:53 |
TheJulia | we've got upwards of a couple thousand lines of test code yaml inbound in the next week, so woudl appreciate a review there. It shoudl actually be a relatively quick review | 15:53 |
TheJulia | on a plus side I'm like half through the API and it doesn't appear we have any access control issues with policy in code enforcement | 15:55 |
*** ociuhandu has quit IRC | 15:56 | |
* TheJulia raises an eyebrow finally | 16:00 | |
* TheJulia pokes the test runner | 16:04 | |
openstackgerrit | Derek Higgins proposed openstack/ironic master: Register all hardware_interfaces together https://review.opendev.org/c/openstack/ironic/+/764911 | 16:15 |
*** jamesden_ is now known as jamesdenton | 16:46 | |
*** gyee has joined #openstack-ironic | 16:50 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Add doc/requirements https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/769196 | 16:50 |
*** ociuhandu has joined #openstack-ironic | 16:51 | |
*** ociuhandu has quit IRC | 16:52 | |
*** ociuhandu has joined #openstack-ironic | 16:52 | |
*** ociuhandu has quit IRC | 16:52 | |
dtantsur | iurygregory: ready for releasing ^^^ | 16:53 |
iurygregory | \o/ | 16:54 |
*** lucasagomes has quit IRC | 16:55 | |
iurygregory | dtantsur, in sushy i just need to add -r {toxinidir}/requirements.txt to docs build right? | 17:01 |
dtantsur | yep | 17:01 |
iurygregory | and add the usedevelop = False hehe | 17:02 |
dtantsur | mm, no | 17:02 |
openstackgerrit | Merged openstack/ironic bugfix/15.2: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic/+/769503 | 17:02 |
dtantsur | it's either | 17:02 |
openstackgerrit | Merged openstack/ironic-inspector bugfix/10.5: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-inspector/+/769505 | 17:02 |
iurygregory | ops D: | 17:02 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/sushy master: Add doc/requirements https://review.opendev.org/c/openstack/sushy/+/769174 | 17:02 |
dtantsur | if the docs build imports the actual modules (as the case with ironic or sushy), you need usedevelop=True and requirements.txt | 17:03 |
dtantsur | otherwise you need usedevelop=False and no requirements.txt | 17:03 |
* dtantsur hopes he makes sense | 17:03 | |
iurygregory | it kinda makes, but pdf-docs works fine without adding... | 17:03 |
iurygregory | this would be expected? | 17:03 |
dtantsur | maybe it doesn't use autodoc? no idea | 17:05 |
iurygregory | hummm | 17:06 |
iurygregory | *magic* | 17:06 |
iurygregory | oh pdf-docs will get the deps from docs | 17:07 |
iurygregory | I've changed to not relay on the docs deps (pdf-docs and releasenotes) | 17:09 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/sushy master: Add doc/requirements https://review.opendev.org/c/openstack/sushy/+/769174 | 17:09 |
iurygregory | let me know wdyt ^ | 17:09 |
*** mgoddard has joined #openstack-ironic | 17:10 | |
openstackgerrit | Merged openstack/bifrost bugfix/10.0: Remove lower-constraints job https://review.opendev.org/c/openstack/bifrost/+/769502 | 17:14 |
iurygregory | molteniron openstack-python3-ussuri-jobs XD | 17:21 |
dtantsur | wow | 17:22 |
dtantsur | TheJulia: do you know who is behind moltenironic nowadays? | 17:22 |
dtantsur | ehmm, molteniron | 17:22 |
iurygregory | I'm going to try to run pep8, py38 locally | 17:23 |
TheJulia | I'm suspecting nobody is using it anymore as it was just tooling for CI that was never intended to be released | 17:23 |
iurygregory | wish me luck | 17:23 |
TheJulia | so I wouldn't burn resources on molteniron | 17:23 |
iurygregory | so we need to talk with openstack-release to maybe "remove" ? | 17:24 |
TheJulia | it should have never been considered for release | 17:24 |
TheJulia | it is not a releasable thing | 17:24 |
* TheJulia wonders what changed | 17:25 | |
iurygregory | they have under independent | 17:25 |
iurygregory | https://opendev.org/openstack/releases/commit/d4986aa9ac1035f4b6c9bb4e33dfad2addd46764 | 17:26 |
*** ociuhandu has joined #openstack-ironic | 17:27 | |
*** dougsz has quit IRC | 17:31 | |
*** ociuhandu has quit IRC | 17:33 | |
TheJulia | so why are they raising an issue? | 17:34 |
dtantsur | can we just kick it out of ironic if nobody is using it? or is it used by 3rd party CI? | 17:35 |
iurygregory | we need to fix the doc issue that they raised on monday | 17:36 |
dtantsur | fwiw if we keep it under our umbrella, have the CI working is a reasonable expectation | 17:36 |
iurygregory | but things are a bit broken hehe | 17:36 |
iurygregory | and since we won't do relases maybe it would be ok to just remove from the releases repo? | 17:37 |
dtantsur | okay from my standpoint, but I'm not necessary aware of all regulations around official projects | 17:37 |
dtantsur | I can easily imagine they want a deliverable per repository unless it's something like specs | 17:37 |
TheJulia | dtantsur: 3rd party ci | 17:38 |
TheJulia | They initially did, but then realized things like small utilities in git made sense even if nobody else would use it | 17:39 |
iurygregory | I was talking with Herve about it | 17:41 |
iurygregory | <hberaud> If you think that this project isn't maintained then I would suggest to you to simply abandon it officially, then it will be removed from the governance and the releases will drop it too | 17:41 |
rpittau | bye everyone! o/ | 17:43 |
iurygregory | https://releases.openstack.org/reference/release_models.html#abandoned | 17:43 |
*** rpittau is now known as rpittau|afk | 17:43 | |
TheJulia | There is a fundamentally broken assumption occurring that repo = project = deliverable | 17:45 |
iurygregory | I will push a patch to change molteniron to it (we can still keep the project around) | 17:45 |
TheJulia | wfm | 17:45 |
iurygregory | cool | 17:45 |
* TheJulia goes back to writing tests | 17:45 | |
arne_wiebalck | bye everyone o/ | 17:46 |
iurygregory | bye arne_wiebalck rpittau|afk | 17:46 |
*** dking has joined #openstack-ironic | 18:24 | |
*** dtantsur is now known as dtantsur|afk | 18:27 | |
dtantsur|afk | o/ | 18:27 |
*** ociuhandu has joined #openstack-ironic | 19:13 | |
*** paras333 has joined #openstack-ironic | 19:14 | |
*** bnemec has quit IRC | 19:18 | |
*** bnemec has joined #openstack-ironic | 19:24 | |
*** ociuhandu has quit IRC | 19:26 | |
*** bnemec has quit IRC | 19:30 | |
*** bnemec has joined #openstack-ironic | 19:32 | |
*** bnemec has quit IRC | 19:37 | |
*** bnemec has joined #openstack-ironic | 19:39 | |
rpioso | dtantsur|afk: As far as I am aware, there is no harm. I confirmed that with a few members of our immediate team. I have not reached out to the firmware team. Our JetPack tooling resets the iDRAC when deploying systems in an OpenStack cluster. When upgrading firmware, we recommend trying it in a limited manner before rolling it out extensively, which seems like a generally best practice :-) | 19:54 |
*** tosin has joined #openstack-ironic | 20:05 | |
*** ociuhandu has joined #openstack-ironic | 20:17 | |
*** ociuhandu has quit IRC | 20:18 | |
*** ociuhandu has joined #openstack-ironic | 20:19 | |
*** ociuhandu has quit IRC | 20:19 | |
*** ociuhandu has joined #openstack-ironic | 20:20 | |
*** ociuhandu has quit IRC | 20:29 | |
*** ociuhandu has joined #openstack-ironic | 20:34 | |
*** ociuhandu has quit IRC | 20:43 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Add separate policies for updating node properties related to boot-from-volume https://review.opendev.org/c/openstack/ironic/+/769619 | 20:55 |
*** ociuhandu has joined #openstack-ironic | 20:56 | |
*** ociuhandu has quit IRC | 21:03 | |
*** ociuhandu has joined #openstack-ironic | 21:36 | |
*** ociuhandu has quit IRC | 21:40 | |
*** ociuhandu has joined #openstack-ironic | 21:44 | |
*** ociuhandu has quit IRC | 21:51 | |
*** ociuhandu has joined #openstack-ironic | 21:54 | |
*** ociuhandu has quit IRC | 21:58 | |
*** ociuhandu has joined #openstack-ironic | 22:10 | |
TheJulia | tzumainn: Please tell me you'll help with some of the testing matrix of that :( | 22:12 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP Start populating existing policy tests https://review.opendev.org/c/openstack/ironic/+/768136 | 22:14 |
TheJulia | brraaaainns | 22:14 |
*** ociuhandu has quit IRC | 22:19 | |
*** rcernin has joined #openstack-ironic | 22:27 | |
*** tosin has quit IRC | 22:30 | |
tzumainn | TheJulia, sure! er, I may not fully understand what testing matrix you mean :) | 22:32 |
TheJulia | positive/negative testing of policy enforcement | 22:32 |
tzumainn | TheJulia, there should be unit tests for that in the patch! | 22:33 |
TheJulia | So take a look at the patch I just updated | 22:33 |
TheJulia | We're working on adding it. It is literally what I've spent all day working on | 22:33 |
*** lbotell has joined #openstack-ironic | 22:34 | |
tzumainn | ah, I see! do the tests check which policy rules are being checked, or is that something to be added? | 22:35 |
tzumainn | (it's the latter, right?) | 22:35 |
TheJulia | It is an effort to have positive/negative testing of all policies before we start reworking policies for system scope support | 22:37 |
tzumainn | TheJulia, oh, I mean - the WIP patch doesn't seem to check that policies are checked, right? or did I miss that part of the patch? | 22:38 |
TheJulia | tzumainn: it is runing based upon response code | 22:38 |
tzumainn | TheJulia, would the desire be to potentially extend it to mock policy.authorize, and verify that certain policies are checked? or would that fall out of the scope of these tests? | 22:42 |
TheJulia | no, not try and mock the policy code at all | 22:43 |
TheJulia | but to actually exercise it as far down as possible | 22:43 |
TheJulia | it is not about checking the policy, but it is about ensuring the policy is effective/doing what is intended | 22:43 |
tzumainn | got it | 22:43 |
*** bnemec has quit IRC | 22:45 | |
TheJulia | the driver is because the policy/project scoping changes will be or'ed | 22:45 |
*** zzzeek has quit IRC | 22:53 | |
*** zzzeek has joined #openstack-ironic | 22:55 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP Start populating existing policy tests https://review.opendev.org/c/openstack/ironic/+/768136 | 22:59 |
*** bnemec has joined #openstack-ironic | 23:01 | |
*** zzzeek has quit IRC | 23:08 | |
*** zzzeek has joined #openstack-ironic | 23:09 | |
*** rh-jelabarre has quit IRC | 23:21 | |
*** ociuhandu has joined #openstack-ironic | 23:25 | |
*** ociuhandu has quit IRC | 23:30 | |
*** eagereagle1 has joined #openstack-ironic | 23:43 | |
eagereagle1 | hi guys! I had a question about updating IPMI during an ironic clean step...and how we would update the driver_info on the node when we do this? | 23:43 |
*** tosky has quit IRC | 23:51 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!