Wednesday, 2020-12-09

*** tosky has quit IRC00:00
*** tkajinam has quit IRC00:16
*** tkajinam has joined #openstack-ironic00:16
iurygregoryhey janders o/00:32
*** rloo has quit IRC00:45
*** ftarasenko has quit IRC00:53
*** benj_ has quit IRC00:54
*** zigo has quit IRC00:54
*** benj_ has joined #openstack-ironic00:54
*** anuradha1904 has quit IRC01:07
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org is being restarted quickly to make heap memory and jgit config adjustments, downtime should be less than 5 minutes01:09
iurygregoryfunny ironic fix complains about pep8,py36... but locally things are fine wtf O.o01:14
*** Goneri has quit IRC01:21
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/ironic/+/76608701:45
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/ironic-python-agent/+/76609801:49
*** zzzeek has quit IRC01:52
*** zzzeek has joined #openstack-ironic01:53
*** zzzeek has quit IRC02:05
*** zzzeek has joined #openstack-ironic02:06
*** lifeless has quit IRC02:25
*** lifeless has joined #openstack-ironic02:27
*** jamesdenton has quit IRC03:17
*** jamesdenton has joined #openstack-ironic03:18
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/ironic/+/76608703:32
*** k_mouza has joined #openstack-ironic03:37
*** k_mouza has quit IRC03:42
openstackgerritSteve Baker proposed openstack/ironic master: Refactor driver_internal_info updates to methods  https://review.opendev.org/c/openstack/ironic/+/76486304:10
openstackgerritSteve Baker proposed openstack/ironic master: WIP Store timestamp on cache_bios_settings  https://review.opendev.org/c/openstack/ironic/+/76439004:10
*** k_mouza has joined #openstack-ironic04:13
*** k_mouza has quit IRC04:17
*** zzzeek has quit IRC04:19
*** ricolin has quit IRC04:21
*** zzzeek has joined #openstack-ironic04:21
*** zzzeek has quit IRC04:26
*** zzzeek has joined #openstack-ironic04:27
*** zzzeek has quit IRC05:26
*** arne_wiebalck has quit IRC05:27
*** arne_wiebalck has joined #openstack-ironic05:29
*** zzzeek has joined #openstack-ironic05:29
*** zzzeek has quit IRC06:15
*** zzzeek has joined #openstack-ironic06:17
*** gyee has quit IRC06:28
*** zzzeek has quit IRC06:49
*** zzzeek has joined #openstack-ironic06:51
*** zzzeek has quit IRC07:21
*** zzzeek has joined #openstack-ironic07:23
arne_wiebalckGood morning, ironic!07:50
*** tosin_ has joined #openstack-ironic07:50
*** lifeless has quit IRC07:54
*** lifeless has joined #openstack-ironic07:56
*** tosin_ has quit IRC08:00
*** tosky has joined #openstack-ironic08:02
*** tosin has joined #openstack-ironic08:05
*** rpittau|afk is now known as rpittau08:14
rpittaugood morning ironic! o/08:14
tosingood morning!08:19
arne_wiebalckHey rpittau and tosin o/08:24
rpittauhey arne_wiebalck :)08:25
*** ociuhandu has joined #openstack-ironic08:42
*** dougsz has joined #openstack-ironic08:50
*** zzzeek has quit IRC08:50
*** zzzeek has joined #openstack-ironic08:50
*** ociuhandu has quit IRC08:52
*** rcernin has quit IRC08:56
*** rcernin has joined #openstack-ironic08:56
*** ociuhandu has joined #openstack-ironic08:59
*** derekh has joined #openstack-ironic09:02
openstackgerritMerged openstack/sushy-cli master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/sushy-cli/+/76607809:03
*** lucasagomes has joined #openstack-ironic09:06
*** tzumainn has quit IRC09:07
*** zzzeek has quit IRC09:16
*** zzzeek has joined #openstack-ironic09:17
*** rcernin has quit IRC09:23
openstackgerritMark Goddard proposed openstack/tenks master: DNM: Test CI  https://review.opendev.org/c/openstack/tenks/+/76616809:26
*** rcernin has joined #openstack-ironic09:32
*** zzzeek has quit IRC09:40
*** k_mouza has joined #openstack-ironic09:40
*** zzzeek has joined #openstack-ironic09:41
*** zzzeek has quit IRC09:46
*** rcernin has quit IRC09:48
*** zzzeek has joined #openstack-ironic09:49
*** rcernin has joined #openstack-ironic09:49
openstackgerritRiccardo Pittau proposed openstack/networking-baremetal master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/networking-baremetal/+/76616909:49
*** zzzeek has quit IRC10:01
*** zzzeek has joined #openstack-ironic10:03
*** rcernin has quit IRC10:31
openstackgerritMark Goddard proposed openstack/tenks master: DNM: Test CI  https://review.opendev.org/c/openstack/tenks/+/76616810:43
*** ociuhandu has quit IRC10:47
*** ociuhandu has joined #openstack-ironic10:51
*** dtantsur|afk is now known as dtantsur10:52
dtantsurmorning ironic10:52
*** rcernin has joined #openstack-ironic10:53
rpittauhey dtantsur :)11:15
rpittaudtantsur: I keep investigating the issues with lower-constraints, in ironic-inspector it still fails but should be close11:15
*** zzzeek has quit IRC11:17
*** zzzeek has joined #openstack-ironic11:18
openstackgerritMark Goddard proposed openstack/tenks master: DNM: Test CI  https://review.opendev.org/c/openstack/tenks/+/76616811:26
openstackgerritVladyslav Drok proposed openstack/networking-generic-switch master: [DO NOT MERGE] Configure physnet for VM bridges  https://review.opendev.org/c/openstack/networking-generic-switch/+/76620211:32
openstackgerritVladyslav Drok proposed openstack/networking-baremetal master: [DO NOT MERGE] Stop supporting VLAN network type  https://review.opendev.org/c/openstack/networking-baremetal/+/76605111:33
*** ociuhandu has quit IRC11:35
*** ociuhandu has joined #openstack-ironic11:49
*** anuradha1904 has joined #openstack-ironic11:50
*** ociuhandu_ has joined #openstack-ironic11:58
*** ociuhandu has quit IRC12:02
*** rcernin has quit IRC12:04
openstackgerritvinay50muddu proposed openstack/ironic master: Add support to manage certificates in iLO  https://review.opendev.org/c/openstack/ironic/+/76057312:05
*** JamesBenson has joined #openstack-ironic12:16
*** tosin_ has joined #openstack-ironic12:49
*** ociuhandu_ has quit IRC12:51
*** ociuhandu has joined #openstack-ironic12:51
*** tosin_ has quit IRC12:54
*** paras333_ has quit IRC13:04
*** ociuhandu has quit IRC13:06
*** ociuhandu has joined #openstack-ironic13:08
*** paras333_ has joined #openstack-ironic13:11
*** ociuhandu has quit IRC13:16
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/victoria: Fix two CI issues  https://review.opendev.org/c/openstack/bifrost/+/76621613:19
*** Goneri has joined #openstack-ironic13:24
*** ociuhandu has joined #openstack-ironic13:24
*** ociuhandu has quit IRC13:24
iurygregorymorning o/13:25
iurygregoryrpittau, dtantsur did you guys found the problem with dib https://zuul.opendev.org/t/openstack/build/60dab31765b24ce985ec269094d5ea55 ?13:25
iurygregorysame problem seems to also affect ipa jobs13:26
dtantsurpreviously it was the 'packaging' package problem13:26
dtantsurnow it seems coming from neutron..13:26
dtantsurno, I haven't investigated that13:26
rpittauit seems neutron is broken, but I haven't dig into that yet13:27
iurygregorygotcha13:27
dtantsuriurygregory: is there any way to make a backportable version of https://review.opendev.org/c/openstack/ironic/+/766087 ? I guess no, but worth asking.13:28
*** k_mouza has quit IRC13:28
*** k_mouza has joined #openstack-ironic13:28
iurygregorywell requirements was complaining =( quite a lot my test was following the commands from the etherpad and I had to update a lot of versions .-.13:29
iurygregorysame for ipa13:29
dtantsuriurygregory: in bifrost I scaled down dogpile.cache to avoid bumping stevedore to 3.013:30
iurygregoryhummmm13:30
iurygregorylet me do some tests locally13:30
rpittauinspector test is still runing after 1 hour, I guess it's time to stop it13:32
iurygregoryrpittau, yeah13:33
iurygregory=(13:33
iurygregoryI gave a try with inspector and I got: loop \o/13:33
dtantsurwow13:34
rpittauI fixed a couple, but it still runs forever13:34
dtantsurrpittau: please keep in mind my comments above, we should try minimizing actual versions bump in case we need to backport this madness13:34
rpittaudtantsur: yeah, that's what I'm trying to do13:35
rpittaukind of hard thoughg as in this case i't six the issue13:36
iurygregorydrop six!13:36
iurygregory\o/13:36
rpittauheh would like that, but other libs use it :)13:36
dtantsurrpittau: if it's not a direct dependency, do whatever you need13:37
dtantsurI'm only worried about bumping requirements.txt13:37
rpittauok :)13:37
openstackgerritDmitry Tantsur proposed openstack/metalsmith master: Move pep8 requirements to tox.ini  https://review.opendev.org/c/openstack/metalsmith/+/76622013:38
rpittauand it doesn't help that I'm dealing with other dependencies mess at the moment.....13:40
dtantsurFYI nova folks are looking at solving this issue for stable branches13:46
rpittauok nice13:47
dtantsurso if we cannot avoid bumping requirements, we should still go for it13:47
iurygregoryack13:47
iurygregorydtantsur, I was looking at https://review.opendev.org/c/openstack/bifrost/+/765985 since you said you scaled down dogpile.cache13:48
iurygregorybut I can't find the scale down O.o13:48
dtantsurso there was a moment where it pulled in newer dogpile.cache, and then I rolled it back..13:49
* dtantsur rubs eyes13:49
dtantsurah, it was in https://review.opendev.org/c/openstack/bifrost/+/765987/2/lower-constraints.txt13:50
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Update lower-constraints from an up-to-date environment  https://review.opendev.org/c/openstack/bifrost/+/76598713:51
rpittauok so I removed one six dependency since I was there, if anyone wants to back me up https://review.opendev.org/c/openstack/automaton/+/76622113:52
iurygregoryI will13:54
*** ricolin has joined #openstack-ironic13:58
*** ociuhandu has joined #openstack-ironic14:00
rpittauthanks!14:06
*** ociuhandu has quit IRC14:07
*** ociuhandu has joined #openstack-ironic14:11
*** rloo has joined #openstack-ironic14:11
rpittauI *think* I fixed inspecto14:18
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/ironic-inspector/+/76622914:20
TheJuliahopefully!14:23
dtantsurmorning TheJulia14:24
*** fdegir0 has joined #openstack-ironic14:24
*** fdegir has quit IRC14:24
iurygregorygood morning TheJulia =)14:25
rpittaugood morning TheJulia :)14:25
TheJuliaarne_wiebalck: thanks for the rbac review14:30
TheJuliaon your first question though, do you mean use system scoped but be able to keep ironic completely project scoped. As in be able to turn on scoping but everyone is still a project member and has rights through that as they would have had before?14:31
arne_wiebalckTheJulia: o/14:31
* arne_wiebalck checks what the question was ...14:32
TheJuliare your second question, it is somewhat implied. admin/member in the project scope with the owner/lessee fields populated should yeild visibility for the users in the model. The thing is today, nova won't record it14:32
TheJuliabut we can have a default disabled option in nova I think14:32
TheJuliaI'm actually wondering if we could get people to be able to also have nova running as a project connecting to the limited scope of the API at some point. It seems feasible even though unintended14:33
arne_wiebalckTheJulia: for the first one, this was mostly to have a smooth transition: how do operators need to configure things in order to basically continue the way they do now ... or is that not feasible?14:34
arne_wiebalckTheJulia: for the second, isn't only when the nodes are instantiated?14:34
* arne_wiebalck is trying to mitigate the lack of flavor quotas14:34
arne_wiebalckTheJulia: Maybe I should describe the use case.14:35
arne_wiebalckTheJulia: As the admin, I'd like to give N nodes to a user. The user will create instances via Nova, but he cannot see how many unless I tell him.14:35
arne_wiebalckTheJulia: If, over time, the user gets more nodes, he will lose track.14:36
*** ociuhandu has quit IRC14:36
TheJulialbragstad: w/r/t oslo policy, is there an eventual planned switch to scope enforcement?14:36
TheJuliaas default14:36
lbragstadTheJulia on the books, no14:37
TheJuliaarne_wiebalck: owner is static set by whoever runs the infra, so in theory that should be visible14:37
arne_wiebalckTheJulia: So, I was thinking if the user can see the nodes I assigned to him, but only these, he can see how many instances he can still create.14:37
TheJulialessee should be nova modified14:37
lbragstadwe've been hesitant to set an explicit date since so many projects are in the early steps14:37
arne_wiebalckTheJulia: The owner would be a project, right?14:37
TheJuliaarne_wiebalck: yes14:37
arne_wiebalckTheJulia: But which role does a user need in order to see only the nodes he owns?14:38
TheJuliaproject admin/member/reader14:38
openstackgerritRiccardo Pittau proposed openstack/ironic-lib master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/ironic-lib/+/76623314:38
TheJuliaso project scope, and any of those three roles14:38
iurygregorydtantsur, seems like for ipa using cliff 3.4.0 will work, I'm doing more tests14:38
arne_wiebalckTheJulia: But don't these role allow to see all nodes?14:38
TheJuliaarne_wiebalck: it is a filtered database query14:38
arne_wiebalckTheJulia: filtered by owner?14:39
TheJuliaif project_id is known on a request and owner OR lessee is populated, the user *should* get back the list of nodes, that is if they have rights to access the API in general and in the new model it would be semi-inherent if their peoject id matches the project id in the database14:40
TheJuliaso if either field is populated, the db query is basically looking for project_id in owner or lessee14:40
arne_wiebalcknice14:40
TheJuliaOn general migration path, I guess a drop in replacement to keep things exactly the same policy file could be defined14:41
arne_wiebalckI was not clear n which nodes would be returned.14:41
TheJuliaarne_wiebalck: yeah, I had to go look at the database queries to confirm it14:41
arne_wiebalckTheJulia: that would solve the flavor quota isse14:41
arne_wiebalckissue14:41
TheJuliaBecause they could do openstack baremetal list, and even if they are a reader, they should be able to see what is in use at that point14:42
arne_wiebalckwhich at least for us is one of the main things users run into14:42
arne_wiebalckTheJulia: right, all nodes, some with instance UUID, some without14:42
TheJuliayup14:42
arne_wiebalckTheJulia: which would be perfect14:42
TheJuliatechnically you can do that today with a custom policy14:42
TheJuliabut the rbac stuff would standardize it14:42
arne_wiebalckcustom policy plus setting owner fields?14:43
*** ociuhandu has joined #openstack-ironic14:43
TheJuliayeah14:43
arne_wiebalckTheJulia: do we have a recipe how to do this?14:43
TheJuliatzumainn likely needs to have a plog post or something14:43
TheJuliahe wrote some docs if memory serves14:43
TheJuliaand there are specs covering it14:43
TheJuliaI'm going to email him14:44
arne_wiebalckawesome, thanks!14:44
arne_wiebalckfor my first point14:44
TheJuliawould something for the baremetal sig where he talks about it be good?14:44
arne_wiebalckoh, yeah!14:44
arne_wiebalckgreat idesa14:44
arne_wiebalckidea14:44
arne_wiebalckfor my first point14:45
arne_wiebalckI don't think it needs to exactly the same, but I am worried that there might be situations when RBAC comes in and access is denied.14:45
arne_wiebalckIf the operator does not understand exactly how things work.14:46
arne_wiebalckSo, default settings that are close to the current defaults may be appreciated :)14:46
TheJuliaFor system scope, they will be default to today14:47
arne_wiebalckAnyway, I think this can all be covered in the docs. But we will need docs :)14:47
arne_wiebalckTheJulia: ok14:47
TheJuliaso you as a system scoped admin should be able to access $things like you do today without any issues, it is project level scoping of $things where $things start to get very restrictive as you've likely seen in the endpoint list.14:47
TheJuliato your project admin change underlying config, that is a good question, and maybe something to work out. Would an operating mode/model where project admin is the same as today's rights also be needed14:48
TheJuliaI guess w/r/t project admins, it could just be a custom policy14:55
arne_wiebalckI guess different deployments have different ways to manage their hardware.14:57
arne_wiebalckIn our case, we have basically one layer for the hardware management, then the users.14:57
arne_wiebalckBut I could imagine scenarios, where there is more than one layer for the hardware.14:58
arne_wiebalcksystem admins at the first layer, project admins at the second14:58
arne_wiebalckwe have this layering for other things already, such as storage14:59
arne_wiebalckwe hand out a huge pools to projects which then sub-divide14:59
arne_wiebalcknot sure this is easy to transfer to hardware15:00
TheJuliaIt is not easy, hence why I still have some TBD and TODO notes and questions in the spec document at this time15:02
openstackgerritRiccardo Pittau proposed openstack/virtualbmc master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/virtualbmc/+/76623515:03
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/ironic-python-agent/+/76609815:04
*** fdegir0 is now known as fdegir15:05
*** akiselev__ has quit IRC15:07
arne_wiebalckTheJulia: Yes. I think we may need to adapt our initial assumptions are confronted with more real world deployments :)15:10
arne_wiebalck*when15:11
*** ociuhandu has quit IRC15:11
*** bburns has quit IRC15:12
*** tzumainn has joined #openstack-ironic15:12
*** hoonetorg has quit IRC15:14
TheJuliatzumainn: We were just atlking about you and a subject that may interest/impact you :)15:14
*** ociuhandu has joined #openstack-ironic15:15
tzumainnTheJulia, hi! what subject would that be?15:15
TheJuliafyi, I updated the rbac spec with more information on model/field anticipations, feedback appreciated15:15
TheJuliatzumainn: rbac and letting lesses and owners view their hardware15:15
TheJuliaI sent you an email btw15:15
tzumainnTheJulia, I'll take a look at the spec, thanks!15:16
TheJuliano problem!15:16
*** bburns has joined #openstack-ironic15:16
tzumainnTheJulia, ah, just saw the email as well - what kind of presentation? if it's recorded, would the one at gave at summit kinda do?15:17
TheJuliaarne_wiebalck: fwiw, I'm thinking with current review bandwidth, at least the system side scoping work of it is do-able15:17
TheJuliatzumainn: we're trying to build up a small collection of baremetal videos for youtube that come from baremetal-sig meetings15:17
TheJuliaso something informal where questions can be asked, etc15:17
tzumainnah, gotcha! when is the next baremetal sig meeting?15:18
TheJuliaa quick 10,000 foot view15:18
TheJuliaJanurary....15:18
* TheJulia pulls up a calendar15:18
TheJuliaJanuary 12th I think15:18
tzumainnI"ll say "yes" because it's far enough away for me to stupidly think that it's someone else's problem :)15:19
tzumainnis there an existing video I could take a look at so I could get a sense of how a presentation might go?15:19
*** rloo has quit IRC15:19
tzumainnoh, and I guess I just realized - you don't mean pre-recorded, you mean a live presentation that's recorded for future persual?15:20
*** ociuhandu has quit IRC15:20
*** rloo has joined #openstack-ironic15:20
TheJuliayeah15:20
TheJuliainformal information/context sharing15:20
TheJuliaand q&a15:20
tzumainnno live demo needed? just a few slides?15:21
TheJuliayup15:22
TheJuliajust a few slides15:22
tzumainnTheJulia, okay! yep, I'll definitely do it then15:22
TheJulia\o/15:23
tzumainnI'll probably ask a few clarifying questions over email, just to make sure I have the answers recorded somewhere15:23
TheJuliaok15:23
*** Goneri has quit IRC15:23
*** Goneri has joined #openstack-ironic15:24
*** ociuhandu has joined #openstack-ironic15:25
arne_wiebalcktzumainn: you *can* do a demo, of course if you like15:29
TheJulialooks like ironic-lib lower constraints are now trying to build cffi15:29
TheJuliaand no finding ffi.h and then *boom*15:29
dtantsurtime to fix bindep?15:30
TheJuliaI guess15:30
*** ociuhandu has quit IRC15:30
tzumainnarne_wiebalck, okay! that kinda depends on the state of my dev environment, which currently is somewhat unhappy15:30
openstackgerritMerged openstack/networking-baremetal master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/networking-baremetal/+/76616915:30
arne_wiebalcktzumainn: understoof15:31
arne_wiebalckd15:31
TheJuliadtantsur: what bindep ;)15:31
arne_wiebalcktzumainn: I'll note this down for Jan 12 and check with you again a week or so before to confirm.15:31
dtantsurmmmm, no bindep for ironic-lib, that explains :)15:31
iurygregory=O15:32
tzumainnarne_wiebalck, okay!15:32
TheJuliaindeed!15:32
iurygregoryin ironic we are going crazy because we don't have oslo-cache on l-c15:32
iurygregoryso it pulls 2.6.1 XD15:32
iurygregorygoing to add 2.5.0 sicne it works with the dogpile.cache 0.9.215:34
TheJuliak15:35
*** ociuhandu has joined #openstack-ironic15:35
TheJuliarpittau: by chance are you looking at ironic-lib bindep, or should I try adding one?15:35
rpittauTheJulia: I'm not looking at that, and I'm scrolling to read the latest comments15:36
rpittauoh yeah, I see the issue with cffi15:37
rpittauTheJulia: I think we need to add that to the pip-resolver patch, I can take care of it15:38
TheJuliarpittau: makes sense to me15:38
TheJuliadtantsur: w/r/t ironic-lib, I'll have a patch I want to backport soonish. Seems we can get two different memory errors out of qemu-img15:39
*** ociuhandu has quit IRC15:39
dtantsur\o/15:39
TheJuliaactually, three! but the check I'm adding covers two of them15:39
* dtantsur cries15:39
TheJuliaYeah... :(15:39
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/ironic/+/76608715:40
iurygregorylet's see what CI will say15:40
TheJuliaI'm also thinking about disabling output caching which reduces it's memory footprint on spawn15:41
*** ociuhandu has joined #openstack-ironic15:43
openstackgerritRiccardo Pittau proposed openstack/ironic-lib master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/ironic-lib/+/76623315:44
rpittau^ this includes bindep now15:44
TheJulia\o/15:46
*** ociuhandu has quit IRC15:47
rpittaulol "ERROR: After October 2020 you may experience errors when installing or updating packages. This is because pip will change the way that it resolves dependency conflicts."15:57
rpittauwell I guess we can't say they didn't warn us15:58
iurygregoryyeah16:01
iurygregoryand problem with the resolver happened once already (if I do remember)16:02
openstackgerritRiccardo Pittau proposed openstack/networking-generic-switch master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/networking-generic-switch/+/76625416:02
dtantsurafter going through all this, I appreciate the wisdom of the rust team who provided package management as a core feature16:06
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/ironic-inspector/+/76622916:20
*** ociuhandu has joined #openstack-ironic16:29
TheJuliarpittau: yeah, I was thinking about it but I thought there was an infra level change to pre-emptively catch it, but I guess it was an experimental job that I didn't think about16:30
openstackgerritRiccardo Pittau proposed openstack/virtualbmc master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/virtualbmc/+/76623516:40
rpittaudtantsur: not sure if you saw this https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/76437016:42
*** lucasagomes has quit IRC17:01
rpittaubye everyone, have a nice rest of the day! o/17:12
*** rpittau is now known as rpittau|afk17:12
TheJuliao/17:13
JayFI landed https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/76437017:19
*** ociuhandu_ has joined #openstack-ironic17:24
*** ociuhandu has quit IRC17:27
*** ociuhandu_ has quit IRC17:28
*** dougsz has quit IRC17:32
TheJuliaWhenever I hear someone says "landed", I think of trying to land on an aircraft carrier17:34
*** ociuhandu has joined #openstack-ironic17:34
iurygregorygerrit down17:34
iurygregory\o/17:34
iurygregoryProxy Error ftw17:35
TheJuliaoh jeeze17:37
iurygregoryyeah17:37
iurygregory*monday*17:37
TheJuliamention it in #opendev17:37
*** ociuhandu has quit IRC17:39
TheJuliaI mentioned it in opendev17:39
*** hoonetorg has joined #openstack-ironic17:41
NobodyCamGood morning Ironic folks, happy hump day17:41
TheJuliagood morning17:42
TheJuliahopefully SN8 will be able to launch today17:42
NobodyCam:)17:44
*** k_mouza has quit IRC17:46
TheJuliaso looks like gerrit's doing GC right now so it might be a little bit of a wait17:48
TheJulianeed to go run an errand, I'll be back in an hour or rwo17:50
*** gyee has joined #openstack-ironic17:57
*** k_mouza has joined #openstack-ironic18:11
*** derekh has quit IRC18:15
*** k_mouza has quit IRC18:15
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org is currently responding slowly or timing out due to resource starvation, investigation is underway18:15
*** k_mouza has joined #openstack-ironic18:17
*** k_mouza has quit IRC18:25
*** dtantsur is now known as dtantsur|afk18:28
dtantsur|afko/18:28
*** dking has quit IRC18:28
openstackgerritMerged openstack/ironic-python-agent-builder master: Workaround for latest pip versions  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/76437018:55
openstackgerritMark Goddard proposed openstack/tenks master: DNM: Test https://github.com/stackhpc/ansible-role-libvirt-host/pull/44  https://review.opendev.org/c/openstack/tenks/+/76627919:03
openstackgerritAija Jauntēva proposed x/sushy-oem-idrac master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/x/sushy-oem-idrac/+/76628419:26
*** dougsz has joined #openstack-ironic19:33
*** dougsz has quit IRC19:47
arne_wiebalckbye everyone o/19:58
*** k_mouza has joined #openstack-ironic19:58
*** k_mouza has quit IRC20:03
*** hoonetorg has quit IRC20:08
TheJuliafyi, in new gerrit, it seems to save when you change pages or occassionally, not immediately :\20:14
*** paras333_ has quit IRC20:20
iurygregoryTheJulia, newbie question, I was looking at the conductors API and we say that hostname is type array but looking at the example I think they are string https://docs.openstack.org/api-ref/baremetal/#conductors-conductors20:24
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Implement System Scoped Role Based Access Control  https://review.opendev.org/c/openstack/ironic-specs/+/76407020:29
TheJuliayeah, that should be a string I think20:30
iurygregoryyeah20:31
*** paras333_ has joined #openstack-ironic20:32
* TheJulia double checks the object model20:32
iurygregoryI was cheking the tests https://github.com/openstack/ironic/blob/2e3245716e6c11f3ec8db026a42efb0b605025a1/ironic/tests/unit/api/controllers/v1/test_conductor.py20:33
TheJuliayup, they are in a list, but yeah20:33
TheJuliaapiref needs to be fixed20:33
TheJuliaugh, pip20:35
iurygregoryack, I was able to notice when looking at the patch on openstacksdk to add the conductors calls20:36
TheJuliaerr, I think we're going to need to drop testing on some of the repos to get job fixes in20:37
iurygregoryyeah .-.20:39
iurygregorythere some problems with diskimage-builder/neutron I think20:40
TheJuliafixing ironic-lib change, I think20:40
TheJuliaI'll check the chagnes before submitting it20:40
TheJuliabut I'm basi9cally at force the fixes in mood20:40
*** ociuhandu has joined #openstack-ironic20:42
TheJuliahmm, they look unrelated failures20:43
openstackgerritVerification of a change to openstack/metalsmith failed: Move pep8 requirements to tox.ini  https://review.opendev.org/c/openstack/metalsmith/+/76622020:43
* TheJulia gives it a spin20:43
openstackgerritJulia Kreger proposed openstack/ironic-lib master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/ironic-lib/+/76623320:43
TheJulialooks like the metalsmith job may just be a race in the test20:46
TheJuliaI think20:46
TheJuliaargument expected order is different which we've seen with py3820:46
TheJuliarechecked it20:46
*** ociuhandu has quit IRC20:46
*** ociuhandu has joined #openstack-ironic20:48
TheJuliaiurygregory: is multinode the only job failing?20:50
* TheJulia looks forward to a test results table20:50
TheJuliagrenade as well20:50
iurygregoryI saw multinode20:51
*** ociuhandu has quit IRC20:52
*** ociuhandu has joined #openstack-ironic20:59
*** ociuhandu has quit IRC21:04
*** ociuhandu has joined #openstack-ironic21:10
openstackgerritMerged openstack/metalsmith master: Move pep8 requirements to tox.ini  https://review.opendev.org/c/openstack/metalsmith/+/76622021:19
TheJuliayay21:20
*** Goneri has quit IRC21:21
TheJuliaiurygregory: I just rechecked the ironic change21:23
TheJuliamultinode... no clue.21:24
TheJuliaI think we'll need to dig deep into that21:24
TheJuliathe grenade looks like a FIP didn't work, it might be neutron, but we can see what is going on21:24
TheJuliasame change on ironic-lib for ipa inbound21:27
TheJuliajust rebuilding my tox env21:27
*** ayoung has joined #openstack-ironic21:44
stevebakergerrit is extra sad today21:50
* stevebaker floats downstream21:52
*** rcernin has joined #openstack-ironic22:02
*** rcernin has quit IRC22:04
*** rcernin has joined #openstack-ironic22:05
*** ociuhandu has quit IRC22:09
iurygregoryTheJulia, tks22:15
iurygregoryif there is any patches we need to merge before let me know =)22:15
*** ociuhandu has joined #openstack-ironic22:15
iurygregoryI was doing some exercises, just got back22:15
TheJuliastevebaker: very sad :(22:16
TheJuliaiurygregory: will take time, unfortuantely with gerrit unahppinesss, it complicates things22:16
TheJuliaI've got a fixed rev, passes locally, but yeah22:17
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/openstack/ironic-python-agent/+/76609822:17
*** ociuhandu has quit IRC22:19
*** ociuhandu has joined #openstack-ironic22:20
openstackgerritRichard G. Pioso proposed x/sushy-oem-idrac master: Fix lower-constraints with the new pip resolver  https://review.opendev.org/c/x/sushy-oem-idrac/+/76628422:21
TheJuliastevebaker: did you get a chance to skim the updated spec yesterday?22:31
stevebakerTheJulia: I was just making my first comment when gerrit said no22:32
stevebakerTheJulia: the comment was going to be: When returned fields are revoked, should we set it to None instead of removing it to reduce the risk of breaking clients22:33
TheJuliaohh, likely22:33
TheJuliayeah, field existance is existed, values being None or empty or {}22:34
TheJuliawouldn't matter22:34
*** ociuhandu has quit IRC22:34
*** ociuhandu has joined #openstack-ironic22:41
*** ociuhandu has quit IRC22:41
TheJuliastevebaker: fwiw, looks like opendev folks are trying to sort things out, unfortunately it is all java GC releated headaches23:05
*** rloo has quit IRC23:32
stevebakerah the java heap, how I don't miss thee23:36
*** k_mouza has joined #openstack-ironic23:41
*** k_mouza has quit IRC23:45

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!