jogo | an ironic image: http://www.angmohdan.com/wp-content/gallery/ironic/ironic-pics-14.jpg | 00:05 |
---|---|---|
*** jrist has joined #openstack-ironic | 00:09 | |
*** krtaylor has joined #openstack-ironic | 00:10 | |
*** naohirot has joined #openstack-ironic | 00:12 | |
*** romcheg has quit IRC | 00:22 | |
naohirot | good morning ironic | 00:22 |
*** lnxnut has joined #openstack-ironic | 00:25 | |
*** davideagnello has quit IRC | 00:27 | |
Shrews | mrda: approved!! | 00:27 |
jroll | py27 recreate: /data/dev/nova/.tox/py27 | 00:28 |
* jroll cries | 00:28 | |
*** anderbubble has joined #openstack-ironic | 00:29 | |
*** Nisha has joined #openstack-ironic | 00:30 | |
Nisha | devananda, around? | 00:31 |
*** davideagnello has joined #openstack-ironic | 00:32 | |
*** lnxnut has quit IRC | 00:33 | |
Nisha | NobodyCam, there? | 00:33 |
*** davideagnello has quit IRC | 00:33 | |
*** davideagnello has joined #openstack-ironic | 00:33 | |
*** david-lyle is now known as david-lyle_afk | 00:37 | |
jlvllal_sfo | Nisha: Discussion going on at the moment | 00:37 |
jlvllal_sfo | At sprint | 00:37 |
Nisha | ohk | 00:37 |
jlvllal_sfo | Nisha: Maybe you can look at my patch :) https://review.openstack.org/155529 | 00:38 |
jlvllal_sfo | Since you have been doing states.py changes. | 00:38 |
*** martini has quit IRC | 00:41 | |
Nisha | jlvllal_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_sfo | Nisha: Thanks! | 00:45 |
Nisha | I dont know how much changes will it bring to my work though :( | 00:46 |
Nisha | if 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 inspectio | 00:47 |
jlvllal_sfo | Nisha: I think very minimal. maybe 1-2 lines | 00:47 |
Nisha | jlvllal_sfo, it would be great if you could also review it | 00:47 |
Nisha | Is gate issue fixed now? | 00:47 |
jlvllal_sfo | Nisha: I'll try. And yes they are from what I have been told | 00:48 |
Nisha | yesterday jenkins was failing for every patch | 00:48 |
Shrews | gate should be fixed | 00:48 |
Nisha | Shrews, thats great if it is fixed. I will repost the patches then later today | 00:49 |
rloo | Shrews: I added a print statement in one of the tests, and looked in .testrepository/* to see if it was printed. | 00:56 |
Shrews | rloo: as did i, but it was not printed :( | 00:56 |
rloo | Shrews: that's odd. | 00:57 |
Shrews | rloo: how did you run the test? | 00:57 |
rloo | Shrews: it is the same mechanism used in ironic. | 00:57 |
rloo | Shrews: I use tox | 00:57 |
rloo | tox -epy27 | 00:57 |
*** Nisha has quit IRC | 00:58 | |
Shrews | yeah, i tried that too. | 00:58 |
rloo | Shrews: do you know if it works for you in ironic? | 00:58 |
Shrews | rloo: i don't know. didn't try there | 00:58 |
rloo | Shrews: 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 |
Shrews | i 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 tomorrow | 00:59 |
rloo | Shrews: hmm. Shouldn't be that difficult. I'll build a new env and test it out again. | 00:59 |
*** takadayuiko has joined #openstack-ironic | 01:02 | |
naohirot | rloo: good morning, I just replied in terms of driver_info['irmc_sensor_method']. | 01:02 |
rloo | Shrews: unless you put the print in a test that doesn't inherit from utils.BaseTestClass | 01:02 |
rloo | hi naohirot | 01:02 |
Shrews | rloo: it did | 01:02 |
rloo | Shrews: hmm. ok, will need a few minutes to set up a dev environment and test... | 01:03 |
Shrews | rloo: it could TOTALLY be something stupid by me. it's been a long day | 01:03 |
naohirot | rloo: please check it again if you got a time. | 01:04 |
rloo | Shrews: stupid or not, worth figuring out ;) | 01:04 |
rloo | Shrews: but maybe it is time for you to work over a beer or two with your buddies there | 01:04 |
Shrews | rloo: if you insist | 01:04 |
rloo | naohirot: will look in a few minutes. I hope. I'm not really working now but I guess I am ;) | 01:05 |
rloo | Shrews: I insist unless your boss sez otherwise ;) | 01:05 |
*** Haomeng|2 has joined #openstack-ironic | 01:06 | |
rloo | naohirot: I think there is some confusion. | 01:06 |
*** Haomeng has quit IRC | 01:06 | |
rloo | naohirot: I agree that parse_driver_info() checks that 'irmc_sensor_method' exists in the dictionary that it returns. | 01:06 |
*** korekhov has joined #openstack-ironic | 01:07 | |
rloo | naohirot: 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 |
rloo | naohirot: https://review.openstack.org/#/c/146803/19..20/ironic/drivers/modules/irmc/management.py | 01:08 |
rloo | naohirot: line 210: sensor_method = task.node.driver_info['irmc_sensor_method'] | 01:08 |
rloo | naohirot: task.node.driver_info['irmc_sensor_method'] may not exist. | 01:09 |
rloo | naohirot: info['irmc_sensor_method'] does exist, if info = irmc_common.parse_driver_info(task.node) | 01:09 |
naohirot | rloo: Umm, I'm thinking | 01:10 |
naohirot | rloo: irmc_common.parse_driver_info(task.node) doesn't make sure sensor_method = task.node.driver_info['irmc_sensor_method']? | 01:13 |
rloo | naohirot: 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 |
rloo | naohirot: didn't you write that code? | 01:14 |
naohirot | rloo: I still don't understand why it doesn't make sure. | 01:14 |
naohirot | rloo: Yes | 01:14 |
rloo | naohirot: where/how do you think the code makes sure that driver_info['irmc_sensor_method'] exists? | 01:15 |
naohirot | rloo: I think task.node.driver_info is singleton. | 01:15 |
rloo | naohirot: what do you mean 'singleton'? .driver_info is a dictionary. | 01:16 |
naohirot | rloo: I mean driver_info exists only one instance | 01:16 |
rloo | naohirot: each node has a driver_info dictionary | 01:17 |
naohirot | rloo: so task.node.driver_info instance is same as info = irmc_common.parse_driver_info(task.node) | 01:18 |
rloo | naohirot: no, it isn't | 01:18 |
rloo | naohirot: https://review.openstack.org/#/c/146803/20/ironic/drivers/modules/irmc/common.py | 01:19 |
naohirot | rloo: but why the test case I added succeed? | 01:19 |
rloo | naohirot: line 98 | 01:19 |
rloo | naohirot: d_info = dict(list(req.items()) + list(opt.items())) | 01:19 |
*** achanda has quit IRC | 01:19 | |
rloo | naohirot: line 118: return d_info | 01:19 |
rloo | naohirot: d_info is not the same as node.driver_info | 01:20 |
naohirot | rloo: I got it | 01:21 |
naohirot | rloo: I'll assign the d_info at line 118 to task.node.driver_info | 01:22 |
rloo | naohirot: I'm glad. because I'm not sure how I could have explained it | 01:22 |
naohirot | rloo: thinks for your insight :) | 01:23 |
*** chenglch has joined #openstack-ironic | 01:23 | |
rloo | naohirot: umm, wait. what did you just say. | 01:25 |
rloo | naohirot: I'll assign the d_info at line 118 to task.node.driver_info | 01:25 |
rloo | naohirot: no, I don't think the code should be modifying driver_info | 01:25 |
naohirot | rloo: you are saying that the d_info at line 118 is abandoned, right? | 01:27 |
*** jlvllal_sfo has quit IRC | 01:27 | |
rloo | naohirot: d_info is returned. the caller of parse_driver_info gets d_info, and can use that information. | 01:27 |
openstackgerrit | Josh Gachnang proposed openstack/ironic: WIP: Implement execute clean steps https://review.openstack.org/155561 | 01:28 |
naohirot | rloo: Yeah, I have to checked the caller how the caller treat the returned the d_info at line 118. | 01:28 |
rloo | the caller can do whatever it wants with d_info, it is just a dictionary/object. it isn't owned by anything. | 01:29 |
naohirot | rloo: 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 |
naohirot | rloo: /could/could not/ | 01:30 |
rloo | naohirot: 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 |
naohirot | rloo: If CONF.irmc.sensor_method is None, then driver_info[irmc_sensor_method] should have 'ipmitool', I think. | 01:32 |
naohirot | rloo: that is the code I wrote. | 01:32 |
naohirot | rloo: anyway I'll add test case which proof driver_info[irmc_sensor_method] and key irmc_sensor_method exit. | 01:34 |
naohirot | rloo: before you leave, I'd like to know your solution if you have in your mind. | 01:36 |
*** korekhov has quit IRC | 01:37 | |
*** korekhov has joined #openstack-ironic | 01:37 | |
naohirot | rloo: good night, thanks for sparing your time :) | 01:38 |
*** Marga_ has quit IRC | 01:39 | |
*** korekhov has quit IRC | 01:42 | |
*** ChuckC has quit IRC | 01:45 | |
*** jerryz has joined #openstack-ironic | 01: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 IRC | 01:48 | |
*** korekhov has joined #openstack-ironic | 01:50 | |
*** ChuckC has joined #openstack-ironic | 01:52 | |
*** davideagnello has quit IRC | 01:52 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/155571 | 01:52 |
*** anderbubble has quit IRC | 01:58 | |
*** davideagnello has joined #openstack-ironic | 01:58 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/155583 | 02:00 |
*** davideagnello has quit IRC | 02:04 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver https://review.openstack.org/146803 | 02:09 |
openstackgerrit | Merged openstack/ironic-python-agent: Log required troubleshooting info on image dl fail https://review.openstack.org/155080 | 02:17 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Virtual Media Deploy module for iRMC Driver https://review.openstack.org/151958 | 02:23 |
*** anderbubble has joined #openstack-ironic | 02:24 | |
*** Haomeng has joined #openstack-ironic | 02:24 | |
*** spandhe has quit IRC | 02:24 | |
*** Haomeng|2 has quit IRC | 02:25 | |
*** mlsoule has joined #openstack-ironic | 02:27 | |
*** mlsoule has quit IRC | 02:28 | |
*** ramineni has joined #openstack-ironic | 02:41 | |
*** anderbubble has quit IRC | 02:47 | |
*** zhenzanz has joined #openstack-ironic | 02:47 | |
*** anderbubble has joined #openstack-ironic | 02:47 | |
*** yuanying_ has joined #openstack-ironic | 02:57 | |
*** yuanying_ has quit IRC | 02:59 | |
*** yuanying_ has joined #openstack-ironic | 02:59 | |
*** yuanying has quit IRC | 03:00 | |
*** korekhov has quit IRC | 03:03 | |
mrda | Has everyone gone to dinner? | 03:07 |
mrda | Shrews: Thanks for the +A | 03:07 |
mrda | (too bad things failed in gate due to (what looks like) a transient network failure :( | 03:07 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver https://review.openstack.org/154808 | 03:19 |
*** yuanying_ has quit IRC | 03:19 | |
*** rloo has quit IRC | 03:23 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver https://review.openstack.org/154814 | 03:23 |
*** yuanying has joined #openstack-ironic | 03:24 | |
*** jlvllal_sfo has joined #openstack-ironic | 03:27 | |
*** yuanying has quit IRC | 03:27 | |
*** harlowja_ is now known as harlowja_away | 03:27 | |
*** yuanying has joined #openstack-ironic | 03:28 | |
*** jlvillal_sfo has joined #openstack-ironic | 03:29 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for agent_ilo driver https://review.openstack.org/154816 | 03:31 |
openstackgerrit | Tan Lin proposed openstack/python-ironicclient: Enable ironicclient to support --ironic-api-version 1.x https://review.openstack.org/155624 | 03:33 |
*** yuanying has quit IRC | 03:34 | |
*** yuanying has joined #openstack-ironic | 03:36 | |
*** yuanying_ has joined #openstack-ironic | 03:38 | |
*** ChuckC_ has joined #openstack-ironic | 03:40 | |
*** yuanying has quit IRC | 03:40 | |
*** Marga_ has joined #openstack-ironic | 03:42 | |
*** ChuckC has quit IRC | 03:44 | |
*** Nisha has joined #openstack-ironic | 03:44 | |
*** jlvillal_sfo_ has joined #openstack-ironic | 03:49 | |
jlvillal_sfo_ | Nisha: Sorry I forgot to tell devananda to look at your patch :( | 03:49 |
Nisha | jlvillal_sfo_, whats the time there? | 03:50 |
jlvillal_sfo_ | 19:50 | 03:50 |
Nisha | is devananda still around? | 03:50 |
Nisha | that is blocking other patches actually | 03:50 |
*** jlvillal_sfo has quit IRC | 03:50 | |
Nisha | jlvillal_sfo_, is devananda NobodyCam around there? | 03:51 |
*** jlvillal_sfo has joined #openstack-ironic | 03:52 | |
jlvillal_sfo | Nisha: I am in my hotel room | 03:52 |
jlvillal_sfo | Nisha: We all left the sprint around 17:30 as they had to use the room for an event. | 03:52 |
Nisha | ohk | 03:52 |
jlvillal_sfo | Nisha: Sorry if dupe messages. Fighting hotel WiFi :( | 03:53 |
Nisha | jlvillal_sfo, ok | 03:53 |
Nisha | :) | 03:53 |
Nisha | ok one thing, recheck no bug initiates the jenkins again? | 03:53 |
Nisha | i 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 something | 03:54 |
Nisha | jlvillal_sfo, ^^^^ | 03:54 |
*** jlvillal_sfo_ has quit IRC | 03:54 | |
mrda | Nisha, yes, "recheck no bug" retriggers jenkins | 03:54 |
jlvillal_sfo | Nisha: I think so | 03:55 |
Nisha | do we need to enclose in quotes? | 03:55 |
jlvillal_sfo | mrda: Aren't you supposed to be flying home right about now ;) | 03:55 |
mrda | gate is fixed - about 4 hours ago | 03:55 |
jlvillal_sfo | Nisha: No quotes needed | 03:55 |
mrda | All flights out of SFO delayed by over an hour | 03:55 |
jlvillal_sfo | From what I have seen | 03:55 |
jlvillal_sfo | mrda: Ouch! | 03:55 |
jlvillal_sfo | You left a long time ago. Sorry you have to wait so long :( | 03:56 |
mrda | oh well, I will get home eventually | 03:56 |
mrda | Just hope I don't miss VA8 as a result of this delay | 03:56 |
mrda | otherwise I miss Valentine's Day, and what I have planned for my wife | 03:57 |
jlvillal_sfo | mrda: Have a great trip home. I hope you make your connections! | 03:57 |
mrda | thanks! Enjoy the meetup tomorrow | 03:57 |
jlvillal_sfo | Thank you. | 03:57 |
*** jlvillal_sfo has quit IRC | 04:03 | |
*** Marga_ has quit IRC | 04:39 | |
*** Marga_ has joined #openstack-ironic | 04:40 | |
*** yog_ has joined #openstack-ironic | 04:42 | |
*** pcrews has quit IRC | 04:45 | |
openstackgerrit | Merged openstack/ironic: Add logical name support to Ironic https://review.openstack.org/141737 | 04:51 |
*** achanda has joined #openstack-ironic | 05:02 | |
*** Haomeng has quit IRC | 05:03 | |
*** pensu has joined #openstack-ironic | 05:03 | |
*** davideagnello has joined #openstack-ironic | 05:04 | |
*** Haomeng has joined #openstack-ironic | 05:04 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add documentation for VirtualBox drivers https://review.openstack.org/146307 | 05:05 |
*** sdake_ has quit IRC | 05:08 | |
*** davideagnello has quit IRC | 05:09 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Add support for mocking with VirtualBox https://review.openstack.org/155647 | 05:36 |
*** lazy_prif has joined #openstack-ironic | 05:41 | |
*** lazy_prif is now known as lazy_prince | 05:41 | |
*** killer_prince has quit IRC | 05:42 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Add support for mocking with VirtualBox https://review.openstack.org/155647 | 05:43 |
*** killer_prince has joined #openstack-ironic | 05:45 | |
*** lazy_prince has quit IRC | 05:46 | |
*** killer_prince is now known as lazy_prince | 05:46 | |
*** Haomeng|2 has joined #openstack-ironic | 05:46 | |
*** Haomeng has quit IRC | 05:47 | |
*** ParsectiX has quit IRC | 05:48 | |
*** sdake has joined #openstack-ironic | 05:48 | |
*** sdake has quit IRC | 05:48 | |
*** sdake has joined #openstack-ironic | 05:48 | |
*** ParsectiX has joined #openstack-ironic | 05:49 | |
*** ParsectiX has quit IRC | 05:50 | |
*** Haomeng|2 has quit IRC | 05:50 | |
*** stendulker has joined #openstack-ironic | 05:51 | |
*** lazy_prince has quit IRC | 05:52 | |
*** killer_prince has joined #openstack-ironic | 05:53 | |
*** killer_prince is now known as lazy_prince | 05:53 | |
*** ParsectiX has joined #openstack-ironic | 05:55 | |
*** Haomeng|2 has joined #openstack-ironic | 05:56 | |
*** achanda has quit IRC | 06:03 | |
*** lazy_prince has quit IRC | 06:07 | |
*** killer_prince has joined #openstack-ironic | 06:07 | |
*** killer_prince is now known as lazy_prince | 06:07 | |
*** ParsectiX has quit IRC | 06:09 | |
*** stendulker_ has joined #openstack-ironic | 06:11 | |
*** achanda has joined #openstack-ironic | 06:11 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Transifex https://review.openstack.org/155657 | 06:14 |
*** stendulker has quit IRC | 06:14 | |
*** coolsvap_ is now known as coolsvap | 06:18 | |
*** anderbubble has quit IRC | 06:21 | |
*** spandhe has joined #openstack-ironic | 06:21 | |
openstackgerrit | Tan Lin proposed openstack/python-ironicclient: Enable ironicclient with --ironic-api-version 1.x https://review.openstack.org/155624 | 06:29 |
*** marios has joined #openstack-ironic | 06:40 | |
*** marios has quit IRC | 06:41 | |
*** marios has joined #openstack-ironic | 06:41 | |
*** jiangfei has joined #openstack-ironic | 06:45 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add driver interface for RAID configuration https://review.openstack.org/155230 | 06:50 |
openstackgerrit | Merged stackforge/proliantutils: Add cleaning operations https://review.openstack.org/154331 | 06:51 |
*** marios has quit IRC | 06:52 | |
*** marios has joined #openstack-ironic | 06:52 | |
*** dlpartain has joined #openstack-ironic | 06:55 | |
*** dlpartain has left #openstack-ironic | 06:58 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Add hpssa module to proliantutils https://review.openstack.org/153519 | 07:02 |
*** yuanying_ has quit IRC | 07:03 | |
*** yuanying has joined #openstack-ironic | 07:03 | |
*** humble_ has joined #openstack-ironic | 07:04 | |
*** stendulker has joined #openstack-ironic | 07:12 | |
*** stendulker_ has quit IRC | 07:12 | |
*** stendulker has quit IRC | 07:25 | |
*** stendulker has joined #openstack-ironic | 07:26 | |
*** Haomeng has joined #openstack-ironic | 07:29 | |
*** Haomeng|2 has quit IRC | 07:30 | |
*** andreykurilin_ has joined #openstack-ironic | 07:42 | |
*** lazy_prince has quit IRC | 07:45 | |
openstackgerrit | Tan Lin proposed openstack/python-ironicclient: Enable ironicclient with --ironic-api-version 1.x https://review.openstack.org/155624 | 07:45 |
*** killer_prince has joined #openstack-ironic | 07:45 | |
*** killer_prince is now known as lazy_prince | 07:45 | |
*** andreykurilin_ has quit IRC | 07:50 | |
*** andreykurilin_ has joined #openstack-ironic | 07:50 | |
*** andreykurilin_ has quit IRC | 08:07 | |
*** jcoufal has joined #openstack-ironic | 08:09 | |
*** andreykurilin_ has joined #openstack-ironic | 08:11 | |
*** spandhe has quit IRC | 08:13 | |
*** ndipanov has joined #openstack-ironic | 08:14 | |
*** ifarkas has joined #openstack-ironic | 08:17 | |
*** oomichi has quit IRC | 08:21 | |
*** achanda has quit IRC | 08:21 | |
*** achanda has joined #openstack-ironic | 08:23 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Add hpssa module to proliantutils https://review.openstack.org/153519 | 08:23 |
*** achanda has quit IRC | 08:38 | |
*** openstackgerrit has quit IRC | 08:42 | |
*** openstackgerrit has joined #openstack-ironic | 08:42 | |
*** achanda has joined #openstack-ironic | 08:44 | |
*** erwan_taf has joined #openstack-ironic | 08:46 | |
*** lazy_prince is now known as killer_prince | 08:55 | |
*** yuanying has quit IRC | 09:01 | |
*** killer_prince is now known as lazy_prince | 09:02 | |
*** lucasagomes has joined #openstack-ironic | 09:03 | |
*** bauzas is now known as bauwser | 09:04 | |
*** dtantsur|afk is now known as dtantsur | 09:06 | |
dtantsur | Morning Ironic, TGIF finally :) | 09:06 |
Nisha | dtantsur, morning | 09:10 |
Nisha | dtantsur, need reviews again on inspetion patches | 09:11 |
Nisha | :) | 09:11 |
dtantsur | sure, lemme wake up :) | 09:13 |
*** humble_ has quit IRC | 09:13 | |
*** achanda has quit IRC | 09:14 | |
*** gilliard is now known as gillllliard | 09:15 | |
*** jistr has joined #openstack-ironic | 09:16 | |
*** romcheg has joined #openstack-ironic | 09:18 | |
*** dlpartain has joined #openstack-ironic | 09:18 | |
dtantsur | Nisha, I'm leaving first patch for devananda to have a look first, as I'm not strong in our state machine... | 09:19 |
Nisha | ok | 09:19 |
Nisha | still you can ad your opinion on same | 09:19 |
*** derekh has joined #openstack-ironic | 09:19 | |
openstackgerrit | Tan Lin proposed openstack/python-ironicclient: Enable ironicclient with --ironic-api-version 1.x https://review.openstack.org/155624 | 09:20 |
*** romcheg has quit IRC | 09:22 | |
*** zhenzanz has quit IRC | 09:26 | |
*** dlpartain has quit IRC | 09:26 | |
*** romcheg has joined #openstack-ironic | 09:27 | |
stendulker | dtantsur: Good Morning, | 09:28 |
stendulker | dtantsur: Can you please review spec for Management Interfaces to support UEFI Secure Boot https://review.openstack.org/#/c/135845/ | 09:29 |
dtantsur | hey hey folks, I see all your patches in my mail :) just need to find some time for them | 09:30 |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Add hpssa module to proliantutils https://review.openstack.org/153519 | 09:30 |
stendulker | dtantsur: :) | 09:30 |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Add hpssa module to proliantutils https://review.openstack.org/153519 | 09:36 |
*** chlong has quit IRC | 09:49 | |
*** vdrok_afk is now known as vdrok | 09:52 | |
*** pas-ha has joined #openstack-ironic | 09:58 | |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/155571 | 10:01 |
openstackgerrit | Merged openstack/ironic: Stop console at first if console is enabled when destroy node https://review.openstack.org/154758 | 10:02 |
*** chenglch has quit IRC | 10:03 | |
*** alexpilotti has joined #openstack-ironic | 10:14 | |
*** romcheg1 has joined #openstack-ironic | 10:16 | |
*** romcheg has quit IRC | 10:16 | |
*** lynxman has quit IRC | 10:19 | |
*** lynxman has joined #openstack-ironic | 10:20 | |
*** romcheg has joined #openstack-ironic | 10:21 | |
*** romcheg1 has quit IRC | 10:21 | |
*** MattMan has joined #openstack-ironic | 10:31 | |
*** Nisha has quit IRC | 10:37 | |
*** stendulker has quit IRC | 11:02 | |
*** pensu has left #openstack-ironic | 11:06 | |
*** Haomeng|2 has joined #openstack-ironic | 11:06 | |
*** ramineni has quit IRC | 11:07 | |
*** Haomeng has quit IRC | 11:07 | |
*** pas-ha has quit IRC | 11:27 | |
*** pas-ha has joined #openstack-ironic | 11:29 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Functional test for setting IPMI credentials https://review.openstack.org/142823 | 11:30 |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Require maintenance mode for setting IPMI credentials https://review.openstack.org/155307 | 11:30 |
openstackgerrit | Diego de Lima Pereira proposed openstack/python-ironicclient: Fix Python Ironic Client Documentation https://review.openstack.org/155714 | 11:34 |
*** pensu has joined #openstack-ironic | 11:42 | |
*** foexle has joined #openstack-ironic | 11:45 | |
foexle | heyho 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 device | 11:47 |
foexle | so udev will choose as eth0 the first device which was found | 11:48 |
foexle | is there any possibilities to tell which MAC/device should be used as eth0 | 11:48 |
lucasagomes | foexle, you mean tell nova which mac (port) should be used for dpeloyment? | 11:49 |
lucasagomes | the thing, we pass to nova a list of ports configured in Ironic | 11:49 |
lucasagomes | and as we have only 1 ip, nova will randomly choose one of the ports and create the network for it | 11:50 |
lucasagomes | it'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 deployment | 11:50 |
lucasagomes | right now you can't :/ | 11:50 |
foexle | lucasagomes: yes ... for this case I've created only one port for the BM host | 11:50 |
lucasagomes | yeah, that's how we workaround it right now | 11:50 |
foexle | so the correct port whith the correct mac | 11:50 |
foexle | PXE gets the IP | 11:51 |
foexle | from dhcp correctly | 11:51 |
lucasagomes | it's also something that need to be discussed, like why we register all ports in Ironic if only 1 is actually used and all | 11:51 |
foexle | will download theimage | 11:51 |
lucasagomes | foexle, right... so you think that having a field like primary=True in the Port resource | 11:51 |
lucasagomes | is alright? | 11:51 |
* lucasagomes have been thinking what would be the best way to do it | 11:51 | |
foexle | yeah ... so i mean i have a couple of available BM hosts | 11:52 |
foexle | and if I'm using ubuntu | 11:52 |
foexle | we notice ... ubuntu will grap randomly the eth0 device (first match) | 11:52 |
foexle | but this "eth0" device is not everytime the same as the used port (mac) | 11:53 |
foexle | so sometimes is use in ironic mac xxxxxx and this device will on boot declared by ubuntu to eth2 | 11:54 |
foexle | sry for my description | 11:54 |
foexle | is a little bit complicated | 11:54 |
foexle | but you understand ? | 11:54 |
lucasagomes | hmm this is after the machine is deployed? | 11:54 |
foexle | yes | 11:54 |
*** mlsoule has joined #openstack-ironic | 11:54 | |
lucasagomes | oh right | 11:54 |
lucasagomes | ok... so the way we do it | 11:54 |
lucasagomes | is using the configdrive to pass some information to the instance | 11:54 |
lucasagomes | and we rename the ethernet devices accondingly | 11:54 |
lucasagomes | this is a pos deploy configuration | 11:55 |
foexle | ahhh | 11:55 |
lucasagomes | we have this patch here in Nova waiting for review /me lemme grab the link | 11:55 |
lucasagomes | https://review.openstack.org/#/c/145302/ | 11:55 |
*** Faizan has joined #openstack-ironic | 11:56 | |
lucasagomes | foexle, ^ 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 that | 11:56 |
lucasagomes | i want to pass info to the instance so it can rename my ethernet devices accordingly | 11:56 |
lucasagomes | that's the only bit that needs to get merged | 11:56 |
foexle | lucasagomes: absolutely .... many thanks for that hint | 11:56 |
lucasagomes | and ofc, you need some tool to configure ur instance | 11:57 |
lucasagomes | cloud-init for example, although I'm not sure if cloud-init rename eth devices at the moment | 11:57 |
lucasagomes | foexle, cool np | 11:57 |
foexle | all right so in Juno i need luck to get the right device or changing dev by hand via IPMI | 11:57 |
lucasagomes | in juno there's no configdrive :/ | 11:58 |
lucasagomes | yeah... unless somehow you can pass info to ur instance, like metadata service or something | 11:58 |
lucasagomes | that would work too | 11:58 |
lucasagomes | but I'm not exactly sure how you can do that | 11:58 |
foexle | lucasagomes: nope that'snotpossible | 11:58 |
foexle | because the device is not up :) | 11:58 |
lucasagomes | oh true | 11:58 |
lucasagomes | yeah, not sure... you could bake something in the image | 11:59 |
foexle | but anyway ... that helps | 11:59 |
lucasagomes | that would check for the network devices | 11:59 |
lucasagomes | see if it's connect to a specific network and if so you rename it | 11:59 |
lucasagomes | like an extra service | 11:59 |
lucasagomes | but, yeah... it's not straight forward :/ | 11:59 |
foexle | :) | 11:59 |
foexle | ok so i think change images would be the best way atm | 12:00 |
foexle | lucasagomes: many thanks for your help | 12:00 |
lucasagomes | foexle, you very welcome | 12:00 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add iscsi extension https://review.openstack.org/155727 | 12:10 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add AgentVendorPasshtru for the PXE driver https://review.openstack.org/155728 | 12:13 |
*** lsmola has joined #openstack-ironic | 12:16 | |
*** pensu has quit IRC | 12:22 | |
*** lucasagomes is now known as lucas-hungry | 12:23 | |
*** athomas has joined #openstack-ironic | 12:25 | |
*** smoriya has quit IRC | 12:28 | |
*** Faizan has quit IRC | 12:33 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node https://review.openstack.org/155731 | 12:35 |
*** athomas has quit IRC | 12:42 | |
*** takadayuiko has quit IRC | 13:09 | |
*** dprince has joined #openstack-ironic | 13:11 | |
*** lazy_prince is now known as killer_prince | 13:19 | |
*** EmilienM|afk is now known as EmilienM | 13:19 | |
openstackgerrit | Merged openstack/ironic: Add support for local boot https://review.openstack.org/146189 | 13:29 |
*** jcoufal_ has joined #openstack-ironic | 13:30 | |
dtantsur | lucas-hungry, ^^^ :) | 13:32 |
*** lucas-hungry is now known as lucasagomes | 13:32 | |
lucasagomes | dtantsur, yushhhhhhh o/ | 13:32 |
*** jcoufal has quit IRC | 13:33 | |
jroll | morning y'all :) | 13:34 |
jroll | nice merge | 13:34 |
lucasagomes | jroll, morning! | 13:34 |
lucasagomes | what was the gate problem in the end? | 13:35 |
jroll | you saw Shrews' revert of making the ironic user non-admin in devstack? | 13:36 |
jroll | needed the same for the swift user | 13:36 |
lucasagomes | yeah, I saw that | 13:36 |
lucasagomes | ohh | 13:36 |
jroll | https://review.openstack.org/#/c/155357/ | 13:36 |
lucasagomes | devstack then | 13:36 |
jroll | yeah | 13:36 |
lucasagomes | gotcha | 13:37 |
jroll | also, using Depends-On: is awesome | 13:37 |
lucasagomes | what is that? | 13:37 |
* lucasagomes feels outdated | 13:38 | |
jroll | jim blair added it | 13:38 |
jroll | you put a change ID there and can have cross project dependencies | 13:38 |
jroll | in gerrit | 13:38 |
jroll | so a nova change can depend on an ironic change or vice versa | 13:38 |
*** jjohnson2 has joined #openstack-ironic | 13:39 | |
lucasagomes | oh holy cow | 13:40 |
lucasagomes | yeah, that was very much needed indeed | 13:40 |
jroll | yep, tis awesome | 13:41 |
jroll | so I made an IPA patch depend on my devstack change to make sure it worked this time :P | 13:41 |
lucasagomes | nice one! yeah I will make the pxe.AgentVendorPasstrhu to depend on the iscsi extension in IPA to test it out :D | 13:44 |
jroll | nice :) | 13:46 |
dtantsur | jroll, morning | 13:48 |
jroll | \o dtantsur :) | 13:49 |
*** Nisha has joined #openstack-ironic | 14:06 | |
Nisha | dtantsur, there? | 14:06 |
Nisha | lucasagomes, hi | 14:09 |
lucasagomes | Nisha, hi there | 14:11 |
Nisha | lucasagomes, hi | 14:11 |
Nisha | do you know if dtantsur is there or not? | 14:11 |
lucasagomes | Nisha, hmm no idea. He might be afk | 14:11 |
Nisha | lucasagomes, request you to review inspection patches | 14:12 |
lucasagomes | (we live in diff countries, so I don't see him very often) | 14:12 |
lucasagomes | Nisha, ack, yeah I have to catch up with the reviews | 14:12 |
Nisha | lucasagomes, oh i thought u guys are in same company and country | 14:16 |
Nisha | :) | 14:16 |
lucasagomes | Nisha, same company :) | 14:18 |
openstackgerrit | Merged openstack/ironic: Imported Translations from Transifex https://review.openstack.org/155657 | 14:18 |
*** david-lyle_afk is now known as david-lyle | 14:22 | |
Nisha | lucasagomes, :) | 14:28 |
jroll | Nisha: you need all bp/ironic-node-properties-discovery patches reviewed, yes? | 14:30 |
Nisha | yes | 14:30 |
jroll | ok | 14:30 |
Nisha | jroll, even this one https://review.openstack.org/151596 | 14:33 |
jroll | Nisha: one thing at a time :) | 14:33 |
Nisha | which is ilo Node properties discovery | 14:33 |
*** annegentle has quit IRC | 14:33 | |
*** yog_ has quit IRC | 14:35 | |
dtantsur | Nisha, o/ | 14:43 |
dtantsur | sorry was on lunch | 14:43 |
*** priteau has joined #openstack-ironic | 14:44 | |
jroll | dtantsur: I'm not sure what your comment means here https://review.openstack.org/#/c/149817/18/ironic/drivers/base.py | 14:45 |
jroll | afaict that hasn't been defined yet? | 14:45 |
dtantsur | jroll, see the next patch actually | 14:45 |
jroll | mmm | 14:45 |
dtantsur | previously it used this return value to distinguish whether inspection was already finished | 14:45 |
dtantsur | (for in-band it won't be the case) | 14:46 |
jroll | right, I see | 14:47 |
jroll | just like deploy | 14:47 |
dtantsur | yep | 14:47 |
jroll | well, currently it doesn't use that return value | 14:47 |
jroll | but I see what you mean | 14:47 |
*** Nisha has quit IRC | 14:48 | |
dtantsur | jroll, it was changed recently to always assume inspection has finished. which I obviously object :) | 14:48 |
*** Nisha has joined #openstack-ironic | 14:48 | |
jroll | yeah :) | 14:48 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove rootfstype=ramfs from PXE templates https://review.openstack.org/155765 | 14:53 |
*** dlpartain has joined #openstack-ironic | 14:55 | |
*** zz_jgrimm is now known as jgrimm | 14:56 | |
*** jcoufal_ has quit IRC | 14:57 | |
*** r-daneel has joined #openstack-ironic | 15:00 | |
lucasagomes | jroll, how I will call the new driver with pxe and agent O.O | 15:02 |
lucasagomes | pxe_agent_ssh!? | 15:02 |
* lucasagomes is bad with names | 15:02 | |
jroll | lol | 15:02 |
jroll | iscsi_ssh? | 15:02 |
Nisha | jroll, | 15:03 |
jroll | and then we can get rid of pxe_ssh eventually | 15:03 |
lucasagomes | jroll, that would be better, but kinda break the pattern where they have pxe_ etc | 15:03 |
jroll | because now pxe driver can local boot :P | 15:03 |
lucasagomes | cool yeah I will add iscsi_ssh | 15:03 |
lucasagomes | yeah it's all complicated | 15:03 |
Nisha | devananda, did not agree to remove INSPECTED or rather i would say *ED states | 15:03 |
jroll | and agent pxe boots the ramdisk so ya | 15:03 |
Nisha | in weekly meeting | 15:03 |
lucasagomes | lol damn... true iscsi_ssh is it | 15:03 |
lucasagomes | <deploy>_<power> methods | 15:03 |
jroll | Nisha: ok, I'll talk with people today, I thought the plan was to get rid of *ED unless needed | 15:04 |
jroll | lucasagomes: +1 | 15:04 |
Nisha | jroll, he said to add them as no-op state | 15:04 |
Nisha | no-op means we may not do any action in that state | 15:04 |
jroll | Nisha: great, can you please put this info on the review in gerrit? | 15:05 |
Nisha | while i need the INSPECTED state as i populate last_inspected field while moving from INSPECTING to INSPECTED | 15:05 |
jroll | Nisha: great, can you please put this info on the review in gerrit? | 15:06 |
* jroll is about to head to the meetup | 15:06 | |
Nisha | in states code patch? | 15:06 |
jroll | yes, where I left the comment | 15:06 |
Nisha | Ok | 15:06 |
jroll | so that 1) everyone can see it, and 2) I don't forget | 15:06 |
Nisha | i will add and post a new patch | 15:06 |
Nisha | is it going to be discussed today? | 15:06 |
jroll | I don't know | 15:07 |
jroll | don't post a new patch if my comments are invalid | 15:07 |
Nisha | ok. | 15:07 |
Nisha | i will add comments per transition | 15:07 |
jroll | thanks | 15:08 |
Nisha | whatever i could think of :) | 15:08 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add AgentVendorPasshtru for the PXE driver https://review.openstack.org/155728 | 15:09 |
*** dlpartain has left #openstack-ironic | 15:09 | |
lucasagomes | 2+2 on https://review.openstack.org/#/c/155316 someone wants to push the button? | 15:10 |
*** MattMan has quit IRC | 15:10 | |
jroll | lucasagomes: boom | 15:15 |
lucasagomes | \o/ | 15:15 |
*** jjohnson2 has quit IRC | 15:16 | |
openstackgerrit | Merged openstack/ironic-specs: Support per Driver Sensor Meters https://review.openstack.org/130074 | 15:16 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Update Ilo drivers to use REST API interface to iLO https://review.openstack.org/154403 | 15:18 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node https://review.openstack.org/155731 | 15:18 |
*** MattMan has joined #openstack-ironic | 15:25 | |
NobodyCam | good mornig Ironic | 15:27 |
dtantsur | NobodyCam, morning. TGIF today! | 15:27 |
lucasagomes | NobodyCam, morning | 15:28 |
*** anderbubble has joined #openstack-ironic | 15:28 | |
NobodyCam | mornig dtantsur & lucasagomes ... and OH ya TGIF!!!! | 15:29 |
openstackgerrit | Mark Goddard proposed openstack/ironic: Avoid deregistering conductor following SIGUSR1. https://review.openstack.org/155785 | 15:31 |
*** datajerk has quit IRC | 15:38 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Functional test for setting IPMI credentials https://review.openstack.org/142823 | 15:39 |
*** datajerk has joined #openstack-ironic | 15:39 | |
dtantsur | ifarkas, 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_afk | 15:51 | |
*** Nisha has quit IRC | 15:53 | |
*** pcrews has joined #openstack-ironic | 15:59 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Common changes for secure boot support https://review.openstack.org/153974 | 16:03 |
ifarkas | dtantsur, sure, I will take look | 16:03 |
lucasagomes | jroll, https://github.com/openstack/nova/blob/master/nova/virt/ironic/patcher.py#L36 :( | 16:04 |
lucasagomes | iscsi_ssh | 16:04 |
lucasagomes | >.< | 16:04 |
jroll | lucasagomes: rage | 16:05 |
lucasagomes | lol | 16:05 |
lucasagomes | same here dude | 16:05 |
lucasagomes | I was like WTH! | 16:05 |
* lucasagomes probably coded that long time ago :/ | 16:05 | |
jroll | lucasagomes: that's just for backwards compat stuff at this point, might be fine to go ahead without changing that | 16:06 |
*** ndipanov is now known as ndipanoff | 16:07 | |
lucasagomes | jroll, yeah, I will look at what that nova is doing and try to make it smarter | 16:09 |
lucasagomes | sigh | 16:09 |
jroll | lucasagomes: I don't remember the deprecation plan for flavor/ramdisk pairing but we may be able to deprecate after kilo | 16:09 |
lucasagomes | jroll, yeah, well we still need to somewhere say what is the kernel/ramdisk for that image | 16:09 |
BadCub | Morning Ironic! | 16:09 |
lucasagomes | we have to think about where it should live | 16:10 |
lucasagomes | now it's glance metadata | 16:10 |
*** jgrimm_ has joined #openstack-ironic | 16:10 | |
lucasagomes | jroll, we already deprecated the deploy ramdisk and kernel being in the flavor | 16:10 |
lucasagomes | but that's something else | 16:10 |
jroll | lucasagomes: the operator specifies it in driver_info | 16:10 |
lucasagomes | that's the ramdisk/kernel for the user image | 16:10 |
jroll | the PXEDriverFields pulls it from driver_info | 16:11 |
lucasagomes | jroll, we only know which image to use when smoeone do a nova boot :/ | 16:11 |
*** maurosr has quit IRC | 16:11 | |
lucasagomes | the deploy ramdisk/kernel is fine to the operator specify | 16:11 |
jroll | eh, no, PXEDriverFields only exists to get deploy ramdisk stuff from flavor | 16:11 |
jroll | ironic gets the instance kernel/ramdisk from glance | 16:11 |
jroll | nova doesn't put it there | 16:11 |
lucasagomes | ohhh | 16:11 |
jroll | https://github.com/openstack/nova/blob/master/nova/virt/ironic/patcher.py#L134-141 | 16:12 |
lucasagomes | damn ok I was confusing it then | 16:12 |
*** mjturek has joined #openstack-ironic | 16:12 | |
lucasagomes | god damn, so devstack should be updated | 16:12 |
jroll | that's the only special part, the pxe ramdisk works fine with generic patcher | 16:12 |
jroll | wait, devstack isn't updated?? | 16:12 |
lucasagomes | jroll, gotcha... hmm not sure | 16:12 |
lucasagomes | too many things in my mind now heh | 16:12 |
lucasagomes | I will check it out | 16:12 |
lucasagomes | I will add 'pxe' to the name for test sake for now | 16:12 |
lucasagomes | (won't be part of the patch) | 16:12 |
jroll | no, devstack is up to date | 16:13 |
jroll | https://github.com/openstack-dev/devstack/blob/master/lib/ironic#L595 | 16:13 |
*** jgrimm- has joined #openstack-ironic | 16:13 | |
*** jgrimm has quit IRC | 16:13 | |
*** mjturek1 has quit IRC | 16:13 | |
lucasagomes | hmm | 16:13 |
*** mjturek1 has joined #openstack-ironic | 16:14 | |
jroll | what kind of trouble is it giving you? | 16:14 |
*** maurosr has joined #openstack-ironic | 16:14 | |
*** pas-ha has quit IRC | 16:14 | |
*** Nisha has joined #openstack-ironic | 16:14 | |
lucasagomes | jroll, checking | 16:16 |
openstackgerrit | Merged openstack/ironic: AgentVendorInterface: Move to a common place https://review.openstack.org/155316 | 16:16 |
openstackgerrit | Merged openstack/ironic: Remove deploy_is_done() from AgentClient https://review.openstack.org/155325 | 16:16 |
*** mjturek has quit IRC | 16:16 | |
*** jgrimm_ has quit IRC | 16:17 | |
jroll | lucasagomes: I'm going to head in to the meetup, I'll be back in like an hour | 16:17 |
lucasagomes | jroll, no worries, enjoy it ! | 16:17 |
jroll | there's no reason it shouldn't just work though, maybe you just need updated devstack? | 16:17 |
jroll | enjoy my commute? yeah right :P | 16:17 |
jroll | see you soon | 16:18 |
lucasagomes | jroll, yeah not sure yet what's going on | 16:18 |
lucasagomes | maybe some stupid mistake I did | 16:18 |
lucasagomes | see ya | 16:18 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver https://review.openstack.org/154808 | 16:21 |
*** kbs1 has quit IRC | 16:22 | |
*** kbs has joined #openstack-ironic | 16:23 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver https://review.openstack.org/154814 | 16:26 |
*** mgoddard1 has joined #openstack-ironic | 16:31 | |
*** mgoddard has quit IRC | 16:33 | |
*** mgoddard1 has quit IRC | 16:35 | |
*** Marga_ has quit IRC | 16:38 | |
*** anderbubble has quit IRC | 16:38 | |
*** jcoufal has joined #openstack-ironic | 16:46 | |
*** marios has quit IRC | 16:47 | |
*** marios has joined #openstack-ironic | 16:47 | |
*** naohirot has quit IRC | 16:50 | |
*** jcoufal has quit IRC | 16:51 | |
openstackgerrit | Merged stackforge/ironic-discoverd: Require maintenance mode for setting IPMI credentials https://review.openstack.org/155307 | 16:51 |
*** jerryz has quit IRC | 16:52 | |
*** EmilienM is now known as EmilienM|afk | 16:56 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Update Ilo drivers to use REST API interface to iLO https://review.openstack.org/154403 | 16:58 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for agent_ilo driver https://review.openstack.org/154816 | 16:58 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Common changes for secure boot support https://review.openstack.org/153974 | 16:58 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver https://review.openstack.org/154814 | 16:58 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver https://review.openstack.org/154808 | 16:58 |
*** derekh has quit IRC | 16:59 | |
*** hemna_ has joined #openstack-ironic | 17:03 | |
*** MattMan has left #openstack-ironic | 17:09 | |
lucasagomes | jroll, ok it was something I did when tweaking stuff, I forgot to update the ipa-driver-name on the pxe config file | 17:09 |
lucasagomes | >.< | 17:09 |
* lucasagomes now it's being generated so, I don't have to manually chage things | 17:09 | |
* lucasagomes facepalm | 17:09 | |
lucasagomes | I will commit what I have now that it's working and take a break, but still need to add unittests :D | 17:10 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add AgentVendorPasshtru for the PXE driver https://review.openstack.org/155728 | 17:11 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove rootfstype=ramfs from PXE templates https://review.openstack.org/155765 | 17:11 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add AgentVendorPasshtru for the PXE driver https://review.openstack.org/155728 | 17:13 |
*** alexpilotti has quit IRC | 17:13 | |
*** alexpilotti has joined #openstack-ironic | 17:15 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add AgentVendorPasshtru for the PXE driver https://review.openstack.org/155728 | 17:15 |
*** alexpilotti has quit IRC | 17:16 | |
*** Marga_ has joined #openstack-ironic | 17:19 | |
lucasagomes | aight good night everyone, I will step out a bit | 17:21 |
*** martini has joined #openstack-ironic | 17:21 | |
*** lucasagomes is now known as lucas-dinner | 17:21 | |
dtantsur | have a nice weekend! | 17:23 |
*** dtantsur is now known as dtantsur|afk | 17:23 | |
lucas-dinner | dtantsur|afk, cheers you too | 17:25 |
lucas-dinner | enjoy it | 17:25 |
*** ekarlso has quit IRC | 17:25 | |
*** ekarlso has joined #openstack-ironic | 17:25 | |
*** alex_xu_ has joined #openstack-ironic | 17:30 | |
*** xu_alex has quit IRC | 17:32 | |
*** yjiang5 is now known as yginger5 | 17:36 | |
*** anderbubble has joined #openstack-ironic | 17:36 | |
*** jistr has quit IRC | 17:41 | |
*** lsmola has quit IRC | 17:42 | |
*** jlvllal_sfo has joined #openstack-ironic | 17:47 | |
*** harlowja_away is now known as harlowja_ | 17:48 | |
*** mgoddard has joined #openstack-ironic | 17:51 | |
*** davideagnello has joined #openstack-ironic | 17:52 | |
*** jogo is now known as flashgordon | 17:54 | |
jroll | lucas-dinner: lol, have a good night | 17:56 |
*** absubram has joined #openstack-ironic | 17:58 | |
* NobodyCam is back | 17:58 | |
*** tchaypo is now known as tchorizo | 17:59 | |
*** spandhe has joined #openstack-ironic | 18:00 | |
jxiaobin | hi all, is there anyone using vm/bm hybrid openstack environment? how do you explicitly ask for a bm/vm? | 18:02 |
priteau | Hi. 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-ironic | 18:03 | |
priteau | Is there some way to cancel the operation that is locking the node? | 18:03 |
jroll | jxiaobin: we run it in cells, where bm/vm cells are separate | 18:04 |
jroll | priteau: update nodes set reservation=null where uuid = '$uuid'; | 18:04 |
jroll | priteau: it's bad, I know :( | 18:04 |
priteau | thanks! | 18:06 |
jxiaobin | my experience is that there is no way to relate a call to the type of compute, say bm/vm | 18:09 |
jroll | jxiaobin: there's something called host aggregates, I don't know anything about them | 18:10 |
jroll | we define our cells to only accept baremetal or VM flavors | 18:10 |
jxiaobin | in a mixed environment, scheduler considers the hypervisor nodes and ironic nodes are the same | 18:10 |
jxiaobin | ok | 18:11 |
jxiaobin | will ironic/nova provide a proper solution? | 18:11 |
jroll | I think host aggregates does solve this, I don't know how they work though | 18:12 |
jxiaobin | say a flavor or image is particularily for BM | 18:12 |
jroll | morgabra: https://github.com/git/git/blob/master/contrib/completion/git-completion.bash | 18:13 |
jxiaobin | thanks jroll! | 18:13 |
jroll | you're welcome :) | 18:13 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add network provider interface and implementations https://review.openstack.org/139687 | 18:16 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add driver and driver_info fields to Port https://review.openstack.org/155848 | 18:16 |
jroll | devananda: ^ | 18:16 |
anderbubble | jroll, 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 |
jroll | clearly not ready yet but yeah | 18:16 |
jroll | anderbubble: they aren't necessary, as far as I know, but rackspace already runs cells for VMs, so we made an ironic cell | 18:17 |
jroll | anderbubble: by default nova will just pick a compute that has resources, you need to use host aggregates or cells to restrict this | 18:17 |
anderbubble | jroll, 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-ironic | 18:21 | |
*** Guest40096 is now known as annegentle | 18:22 | |
*** dtantsur|afk has quit IRC | 18:24 | |
*** achanda has joined #openstack-ironic | 18:25 | |
jlvllal_sfo | JoshNang: ironic]$ tools/states_to_dot.py | 18:25 |
JoshNang | jlvllal_sfo: thanks | 18:25 |
*** dtantsur has joined #openstack-ironic | 18:26 | |
*** tchorizo has left #openstack-ironic | 18:30 | |
*** EmilienM|afk is now known as EmilienM | 18:32 | |
*** andreykurilin_ has quit IRC | 18:32 | |
*** foexle has quit IRC | 18:32 | |
*** Marga_ has quit IRC | 18:32 | |
*** Marga_ has joined #openstack-ironic | 18:33 | |
*** Marga_ has quit IRC | 18:34 | |
*** eghobo has joined #openstack-ironic | 18:35 | |
*** Marga_ has joined #openstack-ironic | 18:35 | |
*** eghobo has quit IRC | 18:35 | |
*** eghobo has joined #openstack-ironic | 18:39 | |
openstackgerrit | Mark Goddard proposed openstack/ironic: Avoid deregistering conductor following SIGUSR1 https://review.openstack.org/155785 | 18:40 |
jroll | anderbubble: I have no idea :| | 18:42 |
*** dprince has quit IRC | 18:45 | |
BadCub | Morning Ironc | 18:46 |
openstackgerrit | John Villalovos proposed openstack/ironic: Add concept of stable states to the state machine https://review.openstack.org/155529 | 18:47 |
jlvllal_sfo | BadCub: Morning | 18:48 |
*** anderbubble has quit IRC | 18:51 | |
JoshNang | jlvllal_sfo: this block of tests was pretty well commented https://github.com/openstack/ironic/blob/master/ironic/tests/conductor/test_manager.py#L1298 | 18:56 |
jlvllal_sfo | JoshNang: Thanks! | 18:56 |
jlvllal_sfo | Do we have anything like this for our REST APIs? http://www.jfrog.com/confluence/display/RTF/Artifactory+REST+API | 18:56 |
*** barra204 has joined #openstack-ironic | 18:57 | |
*** barra204 is now known as shamamunyi | 18:58 | |
*** shamamunyi is now known as shakamunyi | 18:58 | |
jlvllal_sfo | And would something like that be useful for Ironic? | 18:58 |
openstackgerrit | John Villalovos proposed openstack/ironic: Add concept of stable states to the state machine https://review.openstack.org/155529 | 18:59 |
*** lucas-dinner has quit IRC | 19:02 | |
*** dprince has joined #openstack-ironic | 19:02 | |
JayF | martini: devananda: RFR https://github.com/rackerlabs/coreos-image-builder | 19:04 |
*** korekhov has quit IRC | 19:04 | |
JayF | Question 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-ironic | 19:05 | |
jroll | I would import it as-is so people can review the refactor :) | 19:05 |
JayF | That's pretty much what I thought | 19:06 |
devananda | JayF: first glance: lgtm | 19:06 |
JayF | Aight; I'll propose to project-config | 19:06 |
JayF | Should that come with an email to the list as well? | 19:07 |
jroll | yesh | 19:07 |
*** bradjones has quit IRC | 19:07 | |
*** eghobo has quit IRC | 19:08 | |
*** korekhov has quit IRC | 19:09 | |
*** korekhov has joined #openstack-ironic | 19:12 | |
JayF | https://review.openstack.org/#/c/155868/ project-config setup for CIB | 19:16 |
*** Nisha has quit IRC | 19:19 | |
*** Nisha has joined #openstack-ironic | 19:20 | |
*** davideagnello has quit IRC | 19:21 | |
*** davideagnello has joined #openstack-ironic | 19:24 | |
*** davideagnello has quit IRC | 19:31 | |
*** davideagnello has joined #openstack-ironic | 19:32 | |
*** ndipanoff has quit IRC | 19:32 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add driver and driver_info fields to Port https://review.openstack.org/155848 | 19:33 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add network provider interface and implementations https://review.openstack.org/139687 | 19:33 |
*** korekhov has quit IRC | 19:34 | |
*** Haomeng has joined #openstack-ironic | 19:34 | |
*** korekhov has joined #openstack-ironic | 19:35 | |
*** Haomeng|2 has quit IRC | 19:35 | |
*** korekhov has quit IRC | 19:39 | |
*** Marga_ has quit IRC | 19:44 | |
*** Marga_ has joined #openstack-ironic | 19:45 | |
*** Nisha has quit IRC | 19:48 | |
*** Nisha has joined #openstack-ironic | 19:48 | |
*** romcheg has quit IRC | 20:00 | |
*** korekhov has joined #openstack-ironic | 20:01 | |
*** barra204 has joined #openstack-ironic | 20:01 | |
*** shakamunyi has quit IRC | 20:02 | |
jlvllal_sfo | JoshNang: 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 IRC | 20:04 | |
*** Raj_ has joined #openstack-ironic | 20:05 | |
JoshNang | jlvllal_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 soon | 20:06 |
jlvllal_sfo | JoshNang: Cool. Thanks. | 20:06 |
*** dprince has quit IRC | 20:07 | |
*** anderbubble has joined #openstack-ironic | 20:07 | |
*** hemna_ has joined #openstack-ironic | 20:10 | |
*** ndipanoff has joined #openstack-ironic | 20:19 | |
*** barra204 has quit IRC | 20:19 | |
*** romcheg has joined #openstack-ironic | 20:20 | |
*** faizan has joined #openstack-ironic | 20:22 | |
*** mgoddard has quit IRC | 20:22 | |
*** jcoufal has joined #openstack-ironic | 20:22 | |
*** mgoddard has joined #openstack-ironic | 20:24 | |
*** achanda has quit IRC | 20:31 | |
*** ifarkas has quit IRC | 20:31 | |
*** lnxnut has joined #openstack-ironic | 20:33 | |
*** barra204 has joined #openstack-ironic | 20:34 | |
*** romcheg1 has joined #openstack-ironic | 20:34 | |
*** barra204 has quit IRC | 20:35 | |
*** romcheg has quit IRC | 20:35 | |
*** bnemec is now known as beekneemech | 20:37 | |
*** mgoddard has quit IRC | 20:38 | |
*** mgoddard has joined #openstack-ironic | 20:39 | |
*** yginger5 is now known as yginger5_away | 20:39 | |
*** lnxnut has quit IRC | 20:41 | |
*** lnxnut has joined #openstack-ironic | 20:41 | |
*** lnxnut_ has joined #openstack-ironic | 20:43 | |
*** achanda has joined #openstack-ironic | 20:43 | |
*** mgoddard has quit IRC | 20:44 | |
*** lnxnut has quit IRC | 20:46 | |
faizan | devananda: Hi | 20:51 |
faizan | jroll: dtantsur: Hi | 20:53 |
*** martini has quit IRC | 20:58 | |
*** jjohnson2 has joined #openstack-ironic | 21:00 | |
jroll | faizan: hi | 21:03 |
faizan | jroll: regarding "partition image support for agent driver", I have submitted the disk_partition code into oslo-incubator project | 21:17 |
faizan | jroll: We need to give maintainers name for this code in oslo-incubator. Shall I give your name as maintainer? | 21:20 |
JayF | Josh Gachnang or Alex Weeks might be a better choice | 21:24 |
JayF | but they're off to lunch right now and jroll is AFK (we're all at the mid-cycle meetup in SF) | 21:24 |
JayF | so let me check with them and I'll get back | 21:24 |
faizan | JayF: Thanks so much, | 21:25 |
JayF | oh disk_partition | 21:25 |
JayF | sorry I thought that was metrics | 21:25 |
JayF | not jim or josh, lol | 21:25 |
JayF | That should be owned generally by the Ironic team | 21:25 |
JayF | if possible | 21:25 |
JayF | if not, then I'd suggest devananda or someone he explicitly authorizes | 21:26 |
faizan | JayF: ok, can you check with devananda and let me know? | 21:27 |
jroll | faizan: is it already in oslo-incubator | 21:27 |
JayF | jogo is here and suggesting that this should go directly into a new oslo project instead of through the incubator | 21:27 |
jroll | ? | 21:27 |
JayF | faizan: not really, he's gone for the mid-cycle anyway :) | 21:27 |
faizan | JayF: sorry I thought you were also there at mid-cycle | 21:28 |
jroll | faizan: deva left for the day | 21:28 |
*** lnxnut_ has quit IRC | 21:31 | |
faizan | JayF: In the spec I have mentioned that we will put it in oslo-incubator first, and then we can graduate it to oslo project | 21:31 |
*** mgoddard has joined #openstack-ironic | 21:33 | |
jroll | faizan: 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-utils | 21:34 |
*** mgoddard has quit IRC | 21:34 | |
*** hemna_ has quit IRC | 21:35 | |
faizan | jroll: I think it will take a longer cycle to put it in oslo project and may not happen with in kilo release time | 21:37 |
*** hemna_ has joined #openstack-ironic | 21:38 | |
faizan | jroll: 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 release | 21:39 |
jroll | faizan: let's just make an ironic-disk-utils repo then | 21:39 |
jroll | no need to wait for people | 21:40 |
*** david-lyle has quit IRC | 21:42 | |
faizan | jroll: 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-ironic | 21:42 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Automate boot iso creation with in ironic for iscsi-ilo https://review.openstack.org/155900 | 21:43 |
*** lnxnut has joined #openstack-ironic | 21:44 | |
faizan | jroll: 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 IRC | 21:47 | |
*** absubram has quit IRC | 21:52 | |
jroll | faizan: I like libironic or ironiclib | 21:57 |
*** faizan has quit IRC | 21:57 | |
jroll | probably ironiclib | 21:57 |
*** martini has joined #openstack-ironic | 21:57 | |
Shrews | ironiclib seems sensible | 21:57 |
jroll | oh | 21:58 |
jroll | 21:57:22 <-- | faizan (6a33893c@gateway/web/freenode/ip.106.51.137.60) has quit (Ping timeout: 246 seconds) | 21:58 |
*** lnxnut has quit IRC | 21:59 | |
*** lnxnut has joined #openstack-ironic | 22:00 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add states required for node-inspection https://review.openstack.org/147857 | 22:03 |
*** lnxnut has quit IRC | 22:04 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add InspectInterface for node-introspection https://review.openstack.org/149817 | 22:06 |
*** jjohnson2 has quit IRC | 22:06 | |
martini | JayF: https://github.com/PeterMartini/ironic-dib-elements | 22:07 |
JayF | I mean, all the history is gone | 22:11 |
JayF | all the commits are from you? | 22:11 |
JayF | or not... | 22:11 |
JayF | hmm. This authored by vs committed in github is confusing | 22:11 |
* JayF checks the repo he made | 22:11 | |
JayF | Mine: https://github.com/rackerlabs/coreos-image-builder/commits/master doesn't show the commited by / authored by split | 22:11 |
JayF | so you did something differently that changed the history, I think? | 22:12 |
jroll | http://i.imgur.com/Q3U4uIa.gif | 22:16 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Generic changes for Node Inspection https://review.openstack.org/149823 | 22:16 |
martini | Yeah, doing apply patch for lots of different cherry picked pieces | 22:17 |
JayF | you might wanna ask folks in #openstack-infra what they've done in the past | 22:17 |
JayF | when breaking pieces out of multiple repos | 22:17 |
martini | huh. 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 |
martini | ironic-agent worked tho | 22:20 |
*** achanda has quit IRC | 22:21 | |
*** achanda has joined #openstack-ironic | 22:23 | |
* mrda is still at the airport waiting to board the flight back to Australia | 22:23 | |
jroll | O.O | 22:23 |
jroll | hi mrda | 22:23 |
mrda | tldr: 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 |
mrda | But | 22:24 |
*** harlowja_ is now known as harlowja_away | 22:24 | |
mrda | Logical names merged, so \o/ | 22:24 |
JayF | mrda: so Rachael told me just now, apparently mtrenish is in the office today | 22:24 |
JayF | and you missed him :( | 22:24 |
mrda | mattoliverau actually | 22:25 |
JayF | aha | 22:25 |
mrda | wrong matt :) | 22:25 |
mrda | You should go say hello, he's at the hotel desks | 22:25 |
JayF | I was told a Matt from -AU | 22:25 |
mrda | (I think) | 22:25 |
JayF | aight, I shall :) | 22:25 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 22:25 |
mrda | I could have joined you guys for another day! | 22:25 |
martini | nevermind, I do get the same results from doing it in just dib, so I clearly just messed up the command | 22:25 |
*** Raj_ has quit IRC | 22:26 | |
mrda | JayF: also, we had breakfasts and dinenrs together this week, so no big deal | 22:26 |
mrda | jroll, JayF, JoshNang, clif_h, aweeks: Thanks for hosting us this week | 22:27 |
JayF | Yeah :( sad you're stuck at the airport | 22:27 |
jroll | mrda: fo sho :) | 22:27 |
JoshNang | mrda: thanks for coming! | 22:27 |
mrda | Glad to spend some time with y'all | 22:27 |
JayF | General 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 IRC | 22:29 | |
mrda | JayF: that's ok, these things happen | 22:29 |
*** lnxnut has joined #openstack-ironic | 22:30 | |
*** lnxnut has quit IRC | 22:31 | |
Shrews | mrda: OMG | 22:32 |
mrda | Shrews: I'm never travelling again :) | 22:32 |
Shrews | mrda: then obviously we all have to come to you next time | 22:32 |
NobodyCam | mrda: :( | 22:32 |
mrda | Shrews: \o/ | 22:32 |
* mrda should be over my disgruntledness by Vancouver | 22:33 | |
NobodyCam | :) | 22:33 |
* mrda just hopes his checked bag makes it to Australia | 22:33 | |
openstackgerrit | Nisha Agarwal proposed openstack/python-ironicclient: enhanced node-set-provision-state https://review.openstack.org/148804 | 22:36 |
mrda | bye y'all, off to line up at checkin | 22:40 |
*** mrda is now known as mrda-travelling | 22:41 | |
*** Nisha has quit IRC | 22:43 | |
* jlvllal_sfo prepares to go to the airport | 22:43 | |
*** jlvllal_sfo has quit IRC | 22:44 | |
openstackgerrit | Merged openstack/python-ironicclient: Fix Python Ironic Client Documentation https://review.openstack.org/155714 | 22:47 |
*** priteau has quit IRC | 22:50 | |
*** martini has quit IRC | 22:51 | |
*** jjohnson2 has joined #openstack-ironic | 22:52 | |
*** yginger5_away is now known as yjiang5 | 23:01 | |
clif_h | mrda-travelling: glad to meet you, have a safe trip | 23:02 |
*** r-daneel has quit IRC | 23:06 | |
*** Haomeng has quit IRC | 23:08 | |
*** zigo has quit IRC | 23:27 | |
*** zigo has joined #openstack-ironic | 23:28 | |
*** jjohnson2 has quit IRC | 23:31 | |
*** anderbubble has quit IRC | 23:37 | |
*** hemna_ is now known as hemnafk | 23:42 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!