Friday, 2015-02-13

jogoan ironic image: http://www.angmohdan.com/wp-content/gallery/ironic/ironic-pics-14.jpg00:05
*** jrist has joined #openstack-ironic00:09
*** krtaylor has joined #openstack-ironic00:10
*** naohirot has joined #openstack-ironic00:12
*** romcheg has quit IRC00:22
naohirotgood morning ironic00:22
*** lnxnut has joined #openstack-ironic00:25
*** davideagnello has quit IRC00:27
Shrewsmrda: approved!!00:27
jrollpy27 recreate: /data/dev/nova/.tox/py2700:28
* jroll cries00:28
*** anderbubble has joined #openstack-ironic00:29
*** Nisha has joined #openstack-ironic00:30
Nishadevananda, around?00:31
*** davideagnello has joined #openstack-ironic00:32
*** lnxnut has quit IRC00:33
NishaNobodyCam, there?00:33
*** davideagnello has quit IRC00:33
*** davideagnello has joined #openstack-ironic00:33
*** david-lyle is now known as david-lyle_afk00:37
jlvllal_sfoNisha: Discussion going on at the moment00:37
jlvllal_sfoAt sprint00:37
Nishaohk00:37
jlvllal_sfoNisha: Maybe you can look at my patch :)  https://review.openstack.org/15552900:38
jlvllal_sfoSince you have been doing states.py changes.00:38
*** martini has quit IRC00:41
Nishajlvllal_sfo, i had a look at it it looks ok as of now. but I will review it in office(i.e.after approx 3hrs)....Right now its early morning here and time to get ready for morning work and get ready for office.00:45
jlvllal_sfoNisha: Thanks!00:45
NishaI dont know how much changes will it bring to my work though :(00:46
Nishaif you have chance to speak to devananda please request him to look at https://review.openstack.org/147857 so that we can get answers to the queries and we can proceed further on inspectio00:47
jlvllal_sfoNisha: I think very minimal.  maybe 1-2 lines00:47
Nishajlvllal_sfo, it would be great if you could also review it00:47
NishaIs gate issue fixed now?00:47
jlvllal_sfoNisha: I'll try.  And yes they are from what I have been told00:48
Nishayesterday jenkins was failing for every patch00:48
Shrewsgate should be fixed00:48
NishaShrews, thats great if it is fixed. I will repost the patches then later today00:49
rlooShrews: I added a print statement in one of the tests, and looked in .testrepository/* to see if it was printed.00:56
Shrewsrloo: as did i, but it was not printed  :(00:56
rlooShrews: that's odd.00:57
Shrewsrloo: how did you run the test?00:57
rlooShrews: it is the same mechanism used in ironic.00:57
rlooShrews: I use tox00:57
rlootox -epy2700:57
*** Nisha has quit IRC00:58
Shrewsyeah, i tried that too.00:58
rlooShrews: do you know if it works for you in ironic?00:58
Shrewsrloo: i don't know. didn't try there00:58
rlooShrews: am wondering if there is something else that has to be done. you don't have those env vars set to 0, do you?00:58
Shrewsi explicitly set them to 1, and tried not setting them at all (should default to 1, right). i spent waaaay too much time trying to figure it out. i may try again tomorrow00:59
rlooShrews: hmm. Shouldn't be that difficult. I'll build a new env and test it out again.00:59
*** takadayuiko has joined #openstack-ironic01:02
naohirotrloo: good morning, I just replied in terms of driver_info['irmc_sensor_method'].01:02
rlooShrews: unless you put the print in a test that doesn't inherit from utils.BaseTestClass01:02
rloohi naohirot01:02
Shrewsrloo: it did01:02
rlooShrews: hmm. ok, will need a few minutes to set up a dev environment and test...01:03
Shrewsrloo: it could TOTALLY be something stupid by me. it's been a long day01:03
naohirotrloo: please check it again if you got a time.01:04
rlooShrews: stupid or not, worth figuring out ;)01:04
rlooShrews: but maybe it is time for you to work over a beer or two with your buddies there01:04
Shrewsrloo: if you insist01:04
rloonaohirot: will look in a few minutes. I hope. I'm not really working now but I guess I am ;)01:05
rlooShrews: I insist unless your boss sez otherwise ;)01:05
*** Haomeng|2 has joined #openstack-ironic01:06
rloonaohirot: I think there is some confusion.01:06
*** Haomeng has quit IRC01:06
rloonaohirot: I agree that parse_driver_info() checks that 'irmc_sensor_method' exists in the dictionary that it returns.01:06
*** korekhov has joined #openstack-ironic01:07
rloonaohirot: but where the code is, you aren't using the dictionary returned from parse_driver_info. Isn't it trying to get it directly from node.driver_info?01:07
rloonaohirot: https://review.openstack.org/#/c/146803/19..20/ironic/drivers/modules/irmc/management.py01:08
rloonaohirot: line 210: sensor_method = task.node.driver_info['irmc_sensor_method']01:08
rloonaohirot: task.node.driver_info['irmc_sensor_method'] may not exist.01:09
rloonaohirot: info['irmc_sensor_method'] does exist, if info = irmc_common.parse_driver_info(task.node)01:09
naohirotrloo: Umm, I'm thinking01:10
naohirotrloo: irmc_common.parse_driver_info(task.node) doesn't make sure sensor_method = task.node.driver_info['irmc_sensor_method']?01:13
rloonaohirot: no, it doesn't. it gets it from driver_info.get('irmc_sensor_method'), and if it is None, it gets the value from the config.01:14
rloonaohirot: didn't you write that code?01:14
naohirotrloo: I still don't understand why it doesn't make sure.01:14
naohirotrloo: Yes01:14
rloonaohirot: where/how do you think the code makes sure that driver_info['irmc_sensor_method'] exists?01:15
naohirotrloo: I think  task.node.driver_info is singleton.01:15
rloonaohirot: what do you mean 'singleton'? .driver_info is a dictionary.01:16
naohirotrloo: I mean driver_info exists only one instance01:16
rloonaohirot: each node has a driver_info dictionary01:17
naohirotrloo: so  task.node.driver_info instance is same as info = irmc_common.parse_driver_info(task.node)01:18
rloonaohirot: no, it isn't01:18
rloonaohirot: https://review.openstack.org/#/c/146803/20/ironic/drivers/modules/irmc/common.py01:19
naohirotrloo: but why the test case I added succeed?01:19
rloonaohirot: line 9801:19
rloonaohirot: d_info = dict(list(req.items()) + list(opt.items()))01:19
*** achanda has quit IRC01:19
rloonaohirot: line 118: return d_info01:19
rloonaohirot: d_info is not the same as node.driver_info01:20
naohirotrloo: I got it01:21
naohirotrloo: I'll assign the d_info at line 118 to task.node.driver_info01:22
rloonaohirot: I'm glad. because I'm not sure how I could have explained it01:22
naohirotrloo: thinks for your insight :)01:23
*** chenglch has joined #openstack-ironic01:23
rloonaohirot: umm, wait. what did you just say.01:25
rloonaohirot: I'll assign the d_info at line 118 to task.node.driver_info01:25
rloonaohirot: no, I don't think the code should be modifying driver_info01:25
naohirotrloo: you are saying that the d_info at line 118 is abandoned, right?01:27
*** jlvllal_sfo has quit IRC01:27
rloonaohirot: d_info is returned. the caller of parse_driver_info gets d_info, and can use that information.01:27
openstackgerritJosh Gachnang proposed openstack/ironic: WIP: Implement execute clean steps  https://review.openstack.org/15556101:28
naohirotrloo: Yeah, I have to checked the caller how the caller treat the returned the d_info at line 118.01:28
rloothe caller can do whatever it wants with d_info, it is just a dictionary/object. it isn't owned by anything.01:29
naohirotrloo: but I understood that you are saying you could find the code the d_info at line 118 is assigned to  task.node.driver_info, am I correct?01:30
naohirotrloo: /could/could not/01:30
rloonaohirot: I don't know what you mean, it is getting late, and I have other things I have to get done. I am saying that you should NOT modify node.driver_info.01:31
naohirotrloo: If CONF.irmc.sensor_method is None, then driver_info[irmc_sensor_method] should have 'ipmitool', I think.01:32
naohirotrloo: that is the code I wrote.01:32
naohirotrloo: anyway I'll add test case which proof driver_info[irmc_sensor_method] and key irmc_sensor_method exit.01:34
naohirotrloo: before you leave, I'd like to know your solution if you have in your mind.01:36
*** korekhov has quit IRC01:37
*** korekhov has joined #openstack-ironic01:37
naohirotrloo: good night, thanks for sparing your time :)01:38
*** Marga_ has quit IRC01:39
*** korekhov has quit IRC01:42
*** ChuckC has quit IRC01:45
*** jerryz has joined #openstack-ironic01:46
* naohirot rloo: I fixed like this: d_info = irmc_common.parse_driver_info(task.node)01:46
naohirot        sensor_method = d_info['irmc_sensor_method']01:46
*** r-daneel has quit IRC01:48
*** korekhov has joined #openstack-ironic01:50
*** ChuckC has joined #openstack-ironic01:52
*** davideagnello has quit IRC01:52
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/15557101:52
*** anderbubble has quit IRC01:58
*** davideagnello has joined #openstack-ironic01:58
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/15558302:00
*** davideagnello has quit IRC02:04
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver  https://review.openstack.org/14680302:09
openstackgerritMerged openstack/ironic-python-agent: Log required troubleshooting info on image dl fail  https://review.openstack.org/15508002:17
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Virtual Media Deploy module for iRMC Driver  https://review.openstack.org/15195802:23
*** anderbubble has joined #openstack-ironic02:24
*** Haomeng has joined #openstack-ironic02:24
*** spandhe has quit IRC02:24
*** Haomeng|2 has quit IRC02:25
*** mlsoule has joined #openstack-ironic02:27
*** mlsoule has quit IRC02:28
*** ramineni has joined #openstack-ironic02:41
*** anderbubble has quit IRC02:47
*** zhenzanz has joined #openstack-ironic02:47
*** anderbubble has joined #openstack-ironic02:47
*** yuanying_ has joined #openstack-ironic02:57
*** yuanying_ has quit IRC02:59
*** yuanying_ has joined #openstack-ironic02:59
*** yuanying has quit IRC03:00
*** korekhov has quit IRC03:03
mrdaHas everyone gone to dinner?03:07
mrdaShrews: Thanks for the +A03:07
mrda(too bad things failed in gate due to (what looks like) a transient network failure :(03:07
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver  https://review.openstack.org/15480803:19
*** yuanying_ has quit IRC03:19
*** rloo has quit IRC03:23
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver  https://review.openstack.org/15481403:23
*** yuanying has joined #openstack-ironic03:24
*** jlvllal_sfo has joined #openstack-ironic03:27
*** yuanying has quit IRC03:27
*** harlowja_ is now known as harlowja_away03:27
*** yuanying has joined #openstack-ironic03:28
*** jlvillal_sfo has joined #openstack-ironic03:29
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for agent_ilo driver  https://review.openstack.org/15481603:31
openstackgerritTan Lin proposed openstack/python-ironicclient: Enable ironicclient to support --ironic-api-version 1.x  https://review.openstack.org/15562403:33
*** yuanying has quit IRC03:34
*** yuanying has joined #openstack-ironic03:36
*** yuanying_ has joined #openstack-ironic03:38
*** ChuckC_ has joined #openstack-ironic03:40
*** yuanying has quit IRC03:40
*** Marga_ has joined #openstack-ironic03:42
*** ChuckC has quit IRC03:44
*** Nisha has joined #openstack-ironic03:44
*** jlvillal_sfo_ has joined #openstack-ironic03:49
jlvillal_sfo_Nisha: Sorry I forgot to tell devananda to look at your patch :(03:49
Nishajlvillal_sfo_, whats the time there?03:50
jlvillal_sfo_19:5003:50
Nishais devananda still around?03:50
Nishathat is blocking other patches actually03:50
*** jlvillal_sfo has quit IRC03:50
Nishajlvillal_sfo_, is devananda NobodyCam around there?03:51
*** jlvillal_sfo has joined #openstack-ironic03:52
jlvillal_sfoNisha: I am in my hotel room03:52
jlvillal_sfoNisha: We all left the sprint around 17:30 as they had to use the room for an event.03:52
Nishaohk03:52
jlvillal_sfoNisha: Sorry if dupe messages.  Fighting hotel WiFi :(03:53
Nishajlvillal_sfo, ok03:53
Nisha:)03:53
Nishaok one thing, recheck no bug initiates the jenkins again?03:53
Nishai want to rerun jenkins if gate issue is fixed, this should pass as there are no changes to code as of now gerrit wont allow me to repost the patch unless i change something03:54
Nishajlvillal_sfo, ^^^^03:54
*** jlvillal_sfo_ has quit IRC03:54
mrdaNisha, yes, "recheck no bug" retriggers jenkins03:54
jlvillal_sfoNisha: I think so03:55
Nishado we need to enclose in quotes?03:55
jlvillal_sfomrda: Aren't you supposed to be flying home right about now ;)03:55
mrdagate is fixed - about 4 hours ago03:55
jlvillal_sfoNisha: No quotes needed03:55
mrdaAll flights out of SFO delayed by over an hour03:55
jlvillal_sfoFrom what I have seen03:55
jlvillal_sfomrda: Ouch!03:55
jlvillal_sfoYou left a long time ago.  Sorry you have to wait so long :(03:56
mrdaoh well, I will get home eventually03:56
mrdaJust hope I don't miss VA8 as a result of this delay03:56
mrdaotherwise I miss Valentine's Day, and what I have planned for my wife03:57
jlvillal_sfomrda: Have a great trip home.  I hope you make your connections!03:57
mrdathanks!  Enjoy the meetup tomorrow03:57
jlvillal_sfoThank you.03:57
*** jlvillal_sfo has quit IRC04:03
*** Marga_ has quit IRC04:39
*** Marga_ has joined #openstack-ironic04:40
*** yog_ has joined #openstack-ironic04:42
*** pcrews has quit IRC04:45
openstackgerritMerged openstack/ironic: Add logical name support to Ironic  https://review.openstack.org/14173704:51
*** achanda has joined #openstack-ironic05:02
*** Haomeng has quit IRC05:03
*** pensu has joined #openstack-ironic05:03
*** davideagnello has joined #openstack-ironic05:04
*** Haomeng has joined #openstack-ironic05:04
openstackgerritRamakrishnan G proposed openstack/ironic: Add documentation for VirtualBox drivers  https://review.openstack.org/14630705:05
*** sdake_ has quit IRC05:08
*** davideagnello has quit IRC05:09
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Add support for mocking with VirtualBox  https://review.openstack.org/15564705:36
*** lazy_prif has joined #openstack-ironic05:41
*** lazy_prif is now known as lazy_prince05:41
*** killer_prince has quit IRC05:42
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Add support for mocking with VirtualBox  https://review.openstack.org/15564705:43
*** killer_prince has joined #openstack-ironic05:45
*** lazy_prince has quit IRC05:46
*** killer_prince is now known as lazy_prince05:46
*** Haomeng|2 has joined #openstack-ironic05:46
*** Haomeng has quit IRC05:47
*** ParsectiX has quit IRC05:48
*** sdake has joined #openstack-ironic05:48
*** sdake has quit IRC05:48
*** sdake has joined #openstack-ironic05:48
*** ParsectiX has joined #openstack-ironic05:49
*** ParsectiX has quit IRC05:50
*** Haomeng|2 has quit IRC05:50
*** stendulker has joined #openstack-ironic05:51
*** lazy_prince has quit IRC05:52
*** killer_prince has joined #openstack-ironic05:53
*** killer_prince is now known as lazy_prince05:53
*** ParsectiX has joined #openstack-ironic05:55
*** Haomeng|2 has joined #openstack-ironic05:56
*** achanda has quit IRC06:03
*** lazy_prince has quit IRC06:07
*** killer_prince has joined #openstack-ironic06:07
*** killer_prince is now known as lazy_prince06:07
*** ParsectiX has quit IRC06:09
*** stendulker_ has joined #openstack-ironic06:11
*** achanda has joined #openstack-ironic06:11
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/15565706:14
*** stendulker has quit IRC06:14
*** coolsvap_ is now known as coolsvap06:18
*** anderbubble has quit IRC06:21
*** spandhe has joined #openstack-ironic06:21
openstackgerritTan Lin proposed openstack/python-ironicclient: Enable ironicclient with --ironic-api-version 1.x  https://review.openstack.org/15562406:29
*** marios has joined #openstack-ironic06:40
*** marios has quit IRC06:41
*** marios has joined #openstack-ironic06:41
*** jiangfei has joined #openstack-ironic06:45
openstackgerritRamakrishnan G proposed openstack/ironic: Add driver interface for RAID configuration  https://review.openstack.org/15523006:50
openstackgerritMerged stackforge/proliantutils: Add cleaning operations  https://review.openstack.org/15433106:51
*** marios has quit IRC06:52
*** marios has joined #openstack-ironic06:52
*** dlpartain has joined #openstack-ironic06:55
*** dlpartain has left #openstack-ironic06:58
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Add hpssa module to proliantutils  https://review.openstack.org/15351907:02
*** yuanying_ has quit IRC07:03
*** yuanying has joined #openstack-ironic07:03
*** humble_ has joined #openstack-ironic07:04
*** stendulker has joined #openstack-ironic07:12
*** stendulker_ has quit IRC07:12
*** stendulker has quit IRC07:25
*** stendulker has joined #openstack-ironic07:26
*** Haomeng has joined #openstack-ironic07:29
*** Haomeng|2 has quit IRC07:30
*** andreykurilin_ has joined #openstack-ironic07:42
*** lazy_prince has quit IRC07:45
openstackgerritTan Lin proposed openstack/python-ironicclient: Enable ironicclient with --ironic-api-version 1.x  https://review.openstack.org/15562407:45
*** killer_prince has joined #openstack-ironic07:45
*** killer_prince is now known as lazy_prince07:45
*** andreykurilin_ has quit IRC07:50
*** andreykurilin_ has joined #openstack-ironic07:50
*** andreykurilin_ has quit IRC08:07
*** jcoufal has joined #openstack-ironic08:09
*** andreykurilin_ has joined #openstack-ironic08:11
*** spandhe has quit IRC08:13
*** ndipanov has joined #openstack-ironic08:14
*** ifarkas has joined #openstack-ironic08:17
*** oomichi has quit IRC08:21
*** achanda has quit IRC08:21
*** achanda has joined #openstack-ironic08:23
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Add hpssa module to proliantutils  https://review.openstack.org/15351908:23
*** achanda has quit IRC08:38
*** openstackgerrit has quit IRC08:42
*** openstackgerrit has joined #openstack-ironic08:42
*** achanda has joined #openstack-ironic08:44
*** erwan_taf has joined #openstack-ironic08:46
*** lazy_prince is now known as killer_prince08:55
*** yuanying has quit IRC09:01
*** killer_prince is now known as lazy_prince09:02
*** lucasagomes has joined #openstack-ironic09:03
*** bauzas is now known as bauwser09:04
*** dtantsur|afk is now known as dtantsur09:06
dtantsurMorning Ironic, TGIF finally :)09:06
Nishadtantsur, morning09:10
Nishadtantsur, need reviews again on inspetion patches09:11
Nisha:)09:11
dtantsursure, lemme wake up :)09:13
*** humble_ has quit IRC09:13
*** achanda has quit IRC09:14
*** gilliard is now known as gillllliard09:15
*** jistr has joined #openstack-ironic09:16
*** romcheg has joined #openstack-ironic09:18
*** dlpartain has joined #openstack-ironic09:18
dtantsurNisha, I'm leaving first patch for devananda to have a look first, as I'm not strong in our state machine...09:19
Nishaok09:19
Nishastill you can ad your opinion on same09:19
*** derekh has joined #openstack-ironic09:19
openstackgerritTan Lin proposed openstack/python-ironicclient: Enable ironicclient with --ironic-api-version 1.x  https://review.openstack.org/15562409:20
*** romcheg has quit IRC09:22
*** zhenzanz has quit IRC09:26
*** dlpartain has quit IRC09:26
*** romcheg has joined #openstack-ironic09:27
stendulkerdtantsur: Good Morning,09:28
stendulkerdtantsur: Can you please review spec for Management Interfaces to support UEFI Secure Boot https://review.openstack.org/#/c/135845/09:29
dtantsurhey hey folks, I see all your patches in my mail :) just need to find some time for them09:30
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Add hpssa module to proliantutils  https://review.openstack.org/15351909:30
stendulkerdtantsur: :)09:30
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Add hpssa module to proliantutils  https://review.openstack.org/15351909:36
*** chlong has quit IRC09:49
*** vdrok_afk is now known as vdrok09:52
*** pas-ha has joined #openstack-ironic09:58
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/15557110:01
openstackgerritMerged openstack/ironic: Stop console at first if console is enabled when destroy node  https://review.openstack.org/15475810:02
*** chenglch has quit IRC10:03
*** alexpilotti has joined #openstack-ironic10:14
*** romcheg1 has joined #openstack-ironic10:16
*** romcheg has quit IRC10:16
*** lynxman has quit IRC10:19
*** lynxman has joined #openstack-ironic10:20
*** romcheg has joined #openstack-ironic10:21
*** romcheg1 has quit IRC10:21
*** MattMan has joined #openstack-ironic10:31
*** Nisha has quit IRC10:37
*** stendulker has quit IRC11:02
*** pensu has left #openstack-ironic11:06
*** Haomeng|2 has joined #openstack-ironic11:06
*** ramineni has quit IRC11:07
*** Haomeng has quit IRC11:07
*** pas-ha has quit IRC11:27
*** pas-ha has joined #openstack-ironic11:29
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Functional test for setting IPMI credentials  https://review.openstack.org/14282311:30
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Require maintenance mode for setting IPMI credentials  https://review.openstack.org/15530711:30
openstackgerritDiego de Lima Pereira proposed openstack/python-ironicclient: Fix Python Ironic Client Documentation  https://review.openstack.org/15571411:34
*** pensu has joined #openstack-ironic11:42
*** foexle has joined #openstack-ironic11:45
foexleheyho guys, just a short question. I'm playing with ironic (Juno) and i can provisioning some hosts. But sometimes, the used boot image (will downloaded from tftpd) uses a wrong eth device11:47
foexleso udev will choose as eth0 the first device which was found11:48
foexleis there any possibilities to tell which MAC/device should be used as eth011:48
lucasagomesfoexle, you mean tell nova which mac (port) should be used for dpeloyment?11:49
lucasagomesthe thing, we pass to nova a list of ports configured in Ironic11:49
lucasagomesand as we have only 1 ip, nova will randomly choose one of the ports and create the network for it11:50
lucasagomesit's a known problem, but yeah, i think that in Ironic we should be able to tell which port we want to use for the deployment11:50
lucasagomesright now you can't :/11:50
foexlelucasagomes: yes ... for this case I've created only one port for the BM host11:50
lucasagomesyeah, that's how we workaround it right now11:50
foexleso the correct port whith the correct mac11:50
foexlePXE gets the IP11:51
foexlefrom dhcp correctly11:51
lucasagomesit's also something that need to be discussed, like why we register all ports in Ironic if only 1 is actually used and all11:51
foexlewill download theimage11:51
lucasagomesfoexle, right... so you think that having a field like primary=True in the Port resource11:51
lucasagomesis alright?11:51
* lucasagomes have been thinking what would be the best way to do it11:51
foexleyeah ... so i mean i have a couple of available BM hosts11:52
foexleand if I'm using ubuntu11:52
foexlewe notice ... ubuntu will grap randomly the eth0 device (first match)11:52
foexlebut this "eth0" device is not everytime the same as the used port (mac)11:53
foexleso sometimes is use in ironic mac xxxxxx and this device will on boot declared by ubuntu to eth211:54
foexlesry for my description11:54
foexleis a little bit complicated11:54
foexlebut you understand ?11:54
lucasagomeshmm this is after the machine is deployed?11:54
foexleyes11:54
*** mlsoule has joined #openstack-ironic11:54
lucasagomesoh right11:54
lucasagomesok... so the way we do it11:54
lucasagomesis using the configdrive to pass some information to the instance11:54
lucasagomesand we rename the ethernet devices accondingly11:54
lucasagomesthis is a pos deploy configuration11:55
foexleahhh11:55
lucasagomeswe have this patch here in Nova waiting for review /me lemme grab the link11:55
lucasagomeshttps://review.openstack.org/#/c/145302/11:55
*** Faizan has joined #openstack-ironic11:56
lucasagomesfoexle, ^ so what I want with that patch is to give operators a way to inject stuff on the config drive, and my reference example is exactly like that11:56
lucasagomesi want to pass info to the instance so it can rename my ethernet devices accordingly11:56
lucasagomesthat's the only bit that needs to get merged11:56
foexlelucasagomes: absolutely .... many thanks for that hint11:56
lucasagomesand ofc, you need some tool to configure ur instance11:57
lucasagomescloud-init for example, although I'm not sure if cloud-init rename eth devices at the moment11:57
lucasagomesfoexle, cool np11:57
foexleall right so in Juno i need luck to get the right device or changing dev by hand via IPMI11:57
lucasagomesin juno there's no configdrive :/11:58
lucasagomesyeah... unless somehow you can pass info to ur instance, like metadata service or something11:58
lucasagomesthat would work too11:58
lucasagomesbut I'm not exactly sure how you can do that11:58
foexlelucasagomes: nope that'snotpossible11:58
foexlebecause the device is not up :)11:58
lucasagomesoh true11:58
lucasagomesyeah, not sure... you could bake something in the image11:59
foexlebut anyway ... that helps11:59
lucasagomesthat would check for the network devices11:59
lucasagomessee if it's connect to a specific network and if so you rename it11:59
lucasagomeslike an extra service11:59
lucasagomesbut, yeah... it's not straight forward :/11:59
foexle:)11:59
foexleok so i think change images would be the best way atm12:00
foexlelucasagomes: many thanks for your help12:00
lucasagomesfoexle, you very welcome12:00
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add iscsi extension  https://review.openstack.org/15572712:10
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add AgentVendorPasshtru for the PXE driver  https://review.openstack.org/15572812:13
*** lsmola has joined #openstack-ironic12:16
*** pensu has quit IRC12:22
*** lucasagomes is now known as lucas-hungry12:23
*** athomas has joined #openstack-ironic12:25
*** smoriya has quit IRC12:28
*** Faizan has quit IRC12:33
openstackgerritShivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node  https://review.openstack.org/15573112:35
*** athomas has quit IRC12:42
*** takadayuiko has quit IRC13:09
*** dprince has joined #openstack-ironic13:11
*** lazy_prince is now known as killer_prince13:19
*** EmilienM|afk is now known as EmilienM13:19
openstackgerritMerged openstack/ironic: Add support for local boot  https://review.openstack.org/14618913:29
*** jcoufal_ has joined #openstack-ironic13:30
dtantsurlucas-hungry, ^^^ :)13:32
*** lucas-hungry is now known as lucasagomes13:32
lucasagomesdtantsur, yushhhhhhh o/13:32
*** jcoufal has quit IRC13:33
jrollmorning y'all :)13:34
jrollnice merge13:34
lucasagomesjroll, morning!13:34
lucasagomeswhat was the gate problem in the end?13:35
jrollyou saw Shrews' revert of making the ironic user non-admin in devstack?13:36
jrollneeded the same for the swift user13:36
lucasagomesyeah, I saw that13:36
lucasagomesohh13:36
jrollhttps://review.openstack.org/#/c/155357/13:36
lucasagomesdevstack then13:36
jrollyeah13:36
lucasagomesgotcha13:37
jrollalso, using Depends-On: is awesome13:37
lucasagomeswhat is that?13:37
* lucasagomes feels outdated13:38
jrolljim blair added it13:38
jrollyou put a change ID there and can have cross project dependencies13:38
jrollin gerrit13:38
jrollso a nova change can depend on an ironic change or vice versa13:38
*** jjohnson2 has joined #openstack-ironic13:39
lucasagomesoh holy cow13:40
lucasagomesyeah, that was very much needed indeed13:40
jrollyep, tis awesome13:41
jrollso I made an IPA patch depend on my devstack change to make sure it worked this time :P13:41
lucasagomesnice one! yeah I will make the pxe.AgentVendorPasstrhu to depend on the iscsi extension in IPA to test it out :D13:44
jrollnice :)13:46
dtantsurjroll, morning13:48
jroll\o dtantsur :)13:49
*** Nisha has joined #openstack-ironic14:06
Nishadtantsur, there?14:06
Nishalucasagomes, hi14:09
lucasagomesNisha, hi there14:11
Nishalucasagomes, hi14:11
Nishado you know if dtantsur is there or not?14:11
lucasagomesNisha, hmm no idea. He might be afk14:11
Nishalucasagomes, request you to review inspection patches14:12
lucasagomes(we live in diff countries, so I don't see him very often)14:12
lucasagomesNisha, ack, yeah I have to catch up with the reviews14:12
Nishalucasagomes, oh i thought u guys are in same company and country14:16
Nisha:)14:16
lucasagomesNisha, same company :)14:18
openstackgerritMerged openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/15565714:18
*** david-lyle_afk is now known as david-lyle14:22
Nishalucasagomes, :)14:28
jrollNisha: you need all bp/ironic-node-properties-discovery patches reviewed, yes?14:30
Nishayes14:30
jrollok14:30
Nishajroll, even this one https://review.openstack.org/15159614:33
jrollNisha: one thing at a time :)14:33
Nishawhich is ilo Node properties discovery14:33
*** annegentle has quit IRC14:33
*** yog_ has quit IRC14:35
dtantsurNisha, o/14:43
dtantsursorry was on lunch14:43
*** priteau has joined #openstack-ironic14:44
jrolldtantsur: I'm not sure what your comment means here https://review.openstack.org/#/c/149817/18/ironic/drivers/base.py14:45
jrollafaict that hasn't been defined yet?14:45
dtantsurjroll, see the next patch actually14:45
jrollmmm14:45
dtantsurpreviously it used this return value to distinguish whether inspection was already finished14:45
dtantsur(for in-band it won't be the case)14:46
jrollright, I see14:47
jrolljust like deploy14:47
dtantsuryep14:47
jrollwell, currently it doesn't use that return value14:47
jrollbut I see what you mean14:47
*** Nisha has quit IRC14:48
dtantsurjroll, it was changed recently to always assume inspection has finished. which I obviously object :)14:48
*** Nisha has joined #openstack-ironic14:48
jrollyeah :)14:48
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Remove rootfstype=ramfs from PXE templates  https://review.openstack.org/15576514:53
*** dlpartain has joined #openstack-ironic14:55
*** zz_jgrimm is now known as jgrimm14:56
*** jcoufal_ has quit IRC14:57
*** r-daneel has joined #openstack-ironic15:00
lucasagomesjroll, how I will call the new driver with pxe and agent O.O15:02
lucasagomespxe_agent_ssh!?15:02
* lucasagomes is bad with names15:02
jrolllol15:02
jrolliscsi_ssh?15:02
Nishajroll,15:03
jrolland then we can get rid of pxe_ssh eventually15:03
lucasagomesjroll, that would be better, but kinda break the pattern where they have pxe_ etc15:03
jrollbecause now pxe driver can local boot :P15:03
lucasagomescool yeah I will add iscsi_ssh15:03
lucasagomesyeah it's all complicated15:03
Nishadevananda, did not agree to remove INSPECTED or rather i would say *ED states15:03
jrolland agent pxe boots the ramdisk so ya15:03
Nishain weekly meeting15:03
lucasagomeslol damn... true iscsi_ssh is it15:03
lucasagomes<deploy>_<power> methods15:03
jrollNisha: ok, I'll talk with people today, I thought the plan was to get rid of *ED unless needed15:04
jrolllucasagomes: +115:04
Nishajroll, he said to add them as no-op state15:04
Nishano-op means we may not do any action in that state15:04
jrollNisha: great, can you please put this info on the review in gerrit?15:05
Nishawhile i need the INSPECTED state as i populate last_inspected field while moving from INSPECTING to INSPECTED15:05
jrollNisha: great, can you please put this info on the review in gerrit?15:06
* jroll is about to head to the meetup15:06
Nishain states code patch?15:06
jrollyes, where I left the comment15:06
NishaOk15:06
jrollso that 1) everyone can see it, and 2) I don't forget15:06
Nishai will add and post a new patch15:06
Nishais it going to be discussed today?15:06
jrollI don't know15:07
jrolldon't post a new patch if my comments are invalid15:07
Nishaok.15:07
Nishai will add comments per transition15:07
jrollthanks15:08
Nishawhatever i could think of :)15:08
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add AgentVendorPasshtru for the PXE driver  https://review.openstack.org/15572815:09
*** dlpartain has left #openstack-ironic15:09
lucasagomes2+2 on https://review.openstack.org/#/c/155316 someone wants to push the button?15:10
*** MattMan has quit IRC15:10
jrolllucasagomes: boom15:15
lucasagomes\o/15:15
*** jjohnson2 has quit IRC15:16
openstackgerritMerged openstack/ironic-specs: Support per Driver Sensor Meters  https://review.openstack.org/13007415:16
openstackgerritShivanand Tendulker proposed openstack/ironic: Update Ilo drivers to use REST API interface to iLO  https://review.openstack.org/15440315:18
openstackgerritShivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node  https://review.openstack.org/15573115:18
*** MattMan has joined #openstack-ironic15:25
NobodyCamgood mornig Ironic15:27
dtantsurNobodyCam, morning. TGIF today!15:27
lucasagomesNobodyCam, morning15:28
*** anderbubble has joined #openstack-ironic15:28
NobodyCammornig dtantsur & lucasagomes ... and OH ya TGIF!!!!15:29
openstackgerritMark Goddard proposed openstack/ironic: Avoid deregistering conductor following SIGUSR1.  https://review.openstack.org/15578515:31
*** datajerk has quit IRC15:38
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Functional test for setting IPMI credentials  https://review.openstack.org/14282315:39
*** datajerk has joined #openstack-ironic15:39
dtantsurifarkas, hi, could you please review https://review.openstack.org/#/c/155307/ ?15:39
*** coolsvap is now known as coolsvap_15:43
*** vdrok is now known as vdrok_afk15:51
*** Nisha has quit IRC15:53
*** pcrews has joined #openstack-ironic15:59
openstackgerritShivanand Tendulker proposed openstack/ironic: Common changes for secure boot support  https://review.openstack.org/15397416:03
ifarkasdtantsur, sure, I will take look16:03
lucasagomesjroll, https://github.com/openstack/nova/blob/master/nova/virt/ironic/patcher.py#L36 :(16:04
lucasagomesiscsi_ssh16:04
lucasagomes>.<16:04
jrolllucasagomes: rage16:05
lucasagomeslol16:05
lucasagomessame here dude16:05
lucasagomesI was like WTH!16:05
* lucasagomes probably coded that long time ago :/16:05
jrolllucasagomes: that's just for backwards compat stuff at this point, might be fine to go ahead without changing that16:06
*** ndipanov is now known as ndipanoff16:07
lucasagomesjroll, yeah, I will look at what that nova is doing and try to make it smarter16:09
lucasagomessigh16:09
jrolllucasagomes: I don't remember the deprecation plan for flavor/ramdisk pairing but we may be able to deprecate after kilo16:09
lucasagomesjroll, yeah, well we still need to somewhere say what is the kernel/ramdisk for that image16:09
BadCubMorning Ironic!16:09
lucasagomeswe have to think about where it should live16:10
lucasagomesnow it's glance metadata16:10
*** jgrimm_ has joined #openstack-ironic16:10
lucasagomesjroll, we already deprecated the deploy ramdisk and kernel being in the flavor16:10
lucasagomesbut that's something else16:10
jrolllucasagomes: the operator specifies it in driver_info16:10
lucasagomesthat's the ramdisk/kernel for the user image16:10
jrollthe PXEDriverFields pulls it from driver_info16:11
lucasagomesjroll, we only know which image to use when smoeone do a nova boot :/16:11
*** maurosr has quit IRC16:11
lucasagomesthe deploy ramdisk/kernel is fine to the operator specify16:11
jrolleh, no, PXEDriverFields only exists to get deploy ramdisk stuff from flavor16:11
jrollironic gets the instance kernel/ramdisk from glance16:11
jrollnova doesn't put it there16:11
lucasagomesohhh16:11
jrollhttps://github.com/openstack/nova/blob/master/nova/virt/ironic/patcher.py#L134-14116:12
lucasagomesdamn ok I was confusing it then16:12
*** mjturek has joined #openstack-ironic16:12
lucasagomesgod damn, so devstack should be updated16:12
jrollthat's the only special part, the pxe ramdisk works fine with generic patcher16:12
jrollwait, devstack isn't updated??16:12
lucasagomesjroll, gotcha... hmm not sure16:12
lucasagomestoo many things in my mind now heh16:12
lucasagomesI will check it out16:12
lucasagomesI will add 'pxe' to the name for test sake for now16:12
lucasagomes(won't be part of the patch)16:12
jrollno, devstack is up to date16:13
jrollhttps://github.com/openstack-dev/devstack/blob/master/lib/ironic#L59516:13
*** jgrimm- has joined #openstack-ironic16:13
*** jgrimm has quit IRC16:13
*** mjturek1 has quit IRC16:13
lucasagomeshmm16:13
*** mjturek1 has joined #openstack-ironic16:14
jrollwhat kind of trouble is it giving you?16:14
*** maurosr has joined #openstack-ironic16:14
*** pas-ha has quit IRC16:14
*** Nisha has joined #openstack-ironic16:14
lucasagomesjroll, checking16:16
openstackgerritMerged openstack/ironic: AgentVendorInterface: Move to a common place  https://review.openstack.org/15531616:16
openstackgerritMerged openstack/ironic: Remove deploy_is_done() from AgentClient  https://review.openstack.org/15532516:16
*** mjturek has quit IRC16:16
*** jgrimm_ has quit IRC16:17
jrolllucasagomes: I'm going to head in to the meetup, I'll be back in like an hour16:17
lucasagomesjroll, no worries, enjoy it !16:17
jrollthere's no reason it shouldn't just work though, maybe you just need updated devstack?16:17
jrollenjoy my commute? yeah right :P16:17
jrollsee you soon16:18
lucasagomesjroll, yeah not sure yet what's going on16:18
lucasagomesmaybe some stupid mistake I did16:18
lucasagomessee ya16:18
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver  https://review.openstack.org/15480816:21
*** kbs1 has quit IRC16:22
*** kbs has joined #openstack-ironic16:23
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver  https://review.openstack.org/15481416:26
*** mgoddard1 has joined #openstack-ironic16:31
*** mgoddard has quit IRC16:33
*** mgoddard1 has quit IRC16:35
*** Marga_ has quit IRC16:38
*** anderbubble has quit IRC16:38
*** jcoufal has joined #openstack-ironic16:46
*** marios has quit IRC16:47
*** marios has joined #openstack-ironic16:47
*** naohirot has quit IRC16:50
*** jcoufal has quit IRC16:51
openstackgerritMerged stackforge/ironic-discoverd: Require maintenance mode for setting IPMI credentials  https://review.openstack.org/15530716:51
*** jerryz has quit IRC16:52
*** EmilienM is now known as EmilienM|afk16:56
openstackgerritShivanand Tendulker proposed openstack/ironic: Update Ilo drivers to use REST API interface to iLO  https://review.openstack.org/15440316:58
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for agent_ilo driver  https://review.openstack.org/15481616:58
openstackgerritShivanand Tendulker proposed openstack/ironic: Common changes for secure boot support  https://review.openstack.org/15397416:58
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver  https://review.openstack.org/15481416:58
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver  https://review.openstack.org/15480816:58
*** derekh has quit IRC16:59
*** hemna_ has joined #openstack-ironic17:03
*** MattMan has left #openstack-ironic17:09
lucasagomesjroll, ok it was something I did when tweaking stuff, I forgot to update the ipa-driver-name on the pxe config file17:09
lucasagomes>.<17:09
* lucasagomes now it's being generated so, I don't have to manually chage things17:09
* lucasagomes facepalm17:09
lucasagomesI will commit what I have now that it's working and take a break, but still need to add unittests :D17:10
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add AgentVendorPasshtru for the PXE driver  https://review.openstack.org/15572817:11
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Remove rootfstype=ramfs from PXE templates  https://review.openstack.org/15576517:11
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add AgentVendorPasshtru for the PXE driver  https://review.openstack.org/15572817:13
*** alexpilotti has quit IRC17:13
*** alexpilotti has joined #openstack-ironic17:15
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add AgentVendorPasshtru for the PXE driver  https://review.openstack.org/15572817:15
*** alexpilotti has quit IRC17:16
*** Marga_ has joined #openstack-ironic17:19
lucasagomesaight good night everyone, I will step out a bit17:21
*** martini has joined #openstack-ironic17:21
*** lucasagomes is now known as lucas-dinner17:21
dtantsurhave a nice weekend!17:23
*** dtantsur is now known as dtantsur|afk17:23
lucas-dinnerdtantsur|afk, cheers you too17:25
lucas-dinnerenjoy it17:25
*** ekarlso has quit IRC17:25
*** ekarlso has joined #openstack-ironic17:25
*** alex_xu_ has joined #openstack-ironic17:30
*** xu_alex has quit IRC17:32
*** yjiang5 is now known as yginger517:36
*** anderbubble has joined #openstack-ironic17:36
*** jistr has quit IRC17:41
*** lsmola has quit IRC17:42
*** jlvllal_sfo has joined #openstack-ironic17:47
*** harlowja_away is now known as harlowja_17:48
*** mgoddard has joined #openstack-ironic17:51
*** davideagnello has joined #openstack-ironic17:52
*** jogo is now known as flashgordon17:54
jrolllucas-dinner: lol, have a good night17:56
*** absubram has joined #openstack-ironic17:58
* NobodyCam is back17:58
*** tchaypo is now known as tchorizo17:59
*** spandhe has joined #openstack-ironic18:00
jxiaobinhi all, is there anyone using vm/bm hybrid openstack environment? how do you explicitly ask for a bm/vm?18:02
priteauHi. We're running Ironic from RDO Juno. We got it to work, but now we're getting nodes stuck in locked state. What can we do about this?18:03
*** korekhov has joined #openstack-ironic18:03
priteauIs there some way to cancel the operation that is locking the node?18:03
jrolljxiaobin: we run it in cells, where bm/vm cells are separate18:04
jrollpriteau: update nodes set reservation=null where uuid = '$uuid';18:04
jrollpriteau: it's bad, I know :(18:04
priteauthanks!18:06
jxiaobinmy experience is that there is no way to relate a call to the type of compute, say bm/vm18:09
jrolljxiaobin: there's something called host aggregates, I don't know anything about them18:10
jrollwe define our cells to only accept baremetal or VM flavors18:10
jxiaobinin a mixed environment, scheduler considers the hypervisor nodes and ironic nodes are the same18:10
jxiaobinok18:11
jxiaobinwill ironic/nova provide a proper solution?18:11
jrollI think host aggregates does solve this, I don't know how they work though18:12
jxiaobinsay a flavor or image is particularily for BM18:12
jrollmorgabra: https://github.com/git/git/blob/master/contrib/completion/git-completion.bash18:13
jxiaobinthanks jroll!18:13
jrollyou're welcome :)18:13
openstackgerritJim Rollenhagen proposed openstack/ironic: Add network provider interface and implementations  https://review.openstack.org/13968718:16
openstackgerritJim Rollenhagen proposed openstack/ironic: Add driver and driver_info fields to Port  https://review.openstack.org/15584818:16
jrolldevananda: ^18:16
anderbubblejroll, can you quickly help me understand why cells are necessary in a mixed vm/bm environment? Why can’t you just have some novas that are configured with ironic, and some novas that are configured with libvirt?18:16
jrollclearly not ready yet but yeah18:16
jrollanderbubble: they aren't necessary, as far as I know, but rackspace already runs cells for VMs, so we made an ironic cell18:17
jrollanderbubble: by default nova will just pick a compute that has resources, you need to use host aggregates or cells to restrict this18:17
anderbubblejroll, ok; I’d just heard people claim that you *have* to do it that way. I haven’t tried mixing, but it seemed like it should be fine.18:17
*** Guest40096 has joined #openstack-ironic18:21
*** Guest40096 is now known as annegentle18:22
*** dtantsur|afk has quit IRC18:24
*** achanda has joined #openstack-ironic18:25
jlvllal_sfoJoshNang:  ironic]$ tools/states_to_dot.py18:25
JoshNangjlvllal_sfo: thanks18:25
*** dtantsur has joined #openstack-ironic18:26
*** tchorizo has left #openstack-ironic18:30
*** EmilienM|afk is now known as EmilienM18:32
*** andreykurilin_ has quit IRC18:32
*** foexle has quit IRC18:32
*** Marga_ has quit IRC18:32
*** Marga_ has joined #openstack-ironic18:33
*** Marga_ has quit IRC18:34
*** eghobo has joined #openstack-ironic18:35
*** Marga_ has joined #openstack-ironic18:35
*** eghobo has quit IRC18:35
*** eghobo has joined #openstack-ironic18:39
openstackgerritMark Goddard proposed openstack/ironic: Avoid deregistering conductor following SIGUSR1  https://review.openstack.org/15578518:40
jrollanderbubble: I have no idea :|18:42
*** dprince has quit IRC18:45
BadCubMorning Ironc18:46
openstackgerritJohn Villalovos proposed openstack/ironic: Add concept of stable states to the state machine  https://review.openstack.org/15552918:47
jlvllal_sfoBadCub: Morning18:48
*** anderbubble has quit IRC18:51
JoshNangjlvllal_sfo: this block of tests was pretty well commented https://github.com/openstack/ironic/blob/master/ironic/tests/conductor/test_manager.py#L129818:56
jlvllal_sfoJoshNang: Thanks!18:56
jlvllal_sfoDo we have anything like this for our REST APIs?  http://www.jfrog.com/confluence/display/RTF/Artifactory+REST+API18:56
*** barra204 has joined #openstack-ironic18:57
*** barra204 is now known as shamamunyi18:58
*** shamamunyi is now known as shakamunyi18:58
jlvllal_sfoAnd would something like that be useful for Ironic?18:58
openstackgerritJohn Villalovos proposed openstack/ironic: Add concept of stable states to the state machine  https://review.openstack.org/15552918:59
*** lucas-dinner has quit IRC19:02
*** dprince has joined #openstack-ironic19:02
JayFmartini: devananda: RFR https://github.com/rackerlabs/coreos-image-builder19:04
*** korekhov has quit IRC19:04
JayFQuestion though: should I improve/refactor it before it gets slurped in, or just have the history slurped in as-is and do all the refactoring in gerritt?19:04
*** korekhov has joined #openstack-ironic19:05
jrollI would import it as-is so people can review the refactor :)19:05
JayFThat's pretty much what I thought19:06
devanandaJayF: first glance: lgtm19:06
JayFAight; I'll propose to project-config19:06
JayFShould that come with an email to the list as well?19:07
jrollyesh19:07
*** bradjones has quit IRC19:07
*** eghobo has quit IRC19:08
*** korekhov has quit IRC19:09
*** korekhov has joined #openstack-ironic19:12
JayFhttps://review.openstack.org/#/c/155868/ project-config setup for CIB19:16
*** Nisha has quit IRC19:19
*** Nisha has joined #openstack-ironic19:20
*** davideagnello has quit IRC19:21
*** davideagnello has joined #openstack-ironic19:24
*** davideagnello has quit IRC19:31
*** davideagnello has joined #openstack-ironic19:32
*** ndipanoff has quit IRC19:32
openstackgerritJim Rollenhagen proposed openstack/ironic: Add driver and driver_info fields to Port  https://review.openstack.org/15584819:33
openstackgerritJim Rollenhagen proposed openstack/ironic: Add network provider interface and implementations  https://review.openstack.org/13968719:33
*** korekhov has quit IRC19:34
*** Haomeng has joined #openstack-ironic19:34
*** korekhov has joined #openstack-ironic19:35
*** Haomeng|2 has quit IRC19:35
*** korekhov has quit IRC19:39
*** Marga_ has quit IRC19:44
*** Marga_ has joined #openstack-ironic19:45
*** Nisha has quit IRC19:48
*** Nisha has joined #openstack-ironic19:48
*** romcheg has quit IRC20:00
*** korekhov has joined #openstack-ironic20:01
*** barra204 has joined #openstack-ironic20:01
*** shakamunyi has quit IRC20:02
jlvllal_sfoJoshNang: Can you think of other things that could do to fsm.py to make life easier for someone adding states?  Help catch errors that people could make.20:04
*** hemna_ has quit IRC20:04
*** Raj_ has joined #openstack-ironic20:05
JoshNangjlvllal_sfo: not off the top of my head, though i have to make changes to drop deleted/cleaned/etc after lunch, so i may have more ideas soon20:06
jlvllal_sfoJoshNang: Cool.  Thanks.20:06
*** dprince has quit IRC20:07
*** anderbubble has joined #openstack-ironic20:07
*** hemna_ has joined #openstack-ironic20:10
*** ndipanoff has joined #openstack-ironic20:19
*** barra204 has quit IRC20:19
*** romcheg has joined #openstack-ironic20:20
*** faizan has joined #openstack-ironic20:22
*** mgoddard has quit IRC20:22
*** jcoufal has joined #openstack-ironic20:22
*** mgoddard has joined #openstack-ironic20:24
*** achanda has quit IRC20:31
*** ifarkas has quit IRC20:31
*** lnxnut has joined #openstack-ironic20:33
*** barra204 has joined #openstack-ironic20:34
*** romcheg1 has joined #openstack-ironic20:34
*** barra204 has quit IRC20:35
*** romcheg has quit IRC20:35
*** bnemec is now known as beekneemech20:37
*** mgoddard has quit IRC20:38
*** mgoddard has joined #openstack-ironic20:39
*** yginger5 is now known as yginger5_away20:39
*** lnxnut has quit IRC20:41
*** lnxnut has joined #openstack-ironic20:41
*** lnxnut_ has joined #openstack-ironic20:43
*** achanda has joined #openstack-ironic20:43
*** mgoddard has quit IRC20:44
*** lnxnut has quit IRC20:46
faizandevananda: Hi20:51
faizanjroll: dtantsur: Hi20:53
*** martini has quit IRC20:58
*** jjohnson2 has joined #openstack-ironic21:00
jrollfaizan: hi21:03
faizanjroll: regarding "partition image support for agent driver", I have submitted the disk_partition code into oslo-incubator project21:17
faizanjroll: We need to give maintainers name for this code in oslo-incubator. Shall I give your name as maintainer?21:20
JayFJosh Gachnang or Alex Weeks might be a better choice21:24
JayFbut they're off to lunch right now and jroll is AFK (we're all at the mid-cycle meetup in SF)21:24
JayFso let me check with them and I'll get back21:24
faizanJayF: Thanks so much,21:25
JayFoh disk_partition21:25
JayFsorry I thought that was metrics21:25
JayFnot jim or josh, lol21:25
JayFThat should be owned generally by the Ironic team21:25
JayFif possible21:25
JayFif not, then I'd suggest devananda or someone he explicitly authorizes21:26
faizanJayF: ok, can you check with devananda and let me know?21:27
jrollfaizan: is it already in oslo-incubator21:27
JayFjogo is here and suggesting that this should go directly into a new oslo project instead of through the incubator21:27
jroll?21:27
JayFfaizan: not really, he's gone for the mid-cycle anyway :)21:27
faizanJayF: sorry I thought you were also there at mid-cycle21:28
jrollfaizan: deva left for the day21:28
*** lnxnut_ has quit IRC21:31
faizanJayF: In the spec I have mentioned that we will put it in oslo-incubator first, and then we can graduate it to oslo project21:31
*** mgoddard has joined #openstack-ironic21:33
jrollfaizan: people are discouraging new things in oslo-incubator, I think we should put it in a "real" oslo project, or even something like openstack/ironic-disk-utils21:34
*** mgoddard has quit IRC21:34
*** hemna_ has quit IRC21:35
faizanjroll: I think it will take a longer cycle to put it in oslo project and may not happen with in kilo release time21:37
*** hemna_ has joined #openstack-ironic21:38
faizanjroll: cant we pursue with both oslo-incubator and then to oslo project at the same time, so that we dont miss this spec for kilo release21:39
jrollfaizan: let's just make an ironic-disk-utils repo then21:39
jrollno need to wait for people21:40
*** david-lyle has quit IRC21:42
faizanjroll: ok let me try creating a repo for ironic-disk-utils, shall I give the name as libironic?21:42
*** david-lyle has joined #openstack-ironic21:42
openstackgerritNisha Agarwal proposed openstack/ironic: Automate boot iso creation with in ironic for iscsi-ilo  https://review.openstack.org/15590021:43
*** lnxnut has joined #openstack-ironic21:44
faizanjroll: I have submitted the patch in oslo-incubator, It is not reviewed yet. https://review.openstack.org/#/c/155190/21:46
*** david-lyle has quit IRC21:47
*** absubram has quit IRC21:52
jrollfaizan: I like libironic or ironiclib21:57
*** faizan has quit IRC21:57
jrollprobably ironiclib21:57
*** martini has joined #openstack-ironic21:57
Shrewsironiclib seems sensible21:57
jrolloh21:58
jroll21:57:22             <-- | faizan (6a33893c@gateway/web/freenode/ip.106.51.137.60) has quit (Ping timeout: 246 seconds)21:58
*** lnxnut has quit IRC21:59
*** lnxnut has joined #openstack-ironic22:00
openstackgerritNisha Agarwal proposed openstack/ironic: Add states required for node-inspection  https://review.openstack.org/14785722:03
*** lnxnut has quit IRC22:04
openstackgerritNisha Agarwal proposed openstack/ironic: Add InspectInterface for node-introspection  https://review.openstack.org/14981722:06
*** jjohnson2 has quit IRC22:06
martiniJayF: https://github.com/PeterMartini/ironic-dib-elements22:07
JayFI mean, all the history is gone22:11
JayFall the commits are from you?22:11
JayFor not...22:11
JayFhmm. This authored by vs committed in github is confusing22:11
* JayF checks the repo he made22:11
JayFMine: https://github.com/rackerlabs/coreos-image-builder/commits/master doesn't show the commited by / authored by split22:11
JayFso you did something differently that changed the history, I think?22:12
jrollhttp://i.imgur.com/Q3U4uIa.gif22:16
openstackgerritNisha Agarwal proposed openstack/ironic: Generic changes for Node Inspection  https://review.openstack.org/14982322:16
martiniYeah, doing apply patch for lots of different cherry picked pieces22:17
JayFyou might wanna ask folks in #openstack-infra what they've done in the past22:17
JayFwhen breaking pieces out of multiple repos22:17
martinihuh.  Looks like it's not actually putting the deploy-ironic pieces in there (though it doesn't show any errors or complaing about the target)22:20
martiniironic-agent worked tho22:20
*** achanda has quit IRC22:21
*** achanda has joined #openstack-ironic22:23
* mrda is still at the airport waiting to board the flight back to Australia22:23
jrollO.O22:23
jrollhi mrda22:23
mrdatldr: SFO->LAX flight delayed by 2 hours, so missed the long-haul.  Hotel overnight, rebooked on same flight tonight at 11pm.  So I've lost a day in airports.22:24
mrdaBut22:24
*** harlowja_ is now known as harlowja_away22:24
mrdaLogical names merged, so \o/22:24
JayFmrda: so Rachael told me just now, apparently mtrenish is in the office today22:24
JayFand you missed him :(22:24
mrdamattoliverau actually22:25
JayFaha22:25
mrdawrong matt :)22:25
mrdaYou should go say hello, he's at the hotel desks22:25
JayFI was told a Matt from -AU22:25
mrda(I think)22:25
JayFaight, I shall :)22:25
openstackgerritNisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection  https://review.openstack.org/15159622:25
mrdaI could have joined you guys for another day!22:25
martininevermind, I do get the same results from doing it in just dib, so I clearly just messed up the command22:25
*** Raj_ has quit IRC22:26
mrdaJayF: also, we had breakfasts and dinenrs together this week, so no big deal22:26
mrdajroll, JayF, JoshNang, clif_h, aweeks: Thanks for hosting us this week22:27
JayFYeah :( sad you're stuck at the airport22:27
jrollmrda: fo sho :)22:27
JoshNangmrda: thanks for coming!22:27
mrdaGlad to spend some time with y'all22:27
JayFGeneral apologies to everyone for the internet being of the LOL variety at times though :)22:27
*** harlowja_away is now known as harlowja_22:28
*** jcoufal has quit IRC22:29
mrdaJayF: that's ok, these things happen22:29
*** lnxnut has joined #openstack-ironic22:30
*** lnxnut has quit IRC22:31
Shrewsmrda: OMG22:32
mrdaShrews: I'm never travelling again :)22:32
Shrewsmrda: then obviously we all have to come to you next time22:32
NobodyCammrda: :(22:32
mrdaShrews: \o/22:32
* mrda should be over my disgruntledness by Vancouver22:33
NobodyCam:)22:33
* mrda just hopes his checked bag makes it to Australia22:33
openstackgerritNisha Agarwal proposed openstack/python-ironicclient: enhanced node-set-provision-state  https://review.openstack.org/14880422:36
mrdabye y'all, off to line up at checkin22:40
*** mrda is now known as mrda-travelling22:41
*** Nisha has quit IRC22:43
* jlvllal_sfo prepares to go to the airport22:43
*** jlvllal_sfo has quit IRC22:44
openstackgerritMerged openstack/python-ironicclient: Fix Python Ironic Client Documentation  https://review.openstack.org/15571422:47
*** priteau has quit IRC22:50
*** martini has quit IRC22:51
*** jjohnson2 has joined #openstack-ironic22:52
*** yginger5_away is now known as yjiang523:01
clif_hmrda-travelling: glad to meet you, have a safe trip23:02
*** r-daneel has quit IRC23:06
*** Haomeng has quit IRC23:08
*** zigo has quit IRC23:27
*** zigo has joined #openstack-ironic23:28
*** jjohnson2 has quit IRC23:31
*** anderbubble has quit IRC23:37
*** hemna_ is now known as hemnafk23:42

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