Friday, 2015-01-16

*** Marga__ has quit IRC00:02
*** Marga_ has joined #openstack-ironic00:02
*** Marga_ has quit IRC00:03
*** Marga_ has joined #openstack-ironic00:04
openstackgerritClif Houck proposed openstack/ironic: Fix Ironic Conductor so it shuts down gracefully  https://review.openstack.org/14732500:05
*** Marga_ has quit IRC00:06
*** Marga_ has joined #openstack-ironic00:06
*** Marga_ has quit IRC00:07
*** Marga_ has joined #openstack-ironic00:07
*** Marga_ has quit IRC00:10
*** Marga_ has joined #openstack-ironic00:11
*** Marga_ has quit IRC00:11
*** Marga_ has joined #openstack-ironic00:11
*** Marga_ has quit IRC00:12
*** Marga_ has joined #openstack-ironic00:12
*** EmilienM is now known as EmilienM|afk00:15
*** romcheg has quit IRC00:17
*** Ng has quit IRC00:21
*** Marga_ has quit IRC00:21
*** Marga_ has joined #openstack-ironic00:21
*** Ng has joined #openstack-ironic00:25
*** david-lyle has joined #openstack-ironic00:25
*** ijw_ has quit IRC00:26
*** Marga_ has quit IRC00:26
*** yuanying has quit IRC00:27
*** yuanying has joined #openstack-ironic00:37
*** sbadia has joined #openstack-ironic00:39
*** rodrigod` is now known as rodrigods00:41
*** cppforlife is now known as cppforlife_00:49
*** omnibus7 is now known as davidlenwell00:49
*** davidlenwell is now known as davidlenwell__00:50
*** davidlenwell__ is now known as davidlenwell00:50
*** achanda has joined #openstack-ironic01:01
*** achanda has quit IRC01:01
*** achanda_ has quit IRC01:04
*** david-lyle has quit IRC01:08
*** ryanpetrello has joined #openstack-ironic01:22
*** hshiina has joined #openstack-ironic01:22
*** david-lyle has joined #openstack-ironic01:26
*** openstack has joined #openstack-ironic01:35
*** ryanpetrello has quit IRC01:35
*** chenglch has joined #openstack-ironic01:36
*** cohn has left #openstack-ironic01:40
*** rloo has quit IRC01:45
*** Haomeng|2 has quit IRC01:53
*** igordcard has quit IRC01:55
*** david-lyle has quit IRC02:01
*** ChuckC has joined #openstack-ironic02:04
*** ChuckC_ has quit IRC02:05
*** dlaube has quit IRC02:26
openstackgerritDan Prince proposed openstack/ironic: iboot power driver: unbound variable error  https://review.openstack.org/14757502:35
*** david-lyle has joined #openstack-ironic02:42
*** david-lyle has quit IRC02:42
*** Marga_ has joined #openstack-ironic02:44
*** Marga_ has quit IRC02:46
*** Marga__ has joined #openstack-ironic02:46
openstackgerritjiangfei proposed openstack/python-ironicclient: ironicclient handle faultstring when using SessionClient  https://review.openstack.org/14202102:48
*** harlowja is now known as harlowja_away02:57
*** Marga__ has quit IRC03:08
openstackgerritMerged openstack/ironic: Rename localrc for local.conf  https://review.openstack.org/14753603:08
jrollhttps://review.openstack.org/#/c/102685/ has 3x+2, someone should land it (cleaning spec)03:23
jrollg'night everyone03:23
*** Nisha has joined #openstack-ironic03:27
*** naohirot has quit IRC03:29
*** hshiina has quit IRC03:30
*** lazy_prince is now known as killer_prince03:38
openstackgerritMerged openstack/ironic: improve the neutron configuration in install-guide  https://review.openstack.org/14334903:43
*** Marga_ has joined #openstack-ironic03:45
*** jerryz has joined #openstack-ironic03:53
*** hshiina has joined #openstack-ironic04:00
*** naohirot has joined #openstack-ironic04:04
*** dvorak has left #openstack-ironic04:31
*** Marga_ has quit IRC04:34
*** achanda has joined #openstack-ironic04:44
*** achanda has quit IRC04:46
*** ChuckC_ has joined #openstack-ironic04:54
openstackgerritDevananda van der Veen proposed openstack/ironic: Improve ironic-dbsync help strings  https://review.openstack.org/14774604:55
*** ChuckC has quit IRC04:55
*** achanda has joined #openstack-ironic04:57
*** achanda has quit IRC04:57
*** jiangfei|2 has quit IRC04:58
*** jiangfei|2 has joined #openstack-ironic04:58
*** jerryz has quit IRC05:02
*** Nisha has quit IRC05:03
*** ryanpetrello has joined #openstack-ironic05:13
*** jerryz has joined #openstack-ironic05:14
*** achanda has joined #openstack-ironic05:19
*** ryanpetrello has quit IRC05:24
*** Nisha has joined #openstack-ironic05:25
*** killer_prince is now known as lazy_prince05:31
*** rakesh_hs has joined #openstack-ironic05:37
*** Haomeng has joined #openstack-ironic05:37
*** anderbubble has joined #openstack-ironic05:40
*** rameshg87 has joined #openstack-ironic05:45
*** pensu has joined #openstack-ironic05:56
*** achanda has quit IRC06:03
*** achanda has joined #openstack-ironic06:05
*** chenglch has quit IRC06:07
*** chenglch has joined #openstack-ironic06:08
*** achanda has quit IRC06:16
*** pensu has left #openstack-ironic06:17
*** rakesh_hs has quit IRC06:20
*** rakesh_hs has joined #openstack-ironic06:21
*** achanda has joined #openstack-ironic06:23
*** yuanying_ has joined #openstack-ironic06:23
*** yuanying has quit IRC06:26
*** Nisha has quit IRC06:37
*** achanda has quit IRC06:46
*** Nisha has joined #openstack-ironic06:47
*** chlong has quit IRC07:20
*** Haomeng has quit IRC07:44
*** yarker has quit IRC07:44
*** Haomeng has joined #openstack-ironic07:45
*** yarker has joined #openstack-ironic07:46
*** anderbubble has quit IRC07:52
*** pensu1 has joined #openstack-ironic07:56
*** teju has joined #openstack-ironic08:01
*** yuanying_ has quit IRC08:06
*** alex_xu_ has quit IRC08:21
*** alex_xu has joined #openstack-ironic08:22
*** aszeszo has left #openstack-ironic08:22
*** dtantsur|afk is now known as dtantsur08:22
dtantsurMorning Ironic, TGIF!08:22
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: General naming and strings clean up  https://review.openstack.org/14744208:29
*** ifarkas has joined #openstack-ironic08:34
*** rwsu has quit IRC08:40
*** alexm has quit IRC08:41
*** jerryz has quit IRC08:41
*** pcrews has quit IRC08:42
*** hshiina has left #openstack-ironic08:46
*** dtantsur is now known as dtantsur|brb08:46
*** pradipta_away is now known as pradipta08:53
*** jerryz has joined #openstack-ironic08:56
*** Haomeng has quit IRC09:01
*** coolsvap|afk is now known as coolsvap09:03
*** Haomeng has joined #openstack-ironic09:03
*** yuanying has joined #openstack-ironic09:08
*** yuanying has quit IRC09:12
*** romcheg has joined #openstack-ironic09:19
*** jistr has joined #openstack-ironic09:19
*** derekh has joined #openstack-ironic09:26
Nishadtantsur|brb, morning09:30
*** Guest49876 is now known as d0ugal09:36
*** d0ugal has quit IRC09:36
*** d0ugal has joined #openstack-ironic09:36
*** pcrews has joined #openstack-ironic09:38
*** erwan_taf has joined #openstack-ironic09:38
*** jcoufal has joined #openstack-ironic09:38
*** pcrews has quit IRC09:43
*** ndipanov_gone is now known as ndipanov09:43
*** ndipanov has joined #openstack-ironic09:43
*** david-lyle has joined #openstack-ironic09:46
*** jiangfei|2 has quit IRC09:50
*** jiangfei|2 has joined #openstack-ironic09:54
*** chenglch has quit IRC09:55
*** andreykurilin_ has joined #openstack-ironic09:59
*** yuanying has joined #openstack-ironic10:04
jerryzhi, ironic, i have a server deployed by ironic and after a power outage, it can not boot up due to failure to call ironic deploy api, token file can not be downloaded from tftp thus no token in header10:06
jerryztftp told no such file10:06
Haomengjerryz: hi, can you check conductor log, and another one that make sure your tftp folder permision10:07
*** david-lyle has quit IRC10:08
*** yuanying has quit IRC10:09
Haomengjerryz: which driver you are using10:09
jerryzHaomeng: ssh pxe10:10
Haomengjerryz: ok, let me check code how to prepare token file10:11
openstackgerritRamakrishnan G proposed openstack/ironic-specs: New driver interface for RAID configuration  https://review.openstack.org/13589910:12
*** athomas has quit IRC10:14
jerryzHaomeng: i was using a vm as seed cloud where ironic was on. that seed was also shutdown due to power outage. FYI10:16
jerryzHaomeng: not much said in conductor log10:17
Haomengjerryz: how did you launch the ironic deploy, from nova? or via ironic command 'node-set-provision-state'10:18
jerryzHaomeng: through heat10:19
jerryzHaomeng: tripleo devtest script10:19
Haomengjerryz: cool, should be same with from nova10:19
Haomengjerryz: can you check your tftp dir10:19
Haomengjerryz: set via ironic.conf tftp_root10:20
Haomengjerryz: based on current code - https://github.com/openstack/ironic/blob/686f2ebaece43dc0d03e958693bb290a4c865491/ironic/drivers/modules/pxe.py#L25410:21
jerryzHaomeng: in ironic.conf it was not specified10:21
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Inband RAID configuration using agent ramdisk  https://review.openstack.org/14780310:21
Haomengjerryz: token is retrieved from context10:21
jerryzHaomeng: just image_path and instance_master_path10:21
jerryzHaomeng: under pxe section10:21
Haomengjerryz: if no specifiled tftp_root, default should be /tftpboot10:21
jerryzHaomeng: i am in10:22
Haomengjerryz: so check if there is token-* file in /tftpboot, if no, that mean your context does not contain auth_token10:23
Haomengjerryz: elseif the token-* file is existing as well, maybe it is the permision or network issue10:24
jerryzHaomeng: no such file10:24
Haomengjerryz: so maybe this is heat+nova+ironic integration issue, can you boot from nova to see if this can be  reproduced10:25
jerryzHaomeng: boot a new instance or reboot the vm through nova?10:26
Haomengjerryz: delete the existing instance and boot again by nova10:26
jerryzHaomeng: there were some configuration on the instance, which is an undercloud10:27
jerryzHaomeng: deleting it may be my last option10:27
Haomengjerryz: ok, did you raise this issue in tripleo irc already?10:28
jerryzHaomeng: not yet10:28
Haomengjerryz: based on the code, the root cause is there is no auth_token in your task.context.auth_token, that is populated by nova I think10:28
Haomengjerryz: https://github.com/openstack/ironic/blob/686f2ebaece43dc0d03e958693bb290a4c865491/ironic/drivers/modules/pxe.py#L25410:28
Haomengjerryz: so think this is context issue, maybe10:29
*** Nisha_away has joined #openstack-ironic10:29
*** Nisha has quit IRC10:29
jerryzHaomeng: thanks for the investigation.10:31
jerryzHaomeng: is there a way to trigger rewriting the token?10:33
Haomengjerryz: maybe have workaround with hands10:33
*** athomas has joined #openstack-ironic10:34
Haomengjerryz: run "ironic --debug node-list"10:35
Haomengjerryz: to get the token and create the token file into /tftp/token-<node_uuid> as the file name, good luck:)10:35
*** Nisha_away has quit IRC10:36
jerryzHaomeng: another question, i am looking to inject some user data to a instance deployed by ironic. is there a way?10:39
jerryzHaomeng: for instance, a chef-client configuration.10:40
Haomengjerryz: did you enable nova metadata and neutron metadata support to nat/route 169 subnet cloud-init request10:44
jerryzHaomeng: not through nova10:44
Haomengjerryz: sorry, no idea10:45
jerryzHaomeng: we want to use a standalone ironic without nova10:45
Haomengjerryz: that is ok, but depends on configdrive support I think, Lucas is working on - https://review.openstack.org/#/c/143510/10:46
jerryzHaomeng: but configdrive needed to be prepared anyway10:47
Haomengjerryz: if we run standalone ironic without nova/neutron/glance, some cloud/openstack feature can not be supported10:47
Haomengjerryz: yes10:47
Haomengjerryz: can you get the system serial control ownership10:48
*** Masahiro has quit IRC10:48
Haomengjerryz: I think this is a *hack* way to inject file into system from the system serial10:48
jerryzHaomeng: can ironic do that?10:49
Haomengjerryz: no:)10:50
Haomengjerryz: you can try with manually steps:)10:50
Haomengjerryz: good luck10:50
jerryzHaomeng: yeah, thanks for the pointer10:50
jerryzHaomeng: remind me of the days to burn a firmware through console10:50
Haomengjerryz: :)10:51
Haomengjerryz: yw10:51
jerryzHaomeng: i wrote the token10:51
jerryzHaomeng: but ironic api rejected it10:52
Haomengjerryz: maybe api need the same context/session token10:52
Haomengjerryz: do you make sure the tokenfile you create is correct?10:54
jerryzHaomeng: i am doubting it10:54
Haomengjerryz: :)10:54
Haomengjerryz: and make sure the token file owner is ironic:ironic10:55
jerryzHaomeng: {SHA1} got encoded into %7BSHA1%7D10:55
Haomengjerryz: ?10:55
jerryzHaomeng: yeah, i changed the ownership10:55
Haomengjerryz: let me give you a reference10:56
Haomengjerryz: this is my tokenfile created by ironc - http://paste.openstack.org/show/158237/10:57
Haomengjerryz: file size is 555210:57
Haomengjerryz: be away for a while, good luck10:59
jerryzHaomeng: see you11:03
jerryzHaomeng: the token i wrote was definitely messed up11:04
*** pelix1 has joined #openstack-ironic11:04
*** lucasagomes has joined #openstack-ironic11:12
*** david-lyle has joined #openstack-ironic11:17
*** andreykurilin_ has quit IRC11:18
*** alexpilotti has joined #openstack-ironic11:23
*** teju has quit IRC11:35
*** rameshg87 has quit IRC11:43
*** dtantsur|brb is now known as dtantsur11:48
*** Masahiro has joined #openstack-ironic11:48
*** Masahiro has quit IRC11:53
*** kbyrne has quit IRC12:06
*** kbyrne has joined #openstack-ironic12:08
*** pradipta is now known as pradipta_away12:14
*** vdrok_afk is now known as vdrok12:17
*** Haomeng has quit IRC12:22
*** Haomeng has joined #openstack-ironic12:24
dtantsurlucasagomes, hey, if you're around, could you have a look at tiny patch https://review.openstack.org/#/c/14749 ?12:25
lucasagomesdtantsur, sure12:25
dtantsurit's about that complex bug with power on/off. I believe it's actually a one-line fix12:25
lucasagomesI think that's the wrong patch12:27
lucasagomesChange default API port12:27
lucasagomesfor ceilometer12:27
* lucasagomes looks at the ironic queue12:27
lucasagomeshttps://review.openstack.org/#/c/147494/12:28
lucasagomesoh yeah, I tried to reproduce that bug, but could not :/12:29
lucasagomesdtantsur, did you test it with that change?12:29
openstackgerritMerged openstack/ironic: PXE: Add configdrive support  https://review.openstack.org/14351012:29
dtantsurlucasagomes, it's really hard to test, so I'm assuming by logs that it's what happened...12:30
lucasagomeswoah merged!12:30
dtantsur:)12:30
*** Haomeng has quit IRC12:31
lucasagomesdtantsur, I see... yeah the change lgtm. Dunno if we should test it more or just get it in and see if the problem doesn't appear again12:31
lucasagomesit's a rare thing to happen, it's quite hard to test12:31
dtantsuryep. on the other hand, I see it as a logical thing to do without any possible harm12:31
lucasagomesyeah12:32
lucasagomesand with more people using it it will be easier to see if the problem is gone or not12:32
lucasagomesif it's not we can reopen that bug12:32
dtantsurthat's what I thought as well. though I'm pretty sure that at least the problem I've seen will be fixed12:33
*** Haomeng has joined #openstack-ironic12:33
dtantsurI can't see how it could happen otherwise12:33
openstackgerritMerged stackforge/ironic-discoverd: General naming and strings clean up  https://review.openstack.org/14744212:34
lucasagomescool12:34
lucasagomes+a12:34
dtantsurthanks12:35
lucasagomesthank u for the fix :)12:41
*** afaranha_ has quit IRC12:49
*** Masahiro has joined #openstack-ironic12:57
*** smoriya has quit IRC13:02
*** Masahiro has quit IRC13:02
*** EmilienM|afk is now known as EmilienM13:05
openstackgerritMichael Kerrin proposed openstack/ironic: Fix ssh _get_power_status as it returned status for wrong node  https://review.openstack.org/14783213:06
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Introduce test-requirements and update to stable/juno  https://review.openstack.org/14783413:08
*** david-lyle has quit IRC13:11
openstackgerritFrederic Lepied proposed stackforge/ironic-discoverd: Add eDeploy plugin  https://review.openstack.org/14659913:14
openstackgerritMerged openstack/ironic: Ensure we don't have stale power state in database after power action  https://review.openstack.org/14749413:14
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Add eDeploy plugin  https://review.openstack.org/14659913:14
*** Marga_ has joined #openstack-ironic13:24
openstackgerritMerged stackforge/ironic-discoverd: Introduce test-requirements and update to stable/juno  https://review.openstack.org/14783413:26
*** dprince has joined #openstack-ironic13:27
*** jiangfei|2 has quit IRC13:38
*** jjohnson2 has joined #openstack-ironic13:38
*** jiangfei|2 has joined #openstack-ironic13:38
*** rakesh_hs has quit IRC13:38
*** naohirot has quit IRC13:40
*** gilliard is now known as gilllliard13:47
*** Nisha has joined #openstack-ironic13:48
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Small sdist-related cleanup  https://review.openstack.org/14785613:56
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Add eDeploy plugin  https://review.openstack.org/14659913:57
Nishadtantsur, hi13:58
dtantsurNisha, o/13:58
*** bauzas is now known as bauwser14:00
openstackgerritNisha Agarwal proposed openstack/ironic: Add states required for node-introspection  https://review.openstack.org/14785714:02
Nishadtantsur, ^^^^14:02
dtantsurwoohoo!14:03
* dtantsur is looking14:03
NishaNeeded your advice14:03
Nishadtantsur, i think with these addition in states we will need to push changes for nova ironic virt driver14:05
dtantsurNisha, I guess devananda or somebody already plan to work on introducing AVAILABLE and MANAGEABLE14:06
Nishadtantsur, but we require MANAGEABLE for inrospection14:07
Nishaand for MANAGEABLE i landed up in adding AVAILABLE14:07
dtantsurNisha, you can include it _temporary_ in your patch and wait for a proper patch to be rebased on14:07
*** athomas has quit IRC14:07
Nishaas the target state for MANAGEABLE is AVAILABLE14:08
NishaOk I will split states spec into two14:08
Nishais that ok14:08
dtantsurif s/spec/patch/ then should be ok. anyway you'd better talk to devananda or NobodyCam, they're following state machine work much closer than me14:09
Nishayes i meant that only14:10
NishaOk....I was thinking this will require the changes in Nova Virt driver also14:10
gilllliarddtantsur: Hi. Can I ask about the suggestion you (and others) made on ttps://review.openstack.org/#/c/142782/7 ?14:11
*** athomas has joined #openstack-ironic14:12
dtantsurNisha, definitely14:12
dtantsurgilllliard, sure, lemme see14:12
dtantsuraha I remember14:12
gilllliardBasically the suggestion is to clear instance_info['kernel'] and similar for ramdisk, here https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L644 right?14:12
dtantsurmaybe not here, lemme remember14:14
dtantsurgilllliard, ah right, judging by https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L234-L238 it might work14:14
gilllliardI think so.14:15
gilllliardSo, we make remove those properties, that causes the pxe driver to hit glance and we notice any changes there.14:16
gilllliardOr, if there's no changes, then we're still good.14:16
gilllliardBut. How to get hold of the instance_info in do_node_deploy?14:17
* gilllliard worries he's missing something obvious14:18
dtantsurthere's node object: https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L63014:19
gilllliardnode.instance_info then?14:20
NobodyCamGood morning Ironic, TGIF14:20
*** romcheg has left #openstack-ironic14:20
gilllliardMorning NobodyCam :)14:20
*** romcheg has joined #openstack-ironic14:20
NobodyCammorning gilllliard :)14:20
dtantsurNobodyCam, morning, very-very TGIF :)14:21
dtantsurgilllliard, should be14:21
gilllliarddtantsur: OK it is done. I'm not really sure what guarantees there are about instance_info's existence on a node.14:22
gilllliarddtantsur: I mean, if it's not a rebuild then we don't have an instance yet...14:23
gilllliardWell, anyway. New PS incoming.14:23
*** afaranha has joined #openstack-ironic14:26
openstackgerritMerged stackforge/ironic-discoverd: Small sdist-related cleanup  https://review.openstack.org/14785614:29
*** ryanpetrello has joined #openstack-ironic14:29
clif_hmorning ironic14:33
NobodyCammorning clif_h :)14:33
NobodyCammornign dtantsur :)14:33
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Do not timeout nodes which already finished introspection  https://review.openstack.org/14786614:35
openstackgerritMatthew Gilliard proposed openstack/ironic: Force glance recheck for kernel/ramdisk on rebuild  https://review.openstack.org/14278214:39
gilllliarddtantsur: like this ^^ ?14:39
dtantsuryeah. I wonder if we support setting image_source which does not have kernel/ramdisk set...14:40
dtantsurNobodyCam, ideas on ^^^?14:40
*** Haomeng has quit IRC14:45
*** Masahiro has joined #openstack-ironic14:46
*** Haomeng has joined #openstack-ironic14:46
lucasagomesmorning all :)14:49
*** Masahiro has quit IRC14:50
NobodyCammorning lucasagomes14:52
NobodyCamdtantsur: you ean like a whole dosk image?14:53
NobodyCams/ean/mean/14:53
*** mjturek has joined #openstack-ironic14:54
victor_lowtherMorning.14:54
NobodyCammorning victor_lowther14:55
*** coolsvap is now known as coolsvap|afk14:55
*** jiangfei|2 has quit IRC14:56
*** jiangfei|2 has joined #openstack-ironic14:56
vdrokhi everyone!14:57
vdrokgilllliard, instance_info will be there14:57
gilllliardHi vdrok!14:57
vdrokgilllliard, otherwise validation will fail14:58
vdrokgilllliard, also, you forgot about unittest :)14:58
gilllliardWell I wanted to check I was making the right change :) UT is on the way, too14:59
gilllliardAlso - do we need to backport this to stable/juno ?14:59
gilllliardThis and jroll's fix in image_cache ?14:59
dtantsurNobodyCam, I'm not sure what I mean :)15:01
dtantsurmorning vdrok, victor_lowther!15:01
vdrokmorning dtantsur15:01
NobodyCammorning vdrok15:02
NobodyCamdtantsur: lol... it is friday :)15:02
dtantsuryep :D15:02
vdrokdtantsur, agent image_source does not have those15:04
vdrokmorning NobodyCam15:04
*** datajerk has joined #openstack-ironic15:04
*** bnemec has quit IRC15:06
*** bnemec has joined #openstack-ironic15:09
*** yjiang5_ is now known as yginger515:14
*** jerryz has quit IRC15:15
NobodyCamlol.. this may be taking things a bit far: http://www.si.com/extra-mustard/2015/01/15/seattle-suburb-bans-cheese-seahawks-packers-game15:19
*** BadCub01_ has joined #openstack-ironic15:25
*** lucasagomes is now known as lucas-hungry15:25
*** bnemec is now known as beekneemech15:26
jrollmorning everybody15:27
jrolltgif and omgtwiafo15:27
jroll(omg, this week is almost finally over)15:27
jrollNobodyCam: lol, don't let JoshNang see that15:28
BadCub01_Morning all15:28
jrollI plugged it last night but the cleaning spec has 3x+2, 3x+1, who wants to land it? :) https://review.openstack.org/#/c/102685/15:28
dtantsurmorning, jroll, BadCub01_15:28
jrollheya BadCub01_ :)15:28
jrolland dtantsur :)15:28
NobodyCammornig jroll :)15:29
BadCub01_tgif!(H)15:29
*** jcoufal_ has joined #openstack-ironic15:30
trowndtantsur: I was thinking of working on https://bugs.launchpad.net/ironic-discoverd/+bug/1411194 were you wanting to import oslo.config for that, or use argparse and implement oslo.config later?15:30
NobodyCamjroll: will look it over in just a couple of minutes15:30
jroll\o/ thanks15:30
dtantsurtrown, for 1.0.0 it's just using argparse; oslo.config will come later15:31
dtantsurtrown, also we need not to break the current way (with issuing deprecation warning)15:32
trowndtantsur: cool deal. have you started on that, or should I go for it?15:32
dtantsurtrown, go for it, I'm busy with another stuff :) thanks15:32
trownnice15:32
*** jcoufal has quit IRC15:33
*** krtaylor has quit IRC15:33
*** jcoufal_ has quit IRC15:40
*** stelfer1 has joined #openstack-ironic15:45
*** jcoufal has joined #openstack-ironic15:45
*** stelfer has quit IRC15:47
NobodyCamjroll: I think I asked this already but forgot the answer. Will the current pxe deploy disk be modifyed to support ereasing as part of this spec... also are you planning on a follow up patch to the spec to fix the nits on it?15:48
jrollNobodyCam: yes and yes15:49
jrollI think it calls out the pxe ramdisk work15:49
NobodyCamit called out on the description section just not work items15:50
NobodyCam:-p I'll add a note on it for the follow up patch15:51
*** pcrews has joined #openstack-ironic15:52
jrollcool, thanks15:52
jrollJoshNang: you're gonna need a major follow up on this :P15:53
*** jiangfei|2 has quit IRC15:55
*** jiangfei|2 has joined #openstack-ironic15:56
*** anderbubble has joined #openstack-ironic15:57
*** Haomeng|2 has joined #openstack-ironic16:01
*** Haomeng has quit IRC16:02
*** xxj has quit IRC16:02
*** xxj has joined #openstack-ironic16:02
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Dump test CLI output as JSON, not as Python repr  https://review.openstack.org/14789416:04
dtantsurifarkas or trown: one of you up for quick review of 2 discoverd bugs found during testing? https://review.openstack.org/#/c/147866/ and ^^^16:05
trownlooking16:05
NobodyCamJoshNang: just checking you are planning on putting up the follow up to 102685 today?16:07
NobodyCamjroll: +2'd ready to +a see comment for why I didn't +a now16:14
NobodyCam:-p16:14
jrolloh lawd16:18
NobodyCam:)16:18
jrollok that's fine16:18
NobodyCamis JoshNang in today16:18
* jroll might do it if he gets a sec before josh shows up16:18
jrollshould be yeah16:18
jrollhe's one of those normal people16:18
jrollthat doesn't start at 716:18
NobodyCamjroll: if your ok with confirming the follow WILL make it up today I'll hit the button now16:19
jrollNobodyCam: meh, let's wait16:19
NobodyCamhehehe16:19
NobodyCam:-p16:19
NobodyCamwhat a slacker not starting at 6 in the morning16:20
NobodyCam:-p16:20
*** lucas-hungry is now known as lucasagomes16:21
jrollikr16:21
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Do not timeout nodes which already finished introspection  https://review.openstack.org/14786616:24
*** achanda has joined #openstack-ironic16:30
*** vdrok is now known as vdrok_afk16:32
*** krtaylor has joined #openstack-ironic16:33
*** Masahiro has joined #openstack-ironic16:35
openstackgerritClif Houck proposed openstack/ironic: Fix RPCService and Ironic Conductor so they shut down gracefully  https://review.openstack.org/14732516:36
*** ndipanov has quit IRC16:36
clif_h^ This should be a full fix for Ironic Conductor graceful shutdown16:38
clif_hprops to jroll for solving the final piece of the puzzle16:39
*** Masahiro has quit IRC16:39
openstackgerritMerged stackforge/ironic-discoverd: Do not timeout nodes which already finished introspection  https://review.openstack.org/14786616:40
openstackgerritMerged stackforge/ironic-discoverd: Dump test CLI output as JSON, not as Python repr  https://review.openstack.org/14789416:40
*** lucasagomes is now known as lucas-brb16:42
NobodyCamclif_h: including db clean up?16:42
clif_hIt appears workers will clean up their locks if given the chance: https://github.com/openstack/ironic/blob/master/ironic/conductor/task_manager.py#L23416:43
clif_hso if we wait on all workers to finish in del_host, then the workers should get their chance to do the right thing16:44
NobodyCamsweet16:44
jrollI believe it was letting everything that wasn't triggered through spawn_after clean up before16:46
NobodyCamSpamapS: https://review.openstack.org/147325 <- would be great if you could take a quick look at that one.16:46
jrollnow everything should get clean16:46
*** rwsu has joined #openstack-ironic16:47
JoshNangNobodyCam: good morning! :( about the no cheese in seattle and yup I can do a followup today16:48
jrollJoshNang: his +A is waiting on the follow up16:48
jroll:P16:49
JoshNangNobodyCam: to be clear, followup patch, not just responses right?16:49
NobodyCammorning JoshNang16:49
NobodyCamyep if we want to land as is.. then address the comments on the follow up16:49
NobodyCamso I can hit the button now.16:50
*** achanda has quit IRC16:50
NobodyCamI'll give you a couple of minutes to respond if you like16:50
JoshNangsure, i'll respond right now (gives me a clear direction for the followup)16:51
*** xxj has quit IRC16:51
*** Haomeng|2 has quit IRC16:51
*** jistr has quit IRC16:52
NobodyCamorderedA NUC last night16:53
*** agordeev has quit IRC16:54
*** romcheg has quit IRC16:54
*** romcheg has joined #openstack-ironic16:54
*** romcheg has quit IRC16:55
*** lazy_prince is now known as killer_prince16:55
JoshNangNobodyCam: added comments16:59
*** rwsu has quit IRC17:00
JoshNangI can work on the followup patch this morning. There's a bunch of protests between me and the office, so I've got some time before I leave.17:01
*** TheJulia_ is now known as TheJulia17:04
NobodyCamlol17:06
NobodyCamJoshNang: jroll: +a'd17:08
NobodyCamwe can clean up after our self now17:08
JoshNang\o/ thanks17:09
NobodyCambrb..17:09
*** Marga_ has quit IRC17:10
openstackgerritMerged openstack/ironic-specs: Implement Cleaning States  https://review.openstack.org/10268517:10
*** rwsu has joined #openstack-ironic17:19
*** Haomeng has joined #openstack-ironic17:23
*** Hefeweizen has quit IRC17:27
*** Hefeweizen has joined #openstack-ironic17:27
*** jrist has quit IRC17:30
*** lazy_prif has joined #openstack-ironic17:31
* NobodyCam is back17:31
*** killer_prince has quit IRC17:31
*** lazy_prif is now known as lazy_prince17:31
*** derekh has quit IRC17:31
*** lazy_prince has quit IRC17:36
openstackgerritMichael Kerrin proposed openstack/ironic: Fix ssh _get_power_status as it returned status for wrong node  https://review.openstack.org/14783217:36
*** spandhe has joined #openstack-ironic17:40
*** Marga_ has joined #openstack-ironic17:41
*** Marga_ has quit IRC17:41
*** Marga_ has joined #openstack-ironic17:42
*** jrist has joined #openstack-ironic17:42
openstackgerritJosh Gachnang proposed openstack/ironic-specs: Revising cleaning spec  https://review.openstack.org/14792517:43
SpamapSNobodyCam: taking a look now, thanks.17:48
*** achanda has joined #openstack-ironic17:50
*** lucas-brb has quit IRC17:50
*** rwsu has quit IRC17:51
NobodyCam:)17:52
openstackgerritMerged openstack/ironic-specs: Rename MANAGED to MANAGEABLE  https://review.openstack.org/14645217:53
*** killer_prince has joined #openstack-ironic17:55
*** killer_prince is now known as lazy_prince17:55
NobodyCamany cores have a quick minute for a 6 line patch... https://review.openstack.org/#/c/142782 (two +2's already)17:57
*** rwsu has joined #openstack-ironic17:58
*** achanda has quit IRC17:59
*** Marga_ has quit IRC18:00
*** Marga_ has joined #openstack-ironic18:01
dtantsurcalling it a day, have a nice weekend!18:01
*** dtantsur is now known as dtantsur|afk18:01
*** shakamunyi has joined #openstack-ironic18:02
*** achanda has joined #openstack-ironic18:02
*** Marga_ has quit IRC18:02
*** achanda has quit IRC18:03
*** Marga_ has joined #openstack-ironic18:03
NobodyCamhave a nice weekend dtantsur|afk18:11
NobodyCamgilllliard: are you still around18:15
SpamapShttp://logs.openstack.org/42/136642/10/gate/gate-tempest-dsvm-ironic-pxe_ssh/dc09621/18:16
SpamapSanybody know whats up?18:16
NobodyCamSpamapS: hummm Quota exceeded for resources:18:17
JayFthat looks unrelated to Ironic18:17
JayFquota exceeded when creating a subnet in tempest18:17
*** harlowja_away is now known as harlowja18:18
*** Masahiro has joined #openstack-ironic18:23
SpamapSThanks18:25
SpamapSwill see if there's something up in the gate18:25
adam_gSpamapS, JayF hmph that seems to be failing across the board18:26
*** Marga_ has quit IRC18:26
*** Marga_ has joined #openstack-ironic18:27
NobodyCam:(18:27
*** stelfer1 has quit IRC18:27
adam_ghttps://review.openstack.org/#/c/123046/ is the culprit, i believe18:27
*** Masahiro has quit IRC18:28
jrollNobodyCam: looking at https://review.openstack.org/#/c/14278218:32
NobodyCam:)18:33
jrollI know if you do something like node.instance_info['foo'] = bar; node.save() it doesn't actually update correctly for whatever reason18:33
jrollI wonder if this would do the same thing18:33
jrollthough conductor logs show lots of glance hits after rebuild is called, so I think it's working as expected18:33
*** Marga_ has quit IRC18:34
*** Marga_ has joined #openstack-ironic18:34
* jroll should test this locally18:34
*** pensu1 has quit IRC18:35
*** Marga__ has joined #openstack-ironic18:35
*** Marga__ has quit IRC18:35
NobodyCamjroll: some of our internal team were hitting a issue where the kernel was getting updaed in glance but issuing a rebuiild continued to use the previous kernel18:35
NobodyCamoh udated: http://www.customink.com/designs/ironicwork/beu0-0015-m60c/hotlink18:36
*** Marga_ has quit IRC18:36
jrollNobodyCam: right, I know what the bug is18:36
*** Marga_ has joined #openstack-ironic18:36
NobodyCam:)18:36
*** athomas has quit IRC18:36
*** pensu has joined #openstack-ironic18:38
*** jcoufal has quit IRC18:41
*** Marga_ has quit IRC18:42
NobodyCambrb wallies18:42
*** Marga_ has joined #openstack-ironic18:42
*** Marga_ has quit IRC18:43
*** Marga_ has joined #openstack-ironic18:43
*** Marga_ has quit IRC18:48
*** Marga_ has joined #openstack-ironic18:48
*** Marga_ has quit IRC18:50
*** Marga_ has joined #openstack-ironic18:50
*** alexpilotti has quit IRC18:52
* NobodyCam is back18:53
*** jcoufal has joined #openstack-ironic18:54
*** Marga_ has quit IRC18:55
*** jcoufal has quit IRC18:55
*** Marga_ has joined #openstack-ironic18:55
NobodyCamTY JoshNang :) +a'd19:00
JoshNangNobodyCam: np!19:00
jrollwoot19:01
JoshNangand thanks :)19:01
jrollnow go write some code ;)19:01
NobodyCamlol19:01
JoshNangheh yeah. i'm gonna start from scratch on the ironic code19:01
JoshNangipa should be semi-close already19:01
NobodyCamJoshNang: your going to rewrite ironic19:01
NobodyCamlol19:01
JoshNangNobodyCam: yup! in go!19:01
JoshNang:P19:01
NobodyCamlol19:02
*** Marga_ has quit IRC19:03
*** Marga_ has joined #openstack-ironic19:03
*** Marga_ has quit IRC19:04
*** Marga_ has joined #openstack-ironic19:04
openstackgerritMerged openstack/ironic-specs: Revising cleaning spec  https://review.openstack.org/14792519:04
*** Marga_ has quit IRC19:10
*** Marga_ has joined #openstack-ironic19:10
NobodyCamoh just checking anyone one have monday off?19:11
jrollnot that I know of?19:12
JayFNobodyCam: my wife, hahaha19:12
JayFNobodyCam: I think she says "long weekend" more than neccessary to rub it in19:12
*** Marga_ has quit IRC19:14
NobodyCamhahahahah19:14
NobodyCamdoes she work for a bank or school?19:14
*** Marga_ has joined #openstack-ironic19:14
JayFteacher :)19:15
*** rwsu has quit IRC19:15
NobodyCam:) nice19:15
*** spandhe has quit IRC19:22
*** xxj has joined #openstack-ironic19:22
JayFJoshNang: wanna land https://review.openstack.org/#/c/146924/ ?19:22
JoshNangJayF: landed19:28
jrollNobodyCam: on https://review.openstack.org/#/c/142782/ if instance_info gets the new kernel id after pointing the image at a new kernel, we're all good, right?19:29
*** Marga_ has quit IRC19:29
*** Marga_ has joined #openstack-ironic19:29
*** Marga_ has quit IRC19:30
jrollseems like it19:30
*** Marga_ has joined #openstack-ironic19:30
* jroll +A19:30
NobodyCamjroll: for rebuild, yes19:30
NobodyCamreboot, I think that could be a issue19:31
jrollgilllliard: +A'd that19:31
jrollNobodyCam: yeah, it comes back19:31
NobodyCam:)19:32
*** rwsu has joined #openstack-ironic19:32
*** spandhe has joined #openstack-ironic19:33
*** Marga_ has quit IRC19:40
*** Marga_ has joined #openstack-ironic19:40
*** PaulCzar has quit IRC19:41
*** PaulCzar has joined #openstack-ironic19:41
*** Marga_ has quit IRC19:49
*** Marga_ has joined #openstack-ironic19:49
*** Marga_ has quit IRC19:53
*** Marga_ has joined #openstack-ironic19:53
*** Marga_ has quit IRC19:57
*** Marga_ has joined #openstack-ironic19:57
*** achanda has joined #openstack-ironic20:00
*** Marga_ has quit IRC20:06
*** Marga_ has joined #openstack-ironic20:07
*** dprince has quit IRC20:10
*** achanda has quit IRC20:10
*** andreykurilin_ has joined #openstack-ironic20:12
*** achanda has joined #openstack-ironic20:12
*** Masahiro has joined #openstack-ironic20:12
*** Masahiro has quit IRC20:17
*** achanda has quit IRC20:17
*** andreykurilin_ has quit IRC20:22
*** Marga_ has quit IRC20:29
*** Marga_ has joined #openstack-ironic20:30
NobodyCamoh wow seems monday is a holiday for hp'ers20:30
*** Marga_ has quit IRC20:30
*** Marga_ has joined #openstack-ironic20:31
*** achanda has joined #openstack-ironic20:36
*** romcheg has joined #openstack-ironic20:38
*** wendar_ is now known as wendar20:38
*** achanda has quit IRC20:42
*** achanda has joined #openstack-ironic20:43
jrollluck you NobodyCam :P20:43
NobodyCamhehehehe20:45
*** Nisha has quit IRC20:48
*** andreykurilin_ has joined #openstack-ironic20:50
*** pensu has quit IRC20:50
*** chuckc__ has joined #openstack-ironic20:52
*** Marga_ has quit IRC20:52
openstackgerritJohn Trowbridge proposed stackforge/ironic-discoverd: Allows passing of config file via --config-file option.  https://review.openstack.org/14798720:52
*** anderbubble has quit IRC20:55
*** anderbubble has joined #openstack-ironic20:57
*** ChuckC_ has quit IRC20:58
*** ChuckC has joined #openstack-ironic21:02
*** chuckc__ has quit IRC21:04
*** anderbubble has joined #openstack-ironic21:06
*** achanda has quit IRC21:09
openstackgerritJohn Trowbridge proposed stackforge/ironic-discoverd: Allows passing of config file via --config-file option.  https://review.openstack.org/14798721:12
adam_ggate should be fixed by https://review.openstack.org/#/c/147966/21:12
*** afaranha has quit IRC21:15
*** achanda has joined #openstack-ironic21:19
*** achanda has quit IRC21:20
NobodyCamw00 hoo21:26
*** afaranha has joined #openstack-ironic21:29
adam_gshould be merging shortly. feel free to recheck against 1411794 when it does21:36
*** pelix1 has quit IRC21:37
* adam_g needs to run early today21:37
adam_ghave a good weekend21:38
openstackgerritMerged openstack/ironic: Force glance recheck for kernel/ramdisk on rebuild  https://review.openstack.org/14278221:40
*** Haomeng|2 has joined #openstack-ironic21:45
*** Haomeng has quit IRC21:46
*** afaranha has quit IRC21:49
NobodyCamif you havent see here are the election results: https://www.bigpulse.com/pollresults?code=4465BagptIqcEjm73VjmEpTk21:49
JayFQuorum \o/21:50
jrollwoot21:51
jrollNobodyCam: do you know how many directors were being elected?21:51
jrollI know there was at least 421:51
JayFI think it was 521:51
JayF*421:51
jrollok21:51
NobodyCam:)21:51
*** afaranha has joined #openstack-ironic22:00
*** Masahiro has joined #openstack-ironic22:01
*** xxj has quit IRC22:02
*** Haomeng|2 has quit IRC22:02
*** Masahiro has quit IRC22:06
*** Haomeng has joined #openstack-ironic22:06
*** jjohnson2 has quit IRC22:07
*** xxj has joined #openstack-ironic22:11
*** ifarkas has quit IRC22:12
*** Marga_ has joined #openstack-ironic22:13
*** achanda has joined #openstack-ironic22:14
*** Marga_ has quit IRC22:15
openstackgerritMichael Turek proposed openstack/ironic-python-agent: Make all IPA error classes inherit from RESTError  https://review.openstack.org/14801422:15
*** Marga_ has joined #openstack-ironic22:16
JayFmjturek: ^ thanks for the IPA patch; I'll take a look now22:16
mjturekJayF thanks22:16
JayFAre you using IPA?22:17
* jroll wishes ironicclient had pluggable auth like novaclient22:17
jrolljust sayin'22:17
mjturekJayF not yet actually22:17
JayFmjturek: lgtm once jenkins passes22:17
mjturekJayF cool, thanks!22:17
mjturekyeah very interested in IPA, just haven't gotten around to playing with it22:17
JayFWell if you have any questions once you get there; feel free to ask.22:18
*** Haomeng has quit IRC22:21
NobodyCambrb22:22
mjturekJayF awesome thanks!!22:22
*** Haomeng has joined #openstack-ironic22:23
*** EmilienM is now known as EmilienM|afk22:26
*** mjturek has quit IRC22:28
*** andreykurilin_ has quit IRC22:30
jrollI made a thing: https://github.com/jimrollenhagen/superironic22:32
jrollbasically makes "superironic $env command" equivalent to "ironic command" ran with the env defined in ~/.superironic (can be multiple defined there22:33
jroll)22:33
jrollshamelessly copied a lot of it from https://github.com/major/supernova22:33
*** achanda has quit IRC22:33
jrollexample config: https://github.com/major/supernova/blob/master/example_configs/supernova_rackspace22:34
*** achanda has joined #openstack-ironic22:40
*** shakamunyi has quit IRC22:40
*** achanda has quit IRC22:43
*** Marga_ has quit IRC22:45
*** Marga_ has joined #openstack-ironic22:45
*** achanda has joined #openstack-ironic22:49
NobodyCamneat22:52
*** andreykurilin_ has joined #openstack-ironic22:53
*** Marga_ has quit IRC22:54
*** Marga_ has joined #openstack-ironic22:54
*** ryanpetrello has quit IRC22:59
*** ryanpetrello has joined #openstack-ironic23:03
*** MattMan1 has quit IRC23:03
*** achanda has quit IRC23:09
*** alexpilotti has joined #openstack-ironic23:17
*** igordcard has joined #openstack-ironic23:19
*** openstackgerrit has quit IRC23:19
*** openstackgerrit has joined #openstack-ironic23:19
*** 17SABIH3M is now known as annegentle23:26
*** anderbubble has quit IRC23:30
*** Marga_ has quit IRC23:39
*** Marga_ has joined #openstack-ironic23:40
*** romcheg has quit IRC23:45
*** ChuckC has quit IRC23:50
*** Masahiro has joined #openstack-ironic23:50
*** Masahiro has quit IRC23:54
*** lazy_prince has quit IRC23:55
*** ryanpetrello has quit IRC23:55
*** andreykurilin_ has quit IRC23:55
* NobodyCam steps akf for a couple of min23:58
*** ryanpetrello has joined #openstack-ironic23:59
*** killer_prince has joined #openstack-ironic23:59
*** killer_prince is now known as lazy_prince23:59

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