Monday, 2016-08-01

« Sunday, 2016-07-31 Index Tuesday, 2016-08-02 »
*** baoli has joined #openstack-ironic00:01
*** sdake has joined #openstack-ironic00:13
*** sdake has quit IRC00:17
*** achanda has quit IRC00:20
*** harshs has quit IRC00:22
*** mbound has joined #openstack-ironic00:26
*** chlong has joined #openstack-ironic00:26
*** chlong has quit IRC00:30
*** mbound has quit IRC00:31
*** harshs has joined #openstack-ironic00:35
*** harshs has quit IRC00:36
*** hoangcx has joined #openstack-ironic00:40
*** baoli has quit IRC00:40
*** chlong has joined #openstack-ironic00:41
*** baoli has joined #openstack-ironic00:45
*** hoangcx_ has joined #openstack-ironic00:56
*** hoangcx_ has quit IRC00:56
*** hoangcx has quit IRC00:57
*** baoli has quit IRC00:59
*** chlong has quit IRC01:10
*** hoangcx has joined #openstack-ironic01:17
*** chlong has joined #openstack-ironic01:23
*** bvandewa has joined #openstack-ironic01:24
*** chlong has quit IRC01:28
*** chlong has joined #openstack-ironic01:30
*** lucasagomes has quit IRC01:32
*** bvandewa has quit IRC01:38
*** lucasagomes has joined #openstack-ironic01:39
*** moshele has joined #openstack-ironic01:44
*** tangchen_ has joined #openstack-ironic01:58
*** tangchen has joined #openstack-ironic01:58
*** rama_y has quit IRC01:59
*** harshs has joined #openstack-ironic02:24
*** harshs has quit IRC02:30
openstackgerritTeng Fei proposed openstack/ironic-inspector: Modify the SQL's string from double quote to single quote.  https://review.openstack.org/34779502:41
*** baoli has joined #openstack-ironic03:01
*** baoli has quit IRC03:06
*** yuanying has quit IRC03:08
*** amotoki has joined #openstack-ironic03:11
*** amotoki has quit IRC03:28
*** hoangcx has quit IRC03:30
*** harshs has joined #openstack-ironic03:32
*** appu103_ has joined #openstack-ironic03:33
*** appu103_ is now known as aparnav03:33
*** hoangcx has joined #openstack-ironic03:36
*** amitkqed has quit IRC03:38
*** amitkqed has joined #openstack-ironic03:38
*** hoangcx_ has joined #openstack-ironic03:39
*** hoangcx has quit IRC03:40
*** achanda has joined #openstack-ironic03:54
*** amotoki has joined #openstack-ironic03:57
*** achanda has quit IRC04:00
*** ljjjustin has quit IRC04:03
*** amotoki_ has joined #openstack-ironic04:06
*** ljjjustin has joined #openstack-ironic04:06
*** amotoki has quit IRC04:09
openstackgerritTang Chen proposed openstack/python-ironicclient: Fix i18n problems in shell.py  https://review.openstack.org/34934304:10
*** chlong has quit IRC04:15
*** amotoki_ has quit IRC04:17
*** amotoki has joined #openstack-ironic04:31
*** chlong has joined #openstack-ironic04:31
*** harshs has quit IRC04:37
*** achanda has joined #openstack-ironic04:57
*** deray has joined #openstack-ironic05:09
openstackgerritTang Chen proposed openstack/python-ironicclient: Fix i18n problems in shell.py  https://review.openstack.org/34934305:14
*** e0ne has joined #openstack-ironic05:27
*** e0ne has quit IRC05:41
*** e0ne has joined #openstack-ironic05:44
*** e0ne has quit IRC05:47
*** e0ne has joined #openstack-ironic05:51
*** vishwanathj has joined #openstack-ironic05:51
*** e0ne has quit IRC05:51
*** vishwanathj has quit IRC05:55
*** mjura has joined #openstack-ironic06:09
*** vishwanathj has joined #openstack-ironic06:21
*** hshiina has joined #openstack-ironic06:34
*** ohamada has joined #openstack-ironic06:35
*** ohamada has quit IRC06:37
*** ohamada has joined #openstack-ironic06:38
*** bks has joined #openstack-ironic06:43
*** edand has joined #openstack-ironic06:44
*** tesseract- has joined #openstack-ironic06:44
*** wlfightup has joined #openstack-ironic06:44
*** wlfightup has left #openstack-ironic06:45
*** wlfightup has joined #openstack-ironic06:45
wlfightup?06:45
*** loki___ has joined #openstack-ironic06:46
*** daemontool_ has joined #openstack-ironic06:53
*** chlong has quit IRC06:59
*** rbartal has joined #openstack-ironic07:01
*** moshele has quit IRC07:01
*** tiendc has joined #openstack-ironic07:06
*** slagle has quit IRC07:06
openstackgerritTang Chen proposed openstack/python-ironicclient: Fix i18n problems in shell.py  https://review.openstack.org/34934307:07
*** vsaienk0 has quit IRC07:08
*** stevebaker has quit IRC07:09
*** chlong has joined #openstack-ironic07:11
*** vsaienk0 has joined #openstack-ironic07:12
*** ccamacho has joined #openstack-ironic07:17
*** edand has quit IRC07:23
*** bvandewa has joined #openstack-ironic07:26
*** chlong has quit IRC07:34
*** Sukhdev has joined #openstack-ironic07:37
*** Sukhdev has quit IRC07:46
*** edand has joined #openstack-ironic07:46
*** phuongnh has joined #openstack-ironic07:53
*** e0ne has joined #openstack-ironic07:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
openstackgerritZhenguo Niu proposed openstack/ironic-inspector: Check whether action value is string before calling format()  https://review.openstack.org/34939208:01
*** achanda_ has joined #openstack-ironic08:05
*** achanda has quit IRC08:07
*** e0ne has quit IRC08:07
*** yuanying has joined #openstack-ironic08:08
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
*** e0ne has joined #openstack-ironic08:15
openstackgerritDao Cong Tien proposed openstack/ironic: Follow-up patch to review 293873  https://review.openstack.org/34940008:22
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643208:26
*** e0ne has quit IRC08:28
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Update devstack plugin with new auth options  https://review.openstack.org/29731208:31
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions  https://review.openstack.org/23698208:31
*** mbound has joined #openstack-ironic08:35
*** Romanenko_K has joined #openstack-ironic08:36
*** e0ne has joined #openstack-ironic08:40
*** bvandewa has quit IRC08:44
*** deray_ has joined #openstack-ironic08:46
*** deray has quit IRC08:49
*** bks has quit IRC08:51
*** achanda_ has quit IRC08:52
*** milan has joined #openstack-ironic08:52
*** deray has joined #openstack-ironic08:54
*** deray_ has quit IRC08:54
milanmorning Ironic!08:54
*** deray_ has joined #openstack-ironic08:59
*** deray has quit IRC08:59
*** deray has joined #openstack-ironic09:01
*** deray_ has quit IRC09:01
openstackgerritTang Chen proposed openstack/ironic: Split API root unit and functional tests  https://review.openstack.org/34896509:02
openstackgerritTang Chen proposed openstack/ironic: Prepare for functional tests  https://review.openstack.org/34942109:02
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Deny some driver_info updates when console is on  https://review.openstack.org/33000309:02
sambetts|afkMorning all09:03
tangchenHi guys, morning. :)09:04
tangchenToday we have an IRC meeting, right ?09:04
sambetts|afktangchen: yes the Ironic meeting is at 5pm UTC09:07
tangchen<milan>, <sambetts|afk>: Hi, guys, are you going to attend today's meeting ? I don't think I can attend it because of the timezone issue. So I'd like you guys to help to review this patches and the related RFEs.09:08
tangchenhttps://review.openstack.org/#/c/349421/09:08
tangchenhttps://review.openstack.org/#/c/348965/09:08
milanmorning tangchen, sambetts|afk :)09:09
milantangchen, I plan to attend09:09
* milan looks into the patches09:09
tangchenI'm working on functional tests in Ironic. I'd love to know what kind of functional tests you want at last. And please tell me if I'm going to the wrong way.09:10
*** sambetts|afk is now known as sambetts09:11
*** hoangcx_ is now known as hoangcx09:16
*** Goneri has joined #openstack-ironic09:16
*** vishwanathj has quit IRC09:22
openstackgerritSam Betts proposed openstack/ironic-specs: Add spec for IPA API versioning support  https://review.openstack.org/34108609:23
*** dtantsur|pto is now known as dtantsur09:23
dtantsurMorning Ironic :)09:23
sambettso/ dtantsur09:23
*** hparekh has quit IRC09:24
* dtantsur pretends that his IRC client does not support scrollback :D09:25
sambetts;)09:25
sambettsdtantsur: what do you think of "Ironic" vs "ironic" in "Ironic Python Agent"?09:26
dtantsurlike in "ironic Python Agent" or like in "ironic python agent"?09:26
dtantsurfwiw, I've always been against lower-casing ironic, but that's an official policy09:27
sambettsyeah :/09:27
*** tangchen has quit IRC09:27
sambettsI like writing it as Ironic Python Agent, because it lines up with the acronym IPA09:27
dtantsuryeah09:28
dtantsurI also tend to write Ironic everywhere except for our docs09:28
*** tangchen_ has quit IRC09:28
sambettsbut I got a comment on one of my specs to s/Ironic/ironic but I don't know if that should affect the Ironic Python Agent or just were I talk about Ironic by itself09:28
dtantsurgood question, I'd raise to a bigger audience on the ML09:29
dtantsurit also concerns Ironic Inspector vs ironic inspector vs ironic-inspector09:29
milandtantsur, sambetts I'd say Ironic is a name ironic is a quality when I see it written09:30
milanI'd vote for Ironic09:30
milandtantsur +1, it reminds me of an email to ML where someone suggested renaming ironic inspector to Clouseau ;)09:31
sambetts:-P09:31
dtantsurlol, we should have done it09:32
milanalso, I think "Good morning, Ironic" vs "Good morning, ironic" might send a different message ;)09:32
dtantsurbut it was a bit too later at that point09:33
dtantsurlol, also true09:33
milandtantsur, yeah, we can rename it once HA lands ;)09:33
*** deray has quit IRC09:33
*** deray_ has joined #openstack-ironic09:33
*** hshiina has quit IRC09:33
openstackgerritZhenguo Niu proposed openstack/ironic: Add nodes tagging support - objs  https://review.openstack.org/23335709:33
openstackgerritZhenguo Niu proposed openstack/ironic: Add DB layer support for node tags filter  https://review.openstack.org/27262409:34
*** hparekh has joined #openstack-ironic09:37
sambettsdtantsur: is it OpenStack policy to write Ironic as ironic/09:39
sambetts?09:39
sambettsor Ironic's polocy?09:39
dtantsurOpenStack's09:40
sambettsdtantsur: seems like the lower case policy is a blanket thing then looking at the https://governance.openstack.org/reference/service-project-naming.html#project-name-guidelines I think we'd need to be ironic inspector and ironic python agent :/09:42
dtantsurmeh09:43
sambettsyeah :/09:44
*** tangchen has joined #openstack-ironic09:46
*** tangchen_ has joined #openstack-ironic09:46
*** achanda has joined #openstack-ironic09:52
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643209:52
vsaienk0Morning sambetts, dtantsur, milan and all Ironic'ers!09:55
milanmorning vsaienk0 :)09:55
*** achanda has quit IRC09:58
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Update devstack plugin with new auth options  https://review.openstack.org/29731210:09
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions  https://review.openstack.org/23698210:09
*** hoangcx has quit IRC10:09
*** bks has joined #openstack-ironic10:13
openstackgerritYuiko Takada Mori proposed openstack/ironic: Add node serial console installation howto  https://review.openstack.org/29387210:19
*** tangchen has quit IRC10:25
*** tangchen_ has quit IRC10:25
*** mgould|afk is now known as mgould10:42
mgouldmorning Ironic!10:42
vdroklate morning everyone!10:43
milanmorning mgould and vdrok ! :)10:43
sambettsdtantsur: mail sent10:43
*** tiendc has quit IRC10:53
dtantsurcool10:56
mgouldmorning vdrok milan sambetts10:59
openstackgerritMerged openstack/ironic-python-agent: Use ironic_lib's execute()  https://review.openstack.org/34802311:01
*** phuongnh has quit IRC11:05
vsaienk0morning vdrok11:07
mgouldmorning vsaienk011:07
vsaienk0morning mgould11:08
vdrokmorning milan mgould and vsaienk0 :)11:08
*** Goneri has quit IRC11:11
*** deray_ has quit IRC11:12
*** tangchen has joined #openstack-ironic11:14
*** tangchen_ has joined #openstack-ironic11:14
*** Goneri has joined #openstack-ironic11:16
*** jcoufal has joined #openstack-ironic11:20
jrollmorning y'all11:21
mgouldmorning jroll11:21
jrolldtantsur: sambetts: shall we figure out this network_interface thing?11:21
jrolltl;dr nova needs to know which network interface is in use, None is not an option (since nova doesn't know the default)11:21
dtantsurjroll, definitely, even though I don't remember where exactly we finished :)11:22
dtantsurooooooooooooouuuch11:22
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic port-list command  https://review.openstack.org/32906811:22
dtantsurthen it has to become a mandatory interface, I guess11:22
dtantsurand we have to bring back magical auto-filling of old values11:22
jroll'mandatory' meaning it always needs to be set?11:23
jrolland if it's not set, error out or set the default?11:23
openstackgerritYuiko Takada Mori proposed openstack/ironic-specs: Follow-up patch of 75290c72f7c3138bfdab151728718f771dfb9685  https://review.openstack.org/33537811:24
milanmorning jroll ! :)11:27
*** mat128|pto is now known as mat12811:27
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/34862411:27
mat128good morning Ironic :D11:27
milanmorning mat128! :)11:28
jrolldtantsur: so you're good with that? see my last question?11:28
dtantsurjroll, well, no11:29
dtantsurjroll, we can't make it mandatory in this sense because of our beloved API versioning. we have to apply magic to convert None -> something11:29
jrolldtantsur: it hasn't been long, I say we just fix that api version11:30
dtantsur1. we should not do that, 2. I'm talking about old API versions which cannot pass network_interface even in theory, 3. this is a breaking change and we promised to devananda to not break node-create any more :)11:31
dtantsur(kidding about promising, but he was actively objecting to the breakign enroll change)11:31
dtantsurjroll, ^^^11:31
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Verify JSON response of driver commands  https://review.openstack.org/33660011:31
*** slagle has joined #openstack-ironic11:31
TheJuliagood morning11:31
dtantsurmorning TheJulia11:32
jrolldtantsur: if we use the default we have right now to set it at create time, it isn't so breaking11:32
jrolllike, at node-create time, find the default and set it11:32
dtantsurright, I was referring to the "error out" option11:33
dtantsursorry, too many emails to read :(11:33
jrollah11:33
jrollfor existing nodes, I want to say write it at first read, but that first node-list would be insane11:33
openstackgerritMerged openstack/ironic: Fix rendering for version 1.14  https://review.openstack.org/34708411:33
dtantsurjroll, I think we did the same for NONE -> AVAILABLE, no?11:34
jrolldtantsur: kind of, we wrote that at first *write* for the node11:35
*** jcoufal has quit IRC11:35
dtantsurwell, we can do the same here, right?11:35
jrollbut nova still needed to be able to read the old one11:35
jrolland since None means "use the config option", I'm not so sure11:35
*** jcoufal has joined #openstack-ironic11:35
dtantsurmmmmm, right11:35
TheJuliaand the logic needs to fundimentally remain unchanged to allow a clean upgrade and the same behavior...11:36
dtantsurso can we end up with always silently replacing None -> <config> on the way to the user, but not changing anything in the database?11:36
jrollwe could, I think?11:36
jrolldo we expect that config to be set for api servers?11:36
TheJuliaI believe so yes11:36
jrollrephrased, do we mandate that is set in the API level11:37
jrollI think we do11:37
TheJuliaI think it would need to11:37
dtantsurit's not impossible11:37
dtantsurI suspect most of deployers do not split ironic.conf, do they?11:37
mgouldmorning TheJulia11:37
jrollprobably not11:37
jrolloh, we brought in enabled_network_interfaces, not default_11:38
*** dprince has joined #openstack-ironic11:38
dtantsuryeah, I don't think we have a decision on the defaults11:38
dtantsurwhich delays the driver composition fwiw11:39
jrollwell, I think this kind of shows how this way works in practice yes?11:39
jrollso perhaps we want to do the same here11:39
mat128morning TheJulia, jroll, dtantsur and mgould11:39
dtantsurmorning mat12811:40
mgouldmorning mat12811:40
dtantsurjroll, define "the same" please :) do we provide configuration defaults for *all* interfaces? only some of them?11:40
dtantsurwhich takes preference: vendor default or configuration?11:40
jrollbut we need to unblock this nova side of networking first, so... 1) return calculated default instead of None in the API, 2) set calculated default during node-create, 3) existing nodes set calculated default on first write of the node11:41
jrolldoes that make sense?11:41
TheJuliaoperator configuration would be the local default11:41
* jroll doesn't want to get into the rest of the interfaces stuff right now, networking is on a time crunch11:41
TheJuliajroll: ++11:41
dtantsurjroll, ++11:41
dtantsurjroll, unrelated thing: was there any reason to not approve https://review.openstack.org/#/c/333964/ ?11:41
jrollso that plan sounds good? I'll email out a note and hack on it now11:41
TheJuliathe only change would really be to save if not already saved as a value then?11:42
dtantsurjroll, re the plan: what if the configuration option changes? are we ok with some nodes left as None (and getting new values), some nodes left with an old value11:42
dtantsur?11:42
jrolldtantsur: I don't remember if I wanted more eyes or what, I could approve it now11:42
jrolldtantsur: yeah, that's the weird part, idk11:43
openstackgerritSam Betts proposed openstack/ironic-specs: Add spec for IPA API versioning support  https://review.openstack.org/34108611:43
dtantsurjroll, so my suggestion would be to never update the database then11:43
dtantsurso database will contain None as "use default", and the default will always be calculcated on fly11:44
jrolldtantsur: how does an operator find out that something is None vs default, if we always return default?11:44
dtantsurjroll, why should they?11:44
TheJuliaI believe that was what was previously agreed upon11:45
TheJuliaan operator would need to choose to move it off the default, or choose to have an entire environment ready to go for tenant networking out of the gate11:45
jrolldtantsur: if I want to change the default, I need to find which nodes are using default vs have 'flat' written11:45
dtantsurmmmm11:46
dtantsurjroll, your initial plan has the same problem fwiw. nobody knows how many nodes were already updated11:46
TheJuliaThat is a good point, perhaps a log entry listing "default x setting  applies to: [nodes]11:47
jrolldtantsur: yeah, we'd need to include a script that makes sure they're all written to db11:47
sambettsisn't that what the migrations do?11:48
jrollor maybe we admit this can't be perfect and try to get as close as possible11:48
dtantsurjroll, ++ for migrations then11:48
jrolloh, this is something we could do in a db migration eh11:48
TheJuliaexcept if the operator changes the default and we base the default upon the migration, then we've possibly killed their ironic deployment11:48
TheJuliaif it is wrong11:49
sambettsdidn't we have the migrations read the config file though?11:49
jrollTheJulia: not sure I follow11:49
sambettsso it'd pick up the opertators default11:49
dtantsursambetts, ironic-dbsync requires configuration file, yes11:49
TheJuliajroll: deploy only supports flat, operator goes to upgrade, pushes new config files with changed default, we migrate the db, they lack all of the config required to carry forth nodes in the new different default11:50
jrollTheJulia: hopefully they'll read the release note and not set a new default, then :)11:50
TheJuliajroll: lol11:50
* jroll will write a good one, promise11:51
TheJuliarealistically, we need to provide visibility, not try and make decisions11:51
jrollyeah11:51
TheJuliaon what the operator would want11:51
TheJuliathey know their needs best, we simply, don't.11:51
dtantsurisn't it what the configuration option is for?11:51
*** zhenguo has quit IRC11:51
TheJuliaYeah, but how many people have you heard mixing up upgrade orders?11:52
jrollwell, if we don't set network_interface, nova won't work right11:52
dtantsurwell.. every time we add a non-trivial upgrade procedure, it has to be followed11:52
jrollwe can't fix people doing upgrades wrong11:52
dtantsurI would love to avoid doing it, but I don't see how11:53
TheJuliaNo, we can't fix people from doing upgrades wrong, but we can help prevent them from doing too much damage if they do11:53
jrollso what's the alternative?11:54
TheJuliawe provide operator visibility into nodes running on interpretted defaults11:54
TheJuliafrom there, they can choose to do something11:55
dtantsurso that's the question: how do we do it?11:55
sambettshow does that work from services like nova reading the node though? We need to output something that can be switched on11:55
*** achanda has joined #openstack-ironic11:56
TheJuliaeasiest would be to dump something to the logs upon conductor startup11:56
jrollright, how does nova handle that11:56
TheJuliawe provide our pre-existing default to nova11:56
TheJuliausing the new standard, so there is no change in behavior unless it is chosen to be different11:56
jrollright, so the problem I have with that is an operator can't know without log-diving. what about a node in maintenance or some state where it won't be used, when do we log that? etc11:57
sambettsas someone using the ironic client if I do node-show xxxx, I see None?11:57
sambettsthats not a great user expereince to know whats going on11:58
TheJuliajroll: then we make it an api call to search or reveal11:58
sambettsisn't that node-show?11:58
TheJuliawell, we have to remember we have users and operators, users have no clue as long as it just works, and this is all concerns in the operators realm11:58
TheJuliathe default on the object masks the None11:59
TheJuliain the DB11:59
TheJuliaso node show presently returns whatever the interpretted default or set default is if no other setting is set on the node11:59
TheJuliaThe only other clean way to do it is to get rid of the intepretted default and just default migrate everyone to flat on an upgrade and ensure there is sufficent doc and information for them to choose to head in their desired direction, since they would already have had to do that anywya12:00
TheJulias/anywya/anyway/12:00
jrollnode-show presently returns NULL if that's what is in the DB12:00
jrollbut, I'd be fine just always setting 'flat' in the DB migration12:01
sambettsthe problem is the standalone case, e.g. Flat vs No-OP12:01
TheJuliauhhh....  did it get changed in the last week or two?  I thought it would return noop12:01
*** achanda has quit IRC12:01
jrollafaik if None is set in the db, that's what is returned, which is the concern here12:01
sambettsso we need to have an interpreted default in the migration for people migrating in a standalone env12:01
jrollif it always returns flat or noop then we're good12:01
sambettsyeah, its the fact that node-show returns None thats the problem as I understand it12:02
sambettsbecause nova can't do anything with that12:02
*** sdake_ has joined #openstack-ironic12:02
* TheJulia fixes her conductor real quick so she can verify this on new node create12:03
* jroll restacks12:05
*** amotoki has quit IRC12:05
TheJuliaugh, I could have sworn this was returning the the interpretted default... :(12:06
jrollno, that's the entire problem here :/12:06
TheJuliaYeah, I get that now :(12:06
*** jcoufal_ has joined #openstack-ironic12:07
*** bfournie has quit IRC12:08
*** jcoufal has quit IRC12:10
jrollok, sent a mail with the plan as I see it, speak now or forever hold your peace12:10
TheJuliaI guess we could assert defaults upon the first provision state request12:13
TheJuliaor any node update12:14
jrollbut, then it may be unusable for scheduling until that happens12:15
TheJuliaI was fine with a database migration earlier on before we tried to keep default as none.  Only additional thing is the description for the default setting will need to be updated12:15
TheJuliafor an older api version, possibly if it is posted in a single post operation12:16
TheJuliaand even then, node creation is purely an api side operation12:16
TheJuliaso the api would need to be aware of the default settings and apply them upon creation12:17
* TheJulia is just rambling outloud12:17
*** tangchen_ has quit IRC12:17
*** tangchen_ has joined #openstack-ironic12:18
*** MattMan has quit IRC12:19
*** MattMan has joined #openstack-ironic12:19
dtantsurjroll, the plan lgtm12:20
*** rbudden has joined #openstack-ironic12:20
TheJuliajroll: ditto12:20
sambettsditto12:21
jrollcool12:21
*** amotoki has joined #openstack-ironic12:21
* jroll hackhackhack12:21
dtantsurjroll, do we have a chance of removing -2 from the nova patch?12:22
*** tangchen_ has quit IRC12:22
*** tangchen has quit IRC12:23
jrolldtantsur: it's possible, if we fix this stuff12:24
jrollnot certain yet12:24
dtantsurI see12:24
*** hoangcx has joined #openstack-ironic12:25
openstackgerritAparna proposed openstack/ironic: Support 'hardware_disk_erase' as a cleaning step  https://review.openstack.org/34948412:25
*** loki___ has quit IRC12:26
*** bks has quit IRC12:31
*** amotoki has quit IRC12:33
*** thrash|wknd is now known as thrash12:34
openstackgerritVasyl Saienko proposed openstack/ironic-inspector: Use OSC in exercise.sh  https://review.openstack.org/34461612:37
*** ishant|away has quit IRC12:38
*** kirankv has quit IRC12:39
*** moshele has joined #openstack-ironic12:42
*** amotoki has joined #openstack-ironic12:43
*** afaranha has joined #openstack-ironic12:48
*** amotoki has quit IRC12:48
*** sdake_ is now known as sdake12:48
*** lazy_prince has quit IRC12:49
*** mjturek1 has joined #openstack-ironic12:50
*** kirankv has joined #openstack-ironic12:53
*** bfournie has joined #openstack-ironic12:53
*** mjturek1 has quit IRC12:54
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Add manage_boot parameter to introspection API  https://review.openstack.org/31680112:56
dtantsursambetts, ^^^12:56
dtantsurwould be cool to merge it reasonable soon, as we're approaching the client release deadline, and there's an associated client change12:56
sambetts++ I'll check it out12:57
*** kirankv has quit IRC12:57
openstackgerritJulia Kreger proposed openstack/ironic-specs: Boot from Volume - Reference Drivers  https://review.openstack.org/29499512:57
*** lazy_prince has joined #openstack-ironic12:57
*** mtanino has joined #openstack-ironic13:00
*** lazy_prince has quit IRC13:02
openstackgerritVasyl Saienko proposed openstack/ironic-inspector: Use OSC in exercise.sh  https://review.openstack.org/34461613:05
*** ccamacho is now known as ccamacho|lunch13:05
openstackgerritVasyl Saienko proposed openstack/ironic-inspector: Use OSC in exercise.sh  https://review.openstack.org/34461613:07
*** baoli has joined #openstack-ironic13:08
*** mjturek1 has joined #openstack-ironic13:09
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Support passing manage_boot argument in Python API  https://review.openstack.org/33947913:09
*** baoli_ has joined #openstack-ironic13:10
dtantsursambetts, also updated ^^^13:10
dtantsurmilan, ^^^13:10
sambettsawesome :)13:10
* milan looks13:11
*** krtaylor has joined #openstack-ironic13:12
*** amotoki has joined #openstack-ironic13:13
*** baoli has quit IRC13:14
xavierrLOG.info(_LI("good morning Ironic"))13:20
mgouldmorning xavierr :-)13:21
*** aparnav has quit IRC13:21
jrolldtantsur: do we have client changes for driver comp stuff?13:23
jrollI'm going to assume so13:23
dtantsurjroll, we will need such changes, I don't have them up though13:24
jrollrather, do we need them along with the newton release?13:24
dtantsurpreferably, but not 100% required13:24
dtantsurand anyway I'd be surprised if the composition itself lands in Newton :(13:24
jrollyeah, but I do think we can do some of the plumbing13:25
jrolldefinitely want to get new IPA APIs in newton13:26
dtantsur++ to both13:26
dtantsure.g. I think we should deprecate attaching periodic tasks to driver objects (as opposed to interfaces)13:26
dtantsurthere won't be a separate driver object in the same sense after the reform...13:26
sambetts++ makes sense13:27
jrollindeed13:30
openstackgerritMichael Turek proposed openstack/ironic: Prevent URL collisions with sub-controllers: nodes/ports  https://review.openstack.org/31451413:33
openstackgerritMichael Turek proposed openstack/ironic: Add classes for Portgroups API enablement  https://review.openstack.org/34754913:33
*** ayoung has joined #openstack-ironic13:36
*** ccamacho|lunch is now known as ccamacho13:39
openstackgerritDmitry Tantsur proposed openstack/ironic: Deprecate putting periodic tasks on a driver object  https://review.openstack.org/34951013:41
*** ametts has joined #openstack-ironic13:41
dtantsurhere it goes ^^^13:41
*** moshele has quit IRC13:44
*** moshele has joined #openstack-ironic13:45
sambettsdtantsur: you responded about the new_ipmi_creds before I could :-P13:47
dtantsurhehe :)13:49
openstackgerritHironori Shiina proposed openstack/ironic-specs: Follow-up patch of 75290c72f7c3138bfdab151728718f771dfb9685  https://review.openstack.org/33537813:49
*** ametts has quit IRC13:51
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Add manage_boot parameter to introspection API  https://review.openstack.org/31680113:56
dtantsursambetts, vsaienk0, updated ^^^ (had to rebase unfortunately)13:57
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Support passing manage_boot argument in Python API  https://review.openstack.org/33947913:57
milanwall.e executes unittests: .EEEEEEEEEEEERROR13:58
*** achanda has joined #openstack-ironic13:59
*** ametts has joined #openstack-ironic14:00
*** anush_ has joined #openstack-ironic14:00
*** rbartal has quit IRC14:01
*** baoli_ has quit IRC14:02
dtantsurlol14:02
*** baoli has joined #openstack-ironic14:03
*** sergek has quit IRC14:03
*** pas-ha has quit IRC14:03
*** izaakk has quit IRC14:03
*** lekha has quit IRC14:03
*** romcheg has quit IRC14:03
*** ildikov has quit IRC14:03
openstackgerritJim Rollenhagen proposed openstack/ironic: Move network_interface selection logic to node object  https://review.openstack.org/34951914:04
openstackgerritJim Rollenhagen proposed openstack/ironic: Explicitly set network_interface on POST/PATCH  https://review.openstack.org/34952014:04
*** NobodyCam has quit IRC14:04
*** mgkwill has quit IRC14:04
jrolldtantsur: TheJulia: sambetts: ^ there's everything but the migration14:04
*** cppforlife_ has quit IRC14:04
*** amit213 has quit IRC14:04
*** fyxim has quit IRC14:04
*** pas-ha has joined #openstack-ironic14:04
*** sergek has joined #openstack-ironic14:04
*** agordeev has quit IRC14:04
*** vdrok has quit IRC14:04
*** kozhukalov has quit IRC14:04
*** betherly has quit IRC14:04
*** tdurakov has quit IRC14:04
*** boris-42 has quit IRC14:04
*** serverascode has quit IRC14:04
*** rajinir has quit IRC14:04
*** patrickeast has quit IRC14:04
jrolloh, we'll need reno updates and doc updates too14:04
*** achanda has quit IRC14:05
*** fyxim has joined #openstack-ironic14:07
vsaienk0jroll: I have a question regarding https://review.openstack.org/#/c/29789514:07
*** afaranha has quit IRC14:08
jrollvsaienk0: ask away14:08
*** hecliunyx has quit IRC14:08
vsaienk0why not return None for all drivers there, if binding is needed we can update host_id lather in Ironic network driver when adding binding profile information14:09
*** rajinir has joined #openstack-ironic14:09
*** vdrok has joined #openstack-ironic14:09
*** NobodyCam has joined #openstack-ironic14:09
jrollvsaienk0: I thought about that too, I think it would also work14:09
jrollI'm not totally sure though14:10
*** tangchen has joined #openstack-ironic14:10
*** tangchen_ has joined #openstack-ironic14:10
vsaienk0even if some installations needs to bind ports with flat driver for example, we can set host_id on ironic side14:10
jrollyeah14:11
jrollI still think setting it explicitly is the right thing to do14:11
jrollwe can do both :)14:11
sambettswe need to set it explicitly in some cases incase someone is running a newer nova with old Ironic14:12
sambettsexplictly in nova that is ^14:12
sambettsbecause in that case Ironic won't understand to set the host_id14:12
jrollmmm, we don't really support that, but it's a good point14:13
jrollactually that won't work since we use API 1.2014:13
*** ildikov has joined #openstack-ironic14:14
jrollI guess if they were using today's HEAD of ironic with future nova, but meh?14:14
sambettsI thought we support +1 nova with -1 ironic for rolling upgrade ?14:14
*** izaakk has joined #openstack-ironic14:14
sambettse.g. newton nova with mitaka ironic14:14
jrollno, we support N+1 ironic with N nova14:14
jrollalways upgrade ironic first14:14
*** lekha has joined #openstack-ironic14:15
*** causten has joined #openstack-ironic14:15
sambettsoh, I thought we supported Nova first too14:15
*** Goneri has quit IRC14:15
*** romcheg has joined #openstack-ironic14:15
jrollnope14:16
*** cppforlife_ has joined #openstack-ironic14:16
openstackgerritAparna proposed openstack/ironic: Support 'hardware_disk_erase' as a cleaning step  https://review.openstack.org/34948414:16
*** Goneri has joined #openstack-ironic14:17
*** mgkwill has joined #openstack-ironic14:18
sambettsif thats the case then I'm +1 to moving logic out of the nova driver14:18
*** boris-42 has joined #openstack-ironic14:18
*** patrickeast has joined #openstack-ironic14:18
jrollsure, I still think we should do both14:18
jrollI think an op should be able to tell from the API which network interface is in use14:18
*** amit213 has joined #openstack-ironic14:18
*** kozhukalov has joined #openstack-ironic14:18
*** tdurakov has joined #openstack-ironic14:19
*** betherly has joined #openstack-ironic14:19
*** hecliunyx has joined #openstack-ironic14:19
*** serverascode has joined #openstack-ironic14:19
*** gabriel-bezerra has quit IRC14:20
* milan imports introspection_state as istate then googles it -> http://lxr.free-electrons.com/source/kernel/irq/internals.h#L1714:20
*** gabriel-bezerra has joined #openstack-ironic14:20
*** agordeev has joined #openstack-ironic14:21
dtantsurgood one :D14:22
milan;)14:23
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Verify JSON response of chassis commands  https://review.openstack.org/33659614:24
openstackgerritJim Rollenhagen proposed openstack/ironic: Add a data migration to fill node.network_interface  https://review.openstack.org/34952814:24
jrolland there's the migration14:24
* jroll stepping away and then will write docs14:24
*** thiagop has joined #openstack-ironic14:25
*** pas-ha has quit IRC14:25
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Verify JSON response of chassis commands  https://review.openstack.org/33659614:26
*** chlong has joined #openstack-ironic14:26
thiagopGood morning, Ironic14:26
vsaienk0sambetts, at the moment ironic virt driver returns None always since https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L1159 is dead code14:27
*** chlong is now known as chlong|gone14:27
*** pas-ha has joined #openstack-ironic14:27
jrolldtantsur: btw, in case you didn't see it, https://review.openstack.org/#/c/348443/14:27
jrollvsaienk0: you mean it always returns super(...)14:28
jrollvsaienk0: see 'none' is the default for that getattr14:28
dtantsurjroll, oh awesome14:28
jrolldtantsur: it totally works, too :D14:29
jrolljust needs a bit of cleanup per dan14:30
dtantsuryeah, cool14:30
dtantsurwe got an FFE for that?14:30
thiagopdtantsur: welcome back14:30
dtantsurhey thiagop, thanks :)14:30
jrolldtantsur: yeah, seems so14:30
jrolldtantsur: I would trade it in for a multitenant networking FFE though :P14:30
vsaienk0jroll, right it will always  returns super(..)14:31
jrollvsaienk0: which is not None14:31
dtantsurjroll, me too :)14:31
* jroll brb14:32
*** PollyZ has joined #openstack-ironic14:32
*** chlong|gone has quit IRC14:32
*** chlong|gone has joined #openstack-ironic14:33
*** chlong|gone is now known as chlong14:34
*** jvgrant_ has joined #openstack-ironic14:34
*** syed_ has joined #openstack-ironic14:35
*** michauds has joined #openstack-ironic14:35
*** kirankv has joined #openstack-ironic14:36
*** rbudden has quit IRC14:37
thiagopsambetts vdrok did the fix on the multitenant job work?14:38
*** hwcomcn has joined #openstack-ironic14:38
*** hwcomcn has quit IRC14:39
*** hwcomcn has joined #openstack-ironic14:39
*** PollyZ has quit IRC14:40
*** hwcomcn has quit IRC14:41
vdrokmorning thiagop, looks like it, I've seen green runs today14:41
thiagopgood to know14:42
*** hwcomcn has joined #openstack-ironic14:42
*** vishwanathj has joined #openstack-ironic14:46
*** kirankv has quit IRC14:46
*** ppiela has joined #openstack-ironic14:47
*** edand has quit IRC14:49
*** lazy_prince has joined #openstack-ironic14:49
*** clenimar has quit IRC14:49
*** kirankv has joined #openstack-ironic14:50
*** jcoufal has joined #openstack-ironic14:50
*** jcoufal_ has quit IRC14:51
openstackgerritJim Rollenhagen proposed openstack/ironic: Add a data migration to fill node.network_interface  https://review.openstack.org/34952814:51
openstackgerritJim Rollenhagen proposed openstack/ironic: Add note about network_interface migration to release notes  https://review.openstack.org/34954614:51
*** clenimar has joined #openstack-ironic14:51
*** hwcomcn_ has joined #openstack-ironic14:52
*** jvgrant_ has quit IRC14:52
*** mjturek1 has quit IRC14:52
*** jvgrant_ has joined #openstack-ironic14:52
*** mjturek1 has joined #openstack-ironic14:52
*** mjturek1 has quit IRC14:54
*** lazy_prince has quit IRC14:54
*** kirankv has quit IRC14:54
*** fumie has quit IRC14:54
*** beekneemech is now known as bnemec14:55
*** hwcomcn has quit IRC14:55
*** rbudden has joined #openstack-ironic14:58
*** jistr is now known as jistr|call15:00
*** sabeen has joined #openstack-ironic15:03
mgouldI'm trying to nova-boot on a node which already has an Instance UUID set15:05
mgouldit's the UUID of an instance that's been deleted in nova15:05
mgoulddo I have to do anything to remove the UUID?15:06
jrollyeah, you need to remove it in ironic15:06
mgouldthe node is already in provisioning state available and maintenance false15:06
jrollthat *should* have been done during set-provision-state delete, though15:06
sambetts^ this15:06
mgouldwhich should have happened when I ran "nova delete"?15:06
sambettsyup15:07
mgouldoh well15:07
mgouldis there a way I can force it to happen now?15:07
jrollironic node-update uuid remove instance_uuid15:07
mgouldthanks!15:08
jrollnp :)15:08
sambettsdid we implemented it so that cleaning would remove instance IDs? so moving a node from avaible to managable and then back to availiable would remove it too?15:09
JayFsambetts: did you see that bug I tagged you on?15:09
sambettsor I remember fixing it before by doing set-provision-state active followed instantly by a set-provision-state deleted15:09
sambettsJayF: https://review.openstack.org/#/c/349432/15:09
sambettsJayF: I wrongly assumed the default was to use the pre-builts15:10
sambettsJayF: I should have checked our devstack plugin15:10
*** dprince has quit IRC15:10
sambettsJayF: that patch makes it explict15:10
JayFsambetts: I looked in both -src and non -src jobs, neither had them set15:10
JayFsambetts: which doesn't seem to map to the problem being fixed there?15:11
thiagopsambetts: what if cleaning is disabled?15:11
sambettsthiagop: yeah, thats why it should do it on the delete15:12
sambettsJayF: it goes through devstack-gate https://github.com/openstack-infra/devstack-gate/blob/8adc996eef576d77c52721117383d04d215fbe2f/devstack-vm-gate.sh#L35415:12
sambettsJayF: which reads the thing we export and only writing it if its the not the default15:12
*** loki___ has joined #openstack-ironic15:12
*** jistr|call is now known as jistr15:12
sambettsJayF: /me is not a fan of devstack-gate15:12
*** clenimar has quit IRC15:13
sambettsbecause of this reason15:13
JayFAh, okay15:13
openstackgerritJim Rollenhagen proposed openstack/ironic: Add note about network_interface migration to release notes  https://review.openstack.org/34954615:13
jlvillalGood morning Ironic15:14
thiagopgood morning jlvillal , welcome back too15:14
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add baremetal port list command to OSC plugin  https://review.openstack.org/34672215:14
jrollmorning jlvillal, welcome back15:14
*** fxpester has joined #openstack-ironic15:15
* jlvillal is just back from 2 weeks of vacation. He forget his backpack with his computer in the rental car he drove to the airport to start his trip. So didn't get to do anything Ironic related for two weeks15:15
jlvillalThanks thiagop jroll :)15:15
* jlvillal did get his backpack and computer this morning from Hertz15:15
*** nicodemos has quit IRC15:15
jrollsambetts: it happens on the delete btw: https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L56015:15
sambettsjlvillal: well I hope you had a great vacation without the tech :)15:15
jroll^^15:15
fxpesterjlvillal: it is nice and powerful ?15:15
jrollyou aren't supposed to do ironic things on vacation :)15:15
sambettsthis ^15:16
jlvillalsambetts: It was good. I had fun :)15:16
*** lucasagomes has quit IRC15:16
sambettsjlvillal: :D15:16
jlvillalfxpester: It is one I already had. A few years old15:16
sambettsjroll: ah right make sense15:16
jlvillaljroll: True. It was good on the one hand to not have it. My phone worked for the most part.15:16
*** mjura has quit IRC15:17
*** stendulker has joined #openstack-ironic15:17
fxpesterjlvillal: I`m glad for you) in Russia it is almost impossible to get back lost items, even cheap shit or documents15:18
jlvillalfxpester: Ah. Well I forget my backup in Portland on my way to Russia.15:18
jlvillals/backup/back-pack/15:18
jlvillalSo I was in Moscow for two weeks without the backpack15:18
fxpesterjlvillal: man, how are you surviving this world with such a habbits ?15:19
jlvillalfxpester: Not too bad :)15:19
dtantsurmorning JayF, jlvillal15:20
jlvillalGood morning dtantsur15:20
fxpesterguys, sorry for lame question, yet: can Ironic boot Standart nova images ? like Cirros and other qcow and img ? or do I need something special ?15:21
JayFgm15:22
fxpestertotally new to ironic but gotr task today)15:22
JayFfxpester: you can use any image as far as ironic is concerned; but many VM images don't have essential pieces needed for bare metal, like drivers15:22
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for --wrap option for driver-properties output  https://review.openstack.org/29707715:22
JayFfxpester: if you're talking about for devstack; we have a devstack howto and I'm fairly certain we use identical images15:22
*** gabriel-bezerra has quit IRC15:24
*** xavierr has quit IRC15:24
jrollwe do15:24
*** thiagop has quit IRC15:24
*** lucasagomes has joined #openstack-ironic15:25
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add test for 'baremetal port create' command  https://review.openstack.org/33288015:29
*** nicodemos has joined #openstack-ironic15:30
*** clenimar has joined #openstack-ironic15:31
*** thiagop has joined #openstack-ironic15:32
sambettsjroll: not going to make the meetings this evening, hopfully I won't be missed15:33
jrollsambetts: no worries15:34
*** sdake has quit IRC15:34
thiagopNobodyCam: around?15:35
jlvillalsambetts: Of course you will be missed! Have a great night15:35
sambettsnight all15:36
*** sambetts is now known as sambetts|afk15:36
NobodyCamNight sambetts|afk15:36
NobodyCamthiagop: just getting moving here15:36
NobodyCamAnd ofc.15:36
NobodyCamGood morning ironic'ers ;p15:37
jlvillalGood morning NobodyCam15:38
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add test for 'baremetal port create' command  https://review.openstack.org/33288015:38
thiagopNobodyCam: Hi there! Have you seen my comments on https://review.openstack.org/#/c/286192/? Do they satisfy your concerns?15:38
NobodyCamMorning jlvillal15:39
NobodyCamthiagop: let me grab the laptop15:39
thiagopOkay15:40
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Verify JSON response of chassis commands  https://review.openstack.org/33659615:40
*** harlowja_at_home has joined #openstack-ironic15:43
vdrokgood morning jlvillal and NobodyCam15:48
jlvillalvdrok: :)15:48
vdrokjlvillal: how did you like Moscow? :)15:48
stendulkerjroll: Hi15:48
jlvillalvdrok: It was good.  I rode a Segway for the first time :)15:49
stendulkerjroll: Had posted reply to your comment for https://review.openstack.org/#/c/338791/ . Please see if it answers your query?15:49
jlvillalvdrok: Plus had fun doing other things!15:49
NobodyCammorning vdrok :)15:50
openstackgerritStephanie Miller proposed openstack/python-ironicclient: Add volume_connector support  https://review.openstack.org/21478615:50
vdrokjlvillal cool! but I thought segway is an american thing :)15:50
NobodyCamthiagop:15:50
*** gabriel-bezerra has joined #openstack-ironic15:50
jlvillalvdrok: So is McDonald's but they had that there too ;)15:50
thiagopNobodyCam: Thanks!15:50
NobodyCamah /me needs more coffee. thiagop thank you for the reply and for looking in to that issue. I have updated my vote to +215:51
thiagopNobodyCam: will look a 3rd "+2" + "+A" now15:51
NobodyCamthiagop: I have not +a'd the patch so other can look at it but it now has two +2's so it can be landed15:52
openstackgerritStephanie Miller proposed openstack/python-ironicclient: Add volume_target support  https://review.openstack.org/28523315:52
jrollstendulker: interesting, is it possible to use a non-self-signed cert for ilo?15:52
*** Sukhdev has joined #openstack-ironic15:52
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for portgroups in ironicclient  https://review.openstack.org/29772715:52
stendulkerjroll: yes, but one needs sign the  iLO self signed certificate using a valid CA certicate15:53
thiagopNobodyCam: I'm sure I can find a core willing to take a last look, tyvm15:53
jrollstendulker: okay15:53
stendulkerjroll: Currently iLO drivers have turned off certificate validation at urllib level15:54
stendulkerjroll: So this is essentially enablement of ilo certificate validation15:54
jrollstendulker: right, +A15:54
stendulkerjroll: Thank you :)15:54
*** hshiina has joined #openstack-ironic15:54
openstackgerritThiago Paiva Brito proposed openstack/ironic: Onetime boot when set_boot_device isn't persistent  https://review.openstack.org/34059615:55
*** loki___ has quit IRC15:59
*** nicodemos has quit IRC16:01
*** jistr is now known as jistr|biab16:01
*** nicodemos has joined #openstack-ironic16:02
*** dtantsur is now known as dtantsur|bbl16:03
*** achanda has joined #openstack-ironic16:03
*** e0ne has quit IRC16:05
*** ayoung has quit IRC16:06
*** ishant|away has joined #openstack-ironic16:06
*** achanda has quit IRC16:08
*** Sukhdev has quit IRC16:10
*** Sukhdev has joined #openstack-ironic16:10
devanandag'morning, all16:10
*** ishant|away has quit IRC16:11
thiagopmorning devananda16:11
openstackgerritJim Rollenhagen proposed openstack/ironic: Add note about network_interface migration to release notes  https://review.openstack.org/34954616:12
openstackgerritJim Rollenhagen proposed openstack/ironic: Add a data migration to fill node.network_interface  https://review.openstack.org/34952816:12
openstackgerritJim Rollenhagen proposed openstack/ironic: Explicitly set network_interface on POST/PATCH  https://review.openstack.org/34952016:12
*** kirankv has joined #openstack-ironic16:13
*** tesseract- has quit IRC16:14
*** lazy_prince has joined #openstack-ironic16:15
*** kirankv has quit IRC16:20
*** hwcomcn_ has quit IRC16:21
*** ayoung has joined #openstack-ironic16:21
*** fellypefca has joined #openstack-ironic16:22
*** moshele has quit IRC16:23
*** lazy_prince has quit IRC16:25
mgouldmorning devananda16:27
mgouldhow can I build a deployment image suitable for UEFI nodes?16:27
jrollmgould: I think there's a uefi element in DIB16:27
jrolldevananda: are there more docs to write about policy or is this now done done?16:27
mgouldjroll: thanks!16:27
JayFjroll: if you missed it; policy didn't land, failed tests, and now has a seemingly valid -1 on it16:28
JayFjroll: plus there are docs in that patch chain16:28
jrollaw, lame16:28
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Deny some driver_info updates when console is on  https://review.openstack.org/33000316:29
*** harlowja_at_home has quit IRC16:30
jrollwhy'd that suddenly start failing16:30
cineramamgould, there are a couple of changes under review regarding adding uefi support to the bootloader element16:30
*** PollyZ has joined #openstack-ironic16:30
JayFjroll: haven't dug into it16:30
cineramamgould, also take a look at bcornec comment here: https://bugs.launchpad.net/diskimage-builder/+bug/1412561/comments/116:30
openstackLaunchpad bug 1412561 in diskimage-builder "IPA/DIB is not compatible to create the uefi capable ISO images" [Medium,Triaged]16:30
jrollJayF: yeah just thinking out loud16:30
devanandaboo. I'll dive into that patchset and see what's going on16:32
mgouldcinerama: OK, so it's currently not supported by our tools?16:32
*** causten has quit IRC16:33
*** dprince has joined #openstack-ironic16:33
devanandajroll: I think the minimal amount of documentation is included in that patch chain, but doing that has caused me to feel like our deployer-facing docs are really lacking16:33
jrolldevananda: I suspect it's related to suddenly using keystone v316:33
devanandaah, could be16:34
cineramamgould, also have a look at the "iso" element16:35
*** Romanenko_K has quit IRC16:35
*** fellypefca has quit IRC16:35
mgouldcinerama: thanks!16:36
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Update devstack plugin with new auth options  https://review.openstack.org/29731216:39
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions  https://review.openstack.org/23698216:39
*** syed_ has quit IRC16:42
*** tesseract- has joined #openstack-ironic16:46
*** ijw has joined #openstack-ironic16:47
*** jvgrant has quit IRC16:48
*** daemontool__ has joined #openstack-ironic16:48
*** bvandewa has joined #openstack-ironic16:48
*** jistr|biab is now known as jistr16:49
mgouldcinerama: now building an .iso image. How can I tell Ironic to use it for deployment? It looks like Ironic wants two settings, a ramdisk and a vmlinuz16:50
*** dtantsur|bbl is now known as dtantsur16:51
mgoulds/ramdisk/initrd16:51
dtantsurmorning devananda16:51
*** rama_y has joined #openstack-ironic16:51
*** daemontool_ has quit IRC16:52
*** ijw has quit IRC16:53
*** ijw has joined #openstack-ironic16:53
*** tesseract- has quit IRC16:54
*** fxpester has quit IRC16:54
openstackgerritDevananda van der Veen proposed openstack/ironic: Add keystone policy support to Ironic  https://review.openstack.org/32559916:55
*** xavierr has joined #openstack-ironic16:57
*** lok has joined #openstack-ironic16:58
*** xavierr has quit IRC16:58
*** tesseract- has joined #openstack-ironic16:58
*** tesseract- has quit IRC16:58
mariojvironic meeting?16:59
jrollyep17:00
*** xavierr has joined #openstack-ironic17:00
*** gabriel-bezerra has quit IRC17:01
*** wajdi has joined #openstack-ironic17:01
*** sdake has joined #openstack-ironic17:02
*** lok has quit IRC17:02
*** nicodemos has quit IRC17:03
*** nicodemos has joined #openstack-ironic17:03
*** clenimar has quit IRC17:03
*** thiagop has quit IRC17:04
*** thiagop has joined #openstack-ironic17:04
*** achanda has joined #openstack-ironic17:06
*** nicodemos has quit IRC17:10
*** achanda has quit IRC17:10
*** ayoung has quit IRC17:11
*** achanda has joined #openstack-ironic17:11
*** ohamada has quit IRC17:11
*** achanda has quit IRC17:11
*** achanda has joined #openstack-ironic17:12
*** nicodemos has joined #openstack-ironic17:12
*** gabriel-bezerra has joined #openstack-ironic17:12
*** Sukhdev has quit IRC17:16
*** sdake has quit IRC17:17
*** xavierr has quit IRC17:18
*** rama_y has quit IRC17:18
*** thiagop has quit IRC17:19
*** nicodemos has quit IRC17:19
*** gabriel-bezerra has quit IRC17:20
*** bvandewa has quit IRC17:20
*** clenimar has joined #openstack-ironic17:21
*** gabriel-bezerra has joined #openstack-ironic17:22
*** rama_y has joined #openstack-ironic17:22
*** xavierr has joined #openstack-ironic17:23
*** ayoung has joined #openstack-ironic17:24
*** nicodemos has joined #openstack-ironic17:24
*** thiagop has joined #openstack-ironic17:24
*** Goneri has quit IRC17:24
*** lok has joined #openstack-ironic17:25
*** TuanLUong has joined #openstack-ironic17:29
openstackgerritMerged openstack/ironic: Support to validate iLO SSL certificate in iLO drivers  https://review.openstack.org/33879117:29
* vdrok has to leave17:31
vdrokgood night everyone!17:31
jroll\o17:31
dtantsursee you tomorrow17:31
lokReview please https://review.openstack.org/#/c/272658/17:31
*** dtantsur is now known as dtantsur|afk17:31
devanandag'night vdrok, dtantsur|afk o/17:32
*** joprovost has joined #openstack-ironic17:33
* jroll bbiab17:34
thiagopDoes anyone have seen what I have put on the open discussion section of the meeting? I think my internet have dropped right when I talked... :/17:34
jrollthiagop: I did not17:34
TheJuliathiagop: Would it happen to be something involving https://review.openstack.org/#/c/286192/ ?17:35
thiagopyah, asking for +A on ^17:36
thiagopIt has 2 +2's already17:36
*** amotoki has quit IRC17:38
*** amotoki has joined #openstack-ironic17:38
*** amotoki has quit IRC17:38
*** hshiina has quit IRC17:40
TheJuliathiagop: I'm really looking forward  to seeing it land since I know you guys have a few things that it is kind of blocking in terms of advancing your oneview driver.17:41
thiagopTheJulia: yeah... :/17:42
*** bvandewa has joined #openstack-ironic17:44
*** bvandewa has quit IRC17:44
openstackgerritStephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information  https://review.openstack.org/20098317:46
TuanLUong#TheJulia: I just wonder why feature  https://review.openstack.org/#/c/286192/ have many failure tempest check17:47
*** bvandewa has joined #openstack-ironic17:47
TuanLUongCan i join to help fix that17:47
*** bvandewa has quit IRC17:47
*** bvandewa has joined #openstack-ironic17:47
*** e0ne has joined #openstack-ironic17:48
openstackgerritStephanie Miller proposed openstack/ironic: Introduce VolumeConnector object  https://review.openstack.org/21458417:51
thiagopTuanLUong: these are all failures on non-voting jobs. Not a big concern..17:51
*** jcoufal has quit IRC17:52
*** bnemec has quit IRC17:53
jrollthiagop: hrm, oneview is failing though17:53
thiagopjroll: Our CI is behaving badly since Jenkins updated to 2.7.117:54
openstackgerritMario Villaplana proposed openstack/ironic: Add power state change notifications  https://review.openstack.org/32186517:54
openstackgerritMario Villaplana proposed openstack/ironic: Add notification base classes and docs  https://review.openstack.org/29846117:54
jrollthiagop: I mean, the power one has a real failure17:54
thiagopjroll: I tryied to look a way to pin it to the previous version, but haven't much time since I was a little sick last week17:55
*** PollyZ has quit IRC17:55
*** bnemec has joined #openstack-ironic17:55
openstackgerritMerged openstack/ironic: Implement new heartbeat for AgentDeploy  https://review.openstack.org/33396417:56
openstackgerritStephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume connector operation  https://review.openstack.org/21458517:57
*** sambetts|afk has quit IRC17:57
*** sambetts has joined #openstack-ironic17:59
*** stendulker has quit IRC17:59
*** wajdi_ has joined #openstack-ironic18:00
*** wajdi has quit IRC18:00
*** wajdi has joined #openstack-ironic18:01
*** TuanLUong has quit IRC18:01
*** lok has quit IRC18:01
thiagopjroll: those power and management tests sometimes fails since there is no delay between setting and getting the node's info18:04
thiagopjroll: if Ironic runs a little slower, it may get the previous state18:04
jrollthiagop: yeah, just pointing out it's scary to land things like this where the CI is failing18:05
thiagopjroll: I plan to re-implement that using tempest features for checking and timeout"ing"18:05
jrollthat said, I'll put it on my list to review18:05
*** mgould is now known as mgould|afk18:06
thiagopjroll: noted. I'll try to sort these problems out asap18:06
*** Goneri has joined #openstack-ironic18:06
jrollthiagop: no rush, was just odd at first look :)18:09
openstackgerritHugo Nicodemos proposed openstack/python-oneviewclient: Add auditing instrumentation for python-oneviewclient  https://review.openstack.org/33728118:09
*** e0ne has quit IRC18:19
*** PollyZ has joined #openstack-ironic18:21
*** e0ne has joined #openstack-ironic18:23
*** ayoung has quit IRC18:30
*** edand has joined #openstack-ironic18:30
*** causten has joined #openstack-ironic18:33
*** jcoufal has joined #openstack-ironic18:33
*** Sukhdev has joined #openstack-ironic18:35
*** rpioso has joined #openstack-ironic18:43
*** tangchen has quit IRC18:47
*** tangchen has joined #openstack-ironic18:47
*** causten has quit IRC18:50
*** syed_ has joined #openstack-ironic18:50
*** causten has joined #openstack-ironic18:53
openstackgerritFellype Cavalcante proposed openstack/python-oneviewclient: Extend python-oneviewclient to UplinkSet  https://review.openstack.org/29598118:55
*** ishant|away has joined #openstack-ironic18:57
*** lazy_prince has joined #openstack-ironic18:57
*** e0ne has quit IRC19:00
*** ijw has quit IRC19:02
*** ishant|away has quit IRC19:02
*** lazy_prince has quit IRC19:03
*** e0ne has joined #openstack-ironic19:04
*** Sukhdev has quit IRC19:09
*** Sukhdev has joined #openstack-ironic19:10
*** mbound has quit IRC19:12
*** harlowja has quit IRC19:14
*** mbound has joined #openstack-ironic19:15
*** harlowja has joined #openstack-ironic19:15
*** sdake has joined #openstack-ironic19:18
*** achanda has quit IRC19:19
*** fellypefca has joined #openstack-ironic19:21
*** moshele has joined #openstack-ironic19:27
*** ametts has quit IRC19:30
*** ametts has joined #openstack-ironic19:36
*** fragatina has joined #openstack-ironic19:45
*** fragatina has quit IRC19:45
*** fragatina has joined #openstack-ironic19:46
* jlvillal just learned about http://docs.openstack.org/developer/debtcollector/ Not sure how well it works.19:49
*** mbound_ has joined #openstack-ironic19:51
*** mbound has quit IRC19:54
*** ametts has quit IRC19:55
*** edand has quit IRC19:59
openstackgerritDmitry Tantsur proposed openstack/ironic: Promote agent vendor passthru to core API  https://review.openstack.org/33001820:00
*** ametts has joined #openstack-ironic20:00
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Use new agent API if available  https://review.openstack.org/33452320:01
*** e0ne has quit IRC20:01
*** moshele has quit IRC20:02
*** jcoufal_ has joined #openstack-ironic20:05
*** e0ne has joined #openstack-ironic20:06
*** jcoufal has quit IRC20:07
*** joprovost has quit IRC20:09
*** e0ne has quit IRC20:12
*** causten has quit IRC20:14
openstackgerritStephanie Miller proposed openstack/ironic: Add volume_targets table to save target information  https://review.openstack.org/28521820:20
*** fragatina has quit IRC20:21
*** ayoung has joined #openstack-ironic20:26
*** anush__ has joined #openstack-ironic20:31
*** anush_ has quit IRC20:32
openstackgerritStephanie Miller proposed openstack/ironic: Introduce VolumeTarget object  https://review.openstack.org/28521920:35
*** ametts has quit IRC20:36
harlowjajlvillal its the best (obviously)20:37
harlowjalol20:37
*** ametts has joined #openstack-ironic20:38
jlvillalharlowja: Heh.  I had just heard about it. It looks interesting :)20:38
harlowjathe best!20:39
harlowjasince sliced bread20:39
*** Goneri has quit IRC20:49
*** milan has quit IRC20:50
*** dprince has quit IRC20:50
*** milan has joined #openstack-ironic20:50
openstackgerritStephanie Miller proposed openstack/ironic: Introduce VolumeTarget object  https://review.openstack.org/28521920:52
openstackgerritJim Rollenhagen proposed openstack/ironic: Add note about network_interface migration to release notes  https://review.openstack.org/34954620:53
openstackgerritJim Rollenhagen proposed openstack/ironic: Add a data migration to fill node.network_interface  https://review.openstack.org/34952820:53
openstackgerritJim Rollenhagen proposed openstack/ironic: Explicitly set network_interface on POST/PATCH  https://review.openstack.org/34952020:53
*** edand has joined #openstack-ironic20:54
openstackgerritStephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations  https://review.openstack.org/28522020:56
*** Goneri has joined #openstack-ironic20:59
thiagopI'm calling it a day20:59
thiagopsee ya20:59
*** thiagop has quit IRC20:59
*** bfournie has quit IRC21:04
*** wajdi_ has joined #openstack-ironic21:11
*** wajdi has quit IRC21:14
*** wajdi_ has quit IRC21:16
*** sdake has quit IRC21:20
*** stevebaker has joined #openstack-ironic21:34
*** edand has quit IRC21:47
*** bfournie has joined #openstack-ironic21:48
*** milan has quit IRC21:50
*** rama_y has quit IRC21:50
*** milan has joined #openstack-ironic21:50
*** causten has joined #openstack-ironic21:53
*** PollyZ has quit IRC21:55
jrolldtantsur|afk: any reason this is a +1 and not +2? just need answers? https://review.openstack.org/#/c/171878/21:56
mrdaMorning Ironic21:56
*** ametts has quit IRC21:56
jrollheya mrda21:57
mrdao/21:57
*** causten has quit IRC22:00
*** causten has joined #openstack-ironic22:04
*** causten has quit IRC22:04
*** PollyZ has joined #openstack-ironic22:09
*** rama_y has joined #openstack-ironic22:10
*** sdake has joined #openstack-ironic22:13
*** jcoufal has joined #openstack-ironic22:14
*** jcoufal_ has quit IRC22:14
*** Goneri has quit IRC22:18
*** mbound_ has quit IRC22:20
*** milan has quit IRC22:20
*** milan has joined #openstack-ironic22:21
*** jcoufal has quit IRC22:24
*** rpioso has quit IRC22:24
*** jcoufal has joined #openstack-ironic22:25
openstackgerritJim Rollenhagen proposed openstack/ironic: Add note about network_interface migration to release notes  https://review.openstack.org/34954622:31
openstackgerritJim Rollenhagen proposed openstack/ironic: Add a data migration to fill node.network_interface  https://review.openstack.org/34952822:31
*** jcoufal has quit IRC22:36
*** jcoufal has joined #openstack-ironic22:36
jrollsomeone wanna kick off that series with landing https://review.openstack.org/#/c/349519/22:36
jrollthe entire chain is ready for review btw22:36
*** r-daneel has joined #openstack-ironic22:47
openstackgerritJim Rollenhagen proposed openstack/python-ironicclient: Use constraints for all the things  https://review.openstack.org/31867222:49
openstackgerritJim Rollenhagen proposed openstack/python-ironicclient: Use constraints for all the things  https://review.openstack.org/31867222:50
devanandajroll: policy is passing jenkins now, fyi22:51
openstackgerritDevananda van der Veen proposed openstack/ironic: Mask configdrive contents in API responses  https://review.openstack.org/32676822:53
openstackgerritJim Rollenhagen proposed openstack/ironic-lib: Use constraints for all the things  https://review.openstack.org/31869222:53
openstackgerritJim Rollenhagen proposed openstack/python-ironic-inspector-client: Use constraints for all the things  https://review.openstack.org/31869322:54
jrolldevananda: awesome22:55
jrolldevananda: http://logs.openstack.org/99/325599/21/check/gate-tempest-dsvm-ironic-multitenant-network-nv/77c1562/logs/devstacklog.txt.gz#_2016-08-01_18_44_04_72622:57
jrollmultitenant job is still busted22:57
jrollno clue why we wrap that in maintenance mode, but we do22:57
*** jcoufal has quit IRC22:57
* jroll -1 for now and goes off to relax22:58
*** jcoufal has joined #openstack-ironic22:58
*** rama_y has quit IRC23:01
openstackgerritAlexander Bashmakov proposed openstack/ironic: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python.  https://review.openstack.org/34683223:01
*** sdake has quit IRC23:01
openstackgerritStephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information  https://review.openstack.org/20098323:01
openstackgerritStephanie Miller proposed openstack/ironic: Introduce VolumeConnector object  https://review.openstack.org/21458423:02
openstackgerritStephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume connector operation  https://review.openstack.org/21458523:02
openstackgerritStephanie Miller proposed openstack/ironic: Add volume_targets table to save target information  https://review.openstack.org/28521823:03
openstackgerritStephanie Miller proposed openstack/ironic: Introduce VolumeTarget object  https://review.openstack.org/28521923:03
openstackgerritStephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations  https://review.openstack.org/28522023:03
devanandajroll: oh, I see -- I should have checked to see if that job was passing for other test again (it is)23:03
*** sdake has joined #openstack-ironic23:03
jrolldevananda: yes, it's fixed up23:04
jrolloh, I bet we do maintenance mode because we don't allow changing network driver in AVAILABLE23:04
jroll>.>23:04
jrollyeah, because we didn't want to deal with ENROLL in devstack23:06
jrollsadface23:06
devanandalol23:06
jrollyep, ENROLL, INSPECTING, MANAGEABLE23:07
*** jcoufal_ has joined #openstack-ironic23:07
jrollwell, that definitely makes me need a drink23:07
jrollsee y'all tomorrow23:07
*** jcoufal has quit IRC23:08
devanandaciao! enjoy!23:08
*** fumie has joined #openstack-ironic23:10
*** jcoufal_ has quit IRC23:15
*** jvgrant_ has quit IRC23:15
*** achanda has joined #openstack-ironic23:16
*** jcoufal has joined #openstack-ironic23:18
*** mbound has joined #openstack-ironic23:20
*** PollyZ has quit IRC23:22
*** mbound has quit IRC23:25
*** jcoufal has quit IRC23:28
*** jcoufal has joined #openstack-ironic23:30
*** sdake has quit IRC23:31
*** achanda has quit IRC23:32
*** sdake has joined #openstack-ironic23:40
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic power I/F for soft reboot and soft power off  https://review.openstack.org/21673023:41
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/32853923:42
openstackgerritStephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume connector operation  https://review.openstack.org/21458523:44
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off  https://review.openstack.org/21673823:44
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off  https://review.openstack.org/21674323:45
openstackgerritStephanie Miller proposed openstack/ironic: Add volume_targets table to save target information  https://review.openstack.org/28521823:48
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/34862623:51
*** hoangcx2 has joined #openstack-ironic23:52
*** anush__ has quit IRC23:54
*** hoangcx has quit IRC23:54
openstackgerritStephanie Miller proposed openstack/ironic: Introduce VolumeTarget object  https://review.openstack.org/28521923:56
openstackgerritStephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations  https://review.openstack.org/28522023:57
*** jcoufal has quit IRC23:57
*** jcoufal has joined #openstack-ironic23:58
« Sunday, 2016-07-31 Index Tuesday, 2016-08-02 »

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