| *** rloo has left #openstack-ironic | 00:00 | |
| *** puranamr has quit IRC | 00:00 | |
| *** puranamr has joined #openstack-ironic | 00:00 | |
| *** achanda_ has joined #openstack-ironic | 00:05 | |
| *** achanda has quit IRC | 00:07 | |
| *** achanda_ has quit IRC | 00:09 | |
| *** alexpilotti has joined #openstack-ironic | 00:11 | |
| lifeless | devananda: injected in the only sensible context I could find | 00:20 |
|---|---|---|
| lifeless | devananda: I don't think the meetings are overlapping with my sleep sadly | 00:20 |
| *** shadower has quit IRC | 00:23 | |
| *** shadower has joined #openstack-ironic | 00:23 | |
| *** puranamr has quit IRC | 00:26 | |
| *** mtanino has quit IRC | 00:32 | |
| *** puranamr has joined #openstack-ironic | 00:33 | |
| *** davideagnello has quit IRC | 00:34 | |
| *** davideagnello has joined #openstack-ironic | 00:34 | |
| *** puranamr has quit IRC | 00:37 | |
| *** davideagnello has quit IRC | 00:38 | |
| *** davideagnello has joined #openstack-ironic | 00:38 | |
| *** davideagnello has quit IRC | 00:41 | |
| *** davideag_ has joined #openstack-ironic | 00:41 | |
| *** baoli has joined #openstack-ironic | 00:46 | |
| *** Sukhdev has quit IRC | 00:50 | |
| *** shoutm has quit IRC | 00:51 | |
| *** baoli has quit IRC | 00:51 | |
| *** praneshp has quit IRC | 00:59 | |
| *** alexpilotti has quit IRC | 01:00 | |
| *** shoutm has joined #openstack-ironic | 01:04 | |
| *** chenke has quit IRC | 01:08 | |
| *** chenke has joined #openstack-ironic | 01:09 | |
| *** mtanino has joined #openstack-ironic | 01:14 | |
| *** zhenguo has joined #openstack-ironic | 01:22 | |
| *** alexpilotti has joined #openstack-ironic | 01:26 | |
| *** penick has quit IRC | 01:27 | |
| *** chenke has quit IRC | 01:29 | |
| *** bizarrochristy has joined #openstack-ironic | 01:30 | |
| *** bizarrochristy has quit IRC | 01:34 | |
| openstackgerrit | chenglch proposed openstack/ironic-specs: Add console log support https://review.openstack.org/164586 | 01:38 |
| *** davideag_ has quit IRC | 01:41 | |
| jroll | krotscheck: do you want to continue using the 2x +2 rule on this stuff? | 01:49 |
| jroll | or should I just land that whole thing | 01:49 |
| krotscheck | jroll: Ask the PTL :) | 01:49 |
| jroll | lol | 01:51 |
| jroll | I mean if other people are going to figure out how this stuff works, and review it, 2x is fine | 01:51 |
| jroll | but I'm pretty sure I'm the only one brave enough to +2 these | 01:51 |
| jroll | devananda: ^ thoughts? | 01:52 |
| jroll | lifeless: devananda: going back to previous discussion, it's not only CD people harmed by "reverting" this change in a new API version - it's also application developers. inspector, bifrost, etc. if we "revert" this, they must support old, now, and new. | 01:56 |
| jroll | lifeless: devananda: also, with how I understand bifrost works, any bifrost user is a CD user and may not be in the community. | 01:57 |
| lifeless | jroll: do they? | 01:57 |
| lifeless | jroll: they only need to care about releases | 01:57 |
| lifeless | jroll: AIUI | 01:57 |
| lifeless | jroll: so old and new, since now hasn't been in a release | 01:57 |
| jroll | lifeless: so inspector shouldn't be work for people doing CD? | 01:57 |
| lifeless | they may have code for now in -tree, supporting folk doing CD | 01:57 |
| jroll | or almost-CD | 01:58 |
| lifeless | but once Ironic releases they can support new, cut a release, and then delete now | 01:58 |
| jroll | what if a deployer deploys now and doesn't deploy again for months | 01:58 |
| jroll | or years | 01:58 |
| jroll | etc | 01:58 |
| lifeless | then they don't upgrade their stuff on top of it either | 01:58 |
| jroll | so they don't get bug fixes to inspector | 01:59 |
| jroll | etc. | 01:59 |
| lifeless | or sure bifrost might choose to support all versions of the API | 01:59 |
| lifeless | but my experience with such tools in the wild in other contexts is that they don't | 01:59 |
| lifeless | they generally support the oldest thing needed to do their job | 01:59 |
| lifeless | and upgrade when their needs change or when the oldest thing stops being supported | 02:00 |
| lifeless | TheJulia is clearly the person to ask in this case though | 02:00 |
| jroll | yeah, you get the idea | 02:00 |
| jroll | it's more complex then breaking CD users (again) | 02:00 |
| lifeless | sure | 02:01 |
| jroll | and it's also opt-in via the api version. | 02:01 |
| lifeless | but this one isn't hard to work through. | 02:01 |
| lifeless | (analyse I mean) | 02:01 |
| jroll | if you aren't ready for this version, you don't bump your requested api version | 02:01 |
| jroll | and when you are ready, you do | 02:01 |
| jroll | whether you deploy every day or every two years | 02:02 |
| lifeless | that is the primary virtue of immutable API versions | 02:02 |
| lifeless | it doesn't speak to the cost of those upgrades for users, and what a breaking change will be | 02:03 |
| lifeless | which is the heart of this specific discussion AIUI | 02:03 |
| jroll | it isn't breaking unless they opt in to it | 02:03 |
| lifeless | I believe that statement to be false | 02:03 |
| jroll | nothing defaults to using >= this api version | 02:04 |
| lifeless | could you help me verify my belief ? | 02:04 |
| lifeless | users that use the CLI tools | 02:04 |
| lifeless | current get an API version chosen for them | 02:04 |
| lifeless | the algorithm doesn't matter | 02:04 |
| jroll | right, so for clarity this is version 1.11 | 02:04 |
| lifeless | today ? | 02:04 |
| jroll | yes | 02:05 |
| lifeless | in the releases api client? | 02:05 |
| jroll | both the CLI and the python client default to 1.6, today. | 02:05 |
| lifeless | ack ok | 02:05 |
| lifeless | so | 02:05 |
| lifeless | either a) we support 1.6 forever and the client choses 1.6 by default forever | 02:05 |
| *** meghal has joined #openstack-ironic | 02:05 | |
| lifeless | or b) at some point the client will choose >=1.11 and users will have an incompatibility occur merely by upgrading their client | 02:06 |
| jroll | right, so we've gone with a.1) the client defaults to the earliest supported API version forever, once we deprecate "1.6 by default" and get to "earliest" | 02:07 |
| *** meghal1 has joined #openstack-ironic | 02:07 | |
| jroll | we've decided to go with* | 02:07 |
| lifeless | that seems to be the thing that is giving devananda pause | 02:08 |
| jroll | I can't speak for him, but it seems that "requires work to go 1.10 -> 1.11" is the thing. | 02:09 |
| lifeless | because the combination is essentially no stability offering for users | 02:09 |
| jroll | why? | 02:09 |
| *** meghal has quit IRC | 02:10 | |
| jroll | fwiw, let me point out that changing the minimum version is also potentially a breaking change. | 02:10 |
| lifeless | the principle he seems to hold - which I hold to - is that users should opt-in to breaking changes | 02:10 |
| jroll | I agree | 02:10 |
| lifeless | if the way that that is achieved is by new API versions | 02:10 |
| lifeless | then the choice to use a new API version must always be explicit | 02:11 |
| lifeless | otherwise opt-in is not occuring | 02:11 |
| jroll | correct | 02:11 |
| *** mtanino has quit IRC | 02:11 | |
| *** mgoddard has quit IRC | 02:12 | |
| lifeless | having the CLI ever change the version of the API will force a non-opt-in change on users | 02:13 |
| jroll | right | 02:14 |
| jroll | clearly we may want to do this at some point | 02:14 |
| jroll | and I think there's probably a reasonable deprecation path, when we do want to do that | 02:15 |
| *** alexpilotti has quit IRC | 02:15 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Refactor agent driver with pxe boot interface https://review.openstack.org/166521 | 02:17 |
| lifeless | so I dunno | 02:17 |
| lifeless | Nova took a route that seems similar | 02:17 |
| lifeless | in that they had breaking API changes | 02:17 |
| lifeless | and their users resoundingly ignored it | 02:17 |
| lifeless | till they deprecated the new thing entirely and went to doing much more careful only-break-when-needed patterns | 02:18 |
| *** shoutm has quit IRC | 02:18 | |
| lifeless | which the ironic design was inspired by | 02:18 |
| jroll | right | 02:19 |
| jroll | but that was an entire API rewrite, AIUI | 02:19 |
| lifeless | not really | 02:19 |
| lifeless | schema on the front, more http compatible and rigorously defined return values // side effects | 02:19 |
| jroll | are we talking about v3? | 02:19 |
| lifeless | yeah | 02:19 |
| lifeless | depends on what you mean by rewrite I suspect | 02:19 |
| jroll | yeah, the semantics were completely different in terms of the data | 02:20 |
| jroll | sure, probably a bad word | 02:20 |
| *** shoutm has joined #openstack-ironic | 02:20 | |
| lifeless | most v2 clients could use v3 without change in all likelyhood | 02:20 |
| *** achanda has joined #openstack-ironic | 02:23 | |
| *** smoriya has joined #openstack-ironic | 02:29 | |
| *** alexpilotti has joined #openstack-ironic | 02:33 | |
| *** alexpilotti has quit IRC | 02:37 | |
| *** alexpilotti has joined #openstack-ironic | 02:44 | |
| *** shoutm has quit IRC | 02:44 | |
| *** logan2 has quit IRC | 02:44 | |
| *** lazy_prince has joined #openstack-ironic | 02:45 | |
| *** hakimo has joined #openstack-ironic | 02:52 | |
| *** hakimo_ has quit IRC | 02:54 | |
| *** zz_natorious is now known as natorious | 02:54 | |
| *** ramineni has joined #openstack-ironic | 02:56 | |
| *** Nisha has joined #openstack-ironic | 02:57 | |
| *** shoutm has joined #openstack-ironic | 03:01 | |
| *** ramineni1 has joined #openstack-ironic | 03:02 | |
| openstackgerrit | Merged openstack/ironic: Add copyright notice to iRMC driver source code. https://review.openstack.org/212973 | 03:02 |
| *** ramineni has quit IRC | 03:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/213466 | 03:06 |
| *** saripurigopi has joined #openstack-ironic | 03:10 | |
| *** natorious is now known as zz_natorious | 03:14 | |
| *** saripurigopi has quit IRC | 03:16 | |
| *** Haomeng has quit IRC | 03:17 | |
| *** Haomeng has joined #openstack-ironic | 03:18 | |
| *** saripurigopi has joined #openstack-ironic | 03:18 | |
| *** alexpilotti has quit IRC | 03:33 | |
| *** lazy_prince has quit IRC | 03:33 | |
| *** logan2 has joined #openstack-ironic | 03:34 | |
| *** praneshp has joined #openstack-ironic | 03:40 | |
| *** bizarrochristy has joined #openstack-ironic | 03:50 | |
| *** bizarrochristy has quit IRC | 03:55 | |
| *** alexpilotti has joined #openstack-ironic | 03:59 | |
| *** shoutm has quit IRC | 04:00 | |
| *** praneshp has quit IRC | 04:03 | |
| *** zhenguo has quit IRC | 04:06 | |
| *** shoutm has joined #openstack-ironic | 04:06 | |
| *** coolsvap|away is now known as coolsvap | 04:10 | |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/python-ironicclient: Introduce openstackclient plugin https://review.openstack.org/171672 | 04:12 |
| *** padiyar has joined #openstack-ironic | 04:16 | |
| *** padiyar has quit IRC | 04:18 | |
| *** praneshp has joined #openstack-ironic | 04:20 | |
| *** PradeepV has joined #openstack-ironic | 04:23 | |
| *** padiyar has joined #openstack-ironic | 04:24 | |
| PradeepV | Good morning all | 04:24 |
| *** padiyar has quit IRC | 04:24 | |
| PradeepV | Need a help, I am getting the docker connection timed out error when deploying a flat network for ironic through devstack http://paste.openstack.org/show/419972/ | 04:25 |
| PradeepV | can any one faced this issue | 04:25 |
| *** shoutm_ has joined #openstack-ironic | 04:27 | |
| *** shoutm has quit IRC | 04:28 | |
| PradeepV | Need a help, I am getting the docker connection timed out error when deploying a flat network for ironic through devstack http://paste.openstack.org/show/419972/ | 04:48 |
| PradeepV | anyone faced the same issue | 04:48 |
| *** alexpilotti has quit IRC | 04:55 | |
| *** mestery has joined #openstack-ironic | 04:59 | |
| *** shoutm_ has quit IRC | 05:03 | |
| *** mestery has quit IRC | 05:09 | |
| *** alexpilotti has joined #openstack-ironic | 05:12 | |
| *** praneshp_ has joined #openstack-ironic | 05:12 | |
| *** praneshp has quit IRC | 05:14 | |
| *** praneshp_ is now known as praneshp | 05:14 | |
| *** alexpilotti has quit IRC | 05:16 | |
| *** smoriya has quit IRC | 05:21 | |
| *** alexpilotti has joined #openstack-ironic | 05:23 | |
| *** meghal1 has quit IRC | 05:25 | |
| *** shoutm has joined #openstack-ironic | 05:34 | |
| *** lsmola has joined #openstack-ironic | 05:39 | |
| *** achanda has quit IRC | 05:43 | |
| *** achanda has joined #openstack-ironic | 05:47 | |
| *** Sukhdev has joined #openstack-ironic | 05:51 | |
| *** bizarrochristy has joined #openstack-ironic | 06:02 | |
| *** shoutm has quit IRC | 06:05 | |
| *** bizarrochristy has quit IRC | 06:07 | |
| *** Marga_ has joined #openstack-ironic | 06:08 | |
| *** Marga_ has quit IRC | 06:09 | |
| *** Marga_ has joined #openstack-ironic | 06:09 | |
| *** shoutm has joined #openstack-ironic | 06:12 | |
| *** alexpilotti has quit IRC | 06:17 | |
| *** coolsvap is now known as coolsvap|away | 06:22 | |
| *** zhenguo has joined #openstack-ironic | 06:24 | |
| *** zhenguo has quit IRC | 06:32 | |
| *** alexpilotti has joined #openstack-ironic | 06:34 | |
| *** alexpilotti has quit IRC | 06:38 | |
| *** shoutm_ has joined #openstack-ironic | 06:39 | |
| *** shoutm has quit IRC | 06:42 | |
| *** openstackgerrit_ has joined #openstack-ironic | 06:43 | |
| Nisha | yuriyz, | 06:43 |
| Nisha | yuriyz, hi | 06:43 |
| *** alexpilotti has joined #openstack-ironic | 06:46 | |
| *** amotoki has quit IRC | 07:01 | |
| *** Vikas has joined #openstack-ironic | 07:03 | |
| *** Sukhdev has quit IRC | 07:07 | |
| *** ifarkas has joined #openstack-ironic | 07:09 | |
| *** PradeepV has quit IRC | 07:12 | |
| *** dguerri` is now known as dguerri | 07:13 | |
| *** dguerri is now known as dguerri` | 07:16 | |
| *** praneshp_ has joined #openstack-ironic | 07:17 | |
| *** praneshp has quit IRC | 07:17 | |
| *** praneshp_ is now known as praneshp | 07:17 | |
| *** dguerri` is now known as dguerri | 07:24 | |
| *** dguerri is now known as dguerri` | 07:32 | |
| *** dguerri` is now known as dguerri | 07:32 | |
| *** achanda has quit IRC | 07:33 | |
| *** dguerri is now known as dguerri` | 07:39 | |
| *** amotoki has joined #openstack-ironic | 07:42 | |
| *** alexpilotti has quit IRC | 07:48 | |
| *** bradjones has quit IRC | 07:53 | |
| *** jistr has joined #openstack-ironic | 07:55 | |
| *** mbound has joined #openstack-ironic | 07:55 | |
| *** bradjones has joined #openstack-ironic | 07:56 | |
| *** bradjones has quit IRC | 07:56 | |
| *** bradjones has joined #openstack-ironic | 07:56 | |
| *** jcoufal has joined #openstack-ironic | 07:56 | |
| *** athomas has joined #openstack-ironic | 08:01 | |
| *** lucasagomes has joined #openstack-ironic | 08:02 | |
| *** alexpilotti has joined #openstack-ironic | 08:06 | |
| *** alexpilotti has quit IRC | 08:11 | |
| *** derekh has joined #openstack-ironic | 08:12 | |
| *** romainh has joined #openstack-ironic | 08:12 | |
| *** derekh has quit IRC | 08:12 | |
| *** derekh has joined #openstack-ironic | 08:13 | |
| *** mgoddard has joined #openstack-ironic | 08:15 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers https://review.openstack.org/198656 | 08:16 |
| *** alexpilotti has joined #openstack-ironic | 08:18 | |
| *** ramineni1 has quit IRC | 08:19 | |
| *** mgoddard1 has joined #openstack-ironic | 08:24 | |
| yuriyz | morning Ironic | 08:24 |
| *** mgoddard has quit IRC | 08:26 | |
| *** ramineni has joined #openstack-ironic | 08:28 | |
| lucasagomes | yuriyz, good morning | 08:28 |
| *** mgoddard1 has quit IRC | 08:28 | |
| yuriyz | morning lucasagomes | 08:29 |
| *** rameshg87 has joined #openstack-ironic | 08:34 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers https://review.openstack.org/198656 | 08:34 |
| Nisha | lucasagomes, yuriyz morning | 08:35 |
| yuriyz | morning Nisha | 08:36 |
| *** padiyar has joined #openstack-ironic | 08:37 | |
| Nisha | lucasagomes, yuriyz :) please review https://review.openstack.org/198656 https://review.openstack.org/201904 | 08:37 |
| Nisha | rameshg87, ^^^ | 08:37 |
| rameshg87 | Nisha: ack | 08:43 |
| openstackgerrit | chenglch proposed openstack/ironic: Add iscsi_opc and agent_opc to manage the OpenPOWER servers https://review.openstack.org/209824 | 08:44 |
| *** PradeepV has joined #openstack-ironic | 08:45 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 08:46 |
| PradeepV | HI All, when i am trying to bring up the devstack setup, getting error as "dial tcp 54.208.162.63:443: connection timed out" | 08:46 |
| PradeepV | this is the error from output, http://paste.openstack.org/show/419972/ | 08:46 |
| *** bizarrochristy has joined #openstack-ironic | 08:47 | |
| *** shoutm_ has quit IRC | 08:48 | |
| *** alexpilotti has quit IRC | 08:51 | |
| rameshg87 | ramineni: are you going to address the node thing in next patch for https://review.openstack.org/#/c/211849/2 ? | 08:51 |
| rameshg87 | ramineni: if so, let's change this to partial-bug and probably try to merge this | 08:52 |
| ramineni | rameshg87: yes, will post one more patch | 08:53 |
| ramineni | rameshg87: i will make it partial-bug | 08:53 |
| rameshg87 | ramineni: just hold on, just checking a few things | 08:54 |
| rameshg87 | ramineni: I will post my vote in a minute | 08:54 |
| rameshg87 | if there is something else I see | 08:54 |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Old value 'updated_at' field returned after update https://review.openstack.org/211849 | 08:54 |
| ramineni | rameshg87: :) , just updated on gerrit anyway | 08:55 |
| rameshg87 | ramineni: np .. | 08:55 |
| ramineni | rameshg87: you can post the comments, if any :) | 08:55 |
| *** Vikas has quit IRC | 08:56 | |
| *** mgoddard has joined #openstack-ironic | 08:59 | |
| rameshg87 | ramineni: lgtm, but commit message seems confusing | 08:59 |
| rameshg87 | ramineni: can we just tell about the actual fix that save() doesn't return the updated object | 08:59 |
| rameshg87 | ramineni: because from the commit, it looks like there was some change in PATCH operation, but it turns out to be somewhere else | 09:00 |
| ramineni | rameshg87: ok, will change that | 09:00 |
| rameshg87 | ramineni: proposing something, hold on | 09:00 |
| ramineni | ok | 09:01 |
| *** shikel has quit IRC | 09:01 | |
| *** romcheg has joined #openstack-ironic | 09:02 | |
| rameshg87 | ramineni: commented, please check and update if required | 09:03 |
| ramineni | rameshg87: ok, thanks | 09:03 |
| ramineni | rameshg87: And also, could you let me know why jenkins is failing on this https://review.openstack.org/#/c/205895/11? | 09:04 |
| ramineni | rameshg87: couldnt figure out :( | 09:04 |
| *** bizarrochristy has quit IRC | 09:05 | |
| *** bizarrochristy has joined #openstack-ironic | 09:05 | |
| rameshg87 | ramineni: docs job was broken a while back | 09:05 |
| *** sh1kel has joined #openstack-ironic | 09:06 | |
| *** MattMan has joined #openstack-ironic | 09:06 | |
| rameshg87 | ramineni: better to recheck once | 09:06 |
| rameshg87 | the result is 5 days old | 09:06 |
| ramineni | rameshg87: hmm, ok, let me try | 09:06 |
| *** MattMan has quit IRC | 09:07 | |
| *** MattMan has joined #openstack-ironic | 09:08 | |
| *** pelix has joined #openstack-ironic | 09:09 | |
| *** bizarrochristy has quit IRC | 09:09 | |
| ramineni | rameshg87: added a comment , please check and let me know if its fine? | 09:13 |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Make states end-point discoverable via nodes API https://review.openstack.org/205895 | 09:14 |
| *** romcheg has quit IRC | 09:21 | |
| betherly | morning Ironic :) | 09:21 |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 09:23 |
| *** mgoddard has quit IRC | 09:26 | |
| *** alexpilotti has joined #openstack-ironic | 09:26 | |
| rameshg87 | ramineni: yeah, but mentioning about PATCH doesn't seem right to me either | 09:27 |
| rameshg87 | ramineni: I thought it doesn't update the object completely and that was the problem | 09:28 |
| rameshg87 | ramineni: let's see others opinions | 09:28 |
| rameshg87 | lucasagomes: yuriyz: have some thoughts on my comment on https://review.openstack.org/#/c/211849/3//COMMIT_MSG | 09:28 |
| rameshg87 | ?? | 09:28 |
| * lucasagomes reads | 09:28 | |
| *** mgoddard has joined #openstack-ironic | 09:29 | |
| lucasagomes | hmm maybe saying "'updated_at' field shows old value after *resource is updated*" instead of PATCH OP then? | 09:30 |
| lucasagomes | rameshg87, ramineni ^ ? | 09:32 |
| rameshg87 | lucasagomes: yea, s/resource is updated/resource is saved/ ? | 09:33 |
| rameshg87 | ramineni: ^^ | 09:33 |
| *** padiyar has quit IRC | 09:33 | |
| lucasagomes | rameshg87, any works | 09:33 |
| *** max_lobur has joined #openstack-ironic | 09:35 | |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: chassis.save() and port.save() doesn't update object after save https://review.openstack.org/211849 | 09:36 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: 'updated_at' field shows old value after resource is saved https://review.openstack.org/211849 | 09:39 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: 'updated_at' field shows old value after resource is saved https://review.openstack.org/211849 | 09:39 |
| ramineni | lucasagomes: rameshg87, missed the chat | 09:40 |
| ramineni | rameshg87: thanks for updating :) | 09:40 |
| rameshg87 | ramineni: wc | 09:40 |
| lucasagomes | rameshg87, about https://review.openstack.org/#/c/211849/3//COMMIT_MSG | 09:40 |
| lucasagomes | oh it was updated already | 09:40 |
| rameshg87 | lucasagomes: yeah, I just updated it as per we discussed in the latest patch | 09:40 |
| lucasagomes | cool | 09:40 |
| * lucasagomes reviwed | 09:40 | |
| *** athomas has quit IRC | 09:44 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 09:47 |
| *** athomas has joined #openstack-ironic | 09:48 | |
| *** jrist has quit IRC | 09:54 | |
| *** alexpilotti has quit IRC | 09:55 | |
| *** naohirot has quit IRC | 09:56 | |
| *** romainh has quit IRC | 10:06 | |
| ramineni | Any idea on this error CRITICAL ironic [-] AttributeError: `pecan.state` is not bound to a context-local context - ironic-docs is failing with this error | 10:08 |
| ramineni | not sure , how to resolve this | 10:08 |
| ramineni | rameshg87: lucasagomes: dtantsur|pto19Au : ^^ | 10:08 |
| lucasagomes | hmm gotta investigate | 10:09 |
| lucasagomes | not sure | 10:09 |
| ramineni | rameshg87: https://review.openstack.org/#/c/205895/ , again failed :( | 10:09 |
| lucasagomes | oh wow | 10:09 |
| ramineni | lucasagomes: ok, just failing on above patch | 10:09 |
| lucasagomes | so gate is broken? | 10:09 |
| lucasagomes | oh ok | 10:09 |
| *** romcheg has joined #openstack-ironic | 10:10 | |
| ramineni | lucasagomes: dont think so, other patches going through fine | 10:10 |
| rameshg87 | ramineni: oh I think because of this | 10:12 |
| rameshg87 | ramineni: https://review.openstack.org/#/c/205895/12/ironic/api/controllers/v1/node.py | 10:12 |
| rameshg87 | ramineni: docs job generates the samples from def sample (L 658) | 10:12 |
| rameshg87 | ramineni: it calls _convert_with_links | 10:12 |
| rameshg87 | ramineni: which intern calls is_acceptable_api_version() which uses pecan.request | 10:13 |
| rameshg87 | ramineni: in docs job, we wouldn't have pecan.request set which ends up in AttributeError | 10:13 |
| lucasagomes | makes sense | 10:14 |
| ramineni | rameshg87: oohok | 10:14 |
| rameshg87 | ramineni: long before we had try/except AttributeError around other code which accessed pecan.request | 10:14 |
| rameshg87 | ramineni: now I realized why it was there :) | 10:14 |
| ramineni | rameshg87: :) , but how to get version number without using it | 10:14 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Periodically checks for nodes being cleaned https://review.openstack.org/213699 | 10:14 |
| rameshg87 | ramineni: I think you can just add a try/except AttributeError around checking pecan.request.version.minor | 10:15 |
| rameshg87 | ramineni: and return True if it's an AttributeError | 10:15 |
| ramineni | rameshg87: ok | 10:16 |
| rameshg87 | just commented | 10:16 |
| rameshg87 | ramineni: and please add a comment why catching AttributeError is required (sometimes back a lot of people wondered why it was there :)) | 10:16 |
| ramineni | rameshg87: ok, sure, but is this only required for docs to go through? | 10:19 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 10:19 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Address final comments on update image cache based on update time https://review.openstack.org/214114 | 10:21 |
| *** Nisha has quit IRC | 10:24 | |
| *** shoutm has joined #openstack-ironic | 10:27 | |
| *** mgoddard has quit IRC | 10:29 | |
| *** alexpilotti has joined #openstack-ironic | 10:30 | |
| rameshg87 | ramineni: yes | 10:37 |
| ramineni | rameshg87: ok, | 10:38 |
| rameshg87 | lucasagomes: hi | 10:41 |
| lucasagomes | rameshg87, hi there | 10:42 |
| rameshg87 | lucasagomes: in your series of patch sets in https://review.openstack.org/#/c/213240/2 | 10:42 |
| rameshg87 | lucasagomes: we are also extending the timeout for out-of-band cleaning operations, right | 10:43 |
| lucasagomes | rameshg87, that's done when the node is in CLEANING right? | 10:43 |
| rameshg87 | lucasagomes: yeah out of band cleaning operations | 10:43 |
| rameshg87 | lucasagomes: which doesn't use agent | 10:43 |
| lucasagomes | this callback timeout is for in-band operations | 10:43 |
| lucasagomes | similar to do DEPLOYWAIT | 10:43 |
| lucasagomes | we perhaps can use the same config option to timeout the out-of-band operations | 10:44 |
| rameshg87 | lucasagomes: but where do we make the distinction ? | 10:44 |
| rameshg87 | lucasagomes: we just check for nodes whole cleaning_update_at is before 30 mins, right ? | 10:44 |
| lucasagomes | rameshg87, and the node is in CLEANWAIT as well | 10:44 |
| rameshg87 | oh | 10:44 |
| rameshg87 | missed that part | 10:45 |
| rameshg87 | thanks | 10:45 |
| lucasagomes | we check if the cleaning is being executed | 10:45 |
| *** bizarrochristy has joined #openstack-ironic | 10:45 | |
| lucasagomes | cause for CLEANING state, we know it's being execute (the conductor is executing it) | 10:45 |
| rameshg87 | yeah | 10:45 |
| rameshg87 | if some out of band driver wants to implement a cleaning task that takes a lot of time, | 10:45 |
| rameshg87 | do we recommend them to put it in CLEANING only, righ ? | 10:46 |
| rameshg87 | I mean even if they poll the status by periodic task | 10:46 |
| lucasagomes | rameshg87, no it's fine, because I will updated cleaning_updated_at with the heartbeat | 10:47 |
| lucasagomes | (see touch_cleaning) | 10:47 |
| lucasagomes | so they can take as much time as needed | 10:47 |
| lucasagomes | (for e.g cleaning a really really long disk) | 10:47 |
| rameshg87 | yeah, but heartbeat is only for in-band cleaning | 10:48 |
| rameshg87 | but if some out of band clean step triggered something and polled the status in a periodic task | 10:48 |
| *** sh1kel has quit IRC | 10:48 | |
| rameshg87 | I think they will have to do touch_cleaning() once they find out that cleaning task is still in progress | 10:48 |
| rameshg87 | right ? | 10:48 |
| lucasagomes | I don't think that's needed, because for out-of-band the provision_state is CLEANING right? | 10:50 |
| lucasagomes | so the periodic task won't track that node | 10:50 |
| lucasagomes | ofc, the conductor should track the state of the out-of-band request | 10:50 |
| *** alexpilotti has quit IRC | 10:50 | |
| lucasagomes | but not necessarily it needs to call touch_cleaning() | 10:50 |
| lucasagomes | also, cleaning_updated_at is updated when we move from CLEANING->CLEANWAIT (or vice versa) | 10:51 |
| lucasagomes | so it will be fine if the next clean step is in-band (after a long out-of-band clean step) | 10:51 |
| *** mgoddard has joined #openstack-ironic | 10:52 | |
| rameshg87 | yeah, if we have out of band cleaning step always using CLEANING | 10:52 |
| rameshg87 | it's fine then .. got it | 10:52 |
| lucasagomes | yeah, that's the idea (that's what it does today) | 10:53 |
| lucasagomes | we distinguish in-band and out-of-band operations with those 2 states | 10:54 |
| lucasagomes | for both deploy and cleaning | 10:54 |
| rameshg87 | okay | 10:54 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add RPCAPIs for RAID configuration https://review.openstack.org/196006 | 11:05 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add APIs for RAID configuration https://review.openstack.org/196007 | 11:11 |
| * rameshg87 goes home | 11:16 | |
| *** rameshg87 has quit IRC | 11:16 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add support for inband raid configuration agent ramdisk https://review.openstack.org/198238 | 11:16 |
| *** baoli has joined #openstack-ironic | 11:17 | |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Make states end-point discoverable via nodes API https://review.openstack.org/205895 | 11:18 |
| *** lucasagomes is now known as lucas-hungry | 11:19 | |
| *** baoli has quit IRC | 11:21 | |
| *** trown|outttypeww is now known as trown | 11:23 | |
| *** alexpilotti has joined #openstack-ironic | 11:25 | |
| *** ramineni has quit IRC | 11:27 | |
| *** romainh has joined #openstack-ironic | 11:37 | |
| *** zsmithnyc has quit IRC | 11:39 | |
| *** alexpilotti has quit IRC | 11:44 | |
| *** amotoki has quit IRC | 11:53 | |
| *** romcheg has quit IRC | 12:01 | |
| *** saripurigopi has quit IRC | 12:04 | |
| *** dprince has joined #openstack-ironic | 12:04 | |
| *** dprince has quit IRC | 12:07 | |
| *** dprince has joined #openstack-ironic | 12:07 | |
| *** romcheg has joined #openstack-ironic | 12:08 | |
| *** alexpilotti has joined #openstack-ironic | 12:11 | |
| *** thiagop has joined #openstack-ironic | 12:22 | |
| thiagop | Good morning ladies and gentlemen | 12:22 |
| betherly | Morning thiagop | 12:22 |
| *** bizarrochristy has quit IRC | 12:22 | |
| *** bizarrochristy has joined #openstack-ironic | 12:23 | |
| *** BadCub has quit IRC | 12:25 | |
| *** bizarrochristy has quit IRC | 12:27 | |
| *** BadCub has joined #openstack-ironic | 12:27 | |
| *** zhenguo has joined #openstack-ironic | 12:29 | |
| *** uggla_ has joined #openstack-ironic | 12:31 | |
| uggla_ | Hello | 12:31 |
| thiagop | morning uggla_ | 12:32 |
| *** lucas-hungry is now known as lucasagomes | 12:34 | |
| *** alexpilotti has quit IRC | 12:41 | |
| *** rloo has joined #openstack-ironic | 12:44 | |
| thiagop | do we have a gate bug on neutron? | 12:48 |
| *** baoli has joined #openstack-ironic | 12:48 | |
| sambetts | Morning all o/ | 12:50 |
| thiagop | morning sambetts | 12:50 |
| thiagop | lucasagomes: the OneView spec is still in your queue, right? | 12:51 |
| *** chlong has joined #openstack-ironic | 12:52 | |
| *** baoli has quit IRC | 12:52 | |
| *** jjohnson2 has joined #openstack-ironic | 12:58 | |
| *** PradeepV has quit IRC | 13:05 | |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic: Add support for inband raid configuration agent ramdisk https://review.openstack.org/198238 | 13:05 |
| thiagop | People, can you take a look at this spec: https://review.openstack.org/#/c/187762/ | 13:05 |
| thiagop | any feedback is appreciated | 13:05 |
| thiagop | jroll: could you? ^ | 13:07 |
| *** alexpilotti has joined #openstack-ironic | 13:07 | |
| *** persia_ is now known as persia | 13:19 | |
| lucasagomes | thiagop, will review! | 13:22 |
| thiagop | just checking that you didn't forget :) | 13:23 |
| lucasagomes | thiagop, np, 1 thing when is the server profile associated with the node? | 13:23 |
| lucasagomes | once it's being deployed? | 13:24 |
| thiagop | previously to enrollment | 13:24 |
| thiagop | we found that option more in line with Ironic current workflow | 13:25 |
| *** Shrews has joined #openstack-ironic | 13:25 | |
| lucasagomes | right, so this is something that will be passed to the driver_info? | 13:25 |
| thiagop | not necessarily. The idea is to apply the server profile on the blade and then enroll it on Ironic, so Ironic doesn't need to know about the profile (or templates) | 13:27 |
| *** bizarrochristy has joined #openstack-ironic | 13:28 | |
| thiagop | in a second moment (after liberty) we'll propose a change to apply it on zapping to free the machine when no instance is associated, but it needs more brainwork | 13:28 |
| thiagop | specially because it doesn't fit exactly with "The Ironic way" | 13:29 |
| *** foexle has joined #openstack-ironic | 13:34 | |
| *** Shrews_ has joined #openstack-ironic | 13:37 | |
| openstackgerrit | Lilia Sampaio proposed openstack/ironic: [WIP]OneView Driver for Ironic https://review.openstack.org/191822 | 13:37 |
| *** alexpilotti has quit IRC | 13:38 | |
| *** Shrews_ has quit IRC | 13:40 | |
| *** Shrews has quit IRC | 13:41 | |
| *** Shrews has joined #openstack-ironic | 13:41 | |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/bifrost: Fix typo in bifrost docs https://review.openstack.org/214192 | 13:44 |
| lucasagomes | thiagop, reviewed... pointed out some stuff I found confusing when reviewing (those things I asked you about) but overall I think it's pretty good | 13:50 |
| lucasagomes | thiagop, voted +1 but it could be a +2 if you feel like it doesn't need to be updated | 13:50 |
| lucasagomes | thiagop, take a look at the comments when you get some time please | 13:50 |
| sinval | lucasagomes: thanks, we gonna look at you comments soon | 13:51 |
| lucasagomes | sinval, cool! If you update the spec (or feel it's not necessary) ping me again and I will re-review it | 13:51 |
| thiagop | lucasagomes: I'll take a look on that right now, thanks | 13:52 |
| lucasagomes | cool thanks folks! | 13:53 |
| liliars_ | lucasagomes, thanks for your review! (: | 13:55 |
| lucasagomes | np | 13:55 |
| jlvillal | Good morning Ironic | 14:01 |
| sambetts | Morning jlvillal | 14:01 |
| jlvillal | sambetts, Thanks. I hope all is well in the UK :) | 14:02 |
| lucasagomes | jlvillal, sambetts morning | 14:03 |
| *** zsmithnyc has joined #openstack-ironic | 14:03 | |
| thiagop | morning jlvillal | 14:03 |
| sambetts | jlvillal: The UK is a bit wet today, but all is well otherwise | 14:03 |
| jlvillal | lucasagomes, thiagop Thanks :) Trying to wake up. | 14:03 |
| lucasagomes | lol | 14:03 |
| lucasagomes | get some coffee | 14:03 |
| * thiagop hands some coffee to jlvillal | 14:04 | |
| jlvillal | sambetts, I remember it being wet when I lived in Huntingdon for three years. That is between Peterborough and Cambridge. | 14:04 |
| * jlvillal agrees about coffee. But I will start with tea first :) | 14:04 | |
| *** alexpilotti has joined #openstack-ironic | 14:04 | |
| *** uggla_ has quit IRC | 14:08 | |
| *** shoutm has quit IRC | 14:20 | |
| NobodyCam | good morning Ironicers :) | 14:22 |
| thiagop | good morning NobodyCam | 14:23 |
| jlvillal | NobodyCam, good morning | 14:25 |
| *** Shrews has quit IRC | 14:25 | |
| *** rameshg87 has joined #openstack-ironic | 14:26 | |
| *** Shrews has joined #openstack-ironic | 14:26 | |
| NobodyCam | good morning thiagop, jlvillal, lucasagomes, sambetts, TheJulia, rloo, rameshg87, jroll, JoshNang and everyone else too :) | 14:26 |
| rameshg87 | morning NobodyCam | 14:28 |
| *** zz_natorious is now known as natorious | 14:29 | |
| lucasagomes | NobodyCam, morning! | 14:30 |
| NobodyCam | :) | 14:30 |
| *** alexpilotti has quit IRC | 14:32 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Refactor agent driver with pxe boot interface https://review.openstack.org/166521 | 14:34 |
| *** mgoddard1 has joined #openstack-ironic | 14:36 | |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands https://review.openstack.org/172517 | 14:37 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create https://review.openstack.org/172461 | 14:37 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add provision state to set command https://review.openstack.org/206119 | 14:37 |
| *** mgoddard has quit IRC | 14:37 | |
| *** jrist has joined #openstack-ironic | 14:38 | |
| *** jrist has quit IRC | 14:38 | |
| *** jrist has joined #openstack-ironic | 14:38 | |
| openstackgerrit | John Trowbridge proposed openstack/ironic-inspector: Store and expose introspection data https://review.openstack.org/213159 | 14:43 |
| TheJulia | good morning everyone | 14:47 |
| zhenguo | TheJulia: o/ | 14:48 |
| *** alexpilotti has joined #openstack-ironic | 14:49 | |
| zhenguo | lucasagomes: can you help to review https://review.openstack.org/#/c/204088/ again please? have addressed your comments | 14:49 |
| *** alexpilotti has quit IRC | 14:53 | |
| mrda | morning Ironic | 14:54 |
| *** rameshg871 has joined #openstack-ironic | 14:55 | |
| *** mtanino has joined #openstack-ironic | 14:55 | |
| *** foexle has quit IRC | 14:56 | |
| *** rameshg87 has quit IRC | 14:57 | |
| NobodyCam | mornign mrda | 14:57 |
| mrda | hey NobodyCam | 14:58 |
| *** dan_ has joined #openstack-ironic | 14:58 | |
| *** dan_ is now known as Guest34051 | 14:58 | |
| lucasagomes | zhenguo, hi there sure | 14:58 |
| lucasagomes | just gimme few minutes to finish up something here | 14:59 |
| thiagop | morning mrda | 14:59 |
| thiagop | morning TheJulia | 14:59 |
| *** rloo_ has joined #openstack-ironic | 14:59 | |
| *** dprince has quit IRC | 14:59 | |
| *** mbound has quit IRC | 14:59 | |
| *** rcarrillocruz has quit IRC | 14:59 | |
| *** rloo has quit IRC | 14:59 | |
| *** early has quit IRC | 14:59 | |
| *** dan has quit IRC | 14:59 | |
| *** Guest34051 is now known as dan | 14:59 | |
| zhenguo | lucasagomes: thanks :-) | 14:59 |
| mrda | hey thiagop | 15:01 |
| *** alexpilotti has joined #openstack-ironic | 15:01 | |
| NobodyCam | mornign TheJulia | 15:01 |
| *** rameshg871 has quit IRC | 15:02 | |
| *** achanda has joined #openstack-ironic | 15:03 | |
| *** rcarrillocruz has joined #openstack-ironic | 15:03 | |
| *** achanda has quit IRC | 15:05 | |
| *** early has joined #openstack-ironic | 15:06 | |
| *** mgoddard1 has quit IRC | 15:07 | |
| *** amotoki has joined #openstack-ironic | 15:08 | |
| *** mgoddard has joined #openstack-ironic | 15:08 | |
| *** dprince has joined #openstack-ironic | 15:12 | |
| *** dprince has quit IRC | 15:16 | |
| *** alexpilotti has quit IRC | 15:25 | |
| *** puranamr has joined #openstack-ironic | 15:26 | |
| *** maurosr has quit IRC | 15:29 | |
| *** zsmithnyc has quit IRC | 15:36 | |
| *** maurosr has joined #openstack-ironic | 15:36 | |
| *** natorious is now known as zz_natorious | 15:37 | |
| mrda | hey, so I'm having a couple of uses with localboot and ironic and amt. If I create the disk image with "disk-image-create ubuntu baremetal dhcp-all-interfaces local-config grub2 -o imagename", and set "ironic node-update my-nuc add instance_info/capabilities='{"boot_option": "local"}'", I'm still getting grub couldn't install errors. Any ideas? | 15:37 |
| *** zz_natorious is now known as natorious | 15:37 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic-specs: OneView Driver for Ironic https://review.openstack.org/187762 | 15:39 |
| thiagop | lucasagomes: concerns addressed, thank you very much for noting^ | 15:39 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic-python-agent: Add base for IPA functional tests https://review.openstack.org/213249 | 15:39 |
| lucasagomes | thiagop, ta much! give me some mins and I will re-review it | 15:39 |
| sambetts | mrda: I was having issues with disk image builder including grub2 correctly in the image, so I ended up using the triple-o localboot element instead of the grub2 one | 15:41 |
| mrda | oh | 15:42 |
| mrda | thanks sambetts (that rings a bell from when I did this last time) | 15:42 |
| sambetts | mrda: http://paste.openstack.org/show/420647/ this is how I did it | 15:42 |
| mrda | sambetts: that'll teach me for not writing down how I got it working two weeks ago :) | 15:43 |
| mrda | sambetts: thanks - you're great! | 15:44 |
| sambetts | mrda: I ran into exactly this problem early last week so its fresh in my mind :-P | 15:44 |
| JoshNang | NobodyCam: good morning! | 15:46 |
| *** mgoddard1 has joined #openstack-ironic | 15:47 | |
| NobodyCam | morning JoshNang | 15:47 |
| thiagop | good morning JoshNang | 15:47 |
| mrda | sambetts: I had it working two weeks before the trip to the USA. But I have a leaky brain. | 15:48 |
| *** mgoddard has quit IRC | 15:48 | |
| *** alexpilotti has joined #openstack-ironic | 15:52 | |
| lucasagomes | JoshNang, mrda morning | 15:52 |
| *** mestery has joined #openstack-ironic | 15:54 | |
| jlvillal | mariojv: ping | 15:54 |
| mariojv | hi \o | 15:54 |
| jlvillal | mariojv: Never mind :) I was thinking code was adding a string and int together. | 15:55 |
| jlvillal | But I see it is not :) | 15:55 |
| mariojv | nope - env vars are always strings | 15:55 |
| jlvillal | mariojv: Personally I'm a fan of 'foo {}'.format(var) vs 'foo ' + var. But both work. Thanks. | 15:56 |
| mariojv | np | 15:56 |
| *** mgoddard1 has quit IRC | 15:56 | |
| jlvillal | mariojv: Though I think I may have found problem... Sorry | 15:56 |
| *** Marga_ has quit IRC | 15:57 | |
| *** mgoddard has joined #openstack-ironic | 15:57 | |
| cinerama | morning ironic! | 15:58 |
| thiagop | good morning cinerama | 15:58 |
| cinerama | hi thiagop! | 15:58 |
| mrda | hey lucasagomes, mariojv, JoshNang, jlvillal, cinerama and anyone I missed :) | 15:58 |
| mariojv | morning mrda \o | 15:58 |
| mariojv | morning cinerama | 15:59 |
| jlvillal | mariojv: Updated | 15:59 |
| mariojv | thanks, taking a look | 15:59 |
| cinerama | hi mariojv! | 15:59 |
| jlvillal | mrda, cinerama, JoshNang, mariojv: Good morning | 16:00 |
| *** lsmola has quit IRC | 16:00 | |
| mariojv | jlvillal: i think your recommended code does the same thing as what's already there | 16:02 |
| jlvillal | mariojv: Not exactly. Using the format operation it takes care of converting the integer into a string. | 16:03 |
| jlvillal | mariojv: 'foo ' + 2 will fail. 'foo {}'.format(2) works | 16:03 |
| mariojv | ah, right - i'll go ahead and switch everything over to .format in next patch set then | 16:04 |
| jlvillal | mariojv: Cool. format usually makes them easier, I think. | 16:04 |
| mariojv | it's cleaner than 'a' + 'b', i was just being lazy :) | 16:04 |
| jlvillal | mariojv: The sign of a true programmer ;) Laziness! | 16:04 |
| jroll | we tend to use %s in ironic, fwiw | 16:04 |
| jlvillal | mariojv: I waste so much time doing scripts because I'm too lazy to repeat stuff | 16:05 |
| jlvillal | jroll: mariojv: %s works good too. Since we use Python 2.7, I slightly prefer format. But both I think are better than doing '+' | 16:05 |
| jroll | %s works in 3.4, though | 16:06 |
| jroll | so we've been just rolling with it | 16:06 |
| jlvillal | In Python 2.6 it was a pain with '{0}' being required :( | 16:06 |
| mariojv | jroll: jlvillal: .format should also work in 3.4 | 16:08 |
| thiagop | Thanks for +2'd the spec lucasagomes | 16:08 |
| jlvillal | Yep | 16:08 |
| jroll | mariojv: of course. but we currently %s all the things. let's keep doing that | 16:08 |
| lucasagomes | thiagop, thanks for writing it! | 16:08 |
| jroll | consistency and such | 16:08 |
| mariojv | will do | 16:08 |
| *** jistr has quit IRC | 16:08 | |
| thiagop | anybody else could take a look on this? https://review.openstack.org/#/c/187762/ | 16:09 |
| lucasagomes | we use %s only if it's 1 argument | 16:09 |
| lucasagomes | for 2 or more we use format | 16:09 |
| lucasagomes | (because the translation team may need to alter the order of things) | 16:09 |
| jroll | oh do we? | 16:09 |
| lucasagomes | yeah | 16:09 |
| lucasagomes | if not we should | 16:09 |
| jroll | I don't think we do | 16:09 |
| jroll | we use %(thing)s | 16:10 |
| lucasagomes | right but that's fine | 16:10 |
| jroll | rather than {thing} | 16:10 |
| lucasagomes | cause you can move %(thing)s around | 16:10 |
| jroll | yeah totally | 16:10 |
| jroll | we were talking about 'foo {}'.format(var) | 16:10 |
| *** praneshp_ has joined #openstack-ironic | 16:10 | |
| jroll | or in your case 'foo {var}'.format(var=var) | 16:10 |
| *** fultonj has joined #openstack-ironic | 16:10 | |
| lucasagomes | oh, right yeah I see that in IPA code | 16:10 |
| lucasagomes | jroll, yeah | 16:10 |
| jroll | oh dang | 16:10 |
| jroll | heh | 16:11 |
| jroll | well then, hooray for consistency. | 16:11 |
| *** meghal has joined #openstack-ironic | 16:11 | |
| lucasagomes | lol | 16:11 |
| *** romainh has quit IRC | 16:12 | |
| lucasagomes | fwiw there are guides about it | 16:12 |
| lucasagomes | http://docs.openstack.org/developer/oslo.i18n/guidelines.html#choosing-a-marker-function | 16:12 |
| *** fultonj has quit IRC | 16:12 | |
| *** zsmithnyc has joined #openstack-ironic | 16:12 | |
| jlvillal | lucasagomes: Cool, thanks. /me had no idea :) | 16:12 |
| JoshNang | fwiw, we use .format all over ipa | 16:12 |
| lucasagomes | Ironic follow those ^ | 16:12 |
| *** praneshp has quit IRC | 16:13 | |
| *** praneshp_ is now known as praneshp | 16:13 | |
| *** romcheg has quit IRC | 16:13 | |
| jroll | mariojv: I guess I was wrong, sorry :( | 16:13 |
| JoshNang | and none of ipa is translated | 16:13 |
| mariojv | jroll: it's all good, also JoshNang %s is used a lot in ipa too | 16:14 |
| JoshNang | heh yep :( | 16:14 |
| jroll | lol | 16:14 |
| JoshNang | though, i think some IPA errors could wind up in last_error..maybe we should be translating | 16:14 |
| TheJulia | mrda: the vm element works as well | 16:14 |
| mariojv | in any case the strings we're talking about have only 1 substitution, so %s is fine (and more compact), going to stick with that | 16:15 |
| mrda | TheJulia: thanks | 16:15 |
| JoshNang | (though last time i checked, ironic had a total of one string translated into like 5 languages, so maybe it doesn't matter if we translate ipa or not) | 16:16 |
| JoshNang | mariojv: ++ | 16:16 |
| jlvillal | JoshNang: There is a little more translation now. I think lucasagomes did some translation into Portuguese :) | 16:17 |
| lucasagomes | jlvillal, yeah like 2% or something | 16:17 |
| lucasagomes | or not even that hah | 16:17 |
| jlvillal | :) | 16:17 |
| JoshNang | oh, that's awesome! | 16:18 |
| lucasagomes | JoshNang, do we have transifex setup for IPA? | 16:19 |
| jroll | we don't translate IPA | 16:20 |
| JoshNang | lucasagomes: no, i submitted a patch to add it a long time ago, but we decided since it is never user facing it wasn't needed | 16:20 |
| jroll | made that decision forever ago for some reason | 16:20 |
| *** jcoufal has quit IRC | 16:20 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic-python-agent: Add base for IPA functional tests https://review.openstack.org/213249 | 16:20 |
| lucasagomes | JoshNang, fair enuff | 16:21 |
| *** alexpilotti has quit IRC | 16:21 | |
| lucasagomes | jroll, yeah makes sense | 16:21 |
| * lucasagomes was looking at the translation things... it's really hard I don't know which terms we use in portuguese for certain stuff | 16:27 | |
| *** romcheg has joined #openstack-ironic | 16:27 | |
| TheJulia | lifeless: jroll: From my perspective it really is what would shade do. Shade is more of a "give me x" model, and I think a similar model would be more friendly to an advanced user of ironic (sure, someone could write directly to the DB... but.. ugh), if a user could post to ironic and say "I want to create a node in y state, and here is everything ironic needs to know". Granted, that means a us | 16:31 |
| TheJulia | er can choose to bypass a workflow, but that workflow may not be required in their use case, and maybe that is a much more advanced use option that we should consider. Granted, I may not be sane. :) | 16:31 |
| jroll | TheJulia: yeah, I tend to think "give me an available node" should make shade enroll a thing and move it to available | 16:32 |
| *** trown is now known as trown|lunch | 16:33 | |
| TheJulia | jroll: which is what my proposed patch does, so it does provide a nice conduit or mechanism to at least hide the workflow differences and make things just magically, happen | 16:34 |
| *** Nisha has joined #openstack-ironic | 16:35 | |
| jroll | TheJulia: which patch? | 16:35 |
| TheJulia | 212070, shrews pointed two things I need to go back and dig into further. The downside of working on shade is I have to delve into "what does it really do" quite a bit. | 16:36 |
| jlvillal | mariojv: +1 from me :) Thanks for the patch. | 16:36 |
| *** romcheg1 has joined #openstack-ironic | 16:36 | |
| mariojv | thanks! | 16:36 |
| *** romcheg has quit IRC | 16:37 | |
| jroll | TheJulia: nice, good stuff | 16:39 |
| *** dprince has joined #openstack-ironic | 16:40 | |
| *** natorious is now known as zz_natorious | 16:42 | |
| *** krtaylor has quit IRC | 16:43 | |
| *** alexpilotti has joined #openstack-ironic | 16:48 | |
| *** praneshp has quit IRC | 16:48 | |
| *** liliars_ has quit IRC | 16:50 | |
| *** davideagnello has joined #openstack-ironic | 16:51 | |
| *** krtaylor has joined #openstack-ironic | 16:56 | |
| krtaylor | mariojv, can you cange the topic back to ipa-functional on 213249, it reverted back to (detached | 16:58 |
| *** ifarkas has quit IRC | 16:58 | |
| mariojv | krtaylor: oops, will do. that's just derived from the branch name right? | 17:00 |
| rloo_ | hi everyone | 17:00 |
| krtaylor | mariojv, yeah, git review without the -t | 17:01 |
| krtaylor | mariojv, you should be able to change it in the gui | 17:01 |
| *** rloo_ has left #openstack-ironic | 17:01 | |
| *** rloo_ has joined #openstack-ironic | 17:01 | |
| jlvillal | lucasagomes: If you are able to review this Nova patch: https://review.openstack.org/#/c/206736/ It would be appreciated. Thanks! (mrda) | 17:03 |
| mariojv | krtaylor: i think JoshNang is the only one who can actually change in the gui, since he's the change owner. tried changing it by switching to a new branch, but it won't let me run git review without new changes besides just the branch change | 17:04 |
| *** penick has joined #openstack-ironic | 17:04 | |
| thiagop | morning rloo_ | 17:04 |
| *** rloo_ is now known as rloo | 17:05 | |
| jlvillal | krtaylor: mariojv: I'm not sure if the topic really matters. I never use the topic since I just do a 'git push origin HEAD:refs/for/master' | 17:05 |
| *** zhenguo has quit IRC | 17:05 | |
| krtaylor | jlvillal, you don't use git review? | 17:05 |
| jlvillal | krtaylor: No. | 17:06 |
| jroll | O.o | 17:06 |
| krtaylor | but no, it doesn't REALLY matter | 17:06 |
| jlvillal | Maybe I'm a glutton for punishment :) I setup a commit-hook to create the change-id and then do a a push to refs/for/master | 17:06 |
| *** penick_ has joined #openstack-ironic | 17:07 | |
| *** mestery has quit IRC | 17:07 | |
| jroll | that's exactly what git-review does for you | 17:07 |
| jroll | :P | 17:07 |
| jlvillal | jroll: :) As I said I'm a glutton for punishment! | 17:07 |
| jroll | clearly | 17:07 |
| jlvillal | jroll: I was doing that when working on Android and then when I moved to OpenStack I already had the commit-hook setup to be copied to all git clones. | 17:08 |
| * jlvillal should try git-review some day | 17:08 | |
| krtaylor | +1 | 17:08 |
| lucasagomes | jlvillal, will do | 17:08 |
| JoshNang | krtaylor: fixed the topic | 17:08 |
| jlvillal | lucasagomes: Thanks! | 17:08 |
| krtaylor | jlvillal, I felt like I knew you from somewhere, maybe it was Android | 17:08 |
| *** max_lobur has quit IRC | 17:09 | |
| jlvillal | krtaylor: Maybe. I work at Intel. I have pushed a few patches upstream to Google. But for infra stuff. A Gerrit patch and a few patches to git-repo | 17:09 |
| lucasagomes | jlvillal, jroll oh, I'm actually good with that. So the consensus is add a config option or new filter later right? | 17:09 |
| *** penick has quit IRC | 17:10 | |
| *** penick_ is now known as penick | 17:10 | |
| lucasagomes | just let's make sure we open the bug once that code is merged | 17:10 |
| lucasagomes | so we track the problem | 17:10 |
| jroll | +1 | 17:11 |
| lucasagomes | cool, ok +1'd that patch in nova | 17:14 |
| lucasagomes | folks I will call it a day | 17:14 |
| lucasagomes | have a great night everyone! | 17:14 |
| jlvillal | lucasagomes: Thanks lucas! Have a great evening :) | 17:14 |
| lucasagomes | you too, enjoy the day! | 17:14 |
| jroll | night lucas :) | 17:14 |
| *** lucasagomes is now known as lucas-dinner | 17:14 | |
| lucas-dinner | see ya | 17:14 |
| jlvillal | jroll: I'm pinged bauzas this morning to recheck the patch. Sounded like he was kind of busy yesterday. | 17:15 |
| jlvillal | s/I'm/I/ | 17:15 |
| jroll | jlvillal: cool | 17:15 |
| jroll | thanks | 17:15 |
| jlvillal | np | 17:15 |
| lucas-dinner | oh last request... this series of patches starting with https://review.openstack.org/#/c/213240 is adding timeout for nodes doing cleaning | 17:16 |
| lucas-dinner | if you have a time pelase take a look :-) | 17:16 |
| lucas-dinner | thanks | 17:16 |
| *** alexpilotti has quit IRC | 17:16 | |
| jlvillal | Hmmm. Intel announces a Xeon chip for laptops/notebooks. Cool, Lenovo will have a laptop with 64GB of memory. Wonder how huge it will be. | 17:16 |
| BadCub | hmmm.. The zBook has a Xeon chip in it | 17:17 |
| BadCub | I wonder if ^^ is a new design to run cooler and not burn the skin off the user lol | 17:18 |
| *** ndipanov has joined #openstack-ironic | 17:18 | |
| jlvillal | BadCub: Maybe :) | 17:19 |
| *** boris-42 has quit IRC | 17:20 | |
| BadCub | jlvillal: that would be awesome, because yeah... I like keeping my skin in-tact | 17:20 |
| *** absubram has joined #openstack-ironic | 17:20 | |
| TheJulia | BadCub: just get a fan pad, leave it on that around the clock :) | 17:20 |
| BadCub | TheJulia: I had one. It works if you leave it on a desk, but on the lap, No Bueno!!!! | 17:21 |
| TheJulia | jroll: out of curiosity, is last_error wiped upon success? | 17:24 |
| jroll | TheJulia: typically, yes, afaik | 17:25 |
| * TheJulia ponders | 17:25 | |
| *** achanda has joined #openstack-ironic | 17:27 | |
| TheJulia | jroll: I guess if last_error appears, then it would be an easy way to know something has/is going wrong, I do like that idea a lot now that I think about it | 17:28 |
| rloo | darn, i fired off an email and forgot to put '[ironic]' in the subject. i guess the only way is to reply/change the subject line? :-( | 17:30 |
| * devananda waves from ops midcycle | 17:31 | |
| * rloo asks devananda to wave for us ironickers, to the ops folks and to ask them for feedback :-) | 17:32 | |
| *** trown|lunch is now known as trown|mtg | 17:33 | |
| devananda | jroll, TheJulia, penick: I think there's a very strong case for allowing a node to be created in any of the stable states (enroll, manageable, available, active) | 17:33 |
| TheJulia | devananda: agreed | 17:33 |
| jroll | active. | 17:34 |
| * jroll x.x | 17:34 | |
| penick | agreed, we definitely need that when we start forklifting our infra into ironic | 17:34 |
| devananda | jroll: because that ^ | 17:34 |
| *** MattMan has quit IRC | 17:34 | |
| jroll | right | 17:34 |
| TheJulia | yup | 17:34 |
| jroll | I understand the use case | 17:34 |
| jroll | I'm not sure if that should be a first-class thing or a somewhat-supported hack | 17:34 |
| jroll | need to think about where that could go wrong or what data we'd require | 17:35 |
| TheJulia | I was thinking that would be the best case, I want ironic to own this node, but it is in use and I don't want it to get recycled right now. | 17:35 |
| TheJulia | might be a good case for a "We know what we're doing, just trust us" option | 17:36 |
| penick | To hack this into our (janky, hacky, never do this, forget it ever happened) nova bm stuff we made a lobotamized version of instance create in nova. Where it’d run through all of the steps, without actually imacting any external systems. It’s ugly, hacky, and really slow. | 17:37 |
| *** romcheg has joined #openstack-ironic | 17:38 | |
| penick | being able to just say “take this node, here’s all the state info, trust me.” would have made things significantly easier | 17:38 |
| jroll | totally | 17:38 |
| jroll | just... needs thought | 17:38 |
| penick | jroll: are you here at the ops mid cycle today? | 17:38 |
| penick | sure | 17:38 |
| jroll | I don't love many entrypoints into the state machine | 17:38 |
| jroll | no :( | 17:38 |
| TheJulia | absoloutely needs thought | 17:38 |
| TheJulia | jroll: :( | 17:39 |
| penick | boo. Get in car. Drive here. Be here. | 17:39 |
| jroll | would be 2 hour drive for me in traffic | 17:39 |
| TheJulia | penick: +1 | 17:39 |
| penick | You can sleep on my couch. | 17:39 |
| jroll | lol | 17:39 |
| jroll | there's also the part where I share a car with my gf | 17:39 |
| jroll | who kinda needs it :P | 17:39 |
| TheJulia | not on the couch in my hotel room though, I need "my" space | 17:39 |
| *** romcheg1 has quit IRC | 17:40 | |
| TheJulia | eh, caltrain | 17:40 |
| penick | ^ +1 | 17:40 |
| jroll | well | 17:40 |
| jroll | an hour to get to caltrain | 17:40 |
| TheJulia | :( | 17:40 |
| jroll | and then another hour on the train | 17:40 |
| jroll | I'm in deep east bay :P | 17:40 |
| penick | Zipcar! | 17:41 |
| penick | Wait, east bay. Steal a car. | 17:41 |
| TheJulia | lol | 17:41 |
| BadCub | would stealing a car be considered baremetal provisioning? | 17:41 |
| penick | Probably | 17:41 |
| penick | it’s a very efficient use of baremetal | 17:42 |
| TheJulia | BadCub: I think so, but only if the car gets painted in the process | 17:42 |
| jroll | lol | 17:42 |
| penick | and options changed | 17:42 |
| BadCub | jroll: could put on a PixieBoots costume and do it! | 17:42 |
| jroll | oh god | 17:42 |
| penick | who knew that grand theft auto pioneered baremetal provisioning | 17:42 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands https://review.openstack.org/172517 | 17:42 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Introduce openstackclient plugin https://review.openstack.org/171672 | 17:42 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create https://review.openstack.org/172461 | 17:42 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add provision state to set command https://review.openstack.org/206119 | 17:42 |
| *** alexpilotti has joined #openstack-ironic | 17:43 | |
| * TheJulia despritly tries to keep from bursting out in laughter | 17:43 | |
| thiagop | LOL | 17:43 |
| BadCub | hehehehe.. the news clips would be awesome! | 17:43 |
| penick | anyone know Dmitry’s IRC nick? | 17:43 |
| jroll | dtantsur|pto19Au: ^ | 17:43 |
| penick | want to keep argu^H^H^H^H discussing https://review.openstack.org/#/c/212206/ | 17:44 |
| penick | cool, danke :) | 17:44 |
| jroll | in general, dtantsur | 17:44 |
| jlvillal | penick: Wait until tomorrow ;) | 17:44 |
| jroll | he's PTO today and in general afk by 10am | 17:44 |
| penick | ah, good to know. thanks :) | 17:44 |
| jroll | yep | 17:44 |
| krotscheck | BadCub: Thanks for throwing me under the bus there :-P | 17:53 |
| rloo | penick, if i understand, we want to import nodes that are active (have instances). even if we add an API to do so in ironic, there's still the issue of hacking nova? | 17:53 |
| sinval | hey ironicers o/ | 17:54 |
| sinval | when you have some time, please take a look at: https://review.openstack.org/#/c/187762/ | 17:55 |
| penick | yeah, some of that will still exist. But we’ll fix some of the problems we have now | 17:55 |
| sinval | we already received some comments and some things evolved, i think the spec is more mature now, thank you guys | 17:55 |
| rloo | penick: ok, devananda wanted me to vote the way you wanted to vote :-) | 17:55 |
| penick | for one, there’s a race condition we have now, where we add a node, then do the fake boot which then tries to grab that node. If someone sends a specific enough boot request during that time they could grab it | 17:55 |
| rloo | penick: makes sense. | 17:57 |
| *** johnthetubaguy has joined #openstack-ironic | 17:58 | |
| openstackgerrit | James Penick proposed openstack/ironic-specs: No IPA to conductor communication https://review.openstack.org/212206 | 17:58 |
| penick | (ignore that, only updated commit message) | 17:58 |
| johnthetubaguy | hey, if I could get some ironic eyes on this nova patch, that would be awesome: https://review.openstack.org/#/c/207236 | 17:58 |
| johnthetubaguy | its about the addition of the cleanwait state | 17:58 |
| rloo | penick: yeah, I'm ignoring until you and dmitry are friends :) | 17:59 |
| jlvillal | johnthetubaguy: I will ping our guys. Thanks. | 17:59 |
| jlvillal | jroll: ^^ If you have time. | 17:59 |
| rloo | johnthetubaguy: I'd look at it but I added it. jroll? ^^ I discussed with lucas-dinner so he knows about it. sorry, i forgot to ask the ironic folks to look at it. | 17:59 |
| johnthetubaguy | cool thanks, I was just going to PM jroll to that | 17:59 |
| jlvillal | rloo: If you can let mrda and I know about patches to Nova that would be great :) | 18:00 |
| rloo | jlvillal: there is a bug associated with it so that's why i didn't mention it. thought you guys were looking at the bugs. | 18:00 |
| jlvillal | rloo: So we can add it to the tracking etherpad | 18:00 |
| johnthetubaguy | it would be awsome if we can get a list of the most important patches for liberty in here: https://etherpad.openstack.org/p/liberty-nova-priorities-tracking | 18:00 |
| jlvillal | rloo: We do once a week. | 18:00 |
| jlvillal | Today is the day :) | 18:00 |
| rloo | jlvillal: well, it wasn't urgent so I didn't want to use my maybe one favour for that :) | 18:01 |
| johnthetubaguy | ah, I see them now | 18:01 |
| jlvillal | rloo: I think you have way more than one! :) | 18:01 |
| jlvillal | johnthetubaguy: We (mrda and I) try to keep the etherpad up to date. | 18:01 |
| rloo | JoshNang can also review that, he knows about cleaning. | 18:01 |
| *** rohit404 has joined #openstack-ironic | 18:02 | |
| BadCub | krots | 18:02 |
| BadCub | ugh | 18:02 |
| JoshNang | rloo: just did! | 18:02 |
| BadCub | krotscheck: huh? | 18:02 |
| krotscheck | BadCub: "I would like Michael to provide absolute clarity on this to ensure that I am not interpreting the two projects incorrectly." | 18:02 |
| rloo | thx JoshNang! | 18:02 |
| krotscheck | Absolute clarity? | 18:02 |
| krotscheck | :-P | 18:02 |
| BadCub | sorry if the wording was incorrect. I was attempting to get these folks to go back to the "original" conversation that you and I initiated | 18:03 |
| johnthetubaguy | so there is this one about adding a new vnic_type, https://review.openstack.org/#/c/213264 not really sure how that works with the current code, it looks a lot like a feature... | 18:03 |
| NobodyCam | krotscheck: https://s-media-cache-ak0.pinimg.com/736x/f0/cc/d7/f0ccd7281a10291bf2ccb116a67fda85.jpg | 18:03 |
| BadCub | No matter how I tried to explain it, they just don't grasp it. | 18:03 |
| jlvillal | johnthetubaguy: jroll and sukhdev know more about the vnic_type. | 18:04 |
| rloo | johnthetubaguy: jroll has been working with sukhdev on the nova/neutron/ironic stuff | 18:04 |
| krotscheck | oooOOOooooo | 18:04 |
| rloo | johnthetubaguy: (whether they will say it is a feature or a bug is a different question) | 18:04 |
| krotscheck | BadCub: No worries. Rae killed the thread, thankfully. I'm talking with her, Piet, and seattle UX on a separate thread now so we can actually get things done. | 18:05 |
| BadCub | krotscheck: I just kinda ran out of ways of saying "yes, we are trying to work with you on the same thing".... | 18:05 |
| BadCub | Yeah, I saw that. thankfully... | 18:06 |
| *** chlong has quit IRC | 18:06 | |
| johnthetubaguy | I mean, if that is all we really need, then sure, we could add that, but that does seem right somehow | 18:06 |
| jroll | johnthetubaguy: in theory it's a bug that neutron supports that vnic_type and nova doesn't, but realistically it's a feature | 18:06 |
| johnthetubaguy | jroll: is that literally all we need to make neutron work? | 18:06 |
| krotscheck | BadCub: I'll forward you my latest internal response. | 18:06 |
| jroll | johnthetubaguy: no :( | 18:06 |
| BadCub | krotscheck: awesome, thank you :) | 18:07 |
| johnthetubaguy | jroll: I was expecting some kind of neturon vif driver in nova? | 18:07 |
| jroll | johnthetubaguy: also needs https://review.openstack.org/#/c/194413/ and a thing to push the right vnic type, maybe that's a vif driver, dunno | 18:07 |
| *** amotoki has quit IRC | 18:07 | |
| *** alexpilotti has quit IRC | 18:08 | |
| johnthetubaguy | jroll: I know you are going to hate me for the review vote on that, but we really need to get all these bugs and priority stuff reviewed :( | 18:09 |
| *** chlong has joined #openstack-ironic | 18:09 | |
| jlvillal | rloo: I'm wondering if the gate is broken. For gate-tempest-dsvm-ironic-pxe_ssh-postgres | 18:11 |
| *** openstackgerrit_ has quit IRC | 18:11 | |
| rloo | jlvillal: is that failing for all patches? | 18:11 |
| jlvillal | rloo: At least two patches: https://review.openstack.org/#/c/209605/ has been rechecked like 3-4 times. | 18:12 |
| *** SplasPood has left #openstack-ironic | 18:12 | |
| jlvillal | rloo: And I just noticed ramesh's patch that you did the recheck on. | 18:12 |
| rloo | jlvillal: might want to ping infra on that then. | 18:13 |
| jlvillal | rloo: Okay, I'll ask them. Thanks. | 18:13 |
| jroll | johnthetubaguy: I completely understand, and honestly it doesn't set us back at all so it's fine | 18:14 |
| jroll | johnthetubaguy: but you know that's the most trivial patch ever :/ | 18:14 |
| rloo | jlvillal: thx for offering! | 18:14 |
| johnthetubaguy | jroll: yeah, I left the VNIC one alone, its just the other one that worries me really | 18:15 |
| jroll | oh | 18:15 |
| jlvillal | rloo: I dropped a note in #openstack-infra | 18:15 |
| jroll | johnthetubaguy: yeah the host_id thing should be bumped to M, I would take it if you allowed it for L but if I were you I would say no | 18:15 |
| johnthetubaguy | jroll: yeah | 18:17 |
| jroll | johnthetubaguy: totally expected a -2 there, I would have done it myself if I could :P | 18:17 |
| jroll | johnthetubaguy: also https://review.openstack.org/#/c/207236/ should totally land, though it only has racker votes on it :/ | 18:20 |
| jroll | (patch is from red hat though) | 18:20 |
| *** sambetts has quit IRC | 18:21 | |
| johnthetubaguy | jroll: its been up long enough, I don't call that a hard rule as such | 18:21 |
| johnthetubaguy | its the stuff that gets up and merged in a day all from the same company that is way too dodgy | 18:21 |
| jroll | right, just pointing it out | 18:22 |
| johnthetubaguy | yeah, thats all good :) | 18:22 |
| *** sambetts has joined #openstack-ironic | 18:23 | |
| *** caronn has joined #openstack-ironic | 18:25 | |
| caronn | http://bit.ly/1J20MjT | 18:25 |
| *** alexpilotti has joined #openstack-ironic | 18:26 | |
| jroll | ^ that link is super shady, do not click | 18:26 |
| *** Shrews has left #openstack-ironic | 18:27 | |
| jroll | resolves to http:// 128.199.217.90 :8082/ 7JCWhLxBQs8XsH | 18:27 |
| jroll | spaces to make it unclickable :) | 18:27 |
| TheJulia | sweet | 18:27 |
| jroll | devananda: you has ops yes? | 18:27 |
| thiagop | spam, bacon, spam | 18:27 |
| devananda | jroll: i has in ops room | 18:27 |
| caronn | Alba naked pics on this site | 18:28 |
| jroll | devananda: I mean ops to kick caronn | 18:28 |
| *** athomas has quit IRC | 18:28 | |
| devananda | oh | 18:28 |
| caronn | (Jessica) | 18:28 |
| devananda | yes, one sec | 18:28 |
| *** ChanServ sets mode: +o devananda | 18:28 | |
| NobodyCam | doh | 18:28 |
| *** caronn was kicked by devananda (caronn) | 18:28 | |
| NobodyCam | ya | 18:28 |
| NobodyCam | Ty devananda | 18:29 |
| rloo | what the ... would be nice if devananda had more powers than just kicking that person out | 18:29 |
| devananda | rloo: what add'l powers do you want me to have? :) | 18:29 |
| BadCub | wow, nudie pics spam. haven't seen that in a while | 18:29 |
| jlvillal | jroll: I was suspicious so I used 'wget' | 18:30 |
| rloo | devananda: guess i shouldn't say in public | 18:30 |
| jlvillal | Maybe 'ban' powers :) | 18:30 |
| *** alexpilotti has quit IRC | 18:30 | |
| jlvillal | Not sure if they can reconnect or not after the kick | 18:30 |
| rloo | jlvillal: no, was thinking of more than that. think physical damage | 18:30 |
| jroll | jlvillal: add a + to the end of bit.ly links to see what it is | 18:30 |
| jlvillal | LOL | 18:30 |
| jlvillal | jroll: Ah, nice tip. Thanks. | 18:30 |
| devananda | i can ban too | 18:30 |
| jroll | thanks for kick devananda | 18:30 |
| rloo | ban would be good, but does it just work on the nick? | 18:31 |
| jroll | /shrug | 18:31 |
| jroll | can ban if it becomes a problem | 18:31 |
| jroll | that's a digital ocean IP fwiw | 18:31 |
| rloo | jroll: as far as I'm concerned, once is enough to be a problem | 18:31 |
| TheJulia | ban can be just nick, or nick + ident + address, or just ident, or just address | 18:31 |
| * TheJulia knows WAY too much about IRC | 18:32 | |
| * jroll too | 18:32 | |
| jroll | if they don't rejoin within 30 seconds, they probably won't rejoin at all | 18:32 |
| rloo | I say full power ban! | 18:32 |
| * jroll adds BOFH to his notes about rloo | 18:33 | |
| jroll | :P | 18:33 |
| jlvillal | hehe. /me has probably been a BOFH in the past | 18:34 |
| rloo | jroll: :D | 18:34 |
| jroll | haven't we all though :) | 18:34 |
| * TheJulia takes off the former IRC operator/has mucked with ircd code hat and hides it | 18:34 | |
| *** athomas has joined #openstack-ironic | 18:34 | |
| rloo | jroll: but i would only do it for good reason! that's why no one should make me a dictator I guess. | 18:34 |
| jroll | devananda: ironic 5.0.0? | 18:35 |
| * jlvillal goes to find 2nd cup of coffee | 18:35 | |
| jroll | devananda: do we want to bug sweep first etc? | 18:35 |
| *** praneshp has joined #openstack-ironic | 18:37 | |
| *** alexpilotti has joined #openstack-ironic | 18:37 | |
| devananda | 4.0.0 because we've had 3 official releases so far | 18:38 |
| devananda | I, J, K | 18:38 |
| thiagop | b[atm]an powers to devananda | 18:39 |
| jroll | er, ok | 18:39 |
| rloo | jroll, devananda, NobodyCam, etc - if I want keyring support in ironicclient -- is a spec needed or is a bug sufficient? | 18:40 |
| jroll | rloo: is there prior art on how to do it? | 18:40 |
| devananda | rloo: define keyring? | 18:40 |
| rloo | jroll: oh, ala novaclient | 18:40 |
| jroll | rloo: I'm fine without a spec then | 18:41 |
| jroll | devananda: creds storage in OS keyring | 18:41 |
| jroll | OS being operating system for the first time ever here | 18:41 |
| NobodyCam | jroll: lol | 18:41 |
| *** puranamr has quit IRC | 18:41 | |
| jroll | :) | 18:42 |
| rloo | devananda: yeah, that ^^ or whatever novaclient does, i don't recall the exact details | 18:42 |
| *** pelix has quit IRC | 18:42 | |
| rloo | we don't yet have any specs for the client only, even though i view this as a feature. so thought i'd ask. | 18:42 |
| jroll | rloo: now I'm wondering if keyring is actually in novaclient, I don't remember that being a thing | 18:43 |
| jroll | (it is in supernova, though) | 18:43 |
| jroll | oh, it is, cool | 18:44 |
| rloo | jroll: what is supernova? | 18:44 |
| jlvillal | A song from Oasis? ;) | 18:45 |
| jroll | rloo: a wrapper around nova to use it with multiple sets of credentials/environments | 18:45 |
| jroll | rloo: https://github.com/major/supernova | 18:45 |
| jroll | it's a lifesaver | 18:45 |
| jroll | better link https://supernova.readthedocs.org/en/latest/ | 18:45 |
| rloo | jroll: oh, didn't know that. I think Yahoo uses novaclient with keyring. | 18:45 |
| jroll | rloo: yeah, I like supernova given I have approximately 34762589 nova installations I might talk to. | 18:46 |
| jroll | maybe exaggerating, but ya know. | 18:46 |
| rloo | jroll: did rackspace do supernova? | 18:46 |
| jroll | rloo: a person from rackspace did, yes | 18:46 |
| jroll | works fine with upstream keystone/nova/etc too | 18:46 |
| rloo | jroll: makes sense :) | 18:47 |
| jroll | should also work with ironic fine (supernova myenv -x ironic node-list, etc) | 18:47 |
| jroll | -x provides the executable | 18:47 |
| rloo | jroll: i suppose if supernova works fine with ironic, then we wouldn't need keyring support in ironicclient. but it would still be useful for those not using supernova. | 18:48 |
| jroll | rloo: yeah, I think keyring in ironicclient is still super useful | 18:48 |
| rloo | jroll: yeah. will open a bug about it etc. maybe fine someone to do the work too :) | 18:49 |
| jlvillal | rloo: As a note, my two comments about broken gate have met with no response in #openstack-infra as of now ... | 18:49 |
| jroll | heh | 18:49 |
| jroll | jlvillal: what's broken about it? | 18:49 |
| rloo | jlvillal: hmm. doesn't seem worth doing a recheck if it always fails. but are some patches passing? | 18:49 |
| jlvillal | jroll: https://review.openstack.org/166521 | 18:49 |
| jroll | looks like they're working on an etherpad incident, fwiw | 18:49 |
| rloo | jroll: if i recall correctly, something about ansible, and 'no hosts found'. | 18:50 |
| jlvillal | jroll: another one: https://review.openstack.org/209605 | 18:50 |
| TheJulia | yeah, ops summit is pushing etherpad to it's limit | 18:50 |
| rloo | jroll: when setting up devstack i think | 18:50 |
| rloo | awesome that ops summit is doing that. (well, sort of awesome.) | 18:50 |
| jroll | I see http://logs.openstack.org/21/166521/18/check/gate-tempest-dsvm-ironic-pxe_ssh-postgres/2742ca4/logs/devstacklog.txt.gz#_2015-08-18_18_01_41_465 | 18:50 |
| jlvillal | rloo: I don't think any patches are passing. I did do a recheck on a patch that had a +1 from Jenkins yesterday. See if it also fails today. | 18:51 |
| jroll | this looks like a neutron bug, just from that | 18:51 |
| TheJulia | (sort of, in a way that is making things difficult since it is impacting discussion.) | 18:51 |
| thiagop | I looken on neutron bugs to see if anybody has filed anything about, with no success so far | 18:51 |
| *** puranamr has joined #openstack-ironic | 18:51 | |
| jroll | same deal | 18:51 |
| jroll | YAY | 18:51 |
| *** Sukhdev has joined #openstack-ironic | 18:52 | |
| jlvillal | jroll: Do we ping neutron team? | 18:52 |
| jroll | jlvillal: on it | 18:52 |
| jlvillal | Awesome! | 18:52 |
| jlvillal | thiagop: You are the lucky one impacted with your patch ;) | 18:53 |
| jlvillal | Just when you get the +2 and +A! | 18:53 |
| * rloo is lousy at figuring out why a test failed | 18:53 | |
| * jlvillal says me too! | 18:53 | |
| thiagop | jlvillal: that happens... | 18:53 |
| thiagop | jlvillal: the most interesting log is here: http://logs.openstack.org/05/209605/8/check/gate-tempest-dsvm-ironic-pxe_ssh-postgres/324ad10/logs/screen-q-svc.txt.gz | 18:56 |
| jroll | sorry, I meant to link that | 18:56 |
| * jlvillal looks | 18:56 | |
| jlvillal | thiagop: jroll: Oh that does interesting. Thanks. | 18:57 |
| thiagop | this commit adds the field expiration to reservations: https://github.com/openstack/neutron/commit/f5344dec5b80e38db8ffea10c944ef59b26ebe48 | 18:58 |
| *** rohit404 has quit IRC | 18:58 | |
| jroll | neutron folks are looking fwiw | 18:58 |
| jroll | dougwig specifically | 18:58 |
| rloo | thx jroll, jlvillal, thiagop. That's why it is good to look into why something fails! | 18:59 |
| jroll | right, no sense in just pinging infra and making them debug our code | 18:59 |
| thiagop | somebody should make a paper on "collective troubleshooting" | 19:00 |
| thiagop | :) | 19:00 |
| *** zz_natorious is now known as natorious | 19:00 | |
| jroll | or just write a book on debugging openstack | 19:00 |
| jroll | you'd make millions | 19:00 |
| rloo | oh, jlvillal, since you pinged infra, maybe let them know they don't need to look into it | 19:00 |
| jlvillal | rloo: Will do | 19:00 |
| rloo | thx jlvillal | 19:00 |
| thiagop | seriously? I've learned a thing or two in the past 1,5 years of working with OS... (and become rich isn't that bad) | 19:01 |
| *** penick has quit IRC | 19:01 | |
| *** romcheg has quit IRC | 19:05 | |
| *** alexpilotti has quit IRC | 19:05 | |
| *** e0ne has joined #openstack-ironic | 19:07 | |
| *** amotoki has joined #openstack-ironic | 19:08 | |
| thiagop | yuriyz: are you there? | 19:09 |
| *** ndipanov has quit IRC | 19:09 | |
| *** romcheg has joined #openstack-ironic | 19:12 | |
| *** amotoki has quit IRC | 19:13 | |
| jroll | fyi: https://review.openstack.org/#/c/214301/ | 19:14 |
| jroll | -1s welcome if we want to do a bug scrub or whatever | 19:14 |
| thiagop | are we releasing this week? | 19:16 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers https://review.openstack.org/198656 | 19:17 |
| jroll | thiagop: trying to | 19:17 |
| thiagop | ^ o/ | 19:17 |
| rloo | jroll: guess i have to read your spec to see if you're following the process you outlined :) | 19:17 |
| jroll | rloo: uh oh | 19:17 |
| jroll | :) | 19:17 |
| rloo | jroll: do we care that we have part of the RAID interface stuff done? | 19:18 |
| rloo | jroll: and part of the boot/deploy interface work done? | 19:19 |
| jroll | rloo: I'm okay with it, I think | 19:19 |
| jroll | could maybe use extra testing, unsure | 19:19 |
| rloo | jroll: what about 'Leading up to all releases, reviewers should honor a “soft freeze” period of a few days to a week'? | 19:20 |
| jroll | rloo: right, like I said -1s welcome | 19:20 |
| rloo | jroll: so personally, i am fine with releasing what we have but thought i'd point that stuff out. | 19:21 |
| jroll | rloo: indeed | 19:21 |
| jroll | maybe devananda will be around and we can chat about it | 19:21 |
| rloo | jroll: also -- ..., i think, not sure, it would be good to send out an email announcing the plan to do this. i don't know whether it makes much difference really except that there was not much communication about it | 19:22 |
| *** achanda has quit IRC | 19:22 | |
| jroll | rloo: yeah, I could email | 19:22 |
| jroll | though we've been trying to figure out how to release for like a week now | 19:22 |
| jroll | talked about it in meetings and such | 19:22 |
| rloo | jroll: yeah, i expected you to have released it last week. i just think we may not be doing a great job informing the masses out there (all 3 of them) that aren't on irc or meetings. | 19:23 |
| jroll | sure | 19:23 |
| rloo | jroll: even if you mention that we're releasing tomorrow :) | 19:23 |
| jroll | yeah | 19:23 |
| jroll | I'll wait for devananda | 19:23 |
| jroll | he'll likely need to rubber stamp that release patch anyway, PTL and all | 19:23 |
| thiagop | jroll: have you seen my ping earlier? Have a slot on your review stack? | 19:30 |
| jroll | thiagop: the oneview spec? | 19:31 |
| thiagop | yep | 19:31 |
| jroll | yeah, it's on my (growing) list | 19:31 |
| jroll | busy week this week for me | 19:31 |
| thiagop | jroll: Ok. I can imagine... | 19:32 |
| *** alexpilotti has joined #openstack-ironic | 19:41 | |
| *** rohit404 has joined #openstack-ironic | 19:41 | |
| *** mestery has joined #openstack-ironic | 19:44 | |
| *** e0ne has quit IRC | 19:44 | |
| *** Marga_ has joined #openstack-ironic | 19:47 | |
| *** Marga_ has quit IRC | 19:52 | |
| *** Marga_ has joined #openstack-ironic | 19:52 | |
| *** alexpilotti has quit IRC | 20:03 | |
| thiagop | jlvillal: nothing yet, right? | 20:06 |
| jroll | someone mind landing this? https://review.openstack.org/#/c/213252/ | 20:06 |
| thiagop | jlvillal: I'm almost proposing a revert... | 20:06 |
| *** mestery has quit IRC | 20:08 | |
| jlvillal | thiagop, I'd vote for a revert :) | 20:09 |
| *** mestery has joined #openstack-ironic | 20:11 | |
| *** rohit404 has quit IRC | 20:11 | |
| *** penick has joined #openstack-ironic | 20:16 | |
| rloo | jroll: maybe NobodyCam? I'm not sure anyone else that can, is around. | 20:17 |
| jroll | yeah, dunno | 20:18 |
| *** boris-42 has joined #openstack-ironic | 20:19 | |
| *** penick has quit IRC | 20:20 | |
| *** puranamr has quit IRC | 20:21 | |
| *** achanda has joined #openstack-ironic | 20:22 | |
| *** achanda has quit IRC | 20:22 | |
| *** achanda has joined #openstack-ironic | 20:23 | |
| *** achanda has quit IRC | 20:28 | |
| *** achanda has joined #openstack-ironic | 20:28 | |
| NobodyCam | I'm here | 20:29 |
| *** alexpilotti has joined #openstack-ironic | 20:30 | |
| NobodyCam | jroll: rloo +a'd | 20:33 |
| jroll | thanks | 20:33 |
| jroll | :) | 20:33 |
| NobodyCam | :) | 20:34 |
| rloo | thx NobodyCam! | 20:34 |
| jlvillal | rloo: jroll: I updated the Whiteboard to say the gate is broken. | 20:35 |
| rloo | jlvillal: thx. No use 'recheck'ing until there is a fix, right? | 20:35 |
| jlvillal | rloo: I don't think so. Seems like it is broken by neutron. | 20:36 |
| jlvillal | Consistent fail from all jobs I have seen. | 20:36 |
| rloo | jlvillal: too bad. Oh well... | 20:36 |
| jlvillal | Yeah :( Only way for us around it would be to make it a non-voting job while we wait for fix. | 20:36 |
| jlvillal | But we should probably be patient :) | 20:37 |
| rloo | jlvillal: don't know if it is worth it. we don't have any pressing deadlines. | 20:37 |
| jlvillal | Agreed | 20:37 |
| *** puranamr has joined #openstack-ironic | 20:39 | |
| *** trown|mtg is now known as trown|outttypeww | 20:45 | |
| *** Sukhdev has quit IRC | 20:52 | |
| *** lucas-dinner has quit IRC | 20:52 | |
| *** derekh has quit IRC | 20:52 | |
| openstackgerrit | Allan Vidal proposed stackforge/pyghmi: Add LED parsing for Lenovo ThinkServers https://review.openstack.org/214345 | 20:54 |
| openstackgerrit | Merged openstack/ironic-specs: Update network provider spec https://review.openstack.org/213252 | 20:55 |
| jlvillal | Nisha: The gate is broken. So recheck won't fix your patch :( | 20:58 |
| Nisha | jlvillal, oh | 20:58 |
| jlvillal | Nisha: https://etherpad.openstack.org/p/IronicWhiteBoard | 20:58 |
| *** mbound has joined #openstack-ironic | 21:00 | |
| *** trown|outttypeww is now known as trown|ptoAug21 | 21:00 | |
| Nisha | jlvillal, thanks | 21:00 |
| jlvillal | You're welcome. Hopefully it will be fixed soon. | 21:00 |
| Nisha | yeah hoping so :) | 21:01 |
| jroll | apparently this will unblock us https://review.openstack.org/#/c/214282/1 | 21:01 |
| jroll | jlvillal: ^ | 21:01 |
| *** Shrews has joined #openstack-ironic | 21:01 | |
| jlvillal | jroll: Great. Hopefully the recheck job passes on it, so it can move to the merge queue :) | 21:01 |
| *** mbound has quit IRC | 21:01 | |
| jroll | yep | 21:01 |
| *** alexpilotti has quit IRC | 21:01 | |
| *** krtaylor has quit IRC | 21:01 | |
| NobodyCam | jenkins -1 that patch? | 21:02 |
| jlvillal | NobodyCam: Yeah, there is a recheck job running now on it | 21:02 |
| NobodyCam | :) | 21:03 |
| jlvillal | NobodyCam: zuul estimates 20 more minutes for the check job to finish. The failing job from before hasn't finished yet when I looked. | 21:03 |
| *** lucas-dinner has joined #openstack-ironic | 21:04 | |
| NobodyCam | ahh :-p | 21:04 |
| thiagop | fingers crossed | 21:04 |
| thiagop | haha | 21:05 |
| NobodyCam | :) | 21:05 |
| thiagop | anybody knows where can I find the docs on how to create user (whole disk) images for agent drivers? | 21:09 |
| *** puranamr has quit IRC | 21:09 | |
| NobodyCam | thiagop: are you using disk image builder? | 21:09 |
| *** amotoki has joined #openstack-ironic | 21:10 | |
| thiagop | yep | 21:10 |
| NobodyCam | just add the "VM" element to you list | 21:10 |
| *** Marga__ has joined #openstack-ironic | 21:10 | |
| gabriel-bezerra | NobodyCam: does it mean that the usual cloud-images work with agent? | 21:11 |
| gabriel-bezerra | such as ubuntu's and fedora's cloud images | 21:11 |
| NobodyCam | the vm element will make whole disk images from dib | 21:11 |
| jroll | NobodyCam: jlvillal devananda neutron is saying that most of the team is afk this week and the fix might be measured in days, not hours | 21:12 |
| jroll | they're wondering, should we temporarily make that job non-voting? | 21:12 |
| jlvillal | jroll: Ouch. | 21:12 |
| jroll | 21:10:37 dougwig | jroll: half of neutron is at linuxcon, half is at the ops meetup, and the rest are here. i'm guessing the fix will be measured in days, not hours. | 21:12 |
| jlvillal | jroll: If days, then I would think should make it non-voting. | 21:12 |
| jlvillal | rloo: ^^^ | 21:13 |
| jroll | I'm worried that we'll never get around to fixing it if we do so | 21:13 |
| NobodyCam | uggh | 21:13 |
| gabriel-bezerra | NobodyCam: I see. Thank you. | 21:13 |
| *** Marga_ has quit IRC | 21:13 | |
| rloo | whoa. what broke that can't be reverted? sigh. | 21:13 |
| thiagop | gabriel-bezerra: yes, it does | 21:13 |
| *** krtaylor has joined #openstack-ironic | 21:14 | |
| jroll | rloo: it apparently can be reverted but not quickly? | 21:15 |
| jroll | idk | 21:15 |
| *** amotoki has quit IRC | 21:15 | |
| gabriel-bezerra | thank you, thiagop | 21:15 |
| jroll | they're getting failures on the reboot | 21:15 |
| jroll | er, revert | 21:15 |
| rloo | jlvillal, jroll: do they know whether there is a bug in the code, or whether the change isn't backwards compatible or whatever. oh well i don't know that i want to know. | 21:18 |
| Nisha | NobodyCam, jroll rloo please review https://review.openstack.org/#/c/198656/ https://review.openstack.org/201904 | 21:18 |
| jroll | rloo: the feature has a bug with postgres. the "revert" stops using the feature while they fix that bug and some other bugs. | 21:18 |
| NobodyCam | Nisha: gate is down atm | 21:19 |
| *** puranamr has joined #openstack-ironic | 21:19 | |
| rloo | jroll: oh. i suppose regardless of which way they go, they need neutron core reviewers. | 21:19 |
| jroll | right | 21:19 |
| Nisha | NobodyCam, may be after that... | 21:19 |
| rloo | i wonder if it only affects ironic | 21:19 |
| NobodyCam | :) | 21:19 |
| *** jjohnson2 has quit IRC | 21:19 | |
| rloo | jroll: btw, wrt our releases, we're going to do a release that coincides with liberty release, right? | 21:20 |
| rloo | jroll: i saw in the spec, mention of mid-sept. | 21:20 |
| jroll | rloo: yeah | 21:21 |
| jroll | something like that | 21:21 |
| rloo | jroll: so am thinking ideally, we'd want all HIGH priority things done by then. | 21:21 |
| jroll | rloo: that's a lot of things :) | 21:21 |
| jroll | you mean from the google doc? | 21:21 |
| rloo | jroll: well, not that much if i exclude the specs that aren't approved yet. yeah, from the google doc. | 21:21 |
| jlvillal | jroll: rloo: I don't think it is a 'real' revert. It is a patch that reverts some things. | 21:21 |
| jlvillal | or changes some things. | 21:22 |
| jlvillal | Or disables some things :) | 21:22 |
| jlvillal | Not a 'git revert xxxxxxxxxxxxxxxxxxx' | 21:22 |
| rloo | jlvillal: well, w/o many neutron core reviewers around... | 21:22 |
| jroll | it removes the use of the feature while leaving the code intact | 21:24 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: WIP: inband discovery in ilo https://review.openstack.org/214357 | 21:24 |
| *** mestery has quit IRC | 21:25 | |
| *** athomas has quit IRC | 21:27 | |
| jaypipes | jroll: heya Jim, lemme know if you have a moment to chat. Have a question for you about how IPA handles predictable network interface assignment... | 21:29 |
| *** Nisha has quit IRC | 21:29 | |
| jroll | jaypipes: shoot | 21:29 |
| jaypipes | jroll: I did :) | 21:29 |
| jroll | I don't see a question :P | 21:30 |
| jroll | you mean, how do we make sure the right nic gets the right things in the instance? | 21:30 |
| jaypipes | jroll: so, I was wondering if the coreos image that goes with IPA uses the systemd v197 predictable interface naming stuff (http://www.freedesktop.org/wiki/Software/systemd/PredictableNetworkInterfaceNames/) | 21:30 |
| jroll | or if the agent image itself is predicatble | 21:30 |
| jaypipes | jroll: how do you ensure that the network interfaces are named the same on a reboot.. | 21:31 |
| jroll | jaypipes: in the agent image, we don't really care. we just dhcp everything | 21:32 |
| *** penick_ has joined #openstack-ironic | 21:32 | |
| *** meghal has quit IRC | 21:32 | |
| jroll | jaypipes: unless there's something I'm missing | 21:32 |
| *** meghal has joined #openstack-ironic | 21:32 | |
| *** meghal has quit IRC | 21:32 | |
| *** meghal has joined #openstack-ironic | 21:33 | |
| jaypipes | jroll: I guess I'm missing something then... is the coreos image just a discovery image of sorts that then communicates with the Ironic server to register the bare metal node with Ironic? | 21:33 |
| jroll | jaypipes: not discovery... it does the deployment | 21:35 |
| jaypipes | jroll: I'm keen to learn how the bare metal nodes that Ironic launches ensure that the network interface assignments (i.e. eth0, eth1, etc) will refer to the same hardware on reboot of the bare metal node. | 21:35 |
| jroll | jaypipes: right, ok, that's up to the "guest" OS, and cloud-init or whatever | 21:36 |
| jroll | afaik | 21:36 |
| jroll | I'm not really sure it needs to | 21:36 |
| jaypipes | jroll: At AT&T, I know we ran into this problem quite a bit, that interface names were assigned asynchronously and sometimes on reboot would be different. We instituted this two-phase boot process where the first stage would lay down the 70-persisten-net.rules file and then reboot the server, and the second phase would pull all the other bits the node needed from the CHef server. | 21:36 |
| jaypipes | jroll: and of course, with heterogenous hardware, the udev rules are all different... | 21:37 |
| jroll | right, I'm trying to remember if/how we do it, I'm not sure we do, or maybe we have cloud-init matching on mac addresses | 21:38 |
| jroll | I think that's the key | 21:38 |
| jroll | and using the network_data.json thing | 21:38 |
| jlvillal | jroll: rloo: Looks like that 'revert' patch passed the Jenkins 'check' job. It is now in the Jenkins 'gate' job. | 21:38 |
| jlvillal | NobodyCam: ^^ | 21:38 |
| *** alexpilotti has joined #openstack-ironic | 21:38 | |
| rloo | jlvillal: nice! | 21:39 |
| jroll | jaypipes: https://github.com/jayofdoom/cloud-init-fedora-pkg/blob/master/cloud-init-0.7.5-onmetal-configdrive.patch | 21:40 |
| *** Marga__ has quit IRC | 21:40 | |
| jroll | jaypipes: that may be revealing | 21:40 |
| * jaypipes scampers off to read... | 21:40 | |
| jaypipes | :) | 21:40 |
| jaypipes | thx jroll | 21:40 |
| jroll | jaypipes: as to how we at rackspace do it, at least. I'm not sure it matters today in upstream | 21:40 |
| jroll | np | 21:40 |
| jroll | jaypipes: context: that reads the new network_data.json thing | 21:40 |
| *** Marga_ has joined #openstack-ironic | 21:40 | |
| jaypipes | jroll: you guys have homogenous hardware, though, yes? | 21:41 |
| jroll | jaypipes: yes | 21:41 |
| jaypipes | jroll: yes, the stuff from JoshNang right? | 21:41 |
| jroll | yep | 21:41 |
| jaypipes | I'm familiar with the patch series, yeah. | 21:41 |
| jaypipes | k, thank you sir :) | 21:41 |
| jroll | np, good luck :) | 21:41 |
| TheJulia | jroll: jaypipes: fwiw, glean will do interfaces by mac address for this very issue. | 21:48 |
| jroll | ++ | 21:49 |
| *** penick_ has quit IRC | 21:52 | |
| *** puranamr has quit IRC | 21:53 | |
| *** meghal has quit IRC | 21:59 | |
| *** meghal has joined #openstack-ironic | 21:59 | |
| *** alexpilotti has quit IRC | 21:59 | |
| *** meghal has quit IRC | 22:00 | |
| *** meghal has joined #openstack-ironic | 22:00 | |
| *** meghal has quit IRC | 22:00 | |
| *** absubram has quit IRC | 22:01 | |
| *** dprince has quit IRC | 22:02 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Create a microversions.py file https://review.openstack.org/213856 | 22:05 |
| *** shoutm has joined #openstack-ironic | 22:09 | |
| *** meghal has joined #openstack-ironic | 22:09 | |
| *** amotoki has joined #openstack-ironic | 22:10 | |
| jlvillal | jroll: Do you know what the ~50 line block of code is in Nova that devananda mentioned that very few people fully understand? | 22:14 |
| jroll | hehehe | 22:14 |
| jroll | sec | 22:14 |
| jlvillal | Sweet, thanks :) | 22:14 |
| *** amotoki has quit IRC | 22:15 | |
| jroll | jlvillal: https://github.com/openstack/nova/blob/master/nova/scheduler/filters/extra_specs_ops.py | 22:15 |
| jlvillal | Thank you! | 22:15 |
| jroll | jlvillal: actually, https://github.com/openstack/nova/blob/4cf6ef68199183697a0209751575f88fe5b2a733/nova/scheduler/filters/compute_capabilities_filter.py | 22:15 |
| jroll | but primarily the former which the latter calls | 22:15 |
| jlvillal | jroll: Thanks. So how they work together. Thanks for the info. | 22:16 |
| jroll | jlvillal: yeah, the feature as a whole | 22:16 |
| *** alexpilotti has joined #openstack-ironic | 22:16 | |
| jroll | jlvillal: tl;dr this matches host_state (which is roughly equivalent to node in our case) to flavor.extra_specs | 22:17 |
| jroll | host state https://github.com/openstack/nova/blob/4cf6ef68199183697a0209751575f88fe5b2a733/nova/scheduler/ironic_host_manager.py#L59 | 22:17 |
| jlvillal | Okay, thanks. | 22:17 |
| * jlvillal brain is starting to hurt ... ;) | 22:18 | |
| jroll | also related https://github.com/openstack/nova/blob/4cf6ef68199183697a0209751575f88fe5b2a733/nova/virt/ironic/driver.py#L234-314 | 22:18 |
| jroll | yeah :) | 22:18 |
| jlvillal | I see krtaylor listed as a contributor of ^^^^ :) | 22:20 |
| *** alexpilotti has quit IRC | 22:20 | |
| jlvillal | Ah, changing LLC to Foundation. | 22:21 |
| *** meghal has quit IRC | 22:22 | |
| *** lucas-dinner has quit IRC | 22:23 | |
| *** Haomeng has quit IRC | 22:23 | |
| thiagop | folks, I'm calling it a day | 22:26 |
| thiagop | good night everyone | 22:26 |
| jlvillal | thiagop: Good night. | 22:26 |
| jroll | night thiagop \o | 22:26 |
| *** alexpilotti has joined #openstack-ironic | 22:29 | |
| *** thiagop has quit IRC | 22:31 | |
| *** puranamr has joined #openstack-ironic | 22:35 | |
| *** praneshp has quit IRC | 22:38 | |
| jlvillal | aarefiev: Be great to hear your ideas about functional testing for python-ironicclient. Just saw the email you sent out last week... | 22:45 |
| *** penick has joined #openstack-ironic | 22:48 | |
| *** meghal has joined #openstack-ironic | 22:48 | |
| *** baoli has joined #openstack-ironic | 22:49 | |
| *** romcheg has quit IRC | 22:50 | |
| *** romcheg has joined #openstack-ironic | 22:51 | |
| *** romcheg has quit IRC | 22:51 | |
| *** natorious is now known as zz_natorious | 22:52 | |
| *** baoli has quit IRC | 22:53 | |
| krotscheck | I have, like, a million patches against ironic webclient that need a second +2. Anyone out there want to do that for me? https://review.openstack.org/#/q/status:open+project:openstack/ironic-webclient,n,z | 22:56 |
| *** alexpilotti has quit IRC | 23:00 | |
| * jroll calls not it as he has first +2 | 23:01 | |
| krotscheck | jroll: Whom should I be annoying on this? | 23:02 |
| jroll | krotscheck: I'm not sure any other ironic-core members have ever heard of gulp etc | 23:03 |
| krotscheck | jroll: Hrm, ok. I'll make betherly look at them and then someone can go through and be a proxy core. | 23:04 |
| jroll | krotscheck: fwiw I totally trust JoshNang's reviews on this stuff | 23:04 |
| JoshNang | :D | 23:04 |
| JoshNang | i think i've +1'ed most of them so far | 23:04 |
| betherly | No probs I'll look at them tomorrow krotscheck | 23:05 |
| betherly | Heading to bed now night all! | 23:05 |
| krotscheck | WHY ARE YOU STILL AWAKE | 23:06 |
| krotscheck | .... actually, nevermind. I do that too. | 23:06 |
| * krotscheck needs to remove the board from his own head first. | 23:06 | |
| *** penick_ has joined #openstack-ironic | 23:09 | |
| *** penick has quit IRC | 23:09 | |
| *** penick_ is now known as penick | 23:09 | |
| *** amotoki has joined #openstack-ironic | 23:11 | |
| * jlvillal grumbles about how slow the gate is... | 23:11 | |
| jlvillal | krotscheck: Add more cores to webclient??? ;) | 23:12 |
| jroll | there isn't a webclient acl | 23:13 |
| jlvillal | Hmmm. Maybe should have one? | 23:13 |
| krotscheck | jlvillal: Sure! Let's have someone other than me involved in the project first though, ok? :) | 23:13 |
| jroll | we talked about it last week | 23:13 |
| jroll | not sure why we didn't jfdi | 23:13 |
| krotscheck | I think devananda tried to make me a core (against my express wishes), and then ran into the lack of an acl and then other things got his attention. | 23:14 |
| jroll | oh | 23:14 |
| jroll | /shrug | 23:14 |
| jlvillal | krotscheck: You might be able to start part of the process yourself. Getting a group made. Not sure though. I've never done it. | 23:14 |
| jroll | I'm fine with current model, but think JoshNang should have core and maybe even just one +2 to merge for now | 23:14 |
| jlvillal | krotscheck: I think TheJulia did it before. | 23:14 |
| jlvillal | For bifrost | 23:15 |
| jlvillal | Or maybe that was all devananda | 23:15 |
| krotscheck | I've done it before for eslint-config-openstack | 23:15 |
| JoshNang | ¯\_(ツ)_/¯ | 23:15 |
| krotscheck | JoshNang: Did I meet you last week? | 23:16 |
| * jlvillal attempts to read that emoticon. Thinks JoshNang is happy about something ;) | 23:16 | |
| *** puranamr has quit IRC | 23:16 | |
| *** amotoki has quit IRC | 23:16 | |
| JoshNang | i write some angular outside of work and can work some decent grub files. not sure how much time i'll have to actually contribute though. i can definitely review | 23:16 |
| JoshNang | krotscheck: yup (though i was doing more IPA things), and we talked a bit in vancouver | 23:16 |
| JoshNang | iirc we talked about your very high ABV mead at the dinner :D | 23:17 |
| JoshNang | jlvillal: that's my shrug alias | 23:17 |
| krotscheck | JoshNang: Wait, were we talking about mead with your wife? | 23:17 |
| krotscheck | YES | 23:17 |
| krotscheck | NOW I FACE YOU | 23:17 |
| JoshNang | yup! | 23:17 |
| krotscheck | I cannot think without face | 23:17 |
| krotscheck | (it's leaky) | 23:17 |
| JoshNang | :D | 23:17 |
| jlvillal | JoshNang: Okay, that makes more sense | 23:18 |
| *** achanda has quit IRC | 23:22 | |
| betherly | krotscheck: ye I know right! Heading to bed now though. Get caught up in things and the tiredness goes... Helpfully until 7am when the alarm goes and my body complains | 23:25 |
| betherly | Maybe I'm nocturnal!!! :o (would explain a lot) | 23:25 |
| * jroll doesn't believe in alarms, the wake up is so much better without | 23:26 | |
| *** alexpilotti has joined #openstack-ironic | 23:26 | |
| betherly | And I know right! The stress of pairing IRC nicks and faces lol :D | 23:26 |
| krotscheck | betherly: Think of it like this: The surf is better at dawn. Work all night, then destress on a board. | 23:26 |
| betherly | jroll: couldn't agree more. Weekends... | 23:26 |
| krotscheck | :D | 23:26 |
| jroll | betherly: every day! | 23:26 |
| betherly | krotscheck: so true. Sadly I no longer live at the beach | 23:27 |
| jroll | krotscheck: or go to bed early and wake before dawn :) | 23:27 |
| betherly | Then I had no issue waking up early | 23:27 |
| betherly | Night guys! Actually going now :) | 23:27 |
| betherly | Ahh all! Need to get out of this 'guys' habit | 23:27 |
| * krotscheck wonders how long he can keep betherly awake just by mentioning her name in channel. | 23:27 | |
| * krotscheck is not always a nice person :D | 23:27 | |
| betherly | Ha! Funny Mr krotscheck | 23:28 |
| *** chlong has quit IRC | 23:29 | |
| krotscheck | I _think_ this is how new ACL lists are added -> https://review.openstack.org/#/c/214381/ | 23:29 |
| *** achanda has joined #openstack-ironic | 23:30 | |
| *** achanda has quit IRC | 23:30 | |
| *** achanda has joined #openstack-ironic | 23:31 | |
| *** puranamr has joined #openstack-ironic | 23:35 | |
| rloo | krotscheck: what is this 'ironic-webclient'? Is that under the ironic umbrella? | 23:43 |
| krotscheck | rloo: Yep. It's a standalone UI that talks to the raw API. | 23:44 |
| jroll | (and also plugs in to horizon) | 23:44 |
| *** praneshp has joined #openstack-ironic | 23:44 | |
| rloo | krotscheck: would be good if someone would send out email about it. | 23:44 |
| krotscheck | (jroll: might not want to advertise features that aren't ready yet ;) ) | 23:44 |
| krotscheck | rloo: It's been discussed several times on list | 23:44 |
| jroll | krotscheck: meh, it's a goal | 23:44 |
| krotscheck | Mostly in conjunction with "Where's the horizon panel?" | 23:45 |
| krotscheck | And my response being "We can't even run JS tests on infra yet, get in line" | 23:45 |
| jroll | heh, true | 23:46 |
| rloo | krotscheck: it has? oh, is that the ironic dashboard? | 23:46 |
| rloo | krotscheck: oh, yeah, i see mention of an ironic-webclient in one of your emails. guess i'm not sure i ever saw an announcement about it but maybe i just skipped/skimmed. is this something that should be added to the ironic wiki? | 23:47 |
| krotscheck | rloo: Once it's releasable, yes. | 23:48 |
| krotscheck | The hit list is here: https://etherpad.openstack.org/p/ironic-webclient-use-cases | 23:48 |
| *** naohirot has joined #openstack-ironic | 23:49 | |
| *** ndipanov has joined #openstack-ironic | 23:50 | |
| rloo | krotscheck: no worries i guess. just that i sometimes feel like i'm in the dark and i wonder if i feel that way, if others also. | 23:50 |
| krotscheck | Amazingly enough, I managed to get some commitments from both Intel and HP's UX teams to help with design :) | 23:50 |
| *** ndipanov has quit IRC | 23:52 | |
| *** ndipanov has joined #openstack-ironic | 23:52 | |
| jroll | krotscheck: so lots of blue then :) | 23:57 |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/bifrost: Fix typos in bifrost docs https://review.openstack.org/214192 | 23:57 |
| krotscheck | jroll: Teal! | 23:57 |
| krotscheck | :D | 23:57 |
| jroll | :) | 23:58 |
| krotscheck | Oh, speaking of which, I have to get the vector image for the ironic favicon I made licensed under openstack's commons license. | 23:58 |
| * krotscheck vaguely waves hands | 23:58 | |
| krotscheck | someone else do that. | 23:58 |
| * krotscheck wonders if there's a use case for an openstack icon font? | 23:59 | |
| jroll | lol | 23:59 |
| TheJulia | oh jeeze | 23:59 |
| *** ndipanov has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
