| *** MentalSiege has quit IRC | 00:06 | |
| *** pmannidi has quit IRC | 00:07 | |
| *** pmannidi has joined #openstack-ironic | 00:13 | |
| *** gmann_afk is now known as gmann | 00:19 | |
| *** pmannidi_ has joined #openstack-ironic | 00:44 | |
| *** pmannidi has quit IRC | 00:45 | |
| *** pmannidi_ has quit IRC | 00:48 | |
| *** pmannidi has joined #openstack-ironic | 01:33 | |
| *** rloo has quit IRC | 01:46 | |
| *** rcernin has quit IRC | 02:31 | |
| openstackgerrit | Bob Fournier proposed openstack/sushy master: Don't log ERROR if GET of /redfish/v1/SessionService fails https://review.opendev.org/c/openstack/sushy/+/774760 | 02:33 |
|---|---|---|
| *** rcernin has joined #openstack-ironic | 02:40 | |
| *** pmannidi has quit IRC | 02:49 | |
| *** pmannidi has joined #openstack-ironic | 02:52 | |
| *** zzzeek has quit IRC | 03:56 | |
| *** zzzeek has joined #openstack-ironic | 03:56 | |
| *** pmannidi has quit IRC | 04:07 | |
| *** pmannidi has joined #openstack-ironic | 04:10 | |
| *** dsneddon has quit IRC | 04:37 | |
| *** hoonetorg has joined #openstack-ironic | 04:40 | |
| *** dsneddon has joined #openstack-ironic | 04:42 | |
| *** mkowalski has quit IRC | 04:46 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic master: Use OOB inspection to fetch MACs for IB inspection https://review.opendev.org/c/openstack/ironic/+/774391 | 04:57 |
| *** openstackstatus has quit IRC | 04:58 | |
| *** openstack has joined #openstack-ironic | 05:00 | |
| *** ChanServ sets mode: +o openstack | 05:00 | |
| *** stendulker has joined #openstack-ironic | 05:02 | |
| *** hoonetorg has quit IRC | 05:05 | |
| *** pmannidi has quit IRC | 05:13 | |
| *** pmannidi has joined #openstack-ironic | 05:16 | |
| *** mkowalski has joined #openstack-ironic | 05:18 | |
| *** hoonetorg has joined #openstack-ironic | 05:19 | |
| *** stendulker has quit IRC | 05:22 | |
| openstackgerrit | vinay50muddu proposed openstack/ironic master: Add support to manage certificates in iLO https://review.opendev.org/c/openstack/ironic/+/760573 | 05:49 |
| *** pmannidi has quit IRC | 06:19 | |
| *** pmannidi has joined #openstack-ironic | 06:22 | |
| *** k_mouza has joined #openstack-ironic | 06:22 | |
| *** k_mouza has quit IRC | 06:26 | |
| *** gyee has quit IRC | 06:54 | |
| *** tzumainn has quit IRC | 07:09 | |
| *** uzumaki has quit IRC | 07:14 | |
| *** pmannidi has quit IRC | 07:26 | |
| *** pmannidi has joined #openstack-ironic | 07:29 | |
| arne_wiebalck | Good morning, ironic! | 07:33 |
| openstackgerrit | Merged openstack/sushy master: Don't log ERROR if GET of /redfish/v1/SessionService fails https://review.opendev.org/c/openstack/sushy/+/774760 | 07:42 |
| *** moshiur has joined #openstack-ironic | 07:47 | |
| *** rpittau|afk is now known as rpittau | 07:51 | |
| rpittau | good morning ironic! o/ | 07:51 |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost stable/victoria: Misplaced variable preventing fast-track after inspection https://review.opendev.org/c/openstack/bifrost/+/774672 | 08:14 |
| *** pmannidi has quit IRC | 08:41 | |
| janders | good morning arne_wiebalck and rpittau o/ | 08:46 |
| rpittau | hey janders :) | 08:46 |
| arne_wiebalck | hey janders and rpittau o/ | 08:46 |
| rpittau | hey arne_wiebalck :) | 08:48 |
| *** pmannidi has joined #openstack-ironic | 08:50 | |
| *** dougsz has joined #openstack-ironic | 08:53 | |
| *** anuradha1904 has joined #openstack-ironic | 08:54 | |
| *** dougsz has quit IRC | 08:54 | |
| *** rcernin has quit IRC | 08:56 | |
| janders | I'm adding unit tests to MAC-OOB patch. Making good progress except this one: https://review.opendev.org/c/openstack/ironic/+/774391/7/ironic/tests/unit/drivers/modules/redfish/test_management.py#1495 | 09:03 |
| janders | I was trying to follow this example: https://opendev.org/openstack/ironic/src/branch/master/ironic/tests/unit/drivers/modules/redfish/test_management.py#L1374 but clearly I'm doing something wrong | 09:05 |
| janders | any hints welcome! :) | 09:05 |
| janders | (and the test is deliberately commented out in the change - I wanted to test the other tests in CI in the meantime) | 09:05 |
| janders | my guess is I'm setting the side_effect wrong - or perhaps the assert is off (wrong scoping - or maybe even wrong method). Little confused with all the layers :) | 09:07 |
| *** lucasagomes has joined #openstack-ironic | 09:09 | |
| *** tosky has joined #openstack-ironic | 09:12 | |
| *** derekh has joined #openstack-ironic | 09:33 | |
| *** stendulker has joined #openstack-ironic | 09:45 | |
| rpittau | janders: one thing I spot is that the side effect exception doesn't the () | 09:51 |
| rpittau | doesn't need* | 09:51 |
| rpittau | janders: also I don't see you calling set_summary anywhere | 09:52 |
| rpittau | janders: mmmm not sure that would work anyway, summary is a property of the ethernet_interfaces, so once you have the system and the ethernet_interfaces, probably don't need a try-except there | 09:55 |
| *** stendulker has quit IRC | 10:05 | |
| *** pmannidi has quit IRC | 10:07 | |
| *** pmannidi has joined #openstack-ironic | 10:10 | |
| *** stendulker has joined #openstack-ironic | 10:13 | |
| rpittau | dtantsur|afk, arne_wiebalck, and everyone interested :) I'm doing some (more) reshuffling between hardware and raid_utils module, just realized that we have a bunch of text in the hardware test module that can be moved out to reduce size and just separate from the code. It should be an easy and quick change, just wanted an opinion if it's worth before moving forward | 10:20 |
| arne_wiebalck | rpittau: text? like comments? | 10:26 |
| rpittau | arne_wiebalck: text like samples output | 10:27 |
| arne_wiebalck | rpittau: ah, yes | 10:27 |
| rpittau | the first 860 lines :) | 10:27 |
| arne_wiebalck | lol | 10:27 |
| arne_wiebalck | all for the tests! | 10:27 |
| rpittau | yeah | 10:27 |
| arne_wiebalck | sure, I am all for moving it to a better place | 10:28 |
| rpittau | since if we keep moving the raid related functions out from hardware we'll end up sharing some of the samples, this could be a good opportunity to put them in a separate place | 10:28 |
| arne_wiebalck | sounds good | 10:28 |
| rpittau | ok | 10:28 |
| *** dtantsur|afk is now known as dtantsur | 10:38 | |
| dtantsur | morning ironic | 10:38 |
| dtantsur | rpittau: makes sense to me | 10:38 |
| *** dsneddon has quit IRC | 10:50 | |
| *** dsneddon has joined #openstack-ironic | 10:51 | |
| *** stendulker has quit IRC | 10:51 | |
| *** dougsz has joined #openstack-ironic | 10:57 | |
| *** dsneddon has quit IRC | 10:57 | |
| *** dsneddon has joined #openstack-ironic | 10:57 | |
| janders | rpittau thank you. So - do you think this test is a bad idea in the first place? | 11:25 |
| *** k_mouza has joined #openstack-ironic | 11:27 | |
| *** dougsz has quit IRC | 11:28 | |
| *** dsneddon has quit IRC | 11:28 | |
| iurygregory | good morning arne_wiebalck janders rpittau dtantsur and Ironic o/ | 11:29 |
| *** dsneddon has joined #openstack-ironic | 11:29 | |
| *** ianychoi__ is now known as ianychoi | 11:30 | |
| janders | good morning iurygregory o/ | 11:36 |
| rpittau | janders: I need to give it another look, but the entire try-except may not be needed as you're just retrieving some info already assigned to the ethernet_interface object. Does it make sense? | 11:41 |
| janders | yes, thank you rpittau | 11:43 |
| janders | it does indeed look like I decided to make my life more difficult writing this bit :) | 11:46 |
| janders | I thought the original code had these clauses and I carried them over while breaking this part out to a separate method, but looking at master I don't see that... must have been me :) | 11:46 |
| openstackgerrit | Merged openstack/bifrost stable/victoria: Misplaced variable preventing fast-track after inspection https://review.opendev.org/c/openstack/bifrost/+/774672 | 11:55 |
| *** dougsz has joined #openstack-ironic | 12:07 | |
| *** dougsz has quit IRC | 12:10 | |
| *** anuradha1904 has quit IRC | 12:14 | |
| *** pmannidi has quit IRC | 12:15 | |
| *** pmannidi has joined #openstack-ironic | 12:18 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Move some raid specific functions to raid_utils https://review.opendev.org/c/openstack/ironic-python-agent/+/774854 | 12:50 |
| *** iurygregory has quit IRC | 12:51 | |
| *** anuradha1904 has joined #openstack-ironic | 12:55 | |
| *** rcernin has joined #openstack-ironic | 13:13 | |
| *** rcernin has quit IRC | 13:18 | |
| *** pmannidi has quit IRC | 13:23 | |
| *** pmannidi has joined #openstack-ironic | 13:27 | |
| janders | see you tomorrow Ironic o/ | 13:27 |
| *** paras333_ has quit IRC | 13:30 | |
| rpittau | bye janders :) | 13:34 |
| *** rloo has joined #openstack-ironic | 14:02 | |
| TheJulia | good morning | 14:08 |
| *** ianychoi has quit IRC | 14:09 | |
| *** iurygregory has joined #openstack-ironic | 14:13 | |
| * iurygregory is back | 14:14 | |
| dtantsur | morning TheJulia | 14:19 |
| iurygregory | good morning TheJulia =) | 14:20 |
| TheJulia | wb iurygregory | 14:24 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-specs master: Implement System Scoped Role Based Access Control https://review.opendev.org/c/openstack/ironic-specs/+/764070 | 14:24 |
| iurygregory | ty =) I was having some problems with my ISP | 14:25 |
| *** pmannidi has quit IRC | 14:34 | |
| *** pmannidi has joined #openstack-ironic | 14:35 | |
| *** tzumainn has joined #openstack-ironic | 14:43 | |
| *** lmcgann has joined #openstack-ironic | 14:48 | |
| *** MentalSiege has joined #openstack-ironic | 15:05 | |
| *** MentalSiege has quit IRC | 15:12 | |
| *** MentalSiege has joined #openstack-ironic | 15:13 | |
| *** rcernin has joined #openstack-ironic | 15:14 | |
| *** rcernin has quit IRC | 15:18 | |
| iurygregory | TheJulia, if possible can you add https://review.opendev.org/c/openstack/openstacksdk/+/768612 to your list this week? =) | 15:34 |
| TheJulia | iurygregory: I'll try for tomorrow, if that is okay | 15:38 |
| *** uzumaki has joined #openstack-ironic | 15:38 | |
| iurygregory | TheJulia, totally fine =) | 15:40 |
| *** pmannidi has quit IRC | 15:43 | |
| *** pmannidi has joined #openstack-ironic | 15:46 | |
| *** moshiur has quit IRC | 16:06 | |
| *** paras333_ has joined #openstack-ironic | 16:11 | |
| *** ianw has quit IRC | 16:39 | |
| *** ianw has joined #openstack-ironic | 16:39 | |
| TheJulia | omg, one line has been driving me crazy since yesterday afternoon | 16:40 |
| iurygregory | TheJulia, want to share the line? not sure if I can help, but I can try =) | 16:42 |
| TheJulia | oh, I finally figured it out | 16:42 |
| TheJulia | I had set a rule to member instead of reader | 16:43 |
| iurygregory | \o/ | 16:43 |
| TheJulia | along the way I realized I needed to flip the parameter order on the authorization check method | 16:43 |
| TheJulia | because that was wrong too | 16:43 |
| iurygregory | nice =D | 16:44 |
| *** lucasagomes has quit IRC | 16:56 | |
| rpittau | if anyone has a minute, ironic-inspector ready to go for train and stein https://review.opendev.org/c/openstack/ironic-inspector/+/773332 https://review.opendev.org/c/openstack/ironic-inspector/+/773091 | 17:03 |
| rpittau | rocky is failing on python3 :/ | 17:03 |
| TheJulia | ugh | 17:07 |
| TheJulia | wheeeeeeee I thought I found an elegent way in python to do a thing... then see the note "Added in python 3.9" | 17:08 |
| rpittau | let's backport it | 17:09 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Fix error message with UEFI-incompatible images https://review.opendev.org/c/openstack/ironic-python-agent/+/772575 | 17:09 |
| dtantsur | TheJulia: if you have a minute ^^ | 17:09 |
| dtantsur | it's small but touches the code you've recently built | 17:09 |
| TheJulia | ack, let me just figure out my string headache | 17:10 |
| dtantsur | oh | 17:10 |
| dtantsur | then ignore it, we'll figure out | 17:10 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost stable/train: Use tag for ipa-builder https://review.opendev.org/c/openstack/bifrost/+/774237 | 17:12 |
| mgoddard | iurygregory: possible fix for the failures ^ | 17:12 |
| iurygregory | mgoddard, hey tks! | 17:12 |
| iurygregory | let's see how it goes, I was still debugging centos8 job | 17:12 |
| mgoddard | iurygregory: I tried installing lxml and libvirt-python in a centos 8 test VM | 17:13 |
| iurygregory | we probably need to drop l-c job =) | 17:13 |
| mgoddard | iurygregory: fails with train UC, works without | 17:13 |
| mgoddard | true | 17:13 |
| mgoddard | I'll add that, just in case it works | 17:13 |
| iurygregory | yeah | 17:14 |
| dtantsur | mgoddard: will likely also need https://review.opendev.org/c/openstack/bifrost/+/766745 | 17:14 |
| iurygregory | and probably move opensuse job to non-voting (not sure if the centos fix will work for it) | 17:14 |
| dtantsur | opensuse is currently broken on all branches | 17:14 |
| mgoddard | oh dear, can of worms | 17:15 |
| dtantsur | no, fixed on master | 17:15 |
| dtantsur | but does not work on older branches | 17:15 |
| dtantsur | yeah | 17:15 |
| iurygregory | yeah... | 17:15 |
| rpittau | don't forget the pip upgrade | 17:15 |
| dtantsur | which reminds me, I wanted to formally propose to stop doing EM for Bifrost | 17:15 |
| dtantsur | let me send an email | 17:15 |
| TheJulia | dtantsur: looking | 17:15 |
| iurygregory | oh god the pip issue also D: | 17:16 |
| rpittau | if you want to have a look at centos8 almost passing in ussuri https://review.opendev.org/c/openstack/bifrost/+/766742 | 17:16 |
| rpittau | still haven't figured out the issue with pxeboot | 17:16 |
| rpittau | locally, first time it doesn't work, second time it works :D | 17:17 |
| iurygregory | *magic* | 17:18 |
| TheJulia | so thinking through this, we may not have a known efi partition id if it was a whole disk image that we deployed | 17:21 |
| dtantsur | s/may not/never/ | 17:21 |
| dtantsur | It's usually None, I think. It *definitely* is None for incompatible whole disk images. | 17:21 |
| TheJulia | yeah | 17:21 |
| TheJulia | I think the change is good | 17:21 |
| TheJulia | because your restricting the lookup if we have the value which we may not have | 17:22 |
| *** dougsz has joined #openstack-ironic | 17:23 | |
| TheJulia | And ++ on the message | 17:24 |
| TheJulia | it shouldn't break the re-call of the method because by that point we should have all the details and would have otherwise blown up earlier | 17:27 |
| *** dougsz has quit IRC | 17:28 | |
| *** pmannidi has quit IRC | 17:30 | |
| *** pmannidi has joined #openstack-ironic | 17:32 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: utils.mounted: retry umount and allow retrying mount https://review.opendev.org/c/openstack/ironic-lib/+/774908 | 17:33 |
| dtantsur | if you're wondering about ^^ and others, I'm trying to build a reasonably stable API for deploy steps to use | 17:33 |
| dtantsur | * out-of-tree deploy steps first and foremost | 17:34 |
| TheJulia | ack | 17:36 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost stable/train: Fix several CI issues (Train) https://review.opendev.org/c/openstack/bifrost/+/774237 | 17:37 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost stable/train: Fix several CI issues (Train) https://review.opendev.org/c/openstack/bifrost/+/774237 | 17:38 |
| * TheJulia crosses her fingers that she hasn't broken hundreds of api tests | 17:39 | |
| openstackgerrit | Aija Jauntēva proposed openstack/sushy master: [WIP] Refactor TaskMonitors https://review.opendev.org/c/openstack/sushy/+/774532 | 17:51 |
| openstackgerrit | Ghanshyam proposed openstack/networking-generic-switch stable/train: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/networking-generic-switch/+/773335 | 17:54 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: Add find_efi_partition, deprecate get_uefi_disk_identifier https://review.opendev.org/c/openstack/ironic-lib/+/774914 | 17:54 |
| TheJulia | That moment when your changes work too well... and you realize that you need to add a deprecated rule to fallback upon | 17:59 |
| TheJulia | (or at least reference one...) | 18:00 |
| *** derekh has quit IRC | 18:00 | |
| openstackgerrit | Aija Jauntēva proposed openstack/sushy master: [WIP] Refactor TaskMonitors https://review.opendev.org/c/openstack/sushy/+/774532 | 18:05 |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/ironic-inspector/+/773091 | 18:12 |
| rpittau | good night o/ | 18:23 |
| *** rpittau is now known as rpittau|afk | 18:23 | |
| *** k_mouza has quit IRC | 18:33 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: [WIP] Clean up partition functions in disk_utils https://review.opendev.org/c/openstack/ironic-lib/+/774914 | 18:38 |
| *** pmannidi has quit IRC | 18:38 | |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/ironic-inspector/+/773332 | 18:39 |
| *** pmannidi has joined #openstack-ironic | 18:43 | |
| *** dtantsur is now known as dtantsur|afk | 18:45 | |
| dtantsur|afk | o/ | 18:45 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: [WIP] Split out a common helper for blkid https://review.opendev.org/c/openstack/ironic-lib/+/774931 | 18:49 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: [WIP] Split out a common helper for blkid https://review.opendev.org/c/openstack/ironic-lib/+/774931 | 18:53 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: Add find_efi_partition, deprecate get_uefi_disk_identifier https://review.opendev.org/c/openstack/ironic-lib/+/774914 | 18:55 |
| *** rloo has quit IRC | 19:25 | |
| *** rloo has joined #openstack-ironic | 19:25 | |
| *** zzzeek has quit IRC | 19:26 | |
| *** zzzeek has joined #openstack-ironic | 19:27 | |
| *** pmannidi has quit IRC | 19:48 | |
| *** pmannidi has joined #openstack-ironic | 19:49 | |
| *** rloo has quit IRC | 19:51 | |
| *** rloo has joined #openstack-ironic | 19:51 | |
| *** zzzeek has quit IRC | 19:53 | |
| *** zzzeek has joined #openstack-ironic | 19:57 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/stein: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/ironic-inspector/+/773091 | 20:21 |
| *** zzzeek has quit IRC | 20:21 | |
| *** gyee has joined #openstack-ironic | 20:21 | |
| *** zzzeek has joined #openstack-ironic | 20:22 | |
| TheJulia | stevebaker: you around for call? | 20:32 |
| stevebaker | coming | 20:40 |
| arne_wiebalck | bye everyone o/ | 20:49 |
| *** pmannidi has quit IRC | 20:53 | |
| *** pmannidi has joined #openstack-ironic | 20:56 | |
| *** uzumaki has quit IRC | 21:05 | |
| *** lmcgann has quit IRC | 21:59 | |
| janders | good morning Ironic o/ | 22:02 |
| *** pmannidi has quit IRC | 22:03 | |
| janders | TheJulia would you have some time to chat about https://review.opendev.org/c/openstack/ironic-python-agent/+/771904 ? | 22:04 |
| *** pmannidi has joined #openstack-ironic | 22:06 | |
| *** anuradha1904 has quit IRC | 22:35 | |
| TheJulia | janders: let me make a cup of coffee first | 22:35 |
| *** zzzeek has quit IRC | 22:37 | |
| *** zzzeek has joined #openstack-ironic | 22:37 | |
| janders | TheJulia no worries, ready when you are (reading your UEFI comments in the Google Doc in the meantime) | 22:40 |
| TheJulia | janders: coffee will be ready in a couple minutes, /msg me with a url | 22:44 |
| TheJulia | yeah, that whole thing has be very worried | 22:46 |
| janders | https://review.opendev.org/c/openstack/ironic-python-agent/+/771904/24/ironic_python_agent/hardware.py#1690 | 22:46 |
| janders | this is the comment I wanted to chat about to make sure I understand things correctly | 22:46 |
| janders | and - firstly - thank you for your feedback regarding the change, it's been very helpful :) | 22:47 |
| TheJulia | so I think we're basically on the same page | 22:48 |
| TheJulia | but the back of my brain is saying the secure erase code doesn't raise exceptions if there is an incompatability, that it just returns false | 22:48 |
| TheJulia | but maybe I'm out of date | 22:48 |
| janders | I will check in a sec - but there is one more thing that's related that I wanted to ask you about | 22:49 |
| TheJulia | https://meet.google.com/bgg-yabg-wvi | 22:49 |
| janders | ok! | 22:49 |
| *** zzzeek has quit IRC | 22:50 | |
| *** zzzeek has joined #openstack-ironic | 22:52 | |
| stevebaker | lbragstad: hey, do you have opinions on packaging default policy files in /etc/<project>? Right now ironic packages /etc/ironic/policy.json containing yaml, which is broken. But the fix could be deleting it entirely or generating /etc/ironic/policy.yaml | 22:58 |
| lbragstad | ^ that's what i would recommend | 22:59 |
| lbragstad | generate the defaults based on the code | 22:59 |
| lbragstad | er - generate the template/sample based on the defaults in code | 22:59 |
| stevebaker | lbragstad: so a generated /etc/ironic/policy.yaml? Cool that was my first thought | 23:00 |
| lbragstad | yeah - exactly | 23:00 |
| stevebaker | but many projects don't package any policy file so I wondered | 23:00 |
| lbragstad | if you need it | 23:00 |
| lbragstad | if you don't need to package it, because ironic runs with the defaults registered in code, by default | 23:01 |
| lbragstad | then you might not need to provide one? | 23:01 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Generate policy.yaml.sample https://review.opendev.org/c/openstack/ironic/+/775039 | 23:01 |
| lbragstad | if that's the case, then the operator can generate one if they want to supply their own overrides | 23:01 |
| lbragstad | in keystone - we don't package or release one | 23:02 |
| *** zzzeek has quit IRC | 23:02 | |
| lbragstad | we just generate sample on-the-fly for documentation | 23:02 |
| lbragstad | samples* | 23:02 |
| *** zzzeek has joined #openstack-ironic | 23:04 | |
| stevebaker | lbragstad: on the other hand, we have a legacy problem for upgrades if we stop packaging /etc/ironic/policy.* the old one will eventually become stale, we can't delete it because it *might* be customized. So if we're being friendly for package upgrades we have to package policy.yaml forever because we packaged policy.json in the past. | 23:12 |
| *** pmannidi has quit IRC | 23:14 | |
| *** pmannidi has joined #openstack-ironic | 23:15 | |
| TheJulia | I *think* the pain on upgrade is going to have to be manually updating that policy as part of the upgrade process. At least that is hoped what will occur with secure rbac movement | 23:24 |
| *** zzzeek has quit IRC | 23:27 | |
| *** zzzeek has joined #openstack-ironic | 23:29 | |
| *** zzzeek has quit IRC | 23:34 | |
| TheJulia | 7994 tests... | 23:35 |
| *** zzzeek has joined #openstack-ironic | 23:36 | |
| *** tzumainn has quit IRC | 23:38 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!