Monday, 2019-05-20

« Sunday, 2019-05-19 Index Tuesday, 2019-05-21 »
*** dsneddon has joined #openstack-ironic00:18
*** dsneddon has quit IRC00:23
*** oyrogerg has joined #openstack-ironic01:12
*** dsneddon has joined #openstack-ironic02:19
*** dsneddon has quit IRC02:33
*** threestrands has joined #openstack-ironic02:34
*** bdodd has quit IRC02:35
*** bdodd has joined #openstack-ironic02:37
*** Lucas_Gray has joined #openstack-ironic02:53
*** Goneri has joined #openstack-ironic03:08
*** whoami-rajat has joined #openstack-ironic03:11
*** dsneddon has joined #openstack-ironic03:29
*** dsneddon has quit IRC03:34
*** dsneddon has joined #openstack-ironic04:30
*** dsneddon has quit IRC04:35
*** Lucas_Gray has quit IRC04:52
*** e0ne has joined #openstack-ironic05:31
*** dsneddon has joined #openstack-ironic05:31
*** dsneddon has quit IRC05:36
*** e0ne has quit IRC06:02
*** rachit7 has joined #openstack-ironic06:26
*** Goneri has quit IRC06:30
*** dsneddon has joined #openstack-ironic06:32
*** dsneddon has quit IRC06:37
arne_wiebalckgood morning, ironic!06:43
*** rpioso has quit IRC06:47
*** TheJulia has quit IRC06:47
*** vdrok has quit IRC06:47
*** TheJulia has joined #openstack-ironic06:47
*** rpioso has joined #openstack-ironic06:47
*** vdrok has joined #openstack-ironic06:47
*** seongsoocho has quit IRC06:47
*** seongsoocho has joined #openstack-ironic06:47
*** johnsom has quit IRC06:48
*** mnasiadka has quit IRC06:48
*** johnsom has joined #openstack-ironic06:48
*** mnasiadka has joined #openstack-ironic06:48
*** S4ren has joined #openstack-ironic06:58
*** S4ren has quit IRC07:07
*** zigo has joined #openstack-ironic07:08
*** rcernin has quit IRC07:08
*** yolanda_ has quit IRC07:13
*** tesseract has joined #openstack-ironic07:13
*** dsneddon has joined #openstack-ironic07:15
*** rpittau|afk is now known as rpittau07:18
rpittaugood morning ironic! o/07:19
*** yolanda_ has joined #openstack-ironic07:20
*** threestrands has quit IRC07:23
*** pcaruana has joined #openstack-ironic07:23
*** tssurya has joined #openstack-ironic07:33
*** verma-varsha has joined #openstack-ironic07:35
*** dtantsur|afk is now known as dtantsur07:35
dtantsurmorning ironic07:35
verma-varshaGood morning ironic! :)07:35
rpittauhey dtantsur :)07:36
rpittauhi verma-varsha :)07:36
*** kandi_ has quit IRC07:37
*** tiendc has joined #openstack-ironic07:38
verma-varsharpittau: :)07:38
dtantsurrpittau: could you review https://review.opendev.org/#/c/653491/ if you have a minute please?07:40
patchbotpatch 653491 - ironic - Add API to allow update allocation name and extra ... - 6 patch sets07:40
rpittaudtantsur: yeah, going through all the pending changes now07:42
dtantsurthx07:43
*** iurygregory has joined #openstack-ironic07:45
rpittauI was a bit puzzled by rocky still failing but I realized that the fix has not merged yet :/07:46
rpittauhttps://review.opendev.org/65877107:46
patchbotpatch 658771 - ironic-python-agent (stable/rocky) - Fixes for building images with CoreOS - 4 patch sets07:46
*** lucasagomes has joined #openstack-ironic07:47
*** e0ne has joined #openstack-ironic07:48
dtantsurrpittau: I took a liberty of approving these fixes so that we have working stable branches again07:48
rpittaudtantsur: thanks!07:49
dtantsurverma-varsha: hi! just for me to understand: what time is it right now for you?07:51
verma-varshadtantsur: Hey! It's 1:22 PM for me right now.07:52
dtantsurverma-varsha: not bad, I thought we had a larger difference07:53
iurygregorymorning all =D08:01
rpittauhey iurygregory :)08:02
*** Lucas_Gray has joined #openstack-ironic08:12
*** dsneddon has quit IRC08:14
dtantsurTheJulia: FYI I'm going to request a feature release of ironic-lib, as well as fixes for ironicclient stable08:14
*** Lucas_Gray has quit IRC08:17
*** Lucas_Gray has joined #openstack-ironic08:20
openstackgerritDmitry Tantsur proposed openstack/ironic-lib stable/stein: Include partiton name and flags from parted output  https://review.opendev.org/66004308:24
openstackgerritDmitry Tantsur proposed openstack/ironic-lib stable/rocky: Include partiton name and flags from parted output  https://review.opendev.org/66004408:24
openstackgerritDmitry Tantsur proposed openstack/ironic-lib stable/queens: Include partiton name and flags from parted output  https://review.opendev.org/66004508:25
*** verma-varsha has quit IRC08:28
*** S4ren has joined #openstack-ironic08:28
*** oyrogerg has quit IRC08:37
*** derekh has joined #openstack-ironic08:37
*** verma-varsha has joined #openstack-ironic08:41
*** Lucas_Gray has quit IRC09:00
*** Lucas_Gray has joined #openstack-ironic09:00
*** Lucas_Gray has quit IRC09:08
*** skazi has joined #openstack-ironic09:20
openstackgerritMerged openstack/ironic master: Add API to allow update allocation name and extra field  https://review.opendev.org/65349109:26
openstackgerritMerged openstack/ironic-python-agent stable/rocky: Fixes for building images with CoreOS  https://review.opendev.org/65877109:26
openstackgerritMerged openstack/ironic-python-agent stable/queens: Fixes for building images with CoreOS  https://review.opendev.org/65877209:26
openstackgerritMerged openstack/ironic-python-agent stable/pike: Fixes for building images with CoreOS  https://review.opendev.org/65877309:26
openstackgerritDmitry Tantsur proposed openstack/ironic master: Do not try to return mock as JSON in unit tests  https://review.opendev.org/66006109:27
*** verma-varsha has quit IRC09:28
dtantsuran easy change to help RDO un-pin WSME ^^09:30
iurygregorynice09:31
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent stable/ocata: Fixes for building images with CoreOS  https://review.opendev.org/66006509:33
*** verma-varsha has joined #openstack-ironic09:48
openstackgerritMerged openstack/metalsmith master: Switch to allocation API instead of reservations via instance_id  https://review.opendev.org/65953509:52
openstackgerritMerged openstack/metalsmith master: Allow disabling clean up on deployment failure  https://review.opendev.org/65977809:52
*** rcernin has joined #openstack-ironic10:03
*** tiendc has quit IRC10:05
*** tiendc has joined #openstack-ironic10:16
*** verma-varsha has quit IRC10:42
*** rcernin has quit IRC10:57
*** yolanda_ has quit IRC11:04
*** yolanda_ has joined #openstack-ironic11:08
*** tiendc has quit IRC11:08
openstackgerritDongcan Ye proposed openstack/ironic-python-agent master: Add more channel number for detecting BMC IPv6 address  https://review.opendev.org/66009411:10
*** Lucas_Gray has joined #openstack-ironic11:32
openstackgerritMerged openstack/ironic master: Remove deprecated option [ilo]power_retry  https://review.opendev.org/65921911:34
*** hjensas is now known as hjensas|afk11:43
*** rh-jelabarre has joined #openstack-ironic11:48
nishagbGreetings ironic! :)11:49
*** Lucas_Gray has quit IRC11:55
dtantsurhi nishagb11:56
*** Lucas_Gray has joined #openstack-ironic11:57
*** rachit7 has quit IRC12:06
*** dsneddon has joined #openstack-ironic12:11
*** dsneddon has quit IRC12:15
*** derekh has quit IRC12:25
*** ianychoi has quit IRC12:28
*** ianychoi has joined #openstack-ironic12:29
*** bfournie has joined #openstack-ironic12:33
jrollmorning everyone :)13:01
iurygregorymorning jroll13:01
*** derekh has joined #openstack-ironic13:04
dtantsurmorning jroll13:06
jroll\o iurygregory dtantsur13:06
rpittauhey jroll  :)13:07
jrollmornign rpittau :)13:07
TheJuliaGood morning everyone13:19
jroll\o TheJulia :)13:20
TheJuliaI'm officially declaring IRC Message Bankruptcy. If there are messages that were sent in the last week or two that I've not responded to already... please message me again.13:21
rpittauhi TheJulia :)13:24
*** jroll has quit IRC13:25
*** jroll has joined #openstack-ironic13:26
TheJuliaarne_wiebalck: you around?13:32
arne_wiebalckTheJulia: yes13:32
arne_wiebalckTheJulia: welcome back!13:32
TheJuliaarne_wiebalck: thanks!13:33
*** pcaruana has quit IRC13:33
TheJuliaI'm working on the list of priorities for the cycle, nad on the etherpad https://etherpad.openstack.org/p/DEN-train-ironic-ptg we wondered if you would kind of be leading the "quarantine/retirement" thread of discussion and work13:33
* TheJulia has not read what dmitry had sent for a ML summary, so she might need to go back and check that13:34
arne_wiebalckSure.13:34
iurygregorymorning TheJulia o/13:34
arne_wiebalckTheJulia: The main issue on this one is probably to find consensus on how to achieve this (see the discussion on the initial spec and the discussion on the meeting last week).13:36
arne_wiebalckTheJulia: But, yes, I can try to move this forward.13:36
TheJuliaarne_wiebalck: awesome, I wll try to read through that htis week. I've got a multi-hour errand I now need to run today (on a plus side, not tomorrow....) and expense reports to do...13:37
*** verma-varsha has joined #openstack-ironic13:38
dtantsurmorning TheJulia13:38
arne_wiebalckTheJulia: Sure, no worries. I'll try to get some more input and make a proposal.13:38
dtantsurTheJulia: tl;dr latest ironicclient releases blew up everything, but we seem to be back to norm now13:39
dtantsuralso finally got IPA stable branches operating13:40
TheJuliawoot13:47
TheJuliaso was this like... detonation style blowing up, or did things just grind to a halt?13:47
rpittauTheJulia: everything started slowly failing, first queens and rocky, then stein and master13:52
rpittaujust to confuse things a little13:53
mjturekhey arne_wiebalck - had a question on a review I left here https://review.opendev.org/#/c/634681/19/ironic_python_agent/hardware.py@1228 . Just want to confirm, in the initial implementation, when software raid is used, it will use all disks on the system?13:53
patchbotpatch 634681 - ironic-python-agent - Software RAID: Create/delete configurations - 20 patch sets13:53
rpittauactually queens didn't fail because the change to the ironicclient was not merged there when the new version came out13:53
dtantsurthen tripleo and RDO went down as well, then people started complaining downstream..13:55
TheJuliarpittau: sweet!13:55
TheJuliawow13:55
dtantsurso I'd say detonation style :)13:55
TheJuliaThat is kind of impressive13:55
rpittauwell detonation followed by avalanche :P13:56
iurygregorynuclear detonation?13:56
* TheJulia tries to figure some words for "why get rid of wsme"13:56
rpittauiurygregory: luckily no fallout though xD13:57
iurygregoryrpittau, yeah13:57
openstackgerritDirk Mueller proposed openstack/bifrost master: Remove openSUSE Leap 42.3  jobs  https://review.opendev.org/66012313:58
TheJuliaetingof: do you happen to have a story link for redfish virtual media?14:02
arne_wiebalckmjturek: Thanks for reviewing!14:03
arne_wiebalckmjturek: Yes, that's correct.14:03
mjturekgot it :)  sorry for the misunderstanding!14:03
mjturekplanning to keep watching the feature arne_wiebalck14:03
arne_wiebalckmjturek: Please do, feedback id welcome!14:04
dtantsurTheJulia: the answer is "why get rid of wsme" is that it's no longer maintained14:06
dtantsurstephenfin and T are not going to do it forever14:06
dtantsur* and I14:06
* etingof is waiting for SB to reveal14:07
etingofTheJulia, https://storyboard.openstack.org/#!/story/152675314:08
TheJuliamjturek: did you raise the note about ppc netbook v6?14:08
etingofTheJulia, that vmedia patch ^ gets as far as to timing out on IPA callback \o/14:10
mjturekTheJulia: on     [TheJulia] UEFI HTTP Boot from URL?14:10
mjturekthat was baha I think14:10
TheJuliabaha: o/14:11
TheJuliamjturek: I think your right by the colors, just... it would be good to understand if ppc supports and adheres to the v6 tftp standards, OR just expects tftp netbooting to work the same as v4... which is going to be problematic.14:11
TheJuliaso if baha has details and can share that context, it would be helpful for the community to understand if we need to make a driver_info/force_ip_version option for network booting...14:12
*** etingof is now known as etingof|brb14:13
*** hjensas|afk is now known as hjensas14:13
mjturekTheJulia: I will ask him! We might need to reach out to our firmware contacts to confirm14:13
TheJuliaSounds like a plan14:13
TheJuliaarne_wiebalck: do you have a link for the power state callbacks to nova work?14:15
* arne_wiebalck checking ...14:15
arne_wiebalckhttps://review.opendev.org/#/c/636132/14:16
patchbotpatch 636132 - nova-specs - Support server power state update through external... - 7 patch sets14:16
openstackgerritMerged openstack/ironic-inspector master: pxe_filter: fix get blacklist error  https://review.opendev.org/65971714:22
openstackgerritMerged openstack/ironic-python-agent master: Add more channel number for detecting BMC IPv6 address  https://review.opendev.org/66009414:23
openstackgerritMerged openstack/ironic master: Do not try to return mock as JSON in unit tests  https://review.opendev.org/66006114:24
openstackgerritDmitry Tantsur proposed openstack/ironic stable/stein: Do not try to return mock as JSON in unit tests  https://review.opendev.org/66013314:25
openstackgerritDmitry Tantsur proposed openstack/ironic stable/rocky: Do not try to return mock as JSON in unit tests  https://review.opendev.org/66013414:26
openstackgerritDmitry Tantsur proposed openstack/ironic stable/queens: Do not try to return mock as JSON in unit tests  https://review.opendev.org/66013514:26
*** rachit7 has joined #openstack-ironic14:37
*** cdearborn has joined #openstack-ironic14:43
bahaTheJulia: I'm unfortunately not certain if our compliance is still at v4 or goes through to v6, I'll ask some people internally!14:50
*** pcaruana has joined #openstack-ironic14:53
*** etingof|brb is now known as etingof14:54
*** dnuka has joined #openstack-ironic14:55
dnukagood morning everyone o/14:55
TheJuliabaha: sounds good, let us know14:57
*** gyee has joined #openstack-ironic14:57
*** skazi has quit IRC14:58
iurygregorymorning dnuka14:59
arne_wiebalckhey dnuka o/14:59
rpittauhi dnuka :)14:59
iurygregoryo/15:00
iurygregoryops too fast15:00
openstackgerritJerry Wang proposed openstack/ironic master: Add DHCP server part to make the document more detail  https://review.opendev.org/64603015:00
dtantsurheh15:00
mjtureko/15:00
dnukahey arne_wiebalck, iurygregory, rpittau o/15:00
jungleboyjo/15:00
dnukao/15:00
mgoddard\o15:00
cdearborn0/15:00
* dtantsur waits for the actual #startmeeting :)15:00
dtantsurTheJulia: ^^15:01
arne_wiebalckdtantsur: this15:01
* iurygregory will use the other hand after the #startmeeting15:01
dtantsurlol15:01
dnuka:)15:01
arne_wiebalckiurygregory: lol15:01
dtantsur#startmeeting ironic15:01
openstackMeeting started Mon May 20 15:01:43 2019 UTC and is due to finish in 60 minutes.  The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot.15:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:01
dtantsur#chair TheJulia15:01
*** openstack changes topic to " (Meeting topic: ironic)"15:01
openstackThe meeting name has been set to 'ironic'15:01
etingofo/15:01
iurygregory\o15:01
openstackCurrent chairs: TheJulia dtantsur15:01
rpioso\o15:01
arne_wiebalcko/15:01
rpittauo/15:01
*** kaifeng has joined #openstack-ironic15:01
dtantsurokay, I could not resist, sorry :)15:01
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Add train priorities  https://review.opendev.org/66014515:01
TheJuliao/15:02
bdoddo/15:02
TheJuliawoot!15:02
dnuka\o/15:02
kaifengo/15:02
dtantsurTheJulia: the mic is yours now :)15:02
*** verma-varsha has quit IRC15:02
TheJuliadtantsur: thanks, I was just finishing up the priorities doc15:02
mgoddardfalse start!15:02
TheJuliao/ everyoen15:02
mgoddard\o15:02
TheJuliaSorry sorry!15:02
* jroll sneaks in the back \o15:02
hjensaso/15:02
dtantsurno worries, last Monday I was 10 mins late :D15:02
TheJulialol15:02
dtantsurit's hard to get back in schedule after PTO..15:02
TheJuliaYeah, and I literally was on PTO last week15:03
jungleboyj:-)15:03
TheJuliaOur agenda this week can be found on the wiki.15:03
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic15:03
*** w14161_1 has joined #openstack-ironic15:03
bahao/15:03
TheJulia#topic Announcements/Reminders15:03
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)"15:03
TheJulia#info Dmitry has written up notes from the PTG (Thanks Dmitry!) and shared them with the world.15:04
TheJulia#link https://dtantsur.github.io/posts/ironic-denver-2019/15:04
cdearborn0/15:04
TheJulia#info The Next PTG will be in Shanghai on November 6th through 8th.15:04
TheJuliaDoes anyone have anything to add or remind us of?15:05
jungleboyjWow, look like good notes.15:05
dtantsur:)15:06
TheJuliaI guess not...15:06
TheJuliaitems to raise that is :)15:06
TheJuliadtantsur: I'm sorry, I've not had time to look at your notes yet. Hopefully later today :)15:07
dtantsurNo problem, they will be waiting for you :)15:07
rpiosoThey're awesome!15:07
TheJulia\o/15:07
dtantsurthx :)15:07
TheJuliaMoving on then!15: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
TheJulia#info We had no formal action items from last week, but I know there was post meeting discussion of someone emailing regarding the baremetal sig.15:08
TheJuliaSince I've barely even started looking at my email, did that take place?15:08
dtantsurarne_wiebalck: I think it was you ^^15:08
dtantsurand I think I've seen some email15:08
arne_wiebalckI contacted Chris.15:08
TheJuliaOkay, then I think we can move along then :)15:08
arne_wiebalckI don't think he's sent sth out.15:08
dtantsurwe also had an implicit action item to fix the CI, and we did it (modulo the non-voting multinode grenade job)15:09
TheJuliaugh15:09
TheJuliaIt does not look like an email went to a mailing list, at least one that shows in my mailing list mailbox inbox15:10
TheJulia#action TheJulia follow-up on baremetal sig communication15:10
TheJuliaMoving on!15:11
TheJulia#topic Review subteam status reports15:11
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:11
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:11
TheJuliaStarting around line 21715:11
TheJuliaLooks like Software raid could use some reviews?15:12
arne_wiebalckYes, please :)15:12
arne_wiebalckSome reviews were already done, thanks!15:13
dtantsur++15:13
TheJuliaI'm going to strike out the bulk of deploy templates, since step decomposition will replace it15:13
dtantsurI was going to get there, but $errands15:13
TheJuliaHas anyone started ropping the py35 tests?15:13
TheJuliawell, test jobs15:13
rpittauyes, I saw at least 2 changes15:14
dtantsurI think most of our projects don't have them15:14
rpittauhttps://review.opendev.org/65238515:14
dtantsurnote that we shouldn't literally drop them15:14
patchbotpatch 652385 - networking-generic-switch - Dropping the py35 testing - 2 patch sets15:14
dtantsurthere is a new template in zuul per version15:14
TheJuliadtantsur: ++15:14
dtantsurI don't recall the name, something like python-train-jobs15:14
TheJuliadtantsur: can you investigate and follow-up for our next meeting for that?15:15
dtantsurI can try15:15
TheJuliaThanks15:15
TheJuliafast track... one day I'll write docs :(15:16
TheJuliaetingof: how is redfish virtual media going?15:16
* dtantsur hides15:16
dnuka:)15:17
etingofTheJulia, simulation is up and running15:17
* etingof is debugging redfish hw type against the simulator15:17
TheJuliaawesome15:17
* etingof gets up to the point where VM is [not] booting15:17
TheJuliaetingof: Could you update the whiteboard in that case?15:17
etingofsure15:17
TheJuliaThanks!15:17
TheJuliaThe locking item is not on the proposed priority list, but it is stil a nice to have general improvement that we have some consensus behind. Looks like I need to rebase that. hopefully this week.15:18
dtantsur++15:18
TheJuliaarne_wiebalck: re nova callbacks, is it just power, or is there work in progress for state at this time?15:18
arne_wiebalckTheJulia: Not sure I understand the question, sorry.15:19
TheJuliaarne_wiebalck: I know you've been invovled in some of the power call back discussikons with nova, does that include state callbacks or just power?15:19
dtantsurI forgot to announce it, but better late than never: two outreachy interns are joining us for summer!15:20
arne_wiebalckOnly power.15:20
jungleboyjdtantsur:  Nice.15:20
TheJuliadtantsur: \o/15:21
dnuka\o/15:21
TheJuliaarne_wiebalck: okay15:21
rpiosodtantsur: Great news :)15:21
dtantsurone will work on sushy, the other - on ironic-python-agent-builder15:21
TheJuliacool15:21
arne_wiebalckTheJulia: The power stuff is difficult enough ;)15:23
TheJuliaThe rest of the items on the whiteboard were dropped from Stein. They are, as far as I'm remembering from typing, not in the proposed patchset for train15:23
TheJuliaarne_wiebalck: Yeah, and it still helps a LOT!15:23
dtantsurTheJulia: maybe for Train we can have two sections: 1. Priorities, 2. Things people care about that progress well15:24
TheJulia++15:24
dtantsurthat's essentially what we had for Stein, but expressed explicitly15:24
TheJuliakind of, but I divided that by size of work15:24
*** jcoufal has joined #openstack-ironic15:24
TheJuliaAnyway, the change for the train priorities is up in review15:24
dtantsurgreat!15:24
TheJulia#link https://review.opendev.org/#/c/660145/15:25
patchbotpatch 660145 - ironic-specs - Add train priorities - 1 patch set15:25
TheJuliaSo I guess we're good to proceed onward?15:26
dtantsuryep15:27
rpittaulet's!15: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.openstack.org/p/IronicWhiteBoard15:27
TheJuliaStarting around line 12315:27
TheJuliaLooks like a few items have merged15:28
TheJuliaA few items are stable-core reviews15:29
TheJuliaSpeaking of which, tonyb, let us know if the overall policy has been updated as discussed in Denver15:29
*** dsneddon has joined #openstack-ironic15:29
TheJuliaI'm going to remove merged items15:29
TheJuliaAre tehre any other items that people feel need to be on this list?15:30
mgoddardadded one for nova15:30
mgoddardhttps://review.opendev.org/65980915:30
patchbotpatch 659809 - nova - [ironic] Don't remove instance info twice in destroy - 1 patch set15:30
bdoddCould use reviews on this sushy patch https://review.opendev.org/#/c/657688/15:31
patchbotpatch 657688 - sushy - Enhance Storage models to support RAID config - 3 patch sets15:31
mgoddardefried has said he will proxy a +1 from an ironic-core15:31
TheJuliaI've not looked at your patch mgoddard, but why just not ignore it?15:31
TheJuliamgoddard: are you thinking from a backport standpoint?15:31
mgoddardTheJulia: it requires you to wait for all the retries15:31
TheJuliaI clearly need more coffee15:32
kaifengI'd like to have some reviews on the spec https://review.opendev.org/#/c/650630/15:32
patchbotpatch 650630 - ironic-inspector-specs - Introduce etcd for service coordination - 4 patch sets15:32
dtantsuroh, this ^^ is a good one15:32
* dtantsur +1 to adding it15:32
* TheJulia adds to the lists15:32
kaifengthanks :)15:32
bdodd\o/15:33
TheJuliamgoddard: looks like you already got your patch15:33
TheJuliaOkay, if there is nothing else, I guess we can mov on to Baremetal sig15:33
TheJulia#topic Baremetal Sig15:35
*** openstack changes topic to "Baremetal Sig (Meeting topic: ironic)"15:35
TheJuliaHmm.. No Chris. :\15:35
arne_wiebalckI think the main task for now would be to complete the white paper.15:36
TheJuliaOkay, Well use case documentation is in the list of goals for train. Has anyone had time to look at or discuss the white paper?15:36
TheJulia++15:36
* TheJulia wonders if "review the whitepaper is a thing we could put on the priorities for the week.15:37
TheJulia"15:37
jungleboyjI have followed up with Lenovo and have a couple of people that would like to get involved.15:37
TheJulia\o/15:37
TheJuliajungleboyj: that is awesome news15:37
jungleboyjYeah, not sure if they have joined the meeting.  May need to get them up to speed.15:37
jungleboyjI got the feeling at the summit though, that we didn't really know what the next steps for the WhitePaper were though.15:38
arne_wiebalckjungleboyj: The work on the document has started.15:38
TheJuliaLargely it was use cases15:38
TheJuliaand operator cases15:38
arne_wiebalckjungleboyj: More use cases are probably needed15:38
jungleboyjOk.15:38
TheJulia#link https://docs.google.com/document/d/1KBhJcmpCTm8hn0BX-jO5MAyBYf2RHM5Jh5ib7GTZogM/edit#heading=h.1jw85s2vpx7q15:39
TheJuliaAlso some context on use, which I think this team can add quite a bit about15:39
TheJuliaMaybe us engineering types should start on the last page and move forward :)_15:40
TheJulias/_//15:40
jungleboyj:-)15:40
TheJuliaAnyhow, if there is nothing else to discuss re the white paper, we can move along to RFE review15:40
jungleboyjI will keep following up with my team.15:41
TheJuliajungleboyj: Awesome, thanks15:41
TheJuliaMoving on to RFE review15:41
TheJulia#topic RFE Review15:41
*** openstack changes topic to "RFE Review (Meeting topic: ironic)"15:41
TheJulia#link https://review.opendev.org/#/c/656799/415:42
patchbotpatch 656799 - ironic-specs - Add support for node retirement - 4 patch sets15:42
TheJuliaI've not had a chance to read this sadly15:42
*** S4ren has quit IRC15:42
arne_wiebalckThis will probably need to be superseded by a more general proposal.15:43
*** verma-varsha has joined #openstack-ironic15:43
arne_wiebalckI'll try to gather more use cases and make a suggestion.15:43
TheJuliaI think we're generally open to general proposals and the overall theme is covered by the train priorities... It hink15:43
TheJuliathink15:43
dtantsurI guess the key point is to avoid inventing the same feature under two slightly different flavors15:44
TheJulia++15:44
dtantsuri.e. retirement and quarantine15:44
*** tzumainn has joined #openstack-ironic15:44
TheJuliaI suspect we'll need separate workflows15:44
arne_wiebalckdtantsur: totally agree15:44
dtantsurI remember discussing it last time and... I don't remember anything from then :(15:44
TheJuliabut overall mechanics under the hood should be similar15:44
TheJuliaSounds like we all need coffeee15:44
TheJulia\o/ coffee15:44
TheJuliaI guess we can talk about the next item then. A deployment API15:45
TheJulia#link https://storyboard.openstack.org/#!/story/200573715:45
rpittauI know we don't want to overload "maintenance", but quarantine sounds like an extended maintenance ? :)15:45
arne_wiebalckOne point will be whether or not this need to be a state (or a flag).15:45
TheJuliarpittau: but it is still a state of being for for the node.. :\15:45
* arne_wiebalck was surprised to see how ops voted at the ptg15:45
rpittauyeah, right15:45
* TheJulia barely remembers that now15:45
arne_wiebalckstate:4, flag: 015:46
arne_wiebalckflag seems more natural to me tbh15:46
arne_wiebalck(and easier :-D)15:46
dtantsura flag it's definitely easier, but also less visible15:47
arne_wiebalckdtantsur: true15:47
TheJuliadtantsur: re your deployment api.... I think your missing instance_uuid since that needs to match nova's already created DB entry15:47
TheJuliaa flag also gets us into lots of corner case code that is less visible when we go through the code :\15:47
dtantsurTheJulia: instance_uuid is implicit there. it's either set in advance or can be set to deployment UUID15:47
dtantsurI will mention it in the spec, if we decide it needs a spec15:48
TheJuliadtantsur: I'm not sure a spec is really required, it is fairly clearly a pass-through (in flask maybe?!?)15:48
arne_wiebalckbut a flag can be carried around while walking the state machine15:48
TheJuliaPOST /v1/deployments/instance_uuid ?15:49
dtantsuryeah, it's largely a pass-thru to Node, although it will probably have to have a short record in the database15:49
* arne_wiebalck wonders if we have two discussions at the same time :) 15:49
dtantsurTheJulia: you mean, make Deployment UUID == Allocation UUID?15:49
TheJuliaarne_wiebalck: yeah... And I think that is where the two features begin to differ15:49
dtantsurarne_wiebalck: we do :)15:49
arne_wiebalckok, let's stop the quarantine one for now15:49
TheJuliadtantsur: well, if one was chosen externally, one has to be able to define it15:49
TheJuliaoh, but I just had a thought :)15:50
TheJuliare quarentine :)15:50
* TheJulia stops now15:50
TheJuliaI seriously need more coffee and we only ahve 10 minutes left15:50
TheJuliaI'm basically good with the rfe for the API. I do think a quick spec is going to be useful since the proposal is a tl;dr :)15:50
dtantsuryeah, I'll probably write something this week15:51
TheJuliak15:52
TheJuliaI guess that leaves us to open discussion!15:52
TheJulia#topic Open Discussion15:52
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:52
TheJuliaDoes anyone have anything to discuss during the meeting?15:52
* TheJulia hears the army of crickets appear15:53
dtantsur:)15:54
TheJuliaWell everyone, thanks!15:58
TheJuliaHave a wonderful week!15:58
jungleboyjThank you.15:58
rpittauthanks! you too!15:58
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent stable/rocky: [WIP|DNM] testing coreos jobs  https://review.opendev.org/65650715:59
kaifengThank you15:59
arne_wiebalckThanks!15:59
rpiosoThank you, and welcome back TheJulia.15:59
dnukathank you TheJulia15:59
TheJulia#endmeeting16:00
*** 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-dev mailing list."16:00
openstackMeeting ended Mon May 20 16:00:11 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:00
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-05-20-15.01.html16:00
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-05-20-15.01.txt16:00
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-05-20-15.01.log.html16:00
dnukao/16:00
*** dnuka has quit IRC16:00
rpiosodtantsur: Now that Rocky appears to be back in fighting shape, please take a look at https://review.opendev.org/#/c/65642516:00
patchbotpatch 656425 - ironic (stable/rocky) - Set boot_mode in node properties during OOB Intros... - 1 patch set16:00
*** kaifeng has quit IRC16:01
rpittauuhmmm yeah about that, I have the very bad feeling rocky is broken again16:02
dtantsuroh noi16:03
rpittauat least I saw IPA failing a lot today16:03
rpittauI guess that will impact ironic also16:03
rpittauI'm not 100% sure, have a test job running now16:04
rpiosoUgh16:04
rpittausorry for the bad news, I hope I'm wrong :/16:05
*** jcoufal has quit IRC16:07
rpittauneed to go, if anyone has time to poke around, this is the test job for IPA:16:10
rpittauhttps://review.opendev.org/65650716:10
patchbotpatch 656507 - ironic-python-agent (stable/rocky) - [WIP|DNM] testing coreos jobs - 8 patch sets16:10
rpittaugood night! o/16:10
*** rpittau is now known as rpittau|afk16:10
*** Lucas_Gray has quit IRC16:12
*** verma-varsha has quit IRC16:22
*** verma-varsha has joined #openstack-ironic16:22
*** itlinux_ has quit IRC16:26
*** lucasagomes has quit IRC16:32
arne_wiebalckbye everyone o/16:34
*** eandersson has joined #openstack-ironic16:55
TheJuliabraaaainnnsss17:01
*** rachit7 has quit IRC17:02
openstackgerritIlya Etingof proposed openstack/ironic master: Fix GRUB config path when building EFI ISO  https://review.opendev.org/65905017:03
openstackgerritIlya Etingof proposed openstack/ironic master: Fix binary file upload to Swift  https://review.opendev.org/65906117:05
*** e0ne has quit IRC17:07
*** dtantsur is now known as dtantsur|afk17:08
dtantsur|afksee you17:08
*** derekh has quit IRC17:11
openstackgerritIlya Etingof proposed openstack/ironic-specs master: Add spec for indicator management  https://review.opendev.org/65568517:16
*** tssurya has quit IRC17:21
TheJuliagoodnight17:24
*** verma-varsha has quit IRC17:30
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Managers resource support  https://review.opendev.org/63868217:35
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Chassis resource support  https://review.opendev.org/65380217:35
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Indicator LED resource emulation  https://review.opendev.org/65593617:35
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Virtual Media support  https://review.opendev.org/65679217:35
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add `get/set_boot_image` to libvirt driver  https://review.opendev.org/65693817:36
*** tesseract has quit IRC17:51
*** tzumainn has quit IRC18:22
NobodyCamGood morning ironic’ers18:30
rpiosoGood morning, NobodyCam18:35
NobodyCamHey hey happy Monday rpioso18:36
rpiosoTheJulia: If someone other than the original owner of a change which is up for review rebases it locally and submits it, does the owner change -- https://review.opendev.org/#/c/650103/ ?18:37
patchbotpatch 650103 - python-dracclient (stable/stein) - Add a real-time option when commit RAID creation/d... - 2 patch sets18:37
TheJuliarpioso: owner does not, committer does18:38
rpiosoTheJulia: Thank you!18:38
*** e0ne has joined #openstack-ironic18:45
*** e0ne has quit IRC18:57
*** tzumainn has joined #openstack-ironic19:06
*** derekh has joined #openstack-ironic19:26
*** derekh has quit IRC19:26
*** toabctl has quit IRC19:43
openstackgerritRichard G. Pioso proposed openstack/ironic stable/rocky: Place upper bound on python-dracclient version  https://review.opendev.org/65165719:46
*** bfournie has quit IRC19:54
*** dsneddon has quit IRC19:54
*** dsneddon has joined #openstack-ironic20:03
openstackgerritDirk Mueller proposed openstack/bifrost master: Remove openSUSE Leap 42.3  jobs  https://review.opendev.org/66012320:11
*** whoami-rajat has quit IRC20:19
*** Lucas_Gray has joined #openstack-ironic20:28
*** bfournie has joined #openstack-ironic21:04
*** itlinux has joined #openstack-ironic21:14
*** pcaruana has quit IRC21:40
*** itlinux has quit IRC21:49
rpiosoTheJulia: If a change's parent has been abandoned,  must it be rebased before it'll merge?22:10
JayFrpioso: yes, because it can't merge without that parent merging in first unless it's rebased on a code that's actually in the repo22:51
JayFyou can't go Step "1, 2, 3" if step 2 doesn't exist anymore.22:51
TheJuliaJayF: Thanks!23:04
*** itlinux has joined #openstack-ironic23:18
*** Lucas_Gray has quit IRC23:20
*** rcernin has joined #openstack-ironic23:22
*** itlinux has quit IRC23:35
openstackgerritMerged openstack/bifrost stable/stein: Switch to Debian Stretch for building DIB images  https://review.opendev.org/65535723:39
openstackgerritDongcan Ye proposed openstack/ironic-inspector stable/stein: pxe_filter: fix get blacklist error  https://review.opendev.org/66022823:44
openstackgerritDongcan Ye proposed openstack/ironic-inspector stable/rocky: pxe_filter: fix get blacklist error  https://review.opendev.org/66022923:44
*** itlinux has joined #openstack-ironic23:49
*** gyee has quit IRC23:51
*** cdearborn has quit IRC23:52
« Sunday, 2019-05-19 Index Tuesday, 2019-05-21 »

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