krotscheck | TheJulia: What? I make things go clicky clicky, icon fonts come with the territory :) | 00:00 |
---|---|---|
TheJulia | true... | 00:00 |
*** achanda_ has joined #openstack-ironic | 00:02 | |
NobodyCam | well there is a electric-boots font.. so whay not a pixie-boots font | 00:02 |
NobodyCam | http://il.static.1001fonts.net/e/l/electric-boots-font-2-big.png | 00:02 |
*** smoriya has joined #openstack-ironic | 00:04 | |
*** achanda has quit IRC | 00:05 | |
krotscheck | I'm a fan of this one -> http://fizzed.com/oss/font-mfizz | 00:07 |
*** achanda_ has quit IRC | 00:07 | |
NobodyCam | oh we need a pixieboots | 00:08 |
NobodyCam | :) | 00:08 |
*** alexpilotti has quit IRC | 00:08 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Replace HTTP 'magic numbers' with constants https://review.openstack.org/214393 | 00:16 |
*** shadower has quit IRC | 00:23 | |
*** shadower has joined #openstack-ironic | 00:23 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Replace HTTP 'magic numbers' with constants https://review.openstack.org/214393 | 00:23 |
*** alexpilotti has joined #openstack-ironic | 00:27 | |
*** alexpilotti has quit IRC | 00:31 | |
*** mtanino has quit IRC | 00:31 | |
openstackgerrit | James Penick proposed openstack/ironic-specs: No IPA to conductor communication https://review.openstack.org/212206 | 00:32 |
*** thrash is now known as thrash|g0ne | 00:33 | |
openstackgerrit | James Penick proposed openstack/ironic-specs: No IPA to conductor communication https://review.openstack.org/212206 | 00:35 |
*** shoutm has quit IRC | 00:36 | |
*** shoutm has joined #openstack-ironic | 00:37 | |
*** alexpilotti has joined #openstack-ironic | 00:38 | |
*** penick has quit IRC | 00:42 | |
*** baoli has joined #openstack-ironic | 00:49 | |
*** baoli has quit IRC | 00:54 | |
*** meghal has quit IRC | 00:58 | |
jlvillal | pshige_: Are you SHIGEMATSU Mitsuhiro? | 01:06 |
*** alexpilotti has quit IRC | 01:08 | |
*** zhenguo has joined #openstack-ironic | 01:09 | |
*** meghal has joined #openstack-ironic | 01:12 | |
*** puranamr has quit IRC | 01:14 | |
*** puranamr has joined #openstack-ironic | 01:15 | |
*** alexpilotti has joined #openstack-ironic | 01:25 | |
*** alexpilotti has quit IRC | 01:29 | |
openstackgerrit | Zhao Lei proposed openstack/ironic-python-agent: Remove execute mode of cloud-config.yaml https://review.openstack.org/210356 | 01:31 |
openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Removed X-Client header https://review.openstack.org/214406 | 01:31 |
*** alexpilotti has joined #openstack-ironic | 01:37 | |
*** Haomeng has joined #openstack-ironic | 01:46 | |
*** lazy_prince has joined #openstack-ironic | 01:48 | |
openstackgerrit | Michael Krotscheck proposed openstack/ironic: Added CORS support middleware to Ironic https://review.openstack.org/199769 | 01:56 |
openstackgerrit | Merged openstack/bifrost: Updated self referential text in the install role https://review.openstack.org/209246 | 01:59 |
*** rloo has quit IRC | 02:08 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Replace HTTP 'magic numbers' with constants https://review.openstack.org/214393 | 02:10 |
*** alexpilotti has quit IRC | 02:11 | |
jlvillal | mariojv, I think you are working too late ;) | 02:13 |
mariojv | jlvillal: sometimes i'm just in the mood to do code reviews :P | 02:14 |
jlvillal | mariojv, I understand. You can always look at mine :P | 02:15 |
mariojv | :) | 02:15 |
*** lazy_prince has quit IRC | 02:21 | |
jlvillal | krotscheck, Looks like requirements.txt change won't work on the patch. | 02:25 |
*** achanda has joined #openstack-ironic | 02:32 | |
jlvillal | mrda, ping? | 02:36 |
*** meghal1 has joined #openstack-ironic | 02:36 | |
*** alexpilotti has joined #openstack-ironic | 02:37 | |
*** meghal has quit IRC | 02:38 | |
jlvillal | FYI: Gate still seems broken. https://review.openstack.org/#/c/214282/ was merged, which hopefully would fix the broken gate. Not sure if there is a delay for the patch to affect Ironic gate or the patch does not fix the issue. | 02:51 |
*** hakimo_ has joined #openstack-ironic | 02:52 | |
mrda | jlvillal: heh | 02:52 |
jlvillal | mrda, I replied to your review :) | 02:53 |
mrda | thanks, and I responded :) | 02:53 |
jlvillal | Woot! | 02:53 |
jlvillal | mrda, Oh and a did the 'magic numbers' patch | 02:54 |
*** ramineni has joined #openstack-ironic | 02:54 | |
*** hakimo has quit IRC | 02:54 | |
jlvillal | mrda, Are you sure you responded? | 02:54 |
jlvillal | I had responded about 15 minutes ago. | 02:54 |
mrda | yeah, just checked... | 02:56 |
mrda | oh, that post took a while | 02:56 |
jlvillal | mrda, I see it now | 02:56 |
*** Nisha has joined #openstack-ironic | 02:56 | |
jlvillal | mrda, I'll push another revision and call it done. I'll go with your name suggestions. Thanks! | 02:57 |
*** ramineni1 has joined #openstack-ironic | 02:57 | |
mrda | jlvillal: cool, let's see if we get any other reviewers :) | 02:58 |
jlvillal | I hope... | 02:58 |
*** ramineni has quit IRC | 02:58 | |
mrda | jlvillal: as for http error codes, I need two monitors so I can cross checked easily, so that'll have to wait until tomorrow | 02:59 |
jlvillal | Works for me. | 02:59 |
mrda | TheJulia: Just FYI, the battery "you made my buy" arrived today, and works a treat. Thanks for the suggestion! | 03:00 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Create a microversions.py file https://review.openstack.org/213856 | 03:02 |
jlvillal | mrda, Who was interested/working on AMT console support? | 03:03 |
jlvillal | lintan, and I would like to know :) | 03:03 |
mrda | jlvillal: I was thinking about it :) but it's low priority | 03:03 |
jlvillal | mrda, Oh. I thought there was a discussion :) | 03:04 |
jroll | jlvillal | pshige_: Are you SHIGEMATSU Mitsuhiro? <- yes | 03:04 |
jlvillal | jroll, Thanks! | 03:04 |
jroll | np :) | 03:04 |
jlvillal | jroll, So far no joy on patch fixing the gate. | 03:04 |
jroll | mariojv: if you know frontend things, you should review krotscheck's ironic-webclient patches | 03:05 |
jlvillal | Not sure if it takes time for patch to affect the gate or it just doesn't fix the issue. | 03:05 |
jroll | jlvillal: boo :/ | 03:05 |
jroll | is it merged now? | 03:05 |
jlvillal | Yes it is | 03:05 |
jroll | and a recheck after merge didn't go through? | 03:05 |
jlvillal | https://review.openstack.org/#/c/214282/ | 03:05 |
jlvillal | I did a few rechecks and still fail. | 03:05 |
jroll | :| | 03:05 |
jlvillal | I didn't go look yet at the messages yet. | 03:06 |
jroll | there shouldn't be delay fwiw | 03:06 |
jroll | example? | 03:06 |
* jlvillal notes he uses yet twice in same sentence | 03:06 | |
jlvillal | jroll, http://logs.openstack.org/93/214393/3/check/gate-tempest-dsvm-ironic-pxe_ssh-postgres/b65aafe/ | 03:07 |
jroll | blah, same error | 03:07 |
jroll | ok, let's revisit tomorrow I guess | 03:08 |
* jroll goes back to manning the stovetop | 03:08 | |
jlvillal | jroll, Thanks | 03:08 |
jroll | thank you! | 03:08 |
jlvillal | jroll, Time for me to quit too! :) | 03:08 |
jlvillal | Good night all. | 03:08 |
jroll | night | 03:08 |
*** ramineni has joined #openstack-ironic | 03:11 | |
*** alexpilotti has quit IRC | 03:12 | |
*** ramineni1 has quit IRC | 03:13 | |
*** bizarrochristy has quit IRC | 03:14 | |
*** bizarrochristy has joined #openstack-ironic | 03:14 | |
*** purp has quit IRC | 03:15 | |
*** bizarrochristy has quit IRC | 03:18 | |
*** purp has joined #openstack-ironic | 03:19 | |
*** raginbajin has quit IRC | 03:27 | |
*** zz_natorious is now known as natorious | 03:28 | |
*** purp has quit IRC | 03:28 | |
*** raginbajin has joined #openstack-ironic | 03:29 | |
*** purp has joined #openstack-ironic | 03:30 | |
*** logan2 has quit IRC | 03:31 | |
*** alexpilotti has joined #openstack-ironic | 03:39 | |
*** natorious is now known as zz_natorious | 03:39 | |
*** ramineni has quit IRC | 03:44 | |
*** shoutm has quit IRC | 03:45 | |
*** ramineni has joined #openstack-ironic | 03:46 | |
*** ramineni1 has joined #openstack-ironic | 03:48 | |
*** ramineni has quit IRC | 03:50 | |
*** shoutm has joined #openstack-ironic | 03:56 | |
*** ramineni has joined #openstack-ironic | 03:59 | |
*** ramineni1 has quit IRC | 04:01 | |
*** ramineni1 has joined #openstack-ironic | 04:04 | |
*** ramineni has quit IRC | 04:06 | |
*** ramineni has joined #openstack-ironic | 04:07 | |
*** ramineni1 has quit IRC | 04:08 | |
*** raginbajin has quit IRC | 04:10 | |
*** purp has quit IRC | 04:12 | |
*** ramineni1 has joined #openstack-ironic | 04:12 | |
*** ramineni has quit IRC | 04:14 | |
*** ramineni has joined #openstack-ironic | 04:17 | |
*** purp has joined #openstack-ironic | 04:18 | |
*** alexpilotti has quit IRC | 04:18 | |
*** raginbajin has joined #openstack-ironic | 04:28 | |
*** puranamr has quit IRC | 04:28 | |
*** puranamr has joined #openstack-ironic | 04:28 | |
*** VikasC has joined #openstack-ironic | 04:33 | |
*** praneshp has quit IRC | 04:36 | |
*** meghal1 has quit IRC | 04:37 | |
*** ramineni has quit IRC | 04:37 | |
*** coolsvap|away is now known as coolsvap | 04:46 | |
*** meghal has joined #openstack-ironic | 04:52 | |
*** alexpilotti has joined #openstack-ironic | 04:54 | |
*** PradeepV has joined #openstack-ironic | 04:55 | |
*** meghal has quit IRC | 04:56 | |
*** irenab has quit IRC | 04:58 | |
*** coolsvap is now known as coolsvap|away | 05:08 | |
*** ramineni has joined #openstack-ironic | 05:10 | |
*** JoshNang has quit IRC | 05:17 | |
*** boris-42 has quit IRC | 05:20 | |
*** alexpilotti has quit IRC | 05:23 | |
*** chenglch has joined #openstack-ironic | 05:23 | |
*** coolsvap|away is now known as coolsvap | 05:23 | |
*** JoshNang has joined #openstack-ironic | 05:25 | |
*** irenab has joined #openstack-ironic | 05:30 | |
*** puranamr has quit IRC | 05:44 | |
*** amotoki has joined #openstack-ironic | 05:50 | |
*** alexpilotti has joined #openstack-ironic | 05:58 | |
jlvillal | Haomeng, The gate is broken :( | 06:07 |
jlvillal | Haomeng, http://bit.ly/ironic-whiteboard | 06:07 |
Haomeng | jlvillal: yes, almosty patch gate failed | 06:07 |
Haomeng | jlvillal: thanks for your info:) | 06:08 |
jlvillal | Haomeng, glad to help. We worked on it today, but no luck in a fix. | 06:08 |
Haomeng | jlvillal: it merged, will be restored today:) | 06:08 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Enhance Power Interface for Soft Power Off and Inject NMI https://review.openstack.org/186700 | 06:08 |
Haomeng | jlvillal: https://review.openstack.org/#/c/214282/ this is merged | 06:09 |
jlvillal | Haomeng, Yeah and didn't fix it :( | 06:09 |
Haomeng | jlvillal: you mean after 214282 is merged, gate still broken? | 06:09 |
Haomeng | jlvillal: oh | 06:09 |
jlvillal | Haomeng, Yes. It appears so. | 06:09 |
Haomeng | Haomeng: neutron issue, ok, got:) | 06:09 |
*** coolsvap is now known as coolsvap|away | 06:10 | |
*** praneshp has joined #openstack-ironic | 06:10 | |
jlvillal | Haomeng, We may want to think about making the neutron / postgres job a non-voting one. If we can't get it fixed soon. | 06:10 |
Haomeng | jlvillal: but ironic has no such neutron gate, we depends on neutron latest code I think | 06:10 |
jlvillal | Haomeng, I'm going to bed. If you can have it fixed by the time I wake up it would be great ;) LOL | 06:10 |
Haomeng | jlvillal: ok, will try | 06:11 |
Haomeng | jlvillal: good night, and nice dreem:) | 06:11 |
jlvillal | Haomeng, The failing job is a neutron postgres job. You can see the patches failing | 06:11 |
Haomeng | jlvillal: yes | 06:11 |
jlvillal | Haomeng, Like https://review.openstack.org/#/c/209605/ | 06:11 |
jlvillal | jroll, myself, and others will try to work on it in the morning our time. But awesome if you or others can make progress before then. | 06:12 |
Haomeng | jlvillal: you mean this job - gate-tempest-dsvm-ironic-pxe_ssh-postgres ,righ? | 06:12 |
jlvillal | Haomeng, correct. | 06:12 |
Haomeng | jlvillal: ok,let me check and try to fix | 06:12 |
jlvillal | Haomeng, Thank you very much. | 06:12 |
*** praneshp_ has joined #openstack-ironic | 06:12 | |
jlvillal | Haomeng, We do believe the fault is neutron | 06:13 |
jroll | it's going to require a neutron change or to make that job non-voting. | 06:13 |
Haomeng | jlvillal: not sure if I can, will try my best:) | 06:13 |
jlvillal | Not Ironic | 06:13 |
Haomeng | jroll: yes | 06:13 |
jroll | Haomeng: some neutron cores have ideas | 06:13 |
* jlvillal thought jroll would be asleep... | 06:13 | |
jroll | Haomeng: you can leave it if you have other things to do :) | 06:13 |
Haomeng | jroll: I think we can not mark gate-tempest-dsvm-ironic-pxe_ssh-postgres as non-voting | 06:14 |
Haomeng | jroll: ok, thanks for your reminder:) | 06:14 |
jroll | Haomeng: we can mark non-voting temporarily, but would rather fix the issue :) | 06:14 |
Haomeng | jroll: yes, it should be neutron issue | 06:14 |
jlvillal | A patch would have to be submitted to do that. Make it non-voting. Infra would approve patch | 06:14 |
Haomeng | jroll: yes, temporaily | 06:14 |
jlvillal | Good night all. | 06:14 |
*** praneshp has quit IRC | 06:14 | |
*** praneshp_ is now known as praneshp | 06:14 | |
*** lintan_ has joined #openstack-ironic | 06:14 | |
Haomeng | jlvillal: good night:) | 06:15 |
jlvillal | jroll, I hope I didn't wake you by using your nick! | 06:15 |
jroll | jlvillal: no, I explicitly don't push irc to phone or whatever | 06:15 |
jroll | just happened to be here for a moment | 06:15 |
jlvillal | smart man | 06:15 |
jroll | here being on the computer at all :P | 06:15 |
jroll | anyway, gnight | 06:16 |
*** lsmola has joined #openstack-ironic | 06:21 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Enhance Power Interface for Soft Power Off and Inject NMI https://review.openstack.org/186700 | 06:27 |
*** alexpilotti has quit IRC | 06:28 | |
*** achanda has quit IRC | 06:34 | |
*** lazy_prince has joined #openstack-ironic | 06:36 | |
*** ramineni has quit IRC | 06:46 | |
*** alexpilotti has joined #openstack-ironic | 06:55 | |
*** jlvillal has quit IRC | 06:58 | |
*** jlvillal has joined #openstack-ironic | 06:58 | |
*** VikasC has quit IRC | 06:59 | |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - RPC https://review.openstack.org/206243 | 07:07 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 07:07 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - net https://review.openstack.org/206245 | 07:07 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - DB https://review.openstack.org/206232 | 07:07 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - objs https://review.openstack.org/206238 | 07:07 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add network provider interface and implementations https://review.openstack.org/139687 | 07:07 |
*** ramineni has joined #openstack-ironic | 07:20 | |
*** Cameron_C has quit IRC | 07:24 | |
*** dtantsur|pto19Au is now known as dtantsur | 07:26 | |
dtantsur | Morning Ironic | 07:26 |
lintan_ | Morning | 07:33 |
*** alexpilotti has quit IRC | 07:36 | |
*** ramineni1 has joined #openstack-ironic | 07:37 | |
zhenguo | dtantsur: o/ | 07:37 |
*** ramineni has quit IRC | 07:38 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Enhance Power Interface for Soft Power Off and Inject NMI https://review.openstack.org/186700 | 07:39 |
*** yog_ has joined #openstack-ironic | 07:40 | |
*** ramineni has joined #openstack-ironic | 07:43 | |
aarefiev | jlvillal: hi, thanks, I would appreciate if you have some ideas or suggestion | 07:44 |
*** ramineni1 has quit IRC | 07:44 | |
*** romainh has joined #openstack-ironic | 07:51 | |
*** ramineni1 has joined #openstack-ironic | 07:55 | |
*** ramineni has quit IRC | 07:56 | |
*** boris-42 has joined #openstack-ironic | 07:58 | |
openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic-specs: Update the State Machine spec (part 2) https://review.openstack.org/212133 | 07:59 |
openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic-specs: Update the State Machine spec (part 1) https://review.openstack.org/212132 | 07:59 |
*** jcoufal has joined #openstack-ironic | 08:02 | |
*** alexpilotti has joined #openstack-ironic | 08:02 | |
*** yog_ has quit IRC | 08:05 | |
*** ramineni has joined #openstack-ironic | 08:06 | |
*** ramineni1 has quit IRC | 08:08 | |
*** mbound has joined #openstack-ironic | 08:12 | |
*** MattMan has joined #openstack-ironic | 08:14 | |
*** praneshp has quit IRC | 08:17 | |
*** timbyr_ has joined #openstack-ironic | 08:21 | |
*** derekh has joined #openstack-ironic | 08:27 | |
*** jistr has joined #openstack-ironic | 08:34 | |
*** lucasagomes has joined #openstack-ironic | 08:35 | |
*** alexpilotti has quit IRC | 08:38 | |
*** ramineni has quit IRC | 08:40 | |
*** shoutm has quit IRC | 08:40 | |
openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/coreos-image-builder: Update README.rst https://review.openstack.org/214512 | 08:40 |
openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/coreos-image-builder: Update README.rst https://review.openstack.org/214512 | 08:43 |
openstackgerrit | Om Kumar proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 08:45 |
*** achanda has joined #openstack-ironic | 08:48 | |
openstackgerrit | Beth Elwell proposed openstack/ironic-webclient: Corrected author in package.json https://review.openstack.org/213718 | 08:57 |
betherly | morning! | 08:59 |
lucasagomes | betherly, good morning | 09:01 |
lucasagomes | the postgres job is broken? | 09:03 |
*** romcheg has joined #openstack-ironic | 09:04 | |
*** alexpilotti has joined #openstack-ironic | 09:05 | |
*** pelix has joined #openstack-ironic | 09:05 | |
betherly | lucasagomes: which one? | 09:06 |
lucasagomes | betherly, gate-tempest-dsvm-ironic-pxe_ssh-postgres | 09:07 |
lucasagomes | betherly, you work in neutron right? | 09:12 |
betherly | nope. just started last monday :) going to be working on horizon panels for ironic | 09:12 |
lucasagomes | betherly, ah gotcha! NICE! | 09:12 |
betherly | were you at the ironic mid-cycle last week? | 09:12 |
lucasagomes | yeah the error seems to come from neutorn in our gate | 09:12 |
lucasagomes | http://logs.openstack.org/56/213856/4/check/gate-tempest-dsvm-ironic-pxe_ssh-postgres/91c7a5c/logs/screen-q-svc.txt.gz | 09:12 |
lucasagomes | betherly, unfortunately I did miss the last mid-cycle | 09:13 |
betherly | sad times | 09:13 |
lucasagomes | I'm going be around in Tokyo so we can grab a pint there :-) | 09:13 |
betherly | still trying to pair up IRC nicks and faces | 09:13 |
lucasagomes | heh yeah it's good | 09:13 |
betherly | sounds like a plan! meeting in real life...wait IRC people exist beyond the realms of internet relay? | 09:13 |
dtantsur | of course no, but we pretend to | 09:14 |
lucasagomes | hah | 09:14 |
lucasagomes | betherly, where are you based on? UK? | 09:15 |
betherly | dtantsur: +1 | 09:15 |
betherly | haha | 09:15 |
betherly | UK :) | 09:15 |
betherly | you? | 09:15 |
lucasagomes | Dublin, Ireland | 09:15 |
betherly | oh nice! not far away | 09:16 |
lucasagomes | we're neighbors | 09:16 |
lucasagomes | yeah | 09:16 |
betherly | woo! | 09:16 |
dtantsur | more people in EU time \o/ | 09:16 |
betherly | i hear the next midcycle might be in Ireland... might! dont quote me ;) | 09:16 |
betherly | where are you dtantsur ? | 09:16 |
dtantsur | betherly, Czech, Brno | 09:17 |
lucasagomes | betherly, that would be awesome! | 09:17 |
betherly | mm. yay for evening meetings living here haha | 09:17 |
betherly | sick as a pigeon on sesame seed loaf | 09:17 |
lucasagomes | we can have the meeting drinking Guinness at the gravity bar! | 09:17 |
*** achanda has quit IRC | 09:17 | |
betherly | (thats a happy sentiment btw) | 09:17 |
betherly | yes! Guinness in Ireland?! Ummmm YE! | 09:18 |
betherly | is it bad that ive never been and i live this close? | 09:18 |
*** ishant has joined #openstack-ironic | 09:19 | |
lazy_prince | betherly: morning.. looks like you are already enjoying IRC.. :) | 09:19 |
*** lintan_ has quit IRC | 09:20 | |
betherly | lazy_prince: morning! yup yup! :) | 09:20 |
dtantsur | hehe | 09:23 |
lucasagomes | https://bugs.launchpad.net/neutron/+bug/1486467 | 09:24 |
openstack | Launchpad bug 1486467 in neutron "Postgres error: column "reservations.expiration" must appear in the GROUP BY clause or be used in an aggregate function" [Undecided,New] | 09:24 |
*** max_lobur has joined #openstack-ironic | 09:30 | |
*** ramineni has joined #openstack-ironic | 09:31 | |
sambetts | Woo EU time FTW! | 09:35 |
sambetts | Morning all o/ | 09:35 |
lucasagomes | sambetts, hi there mroning | 09:37 |
lucasagomes | morning* | 09:37 |
sambetts | How you doing lucasagomes? | 09:38 |
lucasagomes | sambetts, all is well, you? | 09:38 |
sambetts | Same thanks :) | 09:39 |
*** ramineni1 has joined #openstack-ironic | 09:39 | |
*** ramineni has quit IRC | 09:41 | |
*** Nisha has quit IRC | 09:41 | |
*** ramineni has joined #openstack-ironic | 09:42 | |
*** ramineni1 has quit IRC | 09:44 | |
*** alexpilotti has quit IRC | 09:44 | |
*** jaypipes has quit IRC | 09:46 | |
betherly | morning sambetts :) | 09:47 |
*** killer_prince has joined #openstack-ironic | 09:48 | |
*** lazy_prince has quit IRC | 09:52 | |
*** ramineni1 has joined #openstack-ironic | 09:53 | |
*** ramineni has quit IRC | 09:54 | |
dtantsur | sambetts, morning! | 09:56 |
sambetts | Morning betherly, dtantsur :) | 09:57 |
*** killer_prince has quit IRC | 09:58 | |
*** naohirot has quit IRC | 09:59 | |
*** ramineni has joined #openstack-ironic | 10:01 | |
*** chenglch has quit IRC | 10:04 | |
*** ramineni1 has quit IRC | 10:04 | |
*** ramineni has quit IRC | 10:06 | |
*** zz_natorious has quit IRC | 10:06 | |
*** eglute has quit IRC | 10:06 | |
*** zz_natorious has joined #openstack-ironic | 10:07 | |
*** zz_natorious is now known as natorious | 10:08 | |
*** eglute has joined #openstack-ironic | 10:08 | |
*** alexpilotti has joined #openstack-ironic | 10:10 | |
*** Ng has quit IRC | 10:12 | |
*** lekha has quit IRC | 10:12 | |
*** lekha has joined #openstack-ironic | 10:13 | |
*** Ng has joined #openstack-ironic | 10:13 | |
*** lazy_prince has joined #openstack-ironic | 10:16 | |
*** shoutm has joined #openstack-ironic | 10:18 | |
sambetts | dtantsur: I've just discovered an interesting bug with the tests in inspector | 10:26 |
dtantsur | mmm? | 10:27 |
sambetts | running tox -e py27 is failing, because of this test test_bad_hostname_errors (ironic_inspector.test.test_utils.TestGetIpmiAddress) | 10:29 |
sambetts | on my machine ip = socket.gethostbyname(value) doesn't Error | 10:30 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Copy gitignore from ironic https://review.openstack.org/212561 | 10:32 |
dtantsur | sambetts, oh interesting. why doesn't it fail for you? | 10:32 |
sambetts | dtantsur: I really don't know, I've tried just running the socket command in the python interpretor too and it returns the ip you pass in just fine | 10:34 |
sambetts | dtantsur: this is my test script http://paste.openstack.org/show/421721/ | 10:35 |
dtantsur | one moment, I'm on the meeting | 10:36 |
sambetts | dtantsur: I've just corrected my test script and run it: http://paste.openstack.org/show/421723/ | 10:40 |
sambetts | dtantsur: looks like the word meow actually resolves to a hostname somehow.. | 10:41 |
dtantsur | LOOOOL | 10:42 |
betherly | sambetts: and why not?! :p | 10:42 |
sambetts | doing some further googling it looks like its my BT home router doing stupid things ... I need to disable that... they are injecting an IP into failed DNS results... | 10:44 |
sambetts | -,- not happy about that... | 10:44 |
*** alexpilotti has quit IRC | 10:44 | |
betherly | all hail google, he knoweth all | 10:44 |
dtantsur | sambetts, probably we should have mocked socket in this test anyway... | 10:45 |
sambetts | dtantsur: Yeah, I think so too | 10:45 |
dtantsur | mind putting up a patch? | 10:45 |
* dtantsur is running around with a damn visa... | 10:46 | |
sambetts | Will do :) | 10:46 |
*** pal has joined #openstack-ironic | 10:59 | |
*** killer_prince has joined #openstack-ironic | 10:59 | |
openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Mock socket in test_bad_hostname_errors https://review.openstack.org/214558 | 11:00 |
dtantsur | cool! | 11:00 |
*** shoutm has quit IRC | 11:01 | |
*** lazy_prince has quit IRC | 11:03 | |
zhenguo | hi dtantsur, could you please have a look at https://review.openstack.org/#/c/204088/ when you have time | 11:04 |
dtantsur | zhenguo, I'll try to, but I suspect I won't have time today :( | 11:05 |
*** baoli has joined #openstack-ironic | 11:05 | |
zhenguo | dtantsur: ok, but please help when you have time :) | 11:07 |
dtantsur | sure | 11:08 |
zhenguo | dtantsur: thanks | 11:08 |
*** ishant has quit IRC | 11:10 | |
*** baoli has quit IRC | 11:10 | |
*** alexpilotti has joined #openstack-ironic | 11:11 | |
*** boris-42 has quit IRC | 11:20 | |
*** ramineni has joined #openstack-ironic | 11:20 | |
*** ramineni has quit IRC | 11:21 | |
*** pal has quit IRC | 11:21 | |
dtantsur | jroll, when you're here, we need your opinion on https://review.openstack.org/#/c/209889 please. | 11:22 |
lucasagomes | betherly, sorry... I missed that last bits. Nah, it's not bad. I've never been to london for example :-) | 11:25 |
*** shoutm has joined #openstack-ironic | 11:26 | |
*** pelix has quit IRC | 11:27 | |
betherly | lucasagomes: there's so much of even England I haven't seen it's embarrassing! I do love London though you should make a trip :) | 11:29 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Test broken gate DO NOT MERGE https://review.openstack.org/214566 | 11:29 |
lucasagomes | this depends on a patch in neutron with the potential gate fix ^ | 11:29 |
lucasagomes | betherly, yeah I hear you! I have to go | 11:31 |
lucasagomes | maybe we should organize the mid-cycle over there (-: | 11:31 |
dtantsur | oh no, lets use schengen part of EU :D | 11:32 |
lucasagomes | dtantsur, :-( yeah visas can be painful I understand | 11:39 |
*** thrash|g0ne is now known as thrash | 11:39 | |
*** lucasagomes is now known as lucas-hungry | 11:40 | |
* lucas-hungry bbl will get some food | 11:41 | |
*** alexpilotti has quit IRC | 11:45 | |
*** jaypipes has joined #openstack-ironic | 11:53 | |
betherly | lucas-hungry: yes! I suggested Bristol last week (my home town) ... :p | 11:54 |
*** irenab has quit IRC | 11:55 | |
sambetts | betherly, lucas-hungry: I would suggest somewhere near me in Kent, but I work from home and can't host that many people in my house :-P | 12:03 |
sambetts | my nearest office is London or Reading | 12:03 |
betherly | sambetts: lol! I hear ya on that one! Another Englishian! Yay! | 12:03 |
*** albertoffb has joined #openstack-ironic | 12:05 | |
*** thiagop has joined #openstack-ironic | 12:06 | |
thiagop | Good morning Ironic | 12:06 |
*** dprince has joined #openstack-ironic | 12:08 | |
*** killer_prince has quit IRC | 12:10 | |
betherly | morning thiagop | 12:11 |
thiagop | betherly: hi | 12:11 |
*** alexpilotti has joined #openstack-ironic | 12:11 | |
*** marios_ has joined #openstack-ironic | 12:14 | |
*** marios_ has quit IRC | 12:15 | |
*** ihrachyshka has joined #openstack-ironic | 12:17 | |
*** amotoki_ has joined #openstack-ironic | 12:17 | |
openstackgerrit | Satoru Moriya proposed openstack/ironic: [WIP] Add volume_connector table to save connector information https://review.openstack.org/200983 | 12:21 |
openstackgerrit | Satoru Moriya proposed openstack/ironic: [WIP] Introduce VolumeConnector object https://review.openstack.org/214584 | 12:21 |
openstackgerrit | Satoru Moriya proposed openstack/ironic: [WIP] Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 12:21 |
openstackgerrit | Satoru Moriya proposed openstack/ironic: [WIP] Add REST API for volume connector operation https://review.openstack.org/214586 | 12:21 |
*** chlong has joined #openstack-ironic | 12:26 | |
*** amotoki_ has quit IRC | 12:27 | |
*** smoriya has quit IRC | 12:36 | |
*** lucas-hungry is now known as lucasagomes | 12:39 | |
*** logan2 has joined #openstack-ironic | 12:39 | |
*** jjohnson2 has joined #openstack-ironic | 12:40 | |
lucasagomes | sambetts, betherly hah I'm up to any of those! | 12:42 |
*** alexpilotti has quit IRC | 12:42 | |
openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM firmware information parsing for Lenovo ThinkServers Parse Version Info and add a command to retrieve it. https://review.openstack.org/214590 | 12:42 |
*** amotoki_ has joined #openstack-ironic | 12:47 | |
*** pal has joined #openstack-ironic | 12:52 | |
*** Nisha has joined #openstack-ironic | 12:55 | |
*** jcoufal_ has joined #openstack-ironic | 12:58 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Refactor the essential prop list of inspect driver https://review.openstack.org/214597 | 12:59 |
*** alexpilotti has joined #openstack-ironic | 12:59 | |
*** irenab has joined #openstack-ironic | 13:01 | |
*** jcoufal has quit IRC | 13:01 | |
*** alexpilotti has quit IRC | 13:03 | |
openstackgerrit | Merged stackforge/pyghmi: Add LED parsing for Lenovo ThinkServers https://review.openstack.org/214345 | 13:05 |
*** Nisha has quit IRC | 13:08 | |
*** Nisha has joined #openstack-ironic | 13:08 | |
*** rloo has joined #openstack-ironic | 13:09 | |
*** alexpilotti has joined #openstack-ironic | 13:11 | |
*** puranamr has joined #openstack-ironic | 13:12 | |
*** pal has quit IRC | 13:13 | |
lucasagomes | jroll, http://mjg59.dreamwidth.org/37113.html at least now it's clear the reason their IP policy is not clear | 13:13 |
lucasagomes | heh | 13:13 |
lucasagomes | the reason why* | 13:13 |
ihrachyshka | lucasagomes, neutron fix in gate | 13:14 |
*** pal has joined #openstack-ironic | 13:14 | |
lucasagomes | ihrachyshka, thanks a lot for looking into it man! Appreciate | 13:14 |
rloo | hi and morning ironic'ers, lucasagomes, ihrachyshka | 13:15 |
rloo | is the gate still broken? | 13:15 |
lucasagomes | rloo, fix is on the way ^ | 13:15 |
ihrachyshka | lucasagomes, I only pulled proper strings. I would fail to fix it quick myself. | 13:15 |
dtantsur | rloo, morning! | 13:17 |
rloo | hi dtantsur, welcome back. I hope you enjoyed your time away from us :-) | 13:18 |
dtantsur | I was missing you, everything else went fine :) | 13:18 |
rloo | lucasagomes: sigh. still broken by neutron then. | 13:19 |
rloo | dtantsur: we missed you too :D | 13:19 |
lucasagomes | rloo, the fix seems to have been approved so we need to wait for it to merge | 13:19 |
rloo | lucasagomes: ok, i'll update the whiteboard. there was a different neutron patch that i guess we hoped would fix things. | 13:20 |
*** pal has quit IRC | 13:20 | |
lucasagomes | rloo, yeah, we were tracking it here https://bugs.launchpad.net/neutron/+bug/1486467 | 13:21 |
openstack | Launchpad bug 1486467 in neutron "Postgres error: column "reservations.expiration" must appear in the GROUP BY clause or be used in an aggregate function" [Critical,In progress] - Assigned to Ann Kamyshnikova (akamyshnikova) | 13:21 |
rloo | lucasagomes: i added that link to the whiteboard too. | 13:22 |
lucasagomes | rloo, thanks | 13:22 |
*** Nisha has quit IRC | 13:22 | |
openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM firmware information parsing for Lenovo ThinkServers. Parse Version Info and add a command to retrieve it. https://review.openstack.org/214590 | 13:22 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Refactor the essential prop list of inspect driver https://review.openstack.org/214597 | 13:27 |
BadCub | Morning Ironic | 13:31 |
sambetts | Morning BadCub o/ | 13:32 |
sambetts | o/ Hey rloo | 13:32 |
rloo | hi sambetts, morning BadCub | 13:32 |
jroll | morning all :) | 13:33 |
jroll | lucasagomes: that's... hilarious | 13:33 |
jroll | in a sad way | 13:33 |
lucasagomes | jroll, I know right? There's some conversation going on G+ as well | 13:33 |
lucasagomes | and morning | 13:33 |
lucasagomes | BadCub, morning too :-) | 13:33 |
openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM firmware info parsing for Lenovo ThinkServers Parse Version Info and add a command to retrieve it. https://review.openstack.org/214590 | 13:34 |
jroll | dtantsur: which opinion of mine do you care to invoke on 209889? | 13:34 |
*** pal has joined #openstack-ironic | 13:35 | |
dtantsur | jroll, hi! about supporting/not supporting ubuntu LTS for now | 13:35 |
dtantsur | BadCub, morning you too | 13:35 |
dtantsur | jroll, and of course review will be appreciated :) | 13:36 |
jroll | dtantsur: I mean, we should support it, why not? | 13:36 |
*** alexpilotti has quit IRC | 13:36 | |
dtantsur | jroll, this will require more code, that nobody seems to care much about :) IIRC JoshNang was afraid that we'll break it anyway | 13:37 |
dtantsur | and I can't test ubuntu there, so I can just write some code that *presumably* works on ubuntu :) | 13:37 |
jroll | sure, we'll also break fedora in josh's model | 13:37 |
jroll | but we should try | 13:38 |
jroll | idk | 13:38 |
betherly | morning BadCub jroll rloo :) | 13:38 |
jroll | hiya betherly | 13:38 |
rloo | hi betherly and mornin' jroll | 13:39 |
jroll | hi rloo :) | 13:41 |
dtantsur | well, at least I can personally test stuff on Fedora :) | 13:41 |
*** Shrews has left #openstack-ironic | 13:42 | |
dtantsur | ok, I'll drop the change about SERIAL for now, it's not a critical requirement | 13:42 |
openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM firmware info parsing for Lenovo ThinkServers https://review.openstack.org/214590 | 13:44 |
*** lazy_prince has joined #openstack-ironic | 13:44 | |
rloo | dtantsur: are you good with this as is, and someone follow up with the tox changes later/whenever it bothers them? | 13:47 |
rloo | dtantsur: https://review.openstack.org/#/c/213249/ | 13:47 |
dtantsur | rloo, yep, I'm fine | 13:48 |
BadCub | morning lucasagomes dtansur betherly | 13:48 |
rloo | thx dtantsur | 13:48 |
betherly | morning BadCub | 13:48 |
BadCub | s/dtansur/dtantsur sorry for typo | 13:48 |
dtantsur | np :) | 13:49 |
*** killer_prince has joined #openstack-ironic | 13:49 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Extend hardware manager with data needed for inspector https://review.openstack.org/209889 | 13:49 |
* BadCub needs more coffee | 13:49 | |
*** jcoufal_ has quit IRC | 13:49 | |
dtantsur | jroll, JoshNang, dropped non-ubuntu-compatible code ^^ | 13:49 |
dtantsur | * ubuntu-incompatible even | 13:50 |
jroll | dtantsur: oh hmm, reading the thing more | 13:51 |
*** jcoufal has joined #openstack-ironic | 13:51 | |
jroll | it sounds like it works with older lsblk, just doesn't actually show that field? | 13:51 |
dtantsur | jroll, I've tried with non-existing field, and it failed for me | 13:51 |
jroll | boo. | 13:52 |
dtantsur | jroll, it's not critical for inspection, lets add it later.. | 13:52 |
*** lazy_prince has quit IRC | 13:52 | |
jroll | dtantsur: ok, sounds good | 13:52 |
*** alexpilotti has joined #openstack-ironic | 13:54 | |
openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM firmware info parsing for Lenovo ThinkServers https://review.openstack.org/214623 | 13:55 |
*** pelix has joined #openstack-ironic | 13:56 | |
mrda | Morning Ironic | 13:56 |
*** alexpilotti has quit IRC | 13:58 | |
dtantsur | mrda, morning | 13:58 |
mrda | hey dtantsur | 13:59 |
lucasagomes | mrda, good morning | 14:02 |
jroll | morning mrda :) | 14:02 |
* jroll injects the idea of breakfast tacos into mrda's thought process | 14:02 | |
lucasagomes | what? hah | 14:02 |
lucasagomes | tacos in the morning | 14:02 |
sambetts | I miss breakfast borritos, nom nom :P | 14:03 |
thiagop | morning mrda | 14:03 |
thiagop | morning too rloo jroll | 14:03 |
*** pal has quit IRC | 14:04 | |
jroll | heya thiagop :) | 14:04 |
thiagop | morning BadCub | 14:04 |
jroll | lucasagomes: it's a texas thing, mrda is in texas. it's also delicious | 14:04 |
lucasagomes | heh right | 14:04 |
rloo | hi thiagop, mrda | 14:04 |
lucasagomes | I think I'm used to the brazilian breakfast (which is just fruits, coffee, milk etc...) light stuff | 14:04 |
lucasagomes | even the irish breakfast is just too much for me | 14:05 |
lucasagomes | (eggs, sausage, beans etc...) | 14:05 |
thiagop | tapioca... | 14:05 |
lucasagomes | ++ | 14:05 |
lucasagomes | I love that! | 14:05 |
*** alexpilotti has joined #openstack-ironic | 14:06 | |
NobodyCam | good morning Ironicers | 14:06 |
* jroll loves a nice hearty breakfast | 14:06 | |
sambetts | I've never had tapioca :/ | 14:06 |
jroll | never had an irish breakfast, but they look delicious | 14:06 |
jroll | morning NobodyCam :) | 14:06 |
dtantsur | NobodyCam, morning | 14:06 |
NobodyCam | hey Welcome back and good morning dtantsur :) | 14:07 |
dtantsur | :) | 14:07 |
thiagop | morning NobodyCam | 14:07 |
NobodyCam | mornign jroll thiagop lucasagomes rloo | 14:07 |
lucasagomes | sambetts, it's good man (-: you should try | 14:07 |
NobodyCam | and sambetts | 14:07 |
*** kbyrne has quit IRC | 14:07 | |
sambetts | o/ NobodyCam | 14:07 |
lucasagomes | jroll, yeah, well if it's a brunch I think the irish breakfast works | 14:07 |
rloo | hiya NobodyCam | 14:07 |
lucasagomes | NobodyCam, morning | 14:07 |
NobodyCam | :) | 14:07 |
*** timbyr_ has quit IRC | 14:07 | |
jroll | lucasagomes: here "brunch" implies drinking, is that true there? | 14:08 |
* thiagop totally agree sambetts should try tapioca | 14:08 | |
mrda | breakfast tacos are not my favourite thing | 14:08 |
sambetts | jroll: its ireland everything implies drinking | 14:08 |
mrda | or burritos | 14:08 |
NobodyCam | morning mrda :) | 14:08 |
lucasagomes | jroll, hah I don't think it does, but well 1~2 pints seems OKish at lunch/brunch | 14:08 |
jroll | sambetts: LOL | 14:09 |
jroll | lucasagomes: totally agree, just curious | 14:09 |
mrda | hi thiagop and NobodyCam | 14:09 |
NobodyCam | :) hows the weather in TX mrda | 14:09 |
*** shoutm has quit IRC | 14:10 | |
mrda | It's a little hot and dry, f'sure | 14:10 |
mrda | so in the 90's, compared with home which is raining and 50's | 14:10 |
mrda | but I'll be back there this weekend \o/ | 14:10 |
*** killer_prince is now known as lazy_prince | 14:11 | |
*** kbyrne has joined #openstack-ironic | 14:11 | |
lazy_prince | irenab: regarding your comment on checking bindingextension supported by neutron, can you plz tell me how to do that.. | 14:12 |
lucasagomes | thiagop, dude, why did you bring tapioca to the conversation? I can't find it here it's pretty sad | 14:12 |
lucasagomes | lol | 14:12 |
NobodyCam | :) hope your having a great time down there in the heat | 14:12 |
irenab | lazy_prince: sure, let me check for the code in nova that does it | 14:12 |
thiagop | lucasagomes: maybe I can send you some by mail? | 14:13 |
thiagop | :) | 14:13 |
lucasagomes | thiagop, that would be awesome! hah | 14:13 |
lazy_prince | irenab: cool.. i can pick up the code from Nova then.. | 14:13 |
jlvillal | jroll, Good morning Ironic | 14:13 |
jroll | BadCub: ohai | 14:13 |
NobodyCam | mornign jlvillal | 14:13 |
jlvillal | Any progress on the gate? | 14:13 |
thiagop | morning jlvillal | 14:13 |
openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM firmware info parsing for Lenovo ThinkServers https://review.openstack.org/214623 | 14:14 |
lucasagomes | jlvillal, fix is on the way | 14:14 |
lucasagomes | jlvillal, https://review.openstack.org/#/c/214564/ | 14:15 |
jlvillal | lucasagomes, Woot! I hope this one works :) | 14:15 |
lucasagomes | *fingers crossed* | 14:15 |
*** boris-42 has joined #openstack-ironic | 14:15 | |
thiagop | that seems to resolve the problem... | 14:16 |
jlvillal | That's good news | 14:16 |
irenab | lazy_prince:just track from here: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L701 | 14:16 |
*** mtanino has joined #openstack-ironic | 14:18 | |
openstackgerrit | Merged openstack/ironic-python-agent: Remove execute mode of cloud-config.yaml https://review.openstack.org/210356 | 14:19 |
*** PradeepV has quit IRC | 14:21 | |
*** absubram has joined #openstack-ironic | 14:23 | |
lazy_prince | irenab: I looked at the link. Now, if for some reason, binding extension is not loaded, should we break with an exception..? or is there a way to recover from that..? | 14:24 |
irenab | lazy_prince: I do not think you can recover, since you expect to populate fields that do not exist | 14:25 |
lazy_prince | jroll: what is your suggestion on ^^ | 14:25 |
jroll | lazy_prince: it's probably insane to use neutron without that extension | 14:26 |
lazy_prince | should we be checking or we always assume that binding extension will be loaded and proceed. As the proposal is based on the assumption of binding extension being configured.. | 14:26 |
lazy_prince | so most of the deployments will have binding extensions loaded.. is it..? | 14:27 |
jroll | I have no idea | 14:27 |
jroll | yes, that's my thought | 14:27 |
jroll | but not 100% sure | 14:27 |
jroll | I suspect that is there for pre-ML2 neutron deployments | 14:27 |
lazy_prince | okay.. will poke sukhdev and see his view on this.. | 14:28 |
irenab | jroll: right. you should not assume ML2 only imho | 14:28 |
jroll | :/ | 14:28 |
jroll | irenab: because older neutron or because neutron without ML2 cnofigured? | 14:29 |
irenab | and as novadoes, you cannot assume working with same neutron. it can be restarted with another backend..so need the refresh logic as in nova | 14:29 |
*** alexpilotti has quit IRC | 14:29 | |
jroll | lazy_prince: so your question is, should we check that in ironic? | 14:29 |
irenab | jroll: ml2 is not the only plugin. for example OVN guys decided to provide monolithic plugin… | 14:30 |
lazy_prince | yes.. i guess, its been checked in nova, so we can safely avoid repeating that in ironic.. | 14:30 |
jroll | lazy_prince: yeah, that's my thought, maybe check binding:profile is sane, that would mean nova is using the portbinding extension | 14:31 |
sinval | dtantsur: when you have some time (I know it's difficult) please take a look at that spec: https://review.openstack.org/#/c/187762/, we updated some points we discussed with you in the past, i believe the spec is more mature now, thanks | 14:32 |
dtantsur | sure | 14:32 |
sinval | :) | 14:33 |
lazy_prince | jroll: so the decision is to skip the check..? | 14:34 |
*** mgoddard1 has joined #openstack-ironic | 14:36 | |
*** mgoddard has quit IRC | 14:36 | |
jroll | lazy_prince: it seems fine, I may think about it more | 14:38 |
lazy_prince | k.. then probably I will hold that change for a later patch set.. | 14:38 |
*** jcoufal_ has joined #openstack-ironic | 14:45 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Refactor the essential prop list of inspect driver https://review.openstack.org/214597 | 14:45 |
*** jcoufal has quit IRC | 14:48 | |
JoshNang | dtantsur: for the DIB IPA images, we're just running IPA without the container right? | 14:49 |
lucasagomes | JoshNang, yes | 14:50 |
*** pal has joined #openstack-ironic | 14:50 | |
JoshNang | lucasagomes: thanks! | 14:51 |
JoshNang | dtantsur: i'm fine landing the inspector patch, just wanted to note since we have no tests running IPA on top of ubuntu/fedora, it's not unlikely we'll land code that breaks them in the future :/ | 14:52 |
dtantsur | JoshNang, yep, without container | 14:56 |
dtantsur | as to fedora, inspector job is running with fedora image, once we have a job with IPA, we'll have fedora covered | 14:56 |
*** puranamr has quit IRC | 14:57 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic: Fix bug sending sensor data for drivers w/o management https://review.openstack.org/213025 | 14:58 |
JoshNang | wfm! | 14:59 |
*** logan2 has quit IRC | 15:01 | |
*** alexpilotti has joined #openstack-ironic | 15:03 | |
*** logan2 has joined #openstack-ironic | 15:05 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Intel Node Manager policies support https://review.openstack.org/214657 | 15:08 |
dtantsur | jroll, could you please (re)review https://review.openstack.org/#/c/209889/ once you have some time? | 15:09 |
jroll | dtantsur: yeah, I'll try to get to it | 15:09 |
dtantsur | awesome! | 15:09 |
*** chadix has quit IRC | 15:16 | |
*** coolsvap|away is now known as coolsvap | 15:18 | |
*** pal has quit IRC | 15:19 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: New driver interface for BIOS configuration https://review.openstack.org/209612 | 15:19 |
*** coolsvap is now known as coolsvap|away | 15:22 | |
*** irenab has quit IRC | 15:22 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Overwrite IronicException message https://review.openstack.org/214662 | 15:22 |
*** mgoddard1 has quit IRC | 15:23 | |
*** mgoddard has joined #openstack-ironic | 15:24 | |
*** alexpilotti has quit IRC | 15:28 | |
*** bizarrochristy has joined #openstack-ironic | 15:29 | |
*** irenab has joined #openstack-ironic | 15:31 | |
krotscheck | Ok, I think we have enough votes on the webclient patches to land them. Can a non-jroll core go through and +A as much of the chain as is appropriate here? https://review.openstack.org/#/c/212715/ | 15:33 |
*** bizarrochristy has quit IRC | 15:34 | |
*** bizarrochristy has joined #openstack-ironic | 15:35 | |
*** pal has joined #openstack-ironic | 15:37 | |
jroll | krotscheck: I really think it's ok for me to just +A them | 15:37 |
jroll | if someone else is just trusting existing votes | 15:38 |
jroll | it's the same thing | 15:38 |
krotscheck | jroll: WFM | 15:38 |
krotscheck | I'm just playing CMA | 15:38 |
jroll | oh | 15:38 |
jroll | it's a standard, not a rule :) | 15:38 |
* jroll does it | 15:38 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic: Fix bug sending sensor data for drivers w/o management https://review.openstack.org/213025 | 15:40 |
jroll | krotscheck: done | 15:40 |
jroll | krotscheck: I think it would be quite silly to block you on waiting for an ironic core to suddenly learn the JS build ecosystem | 15:41 |
jroll | just to satisfy the 2x+2 rule | 15:41 |
thiagop | is it me, or the gate is terribly slow today? | 15:41 |
*** bizarrochristy has quit IRC | 15:41 | |
*** pal has quit IRC | 15:44 | |
openstackgerrit | Om Kumar proposed openstack/ironic: Add network provider interface and implementations https://review.openstack.org/139687 | 15:45 |
krotscheck | jroll: Woo! | 15:45 |
*** pal has joined #openstack-ironic | 15:46 | |
*** mgoddard1 has joined #openstack-ironic | 15:46 | |
krotscheck | thiagop: #openstack-infra's the place to ask that. Personally, it looks like the merge-check job is fubar. | 15:46 |
*** mgoddard has quit IRC | 15:46 | |
*** dprince has quit IRC | 15:51 | |
zhenguo | jroll: can you help to review https://review.openstack.org/#/c/204088/ again please, as it's a nova driver patch, needs ironic guys review first. | 15:53 |
*** alexpilotti has joined #openstack-ironic | 15:54 | |
*** Sukhdev has joined #openstack-ironic | 15:54 | |
jroll | zhenguo: on my list | 15:55 |
zhenguo | jroll: thanks | 15:55 |
openstackgerrit | Merged openstack/ironic-webclient: Switched package to prepublish https://review.openstack.org/212715 | 16:01 |
openstackgerrit | Merged openstack/ironic-webclient: Corrected description in package.json https://review.openstack.org/212716 | 16:01 |
openstackgerrit | Merged openstack/ironic-webclient: Updated 'serve' to 'start' https://review.openstack.org/212717 | 16:02 |
openstackgerrit | Merged openstack/ironic-webclient: Restricted tarball to packaged artifacts https://review.openstack.org/212807 | 16:02 |
openstackgerrit | Merged openstack/ironic-webclient: Fixed driver list https://review.openstack.org/212808 | 16:02 |
openstackgerrit | Merged openstack/ironic-webclient: Removed Chassis https://review.openstack.org/212812 | 16:02 |
openstackgerrit | Merged openstack/ironic-webclient: Separated font management https://review.openstack.org/212844 | 16:02 |
openstackgerrit | Merged openstack/ironic-webclient: Switched to using SCSS https://review.openstack.org/213322 | 16:02 |
openstackgerrit | Merged openstack/ironic-webclient: Explicitly include JS libraries https://review.openstack.org/213323 | 16:02 |
openstackgerrit | Merged openstack/ironic-webclient: Removed gulp update_dependencies and gulp bower. https://review.openstack.org/213324 | 16:02 |
*** mgoddard1 has quit IRC | 16:02 | |
openstackgerrit | Merged openstack/ironic-webclient: Removed dependencies. https://review.openstack.org/213325 | 16:02 |
openstackgerrit | Merged openstack/ironic-webclient: Removed reference to github https://review.openstack.org/213341 | 16:02 |
openstackgerrit | Merged openstack/ironic-webclient: Updated all dependencies. https://review.openstack.org/213357 | 16:03 |
openstackgerrit | Merged openstack/ironic-webclient: Added extension note for package maintainers. https://review.openstack.org/213451 | 16:03 |
NobodyCam | wow | 16:03 |
openstackgerrit | Merged openstack/ironic-webclient: Added code coverage report to ironic-webclient https://review.openstack.org/213810 | 16:03 |
*** mgoddard has joined #openstack-ironic | 16:03 | |
*** mestery has joined #openstack-ironic | 16:04 | |
*** mbound has quit IRC | 16:05 | |
*** irenab has quit IRC | 16:07 | |
dtantsur | krotscheck, why not have a separate core team (like inspector does)? | 16:08 |
*** irenab has joined #openstack-ironic | 16:08 | |
jlvillal | dtantsur, We mentioned that last night. I think krotscheck submitted a patch to start the process. | 16:16 |
dtantsur | good | 16:16 |
jlvillal | dtantsur, https://review.openstack.org/#/c/214381/ | 16:17 |
jlvillal | dtantsur, And welcome back! Hope you had a fun PTO | 16:18 |
*** jcoufal_ has quit IRC | 16:18 | |
dtantsur | yeah, thanks :) | 16:18 |
*** dprince has joined #openstack-ironic | 16:19 | |
jlvillal | dtantsur, I see there is a Pycon CZ coming to Brno :) | 16:19 |
dtantsur | oh really? | 16:19 |
jlvillal | And I noticed Portland hosts the next two North American Pycons | 16:19 |
jlvillal | dtantsur, yep | 16:19 |
jlvillal | dtantsur, http://cz.pycon.org/ | 16:20 |
jlvillal | In November | 16:20 |
dtantsur | cool. but knowing locals, it will be useless to go there without knowing czech... | 16:20 |
jlvillal | dtantsur, Hmmm. I would think it would be mostly in English. | 16:21 |
dtantsur | I hope so :) | 16:21 |
*** alexpilotti has quit IRC | 16:23 | |
*** lazy_prince has quit IRC | 16:26 | |
*** penick has joined #openstack-ironic | 16:26 | |
krotscheck | jlvillal, dtantsur: https://review.openstack.org/#/c/214381/ | 16:28 |
cinerama | morning ironic! | 16:28 |
NobodyCam | mornign cinerama | 16:29 |
*** jistr has quit IRC | 16:30 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Expose Node's cleaning_updated_at in the API https://review.openstack.org/213241 | 16:37 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add cleaning_updated_at in the Nodes database https://review.openstack.org/213240 | 16:37 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Keep cleaning_updated_at updated during cleaning https://review.openstack.org/213698 | 16:37 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Periodically checks for nodes being cleaned https://review.openstack.org/213699 | 16:37 |
* lucasagomes is watching the neutron fix for our gate... almost merged folks | 16:38 | |
lucasagomes | gate's slow! | 16:38 |
rloo | hi lucasagomes, wrt 213240. when/what is provision_updated_at used for? | 16:38 |
lucasagomes | rloo, that's used for 2 things | 16:38 |
lucasagomes | well maybe 1 | 16:38 |
lucasagomes | when provision_state is updated | 16:39 |
*** puranamr has joined #openstack-ironic | 16:39 | |
lucasagomes | and also we update it on the agent's heartbeat when the deployment is happening there (i.e IPA writing the image to the disk) | 16:39 |
lucasagomes | so that the Ironic conductor knows that the deployment is going on | 16:39 |
lucasagomes | rloo, so it's basically an indication that the deployment of the node is up and running | 16:40 |
*** puranamr has quit IRC | 16:40 | |
lucasagomes | (say we started the deployment but the ramdisk never came up, that's how we will time it out) | 16:40 |
rloo | lucasagomes: your code to add cleaning_updated_at -- that will basically have the same value as provision_updated_at because you're setting cleaning_updated_at when the provision state changes to CLEAN or CLEANWAIT? | 16:40 |
SpamapS | iLO 3 Standard 1.20 at Mar 14 2011 | 16:40 |
SpamapS | greghaynes: ^ ugggh | 16:40 |
SpamapS | thats oold | 16:40 |
*** puranamr has joined #openstack-ironic | 16:41 | |
lucasagomes | rloo, as well... but look at other patches in the series | 16:41 |
lucasagomes | there's a touch_cleanning() method | 16:41 |
greghaynes | SpamapS: haha | 16:41 |
lucasagomes | which updates that field while cleaning is running | 16:41 |
SpamapS | greghaynes: that one is locked up :-P | 16:41 |
lucasagomes | rloo, https://review.openstack.org/#/c/213698/3/ironic/drivers/modules/agent_base_vendor.py | 16:41 |
greghaynes | SpamapS: theres a couple that were noted as locked up which we need to file tickets for | 16:41 |
rloo | lucasagomes: but that's similar to us updating provision_updated_at when deploying/heartbeating? | 16:41 |
lucasagomes | rloo, yes | 16:42 |
lucasagomes | jsut saying the values will be different on both fields | 16:42 |
rloo | lucasagomes: I would rather avoid adding a new field for cleaning_updated_at when we could use provision_updated_at | 16:42 |
lucasagomes | rloo, right... I did it because cleaning is not provisioning? | 16:43 |
lucasagomes | so, actually using it sounded a bit misleading | 16:43 |
rloo | lucasagomes: well, that depends on what 'provision' means. cuz we have provision_state=cleaning! | 16:43 |
lucasagomes | rloo, but yeah I can change the code to do that | 16:43 |
lucasagomes | (that will actually simplify it | 16:43 |
lucasagomes | )* | 16:43 |
lucasagomes | rloo, right... ok | 16:43 |
lucasagomes | JoshNang, ^ you OK with that too? | 16:43 |
rloo | lucasagomes: thx | 16:43 |
lucasagomes | rloo, I will mark the series as WIP | 16:44 |
rloo | thx lucasagomes. do/should i put a -2 on the first one, just to make my stats look better? just kidding. | 16:44 |
lucasagomes | rloo, marked it as WIP | 16:44 |
*** pelix has quit IRC | 16:44 | |
rloo | lucasagomes: although i'll add a comment so others will understand. | 16:44 |
lucasagomes | ack | 16:45 |
SpamapS | greghaynes: oh, and yeah, that one has verified: false in your inventory | 16:45 |
SpamapS | greghaynes: perhaps gh2bf.py should skip thos | 16:46 |
SpamapS | those | 16:46 |
*** romainh has quit IRC | 16:46 | |
SpamapS | ugh, that would give me only 22 boxes | 16:47 |
greghaynes | SpamapS: that includes the nodes there were shut down 'just because' | 16:47 |
*** Marga_ has quit IRC | 16:47 | |
*** praneshp has joined #openstack-ironic | 16:47 | |
SpamapS | greghaynes: ok.. so thats no help. ;) | 16:47 |
*** romcheg has quit IRC | 16:47 | |
jlvillal | lucasagomes: +1 on slow gate :( Plus a parent job just failed ;( | 16:49 |
*** alexpilotti has joined #openstack-ironic | 16:49 | |
*** praneshp_ has joined #openstack-ironic | 16:50 | |
lucasagomes | jlvillal, damn | 16:50 |
*** derekh has quit IRC | 16:51 | |
*** praneshp has quit IRC | 16:51 | |
*** praneshp_ is now known as praneshp | 16:51 | |
JoshNang | lucasagomes: hmm | 16:52 |
JoshNang | lucasagomes: i think it sounds ok to just update the provision_updated_at field. i'm trying to think if there's a time we'd care about the total length of cleaning, which we could get if we don't update provision_state_updated_at.. | 16:53 |
JoshNang | well..nvm. now that we have clean_wait (so that'll get updated..) i guess that's not an option anyway | 16:53 |
lucasagomes | JoshNang, right, yeah that's an info that we lose by using the same field | 16:54 |
JoshNang | so, ok! works for me! | 16:54 |
lucasagomes | JoshNang, ok | 16:54 |
rloo | JoshNang: you can't get it from provision_state_updated_at. it gets updated when the provision state changes to cleaning/cleanwait. | 16:54 |
lucasagomes | JoshNang, if the total time is a problem as well it can be extended as part of other patches | 16:54 |
JoshNang | rloo: right | 16:55 |
lucasagomes | inspecting there's a started_at and finished_at field | 16:55 |
JoshNang | that might be handy info for operators, but something we can add later | 16:55 |
openstackgerrit | Merged openstack/ironic-python-agent: Add base for IPA functional tests https://review.openstack.org/213249 | 16:55 |
rloo | JoshNang: i think if you want total time, we should replace eg inspector_started_at etc to be more generic, or add a clean_started_at etc. | 16:55 |
JoshNang | \o/ funcational tests! | 16:55 |
lucasagomes | JoshNang, sure yeah, in the scope of that bug the problem is the timeout and not really the total time | 16:55 |
lucasagomes | but +1 for that enhancement | 16:55 |
JoshNang | ++ | 16:56 |
*** meghal has joined #openstack-ironic | 16:59 | |
*** pelix has joined #openstack-ironic | 17:01 | |
*** pal has quit IRC | 17:05 | |
dtantsur | JoshNang, these new functional tests fail and hang for me locally... any ways to disable them without messing with tox.ini? | 17:12 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Extend hardware manager with data needed for inspector https://review.openstack.org/209889 | 17:12 |
JoshNang | dtantsur: :| | 17:13 |
JoshNang | umm, let me make a thing | 17:13 |
*** mestery has quit IRC | 17:14 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Use addCleanup instead of tearDown https://review.openstack.org/214717 | 17:16 |
dtantsur | JoshNang, ^^ | 17:16 |
JoshNang | dtantsur: ahhh | 17:16 |
dtantsur | JoshNang, and it fails because 2 seconds is not enough for me. mind if I bump it to say 5 seconds? | 17:17 |
rloo | dtantsur: ha ha. I wondered if 2 secs was enough. | 17:17 |
JoshNang | dtantsur: yeah, 5 is fine | 17:17 |
*** ndipanov has joined #openstack-ironic | 17:17 | |
*** ndipanov_ has joined #openstack-ironic | 17:17 | |
JoshNang | i'm gonna break it out into its own tox env as well | 17:17 |
*** ndipanov_ has quit IRC | 17:18 | |
*** natorious is now known as zz_natorious | 17:18 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Bump wait timeout in functional tests to 5 seconds https://review.openstack.org/214718 | 17:18 |
JoshNang | we kinda wondered, but i was like "nahhh ipa starts quickly!" :( | 17:18 |
dtantsur | lol | 17:18 |
dtantsur | ok, I've put patches up, now calling it a day :) | 17:18 |
dtantsur | see you tomorrow | 17:18 |
rloo | ciao dtantsur | 17:19 |
JoshNang | dtantsur: thanks for the patches! i'll break up tox today. have a good night! | 17:19 |
*** dtantsur is now known as dtantsur|afk | 17:19 | |
*** alexpilotti has quit IRC | 17:22 | |
*** pelix has quit IRC | 17:23 | |
thiagop | night dtantsur|afk | 17:23 |
*** puranamr has quit IRC | 17:27 | |
*** puranamr has joined #openstack-ironic | 17:27 | |
openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Removed X-Client header https://review.openstack.org/214406 | 17:28 |
lucasagomes | folks I'm going to call it a day as well | 17:29 |
lucasagomes | have a good night everyone | 17:29 |
*** lucasagomes is now known as lucas-hungry | 17:29 | |
JoshNang | o/ | 17:29 |
*** romcheg has joined #openstack-ironic | 17:30 | |
thiagop | good night lucas-hungry | 17:30 |
*** max_lobur has quit IRC | 17:36 | |
*** krtaylor has quit IRC | 17:37 | |
*** zz_natorious is now known as natorious | 17:39 | |
NobodyCam | night lucas-hungry | 17:40 |
*** achanda has joined #openstack-ironic | 17:42 | |
*** natorious is now known as zz_natorious | 17:45 | |
*** romcheg has quit IRC | 17:47 | |
NobodyCam | TheJulia: happen to have a free minute for https://review.openstack.org/#/c/214192 ? | 17:47 |
*** ihrachyshka has quit IRC | 17:48 | |
*** alexpilotti has joined #openstack-ironic | 17:49 | |
openstackgerrit | James Penick proposed openstack/ironic-specs: No IPA to conductor communication https://review.openstack.org/212206 | 17:50 |
*** zz_natorious is now known as natorious | 17:51 | |
TheJulia | NobodyCam: will in a little bit, wrapping my brain around something else at the moment | 17:51 |
NobodyCam | ack :) | 17:52 |
*** achanda_ has joined #openstack-ironic | 17:54 | |
*** Sukhdev has quit IRC | 17:54 | |
*** achanda has quit IRC | 17:55 | |
openstackgerrit | James Penick proposed openstack/ironic-specs: No IPA to conductor communication https://review.openstack.org/212206 | 17:55 |
*** Sukhdev has joined #openstack-ironic | 17:56 | |
* jlvillal hopes that Neutron patch actually merges in 52 minutes as Zuul is predicting... | 17:56 | |
TheJulia | NobodyCam: reviewed | 17:57 |
NobodyCam | :) | 17:57 |
cinerama | rereviewed | 17:58 |
openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Split functional and py27 tests https://review.openstack.org/214735 | 17:59 |
openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Sending OneView Client implementation https://review.openstack.org/212796 | 18:03 |
* jlvillal Imagines JoshNang is grumbling about jlvillal ... | 18:07 | |
openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Sending OneView Client implementation https://review.openstack.org/212796 | 18:09 |
JoshNang | jlvillal: nah, that's a good point. copy/paste fail! | 18:10 |
jlvillal | JoshNang: Thanks. I learned about .testr.conf for the first time. | 18:11 |
JoshNang | heh, i was vaguely aware of it, but how often do you really need to go in there? | 18:12 |
jlvillal | JoshNang: So far in my entire lifetime... Zero times ;) | 18:12 |
JoshNang | as it should be! | 18:12 |
JoshNang | set it up once, shouldn't need to touch it again | 18:13 |
*** puranamr has quit IRC | 18:13 | |
jlvillal | Yep | 18:13 |
*** krtaylor has joined #openstack-ironic | 18:13 | |
betherly | Ok I'm off home! Have a great day everyone :) | 18:13 |
JoshNang | o/ | 18:13 |
openstackgerrit | Merged openstack/bifrost: Fix typos in bifrost docs https://review.openstack.org/214192 | 18:13 |
*** puranamr has joined #openstack-ironic | 18:13 | |
jlvillal | betherly: Good night | 18:13 |
NobodyCam | night betherly | 18:14 |
thiagop | good ugt night betherly | 18:14 |
openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Split functional and py27 tests https://review.openstack.org/214735 | 18:14 |
* jlvillal 'betherly' reminds him of 'loverly'. Yes jlvillal does watch musicals! | 18:14 | |
* betherly "all she wants is a room somewhere... Tralala!" | 18:15 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Split functional and py27 tests https://review.openstack.org/214735 | 18:16 |
* thiagop thinks musicals are strage | 18:17 | |
thiagop | strange* | 18:17 |
jlvillal | betherly: :) | 18:17 |
*** puranamr has quit IRC | 18:18 | |
*** puranamr has joined #openstack-ironic | 18:18 | |
*** natorious is now known as zz_natorious | 18:18 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Split functional and py27 tests https://review.openstack.org/214735 | 18:21 |
JoshNang | jlvillal: ^ i think i finally got rid of all the trailing whitespace :P | 18:21 |
jlvillal | Woot! | 18:21 |
*** penick has quit IRC | 18:22 | |
*** max_lobur has joined #openstack-ironic | 18:22 | |
jlvillal | JoshNang: I assume there will be a follow on patch to add it to the gate? The functional testing that is. | 18:22 |
openstackgerrit | Merged openstack/bifrost: Test script to drive image testing https://review.openstack.org/207184 | 18:22 |
*** alexpilotti has quit IRC | 18:23 | |
JoshNang | jlvillal: yup. probably won't get to it until tomorrow though | 18:23 |
jlvillal | Excellent. Thanks. | 18:24 |
jlvillal | JoshNang: What version of Python do functional tests run under? | 18:26 |
*** max_lobur has quit IRC | 18:26 | |
krotscheck | There's an ironic driver for nova, yes? | 18:26 |
JoshNang | krotscheck: yup | 18:26 |
* krotscheck is in a meeting with people who want to do ironic things in horizon | 18:26 | |
krotscheck | Cool. | 18:26 |
jlvillal | Wondering if it should be 'func27' 'func34' or something like that? | 18:26 |
JoshNang | krotscheck: https://github.com/openstack/nova/tree/master/nova/virt/ironic | 18:26 |
krotscheck | I suspect I'm going to recommend to them that they do that. | 18:27 |
krotscheck | instead of reimplementing ironic things in horizon | 18:27 |
JoshNang | jlvillal: i think unittesting in py34 and py27 should cover "does this code run under either". functional is "when i call the api, does this side effect happen and i get a proper api response"..if different versions of python break that, we've failed our unittesting (IMO) | 18:29 |
*** max_lobur has joined #openstack-ironic | 18:30 | |
jlvillal | JoshNang: Makes sense to me. I'll leave as +0 for now, see if other comments. I could go either way. | 18:30 |
jroll | JoshNang: +1 | 18:30 |
* jlvillal wonders if it is bring your child to work day today... | 18:30 | |
*** bapalm has quit IRC | 18:31 | |
*** bapalm has joined #openstack-ironic | 18:32 | |
betherly | krotscheck: looks like that would work well then! Happy days | 18:35 |
*** Marga_ has joined #openstack-ironic | 18:35 | |
*** MattMan has left #openstack-ironic | 18:46 | |
*** alexpilotti has joined #openstack-ironic | 18:49 | |
*** romcheg has joined #openstack-ironic | 18:49 | |
*** puranamr has quit IRC | 18:50 | |
*** puranamr has joined #openstack-ironic | 18:50 | |
*** puranamr has quit IRC | 18:52 | |
*** puranamr has joined #openstack-ironic | 18:52 | |
thiagop | guys, I'm having some trouble booting an instance with the CoreOS images | 18:57 |
thiagop | it keep throwing "PTE Read access is not set" on the terminal | 18:57 |
thiagop | somebody has a clue? | 18:58 |
*** pal has joined #openstack-ironic | 18:58 | |
JoshNang | thiagop: haven't run across that, but a quick google search suggests its not necessarily related to coreos/ironic | 19:04 |
thiagop | I'm seeing something about intel_iommu | 19:04 |
thiagop | rcarrillocruz: ping | 19:08 |
rcarrillocruz | thiagop: yup | 19:08 |
rcarrillocruz | i pushed a change to accept kernel boot params | 19:09 |
rcarrillocruz | just to circumenvent that, i hit that myself | 19:09 |
rcarrillocruz | you need | 19:09 |
rcarrillocruz | intel_iommu=off | 19:09 |
*** max_lobur has quit IRC | 19:09 | |
thiagop | where do I put that? pxe_append_params? | 19:10 |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Lesson degree of admin backoff https://review.openstack.org/214761 | 19:10 |
* jlvillal goes for lunch. Hopes neutron patch will be merged by the time he gets back. | 19:13 | |
*** alexpilotti has quit IRC | 19:21 | |
*** Sukhdev has quit IRC | 19:23 | |
rloo | NobodyCam, BadCub: if you're around, would one of you please update the 'Read the full specification' link in https://blueprints.launchpad.net/ironic/+spec/ironic-generic-raid-interface, to https://git.openstack.org/cgit/openstack/ironic-specs/tree/specs/approved/ironic-generic-raid-interface.rst | 19:23 |
rloo | jroll: the spec links are a mess. We put spec links in the BP, eg ^^, but once implemented, the spec will move from approved to *-implemented and the link will be broken :-( | 19:24 |
jroll | rloo: I hate launchpad so much | 19:25 |
rloo | jroll: I *just* want a quick way to get a spec when reviewing the patches :-( | 19:25 |
BadCub | rloo: ^^ done :) | 19:26 |
jroll | rloo: totally agree :/ | 19:26 |
*** max_lobur has joined #openstack-ironic | 19:26 | |
rloo | thx BadCub! | 19:26 |
BadCub | yw :) | 19:26 |
rloo | jroll: we basically need a pointer to the spec, and we update the pointer as needed. or we put the spec in one place and never move it. or ... Anyway, head-in-sand-and-reviewing... | 19:27 |
jroll | rloo: yeah, plan is old location would point to new | 19:28 |
thiagop | gerrit could have a way to indicate a spec instead of a blueprint... | 19:28 |
rloo | jroll: but we just changed the approved ones to 'approved'. maybe we need to put them back to the best-guess-release, 'liberty'... | 19:29 |
jroll | rloo: and we put placeholders there | 19:29 |
jroll | where they were before | 19:29 |
rloo | jroll: or we leave placeholders in approved | 19:30 |
jroll | rloo: but we don't know when they'll be implemented | 19:30 |
rloo | jroll: i know. that's the problem. i wonder what other projects do. maybe they ignore the problem. | 19:31 |
rloo | jroll: like i'm going to do now. i feel like i passed the potato to you... | 19:31 |
rcarrillocruz | thiagop: extra_kernel_options variable, from ironic_install role | 19:32 |
jroll | rloo: haha. | 19:32 |
rcarrillocruz | that ^ bifrost btw | 19:32 |
thiagop | rcarrillocruz: Ok, I'm using devstack for now :) | 19:32 |
rcarrillocruz | k | 19:32 |
rcarrillocruz | then | 19:32 |
rcarrillocruz | give me a sec | 19:33 |
* TheJulia tries to collect context | 19:33 | |
thiagop | rcarrillocruz: seems that pxe_append_params works for me | 19:33 |
thiagop | double checking now | 19:33 |
rcarrillocruz | TheJulia: btw, i'm back from vaca and from oncall stuff tomorrow | 19:37 |
rcarrillocruz | how are things? | 19:37 |
rcarrillocruz | you started the ironic shade stuff for rebuilding? | 19:37 |
*** pal has quit IRC | 19:37 | |
rcarrillocruz | if not, i can take a stab at it | 19:37 |
*** achanda_ has quit IRC | 19:37 | |
*** alexpilotti has joined #openstack-ironic | 19:38 | |
*** alexpilotti has quit IRC | 19:42 | |
*** Sukhdev has joined #openstack-ironic | 19:43 | |
*** ndipanov has quit IRC | 19:43 | |
*** e0ne has joined #openstack-ironic | 19:45 | |
*** zz_natorious is now known as natorious | 19:45 | |
*** e0ne has quit IRC | 19:47 | |
*** alexpilotti has joined #openstack-ironic | 19:50 | |
*** achanda has joined #openstack-ironic | 19:51 | |
*** natorious is now known as zz_natorious | 19:52 | |
*** puranamr has quit IRC | 19:55 | |
*** puranamr has joined #openstack-ironic | 19:56 | |
*** meghal has quit IRC | 20:00 | |
*** e0ne has joined #openstack-ironic | 20:02 | |
TheJulia | rcarrillocruz: so shade needs no changes | 20:07 |
TheJulia | node_set_provision_state is just a pass-through, although I will be adding wait very soon | 20:07 |
*** puranamr has quit IRC | 20:08 | |
*** puranamr has joined #openstack-ironic | 20:08 | |
thiagop | TheJulia: I found a bug where you said you logged in into the coreos deploy image. Does it have a defult user/pass? | 20:16 |
jlvillal | jroll, rloo NobodyCam, thiagop and everyone else. Neutron patch ( https://review.openstack.org/#/c/214564/ ) got merged. Hopefully it unbreaks the gate. | 20:17 |
thiagop | wow | 20:17 |
thiagop | jlvillal: awesome | 20:17 |
rloo | jlvillal: thx. here's hopin'... Anyone see our gates workin', let us know! | 20:17 |
*** e0ne has quit IRC | 20:18 | |
jlvillal | thiagop, Only took 5 hours and 50 minutes to go through the gate... Is it just me or does that seem like a long time. | 20:18 |
TheJulia | thiagop: coreos, you need to feed in an ssh key and login with the coreos user. | 20:18 |
openstackgerrit | Merged openstack/ironic-python-agent: Use addCleanup instead of tearDown https://review.openstack.org/214717 | 20:18 |
TheJulia | thiagop: Try the troubleshooting doc | 20:18 |
TheJulia | I think its in there | 20:19 |
*** alexpilotti has quit IRC | 20:21 | |
*** r-daneel has joined #openstack-ironic | 20:24 | |
*** max_lobur has quit IRC | 20:24 | |
thiagop | TheJulia: Ok | 20:26 |
*** e0ne has joined #openstack-ironic | 20:29 | |
*** albertoffb has quit IRC | 20:33 | |
jroll | jlvillal: woot | 20:34 |
*** e0ne has quit IRC | 20:35 | |
*** e0ne has joined #openstack-ironic | 20:36 | |
*** meghal has joined #openstack-ironic | 20:39 | |
*** achanda has quit IRC | 20:47 | |
*** rloo has quit IRC | 20:48 | |
*** rloo has joined #openstack-ironic | 20:48 | |
openstackgerrit | Tomoki Sekiyama proposed openstack/python-ironicclient: Add volume_connector support https://review.openstack.org/214786 | 20:49 |
*** achanda has joined #openstack-ironic | 20:52 | |
*** alexpilotti has joined #openstack-ironic | 20:57 | |
*** meghal has quit IRC | 20:59 | |
*** meghal has joined #openstack-ironic | 20:59 | |
*** meghal has quit IRC | 20:59 | |
*** meghal has joined #openstack-ironic | 21:00 | |
*** Marga_ has quit IRC | 21:06 | |
*** achanda has quit IRC | 21:06 | |
jaypipes | jroll: the CoreOS image that is built in the ironic-python-agent repository... is that *just* for hardware discovery purposes, or is that actually used for the baremetal tenant nodes' operating system? (sorry if this is an odd question or may seem like I don't understand something basic here... I very well could be completely lost.) | 21:13 |
*** amotoki_ has quit IRC | 21:14 | |
jroll | jaypipes: it's primarily for the deployment of the instance | 21:14 |
jroll | jaypipes: right now there's some work going on for hardware discovery things via ironic-inspector | 21:14 |
jroll | thiagop: +2 with a couple questions on https://review.openstack.org/#/c/187762/ | 21:15 |
jaypipes | jroll: ok. could you walk me through exactly what the IPA does with that coreos image though? does the BM node netboot/PXE boot into that coreos image and then a *different* tenant image is pulled down (from swift or some other location) and then the BM node is rebooted into that other image? | 21:15 |
*** e0ne has quit IRC | 21:15 | |
jroll | thiagop: also, going to do vmedia eventually? (just curious) | 21:16 |
thiagop | jroll: oh, wow! thanks! | 21:16 |
jroll | jaypipes: spot on. + configdrive | 21:16 |
thiagop | jroll: that's on our "next steps" | 21:16 |
thiagop | vmedia | 21:16 |
jaypipes | jroll: OK, thank you sir. wanted to make sure I wasn't nuts. :) | 21:16 |
jaypipes | jroll: and the network_data.json stuff from JoshNang is to allow network configuration/setup *without* config-drive support, yes? | 21:17 |
*** krtaylor has quit IRC | 21:18 | |
jaypipes | i.e. so cloud-init can "do its thang"? | 21:18 |
jroll | jaypipes: network_data.json goes in the configdrive (or metadata service) | 21:18 |
jroll | jaypipes: and then yeah cloud-init or glean or whatever reads that and does stuff | 21:18 |
jaypipes | jroll: got it. all clear. thanks again, really appreciated :) | 21:18 |
jroll | np :) | 21:18 |
thiagop | jroll: answering... | 21:19 |
*** e0ne has joined #openstack-ironic | 21:19 | |
JoshNang | yup, spot on | 21:19 |
thiagop | jroll: in the comment about power off/on, you mean change the actual AgentVendorInterface method to do what we intend to do? | 21:21 |
jroll | thiagop: yes | 21:24 |
*** alexpilotti has quit IRC | 21:25 | |
*** ndipanov has joined #openstack-ironic | 21:26 | |
*** meghal has quit IRC | 21:26 | |
*** meghal has joined #openstack-ironic | 21:27 | |
thiagop | jroll: uhmmm... if you think it is indeed useful, we can do that in a blink and submit submit a following patch to remove the custom vendor interface (assuming it shall be merged before the change) | 21:27 |
*** meghal has quit IRC | 21:27 | |
*** meghal has joined #openstack-ironic | 21:27 | |
*** e0ne has quit IRC | 21:28 | |
jroll | thiagop: +1 | 21:32 |
thiagop | jroll: ok. answered the review just to keep record. ;) | 21:32 |
*** jjohnson2 has quit IRC | 21:33 | |
jroll | thanks! | 21:34 |
openstackgerrit | Ruby Loo proposed openstack/ironic: Reset clean_step if error occurs in CLEANWAIT https://review.openstack.org/214808 | 21:38 |
thiagop | Calling it a day folks | 21:41 |
TheJulia | goodnight | 21:41 |
thiagop | good night! | 21:41 |
jlvillal | thiagop: Good night | 21:42 |
*** Sukhdev has quit IRC | 21:43 | |
*** romcheg1 has joined #openstack-ironic | 21:45 | |
*** thiagop has quit IRC | 21:45 | |
*** romcheg has quit IRC | 21:47 | |
*** thrash is now known as thrash|g0ne | 21:47 | |
*** romcheg has joined #openstack-ironic | 21:50 | |
*** sambetts has quit IRC | 21:51 | |
*** romcheg1 has quit IRC | 21:51 | |
*** sambetts has joined #openstack-ironic | 21:51 | |
*** alexpilotti has joined #openstack-ironic | 21:52 | |
*** meghal has left #openstack-ironic | 21:53 | |
*** dprince has quit IRC | 21:59 | |
*** krtaylor has joined #openstack-ironic | 22:00 | |
*** absubram has quit IRC | 22:03 | |
*** sinval_ has joined #openstack-ironic | 22:08 | |
openstackgerrit | John L. Villalovos proposed stackforge/pyghmi: Lessen degree of admin backoff https://review.openstack.org/214761 | 22:13 |
*** Sukhdev has joined #openstack-ironic | 22:13 | |
*** romcheg has quit IRC | 22:14 | |
*** amotoki_ has joined #openstack-ironic | 22:14 | |
*** amotoki_ has quit IRC | 22:19 | |
openstackgerrit | Merged openstack/ironic-python-agent: Cleaning with custom hardware manager docs https://review.openstack.org/200307 | 22:20 |
*** alexpilotti has quit IRC | 22:27 | |
jlvillal | jroll: rloo: The gate is unbroken :) | 22:33 |
jlvillal | e.g. https://review.openstack.org/#/c/213856/ | 22:34 |
rloo | jlvillal: Yay! | 22:34 |
rloo | jlvillal: for your help in the matter, i'll review that patch of yours :) | 22:35 |
jroll | woot, thanks jlvillal | 22:35 |
jlvillal | rloo: Woo hoo :) | 22:43 |
jlvillal | jroll: Glad to help. Which seems to have been mainly monitoring zuul all day :P | 22:43 |
jlvillal | I'm glad the neutron folks got a fix in | 22:44 |
jroll | hah | 22:44 |
jlvillal | jroll: I just thanked them for fixing it in #openstack-neutron | 22:46 |
NobodyCam | jlvillal: Ty for that | 22:46 |
jlvillal | NobodyCam: :) | 22:47 |
NobodyCam | :) | 22:47 |
*** r-daneel has quit IRC | 22:49 | |
*** baoli has joined #openstack-ironic | 22:51 | |
*** baoli has quit IRC | 22:56 | |
*** bnemec has quit IRC | 22:56 | |
rloo | jlvillal: not sure I did you a favour but anyway... | 23:02 |
*** alexpilotti has joined #openstack-ironic | 23:03 | |
jlvillal | rloo: Thanks for the feedback! | 23:03 |
rloo | jlvillal: :) | 23:03 |
rloo | jlvillal: I updated the whiteboard wrt gate status. I left the info about the neutron stuff, just in case. if things are good, we can delete that stuff tomorrow. | 23:05 |
jlvillal | rloo: thx | 23:05 |
*** Sukhdev has quit IRC | 23:05 | |
*** lucas-hungry has quit IRC | 23:08 | |
*** rloo has quit IRC | 23:13 | |
*** amotoki_ has joined #openstack-ironic | 23:15 | |
*** amotoki_ has quit IRC | 23:19 | |
*** bnemec has joined #openstack-ironic | 23:20 | |
*** chlong has quit IRC | 23:28 | |
*** alexpilotti has quit IRC | 23:30 | |
*** achanda has joined #openstack-ironic | 23:33 | |
*** shoutm has joined #openstack-ironic | 23:37 | |
*** puranamr has quit IRC | 23:39 | |
*** bnemec has quit IRC | 23:43 | |
*** bnemec has joined #openstack-ironic | 23:45 | |
*** ndipanov has quit IRC | 23:45 | |
*** Haomeng|2 has joined #openstack-ironic | 23:55 | |
*** alexpilotti has joined #openstack-ironic | 23:56 | |
*** bnemec has quit IRC | 23:58 | |
*** Haomeng has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!