| opendevreview | Merged openstack/ironic stable/zed: CI: Lock Zed CI to Zed IPA https://review.opendev.org/c/openstack/ironic/+/914243 | 00:14 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/ironic-python-agent master: Step to clean UEFI NVRAM entries https://review.opendev.org/c/openstack/ironic-python-agent/+/914563 | 01:22 |
| opendevreview | cid proposed openstack/ironic-inspector master: Use the new microversion-parse library to parse the version headers https://review.opendev.org/c/openstack/ironic-inspector/+/913794 | 04:18 |
| opendevreview | cid proposed openstack/ironic master: Special treatment of .json is now disabled for nodes with .json extension. https://review.opendev.org/c/openstack/ironic/+/913467 | 05:28 |
| opendevreview | cid proposed openstack/ironic master: Use the new microversion-parse library to parse the version headers https://review.opendev.org/c/openstack/ironic/+/913793 | 05:32 |
| opendevreview | cid proposed openstack/ironic-inspector master: Use the new microversion-parse library to parse the version headers https://review.opendev.org/c/openstack/ironic-inspector/+/913794 | 05:46 |
| opendevreview | Jacob Anders proposed openstack/sushy-oem-idrac master: Add jobs in Paused state to the list of unfinished jobs https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914605 | 07:29 |
| opendevreview | Jacob Anders proposed openstack/sushy-oem-idrac master: Remove Id field from DellJobCollection class https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914608 | 07:35 |
| janders | ^^ WIP changes in attempt to create a mechanism of handling the issue of duplicate iDRAC jobs of the same type | 07:36 |
| janders | (not an attempt to fix the main problem, just trying to make this code usable - currently doesn't seem to be the case) | 07:37 |
| janders | CC dtantsur rpittau, more to follo | 07:37 |
| rpittau | good morning ironic! o/ | 08:07 |
| rpittau | janders: thanks, I'll have a look during the day | 08:07 |
| rpittau | I'd like to merge https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914515 first | 08:07 |
| janders | agreed | 09:26 |
| dtantsur | rpittau: that has odd unrelated changes, of which one is wrong - see inline | 09:54 |
| opendevreview | Stephen Finucane proposed openstack/python-ironicclient master: Remove use of distutils https://review.opendev.org/c/openstack/python-ironicclient/+/914523 | 09:55 |
| opendevreview | Stephen Finucane proposed openstack/python-ironicclient master: trivial: Remove note about requirements ordering https://review.opendev.org/c/openstack/python-ironicclient/+/914524 | 09:55 |
| stephenfin | dtantsur: Would appreciate reviews on the first of those when you've a chance ^^ ironicclient's osc plugin is currently non-functional on the latest Fedora (39) | 09:56 |
| dtantsur | looking | 09:57 |
| rpittau | dtantsur: if the summary line is not split pep8 will complain because it's too long :D | 09:59 |
| rpittau | I'll try to make it shorter | 09:59 |
| dtantsur | rpittau: yep. other cases are not too long though, so not sure why these got changed | 10:00 |
| dtantsur | rpittau: in this case, just do "Reference to blah-blah" | 10:00 |
| dtantsur | no need to say it's a property | 10:00 |
| rpittau | mmm I have no idea how the other lines got changed..... | 10:01 |
| rpittau | ok, thanks for the suggestion | 10:01 |
| rpittau | oh I see! | 10:02 |
| rpittau | freaking IDE.... | 10:03 |
| opendevreview | Riccardo Pittau proposed openstack/sushy-oem-idrac master: Update setup and tox config https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914515 | 10:04 |
| opendevreview | Riccardo Pittau proposed openstack/sushy-oem-idrac master: Add codespell non voting test https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914517 | 10:05 |
| opendevreview | Verification of a change to openstack/ironic master failed: ci: automatically set the branch for IPA to match TARGET_BRANCH https://review.opendev.org/c/openstack/ironic/+/914250 | 10:16 |
| opendevreview | Jacob Anders proposed openstack/sushy-oem-idrac master: Remove Id field from DellJobCollection class https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914608 | 11:46 |
| mgoddard | JayF: it's not really on my immediate todo list. If someone else wants to pick it up they would be welcome to | 11:48 |
| opendevreview | cid proposed openstack/ironic-tempest-plugin master: Patch to enforce json extension works in existing API behaviour https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/913926 | 12:06 |
| iurygregory | good morning Ironic | 12:23 |
| opendevreview | Jacob Anders proposed openstack/sushy-oem-idrac master: Add jobs in Paused state to the list of unfinished jobs https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914605 | 12:43 |
| iurygregory | so, I got a question downstream regarding Accept-Encondig... would we be ok in setting to all possible values instead of just identity? self._session.headers['Accept-Encoding'] = 'identity, deflate, compress, gzip' | 13:06 |
| iurygregory | they mentioned that the response content type only needs to match one of them, so we wouldn't need to retry... | 13:07 |
| iurygregory | Just thinking is some BMC could go crazy if we set all values in Accept-Enconding.. wdyt? | 13:08 |
| TheJulia | iurygregory: is there a bug or something trying to explain why it is needed. I guess I'm not super concerned about the bmcs, the issue which concerns me is if we're setting it, we need to ensure it actually works, and generally encoding is an area where things are not that complex | 13:25 |
| TheJulia | the issue with having *any* compression though, is you can never get a stable etag | 13:25 |
| iurygregory | TheJulia, yep! it's the one dtantsur fixed https://review.opendev.org/c/openstack/sushy/+/914118 | 13:25 |
| iurygregory | TheJulia, oh right! the eTag... | 13:25 |
| iurygregory | so if we have all of them we could still face the issue | 13:26 |
| TheJulia | well | 13:26 |
| TheJulia | it would depend on the replying bmc | 13:26 |
| dtantsur | If I understood it right, we'd need to provide a list with priorities, putting identity higher | 13:26 |
| TheJulia | say if it chose compress, deflate, or gzip | 13:26 |
| dtantsur | Which I considered doing, but then how many BMCs will get confused? | 13:26 |
| iurygregory | dtantsur, yeah, this is the point I would be afraid... | 13:27 |
| TheJulia | its a huge risk since we've seen encoding issues bite us around etags and response compression before | 13:27 |
| TheJulia | I clearly need way more coffee and not to be worrying about my father this morning | 13:27 |
| iurygregory | now I have a clear answer to provide to them, it's better one retry than BMCs trowing encoding problems to us :D | 13:28 |
| TheJulia | If memory serves, the reason we went for identity to begin with was to ensure our etags were strong | 13:28 |
| iurygregory | TheJulia, take the day off if you need, so you can try to relax a bit | 13:29 |
| TheJulia | I need to call a repair shop to see if I can get my motorhome in for repairs, I won't have any updates from the hospital until 9-10 AM at the earliest | 13:29 |
| TheJulia | so anyway, waking up | 13:30 |
| iurygregory | ack | 13:30 |
| TheJulia | ... I wonder if the same error dmitry fixed is the root issue on interacting with virtual media on the gen10 plus server I had a customer try with | 13:37 |
| TheJulia | It spat back an error indicating something was not acceptable, but didn't actually indicate the missing field or value | 13:38 |
| TheJulia | oh. | 13:38 |
| TheJulia | I bet it was licensing now that I think about it | 13:38 |
| iurygregory | mgoddard, hey you around? I have someone deploying kolla, and they noticed the ironic.conf [dhcp]dhcp_provider = none, do you know where to change this config? | 14:26 |
| iurygregory | TheJulia, it can be I would say | 14:26 |
| TheJulia | Well, standalone ironic right? | 14:33 |
| iurygregory | yeah | 14:34 |
| iurygregory | they don't want neutron, seems like when you have enable_neutron = false it will set to none | 14:35 |
| iurygregory | https://opendev.org/openstack/kolla-ansible/src/branch/master/ansible/roles/ironic/templates/ironic.conf.j2#L219 | 14:35 |
| TheJulia | I guess they are sort of hoping to use the integrated dhcp stuffs? | 14:40 |
| iurygregory | yup | 14:40 |
| iurygregory | I told them now to hardcode in this file https://opendev.org/openstack/kolla-ansible/src/branch/master/ansible/roles/ironic/templates/ironic.conf.j2#L219 | 14:41 |
| iurygregory | in a few min they will tell me if worked | 14:41 |
| JayF | TheJulia: I know you have family stuff going on; just lmk if/when you're available to talk about self-serve templates | 16:48 |
| TheJulia | JayF: it is mostly just in waiting | 16:48 |
| TheJulia | I'm semi-in the middle of something but almost wrapping that up now | 16:48 |
| JayF | ack | 16:48 |
| JayF | actually, it's almost 10am and I skipped breakfast | 16:49 |
| JayF | I'll remedy that and hopefully we'll be good to go after | 16:49 |
| TheJulia | +++++++++ | 16:49 |
| rpittau | good night! o/ | 17:02 |
| TheJulia | JayF: so.... | 17:17 |
| JayF | I need 5 more mins | 17:18 |
| TheJulia | ok | 17:18 |
| opendevreview | Jay Faulkner proposed openstack/ironic-specs master: Self-Service via Templates https://review.opendev.org/c/openstack/ironic-specs/+/890164 | 19:34 |
| opendevreview | Jay Faulkner proposed openstack/ironic-specs master: CI: Make old spec compliant with new lint https://review.opendev.org/c/openstack/ironic-specs/+/914703 | 19:34 |
| JayF | 914703 is a super trivial change to fix CI on ironic-specs | 19:34 |
| JayF | someone please land it | 19:34 |
| JayF | the self-service templates spec is also ready for further review | 19:34 |
| JayF | I'll note self-service templates is not on PTG schedule; but I'm not sure it needs to be? | 19:35 |
| JayF | TheJulia: did you miss https://review.opendev.org/c/openstack/python-ironicclient/+/908534/1#message-a3b13c5ace8e031fc2ce113b10d621b704cbf68a on this change? (Dmitry asking for a release note). Do you want me to JFDI the update at the cost of losing my +2? | 20:02 |
| TheJulia | Didn't even see his comment | 20:09 |
| JayF | yeah that's what I figured | 20:10 |
| opendevreview | Julia Kreger proposed openstack/python-ironicclient master: Fix on-creation ability https://review.opendev.org/c/openstack/python-ironicclient/+/908534 | 20:16 |
| JayF | I also think https://review.opendev.org/c/openstack/ironic/+/900846 is landable | 20:16 |
| JayF | I think https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/913926 is further evidence our skip logic is screwed up around microversions :/ | 20:35 |
| JayF | I'll try to find time to loook at that before I disappear this weekedn | 20:36 |
| TheJulia | so I looked at the underlying code, and it is config driven, not remote driven | 20:49 |
| TheJulia | which is inherently breaky | 20:49 |
| JayF | oooh | 20:58 |
| JayF | and our changes to stable branches | 20:59 |
| JayF | don't propogate back to i-t-p tests | 20:59 |
| JayF | at least, they don't *seem* to? | 20:59 |
| TheJulia | tempest is always master | 21:04 |
| TheJulia | at some points, we pin older branches to older tempest plugin releases | 21:04 |
| TheJulia | but that is stupidly rare | 21:04 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: leverage remote microversion without requiring human config https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/914270 | 21:10 |
| TheJulia | doh | 21:10 |
| JayF | This is the first time I've really "gotten" why the config is painful, haven't thought about it from the i-t-p side before | 21:14 |
| opendevreview | Merged openstack/ironic-python-agent master: USB device discovery https://review.opendev.org/c/openstack/ironic-python-agent/+/902769 | 21:22 |
| TheJulia | I'm sort of hoping cls.client exists before skiptest is called | 21:25 |
| TheJulia | if so, then we're in good shape | 21:25 |
| JayF | If someone approves https://review.opendev.org/c/openstack/ironic-python-agent/+/913208/ , that whole chain from dking should merge | 21:28 |
| TheJulia | nope, no client | 21:30 |
| opendevreview | Merged openstack/ironic master: Fix Reopen Web Console Duplicate Sol Session https://review.opendev.org/c/openstack/ironic/+/900846 | 21:41 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: leverage remote microversion without requiring human config https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/914270 | 21:45 |
| * TheJulia crosses her fingers | 21:45 | |
| opendevreview | Merged openstack/ironic stable/zed: Special case lenovo UEFI boot setup https://review.opendev.org/c/openstack/ironic/+/910448 | 21:49 |
| TheJulia | so thinking about the step templating, we likely ought to block some time for the topic | 21:59 |
| TheJulia | rpittau: fyi ^ | 22:00 |
| JayF | ack; I'll add it to the PTG schedule | 22:01 |
| JayF | we were "working together" on it this morning, and by that, I mean I told him I'd be around but my disabled-IRC-notifications made me more of a validator than a helper :| | 22:02 |
| JayF | rpittau: TheJulia: I added it to the 1500 UTC Wednesday PTG block with one of my topics which shouldn't need a whole hour | 22:04 |
| TheJulia | not quite, but not blowing up quite yet | 22:13 |
| TheJulia | oh, heh | 22:23 |
| TheJulia | I see what is going on | 22:23 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: leverage remote microversion without requiring human config https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/914270 | 23:10 |
| opendevreview | Merged openstack/ironic stable/zed: Fix system scoped manageable node network failure https://review.opendev.org/c/openstack/ironic/+/907564 | 23:27 |
| TheJulia | JayF: so my change might do the trick to make it auto-detected :) | 23:46 |
| TheJulia | logs confirm it is getting the /baremetal resource endpoint and getting version data off of it for each class trigger, so that is a good sign | 23:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!