Monday, 2015-11-23

*** sacharya has joined #openstack-ironic00:33
openstackgerritNaohiro Tamura proposed openstack/ironic: Refactor IRMCVirtualMediaIscsiDeploy by applying new BootInterface  https://review.openstack.org/22137100:50
openstackgerritNaohiro Tamura proposed openstack/ironic: Refactor IRMCVirtualMediaAgentDeploy by applying new BootInterface  https://review.openstack.org/22157700:51
*** Sukhdev has quit IRC00:58
lintanmorning Ironic00:58
mrdalintan: o/01:03
lintanhi mrda :)01:05
*** sacharya has quit IRC01:13
*** masterbound has quit IRC01:31
*** jerrygb has joined #openstack-ironic01:44
*** zhenguo has joined #openstack-ironic01:48
*** aginwala has joined #openstack-ironic01:51
*** aginwala has quit IRC01:58
*** jerrygb has quit IRC02:05
*** jerrygb has joined #openstack-ironic02:05
*** jerrygb has quit IRC02:10
*** harshs has joined #openstack-ironic02:27
tangchen__Hi, morning Ironic. :002:28
tangchen__Would you guys please help  to review this patch ?02:28
tangchen__https://review.openstack.org/#/c/247903/02:28
tangchen__Thx02:28
*** masterbound has joined #openstack-ironic02:32
*** masterbound has quit IRC02:37
*** jerrygb has joined #openstack-ironic03:06
*** jerrygb has quit IRC03:11
*** phschwartz has quit IRC03:16
*** coolsvap has quit IRC03:16
*** phschwartz has joined #openstack-ironic03:16
*** coolsvap has joined #openstack-ironic03:19
*** harshs has quit IRC03:28
openstackgerritZhenguo Niu proposed openstack/ironic: Add console feature to ssh driver  https://review.openstack.org/24145303:31
*** Sukhdev has joined #openstack-ironic03:36
*** MattMan_ has quit IRC03:38
*** MattMan has joined #openstack-ironic03:39
*** wuhg has joined #openstack-ironic03:55
*** harshs has joined #openstack-ironic03:56
*** achanda has joined #openstack-ironic03:58
*** achanda has quit IRC04:10
*** kalpase has joined #openstack-ironic04:11
openstackgerritTan Lin proposed openstack/ironic: Isolate Ironic API from DB  https://review.openstack.org/24349704:12
*** harshs has quit IRC04:13
*** sacharya has joined #openstack-ironic04:24
*** achanda has joined #openstack-ironic05:01
*** achanda has quit IRC05:04
*** Sukhdev has quit IRC05:09
*** kalpase has quit IRC05:20
*** kalpase has joined #openstack-ironic05:22
openstackgerritTang Chen proposed openstack/python-ironicclient: Use API to translate node power states to json data.  https://review.openstack.org/24790305:30
openstackgerritTang Chen proposed openstack/python-ironicclient: Add inject nmi power state.  https://review.openstack.org/24790505:32
openstackgerritTang Chen proposed openstack/python-ironicclient: Add soft reboot/poweroff power states.  https://review.openstack.org/24790405:32
*** mdbooth has quit IRC05:33
*** gridinv has quit IRC05:35
*** rcernin has joined #openstack-ironic05:36
*** mdbooth has joined #openstack-ironic05:39
*** rcernin has quit IRC05:42
*** aginwala has joined #openstack-ironic05:51
*** aginwala has quit IRC06:02
*** sacharya has quit IRC06:12
*** sacharya has joined #openstack-ironic06:13
openstackgerritTan Lin proposed openstack/ironic: Isolate Ironic API from DB  https://review.openstack.org/24349706:18
*** aginwala has joined #openstack-ironic06:27
*** kalpase has quit IRC06:32
*** sacharya has quit IRC06:36
*** Sukhdev has joined #openstack-ironic06:40
*** aginwala has quit IRC06:54
*** aginwala has joined #openstack-ironic07:00
*** rcernin has joined #openstack-ironic07:07
*** openstack has joined #openstack-ironic07:20
*** Sukhdev has quit IRC07:24
*** ukalifon has joined #openstack-ironic07:27
*** ijw has joined #openstack-ironic07:31
*** sacharya has joined #openstack-ironic07:37
*** sacharya has quit IRC07:41
*** ijw has quit IRC07:51
*** jerrygb has joined #openstack-ironic07:51
*** ijw has joined #openstack-ironic07:52
*** jerrygb has quit IRC07:57
*** ijw has quit IRC07:57
*** aginwala has quit IRC07:58
*** aginwala has joined #openstack-ironic08:04
*** pas-ha has joined #openstack-ironic08:05
openstackgerritTan Lin proposed openstack/ironic: Isolate Ironic API from DB  https://review.openstack.org/24349708:21
*** kalpase has joined #openstack-ironic08:25
Haomengzhenguo: zhenguo, I have address your comments, can you help to review,m thanks -  - https://review.openstack.org/#/q/owner:Haomeng%252CWang+status:open,n,z08:32
zhenguoHaomeng: yeah, will continue to review the other patches :)08:37
pas-hamorning all08:38
*** aginwala has quit IRC08:44
betherlyMorning!08:45
openstackgerritxiexs proposed openstack/python-ironicclient: Add --wrap option for "ironic driver-properties"  https://review.openstack.org/24793308:46
*** xek has joined #openstack-ironic08:46
*** thehoffau has quit IRC08:49
Haomengzhenguo: thank you08:52
Haomengzhenguo: :)08:52
*** ijw has joined #openstack-ironic08:53
*** ijw has quit IRC08:58
*** athomas has joined #openstack-ironic09:05
*** dtantsur|afk is now known as dtantsur09:09
dtantsurMorning Ironic!09:09
*** jcoufal has joined #openstack-ironic09:11
*** sturivnyi has quit IRC09:12
openstackgerritMerged openstack/ironic: Add missing packages to dev-quickstart  https://review.openstack.org/24598909:23
*** mgoddard has joined #openstack-ironic09:23
*** jistr has joined #openstack-ironic09:27
*** e0ne has joined #openstack-ironic09:28
*** max_lobur has joined #openstack-ironic09:32
*** max_lobur has left #openstack-ironic09:32
*** PradeepV has joined #openstack-ironic09:37
*** derekh has joined #openstack-ironic09:40
sergekMorning All!09:42
*** masterbound has joined #openstack-ironic09:42
*** kalpase has left #openstack-ironic09:50
*** ijw has joined #openstack-ironic09:54
*** zhenguo has quit IRC09:56
*** sacharya has joined #openstack-ironic09:56
*** ijw has quit IRC09:59
*** ekarlso- has joined #openstack-ironic09:59
*** ekarlso has quit IRC10:00
*** ekarlso- is now known as ekarlso10:00
lucasagomessergek, morning10:00
*** ijw has joined #openstack-ironic10:00
*** sacharya has quit IRC10:01
dtantsurmorning lucasagomes, sergek10:01
lucasagomesdtantsur, morning!10:02
sergeklucasagomes: dtantsur morning! :))10:04
lucasagomesdtantsur, btw, re collecting logs in IPA. I see you have a module right now for inspector10:05
*** ijw has quit IRC10:05
dtantsurlucasagomes, yep10:05
lucasagomesdtantsur, I was thinking if we could reuse that for deploy. Having a idk, config option in ironic that collect the logs of that deployment (gzip base64 it) if the instance fails to deploy10:06
dtantsurlucasagomes, the problem is the definition of "logs of that deployment". for inspection we just send all logs.10:06
dtantsurfor long-running ramdisks, however, it can be difficult10:06
lucasagomesdtantsur, right, you just grab all the logs from journald right?10:07
dtantsurlucasagomes, well, I take some tail of it, usually big enough to fit all logs10:07
lucasagomesyeah for long-running it can be complicated. Perhaps we can filter it, and grab the logs for IPA and some other services related (iscsi etc)10:07
lucasagomesdtantsur, fair enough10:07
dtantsurhttps://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/inspector.py#L276-L27710:07
*** jerrygb has joined #openstack-ironic10:08
dtantsurnote that it does not work all that well right now: https://review.openstack.org/#/c/247514/10:08
lucasagomesdtantsur, will take a look10:08
*** mgould has joined #openstack-ironic10:09
mgouldmorning, Ironicers!10:10
lucasagomesmgould, g'morning!10:11
dtantsurmorning mgould10:11
*** kromanenko has quit IRC10:11
*** jerrygb has quit IRC10:13
*** daemontool has joined #openstack-ironic10:19
*** _degorenko|afk is now known as degorenko10:20
openstackgerritVladyslav Drok proposed openstack/ironic: Add proxy related parameters in agent driver  https://review.openstack.org/23549310:22
sergekmgo10:27
sergekmgould: morning!10:27
mgouldsergek, lucasagomes dtantsur morning!10:29
openstackgerritTang Chen proposed openstack/python-ironicclient: Use a dict to translate node power states to json data.  https://review.openstack.org/24790310:31
*** daemontool has quit IRC10:34
*** daemontool has joined #openstack-ironic10:35
*** sambetts-afk is now known as sambetts10:45
sambettsMorning all o/10:45
*** mkovacik has joined #openstack-ironic10:46
sergeksambetts: :)10:47
sambettso/ sergek10:47
*** gridinv has joined #openstack-ironic10:48
dtantsurmorning sambetts10:53
sambettsMorning dtantsur10:59
*** ijw has joined #openstack-ironic11:01
*** ijw has quit IRC11:05
openstackgerritDmitry Tantsur proposed openstack/ironic: Drop some outdated information from our quick start guide  https://review.openstack.org/24572411:11
sinvalgood morning sambetts, dtantsur, sergek, lucasagomes, mgould and Ironicers o/11:22
lucasagomessinval, hi there! Good morning11:22
sambettso/ sinval lucasagomes11:23
dtantsuro/11:26
caioboGood morning sambetts, dtantsur, sergek, lucasagomes,  mgould, sinval and the rest of  Ironic  :)11:38
dtantsuro/11:39
sinvalhey caiobo :D11:41
mgouldcaiobo, morning!11:44
dtantsurbad karma I guess: installing ubuntu 12.04 server with all default setting resulted in broken apt >_<11:48
*** hoangcx has joined #openstack-ironic11:48
mgoulddtantsur, ouch11:49
sinvalhey dtantsur, sorry, I did not understand your first comment about the class-level attributes of ironicclient/v1/port.py, it is reasonable, I'll update the patch soon11:51
*** masterbound is now known as mbound_11:53
dtantsursinval, thanks!11:53
*** PradeepV has quit IRC11:56
*** baoli has quit IRC11:58
openstackgerritSinval Vieira Mendes Neto proposed openstack/python-ironicclient: Refactoring and removing duplicate code of "base.Manager" heirs  https://review.openstack.org/24669212:00
sinvaldtantsur, there you go ^12:01
*** ijw has joined #openstack-ironic12:02
lucasagomesmorning caiobo12:02
dtantsurawesome12:02
openstackgerritTan Lin proposed openstack/ironic: Add Code Review Guide for Ironic  https://review.openstack.org/24864712:04
*** sacharya has joined #openstack-ironic12:05
*** ijw has quit IRC12:06
*** jaypipes has joined #openstack-ironic12:08
*** sacharya has quit IRC12:10
openstackgerritTan Lin proposed openstack/ironic: Add Code Review Guide for Ironic  https://review.openstack.org/24864712:10
*** ionutbalutoiu has joined #openstack-ironic12:14
openstackgerritSyed Ismail Faizan Barmawer proposed openstack/ironic: Refactor disk partitioner code from ironic and use ironic-lib.  https://review.openstack.org/18444312:18
*** ionutbalutoiu has quit IRC12:21
*** mbound_ has quit IRC12:28
*** mbound_ has joined #openstack-ironic12:31
*** mbound_ has quit IRC12:35
*** hoangcx has quit IRC12:37
*** thrash|wknd is now known as thrash12:43
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Use Reno for release notes management  https://review.openstack.org/24867412:48
dtantsurjroll, when you're online, could you sanity-check ^^ please?12:48
*** electrofelix has joined #openstack-ironic12:54
*** jerrygb has joined #openstack-ironic12:59
*** jerrygb has quit IRC12:59
*** jerrygb has joined #openstack-ironic12:59
*** jerrygb_ has joined #openstack-ironic13:00
*** ijw has joined #openstack-ironic13:02
*** jerrygb has quit IRC13:03
openstackgerritMerged openstack/ironic: Get mandatory patch attrs from WSME properties  https://review.openstack.org/24020213:05
*** ijw has quit IRC13:08
*** baoli has joined #openstack-ironic13:09
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [WIP] Support Linux-IO in addition to tgtd  https://review.openstack.org/23486113:17
*** trown|outttypeww is now known as trown13:20
*** dprince has joined #openstack-ironic13:20
*** daemontool has quit IRC13:25
*** mrmoje has joined #openstack-ironic13:26
jrolldtantsur: left a couple notes, thanks for doing that13:26
jrollmorning everyone :)13:26
*** lucasagomes is now known as lucas-hungry13:28
lucas-hungryjroll, morning13:29
* lucas-hungry goes grab a quick lunch13:29
*** jaypipes has quit IRC13:29
jrolllucas-hungry: somewhat related to earlier conversation about logs: https://review.openstack.org/#/c/164586/13:30
jrollwait, not the one I'm thinking of13:30
dtantsurmorning jroll, thanks!13:31
jrollthere used to be a spec to send agent logs, not seeing it now :/13:32
*** jaypipes has joined #openstack-ironic13:32
jrolloh13:32
jrolllucas-hungry: https://review.openstack.org/#/c/168799/13:33
mgouldwhoa, has my first code patch just been merged? Awesome! \o/13:35
jrollmgould: \o/13:36
* mgould rolls up his sleeves and gets to work on the next one13:36
jrollheh13:36
*** daemontool has joined #openstack-ironic13:38
*** mbound_ has joined #openstack-ironic13:43
*** jistr has quit IRC13:46
openstackgerritMerged openstack/python-ironicclient: Use a dict to translate node power states to json data.  https://review.openstack.org/24790313:52
*** baoli has quit IRC13:56
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Use Reno for release notes management  https://review.openstack.org/24867413:56
dtantsurjroll, ^^13:56
jrollsweet, thanks13:57
openstackgerritSinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Adding port mac validation tests  https://review.openstack.org/24870913:58
*** athomas has quit IRC13:59
* dtantsur reads about next steps13:59
dtantsurjroll, we don't backport the whole patch to liberty, do we? just something like https://review.openstack.org/#/c/241322/, right?14:00
jrolldtantsur: one more thing :(14:00
jrollI thought you would see it heh14:00
jrolland no, we do need to backport it14:00
dtantsuroh dat copy-paste :)14:00
jrollhttps://review.openstack.org/#/c/245866/14:00
jrollbecause jobs will run on liberty14:00
vdrokmorning jroll dtantsur mgould and lucas-hungry14:01
*** liliars_ has joined #openstack-ironic14:01
jrollhi vdrok :)14:01
vdrokjroll, as for this one - https://review.openstack.org/#/c/235493/12 - this is the same behavior as before14:01
sinvalmorning vdrok14:01
jrollvdrok: oh, well, it's still a problem :)14:02
dtantsurmorning vdrok14:02
vdrokjroll, if the temp url is generated for too short time it would fail now too14:02
vdroksinval, morning14:02
jrollvdrok: right, but operators can fix that, they can't fix this issue14:02
mgouldvdrok, morning14:03
vdrokjroll, why not, they can change CONF.glance.swift_temp_url_duration and redeploy and everything will work14:03
*** ijw has joined #openstack-ironic14:03
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Use Reno for release notes management  https://review.openstack.org/24867414:03
*** ionutbalutoiu has joined #openstack-ironic14:04
jrollvdrok: no matter what the duration is, if it is cached, there will always be a time where it is about to expire and deploys will fail14:04
*** athomas has joined #openstack-ironic14:04
vdrokjroll, oh, that's right, so you suggest some 'agent_startup_time' config?14:05
jrollvdrok: let's say the duration is 60 minutes. at t=0, server x is deployed, works fine. at t=58, server y is deployed, it fails14:05
jrollvdrok: maybe, that specifically feels weird, but something to adjust the "early" expiration time14:06
vdrokjroll, ok, will change it, thanks14:06
jrollthank you!14:07
liliars_hey folks, good morning! stoping by just to say I’m going to be a bit off this week. last Saturday I unfortunately was in a car accident, but everything is getting better now, thank god. just need a few days to recover from some injuries (not doing so great being a left handed for some days whilst my right arm recovers heh).14:07
*** ijw has quit IRC14:08
liliars_some of the commitments I had for this week were also being handled by other teammates of mine so cool, but other things will have to wait for me :( anyways, will be back as soon as I can. thanks for understanding! see you all very soon #fingerscrossed14:08
NobodyCamgood morning Ironic folks14:08
liliars_jroll: this also means no docs advance this week, sorry :( was planning on talking to them someday this week but.. :(( talk to you when I manage to do this. thanks!14:09
*** amotoki_ has joined #openstack-ironic14:09
sinvalliliars_, I hope you get weel soon :)14:09
NobodyCamTraveling back home today, will miss the meeting as todays flights do not have wifi :p14:09
sinvalmorning NobodyCam o/14:09
NobodyCammorning sinval14:10
sinvalliliars_: well*14:10
dtantsurjroll, btw do you plan on doing http://lists.openstack.org/pipermail/openstack-dev/2015-November/079542.html for ironic?14:10
dtantsurmorning NobodyCam, liliars_14:10
NobodyCammorning dtantsur14:10
NobodyCam:)14:10
jrollliliars_: oh no, I'm sorry! :( get well and obviously take as much time as you need14:10
dtantsurliliars_, oh, get well please!14:11
jrolldtantsur: yes, we need CI jobs first. grenade for the first two, grenade-partial for the third14:11
NobodyCamwanted to say thank you for well wishes durning the past wee14:12
NobodyCamweek even :p14:12
*** amotoki has quit IRC14:12
*** amotoki_ is now known as amotoki14:12
*** daemontool has quit IRC14:12
*** daemontool has joined #openstack-ironic14:13
NobodyCamI expect to be back to normal tomorrow14:14
jrolldtantsur: +2 on that stuff14:16
dtantsurthanks!14:16
NobodyCamshuting down to get ready to board14:18
liliars_jroll, dtantsur, sinval thanks guys! see you all soon :)14:18
*** liliars_ has quit IRC14:18
*** lucas-hungry is now known as lucasagomes14:18
jrollNobodyCam: safe travels!14:18
lucasagomesjroll, oh it's abandoned14:19
lucasagomesNobodyCam, safe flight!14:19
lucasagomesjroll, tho, reading that spec. I don't wanna the logs on every heartbeat. I just want it if the deployment fails14:19
jrolllucasagomes: yeah, just a related thing14:20
lucasagomescause right now we power off the node after it fails, I remember I had a patch that left the node on14:20
mgouldliliars: yikes! glad to hear you're recovering - get well soon!14:20
lucasagomesbut someone was suggesting collecting the logs14:20
lucasagomesjroll, what you think about it ?14:20
* lucasagomes is about to give it a whirl 14:20
jrolllucasagomes: I'm not opposed, just thinking where to store them etc14:21
jrolland also trying to avoid large POST requests :P14:21
lucasagomesjroll, I was thinking about something like configdrive... we can either live it as a string somwhere in the node object14:21
lucasagomesor put it in swift14:21
lucasagomeswe can put it in the internal_driver_info, and delete after the node is recycled14:22
jrollyeah, it's an option, I just hate large blobs in the db :/14:22
lucasagomesright, it's just temporary14:22
*** baoli has joined #openstack-ironic14:23
dtantsurjroll, lucasagomes, maybe we use pull model instead of push?14:26
dtantsurwe're using push model in inspector because we don't know IP address (it could be changed though)14:26
dtantsurfor failed deploy we can ask agent for logs14:26
*** HimanshuGarg has joined #openstack-ironic14:27
lucasagomesright, I was thinking about having a command for IPA that Ironic can calls (just like prepare_image()) that collect the logs14:27
lucasagomesand the behavior can be enabled/disabled in Ironic14:28
dtantsuryeah, because with pull model you can specify exactly how much logs you need (aka -n argument to journalctl)14:28
dtantsuryou don't have to worry about huge POST's either14:29
lucasagomesdtantsur, right yeah, we could have a way to configure it14:29
lucasagomesdtantsur, like a parameter in the command, so Ironic can specify the maximum no of lines14:30
lucasagomesjroll, ^ ?14:30
*** mgoddard_ has joined #openstack-ironic14:30
lucasagomesdtantsur, tho, you gzip it before sending right? It shouldn't be that big then14:31
dtantsuryeah, but it can be much bigger for deploy14:31
*** mgoddard has quit IRC14:33
*** jasondotstar_afk is now known as jasondotstar14:33
jrolloh yeah, ++ for pull14:33
*** superdan is now known as dansmith14:35
*** sinval has quit IRC14:40
*** rloo has joined #openstack-ironic14:42
pas-hajroll, regarding scciclient: https://bugs.launchpad.net/ironic/+bug/1518999 with a repro14:45
openstackLaunchpad bug 1518999 in Ironic "just installing scciclient may break ironic-conductor" [Undecided,New]14:45
jrollpas-ha: nice, thanks!14:45
openstackgerritMerged openstack/ironic: Adds test case for the iscsi_ilo recreate boot iso  https://review.openstack.org/23962714:46
pas-hatook some time until figured out ironic should be installed as non-development package14:46
*** achanda has joined #openstack-ironic14:53
*** subscope has joined #openstack-ironic14:58
*** mjturek1 has quit IRC14:58
*** mjturek1 has joined #openstack-ironic14:58
*** mjturek1 has quit IRC15:01
*** mrmoje has quit IRC15:01
*** openstackgerrit has quit IRC15:02
*** openstackgerrit has joined #openstack-ironic15:03
*** mrmoje has joined #openstack-ironic15:03
*** ijw has joined #openstack-ironic15:04
*** mjturek1 has joined #openstack-ironic15:05
*** ijw has quit IRC15:09
openstackgerritRuby Loo proposed openstack/ironic: Conductor handles manual cleaning  https://review.openstack.org/24728515:10
sergekdtantsur: Do you want to save HTTP-API on its place like README and CONTRIBUTING? It would be easier to move it to 'doc' folder15:11
*** foexle has joined #openstack-ironic15:12
*** HimanshuGarg has quit IRC15:12
*** HimanshuGarg has joined #openstack-ironic15:14
sambettsMorning jroll and anyone else I've missed o/15:14
jrollheya sambetts :)15:14
dtantsursergek, not necessary. I'm a bit afraid of broken links, so we might leave a placeholder there saying that docs has moved15:14
rlooyawn, stretch. Hi and morning ironickers, sambetts, dtantsur, jroll, sergek15:17
sambettso/ rloo15:17
jrollmorning rloo :)15:18
sambettslucasagomes, dtantsur, jroll RE: IPA logs, if you wanted to avoid large requests between ironic and IPA, could IPA itself put the logs into swift on request, then simply return the swift object reference to ironic? Not sure if that solves any of the problems :/15:19
sergekdtantsur: I am fighting with the broken links last few days :) you are right I could miss something. Anyway I made it for Readme and Contributing15:19
jrollsambetts: that would work except we probably don't want swift creds in the agent :)15:19
lucasagomessambetts, problem is how the ramdisk talks to swift15:19
lucasagomeswe can put it in swift after it's being passed to ironic tho15:20
sambettslucasagomes: thats what inspector does for when we pull more data than we're willing to store in the DB15:20
lucasagomessambetts, right, yeah that should be fine15:21
*** mdbooth has quit IRC15:21
rloohey jroll, wrt reno release notes -- you prefer us to have one patch with code changes + reno release updates? (wrt https://review.openstack.org/#/c/247701/)15:21
rloojroll: i was going to do one patch for just reno release notes but would be good to have an idea on how we want to do this.15:22
*** r-daneel has joined #openstack-ironic15:22
rloojroll: i mean one patch that mentions the feature, deprecated configs etc. but maybe it makes sense to have with the code change.15:23
jrollrloo: yeah, I'd prefer with the code change, but for long feature additions like this one the "we have a new feature!" release note could be at the end of the chain15:24
rloojroll: okey dokey :)15:25
jrollrloo: but since that's a config change I'd like to keep that part with that patch15:25
jrollif that makes sense15:25
rloojroll: yeah, both ways make sense in this case. but might be a good habit to do with config change so we know to do that in the future.15:25
jrollrloo: yeah, exactly :)15:26
jrollrloo: I think of it as generally "add the reno with the change that calls a thing done" - so for this config change, that's calling the config change done. for the feature as a whole maybe add it with the docs :)15:26
rloojroll: yup. either with docs or separate. depends on how big the doc patch is :)15:27
jrollyep!15:28
*** mgoddard has joined #openstack-ironic15:29
*** mgoddard_ has quit IRC15:29
dtantsurmorning rloo15:31
*** willsama has joined #openstack-ironic15:31
*** willsama has quit IRC15:31
*** mdbooth has joined #openstack-ironic15:32
openstackgerritSerge Kovaleff proposed openstack/ironic-inspector: Generate Sphinx docs  https://review.openstack.org/24698515:34
*** Sukhdev has joined #openstack-ironic15:40
*** mdbooth has quit IRC15:40
*** mdbooth has joined #openstack-ironic15:40
*** mgould has quit IRC15:42
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [WIP] Support Linux-IO in addition to tgtd  https://review.openstack.org/23486115:46
*** daemontool_ has joined #openstack-ironic15:48
*** Sukhdev_ has joined #openstack-ironic15:49
*** lazy_prince has joined #openstack-ironic15:49
openstackgerritMerged openstack/python-oneviewclient: Manage error when Server Profile has no boot in connections Code and tests added to deal with Server Profiles without boot in the connections.  https://review.openstack.org/24811315:50
*** daemontool has quit IRC15:52
*** Piet has joined #openstack-ironic15:52
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Use Reno for release notes management  https://review.openstack.org/24867415:53
yuriyzhi Ironic the same about slow speed of spec review, welcome to all (not only cores) to review https://review.openstack.org/#/c/246965/ and https://review.openstack.org/#/c/209612/15:53
openstackgerritBrad P. Crochet proposed openstack/ironic-specs: OpenStackClient plugin spec  https://review.openstack.org/24753915:56
*** mgould has joined #openstack-ironic15:57
openstackgerritRuby Loo proposed openstack/ironic-lib: Replace deprecated LOG.warn with LOG.warning  https://review.openstack.org/24878615:58
*** meghal has joined #openstack-ironic16:00
*** foexle has quit IRC16:01
*** harshs has joined #openstack-ironic16:03
*** mitchjameson_ has joined #openstack-ironic16:03
krotscheckGood morning, ironic! Anyone out there available to review some javascript before betherly looks at things? https://review.openstack.org/#/c/228571/416:03
*** ijw has joined #openstack-ironic16:05
*** achanda_ has joined #openstack-ironic16:07
*** achanda has quit IRC16:08
*** rcernin has quit IRC16:08
rloojroll, dtantsur: wrt ironic-lib 0.4.0 (that we released last week), besides bumping upper-constraints to 0.4.0, we also have to bump it in global-requirements too, right?16:09
*** ijw has quit IRC16:09
dtantsurrloo, yes16:09
rloodtantsur: thx.16:10
jrollrloo: oh, yes we should before we land the patch to cut over. but now that patch will be tested against 0.4.0 if we want to hack on it16:10
dtantsurwe bump u-c every release and g-r every time we're going to use features from that release in other projects16:10
rloodtantsur: ok. looks like there is a bot to bump u-c. but we/I forgot to bump g-r16:11
*** meghal has quit IRC16:12
*** Pablo|off| has joined #openstack-ironic16:14
dtantsurdamn, this "no valid host found" error is a serious enough reason to stop using nova with ironic....16:14
* dtantsur is angry16:14
*** Pablo|off| is now known as pcaruana16:15
openstackgerritBrad P. Crochet proposed openstack/ironic-specs: OpenStackClient plugin spec  https://review.openstack.org/24753916:20
jrolldtantsur: what about it, just the vague error or?16:21
openstackgerritBrad P. Crochet proposed openstack/ironic-specs: OpenStackClient plugin spec  https://review.openstack.org/24753916:22
dtantsurjroll, extremely vague, no hints how to debug :)(16:22
dtantsur:(16:22
jrollyeah16:22
jrolldtantsur: fwiw, that's true for virt too, it's just easier to fail ironic builds16:22
dtantsuryeah16:22
dtantsuron a bright side: I suspect inspector IPA job fails due to low memory: I forgot to bump it to 1G like for ironic jobs16:23
dtantsurhttps://review.openstack.org/#/c/248799/16:23
*** jistr has joined #openstack-ironic16:33
sambettsdtantsur: tinyipa ;)16:34
sambettshaha16:34
dtantsursambetts, I'm all for using tinyipa instead of coreos. It's pretty possible I won't be able to add Linux-IO support for the coreos version...16:34
dtantsurI can't make the damn thing mount configfs inside the container16:35
*** mkovacik has quit IRC16:35
sambettsI'm looking forward to trying to see if linux-io works for tinyipa because it means I can ditch a package16:35
*** mgould has quit IRC16:38
*** sinval has joined #openstack-ironic16:42
dtantsursambetts, you're welcome to play with https://review.openstack.org/#/c/234861/16:43
dtantsurI'm trying to make it work with DIB builder in the meanwhile16:43
*** harshs has quit IRC16:43
sambettsdtantsur: Awesome :D16:43
*** stendulker has joined #openstack-ironic16:44
*** vishwanathj has joined #openstack-ironic16:45
dtantsurrloo, jroll, OSC spec is up: https://review.openstack.org/247539 please take a look16:46
jrollit's on my list :)16:46
rloodtantsur: thx. this week...16:46
rloomeeting in 15 minutes...16:46
dtantsurI see we have a confusion between "node delete" (removing a node from the database) and "node delete" (applying provision state "deleted")16:46
dtantsurany ideas are welcome..16:46
*** rpioso has joined #openstack-ironic16:47
*** trown is now known as trown|lunch16:49
*** subscope has quit IRC16:50
*** mgould has joined #openstack-ironic16:50
rloodtantsur: is that in the OSC spec? I seem to recall devananda using the term 'register' when doing a node-create, so maybe unregister?16:52
dtantsurrloo, using "create" and "delete" for CRUD operation is part of OSC guidelines, we should have a serious reason for breaking them..16:53
*** thiagop has joined #openstack-ironic16:53
rloodtantsur: oh, ok. i should read that spec soon...16:53
thiagopgood morning Ironic16:53
dtantsurmorning thiagop16:54
mgouldthiagop, morning16:54
*** yonglihe has joined #openstack-ironic16:54
jrollmeeting agenda looks fantastic today \o/16:54
thiagoplooks short...16:55
*** Piet has quit IRC16:55
rloothiagop: 'short' == fantastic :D16:55
thiagoplol16:55
*** sacharya has joined #openstack-ironic16:55
sambettsdtantsur: I think that 'teardown' or something like that makes sense16:56
*** cdearborn has joined #openstack-ironic16:56
dtantsur++ for tear down16:56
*** mgoddard_ has joined #openstack-ironic17:00
*** sacharya has quit IRC17:00
openstackgerritSinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Adding port mac validation tests  https://review.openstack.org/24870917:00
*** Nisha has joined #openstack-ironic17:01
jlvillalkrtaylor, Do you have update to put into the whiteboard?17:03
*** mgoddard has quit IRC17:03
openstackgerritSinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Adding port mac validation tests  https://review.openstack.org/24870917:05
krtaylorjlvillal, not really, the spec is making progress, I'll add that17:05
jlvillalkrtaylor, Thanks!17:05
*** ijw has joined #openstack-ironic17:05
*** daemontool_ has quit IRC17:06
*** daemontool has joined #openstack-ironic17:08
*** ijw has quit IRC17:10
*** ir2ivps9 has joined #openstack-ironic17:14
devanandaugh ... ran out of coffee ...17:14
*** mdbooth has quit IRC17:18
*** mitchjameson_ has quit IRC17:20
*** rcernin has joined #openstack-ironic17:21
* lucasagomes completely forgot the meeting17:23
openstackgerritVladyslav Drok proposed openstack/ironic: Add proxy related parameters in agent driver  https://review.openstack.org/23549317:24
*** jerrygb_ has quit IRC17:26
openstackgerritMerged openstack/ironic-webclient: Dynamically reconfiguring resources  https://review.openstack.org/22857117:27
openstackgerritMerged openstack/ironic-webclient: Introduced Root State  https://review.openstack.org/23164117:27
*** lazy_prince has quit IRC17:28
openstackgerritMerged openstack/ironic-webclient: Moved node list to default application state  https://review.openstack.org/23164217:28
*** aginwala has joined #openstack-ironic17:29
*** aginwala has quit IRC17:30
*** derekh has quit IRC17:31
krotscheckUrgh, forgot the meeting too.17:33
*** jerrygb has joined #openstack-ironic17:33
*** rebase has joined #openstack-ironic17:35
*** albertoffb has joined #openstack-ironic17:35
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add logs extension  https://review.openstack.org/24883217:35
*** afaranha has joined #openstack-ironic17:36
*** jerrygb has quit IRC17:36
*** davideagnello has joined #openstack-ironic17:39
*** e0ne has quit IRC17:39
*** jerrygb has joined #openstack-ironic17:40
jrollkrotscheck: devstack?17:40
krotscheckjroll: Maybe? I've got a dummy vm with the fake driver right now, but that's on HP Public cloud which is going away.17:41
jrollkrotscheck: should work. or even easier, I think bifrost has a vagrant file17:41
krotscheckCan devstack do an "Only Ironic" setup with none of that keystone nonsense?17:41
jrollwhich will only be ironic17:42
jrollheh, no17:42
jrollbifrost sounds like the thing to do17:42
lucasagomeskrotscheck, I think that not right now, needs to be worked out17:42
krotscheckjroll: Using the vagrant file will only run things locally, I need to be able to have it be publicly available so non-technical UX people in london can throw people at it.17:43
devanandakrotscheck: the vagrant file just helps set defaults for local use17:44
jrollkrotscheck: vagrant has support for various clouds as well17:44
krotscheckIt does?17:44
* krotscheck learns something new every day17:44
devanandayou can run bifrost with different inputs and get it on a public IP17:44
jrollyou could probably tweak the vagrantfile to spin it up on rackspace cloud or whatever17:45
jrollkrotscheck: for example https://github.com/mitchellh/vagrant-rackspace17:45
*** jerrygb has quit IRC17:45
*** sc68cal has quit IRC17:46
devanandakrotscheck: you just want to run ironic with the fake driver so folks can test UX things, right?17:46
krotscheckjroll: Neat! Can I have a free account on rackspace to run that? My HP cloud is going away in january :D17:46
*** nicodemos has quit IRC17:46
*** sc68cal has joined #openstack-ironic17:47
krotscheckdevananda: For now. Eventually we'll need people to test ux on a small set of known drivers, since we need to bake form validation into the client rather than get some kind of validation metrics from /drivers/properties17:47
dtantsurg'night!17:47
*** dtantsur is now known as dtantsur|afk17:47
devanandag'night, dtantsur17:47
jrollkrotscheck: https://developer.rackspace.com/17:47
krotscheckdtantsur|afk: toodles!17:48
openstackgerritSerge Kovaleff proposed openstack/ironic-inspector: Generate Sphinx docs  https://review.openstack.org/24698517:48
devanandakrotscheck: you can configure it to start other drivers, but testing those against real hardware is another story. you'd need a hardware lab for that17:48
*** stendulker has quit IRC17:49
*** achanda_ has quit IRC17:49
sambettso/ dtantsur|afk17:50
lucasagomesdtantsur|afk, g'night17:50
mgoulddtantsur|afk, gnight!17:50
*** nicodemos has joined #openstack-ironic17:50
thiagopgood night dtantsur|afk17:52
* mgould -> home: good night!17:54
thiagopnight mgould17:54
*** mgould has quit IRC17:54
*** meghal has joined #openstack-ironic17:54
devanandaback in a bit17:55
*** harshs has joined #openstack-ironic17:58
*** achanda has joined #openstack-ironic18:00
ionutbalutoiuHello guys, does this patch need anything else to be approved ? https://review.openstack.org/#/c/245911/18:00
*** mgoddard_ has quit IRC18:01
*** mgoddard has joined #openstack-ironic18:01
*** aginwala has joined #openstack-ironic18:01
lucasagomesfolks if you have sometime, mind taking a look at ? https://review.openstack.org/#/c/248153/ should be quick18:03
lucasagomesionutbalutoiu, looking now18:03
jrollionutbalutoiu: not anymore! :)18:04
krotscheckdevananda: Yeah, I'm going to talk to BadCub as soon as he's back and ask about some moonshot racks that I heard about. Might be able to turn that into a hardware lab :)18:04
*** achanda_ has joined #openstack-ironic18:04
ionutbalutoiuNice, thanks guys :)18:04
jrolllucasagomes: commented18:05
lucasagomesjroll, thanks!18:05
jrolllucasagomes: I may be missing something that translates that, idk18:05
lucasagomesjroll, yeah, the API converts from None to {}18:05
* lucasagomes finds the code18:05
lucasagomesit's the json patch type18:06
jrollah ok18:06
jrolllucasagomes: well, if the API does it18:06
lucasagomesjson type*18:06
jrollwhy does it happen here18:06
jrollthis isn't an api call18:06
*** ijw has joined #openstack-ironic18:06
jrollor is it at the objects layer18:06
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Node  https://review.openstack.org/23950618:06
*** achanda has quit IRC18:07
jrolloh, I see it, my fault18:08
*** Sukhdev_ has quit IRC18:09
lucasagomesjroll, I think it's on the json type of the db... lemme try to find the code18:09
jrolllucasagomes: I found it, it's on the field18:10
jroll+A for you18:10
lucasagomescheerios!18:10
*** Sukhdev has quit IRC18:10
*** trown|lunch is now known as trown18:11
*** ijw has quit IRC18:11
lucasagomesjroll, oh yeah "FlexibleDictField"18:11
* lucasagomes too many conversions18:11
jrollyep18:11
*** sacharya has joined #openstack-ironic18:13
*** nicodemos has quit IRC18:13
krotscheckjroll: Rackspace dev accounts require a credit card18:14
* krotscheck grumbles18:14
jrollkrotscheck: yeah, they don't limit your spend so we like to be able to bill you if you overspend18:15
jrollnothing I can change though :/18:15
openstackgerritVladyslav Drok proposed openstack/ironic: Add proxy related parameters in agent driver  https://review.openstack.org/23549318:17
*** ionutbalutoiu has quit IRC18:19
*** cdearborn has quit IRC18:19
vdrokgood night everyone18:20
*** praneshp has joined #openstack-ironic18:20
thiagopvdrok: good night. see ya18:20
*** mitchjameson has joined #openstack-ironic18:20
sinvalvdrok, good night o/18:20
*** e0ne has joined #openstack-ironic18:22
*** sacharya has quit IRC18:24
openstackgerritAndre Aranha proposed openstack/python-oneviewclient: Refactoring python-oneviewclient to use domain models of OneView as objects  https://review.openstack.org/23098818:26
*** jerrygb has joined #openstack-ironic18:27
*** ukalifon has quit IRC18:28
*** e0ne has quit IRC18:29
*** ijw has joined #openstack-ironic18:29
*** e0ne has joined #openstack-ironic18:29
*** mitchjameson has quit IRC18:32
*** ijw_ has joined #openstack-ironic18:33
*** ijw has quit IRC18:33
* devananda returns18:33
devanandakrotscheck: those are on the corp net. you won't be able to give public access to them18:35
krotscheckdevananda: That's less helpful then :/18:35
devanandakrotscheck: BadCub may have another location that is suitable for public usage, but you'd need to physically move the servers from Seattle to Ft Collins, IIRC18:36
*** aginwala has quit IRC18:36
krotscheckWell, I've been meaning to visit ft. collins :)18:36
devanandakrotscheck: good luck checking one of those chassis in your luggage :p18:36
krotscheckdevananda: Oh please, I don't check luggage. Those'll fit in an overhead compartment, right? :D18:37
*** meghal has quit IRC18:38
*** degorenko is now known as _degorenko|afk18:39
*** dprince has quit IRC18:39
*** cdearborn has joined #openstack-ironic18:40
*** wuhg has quit IRC18:41
*** jerrygb has quit IRC18:42
*** nicodemos has joined #openstack-ironic18:43
*** meghal has joined #openstack-ironic18:45
devanandakrotscheck: suuuuure18:45
krotscheckMaybe I should just fill a stationwagon with USB sticks.18:46
*** jerrygb has joined #openstack-ironic18:46
*** lazy_prince has joined #openstack-ironic18:46
*** boris-42 has quit IRC18:48
lucasagomesfolks, calling it a day! Have a great night everyone18:51
*** jerrygb has quit IRC18:51
*** ijw has joined #openstack-ironic18:52
*** ijw_ has quit IRC18:52
*** spandhe has joined #openstack-ironic18:52
*** daemontool has quit IRC18:54
*** electrofelix has quit IRC18:57
*** lucasagomes is now known as lucas-dinner18:57
openstackgerritMerged openstack/ironic: Add clean up method for the DHCP factory  https://review.openstack.org/24591118:57
openstackgerritMerged openstack/ironic: Fix: Next cleaning hangs if the previous cleaning was aborted  https://review.openstack.org/24815319:00
*** pas-ha has quit IRC19:02
*** lazy_prince has quit IRC19:02
*** jerrygb has joined #openstack-ironic19:03
*** Nisha has quit IRC19:09
*** Nisha has joined #openstack-ironic19:09
openstackgerritAndre Aranha proposed openstack/python-oneviewclient: wRefactoring python-oneviewclient to use domain models of OneView as objects  https://review.openstack.org/23098819:19
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Switched power state and provision state.  https://review.openstack.org/24888419:21
openstackgerritMario Villaplana proposed openstack/ironic-specs: Add notification support to Ironic  https://review.openstack.org/24888519:22
mariojvlucas-dinner: there's the initial spec for notifications in ironic ^ feel free to make changes + add yourself as co-author19:24
openstackgerritAndre Aranha proposed openstack/python-oneviewclient: Refactoring python-oneviewclient to use domain models of OneView as objects  https://review.openstack.org/23098819:24
mariojvi think the event_type and payload definitions are the sections that may need the most feedback19:25
*** jcoufal has quit IRC19:27
JayFjroll: around? Can you spare a few minutes either now || this afternoon to work on Rescue spec/19:35
*** aginwala has joined #openstack-ironic19:39
*** mitchjameson has joined #openstack-ironic19:44
*** aginwala has quit IRC19:45
*** ionutbalutoiu has joined #openstack-ironic19:45
openstackgerritJay Faulkner proposed openstack/ironic-specs: Implement rescue mode  https://review.openstack.org/17187819:46
JayFmariojv: jroll ^ would appreciate your eyes there. Still work to do though.19:48
*** aginwala has joined #openstack-ironic19:54
*** mitchjameson has quit IRC19:55
*** e0ne has quit IRC19:55
*** cdearborn has quit IRC19:56
*** ijw has quit IRC19:56
*** dims has joined #openstack-ironic19:56
*** ijw has joined #openstack-ironic19:56
*** ijw_ has joined #openstack-ironic19:57
*** ijw has quit IRC20:00
*** achanda_ has quit IRC20:04
devanandaMattMan: around?20:05
*** ionutbalutoiu has quit IRC20:06
*** cdearborn has joined #openstack-ironic20:12
zer0c00lMorning20:17
zer0c00lping jroll JayF20:20
zer0c00lhttps://review.openstack.org/#/c/191505/ Finally got the build timeouts sorted20:21
zer0c00lHad to rewrite the shutdown logic20:21
zer0c00lLet me know if you have any comments20:21
*** jistr has quit IRC20:23
*** baoli has quit IRC20:27
*** meghal has quit IRC20:34
*** e0ne has joined #openstack-ironic20:42
*** baoli has joined #openstack-ironic20:46
*** baoli has quit IRC20:47
*** baoli has joined #openstack-ironic20:47
*** albertoffb has quit IRC20:49
jrollJayF: yeah, I can help out this afternoon20:51
jrollzer0c00l: woo20:51
*** baoli has quit IRC20:52
*** baoli has joined #openstack-ironic20:55
*** baoli has quit IRC20:56
*** baoli has joined #openstack-ironic20:57
*** spandhe has quit IRC21:01
*** spandhe has joined #openstack-ironic21:04
*** Nisha has quit IRC21:07
*** e0ne has quit IRC21:08
*** meghal has joined #openstack-ironic21:10
*** krotscheck has quit IRC21:10
*** baoli has quit IRC21:11
*** anteaya has quit IRC21:11
*** baoli has joined #openstack-ironic21:11
*** jasondotstar has quit IRC21:12
*** baoli has quit IRC21:12
*** baoli has joined #openstack-ironic21:12
*** krotscheck has joined #openstack-ironic21:17
mrdaMorning Ironic21:17
*** e0ne has joined #openstack-ironic21:17
thiagopmorning mrda21:18
*** jasondotstar has joined #openstack-ironic21:18
mrdahey thiagop21:19
*** jerrygb has quit IRC21:22
*** baoli has quit IRC21:22
BadCubkrotscheck: I have a pending equipment order that will fill the Ironic rack. FtC is pretty much the best option for public connection though21:24
*** anteaya has joined #openstack-ironic21:26
*** jerrygb has joined #openstack-ironic21:26
*** bradjones has quit IRC21:27
*** bradjones has joined #openstack-ironic21:29
*** bradjones has quit IRC21:29
*** bradjones has joined #openstack-ironic21:29
rloomorning mrda21:30
*** mjturek1 has quit IRC21:30
mrdahi rloo21:30
rloojroll: if you have a minute. wrt doing manual cleaning when node is in maintenance. You're not OK with it? https://review.openstack.org/#/c/247695/21:31
*** jerrygb has quit IRC21:31
*** david-lyle has quit IRC21:31
jrollrloo: I don't see why we would, we don't touch the nodes in any other way when in maint mode21:31
rloojroll: well, i figured it is manual clean so the operator knows what they're doing.21:32
jrollit's in manageable so it isn't like it would be deployed to21:32
rloojroll: i commented.21:32
jrollrloo: as someone that operates ironic and has no idea what I'm doing, I disagree :D21:32
rloojroll: what if the node is in maintenance and they want to do some magic cleaning.21:32
rloojroll: ok, i'll go with reason 2. Yahoo wants it.21:32
jrolltake it out of maintenance?21:32
jrolllol21:33
jrollbut why21:33
rloojroll: yahoo lets the user 'put' a node in maintenance if the user thinks there is something wrong with it21:33
JayFrloo: here's the traditional objection to that21:33
rloojroll: i guess yahoo wants to do some cleaning but don't want to lose the fact that the user put it in maintenance. until they've verified that things are OK21:33
jrollhm21:33
*** krtaylor has quit IRC21:33
JayFrloo: I'm an operator, I want to replace a widget in the node, so I maintenance it. Operator #2 calls manual-clean-firmware-update21:33
rlooJayF: Oh. didn't know there was a tradition of this.21:33
JayFrloo: and now I've broken a machine :)21:34
JayFrloo: well just saying the traditional argument about this is the "scary firmware upgrade"21:34
*** david-lyle has joined #openstack-ironic21:34
JayFrloo: and the idea that "maintenance" is the only way to indicate a hardware maintenance21:34
Madasithe line between maintenance and managed is very blurry21:35
rlooso we either protect from operators maybe doing something stupid cuz they didn't notice/read why something is in maintenance...21:35
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Node  https://review.openstack.org/23950621:35
rlooJayF, jroll: ok, so I can change that patch to not allow for maintenance and then sic some yahoo folks on you.21:35
JayFI mean, I'm on the fence about maint mode and always have been21:36
JayFfor us the painful piece is when a node goes into maint due to power status loop or something else Ironic-initiated21:36
rloowell, i'm not an ops person so I don't really know how maint is used21:36
JayFthere's no automated way in Ironic to "retrieve" the node from maintenance21:36
Madasirloo: maintenance mode is generally used like a giant pause button for the state machine on that node21:37
MadasiIMO21:37
JayFMadasi: Well, except you can delete nodes in maintenance.21:37
JayFMadasi: they will 'clean fail', but you can delete them21:37
Madasinodes or instances?21:37
JayFthe instance will delete21:37
jrollthey don't clean fail, they heartbeat until you remove maint mode21:37
*** sinvalv has joined #openstack-ironic21:37
JayFoooooh21:37
Madasiyeah21:37
*** mkovacik has joined #openstack-ironic21:37
JayFthat's even more sensible21:38
openstackgerritMatthew Treinish proposed openstack/ironic: Workaround eventlet bug with subunit on python 3  https://review.openstack.org/22943621:38
*** baoli has joined #openstack-ironic21:38
rloohmm, so it seems like it kind of depends on how maintenance mode is being used.21:38
jrollI'm still conflicted on this but still leaning toward "not allowing it in maint mode"21:38
*** trown is now known as trown|outttypeww21:39
jrollrloo: also in a meeting so you don't have my full attention right now :(21:39
*** sinval has quit IRC21:39
Madasimanaged state really seems like a better fit, but it forces a node through full cleaning on the way out b/c you can only transition to available from there21:39
*** Sukhdev has joined #openstack-ironic21:40
rloojroll, JayF, Madasi: for now, I'll change it to not allow manual cleaning in maintenance. Seems like something we should discuss and I don't think we need to hold up manual cleaning cuz of this.21:41
jrollrloo: yeah, agree, I think this is a bigger conversation21:42
thiagopfolks, I'm calling it a day21:43
*** dims_ has joined #openstack-ironic21:43
thiagopsee ya21:43
Madasimaking it consistent with existing behavior sounds like a good short term path, with a conversation to be held down the road21:43
*** thiagop has quit IRC21:43
*** e0ne has quit IRC21:43
JayFI think the idea of manual cleaning nodes in maint is interesting ... but only if we could make them unmaint on success21:44
JayFthen we'd finally have a lever in ironic for "automation" of getting nodes out of maintenance21:44
Madasithat's messy too21:44
* JayF invisions a manual cleaning run doing troubelshooting steps21:44
*** dims has quit IRC21:44
Madasiif you are manually cleaning, you may want to manually confirm success as well21:44
Madasibut it's situation specific21:44
spandherloo: jroll , I agree that we dont need to hold the current patch. But I would like to explain our use case once in detail21:45
*** Piet has joined #openstack-ironic21:45
jrollspandhe: feel free, I'm in a meeting but will read21:45
rloospandhe: i think it would be good to describe that, but i think it might be worth doing in email for folks that aren't 'here' right now.21:46
spandherloo: sure21:46
rloospandhe: i would be interested in hearing from others about how they are/might be using maintenance mode. your email may 'draw' them out :)21:47
*** krtaylor has joined #openstack-ironic21:47
spandherloo: cool21:47
*** david-lyle has quit IRC21:55
*** harshs has quit IRC21:58
*** harlowja_ has joined #openstack-ironic21:59
*** harlowja has quit IRC21:59
*** krtaylor has quit IRC22:03
*** david-lyle has joined #openstack-ironic22:03
*** david-lyle has quit IRC22:04
*** harshs has joined #openstack-ironic22:05
*** jerrygb has joined #openstack-ironic22:11
*** Pavo_ has joined #openstack-ironic22:13
*** thrash is now known as thrash|g0ne22:13
*** krtaylor has joined #openstack-ironic22:15
*** mjturek1 has joined #openstack-ironic22:15
*** Pavo has quit IRC22:16
*** aginwala_ has joined #openstack-ironic22:16
*** Pavo_ is now known as Pavo22:17
*** Pavo has quit IRC22:18
*** rcernin has quit IRC22:20
*** aginwala has quit IRC22:20
*** aginwala_ has quit IRC22:21
*** david-lyle has joined #openstack-ironic22:21
*** aginwala has joined #openstack-ironic22:21
*** david-lyle has quit IRC22:25
*** lucas-dinner has quit IRC22:36
*** athomas has quit IRC22:41
*** lucasagomes has joined #openstack-ironic22:41
*** lucasagomes is now known as lucas-afk22:42
*** sambetts is now known as sambetts-afk22:45
openstackgerritRuby Loo proposed openstack/ironic: Agent vendor handles manual cleaning  https://review.openstack.org/24769522:51
*** cdearborn has quit IRC22:52
*** dims_ has quit IRC22:55
*** amotoki has quit IRC22:58
*** david-lyle has joined #openstack-ironic23:03
*** david-lyle has quit IRC23:05
*** HimanshuGarg has quit IRC23:07
*** Marga_ has joined #openstack-ironic23:14
*** mkovacik has quit IRC23:17
*** david-lyle has joined #openstack-ironic23:23
*** dims_ has joined #openstack-ironic23:25
*** david-lyle has quit IRC23:25
*** dims_ has quit IRC23:26
*** davideagnello has quit IRC23:27
*** davideagnello has joined #openstack-ironic23:29
*** davideagnello has quit IRC23:31
*** davideagnello has joined #openstack-ironic23:35
*** david-lyle has joined #openstack-ironic23:35
*** rpioso has quit IRC23:35
*** meghal has quit IRC23:40
*** meghal has joined #openstack-ironic23:40
*** david-lyle has quit IRC23:41
*** david-ly_ has joined #openstack-ironic23:41
*** sacharya has joined #openstack-ironic23:55
*** aginwala has quit IRC23:56
*** aginwala has joined #openstack-ironic23:58

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