| opendevreview | Verification of a change to openstack/ironic-python-agent bugfix/8.6 failed: Fix CI for bugfix/8.6 https://review.opendev.org/c/openstack/ironic-python-agent/+/868065 | 00:10 |
|---|---|---|
| opendevreview | Verification of a change to openstack/ironic-python-agent bugfix/8.6 failed: Drop python2 from bindep.txt https://review.opendev.org/c/openstack/ironic-python-agent/+/867915 | 00:10 |
| TheJulia | JayF: technically yes, but i don't think it would really need to be version guarded, more as an annoucement when done | 00:26 |
| TheJulia | since previously it just wouldn't have worked and all | 00:26 |
| opendevreview | Merged openstack/ironic master: CI: Reset VM footprint to 2.6GB https://review.opendev.org/c/openstack/ironic/+/869136 | 00:48 |
| JayF | yeah, that's what I thought. It really will be trivial to implement. | 01:04 |
| arne_wiebalck | Good morning, Ironic! | 07:26 |
| rpittau | good morning ironic! o/ | 08:43 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.6: Fix CI for bugfix/8.6 https://review.opendev.org/c/openstack/ironic-python-agent/+/868065 | 09:15 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.6: Drop python2 from bindep.txt https://review.opendev.org/c/openstack/ironic-python-agent/+/867915 | 09:16 |
| dtantsur | JayF: re API version, if I understand you right, the answer is "absolutely yes" | 09:44 |
| dtantsur | and I'm not sure why TheJulia says it does not have to be guarded. per our rules, it absolutely does, it's a new feature. | 09:44 |
| dtantsur | mastodon folks, boosting very welcome: https://mastodon.online/@creepy_owlet/109669997638869827 | 10:03 |
| iurygregory | good morning Ironic | 11:49 |
| rpittau | hey iurygregory :) | 11:56 |
| opendevreview | waleed mousa proposed openstack/ironic-python-agent master: update NVIDIA NIC firmware images and settings by ironic-python-agent https://review.opendev.org/c/openstack/ironic-python-agent/+/566544 | 12:24 |
| iurygregory | rpittau, o/ | 12:34 |
| kubajj | morning everyone | 12:47 |
| iurygregory | morning kubajj | 12:55 |
| TheJulia | dtantsur: I was more thinking a) high cost for overhead area of the api, b) rules seem written to guard against a case of usage were a bit unlikely to see then, granted, it would mean it could now work if manually tried on an older client when bad input wouldn't have worked previously | 13:32 |
| dtantsur | TheJulia: the goal of the rules is also discoverability of API changes coupled with enforced usage of API versions | 13:34 |
| dtantsur | as you know, I'm not exactly the biggest fan of microversions, but we've committed to them :) | 13:34 |
| TheJulia | descoverability I absolutely agree with | 13:34 |
| TheJulia | I think we should at least bump the version, we don't necessarily need to do "are there any commas" and then make decisions based upon the version if we don't necessarily have to | 13:35 |
| TheJulia | then again | 13:36 |
| TheJulia | there is the flip side, run the logic based upon the api version | 13:36 |
| TheJulia | that *should* be much lower overhead | 13:36 |
| TheJulia | at least until everyone is on the newest/latest client :) | 13:37 |
| opendevreview | waleed mousa proposed openstack/ironic-python-agent master: update NVIDIA NIC firmware images and settings by ironic-python-agent https://review.opendev.org/c/openstack/ironic-python-agent/+/566544 | 14:00 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix selinux context of published image hardlink https://review.opendev.org/c/openstack/ironic/+/868521 | 14:58 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix selinux context of published image hardlink https://review.opendev.org/c/openstack/ironic/+/868521 | 15:00 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add `service` role RBAC policy support https://review.opendev.org/c/openstack/ironic/+/869614 | 15:01 |
| arne_wiebalck | Good morning iurygregory rpittau dtantsur TheJulia kubajj o/ | 15:14 |
| TheJulia | good morning arne_wiebalck | 15:15 |
| iurygregory | arne_wiebalck, o/ | 15:15 |
| arne_wiebalck | preparing a visit of another community here at CERN brings back fond memories of the Ironic team down in the CMS cavern :) | 15:16 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix grub config path default https://review.opendev.org/c/openstack/ironic/+/869134 | 15:18 |
| TheJulia | :) | 15:18 |
| TheJulia | That was neat to see, giving us the whole scope of use really helped | 15:19 |
| opendevreview | Julia Kreger proposed openstack/ironic-inspector master: setup.py: Remove 'py_modules' https://review.opendev.org/c/openstack/ironic-inspector/+/869198 | 15:30 |
| opendevreview | Merged openstack/sushy-tools master: Fix tox4 error https://review.opendev.org/c/openstack/sushy-tools/+/868760 | 15:53 |
| rpittau | goodnight! o/ | 16:13 |
| opendevreview | Julia Kreger proposed openstack/ironic-inspector master: setup.py: Remove 'py_modules' https://review.opendev.org/c/openstack/ironic-inspector/+/869198 | 16:18 |
| opendevreview | Julia Kreger proposed openstack/ironic-inspector master: setup.py: Remove 'py_modules' https://review.opendev.org/c/openstack/ironic-inspector/+/869198 | 16:23 |
| opendevreview | Jay Faulkner proposed openstack/sushy stable/zed: Fix tox4 and setuptools errors https://review.opendev.org/c/openstack/sushy/+/868785 | 16:24 |
| * TheJulia hopes the inspector patch makes it happier | 16:30 | |
| *** dmellado_ is now known as dmellado | 17:03 | |
| TheJulia | Do we have any dell folks hanging out on irc on a regular basis these days? | 17:59 |
| samuelkunkel[m] | Guys working for dell? Or user(s) of their products? | 18:00 |
| TheJulia | dell employees | 18:07 |
| TheJulia | not everyone here is a guy :) | 18:07 |
| TheJulia | some in channel are even gender neutral bots | 18:08 |
| * TheJulia looks at opendevstatus | 18:08 | |
| samuelkunkel[m] | I know, was just the easiest to type quickly. Sorry. | 18:08 |
| TheJulia | no worries :) | 18:09 |
| TheJulia | Just getting a smile | 18:09 |
| samuelkunkel[m] | ;) - but anyway, nothing I could help. If you look for users of their product, thats were I could rather help as we have tons of them | 18:11 |
| JayF | Honestly it's nice to see a new name talking in chat | 18:14 |
| TheJulia | samuelkunkel[m]: Thanks :) By chance what is the latest firmware version your using? | 18:17 |
| TheJulia | for the idrac specifically | 18:17 |
| samuelkunkel[m] | 6.00 and some 5.10 | 18:20 |
| samuelkunkel[m] | Do you need it more specific? | 18:20 |
| samuelkunkel[m] | We do not run the latest (6.10) yet | 18:20 |
| samuelkunkel[m] | Also I dont know who created that versioning schema | 18:24 |
| TheJulia | yeah, apparently the report I have is 6.10 | 18:25 |
| JayF | samuelkunkel[m]: if you're looking for people to talk about how firmware doesn't make sense, we're professionals at that ;) | 18:25 |
| TheJulia | ... yeah, I've had some "huh?" "nobody knows" dialogs with them | 18:26 |
| TheJulia | regarding versioning | 18:26 |
| samuelkunkel[m] | JayF: I will get back to this offer, trust me ;) | 18:44 |
| samuelkunkel[m] | TheJulia: yes, 6.10 is sounds correct as the latest. We are testing the new version somewhere during the next week(s). Not expecting big changes… from 5.10 to 6.0 only thing we noticed are bugs in regards to dhcp dns registration of the idrac… | 18:47 |
| opendevreview | Merged openstack/ironic master: setup.py: Remove 'py_modules' https://review.opendev.org/c/openstack/ironic/+/869197 | 18:54 |
| TheJulia | I know we've and some dell folks have spotted a few things, but overall it seems kind of minor in the grand scheme, what I've got a report of is that boot devices can no longer be set, but I need logging data to be able to determine if it has or has not already been fixed upstream, so you might be a good data point :) | 18:57 |
| samuelkunkel[m] | So during the deploy stage of the ipa? After installing the os to the selected disk? (As there is, to my knowledge, the change of the boot device) | 19:00 |
| TheJulia | they hit it upon trying to introspect before even deploying | 19:00 |
| samuelkunkel[m] | Hmm, I can surely test this sometime next week. Does the interface matter? As we are running purely ipmitool (no redfish currently) | 19:02 |
| TheJulia | oh, so redfish | 19:02 |
| samuelkunkel[m] | Ok, I can change that for a test (would be a good one anyway as we want to switch for most newer stuff anyway) | 19:03 |
| TheJulia | cool, much appreciated! | 19:03 |
| samuelkunkel[m] | I can have a look into it on monday. | 19:06 |
| samuelkunkel[m] | Just for further clarification, whats the state of the server before starting the introspection? (I assume its started manually in this case?) | 19:06 |
| samuelkunkel[m] | I assume manageable? | 19:06 |
| samuelkunkel[m] | And then inspect is executed and fails because it can not change the boot device? | 19:07 |
| TheJulia | Yeah, that appears to be the ccase | 19:09 |
| opendevreview | Merged openstack/ironic-inspector master: setup.py: Remove 'py_modules' https://review.opendev.org/c/openstack/ironic-inspector/+/869198 | 19:27 |
| TheJulia | fun == when you write code to write tests to write code | 20:48 |
| JayF | I'm executing the bugfix branch retirements now in ironic | 20:56 |
| JayF | completed ironic, beginning IPA | 20:57 |
| TheJulia | \o/ | 20:57 |
| JayF | IPA complete, moving to ironic-inspector | 21:00 |
| JayF | issues with inspector, pausing | 21:01 |
| JayF | looks like maybe permissinos in gerrit | 21:02 |
| TheJulia | gah, our internal naming inconsistencies are going to drive me crazy | 21:28 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Make metrics names a little more consistent https://review.opendev.org/c/openstack/ironic/+/869879 | 22:52 |
| JayF | that is a change that is constantly in a state of superposition | 22:56 |
| JayF | I love and hate is simultaneously | 22:56 |
| JayF | (it's better; but if I were an operator I'd be extremely unhappy about having to align with a new metric name) | 22:57 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add release note for association proxy changes https://review.opendev.org/c/openstack/ironic/+/869881 | 22:59 |
| TheJulia | JayF: internal and non-contracted naming... *shrugs* | 23:00 |
| TheJulia | I.... really want to rewrite all our metric names | 23:00 |
| TheJulia | ... like... a lot | 23:00 |
| JayF | you wouldn't consider changing a metrics name some kind of api break?! | 23:00 |
| TheJulia | it happens anytime code is shuffled | 23:01 |
| TheJulia | because the actual decorator puts the file path in | 23:01 |
| JayF | I'm dead serious when I say, if you had done that (rewrite all our metric names) when I was operating dashboards against ironic, you would've created days of downstream work | 23:01 |
| TheJulia | and the classpath | 23:01 |
| JayF | maybe less so if they were all changed in a similar/scriptable manner | 23:01 |
| TheJulia | so it is like ironic.conductor.manager.ConductorManager.whateverthehumanset | 23:01 |
| TheJulia | Anyway, I've got a migraine after writing code to massage ^^^^ that ^^^ into something promethus friendly | 23:02 |
| TheJulia | ironic_conductormanager_check_rescuewait_timeouts_sum{component="conductor",hostname="devstack",service="ironic"} 119.98176574707031 | 23:02 |
| JayF | I guess we should be glad that metrics moved away from this kind.of.dot.separated.shenanigans | 23:03 |
| JayF | like, tbh, if you wanted to change all the metric names | 23:03 |
| JayF | that gets into "then we should see if anyone actually uses statsd and change the nature of ironic metrics" territory | 23:03 |
| JayF | which I can be onboard for | 23:03 |
| TheJulia | dunno, it could mean we could just directly leverage *some* of the promethus client stuff internally, and somewhat directly, which would make some things less painful, but other things with it are... still painful | 23:04 |
| TheJulia | I should hit a stopping point with fresh code to review tomorrow | 23:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!