| opendevreview | Merged openstack/ironic stable/train: Remove redfish cache entry upon errors https://review.opendev.org/c/openstack/ironic/+/820937 | 01:02 |
|---|---|---|
| *** pmannidi is now known as pmannidi|Away | 01:43 | |
| *** pmannidi|Away is now known as pmannidi | 01:44 | |
| *** pmannidi is now known as pmannidi|Away | 01:49 | |
| *** pmannidi|Away is now known as pmannidi | 02:34 | |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Adds create_csr and add_https_certificate clean step https://review.opendev.org/c/openstack/ironic/+/804486 | 04:30 |
| rpittau | good morning ironic! o/ | 06:57 |
| arne_wiebalck | Good morning rpittau and Ironic! | 07:18 |
| rpittau | hey arne_wiebalck :) | 07:21 |
| arne_wiebalck | dtantsur: https://review.opendev.org/c/openstack/ironic-python-agent/+/826909, by "merge" you mean to make one backport patch out of our two patches in master ? | 07:49 |
| janders | hey rpittau arne_wiebalck and Ironic o/ | 08:59 |
| janders | rpittau arne_wiebalck I will try to push the doco addition for hybrid cleaning today | 08:59 |
| rpittau | hey janders :) | 09:00 |
| arne_wiebalck | hey janders o/ | 09:00 |
| arne_wiebalck | cool, thanks a lot for taking care of this! | 09:01 |
| janders | no worries | 09:01 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Improve efficiency of storage cleaning in mixed media envs - documentation https://review.opendev.org/c/openstack/ironic/+/833986 | 10:44 |
| janders | arne_wiebalck rpittau please let me know what you think about ^^. I added a fair bit more than just configuration advice for erase_devices_express as I did not see much information on any storage cleaning steps and thought we probably need to explain the differences between them. | 10:48 |
| rpittau | thanks janders, I'll check later today | 10:50 |
| janders | rpittau thank you | 10:50 |
| janders | I also think we need to re-work info on setting priorities (section about erase_devices_priority is out of sync with section on priority_step_overrides) but that needs to be a separate change | 10:51 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Improve efficiency of storage cleaning in mixed media envs - documentation https://review.opendev.org/c/openstack/ironic/+/833986 | 11:07 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Improve efficiency of storage cleaning in mixed media envs - documentation https://review.opendev.org/c/openstack/ironic/+/833986 | 11:08 |
| janders | see you tomorrow Ironic o/ | 11:28 |
| arne_wiebalck | bye janders o/ | 11:29 |
| iurygregory | good morning Ironic o/ | 11:31 |
| MahnoorAsghar | Good morning | 11:44 |
| dtantsur | arne_wiebalck: yep, correct | 12:03 |
| arne_wiebalck | dtantsur: why not two separate backports? | 12:27 |
| dtantsur | arne_wiebalck: because the first is breaking | 12:34 |
| arne_wiebalck | dtantsur: wouldn't it be just a question of ordering then? happy to merge them, it is just that the backport would be different content-wise | 12:37 |
| arne_wiebalck | dtantsur: I mean not only adapted to the older code, but really different | 12:37 |
| * arne_wiebalck needs to step away for a bit | 12:38 | |
| dtantsur | arne_wiebalck: mm, yes, if they can be reordered, it's cool | 12:38 |
| arne_wiebalck | dtantsur: well, I guess if yours goes in first, mine will need a rebase | 12:38 |
| arne_wiebalck | dtantsur: anyway, happy to merge them, the option was just new to me :) | 12:39 |
| dtantsur | up to you, I'm only worried about merging one without the other (CI issues, reviews, etc) | 12:54 |
| TheJulia | The furchildren decided I had to be woken up a few times last night... I really wish I could just sleep the day away | 13:30 |
| dtantsur | furchildren can be like that.. morning TheJulia | 13:30 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent stable/xena: SoftwareRAID: Use efibootmgr (and drop grub2-install) https://review.opendev.org/c/openstack/ironic-python-agent/+/826909 | 14:00 |
| arne_wiebalck | dtantsur: ^^ ... the tests pass locally, have a look, and then we see what the CI thinks :) | 14:02 |
| dtantsur | thanks! I may not get to it today unfortunately: too many other things | 14:03 |
| arne_wiebalck | no problem, I don't think there is any rush | 14:24 |
| arne_wiebalck | rpittau: maybe as well if time permits (as you reviewed my half already and did some of the refactoring around disk and raid utils) | 14:25 |
| rpittau | arne_wiebalck: I'll see if I have some time later, busy days :/ | 14:33 |
| arne_wiebalck | rpittau: great, thanks, no rush at all :) | 14:40 |
| iurygregory | I will add to my list to review =) | 14:41 |
| arne_wiebalck | thanks iurygregory ! | 14:43 |
| arne_wiebalck | TheJulia: I've started to expose some multi-tenancy to our users ... very good feedback so far :) | 14:45 |
| arne_wiebalck | TheJulia: And the use cases are piling up. | 14:46 |
| * arne_wiebalck some call it use cases, some outsourcing tasks to users ;) | 14:46 | |
| arne_wiebalck | TheJulia: Anyway, the ownership/multi-tenancy/RBAC package seems quite useful so far. | 14:47 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/bifrost master: [DNM] Testing CI before creating stable/yoga https://review.opendev.org/c/openstack/bifrost/+/834030 | 14:48 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: [DNM] Testing CI before creating stable/yoga https://review.opendev.org/c/openstack/ironic-inspector/+/834033 | 14:53 |
| ameya49 | iurygregory : Hi, Hope you are doing good :). Thanks for the reviews for tempest test cases patches for introspection and BIOS cleaning step. | 14:54 |
| iurygregory | ameya49, yw =) | 14:55 |
| ameya49 | Can we also have a review for : https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/826646 (Have addressed Julia's comments) | 14:57 |
| TheJulia | arne_wiebalck: Excellent! | 14:59 |
| TheJulia | review the automatic assignment change :) | 14:59 |
| arne_wiebalck | TheJulia: heh, will do :) | 15:00 |
| rpittau | arne_wiebalck: lgtm | 15:04 |
| arne_wiebalck | rpittau: great, thanks for checking | 15:05 |
| opendevreview | Merged openstack/ironic master: Improve efficiency of storage cleaning in mixed media envs - documentation https://review.opendev.org/c/openstack/ironic/+/833986 | 15:12 |
| rpittau | good night! o/ | 16:46 |
| TheJulia | https://www.openssl.org/news/secadv/20220315.txt <-- Wheeeeeee | 17:36 |
| stevebaker[m] | good morning! | 19:36 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Create API documentation from docstrings https://review.opendev.org/c/openstack/ironic/+/827200 | 20:07 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Create API documentation from docstrings https://review.opendev.org/c/openstack/ironic/+/827200 | 20:10 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Add docstring input for API documentation https://review.opendev.org/c/openstack/ironic/+/834068 | 20:10 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Create API documentation from docstrings https://review.opendev.org/c/openstack/ironic/+/827200 | 20:23 |
| TheJulia | Good morning stevebaker[m] | 20:36 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!