| TheJulia | o/ janders | 00:24 |
|---|---|---|
| opendevreview | Merged openstack/bifrost master: Enable HTTP2 for Keystone when TLS is enabled https://review.opendev.org/c/openstack/bifrost/+/820394 | 05:59 |
| arne_wiebalck | Good morning, Ironic! | 07:57 |
| dtantsur | morning ironic | 08:20 |
| janders | good morning arne_wiebalck dtantsur and Ironic o/ | 08:29 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: All-in-one Ironic service with a local RPC bus https://review.opendev.org/c/openstack/ironic/+/819620 | 08:31 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add "none" RPC transport that disables the RPC bus https://review.opendev.org/c/openstack/ironic/+/820036 | 08:33 |
| dtantsur | review appreciated ^^ | 08:33 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/xena: Trivial: log current state when continuing cleaning https://review.opendev.org/c/openstack/ironic/+/820613 | 08:34 |
| arne_wiebalck | Hey janders and dtantsur o/ | 08:47 |
| rpittau | good morning ironic! o/ | 09:13 |
| *** sshnaidm|afk is now known as sshnaidm | 09:20 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Prepare for bugfix release https://review.opendev.org/c/openstack/ironic-python-agent/+/820489 | 09:39 |
| opendevreview | Merged openstack/ironic master: Trivial: minor follow-up to redfish fix https://review.opendev.org/c/openstack/ironic/+/820564 | 10:09 |
| *** redrobot8 is now known as redrobot | 10:20 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/18.1: Trivial: log current state when continuing cleaning https://review.opendev.org/c/openstack/ironic/+/820614 | 10:20 |
| opendevreview | Merged openstack/ironic master: Add missing mode setting on pxe created directories https://review.opendev.org/c/openstack/ironic/+/809294 | 10:28 |
| opendevreview | Merged openstack/sushy stable/xena: Raise an AccessError with SessionService init https://review.opendev.org/c/openstack/sushy/+/820598 | 10:51 |
| opendevreview | Verification of a change to openstack/sushy stable/wallaby failed: Raise an AccessError with SessionService init https://review.opendev.org/c/openstack/sushy/+/820599 | 10:51 |
| opendevreview | Merged openstack/ironic master: Refactor common configuration bits from service commands https://review.opendev.org/c/openstack/ironic/+/820017 | 12:10 |
| opendevreview | Merged openstack/sushy stable/victoria: Raise an AccessError with SessionService init https://review.opendev.org/c/openstack/sushy/+/820600 | 12:27 |
| opendevreview | Arne Wiebalck proposed openstack/sushy master: Use Etag in precondition header when setting boot options https://review.opendev.org/c/openstack/sushy/+/818114 | 13:14 |
| opendevreview | Arne Wiebalck proposed openstack/sushy master: Use Etag in precondition header when setting boot options https://review.opendev.org/c/openstack/sushy/+/818114 | 13:17 |
| arne_wiebalck | dtantsur: thanks ^^, now even with nit fix :) | 13:17 |
| dtantsur | nice! | 13:18 |
| TheJulia | good morning | 13:59 |
| rpittau | good morning TheJulia :) | 14:02 |
| opendevreview | Verification of a change to openstack/ironic stable/xena failed: Trivial: log current state when continuing cleaning https://review.opendev.org/c/openstack/ironic/+/820613 | 14:07 |
| TheJulia | hmm redfish pxe jobs are not happy | 14:10 |
| TheJulia | ugh, the invalid datapath bug | 14:15 |
| dtantsur | brrr | 14:19 |
| dtantsur | morning TheJulia | 14:19 |
| dtantsur | I might have addressed your comments on the combined ironic patch. if I understood them right. | 14:19 |
| TheJulia | https://1662b427aebb610c1844-4e71828f84900af50a9a26357b84a827.ssl.cf5.rackcdn.com/820602/1/check/sushy-tempest-ironic-partition-redfish-src/9624b1d/controller/logs/screen-redfish-emulator.txt | 14:19 |
| TheJulia | le-sigh | 14:19 |
| dtantsur | could you maybe ping me within the next couple of hours if you have more concerns/ | 14:19 |
| TheJulia | dtantsur: okay, I'll try to take a look | 14:20 |
| dtantsur | ImportError: cannot import name 'auth_basic' | 14:20 |
| dtantsur | O__o | 14:20 |
| dtantsur | old ironic-lib somehow? | 14:20 |
| dtantsur | wait, no, it cannot be THAT old | 14:20 |
| dtantsur | ooooor... | 14:21 |
| dtantsur | yes, it can | 14:21 |
| dtantsur | :( | 14:21 |
| dtantsur | TheJulia: sushy-tools is unbranched, ironic-lib is branched. sorry, haven't thought about it | 14:21 |
| dtantsur | we probably need to cap sushy-tools on branches that don't have the required ironic-lib | 14:22 |
| opendevreview | Julia Kreger proposed openstack/sushy stable/ussuri: CI: Pin sushy-tools to fix CI https://review.opendev.org/c/openstack/sushy/+/820868 | 14:23 |
| TheJulia | we've hit it before | 14:23 |
| TheJulia | so.. yeah | 14:23 |
| TheJulia | I double checked ironic and the same exact thing is present there | 14:24 |
| TheJulia | on ussuri | 14:24 |
| dtantsur | :( | 14:24 |
| opendevreview | Julia Kreger proposed openstack/sushy stable/train: CI: Pin sushy-tools to fix CI https://review.opendev.org/c/openstack/sushy/+/820619 | 14:24 |
| TheJulia | c'est la vie | 14:24 |
| TheJulia | at least the failures are simple | 14:24 |
| dtantsur | TheJulia: hmm, you have duplicated required-projects in https://review.opendev.org/c/openstack/sushy/+/820868/1/zuul.d/sushy-jobs.yaml | 14:25 |
| TheJulia | except the neutron one | 14:25 |
| TheJulia | doh | 14:25 |
| dtantsur | I don't think it's going to work | 14:25 |
| dtantsur | just join them | 14:25 |
| TheJulia | yup | 14:25 |
| opendevreview | Julia Kreger proposed openstack/sushy stable/ussuri: CI: Pin sushy-tools to fix CI https://review.opendev.org/c/openstack/sushy/+/820868 | 14:25 |
| TheJulia | that is what I get for the 9pt font on my copy/paste terminal profile | 14:25 |
| dtantsur | heh, that's not much | 14:26 |
| opendevreview | Julia Kreger proposed openstack/sushy stable/train: CI: Pin sushy-tools to fix CI https://review.opendev.org/c/openstack/sushy/+/820619 | 14:26 |
| dtantsur | I'm using 12 | 14:26 |
| dtantsur | TheJulia: a potentially better path could be to update global-requirements to cap sushy-tools | 14:27 |
| dtantsur | but may cause objections from the release team | 14:27 |
| TheJulia | Yeah, it is purely for CI though | 14:29 |
| dtantsur | ah, we don't have sushy-tools in upper-constraints | 14:29 |
| dtantsur | for... reasons? | 14:29 |
| TheJulia | likely | 14:29 |
| TheJulia | not that we will ever remember them | 14:29 |
| dtantsur | I wonder what the release team if we try to backfill this requirements for ALL branches | 14:30 |
| dtantsur | * what the requirements team says | 14:30 |
| dtantsur | I'm completely incapable of thinking today | 14:30 |
| TheJulia | likely loose their minds | 14:30 |
| dtantsur | I wonder why the pip resolver does not complain or do the right thing | 14:31 |
| dtantsur | maybe old pip? | 14:31 |
| dtantsur | ah, my old enemies: chicken and egg | 14:47 |
| opendevreview | Aija Jauntēva proposed x/sushy-oem-idrac master: Migrate constants to enums https://review.opendev.org/c/x/sushy-oem-idrac/+/817028 | 15:09 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Use more granular lock in BaseDriverFactory https://review.opendev.org/c/openstack/ironic/+/820904 | 15:33 |
| TheJulia | ajya: replied to https://review.opendev.org/c/openstack/sushy/+/820076 I'm really struggling to understand what your percieving as a workaround to just make things work given the break | 16:05 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: WIP automatically configure interfaces https://review.opendev.org/c/openstack/ironic/+/820909 | 16:08 |
| dtantsur | TheJulia: we're getting pinged re https://review.opendev.org/c/openstack/releases/+/817593 (ussury em) | 16:09 |
| dtantsur | I seem to recall you planned on a round of releases? | 16:09 |
| TheJulia | Yeah, we should do that | 16:10 |
| jingvar | can I get autodiscovering with Bifrost? | 16:37 |
| TheJulia | I think so | 16:39 |
| TheJulia | I did it a long time ago, you will need to set credentials if memory serves | 16:39 |
| dtantsur | I think you need to enable it as well | 16:40 |
| dtantsur | https://docs.openstack.org/ironic-inspector/latest/user/usage.html#discovery is the general docs | 16:40 |
| jingvar | thanks, but it is not clear for me | 16:44 |
| TheJulia | dtantsur: yeah... I think there was a setting for it | 16:44 |
| jingvar | I thought it just a button - like has MAAS | 16:45 |
| TheJulia | jingvar: when installed with the setting enabled, it is always on | 16:49 |
| jingvar | hmm, it means with default installation for the doc, any powered on node (of course pxe, linked with bifrost port) will come into nodes with a name | 16:51 |
| jingvar | I have old proliant g7 and can't access to bios and remote console, only have halfworked ilo | 16:52 |
| jingvar | I pushed it from bifrost, but inspection failed, deploy failed (i don't know mac of pxe nic :) ) | 16:54 |
| TheJulia | hm | 17:01 |
| TheJulia | yeah, somehow you need to get into that machine to find the mac out in that case | 17:01 |
| jingvar | I have all macs, but don't know with is pxe | 17:02 |
| TheJulia | which one is for PXE? | 17:02 |
| jingvar | yep | 17:02 |
| TheJulia | just add them all | 17:02 |
| jingvar | ? | 17:02 |
| TheJulia | ironic will create records for each one | 17:02 |
| jingvar | I understand | 17:02 |
| jingvar | add into ? | 17:02 |
| TheJulia | create ports, and that should allow ironic to identify the machine as well at deployment time or managed introspection time, ironic will create specific pxe records for each port on the filesystem so the chain loader can possibly find a configuration to use | 17:03 |
| jingvar | hmm | 17:04 |
| jingvar | how it can be done via inventory? | 17:05 |
| jingvar | nics: | 17:05 |
| jingvar | - mac: $TARGET_MAC | 17:05 |
| TheJulia | just add more entries with mac addresses | 17:05 |
| TheJulia | - mac: $TARGET_MAC2 | 17:05 |
| TheJulia | etc | 17:05 |
| jingvar | thanks. will try | 17:09 |
| ajya | thanks, TheJulia, commented. The SessionService instance was not None, that's the jist of all this workaround. | 17:21 |
| TheJulia | I don't see how that was ever the case | 17:22 |
| ajya | I'm signing off, if there are more details necessary, can play around the code and see what happens when exception from parent's init is handled in child's init - child instance still will be created | 17:22 |
| ajya | or run the code with old version and add more debug messages | 17:22 |
| TheJulia | ajya: by all means tomorrow | 17:22 |
| * TheJulia goes back to trying to figure out why backports on ironic hate me | 17:23 | |
| arne_wiebalck | dtantsur: TheJulia: zuul is now happy with https://review.opendev.org/c/openstack/sushy/+/818114 | 17:46 |
| arne_wiebalck | bye everyone, see you tomorrow o/ | 17:47 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/xena: Remove redfish cache entry upon errors https://review.opendev.org/c/openstack/ironic/+/820588 | 17:54 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/xena: Explicitly pin upper sushy version https://review.opendev.org/c/openstack/ironic/+/820923 | 17:54 |
| TheJulia | sigh | 17:55 |
| dtantsur | TheJulia: you'll need a g-r change for ^^^ | 17:57 |
| TheJulia | its already pinned down | 17:57 |
| TheJulia | because we do explicitly it was still open from what I could tell | 17:57 |
| dtantsur | TheJulia: doesn't seem to? https://github.com/openstack/requirements/blob/stable/xena/global-requirements.txt#L468 | 17:58 |
| TheJulia | https://github.com/openstack/requirements/blob/stable/xena/upper-constraints.txt | 17:58 |
| TheJulia | bugfix change also inbound | 17:58 |
| TheJulia | since it was referring to master upper constraints | 17:59 |
| dtantsur | TheJulia: I'm not talking about u-c though | 17:59 |
| dtantsur | although.. driver-requirements is not guarded by global-requirements, so it may be fine | 17:59 |
| dtantsur | or maybe it is? I'm not sure. | 17:59 |
| dtantsur | the CI will tell :) | 17:59 |
| TheJulia | it isent, which is why I think it happens | 17:59 |
| TheJulia | it gets slightly weird if I remember correctly | 17:59 |
| TheJulia | unit tests seem to be working for bugfix/18.1 | 18:00 |
| dtantsur | yeah, the check-requirements job may complain | 18:00 |
| dtantsur | arguably, it's friendlier to downstream packagers to update g-r with <4.0.0 | 18:00 |
| TheJulia | that is likely also needed, ultimately | 18:01 |
| TheJulia | But I don't think pipeline ever considers major package versions | 18:01 |
| dtantsur | if we had sushy in requirements.txt, it would even be required for your patch to pass | 18:01 |
| TheJulia | yeah | 18:01 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/18.1: Remove redfish cache entry upon errors https://review.opendev.org/c/openstack/ironic/+/820589 | 18:02 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/18.1: Explicitly pin upper sushy version https://review.opendev.org/c/openstack/ironic/+/820927 | 18:02 |
| TheJulia | hmm, reverse order | 18:02 |
| dtantsur | hmm, the requirements-check job has started on your patch. I'm curious :) | 18:03 |
| TheJulia | setup running | 18:07 |
| dtantsur | succeeded | 18:10 |
| TheJulia | yup | 18:11 |
| TheJulia | \o/ | 18:11 |
| dtantsur | so I guess the job is set up to run on *requirements.txt, but actually checks a pre-defined list of files | 18:11 |
| TheJulia | and g-r itself largely only has upper bounds defined for things like navigating whole versions | 18:12 |
| TheJulia | like py36 support vs not supported | 18:12 |
| opendevreview | Verification of a change to openstack/ironic-inspector master failed: Add support for state selector in the list introspection https://review.opendev.org/c/openstack/ironic-inspector/+/807578 | 18:15 |
| dtantsur | of course it failed grenade, why wouldn't it? | 18:16 |
| TheJulia | because the universe hates us | 18:21 |
| TheJulia | ... or loves us | 18:21 |
| TheJulia | The universe is bipolar | 18:21 |
| dtantsur | quite possibly! (omg please tweet it) | 18:21 |
| TheJulia | like... quoting irc handles? | 18:22 |
| dtantsur | I'll leave it up to your artistic instinct! | 18:23 |
| TheJulia | done | 18:24 |
| TheJulia | Twitter, the new bash.org | 18:24 |
| TheJulia | so, lets see... wallaby unit tests | 18:25 |
| TheJulia | dtantsur: any objection to backporting https://review.opendev.org/c/openstack/ironic/+/784775/5/ironic/drivers/modules/redfish/utils.py ? | 18:27 |
| TheJulia | oh, wait | 18:27 |
| TheJulia | no | 18:27 |
| TheJulia | nevermind | 18:27 |
| TheJulia | thats not right | 18:27 |
| * TheJulia tests wallaby | 18:30 | |
| dtantsur | see you tomorrow o/ | 18:33 |
| TheJulia | goodnight! | 18:35 |
| rpittau | goodnight! o/ | 18:51 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: Remove redfish cache entry upon errors https://review.opendev.org/c/openstack/ironic/+/820597 | 18:52 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/victoria: Remove redfish cache entry upon errors https://review.opendev.org/c/openstack/ironic/+/820934 | 19:03 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/ussuri: Remove redfish cache entry upon errors https://review.opendev.org/c/openstack/ironic/+/820936 | 19:08 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/train: Remove redfish cache entry upon errors https://review.opendev.org/c/openstack/ironic/+/820937 | 19:09 |
| opendevreview | Merged openstack/metalsmith master: Use only Yoga tests https://review.opendev.org/c/openstack/metalsmith/+/820323 | 19:18 |
| opendevreview | Julia Kreger proposed openstack/sushy stable/train: CI: Pin sushy-tools to fix CI https://review.opendev.org/c/openstack/sushy/+/820619 | 19:19 |
| opendevreview | Merged openstack/sushy stable/wallaby: Raise an AccessError with SessionService init https://review.opendev.org/c/openstack/sushy/+/820599 | 19:25 |
| opendevreview | Verification of a change to openstack/ironic stable/xena failed: Trivial: log current state when continuing cleaning https://review.opendev.org/c/openstack/ironic/+/820613 | 19:34 |
| opendevreview | Merged openstack/sushy stable/ussuri: CI: Pin sushy-tools to fix CI https://review.opendev.org/c/openstack/sushy/+/820868 | 19:53 |
| stevebaker[m] | good morngin | 19:54 |
| stevebaker[m] | ing | 19:54 |
| opendevreview | Merged openstack/ironic master: Avoid RPC notify_conductor_resume_{deploy,clean} in agent_base https://review.opendev.org/c/openstack/ironic/+/756354 | 20:18 |
| opendevreview | Merged openstack/ironic master: Fix restricted allocation creation for old policy defaults https://review.opendev.org/c/openstack/ironic/+/812007 | 20:18 |
| TheJulia | good morning stevebaker[m] | 20:28 |
| opendevreview | Merged openstack/ironic master: Refactor driver_internal_info updates to methods https://review.opendev.org/c/openstack/ironic/+/764863 | 20:42 |
| opendevreview | Julia Kreger proposed openstack/sushy stable/train: CI: Pin sushy-tools to fix CI https://review.opendev.org/c/openstack/sushy/+/820619 | 20:51 |
| opendevreview | Merged openstack/ironic stable/xena: Trivial: log current state when continuing cleaning https://review.opendev.org/c/openstack/ironic/+/820613 | 21:10 |
| opendevreview | Merged openstack/sushy master: Use Etag in precondition header when setting boot options https://review.opendev.org/c/openstack/sushy/+/818114 | 21:16 |
| * TheJulia issues a couple rechecks and hopes ci will be happier | 21:32 | |
| TheJulia | It looks like we're starting to see the invalid datapath issue a lot as of recent :( | 21:34 |
| TheJulia | dtantsur: I'm going to file https://zuul.opendev.org/t/openstack/build/143f0f31adba4b658dc003591166def8 under "wut?!?" | 21:51 |
| opendevreview | Steve Baker proposed openstack/ironic master: Ensure desired permissions on all written pxe files https://review.opendev.org/c/openstack/ironic/+/809295 | 22:26 |
| opendevreview | Steve Baker proposed openstack/ironic master: Move place_loaders_for_boot to boot driver __init__ https://review.opendev.org/c/openstack/ironic/+/819729 | 22:26 |
| opendevreview | Steve Baker proposed openstack/ironic master: Write initial grub config on startup https://review.opendev.org/c/openstack/ironic/+/815580 | 22:26 |
| opendevreview | Steve Baker proposed openstack/ironic master: Capture [pxe]loader_file_paths for distros https://review.opendev.org/c/openstack/ironic/+/815392 | 22:26 |
| TheJulia | stevebaker[m]: review on https://review.opendev.org/c/openstack/sushy/+/820619 would be appreciated. I suspect you could just +2+W it, tbh since it is just job config related | 22:42 |
| stevebaker[m] | oki | 22:43 |
| TheJulia | much appreciated | 22:45 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/820923 may also be good, fwiw | 22:48 |
| TheJulia | since sushy 4.0.0 has all of the enums work and breaks backwards compatability | 22:48 |
| stevebaker[m] | TheJulia: hey do you recall the reason for this master_images change? It seems "security enhancements in Fedora" is the reason I'm changing it back :) https://review.opendev.org/c/openstack/bifrost/+/471750/12/releasenotes/notes/fedora-core-25-support-b9f8f258a9738a06.yaml | 22:49 |
| stevebaker[m] | done | 22:51 |
| TheJulia | stevebaker[m]: uhh basically if memory serves files couldn't be linked | 22:51 |
| TheJulia | but I *think* that got chaned to copy | 22:52 |
| stevebaker[m] | from /tftpboot/master_images to /tftpboot/something/something? | 22:52 |
| TheJulia | from /var/lib/ironic/images/blah to /tftpboot/something/something | 22:52 |
| stevebaker[m] | ok | 22:57 |
| opendevreview | Merged openstack/sushy stable/train: CI: Pin sushy-tools to fix CI https://review.opendev.org/c/openstack/sushy/+/820619 | 23:52 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!