| *** lennyb has quit IRC | 00:02 | |
| *** lennyb has joined #openstack-ironic | 00:04 | |
| *** Goneri has quit IRC | 01:25 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add NTP time sync https://review.opendev.org/707911 | 01:50 |
|---|---|---|
| *** Lucas_Gray has joined #openstack-ironic | 01:59 | |
| *** ociuhandu has joined #openstack-ironic | 02:12 | |
| *** ociuhandu has quit IRC | 02:16 | |
| *** cdearborn has quit IRC | 02:17 | |
| *** igordc has quit IRC | 02:17 | |
| *** Lucas_Gray has quit IRC | 02:25 | |
| *** mkrai has joined #openstack-ironic | 02:50 | |
| *** adrianc has quit IRC | 02:58 | |
| *** adrianc has joined #openstack-ironic | 03:01 | |
| *** ijw_ has quit IRC | 03:22 | |
| *** ijw has joined #openstack-ironic | 03:25 | |
| *** ijw has quit IRC | 03:29 | |
| *** mkrai has quit IRC | 03:46 | |
| *** mkrai has joined #openstack-ironic | 04:02 | |
| *** tzumainn has quit IRC | 05:00 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Added scope to introspection rules. https://review.opendev.org/702215 | 05:45 |
| *** billp_ has joined #openstack-ironic | 06:14 | |
| *** billp has quit IRC | 06:16 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Ignore pyudev errors about device number https://review.opendev.org/706993 | 06:20 |
| iurygregory | good morning Ironic o/ | 06:27 |
| openstackgerrit | Maël Kimmerlin proposed openstack/ironic master: Redfish: Add root_prefix to Sushy https://review.opendev.org/707342 | 07:08 |
| arne_wiebalck | Good morning iurygregory and ironic! | 07:14 |
| iurygregory | arne_wiebalck, o/ | 07:18 |
| openstackgerrit | Maël Kimmerlin proposed openstack/ironic master: Redfish: Add root_prefix to Sushy https://review.opendev.org/707342 | 07:21 |
| *** Lucas_Gray has joined #openstack-ironic | 07:40 | |
| *** mbeierl has quit IRC | 07:46 | |
| *** rpittau|afk is now known as rpittau | 07:58 | |
| rpittau | good morning ironic! o/ | 07:58 |
| *** tesseract has joined #openstack-ironic | 08:02 | |
| iurygregory | good morning rpittau o/ | 08:06 |
| rpittau | hey iurygregory :) | 08:06 |
| rpittau | iurygregory: glad the job is working, but it seems it needs more troubleshooting :) | 08:07 |
| iurygregory | rpittau, i think the reason why we don't see the requests is just because we are running with -D | 08:07 |
| iurygregory | or we can now just add --log-file | 08:07 |
| iurygregory | to ensure we have something showing we did the request | 08:08 |
| rpittau | iurygregory: the -D could definitely be the reason, have tou tried locally? | 08:08 |
| iurygregory | im deploying a new env now =) | 08:08 |
| *** tkajinam has quit IRC | 08:31 | |
| *** amoralej|off is now known as amoralej | 08:32 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Start removing ipxe support from the pxe interface https://review.opendev.org/696666 | 08:38 |
| *** mkrai has quit IRC | 08:56 | |
| *** priteau has joined #openstack-ironic | 08:58 | |
| *** lucasagomes has joined #openstack-ironic | 09:08 | |
| *** mkrai has joined #openstack-ironic | 09:11 | |
| *** rcernin has quit IRC | 09:12 | |
| *** dougsz has joined #openstack-ironic | 09:13 | |
| *** k_mouza has joined #openstack-ironic | 09:15 | |
| *** derekh has joined #openstack-ironic | 09:25 | |
| iurygregory | rpittau,the -D it's not necessary to make it work, I've tested locally and the app still works =), but after removal the journalctl for the service doesn't show that I did requests | 09:27 |
| *** mbeierl has joined #openstack-ironic | 09:31 | |
| rpittau | iurygregory: yeah, that is the daemonize option, not needed for systemd | 09:33 |
| iurygregory | rpittau, got it, I've try adding log-file option to see if the request would show up there, but didn't work | 09:34 |
| *** Lucas_Gray has quit IRC | 09:35 | |
| *** Lucas_Gray has joined #openstack-ironic | 09:41 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Native zuulv3 grenade job for ironic https://review.opendev.org/703098 | 09:52 |
| iurygregory | rpittau, do you think it's worth try something else or we can let the i-p-e like that? =) | 09:52 |
| *** Wryhder has joined #openstack-ironic | 10:01 | |
| *** Lucas_Gray has quit IRC | 10:01 | |
| *** Wryhder is now known as Lucas_Gray | 10:01 | |
| *** dtantsur|afk is now known as dtantsur | 10:05 | |
| dtantsur | morning ironic | 10:05 |
| rpittau | iurygregory: I'm not against leaving it like that for the time being, as far as the job actually works, but would be nice to understand why we're not getting logs, and maybe fix it :) | 10:07 |
| iurygregory | rpittau, agree o/ | 10:08 |
| iurygregory | I'll keep testing locally to see if can identify the reason | 10:08 |
| iurygregory | morning dtantsur | 10:09 |
| dtantsur | etingof: morning! did you have a chance for a project-config fix for sushy-cli? | 10:09 |
| etingof | dtantsur, o/ on that | 10:10 |
| dtantsur | cool, yeah. let's try to make infra merge it quickly or we cannot release | 10:10 |
| etingof | yup | 10:11 |
| openstackgerrit | Riccardo Pittau proposed openstack/sushy-cli master: Update commands to generate docs, renos and pdf https://review.opendev.org/706826 | 10:13 |
| etingof | dtantsur, https://review.opendev.org/708329 | 10:19 |
| patchbot | patch 708329 - project-config - Add publish-to-pypi template for sushy-cli - 1 patch set | 10:19 |
| dtantsur | thanks! | 10:20 |
| etingof | +2-empowered folks have a chance to execute their super-powers here - https://review.opendev.org/#/c/706005/ https://review.opendev.org/706461 | 10:21 |
| patchbot | patch 706005 - sushy-cli - Add system/manager/chassis inventory show command - 10 patch sets | 10:21 |
| patchbot | patch 706461 - sushy-cli - Add systems, managers and chassis listing commands - 8 patch sets | 10:21 |
| *** mkrai_ has joined #openstack-ironic | 10:42 | |
| *** mkrai has quit IRC | 10:45 | |
| *** Lucas_Gray has quit IRC | 10:51 | |
| openstackgerrit | Maël Kimmerlin proposed openstack/ironic master: Redfish: Add root_prefix to Sushy https://review.opendev.org/707342 | 10:53 |
| *** mkrai_ has quit IRC | 10:54 | |
| *** Lucas_Gray has joined #openstack-ironic | 10:56 | |
| *** Lucas_Gray has quit IRC | 10:58 | |
| dtantsur | rpittau: hey, an easy stable review for your stats: https://review.opendev.org/#/c/707369/ :) | 10:59 |
| patchbot | patch 707369 - ironic-inspector (stable/train) - Replace references to the deprecated ironic-agent ... - 1 patch set | 10:59 |
| rpittau | mmm weird I didn't notice that one | 11:00 |
| *** Lucas_Gray has joined #openstack-ironic | 11:20 | |
| *** Lucas_Gray has quit IRC | 11:20 | |
| *** k_mouza has quit IRC | 11:24 | |
| *** Lucas_Gray has joined #openstack-ironic | 11:24 | |
| *** k_mouza has joined #openstack-ironic | 11:25 | |
| *** Lucas_Gray has quit IRC | 11:33 | |
| *** Lucas_Gray has joined #openstack-ironic | 11:34 | |
| *** Lucas_Gray has quit IRC | 11:41 | |
| *** mkrai_ has joined #openstack-ironic | 11:46 | |
| *** Lucas_Gray has joined #openstack-ironic | 11:51 | |
| *** amoralej is now known as amoralej|lunch | 11:59 | |
| *** rpittau is now known as rpittau|bbl | 12:07 | |
| *** tkajinam has joined #openstack-ironic | 12:23 | |
| *** Wryhder has joined #openstack-ironic | 12:39 | |
| *** mkrai_ has quit IRC | 12:41 | |
| *** Lucas_Gray has quit IRC | 12:41 | |
| *** Wryhder is now known as Lucas_Gray | 12:41 | |
| *** mkrai_ has joined #openstack-ironic | 12:44 | |
| *** lucasagomes has quit IRC | 12:51 | |
| openstackgerrit | Ankit Kumar proposed openstack/ironic-tempest-plugin master: Test case for iLO driver in standalone ironic https://review.opendev.org/708379 | 12:54 |
| *** lucasagomes has joined #openstack-ironic | 12:55 | |
| *** rh-jelabarre has joined #openstack-ironic | 13:02 | |
| *** rpittau|bbl is now known as rpittau | 13:06 | |
| *** amoralej|lunch is now known as amoralej | 13:08 | |
| *** mkrai_ has quit IRC | 13:11 | |
| *** jdandrea has joined #openstack-ironic | 13:27 | |
| *** jdandrea has quit IRC | 13:27 | |
| *** jdandrea has joined #openstack-ironic | 13:28 | |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: [DNM] getting some libvirt info https://review.opendev.org/707162 | 13:31 |
| openstackgerrit | Merged openstack/ironic-inspector stable/train: Replace references to the deprecated ironic-agent element https://review.opendev.org/707369 | 13:43 |
| *** mbeierl has quit IRC | 13:45 | |
| *** Goneri has joined #openstack-ironic | 13:46 | |
| *** mbeierl has joined #openstack-ironic | 13:47 | |
| *** mbeierl has quit IRC | 13:52 | |
| iurygregory | rpittau, I've found a option to use =D --access-logfile, 192.168.0.17 - - [18/Feb/2020:13:58:42 +0000] "GET /metrics HTTP/1.1" 200 1563 "-" "curl/7.58.0" | 13:59 |
| iurygregory | but it won't to to the screen of the service .-. | 14:00 |
| rpittau | iurygregory: if it logs to the file is still better than nothing :) | 14:00 |
| iurygregory | rpittau, yeah XD | 14:01 |
| iurygregory | going to update the patch | 14:01 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Use FIPS-compatible SHA256 for comparing files https://review.opendev.org/708388 | 14:01 |
| dtantsur | this is pretty stupid, but anyway ^^^ | 14:01 |
| iurygregory | dtantsur, stupid but necessary right? | 14:02 |
| dtantsur | seems so | 14:02 |
| dtantsur | if anybody could tell me whether SHA256 is actually compatible with FIPS... | 14:03 |
| TheJulia | good morning | 14:03 |
| dtantsur | maybe TheJulia can! good morning :) | 14:03 |
| TheJulia | dtantsur: looking it up | 14:03 |
| iurygregory | good morning TheJulia | 14:04 |
| *** igordc has joined #openstack-ironic | 14:04 | |
| *** jdandrea has quit IRC | 14:04 | |
| dtantsur | thanks! I've seen a lot of discussions but no references | 14:04 |
| TheJulia | dtantsur: issue primarily is that it is not algorithms, but implementations of algorithms that get stamped with approval | 14:05 |
| dtantsur | ugh | 14:05 |
| dtantsur | I'd rather opt out of this check, but it's only in python 3.9.. | 14:05 |
| *** jdandrea has joined #openstack-ironic | 14:05 | |
| TheJulia | sha3-256 is algorithm implementations are good | 14:06 |
| TheJulia | according to the FIPS 140-2 Annex A document | 14:06 |
| dtantsur | from the discussions (e.g. in https://bugzilla.redhat.com/show_bug.cgi?id=1731424) it seems that the python's sha256 is fine | 14:07 |
| openstack | bugzilla.redhat.com bug 1731424 in python3 "hmac.HMAC() is not FIPS compliant" [Unspecified,Closed: errata] - Assigned to torsava | 14:07 |
| rpittau | I was going to say, it really depends on the implementation, as the algorithm is compliant in general | 14:07 |
| TheJulia | yeah, sha-3 is sha-256 and sha-512 | 14:08 |
| rpittau | so that should be fine | 14:08 |
| dtantsur | okay, so https://review.opendev.org/#/c/708388/ is an easy win (dunno if it's the only issue we have, but this prevents even start-up) | 14:08 |
| patchbot | patch 708388 - ironic - Use FIPS-compatible SHA256 for comparing files - 1 patch set | 14:08 |
| * TheJulia has not had enough coffee to do anything but spittake | 14:09 | |
| openstackgerrit | Maël Kimmerlin proposed openstack/ironic master: Redfish: Add root_prefix to Sushy https://review.opendev.org/707342 | 14:11 |
| rpittau | dtantsur: before approving, wondering if a story could be useful ? | 14:11 |
| TheJulia | dtantsur: can we parameterize it from configuration because this is one of those breaking things that people need to be able to... step back from | 14:11 |
| dtantsur | TheJulia: this is an internal routine to check if a file has changed | 14:11 |
| dtantsur | I don't see how it could be exposed to a user | 14:11 |
| TheJulia | no comparison to image_checksum? | 14:12 |
| dtantsur | this seems only used when writing boot.ipxe | 14:12 |
| dtantsur | honestly, I'm not even sure why we do it instead of just writing the file... | 14:12 |
| TheJulia | okay then! | 14:12 |
| dtantsur | we probably waste more resources by reading the file | 14:12 |
| TheJulia | yeah | 14:12 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: DevStack support and Redfish job https://review.opendev.org/685888 | 14:13 |
| dtantsur | mmm, maybe because writing is not atomic? | 14:13 |
| dtantsur | and doing the usual write-then-hardlink dance is a bit annoying | 14:14 |
| TheJulia | yeah, that change is safe, just utils for pxe | 14:14 |
| openstackgerrit | Merged openstack/ironic master: Automatic port allocation for the serial console https://review.opendev.org/706445 | 14:14 |
| dtantsur | I can file a story if needed | 14:14 |
| TheJulia | yeah, that whole dance is a headache | 14:14 |
| TheJulia | I don't think it is needed, but we likely still need to provide an overall fips complaint mode ? | 14:15 |
| TheJulia | this will likely break bifrost, fwiw | 14:15 |
| dtantsur | baby steps. this is an obvious thing to fix, I'm not sure what else may come up. | 14:15 |
| TheJulia | yeah | 14:18 |
| TheJulia | should be a goal for next cycle | 14:18 |
| TheJulia | or if someone just wants to toss patches up, I'll review them | 14:18 |
| TheJulia | arne_wiebalck: thanks for the quick review on the ntp patch. I feel like maybe two more patches are needed so it can be made conductor configurable | 14:23 |
| *** tzumainn has joined #openstack-ironic | 14:23 | |
| arne_wiebalck | TheJulia: yw, what will be configurable? | 14:24 |
| TheJulia | I think the server address... | 14:24 |
| TheJulia | I think | 14:24 |
| TheJulia | it could be done from the pxe command line though | 14:24 |
| TheJulia | that may just be best, maybe | 14:25 |
| arne_wiebalck | let me check our code, but I guess we rely on whatever the ramdisk image uses ... | 14:27 |
| TheJulia | ++ whatever makes it easier to leverage | 14:27 |
| *** rloo has joined #openstack-ironic | 14:28 | |
| arne_wiebalck | we currently have this functionality in our h/w manager where we hardcode our NTP servers | 14:28 |
| TheJulia | so pretty much anything at that point is going to work for adopting it | 14:29 |
| dtantsur | TheJulia: do we have an RFE for this NTP work? | 14:29 |
| TheJulia | dtantsur: Non-specific rfe filed ages ago, where the author was told to write an entire spec and it died somewhere after that. I marked it rfe-approved because the idea/need of something is what is needed as opposed to something with exhaustively painful design review | 14:31 |
| dtantsur | TheJulia: I guess my question is about the specific need (not that I object to the idea..) | 14:32 |
| TheJulia | dtantsur: beyond with us using crypto and possibly clocks being off, turns out deployed OSes and things like rhcos booting needing to use short term certs with etcd need to have time relatively close | 14:33 |
| dtantsur | TheJulia: well, we cannot affect the deployed OS, it's only about IPA, no? | 14:33 |
| TheJulia | dtantsur: but we CAN sync to the hardware clock | 14:33 |
| dtantsur | ah | 14:34 |
| TheJulia | and that is a feature of the patch that I put up | 14:34 |
| dtantsur | that's a good idea | 14:34 |
| dtantsur | I wonder if we should make it a deploy step though | 14:34 |
| dtantsur | (which will force us to finish in-band deploy steps as a neat side effect) | 14:34 |
| TheJulia | so, I _did_ break out the IPA code to eventually be callable throught that method | 14:34 |
| arne_wiebalck | we have this since a while as nodes reached us with h/w clocks that were 2 hours off which prevented the regitration with some of our internal systems | 14:35 |
| dtantsur | I guess I should review before commenting, who could imagine? :) | 14:35 |
| TheJulia | yeah, when off systems can drift like crazy | 14:35 |
| dtantsur | doing with deploy steps will prevent questions like "how do we configure it" | 14:35 |
| dtantsur | you just put NTP servers in your deploy template | 14:35 |
| * rpittau still has nightmares because of puppet servers not synced | 14:35 | |
| TheJulia | dtantsur: but that is also over designing and my intent was minimal configuration required as a start, we can always it4erate | 14:35 |
| TheJulia | go review it :) | 14:36 |
| dtantsur | TheJulia: how do you do it without knowing NTP servers? use predefined? | 14:36 |
| TheJulia | predefined in config or on kernel command line right now | 14:36 |
| TheJulia | trying to keep it simple | 14:36 |
| dtantsur | I think that's the point we should start -1 things that must be deploy steps but we're too lazy to finish deploy steps | 14:36 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: DevStack support and Redfish job https://review.opendev.org/685888 | 14:36 |
| dtantsur | sorry | 14:36 |
| *** cdearborn has joined #openstack-ironic | 14:36 | |
| TheJulia | Seriously? | 14:37 |
| dtantsur | otherwise we can scrap the idea of deploy steps and just hardcode everything in a similar fashion | 14:37 |
| dtantsur | (which is what many operators are doing downstream apparently) | 14:37 |
| arne_wiebalck | it is probably sensible to not require operators to use deploy steps for sth relatively basic such as getting the correct time | 14:38 |
| TheJulia | Deploy steps was intended for more flexible deployment operations, not for forcing operators to articulate everything to facilitate a deployment with a common-sense action | 14:38 |
| dtantsur | deploy steps are not a crazily complex advanced feature. they ARE the way to provide non-standard input into deployment. | 14:38 |
| dtantsur | note that providing NTP servers via a kernel command line is not entirely simpler | 14:39 |
| TheJulia | it is flexible | 14:39 |
| dtantsur | at the very least it requires changing a non-trivial configuration option in ironic.conf | 14:39 |
| dtantsur | and that's only for PXE. can someone here name all options for all boot interfaces? | 14:39 |
| TheJulia | I guess if we're really going to force this, we won't have this functionality for... 6 months? | 14:39 |
| dtantsur | or we finish the deploy steps asap and have it in ussuri | 14:40 |
| arne_wiebalck | FWIU deploy steps require traits which do not work at scale atm (which is why we have them completely disabled) | 14:40 |
| dtantsur | don't we have default deploy steps? | 14:41 |
| * arne_wiebalck is not against finishing deploy steps | 14:41 | |
| TheJulia | and allowing non-infrastructure operators to make clock syncing decisions is not a great idea | 14:41 |
| dtantsur | (seriously asking, I don't remember) | 14:41 |
| TheJulia | the default is supposed to be auto-determined | 14:41 |
| dtantsur | well, we allow them to change firmware :) | 14:42 |
| TheJulia | so we would in essence add a decorator | 14:42 |
| TheJulia | to the ntp patch | 14:42 |
| *** zigo has quit IRC | 14:42 | |
| * TheJulia guesses it is time to go update internal jira to change a 3 pt story to a 21 point story | 14:43 | |
| dtantsur | TheJulia: will all the sympathy, I don't think the downstream planning concerns should be the driving force here | 14:44 |
| dtantsur | before this comment I was about to agree with a non-default step approach | 14:44 |
| * iurygregory "from 3 to 21 HOLY ****" | 14:45 | |
| dtantsur | now I'm afraid to set a precedent | 14:45 |
| dtantsur | s/default/deploy/ | 14:45 |
| TheJulia | dtantsur: more so about communicating increase in scope visibility | 14:46 |
| dtantsur | I haven't even insisted (yet), just listening for the arguments | 14:47 |
| TheJulia | we go from something relatively small to joining it in with a bunch of other work, that is going to elongate the first piece of work a substantial amount of time | 14:47 |
| dtantsur | since this RFE has never (?) been discussed in a wider circle | 14:47 |
| dtantsur | TheJulia: similar argument can be applied to other features hold up by deploy steps: deploy-time RAID, partitioning, etc | 14:47 |
| TheJulia | last time it was discussed in IRC, I distinctly remember people saying "that is not ironic's problem, people should do custom hardware managers" which it turns out few actually do, and general usage is needed | 14:47 |
| * TheJulia wishes to avoid generalized partitioning | 14:48 | |
| dtantsur | I wish it to, I'm not sure we can long-term | 14:48 |
| dtantsur | a bit philosophical question that I have now is where the border between deploy/clean steps and built-in features is | 14:49 |
| TheJulia | I believe the consensus when the topic was last visited was that there really was no border except in name only | 14:50 |
| TheJulia | being that removing explicit naming meant possibly creating confusion over when, and i believe the spec we end up having to point a more "when does this apply, cleaning or deploy" field which I don't think was popular when that spec was being reviewed early on | 14:50 |
| dtantsur | TheJulia: I think in this case the border lies in whether the operation is required for IPA functioning itself. It seems to, so I guess I agree with the start-up time approach. | 14:51 |
| *** Lucas_Gray has quit IRC | 14:51 | |
| dtantsur | TheJulia: what if we split your IPA patch into two: start-up NTP sync (not controversial from my position) and the on-demand one (that should be a deploy steps IMO). | 14:51 |
| dtantsur | Will it solve your immediate problem? | 14:51 |
| TheJulia | dtantsur: uhh... why not just have the step there as is in the existing patch, once we patch in we just add the decorator and remove the explicit call if available upon poweroff in the code? | 14:52 |
| dtantsur | what does it win for us? it seems that we're going to have two changes of the approach within months | 14:53 |
| dtantsur | AFAIR deploy steps are not implemented via IPA commands. so you'll have to remove the command (with compatibility if we cross the release border) | 14:54 |
| TheJulia | I don't think that is correct | 14:54 |
| *** Lucas_Gray has joined #openstack-ironic | 14:55 | |
| TheJulia | I'm looking for the wip patches now | 14:56 |
| dtantsur | if we go down the kernel arguments path, we may need to consolidate various options that modify it | 14:56 |
| dtantsur | so that people don't have to remember to use another option if they use redfish-virtual-media.. | 14:57 |
| TheJulia | ipa: https://review.opendev.org/698770 <-- looks like it just returns the step data across the extension via a command to the conductor | 14:57 |
| patchbot | patch 698770 - ironic-python-agent - WIP: In-band deploy steps - 1 patch set | 14:57 |
| *** mkrai has joined #openstack-ironic | 14:58 | |
| dtantsur | yep, similar to clean steps | 14:58 |
| TheJulia | yup, which means the command decorator would stil be used | 14:59 |
| dtantsur | so an IPA extension call won't immediately become a deploy step | 14:59 |
| TheJulia | well, step decorator | 14:59 |
| TheJulia | https://review.opendev.org/#/c/698309/4/ironic/drivers/modules/agent.py | 14:59 |
| patchbot | patch 698309 - ironic - WIP: Decompose core deploy step - 4 patch sets | 14:59 |
| TheJulia | no, it won't. Looks like there would still be even more work to get there properly, and we still have the issue with steps where we can't collect/condense until the agent is online | 14:59 |
| dtantsur | I think we're taking the same approach as with cleaning: first boot IPA, then see | 15:00 |
| * dtantsur needs to refresh his memory of these patches | 15:00 | |
| TheJulia | yeah, failures there shouldn't conflict | 15:00 |
| TheJulia | err | 15:00 |
| *** igordc has quit IRC | 15:00 | |
| TheJulia | cases there shouldn't, while there will likely be merge failures and rebases becaused lean step code needs to be revisited to be aware of "if I actually need to power the machine on first" | 15:01 |
| TheJulia | What as decided in shanghai, seemed simple enough | 15:01 |
| *** Goneri has quit IRC | 15:03 | |
| TheJulia | gah, still a hard coded list for the agent :( | 15:04 |
| TheJulia | so I feel like we're conflating feature usefulness with each other, the vast majority of the code wouldn't change once deploy steps is in place, testing would be slightly different in terms of opportunistic behavior would be abel to be split away from shutdown/poweroff | 15:05 |
| * dtantsur has commented on the patch | 15:06 | |
| TheJulia | okay | 15:08 |
| *** Goneri has joined #openstack-ironic | 15:11 | |
| dtantsur | mgoddard: hey, I seem to recall you're not around this week? I wonder if I can give the deploy step patches a round of updates. | 15:14 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [DNM] test bionic job https://review.opendev.org/708413 | 15:23 |
| openstackgerrit | Riccardo Pittau proposed openstack/sushy-cli master: Update commands to generate docs, renos and pdf https://review.opendev.org/706826 | 15:29 |
| *** Lucas_Gray has quit IRC | 15:41 | |
| *** stendulker has joined #openstack-ironic | 15:42 | |
| *** Lucas_Gray has joined #openstack-ironic | 15:43 | |
| *** tesseract has quit IRC | 15:55 | |
| *** TxGirlGeek has joined #openstack-ironic | 15:57 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: DevStack support and Redfish job https://review.opendev.org/685888 | 15:57 |
| * rpittau needs to split | 16:06 | |
| rpittau | bye all, see you tomorro! o/ | 16:06 |
| *** rpittau is now known as rpittau|afk | 16:06 | |
| *** zigo has joined #openstack-ironic | 16:08 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Generalize clean step functions to support deploy steps https://review.opendev.org/698768 | 16:15 |
| *** stendulker has quit IRC | 16:21 | |
| *** Lucas_Gray has quit IRC | 16:29 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Generalize clean step functions to support deploy steps https://review.opendev.org/698768 | 16:34 |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Add ntpdate/ntpclient/chrony https://review.opendev.org/707913 | 16:40 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Generalize clean step functions to support deploy steps https://review.opendev.org/698768 | 16:40 |
| openstackgerrit | Sean McGinnis proposed openstack/sushy-cli master: Fix gitreview host https://review.opendev.org/708448 | 16:49 |
| dtantsur | yay, we screwed up the sushy-cli release :D | 16:52 |
| gudrutis2 | dtantsur: hi, what is prefered way to report bug for ironic-python-agent-builder ? | 16:57 |
| dtantsur | gudrutis2: https://storyboard.openstack.org/ probably with a heads-up here | 16:57 |
| *** gyee has joined #openstack-ironic | 17:00 | |
| gudrutis2 | dtantsur: ok. Anyway, are you familiar that IPAB ignores `ELEMENTS_PATH` ? In the documentation, it states I can use it, but it is being ignored. And I see that its because this line overrides it https://opendev.org/openstack/ironic-python-agent-builder/src/branch/master/ironic_python_agent_builder/__init__.py#L59 | 17:00 |
| dtantsur | mmm, right, it's not ideal | 17:01 |
| dtantsur | gudrutis2: I can try to get to it today or tomorrow, or you're very welcome to propose a patch | 17:01 |
| *** mkrai has quit IRC | 17:03 | |
| dtantsur | (tomorrow, since I need to go now) | 17:06 |
| *** dtantsur is now known as dtantsur|afk | 17:06 | |
| dtantsur|afk | o/ | 17:06 |
| *** igordc has joined #openstack-ironic | 17:09 | |
| *** lucasagomes has quit IRC | 17:13 | |
| openstackgerrit | Merged openstack/sushy-cli master: Fix gitreview host https://review.opendev.org/708448 | 17:27 |
| *** diurnalist has joined #openstack-ironic | 17:32 | |
| *** k_mouza has quit IRC | 17:32 | |
| arne_wiebalck | bye everyone o? | 17:35 |
| * arne_wiebalck thinks this looks like me waving with a hook | 17:35 | |
| arne_wiebalck | o/ | 17:35 |
| iurygregory | bye arne_wiebalck o/ | 17:42 |
| iurygregory | YAY https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_fbd/685888/34/check/ironic-prometheus-exporter-redfish/fbdf0e1/controller/logs/ironic-bm-logs/ipe_access_log.txt it's working | 17:43 |
| *** diurnalist has quit IRC | 17:47 | |
| *** TxGirlGeek has quit IRC | 17:59 | |
| TheJulia | etingof: Out of curiosity, do you know if anything has or is going to be submitted for the next outreachy round? | 18:00 |
| *** dougsz has quit IRC | 18:01 | |
| etingof | TheJulia, I am not aware if anyone is going to submit anything, but I think we should do that before it's too late (25.02) | 18:04 |
| *** derekh has quit IRC | 18:04 | |
| * etingof always has a bowl of fishy stuff to pick from | 18:05 | |
| etingof | TheJulia, anything specific you have in mind? | 18:05 |
| TheJulia | Well, arne_wiebalck has a thought about inspector time series data rfe, so I'm wondering if some collaboration makes sense | 18:07 |
| etingof | do you have a url handy? | 18:08 |
| etingof | or is it all above ^? | 18:08 |
| TheJulia | https://storyboard.openstack.org/#!/story/2007246 I think he might want to submit it | 18:10 |
| etingof | the submission url is this -- https://www.outreachy.org/outreachy-may-2020-internship-round/communities/openstack/submit-project/ | 18:12 |
| *** diurnalist has joined #openstack-ironic | 18:14 | |
| *** diurnalist has quit IRC | 18:19 | |
| *** jdandrea has quit IRC | 18:27 | |
| *** jdandrea has joined #openstack-ironic | 18:28 | |
| *** priteau has quit IRC | 18:39 | |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: Fix hanging on shell-pipe https://review.opendev.org/708483 | 18:40 |
| *** amoralej is now known as amoralej|off | 18:41 | |
| *** diurnalist has joined #openstack-ironic | 18:48 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Catch ValueError for FIPS 140-2 mode https://review.opendev.org/708492 | 18:52 |
| TheJulia | dtantsur|afk: re fips: https://storyboard.openstack.org/#!/story/2007306 as a general purpose fix items story. IPA was easy, it is just a catch | 18:53 |
| *** TxGirlGeek has joined #openstack-ironic | 18:53 | |
| arne_wiebalck | Thanks TheJulia & etingof. I checked with dtantsur|afk earlier this week and understand he thinks the overall feature is neat, so I suggest we submit the proposal now (and discuss details next week). Sounds ok? | 18:54 |
| etingof | arne_wiebalck++ | 18:54 |
| TheJulia | sounds good to me | 18:54 |
| arne_wiebalck | Great, thanks! See you tmrw o/ | 18:55 |
| *** k_mouza has joined #openstack-ironic | 18:56 | |
| *** k_mouza has quit IRC | 18:57 | |
| *** k_mouza has joined #openstack-ironic | 18:57 | |
| *** TxGirlGeek has quit IRC | 18:58 | |
| *** k_mouza has quit IRC | 19:02 | |
| *** diurnalist has quit IRC | 19:10 | |
| *** TxGirlGeek has joined #openstack-ironic | 19:12 | |
| openstackgerrit | khansa proposed openstack/sushy-tools master: Change logger call https://review.opendev.org/708495 | 19:22 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Remove the SW RAID configuration https://review.opendev.org/706961 | 19:32 |
| *** ijw has joined #openstack-ironic | 19:37 | |
| *** ijw has quit IRC | 19:42 | |
| *** ijw has joined #openstack-ironic | 19:45 | |
| *** Goneri has quit IRC | 19:58 | |
| *** TxGirlGeek has quit IRC | 20:02 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Handle FIPS 140-2 failure with force_raw https://review.opendev.org/708499 | 20:02 |
| * etingof is getting better at cooking than at coding | 20:05 | |
| TheJulia | dtantsur|afk: fyi ^^^ It likely needs some tests, but it should allow the conductor to gracefully handle fips 140-2 node with force_raw. I think we should likely, for next cycle, consider performing an actual code audit and implement a proper FIPS 140-2 compliance mode flag where we will reject use of the checksum field | 20:06 |
| TheJulia | etingof: I bombed at both pizza and pumpkin pie last night | 20:06 |
| TheJulia | I suspect programming is getting better at my cooking skills | 20:06 |
| TheJulia | Then again! I have been trying new things | 20:07 |
| TheJulia | dtantsur|afk: the only other thing that immediately stands out is the StringFieldThatAcceptsCallable class in objects/fields.py but that is only used in testing | 20:11 |
| etingof | zero sum of cooking and programming talents | 20:12 |
| etingof | we got network_data.json schema merged to nova lately | 20:14 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: tgtd session detach test fails on Centos7 https://review.opendev.org/686588 | 20:15 |
| *** TxGirlGeek has joined #openstack-ironic | 20:17 | |
| *** TxGirlGeek has quit IRC | 20:24 | |
| *** TxGirlGeek has joined #openstack-ironic | 20:26 | |
| *** TxGirlGeek has quit IRC | 20:27 | |
| *** TxGirlGeek has joined #openstack-ironic | 20:32 | |
| *** TxGirlGeek has quit IRC | 20:32 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Use FIPS-compatible SHA256 for comparing files https://review.opendev.org/708388 | 20:37 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Project Contributing updates for Goal https://review.opendev.org/707930 | 20:57 |
| *** k_mouza has joined #openstack-ironic | 20:58 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Hash the rescue_password https://review.opendev.org/691413 | 21:01 |
| *** jdandrea has quit IRC | 21:02 | |
| *** k_mouza has quit IRC | 21:03 | |
| *** TxGirlGeek has joined #openstack-ironic | 21:03 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local https://review.opendev.org/492114 | 21:05 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local https://review.opendev.org/492114 | 21:05 |
| *** mbeierl has joined #openstack-ironic | 21:05 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Rescan devices before performing uefi checks https://review.opendev.org/706667 | 21:08 |
| *** Goneri has joined #openstack-ironic | 21:08 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector master: Change CoreOS to Centos for IPA https://review.opendev.org/707174 | 21:17 |
| TheJulia | etingof: \o/ | 21:23 |
| *** TxGirlGeek has quit IRC | 21:27 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Project Contributing updates for Goal https://review.opendev.org/707930 | 21:27 |
| *** jdandrea has joined #openstack-ironic | 21:43 | |
| *** rh-jelabarre has quit IRC | 22:14 | |
| *** rcernin has joined #openstack-ironic | 22:15 | |
| *** k_mouza has joined #openstack-ironic | 22:16 | |
| *** k_mouza has quit IRC | 22:19 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Pre-shared agent token https://review.opendev.org/692609 | 22:38 |
| *** rcernin has quit IRC | 22:49 | |
| *** rcernin has joined #openstack-ironic | 22:50 | |
| *** Goneri has quit IRC | 22:54 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Enable agent_token for virtual media boot https://review.opendev.org/697584 | 22:57 |
| *** iurygregory has quit IRC | 23:21 | |
| *** TxGirlGeek has joined #openstack-ironic | 23:36 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Add --no-retired list option https://review.opendev.org/707196 | 23:49 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!