| *** zzzeek has quit IRC | 00:04 | |
| openstackgerrit | Bill Dodd proposed openstack/sushy master: Fix TaskMonitor constructor calls in volume.py https://review.opendev.org/c/openstack/sushy/+/773273 | 00:05 |
|---|---|---|
| *** zzzeek has joined #openstack-ironic | 00:05 | |
| *** ociuhandu has joined #openstack-ironic | 00:42 | |
| *** ociuhandu has quit IRC | 00:46 | |
| *** uzumaki has quit IRC | 00:58 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-lib master: Add initial privsep structure https://review.opendev.org/c/openstack/ironic-lib/+/745536 | 01:04 |
| openstackgerrit | Vanou Ishii proposed openstack/ironic master: Fix Mis-Ordering of Bash Variable Definition in DevStack https://review.opendev.org/c/openstack/ironic/+/773272 | 01:22 |
| *** uzumaki has joined #openstack-ironic | 01:43 | |
| *** rcernin has quit IRC | 02:21 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 02:28 |
| *** ociuhandu has joined #openstack-ironic | 02:30 | |
| *** ociuhandu has quit IRC | 02:35 | |
| *** paras333_ has joined #openstack-ironic | 02:36 | |
| *** rcernin has joined #openstack-ironic | 02:36 | |
| *** paras333_ has quit IRC | 02:38 | |
| *** rcernin has quit IRC | 02:45 | |
| *** rcernin has joined #openstack-ironic | 02:45 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-lib master: Add initial privsep structure https://review.opendev.org/c/openstack/ironic-lib/+/745536 | 03:21 |
| *** ociuhandu has joined #openstack-ironic | 03:22 | |
| *** ociuhandu has quit IRC | 03:26 | |
| *** mkrai has joined #openstack-ironic | 03:27 | |
| *** Qianbiao has joined #openstack-ironic | 03:56 | |
| *** rcernin has quit IRC | 04:00 | |
| *** rcernin has joined #openstack-ironic | 04:02 | |
| *** ociuhandu has joined #openstack-ironic | 04:23 | |
| *** rcernin has quit IRC | 04:27 | |
| *** ociuhandu has quit IRC | 04:29 | |
| *** rcernin has joined #openstack-ironic | 04:35 | |
| *** uzumaki has quit IRC | 04:43 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: [DNM] fix test_write_partition_image unit test https://review.opendev.org/c/openstack/ironic-python-agent/+/772989 | 04:43 |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 05:12 |
| *** Qianbiao has quit IRC | 05:13 | |
| *** Qianbiao has joined #openstack-ironic | 05:13 | |
| *** ociuhandu has joined #openstack-ironic | 05:30 | |
| *** ociuhandu has quit IRC | 05:34 | |
| *** Qianbiao has quit IRC | 05:56 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 06:02 |
| *** paras333_ has joined #openstack-ironic | 06:02 | |
| *** paras333_ has quit IRC | 06:06 | |
| *** Qianbiao has joined #openstack-ironic | 06:34 | |
| *** uzumaki has joined #openstack-ironic | 06:36 | |
| *** anuradha1904 has joined #openstack-ironic | 06:39 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 06:41 |
| *** mkrai has quit IRC | 06:43 | |
| *** TheJulia has quit IRC | 06:44 | |
| *** TheJulia has joined #openstack-ironic | 06:45 | |
| *** gmann has quit IRC | 06:45 | |
| *** jungleboyj has quit IRC | 06:45 | |
| *** jungleboyj has joined #openstack-ironic | 06:46 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 06:46 |
| *** gmann has joined #openstack-ironic | 06:46 | |
| *** ociuhandu has joined #openstack-ironic | 07:02 | |
| *** ociuhandu has quit IRC | 07:06 | |
| *** moshiur has joined #openstack-ironic | 07:22 | |
| *** rcernin has quit IRC | 07:25 | |
| arne_wiebalck | Good morning, ironic! | 07:26 |
| *** rcernin has joined #openstack-ironic | 08:07 | |
| erbarr | good morning arne_wiebalck | 08:08 |
| arne_wiebalck | Hey erbarr o/ | 08:08 |
| *** rpittau|afk is now known as rpittau | 08:11 | |
| rpittau | good morning ironic! o/ | 08:11 |
| *** rcernin has quit IRC | 08:24 | |
| *** rcernin has joined #openstack-ironic | 08:26 | |
| *** uzumaki has quit IRC | 08:31 | |
| *** rcernin has quit IRC | 08:31 | |
| *** mkrai has joined #openstack-ironic | 08:35 | |
| *** rcernin has joined #openstack-ironic | 08:37 | |
| *** tosky has joined #openstack-ironic | 08:43 | |
| *** ociuhandu has joined #openstack-ironic | 08:45 | |
| *** ociuhandu has quit IRC | 08:45 | |
| *** ociuhandu has joined #openstack-ironic | 08:45 | |
| *** ociuhandu has quit IRC | 08:49 | |
| *** lucasagomes has joined #openstack-ironic | 08:50 | |
| *** ociuhandu has joined #openstack-ironic | 09:00 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Add release version to release notes https://review.opendev.org/c/openstack/ironic/+/773314 | 09:08 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Add release version to release notes https://review.opendev.org/c/openstack/ironic-inspector/+/773316 | 09:10 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Add release version to release notes https://review.opendev.org/c/openstack/ironic-python-agent/+/773317 | 09:12 |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Add release version to release notes https://review.opendev.org/c/openstack/bifrost/+/773318 | 09:14 |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost stable/ussuri: [DNM] test ci ussuri https://review.opendev.org/c/openstack/bifrost/+/773319 | 09:21 |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost stable/victoria: [DNM] test CI victoria https://review.opendev.org/c/openstack/bifrost/+/772926 | 09:22 |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost stable/ussuri: [DNM] test CI ussuri https://review.opendev.org/c/openstack/bifrost/+/773319 | 09:22 |
| *** rcernin has quit IRC | 09:27 | |
| *** benj_ has quit IRC | 09:28 | |
| *** benj_ has joined #openstack-ironic | 09:30 | |
| *** derekh has joined #openstack-ironic | 09:30 | |
| *** ociuhandu has quit IRC | 09:36 | |
| openstackgerrit | Riccardo Pittau proposed openstack/networking-generic-switch stable/stein: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/networking-generic-switch/+/773324 | 09:37 |
| *** rcernin has joined #openstack-ironic | 09:39 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/train: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/ironic/+/773330 | 09:43 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector stable/train: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/ironic-inspector/+/773332 | 09:44 |
| openstackgerrit | Riccardo Pittau proposed openstack/networking-generic-switch stable/train: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/networking-generic-switch/+/773335 | 09:46 |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 09:50 |
| *** k_mouza has joined #openstack-ironic | 09:53 | |
| *** ociuhandu has joined #openstack-ironic | 10:01 | |
| *** rcernin has quit IRC | 10:06 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 10:06 |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: [DNM] fix test_write_partition_image unit test https://review.opendev.org/c/openstack/ironic-python-agent/+/772989 | 10:33 |
| *** sshnaidm|off is now known as sshnaidm|ruck | 10:35 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 10:39 |
| iurygregory | good morning Ironic! | 10:45 |
| rpittau | hey iurygregory :) | 10:48 |
| *** dtantsur|afk is now known as dtantsur | 10:54 | |
| * dtantsur yawns | 10:54 | |
| dtantsur | happy Monday, ironic. what's up? | 10:54 |
| rpittau | hey dtantsur :) | 10:55 |
| openstackgerrit | Mudit proposed openstack/ironic master: More GPU support in idrac-wsman inspect interface https://review.opendev.org/c/openstack/ironic/+/773359 | 11:02 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector stable/train: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/ironic-inspector/+/773332 | 11:14 |
| iurygregory | rpittau, o/ | 11:15 |
| *** rcernin has joined #openstack-ironic | 11:16 | |
| *** ociuhandu has quit IRC | 11:30 | |
| *** ociuhandu has joined #openstack-ironic | 11:30 | |
| *** ociuhandu has quit IRC | 11:36 | |
| openstackgerrit | Riccardo Pittau proposed openstack/networking-generic-switch stable/train: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/networking-generic-switch/+/773335 | 11:43 |
| *** mkrai has quit IRC | 11:48 | |
| viks____ | hi, i created user image as per https://docs.openstack.org/ironic/train/install/creating-images.html. i.e. whole disk image.. i'm able to ping the instance but not able to ssh.. i have added the key during server creation.. but i get `Permission denied (publickey).`. Do i need to enable any special setting in case of baremetal? | 11:57 |
| rpittau | viks____: are you using train version? | 12:00 |
| viks____ | yes | 12:01 |
| viks____ | rpittau: do we have some issue related to ssh in train? | 12:02 |
| rpittau | not that I'm aware of, it's just a bit old at this point | 12:03 |
| *** uzumaki has joined #openstack-ironic | 12:07 | |
| viks____ | rpittau: also i'm not sure if cloud-init is running in the user image created like above.. i'm not able to set root password as well... so i can't log in via kvm console also.. so i'm stuck.. is there any way i can findout? | 12:07 |
| janders | good morning/afternoon arne_wiebalck rpittau iurygregory dtantsur and Ironic o/ | 12:23 |
| iurygregory | hey janders o/ | 12:23 |
| openstackgerrit | Mudit proposed openstack/ironic master: More GPU support in idrac-wsman inspect interface https://review.opendev.org/c/openstack/ironic/+/773359 | 12:25 |
| arne_wiebalck | hey janders o/ | 12:28 |
| janders | rpittau I tracked down the IPA standby unit test problem a bit further... it seems it only passes in CI with boot_mode='bios' and disk_label='msdos' while on my Fedora it needs uefi/gpt... I wonder if it's inherited from the host | 12:28 |
| janders | https://zuul.opendev.org/t/openstack/build/2f30fa4517dd4678a46f8c9f0c850a70 | 12:28 |
| janders | it will be interesting if the NVMe patch passes CI now, as I tweaked it to match | 12:29 |
| arne_wiebalck | viks____: do you have access to the console of the node? if so, you could follow it to see if cloud-init runs (or boot the node into single user mode to set a password) | 12:30 |
| viks____ | i do not see any cloud-init in the logs in kvm console.. also i do not knw how to boot the node into single user mode to set a password.. | 12:32 |
| *** rcernin has quit IRC | 12:33 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Apply force_persistent_boot_device to all boot interfaces https://review.opendev.org/c/openstack/ironic/+/773382 | 12:38 |
| dtantsur | viks____: not sure if it helps, but I had troubles making an ubuntu image work. debian and centos worked for me though. | 12:42 |
| arne_wiebalck | viks____: there are recipes for this on the web, look for 'reset root password single user mode ubuntu' or similar | 12:44 |
| *** ociuhandu has joined #openstack-ironic | 12:45 | |
| *** ociuhandu has quit IRC | 12:46 | |
| *** ociuhandu has joined #openstack-ironic | 12:47 | |
| *** rcernin has joined #openstack-ironic | 12:47 | |
| bfournie | dtantsur: good morning, ok if I backport the vendor detect refactoring to Victoria? | 12:54 |
| dtantsur | bfournie: sure, go ahead | 12:54 |
| bfournie | dtantsur: thx | 12:54 |
| * dtantsur is in the middle of one of his regular attempts to understand secure boot | 12:55 | |
| bfournie | good luck | 12:56 |
| dtantsur | thanks.. | 12:56 |
| viks____ | dtantsur: ok.. will try out other images.. is there any difference b/n prebuilt cloud images (https://cloud-images.ubuntu.com/bionic/current/)and and the user image built as per https://docs.openstack.org/ironic/train/install/creating-images.html? | 12:57 |
| dtantsur | the content of images you build is somewhat more predictable | 12:57 |
| *** uzumaki has quit IRC | 12:58 | |
| *** dougsz has joined #openstack-ironic | 13:01 | |
| viks____ | dtantsur: ok.. so the prebuilt images in the above link can be used in ironic as well right? i mean i used them for non baremetal vm's which works fine for me for ssh as well as root password reset.. | 13:02 |
| *** paras333 has joined #openstack-ironic | 13:03 | |
| dtantsur | usually yes, although they don't normally have a root password set | 13:03 |
| dtantsur | you'll still need cloud-init (which is normally included) | 13:03 |
| *** paras333 has quit IRC | 13:07 | |
| viks____ | dtantsur: ok... yes they do not have root password set... but resetting works using `nova set-password`when `hw_qemu_guest_agent=yes` property is set on the image | 13:07 |
| dtantsur | "guest agent" is a keyword. I'm not sure how Nova does it, but it may or may not work with bare metal. | 13:08 |
| * dtantsur needs to read the UEFI specification, sorry | 13:08 | |
| *** uzumaki has joined #openstack-ironic | 13:09 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic stable/victoria: Refactor vendor detection and add Redfish implementation https://review.opendev.org/c/openstack/ironic/+/773388 | 13:10 |
| *** rh-jelabarre has joined #openstack-ironic | 13:24 | |
| *** rh-jelabarre has quit IRC | 13:25 | |
| *** rh-jelabarre has joined #openstack-ironic | 13:25 | |
| *** tosin has joined #openstack-ironic | 13:29 | |
| Qianbiao | viks____ you can try build an image with fixed password at first. | 13:34 |
| Qianbiao | and login to fix your issue. i guess cloud-init is not run correctly in your case. | 13:35 |
| *** rcernin has quit IRC | 13:38 | |
| viks____ | Qianbiao: i used the prebuilt images which has qemu-guest-agent in it, which works fine for me on nova comupute(non-baremetal)... but not sure why the same image doesnt work for baremetal.. can you let me know how to build an image with fixed password? what additional parmas do i need to pass to say `disk-image-create ubuntu vm dhcp-all-interfaces -o my-image` while building | 13:40 |
| Qianbiao | viks____ baremetal does not has an agent. | 13:45 |
| viks____ | Qianbiao: Oh! then how do i set/reset root password or enable ssh login? | 13:50 |
| *** rloo has joined #openstack-ironic | 13:54 | |
| *** ociuhandu has quit IRC | 13:55 | |
| *** ociuhandu has joined #openstack-ironic | 13:56 | |
| iurygregory | neutron is going crazy with the privsep changes... .-. | 14:03 |
| iurygregory | https://zuul.opendev.org/t/openstack/build/6f287ee742994721afbbf62e925282c9 | 14:03 |
| iurygregory | any ideas? .-. | 14:04 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy master: Initial support for secure boot databases https://review.opendev.org/c/openstack/sushy/+/773397 | 14:04 |
| Qianbiao | viks____ check https://docs.openstack.org/ironic/train/install/creating-images.html and find elements you required in DIB | 14:08 |
| Qianbiao | like ssh-server | 14:08 |
| Qianbiao | cloud-init | 14:08 |
| Qianbiao | i think you need them to ssh login to the server | 14:09 |
| *** lbragstad has joined #openstack-ironic | 14:09 | |
| *** uzumaki has quit IRC | 14:10 | |
| TheJulia | iurygregory: our neutron? | 14:10 |
| TheJulia | Also, good morning everyone | 14:10 |
| iurygregory | TheJulia, yeah ebtables v1.8.4 (nf_tables): table `broute' is incompatible, use 'nft' tool. | 14:11 |
| rpittau | iurygregory I don't think that's related to privsep change | 14:12 |
| iurygregory | looking at neutron ebtables is not in their privsep (not sure if would block us) O.o | 14:12 |
| iurygregory | rpittau, you saw this before? | 14:12 |
| rpittau | yeah | 14:12 |
| iurygregory | if you have some idea let me know o/ | 14:13 |
| dtantsur | morning TheJulia | 14:14 |
| dtantsur | iurygregory: I've seen this message on normal runs | 14:14 |
| iurygregory | O.o oh ok | 14:14 |
| rpittau | yes, me too, I think it's more a watning | 14:14 |
| rpittau | warning* | 14:14 |
| * iurygregory checking more logs | 14:15 | |
| *** lbragstad has quit IRC | 14:16 | |
| viks____ | Qianbiao: ok.. but my image has ssh running because it prompts for password and not letting me in eventhough i had passed my public key during instance creation | 14:18 |
| *** ociuhandu has quit IRC | 14:19 | |
| *** ociuhandu has joined #openstack-ironic | 14:19 | |
| *** lbragstad has joined #openstack-ironic | 14:22 | |
| *** ociuhandu has quit IRC | 14:29 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic stable/victoria: Refactor vendor detection and add Redfish implementation https://review.opendev.org/c/openstack/ironic/+/773388 | 14:29 |
| *** ociuhandu has joined #openstack-ironic | 14:30 | |
| *** ociuhandu has quit IRC | 14:35 | |
| *** ociuhandu has joined #openstack-ironic | 14:37 | |
| *** ociuhandu has quit IRC | 14:39 | |
| iurygregory | found the problem https://zuul.opendev.org/t/openstack/build/6f287ee742994721afbbf62e925282c9/log/controller/logs/ironic-bm-logs/node-0_console_2021-02-01-04:39:25_log.txt | 14:39 |
| *** ociuhandu has joined #openstack-ironic | 14:39 | |
| iurygregory | http://paste.openstack.org/show/802176/ | 14:40 |
| *** tzumainn has joined #openstack-ironic | 14:44 | |
| *** moshiur has quit IRC | 14:44 | |
| TheJulia | iurygregory: do you remember if the ironic-standalone-redfish executes jobs in parallel or in series. | 14:49 |
| *** uzumaki has joined #openstack-ironic | 14:49 | |
| TheJulia | iurygregory: wow :\ | 14:49 |
| iurygregory | TheJulia, we set to run with concurrency 2 | 14:50 |
| *** kaifeng has joined #openstack-ironic | 14:50 | |
| iurygregory | so should run in parallel if the machine can =) | 14:50 |
| dtantsur | the error looks.. funny | 14:52 |
| *** sdanni has joined #openstack-ironic | 14:54 | |
| *** lmcgann has joined #openstack-ironic | 14:54 | |
| *** mkrai has joined #openstack-ironic | 14:55 | |
| iurygregory | yup! | 14:56 |
| TheJulia | iurygregory: ahh, I bet that is why it likes failing, I bet we're running out of ram. | 14:57 |
| dtantsur | there goes our attempt to reduce the number of jobs :( | 14:58 |
| iurygregory | TheJulia, yeah we can just set tempest_concurrency for it to see | 14:58 |
| TheJulia | dtantsur: huh? | 14:59 |
| dtantsur | TheJulia: if we need to reduce concurrency, we need to increase the number of jobs again, otherwise we're going to hit even more timeouts | 15:00 |
| TheJulia | dtantsur: we can adjust some of the timeouts on jobs like the standalone jobs, besides they present the most value because they cover the most paths and interactions. Just the job failing an hour and fifteen minutes in with no apparent reason at the surface screams memory issues :\ | 15:01 |
| iurygregory | yup | 15:01 |
| TheJulia | It is all a dance | 15:01 |
| TheJulia | unfortunately | 15:01 |
| TheJulia | I'd rather kill the pxe_redfish_blah blah jobs before the standalone jobs | 15:01 |
| iurygregory | if we move the concurrency it should increase the job time (but we have jobs with 3hrs + etc) | 15:01 |
| TheJulia | Anyway! | 15:01 |
| TheJulia | It is time for our meeting | 15:01 |
| TheJulia | #startmeeting ironic | 15:01 |
| *** stendulker has joined #openstack-ironic | 15:01 | |
| openstack | Meeting started Mon Feb 1 15:01:58 2021 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| TheJulia | Good morning everyone! | 15:02 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:02 | |
| iurygregory | o/ | 15:02 |
| openstack | The meeting name has been set to 'ironic' | 15:02 |
| stendulker | o/ | 15:02 |
| bdodd | o/ | 15:02 |
| kaifeng | o/ | 15:02 |
| rpioso | o/ | 15:02 |
| rpittau | o/ | 15:02 |
| TheJulia | Our agenda can be found on the wiki, as always. | 15:02 |
| TheJulia | We should likely discuss what to do without a wiki one day... | 15:02 |
| TheJulia | Anyway | 15:02 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:02 |
| rloo | o/ | 15:02 |
| * TheJulia makes a quick last minute fix | 15:02 | |
| ajya | o/ | 15:02 |
| *** bburns has quit IRC | 15:03 | |
| TheJulia | Okay! | 15:03 |
| bfournie | o/ | 15:03 |
| TheJulia | #topic Announcements / Reminders | 15:03 |
| *** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)" | 15:03 | |
| erbarr | o/ | 15:03 |
| TheJulia | #info This week is R-10 for the Wallaby release. Again, R-6 is non-client library freeze, R-5 is client library and requirements freeze. | 15:03 |
| *** MentalSiege has joined #openstack-ironic | 15:03 | |
| TheJulia | #info Second sprint ends next week with hopefully a release. CI of course permitting :) | 15:04 |
| TheJulia | #info Review Jam poll link has been posted. More votes will be appreciated. If your interested and have not voted, please do so today. | 15:04 |
| TheJulia | #link https://doodle.com/poll/mdv6vpw6qdfzteg2 | 15:04 |
| TheJulia | Does anyone have anything else to announce or remind us of? | 15:04 |
| stendulker | An iLO CI Gate has been added to Sushy. Test ran successfully for https://review.opendev.org/c/openstack/sushy/+/773273 | 15:05 |
| dtantsur | nice! | 15:05 |
| TheJulia | Excellent! | 15:05 |
| iurygregory | would be good for the ironic-cores to review https://review.opendev.org/c/openstack/project-config/+/772427 =) | 15:05 |
| TheJulia | stendulker: That almost sounds worthy of #success | 15:05 |
| iurygregory | maybe I missed something to add the support for editHashtag =) | 15:05 |
| stendulker | Thank you :) | 15:05 |
| rpittau | iurygregory: that looks good, I'll add my +1 | 15:06 |
| TheJulia | iurygregory: please add that link to the list of priority reviews | 15:06 |
| iurygregory | TheJulia, ack o/ | 15:06 |
| TheJulia | Does anyone else have anything else to announce or remind us of this week? | 15:06 |
| TheJulia | I wanted to take a quick moment to thank everyone for midcycle participation last week! It was great to hear everyone's voices. :) | 15:07 |
| TheJulia | So last week didn't have any action items, so I believe we can proceed to subteam status reports if there are no objections. | 15:08 |
| TheJulia | #topic Review subteam status reports | 15:08 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:08 | |
| TheJulia | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:08 |
| TheJulia | Starting at line 260 | 15:09 |
| iurygregory | we had action items from the midcycle, does it counts? | 15:09 |
| TheJulia | Some of them are longer items, I guess we can circle back later on | 15:10 |
| TheJulia | We've got a lot on the agenda | 15:10 |
| iurygregory | ++ | 15:10 |
| TheJulia | bdodd: thanks for the update on redfish raid | 15:12 |
| bdodd | TheJulia Of course | 15:12 |
| TheJulia | iurygregory: still fighting with privsep I guess? :) | 15:12 |
| iurygregory | TheJulia, correct =) | 15:13 |
| iurygregory | will try to add notes after finding something about the error | 15:14 |
| TheJulia | Okay | 15:14 |
| TheJulia | I think that is pretty good, are we good to proceed | 15:14 |
| TheJulia | ? | 15:14 |
| dtantsur | ++ | 15:14 |
| rpittau | let's | 15:15 |
| TheJulia | #topic Deciding on priorities for the coming week | 15:15 |
| *** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:15 | |
| TheJulia | So I went through and checked patches to see if they had merged/been approved/etc | 15:15 |
| TheJulia | I have not added patches outside of the nvme erase code | 15:15 |
| TheJulia | I'll clean up the list of merged patches if others will post their links for the week | 15:16 |
| *** paras333 has joined #openstack-ironic | 15:18 | |
| TheJulia | Anything else to add? | 15:18 |
| *** MentalSiege has quit IRC | 15:19 | |
| TheJulia | I guess that looks good to me | 15:19 |
| rpittau | should be good | 15:20 |
| TheJulia | Onward then! | 15:20 |
| TheJulia | #topic Discussion | 15:20 |
| *** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:20 | |
| TheJulia | dtantsur: looks like this item is something you added? | 15:20 |
| TheJulia | Train branch status? | 15:21 |
| dtantsur | yes, thank you | 15:21 |
| dtantsur | It's a follow-up to the midcycle topic. I'm holding up the release proposed by the release team for ironic. | 15:21 |
| dtantsur | Since we have a few outstanding patches, and the gate does not seem to work. | 15:21 |
| dtantsur | So the pressing question is: should we fix it asap or should I give them go-ahead with the proposed release? | 15:21 |
| TheJulia | Do we have a list of issues or failed example jobs? | 15:21 |
| dtantsur | lemme see | 15:23 |
| TheJulia | It is a hard question to answer without being consciously aware the failures and what is outstanding | 15:23 |
| dtantsur | actually, the patch to remove grenade passed: https://review.opendev.org/c/openstack/ironic/+/773330/ | 15:23 |
| dtantsur | maybe it boils down to landing it | 15:23 |
| TheJulia | ship it! | 15:24 |
| * TheJulia adds +2 | 15:24 | |
| rpittau | yeah ironic in train doesn't look that bad | 15:24 |
| dtantsur | okay, we're fine that. let's make sure to recheck any outstanding patches when this one lands | 15:24 |
| TheJulia | k | 15:24 |
| TheJulia | ++ | 15:24 |
| * dtantsur is ready to move on | 15:24 | |
| *** MentalSiege has joined #openstack-ironic | 15:24 | |
| TheJulia | Excellent! | 15:24 |
| TheJulia | Onward to the SIG | 15:24 |
| TheJulia | #topic Baremetal SIG | 15:24 |
| *** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)" | 15:24 | |
| TheJulia | arne_wiebalck couldn't join us today, but he gave me a heads up for the SIG's next topic! | 15:25 |
| dtantsur | yep :) | 15:25 |
| TheJulia | Next week, dtantsur will be presenting deploy steps to the SIG! Hopefully we'll also see the previous recordings posted to youtube this week. | 15:25 |
| TheJulia | \o/ | 15:25 |
| TheJulia | Onward then! | 15:26 |
| TheJulia | #topic RFE Review | 15:26 |
| *** openstack changes topic to "RFE Review (Meeting topic: ironic)" | 15:26 | |
| TheJulia | dtantsur: looks like you've proposed these three, do you want to walk us through them? | 15:26 |
| dtantsur | yup | 15:26 |
| dtantsur | #link https://storyboard.openstack.org/#!/story/2008567 Expose boot mode and secure boot status in the API | 15:26 |
| dtantsur | This should be more or less self-explanatory | 15:27 |
| TheJulia | Reading it, it seems reasonable | 15:27 |
| dtantsur | The only non-trivial bits: I'm proposing changing to be asynchronous (so unlike boot device and like power states) | 15:27 |
| TheJulia | Seems like it might be a bit unreliable on ipmi based machines, at least for boot_mode. | 15:27 |
| TheJulia | (because ipmi bmc response is not super reliable) | 15:27 |
| dtantsur | Boot mode API is not implemented for ipmitool | 15:27 |
| TheJulia | \o/ | 15:28 |
| TheJulia | nevermind then! | 15:28 |
| dtantsur | :) | 15:28 |
| TheJulia | I think it is good, I would consider these purely administrative endpoints for humans, not programmatic interaction which might be useful say as a member | 15:28 |
| TheJulia | of the system that is | 15:28 |
| * TheJulia now sees everything in RBAC terms :( | 15:28 | |
| dtantsur | yep. and seeing these values in node listing may be really helpful for debugging | 15:28 |
| TheJulia | I have no objection to this change. | 15:29 |
| dtantsur | any other opinions? | 15:29 |
| rloo | the PUT API endpoints. | 15:29 |
| rloo | are they avail any time? | 15:29 |
| rloo | or only when the node is in certain states? | 15:29 |
| dtantsur | rloo: I'll check what we do for other actions, but likely in stable states only | 15:29 |
| Qianbiao | i think it's depended? | 15:30 |
| dtantsur | (and without a reservation, of course) | 15:30 |
| rloo | ok thx, as long as consistent and makes sense, i think i'm good with it :) | 15:30 |
| rloo | if there is a failure, will last_error be updated? | 15:30 |
| rloo | guessing yes, that's what you mean by 'the only way... will be to poll the node'. | 15:31 |
| dtantsur | yep | 15:31 |
| TheJulia | Seems reasonable and hopefully we'll have node history soon() | 15:31 |
| TheJulia | Onward to the next rfe since this one seems "approved" | 15:31 |
| dtantsur | #link https://storyboard.openstack.org/#!/story/2008366 BMC event framework for ironic (reduced scope) | 15:32 |
| dtantsur | This is a lighter version of the RFE I've already proposed once | 15:32 |
| dtantsur | MVP without proxying by ironic | 15:32 |
| TheJulia | dtantsur: my feeling is "yes" to both of the questions at thebottom of the rfe | 15:33 |
| TheJulia | Seems reasonable to build one piece at a time. | 15:33 |
| rloo | why just active nodes. | 15:35 |
| TheJulia | That is a good question, but I think it helps curtail the action of having to remove/cleanup on deploy | 15:35 |
| rloo | i was wondering whether that info would be useful for debugging, if clean/deploy failed :) | 15:36 |
| dtantsur | hmm | 15:36 |
| TheJulia | so lets do this.. I think dmitry is trying to keep it skimmed down, but there may be value in persistent | 15:36 |
| rloo | am ok with the rfe. could start with active. and just stick with active. | 15:36 |
| TheJulia | so I think we need a class of two | 15:36 |
| dtantsur | yeah, relaxing restrictions is easier than adding them | 15:37 |
| dtantsur | (I don't remember why I settled on active only, probably for an absolute MVP) | 15:37 |
| rloo | would just like it documented why 'only' active. if it is for now only. | 15:37 |
| TheJulia | I think active only is fine for first pass and then we add a field for persistent or something | 15:37 |
| dtantsur | I'm fine with dropping this requirement, one fewer check in the code | 15:37 |
| TheJulia | or we could just drop the constraint | 15:37 |
| * TheJulia is fine either way and has no hard opinion | 15:37 | |
| iurygregory | ++ for dropping =) | 15:38 |
| TheJulia | dtantsur: I guess update the rfe and your good to go | 15:39 |
| dtantsur | yup, will do after the meeting | 15:39 |
| rloo | so if i understand it. this provides an event framework. but at the end of this, one can't do much cuz the actual events won't be avail? | 15:39 |
| TheJulia | rloo: not the framework yet, more so the ability to change the knobs on the hardware | 15:39 |
| TheJulia | to eventually reach a framework | 15:39 |
| Qianbiao | dtantsur if node is none-active, maybe subscribtion will fails? | 15:39 |
| dtantsur | Qianbiao: subscriptions themselves are not related to our states | 15:40 |
| kaifeng | Does creating a subscription mean registering an event subscriber? | 15:40 |
| Qianbiao | like wrong credential | 15:40 |
| iurygregory | it will fail if the BMC has some problem =) | 15:40 |
| *** uzumaki has quit IRC | 15:40 | |
| iurygregory | or your request has some problem to create the subscription | 15:40 |
| dtantsur | kaifeng: yep. you're providing a URL that will be called by the BMC in case of a hardware event (overheating etc) | 15:40 |
| *** uzumaki has joined #openstack-ironic | 15:41 | |
| Qianbiao | like some admin tools of BMC :) | 15:41 |
| kaifeng | wow, I don't know BMC can do such a thing | 15:41 |
| stendulker | What kind of events are expected from BMC? | 15:41 |
| dtantsur | things like overheating | 15:42 |
| TheJulia | I feel like the disconnect is modeled around the redfish events functionality | 15:42 |
| TheJulia | where the bmc can broadcast $things | 15:42 |
| dtantsur | but yeah, I don't define the data model, merely exposing what can be done already | 15:42 |
| TheJulia | ++ | 15:42 |
| dtantsur | if your BMC can do it, we can implement that as well | 15:42 |
| stendulker | So will there be event catalog from Ironic and drivers needs to map it to their BMC and create a subscription? | 15:42 |
| TheJulia | there cannot be | 15:43 |
| TheJulia | so vendors have an open-ended capability to define these things, dmitry is just trying to propose the ability to add/remove subscriptions which is a defined feature of the standard | 15:43 |
| stendulker | The events supported by BMC could differ from vendor to vendor... | 15:43 |
| TheJulia | the bmc then enumerates through the endpoints and transmits data to whereever the subscription defines | 15:43 |
| dtantsur | from family to family and even from model to model | 15:43 |
| TheJulia | in this case, it is not transmitting to ironic from what I understand | 15:44 |
| dtantsur | same situation as with BIOS settings: I don't think we can provide a catalog | 15:44 |
| kaifeng | I wonder who consumes the driver-specific-filters? | 15:44 |
| stendulker | So 'filters' in the RFE is going to driver specific ? | 15:44 |
| dtantsur | kaifeng: the BMC at this point | 15:44 |
| dtantsur | yes | 15:44 |
| stendulker | Then do we need some kind listing of supported filters by a driver? Or just refer doc for that | 15:45 |
| kaifeng | okay, if i get this rignt, the rfe is to implement a passthrough api to set up some event callback to the bmc | 15:45 |
| dtantsur | kaifeng: correct | 15:45 |
| kaifeng | sounds useful, so this is only applicable for the redfish right, do we have consistent support from the standard? | 15:46 |
| dtantsur | I haven't checked the real-life status, I must admit | 15:47 |
| TheJulia | I checked like a year ago and I think at least two vendors supported it | 15:47 |
| TheJulia | I think a third, you could see it was stubbed into place, but maybe not usable | 15:48 |
| TheJulia | I didn't dig much further | 15:48 |
| TheJulia | Anyway, I don't see any objections, just scoping questions | 15:49 |
| TheJulia | dtantsur: maybe a slight bit more clarity in the rfe and I suspect you'll be good | 15:49 |
| dtantsur | okay, I'll try to type some words and ping you later | 15:49 |
| Qianbiao | TheJulia ibmc supports subscribtion | 15:50 |
| TheJulia | Qianbiao: good to know, thanks | 15:50 |
| TheJulia | Anything else? | 15:50 |
| TheJulia | dtantsur: ^^ | 15:50 |
| dtantsur | I have another one if we have time | 15:50 |
| TheJulia | I think we do | 15:50 |
| dtantsur | #link https://storyboard.openstack.org/#!/story/2008491 An ability to run manual cleaning without booting a ramdisk | 15:50 |
| dtantsur | This has been discussed many times as a nice-to-have thing | 15:51 |
| dtantsur | This is a concrete proposal | 15:51 |
| TheJulia | ++ | 15:51 |
| TheJulia | we've discussed this enough times I feel like this is good | 15:51 |
| Qianbiao | this is a great feature | 15:51 |
| kaifeng | i tried some raid work last year and it appears that idrac actually doesn't need in-band at all | 15:52 |
| Qianbiao | yes kaifeng | 15:52 |
| Qianbiao | ibmc is the same. | 15:52 |
| kaifeng | for pure oob it would be a good enhancement | 15:52 |
| TheJulia | the major complaint in the past has been reset of the bmc where no other commands are permitted | 15:53 |
| rpioso | We would very much appreciate this change. | 15:53 |
| TheJulia | #chair dtantsur | 15:53 |
| openstack | Current chairs: TheJulia dtantsur | 15:53 |
| TheJulia | I need to step away | 15:53 |
| dtantsur | k | 15:54 |
| dtantsur | I think we've agreed with the direction, what about the specific implementation proposal? | 15:55 |
| kaifeng | Qianbiao: good to know | 15:55 |
| * dtantsur usually assumes that crickets agree with him | 15:56 | |
| rpioso | For us, it is much more than a nice to have feature. The iDRAC can be placed in a a state in which it cannot be removed unless this feature is added. | 15:56 |
| rpioso | If memory serves, it is called recovery mode. | 15:57 |
| dtantsur | yup | 15:58 |
| TheJulia | okay, back | 15:58 |
| openstackgerrit | Merged openstack/sushy master: Fix TaskMonitor constructor calls in volume.py https://review.opendev.org/c/openstack/sushy/+/773273 | 15:58 |
| TheJulia | Sounds good, time to move on? | 15:58 |
| TheJulia | #topic Open Discussion | 15:59 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:59 | |
| TheJulia | Any open items? | 15:59 |
| TheJulia | s/open/other? | 15:59 |
| * TheJulia hears crickets and thinks everyone has gone off to get more coffee | 16:00 | |
| TheJulia | Well everyone! Thanks! | 16:01 |
| TheJulia | #endmeeting | 16:01 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list." | 16:01 | |
| openstack | Meeting ended Mon Feb 1 16:01:22 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-02-01-15.01.html | 16:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-02-01-15.01.txt | 16:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-02-01-15.01.log.html | 16:01 |
| Qianbiao | THanks TheJulia dtantsur | 16:01 |
| Qianbiao | perfect timeing, 12:00 | 16:01 |
| iurygregory | fyi I've talked with oslo folks about the Backport Candidate label, "IMO yes it's a bit useful. It's not the solution to all the problem but it help to vizualize and don't forget things" "However I'm not convinced that +1 & +2 is really useful on that I think that only +1 could be enough" | 16:01 |
| *** Qianbiao has quit IRC | 16:02 | |
| *** Qianbiao has joined #openstack-ironic | 16:02 | |
| TheJulia | +1 could be enough for the flag? | 16:03 |
| TheJulia | Most likely | 16:03 |
| iurygregory | yeah | 16:03 |
| *** Qianbiao has quit IRC | 16:03 | |
| iurygregory | I'm looking at some gerrit clients to see if I can have something that would detect the label and would add the hashtag ironic-backport | 16:04 |
| iurygregory | so we can have a search with "hashtag:ironic-backport" to track things | 16:04 |
| iurygregory | not sure if we can have a bot that would do this for us... | 16:04 |
| kaifeng | btw what does hashtag used for? something like topic? | 16:04 |
| iurygregory | can be used to filter your search, and you can have multiple | 16:05 |
| iurygregory | topic you can only have 1 | 16:05 |
| * iurygregory brb lunch time | 16:06 | |
| kaifeng | iurygregory thanks! | 16:06 |
| * TheJulia finally has breakfast | 16:07 | |
| *** stendulker has quit IRC | 16:12 | |
| *** mkrai has quit IRC | 16:15 | |
| dtantsur | TheJulia: updated https://storyboard.openstack.org/#!/story/2008366 | 16:21 |
| *** alexmcleod has joined #openstack-ironic | 16:22 | |
| TheJulia | dtantsur: thanks | 16:22 |
| dtantsur | an argument for only allowing subscriptions for ACTIVE nodes could be multi-tenancy | 16:26 |
| dtantsur | to make sure people don't start receiving events for nodes that are no longer leased to them | 16:27 |
| TheJulia | Feasible | 16:27 |
| TheJulia | as long as we unwind any | 16:27 |
| TheJulia | then again, lessee admin is not getting a ton of adminy rights | 16:27 |
| TheJulia | owners on the other hand... | 16:27 |
| dtantsur | yeah.. | 16:27 |
| TheJulia | 'My hardware!" | 16:28 |
| dtantsur | maybe we need a subscription owner eventually (similarly to allocations) | 16:28 |
| TheJulia | Maybe, I'm kind of avoiding allocation owners for now and just keeping it system scoped until I can wrap my head around it | 16:28 |
| TheJulia | which is likely a talk to you point | 16:28 |
| dtantsur | allocations are very much designed to be used by lessees | 16:29 |
| dtantsur | (and non-admin owners) | 16:29 |
| TheJulia | err, then we need to add another field then | 16:29 |
| dtantsur | why? | 16:29 |
| TheJulia | we will need to somehow determine how to associate what records should be visible to them | 16:30 |
| TheJulia | and if they are reating it, we'll need to store it | 16:30 |
| dtantsur | project_id == allocation.owner | 16:30 |
| TheJulia | unless we just store the lessee project id into the owner field | 16:30 |
| dtantsur | exactly | 16:30 |
| dtantsur | owner is the allocation owner, not the node owner | 16:30 |
| TheJulia | okay, regardless, I'm going and making a note int he spec | 16:30 |
| dtantsur | (there is no leasing of allocations, you create one yourself) | 16:31 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/726483 <-- wants to merge and help protect from OOMs | 16:32 |
| TheJulia | comment added to the spec so we can sort that out on allocations | 16:34 |
| TheJulia | Thanks dtantsur | 16:34 |
| dtantsur | yw | 16:35 |
| dtantsur | bdodd: I assume we need a new release because of https://review.opendev.org/c/openstack/sushy/+/773273 ? | 16:36 |
| *** ociuhandu_ has joined #openstack-ironic | 16:36 | |
| bdodd | dtantsur Yes, indeed | 16:36 |
| TheJulia | dtantsur: I believe so yes | 16:36 |
| dtantsur | I can request it unless somebody wants to beat me to it | 16:37 |
| TheJulia | no reason to juggle things but I am hoping to review/approve redfish raid this week | 16:37 |
| *** ociuhandu has quit IRC | 16:39 | |
| bdodd | If someone can point me to some info on how to make the request, I can do it. (Would be good to know how to do it.) | 16:40 |
| *** ociuhandu_ has quit IRC | 16:40 | |
| TheJulia | ohhh | 16:40 |
| TheJulia | yes!!! | 16:40 |
| openstackgerrit | Bob Fournier proposed openstack/ironic stable/victoria: Refactor vendor detection and add Redfish implementation https://review.opendev.org/c/openstack/ironic/+/773388 | 16:41 |
| *** bburns has joined #openstack-ironic | 16:41 | |
| TheJulia | bdodd: so the tl;dr is you edit https://opendev.org/openstack/releases/src/branch/master/deliverables/wallaby/sushy.yaml | 16:41 |
| * iurygregory back | 16:41 | |
| TheJulia | you add the git commit hash with a version, submit the change | 16:41 |
| TheJulia | it requires one of the current or past ptls or designated person to approve it, and magic happens once the release team approves it | 16:42 |
| dtantsur | mm, no need to edit manually | 16:42 |
| dtantsur | or figure out hashes | 16:42 |
| dtantsur | bdodd: https://docs.openstack.org/ironic/latest/contributor/releasing.html#how-to-propose-a-release | 16:43 |
| *** sdanni has quit IRC | 16:43 | |
| dtantsur | and don't worry, one of us will be pinged to review your request anyway | 16:43 |
| bdodd | dtantsur Thanks! I'll read and follow those steps. | 16:44 |
| iurygregory | bdodd, https://docs.openstack.org/ironic/latest/contributor/releasing.html#how-to-propose-a-release | 16:45 |
| iurygregory | oh just saw dtantsur added the link =) | 16:45 |
| bdodd | iurygregory Got it! | 16:45 |
| iurygregory | let us know if you face any problems =) | 16:46 |
| bdodd | Will do | 16:46 |
| *** ociuhandu has joined #openstack-ironic | 16:46 | |
| JayF | o/ haven't seen you around bdodd nice to meet you and thanks for helping | 16:49 |
| bdodd | JayF \o Thanks. Nice to me you. | 16:50 |
| *** ociuhandu has quit IRC | 16:51 | |
| openstackgerrit | Merged openstack/ironic stable/train: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/ironic/+/773330 | 16:53 |
| *** dtantsur is now known as dtantsur|afk | 17:00 | |
| dtantsur|afk | o/ | 17:00 |
| *** lucasagomes has quit IRC | 17:02 | |
| iurygregory | the acl for Backport Candidate label should be for Registered Users or only cores? | 17:03 |
| iurygregory | if we want to only use +1/-1 | 17:03 |
| rpittau | probably cores ? | 17:07 |
| iurygregory | I think it makes sense to be only for cores in case we don't want another label +2/-2 | 17:11 |
| arne_wiebalck | JayF: was it you who was re-deploying nodes while keeping the data disks? | 17:11 |
| JayF | We have that capability in our downstream Anaconda driver. I do not anticipate it being implemented for the upstream version, at least initially, but zer0c00l would know better. | 17:12 |
| arne_wiebalck | JayF: We have this in our ks files as, but not in Ironic, but since more and more nodes are moving to Ironic, the question came up. | 17:13 |
| JayF | Yeah; you should be able to do it direct via KS with our upstream version, that's right | 17:14 |
| arne_wiebalck | JayF: I was wondering if it would make sense to set root device hints right after successful deployment. | 17:14 |
| arne_wiebalck | JayF: automatically | 17:15 |
| JayF | root device hints are not something that the KS driver will respect or understand, and that's a driver specific ironic setting, right? | 17:15 |
| arne_wiebalck | JayF: So that next time the same disk is picked. | 17:15 |
| arne_wiebalck | JayF: I am not talking about ks, but Ironic proper. | 17:15 |
| arne_wiebalck | JayF: To avoid I rely on the algorithm to give me the same result. | 17:16 |
| JayF | I'm not certain, but I'm trying to assert that "root device hints" is not an ironic-proper thing | 17:16 |
| JayF | it's a direct deploy driver thing | 17:16 |
| JayF | so why should the KS driver cater to the direct driver? | 17:16 |
| arne_wiebalck | JayF: root device hints are not only direct, are they? | 17:17 |
| JayF | I thought they were, but there's a reason I said I wasn't certain :) | 17:17 |
| JayF | I think it's direct and iscsi | 17:18 |
| JayF | and iscsi is deprecated | 17:18 |
| arne_wiebalck | not yet :) | 17:18 |
| JayF | it's deprecated already, but not removed | 17:18 |
| JayF | right? | 17:18 |
| arne_wiebalck | JayF: ha ha ha, let's start that longish discussion once more :-D | 17:19 |
| bdodd | dtantsur|afk I submitted this sushy release request: https://review.opendev.org/c/openstack/releases/+/773453 | 17:19 |
| arne_wiebalck | JayF: I guess that many rely on direct at the moment. | 17:19 |
| JayF | sure, but I think my point still stands | 17:20 |
| JayF | do we encode the ansible driver to leave behind metadata for !ansible drivers? | 17:20 |
| arne_wiebalck | JayF: I assume one will not change drivers for a node. | 17:20 |
| arne_wiebalck | JayF: So, if I build with direct today, I will rebuild with direct tomorrow. | 17:21 |
| rpittau | good night! o/ | 17:21 |
| arne_wiebalck | JayF: Maybe I miss your point ... | 17:21 |
| *** rpittau is now known as rpittau|afk | 17:21 | |
| JayF | You said: Maybe we should have KS driver populate root device hunts after a successful deployment | 17:22 |
| JayF | I said: It doesn't make sense for KS driver to put metadata down that only other [deploy interface] drivers support | 17:22 |
| arne_wiebalck | JayF: I did not say ks driver. | 17:22 |
| JayF | ahhh, I pulled that in from context | 17:22 |
| arne_wiebalck | JayF: sorry for the confusion | 17:22 |
| JayF | why you thought this was a good idea now makes more sense, loool | 17:22 |
| arne_wiebalck | JayF: LOL | 17:22 |
| JayF | If we want to have direct/iscsi drivers do that, I don't have an opinion, sounds like a good idea | 17:22 |
| arne_wiebalck | JayF: Yeah, this would be for direct, sorry. | 17:23 |
| arne_wiebalck | JayF: Basically, ensure rebuild works with direct and does not touch data disks. | 17:23 |
| arne_wiebalck | JayF: By "burning" hints into the node :) | 17:23 |
| arne_wiebalck | TheJulia: was "setting root device hints automatically" already discussed and discarded in the history of Ironic? (before I file a story :)) | 17:32 |
| *** dougsz has quit IRC | 17:48 | |
| *** derekh has quit IRC | 17:56 | |
| TheJulia | I wonder if ironic should tell nova that we support aki/ari/ami format | 17:58 |
| arne_wiebalck | bye everyone o/ | 18:13 |
| *** tosin has quit IRC | 18:29 | |
| *** kaifeng has quit IRC | 18:30 | |
| *** dsneddon has joined #openstack-ironic | 18:42 | |
| *** k_mouza has quit IRC | 18:51 | |
| *** k_mouza has joined #openstack-ironic | 18:52 | |
| *** k_mouza has quit IRC | 18:52 | |
| *** k_mouza has joined #openstack-ironic | 18:52 | |
| *** k_mouza has quit IRC | 18:53 | |
| *** k_mouza has joined #openstack-ironic | 18:53 | |
| *** k_mouza has quit IRC | 18:58 | |
| TheJulia | lbragstad: w/r/t https://review.opendev.org/c/openstack/ironic/+/763257, I'm going to edit it locally and upload it along with re-stacking the changes | 20:09 |
| lbragstad | TheJulia ok - so rule:is_observer was really just a workaround for read-only access to system-level baremetal resources, right? | 20:10 |
| TheJulia | yes | 20:10 |
| lbragstad | ok - perfect | 20:10 |
| TheJulia | I just commented to that effect on the change | 20:11 |
| *** knikolla_ has joined #openstack-ironic | 20:18 | |
| *** guilhermesp__ has joined #openstack-ironic | 20:18 | |
| *** andy__ has joined #openstack-ironic | 20:23 | |
| *** tzumainn has quit IRC | 20:26 | |
| *** knikolla has quit IRC | 20:26 | |
| *** guilhermesp has quit IRC | 20:26 | |
| *** andy_ has quit IRC | 20:26 | |
| *** zigo has quit IRC | 20:26 | |
| *** knikolla_ is now known as knikolla | 20:26 | |
| *** andy__ is now known as andy_ | 20:26 | |
| *** guilhermesp__ is now known as guilhermesp | 20:26 | |
| *** tzumainn has joined #openstack-ironic | 20:31 | |
| *** zigo has joined #openstack-ironic | 20:32 | |
| *** dsneddon has quit IRC | 21:15 | |
| *** gyee has joined #openstack-ironic | 21:16 | |
| janders | good morning Ironic o/ | 21:33 |
| *** anuradha1904 has quit IRC | 21:42 | |
| TheJulia | good morning janders | 22:10 |
| TheJulia | hmmmm yay rbac | 22:11 |
| janders | I see there used to be a Dockerfile in IPA repo which could be modified to add extra tools in (e.g. hdparm - or in my case nvme-cli). Sample change leveraging it: https://opendev.org/openstack/ironic-python-agent/commit/dff46583d354ced83a056d404abc70fd3ac92de3 I don't see it anymore - has some other part of the repo replaced this functionality? | 22:14 |
| TheJulia | janders: everything was moved to ironic-python-agent-builder | 22:18 |
| TheJulia | janders: the dockerfile was for CoreOS (the original) | 22:18 |
| janders | thank you TheJulia, cloning the repo :) | 22:19 |
| TheJulia | lbragstad: you around for a public api question | 22:19 |
| *** rcernin has joined #openstack-ironic | 22:20 | |
| lbragstad | i am | 22:21 |
| TheJulia | is there a canolical way to define an api endpoint as a public api which someone could later use a custom policy to restrict? | 22:22 |
| * lbragstad digs for an example | 22:22 | |
| TheJulia | or is literally "yes, it is project stcoped although not really so it doesn't get caught with that check and no rule? | 22:23 |
| lbragstad | https://opendev.org/openstack/keystone/src/branch/master/keystone/common/policies/auth.py#L33-L48 | 22:23 |
| lbragstad | so - in there, we at the very least register a default and set it to an empty string | 22:24 |
| lbragstad | but - if someone really wanted to lock it down, they could override it with a custom policy | 22:24 |
| TheJulia | seems kind of pointless to define a scope if there is no scope? | 22:25 |
| lbragstad | well - we could define scope_types in that example as ['project', 'domain', 'system'] | 22:26 |
| lbragstad | but we didn't | 22:26 |
| TheJulia | would it default to one? out of curiosity | 22:26 |
| lbragstad | it wouldn't, no | 22:26 |
| lbragstad | which ironic policy are you thinking about? | 22:27 |
| TheJulia | because I feel like an "authenticated" user trying to access the utility APIs needs to be told that the API is a teapot | 22:27 |
| TheJulia | ipa_lookup and ipa_heartbeat | 22:27 |
| lbragstad | mmm | 22:29 |
| lbragstad | ok - so is_public_api gets set on context objects for that particular API, and the rule really just checks to see if that attribute is true | 22:31 |
| TheJulia | yeah, at least in the older rules | 22:31 |
| lbragstad | ok - if i'm understanding you correctly, you might be able to use this | 22:32 |
| lbragstad | http://paste.openstack.org/show/802210/ | 22:32 |
| lbragstad | which will allow anyone with a system-scoped or project-scoped token to access that API when `ironic.conf [oslo_policy] enforce_scope=True` | 22:33 |
| TheJulia | I'm removing scope_types | 22:33 |
| TheJulia | since it is pointless | 22:33 |
| lbragstad | and that should also be true when `ironic.conf [oslo_policy] enforce_scope=False`, too | 22:33 |
| lbragstad | yeah | 22:33 |
| lbragstad | that works | 22:33 |
| lbragstad | if you end up refactoring the rule:public_api bits out, you might be able to remove that kwarg from the context object, too | 22:34 |
| TheJulia | I still think the api may need to see if it is an actual user and reply with 418 for fun | 22:34 |
| TheJulia | yeah, it seems totally removable when we nuke the old policy support | 22:34 |
| janders | TheJulia i/r/t adding nvme-cli to the IPA, image, is it correct to say that all I need to do is to add nvme-cli to ./dib/extra-hardware/package-installs.yaml (+ appropriate release note)? | 22:42 |
| *** dsneddon has joined #openstack-ironic | 22:46 | |
| TheJulia | janders: mostly yes, if nvme-cli is available for tinycore, it would be good to add there as well | 22:46 |
| janders | TheJulia I don't see nvme-cli (or *nvme*) in this repo: http://www.tinycorelinux.net/11.x/x86/tcz/ so I believe that tinycore don't provide nvme-cli. Is looking at this URL a reasonable way to check, or do I need to look elsewhere? (I'm completely new to tinycore) | 22:52 |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent-builder master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/773538 | 22:56 |
| iurygregory | I don't see any nvme in http://www.tinycorelinux.net/11.x/x86_64/tcz/ | 23:02 |
| iurygregory | maybe it would be under other name .-. | 23:02 |
| openstackgerrit | Jacob Anders proposed openstack/ironic master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic/+/771905 | 23:11 |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 23:14 |
| openstackgerrit | Jacob Anders proposed openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 23:31 |
| TheJulia | iurygregory: sadness :( | 23:41 |
| *** tosky has quit IRC | 23:52 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!