*** tosky has quit IRC | 00:02 | |
*** Goneri has joined #openstack-ironic | 00:02 | |
*** Goneri has quit IRC | 00:29 | |
*** benj_- has joined #openstack-ironic | 00:53 | |
*** benj_ has quit IRC | 00:54 | |
*** benj_- is now known as benj_ | 00:54 | |
*** anuradha1904 has quit IRC | 01:09 | |
*** ricolin has quit IRC | 01:35 | |
*** dking has joined #openstack-ironic | 01:55 | |
*** k_mouza has joined #openstack-ironic | 02:28 | |
*** k_mouza has quit IRC | 02:33 | |
*** Qianbiao has joined #openstack-ironic | 02:47 | |
*** priteau has quit IRC | 03:03 | |
openstackgerrit | Verification of a change to openstack/ironic-lib failed: Fix lower-constraints with the new pip resolver https://review.opendev.org/c/openstack/ironic-lib/+/766233 | 03:18 |
---|---|---|
*** openstackgerrit has quit IRC | 03:22 | |
*** gyee has quit IRC | 04:00 | |
*** zzzeek has quit IRC | 05:06 | |
*** zzzeek has joined #openstack-ironic | 05:08 | |
*** JamesBenson has quit IRC | 05:33 | |
*** JamesBenson has joined #openstack-ironic | 05:35 | |
*** JamesBenson has quit IRC | 05:41 | |
*** rpittau|afk has quit IRC | 06:11 | |
*** mnaser has quit IRC | 06:11 | |
*** mnaser has joined #openstack-ironic | 06:11 | |
*** rpittau|afk has joined #openstack-ironic | 06:11 | |
*** zzzeek has quit IRC | 06:17 | |
*** JamesBenson has joined #openstack-ironic | 06:17 | |
*** zzzeek has joined #openstack-ironic | 06:18 | |
*** tzumainn has quit IRC | 06:20 | |
*** JamesBenson has quit IRC | 06:22 | |
*** openstackgerrit has joined #openstack-ironic | 06:28 | |
openstackgerrit | Merged openstack/metalsmith master: Add metalsmith_instances instance option config_drive https://review.opendev.org/c/openstack/metalsmith/+/761850 | 06:28 |
*** ayoung has quit IRC | 07:05 | |
*** rcernin has quit IRC | 07:10 | |
*** ayoung has joined #openstack-ironic | 07:11 | |
*** rcernin has joined #openstack-ironic | 07:29 | |
arne_wiebalck | Good morning, ironic! | 07:41 |
openstackgerrit | sonali bhausaheb borkar proposed x/sushy-oem-idrac master: Add export system configuration https://review.opendev.org/c/x/sushy-oem-idrac/+/710327 | 07:46 |
*** rcernin has quit IRC | 08:07 | |
openstackgerrit | sonali bhausaheb borkar proposed x/sushy-oem-idrac master: Add export system configuration https://review.opendev.org/c/x/sushy-oem-idrac/+/710327 | 08:09 |
*** rpittau|afk is now known as rpittau | 08:17 | |
rpittau | good morning ironic! o/ | 08:17 |
*** JamesBenson has joined #openstack-ironic | 08:18 | |
*** JamesBenson has quit IRC | 08:23 | |
*** zzzeek has quit IRC | 08:36 | |
*** zzzeek has joined #openstack-ironic | 08:39 | |
janders | good morning arne_wiebalck and rpittau | 08:44 |
*** dougsz has joined #openstack-ironic | 08:45 | |
arne_wiebalck | hey janders o/ | 08:46 |
*** tosky has joined #openstack-ironic | 08:47 | |
*** rcernin has joined #openstack-ironic | 08:56 | |
*** derekh has joined #openstack-ironic | 08:58 | |
openstackgerrit | sonali bhausaheb borkar proposed x/sushy-oem-idrac master: Add export system configuration https://review.opendev.org/c/x/sushy-oem-idrac/+/710327 | 09:03 |
*** lucasagomes has joined #openstack-ironic | 09:06 | |
*** rcernin has quit IRC | 09:13 | |
*** k_mouza has joined #openstack-ironic | 09:17 | |
rpittau | hey janders :) | 09:22 |
*** zzzeek has quit IRC | 09:32 | |
*** zzzeek has joined #openstack-ironic | 09:33 | |
openstackgerrit | Pranjali Srivastava proposed openstack/ironic master: WIP: Create 'redfish' driver Redfish Interop Profile https://review.opendev.org/c/openstack/ironic/+/754061 | 09:46 |
*** zzzeek has quit IRC | 10:13 | |
*** zzzeek has joined #openstack-ironic | 10:14 | |
*** JamesBenson has joined #openstack-ironic | 10:19 | |
*** JamesBenson has quit IRC | 10:23 | |
*** zzzeek has quit IRC | 10:39 | |
*** ociuhandu has joined #openstack-ironic | 10:40 | |
*** zzzeek has joined #openstack-ironic | 10:41 | |
*** k_mouza has quit IRC | 10:46 | |
*** k_mouza has joined #openstack-ironic | 10:48 | |
*** ociuhandu has quit IRC | 10:54 | |
*** rcernin has joined #openstack-ironic | 10:57 | |
*** fmuyassarov has joined #openstack-ironic | 11:10 | |
janders | trying to address Steve's comment: https://storyboard.openstack.org/#!/story/2008038#comment-167637 | 11:27 |
janders | but when I try to pass the param in the new format ( lookup=['mac:aa:bb:cc:dd:ee:ff', 'mac:bb:cc:dd:ee:ff:00']), it looks good up to https://review.opendev.org/c/openstack/ironic/+/750943/16/ironic/drivers/modules/inspector.py#276 but in https://review.opendev.org/c/openstack/ironic-inspector/+/758994/9/ironic_inspector/main.py#357 I only get the | 11:28 |
janders | first value | 11:28 |
janders | http://paste.openstack.org/show/800929/ | 11:29 |
janders | I'd appreciate any pointers on what extra bits may be needed (or what am I doing wrong) | 11:29 |
*** anuradha1904 has joined #openstack-ironic | 11:32 | |
*** dtantsur|afk is now known as dtantsur | 11:33 | |
dtantsur | good.. whatever it is, ironic | 11:33 |
janders | hey dtantsur | 11:35 |
*** JamesBenson has joined #openstack-ironic | 12:00 | |
*** ociuhandu has joined #openstack-ironic | 12:02 | |
*** ociuhandu has quit IRC | 12:03 | |
*** ociuhandu has joined #openstack-ironic | 12:03 | |
*** JamesBenson has quit IRC | 12:06 | |
*** JamesBenson has joined #openstack-ironic | 12:06 | |
*** rcernin has quit IRC | 12:13 | |
fmuyassarov | Hi dtantsur , Would you mind to review https://github.com/metal3-io/metal3-docs/pull/151 whenever possible. Thanks! | 12:14 |
*** zzzeek has quit IRC | 12:14 | |
dtantsur | ack | 12:15 |
*** zzzeek has joined #openstack-ironic | 12:17 | |
*** ociuhandu has quit IRC | 12:23 | |
*** ociuhandu has joined #openstack-ironic | 12:31 | |
*** zzzeek has quit IRC | 12:40 | |
*** zzzeek has joined #openstack-ironic | 12:41 | |
openstackgerrit | Merged openstack/ironic-lib master: Fix lower-constraints with the new pip resolver https://review.opendev.org/c/openstack/ironic-lib/+/766233 | 12:49 |
openstackgerrit | Merged openstack/sushy master: Raise exception when async operation fails https://review.opendev.org/c/openstack/sushy/+/764229 | 12:51 |
*** ociuhandu has quit IRC | 12:55 | |
*** ricolin has joined #openstack-ironic | 12:56 | |
*** ociuhandu has joined #openstack-ironic | 13:05 | |
*** sshnaidm has quit IRC | 13:09 | |
*** sshnaidm has joined #openstack-ironic | 13:09 | |
*** priteau has joined #openstack-ironic | 13:10 | |
*** ociuhandu has quit IRC | 13:15 | |
TheJulia | good morning | 13:28 |
dtantsur | morning TheJulia | 13:32 |
*** k_mouza_ has joined #openstack-ironic | 13:33 | |
*** k_mouza has quit IRC | 13:33 | |
*** hjensas has quit IRC | 13:35 | |
rpittau | good morning TheJulia :) | 13:36 |
*** paras333_ has quit IRC | 13:36 | |
*** paras333_ has joined #openstack-ironic | 13:37 | |
TheJulia | iurygregory: When you have a minute, could you take a look at https://review.opendev.org/c/openstack/puppet-ironic/+/766100 ? All of the failures *seem* to be unrealted :\ | 13:37 |
dtantsur | Execution of '/bin/dnf -d 0 -e 1 -y install python3-neutron-tests-tempest' returned 1 | 13:40 |
TheJulia | I didn't catch that one :( | 13:41 |
dtantsur | TheJulia: the patch looks solid to me, I suggest rechecking | 13:42 |
iurygregory | TheJulia sure =) | 13:42 |
iurygregory | I already have coffee | 13:42 |
rpittau | I added the issue with multinode job in the whiteboard, just wanted to know if there was any development on that | 13:42 |
TheJulia | rpittau: br-ex not found? | 13:43 |
rpittau | nope, the missing ca-bundle | 13:44 |
TheJulia | nope | 13:44 |
iurygregory | well it's broken \o/ | 13:44 |
rpittau | yay | 13:44 |
iurygregory | [puppet] Broken CI because of missing powertools repo | 13:44 |
dtantsur | \o/ | 13:44 |
TheJulia | sweet | 13:44 |
TheJulia | well, at least the ironic-lib fix should be the only thing *needed* | 13:44 |
iurygregory | litmus jobs hehe | 13:45 |
TheJulia | re rsync, I wonder if devstack's getting an override environment variable for something it doesn't expect | 13:45 |
TheJulia | since it is trying to stat something zuul related... | 13:46 |
iurygregory | I gave +2 on takashi patch | 13:46 |
iurygregory | it will probably merge today | 13:46 |
* TheJulia goes and gets coffee | 13:46 | |
*** rloo has joined #openstack-ironic | 13:48 | |
*** zigo has joined #openstack-ironic | 13:52 | |
*** ociuhandu has joined #openstack-ironic | 13:52 | |
TheJulia | Ugh, I need to start wearing glasses during the day | 13:54 |
*** hjensas has joined #openstack-ironic | 13:56 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-inspector master: Fix lower-constraints with the new pip resolver https://review.opendev.org/c/openstack/ironic-inspector/+/766229 | 14:04 |
TheJulia | rpittau: doc8 addition so tox -epep8 passes | 14:04 |
rpittau | TheJulia: ok, I missed it in another place, can we actually get that to 0.8.1 ? for python3.6 support | 14:05 |
*** ociuhandu has quit IRC | 14:06 | |
TheJulia | feel free to change, I just would like to get CI operational again | 14:06 |
TheJulia | sounds like doc8 revs need to be changed across the board then which seems better as a separate series | 14:07 |
rpittau | ok, that's fine, we'll get to that, let's make the CI pass first | 14:08 |
*** ociuhandu has joined #openstack-ironic | 14:09 | |
*** ociuhandu has quit IRC | 14:13 | |
*** ociuhandu has joined #openstack-ironic | 14:15 | |
*** ociuhandu has quit IRC | 14:19 | |
*** ociuhandu has joined #openstack-ironic | 14:28 | |
*** k_mouza_ has quit IRC | 14:38 | |
*** k_mouza has joined #openstack-ironic | 14:39 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Option to enable bootloader config failure bypass https://review.opendev.org/c/openstack/ironic-python-agent/+/764016 | 14:43 |
*** tzumainn has joined #openstack-ironic | 14:43 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Switch images to CentOS Stream 8 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/766486 | 14:49 |
*** k_mouza has quit IRC | 14:52 | |
openstackgerrit | sonali bhausaheb borkar proposed openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port https://review.opendev.org/c/openstack/ironic/+/693328 | 14:54 |
*** k_mouza has joined #openstack-ironic | 14:56 | |
*** ociuhandu has quit IRC | 14:57 | |
arne_wiebalck | When a node is deleted from Ironic, will this also delete its inspection data ? | 14:57 |
dtantsur | arne_wiebalck: eventually | 14:58 |
arne_wiebalck | dtantsur: there is some GC? | 14:58 |
dtantsur | a periodic task in inspector | 14:58 |
arne_wiebalck | nice, thanks! | 14:58 |
dtantsur | oh, the TLS support poses an interesting challenge. If the clocks are not synchronized, it goes kaboom.. | 14:58 |
dtantsur | TheJulia: ^^ | 14:58 |
TheJulia | command line on boot? | 14:59 |
TheJulia | I believe if populated we try that before we even do lookup | 15:00 |
dtantsur | yep, but it's not populated (we're only looking at this for openshift) | 15:00 |
TheJulia | That would explain amy's problem | 15:00 |
dtantsur | what would you say about allowing some graceful period? 1 hour or anything, not sure | 15:00 |
TheJulia | 1 hour grace is likely okay | 15:01 |
* dtantsur opens vim | 15:01 | |
TheJulia | once managed, it should never again be a problem, in theory | 15:01 |
TheJulia | older systems did skew horribly though | 15:01 |
* TheJulia doesn't have a fleet of baremetal without ntp access anymore | 15:01 | |
TheJulia | spotz: your tls issues, could have been time related ^^^ | 15:04 |
*** ociuhandu has joined #openstack-ironic | 15:05 | |
*** fmuyassarov has quit IRC | 15:05 | |
*** ociuhandu has quit IRC | 15:10 | |
*** ociuhandu has joined #openstack-ironic | 15:11 | |
*** Qianbiao has quit IRC | 15:14 | |
*** Qianbiao has joined #openstack-ironic | 15:14 | |
*** ociuhandu has quit IRC | 15:15 | |
*** ociuhandu has joined #openstack-ironic | 15:20 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Generate TLS certificates with validity time in the past https://review.opendev.org/c/openstack/ironic-python-agent/+/766498 | 15:22 |
*** k_mouza has quit IRC | 15:24 | |
*** Goneri has joined #openstack-ironic | 15:38 | |
*** k_mouza has joined #openstack-ironic | 15:49 | |
*** Qianbiao has quit IRC | 16:14 | |
*** dougsz has quit IRC | 16:17 | |
arne_wiebalck | dtantsur: about the inspector clean up, sorry: I was referring to the data sent to the store_data backend, e.g. swift, not the node cache. Is your statement about the periodic task still true? | 16:23 |
dtantsur | mmm, a good question | 16:24 |
dtantsur | arne_wiebalck: I think the data will stay in swift (but not in database). I guess it's something we missed. | 16:25 |
* TheJulia is trying to resist major improvements to the fallback logic on ipa commands | 16:25 | |
arne_wiebalck | dtantsur: ok, thanks | 16:30 |
* arne_wiebalck checked the code and could not find any clean up | 16:30 | |
TheJulia | iurygregory: can you hear me? | 16:31 |
iurygregory | TheJulia, I can | 16:31 |
iurygregory | can you hear me? | 16:31 |
*** dougsz has joined #openstack-ironic | 16:31 | |
TheJulia | no, I cannot | 16:31 |
*** ociuhandu has quit IRC | 16:31 | |
iurygregory | =( | 16:31 |
iurygregory | going to join again | 16:31 |
*** k_mouza has quit IRC | 16:33 | |
*** k_mouza has joined #openstack-ironic | 16:35 | |
*** ociuhandu has joined #openstack-ironic | 16:36 | |
*** ociuhandu has quit IRC | 16:48 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add TLS troubleshooting guide entry https://review.opendev.org/c/openstack/ironic/+/766505 | 16:52 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity https://review.opendev.org/c/openstack/ironic/+/376574 | 16:52 |
dtantsur | TheJulia: ^^^ | 16:52 |
dtantsur | typed some text, could you check the part about hardware clock for correctness? | 16:52 |
*** ociuhandu has joined #openstack-ironic | 16:53 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity https://review.opendev.org/c/openstack/ironic/+/376574 | 16:53 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add TLS troubleshooting guide entry https://review.opendev.org/c/openstack/ironic/+/766505 | 16:58 |
*** ociuhandu_ has joined #openstack-ironic | 16:59 | |
*** ociuhandu has quit IRC | 17:03 | |
TheJulia | dtantsur: thanks | 17:05 |
*** dougsz has quit IRC | 17:31 | |
arne_wiebalck | dtantsur: I created https://storyboard.openstack.org/#!/story/2008441 so we do not forget, shouldn't be too hard to add | 17:32 |
arne_wiebalck | bye everyone o/ | 17:33 |
TheJulia | tzumainn: on the current call, can we chat for another minute or three afterwards? | 17:35 |
tzumainn | TheJulia, definitely! | 17:35 |
tzumainn | sdanni, ^ | 17:35 |
TheJulia | dtantsur: ^ | 17:35 |
sdanni | sure | 17:36 |
*** lmcgann has joined #openstack-ironic | 17:36 | |
spotz | thejulia - also found out that the repos and containers are sort of a moving target:( | 17:37 |
*** ociuhandu_ has quit IRC | 17:42 | |
openstackgerrit | Merged openstack/sushy-tools master: Make memoize behave better in concurrent environments https://review.opendev.org/c/openstack/sushy-tools/+/765333 | 17:44 |
TheJulia | do'oh | 17:44 |
dtantsur | would anyone like to propose a sushy-tools release? | 17:45 |
TheJulia | like as in "please do it?" | 17:46 |
TheJulia | ? | 17:46 |
iurygregory | I can push in a few | 17:46 |
TheJulia | ok | 17:46 |
iurygregory | need to wash the dishes XD | 17:46 |
TheJulia | :) | 17:46 |
iurygregory | or my mother will kill me XD | 17:46 |
* TheJulia can then go back and looking at why the unit test no work | 17:46 | |
dtantsur | is gerrit opening for anyone? | 17:52 |
* TheJulia gets out a magic8 ball | 17:53 | |
TheJulia | Timed out | 17:53 |
*** ociuhandu has joined #openstack-ironic | 17:54 | |
TheJulia | looks like they are hunting | 17:54 |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP: Register all hardware_interfaces together https://review.opendev.org/c/openstack/ironic/+/764911 | 17:55 |
*** ociuhandu has quit IRC | 17:58 | |
*** derekh has quit IRC | 18:01 | |
openstackgerrit | Merged openstack/ironic-inspector master: Fix lower-constraints with the new pip resolver https://review.opendev.org/c/openstack/ironic-inspector/+/766229 | 18:01 |
rpittau | good night! o/ | 18:04 |
*** rpittau is now known as rpittau|afk | 18:04 | |
TheJulia | ever have a day where you feel like mock is just like doing some sort of drug and you can't figure out how in the world it is perceiving what it is which doesn't grok a bunch a print statements?!? | 18:05 |
dtantsur | once a months or so | 18:05 |
dtantsur | usually mocking a wrong thing is the cause. double-mocking is rarer but more vicious | 18:06 |
TheJulia | its just freaky because I'm printing the parameter that I'm calling it with right before and it is what I expect | 18:07 |
* TheJulia takes a break... maybe makes some pizza sauce | 18:07 | |
*** lucasagomes has quit IRC | 18:11 | |
dking | Before I dig into this issue deeper, would anybody know what to check if Ironic Python Agent Builder is failing and complains about using an older version of pip (9.0.3), even though the current venv is using the latest (20.3.1)? | 18:30 |
dking | Could it be caused by me trying to run this command on CentOS 7? | 18:31 |
TheJulia | dking: do you have https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/764370 ? | 18:34 |
TheJulia | 20.3 has a headache of a change in it | 18:34 |
dking | I was just using https://tarballs.opendev.org/openstack/ironic-python-agent-builder/ironic-python-agent-builder-2.3.0.tar.gz | 18:35 |
TheJulia | pull master branch down, we've not cut a release with that fix yet | 18:36 |
TheJulia | it is that much of a headahce that I think we still have patches outstanding | 18:36 |
dking | Yeah, that looks crazy. I notice that looking back in the error output a bit further, it's complaining because the "packaging" package doesn't contain a setup.py. It looks like it was trying to run "python setup.py egg_info" | 18:37 |
dking | Ah, okay. I'll do that. I was assuming that was the latest stable. I was using that because the one in pip didn't have the extra-hardware element included. It looked old. | 18:38 |
TheJulia | hmm | 18:39 |
*** dtantsur is now known as dtantsur|afk | 18:40 | |
dtantsur|afk | dking: re packaging: https://review.opendev.org/c/openstack/requirements/+/765785 | 18:41 |
dtantsur|afk | they reverted the change but haven't issued a new release, I think | 18:41 |
* dtantsur|afk le sigh and see you tomorrow | 18:41 | |
*** k_mouza has quit IRC | 18:48 | |
* TheJulia finds out why the unit test hates her in the mock documentation | 19:17 | |
* iurygregory is back | 19:26 | |
iurygregory | going to do the release | 19:26 |
openstackgerrit | Bill Dodd proposed openstack/sushy master: Add OperationApplyTime support to Volume methods https://review.opendev.org/c/openstack/sushy/+/758874 | 19:38 |
iurygregory | TheJulia, fyi sushy-tools release https://review.opendev.org/c/openstack/releases/+/766537 | 19:41 |
TheJulia | much appreciated | 19:43 |
openstackgerrit | Merged x/sushy-oem-idrac master: Fix lower-constraints with the new pip resolver https://review.opendev.org/c/x/sushy-oem-idrac/+/766284 | 19:50 |
openstackgerrit | Verification of a change to x/sushy-oem-idrac failed: Update .gitreview for stable/victoria https://review.opendev.org/c/x/sushy-oem-idrac/+/764111 | 19:51 |
openstackgerrit | Verification of a change to x/sushy-oem-idrac failed: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/c/x/sushy-oem-idrac/+/764112 | 19:51 |
rpioso | Good morning, ironic! | 19:56 |
rpioso | TheJulia: Are fixes for the new pip resolver needed on stable branches? | 19:57 |
TheJulia | I unfortuantely do not know, but I've not tried to backport anything yet | 19:58 |
rpioso | TheJulia: The 2 verification failures in the immediately above scrollback were caused by openstack-tox-lower-constraints. | 20:00 |
rpioso | TheJulia: The log complains about hacking, pycodestyle, and flake8, so it seems like it applies there, too. | 20:04 |
rpioso | iurygregory: Familiar ^^^? | 20:05 |
*** paras333_ has quit IRC | 20:16 | |
iurygregory | rpioso, move to tox.ini | 20:20 |
iurygregory | =) | 20:20 |
iurygregory | some patches we did move pep8 deps to tox.ini | 20:20 |
stevebaker | morning | 20:20 |
*** rcernin has joined #openstack-ironic | 20:21 | |
rpioso | iurygregory: Thank you! | 20:21 |
iurygregory | rpioso, yw | 20:25 |
stevebaker | janders: you might need to post your current WIP to see what it is doing | 20:26 |
openstackgerrit | Mark Goddard proposed openstack/tenks master: WIP: Support libvirt pool host path https://review.opendev.org/c/openstack/tenks/+/766279 | 20:29 |
*** paras333_ has joined #openstack-ironic | 20:34 | |
openstackgerrit | Merged openstack/bifrost stable/victoria: Fix two CI issues https://review.opendev.org/c/openstack/bifrost/+/766216 | 21:14 |
*** paras333_ has quit IRC | 21:20 | |
openstackgerrit | Merged openstack/bifrost master: Update lower-constraints from an up-to-date environment https://review.opendev.org/c/openstack/bifrost/+/765987 | 21:20 |
*** rcernin has quit IRC | 21:31 | |
*** rcernin has joined #openstack-ironic | 21:34 | |
*** ociuhandu has joined #openstack-ironic | 21:40 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: Enable node level control of ignoring boot loader failure https://review.opendev.org/c/openstack/ironic/+/766620 | 21:43 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: Enable node level control of ignoring boot loader failure https://review.opendev.org/c/openstack/ironic/+/766620 | 21:44 |
*** rcernin has quit IRC | 21:51 | |
TheJulia | bfournie: fyi, https://review.opendev.org/c/openstack/ironic-python-agent/+/765294's reno likely needs adjustment to frame it so we don't have to argue about stable policy | 21:52 |
*** rcernin has joined #openstack-ironic | 21:53 | |
bfournie | TheJulia: sure I can add a note to reno. Should I add an additional note under "features" or as a separate item? | 21:58 |
TheJulia | Start the feature note explaining why it is useful to operators, or refame it as a fix for environments that are using vlan enabled networks for with no default tagging | 22:00 |
zaneb | say you deprovision a node and it ends up in the "clean failed" state, so you move it to manageable... it's not safe to say that anything has happened at all, right? the provisioned image may still be running? | 22:04 |
TheJulia | zaneb: why did the node end up in clean failed? are the disks broken? shouldn't someone investigate that? | 22:05 |
TheJulia | the provisioned image could be, if it never powered down to begin with | 22:06 |
TheJulia | or if it is rebooted out of the ramdisk it could be | 22:06 |
zaneb | speaking purely hypothetically | 22:06 |
zaneb | reference: https://github.com/metal3-io/baremetal-operator/blob/master/pkg/provisioner/ironic/ironic.go#L1344-L1352 | 22:06 |
TheJulia | cleaning typically fails due to a communications failure OR the hardware has a fault | 22:06 |
TheJulia | communications wise, that is often something is physically wrong, the machine didn't boot to the network or something like that | 22:07 |
TheJulia | or miscabled, or whatever | 22:07 |
zaneb | thanks, that answers my question. we are Doing It Wrong | 22:07 |
* TheJulia looks at the go code and sighs | 22:07 | |
zaneb | so if e.g. we can't contact the BMC, that code ^ could move us to maintenance mode without us actually having powered down the host | 22:08 |
zaneb | which is ok so far as it goes (we'll try to clean again next time we provision), but not consistent with the guarantees the metal3 API provides about the deprovisioning state | 22:10 |
TheJulia | zaneb: I _think_ youd end up in delete failed | 22:10 |
TheJulia | which the only option to is delete again | 22:10 |
zaneb | oh, that's a thing? | 22:11 |
TheJulia | but if the bmc loccked up or went away between delete and the boot of the ramdisk | 22:11 |
TheJulia | yeah, it is | 22:11 |
zaneb | oh, it's called error? | 22:11 |
TheJulia | yeah, error | 22:11 |
TheJulia | sorry | 22:11 |
zaneb | np, had a small moment of panic when I saw that wasn't handled at all :D | 22:11 |
TheJulia | sorry! | 22:12 |
zaneb | we have handled error so badly (by trying to go back to manage) that we might as well not have handled it | 22:12 |
TheJulia | :( | 22:13 |
TheJulia | state machine diagram is your friend | 22:13 |
TheJulia | and perhaps a nice whiskey | 22:13 |
zaneb | yeah, I dunno why I didn't see that on there | 22:14 |
zaneb | ok, so we think that if we fail to reboot into IPA we would end up in error, and clean failed will only happen if it actually fails in cleaning | 22:15 |
zaneb | this is good news | 22:15 |
*** lmcgann has quit IRC | 22:16 | |
TheJulia | or never checks in from the ramdisk while in clean wait | 22:20 |
zaneb | yeah. we can always fix that later by cleaning again. but we really need to know if there is a zombie deployment keeping on trucking out there :) | 22:21 |
openstackgerrit | Bob Fournier proposed openstack/ironic-python-agent stable/victoria: Bring up VLAN interfaces and include in introspection report https://review.opendev.org/c/openstack/ironic-python-agent/+/765294 | 22:45 |
bfournie | TheJulia: I've updated the reno, let me know if that provides more clarity. This was needed for metal3 where we need to get the vlan IPs in order to do CSRs after introspection | 22:47 |
*** paras333 has joined #openstack-ironic | 22:59 | |
*** ociuhandu has quit IRC | 22:59 | |
*** paras333 has quit IRC | 23:03 | |
*** anuradha1904 has quit IRC | 23:20 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Fix lower-constraints with the new pip resolver https://review.opendev.org/c/openstack/ironic-python-agent/+/766098 | 23:28 |
*** tzumainn has quit IRC | 23:40 | |
*** Goneri has quit IRC | 23:40 | |
*** tzumainn has joined #openstack-ironic | 23:44 | |
janders | stevebaker thanks, will upload the WIP patch. I'm taking 4hr RHAT exam today and then I'm off till 4th of Jan so I might do it then. When are you back from xmas holiday? | 23:45 |
*** tosky has quit IRC | 23:53 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!