Thursday, 2020-11-19

*** k_mouza has joined #openstack-ironic00:00
*** k_mouza has quit IRC00:05
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: WIP Add upgrade check, and json2yaml policy handling  https://review.opendev.org/76328600:06
TheJuliagmann: a quick glance would be appreciated since ^^^^ is also adding the status command00:07
*** tosky has quit IRC00:14
*** ayoung has quit IRC00:26
*** ayoung has joined #openstack-ironic00:28
*** ayoung has quit IRC00:35
*** ayoung has joined #openstack-ironic00:37
*** ricolin has joined #openstack-ironic00:37
openstackgerritBob Fournier proposed openstack/ironic master: Add vendor_passthru method for virtual media  https://review.opendev.org/76300701:10
*** Goneri has quit IRC01:29
*** ayoung has quit IRC01:36
*** ayoung has joined #openstack-ironic01:36
*** gyee has quit IRC01:38
*** paras333 has quit IRC01:40
*** alexmcleod_ has joined #openstack-ironic02:00
*** alexmcleod has quit IRC02:03
*** zzzeek has quit IRC02:17
*** zzzeek has joined #openstack-ironic02:19
*** rcernin has quit IRC02:22
*** rcernin has joined #openstack-ironic02:26
*** zzzeek has quit IRC02:35
*** zzzeek has joined #openstack-ironic02:36
*** ayoung has quit IRC02:39
*** ayoung has joined #openstack-ironic02:41
*** mkrai has joined #openstack-ironic02:50
*** rloo has quit IRC02:50
*** ayoung has quit IRC03:07
*** ayoung has joined #openstack-ironic03:07
*** mkrai has quit IRC03:14
*** mkrai_ has joined #openstack-ironic03:14
*** ayoung has quit IRC03:49
*** ayoung has joined #openstack-ironic03:49
*** ianychoi has joined #openstack-ironic04:36
*** mkrai_ has quit IRC04:49
*** mkrai__ has joined #openstack-ironic04:49
*** rh-jlabarre has quit IRC04:58
*** tzumainn has quit IRC05:16
*** ayoung has quit IRC05:20
*** ayoung has joined #openstack-ironic05:20
*** fmuyassarov has joined #openstack-ironic06:00
*** k_mouza has joined #openstack-ironic06:01
*** JamesBenson has quit IRC06:03
*** ayoung has quit IRC06:03
*** ayoung has joined #openstack-ironic06:04
*** JamesBenson has joined #openstack-ironic06:04
*** k_mouza has quit IRC06:06
*** JamesBenson has quit IRC06:09
*** JamesBenson has joined #openstack-ironic06:15
*** ayoung has quit IRC06:20
*** ayoung has joined #openstack-ironic06:20
*** ayoung has quit IRC06:37
*** uzumaki has joined #openstack-ironic06:40
*** ayoung has joined #openstack-ironic06:40
*** mkrai__ has quit IRC06:49
*** mkrai__ has joined #openstack-ironic06:52
openstackgerritMerged openstack/ironic stable/victoria: Fix incorrect network_data.json location  https://review.opendev.org/76263806:55
openstackgerritVerification of a change to openstack/ironic failed: Limit the default value of [api]api_workers to 4  https://review.opendev.org/76313306:55
*** ayoung has quit IRC07:28
*** ayoung has joined #openstack-ironic07:29
iurygregorygood morning Ironic!07:33
*** mkrai__ has quit IRC07:34
*** ayoung has quit IRC07:37
*** ayoung has joined #openstack-ironic07:37
arne_wiebalckGood morning, iurygregory and ironic!07:39
iurygregoryhey arne_wiebalck o/07:40
*** Qianbiao has joined #openstack-ironic07:47
*** ddurst has quit IRC07:48
*** ddurst has joined #openstack-ironic07:48
*** bhagyashris|off is now known as bhagyashris07:55
*** mkrai__ has joined #openstack-ironic08:01
*** uzumaki has quit IRC08:02
*** rpittau|afk is now known as rpittau08:03
rpittaugood morning ironic! o/08:03
iurygregoryhey rpittau o/08:03
rpittauhey iurygregory :)08:05
rpittauif someone has a minute please check https://review.opendev.org/763068 and https://review.opendev.org/76301608:09
patchbotpatch 763068 - sushy-tools - Allow setting debug mode from command line - 3 patch sets08:09
openstackgerritVerification of a change to openstack/ironic failed: Fix disk label to account for UEFI  https://review.opendev.org/76254008:09
patchbotpatch 763016 - ironic - Remove from_dict function from context - 1 patch set08:09
iurygregoryrpittau, done08:16
rpittauthanks iurygregory :)08:16
*** ayoung has quit IRC08:30
*** ayoung has joined #openstack-ironic08:32
*** rcernin has quit IRC08:37
*** tosky has joined #openstack-ironic08:44
*** mgoddard has joined #openstack-ironic08:58
*** lucasagomes has joined #openstack-ironic09:02
openstackgerritMerged openstack/sushy-tools master: Allow setting debug mode from command line  https://review.opendev.org/76306809:18
*** ricolin has quit IRC09:38
*** ociuhandu has joined #openstack-ironic09:41
*** dtantsur|afk is now known as dtantsur09:41
dtantsurgood morning folks09:41
dtantsurTheJulia, JayF, any JSON is a valid YAML, I guess they're changing formats...09:43
iurygregorymorning dtantsur09:45
*** k_mouza has joined #openstack-ironic09:57
openstackgerritVerification of a change to openstack/ironic failed: Remove from_dict function from context  https://review.opendev.org/76301610:04
jandersgood morning iurygregory arne_wiebalck rpittau dtantsur and Ironic o/10:04
rpittauhey janders :)10:04
iurygregoryjanders, o/10:04
dtantsurhey-hey10:12
arne_wiebalckhey janders o/10:18
openstackgerritAnkit Kumar proposed openstack/ironic-tempest-plugin master: Adding firmware update standalone gate  https://review.opendev.org/76334010:24
openstackgerritAnkit Kumar proposed openstack/ironic master: Adds config params to support fw-update gate  https://review.opendev.org/76334110:25
*** ociuhandu has quit IRC10:36
*** ayoung has quit IRC10:39
*** mkrai__ has quit IRC10:40
*** ayoung has joined #openstack-ironic10:41
*** ociuhandu has joined #openstack-ironic10:43
*** mkrai__ has joined #openstack-ironic10:48
*** uzumaki has joined #openstack-ironic10:57
*** bnemec has quit IRC11:00
*** bnemec has joined #openstack-ironic11:01
*** ociuhandu has quit IRC11:08
*** thiagop has joined #openstack-ironic11:10
*** Qianbiao has quit IRC11:13
*** Qianbiao has joined #openstack-ironic11:14
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Copy any configuration from the virtual media  https://review.opendev.org/76320711:21
*** ayoung has quit IRC11:24
*** ayoung has joined #openstack-ironic11:27
*** ociuhandu has joined #openstack-ironic11:28
*** tkajinam has quit IRC11:29
*** tkajinam has joined #openstack-ironic11:30
*** mushrushu has quit IRC11:40
*** ayoung has quit IRC11:40
*** ociuhandu has quit IRC11:41
*** ociuhandu has joined #openstack-ironic11:42
*** mushrushu has joined #openstack-ironic11:43
*** ayoung has joined #openstack-ironic11:44
*** mkrai__ has quit IRC11:47
openstackgerritMerged openstack/ironic stable/victoria: Limit the default value of [api]api_workers to 4  https://review.opendev.org/76313311:59
openstackgerritMerged openstack/ironic-tempest-plugin master: Run redfish victoria job on focal  https://review.opendev.org/76010212:07
*** ayoung has quit IRC12:13
*** ayoung has joined #openstack-ironic12:14
*** ricolin has joined #openstack-ironic12:18
*** ociuhandu has quit IRC12:22
*** thiagop has quit IRC12:25
*** thiagop has joined #openstack-ironic12:25
openstackgerritBob Fournier proposed openstack/ironic master: Add vendor_passthru method for virtual media  https://review.opendev.org/76300712:33
*** fmuyassarov has quit IRC12:43
*** zzzeek has quit IRC12:45
*** zzzeek has joined #openstack-ironic12:47
*** ociuhandu has joined #openstack-ironic12:52
*** fmuyassarov has joined #openstack-ironic12:56
*** ociuhandu has quit IRC12:57
*** rh-jlabarre has joined #openstack-ironic13:07
*** ayoung has quit IRC13:15
*** rpittau is now known as rpittau|brb13:16
* rpittau|brb upgrade && reboot13:16
*** ayoung has joined #openstack-ironic13:17
*** ayoung has quit IRC13:21
*** ayoung has joined #openstack-ironic13:22
*** ociuhandu has joined #openstack-ironic13:24
*** uzumaki has quit IRC13:36
*** fmuyassarov has quit IRC13:42
*** Goneri has joined #openstack-ironic13:47
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Copy any configuration from the virtual media  https://review.opendev.org/76320713:49
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] inject TLS certificate when using virtual media  https://review.opendev.org/75842713:51
*** ociuhandu has quit IRC14:02
*** ociuhandu has joined #openstack-ironic14:02
*** rloo has joined #openstack-ironic14:15
TheJuliagood morning14:18
TheJuliahttps://review.opendev.org/#/c/748049/ would love reviews14:19
patchbotpatch 748049 - ironic-python-agent - Support partition image efi contents - 27 patch sets14:19
TheJuliadtantsur: formats because of the secure rbac work14:19
iurygregorygood morning TheJulia14:26
TheJuliaajya: configuration molds call in like a half an hour?14:31
ajyaTheJulia: good morning, yes14:32
dtantsurmorning TheJulia14:32
dtantsurack re formats14:32
*** rloo has quit IRC14:34
*** rloo has joined #openstack-ironic14:34
*** ociuhandu has quit IRC14:39
*** yolanda has quit IRC14:40
*** tzumainn has joined #openstack-ironic14:41
* TheJulia needs more coffee14:44
openstackgerritMerged openstack/ironic master: Don't allow patching port internal_info  https://review.opendev.org/76327814:46
*** ayoung has quit IRC14:47
*** mgoddard has quit IRC14:48
*** ayoung has joined #openstack-ironic14:48
*** rpittau|brb is now known as rpittau14:51
ajyaconfiguration mold meeting now at https://Dell.zoom.us/j/92026404383?pwd=MzMvUmtXSmZVbTdYN3dwdFl6SG1VUT0915:00
dtantsurajya: "this meeting is for authorized attendees only"15:01
ajyadtantsur: will check15:01
TheJuliaI retried and it worked15:01
dtantsurnope15:02
TheJuliado we want to just use meetpad?15:03
TheJuliahttps://meetpad.opendev.org/ironic15:03
rpioso+115:03
dtantsurtried logging in with my @rh address, but the meeting has been ended?15:03
*** mkrai has joined #openstack-ironic15:04
TheJuliazoom is evil15:04
*** ociuhandu has joined #openstack-ironic15:05
*** mgoddard has joined #openstack-ironic15:07
*** k_mouza has quit IRC15:15
*** tosky has quit IRC15:18
*** tosky has joined #openstack-ironic15:22
openstackgerritMerged openstack/ironic master: Convert volume/connectors endpoint to plain JSON  https://review.opendev.org/75085915:23
openstackgerritMerged openstack/ironic master: Convert volume/targets endpoint to plain JSON  https://review.opendev.org/75086015:23
*** tosky has quit IRC15:26
*** k_mouza has joined #openstack-ironic15:28
*** tosky has joined #openstack-ironic15:50
*** mkrai has quit IRC15:50
gmannTheJulia: overall lgtm, same comment to replace Victoria->Wallaby for deprecation cycle15:59
*** ociuhandu_ has joined #openstack-ironic16:00
TheJuliagmann: ack16:02
TheJuliagmann: any plans of rallying any resources to look for and stamp out embedded json policies in tests?16:03
TheJuliaI've not looked at ironic-inspector's yet but I noticed a lot of deprecation notes when I ran unit tests last night. :\16:03
*** ociuhandu has quit IRC16:03
*** diurnalist has joined #openstack-ironic16:03
gmannTheJulia: right, i forgot but if can change here then warning should go away - https://github.com/openstack/ironic-inspector/blob/e7c3218f718bf1bef17ff38dc6fee945db29a2c1/ironic_inspector/test/unit/policy_fixture.py#L3416:04
*** diurnalist has quit IRC16:04
gmannsame for ironic too.16:04
*** diurnalist has joined #openstack-ironic16:04
TheJuliaheh, I think there is actual json content embedded int he tests, but that is for when I crack the code back open today.16:07
TheJuliaFirst.. breakfast, then back into the code mill16:07
gmann+116:08
*** Qianbiao has quit IRC16:14
TheJuliagmann: out of curiosity, any known issues with jsonschema on lower-requirements?16:25
*** ayoung has quit IRC16:26
gmannTheJulia: there was version compatible one few month back, let me search16:26
*** ayoung has joined #openstack-ironic16:28
gmannTheJulia: i remember this one but Ironic also have this version bump - https://review.opendev.org/#/c/730960/16:29
patchbotpatch 730960 - nova - Cap jsonschema 3.2.0 as the minimal version (MERGED) - 1 patch set16:29
gmann3.2.0 is min version required16:29
openstackgerritVladyslav Drok proposed openstack/ironic-python-agent stable/victoria: Fix vendor info retrieval for some versions of lshw  https://review.opendev.org/76340616:31
*** gyee has joined #openstack-ironic16:34
TheJuliayeah, Interesting I'm seeing some odd behavior on inspector16:38
* TheJulia nukes .tox and starts over to be on the super safe side16:38
*** ociuhandu_ has quit IRC16:40
*** ociuhandu has joined #openstack-ironic16:42
*** ociuhandu has quit IRC16:48
*** gyee has quit IRC16:50
rpittaugood night! o/16:52
*** rpittau is now known as rpittau|afk16:52
*** ociuhandu has joined #openstack-ironic16:52
*** lucasagomes has quit IRC17:01
TheJuliagmann: when exactly is oslo.policy going to remove json support?17:03
*** gyee has joined #openstack-ironic17:05
gmannTheJulia: plan is after all the services finish this deprecation which is in Wallaby so may be in X or Y we can remove it completely from oslo_policy.17:17
TheJuliaokay, so I get your comment regarding stating Wallaby instead of victoria because the underlying library deprecated17:18
gmannI prefer to do in X but that is something we can discuss in next PTG or so and by operator feedback17:18
gmannTheJulia: 1 sec.17:19
openstackgerritJulia Kreger proposed openstack/ironic master: Policy json to yaml migration  https://review.opendev.org/76326217:20
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: WIP Add upgrade check, and json2yaml policy handling  https://review.opendev.org/76328617:22
gmannTheJulia: we can use the sample text mentioned in goal doc (i commented in your patch) to make it clear that it is deprecation from Ironic17:24
* TheJulia goes and looks at your comment17:25
arne_wiebalckbye everyone o/17:26
TheJuliaso oslo.policy's release notes say it was deprecated in victoria17:26
gmannyeah, deprecation from service side can be explicit on when service deprecate it.17:27
*** ociuhandu_ has joined #openstack-ironic17:28
TheJuliait is not necessarilly the service though that is forcing it, it is oslo.policy that is forcing this. Projects have little choice in the mater.17:29
gmannbecause default value of oslo_policy.policy_file config option and its default value is per service and changing that value to policy.yaml + deprecating it from service side .17:29
TheJuliaTrue17:29
*** ociuhandu has quit IRC17:31
TheJuliaAhh, I see your comment. You raise a good point, realistically both need to be noted as there are people who don't always increment requirements automatically.17:31
gmannyeah17:32
gmannthat is why it is tricky to handle just from oslo side17:32
*** ociuhandu_ has quit IRC17:33
*** JamesBenson has quit IRC17:33
TheJuliaIt is really just a very long deprecation which is good and bad17:38
TheJuliait doesn't encourage operators/vendors/packagers to move quickly :\17:38
openstackgerritJulia Kreger proposed openstack/ironic master: Policy json to yaml migration  https://review.opendev.org/76326217:39
*** ayoung has quit IRC17:46
*** diurnalist has quit IRC17:46
*** ayoung has joined #openstack-ironic17:48
*** ayoung has quit IRC17:54
*** ayoung has joined #openstack-ironic17:58
*** mgoddard has quit IRC18:05
*** uzumaki has joined #openstack-ironic18:08
TheJuliawow, eventlet fun18:17
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Copy any configuration from the virtual media  https://review.opendev.org/76320718:21
dtantsurthis is also quite funny ^^18:21
dtantsurTheJulia: btw, I've updated https://storyboard.openstack.org/#!/story/2008366 based on conversations with dhellmann18:23
dtantsurfrom their side they're mostly interested in the 'direct' mode, but based on our conversations it feels like the proxy mode is helpful as well?18:23
dtantsurI actually need to go, but will check any comments tomorrow18:23
*** dtantsur is now known as dtantsur|afk18:23
dtantsur|afko/18:23
TheJuliadtantsur: ack, I'll take a look at it, working on upgrade check/ json->yaml18:23
*** fmuyassarov has joined #openstack-ironic18:29
*** k_mouza has quit IRC18:34
NobodyCamGood Morning Ironic folks18:36
*** jslater has joined #openstack-ironic18:38
TheJuliagood morning NobodyCam18:43
*** uzumaki has quit IRC18:43
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Add upgrade check, and json2yaml policy handling  https://review.opendev.org/76328618:48
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Add upgrade check, and json2yaml policy handling  https://review.opendev.org/76328618:51
TheJuliablarg18:51
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Prevent broken partition image UEFI deploys  https://review.opendev.org/74804919:17
*** mgoddard has joined #openstack-ironic19:27
trandlesHi Ironic. I'm deploying some new systems and it appears my rootfs isn't being resized. instance_info has root_gb=100 but after deployment the rootfs is only 1.1GB. What might be causing this? I'm at my wits end...to the point that I've had my non-technical spouse reading the screen just to confirm she's seeing the same things I'm seeing.19:28
trandlesI'm using the ipa-centos8-stable-ussuri deploy image downloaded from tarballs.opendev.org.19:30
trandlesFWIW Inspector is filling in the correct node properties and has "local_gb: 930" for the systems in question19:34
*** mgoddard has quit IRC19:39
trandleslooks like the partition created was the correct size (100GB) so it's a growfs or resizefs that's failing19:40
TheJuliatrandles: so... is this a whole disk image or a partition image?19:44
trandlesoof, nvm, I think I figured it out19:44
trandlesHi TheJulia it's full disk19:45
trandlesI need next week's holiday. I spend just enough time with Ironic to get comfy and then get yanked to another project for 2-3 weeks and feel like I'm starting over when I return. :(19:50
NobodyCamGood Morning TheJulia :) happy almost Friday ;p19:51
trandlesTheJulia: growroot DIB module did what I wanted.20:10
*** ayoung has quit IRC20:20
*** rcernin has joined #openstack-ironic20:33
*** k_mouza has joined #openstack-ironic20:35
*** rcernin has quit IRC20:37
*** k_mouza has quit IRC20:39
jslaterHey all, I've created a custom hardware manager via https://docs.openstack.org/ironic-python-agent/latest/contributor/hardware_managers.html. Now I need to build the IPA image that includes this hardware manager code in it. I see in ironic-python-agent-builder there is a --branch option in the command but I do not see how to set a custom repository to point to the fork that contains my hardware manager. Am I missing something in how to use the20:54
jslaterIPA builder to include the custom hardware manager?20:54
*** rcernin has joined #openstack-ironic21:02
*** ayoung has joined #openstack-ironic21:08
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Copy any configuration from the virtual media  https://review.opendev.org/76320721:24
*** ayoung has quit IRC22:08
TheJuliaarne_wiebalck: maybe you can point jslater in the right direction?22:11
openstackgerritJulia Kreger proposed openstack/ironic master: Policy json to yaml migration  https://review.opendev.org/76326222:14
TheJuliajslater: ideally you would have a custom diskimage-builder element that would be included in the ELEMENTS_PATH which could then be invoked as part of the builder22:15
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Add upgrade check, and json2yaml policy handling  https://review.opendev.org/76328622:20
jslaterThanks your response TheJulia. The caveat I am finding with using disk-imagebuilder to create the IPA ramdisk image, is that the ELEMENTS_PATH is required to the dib folder installed by the ironic-python-agent-builder package. Wondering if I also need to maintain a fork of ironic-python-agent-builder with the customizations in the dib folder22:21
*** ayoung has joined #openstack-ironic22:22
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Prevent broken partition image UEFI deploys  https://review.opendev.org/74804922:23
TheJuliajslater: you should be able to separate that and define it like your $PATH variable22:23
TheJuliaELEMENTS_PATH=/path/to/diskimage-builder/elements:/path/to/ipa-b/elements:/path/to/tripleo-image-elements/elements22:24
TheJuliaas an example22:24
jslaterExcellent wasn't aware it could take multiples, will try that.22:26
TheJuliaOkay!22:38
*** fmuyassarov has quit IRC22:56
jandersgood morning Ironic o/23:04
*** Goneri has quit IRC23:10
*** rloo has quit IRC23:18
trandlesHi janders23:19
janderstrandles have you seen StackHPC's Ironic machine in the Top100?23:23
trandlesI did. I asked John for some details and he pointed to his Summit talk. It only left me with more questions.23:25
trandlesOff for the night. o/23:28
*** thiagop has quit IRC23:32
jandershave a good one trandles23:32
*** thiagop has joined #openstack-ironic23:32
openstackgerritMerged openstack/ironic master: Fix disk label to account for UEFI  https://review.opendev.org/76254023:34
*** rcernin has quit IRC23:35
*** rcernin has joined #openstack-ironic23:35

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