Monday, 2018-10-01

*** hshiina has joined #openstack-ironic00:09
*** hoangcx has joined #openstack-ironic00:47
*** phuongnh has joined #openstack-ironic01:06
*** JudeCross has joined #openstack-ironic01:20
*** JudeCross has quit IRC01:24
*** ijw has joined #openstack-ironic01:32
*** bfournie has joined #openstack-ironic02:16
openstackgerritNguyen Van Trung proposed openstack/ironic master: Fix not detach boot image for virtual CDROM  https://review.openstack.org/60470403:16
*** JudeCross has joined #openstack-ironic03:21
*** JudeCross has quit IRC03:26
*** ijw has quit IRC03:34
*** olivierb has joined #openstack-ironic04:01
*** pcaruana has joined #openstack-ironic04:06
*** JudeCross has joined #openstack-ironic04:06
*** dnuka has joined #openstack-ironic04:10
*** trungnv has joined #openstack-ironic04:12
openstackgerritShivanand Tendulker proposed openstack/ironic master: Adds support for soft power operations to 'ilo' power interface  https://review.openstack.org/60235804:15
*** pcaruana has quit IRC04:23
*** jaganathan has joined #openstack-ironic04:25
*** skazi has quit IRC04:40
*** gabys has quit IRC05:12
*** phuongnh has quit IRC05:36
*** phuongnh has joined #openstack-ironic05:36
*** dnuka has quit IRC05:39
*** skazi has joined #openstack-ironic05:39
*** pcaruana has joined #openstack-ironic05:51
*** phuongnh has left #openstack-ironic06:11
*** olivierb has quit IRC06:26
*** olivierb has joined #openstack-ironic07:04
*** lenka has joined #openstack-ironic07:08
*** jtomasek has joined #openstack-ironic07:09
*** gabys has joined #openstack-ironic07:17
*** MattMan_1 has quit IRC07:18
*** MattMan_1 has joined #openstack-ironic07:18
*** gabys has quit IRC07:22
*** serlex has joined #openstack-ironic07:39
*** serlex has quit IRC07:48
*** d0ugal has joined #openstack-ironic07:52
*** ijw has joined #openstack-ironic07:56
*** ijw has quit IRC08:01
*** dougsz has joined #openstack-ironic08:05
*** S4ren has joined #openstack-ironic08:09
*** tssurya has joined #openstack-ironic08:11
*** JudeCross has quit IRC08:20
*** ijw has joined #openstack-ironic08:32
*** dnuka has joined #openstack-ironic08:35
*** ijw has quit IRC08:37
*** gabys has joined #openstack-ironic08:40
*** gabys has quit IRC08:45
*** mgoddard has joined #openstack-ironic08:46
openstackgerritMerged openstack/ironic master: Prevent HTML from appearing in API error messages  https://review.openstack.org/59711708:48
*** olivierb has quit IRC08:53
*** olivierb has joined #openstack-ironic08:53
*** ijw has joined #openstack-ironic08:54
openstackgerritMallikarjuna Reddy Kolagatla proposed openstack/ironic stable/rocky: Add admin documentation for rescue interface  https://review.openstack.org/60690508:54
*** trungnv has quit IRC08:54
*** gabys has joined #openstack-ironic08:55
zigoHi there! Could someone help me fix https://bugs.debian.org/909988 ?08:58
openstackDebian bug 909988 in src:ironic "ironic: FTBFS (Failures during discovery)" [Serious,Open]08:58
zigoI don't really get what's happening...08:58
*** ijw has quit IRC08:58
*** e0ne has joined #openstack-ironic08:59
*** derekh has joined #openstack-ironic09:00
*** gabys has quit IRC09:00
zigoI've removed the test file there, because I don't think it's worse spending my time on it, but it'd be nice if someone could have a look.09:08
*** priteau has joined #openstack-ironic09:16
*** gabys has joined #openstack-ironic09:18
*** gabys has quit IRC09:23
*** ijw has joined #openstack-ironic09:30
*** ijw has quit IRC09:35
*** gabys has joined #openstack-ironic09:41
*** gabys has quit IRC09:46
*** dtantsur|afk is now known as dtantsur09:48
dtantsurmorning ironic09:48
dtantsurzigo: I suspect you may have an old version of scciclient (in which case "just disable the test" probably broke your users)09:49
dtantsurzigo: judging by https://packages.debian.org/search?keywords=scciclient&searchon=names&suite=all&section=all you have 0.7.2, while master needs 0.8.0. it may be the cause.09:53
*** ijw has joined #openstack-ironic09:56
*** gabys has joined #openstack-ironic09:56
openstackgerritIlya Etingof proposed openstack/ironic master: Reuse Redfish session tokens  https://review.openstack.org/60540809:56
*** mgoddard has quit IRC09:59
*** ijw has quit IRC10:00
*** gabys has quit IRC10:01
*** hshiina has quit IRC10:01
*** stendulker has joined #openstack-ironic10:10
openstackgerrityolanda.robla proposed openstack/ironic master: Enable configuration of conversion flags for iscsi  https://review.openstack.org/60479410:17
*** gabys has joined #openstack-ironic10:18
*** gabys has quit IRC10:22
*** lenka has quit IRC10:27
*** ijw has joined #openstack-ironic10:32
dtantsurolivierb: out of curiosity, what is https://review.openstack.org/#/c/599544/19/playbooks/roles/bifrost-ironic-install/templates/ironic.conf.j2 doing?10:35
patchbotpatch 599544 - bifrost - Add voting and gating test with Keystone enabled - 19 patch sets10:35
*** ijw has quit IRC10:37
*** gabys has joined #openstack-ironic10:37
*** gabys has quit IRC10:42
openstackgerritMerged openstack/bifrost master: Fix deployment when improper version of python-openssl is installed  https://review.openstack.org/59164610:42
openstackgerritMerged openstack/ironic-python-agent-builder master: fix tox python3 overrides  https://review.openstack.org/60664110:48
openstackgerritIlya Etingof proposed openstack/ironic master: Add configurable Redfish client authentication  https://review.openstack.org/60381810:49
openstackgerritMerged openstack/ironic-inspector-specs master: fix tox python3 overrides  https://review.openstack.org/60664010:51
*** gabys has joined #openstack-ironic10:52
*** ijw has joined #openstack-ironic10:54
openstackgerritDhanuka proposed openstack/sushy master: Add product property to the root Sushy object  https://review.openstack.org/60612510:55
*** stendulker has quit IRC10:56
*** gabys has quit IRC10:57
*** ijw has quit IRC10:58
olivierbdtantsur I was experimenting seing imact on the change on CI and it seems that this impacts result of new test11:08
*** gabys has joined #openstack-ironic11:14
*** gabys has quit IRC11:19
*** mgoddard has joined #openstack-ironic11:23
*** dougsz has quit IRC11:29
*** ijw has joined #openstack-ironic11:30
*** gabys has joined #openstack-ironic11:32
openstackgerritDerek Higgins proposed openstack/ironic master: [WIP] Switch ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode  https://review.openstack.org/58736511:33
*** ijw has quit IRC11:35
*** gabys has quit IRC11:36
*** rh-jelabarre has joined #openstack-ironic11:53
*** gabys has joined #openstack-ironic11:53
*** ijw has joined #openstack-ironic11:54
*** gabys has quit IRC11:58
*** ijw has quit IRC11:58
*** e0ne has quit IRC12:01
*** e0ne has joined #openstack-ironic12:08
*** dougsz has joined #openstack-ironic12:11
*** gabys has joined #openstack-ironic12:12
*** gabys has quit IRC12:17
*** mmethot has joined #openstack-ironic12:21
*** lucasagomes has joined #openstack-ironic12:26
*** gabys has joined #openstack-ironic12:36
*** skazi has quit IRC12:38
openstackgerritMerged openstack/sushy master: Add product property to the root Sushy object  https://review.openstack.org/60612512:39
*** gabys has quit IRC12:41
*** S4ren has quit IRC12:41
*** S4ren has joined #openstack-ironic12:42
openstackgerritIlya Etingof proposed openstack/ironic master: Add configurable Redfish client authentication  https://review.openstack.org/60381812:42
*** jcoufal has joined #openstack-ironic12:51
*** jroll has quit IRC12:54
*** jroll has joined #openstack-ironic12:55
*** mmethot_ has joined #openstack-ironic12:57
openstackgerritMerged openstack/ironic master: pxe/ipxe: Move common calls out pxe.py  https://review.openstack.org/60027612:57
openstackgerritMerged openstack/ironic master: Remove tox checkconfig  https://review.openstack.org/60631212:57
openstackgerritMerged openstack/python-ironicclient master: Minor fixes to README.rst  https://review.openstack.org/59162412:57
*** rloo has joined #openstack-ironic12:58
*** mmethot has quit IRC12:58
*** gabys has joined #openstack-ironic12:58
*** mjturek has joined #openstack-ironic12:59
*** baha has joined #openstack-ironic13:00
*** gabys has quit IRC13:03
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Use oslo_utils for string to bool conversion  https://review.openstack.org/60694513:07
openstackgerritMerged openstack/networking-generic-switch master: Support disabling inactive links  https://review.openstack.org/59899613:15
openstackgerritMerged openstack/networking-generic-switch master: Support disabling inactive links for Juniper  https://review.openstack.org/59899713:15
*** gabys has joined #openstack-ironic13:22
*** S4ren has quit IRC13:23
*** gabys has quit IRC13:26
*** lenka has joined #openstack-ironic13:29
*** skazi has joined #openstack-ironic13:32
*** gabys has joined #openstack-ironic13:38
*** gabys has quit IRC13:43
*** jovial[m] has joined #openstack-ironic13:52
*** gabys has joined #openstack-ironic13:59
*** gabys has quit IRC14:04
openstackgerritMerged openstack/ironic master: Add openstack/placement as a required project for ironic-grenade*  https://review.openstack.org/60685014:04
*** S4ren has joined #openstack-ironic14:14
*** leakypipes is now known as jaypipes14:15
*** beekneemech is now known as bnemec14:21
openstackgerritMadhuri Kumari proposed openstack/ironic master: Implement basic interfaces for GraphicalConsole Interface  https://review.openstack.org/54735614:22
jrolldid I say good morning? I can't remember14:43
openstackgerritDerek Higgins proposed openstack/ironic master: Remove token expiration  https://review.openstack.org/60450714:43
jrollif not, morning everyone14:43
*** hshiina has joined #openstack-ironic14:44
rloogood morning jroll and everyone else :)14:45
etingofis it a bad taste to dup a good morning greeting, jroll? o/14:46
dtantsurmorning jroll and rloo14:46
etingofgood morning rloo ;)14:46
jrolletingof: of course not :) just thinking out loud14:47
TheJuliagood morning!14:47
rloosambetts|afk: have you had a chance to look at the priorities, you're the only core that hasn't yet reviewed: https://review.openstack.org/#/c/603161/14:47
patchbotpatch 603161 - ironic-specs - Stein priorites - 4 patch sets14:47
dtantsurmorning TheJulia14:47
rlooguten morgen dtantsur, TheJulia, und etingof!14:47
mbeierlmorning all14:47
jrollby not voting, sambetts|afk is ensuring that we don't have priorities, and thus a nice relaxed cycle ;D14:47
rloojroll: either that or chaos reigns. both are different and interesting. Shall we remove our votes too? :D14:48
*** etingof is now known as etingof|brb14:48
rloomorning mbeierl!14:48
rpiosoGood morning, ironicers :-)14:49
sambetts|afktaking a look now, I did read through them once but they were still being updated lastI checked14:49
*** cdearborn has joined #openstack-ironic14:50
*** stendulker has joined #openstack-ironic14:52
*** baha has quit IRC14:52
*** chris_j has joined #openstack-ironic14:53
olivierbgood morning TheJulia14:54
sambetts|afkrloo: +2 from me14:54
rloothx sambetts|afk! I'm approving...14:55
*** etingof|brb has quit IRC14:55
TheJulia\o//14:55
TheJuliaI'll do the follow-up rev after the meeting14:55
rloothx TheJulia! :)14:56
olivierbI asked dtantsur to have a deeper look at https://review.openstack.org/599544 but we do not understand the impact of endpoint_override in ironic.conf.j2 which makes the test work if outside of the if/else and the VM does not boot if within if/else (current unchanged code). Furthermore this has to be coupled with https://review.openstack.org/605338 and the fact that https://review.openstack.org/#/c/599544/19/scripts/test-bifrost.sh line 20314:58
olivierb -e use_public_urls=${ENABLE_KEYSTONE}  enables the public URI in endpoints. Could you please shed some light on this ?14:58
patchbotpatch 599544 - bifrost - Add voting and gating test with Keystone enabled - 19 patch sets14:58
patchbotpatch 605338 - bifrost - [DNM] Revert "Add possibility to use proper public... (ABANDONED) - 2 patch sets14:58
patchbotpatch 599544 - bifrost - Add voting and gating test with Keystone enabled - 19 patch sets14:58
olivierbsorry proper link is https://review.openstack.org/59961214:58
patchbotpatch 599612 - bifrost - Add possibility to use proper public URLs for endp... (MERGED) - 5 patch sets14:58
olivierbnot the DNM/Abandonned stuff of course14:59
TheJuliaI suspect, but I'd have to dig throguh ironic's code15:00
TheJuliaAnyway! Time for a meeting!15:00
TheJulia#startmeeting ironic15:00
TheJuliao/15:00
openstackMeeting started Mon Oct  1 15:00:21 2018 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
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
jroll\o15:00
cdearborno/15:00
bdoddo/15:00
stendulkero/15:00
rpiosoo/15:01
hshiinao/15:01
TheJuliaOur agenda can be found on the wiki, as always15:01
mgoddardo/15:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
TheJuliaWe have a few things on the agenda today, but hopefully they will be fairly quick.15:01
rlooo/15:01
TheJulia#topic Announcements / Reminders15:01
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:01
TheJulia#info During the PTG, we discussed Python3, and 3rd party CI.15:02
TheJulia#info The basic consensus after consulting with one of the python3 goal champions is that we should ensure that we at least have some 3rd party CI driver coverage for python3.15:03
rajiniro/15:03
mjturek\o15:03
rajiniro/15:04
sambetts|afkTheJulia: that is something we can look into once we get our driver's SDKs updated15:04
rlooTheJulia: does that mean, each 3rd party CI driver should have some, or does it mean if any 3rd-party CI driver has some?15:04
TheJulia#info As such, third party CI operators should switch at least one CI job to be Python3 native. We don't need full coverage for all scenarios, just need to ensure that the driver generally works, and we can slowly phase all CI jobs over to python315:04
rloo(that explains it, thx!)15:04
TheJulia\o/15:04
TheJuliaSorry, that was a lot of words to type out15:04
TheJuliatl;dr: duplication of jobs is not required, just reasonable coverage/conversion for now.15:05
mjturekTheJulia: I'm going to talk to our pkvmci guru this week and see if he sees any problems there. I think we'll be able to do it though.15:05
TheJuliaThere is only one case that requires duplication, which is mjturek's case :(15:05
TheJuliamjturek: Awesome!15:05
*** etingof has joined #openstack-ironic15:05
openstackgerritMerged openstack/ironic-specs master: Stein priorites  https://review.openstack.org/60316115:05
TheJuliaAnyone else have anything for announcement or reminders?15:06
rloodid the time for forum proposals pass?15:06
TheJuliaOh, I'm likely going to take Thursday off this week, and may take part Wednesday, or tonight. Silly inbound weather :(15:06
rloooh yeah, congrats TheJulia for being on TC. Hope this doesn't take away your time from being our PTL :D15:06
TheJuliarloo: It did, four sessions were submitted15:06
TheJuliarloo: was there another session you were desiring that was not on https://etherpad.openstack.org/p/BER-stein-forum-ironic-brainstorming ?15:07
rlooTheJulia: no, just wanted a FYI thing about it :)15:07
TheJuliaOkay15:07
TheJuliaWell then, I think it is time to move on15:07
TheJulia#topic Review action items from previous meeting15:08
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"15:08
TheJuliaI don't think we had any action items last week, double checking15:08
TheJuliaNope, no action items15:08
TheJulia#topic Review subteam status reports15:09
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:09
TheJuliaI started to put the priorities down in the ironic whiteboard this morning, starting at line 173.15:09
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:09
TheJuliaEach group that has volunteered will want to make sure their information is up to date and that they are tracking anything pertinent to that effort under the item.15:10
TheJuliaAny questions!15:10
TheJulia?15:10
*** munimeha1 has joined #openstack-ironic15:11
rlooTheJulia: are you going to leave it in that format in the etherpad? with the projects/features as items under 'shorter/full' sections?15:11
TheJuliarloo: yeah, that was kind of what I was thinking15:11
TheJuliaI figure we can shuffle items around as we get closer15:12
TheJuliato landing their code or remove items from the list15:12
rlooTheJulia: i'm concerned that it will make it harder to read, by the time i get to DHCP-less.. one, I am guessing the 'Shorter term' part will have scrolled off my window/screen. And people will get the indents wrong, but whatever works for you/the team.15:12
TheJuliaI can see that15:13
TheJuliaI'll just make it one list then15:13
rlooit is somewhat an administrivia thing, i'm only concerned with readability :)15:13
TheJuliaBut very valid :)15:13
rloogiven that we have stories and stories are supposed to be good at tracking stories/features/ ?, we should try to keep the stories updated with what needs to be done, and this etherpad with latest status?15:14
TheJuliaSince priorities are in the process of merging, is everyone good to proceed, or do we need further discussion or even time to add text to the etherpad?15:14
rloopriorities have merged :)15:14
TheJulia\o/15:14
TheJuliaI was going to go through and made sure everything has a story later today15:15
rlooso my ask is for folks to please use the stories to list their subtasks etc, to getting things done :)15:15
TheJulia++++15:15
TheJuliaThat has actually helped me a lot for a few times I've added more subtasks. "Oh, I still need to do that one thing"15:15
jrollditto15:16
rloo(otherwise, I'm good with this. Unless someone wants to highlight something related to a particular item?)15:16
rloolike... exciting/interesting read on devlist about deploy templates, traits and placement ;)15:17
* etingof is wondering is there any difference between a task and a subtask15:17
TheJuliaI've not gotten to my email but there was a long discussion on friday about that as well15:18
mgoddardetingof: in Jira yes, storyboard no AFAIK15:18
jrollTheJulia: ohhhh, it continues :)15:18
openstackgerritDhanuka proposed openstack/sushy master: Add protocol_features_supported property to the root Sushy object  https://review.openstack.org/60697415:18
etingofbut we are talking about storyboard here?15:18
mgoddardetingof: yes15:18
TheJuliajroll: oh noes :(15:18
etingofthank you, mgoddard15:18
TheJuliajroll: I guess I'm jumping into that fray again later today15:19
*** gabys has joined #openstack-ironic15:19
TheJuliaAnyway, Shall we move on?15:19
rloo++15:19
dtantsur++15:19
jrollplease15:19
TheJulia#topic Priorities for this week15:19
*** openstack changes topic to "Priorities for this week (Meeting topic: ironic)"15:19
TheJuliaSo I went through and struck out items that were no longer needed on the list. Does anyone have any preference for things to add?15:20
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:20
TheJuliaLine 11815:20
openstackgerritMerged openstack/ironic master: Switch ironic-tempest-dsvm-functional-python2 to zuulv3  https://review.openstack.org/60009315:20
rlooi will point out that we got most of our priorities done this past week, yay! :)15:20
openstackgerritMerged openstack/ironic stable/rocky: Add admin documentation for rescue interface  https://review.openstack.org/60690515:20
TheJuliarloo: that is very \o/15:21
TheJuliaI'd kind of like to make the ipxe stuff a priority this week, I should be able to revise the patches again today15:21
* TheJulia wonders how specs are looking15:22
rloois there anything python3 related that needs reviewing?15:23
TheJulianot that I'm aware of at the moment15:23
TheJuliaWe could keep the bucket and I could go hunting15:23
*** gabys has quit IRC15:23
TheJuliamgoddard: jroll: with the ongoing ml thread re traits, should we wait on trying to get more eyes  on deployo templates?15:23
TheJulias/deployo/deploy/15:23
jrollTheJulia: I would say wait at least until jay's proposal is up15:24
rlooTheJulia: that's fine, we can wait til next week when etherpad/story is updated, will give us an idea of what is still outstanding wrt py315:24
rloo+1 wait for jay :)15:24
dtantsuryeah, his idea looks interesting15:24
dtantsur(if nobody comes at say, "No, this is not how you're supposed to use $THING" again :D)15:24
mgoddard+1 I'd like to see what he is cooking up before proceding15:24
mgoddardI'm a bit concerned that it's not going to tie into deploy steps, but we'll see15:25
TheJuliaWe could do step visibility as a priority for the week?15:25
rlooif people dont' have much to do, would a bug bash this week be useful? no, i'm not offering to organize or attend ;)15:25
rlooi find it hard to believe we have 256 (out of 303) not triaged :-(15:26
*** sthussey has joined #openstack-ironic15:26
jrollmgoddard: I've been considering a blend where the operator publishes the config things in glance (which are basically deploy templates), and the user requests those templates15:26
TheJuliarloo: agreed, I guess perhaps we should spend time on that this week15:27
dtantsurjroll: yeah, we could store deploy templates there, and pass them to our provisioning API from the virt driver15:28
TheJuliaI guess we're good to move on to discussion if there are no objections?15:28
rloo+1 wrt priorities. fine to have a relaxing week :)15:28
mgoddardjroll: that could work15:29
rloo(well, ipxe/pxe isnt' that relaxing)15:29
TheJuliaEh, but it is a lot of clean up in the end :)15:29
TheJulia(And something has to be hard in that list;))15:29
rlooTheJulia: right :) (CleaningIsHard TM)15:29
TheJulia:)15:30
TheJuliaOkay, lets move on then15:30
TheJulia#topic Discussion15:30
*** openstack changes topic to "Discussion (Meeting topic: ironic)"15:30
*** pcaruana has quit IRC15:30
rloowe didn't announce it but did i see that we're removing a driver?15:30
jrollrloo: yeah, oneview15:30
TheJuliarloo: disclosed quite some time ago15:31
rloojroll: ok, worth mentioning :) ^^ done :)15:31
rlooTheJulia: right, we said we were going to remove or something like that, but we've removed now?15:31
TheJuliaI put up two topics for discussion this week. The first because I wanted a wider discussion/audience to look at it.15:31
jrollwe have not removed yet15:31
TheJulianot yet15:31
jrolloh, discussion, not open discussion :)15:31
rlooOh, ok.15:32
jaypipesjroll: isn't that precisely what my example solution was on the ML thread?15:32
TheJuliaA contributor proposed an update to IPA to enable parallel erasure of devices (This is a very good thing) - https://review.openstack.org/#/c/34672315:32
patchbotpatch 346723 - ironic-python-agent - Parallel erase disk devices - 13 patch sets15:32
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Completely remove support for Glance V1  https://review.openstack.org/60697715:32
dtantsurJFYI ^^15:32
jaypipesjroll: "I've been considering a blend where the operator publishes the config things in glance (which are basically deploy templates), and the user requests those templates" <-- pretty much what I was proposing, no?15:32
jrolljaypipes: we're mid-meeting right now - it's pretty close, let's loop back :)15:32
jaypipesyikes, sorry15:33
jrollno worries15:33
TheJuliajaypipes: No worries :)15:33
TheJuliaAnyway, Does anyone have any strong feels regarding parallel device erasure?15:33
* TheJulia hears crickets15:34
chris_jonly strongly possitive here from the field15:34
rlooomg, that was proposed in 201615:34
TheJuliachris_j: Awesome15:34
etingofwould that cause power consumption spikes and the whole DC lights out... ;)15:34
mgoddard+1 from me15:34
rlooTheJulia: is it the configuration thing that you wanted to discuss today?15:35
mgoddardetingof: how power hungry is erasing?15:35
TheJuliarloo: yeah, I think so15:35
etingofmgoddard, like sustained writes, no?15:35
TheJuliamgoddard: .5A@12v x $NUM_SPINNING_RUST?15:36
jrollTheJulia: what needs to be discussed about the config?15:36
TheJuliaThey just wanted to add it as a kernel argument15:36
jrollwell, it needs a way to be set15:37
TheJuliaWe typically wire in through driver info to allow the conductor configuration to have a flag. I'm generally okay with that, but I think the config file would be better15:37
TheJuliayeah15:37
jrollyeah, I don't see any reason not to make it a global config15:37
TheJuliaI concur15:38
TheJuliaOkay I think that works then, I'll follow-up on my review and we can go from there15:38
mgoddardwhat about if some nodes don't support secure erase?15:38
rloothe default is true. should it be false to preserve existing behavior?15:38
jrollmgoddard: this is regardless of secure erase or not15:38
TheJuliarloo: also true, I agree that is likely best :(15:39
mgoddardjroll: yeah, I'm just wondering if you might want different behaviour dependent upon whether nodes support secure erase. Perhaps not15:39
*** gabys has joined #openstack-ironic15:39
jrollmgoddard: well, secure erase is instant, so this only seems useful for drives that don't support it15:39
TheJuliaNext topic: mgoddard and friends have proposed TENKS https://review.openstack.org/#/c/579583/ and have proposed it to live under ironic's governance. I'm +2 for this because we have a mix of VM creation scripts out there, and this would help drive consistency.15:40
patchbotpatch 579583 - ironic-specs - Add virtual Bare Metal Clusters spec - 10 patch sets15:40
jrollmmm15:40
jrollcan we punt this one to the ML? I haven't looked at it at all15:40
jrollunless you want to wait for me to go read code and code reviews and such :)15:41
mgoddardjovial[m] has picked up the work on tenks after w-miller went back to university15:41
mgoddardI could write something up for the ML15:41
TheJuliaYeah, I think that would be good, I just searched the archives and it had not been mentioned on the lsit15:42
TheJuliaOkay, well I think we're good to move to Open Discussion15:42
mgoddardshould have done before now really, was just hoping for a few more spec reviewers15:42
jovial[m]\o15:42
jrollthanks mgoddard15:43
TheJuliamgoddard: I might be looking at it through rose colored glasses thinking of all the spoons I'll get... but I thought the spec reads well.15:43
TheJuliaanyway, thanks mgoddard and greetings jovial[m]15:43
TheJuliaWith that, Time for Open Discussion15:43
mgoddardTheJulia: that's the plan - lure you in with the spoons15:43
TheJulia#topic Open Discussion15:43
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:43
TheJuliamgoddard: That is crafty.... Very craft.15:44
TheJuliacrafty15:44
* TheJulia can't type today15:44
TheJuliaDoes anyone have anything they wish to discuss?15:44
*** gabys has quit IRC15:44
TheJuliaOh, rloo, you might find this interesting https://review.openstack.org/#/c/606199/15:45
patchbotpatch 606199 - ironic-specs - A fresh way of looking at step retrieval - 1 patch set15:45
rlooTheJulia: i put on my list for this week, thx :)15:46
TheJuliano problem :)15:46
TheJuliaWell if nobody has anything else, we can proceed with taking over the world15:46
dtantsur++15:47
TheJuliaExcellent then!15:47
TheJuliaHave a wonderful day everyone! Thanks!15:47
jrolljaypipes: so yeah, what I said about those templates is pretty much what you described, with some schema differences and the key fact that only the operator can publish templates (I'm not sure if this is true for your ML post)15:48
jrollthanks TheJulia :)15:48
TheJulia#endmeeting15:48
chris_jthanks TheJulia!15:48
*** 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."15:48
openstackMeeting ended Mon Oct  1 15:48:42 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:48
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-10-01-15.00.html15:48
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-10-01-15.00.txt15:48
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-10-01-15.00.log.html15:48
*** chris_j has left #openstack-ironic15:48
*** hshiina has quit IRC15:49
* jroll wonders why he says things like this right before he wants to get lunch15:49
openstackgerritMerged openstack/ironic-lib master: Prevent configuration changes in unit tests from affecting each other  https://review.openstack.org/60564515:49
TheJuliajroll: no idea15:53
TheJuliaLunch sounds really good....15:53
openstackgerritDmitry Tantsur proposed openstack/ironic-lib stable/rocky: Prevent configuration changes in unit tests from affecting each other  https://review.openstack.org/60698715:54
openstackgerritDmitry Tantsur proposed openstack/ironic-lib stable/queens: Prevent configuration changes in unit tests from affecting each other  https://review.openstack.org/60698815:54
openstackgerritMerged openstack/ironic master: Add admin documentation for rescue mode in iLO driver  https://review.openstack.org/60564715:58
*** e0ne has quit IRC15:58
*** gabys has joined #openstack-ironic16:00
jaypipesjroll: yup, only the op would create deploy templates/partitioning schemes/raid configs16:01
*** gabys has quit IRC16:04
*** stendulker has quit IRC16:05
*** olivierb has quit IRC16:08
*** olivierb has joined #openstack-ironic16:08
*** gabys has joined #openstack-ironic16:09
*** etingof has quit IRC16:10
*** gabys has quit IRC16:13
*** etingof has joined #openstack-ironic16:24
*** gabys has joined #openstack-ironic16:26
*** baha has joined #openstack-ironic16:28
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Allow using ironic-lib from source when building tinyipa  https://review.openstack.org/60700016:28
openstackgerritDhanuka proposed openstack/sushy master: Add protocol_features_supported property to the root Sushy object  https://review.openstack.org/60697416:28
*** gyee has joined #openstack-ironic16:30
*** gabys has quit IRC16:30
jrolljaypipes: excellent, then the only real debate is how to "choose" those templates16:31
*** olivierb has quit IRC16:33
*** gabys has joined #openstack-ironic16:45
*** gabys has quit IRC16:49
jaypipesjroll: right.16:52
*** zzzeek_ has joined #openstack-ironic16:53
*** d0ugal has quit IRC16:54
*** tssurya has quit IRC16:54
*** pcaruana has joined #openstack-ironic16:56
*** gabys has joined #openstack-ironic16:58
*** dougsz has quit IRC16:59
*** derekh has quit IRC17:00
*** gabys has quit IRC17:03
*** dtantsur is now known as dtantsur|afk17:03
dtantsur|afksee you tomorrow17:03
*** mgoddard has quit IRC17:04
*** olivierb has joined #openstack-ironic17:08
*** dnuka has quit IRC17:08
*** dsneddon_ has quit IRC17:12
*** S4ren has quit IRC17:13
*** dsneddon has joined #openstack-ironic17:14
*** gabys has joined #openstack-ironic17:16
TheJulia goodnight17:18
*** gabys has quit IRC17:20
*** e0ne has joined #openstack-ironic17:24
*** gabys has joined #openstack-ironic17:24
*** trown is now known as trown|lunch17:24
openstackgerritMerged openstack/bifrost stable/pike: Pin get-pip.py to 3.2  https://review.openstack.org/59453017:26
*** JudeCross has joined #openstack-ironic17:31
*** e0ne has quit IRC17:37
*** e0ne has joined #openstack-ironic17:40
*** e0ne has quit IRC17:41
*** mgoddard has joined #openstack-ironic17:41
openstackgerritMerged openstack/networking-generic-switch master: Use oslo_utils for string to bool conversion  https://review.openstack.org/60694517:45
openstackgerritJulia Kreger proposed openstack/ironic-specs master: A fresh way of looking at step retrieval  https://review.openstack.org/60619918:00
eanderssonAnyone that has used the Cisco Nexus driver and Ironic?18:05
eanderssonhttps://github.com/openstack/networking-cisco/blob/3.2.0/networking_cisco/plugins/ml2/drivers/cisco/nexus/mech_cisco_nexus.py#L54218:05
eanderssonTrying to figure out how to feed it the proper command line arguments18:05
*** mgoddard has quit IRC18:06
eanderssonAnd we just keep getting json errors from the Cisco driver18:07
jrollsambetts|afk: ^ if you're around18:09
eanderssonGuessing it's just something silly we are missing18:12
eanderssonhttp://paste.openstack.org/show/7GhEZr3eQGm1vRZbPZ9q/18:12
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Follow-up to Stein priorities  https://review.openstack.org/60701618:14
*** e0ne has joined #openstack-ironic18:14
* TheJulia goes and marks that wf-1 and goes to grab lunch18:14
eanderssonBasically trying to get switch_info to take a dictionary18:14
*** e0ne has quit IRC18:14
*** mgoddard has joined #openstack-ironic18:15
TheJuliaeandersson: have you tried escaping it? This seems super familiar but it is just not coming to me at the moment18:16
jrolloh yeah, that isn't just a cisco question18:17
eanderssonWe started with escaping, but I guess we could have missed something18:18
jrolland I also can't recall the answer :(18:18
jrolloh wait18:18
jrollI think you need to do something like --local-link-connection switch_info/switch_ip=<ip>18:18
eanderssonoh18:19
jrollthe / gets you down a level in the dict blob18:19
jrollor 'blobject' (c) jay pipes18:19
jaypipes:)18:19
eanderssoninvalid keys :'(18:21
jrollO_o18:21
eandersson>set(['switch_info/switch_ip']) are invalid keys (HTTP 400)18:21
jrollhrm18:21
* jroll looks around18:22
jrollwonder if we have a bug there18:23
eanderssonBack at escaping18:24
eanderssonIf anything I wish there was a single example haha18:24
jrollI would also maybe try ='"{\"switch_ip\": \"<ip>\"}"'18:24
eanderssonI always send the hound out to search for info, but nothing sticks out18:25
jrollbut idk that it will work18:25
eanderssonhttp://codesearch.openstack.org/?q=switch_info&i=nope&files=&repos=18:25
jrollagree with examples18:25
eandersson> switch_info=\"\{\"switch_ip\":\"<ip>\"\}\"18:25
eanderssonmy current attempt :D18:25
jrollheh18:25
* jroll jumps into devstack18:26
jrollit does look like we only accept a string there18:26
jrolleandersson: looks like my hunch was right: https://gist.github.com/jimrollenhagen/fa74c3130609a9f89643ed9711a0617118:32
eanderssonLet me try it again18:32
eanderssonjaypipes, impl would be great here :D18:33
* jroll has to run, bbl18:34
*** e0ne has joined #openstack-ironic18:37
*** Sukhdev has joined #openstack-ironic18:45
*** trown|lunch is now known as trown18:51
openstackgerritMerged openstack/ironic master: Adds support for 'ramdisk' deploy with 'ilo-virtual-media' boot  https://review.openstack.org/59744819:18
*** Sukhdev has quit IRC19:42
eanderssonjroll, yep worked btw20:04
eanderssonMaybe a silly question, but do you need the ml2 plugin baremetal for neutron?20:05
eanderssonI assumed just adding the cisco networking plugin would be enough20:06
TheJuliaeandersson: do you mean networking-baremetal?20:09
eanderssonYea - I assume so20:11
TheJuliaeandersson: If so, yes, it fixes the port state in neutron in some cases20:11
TheJuliathe cisco plugin should support the network port vnic type of baremetal20:11
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Change nic driver to e1000 for CoreOS jobs  https://review.openstack.org/60612720:20
*** jcoufal has quit IRC20:25
*** Sukhdev has joined #openstack-ironic20:30
TheJuliajroll: <3 your reply on the ml regarding the UEFI case which fairly concisely explains why we're on this path.20:34
jrollTheJulia: thanks :) baby steps and all20:35
*** pcaruana has quit IRC20:43
*** efried has joined #openstack-ironic20:51
*** baha has quit IRC20:53
*** e0ne_ has joined #openstack-ironic20:54
*** e0ne has quit IRC20:54
*** efried has left #openstack-ironic20:58
*** v12aml has quit IRC21:05
*** trown is now known as trown|outtypewww21:07
*** v12aml has joined #openstack-ironic21:09
*** priteau has quit IRC21:13
*** priteau has joined #openstack-ironic21:16
*** priteau has quit IRC21:18
*** munimeha1 has quit IRC21:41
*** cdearborn has quit IRC22:01
*** e0ne_ has quit IRC22:04
*** e0ne has joined #openstack-ironic22:05
*** e0ne has quit IRC22:06
*** ianw is now known as ianw_pto22:17
*** jtomasek has quit IRC22:20
*** jcoufal has joined #openstack-ironic22:33
*** lenka has quit IRC22:37
*** Sukhdev has quit IRC22:41
*** threestrands has joined #openstack-ironic22:41
*** rcernin has joined #openstack-ironic22:49
*** jcoufal has quit IRC23:05
*** jcoufal has joined #openstack-ironic23:07
*** jcoufal has quit IRC23:41
*** rh-jelabarre has quit IRC23:43
*** gyee has quit IRC23:47
*** mjturek has quit IRC23:52

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