openstackgerrit | David Hill proposed openstack/ironic-python-agent master: Skip nic numa_node discovery if it's not assigned to a numa_node https://review.opendev.org/702990 | 00:11 |
---|---|---|
openstackgerrit | David Hill proposed openstack/ironic-python-agent master: Skip nic numa_node discovery if it's not assigned to a numa_node https://review.opendev.org/702990 | 00:17 |
openstackgerrit | David Hill proposed openstack/ironic-python-agent master: Skip nic numa_node discovery if it's not assigned to a numa_node https://review.opendev.org/702990 | 00:27 |
*** adrianc_ has quit IRC | 00:30 | |
*** cjloader has quit IRC | 00:32 | |
*** adrianc has joined #openstack-ironic | 00:33 | |
*** yedongcan has joined #openstack-ironic | 00:50 | |
*** openstackgerrit has quit IRC | 00:57 | |
*** rloo has quit IRC | 00:59 | |
*** yedongcan has quit IRC | 01:12 | |
*** tzumainn has quit IRC | 01:12 | |
*** openstackgerrit has joined #openstack-ironic | 01:22 | |
openstackgerrit | David Hill proposed openstack/ironic-python-agent master: Skip nic numa_node discovery if it's not assigned to a numa_node https://review.opendev.org/702990 | 01:22 |
*** ociuhandu has joined #openstack-ironic | 01:31 | |
*** bnemec has quit IRC | 01:31 | |
*** ociuhandu has quit IRC | 01:35 | |
rpioso | etingof: I have some progress to share. tinycore boots successfully \o/ However, it appears IPA doesn't execute. Also the server has no IPv4 addresses assigned to its network interfaces, except for its loopback interface. | 01:50 |
*** gyee has quit IRC | 02:07 | |
*** jawad_axd has joined #openstack-ironic | 02:39 | |
*** jawad_axd has quit IRC | 02:43 | |
*** logan- has quit IRC | 02:48 | |
*** logan_ has joined #openstack-ironic | 02:50 | |
*** logan_ is now known as logan- | 02:50 | |
*** jawad_axd has joined #openstack-ironic | 03:00 | |
*** jawad_axd has quit IRC | 03:04 | |
*** TxGirlGeek has quit IRC | 03:08 | |
*** rh-jelabarre has quit IRC | 03:22 | |
*** mkrai_ has joined #openstack-ironic | 04:09 | |
*** mkrai_ has quit IRC | 04:47 | |
*** mkrai__ has joined #openstack-ironic | 04:47 | |
*** dustinc is now known as dustinc|PTO | 04:52 | |
*** TxGirlGeek has joined #openstack-ironic | 05:00 | |
*** zaneb has quit IRC | 05:08 | |
*** jdandrea has quit IRC | 05:23 | |
*** ociuhandu has joined #openstack-ironic | 05:30 | |
*** rpittau|afk has quit IRC | 05:34 | |
*** rpittau|afk has joined #openstack-ironic | 05:34 | |
*** ociuhandu has quit IRC | 05:35 | |
*** mkrai__ has quit IRC | 05:39 | |
*** mkrai_ has joined #openstack-ironic | 05:39 | |
*** mkrai_ has quit IRC | 05:40 | |
*** mkrai_ has joined #openstack-ironic | 05:40 | |
etingof | rpioso, good news! do yo have a DHCP server running on any of the networks local to the node? | 05:42 |
*** TxGirlGeek has quit IRC | 05:54 | |
*** TxGirlGeek has joined #openstack-ironic | 05:59 | |
*** TxGirlGeek has quit IRC | 06:10 | |
rpioso | etingof: devstack’s? | 06:56 |
mkrai_ | mgoddard, Hi o/ | 07:01 |
rpioso | etingof: Please note the ramdisk was built by stack.sh, because IRONIC_BUILD_DEPLOY_RAMDISK was set to True. That was probably unnecessary. Fetching a prebuilt tinyipa would have been faster. | 07:01 |
mkrai_ | good morning rpioso etingof | 07:01 |
rpioso | mkrai_: Good very early morning. | 07:02 |
mkrai_ | rpioso, :) | 07:02 |
* rpioso -> sleep | 07:03 | |
*** zaneb has joined #openstack-ironic | 07:09 | |
etingof | mkrai_, o/ | 07:17 |
*** samc-bbc has quit IRC | 07:18 | |
*** NobodyCam has quit IRC | 07:18 | |
*** trident has quit IRC | 07:18 | |
*** jrist has quit IRC | 07:19 | |
*** lifeless has quit IRC | 07:19 | |
*** JayF has quit IRC | 07:19 | |
*** ianw has quit IRC | 07:19 | |
*** mjturek has quit IRC | 07:19 | |
*** zigo has quit IRC | 07:19 | |
*** rnoriega_ has quit IRC | 07:19 | |
*** antonym has quit IRC | 07:19 | |
*** brtknr has quit IRC | 07:19 | |
etingof | rpioso, I guess your devstack resides at a different L2 broadcast domain than your baremetal. In that case devstack's dhcpd won't be reachable. So to have your baremetal on the network you either have a DHCP responder nearby or engage in dhcp-less deploy testing. ;-) | 07:19 |
*** openstackstatus has quit IRC | 07:20 | |
*** kiseok7 has joined #openstack-ironic | 07:23 | |
*** hwoarang has quit IRC | 07:34 | |
*** hwoarang_ has joined #openstack-ironic | 07:34 | |
arne_wiebalck | Good morning, ironic! | 07:47 |
sziviani | Good morninr, Ironic :) | 07:58 |
*** jawad_axd has joined #openstack-ironic | 08:04 | |
*** samc-bbc has joined #openstack-ironic | 08:05 | |
*** NobodyCam has joined #openstack-ironic | 08:05 | |
*** trident has joined #openstack-ironic | 08:05 | |
*** jrist has joined #openstack-ironic | 08:05 | |
*** lifeless has joined #openstack-ironic | 08:05 | |
*** JayF has joined #openstack-ironic | 08:05 | |
*** mjturek has joined #openstack-ironic | 08:05 | |
*** ianw has joined #openstack-ironic | 08:05 | |
*** zigo has joined #openstack-ironic | 08:05 | |
*** rnoriega_ has joined #openstack-ironic | 08:05 | |
*** antonym has joined #openstack-ironic | 08:05 | |
*** brtknr has joined #openstack-ironic | 08:05 | |
*** zzzeek has quit IRC | 08:08 | |
*** zzzeek has joined #openstack-ironic | 08:11 | |
*** tesseract has joined #openstack-ironic | 08:16 | |
*** rpittau|afk is now known as rpittau | 08:28 | |
rpittau | good morning ironic! o/ | 08:28 |
rpittau | TGIF! | 08:29 |
*** iurygregory has joined #openstack-ironic | 08:32 | |
iurygregory | good morning o/ | 08:32 |
rpittau | hey iurygregory :) | 08:33 |
iurygregory | rpittau, o/ | 08:33 |
*** dmellado has quit IRC | 08:34 | |
*** dmellado has joined #openstack-ironic | 08:35 | |
*** iurygregory_ has joined #openstack-ironic | 09:01 | |
*** iurygregory has quit IRC | 09:03 | |
*** khansa has joined #openstack-ironic | 09:03 | |
*** rcernin_ has joined #openstack-ironic | 09:04 | |
*** rcernin has quit IRC | 09:04 | |
*** Lucas_Gray has joined #openstack-ironic | 09:13 | |
*** lucasagomes has joined #openstack-ironic | 09:16 | |
*** k_mouza has joined #openstack-ironic | 09:17 | |
*** dougsz has joined #openstack-ironic | 09:18 | |
*** Lucas_Gray has quit IRC | 09:28 | |
*** Lucas_Gray has joined #openstack-ironic | 09:29 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [DNM] Test CI https://review.opendev.org/703043 | 09:33 |
openstackgerrit | Merged openstack/ironic-ui stable/stein: Fix horizon dependency https://review.opendev.org/701686 | 09:36 |
*** derekh has joined #openstack-ironic | 09:36 | |
*** ociuhandu has joined #openstack-ironic | 09:38 | |
*** dougsz has quit IRC | 09:38 | |
*** ociuhandu has quit IRC | 09:39 | |
*** mkrai_ has quit IRC | 09:39 | |
*** Lucas_Gray has quit IRC | 09:44 | |
*** kaifeng has quit IRC | 09:54 | |
*** alexmcleod has joined #openstack-ironic | 09:56 | |
*** ociuhandu has joined #openstack-ironic | 10:00 | |
*** Lucas_Gray has joined #openstack-ironic | 10:01 | |
*** dtantsur|afk is now known as dtantsur | 10:06 | |
dtantsur | morning ironic | 10:06 |
iurygregory_ | morning dtantsur | 10:08 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Skip nic numa_node discovery if it's not assigned to a numa_node https://review.opendev.org/702990 | 10:15 |
yolanda | hi etingof | 10:18 |
etingof | yolanda, o/ | 10:18 |
yolanda | http://paste.openstack.org/show/788520/ | 10:18 |
yolanda | those are the last bits i get on the conductor log | 10:18 |
*** ociuhandu has quit IRC | 10:19 | |
yolanda | from what i can see the return message of 2020-01-17 10:02:09.300 1 DEBUG sushy.connector [req-925faea9-6ee7-4c7b-ace1-f6f03dfa534d - - - - -] HTTP response for GET https://[fd35:919d:4042::1102]:443/redfish/v1/RegistryStore/registries/en/iLO.json: status code: 200 _op /usr/lib/python3.6/site-packages/sushy/connector.py:116 | 10:19 |
yolanda | are truncated | 10:19 |
yolanda | i tried that from command line but seems i get a binary.. | 10:20 |
etingof | yolanda, try curl -H "Accept: application/json" -H "Content-Type: application/json" | 10:23 |
yolanda | same | 10:23 |
etingof | to trick iLO into serving JSON | 10:23 |
yolanda | but from the python logs seems it's extracting ok | 10:23 |
etingof | yolanda, what's in the response headers? content-type-wise | 10:24 |
yolanda | cat header.txt | 10:25 |
yolanda | HTTP/1.1 200 OK | 10:25 |
yolanda | Allow: GET, HEAD | 10:25 |
yolanda | Cache-Control: no-cache | 10:25 |
yolanda | Content-Encoding: gzip | 10:25 |
yolanda | Content-type: application/json; charset=utf-8 | 10:25 |
yolanda | Date: Fri, 17 Jan 2020 10:20:55 GMT | 10:25 |
yolanda | ETag: "3FB1CBCD" | 10:25 |
yolanda | OData-Version: 4.0 | 10:25 |
yolanda | Transfer-Encoding: chunked | 10:25 |
yolanda | X-Frame-Options: sameorigin | 10:25 |
yolanda | X_HP-CHRP-Service-Version: 1.0.3 | 10:25 |
yolanda | gzip... | 10:25 |
etingof | so it it a gzipped json? | 10:25 |
etingof | curl ... | gzip -dc | 10:25 |
etingof | perhaps curl -s | 10:26 |
yolanda | ok i can get the content | 10:26 |
etingof | \o/ | 10:26 |
etingof | ... | python -m json.tool | 10:27 |
yolanda | big one, just a moment and i'll paste | 10:27 |
*** ociuhandu has joined #openstack-ironic | 10:28 | |
*** ociuhandu has quit IRC | 10:29 | |
* etingof imagines this fat json is getting pulled by ironic on power sync | 10:30 | |
yolanda | https://paste.centos.org/view/73426d32 | 10:32 |
yolanda | and the error i get from ironic node is | 10:32 |
yolanda | Failed to get power state for node xxx. Error: The attribute Messages/Description is missing from the resource /redfish/v1/RegistryStore/registries/en/iLO.json | 10:33 |
*** dougsz has joined #openstack-ironic | 10:34 | |
yolanda | but maybe due to that truncation that is seen in the logs? | 10:34 |
*** rcernin_ has quit IRC | 10:34 | |
*** Lucas_Gray has quit IRC | 10:38 | |
*** Lucas_Gray has joined #openstack-ironic | 10:42 | |
mgoddard | mkrai_: if you read this, see https://www.stackhpc.com/bare-metal-infiniband.html | 10:43 |
etingof | yolanda, I am still looking into this JSON madness... my guess is that there is some inconsistency either in sushy models or iLO message registry | 10:44 |
yolanda | why is that json picked and being used? | 10:45 |
yolanda | when i look at logs, i see lots of json files like that being downloaded | 10:45 |
etingof | right, that's the way how redfish is supposed to communicate BMC failures to the client | 10:46 |
*** ociuhandu has joined #openstack-ironic | 10:46 | |
etingof | BMC can fail in many curious ways, to describe at least some, they set up the context in anticipation of the upcoming failure | 10:47 |
yolanda | so we fail in preparation of that context | 10:47 |
etingof | ironically, the failure happens in preparation to describing the failure | 10:48 |
etingof | do you already like IPMI better? ;) | 10:48 |
yolanda | cannot be due to that truncation ? or does it just truncates on logs? | 10:49 |
yolanda | i apologize for all the times i complained about ipmi :) | 10:49 |
etingof | well, JSON is not chunkable by definition. if you attempt to chunk/cut JSON, you should get parsing failure | 10:51 |
*** ociuhandu has quit IRC | 10:51 | |
yolanda | yep, looking at the full logs, i see truncations in other places as well | 10:51 |
etingof | rather than this schema inconsistency problem | 10:51 |
etingof | so the good way to debug this is to take sushy in one hand, this iLO JSON into the other and merry these beasts | 10:52 |
etingof | they should explode in the same way as you observe under ironic | 10:52 |
etingof | a hackerish workaround could be to flip True to False in here -- https://github.com/openstack/sushy/blob/master/sushy/resources/registry/message_registry.py#L23 | 10:53 |
yolanda | mm look | 10:53 |
yolanda | https://paste.centos.org/view/73426d32#L6 | 10:53 |
yolanda | no Description there in the upper level | 10:54 |
yolanda | but in other queries i have: | 10:54 |
yolanda | http://paste.openstack.org/show/788522/ | 10:54 |
etingof | yes, but it does not seem to be required by the schema...? -- https://redfish.dmtf.org/schemas/v1/MessageRegistry.v1_1_1.json | 10:54 |
yolanda | there is a Description entry | 10:54 |
dtantsur | etingof: why do we even make registries required? | 10:54 |
etingof | dtantsur, because we are good citizens - I think that's what compliant redfish client has to do... | 10:55 |
dtantsur | in an ideal world - yes... | 10:58 |
dtantsur | given the reality of hardware, I think we should adopt the good old "be flexible with input and strict with output" motto | 10:58 |
*** Lucas_Gray has quit IRC | 10:59 | |
etingof | dtantsur, that means that schema violations would not be noticed so that sushy will explode later on unless we add some fuzziness to compensate for missing schema validation | 11:00 |
etingof | yolanda, so I am looking at MessageRegistry.properties.Description - this field is not among required | 11:00 |
yolanda | weird, why does it explode ?! | 11:01 |
dtantsur | you have a point there... | 11:01 |
etingof | yolanda, that's curious because in sushy that field is not required either -- https://github.com/openstack/sushy/blob/master/sushy/resources/registry/message_registry.py#L61 | 11:01 |
yolanda | etingof, is that the schema being picked? | 11:01 |
etingof | yolanda, supposedly, yes -- https://github.com/openstack/sushy/blob/master/sushy/resources/registry/message_registry.py#L14 | 11:02 |
dtantsur | so is it required or is it not in the end? | 11:03 |
* dtantsur is confused | 11:03 | |
*** ociuhandu has joined #openstack-ironic | 11:03 | |
yolanda | indeed, confusing | 11:04 |
*** Lucas_Gray has joined #openstack-ironic | 11:04 | |
etingof | there are two descriptions in the schema. one is required, the other is not | 11:04 |
yolanda | the inner one is required, right? | 11:04 |
etingof | yes | 11:04 |
yolanda | ah, then maybe some of those internal messages is missing a description? | 11:04 |
etingof | I am suspicious about this definition though -- https://github.com/openstack/sushy/blob/master/sushy/resources/registry/message_registry.py#L80 | 11:05 |
etingof | well, error message mentions 'Messages' object, not something inner | 11:05 |
yolanda | so some of the entries inside Messages dictionary may be missing a Description... | 11:06 |
etingof | yolanda, however, there are a lot of description-less messages in your paste -- https://paste.centos.org/view/73426d32#L1232 | 11:07 |
yolanda | then that's the issue | 11:07 |
yolanda | but that's what the hardware is giving... | 11:07 |
yolanda | seems it doesn't honor the schema then | 11:08 |
etingof | they also seem to miss the required Severity field | 11:08 |
yolanda | amazing | 11:08 |
etingof | yeah, perhaps | 11:08 |
yolanda | dtantsur, what do you do in those cases? are you reporting the vendor somehow? | 11:08 |
etingof | so we fail fast, what's the whole purpose of all these checks \o/ | 11:08 |
yolanda | all my boxes have the latest firmware | 11:09 |
dtantsur | well, in the ideal world we report to vendors and they fix it | 11:09 |
dtantsur | in our world, in the unlikely event of a particular luck, you'll get new firmware in a few months | 11:09 |
dtantsur | which means, we may have to hack around it... | 11:09 |
dtantsur | etingof: to avoid getting relaxed with the schema, I'd simply fall back to NOT using registries | 11:10 |
dtantsur | with an ugly warning | 11:10 |
dtantsur | after all, understanding registries is not 100% required for a client, as far as I can get | 11:10 |
etingof | yeah, perhaps that's a way out | 11:11 |
yolanda | do we have any point of contact for HPE redfish? | 11:11 |
etingof | however I seem to recall that we wanted these registries so much when BIOS work was on the table... | 11:11 |
dtantsur | yolanda: stendulker? (doesn't seem to be online) | 11:12 |
* dtantsur tries to remember | 11:12 | |
dtantsur | etingof: well, to me it's better to say "Error happened, we could not display the exact message because the registries implementation is non-compliant" | 11:13 |
dtantsur | rather than outright refusing to work with the machine | 11:13 |
etingof | arguably, dropping all registries because of a few malformed entries obscures the whole BMC reporting... | 11:13 |
dtantsur | etingof: are only a few entries malformed? | 11:13 |
dtantsur | then we can probably skip them... | 11:14 |
* dtantsur senses an ML thread | 11:14 | |
etingof | about a dozen according to my brain-resident JSON parser | 11:14 |
rpittau | dtantsur: hey, sorry for change of context, I don't know if you saw this https://opendev.org/openstack/diskimage-builder/commit/ae2be0b464392256df7009f7030281963c06aa81 dib is broken for Centos 8 until upper-constraints is updated, and by consequence IPA-B CI | 11:14 |
dtantsur | rpittau: yeah, I'm aware. There's nothing we can do but to wait :( | 11:14 |
rpittau | ok :/ | 11:14 |
dtantsur | This blends... "nicely".. into the discussion I wanted to have about voting DIB jobs :( | 11:15 |
rpittau | :D | 11:15 |
dtantsur | it's even worse that it's not DIB's fault | 11:15 |
dtantsur | we can gate on DIB changes, we cannot gate on CentOS changing underneath | 11:15 |
rpittau | yep, exactly | 11:15 |
rpittau | that change was subtle | 11:16 |
etingof | skipping bad messages would be less harmful indeed, however sushy object model is not well-suited for that kind of constraint relaxation. it's currently all or nothing | 11:16 |
etingof | but I guess yolanda should have a way forward once True becomes False there... | 11:17 |
dtantsur | etingof: well, it may boil down to one (optional) try..except in our list implementation? | 11:17 |
dtantsur | here https://opendev.org/openstack/sushy/src/branch/master/sushy/resources/base.py#L209 | 11:18 |
etingof | that will affect all fields of that type in all JSON documents | 11:18 |
dtantsur | we can make it optional | 11:19 |
dtantsur | like ListField('Name', skip_broken_elements=True) | 11:19 |
dtantsur | etingof: ^^ | 11:20 |
etingof | that's a way, however it may lead to some duplication in definitions effectively making them context-specific... generally speaking | 11:23 |
dtantsur | I'm not sure I get your comment | 11:23 |
dtantsur | I don't quite envision any duplications.. | 11:24 |
dtantsur | shell I make a patch to show the approach? | 11:24 |
etingof | e.g. if we presently have a single Message definition used everywhere, then we discover that in one particular document is can be broken we'd have to introduce another Message which is exactly the same but with this field flipped | 11:24 |
etingof | your proposal is clear to me | 11:25 |
dtantsur | etingof: do we use Messages in contexts other than error reporting? | 11:25 |
dtantsur | in other words, how safe is it to just change the Message definition for everything? | 11:26 |
*** ociuhandu_ has joined #openstack-ironic | 11:26 | |
etingof | I do not know that readily | 11:26 |
etingof | I am just trying to evaluate the approach | 11:27 |
*** ociuhandu_ has quit IRC | 11:28 | |
*** ociuhandu has quit IRC | 11:28 | |
dtantsur | okay, what other options do we have? | 11:28 |
etingof | another possible hack might be to default missing description | 11:28 |
*** ociuhandu has joined #openstack-ironic | 11:28 | |
dtantsur | do we have a sane default for it? | 11:28 |
etingof | default to some swearing of course | 11:28 |
dtantsur | :D | 11:28 |
dtantsur | sigh, I'm so sad we cannot do it | 11:29 |
yolanda | but etingof , you mentioned another field not being honored right, not just Description | 11:29 |
etingof | thanks to OEM, we can even expand vendor IDs there | 11:29 |
dtantsur | ++ | 11:29 |
dtantsur | do we have a default Severity? | 11:29 |
etingof | yeah, that's my impression that Severity is required by the schema | 11:30 |
etingof | perhaps - low to minimize the impact | 11:30 |
dtantsur | it's a very valid impression, without Severity we cannot even distinguish errors from success | 11:30 |
dtantsur | mmm, judging by the code it's more of Ok, Warning, Error, etc | 11:30 |
dtantsur | Ok, Warning, Critical, to be precise. if so, there's no default. | 11:31 |
etingof | of course, not. may be we can add "warning" as a default in case it's missing... heh | 11:32 |
dtantsur | this ^^ will not work well for BIOS, will it? we need to detect errors there. | 11:32 |
etingof | sure, but the alternative is to drop the entire message(s) if Severity is not there | 11:33 |
dtantsur | we could default to some Unknown severity. then if we actually see a record referencing it, we raise an error. | 11:33 |
dtantsur | mmm, I don't explain it well, I guess... | 11:33 |
etingof | I got it I think | 11:33 |
dtantsur | I mean, we do parse the registries, we accept Severity==None. But if we get a message referencing it, we raise an error, mentioning both the message and the fact that severity is unknown. | 11:34 |
etingof | yes, how is it different from having Severity-Critical? | 11:34 |
dtantsur | an additional message that we could not parse the severity? | 11:35 |
dtantsur | so that we don't end up with "Error: operation ended successfully" :) | 11:35 |
*** jawad_axd has quit IRC | 11:35 | |
*** Lucas_Gray has quit IRC | 11:35 | |
etingof | yeah, alternatively, we could add something to Description... ;) | 11:35 |
etingof | "Error: operation ended successfully. Replace your vendor and hit any key..." | 11:36 |
yolanda | go back to IPMI! | 11:36 |
dtantsur | :D | 11:36 |
dtantsur | :D :D :D | 11:36 |
dtantsur | "Error: computers were a mistake, go do some real life fishing instead" | 11:37 |
etingof | so how critical is this for yolanda to get working? | 11:37 |
*** Lucas_Gray has joined #openstack-ironic | 11:38 | |
etingof | I mean, would a local hack suffice for the time being? | 11:38 |
yolanda | it is critical as the boxes don't start . But i can go with a local hack | 11:40 |
etingof | I can probably propose some official hackaround as discussed here as time permits | 11:42 |
dtantsur | etingof: it would be great. If new HPE boxes don't work at all... it's quite a priority. | 11:44 |
* dtantsur will review | 11:44 | |
etingof | sure | 11:45 |
dtantsur | thanks! | 11:45 |
* etingof is presently busy telling the world how Redfish is better than IPMI | 11:45 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: [WIP] use openstacksdk for ironic module https://review.opendev.org/672179 | 11:50 |
*** pcaruana has joined #openstack-ironic | 11:59 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/train: Respect driver_info.force_persistent_boot_device https://review.opendev.org/703062 | 12:02 |
yolanda | etingof, don't mention the schema issues then! | 12:09 |
* etingof is not sure what yolanda means here | 12:10 | |
*** jawad_axd has joined #openstack-ironic | 12:10 | |
etingof | ah, regarding redfish superiority?! | 12:10 |
*** dsneddon has quit IRC | 12:12 | |
*** mkrai_ has joined #openstack-ironic | 12:13 | |
dtantsur | hehe | 12:13 |
dtantsur | "Redfish works awesome.. when YOU implement it RIGHT!!" | 12:14 |
* etingof can's resist reminding dtantsur of managers<->virtual media feature | 12:15 | |
dtantsur | don't remind me, I still don't like it :) | 12:15 |
*** ociuhandu has quit IRC | 12:23 | |
*** quoth has joined #openstack-ironic | 12:23 | |
*** Lucas_Gray has quit IRC | 12:24 | |
*** Lucas_Gray has joined #openstack-ironic | 12:26 | |
quoth | Is there a current list of known good host OS's for installing bifrost to? I have tried CentOS today but pip is failing to install python-ironicclient because more_itertools doesn't support python 2 anymore. So I also tried to switch to python3 but that is failing because the ansible playbooks are trying to install python2 libs since it is CentOS 7 | 12:31 |
quoth | * I have tried CentOS 7 .. | 12:31 |
dtantsur | quoth: which version of bifrost are you installing? Train was the last to support Python 2. | 12:36 |
quoth | I have hit the issue using both the stable/stein and stable/train branches | 12:36 |
dtantsur | hmmm | 12:36 |
dtantsur | quoth: these are supposed to work with python 2. could you show us the exact error you're hitting (feel free to use paste.openstack.org) | 12:37 |
dtantsur | more-itertools seems properly constrained https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L491-L495 | 12:38 |
quoth | http://paste.openstack.org/raw/788530/ | 12:38 |
quoth | That is the output from the failing ansible task | 12:39 |
dtantsur | OUCH https://github.com/openstack/bifrost/blob/e2a919586272946ca081b597014be86d370ceff8/playbooks/roles/bifrost-ironic-install/tasks/install.yml#L90-L91 | 12:39 |
*** iurygregory_ is now known as iurygregory | 12:40 | |
dtantsur | TheJulia: ^^^ | 12:40 |
*** dmellado has quit IRC | 12:41 | |
dtantsur | quoth: so, it's a legitimate issue. I wonder if you can work around it by setting ironicclient_source_install to true. | 12:44 |
dtantsur | answering myself: no, probably not | 12:44 |
*** dmellado has joined #openstack-ironic | 12:44 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Use upper-constraints for ironicclient and sushy https://review.opendev.org/703069 | 12:47 |
dtantsur | quoth: try this patch ^^^ let me know if it works please | 12:47 |
quoth | dtantsur: Will do. I need to head out of the office for a while but I will try the patch this afternoon and let you know | 12:47 |
quoth | thanks | 12:47 |
dtantsur | cool | 12:47 |
*** dsneddon has joined #openstack-ironic | 12:48 | |
*** rh-jelabarre has joined #openstack-ironic | 12:51 | |
mkrai_ | Hi, have anyone tried running infiniband with ironic? | 12:53 |
mkrai_ | I am facing issue where cloud-init fails to configure network interfaces on the node because of the fake mac address of IB port | 12:55 |
*** jdandrea has joined #openstack-ironic | 12:55 | |
*** dsneddon has quit IRC | 12:56 | |
*** ociuhandu has joined #openstack-ironic | 12:57 | |
*** jdandrea has quit IRC | 12:57 | |
*** ociuhandu has quit IRC | 13:03 | |
*** dsneddon has joined #openstack-ironic | 13:06 | |
*** mmethot has quit IRC | 13:08 | |
* rpittau needs more coffee | 13:13 | |
*** mkrai_ has quit IRC | 13:19 | |
*** dougsz has quit IRC | 13:22 | |
*** ricolin has joined #openstack-ironic | 13:30 | |
*** Lucas_Gray has quit IRC | 13:31 | |
*** derekh has quit IRC | 13:32 | |
arne_wiebalck | rpittau: dtantsur TheJulia On (manual, ie. recurring) inspection and inventory: from what I see, there are no means at the moment to collect a time series of inspection data. New data for a node simply overwrites old data. We can solve this on the backend with object versioning, but then access to this data also becomes backend specific (and different from accessing the latest data). | 13:32 |
arne_wiebalck | Would it make sense to consider to make the inspector more sophisticated when storing data into its backend? | 13:33 |
arne_wiebalck | Having a time series would be neat to track changes, such as serial numbers or f/w versions. | 13:34 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Use variables for ssh paths https://review.opendev.org/697654 | 13:39 |
*** kozhukalov has joined #openstack-ironic | 13:39 | |
dtantsur | arne_wiebalck: this sounds interesting. I'd read an RFE on that. My first concern would be garbage collection, especially when used with the DB backend. | 13:41 |
arne_wiebalck | dtantsur: sth to discuss at the mid cycle maybe? | 13:42 |
rpittau | arne_wiebalck: probably an interesting discussion for the mid cycle, yeah | 13:43 |
*** pcaruana has quit IRC | 13:45 | |
rpittau | mmmm review.opendev.org is not working for me | 13:45 |
rpittau | oh nvm, it came back | 13:46 |
yolanda | hi etingof , i'm getting this now : Failed to get power state for node 312a7e06-79ed-412b-93af-b5a5154de71d. Error: Redfish exception occurred. Error: Resource https://[xxxx]:443/redfish/v1/Systems/System.Embedded.1 not found | 13:48 |
yolanda | how can i know the url to use? | 13:48 |
yolanda | mm seems it's /redfish/v1/Systems/1/ . Does it change depending on vendor? | 13:52 |
*** mkrai_ has joined #openstack-ironic | 13:53 | |
dtantsur | arne_wiebalck: good idea | 13:57 |
arne_wiebalck | rpittau: dtantsur thanks, I'll add this to the etherpad | 13:57 |
dtantsur | ++ | 13:57 |
etingof | yolanda, it is up to the vendors to enumerate their systems | 13:58 |
*** rloo has joined #openstack-ironic | 13:58 | |
etingof | yolanda, the name can be anything, but it obviously needs to be consistent link-wise | 13:58 |
etingof | yolanda, so if the official name is /redfish/v1/Systems/1 that's alright, but then it should be '1' everywhere else or BMC should be able to accept non-canonical names as well | 13:59 |
etingof | yolanda, however, may be that '1' is coming from your node config rather than from some hyperlink? | 14:02 |
*** derekh has joined #openstack-ironic | 14:02 | |
etingof | s/1/System.Embedded.1/ | 14:03 |
etingof | you should have /redfish/v1/Systems/System.Embedded.1 in your ironic driver_info | 14:03 |
TheJulia | good morning | 14:03 |
etingof | good morning, TheJulia o/ | 14:04 |
TheJulia | dtantsur: for python-ironicclient, we likely can now apply constraints | 14:04 |
dtantsur | TheJulia: cool, see https://review.opendev.org/#/c/703069/ (and the concern rpittau voiced) | 14:05 |
patchbot | patch 703069 - bifrost - Use upper-constraints for ironicclient and sushy - 1 patch set | 14:05 |
dtantsur | and good morning :) | 14:05 |
rpittau | good morning TheJulia :) | 14:05 |
TheJulia | dtantsur: should be fine now, the ansible stuff doesn't even use python-ironicclient anymore | 14:05 |
dtantsur | I'll propose a train backport to see it on a working CI | 14:05 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/train: Use upper-constraints for ironicclient and sushy https://review.opendev.org/703086 | 14:06 |
yolanda | etingof, i checked /redfish/v1/systems and i got : {"@odata.context":"/redfish/v1/$metadata#ComputerSystemCollection.ComputerSystemCollection","@odata.etag":"W/\"AA6D42B0\"","@odata.id":"/redfish/v1/Systems/","@odata.type":"#ComputerSystemCollection.ComputerSystemCollection","Description":"Computer Systems view","Name":"Computer Systems","Members":[{"@odata.id":"/redfish/v1/Systems/1/"}],"Members@odata.count":1} | 14:12 |
etingof | so it seems '1' is the canonical name | 14:13 |
*** dougsz has joined #openstack-ironic | 14:13 | |
etingof | then you should probably have it in your driver_info, however the question is where System.Embedded.1 is coming from? | 14:14 |
yolanda | i think i copied that from derekh | 14:14 |
etingof | ah, perhaps Dell does not mix well with HP! | 14:14 |
*** ociuhandu has joined #openstack-ironic | 14:14 | |
derekh | etingof: ya, I got it from your Dell notes | 14:15 |
yolanda | that url allowed me to progress a bit more... | 14:15 |
yolanda | mm now | 14:16 |
yolanda | | last_error | Failed to prepare to deploy: Redfish exception occurred. Error: Setting boot mode to uefi failed for node 9d2c305e-4753-4a06-8e70-f1affd6f6a67. Error: The parameter "target" value "None" is invalid. Valid values are: {'utilities', 'diags', 'usb', 'hdd', 'pxe', 'uefi shell', 'cd', 'none', 'bios setup'} | 14:16 |
yolanda | derekh, you know? uefi shell? | 14:16 |
*** hjensas has quit IRC | 14:19 | |
etingof | yolanda, this is coming from ironic... when it tries to set both boot mode and boot device at once (because Redfish). seems like boot device is somehow missing... | 14:19 |
*** k_mouza has quit IRC | 14:20 | |
* derekh is glad etingof had an answer | 14:20 | |
*** k_mouza has joined #openstack-ironic | 14:20 | |
yolanda | do i need to instruct to use boot device somehow? | 14:21 |
etingof | not really, iirc it should be taken from the currently set boot device and applied back (together with the new boot mode) | 14:21 |
etingof | let me dig into the scary depths of that code | 14:22 |
yolanda | mm i think this machine is set to bios boot, not uefi. Can it be a problem? | 14:22 |
yolanda | but that's the point of that code right? changing the boot mode.... | 14:22 |
*** pcaruana has joined #openstack-ironic | 14:22 | |
etingof | yes, ironic tried to change boot mode | 14:23 |
etingof | btw, if you set the node to uefi by hand, chances are that would get around that error | 14:23 |
yolanda | cheating! | 14:24 |
yolanda | but i prefer to cheat :) | 14:24 |
* etingof is just back from fishing with this trophy -- https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/redfish/management.py#L215 | 14:25 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Skip nic numa_node discovery if it's not assigned to a numa_node https://review.opendev.org/702990 | 14:26 |
etingof | I think `target` is None because boot device is not set in the Systems/1 document | 14:27 |
*** ociuhandu has quit IRC | 14:27 | |
yolanda | let me check that output | 14:27 |
yolanda | seems HPE is not behaving so well with that documents... | 14:27 |
*** mkrai_ has quit IRC | 14:28 | |
etingof | so may be if you get your node reporting somehow reporting any boot device via redfish system object, that would get you to the next problem | 14:28 |
dtantsur | etingof: oh, have you seen https://bugzilla.redhat.com/show_bug.cgi?id=1791392? Looks quite similar to what yolanda saw. | 14:29 |
openstack | bugzilla.redhat.com bug 1791392 in rhosp-director "Redfish PM driver not working with Supermicro" [Unspecified,New] - Assigned to rhos-maint | 14:29 |
TheJulia | I wonder if the hardware is defaulted so setting boot mode gets weird | 14:29 |
etingof | dtantsur, I think HP made it a bit further than Supermicro - the latter misses the top-level attribute | 14:30 |
yolanda | https://paste.centos.org/view/93c7c359#L1 | 14:30 |
dtantsur | etingof: lovely... | 14:30 |
dtantsur | we have to start being more relaxed about it indeed | 14:31 |
yolanda | so that's the content i get from Systems/1 ... does it help ? | 14:31 |
*** mkrai_ has joined #openstack-ironic | 14:32 | |
etingof | yolanda, yes, hold on | 14:32 |
*** ociuhandu has joined #openstack-ironic | 14:33 | |
dtantsur | etingof: I put the bug on your plate as well, it can probably be fixed in the same patch.. | 14:33 |
yolanda | etingof, i'm going to be your nightmare today :) | 14:34 |
yolanda | going for lunch | 14:34 |
etingof | yolanda, do you have the PATCH call in ironic debug log that leads to this failure? | 14:35 |
*** mmethot has joined #openstack-ironic | 14:35 | |
*** aedc has joined #openstack-ironic | 14:36 | |
etingof | you will be competing with jsonschema in my nightmares | 14:36 |
*** ociuhandu has quit IRC | 14:37 | |
dtantsur | "the schema of my nightmares" | 14:38 |
*** ociuhandu has joined #openstack-ironic | 14:38 | |
TheJulia | dtantsur: I remember arguments about that, but yeah, definitely need to :\ | 14:48 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Use upper-constraints for ironicclient, sushy and DIB https://review.opendev.org/703069 | 14:51 |
dtantsur | TheJulia, rpittau, borrowed the hack that I'm using in the metalsmith CI ^^^ | 14:51 |
* dtantsur updates the train patch to test it | 14:51 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/train: Use upper-constraints for ironicclient, sushy and DIB https://review.opendev.org/703086 | 14:52 |
*** mkrai_ has quit IRC | 14:52 | |
*** jdandrea has joined #openstack-ironic | 14:53 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent stable/train: Skip nic numa_node discovery if it's not assigned to a numa_node https://review.opendev.org/703096 | 14:54 |
*** TxGirlGeek has joined #openstack-ironic | 14:55 | |
*** aedc has quit IRC | 14:55 | |
yolanda | etingof, let me see logs | 14:57 |
*** dougsz has quit IRC | 14:58 | |
yolanda | where should i see that? i don't see anything relevant on conductor, just lots of calls to /Systems/1 url | 14:59 |
yolanda | i searched conductor logs for PATCH and cannot see anything | 14:59 |
etingof | there should be PATCH against Systems/1 | 15:00 |
etingof | alright then | 15:00 |
yolanda | it may not be logged ... just GET calls | 15:00 |
etingof | right, yes | 15:00 |
yolanda | looking at the paste... there doesn't seem to exist the uefi mode | 15:02 |
etingof | yolanda, I think the failure comes from here -- https://github.com/openstack/sushy/blob/master/sushy/resources/system/system.py#L229 | 15:02 |
yolanda | just | 15:02 |
yolanda | "BootSourceOverrideTarget": "None", "BootSourceOverrideTarget@Redfish.AllowableValues": ["None", "Cd", "Hdd", "Usb", "Utilities", "Diags", "BiosSetup", "Pxe", "UefiShell"]} | 15:02 |
etingof | that's not about boot mode, but about non-default boot device | 15:03 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: WIP - Native zuulv3 grenade job for ironic https://review.opendev.org/703098 | 15:04 |
*** ociuhandu has quit IRC | 15:04 | |
etingof | yolanda, let me debug sushy against your json | 15:04 |
*** jawad_axd has quit IRC | 15:05 | |
yolanda | what's the key for default boot device? looking at sushy mappings but cannot see it | 15:05 |
etingof | yolanda, here you go -- https://github.com/openstack/sushy/blob/master/sushy/resources/system/constants.py#L77 | 15:07 |
yolanda | i have that on my paste ... "BootSourceOverrideTarget": "None", | 15:07 |
*** jawad_axd has joined #openstack-ironic | 15:08 | |
etingof | yes, that makes me suspect sushy bug. but let me debug it, I do not see what's wrong so far | 15:08 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Use upper-constraints for ironicclient, sushy and DIB https://review.opendev.org/703069 | 15:12 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/train: Use upper-constraints for ironicclient, sushy and DIB https://review.opendev.org/703086 | 15:13 |
* dtantsur cannot ansible, send help | 15:13 | |
*** jawad_ax_ has joined #openstack-ironic | 15:13 | |
*** jawad_axd has quit IRC | 15:13 | |
* rpittau needs much more coffee | 15:13 | |
* TheJulia appears with her ears in conference calls | 15:13 | |
*** hwoarang_ is now known as hwoarang | 15:16 | |
*** jawad_ax_ has quit IRC | 15:18 | |
etingof | yolanda, are you running tips of ironic/sushy master branches? | 15:19 |
yolanda | everything is master, i built the image locally | 15:20 |
*** kozhukalov has quit IRC | 15:23 | |
etingof | yolanda, ack, on it | 15:24 |
yolanda | cool thx | 15:24 |
*** dougsz has joined #openstack-ironic | 15:26 | |
etingof | yolanda, the bug is here (do not look, it's too weird for a sane human being) -- https://github.com/openstack/sushy/blob/master/sushy/resources/base.py#L109 | 15:29 |
*** mkrai_ has joined #openstack-ironic | 15:29 | |
quoth | dtantsur: Was going to say I tried the patch for bifrost with no luck but I see from above you are uploading more patches for the issue so I shall keep an eye on them. | 15:29 |
etingof | effectively, this 'None' literal from JSON document gets mixed up with Python 'None' type representation not normalized as it should | 15:29 |
etingof | tl;dr; sushy does not work well with anything 'None' | 15:30 |
etingof | yolanda, could you please set you machine to boot from anything other than None? | 15:30 |
*** ociuhandu has joined #openstack-ironic | 15:31 | |
*** ociuhandu has quit IRC | 15:32 | |
*** ociuhandu has joined #openstack-ironic | 15:32 | |
yolanda | doh! what a bug | 15:34 |
yolanda | ok i'll do | 15:35 |
yolanda | mm my boot settings are boot from legacy bios | 15:38 |
yolanda | and then in server boot order i have cd/dvd, usb, hard disk, network device | 15:38 |
yolanda | i can try changing from bios to uefi | 15:39 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Use upper-constraints for ironicclient, sushy and DIB https://review.opendev.org/703069 | 15:41 |
etingof | yolanda, actually, I am suggesting that you have BootOverrideTarget set to anything other than None | 15:41 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/train: Use upper-constraints for ironicclient, sushy and DIB https://review.opendev.org/703086 | 15:42 |
etingof | alternatively, you can avoid this whole code path by setting your node to uefi | 15:42 |
dtantsur | quoth: yeah, I could not make it right from the first attempt. Getting there though. | 15:42 |
etingof | so you have a variety of cheating opportunities | 15:43 |
yolanda | etingof, i was looking at the ilo 5 web... BootOverrideTarget is one time boot? not sure where i should set it | 15:43 |
yolanda | i need to step out for a while, bbl | 15:45 |
*** mkrai_ has quit IRC | 15:50 | |
etingof | yolanda, it is only one-time if BootSourceOverrideMode=Once, otherwise it's forever | 15:50 |
*** aedc has joined #openstack-ironic | 15:52 | |
*** TxGirlGeek has quit IRC | 15:53 | |
*** jawad_axd has joined #openstack-ironic | 15:54 | |
*** jawad_axd has quit IRC | 15:59 | |
*** jawad_axd has joined #openstack-ironic | 16:15 | |
*** TxGirlGeek has joined #openstack-ironic | 16:19 | |
*** jawad_axd has quit IRC | 16:19 | |
etingof | yolanda, from redfish perspective, BootOverrideTarget setting overrides "main" boot device which is configured somewhere else, perhaps in BIOS/nvram. I feel that this booting business is complicated, may be that explains the cascading of these settings... | 16:24 |
*** belmoreira has joined #openstack-ironic | 16:24 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Use upper-constraints for ironicclient, sushy and DIB https://review.opendev.org/703069 | 16:25 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/train: Use upper-constraints for ironicclient, sushy and DIB https://review.opendev.org/703086 | 16:26 |
dtantsur | The hacks are piling up :( | 16:26 |
*** gyee has joined #openstack-ironic | 16:27 | |
TheJulia | :( | 16:29 |
etingof | on the other hand, genetically speaking, we are piles of hacks and workarounds | 16:30 |
dtantsur | we are for sure | 16:30 |
dtantsur | computers are getting close though | 16:30 |
dtantsur | TheJulia: does the current version look at least remotely acceptable? (I'm not yet sure if it works) | 16:31 |
TheJulia | dtantsur: looking | 16:42 |
rpittau | dtantsur: I was going to say, if you change the extra_args logic, you need to change it everywhere | 16:42 |
TheJulia | not 1000% sure, but I _think_ so | 16:43 |
dtantsur | rpittau: not necessary, this part is only used for ironic installation | 16:43 |
dtantsur | otherwise this patch is going to blow up | 16:43 |
TheJulia | cp pip_install.yml ../.../bifrost-keystone-install/tasks/ | 16:43 |
dtantsur | ideally, yes. practically we seem to only have problem with ironicclient | 16:43 |
rpittau | dtantsur: oh yeah, my bad, I mean everywhere you use pip_install.yaml | 16:43 |
dtantsur | since we'll have to backport this patch all the way, I'd prefer to minimize the scope of it | 16:44 |
dtantsur | rpittau: what exactly place do you have in mind? | 16:44 |
dtantsur | (my only concern now is ansible scoping rules and extra_args bleeding all over the place) | 16:45 |
rpittau | dtantsur: that exact file :) my comment is old, I think it should work like that | 16:45 |
dtantsur | ah | 16:45 |
*** goldyfruit has joined #openstack-ironic | 16:45 | |
goldyfruit | Hi guys | 16:46 |
TheJulia | o/ long time no see | 16:46 |
TheJulia | goldyfruit: hope things are well | 16:46 |
dtantsur | goldyfruit: o/ | 16:46 |
goldyfruit | TheJulia all good! I'm working for IBM now | 16:46 |
goldyfruit | Red Hat -> Ormuco -> IBM | 16:46 |
TheJulia | goldyfruit: welcome to the fold... | 16:47 |
goldyfruit | Kind of back to Red Hat ? :p | 16:47 |
TheJulia | Kind of... does that mean you get purple shirts? | 16:47 |
* TheJulia is actually wearing a purple RH shirt this morning | 16:47 | |
goldyfruit | There are not there yet I guess :p | 16:47 |
dtantsur | :D | 16:47 |
goldyfruit | I have a question about fast-track, I found the RFE but I'm not sure to understand what is used for. Could you guys could give me some light on it, please? | 16:49 |
TheJulia | Heh, I need to look at a bug about it today | 16:50 |
TheJulia | so the idea with fast track is to allow an operator to choose to let their ironic deployment move machines between certain states where we know we likely don't really need to power down. | 16:50 |
TheJulia | so If cleaning and provisioning network are the same, then fast track would allow the node to be left powered on with the agent heartbeating such that the deploy could start upon the next heartbeat and we don't need to wait for the power-on self test to complete | 16:51 |
rpittau | bye all, have a nice weekend! o/ | 16:53 |
*** rpittau is now known as rpittau|afk | 16:53 | |
goldyfruit | TheJulia so basically the cleaning and the provisioning will appends on the same power-on cycle ? | 16:53 |
dtantsur | and inspection, if used | 16:54 |
goldyfruit | Oh!!! | 16:54 |
TheJulia | Did we do inspection too? | 16:54 |
TheJulia | This might be why my last fix didn't take care of it! | 16:54 |
TheJulia | goldyfruit: great timing! | 16:54 |
dtantsur | TheJulia: I think so? It requires some configuration, but is doable in theory. | 16:55 |
goldyfruit | That will save me some time on Dell and HPE which are pretty long to boot due the BIOS post blablablabla | 16:55 |
dtantsur | goldyfruit: here goes my regular complaint that vendors don't provide such thing as "hot reboot". | 16:55 |
dtantsur | would be life-changing for ironic | 16:55 |
TheJulia | ++++ | 16:55 |
goldyfruit | I agree! | 16:56 |
goldyfruit | Thanks for the light guys! | 16:56 |
TheJulia | goldyfruit: so long story, abbreviated, there may be bugs there. We did it for the standalone use case really and... yeah. | 16:56 |
dtantsur | :) | 16:56 |
TheJulia | but it is also increadibly simple | 16:57 |
dtantsur | there'll be bugs until people try it and we fix it :) | 16:57 |
TheJulia | exactly | 16:57 |
TheJulia | other use patterns that we never saw since things can vary so much | 16:57 |
dtantsur | on this positive note, I need to go enjoy the queues in the Friday evening supermarket :) | 16:57 |
dtantsur | have a wonderful weekend | 16:57 |
*** dtantsur is now known as dtantsur|afk | 16:58 | |
TheJulia | dtantsur: enjoy! | 16:58 |
goldyfruit | dtantsur|afk o/ | 16:58 |
* etingof thinks that you guys underestimate the significance of a healthy reboot for the whole world not to fall apart | 16:58 | |
*** quoth has left #openstack-ironic | 16:58 | |
* dtantsur|afk removes standby functionality from etingof's laptop and leaves | 16:58 | |
TheJulia | ++ | 16:58 |
* TheJulia wonders if that removal was just a change to suspend to disk | 16:59 | |
*** lucasagomes has quit IRC | 16:59 | |
iurygregory | TheJulia, can you put https://review.opendev.org/#/c/696914/ on your list? *-* | 16:59 |
patchbot | patch 696914 - ironic-python-agent - Avoid grub2-install when on UEFI boot mode - 16 patch sets | 16:59 |
TheJulia | iurygregory: I think it already is on my list | 16:59 |
TheJulia | iurygregory: Haven't looked yet today though... early morning meeting prep and meetings \o/ | 17:00 |
iurygregory | coffee first! | 17:00 |
iurygregory | =D | 17:00 |
*** dougsz has quit IRC | 17:04 | |
TheJulia | well, I coffeeed now email and | 17:04 |
TheJulia | wow... I just noticed the time | 17:04 |
*** alexmcleod has quit IRC | 17:05 | |
TheJulia | sadly bigpulse does not provide insight into the data as a user :) | 17:06 |
*** goldyfruit has quit IRC | 17:10 | |
*** goldyfruit has joined #openstack-ironic | 17:11 | |
arne_wiebalck | bye everyone o/ | 17:21 |
*** ociuhandu_ has joined #openstack-ironic | 17:25 | |
*** ociuhandu has quit IRC | 17:27 | |
*** ociuhandu_ has quit IRC | 17:29 | |
*** k_mouza has quit IRC | 17:32 | |
*** aedc has quit IRC | 17:32 | |
etingof | the wonders of caffeinated brainpower -- https://review.opendev.org/703133 | 17:35 |
patchbot | patch 703133 - nova - [WIP] Add JSON schema for network_data.json - 1 patch set | 17:35 |
* etingof is wondering if some sort of unit testing would be proper? | 17:35 | |
*** afasano has quit IRC | 17:47 | |
openstackgerrit | khansa proposed openstack/sushy-cli master: Update toxi.ini file https://review.opendev.org/703139 | 17:48 |
*** afasano has joined #openstack-ironic | 17:48 | |
TheJulia | etingof: most likely just that data gets passed through | 17:49 |
etingof | there are tools that can generate data samples based on the schema | 17:50 |
etingof | so we can take these samples and wrap into a unitest runner | 17:50 |
TheJulia | yeah, but is the data being massaged anywhere the nova.virt.ironic.* code paths? | 17:50 |
TheJulia | because if not, doing full schema seems... like overkill | 17:51 |
*** khansa has quit IRC | 17:51 | |
*** khansa has joined #openstack-ironic | 17:51 | |
etingof | this schema covers exactly the things Nova can produce (minus IPv6, I run out of coffee to finish that off) | 17:52 |
*** khansa has quit IRC | 17:56 | |
TheJulia | etingof: yolanda: I _feel_ like the translation registry entry should be optional, but I might be completely off my rocking chair | 17:58 |
*** aedc has joined #openstack-ironic | 17:59 | |
*** derekh has quit IRC | 18:00 | |
etingof | what exactly do you mean by 'translation registry entry'? that missing 'description' attribute? | 18:00 |
*** ociuhandu has joined #openstack-ironic | 18:03 | |
*** aedc has quit IRC | 18:06 | |
*** tenbrae has joined #openstack-ironic | 18:08 | |
TheJulia | rpioso: etingof: w/r/t not getting a dhcp not working, is there other interfaces on the tinycore machine? i.e. the lack of drivers issue that I think has forced different ipa images in prior testing. | 18:13 |
*** ociuhandu has quit IRC | 18:18 | |
*** ociuhandu has joined #openstack-ironic | 18:18 | |
*** ociuhandu has quit IRC | 18:19 | |
*** ociuhandu has joined #openstack-ironic | 18:20 | |
*** ociuhandu has quit IRC | 18:21 | |
*** ociuhandu has joined #openstack-ironic | 18:21 | |
*** afasano has quit IRC | 18:25 | |
*** ociuhandu has quit IRC | 18:26 | |
*** k_mouza has joined #openstack-ironic | 18:41 | |
*** bnemec has joined #openstack-ironic | 18:44 | |
*** bnemec is now known as beekneemech | 18:45 | |
*** k_mouza has quit IRC | 18:46 | |
*** ociuhandu has joined #openstack-ironic | 18:46 | |
*** pcaruana has quit IRC | 18:51 | |
*** ociuhandu has quit IRC | 18:52 | |
*** jawad_axd has joined #openstack-ironic | 18:53 | |
*** jawad_axd has quit IRC | 18:58 | |
*** jawad_axd has joined #openstack-ironic | 19:13 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix entry paths for cleaning and deployment https://review.opendev.org/703160 | 19:16 |
*** jawad_axd has quit IRC | 19:18 | |
*** TxGirlGeek has quit IRC | 19:25 | |
*** TxGirlGeek has joined #openstack-ironic | 19:26 | |
*** TxGirlGeek has quit IRC | 19:27 | |
*** TxGirlGeek has joined #openstack-ironic | 19:28 | |
*** ociuhandu has joined #openstack-ironic | 19:30 | |
*** tesseract has quit IRC | 19:32 | |
*** k_mouza has joined #openstack-ironic | 19:33 | |
*** jawad_axd has joined #openstack-ironic | 19:34 | |
*** ociuhandu has quit IRC | 19:35 | |
*** k_mouza has quit IRC | 19:38 | |
*** jawad_axd has quit IRC | 19:38 | |
TheJulia | Taking wife to lunch and then an errand, bbl | 19:46 |
*** belmoreira has quit IRC | 19:49 | |
*** aedc has joined #openstack-ironic | 19:52 | |
*** jawad_axd has joined #openstack-ironic | 19:55 | |
*** TxGirlGeek has quit IRC | 19:57 | |
*** TxGirlGeek has joined #openstack-ironic | 19:58 | |
*** jawad_axd has quit IRC | 20:00 | |
*** TxGirlGeek has quit IRC | 20:00 | |
*** openstackstatus has joined #openstack-ironic | 20:01 | |
*** ChanServ sets mode: +v openstackstatus | 20:01 | |
*** TxGirlGeek has joined #openstack-ironic | 20:02 | |
*** TxGirlGeek has quit IRC | 20:03 | |
*** TxGirlGeek has joined #openstack-ironic | 20:03 | |
*** jtomasek has quit IRC | 20:03 | |
etingof | congrats to TheJulia on the election results! \o/ | 20:10 |
*** jawad_axd has joined #openstack-ironic | 20:21 | |
*** jawad_axd has quit IRC | 20:25 | |
*** aedc has quit IRC | 20:26 | |
*** rh-jelabarre has quit IRC | 20:36 | |
*** aedc has joined #openstack-ironic | 20:36 | |
*** belmoreira has joined #openstack-ironic | 20:38 | |
*** belmoreira has quit IRC | 20:38 | |
*** jawad_axd has joined #openstack-ironic | 20:42 | |
*** jawad_axd has quit IRC | 20:46 | |
*** ociuhandu has joined #openstack-ironic | 20:46 | |
*** ociuhandu has quit IRC | 20:48 | |
*** ociuhandu has joined #openstack-ironic | 20:48 | |
*** mmethot has quit IRC | 20:51 | |
*** jawad_axd has joined #openstack-ironic | 21:02 | |
*** ociuhandu has quit IRC | 21:03 | |
*** ociuhandu has joined #openstack-ironic | 21:04 | |
*** jawad_axd has quit IRC | 21:07 | |
*** hwoarang has quit IRC | 21:09 | |
*** ociuhandu has quit IRC | 21:13 | |
*** hwoarang has joined #openstack-ironic | 21:15 | |
*** jawad_axd has joined #openstack-ironic | 21:23 | |
*** goldyfruit has quit IRC | 21:23 | |
*** jawad_axd has quit IRC | 21:27 | |
*** aedc has quit IRC | 21:42 | |
*** ociuhandu has joined #openstack-ironic | 21:44 | |
*** jawad_axd has joined #openstack-ironic | 21:44 | |
*** ociuhandu has quit IRC | 21:48 | |
*** jawad_axd has quit IRC | 21:49 | |
*** kozhukalov has joined #openstack-ironic | 22:00 | |
*** mmethot has joined #openstack-ironic | 22:01 | |
*** jawad_axd has joined #openstack-ironic | 22:04 | |
*** jawad_axd has quit IRC | 22:09 | |
*** TxGirlGeek has quit IRC | 22:20 | |
*** jawad_axd has joined #openstack-ironic | 22:25 | |
*** jawad_axd has quit IRC | 22:30 | |
*** rh-jelabarre has joined #openstack-ironic | 22:33 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM try further reducing RAM in DIB jobs https://review.opendev.org/702702 | 22:34 |
*** ociuhandu has joined #openstack-ironic | 22:35 | |
*** ociuhandu has quit IRC | 22:40 | |
*** jawad_axd has joined #openstack-ironic | 22:46 | |
*** jawad_axd has quit IRC | 22:50 | |
*** jawad_axd has joined #openstack-ironic | 23:07 | |
*** jawad_axd has quit IRC | 23:12 | |
*** kozhukalov has quit IRC | 23:13 | |
*** rh-jelabarre has quit IRC | 23:27 | |
*** jawad_axd has joined #openstack-ironic | 23:27 | |
*** jawad_axd has quit IRC | 23:32 | |
*** hwoarang has quit IRC | 23:33 | |
*** k_mouza has joined #openstack-ironic | 23:34 | |
*** hwoarang has joined #openstack-ironic | 23:34 | |
*** k_mouza has quit IRC | 23:39 | |
*** jawad_axd has joined #openstack-ironic | 23:48 | |
*** jawad_axd has quit IRC | 23:53 | |
*** rcernin_ has joined #openstack-ironic | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!