opendevreview | Steve Baker proposed openstack/ironic master: Refactor driver_internal_info updates to methods https://review.opendev.org/c/openstack/ironic/+/764863 | 04:22 |
---|---|---|
opendevreview | Steve Baker proposed openstack/ironic master: Use driver_internal_info methods for driver utils https://review.opendev.org/c/openstack/ironic/+/818505 | 04:22 |
opendevreview | Steve Baker proposed openstack/ironic master: Use driver_internal_info methods for drac driver https://review.opendev.org/c/openstack/ironic/+/818506 | 04:22 |
opendevreview | Steve Baker proposed openstack/ironic master: Use driver_internal_info methods for ilo driver https://review.opendev.org/c/openstack/ironic/+/818507 | 04:22 |
opendevreview | Steve Baker proposed openstack/ironic master: Use driver_internal_info methods for redfish driver https://review.opendev.org/c/openstack/ironic/+/818508 | 04:22 |
opendevreview | Steve Baker proposed openstack/ironic master: Use driver_internal_info methods for other drivers https://review.opendev.org/c/openstack/ironic/+/818509 | 04:22 |
janders | Dear Sushy maintainers. When you have time, I would really appreciate reviews on https://review.opendev.org/c/openstack/sushy/+/818369 and https://review.opendev.org/c/openstack/sushy/+/818364. These are "clean" (no other changes) backports of already-merged master change: https://review.opendev.org/c/openstack/sushy/+/817137 . Thanks in advance! | 04:50 |
arne_wiebalck | Good morning, Ironic! | 07:06 |
muellerbe | good morning ironic o/ | 07:15 |
janders | hey arne_wiebalck and muellerbe | 07:21 |
janders | rpittau arne_wiebalck thank you for the +2s! | 07:22 |
arne_wiebalck | hey muellerbe o/ | 07:38 |
arne_wiebalck | janders: np :) | 07:38 |
arne_wiebalck | janders: will send you some sushy patches as well :-P | 07:38 |
muellerbe | hey janders and arne_wiebalck o/ | 07:38 |
iurygregory | good morning janders arne_wiebalck muellerbe and Ironic, happy friday! | 07:42 |
janders | iurygregory Happy Friday! :) | 07:43 |
arne_wiebalck | hey iurygregory o/ | 07:44 |
rpittau | good morning ironic! o/ | 08:12 |
rpittau | np janders :) | 08:12 |
opendevreview | Merged openstack/sushy stable/xena: Changing boot device string for vmedia on SuperMicro https://review.opendev.org/c/openstack/sushy/+/818369 | 08:13 |
opendevreview | Merged openstack/sushy stable/wallaby: Changing boot device string for vmedia on SuperMicro https://review.opendev.org/c/openstack/sushy/+/818364 | 08:13 |
iurygregory | morning rpittau o/ | 08:26 |
rpittau | hey iurygregory :) | 08:26 |
*** akahat|rover is now known as akahat|lunch | 08:36 | |
rpittau | if anyone has a minute please review https://review.opendev.org/c/openstack/ironic-lib/+/817984 and https://review.opendev.org/c/openstack/ironic-lib/+/817983 so we can ask a release for ironic-lib in xena and wallaby | 08:45 |
iurygregory | rpittau, done =) | 09:05 |
rpittau | thanks! | 09:05 |
iurygregory | yw =) | 09:06 |
opendevreview | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Improve testing https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/818523 | 09:16 |
opendevreview | Riccardo Pittau proposed openstack/ironic master: Use default test dir https://review.opendev.org/c/openstack/ironic/+/818524 | 09:40 |
*** akahat|lunch is now known as akahat|rover | 09:59 | |
dtantsur | morning ironic | 10:57 |
dtantsur | finally Friday, yeah? | 10:57 |
rpittau | TGIF \o/ | 10:57 |
opendevreview | Merged openstack/ironic-lib stable/xena: Restore blkid compatibility with Centos 7 https://review.opendev.org/c/openstack/ironic-lib/+/817983 | 11:03 |
opendevreview | Merged openstack/ironic-lib stable/wallaby: Restore blkid compatibility with Centos 7 https://review.opendev.org/c/openstack/ironic-lib/+/817984 | 11:03 |
opendevreview | Merged openstack/ironic-python-agent master: Remove metalsmith centos7 job https://review.opendev.org/c/openstack/ironic-python-agent/+/817472 | 11:08 |
rpittau | dtantsur:, TheJulia, iurygregory, I'm thinking to ask for a bugfix release of ironic-lib for xena and wallaby to include the fix for centos7, wdyt? | 11:44 |
dtantsur | makes sense | 11:45 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Trivial: split away efibootmgr helpers https://review.opendev.org/c/openstack/ironic-python-agent/+/818537 | 11:45 |
iurygregory | rpittau, ++ | 11:45 |
rpittau | ok, thanks | 11:46 |
dtantsur | okay, now we seem to have bifrost keystone jobs broken #tableflip | 12:24 |
dtantsur | maybe, just maybe, it's broken by the change to UEFI by default | 12:56 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use UEFI by default and deprecate legacy boot https://review.opendev.org/c/openstack/bifrost/+/818549 | 13:05 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use UEFI by default and deprecate legacy boot https://review.opendev.org/c/openstack/bifrost/+/818549 | 13:08 |
janders | see you on Monday Ironic, have a great weekend everyone o/ | 13:09 |
dtantsur | "Ansible will require Python 3.8 or newer on the controller starting with Ansible 2.12" | 13:11 |
rpittau | bye janders | 13:12 |
dtantsur | I wonder if we should drop 3.6 support or make sure we cap ansible well enough | 13:12 |
rpittau | mmm centos 8 still has 3.6 right ? | 13:12 |
dtantsur | I think it also has other versions, although their support promise may be limited | 13:12 |
rpittau | oh yeah, I mean the system default | 13:12 |
dtantsur | correct | 13:13 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Revert "Explicitly trap on ERR" and fix exit code hanlding https://review.opendev.org/c/openstack/bifrost/+/817978 | 13:14 |
TheJulia | good morning | 13:29 |
dtantsur | morning TheJulia | 13:30 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add a unit test job with Sushy from source https://review.opendev.org/c/openstack/ironic/+/818553 | 13:30 |
* TheJulia wipes sleep from her eyes | 13:30 | |
iurygregory | good morning TheJulia, tgif =) | 13:31 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add a unit test job with Sushy from source https://review.opendev.org/c/openstack/ironic/+/818553 | 13:32 |
opendevreview | Dmitry Tantsur proposed openstack/sushy master: Run Ironic unit tests to avoid regressions https://review.opendev.org/c/openstack/sushy/+/818554 | 13:34 |
dtantsur | this is an experiment, I wonder if it works in the end ^^ | 13:34 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add a unit test job with Sushy from source https://review.opendev.org/c/openstack/ironic/+/818553 | 13:35 |
TheJulia | I think we're going to need to disable voting for a couple jobs to fix them | 13:36 |
dtantsur | TheJulia: which ones do you have in mind? | 13:36 |
TheJulia | I just had it in front of me and lost the window | 13:37 |
* TheJulia goes back | 13:37 | |
rpittau | dtantsur: I'd be ok dropping python 3.6 support, also considering that its full support ends in 1 month | 13:38 |
TheJulia | https://review.opendev.org/c/openstack/ironic/+/818458 <-- Specifically we seemt o have unexplained/weird failures in the metalsmith job, and the grenade job | 13:38 |
dtantsur | Error: Could not verify uefi on device /dev/vdafailed with Unexpected error while running command. | 13:39 |
TheJulia | is taht from metalsmith? | 13:39 |
dtantsur | yeah | 13:39 |
TheJulia | I was wondering because I couldn't see anything obvious | 13:39 |
TheJulia | not caffinated yet | 13:39 |
dtantsur | https://zuul.opendev.org/t/openstack/build/284b4d0d8e0b45d9a6bc40eff1d7df3f/log/controller/ironic-bm-logs/node-0_no_ansi_2021-11-19-00:53:27.log#3072 | 13:39 |
TheJulia | that eems entirely unrelated | 13:39 |
TheJulia | but there is also a vif detach error | 13:39 |
dtantsur | mount: /tmp/tmpk_scf0vo/boot/efi: bad option; for several filesystems (e.g. nfs, cifs) you might need a /sbin/mount.<type> helper program | 13:40 |
dtantsur | wtf | 13:40 |
dtantsur | wait, why do we have JSON in the command? O____o | 13:40 |
dtantsur | it could be a regression in my efi_utils patch, lemme stare at it | 13:41 |
TheJulia | ok | 13:41 |
TheJulia | I've got grenade up now trying to figure out wtf broke there | 13:41 |
dtantsur | I wonder why the metalsmith job on IPA itself didn't catch it | 13:41 |
TheJulia | 2021-11-19 00:28:16.954 | + /opt/stack/new/ironic/devstack/lib/ironic:create_ovs_taps:2048 : openstack --os-cloud port delete 2d1885b4-1c45-4431-b42b-b8e4ae5d4d90 | 13:42 |
TheJulia | le-sigh | 13:42 |
opendevreview | Julia Kreger proposed openstack/ironic master: CI: Fix devstack plugin with RBAC changes https://review.opendev.org/c/openstack/ironic/+/818458 | 13:49 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Fix compatibility with disk_utils.find_efi_partition https://review.opendev.org/c/openstack/ironic-python-agent/+/818556 | 13:49 |
dtantsur | TheJulia: ^^^ | 13:49 |
dtantsur | sorry for that | 13:49 |
dtantsur | fg | 13:49 |
TheJulia | dtantsur: stuff happens | 13:50 |
TheJulia | hopefully those jobs "just work" | 13:50 |
dtantsur | does not explain why the metalsmith job didn't catch it on IPA... | 13:50 |
TheJulia | Some times, it is life | 13:50 |
TheJulia | unfortunately | 13:50 |
dtantsur | ah, it's non-voting :) | 13:50 |
TheJulia | lol | 13:50 |
TheJulia | That woudl do it | 13:50 |
dtantsur | OUCH | 13:50 |
TheJulia | I suspect we may need to fix ironic first | 13:50 |
dtantsur | we have different jobs in check and gate Oo | 13:50 |
dtantsur | I'll fix that as well | 13:51 |
TheJulia | ok | 13:51 |
dtantsur | TheJulia: we need to run the UEFI one, right? | 13:51 |
opendevreview | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Improve testing https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/818523 | 13:51 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Fix compatibility with disk_utils.find_efi_partition https://review.opendev.org/c/openstack/ironic-python-agent/+/818556 | 13:51 |
dtantsur | TheJulia: like this ^^ | 13:52 |
TheJulia | dtantsur: efi, yes | 13:53 |
TheJulia | we should be banishing bios stuffs | 13:53 |
dtantsur | yeah, I proposed switching Bifrost today | 13:54 |
TheJulia | Or at least sending them to Zahadoom | 13:54 |
TheJulia | err, or maybe Z'ha'dum | 13:54 |
TheJulia | Anyway, we send them to the planet of the shadows and don't look back | 13:54 |
dtantsur | oh damn, https://review.opendev.org/c/openstack/ironic/+/816721 hasn't merged yet | 13:55 |
dtantsur | c'mon people, you don't like redfish? :) | 13:55 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add a unit test job with Sushy from source https://review.opendev.org/c/openstack/ironic/+/818553 | 13:56 |
opendevreview | Julia Kreger proposed openstack/ironic stable/ussuri: CI: Work around tinyipa build issues https://review.opendev.org/c/openstack/ironic/+/818394 | 14:02 |
TheJulia | there, it can be +1'ed once the gate is happy | 14:02 |
* TheJulia suddenly has a question for jammielennox and wonders how he is doing | 14:16 | |
opendevreview | Dmitry Tantsur proposed openstack/sushy master: DNM try breaking Ironic tests (more than I already did) https://review.opendev.org/c/openstack/sushy/+/818557 | 14:19 |
dtantsur | does this ring any bells? https://zuul.opendev.org/t/openstack/build/db7937c6b09a4f93a787f8cf0561908e/log/logs/testvm1_console.log | 14:44 |
dtantsur | this is haunting bifrost jobs on ubuntu, and my UEFI patch doesn't make it any better (if not worse) | 14:44 |
TheJulia | file size of zero bytes | 14:45 |
TheJulia | odd | 14:45 |
TheJulia | Someone else mentioned that recently | 14:45 |
TheJulia | well, like 3 months ago | 14:45 |
dtantsur | I wonder if the file name is simply wrong | 14:47 |
dtantsur | hmm, doesn't seem to | 14:47 |
TheJulia | can we confirm that it actually has a real size? | 14:50 |
dtantsur | completely accidentally, we put the whole file in the job logs :) | 14:50 |
dtantsur | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_db7/817978/4/check/bifrost-integration-tinyipa-ubuntu-bionic/db7937c/logs/pxe/ipxe.efi | 14:50 |
opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-specs master: Yoga Themes https://review.opendev.org/c/openstack/ironic-specs/+/815308 | 14:51 |
TheJulia | heh | 14:52 |
TheJulia | nice | 14:52 |
TheJulia | dtantsur: could it be that the access to the file is being denied by the OS? | 14:53 |
TheJulia | file exists, but file open would fail | 14:53 |
dtantsur | possible, but I see no traces of it (and ubuntu has no selinux) | 14:53 |
TheJulia | and be zero bytes depending on the exchange. | 14:53 |
TheJulia | hmm, true | 14:53 |
dtantsur | it could be simple ACL of course | 14:53 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Revert "Explicitly trap on ERR" and fix exit code hanlding https://review.opendev.org/c/openstack/bifrost/+/817978 | 14:55 |
rpittau | bye everyone, have a great weekend! o/ | 15:00 |
TheJulia | o/ rpittau | 15:00 |
arne_wiebalck | rpittau: o/ | 15:01 |
artom | TheJulia, so the more I think about https://review.opendev.org/c/openstack/nova/+/813897, the more I become convinced it has to be done in the conductor | 15:38 |
artom | Tangentially, I did write (but not yet send) the following in response to your "bit much" comment: | 15:39 |
artom | Sorry, I realize this was an unfriendly way to start a review. In the past just leaving comments on Nova patches would often result in no reaction, so I've developed a habit (and been advised by others) to be a bit "in your face" about it, and not feel bad about saying "-1 for question visibility". Which is what I attempted to do here, just clumsily. Sorry again. | 15:39 |
artom | The main thing that convinced me is that a nova-compute service will always add itself to the list of services passed to the hash ring, because it may not yet have registered itself in the service group. | 15:40 |
artom | Which means that, if another nova-compute service is refreshing its cache at the same time, the hash ring will have different results for both. | 15:40 |
artom | I fully realize that doing it in the conductor is not backportable, but I feel like it's the only logical place. | 15:41 |
artom | I also have no idea how to properly abstract what it essentially an Ironic-only concept. | 15:41 |
TheJulia | artom: in the middle of something else, I'll read in a little bit | 15:42 |
artom | TheJulia, sure, I'm still mulling this over myself. | 15:43 |
dtantsur | -rw-r--r-- 1 ironic ironic 89 Nov 19 15:10 ipa.kernel.sha256 | 15:46 |
dtantsur | -rw-r--r-- 1 ironic ironic 1002336 Jul 16 2020 ipxe.efi | 15:46 |
dtantsur | it's not permissions then | 15:46 |
dtantsur | bloody UEFI, bloody PXE | 15:46 |
dtantsur | interestingly, the only UEFI jobs we had before on Bifrost were virtual media jobs | 15:52 |
dtantsur | sigh, I'll revert to BIOS for the time being, I guess | 15:52 |
dtantsur | iptables dropped: IN=virbr0 OUT= MAC= SRC=192.168.122.1 DST=255.255.255.255 LEN=328 TOS=0x00 PREC=0xC0 TTL=64 ID=43050 PROTO=UDP SPT=67 DPT=68 LEN=308 | 15:57 |
dtantsur | maybe not be the cause, but should not be happening | 15:57 |
TheJulia | new agenda item, we need islands of ironic. Dan Smith requests joining our exclusive resort | 15:59 |
dtantsur | you mean, the asylum? | 16:00 |
TheJulia | ?maybe? | 16:00 |
TheJulia | "resort" | 16:01 |
artom | dansmith just looking to be core on all of OpenStack now? First Nova, then Glance, now Ironic? ;) | 16:02 |
dtantsur | wait, what, we can assign work to dansmith now? | 16:03 |
iurygregory | TheJulia, were the resort will be located? :D | 16:03 |
dansmith | not seeking core, just seeking an invite to the exclusive resort | 16:03 |
dansmith | iurygregory: obviously on some group of islands | 16:03 |
dtantsur | Spitsbergen? :D | 16:04 |
iurygregory | dansmith, we need to discuss in the upstream meeting =P | 16:04 |
iurygregory | I know rpittau will vote for Sardegna :D | 16:04 |
dtantsur | a resort at rpittau's place works for me | 16:05 |
TheJulia | dtantsur: nah, I odn't think we can necessarily assign work to dansmith ;) | 16:05 |
TheJulia | dtantsur: ++ | 16:05 |
iurygregory | dtantsur, very interesting the Spitsbergen island :D | 16:05 |
dtantsur | iurygregory: it's one of my dream destinations | 16:05 |
iurygregory | nice! | 16:06 |
TheJulia | Spitsbergen seems like it would be an excellent location for an evil fortress | 16:06 |
TheJulia | artom: reading | 16:06 |
dtantsur | not really. I think you're not to die there: too troublesome to bury anyone | 16:06 |
dtantsur | * not allowed | 16:06 |
TheJulia | artom: no worries regarding the "bit much" stuff. | 16:06 |
dtantsur | okay, I think we (I?) messed up iptables in bifrost | 16:07 |
iurygregory | TheJulia, well.. we are going to take over the world, no? :D | 16:07 |
iurygregory | evil fortress seems very legit to me :D | 16:07 |
TheJulia | artom: to nova-compute, what does the service group really mean, just so I'm on the same page as you? | 16:07 |
TheJulia | artom: I ask because I think it is relatively easy to change the pattern to "ease" into the hash ring | 16:08 |
TheJulia | that may make many super happy | 16:08 |
TheJulia | iurygregory: this means we need to hire an evil architect and get an evil construction company to build the evil fortress of bare metal solitude | 16:08 |
TheJulia | s/bare/bear/ | 16:09 |
TheJulia | of course! there are all the design themes! | 16:09 |
iurygregory | TheJulia, that makes a lot of sense! | 16:09 |
TheJulia | We'll just need to "borrow" artwork styles from bearlin | 16:09 |
artom | TheJulia, just a logical abstraction around getting lists of services, as far as I can tell | 16:09 |
dtantsur | even architecture: https://l13.alamy.com/360/PWNBG7/sedlec-ossuary-bone-chapel-kutna-hora-czech-republic-PWNBG7.jpg | 16:09 |
TheJulia | artom: do you know when the entry gets added to the db table? | 16:09 |
dtantsur | * evil (I should stop talking to no longer embarrass myself) | 16:10 |
artom | Sorry, not quite - an API for getting a service's UP status | 16:10 |
iurygregory | I like how fast dtantsur is to find this things... :D | 16:10 |
dtantsur | an even better shot: https://static.thousandwonders.net/Sedlec.Ossuary.original.32979.jpg | 16:10 |
artom | What I said initially is the ServiceList | 16:10 |
dtantsur | iurygregory: it may be my secret desires... or image search in duckduckgo. you'll never know for sure. | 16:10 |
artom | TheJulia, at some point during service startup... do we need to be more precise than that? | 16:10 |
TheJulia | artom: maybe, but also maybe not | 16:11 |
artom | TheJulia, also, what do you mean by "ease" into the hash ring? | 16:11 |
* iurygregory never used duckduckgo ... | 16:11 | |
TheJulia | I can dig further | 16:11 |
TheJulia | artom: I mean, not necessarilly immediatley start assuming remote nodes are managed by the hash ring, but I *suspet* that will have cache impliations and get_available_nodes | 16:11 |
TheJulia | (this is why I want zer0c00l to ponder this as well) | 16:12 |
dtantsur | oh of course it was me who screwed it up the last time, sigh: https://opendev.org/openstack/bifrost/commit/acda312aba6434ef25ca7bb092af10f7106f9d39 | 16:12 |
TheJulia | and nobodycam, and arne | 16:12 |
TheJulia | and the kitten just tried to explore the top of the router and the house SAN | 16:12 |
TheJulia | yay! | 16:12 |
dtantsur | \o/ | 16:12 |
TheJulia | \o/ still connected | 16:12 |
artom | TheJulia, as someone who fell into this review by more or less chance, and has absolutely no Ironic review credentials, my simple mind immediately goes to "can we not make the hash ring the absolute source of truth"? | 16:13 |
TheJulia | artom: I guess trying to move into the conductor seems like a huge lift of complexity when it seems like it is all resolvable and able to be navigated at the compute service level | 16:13 |
artom | And then rejigger instance.host and compute_node.host records accordingly. It would become *so* much easier to reason about it. | 16:13 |
TheJulia | artom: that is actually what I want, just make the hash ring the source of truth and reconcile it from there as the ring changes | 16:13 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Fix the DHCP port in iptables (68 is for clients) https://review.opendev.org/c/openstack/bifrost/+/818592 | 16:14 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use UEFI by default and deprecate legacy boot https://review.opendev.org/c/openstack/bifrost/+/818549 | 16:14 |
TheJulia | I *think* the case I'm trying to guard is *un-necessarily* doing it based on service restarts which are common, like nobodycam has to do it every few weeks AIUI | 16:14 |
artom | TheJulia, yeah, I guess if we don't add CONF.host to the list of services we pass to the hash ring, that should help make things consistent. | 16:14 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Revert "Explicitly trap on ERR" and fix exit code hanlding https://review.opendev.org/c/openstack/bifrost/+/817978 | 16:14 |
TheJulia | artom: exactly, it would delay the node actually picking things up a little, I think | 16:15 |
artom | Which is... fine? | 16:15 |
TheJulia | ... I think so | 16:15 |
TheJulia | but the folks who run this at scale need to chime in on that | 16:15 |
TheJulia | since this is their pain point | 16:15 |
TheJulia | well, mostly their pain point, the fact we can hit a condition which is sparked from this downstream is kind of funny | 16:16 |
TheJulia | but like... all of the major pain points have resolved around this area of the code | 16:16 |
artom | I'm... not surprised :P | 16:16 |
TheJulia | :) | 16:16 |
dtantsur | TheJulia, iurygregory, https://review.opendev.org/c/openstack/ironic-python-agent/+/818556 fixes the metalsmith job | 16:16 |
TheJulia | +2'ed | 16:17 |
TheJulia | now that I'm not on a meeting | 16:17 |
* TheJulia checks changes from this morning | 16:17 | |
TheJulia | hmm 817019 is still running | 16:18 |
TheJulia | wish it... luck?!? | 16:18 |
TheJulia | hmmm... grenade is still failing on 818458 | 16:19 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: [WIP] iPXE fallback script https://review.opendev.org/c/openstack/bifrost/+/818000 | 16:20 |
dtantsur | TheJulia: what's the failure? | 16:21 |
TheJulia | OS_CLOUD not present | 16:21 |
dtantsur | O____O | 16:21 |
TheJulia | oh | 16:21 |
TheJulia | i see why | 16:21 |
TheJulia | nevermind | 16:21 |
opendevreview | Julia Kreger proposed openstack/ironic master: CI: Fix devstack plugin with RBAC changes https://review.opendev.org/c/openstack/ironic/+/818458 | 16:23 |
TheJulia | now to wait on ci | 16:23 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Trivial: split away efibootmgr helpers https://review.opendev.org/c/openstack/ironic-python-agent/+/818537 | 16:27 |
NobodyCam | Good morning Ironic’ers | 16:30 |
TheJulia | good morning NobodyCam | 16:32 |
dtantsur | morning NobodyCam! TGIF, isn't it? :) | 16:32 |
NobodyCam | Good Morning TheJulia and dtantsur ...It is.. and I haven't had enough coffee yet... | 16:33 |
TheJulia | dtantsur: looks like the stable/ussuri sed changes have finally worked | 16:34 |
dtantsur | \o/ | 16:34 |
TheJulia | just watched one of the jobs upload the ramdisk to glance \o/ | 16:34 |
TheJulia | and conductor is being started | 16:34 |
TheJulia | speaking of coffee, i think I'm in need of more | 16:35 |
iurygregory | morning NobodyCam o/ | 16:36 |
NobodyCam | good morning iurygregory :) o/ | 16:37 |
dtantsur | I could swear there were some talks about bullseye support in IPA-builder.. I guess I need to ask rpittau on Monday. | 16:44 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Revert the default DIB distribution to Buster https://review.opendev.org/c/openstack/bifrost/+/818600 | 16:50 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Revert the default DIB distribution to Buster https://review.opendev.org/c/openstack/bifrost/+/818600 | 16:50 |
TheJulia | yup https://review.opendev.org/c/openstack/ironic/+/818394/ got stable/ussuri ipa jobs past build and one just passed test_baremetal_server_ops_partition_image | 16:51 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Fix the DHCP port in iptables (68 is for clients) https://review.opendev.org/c/openstack/bifrost/+/818592 | 16:54 |
dtantsur | I can be impressively stupid at times ^^ | 16:54 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use UEFI by default and deprecate legacy boot https://review.opendev.org/c/openstack/bifrost/+/818549 | 16:55 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Revert "Explicitly trap on ERR" and fix exit code hanlding https://review.opendev.org/c/openstack/bifrost/+/817978 | 16:55 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Revert the default DIB distribution to Buster https://review.opendev.org/c/openstack/bifrost/+/818600 | 16:56 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: [WIP] iPXE fallback script https://review.opendev.org/c/openstack/bifrost/+/818000 | 16:56 |
dtantsur | I think I should go before I discover another bug I introduced :) | 17:10 |
dtantsur | have a nice weekend y'all | 17:10 |
iurygregory | bye dtantsur o/ | 17:21 |
arne_wiebalck | bye everyone, see you next week o/ | 17:40 |
*** janders3 is now known as janders | 17:42 | |
iurygregory | and we will be default to py3.8 py3.9 in Yoga :D | 17:47 |
iurygregory | bye bye py36.... | 17:47 |
iurygregory | [all][tc] Yoga testing runtime update. Action needed if py3.9 job failing for your project | 17:47 |
iurygregory | \o/ | 17:47 |
opendevreview | Merged openstack/ironic-python-agent master: Fix compatibility with disk_utils.find_efi_partition https://review.opendev.org/c/openstack/ironic-python-agent/+/818556 | 17:51 |
opendevreview | Merged openstack/ironic stable/ussuri: CI: Work around tinyipa build issues https://review.opendev.org/c/openstack/ironic/+/818394 | 18:53 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Delete EFI boot entry duplicate labels first https://review.opendev.org/c/openstack/ironic-python-agent/+/817020 | 19:35 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Fix UEFI record regex https://review.opendev.org/c/openstack/ironic-python-agent/+/817021 | 19:35 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Auto-populate lessee for deployments https://review.opendev.org/c/openstack/ironic/+/818641 | 20:55 |
NobodyCam | crazy question: | 21:36 |
NobodyCam | anyone ever encounter a situation where IPA fails with `Failed looking up node with address ...` and the the Mac address matches the bare metal port Mac.. other nodes are working, this node use to work.. | 21:38 |
TheJulia | NobodyCam: unicode? | 23:00 |
NobodyCam | been not that I can tell.. | 23:01 |
NobodyCam | just deleted and re-enrolled | 23:01 |
NobodyCam | still | 23:02 |
NobodyCam | the same | 23:02 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Auto-populate lessee for deployments https://review.opendev.org/c/openstack/ironic/+/818641 | 23:06 |
TheJulia | did you copy/paste the mac? | 23:25 |
TheJulia | or is it from somewhere else? | 23:25 |
TheJulia | NobodyCam: so come to think of it, I did once, but it was something funky with the payload and the nics | 23:34 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!