Thursday, 2016-10-06

« Wednesday, 2016-10-05 Index Friday, 2016-10-07 »
*** rama_y has quit IRC00:00
*** rpioso has quit IRC00:01
*** ijw has joined #openstack-ironic00:11
*** baoli has quit IRC00:15
*** ijw has quit IRC00:16
*** Vinsh has joined #openstack-ironic00:34
*** syed_ has quit IRC00:34
*** thrash is now known as thrash|g0ne00:38
openstackgerritHironori Shiina proposed openstack/ironic: Keep numbering of list in Install Guide  https://review.openstack.org/38269400:38
*** hoangcx has joined #openstack-ironic00:45
*** anush has quit IRC00:52
*** baoli has joined #openstack-ironic00:54
*** phuongnh has joined #openstack-ironic01:00
openstackgerritBob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands  https://review.openstack.org/37438101:04
*** joprovost has joined #openstack-ironic01:07
*** ijw has joined #openstack-ironic01:13
*** Sukhdev has quit IRC01:15
*** watanabe_isao has joined #openstack-ironic01:17
*** ijw has quit IRC01:18
*** raghusabar has joined #openstack-ironic01:20
*** wajdi has joined #openstack-ironic01:35
*** tuanluong has joined #openstack-ironic01:55
*** baoli has quit IRC02:05
*** amotoki has joined #openstack-ironic02:12
*** ijw has joined #openstack-ironic02:14
*** joprovost has quit IRC02:18
*** ijw has quit IRC02:20
*** yuanying has quit IRC02:27
*** raghusabar has quit IRC02:38
*** tiendc has joined #openstack-ironic02:43
*** syed_ has joined #openstack-ironic02:52
*** rbartal has joined #openstack-ironic03:01
*** phschwartz has quit IRC03:03
*** phschwartz has joined #openstack-ironic03:06
*** sdake has joined #openstack-ironic03:14
*** ijw has joined #openstack-ironic03:16
*** vikrant has joined #openstack-ironic03:20
*** ijw has quit IRC03:21
openstackgerritMerged openstack/ironic: Clean exceptions handling in conductor manager  https://review.openstack.org/38155403:32
*** sdake_ has joined #openstack-ironic03:45
*** sdake has quit IRC03:47
*** links has joined #openstack-ironic03:47
*** pmannidi has quit IRC03:48
*** pmannidi has joined #openstack-ironic03:50
*** rama_y has joined #openstack-ironic03:55
*** vmud213 has joined #openstack-ironic04:00
*** mtanino has quit IRC04:03
*** yuanying has joined #openstack-ironic04:13
*** ijw has joined #openstack-ironic04:17
openstackgerritpawnesh kumar proposed openstack/ironic: Fix doc build if git is absent  https://review.openstack.org/38271304:20
*** ijw has quit IRC04:23
*** Sukhdev has joined #openstack-ironic04:25
*** jaosorior has joined #openstack-ironic04:25
openstackgerritJohn L. Villalovos proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817304:30
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC management driver for Inject NMI  https://review.openstack.org/34873204:33
*** rajinir has quit IRC04:35
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI  https://review.openstack.org/34872104:37
*** yuanying has quit IRC04:45
openstackgerritpawnesh kumar proposed openstack/ironic: Fix doc build if git is absent  https://review.openstack.org/38271304:50
*** pawnesh has joined #openstack-ironic04:52
*** yuanying has joined #openstack-ironic04:54
*** makowals has quit IRC05:05
*** bjolo_ has joined #openstack-ironic05:06
*** pmannidi has quit IRC05:19
*** ijw has joined #openstack-ironic05:19
openstackgerritpawnesh kumar proposed openstack/ironic: Fix doc build if git is absent  https://review.openstack.org/38271305:20
*** ijw has quit IRC05:24
*** pmannidi has joined #openstack-ironic05:35
*** aNuposic has joined #openstack-ironic05:41
*** alex_xu has quit IRC05:48
*** vsaienko has joined #openstack-ironic05:49
*** alex_xu has joined #openstack-ironic05:51
*** Sukhdev has quit IRC05:51
*** jaosorior has quit IRC05:55
*** vsaienko has quit IRC05:56
*** jaosorior has joined #openstack-ironic05:57
*** vsaienko has joined #openstack-ironic05:57
*** yuanying has quit IRC06:02
*** sdake_ has quit IRC06:11
*** moshele has joined #openstack-ironic06:15
*** rcernin has joined #openstack-ironic06:15
pas-hamorning Ironic06:19
*** makowals has joined #openstack-ironic06:19
*** ijw has joined #openstack-ironic06:20
tuanluongmorning pas-ha06:24
*** e0ne has joined #openstack-ironic06:25
*** ijw has quit IRC06:26
*** aNuposic has quit IRC06:28
*** ohamada has joined #openstack-ironic06:29
*** vsaienko has quit IRC06:32
*** pmannidi_ has joined #openstack-ironic06:32
*** hshiina has joined #openstack-ironic06:32
*** pmannidi has quit IRC06:33
*** ccamacho has quit IRC06:34
openstackgerritNaohiro Tamura proposed openstack/ironic-lib: Use 'code-block' for pieces of code  https://review.openstack.org/37864306:37
*** wajdi_ has joined #openstack-ironic06:42
*** rama_y has quit IRC06:43
*** wajdi has quit IRC06:44
*** athomas has joined #openstack-ironic06:44
*** wajdi_ has quit IRC07:00
*** hoangcx has quit IRC07:06
*** tesseract- has joined #openstack-ironic07:11
*** yuanying has joined #openstack-ironic07:14
*** ijw has joined #openstack-ironic07:22
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817307:23
*** e0ne has quit IRC07:24
*** pcaruana has joined #openstack-ironic07:26
*** ijw has quit IRC07:28
*** ralonsoh has joined #openstack-ironic07:32
*** syed_ has quit IRC07:34
*** bjolo_ has quit IRC07:42
*** jpich has joined #openstack-ironic07:44
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:01
*** hoangcx has joined #openstack-ironic08:03
*** skazi has quit IRC08:03
aarefievmorning!08:09
*** skazi has joined #openstack-ironic08:15
*** Romanenko_K has joined #openstack-ironic08:15
*** jaosorior is now known as jaosorior_lunch08:17
*** mgoddard has joined #openstack-ironic08:19
*** ijw has joined #openstack-ironic08:24
*** ijw has quit IRC08:29
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902108:30
*** e0ne has joined #openstack-ironic08:37
*** derekh has joined #openstack-ironic08:38
*** dtantsur|afk is now known as dtantsur08:49
dtantsurMorning Ironic08:49
vsaienk0morning pas-ha, dtantsur, aarefiev, lucasagomes and all Ironic'ers!08:52
aarefievmorning dtantsur, vsaienk008:52
*** _milan_ has joined #openstack-ironic08:59
_milan_morning Ironic! :)08:59
*** bjolo_ has joined #openstack-ironic09:02
vdrokmorning ironic, pas-ha tuanluong aarefiev dtantsur vsaienk0 _milan_09:03
dtantsurhey _milan_, vdrok, vsaienk0, aarefiev, pas-ha09:03
dtantsur55 minutes was required for full cleaning to finish on a virtual drive #neveragain >_<09:03
aarefievmorning: vdrok, _milan_09:04
*** ralonsoh_ has joined #openstack-ironic09:04
_milan_morning aarefiev dtantsur pas-ha vdrok vsaienk0! :)09:05
*** ralonsoh has quit IRC09:05
*** karthiks has joined #openstack-ironic09:13
openstackgerritDmitry Tantsur proposed openstack/ironic: Log node state transitions at INFO level  https://review.openstack.org/38279409:15
dtantsurdunno what people think about that, but I'd find it convenient ^^09:15
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui: Imported Translations from Zanata  https://review.openstack.org/38002709:17
sambetts|afkMorning all09:17
*** sambetts|afk is now known as sambetts09:17
dtantsurmorning sambetts09:20
aarefievmorning sambetts09:20
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902109:20
aarefievdtantsur: why not just changing log level on existing msg09:21
dtantsuraarefiev, which ones?09:22
sambettsHey aarefiev, dtantsur09:22
aarefievdtantsur: there is debug logs about changing state somewhere, old state - foo, new one - bar09:22
_milan_morning sambetts! :)09:22
aarefievafair09:22
sambettsHi milan!09:23
_milan_so, decision making time? ;)09:23
dtantsuraarefiev, there is one place, but it can't log node UUID, so I find it pretty useless09:23
dtantsurunless I'm missing something09:23
aarefievaha, ok, I don't remember about uuid, looking09:24
*** ijw has joined #openstack-ironic09:25
*** strigazi_AFK is now known as strigazi09:25
_milan_dtantsur, I think state change should be INFO level, also, won't be too frequent for the logs to get bloated, +109:26
aarefievyeah, it's on_enter, on_exit watchers and no uuids09:28
sambettsaarefiev, dtantsur: here https://github.com/openstack/ironic/blob/master/ironic/common/states.py#L21609:30
dtantsursambetts, aarefiev, yes, these ones do exist, but without node UUID they seem completely useless to me09:30
sambettsaarefiev, dtantsur: IMO those logs should be changed to be able to include the node UUID, because as you say they are useless without it09:30
dtantsuralso I'd prefer not to log events at INFO level, as many events are internal-only09:30
*** ijw has quit IRC09:31
dtantsursambetts, I'm also not sure about the moment they're logged. judging by the message, they happen *before* the transition, while we need to do INFO logging after it's finished09:31
dtantsurhmm, it's not our code that processes watchers, it comes from automaton lib... and node only exists two levels upper: in task manager09:33
*** hshiina has quit IRC09:33
sambettsdtantsur: we manage to include the request ID in the logs though, so can't we get the node from that?09:34
dtantsursambetts, I doubt that the request ID is always (or even ever) bound to a node09:34
dtantsurespecially when transitions happen due to internal events, like timeouts09:35
*** jaosorior_lunch has quit IRC09:36
*** jaosorior_lunch has joined #openstack-ironic09:36
sambettsyeah :/ I actually think those logs cause more confusion than they help, I got throughly lost attempting to debug a race condition a month or so back where several nodes were all going through the state machine at the same time, and I couldn't tell which logs tied to which nodes09:39
sambettsdtantsur: I like your patch09:40
dtantsurcool09:43
*** jaosorior_lunch is now known as jaosorior09:44
*** e0ne_ has joined #openstack-ironic09:49
*** Romanenko_K has quit IRC09:50
*** aarefiev has quit IRC09:50
*** e0ne has quit IRC09:50
*** Romanenko_K has joined #openstack-ironic09:50
*** ralonsoh_ is now known as ralonsoh09:50
*** aarefiev has joined #openstack-ironic09:51
aarefievsambetts, dtantsur: the request id is there, but it doesn't mapped to node09:52
dtantsurI see09:52
aarefievI like inspector log contex09:52
aarefievfor this point09:53
sambettsIncluding the node id when ever we can in the [] part ??09:53
dtantsuryeah, but I guess it will require some effort to do the same in ironic09:53
aarefievyeah09:53
sambettsthe log context for ironic seems to be just the default one from oslo logging09:54
*** awiddersheim has quit IRC09:55
aarefievit is, but I'm wondering is it possible to pass custom values to it09:56
aarefievmore interesting we have instance value in context template09:57
aarefievin ironic09:57
*** hoangcx has quit IRC10:11
*** hoangcx has joined #openstack-ironic10:12
*** hoangcx has quit IRC10:12
*** awiddersheim has joined #openstack-ironic10:12
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902110:14
*** ccamacho has joined #openstack-ironic10:17
*** ijw has joined #openstack-ironic10:26
_milan_aarefiev, dtantsur, sambetts bike shedding time? ;) https://review.openstack.org/#/c/375045/710:29
patchbotpatch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses10:29
dtantsurno, lunch time :-P10:29
*** thrash|g0ne is now known as thrash10:29
_milan_dtantsur, excuses :P10:29
aarefiev:)10:29
*** ijw has quit IRC10:32
jrollmorning y'all10:33
jrollfeeling much better today, time to do some work \o/10:33
dtantsurmorning jroll, glad to hear it :)10:35
*** tiendc has quit IRC10:35
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817310:41
sergekMorning all10:41
aarefievmorning jroll, sergek10:42
aarefievjroll, glad to hear, keep on :)10:42
dtantsurI like how it looks now: http://logs.openstack.org/94/382794/1/check/gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa/5ade1c8/logs/screen-ir-cond.txt.gz?level=INFO10:43
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817310:43
dtantsure.g. it's now clear when exactly the deployment is starting: http://logs.openstack.org/94/382794/1/check/gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa/5ade1c8/logs/screen-ir-cond.txt.gz?level=INFO#_2016-10-06_09_34_16_19010:43
aarefievnice!10:44
_milan_dtantsur, ++10:47
_milan_morning jroll sergek! :)10:47
jrollmorning :)10:47
jrolloh, logging talk reminds me of something I noticed yesterday, can't tell if it's bad10:48
jroll/usr/local/lib/python2.7/dist-packages/pecan/routing.py:183: RuntimeWarning: Got exception calling lookup(): _lookup() takes at least 3 arguments (2 given) (('_lookup() takes at least 3 arguments (2 given)',))10:48
jrollright below this line http://logs.openstack.org/94/382794/1/check/gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa/5ade1c8/logs/screen-ir-api.txt.gz#_2016-10-06_09_32_26_04110:48
jrollsambetts: ^ I think you hacked on that, you might be interested?10:49
jrollafaict it's this blowing up https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L106610:49
sambettsjroll: oh yeah I saw this! Luckly it doesn't actually break anything, because we only get that warning in cases when lookup wouldn't be doing anything anyway10:51
sambettsjroll: but we should definatly fix it10:51
jrollsambetts: yeah, seems odd10:52
jrollthat's a magic pecan method, right?10:52
sambettsjroll: basically pecan chops up the request URL, and passes it into the _lookup function, if there isn't enough parts to the URL to fill in all the parameters it blows up10:53
jrollsambetts: then it just routes as usual before that was there?10:53
sambettsjroll: yup10:53
jrollcool10:53
* jroll reads docs10:53
sambettsjroll: I think it was just a mistake on my part because I missed a =None in the parameter list10:54
jrollsambetts: sounds like that's probably just for /v1/nodes/ that it blows up, given it splits on /10:54
sambettsyeah, thats the only URL with not enough parts10:55
jrollnod10:55
jrollsambetts: I guess another way would be remove the subres arg, then if remainder: subcontroller = self._subcontroller_map.get(remainder[0])10:56
sambettsjroll: yeah10:56
* jroll just does it10:57
sambetts++ thanks10:57
jrollglad you agree :D10:57
*** mjura has joined #openstack-ironic10:57
sambettsjroll: you might need an "if remainder" check in there10:57
jrollthat's what I said no? :P10:57
sambettsoh yeah XD I missed that bit ;)10:58
jrollso while that runs tests real quick11:03
jrollit appears this fixes the IPA gate breakage https://review.openstack.org/#/c/381934/11:03
patchbotpatch 381934 - ironic - DNM: testing SUBNETPOOL_PREFIX_V4 outside of 10/811:03
jrollunless that was simultaneously fixed11:03
jrollas evidenced by https://review.openstack.org/#/c/380351/11:03
patchbotpatch 380351 - ironic-python-agent - DNM: testing 38002311:03
jroll(failure is unrelated)11:03
jrollnow we shouldn't merge it as-is, but we can update project-config to set that11:03
sambetts++ I had to put the SUBNETPOOL fix into my third party CI to fix the same issue11:03
*** mjura has quit IRC11:04
*** hshiina has joined #openstack-ironic11:05
*** d0ugal has quit IRC11:05
*** d0ugal has joined #openstack-ironic11:06
*** d0ugal has quit IRC11:06
*** d0ugal has joined #openstack-ironic11:06
openstackgerritJim Rollenhagen proposed openstack/ironic: Fix _lookup() method for node API routing  https://review.openstack.org/38285011:06
jrollokay, there's that thing, let's not merge until we see evidence in logs11:06
jrollsambetts: so what do you think about hardcoding some range in project-config, versus something like https://review.openstack.org/#/c/381940/11:08
patchbotpatch 381940 - magnum - devstack: Fix neutron configuration to run in OSIC (MERGED)11:08
jrollas far as third-party CI and such, think the latter is okay?11:08
*** kun_huang has quit IRC11:10
*** kun_huang has joined #openstack-ironic11:11
sambettsjroll: In my case I have to override it anyway because the default conflicts with my ex-net, my only concern with hardcoding it in project-config would be if they were to change something in infra that might conflict with it11:11
*** vsaienko has joined #openstack-ironic11:11
jrollright11:12
jrollI'll just set it =$FIXED_RANGE11:12
*** mgould|afk is now known as mgould11:12
mgouldmorning Ironic!11:12
dtantsuroh, btw, official release today, right?11:15
dtantsurand morning mgould11:15
jrolldtantsur: indeed \o/11:16
dtantsurhmm, this looks strange IMO: http://docs.openstack.org/admin-guide/baremetal.html11:16
*** mjura has joined #openstack-ironic11:17
openstackgerritJim Rollenhagen proposed openstack/ironic: Set SUBNETPOOL_PREFIX_V4 to FIXED_RANGE  https://review.openstack.org/38193411:17
*** ralonsoh has quit IRC11:17
*** ralonsoh has joined #openstack-ironic11:18
jrolldtantsur: well, what about it? that's still published from docs repos, not our tree11:18
mat128good morning11:18
jrollthat's how it's looked for a while11:18
dtantsurjroll, ah, I see. looks meaningless, but if it was the case previously, then fine11:18
mat128dtantsur: re: 55 minute "long" virtual disk shredding - try wiping multiple 4TB drives :)11:19
jrolldtantsur: indeed, I think that guide is on docs list to move to in-tree plugins, but not totally sure11:19
jrollwe should probably add a link back to... something11:19
mat128morning mgould11:19
dtantsurjroll, yeah, maybe to install-guide and then to our dev docs11:19
dtantsurmat128, Oo11:19
jrolldtantsur: yeah, I think install-guide is intended to be for "deployers" and admin-guide for "operators". but it's a bit unclear, especially on an admin-only service :D11:20
TheJuliaGood morning11:22
mat128Hello TheJulia11:22
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: devstack: remove old exercise scripts  https://review.openstack.org/38285911:23
dtantsurI've been dreaming of this ^^^ for a while11:23
dtantsurjroll, right :)11:23
dtantsurmorning TheJulia11:23
jrolldtantsur: \o/11:23
mgouldmorning TheJulia11:25
mgoulddtantsur: not sure if that's better or worse than the time I had a dream in first-order predicate logic...11:26
*** watanabe_isao has quit IRC11:26
mat128TheJulia: do you think https://review.openstack.org/#/c/382856/ could be turned into a bifrost bug? Check the commit message11:26
patchbotpatch 382856 - openstack-infra/puppet-infracloud - Manage /root.ssh directory from bifrost11:26
mat128TheJulia: found that in -infra :)11:26
*** pawnesh has quit IRC11:26
dtantsurmgould, O_o11:26
mgouldmorning mat12811:26
mgouldthe joys of studying for exams :-)11:27
*** ijw has joined #openstack-ironic11:28
TheJuliamgould: that really seems like a deployed machine booting bug in glean11:28
mgould^^^ I think that was meant for mat12811:29
TheJuliaerr, yup11:29
TheJuliasorry!11:29
* TheJulia has not had any form of coffee yet11:29
mat128TheJulia: don't worry11:29
mgouldtbf, *I've* never seen us in the same room at the same time...11:29
sambettshaha11:29
*** ipukha has quit IRC11:30
* TheJulia totally doesn't get it... and seemingly needs something very strong this morning11:30
mat128mgould: Does that mean someone else might have seen us together?11:30
mat128mgould: but we couldnt see each other?11:30
mgouldno, it means there's no proof that we're not the same person11:30
mat128ah11:31
*** baoli has joined #openstack-ironic11:31
*** vsaienko has quit IRC11:31
*** vsaienko has joined #openstack-ironic11:32
mat128TheJulia: Ahh my bad11:32
TheJuliamat128: for what? :)11:32
mat128TheJulia: It's just the puppet thing thats doing the wrong thing. They did file { '/root/.ssh/id_rsa': } without having created the directory11:32
mat128TheJulia: it has *nothing* to do with bifrost11:32
mat128TheJulia: I was misled by the comments right above what was added11:33
TheJuliayup :(  The context is infracloud and similar deployments11:33
mat128TheJulia: coffee time for me too I guess11:33
*** ijw has quit IRC11:34
mgouldmat128: the commit message is IMHO misleading11:34
TheJuliawhich reminds me, I need to fix some template issue that I've created for myself, and I should have a fully working keystonified bifrost today11:34
mgouldTheJulia: \o/11:34
aarefievdtantsur: did you switch ipa job on tempest?11:36
dtantsuraarefiev, yes, we've switched both jobs11:36
aarefievon IPA11:36
*** phuongnh has quit IRC11:37
mgouldmat128: could you add a comment on the patch explaining what it's actually fixing? I'm not sure I quite understood11:37
mat128mgould: I can ask, but it's not mine11:38
mat128mgould: I mean, everyone can push updates on other reviews... but I dont think it will be appreciated11:39
mgouldmat128: sure, see my comment on https://review.openstack.org/#/c/382856/111:39
patchbotpatch 382856 - openstack-infra/puppet-infracloud - Manage /root.ssh directory from bifrost11:39
* mgould also needs MOAR COFFEE today11:39
mat128ah11:39
mat128re: your comment11:39
mat128yes11:39
mat128will do right now11:39
mgouldhttps://www.youtube.com/watch?v=XrivHBZhxD411:39
mgouldmat128: thanks!11:39
*** dprince has joined #openstack-ironic11:39
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902111:40
*** vsaienko has quit IRC11:41
aarefievdtantsur: I see gate-ironic-inspector-dsvm-ipa-src-nv, but it's non voting11:41
dtantsuraarefiev, none of these jobs run on newton or master11:41
dtantsurat least this is how it supposed to work right now, and I haven't seen them indeed :)11:42
TheJuliamat128: mgould I commennted, but given it is an -infra change, I'm personally less worried about the commit message :)11:42
aarefievdtantsur : ttps://review.openstack.org/#/c/296466/11:42
mat128TheJulia: code is code :)11:42
aarefiev https://review.openstack.org/#/c/296466/11:42
patchbotpatch 296466 - ironic-python-agent - Use ironic-lib to create configdrive11:42
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification  https://review.openstack.org/34843711:42
dtantsuraarefiev, Sep 14 was before the switch IIRC11:43
aarefievahh, ok11:43
dtantsuraarefiev, https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L3266-L327011:43
aarefievthanks for pointing, yeah it works for all jobs11:45
vdrokmorning sambetts jroll TheJulia mat128 mgould11:46
*** ipukha has joined #openstack-ironic11:47
mgouldmorning vdrok11:49
openstackgerritChangBo Guo(gcb) proposed openstack/ironic: Enable DeprecationWarning in test environments  https://review.openstack.org/37958111:49
mgouldmat128, TheJulia: thanks!11:49
* mgould belatedly realises that the commit message is just a grouping error11:49
*** gcb has joined #openstack-ironic11:49
*** vsaienko has joined #openstack-ironic11:49
openstackgerritMerged openstack/bifrost: Serialize localhost actions during final host deployment  https://review.openstack.org/38154111:49
mgouldmanage ($dir from bifrost) rather than (manage $dir) from bifrost11:50
mgouldsometimes I think Sumerian had the right idea by providing no facility for nested clauses11:50
*** vmud213 has quit IRC11:54
*** bfournie has quit IRC11:55
TheJuliamgould: or maybe an overzelous autocorrect in a text editor changing for to from.....11:58
*** vsaienko has quit IRC11:59
*** vsaienko has joined #openstack-ironic12:00
TheJuliaTIL, infra apparently has review guidelines for their project12:02
TheJuliammmm coffeeeeeeeeee12:03
_milan_morning mat128, mgould, TheJulia! :)12:03
*** trown|outtypewww is now known as trown12:03
TheJuliagood morning _milan_12:04
*** tuanluong has quit IRC12:04
_milan_aarefiev, dtantsur, sambetts so what about the bike shedding? I hope dtantsur had his lunch already :P12:04
vdrokjroll: dtantsur or everyone else interested - https://review.openstack.org/#/c/295371/13/ironic/common/context.py - what is your opinion on changing to_dict/from dict context methods?12:05
patchbotpatch 295371 - ironic - Add user and project domains to ironic context12:05
dtantsurwhat are we going to bikeshed about in the end?12:05
vdrokit may break an older service as it will try to deserialize a context and fail12:05
_milan_dtantsur, whether or not to accept current proposal of the queries in https://review.openstack.org/#/c/375045/712:05
patchbotpatch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses12:05
dtantsurvdrok, breaking older services doesn't sound cool for sure12:05
vdrokdo we care that eg in the conductor we won't have user_domain/project_domain12:05
dtantsur_milan_, which one is it compatible with now: ironic or API-WG?12:06
vdrokdtantsur: not other, ourselves :)12:06
vdrokdtantsur: the easy way to fix it would be to backport addition  of **kwargs in context constructor so that it does not fail12:06
vdrokbut that's super ugly12:06
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Cleanup unused (i)PXE kernel parameters  https://review.openstack.org/38288912:07
vdrokwe don't actually use the context fields on the conductor side, apart from maybe token, so do we care about fixing its dictionary representation?12:07
*** lucas-afk is now known as lucasagomes12:07
_milan_dtantsur, none and both. Ironic supports single sort key and single direction queries in pagination and doesn't support the fancy API WG filtering12:08
*** mgoddard_ has joined #openstack-ironic12:08
dtantsur_milan_, so in the end, if we don't go the API-WG way, I'd prefer something that we won't deprecate when we do eventually12:09
dtantsur_milan_, which makes sort more or less fine, but rules out finished_after12:09
dtantsuror we can implement any compatible subset of API-WG thing12:09
_milan_dtantsur, so '?finished_at=inter/val&finished_at=inter/val' wouldn't fit?12:10
dtantsur_milan_, how do you see it evolving into finished=gt:val (or what do they have?)?12:10
_milan_dtantsur, the WG spec doesn't support intervals at all12:11
dtantsurbut it supports operators, no?12:11
aarefievI guess Ironic don't support it because it was done before this  API-WG guidelines12:11
_milan_dtantsur, i think only single operator per query12:11
*** mgoddard has quit IRC12:11
dtantsuraarefiev, yeah, my main point is to not reinvent the wheel, especially in an incompatible way12:12
*** alex_xu has quit IRC12:12
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817312:12
aarefievalso I see ironic in list of projects to fix this12:13
sambettsthe WG way would support intervals by doing finished_at=gt:time lt:time right?12:13
_milan_for https://github.com/openstack/api-wg/blob/master/guidelines/pagination_filter_sort.rst#filtering for others12:13
*** rloo has joined #openstack-ironic12:13
_milan_sambetts, it is not clear to me how to express that, if I get it correctly a query supports single op12:14
sambettsmilan: I meant: finished_at=gt:time&finished_at=lt:time12:14
*** alex_xu has joined #openstack-ironic12:14
_milan_sambetts, that makes sense but is it supported in the spec?12:15
_milan_* WG spec12:15
mgouldcan we ask the API-WG how they'd express that query?12:15
mgouldalso, do we want union of disjoint intervals?12:15
mgouldie "last Tuesday OR today"12:15
*** mgoddard_ has quit IRC12:15
mgould_milan_: https://review.openstack.org/#/c/346923/ has just merged :-)12:16
patchbotpatch 346923 - openstack-dev/grenade - Document TARGET_RUN_SMOKE flag (MERGED)12:16
_milan_\o/ :)12:16
mgouldbrb12:16
sambettsmgould: disjoint intervals works in our suggestion but is indeed hard to represent in the WG one12:16
_milan_what #channel to use to ping those guys?12:16
*** vsaienko has quit IRC12:17
dtantsurhttps://wiki.openstack.org/wiki/API_Working_Group12:17
dtantsur_milan_, seems like #openstack-sdks12:18
_milan_dtantsur, thx12:18
dtantsurmgould, _milan_, for the record: I don't care about filtering by finished/started at ALL (and I'm pretty sad that the important part is blocked by it)12:18
aarefiev+112:18
_milan_dtantsur, but if we're specifying the endpoint, shouldn't we describe it thoroughly?12:19
dtantsur_milan_, we should, I'm just saying I'll live happily without this feature12:19
sambettsdo we split the list API itself and the support filtering  on that API into 2 feautes?12:19
jrollvdrok: commented12:20
* jroll bbiab12:20
aarefievActually I thought we will add only basic one pagination, filtering at first12:20
_milan_sambetts, would that help the spec?12:20
sambettswell the list nodes statuses I think is +2 to go, its only being blocked by the filtering which could be added on later right?12:21
aarefiev+112:21
_milan_sambetts, so should we remove the filtering part from the spec?12:22
aarefievwe can put only pagination to list introspection patch12:22
*** mgoddard has joined #openstack-ironic12:22
sambettsI think it might be worth making it, its own spec if we have a lot of bikeshedding to do12:23
dtantsurI would even say that the pagination part does not require a spec12:23
aarefievwe can merge base patch w/o spec, and discuss filtering, sorting in spec as separated feature12:23
dtantsurit is the filtering where we need to figure out a proper design12:24
aarefievdtantsur: yeah, it what I mean12:24
_milan_dtantsur, aarefiev, sambetts OK12:24
_milan_dtantsur, please remove the -2 on the patch12:24
sambetts_milan_, lucasagomes: supporting multiple operators in a query is something that affects the IPA devices hints right?12:24
_milan_I'll remove the filtering from it12:24
dtantsur_milan_, link handy?12:24
*** vsaienko has joined #openstack-ironic12:25
lucasagomessambetts, which query ?12:25
vdrokjroll: thank you, but it does not seem to be the case :(12:25
* lucasagomes reads scrollback12:25
_milan_dtantsur, https://review.openstack.org/#/c/344921/12:25
patchbotpatch 344921 - ironic-inspector - Add API for listing all introspection statuses12:25
vdrokmorning lucasagomes12:25
lucasagomesvdrok, hi there12:25
sambettslucasagomes: the whole gt and lt style thingys12:25
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902112:25
dtantsurdone12:26
_milan_dtantsur, thx12:26
aarefievhow does device hints query looks like now ?12:26
lucasagomessambetts, aarefiev so the device hints query is not passed via the IPA. It lives in the node object12:27
lucasagomese.g12:27
*** alex_xu has quit IRC12:27
lucasagomesproperties/root_device={'size': '>= 15G', 'model': '<or> model1 <or> model2 <or> model3'}12:28
_milan_sambetts, I like your point about the intervals query, that it would be difficult to express in the WG way  as it doesn't define more operators and nesting12:28
lucasagomesIPA get's the node representation and just parse that dict ^12:28
sambettslucasagomes: I thought we used the "gt" and "lt" things now, my mistake12:29
lucasagomessambetts, yeah not really, we are using the same as nova had for the filters12:29
aarefievaha, thanks lucasagomes12:29
*** alex_xu has joined #openstack-ironic12:29
*** ijw has joined #openstack-ironic12:29
sambettsmilan: if you could do finished_at=gt:12,lt:14&finished_at=gt:25,lt:60 then you could do distict intervals12:29
lucasagomesthat said, ceilometer does a lot of queries via the API I believe12:29
lucasagomesyou may want to take a look at how they do that ? (They also use wsme, so I bet we could share some code)12:30
_milan_lucasagomes, question is whether they're compatible w/ the WG suggestion or going to fix it12:30
aarefievtbh I like proposed time intervals more12:30
sambetts++12:30
sambettsaarefiev: well its a ISO standard12:31
lucasagomes_milan_, I see, yeah I don't know... Where's the API-WG suggestion for it ? (they already have one?)12:31
aarefievmaybe we should discuss it with  API-WG folks12:32
aarefievyeah12:32
sambettslucasagomes: https://github.com/openstack/api-wg/blob/master/guidelines/pagination_filter_sort.rst#filtering12:32
*** bfournie has joined #openstack-ironic12:32
lucasagomessambetts, ty12:33
_milan_aarefiev, I'm composing a question atm12:34
_milan_I guess I'll ask on mailing list and ping on channel later12:35
*** ijw has quit IRC12:35
aarefievthanks12:35
*** jcoufal has joined #openstack-ironic12:40
*** vsaienko has quit IRC12:41
jrollvdrok: ah, I see :( so it's still possible, just add a cycle to change from_dict first, right?12:41
*** jaosorior has quit IRC12:41
*** xavierr|afk has quit IRC12:41
vdrokjroll: you mean, wait for a cycle and then change the to/from_dict?12:42
vdrokyes, it is possible12:42
vdrokjroll: ah, I get it12:42
vdrokto iterate only through the values we know12:43
jrollvdrok: yeah, change from_dict to either **kwargs or accept all the new values as well, then do what I talked about but move everything back a cycle12:44
*** xavierr has joined #openstack-ironic12:45
*** anush has joined #openstack-ironic12:45
*** vsaienko has joined #openstack-ironic12:46
vdrokjroll: hm, but that's basically the same as leaving to_dict/from_dict untouched now, and then rewriting them after ocata releases?12:46
vdrokwhen the previous release will have **kwargs in the context constructor12:46
jrollvdrok: we would update from_dict in ocata to take the new values, update to_dict to send both in pike, remove the old in q12:47
vdrokjroll: yup, that is kind of what I meant :)12:48
jrollyeah, I think so12:48
vdrokjroll: ok, thank you, I'll update then!12:48
jrollvdrok: awesome :)12:48
xavierrmorning Ironic12:48
vdrokmorning xavierr12:49
*** links has quit IRC12:50
openstackgerritJim Rollenhagen proposed openstack/ironic: Set SUBNETPOOL_PREFIX_V4 to FIXED_RANGE  https://review.openstack.org/38193412:53
jrollthis makes IPA gates working again, just added a comment ^^ sambetts dtantsur12:53
*** anush has quit IRC12:56
jrollsambetts: this also appears to work as expected, if you want to review https://review.openstack.org/#/c/382850/112:56
patchbotpatch 382850 - ironic - Fix _lookup() method for node API routing12:56
*** Goneri has joined #openstack-ironic12:58
_milan_aarefiev, done: http://lists.openstack.org/pipermail/openstack-dev/2016-October/105180.html12:59
_milan_let's see12:59
*** jtomasek_ has joined #openstack-ironic13:00
aarefievlets see, thanks13:01
*** jheroux has joined #openstack-ironic13:02
*** [1]cdearborn has joined #openstack-ironic13:03
* _milan_ coffee \o/ :D13:04
*** moshele has quit IRC13:04
*** moshele has joined #openstack-ironic13:05
*** links has joined #openstack-ironic13:05
*** vsaienko has quit IRC13:06
*** vishwanathj has joined #openstack-ironic13:07
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902113:08
*** vsaienko has joined #openstack-ironic13:09
lucasagomesjroll, btw, for #382850, maybe we should add some unittest ?13:13
* lucasagomes +2'd anyway13:14
jrolllucasagomes: yeah, see comments, the weird thing is we can only assume how pecan uses that so idk if it's valuable13:14
lucasagomesoh13:14
jrolllucasagomes: I think the rest of our API unit tests do make sure the API works, so /shrug13:14
lucasagomesgotcha13:14
lucasagomesI missed the comments there13:15
sambettsand we have tempest API test to test it13:15
sambetts+A from me :)13:15
*** gcb has quit IRC13:15
*** vsaienko has quit IRC13:18
*** vsaienko has joined #openstack-ironic13:19
*** moshele has quit IRC13:26
*** vikrant has quit IRC13:26
*** jtomasek_ has quit IRC13:26
*** rbudden has joined #openstack-ironic13:28
*** ijw has joined #openstack-ironic13:31
*** baoli_ has joined #openstack-ironic13:32
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817313:34
*** baoli has quit IRC13:35
*** MattMan has quit IRC13:35
*** bnemec has joined #openstack-ironic13:36
*** ijw has quit IRC13:37
*** joprovost has joined #openstack-ironic13:39
openstackgerritBob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands  https://review.openstack.org/37438113:41
*** joprovost1 has joined #openstack-ironic13:42
*** joprovost has quit IRC13:43
*** joprovost1 is now known as joprovost13:43
*** vsaienko has quit IRC13:44
*** vsaienko has joined #openstack-ironic13:46
openstackgerritBob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands  https://review.openstack.org/37438113:50
openstackgerritVladyslav Drok proposed openstack/ironic: Add user and project domains to ironic context  https://review.openstack.org/29537113:50
*** hshiina has quit IRC13:51
*** vsaienko has quit IRC13:56
*** _mbelanger has joined #openstack-ironic13:56
*** mtanino has joined #openstack-ironic13:56
*** sdake has joined #openstack-ironic13:59
*** MattMan has joined #openstack-ironic14:00
*** anush has joined #openstack-ironic14:01
*** vsaienko has joined #openstack-ironic14:02
rloohi and morning all, vdrok, vsaienk0, sambetts, lucasagomes, jroll, xavierr, dtantsur, _milan_, aarefiev14:05
*** joprovost has quit IRC14:05
_milan_morning rloo! :)14:05
jrollmorning rloo14:05
aarefievmorning rloo14:06
rloojroll, dtantsur: your advice on this backport https://review.openstack.org/#/c/38269414:06
patchbotpatch 382694 - ironic (stable/newton) - Keep numbering of list in Install Guide14:06
vdrokgood morning rloo14:06
rloojroll, dtantsur: or should I just be reviewing based on technical accuracy of backport, and leave the decision making to you?14:07
jrollrloo: so stable policy as far as "is it worth backporting", is basically "do the benefits outweigh the risk"14:08
jrollthere's very little risk here14:08
rloojroll: ok. docs are a 'new' thing though so thought i should check.14:08
jrollya, fair14:09
rloojroll: i mean, what's the risk of updating a doc? :) but is that going to mean a big hassle, lots of minor doc changes being backported?14:09
jrollrloo: it's only a hassle for stable reviewers :P14:09
jrollthere's always a risk that the doc update is wrong14:09
rloojroll: yeah, which is why i'm asking you :) ok, so you're fine with it. It just needs a depends on thing then?14:10
jrollrloo: looks like it is chained properly14:10
*** MattMan has joined #openstack-ironic14:10
jrollsee 'related changes'14:10
sambettso/ rloo14:10
*** anush has quit IRC14:10
rloojroll: oh, i missed that. ok then it is good to go. (from my point of view!) thx.14:10
jrollrloo: :)14:10
lucasagomeshey rloo !14:11
*** vsaienko has quit IRC14:11
*** zhenguo has joined #openstack-ironic14:12
*** syed_ has joined #openstack-ironic14:13
rloojroll: wrt summit sessions. i most likely will not be in monday's meeting. is that when they will be decided?14:14
jrollrloo: I hope so14:14
rloojroll: ok, i'll add my feedback/questions to the etherpad then.14:14
jrollok14:14
jrollI just talked to nova people about #5, going to add details14:14
*** sdake has quit IRC14:15
*** rama_y has joined #openstack-ironic14:15
*** vsaienko has joined #openstack-ironic14:15
lucasagomesyeah I'll arrive on monday actually14:15
*** snecklifter has quit IRC14:15
lucasagomesmonday afternoon*14:16
rloolucasagomes: you flying back to dublin?14:16
jrolllucasagomes: rloo means next monday irc meeting14:16
lucasagomesrloo, yes, on friday14:16
lucasagomesright after the summit14:16
rloolucasagomes: oh, I think there is a disconnect here :) I mean this coming monday, our weekly ironic meeting. to discuss/decide which proposals for design sessions for the summit :)14:17
lucasagomesoh, damn I totally misread it all14:18
lucasagomessorry for that14:18
*** bjolo_ has quit IRC14:18
rloolucasagomes: no worries14:18
dtantsurmorning rloo14:18
*** athomas has quit IRC14:19
*** moshele has joined #openstack-ironic14:19
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Cleanup unused (i)PXE kernel parameters  https://review.openstack.org/38288914:19
dtantsurlucasagomes, FYI ^^^14:20
lucasagomesoh /me looks14:20
openstackgerritMerged openstack/ironic: Fix _lookup() method for node API routing  https://review.openstack.org/38285014:23
openstackgerritVladyslav Drok proposed openstack/ironic: Add user and project domains to ironic context  https://review.openstack.org/29537114:24
openstackgerritMerged openstack/ironic: Enable DeprecationWarning in test environments  https://review.openstack.org/37958114:24
*** athomas has joined #openstack-ironic14:25
*** mgoddard_ has joined #openstack-ironic14:29
*** mgoddard has quit IRC14:31
*** ijw has joined #openstack-ironic14:33
*** vsaienko has quit IRC14:34
sambettsvsaienk0: is this still a valid bug? https://bugs.launchpad.net/neutron/+bug/1492254 I assume with the new networking stuff its not relevent?14:34
openstackLaunchpad bug 1492254 in neutron "neutron should not try to bind port on compute with hypervisor_type ironic" [High,Confirmed] - Assigned to Sandhya Dasu (sadasu)14:34
vsaienk0sambetts: yes with multitenancy it is not relevant14:35
*** vsaienko has joined #openstack-ironic14:35
openstackgerritVadim Hmyrov proposed openstack/python-ironicclient: List required arguments in '--help' message in Ironic Client.  https://review.openstack.org/38299914:36
*** anush has joined #openstack-ironic14:36
*** hj-hpe has joined #openstack-ironic14:37
sambettsvsaienk0: do we want to close out this bug? Or do you still think it needs fixing? the assigny is on my team and wondering if its still open14:38
sambettsassignee*14:38
*** ijw has quit IRC14:38
*** anush has quit IRC14:43
vsaienk0sambetts: I think that it still should be fixed, we didn't deprecate flat case right?14:43
lucasagomesso strange, the ironic virt driver seems to be the only one implementing "rebuild" form the base virt interface http://paste.openstack.org/show/584694/14:44
* lucasagomes wonders how other drivers does it14:44
jrolllucasagomes: others don't support --preserve-ephemeral, so it's just delete/spawn on the same host14:44
jrollthat's the only reason we override that14:44
lucasagomesoh14:45
* lucasagomes disliked that flag already, now...14:45
*** _mbelanger has quit IRC14:47
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817314:47
*** wajdi has joined #openstack-ironic14:48
sambettsvsaienk0: no we didn't deprecate it, ideally I think we should be changing the vnic type to baremetal and having a always bind ML2 driver that checks if vnic_type is baremetal and device_owner is compute:None14:49
sambettsbut :/14:49
sambettsi think maybe we should have a special local link information case for when we're binding a statically configured flat network, so we don't pass any switch information14:51
openstackgerritStephanie Miller proposed openstack/bifrost: Add ability to write clouds.yaml  https://review.openstack.org/38022714:52
sambettse.g. local_link_information { switch_id: "static", port_id: "static", switch_info: "" }14:53
sambettsand have a special ml2 driver that only handles that ^ LLC info14:53
sambettsso it can always bind it14:54
*** ildikov has quit IRC14:54
*** anush has joined #openstack-ironic14:55
*** mgkwill has quit IRC14:55
*** boris-42 has quit IRC14:55
*** izaakk has quit IRC14:55
*** pas-ha has quit IRC14:55
*** patrickeast has quit IRC14:55
vsaienk0sambetts: it is only related to flat network I already propose it https://review.openstack.org/#/c/339129/8/neutron/plugins/ml2/drivers/mech_baremetal/mech_driver/mech_baremetal.py14:55
*** fyxim has quit IRC14:55
patchbotpatch 339129 - neutron - Add BaremetalMechanismDriver driver.14:55
*** _vsaienko has joined #openstack-ironic14:56
*** vsaienko has quit IRC14:56
sambettsNice! then I would say that this is the fix for that bug14:57
vsaienk0sambetts: could you please ask your team to review it than :)14:57
xavierranyone knows what that means? https://releases.openstack.org/newton/schedule.html#newton-cycle-trailing-deadline14:58
sambettsxavierr: its for projects that need to wait until the current OpenStack release has already release to release their project14:59
*** anush has quit IRC14:59
xavierrsambetts: ohh, ty sam15:00
*** anush_ is now known as anush15:00
*** anush_ has joined #openstack-ironic15:00
*** ccamacho has quit IRC15:01
*** mjura has quit IRC15:03
*** bdeschenes has joined #openstack-ironic15:03
*** _vsaienko has quit IRC15:08
*** aNuposic has joined #openstack-ironic15:10
*** rajinir has joined #openstack-ironic15:10
xavierrany idea in when will be the deadline for solving bugs patches that will be backported to newton?15:11
vsaienk0sambetts: just checked comments on https://review.openstack.org/#/c/339129 I'm not sure that neutron folks will be ok to keep ironic related things in newtron project, we can use networking-generic-switch for that https://review.openstack.org/#/c/35777915:11
*** fyxim has joined #openstack-ironic15:11
patchbotpatch 339129 - neutron - Add BaremetalMechanismDriver driver.15:11
*** _vsaienko has joined #openstack-ironic15:11
patchbotpatch 357779 - networking-generic-switch - Fake binding of baremetal port in FLAT net15:11
vsaienk0jroll what do you think ^^15:12
sambettsvsaienk0: it seems what Ihar is suggesting is that we put that ml2 driver in the Ironic tree15:12
sambettsvsaienk0: because they don't carry any vendor/project specifc code in their repo anymore15:13
vsaienk0sambetts: I don't think we should keep it in ironic tree, it will require installing ironic on node with neutron-server15:13
jrollyeah, pls not ironic tree15:13
vsaienk0sambetts: I wondered if we can use networking-generic-switch for that, we already using it for gate testing15:14
jrollI'm fine with that, though seems like maybe it should be a separate thing?15:14
sambettsI think this ^15:14
jrollopenstack/ironic-flat-ml2 or something15:14
*** _milan_ has quit IRC15:14
xavierrhey jroll any idea to my last question? :)15:14
*** milan has joined #openstack-ironic15:15
jrollxavierr: the deadline would be whenever newton is end-of-life, typically 15 months or so15:15
*** lucasagomes is now known as lucas-hungry15:15
jrollxavierr: there's different levels of stable maintenance, though, after ocata release newton will likely be critical/security fixes only15:15
jrollxavierr: again, that's listed on the stable branch guide http://docs.openstack.org/project-team-guide/stable-branches.html15:15
mariojvrloo: hey, thanks for reviewing again. on https://review.openstack.org/#/c/321865/38/ironic/conductor/notification_utils.py, do you think we should just catch a generic Exception if we merge the messages?15:15
patchbotpatch 321865 - ironic - Add power state change notifications15:15
*** rbrud has joined #openstack-ironic15:16
mariojvi think LOG.exception is better to keep a traceback, but imbw15:16
rloohi mariojv. i dunno. i hate dealing with exceptions.15:16
vsaienk0jroll: why not use networking-generic-switch we already have a repo, and it helps us to test multitenancy15:16
rloomariojv: what do you think about spamming the log file with lots of these errors?15:16
rloomariojv: is that going to be a problem?15:16
mariojvrloo: i think it's a good thing15:16
mariojvrloo: if something's wrong, you may be spamming the message bus inappropriately15:16
mariojvrloo: i'm +1 to merging the messages though15:17
jrollvsaienk0: it means people must install all of that repo to handle flat networks - which isn't terrible but it seems like it would be nice to say "if you're using flat, use foo, for multitenant, use bar"15:17
*** r-daneel has joined #openstack-ironic15:17
xavierrok jroll, thanks :)15:18
rloomariojv: do you think that all those exceptions are appropriate for spamming the log file?15:18
vsaienk0jroll: there is another thing, I've worked on ironic-neutron event synchronization, and It may be done in ML2, so in future we can use ML2 to sync events with neutron15:18
*** ccamacho has joined #openstack-ironic15:18
mariojvrloo: yes - most of those are coding failures, so they'd be useful during development15:18
vsaienk0jroll: actually I've implemented them on top of ML2 https://review.openstack.org/#/c/357780/15:19
patchbotpatch 357780 - networking-generic-switch - WIP: Enable ironic notifications15:19
mariojvrloo: the only one that's not a coding error is MessageDeliveryFailure, which indicates an issue with the message bus, or generic Exception15:19
rloomariojv: in that case, there is no real diff between the two except's, right?15:19
mariojvrloo: yeah - i'm fine with  removing "Unknown error"15:19
mariojvi think it's redundant15:19
jrollvsaienk0: awesome, maybe that's a reason to do one ML2 plugin, though we could make it a library too15:19
mariojvrloo: also, not sure if ValueError subclasses Exception (I think it does, from python docs)15:20
mariojvrloo: but, isinstance(ValueError, Exception) returns False15:20
rloomariojv: it is good to know what exceptions we might expect, vs what exceptions happen that aren't expected.15:20
vsaienk0jroll since we already have networking-generic-switch I'm wondered why we need to create new ironic-** :)15:20
mariojvrloo: okay - so you want to keep the "Unknown error" piece?15:20
rloomariojv: i thought all exceptions were handled by Exception15:20
mariojvrloo: they are15:21
mariojvrloo: oh, for the ValueError thing15:21
mariojvrloo: yeah, i think that's accurate15:21
vsaienk0jroll: probably it is nicer to have ironic- in the project name15:21
jrollvsaienk0: I just told you the reason I have. I'm not opposed to either way, would be curious what others think15:21
rloomariojv: no, i think the log string can be the same. i am just wondering about the coding. one might just do except Exception.., but i think it is worthwhile listing the Notification*, along with Exception. but if we did that, someone will come along and delete the Notification* ones cuz you know, Exception handles them all :)15:22
mariojvhaha. ok, i think it's useful to list them out too15:22
mariojvrloo: so, the only thing i will change there is removing the isinstance check. sound ok?15:22
vsaienk0jroll: I will rise this question at openstack-dev if you do not mind than15:22
rloomariojv: we need to think about future maintainability of our code... (seriously)15:22
jrollvsaienk0: sure15:22
mariojvrloo: is there some openstack guideline about this? i'll look15:23
rloomariojv: yeah, remove the isinstance check and maybe combine the two excepts into one, with a comment about leaving the non Exception exceptions there :)15:23
mariojvoh, the two excepts are already combined i think15:23
mariojvi will definitely leave the comment15:23
rloomariojv: no guideline. my thoughts based on seeing patches...15:24
mariojvi think it's a good one15:24
dtantsurmariojv, isinstance(ValueError, Exception) is definitely False, but I'd expect issubclass(ValueError, Exception) to be True ;)15:24
*** boris-42 has joined #openstack-ironic15:24
rloomariojv: i mean, no guideline written down that i know of :)15:24
dtantsurmariojv, only StopIteration and SystemExit do not inherit Exception IIRC15:24
mariojvthanks dtantsur15:24
dtantsur... and maybe KeyboardInterrupt too, don't quite remember15:25
JayFtil15:26
JayFdtantsur: why not? (RTFM is an OK answer if you want :P)15:26
dtantsurJayF, these exceptions are "special", that's why in Python 2 they only inherited BaseException, not sure about Python 315:26
dtantsuralso note that I can be wrong, worth double-checking :)15:27
JayFdtantsur: I guess so if someone does except Exception they still break out?15:27
* JayF wondering how much "fun" you could have except BaseException15:27
rloooh, hi JayF. Did you know about our admin guide: http://docs.openstack.org/admin-guide/baremetal.html15:27
dtantsurright. e.g. so that sys.exit still work15:27
*** aNuposic has quit IRC15:27
dtantsurJayF, just plain "except" is the same, hence pep8 doesn't like it15:27
JayFrloo: yep, I'm talking to docs team, trying to see what work is required to move that in-tree15:27
rlooJayF: thx!15:27
*** mgkwill has joined #openstack-ironic15:27
JayFrloo: basically same as install guide, it's a completely separate repo and they have to do the work to distribute it across. it's on my list of docs-things I'd like in ocata, but we're mostly just along for the ride for that one15:28
* dtantsur checked15:28
rlooJayF: yeah, would be nice if the docs folks had told us about that, instead of some people adding stuff there that is from our developer docs.15:28
dtantsurmy bad, folks, StopIteration is an Exception actually. everything else seems correct.15:28
JayFrloo: Yeah. Perhaps a good idea to remove some of that and just link back to us for the meantime?15:29
rlooJayF: oh, don't ask me. seems like 3 options: 1. leave status quo; 2. remove that/link back; 3. leave that, remove developer & link to15:31
JayFYeah. I'll dig in and try to find out what a timeline (if any) for doing admin-guide in-tree.15:32
JayFthat should inform whatever action we take15:33
rlooJayF: makes sense. thx for looking after that!15:34
JayFno problem, trying to do a little better job of tracking all this. So far I have a list, but nothing scratched off. lol15:34
*** ijw has joined #openstack-ironic15:34
rlooJayF: you could open bugs/rfes for whatever is on your list...15:36
JayFI should for the actionable items. /me adds that to the list15:36
JayFSome of it is stuff like "find out the timeline for admin guide in tree" which isn't really bug-trackable15:36
rlooJayF: being a doc liaison doesn't mean you have to actually do the changes or whatever. (although great that you are).15:36
JayFOh, believe me, I think I do a pretty solid job of the file-bug-get-my-problem-solved trick15:37
JayFlol15:37
*** patrickeast has joined #openstack-ironic15:37
JayFit's the magic of working on open source. your problems magically become everyone's problems just by filling out a form :P15:37
rlooJayF: :D15:37
*** ildikov has joined #openstack-ironic15:38
*** izaakk_ has joined #openstack-ironic15:38
*** [1]cdearborn has quit IRC15:39
*** ijw has quit IRC15:40
openstackgerritRuby Loo proposed openstack/ironic-lib: followup to disk_utils.count_mbr_partitions()  https://review.openstack.org/38304315:41
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for openstack baremetal node commands.  https://review.openstack.org/38304415:41
*** pas-ha has joined #openstack-ironic15:41
*** moshele has quit IRC15:42
rloolucas-hungry: ^^ https://review.openstack.org/383043 if you have a minute. literally, maybe 2 minutes :)15:43
patchbotpatch 383043 - ironic-lib - followup to disk_utils.count_mbr_partitions()15:43
JayFI always feel bad when I miss a typo in something I landed /o\15:44
JayFty for fixing it rloo, +2'd15:44
rlooJayF: easy to miss those15:45
*** karthi has quit IRC15:50
*** ohamada has quit IRC15:53
*** _vsaienko has quit IRC15:58
*** rcernin has quit IRC16:01
*** vsaienko has joined #openstack-ironic16:01
*** izaakk_ has quit IRC16:02
*** jheroux has quit IRC16:02
*** izaakk has joined #openstack-ironic16:02
*** rpioso has joined #openstack-ironic16:03
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent: Remove Python 2.6 format style  https://review.openstack.org/38261616:05
*** cdearborn has joined #openstack-ironic16:05
*** jheroux has joined #openstack-ironic16:06
jlvillaljroll: dtantsur: Stable patches: https://review.openstack.org/#/c/375585/  &  https://review.openstack.org/#/c/375590/16:07
patchbotpatch 375585 - ironic (stable/mitaka) - Increase timeout for migration-related tests16:07
patchbotpatch 375590 - ironic (stable/newton) - Increase timeout for migration-related tests16:07
openstackgerritVasyl Saienko proposed openstack/ironic: Ironic devstack portgroup support  https://review.openstack.org/38174316:08
openstackgerritVasyl Saienko proposed openstack/ironic: Tempest tests for portgroups  https://review.openstack.org/38247616:08
dtantsurjlvillal, +A16:09
jlvillaldtantsur: Thanks16:09
*** e0ne_ has quit IRC16:09
*** lucas-hungry is now known as lucasagomes16:12
lucasagomesrloo, thanks! will take a look16:12
*** jistr is now known as jistr|afk16:14
*** vsaienko has quit IRC16:15
NobodyCamGood Morning Ironic'ers :)16:16
dtantsurmorning NobodyCam16:16
NobodyCamhey hey dtantsur :)16:16
jlvillalmariojv: isinstance(ValueError, Exception) is True. Sort of16:16
*** jcoufal has quit IRC16:16
*** strigazi is now known as strigazi_AFK16:17
jlvillalmariojv: http://paste.openstack.org/show/584715/16:17
openstackgerritMerged openstack/ironic: Increase timeout for migration-related tests  https://review.openstack.org/37559016:17
openstackgerritMerged openstack/ironic: Increase timeout for migration-related tests  https://review.openstack.org/37558516:17
jlvillalmariojv: >>> isinstance(ValueError(), Exception)16:18
jlvillalTrue16:18
openstackgerritharis proposed openstack/python-ironicclient: Enables ironic list commands to use labels  https://review.openstack.org/38067116:18
mariojvoh, weird16:18
mariojvthat kind of makes sense actually16:18
jlvillalmariojv: Just 'ValueError' is the function, not the object. I think.16:18
mariojvjlvillal: that's exactly what's happening16:18
mariojvisinstance(ValueError, Exception) without instantiating the object is False16:19
jlvillalYep16:19
*** vsaienko has joined #openstack-ironic16:19
mtaninoHi, quick question. Are there a way to recover an instance from "clean wait" Provisioning State?16:19
mtaninolike reset-state command.16:20
*** jcoufal has joined #openstack-ironic16:20
jlvillalmtanino: That's a classic question! I think have to wait for it to time-out :(16:20
lucasagomesrloo, opssss thanks for that16:21
jlvillalmtanino: Or be reckless and daring and go modify the database...16:21
* jlvillal wonders if we need an FAQ for that.16:21
jlvillalmtanino: Hopefully if I'm wrong, someone will correct me :)16:21
rloolucasagomes: yw16:21
mtaninojlvillal: I see. seems need on FAQ!16:21
mtaninojistr|afk: change the Provisioning State value to "available", right?16:22
mtaninosorry jistr|afk16:22
mtaninojlvillal: change the Provisioning State value to "available", right?16:23
mtaninoand then power off16:23
dtantsurmtanino, I think you can try to "abort" cleaning16:23
NobodyCammorning lucasagomes rloo jlvillal milan16:23
lucasagomesNobodyCam, hi there16:23
dtantsurlike, $ ironic node-set-provision-state $NODE abort16:24
*** penick has joined #openstack-ironic16:24
milanmorning NobodyCam! :)16:24
NobodyCam:)16:24
mtaninodtantsur: thanks, let me try the command.16:24
rloomorning NobodyCam!16:24
lucasagomesdtantsur, mtanino I think it's only cleanwait, but worth trying16:24
NobodyCam:)16:24
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817316:24
dtantsurlucasagomes, well, we're talking about cleanwait, no?16:24
mtaninolucasagomes: sure.16:24
*** ccamacho1 has joined #openstack-ironic16:25
vdrokmorning NobodyCam and jlvillal16:25
NobodyCammorning vdrok :)16:25
*** ccamacho has quit IRC16:26
jlvillalHi vdrok :)16:27
*** vsaienko has quit IRC16:28
*** kencjohnston has joined #openstack-ironic16:29
*** joprovost has joined #openstack-ironic16:30
mtaninodtantsur: The status was changed to "clean failed"... so then need to edit database directly?16:30
vgadirajmtanino, see if this state machine diagram for Ironic node is of any help http://docs.openstack.org/developer/ironic/_images/states.svg16:30
*** vsaienko has joined #openstack-ironic16:31
mtaninovgadiraj: thanks. There is no arrow to recover from clean failed...16:31
dtantsurmtanino, I highly recommend you never edit database directly. now IIRC you can do $ ironic node-set-provision-state $NODE manage16:31
*** tesseract- has quit IRC16:31
vgadirajthere is not :(16:31
dtantsurthen your nodes will be in manageable, you can figure out what went wrong during cleaning, fix it, retry..16:32
mtaninodtantsur: vgadiraj: Good the status is changed to "manageable", so I think I can try the inspect command again16:33
*** zhenguo has quit IRC16:33
jlvillalvgadiraj: mtanino: Can go from "Clean Failed" to manageable. Based on diagram16:33
mat128mtanino, vgadiraj you can go back to manageable via "manage"16:33
mat128jlvillal: ;)16:33
vgadirajyes i was about to say that16:33
jlvillalvgadiraj: :)16:33
mtaninomat128: :)16:33
vgadirajgood catch :)16:33
mat128mtanino, vgadiraj: there is no dead end16:34
mtaninonow we are on same page ;)16:34
*** mat128 is now known as mat128|lunch16:34
jlvillalmat128|lunch: http://www.funny-potato.com/images/dead-end.jpg16:34
mtaninomat128|lunch: ah, now I find "manage" arrow on the figure.16:34
*** joprovost1 has joined #openstack-ironic16:35
vgadirajmat128 is right, there is no dead end16:35
*** ijw has joined #openstack-ironic16:36
*** joprovost has quit IRC16:36
*** joprovost1 is now known as joprovost16:36
*** ijw has quit IRC16:40
*** Romanenko_K has quit IRC16:41
jlvillaljroll: Thanks for: https://review.openstack.org/#/c/381934/  I just did +A.  Tested it on my IPA patch and it is passing the gate jobs :)16:42
patchbotpatch 381934 - ironic - Set SUBNETPOOL_PREFIX_V4 to FIXED_RANGE16:42
JayF\o/16:43
openstackgerritMerged openstack/ironic-lib: followup to disk_utils.count_mbr_partitions()  https://review.openstack.org/38304316:45
jrolljlvillal: awesome, ty16:47
jlvillal:)16:48
jlvillalrloo: Nice catch on the mbr_partition patch :)16:51
* jlvillal reviewed it about six times and missed that missing variable in the string :(16:52
rloojlvillal: thx :)16:52
rloojlvillal: i find that after reviewing something more than twice, i'm not as effective any more.16:52
jlvillal+116:52
rloojlvillal: we should all just get our patches right the first time! :)16:52
JayFNothing in my job makes me more nervous than a patch being accepted without negative reviews ever.16:54
JayFIt doesn't happen often, but it just makes me nervous as hell16:54
JayF"I had to have done something wrong! Look harder!"16:54
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/38311216:55
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/38311316:55
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/37744616:55
*** derekh has quit IRC16:56
*** jtomasek_ has joined #openstack-ironic16:56
*** jistr|afk is now known as jistr16:57
*** vsaienko has quit IRC16:57
jrolllots of ironic shoutouts here \o/ https://www.openstack.org/news/view/259/openstack%C2%AE-newton-release-advances-one-cloud-platform-for-virtualization,-bare-metal-and-containers16:58
*** jcoufal_ has joined #openstack-ironic16:59
jrollbtw, if you hadn't heard yet, newton final just shipped16:59
*** vsaienko has joined #openstack-ironic16:59
rlooJayF: you have to think positively. I look forward to patches that have no negative reviews!16:59
jrollgreat work from everyone this cycle, we did lots of awesome things <316:59
rloojroll: that's great!16:59
*** ralonsoh has quit IRC16:59
JayFIn honor of newton, everyone hit yourself on the head with an apple, then yell "eureka!" and eat it.16:59
dtantsur\o/17:00
* rloo thinks americans have strange customs17:00
jrollrloo: only JayF, don't worry17:00
JayFrloo: In reference to one of our strange customs that have leaked into Canada, congrats to the Blue Jays \o/17:00
rlooJayF: did they win? not yet, right?17:01
mariojveveryone code on this thing for next release https://en.wikipedia.org/wiki/Apple_Newton17:01
JayFrloo: they won the wild-card play in game, so they're in the best-of-5 divisional series17:01
*** rama_y has quit IRC17:01
*** sdake has joined #openstack-ironic17:01
rlooJayF: i only cheer for the winner! :)17:01
*** jcoufal has quit IRC17:01
dtantsursee you tomorrow17:02
*** dtantsur is now known as dtantsur|afk17:02
NobodyCamnight dtantsur17:02
rloobye dtantsur|afk17:02
milannight dtantsur|afk! :)17:03
*** vsaienko has quit IRC17:11
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817317:11
*** jpich has quit IRC17:14
*** cdearborn has quit IRC17:18
*** vsaienko has joined #openstack-ironic17:18
*** rbartal has quit IRC17:23
*** bnemec has quit IRC17:25
*** jvgrant_ has quit IRC17:28
*** rbrud has quit IRC17:31
jlvillaljroll: Cool to see us getting some love :)17:33
jlvillaljroll: Not sure if you saw:  https://youtu.be/z6ftW7fUdp4?t=2m28s  Half of the video is on Ironic.17:33
*** cdearborn has joined #openstack-ironic17:34
*** vsaienko has quit IRC17:35
*** ijw has joined #openstack-ironic17:36
*** athomas has quit IRC17:38
*** bnemec has joined #openstack-ironic17:38
jlvillalvsaienk0: Are you here?17:40
*** vsaienko has joined #openstack-ironic17:40
*** Vinsh has quit IRC17:41
*** jjohnson2 has quit IRC17:41
*** trown is now known as trown|lunch17:41
*** ijw has quit IRC17:41
milanGood night Ironic! :)17:45
* milan gone17:45
*** milan has quit IRC17:45
*** bnemec has quit IRC17:47
openstackgerritMerged openstack/ironic: Set SUBNETPOOL_PREFIX_V4 to FIXED_RANGE  https://review.openstack.org/38193417:47
openstackgerritMerged openstack/ironic: Have bashate run for entire project  https://review.openstack.org/37426917:48
jlvillalSweet. IPA gate works again :)17:48
*** vsaienko has quit IRC17:48
sambettszo/17:48
sambetts\o/17:48
mat128|lunchjlvillal: nice video! :)17:50
jlvillalmat128|lunch: I liked seeing it :)17:50
openstackgerritVladyslav Drok proposed openstack/ironic: Remove unnecessary option from plugin settings  https://review.openstack.org/37550117:53
vdrokanother easy one ^^17:54
*** bnemec has joined #openstack-ironic17:57
openstackgerritVladyslav Drok proposed openstack/ironic: Clean up provision ports when reattempting deploy  https://review.openstack.org/34839217:57
*** rama_y has joined #openstack-ironic17:57
*** rbrud has joined #openstack-ironic17:59
*** priteau has joined #openstack-ironic18:01
*** aNuposic has joined #openstack-ironic18:02
*** bnemec has quit IRC18:03
openstackgerritVladyslav Drok proposed openstack/python-ironicclient: Add docs for create command  https://review.openstack.org/35802518:04
jlvillalvdrok: Famous last words :P18:05
vdrokheh18:05
*** cdearborn has quit IRC18:06
vdrokgood night everyone@!18:06
jlvillalvdrok: Paka paka18:07
rloovdrok: baka baka (no idea what that means)18:08
jlvillalheh rloo18:08
*** bnemec has joined #openstack-ironic18:08
*** vsaienko has joined #openstack-ironic18:09
jlvillalAnother video: https://www.youtube.com/watch?v=I7rXYQZwM-k18:10
*** cdearborn has joined #openstack-ironic18:10
*** vsaienko has quit IRC18:14
*** vsaienko has joined #openstack-ironic18:17
*** bnemec has quit IRC18:17
openstackgerritAndreas Jaeger proposed openstack/bifrost: Enable release notes translation  https://review.openstack.org/38320218:24
*** mat128|lunch is now known as mat12818:27
openstackgerritAndreas Jaeger proposed openstack/ironic: Enable release notes translation  https://review.openstack.org/38321718:27
*** bnemec has joined #openstack-ironic18:27
openstackgerritAndreas Jaeger proposed openstack/ironic-inspector: Enable release notes translation  https://review.openstack.org/38321818:27
openstackgerritAndreas Jaeger proposed openstack/ironic-python-agent: Enable release notes translation  https://review.openstack.org/38321918:27
openstackgerritAndreas Jaeger proposed openstack/ironic-ui: Enable release notes translation  https://review.openstack.org/38322118:28
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM: Testing multi-node  https://review.openstack.org/38324118:31
TheJuliamariojv: Thank you for the review on the BFV spec.  Out of curiosity, will you be at the BCN summit?18:32
mariojvTheJulia: unfortunatley not18:33
mariojv*unfortunately18:33
*** bnemec has quit IRC18:33
TheJuliaAww :(18:33
*** bnemec has joined #openstack-ironic18:34
TheJuliamariojv: Anyway, I think I was a bit verbose on my replies, but again thank you for the awesome questions :)18:36
*** Sukhdev has joined #openstack-ironic18:36
mariojvno worries! I am also usually really verbose, thanks for responding18:36
mariojvi'll try to review again today or tomorrow18:36
*** ijw has joined #openstack-ironic18:37
*** trown|lunch is now known as trown18:39
*** bnemec has quit IRC18:41
*** vsaienko has quit IRC18:44
*** ijw has quit IRC18:45
xavierrJayF around?18:46
JayFI am, but about to go to lunch shortly18:46
JayFwhat's up?18:46
xavierryou posted a comment in OneView inspect interface patch asking for the docs18:47
rama_yHi mariojv: I am trying to build and test IPA; I built an iso image using tinyipa and uploaded it to glance and then updated a baremetal node with the new glance image18:47
JayFxavierr: yes :D18:47
xavierrhahaha18:47
mariojvrama_y: hi!18:47
rama_ymariojv: Hi18:47
mariojvrama_y: why don't we work on it in a video chat? i'll pm you details18:47
xavierrJayF: this is the docs I've asked you to look https://review.openstack.org/#/c/369737/18:47
patchbotpatch 369737 - ironic - OneView driver docs explaining hardware inspection18:47
xavierrJayF: I'll do a rebase after your review, right?18:48
rama_ymariojv: the node failed to deploy when I tried node set-provision-state active18:48
xavierr:)18:48
JayFsure, I'll look18:48
xavierro/18:48
rama_ymariojv: thanks.18:48
*** krtaylor has joined #openstack-ironic18:48
JayFxavierr: yep, that looks like exactly what I was looking for, I'll proofread it and let you know18:48
*** vsaienko has joined #openstack-ironic18:49
xavierrawesome, JayF18:49
openstackgerritAndreas Jaeger proposed openstack/python-ironicclient: Enable release notes translation  https://review.openstack.org/38337218:50
JayFxavierr: one super minor nit inline, which I think might even be a UK vs US english thing so nbd18:50
openstackgerritAndreas Jaeger proposed openstack/python-ironic-inspector-client: Enable release notes translation  https://review.openstack.org/38337318:50
mariojvrama_y: you may find this state diagram useful; it should you what states ironic should go thorugh during its entire lifecycle: http://docs.openstack.org/developer/ironic/_images/states.svg18:50
*** harlowja has quit IRC18:50
mariojvrama_y: i've never attempted to deploy a node with ironic without nova, but let's look into the specific errorss18:50
mariojv*errors18:50
jrolljlvillal: oh wow, neat! hogepodge, is that newton demo your thing?18:51
xavierrJayF: I'll do it now, ty!18:51
*** hwoarang has quit IRC18:51
*** hwoarang has joined #openstack-ironic18:52
TheJuliarama_y: Was there anything in the node's last_error field?18:53
*** vsaienko has quit IRC18:53
*** bnemec has joined #openstack-ironic18:54
xavierrmorning TheJulia :D18:55
TheJuliagood afternoon xavierr18:55
*** mkam has joined #openstack-ironic18:55
xavierro/18:55
*** mkam has quit IRC18:55
aNuposicrama_y: what errors are you getting?18:56
*** jcoufal_ has quit IRC18:56
*** hwoarang has quit IRC18:57
*** hwoarang has joined #openstack-ironic18:58
*** jtomasek_ has quit IRC19:00
*** aNuposic has quit IRC19:05
*** vsaienko has joined #openstack-ironic19:06
*** jcoufal has joined #openstack-ironic19:06
openstackgerritXavier proposed openstack/ironic: OneView driver docs explaining hardware inspection  https://review.openstack.org/36973719:06
*** absubram has joined #openstack-ironic19:08
*** mgoddard has joined #openstack-ironic19:10
*** mgoddard_ has quit IRC19:10
*** bnemec has quit IRC19:10
*** mkam has joined #openstack-ironic19:10
*** mkam has quit IRC19:10
*** bnemec has joined #openstack-ironic19:11
*** vsaienko has quit IRC19:12
*** mgoddard has quit IRC19:15
* sambetts -> home19:15
sambettsnight all19:15
*** sambetts is now known as sambetts|afk19:16
jlvillalsambetts|afk, Good night19:16
mariojvTheJulia: was just an issue with trying to use IPA as a deploy image instead of as an agent19:16
sambetts|afknight jlvillal19:16
mariojvworking fine now, i think :)19:16
mariojvare there IPA API docs?19:16
mariojvbesides the generated docstrings19:16
jrollnah19:16
TheJuliamariojv: I was suspecting as such for the deployment based on the comments, but I wasn't sure19:16
mariojvi suspected not, since it's such a small api19:17
mariojvimage wasn't built to allow SSH, so we're just running GET /v1/commands to verify it's running19:17
jrollmmm19:17
mariojvi think that's possible with default devstack installation, not sure though19:17
jrollyeah, there's no config for the API or anything19:18
rama_yaNuposic: IPMI call failed: power status.19:18
*** ElCoyote_ has left #openstack-ironic19:19
rama_yTheJulia: yes, there is something in last_error: Failed to prepare to deploy. Error: IPMI call failed: power status.19:20
rama_yTheJulia, aNuposic: sorry for the late response.19:21
*** ElCoyote_ has joined #openstack-ironic19:24
*** hwoarang has quit IRC19:25
*** hamzy has quit IRC19:26
*** RomanenkoK has joined #openstack-ironic19:27
*** joprovost has quit IRC19:28
*** Goneri has quit IRC19:28
*** rama_y has quit IRC19:28
*** rloo has quit IRC19:29
*** hwoarang has joined #openstack-ironic19:32
*** penick has quit IRC19:34
*** hamzy has joined #openstack-ironic19:37
*** ijw has joined #openstack-ironic19:37
openstackgerritJulia Kreger proposed openstack/bifrost: Adding keystone support to inspector config  https://review.openstack.org/38342519:40
openstackgerritJulia Kreger proposed openstack/bifrost: Add test script for Bifrost using keystone  https://review.openstack.org/38342619:40
openstackgerritJulia Kreger proposed openstack/bifrost: Update the task label for inspection  https://review.openstack.org/38342719:40
jrollgonna dump some easy reviews here19:41
jrollhttps://review.openstack.org/#/c/383217/119:41
patchbotpatch 383217 - ironic - Enable release notes translation19:41
jrollhttps://review.openstack.org/#/c/383372/119:42
patchbotpatch 383372 - python-ironicclient - Enable release notes translation19:42
openstackgerritJulia Kreger proposed openstack/bifrost: Adding keystone support to inspector config  https://review.openstack.org/38342519:42
jrollbetherly: https://review.openstack.org/#/c/383221/119:43
patchbotpatch 383221 - ironic-ui - Enable release notes translation19:43
jrollor TheJulia ^ TIL you're core there :)19:43
jrolland inspector cores, https://review.openstack.org/#/c/383218/119:43
patchbotpatch 383218 - ironic-inspector - Enable release notes translation19:43
TheJuliajroll: you wnat me to +A them ? :)19:43
jrollTheJulia: yes please :)19:43
TheJuliak19:44
jrollthere's a dev list thread about it19:44
*** vsaienko has joined #openstack-ironic19:44
* jroll wishes dev list had a header with a link to the archives19:44
TheJuliaahh, I've had my head down making bifrost speak keystone19:44
*** rbrud has quit IRC19:44
TheJuliaof sorts19:44
jrollnice19:44
jrollthis email is less than an hour old so I didn't expect you saw it yet :P19:45
*** penick has joined #openstack-ironic19:47
*** vsaienko has quit IRC19:48
TheJuliaexcellent19:49
TheJuliaand \o/ ironic-ui translation in russian19:49
* TheJulia did not expect that19:50
jrolloh nice19:51
*** Sukhdev has quit IRC19:51
*** ijw has quit IRC19:52
*** penick has quit IRC19:53
*** penick has joined #openstack-ironic19:54
openstackgerritMerged openstack/ironic: Enable release notes translation  https://review.openstack.org/38321719:56
openstackgerritMerged openstack/ironic-ui: Enable release notes translation  https://review.openstack.org/38322119:56
openstackgerritMerged openstack/python-ironicclient: Enable release notes translation  https://review.openstack.org/38337219:56
*** rama_y has joined #openstack-ironic19:56
*** rbrud has joined #openstack-ironic19:59
*** jcoufal_ has joined #openstack-ironic20:00
openstackgerritJulia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml  https://review.openstack.org/38022720:03
*** jcoufal has quit IRC20:03
*** jcoufal_ has quit IRC20:05
*** sdake has quit IRC20:09
xavierrJayF: Jenkins voted. Could you +2 it? https://review.openstack.org/#/c/369737/20:11
patchbotpatch 369737 - ironic - OneView driver docs explaining hardware inspection20:11
*** harlowja has joined #openstack-ironic20:17
*** devananda|afk is now known as devananda20:18
*** e0ne has joined #openstack-ironic20:20
*** e0ne has quit IRC20:22
*** anush_ has quit IRC20:25
*** rloo has joined #openstack-ironic20:27
xavierrlucasagomes, are you ok +2 it? https://review.openstack.org/#/c/367065/ :)20:28
patchbotpatch 367065 - ironic - Add inspection feature for the OneView drivers20:28
lucasagomesxavierr, hi there20:29
lucasagomesxavierr, you mean +a ? I'm good with it (hence my +2 there)20:29
lucasagomesJayF, any concerns ?20:30
xavierrlucasagomes: sure +A :)20:30
*** rpioso has quit IRC20:33
xavierrty lucasagomes o/20:39
JayFlucasagomes: none, just was going to wait20:41
xavierrhi rloo, could you reapply your +2 again in this doc? https://review.openstack.org/#/c/369737/20:41
patchbotpatch 369737 - ironic - OneView driver docs explaining hardware inspection20:41
lucasagomesJayF, cool ty20:41
xavierrrloo: if possible +A it ;)20:41
JayFlucasagomes: aslezil is looking at the "make tempest verify cleaning actually happened" bug, I think she's reading some cleaning+tempest docs there, but if you have any ideas you might want to toss them her way or into the bug (https://bugs.launchpad.net/ironic/+bug/1586173) -- I know it's something you and jim had talked about at length at one point20:42
openstackLaunchpad bug 1586173 in Ironic "Tempest tests don't verify CLEANING state" [High,Triaged] - Assigned to Annie Lezil (annie-lezil)20:42
*** bfournie has quit IRC20:42
lucasagomesJayF, yes we did! I will add a comment20:43
lucasagomesthanks for pointing it out20:43
JayFno problem!20:43
*** rloo has quit IRC20:46
*** baoli_ has quit IRC20:46
*** aNuposic has joined #openstack-ironic20:46
*** baoli has joined #openstack-ironic20:48
*** lucasagomes is now known as lucas-afk20:53
aslezilJayF: Thank you :)20:54
JayFnp20:55
aslezillucasagomes: Thank you :)20:55
mrdaMorning Ironic20:59
hogepodgejroll: jlvillal: yeah, I've been really digging on ironic and dib (and bifrost and kolla) over the last few weeks21:01
hogepodgethere was more hand-waving than I would really normally want, but I wanted to show how much I like Ironic.21:03
* TheJulia smiles21:04
*** ijw has joined #openstack-ironic21:04
jrollhogepodge: awesome, thanks for featuring us, I hear there's more to come :)21:04
jlvillalhogepodge: Thanks :)21:04
*** penick has quit IRC21:05
hogepodgeWe're going to have a foundation lounge at the summit, and I just signed up to do a 15 minute live demo of Ironic on Wednesday.21:07
*** penick has joined #openstack-ironic21:07
*** ijw has quit IRC21:10
*** trown is now known as trown|outtypewww21:11
*** baoli_ has joined #openstack-ironic21:13
*** rbrud has quit IRC21:15
*** baoli has quit IRC21:17
*** harlowja has quit IRC21:17
*** michauds has joined #openstack-ironic21:18
*** dprince has quit IRC21:21
xavierrmorning mrda21:23
*** baoli_ has quit IRC21:23
aNuposicHi Folks, has anyone retried this bug https://bugs.launchpad.net/ironic/+bug/1626540 to check if it still exists?21:25
openstackLaunchpad bug 1626540 in Ironic "partprobe sometime fails with Device or resource busy" [Undecided,Incomplete]21:25
JayFDoesn't look like anyone has commented with attempts to reproduce.21:26
mrdao/21:26
JayFGiven it's marked incomplete, that usually means the reporter needs to provide more information in order to reproduce.21:26
*** bdeschenes has quit IRC21:29
*** rloo has joined #openstack-ironic21:32
openstackgerritJohn L. Villalovos proposed openstack/ironic: Increase verbosity of devstack/lib/ironic  https://review.openstack.org/37049221:34
*** rloo_ has joined #openstack-ironic21:34
jlvillalI would like to get ^^^^ in to help debugging. As currently we can't see what the values are for the various variables in devstack/lib/ironic21:37
*** cdearborn has quit IRC21:38
*** harlowja has joined #openstack-ironic21:39
*** harlowja has quit IRC21:39
*** harlowja has joined #openstack-ironic21:39
aNuposicJayF: I see. Yeah then will wait for some steps to reproduce21:40
*** moshele has joined #openstack-ironic21:41
*** krtaylor has quit IRC21:42
openstackgerritMerged openstack/ironic: Add inspection feature for the OneView drivers  https://review.openstack.org/36706521:42
openstackgerritMerged openstack/ironic: OneView driver docs explaining hardware inspection  https://review.openstack.org/36973721:43
openstackgerritJohn L. Villalovos proposed openstack/ironic: Use the existing $HOME/.ssh/id_rsa public key  https://review.openstack.org/38348721:46
*** rcernin has joined #openstack-ironic21:47
*** bnemec has quit IRC21:47
*** jheroux has quit IRC21:49
*** aNuposic has quit IRC21:50
*** hj-hpe has quit IRC21:51
*** bdeschenes has joined #openstack-ironic21:55
*** rpioso has joined #openstack-ironic21:58
*** penick has quit IRC21:59
*** rcernin has quit IRC21:59
*** rcernin has joined #openstack-ironic21:59
*** absubram has quit IRC22:01
*** penick has joined #openstack-ironic22:04
*** ijw has joined #openstack-ironic22:06
*** raghusab has joined #openstack-ironic22:10
*** moshele has quit IRC22:11
*** ijw has quit IRC22:11
*** wajdi_ has joined #openstack-ironic22:12
*** sdake has joined #openstack-ironic22:13
*** ccamacho1 has quit IRC22:13
*** wajdi has quit IRC22:15
*** bdeschenes has quit IRC22:15
*** wajdi_ has quit IRC22:16
*** aNuposic has joined #openstack-ironic22:27
*** krtaylor has joined #openstack-ironic22:31
*** vishwanathj has quit IRC22:32
openstackgerritJohn L. Villalovos proposed openstack/ironic: Increase verbosity of devstack/lib/ironic  https://review.openstack.org/37049222:41
*** michauds has quit IRC22:44
*** rbudden has quit IRC22:52
*** RomanenkoK has quit IRC23:00
*** ijw has joined #openstack-ironic23:07
*** jcoufal has joined #openstack-ironic23:13
*** ijw has quit IRC23:14
mtaninoHi, does anyone have an experience(error) to build ironic environment on VM? http://paste.openstack.org/show/584748/23:19
mtaninoexperience of the error23:23
*** amotoki has quit IRC23:24
*** priteau has quit IRC23:26
*** rpioso has quit IRC23:27
*** thrash is now known as thrash|g0ne23:33
*** rcernin has quit IRC23:36
openstackgerritMerged openstack/ironic: Remove unnecessary option from plugin settings  https://review.openstack.org/37550123:37
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM: Testing multi-node  https://review.openstack.org/38324123:43
*** dtantsur|afk has quit IRC23:53
*** jcoufal has quit IRC23:54
*** pcaruana has quit IRC23:55
*** jcoufal has joined #openstack-ironic23:57
« Wednesday, 2016-10-05 Index Friday, 2016-10-07 »

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