*** k_mouza has quit IRC | 00:02 | |
*** JamesBenson has quit IRC | 00:08 | |
TheJulia | So to the question if the larger proposed feature is acceptable. If the behavior contract of the raid configuration field is changed accordingly as discussed above then it does not have bearing on the overall larger piece of work's acceptability. Fair warning though, it is going to face greater scrutiny in the review process because the larger feature will be a larger piece of work and much more code outside | 00:08 |
---|---|---|
TheJulia | of the driver's specific code and driver_internal_info field will need to be touched/manipulated. As mentioned on the call I highly suggest to submit patches in incremental steps to reach your desired end state and feature complete status. This having been completely stated, I still believe it would be good to have another call so that this is revisited by contributors since we did not discuss changing the | 00:08 |
TheJulia | raid_config field behavior contract and on the call and ultimately the contract offered by these interfaces needs to be explicitly stated in these terms such that it non-destructively updates it's relevant areas based on what it has available, not deleting any field value. This will likely require additional Restful API functionality in terms of CRUD operations on what I believe are presently read-only | 00:08 |
TheJulia | fields/data. This also being said, IF this discussion is framed with generic version being available to at least _save_/_restore_ existing configuration data in the same base mechanism, then this becomes even easier for the community grok and see usefulness of a larger mechanism beyond the vendor specific steps/methods that would be part of a vendor specific interface. | 00:08 |
* rpioso puts on masks before entering grocery store zone. | 00:08 | |
*** JamesBenson has joined #openstack-ironic | 00:09 | |
TheJulia | rpioso: +++++++ | 00:11 |
TheJulia | rpioso: I'm going to go assemble a shelf now, have a wonderful and safe evening! | 00:11 |
rpioso | TheJulia: Sound good. Let’s have that discussion. First though, I need to arrange an update on Redfish Interop Profiles :) | 00:12 |
TheJulia | ++ | 00:13 |
TheJulia | awesome | 00:13 |
TheJulia | remind me tomorrow and I'll put a thing on the meeting for monday | 00:13 |
rpioso | TheJulia: Thank you! And you as well. | 00:13 |
rpioso | TheJulia: We may want to wait until ajya and I post a spec update. That’s on the back burner at the moment, as we approach feature freeze. | 00:15 |
rpioso | TheJulia: We could discuss RAID contract before revisiting the meta step proposal, though. | 00:17 |
*** k_mouza has joined #openstack-ironic | 00:29 | |
*** hoonetorg has quit IRC | 00:29 | |
*** k_mouza has quit IRC | 00:33 | |
*** hoonetorg has joined #openstack-ironic | 00:41 | |
erbarr | there hasn't been any changes that would cause PXE to not boot, correct? I haven't seen any so far | 00:42 |
*** lifeless has joined #openstack-ironic | 01:02 | |
janders | I'm looking at testing redfish OOB introspection on bifrost/VMs. Does anyone here have any experience with this? I'm particularly after any reconfiguration of the bifrost-deployed vbmc that might be required | 01:07 |
*** zzzeek has quit IRC | 01:11 | |
*** zzzeek has joined #openstack-ironic | 01:14 | |
*** JamesBenson has quit IRC | 01:35 | |
*** stevebaker has joined #openstack-ironic | 01:54 | |
openstackgerrit | Christopher Dearborn proposed openstack/sushy master: WIP: Allow monitoring progress of a firmware update https://review.opendev.org/745950 | 02:04 |
openstackgerrit | Christopher Dearborn proposed openstack/ironic master: WIP: Redfish driver firmware update https://review.opendev.org/749619 | 02:12 |
janders | dtantsur|afk rpittau iurygregory regarding https://storyboard.openstack.org/#!/story/2008038 do I need a real-baremetal in order to be able to use OOB introspection? I tried setting up an environment using sushy-emulator and bifrost VMs but I hit this: http://paste.openstack.org/show/797405/ - looks like the response to | 02:27 |
janders | http://localhost:9132/redfish/v1/Systems/<UUID>/Processors is not implemented (I don't see a template for it in sushy_tools/emulator/templates). I will try to progress other parts of the patch now but it would be great to have your thoughts on this when your timezone comes online. Thanks! :) | 02:27 |
janders | I see https://storyboard.openstack.org/#!/story/2007454 is still in TODO which would explain it | 02:28 |
*** ijw has quit IRC | 02:47 | |
*** ijw has joined #openstack-ironic | 02:48 | |
*** tzumainn has quit IRC | 02:49 | |
*** JamesBenson has joined #openstack-ironic | 03:05 | |
*** JamesBen_ has joined #openstack-ironic | 03:09 | |
*** JamesBenson has quit IRC | 03:10 | |
*** hjensas has joined #openstack-ironic | 04:37 | |
*** JamesBen_ has quit IRC | 04:39 | |
*** m1p has quit IRC | 04:39 | |
*** TheJulia has quit IRC | 04:39 | |
*** m1p has joined #openstack-ironic | 04:40 | |
*** TheJulia has joined #openstack-ironic | 04:40 | |
*** hjensas has quit IRC | 04:58 | |
*** JamesBenson has joined #openstack-ironic | 05:00 | |
*** janders has quit IRC | 05:11 | |
openstackgerrit | Merged openstack/ironic master: Support file:/// images for the direct deploy https://review.opendev.org/748379 | 05:26 |
*** hjensas has joined #openstack-ironic | 05:27 | |
*** hjensas has quit IRC | 05:28 | |
*** JamesBenson has quit IRC | 05:32 | |
*** JamesBenson has joined #openstack-ironic | 05:33 | |
*** hjensas has joined #openstack-ironic | 05:36 | |
*** JamesBenson has quit IRC | 05:38 | |
*** jtomasek has joined #openstack-ironic | 05:57 | |
*** janders has joined #openstack-ironic | 06:01 | |
openstackgerrit | likui proposed openstack/ironic-python-agent master: Remove Babel lower-requirement https://review.opendev.org/749642 | 06:09 |
*** Qianbiao has joined #openstack-ironic | 06:30 | |
*** belmoreira has joined #openstack-ironic | 06:36 | |
*** JamesBenson has joined #openstack-ironic | 06:38 | |
*** JamesBenson has quit IRC | 06:43 | |
arne_wiebalck | Good morning, ironic! | 06:44 |
*** sshnaidm|afk is now known as sshnaidm | 06:50 | |
openstackgerrit | Ankit Kumar proposed openstack/ironic master: Enhance certificate verification for ilo harware type https://review.opendev.org/743490 | 06:52 |
*** Nisha_Agarwal has joined #openstack-ironic | 06:55 | |
*** JamesBenson has joined #openstack-ironic | 07:06 | |
*** JamesBenson has quit IRC | 07:24 | |
*** tosky has joined #openstack-ironic | 07:40 | |
*** Nisha_Agarwal has quit IRC | 07:42 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Increase memory of tinyipa vms https://review.opendev.org/747633 | 07:46 |
*** dougsz has joined #openstack-ironic | 07:52 | |
rpittau | good morning ironic! o/ | 07:52 |
iurygregory | good morning rpittau arne_wiebalck janders and Ironic o/ | 07:56 |
rpittau | hey iurygregory :) | 07:57 |
*** gyee has quit IRC | 07:57 | |
*** dougsz has quit IRC | 07:57 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Use tinycore 11 to build tinyipa https://review.opendev.org/711647 | 07:58 |
*** janders has quit IRC | 08:00 | |
*** janders has joined #openstack-ironic | 08:07 | |
*** lucasagomes has joined #openstack-ironic | 08:09 | |
arne_wiebalck | hey iurygregory o/ | 08:19 |
*** derekh has joined #openstack-ironic | 08:47 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity https://review.opendev.org/376574 | 08:47 |
*** priteau has joined #openstack-ironic | 08:49 | |
*** mkowalski has quit IRC | 09:00 | |
*** dtantsur|afk is now known as dtantsur | 09:06 | |
dtantsur | good morning ironic | 09:06 |
iurygregory | good morning dtantsur =) | 09:06 |
dtantsur | janders: maybe? I thought etingof implemented OOB introspection, but given the state of sushy-tools.. | 09:06 |
dtantsur | you only need MAC addresses though and THAT part should have been implemented | 09:07 |
iurygregory | lucasagomes, approved =) sorry forgot to look yesterday the result hehe | 09:08 |
lucasagomes | iurygregory++ no rush! | 09:08 |
lucasagomes | thanks for reviewing it | 09:08 |
iurygregory | yw =) | 09:08 |
*** dougsz has joined #openstack-ironic | 09:08 | |
*** dougsz has quit IRC | 09:13 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/stein: [Train and older] Fix several CI problems https://review.opendev.org/749014 | 09:18 |
janders | g'day Ironic! o/ | 09:18 |
janders | dtantsur I think OOB introspection works, it's the sushy emulator that's at fault AFAIK | 09:19 |
dtantsur | why am I not surprised?.. | 09:19 |
janders | when I work on redfish stuff, should I aim to work with a redfish-enabled real-baremetal? | 09:19 |
janders | or is sushy-emulator reasonably complete and worth a try? | 09:20 |
rpittau | janders: if you have a real baremetla server with redfish is the ideal scenario :) | 09:20 |
rpittau | sushy emulator can work, with its limitations | 09:20 |
rpittau | like as you saw the processors is not implemented at the moment | 09:21 |
rpittau | in theory with the emulator you should have a close-to-real implementation of the redfish standard, on real baremetal, depending on the vendor, certain things can be "off" | 09:23 |
*** dougsz has joined #openstack-ironic | 09:27 | |
*** ricolin has quit IRC | 09:29 | |
*** janders9 has joined #openstack-ironic | 09:31 | |
*** janders has quit IRC | 09:33 | |
*** janders9 is now known as janders | 09:33 | |
*** janders has quit IRC | 09:36 | |
*** janders has joined #openstack-ironic | 09:39 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Update the cache if we don't have a root device hint https://review.opendev.org/747072 | 09:41 |
openstackgerrit | Merged openstack/ironic-python-agent bugfix/6.2: Make WSGI server respect listen_* directives https://review.opendev.org/749599 | 09:42 |
openstackgerrit | Merged openstack/ironic-python-agent bugfix/6.3: Make WSGI server respect listen_* directives https://review.opendev.org/749598 | 09:42 |
*** Nisha_Agarwal has joined #openstack-ironic | 09:50 | |
Nisha_Agarwal | dtantsur, Good morning... | 09:50 |
dtantsur | hi Nisha_Agarwal | 09:50 |
Nisha_Agarwal | dtantsur, after a long time working back in ironic :) | 09:51 |
dtantsur | Nisha_Agarwal: congrats :) | 09:51 |
Nisha_Agarwal | dtantsur, a very small patch https://review.opendev.org/#/c/746416/ if u have time .... | 09:51 |
patchbot | patch 746416 - ironic - Adds few of the security dashboard parameters to c... - 10 patch sets | 09:51 |
Nisha_Agarwal | dtantsur, thanks :) | 09:52 |
dtantsur | will try to get there, a bit busy day today though | 09:52 |
Nisha_Agarwal | dtantsur, looking at more contribution now :) | 09:52 |
Nisha_Agarwal | dtantsur, sure | 09:52 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: documentation: follow-up to file:// support in direct deploy https://review.opendev.org/749695 | 09:56 |
*** rcernin has quit IRC | 10:01 | |
*** rcernin has joined #openstack-ironic | 10:06 | |
*** hjensas is now known as hjensas|afk | 10:11 | |
*** rcernin has quit IRC | 10:11 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Adds few of the security dashboard parameters to capabilities https://review.opendev.org/746416 | 10:18 |
*** rcernin has joined #openstack-ironic | 10:19 | |
dtantsur | WTF https://zuul.opendev.org/t/openstack/build/9d2905cea75e4297b82dcec95e1620da/log/logs/ironic-inspector.log | 10:21 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Adds few of the security dashboard parameters to capabilities https://review.opendev.org/746416 | 10:21 |
iurygregory | inspector broken? | 10:21 |
dtantsur | iurygregory: it's on Stein, fortunately | 10:21 |
dtantsur | but yeah, something is very wrong | 10:21 |
dtantsur | ah, of course it's eventlet, what else? https://bugs.launchpad.net/glance/+bug/1711123 | 10:22 |
openstack | Launchpad bug 1711123 in Glance "Failed to start openstack image service glance (openstack-glance-api.service)." [Undecided,Invalid] | 10:22 |
iurygregory | eventlet -.-' | 10:23 |
*** rcernin has quit IRC | 10:24 | |
dtantsur | mnasiadka: I'm seriously not sure if I can revive bifrost stein ^^ | 10:27 |
* dtantsur has posted a call for help on the ML | 10:31 | |
*** rcernin has joined #openstack-ironic | 10:31 | |
*** priteau has quit IRC | 10:33 | |
dtantsur | iurygregory: I guess the privsep stuff on ironic-lib won't be ready for Victoria? | 10:34 |
*** rcernin has quit IRC | 10:36 | |
*** Lucas_Gray has joined #openstack-ironic | 10:38 | |
mnasiadka | dtantsur: well, we just marked it as unbuildable :) | 10:40 |
dtantsur | k | 10:43 |
dtantsur | ironic folks, please pay attention to any outstanding ironic-lib, metalsmith and sushy patches. the release deadline is, well, today. | 10:44 |
*** priteau has joined #openstack-ironic | 10:49 | |
openstackgerrit | Merged openstack/metalsmith master: Explicitly set jobs to ML2/OVS https://review.opendev.org/749346 | 10:52 |
openstackgerrit | Merged openstack/metalsmith master: metalsmith_instances module cleanups https://review.opendev.org/748544 | 10:52 |
iurygregory | dtantsur, yep, we decided last week in the weekly meeting | 10:53 |
dtantsur | got it | 10:54 |
dtantsur | then there are outstanding sushy and metalsmith patches that could be reviewed | 10:54 |
*** priteau has quit IRC | 10:54 | |
iurygregory | I will look at them after the meeting o/ | 10:54 |
dtantsur | I think ironic-lib is fine, so anyone is welcome to propose a release | 10:54 |
iurygregory | ack | 10:55 |
dtantsur | I also think it may be the time to release IPA-builder | 10:55 |
iurygregory | I can check everything in about 45min hehe | 10:56 |
* iurygregory grabs something to eat before the meeting | 10:56 | |
openstackgerrit | Merged openstack/metalsmith master: Document metalsmith_instances https://review.opendev.org/738131 | 10:59 |
*** JamesBenson has joined #openstack-ironic | 11:00 | |
*** priteau has joined #openstack-ironic | 11:03 | |
*** k_mouza has joined #openstack-ironic | 11:08 | |
*** priteau has quit IRC | 11:11 | |
openstackgerrit | Merged openstack/ironic master: Remove absolute path with iptables when L3 enabled https://review.opendev.org/748930 | 11:12 |
*** k_mouza has quit IRC | 11:14 | |
*** hjensas|afk is now known as hjensas | 11:15 | |
*** vmud213 has joined #openstack-ironic | 11:22 | |
openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Adds ilo-uefi-https boot interface to ilo5 https://review.opendev.org/745501 | 11:22 |
*** vmud213 has quit IRC | 11:34 | |
openstackgerrit | Merged openstack/sushy stable/ussuri: Use Sessions URL from root service if it's provided https://review.opendev.org/748972 | 11:46 |
mnaser | o/ has anyone played around with smartnics with ironic (i assume since a spec was implemented, someone was testing it on $vendor cards?) | 11:48 |
openstackgerrit | Merged openstack/sushy stable/train: Use Sessions URL from root service if it's provided https://review.opendev.org/748973 | 11:49 |
iurygregory | dtantsur, for metalsmith the only patches I saw https://review.opendev.org/748315 https://review.opendev.org/737888 | 11:51 |
patchbot | patch 748315 - metalsmith - Replace the Centos7 image job with a Centos8 UEFI ... - 6 patch sets | 11:51 |
patchbot | patch 737888 - metalsmith - metalsmith_deployment role switch to metalsmith_in... - 17 patch sets | 11:51 |
dtantsur | iurygregory: ack. maybe we should released metalsmith (without branching) now and see if we can fit these in quickly enough | 11:52 |
dtantsur | stevebaker: FYI ^^^ | 11:52 |
openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Follow up on "Fix error 601" https://review.opendev.org/749499 | 11:54 |
iurygregory | can be, and we can do another release + branch after we merge 737888... | 11:54 |
iurygregory | sushy https://review.opendev.org/#/q/project:openstack/sushy+status:open+branch:master WOW | 11:55 |
rpittau | oh yes, tinyipa built on focal with tinycore 11 in ironic-standalone-ipa-src | 11:56 |
* rpittau more coffee AND ice-cream needed | 11:59 | |
*** priteau has joined #openstack-ironic | 12:01 | |
*** JamesBenson has quit IRC | 12:04 | |
*** uzumaki has joined #openstack-ironic | 12:08 | |
dtantsur | actually, I may have a small fix for metalsmith, let me see | 12:09 |
*** ricolin has joined #openstack-ironic | 12:10 | |
iurygregory | dtantsur, ack I will go with ironic-lib | 12:14 |
iurygregory | and I can push metalsmith later today | 12:14 |
*** JamesBenson has joined #openstack-ironic | 12:19 | |
*** JamesBenson has quit IRC | 12:23 | |
openstackgerrit | Dmitry Tantsur proposed openstack/metalsmith master: Stop requiring checksums with file images https://review.opendev.org/749719 | 12:24 |
dtantsur | iurygregory, rpittau, please ^^ | 12:26 |
*** Goneri has joined #openstack-ironic | 12:31 | |
*** uzumaki has quit IRC | 12:32 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Prepare to move ipa-builder jobs https://review.opendev.org/746766 | 12:32 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Use converted ipa jobs https://review.opendev.org/749723 | 12:35 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Use converted ipa jobs https://review.opendev.org/749723 | 12:36 |
Qianbiao | hello Ironic~ | 12:42 |
Qianbiao | hi what share picture service is used usually. | 12:43 |
*** vmud213 has joined #openstack-ironic | 12:44 | |
*** thiagop has joined #openstack-ironic | 12:44 | |
Qianbiao | a baremetal server failed to boot from pxe ramdisk and display some Garbage characters | 12:45 |
Qianbiao | i want to share the screenshoot, and may be some of you know the issue | 12:46 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Remove unused jobs https://review.opendev.org/749725 | 12:48 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Remove unused jobs https://review.opendev.org/749725 | 12:49 |
rpittau | Qianbiao: this should work https://pasteboard.co/ | 12:50 |
Qianbiao | rpittau thanks. | 12:50 |
Qianbiao | ok this is it : https://pasteboard.co/Jpn13aR.png | 12:52 |
Qianbiao | I can it dhcp finished, loading kernal file finished. and then show those garbage chars. | 12:52 |
Qianbiao | and after a while(10s maybe), server auto restart. | 12:52 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Use tinycore 11 to build tinyipa https://review.opendev.org/711647 | 12:53 |
dtantsur | Qianbiao: never seen anything like that, but it does start booting the linux kernel, so whatever happens must have something to do with it. | 12:56 |
Qianbiao | <dtantsur> may it be hardware compatible issue? | 12:57 |
Qianbiao | I am using the default pxe options from ironic devstack. | 12:58 |
openstackgerrit | Merged openstack/ironic master: [trivial] remove emacs config from devstack script https://review.opendev.org/748978 | 12:58 |
openstackgerrit | Merged openstack/ironic master: Update nodes in notifications documentation https://review.opendev.org/748423 | 12:58 |
*** rcernin has joined #openstack-ironic | 12:59 | |
dtantsur | Qianbiao: you mean want to tweak these options to make sure you actually see the kernel console output | 13:00 |
dtantsur | it's probably a kernel fault | 13:01 |
openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: DNM reproduce missing PowerState https://review.opendev.org/749727 | 13:01 |
dtantsur | bfournie: trying to reproduce the dell issue ^^^ | 13:01 |
dtantsur | otherwise I cannot understand what is going on | 13:01 |
*** rloo has joined #openstack-ironic | 13:01 | |
Qianbiao | dtantsur yes, if i can see what is happening, maybe i can figure out how to fix it. | 13:03 |
bfournie | dtantsur: good, btw wonder if we should handle the exception if try to set OFF and get back "already powered OFF" | 13:04 |
*** rcernin has quit IRC | 13:04 | |
dtantsur | bfournie: we should, I'm just not sure how.. especially since we probably cannot rely on the text to be in English. | 13:05 |
bfournie | dtantsur: hmm, yeah | 13:05 |
dtantsur | maybe if we get HTTP 409 on power off, try the reboot command? | 13:06 |
bfournie | dtantsur: yeah that would revert to previous behaviour | 13:06 |
dtantsur | the thing is, if we get no power state from the machine, I'd expect ironic to complain that it cannot get power state | 13:07 |
dtantsur | I don't understand how we end up assuming it's powered on... | 13:07 |
dtantsur | I think we should catch rpioso later today | 13:08 |
bfournie | dtantsur: sure, does it depend on sync_power_state_interval? | 13:08 |
dtantsur | bfournie: it's a possibility. maybe some action from our side removes PowerState from the System, but the power sync loop does not notice it quickly enough? | 13:08 |
dtantsur | does not explain why we assume None==On | 13:09 |
* dtantsur is extremely confused | 13:09 | |
bfournie | dtantsur: yeah | 13:09 |
iurygregory | None==On WHAT? | 13:09 |
iurygregory | https://media.giphy.com/media/SqmkZ5IdwzTP2/giphy.gif | 13:09 |
dtantsur | iurygregory: https://bugzilla.redhat.com/show_bug.cgi?id=1873305 | 13:10 |
openstack | bugzilla.redhat.com bug 1873305 in Bare Metal Hardware Provisioning "Failed to power on /inspect node when using Redfish protocol" [High,New] - Assigned to tsedovic | 13:10 |
* iurygregory looks | 13:10 | |
* dtantsur sees that shadower is assigned and assumes he'll solve it! :-P | 13:10 | |
iurygregory | dtantsur, LOL | 13:10 |
shadower | Hi! I'm your friendly load balancer for bugs assigned to the Ironic subcomponent of OCP | 13:12 |
* dtantsur likes friendly load balancers | 13:16 | |
openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Allow HttpImageService to accept custom certificate https://review.opendev.org/742936 | 13:17 |
TheJulia | heh | 13:19 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Update deploy steps documentation https://review.opendev.org/749729 | 13:21 |
dtantsur | morning TheJulia | 13:21 |
iurygregory | good morning TheJulia =) | 13:21 |
rpittau | good morning TheJulia :) | 13:23 |
Nisha_Agarwal | morning TheJulia | 13:24 |
*** JamesBenson has joined #openstack-ironic | 13:25 | |
bfournie | dtantsur: I see what you mean now, the response for /redfish/v1/Systems/System.Embedded.1 doesn't contain 'PowerState' when the power is Off, we only see it for the node which was manually powered On in the description (mgmt-e16-h12-b02-fc640) - 'PowerState': 'On' | 13:30 |
openstackgerrit | Christopher Dearborn proposed openstack/sushy master: Make message parsing more resilient https://review.opendev.org/748927 | 13:32 |
dtantsur | bfournie: yeah, it's super weird | 13:34 |
*** tzumainn has joined #openstack-ironic | 13:35 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity https://review.opendev.org/376574 | 13:36 |
dtantsur | looking for reviews on https://review.opendev.org/#/c/749285/ it's probably another thing we'll want to backport | 13:39 |
patchbot | patch 749285 - bifrost - Create our own firewalld zone and use it on real b... - 2 patch sets | 13:39 |
iurygregory | dtantsur, https://review.opendev.org/#/c/749734/ ironic-lib | 13:39 |
patchbot | patch 749734 - releases - Release ironic-lib and create stable/victoria - 1 patch set | 13:39 |
dtantsur | thx! | 13:40 |
openstackgerrit | Merged openstack/ironic master: Support caching http:// images locally in the direct deploy https://review.opendev.org/749496 | 13:43 |
openstackgerrit | Merged openstack/ironic master: Allow setting image_download_source per node https://review.opendev.org/749504 | 13:50 |
openstackgerrit | Merged openstack/ironic master: documentation: follow-up to file:// support in direct deploy https://review.opendev.org/749695 | 13:50 |
openstackgerrit | Merged openstack/ironic master: Add L3 boot section to the docs https://review.opendev.org/689844 | 13:54 |
iurygregory | lol release patch complains about stable/victoria o.o | 14:06 |
dtantsur | iurygregory: mm, no, it complains about invalid hash | 14:07 |
iurygregory | o.o | 14:07 |
dtantsur | how did you end up with it? for me HEAD is different | 14:07 |
iurygregory | tox -e venv -- list-unreleased-changes victoria openstack/ironic-lib | 14:08 |
dtantsur | iurygregory: try using scripts for proposing releases, not editing yaml manually | 14:09 |
iurygregory | I did | 14:09 |
dtantsur | ah, hmmm. then I don't know, probably worth mentioning on #openstack-release | 14:09 |
iurygregory | tox -e venv -- new-release victoria ironic-lib feature --stable-branch | 14:10 |
dtantsur | wow | 14:10 |
iurygregory | lol running again locally the same command I got the other commit | 14:13 |
iurygregory | 07d98520f09f65dc13c37150c91733527a105acf WTF | 14:13 |
rpittau | iurygregory: yeah, I just tested as well and I got the correct hash | 14:13 |
iurygregory | magic | 14:13 |
iurygregory | O.o | 14:14 |
iurygregory | updated | 14:16 |
*** cdearborn has joined #openstack-ironic | 14:16 | |
cdearborn | Good morning Ironic! | 14:17 |
iurygregory | morning cdearborn | 14:17 |
rpioso | Good morning, ironic | 14:17 |
iurygregory | morning rpioso | 14:17 |
dtantsur | hi cdearborn, rpioso, may I use some of your time? | 14:18 |
rpioso | bfournie, dtantsur: Could you try iDRAC firmware 4.10.10.10, assuming that version is available for FC640? | 14:18 |
dtantsur | aha, lemme pass it on | 14:18 |
rpittau | hey cdearborn rpioso :) | 14:19 |
cdearborn | dtantsur, sure | 14:19 |
dtantsur | cdearborn: rpioso pretty much answered my question, but it was about https://bugzilla.redhat.com/show_bug.cgi?id=1873305 | 14:19 |
openstack | bugzilla.redhat.com bug 1873305 in Bare Metal Hardware Provisioning "Failed to power on /inspect node when using Redfish protocol" [High,New] - Assigned to dtantsur | 14:19 |
dtantsur | seeing something truly weird there | 14:20 |
rpioso | dtantsur: Yeah, I examined the attached log. The System resource does not consistently contain the PowerState property. | 14:21 |
cdearborn | dtantsur: aok - let me know if you need me | 14:22 |
rpioso | rpittau: o/ | 14:22 |
rpioso | dtantsur: Thank you for passing my ask along. | 14:23 |
dtantsur | thank you for looking into it | 14:23 |
rpioso | dtantsur: yw | 14:27 |
*** Nisha_Agarwal has quit IRC | 14:45 | |
Qianbiao | hi dtantsur, i enabled console redirect log and debug log. | 14:45 |
Qianbiao | this is the log: http://paste.openstack.org/show/797425/ | 14:45 |
Qianbiao | do u have a minute look at it. | 14:46 |
*** Lucas_Gray has quit IRC | 14:46 | |
JayF | If someone gets a chance, I'd love to get a +2A on https://review.opendev.org/#/c/747193/ (Thanks for the +2 dmitry) | 14:46 |
patchbot | patch 747193 - ironic-python-agent - If listen_tls is true, enable TLS on wsgi server - 10 patch sets | 14:46 |
dtantsur | Qianbiao: I'm not a kernel specialist, so I only have a random guess: if you're using tinyIPA as your ramdisk, you may want to try a DIB-built image | 14:48 |
Qianbiao | <dtantsur> thanks, will try. | 14:48 |
Qianbiao | i guess it maybe hardware compatibility issue. | 14:49 |
Qianbiao | But not sure where. | 14:49 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity https://review.opendev.org/376574 | 14:49 |
dtantsur | I'm not sure either, unfortunately | 14:49 |
Qianbiao | :) You are still my hero | 14:49 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity https://review.opendev.org/376574 | 14:49 |
dtantsur | Qianbiao: if nothing helps, you may try building IPA images with different base OS and seeing if it makes any difference | 14:49 |
dtantsur | :) | 14:50 |
Qianbiao | ok | 14:50 |
openstackgerrit | Merged openstack/bifrost master: Create our own firewalld zone and use it on real bare metal https://review.opendev.org/749285 | 14:50 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Trivial: fix minor typo on RFC number https://review.opendev.org/749754 | 14:50 |
*** Lucas_Gray has joined #openstack-ironic | 14:51 | |
*** belmoreira has quit IRC | 15:00 | |
*** rloo has quit IRC | 15:01 | |
*** rloo has joined #openstack-ironic | 15:01 | |
*** rloo has quit IRC | 15:02 | |
*** rloo has joined #openstack-ironic | 15:02 | |
*** Qianbiao has quit IRC | 15:04 | |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Rework jobs so failed devstack does not retry the job https://review.opendev.org/749758 | 15:04 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost bugfix/8.3: Create our own firewalld zone and use it on real bare metal https://review.opendev.org/749759 | 15:05 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/ussuri: Create our own firewalld zone and use it on real bare metal https://review.opendev.org/749760 | 15:08 |
TheJulia | hmmmm | 15:12 |
dtantsur | mmmm? | 15:13 |
TheJulia | trying to figure out why cleaning fails in uefi mode on metalsmith | 15:13 |
*** vmud213 has quit IRC | 15:13 | |
TheJulia | Got the VM to boot with the agent though \o/... It is just took a really long time because the job would be retried three times | 15:14 |
TheJulia | also, https://review.opendev.org/#/c/749407/2 apparently I put W* in the config option, which I don't think was intentional | 15:14 |
patchbot | patch 749407 - ironic - Remove token-less agent support - 2 patch sets | 15:14 |
dtantsur | fun | 15:14 |
*** iurygregory has quit IRC | 15:24 | |
dtantsur | going a bit earlier today, see you tomorrow | 15:31 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Accept and use a TLS certificate from the agent https://review.opendev.org/747136 | 15:31 |
*** dtantsur is now known as dtantsur|afk | 15:31 | |
dtantsur|afk | o/ | 15:31 |
rpioso | cdearborn: Wondering if https://review.opendev.org/#/c/748927 should pass Dell Ironic CI before being merged? If so, please Workflow-1 it. iurygregory appears to be out. | 15:34 |
patchbot | patch 748927 - sushy - Make message parsing more resilient - 3 patch sets | 15:34 |
*** vmud213 has joined #openstack-ironic | 15:35 | |
vmud213 | Hello Ironic! | 15:36 |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job https://review.opendev.org/748315 | 15:38 |
erbarr | rpioso, looks like when that run it was hitting the setuptools issue, right now master is failing, I looked at nodes running and it looks like they're failing to pxe boot into ipxe, I'm looking to see what's going on | 15:38 |
TheJulia | rpittau: I blocked workflow on https://review.opendev.org/#/c/748927 for the time being | 15:40 |
patchbot | patch 748927 - sushy - Make message parsing more resilient - 3 patch sets | 15:40 |
vmud213 | TheJulia: Mind taking a look into https://review.opendev.org/#/c/745501/ when time permits? | 15:40 |
patchbot | patch 745501 - ironic - Adds ilo-uefi-https boot interface to ilo5 - 10 patch sets | 15:40 |
TheJulia | vmud213: sure, I'll try to look at it soon() | 15:41 |
vmud213 | Also Also this one Thanks. | 15:41 |
vmud213 | https://review.opendev.org/#/c/742936/ | 15:42 |
patchbot | patch 742936 - ironic - Allow HttpImageService to accept custom certificate - 8 patch sets | 15:42 |
vmud213 | Thanks | 15:42 |
*** iurygregory has joined #openstack-ironic | 15:42 | |
rpioso | erbarr: Thank you! | 15:47 |
rpioso | TheJulia: Thanks | 15:47 |
erbarr | anybody got any idea what could be causing it? started yesterday at around 8am | 15:48 |
TheJulia | ugh zuul job names | 15:48 |
TheJulia | erbarr: causing what? | 15:48 |
openstackgerrit | Merged openstack/metalsmith master: Stop requiring checksums with file images https://review.opendev.org/749719 | 15:48 |
erbarr | dell jobs stopped pxe booting on master | 15:49 |
TheJulia | OVN? | 15:50 |
TheJulia | lucasagomes: did the OVN default change merge to devstack? | 15:50 |
lucasagomes | TheJulia, not yet no | 15:50 |
lucasagomes | TheJulia, there are a few more things to fix, what we first did was to move the devstack code to deploy OVN to devstack itself | 15:51 |
TheJulia | erbarr: yesterday 8 am what timzone? | 15:51 |
erbarr | central | 15:51 |
lucasagomes | now I am sending patches around with a depends-on which changes the default to OVN | 15:51 |
TheJulia | UTC-5 then | 15:51 |
lucasagomes | making sure everything is ok before we go ahead and actually changing it | 15:52 |
lucasagomes | fixing whatever I find in between as well | 15:52 |
openstackgerrit | Merged openstack/bifrost master: TLS support for API services https://review.opendev.org/747921 | 15:53 |
iurygregory | erbarr, do you have some logs? (sorry I was disconnected) | 15:55 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Change integration test for Metalsmith to use Centos8/UEFI https://review.opendev.org/749772 | 15:56 |
TheJulia | erbarr: can you confirm what the PATH variable devstack is being executed with? | 15:57 |
erbarr | iurygregory, I'm trying to dig through the bulk here https://stash.dellemc-community.org/logs/dellemc-poweredge-R730-R630-tempest-dsvm-ironic-idrac/2020-09-02:21:02/743868/5//logs/ | 15:57 |
TheJulia | https://review.opendev.org/#/c/748930/ may be the cause | 15:58 |
patchbot | patch 748930 - ironic - Remove absolute path with iptables when L3 enabled (MERGED) - 1 patch set | 15:58 |
TheJulia | it is around the time window | 15:58 |
TheJulia | hmm.. no ip netns in the logs | 16:01 |
*** Lucas_Gray has quit IRC | 16:02 | |
erbarr | TheJulia, it started yesterday morning, i looked at devstack and didn't see any merges around that time but maybe neutron | 16:02 |
iurygregory | WUT?! | 16:04 |
iurygregory | DevStack Version: victoria | 16:04 |
iurygregory | Change: ae21b3556ff215c0f73bc0ebbc90d72fa59a3459 move cleanup of pyc files 2020-09-02 04:48:31 +0000 | 16:04 |
iurygregory | OS Version: Ubuntu 16.04 xenial | 16:04 |
iurygregory | Ubuntu Xenial in Victoria ?! | 16:04 |
iurygregory | this doesn't seem right.. | 16:05 |
*** gyee has joined #openstack-ironic | 16:05 | |
iurygregory | erbarr, any reason why it's not Bionic? | 16:06 |
erbarr | iurygregory, yea i know, I'm trying to update to zuul v3 and having trouble installing it | 16:07 |
iurygregory | ouch =( | 16:07 |
iurygregory | maybe people on #zuul would be able to help | 16:07 |
erbarr | yea, any pointers as to why nodepool would launch and then instantly delete a node? | 16:07 |
erbarr | yea, i've talked to them a little, maybe i should more | 16:08 |
iurygregory | no idea =( | 16:08 |
iurygregory | any chance the jobs can run on bionic without moving to zuulv3? (just wondering) | 16:09 |
tosky | (the move to zuul v3 is much needed at this point regardless) | 16:11 |
iurygregory | yeah | 16:11 |
erbarr | tosky, agreed and I would love to do so, i'm trying | 16:11 |
iurygregory | https://developer.ibm.com/tutorials/migrating-openstack-from-zuul-2-to-3/ maybe would help | 16:12 |
iurygregory | not 100% sure | 16:12 |
erbarr | for just bionic, i understand that was tried as a simple nodepool change to dib bionic but that had some problems, i haven't tried it myself though | 16:13 |
*** lucasagomes has quit IRC | 16:14 | |
TheJulia | looking at the neutron log, dnsmasq is reporting no address available | 16:20 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-lib stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/749779 | 16:22 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-lib stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/749780 | 16:22 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-lib master: Add Python3 wallaby unit tests https://review.opendev.org/749781 | 16:22 |
erbarr | mmmnnnnn, ok, I'll look to see where those should come from and what would be different between ussuri and master | 16:22 |
erbarr | iurygregory: thanks, i'll take a look, I understand the overall picture and the flow, kind of just wish there was an easy install button lol, i liked the tutorial for zuul v3 with the containers, that was easy to follow | 16:24 |
*** priteau has quit IRC | 16:25 | |
*** adrianc has quit IRC | 16:25 | |
iurygregory | yeah install button would be good hehe | 16:26 |
*** adrianc has joined #openstack-ironic | 16:27 | |
tosky | the Software Factory project provides a prepackaged "zuul distribution" | 16:28 |
*** dougsz has quit IRC | 16:30 | |
erbarr | tosky: cool, i'll take a look thanks! | 16:31 |
*** rloo has quit IRC | 16:31 | |
TheJulia | erbarr: is fa:16:3e:aa:52:43 a valid mac in your environment? | 16:32 |
*** rloo has joined #openstack-ironic | 16:33 | |
*** rloo has quit IRC | 16:33 | |
*** rloo has joined #openstack-ironic | 16:34 | |
erbarr | TheJulia, checking, with environment you mean baremetals? | 16:34 |
TheJulia | erbarr: yes | 16:34 |
erbarr | ok | 16:35 |
TheJulia | any hosts have that mac, because that is the mac that was recorded for use to dhcp with | 16:35 |
erbarr | should have been 44:A8:42:24:24:66 on that job, that's the target mac address for the nic | 16:38 |
TheJulia | erbarr: ignore that request, it looks like it is the dhcp address | 16:38 |
erbarr | ok | 16:38 |
* TheJulia looks for that | 16:38 | |
TheJulia | found it | 16:39 |
*** ijw has quit IRC | 16:39 | |
*** ijw_ has joined #openstack-ironic | 16:39 | |
erbarr | cool | 16:42 |
TheJulia | it _looks_ like neutron never picked up that something had to change | 16:43 |
TheJulia | for the dhcp configuration | 16:44 |
TheJulia | and thus tftp failed | 16:44 |
TheJulia | we should see something the dhcp log when the port was last updated with all of the tftp data around 15:14:41 | 16:44 |
TheJulia | but there is zilch | 16:44 |
erbarr | ouch | 16:46 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity https://review.opendev.org/376574 | 16:47 |
TheJulia | the primary neutron service logs indiacate it put the message on the message bus | 16:47 |
TheJulia | well, triggered the callback which it should | 16:48 |
TheJulia | erbarr: are you in #openstack-neutron? | 16:51 |
erbarr | joining | 16:51 |
erbarr | i'm in there now | 16:52 |
*** derekh has quit IRC | 16:55 | |
erbarr | TheJulia, thanks! | 17:01 |
*** gregwork has quit IRC | 17:29 | |
*** vmud213 has quit IRC | 17:45 | |
TheJulia | erbarr: are all jobs failing now on dell's CI? | 18:02 |
TheJulia | at least master branch ones? | 18:02 |
erbarr | yea | 18:03 |
TheJulia | It does look like there was a change in notifications yesterday, but it looks completely unrelated area specifically with security groups... yet it is in update_port | 18:12 |
TheJulia | https://review.opendev.org/#/c/508446/5/neutron/plugins/ml2/plugin.py | 18:12 |
patchbot | patch 508446 - neutron - Fix unnecessary security_groups_member_updated not... (MERGED) - 5 patch sets | 18:12 |
TheJulia | oh, no | 18:15 |
TheJulia | that was in 2018 | 18:15 |
TheJulia | they cherry picked that yesterday | 18:15 |
TheJulia | erbarr: when was the first failure exactly and when was the last success before that? | 18:16 |
*** priteau has joined #openstack-ironic | 18:24 | |
erbarr | the first failure I saw started at this review https://review.opendev.org/749357 for patchset 2 | 18:32 |
patchbot | patch 749357 - ironic - Explicitely do not allocate initial space for virt... (MERGED) - 2 patch sets | 18:32 |
erbarr | this was the last pass https://review.opendev.org/749459 patchset 1 | 18:33 |
patchbot | patch 749459 - ironic - [DNM] Test multinode - 1 patch set | 18:33 |
erbarr | passing finished running at 7:19 am central time, failed job started running at 9:11 am, there was a job cancelled between the 2 due to another patchset | 18:35 |
erbarr | actually, found another job for that same failed trigger that started running at 8:18 | 18:38 |
*** bdodd_ has joined #openstack-ironic | 18:44 | |
*** bdodd has quit IRC | 18:46 | |
*** priteau has quit IRC | 18:47 | |
*** sshnaidm is now known as sshnaidm|off | 18:48 | |
openstackgerrit | Aija Jaunteva proposed openstack/sushy master: WIP: Allow monitoring progress of a firmware update https://review.opendev.org/745950 | 18:51 |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job https://review.opendev.org/748315 | 18:57 |
openstackgerrit | Merged openstack/ironic-python-agent master: If listen_tls is true, enable TLS on wsgi server https://review.opendev.org/747193 | 18:59 |
JayF | https://review.opendev.org/#/c/747309/ should be ready for review now. Adding support in ipa-builder for enabling ^^ that feature. | 19:02 |
patchbot | patch 747309 - ironic-python-agent-builder - Add element to configure IPA with TLS, use configdir - 3 patch sets | 19:02 |
JayF | I'll retest this tomorrow as well | 19:02 |
*** jtomasek has quit IRC | 19:05 | |
*** jtomasek has joined #openstack-ironic | 19:17 | |
*** jtomasek has quit IRC | 19:21 | |
*** tosky has quit IRC | 19:45 | |
openstackgerrit | Christopher Dearborn proposed openstack/ironic master: WIP: Redfish driver firmware update https://review.opendev.org/749619 | 19:47 |
*** thiagop has quit IRC | 19:59 | |
*** zzzeek has quit IRC | 20:08 | |
*** zzzeek has joined #openstack-ironic | 20:12 | |
*** zaneb has quit IRC | 20:26 | |
TheJulia | erbarr: unfortunately looking at neutron nothing seems to fall in that window :\ | 20:34 |
erbarr | TheJulia, thanks, I really appreciate the help | 20:39 |
openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job https://review.opendev.org/748315 | 20:40 |
*** Lucas_Gray has joined #openstack-ironic | 20:41 | |
TheJulia | erbarr: the alternative idea may be to just start to revert neutron and related patches until we find the right one :\ | 20:41 |
openstackgerrit | Merged openstack/ironic-python-agent stable/ussuri: Increase memory for tinyipa jobs https://review.opendev.org/747634 | 20:42 |
erbarr | TheJulia, yea I was thinking on checking patch differences between master branch and ussuri and reverting one by one | 20:43 |
*** zaneb has joined #openstack-ironic | 20:44 | |
*** zbitter has joined #openstack-ironic | 20:48 | |
*** zaneb has quit IRC | 20:48 | |
*** JamesBenson has quit IRC | 20:51 | |
TheJulia | erbarr: you can likely just start reverting off neutron master, only 6 patches or so | 20:52 |
erbarr | TheJulia, ok, is there an easy way via local.conf that you know of? | 20:54 |
TheJulia | nope | 20:54 |
TheJulia | you need to start nuking the patches on the neutron clone and restacking | 20:54 |
erbarr | ok, thanks! | 20:56 |
*** JamesBenson has joined #openstack-ironic | 21:08 | |
stevebaker | morning | 21:25 |
stevebaker | dtantsur|afk: I'll refresh https://review.opendev.org/#/c/737888 but I don't think its a huge deal if it misses the release | 21:25 |
patchbot | patch 737888 - metalsmith - metalsmith_deployment role switch to metalsmith_in... - 17 patch sets | 21:25 |
openstackgerrit | Merged openstack/ironic-python-agent stable/ussuri: Make WSGI server respect listen_* directives https://review.opendev.org/749597 | 21:26 |
*** adrianc has quit IRC | 21:35 | |
*** adrianc has joined #openstack-ironic | 21:35 | |
*** k_mouza has joined #openstack-ironic | 21:38 | |
openstackgerrit | Christopher Dearborn proposed openstack/ironic master: WIP: Redfish driver firmware update https://review.opendev.org/749619 | 21:41 |
*** bfournie has quit IRC | 21:54 | |
*** k_mouza has quit IRC | 22:09 | |
*** ijw_ has quit IRC | 22:25 | |
*** ijw has joined #openstack-ironic | 22:25 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-inspector master: Power off before inspection https://review.opendev.org/749845 | 22:46 |
*** janders has quit IRC | 22:53 | |
*** janders has joined #openstack-ironic | 22:55 | |
janders | good morning Ironic o/ | 22:55 |
*** hjensas has quit IRC | 22:56 | |
janders | migrating to a new laptop. Managed to migrate my IRC setup and all the history is still there, woohoo :) | 22:56 |
TheJulia | o/ | 22:59 |
*** Lucas_Gray has quit IRC | 23:02 | |
*** zzzeek has quit IRC | 23:05 | |
*** zzzeek has joined #openstack-ironic | 23:06 | |
*** Lucas_Gray has joined #openstack-ironic | 23:07 | |
*** Goneri has quit IRC | 23:16 | |
*** rcernin has joined #openstack-ironic | 23:24 | |
*** Lucas_Gray has quit IRC | 23:25 | |
*** ijw has quit IRC | 23:25 | |
*** ijw has joined #openstack-ironic | 23:25 | |
*** rcernin has quit IRC | 23:31 | |
*** rcernin has joined #openstack-ironic | 23:31 | |
*** zzzeek has quit IRC | 23:39 | |
*** zzzeek has joined #openstack-ironic | 23:40 | |
*** jamesden_ has joined #openstack-ironic | 23:53 | |
*** jamesdenton has quit IRC | 23:56 | |
*** zzzeek has quit IRC | 23:57 | |
*** zzzeek has joined #openstack-ironic | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!