opendevreview | Verification of a change to openstack/ironic stable/xena failed: Fix ironic-lib from source and branch detection in IPA builds https://review.opendev.org/c/openstack/ironic/+/839984 | 02:50 |
---|---|---|
*** rcastillo_ is now known as rcastillo | 02:56 | |
dtantsur | morning folks! https://review.opendev.org/c/openstack/metalsmith/+/843112 works around the metalsmith CI breakage, please review ASAP | 09:47 |
dtantsur | cc arne_wiebalck iurygregory stendulker ^^^ | 09:48 |
opendevreview | Verification of a change to openstack/ironic stable/xena failed: Fix ironic-lib from source and branch detection in IPA builds https://review.opendev.org/c/openstack/ironic/+/839984 | 10:04 |
jm1 | dtantsur: hey :) we have two bifrost jobs in ansible openstack collection which both are currently broken in our stable/1.0.0 branch. they are used as non-voting check jobs in bifrost. are those still actively monitored in bifrost? or do we want to get rid of them? | 10:04 |
dtantsur | hi jm1! I suspect these are centos8 jobs which no longer work with openstack zed and should be replaced with centos9 | 10:27 |
dtantsur | hmm, no, these are ubuntu jobs. have a link handy | 10:28 |
dtantsur | : | 10:28 |
dtantsur | ? | 10:28 |
dtantsur | (and they are voting on bifrost) | 10:30 |
dtantsur | (I mean, the parent jobs) | 10:31 |
dtantsur | the collections jobs specifically passed on https://review.opendev.org/c/openstack/bifrost/+/842332 just a few days ago | 10:33 |
dtantsur | jm1: if you take a look at the failing job, you'll see that https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839693 broke it | 10:36 |
arne_wiebalck | dtantsur: the deprecation warning goes to stdout (rather than stderr?), the patch redirects stderr to a variable ... how does this fix your issue? (happy to move this fwd, just trying to understand :)) | 11:07 |
iurygregory | good morning Ironic, happy towel day! | 11:08 |
iurygregory | dtantsur, +2 | 11:08 |
jm1 | dtantsur: bifrost pulls latest openstacksdk from pypi which is 0.99.x atm. This breaks backwards compatibity to older sdk versions and thus also breaks older releases of ansible openstack collection. | 11:28 |
jm1 | dtantsur: we decided to implement this error message about incompatible sdk versions in ansible openstack collection to prevent more subtle errors in user code. in master branch i reverted this error for now because it broke a lot of not-yet-updated-code but we will (have to) apply it again | 11:30 |
opendevreview | Merged openstack/bifrost master: [CI] Test Kolla on Ubuntu https://review.opendev.org/c/openstack/bifrost/+/842332 | 11:32 |
jm1 | dtantsur: wrt to our bifrost ci jobs in aoc: if nobody would be monitoring them anyway (they are non-voting), then we could drop them from bifrost and aoc. If you rely on them, we need some idea how to fix them for our two branches in aoc: master (sdk>=0.99.0) and stable/1.0.0 (sdk<0.99.0) | 11:35 |
jm1 | dtantsur: maybe it is enough to use bifrost's stable/yoga branch in our bifrost job def in our stable/1.0.0 branch. will test that later | 11:36 |
opendevreview | Takashi Kajinami proposed openstack/python-ironicclient master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/python-ironicclient/+/840071 | 12:23 |
dtantsur | arne_wiebalck: zuul merges stdout and stderr, breaking JSON parsing. so when there is no error, I'm getting rid of stderr completely | 12:52 |
dtantsur | jm1: it's not a question for us if you're going to ignore these jobs further.. | 12:52 |
* TheJulia yawns | 12:52 | |
TheJulia | good morning everyone | 12:53 |
dtantsur | morning TheJulia | 12:53 |
dtantsur | jm1: sorry for being harsh, but your messages seem to be misdirected. nothing in our jobs was broken, neither before 0.99.0 nor after, before the ansible collection started blowing up | 12:58 |
* TheJulia wonders if somebody replaced dtantsur's coffee | 13:01 | |
TheJulia | :) | 13:02 |
dtantsur | well, I'm just trying to prevent the bifrost gate from catching fire AGAIN | 13:02 |
TheJulia | yeah, gates on fire is no good | 13:03 |
TheJulia | speaking of, anyone looked at IPA's general unhappiness with dib yet? | 13:03 |
TheJulia | gah | 13:05 |
TheJulia | I had a unit test failing last night because of a typo | 13:05 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Exclude ansible collection >1.8.0,<2.0.0 https://review.opendev.org/c/openstack/bifrost/+/843281 | 13:06 |
dtantsur | okay, this is hopefully it ^^ | 13:06 |
dtantsur | now I need to figure out why my vim is suddenly using a completely different theme | 13:06 |
TheJulia | O.o | 13:09 |
dtantsur | I think the right answer is to piss me off, but needs verifying :) | 13:09 |
TheJulia | excluding the collection feels like an antipattern to me, but given what I'm aware of with my limited caffination it seems like the only choice to me | 13:10 |
dtantsur | yeah, I'm trying to contain the damage | 13:11 |
TheJulia | yeah | 13:11 |
dtantsur | once we have collections 2.0.0, we can hopefully drop it | 13:11 |
TheJulia | dtantsur: happy thoughts... happy thoughts | 13:11 |
TheJulia | brewdog... happy thoughts | 13:11 |
dtantsur | \o/ | 13:19 |
iurygregory | happy thoughts is the name of a beer we can find in brewdog? | 13:19 |
iurygregory | good morning TheJulia =) | 13:20 |
TheJulia | no, just to have happy thoughts | 13:20 |
dtantsur | iurygregory: I can easily imagine that tbh | 13:20 |
TheJulia | well, they might for all I know have a beer on tap called happy thoughts | 13:20 |
TheJulia | heh | 13:20 |
iurygregory | \o/ | 13:20 |
opendevreview | Harald Jensås proposed openstack/networking-baremetal master: DNM - Testing CI - neutron segment host mapping issue https://review.opendev.org/c/openstack/networking-baremetal/+/842578 | 13:23 |
TheJulia | hi back, what did I do to you to make you unhappy | 13:23 |
opendevreview | Merged openstack/ironic stable/xena: Fix ironic-lib from source and branch detection in IPA builds https://review.opendev.org/c/openstack/ironic/+/839984 | 13:30 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Guard shared device/cluster filesystems https://review.opendev.org/c/openstack/ironic-python-agent/+/839084 | 13:55 |
*** mat_fechner is now known as matfechner | 13:57 | |
TheJulia | hmm, why are we trying to build a centos ramdisk when the job says to build tinyipa... | 14:05 |
TheJulia | nevermind | 14:06 |
TheJulia | I'm blind | 14:06 |
TheJulia | looks like we need to get dmitry's centos9 patch merged | 14:13 |
TheJulia | but Fatal glibc error: CPU does not support x86-64-v2 | 14:13 |
TheJulia | https://developers.redhat.com/blog/2021/01/05/building-red-hat-enterprise-linux-9-for-the-x86-64-v2-microarchitecture-level#architectural_considerations_for_rhel_9 | 14:15 |
jm1 | dtantsur: not sure what you mean. i am trying to evaluate with my questions how much value our bifrost jobs actually have to the bifrost project. since i started hacking on aoc they have been non-voting in aoc and bifrost, which could be an indicator that they are not important for you. | 14:21 |
dtantsur | jm1: they are, we're just being careful with not creating deadlocks | 14:24 |
opendevreview | Julia Kreger proposed openstack/ironic master: devstack: use CentOS 9 for DIB IPA builds https://review.opendev.org/c/openstack/ironic/+/842277 | 14:31 |
TheJulia | dtantsur: ^ hope that will work... the requiring explicit cpu models/features thing is kind of a major break for us | 14:31 |
dtantsur | ouch | 14:32 |
dtantsur | I wonder if we had to do it for bifrost | 14:32 |
TheJulia | dunno | 14:33 |
opendevreview | Merged openstack/metalsmith master: Provide a work around to fix the CI https://review.opendev.org/c/openstack/metalsmith/+/843112 | 14:51 |
jm1 | dtantsur: ack, then we will fix our bifrost jobs in our stable/1.0.0 branch | 14:52 |
dtantsur | jm1: using yoga is probably fine for this case | 14:54 |
dtantsur | meanwhile, the metalsmith job should be fixed. recheck away! | 14:54 |
ameya49 | Hi ironic | 15:30 |
JayF | hi ameya49 | 15:31 |
JayF | Feel free to just ask your question, someone will help if they can :) | 15:31 |
ameya49 | to give a reference and quick recap, was working on introspection test case (ironic_tempest_plugin), where we discussed the nova quota issue. | 15:33 |
ameya49 | Link : https://meetings.opendev.org/irclogs/%23openstack-ironic/%23openstack-ironic.2022-04-11.log.html#t2022-04-11T15:32:41 | 15:33 |
ameya49 | so as a workaround we configured the required quota parameters (cpu, cores) while stacking the environment (devstack) in local.conf and I was able to proceed ahead through that part of that code | 15:34 |
ameya49 | Later, also observed that enabled filters (nova.conf) were filtering the node out causing the test case to fail. Hence, had set "AllHostsFilter" which allows all available host to pass. | 15:43 |
ameya49 | Wanted to know of above 2 approaches for respective scenarios are applicable or any story that needs to be created to check interaction between nova and ironic for investigating/implementing solution above scenarios | 15:44 |
TheJulia | ugh, so ubuntu libvirt doesn't grok maximum | 15:52 |
opendevreview | Julia Kreger proposed openstack/ironic master: devstack: use CentOS 9 for DIB IPA builds https://review.opendev.org/c/openstack/ironic/+/842277 | 15:58 |
TheJulia | lets see if that works... | 15:58 |
TheJulia | ameya49: what exactly are you trying to test that requires nova in the mix for tempest? | 15:59 |
TheJulia | Keep in mind, I'm asking because tempest is API contract testing. Adding tests which require explicit settings to pass in nova... is not ideal. | 16:01 |
dtantsur | we have a long weekend here, see you on Monday o/ | 16:12 |
TheJulia | o/ | 16:12 |
* TheJulia needs a long weekend | 16:12 | |
* TheJulia needs afew weeks | 16:12 | |
TheJulia | stevebaker[m]: you around? | 19:44 |
stevebaker[m] | I will be, having a 1:1 in 15 minutes | 19:45 |
TheJulia | stevebaker[m]: ack | 19:46 |
TheJulia | https://review.opendev.org/c/openstack/ironic/+/842277 seems like it will make CI happy | 19:46 |
TheJulia | hjensas: ^^^ if your around | 19:46 |
stevebaker[m] | TheJulia: reviewed. Also thanks for tackling https://review.opendev.org/c/openstack/diskimage-builder/+/843297 | 21:01 |
stevebaker[m] | TheJulia: pep8 is complaining about tab characters | 21:01 |
TheJulia | gaaah | 21:02 |
TheJulia | I thought I fixed that | 21:02 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Concurrent Distructive/Intensive ops limits https://review.opendev.org/c/openstack/ironic/+/840006 | 21:03 |
TheJulia | arne_wiebalck: ^ okay, passes unit tests. Feedback would be appreciated | 21:03 |
TheJulia | stevebaker[m]: so weird | 21:04 |
TheJulia | it doesn't report locally | 21:04 |
stevebaker[m] | how odd | 21:06 |
TheJulia | stevebaker[m]: so zuul voted +1 | 21:07 |
TheJulia | I must have uploaded it and maybe you looked at the first revision?!? | 21:07 |
TheJulia | *shrugs* | 21:07 |
TheJulia | I dunno | 21:07 |
stevebaker[m] | oh, gerrit was hiding comment lines, including "Uploaded patch set 2." | 21:08 |
stevebaker[m] | +2! | 21:08 |
TheJulia | oh, I super dislike the persistent comment stuff | 21:09 |
TheJulia | if I could burn one thing in the technology universe with fire, it might be that | 21:09 |
stevebaker[m] | show me everything all the time please | 21:10 |
TheJulia | lol | 21:11 |
*** tosky_ is now known as tosky | 21:49 | |
opendevreview | Verification of a change to openstack/ironic master failed: devstack: use CentOS 9 for DIB IPA builds https://review.opendev.org/c/openstack/ironic/+/842277 | 22:26 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!