Monday, 2021-02-22

« Sunday, 2021-02-21 Index Tuesday, 2021-02-23 »
stevebakerjanders: very quiet, just kicking around home mostly. how about yourself?00:14
*** zzzeek has quit IRC00:15
*** zzzeek has joined #openstack-ironic00:15
jandersstevebaker that's good. My Sunday was similar - and Saturday went for a road trip that got cut short by tough weather (but it was fun anyway! :) We literally drove through some waterfalls (which were running over the road due to rain). Not something I can say I've done before :)00:16
*** zzzeek has quit IRC00:23
*** zzzeek has joined #openstack-ironic00:26
stevebakerjanders: yikes!00:43
*** tosky has quit IRC00:53
iurygregorymorning janders stevebaker o/00:55
jandershey iurygregory00:55
stevebakerhi iurygregory01:05
*** ricolin has joined #openstack-ironic02:52
*** rcernin has quit IRC02:55
*** maelk has quit IRC02:57
*** fdegir has quit IRC02:58
*** maelk has joined #openstack-ironic02:58
*** rcernin has joined #openstack-ironic03:07
*** rcernin has quit IRC03:20
*** rcernin has joined #openstack-ironic03:25
*** ociuhandu has joined #openstack-ironic03:29
*** ociuhandu has quit IRC03:33
*** ricolin has quit IRC03:38
*** mkrai has joined #openstack-ironic03:39
*** lbragstad_ has joined #openstack-ironic03:41
*** lbragstad has quit IRC03:43
*** zzzeek has quit IRC03:47
*** zzzeek has joined #openstack-ironic03:49
*** zzzeek has quit IRC03:53
*** zzzeek has joined #openstack-ironic03:55
*** mkrai has quit IRC03:57
*** mkrai_ has joined #openstack-ironic03:57
*** ociuhandu has joined #openstack-ironic03:57
*** zzzeek has quit IRC03:59
*** zzzeek has joined #openstack-ironic04:01
*** ricolin has joined #openstack-ironic04:01
*** ociuhandu has quit IRC04:02
*** bdodd has joined #openstack-ironic04:06
*** zzzeek has quit IRC04:14
*** zzzeek has joined #openstack-ironic04:18
*** rcernin has quit IRC04:37
*** zzzeek has quit IRC04:57
*** zzzeek has joined #openstack-ironic04:58
*** rcernin has joined #openstack-ironic05:03
*** zzzeek has quit IRC05:03
*** zzzeek has joined #openstack-ironic05:05
*** ociuhandu has joined #openstack-ironic05:20
*** ociuhandu has quit IRC05:24
openstackgerritMerged openstack/ironic master: Trivial: fix incorrect ordering in iLO tests  https://review.opendev.org/c/openstack/ironic/+/77666105:37
*** rcernin has quit IRC05:42
*** rcernin has joined #openstack-ironic05:42
*** dtantsur|afk is now known as dtantsur06:14
dtantsurgood (really early) morning ironic06:14
*** benj_ has quit IRC06:48
*** bnemec has quit IRC06:48
jandersgood morning dtantsur o/06:52
jandershow was your weekend?06:52
*** benj_ has joined #openstack-ironic06:52
*** bnemec has joined #openstack-ironic06:54
dtantsurjanders: hey! not bad at all, enjoyed the nice weather :) yours?07:00
*** ociuhandu has joined #openstack-ironic07:03
*** dtantsur is now known as dtantsur|brb07:05
*** ociuhandu has quit IRC07:07
*** rcernin has quit IRC07:10
*** rcernin has joined #openstack-ironic07:18
*** mkrai_ has quit IRC07:19
arne_wiebalckGood morning, ironic!07:31
*** tosin has joined #openstack-ironic07:39
*** ociuhandu has joined #openstack-ironic07:45
*** rcernin has quit IRC07:51
*** rcernin has joined #openstack-ironic07:52
*** anuradha1904 has joined #openstack-ironic07:56
*** rcernin has quit IRC08:00
*** rcernin has joined #openstack-ironic08:03
jandersgood morning arne_wiebalck o/08:04
jandersdtantsur|brb that's great to hear! :) we had some really heavy rain, so had to cut the road trip we were planning short - the creek we had to cross turned out impassable, too deep and too fast flowing08:05
jandersSo a two-day trip turned into a day trip. Quite fun nonetheless. First time I literally drove through a waterfall :)08:05
*** rcernin has quit IRC08:09
*** rcernin has joined #openstack-ironic08:10
*** mkrai_ has joined #openstack-ironic08:12
openstackgerritRotan proposed openstack/ironic master: Address the console port confilct.  https://review.opendev.org/c/openstack/ironic/+/77689908:15
*** rcernin has quit IRC08:16
*** rcernin has joined #openstack-ironic08:26
*** rcernin has quit IRC08:32
*** rpittau|afk is now known as rpittau08:34
rpittaugood morning ironic! o/08:34
*** ociuhandu has quit IRC08:35
*** rcernin has joined #openstack-ironic08:36
*** rcernin has quit IRC08:41
*** tosky has joined #openstack-ironic08:43
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Use tinycore 12 to build tinyipa  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/77658708:47
*** ociuhandu has joined #openstack-ironic08:57
*** lucasagomes has joined #openstack-ironic08:57
*** k_mouza has joined #openstack-ironic08:58
openstackgerritMerged openstack/ironic-lib master: Add a helper for getting partition path  https://review.opendev.org/c/openstack/ironic-lib/+/77450209:01
*** k_mouza has quit IRC09:05
*** k_mouza has joined #openstack-ironic09:08
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Make sure we can create tinyipa isos  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/77664209:08
jandersdtantsur|brb rpittau if you have time, I'd appreciate if you can review https://review.opendev.org/c/openstack/ironic/+/771905 (this is Ironic-API side of the NVMe cleaning change). It's a small change. Happy to tweak it as needed. Having both merged would allow me to tick this off and start working on the demo.09:31
janders(it's mostly around adding the config option and deprecating the ATA specific one)09:31
openstackgerritRotan proposed openstack/ironic-inspector master: [WIP]Support AEP, sst at introspection data  https://review.opendev.org/c/openstack/ironic-inspector/+/76813809:33
openstackgerritRotan proposed openstack/ironic-python-agent master: [WIP]Support AEP, sst at introspection data  https://review.opendev.org/c/openstack/ironic-python-agent/+/76769209:42
*** derekh has joined #openstack-ironic09:44
openstackgerritRotan proposed openstack/ironic-inspector master: [WIP]Support AEP, sst at introspection data  https://review.opendev.org/c/openstack/ironic-inspector/+/76813809:50
*** mkrai_ has quit IRC09:54
*** mkrai_ has joined #openstack-ironic09:56
*** zzzeek has quit IRC09:59
*** zzzeek has joined #openstack-ironic10:01
openstackgerritRotan proposed openstack/ironic-python-agent master: [WIP]Support AEP, sst at introspection data  https://review.opendev.org/c/openstack/ironic-python-agent/+/76769210:01
*** ociuhandu has quit IRC10:12
*** ociuhandu has joined #openstack-ironic10:17
*** k_mouza has quit IRC10:21
*** sshnaidm__ is now known as sshnaidm10:22
openstackgerritDerek Higgins proposed openstack/sushy-tools master: Fix Unit test to test for boot index  https://review.opendev.org/c/openstack/sushy-tools/+/77665510:33
*** jhesketh has quit IRC10:37
*** jhesketh has joined #openstack-ironic10:37
*** k_mouza has joined #openstack-ironic10:38
iurygregorygood morning Ironic o/10:51
rpittauhey iurygregory :)10:52
*** tosky_ has joined #openstack-ironic10:53
*** tosky has quit IRC10:54
*** tosky_ is now known as tosky10:54
jandershey iurygregory o/10:58
*** k_mouza has quit IRC11:01
*** mkrai_ has quit IRC11:01
openstackgerritRiccardo Pittau proposed openstack/networking-generic-switch stable/train: Remove grenade jobs from old stable branches  https://review.opendev.org/c/openstack/networking-generic-switch/+/77333511:06
*** k_mouza has joined #openstack-ironic11:06
*** fmuyassarov has joined #openstack-ironic11:13
*** k_mouza has quit IRC11:17
openstackgerritRiccardo Pittau proposed openstack/networking-generic-switch stable/train: Remove grenade jobs from old stable branches  https://review.opendev.org/c/openstack/networking-generic-switch/+/77333511:18
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Make sure we can create tinyipa isos  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/77664211:24
*** k_mouza has joined #openstack-ironic11:32
*** fmuyassarov has quit IRC11:42
openstackgerritRotan proposed openstack/ironic master: Address the console port confilct.  https://review.opendev.org/c/openstack/ironic/+/77689911:44
*** ociuhandu has quit IRC11:44
dtantsur|brbyay, electricity is back \o/11:50
*** dtantsur|brb is now known as dtantsur11:50
iurygregoryyay11:54
jandersrpittau thank you for your review of the NVMe Ironic patch. Good point. What is the typical approach to deprecation/removal? If something is deprecated in N, is it typically removed in N+1?11:56
iurygregoryI think it's N+211:59
iurygregoryor maybe it depends XD12:00
jandersso - deprecation in Wallaby, removal in "Y"?12:00
iurygregoryhuumm it's not a feature so maybe N+1 is fine12:01
iurygregoryfor example https://github.com/openstack/ironic/blob/stable/queens/releasenotes/notes/classic-drivers-deprecation-de464065187d4c14.yaml was deprecated in queens12:02
dtantsurN+1 is fine for all but the most cornerstone features12:03
dtantsur(we opted for N+2 for iSCSI because of how ubiquitous it is)12:04
dtantsurspeaking of reviews, could i get some on https://review.opendev.org/c/openstack/ironic-python-agent/+/775428 please? It's less scary than it seems, the code is very straightforward.12:04
jandersthank you iurygregory and dtantsur.12:04
iurygregorydtantsur, looking12:05
*** ociuhandu has joined #openstack-ironic12:05
*** ociuhandu has quit IRC12:05
janderswe don't know what the Y release will be called yet right?12:07
rpittaujanders: "Y cycle" is fine :)12:10
openstackgerritJacob Anders proposed openstack/ironic master: Add support for using NVMe specific cleaning  https://review.opendev.org/c/openstack/ironic/+/77190512:14
*** ociuhandu has joined #openstack-ironic12:15
*** ociuhandu has quit IRC12:15
*** ociuhandu has joined #openstack-ironic12:16
iurygregorywell we are deprecating in Wallaby so wouldn't be removed in Xena?12:17
* dtantsur votes for Xena+ as well12:18
* rpittau check alphabet again12:18
rpittauIt's lunch time, I'm hungry, I don't know what comes after W12:18
dtantsur:D12:19
*** ociuhandu has quit IRC12:20
arne_wiebalckhttps://usercontent.irccloud-cdn.com/file/E0NOL1R0/nodes_only.png12:21
dtantsurWOW12:21
arne_wiebalckdtantsur: TheJulia: JayF: I patched the power sync job. This reduced the number of DB accesses by two thirds.12:21
dtantsur#shipit12:21
arne_wiebalckdtantsur: this is a nodes_only parameter now12:22
dtantsurI would expect lazy loading to have an even bigger impact12:22
arne_wiebalckdtantsur: I agree12:23
iurygregoryWorld Of Warcraft =D12:23
iurygregoryXD12:23
arne_wiebalckdtantsur: I will submit this patch now none the less12:23
arne_wiebalckdtantsur: it is a very simple fix with quite an impact12:23
arne_wiebalckdtantsur: I looked at the remaining accesses12:24
iurygregoryI have the feeling this will be in a blog post =)12:24
arne_wiebalckdtantsur: per cycle, there is a select on nodes (as expected)12:24
arne_wiebalckdtantsur: and roughly one on allocations12:24
arne_wiebalckdtantsur: these are the two main contributors12:24
*** ociuhandu has joined #openstack-ironic12:25
dtantsurright, because we need an allocation UUID12:28
*** anuradha1904 has quit IRC12:32
dtantsurjanders: one comment re the release note (inline)12:32
*** rh-jelabarre has joined #openstack-ironic12:38
*** ociuhandu has quit IRC12:40
jandersthank you dtantsur, fixing this now12:44
arne_wiebalckiurygregory: the graph looks nice, but the patch is really simple: "selecting what you do not need does help with scaling"12:52
arne_wiebalckoops: *does not12:52
*** ociuhandu has joined #openstack-ironic12:53
openstackgerritJacob Anders proposed openstack/ironic master: Add support for using NVMe specific cleaning  https://review.opendev.org/c/openstack/ironic/+/77190512:54
janders^ release note sorted out12:54
jandershmm I might have spotted inconsistency in the new config option naming12:55
janderschecking12:55
iurygregoryarne_wiebalck, yeah =)12:56
jandersactually the config options don't change, just the agent_info variables12:57
dtantsurjanders: sorry, a comment for the updated version12:59
dtantsurwe need to be careful about wording with two types of options: agent options (which the operator does not see) and ironic.conf options12:59
jandersdtantsur I haven't renamed ``[deploy]/continue_if_disk_secure_erase_fails`` (only changed the help string in ironic/conf/deploy.py). It's the agent_info variables that had _ata_ in their name12:59
jandersin which case, maybe we don't need most (or any) of the deprecation notice?13:00
dtantsurright, hence my comment13:00
dtantsurwe need to make it clear that this option will break with Xena Ironic and Victoria IPA13:00
jandersyeah I agree13:00
openstackgerritJacob Anders proposed openstack/ironic master: Add support for using NVMe specific cleaning  https://review.opendev.org/c/openstack/ironic/+/77190513:03
tosinhi dtantsur, I'm not sure  understand your comment on line 255 here https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774089/4/plugins/modules/baremetal_node_action.py#25513:04
dtantsurtosin: hi! the question is: what will happen if somebody uses this module without setting "traits" on a node that has traits already set?13:05
*** zzzeek has quit IRC13:05
tosinoh that makes sense. I'll work on it. Thank you13:06
*** zzzeek has joined #openstack-ironic13:07
openstackgerritArne Wiebalck proposed openstack/ironic master: For power sync, get only the nodes from the DB  https://review.opendev.org/c/openstack/ironic/+/77693013:10
*** uzumaki has joined #openstack-ironic13:11
*** iurygregory_ has joined #openstack-ironic13:14
*** iurygregory has quit IRC13:15
*** ociuhandu has quit IRC13:16
*** lbragstad_ is now known as lbragstad13:16
*** tosky has quit IRC13:18
*** ociuhandu has joined #openstack-ironic13:19
TheJuliagood morning13:21
*** bburns has quit IRC13:22
*** iurygregory_ is now known as iurygregory13:24
*** tosky has joined #openstack-ironic13:24
*** bburns has joined #openstack-ironic13:24
iurygregorygood morning TheJulia13:24
rpittaugood morning TheJulia :)13:25
*** ociuhandu has quit IRC13:27
arne_wiebalckFor the small partitions/EBR patch in https://github.com/openstack/ironic-lib/commit/9a60d3853fc5b36cd3715e9419568cc9c51f5a8e : would it make sense to backport this to Ussuri as well? Otherwise, I would need to start building ironic-lib locally ... :-S13:28
iurygregoryarne_wiebalck, I would say it does make sense since it's a fix =)13:29
*** ociuhandu has joined #openstack-ironic13:31
TheJuliaI concur but I think it is fine just to backport and upload to CI13:31
TheJuliaerr13:31
TheJuliastart building locally? err13:32
TheJuliaI guess older ironci version13:32
* TheJulia has words with the cat that is trying to be super affectionate 13:32
*** ociuhandu has quit IRC13:32
iurygregorythe corgi is probably mad...13:33
iurygregoryor angry13:33
*** ociuhandu has joined #openstack-ironic13:33
* arne_wiebalck rebuilds most of Ironic locally13:33
arne_wiebalckour deployment is still on Train, a Ussuri backport will allow me to say "it comes with Ussuri!" :-D13:34
iurygregoryTheJulia, https://www.teeturtle.com/products/paw-painting-corgi?variant=19577791465884 =)13:34
iurygregoryarne_wiebalck, LOL13:34
*** lbragstad has quit IRC13:35
TheJuliaiurygregory: adorable13:36
*** lbragstad has joined #openstack-ironic13:37
*** dougsz has joined #openstack-ironic13:41
dtantsurmorning TheJulia13:49
openstackgerritArne Wiebalck proposed openstack/ironic-lib stable/ussuri: Handle EBRs and tiny partitions when removing metadata  https://review.opendev.org/c/openstack/ironic-lib/+/77693613:51
*** uzumaki has quit IRC13:52
openstackgerritJulia Kreger proposed openstack/ironic master: Implement "system" scoped RBAC for the node endpoint  https://review.opendev.org/c/openstack/ironic/+/76325713:59
openstackgerritJulia Kreger proposed openstack/ironic master: Implement "system" scoped RBAC for ports  https://review.opendev.org/c/openstack/ironic/+/76326713:59
openstackgerritJulia Kreger proposed openstack/ironic master: Implement system scoped RBAC for port groups  https://review.opendev.org/c/openstack/ironic/+/76326813:59
openstackgerritJulia Kreger proposed openstack/ironic master: Implement system scoped RBAC for chassis  https://review.opendev.org/c/openstack/ironic/+/76326913:59
openstackgerritJulia Kreger proposed openstack/ironic master: Implement system scoped RBAC for baremetal drivers  https://review.opendev.org/c/openstack/ironic/+/76327013:59
openstackgerritJulia Kreger proposed openstack/ironic master: Implement system scoped RBAC for node and driver passthru  https://review.opendev.org/c/openstack/ironic/+/76327113:59
openstackgerritJulia Kreger proposed openstack/ironic master: Implement system scoped RBAC for utility APIs  https://review.opendev.org/c/openstack/ironic/+/76327213:59
openstackgerritJulia Kreger proposed openstack/ironic master: Implement system scoped RBAC for volume APIs  https://review.opendev.org/c/openstack/ironic/+/76327313:59
openstackgerritJulia Kreger proposed openstack/ironic master: Implement system scoped RBAC for conductor APIs  https://review.opendev.org/c/openstack/ironic/+/76327413:59
openstackgerritJulia Kreger proposed openstack/ironic master: Implement system scoped RBAC for the allocation APIs  https://review.opendev.org/c/openstack/ironic/+/76327513:59
openstackgerritJulia Kreger proposed openstack/ironic master: Implement system scoped RBAC for the event APIs  https://review.opendev.org/c/openstack/ironic/+/76327613:59
openstackgerritJulia Kreger proposed openstack/ironic master: Implement system scoped RBAC for the deploy templates APIs  https://review.opendev.org/c/openstack/ironic/+/76327713:59
TheJuliareview jam anyone?14:01
* dtantsur needs 5 minutes more14:02
TheJuliano worries14:02
iurygregorygoing14:04
* iurygregory looks for the headphone14:04
openstackgerritJulia Kreger proposed openstack/ironic master: RBAC System Scope: observer -> reader  https://review.opendev.org/c/openstack/ironic/+/77245014:04
openstackgerritJulia Kreger proposed openstack/ironic master: Initial Project scoped tests  https://review.opendev.org/c/openstack/ironic/+/77245114:04
openstackgerritJulia Kreger proposed openstack/ironic master: Project Scoping Node endpoint  https://review.opendev.org/c/openstack/ironic/+/77392414:04
*** rloo has joined #openstack-ironic14:06
openstackgerritMerged openstack/metalsmith master: Uncap PrettyTable  https://review.opendev.org/c/openstack/metalsmith/+/77677914:07
*** zzzeek has quit IRC14:28
*** zzzeek has joined #openstack-ironic14:29
*** tosky has quit IRC14:29
*** tosky has joined #openstack-ironic14:33
*** anuradha1904 has joined #openstack-ironic14:35
*** tzumainn has joined #openstack-ironic14:43
*** Qianbiao has joined #openstack-ironic14:46
*** kaifeng has joined #openstack-ironic14:51
*** sdanni has joined #openstack-ironic14:51
iurygregorydtantsur, I only noticed now that we merged the support for deploy_templates on openstacksdk without adding tests to https://github.com/openstack/openstacksdk/blob/master/openstack/tests/unit/baremetal/v1/test_proxy.py D: my bad14:54
dtantsuriurygregory: oops :) honestly, I can about functional tests much more, these ones are mostly overmocked14:55
iurygregorydtantsur, yeah having functional tests gives more credibility XD14:56
*** lmcgann has joined #openstack-ironic14:56
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Make sure we can create tinyipa isos  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/77664214:57
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Make sure we can create tinyipa isos  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/77664214:59
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Feb 22 15:00:30 2021 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
TheJuliao/15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
rpittauo/15:00
iurygregoryo/15:00
ajyao/15:00
rpiosoo/15:00
erbarro/15:00
kaifengo/15:00
TheJuliaGood morning everyone!15:00
bdoddo/15:00
rlooo/15:00
arne_wiebalcko/15:01
*** stendulker has joined #openstack-ironic15:01
stendulkero/15:01
TheJuliaOur agenda can be found on the wiki.15:01
TheJulia#link15:01
TheJuliaerr15:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
bfournieo/15:01
* TheJulia needs more coffee15:01
dtantsuro/15:02
TheJuliaLooks like we have a fairly standard agenda, so hopefully this will go nice and quick :)15:02
TheJulia#topic Announcements / Reminders15:02
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:02
TheJulia#info this week is R-7 in the wallaby release cycle.15:02
TheJulia#info Next week is R-6 and non-client library freeze. R-5 will be the client library and requirements freeze.15:03
TheJuliaSo in other words, either the end of this week or very early next week we need to do our releases for metalsmith, ironic-lib, sushy15:03
TheJuliaDoes anyone else have anything to announce or remind us of?15:04
TheJuliaDoes anyone have anything to announce?15:04
TheJuliaerr yeah15:04
TheJuliaClearly, I need caffine.15:05
TheJuliaI guess one thing to note, I'll be taking some time off at some point in the next few weeks, I don't have any specifics, but it is looking like in ~2 weeks.15:06
rpittauI see there are not a lot of things to merge in sushy/ironic-lib, we should be ok15:06
TheJulia\o/15:06
TheJuliaMoving on then!15:06
dtantsurI'm out next week15:06
TheJuliadtantsur: enjoy!15:06
dtantsurthx :)15:06
TheJulia#topi Review action items from previous meeting15:07
TheJuliablah15:07
TheJulia#topic Review action items from previous meeting15:07
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"15:07
TheJuliaWe didn't have a formal action item, but Iury volunteered15:07
TheJuliaiurygregory: How did getting things tagged go for helping to replace the review priority list on the etherpad?15:08
iurygregoryTheJulia, for me it helped =) (adding the hashtag)15:08
TheJuliaiurygregory: where the be some sort of class or session to walk us through its use? "Practical process 101?"15:09
TheJulia:)15:09
iurygregoryTheJulia, totally =)15:09
iurygregoryshould I do now or maybe write an email?15:10
TheJuliaiurygregory: an email may be besst15:10
TheJuliabest, if you want to do something else, by all means. It could also just be well timed call, but getting all of our core reviewers around at the same time is difficult15:11
iurygregoryyeah, I can do an email and schedule a call to explain15:11
TheJuliaawesome15:11
iurygregoryeven two to cover the timezones =)15:11
TheJuliaWe may need some links to help us on the whiteboard a little later in the meeting :)15:11
TheJuliaOnward!15:12
TheJulia#topic Review subteam status reports15:12
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:12
TheJulia#link https://etherpad.opendev.org/p/IronicWhiteBoard15:12
*** Qianbiao has quit IRC15:12
TheJuliaStarting at line 28215:12
*** Qianbiao has joined #openstack-ironic15:13
TheJuliaLooks like NVMe cleaning is on its way15:13
TheJuliakaifeng: any progress on node error history?15:14
kaifengTheJulia no progress yet15:14
*** tosky has quit IRC15:15
TheJuliaI feel like this is going to be an item to move to the next cycle, which maybe we can also roll into "error codes" or something to address sensitive details like conductor names being logged15:15
*** tosky has joined #openstack-ironic15:15
TheJuliaLooks like Redfish raid needs some eyes15:15
ajyayes, need to review the sushy patch15:16
ajyahttps://review.opendev.org/c/openstack/sushy/+/77453215:16
rpittauwe need to get to that this week15:16
TheJuliaoh yes, is that on the list of priorities for this coming week? if the answer is no, then it needs to be on it15:16
rpittauit has the prio tag15:16
ajyait was added last week as prio15:16
TheJuliaack15:17
TheJuliaiurygregory: fighting some more privsep issues?15:17
iurygregorya "little" one it seems15:18
iurygregorylooking at the code privsep didn't gave any errors in the process15:18
TheJulialooks like raw image conversion's link doesn't like us15:18
*** tosky_ has joined #openstack-ironic15:18
TheJuliawhich kind of makes sense15:18
iurygregoryyeah15:18
TheJulianew method needed?15:18
*** tosky is now known as Guest4362915:19
*** tosky_ is now known as tosky15:19
TheJuliato provide elevated access to make a link15:19
iurygregoryI'm wondering if I'm missing a required capabilities to make it work with link15:19
TheJuliathat could be15:19
TheJuliaselinux also has a thing about linking with files copied based on permission inheratinence15:20
openstackgerritMerged openstack/metalsmith master: Update TOX_CONSTRAINTS_FILE  https://review.opendev.org/c/openstack/metalsmith/+/76691915:20
iurygregoryor maybe we need to add something to the os.link call15:20
TheJuliabut that... shouldn't be occuring on a CI node.15:20
iurygregorysince it's not something we are processutils to execute15:21
*** Guest43629 has quit IRC15:21
TheJuliayeah15:21
TheJuliais selinux logging anything?15:21
TheJuliaajya: Thank you for the update15:21
iurygregorychecking15:21
TheJuliazer0c00l: your around to provide an update for Anaconda?15:22
TheJuliarpioso: w/r/t redfish interop profiles, are you expecting any forward prog15:22
TheJuliare15:22
TheJuliaerr, thanks cat15:22
TheJuliaforward progress in the next 3-4 weeks?15:22
rpiosoTheJulia: We're continuing to make forward progress, so yes :)15:23
TheJuliaokay, cool15:23
arne_wiebalckrpioso: what is missing ?15:23
rpiosoarne_wiebalck: The log errors are being analyzed and results compared to iDRAC.15:24
TheJuliadtantsur: I just spotted https://review.opendev.org/c/openstack/sushy/+/773397, would it make sense to try and get that into sushy this cycle?15:24
rpittauTheJulia: just added one short comment for the anaconda interface, patch was updated so just waiting for reviews15:24
arne_wiebalckrpioso: thanks15:24
TheJuliarpittau: awesome15:24
TheJuliarpittau: thanks for updating the etherpad15:24
TheJuliaOnward?15:24
dtantsurTheJulia: if possible15:25
dtantsurit's nor wired anywhere, so I don't care much15:25
dtantsur(I wrote it while figuring out how these things work at all)15:25
TheJuliaahh15:26
dtantsurI'd be more interested to finish https://review.opendev.org/c/openstack/ironic-lib/+/774354 if I have time15:26
dtantsurthen we can start looking into JSON RPC for inspector15:26
TheJuliaOkay15:27
TheJuliaOnward to priorities15:27
TheJulia#topic Deciding on priorities for the coming week15:27
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:27
TheJulia#link https://etherpad.opendev.org/p/IronicWhiteBoard15:27
TheJuliadoes anyone have new items to add to this list?15:28
TheJuliaalso, iurygregory can we get links to the tagged searches?15:28
dtantsurI added one bug fix15:29
iurygregoryTheJulia, sure15:29
TheJuliaA lot on there got merged last week, actually kind of impressive15:30
iurygregoryhttps://review.opendev.org/q/hashtag:ironic-week-prio+status:open15:30
rpittauwe hsould probably remove the tag from merged patches15:30
TheJuliapossibly, although status:open means we shouldn't really need to15:31
iurygregoryyeah =)15:31
TheJuliaor maybe a bot that does it after a week or two?15:31
rpittauyeah, just wondering what will happen with gerrit filter when we'll have hundreds of them :)15:31
iurygregorywe would need to give the bot core rights...15:32
iurygregory=X15:32
iurygregoryor maybe add an exception in the ACL?15:32
dtantsurit can be a script in python15:32
dtantsurthat somebody just runs whenever they remember15:32
rpittaua simple script that can be ran15:32
TheJulia++15:32
TheJulialets keep it simple15:32
iurygregorydtantsur, I'm working on one to add the backport tag15:32
iurygregoryso I can test to see if remove will work fine =)15:33
TheJuliaLooks like I need to add the rbac stuff, but aside from that we appear to be in good shape with listed priorities15:33
iurygregoryand I can share the code on github15:33
TheJuliaAre we good to proceed forward?15:33
dtantsuryep15:33
rpittaulet's15:33
iurygregory++15:34
dtantsuroh, the secure erase patch has been approved15:34
* dtantsur has fixed15:34
TheJulia\o/15:34
* TheJulia moves the cat for the 15th time today15:34
TheJuliaThe fur children are driving me crazy... err.. more crazy15:35
rpittauwe need a bot to move the cat15:35
TheJulia+100015:35
* TheJulia thinks it may be just a laserpointer15:35
iurygregoryhehehe15:35
TheJulia#topic Discussion15:35
*** openstack changes topic to "Discussion (Meeting topic: ironic)"15:35
dtantsura vacuum cleaner robot with a laserpointer?15:35
TheJuliaiurygregory: Is there anything to really revisit w/r/t hashtags beyond getting the links into place for visibility?15:35
TheJuliadtantsur: this sounds like a marketable product15:35
TheJuliadtantsur: as long as there is a cat bed box or circular bed they can sit in as the robot vacuums as well15:36
iurygregoryTheJulia, no =)15:36
TheJuliaiurygregory: Okay then!15:36
TheJulia#topic Baremetal SIG15:36
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)"15:36
TheJuliaarne_wiebalck: anything with the sig this week?15:37
arne_wiebalckI don't think so.15:37
TheJuliaarne_wiebalck: I guess it would be helpful if the sig could maybe see if we could collect/identify operator focused needs/items for the next development cycle15:38
TheJuliaJust thinking brainstorming sort of stuff for PTG input.15:38
arne_wiebalckTheJulia: sure, we can either schedule a session or have a mail sent15:38
TheJuliaI think the next scheduled session should be fine, timeline wise15:39
arne_wiebalckOk, we should announce this then.15:39
TheJuliaSince there is nothign else, it seems like we can proceed to Open Discussion15:39
TheJuliaarne_wiebalck: ++15:39
arne_wiebalckI'll also update the agenda.15:39
TheJuliaarne_wiebalck: thanks15:39
TheJulia#topic Open Discussion15:40
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:40
TheJuliaSo besides robots to keep the cat and the corgi busy!15:40
TheJuliaDoes anyone have anything else to discuss/bring up?15:40
sdanniHi everyone! lmcgann and I are working on updating the spec https://review.opendev.org/c/openstack/ironic-specs/+/576718/20/specs/approved/security-interface.rst. It looks like it's in good shape. Do you have any suggestions about what else we can update it with?15:40
TheJuliaLooks like review bandwidth is what is needed15:41
TheJuliawould a review jam session help?15:41
TheJuliaWe could likely cover it partially tomorrow15:41
sdannithat would be great!15:42
TheJuliasdanni: if you or lmcgann can be there to discuss it with folks on the call, that will be helpful15:43
TheJuliahttps://wiki.openstack.org/wiki/Meetings/Ironic#Review_Jams15:44
TheJuliaif you want to add it to the agenda, that would be good15:44
TheJuliaIs there anything else to discuss today?15:44
sdanniTheJulia: thanks! We'll attend the meeting15:45
TheJuliaExcellent!15:45
TheJuliaWell, if there is nothing else everyone, Thanks and have a wonderful week!15:45
TheJuliao/15:46
TheJuliaThanks everyone!15:47
arne_wiebalckThanks TheJulia !15:47
TheJulia#endmeeting15:47
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list."15:47
openstackMeeting ended Mon Feb 22 15:47:09 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:47
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-02-22-15.00.html15:47
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-02-22-15.00.txt15:47
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-02-22-15.00.log.html15:47
*** stendulker has quit IRC16:03
*** uzumaki has joined #openstack-ironic16:13
openstackgerritYogesh proposed openstack/ironic master: Add idrac HW type IPMI interface support  https://review.opendev.org/c/openstack/ironic/+/77186216:21
*** Qianbiao has quit IRC16:27
openstackgerritYogesh proposed openstack/ironic master: Add idrac HW type IPMI interface support  https://review.opendev.org/c/openstack/ironic/+/77186216:29
*** dtantsur is now known as dtantsur|afk16:38
dtantsur|afko/16:38
*** lucasagomes has quit IRC16:55
*** ociuhandu has quit IRC17:03
*** ociuhandu has joined #openstack-ironic17:04
arne_wiebalckJayF: TheJulia: In case you missed this earlier. The effect of the DB power sync patch in https://review.opendev.org/c/openstack/ironic/+/776930 https://usercontent.irccloud-cdn.com/file/PaTg4UmU/nodes_only.png17:06
JayFI find myself wondering if we should expose something like this via the API (or imply it if given a `fields` argument that excludes those)17:07
JayFso that the same enhancement can be applied to the resource tracker calls done from the nova-compute driver17:08
*** ociuhandu has quit IRC17:08
*** ociuhandu has joined #openstack-ironic17:08
arne_wiebalckdoes the resource tracker hit the DB?17:08
TheJuliait does17:09
TheJuliaso we could imply it based upon fields17:09
TheJuliaat least based on the code17:09
TheJuliaI was looking at it last week17:10
TheJuliafields *should* only be returning the columns as well, but there is some weirdness with nodes most likely because of how they are constructed17:10
arne_wiebalckthe resource tracker has a longer cycle, this is maybe why I do not see this in my 6min trace17:11
TheJulialikely17:11
arne_wiebalckthis is still 2 selects for each node in the 5min power sync cycle17:11
arne_wiebalcknodes & allocations17:12
arne_wiebalckwell, it seems getting only what we need has some potential :-D17:13
*** ociuhandu has quit IRC17:23
*** dougsz has quit IRC17:40
*** ociuhandu has joined #openstack-ironic17:48
openstackgerritVerification of a change to openstack/ironic failed: Add support for using NVMe specific cleaning  https://review.opendev.org/c/openstack/ironic/+/77190517:48
rpittaugood night! o/17:49
*** tosin has quit IRC17:49
*** rpittau is now known as rpittau|afk17:49
*** k_mouza has quit IRC17:52
*** ociuhandu has quit IRC17:52
*** k_mouza has joined #openstack-ironic17:53
*** derekh has quit IRC18:01
*** rh-jelabarre has quit IRC18:12
*** kaifeng has quit IRC18:20
*** rh-jelabarre has joined #openstack-ironic18:28
arne_wiebalckbye everyone o/18:44
openstackgerritMerged openstack/ironic-python-agent master: New deploy step for injecting arbitrary files  https://review.opendev.org/c/openstack/ironic-python-agent/+/77542818:48
openstackgerritVerification of a change to openstack/ironic-inspector failed: Implement secure RBAC  https://review.opendev.org/c/openstack/ironic-inspector/+/77277118:48
openstackgerritVerification of a change to openstack/ironic failed: Implement "system" scoped RBAC for the node endpoint  https://review.opendev.org/c/openstack/ironic/+/76325719:12
openstackgerritVerification of a change to openstack/ironic failed: Implement "system" scoped RBAC for ports  https://review.opendev.org/c/openstack/ironic/+/76326719:12
openstackgerritVerification of a change to openstack/ironic failed: Implement system scoped RBAC for port groups  https://review.opendev.org/c/openstack/ironic/+/76326819:12
openstackgerritVerification of a change to openstack/ironic failed: Implement system scoped RBAC for chassis  https://review.opendev.org/c/openstack/ironic/+/76326919:12
openstackgerritVerification of a change to openstack/ironic failed: Implement system scoped RBAC for baremetal drivers  https://review.opendev.org/c/openstack/ironic/+/76327019:13
openstackgerritVerification of a change to openstack/ironic failed: Implement system scoped RBAC for node and driver passthru  https://review.opendev.org/c/openstack/ironic/+/76327119:13
openstackgerritVerification of a change to openstack/ironic failed: Implement system scoped RBAC for utility APIs  https://review.opendev.org/c/openstack/ironic/+/76327219:13
openstackgerritVerification of a change to openstack/ironic failed: Implement system scoped RBAC for volume APIs  https://review.opendev.org/c/openstack/ironic/+/76327319:13
openstackgerritVerification of a change to openstack/ironic failed: Implement system scoped RBAC for conductor APIs  https://review.opendev.org/c/openstack/ironic/+/76327419:14
*** k_mouza has quit IRC19:27
stevebakermorning!19:34
iurygregorymorning stevebaker o/19:35
*** gyee has joined #openstack-ironic19:53
*** bdodd has quit IRC20:14
*** ociuhandu has joined #openstack-ironic20:17
*** bdodd has joined #openstack-ironic20:19
*** bdodd has quit IRC20:32
*** bdodd has joined #openstack-ironic20:34
openstackgerritVerification of a change to openstack/ironic failed: Add support for using NVMe specific cleaning  https://review.opendev.org/c/openstack/ironic/+/77190520:51
TheJuliaugh20:55
TheJuliaThat seems a bit more than random failures :( at eleast fate wise20:56
jandersgood morning stevebaker good afternoon TheJulia good evening iurygregory o/21:11
janderscross timezone greetings!21:11
*** ociuhandu has quit IRC21:11
TheJuliaGood morning janders21:12
iurygregorymorning janders o/21:13
TheJuliastevebaker: looks like the ironic-inspector one needs a rebase21:18
* stevebaker looks21:21
openstackgerritSteve Baker proposed openstack/ironic-inspector master: Implement secure RBAC  https://review.opendev.org/c/openstack/ironic-inspector/+/77277121:25
TheJuliarutro, looks like there may have been a grenade breaking change21:26
*** zzzeek has quit IRC21:26
*** zzzeek has joined #openstack-ironic21:28
iurygregorygrenade *BOOM*21:28
TheJuliaso21:32
TheJuliathe problem appears to be that ping_check_public in grenade is mandatory and only has 30 seconds to pass21:32
openstackgerritVerification of a change to openstack/ironic failed: Implement "system" scoped RBAC for the node endpoint  https://review.opendev.org/c/openstack/ironic/+/76325721:36
jandersiurygregory LOL!21:39
jandersthat's kind of grenades do...21:39
janders*what grenades do21:39
TheJuliaBest I can figure21:45
TheJuliaNeutron *never* processed the FIP creation21:46
TheJuliajanders: your nvme patch failed on mirror connectivity issues21:48
jandersTheJulia yeah I saw that too21:48
jandersI think it's karma for making all the jokes about nobuntu and unbuntu21:48
TheJuliaI've fired off a few rechecks21:49
janders(and having some sarcastic discussions with MarkS at Summits)21:49
iurygregoryjanders, if you try to look at grenade multinode you will be afraid of grenade jobs XD21:52
jandersone grenade is kinda scary - it can't get any better with multiple grenades...21:53
*** lmcgann has quit IRC21:54
TheJuliaMy liver is calling Red to this discusison of multinode grenade21:56
*** rcernin has joined #openstack-ironic22:00
*** rcernin has quit IRC22:01
*** rcernin has joined #openstack-ironic22:01
jandersas soon as this merges I will start working on a baremetal lab setup involving some NVMe nodes for testing & demos... do I need to create a custom IPA image with master codebase, or will bifrost automatically pull in the latest IPA/master codebase as the IPA image?22:04
*** fmuyassarov has joined #openstack-ironic22:10
openstackgerritVerification of a change to openstack/ironic failed: Implement system scoped RBAC for port groups  https://review.opendev.org/c/openstack/ironic/+/76326822:11
openstackgerritVerification of a change to openstack/ironic failed: Implement "system" scoped RBAC for ports  https://review.opendev.org/c/openstack/ironic/+/76326722:11
* TheJulia dreads the evening vehicle project22:19
janders TheJulia what's the project? :)22:20
TheJuliaswap diesel filters22:20
jandersright!22:20
jandersfactory, or pre-filter?22:20
TheJuliaI just went to drain out the fuel/water separator to check and see if the diesel was in general good (looks great to my untrained eye) but still got a little on my hands and I loath the smell\22:21
TheJuliaboth prefilter and actual final engine filter22:21
TheJuliaboth super easy to access though, so that is a huge plus22:21
jandersYeah I can relate. I like the idea of diesel fuel, but I hate having anything to do with diesel itself. It stinks and tends to soak into things, making the smell hang around for weeks22:22
jandersdo the filters have a 'water trap' light on the dash - or a glass bottom where you can see the captured contaminants?22:23
TheJuliaBoth :)22:24
janderscool! if that's all clear and it looks good it should be no worries at all22:24
jandersdo you have an in-line priming pump in the fuel system?22:24
TheJulia*looks* like there is an in-line pump22:25
TheJuliabut the actual running pump is purely mechanical from what I've read22:25
jandersyeah that makes sense - if there is the inline manual one it can help getting everything going after the work22:27
fmuyassarovHello folks. How can check the microversion that introduced a new field? In particular, I'm interested in node.automated_clean field, which was added by https://opendev.org/openstack/ironic/commit/f11c949a111bff0b34ce14dd219ec7cf23e0d082. I can see this text "# v1.47: Add automated_clean to the node object.", so can I say that it was 1.47 microversion that added this field?22:29
TheJuliathe destructions ay to pre-fill the filter22:29
jandersis it a common rail motor?22:29
TheJuliaafaik yes22:29
jandersokay - same with mine22:30
TheJuliafmuyassarov: umm.. we put the comments in to that when we make the effective change to introduce a thing, why22:30
jandersthey probbaly want to avoid pumping air through the fuel system22:30
TheJuliaCummins 6.7L ISB... or ISL22:30
TheJuliaI don't remember which it actually is22:30
jandersmay be worthwhile following the destructions22:30
TheJuliafmuyassarov: is the field just not visible in your client?22:31
jandersbut if the motor didn't want to start despite I'd hand pump till it won't go anymore22:31
jandersand try again22:31
TheJuliaoh, the priming motor is electic22:32
TheJuliaalthough, half forum posts say its not there22:32
TheJuliabut I can visually see a small electric motor, and the other half of folk have shared how to tell it to prime the system but not start22:33
fmuyassarovTheJulia, it is related to my PR in gophercloud (related to Metal3 project) where we are missing automated_clean field. So, I was asked to add a note to mention the microversion that added this field. But I'm not familiar with microversions TBH. I only know the PR above, that added the field. You asked if the field isn't visible in my client, my answer is I don't have Ironic setup right now, but I can run it to check22:33
fmuyassarovthat22:33
jandersah okay22:34
TheJuliahttps://docs.openstack.org/ironic/latest/contributor/webapi-version-history.html#id1522:34
jandersthen maybe just switching the ignition on but not starting and leaving it like that for some time will prime it22:35
janders(some time = 15-60 seconds)22:35
TheJuliajanders: 3x that in rapid succesion22:35
jandersyep22:35
openstackgerritJulia Kreger proposed openstack/ironic master: Review feedback follow-up on Node System Scoped RBAC  https://review.opendev.org/c/openstack/ironic/+/77701622:36
fmuyassarovTheJulia, thank you for the link!22:37
*** fmuyassarov has quit IRC22:45
jandersTheJulia iurygregory I think my question got buried in IRC history hence asking again before you go offline :) - when I install bifrost, does it use the latest IPA/master codebase or do I need to rebuild the ramdisk to have the latest code? (context: NVMe cleaning testing/demo)23:02
TheJuliaI think it reuses23:03
TheJuliaif you remove the cached file it should pull it down23:03
jandersACK thank you TheJulia23:08
TheJuliaugh, I wish we settled 403 vs 404 earlier23:08
iurygregoryjanders, what TheJulia said =) (you can try to re install brifrost from master just to ensure it will pull the latest bits )23:14
iurygregoryI had to check why the dog was barking a lot (she was just asking for attention hehehe)23:15
TheJuliaoh, for a seocnd there I was liek "do I have a hot mic someplace?23:16
stevebakerjanders: building your own IPA isn't that difficult, and lets you add bit to help with dev23:17
stevebakerbitS23:17
TheJulia++23:23
stevebakerhaving said that, I seem to have some centos-stream issues23:24
openstackgerritMerged openstack/ironic master: Add support for using NVMe specific cleaning  https://review.opendev.org/c/openstack/ironic/+/77190523:37
*** anuradha1904 has quit IRC23:42
janders^ \o/23:42
jandersthank you iurygregory stevebaker23:43
« Sunday, 2021-02-21 Index Tuesday, 2021-02-23 »

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