| openstackgerrit | Steve Baker proposed openstack/ironic master: Align the indicators API with the spec https://review.opendev.org/c/openstack/ironic/+/776540 | 01:04 |
|---|---|---|
| *** rcernin has quit IRC | 01:29 | |
| *** zzzeek has quit IRC | 01:30 | |
| *** zzzeek has joined #openstack-ironic | 01:34 | |
| *** rcernin has joined #openstack-ironic | 01:35 | |
| *** tzumainn has quit IRC | 02:08 | |
| *** mkrai has joined #openstack-ironic | 02:15 | |
| *** rcernin has quit IRC | 02:38 | |
| *** rcernin has joined #openstack-ironic | 02:39 | |
| *** rcernin has quit IRC | 02:50 | |
| openstackgerrit | Steve Baker proposed openstack/python-ironicclient master: Implement client commands for node indicators https://review.opendev.org/c/openstack/python-ironicclient/+/776569 | 02:51 |
| openstackgerrit | Steve Baker proposed openstack/python-ironicclient master: Implement client commands for node indicators https://review.opendev.org/c/openstack/python-ironicclient/+/776569 | 03:03 |
| *** rcernin has joined #openstack-ironic | 03:03 | |
| *** rcernin has quit IRC | 03:17 | |
| *** rcernin has joined #openstack-ironic | 03:26 | |
| *** rcernin has quit IRC | 03:37 | |
| *** ricolin_ has joined #openstack-ironic | 03:39 | |
| *** rcernin has joined #openstack-ironic | 03:52 | |
| *** TheJulia has quit IRC | 04:01 | |
| *** buhman has quit IRC | 04:02 | |
| *** vdrok has quit IRC | 04:02 | |
| *** vdrok has joined #openstack-ironic | 04:03 | |
| *** buhman has joined #openstack-ironic | 04:03 | |
| *** TheJulia has joined #openstack-ironic | 04:03 | |
| *** paras333 has joined #openstack-ironic | 04:16 | |
| *** paras333 has quit IRC | 04:33 | |
| *** ricolin_ has quit IRC | 04:56 | |
| *** ricolin_ has joined #openstack-ironic | 04:58 | |
| *** gyee has quit IRC | 05:23 | |
| *** mkrai has quit IRC | 05:32 | |
| *** mkrai has joined #openstack-ironic | 05:32 | |
| *** benj_ has quit IRC | 06:12 | |
| *** benj_ has joined #openstack-ironic | 06:19 | |
| *** mkrai has quit IRC | 06:23 | |
| *** iurygregory_ has joined #openstack-ironic | 06:26 | |
| *** iurygregory has quit IRC | 06:27 | |
| *** ricolin_ has quit IRC | 06:52 | |
| *** moshiur has joined #openstack-ironic | 07:06 | |
| *** uzumaki has joined #openstack-ironic | 07:14 | |
| arne_wiebalck | Good morning, ironic! | 07:22 |
| *** rcernin has quit IRC | 07:29 | |
| openstackgerrit | Merged openstack/ironic master: Validate configdrive string format https://review.opendev.org/c/openstack/ironic/+/775922 | 07:51 |
| *** rpittau|afk is now known as rpittau | 08:06 | |
| rpittau | good morning ironic! Happy Friday! o/ | 08:06 |
| *** mkowalski has quit IRC | 08:26 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Use tinycore 12 to build tinyipa https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776587 | 08:29 |
| *** ociuhandu has joined #openstack-ironic | 08:31 | |
| *** tosky has joined #openstack-ironic | 08:40 | |
| *** ociuhandu has quit IRC | 08:41 | |
| *** ociuhandu has joined #openstack-ironic | 08:45 | |
| *** mgoddard has quit IRC | 08:48 | |
| *** ociuhandu has quit IRC | 08:50 | |
| *** dougsz has joined #openstack-ironic | 08:56 | |
| *** ociuhandu has joined #openstack-ironic | 09:08 | |
| *** ociuhandu has quit IRC | 09:21 | |
| *** ociuhandu has joined #openstack-ironic | 09:22 | |
| *** mkowalski has joined #openstack-ironic | 09:23 | |
| *** ociuhandu has quit IRC | 09:27 | |
| *** uzumaki has quit IRC | 09:32 | |
| *** ociuhandu has joined #openstack-ironic | 09:35 | |
| *** dtantsur|afk is now known as dtantsur | 09:42 | |
| dtantsur | morning ironic! Finally end of week, yeah? | 09:42 |
| dtantsur | could I get another +2 on https://review.opendev.org/c/openstack/ironic/+/776505 please? | 09:44 |
| *** derekh has joined #openstack-ironic | 09:45 | |
| openstackgerrit | Merged openstack/bifrost master: More compact README.rst https://review.opendev.org/c/openstack/bifrost/+/776107 | 09:54 |
| dtantsur | SPUC anyone? janders, rpittau, arne_wiebalck, ajya? | 10:01 |
| rpittau | dtantsur: I will join in a couple of minutes | 10:01 |
| *** k_mouza has joined #openstack-ironic | 10:01 | |
| openstackgerrit | Aija Jauntēva proposed openstack/ironic master: Add Redfish RAID management to Ironic https://review.opendev.org/c/openstack/ironic/+/776596 | 10:12 |
| *** uzumaki has joined #openstack-ironic | 10:12 | |
| openstackgerrit | Aija Jauntēva proposed openstack/ironic master: [WIP] Add Redfish RAID interface to idrac HW type https://review.opendev.org/c/openstack/ironic/+/774433 | 10:23 |
| *** dtantsur is now known as dtantsur|brb | 10:55 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_node https://review.opendev.org/c/openstack/ironic/+/776479 | 10:59 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_notification_utils module https://review.opendev.org/c/openstack/ironic/+/776603 | 10:59 |
| openstackgerrit | Takashi Kajinami proposed openstack/ironic-ui master: Use permissions attribute to detect ironic service https://review.opendev.org/c/openstack/ironic-ui/+/776486 | 11:12 |
| *** ociuhandu has quit IRC | 11:17 | |
| *** ociuhandu has joined #openstack-ironic | 11:18 | |
| *** k_mouza has quit IRC | 11:20 | |
| *** k_mouza has joined #openstack-ironic | 11:20 | |
| openstackgerrit | Aija Jauntēva proposed openstack/ironic master: Add Redfish RAID management to Ironic https://review.opendev.org/c/openstack/ironic/+/730000 | 11:21 |
| *** ociuhandu has quit IRC | 11:23 | |
| *** tkajinam has quit IRC | 11:25 | |
| openstackgerrit | Merged openstack/ironic master: Correct release mappings for 16.2 https://review.opendev.org/c/openstack/ironic/+/776505 | 11:42 |
| *** tosin has joined #openstack-ironic | 11:43 | |
| iurygregory_ | good morning Ironic | 11:46 |
| *** NobodyCam has quit IRC | 12:08 | |
| *** rpittau has quit IRC | 12:08 | |
| *** johnsom has quit IRC | 12:08 | |
| *** buhman has quit IRC | 12:08 | |
| *** vdrok has quit IRC | 12:08 | |
| *** TheJulia has quit IRC | 12:08 | |
| *** k_mouza_ has joined #openstack-ironic | 12:11 | |
| *** k_mouza_ has quit IRC | 12:12 | |
| *** k_mouza_ has joined #openstack-ironic | 12:12 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Add support for using NVMe specific cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 | 12:13 |
| *** k_mouza has quit IRC | 12:15 | |
| *** vdrok has joined #openstack-ironic | 12:18 | |
| *** NobodyCam has joined #openstack-ironic | 12:18 | |
| *** TheJulia has joined #openstack-ironic | 12:19 | |
| *** rpittau has joined #openstack-ironic | 12:21 | |
| *** johnsom has joined #openstack-ironic | 12:23 | |
| *** buhman has joined #openstack-ironic | 12:26 | |
| *** dtantsur|brb is now known as dtantsur | 12:29 | |
| *** mgoddard has joined #openstack-ironic | 12:30 | |
| *** sshnaidm is now known as sshnaidm|off | 12:36 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic bugfix/16.2: Correct release mappings for 16.2 https://review.opendev.org/c/openstack/ironic/+/776616 | 12:39 |
| *** iurygregory_ is now known as iurygregory | 12:49 | |
| iurygregory | HPE CI has been reporting Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. =( | 12:49 |
| *** ociuhandu has joined #openstack-ironic | 12:55 | |
| arne_wiebalck | dtantsur: sorry, was in meetings all morning | 12:57 |
| *** knikolla has quit IRC | 13:06 | |
| *** ociuhandu has quit IRC | 13:06 | |
| *** ociuhandu has joined #openstack-ironic | 13:07 | |
| *** knikolla has joined #openstack-ironic | 13:07 | |
| *** ociuhandu has quit IRC | 13:11 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_node https://review.opendev.org/c/openstack/ironic/+/776479 | 13:16 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_notification_utils module https://review.opendev.org/c/openstack/ironic/+/776603 | 13:16 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Correct release mappings for 16.2 https://review.opendev.org/c/openstack/ironic/+/776616 | 13:19 |
| *** rcernin has joined #openstack-ironic | 13:26 | |
| *** k_mouza_ has quit IRC | 13:28 | |
| *** k_mouza has joined #openstack-ironic | 13:28 | |
| *** rcernin has quit IRC | 13:30 | |
| *** uzumaki has quit IRC | 13:38 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Correct release mappings for 16.2 https://review.opendev.org/c/openstack/ironic/+/776616 | 13:41 |
| *** bburns has quit IRC | 13:41 | |
| *** bburns has joined #openstack-ironic | 13:44 | |
| *** ociuhandu has joined #openstack-ironic | 13:44 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Use tinycore 12 to build tinyipa https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776587 | 13:46 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Use tinycore 12 to build tinyipa https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776587 | 13:47 |
| dtantsur | iurygregory: I suspect they have some downstream patches that no longer apply | 13:50 |
| iurygregory | dtantsur, gotcha | 13:51 |
| *** rloo has joined #openstack-ironic | 13:53 | |
| *** anuradha1904 has joined #openstack-ironic | 13:53 | |
| *** dougsz has quit IRC | 13:53 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Fix isolinux.bin path https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776642 | 13:55 |
| *** alexmcleod has quit IRC | 13:57 | |
| *** alexmcleod has joined #openstack-ironic | 13:58 | |
| rpittau | this "ERROR: Cannot find Ninja" should not be an error, Ninja is doing his job right | 13:59 |
| dtantsur | :D | 14:00 |
| dtantsur | rpittau: oh, may I volunteer you for additional work? | 14:00 |
| dtantsur | could you make sure this build-iso thing runs in the CI? | 14:00 |
| rpittau | dtantsur: about ninjas? :D | 14:01 |
| *** rcernin has joined #openstack-ironic | 14:01 | |
| rpittau | dtantsur: yeah, we should have that | 14:01 |
| rpittau | I'll get to that | 14:01 |
| dtantsur | the same patch or later? | 14:01 |
| rpittau | either is fine, I tested locally and it works, probably need to verify that all the dependencies are installed though | 14:03 |
| iurygregory | ninja++ | 14:05 |
| dtantsur | TIL about https://tools.ietf.org/html/rfc7807, I wish we have had it when ironic was started | 14:07 |
| *** ociuhandu has quit IRC | 14:07 | |
| dtantsur | rpittau: let's do in this patch so that we can see it working? | 14:08 |
| rpittau | dtantsur: sounds good | 14:08 |
| *** ociuhandu has joined #openstack-ironic | 14:08 | |
| openstackgerrit | Merged openstack/ironic bugfix/16.2: Correct release mappings for 16.2 https://review.opendev.org/c/openstack/ironic/+/776616 | 14:09 |
| openstackgerrit | Aija Jauntēva proposed openstack/ironic master: Add Redfish RAID management to Ironic https://review.opendev.org/c/openstack/ironic/+/730000 | 14:10 |
| *** ociuhandu has quit IRC | 14:12 | |
| TheJulia | good morning everyone | 14:16 |
| iurygregory | good morning TheJulia | 14:17 |
| arne_wiebalck | for auto-registration to set a nodename we will add the name to introspection data in our h/w manager (as we will read it from the FRU/BMC) and then extract it during the node_not_found hook so it becomes part of the enrollment API call ... sounds ok? | 14:22 |
| *** k_mouza_ has joined #openstack-ironic | 14:23 | |
| arne_wiebalck | I'd think the part in the hook to check if there is a field 'node_name' in the introspection data may be upstreamable. | 14:24 |
| *** ociuhandu has joined #openstack-ironic | 14:24 | |
| arne_wiebalck | I think we were discussing sth like this a few months ago ... with dking maybe? | 14:24 |
| TheJulia | node_name as in from dhcp or node_name in as to what the bmc thinks the chassis's name is? | 14:26 |
| *** k_mouza has quit IRC | 14:27 | |
| *** ociuhandu has quit IRC | 14:28 | |
| arne_wiebalck | the latter, this is before dhcp | 14:28 |
| *** ociuhandu has joined #openstack-ironic | 14:28 | |
| arne_wiebalck | sorry, before DNS | 14:29 |
| arne_wiebalck | in any case, it is from the BMC | 14:29 |
| arne_wiebalck | the node boots up for the first time in the data center | 14:30 |
| arne_wiebalck | at the moment this is all done with an in-house image, but I would like to move these early stages to Ironic | 14:31 |
| arne_wiebalck | and rather than have the node registered everywhere, then enrolled, then inspected, I would like to have the node inspected, enrolled, and then registered everywhere ... if that makes sense :) | 14:32 |
| *** uzumaki has joined #openstack-ironic | 14:32 | |
| *** rcernin has quit IRC | 14:35 | |
| dtantsur | morning TheJulia | 14:35 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Make sure we can create tinyipa isos https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776642 | 14:36 |
| rpittau | dtantsur: ^ I'm adding a new job for this, maybe a bit overkill ? | 14:37 |
| *** ociuhandu has quit IRC | 14:40 | |
| *** ociuhandu has joined #openstack-ironic | 14:40 | |
| *** tzumainn has joined #openstack-ironic | 14:41 | |
| dtantsur | rpittau: cannot we run it in the existing tinyipa-check job? | 14:42 |
| TheJulia | run it in existing jobs please | 14:46 |
| TheJulia | mastering the iso is not that much work one the image and assets are built | 14:46 |
| TheJulia | arne_wiebalck: makes sense, sounds a lot like what NobodyCam did | 14:48 |
| *** ociuhandu has quit IRC | 14:49 | |
| rpittau | dtantsur, TheJulia, yes, we can, I will just do that | 14:54 |
| dtantsur | (moving discussion from downstream slack) iurygregory and I have been discussing a way to run extended (= using BMC) validations for nodes | 14:55 |
| dtantsur | I'm pondering a new BaseInterface call that is invoked asynchronously after updating driver_info/driver/*_interface and updates last_error | 14:56 |
| dtantsur | which is.. very implicit :( | 14:56 |
| dtantsur | we can also run it on 'manage', which is fine for our downstream case, but won't handle later updates | 14:56 |
| dtantsur | thoughts welcome | 14:56 |
| TheJulia | I like the "sanity check/fix" | 14:58 |
| dtantsur | oh, use case: check that the Redfish BMC has virtual media support before we try to deploy/clean/inspect with it | 14:58 |
| TheJulia | because that gives us a place to shunt from generic redfish to idrac or ilo if there is an issue | 14:59 |
| TheJulia | "oh, this is never going to work, here use the vendor driver" | 14:59 |
| dtantsur | but what if it's just not present? | 14:59 |
| dtantsur | no virtual media support, no CD device.. | 14:59 |
| TheJulia | I think that would need to be a second level of checks | 14:59 |
| dtantsur | sure, I'm trying to image how they could look | 15:00 |
| TheJulia | for now, we know the higher level configurations that just won't work as people would expect them to | 15:00 |
| TheJulia | and it is not a matter of documentaiton, it is all about operator perception and vendor perception being on different pages | 15:00 |
| dtantsur | the driver fix-up is cool, but right now I'm curious about a different thing | 15:00 |
| dtantsur | (unless they can be done though the same change, which I doubt) | 15:01 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Make sure we can create tinyipa isos https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776642 | 15:01 |
| TheJulia | could we add interfaces to do basic checks on the management interface .can_do_uefi .can_do_virtual_media | 15:01 |
| TheJulia | would be past a basic validation, but we could also do things on validation because we've built this pattern for people to always try calling validation before deployment | 15:01 |
| TheJulia | (not sure if metal3 is doing that though) | 15:02 |
| TheJulia | ugh, went from 3 failing tests to 9 | 15:02 |
| dtantsur | we cannot call these from validation - it's synchronous | 15:02 |
| dtantsur | that's the whole conundrum | 15:03 |
| dtantsur | (otherwise it would be a non-brainer) | 15:03 |
| iurygregory | basically to check if redfish from the node has vmedia support we will be doing https://github.com/metal3-io/baremetal-operator/blob/b5d7e09c3115afc1e2e62063c31ad8009213c648/pkg/bmc/virtualmedia_validation.go | 15:03 |
| *** ociuhandu has joined #openstack-ironic | 15:03 | |
| iurygregory | on python code for Dell this is https://github.com/dell/iDRAC-Redfish-Scripting/blob/d3e674b05b7b699c4af185ebb9c4993a904e4128/Redfish%20Python/InsertEjectVirtualMediaREDFISH.py#L42-L55 | 15:03 |
| TheJulia | is the hanging api client the only reason to avoid synchronous? | 15:03 |
| dtantsur | this and occupying an API thread as a result | 15:04 |
| TheJulia | (I know there are other reasons, but realistically) | 15:04 |
| dtantsur | currently validation is basically zero-cost | 15:04 |
| TheJulia | it is not very restful to return data and then change things or fail things after the fact | 15:04 |
| TheJulia | basically yeah | 15:04 |
| dtantsur | we're not very restful at all :) | 15:04 |
| iurygregory | hehehe | 15:04 |
| TheJulia | heh | 15:05 |
| * TheJulia somehow typed poolicy instead of policy | 15:05 | |
| * TheJulia suspects a pool would be nice | 15:05 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Make sure we can create tinyipa isos https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776642 | 15:05 |
| openstackgerrit | Tosin Farai proposed openstack/bifrost master: Test function to set provision state https://review.opendev.org/c/openstack/bifrost/+/774081 | 15:05 |
| dtantsur | poo-licy, lol | 15:05 |
| iurygregory | it's a pool of policy =) | 15:05 |
| iurygregory | it makes sense | 15:05 |
| dtantsur | back on topic, validating on ->manage-> sounds like an easy win | 15:06 |
| dtantsur | it won't help people who change drivers on fly though | 15:06 |
| *** ociuhandu has quit IRC | 15:06 | |
| dtantsur | another possibility is a periodic task and setting node.fault | 15:06 |
| *** ociuhandu has joined #openstack-ironic | 15:07 | |
| dtantsur | (the two options are not mutually exclusive) | 15:07 |
| iurygregory | periodic task wouldn't be too much? | 15:07 |
| dtantsur | the downside of node.fault is that nobody cares about it | 15:07 |
| dtantsur | well.. scaling conductors is a known problem | 15:07 |
| *** moshiur has quit IRC | 15:07 | |
| rpittau | bye everyone! have a great weekend! o/ | 15:07 |
| *** rpittau is now known as rpittau|afk | 15:07 | |
| dtantsur | and we can use last_updated to check if there was a validation since the last update | 15:08 |
| iurygregory | I do think validation can happen when the node goes to manage... | 15:08 |
| iurygregory | if fails we just update the information and move the node to a failed state | 15:08 |
| dtantsur | validation on manageable + a periodic task matches nicely the power state handling | 15:08 |
| dtantsur | so there is at least an established pattern | 15:08 |
| TheJulia | also, I suspect I'm going to be spending time looking at db stuff soon | 15:09 |
| TheJulia | so we might be able to improve things a little bit | 15:09 |
| TheJulia | and periodics can be disabled | 15:09 |
| dtantsur | yep, this is also important | 15:09 |
| TheJulia | but the on managable would catch things in your case | 15:09 |
| dtantsur | well, I suspect it's possible to update driver in BMO.. but here be dragons, let's take baby steps, etc, etc | 15:10 |
| dtantsur | now the hardest part: come up with a new method name :) | 15:10 |
| iurygregory | ++ baby steps | 15:11 |
| iurygregory | I'm looking at the code to understand the workflow you are proposing | 15:12 |
| dtantsur | iurygregory: wanna draft an RFE? | 15:12 |
| dtantsur | it's going to be very close to power state sync, so check it | 15:12 |
| * TheJulia takes corgi overlord out for the 32767th time this morning | 15:12 | |
| dtantsur | corgi \o/ | 15:13 |
| TheJulia | maybe also do it as part of the sync | 15:13 |
| TheJulia | or the same loop | 15:13 |
| TheJulia | or maybe same loop but skew it | 15:13 |
| TheJulia | but don't run it *as* often | 15:13 |
| dtantsur | doing it in the same loop will not allow disablign this periodic | 15:13 |
| TheJulia | true | 15:13 |
| TheJulia | it doesn't need to try and check things every five minutes | 15:13 |
| TheJulia | once an hour or something like that might be good | 15:14 |
| TheJulia | OR | 15:14 |
| dtantsur | furthermore, it can check node.last_updated | 15:14 |
| TheJulia | only run on recently changed nodes | 15:14 |
| TheJulia | yup | 15:14 |
| dtantsur | right ;) | 15:14 |
| dtantsur | take a shared lock, check last_updated, do not run if not updated recently | 15:14 |
| dtantsur | (always run on 'manage') | 15:14 |
| TheJulia | oh, but full scan on first loop maybe | 15:14 |
| TheJulia | then go to the last_updated | 15:14 |
| TheJulia | that captures if something went sideways | 15:14 |
| dtantsur | we can assume that full scan is done on ->manageable | 15:14 |
| TheJulia | or someone did manual database evil | 15:14 |
| dtantsur | it doesn't have to be 100% robust in the end | 15:15 |
| dtantsur | oh, a missing part. if the validation fails in the periodic task, then what? | 15:15 |
| dtantsur | set a fault? only last_error? | 15:15 |
| dtantsur | remove all secure boot keys? | 15:16 |
| dtantsur | (the latter was not 100% serious, jfyi) | 15:16 |
| iurygregory | dtantsur, sure I will do, let me just finish to understand the workflow you are proposing =) | 15:16 |
| dtantsur | well, I also need to finish understanding it :) | 15:16 |
| iurygregory | corgi is the new method name? | 15:16 |
| iurygregory | LOL | 15:16 |
| dtantsur | if task.driver.boot.corgi('bork') | 15:17 |
| TheJulia | +1 | 15:18 |
| dtantsur | the more I think, the more I'm inclined to omit the periodic part in the 1st iteration | 15:18 |
| TheJulia | task.driver.management.boop_the_corgi_snoot(task) | 15:18 |
| TheJulia | ++ | 15:19 |
| dtantsur | and only make ->manageable call some BaseInterface.validate_full | 15:19 |
| TheJulia | i is easy to wire up later | 15:19 |
| dtantsur | right | 15:19 |
| TheJulia | s/i /it / | 15:19 |
| dtantsur | and boot_the_corgi++ | 15:19 |
| dtantsur | wait, boot? that was boop | 15:19 |
| * dtantsur is out of words, send hel^Wcorgis | 15:19 | |
| TheJulia | https://www.youtube.com/watch?v=LkykPTM9TGo | 15:20 |
| dtantsur | very nice! | 15:21 |
| openstackgerrit | Derek Higgins proposed openstack/sushy-tools master: Fix Unit test to test for success https://review.opendev.org/c/openstack/sushy-tools/+/776655 | 15:21 |
| TheJulia | :returns: BarkException if something is out of place or has moved an inch. | 15:22 |
| TheJulia | :returns: WimperException when sadness exists. | 15:22 |
| TheJulia | I made the unfortunate mistake of moving the wine fridge we got at the start of the pandemic... and it greatly upset the corgi overlord last night | 15:23 |
| iurygregory | aww =( | 15:24 |
| TheJulia | :returns: TheCatIsDoingSomethingItShouldntException is raised when an intermediate state exists. Example: cat in the window. | 15:24 |
| dtantsur | :D | 15:25 |
| *** ociuhandu has quit IRC | 15:26 | |
| NobodyCam | What did I do? | 15:26 |
| dtantsur | NobodyCam: did you boop the corgi? | 15:26 |
| TheJulia | NobodyCam: Nothing, we signed you up for everything though. | 15:26 |
| NobodyCam | And of course TGIF | 15:26 |
| dtantsur | ... for not booping the corgi enough | 15:26 |
| *** ociuhandu has joined #openstack-ironic | 15:26 | |
| NobodyCam | I have booped the corgi | 15:27 |
| NobodyCam | Hehehhe | 15:28 |
| dtantsur | NOT ENOUGH!!1 | 15:28 |
| TheJulia | lol | 15:28 |
| NobodyCam | LoL | 15:28 |
| TheJulia | At this rate, CNBaaS will be a thing | 15:29 |
| NobodyCam | LoL | 15:29 |
| NobodyCam | I would vote to expand the scope to NBaas | 15:30 |
| TheJulia | To cover Great Danes? | 15:31 |
| NobodyCam | Of course | 15:31 |
| NobodyCam | But really all the puppets | 15:31 |
| NobodyCam | Gah | 15:31 |
| NobodyCam | Puppers | 15:32 |
| *** ociuhandu has quit IRC | 15:32 | |
| openstackgerrit | Derek Higgins proposed openstack/sushy-tools master: New option to prevent libvirt defining boot order https://review.opendev.org/c/openstack/sushy-tools/+/776660 | 15:32 |
| NobodyCam | 13 out of 10. Would boop again | 15:35 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Trivial: fix incorrect ordering in iLO tests https://review.opendev.org/c/openstack/ironic/+/776661 | 15:47 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Trivial: fix incorrect ordering in iLO tests https://review.opendev.org/c/openstack/ironic/+/776661 | 15:52 |
| TheJulia | tzumainn: you around? | 15:57 |
| *** ociuhandu has joined #openstack-ironic | 15:57 | |
| tzumainn | TheJulia, yep, what's up? | 15:58 |
| TheJulia | thoughts on owners and volume targets.... | 15:58 |
| TheJulia | err | 15:58 |
| TheJulia | lessees | 15:58 |
| tzumainn | ah, yep! | 15:58 |
| tzumainn | I kinda just thought they'd be handled the same way as baremetal ports - derive owner/lessee from the node | 15:59 |
| tzumainn | but maybe it's more complex than that? | 15:59 |
| TheJulia | well, that is how I'm coding it | 15:59 |
| TheJulia | but I'm wondering of lesses should be able to | 15:59 |
| TheJulia | create/delete volume targets | 15:59 |
| TheJulia | or connectors, which is far more forgiving than ports | 16:00 |
| *** ociuhandu has quit IRC | 16:00 | |
| TheJulia | but most of the time this is iscsi stuff and not like a fibrechannel adapter or soemthing like that | 16:00 |
| *** ociuhandu has joined #openstack-ironic | 16:00 | |
| TheJulia | . o (does anyone still sell fc gear?) | 16:00 |
| TheJulia | looks like fabric directors are still a thing | 16:01 |
| TheJulia | so yes | 16:01 |
| tzumainn | TheJulia, I think the answer should be yes - it seems like the sort of thing a lessee might reasonably want to do | 16:03 |
| tzumainn | and I don't see how it messes with the node too much; the volume target is deleted when the node is undeployed | 16:03 |
| TheJulia | yeah | 16:03 |
| TheJulia | connectors are more static, but easily recreated | 16:03 |
| tzumainn | is https://review.opendev.org/c/openstack/ironic/+/767631 useful to what you're doing, or is it superceded? I don't care too much either way | 16:03 |
| TheJulia | I don't see a reason to split them | 16:04 |
| TheJulia | looking | 16:04 |
| TheJulia | very similar to what I've done | 16:05 |
| TheJulia | very very similar | 16:05 |
| TheJulia | I didn't rename one of the methods, likely should in the grand scheme, but I'm still just trying to get the first pass of this done | 16:06 |
| tzumainn | TheJulia, well, if you'd like I can resolve this merge conflict, or I can just close it if you'd like! | 16:06 |
| TheJulia | I'm 16+ patches deep | 16:06 |
| TheJulia | so *shrugs* | 16:06 |
| tzumainn | hahaha, okay - well, I'l lresolve the merge conflict and let you decide later, it's no big deal to me either way! | 16:07 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Trivial: fix incorrect ordering in iLO tests https://review.opendev.org/c/openstack/ironic/+/776661 | 16:07 |
| TheJulia | I've got a lot of testing that I've put in place too in the entire series of patches that has helped me find a few issues. That likely kept me from just doing the method rename | 16:07 |
| *** k_mouza_ has quit IRC | 16:12 | |
| *** k_mouza has joined #openstack-ironic | 16:13 | |
| arne_wiebalck | bye everyone, have a good weekend o/ | 16:22 |
| TheJulia | have a good weekend arne_wiebalck | 16:22 |
| TheJulia | dtantsur: are you going to be on the spuc? | 16:27 |
| dtantsur | TheJulia: planning on. why? | 16:27 |
| TheJulia | I need a tiny bit of context on allocations and talking to you is likely easiest path | 16:28 |
| TheJulia | since I'll likely be getting to that next week | 16:28 |
| dtantsur | ack | 16:28 |
| TheJulia | I'm like 60% through volume connectors/targets | 16:28 |
| TheJulia | I *should* be done with it today | 16:29 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Switch iLO and iRMC to the new secure boot framework https://review.opendev.org/c/openstack/ironic/+/770122 | 16:33 |
| *** bdodd has joined #openstack-ironic | 16:40 | |
| TheJulia | o/ bdodd | 16:40 |
| TheJulia | hopefully you've thawed out :) | 16:40 |
| *** gyee has joined #openstack-ironic | 16:41 | |
| bdodd | TheJulia Thanks! Yes, I have power and heat now and am back online. :-) Still without running water, but situation much improved. | 16:42 |
| TheJulia | ugh, well, hopefully you'll get water back soon. | 16:42 |
| bdodd | Hope so. | 16:42 |
| ajya | hi bdodd, I did some updates on RAID patch, feel free to make adjustments :) | 16:56 |
| bdodd | Hi, aija. Thanks very much for working on those updates! I'm reviewing them right now. | 16:57 |
| TheJulia | https://bluejeans.com/250125662 <-- Time for the SPUC? | 17:00 |
| JayF | I'll be in SPUC shortly TheJulia | 17:03 |
| *** k_mouza_ has joined #openstack-ironic | 17:11 | |
| *** jmccrory_ has joined #openstack-ironic | 17:11 | |
| -openstackstatus- NOTICE: All jobs are failing with POST_FAILURE due to a backward incompatible change made in the swift log upload libarary role. Working on a fix now. | 17:17 | |
| dtantsur | ouch, just rechecked | 17:17 |
| *** k_mouza has quit IRC | 17:18 | |
| *** lourot has quit IRC | 17:18 | |
| *** ricolin has quit IRC | 17:18 | |
| *** jmccrory has quit IRC | 17:18 | |
| *** jmccrory_ is now known as jmccrory | 17:18 | |
| *** k_mouza has joined #openstack-ironic | 17:21 | |
| *** lourot has joined #openstack-ironic | 17:21 | |
| *** k_mouza has quit IRC | 17:22 | |
| *** ociuhandu_ has joined #openstack-ironic | 17:27 | |
| *** ociuhandu has quit IRC | 17:30 | |
| *** ociuhandu_ has quit IRC | 17:31 | |
| -openstackstatus- NOTICE: The change to the upload role has been reverted. Jobs started since the revert appear to be functioning normally. You can recheck changes that failed for builds started between 16:12 and 17:13UTC reporting POST_FAILURE safely now. | 17:57 | |
| *** derekh has quit IRC | 18:01 | |
| *** dtantsur is now known as dtantsur|afk | 18:10 | |
| dtantsur|afk | have a great weekend! | 18:14 |
| TheJulia | o/ | 18:14 |
| JayF | TheJulia: dtantsur|afk: https://storyboard.openstack.org/#!/story/1547139 | 18:15 |
| JayF | TheJulia: if you can find that CERN story, I'd love to see it too | 18:17 |
| *** k_mouza_ has quit IRC | 18:17 | |
| TheJulia | JayF: oh, I pasted it to you | 18:19 |
| TheJulia | https://storyboard.openstack.org/#!/story/2008626 | 18:19 |
| JayF | thanks | 18:19 |
| *** ayoung has quit IRC | 18:29 | |
| *** moshiur has joined #openstack-ironic | 18:31 | |
| *** moshiur has quit IRC | 18:33 | |
| openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic master: Expose owner and lessee in baremetal volume policy https://review.opendev.org/c/openstack/ironic/+/767631 | 19:01 |
| tzumainn | TheJulia, just a note regarding the updated patch; I found it necessary to update fake_db_node to have the uuid that's hardcoded in the acl yaml file | 19:03 |
| tzumainn | that seems to allow a bunch of tests that previously returned 400s because they couldn't find the referenced node, to return 201 instead | 19:04 |
| TheJulia | Yeah | 19:07 |
| TheJulia | I've been slowly going through those and finding those as I hit each controller | 19:07 |
| *** gmann is now known as gmann_afk | 19:19 | |
| *** jmccrory has quit IRC | 19:36 | |
| *** mkowalski has quit IRC | 19:36 | |
| *** larsks has quit IRC | 19:36 | |
| *** mrda has quit IRC | 19:36 | |
| *** ddurst has quit IRC | 19:36 | |
| *** jhesketh has quit IRC | 19:36 | |
| *** yoctozepto has quit IRC | 19:36 | |
| *** lourot has quit IRC | 19:36 | |
| *** bdodd has quit IRC | 19:36 | |
| *** pmannidi has quit IRC | 19:36 | |
| *** hjensas has quit IRC | 19:36 | |
| *** bnemec has quit IRC | 19:36 | |
| *** irclogbot_0 has quit IRC | 19:36 | |
| *** stevebaker has quit IRC | 19:36 | |
| *** mcarden has quit IRC | 19:36 | |
| *** jlvillal has quit IRC | 19:36 | |
| *** zzzeek has quit IRC | 19:36 | |
| *** gouthamr has quit IRC | 19:36 | |
| *** sshnaidm|off has quit IRC | 19:36 | |
| *** openstackgerrit has quit IRC | 19:36 | |
| *** adrianc has quit IRC | 19:36 | |
| *** strigazi has quit IRC | 19:36 | |
| *** gmann_afk has quit IRC | 19:36 | |
| *** jungleboyj has quit IRC | 19:36 | |
| *** sri_ has quit IRC | 19:36 | |
| *** portdirect has quit IRC | 19:36 | |
| *** phrobb has quit IRC | 19:36 | |
| *** gryf has quit IRC | 19:36 | |
| *** uzumaki has quit IRC | 19:36 | |
| *** mgoddard has quit IRC | 19:36 | |
| *** mkowalsk- has quit IRC | 19:36 | |
| *** parallax has quit IRC | 19:36 | |
| *** zigo has quit IRC | 19:36 | |
| *** fdegir has quit IRC | 19:36 | |
| *** mnasiadka has quit IRC | 19:36 | |
| *** early has quit IRC | 19:36 | |
| *** trandles has quit IRC | 19:36 | |
| *** eandersson has quit IRC | 19:36 | |
| *** mushrushu has quit IRC | 19:36 | |
| *** ianychoi_ has quit IRC | 19:36 | |
| *** naved001 has quit IRC | 19:36 | |
| *** baha has quit IRC | 19:36 | |
| *** tzumainn has quit IRC | 19:36 | |
| *** anuradha1904 has quit IRC | 19:36 | |
| *** buhman has quit IRC | 19:36 | |
| *** arne_wiebalck has quit IRC | 19:36 | |
| *** mnaser has quit IRC | 19:36 | |
| *** mugsie has quit IRC | 19:36 | |
| *** PrinzElvis has quit IRC | 19:36 | |
| *** erbarr has quit IRC | 19:36 | |
| *** ftarasenko has quit IRC | 19:36 | |
| *** persia has quit IRC | 19:36 | |
| *** melwitt has quit IRC | 19:36 | |
| *** dhellmann has quit IRC | 19:36 | |
| *** dansmith has quit IRC | 19:36 | |
| *** valleedelisle has quit IRC | 19:36 | |
| *** rloo has quit IRC | 19:36 | |
| *** knikolla has quit IRC | 19:36 | |
| *** johnsom has quit IRC | 19:36 | |
| *** TheJulia has quit IRC | 19:36 | |
| *** tosin has quit IRC | 19:36 | |
| *** rajinir has quit IRC | 19:36 | |
| *** andrein has quit IRC | 19:36 | |
| *** ildikov has quit IRC | 19:36 | |
| *** bbezak has quit IRC | 19:36 | |
| *** seongsoocho has quit IRC | 19:36 | |
| *** ajya has quit IRC | 19:36 | |
| *** jroll has quit IRC | 19:36 | |
| *** gyee has quit IRC | 19:36 | |
| *** dking has quit IRC | 19:36 | |
| *** logan- has quit IRC | 19:36 | |
| *** andy_ has quit IRC | 19:36 | |
| *** benj_ has quit IRC | 19:36 | |
| *** trident has quit IRC | 19:36 | |
| *** rpittau|afk has quit IRC | 19:36 | |
| *** NobodyCam has quit IRC | 19:36 | |
| *** vdrok has quit IRC | 19:36 | |
| *** iurygregory has quit IRC | 19:36 | |
| *** lifeless has quit IRC | 19:36 | |
| *** dtantsur|afk has quit IRC | 19:36 | |
| *** seba has quit IRC | 19:36 | |
| *** fungi has quit IRC | 19:36 | |
| *** ccstone has quit IRC | 19:36 | |
| *** ianw has quit IRC | 19:36 | |
| *** SpamapS has quit IRC | 19:36 | |
| *** cp- has quit IRC | 19:36 | |
| *** lbragstad has quit IRC | 19:36 | |
| *** coreycb has quit IRC | 19:36 | |
| *** guilhermesp has quit IRC | 19:36 | |
| *** maelk has quit IRC | 19:36 | |
| *** zer0c00l has quit IRC | 19:36 | |
| *** mjturek has quit IRC | 19:36 | |
| *** shadower has quit IRC | 19:36 | |
| *** janders has quit IRC | 19:36 | |
| *** bburns has quit IRC | 19:36 | |
| *** priteau has quit IRC | 19:36 | |
| *** dtruong has quit IRC | 19:36 | |
| *** dsneddon has quit IRC | 19:36 | |
| *** rpioso has quit IRC | 19:36 | |
| *** akahat has quit IRC | 19:36 | |
| *** donnyd has quit IRC | 19:36 | |
| *** v1k0d3n has quit IRC | 19:36 | |
| *** tosky has quit IRC | 19:36 | |
| *** vkmc has quit IRC | 19:36 | |
| *** lourot has joined #openstack-ironic | 19:40 | |
| *** jmccrory has joined #openstack-ironic | 19:40 | |
| *** gyee has joined #openstack-ironic | 19:40 | |
| *** bdodd has joined #openstack-ironic | 19:40 | |
| *** tzumainn has joined #openstack-ironic | 19:40 | |
| *** uzumaki has joined #openstack-ironic | 19:40 | |
| *** anuradha1904 has joined #openstack-ironic | 19:40 | |
| *** rloo has joined #openstack-ironic | 19:40 | |
| *** bburns has joined #openstack-ironic | 19:40 | |
| *** knikolla has joined #openstack-ironic | 19:40 | |
| *** buhman has joined #openstack-ironic | 19:40 | |
| *** johnsom has joined #openstack-ironic | 19:40 | |
| *** rpittau|afk has joined #openstack-ironic | 19:40 | |
| *** TheJulia has joined #openstack-ironic | 19:40 | |
| *** NobodyCam has joined #openstack-ironic | 19:40 | |
| *** vdrok has joined #openstack-ironic | 19:40 | |
| *** tosin has joined #openstack-ironic | 19:40 | |
| *** mkowalski has joined #openstack-ironic | 19:40 | |
| *** tosky has joined #openstack-ironic | 19:40 | |
| *** iurygregory has joined #openstack-ironic | 19:40 | |
| *** benj_ has joined #openstack-ironic | 19:40 | |
| *** zzzeek has joined #openstack-ironic | 19:40 | |
| *** arne_wiebalck has joined #openstack-ironic | 19:40 | |
| *** rajinir has joined #openstack-ironic | 19:40 | |
| *** dking has joined #openstack-ironic | 19:40 | |
| *** priteau has joined #openstack-ironic | 19:40 | |
| *** eandersson has joined #openstack-ironic | 19:40 | |
| *** ccstone has joined #openstack-ironic | 19:40 | |
| *** gouthamr has joined #openstack-ironic | 19:40 | |
| *** ianw has joined #openstack-ironic | 19:40 | |
| *** SpamapS has joined #openstack-ironic | 19:40 | |
| *** mkowalsk- has joined #openstack-ironic | 19:40 | |
| *** jhesketh has joined #openstack-ironic | 19:40 | |
| *** sshnaidm|off has joined #openstack-ironic | 19:40 | |
| *** mnaser has joined #openstack-ironic | 19:40 | |
| *** dtruong has joined #openstack-ironic | 19:40 | |
| *** mugsie has joined #openstack-ironic | 19:40 | |
| *** pmannidi has joined #openstack-ironic | 19:40 | |
| *** openstackgerrit has joined #openstack-ironic | 19:40 | |
| *** mushrushu has joined #openstack-ironic | 19:40 | |
| *** ianychoi_ has joined #openstack-ironic | 19:40 | |
| *** adrianc has joined #openstack-ironic | 19:40 | |
| *** larsks has joined #openstack-ironic | 19:40 | |
| *** logan- has joined #openstack-ironic | 19:40 | |
| *** cp- has joined #openstack-ironic | 19:40 | |
| *** ddurst has joined #openstack-ironic | 19:40 | |
| *** mrda has joined #openstack-ironic | 19:40 | |
| *** andrein has joined #openstack-ironic | 19:40 | |
| *** jlvillal has joined #openstack-ironic | 19:40 | |
| *** mcarden has joined #openstack-ironic | 19:40 | |
| *** stevebaker has joined #openstack-ironic | 19:40 | |
| *** irclogbot_0 has joined #openstack-ironic | 19:40 | |
| *** bnemec has joined #openstack-ironic | 19:40 | |
| *** hjensas has joined #openstack-ironic | 19:40 | |
| *** gryf has joined #openstack-ironic | 19:40 | |
| *** phrobb has joined #openstack-ironic | 19:40 | |
| *** portdirect has joined #openstack-ironic | 19:40 | |
| *** jungleboyj has joined #openstack-ironic | 19:40 | |
| *** sri_ has joined #openstack-ironic | 19:40 | |
| *** gmann_afk has joined #openstack-ironic | 19:40 | |
| *** strigazi has joined #openstack-ironic | 19:40 | |
| *** dsneddon has joined #openstack-ironic | 19:40 | |
| *** lbragstad has joined #openstack-ironic | 19:40 | |
| *** PrinzElvis has joined #openstack-ironic | 19:40 | |
| *** ildikov has joined #openstack-ironic | 19:40 | |
| *** coreycb has joined #openstack-ironic | 19:40 | |
| *** guilhermesp has joined #openstack-ironic | 19:40 | |
| *** rpioso has joined #openstack-ironic | 19:40 | |
| *** parallax has joined #openstack-ironic | 19:40 | |
| *** naved001 has joined #openstack-ironic | 19:40 | |
| *** zigo has joined #openstack-ironic | 19:40 | |
| *** andy_ has joined #openstack-ironic | 19:40 | |
| *** akahat has joined #openstack-ironic | 19:40 | |
| *** fdegir has joined #openstack-ironic | 19:40 | |
| *** lifeless has joined #openstack-ironic | 19:40 | |
| *** yoctozepto has joined #openstack-ironic | 19:40 | |
| *** mnasiadka has joined #openstack-ironic | 19:40 | |
| *** bbezak has joined #openstack-ironic | 19:40 | |
| *** donnyd has joined #openstack-ironic | 19:40 | |
| *** seongsoocho has joined #openstack-ironic | 19:40 | |
| *** v1k0d3n has joined #openstack-ironic | 19:40 | |
| *** erbarr has joined #openstack-ironic | 19:40 | |
| *** vkmc has joined #openstack-ironic | 19:40 | |
| *** early has joined #openstack-ironic | 19:40 | |
| *** ftarasenko has joined #openstack-ironic | 19:40 | |
| *** maelk has joined #openstack-ironic | 19:40 | |
| *** janders has joined #openstack-ironic | 19:40 | |
| *** fungi has joined #openstack-ironic | 19:40 | |
| *** seba has joined #openstack-ironic | 19:40 | |
| *** shadower has joined #openstack-ironic | 19:40 | |
| *** dtantsur|afk has joined #openstack-ironic | 19:40 | |
| *** trident has joined #openstack-ironic | 19:40 | |
| *** ajya has joined #openstack-ironic | 19:40 | |
| *** jroll has joined #openstack-ironic | 19:40 | |
| *** dansmith has joined #openstack-ironic | 19:40 | |
| *** valleedelisle has joined #openstack-ironic | 19:40 | |
| *** mjturek has joined #openstack-ironic | 19:40 | |
| *** dhellmann has joined #openstack-ironic | 19:40 | |
| *** zer0c00l has joined #openstack-ironic | 19:40 | |
| *** persia has joined #openstack-ironic | 19:40 | |
| *** melwitt has joined #openstack-ironic | 19:40 | |
| *** trandles has joined #openstack-ironic | 19:40 | |
| *** baha has joined #openstack-ironic | 19:40 | |
| *** mgoddard has joined #openstack-ironic | 19:41 | |
| arne_wiebalck | JayF: I have a patched version of the power sync job, will try it next week to see how it affects the sustained DB accesses. The hope is a good reduction, ofc. Will report the findings :) | 19:45 |
| JayF | arne_wiebalck: we run with it completely disabled :/ | 19:45 |
| arne_wiebalck | JayF: yeah, I remember ... from my scale I can guess it would not work at yours at all | 19:46 |
| JayF | Hopefully that won't be the case forever; and I appreciate you blazing the trail | 19:46 |
| arne_wiebalck | JayF: this is mostly to shown the gain, TheJulia and dtantsur|afk have suggested to move to some lazy loading instead (rather than no loading when you do not need it); picking what you need seems easier as a first step | 19:48 |
| * arne_wiebalck leaves for the weekend o/ | 19:51 | |
| *** mkowalsk- has quit IRC | 20:04 | |
| *** mkowalski_ has joined #openstack-ironic | 20:16 | |
| *** mkowalski_ has quit IRC | 20:27 | |
| *** zzzeek has quit IRC | 20:28 | |
| *** zzzeek has joined #openstack-ironic | 20:30 | |
| *** rcernin has joined #openstack-ironic | 20:32 | |
| iurygregory | rloo, hey if you have a chance I've answered your questions in https://review.opendev.org/c/openstack/project-config/+/775244 =) | 20:38 |
| rloo | looking... | 20:39 |
| *** rcernin has quit IRC | 20:40 | |
| *** ociuhandu has joined #openstack-ironic | 20:41 | |
| rloo | iurygregory: even if we don't want to use +2/-1 (I don't actually care). why is it that our -1/+1 are different from eg oslo.config's? | 20:42 |
| rloo | i am not happy that there is inconsistency among the different projects in openstack. that is a very bad user experience. | 20:43 |
| *** tosin has quit IRC | 20:44 | |
| iurygregory | rloo, basically oslo folks told me that +2/-2 doesn't add anything (+1/-1 would have the same effect) | 20:44 |
| iurygregory | I just moved the name from +2/-2 to our +1/-1 | 20:45 |
| iurygregory | and added restriction to Backport Candidate label to only cores (this would be the main thing to discuss, i.e do we want to let it open our restricted?) | 20:46 |
| rloo | iurygregory: i commented. so sorry. i feel like it makes things very inconsistent. and i don't have time to try to figure out and chase down who is doing all this copy/pasting. | 20:46 |
| *** mkowalski_ has joined #openstack-ironic | 20:47 | |
| rloo | iurygregory: my point being that if this project-config repo is meant to be used across all the openstack projects, why is each project defining what 'backport candidate' means to them. | 20:47 |
| iurygregory | rloo, I totally understand =), but oslo defined for their project, I'm ok using the same definition they did if this is what we want =) | 20:49 |
| rloo | iurygregory: also very very likely i don't understand what is going on. so I removed my -1. | 20:49 |
| rloo | what I want is for one place in project-config, where [label "Backport Candidate"] is defined. then maybe on a per-project basis, it can be overridden. but all this copy/paste seems wrong to me. | 20:49 |
| iurygregory | I see | 20:50 |
| iurygregory | I will talk with clarkb about it | 20:51 |
| *** mkowalski_ has quit IRC | 21:09 | |
| *** anuradha1904 has quit IRC | 22:23 | |
| *** mkowalski_ has joined #openstack-ironic | 22:55 | |
| *** rcernin has joined #openstack-ironic | 22:59 | |
| *** dking has quit IRC | 23:13 | |
| *** rloo has quit IRC | 23:18 | |
| *** gmann_afk is now known as gmann | 23:21 | |
| *** rcernin has quit IRC | 23:26 | |
| *** ociuhandu has quit IRC | 23:27 | |
| *** uzumaki has quit IRC | 23:52 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!