| *** eghobo has quit IRC | 01:19 | |
| openstackgerrit | lifeless proposed a change to openstack/ironic: Simplify error handling https://review.openstack.org/105050 | 01:19 |
|---|---|---|
| *** nosnos has joined #openstack-ironic | 01:37 | |
| *** ccrouch has quit IRC | 02:36 | |
| *** pcrews has quit IRC | 02:36 | |
| openstackgerrit | A change was merged to openstack/ironic: Simplify error handling https://review.openstack.org/105050 | 02:42 |
| *** ccrouch has joined #openstack-ironic | 02:45 | |
| *** ramineni has joined #openstack-ironic | 02:49 | |
| *** coolsvap|afk is now known as coolsvap | 03:05 | |
| *** aswadr has joined #openstack-ironic | 03:09 | |
| *** nosnos has quit IRC | 03:25 | |
| openstackgerrit | Daryl Walleck proposed a change to openstack/ironic: Updates the Ironic on Devstack dev documentation https://review.openstack.org/104469 | 03:35 |
| *** takadayuiko has joined #openstack-ironic | 03:41 | |
| *** Poornima has joined #openstack-ironic | 03:44 | |
| takadayuiko | lifeless: is here? | 03:57 |
| lifeless | I am | 03:59 |
| takadayuiko | lifeless: Hi, thank you for reviewing my patch 73121. Sorry, I cannot understand what you means. I would like to talk about it. | 04:00 |
| *** coolsvap is now known as coolsvap|afk | 04:00 | |
| *** radsy has quit IRC | 04:01 | |
| *** nosnos has joined #openstack-ironic | 04:02 | |
| lifeless | takadayuiko: I haven't reviewed 73121 - DB details leaking to API on error | 04:03 |
| takadayuiko | Lifeless, Oops, I mistook. 104738. | 04:03 |
| *** eghobo has joined #openstack-ironic | 04:07 | |
| lifeless | takadayuiko: ok, which bits are unclear? | 04:12 |
| takadayuiko | lifeless: thank you. Do you mean that it's better to write neutron endpoint url as "http://192.168.0.10:9696" instead of "http" + "://" * "192.168.0.10" + ":" 9696 ? | 04:23 |
| lifeless | takadayuiko: if it has to be written at all, yes. | 04:25 |
| lifeless | takadayuiko: but thats opinion; if there is a consensus elsewhere, or a good reason to change - thats fine | 04:27 |
| lifeless | takadayuiko: but - it needs to be backwards compatible; and it really shouldn't need to be set at all as keystone knows the neutron URL. | 04:27 |
| takadayuiko | lifeless: OK, I understood what you said. I myself think this method is better same as glance(glance_host, glance_port, glance_protocol) | 04:29 |
| takadayuiko | lifeless: By changing the method of writing URL, backwards compatible will be lacked, right? | 04:31 |
| *** eghobo has quit IRC | 04:37 | |
| lifeless | sorry, whats the relevance of glance here? | 04:39 |
| lifeless | yes, if we stop supporting the url format its a backwards incompatible change | 04:39 |
| lifeless | we should have a good reason to do that | 04:39 |
| *** eghobo has joined #openstack-ironic | 04:41 | |
| takadayuiko | lifeless, Ah, there is no relevance. But the way of writing of Glance URL looks smart. | 04:41 |
| takadayuiko | I understood the importance of backwards incompatible. And my main purpose is setting my_ip as neutron URL instead of localhost. so the writing style is out of scope. I think I'll update my patch. | 04:44 |
| *** k4n0 has joined #openstack-ironic | 04:45 | |
| takadayuiko | Mmm... but Lucas took this patch as the agenda of the next meeting. I think it's better to discuss about it together with everyone. | 04:47 |
| *** sabah has joined #openstack-ironic | 04:49 | |
| *** ajc_ has joined #openstack-ironic | 04:52 | |
| *** geekyogi has joined #openstack-ironic | 04:58 | |
| lifeless | takadayuiko: I think the default should be to have no url set at all and read neutron out of keystone. | 04:58 |
| openstackgerrit | Anusha Ramineni proposed a change to openstack/ironic-specs: Out-Of-Band firmware update for Baremetal Servers https://review.openstack.org/100842 | 04:58 |
| takadayuiko | lifeless, about this your idea, I'm not sure, sorry. We should discuss about it. | 05:00 |
| takadayuiko | And I have to leave here. thank you lifeless! :) | 05:01 |
| *** ramineni has quit IRC | 05:04 | |
| *** takadayuiko has quit IRC | 05:05 | |
| *** amitpp has joined #openstack-ironic | 05:07 | |
| *** coolsvap|afk is now known as coolsvap | 05:30 | |
| *** killer_prince is now known as lazy_prince | 05:35 | |
| *** geekyogi has quit IRC | 05:55 | |
| *** bvivek has joined #openstack-ironic | 06:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/103291 | 06:02 |
| *** lazy_prince is now known as killer_prince | 06:06 | |
| *** eghobo has quit IRC | 06:15 | |
| *** eghobo has joined #openstack-ironic | 06:15 | |
| *** eghobo has quit IRC | 06:16 | |
| *** pradipta_away is now known as pradipta | 06:23 | |
| *** bvivek has quit IRC | 06:39 | |
| *** bvivek has joined #openstack-ironic | 06:40 | |
| *** killer_prince is now known as lazy_prince | 06:43 | |
| *** shardy_afk is now known as shardy | 06:56 | |
| *** jcoufal has joined #openstack-ironic | 07:15 | |
| openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: Sync Oslo notifier module to Ironic https://review.openstack.org/105076 | 07:34 |
| *** mkerrin has quit IRC | 07:41 | |
| *** mitz_ has quit IRC | 07:41 | |
| *** mkerrin has joined #openstack-ironic | 07:45 | |
| *** rakesh_hs has joined #openstack-ironic | 07:48 | |
| *** ndipanov has joined #openstack-ironic | 07:51 | |
| *** derekh_ has joined #openstack-ironic | 08:11 | |
| *** bvivek has quit IRC | 08:14 | |
| *** mitz_ has joined #openstack-ironic | 08:23 | |
| *** bvivek has joined #openstack-ironic | 08:26 | |
| *** lucasagomes has joined #openstack-ironic | 08:27 | |
| *** athomas has joined #openstack-ironic | 08:36 | |
| *** Poornima has quit IRC | 08:44 | |
| *** romcheg has joined #openstack-ironic | 08:51 | |
| *** coolsvap is now known as coolsvap|afk | 08:52 | |
| *** geekyogi has joined #openstack-ironic | 08:54 | |
| *** amitpp has quit IRC | 09:00 | |
| *** coolsvap|afk is now known as coolsvap | 09:04 | |
| *** lazy_prince is now known as killer_prince | 09:12 | |
| *** pelix has joined #openstack-ironic | 09:18 | |
| *** amitpp has joined #openstack-ironic | 09:23 | |
| *** martyntaylor has joined #openstack-ironic | 09:30 | |
| *** andreykurilin has quit IRC | 09:32 | |
| *** killer_prince is now known as lazy_prince | 09:32 | |
| romcheg | Good morning Ironic! | 09:38 |
| *** athomas has quit IRC | 09:39 | |
| *** athomas has joined #openstack-ironic | 09:39 | |
| *** lazy_prince has quit IRC | 09:52 | |
| *** subah has joined #openstack-ironic | 10:08 | |
| *** sabah has quit IRC | 10:08 | |
| *** k4n0 has quit IRC | 10:13 | |
| *** romcheg has quit IRC | 10:16 | |
| *** sabah has joined #openstack-ironic | 10:22 | |
| *** subah has quit IRC | 10:22 | |
| *** k4n0 has joined #openstack-ironic | 10:26 | |
| *** k4n0 has quit IRC | 10:38 | |
| *** lazy_prince has joined #openstack-ironic | 10:38 | |
| *** overlayer has joined #openstack-ironic | 10:38 | |
| *** overlayer has quit IRC | 10:48 | |
| *** bvivek has quit IRC | 10:50 | |
| *** Haomeng|2 has joined #openstack-ironic | 10:51 | |
| *** Haomeng has quit IRC | 10:52 | |
| *** k4n0 has joined #openstack-ironic | 11:01 | |
| *** coolsvap is now known as coolsvap|afk | 11:08 | |
| *** pradipta is now known as pradipta_away | 11:15 | |
| *** coolsvap|afk is now known as coolsvap | 11:16 | |
| *** romcheg has joined #openstack-ironic | 11:29 | |
| *** geekyogi1 has joined #openstack-ironic | 11:29 | |
| *** k4n0 has quit IRC | 11:31 | |
| *** geekyogi has quit IRC | 11:32 | |
| *** lucasagomes is now known as lucas-hungry | 11:34 | |
| *** subah has joined #openstack-ironic | 11:43 | |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Initial oslo.i18n migration https://review.openstack.org/105132 | 11:44 |
| *** subah has quit IRC | 11:46 | |
| *** subah has joined #openstack-ironic | 11:46 | |
| *** sabah has quit IRC | 11:46 | |
| *** coolsvap is now known as coolsvap|afk | 11:46 | |
| *** subah has quit IRC | 11:51 | |
| *** krtaylor_away is now known as krtaylor | 11:51 | |
| *** Haomeng has joined #openstack-ironic | 11:54 | |
| *** Haomeng|2 has quit IRC | 11:55 | |
| *** bvivek has joined #openstack-ironic | 12:03 | |
| *** jcoufal has quit IRC | 12:12 | |
| *** geekyogi1 has quit IRC | 12:12 | |
| *** ccrouch has left #openstack-ironic | 12:16 | |
| *** jcoufal has joined #openstack-ironic | 12:17 | |
| *** bvivek has quit IRC | 12:20 | |
| *** Shrews has quit IRC | 12:24 | |
| *** Shrews has joined #openstack-ironic | 12:25 | |
| *** lucas-hungry is now known as lucasagomes | 12:26 | |
| *** mgagne has quit IRC | 12:28 | |
| *** aweeks has quit IRC | 12:29 | |
| *** wendar has quit IRC | 12:29 | |
| *** wendar has joined #openstack-ironic | 12:29 | |
| *** coolsvapl has joined #openstack-ironic | 12:30 | |
| *** GheRiver1 has joined #openstack-ironic | 12:30 | |
| *** BadCub_ has joined #openstack-ironic | 12:30 | |
| *** coolsvap|afk has quit IRC | 12:30 | |
| *** antonym has quit IRC | 12:30 | |
| *** GheRivero has quit IRC | 12:31 | |
| *** yuriyz has quit IRC | 12:31 | |
| *** BadCub has quit IRC | 12:31 | |
| *** aweeks_ has joined #openstack-ironic | 12:31 | |
| *** mgagne has joined #openstack-ironic | 12:31 | |
| *** coolsvapl has quit IRC | 12:31 | |
| *** coolsvapl has joined #openstack-ironic | 12:31 | |
| *** aweeks_ has quit IRC | 12:31 | |
| *** aweeks_ has joined #openstack-ironic | 12:31 | |
| *** yuriyz has joined #openstack-ironic | 12:32 | |
| *** antonym has joined #openstack-ironic | 12:32 | |
| *** ajc_ has quit IRC | 12:33 | |
| *** k4n0 has joined #openstack-ironic | 12:44 | |
| *** k4n0 has quit IRC | 12:47 | |
| *** rakesh_hs has quit IRC | 12:56 | |
| *** nosnos has quit IRC | 13:02 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add set_spawn_error_hook to TaskManager https://review.openstack.org/100957 | 13:04 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Fix nodes left in an incosistent state if no workers https://review.openstack.org/100958 | 13:04 |
| *** amitpp has quit IRC | 13:05 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Fix nodes left in an incosistent state if no workers https://review.openstack.org/100958 | 13:08 |
| *** jbjohnso_ has joined #openstack-ironic | 13:18 | |
| * romcheg written some changes to grenade | 13:21 | |
| romcheg | But I have no idea what I just did, because there's no docs for this stuff :( | 13:23 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/105167 | 13:28 |
| *** dhellmann_ is now known as dhellmann | 13:29 | |
| *** matty_dubs|gone is now known as matty_dubs | 13:43 | |
| NobodyCam | good morning ironic | 13:46 |
| Shrews | NobodyCam: morning | 13:47 |
| NobodyCam | :) morning Shrews | 13:47 |
| lucasagomes | morning romcheg NobodyCam Shrews | 13:51 |
| *** dhellmann is now known as dhellmann_ | 13:51 | |
| romcheg | Morning NobodyCam Shrews lucasagomes! | 13:51 |
| Shrews | o/ | 13:51 |
| matty_dubs | Howdy fellers | 13:51 |
| lucasagomes | morning matty_dubs | 13:51 |
| * romcheg is about to start killing people around | 13:59 | |
| romcheg | When I was struggling with infra integration, I didn't know that there is Grenade… : | 13:59 |
| lucasagomes | lol? | 13:59 |
| romcheg | :) | 13:59 |
| NobodyCam | morning lucasagomes romcheg and matty_dubs | 13:59 |
| lucasagomes | romcheg, ah thought it was neutron :P | 14:00 |
| * matty_dubs steps back from IRC, out of romcheg's view | 14:00 | |
| lucasagomes | romcheg, btw, it's now working for me | 14:00 |
| lucasagomes | romcheg, don't ask me what I didnt, I rebuilt my devstack env | 14:00 |
| lucasagomes | and now neutron works | 14:00 |
| *** rloo has joined #openstack-ironic | 14:05 | |
| romcheg | lucasagomes: It keeps failing in random places for me :) | 14:06 |
| lucasagomes | romcheg, heh | 14:11 |
| *** jgrimm has joined #openstack-ironic | 14:20 | |
| *** geekyogi has joined #openstack-ironic | 14:26 | |
| *** dhellmann_ is now known as dhellmann | 14:26 | |
| *** lazy_prince has quit IRC | 14:37 | |
| BadCub_ | Morning :-p | 14:38 |
| NobodyCam | morning BadCub_ | 14:43 |
| NobodyCam | lucasagomes: love the var cat="meow" | 14:43 |
| lucasagomes | NobodyCam, lol | 14:43 |
| NobodyCam | :) | 14:44 |
| NobodyCam | lucasagomes: do you see _set_states_on_error getting used else where? | 14:50 |
| lucasagomes | NobodyCam, apart from deploy and tear down? | 14:51 |
| * lucasagomes thinking | 14:52 | |
| lucasagomes | NobodyCam, I don't think so because that's touch the provisioning state part | 14:53 |
| NobodyCam | I was woundering if it would be worth a name change, maybe something like: _set_node_state_after_error | 14:53 |
| NobodyCam | ??? | 14:53 |
| NobodyCam | but ... maybe not | 14:53 |
| NobodyCam | not really sure if thats more clear or not | 14:54 |
| NobodyCam | got a conf call.. now... bbiab | 14:55 |
| lucasagomes | NobodyCam, right, yeah naming things is not my thing really :( | 14:58 |
| NobodyCam | :) its a good name .. but may be a bit unclear | 14:59 |
| NobodyCam | I'm trying to think | 15:00 |
| openstackgerrit | A change was merged to openstack/ironic: Updates the Ironic on Devstack dev documentation https://review.openstack.org/104469 | 15:01 |
| *** dwalleck has joined #openstack-ironic | 15:05 | |
| *** killer_prince has joined #openstack-ironic | 15:13 | |
| *** killer_prince is now known as lazy_prince | 15:14 | |
| *** overlayer has joined #openstack-ironic | 15:19 | |
| *** jcoufal has quit IRC | 15:22 | |
| *** jcoufal has joined #openstack-ironic | 15:24 | |
| jroll | mornin y'all | 15:29 |
| matty_dubs | Howdy jroll | 15:29 |
| romcheg | lucasagomes: Did grenade fail for you because of version mismatch for six? | 15:30 |
| NobodyCam | morning jroll | 15:34 |
| NobodyCam | have a good weekend | 15:34 |
| jroll | heya NobodyCam | 15:34 |
| jroll | twas great, you? | 15:34 |
| *** coolsvapl is now known as coolsvap | 15:37 | |
| *** overlayer has quit IRC | 15:41 | |
| *** aswadr has quit IRC | 15:42 | |
| NobodyCam | was nice... | 15:42 |
| NobodyCam | :) | 15:42 |
| lucasagomes | romcheg, where? | 15:43 |
| romcheg | lucasagomes: grenade.sh when installing icehouse | 15:43 |
| lucasagomes | didn't try it :/ | 15:43 |
| lucasagomes | lemme check | 15:43 |
| *** rwsu has joined #openstack-ironic | 15:47 | |
| matty_dubs | InstanceDeployFailure: Disk partitioning failed on device /tmp/tmpXqo2W6/tmp24f0J7. Fuser exited with "Cannot stat file /proc/21639/fd/0: Permission denied | 15:55 |
| *** Nisha has joined #openstack-ironic | 15:55 | |
| matty_dubs | Haven't seen that before. Do I maybe need to update root_wrap? | 15:56 |
| lucasagomes | matty_dubs, yes, ifarkas patch to use fuser to verify if the disk is not been held by another process was merged last friday | 15:57 |
| lucasagomes | so fuser should be in ur rootwrap | 15:57 |
| matty_dubs | Oh, I see. I just need to update what's in git | 15:59 |
| *** pcrews has joined #openstack-ironic | 15:59 | |
| *** matty_dubs is now known as matty_dubs|lunch | 16:01 | |
| *** jcoufal has quit IRC | 16:05 | |
| *** blamar has joined #openstack-ironic | 16:09 | |
| *** dwalleck has quit IRC | 16:11 | |
| *** coolsvap is now known as coolsvap|afk | 16:16 | |
| NobodyCam | bbt... brb | 16:22 |
| *** derekh_ has quit IRC | 16:28 | |
| *** datajerk has quit IRC | 16:30 | |
| *** jcoufal has joined #openstack-ironic | 16:32 | |
| *** dwalleck has joined #openstack-ironic | 16:34 | |
| *** datajerk has joined #openstack-ironic | 16:35 | |
| *** martyntaylor has quit IRC | 16:37 | |
| *** ndipanov has quit IRC | 16:41 | |
| *** ramineni has joined #openstack-ironic | 16:42 | |
| *** romcheg has quit IRC | 16:45 | |
| *** ramineni has quit IRC | 16:48 | |
| *** martyntaylor has joined #openstack-ironic | 16:51 | |
| NobodyCam | no distributions at all found for lxml >=2.3 hummmmm..... | 16:52 |
| Shrews | jroll: the more i look into this concept of "hypervisor uptime" and how it relates to ironic makes me want to kick puppies | 16:53 |
| jroll | Shrews: :) | 16:56 |
| *** lucasagomes is now known as lucas-afk | 16:58 | |
| *** harlowja_away is now known as harlowja | 17:01 | |
| devananda | morning, all | 17:03 |
| *** rloo has quit IRC | 17:11 | |
| jroll | morning devananda | 17:13 |
| *** romcheg has joined #openstack-ironic | 17:18 | |
| *** lazy_prince is now known as killer_prince | 17:19 | |
| NobodyCam | good mornign devananda | 17:22 |
| Shrews | FYI, I plan to ask for advice re: this bug during today's meeting (https://bugs.launchpad.net/ironic/+bug/1337460), if people want to read it ahead of time. :) | 17:22 |
| *** matty_dubs|lunch is now known as matty_dubs | 17:26 | |
| adam_g | anyone know whats up with my tox py27 environment? http://paste.ubuntu.com/7761142/ | 17:26 |
| *** eghobo has joined #openstack-ironic | 17:35 | |
| devananda | Shrews: that's fun. Im curious what the nova team will think | 17:37 |
| devananda | Shrews: it's returning the uptime for each compute host, right? | 17:38 |
| jroll | is that actually useful info, though? | 17:39 |
| jroll | for any of the hypervisor-* commands, not just -uptime | 17:40 |
| Shrews | devananda: best as i can tell (from this http://docs.openstack.org/api/openstack-compute/2/content/ext-os-hypervisors.html), it's supposed to. But not even the libvirt driver does it properly | 17:41 |
| devananda | jroll: stats and list are, though they don' tmean the normal things | 17:41 |
| matty_dubs | It feels like the 'right' thing is to report Ironic's uptime, but I'm not sure what one might do with that information. | 17:41 |
| devananda | or i'm thinking of something else | 17:42 |
| JayF | if you define it that way, you rapidly get into 'what does uptime mean' :) | 17:42 |
| devananda | yea | 17:42 |
| devananda | it seems like returning the instance uptime is the wrong thing here | 17:42 |
| jroll | devananda: welllll, those display node info, not n-cpu info | 17:42 |
| devananda | jroll: right | 17:43 |
| devananda | jroll: which can be useful :) | 17:43 |
| jroll | seems like hypervisor-uptime would be equivalent to 'node uptime' | 17:43 |
| devananda | but that's really equivlane to instance uptime | 17:43 |
| jroll | today. | 17:43 |
| jroll | :) | 17:44 |
| devananda | ah, heh | 17:44 |
| jroll | idk. maybe it should be 'time since provisioned' for now... think about 'ramdisk uptime' later | 17:45 |
| devananda | so this is a good example where ironic doesn't really fit in nova's API | 17:45 |
| devananda | rather than shoehorn it in very awkardly | 17:45 |
| devananda | and confuse users | 17:45 |
| jroll | right | 17:45 |
| jroll | just don't implement | 17:45 |
| jroll | (imo) | 17:45 |
| *** hemna has quit IRC | 17:45 | |
| devananda | I think we should punt to Nova | 17:45 |
| Shrews | I mean, we could just return an empty string | 17:45 |
| devananda | is there a way to indicate that "this is intentionallynot implemented" ? | 17:45 |
| *** hemna has joined #openstack-ironic | 17:45 | |
| Shrews | devananda: raise NotImplementedException | 17:45 |
| Shrews | :) | 17:46 |
| devananda | or, alternately, we have these only return the info for the n=cpu host itself | 17:46 |
| devananda | which is less useful from *our* POV but probably the correct thing from Nova's POV | 17:46 |
| *** pelix has quit IRC | 17:46 | |
| devananda | Shrews: right. and then tempest complains :) | 17:46 |
| Shrews | well, a <space> then :-P | 17:46 |
| devananda | heh | 17:47 |
| Shrews | but like i said, not even libvirt does the "right thing" | 17:47 |
| Shrews | seems like an api someone once thought might be useful, but turns out it wasn't, so it's forgotten about | 17:48 |
| devananda | except it's not forgotten about by tempest | 17:48 |
| rameshg87 | hello devananda | 17:49 |
| devananda | rameshg87: hi | 17:49 |
| Shrews | dprince doesn't think any driver does it right: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4834-L4835 | 17:50 |
| *** rloo has joined #openstack-ironic | 17:50 | |
| rameshg87 | devananda: a question regarding ilo power driver spec https://review.openstack.org/#/c/97455/ | 17:51 |
| *** rloo has quit IRC | 17:51 | |
| rameshg87 | devananda: do we need approval from one more core reviewer for this spec ? | 17:51 |
| *** rloo has joined #openstack-ironic | 17:52 | |
| devananda | Shrews: hm, is "host" there the "hypervisor_hostname" ? | 17:54 |
| NobodyCam | I have a spec question: RE: spec dependenceies. as it pertains to my last comment on https://review.openstack.org/#/c/100951 | 17:54 |
| devananda | Shrews: if so, then most hypervisors will be ignoring it at that layer, since only baremetal/ironic/vmware support >1 hypervisor_hostname per n-cpu | 17:54 |
| Shrews | devananda: i believe so, from the compute_nodes table | 17:55 |
| devananda | Shrews: right. then that's why it's ignored there. | 17:55 |
| *** pelix has joined #openstack-ironic | 17:56 | |
| Shrews | devananda: vmware does... return 'Please refer to %s for the uptime' % CONF.vmware.host_ip | 17:57 |
| Shrews | so they defer the work.. hehe.. nice | 17:57 |
| *** rloo has quit IRC | 17:58 | |
| *** rloo has joined #openstack-ironic | 17:58 | |
| *** rloo has quit IRC | 18:00 | |
| devananda | russell_h: ping | 18:00 |
| jroll | devananda: russell_h is out for the next two weeks | 18:01 |
| jroll | getting married / honeymoon | 18:01 |
| devananda | jroll: ahh. seems to be taht time of year :) | 18:01 |
| jroll | heh | 18:02 |
| jroll | anything I can help with? | 18:02 |
| devananda | jroll: specs-core team has 2/5 members out for two weeks ... | 18:02 |
| jroll | gah | 18:02 |
| *** rloo_ has joined #openstack-ironic | 18:03 | |
| *** rloo_ has left #openstack-ironic | 18:03 | |
| *** rloo has joined #openstack-ironic | 18:05 | |
| NobodyCam | devananda: woud you have a free minute to take a look at my last comment on https://review.openstack.org/#/c/100951 ? | 18:06 |
| rameshg87 | devananda: will we need to wait for specs-core reviewers to be back ? | 18:07 |
| *** jgrimm has quit IRC | 18:08 | |
| jroll | rameshg87: there are still 3 spec cores around | 18:08 |
| jroll | 2 is enough for approval | 18:08 |
| rameshg87 | jroll: one basic questions, is the core reviewers list available somewhere ? :-) | 18:09 |
| jroll | rameshg87: yes, looking | 18:10 |
| *** eghobo has quit IRC | 18:11 | |
| JayF | It's a gerrit group | 18:13 |
| JayF | but I always have a ton of trouble 'discovering' gerrit groups, or finding the list, even knowing the name :x | 18:13 |
| jroll | yeah :/ | 18:14 |
| rameshg87 | devananda: please have a look at the ilo deploy spec https://review.openstack.org/#/c/97744/ | 18:14 |
| rameshg87 | devananda: i have revised it after your comments. | 18:15 |
| *** chuckC has quit IRC | 18:15 | |
| jroll | rameshg87: I can't find the list, sorry :/ | 18:17 |
| rameshg87 | jroll: np :-) | 18:18 |
| *** rameshg87 is now known as rameshg87_afk | 18:22 | |
| *** eghobo has joined #openstack-ironic | 18:22 | |
| *** eghobo has quit IRC | 18:23 | |
| *** eghobo has joined #openstack-ironic | 18:24 | |
| NobodyCam | rameshg87_afk: JayF: jroll: https://review.openstack.org/#/admin/groups/352,members | 18:24 |
| rloo | rameshg87_afk, jroll : https://review.openstack.org/#/admin/groups/352,members | 18:24 |
| NobodyCam | lol | 18:25 |
| rloo | NobodyCam: it was a draw ;) | 18:25 |
| NobodyCam | hehehe | 18:25 |
| NobodyCam | great minds .... | 18:25 |
| NobodyCam | :-p | 18:25 |
| jroll | ah ha | 18:25 |
| rloo | that too :D | 18:25 |
| jroll | thanks NobodyCam | 18:25 |
| *** eghobo has quit IRC | 18:28 | |
| NobodyCam | bb.. b4 meeting | 18:31 |
| *** dwalleck has quit IRC | 18:31 | |
| devananda | re gerrit groups, the full list is browsable here | 18:31 |
| devananda | https://review.openstack.org/#/admin/groups/ | 18:31 |
| devananda | just type in "ironic-specs" in the filter and you'l find it | 18:32 |
| *** foexle has joined #openstack-ironic | 18:32 | |
| rloo | jroll, NobodyCam, rameshg87_afk, devananda: I just added a link to that in our wiki: https://wiki.openstack.org/wiki/Ironic/Specs_Process#Ironic_Specs_Process | 18:32 |
| jroll | rloo: thanks | 18:32 |
| JayF | nice, thanks devananda (and thanks to NobodyCam and rloo for the direct link) | 18:36 |
| matty_dubs | How are core reviewers for the specs chosen? | 18:38 |
| matty_dubs | (Not that I have any issue with the current list; I'm just curious.) | 18:38 |
| NobodyCam | matty_dubs: they volunteer | 18:39 |
| devananda | matty_dubs: i throw spaghetti at them | 18:39 |
| devananda | matty_dubs: if it sticks, they're on the team | 18:39 |
| matty_dubs | lol | 18:39 |
| devananda | :) | 18:39 |
| NobodyCam | lol | 18:39 |
| NobodyCam | spaghetti is better then a wet cat (which has also been throwen at /me) | 18:40 |
| devananda | ouch | 18:40 |
| NobodyCam | lol | 18:40 |
| matty_dubs | Spaghetti's a lot less likely to claw you. Also, a tasty snack. | 18:41 |
| *** dwalleck has joined #openstack-ironic | 18:42 | |
| devananda | lucas-afk: have time to review /approve some specs tomorrow? | 18:42 |
| *** tatyana has joined #openstack-ironic | 18:43 | |
| lucas-afk | devananda, hey morning, yes! | 18:45 |
| lucas-afk | devananda, tomorrow sounds good to me | 18:46 |
| devananda | lucas-afk: cool. lemme get some #'s for ya | 18:46 |
| lucas-afk | ack | 18:46 |
| *** lucas-afk is now known as lucasagomes | 18:46 | |
| devananda | lucasagomes: https://review.openstack.org/102914 https://review.openstack.org/97455 https://review.openstack.org/98904 | 18:49 |
| devananda | lucasagomes: those three look ready to me | 18:49 |
| lucasagomes | devananda, ack, I will take a look tomorrow morning | 18:49 |
| devananda | thanks much! | 18:49 |
| lucasagomes | yvw | 18:50 |
| NobodyCam | 10 minutes til meeting, last chance for coffee, soda, and other rest stop activities | 18:52 |
| *** takadayuiko has joined #openstack-ironic | 18:53 | |
| takadayuiko | Hi, ironic :) | 18:53 |
| NobodyCam | good morning takadayuiko | 18:53 |
| takadayuiko | NobodyCam, good morning! | 18:54 |
| sirushti | devananda, Hi, could you please take a look at https://review.openstack.org/#/c/97150/? I've responded to your comments | 18:54 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Specify rootfstype=ramfs deploy kernel parameter https://review.openstack.org/104728 | 18:54 |
| NobodyCam | last chance to update the agenda | 18:55 |
| *** eghobo has joined #openstack-ironic | 18:56 | |
| *** eghobo has quit IRC | 18:57 | |
| *** eghobo has joined #openstack-ironic | 18:58 | |
| * lucasagomes looks at the agenda | 18:58 | |
| lucasagomes | takadayuiko, morning, wow it's quite late there no? | 18:58 |
| lucasagomes | or quite early... | 18:59 |
| devananda | sirushti: the claim in that paragraph is still false | 18:59 |
| devananda | sirushti: it is possible to deploy windows today using partition image and pxe boot | 18:59 |
| devananda | sirushti: * with some changes to the template | 18:59 |
| sirushti | devananda, That sounds cool, is there a link for this? | 18:59 |
| takadayuiko | lucasgomes, good morning! yes, it's 4:00 AM, super early :O | 18:59 |
| devananda | sirushti: I put a link in my comment on the spec last week | 19:00 |
| sirushti | devananda, that's just chainloading | 19:00 |
| devananda | sirushti: correct | 19:00 |
| sirushti | not really pxe-booting, right? | 19:00 |
| devananda | sirushti: let's continue after meeting | 19:00 |
| sirushti | devananda, Sure | 19:00 |
| NobodyCam | meeting time | 19:01 |
| *** foexle has quit IRC | 19:02 | |
| *** tatyana has quit IRC | 19:04 | |
| *** tatyana has joined #openstack-ironic | 19:09 | |
| *** jgrimm has joined #openstack-ironic | 19:09 | |
| *** martyntaylor has quit IRC | 19:21 | |
| *** mkerrin1 has joined #openstack-ironic | 19:28 | |
| *** mkerrin has quit IRC | 19:29 | |
| *** Nisha has quit IRC | 19:37 | |
| *** pelix has quit IRC | 19:38 | |
| *** chuckC has joined #openstack-ironic | 19:39 | |
| *** sseago has quit IRC | 19:49 | |
| *** Penick has joined #openstack-ironic | 19:49 | |
| jroll | wanyen... isn't here. | 20:01 |
| NobodyCam | great meeting all | 20:01 |
| BadCub_ | good meeting :-) | 20:01 |
| NobodyCam | looks like he logged off | 20:01 |
| jroll | well, he wasn't here before thay | 20:01 |
| jroll | that | 20:01 |
| *** wanyen has joined #openstack-ironic | 20:01 | |
| devananda | on the generic spec -> specific spec dependnecy | 20:01 |
| devananda | I asked for things like hardware discuvery to be written up in a generic way specifically so that they are addressed in a way that works for more than one vendor | 20:02 |
| devananda | i do not want to approve the spec (or the code) for one vendor before the generic spec | 20:02 |
| NobodyCam | ++ | 20:02 |
| devananda | because it is likely to create technical debt -- we'll have to refactor it to make it fit the generic model we eventually adopt (or remove it) | 20:03 |
| devananda | OTOH, that is the main purpose for the /vendor_passthru/ API | 20:03 |
| devananda | so I have no objection to a vendor implementing it *there* | 20:03 |
| devananda | when ever they want to | 20:03 |
| *** rloo has quit IRC | 20:04 | |
| jroll | +1 | 20:04 |
| *** rloo has joined #openstack-ironic | 20:04 | |
| NobodyCam | devananda: my concern is that we will approve a spec like this and then the dependent spce will change | 20:04 |
| wanyen | So, I have concern that no other discovery code can be approved until the generic hw discovery bit code has been approved. because the dependency is the inventory field should it just be dependencnt on the lading of tnat data structure? | 20:05 |
| NobodyCam | for things like changing the datamodel | 20:05 |
| NobodyCam | ie adding inventor field | 20:05 |
| devananda | NobodyCam: that's one reason I dont think they should be approved until the common spec is approved | 20:05 |
| *** tatyana has quit IRC | 20:05 | |
| jroll | wanyen: the common spec also defines the API interfaces | 20:05 |
| devananda | wanyen: did you see my comments here just after the meeting? | 20:05 |
| jroll | for discovery | 20:05 |
| jroll | what's the point of implementing "discovery on node-create" if the discovery code doesn't exist? | 20:06 |
| devananda | wanyen: http://paste.openstack.org/show/85613/ | 20:06 |
| lucasagomes | devananda, wanyen just approved the iLO power driver spec | 20:07 |
| *** takadayuiko has quit IRC | 20:07 | |
| lucasagomes | thanks for that, small and direct spec | 20:07 |
| wanyen | deva> yes. I saw taht . | 20:07 |
| devananda | jroll: why is https://review.openstack.org/#/c/95551 a dependency for ipa driver? | 20:08 |
| NobodyCam | gah ... brb | 20:08 |
| lucasagomes | I will take a look at the other specs tomorrow, it's late here | 20:08 |
| jroll | devananda: it shouldn't be, any more. did you get that from ipa-todos or did I forget to rebase? | 20:08 |
| devananda | lucasagomes: indeed it is :) have a good evening! | 20:08 |
| lucasagomes | (and I will still tired from the trip to the ceilometer midcycle) | 20:08 |
| wanyen | >deva> it seems to that the generic hw discovery bit focus on in-band discovery which can be very different from oob discovery | 20:08 |
| devananda | jroll: ipa-todos | 20:09 |
| lucasagomes | devananda, thanks you too | 20:09 |
| lucasagomes | have a great night everybody | 20:09 |
| jroll | devananda: right, no longer a dep | 20:09 |
| devananda | jroll: great | 20:09 |
| *** lucasagomes is now known as lucas-dinner | 20:09 | |
| devananda | jroll: i'll stop reading it until ya'll update it :) | 20:09 |
| jroll | I just removed that line | 20:09 |
| jroll | devananda: that section is accurate now | 20:09 |
| jroll | devananda: | 20:09 |
| jroll | oops | 20:09 |
| devananda | oh, reminder for folks that will be at the midcyucle (i should have pointed this out in the meeting) | 20:10 |
| devananda | I'm tracking agenda topics here: https://etherpad.openstack.org/p/juno-ironic-sprint | 20:10 |
| jroll | devananda: you can look at the reviews in that section... then the bottom needs to be updated for the branches coming off the main agent driver spec | 20:10 |
| jroll | or I can just ping you when I update :P | 20:10 |
| openstackgerrit | A change was merged to openstack/ironic-specs: iLO Power Driver for Ironic https://review.openstack.org/97455 | 20:10 |
| devananda | jroll: i doubt i'll get much time for that today, but hopefully this week I can give them a read agaon | 20:11 |
| devananda | again | 20:11 |
| jroll | devananda: sounds good :) I have specs to update too | 20:11 |
| wanyen | lucasagomes> ty very much for approving the ilo power spec. | 20:11 |
| devananda | jroll: i'm eagerly awaiting the day when you point me at a walkthrough to get devstack up and running with it :) | 20:11 |
| jroll | devananda: soon (this week?). my plan is to point at our devstack docs :) | 20:12 |
| * jroll brb | 20:13 | |
| sirushti | devananda, Hi, could we continue our discussion? | 20:16 |
| *** dwalleck has quit IRC | 20:27 | |
| *** dwalleck_ has joined #openstack-ironic | 20:30 | |
| devananda | sirushti: hi! i have a few minutes | 20:31 |
| devananda | sirushti: so it is pxe booting in that the server issues a DHCP BOOT request, and the response is controllable by Ironic | 20:32 |
| sirushti | devananda, that should do :-) so, as I was saying one can't pxe-boot windows using partition images+chainloading | 20:32 |
| sirushti | devananda, ok | 20:32 |
| sirushti | devananda, so anything involving pxelinux.0 is assumed to be pxe-boot? | 20:33 |
| sirushti | I thought pxe-boot is pxe acting as the bootloader and not something on the disk to be called pxe boot. Am I missing | 20:33 |
| sirushti | because chain-load actually gives its responsibility to the node and the node boots using the local bootloader | 20:34 |
| *** Hefeweizen has joined #openstack-ironic | 20:35 | |
| sirushti | *s/missing/missing something/ | 20:36 |
| *** foexle has joined #openstack-ironic | 20:40 | |
| openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic-specs: Whole Disk Image Support for PXE Deploy Driver https://review.openstack.org/97150 | 20:41 |
| devananda | sirushti: as i understand it, this will work even if the local HDD has no boot flag set | 20:42 |
| JayF | I saw that assumption in the spec. It's not awesome to assume that HDD will boot if PXE times out (because that's not always true) | 20:43 |
| JayF | but I don't think that's a good reason to disapprove the spec | 20:43 |
| sirushti | devananda, this = the link you provided? | 20:43 |
| JayF | Opportunistically supporting boxes that /can/ have their bios set to fallback but have no IPMI is a great idea, and I actually loved seeing it in the spec | 20:43 |
| JayF | sounds like a good way to support other things (like the agent) on boxes without explicit IPMI support (but maybe having power_driver hooked up to a ups, for instance) | 20:44 |
| sirushti | JayF, thats only until we get the changes into the ssh driver to change the boot order though | 20:44 |
| JayF | sirushti: I think it's a good fallback scenario to be aware of in any situation. Earlier in the life of onmetal, we had thought of doing just that: boxes that always pxe'd, and had a power strip hooked up with a power driver, and just letting the fallback work | 20:45 |
| sirushti | JayF, ooh, nice :-) | 20:46 |
| devananda | sirushti: ssh power driver's ability to change the boot order on VMs will enable some better testing in virtual environments | 20:47 |
| devananda | sirushti: but does not cover other power drivers (like PDU) which will not have such an ability | 20:47 |
| JayF | I look forward to finding even funkier hardware to boot with Ironic | 20:48 |
| devananda | so, to rephrase my objection to rev.11 | 20:48 |
| devananda | this statement is illogical: "Since Windows cannot be PXE-Booted via a certain25 kernel/ramdisk, there's no choice but to deploy whole disk Windows images. | 20:48 |
| devananda | " | 20:48 |
| JayF | I honestly thought of doing an ironic-provisioned cluster of raspberry pis once, but they don't have any pxe support whatsoever (without putting an OS on that chains into pxe) | 20:48 |
| sirushti | devananda, hmm, perhaps, I should just remove that line altogether :-p I only wanted to emphasize that windows would be a good use case | 20:49 |
| devananda | JayF: stick a uboot image on the pi's sdcard that chains into ipxe ;) | 20:49 |
| JayF | devananda: yeah but then I can't have IPA write a full image to it :P | 20:50 |
| JayF | devananda: what I really want to do is buy an Orange Box from Ubuntu, make it work with Ironic, and show them how well their box works with a /real/ openstack provisioning method :) | 20:50 |
| devananda | JayF: right. well.... | 20:50 |
| devananda | JayF: the NUCs are available on amazon | 20:51 |
| JayF | well yeah, and it'd be sweet, but for now I'm spinning up bigger boxes :P | 20:51 |
| devananda | :) | 20:51 |
| devananda | sirushti: i agree that windows is a good use case :) | 20:51 |
| devananda | sirushti: just not that it is impossible to use windows without the changes that spec describes | 20:51 |
| devananda | which is how I interpret that sentence | 20:52 |
| Shrews | devananda: about the tempest test for uptime... looks like we're just going to have to skip it if baremetal driver is enabled. We can't make it consider 501 a valid response w/o affecting the whole test suite. | 20:52 |
| devananda | :( | 20:52 |
| Shrews | which makes me "grrrrr" | 20:52 |
| sirushti | devananda, I was actually looking out for resources that would let me pxe-boot w/ windows and still have preserve_ephemeral, not much luck really | 20:52 |
| sirushti | but there probably is a way, so I suppose it is not right to say that it is impossible :-) | 20:53 |
| devananda | sirushti: as it is written today, preserve ephemeral won't work at all with whole-disk image | 20:54 |
| sirushti | devananda, right, that was just off topic trying to find a way getting windows to be partition-booted | 20:55 |
| devananda | lunch time, then i have a bunch of paperwork to do .... probably lurking the rest of the day, but not really here | 20:55 |
| NobodyCam | ack enjoy lunch devananda | 20:55 |
| *** dwalleck_ has quit IRC | 20:56 | |
| *** dwalleck has joined #openstack-ironic | 20:57 | |
| openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic-specs: Whole Disk Image Support for PXE Deploy Driver https://review.openstack.org/97150 | 20:58 |
| *** dwalleck_ has joined #openstack-ironic | 20:58 | |
| sirushti | devananda, JayF thanks for the input :-) Please do have a look https://review.openstack.org/97150 | 20:58 |
| *** dwalleck has quit IRC | 21:01 | |
| *** harlowja is now known as harlowja_away | 21:02 | |
| *** GheRiver1 is now known as GheRivero | 21:05 | |
| *** jbjohnso_ has quit IRC | 21:06 | |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic: Add more INFO logging to ironic/conductor https://review.openstack.org/99744 | 21:11 |
| jroll | how do I specify an option as None in ironic.conf? :/ | 21:16 |
| jroll | option=, option=None, option=<None> do not work | 21:16 |
| jroll | all produce strings | 21:16 |
| *** jcoufal has quit IRC | 21:18 | |
| *** dwalleck_ has quit IRC | 21:19 | |
| Shrews | jroll: is there a default? | 21:23 |
| *** dwalleck has joined #openstack-ironic | 21:23 | |
| jroll | Shrews: nope | 21:23 |
| Shrews | if not, then don't specify it... or is that broken? | 21:23 |
| jroll | looking at oslo.config | 21:23 |
| jroll | oh | 21:24 |
| jroll | could work | 21:24 |
| jroll | hmm /me tries | 21:24 |
| * NobodyCam thinks devananda's idea about food was a good thing, and goes off to look for soething to eat.. bbiaf | 21:30 | |
| *** foexle has quit IRC | 21:31 | |
| jroll | Shrews: that did it, thanks | 21:32 |
| * jroll fights keystone now | 21:32 | |
| *** matty_dubs is now known as matty_dubs|gone | 21:33 | |
| Shrews | jroll: i iz a jeenys | 21:33 |
| jroll | :P | 21:33 |
| Shrews | jeenyus... geez, i can't spell | 21:34 |
| Shrews | :) | 21:34 |
| rloo | Shrews, do you have a few min to look at https://review.openstack.org/#/c/100957/? I want to make sure you're OK with it. | 21:49 |
| *** dwalleck has quit IRC | 21:50 | |
| *** dwalleck has joined #openstack-ironic | 21:52 | |
| NobodyCam | jroll: was looking at the tests for https://review.openstack.org/#/c/100364 | 21:54 |
| *** max_lobur has joined #openstack-ironic | 21:55 | |
| jroll | wow, so many -1's | 21:55 |
| NobodyCam | several of the test have no assert at all? | 21:55 |
| jroll | NobodyCam: what's up? | 21:55 |
| jroll | uh | 21:56 |
| NobodyCam | https://review.openstack.org/#/c/100364/8/ironic/tests/drivers/test_ipmitool.py line 768 | 21:56 |
| jroll | NobodyCam: which? | 21:56 |
| jroll | oh | 21:56 |
| NobodyCam | 747 | 21:56 |
| jroll | oops :) | 21:56 |
| NobodyCam | hehehe note I did not point that on the review | 21:57 |
| NobodyCam | :-p | 21:57 |
| jroll | oh wait | 21:57 |
| jroll | hrmmm | 21:57 |
| *** mkerrin1 has quit IRC | 21:57 | |
| *** mkerrin has joined #openstack-ironic | 21:57 | |
| jroll | NobodyCam: thanks, will fix | 21:58 |
| NobodyCam | I think thats actually okay. but I always try and assert something | 21:58 |
| jroll | right | 21:58 |
| jroll | actually, ok, I remember | 21:58 |
| jroll | NobodyCam: that just calls vendor.validate() | 21:59 |
| jroll | which just validates args or whatever | 21:59 |
| jroll | so, it not raising an exception is the only way you can tell it failed | 21:59 |
| jroll | s/failed/passed/ | 21:59 |
| jroll | brb | 22:00 |
| *** harlowja_away is now known as harlowja | 22:04 | |
| *** romcheg has quit IRC | 22:10 | |
| *** dkehnx1 has joined #openstack-ironic | 22:16 | |
| NobodyCam | jroll: when your back please have a look at https://review.openstack.org/#/c/103595 with reguards to https://review.openstack.org/#/c/100734 | 22:26 |
| jroll | gah | 22:27 |
| jroll | it would be nice if that depended on my patch or vice versa | 22:28 |
| NobodyCam | :) we can poke lucas tomorrow | 22:29 |
| NobodyCam | hehehe | 22:29 |
| jroll | yeah | 22:29 |
| jroll | he even mentioned it in a comment | 22:29 |
| NobodyCam | ya | 22:29 |
| jroll | so much to do | 22:29 |
| NobodyCam | that series of patches is quite long :-p | 22:30 |
| openstackgerrit | A change was merged to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/103291 | 22:30 |
| jroll | only the first is getting any love | 22:30 |
| NobodyCam | :) | 22:34 |
| NobodyCam | I'm looking :) | 22:35 |
| NobodyCam | hehehe ,,, but gota run to the store... brb | 22:35 |
| *** Penick has quit IRC | 22:41 | |
| *** steveh1 has joined #openstack-ironic | 22:45 | |
| Shrews | rloo: re: 100957, not the way i'd do it (i consider it a design anti-pattern), but not here to be preachy about such things. fine to +A if you guys want. :) | 22:49 |
| *** dwalleck has quit IRC | 22:51 | |
| rloo | thx Shrews. | 23:14 |
| * NobodyCam wanders afk... /me need to touch up some paint on the house | 23:28 | |
| *** lucas-dinner has quit IRC | 23:41 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!