arne_wiebalck | Good morning, Ironic! | 07:08 |
---|---|---|
opendevreview | Merged openstack/ironic-tempest-plugin master: Fix for verifying introspected data https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/831958 | 07:13 |
rpittau | good morning ironic! o/ | 07:24 |
arne_wiebalck | hey rpittau o/ | 07:43 |
rpittau | hey arne_wiebalck :) | 07:43 |
janders | hey arne_wiebalck rpittau and Ironic o/ | 07:54 |
arne_wiebalck | hey janders o/ | 07:56 |
rpittau | hey janders :) | 07:58 |
rpittau | mmm is there an issue with the cirros image in networking-baremetal job? | 07:59 |
rpittau | if we look at https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_139/832697/1/check/networking-baremetal-multitenant-vlans/1396232/controller/logs/ironic-bm-logs/node-2_console_2022-03-09-03%3A43%3A29_log.txt | 07:59 |
rpittau | we can see https://paste.openstack.org/show/b9CDfkeIrELTycXmcRNq/ | 07:59 |
dtantsur | rpittau: no configdrive | 08:14 |
dtantsur | I thought I fixed the multinode job, but maybe n-b has its own configuration | 08:14 |
dtantsur | we need FORCE_CONFIG_DRIVE=true (or how it's called) | 08:14 |
dtantsur | morning folks | 08:16 |
dtantsur | `FORCE_CONFIG_DRIVE: False` WUT | 08:16 |
rpittau | ah! | 08:16 |
rpittau | mornig dtantsur :) | 08:17 |
dtantsur | I wonder if it was accidental | 08:17 |
dtantsur | the partition image I build does not work with nova metadata server, only with configdrive | 08:18 |
dtantsur | maybe I can fix it... | 08:18 |
dtantsur | but in reality, configdrive is required for everything but BFV/ramdisk | 08:18 |
opendevreview | Jacob Anders proposed openstack/ironic-python-agent master: Improve efficiency of storage cleaning in mixed media envs https://review.opendev.org/c/openstack/ironic-python-agent/+/818712 | 10:44 |
iurygregory | good morning Ironic | 11:11 |
dtantsur | morning iurygregory | 11:12 |
dtantsur | I need to restart my lab machine with the IRC bouncer, hopefully back online in a few minutes. | 11:20 |
janders | hey iurygregory | 11:21 |
opendevreview | Jacob Anders proposed openstack/ironic-python-agent master: Improve efficiency of storage cleaning in mixed media envs https://review.opendev.org/c/openstack/ironic-python-agent/+/818712 | 11:35 |
dtantsur | seemingly back :) | 11:37 |
iurygregory | stendulker, hey you around? we have an interesting bug happening in HPE machines when creating a redfish subscription https://bugzilla.redhat.com/show_bug.cgi?id=2060587 | 11:41 |
iurygregory | do you know if is expected that after rebooting the machine the subscription is deleted from the BMC? | 11:41 |
janders | see you tomorrow Ironic o/ | 12:21 |
iurygregory_ | bye janders o/ | 12:28 |
*** iurygregory_ is now known as iurygregory | 13:48 | |
rpittau | hey if someone has a moment, please check https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/830909 to have cs9 jobs voting in ipa-builder, thanks! | 14:17 |
rloo | +100 | 14:21 |
TheJulia | good morning | 14:34 |
MahnoorAsghar | Good morning Julia | 14:35 |
dtantsur | morning rloo, TheJulia, MahnoorAsghar | 14:50 |
MahnoorAsghar | o/ | 14:51 |
rloo | mornin' dtantsur, TheJulia, MahnoorAsghar, rpittau :) | 14:52 |
rpittau | hey rloo :) | 14:52 |
rpittau | good morning TheJulia MahnoorAsghar :) | 14:52 |
MahnoorAsghar | I am working on adding input into my extension - and there is a question | 14:54 |
MahnoorAsghar | All the API methods that are meant to be exposed, are marked with the @method.expose() decorator, right? | 14:54 |
TheJulia | that is *likely* a question best answered by sbaker | 14:55 |
TheJulia | err, stevebaker[m] | 14:55 |
TheJulia | so.. the secret in the framework is that it has some static method names it looks for | 14:56 |
MahnoorAsghar | okay | 14:56 |
TheJulia | and there is some code which helps it route by path of the URL to the file and directory | 14:56 |
MahnoorAsghar | ahan | 14:56 |
MahnoorAsghar | I should look for that code | 14:56 |
TheJulia | so "GET /" always translates to get(), "GET /<value>" translates out to get_one() | 14:57 |
TheJulia | err | 14:57 |
MahnoorAsghar | aah, I see | 14:57 |
TheJulia | actually s/get()/get_all()/ | 14:57 |
TheJulia | or soemthing like that | 14:57 |
MahnoorAsghar | yes I see what you mean | 14:57 |
TheJulia | it requires the brain to make that assocation which... is sometimes not easy | 14:57 |
MahnoorAsghar | okay | 14:58 |
opendevreview | Merged openstack/ironic master: Ignore fake nodes in the power sync loop https://review.opendev.org/c/openstack/ironic/+/821215 | 15:50 |
opendevreview | Merged openstack/ironic-python-agent-builder master: Make centos9 jobs voting, add to gate https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/830909 | 15:58 |
iurygregory | TheJulia, dtantsur rpittau any thoughts on Elod comment's https://review.opendev.org/c/openstack/releases/+/825662 ? | 16:33 |
TheJulia | I'd tend to agree with the comment | 16:34 |
TheJulia | I'd also like to highlight the general inconsistency which has been applied in openstack's history as to such releases | 16:34 |
TheJulia | whcih is obviously frustrating | 16:34 |
iurygregory | yeah, it also makes sense to me | 16:39 |
rpittau | bye everyone, see you tomorrow o/ | 16:43 |
MahnoorAsghar | o/ | 16:43 |
rpioso | Good morning, ironic :) | 16:44 |
rpioso | iurygregory: Thank you for the congrats on my accepted Summit presentation proposal. Better late than never :-/ | 16:45 |
iurygregory | :D | 16:45 |
* TheJulia blinks | 19:39 | |
* TheJulia wonders where the day has gone off to | 19:39 | |
stevebaker[m] | good morning | 19:42 |
TheJulia | good morning! | 19:54 |
frigo | Hello Ironic! I hope you are doing great :) | 20:21 |
frigo | I am thinking about deploying a stack using routed subnets, and IPv6 (I've never used any of these). I am wondering how ambitious this is. | 20:22 |
TheJulia | frigo: with dhcp forwarding? | 20:31 |
frigo | yes, should be a mix of router advertisements and dhcp relaying | 20:32 |
TheJulia | so, neutron *does* support dhcpv6, you'll need to operate in stateful mode | 20:35 |
TheJulia | I guess that should work | 20:35 |
TheJulia | my only concern, w/r/t baremetal is *actual* hardware support. | 20:35 |
TheJulia | some firmware in cards doesn't support v6 | 20:35 |
TheJulia | Some UEFI network stacks don't know about v6 either | 20:35 |
TheJulia | but they are generally getting better | 20:35 |
frigo | normally my hardware is recent enough. I am trying both (stateful, stateless) but I did not go very far yet. Is there a reason for using stateful mode instead of stateless ? | 20:36 |
frigo | I have had issues with both today :D I used a single vlan, with different subnets per rack. With ipv6, the router advertisement get multicasted across the racks and PXE does not seem to use the right one. With stateful, ironic-inspector gave me an IP.. but from the wrong subnet. I need to try harder | 20:39 |
TheJulia | frigo: stateless i guess is viable, but you need to use dhcpv6 and your routers need to explicitly announce it | 20:44 |
TheJulia | with stateless, they can just go "oh, here is my ip based on the announcer" | 20:45 |
TheJulia | stateful, it has to ask dhcpv6 | 20:45 |
frigo | ok I guess that both should work if configured correctly :D | 20:46 |
opendevreview | Julia Kreger proposed openstack/networking-baremetal master: Remove config drive override https://review.opendev.org/c/openstack/networking-baremetal/+/832857 | 20:56 |
opendevreview | Julia Kreger proposed openstack/networking-baremetal master: CI: Remove config drive override https://review.opendev.org/c/openstack/networking-baremetal/+/832857 | 20:58 |
TheJulia | gah | 21:09 |
TheJulia | pointless now | 21:09 |
TheJulia | since ci now fails | 21:09 |
opendevreview | Julia Kreger proposed openstack/networking-baremetal master: Set agent_type in tests https://review.opendev.org/c/openstack/networking-baremetal/+/832697 | 21:11 |
hjensas | TheJulia: Thanks for that, I was trying to reproduce the issues locally. Let's hope you nailed it! | 21:14 |
TheJulia | I read the prior discussion from right before I woke up and looked at it... and went "wait a second!" | 21:15 |
TheJulia | and then started looking through git history | 21:15 |
TheJulia | it looks like a mistake, to me | 21:15 |
TheJulia | or copy/paste | 21:15 |
TheJulia | dunno why it was there really | 21:15 |
* hjensas guesses self is to blame. :) Makes sense to remove it. | 21:17 | |
TheJulia | hjensas: nah ;) | 21:23 |
TheJulia | any core reviewers up for a relatively quick ipa reivew? https://review.opendev.org/c/openstack/ironic-python-agent/+/831029 | 21:25 |
opendevreview | Julia Kreger proposed openstack/networking-baremetal stable/yoga: Set agent_type in tests https://review.opendev.org/c/openstack/networking-baremetal/+/832858 | 21:27 |
TheJulia | hjensas: ^^ to unbrick stable/yoga | 21:27 |
TheJulia | we likely need to backport further, maybe | 21:28 |
hjensas | possibly yes, I think the option was deprecated in ussuri so might take it atleast there. | 21:29 |
opendevreview | Julia Kreger proposed openstack/sushy stable/yoga: Fix session authentication issues https://review.opendev.org/c/openstack/sushy/+/832859 | 21:31 |
opendevreview | Julia Kreger proposed openstack/sushy stable/xena: Fix session authentication issues https://review.opendev.org/c/openstack/sushy/+/832860 | 21:32 |
opendevreview | Julia Kreger proposed openstack/sushy stable/wallaby: Fix session authentication issues https://review.opendev.org/c/openstack/sushy/+/832862 | 21:47 |
opendevreview | Julia Kreger proposed openstack/sushy stable/victoria: Fix session authentication issues https://review.opendev.org/c/openstack/sushy/+/832864 | 22:00 |
opendevreview | Julia Kreger proposed openstack/sushy stable/ussuri: Protect Connector against empty auth object https://review.opendev.org/c/openstack/sushy/+/832826 | 22:08 |
opendevreview | Julia Kreger proposed openstack/sushy stable/ussuri: Fix session authentication issues https://review.opendev.org/c/openstack/sushy/+/832866 | 22:11 |
opendevreview | Julia Kreger proposed openstack/sushy stable/train: Protect Connector against empty auth object https://review.opendev.org/c/openstack/sushy/+/832827 | 22:12 |
TheJulia | ooh ahh you can now cherry pick a merge conflict | 22:12 |
TheJulia | in the UI | 22:13 |
opendevreview | Julia Kreger proposed openstack/sushy stable/train: Protect Connector against empty auth object https://review.opendev.org/c/openstack/sushy/+/832827 | 22:16 |
opendevreview | Julia Kreger proposed openstack/sushy stable/train: Fix session authentication issues https://review.opendev.org/c/openstack/sushy/+/832867 | 22:17 |
TheJulia | any cores around to do some quick reviews on https://review.opendev.org/q/I171845e556a45746d9435a11390e510c731f6d5c | 23:11 |
TheJulia | we need to cut a new fix release for networking-baremetal relatively soon | 23:11 |
JayF | TheJulia: +2, leaving you to +a | 23:14 |
JayF | you need the same on the stable one? | 23:14 |
TheJulia | yeah | 23:15 |
JayF | change seems simple and safe enough :shrug: | 23:15 |
TheJulia | JayF: and it is broken at present | 23:15 |
TheJulia | hjensas: fyi^^ | 23:15 |
JayF | well, that's literally the first ever review I've done on that repo | 23:15 |
JayF | and I don't approve anything but simple changes now | 23:15 |
TheJulia | ahh yeah | 23:15 |
JayF | it has the +2s, you can land if/when you wish | 23:16 |
TheJulia | hjensas: also, we should talk to iury about core rights :) | 23:20 |
JayF | honestly, we should talk to iury about taking away mine, too | 23:20 |
JayF | you can give them to hjensas lol | 23:21 |
TheJulia | JayF: that is evil.... and might just work! | 23:22 |
JayF | also nice to meet you hjensas lol | 23:22 |
* TheJulia needs to go to the mechanic and give them money for removing the transmission from the errand car | 23:23 | |
opendevreview | Jacob Anders proposed openstack/ironic-python-agent master: Improve efficiency of storage cleaning in mixed media envs https://review.opendev.org/c/openstack/ironic-python-agent/+/818712 | 23:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!