| *** Sukhdev has joined #openstack-ironic | 00:00 | |
| rloo | jroll: so devananda is right, the install guide is wrong. it was wrong the minute we updated it beyond kilo. | 00:00 |
|---|---|---|
| rloo | jroll: the problem is that we don't have a copy anywhere of what *was* in a particular release. | 00:01 |
| jroll | rloo: right, we need to work with the docs team on that. wondering if pshige_ can take that on | 00:01 |
| rloo | jroll: and it seems to me that until we have urls for our docs that reflect each release (new semver releases or 6-month-named cycles) it is too confusing. | 00:02 |
| *** naohirot has joined #openstack-ironic | 00:06 | |
| *** sirushti has quit IRC | 00:07 | |
| *** Sukhdev has quit IRC | 00:21 | |
| jroll | mrda: oh btw, something is broken with that dashboard creator. I haven't debugged yet | 00:21 |
| jroll | mrda: even before your commit :( | 00:21 |
| jroll | I also broke the perm.ly link | 00:22 |
| jroll | feels bad :( | 00:22 |
| *** shadower has quit IRC | 00:23 | |
| *** shadower has joined #openstack-ironic | 00:23 | |
| mrda | jroll: orly? sorry | 00:24 |
| mrda | it worked locally | 00:24 |
| rloo | jroll: why is there a funny string in the commit? https://review.openstack.org/#/c/224079/24//COMMIT_MSG | 00:24 |
| mrda | i'll look today | 00:24 |
| rloo | jroll: line 14 | 00:24 |
| *** rbudden has quit IRC | 00:24 | |
| jroll | mrda: it works, the link doesn't, it wasn't your commit that broke it though | 00:25 |
| jroll | mrda: the commit before yours is similarly broken | 00:25 |
| jroll | rloo: wtf | 00:25 |
| jroll | idk | 00:25 |
| rloo | jroll: I'm ready to +2/+A. | 00:26 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Implement indirection_api https://review.openstack.org/224079 | 00:26 |
| jroll | rloo: fixed and +2'd | 00:26 |
| rloo | oh. maybe i should wait for jenkins. | 00:27 |
| jroll | /shrug | 00:27 |
| jroll | it passed before, I don't expect any surprisesd | 00:27 |
| jroll | anyway, I'm out for the night, hoping to come back to these last two patches merged tomorrow :) | 00:28 |
| rloo | ok, we'll see what we'll see. night. | 00:28 |
| jroll | night | 00:28 |
| rloo | oh, the depends on didn't merge yet so this won't merge til after that. | 00:29 |
| *** Sukhdev has joined #openstack-ironic | 00:32 | |
| *** toabctl has quit IRC | 00:32 | |
| *** trown is now known as trown|outttypeww | 00:33 | |
| *** _fortis has quit IRC | 00:33 | |
| *** achanda has joined #openstack-ironic | 00:33 | |
| *** toabctl has joined #openstack-ironic | 00:35 | |
| *** d0ugal has quit IRC | 00:35 | |
| *** achanda has quit IRC | 00:38 | |
| *** d0ugal has joined #openstack-ironic | 00:40 | |
| *** d0ugal is now known as Guest65853 | 00:40 | |
| *** tsekiyam_ has joined #openstack-ironic | 00:41 | |
| *** tsekiyama has quit IRC | 00:44 | |
| *** Sukhdev has quit IRC | 00:44 | |
| *** Sukhdev has joined #openstack-ironic | 00:44 | |
| *** Sukhdev has quit IRC | 00:44 | |
| *** tsekiyam_ has quit IRC | 00:45 | |
| *** mtanino has quit IRC | 00:46 | |
| *** _fortis has joined #openstack-ironic | 00:46 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Update install guide to reflect latest code https://review.openstack.org/227029 | 00:47 |
| *** boris-42 has quit IRC | 00:49 | |
| *** rloo has quit IRC | 01:01 | |
| *** sdake has joined #openstack-ironic | 01:03 | |
| *** penick has quit IRC | 01:22 | |
| *** Marga__ has joined #openstack-ironic | 01:29 | |
| *** Marga_ has quit IRC | 01:32 | |
| *** Marga__ has quit IRC | 01:33 | |
| *** baoli has joined #openstack-ironic | 01:35 | |
| mrda | jroll: how can you detect that it's broken? Where do you see that borkedness? | 01:39 |
| *** baoli has quit IRC | 01:40 | |
| *** zhenguo has joined #openstack-ironic | 01:43 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Use of 'the Bare Metal service' in guide https://review.openstack.org/227080 | 01:44 |
| *** harshs has quit IRC | 01:46 | |
| *** bizarrochristy has quit IRC | 01:54 | |
| *** bizarrochristy has joined #openstack-ironic | 01:54 | |
| jroll | mrda: I run './gerrit-dash-creator dashboards/ironic.dash' and then click the URL it outputs. gerrit loads and gives me an error | 01:59 |
| *** ijw has quit IRC | 02:00 | |
| *** achanda has joined #openstack-ironic | 02:01 | |
| mrda | jroll: huh. I cut and paste the url to a broiwser and it works... | 02:06 |
| *** praneshp has quit IRC | 02:06 | |
| *** sdake has quit IRC | 02:06 | |
| mrda | s/broiwser/browser | 02:06 |
| jroll | whaaat | 02:06 |
| jroll | ff or chrome? | 02:06 |
| mrda | Chrome on Mac OS X, but I will now try other browsers/OSes | 02:07 |
| jroll | ok, I'm in firefox | 02:07 |
| jroll | but the old dashboard totally worked | 02:07 |
| * mrda hates browser differences :/ | 02:07 | |
| * jroll tries chrome | 02:07 | |
| mrda | jroll: Just FYI, if I hadn't tested it, I wouldn't have pushed the patch up | 02:08 |
| jroll | mrda: totally | 02:08 |
| jroll | I never thought it was you | 02:08 |
| jroll | like I said, I checked it out before your commit and it was also broken | 02:08 |
| jroll | wtf, totally works in chrome | 02:09 |
| jroll | :( | 02:09 |
| mrda | ok, I'll look at this this afternoon. probably in the generation code | 02:09 |
| jroll | mrda: nah, I think it may be a gerrit thing | 02:10 |
| jroll | I also just checked http://perm.ly/ironic-review-dashboard which is broken in FF | 02:10 |
| jroll | it is the same as the old link | 02:10 |
| jroll | so I'll poke infra tomorrow to see if there was a gerrit update, because that used to work in FF for me | 02:10 |
| mrda | right | 02:10 |
| mrda | I will see if I can spot the difference in the gernation code | 02:10 |
| jroll | sorry for the trouble :( | 02:10 |
| jroll | well | 02:11 |
| mrda | not at all - you have every expectation that it should work :) | 02:11 |
| jroll | I think it's gerrit+FF, not the generation code | 02:11 |
| jroll | the perm.ly link did NOT get changed and is now broken for me in FF | 02:11 |
| jroll | so don't worry about it, I'll bug folks | 02:11 |
| mrda | so gerrit-dash-creator should adjust to gerrit + {common browsers} | 02:11 |
| jroll | no, I think complex queries are broken with gerrit+ff | 02:12 |
| mrda | :( | 02:12 |
| jroll | or the old ironic dashboard query, at least | 02:13 |
| jroll | (and the new) | 02:13 |
| *** rbudden has joined #openstack-ironic | 02:16 | |
| *** baoli has joined #openstack-ironic | 02:17 | |
| *** priteau has joined #openstack-ironic | 02:19 | |
| *** priteau has quit IRC | 02:24 | |
| *** r-daneel has quit IRC | 02:33 | |
| *** gabriel-bezerra has quit IRC | 02:47 | |
| *** solid-sinval has quit IRC | 02:47 | |
| *** solid-sinval has joined #openstack-ironic | 02:53 | |
| *** gabriel-bezerra has joined #openstack-ironic | 02:53 | |
| *** Nisha has joined #openstack-ironic | 02:56 | |
| *** chenglch has joined #openstack-ironic | 02:56 | |
| *** harshs has joined #openstack-ironic | 03:01 | |
| *** rbudden has quit IRC | 03:02 | |
| *** bizarrochristy has quit IRC | 03:04 | |
| *** saripurigopi has joined #openstack-ironic | 03:23 | |
| *** stendulker has joined #openstack-ironic | 03:23 | |
| *** sirushti has joined #openstack-ironic | 03:27 | |
| saripurigopi | Morning Ironic | 03:33 |
| openstackgerrit | Merged openstack/ironic: Implement indirection_api https://review.openstack.org/224079 | 03:33 |
| *** shuquan has joined #openstack-ironic | 03:34 | |
| *** sirushti has quit IRC | 03:35 | |
| *** sirushti has joined #openstack-ironic | 03:35 | |
| *** baoli has quit IRC | 03:47 | |
| *** harshs has quit IRC | 03:49 | |
| *** Nisha has quit IRC | 03:50 | |
| *** shuquan has quit IRC | 03:51 | |
| *** Nisha has joined #openstack-ironic | 03:52 | |
| *** achanda has quit IRC | 03:55 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add db migration for tags table https://review.openstack.org/192576 | 03:55 |
| *** shuquan has joined #openstack-ironic | 03:56 | |
| *** praneshp has joined #openstack-ironic | 03:57 | |
| *** rameshg87 has joined #openstack-ironic | 04:00 | |
| *** harshs has joined #openstack-ironic | 04:11 | |
| *** saripurigopi has quit IRC | 04:12 | |
| *** saripurigopi has joined #openstack-ironic | 04:12 | |
| *** VikasC has joined #openstack-ironic | 04:16 | |
| *** Nisha_away has joined #openstack-ironic | 04:29 | |
| *** Nisha has quit IRC | 04:32 | |
| *** garthb has joined #openstack-ironic | 04:39 | |
| *** Marga_ has joined #openstack-ironic | 04:42 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Fixes config drive creation failure in UEFI boot mode https://review.openstack.org/215097 | 04:42 |
| *** Marga_ has quit IRC | 04:43 | |
| *** Marga_ has joined #openstack-ironic | 04:43 | |
| *** sdake has joined #openstack-ironic | 04:55 | |
| *** achanda has joined #openstack-ironic | 04:55 | |
| *** chenglch has quit IRC | 04:58 | |
| *** shuquan has quit IRC | 05:03 | |
| *** shuquan has joined #openstack-ironic | 05:03 | |
| *** ifarkas has quit IRC | 05:07 | |
| *** saripurigopi has quit IRC | 05:11 | |
| *** garthb has quit IRC | 05:13 | |
| *** saripurigopi has joined #openstack-ironic | 05:17 | |
| *** shuquan has quit IRC | 05:18 | |
| *** yuanying has joined #openstack-ironic | 05:18 | |
| *** puranamr has joined #openstack-ironic | 05:22 | |
| *** vinbs has joined #openstack-ironic | 05:24 | |
| *** puranamr has quit IRC | 05:26 | |
| *** puranamr has joined #openstack-ironic | 05:27 | |
| *** sdake has quit IRC | 05:40 | |
| *** puranamr has quit IRC | 05:52 | |
| *** deray has joined #openstack-ironic | 06:19 | |
| *** praneshp has quit IRC | 06:23 | |
| *** praneshp has joined #openstack-ironic | 06:25 | |
| *** praneshp_ has joined #openstack-ironic | 06:31 | |
| *** Nisha_away has quit IRC | 06:33 | |
| *** boris-42 has joined #openstack-ironic | 06:34 | |
| *** praneshp has quit IRC | 06:34 | |
| *** praneshp_ is now known as praneshp | 06:34 | |
| *** saripurigopi has quit IRC | 06:42 | |
| *** harshs has quit IRC | 06:42 | |
| *** pradipta has joined #openstack-ironic | 06:43 | |
| *** saripurigopi has joined #openstack-ironic | 06:44 | |
| *** Nisha_away has joined #openstack-ironic | 06:47 | |
| *** saripurigopi has quit IRC | 06:50 | |
| *** chenglch has joined #openstack-ironic | 06:54 | |
| *** jrist has quit IRC | 06:56 | |
| *** jrist has joined #openstack-ironic | 06:59 | |
| *** romainh has joined #openstack-ironic | 07:02 | |
| *** yog_ has joined #openstack-ironic | 07:17 | |
| *** Nisha_away has quit IRC | 07:20 | |
| xek | morning :) | 07:26 |
| *** achanda has quit IRC | 07:28 | |
| *** karimb has joined #openstack-ironic | 07:33 | |
| *** Nisha_away has joined #openstack-ironic | 07:36 | |
| *** ifarkas has joined #openstack-ironic | 07:46 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Make raid clean steps configurable https://review.openstack.org/224938 | 07:51 |
| *** jamielennox is now known as jamielennox|away | 07:53 | |
| *** saripurigopi has joined #openstack-ironic | 07:58 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Update proliantutils version required for L release https://review.openstack.org/227165 | 08:06 |
| *** Guest65853 is now known as d0ugal | 08:08 | |
| *** athomas has joined #openstack-ironic | 08:08 | |
| *** d0ugal has quit IRC | 08:08 | |
| *** d0ugal has joined #openstack-ironic | 08:08 | |
| *** priteau has joined #openstack-ironic | 08:21 | |
| naohirot | rameshg87: hello | 08:23 |
| *** Nisha_away has quit IRC | 08:27 | |
| *** dtantsur|afk is now known as dtantsur | 08:30 | |
| dtantsur | Morning Ironic | 08:30 |
| *** lucasagomes has joined #openstack-ironic | 08:30 | |
| *** pradipta has quit IRC | 08:31 | |
| dtantsur | how is our gate, folks? | 08:32 |
| *** Nisha_away has joined #openstack-ironic | 08:33 | |
| *** BobBall has left #openstack-ironic | 08:34 | |
| lucasagomes | dtantsur, no idea, checking | 08:37 |
| dtantsur | looks like it's fine | 08:38 |
| lucasagomes | yeah looks alright | 08:38 |
| * dtantsur rechecks everything | 08:40 | |
| *** shoutm has joined #openstack-ironic | 08:41 | |
| xek | looks like this patch fixed it https://review.openstack.org/#/c/226995/ | 08:41 |
| rameshg87 | naohirot: hello | 08:42 |
| *** lifeless has quit IRC | 08:49 | |
| *** getvasanth has joined #openstack-ironic | 08:50 | |
| *** derekh has joined #openstack-ironic | 08:51 | |
| *** saripurigopi has quit IRC | 08:53 | |
| *** saripurigopi has joined #openstack-ironic | 08:53 | |
| *** athomas has quit IRC | 08:54 | |
| rameshg87 | dtantsur: lucasagomes: just a quick query regarding fedora | 08:56 |
| dtantsur | yep | 08:56 |
| rameshg87 | dtantsur: lucasagomes: can I get the contents of kernel-core packages on 4.1.5-200.fc22-x86_64 and 4.1.6-201.fc22.x86_64 | 08:57 |
| rameshg87 | dtantsur: I mean from the web can I get | 08:58 |
| lucasagomes | rameshg87, https://apps.fedoraproject.org/packages/kernel-core/ | 08:58 |
| rameshg87 | dtantsur: I am seeing a uefi localboot failure on the agent ramdisk built on latter but passes on the agent ramdisk built on former | 08:58 |
| lucasagomes | tab "contents" | 08:58 |
| lucasagomes | rameshg87, tho I don't like much the webinterface on that website | 08:59 |
| dtantsur | oh yeah... | 08:59 |
| dtantsur | btw, lucasagomes, mind reviewing IPA patch https://review.openstack.org/#/c/225092 please? | 08:59 |
| dtantsur | that's the plugins we used to have for inspection | 09:00 |
| lucasagomes | dtantsur, will do, 1 sec | 09:00 |
| *** lifeless has joined #openstack-ironic | 09:03 | |
| *** athomas has joined #openstack-ironic | 09:03 | |
| rameshg87 | lucasagomes: the web interface seems not so good :( | 09:04 |
| rameshg87 | lucasagomes: is it possible to get a quick changelog of what changed between 4.1.5-200.fc22-x86_64 and 4.1.6-201.fc22.x86_64 | 09:04 |
| rameshg87 | lucasagomes: in a faster way | 09:04 |
| dtantsur | s/not so good/pretty awful/ | 09:04 |
| rameshg87 | lucasagomes: because I am just looking for an information if one of kernel modules named efivars stopped being in builtin kernel in 4.1.6 | 09:04 |
| rameshg87 | looks like fedora switch to using efivarfs (a newer method) | 09:05 |
| rameshg87 | but I need a confirmation | 09:05 |
| dtantsur | rameshg87, maybe here https://admin.fedoraproject.org/updates/kernel | 09:05 |
| * rameshg87 checks | 09:06 | |
| rameshg87 | dtantsur: oh yeah | 09:06 |
| rameshg87 | dtantsur: https://bugzilla.redhat.com/show_bug.cgi?id=1252137 | 09:06 |
| openstack | bugzilla.redhat.com bug 1252137 in kernel "Apparently both CONFIG_EFI_VARS and CONFIG_EFIVAR_FS should not both be enabled..." [Unspecified,On_qa] - Assigned to kernel-maint | 09:06 |
| rameshg87 | I've disabled EFI_VARS in all the branches. :( | 09:07 |
| lucasagomes | rameshg87, hmm perhaps cloning it and looking at the tags (http://pkgs.fedoraproject.org/cgit/kernel.git/) | 09:07 |
| rameshg87 | lucasagomes: got it | 09:07 |
| lucasagomes | rameshg87, you can ask in #fedora too | 09:07 |
| rameshg87 | lucasagomes: confirmed as well ^^ | 09:07 |
| lucasagomes | cool | 09:07 |
| rameshg87 | lucasagomes: we used a tool efibootmgr which depended on this kernel module | 09:07 |
| rameshg87 | lucasagomes: now it seems disabled by default in newer kernels after 4.16 :( | 09:07 |
| rameshg87 | lucasagomes: so uefi localboot fails | 09:08 |
| lucasagomes | damn | 09:08 |
| dtantsur | wow, that's not good | 09:08 |
| * rameshg87 thinks what can be done | 09:08 | |
| rameshg87 | lucasagomes: dtantsur: efibootmgr is being used internally by grub (we don't use it directly) | 09:08 |
| rameshg87 | lucasagomes: dtantsur: and we use efibootmgr in instance image. so we don't have control as well :( | 09:09 |
| lucasagomes | rameshg87, and the cloud images are using 4.1.6? | 09:09 |
| lucasagomes | rameshg87, perahps at built time you can downgrade that kernel version? | 09:10 |
| lucasagomes | with a big ass FIXME on the code | 09:10 |
| rameshg87 | lucasagomes: we use the kernel on agent ramdisk while running grub-install (we use chroot) | 09:10 |
| rameshg87 | lucasagomes: so we should degrade the kernel to 4.1.5 while building agent ramdisk based on fedora | 09:10 |
| rameshg87 | while building from dib | 09:10 |
| lucasagomes | right | 09:10 |
| rameshg87 | dib provides an option I think | 09:11 |
| lucasagomes | yeah perhaps this is something we can add to the ironic-agent ramdisk | 09:11 |
| lucasagomes | s/ramdisk/element | 09:11 |
| rameshg87 | lucasagomes: to downgrade the kernel ? | 09:11 |
| lucasagomes | yeah | 09:11 |
| *** pelix has joined #openstack-ironic | 09:11 | |
| * rameshg87 checks if it's possible | 09:11 | |
| rameshg87 | lucasagomes: let me check | 09:11 |
| rameshg87 | lucasagomes: and the good thing is grub doesn't throw error if efibootmgr fails | 09:12 |
| lucasagomes | dnf downgrade <package> | 09:12 |
| rameshg87 | lucasagomes: so deploy goes through fine, but instance never boots :D | 09:12 |
| lucasagomes | I think you can set a specific version too | 09:12 |
| lucasagomes | :-( | 09:12 |
| lucasagomes | rameshg87, yeah grub is known for sucking when installing etc | 09:12 |
| lucasagomes | compared with syslinux | 09:12 |
| rameshg87 | :) | 09:13 |
| lucasagomes | grub had a script to generate another script (y) | 09:13 |
| rameshg87 | :D | 09:13 |
| naohirot | rameshg87: dtantsur: lucasagomes: good morning | 09:13 |
| lucasagomes | naohirot, morning | 09:13 |
| rameshg87 | naohirot: morning | 09:13 |
| naohirot | lucasagomes: rameshg87: hello | 09:14 |
| naohirot | rameshg87: can I talk? | 09:14 |
| rameshg87 | naohirot: yeah | 09:15 |
| rameshg87 | naohirot: might leave desk in a min, but till then it's okay | 09:15 |
| *** romcheg has joined #openstack-ironic | 09:15 | |
| naohirot | rameshg87: Okay, I hope I'm not interrupting your conversation with lucasagomes | 09:15 |
| naohirot | rameshg87: Yes. | 09:15 |
| naohirot | rameshg87: I'd like to know the status of new boot I/F. | 09:16 |
| naohirot | rameshg87: I think today is the last day to release 4.2.0, right? | 09:16 |
| naohirot | rameshg87: are you going to review the patches from now on? | 09:17 |
| rameshg87 | naohirot: sure, will do, I haven't got back a chance to look at the patches, will start doing it | 09:18 |
| rameshg87 | naohirot: I was busy involved in something else internally and upstream | 09:18 |
| naohirot | rameshg87: Okay, here is already 6PM, and what time will be the final dead line if I needed to update, I believe the case | 09:19 |
| naohirot | rameshg87: should I stand by? I can do that. | 09:20 |
| dtantsur | ok, inspector gate is STILL broken, now probably by nova >_< | 09:22 |
| naohirot | dtantsur: hi :) | 09:23 |
| dtantsur | o/ | 09:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Fix rootwrap configuration documentation https://review.openstack.org/227193 | 09:23 |
| dtantsur | lucasagomes, have you even seen http://logs.openstack.org/05/226805/2/check/gate-ironic-inspector-dsvm/5cd5071/logs/screen-n-cond.txt.gz?level=WARNING ? | 09:24 |
| lucasagomes | dtantsur, uu I haven't no | 09:25 |
| lucasagomes | seems metrics is missconfigured (perhaps it should default to an empty list and not None) | 09:26 |
| *** ndipanov has joined #openstack-ironic | 09:27 | |
| *** MattMan has joined #openstack-ironic | 09:27 | |
| dtantsur | jesus, I'll never release anything... | 09:29 |
| dtantsur | ok, looks like nova folks know what got broken :) | 09:34 |
| naohirot | rameshg87: I have to leave at 10 o'clock UTC. I'd like to just let you know that I fully tested the code with real hardware, and confirmed that the code worked fine except configdrive of iscsi case which has been already reported in the bugdb. | 09:46 |
| sambetts | dtantsur: not the gate again :( | 09:47 |
| dtantsur | sambetts, yeah, we got broken by nova... could you please join #openstack-nova and keep an eye on them? I'll be back in 1 hour | 09:49 |
| sambetts | dtantsur: Ok :) | 09:49 |
| rameshg87 | naohirot: I am sorry, was away from desk | 09:50 |
| rameshg87 | naohirot: I don't think boot interface will be completed in 4.2.0 | 09:50 |
| rameshg87 | naohirot: I suppose the rest of the refactoring will be done on the next release 4.3.0 | 09:50 |
| rameshg87 | naohirot: so we might have plenty of time :) | 09:50 |
| *** dtantsur is now known as dtantsur|brb | 09:50 | |
| naohirot | rameshg87: I see | 09:50 |
| naohirot | rameshg87: If it is the case, I'll come back tomorrow. If not, I intended to work all night :) | 09:52 |
| rameshg87 | naohirot: oh, please go head, it's not going to 4.2.0 | 09:53 |
| rameshg87 | naohirot: even ilo patches won't be merged in 4.2.0 | 09:53 |
| naohirot | rameshg87: anyway thanks! | 09:53 |
| rameshg87 | naohirot: thank you :) | 09:53 |
| rameshg87 | naohirot: sorry to have kept you waiting | 09:53 |
| naohirot | rameshg87: no problem :) | 09:53 |
| rameshg87 | lucasagomes: dtantsur|brb: I just thought of something regarding raid - https://review.openstack.org/#/c/224938/ | 09:55 |
| rameshg87 | please have a look at the above and see if it makes sense for 4.2.0 | 09:55 |
| lucasagomes | rameshg87, will do | 09:56 |
| *** VikasC has quit IRC | 09:57 | |
| *** naohirot has quit IRC | 09:58 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add i18n _ import to cimc common https://review.openstack.org/227211 | 10:19 |
| sambetts | lucasagomes: ^ I don't know why none of the tests picked that up | 10:19 |
| sambetts | lucasagomes: :'( | 10:20 |
| sambetts | lucasagomes: even my system testing didn't fail, I dont get it ... | 10:20 |
| *** e0ne has joined #openstack-ironic | 10:23 | |
| *** e0ne has quit IRC | 10:24 | |
| sambetts | dtantsur|brb: https://review.openstack.org/#/c/227199/ <- nova gate fix | 10:24 |
| rameshg87 | lucasagomes: can you please have a look at https://review.openstack.org/#/c/227165/ | 10:24 |
| rameshg87 | lucasagomes: I hope it isn't too late to change dependencies (proliantutils version) | 10:25 |
| rameshg87 | sambetts: wondering how does unit test pass without https://review.openstack.org/227211 | 10:32 |
| rameshg87 | sambetts: do you know why ? | 10:32 |
| sambetts | rameshg87: I really don't know :( the code even works properly without it ... | 10:32 |
| rameshg87 | sambetts: yeah, strange | 10:32 |
| rameshg87 | sambetts: debugger shows this | 10:33 |
| rameshg87 | sambetts: (Pdb) _ | 10:33 |
| rameshg87 | <bound method NullTranslations.gettext of <gettext.NullTranslations instance at 0x7f760c557170>> | 10:33 |
| rameshg87 | sambetts: without your patch | 10:33 |
| lucasagomes | rameshg87, right yeah maybe wait for jroll since he's the release liaison | 10:33 |
| lucasagomes | but LGTM | 10:33 |
| rameshg87 | lucasagomes: ^^ do you know reason why it works without importing _ | 10:34 |
| lucasagomes | sambetts, it didn't get caught perhaps because the driver won't load if the library is not installed? | 10:34 |
| rameshg87 | lucasagomes: unit tests run | 10:35 |
| lucasagomes | and since the library is not part of the test-requirements | 10:35 |
| *** ndipanov has quit IRC | 10:35 | |
| sambetts | lucasagomes: But it works ... like I have the code running in my lab | 10:35 |
| lucasagomes | rameshg87, it runs with a mock library right? | 10:35 |
| lucasagomes | oh | 10:35 |
| * lucasagomes no idea then | 10:35 | |
| rameshg87 | lucasagomes: but this is in common.py | 10:35 |
| rameshg87 | lucasagomes: wihtout sambetts patch, _ is <bound method NullTranslations.gettext of <gettext.NullTranslations instance at 0x7f760c557170>> | 10:35 |
| rameshg87 | lucasagomes: I am wondering where it got this | 10:35 |
| sambetts | I saw this issue come up in another patch recently, someone forgot to import _ and it passed every test fine | 10:36 |
| lucasagomes | it's odd that it also pass the pep8 stuff | 10:37 |
| lucasagomes | really odd, but cool to investigate | 10:37 |
| sambetts | lucasagomes: :/ exactly, it really shouldn't, it sourcing _ from somewhere | 10:37 |
| lucasagomes | yeah | 10:38 |
| lucasagomes | probably the way python import stuff (just stacking the loaded libs at sys.modules) ? | 10:38 |
| sambetts | lucasagomes: without an import at the top its almost like its registers as a python builtin func | 10:39 |
| sambetts | it must be something that ironic is doing though, because I tried things like importing oslo_i18n in my code then importing common.py and it fails | 10:41 |
| sambetts | but inside the ironic tests its fine | 10:41 |
| *** amotoki has joined #openstack-ironic | 10:41 | |
| *** zhenguo has quit IRC | 10:42 | |
| sambetts | :/ | 10:42 |
| lucasagomes | yeah needs some investigation... those are a bit obscure but can be fun to figure out why it happens | 10:43 |
| sambetts | lucasagomes: https://review.openstack.org/#/c/215505/11 patch set 11 here I left a comment about _ | 10:45 |
| *** saripurigopi has quit IRC | 10:47 | |
| *** saripurigopi has joined #openstack-ironic | 10:47 | |
| sambetts | lucasagomes: Found it! | 10:47 |
| lucasagomes | sambetts, oh where's it? | 10:48 |
| sambetts | gettext.install in ironic/tests/objects/test_objects.py | 10:48 |
| sambetts | https://docs.python.org/2/library/gettext.html#gettext.NullTranslations.install | 10:48 |
| sambetts | it makes _ avaliable globally | 10:49 |
| *** saripurigopi has quit IRC | 10:49 | |
| lucasagomes | sambetts, a-ha | 10:49 |
| *** saripurigopi has joined #openstack-ironic | 10:49 | |
| lucasagomes | ironic/cmd/__init__.py:i18n.install('ironic') | 10:49 |
| lucasagomes | perhaps? | 10:49 |
| sambetts | I would guess so! | 10:50 |
| sambetts | so you don't have to import _ everytime! How weird, and also breaks if you ever import stuff from outside the ironic module... | 10:51 |
| lucasagomes | hah yeah, black magic | 10:51 |
| sambetts | :'( I understand python black magic this is voodoo :-P | 10:52 |
| lucasagomes | lol | 10:52 |
| *** dtantsur|brb is now known as dtantsur | 10:53 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Fix rootwrap configuration documentation https://review.openstack.org/227193 | 11:01 |
| dtantsur | sambetts, this patch ^^ Depends-On the fix | 11:01 |
| sambetts | :) | 11:02 |
| * rameshg87 goes home | 11:04 | |
| sambetts | cya rameshg87 | 11:04 |
| rameshg87 | sambetts: see you .. bye | 11:05 |
| *** rameshg87 has quit IRC | 11:05 | |
| *** deray has quit IRC | 11:10 | |
| *** Mark_Melb has joined #openstack-ironic | 11:13 | |
| *** Mark_Melb has quit IRC | 11:15 | |
| *** Mark_Melb has joined #openstack-ironic | 11:15 | |
| *** saripurigopi has quit IRC | 11:16 | |
| *** mgoddard has quit IRC | 11:16 | |
| *** mgoddard has joined #openstack-ironic | 11:16 | |
| Mark_Melb | Hi, is anyone around that could possibly confirm if a recent commit I am seeing to Nova scheduler may be behind an exception I am getting with Devstack launching a baremetal instance? | 11:17 |
| sambetts | Mark_Melb: https://review.openstack.org/#/c/227199/ | 11:20 |
| Mark_Melb | Bingo | 11:21 |
| Mark_Melb | Thank you @sambetts | 11:21 |
| ifarkas | dtantsur, lucasagomes, hi, could you please review https://review.openstack.org/#/c/222623/ ? | 11:25 |
| dtantsur | ifarkas, sure, just gimme some time | 11:26 |
| ifarkas | dtantsur, sure, thanks | 11:26 |
| *** achanda has joined #openstack-ironic | 11:29 | |
| TheJulia | Good morning | 11:30 |
| dtantsur | morning, TheJulia | 11:31 |
| *** chenglch has quit IRC | 11:37 | |
| *** Nisha_away has quit IRC | 11:38 | |
| *** alexpilotti has joined #openstack-ironic | 11:41 | |
| *** achanda has quit IRC | 11:41 | |
| *** stendulker has quit IRC | 11:44 | |
| sambetts | o/ TheJulia | 11:55 |
| *** baoli has joined #openstack-ironic | 11:56 | |
| solid-sinval | good morning Ironicers | 12:00 |
| openstackgerrit | Merged openstack/ironic: Add i18n _ import to cimc common https://review.openstack.org/227211 | 12:04 |
| dtantsur | morning solid-sinval | 12:08 |
| *** thrash has quit IRC | 12:08 | |
| *** romcheg has quit IRC | 12:08 | |
| *** romcheg has joined #openstack-ironic | 12:09 | |
| *** thrash has joined #openstack-ironic | 12:11 | |
| *** thrash has quit IRC | 12:11 | |
| *** thrash has joined #openstack-ironic | 12:11 | |
| solid-sinval | hey guys, what are the review priorities for today? i took a look at the launchpad but all priorities were already merged... | 12:15 |
| dtantsur | solid-sinval, https://review.openstack.org/227033 https://review.openstack.org/224938 | 12:16 |
| lucasagomes | dtantsur, the version of pyghmi we have packaged does not work | 12:19 |
| lucasagomes | dtantsur, that bug udi opened I could reproduce on host07 | 12:19 |
| solid-sinval | dtantsur, thanks | 12:19 |
| dtantsur | lucasagomes, facepalm.. | 12:19 |
| lucasagomes | tl;dr new version of pyghmi works | 12:19 |
| lucasagomes | yeah | 12:19 |
| *** dims__ has quit IRC | 12:19 | |
| lucasagomes | dtantsur, http://paste.openstack.org/show/473861/ | 12:19 |
| *** dims_ has joined #openstack-ironic | 12:20 | |
| lucasagomes | ifarkas, will do | 12:20 |
| *** trown|outttypeww is now known as trown | 12:23 | |
| ifarkas | lucasagomes, thanks | 12:25 |
| *** cdearborn has joined #openstack-ironic | 12:27 | |
| openstackgerrit | Merged openstack/ironic: Add 'abort' to state machine diagram https://review.openstack.org/226997 | 12:29 |
| *** nicodemos has joined #openstack-ironic | 12:37 | |
| *** jamielennox|away is now known as jamielennox | 12:37 | |
| sambetts | gerrit slow for anyone else today? Or just me again? | 12:38 |
| dtantsur | everyone | 12:38 |
| dtantsur | there are even posts on ML about it | 12:38 |
| sambetts | oh, i've not seen them, what tag did they have in the subject? | 12:40 |
| dtantsur | sambetts, [all] | 12:41 |
| *** liliars has joined #openstack-ironic | 12:41 | |
| sambetts | dtantsur: Ah they were taking a while to sync to my inbox, I see them now | 12:43 |
| lucasagomes | sambetts, it's very slow here | 12:49 |
| sambetts | :( | 12:50 |
| jroll | omg so slowwwwwwww | 12:53 |
| jroll | morning y'all | 12:53 |
| trown | o/ | 12:54 |
| sambetts | o/ jroll | 12:54 |
| lucasagomes | morning jroll | 12:55 |
| TheJulia | good morning jroll | 12:55 |
| jroll | hey lucasagomes TheJulia sambetts trown dtantsur \o | 12:56 |
| dtantsur | morning jroll, trown | 12:57 |
| dtantsur | jroll, at least ironic gate was fixed (unlike inspector one) | 12:58 |
| jroll | zigo: hey, in case you were starting on packaging dependencies yet, I just landed https://review.openstack.org/#/c/227165/ | 12:58 |
| jroll | dtantsur: :( | 12:58 |
| trown | ya thanks jroll for driving that with neutron folks last night | 12:58 |
| jroll | np! | 12:58 |
| jroll | so I think we have a +A on everything we need for 4.2.0, with the exception of release notes: https://review.openstack.org/#/c/227033/ | 12:59 |
| jroll | which I'm about to update | 12:59 |
| jroll | does anyone disagree? | 12:59 |
| zigo | jroll: Thans for the heads up. FYI, I have already version 2.1.4, so I'll do the update. | 13:00 |
| jroll | zigo: cool, thanks :) | 13:00 |
| zigo | jroll: What will be the release number for final Liberty? | 13:00 |
| zigo | FYI, I was very much against the new scheme for version numbers, as I think it's very confusing. | 13:01 |
| jroll | zigo: 4.2.0 is the proposed liberty release; I'm tagging it today. If we end up with backports, there will be a 4.2.1 and so on until the 'final' release | 13:01 |
| zigo | Ok, thanks. | 13:01 |
| jroll | sure thing :) | 13:01 |
| zigo | For now, I'm focussing on running tempest on an all-in-one deployement... | 13:01 |
| jroll | in general, I think you can just watch this file: https://github.com/openstack/releases/blob/master/deliverables/liberty/ironic.yaml | 13:02 |
| zigo | (packages based only, a bit like packstack, if you want, but for Debian, and running on a Xen domU) | 13:02 |
| trown | zigo: btw there are a couple late changes to inspector that will require packaging changes | 13:02 |
| jroll | whatever is there at 'liberty final' time should be right | 13:02 |
| jroll | zigo: fun, good luck :) | 13:02 |
| trown | zigo, https://review.gerrithub.io/#/c/248123/ and https://review.openstack.org/#/c/223665/ | 13:03 |
| zigo | The main issue is, since Kilo, Tempest changed *a lot*, so my setup script to edit the config file is to be reworked. | 13:03 |
| zigo | Oh, that's awesome that we have rootwrap support! | 13:04 |
| zigo | To tell the truth, I kept inspector to run as inspector user/group, in the hope it could make it upstream soon. | 13:04 |
| zigo | Thanks for working on this. | 13:05 |
| *** rameshg87 has joined #openstack-ironic | 13:05 | |
| dtantsur | now if we manage to release 2.2.0, you'll be able to use it.... | 13:07 |
| trown | whoops realized i linked to the packaging change rather than the code change: https://review.openstack.org/#/c/223930/ | 13:08 |
| rameshg87 | folks, can someone please have a look at https://review.openstack.org/#/c/224938/7 to see if how it looks for 4.2.0 | 13:09 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Ansible 2.0 compatibility https://review.openstack.org/196832 | 13:11 |
| jroll | rameshg87: why should we land this after bumping the feature? | 13:11 |
| rameshg87 | jroll: I wanted to check if it makes sense | 13:11 |
| rameshg87 | jroll: the effect is still same, it is disabled. but this gives an oppurtunity for someone to use it for homogenous hardware | 13:12 |
| rameshg87 | jroll: we have talked a lot about this, but does it make sense to at least enable it for experimental use ? | 13:12 |
| jroll | rameshg87: it doesn't really make sense without any driver/agent support :( | 13:12 |
| *** rbudden has joined #openstack-ironic | 13:12 | |
| jroll | rameshg87: it's also missing unit test changes | 13:13 |
| rameshg87 | jroll: it makes sense, all it needs is this patch for someone to use it for cleaning if they have a hardware manager and all of their hardware is having raid capability | 13:13 |
| *** shoutm has quit IRC | 13:13 | |
| rameshg87 | jroll: the current unit tests are sufficient | 13:13 |
| rameshg87 | it already has | 13:13 |
| jroll | hmm | 13:14 |
| jroll | I'm leaning towards no, but I'll defer to others | 13:14 |
| rameshg87 | oh may be if we really read from CONF or not, we mgiht need unit test | 13:14 |
| rameshg87 | I can add that .. | 13:14 |
| rameshg87 | jroll: okay, all I want is some feedback, ready to accept any :) | 13:14 |
| rameshg87 | jroll: brb | 13:14 |
| rameshg87 | lucasagomes: dtantsur: ^^^ | 13:15 |
| *** rameshg87 is now known as rameshg87-brb | 13:15 | |
| jroll | rameshg87-brb: yeah, I'm -0 right now, you should add unit tests asap in case others want to land it :) | 13:15 |
| *** rameshg87-brb is now known as rameshg87 | 13:19 | |
| dtantsur | rameshg87, jroll, didn't see the code yet, but if it enables RAID potentially, I'd say yes. just because we landed so much code already, we even exposed some APIs... | 13:19 |
| rameshg87 | dtantsur: okay | 13:20 |
| * rameshg87 goes to update the patch | 13:20 | |
| dtantsur | but yeah, let's talk with unit tests present :) | 13:20 |
| jroll | dtantsur: it's a small patch https://review.openstack.org/#/c/224938/7/ironic/drivers/modules/agent.py | 13:20 |
| jroll | just adds clean step priority for in-band raid stuff | 13:20 |
| jroll | devananda: rloo ^ your thoughts would be lovely here as well | 13:24 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add 4.2.0 release notes https://review.openstack.org/227033 | 13:27 |
| jroll | ^ WIP until we decide on rameshg87's patch but reviews welcome anyway | 13:28 |
| jroll | (if you can load it) | 13:28 |
| BadCub | morning folks | 13:29 |
| jroll | mornin' BadCub | 13:29 |
| BadCub | heya jroll | 13:29 |
| *** vinbs has quit IRC | 13:30 | |
| dtantsur | jroll, yeah, looks good to land, but half-dead gerrit doesn't allow me to vote... | 13:31 |
| jroll | ya :( | 13:32 |
| dtantsur | rameshg87, jroll, maybe update the options help strings to call RAID support "experimental"? | 13:33 |
| *** rloo has joined #openstack-ironic | 13:33 | |
| dtantsur | and then we can polish it in Mitaka and drop this comment | 13:33 |
| rameshg87 | dtantsur: ack | 13:33 |
| jroll | I'm good with that | 13:33 |
| sambetts | :'( gerrit giving me errors now | 13:36 |
| dtantsur | yeah, #infra is going to restart it | 13:37 |
| *** romainh has left #openstack-ironic | 13:37 | |
| *** Mark_Melb_ has joined #openstack-ironic | 13:38 | |
| *** Mark_Melb has quit IRC | 13:38 | |
| *** Mark_Melb_ is now known as Mark_Melb | 13:38 | |
| jroll | appears to be back \o/ | 13:38 |
| dtantsur | yeah | 13:39 |
| sambetts | woo \o/ | 13:39 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Make raid clean steps configurable https://review.openstack.org/224938 | 13:41 |
| jroll | woo | 13:41 |
| dtantsur | sambetts, nova patch got approved, fingers crossed! | 13:42 |
| rameshg87 | lucasagomes: mind helping me again with CONF thing if possible :) ^^ | 13:42 |
| jroll | rameshg87: weird conf thing | 13:42 |
| jroll | heh | 13:42 |
| sambetts | dtantsur: nice! The one you added Depends-On passed so lets hope! :D | 13:43 |
| lucasagomes | hi yeah... | 13:43 |
| * lucasagomes was looking at fedora pkg | 13:43 | |
| * lucasagomes dunno how to update a package there | 13:43 | |
| rameshg87 | jroll: I just noticed it | 13:43 |
| rameshg87 | jroll: I don't know why it happened :( | 13:43 |
| jroll | yeah :( | 13:43 |
| *** harshs has joined #openstack-ironic | 13:43 | |
| rameshg87 | [cisco_ucs] should have been up | 13:43 |
| NobodyCam | good morning From Palm Springs Ironicers | 13:43 |
| jroll | I can update it if you're busy lucasagomes | 13:43 |
| lucasagomes | rameshg87, do you have a fresh envrioment when you generate the config? | 13:43 |
| lucasagomes | jroll, it's cool | 13:43 |
| jroll | ok | 13:44 |
| dtantsur | NobodyCam, hey morning! | 13:44 |
| * lucasagomes updates the config | 13:44 | |
| NobodyCam | hey dtantsur nice to not be on the road | 13:45 |
| rameshg87 | lucasagomes: just created it today :( | 13:45 |
| dtantsur | :) | 13:45 |
| NobodyCam | :) | 13:45 |
| lucasagomes | rameshg87, odd... do you rm -r .tox | 13:45 |
| lucasagomes | and then create it again? | 13:45 |
| dtantsur | tox -r is enough :) | 13:46 |
| dtantsur | -r means rebuild | 13:46 |
| lucasagomes | a-ha will try that later | 13:46 |
| lucasagomes | I just remove it and recreate | 13:46 |
| rameshg87 | lucasagomes: yes | 13:46 |
| lucasagomes | idk I stopped trusting those tools by now | 13:46 |
| * lucasagomes trusts "rm" more | 13:46 | |
| dtantsur | lucasagomes, and this way you can rebuild only one: $ tox -r -eenv | 13:46 |
| jroll | rameshg87: left one more comment | 13:46 |
| dtantsur | lol | 13:46 |
| rameshg87 | jroll: checking again | 13:47 |
| lucasagomes | rameshg87, no idea then :-/ | 13:47 |
| TheJulia | good morning NobodyCam | 13:51 |
| rloo | hi lucasagomes, dtantsur, jroll, NobodyCam, rameshg87, sambetts, TheJulia and everyone else | 13:51 |
| lucasagomes | rloo, hi there | 13:51 |
| rameshg87 | rloo: o/ | 13:51 |
| rloo | so I guess I should look at that RAID patch... | 13:51 |
| jroll | rameshg87: also... do we want to sanity check that the create priority makes it come after the delete priority? | 13:51 |
| lucasagomes | jroll, I wonder if reload() is the right thing to do on tests? | 13:51 |
| lucasagomes | can't it mess up with another tests? | 13:52 |
| *** karimb has quit IRC | 13:52 | |
| liliars | morning rloo NobodyCam and everyone else (: | 13:52 |
| rameshg87 | lucasagomes: I guess reload isn't required | 13:52 |
| rameshg87 | lucasagomes: just checking | 13:52 |
| liliars | so I just checked this doc http://docs.openstack.org/developer/ironic/dev/architecture.html#drivers and it doesn't contain info about the boot interface when mentioning the existing driver interfaces | 13:52 |
| rameshg87 | I might have been wrong | 13:52 |
| rloo | morning liliars | 13:52 |
| lucasagomes | yeah it seems we should not do that | 13:52 |
| jroll | lucasagomes: yeah, not sure. it's weird because that CONF is in a decorator so it's loaded at module time | 13:52 |
| dtantsur | morning rloo, liliars | 13:52 |
| liliars | is it because not all drivers use this interface yet, or we should be mentioning it anyways? | 13:52 |
| rloo | liliars: it is cuz no one updated the docs | 13:52 |
| liliars | morning dtantsur | 13:52 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Ansible 2.0 compatibility https://review.openstack.org/196832 | 13:52 |
| lucasagomes | jroll, right but to test those create_configuration() and delete_configuration() we can just call it directly | 13:53 |
| lucasagomes | I think that test is testing the @clean_step decorator only | 13:53 |
| lucasagomes | which is already tested | 13:53 |
| liliars | rloo haha ok then I'm doing it | 13:53 |
| rloo | thx liliars | 13:53 |
| openstackgerrit | Merged openstack/ironic: Update proliantutils version required for L release https://review.openstack.org/227165 | 13:53 |
| jroll | lucasagomes: mmm, I see | 13:54 |
| NobodyCam | morning TheJulia, rloo, lucasagomes, jroll | 13:54 |
| rloo | liliars: please add RAID interface too. I think that's all we added... | 13:54 |
| NobodyCam | and everyone else not listed | 13:54 |
| jroll | heya NobodyCam :) | 13:55 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Make raid clean steps configurable https://review.openstack.org/224938 | 13:55 |
| lucasagomes | rameshg87, ^ with the config sample updated | 13:55 |
| rameshg87 | lucasagomes: it's required | 13:55 |
| * lucasagomes haven't changed the tests | 13:55 | |
| rameshg87 | lucasagomes: thanks | 13:55 |
| NobodyCam | oh so noice to not be traveling | 13:55 |
| rameshg87 | lucasagomes: just changing tests | 13:55 |
| lucasagomes | rameshg87, I don't think we should do that test at all tbh | 13:55 |
| lucasagomes | rameshg87, what is that testing? if the @clean_step decorator add the raid steps? | 13:55 |
| lucasagomes | that's only retesting the @clean_step decorator | 13:55 |
| rameshg87 | lucasagomes: it's testing if the priority comes from the CONF variable or not | 13:56 |
| lucasagomes | rameshg87, it does cause it's hardcoded in the code | 13:56 |
| lucasagomes | idk I think it may be dangerous to call reload | 13:56 |
| lucasagomes | specially with tests running in parallel | 13:57 |
| rameshg87 | lucasagomes: today, it's hardcoded, but patch changes it to a CONF variable | 13:57 |
| rameshg87 | lucasagomes: but testr runs in multiple processes iiuc | 13:57 |
| lucasagomes | what if test_get_clean_steps_configured() runs before than test_get_clean_steps_default() ? | 13:57 |
| liliars | rloo ok, will do | 13:57 |
| lucasagomes | try to change the order of those tests and run everything in a single thread | 13:57 |
| rameshg87 | lucasagomes: hmm yeah | 13:57 |
| *** achanda has joined #openstack-ironic | 13:57 | |
| rameshg87 | lucasagomes: so the test should reload the module back after changing the CONF variables back | 13:58 |
| rameshg87 | :( | 13:58 |
| rameshg87 | it is risky | 13:58 |
| rloo | rameshg87, lucasagomes, dtantsur, jroll: are you sure you want to add in these config values? I'm not convinced. | 13:58 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add 4.2.0 release notes https://review.openstack.org/227033 | 13:59 |
| *** achanda has quit IRC | 13:59 | |
| rloo | even though it says 'experimental', it makes it look like if you configure it, it will work (experimentally). but it won't. | 13:59 |
| rameshg87 | rloo: it at least enables it to be used on an experimental basis as it can cover one use case - people who have homogenous hardware all supporting raid | 13:59 |
| lucasagomes | rameshg87, fyi http://paste.openstack.org/show/473885/ | 13:59 |
| dtantsur | rloo, why not? we just need to warn about the limitations | 13:59 |
| lucasagomes | I don't thin the tests are needed, that's risky | 13:59 |
| jroll | rloo: we sohuld probably also add a note that your agent must support these methods | 13:59 |
| jroll | rloo: or just wait. | 13:59 |
| jroll | it would be nice to make it usable | 14:00 |
| rameshg87 | lucasagomes: dropping the tests | 14:00 |
| * rloo thinks about it | 14:00 | |
| *** baoli has quit IRC | 14:00 | |
| lucasagomes | rameshg87, thanks | 14:00 |
| rameshg87 | lucasagomes: thanks for catching it | 14:00 |
| dtantsur | just let's be very clear about what the current implementation supports | 14:00 |
| lucasagomes | rameshg87, no problem | 14:00 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add 4.2.0 release notes https://review.openstack.org/227033 | 14:01 |
| dtantsur | and what it does not support | 14:01 |
| jroll | ^ more clarity in release notes about this | 14:01 |
| jroll | should also add clarity to the confgis | 14:01 |
| jroll | configs, as well | 14:01 |
| *** baoli has joined #openstack-ironic | 14:01 | |
| *** karimb has joined #openstack-ironic | 14:01 | |
| * lucasagomes brb | 14:02 | |
| rloo | jroll, dtantsur, rameshg87, lucasagomes: when it is no longer experimental, what would it mean. that we'd need to make it backwards compatible with experimental behaviour? | 14:03 |
| rameshg87 | rloo: probably when it's no longer experimental, we will better the coverage | 14:03 |
| rameshg87 | rloo: today's limitation - when raid is enabled in cleaning, we can support only homogenous hardware all of which support raid | 14:04 |
| rloo | rameshg87: I mean, what if we change the behaviour. If we add more converage that is different. | 14:04 |
| dtantsur | jroll, could you also separately mention RAID interface in release notes? for example, I'd like to point that drivers can start implementing it now, not only via agent | 14:04 |
| lucasagomes | rloo, tbh I'm not sure... | 14:04 |
| jroll | dtantsur: ya | 14:04 |
| rameshg87 | rloo: probably when it is no longer experimental, it is applicable to all types of mixed hardware which supports raid and non-raid, we would ideally provide support for some mechanism of opt-out for non-raid hardware | 14:05 |
| lucasagomes | rloo, now it would be good to to say that experimental doesn't guarantee backward compat, just to be safe | 14:05 |
| rloo | rameshg87: i don't think you understand what I mean. lucasagomes ^^ answered my question. | 14:05 |
| dtantsur | yeah, ++ to warning about *potential* backward incompatibility | 14:06 |
| rloo | rameshg87: I don't want to have to muck around with warnings/whatever/microversion bumps to deal with backward incompatibilities | 14:06 |
| *** ionutbalutoiu has joined #openstack-ironic | 14:06 | |
| rameshg87 | okay | 14:06 |
| rloo | rameshg87: or if we have to muck around with that, we should all know that we will have to deal with it. | 14:07 |
| *** Mark_Melb has quit IRC | 14:07 | |
| rameshg87 | so that means capturing potential backward incompatibility in the docs ? | 14:07 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add 4.2.0 release notes https://review.openstack.org/227033 | 14:08 |
| rloo | my preference is not to put it in because of the unknown but anyway. | 14:08 |
| jroll | ^ covered in release notes | 14:08 |
| jroll | I can back all this out pretty quick if we decide not to do this, btw | 14:08 |
| rloo | rameshg87: right, we should be clear about what experimental means and what support or none in the future for it. | 14:08 |
| * jroll thinks hard about the implications | 14:08 | |
| rloo | honestly, we have a new versioning thing. we should just get it right and do a release. | 14:09 |
| rameshg87 | idk I will leave it upto you folks to decide. I just made a proposition :) | 14:09 |
| *** priteau has quit IRC | 14:09 | |
| rloo | i mean, i am torn cuz i'd like to get it in. i just don't know if that's a good enough reason. | 14:10 |
| rloo | if we look at nova-bare-metal as an example. was it a prototype then? | 14:10 |
| jroll | I'm still about a +0 on this | 14:10 |
| jroll | yeah, nova-bm was always marked experimental | 14:10 |
| jroll | and then yahoo went and started deploying it ;D | 14:11 |
| *** dims__ has joined #openstack-ironic | 14:11 | |
| rloo | jroll: shhhhh | 14:11 |
| jroll | hehehe | 14:11 |
| rloo | i'd like devananda to decide :) | 14:12 |
| jroll | +1 | 14:12 |
| *** dims_ has quit IRC | 14:14 | |
| *** shoutm has joined #openstack-ironic | 14:16 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Make raid clean steps configurable https://review.openstack.org/224938 | 14:17 |
| rameshg87 | lucasagomes: ^^ sorry to bother again. I just added a note on backward incompatibility | 14:17 |
| rameshg87 | lucasagomes: it just doesn't work for me | 14:17 |
| * rameshg87 promises I will attempt in fixing the config bug tomorrow | 14:17 | |
| lucasagomes | rameshg87, heh it's all good | 14:17 |
| lucasagomes | 1 sec | 14:17 |
| jroll | backward incompatibility, even for experimental things, makes me sad :( | 14:18 |
| jroll | rameshg87: in the future, 'git add -p' is your friend | 14:18 |
| jroll | you can choose which hunks to add | 14:18 |
| lucasagomes | jroll, me too, but I don't wanna promise backward compat for it | 14:18 |
| lucasagomes | if we have to I would prefer to not merge it | 14:18 |
| jroll | we don't have to do anything | 14:18 |
| jroll | but any incompatibility makes me sad | 14:18 |
| lucasagomes | yeah | 14:19 |
| rameshg87 | jroll: thanks for add -p :) | 14:19 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update docs for Fedora 22 https://review.openstack.org/208924 | 14:19 |
| jroll | I suspect devananda is going to agree that we shouldn't land it and we should just move on | 14:20 |
| lucasagomes | right | 14:20 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add 4.2.0 release notes https://review.openstack.org/227033 | 14:21 |
| jroll | ok, updated release notes assuming it won't land | 14:21 |
| *** cdearborn has quit IRC | 14:21 | |
| jroll | I can roll back a patchset if we decide to do so | 14:21 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Make raid clean steps configurable https://review.openstack.org/224938 | 14:22 |
| * lucasagomes updates sample config | 14:22 | |
| rameshg87 | lucasagomes: thanks | 14:22 |
| ionutbalutoiu | Hello again, guys! I'm starting a new implementation for a dhcp provider (mainly to be used for Ironic standalone). Will push it upstream after I finish the code. Briefly talked with lucasgomes yesterday about it. I'm using isc-dhcp-server service from Linux. So, when a node is started, the ironic-conductor writes in the config file the MAC - IP reservation for that node and reloads the dhcp service. I'm using a sql database as a | 14:22 |
| ionutbalutoiu | backend to store the dhcp reservation (I think is cleaner and it should allow us to have more than one DHCP server). But, I'm not sure how can I scale this out to provide HA/failover. Do you guys have any idea ? | 14:22 |
| lucasagomes | np | 14:23 |
| rameshg87 | jroll: why do you need to roll back patch set ? | 14:23 |
| rameshg87 | jroll: the latest one looks okay to me now. it just says RAIDInterface has been implemented | 14:23 |
| rameshg87 | jroll: we might also have to tag https://blueprints.launchpad.net/ironic/+spec/ironic-generic-raid-interface to 4.2.0 | 14:24 |
| jroll | rameshg87: the latest patch set assumes we don't land your patch, the previous assumes we do, look at the difference in the API version 1.12 part | 14:24 |
| rameshg87 | jroll: okay | 14:24 |
| jroll | rameshg87: deva decided not to tag that as 4.2.0 since it isn't usable, doesn't have docs, doesn't have client support | 14:24 |
| rameshg87 | okay | 14:24 |
| sambetts | ionutbalutoiu: I suggest sending an email out on the mailing list to prevent your request getting lost in the stream on here | 14:24 |
| jroll | ionutbalutoiu: I also agree head to the mailing list (or a spec) - at first thought you could run isc-dhcp in active/passive, talking to your DB, this would be all left to the deployer | 14:25 |
| *** marzif has joined #openstack-ironic | 14:26 | |
| *** jamielennox is now known as jamielennox|away | 14:26 | |
| *** ndipanov has joined #openstack-ironic | 14:26 | |
| rameshg87 | jroll: and one more thing | 14:28 |
| rameshg87 | jroll: https://blueprints.launchpad.net/ironic/+spec/remove-swift-dependency-for-ilo-drivers | 14:28 |
| rameshg87 | jroll: this is merged in 4.2.0 | 14:28 |
| rameshg87 | jroll: but this blueprint has some problem, it's not listing in ironic blueprints | 14:28 |
| * rameshg87 can't find why | 14:28 | |
| dtantsur | rameshg87, it does not have milestone target | 14:29 |
| dtantsur | nor series goal | 14:29 |
| rameshg87 | dtantsur: but at least it should get listed in https://blueprints.launchpad.net/ironic | 14:29 |
| jroll | rameshg87: there's two pages of ironic blueprints, maybe on second page? | 14:29 |
| jroll | I'll target it though, thanks | 14:30 |
| jroll | rameshg87: do you mind adding that to my release notes patch? :) | 14:30 |
| rameshg87 | jroll: will update | 14:30 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Ansible 2.0 compatibility https://review.openstack.org/196832 | 14:30 |
| jroll | thanks | 14:30 |
| rameshg87 | jroll: dtantsur: but please check if you see something wrong with that blueprint other than it doesn't have some fields | 14:30 |
| * rameshg87 suspects something else | 14:31 | |
| jroll | rameshg87: did you check page 2 on the blueprints page? | 14:31 |
| rameshg87 | jroll: yes, I did. it's not there. | 14:31 |
| jroll | huh. | 14:32 |
| *** mgoddard1 has joined #openstack-ironic | 14:32 | |
| jroll | wtf launchpad | 14:32 |
| rameshg87 | dtantsur: do you really want to re-iterate "no in-tree drivers....." | 14:32 |
| dtantsur | yep | 14:32 |
| rameshg87 | dtantsur: it's in the api section no | 14:32 |
| dtantsur | rameshg87, what is written there no I read as "this API has no effect", which is not true in all cases | 14:33 |
| jroll | lucasagomes: oops, this should have been targeted for some milestone, it's done right? https://blueprints.launchpad.net/ironic/+spec/ipxe-dynamic-config | 14:33 |
| *** mgoddard has quit IRC | 14:34 | |
| rameshg87 | dtantsur: right. some out-of-tree driver can still implement it | 14:34 |
| * jroll is going to totally overhaul our launchpad blueprints page next week | 14:34 | |
| lucasagomes | jroll, it's not completed no :-( | 14:34 |
| lucasagomes | jroll, 4.3.0 | 14:34 |
| jroll | orly | 14:34 |
| jroll | ok | 14:34 |
| * lucasagomes had to complete abandon that to prioritize other stuff | 14:34 | |
| *** marzif_ has joined #openstack-ironic | 14:34 | |
| jroll | I just saw those patches were merged, I didn't look at what they did :P | 14:34 |
| jroll | I guess it's just plumbing stuff | 14:34 |
| lucasagomes | jroll, the base stuff is done, just need to work on the drivers to return the pxe config as a string | 14:34 |
| lucasagomes | yeah | 14:34 |
| jroll | yep | 14:34 |
| jroll | cool thanks | 14:35 |
| lucasagomes | from now on should be fine, I just didn't have the time | 14:35 |
| jroll | rameshg87: I -2'd for now to make sure that doesn't accidentally land, please do keep working on it | 14:36 |
| jroll | stepping away for a bit | 14:36 |
| rameshg87 | jroll: sure | 14:36 |
| *** marzif has quit IRC | 14:37 | |
| rameshg87 | rloo: https://review.openstack.org/#/c/227033/8/doc/source/releasenotes/index.rst L68 | 14:38 |
| rameshg87 | rloo: you want to add something at the end | 14:38 |
| rameshg87 | something like | 14:38 |
| rameshg87 | how much RAM to reserve67 for running the agent when doing this check before starting deploy. | 14:39 |
| rameshg87 | ?? | 14:39 |
| rameshg87 | s/67// | 14:39 |
| jroll | dtantsur: so going back to this out-of-tree thing... I think that 1) even if an out-of-tree driver implements something, it still isn't quite 'supported' because we don't have zapping (of course they could do it as cleaning); 2) 99% of our users won't be using an out-of-tree driver, I don't want to confuse them | 14:39 |
| rloo | rameshg87: did i ever say i hate documentation? | 14:39 |
| rloo | rameshg87: thinking | 14:39 |
| jroll | I'm not opposed to adding it, I just think it's a bit odd | 14:39 |
| dtantsur | jroll, well, I'm trying to be precise on it | 14:39 |
| jroll | yeah, I guess I'll change it | 14:40 |
| rloo | rameshg87: When ironic does this check, it can factor in the RAM to reserve for running the agent via the optional config... ? | 14:40 |
| *** garthb has joined #openstack-ironic | 14:40 | |
| rameshg87 | okay | 14:41 |
| jroll | rloo: When ironic does this check, this config option may be set to factor in the amount of RAM to reserve for running the agent? | 14:42 |
| rloo | rameshg87: and could also s/As such,/As such, there must be enough RAM for running the agent and to fit the image/ ? | 14:42 |
| * rameshg87 feels we need etherpad editing | 14:42 | |
| rloo | jroll: yeah, that's good enough. maybe with my other suggestion. should be obvious then. | 14:43 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add 4.2.0 release notes https://review.openstack.org/227033 | 14:43 |
| * rloo is thinking about a career in tech writing. NOT. | 14:43 | |
| jroll | ^ there you are | 14:44 |
| jroll | lol | 14:44 |
| dtantsur | sambetts, ifarkas, gate fixing patch has merged, could you review the remaining patches: https://review.openstack.org/226873 and https://review.openstack.org/227193 | 14:44 |
| sambetts | dtantsur: w00p will do | 14:44 |
| sambetts | :D | 14:44 |
| *** mtanino has joined #openstack-ironic | 14:45 | |
| mordred | jroll, devananda: I was just chatting with TheJulia about a patch that just got landed in ironicclient/master | 14:47 |
| jroll | mhmmm? | 14:47 |
| mordred | that needs a backport into stable/liberty | 14:47 |
| jroll | mordred: https://github.com/openstack/python-ironicclient/commit/5afd4833468ff25d0071e445561881e1d560287e ? | 14:47 |
| mordred | yup | 14:48 |
| jroll | got a backport proposed? | 14:48 |
| NobodyCam | morning mordred :) | 14:48 |
| BadCub | hiya mordred :) | 14:48 |
| mordred | nah - I think TheJulia is working on it - I just wanted to pipe up about it and make sure it was reasonable to folks before I landed a shade patch which requires the ironicclient patch | 14:48 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add 4.2.0 release notes https://review.openstack.org/227033 | 14:48 |
| rameshg87 | jroll: ^^ | 14:48 |
| jroll | mordred: I don't see a reason why not | 14:49 |
| mordred | cool | 14:49 |
| mordred | morning BadCub NobodyCam | 14:49 |
| * rameshg87 needs to leave for the day | 14:49 | |
| rameshg87 | have a good day folks | 14:49 |
| *** rameshg87 has quit IRC | 14:49 | |
| NobodyCam | :) | 14:49 |
| NobodyCam | night rameshg | 14:49 |
| dtantsur | jroll, mordred, last time I asked devananda he told me that ironicclient backports are only for fixing the gate.. | 14:50 |
| jroll | dtantsur: seems like a policy like that should be written down :) | 14:51 |
| dtantsur | yeah | 14:51 |
| jroll | idk, maybe not then, I'd love to hear the reasoning | 14:51 |
| jroll | I'll leave it to him | 14:51 |
| *** cdearborn has joined #openstack-ironic | 14:51 | |
| ifarkas | dtantsur, sure thing | 14:53 |
| rloo | jroll: one of the problems with LP, is that it requires manual updating. Sigh. This was approved: https://blueprints.launchpad.net/ironic/+spec/bare-metal-trust-using-intel-txt | 14:53 |
| jroll | rloo: it didn't land yet, right? | 14:54 |
| jroll | rloo: I'm going to clean up all the things after this release... but if you see a BP that's implemented please do let me know | 14:54 |
| rloo | jroll: no. partially (maybe) cuz I didn't know/remember to look at the other patch. | 14:54 |
| jroll | heh | 14:54 |
| jroll | ok, I'll deal with it later then :) | 14:54 |
| TheJulia | jroll: mordred https://review.openstack.org/#/c/227347/ | 14:55 |
| rloo | jroll: well, seriously. i feel bad about that. | 14:55 |
| *** tsekiyama has joined #openstack-ironic | 14:56 | |
| jroll | rloo: yeah, I know the feeling. we need to get organized. | 14:56 |
| rloo | jroll: I think this may be done even though we never approved it: https://blueprints.launchpad.net/ironic/+spec/use-oslo.versionedobjects | 14:57 |
| *** mgoddard1 has quit IRC | 14:57 | |
| jroll | rloo: it's hard to say | 14:57 |
| dtantsur | at least big part of it is done | 14:57 |
| rloo | jroll: and/or maybe it includes rolling upgrades :). Ok, I'll stop looking at them. | 14:57 |
| jroll | rloo: I'd like to just wait until this release is done to look at this stuff | 14:57 |
| jroll | heh, right | 14:57 |
| jroll | there's still work to do, there's no spec or whatever so idk what the scope is. | 14:58 |
| *** mgoddard has joined #openstack-ironic | 14:58 | |
| rloo | jroll: yeah, but we've been merging patches related to that :-( | 14:58 |
| dtantsur | rloo, jroll, we can say it's done, as we did migrate to oslo.ov. maybe some work is to be done, but the essence of the bp is finished | 14:58 |
| jroll | dtantsur: "to help dealing with upgrades" isn't done. | 14:59 |
| dtantsur | ah, this part | 14:59 |
| jroll | maybe it is | 14:59 |
| jroll | hard to tell :) | 14:59 |
| jroll | ok, actually stepping away now, bbiab | 15:02 |
| *** marzif_ has quit IRC | 15:03 | |
| rloo | easy doc change to review for the release: https://review.openstack.org/#/c/227029/ | 15:04 |
| rloo | (only easy if you agree :-)) | 15:04 |
| *** shoutm has quit IRC | 15:05 | |
| *** marzif_ has joined #openstack-ironic | 15:07 | |
| *** harshs has quit IRC | 15:09 | |
| dtantsur | rloo, s/look at earlier versions/look at the released version/ ? | 15:09 |
| dtantsur | rloo, also, released version will also contain the same disclaimer :D | 15:10 |
| rloo | dtantsur: well, it depends on whether they want a not-current-master but not-yet-released version. | 15:10 |
| rloo | dtantsur: basically it is a mess | 15:10 |
| dtantsur | oh yeah, hmm... | 15:10 |
| rloo | dtantsur: if we assume they are interested in current-master or a released version. which might be a valid assumption. | 15:10 |
| rloo | dtantsur: oh, that string was already there, I didn't add it. | 15:12 |
| dtantsur | yeah, I know | 15:12 |
| rloo | dtantsur: i think it is fine. if we say 'released version' folks will ask, where are the released versions. vague might be better now. | 15:13 |
| rloo | dtantsur: i really want some url .. ironic/4.1.0/install-guide... | 15:13 |
| dtantsur | +100500! | 15:15 |
| dtantsur | I have to point people to github stable/kilo... | 15:15 |
| rloo | dtantsur: and tell them how to generate? | 15:15 |
| dtantsur | rloo, no, github does some basic rendering | 15:15 |
| rloo | dtantsur: oh, that's good. i don't think i've ever tried looking at our docs that way. | 15:16 |
| sambetts | dtantsur: it would be nice if git.openstack.org rendered rsts like github does | 15:16 |
| dtantsur | yeah | 15:17 |
| sambetts | dtantsur: although it can't render the sphinx specfic magic | 15:17 |
| dtantsur | rloo, I had to, as I had to point people at Kilo version of UEFI docs | 15:17 |
| *** ionutbalutoiu has quit IRC | 15:17 | |
| *** ionutbalutoiu has joined #openstack-ironic | 15:17 | |
| dtantsur | sambetts, that's true, still better than nothing | 15:17 |
| sambetts | dtantsur: definatly :D | 15:17 |
| rloo | dtantsur: (and then hope we updated the doc by the time we cut the stable/x branch) | 15:18 |
| dtantsur | yeah... | 15:18 |
| *** marzif_ has quit IRC | 15:18 | |
| openstackgerrit | Lilia Sampaio proposed openstack/ironic: Update architecture docs to mention new driver interfaces https://review.openstack.org/227370 | 15:20 |
| sambetts | ifarkas: Any reason why you didn't +A the inspector devstack patch? I'm just reviewing it now | 15:20 |
| ifarkas | sambetts, yes, because of failing check | 15:21 |
| sambetts | ifarkas: Ah, cool, we'll just wait for jenkins then :) | 15:22 |
| ifarkas | sambetts, yup ;-) | 15:23 |
| dtantsur | ifarkas, sambetts, currently our gate and check jobs are the same, so it's safe to approve any patch | 15:23 |
| dtantsur | if it's broken, it won't pass the gate | 15:23 |
| dtantsur | just FYI :) | 15:23 |
| *** alexpilotti has quit IRC | 15:23 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Fix rootwrap configuration documentation https://review.openstack.org/227193 | 15:24 |
| dtantsur | sambetts, ^^ | 15:24 |
| ifarkas | dtantsur, sambetts, ok, approved | 15:24 |
| sambetts | :D | 15:25 |
| dtantsur | now I hope we won't get too many merge conflicts :D | 15:25 |
| *** shoutm has joined #openstack-ironic | 15:28 | |
| dtantsur | going for now, will get back to check if everything is merged. we'll probably have to release tomorrow (yeah, release on Friday \o/) | 15:29 |
| *** [1]cdearborn has joined #openstack-ironic | 15:29 | |
| dtantsur | (that's about inspector release ofc) | 15:30 |
| sambetts | \o/ :D | 15:30 |
| rloo | bye dtantsur | 15:30 |
| *** dtantsur is now known as dtantsur|afk | 15:31 | |
| *** harshs has joined #openstack-ironic | 15:31 | |
| *** lsmola has quit IRC | 15:32 | |
| *** Nisha_away has joined #openstack-ironic | 15:34 | |
| rloo | the install guide doesn't mention bifrost, ironic-webclient. Do we want it to? TheJulia? | 15:36 |
| *** getvasanth has quit IRC | 15:37 | |
| lucasagomes | JayF, hi there? Do you have any experience with ipv6 and IPMI? | 15:38 |
| lucasagomes | JayF, do you know if it's common for hardware vendors to allow using IPv6 for their OOBM? | 15:39 |
| BadCub | hiya rloo and lucasagomes :) | 15:40 |
| lucasagomes | BadCub, hi there! | 15:40 |
| rloo | jroll: wrt trusted boot BP - we have documentation on that. So I don't know what is or is not done: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#trusted-boot-with-partition-image | 15:40 |
| rloo | morning BadCub | 15:40 |
| *** krtaylor has quit IRC | 15:41 | |
| BadCub | how have things been in Ironicville? | 15:41 |
| rloo | jlvillal, lintan: either of you know what the status is for trusted boot? | 15:41 |
| *** romcheg has quit IRC | 15:42 | |
| *** cdearborn has quit IRC | 15:42 | |
| *** trown is now known as trown|lunch | 15:43 | |
| jlvillal | rloo, I think all of lintan's patches have landed. | 15:43 |
| *** puranamr has joined #openstack-ironic | 15:43 | |
| *** ifarkas has quit IRC | 15:43 | |
| jlvillal | So I believe it is done. | 15:43 |
| rloo | jlvillal: we haven't mentioned it at all in any of the release notes, and the BP wasn't even marked as approved :-( | 15:44 |
| rloo | jlvillal: the BP shows another patch. | 15:44 |
| jlvillal | rloo, The spec was approved though. | 15:44 |
| openstackgerrit | Zhao Lei proposed openstack/ironic-python-agent: Use cloud-config.yaml instead of cloud-config.yml for coreos https://review.openstack.org/210355 | 15:45 |
| rloo | jlvillal: https://blueprints.launchpad.net/ironic/+spec/bare-metal-trust-using-intel-txt | 15:45 |
| jlvillal | rloo, lintan knows all the details. I unfortunately do not :( | 15:45 |
| *** harshs has quit IRC | 15:45 | |
| rloo | jlvillal: there is a patch there 'Support trusted boot with iPXE' that hasn't merged | 15:45 |
| rloo | jlvillal: just wondering if we should put something in the release notes. | 15:46 |
| jlvillal | rloo, bare-metal-trust-using-intel-txt.rst | 15:46 |
| rloo | jlvillal: ? is that the spec? | 15:47 |
| jroll | rloo: I'm curious what cycle that stuff landed too :/ | 15:47 |
| jlvillal | rloo, https://github.com/openstack/ironic-specs/blob/master/specs/liberty/bare-metal-trust-using-intel-txt.rst | 15:47 |
| jlvillal | Which references the blueprint you mentioned | 15:47 |
| rloo | jlvillal: yeah, i know the spec landed. and i know i reviewed the first patch. | 15:47 |
| rloo | jlvillal: yeah, the BP should have been marked approved. that was an oversight. | 15:48 |
| NobodyCam | mornign jlvillal :) | 15:48 |
| jlvillal | NobodyCam, morning | 15:48 |
| rloo | jlvillal: the question is whether it is done, should we mention anything in the release notes? | 15:48 |
| *** puranamr has quit IRC | 15:48 | |
| NobodyCam | :) | 15:48 |
| BadCub | hiya jlvillal | 15:48 |
| jlvillal | rloo, I believe it is done. I have seen status reports saying it is done. | 15:48 |
| rloo | jlvillal: the default (right now) is no mention. | 15:48 |
| jlvillal | BadCub, Good morning. | 15:48 |
| rloo | jroll: ^^ release meister, will leave it for you to add or not, something about trusted boot :) | 15:49 |
| jlvillal | rloo, But need lintan to confirm it | 15:49 |
| jlvillal | rloo, Though it does appear one patch is still not merged that seems related to it. iPXE https://review.openstack.org/207278 (which you mentioned). I don't see any other patches from lintan that seem related to trusted boot | 15:51 |
| jroll | rloo: it appears it was released with 4.0.0, I think? | 15:51 |
| jroll | rloo: I'm inclined to wait for lintan and then update the release notes later | 15:51 |
| jroll | also, seems like we should have ipxe support before calling it done. idk. | 15:52 |
| jlvillal | jroll, Would you agree that AMT doesn't yet work with IPA? A driver is needed to be created first, is that correct? | 15:53 |
| jroll | jlvillal: let me look | 15:53 |
| *** krtaylor has joined #openstack-ironic | 15:54 | |
| jroll | jlvillal: it appears you are correct | 15:54 |
| jlvillal | jroll, Thanks. I will give mrda the credit :) He told me we need to create an agent_amt driver. | 15:54 |
| jroll | jlvillal: yep. we should do that. | 15:56 |
| jroll | jlvillal: something like https://github.com/openstack/ironic/blob/master/ironic/drivers/pxe.py#L285 | 15:56 |
| jlvillal | jroll, Thanks | 15:57 |
| *** thiagop has joined #openstack-ironic | 16:00 | |
| thiagop | Good morning folks | 16:00 |
| NobodyCam | mornign thiagop | 16:00 |
| *** degorenko has quit IRC | 16:00 | |
| thiagop | Hey NobodyCam! | 16:01 |
| devananda | gmorning all | 16:01 |
| thiagop | morning devananda | 16:01 |
| *** degorenko has joined #openstack-ironic | 16:01 | |
| jroll | devananda: \o/ | 16:02 |
| jroll | I have things for you | 16:02 |
| devananda | great! I like things :) | 16:02 |
| jroll | 1) opinions on landing https://review.openstack.org/#/c/224938/ as 'super-experimental and may have incompatible changes later' - I'm leaning toward no | 16:02 |
| jroll | 2) updated release notes: https://review.openstack.org/#/c/227033/10 | 16:02 |
| *** marzif has joined #openstack-ironic | 16:03 | |
| jroll | 3) TheJulia has a client backport already, but idk our client backport policy, dtantsur|afk mentioned gate fixes only? https://review.openstack.org/#/c/227347/ | 16:03 |
| jroll | dat's it :) | 16:03 |
| *** puranamr has joined #openstack-ironic | 16:04 | |
| devananda | 3) why backport? why not release new? | 16:04 |
| devananda | 1 & 2 -- looking | 16:04 |
| jroll | devananda: because apparently stable clients are a thing people care about or something | 16:04 |
| jroll | we should also release new, but ya know | 16:04 |
| lucasagomes | thiagop, yo morning | 16:05 |
| lucasagomes | devananda, morning too :-) | 16:06 |
| lucasagomes | folks I'm heading home earlier today | 16:06 |
| thiagop | lucasagomes: o/ | 16:06 |
| lucasagomes | came to the office today | 16:06 |
| lucasagomes | have a good night everyone! | 16:06 |
| thiagop | g'night | 16:06 |
| sambetts | cya lucasagomes | 16:07 |
| sambetts | o/ devananda | 16:07 |
| devananda | jroll: is there a particular bug / release in question here? | 16:07 |
| jroll | devananda: for the backport? | 16:08 |
| jroll | night lucasagomes | 16:08 |
| devananda | I believe dtantsur|afk 's recollection matches mine. I don't think packagers are doing new point releases for older openstack clients .... zigo ? | 16:08 |
| *** puranamr has quit IRC | 16:08 | |
| devananda | g'night, lucasagomes ! | 16:08 |
| *** derekh has quit IRC | 16:09 | |
| zigo | devananda: Point releases of clients? Is there such a thing upstream? | 16:09 |
| devananda | zigo: client libs have been semver for a while, but AFAIK no one's doing stable branches of them | 16:10 |
| zigo | I already barely have enough time to work on CVEs and such, so ... | 16:10 |
| devananda | heh | 16:10 |
| jroll | devananda: there are stable branches | 16:11 |
| devananda | i'll take that as a no :) | 16:11 |
| jroll | I'm goign to poke my head in relmgr office | 16:11 |
| devananda | jroll: yes, for gate reasons. | 16:11 |
| openstackgerrit | Merged openstack/ironic-inspector: Add alembic migrations for the inspector database https://review.openstack.org/223665 | 16:11 |
| openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/226805 | 16:11 |
| devananda | like dependency resolving | 16:11 |
| jroll | devananda: yeah, double checking with rel mgmt people but you're probably right | 16:11 |
| *** ndipanov has quit IRC | 16:11 | |
| devananda | jroll: cool, ty | 16:12 |
| jroll | in the meantime, I'll look at releasing a client | 16:12 |
| *** rloo has quit IRC | 16:12 | |
| TheJulia | \o/ | 16:12 |
| *** lucasagomes has quit IRC | 16:12 | |
| * TheJulia goes back to conference call | 16:12 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Expanded clean command. https://review.openstack.org/227404 | 16:13 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added phantomjs as dependency https://review.openstack.org/227405 | 16:13 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Upgraded bower dependencies https://review.openstack.org/227406 | 16:13 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added "gulp package" to pretest https://review.openstack.org/227407 | 16:13 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Split storage components into individual files https://review.openstack.org/227408 | 16:13 |
| openstackgerrit | Merged openstack/ironic-inspector: Support IPA in raid_device plugin https://review.openstack.org/225658 | 16:13 |
| devananda | jroll: re: RAID clean patch, this looks like something we'll have to break to make generic for the RAID interface as a whole | 16:14 |
| *** mgoddard has quit IRC | 16:14 | |
| mariojv | good morning ironic! | 16:14 |
| jroll | devananda: yeah, I'm inclined not to land it | 16:14 |
| openstackgerrit | Merged openstack/ironic: Update docs for Fedora 22 https://review.openstack.org/208924 | 16:14 |
| thiagop | morning mariojv | 16:14 |
| mariojv | does anyone know why this might be happening? http://logs.openstack.org/58/226058/2/check/gate-tempest-dsvm-ironic-agent_ssh/89207e4/logs/screen-ir-cond.txt.gz#_2015-09-24_12_47_39_408 | 16:14 |
| mariojv | in the context of this patch: https://review.openstack.org/#/c/226058/ | 16:14 |
| jroll | mariojv: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L371 | 16:14 |
| *** mgoddard has joined #openstack-ironic | 16:15 | |
| mariojv | ah, that works | 16:15 |
| mariojv | didn't see that class, thanks jroll | 16:15 |
| *** bcornec has quit IRC | 16:15 | |
| jroll | yeah np | 16:15 |
| jroll | that base class is shared by agent.py and pxe.py iirc | 16:15 |
| jroll | devananda: woo, this release is going to have some osc commands | 16:20 |
| devananda | nice! | 16:21 |
| *** mgoddard1 has joined #openstack-ironic | 16:21 | |
| devananda | jroll: release notes are looking better | 16:22 |
| JayF | jroll: you see the openstack-dev post? | 16:23 |
| JayF | jroll: someone wants to build an external dhcp provider for isc-dhcp-server | 16:23 |
| devananda | do we have any specific upgrade instructions forthis release? (not afaik) | 16:23 |
| JayF | jroll: only sad thing is his prototype is using a secondary db outside of ironic | 16:23 |
| jroll | JayF: I asked them to post on the ML \o/ | 16:23 |
| devananda | also - we should perhaps have a section in our docs dedicated to "how to upgrade safely" | 16:23 |
| jroll | devananda: not that I know of | 16:23 |
| JayF | jroll: I asked him to come to IRC in my reply | 16:23 |
| jroll | JayF: but I haven't read it yet | 16:23 |
| JayF | jroll: so lololol looping replies | 16:23 |
| jroll | hah | 16:23 |
| jroll | well, we're busy with release stuff | 16:23 |
| jroll | I'm not going to help architect that today heh | 16:23 |
| *** notq has quit IRC | 16:24 | |
| *** gabriel-bezerra has quit IRC | 16:24 | |
| *** mjturek1 has quit IRC | 16:24 | |
| *** x3k` has joined #openstack-ironic | 16:24 | |
| *** mgoddard has quit IRC | 16:24 | |
| *** nicodemos has quit IRC | 16:24 | |
| *** x3k has quit IRC | 16:24 | |
| *** purp has quit IRC | 16:24 | |
| *** kbyrne has quit IRC | 16:24 | |
| *** purp has joined #openstack-ironic | 16:24 | |
| *** kbyrne has joined #openstack-ironic | 16:24 | |
| JayF | jroll: yeah; I just gave the basic "put the code up even if it might not merge, file a spec if it's important, come chat w/us in IRC" | 16:24 |
| jroll | yah | 16:24 |
| JayF | I do think that's one thing we did well in early days of agent | 16:25 |
| *** gabriel-bezerra has joined #openstack-ironic | 16:25 | |
| JayF | having all our patches upstream | 16:25 |
| *** nicodemos has joined #openstack-ironic | 16:25 | |
| devananda | jroll: everything on https://launchpad.net/ironic/+milestone/4.2.0 is green -- are we waiting for anything besides the release notes patch to land now? | 16:25 |
| *** mjturek1 has joined #openstack-ironic | 16:25 | |
| jroll | devananda: just rel notes, assuming we don't land that raid thing | 16:25 |
| BadCub | morning devananda :) | 16:25 |
| devananda | BadCub: g'morning! | 16:25 |
| devananda | jroll: I -1'd the raid thing too | 16:25 |
| jroll | JayF: what's also interesting is that yahoo is working on an isc-dhcp backend for neutron | 16:26 |
| jroll | anyway, need to defer my thuoghts and excitement to later on this :P | 16:26 |
| jroll | devananda: cool, let's work out the release notes then. not sure if there are docs where ruby and myself commented | 16:26 |
| sambetts | dtantsur|afk: We're all green for 2.2.0 :) | 16:26 |
| * JayF finds and reviews said release notes patch | 16:27 | |
| devananda | jroll: thoughts on landing 227029 now as well? | 16:29 |
| *** puranamr has joined #openstack-ironic | 16:29 | |
| devananda | I mean, you +2'd but didn't +A it | 16:29 |
| jroll | devananda: I was first +2 | 16:29 |
| devananda | ah | 16:30 |
| * devananda pushes the button | 16:30 | |
| jroll | thanks | 16:30 |
| *** yog_ has quit IRC | 16:33 | |
| devananda | CIMS has docs | 16:35 |
| devananda | jroll: are you updating the patch, or shall I? | 16:35 |
| jroll | devananda: either way | 16:36 |
| jroll | I have a meeting at 10 | 16:36 |
| jroll | devananda: if you have links already go for it | 16:37 |
| devananda | k | 16:37 |
| *** karimb has quit IRC | 16:40 | |
| devananda | Nisha_away: is there any documentation up for https://blueprints.launchpad.net/ironic/+spec/remove-swift-dependency-for-ilo-drivers ? | 16:42 |
| Nisha_away | devananda, No as of now it is not there...will push it by today | 16:42 |
| devananda | Nisha_away: ok, thanks. we'll link to the general ilo driver doc page for now, then | 16:43 |
| Nisha_away | Ok | 16:43 |
| *** trown|lunch is now known as trown | 16:45 | |
| *** ionutbalutoiu has quit IRC | 16:45 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Add 4.2.0 release notes https://review.openstack.org/227033 | 16:52 |
| jroll | woo | 16:52 |
| devananda | I didn't add upgrade notes cause, well, I haven't tested that upgrade yet | 16:54 |
| openstackgerrit | Merged openstack/ironic-inspector: devstack: allow nodes to be already MANAGEABLE in exercise https://review.openstack.org/226873 | 16:54 |
| devananda | we should do that soon tho | 16:54 |
| jroll | devananda: yeah, I'd say that should also be a separate doc | 16:55 |
| jroll | I +2d that release notes change because excite | 16:55 |
| jroll | should probably get 2 other +2 | 16:55 |
| jroll | where's rloo :( | 16:55 |
| devananda | we've all written parts of it, lol. /me adds a +2 | 16:55 |
| NobodyCam | morning devananda | 16:56 |
| devananda | g'morning! | 16:56 |
| jroll | meeting, be back in an hour | 16:57 |
| devananda | ciao! | 16:57 |
| NobodyCam | devananda: have a free minute to take a look at a stable/kilo backport? https://review.openstack.org/#/c/222544 | 16:59 |
| jroll | is that a bug in kilo? | 17:00 |
| jroll | I thought that broke with the boot interface or something :( | 17:00 |
| jroll | amazing :( | 17:00 |
| *** achanda has joined #openstack-ironic | 17:00 | |
| *** Nisha_away has quit IRC | 17:00 | |
| *** puranamr has quit IRC | 17:01 | |
| NobodyCam | :p | 17:01 |
| *** marzif has quit IRC | 17:05 | |
| devananda | NobodyCam: replied with -1 for process issues | 17:05 |
| *** marzif has joined #openstack-ironic | 17:05 | |
| NobodyCam | thank you devananda :) | 17:05 |
| devananda | dtantsur|afk: I left feedback on https://review.openstack.org/#/c/196037/ a while ago -- do you still care about this kilo backport? it looks like a reasonable fix to me, just -1'd for process reasons | 17:10 |
| *** penick has joined #openstack-ironic | 17:11 | |
| *** shoutm has quit IRC | 17:11 | |
| TheJulia | seems like it should almost be git review -X instead of git cherry-pick -x in the instructions | 17:15 |
| *** nicodemos has quit IRC | 17:17 | |
| *** MattMan has quit IRC | 17:19 | |
| *** puranamr has joined #openstack-ironic | 17:22 | |
| *** rloo has joined #openstack-ironic | 17:23 | |
| *** puranamr has quit IRC | 17:26 | |
| *** harshs has joined #openstack-ironic | 17:30 | |
| rloo | jroll, devananda: wrt release notes https://review.openstack.org/#/c/227033/, you're OK if we don't have 'key new features', 'known issues' and 'upgrade notes' like in previous 20XY.Z release notes? | 17:30 |
| openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Allow hardware managers to override clean step priority https://review.openstack.org/189969 | 17:31 |
| *** ijw has joined #openstack-ironic | 17:32 | |
| *** achanda has quit IRC | 17:36 | |
| jlvillal | JoshNang: Good to see you on your final day :) | 17:41 |
| JoshNang | :D | 17:41 |
| JoshNang | figured i'd try to close out the simple patches i have left over | 17:42 |
| jlvillal | Thanks! | 17:42 |
| krotscheck | JoshNang: Oh? Where're you headed? | 17:42 |
| *** puranamr has joined #openstack-ironic | 17:43 | |
| JoshNang | krotscheck: triggr health in chicago. an app for helping people in drug/alcohol rehab | 17:44 |
| krotscheck | JoshNang: Gotcha! So... no more javascript reviews from you then, eh? | 17:44 |
| krotscheck | :) | 17:44 |
| JoshNang | nope, sorry :( | 17:44 |
| JoshNang | though i'll be doing more javascript dev at the new place! | 17:45 |
| krotscheck | JoshNang: No worries, I'll just make jroll learn javascript :) | 17:45 |
| jroll | krotscheck: I know javascript! | 17:45 |
| JoshNang | heh i think he knows it...just doesn't want to know it :P | 17:45 |
| jroll | I'm just crusty :) | 17:45 |
| jroll | like idk ES6 | 17:45 |
| krotscheck | JoshNang: That's more or less the case for all of openstack ;) | 17:45 |
| JoshNang | heh | 17:45 |
| krotscheck | jroll: Prove it! There's a stack of patches that starts here -> https://review.openstack.org/#/c/227404/ :) | 17:46 |
| krotscheck | :D | 17:46 |
| jroll | krotscheck: D: | 17:46 |
| krotscheck | So, who's my second +2 going to be? | 17:46 |
| * jroll runs away | 17:46 | |
| jroll | krotscheck: I will get to it though | 17:46 |
| jroll | later. too much code review this week. | 17:47 |
| krotscheck | jroll: No rush. I'm still... 2 patches away, maybe 3, from making node provisioning work. | 17:47 |
| jroll | great question, I still think we need a core group for webclient | 17:47 |
| TheJulia | krotscheck: I would, but I'm not in the right group :) | 17:47 |
| *** puranamr has quit IRC | 17:47 | |
| TheJulia | jroll: ++ | 17:47 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Split storage components into individual files https://review.openstack.org/227408 | 17:48 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added $$resourceCache service. https://review.openstack.org/227444 | 17:48 |
| NobodyCam | krotscheck: are you back??? | 17:49 |
| krotscheck | NobodyCam: Yes, though my work day will be "8-hours-as-interrupted-by-diapers" for the forseeable future. | 17:50 |
| NobodyCam | lol... +++ welcome back | 17:50 |
| krotscheck | NobodyCam: Thanks! | 17:50 |
| *** Marga_ has quit IRC | 17:50 | |
| *** achanda has joined #openstack-ironic | 17:51 | |
| *** puranamr has joined #openstack-ironic | 17:51 | |
| jroll | rloo: I'd like to add the 'known issues' and 'upgrade notes' sometime between today and liberty-official | 17:53 |
| * krotscheck keeps forgetting that 'delete' is a reserved word in javascript. | 17:53 | |
| rloo | jroll: ok, so you're OK with those release notes now? I see there are two +2s already. | 17:54 |
| rloo | jroll: you just waiting for someone else to approve? | 17:54 |
| jroll | rloo: yep | 17:54 |
| *** puranamr has quit IRC | 17:55 | |
| *** puranamr has joined #openstack-ironic | 17:55 | |
| rloo | jroll: done. | 17:55 |
| jroll | \o/ | 17:56 |
| jroll | I'm going to release once that merges | 17:56 |
| JayF | RELEASE THE HOUN^W IRONIC | 17:56 |
| jroll | s/HOUN/KRAKEN | 17:56 |
| rloo | jroll: should also wait for this to merge: https://review.openstack.org/#/c/227029/. which should merge before the release notes does. | 17:57 |
| jroll | right right | 17:58 |
| rloo | jroll: there is this other doc change that i -1'd, don't know if we want it in. not that important I don't think: https://review.openstack.org/#/c/227370/ | 18:00 |
| jroll | yeah | 18:01 |
| jroll | would be nice though, liliars do you mind updating that? | 18:01 |
| jroll | I will if they don't respond | 18:01 |
| *** puranamr has quit IRC | 18:01 | |
| rloo | jroll: go fer it. | 18:02 |
| solid-sinval | jroll: liliars is afk right now, you can update it | 18:02 |
| rloo | or solid-sinval, you can do it if you want :) | 18:02 |
| jroll | I'll grab it | 18:02 |
| solid-sinval | :) | 18:03 |
| jroll | I hate line wrapping sometimes | 18:03 |
| * sambetts too | 18:03 | |
| *** athomas has quit IRC | 18:04 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Update architecture docs to mention new driver interfaces https://review.openstack.org/227370 | 18:04 |
| jroll | bam, rloo ^ | 18:04 |
| *** Sukhdev has joined #openstack-ironic | 18:04 | |
| jroll | devananda: ^ would also like your eyes on that | 18:05 |
| rloo | jroll: +2'd. | 18:06 |
| jroll | thanks | 18:06 |
| * devananda looks | 18:07 | |
| devananda | I suspect a lot of our docs could use some careful rereading and cleanup soon ... statements like this no longer seem, well, right. | 18:09 |
| devananda | "`Standard` interfaces provide functionality beyond the needs of OpenStack" | 18:09 |
| *** penick has quit IRC | 18:10 | |
| jroll | heh | 18:11 |
| jroll | +1 | 18:11 |
| *** penick has joined #openstack-ironic | 18:11 | |
| rloo | wrt docs, I had this thought to make everyone read a different page, but I decided that was too much torture. | 18:12 |
| *** pelix has quit IRC | 18:14 | |
| rloo | do we think that when all intree drivers have a boot interface in addition to a deploy interface, that boot interface would be a core interface instead of a standard interface? | 18:16 |
| jroll | it's almost like we need someone explicitly working on docs :| | 18:17 |
| jroll | rloo: that's a good question, it would need some sort of 'deprecation' period for out-of-tree drivers | 18:17 |
| *** mgoddard1 has left #openstack-ironic | 18:17 | |
| *** notq has joined #openstack-ironic | 18:18 | |
| rloo | i was wondering whether it was easier to put boot in core now with a hack so it doesn't bomb if it isn't there, vs moving boot from standard to core later. | 18:18 |
| rloo | jroll: then i thought that i shouldn't be thinking about these things now. | 18:18 |
| jroll | rloo: :) | 18:18 |
| devananda | rloo: yes. after deprecation period where we LOG a warning for drivers that don't implement it | 18:19 |
| devananda | rloo: that is, standard now since we just introduced it. leave it there for a cycle. make it core in N and deprecate in O or P | 18:19 |
| devananda | so core interface list will become power, boot, deploy | 18:20 |
| rloo | devananda: yes, that makes sense. | 18:20 |
| Madasi | q/buffer *shell | 18:21 |
| Madasi | sorry, misfire | 18:21 |
| rloo | devananda: i am thinking of adding a comment/TODO? with that plan. Cuz otherwise, I think we may forget (or not be around) | 18:21 |
| devananda | rloo: good idea | 18:22 |
| devananda | I'd add it in drivers/base.py | 18:22 |
| rloo | devananda: yup | 18:22 |
| *** trown has quit IRC | 18:23 | |
| rloo | devananda: sorry, i didn't get it. if we make it core in N, we'd want to deprecate it from standard in N, and delete from standard in O or P? (which means hacking something in N so things don't barf if a driver doesn't have it in core but in standard) | 18:28 |
| jroll | rloo: add a LOG.warning in N, remove in O | 18:29 |
| jroll | make it core in O | 18:29 |
| jroll | sorry | 18:29 |
| *** athomas has joined #openstack-ironic | 18:29 | |
| *** achanda has quit IRC | 18:29 | |
| rloo | jroll: it seems like we ought to have/allow? it in core & standard during the transition. | 18:30 |
| jroll | rloo: I guess I don't actually know what the difference is other than checking if driver.boot exists | 18:31 |
| jroll | I don't think it's a driver change to move it to core | 18:31 |
| *** Marga_ has joined #openstack-ironic | 18:31 | |
| *** achanda has joined #openstack-ironic | 18:31 | |
| rloo | jroll: yeah, core means that the interface has to exist in every driver. | 18:32 |
| jroll | rloo: right, I don't think we can have it in core and standard | 18:32 |
| rloo | jroll: but what does 'deprecation' period mean then? | 18:32 |
| devananda | rloo: the access path would be the same once the driver is loaded, regardless of whether it is loaded through core or standard | 18:32 |
| devananda | I think we'll need to do a little checking on driver loading / initialization merely so we can inform users that they should update the driver | 18:33 |
| devananda | but I don't think the change from standard -> core would actually be incompatible | 18:33 |
| jroll | rloo: all it means is we'll log a warning | 18:33 |
| devananda | rloo: during deprecation period, we'll allow drivers to run that haven't defined a 'boot' interface, and after that period, we'll start enforcing that the driver must have all three interfaces instantiated | 18:34 |
| jroll | actually... do we even validate this? wtf | 18:34 |
| * jroll reading code | 18:34 | |
| jroll | O dpm | 18:35 |
| jroll | I don't think we do | 18:35 |
| jroll | well that answers that question | 18:36 |
| jroll | just move it and log | 18:36 |
| rloo | devananda, jroll: ok, got it. brain fart, i thought all drivers had to do something to explicitly move their boot interface from standard to core, but that's in base. | 18:36 |
| *** trown has joined #openstack-ironic | 18:36 | |
| jroll | rloo: yep :) | 18:37 |
| *** r-daneel has joined #openstack-ironic | 18:37 | |
| *** Sukhdev has quit IRC | 18:37 | |
| *** Sukhdev has joined #openstack-ironic | 18:39 | |
| devananda | jroll: yea, heh, i just read that too. derp. we should validate it :( | 18:39 |
| jroll | heh | 18:40 |
| *** Marga_ has quit IRC | 18:40 | |
| *** Marga_ has joined #openstack-ironic | 18:40 | |
| *** Sukhdev has quit IRC | 18:41 | |
| *** athomas has quit IRC | 18:42 | |
| rloo | jroll, devananda: we don't do much/strict validation. I think what we did is assume that the core interfaces exist so we don't check for existence before accessing. | 18:44 |
| *** Sukhdev has joined #openstack-ironic | 18:45 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Comment with plan to make boot interface mandatory https://review.openstack.org/227468 | 18:48 |
| jroll | openstackgerrit: where are my merges, pls 2 hurry | 18:51 |
| openstackgerrit | Merged openstack/ironic: Update install guide to reflect latest code https://review.openstack.org/227029 | 18:55 |
| jroll | othx | 18:55 |
| jroll | 1/3 | 18:55 |
| rloo | jroll: those don't even run any tests and they're taking so long :-( | 18:56 |
| openstackgerrit | Merged openstack/ironic: Use of 'the Bare Metal service' in guide https://review.openstack.org/227080 | 18:56 |
| openstackgerrit | Merged openstack/ironic: Unit test environment setup clarification https://review.openstack.org/226445 | 18:57 |
| jroll | rloo: :/ | 18:57 |
| jroll | I didn't know about those changes but \o/ | 18:57 |
| rloo | jroll: yeah, documentation changes. not that important but nice to get in if they make it. | 18:57 |
| jroll | yeah | 18:57 |
| jroll | the gate is really interesting | 18:58 |
| jroll | tests already passed on those | 18:58 |
| jroll | but they're waiting in line for neutron etc changes | 18:58 |
| devananda | jroll: wwhy we need to keep our gate run times down -- other projects wait on us, too | 19:01 |
| jroll | yep | 19:01 |
| rloo | jroll: you did see my comment about the upgrade kilo->Liberty for inspection? http://docs.openstack.org/developer/ironic/deploy/upgrade-guide.html | 19:01 |
| *** ijw has quit IRC | 19:01 | |
| jroll | rloo: I did not, good one! | 19:01 |
| rloo | jroll: Oh. I added a comment about that to the release notes patch and figured you saw it. | 19:02 |
| jroll | rloo: nope :( | 19:02 |
| jroll | I think that's the right place for it though | 19:03 |
| rloo | jroll: sorry, should have checked with you about it sooner. you're going to update the release notes anyway right? | 19:03 |
| jroll | rloo: I mean, I tend to think if people are upgrading, they should read the upgrade guide | 19:04 |
| rloo | jroll: I mean, after the candidate branch gets cut. | 19:04 |
| jroll | idk if it needs to be in release notes | 19:04 |
| jroll | but we can add it later yeah | 19:04 |
| rloo | jroll: a link from the release notes. | 19:04 |
| jroll | even if we don't do another release, I'll update it to say this is the final liberty release or whatever | 19:04 |
| jroll | yeah | 19:04 |
| jroll | I'm going to grab some lunch. bbl | 19:05 |
| *** krtaylor has quit IRC | 19:15 | |
| devananda | ditto | 19:23 |
| *** Sukhdev has quit IRC | 19:24 | |
| openstackgerrit | Andreas Jaeger proposed openstack/ironic: Cleanup of Translations https://review.openstack.org/227498 | 19:24 |
| *** krtaylor has joined #openstack-ironic | 19:28 | |
| *** penick has quit IRC | 19:30 | |
| openstackgerrit | Merged openstack/ironic: Add 4.2.0 release notes https://review.openstack.org/227033 | 19:31 |
| openstackgerrit | Merged openstack/ironic: Update architecture docs to mention new driver interfaces https://review.openstack.org/227370 | 19:31 |
| *** penick has joined #openstack-ironic | 19:33 | |
| *** bnemec has quit IRC | 19:34 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Rename 'message' attribute to '_msg_fmt' in IronicException https://review.openstack.org/215362 | 19:49 |
| * jlvillal feels good to get back and fixup some old patches :) | 19:49 | |
| *** ijw has joined #openstack-ironic | 19:56 | |
| *** romcheg has joined #openstack-ironic | 19:56 | |
| *** ijw has quit IRC | 20:01 | |
| * devananda heads out to post office to mail some things | 20:08 | |
| *** absubram has joined #openstack-ironic | 20:08 | |
| NobodyCam | :) | 20:08 |
| NobodyCam | enjoy | 20:08 |
| *** bnemec has joined #openstack-ironic | 20:24 | |
| *** trown is now known as trown|outttypeww | 20:28 | |
| *** baoli has quit IRC | 20:28 | |
| *** achanda has quit IRC | 20:30 | |
| rloo | we might want this change in for the release candidate. NobodyCam, do you have time to take a look? https://review.openstack.org/#/c/227498/ | 20:44 |
| NobodyCam | rloo: one minute | 20:45 |
| NobodyCam | rloo: lgtm, I've only +2'd as i did not test locally | 20:53 |
| rloo | NobodyCam: ? I never test that stuff. | 20:54 |
| NobodyCam | it all looks good.. | 20:54 |
| *** lucasagomes has joined #openstack-ironic | 20:54 | |
| NobodyCam | we can +a it | 20:54 |
| rloo | NobodyCam: I think so. even though it doesn't remove the existing .po files. | 20:55 |
| NobodyCam | ya. by reading the commit message that seemed like it was for broken files, ours may not have been that bad | 20:56 |
| NobodyCam | =a'd | 20:56 |
| NobodyCam | +a'd :-p | 20:56 |
| rloo | NobodyCam: ha ha, we both +a'd | 20:56 |
| NobodyCam | hehehe | 20:57 |
| *** ijw has joined #openstack-ironic | 20:57 | |
| NobodyCam | brb | 20:57 |
| * NobodyCam is back | 21:01 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Refactor list_all_block_devices & add block_type param https://review.openstack.org/218404 | 21:01 |
| *** mtanino has quit IRC | 21:01 | |
| * jlvillal wonders if JoshNang is still around and wants to look at his patch ^^^^ :) | 21:02 | |
| *** ijw has quit IRC | 21:02 | |
| * TheJulia suspects, based on twitter, likely not | 21:03 | |
| jlvillal | TheJulia: So 'JoshNang has left the building. JoshNang has left the building' ;) | 21:04 |
| TheJulia | yeah, post says he is on his way to vegas | 21:05 |
| *** achanda has joined #openstack-ironic | 21:05 | |
| *** hemna has joined #openstack-ironic | 21:05 | |
| jlvillal | Sounds like fun. | 21:05 |
| TheJulia | indeed | 21:05 |
| * jlvillal has never gone to Vegas. Sadly.... | 21:06 | |
| *** ijw has joined #openstack-ironic | 21:06 | |
| TheJulia | i have only been once sadly | 21:06 |
| *** achanda has quit IRC | 21:09 | |
| *** mtanino has joined #openstack-ironic | 21:11 | |
| thiagop | 4.2.0 is already out? | 21:19 |
| rloo | thiagop: no, not yet | 21:20 |
| thiagop | did we have problems with the release? | 21:20 |
| *** derekh has joined #openstack-ironic | 21:21 | |
| *** absubram is now known as absubram_afk | 21:23 | |
| rloo | thiagop: no, no problem. i don't think jroll is back from lunch. he has to do something to make it happen. | 21:24 |
| thiagop | ow | 21:24 |
| thiagop | I'm about to dinner | 21:24 |
| thiagop | :) | 21:24 |
| rloo | thiagop: are you waiting for the release? | 21:25 |
| thiagop | rloo: no, just curiosity | 21:25 |
| rloo | thiagop: :) | 21:25 |
| *** dims__ has quit IRC | 21:29 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Rename 'message' attribute to '_msg_fmt' in IronicException https://review.openstack.org/215362 | 21:31 |
| *** rbudden has quit IRC | 21:32 | |
| *** rbudden has joined #openstack-ironic | 21:33 | |
| mrda | Morning Ironic | 21:34 |
| jroll | rloo: hm, are we waiting for 218404 now? | 21:34 |
| thiagop | morning mrda | 21:35 |
| mrda | jlvillal: I have the compsoite agent_amt driver ready to go, except I keep tripping on the openwsman conductor lock-up problem. That's why I haven't pushed a patch. | 21:36 |
| mrda | thiagop: o/ | 21:36 |
| rloo | jroll: why do we need 218404? that's ipa. | 21:36 |
| rloo | jroll: but maybe https://review.openstack.org/#/c/227498/ | 21:36 |
| jlvillal | mrda: Yeah, I think we need to rip out that openwsman stuff | 21:37 |
| jroll | rloo: I can't read. yes that one. | 21:37 |
| NobodyCam | anyone happen to have a nova configured with the ironic driver handy? | 21:37 |
| jlvillal | mrda: Maybe should assign this to you then? https://bugs.launchpad.net/ironic/+bug/1499489 | 21:37 |
| openstack | Launchpad bug 1499489 in Ironic "AMT is not supported with Ironic-Python-Agent" [Undecided,New] - Assigned to John L. Villalovos (happycamp) | 21:37 |
| mrda | jlvillal: so hos's that going? | 21:37 |
| mrda | /hos/how/ | 21:37 |
| rloo | jroll: i hae no idea how necessary it is. given that we have almost 0 translations it doesn't seem to me to matter much but ?? | 21:38 |
| jlvillal | mrda: I think I could put that at the top of my list. | 21:38 |
| NobodyCam | morning mrda :) | 21:38 |
| jroll | rloo: yeah idk. if it's already +A... might as well | 21:38 |
| mrda | NobodyCam: o/ | 21:38 |
| jlvillal | mrda: And good morning! :) | 21:39 |
| rloo | jroll: we're not in any hurry, right? | 21:39 |
| *** Sukhdev has joined #openstack-ironic | 21:39 | |
| mrda | jlvillal: o/ | 21:39 |
| jroll | rloo: I guess not, no | 21:39 |
| rloo | jroll: although all our fans are out there, waiting in anticipation... | 21:40 |
| mrda | jlvillal: So with sdague's new thing, what needs to be done so it could be a drop in for the amt_driver? | 21:40 |
| jroll | right? | 21:40 |
| jlvillal | mrda: It needs a few more functions added basically. | 21:40 |
| jroll | rloo: there's also the whole "we should go home at some point today" thing | 21:40 |
| *** marzif has quit IRC | 21:40 | |
| rloo | jroll: definitely. can you ask andreas if it is important? | 21:41 |
| *** thrash is now known as thrash|g0ne | 21:41 | |
| mrda | jlvillal: do we know what functions are needed? | 21:41 |
| jlvillal | mrda: I will put that at the top of my list. I should be able to spend most of tomorrow on it. | 21:41 |
| jroll | rloo: sure | 21:41 |
| jlvillal | mrda: Not too hard to figure out. Basically any pywsman calls need to be replaced | 21:41 |
| jlvillal | mrda: Well for AMT that is :) | 21:42 |
| mrda | :) | 21:42 |
| jroll | rloo: I lied, they have disappeared from irc | 21:42 |
| jlvillal | It is used by other drivers too. But I will focus on AMT | 21:42 |
| *** rbudden has quit IRC | 21:42 | |
| rloo | jroll: who has disappeared? | 21:42 |
| jlvillal | mrda: List if fairly small | 21:43 |
| *** derekh has quit IRC | 21:43 | |
| jroll | rloo: AJaeger | 21:43 |
| rloo | jroll: my take on it is that if it was urgent AJaeger would have ping'd us on irc. | 21:43 |
| jroll | sure | 21:43 |
| jroll | rloo: the queue isn't too long. I'll wait a bit | 21:44 |
| jroll | dhellmann: how late are you around today? | 21:44 |
| rloo | jroll: i can basically argue either way. but really, it isn't that important. and if it is, it can always be added to stable branch. | 21:44 |
| jlvillal | mrda: I count about 8 calls to pywsman from the amt driver. | 21:46 |
| mrda | thanks jlvillal - hoping to find time to help out here | 21:48 |
| jroll | so my developer friend just played with a BMC for the first time, it's hilarious watching him find out how cool OOB management is | 21:49 |
| mrda | :) | 21:50 |
| mrda | what it works :P | 21:50 |
| *** jlvillal has quit IRC | 21:51 | |
| *** dims_ has joined #openstack-ironic | 21:52 | |
| NobodyCam | jroll: ++++ pulling another sole in to the dark side :-p | 21:53 |
| jroll | hehehe | 21:54 |
| *** e0ne has joined #openstack-ironic | 21:56 | |
| *** jlvillal has joined #openstack-ironic | 21:57 | |
| jlvillal | mrda: I count about 8 calls to pywsman from the amt driver. | 22:01 |
| jlvillal | mrda: I'll go dig in some more | 22:01 |
| * jlvillal grumbles that his ZNC went down for five minutes... | 22:01 | |
| NobodyCam | jlvillal: any discounts on the correct nuc model for ironic devs? | 22:03 |
| NobodyCam | :-p | 22:03 |
| * jroll points NobodyCam towards BadCub's p card | 22:03 | |
| NobodyCam | hahahhahahaha | 22:04 |
| NobodyCam | :-p | 22:04 |
| * BadCub frowns and hisses | 22:04 | |
| NobodyCam | lol | 22:04 |
| thiagop | lol | 22:05 |
| *** achanda has joined #openstack-ironic | 22:06 | |
| jlvillal | NobodyCam: I wish! | 22:06 |
| NobodyCam | :) gota try | 22:06 |
| jlvillal | NobodyCam: Our employee purchase program doesn't even carry the AMT model :( | 22:06 |
| BadCub | ouch | 22:06 |
| NobodyCam | :( | 22:06 |
| jlvillal | NobodyCam: I think sometimes we buy them on Amazon.com for our developers. Crazy. | 22:07 |
| NobodyCam | hahhaha | 22:07 |
| jlvillal | Big companies.... | 22:07 |
| NobodyCam | yep | 22:07 |
| mrda | Don't make me tell you the price I had to pay to get mine - NUCs aren't common in AU and you pay a premium for them. | 22:10 |
| *** hemna has quit IRC | 22:10 | |
| NobodyCam | mrda: uggh | 22:10 |
| NobodyCam | :( | 22:10 |
| * mrda wishes he could've bought on Amazon | 22:10 | |
| BadCub | heh.. try ordering HP equipment for internal use! Madness!!!!! | 22:11 |
| mrda | BadCub: Try ordering HP equipment as a customer! Madness!!!!! | 22:13 |
| mrda | :-) | 22:13 |
| NobodyCam | doh | 22:13 |
| mrda | lol | 22:13 |
| jroll | lol | 22:14 |
| BadCub | mrda: LOL. I have ordered for personal use.... I remember the pain.... | 22:14 |
| mrda | it's popular stuff - that popularity is reflected in lead times... | 22:14 |
| BadCub | I ordered new Dev machines for my team 4-months-ago.... And they just arrived today..... | 22:14 |
| mrda | \o/ | 22:14 |
| jlvillal | NobodyCam: That mini server from HP is pretty nice. For a NAS type system. | 22:14 |
| NobodyCam | jlvillal: ya it actually is | 22:15 |
| BadCub | for personal machine I have the Envy Phoenix. It's a pretty nasty gaming tower | 22:15 |
| *** garthb has quit IRC | 22:15 | |
| *** e0ne has quit IRC | 22:17 | |
| jlvillal | NobodyCam: Okay 'microserver' ... | 22:17 |
| * NobodyCam is waiting for the ^^^^^ dev systems... something like 48 cores with 128 gb of ram | 22:18 | |
| BadCub | Yes, I want to game on said dev machine lol | 22:19 |
| jroll | o.o | 22:19 |
| jlvillal | mrda: Not sure if you know about: pywsman.set_debug(4) | 22:21 |
| jlvillal | mrda: Makes pywsman/openwsman spew a lot of debug messages. If you need it. | 22:21 |
| mrda | jlvillal: cool! | 22:24 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Refactor list_all_block_devices & add block_type param https://review.openstack.org/218404 | 22:31 |
| *** absubram_afk has quit IRC | 22:32 | |
| * devananda returns from shopping and shipping | 22:35 | |
| NobodyCam | wb devananda | 22:35 |
| NobodyCam | hope everything is in ship shop shape | 22:36 |
| NobodyCam | lol | 22:36 |
| jroll | hehehe | 22:39 |
| devananda | jroll: rloo: anything I can do to help the release along right now? | 22:39 |
| jroll | devananda: kick the gate? | 22:39 |
| devananda | heh | 22:39 |
| jroll | waiting on one more patch | 22:39 |
| devananda | I do not have those powers | 22:39 |
| jroll | it's #2 in the queue | 22:40 |
| *** penick has quit IRC | 22:40 | |
| BadCub | I am calling it quits for the day. Have a good night folks!!! | 22:41 |
| jroll | \o later | 22:43 |
| devananda | BadCub: g'night! | 22:45 |
| BadCub | gn devananda :) | 22:45 |
| jlvillal | BadCub: ciao! | 22:50 |
| jlvillal | mrda: I have got something very ugly working that no longer uses pywsman for AMT. | 22:51 |
| mrda | great job jlvillal! | 22:52 |
| jlvillal | mrda: I said it was very ugly ;) But system did install and boot. So I think that is a good sing. | 22:52 |
| jlvillal | s/sing/sign/ | 22:52 |
| mrda | jlvillal: if you push it to github I'll have a play today | 22:54 |
| jroll | neutron plsssss | 22:54 |
| jroll | push it to gerrit! | 22:54 |
| jlvillal | mrda: Let me see | 22:55 |
| mrda | jroll: \o/ | 22:55 |
| mrda | that's what the WIP function is for, right? | 22:56 |
| openstackgerrit | Merged openstack/ironic: Cleanup of Translations https://review.openstack.org/227498 | 22:58 |
| openstackgerrit | Merged openstack/ironic: Cleanup of Translations https://review.openstack.org/227498 | 22:58 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Work in progress, do not merge https://review.openstack.org/227581 | 23:03 |
| jlvillal | mrda: ^^^ Ugly | 23:04 |
| mrda | thanks jlvillal | 23:05 |
| *** dims_ has quit IRC | 23:05 | |
| jroll | devananda: boom https://review.openstack.org/#/c/227582/ | 23:06 |
| NobodyCam | nice | 23:07 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Work in progress, do not merge https://review.openstack.org/227581 | 23:07 |
| devananda | jroll: nice | 23:07 |
| jlvillal | mrda: Updated again. Removed all the 'pywsman' references. | 23:08 |
| jlvillal | jroll: Woo hoo. New release :) | 23:09 |
| jroll | \o/ | 23:09 |
| *** Sukhdev has quit IRC | 23:09 | |
| thiagop | release woohoo | 23:19 |
| thiagop | ! | 23:19 |
| jroll | devananda: I guess we can open M now that we've identified that SHA, yeah? | 23:20 |
| jroll | devananda: still need to move specs around too :) | 23:20 |
| devananda | jroll: yah | 23:21 |
| devananda | oh, spec patch, here's a link | 23:21 |
| devananda | https://review.openstack.org/#/c/226606/ | 23:22 |
| * devananda fixes and unwips | 23:22 | |
| jroll | devananda: on a lighter note you may enjoy https://www.youtube.com/watch?v=oAiVsbXVP6k | 23:22 |
| *** [1]cdearborn has quit IRC | 23:22 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: Update spec file locations for 4.2 / Liberty https://review.openstack.org/226606 | 23:25 |
| jroll | devananda: do we want placeholders from approved/ ? | 23:26 |
| devananda | jroll: uh, I don't think so? | 23:26 |
| devananda | jroll: approved == work in progress | 23:26 |
| jroll | there are links out there to the approved spec | 23:27 |
| devananda | we did placeholders for the old /<name>/ directories becayuse of hard links elsewhere | 23:27 |
| devananda | gah | 23:27 |
| *** romcheg has quit IRC | 23:27 | |
| NobodyCam | jroll: omg roflmao | 23:27 |
| jroll | :D | 23:27 |
| jroll | devananda: also https://blueprints.launchpad.net/ironic/+spec/remove-swift-dependency-for-ilo-drivers | 23:27 |
| jlvillal | mrda: Do you think we should create a blueprint related to removing pywsman from Ironic? I know we want it out of the amt driver. And the drac driver is doing the same thing. | 23:29 |
| devananda | jroll: LOL nice! | 23:29 |
| *** lucasagomes has quit IRC | 23:38 | |
| *** shoutm has joined #openstack-ironic | 23:45 | |
| *** davideagnello has quit IRC | 23:46 | |
| *** rbudden has joined #openstack-ironic | 23:47 | |
| devananda | jroll: oh - right - we're calling that one done, even w/o docs | 23:48 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: Update spec file locations for 4.2 / Liberty https://review.openstack.org/226606 | 23:52 |
| *** rbudden has quit IRC | 23:52 | |
| devananda | jlvillal: or a bug, perhaps | 23:52 |
| devananda | jlvillal: what is it being replaced with? | 23:52 |
| devananda | also I think I filed a bug about it months ago -- maybe just update that? | 23:52 |
| devananda | brb, rebooting | 23:53 |
| jlvillal | devananda: The goal will be to use sdague's amt library for the AMT driver. But ifarkas is working on a wsman.py library | 23:53 |
| *** vishwana_ has joined #openstack-ironic | 23:54 | |
| *** jamielennox|away is now known as jamielennox | 23:56 | |
| *** vishwanathj has quit IRC | 23:56 | |
| *** harlowja has quit IRC | 23:56 | |
| *** BadCub has quit IRC | 23:56 | |
| *** r-daneel has quit IRC | 23:57 | |
| *** davidlenwell has quit IRC | 23:57 | |
| *** harlowja has joined #openstack-ironic | 23:57 | |
| *** davidlenwell has joined #openstack-ironic | 23:58 | |
| *** naohirot has joined #openstack-ironic | 23:58 | |
| *** r-daneel has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!