*** mat_fechner is now known as matfechner | 01:53 | |
arne_wiebalck | Good morning Ironic! | 06:27 |
---|---|---|
dtantsur | morning folks | 07:37 |
hjensas | good morning | 07:45 |
muellerbe | hello ironic | 08:34 |
dtantsur | o/ | 08:34 |
muellerbe | I have one question, is there a way to import a cert for e.g. opendev.org git clone? in my actual environment, there is a firewall, adding their cert to the chain 😱️ | 08:36 |
muellerbe | oh, sorry, during the bifrost deploy | 08:36 |
muellerbe | oh, sorry, for the bifrost deploy | 08:36 |
dtantsur | hmmmm | 08:36 |
dtantsur | cannot you add it as trusted to the OS cert storage? | 08:37 |
muellerbe | yes, this is done, ah, ok, we use bifrost deploy in a container, so I have to add this to the container via kolla-ansible | 08:37 |
dtantsur | abstraction layers for the win :D | 08:38 |
dtantsur | well, at least it won't affect your host system | 08:38 |
muellerbe | you are wright | 08:38 |
muellerbe | you are right | 08:38 |
muellerbe | yes, the host system can access through the firewall, but not kolla-ansible in this step | 08:39 |
muellerbe | I will try something, and maybe change to kolla-ansible IRC, thanks | 08:41 |
muellerbe | I imported the cert manual via /usr/local/share/ca-certificates/ and update-ca-certificates, but now I have to investigate, how to do this via ansible, but I think I will discuss this in kolla-ansible channel, so thanks for the hints and have a nice day | 09:31 |
dtantsur | sure, glad that it worked | 09:34 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Swap the metalsmith UEFI job for a legacy one https://review.opendev.org/c/openstack/ironic/+/842083 | 10:47 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Add a metalsmith job with legacy boot https://review.opendev.org/c/openstack/ironic-python-agent/+/842085 | 10:52 |
dtantsur | iurygregory: Arx and I got you a ton of options for https://etherpad.opendev.org/p/ironic-dinner-ois2022Berlin | 11:17 |
iurygregory | dtantsur, tks! | 11:34 |
iurygregory | good morning Ironic o/ | 11:34 |
opendevreview | Bernd Mueller proposed openstack/bifrost master: add sha256sum files for ipa images https://review.opendev.org/c/openstack/bifrost/+/842092 | 11:50 |
opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent bugfix/8.4: DNM Test CI https://review.opendev.org/c/openstack/ironic-python-agent/+/842093 | 11:52 |
iurygregory | oh we don't have all jobs running in 8.4 =) | 11:58 |
opendevreview | Merged openstack/ironic-python-agent master: Remove support for distributions with Python 3.6 https://review.opendev.org/c/openstack/ironic-python-agent/+/841563 | 12:01 |
dtantsur | yeah, probably haven't done the necessary change | 12:01 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.3: Use Yoga constraints for bugfix/8.3 https://review.opendev.org/c/openstack/ironic-python-agent/+/841291 | 12:02 |
dtantsur | I wonder if this ^^^ is finally enough | 12:02 |
TheJulia | brraaaainns | 12:19 |
TheJulia | also covid booster braaaaains | 12:19 |
dtantsur | oh wow! I hope it goes smoothly | 12:20 |
TheJulia | a bit rough so far | 12:20 |
TheJulia | but I got too many plates to keep spinning | 12:20 |
TheJulia | or to have others hold the plate, and spin it | 12:21 |
iurygregory | booster == 3rd dose? or 4th? | 12:21 |
TheJulia | 4th | 12:21 |
iurygregory | oh wow | 12:21 |
TheJulia | Germany's considers validity of boosters to be only 9 months the last time I looked | 12:21 |
TheJulia | And I got mine early here in the states for reasons() | 12:21 |
dtantsur | s/Germany/EU/ I think | 12:22 |
TheJulia | is it the whole of the EU? | 12:22 |
dtantsur | I think they at least tried to come up with a single regulation | 12:22 |
iurygregory | I got my booster in December (still waiting for their certificate....) | 12:22 |
iurygregory | going to use the recharge day to solve this XD | 12:22 |
dtantsur | at this point you may only need it on arrival (dunno) | 12:23 |
TheJulia | I think my airline will require it for me to board the aircraft | 12:23 |
iurygregory | some airlines here in Brazil still requires PCR test before traveling | 12:24 |
TheJulia | I need PCR before I can fly home | 12:25 |
dtantsur | https://www.auswaertiges-amt.de/en/coronavirus/2317268 is the official link | 12:25 |
iurygregory | Fingers crossed that they won't complain that I got the first 2 doses in CZ and the 3rd in Brazil :D | 12:27 |
dtantsur | well, the rules are only talking about the vaccine itself, not where it was done | 12:28 |
TheJulia | at least it is revised may 2nd, as opposed to everything being listed march 1st or something like that | 12:29 |
TheJulia | oh wow, so... the language is slightly different, but starting October 1, 3 doses is absolutely required | 12:32 |
muellerbe | I think in germany the vaccine product is important, e.g. vaccination with Astra and maybe Johnson is not valid | 13:04 |
dtantsur | both are valid even though they're not longer actively used | 13:04 |
dtantsur | Johnson is counted as 1 vaccine nowadays, so not complete | 13:05 |
iurygregory | dtantsur, it worked re https://review.opendev.org/c/openstack/ironic-python-agent/+/841291 only some non-voting jobs failed (we can investigate later I would say) | 14:03 |
dtantsur | w00t! | 14:06 |
iurygregory | yeah =) | 14:06 |
dtantsur | I must admit, maintaining bugfix branches is more painful than I hoped for | 14:06 |
iurygregory | yeah, the amount of config we need to change... Oh God WHY | 14:06 |
dtantsur | okay, I'm updating https://review.opendev.org/c/openstack/ironic-python-agent/+/841293 accordingly | 14:07 |
iurygregory | ++, I'm working on updating 8.4 since we don't have jobs running | 14:07 |
dtantsur | yeah, another side effect of branch mismatches | 14:08 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.1: Use Xena constraints for bugfix/8.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/841293 | 14:10 |
opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent bugfix/8.4: Run Jobs on bugfix 8.4 https://review.opendev.org/c/openstack/ironic-python-agent/+/842093 | 14:15 |
dtantsur | iurygregory: good that we don't have to do the dance around unit tests :) | 14:19 |
iurygregory | yes! | 14:19 |
iurygregory | I think the fix for the ironic-standalone-ipa-src is just override the branches also (since is the json error from werkzeug) | 14:22 |
iurygregory | https://twitter.com/knelson92/status/1526557919801749504 | 14:22 |
iurygregory | \o/ \o/ \o/ \o/ \o/ | 14:22 |
dtantsur | I'm more anxious to be honest :) | 14:23 |
jeyoung | Good Morning, I'm working on enabling arm on openshift, and would like to add some arm support to sushy-tools: https://storyboard.openstack.org/#!/story/2009962 | 14:30 |
jeyoung | Was wondering if anyone had thoughts/objections to this fix? | 14:30 |
iurygregory | interesting.. | 14:33 |
iurygregory | I think it would be good to add support on sushy-tools for arm | 14:34 |
dtantsur | jeyoung: I guess we've already chatted about it, but I can't imagine why we would object | 14:34 |
* iurygregory wondering about CI job if we can add one... | 14:35 | |
dtantsur | I would actually like that | 14:36 |
dtantsur | probably Bifrost-based to avoid the effort of porting nova and other stuff :) | 14:37 |
opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent bugfix/8.3: Fix ironic-standalone-ipa-src in 8.3 https://review.opendev.org/c/openstack/ironic-python-agent/+/842125 | 14:37 |
iurygregory | ++ | 14:37 |
jeyoung | dtantsur: Thanks..I remember talking about it but its been while, so wanted to double check before I moved forward. | 14:41 |
jeyoung | do you all have the ability to build sushy-tools on arm and manifest list? | 14:41 |
dtantsur | not sure I get the question.. most of us don't have access to arm hardware | 14:43 |
* TheJulia suddenly notices another nick with young in the name and gets super confused for about 0.65 seconds | 14:46 | |
iurygregory | dtantsur, I'm a bit lost with https://review.opendev.org/c/openstack/ironic-python-agent/+/842085 , you are adding a new job metalsmith-integration-ipa-src-legacy using metalsmith-integration-glance-centos8-legacy as parent job, but I can't find the parent job in any repo.. O.o | 14:51 |
TheJulia | outstanding metalsmith change? | 14:53 |
iurygregory | ohh right | 14:53 |
iurygregory | forgot to check "Depends-On" | 14:53 |
* iurygregory facepalm | 14:54 | |
dtantsur | :) | 15:03 |
jeyoung | dtantsur: Was asking what it would take to get quay.io/metal3-io/sushy-tools built and published on arm. | 15:24 |
dtantsur | jeyoung: let's move this particular conversation to #cluster-api-baremetal on the kubernetes slack if possible | 15:25 |
dtantsur | I don't really know how multi-arch images are published.. but if you tell us how, we can do it | 15:25 |
jeyoung | will do, thanks! | 15:26 |
opendevreview | Radosław Piliszek proposed openstack/bifrost stable/yoga: Set git_branch to stable/yoga https://review.opendev.org/c/openstack/bifrost/+/842151 | 16:02 |
opendevreview | Merged openstack/ironic-python-agent bugfix/8.3: Use Yoga constraints for bugfix/8.3 https://review.opendev.org/c/openstack/ironic-python-agent/+/841291 | 16:07 |
opendevreview | Merged openstack/ironic-inspector master: The Python 3.6 and Python 3.7 Support has been dropped since zed https://review.opendev.org/c/openstack/ironic-inspector/+/841515 | 16:07 |
opendevreview | Merged openstack/ironic-prometheus-exporter master: The Python 3.6 and Python 3.7 Support has been dropped since zed https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/841516 | 16:07 |
admiyo | doe tox -e pep8 not actually run pep8? | 16:24 |
admiyo | tools/flake8wrap.sh | 16:24 |
iurygregory | metalsmith not working in IPA 8.1 =( | 16:25 |
iurygregory | Cannot install keystone==21.1.0.dev13 because these package versions have conflicting dependencies. | 16:25 |
iurygregory | '-' | 16:26 |
dtantsur | admiyo: no :) | 16:26 |
dtantsur | iurygregory: O___o | 16:26 |
iurygregory | dtantsur, yeah (same reaction here) | 16:26 |
dtantsur | well.. I guess we need to override keystone as well | 16:26 |
iurygregory | https://zuul.opendev.org/t/openstack/build/57f6915db1974bec95ccac0c692ff4e1/log/job-output.txt | 16:26 |
admiyo | Keystone is on lifesupport AFAICT | 16:27 |
admiyo | I just tried running the pep8 code in keystone and i asplodes on Ubuntu LTS | 16:27 |
admiyo | I think python39 vs 3.10 | 16:28 |
dtantsur | Keystone on lifesupport? Oo | 16:28 |
*** janders9 is now known as janders | 16:29 | |
iurygregory | a lot of projects will fail when running py3.10 in the new ubuntu | 16:31 |
iurygregory | =) | 16:31 |
dtantsur | yeah, I also have this problem on Fedora | 16:31 |
dtantsur | `tox -epep8` uses the default Python, which is 3.10 | 16:31 |
dtantsur | admiyo: you can temporary change basepython in tox.ini | 16:31 |
iurygregory | yup =) ^ | 16:31 |
*** emilien-oftc is now known as EmilienM | 16:34 | |
dtantsur | libvirt-python is a bloody nightmare, wtf | 16:37 |
dtantsur | or is it stream9? | 16:38 |
admiyo | dtantsur, but it looks like it uses the packages from the distro. I thought venv could install its own version of pythn | 16:39 |
admiyo | if I just change the basepython, it complains | 16:40 |
admiyo | ironic sets basepython = python3 | 16:40 |
dtantsur | admiyo: python3 is your default python | 16:40 |
dtantsur | if you want a specific one (e.g. because the default 3.10 does not work), you need to be specific | 16:40 |
admiyo | that leaves it wide ope, from 3.0 to 3.11 | 16:40 |
admiyo | or whatever | 16:40 |
admiyo | so on ubuntu LTS (22?) it runs 3.10 | 16:41 |
admiyo | seems to be ok, just not over on keystone | 16:41 |
dtantsur | 3.10 is not yet supported officially by openstack | 16:41 |
admiyo | $ which python | 16:41 |
dtantsur | stuff may or may not work | 16:41 |
admiyo | $ python --version | 16:41 |
admiyo | Python 3.10.4 | 16:41 |
admiyo | Yeah, I know that | 16:41 |
dtantsur | if you want to run tests, install an older python and change basepython temporary | 16:42 |
admiyo | and yet there seems to be no way to get a supported version for Ubuntu | 16:42 |
dtantsur | deadsnakes? | 16:42 |
admiyo | the older versions of Python come out of a ppa...the equivalent of a copr | 16:42 |
dtantsur | https://launchpad.net/~deadsnakes/+archive/ubuntu/ppa | 16:42 |
dtantsur | has 3.9 for jammy | 16:42 |
admiyo | yeah, that is the ppa | 16:42 |
admiyo | I'm working through getting that to work for Keystone. But it means that even to test on ironic I need to hack the tox.ini. | 16:43 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Install wheel to ensure we can install wheels https://review.opendev.org/c/openstack/bifrost/+/842154 | 16:45 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/yoga: Install wheel to ensure we can install wheels https://review.opendev.org/c/openstack/bifrost/+/842155 | 16:46 |
* TheJulia begins the next round of flight hunting | 16:46 | |
dtantsur | admiyo: ironic? you mentioned keystone before? `tox -epep8` passes for me on ironic on python 3.10 | 16:46 |
admiyo | dtantsur, I'm pingponging back and forth between keystone and ironic....bifrost setup | 16:47 |
admiyo | if I try to force an older python on ironic, I get | 16:47 |
admiyo | ERROR: could not install deps [hacking>=4.1.0,<5.0.0, doc8>=0.6.0, pycodestyle>=2.0.0,<3.0.0, flake8-import-order>=0.17.1, Pygments>=2.2.0, bashate>=0.5.1]; v = InvocationError("/opt/openstack/ironic/.tox/pep8/bin/python -m pip install 'hacking>=4.1.0,<5.0.0' 'doc8>=0.6.0' 'pycodestyle>=2.0.0,<3.0.0' 'flake8-import-order>=0.17.1' 'Pygments>=2.2.0' 'bashate>=0.5.1'", 1) | 16:48 |
admiyo | [testenv] | 16:48 |
admiyo | usedevelop = True | 16:48 |
admiyo | basepython = /usr/bin/python3.9 | 16:48 |
dtantsur | why force an older python on *ironic*? you have problems with keystone | 16:48 |
admiyo | Because, as you just pointed out, python 3.10 is not supported | 16:49 |
admiyo | so I am running my tsts against the wrong version of python. Tryhing to figure out how to get a supported version | 16:49 |
admiyo | but it seems the basepython approach I am using is naive | 16:50 |
dtantsur | pep8 passes just fine | 16:51 |
dtantsur | for unit tests, you can specify a version, like in `tox -epy39` (I do it all the time) | 16:51 |
admiyo | I can't actually run them against 39, just 310 | 16:51 |
admiyo | tox -e py3 runs against the system python (3.10) | 16:53 |
dtantsur | yes, but I suggested a different command | 16:53 |
admiyo | dtantsur, I ran it. I got the error I showed above | 16:54 |
admiyo | are developing on Fedora or Centos? | 16:54 |
dtantsur | I have Fedora 35, CI runs mostly on Ubuntu Focal (20.04) | 16:54 |
*** dansmith_ is now known as dansmith | 16:55 | |
admiyo | let me see what happens on Fedora... | 16:55 |
dtantsur | meanwhile, I'm calling it a day, see you tomorrow folks | 16:56 |
admiyo | dtantsur, thanks. The issues seem to be Ubuntu Specific | 16:58 |
admiyo | I actually have a Fedora 35 setup I can use for Ironic and Keystone dev, it just does not line up with what we want to use in production...which will be Ubuntu 20.04 anyway. | 16:59 |
opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Multipath Hardware path handling https://review.opendev.org/c/openstack/ironic-python-agent/+/837039 | 17:10 |
* TheJulia sits on the endless hold with the travel agency | 17:15 | |
iurygregory | now I'm wondering how unit tests were green in check and red in gate '-' | 17:27 |
TheJulia | rpittau: could you pull your wf-1 off of https://review.opendev.org/c/openstack/networking-generic-switch/+/841472, patch has merged on master | 17:37 |
*** melwitt_ is now known as melwitt | 18:08 | |
admiyo | cleaning is going to get its config from hrere, right? /var/lib/ironic/httpboot/pxelinux.cfg/default | 18:12 |
admiyo | and that has | 18:12 |
admiyo | at the end of the kernel config line ipa-ntp-server=1 | 18:12 |
admiyo | Does seem to be better, tho | 18:20 |
TheJulia | truncated?!? | 18:24 |
TheJulia | I thought we added ntp server support into bifrost at some point | 18:24 |
TheJulia | but I dunno | 18:24 |
TheJulia | sooooo long ago | 18:24 |
admiyo | do I have it right, with the =1 ? | 18:25 |
TheJulia | if memory serves, it should be an ip address | 18:26 |
admiyo | So if I change it in the conf file, restart Ironic, I should regen the default file, right? | 18:26 |
admiyo | and the ip would be my controller | 18:26 |
TheJulia | that one, I don't remember | 18:27 |
TheJulia | we don't repave the default fallback files | 18:27 |
TheJulia | since customization tends to be heavy on them | 18:27 |
admiyo | so hack it into place there as well...got it\ | 18:28 |
admiyo | danke | 18:28 |
opendevreview | Julia Kreger proposed openstack/ironic master: Deprecate syslinux https://review.opendev.org/c/openstack/ironic/+/842229 | 20:34 |
TheJulia | stevebaker[m]: ^ | 20:34 |
stevebaker[m] | Cool, thanks | 20:42 |
stevebaker[m] | TheJulia: is there an option for a single grub binary for pxe legacy bios? I'd really rather not have to use grub-mknetdir | 20:44 |
TheJulia | Sometimes in some distros yes…. But you still need the modules which that command places. | 20:46 |
* TheJulia awaits a cat or corgi | 20:46 | |
stevebaker[m] | maybe its time to deprecate legacy bios | 20:46 |
TheJulia | Possibly | 20:46 |
TheJulia | Or start | 20:47 |
TheJulia | The cats are all in low energy mode at the moment, avoiding me :( | 20:47 |
stevebaker[m] | evil mode: paused | 20:59 |
stevebaker[m] | pawsed? | 20:59 |
opendevreview | Merged openstack/metalsmith master: Add a CentOS job with legacy local boot https://review.opendev.org/c/openstack/metalsmith/+/841911 | 21:58 |
iurygregory | TheJulia, rpittau is on PTO this week afaik re https://review.opendev.org/c/openstack/networking-generic-switch/+/841472 | 22:05 |
* iurygregory trying to figure out what changed to make the unit test failure in the multipath patch | 22:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!