openstackgerrit | Michael Chapman proposed openstack/ironic master: Support centos 7 rootwrap data directory https://review.opendev.org/713765 | 00:11 |
---|---|---|
*** diurnalist has quit IRC | 00:12 | |
*** k_mouza has joined #openstack-ironic | 00:16 | |
*** k_mouza has quit IRC | 00:20 | |
*** kaifeng has joined #openstack-ironic | 00:52 | |
*** rloo has quit IRC | 00:53 | |
*** TxGirlGeek has quit IRC | 00:53 | |
*** ianychoi_ has joined #openstack-ironic | 01:38 | |
*** ianychoi has quit IRC | 01:40 | |
*** mgoddard has quit IRC | 01:49 | |
*** mgoddard has joined #openstack-ironic | 01:57 | |
*** ociuhandu has joined #openstack-ironic | 02:04 | |
*** diurnalist has joined #openstack-ironic | 02:09 | |
*** ociuhandu has quit IRC | 02:09 | |
*** diurnalist has quit IRC | 02:14 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Use python3 binary for all operations https://review.opendev.org/713477 | 02:42 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent-builder failed: Use variables for ssh paths https://review.opendev.org/697654 | 02:47 |
*** k_mouza has joined #openstack-ironic | 03:03 | |
*** k_mouza has quit IRC | 03:08 | |
*** rh-jelabarre has quit IRC | 03:15 | |
*** mkrai has joined #openstack-ironic | 03:17 | |
*** diurnalist has joined #openstack-ironic | 03:42 | |
*** gyee has quit IRC | 04:08 | |
*** dsneddon_ has quit IRC | 04:41 | |
*** diurnalist has quit IRC | 05:57 | |
*** michchap has quit IRC | 06:10 | |
*** diurnalist has joined #openstack-ironic | 06:11 | |
*** tzumainn has quit IRC | 06:12 | |
*** riuzen has joined #openstack-ironic | 06:15 | |
*** diurnalist has quit IRC | 06:16 | |
*** dsneddon_ has joined #openstack-ironic | 06:21 | |
*** dsneddon_ has quit IRC | 06:26 | |
*** mkrai has quit IRC | 06:35 | |
*** mkrai has joined #openstack-ironic | 06:40 | |
*** yedongcan has joined #openstack-ironic | 06:47 | |
*** mkrai has quit IRC | 07:02 | |
*** mkrai_ has joined #openstack-ironic | 07:02 | |
*** k_mouza has joined #openstack-ironic | 07:04 | |
arne_wiebalck | Good morning, ironic! | 07:05 |
*** k_mouza has quit IRC | 07:08 | |
*** ociuhandu has joined #openstack-ironic | 07:28 | |
*** amoralej|off is now known as amoralej | 07:32 | |
*** ociuhandu has quit IRC | 07:33 | |
*** riuzen has quit IRC | 07:36 | |
*** tesseract has joined #openstack-ironic | 07:51 | |
*** jawad_axd has joined #openstack-ironic | 07:51 | |
iurygregory | good morning arne_wiebalck and Ironic o/ | 07:55 |
arne_wiebalck | hey iurygregory o/ | 07:55 |
*** sziviani_ has joined #openstack-ironic | 07:59 | |
*** threestrands has quit IRC | 08:01 | |
*** mbeierl has quit IRC | 08:03 | |
*** mbeierl has joined #openstack-ironic | 08:04 | |
*** portdirect has quit IRC | 08:08 | |
*** guilhermesp has quit IRC | 08:09 | |
*** phrobb has quit IRC | 08:09 | |
*** portdirect has joined #openstack-ironic | 08:10 | |
*** mnaser has quit IRC | 08:10 | |
*** phrobb has joined #openstack-ironic | 08:10 | |
*** mnaser has joined #openstack-ironic | 08:11 | |
*** rpittau|afk is now known as rpittau | 08:11 | |
*** guilhermesp has joined #openstack-ironic | 08:11 | |
rpittau | good morning ironic! o/ | 08:11 |
*** tkajinam has quit IRC | 08:12 | |
*** dsneddon_ has joined #openstack-ironic | 08:21 | |
*** dsneddon_ has quit IRC | 08:26 | |
*** dougsz has joined #openstack-ironic | 08:38 | |
*** riuzen has joined #openstack-ironic | 08:44 | |
iurygregory | morning rpittau o/ | 08:46 |
rpittau | hey iurygregory :) | 08:46 |
rpittau | for the tentacles of cthulhu, what's wrong with centos8 now.... | 08:47 |
iurygregory | we can ask on rdo =) | 08:50 |
*** lucasagomes has joined #openstack-ironic | 08:58 | |
riuzen | hi, so I try to install baremetal using ironic. Actually I simulate it under KVM environment. I use a VM with 20GB raw disk as baremetal node. When I exec command openstack baremetal node inspect, and VM start to boot it's said that no space left on device. | 08:59 |
riuzen | Did I missing something? | 08:59 |
iurygregory | maybe you can provide more logs, only by no space left is a bit tricky to debug | 09:04 |
*** yoctozepto has quit IRC | 09:07 | |
*** ociuhandu has joined #openstack-ironic | 09:10 | |
*** diurnalist has joined #openstack-ironic | 09:12 | |
*** uzumaki has joined #openstack-ironic | 09:14 | |
*** yoctozepto0 has joined #openstack-ironic | 09:16 | |
*** diurnalist has quit IRC | 09:18 | |
*** ociuhandu has quit IRC | 09:22 | |
*** yoctozepto9 has joined #openstack-ironic | 09:27 | |
*** yoctozepto0 has quit IRC | 09:27 | |
*** riuzen has quit IRC | 09:34 | |
*** dtantsur|afk is now known as dtantsur | 09:39 | |
dtantsur | morning ironic | 09:40 |
rpittau | hey dtantsur :) | 09:40 |
uzumaki | hey dmitry! | 09:41 |
uzumaki | how you doing? | 09:41 |
dtantsur | surviving somehow :) you? | 09:43 |
*** derekh has joined #openstack-ironic | 09:43 | |
iurygregory | morning dtantsur | 09:45 |
*** riuzen has joined #openstack-ironic | 09:53 | |
*** ociuhandu has joined #openstack-ironic | 09:54 | |
uzumaki | Well, can't complain, been well, had a flu but now much better. This corona thing is getting on everybody's nerves more than it is in their respiratory system | 09:55 |
*** dsneddon_ has joined #openstack-ironic | 09:55 | |
iurygregory | uzumaki++ | 09:56 |
uzumaki | anyway, anybody got an idea what's the purpose of async_step_flags that we set in clean steps? Who consumes those? | 09:57 |
*** ktibi has joined #openstack-ironic | 09:57 | |
rpittau | uzumaki: I tend do disagree, all considered | 09:57 |
ktibi | Hello all, I replace manually the deploy_image in my ironic. Now, IPA have error, different checksum between /tmp/deploy_image and checksum give by ironic. How or Where can I refresh checksum send by ironic ? | 09:58 |
uzumaki | i think you put it in the same directory as a checksum file? as the image? | 09:58 |
ktibi | uzumaki, now I don't have checksum file but IPA give me the old checkum, like ironic send it by API | 10:00 |
*** dsneddon_ has quit IRC | 10:00 | |
ktibi | I can see command like "ironic node-update $UUID add instance_info/image_checksum=$MD5SUM" | 10:01 |
ktibi | but I don't know if it's the good way | 10:01 |
dtantsur | image_checksum is a checksum of the user image, not the deploy one | 10:01 |
dtantsur | I don't think we checksum the deploy image | 10:02 |
ktibi | dtantsur, error from IPA during deploy step | 10:02 |
ktibi | Image failed to verify against checksum. location: /tmp/deployment_image.qcow2; image ID: deployment_image.qcow2; image checksum: 3366a6bf41b91baeac4aa51fb5372a5d; verification checksum: 7c7b252fa5b42736d565f5fb4e19d666 | 10:03 |
dtantsur | ktibi: I guess it's just a name, you need a checksum of instance_info/image_source | 10:04 |
dtantsur | uzumaki: async_step_flags are set by steps and regulate how ironic-conductor processes them | 10:04 |
dtantsur | unfortunately, the docstrings are not the most obvious.. | 10:05 |
uzumaki | dtantsur, ah, I see. I can add comments in my code in case somebody will need to look it up then | 10:05 |
uzumaki | Then I believe if I set appropriate flags in my steps, I'm good to go, no need to consume them manually | 10:05 |
dtantsur | yep | 10:06 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP Raise human-friendly messages on attempt to use pre-deploy steps drivers https://review.opendev.org/713594 | 10:06 |
iurygregory | ktibi, maybe you need to update the node instance-info image_checksum | 10:11 |
uzumaki | perhaps by manually computing the checksum and putting it in the instance-info? | 10:12 |
*** amoralej is now known as amoralej|afk | 10:13 | |
ktibi | iurygregory, Yes I'll try that | 10:13 |
iurygregory | ktibi, cool let us know the result =) | 10:15 |
ktibi | iurygregory, deploy is failed, so only way is stuck in wait call-back so I deleted the node but before that I checked the instance info value and I can see the wrong checksum | 10:18 |
*** mkrai_ has quit IRC | 10:19 | |
uzumaki | correcting the checksum in wait call-back doesn't automatically allow recovery? | 10:20 |
iurygregory | I don't think so | 10:20 |
iurygregory | you probably will need to update the node manually to move from deploy failed | 10:20 |
iurygregory | set the new info and try again? | 10:21 |
ktibi | uzumaki, can't modify if node is not in active or manageable state | 10:21 |
iurygregory | ^ yup | 10:22 |
iurygregory | https://docs.openstack.org/ironic/train/contributor/states.html | 10:23 |
uzumaki | ah, I see. Shouldn't known. I remember now | 10:24 |
uzumaki | should've* | 10:24 |
ktibi | strange schema because when I use delete command, node are not in active or manageable after cleaning but just totaly deleted :/ | 10:25 |
ktibi | node list return me empty list | 10:25 |
dtantsur | the delete command deletes a node, did you mean undeploy? | 10:25 |
ktibi | ok deleting state is not the state after delete command ? | 10:26 |
ktibi | deleting state in the schema if after undeploy ? | 10:26 |
ktibi | is after* | 10:26 |
dtantsur | nope, the namings somewhat mismatch. I'd suggest you rely on ironicclient documentation for checking the right commands. | 10:27 |
*** mkrai_ has joined #openstack-ironic | 10:30 | |
*** mkrai has joined #openstack-ironic | 10:32 | |
*** mkrai_ has quit IRC | 10:35 | |
*** tesseract has quit IRC | 10:36 | |
*** yoctozepto9 is now known as yoctozepto | 10:37 | |
*** k_mouza has joined #openstack-ironic | 10:38 | |
*** tesseract has joined #openstack-ironic | 10:42 | |
*** tesseract has quit IRC | 10:46 | |
*** tesseract has joined #openstack-ironic | 10:48 | |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver https://review.opendev.org/713450 | 10:48 |
*** riuzen has quit IRC | 10:51 | |
iurygregory | dtantsur, I'm wondering if we want to generate to token with any characters (before we are only considering ascii_letters and digits) and now we include - and _ | 10:54 |
dtantsur | iurygregory: we don't check the character set anywhere | 10:54 |
iurygregory | do we want to also generate the salt using token_urlsafe? | 10:55 |
dtantsur | iurygregory: mm, no, salt should be generated with mksalt or whatever is designed for it | 10:55 |
iurygregory | https://review.opendev.org/#/c/713690/1/ironic_python_agent/extensions/rescue.py | 10:55 |
patchbot | patch 713690 - ironic-python-agent - Use secrets to generate salt - 1 patch set | 10:55 |
dtantsur | it probably does not matter, but let's use clearly targeted functions | 10:55 |
iurygregory | ack | 10:56 |
ktibi | oh yeah !! RED Screen on my proliant after the deploy \o/ | 10:56 |
dtantsur | iurygregory: i.e. this one https://docs.python.org/3.5/library/crypt.html#crypt.mksalt | 10:56 |
iurygregory | dtantsur, I'll change ipa to use that | 10:58 |
*** ociuhandu has quit IRC | 10:58 | |
dtantsur | thx! | 10:58 |
iurygregory | I will remove the function make_salt... since we can just call from crypt | 10:58 |
*** tesseract-RH has joined #openstack-ironic | 11:00 | |
openstackgerrit | Mark Goddard proposed openstack/bifrost stable/train: WIP: CentOS 8 support (Train) https://review.opendev.org/713508 | 11:00 |
*** yolanda has quit IRC | 11:01 | |
*** tesseract-RH has left #openstack-ironic | 11:02 | |
*** tesseract has quit IRC | 11:03 | |
*** ociuhandu has joined #openstack-ironic | 11:03 | |
uzumaki | ktibi, is that a good thing? | 11:04 |
*** ociuhandu has quit IRC | 11:04 | |
*** ociuhandu has joined #openstack-ironic | 11:05 | |
*** riuzen has joined #openstack-ironic | 11:06 | |
ktibi | uzumaki, no, it's like blue screen but for hardware :p | 11:07 |
iurygregory | dtantsur, any problem if we remove the salt by default crypt.crypt will generate one if we don't pass any, wdyt? https://docs.python.org/3/library/crypt.html#crypt.crypt | 11:09 |
uzumaki | ktibi, aha! I knew it was sarcasm :D | 11:10 |
dtantsur | iurygregory: seems that we'll need to pass crypt.METHOD_CRYPT as that's what we're currently using | 11:11 |
dtantsur | (judging by 2-character salt) | 11:11 |
dtantsur | otherwise, I agree | 11:11 |
iurygregory | dtantsur, we can or we can let it choose the strongest it can find | 11:12 |
iurygregory | https://github.com/python/cpython/blob/0d3fe8ae4961bf551e7d5e42559e2ede1a08fd7c/Lib/crypt.py#L70-L82 | 11:12 |
dtantsur | iurygregory: I understand, but I don't know if it really checks what the underlying OS supports.. | 11:13 |
*** yolanda has joined #openstack-ironic | 11:13 | |
*** Lucas_Gray has joined #openstack-ironic | 11:13 | |
ktibi | during the deploy, ipa get error on oslo.service.loopingcall. just before dhcp call. After that I can start manualy IPA and ipa install image and shutdown node. Any suggestion for debug that ? I turn on debug mode on IPA but can't see log of oslo in log (just in console during boot) | 11:16 |
*** dtantsur is now known as dtantsur|afk | 11:20 | |
dtantsur|afk | not feeling well today, will go afk for recovery. likely talk to you tomorrow. | 11:21 |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver https://review.opendev.org/713450 | 11:21 |
rpittau | dtantsur|afk: take care | 11:21 |
uzumaki | dtantsur|afk, take care! gws | 11:21 |
iurygregory | dtantsur|afk, take care o/ | 11:22 |
*** riuzen has quit IRC | 11:26 | |
*** rpittau is now known as rpittau|bbl | 11:30 | |
ktibi | uzumaki, yes ,) sanboot command from ironic make a redscreen ^^ | 11:33 |
openstackgerrit | Ilya Etingof proposed openstack/sushy master: Lazily load message registries https://review.opendev.org/713512 | 11:34 |
openstackgerrit | Merged openstack/bifrost stable/train: Ensure DIB is installed from source https://review.opendev.org/713443 | 11:50 |
*** ociuhandu has quit IRC | 11:50 | |
uzumaki | ktibi, never tried a SAN boot, must be interesting seeing that redscreen thing (not to say I wish to see one when I'm testing my ironic code :D) | 11:53 |
ktibi | uzumaki, sanboot is command in pxe for boot on localdisk :p | 11:54 |
*** rh-jelabarre has joined #openstack-ironic | 11:54 | |
ktibi | But before fix this issue, I try to understand why IPA run before DHCP request and fail with "max retries exceded" | 11:55 |
*** dsneddon_ has joined #openstack-ironic | 11:57 | |
*** michchap has joined #openstack-ironic | 11:58 | |
*** dsneddon_ has quit IRC | 12:01 | |
*** k_mouza has quit IRC | 12:06 | |
*** k_mouza has joined #openstack-ironic | 12:07 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Use secrets to generate salt https://review.opendev.org/713690 | 12:14 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Do not use random to generate token https://review.opendev.org/713687 | 12:15 |
openstackgerrit | Merged openstack/sushy master: Add `Connector(..., response_callback=None)`` parameter https://review.opendev.org/713315 | 12:18 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Use crypt to generate salt https://review.opendev.org/713690 | 12:19 |
*** ociuhandu has joined #openstack-ironic | 12:22 | |
*** amoralej|afk is now known as amoralej | 12:28 | |
*** ociuhandu has quit IRC | 12:28 | |
*** hjensas has quit IRC | 12:33 | |
*** hjensas has joined #openstack-ironic | 12:35 | |
*** k_mouza has quit IRC | 12:42 | |
*** jdandrea has joined #openstack-ironic | 12:45 | |
openstackgerrit | Mark Goddard proposed openstack/bifrost stable/train: WIP: CentOS 8 support (Train) https://review.opendev.org/713508 | 12:47 |
*** Wryhder has joined #openstack-ironic | 12:51 | |
*** k_mouza has joined #openstack-ironic | 12:52 | |
*** Lucas_Gray has quit IRC | 12:53 | |
*** Wryhder is now known as Lucas_Gray | 12:53 | |
uzumaki | ktibi, sanboot - boot from local disk? Oh gosh. What is the world becoming... | 12:54 |
*** jdandrea has quit IRC | 12:56 | |
*** rloo has joined #openstack-ironic | 12:59 | |
*** rpittau|bbl is now known as rpittau | 12:59 | |
openstackgerrit | Mark Goddard proposed openstack/tenks master: Refactor virtualbmc-domain tasks into a module https://review.opendev.org/713043 | 13:00 |
*** jdandrea has joined #openstack-ironic | 13:03 | |
*** mkrai has quit IRC | 13:09 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Workaround for missing adjtime file https://review.opendev.org/713651 | 13:14 |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Use variables for ssh paths https://review.opendev.org/697654 | 13:14 |
*** tzumainn has joined #openstack-ironic | 13:22 | |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver https://review.opendev.org/713450 | 13:24 |
*** derekh has quit IRC | 13:26 | |
TheJulia | rpittau: You have invoked the dark lord. Now I'm worried ;) | 13:33 |
rpittau | TheJulia: IT always keep an eye on me, on my desk :) | 13:34 |
TheJulia | iurygregory: Please add a release note, a story id, and a task id to https://review.opendev.org/#/c/713690/3 | 13:38 |
patchbot | patch 713690 - ironic-python-agent - Use crypt to generate salt - 3 patch sets | 13:38 |
iurygregory | TheJulia, ack o/ | 13:39 |
rpittau | probably needed for https://review.opendev.org/713687 too | 13:39 |
patchbot | patch 713687 - ironic - Do not use random to generate token - 2 patch sets | 13:39 |
iurygregory | rpittau, sure | 13:40 |
TheJulia | one concern regarding switching to token_urlsafe is the shorter length | 13:44 |
iurygregory | TheJulia, we can say the bytes we want | 13:45 |
iurygregory | if you want 128 chars I just need to add 96 or 95 as parameter | 13:45 |
TheJulia | I'm also generally slightly paranoid | 13:45 |
iurygregory | want me to update to keep the 128 chars? =) | 13:46 |
uzumaki | anybody take a quick look at the BIOSInterface code, I've finished the main part, wanna know your thoughts if anybody has the time | 13:46 |
uzumaki | https://review.opendev.org/#/c/713450 | 13:46 |
patchbot | patch 713450 - ironic - WIP: [RFE] Implement the bios-interface for idrac-... - 10 patch sets | 13:46 |
TheJulia | I made it 128 chars just to be near impossible to copy it easily | 13:46 |
TheJulia | so it would scroll of a screen, wrap in a terminal | 13:46 |
* TheJulia plays the because I'm evil song | 13:46 | |
iurygregory | TheJulia, I'm ok with having 128 chars =) it won't change much | 13:47 |
uzumaki | most of the patchsets are pep8 compliance stuff, 2/3 major ones | 13:47 |
iurygregory | but do I need to update the results in test to have 128 chars also? hehehe | 13:47 |
TheJulia | https://www.youtube.com/watch?v=tWYCS6k1IOA | 13:47 |
TheJulia | iurygregory: likely :) maybe at least a length or something... just in case token_urlsafe ever chagnes | 13:48 |
* iurygregory opens the link in a new tab | 13:48 | |
*** mkrai has joined #openstack-ironic | 13:48 | |
* uzumaki opens the link as well, sounds wonderful (even though it's evil) | 13:49 | |
TheJulia | uzumaki: yay for pep8 :) | 13:49 |
TheJulia | uzumaki: I guess net is unit tests? | 13:50 |
TheJulia | s/net/next | 13:50 |
uzumaki | TheJulia, used to be a pain, but now I'm getting used to writing pep8 complaint code, not having to go through the editing pain later, getting better.. | 13:50 |
uzumaki | TheJulia, indeed! Unit tests coming up | 13:50 |
iurygregory | this song is from a movie right? O.o | 13:50 |
uzumaki | iurygregory, an evil movie, i'm sure | 13:51 |
iurygregory | uzumaki, hehehehe | 13:51 |
TheJulia | iurygregory: it is an entire albumn by voltaire | 13:51 |
uzumaki | TheJulia, voltaire as in Voltaire the philosopher? | 13:51 |
TheJulia | the musician | 13:51 |
uzumaki | too bad, i would've liked to hear Voltaire sing :D | 13:52 |
rpittau | uzumaki: I'm pretty sure he would not have a nice voice at the moment | 13:52 |
TheJulia | https://www.youtube.com/watch?v=8DxUHeiFHmo&list=PL75F59375E0AA6E1A <-- the entire album | 13:52 |
rpittau | mostly sounds like braaaainnssss | 13:53 |
*** amoralej is now known as amoralej|lunch | 13:53 | |
TheJulia | Actually, this might be good music for the day | 13:53 |
*** mkrai has quit IRC | 13:54 | |
* TheJulia realizes that she has revealed how dark her soul is to IRC | 13:54 | |
TheJulia | DOH! | 13:54 |
* uzumaki thinks Julia can never fully reveal how dark she _actually_ is | 13:55 | |
uzumaki | rpittau, lol for the 'at the moment' part | 13:55 |
*** k_mouza has quit IRC | 13:55 | |
*** Lucas_Gray has quit IRC | 13:56 | |
TheJulia | Well, time to begin talking about a virtual ptg I guess https://www.openstack.org/events/covid-19-coronavirus-disease-updates | 13:57 |
*** dsneddon_ has joined #openstack-ironic | 13:57 | |
*** k_mouza has joined #openstack-ironic | 13:57 | |
uzumaki | yeah, just saw that this morning :( | 13:57 |
rpittau | considering that I was not coming to Vancouver, I'm happy(?) about that | 13:57 |
uzumaki | rpittau, what's wrong with vancouver? (besides it being in canada) | 13:58 |
TheJulia | heh | 13:58 |
* TheJulia would get popcorn if she could eat it | 13:58 | |
rpittau | nothing, I've beene there and I ovled it, just had other plans for June | 13:58 |
uzumaki | rpittau, ah, I see. I wanna be there as well, apparently, I won't be able to :/ | 13:58 |
openstackgerrit | Mark Goddard proposed openstack/bifrost stable/train: CentOS 8 support (Train) https://review.opendev.org/713508 | 13:58 |
* uzumaki thinks Julia could eat it all the same | 13:58 | |
TheJulia | uzumaki: corn allergy | 13:59 |
rpittau | uzumaki: last time I was there for the summit I actually got gifts from the city tourism office for one of my pictures on instagram :) | 13:59 |
mgoddard | ^ Bifrost train CentOS 8 support ready to go | 13:59 |
TheJulia | So I leave popcorn for my wife, in accordance with the prophecy | 13:59 |
TheJulia | mgoddard: <3 | 13:59 |
rpittau | mgoddard: awesome! | 13:59 |
uzumaki | rpittau, Woahh, that's so cool! | 13:59 |
TheJulia | mgoddard: Will review it after I am no longer wearing pajamas, which should be shortly | 13:59 |
rpittau | yeah, they're super nice :) | 13:59 |
mgoddard | TheJulia: great :) | 14:00 |
*** derekh has joined #openstack-ironic | 14:00 | |
rpittau | mgoddard: I will review even if I'm on my pajamas :P | 14:00 |
*** Lucas_Gray has joined #openstack-ironic | 14:00 | |
uzumaki | TheJulia, Oh! too bad. which prophecy is this? | 14:00 |
uzumaki | rpittau, lol!! | 14:00 |
mgoddard | WFH == pajamas | 14:00 |
mgoddard | at best | 14:00 |
rpittau | \o/ | 14:00 |
iurygregory | mgoddard, ++ | 14:01 |
uzumaki | mgoddard, lol for 'at best' | 14:01 |
*** dsneddon_ has quit IRC | 14:01 | |
* rloo hears Canada and perks up. Phew, only Canada lovers out there. | 14:01 | |
uzumaki | rloo, I really wanted to visit, given the 'coronic' circumstances, I can't unfortunately :( | 14:02 |
rloo | uzumaki: the vancouver area is lovely (if you love outdoors, nature). Hopefully there will be another time. | 14:02 |
uzumaki | rloo, totally looking forward to it! | 14:03 |
rloo | uzumaki: :) | 14:03 |
iurygregory | TheJulia, for the ipa change, should I use security or fixes in the release note? I have mixed feelings .. | 14:04 |
TheJulia | If it wasn't for the double taxing being a us citizen, I'd definitely try to move to canada. Maybe when I'm retired. | 14:04 |
TheJulia | iurygregory: yeah, might as well | 14:04 |
TheJulia | mgoddard: pajama WFH gets old so quick.... so so quick. | 14:05 |
uzumaki | TheJulia, retired? that's never happening, redundant conditional in the code! | 14:05 |
TheJulia | uzumaki: I had hope ;) | 14:06 |
rpittau | mgoddard: one thing, did you not add the tinyipa integration job on purpose to project.yaml ? | 14:07 |
rpittau | ah no nvm | 14:07 |
rpittau | I'm blind | 14:07 |
mgoddard | :0 | 14:08 |
mgoddard | :) | 14:08 |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: [WIP] Add `network_data` field to ports and portgroups https://review.opendev.org/687910 | 14:10 |
* uzumaki doesn't like when the ironic channel is quiet | 14:17 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Use crypt to generate salt https://review.opendev.org/713690 | 14:18 |
*** jcoufal has joined #openstack-ironic | 14:18 | |
*** amoralej|lunch is now known as amoralej | 14:22 | |
rpittau | uzumaki: we need silence and peace to properly plan world conquering | 14:22 |
*** diurnalist has joined #openstack-ironic | 14:23 | |
iurygregory | TheJulia, I do agree with dtantsur|afk 's comment that we get a bonus by using the default of token_urlsafe =) | 14:23 |
iurygregory | but if you think we should increase I can update now together with the release note | 14:24 |
TheJulia | iurygregory: might as well | 14:27 |
* uzumaki accedes to the mighty cause rpittau has mentioned | 14:27 | |
*** dking_desktop has quit IRC | 14:28 | |
uzumaki | just curious, do we have documentation on writing ironic unit tests? I am aware of a python project guide, and ironic testing guide | 14:28 |
TheJulia | uzumaki: nothing terribly specific, there are existing patterns in the other driver modules that should prove informative | 14:29 |
uzumaki | TheJulia, great. I'll take a look at those. thanks | 14:29 |
TheJulia | Basically base test cases should serve most needs, and you'll likely just want/need to mock some backend library methods | 14:30 |
uzumaki | that sounds even better, let's see | 14:32 |
*** diurnalist has quit IRC | 14:32 | |
TheJulia | etingof: any chance we could get some more detail on failure types? I'm kind of in this mixed feeling area because I'm lacking something to the puzzle. | 14:40 |
* etingof still misses the context | 14:41 | |
*** diurnalist has joined #openstack-ironic | 14:41 | |
etingof | any quick link/pointer handy? | 14:41 |
TheJulia | etingof: I guess a story/task would help as an example of what is occuring so I can feel more confidence to the change | 14:41 |
TheJulia | oh! | 14:41 |
TheJulia | I forgot the bugid | 14:41 |
TheJulia | err | 14:41 |
TheJulia | changeset | 14:41 |
TheJulia | https://review.opendev.org/#/c/712655/ | 14:41 |
patchbot | patch 712655 - sushy - Ignore failing message registry download - 1 patch set | 14:41 |
etingof | no worries | 14:41 |
etingof | ah, sure! that's iLO serving a dict where a string is expected blowing up urllib call inside sushy | 14:43 |
etingof | let me do a Story and update the patch | 14:43 |
TheJulia | SWEET | 14:44 |
TheJulia | could you put that in a story with some example data? | 14:44 |
etingof | yes, indeed | 14:46 |
etingof | btw, we also have it noted here -- https://bugzilla.redhat.com/show_bug.cgi?id=1804127#c11 | 14:47 |
openstack | bugzilla.redhat.com bug 1804127 in python-sushy "Importing results in "Error: Cannot mix str and non-str arguments"" [Medium,New] - Assigned to ietingof | 14:47 |
*** cdearborn has joined #openstack-ironic | 14:47 | |
TheJulia | sweet! I'd just note that in the commit message then | 14:50 |
etingof | indeed! there has been a discussion on the channel that unless Redfish client is sending the specific odata-version, Redfish agent is free to represent some things (url included) in its own weird way | 14:51 |
TheJulia | ahh yeah, that discussion | 14:51 |
etingof | trouble is that sushy is sending that magical header | 14:51 |
TheJulia | Oo | 14:54 |
TheJulia | Yeah, I thought it was, but yeah | 14:54 |
*** jcoufal_ has joined #openstack-ironic | 15:01 | |
*** jcoufal has quit IRC | 15:03 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Do not use random to generate token https://review.opendev.org/713687 | 15:04 |
*** ricolin has quit IRC | 15:08 | |
*** ricolin has joined #openstack-ironic | 15:13 | |
openstackgerrit | Merged openstack/ironic stable/train: Documentation clarifications for software RAID https://review.opendev.org/713645 | 15:19 |
*** jawad_axd has quit IRC | 15:31 | |
*** jawad_axd has joined #openstack-ironic | 15:42 | |
*** dking_desktop has joined #openstack-ironic | 15:57 | |
*** dsneddon_ has joined #openstack-ironic | 15:57 | |
*** Lucas_Gray has quit IRC | 15:59 | |
*** dsneddon_ has quit IRC | 16:02 | |
*** ociuhandu has joined #openstack-ironic | 16:04 | |
*** ociuhandu has quit IRC | 16:06 | |
*** ociuhandu_ has joined #openstack-ironic | 16:06 | |
*** k_mouza has quit IRC | 16:07 | |
*** Lucas_Gray has joined #openstack-ironic | 16:08 | |
*** k_mouza has joined #openstack-ironic | 16:08 | |
*** yedongcan has left #openstack-ironic | 16:15 | |
rpioso | Good morning, ironic | 16:17 |
rpioso | etingof: o/ | 16:17 |
rpioso | iurygregory: \o | 16:17 |
iurygregory | rpioso, o/ | 16:17 |
iurygregory | sorry for the wrong ping other day hehehe | 16:17 |
rpioso | etingof: When you have a moment, please review https://review.opendev.org/#/c/710416/ | 16:18 |
patchbot | patch 710416 - x/sushy-oem-idrac - Drop support for Python 2.7 - 6 patch sets | 16:18 |
rpioso | etingof: I believe rpittau addressed all of your previous comments. | 16:18 |
rpioso | iurygregory: No worries at all. | 16:18 |
*** k_mouza has quit IRC | 16:18 | |
rpioso | :-) | 16:18 |
uzumaki | hey rpioso ! | 16:19 |
uzumaki | morning | 16:19 |
rpioso | uzumaki: :-) | 16:19 |
rpioso | uzumaki: And good evening to you! | 16:19 |
*** k_mouza has joined #openstack-ironic | 16:19 | |
uzumaki | be sure to have a look https://review.opendev.org/#/c/713450 I'm adding the unit tests, shall be done tomorrow, then with the release note and doc update, will finish it | 16:19 |
patchbot | patch 713450 - ironic - WIP: [RFE] Implement the bios-interface for idrac-... - 10 patch sets | 16:19 |
*** Lucas_Gray has quit IRC | 16:21 | |
uzumaki | any ideas why stash.dellemc-community.org/logs throws 404 Not Found when I try to look at the CI logs? | 16:21 |
rpioso | uzumaki: Do you have the entire link? | 16:23 |
uzumaki | well, you can look at the change I sent, 3 Dell ironic CI jobs have failed, I'm unable to access their logs | 16:23 |
*** Lucas_Gray has joined #openstack-ironic | 16:24 | |
iurygregory | TheJulia, do you know for how long HP CI keep the logs? also wondering if we should tell them to drop py2 jobs in CI =) | 16:25 |
*** erbarr has joined #openstack-ironic | 16:26 | |
etingof | rpioso, sure o/ | 16:27 |
TheJulia | iurygregory: twoish weeks I think and I thought they had when they brought CI back but I guess not? | 16:27 |
iurygregory | I guess not also =) | 16:28 |
iurygregory | maybe a topic to raise in the next weekly meeting? | 16:28 |
TheJulia | +++++ | 16:29 |
TheJulia | Yes | 16:29 |
* iurygregory adding to the wiki | 16:30 | |
TheJulia | iurygregory: thanks! | 16:31 |
rpioso | etingof: ty | 16:34 |
*** jdandrea has quit IRC | 16:35 | |
rpioso | etingof: I see I missed a ping of yours yesterday. Is it still relevant? | 16:35 |
*** dsneddon_ has joined #openstack-ironic | 16:38 | |
etingof | rpioso, well, I can't grok how bios works so far | 16:41 |
etingof | also, are all bios attributes treated the same? can some be set right away, but some require reboot? | 16:42 |
*** dsneddon_ has quit IRC | 16:44 | |
*** jcoufal_ has quit IRC | 16:46 | |
*** TxGirlGeek has joined #openstack-ironic | 16:48 | |
TheJulia | Is there consistency at all? or are only some consistent one way and others another way? | 16:49 |
TheJulia | What is the pattern of consistency that we need to adhere to, seems like the question to me. | 16:49 |
*** lucasagomes has quit IRC | 16:50 | |
*** k_mouza has quit IRC | 16:51 | |
*** k_mouza has joined #openstack-ironic | 16:52 | |
uzumaki | are we talking about consistency in the handling of BIOS parameters? or in their names? TheJulia | 16:52 |
openstackgerrit | Merged openstack/ironic-tempest-plugin master: Remove check for API version in validate_scheduling https://review.opendev.org/712030 | 17:04 |
iurygregory | TheJulia, enabled_drivers in the tempest.conf will always be empty? | 17:06 |
openstackgerrit | Ilya Etingof proposed openstack/sushy master: Ignore failing message registry download https://review.opendev.org/712655 | 17:06 |
*** adrianc_ has quit IRC | 17:06 | |
iurygregory | I'm looking at many jobs that runs tempest and it's always empty (but we have default values lol) | 17:06 |
*** adrianc has joined #openstack-ironic | 17:06 | |
openstackgerrit | Mark Goddard proposed openstack/bifrost stable/train: CentOS 8 support (Train) https://review.opendev.org/713508 | 17:10 |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver https://review.opendev.org/713450 | 17:15 |
*** gyee has joined #openstack-ironic | 17:19 | |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver https://review.opendev.org/713450 | 17:21 |
openstackgerrit | Merged x/sushy-oem-idrac master: Drop support for Python 2.7 https://review.opendev.org/710416 | 17:21 |
etingof | rpioso, ^ | 17:28 |
rpioso | etingof: They all require a reboot to be applied. | 17:31 |
etingof | surprisingly, reboot does not change things | 17:31 |
* etingof tried to turn it off and on again already | 17:32 | |
*** alexmcleod has quit IRC | 17:37 | |
*** ociuhandu_ has quit IRC | 17:42 | |
*** ociuhandu has joined #openstack-ironic | 17:42 | |
*** k_mouza has quit IRC | 17:43 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: [WIP] Get driver from config https://review.opendev.org/712041 | 17:45 |
rpioso | erbarr: uzumaki gets a 404 Not Found error when he tries to access the logs of the most recent Dell EMC 3d Party CI job failures for https://review.opendev.org/#/c/713450/. So do I :-( | 17:47 |
patchbot | patch 713450 - ironic - WIP: [RFE] Implement the bios-interface for idrac-... - 12 patch sets | 17:47 |
*** ociuhandu has quit IRC | 17:48 | |
rpioso | erbarr: Well, I was receiving that error. Now I'm not \o/ | 17:48 |
erbarr | rpioso, mmmnn the transfer of the logs happens every minute i think, then they get deleted after 30 days | 17:49 |
*** k_mouza has joined #openstack-ironic | 17:50 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Do not use random to generate token https://review.opendev.org/713687 | 17:58 |
*** ktibi has quit IRC | 17:58 | |
*** ociuhandu has joined #openstack-ironic | 18:01 | |
*** dougsz has quit IRC | 18:02 | |
rpioso | etingof: How did you try to set them? | 18:02 |
*** derekh has quit IRC | 18:05 | |
etingof | rpioso, by PATCH'ing BIOS document I guess... I am calling sushy | 18:05 |
rpioso | That's insufficient. | 18:06 |
rpittau | good night! o/ | 18:06 |
etingof | + rebooting! | 18:06 |
*** rpittau is now known as rpittau|afk | 18:06 | |
rpioso | etingof: A config job is needed to apply them at reboot. | 18:06 |
etingof | so plain Redfish will not work? | 18:07 |
etingof | it does not refuse my PATCH though, just ignores me | 18:07 |
rpioso | etingof: It doesn't ignore you! | 18:07 |
etingof | well, may be it watches me | 18:08 |
etingof | so what's the right approach to manage bios attributes over redfish bios branch? | 18:09 |
etingof | should we push vendor-specific fishiness to sushy-oem-idrac again? | 18:09 |
etingof | TheJulia, speaking of prospective ironic BIOS use - looking at your crystal ball, what would be the most likely bios attributes that the operator might want to tackle? besides reset to factory default | 18:13 |
TheJulia | etingof: i need to get past meetings before I can order into the crystal ball today :( | 18:15 |
etingof | no worries! | 18:16 |
* rpioso believes etingof can't be ignored by a BMC | 18:16 | |
* etingof should authenticate as etingof rather than root, may be that explains my misery | 18:17 | |
*** dsneddon_ has joined #openstack-ironic | 18:18 | |
*** dsneddon has quit IRC | 18:20 | |
*** dsneddon has joined #openstack-ironic | 18:21 | |
*** dsneddon has quit IRC | 18:22 | |
*** dsneddon has joined #openstack-ironic | 18:22 | |
*** dsneddon has quit IRC | 18:22 | |
*** dsneddon has joined #openstack-ironic | 18:22 | |
*** Lucas_Gray has quit IRC | 18:25 | |
*** ociuhandu has quit IRC | 18:27 | |
*** ociuhandu has joined #openstack-ironic | 18:28 | |
*** k_mouza has quit IRC | 18:29 | |
*** k_mouza has joined #openstack-ironic | 18:32 | |
rpioso | etingof: lol | 18:33 |
*** ociuhandu has quit IRC | 18:33 | |
*** gyee has quit IRC | 18:44 | |
TheJulia | etingof: so by attributes, do you mean more so actions like resetting to factory defaults or do you mean attributes like explicit "turn on xyz" or "set y profile" knobs? The structure was already geared for key/value setting exposure/set capability, which means your question as posed is slight confusing and compounding my headache. | 18:48 |
etingof | TheJulia, I am sorry for causing more headache! | 18:49 |
TheJulia | No worries | 18:49 |
etingof | Let's return to this discussion when you will feel better | 18:49 |
etingof | it's not blocking anything so far | 18:49 |
TheJulia | No better time than noe | 18:50 |
TheJulia | Now | 18:50 |
TheJulia | Besides, maybe it will reinvigorate my brain! | 18:50 |
etingof | okay, I mean which keys are most likely be used | 18:50 |
uzumaki | I'm off now guys. See y'all tomorrow! o/ Hoping to finish my story tomorrow | 18:50 |
*** gyee has joined #openstack-ironic | 18:51 | |
TheJulia | etingof: ones for virtualization, memory mapping behavior, execute bit are there that come to mind. Security related ones too, but the issue with the keys is they are vendor specific... Like where Dell may have an st-iov setting, suoermicro may not and vice versa | 18:54 |
TheJulia | Sr-iov | 18:55 |
etingof | yeah, okay, I will play with those | 18:55 |
*** uzumaki has quit IRC | 18:55 | |
etingof | even if those keys were standardized, the vendors would make them vendor-specific anyway | 18:56 |
etingof | so no worries | 18:56 |
*** bfournie has quit IRC | 19:01 | |
*** jawad_axd has quit IRC | 19:04 | |
TheJulia | Yeah :( | 19:05 |
TheJulia | etingof: are you trying to navigate a passthrough for the actual settings object, not just the bios object? | 19:05 |
*** k_mouza has quit IRC | 19:07 | |
etingof | TheJulia, I am using sushy to change BIOS attributes in exactly the same way as ironic does. I can see in debugging that sushy respects the Settings object when PATCH'ing iDRAC | 19:10 |
TheJulia | Okay, I was going to say because settinngd and attributes are very different in the idracs from what I have read | 19:12 |
etingof | sushy takes it simple - just PATCHes by the URL in the Settings object including only the desired key-value pairs to the message body | 19:13 |
*** TxGirlGeek has quit IRC | 19:14 | |
etingof | if you have a link handy of what you've read about bios@idrac - that would be helpful | 19:15 |
*** dsneddon has quit IRC | 19:15 | |
*** dsneddon has joined #openstack-ironic | 19:16 | |
TheJulia | etingof: Just the dell published api spec | 19:25 |
etingof | okay | 19:25 |
*** gyee has quit IRC | 20:03 | |
*** Lucas_Gray has joined #openstack-ironic | 20:20 | |
*** dsneddon_ has quit IRC | 20:57 | |
*** ijw has quit IRC | 20:57 | |
*** ijw has joined #openstack-ironic | 21:11 | |
*** ijw has quit IRC | 21:15 | |
*** ociuhandu has joined #openstack-ironic | 21:33 | |
*** ijw has joined #openstack-ironic | 21:37 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Fix agent token vmedia use https://review.opendev.org/713989 | 21:38 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Signal agent token is required https://review.opendev.org/699304 | 21:38 |
TheJulia | dtantsur|afk: I'd appreciate a quick look at https://review.opendev.org/713989 tomorrow. \o/ | 21:39 |
patchbot | patch 713989 - ironic-python-agent - Fix agent token vmedia use - 1 patch set | 21:39 |
openstackgerrit | Harald Jensås proposed openstack/ironic master: Additional IP addresses to IPv6 stateful ports https://review.opendev.org/700002 | 21:40 |
*** dhellmann_ has joined #openstack-ironic | 21:53 | |
*** dhellmann has quit IRC | 21:54 | |
*** dhellmann_ is now known as dhellmann | 21:54 | |
*** ktibi has joined #openstack-ironic | 21:54 | |
openstackgerrit | Harald Jensås proposed openstack/ironic master: Additional IP addresses to IPv6 stateful ports https://review.opendev.org/700002 | 21:58 |
*** ktibi has quit IRC | 21:58 | |
*** bfournie has joined #openstack-ironic | 22:00 | |
*** gyee has joined #openstack-ironic | 22:00 | |
*** k_mouza has joined #openstack-ironic | 22:09 | |
*** ociuhandu has quit IRC | 22:09 | |
*** ociuhandu has joined #openstack-ironic | 22:10 | |
*** k_mouza has quit IRC | 22:14 | |
*** ociuhandu has quit IRC | 22:15 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Clean up nits from adding additional node update policies https://review.opendev.org/713462 | 22:20 |
*** zigo has quit IRC | 22:22 | |
*** sziviani_ has quit IRC | 22:23 | |
*** Lucas_Gray has quit IRC | 22:31 | |
*** zigo has joined #openstack-ironic | 22:31 | |
*** diurnalist has quit IRC | 23:12 | |
*** k_mouza has joined #openstack-ironic | 23:25 | |
*** k_mouza has quit IRC | 23:29 | |
openstackgerrit | Merged openstack/ironic master: Support centos 7 rootwrap data directory https://review.opendev.org/713765 | 23:47 |
openstackgerrit | Verification of a change to openstack/ironic failed: Don't require root partition when installing a whole disk image https://review.opendev.org/707676 | 23:47 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!