*** spandhe has quit IRC | 00:01 | |
*** naohirot has joined #openstack-ironic | 00:03 | |
*** spandhe has joined #openstack-ironic | 00:03 | |
devananda | hrm. I didn't review the code when it landed in juno ... https://review.openstack.org/#/c/90151/ | 00:05 |
---|---|---|
*** Haomeng|2 has joined #openstack-ironic | 00:05 | |
*** Haomeng has quit IRC | 00:06 | |
openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Add scripts to manage translations https://review.openstack.org/158981 | 00:08 |
NobodyCam | ugg I landed that :( | 00:13 |
*** jlvillal_ has joined #openstack-ironic | 00:15 | |
*** ijw_ has quit IRC | 00:16 | |
*** jlvillal_ has left #openstack-ironic | 00:23 | |
devananda | https://bugs.launchpad.net/ironic/+bug/1427923 | 00:25 |
openstack | Launchpad bug 1427923 in Ironic "boot device API blocks while waiting on the BMC" [Medium,Confirmed] | 00:25 |
* NobodyCam steps afk for a few | 00:41 | |
*** dmellado has quit IRC | 00:44 | |
*** BadCub has quit IRC | 00:44 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Make set-boot-device asynchronous https://review.openstack.org/161046 | 00:44 |
*** dmellado has joined #openstack-ironic | 00:48 | |
*** Haomeng has joined #openstack-ironic | 00:49 | |
*** Haomeng|2 has quit IRC | 00:50 | |
*** mtanino has quit IRC | 00:50 | |
NobodyCam | devananda: last I checked the gate is running 3 1/2 to 4 hours for checks today | 00:50 |
devananda | ouch | 00:54 |
*** jlvillal_pdx has quit IRC | 00:58 | |
rloo_afk | devananda: wrt https://review.openstack.org/#/c/90151/, I did ask about the sync thingy in one of my comments. Looks like it was mentioned as sync in the spec too :-( Hard to know/remember what is 'set in stone' and what isn't. Sorry... | 01:00 |
*** rloo_afk has quit IRC | 01:01 | |
*** Marga_ has joined #openstack-ironic | 01:07 | |
* devananda goes afk for dinner & such | 01:13 | |
*** jerryz_ has joined #openstack-ironic | 01:32 | |
*** Marga_ has quit IRC | 01:33 | |
*** achanda has quit IRC | 01:35 | |
*** achanda has joined #openstack-ironic | 01:41 | |
*** wshao has joined #openstack-ironic | 01:41 | |
*** Marga_ has joined #openstack-ironic | 01:45 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Add cleaning/zapping support to IPA https://review.openstack.org/161066 | 01:51 |
*** chenglch has joined #openstack-ironic | 01:53 | |
*** ChuckC has quit IRC | 02:04 | |
*** Haomeng|2 has joined #openstack-ironic | 02:18 | |
*** Haomeng has quit IRC | 02:18 | |
*** Marga_ has quit IRC | 02:21 | |
*** achanda has quit IRC | 02:25 | |
*** achanda has joined #openstack-ironic | 02:33 | |
openstackgerrit | Haomeng,Wang proposed openstack/ironic: raises exception if can not get uuid of root filesystem https://review.openstack.org/143919 | 02:41 |
*** ChuckC has joined #openstack-ironic | 02:49 | |
*** ChuckC has quit IRC | 02:54 | |
*** ChuckC has joined #openstack-ironic | 02:54 | |
openstackgerrit | Haomeng,Wang proposed openstack/ironic: Add tftp mapfile configuration in install-guide https://review.openstack.org/161074 | 03:02 |
*** achanda has quit IRC | 03:05 | |
*** ChuckC has quit IRC | 03:06 | |
*** ChuckC has joined #openstack-ironic | 03:06 | |
*** wshao has quit IRC | 03:06 | |
*** Haomeng|2 has quit IRC | 03:09 | |
*** achanda has joined #openstack-ironic | 03:10 | |
*** spandhe has quit IRC | 03:12 | |
*** harlowja_ is now known as harlowja_away | 03:21 | |
lintan | Hi NobodyCam, | 03:23 |
NobodyCam | hi lintan | 03:24 |
NobodyCam | :) | 03:24 |
lintan | Hi NobodyCam, I see you call me some time before, what can I do for you :) | 03:25 |
NobodyCam | lintan: I had a question on 145690, is that any admin user with multi-networks? | 03:25 |
*** green_ has joined #openstack-ironic | 03:26 | |
lintan | NobodyCam, yes, we create two networks following the guide | 03:27 |
NobodyCam | :) I'd really like to see that added to the note on 145690 | 03:28 |
lintan | NobodyCam, OK for sure | 03:28 |
NobodyCam | I agree with haomeng commented on it and then removed his -1 | 03:28 |
NobodyCam | :) | 03:28 |
NobodyCam | i'll +2 | 03:29 |
NobodyCam | thank you lintan | 03:29 |
lintan | NobodyCam, Thanks, btw, I update AMT patch with your comments, please have a look if you have time :) | 03:30 |
*** achanda has quit IRC | 03:30 | |
NobodyCam | #? | 03:30 |
*** kkoski has joined #openstack-ironic | 03:30 | |
lintan | NobodyCam, #152858 | 03:31 |
mrda | hi lintan | 03:40 |
mrda | We had discussions on microversions earlier today here in IRC | 03:41 |
lintan | hi mrda, I see the log, it's very helpful | 03:41 |
mrda | I'm putting together a spec to detail what behaviour we're all agreeing to. | 03:41 |
mrda | I'll ping you when it's done for your comments. | 03:41 |
mrda | (and devananda and rloo) | 03:41 |
lintan | mrda, cool, it's great. | 03:42 |
*** Haomeng has joined #openstack-ironic | 03:42 | |
mrda | I think the path we're on is fine, except defaulting to latest, rather than earliest | 03:42 |
mrda | as per deva's comment | 03:42 |
lintan | mrda, yes, I think we need more agreement on that, | 03:43 |
mrda | agreed | 03:44 |
openstackgerrit | Tan Lin proposed openstack/ironic: Add a note to dev-quickstart https://review.openstack.org/145690 | 03:46 |
Haomeng | lintan: got your new patch, thanks | 03:50 |
lintan | Haomeng, welcome back from Chinese new year :) | 03:51 |
*** achanda has joined #openstack-ironic | 03:55 | |
*** kkoski has quit IRC | 04:13 | |
*** pradipta has joined #openstack-ironic | 04:14 | |
mrda | I'm trying to find the original code that landed microversions in ironic itself. Does anyone have a bug number or review? | 04:15 |
* mrda can't find it | 04:15 | |
*** adam_g has quit IRC | 04:20 | |
Haomeng | lintan: same to you:) | 04:22 |
Haomeng | lintan: :) | 04:23 |
lintan | Hi Haomeng | 04:23 |
lintan | mrda: https://review.openstack.org/#/c/150821/ this is what you need | 04:23 |
mrda | thanks lintan - was scrolling back in commit history trying to find it :-S | 04:24 |
* mrda just realises why he never saw it - he was on leave during the whole review timeframe of that patch | 04:26 | |
*** adam_g has joined #openstack-ironic | 04:28 | |
*** adam_g has quit IRC | 04:28 | |
*** adam_g has joined #openstack-ironic | 04:28 | |
*** mgagne has quit IRC | 04:31 | |
*** rameshg87 has joined #openstack-ironic | 04:34 | |
rameshg87 | good morning ironic | 04:35 |
mrda | hi rameshg87 | 04:36 |
rameshg87 | hello mrda | 04:36 |
*** lazy_prince has quit IRC | 04:38 | |
Haomeng | rameshg87: morning:) | 04:42 |
*** mgagne has joined #openstack-ironic | 04:45 | |
*** mgagne is now known as Guest75711 | 04:45 | |
*** killer_prince has joined #openstack-ironic | 04:46 | |
*** killer_prince is now known as lazy_prince | 04:46 | |
*** clif_h has quit IRC | 04:46 | |
*** openstackgerrit has quit IRC | 04:46 | |
rameshg87 | hello Haomeng | 04:47 |
rameshg87 | Haomeng, after long vacation ? :) | 04:47 |
Haomeng | rameshg87: yes:) | 04:48 |
Haomeng | rameshg87: :) | 04:48 |
*** openstackgerrit has joined #openstack-ironic | 04:53 | |
*** Marga_ has joined #openstack-ironic | 04:57 | |
*** comstud has quit IRC | 05:00 | |
*** achanda has quit IRC | 05:28 | |
*** sandhya has joined #openstack-ironic | 05:28 | |
sandhya | NobodyCam, devananda - I have pushed code for Chassis Discovery Tool as we discussed in the irc meeting - https://review.openstack.org/#/c/158577/. Kindly review and give your feedback. We could also reach on an agreement... | 05:30 |
*** openstack has joined #openstack-ironic | 05:36 | |
*** jlvllal_rem has left #openstack-ironic | 05:47 | |
Haomeng | sandhya: thank you | 05:50 |
Haomeng | sandhya: your ' Chassis Level Discovery Tool' is good starting, great tool! Have a quick review and left some comments inline. | 05:50 |
sandhya | Haomeng: Thank you. Just going though your comments. I will work on them. | 05:51 |
*** Haomeng|2 has joined #openstack-ironic | 05:53 | |
*** Haomeng has quit IRC | 05:53 | |
Haomeng|2 | sandhya: and can we add the execute script chassis_discovery_tool.py this into setup.cfg as a new console_scripts? | 05:54 |
Haomeng|2 | sandhya: in entry_points section | 05:55 |
Haomeng|2 | sandhya: maybe this is not ironic script, so it is a seperated tool, make sense | 06:01 |
sandhya | Haomeng: entry points in setup.cfg is meant for the plugin discovery mechanism that is to be loaded when the tool is called with it. What is your thought process behind adding the tool script as one of the entry points? | 06:02 |
Haomeng|2 | sandhya: yes, make sense, so ignore my last comments:) | 06:06 |
Haomeng|2 | sandhya: :) | 06:06 |
sandhya | Haomeng: Thank you! :) | 06:06 |
Haomeng|2 | sandhya: welcome:) | 06:06 |
Haomeng|2 | sandhya: :) | 06:06 |
*** achanda has joined #openstack-ironic | 06:16 | |
*** JayF has quit IRC | 06:17 | |
*** pcrews has quit IRC | 06:19 | |
*** yog_ has joined #openstack-ironic | 06:19 | |
*** gridinv has quit IRC | 06:40 | |
*** erwan_taf has quit IRC | 06:42 | |
openstackgerrit | Michael Davies proposed openstack/ironic-specs: API Microversions https://review.openstack.org/161110 | 06:46 |
mrda | lintan, rloo, devananda: https://review.openstack.org/#/c/161110 <--- API Microversions spec to capture our discussions and to gain consensus. | 06:46 |
*** jogo has quit IRC | 06:53 | |
openstackgerrit | sandhya proposed openstack/ironic: Implement Chassis Level Discovery Tool https://review.openstack.org/158577 | 07:02 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Follow-up patch for generic node inspection https://review.openstack.org/160665 | 07:04 |
*** Nisha has joined #openstack-ironic | 07:08 | |
*** pcrews has joined #openstack-ironic | 07:09 | |
*** ukalifon1 has joined #openstack-ironic | 07:18 | |
*** david-lyle_afk has joined #openstack-ironic | 07:33 | |
*** stendulker has joined #openstack-ironic | 07:36 | |
*** saripurigopi has joined #openstack-ironic | 07:38 | |
*** Marga_ has quit IRC | 07:40 | |
*** jcoufal has joined #openstack-ironic | 07:41 | |
*** openstackgerrit has quit IRC | 07:49 | |
*** openstackgerrit has joined #openstack-ironic | 07:50 | |
*** Marga_ has joined #openstack-ironic | 07:50 | |
*** dlpartain has joined #openstack-ironic | 07:53 | |
dtantsur | Morning! | 08:05 |
lintan | Morning dtantsur :) | 08:07 |
stendulker | Morning dtantsur :) | 08:07 |
*** Marga_ has quit IRC | 08:11 | |
*** pas-ha has joined #openstack-ironic | 08:11 | |
*** achanda has quit IRC | 08:12 | |
*** ifarkas has joined #openstack-ironic | 08:15 | |
*** chlong has quit IRC | 08:20 | |
*** erwan_taf has joined #openstack-ironic | 08:26 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: Add support for sharing physical disks https://review.openstack.org/160385 | 08:28 |
-openstackstatus- NOTICE: Zuul check queue stuck due to reboot maintenance window at one of our cloud providers - no need to recheck changes at the moment, they won't move forward. | 08:30 | |
*** ChanServ changes topic to "Zuul check queue stuck due to reboot maintenance window at one of our cloud providers - no need to recheck changes at the moment, they won't move forward." | 08:30 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Add server-supported min and max API version to HTTPNotAcceptable(406) https://review.openstack.org/160758 | 08:32 |
dtantsur | Haomeng|2, hi, still around? Mind reviewing https://review.openstack.org/#/c/159100/ ? | 08:37 |
*** pradipta has quit IRC | 08:39 | |
Haomeng|2 | dtantsur: yes, sure, reviewed and +a, thanks Dmitry to add such useful common method:) | 08:46 |
dtantsur | thank you! | 08:46 |
Haomeng|2 | dtantsur: welcome:) | 08:46 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Start using the in-band inspection https://review.openstack.org/161132 | 08:48 |
Haomeng|2 | dtantsur: but the jenkins run slowly today | 08:49 |
Haomeng|2 | dtantsur: maybe jenkins env issue | 08:49 |
dtantsur | Haomeng|2, gate is OK, as I understood, check job do not work - see this channel title | 08:49 |
Haomeng|2 | dtantsur: ok | 08:49 |
*** jistr has joined #openstack-ironic | 08:50 | |
*** achanda has joined #openstack-ironic | 08:52 | |
*** dlpartain has left #openstack-ironic | 08:54 | |
*** lucasagomes has joined #openstack-ironic | 08:57 | |
naohirot | good morning all | 08:58 |
naohirot | rameshg87: can I ask a question about agent_ilo? | 08:59 |
naohirot | dtantsur: Haomeng|2: In case of agent deploy, where should I put user boot image, not deploy image ? | 09:03 |
*** rainya has quit IRC | 09:03 | |
rameshg87 | naohirot, sure | 09:03 |
openstackgerrit | Merged stackforge/ironic-discoverd: Add scripts to manage translations https://review.openstack.org/158981 | 09:03 |
rameshg87 | naohirot, was that the same question to me as well ? | 09:04 |
rameshg87 | naohirot, if so, you can put in glance, but agent has requirement that glance should be backed by swift | 09:04 |
naohirot | dtantsur: Haomeng|2: I couldn't find any document which specifically mentions where the boot image should be put. | 09:04 |
naohirot | rameshg87: Hi, yes. it is same question. | 09:04 |
rameshg87 | naohirot, i assumed you meant boot image = <image to be deployed> | 09:05 |
naohirot | naohirot: really, the symptom currently I'm having is that agent deploy coreos image works fine | 09:06 |
naohirot | rameshg87: but conductor cannot find user boot image. | 09:07 |
naohirot | rameshg87: the error is at | 09:07 |
*** openstack has joined #openstack-ironic | 15:28 | |
-sendak.freenode.net- [freenode-info] channel flooding and no channel staff around to help? Please check with freenode support: http://freenode.net/faq.shtml#gettinghelp | 15:28 | |
Kinnison | devananda: I've been working through that, currently branched off trying to learn about keystone | 15:29 |
devananda | vdrok: we need to still test glance images, so changing test_baremetal_basic_ops is not the right way to go. a new test would be better | 15:29 |
devananda | vdrok: it should still use Nova, though. just not glance | 15:29 |
Kinnison | sadly I need to pop off now for a doctor's appointment, but I'll be back a little later and raring to learn more :) | 15:29 |
devananda | Kinnison: great. you're on the right track | 15:29 |
vdrok | devananda, hm, but nova boot requires glance image_source? | 15:29 |
vdrok | devananda, i mean --image parameter | 15:30 |
devananda | oh. right :-/ | 15:30 |
vdrok | devananda, so we should either have a new test without nova (which will take another ~10 minutes), or we can change the one that we have (but then it won't be fully tested, only deploy images will be non-glance) | 15:32 |
*** stendulker has joined #openstack-ironic | 15:33 | |
vdrok | devananda, which you think is better? | 15:33 |
vdrok | devananda, or are there other options? | 15:33 |
*** saripurigopi has joined #openstack-ironic | 15:33 | |
devananda | vdrok: so we definitely need to continue testing with nova + glance + neutron. dropping that from our test is not realy an option | 15:35 |
* rameshg87 goes to sleep | 15:36 | |
rameshg87 | good night ironic | 15:36 |
devananda | vdrok: when you say "new test" do you mean a new jenkins job, or just a new tempest scenario? | 15:36 |
vdrok | night rameshg87 | 15:36 |
*** rameshg87 has quit IRC | 15:37 | |
vdrok | devananda, new scenario | 15:37 |
NobodyCam | night ramshg87 | 15:37 |
vdrok | the one like test_baremetal_basic_ops but using ironic api rather then nova | 15:38 |
vdrok | morning NobodyCam :) | 15:38 |
NobodyCam | mornign vdrok :) | 15:39 |
devananda | vdrok: that should be fine | 15:41 |
devananda | vdrok: so another component of this -- we should start thinking about moving our functional tests into ironic, and using tempest-lib | 15:42 |
vdrok | devananda, ok, i'll look into it in more detail this week, never written tempest tests before | 15:44 |
vdrok | devananda, if i'll understand how to do it in ironic with tempest-lib i'll do it :) | 15:45 |
*** clif has joined #openstack-ironic | 15:46 | |
*** clif is now known as clif_h | 15:46 | |
devananda | vdrok: great, thanks! don't stress over the tempest-lib stuff yet - it's new and we aren't using it at all at the moment. I think you'll have plenty to do just creating a "test_baremetal_standalone_ops" test | 15:48 |
* dtantsur researches why raising IronicException in validate() causes error 500 | 15:51 | |
devananda | vdrok: you also might ask Shrews for advice if you run into challenges writing the tempest test | 15:52 |
vdrok | devananda, ok, thanks :) | 15:52 |
* Shrews hides | 15:52 | |
vdrok | morning Shrews :) | 15:52 |
Shrews | hello vdrok | 15:53 |
vdrok | no challenges at the moment :) | 15:53 |
*** anderbubble has joined #openstack-ironic | 15:54 | |
*** ChuckC_ has joined #openstack-ironic | 15:54 | |
NobodyCam | morning Shrews :) | 15:55 |
*** rainya has joined #openstack-ironic | 15:55 | |
BadCub | mornin Shrews | 15:55 |
Shrews | hey hey NobodyCam & BadCub | 15:56 |
NobodyCam | :) | 15:56 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix nits for supporting non-glance images https://review.openstack.org/160732 | 15:57 |
* BadCub has to run to conference call | 15:58 | |
*** gridinv has joined #openstack-ironic | 16:00 | |
*** hj-hp has joined #openstack-ironic | 16:00 | |
*** saripurigopi has quit IRC | 16:01 | |
*** mgoddard has joined #openstack-ironic | 16:03 | |
*** gridinv has quit IRC | 16:05 | |
*** ukalifon1 has quit IRC | 16:06 | |
*** saripurigopi has joined #openstack-ironic | 16:07 | |
*** jerryz_ has quit IRC | 16:11 | |
*** absubram has joined #openstack-ironic | 16:15 | |
*** yjiang5 has quit IRC | 16:15 | |
*** romcheg has quit IRC | 16:15 | |
*** gridinv has joined #openstack-ironic | 16:16 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add module for in-band inspection using ironic-discoverd https://review.openstack.org/156562 | 16:17 |
*** gridinv has quit IRC | 16:22 | |
*** ParsectiX has joined #openstack-ironic | 16:28 | |
*** stendulker has quit IRC | 16:28 | |
*** saripurigopi has quit IRC | 16:30 | |
*** vdrok is now known as vdrok_afk | 16:38 | |
*** rwsu-afk is now known as rwsu | 16:42 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add the image extension (for local boot) https://review.openstack.org/157469 | 16:43 |
*** mrda has quit IRC | 16:43 | |
*** jlvillal_pdx has joined #openstack-ironic | 16:47 | |
*** jlvillal_pdx is now known as jlvillal_train | 16:48 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add python-openstackclient plugin for ironic-discoverd https://review.openstack.org/157448 | 16:48 |
Kinnison | devananda: Okay, so I'm starting to get a handle on the vocabulary for OpenStack (and wow isn't there a lot of it) -- If I wanted to have a go at persuading Ironic to talk to a Moonshot chassis would I need trunk/Kilo or would whatever is in the HP Cloud OS on these M400 cartridges be likely capable of it? | 16:48 |
*** anderbubble has quit IRC | 16:48 | |
dtantsur | trown, https://review.openstack.org/157448 seems to work now | 16:49 |
trown | dtantsur: sweet, I will give it a try in a bit | 16:50 |
dtantsur | thanks! | 16:51 |
dtantsur | and now I go home :) | 16:51 |
dtantsur | see you all tomorrow | 16:51 |
*** dtantsur is now known as dtantsur|afk | 16:51 | |
NobodyCam | night dtantsur|afk | 16:51 |
trown | have a good night | 16:51 |
jroll | morning everybody :) | 16:52 |
NobodyCam | mornign jroll :) | 16:52 |
*** erwan_taf has quit IRC | 16:54 | |
*** erwan_taf has joined #openstack-ironic | 16:54 | |
*** anderbubble has joined #openstack-ironic | 16:55 | |
devananda | Kinnison: trunk/kilo should(tm) work. as far as CloudOS is concerned, I can't say - you'll need to go through HP support for that | 16:57 |
Kinnison | fair enough | 16:57 |
Kinnison | I suppose I can fire up a VM, pop keystone etc onto it, and point it at the chassis | 16:57 |
Kinnison | I'll have to try and do that tomorrow | 16:58 |
Kinnison | Do you have any other hints for making things work well for the moonshot? | 16:58 |
*** mgoddard1 has joined #openstack-ironic | 16:59 | |
*** mgoddard has quit IRC | 17:00 | |
*** tonyb has quit IRC | 17:03 | |
*** mikal has quit IRC | 17:03 | |
devananda | lucasagomes: ping | 17:06 |
lucasagomes | devananda, pong | 17:06 |
devananda | lucasagomes: have a couple minutes to help TheJulia and I debug some ipxe things? | 17:07 |
lucasagomes | devananda, yeah ofc | 17:07 |
lucasagomes | TheJulia, hi there | 17:07 |
devananda | lucasagomes: hangout? | 17:08 |
NobodyCam | hummm should our unit test actualy create files: https://github.com/openstack/ironic/blob/master/ironic/tests/drivers/test_ipmitool.py#L226-L240 | 17:08 |
lucasagomes | devananda, right, can you give me 15 min then? | 17:09 |
devananda | lucasagomes: sure. we're having trouble with the chain to ipxe not working, and hte node just hits tftp indefinitely... | 17:11 |
*** Hefeweizen has quit IRC | 17:11 | |
*** kkoski has joined #openstack-ironic | 17:12 | |
lucasagomes | devananda, right hangout :) | 17:12 |
lucasagomes | I was in a call but its finished now | 17:12 |
*** jistr has quit IRC | 17:13 | |
*** hj-hp has quit IRC | 17:15 | |
-openstackstatus- NOTICE: Issue solved, gate slowly digesting accumulated changes | 17:15 | |
*** kkoski has quit IRC | 17:16 | |
openstackgerrit | Sirushti Murugesan proposed openstack/ironic: Adds support for deploying whole disk images https://review.openstack.org/150142 | 17:16 |
*** hj-hp has joined #openstack-ironic | 17:22 | |
lucasagomes | devananda, TheJulia https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L408-L412 | 17:22 |
*** BadCub is now known as BadCub_Breakfast | 17:23 | |
devananda | BadCub_Breakfast: still wnt to talk soon? | 17:23 |
lucasagomes | devananda, TheJulia and then you update the dhcp server with that info: https://github.com/openstack/ironic/blob/master/ironic/common/pxe_utils.py#L233-L242 | 17:23 |
BadCub_Breakfast | devananda yeah, gonna choke down breakfast real fast | 17:24 |
devananda | lucasagomes: so we're not running neutron here | 17:24 |
jroll | oh | 17:25 |
jroll | devananda: want dhcp configs for ipxe stuff? | 17:25 |
devananda | yup | 17:25 |
devananda | jroll: static dhcp | 17:25 |
lucasagomes | devananda, sure, but it's just to show that the DHCP server has to point to it | 17:25 |
lucasagomes | even if it's static | 17:25 |
jroll | devananda: yeah, we have them | 17:25 |
jroll | sec | 17:25 |
devananda | lucasagomes: ah, gotcha | 17:25 |
jroll | devananda: https://gist.github.com/jimrollenhagen/e09f06f42fe980b9f5fe | 17:28 |
jroll | something like that | 17:28 |
jroll | is what we do | 17:28 |
jroll | this is isc-dhcp-server if that matters | 17:28 |
devananda | TheJulia: ^ | 17:29 |
devananda | jroll: cheers | 17:29 |
* jroll would love to open source all the config management | 17:29 | |
jroll | np | 17:29 |
lucasagomes | jroll, +1 :) | 17:29 |
lucasagomes | jroll, and good morning :D | 17:30 |
jroll | morning lucasagomes :D | 17:30 |
lucasagomes | nice that isc allows you to do it user-class = "iPXE" | 17:30 |
lucasagomes | instead of using !175 | 17:30 |
lucasagomes | and other weird flags | 17:30 |
jroll | lol yeah | 17:30 |
*** killer_prince has quit IRC | 17:31 | |
*** BadCub_Breakfast is now known as BadCub | 17:32 | |
*** killer_prince has joined #openstack-ironic | 17:34 | |
*** killer_prince is now known as lazy_prince | 17:34 | |
*** romcheg has joined #openstack-ironic | 17:35 | |
TheJulia | lucasagomes: Thank you for the help! getting much closer now :) | 17:36 |
lucasagomes | TheJulia, you're very welcome | 17:37 |
BadCub | devananda I am back now. | 17:38 |
*** ndipanov has quit IRC | 17:38 | |
openstackgerrit | Merged openstack/ironic-python-agent: Fix incorrect IPA log message https://review.openstack.org/160977 | 17:39 |
*** kkoski has joined #openstack-ironic | 17:39 | |
devananda | BadCub: hangout? | 17:41 |
BadCub | sure! | 17:41 |
*** spandhe has joined #openstack-ironic | 17:44 | |
*** ijw has joined #openstack-ironic | 17:44 | |
TheJulia | lucasagomes: Quick question, should the deploy config have hard coded paths to the files on local disk if we want them to load via http? I'm guessing not, but wanted to make sure first | 17:44 |
*** kkoski has quit IRC | 17:45 | |
*** Marga_ has joined #openstack-ironic | 17:46 | |
lucasagomes | TheJulia, the files you mean the deploy ramdisk and kernel? | 17:46 |
TheJulia | yes | 17:46 |
lucasagomes | no, you should really point to a URL | 17:47 |
lucasagomes | where you can download the images from | 17:47 |
TheJulia | lucasagomes: thats what I figured, thanks! | 17:47 |
lucasagomes | TheJulia, no problem | 17:47 |
jroll | TheJulia: if you're using static dhcp, those should be ignored anyway | 17:48 |
*** jcoufal_ has quit IRC | 17:48 | |
*** kkoski has joined #openstack-ironic | 17:48 | |
jroll | (the node.driver_info things) | 17:48 |
*** mgoddard1 has quit IRC | 17:51 | |
*** mgoddard has joined #openstack-ironic | 17:53 | |
*** achanda has joined #openstack-ironic | 17:54 | |
*** a1exhughe5 has quit IRC | 18:00 | |
*** gridinv has joined #openstack-ironic | 18:00 | |
*** kkoski has quit IRC | 18:01 | |
*** hj-hp has quit IRC | 18:02 | |
*** anderbubble has quit IRC | 18:02 | |
*** hj-hp has joined #openstack-ironic | 18:04 | |
*** ijw_ has joined #openstack-ironic | 18:08 | |
devananda | JoshNang: hi! any more work left to do for https://blueprints.launchpad.net/ironic/+spec/implement-zapping-states ? | 18:10 |
*** igordcard has quit IRC | 18:11 | |
*** ijw has quit IRC | 18:11 | |
lucasagomes | https://review.openstack.org/#/c/159178/ is now merged! | 18:11 |
JoshNang | devananda: hi! yeah, it still needs the API endpoint. on my list for today | 18:11 |
devananda | JoshNang: k, thanks | 18:11 |
lucasagomes | devananda, ^ we need to create a job in the gate to test the IPA + PXE. You know where I can do that? (never done it before) | 18:12 |
devananda | lucasagomes: yea... that would require a different devstack run, so i think it needs a new jenkins job with slightly different config, but the same test_baremetal_basic_ops tempest test | 18:13 |
devananda | also I thought we already had a test for that? jroll ^ ? | 18:13 |
lucasagomes | devananda, yeah, just need to export one flag to devstack | 18:13 |
lucasagomes | devananda, oh this is about testing IPA ramdisk with the pxe_* drivers via iscsi | 18:14 |
lucasagomes | deploying with iscsi* | 18:14 |
devananda | ah right | 18:14 |
jroll | yeah, what he said | 18:14 |
lucasagomes | we got all the basics merged, in ipa, ironic and now devstack | 18:14 |
JoshNang | devananda: but everyone trying to base code on zapping should have all the code they need (in ironic and IPA) | 18:14 |
devananda | lucasagomes: that's related to this BP, right? https://blueprints.launchpad.net/ironic/+spec/partition-image-support-for-agent-driver | 18:14 |
devananda | lucasagomes: or were you tracking it somewhere else? | 18:15 |
lucasagomes | devananda, well, I think that bp is more about agent_ drivers being able to deploy partition images | 18:15 |
lucasagomes | I was mostly trying to do ipa as default ramdisk | 18:15 |
lucasagomes | devananda, I don't have a bp :( just a topic I put on the patches ipa-as-default-ramdisk | 18:16 |
devananda | lucasagomes: hm. well, this lets IPA deploy a partition image -- albeit using iscsi, rather than local | 18:16 |
lucasagomes | maybe I should put a bp up for that | 18:16 |
devananda | lucasagomes: heh... | 18:16 |
lucasagomes | devananda, I didn't because I thought we already have agreed on having IPA as default ramdisk | 18:17 |
lucasagomes | so I just coded :) | 18:17 |
lucasagomes | but I will put a bp to track the work | 18:17 |
lucasagomes | and link the patches already merged there | 18:17 |
* lucasagomes writes it down | 18:17 | |
devananda | lucasagomes: so you just implemented the "Alternative" here V | 18:18 |
devananda | http://git.openstack.org/cgit/openstack/ironic-specs/plain/specs/kilo/partition-image-support-for-agent-driver.rst | 18:18 |
lucasagomes | devananda, pretty much | 18:18 |
lucasagomes | hah | 18:18 |
devananda | so, um ... | 18:18 |
*** logan2 has quit IRC | 18:18 | |
devananda | we approved a spec and said "here's the alternative" ... and then we did THAT ? | 18:18 |
lucasagomes | I see advantages on having both methods tho | 18:18 |
devananda | LOL | 18:18 |
lucasagomes | not really an "alternative" | 18:18 |
lucasagomes | if you want to support IPA ramdisk with no swift | 18:18 |
lucasagomes | you can do it with iscsi | 18:18 |
devananda | right | 18:19 |
lucasagomes | if you want to have a more distributed way of deploying ur images (not depending on the conductor to partition it for u etc) | 18:19 |
lucasagomes | you can use agent | 18:19 |
*** harlowja_away is now known as harlowja_ | 18:19 | |
devananda | ok - so syed has proposed the library for splitting out the partition manipulation, but it seems like there's no tmuch progress there | 18:19 |
devananda | JayF: you were working on that too i think? | 18:20 |
devananda | trying to update the status of the BP | 18:20 |
lucasagomes | devananda, yeah, as a context. We want to have IPA for our stuff downstream, but we can't depend on swift | 18:20 |
devananda | lucasagomes: if you could toss up a BP to describe what you've done, it'd help in tracking the release ... | 18:20 |
lucasagomes | so I had to do it anyway | 18:20 |
lucasagomes | and thought about putting it upstream | 18:20 |
devananda | lucasagomes: totally agree. I want iscsi support in IPA for all the reasons. it's great | 18:21 |
devananda | I just realized the apparent conflict -- I dont think we've done this before | 18:21 |
openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is writable https://review.openstack.org/160383 | 18:21 |
devananda | where we approve a spec, then go implement the Alternative first | 18:21 |
lucasagomes | devananda, a bp is on the way :) | 18:21 |
devananda | cheers | 18:21 |
devananda | ping BadCub - he'll update the tracking things | 18:21 |
BadCub | lucasagomes would you mind pinging me when you have the bp up for that? | 18:21 |
lucasagomes | hah yeah I think it was an exception case indeed | 18:21 |
lucasagomes | BadCub, sure | 18:21 |
BadCub | :-) | 18:22 |
lucasagomes | devananda, and the origional question, you think we should create a gate job to test it? | 18:23 |
devananda | yes | 18:23 |
lucasagomes | cool | 18:23 |
* lucasagomes looks how to do it | 18:23 | |
lucasagomes | the beauty of it, is that pxe_* drivers works with both ramdisks | 18:23 |
lucasagomes | just a drop in replacement, you don't need any extra config anywhere | 18:23 |
devananda | exactly | 18:24 |
lucasagomes | the pxe configs etc work with both | 18:24 |
*** lazy_prince has quit IRC | 18:27 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is writable https://review.openstack.org/160383 | 18:27 |
devananda | NobodyCam: have you looked at jmankovich's specs on ceilometer integration lately? https://review.openstack.org/#/q/owner:%22Jim+Mankovich+%253Cjmank%2540hp.com%253E%22,n,z | 18:32 |
NobodyCam | I have not ... but will now! | 18:32 |
devananda | NobodyCam: they seem to be stalled waiting on reviews -- and on some contention with ceilometer design (or they were stuck onthat a while ago) | 18:33 |
devananda | NobodyCam: if you can help sort that out, that'd be great. it seems like we (ironic spec cores) haven't given it much attention all cycle | 18:33 |
NobodyCam | :( | 18:33 |
NobodyCam | devananda: do we have hopes that the ceilometer spec will also land: https://review.openstack.org/#/c/130359 | 18:37 |
*** ijw has joined #openstack-ironic | 18:38 | |
*** killer_prince has joined #openstack-ironic | 18:39 | |
*** killer_prince is now known as lazy_prince | 18:39 | |
*** ijw_ has quit IRC | 18:39 | |
*** ChuckC_ has quit IRC | 18:46 | |
lucasagomes | devananda, BadCub https://blueprints.launchpad.net/ironic/+spec/ipa-as-default-ramdisk (will now add the already merged specs | 18:46 |
lucasagomes | s/specs/patches | 18:46 |
devananda | NobodyCam: I have hopes that you can help answer that question :) | 18:46 |
devananda | lucasagomes: cheers! | 18:47 |
adam_g | lucasagomes, was removing rootfstype=ramfs required @ https://review.openstack.org/#/c/155728/16/ironic/tests/drivers/pxe_config.template ? | 18:48 |
BadCub | gotcha covered lucasagomes :-) | 18:49 |
lucasagomes | adam_g, yes, because ipa uses tmpfs | 18:50 |
lucasagomes | adam_g, now. You can pass rootfstype=ramfs as extra pxe config | 18:50 |
lucasagomes | for the parallel tests | 18:50 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: PXE driver: Deprecate pxe_deploy_{ramdisk, kernel} https://review.openstack.org/158644 | 18:50 |
lucasagomes | since we can't have more than 512MB ram per machine on those tests | 18:50 |
* devananda updates the commit message | 18:50 | |
lucasagomes | BadCub, thanks :D | 18:51 |
adam_g | lucasagomes, ack, okay | 18:51 |
BadCub | yw :-d | 18:51 |
jroll | devananda: can you review the direction on https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:bp/network-provider,n,z | 18:57 |
* devananda looks | 18:57 | |
jroll | it isn't done but I'd like to know if there's things you hate about it | 18:58 |
jroll | (should be close though, maybe a day of work) | 18:58 |
lucasagomes | folks I will call it a day | 18:58 |
lucasagomes | have a good night everyone! see y'all tomorrow | 18:58 |
BadCub | have a good night lucasagomes | 18:58 |
jroll | night lucasagomes :) | 18:59 |
*** lucasagomes is now known as lucas-dinner | 18:59 | |
NobodyCam | night lucas-dinner | 19:00 |
*** alexpilotti_ has joined #openstack-ironic | 19:02 | |
devananda | jroll: we've got ~5 weeks until RC. I'm questioning whether now is a good time to land this big a change in architecture - especially given that everyone seems to have other priorities | 19:02 |
devananda | jroll: should we make it a top priority? or put it on the short list for 'as soon as L opens" | 19:03 |
jroll | devananda: yeah, I'm fine with bumping it if that's what you'd like to do (though I'd argue it's not a major architecture change) | 19:03 |
jroll | either way | 19:03 |
*** alexpilotti has quit IRC | 19:03 | |
*** alexpilotti_ is now known as alexpilotti | 19:03 | |
jroll | devananda: to me right now, it's just downstream debt | 19:04 |
jroll | though with localboot landed it's much more useful | 19:04 |
devananda | jroll: I say major because it opens the door for new interactions with other network provideres that we're not testing or documenting yet upstream | 19:04 |
jroll | sure | 19:04 |
devananda | jroll: landing it now means adding that to our test and doc backlog ... which is already quite large and untracked | 19:04 |
jroll | yep | 19:05 |
devananda | so it's just a question of when | 19:05 |
jroll | right | 19:05 |
jroll | whatever you feel is right, I've been distracted downstream but could focus on it sooner if you want to land it | 19:05 |
jroll | I have no issues with bumping it | 19:05 |
jroll | well, not *no* issues, but I won't fight you about it :P | 19:06 |
devananda | honestly, right now, I'd rather make sure the other major things we've already landed this cycle are polished | 19:06 |
devananda | eg, zapping, cleaning, RAID, better IPA support, etc | 19:06 |
jroll | +1 | 19:06 |
devananda | cool | 19:06 |
devananda | bumping it | 19:06 |
jroll | devananda: what do you want to do about kilo-approved specs in L? | 19:07 |
jroll | fast track, full process, something else? | 19:07 |
devananda | jroll: why'd you have to ask that? :P | 19:07 |
jroll | :D | 19:07 |
devananda | fast track | 19:07 |
jroll | cool | 19:07 |
devananda | jroll: I feel awkward -2'ing your patches ... want to WIP them? | 19:09 |
jroll | lol | 19:10 |
jroll | I'll -2 | 19:10 |
devananda | hah ok | 19:10 |
*** anderbubble has joined #openstack-ironic | 19:10 | |
*** hj-hp has quit IRC | 19:10 | |
*** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 19:16 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Fix nits in cleaning https://review.openstack.org/160599 | 19:16 |
*** ParsectiX has quit IRC | 19:17 | |
*** ParsectiX has joined #openstack-ironic | 19:17 | |
devananda | sirushti: hi! have a minute to discuss whole-disk-image-support? | 19:21 |
sirushti | devananda, Hi, sure | 19:21 |
NobodyCam | devananda: I now have questions out on the ML and in the ceilometer channel about our dependent spec | 19:21 |
devananda | sirushti: it looks like nova has rejeceted the filter changes for kilo | 19:21 |
devananda | sirushti: do you think it's advisable to land the changes in ironic without that support in nova? | 19:22 |
devananda | NobodyCam: thanks! | 19:22 |
sirushti | devananda, sure, I think that's only an optimisation to fail fast. We still duplicate that validation bit in Ironic's DeployInterface.Validate | 19:22 |
jroll | scheduler filters can be out of tree as well, right? we could put them in our tree and document appropriately | 19:23 |
jroll | I think we still have shims for the filter tree too | 19:23 |
devananda | yah | 19:23 |
devananda | sirushti: ok, cool. is there any work following https://review.openstack.org/#/c/150142/ thta is not proposed yet for review? | 19:25 |
sirushti | devananda, just one bit, getting the agent deploy driver to use the same inference pattern to deploy whole disk images to maintain consistency | 19:26 |
devananda | sirushti: ok - if you could post that soon -- even in an early work-in-progress state -- that would be helpful for our tracking | 19:26 |
sirushti | devananda, will do | 19:27 |
*** gridinv has quit IRC | 19:31 | |
NobodyCam | devananda: do you know Jim Mankovich's irc handle? | 19:31 |
jroll | jmankovich or something | 19:31 |
devananda | NobodyCam: jmank*something | 19:31 |
NobodyCam | :) | 19:32 |
devananda | but I dont think he's on all that often | 19:32 |
NobodyCam | I'll email him | 19:32 |
devananda | jroll, JoshNang: status of in-band raid config work in IPA ? | 19:35 |
devananda | looks like https://review.openstack.org/#/c/157316/ is stalled for the last week? | 19:35 |
jroll | devananda: I have no clue | 19:35 |
jroll | oh, it's waiting on josh's stuff | 19:35 |
JoshNang | devananda: https://review.openstack.org/#/c/161066/ should give them direction on how to make it a hardware manager | 19:35 |
* JoshNang comments on the inband raid patch | 19:36 | |
jroll | I just did that :P | 19:36 |
JoshNang | ha thanks | 19:36 |
devananda | so, if that is not likely to land in Kilo, I'd be hesitant to land the foundational API changes in https://blueprints.launchpad.net/ironic/+spec/ironic-generic-raid-interface | 19:37 |
JoshNang | actually, as that patch stands, it doesn't need to exist. | 19:37 |
devananda | JoshNang: all the RAID stuff is blocked, pending your work on zapping, right? | 19:39 |
JoshNang | dispatching those commands to the hardware managers is supported by the patch i posted, they'll just need to have a hardware manager that returns raid steps. whether that lives in IPA or out of tree, i'm indifferent | 19:39 |
JoshNang | devananda: it should be unblocked now. adding RAID to zapping just means adding a function with @clean_step, which is meged | 19:39 |
JoshNang | merged, even | 19:39 |
devananda | ok | 19:40 |
devananda | there's a pretty large change proposed | 19:40 |
devananda | https://review.openstack.org/#/c/155230/ is ~1400 lines | 19:40 |
JoshNang | oh wow | 19:41 |
devananda | .... https://review.openstack.org/#/c/155230/17/ironic/common/raid.py,unified | 19:42 |
*** mikal has joined #openstack-ironic | 19:46 | |
*** r-daneel has joined #openstack-ironic | 19:47 | |
*** pas-ha has quit IRC | 19:48 | |
*** dprince has quit IRC | 19:49 | |
*** lucas-dinner has quit IRC | 19:49 | |
*** zz_jgrimm is now known as jgrimm | 19:51 | |
devananda | JoshNang: so a quick skim of ^ and things don't look bad. however, I don't see any drivers proposed that use it? | 19:51 |
devananda | sirushti: hi again! are you aware of any work to add RAID support for ilo driver, based on top of https://review.openstack.org/#/c/155230/17 ? | 19:52 |
JoshNang | devananda: yeah i'm assuming an iLO patch will be coming that will use agent inband raid? | 19:54 |
devananda | JoshNang: right - I assume so, but I dont see it yet | 19:55 |
devananda | JoshNang: oh hm. that'll be a feature only of the agent_ilo river -- and not, say, pxe_ilo or agent_ipmitool, right? | 19:56 |
devananda | isn't that a bit odd? | 19:56 |
*** EmilienM is now known as EmilienM|afk | 19:56 | |
*** harlowja_ has quit IRC | 19:57 | |
devananda | JoshNang: or is there a patch up for any agent_* driver to use the inband raid and I've just skipped it somewhere? | 19:57 |
*** devlaps has joined #openstack-ironic | 19:58 | |
JoshNang | devananda: nope, that's one of the last bits to finish up cleaning :/ | 19:58 |
devananda | ah. we should be tracking that | 19:59 |
JoshNang | like what's left? | 19:59 |
devananda | JoshNang: right. like what code is outstanding before the spec is done() | 19:59 |
devananda | JoshNang: could you post something and WIP it, tagging it to https://blueprints.launchpad.net/ironic/+spec/implement-cleaning-states ? | 19:59 |
sirushti | devananda, nope, best ask rameshg87 for that. | 20:00 |
JoshNang | devananda: sure. i'm gonna finish reviewing this raid patch first. | 20:00 |
sirushti | devananda, I saw you updated the whole-disk-image spec to also add the filter to the Ironic tree. | 20:00 |
devananda | JoshNang: sure. thanks | 20:00 |
sirushti | devananda, does that mean the Ironic tree would once again have to start running nova tests? | 20:00 |
*** harlowja has joined #openstack-ironic | 20:01 | |
devananda | sirushti: nope ... but when testing that, you would have to configure nova appropriately to use that filter | 20:01 |
devananda | sirushti: OR we simply don't land it this cycle and just leave the filter proposed to nova | 20:02 |
sirushti | devananda, so unit tests for filters just won't be there until it lands in Nova | 20:02 |
sirushti | cool | 20:02 |
*** kkoski has joined #openstack-ironic | 20:04 | |
*** kkoski has quit IRC | 20:07 | |
*** achanda has quit IRC | 20:07 | |
*** kkoski has joined #openstack-ironic | 20:08 | |
*** andreykurilin_ has joined #openstack-ironic | 20:13 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic-python-agent: Add cleaning/zapping support to IPA https://review.openstack.org/161066 | 20:15 |
devananda | JoshNang: tagging the BP on ^ | 20:15 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/161389 | 20:15 |
JoshNang | devananda: thanks! can i tag multiple bps on there? it's code required for cleaning and zapping too | 20:16 |
devananda | yup | 20:16 |
JoshNang | awesome | 20:16 |
devananda | hm actually, i think the automation will only keep one BP up to date | 20:16 |
devananda | but you can add the link to the wihteboard on any BP you want | 20:16 |
JoshNang | gotcha | 20:17 |
*** ijw has quit IRC | 20:17 | |
*** kkoski has quit IRC | 20:18 | |
*** logan2 has joined #openstack-ironic | 20:29 | |
*** chlong has joined #openstack-ironic | 20:30 | |
*** andreykurilin_ has quit IRC | 20:30 | |
*** kkoski has joined #openstack-ironic | 20:34 | |
*** mrda has joined #openstack-ironic | 20:37 | |
mrda | Morning Ironic | 20:38 |
NobodyCam | mornign mrda | 20:38 |
*** achanda has joined #openstack-ironic | 20:41 | |
NobodyCam | devananda: should https://bugs.launchpad.net/ironic/+bug/1427923 be targeted to K-3? | 20:45 |
openstack | Launchpad bug 1427923 in Ironic "boot device API blocks while waiting on the BMC" [Medium,Confirmed] | 20:45 |
*** Marga_ has quit IRC | 20:57 | |
*** Marga_ has joined #openstack-ironic | 20:57 | |
*** erwan_taf has quit IRC | 20:58 | |
*** ParsectiX has quit IRC | 21:02 | |
*** ParsectiX has joined #openstack-ironic | 21:02 | |
mrda | thanks lintan, dtantsur, rloo for the reviews on ironicclient & microversions | 21:03 |
openstackgerrit | Merged openstack/ironic: Follow-up patch for generic node inspection https://review.openstack.org/160665 | 21:04 |
rloo | mrda: thank YOU for writing it. | 21:04 |
mrda | I'll try and get a revised version up in the next couple of hours | 21:04 |
* mrda goes looking for breakfast | 21:04 | |
*** anderbubble has quit IRC | 21:10 | |
devananda | NobodyCam: maybe? not going to worry about it yet | 21:13 |
devananda | NobodyCam: BadCub and I went through about half of the BP's today, starting from the high priority. we'll cover the other half tomorrow | 21:13 |
devananda | I'll think more about bug targeting once we've passed feature freeze (two weeks) | 21:13 |
*** kkoski has quit IRC | 21:14 | |
mrda | hey devananda, are you happy with the format of the api microversion stuff I wrote up yesterday? (https://review.openstack.org/#/c/161110/) I know there's review comments, but want to make sure this is an acceptible way to capture the expected behaviour | 21:17 |
*** alexpilotti has quit IRC | 21:17 | |
mrda | ...before I spend too much time polishing :) | 21:17 |
*** achanda has quit IRC | 21:19 | |
*** Marga_ has quit IRC | 21:23 | |
*** ifarkas has quit IRC | 21:23 | |
*** Marga_ has joined #openstack-ironic | 21:23 | |
NobodyCam | devananda: :) | 21:24 |
devananda | mrda: /me looks | 21:25 |
*** ChuckC has joined #openstack-ironic | 21:32 | |
*** Marga_ has quit IRC | 21:34 | |
*** Marga_ has joined #openstack-ironic | 21:34 | |
NobodyCam | devananda: you should have a reply from Jim Mankovich in your email. based on that looks like we should bump ceilometer spec. | 21:37 |
TheJulia | So I'm trying to deploy configdrives into instances with the agent driver, I've got what at least from what I've read through code and doc is a correctly formatted configdrive file, ISO9660, base64'ed and gzipped. But I'm getting "Not a gzipped file", which appears after my provisioning node boots and pulls the configdrive file down. | 21:38 |
*** Marga_ has quit IRC | 21:38 | |
*** romcheg has quit IRC | 21:39 | |
TheJulia | Anyone happen to have any pointers on hunting that down? | 21:39 |
*** Marga_ has joined #openstack-ironic | 21:39 | |
*** EmilienM|afk is now known as EmilienM | 21:43 | |
jlvillal_train | TheJulia: Maybe dumb thought, but what does 'file' say about the configdrive file? Hopefully it says it is gzipped | 21:43 |
jroll | TheJulia: did you gzip then base64? | 21:44 |
jroll | that's the order it should be | 21:44 |
TheJulia | oh :( | 21:44 |
* TheJulia flips the order around | 21:44 | |
jroll | heh | 21:45 |
jroll | sorry for lack of docs on this :( | 21:46 |
TheJulia | I thought it was a bit weird, but It was really late when I was reading that, and now it makes way more sense | 21:46 |
jroll | yeah | 21:47 |
jroll | I mean, it's still weird | 21:47 |
*** ijw has joined #openstack-ironic | 21:47 | |
*** ijw has quit IRC | 21:48 | |
*** ijw has joined #openstack-ironic | 21:50 | |
*** dhellmann has quit IRC | 21:52 | |
*** dhellmann has joined #openstack-ironic | 21:55 | |
*** dhellmann has quit IRC | 21:56 | |
*** dhellmann has joined #openstack-ironic | 21:57 | |
NobodyCam | brb | 21:57 |
*** pelix has quit IRC | 21:57 | |
*** harlowja has quit IRC | 21:58 | |
devananda | mrda: one nit - line wrapping. 80 chars pls | 22:00 |
*** ijw has quit IRC | 22:01 | |
mrda | sure, I wrote that spec in a hurry :) | 22:01 |
mrda | and I knew I'd have to come back and fix a few things today | 22:01 |
*** ijw has joined #openstack-ironic | 22:01 | |
devananda | sure :) | 22:02 |
devananda | just easier fo rme to read when it's wrapped | 22:02 |
devananda | i'm adding comments now | 22:02 |
mrda | sorry about that | 22:03 |
*** ijw_ has joined #openstack-ironic | 22:06 | |
*** ijw__ has joined #openstack-ironic | 22:08 | |
*** ijw has quit IRC | 22:08 | |
*** jlvillal_train has quit IRC | 22:09 | |
*** ijw_ has quit IRC | 22:11 | |
*** ChuckC has quit IRC | 22:11 | |
*** ijw__ has quit IRC | 22:12 | |
*** chlong has quit IRC | 22:12 | |
*** ijw has joined #openstack-ironic | 22:13 | |
openstackgerrit | Merged openstack/ironic: Spelling error in Comment https://review.openstack.org/158970 | 22:14 |
openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is writable https://review.openstack.org/160383 | 22:15 |
*** athomas has quit IRC | 22:17 | |
*** achanda has joined #openstack-ironic | 22:20 | |
*** jgrimm is now known as zz_jgrimm | 22:23 | |
openstackgerrit | John Trowbridge proposed stackforge/ironic-discoverd: Add python-openstackclient plugin for ironic-discoverd https://review.openstack.org/157448 | 22:25 |
*** jlvillal_train has joined #openstack-ironic | 22:25 | |
*** achanda has quit IRC | 22:26 | |
*** harlowja has joined #openstack-ironic | 22:28 | |
*** Marga_ has quit IRC | 22:31 | |
*** Marga_ has joined #openstack-ironic | 22:31 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement cleaning/zapping for the agent driver https://review.openstack.org/161453 | 22:32 |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement cleaning/zapping for the agent driver https://review.openstack.org/161453 | 22:32 |
devananda | jroll, NobodyCam: is the dib + ipa support building any artifacts in the way that the coreos + ipa job does? | 22:35 |
devananda | cause I think that'd be great, but I dont see any | 22:35 |
jroll | I have no idea about dib | 22:35 |
jroll | it should build artifacts | 22:35 |
devananda | also, if we dont have a job to test that, why not? | 22:35 |
NobodyCam | devananda: last I checked no.. dib ipa element was very crude and only worked with fedora | 22:35 |
devananda | ugh | 22:36 |
NobodyCam | on nodes with 4 gb of ram | 22:36 |
*** ijw_ has joined #openstack-ironic | 22:36 | |
NobodyCam | that was a little while ago | 22:36 |
devananda | sure - it doesn't have to run in the gate in parallel -- we could wedge a single 4GB VM into the gate | 22:36 |
devananda | just to test the building and booting of it | 22:36 |
devananda | jroll: so it seems we've hit a kernel bug in coreos that's preventing /dev/sda from being found | 22:37 |
devananda | jroll: ring any bells? | 22:37 |
jroll | devananda: no, never heard of that, we run the same version of coreos downstream | 22:37 |
NobodyCam | devananda: is this on the moonshot | 22:38 |
devananda | :-/ | 22:38 |
devananda | NobodyCam: yes | 22:38 |
NobodyCam | the disks come up as sda or do they do something strange like /dev/cciss/c0d0 ? | 22:39 |
devananda | TheJulia: ^ ? | 22:39 |
*** ijw has quit IRC | 22:39 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Add zapping API https://review.openstack.org/161457 | 22:40 |
NobodyCam | devananda: there is a sda on the c1n1 node | 22:43 |
TheJulia | NobodyCam: in this case they are /dev/sda on ubuntu, standard SATA adapters | 22:43 |
*** romcheg has joined #openstack-ironic | 22:43 | |
NobodyCam | TheJulia: yea I just looked | 22:43 |
TheJulia | NobodyCam: sorry, I stepped away for a moment | 22:44 |
NobodyCam | TheJulia: do you have the boot up log | 22:44 |
NobodyCam | TheJulia: NP at all | 22:44 |
jroll | devananda: TheJulia: I guess this could be a coreos bug, now that I remember it's moonshot | 22:44 |
TheJulia | NobodyCam: kern.log on cartrage1 | 22:46 |
jroll | coreos people are awesome, btw | 22:48 |
devananda | having only really interacted with redbeard - i share jroll's impression | 22:48 |
NobodyCam | humm: TheJulia did you see this: Mar 3 01:17:04 c1n1 kernel: [ 0.000000] Booting paravirtualized kernel on bare hardware | 22:48 |
devananda | he was fantastically helpful and interested in what we're doing when i ran into him at FOSDEM | 22:48 |
jroll | they've helped us with coreos and systemd bugs at this point | 22:49 |
TheJulia | NobodyCam: nice! | 22:50 |
NobodyCam | TheJulia: your image is atom compatiable? | 22:50 |
NobodyCam | smpboot: CPU0: Intel(R) Atom(TM) | 22:50 |
*** rlr0329 has joined #openstack-ironic | 22:52 | |
TheJulia | NobodyCam: it boots :) It tosses back that it can't deploy the configdrive. http://paste.openstack.org/show/188215/ | 22:52 |
devananda | NobodyCam: it's x86_64 ... | 22:52 |
devananda | TheJulia: do you have the kernel log from the coreos image's boot? | 22:54 |
NobodyCam | ya the one I'm looking at looks ok: Mar 3 01:17:04 c1n1 kernel: [ 27.251360] sd 0:0:0:0: [sda] Attached SCSI disk | 22:55 |
TheJulia | I thought that it was a black box that one couldn't get such things out of? | 22:55 |
devananda | TheJulia: IPA ramdisk can be booted and ssh'd into during the deploy phase :) | 22:55 |
NobodyCam | :) | 22:55 |
jroll | however | 22:56 |
jroll | upstream ironic will power it off as soon as deploy fails | 22:56 |
jroll | TheJulia: ^ | 22:56 |
jroll | devananda: & | 22:56 |
jroll | s/&/^/ | 22:56 |
devananda | jroll: did ya'll upstream the "ohhai, deploy failed, ship the logs somewhere" stuff? | 22:56 |
devananda | or was your solution to just not power off, and let a human ssh in? | 22:57 |
jroll | devananda: we haven't even downstreamed that yet :/ | 22:57 |
devananda | ick | 22:57 |
jroll | ya. | 22:57 |
jroll | it's a hardish problem | 22:57 |
jroll | or to do it right is, anyway | 22:57 |
jroll | OTOH | 22:57 |
jroll | we never really see deploy failures unless swift blips. | 22:58 |
devananda | hah | 22:58 |
devananda | well, that's good to hear :) | 22:58 |
rlr0329 | Question. Since IPA doesn't know how to partition disks yet how could cloud-init be used to expand the root filesystem to full size, i.e. how to determine when the deploy is finished? | 22:59 |
rlr0329 | so that root filesystem can be expanded | 22:59 |
*** ParsectiX has quit IRC | 22:59 | |
devananda | rlr0329: growroot happens during boot of the user image, not during deploy | 22:59 |
jroll | ^ | 23:00 |
devananda | and the / partition should be the last one in the disk image. so it can grow | 23:00 |
*** ParsectiX has joined #openstack-ironic | 23:00 | |
devananda | jroll: ^ right? | 23:00 |
jroll | in the image, yeah | 23:00 |
jroll | and then we stick the configdrive at the end of the disk | 23:00 |
rlr0329 | so growroot would need to be added to the boot of the user_image? | 23:00 |
rlr0329 | jroll: what exactly is the configdrive? | 23:01 |
openstackgerrit | Haomeng,Wang proposed openstack/ironic: Add tftp mapfile configuration in install-guide https://review.openstack.org/161074 | 23:02 |
jroll | rlr0329: basically gives same info as the metadata service | 23:02 |
jroll | but as a partition rather than an API call | 23:02 |
rlr0329 | jroll: That is what I thought but how do I configure it? | 23:03 |
jroll | rlr0329: nova boot --configdrive true | 23:04 |
NobodyCam | Haomeng: is that only for one os? | 23:05 |
NobodyCam | I have not had to do that on ubuntu | 23:05 |
rlr0329 | jroll: ok cool. Sorry for the simple question | 23:05 |
jroll | no worries :) | 23:06 |
*** alexpilotti has joined #openstack-ironic | 23:06 | |
Haomeng | NobodyCam: :) | 23:08 |
Haomeng | NobodyCam: I tested it with ubuntu and rehat, same tftp configurations | 23:08 |
Haomeng | NobodyCam: both works | 23:08 |
NobodyCam | Haomeng: ack thank you | 23:08 |
*** yuanying has joined #openstack-ironic | 23:09 | |
Haomeng | NobodyCam: without the mapfile configuration, some pxe client will not get tftp file with virtual folder path | 23:09 |
Haomeng | NobodyCam: :) | 23:09 |
NobodyCam | Haomeng: yep | 23:12 |
NobodyCam | I voted +2 but gate is taking like 9 hours today | 23:12 |
Haomeng | NobodyCam: thank you, gate is very slowly these days | 23:13 |
jroll | our bad :x | 23:13 |
jroll | turns out rebooting an entire cloud will impact the gate :P | 23:14 |
NobodyCam | lol | 23:14 |
* NobodyCam pokes jroll with a dull stick | 23:14 | |
jroll | hehe | 23:15 |
NobodyCam | lol | 23:15 |
NobodyCam | j/k | 23:15 |
jroll | to be fair, I didn't reboot anything | 23:15 |
NobodyCam | :) | 23:15 |
*** lazy_prince has quit IRC | 23:20 | |
*** killer_prince has joined #openstack-ironic | 23:22 | |
*** killer_prince is now known as lazy_prince | 23:22 | |
*** alexpilotti has quit IRC | 23:26 | |
* NobodyCam kills his devtest setup and attempts to run devstack with ipa | 23:26 | |
*** dttocs has joined #openstack-ironic | 23:27 | |
*** jlvillal_train has quit IRC | 23:28 | |
TheJulia | NobodyCam: http://paste.openstack.org/show/188254/ | 23:31 |
jroll | [ 10.827236] sda: unknown partition table | 23:32 |
jroll | [ 10.828235] sd 0:0:0:0: [sda] Attached SCSI disk | 23:32 |
jroll | seems like it's seeing it | 23:32 |
TheJulia | yeah... | 23:32 |
jroll | so what errors were you getting? | 23:32 |
TheJulia | looks like partx just does not like the drive, where as parted goes "unknown partition table" | 23:34 |
jroll | mmm. | 23:34 |
jroll | so we used to use something else... | 23:35 |
jroll | though I feel like this maybe is a bad partition table? | 23:35 |
NobodyCam | ya | 23:35 |
jroll | https://review.openstack.org/#/c/153592/2/ironic_python_agent/shell/copy_configdrive_to_disk.sh | 23:35 |
TheJulia | so the fun thing is, somehow it created it :( | 23:35 |
jroll | might be worth reverting that just to try | 23:36 |
NobodyCam | TheJulia: is this a whole disk image or partition image? | 23:36 |
jroll | TheJulia: well, it's a full disk image no? | 23:36 |
TheJulia | this had a working ubuntu installation before I tried a deploy on it | 23:36 |
TheJulia | partition images I think | 23:36 |
jroll | oh, hm | 23:37 |
jroll | so you're using the iscsi_agent driver or whatever? | 23:37 |
jroll | I guess it's still pxe_* maybe, idk | 23:37 |
TheJulia | ahh, I think its the image, apparently i didn't prep it correctly in my playbook | 23:38 |
jroll | \o/ I didn't break a thing | 23:38 |
TheJulia | lol | 23:38 |
* TheJulia goes and massages the playbooks and the servers to see what happens | 23:39 | |
*** ijw has joined #openstack-ironic | 23:39 | |
NobodyCam | TheJulia: if your building the image with dib I think the VM element will create a workable image | 23:40 |
*** penick has joined #openstack-ironic | 23:40 | |
*** jlvillal_train has joined #openstack-ironic | 23:40 | |
*** jlvillal_train has quit IRC | 23:40 | |
*** achanda has joined #openstack-ironic | 23:40 | |
TheJulia | NobodyCam: pre-canned image, i derped on it because I was rushing | 23:41 |
* NobodyCam notes he will need to leave at 4 today to meet contractor at new home location | 23:41 | |
*** ijw_ has quit IRC | 23:42 | |
NobodyCam | TheJulia: :) | 23:42 |
*** chlong has joined #openstack-ironic | 23:42 | |
*** absubram has quit IRC | 23:43 | |
* NobodyCam also wounders if stack.sh will finish before he has to leave | 23:44 | |
*** jlvillal has left #openstack-ironic | 23:45 | |
*** chlong has quit IRC | 23:46 | |
*** chlong has joined #openstack-ironic | 23:46 | |
TheJulia | NobodyCam: have fun! | 23:47 |
NobodyCam | :) | 23:48 |
openstackgerrit | Adam Gandelman proposed openstack/ironic: Check-in Ironic's devstack code as in-tree plugin https://review.openstack.org/161485 | 23:54 |
*** kkoski has joined #openstack-ironic | 23:55 | |
*** jlvillal_ has joined #openstack-ironic | 23:55 | |
*** chuckC_ has joined #openstack-ironic | 23:56 | |
*** EmilienM is now known as EmilienM|afk | 23:56 | |
NobodyCam | humm: http://paste.openstack.org/show/OQBNCLtVmcPUtSnM3aLD/ | 23:56 |
NobodyCam | ^^^ that seems not right... | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!