Monday, 2015-02-02

*** pcrews has joined #openstack-ironic00:03
*** igordcard has quit IRC00:10
*** Marga_ has joined #openstack-ironic00:11
*** Marga_ has quit IRC00:14
*** Marga_ has joined #openstack-ironic00:15
*** Marga_ has quit IRC00:19
*** Marga_ has joined #openstack-ironic00:21
*** Marga_ has quit IRC00:22
*** Marga_ has joined #openstack-ironic00:22
*** eghobo has joined #openstack-ironic00:24
*** Marga_ has quit IRC00:27
*** Haomeng|2 has joined #openstack-ironic00:28
*** Haomeng has quit IRC00:29
*** Marga_ has joined #openstack-ironic00:32
*** eghobo has quit IRC00:45
mrdahi naohirot00:50
naohirotmrda: hi good morning :)00:50
jroll\o mrda and naohirot :)00:53
naohirotjroll: good evening00:53
*** Marga_ has quit IRC00:55
*** Marga_ has joined #openstack-ironic00:56
*** chenleji has quit IRC00:56
*** chenleji has joined #openstack-ironic00:56
*** eghobo has joined #openstack-ironic00:56
*** eghobo has quit IRC00:58
mrdahey jroll01:06
jrollhiya mrda, how goes?01:06
mrdagood - sorry about the slow progress on logical names.  Was on leave last week (before the kids went back to school)01:06
mrdaAnd I had the LCA conference before that.  But will get the ironic portion finished off today and a new patch to review.01:07
mrdaThen the python-ironicclient01:07
jrollit's all good, I just hadn't heard about it for a while, I asked before I actually looked at the last update :P01:07
mrdanp, it's in good shape, just a few niggles01:07
jrollsweet01:08
* jroll will review when it's his monday01:08
mrda:)01:08
*** zhenzanz has joined #openstack-ironic01:38
*** Marga_ has quit IRC01:41
*** jerryz has joined #openstack-ironic01:53
*** chenglch has joined #openstack-ironic02:09
*** jiangfei has joined #openstack-ironic02:17
openstackgerritMerged openstack/ironic: Drop deprecated namespace for oslo.rootwrap  https://review.openstack.org/15185102:17
lintan__Hi jiangfei02:19
jiangfeilintan_: hi,  we add --instance option on node-show, why not no node-list, it has some reason?02:20
lintan__jiangfei: Hmm, I am not familiar with this :(  You can ask others02:23
jiangfeilintan__: I saw that you submit commit :).02:24
jiangfeilintan_:sorry, it's linggao.02:26
lintan__jiangfei, :)02:26
jiangfeilintan_:  :)02:27
*** ramineni has joined #openstack-ironic02:43
*** achanda has joined #openstack-ironic02:48
*** achanda has quit IRC02:50
*** achanda has joined #openstack-ironic02:52
*** achanda has quit IRC02:54
openstackgerritTan Lin proposed openstack/ironic: Add Node.driver_internal_info  https://review.openstack.org/15078403:02
*** achanda has joined #openstack-ironic03:05
*** achanda has quit IRC03:09
*** achanda has joined #openstack-ironic03:09
*** achanda has quit IRC03:28
*** achanda has joined #openstack-ironic03:35
*** achanda has quit IRC03:38
*** pcrews has quit IRC03:40
*** achanda has joined #openstack-ironic03:43
*** eghobo has joined #openstack-ironic03:45
*** eghobo has quit IRC03:58
*** achanda has quit IRC04:12
*** rameshg87 has joined #openstack-ironic04:20
*** eghobo has joined #openstack-ironic04:23
*** achanda has joined #openstack-ironic04:25
*** achanda has quit IRC04:27
*** eghobo has quit IRC04:32
*** spandhe has joined #openstack-ironic04:33
*** eghobo has joined #openstack-ironic04:34
*** anderbubble has joined #openstack-ironic04:46
*** ramineni1 has joined #openstack-ironic05:00
*** ramineni has quit IRC05:02
*** SpamapS_ is now known as SpamapS05:09
*** SpamapS has quit IRC05:09
*** SpamapS has joined #openstack-ironic05:09
*** pensu has joined #openstack-ironic05:14
*** Marga_ has joined #openstack-ironic05:26
*** Marga_ has quit IRC05:37
*** Marga__ has joined #openstack-ironic05:37
*** lazy_prince has quit IRC05:46
openstackgerritTan Lin proposed openstack/ironic: Add Node.driver_internal_info  https://review.openstack.org/15078405:46
openstackgerritTan Lin proposed openstack/ironic: Clean driver_internal_info when changes nodes' driver.  https://review.openstack.org/15199005:46
*** rakesh_hs has joined #openstack-ironic05:51
*** killer_prince has joined #openstack-ironic05:51
*** killer_prince is now known as lazy_prince05:51
*** anderbubble has quit IRC06:00
openstackgerritjiangfei proposed openstack/python-ironicclient: node-list support get-node-by-instance_id  https://review.openstack.org/15186406:00
openstackgerritTan Lin proposed openstack/ironic: Clean driver_internal_info when changes nodes' driver  https://review.openstack.org/15199006:01
*** Marga__ has quit IRC06:22
*** Marga_ has joined #openstack-ironic06:22
*** lazy_prince has quit IRC06:26
*** Marga_ has quit IRC06:26
*** ramineni1 has quit IRC06:44
*** killer_prince has joined #openstack-ironic06:50
*** killer_prince is now known as lazy_prince06:50
*** achanda has joined #openstack-ironic06:52
*** Marga_ has joined #openstack-ironic06:53
*** ramineni has joined #openstack-ironic06:56
*** achanda has quit IRC06:56
*** yog_ has joined #openstack-ironic06:56
*** Marga_ has quit IRC06:57
*** jcoufal has joined #openstack-ironic07:00
*** jiangfei has quit IRC07:02
*** jiangfei has joined #openstack-ironic07:02
*** lazy_prince has quit IRC07:04
openstackgerritMichael Davies proposed openstack/ironic: Add logical name support to Ironic  https://review.openstack.org/14173707:07
*** achanda has joined #openstack-ironic07:07
*** chuckC__ has joined #openstack-ironic07:11
*** jmanko has joined #openstack-ironic07:11
*** chuckC_ has quit IRC07:14
*** chuckC__ is now known as chuckC_07:14
*** jmank has quit IRC07:14
*** mrda is now known as mrda-away07:14
*** Marga_ has joined #openstack-ironic07:23
*** Marga_ has quit IRC07:27
*** eghobo has quit IRC07:31
*** killer_prince has joined #openstack-ironic07:32
*** killer_prince has quit IRC07:32
*** lazy_prince has joined #openstack-ironic07:33
*** ijw has joined #openstack-ironic07:43
*** lazy_prince has quit IRC07:45
*** jcoufal_ has joined #openstack-ironic07:49
*** jcoufal has quit IRC07:52
*** lazy_prince has joined #openstack-ironic07:58
*** lazy_prince has quit IRC07:58
*** lazy_prince has joined #openstack-ironic07:59
*** chlong has quit IRC08:00
*** mkerrin has quit IRC08:00
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Virtual Media Deploy module for iRMC Driver  https://review.openstack.org/15195808:01
*** dtantsur|afk is now known as dtantsur08:05
*** spandhe has quit IRC08:06
dtantsurMorning Ironic08:06
naohirotdtantsur: good morning :)08:07
dtantsurnaohirot, g'afternoon (right?)08:07
naohirotdtantsur: Yeah, getting dark, 5PM08:08
*** stendulker has joined #openstack-ironic08:14
*** achanda has quit IRC08:16
*** ndipanov has joined #openstack-ironic08:17
lintan__hi dtantsur08:17
dtantsuro/08:18
openstackgerritjiangfei proposed openstack/python-ironicclient: node-list support get nodes from chassis_uuid  https://review.openstack.org/15185908:18
lintan__I try updates['driver_internal_info'] = {} at first, but node.driver_internal_info doesn't change immediately08:19
lintan__dtantsur, so I change node.driver_internal_info and get updates again08:21
dtantsurlintan__, ok, looks valid08:22
* dtantsur changes vote08:22
*** Marga_ has joined #openstack-ironic08:24
*** Marga_ has quit IRC08:26
*** Marga_ has joined #openstack-ironic08:26
*** achanda has joined #openstack-ironic08:30
*** Marga_ has quit IRC08:31
*** bauwser is now known as bauzas08:32
openstackgerritjiangfei proposed openstack/python-ironicclient: port-list support get-port-from-mac  https://review.openstack.org/15186808:42
*** achanda has quit IRC08:44
*** rwsu has joined #openstack-ironic08:49
*** rwsu is now known as rwsu-afk08:50
*** jistr has joined #openstack-ironic08:58
*** erwan_taf has joined #openstack-ironic08:59
openstackgerritDmitry Tantsur proposed openstack/ironic: Add support for driver-specific periodic tasks  https://review.openstack.org/14752309:06
openstackgerritjiangfei proposed openstack/python-ironicclient: node-list support get nodes from chassis_uuid  https://review.openstack.org/15185909:12
*** derekh has joined #openstack-ironic09:12
*** zhenzanz has quit IRC09:13
*** pensu has quit IRC09:13
openstackgerritTan Lin proposed openstack/ironic: Update agent driver with new field driver_internal_info  https://review.openstack.org/15205509:17
openstackgerritDevananda van der Veen proposed openstack/ironic: Create a new python entrypoint for testing drivers  https://review.openstack.org/15205609:18
*** MattMan has joined #openstack-ironic09:19
*** andreykurilin_ has joined #openstack-ironic09:21
*** GheRiver1 is now known as GheRivero09:22
*** athomas has joined #openstack-ironic09:25
*** Marga_ has joined #openstack-ironic09:27
*** Marga__ has joined #openstack-ironic09:29
openstackgerritsandhya proposed openstack/ironic-specs: Chassis Level Discovery Tool  https://review.openstack.org/13486609:29
*** subscope has joined #openstack-ironic09:29
openstackgerritsandhya proposed openstack/ironic-specs: Chassis Level Discovery Tool  https://review.openstack.org/13486609:31
*** Marga_ has quit IRC09:32
dtantsurjroll, please have a look at https://review.openstack.org/#/c/152055/ (agent driver change) once you're here09:32
*** Marga__ has quit IRC09:33
*** pensu has joined #openstack-ironic09:33
*** ijw has quit IRC09:37
*** andreykurilin_ has quit IRC09:39
*** andreykurilin_ has joined #openstack-ironic09:39
*** yog_ has quit IRC09:40
*** pensu has quit IRC09:49
*** mkerrin has joined #openstack-ironic09:54
*** yog_ has joined #openstack-ironic09:56
*** pensu has joined #openstack-ironic10:00
*** pensu has quit IRC10:04
rameshg87dtantsur, hi10:14
*** yog_ has quit IRC10:15
rameshg87dtantsur, regarding https://review.openstack.org/#/c/151851/110:15
rameshg87dtantsur, shouldn't we actually change requirements.txt to oslo.rootwrap>=1.5.0 ?10:15
rameshg87dtantsur, oslo_rootwrap seem to be available only from oslo.rootwrap==1.5.010:16
*** pelix has joined #openstack-ironic10:16
dtantsurrameshg87, we probably should, but we can't change this file directly10:17
dtantsurrameshg87, it should go to global-requirements10:17
dtantsurhttps://github.com/openstack/requirements/blob/master/global-requirements.txt10:18
*** yog_ has joined #openstack-ironic10:18
dtantsurbut yeah, maybe we landed that patch a bit too early...10:18
openstackgerritsandhya proposed openstack/ironic-specs: Chassis Level Discovery Tool  https://review.openstack.org/13486610:18
rameshg87dtantsur, yeah10:19
rameshg87dtantsur, if someone does devstack now, i guess it won't pick up the latest oslo.rootwrap if they had oslo.rootwrap=1.3.0 installed10:20
rameshg87dtantsur, and starting conductor might fail10:20
dtantsurwe might want to check with infra about bumping global-requirements... and I don't if we should revert this one10:21
dtantsursorry, brb10:21
rameshg87dtantsur, have added a comment in that review10:22
rameshg87dtantsur, infra is still at 1.3.0: https://github.com/openstack/requirements/blob/master/global-requirements.txt#L6610:23
rameshg87dtantsur, might need to change that first10:23
*** yog_ has quit IRC10:23
*** andreykurilin_ has quit IRC10:24
openstackgerritsandhya proposed openstack/ironic-specs: Chassis Level Discovery Tool  https://review.openstack.org/13486610:26
*** Marga_ has joined #openstack-ironic10:29
*** Marga_ has quit IRC10:35
*** yog_ has joined #openstack-ironic10:36
*** dtantsur is now known as dtantsur|brb10:48
*** dlpartain has joined #openstack-ironic11:01
*** dlpartain has left #openstack-ironic11:01
*** ramineni has quit IRC11:05
openstackgerritMichael Davies proposed openstack/ironic: Add logical name support to Ironic  https://review.openstack.org/14173711:10
* mrda-away says to himself, "This time for sure!"11:12
*** kbyrne has joined #openstack-ironic11:14
*** rameshg87 has quit IRC11:22
*** stendulker has quit IRC11:30
*** Marga_ has joined #openstack-ironic11:30
*** Marga_ has quit IRC11:35
*** dtantsur|brb is now known as dtantsur12:00
*** Haomeng has joined #openstack-ironic12:01
*** Haomeng|2 has quit IRC12:03
*** mrmoje has joined #openstack-ironic12:11
mrmojejoin #openstack-oslo12:12
*** pensu has joined #openstack-ironic12:17
*** subscope has quit IRC12:21
*** Marga_ has joined #openstack-ironic12:23
*** Marga_ has quit IRC12:24
*** Marga_ has joined #openstack-ironic12:24
*** subscope has joined #openstack-ironic12:31
*** mlsoule has joined #openstack-ironic12:46
*** dprince has joined #openstack-ironic13:01
*** Marga_ has quit IRC13:04
*** Marga_ has joined #openstack-ironic13:05
*** rakesh_hs has quit IRC13:22
*** mjturek has joined #openstack-ironic13:32
*** rloo has joined #openstack-ironic13:43
*** stendulker has joined #openstack-ironic13:51
*** jjohnson2 has joined #openstack-ironic14:00
jrollmorning all :)14:02
Shrewsjroll: morning14:02
jrollnice catch by ramesh... dtantsur do we want to revert that? I see global reqs got bumped14:03
jrollhiya Shrews :)14:03
dtantsurShrews, jroll, o/14:04
dtantsurif global-reqs are ok, we should just wait for the bot to come IMO14:04
Shrewshey dtantsur14:05
jrollyeah, agree14:05
Shrewsjroll: isn't it, like, 6am for you???14:05
jroll... yes14:05
* jroll is a morning person :P14:05
Shrewsjroll: morning people are weird14:05
jrolldtantsur: that agent patch has $heartbeat_period seconds of backwards incompatibility :(14:06
jrollShrews: indeed14:06
derekhHi all, tripleo overcloud on F20 is broken , I think its an ironic regression https://bugs.launchpad.net/tripleo/+bug/141702614:06
jrollderekh: :/14:07
jrollderekh: you have more info?14:07
jrollone thing we realized, is we landed the change to move oslo.rootwrap to oslo_rootwrap before we picked up the new version14:07
jrollit's in global reqs at this point but I could propose a bump before the bot does to see what happens14:08
derekhjroll: screen show of what I'm  seeing when trying to boot a f20 image https://launchpadlibrarian.net/196381334/Screenshot%20from%202015-02-02%2011%3A31%3A25.png14:08
derekhjroll:  that made me try a test revert of https://review.openstack.org/#/c/141148/614:09
*** chenglch has quit IRC14:09
stendulkerHi jroll14:09
jrollmorning stendulker14:09
derekhjrist: which passed, the first F20 overcloud pass we have had since friday14:09
stendulkermorning jroll !14:09
stendulkerjroll: Its evening for me.14:10
jrollderekh: strange to me that this commit only broke f20 :/14:10
jrollstendulker: oops, good evening :)14:10
stendulkerjroll: For ILOAgent driver, for secure boot, I need to enable  secure boot once deploy is complete.14:10
derekhjroll: yup, I thought so to but that seems to be whats happening, see http://goodsquishy.com/downloads/tripleo-jobs.html14:11
jrollderekh: does the f20 environment have dhcp?14:11
stendulkerjroll: For this one option is to override heartbeat method and check for deploy_is_done() and perform the task14:11
openstackgerritJim Rollenhagen proposed openstack/ironic: Bump oslo.rootwrap to 1.5.0  https://review.openstack.org/15212714:12
jrolldtantsur: ^ in case we want to be proactive14:13
stendulkerjroll: Would it be better to have a interface in AgentVendorInterface which could be redefined by drivers to perform things upon deploy complete or in vendor passthrough method but not to get called regularly like heartbeat()14:13
jrollderekh: hrm, it must have dhcp because pxe14:13
derekhjroll: I thought so but maybe not14:13
jrollstendulker: you could override _reboot_to_instance() no?14:14
stendulkerjroll: Any suggestion on this?14:15
jrollstendulker: _reboot_to_instance is called when the deploy is done, just override that method instead https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L406-42514:16
stendulkerjroll: this sounds good14:17
openstackgerritJim Rollenhagen proposed openstack/ironic: Revert "Do not pass PXE net config from bootloader to ramdisk"  https://review.openstack.org/15212914:17
jrollderekh: I'm not sure if this is the fix we should land, but ^14:18
jrollderekh: I'd rather fix that environment, might be ok with a revert in the meantime though14:18
rlooHappy Monday ironickers14:18
jrollderekh: what's different about that env? the undercloud runs on fedora 20?14:18
jrollmorning rloo, "happy" is a fun word :P14:18
rlooevening stendulker, I was looking at your comments for https://review.openstack.org/#/c/135845/14:19
stendulkerthanks jroll14:19
derekhjroll: the difference between that env for the F20 ci job when compared to the ubuntu one ?14:19
rloojroll: heh, am still waking up. hoping it'll be a happy week ;)14:19
jrollderekh: correct14:19
jrollrloo: indeed :)14:19
derekhjroll: its a F20 undercloud booting a F20 image  , the passing job is a ununtu undercloud booting ubunutu image14:20
rloostendulker: where in https://review.openstack.org/#/c/135228/3/specs/kilo/uefi-secure-boot.rst is there mention of set_secure_boot_state? (oh, maybe I should do a grep, i was just skimming)14:20
jrollderekh: hrm, everything else is the same? ramdisks are the same?14:20
stendulkerrloo: Its in comments14:21
rloostendulker: oh, let me see14:21
derekhjroll: ramdisk would be built with F20 aswell (as opposed to Ubuntu)14:21
jrollderekh: aha14:22
rloostendulker: ok, I see the comments now. They were just comments; ie, not proposed. I don't think that those settings are 'states'.14:22
rloostendulker: they don't reflect the current state of secure_boot, do they?14:22
rloostendulker: i'll comment in the spec14:23
stendulkerrloo: secure boot can have 2 states. viz enabled or disabled14:23
rloostendulker: i'm objecting to the word 'state'. Can't we use something like 'mode'?14:24
rloostendulker: just cuz we already use state in provision_state and power_state.14:25
jrollderekh: I'm thinking the f20 ramdisk needs to be fixed somehow, maybe there's a different code path compared to https://review.openstack.org/#/c/140870/14:28
jrollderekh: I'm fine with reverting that for now, might like to hear from others, adam_g etc14:29
*** Haomeng has quit IRC14:29
*** stendulker_ has joined #openstack-ironic14:30
derekhjroll: ok, makes sense, in the meantime I'll revert it in tripleo-ci14:30
derekhjroll: thanks14:30
stendulker_rloo: If naming could ause confusion, we can change the names of APIs14:30
*** BadCub_ has joined #openstack-ironic14:31
NobodyCamgood morning Ironic :)14:31
jrollderekh: hm, wondering if we should revert in ironic proper, there could be other people using the f20 ramdisk14:31
jrollhiya NobodyCam :)14:31
BadCub_Morning Iroinic14:31
BadCub_Ironic even14:31
*** zigo has quit IRC14:31
jrollmornin BadCub_ :)14:31
NobodyCammorning jroll and BadCub_14:31
NobodyCam:)14:31
naohirotNobodyCam: jroll: BadCub_: good morning14:32
GheRiveromorning all14:32
naohirotNobodyCam: I'd like to know one thing before going to bed :)14:32
derekhjroll: yup, my suggestion of reverting in tripleo-ci is only a temp measure to allow our ci to keep moving, while a permanent solution is settled on14:32
rloostendulker_: that's my opinion. What do you/others think? I've commented in the spec. thx.14:33
stendulker_rloo: By having get_secure_boot_mode(), one should not assume its yet  boot mode in addition to  (bios, uefi)14:33
*** stendulker has quit IRC14:33
NobodyCammorning GheRivero naohirot rloo and derekh14:33
jrollderekh: right, ok, thanks14:33
jroll\o GheRivero14:33
naohirotNobodyCam: do we have IRC meeting tomorrow?14:33
rloomorning NobodyCam14:33
NobodyCamI think we do14:33
rlooGheRivero: hi! Have you been away?14:33
naohirotNobodyCam: or there is no IRC meeting due to mid cycle?14:33
*** xxj has quit IRC14:34
naohirotNobodyCam: Okay14:34
jrollI think we should still have a meeting but idk what the actual plan is14:34
NobodyCamnaohirot: :)14:34
jrollthis week is the tuesday time right?14:34
jroll(tuesday utc)14:34
naohirotjroll: yes, i think So14:34
*** zigo has joined #openstack-ironic14:35
GheRiverorloo: kind of. Focusing on other things14:35
rlooGheRivero: welcome back ;)14:35
naohirotNobodyCam: jroll: all: see you good night!14:35
*** naohirot has quit IRC14:35
dtantsurNobodyCam, rloo, morning14:35
NobodyCammorning dtantsur14:36
rloostendulker_: maybe some other word than 'mode'? I didn't think much beyond not liking 'state'.14:36
stendulker_rloo: How about naming them as get_uefi_secure_boot_mode() and set_uefi_Secure_boot_mode()? This would avoid confusion and also associate it with the uefi boot mode14:36
*** vipul has quit IRC14:37
rloostendulker_: this is in the management interface right? I don't think we want 'uefi' there if setting/getting this secure boot mode could be used non-uefi.14:38
stendulker_rloo: secure boot is applicable only to UEFI boot mode14:38
openstackgerritJim Rollenhagen proposed openstack/ironic: Move oslo.config references to oslo_config  https://review.openstack.org/15195714:39
*** jjohnson2 has quit IRC14:40
*** Haomeng has joined #openstack-ironic14:40
openstackgerritJim Rollenhagen proposed openstack/ironic: Move oslo.db references to oslo_db  https://review.openstack.org/15195514:40
*** vipul has joined #openstack-ironic14:41
*** Marga_ has quit IRC14:42
rloostendulker_: how about instead of true/false, the value is none or 'uefi'?14:42
stendulker_rloo: UEFI can still work without secure boot14:43
rloostendulker_: right, so in that case, secure_boot_mode would be none14:44
stendulker_rloo: In UEFI boot mode, secure boot can be in enabled or disabled state.14:44
rloostendulker_: do we have any other properties that are only related to uefi or is this the first one? (other than boot mode)14:45
dtantsurso, today http://ironic-bugs.divius.net/ is broken due to unicode problems... one more reason to have Py3k ASAP >_<14:45
stendulker_rloo: do you mean true=uefi and false=none for secure boot state?14:45
*** xxj has joined #openstack-ironic14:45
stendulker_rloo: this is the first one that differs from bios14:46
rloostendulker_: yes. I mean, set_secure_boot_mode() (or whatever we call it), can have values None or 'uefi'.14:46
rloostendulker_: if that makes sense. I don't have a good idea right now, of how we want to do the api so it is generalized cuz I'm not sure what the future holds. I don't want us to end up with eg get_uefi_secure_boot_mode, get_uefi_propertyZ_mode, get_uefi_properyY_mode...14:47
stendulker_rloo: I feel, such values (none, uefi) do not give correct interpretation of secure boot mode.14:47
rloostendulker_: and I don't know if we'll ever have secure_boot in the context of something outside uefi. if we do, I don't know that I want 'uefi' in the name of the method.14:48
stendulker_rloo: We can drop uefi from the name of the API and it cane be get/set_secure_boot_mode()14:50
rloostendulker_: that's fine with me but I thought you were concerned about it.14:51
openstackgerritJim Rollenhagen proposed openstack/ironic: Move oslo.config references to oslo_config  https://review.openstack.org/15195714:51
jrollholy cow, that's a lot of oslo.config imports14:51
stendulker_rloo: i was trying to be more explicit, but its ok with me not to have uefi14:52
*** openstackgerrit has quit IRC14:52
stendulker_rloo: Will change the API names and upload the new patch for same.14:52
rloostendulker_: I'd rather not have uefi in the name. You're going to hate this, but I'm wondering why we even want a method in the management interface for this.14:52
jrollstendulker_: rloo I like the (none, uefi) thing, that would give us explicit-ness14:52
*** openstackgerrit has joined #openstack-ironic14:52
jrollrloo: where would you put it? boot interface?14:53
rloojroll, NobodyCam, anyone. How are we matching/mapping information in 'capabilities' (that eg nova driver sets) to our nodes?14:53
jrollrloo: it's in nova, let me find it14:54
rloojroll: looking at the uefi secure boot spec, it seems to indicate that the flavor will have capabilities:secure_boot:True14:54
jrollrloo: I think it's this https://github.com/openstack/nova/blob/master/nova/scheduler/filters/compute_capabilities_filter.py14:55
jrolloh, flavor extra specs, mmm14:55
jrollyeah, that's the flavor filter14:56
rloojroll: so for each capability that can be set, are we going to have code that sez: if node.capability['secure_boot'] and driver.management.get_secure_boot()...14:56
*** jerryz has quit IRC14:56
jrollrloo: and the mapping here https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L223-23614:57
jrollmmm, I see14:57
jrollI think it depends on the capability14:58
stendulker_jroll: There is a spec that will pass the capabilities as part of instance_info14:58
jrollfor example we might have capabilities['awesome_video_card']14:58
stendulker_jroll: https://review.openstack.org/#/c/136104/14:58
rloowhat do you do when it is -10 and storming outside and your cat comes in with a (dead I think) bird... go back to work...14:58
jrollwhich is just a filtering thing14:58
jrollrloo: lollllll14:58
jrollhere's the actual patch passing capabilities along https://review.openstack.org/#/c/141012/14:59
rloojroll: right, so it will be in the node's instance info. then what?15:00
jrollidk, just narrating :P15:01
jrollI mean, we'll need code to do things with it15:01
rloojroll: we treat them all individually/specially? I think now, that I didn't agree with a spec that wanted to add some method to mgt interface.15:01
jrollbut it's going to depend what the capability is15:01
NobodyCamnice HipChat reset all there passwords :(15:01
rloojroll: ah yes, get/set boot mode to mgt interface (https://review.openstack.org/#/c/129529/)15:01
jrollNobodyCam: /me wonders what other data was compromised15:02
rloojroll: yeah, at some point it is going to depend on the capability.15:02
NobodyCamjroll: yep!15:02
rloojroll: Are we OK with adding a new set of methods get/set_capabilityX() to mgt interface for each capabililty (worst case I guess)15:02
jrollrloo: I'm ok with that for now. if it gets to be too many we can pretty easily make that generic15:03
jrollor make it generic now, idk15:03
jrollset_capability(capability)15:03
rloojroll: or is it more extensible to make it get/set_capability(capability name, value)15:03
NobodyCamrloo: ++15:03
rlooI think we should make it generic now.15:03
jrollare we confident there will be many of them?15:04
jrollit's easy to make that generic, set_capability_uefi = functools.partial(set_capability, 'uefi')15:05
rloojroll: didn't people mention at the summit sessions, that they wanted to pass down capabilities from nova/flavour?15:05
jroll(for backwards compat)15:05
jrollrloo: yeah, this is exactly that15:05
rloobut why bother doing the backwards compatibility if we can do the generic one now.15:05
rloois there any reason not to have a generic now, and to add set_boot_mode(), set_secure_boot_mode() now?15:05
*** yog_ has quit IRC15:06
rloothe fewer backwards compat stuff we have the better.15:06
jrollyeah, I suppose15:10
*** jerryz has joined #openstack-ironic15:12
jrollrloo: did you see devananda's comments here? https://review.openstack.org/#/c/129529/15:12
victor_lowtherGood morning, Ironic15:12
jrollheya victor_lowther :)15:12
*** erwan_taf has quit IRC15:13
rloojroll: thx, didn't see that (haven't gone through all my email)15:13
jrollnp15:14
rloojroll: that's fine then. although my comment was dec 8, and it seems a shame that the spec was held up for 2 months+...15:15
jrollrloo: yeah, your comments could have been addressed in that time :P15:15
jrollI think the hold up is actually people reviewing it15:16
rloojroll: 'people', not reviewing it. that's us :-(15:16
jrollI know15:16
jroll:/15:16
jrollreviewing is hard15:16
rlooit is hard and to be honest, i have no idea what to comment on any more.15:17
jrollI need to start using that dashboard15:17
openstackgerritShivanand Tendulker proposed openstack/ironic-specs: Ironic Management Interfaces to support UEFI Secure Boot  https://review.openstack.org/13584515:17
stendulker_rloo: Have updated the spec by changing the API names as get_secure_boot_mode() and set_secure_boot_mode()15:22
rloothx stendulker_. will look at it in a few minutes.15:22
jrolldevananda: want to unblock https://review.openstack.org/#/c/150784/4 ?15:23
stendulker_rloo: Thank you for all your comments. They are really valuable.15:23
rloostendulker_: yw.15:23
stendulker_Hi NobodyCam15:24
stendulker_NobodyCam: Good morning :)15:25
jrolldtantsur: https://review.openstack.org/#/c/151686/1/ironic/api/controllers/base.py did you want to actually move MIN_VER and MAX_VER or just the config stuff?15:25
dtantsurjroll, both15:26
jrolldtantsur: it's min/max for v1, seems like it should go there so we don't need to move stuff when we do a v215:26
dtantsurso that you don't have circular imports15:26
jrollyeah15:26
dtantsurjroll, not it's not for v1 :)15:26
jrollit is15:26
dtantsurit is in base class and applies to everything15:26
jrollthose are min/max minor versions15:26
dtantsurjroll, if it's for v1 it should not go to base.py15:27
dtantsurthe way it's implemented now is that it applies to all (1 of 1) version :)15:27
jrollwell15:27
*** erwan_taf has joined #openstack-ironic15:27
jrollso base.py handles version things15:27
jrollwe could theoretically have a min of 1.3 and max of 2.515:27
jrollwe need to parse version headers and whatnot whether it's v1 or something else15:28
jrollso that bit *should* be in base.py15:28
jrollidk, it's weird15:28
dtantsurjroll, ok, then let's move MIN_VER to base.py15:28
dtantsurespecially if you envision things like "a min of 1.3 and max of 2.5"15:29
jrollit's really MIN_V1_VER15:29
jrollbecause it's just the minor version that MIN_VER specifies15:29
jrollidk, maybe we should make it total version15:29
jrollyeah, let's do that15:29
*** achanda has joined #openstack-ironic15:30
NobodyCammorning stendulker_ :)15:30
*** jerryz has quit IRC15:31
adam_gderekh, does tripleo ci build a new deploy ramdisk each run? if so, is it using the most recent DIB release?15:31
derekhadam_g: yes and yes15:31
jrolladam_g: just blame fedora :P15:31
derekhadam_g: or at least if it isn't we have a problem15:32
stendulker_NobodyCam : Fedora element on DIB seems broken with Fedora stopped hosting cloud images at http://cloud.fedoraproject.org15:32
adam_gjroll, it shouldn't matter15:32
jrolladam_g: I know, I was joking15:32
jrollX-OpenStack-Ironic-API-Minimum-Version: 015:32
derekhthis is a serious channel15:32
jrollX-OpenStack-Ironic-API-Maximum-Version: 115:32
jrolldtantsur: ^ I don't love this15:32
NobodyCamstendulker_: humm I'd check in the #tripleo channel15:32
stendulker_NobodyCam: They are hosting at http://download.fedoraproject.org and image type is raw instead of qcow215:32
NobodyCamthere may be a patch already for that15:33
dtantsurstendulker_, I'm constantly using fedora element for building ramdisk, what do you mean by broken?15:33
stendulker_NobodyCam: Ok. I will sheck at  #tripleo channel15:33
stendulker_dtantsur: When I was trying to build new agent ramdisk, it complained. If you try to open http://cloud.fedoraproject.org you get redirected to http://download.fedoraproject.org15:34
dtantsurjroll, well.. honestly I'm a bit afraid that this micro-version stuff is going to complicate things a lot15:34
jrolldtantsur: why?15:35
stendulker_dtantsur: The URL and image types have changed15:35
dtantsurjroll, too many combinations to keep in mind and test15:35
adam_gderekh, cant seem to find what DIB version is being used here: http://logs.openstack.org/28/137028/11/check-tripleo/check-tripleo-ironic-overcloud-f20-nonha/c82decb/15:35
adam_gpip-freeze.txt is empty15:35
dtantsurstendulker_, curl/wget should handle the redirect... when did you last try?15:35
stendulker_dtantsur: Some 6hrs back.15:36
jrolldtantsur: yeah... I think we should move functional API testing stuff to our tree as well, I don't think it will be too bad, just need more context in the head15:36
dtantsurstendulker_, hmmm... strange, I've build a ramdisk with 'fedora ironic-discoverd-ramdisk' like 4hrs ago...15:37
adam_gderekh, i dont suppose you have some way of pulling a ramdisk from a failed run before its torn down?15:37
derekhadam_g: no, but I can run it locally and get you the ramdisk15:38
derekhadam_g: could take a little time though15:38
adam_gderekh, cool. that would be helpful. also, if you are running with libvirt, and you can install and use sgabios, you can capture console logs too15:38
derekhadam_g: ok, will give it a go15:39
adam_gderekh, thanks!15:39
derekhadam_g: I havn't use sgabios before is there anything I need to run (besides install it)15:40
stendulker_dtantsur: Its building now. I think curl was hanging 6hrs back and was not getting redirected.15:41
stendulker_dtantsur: Thank you :)15:41
adam_gderekh, install it, and  add <bios useserial='yes'/> to the <os> section of the libvirt domain15:42
adam_gderekh, oh wait, nvm thats not needed to capture the ramdisk output15:42
adam_gderekh, you can just set the vm to log console to a file15:43
*** achanda has quit IRC15:43
adam_gderekh, for example http://paste.ubuntu.com/10017379/15:44
derekhadam_g: ok, will give it a go, have to hack this into devtest15:46
adam_gderekh, FYI this is the related DIB change: https://review.openstack.org/#/c/140870/15:48
adam_gwhich was released in 0.1.3915:49
adam_gbut without that, i'd expect the error your seeing (BOOT_SERVER not set)15:49
derekhadam_g: ack15:50
mariosdtantsur: hey man, is this a known thing ? (from running devtest_undercloud waiting for the undercloud instance to come up, devtest_undercloud.sh. )  ironic conductor errors like http://paste.openstack.org/show/165639/15:52
marios(anyone else)15:52
* dtantsur is looking15:52
dtantsurderekh, does it ^^^ look similar?15:53
jrollthat would come from the current tripleo issues, yes15:54
mariosdtantsur: thanks gents. ftr it ran to completion on friday. i just reprovisioned the box from scratch so may be something that landed recently15:54
jrollmarios: is that the fedora job?15:54
mariosjroll: no on my lab box15:54
jrollmarios: oh, make sure your ramdisk is build with DIB 0.1.3915:54
mariosjroll: http://docs.openstack.org/developer/tripleo-incubator/devtest.html15:54
dtantsurbut still Fedora, I suspect?15:54
mariosdtantsur: yes15:54
dtantsuraha15:54
dtantsurI guess you're broken by the same thing15:54
dtantsurmarios, can you attach to the vm and see what's going on there?15:55
derekhYup, looks like https://bugs.launchpad.net/ironic/+bug/141702615:55
jrollany cores want to approve an early requirements bump? :) https://review.openstack.org/#/c/152127/15:55
jrollNobodyCam: ^^15:55
mariosdtantsur: yeah i am. the log is from the seed /var/log/messages15:56
adam_gjroll, dtantsur im fine with the revert at https://review.openstack.org/#/c/152129/15:56
mariosderekh: yes thanks, indeed it does15:56
*** jjohnson2 has joined #openstack-ironic15:56
jrolladam_g: then +1 it? :P15:57
adam_gdoh15:57
dtantsurjroll, adam_g, check-tripleo didn't pass on https://review.openstack.org/#/c/152129/15:57
dtantsurshould we recheck?15:58
adam_gdtantsur, is it the same issue? im having trouble finding any useful debug info in those jobs logs15:58
jrolldtantsur: yeah, let's recheck15:58
jrollit looks like setup scripts failed15:58
dtantsuradam_g, I don't think so, that's why I suggest rechecking15:59
*** ndipanov has quit IRC15:59
adam_gthat one looks like some devtest error with bootstrapping the seed node15:59
jrolljust submitted a recheck there15:59
jrollderekh: any news on if the revert worked for f20?15:59
adam_gi forgot my euro power adapter and need to go find one. back in a bit16:00
*** beekneemech is now known as bnemec16:00
*** ndipanov has joined #openstack-ironic16:01
derekhjroll: yup it did, added comment to bug16:01
jrollsweet16:01
jrolllet's get 152129 done then16:02
jrollit's rechecking now16:02
*** Marga_ has joined #openstack-ironic16:02
tteggel_hey ironiciers16:04
*** pcrews has joined #openstack-ironic16:04
*** tteggel_ is now known as tteggel16:04
jrollheya tteggel :)16:04
jrolloh that reminds me, you need juno patches approved yes?16:05
tteggelI have a stable patch that previously had a +W but needed a rebase, yeah16:05
tteggeljust 1 noe16:05
*** achanda has joined #openstack-ironic16:05
tteggelwhich is https://review.openstack.org/#/c/149876/16:05
jrolldevananda: adam_g ^16:05
jrolltteggel: those are all 2 of our stable cores :P16:06
tteggelthat's good to know16:06
NobodyCamthere was a reason we waited before landing that I seem to recall16:06
jrolloh?16:06
jrollhuh16:06
*** jjohnson2 has quit IRC16:07
tteggeladam_g: thanks :)16:07
*** anderbubble has joined #openstack-ironic16:10
*** achanda has quit IRC16:10
tteggelNobodyCam: one of the backports was a bit of a mess. but i cleaned up after myself16:11
*** zz_jgrimm is now known as jgrimm16:12
NobodyCamtteggel: ahh :)16:16
NobodyCamtteggel: Thank you :)16:16
*** david-lyle_afk is now known as david-lyle16:19
NobodyCambrb16:23
rloohey jroll, wrt https://review.openstack.org/152127, the line in global_requirements has '# Apache-2.0'16:25
rloojroll: does it matter? will it confuse the bot or whatever that generates those patches?16:25
*** lazy_prince is now known as killer_prince16:31
*** rameshg87 has joined #openstack-ironic16:36
*** rwsu-afk is now known as rwsu16:40
*** anderbubble has quit IRC16:40
jrollrloo: it should be fine but I can add it if you want16:43
rloojroll: if it is fine, then I'm ok. dtantsur said it was fine so16:44
jrollrloo: ok, cool16:44
rloojroll: done...16:44
jrollwoot16:45
jrollthanks!16:45
*** vdrok_ has joined #openstack-ironic16:45
*** rameshg87 has quit IRC16:46
derekhadam_g: my attempt to log the console resulted in a blank file, but there is the randisk and some screen shots http://goodsquishy.com/downloads/image-bug1417026/16:47
*** anderbubble has joined #openstack-ironic16:47
jrollone more patch for cores to +A, fixes tripleo f20 jobs: https://review.openstack.org/#/c/152129/16:47
jrollit's rechecking because tripleo failed, but derekh confirmed it works16:48
Shrewsjroll: hm, interesting16:51
jrollShrews: what's interesting about it?16:52
Shrewsjroll: that it only broke fedora16:52
jrollShrews: yeah...16:52
jrollweird16:53
* Shrews would be interested in seeing an analysis of "why"16:53
jrollI think that's what derekh is working on right now :P16:53
Shrewsi guess bug 1401298 needs reopened now16:54
derekhjroll: I havn't looked into why yet, but have it reproduced locally, I can dig into it later if needed but at the moment have some other tripleo things to deal with16:55
*** killer_prince has quit IRC16:55
*** killer_prince has joined #openstack-ironic16:56
*** killer_prince is now known as lazy_prince16:56
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Bump randomly set retry count and delay  https://review.openstack.org/15217916:57
*** lazy_prince has quit IRC16:58
*** killer_prince has joined #openstack-ironic17:01
*** killer_prince is now known as lazy_prince17:01
*** Marga_ has quit IRC17:02
Shrewswe don't have a meeting now?17:02
*** mjturek has quit IRC17:03
* Shrews is off by a week, methinks17:03
*** krtaylor has quit IRC17:04
jrollyou are :P17:04
jrollit's tonight17:04
*** ndipanov has quit IRC17:07
NobodyCambrb...bbt17:07
jrolladam_g: I re-opened https://bugs.launchpad.net/ironic/+bug/1401298 and left it assigned to you, hope that's ok17:08
*** eghobo has joined #openstack-ironic17:08
*** coolsvap is now known as coolsvap|afk17:12
*** chuckC_ has quit IRC17:12
*** krtaylor has joined #openstack-ironic17:12
*** eghobo has quit IRC17:13
*** ChuckC_ has joined #openstack-ironic17:14
*** eghobo has joined #openstack-ironic17:15
*** ChuckC has quit IRC17:18
*** EmilienM is now known as EmilienM|afk17:18
*** dtantsur is now known as dtantsur|afk17:23
openstackgerritMerged stackforge/ironic-discoverd: Bump randomly set retry count and delay  https://review.openstack.org/15217917:29
*** jistr has quit IRC17:39
*** jcoufal_ has quit IRC17:46
*** jlvillal has joined #openstack-ironic17:46
*** coolsvap|afk is now known as coolsvap17:52
*** derekh has quit IRC17:53
*** Haomeng has quit IRC17:56
*** Haomeng has joined #openstack-ironic17:57
*** davideagnello has joined #openstack-ironic17:58
*** spandhe has joined #openstack-ironic17:59
*** Marga_ has joined #openstack-ironic17:59
*** eghobo has quit IRC17:59
*** anderbubble has quit IRC18:00
*** EmilienM|afk is now known as EmilienM18:00
*** stendulker_ has quit IRC18:01
*** anderbubble has joined #openstack-ironic18:05
*** ijw has joined #openstack-ironic18:06
*** harlowja has joined #openstack-ironic18:11
*** athomas has quit IRC18:11
*** rlr0329 has joined #openstack-ironic18:15
rlr0329We have successfully deployed a 35-node site using Juno release of ironic with IPA.  Now looking to productionize the ironic services.  Any tips or best practices?18:15
NobodyCamrlr0329: awesome!!!18:17
openstackgerritMerged openstack/ironic: Simplify policy.json  https://review.openstack.org/14838118:17
JayFgreat job rlr032918:17
*** ijw has quit IRC18:17
rlr0329Thanks :-)18:18
openstackgerritMerged openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/14997818:18
*** ijw has joined #openstack-ironic18:18
anderbubbleForgive me, as I expect this is meant to be clear; but what does “IPMI terminal port not supplied to IPMI driver.” mean I should do?18:19
anderbubble(in node-validate, for console)18:19
NobodyCamanderbubble: sounds like the port is not set18:19
anderbubbleI found the bug at https://bugs.launchpad.net/ironic/+bug/1379911 requesting added clarity; but I don’t know where or how to set the port.18:20
*** Haomeng|2 has joined #openstack-ironic18:20
*** ChuckC has joined #openstack-ironic18:21
*** Haomeng has quit IRC18:21
anderbubbleHeh: looks like this error message also appears in xcat… how could that code have possibly gotten in there? ;)18:22
*** achanda has joined #openstack-ironic18:23
*** ChuckC_ has quit IRC18:24
anderbubbleNobodyCam, so I see that the node apparently needs a driver_info/port; but what port is that? Like COM1? How do I determine the correct port to put there? (Sorry: I haven’t ever really grok’d SoL)18:25
*** jjohnson2 has joined #openstack-ironic18:25
NobodyCamanderbubble: what driver?18:27
anderbubblepxe_ipmitool18:27
NobodyCamanderbubble: yep: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipmitool.py#L100218:28
NobodyCamthat will be the shellinabox port18:28
*** ChuckC has quit IRC18:28
anderbubbleOh; so this is a tcp port?18:29
NobodyCamanderbubble: https://help.ubuntu.com/community/shellinabox18:29
anderbubbleThat is *not at all* what I thought it was asking for. :p18:29
jjohnson2shellinabox is crappy18:29
NobodyCamlol morning jjohnson218:30
NobodyCam:)18:30
jjohnson2I'm my usual polite self18:30
anderbubbleNobodyCam, jjohnson2, So, presumably each node needs its own port, then, if you want to be able to access nodes simultaneously?18:30
jjohnson2NobodyCam, morning18:30
jjohnson2anderbubble, so I needto make a recording...18:30
anderbubbleAnd I’m my usual “please help me I can’t get Ironic working” self ;)18:30
*** Marga_ has quit IRC18:30
jjohnson2anderbubble, and I might be distracting from ironic specifically, though I think ironic would benefit through the thing I'm about to record18:31
jjohnson2I came in late to the conversation, but I'm guessing we are talking about console access18:32
anderbubbleFor the record, though: I actually got a node to almost completely provision at the very end of the day on Friday; though the setup didn’t work (it didn’t get a hostname, for example)18:32
jjohnson2web based or cli based or both?18:32
anderbubbleso I’m trying to get serial console working so I don’t have to keep walking to the DC ;)18:32
anderbubblejjohnson2, cli18:32
NobodyCamanderbubble: that sounds like a cloud-init thing18:32
jjohnson2so I got a present for you then18:32
jjohnson2oh, you know about my present18:32
jjohnson2the thing I mentioned the other day18:32
anderbubbleI’ll try horizon once I get it working reliably from the console18:33
anderbubblejjohnson2, yeah, I haven’t looked at it yet; but it did make it into org-mode ;)18:33
*** eghobo has joined #openstack-ironic18:33
jjohnson2yeah, install confluent rpm18:33
jjohnson2then start confluent service18:33
openstackgerritMerged openstack/ironic: Bump oslo.rootwrap to 1.5.0  https://review.openstack.org/15212718:33
jjohnson2then you can confetty create /nodes/thisnodeisbob secret.hardwaremanagementuser=bigguy secret.hardwaremanagementpassword=supersecret hardwaremanagement.manager=my.bmc.here18:34
jjohnson2oh and console.method=ipmi18:34
jjohnson2then you can 'confetty thisnodeisbob'18:34
jjohnson2and you hav econsole18:34
openstackgerritMerged openstack/ironic: Revert "Do not pass PXE net config from bootloader to ramdisk"  https://review.openstack.org/15212918:35
anderbubbleNobodyCam, so is this a server port that I want shellinabox to use; or is it a client port that I need to have shellinabox already running on to connect to?18:36
*** wanyen has joined #openstack-ironic18:36
anderbubbleNobodyCam, that is, am I defining a new port here, or referencing an existing port (and need to go set up shellinabox out-of-band)?18:36
*** Marga_ has joined #openstack-ironic18:38
wanyenNobodyCam, Just want to confirm that we wil have ironic weekly meeting today at 0500 UTC?  The  ironic wiki https://wiki.openstack.org/wiki/Meetings/Ironic has not been updated yet. It still says next meeting is 01/26.18:38
NobodyCamwanyen: gah... yes I beleieve we are having a meeting18:39
wanyenNobodyCam, okay.  I just want o make sure because I invited Jim to attend.  It wil be 10pm his time.18:40
*** rlr0329 has quit IRC18:43
*** ChuckC has joined #openstack-ironic18:47
anderbubblejjohnson2, I see that the ipminative.py module is already in the ironic that I have installed; so can it be used with pyghmi already in Ironic?18:50
*** achanda has quit IRC18:53
*** mrmoje has quit IRC18:53
anderbubblejjohnson2, also, why isn’t pxe_ipminative being “tested in the upstream gate” ?18:54
jjohnson2anderbubble, yeah, current pyghmi should be good for both18:55
jjohnson2I don't know about pxe_ipminative testing at the moment,  I will confess to not being up to date on how pyghmi is being consumed18:55
*** mrmoje has joined #openstack-ironic18:55
jjohnson20.6.23 was released to address some issues in someones CI efforts...18:56
*** achanda has joined #openstack-ironic18:57
*** ChuckC has quit IRC19:01
*** ChuckC has joined #openstack-ironic19:02
*** ijw has quit IRC19:02
*** chuckC_ has joined #openstack-ironic19:04
*** dprince has quit IRC19:06
jrollanderbubble: does 'ipmitool... sol activate' not work?19:07
*** wanyen has quit IRC19:07
jrollanderbubble: pyghmi isn't tested in the upstream gate because the upstream gate doesn't have real hardware19:07
* NobodyCam need to make the time to play with jjohnson2's ipmi listening stuff19:08
*** ijw has joined #openstack-ironic19:08
*** pelix has quit IRC19:09
anderbubblejroll, it connected; but no echo19:10
jrollhrm19:10
anderbubblenot sure why; I’m going to try reprovisioning the node and see if maybe the provisioned image just doesn’t report on the serial console19:10
jrollmight need to set the console device differently in your kernel command19:11
jrollor the baud rate19:12
jrolle.g. console=ttyS4,115200n819:12
anderbubblejroll, where can I pass kernel parameters?19:12
* anderbubble should just google that19:12
jrollthere's a config option for extra parameters19:12
jrollbecause ironic does it19:12
jjohnson2NobodyCam, yeah, I'm trying to figure out how to record my desktop sanely19:13
jrollanderbubble: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L1189-119119:13
jjohnson2last time I was in Linux and it was straightforward, now that I'm on windows, it's harder19:13
anderbubblejroll, thanks19:13
jrollnp19:13
openstackgerritMerged openstack/python-ironicclient: Check if --config-drive is only used with provision state "active"  https://review.openstack.org/15127019:16
*** andreykurilin_ has joined #openstack-ironic19:19
*** harmw has joined #openstack-ironic19:20
harmwis there a dummy power driver for when I want to boot a machine my manually pressing the pwr button? :)19:21
harmwor do I rly need to build this myself19:21
openstackgerritMerged openstack/python-ironicclient: ironicclient node-set-maintenance to accept true/false  https://review.openstack.org/15069219:21
*** ChuckC has quit IRC19:23
*** EmilienM is now known as EmilienM|afk19:25
*** andreykurilin_ has quit IRC19:26
jrollharmw: I don't believe there is in tree, maybe the fake power driver19:30
jrollhas anyone thought about suspend/resume yet?19:30
jrollI know that's VM terminology, but would be nice to support it somehow (even if it's just power off/on)19:30
*** dprince has joined #openstack-ironic19:32
harmwjroll: so there is atleast something I could base it off of?19:33
jrollharmw: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/fake.py#L3419:34
harmwyea, I'm just reading that19:34
*** Marga_ has quit IRC19:34
NobodyCamharmw: there was this awhile ago: https://review.openstack.org/#/c/98950/19:34
*** Marga_ has joined #openstack-ironic19:35
*** penick has joined #openstack-ironic19:35
harmwNobodyCam: thats exactly what I'm after19:35
NobodyCam:p19:36
*** coolsvap is now known as coolsvap|afk19:36
harmwto bad it's abandonent19:37
NobodyCamoh brb... quick walkies19:38
*** Marga_ has quit IRC19:38
*** Marga__ has joined #openstack-ironic19:38
*** Marga__ has quit IRC19:39
*** Marga_ has joined #openstack-ironic19:39
*** Marga_ has quit IRC19:43
*** Marga_ has joined #openstack-ironic19:43
*** Marga_ has quit IRC19:45
*** Marga_ has joined #openstack-ironic19:46
*** EmilienM|afk is now known as EmilienM19:50
*** Marga_ has quit IRC19:54
*** Marga_ has joined #openstack-ironic19:54
*** eghobo has quit IRC19:54
*** achanda has quit IRC19:57
*** Marga_ has quit IRC19:59
*** anderbubble has quit IRC20:01
*** ijw_ has joined #openstack-ironic20:06
*** achanda has joined #openstack-ironic20:06
*** ijw has quit IRC20:08
*** anderbubble has joined #openstack-ironic20:13
*** dprince has quit IRC20:13
harmwlifeless: will you ever complete that pxe_logpower driver?20:15
harmwor is it almost identical to FakePower?20:16
*** mrda-away is now known as mrda20:16
mrdaMorning Ironic20:16
NobodyCamharmw: I do not expect that to ever land in tree20:17
NobodyCammorning mrda20:17
NobodyCam:)20:17
spandhehello!20:17
*** ijw_ has quit IRC20:18
NobodyCamhi20:19
*** pensu has quit IRC20:21
harmwNobodyCam: me neither :) how should I enable the fake driver under enabled_drivers?20:22
*** mjturek has joined #openstack-ironic20:22
NobodyCamharmw: add the fake version of the driver you want to use: https://github.com/openstack/ironic/blob/master/setup.cfg#L41-L5220:23
harmwah20:23
NobodyCamto enabled_drivers20:24
harmwshould've thought of that myself20:24
*** Marga_ has joined #openstack-ironic20:25
NobodyCam:)20:25
*** pensu has joined #openstack-ironic20:28
*** Marga_ has quit IRC20:30
harmwhm, and how would I be able to select the Fake driver when registering a new node?20:30
harmwenabled_drivers = pxe_ssh,pxe_ipmitool,fake20:31
*** ijw has joined #openstack-ironic20:32
NobodyCamharmw: fake has all interfaces mocked.. If your wanting to do something like an actual deploy you want a driver like fake_ipmitool20:36
*** dprince has joined #openstack-ironic20:37
NobodyCamoh wait20:38
NobodyCamhttps://github.com/openstack/ironic/blob/master/ironic/drivers/fake.py#L7020:38
NobodyCamfake_pxe20:38
NobodyCam:-p20:38
*** krtaylor has quit IRC20:40
NobodyCambrb20:42
Shrewsgah20:43
Shrewsi will never understand how infra gets devstack to run on hp instances. i can NEVER get it to work "out of the box"20:44
harmwah cool20:46
harmwand shouldn't tuskar display it as an option?20:47
*** achanda has quit IRC20:50
harmwhm, so tuskar doesn't use the enabled_drivers from ironic? the dropdown isn't updated..20:51
NobodyCamharmw: I don't use taskar. :(20:54
NobodyCammaybe check in the TripleO channel20:54
*** ChuckC has joined #openstack-ironic20:57
*** dprince has quit IRC21:00
harmwok sure21:03
*** eghobo has joined #openstack-ironic21:05
*** achanda has joined #openstack-ironic21:13
*** krtaylor has joined #openstack-ironic21:13
NobodyCamtrown: you around?21:14
trownindeed21:14
NobodyCamhey hey :)21:14
anderbubbleany reason why the adminPass displayed by nova wouldn’t work in the baremetal instance?21:15
NobodyCamjust checking you have a item on the agenda ... is that still open? can I remove it?21:15
anderbubbleDoes cloud-init need to have been able to use the network to set it?21:15
NobodyCamanderbubble: sounds like cloud-init not working21:15
NobodyCamyep21:16
jrollanderbubble: correct, it needs to be able to reach the metadata service21:16
trownNobodyCam: it is still open, I got sidetracked with some other stuff, I will finish it up this week though21:16
trownoh, the agenda item is done with21:16
NobodyCamok so I should leave it on the agenda21:16
anderbubblenetwork is failing to start in my baremetal instance, and I don’t know how to debug it without being able to get into the instance in its provisioned state :/21:16
jrolltrown: do you need it on this week's... ok21:16
NobodyCamahh ok21:16
jrollwe can remove it then21:16
trownI have action items, no need for discussion21:16
NobodyCami'm removing21:16
trowncool21:16
jjohnson2all right, the most boring youtube video ever is about to be live21:16
NobodyCamjjohnson2: not borning if its setting up the ipmi listener21:18
jjohnson2NobodyCam, it's 15 minutes, that's like 14 minutes and 50 seconds longer than my attention span21:18
jjohnson2people have told me to talk more slowly and elaborate more on things, so I made a very concious effort21:19
NobodyCam:)21:19
jjohnson2NobodyCam, anderbubble: youtu.be/8NEgyH_4gfo21:21
anderbubblejjohnson2, I’ll take a look once I’m not in the dc ;)21:21
*** Haomeng|2 has quit IRC21:26
*** Marga_ has joined #openstack-ironic21:26
*** Haomeng|2 has joined #openstack-ironic21:28
*** Marga_ has quit IRC21:31
openstackgerritMichael Davies proposed openstack/ironic: Add logical name support to Ironic  https://review.openstack.org/14173721:32
NobodyCambrb21:35
*** Marga_ has joined #openstack-ironic21:41
*** kevinbenton_ is now known as kevinbenton21:41
jjohnson2and now someone is saying I should have made it more terse... oh well, I can't do a video to please everyone21:41
NobodyCamlol21:42
*** davideagnello has quit IRC21:43
*** pensu has quit IRC21:45
*** bradjones has quit IRC21:57
*** bradjones has joined #openstack-ironic21:58
*** bradjones has quit IRC21:58
*** bradjones has joined #openstack-ironic21:58
*** penick has quit IRC22:01
anderbubblejjohnson2, you should make a video that tells me what I need to know without me having to watch it ;)22:01
jrolldevananda: ping22:02
jjohnson2anderbubble, well to be fare I go off topic to point out the stuff that I think is noteworthy for not normally being noticable22:02
jrolldevananda: wondering what you think about returning e.g. 1.0 for api minimum version, rather than just 022:02
*** andreykurilin_ has joined #openstack-ironic22:04
NobodyCamjroll: its late at night for him now22:04
jrollNobodyCam: I'm aware22:05
jrollhe can respond as needed22:05
NobodyCam:) ok heheeh22:05
anderbubbleok, so here’s the most recent reason my ironic node won’t boot:22:06
anderbubbleit’s trying to configure eth0; but my node doesn’t have an eth0; it has an eno122:07
anderbubble*provisioning* works; it’s only the actual boot into the provisioned os that fails, which, of course, causes cloud-init to fail22:07
anderbubblepresumably that’s the fault of the actual boot image, yes?22:08
NobodyCamanderbubble: sounds like it22:08
anderbubbleso, who’s the right person to talk to about it being broken?22:10
anderbubbleand if I try a different image, is it likely to have the correct interface in it? or is this something I can configure in my own diskimage-builder element?22:10
NobodyCamanderbubble: https://github.com/coreos/bugs/issues/3622:11
NobodyCamlook like your issue?22:11
anderbubbleno; my issue is that the provisioned image only has a /etc/sysconfig/network-scripts/ifcfg-eth0; but the same file should, in stead, actually be ifcfg-eno1. The device just has a different name than the provisioned image expects.22:12
anderbubbleI don’t know where the responsibility for configuring the network in the provisioned image lies.22:13
jrollwhere did your image come from?22:13
anderbubbledisk-image-create centos7 baremetal -o my-image22:13
jrollmmm22:14
jrollyeah, I'd poke #tripleo about it then, I think22:14
anderbubblewill do! thanks as always!22:14
jroll:)22:14
*** ijw has quit IRC22:15
*** anderbubble has quit IRC22:15
*** andreykurilin|ho has joined #openstack-ironic22:17
*** andreykurilin_ has quit IRC22:17
*** anderbubble has joined #openstack-ironic22:22
*** andreykurilin|ho has quit IRC22:23
*** andreykurilin_ has joined #openstack-ironic22:24
*** jjohnson2 has quit IRC22:28
anderbubbleironic node-list shows two of my nodes in “maintenance” mode, but ironic node-set-maintenance claims it doesn’t exist! :/22:39
*** davideagnello has joined #openstack-ironic22:42
anderbubble(by which I mean ironic claims the command node-set-maintenance doesn’t exist)22:45
jrollupdate your client maybe?22:46
devanandajroll: oh. we're going to need to add microversion awareness to the client, which will be a little trickier22:47
devanandajroll: like auto discovering what commands, or states, or things, a given endpoint supports22:48
jrolldevananda: right...22:48
devanandaI mean,  handling multiple endpoints in a way that doesn't absolutely make our users want to kill us ...22:48
jrollright22:49
jrolldevananda: also, wdyt about returning the full versino for min/max version?22:49
devanandawe probably need some local file cache of endpoint,min,max and decorators around functions, CLI commands, etc ... justlike the server is going to need22:49
devanandajroll: ++22:49
jroll1.0 rather than 022:49
jrollok22:49
devanandajroll: total oversight on my part.22:49
jrollheh22:50
jrolland the 4 +2'ers :P22:50
devanandahehe22:52
anderbubblehow can I clear a node reservation when the conductor has failed to clean it up?22:56
jrollupdate nodes set reservation=null where uuid='blah';22:57
jroll:/22:57
NobodyCamanderbubble: ^^^22:57
jrollerr22:57
NobodyCamty jroll :)22:57
jrollanderbubble: was the conductor restarted?22:57
anderbubblejroll, yes22:57
jrollok yeah22:57
jrollthere's also a recent patch to clear on startup22:58
anderbubbleah22:58
jrollidk what version you're running22:58
NobodyCamanderbubble: are you trunk or packager version?22:58
jrollhttps://review.openstack.org/#/c/149038/22:58
anderbubblejroll, openstack-ironic-conductor-2014.2-2.el7.centos.noarch22:58
jrollI think that actually got backported to stable/juno22:58
*** yjiang5 has quit IRC22:58
*** alex_xu_ has quit IRC22:58
jrollbut idk if released yet22:58
jrollno idea how backports work22:58
openstackgerritDan Prince proposed openstack/ironic: Partial revert of 4606716 until we debug further  https://review.openstack.org/15232822:59
devanandait has not been released yet22:59
devanandaa new rev of the stable/juno branch will get tagged this week, which will then take some time to trickle through the distros23:00
jrolldevananda: can you remove your -2 here https://review.openstack.org/#/c/150784/23:00
jrollplease and thanks23:00
* anderbubble feels dirty when he mixes SQL and openstack23:00
Shrewsdevananda: i think my brain is suffering burnout on your MANAGEABLE state change, possibly not through any fault of yours  :(23:00
*** yjiang5 has joined #openstack-ironic23:01
*** alex_xu has joined #openstack-ironic23:01
Shrewsdevananda: does this stack trace make any sense to you? (from a failed deploy using your change)   http://paste.openstack.org/show/165812/23:01
Shrewsam i seeing a failed deploy trying to deploy again? or do i need to get away from my laptop?23:02
devanandajroll: done23:02
jrollthank you23:02
devanandaShrews: I'm sad that my patch is in any way hurting your brain23:03
jrollShrews: that could just be a race, where the token expires between the request to ironic and the request from ironic to say glance23:03
Shrewsdevananda: it's only the surrounding testing of your patch, which i blame on setting up devstack on HP cloud instances23:03
devanandaShrews: my first thought was what jroll just said23:04
jrollShrews: devstack works fine on rackspace cloud :P23:04
devanandaShrews: how long did the deploy take (time from "nova boot" until this crash)23:04
Shrewstoo long. i'm sure it hit the timeout.  30min ?23:04
devanandayup23:04
devanandasounds right23:04
Shrewswell, on the plus side, the state changes seem to be sane23:07
devanandahehe23:08
NobodyCamoh wow just saw dan's bug and patch23:09
devanandaanyone want to critique a talk abstract i'm about to submit to oscon?23:09
devanandaNobodyCam: eh?23:10
jrollNobodyCam: hrm23:10
NobodyCamdevananda: https://bugs.launchpad.net/ironic/+bug/1417307 / https://review.openstack.org/15232823:10
jrollNobodyCam: so it wasn't the pxe thing that broke them?23:11
jrollor maybe both23:11
*** chlong has joined #openstack-ironic23:11
NobodyCamI don't know whats up yet23:12
devanandaso adam_g broke something in tripleo ci recently23:12
devanandai wonder if this is related23:12
Shrewsdevananda: we reverted one of adam's changes this morning23:12
jrolldevananda: we reverted adam's thing already, jfyi23:13
devanandaShrews: k k23:13
jrollbut that's a good point23:13
jrollif networking wasn't coming up on the ramdisk, iscsi will never succeed23:13
jrollsucceed/connect23:13
jrollbut would it even try without getting the callback?23:13
NobodyCamjroll: I would hope not23:14
NobodyCam:-p23:14
jrollright, ok23:14
jrollso maybe both things broke their f20 jobs23:14
jrollbut they couldn't see this one until the first was fixed23:14
NobodyCamyea ..23:15
NobodyCamthat would make the most sense23:15
NobodyCamjust strange that the sleep fixes the issue where activlty checking and forcing a re-read of the luns causes it to break23:16
jrollyeah, that is weird23:17
jrollmaybe fedora has a newer iscsid or whatever?23:17
jrollwith different options23:17
NobodyCamlazy_prince: you happen to have a log you can point me at23:17
jrollidk23:17
NobodyCamya23:17
NobodyCamthat we have seen before23:17
*** penick has joined #openstack-ironic23:20
jrolldevananda: NobodyCam any opposition to me making the agent_ssh job vote on ironic patches?23:23
jrollopposition/objection?23:23
devanandajroll: yup. transitive dependencies23:23
jrolleh?23:23
devanandajroll: AFAIK, doing that today will put those jobs in the global gate queue23:24
jrollfor other projects?23:24
jrollsigh23:24
devanandanot that they'll vote on nova23:24
devanandano23:24
jrolloh23:24
jrollso what's the problem?23:24
devanandait makes the gate longer23:25
devanandathe grand unified gate23:25
jrollso it does run on other projects?23:25
devanandano23:25
jroll(if it votes on ironic)23:25
jrollso uh23:25
jrollhow does it make the gate that much longer?23:25
jrollit will increase it just by n * m, where n is number of ironic jobs and m is the time to run the agent job, right?23:26
devanandayup23:26
devanandaand any instability in that job will exponentially effect all projects23:26
jrollis that really a reason to make it not vote, though?23:27
devanandatldr; i am proxying for clark and sdague right now, because i understand their concerns (or think I do)23:27
jrollthe only instability we've ever seen in that job is general ironic instability23:27
devanandaif they are willing to accept it, I'm fine with it voting on ironic23:27
jrollyeah, I mean right now we effectively aren't testing half of our deploy drivers23:28
devanandaright23:28
*** ijw has joined #openstack-ironic23:28
devanandai maen, I want it to be voting on ironic23:28
jrollwhich is pretty lame23:28
jrollimho23:28
jrollok23:28
devanandabut i also dont want to put more strain on infra/qa23:29
jrollyeah23:29
*** anderbubble has quit IRC23:33
*** mjturek has quit IRC23:35
*** vdrok_ has quit IRC23:35
*** mlsoule has quit IRC23:38
*** derekh has joined #openstack-ironic23:39
devanandajroll: adam_g: have I been imaging it, or is check-tempest-dsvm-ironic-parallel-nv is looking fairly stable lately?23:45
jrollI haven't been paying a ton of attention but I feel like I see it failing somewhat regularly23:46
devananda:-/23:47
jrollI might only notice when it fails23:48
jrolldon't rely on my memory :x23:48
* devananda relies more on adam's stats than his own memory, too23:52

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