| opendevreview | Julia Kreger proposed openstack/ironic master: Update RAID docs https://review.opendev.org/c/openstack/ironic/+/821782 | 00:02 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/ironic-python-agent stable/ussuri: Re-read the partition table with partx -a https://review.opendev.org/c/openstack/ironic-python-agent/+/821785 | 01:45 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent stable/ussuri: Re-read the partition table with partx -a, part 2 https://review.opendev.org/c/openstack/ironic-python-agent/+/821786 | 01:45 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent stable/victoria: Re-read the partition table with partx -a https://review.opendev.org/c/openstack/ironic-python-agent/+/821787 | 01:52 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent stable/victoria: Re-read the partition table with partx -a, part 2 https://review.opendev.org/c/openstack/ironic-python-agent/+/821788 | 01:52 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent stable/train: Re-read the partition table with partx -a https://review.opendev.org/c/openstack/ironic-python-agent/+/821791 | 01:58 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent stable/train: Re-read the partition table with partx -a, part 2 https://review.opendev.org/c/openstack/ironic-python-agent/+/821792 | 01:58 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Fix Mock objects as specs https://review.opendev.org/c/openstack/ironic/+/821370 | 05:58 |
| arne_wiebalck | Good morning Ironic! | 07:25 |
| arne_wiebalck | stevebaker[m]: Awesome, thanks a lot! | 07:26 |
| janders | good morning arne_wiebalck and Ironic o/ | 07:58 |
| arne_wiebalck | hey janders o/ | 07:58 |
| *** amoralej|off is now known as amoralej | 07:59 | |
| rpittau | good morning ironic! o/ | 08:00 |
| arne_wiebalck | hey rpittau o/ | 08:08 |
| rpittau | hey arne_wiebalck :) | 08:08 |
| dtantsur | morning ironic | 08:51 |
| rpittau | hey dtantsur :) | 08:57 |
| dtantsur | has anyone observed any improvements in the pypi issues? | 08:58 |
| rpittau | dtantsur: which pypi issues? | 09:30 |
| *** sshnaidm is now known as sshnaidm|afk | 09:33 | |
| dtantsur | well, we have most of the jobs failing because of missing packages :) since yesterday | 09:39 |
| rpittau | oh ok, that issue, I rechecked some jobs and it seems they went through, just unit tests though | 09:41 |
| arne_wiebalck | Could someone (briefly :-) remind me why we removed the lower-constraints file in stable/wallaby for the IPA, but then put it back ? | 09:54 |
| rpittau | arne_wiebalck: we're testing lower-constraints in master only, the reason we put it back is to remind us to update the minimum versions of the dependencies that actually work | 10:06 |
| arne_wiebalck | rpittau: ok, thanks ... when porting patches we got confused when there was no l-c in stable/wallaby, but in vic and xena. | 10:08 |
| arne_wiebalck | stable/victoria and stable/xena | 10:08 |
| rpittau | yeah, we haven't put the file back there | 10:09 |
| *** redrobot6 is now known as redrobot | 10:34 | |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/19.0: Adoption: do not validate boot interface when local booting https://review.opendev.org/c/openstack/ironic/+/821820 | 10:55 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/xena: Adoption: do not validate boot interface when local booting https://review.opendev.org/c/openstack/ironic/+/821821 | 10:57 |
| holtgrewe | Hi. I'm using the qcow2 cloud image for CentOS 7.9 but get the following error in ironic conductor logs. What could be going wrong here? >>Could not install bootloader for whole disk image for node 15cc3734-57fa-43be-bc17-7851183e654b, Error: No EFI partition could be detected on device /dev/sda and EFI partition UUID has not been recorded during deployment (which is often the case for whole | 10:58 |
| holtgrewe | disk images). Are you using a UEFI-compatible image?<< | 10:58 |
| dtantsur | holtgrewe: the latter question is one you need answering | 10:59 |
| dtantsur | I think CentOS images do not have an EFI partition | 10:59 |
| dtantsur | you can build one yourself using diskimage-builder with the block-device-efi element | 11:00 |
| dtantsur | see https://docs.openstack.org/ironic/latest/user/creating-images.html | 11:00 |
| *** sshnaidm|afk is now known as sshnaidm | 11:26 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Fix Mock objects as specs https://review.opendev.org/c/openstack/ironic/+/821370 | 11:33 |
| *** tkajinam is now known as Guest8517 | 11:58 | |
| arne_wiebalck | dtantsur: dynamic pairing for network burn-in, "please make tooz/kazoo optional": you mean via an extra-requirements.txt file ? | 12:08 |
| dtantsur | arne_wiebalck: rather via setup.cfg, see ironic-lib | 12:24 |
| dtantsur | https://opendev.org/openstack/ironic-lib/src/branch/master/setup.cfg#L40 | 12:24 |
| holtgrewe | dtantsur: thanks | 12:25 |
| janders | see you tomorrow Ironic o/ | 12:30 |
| opendevreview | Merged openstack/ironic master: Adoption: do not validate boot interface when local booting https://review.opendev.org/c/openstack/ironic/+/821537 | 12:45 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Fix Mock objects as specs https://review.opendev.org/c/openstack/ironic/+/821370 | 13:10 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Fix Mock objects as specs https://review.opendev.org/c/openstack/ironic/+/821370 | 13:12 |
| Alex-dev | hello | 13:21 |
| Alex-dev | I'm trying to inspect new nodes with ironic (tripleo victoria) and got error: | 13:22 |
| Alex-dev | The PXE filter driver DnsmasqFilter, state=uninitialized: my fsm encountered an exception: Can not transition from state 'uninitialized' on event 'sync' | 13:22 |
| dtantsur | Alex-dev: check if you have any errors before that. state=uninitialized is suspicious. | 13:23 |
| *** amoralej is now known as amoralej|lunch | 13:25 | |
| arne_wiebalck | dtantsur: thanks ... not sure I understand what needs to be done, though, sorry: what is the relation of the extra group in setup.cfg with the extra-requirements.txt file? is 'extra' | 13:25 |
| arne_wiebalck | a keyword? are the group names free to chose? what determines whether or not the groups in the extra section are taken into account? does tox.ini need patching as well? (I looked at some docs and patches touching these files for some time now, but that does not seem very efficient) | 13:25 |
| dtantsur | arne_wiebalck: there is no extra-requirements.txt | 13:26 |
| dtantsur | it's just an alternative to using requirements.txt | 13:26 |
| dtantsur | you can use any extra group with `pip install ironic[foobar]` | 13:27 |
| arne_wiebalck | dtantsur: ironic-lib has an extra-requirements.txt | 13:28 |
| dtantsur | I guess we're just using it for unit tests convenient | 13:28 |
| dtantsur | * convenience | 13:28 |
| arne_wiebalck | ok | 13:28 |
| arne_wiebalck | so, all that needs to be done is add a section to setup.cfg? | 13:29 |
| * arne_wiebalck has the feeling this cannot be it :) | 13:29 | |
| arne_wiebalck | I guess the image building will also need to be adapted? | 13:31 |
| rpittau | arne_wiebalck: I actually don't see the need for kazoo in requirements | 13:34 |
| arne_wiebalck | rpittau: didn't work without | 13:34 |
| rpittau | well we don't have any direct import of kazoo so that's weird | 13:35 |
| arne_wiebalck | done from tooz once it figures out which driver we want to use? | 13:36 |
| dtantsur | mmm, a good point from rpittau. we don't e.g. depend on pymysql | 13:36 |
| dtantsur | maybe someone wants to use etcd | 13:37 |
| arne_wiebalck | how we do this in the inspector? | 13:37 |
| arne_wiebalck | which also uses tooz | 13:37 |
| arne_wiebalck | or can use it | 13:37 |
| arne_wiebalck | we do not specify kazoo there | 13:38 |
| rpittau | arne_wiebalck: kazoo is an extra dependency of tooz, it really depends on the driver used there; since you're testing with zookeeper, it requires kazoo | 13:45 |
| arne_wiebalck | right, so this should be a test requirement only? | 13:46 |
| rpittau | yeah | 13:46 |
| rpittau | wondering if the docs also need to mention that | 13:46 |
| rpittau | that will work only if the reqs for the chosen driver are installed in the ipa image | 13:47 |
| arne_wiebalck | right | 13:47 |
| arne_wiebalck | there are no docs yet | 13:47 |
| arne_wiebalck | :-D | 13:47 |
| rpittau | :) | 13:48 |
| arne_wiebalck | it is planned ;) | 13:48 |
| arne_wiebalck | ok, so how can I move fwd: remove kazoo from requirements and lower-constraints, add it to test-requirements? | 13:48 |
| rpittau | I would mention something in the patch somewhere anyway, like in the test suite or in the test-requirements, maybe add one more case at least for another driver | 13:49 |
| rpittau | arne_wiebalck: move kazoo to test-requirements but leave it in l-c | 13:49 |
| arne_wiebalck | rpittau: ok, let's start with this ... then I will need to add it to the image (either explicitly or by adding it as a local patch to requirements.txt ;) ? | 13:51 |
| rpittau | maybe something on ipa-builder side ? | 13:52 |
| dtantsur | TheJulia: oh, wait, node history is enabled by default? I hope we made it very clear in the release notes. | 13:52 |
| rpittau | TheJulia: btw I have 2 PRs open for the ironic-website when you have a minute :) | 13:54 |
| arne_wiebalck | rpittau: yeah, we tried this initially | 13:54 |
| arne_wiebalck | rpittau: did not work immediately (installed in the wrong env, wasn't sure how to get into the correct one in an elegant way), when I figured that adding it to requirements.txt does it :) | 13:55 |
| arne_wiebalck | rpittau: moving kazoo over to test-requirements worked fine, all tests still passing :) | 13:57 |
| rpittau | great :) | 13:57 |
| rpittau | I still think going through ipa-builder is a better solution | 13:57 |
| arne_wiebalck | ok | 13:57 |
| arne_wiebalck | is there a place where this could be added easily ? as mentioned, it seemed this needs to be installed differently than other packages | 13:58 |
| dtantsur | check the extra-hardware element, it also installs a python package | 13:59 |
| arne_wiebalck | dtantsur: thanks! | 13:59 |
| arne_wiebalck | dtantsur: will have a look | 13:59 |
| *** amoralej|lunch is now known as amoralej | 14:00 | |
| arne_wiebalck | ok, so what remains for the network pairing change is making tooz optional | 14:00 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Burn-in: Dynamic network pairing https://review.opendev.org/c/openstack/ironic-python-agent/+/821244 | 14:02 |
| arne_wiebalck | I uploaded the current state of the patch, so it is easier to guide me ;-) | 14:04 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable foreign keys in SQLite https://review.opendev.org/c/openstack/ironic/+/821219 | 14:05 |
| TheJulia | dtantsur: auto-truncates if memory serves at this point. I'd <3 to get client code moving forward | 14:21 |
| dtantsur | okie, cool. I guess we'll disable it for to save some db writes. | 14:22 |
| dtantsur | and good morning | 14:22 |
| TheJulia | good morning | 14:22 |
| TheJulia | dtantsur: could also be useful, but to each their own | 14:23 |
| * TheJulia feels exhausted | 14:23 | |
| dtantsur | yeah, well, we have some history in BMO already :) | 14:23 |
| dtantsur | ouch | 14:23 |
| dtantsur | a bit more sleep maybe? | 14:23 |
| TheJulia | I've got to leave a little early at 1 PM today | 14:27 |
| TheJulia | I'm getting a hair cut, and then going and having a fancy dinner as like... Yay we survivied nearly two years without going to the favorite resturant and now things are going to get worse again 😠| 14:28 |
| TheJulia | I also worked a bit later than normal last night. I looked up and all of a sudden it was like 3 hours later than when I normally try to wrap my day up | 14:29 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Update RAID docs https://review.opendev.org/c/openstack/ironic/+/821782 | 14:42 |
| TheJulia | arne_wiebalck: rpittau: ^^ added you both to take a look at that. It is just a doc change so only one core required. Merging it would be rather helpful for me as I have people reading the master branch docs so I'll likely actually backport some of those items. | 14:44 |
| arne_wiebalck | TheJulia: lgtm, I found some typos and added them to the review | 14:57 |
| TheJulia | arne_wiebalck: ack, I can revise in a minute | 14:57 |
| rpittau | same | 14:57 |
| rpittau | :) | 14:57 |
| *** mnaser_ is now known as mnaser | 14:58 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix Mock objects as specs https://review.opendev.org/c/openstack/ironic/+/821370 | 14:59 |
| *** johnsom_ is now known as johnsom | 15:00 | |
| *** viks___ is now known as viks__ | 15:00 | |
| *** bbezak_ is now known as bbezak | 15:00 | |
| *** erbarr_ is now known as erbarr | 15:01 | |
| opendevreview | Merged openstack/ironic stable/xena: Adoption: do not validate boot interface when local booting https://review.opendev.org/c/openstack/ironic/+/821821 | 15:01 |
| *** parallax_ is now known as parallax` | 15:01 | |
| opendevreview | Merged openstack/ironic bugfix/19.0: Adoption: do not validate boot interface when local booting https://review.opendev.org/c/openstack/ironic/+/821820 | 15:01 |
| *** NobodyCam_ is now known as NobodyCam | 15:02 | |
| *** parallax` is now known as parallax | 15:02 | |
| *** parallax is now known as Guest8535 | 15:03 | |
| TheJulia | arne_wiebalck: you mentioned having to re-install the bios bootloader on machines once, did you ever post anything on that? | 15:06 |
| TheJulia | eh, looks like just grub2-install /path/to/device | 15:08 |
| arne_wiebalck | TheJulia: https://techblog.web.cern.ch/techblog/post/longterm-maintenance-of-bootable-raid-devices/ | 15:08 |
| TheJulia | ahh yes, the missing detail, thanks | 15:11 |
| dtantsur | fancy dinner, nice! I go out nearly exclusively for climbing. | 15:15 |
| *** dking is now known as Guest8538 | 15:15 | |
| arne_wiebalck | Redfish fun: updating the f/w (to see if etags work according to the http rfc on newer versions) removed the BootSourceOverrideMode property ... so now etags still do not work *and* there is a new issue ... not sure if I shall laugh or cry. | 15:16 |
| dtantsur | \o/ | 15:17 |
| TheJulia | arne_wiebalck: oh no... they based a version on the late ?2018? revision? | 15:18 |
| arne_wiebalck | TheJulia: no, seems like mid 2019 | 15:20 |
| TheJulia | There was a version where some vendors interpretted as "remove the setting!" | 15:21 |
| arne_wiebalck | TheJulia: v1.7 | 15:21 |
| TheJulia | others interpretted it as "limit the setting" | 15:21 |
| TheJulia | yeah | 15:21 |
| TheJulia | sounds about right | 15:21 |
| arne_wiebalck | sigh | 15:21 |
| TheJulia | I think they fixed it in 1.8 after the dmtf realized what had happened | 15:22 |
| arne_wiebalck | TheJulia: was this corrected/clarified in newer versions? | 15:22 |
| TheJulia | well, that doesn't mean the vendors did | 15:22 |
| arne_wiebalck | TheJulia: ah, ok | 15:22 |
| TheJulia | I *believe* so, but I think comms on that were restricted to the redfish forum | 15:22 |
| arne_wiebalck | the f/w version is from Sep '21 | 15:22 |
| arne_wiebalck | but the redfish spec they used seems 1.7 | 15:23 |
| rpittau | so it's based on 2 years old spec? ok.... | 15:23 |
| TheJulia | not terribly uncommon, just lagging | 15:23 |
| TheJulia | on a plus side, it doesn't say v1.6 and have v1.8 fields | 15:24 |
| TheJulia | or vise versa! | 15:24 |
| arne_wiebalck | :-D | 15:25 |
| *** parallax_ is now known as parallax | 15:30 | |
| * TheJulia finally eats breakfast | 15:55 | |
| dtantsur | o/ | 16:05 |
| opendevreview | Merged openstack/ironic stable/xena: CI: reduce api worker processes to 1 https://review.opendev.org/c/openstack/ironic/+/821037 | 16:14 |
| opendevreview | Merged openstack/ironic master: Allow enabling fast-track per node https://review.opendev.org/c/openstack/ironic/+/821010 | 16:39 |
| *** Guest8538 is now known as dking | 16:53 | |
| *** sshnaidm is now known as sshnaidm|afk | 17:14 | |
| rpittau | good night! o/ | 17:23 |
| *** amoralej is now known as amoralej|off | 17:42 | |
| arne_wiebalck | bye everyone, see you tomorrow o/ | 18:03 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Update RAID docs https://review.opendev.org/c/openstack/ironic/+/821782 | 18:04 |
| stevebaker[m] | good morning | 19:24 |
| TheJulia | stevebaker[m]: good morning | 19:27 |
| opendevreview | Verification of a change to openstack/ironic stable/victoria failed: CI: Lower test VM memory by 400MB https://review.opendev.org/c/openstack/ironic/+/821140 | 19:42 |
| opendevreview | Merged openstack/metalsmith master: Add unit tests for metalsmith/_nics.py https://review.opendev.org/c/openstack/metalsmith/+/821432 | 20:00 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent stable/victoria: Re-read the partition table with partx -a https://review.opendev.org/c/openstack/ironic-python-agent/+/821787 | 20:09 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent stable/victoria: Re-read the partition table with partx -a, part 2 https://review.opendev.org/c/openstack/ironic-python-agent/+/821788 | 20:09 |
| opendevreview | Steve Baker proposed openstack/python-ironicclient master: Improve description of "node boot mode set" https://review.opendev.org/c/openstack/python-ironicclient/+/821877 | 20:37 |
| stevebaker[m] | Here is a trivial review ^^ | 20:39 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/18.1: Explicitly pin upper sushy, reduce job count https://review.opendev.org/c/openstack/ironic/+/820927 | 20:41 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/18.1: Explicitly pin upper sushy, reduce job count https://review.opendev.org/c/openstack/ironic/+/820927 | 20:41 |
| TheJulia | stevebaker[m]: for some reason I thought it would go all the way through, but I don't remember it anymore :( | 20:42 |
| stevebaker[m] | TheJulia: what would go all the way through what? | 20:44 |
| TheJulia | to sending it to the bmc ad-hoc | 20:44 |
| TheJulia | but I guess it may not have done that | 20:45 |
| * TheJulia prods a couple jobs | 20:48 | |
| TheJulia | I noticed something weird which might be a bug on victoria | 21:01 |
| TheJulia | but... with iscsi deploy | 21:01 |
| TheJulia | I'm likely not going to dig at it | 21:02 |
| TheJulia | stevebaker[m]: a set of eyes on https://review.opendev.org/c/openstack/sushy/+/821324 would be appreciated since I reverted out the other sushy patch | 21:04 |
| TheJulia | and with that, I'm going to go look for my keys and go get a hair cut | 21:05 |
| stevebaker[m] | TheJulia: ok I'll take a look | 21:05 |
| NobodyCam | Good Afternoon Ironic folks! Happy Hump day | 22:42 |
| opendevreview | Merged openstack/python-ironicclient master: Improve description of "node boot mode set" https://review.opendev.org/c/openstack/python-ironicclient/+/821877 | 23:17 |
| NobodyCam | anyone happen to know where in IPA we issue the shutdown before reboot on deploy | 23:25 |
| opendevreview | Merged openstack/ironic stable/victoria: CI: Lower test VM memory by 400MB https://review.opendev.org/c/openstack/ironic/+/821140 | 23:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!