opendevreview | OpenStack Proposal Bot proposed openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/861555 | 03:42 |
---|---|---|
rpittau | good morning ironic! o/ | 07:32 |
arne_wiebalck | Good morning rpittau and Ironic! | 08:00 |
rpittau | hey arne_wiebalck :) | 08:00 |
jpodivin | Hi. I wanted to ask how should I get info about the storage devices used by a node. I've seen "local_gb" in the properties dictionary I get when running `node show` ... but what if there are multiple disks? | 08:20 |
janders | good morning rpittau arne_wiebalck jpodivin and Ironic o/ | 08:40 |
rpittau | hey janders :) | 08:40 |
arne_wiebalck | good morning janders o/ | 08:40 |
* dtantsur checks the ptg etherpad | 08:58 | |
dtantsur | we don't have the normal meeting today, do we? | 09:07 |
rpittau | we don't | 09:08 |
rpittau | if anyone has a moment please review the bifrost migration to jammy https://review.opendev.org/c/openstack/bifrost/+/848828 and upgrading from zed https://review.opendev.org/c/openstack/bifrost/+/859075 thanks! | 09:16 |
jpodivin | janders: do you happen to know how to get the disk info? | 09:20 |
opendevreview | Verification of a change to openstack/ironic master failed: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/861555 | 09:44 |
dtantsur | jpodivin: if you use ironic-inspector, it stores some data that can be accessed via its API/CLI | 10:13 |
dtantsur | if not, it's not possible | 10:13 |
jpodivin | dtantsur: I see. So there is no way to find out how many disks are on an active node? | 10:27 |
dtantsur | jpodivin: if you don't use ironic-inspector (or don't have access to it) - no. | 10:27 |
dtantsur | (note that we don't provide or support agents that run on active instances) | 10:27 |
jpodivin | dtantsur: including inspector? | 10:27 |
dtantsur | sorry? | 10:28 |
dtantsur | (okay, there is limited support for re-introspection of active instances - forgot about it) | 10:28 |
jpodivin | So if I have ironic inspector I can get info about the disks? | 10:29 |
dtantsur | jpodivin: if you ran introspection for the nodes - yes | 10:29 |
dtantsur | `openstack baremetal introspection data save <node>` | 10:29 |
jpodivin | dtantsur: but that doesn't work on active nodes ... hm | 10:29 |
dtantsur | jpodivin: what's your case? do you expect the disks to change? | 10:30 |
jpodivin | dtantsur: not necessarily, I just need to know how many there are and which one is root. On a working cloud that is, with active nodes. | 10:31 |
dtantsur | jpodivin: it's not something we have out-of-box. I think arne_wiebalck is doing something like this: a container is running on the nodes that periodically posts updated inventory to ironic-inspector. Right Arne? | 10:32 |
jpodivin | dtantsur: ok, if there is no way to get it without introspection we'll work with that. The introspection data are stored automatically right? They can be called back at any point after it's finished and node is active? | 10:37 |
dtantsur | jpodivin: correct | 10:58 |
dtantsur | JayF, iurygregory, TheJulia, I wonder if it's too late to swap "Merging inspector into Ironic" and e.g. "OpenInfra Labs collab". I have a hard stop at 16:00 UTC on Tue/Wed, so it's somewhat risky for me. | 10:59 |
iurygregory | good morning Ironic | 11:06 |
iurygregory | dtantsur, fine with me | 11:06 |
arne_wiebalck | jpodivin: we do not do this periodically/constantly, but we have the mechanism in place in case we have to, yes | 11:14 |
arne_wiebalck | jpodivin: e.g. when we need to update the serial numbers of nodes | 11:14 |
arne_wiebalck | jpodivin: and we were able to run introspection on a few 100 nodes in parallel against our APIs, seemed ok :) | 11:15 |
kubajj | Good afternoon Ironic, I had a wee chat with TheJulia on Friday and I was suggested to share my devstack local.conf and the error block of output that I think says what the error is. If anybody has any suggestions on what could be the problem, please let me know. | 13:03 |
kubajj | https://www.irccloud.com/pastebin/VKmADRDT/ | 13:04 |
kubajj | https://www.irccloud.com/pastebin/knWJYNrZ/ | 13:04 |
kubajj | (It seems that the paste bins are not there anymore, I'll redo them) | 13:06 |
kubajj | Local.conf: https://paste.opendev.org/show/bLnk28LuR4A5NrZTVL8h/ | 13:08 |
kubajj | Devstack output: https://paste.opendev.org/show/b341YTBEYGexAATLPNrJ/ | 13:08 |
iurygregory | kubajj, so the error is on neutron, so probably some of the configs you have in your local.conf should be changed | 13:48 |
JayF | dtantsur: sure; I'm OK with that. You wanna edit the etherpad directly or shall I? | 13:48 |
iurygregory | in the devstack jobs checked the local.conf we have FIXED_RANGE="10.1.0.0/20" and we don't have any value set for FIXED_NETWORK_SIZE | 13:49 |
iurygregory | I was looking at https://0f0a489eb876d8f6c64b-af7203ec313049b47d68440a509d433a.ssl.cf5.rackcdn.com/861496/1/check/ironic-tempest-uefi-redfish-vmedia/865590e/controller/logs/local_conf.txt | 13:49 |
iurygregory | if you have neutron enabled you probably need to set some extra params like the "Q_***" variables | 13:50 |
iurygregory | I think neutron has some defaults with the plugin that we don't support, so you need to have the variable set | 13:55 |
iurygregory | 4min for our session with nova right? | 13:56 |
JayF | yes | 13:57 |
dtantsur | JayF: changed | 14:53 |
dtantsur | FYI kubajj, I've moved the inspector session to Wednesday because of a conflict | 14:53 |
kubajj | dtantsur: thanks, I think I should be able to make it | 14:54 |
JayF | thank you dtantsur; happy you'll be there | 14:54 |
* TheJulia sighs | 16:06 | |
TheJulia | off calls, and now to call to figure out flight options :( | 16:06 |
rpittau | bye everyone! o/ | 16:16 |
iurygregory | do we have support for UEFI HTTP boot? | 17:51 |
iurygregory | someone just asked me this if we have this type of support in redfsih when using ironic | 17:51 |
TheJulia | depends on which style, technically that is what ipxe+ipv6 is, fwiw | 17:51 |
iurygregory | humm ok | 17:52 |
TheJulia | but there is a version which is tell the BMC what url to use | 17:52 |
TheJulia | there is a DHCP version | 17:52 |
TheJulia | which we don't explicitly support unless it is v6 | 17:52 |
TheJulia | and ipv6, since we can hand back a full url | 17:52 |
iurygregory | I see | 17:52 |
iurygregory | thanks TheJulia | 17:52 |
TheJulia | Yeah, yay standardization or lack their of | 17:52 |
TheJulia | or multiple standards! | 17:52 |
JayF | Do I even need to link the xkcd here? | 17:53 |
TheJulia | in redfish would be via the bmc, and we really don't from the standpoint of "set a URL" | 17:53 |
TheJulia | was not widely supported when I did some wip patches for sushy a couple years back | 17:53 |
TheJulia | JayF: absolutely | 17:53 |
JayF | https://xkcd.com/927/ | 17:53 |
JayF | I can't wait for onefish, twofish, and bluefish to go with redfish ;) | 17:54 |
TheJulia | swordfish | 17:55 |
iurygregory | TheJulia, so in redfish scenario we don't have support for it? | 17:55 |
TheJulia | not kidding at all: https://www.snia.org/forums/smi/swordfish | 17:55 |
TheJulia | not for a direct url to boot from | 17:55 |
JayF | aw hell, can't they at least make it seussian :P | 17:56 |
* JayF afk | 17:56 | |
TheJulia | it wouldn't be super hard to add support for, and maybe then teach ironic to fallback via the DHCP path | 17:56 |
TheJulia | but setting via url is basically virtual media boot | 17:56 |
TheJulia | similar construct and mechanics | 17:56 |
stevebaker[m] | good morning | 19:15 |
TheJulia | Good morning stevebaker[m] | 19:16 |
TheJulia | stevebaker[m]: https://review.opendev.org/c/openstack/ironic/+/861305 would likely help with fixing older ci | 20:57 |
stevebaker[m] | Looking | 20:57 |
JayF | TheJulia: that commit message says train; but the patch is on ussuri | 21:01 |
TheJulia | oh, well, same exact change is required | 21:01 |
JayF | I think commit message here matters? At least it'd confuse me if I saw that on a !train branch :( | 21:02 |
TheJulia | waiting for it to laod to fix it | 21:02 |
JayF | ack ty | 21:02 |
opendevreview | Julia Kreger proposed openstack/ironic stable/ussuri: CI: Pin ussuri to use only basic standalone tests https://review.opendev.org/c/openstack/ironic/+/861305 | 21:06 |
kubajj | TheJulia: any suggestions on the local.conf? I tried adding some of the Q_* variables as iury suggested https://paste.opendev.org/show/btVaKadYmu5eSWwXux4E/ and I am getting a different error, but I am clueless at this point (the error: https://paste.opendev.org/show/bfAqH0g5ZkYQ6DQGVTMX/ ) | 21:09 |
JayF | > 503: Egress is over the account limit | 21:09 |
JayF | looks like you might have some quota at maximum? | 21:09 |
JayF | oh, wait, that's fomr github etcd | 21:09 |
JayF | that looks like etcd's download is broken upstream | 21:10 |
TheJulia | sigh | 21:10 |
JayF | kubajj: try re-running ./stack.sh with the exact same config on the exact same machine | 21:10 |
kubajj | JayF: this did not take even 30 seconds to fail | 21:11 |
JayF | yeah, if you look at that error log you posted | 21:11 |
JayF | it's just downloading and installing requirements | 21:12 |
JayF | and it dies early because the download from github fails | 21:12 |
JayF | not much devstack can do about that :( | 21:12 |
kubajj | I see | 21:12 |
kubajj | I will try another VM then | 21:12 |
JayF | don't use a nnew one | 21:12 |
JayF | if you can help it | 21:12 |
JayF | devstack should be eventually consistent; if it errors and you fix it you can re-run stack.sh and it'll just do what's needed | 21:13 |
kubajj | this is the error after re-running it: https://paste.opendev.org/show/bBOuZ5tkzHOvFFo9tMve/ | 21:14 |
JayF | kubajj: so lets read through it: it's saying ~kuba/devstack/files/etcd{}.tar.gz checksum doesn't match | 21:14 |
JayF | so that implies to me that it might have partially downloaded earlier when it failed | 21:15 |
JayF | so you might be able to remove that file and retry, so it re-fetches it | 21:15 |
kubajj | I don't know why it did not delete it when it says it in the paste above, but after calling the rm manually, it is running again now | 21:16 |
JayF | well that's basically what it's checking | 21:17 |
JayF | to see if the file already there is good | 21:17 |
JayF | in the gate; for instance, many of those things are baked into the image | 21:17 |
JayF | so it can just check to make sure the file is good and skip the download | 21:17 |
JayF | you don't get that cache for a single local devstack run :) | 21:17 |
TheJulia | is gerrit being painfully slow for anyone else? | 21:27 |
kubajj | I am not going to lie, neutron is slowly becoming my least favourite service | 21:28 |
kubajj | TheJulia: for me it is quite fast now | 21:28 |
TheJulia | hmm, now it is | 21:30 |
opendevreview | Julia Kreger proposed openstack/ironic master: Phase 3 - SQLAlchemy 2.0 Compatability https://review.opendev.org/c/openstack/ironic/+/857932 | 21:31 |
opendevreview | Merged openstack/ironic-python-agent bugfix/8.1: CI: Zuul no longer respects queue param https://review.opendev.org/c/openstack/ironic-python-agent/+/860194 | 22:17 |
opendevreview | Verification of a change to openstack/ironic-python-agent stable/wallaby failed: Use utf-16-le if BOM not present https://review.opendev.org/c/openstack/ironic-python-agent/+/861063 | 22:17 |
opendevreview | Verification of a change to openstack/ironic-python-agent stable/victoria failed: Use utf-16-le if BOM not present https://review.opendev.org/c/openstack/ironic-python-agent/+/861064 | 22:40 |
opendevreview | Merged openstack/ironic-python-agent-builder stable/xena: dhcp-all-interfaces: let NetworkManager doit. https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/860432 | 22:52 |
opendevreview | Verification of a change to openstack/ironic master failed: Use project scoped token for cinder, glance services https://review.opendev.org/c/openstack/ironic/+/861496 | 22:55 |
opendevreview | Merged openstack/ironic master: Phase 2 - SQLAlchemy 2.0 Compatability https://review.opendev.org/c/openstack/ironic/+/857516 | 23:27 |
opendevreview | Merged openstack/ironic master: Phase 3 - SQLAlchemy 2.0 Compatability https://review.opendev.org/c/openstack/ironic/+/857932 | 23:27 |
TheJulia | that... feels... good | 23:44 |
opendevreview | Merged openstack/ironic-python-agent master: Fix failure of bind mount in _install_grub2 https://review.opendev.org/c/openstack/ironic-python-agent/+/857271 | 23:46 |
opendevreview | Verification of a change to openstack/ironic stable/ussuri failed: CI: Pin ussuri to use only basic standalone tests https://review.opendev.org/c/openstack/ironic/+/861305 | 23:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!