Thursday, 2020-03-19

openstackgerritMichael Chapman proposed openstack/ironic master: Support centos 7 rootwrap data directory  https://review.opendev.org/71376500:11
*** diurnalist has quit IRC00:12
*** k_mouza has joined #openstack-ironic00:16
*** k_mouza has quit IRC00:20
*** kaifeng has joined #openstack-ironic00:52
*** rloo has quit IRC00:53
*** TxGirlGeek has quit IRC00:53
*** ianychoi_ has joined #openstack-ironic01:38
*** ianychoi has quit IRC01:40
*** mgoddard has quit IRC01:49
*** mgoddard has joined #openstack-ironic01:57
*** ociuhandu has joined #openstack-ironic02:04
*** diurnalist has joined #openstack-ironic02:09
*** ociuhandu has quit IRC02:09
*** diurnalist has quit IRC02:14
openstackgerritMerged openstack/ironic-python-agent-builder master: Use python3 binary for all operations  https://review.opendev.org/71347702:42
openstackgerritVerification of a change to openstack/ironic-python-agent-builder failed: Use variables for ssh paths  https://review.opendev.org/69765402:47
*** k_mouza has joined #openstack-ironic03:03
*** k_mouza has quit IRC03:08
*** rh-jelabarre has quit IRC03:15
*** mkrai has joined #openstack-ironic03:17
*** diurnalist has joined #openstack-ironic03:42
*** gyee has quit IRC04:08
*** dsneddon_ has quit IRC04:41
*** diurnalist has quit IRC05:57
*** michchap has quit IRC06:10
*** diurnalist has joined #openstack-ironic06:11
*** tzumainn has quit IRC06:12
*** riuzen has joined #openstack-ironic06:15
*** diurnalist has quit IRC06:16
*** dsneddon_ has joined #openstack-ironic06:21
*** dsneddon_ has quit IRC06:26
*** mkrai has quit IRC06:35
*** mkrai has joined #openstack-ironic06:40
*** yedongcan has joined #openstack-ironic06:47
*** mkrai has quit IRC07:02
*** mkrai_ has joined #openstack-ironic07:02
*** k_mouza has joined #openstack-ironic07:04
arne_wiebalckGood morning, ironic!07:05
*** k_mouza has quit IRC07:08
*** ociuhandu has joined #openstack-ironic07:28
*** amoralej|off is now known as amoralej07:32
*** ociuhandu has quit IRC07:33
*** riuzen has quit IRC07:36
*** tesseract has joined #openstack-ironic07:51
*** jawad_axd has joined #openstack-ironic07:51
iurygregorygood morning arne_wiebalck and Ironic o/07:55
arne_wiebalckhey iurygregory o/07:55
*** sziviani_ has joined #openstack-ironic07:59
*** threestrands has quit IRC08:01
*** mbeierl has quit IRC08:03
*** mbeierl has joined #openstack-ironic08:04
*** portdirect has quit IRC08:08
*** guilhermesp has quit IRC08:09
*** phrobb has quit IRC08:09
*** portdirect has joined #openstack-ironic08:10
*** mnaser has quit IRC08:10
*** phrobb has joined #openstack-ironic08:10
*** mnaser has joined #openstack-ironic08:11
*** rpittau|afk is now known as rpittau08:11
*** guilhermesp has joined #openstack-ironic08:11
rpittaugood morning ironic! o/08:11
*** tkajinam has quit IRC08:12
*** dsneddon_ has joined #openstack-ironic08:21
*** dsneddon_ has quit IRC08:26
*** dougsz has joined #openstack-ironic08:38
*** riuzen has joined #openstack-ironic08:44
iurygregorymorning rpittau o/08:46
rpittauhey iurygregory :)08:46
rpittaufor the tentacles of cthulhu, what's wrong with centos8 now....08:47
iurygregorywe can ask on rdo =)08:50
*** lucasagomes has joined #openstack-ironic08:58
riuzenhi, 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
riuzenDid I missing something?08:59
iurygregorymaybe you can provide more logs, only by no space left is a bit tricky to debug09:04
*** yoctozepto has quit IRC09:07
*** ociuhandu has joined #openstack-ironic09:10
*** diurnalist has joined #openstack-ironic09:12
*** uzumaki has joined #openstack-ironic09:14
*** yoctozepto0 has joined #openstack-ironic09:16
*** diurnalist has quit IRC09:18
*** ociuhandu has quit IRC09:22
*** yoctozepto9 has joined #openstack-ironic09:27
*** yoctozepto0 has quit IRC09:27
*** riuzen has quit IRC09:34
*** dtantsur|afk is now known as dtantsur09:39
dtantsurmorning ironic09:40
rpittauhey dtantsur :)09:40
uzumakihey dmitry!09:41
uzumakihow you doing?09:41
dtantsursurviving somehow :) you?09:43
*** derekh has joined #openstack-ironic09:43
iurygregorymorning dtantsur09:45
*** riuzen has joined #openstack-ironic09:53
*** ociuhandu has joined #openstack-ironic09:54
uzumakiWell, 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 system09:55
*** dsneddon_ has joined #openstack-ironic09:55
iurygregoryuzumaki++09:56
uzumakianyway, 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-ironic09:57
rpittauuzumaki: I tend do disagree, all considered09:57
ktibiHello 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
uzumakii think you put it in the same directory as a checksum file? as the image?09:58
ktibiuzumaki, now I don't have checksum file but IPA give me the old checkum, like ironic send it by API10:00
*** dsneddon_ has quit IRC10:00
ktibiI can see command like "ironic node-update $UUID add instance_info/image_checksum=$MD5SUM"10:01
ktibibut I don't know if it's the good way10:01
dtantsurimage_checksum is a checksum of the user image, not the deploy one10:01
dtantsurI don't think we checksum the deploy image10:02
ktibidtantsur, error from IPA during deploy step10:02
ktibiImage failed to verify against checksum. location: /tmp/deployment_image.qcow2; image ID: deployment_image.qcow2; image checksum: 3366a6bf41b91baeac4aa51fb5372a5d; verification checksum: 7c7b252fa5b42736d565f5fb4e19d66610:03
dtantsurktibi: I guess it's just a name, you need a checksum of instance_info/image_source10:04
dtantsuruzumaki: async_step_flags are set by steps and regulate how ironic-conductor processes them10:04
dtantsurunfortunately, the docstrings are not the most obvious..10:05
uzumakidtantsur, ah, I see. I can add comments in my code in case somebody will need to look it up then10:05
uzumakiThen I believe if I set appropriate flags in my steps, I'm good to go, no need to consume them manually10:05
dtantsuryep10:06
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP Raise human-friendly messages on attempt to use pre-deploy steps drivers  https://review.opendev.org/71359410:06
iurygregoryktibi, maybe you need to update the node instance-info image_checksum10:11
uzumakiperhaps by manually computing the checksum and putting it in the instance-info?10:12
*** amoralej is now known as amoralej|afk10:13
ktibiiurygregory, Yes I'll try that10:13
iurygregoryktibi, cool let us know the result =)10:15
ktibiiurygregory, 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 checksum10:18
*** mkrai_ has quit IRC10:19
uzumakicorrecting the checksum in wait call-back doesn't automatically allow recovery?10:20
iurygregoryI don't think so10:20
iurygregoryyou probably will need to update the node manually to move from deploy failed10:20
iurygregoryset the new info and try again?10:21
ktibiuzumaki, can't modify if node is not in active or manageable state10:21
iurygregory^ yup10:22
iurygregoryhttps://docs.openstack.org/ironic/train/contributor/states.html10:23
uzumakiah, I see. Shouldn't known. I remember now10:24
uzumakishould've*10:24
ktibistrange schema because when I use delete command, node are not in active or manageable after cleaning but just totaly deleted :/10:25
ktibinode list return me empty list10:25
dtantsurthe delete command deletes a node, did you mean undeploy?10:25
ktibiok deleting state is not the state after delete command ?10:26
ktibideleting state in the schema if after undeploy ?10:26
ktibiis after*10:26
dtantsurnope, the namings somewhat mismatch. I'd suggest you rely on ironicclient documentation for checking the right commands.10:27
*** mkrai_ has joined #openstack-ironic10:30
*** mkrai has joined #openstack-ironic10:32
*** mkrai_ has quit IRC10:35
*** tesseract has quit IRC10:36
*** yoctozepto9 is now known as yoctozepto10:37
*** k_mouza has joined #openstack-ironic10:38
*** tesseract has joined #openstack-ironic10:42
*** tesseract has quit IRC10:46
*** tesseract has joined #openstack-ironic10:48
openstackgerritNoor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver  https://review.opendev.org/71345010:48
*** riuzen has quit IRC10:51
iurygregorydtantsur, 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
dtantsuriurygregory: we don't check the character set anywhere10:54
iurygregorydo we want to also generate the salt using token_urlsafe?10:55
dtantsuriurygregory: mm, no, salt should be generated with mksalt or whatever is designed for it10:55
iurygregoryhttps://review.opendev.org/#/c/713690/1/ironic_python_agent/extensions/rescue.py10:55
patchbotpatch 713690 - ironic-python-agent - Use secrets to generate salt - 1 patch set10:55
dtantsurit probably does not matter, but let's use clearly targeted functions10:55
iurygregoryack10:56
ktibioh yeah !! RED Screen on my proliant after the deploy \o/10:56
dtantsuriurygregory: i.e. this one https://docs.python.org/3.5/library/crypt.html#crypt.mksalt10:56
iurygregorydtantsur, I'll change ipa to use that10:58
*** ociuhandu has quit IRC10:58
dtantsurthx!10:58
iurygregoryI will remove the function make_salt... since we can just call from crypt10:58
*** tesseract-RH has joined #openstack-ironic11:00
openstackgerritMark Goddard proposed openstack/bifrost stable/train: WIP: CentOS 8 support (Train)  https://review.opendev.org/71350811:00
*** yolanda has quit IRC11:01
*** tesseract-RH has left #openstack-ironic11:02
*** tesseract has quit IRC11:03
*** ociuhandu has joined #openstack-ironic11:03
uzumakiktibi, is that a good thing?11:04
*** ociuhandu has quit IRC11:04
*** ociuhandu has joined #openstack-ironic11:05
*** riuzen has joined #openstack-ironic11:06
ktibiuzumaki, no, it's like blue screen but for hardware :p11:07
iurygregorydtantsur, 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.crypt11:09
uzumakiktibi, aha! I knew it was sarcasm :D11:10
dtantsuriurygregory: seems that we'll need to pass crypt.METHOD_CRYPT as that's what we're currently using11:11
dtantsur(judging by 2-character salt)11:11
dtantsurotherwise, I agree11:11
iurygregorydtantsur, we can or we can let it choose the strongest it can find11:12
iurygregoryhttps://github.com/python/cpython/blob/0d3fe8ae4961bf551e7d5e42559e2ede1a08fd7c/Lib/crypt.py#L70-L8211:12
dtantsuriurygregory: I understand, but I don't know if it really checks what the underlying OS supports..11:13
*** yolanda has joined #openstack-ironic11:13
*** Lucas_Gray has joined #openstack-ironic11:13
ktibiduring 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|afk11:20
dtantsur|afknot feeling well today, will go afk for recovery. likely talk to you tomorrow.11:21
openstackgerritNoor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver  https://review.opendev.org/71345011:21
rpittaudtantsur|afk: take care11:21
uzumakidtantsur|afk, take care! gws11:21
iurygregorydtantsur|afk, take care o/11:22
*** riuzen has quit IRC11:26
*** rpittau is now known as rpittau|bbl11:30
ktibiuzumaki, yes ,) sanboot command from ironic make a redscreen ^^11:33
openstackgerritIlya Etingof proposed openstack/sushy master: Lazily load message registries  https://review.opendev.org/71351211:34
openstackgerritMerged openstack/bifrost stable/train: Ensure DIB is installed from source  https://review.opendev.org/71344311:50
*** ociuhandu has quit IRC11:50
uzumakiktibi, 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
ktibiuzumaki, sanboot is command in pxe for boot on localdisk :p11:54
*** rh-jelabarre has joined #openstack-ironic11:54
ktibiBut 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-ironic11:57
*** michchap has joined #openstack-ironic11:58
*** dsneddon_ has quit IRC12:01
*** k_mouza has quit IRC12:06
*** k_mouza has joined #openstack-ironic12:07
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Use secrets to generate salt  https://review.opendev.org/71369012:14
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Do not use random to generate token  https://review.opendev.org/71368712:15
openstackgerritMerged openstack/sushy master: Add `Connector(..., response_callback=None)`` parameter  https://review.opendev.org/71331512:18
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Use crypt to generate salt  https://review.opendev.org/71369012:19
*** ociuhandu has joined #openstack-ironic12:22
*** amoralej|afk is now known as amoralej12:28
*** ociuhandu has quit IRC12:28
*** hjensas has quit IRC12:33
*** hjensas has joined #openstack-ironic12:35
*** k_mouza has quit IRC12:42
*** jdandrea has joined #openstack-ironic12:45
openstackgerritMark Goddard proposed openstack/bifrost stable/train: WIP: CentOS 8 support (Train)  https://review.opendev.org/71350812:47
*** Wryhder has joined #openstack-ironic12:51
*** k_mouza has joined #openstack-ironic12:52
*** Lucas_Gray has quit IRC12:53
*** Wryhder is now known as Lucas_Gray12:53
uzumakiktibi, sanboot - boot from local disk? Oh gosh. What is the world becoming...12:54
*** jdandrea has quit IRC12:56
*** rloo has joined #openstack-ironic12:59
*** rpittau|bbl is now known as rpittau12:59
openstackgerritMark Goddard proposed openstack/tenks master: Refactor virtualbmc-domain tasks into a module  https://review.opendev.org/71304313:00
*** jdandrea has joined #openstack-ironic13:03
*** mkrai has quit IRC13:09
openstackgerritMerged openstack/ironic-python-agent-builder master: Workaround for missing adjtime file  https://review.opendev.org/71365113:14
openstackgerritMerged openstack/ironic-python-agent-builder master: Use variables for ssh paths  https://review.opendev.org/69765413:14
*** tzumainn has joined #openstack-ironic13:22
openstackgerritNoor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver  https://review.opendev.org/71345013:24
*** derekh has quit IRC13:26
TheJuliarpittau: You have invoked the dark lord. Now I'm worried ;)13:33
rpittauTheJulia: IT always keep an eye on me, on my desk :)13:34
TheJuliaiurygregory: Please add a release note, a story id, and a task id to https://review.opendev.org/#/c/713690/313:38
patchbotpatch 713690 - ironic-python-agent - Use crypt to generate salt - 3 patch sets13:38
iurygregoryTheJulia, ack o/13:39
rpittauprobably needed for https://review.opendev.org/713687 too13:39
patchbotpatch 713687 - ironic - Do not use random to generate token - 2 patch sets13:39
iurygregoryrpittau, sure13:40
TheJuliaone concern regarding switching to token_urlsafe is the shorter length13:44
iurygregoryTheJulia, we can say the bytes we want13:45
iurygregoryif you want 128 chars I just need to add 96 or 95 as parameter13:45
TheJuliaI'm also generally slightly paranoid13:45
iurygregorywant me to update to keep the 128 chars? =)13:46
uzumakianybody take a quick look at the BIOSInterface code, I've finished the main part, wanna know your thoughts if anybody has the time13:46
uzumakihttps://review.opendev.org/#/c/71345013:46
patchbotpatch 713450 - ironic - WIP: [RFE] Implement the bios-interface for idrac-... - 10 patch sets13:46
TheJuliaI made it 128 chars just to be near impossible to copy it easily13:46
TheJuliaso it would scroll of a screen, wrap in a terminal13:46
* TheJulia plays the because I'm evil song13:46
iurygregoryTheJulia, I'm ok with having 128 chars =) it won't change much13:47
uzumakimost of the patchsets are pep8 compliance stuff, 2/3 major ones13:47
iurygregorybut do I need to update the results in test to have 128 chars also? hehehe13:47
TheJuliahttps://www.youtube.com/watch?v=tWYCS6k1IOA13:47
TheJuliaiurygregory: likely :) maybe at least a length or something... just in case token_urlsafe ever chagnes13:48
* iurygregory opens the link in a new tab13:48
*** mkrai has joined #openstack-ironic13:48
* uzumaki opens the link as well, sounds wonderful (even though it's evil)13:49
TheJuliauzumaki: yay for pep8 :)13:49
TheJuliauzumaki:  I guess net is unit tests?13:50
TheJulias/net/next13:50
uzumakiTheJulia, 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
uzumakiTheJulia, indeed! Unit tests coming up13:50
iurygregorythis song is from a movie right? O.o13:50
uzumakiiurygregory, an evil movie, i'm sure13:51
iurygregoryuzumaki, hehehehe13:51
TheJuliaiurygregory: it is an entire albumn by voltaire13:51
uzumakiTheJulia, voltaire as in Voltaire the philosopher?13:51
TheJuliathe musician13:51
uzumakitoo bad, i would've liked to hear Voltaire sing :D13:52
rpittauuzumaki: I'm pretty sure he would not have a nice voice at the moment13:52
TheJuliahttps://www.youtube.com/watch?v=8DxUHeiFHmo&list=PL75F59375E0AA6E1A <-- the entire album13:52
rpittaumostly sounds like braaaainnssss13:53
*** amoralej is now known as amoralej|lunch13:53
TheJuliaActually, this might be good music for the day13:53
*** mkrai has quit IRC13:54
* TheJulia realizes that she has revealed how dark her soul is to IRC13:54
TheJuliaDOH!13:54
* uzumaki thinks Julia can never fully reveal how dark she _actually_ is13:55
uzumakirpittau, lol for the 'at the moment' part13:55
*** k_mouza has quit IRC13:55
*** Lucas_Gray has quit IRC13:56
TheJuliaWell, time to begin talking about a virtual ptg I guess https://www.openstack.org/events/covid-19-coronavirus-disease-updates13:57
*** dsneddon_ has joined #openstack-ironic13:57
*** k_mouza has joined #openstack-ironic13:57
uzumakiyeah, just saw that this morning :(13:57
rpittauconsidering that I was not coming to Vancouver, I'm happy(?) about that13:57
uzumakirpittau, what's wrong with vancouver? (besides it being in canada)13:58
TheJuliaheh13:58
* TheJulia would get popcorn if she could eat it13:58
rpittaunothing, I've beene there and I ovled it, just had other plans for June13:58
uzumakirpittau, ah, I see. I wanna be there as well, apparently, I won't be able to :/13:58
openstackgerritMark Goddard proposed openstack/bifrost stable/train: CentOS 8 support (Train)  https://review.opendev.org/71350813:58
* uzumaki thinks Julia could eat it all the same13:58
TheJuliauzumaki: corn allergy13:59
rpittauuzumaki: 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 go13:59
TheJuliaSo I leave popcorn for my wife, in accordance with the prophecy13:59
TheJuliamgoddard: <313:59
rpittaumgoddard: awesome!13:59
uzumakirpittau, Woahh, that's so cool!13:59
TheJuliamgoddard: Will review it after I am no longer wearing pajamas, which should be shortly13:59
rpittauyeah, they're super nice :)13:59
mgoddardTheJulia: great :)14:00
*** derekh has joined #openstack-ironic14:00
rpittaumgoddard: I will review even if I'm on my pajamas :P14:00
*** Lucas_Gray has joined #openstack-ironic14:00
uzumakiTheJulia, Oh! too bad. which prophecy is this?14:00
uzumakirpittau, lol!!14:00
mgoddardWFH == pajamas14:00
mgoddardat best14:00
rpittau\o/14:00
iurygregorymgoddard, ++14:01
uzumakimgoddard, lol for 'at best'14:01
*** dsneddon_ has quit IRC14:01
* rloo hears Canada and perks up. Phew, only Canada lovers out there.14:01
uzumakirloo, I really wanted to visit, given the 'coronic' circumstances, I can't unfortunately :(14:02
rloouzumaki: the vancouver area is lovely (if you love outdoors, nature). Hopefully there will be another time.14:02
uzumakirloo, totally looking forward to it!14:03
rloouzumaki: :)14:03
iurygregoryTheJulia, for the ipa change, should I use security or fixes in the release note? I have mixed feelings ..14:04
TheJuliaIf 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
TheJuliaiurygregory: yeah, might as well14:04
TheJuliamgoddard: pajama WFH gets old so quick.... so so quick.14:05
uzumakiTheJulia, retired? that's never happening, redundant conditional in the code!14:05
TheJuliauzumaki: I had hope ;)14:06
rpittaumgoddard: one thing, did you not add the tinyipa integration job on purpose to project.yaml ?14:07
rpittauah no nvm14:07
rpittauI'm blind14:07
mgoddard:014:08
mgoddard:)14:08
openstackgerritIlya Etingof proposed openstack/ironic master: [WIP] Add `network_data` field to ports and portgroups  https://review.opendev.org/68791014:10
* uzumaki doesn't like when the ironic channel is quiet14:17
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Use crypt to generate salt  https://review.opendev.org/71369014:18
*** jcoufal has joined #openstack-ironic14:18
*** amoralej|lunch is now known as amoralej14:22
rpittauuzumaki: we need silence and peace to properly plan world conquering14:22
*** diurnalist has joined #openstack-ironic14:23
iurygregoryTheJulia, I do agree with dtantsur|afk 's comment that we get a bonus by using the default of token_urlsafe =)14:23
iurygregorybut if you think we should increase I can update now together with the release note14:24
TheJuliaiurygregory: might as well14:27
* uzumaki accedes to the mighty cause rpittau has mentioned14:27
*** dking_desktop has quit IRC14:28
uzumakijust curious, do we have documentation on writing ironic unit tests? I am aware of a python project guide, and ironic testing guide14:28
TheJuliauzumaki: nothing terribly specific, there are existing patterns in the other driver modules that should prove informative14:29
uzumakiTheJulia, great. I'll take a look at those. thanks14:29
TheJuliaBasically base test cases should serve most needs, and you'll likely just want/need to mock some backend library methods14:30
uzumakithat sounds even better, let's see14:32
*** diurnalist has quit IRC14:32
TheJuliaetingof: 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-ironic14:41
etingofany quick link/pointer handy?14:41
TheJuliaetingof: I guess a story/task would help as an example of what is occuring so I can feel more confidence to the change14:41
TheJuliaoh!14:41
TheJuliaI forgot the bugid14:41
TheJuliaerr14:41
TheJuliachangeset14:41
TheJuliahttps://review.opendev.org/#/c/712655/14:41
patchbotpatch 712655 - sushy - Ignore failing message registry download - 1 patch set14:41
etingofno worries14:41
etingofah, sure! that's iLO serving a dict where a string is expected blowing up urllib call inside sushy14:43
etingoflet me do a Story and update the patch14:43
TheJuliaSWEET14:44
TheJuliacould you put that in a story with some example data?14:44
etingofyes, indeed14:46
etingofbtw, we also have it noted here -- https://bugzilla.redhat.com/show_bug.cgi?id=1804127#c1114:47
openstackbugzilla.redhat.com bug 1804127 in python-sushy "Importing results in "Error: Cannot mix str and non-str arguments"" [Medium,New] - Assigned to ietingof14:47
*** cdearborn has joined #openstack-ironic14:47
TheJuliasweet! I'd just note that in the commit message then14:50
etingofindeed! 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 way14:51
TheJuliaahh yeah, that discussion14:51
etingoftrouble is that sushy is sending that magical header14:51
TheJuliaOo14:54
TheJuliaYeah, I thought it was, but yeah14:54
*** jcoufal_ has joined #openstack-ironic15:01
*** jcoufal has quit IRC15:03
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Do not use random to generate token  https://review.opendev.org/71368715:04
*** ricolin has quit IRC15:08
*** ricolin has joined #openstack-ironic15:13
openstackgerritMerged openstack/ironic stable/train: Documentation clarifications for software RAID  https://review.opendev.org/71364515:19
*** jawad_axd has quit IRC15:31
*** jawad_axd has joined #openstack-ironic15:42
*** dking_desktop has joined #openstack-ironic15:57
*** dsneddon_ has joined #openstack-ironic15:57
*** Lucas_Gray has quit IRC15:59
*** dsneddon_ has quit IRC16:02
*** ociuhandu has joined #openstack-ironic16:04
*** ociuhandu has quit IRC16:06
*** ociuhandu_ has joined #openstack-ironic16:06
*** k_mouza has quit IRC16:07
*** Lucas_Gray has joined #openstack-ironic16:08
*** k_mouza has joined #openstack-ironic16:08
*** yedongcan has left #openstack-ironic16:15
rpiosoGood morning, ironic16:17
rpiosoetingof: o/16:17
rpiosoiurygregory: \o16:17
iurygregoryrpioso, o/16:17
iurygregorysorry for the wrong ping other day hehehe16:17
rpiosoetingof: When you have a moment, please review https://review.opendev.org/#/c/710416/16:18
patchbotpatch 710416 - x/sushy-oem-idrac - Drop support for Python 2.7 - 6 patch sets16:18
rpiosoetingof: I believe rpittau addressed all of your previous comments.16:18
rpiosoiurygregory: No worries at all.16:18
*** k_mouza has quit IRC16:18
rpioso:-)16:18
uzumakihey rpioso !16:19
uzumakimorning16:19
rpiosouzumaki: :-)16:19
rpiosouzumaki: And good evening to you!16:19
*** k_mouza has joined #openstack-ironic16:19
uzumakibe 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 it16:19
patchbotpatch 713450 - ironic - WIP: [RFE] Implement the bios-interface for idrac-... - 10 patch sets16:19
*** Lucas_Gray has quit IRC16:21
uzumakiany ideas why stash.dellemc-community.org/logs throws 404 Not Found when I try to look at the CI logs?16:21
rpiosouzumaki: Do you have the entire link?16:23
uzumakiwell, you can look at the change I sent, 3 Dell ironic CI jobs have failed, I'm unable to access their logs16:23
*** Lucas_Gray has joined #openstack-ironic16:24
iurygregoryTheJulia, 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-ironic16:26
etingofrpioso, sure o/16:27
TheJuliaiurygregory: twoish weeks I think and I thought they had when they brought CI back but I guess not?16:27
iurygregoryI guess not also =)16:28
iurygregorymaybe a topic to raise in the next weekly meeting?16:28
TheJulia+++++16:29
TheJuliaYes16:29
* iurygregory adding to the wiki16:30
TheJuliaiurygregory: thanks!16:31
rpiosoetingof: ty16:34
*** jdandrea has quit IRC16:35
rpiosoetingof: I see I missed a ping of yours yesterday. Is it still relevant?16:35
*** dsneddon_ has joined #openstack-ironic16:38
etingofrpioso, well, I can't grok how bios works so far16:41
etingofalso, are all bios attributes treated the same? can some be set right away, but some require reboot?16:42
*** dsneddon_ has quit IRC16:44
*** jcoufal_ has quit IRC16:46
*** TxGirlGeek has joined #openstack-ironic16:48
TheJuliaIs there consistency at all? or are only some consistent one way and others another way?16:49
TheJuliaWhat is the pattern of consistency that we need to adhere to, seems like the question to me.16:49
*** lucasagomes has quit IRC16:50
*** k_mouza has quit IRC16:51
*** k_mouza has joined #openstack-ironic16:52
uzumakiare we talking about consistency in the handling of BIOS parameters? or in their names? TheJulia16:52
openstackgerritMerged openstack/ironic-tempest-plugin master: Remove check for API version in validate_scheduling  https://review.opendev.org/71203017:04
iurygregoryTheJulia, enabled_drivers in the tempest.conf will always be empty?17:06
openstackgerritIlya Etingof proposed openstack/sushy master: Ignore failing message registry download  https://review.opendev.org/71265517:06
*** adrianc_ has quit IRC17:06
iurygregoryI'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-ironic17:06
openstackgerritMark Goddard proposed openstack/bifrost stable/train: CentOS 8 support (Train)  https://review.opendev.org/71350817:10
openstackgerritNoor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver  https://review.opendev.org/71345017:15
*** gyee has joined #openstack-ironic17:19
openstackgerritNoor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver  https://review.opendev.org/71345017:21
openstackgerritMerged x/sushy-oem-idrac master: Drop support for Python 2.7  https://review.opendev.org/71041617:21
etingofrpioso, ^17:28
rpiosoetingof: They all require a reboot to be applied.17:31
etingofsurprisingly, reboot does not change things17:31
* etingof tried to turn it off and on again already17:32
*** alexmcleod has quit IRC17:37
*** ociuhandu_ has quit IRC17:42
*** ociuhandu has joined #openstack-ironic17:42
*** k_mouza has quit IRC17:43
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: [WIP] Get driver from config  https://review.opendev.org/71204117:45
rpiosoerbarr: 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
patchbotpatch 713450 - ironic - WIP: [RFE] Implement the bios-interface for idrac-... - 12 patch sets17:47
*** ociuhandu has quit IRC17:48
rpiosoerbarr: Well, I was receiving that error. Now I'm not \o/17:48
erbarrrpioso, mmmnn the transfer of the logs happens every minute i think, then they get deleted after 30 days17:49
*** k_mouza has joined #openstack-ironic17:50
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Do not use random to generate token  https://review.opendev.org/71368717:58
*** ktibi has quit IRC17:58
*** ociuhandu has joined #openstack-ironic18:01
*** dougsz has quit IRC18:02
rpiosoetingof: How did you try to set them?18:02
*** derekh has quit IRC18:05
etingofrpioso, by PATCH'ing BIOS document I guess... I am calling sushy18:05
rpiosoThat's insufficient.18:06
rpittaugood night! o/18:06
etingof+ rebooting!18:06
*** rpittau is now known as rpittau|afk18:06
rpiosoetingof: A config job is needed to apply them at reboot.18:06
etingofso plain Redfish will not work?18:07
etingofit does not refuse my PATCH though, just ignores me18:07
rpiosoetingof: It doesn't ignore you!18:07
etingofwell, may be it watches me18:08
etingofso what's the right approach to manage bios attributes over redfish bios branch?18:09
etingofshould we push vendor-specific fishiness to sushy-oem-idrac again?18:09
etingofTheJulia, 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 default18:13
TheJuliaetingof: i need to get past meetings before I can order into the crystal ball today :(18:15
etingofno worries!18:16
* rpioso believes etingof can't be ignored by a BMC18:16
* etingof should authenticate as etingof rather than root, may be that explains my misery18:17
*** dsneddon_ has joined #openstack-ironic18:18
*** dsneddon has quit IRC18:20
*** dsneddon has joined #openstack-ironic18:21
*** dsneddon has quit IRC18:22
*** dsneddon has joined #openstack-ironic18:22
*** dsneddon has quit IRC18:22
*** dsneddon has joined #openstack-ironic18:22
*** Lucas_Gray has quit IRC18:25
*** ociuhandu has quit IRC18:27
*** ociuhandu has joined #openstack-ironic18:28
*** k_mouza has quit IRC18:29
*** k_mouza has joined #openstack-ironic18:32
rpiosoetingof: lol18:33
*** ociuhandu has quit IRC18:33
*** gyee has quit IRC18:44
TheJuliaetingof: 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
etingofTheJulia, I am sorry for causing more headache!18:49
TheJuliaNo worries18:49
etingofLet's return to this discussion when you will feel better18:49
etingofit's not blocking anything so far18:49
TheJuliaNo better time than noe18:50
TheJuliaNow18:50
TheJuliaBesides, maybe it will reinvigorate my brain!18:50
etingofokay, I mean which keys are most likely be used18:50
uzumakiI'm off now guys. See y'all tomorrow! o/ Hoping to finish my story tomorrow18:50
*** gyee has joined #openstack-ironic18:51
TheJuliaetingof: 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 versa18:54
TheJuliaSr-iov18:55
etingofyeah, okay, I will play with those18:55
*** uzumaki has quit IRC18:55
etingofeven if those keys were standardized, the vendors would make them vendor-specific anyway18:56
etingofso no worries18:56
*** bfournie has quit IRC19:01
*** jawad_axd has quit IRC19:04
TheJuliaYeah :(19:05
TheJuliaetingof: are you trying to navigate a passthrough for the actual settings object, not just the bios object?19:05
*** k_mouza has quit IRC19:07
etingofTheJulia, 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 iDRAC19:10
TheJuliaOkay, I was going to say because settinngd and attributes are very different in the idracs from what I have read19:12
etingofsushy takes it simple - just PATCHes by the URL in the Settings object including only the desired key-value pairs to the message body19:13
*** TxGirlGeek has quit IRC19:14
etingofif you have a link handy of what you've read about bios@idrac - that would be helpful19:15
*** dsneddon has quit IRC19:15
*** dsneddon has joined #openstack-ironic19:16
TheJuliaetingof: Just the dell published api spec19:25
etingofokay19:25
*** gyee has quit IRC20:03
*** Lucas_Gray has joined #openstack-ironic20:20
*** dsneddon_ has quit IRC20:57
*** ijw has quit IRC20:57
*** ijw has joined #openstack-ironic21:11
*** ijw has quit IRC21:15
*** ociuhandu has joined #openstack-ironic21:33
*** ijw has joined #openstack-ironic21:37
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Fix agent token vmedia use  https://review.opendev.org/71398921:38
openstackgerritJulia Kreger proposed openstack/ironic master: Signal agent token is required  https://review.opendev.org/69930421:38
TheJuliadtantsur|afk: I'd appreciate a quick look at https://review.opendev.org/713989 tomorrow. \o/21:39
patchbotpatch 713989 - ironic-python-agent - Fix agent token vmedia use - 1 patch set21:39
openstackgerritHarald Jensås proposed openstack/ironic master: Additional IP addresses to IPv6 stateful ports  https://review.opendev.org/70000221:40
*** dhellmann_ has joined #openstack-ironic21:53
*** dhellmann has quit IRC21:54
*** dhellmann_ is now known as dhellmann21:54
*** ktibi has joined #openstack-ironic21:54
openstackgerritHarald Jensås proposed openstack/ironic master: Additional IP addresses to IPv6 stateful ports  https://review.opendev.org/70000221:58
*** ktibi has quit IRC21:58
*** bfournie has joined #openstack-ironic22:00
*** gyee has joined #openstack-ironic22:00
*** k_mouza has joined #openstack-ironic22:09
*** ociuhandu has quit IRC22:09
*** ociuhandu has joined #openstack-ironic22:10
*** k_mouza has quit IRC22:14
*** ociuhandu has quit IRC22:15
openstackgerritVerification of a change to openstack/ironic failed: Clean up nits from adding additional node update policies  https://review.opendev.org/71346222:20
*** zigo has quit IRC22:22
*** sziviani_ has quit IRC22:23
*** Lucas_Gray has quit IRC22:31
*** zigo has joined #openstack-ironic22:31
*** diurnalist has quit IRC23:12
*** k_mouza has joined #openstack-ironic23:25
*** k_mouza has quit IRC23:29
openstackgerritMerged openstack/ironic master: Support centos 7 rootwrap data directory  https://review.opendev.org/71376523:47
openstackgerritVerification of a change to openstack/ironic failed: Don't require root partition when installing a whole disk image  https://review.opendev.org/70767623:47

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!