opendevreview | Iury Gregory Melo Ferreira proposed openstack/sushy master: Handle non-default language for registries https://review.opendev.org/c/openstack/sushy/+/872049 | 00:06 |
---|---|---|
opendevreview | Verification of a change to openstack/bifrost master failed: CI fixes and workarounds https://review.opendev.org/c/openstack/bifrost/+/874650 | 01:07 |
opendevreview | Steve Baker proposed openstack/ironic master: Document [fake] delay config values https://review.opendev.org/c/openstack/ironic/+/868053 | 02:07 |
opendevreview | Merged openstack/bifrost master: CI fixes and workarounds https://review.opendev.org/c/openstack/bifrost/+/874650 | 04:25 |
opendevreview | Ebbex proposed openstack/bifrost master: remove unused variables https://review.opendev.org/c/openstack/bifrost/+/874518 | 06:45 |
rpittau | good morning ironic! o/ | 08:06 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: Finally fix jinja[spacing] https://review.opendev.org/c/openstack/bifrost/+/872634 | 08:08 |
kubajj | Morning rpittau and ironic | 08:09 |
rpittau | hey kubajj :) | 08:09 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: Fixes "too many spaces after colon" lint error https://review.opendev.org/c/openstack/bifrost/+/872633 | 08:15 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: Fixes "too many spaces after colon" lint error https://review.opendev.org/c/openstack/bifrost/+/872633 | 08:16 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: Finally fix jinja[spacing] https://review.opendev.org/c/openstack/bifrost/+/872634 | 08:16 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: Finally fix jinja[spacing] https://review.opendev.org/c/openstack/bifrost/+/872634 | 08:17 |
arne_wiebalck | Good morning rpittau kubajj and Ironic! | 08:29 |
rpittau | hey arne_wiebalck :) | 08:29 |
opendevreview | Ebbex proposed openstack/bifrost master: Use a more traditional ansible approach to include_vars https://review.opendev.org/c/openstack/bifrost/+/855806 | 09:43 |
opendevreview | Ebbex proposed openstack/bifrost master: Refactor use of include_vars https://review.opendev.org/c/openstack/bifrost/+/855807 | 09:43 |
jrosser | ebbex: you saw i had to improve that vars loading to be specific about which directory is intended? https://github.com/openstack/openstack-ansible-os_glance/blob/master/tasks/main.yml#L17 | 09:49 |
jrosser | the sequence of places ansible will search for files of those names can lead to unexpected results | 09:50 |
ebbex | jrosser: yeah, hope you don't mind me borrowing that here? I think it's a pretty neat and well explained use-case. | 09:55 |
jrosser | oh sure its fine | 09:55 |
iurygregory | good morning Ironic | 11:17 |
arne_wiebalck | hey iurygregory o/ | 12:04 |
iurygregory | arne_wiebalck, hey o/ | 12:06 |
dtantsur | sschmitt: networking-baremetal was initially an internal ML2 plugin that allowed binding to work correctly for bare metal ports. | 12:13 |
dtantsur | I suspect the folks have more plans for its development, but initially it was just that: an internal helper | 12:13 |
dtantsur | networking-baremetal is designed to work with the traditional OVS | 12:13 |
dtantsur | sschmitt: n-g-s is also an ML2 plugin, but its goal is to interact with hardware switches, so do some actual non-trivial job when binding baremetal ports. | 12:14 |
dtantsur | the non-trivial job was, most of the time, putting ports on right VLANs | 12:14 |
dtantsur | I hope it helps | 12:14 |
iurygregory | rpittau, the IPE patch just need a new set of eyes https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/869509 | 12:19 |
rpittau | iurygregory: lot of comments are unresolved, are we ok with moving forward anyway ? | 13:16 |
kubajj | dtantsur: just checking, is the inventory API going to be included in Antelope release? (Just so I don't lie in my dissertation. 😉 ) | 13:54 |
dtantsur | kubajj: I think so, unless I completely lost track of time. | 13:54 |
kubajj | dtantsur: ok, thanks | 13:55 |
kubajj | dtantsur: should I also email you the questions for the evaluation? (I am going to finalise them tomorrow with my supervisor) | 13:55 |
dtantsur | kubajj: yep, absolutely. do you know my email? | 13:59 |
iurygregory | rpittau, they aren't blockers | 14:16 |
iurygregory | minor improvements that can be done as follow-up (kinda low-hanging fruit bugs) | 14:17 |
kubajj | dtantsur: the protonmail one? | 14:20 |
TheJulia | iurygregory: given the low level of contribution on i-p-e, I suspect it might be okay to single core approve on it. just, thinking outloud | 14:32 |
dtantsur | kubajj: yes, or use dtantsur@redhat.com if you want it a bit more official | 14:42 |
kubajj | ok, thans | 14:42 |
rpittau | iurygregory: approved | 14:58 |
opendevreview | Derek Higgins proposed openstack/sushy stable/zed: Add TransferProtocolType for any general error that mentions it https://review.opendev.org/c/openstack/sushy/+/875744 | 15:12 |
opendevreview | Merged openstack/ironic master: Fix expired links https://review.opendev.org/c/openstack/ironic/+/873896 | 15:28 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: Support ansible-collections-openstack 2 and later https://review.opendev.org/c/openstack/bifrost/+/875750 | 15:33 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: Support ansible-collections-openstack 2 and later https://review.opendev.org/c/openstack/bifrost/+/875750 | 15:44 |
opendevreview | Merged openstack/ironic-prometheus-exporter master: Support extraction of ironic internal metrics https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/869509 | 15:51 |
iurygregory | dtantsur, tks for the review in the sushy patch | 15:58 |
opendevreview | Iury Gregory Melo Ferreira proposed openstack/sushy master: Handle non-default language for registries https://review.opendev.org/c/openstack/sushy/+/872049 | 15:59 |
opendevreview | Merged openstack/ironic-specs master: Enumerate retired specs https://review.opendev.org/c/openstack/ironic-specs/+/873308 | 16:07 |
rpittau | good night! o/ | 16:50 |
opendevreview | Merged openstack/metalsmith stable/zed: list_instances - cache allocations https://review.opendev.org/c/openstack/metalsmith/+/873589 | 17:10 |
opendevreview | Dmitry Tantsur proposed openstack/python-ironicclient master: Allow several nodes for most node actions https://review.opendev.org/c/openstack/python-ironicclient/+/875756 | 17:12 |
dtantsur | I've resisted doing this ^^^ for a long time | 17:12 |
dtantsur | but I finally got too annoyed by it | 17:12 |
TheJulia | heh | 17:12 |
* dtantsur does not remember if it's already too late for Antelope client changes | 17:13 | |
JayF | yes, it is | 17:13 |
dtantsur | alas! my bifrost is permanently on master anyway :) (and so it metal3) | 17:14 |
JayF | I'm working on really my first sizable client change | 17:20 |
JayF | and I really don't grasp the value of some of the unit tests in that repo | 17:20 |
JayF | looks like a lot of copy-paste-itis | 17:20 |
JayF | but I assume when I feel this way about something; there's gotta be a good reason it's done that way, something we're guarding against I'm not familiar with | 17:20 |
dtantsur | I was just thinking the same thing.. a lot of copy-paste | 17:43 |
JayF | I usually follow boy scout rules when working in a repo I've not worked in a lot before | 17:46 |
JayF | just leave the campsite clean and don't mess with nature | 17:46 |
JayF | so I'm going to assume whoever came before knew what they were doing and just exercise my clipboard LOL | 17:47 |
opendevreview | Merged openstack/ironic master: Respond to rpc requests on stop until hash ring reset https://review.opendev.org/c/openstack/ironic/+/872601 | 17:47 |
opendevreview | Merged openstack/bifrost master: Fixes "too many spaces after colon" lint error https://review.opendev.org/c/openstack/bifrost/+/872633 | 18:14 |
opendevreview | Julia Kreger proposed openstack/ironic master: Clean out agent token even if power is already off https://review.opendev.org/c/openstack/ironic/+/875778 | 18:21 |
stevebaker[m] | good morning | 19:12 |
fungi | rpittau: i got opendev consensus for the virtualpdu hand-off plan. tomorrow i'll post to service announce and openstack-discuss, cc'ing the existing maintainers on the latter, with a notice of intent to add the ironic ptl to the group(s) for that repository in one week, barring any objections | 20:14 |
iurygregory | \o/ great news | 20:58 |
JayF | fungi: my preference is that we just add ironic's core group to the cores there, if there's any desire to cut out a middle-step | 21:15 |
JayF | especially since timing wise; I might be at a conference when the cutover happens | 21:15 |
JayF | I guess that's a PR, not just a gerrit perms change | 21:16 |
fungi | that can work. normally we treat the ptl as the representative for the team and let them handle whatever delegation they intend afterward | 21:19 |
JayF | yeah; in this case the handover will likely be quickly followed by a governance proposal to move it from x/ to openstack/ and officially adopt it as bare metal | 21:20 |
JayF | assuming that step isn't included in what you're doing | 21:20 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!