Monday, 2021-02-15

« Sunday, 2021-02-14 Index Tuesday, 2021-02-16 »
*** tosky has quit IRC00:09
*** bdodd has joined #openstack-ironic01:37
*** bdodd has quit IRC01:51
openstackgerritBill Dodd proposed openstack/ironic master: Add Redfish RAID management to Ironic  https://review.opendev.org/c/openstack/ironic/+/73000002:27
*** jhesketh has joined #openstack-ironic02:34
*** rcernin has quit IRC02:42
*** rcernin has joined #openstack-ironic02:54
*** k_mouza has joined #openstack-ironic03:08
*** k_mouza has quit IRC03:12
*** uzumaki has quit IRC03:28
*** frigo has joined #openstack-ironic03:30
*** MentalSiege has joined #openstack-ironic04:19
*** MentalSiege has quit IRC04:24
*** rcernin has quit IRC04:51
*** rcernin has joined #openstack-ironic04:56
*** vdrok has quit IRC04:58
*** johnsom has quit IRC04:58
*** TheJulia has quit IRC04:58
*** buhman has quit IRC04:58
*** johnsom has joined #openstack-ironic04:58
*** vdrok has joined #openstack-ironic04:58
*** TheJulia has joined #openstack-ironic04:58
*** buhman has joined #openstack-ironic04:58
*** k_mouza has joined #openstack-ironic05:34
*** k_mouza has quit IRC05:38
*** k_mouza has joined #openstack-ironic05:45
*** k_mouza has quit IRC05:50
*** anuradha1904 has joined #openstack-ironic06:06
openstackgerritMerged openstack/ironic stable/victoria: Fixes issue of redfish firmware update  https://review.opendev.org/c/openstack/ironic/+/77391006:47
*** uzumaki has joined #openstack-ironic07:01
*** moshiur has joined #openstack-ironic07:04
openstackgerritMerged openstack/ironic master: Address some rbac review feedback in merged patches  https://review.opendev.org/c/openstack/ironic/+/77539907:04
*** uzumaki has quit IRC07:06
*** rcernin has quit IRC07:16
*** rcernin has joined #openstack-ironic07:19
arne_wiebalckGood morning, ironic!07:22
*** rcernin has quit IRC07:23
stevebakergood evening!07:23
arne_wiebalckHey stevebaker o/07:27
stevebakerhey07:27
openstackgerritankit proposed openstack/ironic master: Adds config parameter kernel_append_param for iLO  https://review.opendev.org/c/openstack/ironic/+/75518907:35
*** rcernin has joined #openstack-ironic07:36
*** k_mouza has joined #openstack-ironic07:50
*** k_mouza has quit IRC07:54
jandersgood morning arne_wiebalck stevebaker and Ironic o/08:17
jandersrecall: good evening stevebaker :)08:18
* stevebaker is about to shuffle off08:18
openstackgerritMerged openstack/ironic-python-agent master: Remove samples from the hardware test module  https://review.opendev.org/c/openstack/ironic-python-agent/+/77516308:26
openstackgerritVerification of a change to openstack/ironic failed: Implement "system" scoped RBAC for the node endpoint  https://review.opendev.org/c/openstack/ironic/+/76325708:26
*** k_mouza has joined #openstack-ironic08:29
arne_wiebalckhey janders o/08:31
*** rcernin has quit IRC08:32
*** k_mouza has quit IRC08:33
*** lucasagomes has joined #openstack-ironic09:00
*** dougsz has joined #openstack-ironic09:03
*** rpittau|afk is now known as rpittau09:03
rpittaugood morning ironic! o/09:03
*** ociuhandu has joined #openstack-ironic09:13
*** ianychoi_ has joined #openstack-ironic09:18
*** mushrushu has quit IRC09:20
*** ianychoi has quit IRC09:21
*** mushrushu has joined #openstack-ironic09:21
*** ociuhandu has quit IRC09:22
*** ociuhandu has joined #openstack-ironic09:22
*** tosin has joined #openstack-ironic09:22
*** tosky has joined #openstack-ironic09:24
*** rcernin has joined #openstack-ironic09:28
*** HardCase has quit IRC09:31
*** ociuhandu has quit IRC09:34
*** dtantsur|afk is now known as dtantsur09:36
dtantsurmorning ironic09:36
rpittaugood morning dtantsur :)09:44
*** sshnaidm|off is now known as sshnaidm09:54
*** derekh has joined #openstack-ironic09:56
*** k_mouza has joined #openstack-ironic10:00
*** rcernin has quit IRC10:02
*** ociuhandu has joined #openstack-ironic10:04
*** ociuhandu has quit IRC10:09
*** ociuhandu has joined #openstack-ironic10:18
*** derekh has quit IRC10:19
dtantsurlooking for a 2nd +2 on https://review.opendev.org/c/openstack/ironic/+/77541010:23
*** derekh has joined #openstack-ironic10:24
*** ociuhandu has quit IRC10:28
*** ociuhandu has joined #openstack-ironic10:29
iurygregorygood morning Ironic10:32
*** ociuhandu has quit IRC10:34
rpittauhey iurygregory :)10:35
iurygregoryrpittau, o/10:36
*** ociuhandu has joined #openstack-ironic10:38
*** ociuhandu has quit IRC10:55
*** ociuhandu has joined #openstack-ironic10:56
*** ociuhandu has quit IRC11:01
*** ociuhandu has joined #openstack-ironic12:00
*** mkowalski_ has quit IRC12:08
openstackgerritMerged openstack/ironic master: Move the IPv6 job to the experimental pipeline  https://review.opendev.org/c/openstack/ironic/+/77541012:13
dtantsurfolks, could anyone try the cherry-pick button on ^^12:19
dtantsurit does nothing for me for some reason12:19
iurygregorytrying12:20
iurygregorysame12:20
* dtantsur asks on #opendev12:21
iurygregorydtantsur, worked now12:21
iurygregoryI had to wait like 20secs12:21
iurygregoryLOL12:21
dtantsurmm, let me try waiting as well12:21
*** ociuhandu has quit IRC12:22
dtantsuryeah12:22
iurygregorydelay++12:22
iurygregory=)12:22
iurygregorytks gerrit12:22
*** ociuhandu has joined #openstack-ironic12:23
dtantsuryeah12:23
dtantsurthe new version is definitely much slower12:24
iurygregoryyup12:24
openstackgerritDmitry Tantsur proposed openstack/ironic stable/victoria: Move the IPv6 job to the experimental pipeline  https://review.opendev.org/c/openstack/ironic/+/77560812:24
*** mkowalski_ has joined #openstack-ironic12:24
*** ociuhandu has quit IRC12:27
iurygregoryrpittau, hey for the ngs problem with stable/train12:29
iurygregoryhttps://github.com/openstack/networking-generic-switch/commit/bcc00173355367387433479486fc8007ab3cd3a6 maybe this will help?12:29
* iurygregory checks if commit is present in stable/train12:30
iurygregoryyeah it's not present12:30
*** mkowalski_ has quit IRC12:33
*** mkowalski_ has joined #openstack-ironic12:34
arne_wiebalckTheJulia: dtantsur: for https://storyboard.openstack.org/#!/story/2008626, see https://usercontent.irccloud-cdn.com/file/hXuH56qh/randomising_conductor_restart.png12:38
*** ociuhandu has joined #openstack-ironic12:39
*** k_mouza has quit IRC12:39
arne_wiebalckthis is ~25 conductors started simultaneously vs with a random delay12:39
*** k_mouza has joined #openstack-ironic12:39
* arne_wiebalck goes for lunch12:39
dtantsurhmm, interesting12:40
dtantsurI don't think people who have 1 conductor will appreciate a start-up delay12:40
arne_wiebalckconfigurable12:41
arne_wiebalck0 by default12:41
dtantsurit feels that we're breaching into the area of orchestration tools12:41
arne_wiebalckto me it feels more like supporting larger-scale deployments12:43
iurygregoryI agree with arne_wiebalck, if this can happen in large-scale we can have some configuration that will help them to prevent the problem12:44
dtantsurwell, it can be done with puppet/ansible/whatever12:44
dtantsurthere is nothing ironic-specific here, other services may exhibit similar issues12:44
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] New deploy step for injecting arbitrary files  https://review.opendev.org/c/openstack/ironic-python-agent/+/77542813:03
arne_wiebalcknot sure about ansible, but puppet as a config management tool is not what I would use to stop/start a service, it merely makes sure it is there and configures it (so I would use it to configure such a delay :-P)13:08
dtantsurmaybe a systemd unit.. dunno, I'm not strongly against having such an option, just seems completely service-independent13:12
arne_wiebalckyeah, I see what you mean13:14
arne_wiebalcka related option would be if the periodics in oslo would allow for initial jitter ... that does not seem to be there, though13:15
arne_wiebalcklike, start this periodic task only after 0..N seconds, there is only immediately or N13:16
dtantsurI think futurists does allow jitter13:17
arne_wiebalckthe thing here is that periodics should be periodic, i.e. with a fixed frequency; this also shifts the issue to identify the "offending" task ... there are pros and cons, I guess13:20
rpittauiurygregory: re ngs: thanks for checking! try and add the backport if you got the time, otherwise I can add it myself13:31
iurygregoryrpittau, sure doing now!13:32
TheJuliagood morning13:34
rpittaugood morning TheJulia :)13:34
dtantsurTheJulia: morning13:35
TheJuliaarne_wiebalck: I was thinking we look at the number of conductors and make some decisions based upon that, fwiw13:35
arne_wiebalckTheJulia: good morning13:37
arne_wiebalckTheJulia: that sounds sophisticated :)13:37
TheJuliait would be13:37
TheJuliaand possibly over engineered13:37
arne_wiebalck:-D13:37
TheJuliabut we do need to auto-skew the periodics somehow13:38
TheJuliapower-sync is just the worst offender possible13:38
arne_wiebalckhttps://usercontent.irccloud-cdn.com/file/569Lty7C/randomising_conducor_starts.png13:38
TheJuliaYeah, I saw that13:38
arne_wiebalckno, wait13:38
TheJuliaoh13:38
arne_wiebalckthis is longer and different13:38
arne_wiebalckfirst part: all started at the same time13:38
arne_wiebalckuntil 12:1513:39
arne_wiebalckthen with 30s fixed delay13:39
arne_wiebalckuntil 13:1513:39
arne_wiebalckthen random(30-90) delay13:39
TheJuliajust modifying power sync?13:39
arne_wiebalckthis is ironic-conductor restarts, I did not try power sync only13:39
TheJuliaah, ok13:40
arne_wiebalckI think we are dominated by power sync13:40
* arne_wiebalck has no direct proof for this statement13:40
TheJuliaiurygregory: for today's meeting, shoudl we revisit hash tags?13:40
TheJuliaBased on the code, I know you are13:40
arne_wiebalck+113:40
TheJulia+1 to hash tags?13:41
arne_wiebalckas well :)13:41
iurygregoryTheJulia, we didn't have much feedback on the etherpad for options we should use, so I would say maybe =)13:41
TheJuliaiurygregory: we are trying to keep it simple\13:42
arne_wiebalckTheJulia: I am not sure on adding jitter to all periodics, or one by one whenever they appear, but I also see dtantsur point on a general delayed start13:42
iurygregoryyup =)13:42
TheJuliaso part of what makes your graph even as spikey as it is is all of the periodics triggering still in rapid succession13:44
TheJuliawe need to smooth the db load out over time13:44
TheJuliaand/or give tunables to enable operators to smooth it13:44
arne_wiebalckTheJulia: yes, there is also a granularity problem with the simple approach of a delayed service start13:45
TheJuliaYeah13:46
arne_wiebalckbut from that graph, this helps quite a bit already13:46
TheJuliayup13:46
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] New deploy step for injecting arbitrary files  https://review.opendev.org/c/openstack/ironic-python-agent/+/77542813:52
rpittauTheJulia: we have the review jam today ?13:53
openstackgerritIury Gregory Melo Ferreira proposed openstack/networking-generic-switch stable/train: Remove grenade jobs from old stable branches  https://review.opendev.org/c/openstack/networking-generic-switch/+/77333513:54
iurygregorylet's see how it goes13:54
rpittauiurygregory: thanks! fingers crossed :)13:54
iurygregoryrpittau, np! I hope it helps =)13:54
TheJuliaYes, review jam is going to get started in a moment14:01
arne_wiebalckTheJulia: the randomisation also reduced the API calls, probably less retries14:05
*** lmcgann has joined #openstack-ironic14:06
arne_wiebalckconductor randomisation started at around 12:00 https://usercontent.irccloud-cdn.com/file/KnmvPzSa/API_requests_on_nova_restart_and_randomisation.png14:06
arne_wiebalckanyway14:06
* arne_wiebalck stops posting graphs now 14:06
TheJulialol14:08
TheJuliaarne_wiebalck: WOW14:08
*** ociuhandu has quit IRC14:12
*** ociuhandu has joined #openstack-ironic14:14
*** tosin has quit IRC14:32
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: New deploy step for injecting arbitrary files  https://review.opendev.org/c/openstack/ironic-python-agent/+/77542814:36
*** tzumainn has joined #openstack-ironic14:45
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Move some raid specific functions to raid_utils  https://review.opendev.org/c/openstack/ironic-python-agent/+/77485414:49
*** tosky has quit IRC14:55
*** ociuhandu has quit IRC14:56
*** stendulker has joined #openstack-ironic14:57
*** ociuhandu has joined #openstack-ironic14:57
TheJulia#startmeeting ironic15:00
TheJuliao/\15:00
openstackMeeting started Mon Feb 15 15:00:25 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
TheJuliaerr15:00
TheJuliao/15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
iurygregoryo/15:00
openstackThe meeting name has been set to 'ironic'15:00
dtantsuro/15:00
ajyao/15:00
stendulkero/15:00
rpittauo/15:00
TheJuliaGood Morning everyone!15:00
rpioso\o15:00
*** MentalSiege has joined #openstack-ironic15:01
TheJuliaThe Meeting Marathon continues with Ironic's Weekly meeting after our third review jam in the last few weeks.15:01
arne_wiebalcko/15:01
TheJuliaOur agenda can be found on the wiki!15:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
TheJulia#topic Announcements / Reminder15:01
*** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)"15:01
TheJulia#Info This week is R-8 in the OpenStack Wallaby release schedule.15:02
TheJuliaSome reminders:15:02
*** anuradha1904 has quit IRC15:02
TheJulia#info R-6 is non-client library freeze, and R-5 is client library + requirements freeze.15:02
TheJuliaWhich means, next week we likely need to get Sushy + ironic lib released.15:03
dtantsuror the week after, right?15:03
TheJulia#info Targetting sushy+ironic-lib+metalsmith releases for next week, patch authors should ensure they have patches proposed for the priorities15:03
dtantsurthe week of Mar 1st15:03
dtantsur(metalsmith has been recently released and has no more outstanding patches)15:04
rpittauwe should prioritize ironic-lib and sushy reviews this week and next then15:04
TheJuliaPreferably end of next week, anticipating they may not actually freeze until the mid-week of R-615:04
TheJuliaWe need to keep in mind, when they say R-6 freeze, they don't mean the last day , they lean towards EOD Wednesday/EOD Thursday.15:05
TheJuliarpittau: ++15:05
TheJuliadtantsur: \o/15:05
dtantsurThursday used to be official15:05
TheJuliaYeah15:05
dtantsuralthough it has been slightly more relaxed recently15:05
TheJuliaYup15:05
TheJuliaAnyhow! One other thing of note!15:05
TheJulia#Info reminder, we're holding review jam's twice a week. Tomorrow we will be revisiting Configuration molds with ajya and next week we'll be back on secure-rbac.15:06
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Review_Jams15:06
TheJuliaDoes anyone have anything else to announce or remind us of this week?15:06
dtantsurThe sprint 2 releases are pending15:06
dtantsurwill hopefully be processed today-tomorrow15:07
TheJuliaAck, thanks!15:07
dtantsuroh15:07
dtantsurand there has not been an inspector sprint 2 release because of lack of changes15:07
dtantsurjust in case somebody will be looking for it :)15:07
iurygregoryI was about to ask about inspector =)15:07
TheJuliaThanks again!15:07
TheJuliaWe should have at least a couple things merged into inspector before the end of Wallaby15:08
TheJuliaIt does not appear that we had any action items from last week, so I propose we go directly into subteam status reports15:08
dtantsur++15:08
TheJulia#topic Review subteam status reports15:09
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:09
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:09
TheJuliaStarting around line 28515:09
TheJuliaHmm, no kaifeng15:10
TheJuliaEek, I just read the redfish raid item and I'm worried about our friends in Austin now15:11
TheJuliaiurygregory: Seems like we're going to need a meeting of a brain trust of ironic cores to dig at the oslo.privsep issues?15:11
iurygregoryTheJulia, probably =(15:12
rpittaushould be more available this week to ahve a look at the privsep stuff if needed15:12
iurygregoryrpittau, tks!15:12
*** MentalSiege has quit IRC15:12
TheJuliaDo we want to try and work something out for tomorrow?15:12
*** anuradha1904 has joined #openstack-ironic15:12
iurygregoryit's a bit weird the logs in different runs for the same change15:12
iurygregoryworks for me15:13
TheJuliaIf you guys want to agree upon a time, I think that will work15:13
iurygregorylet's follow-up after the meeting?15:13
TheJuliaworks for me15:14
rpittauyup15:14
iurygregoryso we can find a slot for tomorrow =)15:14
*** juanoterocas has joined #openstack-ironic15:14
* TheJulia make notes on the json to yaml effort and the secure rbac work15:16
TheJuliarpittau: thank you for updating on the Anaconda deploy interface15:18
rpittaunp15:18
TheJuliaOkay, I guess that is good, onward to priorities for the week?15:18
rpittaulet's15:18
TheJulia#topic Deciding on priorities for the coming week15:18
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:18
TheJulia#link https://etherpad.opendev.org/p/IronicWhiteBoard15:19
TheJuliaStarting around line 13015:19
ajyacan this patch be added - https://review.opendev.org/c/openstack/sushy/+/774532 for Redfish RAID?15:19
TheJuliaplease15:19
openstackgerritMoisés Guimarães de Medeiros proposed openstack/ironic-inspector stable/ussuri: Fix memcached host address.  https://review.opendev.org/c/openstack/ironic-inspector/+/77564015:20
TheJuliaAny chance someone would volunteer to review https://review.opendev.org/c/openstack/ironic/+/762873 ?15:20
TheJuliait is just a doc update for BFV15:20
openstackgerritDmitry Tantsur proposed openstack/ironic master: Switch multinode jobs to 512M RAM  https://review.opendev.org/c/openstack/ironic/+/77566615:21
dtantsur^^^15:21
TheJuliaLooks like I need to update the rbac work items list15:22
TheJuliaI think I'm good with the list of new items15:23
TheJuliaAny objections? Additions?15:23
rpittausohuld be fine15:23
TheJuliaI'll clean-up and reconcile the list after the meeting15:24
dtantsuras the author of all added patches, I agree :D15:24
iurygregorylol15:24
TheJuliaheh15:24
TheJuliaOnward to discussion then?15:24
rpittauhope to have a little more time for reviews this week (but I guess I just jinxed it)15:24
rpittaulet's !15:25
TheJuliarpittau: you did jinx yourself :)15:25
rpittauyeah..... :/15:25
TheJuliac'est la vie15:25
TheJulia#topic Discussion15:25
*** openstack changes topic to "Discussion (Meeting topic: ironic)"15:25
TheJuliaTwo topics, first up is iurygregory and hashtags15:25
iurygregoryso, we had two ideas for hashtags to track the priorities15:26
*** sdanni has joined #openstack-ironic15:26
iurygregoryironic-current-sprint and ironic-week-prio15:26
TheJuliaand I believe the prio one has merged?15:26
*** dougsz has quit IRC15:26
TheJuliamaybe not15:27
iurygregoryI have the feeling week-prio would make more sense15:27
rpittauthe idea on the prio was to at some point use the tags instead of the list in the etherpad15:27
iurygregoryand for works we have for sprints we can have ironic-sprint-X? maybe15:27
TheJulia++ I think, at least I am... missing the process to go from one to the other15:27
TheJuliaI'd prefer to keep it simple and avoid sprint targetting reviews15:28
rpittaumaybe having a period where we use both tag + etherpad and then gradually remove the list ?15:28
TheJuliaEvery time we've tried to confine the scope, we begin pushing back on generally useful minor fixes and the like, and that is no good for the rest of the community15:28
TheJuliarpittau: I think that is the only reasonable path15:28
*** bburns has quit IRC15:29
rpittauI guess at some point we just add a url with the filter15:29
TheJulia++15:29
rpittauiurygregory: thanks for taking care of that btw, seems very useful :)15:29
TheJuliaOh ,here we go, I finally see "Add hashtags"15:29
*** iurygregory_ has joined #openstack-ironic15:29
TheJuliarutro, we lost iury somewhere15:29
rpittau:D15:30
*** iurygregory has quit IRC15:30
iurygregory_enr bad internet15:30
TheJuliaiurygregory_: so ironic-week-prio? as the hash tag?15:30
*** iurygregory_ is now known as iurygregory15:30
iurygregoryso if we agree on the tags I can update our docs/wiki with the necessary info15:30
iurygregory=)15:30
iurygregoryTheJulia, I think it would be better15:30
iurygregorysince not all patches have an sprint item15:31
TheJuliaYeah, and I'm against trying to do sprint tied patch listing15:31
iurygregorywe can add hash tags for sprint items also so it will help to track =)15:31
*** bburns has joined #openstack-ironic15:31
iurygregoryack15:32
TheJuliaAt least, trying to rely upon it. My worry is we hyperfocus on that and start ignoring generally useful fixes/additions15:32
iurygregoryyeah15:32
openstackgerritMoisés Guimarães de Medeiros proposed openstack/ironic-inspector stable/ussuri: Fix memcached host address.  https://review.opendev.org/c/openstack/ironic-inspector/+/77564015:32
TheJuliaSince those take any form, like... as simple as the patch just uploaded15:32
TheJuliaOkay, is there anything else to discuss other than just do it?15:32
iurygregoryone more thing15:33
*** moshiur has quit IRC15:33
iurygregoryironic-cores are welcome to provide feedback on https://review.opendev.org/c/openstack/project-config/+/77524415:33
iurygregoryso we can have the backport candidate label15:33
TheJuliaI like it15:34
rpittauoh yeah cool15:34
TheJulia#link https://review.opendev.org/c/openstack/project-config/+/77524415:34
* iurygregory hopes my test to auto add the backport hash tag patches with the label will work =)15:34
TheJuliaiurygregory: crafty :)15:34
* iurygregory *his test*15:34
TheJuliaI guess I can take the action item of beginning to migrate the list of patches?15:35
TheJuliaI'm still very heads down on rbac, but I'll need some breaks this week15:35
iurygregoryI can add the tags to the patches15:35
iurygregory=)15:35
TheJuliaiurygregory: okay!15:35
* TheJulia senses the universe just wants me to wrap up the rbac stuff15:35
iurygregoryhahaha15:35
openstackgerritMoisés Guimarães de Medeiros proposed openstack/ironic-inspector stable/train: Fix memcached host address.  https://review.opendev.org/c/openstack/ironic-inspector/+/77564115:35
TheJuliasooo close15:36
iurygregorywe can move on =)15:36
TheJuliaOkay, next discussion topic15:36
TheJuliaIn a couple weeks, if memory serves, we begin to hit the nemesis of daylight savings time changes15:36
rpittaumarch 28 for europe15:37
TheJuliaI would propose that we move the Monday review jam to after the weekly meeting, or if we want to another day entirely15:37
dtantsurI'm fine with swapping them15:37
iurygregoryI'm fine also15:37
rpittauso mm that means it will go to 6pm ?15:38
TheJuliaWell, moving the weekly meeting earlier is not exactly Julia/US-Pacific friendly15:38
TheJuliaThen again, maybe I'm thinking of calendars in the wrong direction.15:38
rpittauI think we should leave it15:38
rpittauit will be 4pm CEST15:38
TheJuliaI just wanted to bring it up since it tends to cause chaos15:38
rpittaunow it's 3pm15:38
*** tosin has joined #openstack-ironic15:39
TheJuliaOkay, I think I'm mixing it up15:39
*** ociuhandu has quit IRC15:39
arne_wiebalckI'm with rpittau on this complicated Gedankenexperiment15:39
rpittauand I just learned a new german word, monday is getting better15:40
dtantsurheh15:40
TheJuliaSince I think the meeting becames 8 AM for me after daylight savings time15:40
TheJuliaheh15:40
*** ociuhandu has joined #openstack-ironic15:40
TheJuliaI'm for banning daylight savings time, but.. yeah15:40
TheJuliaOkay, onward!15:40
dtantsurfor us the meeting time is reasonable either way15:40
TheJulia#topic Baremetal SIG15:40
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)"15:40
arne_wiebalckI don't think there is anything to report.15:40
TheJuliadtantsur: I only got up at 5AM local this morning to begin preparing for hours of meetings.15:41
iurygregoryarne_wiebalck, one question =)15:41
TheJuliaarne_wiebalck: Well, last week Dmitry did have a presentation :)15:41
dtantsurTheJulia: exactly, so let us know what works better for you :)15:41
openstackgerritMerged openstack/ironic master: Add some tuning documentation  https://review.opendev.org/c/openstack/ironic/+/73978315:41
TheJulia#link https://www.youtube.com/playlist?list=PLKqaoAnDyfgoBFAjUvZGjKXQjogWZBLL_15:41
iurygregorywhen it will be the next meeting?15:41
arne_wiebalckMeeting #6 Tue March 9, 2021 at 2pm UTC15:42
TheJulia#link https://etherpad.opendev.org/p/bare-metal-sig15:42
iurygregoryack15:42
*** dougsz has joined #openstack-ironic15:42
TheJuliaAnyone want the April spot? I could talk about RBAC I guess15:42
TheJuliaThat is if it is all merged by then15:42
arne_wiebalckTheJulia: all yours then :)15:43
TheJuliaLooks like we have no RFE's to review today in the meeting, so next up is Open Discussion15:43
TheJulia#topic Open Discussion15:43
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:43
TheJuliaAnything for us to discuss in Open Discussion?15:44
TheJuliaor should I just resume going crazy before my next meeting in 15 minutes?15:44
*** ociuhandu has quit IRC15:44
* iurygregory also has a meeting in 15min15:45
* TheJulia hears crickets15:46
*** ociuhandu has joined #openstack-ironic15:46
rpittauTheJulia: I'm sorry, for the review jam on monday, I realized it is very early for you now, I guess we could pjust postpone it to after the meeting15:46
TheJuliaIt may be fine, lets see how it goes15:47
rpittauok15:47
TheJuliaIt is a little difficult ot keep the mental context of "Was I getting up earlier or later"15:47
TheJuliaUntil I begin to grumble about getting up when it is dark.15:47
*** Qianbiao has joined #openstack-ironic15:48
TheJuliaWell, google finally played the sound of crickets for me15:48
TheJuliaso I guess today's meeting is done. Thanks everyone15:48
rpiosoThank you, Julia!15:48
rpittauthank you!15:48
iurygregoryty!15:48
TheJuliaHave a wonderful week everyone! And remember to review all the things!15:49
TheJulia#endmeeting15:49
*** 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:49
openstackMeeting ended Mon Feb 15 15:49:17 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:49
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-02-15-15.00.html15:49
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-02-15-15.00.txt15:49
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-02-15-15.00.log.html15:49
*** Qianbiao has quit IRC15:50
*** ociuhandu has quit IRC15:51
*** ociuhandu has joined #openstack-ironic15:52
TheJuliawheeee, looks like I need to rebase the secure rbac stuffs15:52
*** mkowalski_ has quit IRC15:54
iurygregoryrpittau, "ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode POST_FAILURE"15:56
iurygregory-.-'15:56
rpittaummm yeah15:56
rpittauit could be the version of paramiko15:56
iurygregory<insert flip table>15:56
iurygregorylet me see with recheck...15:57
*** mkowalski_ has joined #openstack-ironic15:57
rpittauiurygregory: check the nwutron logs first15:57
iurygregoryPOST_FAILURE normally is infra problem no?15:58
iurygregoryI will check after the meeting I have in 1 min15:59
*** ociuhandu has quit IRC15:59
*** ociuhandu has joined #openstack-ironic16:00
rpittauiurygregory: ok, same error as before btw http://paste.openstack.org/show/802651/16:01
*** tosky has joined #openstack-ironic16:02
openstackgerritRiccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity  https://review.opendev.org/c/openstack/ironic/+/37657416:07
*** mkowalski_ has quit IRC16:07
*** mkowalski_ has joined #openstack-ironic16:10
openstackgerritRiccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity  https://review.opendev.org/c/openstack/ironic/+/37657416:11
TheJuliaparamiko?!?16:16
TheJuliawhat... how... why?!?16:17
dtantsurit's used by tempest for validating access to the nodes IIRC :)16:18
*** stendulker has quit IRC16:19
iurygregoryrpittau, ouch =(16:24
*** ociuhandu has quit IRC16:29
*** ociuhandu has joined #openstack-ironic16:29
* iurygregory brb lunch time16:32
*** ociuhandu has quit IRC16:34
*** ociuhandu has joined #openstack-ironic16:37
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Restore the conditional verbosity in bash scripts  https://review.opendev.org/c/openstack/bifrost/+/77567816:45
dtantsurrpittau: if you have a second ^^16:46
openstackgerritMerged openstack/ironic-specs master: Implement System Scoped Role Based Access Control  https://review.opendev.org/c/openstack/ironic-specs/+/76407016:51
arne_wiebalckbye everyone o/16:52
iurygregoryrpittau, TheJulia 4pm UTC would work?16:55
TheJuliayeah, that should work16:55
rpittauiurygregory: good for me16:56
iurygregoryack!16:56
iurygregorywant me to send an invitation to your calendar?16:57
TheJuliaplease17:01
iurygregorydone17:03
rpittauiurygregory: thanks!17:04
*** lucasagomes has quit IRC17:05
*** ociuhandu has quit IRC17:07
*** ociuhandu has joined #openstack-ironic17:08
TheJuliaiurygregory: actually, can we off shift that by a half hour in either direction?17:10
TheJuliaI've got a 4:00 to 4:30 call tomorrow that I need to be on17:10
* TheJulia was not looking at that calendar when she said 4pm would work17:10
*** ociuhandu has quit IRC17:12
*** ociuhandu has joined #openstack-ironic17:15
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: [WIP] Use json for lsblk output  https://review.opendev.org/c/openstack/ironic-python-agent/+/77539117:18
*** ociuhandu_ has joined #openstack-ironic17:22
*** ociuhandu_ has quit IRC17:23
*** ociuhandu has quit IRC17:24
openstackgerritTzu-Mainn Chen proposed openstack/ironic master: Allow support for multipath volumes  https://review.opendev.org/c/openstack/ironic/+/77568117:25
TheJuliatzumainn: fixing it since ipxe landed proper mpath data passing?17:25
tzumainnTheJulia, from our testing, if target_portals are present, all that's needed is the construction of a different iscsi url17:26
tzumainnit is very possible that I am not fully considering the ramifications of the change17:26
tzumainnbut I thought maybe the easiest way to discuss would be to submit the code and let others correct my misunderstandings :)17:27
TheJuliatzumainn: if you  could, add an ipxe template to match what it *should* be and run a test against that17:28
TheJuliajust so we can ensure we see/get what we expect17:29
TheJuliaI guess my disconnect is I'm, not seeing how it results in what gets rendered for ipxe17:29
tzumainnTheJulia, ah, does the updated test not show that?17:30
TheJuliaI don't think so17:30
TheJuliawe have a couple tests where we actually compare hand crafted file to the generated output17:30
TheJuliathat hand crafted file is of course static and is what we expect17:30
tzumainnah, okay! I see them now - in ironic/tests/unit/drivers?17:31
*** dougsz has quit IRC17:39
TheJuliatzumainn: yup17:42
rpittaugood night! o/17:47
TheJuliagoodnight17:47
*** rpittau is now known as rpittau|afk17:47
openstackgerritTzu-Mainn Chen proposed openstack/ironic master: Allow support for multipath volumes  https://review.opendev.org/c/openstack/ironic/+/77568117:50
*** rcernin has joined #openstack-ironic17:59
*** derekh has quit IRC18:00
*** johnsom has quit IRC18:01
*** rpittau|afk has quit IRC18:01
*** johnsom has joined #openstack-ironic18:02
*** rpittau|afk has joined #openstack-ironic18:04
*** rcernin has quit IRC18:04
TheJuliadtantsur: replied to some of your comments on the rbac spec, generally in agreement. Your changing types concern does concern me, only because we can have null strings that are represented as null today, which makes me wonder is how is that handled today?18:16
TheJuliain rust18:16
dtantsurTheJulia: it's one thing if a value is optional (e.g. instance_uuid), another - if it's de facto mandatory now in the right microversion (conductor)18:16
dtantsurin rust-openstack I expect everything that is not UUID to be optional :) but I can imagine somebody acting differently18:17
dtantsuror even if python doing things like: `if "localhost" in node.reservation` :)18:17
TheJuliaMaybe we just wanted to explicitly indicate "redacted"18:18
dtantsuryep18:18
TheJuliaor ******18:18
TheJulia"**value redacted**18:18
TheJulia"18:18
dtantsureven better18:18
TheJuliawhich might be a solid hint of "you need more access if you need this", "please call your friendly system admin18:18
TheJulia"18:18
TheJuliafor dictionaries, I think it is fine if it is just empty18:20
TheJulia{}18:20
* TheJulia goes and hunts that patch down to make notes18:22
TheJuliahttps://review.opendev.org/c/openstack/ironic/+/773924 <-- notes added18:25
dtantsuryep, an empty dict is totally fine18:32
dtantsuractually, driver_info can be empty in some normal situations18:32
*** dtantsur is now known as dtantsur|afk18:32
dtantsur|afko/18:32
*** openstackgerrit has quit IRC18:38
TheJuliaYeah, that was what I was thinkign18:58
*** rcernin has joined #openstack-ironic18:59
*** tosin has quit IRC19:02
*** k_mouza has quit IRC19:03
*** juanoterocas has quit IRC19:08
*** juanoterocas has joined #openstack-ironic19:09
iurygregoryTheJulia, sure I will shift =)19:10
*** openstackgerrit has joined #openstack-ironic19:10
openstackgerritMerged openstack/ironic master: Switch multinode jobs to 512M RAM  https://review.opendev.org/c/openstack/ironic/+/77566619:10
*** rcernin has quit IRC19:12
iurygregorylol promote-openstack-tox-docs failed in https://review.opendev.org/c/openstack/ironic/+/775666 O.o19:34
TheJuliaiurygregory: <319:41
TheJuliawut...19:41
iurygregoryyeah weird, also inspector CI is showing problem in ussuri/train in the docs jobs19:41
*** rcernin has joined #openstack-ironic19:48
*** anuradha1904 has quit IRC19:52
iurygregoryoh seems like a bug in sphinx version..19:59
iurygregoryhttps://github.com/sphinx-doc/sphinx/issues/888019:59
iurygregoryhttps://zuul.opendev.org/t/openstack/build/d191928f1980411eb1cf7e83783a04f5/log/tox/docs-3.log#12720:01
iurygregoryyeah...20:01
stevebakergood morning20:03
*** rcernin has quit IRC20:05
iurygregorygood morning stevebaker20:05
*** alexmcleod has quit IRC20:05
*** openstackgerrit has quit IRC20:06
*** rcernin has joined #openstack-ironic20:21
*** openstackgerrit has joined #openstack-ironic20:36
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-inspector stable/victoria: Avoid a full install in tox environments that do not need it  https://review.opendev.org/c/openstack/ironic-inspector/+/77570420:36
iurygregoryI hate the fact that I need to change tox.ini basepython in ussuri or older to run tests =(20:53
*** rh-jelabarre has quit IRC20:55
*** rcernin has quit IRC20:57
*** HardCase has joined #openstack-ironic20:58
*** rh-jelabarre has joined #openstack-ironic21:01
*** rh-jelabarre has quit IRC21:01
*** rh-jelabarre has joined #openstack-ironic21:02
*** rcernin has joined #openstack-ironic21:23
* TheJulia sighs21:28
stevebakerTheJulia: \o21:32
*** rcernin has quit IRC21:38
*** rcernin has joined #openstack-ironic21:38
TheJuliastevebaker: o/21:43
stevebakerTheJulia: I'm looking at the indicators breakage now, I'm assuming you haven't21:45
TheJuliaI have not21:48
TheJuliaI've been slammed21:48
TheJuliaI made it mostly through ports/portgroups21:48
TheJuliaI'm not in a headspace to pick that back up today21:48
stevebakerTheJulia: all good21:55
*** sdanni has quit IRC22:11
stevebakerTheJulia: I see ironicclient doesn't support the indicators endpoint, and the original spec intended to. I might just go do that https://specs.openstack.org/openstack/ironic-specs/specs/15.0/expose-hardware-indicators.html#openstack-baremetal-cli22:23
iurygregorystevebaker, I do remember we had an RFE to add this support I (don't remember atm who said it was going to add the support...)22:25
* stevebaker looks for the RFE22:26
stevebakeriurygregory: I can't see anything in storyboard, it probably just needs one more task in https://storyboard.openstack.org/#!/story/200534222:29
iurygregoryyeah22:33
iurygregoryI do remember last year we talked about the support on the client...22:33
iurygregoryduring some of the upstream meetings22:33
stevebakermaybe it didn't quite make it to being tracked. And there is no outstanding ironicclient reviews for it22:34
iurygregoryyeah22:36
* stevebaker stumbles into a field of yaks22:42
*** clarkb has quit IRC22:42
zer0c00lTheJulia: I see the kernel and ramdisk getting fetched and cached. Does the OS image (image_source) gets fetched and cached on the conductor?22:55
zer0c00li do not see any code doing that. Does IPA directly fetches the os image (image_source) ?22:56
TheJuliaI'm somehow not surprised the client neer got handled22:58
TheJulia:(22:58
TheJuliazer0c00l: there is a hook to pull it down for http(s) download through the condcutor23:01
TheJuliaor the agent can go directly to image_source23:01
*** juanoterocas has quit IRC23:04
zer0c00lTheJulia: i see. will "look around" in the code. thanks.23:04
TheJuliaI think the conductor will download the image in the base prepare, but I'm not 100% sure anymore23:05
TheJuliaits all kind of fuzzy at the moment23:05
zer0c00lthere are two prepare_* methods, i see prepare_ramdisk and prepare_instance23:08
zer0c00li have been mostly looking at prepare_instance()23:08
TheJuliahmm23:08
TheJuliapulling it up23:08
zer0c00li so far do not see anything fetching the "image_source" itself.23:08
*** HardCase has quit IRC23:08
zer0c00li think it see it.23:12
TheJuliacache_ramdisk_kernel23:13
TheJuliaI think it gets a disk image as well23:14
zer0c00lTheJulia: the 'instance_image_info' is passed to cache_ramdisk_kernel23:16
TheJuliayup23:16
zer0c00lbut as far as i could see instance_image_info never includes the image_source23:16
TheJuliayeah, I think there is a conditional for the ipxe driver if the http download feature is enabled23:17
TheJuliawhich flows things through the conductor23:17
zer0c00lTheJulia: perhaps. but i am not very sure if cache_ramdisk_and_kernel is responsible for downloading the image_source23:19
zer0c00lbecause get_instance_image_info() only deals with kernel and ramdisk23:19
zer0c00lhttps://github.com/openstack/ironic/blob/master/ironic/common/pxe_utils.py#L63623:19
TheJuliathere is a easy way to figure this out23:19
TheJulia[agent]image_download_source23:21
TheJuliahttps://github.com/openstack/ironic/blob/3db362e5aacaaffbf10a91e083f46bd3807349c1/ironic/drivers/modules/deploy_utils.py#L106623:21
zer0c00lhttps://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L55523:23
zer0c00lyes23:23
zer0c00lTheJulia: thanks23:23
zer0c00lkind of all over the place. kernel/ramdisk gets download in pxe - boot phase23:23
zer0c00limage_source downloaded in agent.py23:24
zer0c00lAppreciate the help :)23:24
TheJuliayeah, there is some indirection due to refactoring23:24
*** lmcgann has quit IRC23:24
TheJuliaand it is all being driven by agent.py23:24
zer0c00lyeah, i figured thats where everything starts!23:24
« Sunday, 2021-02-14 Index Tuesday, 2021-02-16 »

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