Tuesday, 2019-09-24

*** rloo has quit IRC01:06
openstackgerritpengyuesheng proposed openstack/virtualbmc master: Update the constraints url  https://review.opendev.org/68416302:08
*** dsneddon has quit IRC02:29
*** ricolin has joined #openstack-ironic02:48
openstackgerritRichard G. Pioso proposed openstack/ironic master: Add Redfish vmedia boot interface to idrac HW type  https://review.opendev.org/67249803:15
openstackgerritMerged openstack/ironic-inspector master: Clean up release notes for 10.0.0  https://review.opendev.org/68399203:46
*** mkrai has joined #openstack-ironic04:22
*** whoami-rajat has joined #openstack-ironic04:33
*** early has quit IRC04:53
*** early` has joined #openstack-ironic05:00
*** bnemec has quit IRC05:09
*** igordc has quit IRC05:12
*** bnemec has joined #openstack-ironic05:15
*** tzumainn has quit IRC05:18
*** hwoarang has quit IRC05:29
*** jtomasek has joined #openstack-ironic05:30
*** hwoarang has joined #openstack-ironic05:32
*** dsneddon has joined #openstack-ironic05:41
*** diga has joined #openstack-ironic05:42
*** dsneddon has quit IRC05:46
arne_wiebalckGood morning, ironic!06:25
arne_wiebalckTheJulia: I'll have a look today. I guess you'd like to have this still in Train?06:26
*** yolanda has quit IRC06:34
*** rpittau|afk is now known as rpittau06:43
rpittaugood morning ironic! o/06:43
*** stendulker has joined #openstack-ironic06:44
*** rachit7 has joined #openstack-ironic06:46
openstackgerritRiccardo Pittau proposed openstack/ironic master: if wholedisk set boot dev to disk  https://review.opendev.org/68326206:53
*** pcaruana has joined #openstack-ironic06:56
*** e0ne has joined #openstack-ironic07:04
*** donnyd has quit IRC07:07
*** tesseract has joined #openstack-ironic07:07
*** donnyd has joined #openstack-ironic07:08
*** e0ne has quit IRC07:09
*** hjensas has quit IRC07:23
*** iurygregory has joined #openstack-ironic07:47
iurygregorygood morning Ironic o/07:51
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Clean up release notes  https://review.opendev.org/68422207:57
rpittauhey iurygregory :)07:57
iurygregoryrpittau, o/07:57
*** priteau has joined #openstack-ironic07:58
*** alexmcleod has joined #openstack-ironic08:05
*** hjensas has joined #openstack-ironic08:09
*** S4ren has joined #openstack-ironic08:13
*** derekh has joined #openstack-ironic08:28
*** mkrai has quit IRC08:41
*** mkrai has joined #openstack-ironic08:42
*** rcernin has quit IRC09:04
*** ricolin_ has joined #openstack-ironic09:05
*** mkowalski__ has quit IRC09:08
*** ricolin has quit IRC09:08
openstackgerritRiccardo Pittau proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection  https://review.opendev.org/68397109:16
openstackgerritMerged openstack/ironic-python-agent master: Clean up release notes  https://review.opendev.org/68422209:22
*** ricolin_ is now known as ricolin09:23
*** mkowalski__ has joined #openstack-ironic09:24
*** ociuhandu has joined #openstack-ironic09:30
*** dsneddon has joined #openstack-ironic09:42
*** dsneddon has quit IRC09:46
*** e0ne has joined #openstack-ironic09:54
*** e0ne has quit IRC09:55
*** e0ne has joined #openstack-ironic09:55
arne_wiebalckTheJulia: dtantsur|afk: I've tried the RAID deletion patch ^^: LGTM. Wasn't clear on whether this is still for Train, so I did not put a W+1.09:58
*** S4ren has quit IRC10:02
*** dsneddon has joined #openstack-ironic10:09
*** openstackstatus has quit IRC10:12
*** openstack has joined #openstack-ironic10:15
*** ChanServ sets mode: +o openstack10:15
openstackgerritThomas Bechtold proposed openstack/ironic master: Drop rootwrap.d/ironic-lib.filters file  https://review.opendev.org/68427010:16
*** dtantsur|afk is now known as dtantsur10:17
dtantsurmorning/afternoon folks10:17
openstackgerritRiccardo Pittau proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection  https://review.opendev.org/68397110:18
dtantsurTheJulia: metalsmith always sets boot_option explicitly (and defaults to local boot)10:20
dtantsurso it is not affected by the suboptimal defaults10:20
*** mkrai_ has joined #openstack-ironic10:22
*** mkrai__ has joined #openstack-ironic10:22
*** pcaruana has quit IRC10:24
*** mkrai has quit IRC10:25
*** mkrai_ has joined #openstack-ironic10:25
dtantsurarne_wiebalck: approved, I think the risk is justified. could you check https://review.opendev.org/#/c/679332/ please?10:25
patchbotpatch 679332 - ironic-python-agent - Attempt tgtd session detach - 4 patch sets10:25
*** mkrai__ has quit IRC10:27
*** bobmel has joined #openstack-ironic10:27
*** bobmel has quit IRC10:30
*** mkrai_ has quit IRC10:30
rpittaudtantsur: I was going to approve this ^^ , I'll wait for arne_wiebalck's feedback10:34
openstackgerritRachit Kapadia proposed openstack/ironic master: DRAC: Fix a bug for delete_config with multiple controllers  https://review.opendev.org/68066010:34
TheJuliadtantsur: that doesn't seem to corrilate with what I remember seeing in logs last week, but *shrug*10:34
*** S4ren has joined #openstack-ironic10:35
arne_wiebalckdtantsur: I agree re the risk. I approved the tgtd patch ^ as well now.10:36
rpittauarne_wiebalck: thanks :)10:36
arne_wiebalckrpittau: :)10:36
*** stendulker has quit IRC10:36
dtantsurTheJulia: it's pretty explicit in this regard: https://github.com/openstack/metalsmith/blob/a77cfb2ce66f965809bb4dd43eb0b03b5787ff6e/metalsmith/_provisioner.py#L40510:37
dtantsurTheJulia: now that I remembered about metalsmith, there is a job that netboots a whole disk image https://021706778c3f130fc2cd-793c55b6fd3f2546ffa515e5af6fce40.ssl.cf2.rackcdn.com/676525/3/check/metalsmith-integration-glance-netboot-cirros-direct/e1732ca/controller/ironic-bm-logs/node-0_no_ansi_2019-08-15-22:55:16.log10:39
dtantsurso silently changing it is probably dangerous10:40
* dtantsur likes how much the metalsmith CI covers :)10:40
*** kaifeng has joined #openstack-ironic10:43
*** pcaruana has joined #openstack-ironic10:48
dtantsurTheJulia, mgoddard, rpittau, kaifeng, arne_wiebalck, (sorry for mass ping) are there any patches that has to be approved before stable/train is branched?10:48
rpittaudtantsur: ironicclient fixes as reported in the etherpad10:48
dtantsuryep, but ironicclient is already branched10:49
* kaifeng has none10:49
rpittauah right :/10:49
mgoddardnot aware of any10:49
rpittauso nothing else, I think10:49
dtantsurmgoddard: why do you think the next ironic-inspector release will be major? I cannot remember breaking changes.10:50
mgoddarddtantsur: you're probably right10:50
dtantsurack, just wanted to double-check10:50
mgoddardstandalone mode is the default?10:50
dtantsuryes, it's still the default10:50
kaifengright10:50
mgoddardok10:50
mgoddardI'll update10:51
kaifengand it still has a hat of experimental in the configuration option :)10:51
arne_wiebalckdtantsur: I am not aware of any.10:51
dtantsurmgoddard: I don't think you need to update anything, the prelude doesn't mention the version10:51
dtantsurjust checking my sanity10:51
mgoddardok, so it was just commit message?10:51
dtantsuryep, and the release note file name (which is not worth bothering updating)10:52
dtantsurmgoddard: okay, I think we can proceed with branching ironic-inspector. would you like to propose the patch?10:52
mgoddarddtantsur: will do10:52
dtantsurironic is pending the release_mapping patch, IPA - the RAID deletion one10:52
dtantsurIPA-builder is branchless, we can do it when everything finally works there :)10:53
dtantsurbifrost.. we need to check that the CI works again. this needs an ironicclient release though.10:53
dtantsurlook, look everyone: DIB images published https://tarballs.openstack.org/ironic-python-agent/dib/files/UPLOAD_RAW/10:54
dtantsur(wrong path though, SIGH)10:54
rpittaulol10:54
kaifeng:D10:54
mgoddarddtantsur: https://review.opendev.org/68427210:56
patchbotpatch 684272 - releases - Ironic inspector: release 9.2.0 and branch stable/... - 1 patch set10:56
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: CI: do not create UPLOAD_* directories when uploading images  https://review.opendev.org/68427410:58
TheJuliaI would <3 boot option to go to local for 13.0, but I guess it will have to be 14 :(10:58
dtantsurokay, fixing the upload ^^^10:58
kaifengdtantsur: the inspector would need packaging change, i guess it's handled by release team?10:58
dtantsurkaifeng: packaging change?10:59
* TheJulia sips coffee and tries to wake up10:59
dtantsurTheJulia: yeah, I don't think we should pull the trigger so late in the cycle :(10:59
dtantsurand g'morning10:59
kaifengbecause of the service split, i guess there would be two or three package just like ironic10:59
kaifengopenstack-ironic-inspector-api, conductor, common, something like that11:00
TheJuliadtantsur: well, we can’t seem to resolve the intermediate issue of the fact that we still try and run a bootloader on whole disk images11:00
dtantsurkaifeng: ah! upstream doesn't handle packaging. I've already updated RDO.11:00
*** ociuhandu has quit IRC11:00
dtantsurTheJulia: yep :( we may need to schedule a call for an hour or two to re-sync on what is happening.11:00
dtantsurwith a few people who understand how this stuff works11:01
TheJuliadtantsur: anyway, to answer the question, nothing that I’m directly aware of at the moment, but jet lag is evil11:01
kaifengdtantsur: cool!11:01
dtantsurTheJulia: jetlag--11:01
TheJuliadtantsur: yeah, kind of wish I had time to do that weeks ago, but it is life.11:01
TheJuliaTwo more weeks... two more weeks11:01
* TheJulia is beginning to question why the hell she is flying home on Thursday11:01
*** ociuhandu has joined #openstack-ironic11:01
dtantsur"have you ever questioned the nature of your reality?"11:03
*** iurygregory_ has joined #openstack-ironic11:03
TheJuliadtantsur: heh!11:04
*** iurygregory has quit IRC11:05
dtantsuralso, can somebody please make me write slides for OpenInfra Nordics? :D it's rapidly approaching.11:05
TheJuliadtantsur: will glaring through the internet help? I was going to ask if someone could whip up what my two speeches should be about today11:06
dtantsur:D11:07
kaifengTheJulia: Thank you for adding me to the project update, just got a notification mail a few days ago :)11:08
*** ociuhandu has quit IRC11:09
*** ociuhandu has joined #openstack-ironic11:09
TheJuliakaifeng: that is good news11:11
*** arxcruz|rover has joined #openstack-ironic11:16
dtantsurrpittau: this may be another fallout of ironicclient 3.0.0 https://logs.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-7-ovb-1ctlr_2comp-featureset020-master/41a87ee/logs/undercloud/home/zuul/overcloud_prep_images.log.txt.gz#_2019-09-23_14_56_3711:16
arxcruz|roverdtantsur: rpittau https://bugs.launchpad.net/tripleo/+bug/1845189 FYI11:17
dtantsurrpittau: probably session is None. could you remind me, do we require a session now?11:17
openstackLaunchpad bug 1845189 in tripleo "overcloud-prep-image.sh failing with 'NoneType' object has no attribute 'get_endpoint'" [Critical,Triaged]11:17
dtantsurmaybe we need to make it explicit?11:17
dtantsuralso can we please just burn ironicclient with fire in Ussuri?11:18
dtantsurtraceback: https://logs.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-7-ovb-1ctlr_2comp-featureset020-master/41a87ee/logs/undercloud/var/log/containers/mistral/executor.log.txt.gz#_2019-09-23_14_56_37_28011:19
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient stable/train: Fix TypeError when using endpoint_override with SessionClient  https://review.opendev.org/68428011:20
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient stable/train: Make the dependency on oslo.config explicit  https://review.opendev.org/68428111:20
*** lucasagomes has joined #openstack-ironic11:21
arxcruz|roverdtantsur: the above patches will fix it ? cuz it's failing on master...11:22
dtantsurarxcruz|rover: no. I think tripleo has to be patched, not ironicclient11:23
dtantsurlooking into it11:23
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Make it clear that a Session is required for v1.client.Client  https://review.opendev.org/68428611:32
dtantsurrpittau: would appreciate a careful review ^^ (esp. that my assumptions are correct)11:32
dtantsurTheJulia: ditto if you have a second11:32
TheJuliaK, Going up to my hotel room in a few minutes11:34
* dtantsur -> lunch11:42
kaifengbye all o/11:49
*** kaifeng has quit IRC11:49
openstackgerritMerged openstack/ironic-python-agent master: Delete_configuration, consider removed raid members as well  https://review.opendev.org/68183811:51
*** rh-jelabarre has joined #openstack-ironic11:54
*** bfournie has quit IRC11:54
rpittaudtantsur: checking now11:56
*** stevebaker has quit IRC11:59
*** stevebaker has joined #openstack-ironic11:59
openstackgerritMerged openstack/ironic-python-agent master: Attempt tgtd session detach  https://review.opendev.org/67933212:00
*** derekh has quit IRC12:00
dtantsurarne_wiebalck: we don't plan on https://review.opendev.org/#/c/674879/ for train, do we?12:02
patchbotpatch 674879 - ironic-python-agent - image extension, install_bootloader improvements - 10 patch sets12:02
dtantsuralso looking for a 2nd review on https://review.opendev.org/#/c/684274/ please (it cannot be checked in the CI, unfortunately)12:03
patchbotpatch 684274 - ironic-python-agent-builder - CI: do not create UPLOAD_* directories when upload... - 1 patch set12:03
rpittaudtantsur: I approved the change, now we just need to decide which combustible to use :)12:04
dtantsurhehe :)12:04
dtantsurTheJulia: it may be a TODO item for Ussuri to do something about ironicclient.. either make SessionClient sane (or at least not so hopelessly insane) or replace it with the SDK12:05
rpittaudtantsur, TheJulia: could be a topic for the PTG12:05
TheJuliaI think the least painful path, honestly is to encourage people to use the SDK12:06
rpittau^^ sustained12:06
TheJuliaThe underlying issue is version negotiation, and that is a tricky issue depending on the environment, and the SDK is where we ultimately want everyone to go to anyway12:06
dtantsuryep12:06
dtantsurSDK is much cleaner in many aspects since it incorporates the experience we've all gained through screwing up python-**clients :)12:07
openstackgerritMerged openstack/ironic-python-agent-builder master: Add documentation on building images  https://review.opendev.org/68037512:07
TheJuliaWe can PTG discuss it, but I'm not sure it is going to get actual traction contributor activity wise. Part of me want us make sure we reserve like half the ptg time for context sharing/building with counterparts, and the other parts for sorting through the work we can actually think we can move forward on in the next cycle12:08
TheJuliaerr, discuss it at the PTG. Brain already failing me this morning12:08
dtantsura random (and probably unpopular) thought: I don't think we can plan cycles as we're used to12:10
dtantsurmost of contributors are affected by downstream priorities too much (even the two of us to an extent)12:11
dtantsurI guess we have to go true-agile12:11
TheJuliadtantsur: I've been trying to resist saying that12:12
dtantsurwell, that's why we have me: to say unpleasant things aloud :D12:12
TheJulia<312:12
*** goldyfruit has quit IRC12:13
dtantsurmgoddard: I think we're good to proceed with IPA now12:17
TheJuliadtantsur: Whatever we do, I want it to lighter weight in the end. OpenStack is boring, and while bare metal is always hopping... because we have to teach the machines to dance in new ways, so there is no real need to inflict pain upon any contributor12:17
dtantsurright12:18
TheJulias/so//12:18
*** rachit7 has quit IRC12:18
TheJuliaI think we're largely back to the real end user problems, to be honest12:19
TheJuliaWhich sounds depressing, but it is a clarity12:19
dtantsurnot too much depressing12:20
*** mkrai has joined #openstack-ironic12:21
TheJuliaso I'm writing notes for one of my speaches, and I joked last time I was in training about taking over the world... and I think I might have been serious.12:21
*** mkrai has quit IRC12:22
*** mkrai has joined #openstack-ironic12:22
dtantsurfinally!12:23
TheJulialol12:24
* TheJulia still has nothing for second speach12:27
* rpittau never jokes about taking over the world :)12:27
*** belmoreira has joined #openstack-ironic12:27
*** S4ren has quit IRC12:30
*** S4ren has joined #openstack-ironic12:30
TheJuliarpittau: but are you plotting it?12:31
TheJuliarpittau: bonus points for volcano fortresses and submarine lairs :)12:31
rpittauTheJulia: well, legendary secret submarine lairs are a real thing in my hometown :D12:32
TheJuliafrom WW2?12:33
rpittauthe origin is not exactly known, could be from WW2 and then reused by NATO12:34
TheJuliaoh wow12:34
TheJuliaso active, nice!12:34
TheJuliaI think I've heard of that one12:34
*** rloo has joined #openstack-ironic12:34
*** rachit7 has joined #openstack-ironic12:36
TheJuliadtantsur: It will be interesting to get feedback from trandles regarding his ironic deployment. Hopefully soonish12:36
TheJuliawell, feedback and perception of needs... above and beyond less reboots which I think eveyrone with large amounts of RAM can agree to.12:37
openstackgerritMerged openstack/ironic master: Update release mappings for Train  https://review.opendev.org/68398412:37
openstackgerritMerged openstack/python-ironicclient stable/train: Make the dependency on oslo.config explicit  https://review.opendev.org/68428112:38
openstackgerritMerged openstack/python-ironicclient stable/train: Fix TypeError when using endpoint_override with SessionClient  https://review.opendev.org/68428012:38
dtantsurmgoddard: I think ironic is now ready as well12:43
*** mkrai has quit IRC12:43
*** bfournie has joined #openstack-ironic12:46
mgoddarddtantsur: ack12:49
openstackgerritMerged openstack/python-ironicclient master: Remove the requirement on either endpoint_override or os_ironic_api_version  https://review.opendev.org/68395212:52
openstackgerritMark Goddard proposed openstack/ironic master: Improve documentation about releasing deliverables  https://review.opendev.org/68430212:52
rpiosoI sent an FFE request to openstack-discuss. How will it be decided?12:53
mgoddardrpioso: it needs 2 cores to sponsor it, and no one has yet. Everything else is ready to go12:56
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add a CI job with a DIB-built ramdisk  https://review.opendev.org/68430412:58
dtantsurand we're about to release ironic...12:58
mgoddarddtantsur: major bump for IPA due to removal of coreos support/13:00
mgoddard?13:00
dtantsurmgoddard: I guess so13:00
mgoddardtrain has had 3.x, 4.x and 5.x13:00
dtantsurlol13:01
dtantsurgotta break'em all!13:01
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient stable/train: Remove the requirement on either endpoint_override or os_ironic_api_version  https://review.opendev.org/68430713:01
* rpioso wonders if there are any cores willing to sponsor https://review.opendev.org/#/c/672498/13:02
patchbotpatch 672498 - ironic - Add Redfish vmedia boot interface to idrac HW type - 21 patch sets13:02
*** ociuhandu has quit IRC13:03
*** mkrai has joined #openstack-ironic13:03
*** henriqueof1 has joined #openstack-ironic13:04
*** henriqueof has quit IRC13:05
mgoddardrpioso: the patch to release ironic has been proposed. I think we're out of time13:05
rpiosomgoddard: Oh well. Onto Ussuri!13:06
mgoddardindeed, it'll be here before we know it13:06
*** jcoufal has joined #openstack-ironic13:07
mgoddardand ironic does intermediate releases, so you could have it released pretty quickly13:07
rpiosoAs far as I understand, it won't be in an LTS until W.13:08
mgoddarddtantsur: inspector, ironic & IPA proposed. Will wait for bifrost. Do we need to release ironic-prometheus-exporter?13:09
* rpioso doesn't understand how intermediate releases apply to new features which aren't backported13:09
dtantsurmgoddard: we do. not sure if it's 1.0.0 or not yet. iurygregory_?13:10
dtantsurmgoddard: I'd probably go with 0.2.0 for now until it sees some early adoption13:10
iurygregory_dtantsur, hey we need to do the release =)13:11
iurygregory_I would say 1.0.013:11
*** iurygregory_ is now known as iurygregory13:11
dtantsuriurygregory: okay, so you expect the current interface and feature set to be pretty stable at this point?13:11
iurygregorydtantsur, yeah for ipmi at least since we validated with real hardware =)13:12
iurygregorynot sure if I'm missing something that openstack would require to be a 1.0.0 release13:13
mgoddard1.0.0 it is13:13
mgoddardbe brave!13:14
iurygregorymgoddard, yeah!13:14
iurygregorydo I need to push a patch for this or only cores/ptl do this?13:15
dtantsuriurygregory: mgoddard will handle the release, I guess13:16
* iurygregory just got an e-mail for review =)13:16
iurygregorydtantsur, tks13:16
dtantsuranyone can do it, but it will require a +1 from the PTL or the current release liaison (me)13:16
iurygregorygotcha13:17
*** sthussey has joined #openstack-ironic13:22
rloodtantsur: how do i stop this from merging so i can update the release note? https://review.opendev.org/#/c/682666/313:23
patchbotpatch 682666 - python-ironicclient - Fix usage of --owner filter during node list - 3 patch sets13:23
dtantsurrloo: W-1 is usually enough. or you can rebase it13:23
rloodtantsur: i did a -1. can't tell if it stopped it or not. well, i'll update it anyway and see what happens.13:24
openstackgerritRuby Loo proposed openstack/python-ironicclient master: Fix usage of --owner filter during node list  https://review.opendev.org/68266613:24
rloodtantsur: well, rebasing worked :)13:24
openstackgerritRuby Loo proposed openstack/python-ironicclient master: Fix usage of --owner filter during node list  https://review.opendev.org/68266613:27
clarkbrloo: dtantsur if you W-1 tests will complete butit wont merge13:27
dtantsurthanks clarkb113:27
dtantsur!13:27
clarkbrebase stops the tests since the old patchset cant merge in gerrit anymore at all13:28
rlooclarkb: good to know. but I don't want any tests to run either (waste of resources)13:28
rlooclarkb: i want a powerful STOP button :D13:28
*** belmoreira has quit IRC13:30
*** belmoreira has joined #openstack-ironic13:31
openstackgerritMerged openstack/python-ironicclient master: Make it clear that a Session is required for v1.client.Client  https://review.opendev.org/68428613:32
*** belmoreira has quit IRC13:32
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient stable/train: Make it clear that a Session is required for v1.client.Client  https://review.opendev.org/68431913:33
*** mkrai has quit IRC13:34
openstackgerritThomas Bechtold proposed openstack/ironic master: Drop rootwrap.d/ironic-lib.filters file  https://review.opendev.org/68427013:35
*** ociuhandu has joined #openstack-ironic13:36
*** goldyfruit has joined #openstack-ironic13:36
*** ociuhandu has quit IRC13:36
*** tzumainn has joined #openstack-ironic13:37
*** ociuhandu has joined #openstack-ironic13:37
*** ociuhandu has quit IRC13:40
*** ociuhandu has joined #openstack-ironic13:41
*** ociuhandu has quit IRC13:53
*** dsneddon has joined #openstack-ironic14:02
goldyfruitrloo, thx for the commit message update14:03
goldyfruitI was commuting14:03
rloogoldyfruit: yw. thx for fixing :)14:03
goldyfruit:014:04
goldyfruit:)14:04
*** ociuhandu has joined #openstack-ironic14:05
*** whoami-rajat has quit IRC14:06
openstackgerritMerged openstack/python-ironicclient stable/train: Make it clear that a Session is required for v1.client.Client  https://review.opendev.org/68431914:07
*** dsneddon has quit IRC14:07
*** belmoreira has joined #openstack-ironic14:08
*** bdodd has joined #openstack-ironic14:09
*** lennyb has quit IRC14:11
*** dsneddon has joined #openstack-ironic14:12
openstackgerritMerged openstack/python-ironicclient stable/train: Remove the requirement on either endpoint_override or os_ironic_api_version  https://review.opendev.org/68430714:12
*** dsneddon has quit IRC14:17
*** whoami-rajat has joined #openstack-ironic14:17
*** dsneddon has joined #openstack-ironic14:17
*** michchap has quit IRC14:19
*** dsneddon has quit IRC14:22
*** lennyb has joined #openstack-ironic14:23
mbuilhey, when I start ironic-inspector service, I get a weird error "ERROR oslo_service.service [-] Error starting thread.: TypeError: expected string or buffer", any idea?14:34
dtantsurmbuil: yes, it's an issue in ironicclient14:36
mbuildtantsur: this is the whole log --> http://paste.openstack.org/show/779188/14:36
dtantsurmbuil: try applying https://review.opendev.org/#/c/684280/ or downgrading ironicclient to < 3.0.014:36
patchbotpatch 684280 - python-ironicclient (stable/train) - Fix TypeError when using endpoint_override with Se... (MERGED) - 1 patch set14:36
*** TxGirlGeek has joined #openstack-ironic14:37
dtantsurrloo, goldyfruit, we should backport it to train asap for it to get into the new release14:38
rloodtantsur: yes sir. was waiting for it to merge, but i can do the needful :)14:40
openstackgerritRuby Loo proposed openstack/python-ironicclient stable/train: Fix usage of --owner filter during node list  https://review.opendev.org/68436314:40
rlooand that ^^ appeared faster than the dialog box (where i clicked the cherrypick thing) disappeared from my browser :D14:41
mbuildtantsur: thanks!14:41
dtantsurrloo: the downside is that it did not add the "cherry-picked" line14:41
dtantsur(it happens when the master patch is not merged yet)14:42
rloodtantsur: dang. i knew there was a reason why i was waiting for it to merge!14:45
*** belmoreira has quit IRC14:50
*** BjoernT_ has joined #openstack-ironic14:50
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: [DNM] testing ironic-inspector CI  https://review.opendev.org/68436714:50
*** belmoreira has joined #openstack-ironic14:51
openstackgerritRuby Loo proposed openstack/python-ironicclient stable/train: Fix usage of --owner filter during node list  https://review.opendev.org/68436314:52
rpittaudtantsur: ironicclient 3.0.0 is also breaking inspector \o/14:53
dtantsurw000t14:53
dtantsurso we've broken pretty much everything?14:53
rpittautested locally, but the dnm I just submitted should fail14:53
rpittauyep14:53
dtantsurwhich side do we have to fix then?14:53
rpittauheh, I was trying to move forward with the sdk migration for ironic-inspector, but at this point it's a bit late14:54
dtantsurrpittau: if inspector needs fixing, we need to tell mgoddard to hold on the release14:56
mgoddardmgoddard in a meeting for 1 hour, can dtantsur handle it?14:56
dtantsursure14:56
rpittaudtantsur: it could be just an issue with the functional tests, tempest is passing14:57
dtantsurah14:57
dtantsurstill needs fixing before we release14:58
rpittauyeah14:58
dtantsurrpittau: do you have an idea of a fix?14:58
*** rachit7 has quit IRC14:58
rpittaudtantsur: not at a glance14:59
dtantsurlemme check locally14:59
*** mkowalski__ has quit IRC15:00
dtantsurrloo: an easy one if you have a second: https://review.opendev.org/#/c/684274/15:00
patchbotpatch 684274 - ironic-python-agent-builder - CI: do not create UPLOAD_* directories when upload... - 1 patch set15:00
*** mkowalski__ has joined #openstack-ironic15:00
rloodtantsur: +A15:01
*** pcaruana has quit IRC15:01
dtantsurthx!15:02
dtantsurrpittau: I think it's the same problem as in bifrost15:06
dtantsuryep, it seems to work with python-ironicclient master15:08
dtantsurrpittau: are there any other issues?15:09
rpittaudtantsur: not that I can see at the moment15:09
dtantsurok15:10
* dtantsur will get really drunk after train is out15:11
dtantsurFYI folks I'm requesting to block ironicclient 3.0.0 from requirements: https://review.opendev.org/#/c/684376/15:17
patchbotpatch 684376 - requirements - Block python-ironicclient 3.0.0 - 1 patch set15:17
mbuildtantsur: I know is a stupid question but... where in the ironicclient source code is the pip version being set? I thought it was in setup.cfg15:19
dtantsurmbuil: we're using a tool called 'pbr' which generates the version from git tags15:20
mbuildtantsur: thanks! I guess this is the key ==> __version__ = pbr.version.VersionInfo('python-ironicclient').version_string()15:22
dtantsuryeah15:22
openstackgerritMerged openstack/python-ironicclient master: Fix usage of --owner filter during node list  https://review.opendev.org/68266615:23
*** lennyb has quit IRC15:25
BjoernT_Hey guys is it feasible these days (stein) to run ironic alongside the normal control plane or are there known issues ?15:27
*** BjoernT_ is now known as BjoernT15:27
dtantsurBjoernT: should be fine. Depends on which sorts of issues you expect. I know people who did that in Newton and before.15:29
BjoernTyeah I recalled everybody was waiting for the cell implementation, to which I did not see a necessary connection15:29
*** lennyb has joined #openstack-ironic15:30
arne_wiebalckBjoernT: we run it with Nova in Stein (and cells), seems to work ok15:30
dtantsurBjoernT: well, before Pike nova tried to use RAM/CPU/disk for scheduling bare metals, which did not always work great15:31
arne_wiebalckBjoernT: this has all move to resource classes15:31
BjoernTok thanks for confirming before I go down the rabbit hole and realizing it shouldn't have worked15:31
dtantsurin Pike an ability to use custom resource classes appeared, and starting with Stein it's the only way to schedule bare metals15:31
BjoernTyes I saw the resource class  changes15:31
*** iurygregory has quit IRC15:32
*** belmoreira has quit IRC15:32
arne_wiebalckBjoernT: "openstack allocation candidate list --resource CUSTOM_BAREMETAL_...=1" will become your friend :)15:35
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Exclude python-ironicclient 3.0.0 from requirements.txt  https://review.opendev.org/68438315:38
*** gyee has joined #openstack-ironic15:44
*** goldyfruit_ has joined #openstack-ironic15:47
*** goldyfruit has quit IRC15:50
*** bobmel has joined #openstack-ironic15:51
openstackgerritMerged openstack/python-ironicclient stable/train: Fix usage of --owner filter during node list  https://review.opendev.org/68436315:52
dtantsurwhy on Earth aren't tempest jobs running on stable/train??15:54
dtantsurso, we've merged a bunch of things that only passed unit tests15:55
dtantsurneat15:55
dtantsurTIL the jobs won't run until ironic is branched >_<15:59
rpittauoh gosh :/16:00
dtantsuryep16:00
goldyfruit_:o16:00
zzzeektoabctl: can you please do a review of https://gerrit.sqlalchemy.org/#/c/sqlalchemy/alembic/+/1477/ so I can do a release16:00
zzzeektoabctl: there's a quaint old system of linking the inspect of those fns to the argparse which i had to hack into in order to accommodate both scenarios16:01
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient stable/train: DNM debug why CI jobs do not run on train  https://review.opendev.org/68439416:03
*** alexmcleod has quit IRC16:06
dtantsurmgoddard: FYI we need to do train ironicclient release (see FFE request on the ML). but we cannot until we figure out why the jobs don't run.16:06
mgoddarddtantsur: ack16:07
* rpittau needs to go16:08
rpittaubye all, good night! o/16:08
*** rpittau is now known as rpittau|afk16:08
*** bobmel has quit IRC16:10
dtantsurmgoddard: update: we cannot do anything apparently :( we need to request an ironicclient release and hope for the better..16:13
dtantsurwill you have time for it today?16:14
* dtantsur has a headache and wants to go home16:14
dtantsurI guess we can think about it tomorrow morning, nothing will change today16:15
*** dtantsur is now known as dtantsur|afk16:15
mgoddarddtantsur|afk: let's discuss tomorrow16:19
openstackgerritMerged openstack/ironic-python-agent-builder master: DIB: do not use /tmp for a virtualenv in the source build  https://review.opendev.org/68392916:19
*** priteau has quit IRC16:19
openstackgerritMark Goddard proposed openstack/ironic master: Improve documentation about releasing deliverables  https://review.opendev.org/68430216:26
openstackgerritMerged openstack/ironic-python-agent-builder master: CI: do not create UPLOAD_* directories when uploading images  https://review.opendev.org/68427416:28
*** yoctozepto has quit IRC16:35
*** yoctozepto has joined #openstack-ironic16:40
*** tesseract has quit IRC16:46
toabctlzzzeek, done. thanks for taking care of it!16:47
*** e0ne has quit IRC16:57
*** ricolin has quit IRC17:03
*** S4ren has quit IRC17:06
*** ociuhandu has quit IRC17:15
*** hjensas has quit IRC17:36
*** sthussey has quit IRC17:39
*** e0ne has joined #openstack-ironic17:47
*** igordc has joined #openstack-ironic17:48
*** diga has quit IRC17:50
*** dsneddon has joined #openstack-ironic18:04
*** e0ne has quit IRC18:09
*** Goneri has joined #openstack-ironic18:10
*** belmoreira has joined #openstack-ironic18:17
*** ociuhandu has joined #openstack-ironic18:18
*** jtomasek has quit IRC18:21
*** bdodd has quit IRC18:22
*** ociuhandu has quit IRC18:23
*** bdodd has joined #openstack-ironic18:24
*** bdodd has quit IRC18:24
*** e0ne has joined #openstack-ironic18:32
*** e0ne has quit IRC18:33
*** Goneri has quit IRC18:37
*** e0ne has joined #openstack-ironic18:54
*** lucasagomes has quit IRC19:00
*** jtwill98 has joined #openstack-ironic19:20
*** sthussey has joined #openstack-ironic19:21
*** hwoarang has quit IRC19:50
*** hwoarang has joined #openstack-ironic19:52
*** whoami-rajat has quit IRC19:56
*** e0ne has quit IRC19:59
*** hwoarang has quit IRC20:00
*** rloo has quit IRC20:03
*** hwoarang has joined #openstack-ironic20:04
*** Lucas_Gray has joined #openstack-ironic20:11
*** jtwill98 has quit IRC20:21
*** belmoreira has quit IRC20:27
*** Lucas_Gray has quit IRC20:27
*** Lucas_Gray has joined #openstack-ironic20:29
*** jcoufal has quit IRC20:34
*** e0ne has joined #openstack-ironic20:38
NobodyCamGood Afternoon Ironic folks20:42
*** rloo has joined #openstack-ironic20:44
rpiosoGood afternoon, NobodyCam21:08
*** rloo has quit IRC21:20
*** bfournie has quit IRC21:47
*** e0ne has quit IRC22:04
*** rcernin has joined #openstack-ironic22:07
*** Lucas_Gray has quit IRC22:12
*** rloo has joined #openstack-ironic22:26
*** rloo has quit IRC22:37
*** rloo has joined #openstack-ironic22:37
*** bfournie has joined #openstack-ironic22:43
*** dtantsur|afk has quit IRC22:51
*** stevebaker has quit IRC22:51
*** stevebaker has joined #openstack-ironic23:06
*** goldyfruit_ has quit IRC23:14
*** BjoernT has quit IRC23:15
*** stevebaker has quit IRC23:19
*** sthussey has quit IRC23:20
*** stevebaker has joined #openstack-ironic23:20
*** BjoernT has joined #openstack-ironic23:45
*** BjoernT_ has joined #openstack-ironic23:49
*** BjoernT has quit IRC23:50

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!