Tuesday, 2022-05-17

*** mat_fechner is now known as matfechner01:53
arne_wiebalckGood morning Ironic!06:27
dtantsurmorning folks07:37
hjensasgood morning07:45
muellerbehello ironic08:34
dtantsuro/08:34
muellerbeI 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
muellerbeoh, sorry, during the bifrost deploy08:36
muellerbeoh, sorry, for the bifrost deploy08:36
dtantsurhmmmm08:36
dtantsurcannot you add it as trusted to the OS cert storage?08:37
muellerbeyes, this is done, ah, ok, we use bifrost deploy in a container, so I have to add this to the container via kolla-ansible08:37
dtantsurabstraction layers for the win :D08:38
dtantsurwell, at least it won't affect your host system08:38
muellerbeyou are wright08:38
muellerbeyou are right08:38
muellerbeyes, the host system can access through the firewall, but not kolla-ansible in this step08:39
muellerbeI will try something, and maybe change to kolla-ansible IRC, thanks08:41
muellerbeI 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 day09:31
dtantsursure, glad that it worked09:34
opendevreviewDmitry Tantsur proposed openstack/ironic master: Swap the metalsmith UEFI job for a legacy one  https://review.opendev.org/c/openstack/ironic/+/84208310:47
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent master: Add a metalsmith job with legacy boot  https://review.opendev.org/c/openstack/ironic-python-agent/+/84208510:52
dtantsuriurygregory: Arx and I got you a ton of options for https://etherpad.opendev.org/p/ironic-dinner-ois2022Berlin11:17
iurygregorydtantsur, tks!11:34
iurygregorygood morning Ironic o/11:34
opendevreviewBernd Mueller proposed openstack/bifrost master: add sha256sum files for ipa images  https://review.opendev.org/c/openstack/bifrost/+/84209211:50
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic-python-agent bugfix/8.4: DNM Test CI  https://review.opendev.org/c/openstack/ironic-python-agent/+/84209311:52
iurygregoryoh we don't have all jobs running in 8.4 =)11:58
opendevreviewMerged openstack/ironic-python-agent master: Remove support for distributions with Python 3.6  https://review.opendev.org/c/openstack/ironic-python-agent/+/84156312:01
dtantsuryeah, probably haven't done the necessary change12:01
opendevreviewDmitry 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/+/84129112:02
dtantsurI wonder if this ^^^ is finally enough12:02
TheJuliabrraaaainns12:19
TheJuliaalso covid booster braaaaains12:19
dtantsuroh wow! I hope it goes smoothly12:20
TheJuliaa bit rough so far12:20
TheJuliabut I got too many plates to keep spinning12:20
TheJuliaor to have others hold the plate, and spin it12:21
iurygregorybooster == 3rd dose? or 4th?12:21
TheJulia4th12:21
iurygregoryoh wow12:21
TheJuliaGermany's considers validity of boosters to be only 9 months the last time I looked12:21
TheJuliaAnd I got mine early here in the states for reasons()12:21
dtantsurs/Germany/EU/ I think12:22
TheJuliais it the whole of the EU?12:22
dtantsurI think they at least tried to come up with a single regulation12:22
iurygregoryI got my booster in December (still waiting for their certificate....)12:22
iurygregorygoing to use the recharge day to solve this XD12:22
dtantsurat this point you may only need it on arrival (dunno)12:23
TheJuliaI think my airline will require it for me to board the aircraft12:23
iurygregorysome airlines here in Brazil still requires PCR test before traveling 12:24
TheJuliaI need PCR before I can fly home12:25
dtantsurhttps://www.auswaertiges-amt.de/en/coronavirus/2317268 is the official link12:25
iurygregoryFingers crossed that they won't complain that I got the first 2 doses in CZ and the 3rd in Brazil :D12:27
dtantsurwell, the rules are only talking about the vaccine itself, not where it was done12:28
TheJuliaat least it is revised may 2nd, as opposed to everything being listed march 1st or something like that12:29
TheJuliaoh wow, so... the language is slightly different, but starting October 1, 3 doses is absolutely required12:32
muellerbeI think in germany the vaccine product is important, e.g. vaccination with Astra and maybe Johnson is not valid13:04
dtantsurboth are valid even though they're not longer actively used13:04
dtantsurJohnson is counted as 1 vaccine nowadays, so not complete13:05
iurygregorydtantsur, 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
dtantsurw00t!14:06
iurygregoryyeah =)14:06
dtantsurI must admit, maintaining bugfix branches is more painful than I hoped for14:06
iurygregoryyeah, the amount of config we need to change... Oh God WHY14:06
dtantsurokay, I'm updating https://review.opendev.org/c/openstack/ironic-python-agent/+/841293 accordingly14:07
iurygregory++, I'm working on updating 8.4 since we don't have jobs running14:07
dtantsuryeah, another side effect of branch mismatches14:08
opendevreviewDmitry 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/+/84129314:10
opendevreviewIury 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/+/84209314:15
dtantsuriurygregory: good that we don't have to do the dance around unit tests :)14:19
iurygregoryyes!14:19
iurygregoryI think the fix for the ironic-standalone-ipa-src is just override the branches also (since is the json error from werkzeug)14:22
iurygregoryhttps://twitter.com/knelson92/status/152655791980174950414:22
iurygregory\o/ \o/ \o/ \o/ \o/14:22
dtantsurI'm more anxious to be honest :)14:23
jeyoungGood 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/200996214:30
jeyoungWas wondering if anyone had thoughts/objections to this fix?14:30
iurygregoryinteresting..14:33
iurygregoryI think it would be good to add support on sushy-tools for arm14:34
dtantsurjeyoung: I guess we've already chatted about it, but I can't imagine why we would object14:34
* iurygregory wondering about CI job if we can add one...14:35
dtantsurI would actually like that14:36
dtantsurprobably Bifrost-based to avoid the effort of porting nova and other stuff :)14:37
opendevreviewIury 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/+/84212514:37
iurygregory++14:37
jeyoungdtantsur: Thanks..I remember talking about it but its been while, so wanted to double check before I moved forward. 14:41
jeyoungdo you all have the ability to build sushy-tools on arm and manifest list?14:41
dtantsurnot sure I get the question.. most of us don't have access to arm hardware14:43
* TheJulia suddenly notices another nick with young in the name and gets super confused for about 0.65 seconds14:46
iurygregorydtantsur, 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.o14:51
TheJuliaoutstanding metalsmith change?14:53
iurygregoryohh right14:53
iurygregoryforgot to check "Depends-On"14:53
* iurygregory facepalm14:54
dtantsur:)15:03
jeyoungdtantsur: Was asking what it would take to get quay.io/metal3-io/sushy-tools built and published on arm.15:24
dtantsurjeyoung: let's move this particular conversation to #cluster-api-baremetal on the kubernetes slack if possible15:25
dtantsurI don't really know how multi-arch images are published.. but if you tell us how, we can do it15:25
jeyoungwill do, thanks!15:26
opendevreviewRadosław Piliszek proposed openstack/bifrost stable/yoga: Set git_branch to stable/yoga  https://review.opendev.org/c/openstack/bifrost/+/84215116:02
opendevreviewMerged openstack/ironic-python-agent bugfix/8.3: Use Yoga constraints for bugfix/8.3  https://review.opendev.org/c/openstack/ironic-python-agent/+/84129116:07
opendevreviewMerged 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/+/84151516:07
opendevreviewMerged 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/+/84151616:07
admiyodoe tox -e pep8 not actually run pep8?16:24
admiyotools/flake8wrap.sh16:24
iurygregorymetalsmith not working in IPA 8.1 =(16:25
iurygregoryCannot install keystone==21.1.0.dev13 because these package versions have conflicting dependencies.16:25
iurygregory'-'16:26
dtantsuradmiyo: no :)16:26
dtantsuriurygregory: O___o16:26
iurygregorydtantsur, yeah (same reaction here)16:26
dtantsurwell.. I guess we need to override keystone as well16:26
iurygregoryhttps://zuul.opendev.org/t/openstack/build/57f6915db1974bec95ccac0c692ff4e1/log/job-output.txt16:26
admiyoKeystone is on lifesupport AFAICT16:27
admiyoI just tried running the pep8 code in keystone and i asplodes on Ubuntu LTS16:27
admiyoI think python39 vs 3.1016:28
dtantsurKeystone on lifesupport? Oo16:28
*** janders9 is now known as janders16:29
iurygregorya lot of projects will fail when running py3.10 in the new ubuntu 16:31
iurygregory=)16:31
dtantsuryeah, I also have this problem on Fedora16:31
dtantsur`tox -epep8` uses the default Python, which is 3.1016:31
dtantsuradmiyo: you can temporary change basepython in tox.ini16:31
iurygregoryyup =) ^16:31
*** emilien-oftc is now known as EmilienM16:34
dtantsurlibvirt-python is a bloody nightmare, wtf16:37
dtantsuror is it stream9?16:38
admiyodtantsur, but it looks like it uses the packages from the distro.  I thought venv could install its own version of pythn16:39
admiyoif I just change the basepython, it complains16:40
admiyoironic sets basepython = python316:40
dtantsuradmiyo: python3 is your default python16:40
dtantsurif you want a specific one (e.g. because the default 3.10 does not work), you need to be specific16:40
admiyothat leaves it wide ope, from 3.0 to 3.11 16:40
admiyoor whatever16:40
admiyoso on ubuntu LTS (22?) it runs 3.1016:41
admiyoseems to be ok, just not over on keystone16:41
dtantsur3.10 is not yet supported officially by openstack16:41
admiyo$ which python16:41
dtantsurstuff may or may not work16:41
admiyo$ python --version16:41
admiyoPython 3.10.416:41
admiyoYeah, I know that16:41
dtantsurif you want to run tests, install an older python and change basepython temporary16:42
admiyoand yet there seems to be no way to get a supported version for Ubuntu16:42
dtantsurdeadsnakes?16:42
admiyothe older versions of Python come out of a ppa...the equivalent of a copr16:42
dtantsurhttps://launchpad.net/~deadsnakes/+archive/ubuntu/ppa16:42
dtantsurhas 3.9 for jammy16:42
admiyoyeah, that is the ppa16:42
admiyoI'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
opendevreviewDmitry Tantsur proposed openstack/bifrost master: Install wheel to ensure we can install wheels  https://review.opendev.org/c/openstack/bifrost/+/84215416:45
opendevreviewDmitry Tantsur proposed openstack/bifrost stable/yoga: Install wheel to ensure we can install wheels  https://review.opendev.org/c/openstack/bifrost/+/84215516:46
* TheJulia begins the next round of flight hunting16:46
dtantsuradmiyo: ironic? you mentioned keystone before? `tox -epep8` passes for me on ironic on python 3.1016:46
admiyodtantsur, I'm pingponging back and forth between keystone and ironic....bifrost setup16:47
admiyoif I try to force an older python on ironic, I get16:47
admiyoERROR: 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
admiyousedevelop = True16:48
admiyobasepython = /usr/bin/python3.916:48
dtantsurwhy force an older python on *ironic*? you have problems with keystone16:48
admiyoBecause, as you just pointed out, python 3.10 is not supported16:49
admiyoso I am running my tsts against the wrong version of python.  Tryhing to figure out how to get a supported version16:49
admiyobut it seems the basepython approach I am using is naive16:50
dtantsurpep8 passes just fine16:51
dtantsurfor unit tests, you can specify a version, like in `tox -epy39` (I do it all the time)16:51
admiyoI can't actually run them against 39, just 31016:51
admiyotox -e py3 runs against the system python (3.10)16:53
dtantsuryes, but I suggested a different command16:53
admiyodtantsur, I ran it.  I got the error I showed above16:54
admiyoare developing on Fedora or Centos?16:54
dtantsurI have Fedora 35, CI runs mostly on Ubuntu Focal (20.04)16:54
*** dansmith_ is now known as dansmith16:55
admiyolet me see what happens on Fedora...16:55
dtantsurmeanwhile, I'm calling it a day, see you tomorrow folks16:56
admiyodtantsur, thanks. The issues seem to be Ubuntu Specific16:58
admiyoI 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
opendevreviewVerification of a change to openstack/ironic-python-agent master failed: Multipath Hardware path handling  https://review.opendev.org/c/openstack/ironic-python-agent/+/83703917:10
* TheJulia sits on the endless hold with the travel agency17:15
iurygregorynow I'm wondering how unit tests were green in check and red in gate '-'17:27
TheJuliarpittau: could you pull your wf-1 off of https://review.opendev.org/c/openstack/networking-generic-switch/+/841472, patch has merged on master17:37
*** melwitt_ is now known as melwitt18:08
admiyocleaning is going to get its config from hrere, right? /var/lib/ironic/httpboot/pxelinux.cfg/default18:12
admiyoand that has 18:12
admiyoat the end of the kernel config line ipa-ntp-server=118:12
admiyoDoes seem to be better, tho18:20
TheJuliatruncated?!?18:24
TheJuliaI thought we added ntp server support into bifrost at some point18:24
TheJuliabut I dunno18:24
TheJuliasooooo long ago18:24
admiyodo I have it right, with the =1 ?18:25
TheJuliaif memory serves, it should be an ip address18:26
admiyoSo if I change it in the conf file, restart Ironic, I should regen the default file, right?18:26
admiyoand the ip would be my controller18:26
TheJuliathat one, I don't remember18:27
TheJuliawe don't repave the default fallback files18:27
TheJuliasince customization tends to be heavy on them18:27
admiyoso hack it into place there as well...got it\18:28
admiyodanke18:28
opendevreviewJulia Kreger proposed openstack/ironic master: Deprecate syslinux  https://review.opendev.org/c/openstack/ironic/+/84222920:34
TheJuliastevebaker[m]: ^20:34
stevebaker[m]Cool,  thanks20: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-mknetdir20:44
TheJuliaSometimes in some distros yes…. But you still need the modules which that command places.20:46
* TheJulia awaits a cat or corgi20:46
stevebaker[m]maybe its time to deprecate legacy bios20:46
TheJuliaPossibly20:46
TheJuliaOr start20:47
TheJuliaThe cats are all in low energy mode at the moment, avoiding me :(20:47
stevebaker[m]evil mode: paused20:59
stevebaker[m]pawsed?20:59
opendevreviewMerged openstack/metalsmith master: Add a CentOS job with legacy local boot  https://review.opendev.org/c/openstack/metalsmith/+/84191121:58
iurygregoryTheJulia, rpittau is on PTO this week afaik re https://review.opendev.org/c/openstack/networking-generic-switch/+/84147222: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/!