| *** praneshp has quit IRC | 00:00 | |
| *** praneshp has joined #openstack-ironic | 00:01 | |
| *** naohirot has joined #openstack-ironic | 00:02 | |
| *** shoutm has joined #openstack-ironic | 00:02 | |
| *** Sukhdev has quit IRC | 00:04 | |
| *** MattMan has left #openstack-ironic | 00:04 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Document GET ...raid/logical_disk_properties https://review.openstack.org/224356 | 00:06 |
|---|---|---|
| *** Sukhdev has joined #openstack-ironic | 00:10 | |
| *** praneshp has quit IRC | 00:10 | |
| *** mdbooth has quit IRC | 00:10 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic-specs: Update RAID spec based on what was coded https://review.openstack.org/224358 | 00:11 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Update descriptions in RAID config schema https://review.openstack.org/224299 | 00:13 |
| *** devlaps has quit IRC | 00:15 | |
| *** harshs has joined #openstack-ironic | 00:16 | |
| *** mdbooth has joined #openstack-ironic | 00:16 | |
| *** smoriya has joined #openstack-ironic | 00:17 | |
| *** tsekiyama has quit IRC | 00:20 | |
| *** mtanino has quit IRC | 00:20 | |
| *** shadower has quit IRC | 00:23 | |
| *** Sukhdev has quit IRC | 00:23 | |
| *** shadower has joined #openstack-ironic | 00:23 | |
| *** achanda_ has quit IRC | 00:31 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Add more info to checksum exception https://review.openstack.org/217369 | 00:31 |
| openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Add more info to checksum exception https://review.openstack.org/217369 | 00:31 |
| *** thrash is now known as thrash|g0ne | 00:31 | |
| *** achanda has joined #openstack-ironic | 00:33 | |
| *** achanda has quit IRC | 00:38 | |
| *** garthb has quit IRC | 00:45 | |
| *** Haomeng|2 has joined #openstack-ironic | 00:46 | |
| *** Haomeng has quit IRC | 00:49 | |
| *** ijw_ has quit IRC | 00:54 | |
| *** dims_ has quit IRC | 01:01 | |
| *** achanda has joined #openstack-ironic | 01:22 | |
| *** dims_ has joined #openstack-ironic | 01:26 | |
| *** zhenguo has joined #openstack-ironic | 01:31 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Base IronicObject on VersionedObject https://review.openstack.org/213602 | 01:37 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Remove unused object function https://review.openstack.org/222082 | 01:37 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Use oslo.versionedobjects remotable decorators https://review.openstack.org/217000 | 01:37 |
| *** achanda has quit IRC | 01:41 | |
| *** chlong has joined #openstack-ironic | 01:47 | |
| *** bizarrochristy has joined #openstack-ironic | 01:51 | |
| *** chlong_ has joined #openstack-ironic | 01:53 | |
| *** yog_ has joined #openstack-ironic | 01:53 | |
| *** shoutm has quit IRC | 01:55 | |
| *** Haomeng has joined #openstack-ironic | 01:57 | |
| *** Haomeng|2 has quit IRC | 02:00 | |
| *** r-daneel has quit IRC | 02:02 | |
| *** bizarrochristy has quit IRC | 02:02 | |
| *** bizarrochristy has joined #openstack-ironic | 02:03 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic-specs: Update RAID spec based on what was coded https://review.openstack.org/224358 | 02:05 |
| *** yog_ has quit IRC | 02:09 | |
| *** shoutm has joined #openstack-ironic | 02:19 | |
| *** rloo has quit IRC | 02:31 | |
| *** david-lyle has joined #openstack-ironic | 02:34 | |
| *** yog_ has joined #openstack-ironic | 02:38 | |
| *** achanda has joined #openstack-ironic | 02:41 | |
| *** harshs_ has joined #openstack-ironic | 02:44 | |
| *** achanda has quit IRC | 02:45 | |
| *** harshs has quit IRC | 02:46 | |
| *** harshs_ is now known as harshs | 02:46 | |
| *** chlong_ has quit IRC | 02:50 | |
| *** VikasC has joined #openstack-ironic | 02:59 | |
| *** achanda has joined #openstack-ironic | 03:03 | |
| *** yog_ has quit IRC | 03:08 | |
| *** yog_ has joined #openstack-ironic | 03:09 | |
| *** coolsvap is now known as coolsvap|holiday | 03:17 | |
| *** bizarrochristy has quit IRC | 03:18 | |
| *** purp has quit IRC | 03:22 | |
| *** purp has joined #openstack-ironic | 03:22 | |
| *** praneshp has joined #openstack-ironic | 03:22 | |
| *** achanda has quit IRC | 03:35 | |
| *** achanda has joined #openstack-ironic | 03:35 | |
| *** penick has joined #openstack-ironic | 03:40 | |
| *** olaph has quit IRC | 03:41 | |
| *** praneshp_ has joined #openstack-ironic | 03:43 | |
| *** praneshp has quit IRC | 03:43 | |
| *** praneshp_ is now known as praneshp | 03:43 | |
| *** boris-42 has quit IRC | 03:50 | |
| *** yog_ has quit IRC | 03:50 | |
| *** boris-42 has joined #openstack-ironic | 03:51 | |
| *** sdake_ has joined #openstack-ironic | 03:52 | |
| *** yog_ has joined #openstack-ironic | 03:52 | |
| *** sdake has quit IRC | 03:54 | |
| *** ijw_ has joined #openstack-ironic | 04:11 | |
| *** harshs has quit IRC | 04:12 | |
| *** ijw_ has quit IRC | 04:15 | |
| *** chlong has quit IRC | 04:16 | |
| *** shoutm_ has joined #openstack-ironic | 04:17 | |
| *** shoutm has quit IRC | 04:19 | |
| *** olaph has joined #openstack-ironic | 04:21 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: [WIP]Support rootwrap https://review.openstack.org/223930 | 04:24 |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: Support rootwrap https://review.openstack.org/223930 | 04:24 |
| *** amotoki has joined #openstack-ironic | 04:25 | |
| *** chlong has joined #openstack-ironic | 04:33 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-specs: Get executable events API https://review.openstack.org/224022 | 04:37 |
| *** dims_ has quit IRC | 04:38 | |
| *** achanda has quit IRC | 04:39 | |
| *** dims_ has joined #openstack-ironic | 04:43 | |
| *** dims_ has quit IRC | 04:44 | |
| *** sdake has joined #openstack-ironic | 04:46 | |
| *** chlong has quit IRC | 04:48 | |
| *** sdake_ has quit IRC | 04:49 | |
| *** shoutm_ has quit IRC | 04:50 | |
| *** shoutm has joined #openstack-ironic | 04:52 | |
| *** shoutm has quit IRC | 05:06 | |
| *** shoutm has joined #openstack-ironic | 05:10 | |
| *** penick has quit IRC | 05:10 | |
| *** achanda has joined #openstack-ironic | 05:20 | |
| *** shoutm has quit IRC | 05:41 | |
| *** shoutm has joined #openstack-ironic | 05:43 | |
| *** dims_ has joined #openstack-ironic | 05:45 | |
| *** dims_ has quit IRC | 05:53 | |
| *** Sukhdev has joined #openstack-ironic | 05:54 | |
| *** ukalifon has joined #openstack-ironic | 06:00 | |
| *** lsmola has joined #openstack-ironic | 06:01 | |
| *** chenglch has joined #openstack-ironic | 06:11 | |
| *** chenke has joined #openstack-ironic | 06:17 | |
| chenke | Haomeng:how are u? | 06:17 |
| *** chlong has joined #openstack-ironic | 06:18 | |
| Haomeng | chenke: fine, and you? | 06:23 |
| chenke | Haomeng: find too. seems we can enable swap while "nova boot" a barematal node, do you know what config should be changed for that? | 06:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Zanata https://review.openstack.org/224240 | 06:25 |
| *** balajin has quit IRC | 06:28 | |
| *** praneshp has quit IRC | 06:29 | |
| *** zer0c00l has quit IRC | 06:29 | |
| *** harlowja has quit IRC | 06:29 | |
| *** zhenguo has quit IRC | 06:30 | |
| *** zhenguo has joined #openstack-ironic | 06:32 | |
| *** zer0c00l has joined #openstack-ironic | 06:33 | |
| *** balajin has joined #openstack-ironic | 06:36 | |
| *** e0ne has joined #openstack-ironic | 06:56 | |
| Haomeng | chenke: should be swap_mb parameters to control the size | 06:57 |
| Haomeng | chenke: you want to disable/enable swap paration? | 06:57 |
| chenke | Haomeng: i want to enable it | 06:58 |
| Haomeng | chenke: yes, set swap_mb properties value can enable | 06:58 |
| Haomeng | chenke: node's properties | 06:58 |
| chenke | Haomeng: now the issue is: the partition is right, but it doesn't mount as swap | 06:58 |
| chenke | Haomeng: you mean set ironic node property to add swap | 06:59 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove DictMatches custom matcher from unit tests https://review.openstack.org/224287 | 06:59 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix string formatting issues https://review.openstack.org/222862 | 06:59 |
| Haomeng | chenke: did you set it already? | 06:59 |
| Haomeng | chenke: I understand your swap paration is created but it is not enabled in os, right? | 07:00 |
| chenke | Haomeng: no, i am not. | 07:00 |
| chenke | Haomeng:yes. | 07:00 |
| Haomeng | chenke: "now the issue is: the partition is right, but it doesn't mount as swap" | 07:00 |
| Haomeng | did you check /etc/fstab | 07:01 |
| chenke | Haomeng: yes. there is no swap configration in /etc/fstab. when i set it, the swap is ok. | 07:02 |
| chenke | Haomeng: but i understand it should be set by ironic while deploy. right? | 07:02 |
| Haomeng | chenke: I can confirm the swap paration is created, but not sure if it is enabled during os starting | 07:03 |
| *** e0ne has quit IRC | 07:03 | |
| chenke | chenke: yes. it should be like what i said. because i verify it just 1 month ago. now my openstack env was deploy again, but it failed set swap. | 07:05 |
| chenke | Haomeng: yes. it should be like what i said. because i verify it just 1 month ago. now my openstack env was deploy again, but it failed set swap. | 07:05 |
| Haomeng | chenke: what driver you are using, agent or ipmitool? | 07:06 |
| Haomeng | chenke: pxe_impitool | 07:06 |
| chenke | Haomeng: yes. pxe | 07:06 |
| Haomeng | chenke: https://github.com/openstack/ironic/blob/68ec9be6a0abe4946edf55fa8337e2c9854bc397/ironic/drivers/modules/deploy_utils.py#L284 | 07:06 |
| chenke | Haomeng: can you confirm that the swap should be set in the node or flavor? | 07:08 |
| Haomeng | chenke: should be node properties I think | 07:08 |
| *** Sukhdev has quit IRC | 07:11 | |
| Haomeng | chenke: let me confirm | 07:16 |
| chenke | Haomeng:tku. i also turn to verify it in my env. | 07:17 |
| Haomeng | chenke: should be in flavors, sorry for confusing | 07:18 |
| Haomeng | chenke: it will be set into instance_info | 07:18 |
| Haomeng | chenke: after nova booting | 07:18 |
| chenke | Haomeng: tku. so i need to check why no config in /etc/fstab. | 07:20 |
| Haomeng | chenke: yes, not sure if it is depending on image | 07:23 |
| Haomeng | chenke: if it can auto-mount | 07:23 |
| Haomeng | chenke: ironic just help to create the swap paration | 07:23 |
| *** dtantsur|afk is now known as dtantsur | 07:27 | |
| dtantsur | Morning Ironic | 07:27 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Fix func test job broken by Inspector func tests changes https://review.openstack.org/224268 | 07:32 |
| *** alex_xu_ is now known as alex_xu | 07:32 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Support for introspection rules https://review.openstack.org/223096 | 07:32 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Support for getting introspection data https://review.openstack.org/224075 | 07:36 |
| *** achanda has quit IRC | 07:38 | |
| *** yuikotakada has joined #openstack-ironic | 07:39 | |
| yuikotakada | morning Ironic | 07:39 |
| *** achanda has joined #openstack-ironic | 07:47 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Add a simple 'introspection rule show' command https://review.openstack.org/224471 | 07:50 |
| *** achanda has quit IRC | 07:50 | |
| *** dims_ has joined #openstack-ironic | 07:51 | |
| *** Marga_ has quit IRC | 07:53 | |
| *** shuquan has joined #openstack-ironic | 07:55 | |
| *** dims_ has quit IRC | 07:57 | |
| *** ifarkas has joined #openstack-ironic | 07:59 | |
| dtantsur | yuikotakada, o/ | 07:59 |
| yuikotakada | dtantsur, o/ | 08:00 |
| *** dlpartain has joined #openstack-ironic | 08:01 | |
| *** dlpartain has left #openstack-ironic | 08:01 | |
| -openstackstatus- NOTICE: Gate is currently stuck, failing grenade upgrade tests due the release of oslo.utils 1.4.1 for Juno. | 08:01 | |
| dtantsur | oh damn ^^ | 08:03 |
| dtantsur | but it does not affect inspector. I guess | 08:03 |
| *** shoutm_ has joined #openstack-ironic | 08:06 | |
| *** shoutm has quit IRC | 08:08 | |
| *** romainh has joined #openstack-ironic | 08:12 | |
| *** jistr has joined #openstack-ironic | 08:20 | |
| dtantsur | yuikotakada, could you please review https://review.openstack.org/#/c/224075/ and https://review.openstack.org/#/c/224471/ ? We need a client release today | 08:20 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Fix upgrades by implementing indirection_api https://review.openstack.org/224079 | 08:21 |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: Support rootwrap https://review.openstack.org/223930 | 08:26 |
| *** ndipanov has joined #openstack-ironic | 08:27 | |
| *** lucasagomes has joined #openstack-ironic | 08:29 | |
| *** derekh has joined #openstack-ironic | 08:33 | |
| *** mgoddard has joined #openstack-ironic | 08:37 | |
| *** Marga_ has joined #openstack-ironic | 08:39 | |
| *** romcheg has joined #openstack-ironic | 08:44 | |
| *** Marga_ has quit IRC | 08:47 | |
| yuikotakada | dtantsur, output of "glance image-list" seems has been changed, do you know? | 08:51 |
| dtantsur | yuikotakada, sorry? that is possible, but is it important for us? | 08:52 |
| yuikotakada | dtantsur, https://github.com/openstack/ironic-inspector/blob/master/devstack/exercise.sh#L191 fails | 08:53 |
| dtantsur | yuikotakada, weird we don't catch it in gate... what's in image-list now? we should raise it to glance folks, if it's really a breaking change | 08:54 |
| yuikotakada | dtantsur, just ID and Name. does it cause only in my env...? I will investigate | 08:55 |
| *** alexpilotti has joined #openstack-ironic | 08:55 | |
| dtantsur | yuikotakada, in my env as well... | 08:56 |
| lucasagomes | glance broke other stuff as well | 08:59 |
| yuikotakada | dtantsur, lucasagomes, that's too bad | 08:59 |
| lucasagomes | they now point to the v2 API by default in their client | 08:59 |
| dtantsur | lucasagomes, damn. that's not very nice thing to do just before the release | 08:59 |
| lucasagomes | dtantsur, yeah I put a doc up these days to fix the image-create command that has changed :-/ | 09:01 |
| lucasagomes | yuikotakada, dtantsur btw https://review.openstack.org/#/c/219802/ | 09:01 |
| dtantsur | yuikotakada, I'll put up a patch | 09:03 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Allow abort for CLEANWAIT states https://review.openstack.org/201552 | 09:06 |
| yuikotakada | dtantsur, thank you :) | 09:08 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Replace glance and keystone commands with OSC equivalents https://review.openstack.org/224501 | 09:08 |
| dtantsur | yuikotakada, ^^ | 09:08 |
| dtantsur | lucasagomes, ^^^ | 09:09 |
| lucasagomes | dtantsur, cool, there are others to be replaced there as well | 09:12 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Allow abort for CLEANWAIT states https://review.openstack.org/201552 | 09:13 |
| dtantsur | lucasagomes, yep, I'll replace them soonish. but these are particularly bad, as keystone utility is deprecated, and glance is going to break us | 09:14 |
| lucasagomes | dtantsur, yeah the keystone there is exactly the same command you just replaced | 09:14 |
| dtantsur | ah, you meant more keystone things, I just read the comment :) | 09:14 |
| lucasagomes | dtantsur, yup | 09:14 |
| lucasagomes | there are two more | 09:14 |
| sambetts | Ah the pythonclients breaking everything again I see :-P | 09:15 |
| dtantsur | lucasagomes, openstack endpoint list is empty :( I remember there was some v2 vs v3 thingy... | 09:15 |
| lucasagomes | dtantsur, oh :-/ | 09:17 |
| yuikotakada | sambetts, o/ | 09:17 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Replace glance and keystone commands with OSC equivalents https://review.openstack.org/224501 | 09:17 |
| *** athomas has joined #openstack-ironic | 09:17 | |
| dtantsur | morning sambetts! yeah, usual fun... and these people are talking about inspector-client feature freeze :D | 09:17 |
| dtantsur | sambetts, I've prepared a 'rule show' command for your reviewing pleasure: https://review.openstack.org/#/c/224471/ | 09:18 |
| dtantsur | :) | 09:18 |
| dtantsur | also, lets land the last 2 patches, and release the damned client finally :) | 09:18 |
| yuikotakada | dtantsur, lucasagomes so that all we can do is Dmitry's patch as is :) | 09:19 |
| dtantsur | yeah... yuikotakada could you please check that it actually fixes things for you? | 09:20 |
| dtantsur | oh, you already did, right/ | 09:20 |
| lucasagomes | I just +a that... are you still testing it? | 09:20 |
| lucasagomes | dtantsur, ^ ? | 09:20 |
| dtantsur | lucasagomes, I've tested only the commands. And if our gate passes, I think we're fine | 09:21 |
| lucasagomes | yeah fair enuff, won't merge if it fails | 09:21 |
| yuikotakada | dtantsur ^^ | 09:21 |
| sambetts | dtantsur: Aweseome | 09:22 |
| * sambetts looks :) | 09:22 | |
| *** pelix has joined #openstack-ironic | 09:27 | |
| sambetts | dtantsur: Oh wow thats confusing!! The Cliff docs refer to rows as columns! http://docs.openstack.org/developer/cliff/demoapp.html#show-py | 09:31 |
| lucasagomes | sambetts, lol! | 09:34 |
| * lucasagomes remembers the javascript api for media | 09:34 | |
| lucasagomes | sambetts, http://www.w3schools.com/jsref/met_video_canplaytype.asp | 09:35 |
| lucasagomes | check the return values | 09:35 |
| *** shuquan has quit IRC | 09:35 | |
| sambetts | lucasagomes: haha | 09:35 |
| dtantsur | sambetts, yeah :) and I wonder why couldn't they just accept a dict | 09:40 |
| sambetts | dtantsur: No idea, its a bit odd :-p | 09:41 |
| dtantsur | OSC is relatively young, a lot of things to improve | 09:42 |
| dtantsur | anyway, it's MUCH better than writing a client from scratch | 09:42 |
| *** shuquan has joined #openstack-ironic | 09:42 | |
| sambetts | dtantsur: waay better :-p, btw I think looking at this I've just noticed something about the rules API, the GET /v1/rules/<uuid> returns { uuid: balh, desc: blash } instead of { rule: { uuid: blah, desc: blah } } which is the way the other APIs do it http://developer.openstack.org/api-ref-networking-v2.html#showNetwork | 09:46 |
| dtantsur | sambetts, well, nesting a dict is probably a matter of taste... we didn't do it with introspection status either. anyway, it's a bit too late to change :) | 09:47 |
| sambetts | dtantsur: Yeah, I just didn't know if it was an OpenStack standard or not, looking at the other APIs, neutron, cinder and nova for their GETs nest, but glance and celiometer don't, so I guess its not a problem | 09:50 |
| *** e0ne has joined #openstack-ironic | 09:51 | |
| sambetts | dtantsur: I've always found the nesting odd anyway | 09:53 |
| dtantsur | yeah | 09:55 |
| *** chenglch has quit IRC | 09:55 | |
| lucasagomes | dtantsur, not that young actually, it's been around since 2012 | 09:56 |
| lucasagomes | (or 2013... not sure) | 09:56 |
| *** naohirot has quit IRC | 09:58 | |
| dtantsur | sambetts, ifarkas, WDYT about yuikotakada's suggestion on https://review.openstack.org/#/c/224075/ ? I'm a bit undecided on this matter... | 09:59 |
| dtantsur | just like with rules, the table representation will look ugly | 09:59 |
| *** shuquan has quit IRC | 09:59 | |
| dtantsur | but we can always use '-f json' to get just JSON | 09:59 |
| *** dims_ has joined #openstack-ironic | 09:59 | |
| dtantsur | here's how it looks for me: http://paste.openstack.org/show/466058/ | 10:00 |
| dtantsur | (actually it will be heavily wrapped) | 10:00 |
| *** e0ne has quit IRC | 10:00 | |
| *** shuquan has joined #openstack-ironic | 10:01 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add IPMINative vendor methods to *IPMINative drivers https://review.openstack.org/224099 | 10:01 |
| *** shuquan has quit IRC | 10:02 | |
| dtantsur | yuikotakada, hmm, I took one more look, seems like '-f json' does not give us an equivalent JSON. maybe split into "data" and "raw data" commands? | 10:02 |
| ifarkas | dtantsur, I vote for landing the patch as it is but I am not opposed having a table view | 10:02 |
| sambetts | dtantsur: I think it would likely be worth making it a ShowOne like the rule one, it can't hurt to enable the flexablity in the command | 10:02 |
| dtantsur | sambetts, that we'll need the 2nd command to output a raw JSON... | 10:03 |
| yuikotakada | dtantsur, wait.. the output you pasted is different from mine... | 10:03 |
| dtantsur | yuikotakada, it depends on how you implement it :) | 10:03 |
| dtantsur | sambetts, '-f json' does not give the original JSON back, it's more like [{"Field": "..", "value": "..."}] | 10:04 |
| dtantsur | here's how it looks with '-f json': http://paste.openstack.org/show/466060/ | 10:05 |
| sambetts | dtantsur: I guess thats because you have to transpose it out of the dict before you return it :( | 10:06 |
| dtantsur | sambetts, I'm using dict2columns, so it should be fine | 10:07 |
| dtantsur | sambetts, here's what I'm testing: http://paste.openstack.org/show/466062/ | 10:07 |
| yuikotakada | dtantsur, http://paste.openstack.org/show/466058/ looks fine enough. mine is not easy to read. http://paste.openstack.org/show/466061/ | 10:08 |
| sambetts | dtantsur: thats not how the docs say it should output | 10:09 |
| sambetts | http://docs.openstack.org/developer/cliff/show_commands.html#json | 10:09 |
| dtantsur | weird... | 10:09 |
| *** e0ne has joined #openstack-ironic | 10:11 | |
| *** ndipanov has quit IRC | 10:11 | |
| dtantsur | sambetts, tested 'image show', it also outputs with "Field" and "Value" | 10:11 |
| sambetts | dtantsur: How odd! I wonder if they changed something in their code! Thats really annoying | 10:12 |
| dtantsur | yep | 10:13 |
| dtantsur | sambetts, yuikotakada, so I'd like us to have something returning data in machine readable format. I'm ok with leaving it as it is, or with splitting into 2 commands: human readable and machine readable | 10:14 |
| -openstackstatus- NOTICE: Gate back to normal, thanks to the backlisting of the problematic version | 10:15 | |
| sambetts | dtantsur: if you do -f shell does it output it properly? | 10:15 |
| dtantsur | sambetts, yep | 10:17 |
| sambetts | dtantsur: Arg! So annoying... Maybe in the future we'll have to create our own formatter as it describes at the bottom of that cliff doc, so we could deal with the wrapping | 10:17 |
| dtantsur | so, I'll change this command to "raw data", and we'll revisit it later. WDYT? | 10:18 |
| sambetts | dtantsur: Ok, we can deprecated it later if we managed to make a successful combined command | 10:19 |
| dtantsur | yeah | 10:19 |
| *** ndipanov has joined #openstack-ironic | 10:20 | |
| *** romainh has quit IRC | 10:21 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Support for getting introspection data https://review.openstack.org/224075 | 10:22 |
| dtantsur | sambetts, yuikotakada ^^ | 10:22 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Do not use matchers for trivial assertions https://review.openstack.org/224560 | 10:25 |
| lucasagomes | vdrok, racing comments hehe | 10:36 |
| lucasagomes | vdrok, I just added a suggestion the commit message on the patch-set #1, but ps #2 is grand as well | 10:36 |
| lucasagomes | +2'd | 10:36 |
| vdrok | lucasagomes, :) thanks | 10:36 |
| vdrok | also morning everyone | 10:37 |
| openstackgerrit | Merged openstack/ironic: Base IronicObject on VersionedObject https://review.openstack.org/213602 | 10:38 |
| openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM domain name command for Lenovo ThinkServers https://review.openstack.org/221911 | 10:38 |
| *** chenke has quit IRC | 10:39 | |
| *** zhenguo has quit IRC | 10:40 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Fix upgrades by implementing indirection_api https://review.openstack.org/224079 | 10:42 |
| openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM domain name command for Lenovo ThinkServers https://review.openstack.org/221911 | 10:44 |
| openstackgerrit | Merged openstack/ironic-inspector: Replace glance and keystone commands with OSC equivalents https://review.openstack.org/224501 | 10:44 |
| *** yuikotakada has quit IRC | 10:50 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Add a simple 'introspection rule show' command https://review.openstack.org/224471 | 10:54 |
| dtantsur | only one patch left before the client release \o/ | 10:55 |
| *** dtantsur is now known as dtantsur|lunch | 10:55 | |
| sambetts | \o/ | 10:57 |
| *** lsmola has quit IRC | 11:01 | |
| *** timbyr_ has joined #openstack-ironic | 11:02 | |
| *** lsmola has joined #openstack-ironic | 11:02 | |
| *** timbyr_ has quit IRC | 11:03 | |
| *** romcheg has quit IRC | 11:07 | |
| *** MattMan has joined #openstack-ironic | 11:11 | |
| *** romainh has joined #openstack-ironic | 11:16 | |
| *** e0ne has quit IRC | 11:23 | |
| *** sdake_ has joined #openstack-ironic | 11:43 | |
| *** sdake has quit IRC | 11:47 | |
| *** romcheg has joined #openstack-ironic | 11:53 | |
| *** trown|outttypeww is now known as trown | 11:55 | |
| *** thrash|g0ne is now known as thrash | 12:01 | |
| openstackgerrit | Merged openstack/ironic: Use oslo.versionedobjects remotable decorators https://review.openstack.org/217000 | 12:04 |
| *** baoli has joined #openstack-ironic | 12:04 | |
| *** e0ne has joined #openstack-ironic | 12:04 | |
| *** amotoki has quit IRC | 12:12 | |
| *** sdake_ is now known as sdake | 12:13 | |
| *** romcheg has quit IRC | 12:23 | |
| sambetts | Is someone around that can explain some ipmitool bridging stuff? | 12:29 |
| TheJulia | sambetts: I have a basic understanding of it | 12:30 |
| TheJulia | Also, good morning | 12:30 |
| *** romcheg has joined #openstack-ironic | 12:31 | |
| *** achanda has joined #openstack-ironic | 12:35 | |
| sambetts | Morning TheJulia, so I'm looking at the ironic ipmitool bridge spec, and I'm interested to know what format the target address is/local addresses are, from the blueprint they don't look like IPs | 12:36 |
| TheJulia | sambetts: so there is the target address of the node, which conceptually you could think about it as a single chassis controller for a number of blades, from there multiple nodes may have the same address, but they would have different bridge addresses | 12:39 |
| TheJulia | Example being, On a moonshot, to manage the nodes, it is the same IP, but I have to use dual bridging, and define node specific parameters for each node's specific control. | 12:40 |
| TheJulia | does that make sense/answer your question? | 12:40 |
| sambetts | TheJulia, so if you were to run "ipmitool lan print" on one of those nodes does it actually return anything? I'm working on adding bridging support to inspector, and thats the command inspector uses to pull the ipmi address from the node, so I'm trying to work out what it would return | 12:43 |
| *** shoutm_ has quit IRC | 12:46 | |
| TheJulia | sambetts: I don't think anything gets printed. I think nodes are slightly unaware. I think we ended up scanning to figure out each cartrige's bridging info | 12:47 |
| *** cdearborn has joined #openstack-ironic | 12:48 | |
| *** mdbooth has quit IRC | 12:48 | |
| *** dtantsur|lunch is now known as dtantsur | 12:48 | |
| BobBall | What should be updating a node's power state? Is there a period task somewhere which will query the power state and update the appropriate nodes? Adding a new SSH PM driver and the command I run manually gives the correct list of running VMs, but ironic's view of the nodes is not updated | 12:49 |
| *** mdbooth has joined #openstack-ironic | 12:50 | |
| *** amotoki has joined #openstack-ironic | 12:50 | |
| sambetts | TheJulia: Ah ok, back the white board I go :-P | 12:50 |
| *** rloo has joined #openstack-ironic | 12:53 | |
| dtantsur | TheJulia, morning | 12:56 |
| lucasagomes | BobBall, yes there's a periodic task that sync the power states | 12:56 |
| TheJulia | good morning dtantsur lucasagomes | 12:56 |
| BobBall | lucasagomes: Run by which process? Trying to track it down so I can debug why the power states aren't being synced | 12:57 |
| lucasagomes | BobBall, and there's another configuration to make the power state to be forced upon the nodes (vm's in ur case) or just passive. For e.g in active move if the DB says the node is "on" but it's actually "off" it will power on the node. Passive mode it will just update the db saying the node is "off" | 12:57 |
| lucasagomes | BobBall, ironic-conductor | 12:57 |
| lucasagomes | BobBall, https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L2229 | 12:58 |
| rloo | hi dtantsur, lucasagomes, BobBall, TheJulia | 12:58 |
| dtantsur | rloo, o/ | 12:58 |
| lucasagomes | BobBall, actually here's the periodic task https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1103 | 12:58 |
| lucasagomes | TheJulia, rloo good morning! | 12:58 |
| BobBall | Howdy | 13:00 |
| BobBall | Thanks lucasagomes - I'll chuck some debugging in there. | 13:00 |
| lucasagomes | BobBall, cool | 13:01 |
| lucasagomes | BobBall, you can see there that some states the nodes are not sync'd (deploywait, cleanwait, enroll) | 13:02 |
| lucasagomes | and if it's in maintenance it doesn't sync either | 13:02 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Refactor IRMCVirtualMediaIscsiDeploy by applying new BootInterface https://review.openstack.org/221371 | 13:02 |
| lucasagomes | s/and/or | 13:02 |
| BobBall | they are all available / not in maintenance | 13:02 |
| lucasagomes | cool yeah should be sync'ing then | 13:02 |
| * lucasagomes suspect it may be something with the get_power_state() which is returning only one value (not failing) | 13:03 | |
| BobBall | My checks show it returning multiple values from the plain SSH command (separated by newline) ... but investigating :) | 13:08 |
| zigo | I'm getting my jenkins build process to die, when it builds ironicclient. | 13:11 |
| zigo | How can I investigate the issue? | 13:11 |
| *** jefabo has joined #openstack-ironic | 13:11 | |
| zigo | It does when doing the CommonParamsForListTest series of test. | 13:12 |
| rloo | zigo: do you have anything more specific? does it dump out where it died? | 13:16 |
| zigo | rloo: That's the issue: it does nothing that gives me a clue. | 13:17 |
| rloo | zigo: :-( | 13:17 |
| zigo | rloo: https://liberty-jessie.pkgs.mirantis.com/job/python-ironicclient/4/console | 13:17 |
| rloo | zigo: does the 'wait: No child processes...' mean that one of the unit tests failed? or is that more of a problem with the build machine | 13:20 |
| *** zhenguo has joined #openstack-ironic | 13:20 | |
| zigo | rloo: I think it's just jenkins saying it's child process died. | 13:21 |
| zigo | FYI, building in plain sid, not using Jenkins, works. | 13:21 |
| zigo | Also, I have the same issue with Nova. | 13:21 |
| zigo | I increased RAM from 4 to 6 GB, and it didn't help | 13:21 |
| rloo | zigo: oh. then I blame it on Nova :-) | 13:21 |
| *** romcheg has quit IRC | 13:22 | |
| rloo | zigo: I don't know much about jenkins and it doesn't look like the problem is due to the unit tests themselves. Else I would think it would have outputted the unit test it was working on when it barfed. | 13:23 |
| rloo | zigo: and everything else in the openstack world works fine except nova and this client? | 13:24 |
| *** romcheg has joined #openstack-ironic | 13:24 | |
| zigo | rloo: Correct. | 13:24 |
| rloo | zigo: hmm, not always good to be special... | 13:24 |
| zigo | rloo: Another thing is that when it is in this state, Python 3.4 takes some CPU. | 13:25 |
| zigo | But I have no clue what this process is doing ... | 13:25 |
| *** romcheg has quit IRC | 13:26 | |
| rloo | zigo: I'm not a py 3.4 person. | 13:26 |
| rloo | zigo: does 'py3versions: no X-Python3-Version in control file, using supported versions' mean anything? | 13:26 |
| zigo | rloo: This is only a warning and it can be ignored. | 13:27 |
| lucasagomes | zigo, "Build killed with signal TERM after 150 minutes of inactivity" is that time configurable? | 13:27 |
| zigo | Basically, it's saying: oh, you haven't declard anything in your package, so I'll just take whatever Python version is supported in this Debian system. | 13:28 |
| *** romcheg has joined #openstack-ironic | 13:28 | |
| zigo | lucasagomes: Probably. That's just a Jenkins timeout, IMO. | 13:28 |
| rloo | zigo: and do you know that the python version is "good"? | 13:28 |
| lucasagomes | zigo, perhaps incress it a bit see how far it gets. Then investigate why it's saying it's inactive | 13:28 |
| zigo | rloo: 2.7 and 3.4 are fine. | 13:28 |
| zigo | lucasagomes: 150 minutes is *A LOT* of time. | 13:28 |
| lucasagomes | indeed | 13:29 |
| zigo | A way more than it takes to build ironicclient. | 13:29 |
| rloo | lucasagomes: I know we test with py3.4, but do we say to the world that ironic/openstack works with py3.4? | 13:29 |
| lucasagomes | zigo, it would be good to have some timestamps for the commands there | 13:29 |
| rloo | lucasagomes: I don't think it should take more than 150 minutes! | 13:29 |
| lucasagomes | to see where it's hangin | 13:29 |
| rloo | zigo: if it is out of child processes, is there a way to increase that? or find out how many child processes it is using? | 13:29 |
| lucasagomes | rloo, it totally shouldn't, but for debug purprose see what happens if u set the time a bit longer, if it stops at the same place or not | 13:30 |
| BobBall | ARGHHH!!!!!! https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ssh.py#L371 Why on earth does sshy.py assume the nodename is __always__ quoted? For XenServer it isn't :) | 13:30 |
| * BobBall bangs his head on the padded walls | 13:30 | |
| *** openstackgerrit has quit IRC | 13:31 | |
| rloo | zigo: is it easy to change jenkins so it runs py3.4 first, before py2.7? | 13:31 |
| zigo | rloo: Yes, I just change something in my debian/rules | 13:31 |
| zigo | (for this package) | 13:32 |
| *** openstackgerrit has joined #openstack-ironic | 13:32 | |
| lucasagomes | zigo, build 5 has a diff error https://liberty-jessie.pkgs.mirantis.com/job/python-ironicclient/5/console | 13:32 |
| *** shoutm has joined #openstack-ironic | 13:32 | |
| lucasagomes | ironicclient.tests.unit.test_utils.CommonParamsForListTest.test_sort_dir_invalid ... FAIL | 13:33 |
| lucasagomes | for py3.4 | 13:33 |
| zigo | That's because I killed it. | 13:33 |
| zigo | lucasagomes: Not relevant IMO. | 13:33 |
| rloo | lucasagomes, zigo: that's basically around where the first jenkins died | 13:33 |
| lucasagomes | oh right ok if you killed yah | 13:33 |
| zigo | Trying to build 3.4 tests first, see how it goes. | 13:34 |
| zigo | If it fails there, then I'll just skip running 3.4 tests. | 13:34 |
| rloo | zigo: is it possible it is creating child processes for all those unit tests, but not freeing them? | 13:34 |
| zigo | Ok, it is stuck at the same place, still. | 13:36 |
| zigo | rloo: What do you mean "not freeing them"? | 13:36 |
| rloo | zigo: am wondering if it is spawning processes/threads for the unit tests, but holding on to those threads even after the test is done. | 13:37 |
| rloo | zigo: if you do some sort of 'ps' command while it is waiting, you should be able to see how many processes are being used. i think. | 13:38 |
| zigo | Well, it's just standard "testr run" stuff... | 13:38 |
| BobBall | lucasagomes: Confirmed - the bug was that ironic requires for running-vms to give a list of quoted node names instead of just the node names... Seems that vmware/virsh adds the quotes in the ssh commands but vbox quotes by default | 13:38 |
| zigo | I only see the discover one. | 13:38 |
| zigo | Not even the testr run. | 13:38 |
| zigo | So my guess is that it really dies. | 13:38 |
| dtantsur | trown, sambetts, ifarkas, WDYT about releasing the client without introspection data change? it fails in zuul for the 2nd time for mirror reasons... | 13:39 |
| lucasagomes | BobBall, I see | 13:39 |
| rloo | zigo: hmm. that's about as much as i know. wonder if the infra folks have more experience with jenkins/testr/blah blah. | 13:39 |
| lucasagomes | BobBall, yeah the ssh driver can be pretty dodge | 13:39 |
| zigo | Maybe. | 13:39 |
| rloo | zigo: unless it has to do with that series of tests. i guess you could delete that test file and see what happens. | 13:39 |
| zigo | rloo: Thanks, I'll try to ask. | 13:39 |
| trown | dtantsur: fine by me... we can get that same thing with swiftclient... so not really a huge deal | 13:39 |
| * BobBall thinks some comments explaining the required output from each command would be __very__ helpful. | 13:39 | |
| ifarkas | dtantsur, you don't want to wait until tomorrow? | 13:39 |
| BobBall | When we add XS support we'll add the comments too. Just to save the next poor unfortunate soul | 13:40 |
| dtantsur | ifarkas, we're past all possible deadlines already tbh... | 13:40 |
| *** achanda has quit IRC | 13:40 | |
| ifarkas | dtantsur, ok then | 13:40 |
| zigo | I can confirm that if not running 3.4 tests, and it finishes building ... | 13:40 |
| lucasagomes | BobBall, hah thanks yes that helps | 13:40 |
| trown | it is also a really small patch for backport later | 13:40 |
| rloo | BobBall: that's a good idea wrt comments. Do you have time to submit a patch for that? | 13:41 |
| lucasagomes | BobBall, are you planning to submit the xenserver virt type support for ssh upstream? | 13:41 |
| zigo | Oh, btw, I forgot to tell a good news: ironic-inspector has been FTP master approved, and it will reach your closest Debian mirror soon (FYI: upload was done in Debian Experimental, and will stay there until the final release of Liberty). | 13:41 |
| trown | zigo: nice one | 13:41 |
| dtantsur | zigo, \o/ | 13:41 |
| BobBall | Yes rloo. And eventually, yes lucasagomes. It makes a lot of sense to me to add that support - but I guess there may be additional requirements (e.g. CI?) that might delay adding it... | 13:41 |
| *** VikasC has quit IRC | 13:41 | |
| sambetts | ifarkas, dtantsur: I think we can do it, without the table view its only a shortcut to the curl anyways :-P | 13:41 |
| dtantsur | hehe, right | 13:42 |
| lucasagomes | BobBall, not really, cause we don't have CI for anything else than virsh right now | 13:42 |
| lucasagomes | so it's def not a requirement | 13:42 |
| BobBall | Ah - well, in that case it might just need a lot more testing + validating before I'm happy with it ;) | 13:42 |
| lucasagomes | BobBall, ++, thanks! Yeah looking fwd to it | 13:42 |
| BobBall | Need a spec for XS support in ssh driver, or wishlist bug? Different projects 'n' all... | 13:43 |
| lucasagomes | BobBall, IMHO, a bug is good enough. This is just yet another hypervisor support for the ssh driver | 13:44 |
| * lucasagomes think parallels only had a bug as well | 13:44 | |
| * lucasagomes checks | 13:44 | |
| rloo | BobBall: just looking at that code in ssh.py; we/you could change it so that it looks for non-quoted names too. | 13:44 |
| lucasagomes | BobBall, yeah https://bugs.launchpad.net/ironic/+bug/1358401 | 13:45 |
| openstack | Launchpad bug 1358401 in Ironic "Add Parallels virtualisation to ssh driver" [Wishlist,Fix released] - Assigned to Davide Guerri (davide-guerri) | 13:45 |
| lucasagomes | it's cool to have a wishlist bug only | 13:45 |
| * lucasagomes tho it's an old bug, but hey, it's grand! | 13:45 | |
| BobBall | rloo: TBH that was my fix as well (node_name == node) or (quote_node_name in node) | 13:49 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Ignore IPMI Address for IPMI Bridged nodes https://review.openstack.org/224127 | 13:49 |
| rloo | BobBall: thx. it is OK to just submit that fix w/o your hypervisor support, since that is a bug. If you want. | 13:50 |
| rloo | BobBall: or you can do it all in one shot or whatever works for you. | 13:50 |
| BobBall | Actually - will do that + comments now. | 13:50 |
| sambetts | dtantsur: I've reworked that one ^, unfortunatly until the migrations one goes in and we can update the DB then we can't store multiple bmc_addresses with the same value | 13:51 |
| BobBall | rloo / lucasagomes: BTW... is the vmware ssh driver out of date? I'm not sure it can work because it does not have get/set boot device? | 13:53 |
| sambetts | dtantsur: I had a solution based on my original one, with the multiple return detection, which would have working, until I realised we couldn't even store it in the DB without changing something | 13:53 |
| dtantsur | sambetts, well, IPMI address is useless there anyway | 13:53 |
| sambetts | dtantsur: Yup | 13:53 |
| rloo | BobBall: I don't know but that is possible. Sorry, head down reviewing code now. | 13:53 |
| *** shoutm has quit IRC | 13:54 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Ignore IPMI Address for IPMI Bridged nodes https://review.openstack.org/224127 | 13:54 |
| BobBall | np! | 13:54 |
| lucasagomes | BobBall, no idea! | 13:55 |
| dtantsur | awesome, now python-ironic-inspector-client release just doesn't work >_< | 13:55 |
| * dtantsur has a "lucky" day as usual | 13:56 | |
| trown | lucky week | 13:56 |
| dtantsur | ok, it's queued now. complaining always works :) | 13:57 |
| *** romainh has quit IRC | 13:57 | |
| sambetts | dtantsur: haha | 13:57 |
| *** absubram has joined #openstack-ironic | 13:57 | |
| *** romainh has joined #openstack-ironic | 13:58 | |
| *** mtanino has joined #openstack-ironic | 13:58 | |
| lucasagomes | dtantsur, :-( | 13:59 |
| *** MattMan has left #openstack-ironic | 14:01 | |
| *** e0ne has quit IRC | 14:01 | |
| *** e0ne has joined #openstack-ironic | 14:06 | |
| *** shoutm has joined #openstack-ironic | 14:08 | |
| *** mjturek1 has joined #openstack-ironic | 14:09 | |
| dtantsur | ok, we released python-ironic-inspector-client 1.2.0 :) lets move on, ironic-inspector-2.2.0 is ~ in a week :) | 14:12 |
| rloo | dtantsur is a stern task master :D | 14:13 |
| trown | dtantsur: aye aye captain | 14:14 |
| trown | nice one on the client release | 14:14 |
| dtantsur | :) | 14:14 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Support for getting introspection data https://review.openstack.org/224075 | 14:20 |
| jroll | morning everyone | 14:20 |
| jroll | dtantsur: \o/ | 14:21 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Refactor IRMCVirtualMediaAgentDeploy by applying new BootInterface https://review.openstack.org/221577 | 14:21 |
| dtantsur | jroll, morning! | 14:21 |
| lucasagomes | jroll, yo | 14:21 |
| *** achanda has joined #openstack-ironic | 14:22 | |
| jroll | random informal poll: what do people think about spec-less blueprints for small features (like bob's xenserver support) | 14:23 |
| dtantsur | jroll, I'm all for fall back approach: if patch is deemed to complex we ask a person to write a spec | 14:23 |
| *** amotoki has quit IRC | 14:23 | |
| jroll | I should give context, I guess | 14:24 |
| jroll | I like having BPs for tracking features in releases, just to differentiate features from bugs | 14:24 |
| jroll | the BP can just be a title and quick blurb like a bug would be, no need to design the feature there | 14:24 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Check image size before provisioning for agent driver https://review.openstack.org/215254 | 14:24 |
| jroll | but being able to tag features to a release as a feature rather than a bug seems useful | 14:25 |
| jlvillal | Sounds reasonable to me, from a gut feeling, I haven't thought about it a lot, standpoint :) | 14:25 |
| *** achanda has quit IRC | 14:25 | |
| jroll | I'd much rather use another system to track it but hey | 14:25 |
| dtantsur | jroll, yeah, looks good at first glance | 14:25 |
| jroll | \o/ | 14:27 |
| jroll | I'd love to get launchpad cleaned up real soon, definitely before the summit | 14:27 |
| sambetts | that sounds pretty good to me, I've written specs when I end up putting the word None in more places than actually explanation | 14:28 |
| dtantsur | oh yeah, that happens | 14:29 |
| dtantsur | jroll, could you have one more look at https://review.openstack.org/#/c/223054 if you have a minute? | 14:30 |
| jroll | yep | 14:30 |
| *** mtanino has quit IRC | 14:31 | |
| jroll | dtantsur: +A | 14:31 |
| dtantsur | thanks! | 14:32 |
| jroll | np | 14:32 |
| * jroll wonders how zapping is doing | 14:32 | |
| jroll | lucasagomes: are you planning to deal with the tripleo CI stuff? | 14:37 |
| lucasagomes | jroll, hi there, yes I will | 14:37 |
| jroll | awesome, thanks | 14:37 |
| lucasagomes | jroll, I got a bit confused because it's passing to a lot of changes right now | 14:37 |
| lucasagomes | it was using devtest before now it uses instack | 14:37 |
| jroll | hm | 14:38 |
| lucasagomes | so the patches I put up before doesn't work anymore | 14:38 |
| jroll | weird that your patch could even possibly make it fail | 14:38 |
| lucasagomes | because it just doesn't use those scripts to generate images | 14:38 |
| jroll | oh | 14:38 |
| lucasagomes | yeah | 14:38 |
| lucasagomes | heh | 14:38 |
| jroll | that's fun | 14:38 |
| lucasagomes | jroll, the tripleo-ci was basically replaced | 14:38 |
| dtantsur | jroll, we're upstreaming a HUGE bunch of tripleo downstream stuff | 14:39 |
| dtantsur | it will be much saner once we finish | 14:39 |
| lucasagomes | from devtest (and tripleo-incubator etc...) to instack-undercloud and python-tripleoclient | 14:39 |
| dtantsur | but now it's a temporary mess :) | 14:39 |
| lucasagomes | jroll, yeah that's a good thing actually | 14:39 |
| lucasagomes | jroll, but yeah I will fix it up for IPA | 14:39 |
| lucasagomes | waiting for it to calm down a little | 14:39 |
| lucasagomes | jroll, anyway, the patch deprecating the deploy-ironic element is now +a | 14:39 |
| rloo | jroll: am fine with BP/spec-less, but I think we need to (try to) make it clear what is a BP-spec, BP+spec, bug. | 14:40 |
| rloo | jroll: and bug-less :) | 14:40 |
| lucasagomes | jroll, hmm... you mean bps for small features that doesn't need a spec? | 14:41 |
| rloo | lucasagomes: I think jroll is saying a feature needs a BP | 14:41 |
| rloo | lucasagomes: whether the BP needs a spec, depends on the feature. | 14:42 |
| lucasagomes | rloo, right, but what's the diff of a feature that needs a bp and the one that needs a bp and spec? | 14:42 |
| rloo | lucasagomes: that's what i mentioned; we need to try to clarify that or we'll have this question again and again | 14:42 |
| lucasagomes | rloo, ++ | 14:42 |
| lucasagomes | jroll, rloo but wouldn't a wishlist bug be better for that? | 14:42 |
| rloo | lucasagomes: I suspect a BP without spec is one that doesn't require any architect/thinking about how it fits in. | 14:42 |
| jroll | lucasagomes: well, for release tracking... we list blueprints as features and bugs as bugs | 14:43 |
| lucasagomes | rloo, yeah, perhaps something like "if the feature does impact in multiple areas of the project (api, database model, rpc, etc...) it needs a spec" | 14:43 |
| jroll | and yes, I also want to document this | 14:44 |
| rloo | lucasagomes: right. so i think (most) cores might know; if i have to ask questions about how someone is going to implement a feature, it needs a spec. | 14:44 |
| lucasagomes | jroll, fair enuff | 14:44 |
| rloo | lucasagomes: and we all know, i ask a lot of questions :D | 14:44 |
| jroll | so here's what I want to do, whether I'm PTL or deva is PTL: | 14:44 |
| jroll | - clean up launchpad, start actually tracking things well there | 14:44 |
| lucasagomes | rloo, lol yeah, it's good tho | 14:44 |
| rloo | jroll: there's still time for someone else to run for ptl? :) | 14:44 |
| lucasagomes | rloo, no it's closed | 14:45 |
| jroll | - for a BP to be approved, we decide if it needs a spec or not | 14:45 |
| rloo | lucasagomes: oh? good, now the fun starts! | 14:45 |
| jroll | - we also document why we would or would not want a spec | 14:45 |
| rloo | jroll: maybe we can discuss this after we freeze liberty. | 14:45 |
| jroll | yeah | 14:45 |
| jroll | just wanted to start the conversation | 14:46 |
| rloo | jroll: cuz i really want to focus on zapping and am still trying to review. | 14:46 |
| jroll | and hear folks' thoughts | 14:46 |
| jroll | yep | 14:46 |
| openstackgerrit | Bob Ball proposed openstack/ironic: Make requirements for ssh power manager clear https://review.openstack.org/224770 | 14:46 |
| jroll | hence "informal poll" | 14:46 |
| lucasagomes | rloo, ++ | 14:46 |
| rloo | jroll: yeah, good conversation, i want to participate but... also. maybe start it on mail list? | 14:46 |
| lucasagomes | jroll, yeah those ideas are fine, I think that's why blueprints exists in launchpad | 14:46 |
| jroll | rloo: let me rephrase, I wanted to hear what dmitry, ruby, and lucas had to say about this | 14:46 |
| lucasagomes | it just that launchpad seems a bit eww to track things | 14:46 |
| jroll | rloo: and take it from there | 14:46 |
| rloo | jroll: my concern is to avoid too many steps for someone trying to get a feature in. if i have to submit BP, then wait for it to be approved, then submit spec, then wait for it to be approved... | 14:47 |
| jroll | lucasagomes: sure, but a text file or google doc is just as eww :) I for one welcome our phabricator overlords | 14:47 |
| lucasagomes | jroll, yeah ML and if needed we take it to the summit | 14:47 |
| lucasagomes | jroll, fair enuff | 14:47 |
| lucasagomes | I'm ok with processes, as long as we have a well defined flow | 14:48 |
| lucasagomes | that is straight fwd | 14:48 |
| jroll | yep | 14:48 |
| rloo | jroll: I *think* we had the idea of short specs for that purpose; to get a quick 'does it need more info or not'. and I don't think anyone has gone that route. | 14:48 |
| jroll | I don't see it as different than "file a wishlist bug", it's still a quick form on launchpad | 14:48 |
| rloo | lucasagomes: the process could be well-defined, but if takes weeks to get a bp approved and weeks to get a spec approved, there is something wrong. | 14:48 |
| lucasagomes | rloo, the short spec was more in the fields of ideas. Sometihng controversial that may not be aligned with the project's scope | 14:48 |
| lucasagomes | rloo, and the idea was to either say: "yeah go ahead and finish up the spec" or "no we don't think the idea is aligned within the project scope" | 14:49 |
| *** penick has joined #openstack-ironic | 14:49 | |
| lucasagomes | rloo, ++ | 14:49 |
| jroll | rloo: my goal would be days or less to get feedback on the blueprint :) | 14:49 |
| rloo | jroll: the wishlist bug, unless we triage and kick it out, means that the person can start coding. and/or we ask person to submit spec for the wishlist bug. | 14:49 |
| rloo | jroll obviously has his release hat on. which is good I guess :) | 14:50 |
| jroll | hehe | 14:50 |
| lucasagomes | :D | 14:51 |
| *** achanda has joined #openstack-ironic | 14:52 | |
| lucasagomes | as a side, one thing that I usually do when writing a spec is to code in parallel, like POC. Because sometimes I just don't know how to solve something unless I'm trying it out | 14:53 |
| *** dims_ has quit IRC | 14:53 | |
| jroll | lucasagomes: +1, I'm the same way | 14:54 |
| jlvillal | dtantsur, After reading up on decorators and looking at Ironic code yesterday, I learned something. You must know a lot about decorators :) | 14:54 |
| dtantsur | hehe | 14:55 |
| dtantsur | jlvillal, morning :) | 14:55 |
| jroll | something something mad scientist | 14:55 |
| jlvillal | dtantsur, In particular decorators with arguments and needing two layers of closure in the decorator :) | 14:55 |
| * jlvillal learned that yesterday. | 14:55 | |
| *** dims_ has joined #openstack-ironic | 14:55 | |
| jlvillal | dtantsur, Good morning | 14:55 |
| dtantsur | jlvillal, I think that's just a problem in design. @decorator should be equivalent to @decorator(). but it's not. | 14:56 |
| jlvillal | Yep. | 14:56 |
| dtantsur | though it would be a bit less flexible, I have to admit | 14:56 |
| jlvillal | And then there is decorators implemented using a class vs the typical function implementation. | 14:57 |
| dtantsur | well, class with __call__ is callable :) | 14:57 |
| dtantsur | I know even worse topic for understanding: metaclasses | 14:58 |
| jlvillal | But if use a decorator implemented using a class on a class function it makes it even weirder. With __get__ being called. My brain started hurting at a certain point :) | 14:58 |
| *** penick has quit IRC | 14:58 | |
| dtantsur | :) | 14:58 |
| * jroll sees metaclasses and runs away | 14:58 | |
| jlvillal | dtantsur, I will weight for a day when I feel like I deserve punishment to learn about metaclasses ;) | 14:59 |
| jlvillal | s/weight/wait/ | 14:59 |
| *** Marga_ has joined #openstack-ironic | 14:59 | |
| * jlvillal goes prepares to go to work... | 14:59 | |
| dtantsur | I was hacking on SQLAlchemy some years ago. A lot of fun :) | 14:59 |
| dtantsur | metaclasses, decorators, descriptors, everything mixed | 15:00 |
| *** Marga_ has quit IRC | 15:00 | |
| *** Marga_ has joined #openstack-ironic | 15:01 | |
| *** jefabo has quit IRC | 15:04 | |
| *** zhenguo has quit IRC | 15:06 | |
| lucasagomes | dtantsur, heh metaclasses are really black magic | 15:08 |
| rloo | omg, pep8 failure, No matching distribution found for hacking<0.11,>=0.10.0 | 15:08 |
| rloo | http://logs.openstack.org/99/224299/2/check/gate-ironic-pep8/92a5fda/console.html | 15:08 |
| * rloo so sad | 15:08 | |
| lucasagomes | :-( | 15:09 |
| jroll | :| | 15:09 |
| jroll | oh wait, I think I may have heard about pypi mirror issues | 15:09 |
| *** achanda has quit IRC | 15:09 | |
| jroll | yeah, that's what it looks like | 15:09 |
| lucasagomes | we are hitting record number of gate breakage the last 2 weeks (non was our fault tho) | 15:10 |
| dtantsur | one of our patches didn't get in the release because of gate... | 15:10 |
| *** ukalifon has quit IRC | 15:10 | |
| dtantsur | yeah, the same error | 15:10 |
| sambetts | dtantsur: :( | 15:11 |
| jroll | yeah, verifying with infra but I saw some things in scrollback there | 15:11 |
| *** mgoddard has quit IRC | 15:12 | |
| *** mgoddard has joined #openstack-ironic | 15:12 | |
| *** alexpilotti has quit IRC | 15:19 | |
| openstackgerrit | Merged openstack/ironic-python-agent: inspection: prepare for future deprecations https://review.openstack.org/223054 | 15:22 |
| *** alexpilotti has joined #openstack-ironic | 15:24 | |
| *** puranamr has joined #openstack-ironic | 15:25 | |
| *** krtaylor has quit IRC | 15:29 | |
| *** devlaps has joined #openstack-ironic | 15:40 | |
| jroll | JoshNang: getting close | 15:50 |
| *** puranamr has quit IRC | 15:51 | |
| *** vishwanathj has joined #openstack-ironic | 15:51 | |
| JoshNang | \o/ | 15:56 |
| * JoshNang goes to fix | 15:56 | |
| *** krtaylor has joined #openstack-ironic | 15:57 | |
| jroll | JoshNang: I'm testing in devstack now btw | 15:59 |
| *** e0ne has quit IRC | 15:59 | |
| jroll | if you want to wait a few, I may end up with more comments | 15:59 |
| *** e0ne has joined #openstack-ironic | 16:00 | |
| JoshNang | wfm. i've gotta rebuild my devstack box today | 16:00 |
| *** purp has quit IRC | 16:02 | |
| *** ijw has joined #openstack-ironic | 16:05 | |
| *** uggla_ has joined #openstack-ironic | 16:05 | |
| jroll | JoshNang: if something gets stuck in zapping due to code bug, that can't be aborted right? | 16:05 |
| *** shoutm has quit IRC | 16:05 | |
| dtantsur | see you tomorrow | 16:06 |
| *** dtantsur is now known as dtantsur|afk | 16:06 | |
| JoshNang | jroll: not yet | 16:07 |
| jroll | k | 16:07 |
| JoshNang | found a code bug? | 16:07 |
| *** mgoddard has quit IRC | 16:09 | |
| jroll | JoshNang: yeah | 16:10 |
| jroll | here, I'll add the comments | 16:10 |
| jroll | conductor line 875 | 16:10 |
| jroll | also... | 16:11 |
| jroll | I submitted '{"target": "zap", "zap_steps": [{"interface": "deploy", "step": "hi"}]}' | 16:11 |
| jroll | it booted the agent and whatnot | 16:11 |
| jroll | and did the step, in theory | 16:11 |
| jroll | don't see any indication that it didn't happen | 16:11 |
| jroll | (because obviously invalid step) | 16:11 |
| JoshNang | weird, wtf | 16:12 |
| jroll | lemme play with it | 16:13 |
| jroll | I wonder if it just skipped the first step altogether | 16:13 |
| *** mgoddard has joined #openstack-ironic | 16:13 | |
| jroll | off by one errors ftw | 16:13 |
| *** purp has joined #openstack-ironic | 16:13 | |
| *** e0ne has quit IRC | 16:15 | |
| *** romcheg has quit IRC | 16:16 | |
| * lucasagomes updated the microversion support patch for the ironic driver in nova, for those interested: https://review.openstack.org/#/c/221848/ | 16:18 | |
| JayF | thanks lucas | 16:18 |
| JayF | jroll: I talked to clif_h yesterday; I think he's interested in picking up improving the agent client as an upstream project | 16:19 |
| JayF | jroll: very exciting \o/ | 16:19 |
| clif_h | I will corroborate JayF's story | 16:19 |
| *** sdake_ has joined #openstack-ironic | 16:19 | |
| JayF | We have to let the entire onmetal team +1 the change that removes the comment from Russell with the #todo(russell) actually error check | 16:20 |
| JayF | or somehting like that | 16:20 |
| JayF | heh | 16:20 |
| *** karimb has joined #openstack-ironic | 16:20 | |
| *** ifarkas has quit IRC | 16:20 | |
| *** sdake has quit IRC | 16:23 | |
| devananda | jroll: if you want to manage LP -- I'd say "go for it" except I think it's such a bad tool for collaborative design that it would hamper the whole team and I'd rather we just not use it. | 16:23 |
| jroll | devananda: I mean, we need something, I'd also rather not use launchpad but it's already integrated with all the release management stuff | 16:24 |
| jroll | devananda: so I'd prefer to take that pain now and move to phabricator ASAP when that's a thing | 16:24 |
| jroll | I definitely don't think we should do design there, if that's actually what you meant | 16:25 |
| *** penick has joined #openstack-ironic | 16:25 | |
| jroll | design can stay to specs, just LP for tracking things | 16:25 |
| devananda | jroll: tracking acceptance status of BP's? | 16:26 |
| devananda | jroll: I thought that was done through the specs process now | 16:26 |
| jroll | devananda: tracking all the steps, and also features that don't need a spec | 16:27 |
| jroll | like "added minor feature X" shouldn't be in the list of bugfixes for a release, but it is now because we use wishlist bugs for that | 16:28 |
| *** romcheg has joined #openstack-ironic | 16:28 | |
| lucasagomes | devananda, morning | 16:29 |
| *** romainh has left #openstack-ironic | 16:31 | |
| devananda | jroll: I agree that using LP to generate release notes is helpful | 16:31 |
| devananda | jroll: so if ttracking minor features with a short BP is helpful, even though it breaks the process of needing a spec for every BP .... maybe? | 16:32 |
| BobBall | Hey all... One more question (I hope just one). Trying to boot a node I get an error in the console of the node saying that it was unable to locate the target IQN. We don't understand this one - and actually are getting confused about where the server is vs the client? | 16:32 |
| devananda | jroll: anyhow, probably a better topic for tokyo | 16:32 |
| lucasagomes | there's no way to filter the wishlist bugs severity and mark it as a feature? (Maybe I got used to the same stuff on bugzilla, but there s/whishlist/enhancement) | 16:33 |
| jroll | devananda: I mean, it can just be a title and a link to a spec, E_DO_NOT_CARE | 16:33 |
| devananda | lucasagomes: LP doesn't differentiate between importance and severity | 16:33 |
| devananda | it's one of the major issues I take with its design | 16:33 |
| lucasagomes | BobBall, you're using pxe_ssh and not agent_ssh right? | 16:33 |
| *** garthb has joined #openstack-ironic | 16:33 | |
| BobBall | Right | 16:34 |
| lucasagomes | BobBall, cause with pxe_ssh, the way the image gets written to the node's disk is: The deploy ramdisk boots up -> expose the local disk as an iscsi target and send it to the Ironic conductor via the API -> ironic conductor writes the image onto the disk over the network | 16:34 |
| BobBall | Just had to double check though! :D | 16:34 |
| lucasagomes | so see if the network stuff is configured properly | 16:34 |
| BobBall | Yes, that's what we thought - which is why we're confused that the node is saying that it can't find the initiator | 16:35 |
| *** jistr has quit IRC | 16:35 | |
| *** sdake has joined #openstack-ironic | 16:35 | |
| lucasagomes | devananda, :-( that's sad | 16:35 |
| BobBall | It connects to the host, but the error is iscsi Login negotiation failed; are there access credentials somewhere? | 16:35 |
| *** david-lyle has quit IRC | 16:36 | |
| lucasagomes | BobBall, hmm there's none. But you may have problems with the time (session expiring) | 16:36 |
| lucasagomes | a ntp server is usually useful for it | 16:36 |
| BobBall | Ah - ok. I don't have an NTP server setup. Good call. | 16:36 |
| * BobBall will investigate | 16:36 | |
| *** ifarkas has joined #openstack-ironic | 16:37 | |
| lucasagomes | BobBall, cool | 16:37 |
| *** sdake_ has quit IRC | 16:38 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/224617 | 16:39 |
| jroll | JoshNang: conductor continue_node_clean always sends clean_steps, left a comment | 16:40 |
| *** Nisha has joined #openstack-ironic | 16:41 | |
| jroll | stepping away for a few then some meetings, will continue testing this as time allows | 16:41 |
| lucasagomes | jroll, JoshNang btw I have a patch touching those parts (the abort one) | 16:41 |
| JoshNang | jroll: ah, good catch | 16:41 |
| JoshNang | lucasagomes: yeah :( but also ++ for abort. | 16:42 |
| lucasagomes | JoshNang, ++ yeah we can add it to zapping too | 16:42 |
| JoshNang | ++ should be an easy follow up | 16:42 |
| lucasagomes | indeed! | 16:43 |
| *** penick has quit IRC | 16:43 | |
| Nisha | lucasagomes, jroll JoshNang please could you have look at https://review.openstack.org/217068 | 16:46 |
| lucasagomes | Nisha, will do /me adds to his todo | 16:46 |
| lucasagomes | I'm going home now (thursday's office day) | 16:47 |
| lucasagomes | have a great night everyone! | 16:47 |
| Nisha | Thanks lucasagomes | 16:47 |
| *** lekha||away is now known as lekha | 16:47 | |
| *** david-lyle has joined #openstack-ironic | 16:47 | |
| *** mtanino has joined #openstack-ironic | 16:48 | |
| lucasagomes | see y'all | 16:48 |
| *** derekh has quit IRC | 16:50 | |
| *** ndipanov has quit IRC | 16:50 | |
| sambetts | Cya lucasagomes | 16:51 |
| *** lucasagomes has quit IRC | 16:53 | |
| *** Nisha_away has joined #openstack-ironic | 16:57 | |
| *** Nisha has quit IRC | 16:57 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add retries to ssh._get_hosts_name_for_node https://review.openstack.org/224828 | 17:02 |
| *** david-lyle has quit IRC | 17:03 | |
| *** Nisha_away has quit IRC | 17:04 | |
| *** harlowja has joined #openstack-ironic | 17:04 | |
| *** Nisha has joined #openstack-ironic | 17:04 | |
| *** trown is now known as trown|lunch | 17:10 | |
| *** ijw has quit IRC | 17:10 | |
| *** romcheg has quit IRC | 17:10 | |
| jroll | JoshNang: something up with the heartbeat thing, I'm not sure what the intended code is | 17:10 |
| JoshNang | oh? | 17:11 |
| jroll | but it looks like we only resume if not clean_step and not zap_step? | 17:11 |
| jroll | I don't see where it's like "oh hey, step is complete" | 17:11 |
| jroll | oh, wait, the else hmm | 17:12 |
| jroll | the log message there is throwing me off | 17:13 |
| JoshNang | right | 17:13 |
| jroll | line 402 | 17:13 |
| JoshNang | yeah, that gets me pretty often | 17:13 |
| JoshNang | s/msg/exc_msg/ or something | 17:13 |
| jroll | OH | 17:14 |
| jroll | heh | 17:14 |
| *** romcheg has joined #openstack-ironic | 17:14 | |
| *** uggla_ has quit IRC | 17:14 | |
| *** Marga_ has quit IRC | 17:15 | |
| *** Marga_ has joined #openstack-ironic | 17:16 | |
| *** lsmola has quit IRC | 17:17 | |
| *** karimb has quit IRC | 17:17 | |
| jroll | MUAHAHA | 17:17 |
| *** ijw has joined #openstack-ironic | 17:19 | |
| jroll | JoshNang: left a comment, but _get_completed_cleaning_command needs more zap | 17:20 |
| JoshNang | ahh gdi | 17:20 |
| JoshNang | in retrospect, we should have just allowed you to pass in a list of steps to cleaning and avoided zapping completely | 17:22 |
| jroll | mhmmm | 17:23 |
| jroll | it's not too late to do that :) | 17:23 |
| JoshNang | it honestly might be easier, but would mean changing the state machine | 17:23 |
| JoshNang | well. maybe. | 17:24 |
| jroll | I'm not sure | 17:24 |
| * jroll thinks about it | 17:24 | |
| rloo | I was thinking about that. Getting rid of 'zap' altogether and just use 'clean'. there are a few things in our state machine that don't like 'clean' states going to different target states. | 17:24 |
| jroll | I do like the idea of keeping the code/api nicer | 17:25 |
| jroll | (but is automated clean + manual clean going to be confusing to users? not sure) | 17:25 |
| rloo | or is zap (manual) and clean (automated) more confusing? | 17:25 |
| JoshNang | right, the biggest problem i see is manageable->clean/zap->manageable | 17:26 |
| rloo | I think it is less confusing to have automated/manual clean. | 17:26 |
| JoshNang | ++ personally | 17:26 |
| *** Nisha has quit IRC | 17:26 | |
| *** Nisha has joined #openstack-ironic | 17:26 | |
| jroll | JoshNang: rloo: I hate to ask this question... maybe (though sad) it would be better to bump this to M and rework it? | 17:27 |
| jroll | rather than dealing with a less-optimal API for years? | 17:27 |
| JoshNang | jroll: i think that's best for the long term success of the project | 17:27 |
| rloo | yeah. it means no RAID though. unless we fix that to work with auto clean. | 17:28 |
| JoshNang | though i'll be sad to not get to finish it. (i'm leaving RS in a week) | 17:28 |
| rloo | JoshNang: WHAT? | 17:28 |
| jroll | rloo: no inband RAID, or none at all? | 17:28 |
| rloo | jroll: no inband RAID. | 17:28 |
| JoshNang | i though raid works with clean right now? | 17:28 |
| jroll | rloo: ok. hm. | 17:28 |
| JoshNang | hrm maybe | 17:28 |
| JoshNang | rloo: yeah :( i'm leaving RS for a healthcare startup next thurs :( :( :( | 17:29 |
| rloo | It had something to do with not knowing how to handle verifying the raid config or something. i've already forgotten but I left a note in the review. let me find it. | 17:29 |
| rloo | JoshNang: Yay and Boo! That is awesome for you. | 17:29 |
| JoshNang | rloo: yeah :) it's an inspiring mission so :) :) :) there, but i'm gonna miss this community | 17:30 |
| rloo | JoshNang: the healthcare startup wants to use OpenStack, right? ;) | 17:30 |
| JoshNang | heh maybe! | 17:30 |
| *** Nisha_away has joined #openstack-ironic | 17:31 | |
| *** Nisha has quit IRC | 17:31 | |
| rloo | wrt RAID & cleaning: https://review.openstack.org/#/c/198238/ | 17:31 |
| *** ifarkas has quit IRC | 17:31 | |
| rloo | I was trying to recall the reasoning for not allowing RAID in cleaning. Dmitry said it was: 1. it won't work with cleaning, as target config is overriden in the end of cleaning; 2. it's unclear how to handle difference between zap and cleaning (i.e. whether to fail if target config is none) | 17:32 |
| jroll | JoshNang: one more thing... this isn't popping the step off the list | 17:32 |
| * jroll shredding a VM forever | 17:32 | |
| rloo | jroll, JoshNang: #1 is not a concern any more. the code was changed so that target confis isn't overridden | 17:32 |
| *** jefabo has joined #openstack-ironic | 17:32 | |
| rloo | jroll, JoshNang: so #2 is what we didn't know so we punted on it. | 17:32 |
| jroll | got it | 17:33 |
| JoshNang | makes sense | 17:33 |
| *** penick has joined #openstack-ironic | 17:34 | |
| JoshNang | so if we kill zapping, make it fail if target config is none? | 17:34 |
| rloo | JoshNang: assuming we add in the configs to set the create/deleteRAID step priority and it is non-zero. | 17:35 |
| rloo | JoshNang: no target config; 1. fail all cleaning or 2. 'skip' the create RAID step | 17:36 |
| *** Nisha_away has quit IRC | 17:37 | |
| *** Nisha_brb has joined #openstack-ironic | 17:37 | |
| *** achanda has joined #openstack-ironic | 17:39 | |
| JoshNang | rloo: hrm i'm gonna have to review this harder | 17:39 |
| rloo | JoshNang: I think it should fail cleaning. BUT maybe for almost all your nodes, you want RAID stuff done, but you have a few special cases. We only have one global config to do raid or not, so you want to use the 'no target config' to indicate whether or not to do it for that particular node. | 17:41 |
| rloo | JoshNang: on the other hand, all the other clean steps (so far) are ON or NOT, depending on the agent/hw manager on the node, right? | 17:41 |
| *** jefabo has quit IRC | 17:41 | |
| *** Nisha_brb has quit IRC | 17:42 | |
| *** Nisha_brb has joined #openstack-ironic | 17:43 | |
| JoshNang | rloo: so hrm. does setting the raid expose any scheduling hints? | 17:44 |
| JoshNang | and yeah, the steps in a hwm are much more dynamic, can be based on what hardware is actually present, etc. so it's less global | 17:46 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Add alembic migrations for the inspector database https://review.openstack.org/223665 | 17:48 |
| jlvillal | JoshNang: WTF!?!?! | 17:48 |
| jlvillal | You are leaving :( | 17:48 |
| jlvillal | So sad... | 17:48 |
| jlvillal | For us. | 17:48 |
| JoshNang | jlvillal: yeah :( but y'all get another week of me! :D | 17:49 |
| jlvillal | I hope you have great fun though! | 17:49 |
| JoshNang | i will...it's a small team...lots of room to make a big impact on our patients' lives | 17:49 |
| jlvillal | JoshNang: Sounds like a great opportunity :) | 17:49 |
| rloo | JoshNang: you ask too much! There isn't any explicit correlation between setting RAID and scheduling hints. Not that I know of anyway. | 17:49 |
| JoshNang | rloo: ahh! dang. if there was, skip would be much more palatable...you'd get no hosts found errors instead of provisioning to a box you thought had raid but doesn't | 17:50 |
| rloo | JoshNang: but presumably, depending on the raid-config per node, that node can be set up appropriately to have its properties/capabilities match a flavor. | 17:50 |
| jlvillal | Personally I find 'zap' vs 'cleaning' confusing. As someone who hasn't read that much about it. 'automated cleaning' vs 'manual cleaning' would be easier to read and more likely to be understood immediately. | 17:51 |
| jlvillal | IMHO | 17:51 |
| JoshNang | jlvillal: ++ | 17:51 |
| * rloo so said that we're all voicing this now instead of 6+ months ago. | 17:51 | |
| jlvillal | True :( | 17:52 |
| rloo | but better late than never | 17:52 |
| jroll | well, I think we've learned a lot since then | 17:52 |
| rloo | so 'zap' isn't a normal term used by operators? | 17:52 |
| JoshNang | LOL | 17:52 |
| jroll | heh | 17:52 |
| jroll | I still don't know where that came from :) | 17:53 |
| * jlvillal doesn't know as he needs operator training. Which comes in November. | 17:53 | |
| JoshNang | i think most operators prefer "pew pew" | 17:53 |
| *** harshs has joined #openstack-ironic | 17:53 | |
| * jroll wonders what operator training is | 17:53 | |
| * jlvillal thinks it is what RackSpace will teach him in November. He hopes it is :) | 17:53 | |
| JoshNang | jroll: i think that's where they throw you at hardware and say "make a cloud!" | 17:53 |
| jroll | "here, stare at these logs and see if you find anything interesting" | 17:53 |
| rloo | jroll: i think it means they're going to send jlvillal to RS | 17:53 |
| JoshNang | :D | 17:53 |
| jroll | "here, I'm going to break your network, try to keep things up" | 17:53 |
| jlvillal | rloo: ding ding ding! | 17:53 |
| rloo | oh, so I was right! ha ha. | 17:54 |
| rloo | jlvillal: can you request to be where jroll and JayF are? :) | 17:54 |
| jlvillal | rloo: Yeah, Rackspace and Intel have a joint agreement. So I will go learn from them in San Antonio. | 17:54 |
| jroll | rloo: I'm not letting him in my house... at least not for work :D | 17:54 |
| jlvillal | rloo: I did request San Francisco and my boss said no. | 17:54 |
| jroll | he can come over for beers, though | 17:54 |
| JoshNang | heh | 17:54 |
| jroll | lol | 17:54 |
| rloo | JoshNang: you sure you want to leave. Look at all the fun you'll be missing... | 17:55 |
| jroll | jlvillal: SF office isn't very openstack-y anyway :P | 17:55 |
| jlvillal | Something about me needing to learn how to ride escalators and go down slides. | 17:55 |
| JoshNang | rloo: ;) | 17:55 |
| jroll | slides \o/ | 17:55 |
| rloo | so to go back to zap vs clean. Are we good with 'clean'? (as the verb/descripto/blah blah) | 17:56 |
| jroll | I tend to prefer 'clean' for both, even if it wipes out days worth of work from josh | 17:56 |
| JoshNang | i think it's good. i'd also prefer not to break the api by changing it | 17:57 |
| *** ijw has quit IRC | 17:57 | |
| jroll | but it's hard to say it could land this cycle :( | 17:57 |
| rloo | jroll, JoshNang: I'm not convinced it wipes out a lot of work of Josh's. We still need that code refactored. | 17:57 |
| JoshNang | jroll: meh. better than a crappy api | 17:57 |
| rloo | it is really the strings etc that were/are annoying, right? | 17:57 |
| jroll | rloo: a lot of that work is s/clean/clean_or_zap/ | 17:57 |
| jroll | yeah, it's just gruntwork | 17:57 |
| jroll | but still work | 17:58 |
| JoshNang | yeah. also a lot of parallel state checks like if zapwait: etc | 17:58 |
| rloo | jroll: that's minor as long as architecturally etc it is good | 17:58 |
| jroll | right | 17:58 |
| jroll | still conflicted on landing it | 17:58 |
| rloo | jroll: oh, i don't think we should land it as is. | 17:58 |
| jroll | rloo: I mean "during this cycle" | 17:58 |
| rloo | jroll: just need to think of what we'd like, and what/if those are minor changes to the existing patch. | 17:58 |
| rloo | jroll: oh, well, if not during this cycle, should be soon after, if we can agree on something. | 17:59 |
| jroll | +2 | 17:59 |
| rloo | like, i think we might agree that the verb should be 'clean', not 'zap'? | 17:59 |
| jroll | *I* agree | 17:59 |
| jroll | idk abuot anyone else :) | 17:59 |
| JoshNang | so we'd need the handling for the optional steps args in the provision state. and we'd need to store those provided steps on the node, then some handling around set_node_clean_steps to prefer the provided steps...and that should be it | 18:00 |
| rloo | I *think* the issue for me is the state machine, and ZAP* states vs CLEAN* states. | 18:00 |
| rloo | JoshNang: that'd be node.clean_steps I think. or node.target_clean_steps? hmm, don't like the target. | 18:01 |
| jroll | yeah... state machine will need to allow manageable -> verb[clean] -> manageable, which doesn't seem that painful | 18:01 |
| rloo | JoshNang: your code already handles the optional step args in the provision state for 'zap', right? | 18:02 |
| rloo | jroll: well, it isn't painful except that we added 'target-state' or something like that to our state machine. | 18:02 |
| *** trown|lunch is now known as trown | 18:03 | |
| jroll | rloo: right, I think that helps if nothing else | 18:03 |
| rloo | jroll: I actually looked at it briefly yesterday or the day before. Guess I should have mentioned it then. I had decided it might not be worth delving into the state machine code to see about how to deal with that target state. | 18:03 |
| rloo | jroll: let me think/look into it. Need a couple hours maybe. | 18:04 |
| jroll | rloo: yeah, I guess I need to read code | 18:04 |
| JoshNang | jroll: oh. manageable -> verb[clean] -> manageable is pretty perfect. | 18:04 |
| jroll | rloo: JoshNang so at some point today we need to make the decision whether to sneak this into 4.2 (liberty) or punt on it until Mitaka opens | 18:05 |
| JoshNang | rloo: yeah, actually that's fine | 18:05 |
| *** jamielennox is now known as jamielennox|away | 18:05 | |
| jroll | I'm leaning toward the latter | 18:05 |
| JoshNang | either wfm. i can start coding it | 18:05 |
| rloo | jroll: https://github.com/openstack/ironic/blob/master/ironic/common/states.py#L230 | 18:05 |
| rloo | jroll: that's what I mean. target=AVAILABLE. | 18:06 |
| jroll | rloo: oh :( | 18:06 |
| JoshNang | ah dang | 18:06 |
| rloo | jroll: in meeting now; let me think about it a bit. if I can figure out something that is simple, I think we can get it in. Otherwise, we punt on it. | 18:06 |
| jroll | rloo: ditto, sounds good. thank you! | 18:06 |
| JoshNang | wfm...i should probably head in to the office anyway :P | 18:07 |
| JoshNang | rloo: jroll thanks for the help here! | 18:07 |
| jroll | np | 18:07 |
| *** e0ne has joined #openstack-ironic | 18:07 | |
| rloo | JoshNang: thank YOU for working on zapping :) | 18:07 |
| devananda | jroll: product WG wants to know our roadmap. forwarding you the email. I'm happy to respond or to etherpad up a response with you, if you'd like | 18:07 |
| jroll | devananda: will read in a moment here | 18:08 |
| JoshNang | :) | 18:08 |
| *** pelix has quit IRC | 18:11 | |
| jroll | devananda: either way works for me | 18:11 |
| devananda | k, i'll fire off a quick reply | 18:13 |
| *** e0ne has quit IRC | 18:17 | |
| jroll | thanks | 18:18 |
| *** sdake has quit IRC | 18:19 | |
| *** athomas has quit IRC | 18:20 | |
| *** e0ne has joined #openstack-ironic | 18:21 | |
| *** sdake has joined #openstack-ironic | 18:23 | |
| sambetts | Night all o/ | 18:25 |
| TheJulia | goodnight | 18:26 |
| *** yog_ has quit IRC | 18:27 | |
| jlvillal | sambetts: Good night! | 18:28 |
| *** sdake has quit IRC | 18:31 | |
| *** sdake has joined #openstack-ironic | 18:34 | |
| *** [1]cdearborn has joined #openstack-ironic | 18:40 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add constraint target to tox.ini https://review.openstack.org/220627 | 18:40 |
| *** penick has quit IRC | 18:41 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add constraint target to tox.ini https://review.openstack.org/220627 | 18:43 |
| *** greghaynes has quit IRC | 18:49 | |
| *** penick has joined #openstack-ironic | 18:51 | |
| *** penick has quit IRC | 18:51 | |
| *** cdearborn has quit IRC | 18:51 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Fix upgrades by implementing indirection_api https://review.openstack.org/224079 | 19:04 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic-python-agent: Adds more functional tests for commands https://review.openstack.org/224247 | 19:08 |
| *** devlaps has quit IRC | 19:08 | |
| *** yog_ has joined #openstack-ironic | 19:10 | |
| *** david-lyle has joined #openstack-ironic | 19:11 | |
| jlvillal | Does anyone else have opinions on imports inside classes inside functions? Should imports be done at the top of the file? | 19:12 |
| * jlvillal thinks the answer is yes they should be at the top of the file and not inside a class function. | 19:13 | |
| *** olaph has quit IRC | 19:18 | |
| jroll | jlvillal: agree, though one might put them in a function to avoid circular imports | 19:21 |
| jlvillal | jroll: Thanks. I don't think it is the case for the patch I was reviewing. I left comment and see if they have strong reasoning for it being there. If so, I think a comment would be good. | 19:21 |
| jroll | +1 | 19:22 |
| * jlvillal likes comments when things deviate from the expected :) | 19:22 | |
| *** david-lyle has quit IRC | 19:25 | |
| *** Marga_ has quit IRC | 19:26 | |
| *** alexpilotti has quit IRC | 19:35 | |
| *** sinval has quit IRC | 19:37 | |
| *** sinval has joined #openstack-ironic | 19:37 | |
| *** Sukhdev has joined #openstack-ironic | 19:39 | |
| *** sinval has quit IRC | 19:42 | |
| *** sinval has joined #openstack-ironic | 19:43 | |
| *** sdake has quit IRC | 19:46 | |
| *** yog_ has quit IRC | 19:46 | |
| *** yog_ has joined #openstack-ironic | 19:47 | |
| *** sdake has joined #openstack-ironic | 19:49 | |
| *** derekh has joined #openstack-ironic | 19:51 | |
| *** Sukhdev has quit IRC | 19:51 | |
| rloo | jroll, JoshNang: here's what I'm thinking/proposing wrt zap -> clean. | 19:52 |
| JoshNang | rloo: thinking more about raid...you have to opt in to raid by changing the priority or hardware manager in the agent. if we have a raid config that is basically "don't raid", then we can fail cleaning if no config is provided | 19:52 |
| rloo | I don't think it is feasible to use the existing CLEAN* states for both manual & automated cleaning, cuz of the target-state stuff | 19:53 |
| JoshNang | where do we use that? | 19:53 |
| *** Sukhdev_ has joined #openstack-ironic | 19:53 | |
| rloo | JoshNang: the node has a target_provision_state | 19:53 |
| JoshNang | sure, but is that actually used? | 19:54 |
| rloo | JoshNang: and that node.target_provision_state is set via the fsm's target_state | 19:54 |
| jroll | rloo: about to jump in a meeting for an hour, sorry :( | 19:54 |
| JoshNang | same :/ | 19:54 |
| rloo | JoshNang: in the short term, I don't want to touch target_provision_state | 19:54 |
| rloo | jroll, JoshNang:ok | 19:54 |
| rloo | jroll, JoshNang: I'll add this in comments to the patch. | 19:55 |
| rloo | jroll, JoshNang: and you can decide what to do based on that :) | 19:55 |
| JoshNang | alrighty. i have most of the patch (minus needing some updated tests) to fix what's commented so far. i'll push it up either way | 19:55 |
| rloo | JoshNang: thx! | 19:55 |
| jroll | rloo: alright, ty | 19:56 |
| *** Nisha_brb has quit IRC | 19:57 | |
| *** Nisha_brb has joined #openstack-ironic | 19:57 | |
| *** sdake has quit IRC | 19:57 | |
| *** sdake has joined #openstack-ironic | 20:00 | |
| *** Nisha_away has joined #openstack-ironic | 20:04 | |
| *** Nisha_brb has quit IRC | 20:04 | |
| *** Sukhdev_ has quit IRC | 20:06 | |
| *** ijw has joined #openstack-ironic | 20:08 | |
| *** Nisha_away has quit IRC | 20:10 | |
| *** Nisha has joined #openstack-ironic | 20:10 | |
| *** Nisha_away has joined #openstack-ironic | 20:16 | |
| *** Nisha_away has quit IRC | 20:17 | |
| *** Nisha has quit IRC | 20:17 | |
| *** lucasagomes has joined #openstack-ironic | 20:17 | |
| *** Nisha has joined #openstack-ironic | 20:18 | |
| *** Nisha_away has joined #openstack-ironic | 20:19 | |
| *** Nisha has quit IRC | 20:19 | |
| *** Nisha_brb has joined #openstack-ironic | 20:21 | |
| *** Nisha_away has quit IRC | 20:21 | |
| *** Nisha_brb has quit IRC | 20:21 | |
| *** penick has joined #openstack-ironic | 20:21 | |
| *** derekh has quit IRC | 20:21 | |
| *** Nisha_brb has joined #openstack-ironic | 20:21 | |
| *** Nisha_away has joined #openstack-ironic | 20:25 | |
| *** Nisha_brb has quit IRC | 20:25 | |
| *** Marga_ has joined #openstack-ironic | 20:26 | |
| *** Nisha_away has quit IRC | 20:26 | |
| *** Nisha_away has joined #openstack-ironic | 20:26 | |
| *** lucasagomes has quit IRC | 20:29 | |
| *** karimb has joined #openstack-ironic | 20:29 | |
| *** penick has quit IRC | 20:31 | |
| openstackgerrit | Merged stackforge/pyghmi: Add custom OEM domain name command for Lenovo ThinkServers https://review.openstack.org/221911 | 20:40 |
| *** Nisha_away has quit IRC | 20:40 | |
| *** Nisha_away has joined #openstack-ironic | 20:41 | |
| *** Sukhdev has joined #openstack-ironic | 20:42 | |
| *** lucasagomes has joined #openstack-ironic | 20:44 | |
| *** krtaylor has quit IRC | 20:45 | |
| *** baoli has quit IRC | 20:46 | |
| *** baoli has joined #openstack-ironic | 20:46 | |
| *** Nisha_away has quit IRC | 20:47 | |
| *** e0ne has quit IRC | 20:50 | |
| *** Nisha has joined #openstack-ironic | 20:56 | |
| *** penick has joined #openstack-ironic | 20:57 | |
| *** penick has quit IRC | 20:59 | |
| *** Nisha has quit IRC | 21:03 | |
| *** blakec has joined #openstack-ironic | 21:04 | |
| *** david-lyle has joined #openstack-ironic | 21:06 | |
| JoshNang | rloo: it seems like 2 is the exact same patch, minus zap_step and slightly different api code. still ugly, still full of conditionals. i'd rather kill target state and have it implicty in the state machine. | 21:06 |
| *** ijw has quit IRC | 21:06 | |
| rloo | JoshNang: Not as many conditionals as before (I mean, the logs/output dont' have to say 'manual clean' vs 'clean' I don't think?) | 21:07 |
| rloo | JoshNang: I didn't see any obvious/quick way to clean up/fix the target state stuff. | 21:07 |
| JoshNang | sure, i don't think there's any quick fix here | 21:08 |
| rloo | JoshNang: w/o hacking something and I don't want to take the chance of breaking our state machine stuff. | 21:08 |
| *** david-lyle has quit IRC | 21:09 | |
| rloo | JoshNang: my 2 was the least risk/good to go into Liberty. | 21:09 |
| *** baoli has quit IRC | 21:09 | |
| rloo | JoshNang: but will let you or jroll make the call. we can just punt it to M* | 21:09 |
| *** baoli has joined #openstack-ironic | 21:09 | |
| JayF | rloo: JoshNang: jroll: Should we put in a PR to revert/change zapping spec if it's getting pushed + perhaps substantially changing? | 21:10 |
| JoshNang | heh fwiw, there's still the followup to the zapping spec pending | 21:10 |
| JoshNang | https://review.openstack.org/#/c/209207/ | 21:10 |
| rloo | JayF: I must be tired. what is a PR? | 21:10 |
| JayF | rloo: github terminology. "Pull Request" | 21:11 |
| JayF | rloo: I should've said gerrit patchset or something more openstacky | 21:11 |
| rloo | JayF: oh. gotcha. i hate pull requests. | 21:11 |
| *** trown is now known as trown|outttypeww | 21:11 | |
| rloo | JayF: yeah, we'd need to change the spec. | 21:11 |
| *** Sukhdev has quit IRC | 21:12 | |
| JoshNang | with that, i'd say this is definitely a punt-to-M | 21:14 |
| *** baoli has quit IRC | 21:14 | |
| *** absubram has quit IRC | 21:16 | |
| *** olaph has joined #openstack-ironic | 21:16 | |
| jroll | JayF: no, we should just ignore it and pretend it didn't happen | 21:18 |
| jroll | ;) | 21:18 |
| jroll | JoshNang: rloo: I take it this means bumping in-band raid as well eh | 21:20 |
| rloo | jroll: not necessarily. just need to get it to work with cleaning. | 21:21 |
| jroll | ok. | 21:21 |
| rloo | jroll: the code is in | 21:21 |
| jroll | rloo: oh, so just some tweaks then? | 21:21 |
| rloo | jroll: I mean, the code for that is merged, to be used with zapping | 21:21 |
| rloo | jroll: yeah. I think. two configs, and whatever I mentioned earlier. | 21:21 |
| jroll | rloo: perfect | 21:22 |
| *** ijw has joined #openstack-ironic | 21:22 | |
| rloo | jroll: what to do if the target config wasn't specified, but create-raid-config is a clean step. | 21:22 |
| jroll | yep | 21:22 |
| rloo | jroll: i don't even have my head wrapped around what clean steps i'd want and what i'd like to see happen if one of them fails, let alone know what to do in the raid situation. if i had to choose, if they configure create-raid as a clean step, they should specify target-config, and if they don't, clean bombs. but that's just me :) | 21:23 |
| *** dims_ has quit IRC | 21:23 | |
| *** jamielennox|away is now known as jamielennox | 21:24 | |
| jroll | rloo: yeah, I need to think about it... I tend to think you're right. if part of the fleet shouldn't get RAID, they could get a different ramdisk without that clean step or something | 21:24 |
| rloo | jroll: I think it is all or nothing. per driver. | 21:25 |
| jroll | rloo: hm, I would think for inband that clean step would come from the agent? | 21:25 |
| jroll | I should like, read this code | 21:25 |
| rloo | jroll: yeah, from the agent. | 21:26 |
| rloo | jroll: and yeah, you should read the code :) | 21:26 |
| rloo | jroll: cuz really, I'm not 'up' on the agent. | 21:26 |
| jroll | rloo: right, so they could build a second agent without that clean step for those nodes | 21:26 |
| jroll | which is a pain but hey | 21:26 |
| rloo | jroll: ok. presumably, that's how you'd/the agent stuff/ would deal with any clean step | 21:27 |
| JoshNang | or have the hardware manager conditionally expose that step (nodes with this raid card have a raid step, nodes without a raid card don't have the step) | 21:27 |
| jroll | yah | 21:27 |
| rloo | so the feature freeze whatever is today. fixing the raid to work with clean can be done next week i think? or did you want it in this week? | 21:28 |
| jroll | I'm okay with whenever for that, I think | 21:28 |
| jroll | I guess I'd have to see the code | 21:29 |
| rloo | jroll: oh, you mean someone needs to write it? | 21:29 |
| jroll | rloo: I assume so :P | 21:29 |
| rloo | i wonder if ramesh is around. | 21:29 |
| jroll | I agree | 21:29 |
| JoshNang | he's in india, right? if so, it's 3am :P | 21:31 |
| JoshNang | (or about that timezone, iirc) | 21:31 |
| jroll | yeah, india | 21:32 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Help patch authors to remember to update version docs https://review.openstack.org/224908 | 21:32 |
| rloo | JoshNang: yeah. Just that I didn't see anything from him last night/earlier today. but he just came back from some time off so I think he's around. | 21:33 |
| JoshNang | ah gotcha | 21:33 |
| jlvillal | JoshNang: On your zapping initial patch. I'm fine if doc change was a follow up, but I am not the decider :) | 21:33 |
| JoshNang | jlvillal: ++ | 21:34 |
| rloo | JoshNang, jroll: I can probably push up a preliminary patch for it if we want something this week. I'm here for another few hours, then gone til Mon. | 21:34 |
| jlvillal | JoshNang: It did motivate me to do the patchset I just uploaded. | 21:34 |
| JoshNang | rloo: sure, i'd be more than happy to review. | 21:34 |
| rloo | JoshNang, jroll: should we mark the zap patches as work in progress or something? | 21:35 |
| jroll | jlvillal: a doc update like this one? https://review.openstack.org/#/c/223295/ | 21:35 |
| jroll | rloo: you already have a -2 on the first one :) | 21:35 |
| rloo | jroll: yeah, but jlvillal just reviewed it. | 21:35 |
| jroll | true | 21:35 |
| rloo | jroll: no sense folks reviewing if we're going to change it. | 21:35 |
| openstackgerrit | Merged openstack/ironic-specs: Extend the state machine spec to cover DEPLOYWAIT and abort https://review.openstack.org/224008 | 21:35 |
| jlvillal | jroll: Ah! Cool. I will get rid of my -1. Thanks. | 21:35 |
| JoshNang | yeah i'll wip em all | 21:36 |
| jroll | JoshNang: let us do it in case you disappear :P | 21:36 |
| JoshNang | heh k :D | 21:36 |
| jroll | rloo: I've WIPd them | 21:36 |
| rloo | jroll: good thinking. although I think wips are easy to get rid of; just do another revision. it's those killer -2s that are more of a problem! | 21:36 |
| rloo | thx jroll | 21:36 |
| jroll | np, ty for suggestion | 21:37 |
| jroll | rloo: it's also now gone from https://launchpad.net/ironic/+milestone/4.2.0 | 21:37 |
| * jroll updates the other BPs there | 21:37 | |
| rloo | jroll: thx | 21:37 |
| jroll | rloo: probably safe to call this done? https://blueprints.launchpad.net/ironic/+spec/ironic-generic-raid-interface | 21:42 |
| rloo | jroll: yesish. | 21:43 |
| jroll | rloo: after your patches? | 21:43 |
| rloo | jroll: i have an update to that spec -- i was reviewing it yesterday and there were a few inconsistencies wrt coding, so figured that'd be the best way to document/check with ramesh. | 21:43 |
| rloo | jroll: after patches. + client + any documentation. | 21:43 |
| jroll | rloo: ok, I'll leave it for now | 21:43 |
| *** jst_ has joined #openstack-ironic | 21:44 | |
| jroll | holy cow, boot interface still has large patches out :( | 21:45 |
| jroll | so much for finishing any features :( | 21:46 |
| *** jaypipes has quit IRC | 21:47 | |
| rloo | jroll: oh yeah. the actual driver changes. i never got around to reviewing those. wanted zapping/raid more. | 21:47 |
| jroll | yep, same | 21:47 |
| jroll | it's just ilo and irmc | 21:48 |
| jroll | "just" | 21:48 |
| *** jst_ has quit IRC | 21:48 | |
| *** dims_ has joined #openstack-ironic | 21:49 | |
| *** karimb has quit IRC | 21:53 | |
| *** thrash is now known as thrash|g0ne | 21:55 | |
| *** Sukhdev has joined #openstack-ironic | 21:57 | |
| *** vishwanathj has quit IRC | 22:04 | |
| *** Sukhdev has quit IRC | 22:05 | |
| *** vishwanathj has joined #openstack-ironic | 22:05 | |
| *** vishwana_ has joined #openstack-ironic | 22:07 | |
| *** vishwanathj has quit IRC | 22:10 | |
| *** achanda has quit IRC | 22:11 | |
| openstackgerrit | Merged openstack/ironic: Imported Translations from Zanata https://review.openstack.org/224240 | 22:13 |
| jlvillal | So how do I hurry things out of the 'wait' state? I know I should already know this :( | 22:14 |
| jlvillal | $ ironic node-set-provision-state nuc1 manage | 22:14 |
| jlvillal | The requested action "manage" can not be performed on node "cf15449c-7d09-4b6c-b2db-7e07795ebc98" while it is in state "wait call-back". (HTTP 400) | 22:14 |
| mrda | ironic node-set-maintenance-state nuc1 on; ironic node-delete nuc1 <-- Is that what you want jlvillal ? | 22:15 |
| mrda | oh, and good morning :) | 22:15 |
| jlvillal | mrda: Thanks! :) Good morning. | 22:15 |
| jlvillal | mrda: Trying to debug my failing ramdisk issue. Got some USB to serial adapters :) | 22:16 |
| jlvillal | mrda: Uh, will that delete my node? I don't want to delete it. Just move it back to manage or something. | 22:16 |
| jlvillal | basically hurry up what would happen after the time-out occurs | 22:17 |
| *** dims_ has quit IRC | 22:17 | |
| *** sinval has quit IRC | 22:19 | |
| jlvillal | mrda: Hmm. Didn't seem to work, though I was too scared to do the 'node-delete' | 22:19 |
| *** sinval has joined #openstack-ironic | 22:19 | |
| openstackgerrit | Merged openstack/ironic: Remove DictMatches custom matcher from unit tests https://review.openstack.org/224287 | 22:20 |
| *** dims_ has joined #openstack-ironic | 22:20 | |
| *** ijw has quit IRC | 22:25 | |
| *** [1]cdearborn has quit IRC | 22:26 | |
| *** achanda has joined #openstack-ironic | 22:26 | |
| jlvillal | mrda: I think my hanging issue was related to console=ttyS0,.... I removed that and the kernel no longer hangs. | 22:30 |
| jlvillal | mrda: Now I see an error about -c not a valid option to tftp | 22:31 |
| jlvillal | And that I don't have a web server on port 80 | 22:31 |
| mrda | hmmm | 22:31 |
| jlvillal | As there is a curl command trying to run | 22:31 |
| jlvillal | mrda: Is there a way to turn on more verbose output of what the DIB image is doing? | 22:32 |
| jlvillal | Or better question, do you know a way to turn on more verbose output. | 22:32 |
| * jlvillal wanders off to tripleo-image-elements web page... | 22:33 | |
| mrda | jlvillal: I don't know of any | 22:44 |
| jlvillal | mrda: Thanks | 22:44 |
| mrda | jlvillal: ...because where we it store that? Unless you could configure a syslog as part of the deploy image | 22:46 |
| mrda | and send it off over the network | 22:46 |
| jlvillal | mrda: Okay, I'm just hoping to make it print out the commands it is running on the console | 22:47 |
| jlvillal | still playing around with it | 22:47 |
| mrda | so you could mount the deploy image locally, independent of the actual deploy, and step through what it's doing... | 22:48 |
| jlvillal | mrda: Oh, I didn't think about that. Thanks. | 22:52 |
| jlvillal | mrda: I would suggest on your blog to not have the console=ttyS0 lines | 22:52 |
| jlvillal | Maybe others might have hanging systems too. | 22:52 |
| mrda | ta | 22:53 |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Sending OneView Client implementation https://review.openstack.org/212796 | 22:54 |
| *** dims_ has quit IRC | 22:59 | |
| *** david-lyle has joined #openstack-ironic | 22:59 | |
| *** krtaylor has joined #openstack-ironic | 23:03 | |
| *** harshs has quit IRC | 23:03 | |
| *** lucasagomes has quit IRC | 23:16 | |
| *** trown|outttypeww is now known as trown | 23:25 | |
| *** baoli has joined #openstack-ironic | 23:25 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Refactor deploy_utils methods https://review.openstack.org/222264 | 23:34 |
| *** penick has joined #openstack-ironic | 23:38 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Refactor deploy_utils methods https://review.openstack.org/222264 | 23:39 |
| *** romcheg has quit IRC | 23:44 | |
| *** alexpilotti has joined #openstack-ironic | 23:49 | |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/ironic: OneView Driver for Ironic https://review.openstack.org/191822 | 23:50 |
| sinval | hey guys o/ | 23:51 |
| sinval | when you have time, please take a look at:https://review.openstack.org/191822 | 23:51 |
| sinval | thank you very much | 23:52 |
| sinval | https://review.openstack.org/191822 | 23:52 |
| *** shoutm has joined #openstack-ironic | 23:56 | |
| *** baoli has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!