| jogo | devananda: https://review.openstack.org/#/c/43913/ | 00:00 |
|---|---|---|
| jogo | is what it should do | 00:00 |
| devananda | jogo: https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/driver.py#L215 | 00:02 |
| devananda | looks like we already do that | 00:02 |
| jogo | yup awesome | 00:02 |
| jogo | I figured you did | 00:02 |
| devananda | cool | 00:02 |
| jogo | but just wanted to double check | 00:02 |
| devananda | jogo: much appreciated | 00:02 |
| jogo | devananda: can you give that patch a quick +1 | 00:02 |
| devananda | added my +1 | 00:02 |
| jogo | sweet thanks | 00:03 |
| devananda | fwiw, we'll need the same patch on https://github.com/openstack/ironic/blob/master/ironic/nova/scheduler/ironic_host_manager.py now | 00:04 |
| * devananda creates a new wiki page | 00:10 | |
| devananda | https://wiki.openstack.org/wiki/Ironic/Drivers | 00:10 |
| *** matsuhashi has quit IRC | 00:13 | |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic: Log exceptions from deploy and tear_down https://review.openstack.org/97318 | 00:21 |
| lifeless | jogo: is host capabilities what lets vm and bm hypervisors co-exist in one scheduler domain ? | 00:57 |
| *** ellenh has quit IRC | 01:00 | |
| *** ryanpetrello has joined #openstack-ironic | 01:05 | |
| ryanpetrello | hey Ironic folks, anybody know what this keystoneclient failure’s about? | 01:06 |
| ryanpetrello | http://logs.openstack.org/44/94244/17/check/gate-pecan-tox-ironic-stable/8afac0e/console.html | 01:06 |
| ryanpetrello | looks like it was resolved in https://bugs.launchpad.net/ironic/+bug/1324655 | 01:07 |
| ryanpetrello | maybe it’s a candidate for backporting, as it’s breaking the pecan tests that are gating on ironic stable/icehouse | 01:08 |
| devananda | ryanpetrello: yep! that looks like it | 01:09 |
| *** nosnos has joined #openstack-ironic | 01:09 | |
| ryanpetrello | I’ll submit a backport | 01:09 |
| devananda | ryanpetrello: tyvm | 01:09 |
| ryanpetrello | hrm | 01:15 |
| ryanpetrello | in https://review.openstack.org/#/c/96576/1 | 01:15 |
| ryanpetrello | are those config changes applicable to stable/icehouse? | 01:15 |
| ryanpetrello | the original commit says, “Sample conf needed regenerated, also, due to changes." | 01:16 |
| ryanpetrello | but I’m unsure what that has to do w/ the test change | 01:16 |
| devananda | ryanpetrello: the keystone change caused the options to change | 01:25 |
| devananda | ryanpetrello: so our unit tests started failing at the same time because of the sample config file check | 01:26 |
| devananda | ryanpetrello: you can regenerate it with tools/conf/generate_sample.sh | 01:26 |
| devananda | ryanpetrello: another solution -- even better actually | 01:26 |
| devananda | ryanpetrello: pin the version of keystoneclient on stable-icehouse | 01:26 |
| devananda | ryanpetrello: the problem is pulling a keystoneclient that's newer than the code in stable-icehouse | 01:27 |
| devananda | ok - heading out for dinner.... ttytm! | 01:27 |
| ryanpetrello | good point | 01:27 |
| jogo | lifeless: not really, host capabilities was an old way of getting info to the sheduler | 01:50 |
| lifeless | jogo: so whats its for now then, I mean? | 01:58 |
| *** Poornima has joined #openstack-ironic | 02:09 | |
| *** rloo has quit IRC | 02:11 | |
| *** shakamunyi has quit IRC | 02:31 | |
| ryanpetrello | devananda: you sure the appropriate response here is to revert to a previous keystoneclient? | 02:35 |
| ryanpetrello | it seems like the arguments in question have been that way for some time: https://github.com/openstack/python-keystoneclient/commit/3374e3c2fa30239a131c3815fea8c2694caca226 | 02:36 |
| openstackgerrit | A change was merged to stackforge/pyghmi: Fix IPv4 operations where v6only behavior is default https://review.openstack.org/97333 | 02:44 |
| *** coolsvap|afk is now known as coolsvap | 03:13 | |
| jogo | lifeless: it doesn't do anything, dead code | 03:18 |
| *** nosnos has quit IRC | 03:20 | |
| *** aswadrangnekar has joined #openstack-ironic | 03:27 | |
| *** eghobo has quit IRC | 03:47 | |
| lifeless | jogo: so why the patch at all? Colour me confused. | 03:58 |
| *** nosnos has joined #openstack-ironic | 04:00 | |
| *** k4n0 has joined #openstack-ironic | 04:11 | |
| jogo | lifeless: remove dead code so we don't confuse folks inthe the future | 04:23 |
| lifeless | ack | 04:24 |
| *** sabah has joined #openstack-ironic | 04:29 | |
| *** eghobo has joined #openstack-ironic | 04:42 | |
| *** rameshg87 has joined #openstack-ironic | 04:47 | |
| *** harlowja is now known as harlowja_away | 04:48 | |
| *** JoshNang has quit IRC | 05:19 | |
| *** rakesh_hs has joined #openstack-ironic | 05:25 | |
| *** jcoufal has joined #openstack-ironic | 05:41 | |
| *** lazy_prince has joined #openstack-ironic | 05:55 | |
| *** coolsvap is now known as coolsvap|afk | 05:56 | |
| *** coolsvap|afk is now known as coolsvap | 06:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/96063 | 06:03 |
| *** sysexit has joined #openstack-ironic | 06:05 | |
| *** ccit has joined #openstack-ironic | 06:11 | |
| *** kevinbenton has quit IRC | 06:19 | |
| GheRivero | morning all | 06:21 |
| openstackgerrit | Christian Berendt proposed a change to openstack/ironic: Test for membership should be 'not in' https://review.openstack.org/97412 | 06:23 |
| *** radsy has quit IRC | 06:39 | |
| *** max_lobur has joined #openstack-ironic | 06:41 | |
| *** pradipta_away is now known as pradipta | 06:44 | |
| *** coolsvap is now known as coolsvap|afk | 06:57 | |
| *** coolsvap|afk is now known as coolsvap | 07:02 | |
| lifeless | NobodyCam: still around ? | 07:05 |
| *** eghobo has quit IRC | 07:18 | |
| *** romcheg has joined #openstack-ironic | 07:20 | |
| *** yuriyz has joined #openstack-ironic | 07:24 | |
| *** JoshNang has joined #openstack-ironic | 07:26 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 07:36 | |
| Mikhail_D_ltp | Good morning all! :) | 07:39 |
| romcheg | Mornind Mikhail_D_ltp and everyone else! | 07:40 |
| *** mrda is now known as mrda-away | 07:40 | |
| *** martyntaylor has joined #openstack-ironic | 07:41 | |
| *** jistr has joined #openstack-ironic | 08:03 | |
| *** ndipanov has joined #openstack-ironic | 08:07 | |
| *** sysexit has quit IRC | 08:08 | |
| dtantsur|afk | morning Ironic, morning Mikhail_D_ltp and romcheg! | 08:12 |
| Mikhail_D_ltp | dtantsur|afk: morning :) | 08:13 |
| *** dtantsur|afk is now known as dtantsur | 08:13 | |
| *** derekh_ has joined #openstack-ironic | 08:20 | |
| *** coolsvap is now known as coolsvap|afk | 08:24 | |
| dtantsur | romcheg, around? | 08:25 |
| dtantsur | romcheg, Mikhail_D_ltp, I would like to try land https://review.openstack.org/#/c/86536/ this morning | 08:25 |
| *** lucasagomes has joined #openstack-ironic | 08:26 | |
| dtantsur | oh, lucasagomes, hi! Maybe we'll use you, if romcheg is not around :) | 08:27 |
| dtantsur | I would like to try and land https://review.openstack.org/#/c/86536/ this morning | 08:27 |
| lucasagomes | dtantsur, :) sure | 08:28 |
| dtantsur | let us see, what we have there, while we're waiting for Jenkins | 08:28 |
| lucasagomes | will review it | 08:28 |
| dtantsur | lucasagomes, my suggestion is to -1 only on something serious, while keeping nits for anything later | 08:30 |
| dtantsur | as this series is around for too long already :-/ | 08:30 |
| lucasagomes | dtantsur, right, well we def need to change the commit message then | 08:31 |
| lucasagomes | part 2 is not merged | 08:31 |
| dtantsur | Mikhail_D_ltp, ^^^ | 08:31 |
| lucasagomes | merging part 1 and part 3 but no part 2 is very awkward | 08:31 |
| openstackgerrit | Sandhya Balakrishnan proposed a change to openstack/ironic: Updates Ironic Guide with deployment information https://review.openstack.org/94604 | 08:32 |
| lucasagomes | dtantsur, what happened with part 2? | 08:33 |
| dtantsur | lucasagomes, it is around, but has lower priority | 08:33 |
| lucasagomes | we can change the commit message for Mikhail_D_ltp if needed, I'm just trying to understand why the dependency was removed from p2 | 08:33 |
| dtantsur | lucasagomes, because this one touches nova driver | 08:33 |
| lucasagomes | ah | 08:33 |
| lucasagomes | right makes sense | 08:33 |
| *** max_lobur has quit IRC | 08:34 | |
| Mikhail_D_ltp | lucasagomes: Ok, I'll change name for this patch :) | 08:34 |
| lucasagomes | Mikhail_D_ltp, I know it's a kind of nit, but that would look very odd in the log | 08:35 |
| dtantsur | Mikhail_D_ltp, test__stop_firewall is not converted. Is it possible to quickly convert as well? | 08:35 |
| dtantsur | Mikhail_D_ltp, also don't remember, was it possible to convert test_list_instance_uuids? | 08:36 |
| dtantsur | lucasagomes, should we insist on these ^^^ ? I'm ok with fixing the remaining in one more patch | 08:37 |
| lucasagomes | dtantsur, yeah it's fine for me to have a following patch as well if needed | 08:37 |
| lucasagomes | if it's quick and u want to fix it in this patch I would be glad, but if not I'm ok with it as well | 08:38 |
| dtantsur | Mikhail_D_ltp ^^^ | 08:38 |
| dtantsur | btw tests passed for me locally. hipe Jenkins will be ok as well | 08:38 |
| dtantsur | * hope even | 08:38 |
| Mikhail_D_ltp | dtantsur: yeah, I'm rebasing now. | 08:39 |
| dtantsur | Mikhail_D_ltp, patch looks good to me. Let me know, when you fix what Lucas found, I'll +2 it. | 08:40 |
| Mikhail_D_ltp | Ok :) | 08:40 |
| *** sabah has quit IRC | 08:42 | |
| *** sysexit has joined #openstack-ironic | 08:44 | |
| *** coolsvap|afk is now known as coolsvap | 08:46 | |
| *** romcheg has quit IRC | 08:49 | |
| dtantsur | random Jenkins failures become more and more annoying >_< | 08:55 |
| *** Mikhail_D_ltp has quit IRC | 08:58 | |
| lucasagomes | dtantsur, +1!!!!!!! | 08:58 |
| dtantsur | we can't land 0-line patch for 2 days already... | 08:59 |
| dtantsur | lucasagomes, to top it all, we fixed only one kind of NodeLocked problems, we still have more possibility | 09:00 |
| dtantsur | Mikhail_D_wk, how are things with the patch? As you see, we have troubles landing things due to random test failures... | 09:01 |
| dtantsur | lucasagomes, in the meanwhile, may I ask you for update on https://bugs.launchpad.net/ironic/+bug/1301279 ? | 09:03 |
| dtantsur | lucasagomes, ... and https://bugs.launchpad.net/ironic/+bug/1308680 | 09:04 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 09:04 | |
| lucasagomes | dtantsur, yeah I saw ur comment I'm going to reply it | 09:05 |
| *** yfujioka has joined #openstack-ironic | 09:05 | |
| dtantsur | thanks a lot :) | 09:05 |
| lucasagomes | yesterday it was holiday here so I'm trying to catch up with everything yet | 09:05 |
| * dtantsur is going to annoy people a lot this week ^_^ | 09:05 | |
| lucasagomes | heh no worries | 09:05 |
| Mikhail_D_wk | dtantsur: I'm a bit stuck with test_list_instance_uuids | 09:07 |
| dtantsur | Mikhail_D_ltp, than skip it please. We can do it later | 09:08 |
| Mikhail_D_wk | ок | 09:08 |
| *** igordcard has joined #openstack-ironic | 09:09 | |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Cleanup mock patch without `with` part 2 https://review.openstack.org/86536 | 09:11 |
| dtantsur | lucasagomes, ^^^ | 09:13 |
| lucasagomes | Mikhail_D_ltp, dtantsur cool will take a look in a min | 09:13 |
| *** pelix has joined #openstack-ironic | 09:14 | |
| *** pradipta is now known as pradipta_away | 09:15 | |
| dtantsur | lucasagomes, I'm ok, just approve, when you're ok with it as well | 09:15 |
| lucasagomes | dtantsur, Mikhail_D_ltp done, thanks for convertng the firewall test as well | 09:20 |
| dtantsur | thanks guys \o/ | 09:20 |
| dtantsur | hope it gets merged eventually :D | 09:20 |
| Mikhail_D_ltp | Thank you guys for review. | 09:22 |
| *** shausy has joined #openstack-ironic | 09:38 | |
| *** ifarkas has joined #openstack-ironic | 09:46 | |
| openstackgerrit | Hans Lindgren proposed a change to openstack/ironic: Fix host manager node detection logic https://review.openstack.org/97447 | 10:10 |
| *** max_lobur has joined #openstack-ironic | 10:15 | |
| dtantsur | Mikhail_D_ltp, if you want to have some more tests refactoring, welcome here: https://bugs.launchpad.net/ironic/+bug/1325906 :) | 10:16 |
| openstackgerrit | Anusha Ramineni proposed a change to openstack/ironic-specs: Add design spec for iLO Power Driver https://review.openstack.org/97455 | 10:17 |
| *** romcheg has joined #openstack-ironic | 10:18 | |
| romcheg | dtantsur: Morning, sorry, meeting at the bank took longer than I expected | 10:22 |
| dtantsur | :) | 10:22 |
| *** coolsvap is now known as coolsvap|afk | 10:24 | |
| dtantsur | folks, there was a discussion about paramiko and evenlet some time ago: https://bugs.launchpad.net/ironic/+bug/1321787 | 10:28 |
| dtantsur | anyone remembers the outcome? | 10:28 |
| lucasagomes | dtantsur, hmm I don't remember any real outcome, I remmeber the sahara project had the same problem | 10:32 |
| lucasagomes | and the way they solved was having the ssh commands in a subprocess or something like that | 10:32 |
| lucasagomes | dtantsur, https://review.openstack.org/45716 | 10:32 |
| dtantsur | lucasagomes, thanks! | 10:33 |
| dtantsur | JoshNang, hi, could you fix status on https://bugs.launchpad.net/ironic/+bug/1308199 https://bugs.launchpad.net/ironic/+bug/1323781 and https://bugs.launchpad.net/ironic/+bug/1323783 ? | 10:39 |
| dtantsur | JoshNang, on the last meeting we decided not to keep bugs assigned w/o patch for more than 7 days. Also, assigned bugs should likely be "In Progress", not "New" | 10:40 |
| *** takadayuiko has quit IRC | 10:43 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Implement security groups and firewall filtering methods https://review.openstack.org/96466 | 10:43 |
| *** takadayuiko has joined #openstack-ironic | 10:51 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Return the HTTP Location for accepted requestes https://review.openstack.org/92616 | 10:53 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Return the HTTP Location for newly created resources https://review.openstack.org/92615 | 10:53 |
| *** k4n0 has quit IRC | 11:02 | |
| NobodyCam | Good morning Ironic | 11:05 |
| lucasagomes | morning NobodyCam | 11:07 |
| NobodyCam | we're in North platte Ne | 11:08 |
| lucasagomes | NobodyCam, :) nice! right in the middle of the coutry | 11:09 |
| lucasagomes | NobodyCam, where are you guys heading now? | 11:10 |
| NobodyCam | on cheyenne wyoming | 11:12 |
| NobodyCam | on to .. | 11:12 |
| lucasagomes | :) | 11:14 |
| *** praefect has joined #openstack-ironic | 11:17 | |
| NobodyCam | lol trying to get ahead of the really bad weather | 11:17 |
| *** lucasagomes is now known as lucas-hungry | 11:22 | |
| *** rameshg87 has left #openstack-ironic | 11:23 | |
| NobodyCam | open quiestion.. do we support ipv6? | 11:25 |
| dtantsur | morning NobodyCam | 11:25 |
| NobodyCam | good morning dtantsur | 11:25 |
| dtantsur | NobodyCam, not yet, I guess | 11:25 |
| NobodyCam | :/ | 11:26 |
| dtantsur | NobodyCam, there were some work on DevStack, IIRC | 11:26 |
| dtantsur | you need it? | 11:26 |
| NobodyCam | we will | 11:27 |
| dtantsur | oh | 11:27 |
| *** dtantsur is now known as dtantsur|lunch | 11:28 | |
| NobodyCam | http://www.weather.com/news/tornado-central/severe-weather-tracker-page | 11:30 |
| NobodyCam | woo hoo, we in North Platte trying to make it to chenneye wy today | 11:30 |
| NobodyCam | and https://bugs.launchpad.net/ironic/+bug/1325942 | 11:33 |
| *** matsuhashi has joined #openstack-ironic | 11:34 | |
| *** Poornima has quit IRC | 11:37 | |
| agordeev | good morning everyone! | 11:39 |
| agordeev | NobodyCam: dtantsur|lunch lucas-hungry :) | 11:39 |
| NobodyCam | morning agordeev | 11:40 |
| *** lazy_prince has quit IRC | 11:47 | |
| *** Jatin360 has joined #openstack-ironic | 11:49 | |
| *** dtantsur|lunch is now known as dtantsur | 11:53 | |
| dtantsur | agordeev, morning | 11:53 |
| NobodyCam | ok going to start packing up. will be back once we land in Wy | 11:54 |
| *** Jatin360 has quit IRC | 11:56 | |
| dtantsur | NobodyCam, good luck! When you're back, could you please give status update on https://bugs.launchpad.net/ironic/+bug/1300589 ? | 12:00 |
| *** romcheg1 has joined #openstack-ironic | 12:01 | |
| *** romcheg has quit IRC | 12:03 | |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Cleanup mock patch without `with` part 3 https://review.openstack.org/73256 | 12:03 |
| *** romcheg has joined #openstack-ironic | 12:04 | |
| *** romcheg1 has quit IRC | 12:05 | |
| *** jdob has joined #openstack-ironic | 12:20 | |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 1 https://review.openstack.org/60025 | 12:29 |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Old value 'updated_at' field returned after update https://review.openstack.org/75430 | 12:40 |
| *** krtaylor has quit IRC | 12:45 | |
| *** radsy has joined #openstack-ironic | 12:46 | |
| *** radsy has joined #openstack-ironic | 12:46 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: PXE driver to validate the requested image in Glance https://review.openstack.org/96710 | 12:52 |
| *** jcoufal has quit IRC | 12:52 | |
| *** jcoufal has joined #openstack-ironic | 12:53 | |
| *** lucas-hungry is now known as lucasagomes | 12:53 | |
| *** takadayuiko has quit IRC | 12:55 | |
| *** jistr has quit IRC | 12:57 | |
| *** praefect has quit IRC | 12:59 | |
| *** Alexei_987 has joined #openstack-ironic | 12:59 | |
| *** praefect has joined #openstack-ironic | 13:00 | |
| *** jistr has joined #openstack-ironic | 13:01 | |
| *** openstackgerrit has quit IRC | 13:06 | |
| *** openstackgerrit has joined #openstack-ironic | 13:07 | |
| *** rloo has joined #openstack-ironic | 13:09 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic-specs: Add instance_info field to Node model https://review.openstack.org/95701 | 13:10 |
| *** matsuhashi has quit IRC | 13:12 | |
| *** nosnos has quit IRC | 13:12 | |
| *** rloo has quit IRC | 13:14 | |
| *** rloo has joined #openstack-ironic | 13:15 | |
| *** linggao has joined #openstack-ironic | 13:16 | |
| *** matty_dubs|gone is now known as matty_dubs | 13:23 | |
| *** rloo has quit IRC | 13:23 | |
| *** rloo has joined #openstack-ironic | 13:23 | |
| jroll | morning y'all | 13:24 |
| jroll | lucasagomes: around? | 13:24 |
| romcheg | Morning jroll, lucasagomes! | 13:24 |
| jroll | heya romcheg | 13:25 |
| *** rloo has quit IRC | 13:25 | |
| dtantsur | morning, jroll! | 13:25 |
| jroll | \o dtantsur | 13:25 |
| sysexit | morning Ironic | 13:26 |
| dtantsur | sysexit, morning | 13:26 |
| *** rloo has joined #openstack-ironic | 13:26 | |
| lucasagomes | morning all | 13:26 |
| lucasagomes | jroll, hey yes | 13:26 |
| jroll | hey! | 13:26 |
| lucasagomes | I will brb for 2 min to get a cup of coffee | 13:26 |
| jroll | no worries | 13:26 |
| jroll | about https://review.openstack.org/#/c/97361/ | 13:26 |
| sysexit | So I have a "pxe_ravello" driver that supports provisioning via PXE and power mgmt/console. It can be found here: https://github.com/ravello/ironic/commit/dff1e3a220da13800c126db2f01bdeeaa0eda672 | 13:27 |
| jroll | I don't think the option itself is confusing to users, but I think the behavior is | 13:27 |
| sysexit | Question is now, what to do with it? :) | 13:27 |
| sysexit | Last time there were some concerns adding drivers for non-physical hardware. | 13:28 |
| jroll | lucasagomes: imagine you don't know about the option, you register your nodes in ironic, and suddenly they power off | 13:28 |
| jroll | lucasagomes: especially in the context of upgrading from nova-bm | 13:28 |
| jroll | sysexit: nice! do you know about openstack's review process? | 13:29 |
| linggao | morning Ironic. | 13:30 |
| jroll | hiya linggao | 13:30 |
| dtantsur | sysexit, I guess you start with writing a spec and also determining, if it's possible to create CI for this method | 13:30 |
| romcheg | Morning linggao! | 13:30 |
| dtantsur | morning, linggao | 13:30 |
| linggao | Hi romcheg jroll dtantsur | 13:30 |
| linggao | I got the following error from Jekins for my patch "Requirement pyghmi>=0.6.11 does not match openstack/requirements value pyghmi>=0.5.8" | 13:31 |
| *** shausy has quit IRC | 13:31 | |
| linggao | does anyone know which project does openstack/requirements come from. | 13:32 |
| sysexit | jroll, dtantsur, I think I understand the review process. I've never submitted anything but I read the docs. | 13:32 |
| linggao | devananda added it for me last year. I need to update it. | 13:32 |
| romcheg | linggao: https://github.com/openstack/requirements | 13:32 |
| sysexit | dtantsur, where can I find more info about the spec process? | 13:33 |
| jroll | linggao: what patch? (also, don't get me started on the global requirements thing) | 13:33 |
| romcheg | linggao: you can find global-requirements.txt there | 13:33 |
| *** jbjohnso has joined #openstack-ironic | 13:33 | |
| sysexit | spec process seems to be in flux | 13:33 |
| jroll | sysexit: https://github.com/openstack/ironic-specs | 13:33 |
| *** jgrimm has joined #openstack-ironic | 13:33 | |
| jroll | sysexit: it's more solid at this point | 13:33 |
| lucasagomes | jroll, yeah I can see ur point, I will try to find the reasoning around that options I'm sure there were a bunch of chats about whether it should be the default behavior or not | 13:34 |
| dtantsur | sysexit, this may help https://github.com/openstack/ironic-specs/blob/master/specs/template.rst | 13:34 |
| linggao | romcheg, jroll. Yes, it is the global requirements. It took long time to get through | 13:34 |
| jroll | sysexit: as far as the actual review process etc, https://wiki.openstack.org/wiki/How_To_Contribute and https://wiki.openstack.org/wiki/Gerrit_Workflow | 13:34 |
| jroll | lucasagomes: cool. I'm fine with it staying as is, but it's something that surprised the heck out of me recently (and I even knew the option existed) | 13:35 |
| sysexit | jroll: thx - IIUC, the first thing is to write a spec, and then submit a Gerrit review request? | 13:35 |
| sysexit | The spec, do I simply submit a PR against ironic-specs? | 13:36 |
| sysexit | Or that goes via Gerrit as well? | 13:36 |
| dtantsur | sysexit, yes, via gerrit | 13:36 |
| jroll | lucasagomes: especially when initially deploying, and poking things manually to make sure everything works correctly, that default is painful | 13:36 |
| sysexit | OK, so two Gerrits :) | 13:36 |
| sysexit | For the spec and for the impl. | 13:36 |
| lucasagomes | jroll, http://paste.openstack.org/show/82594/ | 13:37 |
| jroll | sysexit: yeah, generally - submit a spec and a placeholder blueprint (https://blueprints.launchpad.net/ironic/) at the same time. once the spec is merged in gerrit, update the blueprint with a link to it. then put up the code patch, the spec will be approved and targeted to a release. then the code is reviewed. | 13:37 |
| sysexit | jroll, thx! ill do that | 13:38 |
| jroll | lucasagomes: ah yeah. I remember that. | 13:39 |
| jroll | <lifeless> devananda: and the admin goes 'fuck yeah, automation' | 13:39 |
| jroll | I disagree | 13:39 |
| lucasagomes | right, as I pointed on that patch I'm not the guy with most hands on experience in the datacenter, but as we had a blueprint and discussions about it | 13:39 |
| lucasagomes | I thought that the current default was already sorted out | 13:40 |
| jroll | yeah, you make a fair point | 13:40 |
| lucasagomes | jroll, do you think that having a ML discussion about it would be better? | 13:40 |
| jroll | maybe | 13:40 |
| jroll | I mean, I really don't care at this point | 13:40 |
| lucasagomes | cause although we put that as default True, we don't have it documented anywhere apart from random IRC logs | 13:40 |
| jroll | because we now have that set to False in our automation :) | 13:41 |
| jroll | so it's settled for me | 13:41 |
| lucasagomes | right, I will just add lifeless and devananda to ur patch then and have them to take a look | 13:41 |
| jroll | just - "think of the users!" is my mindset | 13:41 |
| *** jistr has quit IRC | 13:41 | |
| lucasagomes | gotcha | 13:41 |
| jroll | :P | 13:41 |
| *** jistr has joined #openstack-ironic | 13:42 | |
| lucasagomes | heh these conversations are kinda complicated heh there's no wrong or right I suppose.... some users will say "fuck yeah automation" others will be like "wtf" | 13:42 |
| lucasagomes | I guess | 13:43 |
| lucasagomes | heh | 13:43 |
| jroll | right | 13:43 |
| jroll | I already have scripts to power on/off all nodes | 13:43 |
| jroll | because I need them regardless | 13:43 |
| jroll | but that may be unique to the 'long-running agent' model | 13:44 |
| *** radsy has quit IRC | 13:49 | |
| *** shakamunyi has joined #openstack-ironic | 13:51 | |
| *** foexle has joined #openstack-ironic | 13:58 | |
| *** Isotopp has joined #openstack-ironic | 13:59 | |
| *** jdob_ has joined #openstack-ironic | 14:00 | |
| Isotopp | Hello. Is there some component or plugin that is part of ironic and that takes part of doing HP firmware updates, | 14:02 |
| Isotopp | such as ilo, disk controller firmware or bios flashes? | 14:02 |
| *** praefect has quit IRC | 14:02 | |
| *** jcoufal has quit IRC | 14:03 | |
| Isotopp | that is, helps to automate the initial install of masses of new hardware whcih may or may not be up to date? | 14:03 |
| *** linggao has quit IRC | 14:04 | |
| *** mat128 has joined #openstack-ironic | 14:05 | |
| dtantsur | Isotopp, hi. I guess there's some work on ilo, see spec https://review.openstack.org/97455 | 14:07 |
| dtantsur | nothing seems to be ready-to-use | 14:07 |
| Isotopp | thank you, i will have a look | 14:08 |
| dtantsur | Folks, it's hard to stop, and now I have a new version of dashboard, which satisfies me, and I hope will satisfy rloo and Shrews: http://perm.ly/ironic-review-dashboard | 14:09 |
| dtantsur | (and yes, it has a nice short link) | 14:09 |
| dtantsur | Mind having a look? | 14:09 |
| * matty_dubs starts a dashboard listing dtantsur's dashboards | 14:09 | |
| matty_dubs | +1 for adding "My Patches Requiring Attention" | 14:10 |
| rloo | dtantsur: so you added a 'Needs Reverify' section? | 14:10 |
| romcheg | +1 | 14:10 |
| dtantsur | rloo, that's one of the most important points. Also your patches include WIP. Also WIP+nonverified section in the end | 14:10 |
| dtantsur | and specs also include WIP (arguable) | 14:11 |
| *** jdob_ has quit IRC | 14:11 | |
| dtantsur | that's also to address sdague's review comments | 14:11 |
| rloo | dtantsur: looks good. I'm fine with it. | 14:12 |
| dtantsur | wow! at this rate we end up with one version only :) | 14:12 |
| *** praefect has joined #openstack-ironic | 14:12 | |
| rloo | dtantsur: One thing I wondered. the 'Needs Approval' includes 'Needs one more +2' as well. It is fine with me as-is, just wondering if others understand it. | 14:12 |
| rloo | dtantsur: are you telling me you like this too? :D | 14:13 |
| dtantsur | rloo, yes, it's my first version updated and I like it :) | 14:13 |
| dtantsur | rloo, I always assumed that +A requires +2 first | 14:14 |
| rloo | dtantsur: well, +A requires two +2s first ;) | 14:15 |
| *** praefect1 has joined #openstack-ironic | 14:15 | |
| rloo | dtantsur: there are cases where some reviews have two or more +2s and really only need the approval ;) | 14:15 |
| rloo | dtantsur: anyway, i am fine the way it is. Just wanted to mention it. | 14:16 |
| dtantsur | rloo, well... yes, but I hope common sense will help in all these cases :) | 14:16 |
| dtantsur | ok, I'll post this one for review | 14:16 |
| *** praefect has quit IRC | 14:16 | |
| rloo | dtantsur: yup ;) | 14:17 |
| *** pradipta_away is now known as pradipta | 14:23 | |
| *** praefect1 has quit IRC | 14:36 | |
| *** uberj has quit IRC | 14:39 | |
| openstackgerrit | Victor Sergeyev proposed a change to openstack/ironic: WIP: use paramiko channel to run ssh command https://review.openstack.org/97523 | 14:48 |
| *** praefect has joined #openstack-ironic | 14:53 | |
| *** rakesh_hs has quit IRC | 14:55 | |
| *** rloo has quit IRC | 14:56 | |
| *** rloo has joined #openstack-ironic | 14:56 | |
| *** igordcard has quit IRC | 14:56 | |
| *** rloo has quit IRC | 14:57 | |
| openstackgerrit | Anusha Ramineni proposed a change to openstack/ironic-specs: iLO Power Driver for Ironic https://review.openstack.org/97455 | 14:57 |
| *** rloo has joined #openstack-ironic | 14:57 | |
| *** rloo has quit IRC | 14:59 | |
| *** rloo has joined #openstack-ironic | 15:00 | |
| *** jcoufal has joined #openstack-ironic | 15:02 | |
| *** jcoufal has quit IRC | 15:03 | |
| *** jcoufal has joined #openstack-ironic | 15:03 | |
| *** rloo has quit IRC | 15:04 | |
| *** rloo has joined #openstack-ironic | 15:04 | |
| *** yfujioka has quit IRC | 15:08 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add/Update docstrings in the Nova Ironic Driver https://review.openstack.org/97536 | 15:16 |
| *** jistr has quit IRC | 15:25 | |
| *** Mikhail_D_ltp has quit IRC | 15:26 | |
| *** romcheg has quit IRC | 15:29 | |
| *** rloo has quit IRC | 15:29 | |
| *** rloo has joined #openstack-ironic | 15:30 | |
| *** eghobo has joined #openstack-ironic | 15:44 | |
| *** romcheg has joined #openstack-ironic | 15:55 | |
| *** pradipta is now known as pradipta_away | 15:55 | |
| *** max_lobur has quit IRC | 15:56 | |
| *** matty_dubs is now known as matty_dubs|lunch | 15:57 | |
| *** derekh_ has quit IRC | 16:00 | |
| devananda | morning, all | 16:00 |
| rloo | morning devananda! | 16:02 |
| *** martyntaylor has left #openstack-ironic | 16:02 | |
| *** jrist-afk is now known as jrist | 16:04 | |
| *** pelix has quit IRC | 16:08 | |
| *** pelix has joined #openstack-ironic | 16:09 | |
| devananda | Isotopp: hi! so there is the iLO driver work, specific to HP hardware, for out-of-band updates to firmware (eventually, I haven't seen any code for that yet). There is the IPA (ironic-python-agent) work, which aims to use an agent to do in-band updates of firmware, etc | 16:11 |
| devananda | *And there is ... | 16:11 |
| dtantsur | morning, devananda | 16:12 |
| dtantsur | brb | 16:12 |
| *** dtantsur is now known as dtantsur|afk | 16:12 | |
| devananda | dtantsur: ++ for putting "my patches needing attention" at the top | 16:12 |
| dtantsur|afk | devananda, good! I got this merged to sdague's repo, will post to ML as well later | 16:13 |
| *** pelix has quit IRC | 16:13 | |
| *** pelix has joined #openstack-ironic | 16:14 | |
| *** hemna has joined #openstack-ironic | 16:17 | |
| *** ellenh has joined #openstack-ironic | 16:18 | |
| *** Alexei_987 has quit IRC | 16:28 | |
| devananda | gotta catch a ride, will bbi30m | 16:32 |
| *** dwalleck has joined #openstack-ironic | 16:35 | |
| *** hemna has quit IRC | 16:43 | |
| *** harlowja_away is now known as harlowja | 16:45 | |
| *** dwalleck has quit IRC | 16:47 | |
| NobodyCam | good morning from cheyenne | 16:47 |
| *** hemna has joined #openstack-ironic | 16:49 | |
| ryanpetrello | ironic folks - any insight into the failure seen here? https://review.openstack.org/#/c/97475/ | 16:49 |
| rloo | ryanpetrello: i've seen a bunch failing but haven't looked into it. Some people have been doing a 'recheck bug 1325689' but i don't know if that helps or not. | 16:53 |
| ryanpetrello | okay, cool | 16:54 |
| ryanpetrello | looks like there’s a bug for it | 16:54 |
| ryanpetrello | thanks! | 16:54 |
| *** ramineni has joined #openstack-ironic | 16:56 | |
| lucasagomes | morning rloo devananda :) | 16:57 |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: VendorPassthru.validate()s call _parse_driver_info https://review.openstack.org/97558 | 16:58 |
| rloo | hi lucasagomes | 16:58 |
| Isotopp | devananda: hi there. thanks. i will try to find all this and have a look at it | 16:58 |
| Isotopp | ... tomorrow, though | 16:58 |
| *** ifarkas has quit IRC | 17:02 | |
| *** lucasagomes is now known as lucas-dinner | 17:04 | |
| NobodyCam | morning rloo lucas-dinner and devananda | 17:05 |
| rloo | hiya NobodyCam in cheyenne ;) | 17:05 |
| NobodyCam | :) | 17:05 |
| NobodyCam | should be out of tornado range now | 17:05 |
| NobodyCam | lol | 17:05 |
| rloo | whew! | 17:06 |
| openstackgerrit | Anusha Ramineni proposed a change to openstack/ironic-specs: iLO Power Driver for Ironic https://review.openstack.org/97455 | 17:06 |
| *** ramineni has left #openstack-ironic | 17:09 | |
| *** rloo has quit IRC | 17:12 | |
| *** rloo has joined #openstack-ironic | 17:12 | |
| *** rloo has quit IRC | 17:12 | |
| *** rloo has joined #openstack-ironic | 17:13 | |
| Shrews | devananda: did you write this regex? b/c it's making my head hurt. https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/jenkins_job_builder/config/devstack-gate.yaml#L587-L602 | 17:18 |
| *** shakamunyi has quit IRC | 17:24 | |
| *** rloo has quit IRC | 17:24 | |
| *** rloo has joined #openstack-ironic | 17:24 | |
| *** matty_dubs|lunch is now known as matty_dubs | 17:27 | |
| *** jogo has left #openstack-ironic | 17:28 | |
| *** ellenh has quit IRC | 17:31 | |
| devananda | Shrews: that's adam_g's handiwork :) | 17:38 |
| *** eghobo has quit IRC | 17:39 | |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic: Log exceptions from deploy and tear_down https://review.openstack.org/97318 | 17:39 |
| Shrews | adam_g: to pick up our new advanced scenario tests, looks like i need to change the last regex to this? r="$r tempest.scenario.test_baremetal_*" | 17:41 |
| * Shrews not sure how one would test that | 17:42 | |
| *** zdin0bot has joined #openstack-ironic | 17:44 | |
| devananda | Shrews: i'm not sure we /can/ test that today. do you have a patch for that change up? if so, we should ask in -infra | 17:45 |
| *** rloo has quit IRC | 17:46 | |
| *** rloo has joined #openstack-ironic | 17:46 | |
| Shrews | devananda: not yet. sean mentioned i needed to change it first before https://review.openstack.org/94439 can be merged | 17:46 |
| Shrews | but i need to run to a high school graduation now, so i can check with infra when i return | 17:47 |
| devananda | Shrews: ah, yep. sdague is right. change regex -> observe that the new tests in the patch run in the check queue -> make sure they pass -> land changes to tempest | 17:49 |
| devananda | Shrews: if we were to land the tempest changes /before/ making the change to infra, we could break our gate for a while | 17:50 |
| devananda | if the tests didn't pass once they were enabled | 17:50 |
| Shrews | devananda: yeah. i just didn't know about that regex until today | 17:50 |
| Shrews | anyway, bbl | 17:50 |
| devananda | ciao | 17:50 |
| NobodyCam | have a good time Shrews | 17:51 |
| adam_g | Shrews, you should be able to test that regex just passing it to testr | 17:51 |
| adam_g | testr list-tests $REGEX | 17:51 |
| NobodyCam | woo hoo we could be HOME as soon as Sunday! | 17:56 |
| devananda | NobodyCam: \o/ | 17:57 |
| NobodyCam | :) | 17:57 |
| devananda | NobodyCam: also, you have email from me yesterday afternoon. do you have a minute to reply? | 17:58 |
| NobodyCam | oh devananda just fyi I am rebuilding my tox env as soon as I have the tests working I have the dont remake partitions when preserve_ephemeral is set patch ready to go | 17:59 |
| devananda | ack | 17:59 |
| NobodyCam | devananda: meetup timing email? | 18:01 |
| *** ndipanov has quit IRC | 18:04 | |
| devananda | NobodyCam: ya | 18:04 |
| NobodyCam | dropped.. :-p | 18:12 |
| NobodyCam | yes I will :) | 18:12 |
| *** ifarkas has joined #openstack-ironic | 18:13 | |
| NobodyCam | devananda: quick reply sent :-p | 18:20 |
| NobodyCam | woo hoo py27: commands succeeded | 18:21 |
| NobodyCam | :-p | 18:21 |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Wipe any metadata from a nodes disk https://review.openstack.org/93133 | 18:29 |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Rework make_partitions logic when preserve_ephemeral is set https://review.openstack.org/97590 | 18:31 |
| NobodyCam | lifeless: devananda: ^^^^^^ | 18:31 |
| NobodyCam | brb | 18:33 |
| *** zdin0bot has quit IRC | 18:36 | |
| *** romcheg has quit IRC | 18:36 | |
| *** max_lobur has joined #openstack-ironic | 18:42 | |
| NobodyCam | oh so nice to yet another timezone closer to home | 18:49 |
| *** rloo has quit IRC | 18:50 | |
| devananda | NobodyCam: did you see lifeless' previous comment on ^ ? | 18:50 |
| devananda | on 93133 | 18:50 |
| *** rloo has joined #openstack-ironic | 18:51 | |
| *** sysexit has quit IRC | 18:53 | |
| *** sysexit has joined #openstack-ironic | 18:54 | |
| NobodyCam | oh poop | 18:56 |
| NobodyCam | fixing :-p | 18:57 |
| *** pelix has quit IRC | 19:00 | |
| *** jistr has joined #openstack-ironic | 19:02 | |
| *** derekh_ has joined #openstack-ironic | 19:02 | |
| *** rloo has quit IRC | 19:09 | |
| *** rloo has joined #openstack-ironic | 19:09 | |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Rework make_partitions logic when preserve_ephemeral is set https://review.openstack.org/97590 | 19:09 |
| devananda | lucas-dinner: i'm not sure what this is doing yet, but it may be interesting for you re: json patch support in pecan -- https://github.com/stackforge/designate/blob/master/designate/api/v2/controllers/records.py#L102 | 19:14 |
| devananda | https://github.com/stackforge/designate/blob/master/designate/api/v2/patches.py | 19:15 |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Wipe any metadata from a nodes disk https://review.openstack.org/93133 | 19:18 |
| NobodyCam | ok I think that should be right | 19:19 |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Wipe any metadata from a nodes disk https://review.openstack.org/93133 | 19:20 |
| * NobodyCam tries the change commit message via web interface | 19:21 | |
| NobodyCam | :-p | 19:21 |
| NobodyCam | devananda: lifeless: order should be correct now... 93133 depends on 97590 | 19:21 |
| NobodyCam | :- | 19:21 |
| *** rloo has quit IRC | 19:23 | |
| *** rloo has joined #openstack-ironic | 19:23 | |
| *** rloo has quit IRC | 19:23 | |
| *** rloo has joined #openstack-ironic | 19:24 | |
| NobodyCam | doh | 19:24 |
| *** rloo has quit IRC | 19:25 | |
| *** rloo has joined #openstack-ironic | 19:25 | |
| * devananda gets lunch and relocates... back soon | 19:26 | |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Wipe any metadata from a nodes disk https://review.openstack.org/93133 | 19:26 |
| NobodyCam | and now with working test too | 19:27 |
| NobodyCam | :-p | 19:27 |
| NobodyCam | ohh zuul gone nuts | 19:30 |
| *** aswadrangnekar has left #openstack-ironic | 19:36 | |
| openstackgerrit | Jay Faulkner proposed a change to openstack/ironic-python-agent: Allow configdrive partition to be precreated https://review.openstack.org/95969 | 19:46 |
| JayF | dtantsur|afk: ^ when you get a chance, addressed your comments there. LMK what you think. Also we all are appreciative of the review attention you've been giving IPA, so thanks :) | 19:47 |
| NobodyCam | devananda: have you seen the comments on https://review.openstack.org/#/c/96558/11/ironic/drivers/modules/ipmitool.py | 19:47 |
| JayF | NobodyCam: those comments look like someone working under python 3, at least a lot of the comments are similar to the things I had to fix when trying to get py3 support for IPA (although it still doesn't work, because eventlet :X) | 19:48 |
| NobodyCam | ahh | 19:48 |
| JayF | that's just a guess though :) | 19:48 |
| NobodyCam | what I find odd id the comments seem to on code out side that patch | 19:49 |
| NobodyCam | s/id/is/ | 19:49 |
| NobodyCam | JayF: would you mind adding that as a comment to the patch so its logged there | 19:51 |
| JayF | It's a wild guess, hence why mentioning it here instead of there :) | 19:51 |
| JayF | not actual knowledge, just pattern matching :) | 19:51 |
| NobodyCam | :) ahhh | 19:51 |
| *** ellenh has joined #openstack-ironic | 19:53 | |
| *** pbrooko has joined #openstack-ironic | 19:58 | |
| NobodyCam | brb | 20:00 |
| openstackgerrit | Jay Faulkner proposed a change to openstack/ironic-python-agent: Readme should point to Openstack, not travis-ci https://review.openstack.org/97611 | 20:01 |
| devananda | NobodyCam: looks like comments are meant for the dependent patch (mine) | 20:04 |
| devananda | NobodyCam: also, we should really test those before landing them :p | 20:04 |
| *** derekh_ has quit IRC | 20:08 | |
| NobodyCam | devananda: yes. :-p | 20:11 |
| *** jcoufal has quit IRC | 20:15 | |
| *** ifarkas has quit IRC | 20:15 | |
| *** romcheg has joined #openstack-ironic | 20:19 | |
| *** jistr has quit IRC | 20:20 | |
| *** ellenh has quit IRC | 20:23 | |
| *** praefect has quit IRC | 20:33 | |
| *** jbjohnso has quit IRC | 20:39 | |
| *** ellenh has joined #openstack-ironic | 20:47 | |
| *** romcheg has left #openstack-ironic | 20:53 | |
| *** praefect has joined #openstack-ironic | 20:55 | |
| *** jdob has quit IRC | 21:00 | |
| *** igordcard has joined #openstack-ironic | 21:03 | |
| *** dhellmann has quit IRC | 21:11 | |
| *** dhellmann has joined #openstack-ironic | 21:13 | |
| *** Faizan has joined #openstack-ironic | 21:15 | |
| *** Faizan has quit IRC | 21:15 | |
| *** sysexit has quit IRC | 21:21 | |
| *** foexle has quit IRC | 21:25 | |
| openstackgerrit | linggao proposed a change to openstack/ironic: Test fake console driver https://review.openstack.org/97629 | 21:28 |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic-python-agent: Add missing methods to base HardwareManager class https://review.openstack.org/97631 | 21:31 |
| *** matty_dubs is now known as matty_dubs|gone | 21:32 | |
| *** mrda-away is now known as mrda | 21:58 | |
| mrda | Morning Ironic! | 22:02 |
| NobodyCam | good morning mrda | 22:02 |
| devananda | morning, mrda ! | 22:06 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Mock seamicroclient lib in unit tests if not present https://review.openstack.org/90174 | 22:14 |
| *** ccit has quit IRC | 22:23 | |
| *** hemna has quit IRC | 22:27 | |
| NobodyCam | devananda: is 96902 on your plate? | 22:28 |
| devananda | NobodyCam: was just looking at that | 22:28 |
| NobodyCam | :) something about great minds and alike :-p | 22:29 |
| devananda | thoughts on support for older ipmitool? | 22:29 |
| NobodyCam | I thihk we should. | 22:30 |
| NobodyCam | the first versions I put up had most of the logic to do the looking with in the code | 22:31 |
| NobodyCam | i kinda think it worth taking the hit on code and looping but be able to support a wider (version) of ipmitool | 22:32 |
| NobodyCam | s/looking/looping/ | 22:33 |
| devananda | NobodyCam: except ipmitool *does* that looping *anyway* | 22:33 |
| devananda | even if we don't pass the options | 22:33 |
| devananda | so doing it the originally proposed way is actually sending more IPMI packets than it appears to be | 22:34 |
| NobodyCam | *if the ipmitool version supports it | 22:34 |
| * NobodyCam is thinking of ways of autodetecting ipmi versions | 22:34 | |
| NobodyCam | :-p | 22:35 |
| devananda | NobodyCam: that's certainly possible, but i'm concerned about marking the bug fixed when it's not | 22:36 |
| devananda | fixed for older versions of ipmitool | 22:36 |
| NobodyCam | if we detected support for -N -R we could then set them and not loop | 22:37 |
| NobodyCam | ourselfs | 22:37 |
| NobodyCam | something simple like ipmitool --help|grep '\-N' | 22:38 |
| NobodyCam | if found set -n / r else loop in code | 22:38 |
| NobodyCam | that way we are "excluding" anyone | 22:40 |
| NobodyCam | s/anyone/distro's/ | 22:40 |
| mrda | I think that makes sense. | 22:40 |
| JayF | I wish it wasn't considered reasonable to run old software that doesn't support things like the -N/-R options for ipmitool :( | 22:41 |
| JayF | someone else already fixed it and went through that effort, seems crazy that we have to fix it too | 22:41 |
| NobodyCam | JayF: and you cann't check the version number because I have a 2.86 something (i think) and the official is 1.8.?? | 22:41 |
| JayF | like I /understand/ but I wish a different design decision w/r/t those things in openstack in general :( | 22:42 |
| NobodyCam | we could throw deprecated log message | 22:43 |
| JayF | +1 | 22:43 |
| JayF | 'WARN: Deprecated, use a modern ipmitool you insensitive clod' | 22:43 |
| NobodyCam | your ipmitool is OLD nag someone | 22:43 |
| mrda | U12.04 has ipmitool version 1.8.11.dell19 which has no -N and -R. Given this is still the Ubuntu people use for many things, we should detect if -N and -R are there for us to use. | 22:43 |
| mrda | (unfortunately) | 22:44 |
| *** jgrimm has quit IRC | 22:44 | |
| *** hemna has joined #openstack-ironic | 22:44 | |
| * JayF checks trusty | 22:44 | |
| JayF | FWIW debian 7 supports -N and -R as version '1.8.14' as reported by --help | 22:45 |
| NobodyCam | I believe .13 added it | 22:46 |
| JayF | as does Ubuntu 14.04 (trusty) 1.8.13 as reported by --help | 22:46 |
| mrda | so | 22:46 |
| mrda | much | 22:46 |
| mrda | fail | 22:46 |
| NobodyCam | but then I have nobodycamAir:openstack NobodyCam$ ipmitool -V | 22:47 |
| NobodyCam | ipmitool version 2.5b1 | 22:47 |
| mrda | major.minor.patch version numbering. How do you add in -N and -R in a patch series? | 22:47 |
| JayF | So I think checking for functionality, dropping a 'deprecated' message if it's missing is probably a good route to go | 22:48 |
| JayF | then we use the deprecated message we throw to guilt ubuntu into putting 1.8.13+ into precise-updates ;0 | 22:48 |
| NobodyCam | JayF: and suse ans fedora 20 | 22:49 |
| NobodyCam | freebds, os x | 22:49 |
| NobodyCam | :-p | 22:49 |
| JayF | Are there two separate commands that can exist as 'ipmitool'? | 22:49 |
| NobodyCam | freebsd* | 22:49 |
| JayF | that's what seems like is happening in some of these cases | 22:49 |
| NobodyCam | there is ipmitool | 22:50 |
| NobodyCam | and openipmi | 22:50 |
| JayF | The one I have on OS X is vastly different than the one in my linux vms (which is why I only run ipmitool from my vm) | 22:50 |
| JayF | I think ipmitool is provided by openipmi in debian 7 and trusty | 22:50 |
| JayF | because apt-get install ipmitool installed openipmi as a dep (maybe ipmitool is a virtual package pointed at openipmi?) | 22:50 |
| *** pbrooko has quit IRC | 22:51 | |
| NobodyCam | devananda: found a good link which I ofc don't have handy | 22:51 |
| NobodyCam | devananda: do you have that link for ipmitool vs openipmi | 22:51 |
| JayF | if he posted it in here I'll check logs | 22:51 |
| NobodyCam | bet it was | 22:52 |
| JayF | https://developer.apple.com/library/mac/documentation/Darwin/Reference/Manpages/man1/ipmitool.1.html vs http://linux.die.net/man/1/ipmitool | 22:52 |
| NobodyCam | I use a mac so I brew installed ipmitool then I just fiddle with my path to get the correct version | 22:53 |
| JayF | ah so ipmitool as installed by brew is the newer one | 22:53 |
| * JayF looks through the paths | 22:53 | |
| NobodyCam | nobodycamAir:openstack NobodyCam$ /usr/local/bin/ipmitool -V | 22:54 |
| NobodyCam | ipmitool version 1.8.13 | 22:54 |
| JayF | aha | 22:54 |
| JayF | that's very helpful, ty | 22:54 |
| NobodyCam | ) | 22:55 |
| JayF | fwiw I checked fedora 20, it has 1.8.13 and supports -N -R | 22:56 |
| devananda | NobodyCam: it's in the patch | 22:57 |
| JayF | docker containers are great for this :) docker run -t -i stackbrew/distro:version /bin/bash ; install ipmitool, find the support | 22:57 |
| NobodyCam | ok its darn near five, the rv park we are at has a BBQ place in the park (http://www.abbbq.com) so going to get food and beer :-p | 22:58 |
| JayF | enjoy, have a nice evening | 22:58 |
| NobodyCam | devananda: doh | 22:59 |
| NobodyCam | :-p | 22:59 |
| devananda | JayF: 1.8.13 on trusty has -N -R | 22:59 |
| devananda | NobodyCam: have a good evening :) | 23:00 |
| NobodyCam | hehehe | 23:00 |
| *** pcrews has joined #openstack-ironic | 23:00 | |
| * NobodyCam wants to be back to home base :-p | 23:00 | |
| * devananda adds a warning to the module load | 23:01 | |
| NobodyCam | olpen is turning on the intertub service as of friday | 23:01 |
| NobodyCam | oh for the ipitool version check | 23:01 |
| NobodyCam | ipmi tool | 23:01 |
| NobodyCam | that would be perfect place | 23:02 |
| NobodyCam | then WARN: Deprecated, you should use pymgi | 23:03 |
| NobodyCam | lol /me being evil would log the warn wiht every ipmitool validate call :-p | 23:09 |
| NobodyCam | with even | 23:09 |
| jroll | psh, log as critical | 23:10 |
| *** rloo has quit IRC | 23:10 | |
| *** rloo has joined #openstack-ironic | 23:10 | |
| *** hemna has quit IRC | 23:10 | |
| NobodyCam | lol | 23:10 |
| devananda | JayF: not that i'm lazy, but do you guys want to fix up that patch? | 23:10 |
| devananda | seems like you've got some opinions on it :) | 23:11 |
| *** rloo_ has joined #openstack-ironic | 23:11 | |
| *** igordcard has quit IRC | 23:11 | |
| *** JayF_ has joined #openstack-ironic | 23:16 | |
| *** JayF has quit IRC | 23:17 | |
| *** JayF_ is now known as JayF | 23:18 | |
| *** EmilienM_ has joined #openstack-ironic | 23:19 | |
| *** dtantsur has joined #openstack-ironic | 23:20 | |
| *** rloo has quit IRC | 23:20 | |
| *** dtantsur|afk has quit IRC | 23:20 | |
| *** EmilienM has quit IRC | 23:20 | |
| *** EmilienM_ is now known as EmilienM | 23:20 | |
| devananda | catching a bus, bbl o/ | 23:39 |
| *** lucas-dinner has quit IRC | 23:51 | |
| openstackgerrit | Jay Faulkner proposed a change to openstack/ironic-python-agent: Allow configdrive partition to be precreated https://review.openstack.org/95969 | 23:52 |
| *** radsy has joined #openstack-ironic | 23:59 | |
| *** radsy has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!