*** sabeen has quit IRC | 00:02 | |
*** baoli has joined #openstack-ironic | 00:22 | |
*** hoangcx2 has quit IRC | 00:22 | |
*** morgabra has quit IRC | 00:41 | |
*** morgabra has joined #openstack-ironic | 00:41 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-inspector: Fix formatting strings in LOG.error https://review.openstack.org/361148 | 00:42 |
---|---|---|
*** fragatina has joined #openstack-ironic | 00:48 | |
*** hoangcx has joined #openstack-ironic | 00:48 | |
*** Goneri has joined #openstack-ironic | 01:00 | |
*** bfournie has joined #openstack-ironic | 01:08 | |
*** vdrok has quit IRC | 01:09 | |
*** serverascode has quit IRC | 01:10 | |
*** TheJulia has quit IRC | 01:10 | |
*** tdurakov has quit IRC | 01:10 | |
*** betherly has quit IRC | 01:11 | |
*** mtanino has joined #openstack-ironic | 01:16 | |
*** TheJulia has joined #openstack-ironic | 01:19 | |
*** vdrok has joined #openstack-ironic | 01:25 | |
*** sdake_ has quit IRC | 01:25 | |
*** tdurakov has joined #openstack-ironic | 01:26 | |
*** serverascode has joined #openstack-ironic | 01:26 | |
*** betherly has joined #openstack-ironic | 01:27 | |
*** sdake has joined #openstack-ironic | 01:30 | |
*** baoli has quit IRC | 01:36 | |
*** baoli has joined #openstack-ironic | 01:44 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Use namedtuple to improve code readability https://review.openstack.org/361788 | 01:44 |
*** Goneri has quit IRC | 01:46 | |
*** baoli has quit IRC | 01:49 | |
*** Goneri has joined #openstack-ironic | 01:52 | |
*** baoli has joined #openstack-ironic | 01:56 | |
*** vishwanathj has joined #openstack-ironic | 02:01 | |
*** annp has joined #openstack-ironic | 02:10 | |
*** vishwanathj is now known as vishwanathj_zzz | 02:10 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: WIP: DO NOT REVIEW (namedtuple playground) https://review.openstack.org/361793 | 02:17 |
openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: WIP: DO NOT REVIEW (namedtuple playground) https://review.openstack.org/361793 | 02:19 |
*** amotoki has joined #openstack-ironic | 02:24 | |
*** Goneri has quit IRC | 02:30 | |
*** vishwanathj_zzz has quit IRC | 02:33 | |
*** vishwanathj_zzz has joined #openstack-ironic | 02:33 | |
*** vishwanathj_zzz has quit IRC | 02:41 | |
*** vishwanathj_zzz has joined #openstack-ironic | 02:41 | |
*** vishwanathj_zzz has quit IRC | 02:44 | |
*** vishwanathj_zzz has joined #openstack-ironic | 02:44 | |
*** yuanying has quit IRC | 02:51 | |
*** yuanying has joined #openstack-ironic | 02:52 | |
*** vishwanathj_zzz has quit IRC | 02:56 | |
*** vishwanathj_zzz has joined #openstack-ironic | 02:56 | |
*** tiendc has joined #openstack-ironic | 02:59 | |
*** tuanla_ has joined #openstack-ironic | 02:59 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 03:01 |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-python-agent: Using assertIsNone() is preferred over assertEqual() https://review.openstack.org/361811 | 03:10 |
*** amotoki has quit IRC | 03:21 | |
*** amotoki has joined #openstack-ironic | 03:40 | |
*** vikrant has joined #openstack-ironic | 03:41 | |
*** yuanying has quit IRC | 03:44 | |
*** aswadr_ has joined #openstack-ironic | 03:45 | |
*** vikrant is now known as vikrant|brb | 03:58 | |
*** chlong has quit IRC | 04:12 | |
*** vikrant|brb is now known as vikrant | 04:14 | |
*** amotoki has quit IRC | 04:21 | |
*** chlong has joined #openstack-ironic | 04:26 | |
*** links has joined #openstack-ironic | 04:33 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Fix incorrect order of params of assertEqual() https://review.openstack.org/359580 | 04:33 |
*** amotoki has joined #openstack-ironic | 04:42 | |
*** fragatin_ has joined #openstack-ironic | 04:45 | |
*** fragati__ has joined #openstack-ironic | 04:48 | |
*** fragatina has quit IRC | 04:48 | |
*** amotoki has quit IRC | 04:48 | |
*** fragatin_ has quit IRC | 04:50 | |
*** jaosorior has joined #openstack-ironic | 04:52 | |
*** fragati__ has quit IRC | 04:53 | |
*** vishwanathj_zzz has quit IRC | 05:01 | |
*** vishwanathj_zzz has joined #openstack-ironic | 05:01 | |
*** amotoki has joined #openstack-ironic | 05:03 | |
*** yuanying has joined #openstack-ironic | 05:03 | |
*** sdake_ has joined #openstack-ironic | 05:10 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-python-agent: Using assertIsNone() is preferred over assertEqual() https://review.openstack.org/361811 | 05:10 |
*** sdake_ is now known as dake | 05:13 | |
*** sdake has quit IRC | 05:13 | |
*** dake is now known as sdake | 05:13 | |
*** mtanino has quit IRC | 05:15 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-python-agent: Using assertIsNone() is preferred over assertEqual() https://review.openstack.org/361811 | 05:22 |
*** sdake has quit IRC | 05:26 | |
openstackgerrit | Aparna proposed openstack/proliantutils: Raise exception when HPSSA controller not available https://review.openstack.org/347791 | 05:54 |
*** ChubYann has quit IRC | 05:57 | |
*** zhangjl has joined #openstack-ironic | 06:02 | |
*** watanabe_isao has joined #openstack-ironic | 06:06 | |
*** mjura has joined #openstack-ironic | 06:06 | |
*** moshele has joined #openstack-ironic | 06:06 | |
*** mjura has quit IRC | 06:12 | |
*** mjura has joined #openstack-ironic | 06:12 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 06:24 |
*** fragatina has joined #openstack-ironic | 06:25 | |
*** fragatina has quit IRC | 06:26 | |
*** fragatina has joined #openstack-ironic | 06:26 | |
*** yuanying has quit IRC | 06:29 | |
*** amotoki has quit IRC | 06:29 | |
*** praneshp has quit IRC | 06:38 | |
*** rcernin has joined #openstack-ironic | 06:47 | |
*** amotoki has joined #openstack-ironic | 06:50 | |
*** amotoki has quit IRC | 06:58 | |
*** pcaruana has joined #openstack-ironic | 07:00 | |
*** tesseract- has joined #openstack-ironic | 07:01 | |
*** aswadr_ has quit IRC | 07:01 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Remove neutron client workarounds https://review.openstack.org/354588 | 07:08 |
*** ohamada has joined #openstack-ironic | 07:09 | |
*** MattMan has quit IRC | 07:12 | |
*** MattMan has joined #openstack-ironic | 07:12 | |
*** ifarkas_afk is now known as ifarkas | 07:16 | |
*** amotoki has joined #openstack-ironic | 07:19 | |
*** amotoki has quit IRC | 07:36 | |
*** jpich has joined #openstack-ironic | 07:37 | |
*** tiendc has quit IRC | 07:38 | |
*** tiendc has joined #openstack-ironic | 07:40 | |
*** amotoki has joined #openstack-ironic | 07:45 | |
*** Sukhdev has quit IRC | 07:46 | |
*** yuanying has joined #openstack-ironic | 07:51 | |
*** vishwanathj_zzz has quit IRC | 07:53 | |
*** vishwanathj_zzz has joined #openstack-ironic | 07:53 | |
*** zzzeek has quit IRC | 08:00 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 08:00 |
*** zzzeek has joined #openstack-ironic | 08:01 | |
*** mpsairam has quit IRC | 08:03 | |
*** fxpester has quit IRC | 08:04 | |
*** e0ne has joined #openstack-ironic | 08:05 | |
*** links has quit IRC | 08:06 | |
*** derekh has joined #openstack-ironic | 08:12 | |
*** greghaynes has quit IRC | 08:13 | |
*** lucas-afk is now known as lucasagomes | 08:21 | |
lucasagomes | morning all | 08:21 |
*** cmyster has joined #openstack-ironic | 08:21 | |
pas-ha | morning lucasagomes and all :) | 08:23 |
tangchen | morning, all | 08:23 |
cmyster | morning | 08:23 |
vdrok | morning lucasagomes pas-ha tangchen and cmyster | 08:33 |
lucasagomes | hi there | 08:34 |
tangchen | morning, vdrok | 08:34 |
cmyster | hi | 08:34 |
vsaienk0 | Morning lucasagomes, pas-ha, vdrok and all Ironic'ers! | 08:38 |
lucasagomes | o/ | 08:38 |
vdrok | hey vsaienk0 | 08:38 |
*** ashestakov has joined #openstack-ironic | 08:39 | |
*** greghaynes has joined #openstack-ironic | 08:40 | |
tangchen | lucasagomes, vdrok: I have updated functional tests patches. Start from here: https://review.openstack.org/#/c/349421/ | 08:41 |
tangchen | lucasagomes, vdrok: Please help to review when you are free. :) | 08:41 |
*** MattMan_ has joined #openstack-ironic | 08:41 | |
vdrok | tangchen: ok will try to | 08:41 |
tangchen | thx | 08:42 |
*** MattMan_ has quit IRC | 08:42 | |
*** MattMan_ has joined #openstack-ironic | 08:42 | |
*** MattMan_ has quit IRC | 08:42 | |
*** MattMan_ has joined #openstack-ironic | 08:42 | |
*** MattMan_ has quit IRC | 08:44 | |
*** MattMan has quit IRC | 08:44 | |
*** MattMan has joined #openstack-ironic | 08:44 | |
*** MattMan has quit IRC | 08:46 | |
*** MattMan has joined #openstack-ironic | 08:51 | |
ifarkas | morning all! | 08:52 |
*** watanabe_isao has quit IRC | 08:53 | |
*** electrofelix has joined #openstack-ironic | 09:00 | |
vdrok | morning ifarkas | 09:03 |
*** MattMan has quit IRC | 09:08 | |
*** wznoinsk_ is now known as wznoinsk | 09:09 | |
*** MattMan has joined #openstack-ironic | 09:09 | |
*** MattMan has quit IRC | 09:10 | |
*** MattMan has joined #openstack-ironic | 09:11 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix pep8 on Python3.5 https://review.openstack.org/361971 | 09:43 |
*** dtantsur|afk is now known as dtantsur | 09:44 | |
dtantsur | Morning Ironic, morning pas-ha, lucasagomes, tangchen, vdrok, vsaienk0, ifarkas | 09:44 |
lucasagomes | dtantsur, hi there | 09:44 |
*** xuao has joined #openstack-ironic | 09:46 | |
*** xuao has quit IRC | 09:47 | |
*** xuao has joined #openstack-ironic | 09:47 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Configure retrieving logs from the deploy ramdisk https://review.openstack.org/360658 | 09:49 |
*** MattMan has quit IRC | 09:52 | |
*** MattMan has joined #openstack-ironic | 09:54 | |
tangchen | morning, dtantsur | 09:55 |
*** zigo_ is now known as zigo | 09:56 | |
*** vishwanathj_zzz has quit IRC | 09:58 | |
*** vishwanathj_zzz has joined #openstack-ironic | 09:58 | |
vdrok | dtantsur: morning! | 10:21 |
*** tiendc has quit IRC | 10:27 | |
*** hoangcx has quit IRC | 10:30 | |
*** e0ne has quit IRC | 10:32 | |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/360889 | 10:41 |
*** amotoki has quit IRC | 10:41 | |
*** amotoki has joined #openstack-ironic | 10:46 | |
*** amotoki has quit IRC | 10:46 | |
*** amotoki has joined #openstack-ironic | 10:50 | |
*** tiendc has joined #openstack-ironic | 10:59 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 11:00 |
*** amotoki has quit IRC | 11:02 | |
*** tuanla_ has quit IRC | 11:05 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix pep8 on Python3.5 https://review.openstack.org/361971 | 11:07 |
*** ashestakov_ has joined #openstack-ironic | 11:10 | |
*** ashestakov has quit IRC | 11:11 | |
*** ashestakov_ has left #openstack-ironic | 11:16 | |
*** ashestakov has joined #openstack-ironic | 11:16 | |
*** thrash|wknd is now known as thrash | 11:23 | |
*** tiendc has quit IRC | 11:40 | |
*** annp has quit IRC | 11:44 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Wish to help with rebasing this patch https://review.openstack.org/362047 | 11:54 |
*** lucasagomes is now known as lucas-hungry | 11:55 | |
openstackgerrit | Merged openstack/ironic: Fix formatting strings in LOG.error https://review.openstack.org/361127 | 11:58 |
openstackgerrit | Merged openstack/ironic: Update documentation for keystone policy support https://review.openstack.org/345556 | 11:58 |
openstackgerrit | Merged openstack/ironic: Add multi-tenancy section to security doc https://review.openstack.org/345557 | 11:58 |
*** dprince has joined #openstack-ironic | 12:00 | |
*** eset- has quit IRC | 12:01 | |
*** jaosorior has quit IRC | 12:02 | |
*** jaosorior has joined #openstack-ironic | 12:03 | |
*** eset has joined #openstack-ironic | 12:03 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Rely on portgroup individual_mode_supported https://review.openstack.org/360647 | 12:06 |
*** eset has quit IRC | 12:06 | |
*** eset has joined #openstack-ironic | 12:06 | |
*** vishwanathj_zzz has quit IRC | 12:08 | |
*** vishwanathj_zzz has joined #openstack-ironic | 12:08 | |
*** aswadr_ has joined #openstack-ironic | 12:08 | |
mat128 | morning Ironic | 12:12 |
*** amotoki has joined #openstack-ironic | 12:17 | |
*** amotoki has quit IRC | 12:17 | |
*** baoli_ has joined #openstack-ironic | 12:18 | |
vdrok | morning mat128 | 12:20 |
*** baoli has quit IRC | 12:21 | |
*** vikrant has quit IRC | 12:28 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Rely on portgroup individual_mode_supported https://review.openstack.org/360647 | 12:31 |
*** appu103_ has joined #openstack-ironic | 12:32 | |
*** appu103_ is now known as aparnav | 12:32 | |
*** appu1034 has joined #openstack-ironic | 12:36 | |
*** aparnav has quit IRC | 12:36 | |
*** nicodemos has joined #openstack-ironic | 12:36 | |
openstackgerrit | Ankit Kumar proposed openstack/proliantutils: Fix log message for exception in Gen7 https://review.openstack.org/353860 | 12:38 |
*** milan has joined #openstack-ironic | 12:40 | |
milan | morning Ironic! :) | 12:40 |
jaosorior | dtantsur: When are RFEs approved? Wandering about the status of https://review.openstack.org/#/c/358457/ | 12:46 |
*** Goneri has joined #openstack-ironic | 12:50 | |
*** xavierr has joined #openstack-ironic | 12:50 | |
*** ricardoas has joined #openstack-ironic | 12:51 | |
*** amotoki has joined #openstack-ironic | 12:51 | |
jroll | jaosorior: it depends, they usually get looked at within a week or two (or when someone asks), but we're close to the end of the cycle (trying to finish already-in-progress features, and IPA launchpad sometimes gets overlooked a bit | 12:53 |
jroll | morning everyone | 12:53 |
jaosorior | jroll, JayF, dtantsur: well, if you have time, can you take a look at https://review.openstack.org/#/c/358457/ ? | 12:54 |
jroll | jaosorior: yeah, I'll try | 12:56 |
milan | morning jroll! :) | 12:56 |
*** lucas-hungry is now known as lucasagomes | 12:56 | |
jroll | dtantsur: JayF: the RFE is here, btw, I'm okay with it https://bugs.launchpad.net/ironic-python-agent/+bug/1616495 | 12:56 |
openstack | Launchpad bug 1616495 in ironic-python-agent "RFE: Need an alternative to trust a certificate without re-building the IPA image" [Undecided,In progress] - Assigned to Juan Antonio Osorio Robles (juan-osorio-robles) | 12:56 |
dtantsur | jaosorior, what's the RFE bug number? we're approving intentions first :) | 12:56 |
* jroll ninja's dtantsur | 12:57 | |
dtantsur | morning jroll, milan, mat128 | 12:57 |
dtantsur | :) | 12:57 |
jroll | morning :) | 12:57 |
milan | morning dtantsur :) | 12:57 |
lucasagomes | morning all | 12:57 |
dtantsur | jroll, +1 for this RFE. I did have some questions about the implementation, but the intention looks clear. | 12:58 |
jroll | dtantsur: cool | 12:58 |
* jroll approves | 12:58 | |
jroll | JayF: when you're around, you can remove your -2 on https://review.openstack.org/#/c/358457/ :) | 12:58 |
xavierr | morning jroll jaosorior dtantsur lucasagomes | 12:59 |
lucasagomes | hi there | 12:59 |
jaosorior | xavierr: morning (afternoon in my case :P ) | 12:59 |
*** rloo has joined #openstack-ironic | 13:00 | |
jaosorior | dtantsur, jroll thanks for checking it out | 13:00 |
jroll | yeah no problem :) | 13:00 |
*** pcaruana has quit IRC | 13:02 | |
xavierr | lucasagomes: I was reading pixie boot bot code this night :) | 13:03 |
*** sdake has joined #openstack-ironic | 13:03 | |
xavierr | lucasagomes: I was making some magic with RabbitMQ and irc lib for python :D | 13:03 |
dtantsur | morning xavierr | 13:03 |
dtantsur | btw, where is Pixie? | 13:03 |
*** strigazi_AFK is now known as strigazi | 13:05 | |
*** daemontool has joined #openstack-ironic | 13:06 | |
*** Goneri has quit IRC | 13:06 | |
*** ricardoas has quit IRC | 13:06 | |
*** appu1034 has quit IRC | 13:08 | |
*** nicodemos has quit IRC | 13:08 | |
*** xavierr has quit IRC | 13:08 | |
*** ashestakov has quit IRC | 13:11 | |
milan | aarefiev, dtantsur , sambetts|afk hey guys, I've been hesitating about asking further review on the inspector state patch but here I go :) I'm open to land suggestions on that patch before feature freeze, please, let me know what you thing! | 13:13 |
*** zhangjl has quit IRC | 13:13 | |
* dtantsur is still unsure about landing that before feature freeze... opinions? | 13:13 | |
dtantsur | s/before/so close to/ | 13:14 |
lucasagomes | dtantsur, the VM was shutdown :-( | 13:14 |
jroll | dtantsur: we'll have 3-4 weeks to deal with any fallout, but I haven't looked at that change so not sure how crazy it is :) | 13:14 |
lucasagomes | dtantsur, have to find another place for i | 13:14 |
lucasagomes | it* | 13:14 |
dtantsur | jroll, it's just rewriting half of ironic-inspector, nothing special :D | 13:15 |
jroll | dtantsur: O_O | 13:15 |
dtantsur | lucasagomes, :( our poor bear is looking for a home | 13:15 |
lucasagomes | yeah | 13:15 |
milan | dtantsur, I'd like to get feedback sooner, from Newton ideally | 13:15 |
dtantsur | milan, yeah. on the other hand, I don't want to break tripleo, for example, so close to their FF | 13:15 |
milan | before other HA bits land | 13:15 |
jroll | dtantsur: this one? https://review.openstack.org/#/c/348943/ | 13:15 |
dtantsur | jroll, yes | 13:16 |
milan | jroll, yup | 13:16 |
dtantsur | +1071, -126 sweet :D | 13:16 |
* milan hides ;) | 13:16 | |
milan | my only excuse is that majority of the change are test cases O:-) | 13:17 |
*** pcaruana has joined #openstack-ironic | 13:17 | |
dtantsur | It's not something bad to have, and I'm still open to merge it before we release N-3-ish inspector. But I'd really listen to other's opinions. | 13:18 |
jroll | code doesn't look that insane, honestly | 13:18 |
jroll | but, it is ripe for bugs | 13:18 |
jroll | especially in systems interacting with inspector | 13:18 |
jroll | does tripleo etc use inspector's REST API? or only ironic? | 13:19 |
dtantsur | jroll, it does | 13:19 |
dtantsur | (use inspector's own API pretty extensively) | 13:19 |
jroll | yeah, I'd be worried then :) | 13:19 |
* milan worried :D | 13:19 | |
milan | feedback from TripleO will be substantial, something tells me ;) | 13:20 |
jroll | it always is :) | 13:20 |
dtantsur | I have my personal reason too: I'm so deeply buried in bringing ironic support to TripleO before the FF, that I won't be able to help with potential fallouts | 13:20 |
jroll | if this was ironic I'd probably wait | 13:20 |
jroll | dtantsur: ++ for calling that out | 13:20 |
*** mjturek has joined #openstack-ironic | 13:22 | |
*** sdake_ has joined #openstack-ironic | 13:23 | |
milan | jroll, dtantsur shall we possibly bring this up on the upstream meeting? | 13:23 |
jroll | milan: you can bring it up in open discussion if you like, can't hurt | 13:24 |
dtantsur | milan, I'd like sambetts|afk and aarefiev's opinions first of all (unfortunately, it's a holiday in UK) | 13:24 |
milan | dtantsur, my intention was to get their opinion there | 13:25 |
* milan didn't know about the holiday | 13:25 | |
*** sdake has quit IRC | 13:25 | |
dtantsur | hell, I need to rebase https://review.openstack.org/#/c/354806/ for ironicclient release... anyone wants some free work? :) | 13:26 |
mjturek | dtantsur: I can rebase it for you if you'd like! | 13:27 |
dtantsur | mjturek, would be awesome! in addition to rebases it also needs unit tests though.. | 13:27 |
mjturek | dtantsur: sure :) I'll work on it! | 13:27 |
jroll | dang, mjturek got tricked | 13:27 |
mjturek | hahaha | 13:28 |
dtantsur | mjturek, thanks! :) | 13:28 |
dtantsur | lol | 13:28 |
*** skazi has quit IRC | 13:32 | |
*** hoonetorg has quit IRC | 13:33 | |
*** itooon has joined #openstack-ironic | 13:37 | |
*** itooon has quit IRC | 13:38 | |
rloo | Happy Monday ironic'ers. This includes you dtantsur, mjturek, milan, jroll, lucasagomes :) | 13:38 |
lucasagomes | rloo, hello there | 13:38 |
milan | morning rloo! :) Happy Monday to you too! | 13:38 |
*** hshiina has joined #openstack-ironic | 13:39 | |
dtantsur | morning rloo | 13:39 |
*** rloo has quit IRC | 13:40 | |
*** nicodemos has joined #openstack-ironic | 13:40 | |
mat128 | o/ rloo | 13:41 |
*** rloo has joined #openstack-ironic | 13:41 | |
mjturek | good morning rloo :) | 13:41 |
*** ricardoas has joined #openstack-ironic | 13:42 | |
ricardoas | morning | 13:42 |
*** xavierr has joined #openstack-ironic | 13:43 | |
*** rbrud has joined #openstack-ironic | 13:47 | |
*** sdake_ has quit IRC | 13:50 | |
*** sdake has joined #openstack-ironic | 13:51 | |
hshiina | hi folks, patches for soft reboot and inject nmi have been posted. | 13:54 |
*** xuao has quit IRC | 13:54 | |
hshiina | could you please review soft reboot changes of power interface and ipmitool driver firstly? | 13:54 |
hshiina | power interface: https://review.openstack.org/#/c/216730/ | 13:54 |
hshiina | ipmitool driver: https://review.openstack.org/#/c/216738/ | 13:54 |
openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Add portgroup support to osc plugin https://review.openstack.org/362130 | 13:55 |
*** pcaruana has quit IRC | 13:55 | |
*** causten has joined #openstack-ironic | 13:56 | |
rloo | dtantsur: wrt https://review.openstack.org/#/c/360787/1/ironicclient/osc/v1/baremetal_node.py, it might be worth mentioning. but what words to use so it is clear? | 13:56 |
dtantsur | rloo, maybe just "RAID configuration is updated separately from the remaining fields"? | 13:58 |
*** sabeen has joined #openstack-ironic | 13:58 | |
rloo | dtantsur: i wonder if it is useful to the user. and if we do that, what about the 'create' command, and the 'node delete' (deletes each node sequentially). | 14:00 |
rloo | dtantsur: i think i want to punt that. we should be consistent with whatever we say. i think it is on my list of things to check with osc. | 14:01 |
dtantsur | I'm fine with not mentioning it for now | 14:01 |
jroll | morning rloo :) | 14:02 |
rloo | dtantsur: ok | 14:02 |
openstackgerrit | Merged openstack/ironic: Mask instance secrets in API responses https://review.openstack.org/326768 | 14:05 |
*** mkoderer has quit IRC | 14:06 | |
openstackgerrit | Merged openstack/ironic: Fix pep8 on Python3.5 https://review.openstack.org/361971 | 14:07 |
*** mtanino has joined #openstack-ironic | 14:07 | |
*** syed_ has joined #openstack-ironic | 14:08 | |
*** mkoderer has joined #openstack-ironic | 14:08 | |
*** pcaruana has joined #openstack-ironic | 14:09 | |
* lucasagomes won't see those annoying pep8 errors anymore ^ yay... | 14:10 | |
openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC RAID configuration https://review.openstack.org/176319 | 14:11 |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Set/unset node's target RAID config via OSC command https://review.openstack.org/360787 | 14:12 |
*** thiagop has joined #openstack-ironic | 14:13 | |
*** Goneri has joined #openstack-ironic | 14:14 | |
thiagop | Good morning, Ironicers | 14:14 |
*** ksumit has joined #openstack-ironic | 14:15 | |
rloo | morning thiagop | 14:15 |
thiagop | rloo: o/ | 14:17 |
*** moshele has quit IRC | 14:18 | |
rloo | dtantsur: another question wrt vendor passthrough & vendor passthru (https://review.openstack.org/#/c/357400/), see the updated spec: https://review.openstack.org/#/c/357505/ | 14:18 |
rloo | dtantsur: some people wanted 'passthru', and others wanted 'passthrough' (more English). | 14:19 |
dtantsur | this does not change my position, though I guess I'm overvoted | 14:19 |
rloo | dtantsur: since osc is meant to be more English, i was going to change it to passthrough. | 14:19 |
dtantsur | having two names for the same command does not sound right to me | 14:19 |
rloo | dtantsur: i am happy to change it again. would like to make you happy :) | 14:20 |
*** vishwanathj_zzz is now known as vishwanathj | 14:20 | |
vdrok | morning milan jroll xavierr thiagop rloo | 14:21 |
thiagop | vdrok: o/ | 14:21 |
xavierr | good morning vdrok :D | 14:22 |
dtantsur | rloo, maybe bring it to the ML and get OSC people opinion too? I'm not sure how to handle this "legacy term used everywhere vs a proper english term" situation | 14:22 |
rloo | dtantsur: sure, could do that. | 14:22 |
*** jaosorior is now known as jaosorior_away | 14:25 | |
TheJulia | Good morning everyone | 14:27 |
openstackgerrit | Merged openstack/ironic: Add metrics for the ipminative driver https://review.openstack.org/352829 | 14:27 |
vdrok | morning TheJulia | 14:29 |
*** jtomasek|afk has quit IRC | 14:30 | |
thiagop | Good morning TheJulia | 14:30 |
milan | morning vdrok :) | 14:32 |
*** clenimar has joined #openstack-ironic | 14:34 | |
*** ksumit has quit IRC | 14:34 | |
*** appu103_ has joined #openstack-ironic | 14:37 | |
xavierr | dtantsur: is there a way inspector don't include hyperthreads in the total number of cpus? | 14:38 |
dtantsur | xavierr, not currently | 14:39 |
mat128 | xavierr, dtantsur: please use 2 separate fields if you don't want it | 14:39 |
mat128 | downstream, we've used names likes "number of threads" instead of "cores" to distinguish | 14:39 |
xavierr | mat128: oh, good to know | 14:40 |
dtantsur | yeah, we can make IPA return two fields, then have an option on ironic-inspector side to pick one for properties/cpus | 14:40 |
*** appu103_ has quit IRC | 14:41 | |
mat128 | xavierr: exerpt from downstream flavor | 14:41 |
mat128 | http://paste.openstack.org/show/564588/ | 14:41 |
*** appu103_ has joined #openstack-ironic | 14:41 | |
mat128 | vcpus is "total threads presented" | 14:41 |
mat128 | then deeper you have actual processor count | 14:41 |
mat128 | we have plenty more data in the ironic node (not on vpn, cant show right now), but flavor only "filters" on those | 14:42 |
xavierr | mat128: I see. I need to do that! | 14:43 |
jroll | neat | 14:43 |
*** ashestakov has joined #openstack-ironic | 14:44 | |
gabriel-bezerra | something to observe: the cpu capabilities could also show the hyperthreading | 14:46 |
TheJulia | ++ since some applications run poorly on hyperthreads, so it might be useful to know in advance | 14:47 |
gabriel-bezerra | could be "ht": "cpu_smt" in the https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/plugins/capabilities.py#L23 | 14:47 |
JayF | jroll: jaosorior_away: I'll pull my -2 | 14:49 |
jroll | thanks | 14:50 |
*** e0ne has joined #openstack-ironic | 14:51 | |
lucasagomes | rloo, commented on the ML. tl;dr I don't think we should keep 2 commands for the same functionality | 14:52 |
lucasagomes | IMO we should deprecate the "passthrough" one | 14:52 |
lucasagomes | because we use passthru in the API, that's the only reason | 14:53 |
*** appu103_ has quit IRC | 14:53 | |
*** jtomasek has joined #openstack-ironic | 14:53 | |
*** anush__ has joined #openstack-ironic | 14:54 | |
JayF | lucasagomes: fwiw there's no "deprecate". passthrough doesn't exist yet | 14:56 |
*** mari0jv has joined #openstack-ironic | 14:56 | |
lucasagomes | JayF, oh I thought it was in osc already | 14:57 |
lucasagomes | thanks for clarifying, so yeah I think we should only add passthru then | 14:57 |
rloo | lucasagomes: thx. so i want us to separate ironic CLI from openstack commands. our openstack commands should conform to openstack guidelines, whatever they are :) | 15:02 |
rloo | lucasagomes: having said that, passthru might be an acceptable english term. | 15:02 |
thiagop | dtantsur: answered your comments on 358041, thank you. Wdyt I should do about that decorator? | 15:02 |
lucasagomes | rloo, gotcha. Is it a queens english vs american english thing ? | 15:02 |
rloo | lucasagomes: and yeah, am asking now before we push out something that we don't want to do :) so thx for your comment. | 15:03 |
rloo | lucasagomes: more like proper English :) | 15:03 |
JayF | I mean, thru was in every dictionary I found :) | 15:03 |
lucasagomes | heh I see | 15:03 |
JayF | through is the base word for thru | 15:03 |
JayF | but it's not like it's improper slang or something | 15:03 |
rloo | JayF: ++. I don't actually care what we do as long as we decide/pick the 'right' thing. I just wanna get osc stuff done this week. | 15:04 |
Mr_T | http://grammarist.com/spelling/through-thru/ | 15:05 |
Mr_T | http://www.merriam-webster.com/words-at-play/how-thru-turned-into-through | 15:05 |
*** mjura has quit IRC | 15:05 | |
Mr_T | this is (almost) literally the definition of bike-shedding, though, so I'll back away now | 15:05 |
mat128 | lmao | 15:06 |
rloo | Mr_T: yes, although not quite bikeshedding, because this is something that is visible to the user. so language is important and consistency. i think that's why osc was started, blah blah. | 15:07 |
*** daemontool has quit IRC | 15:07 | |
rloo | honestly, i would have left it as passthru, but i saw dean troyer mention it as passthrough :) | 15:07 |
Mr_T | after reading those articles, I vote "drive-thru" | 15:08 |
JayF | rloo: if you haven't met him, Mr_T is the downstream product manager for OnMetal | 15:09 |
* Mr_T waves | 15:09 | |
rloo | hi Mr_T! thx for your vote. | 15:09 |
Mr_T | JayF has to put up with a lot of this kind of crap, so y'all can empathize with him now :-) | 15:09 |
*** daemontool has joined #openstack-ironic | 15:09 | |
rloo | operator and user feedback is more appreciated than developer feedback wrt 'words' :) | 15:09 |
JayF | Mr_T: on either Thursday or Friday I broke out the OED and looked up "thru", lol | 15:10 |
*** jaosorior_away is now known as jaosorior | 15:10 | |
jaosorior | JayF: thanks | 15:10 |
Mr_T | JayF - as long as you go with a respected descriptivist dictionary, it sounds like you're safe :-) | 15:10 |
JayF | Mr_T: *blink* | 15:11 |
JayF | Mr_T: I know those words, just not what they mean all together | 15:11 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow suppressing ramdisk logs collection https://review.openstack.org/359381 | 15:11 |
mat128 | :P | 15:11 |
*** dprince has quit IRC | 15:11 | |
jroll | rloo: sooooo, everything is thru. we're always going to have thru unless we change our API. and the ironic CLI. so can we just go with thru for now and if we want to switch to through later, we just do it? | 15:11 |
mat128 | +1 ^ | 15:12 |
Mr_T | JayF - descriptivists call the language as they see it - prescriptionists set the language in stone | 15:12 |
rloo | jroll: i'm fine with that. (although i'm not sure about switching things in osc; something we'll need to find out wrt deprecations etc) | 15:12 |
jroll | rloo: it's possible, even if we don't know the process | 15:12 |
JayF | Mr_T: ah, my wife (who has an MA in English) always told me language is defined by what people say, not the other way around. I guess that means she's a descriptivist? | 15:12 |
Mr_T | precisely! | 15:13 |
*** hshiina has quit IRC | 15:13 | |
*** persia has quit IRC | 15:13 | |
jlvillal | Good morning Ironic | 15:14 |
thiagop | morning jlvillal | 15:14 |
*** michauds has joined #openstack-ironic | 15:15 | |
*** persia has joined #openstack-ironic | 15:16 | |
lucasagomes | jlvillal, thiagop morning | 15:16 |
*** praneshp has joined #openstack-ironic | 15:16 | |
jlvillal | :) | 15:16 |
thiagop | lucasagomes: o/ | 15:16 |
*** hoonetorg has joined #openstack-ironic | 15:16 | |
rloo | morning jlvillal | 15:17 |
jroll | TheJulia: hey, can we do a bifrost release this week? | 15:17 |
*** sdake_ has joined #openstack-ironic | 15:18 | |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 15:18 |
thiagop | Folks, this one is hanging around for a while. I'd love if some cores give some love to it before I leave: https://review.openstack.org/#/c/340596/ | 15:19 |
vdrok | morning jlvillal | 15:19 |
thiagop | btw, it's failing one of our jobs due to a timeout downloading IPA images, but I did rechecked it | 15:19 |
thiagop | I did recheck* | 15:19 |
*** sdake has quit IRC | 15:20 | |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Bring configurations from tempest to ironic_tempest_plugin https://review.openstack.org/355591 | 15:24 |
*** sdake has joined #openstack-ironic | 15:24 | |
*** sdake_ has quit IRC | 15:26 | |
*** Sukhdev has joined #openstack-ironic | 15:31 | |
*** ChubYann has joined #openstack-ironic | 15:31 | |
*** harlowja_at_home has joined #openstack-ironic | 15:32 | |
TheJulia | jroll: I don't see why not, it wouldn't be like a 2.0 from a functionality standpoint not much has really changed this cycle | 15:33 |
*** thrash is now known as thrash|biab | 15:33 | |
*** rpioso has joined #openstack-ironic | 15:36 | |
*** xuao has joined #openstack-ironic | 15:38 | |
*** tesseract- has quit IRC | 15:42 | |
*** pcaruana has quit IRC | 15:43 | |
*** rcernin has quit IRC | 15:44 | |
jroll | TheJulia: okay, release team would like one, just to be sure there will be one this cycle (email from doug on -dev) | 15:45 |
TheJulia | k, I've not even gotten through all of my email this morning.... never schedule an 8am lab appointment on the day schools go back into session. | 15:47 |
jroll | heh | 15:47 |
jroll | yeah no rush, just this week sometime | 15:47 |
jroll | ironic/neutron meeting in 10, I'm running it so this is more reminding myself than anyone else :P | 15:50 |
*** wajdi has joined #openstack-ironic | 15:53 | |
*** amotoki has quit IRC | 15:55 | |
dtantsur | thiagop, sorry, on the meeings now, will answer in a few | 16:00 |
*** thrash|biab is now known as thrash | 16:01 | |
* jlvillal wonders what Ilya Shakhat's IRC nick is. To ask about Stackalytics REST API strangeness | 16:01 | |
*** david-lyle_ is now known as david-lyle | 16:01 | |
jroll | jlvillal: https://launchpad.net/~shakhat | 16:02 |
jlvillal | jroll: Thanks. I'm guessing he is offline or that is not his IRC nick. | 16:02 |
dtantsur | morning jlvillal, JayF and everyone I missed while on a meeting | 16:03 |
JayF | o/ | 16:03 |
vsaienk0 | jroll: I wondered if we want to add portgroups support to bootstrap also in long time perspective. it will allow to download user images faster. What do you think? | 16:04 |
jlvillal | \o/ | 16:04 |
jroll | vsaienk0: in ironic/neutron meeting now | 16:04 |
devananda|OPS | morning, all | 16:04 |
*** devananda|OPS is now known as devananda | 16:05 | |
dtantsur | morning devananda | 16:05 |
openstackgerrit | Merged openstack/python-ironicclient: Add baremetal port list command to OSC plugin https://review.openstack.org/346722 | 16:05 |
pas-ha | jlvillal: Ilya's nick is "ilyashakhat" | 16:06 |
pas-ha | he seems busy ATM though | 16:07 |
openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 16:07 |
jlvillal | pas-ha: Thanks! | 16:08 |
*** ifarkas is now known as ifarkas_afk | 16:11 | |
*** e0ne has quit IRC | 16:14 | |
*** ppiela has joined #openstack-ironic | 16:18 | |
*** sdake has quit IRC | 16:19 | |
*** ohamada has quit IRC | 16:19 | |
*** rama_y has joined #openstack-ironic | 16:20 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Update openstack baremetal driver commands https://review.openstack.org/362047 | 16:20 |
*** dprince has joined #openstack-ironic | 16:21 | |
*** e0ne has joined #openstack-ironic | 16:23 | |
*** jpich has quit IRC | 16:25 | |
NobodyCam | Good Morning Ironic'ers :) | 16:26 |
*** sdake has joined #openstack-ironic | 16:30 | |
lucasagomes | NobodyCam, hi there | 16:33 |
lucasagomes | devananda, morning | 16:33 |
dtantsur | morning NobodyCam | 16:33 |
openstackgerrit | Rajini Ram proposed openstack/ironic: Option to enroll nodes with drac driver https://review.openstack.org/362267 | 16:33 |
openstackgerrit | Rajini Ram proposed openstack/ironic: Option to enroll nodes with drac driver https://review.openstack.org/362267 | 16:34 |
NobodyCam | Morning lucasagomes, dtantsur | 16:34 |
*** aNupoisc has joined #openstack-ironic | 16:36 | |
*** Guest35918 is now known as mgagne | 16:39 | |
*** mgagne has quit IRC | 16:39 | |
*** mgagne has joined #openstack-ironic | 16:39 | |
*** harlowja_at_home has quit IRC | 16:39 | |
openstackgerrit | Rodion Promyshlennikov proposed openstack/python-ironicclient: Add openstack baremetal driver commands https://review.openstack.org/350050 | 16:40 |
openstackgerrit | Rodion Promyshlennikov proposed openstack/python-ironicclient: Add openstack baremetal driver commands https://review.openstack.org/350050 | 16:44 |
*** lucasagomes is now known as lucas-afk | 16:45 | |
* lucas-afk will be back for the meeting | 16:45 | |
*** jcoufal has joined #openstack-ironic | 16:46 | |
jlvillal | pas-ha: I don't think he believed me that something strange is going on :) At first I thought it was a proxy server issue because I am at work. But a VM on the internet is also getting strange results :( | 16:46 |
thiagop | jroll: 355591 is now failing only on the multitenant job because security-groups feature is enabled. Shouldn't we have that disabled in our jobs? | 16:46 |
jlvillal | pas-ha: Regarding stackalytics REST API | 16:47 |
jroll | thiagop: security groups do work | 16:47 |
jroll | (or should?) | 16:47 |
thiagop | jroll: so not-running the security group tests is the real problem. Will remove the check then. | 16:49 |
jroll | thiagop: sounds like it | 16:49 |
*** jtomasek has quit IRC | 16:50 | |
*** rbrud has quit IRC | 16:51 | |
*** gfhellma has joined #openstack-ironic | 16:54 | |
vdrok | morning NobodyCam | 16:55 |
NobodyCam | morning vdrok :) | 16:55 |
*** lucas-afk is now known as lucasagomes | 17:00 | |
*** jaosorior has quit IRC | 17:00 | |
vdrok | jroll: meeting time :) | 17:00 |
jroll | heh | 17:00 |
* jroll easily distracted | 17:00 | |
*** jtomasek has joined #openstack-ironic | 17:10 | |
JayF | mat128: I'll be honest, part of why I haven't reviewed that upper-constraints is because it's a little hard to grok | 17:23 |
JayF | mat128: I presume you built this and checked pip freeze inside it? | 17:23 |
mat128 | You can see the pip install process installing the right versions | 17:23 |
mat128 | I can pinpoint lines | 17:23 |
mat128 | 1 sec | 17:23 |
JayF | thanks; I was about to look for that myself \o/ | 17:23 |
JayF | I can land this before the end of the meeting once you show me that | 17:23 |
JayF | I've looked over the code | 17:23 |
mat128 | I think I liked it in the comments | 17:24 |
JayF | jcook: jroll ^ we should make sure to implement the upper-constraints stuff in our downstream ipa build repo | 17:24 |
mat128 | stable/liberty http://logs.openstack.org/93/360793/1/check/gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-coreos-src/e20f91d/logs/devstacklog.txt.gz#_2016-08-25_21_37_45_658 | 17:24 |
*** sabeen has quit IRC | 17:25 | |
mat128 | Pint 0.6 | 17:25 |
mat128 | https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h=stable/liberty | 17:25 |
mat128 | otherwise would be 0.7.2 (https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt) | 17:25 |
mat128 | or even later | 17:25 |
JayF | ah | 17:25 |
JayF | I'll review the code again and land it then | 17:25 |
mat128 | it basically boils down to: introduce a tool to generate a "correct" upper-constraints (one that follows the rules we have defined), change builders to dump a u-c file in the root of IPA then pip install -c $uc_file | 17:27 |
JayF | mat128: landing | 17:31 |
*** trown is now known as trown|lunch | 17:32 | |
mat128 | :D | 17:36 |
mat128 | In that case, backports are already based on the right commit (let me double check) | 17:36 |
*** electrofelix has quit IRC | 17:37 | |
*** praefect has joined #openstack-ironic | 17:40 | |
*** fragatina has quit IRC | 17:44 | |
*** Sukhdev has quit IRC | 17:45 | |
*** praefect has quit IRC | 17:50 | |
*** bdeschenes has joined #openstack-ironic | 17:52 | |
*** xuao has quit IRC | 17:54 | |
*** rajinir has joined #openstack-ironic | 17:55 | |
rloo | thx devananda | 18:00 |
jroll | devananda: feel free to add that dash to the whiteboard | 18:00 |
devananda | jroll: I did | 18:00 |
jroll | woo | 18:00 |
dtantsur | devananda, "small changes" is a good idea. gertty doesn't support it? | 18:00 |
NobodyCam | thank you :) | 18:00 |
devananda | jroll: question is - should it replace our official dashboard? | 18:00 |
devananda | dtantsur: nope | 18:01 |
vdrok | time to leave for me, good night everyone! | 18:01 |
dtantsur | we should ask them to support it for sure.. | 18:01 |
dtantsur | g'night vdrok | 18:01 |
jroll | devananda: define official :) | 18:01 |
rloo | devananda: are 'small things' also included in 'recently proposed to master'? | 18:01 |
* dtantsur is leaving as well, g'night | 18:01 | |
lucasagomes | devananda, unfortunately replacing the module in sys.module is causing other tests to fail when runnning in parallel | 18:01 |
jroll | devananda: honestly, I think we should merge them, both have some useful categories | 18:01 |
devananda | jroll: https://github.com/openstack/gerrit-dash-creator/blob/master/dashboards/ironic.dash | 18:01 |
*** Sukhdev has joined #openstack-ironic | 18:01 | |
devananda | official :-D | 18:01 |
devananda | lucasagomes: yep | 18:01 |
devananda | rloo: the queries are separate. it's possible for there to be overlap. | 18:02 |
*** moshele has joined #openstack-ironic | 18:02 | |
devananda | rloo: something could be: starred; small; recent. and appear in all three. | 18:02 |
*** fragatina has joined #openstack-ironic | 18:02 | |
mat128 | devananda: I'd love your new categories to appear in gerrit-dash-reator | 18:02 |
jroll | devananda: yeah, I'd suggest adding some of those categories to the 'official' one | 18:02 |
*** dtantsur is now known as dtantsur|afk | 18:02 | |
devananda | k | 18:02 |
mat128 | rloo: you can always click on a sub-category to see the query | 18:03 |
devananda | jroll: I'm particularly interested to share with you the "starred by PTL" list, and how Swift is using that | 18:03 |
rloo | devananda: ok, then I'm happy with it. Would just like it if you added eg 'Recently' -> 'Recently (past week)', and 'Small' -> 'Small (< 25 LOC)' | 18:03 |
mat128 | Small things will tell you "delta" | 18:03 |
*** sabeen has joined #openstack-ironic | 18:03 | |
devananda | rloo: easy enough :) | 18:03 |
jroll | devananda: I'm not good enough at keeping my stars up to date to do that right now :) | 18:03 |
rloo | mat128: i didn't know they were clickable. but still, makes it easier for my poor brain. | 18:04 |
rloo | jroll: what about a starred by core? ha ha. | 18:04 |
mat128 | rloo: maybe you can do reviewers.contains(one_of(ironic_cores())) | 18:05 |
jroll | rloo: I mean, I think we could make some good dashboards if we paid attention to how we star things and such | 18:05 |
rloo | jroll: i suppose. i worry about the patches that have fallen down the list. eg tagging. its been on my own list for many weeks now. | 18:06 |
mat128 | ^thats what worries me with the "starred by PTL" category | 18:06 |
mat128 | if people only look at those | 18:06 |
rloo | jroll: if the first thing i do is look at 'small', 'needs approval', 'starred', i may never get to them. on the other hand i haven't gotten to them. sigh. | 18:06 |
jroll | rloo: yeah, I hear ya, but I guess it depends what you look at | 18:07 |
jroll | rloo: if the first thing you do is think "what do I feel like looking at", I can tell you which ones will get looked at | 18:07 |
devananda | swift project has a dashboard that ranks all reviews by a sum() of the relative weight of the folks who star it, where weight is a result of activity in the project. | 18:07 |
mat128 | rloo: on the bright side, small might be easier to go through | 18:07 |
jroll | rloo: (small, needs approval, cores care) | 18:07 |
rloo | jroll: right. so those dashboards are useful/sort of indicate/importance based on order of the categories and where they are situated :) | 18:07 |
jroll | rloo: I think so, yeah | 18:08 |
devananda | just a guess, but perhaps something like that could help us to focus on patches we collectively feel are important | 18:08 |
jroll | yeah, I agree | 18:09 |
jroll | instead of asking people in irc | 18:09 |
mat128 | devananda: https://github.com/openstack/gerrit-dash-creator/blob/master/dashboards/swift.dash | 18:09 |
rloo | to date, i try to follow (apart from intel priorities) what jroll/PTL sez is important. so upstream priorities, blah blah. | 18:09 |
devananda | I spend a fair amount of time starring patches as I run across them, and then pull up my list of "starred patches" when I'm looking for things to follow up on | 18:09 |
jroll | *if* we're diligent about maintaining stars | 18:09 |
devananda | rloo: right, me too, except it takes me effort to find those patches three days later, or while I'm offline / at a cafe / on a plane | 18:10 |
rloo | devananda: oh, yes, that's true. | 18:10 |
devananda | that's my own failing at tracking them, in a sense, so I've started using stars to keep track | 18:10 |
mat128 | devananda: do you have a link to the swift ranking? Can't find it in gerrit-dash-creator | 18:10 |
rloo | devananda: no, i don't think that's your own failing. i think it means we should provide some way to find them easily w/o looking at meeting minutes or whatever. | 18:10 |
*** vishwanathj has quit IRC | 18:11 | |
*** trown|lunch is now known as trown | 18:11 | |
*** jcoufal has quit IRC | 18:11 | |
devananda | mat128: searching, one sec | 18:11 |
*** vishwanathj has joined #openstack-ironic | 18:11 | |
devananda | mat128: http://d.not.mn/swift_community_dashboard.html | 18:12 |
devananda | mat128: dash on the right, "community starred patches" | 18:12 |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Metrics for ConductorManager https://review.openstack.org/352648 | 18:13 |
mat128 | thank you | 18:13 |
mat128 | interesting | 18:13 |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Bring configurations from tempest to ironic_tempest_plugin https://review.openstack.org/355591 | 18:13 |
*** rbrud has joined #openstack-ironic | 18:14 | |
devananda | jroll, rloo: which of the sections in the current dash do you find valuable / want to keep? | 18:16 |
rloo | devananda: comparing... | 18:18 |
JayF | devananda: needs approval is my favorite section, but I think your new dash has that | 18:18 |
*** jvgrant__ has quit IRC | 18:18 | |
*** jvgrant has joined #openstack-ironic | 18:18 | |
devananda | JayF: yep | 18:18 |
jroll | devananda: "Needs Reverify" is useful, and "5 Days Without Feedback" *can* be useful but often isn't | 18:18 |
devananda | jroll: keeping reverify, I agree that's useful | 18:19 |
JayF | 5d without feedback could be named "unpopular/hard things to review" | 18:19 |
*** moshele has quit IRC | 18:19 | |
JayF | that's almost always what's in the list, stuff that's driver specific or extra sized | 18:19 |
jroll | yeah | 18:19 |
*** jvgrant_ has joined #openstack-ironic | 18:19 | |
rloo | devananda: the ones that aren't in your proposed: needs reverify, my patches requiring attention (only to guilt me) | 18:20 |
devananda | is "open backports" going to be useful to enough folks to put on the dash? | 18:20 |
JayF | honestly, backports in the review dash are frustrating | 18:20 |
JayF | because I can't land them | 18:20 |
rloo | devananda: 5 days w/o feedback used to be very useful to me. when i tried to be 'fair' and give feedback to folks that hadn't heard from anyone. but i couldn't sustain it so stopped doing it. | 18:20 |
devananda | JayF: k, that's what I thought. | 18:21 |
rloo | open backports are useful but close to bottom of the list for me. but useful for others. doesn't matter to me, i can ignore. | 18:21 |
devananda | so, it sounds like ya'll would prefer age:5d to be removed | 18:21 |
*** sabeen has quit IRC | 18:22 | |
jroll | I would want backports in review dash because he always forgets them | 18:22 |
jroll | s/he/I/ | 18:22 |
devananda | JayF: would your frustration at seeing it inspire you to nag me or jroll? :) | 18:22 |
jroll | heh | 18:23 |
JayF | I usually end up nagging tonyb instead, by the time I do reviews jroll is gone and you don't check IRC with a low enough latency :P | 18:23 |
jroll | ++ | 18:23 |
*** jvgrant has quit IRC | 18:23 | |
JayF | Or I could just volunteer to be a stable maintainer for IPA... | 18:23 |
rloo | why can't the dashboard be sophisticated enough to have expand/collapse thingies next to each grouping, and allow folks to select/remember the views they want? | 18:23 |
mat128 | ^YES! this | 18:23 |
mat128 | lol | 18:23 |
devananda | rloo: I wish :) | 18:24 |
mat128 | #gerrit | 18:24 |
lucasagomes | devananda, thinking of a way to do what you want, I think we could mock the "wsexpose" method from wsme and reload the module | 18:25 |
mat128 | what world is this | 18:25 |
lucasagomes | devananda, then check the call_list() | 18:25 |
mat128 | gerrit project not even using gerrit | 18:25 |
jroll | mat128: wat | 18:25 |
*** jcoufal has joined #openstack-ironic | 18:25 | |
mat128 | jroll: was looking for gerrit's source code | 18:25 |
mat128 | jroll: https://gerrit.googlesource.com/gerrit/ | 18:26 |
mat128 | all I can find .. | 18:26 |
JayF | yep and https://www.gerritcodereview.com/ is the official site | 18:26 |
mat128 | yes | 18:26 |
jroll | https://gerrit-review.googlesource.com/#/projects/gerrit,dashboards/default | 18:26 |
mat128 | they want you to use git push bleh:/refs/for/ | 18:27 |
jroll | even on their homepage in the 'reviews' link | 18:27 |
*** moshele has joined #openstack-ironic | 18:27 | |
mat128 | oh wow | 18:28 |
* mat128 fails | 18:28 | |
jroll | :P | 18:28 |
jroll | it is weird they don't recommend git-review though | 18:28 |
devananda | JayF: does IPA have stable branches? | 18:28 |
mat128 | jroll: git-review https://pypi.python.org/pypi/git-review/1.25.0 | 18:28 |
JayF | devananda: absolutely | 18:29 |
mat128 | openstack-led project | 18:29 |
jroll | mat128: heh, go figure | 18:29 |
lucasagomes | folks calling it a day | 18:29 |
lucasagomes | have a great evening all | 18:29 |
JayF | o/ | 18:29 |
mat128 | cya | 18:29 |
lucasagomes | devananda, left a comment, it's late here so I haven't tested | 18:29 |
devananda | lucasagomes: I was looking at using a test fixture. your approach could work if done early on, but not from within the test class -- i think that would have a similar issue to my approach | 18:29 |
lucasagomes | but I think the idea might work | 18:29 |
devananda | lucasagomes: thanks much. I'll give it a shot | 18:29 |
lucasagomes | devananda, even if we reload the module? | 18:29 |
*** moshele has quit IRC | 18:30 | |
lucasagomes | devananda, http://paste.openstack.org/show/564635/ | 18:31 |
devananda | lucasagomes: yea. I tried replacing wsexpose in another attempt, which I didn't post to gerrit | 18:32 |
devananda | because it had the same problem | 18:32 |
lucasagomes | devananda, I can see the functions there, probably we need to sanitize the list of calls and check if they are functions or not | 18:32 |
lucasagomes | devananda, I se e:-( | 18:32 |
lucasagomes | I will try more tomorrow if you don't find a way tonight | 18:32 |
devananda | I think it stems from the way I'm overloading sys.modules entries | 18:32 |
devananda | new dashboard combination: http://goo.gl/dg2lDE | 18:32 |
devananda | if ya'll like, I'll propose that to gerrit-dash-creator | 18:33 |
devananda | lucasagomes: thanks. have a good night! | 18:33 |
lucasagomes | you too | 18:33 |
lucasagomes | see yall | 18:33 |
*** lucasagomes is now known as lucas-dinner | 18:33 | |
*** causten has quit IRC | 18:34 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Add metrics for IPA extensions https://review.openstack.org/360111 | 18:35 |
devananda | jroll, rloo: new dash: http://goo.gl/dg2lDE . feedback requested before I propose it. | 18:35 |
*** jcoufal has quit IRC | 18:36 | |
jroll | devananda: can we add not jenkins -1 to the "Older (>1wk) Open Patches" ? | 18:36 |
devananda | jroll: ah. yes :) | 18:36 |
jroll | nice | 18:36 |
jroll | other than that lgtm | 18:37 |
* jroll runs an errand | 18:37 | |
*** jcoufal has joined #openstack-ironic | 18:37 | |
*** jvgrant_ has quit IRC | 18:38 | |
*** gfhellma has quit IRC | 18:39 | |
*** jvgrant_ has joined #openstack-ironic | 18:39 | |
*** causten has joined #openstack-ironic | 18:40 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Add metrics to administrator guide https://review.openstack.org/356811 | 18:40 |
JayF | ^ should be a quick review if anyone has a moment :) | 18:41 |
rloo | devananda: the newer ones -- is that a complete list or does it take the top X? similar question for older patches. | 18:41 |
rloo | devananda: otherwise, looks good to me, thx! (wrt the other question, cuz you had 'limit 10' for small which i like) | 18:42 |
openstackgerrit | Merged openstack/ironic-python-agent: Changed an assert to more specific assert method https://review.openstack.org/362124 | 18:42 |
openstackgerrit | Merged openstack/ironic-python-agent: Using assertIsNone() is preferred over assertEqual() https://review.openstack.org/361811 | 18:43 |
*** Sukhdev has quit IRC | 18:46 | |
jcook | JayF upper constraints stuff, sorry, need a tl;dr | 18:49 |
JayF | jcook: pretty much upstream OpenStack manages python dependencies two ways: 1) requirements.txt and 2) upper-constraints.txt. We just merged a patch upstream to IPA making the ramdisk build support upper-constraints.txt, ensuring the lib versions we test against are the same that get installed in the ramdisk | 18:50 |
JayF | jcook: since we build IPA out of a separate repo downstream, we need to port that work downstream (or not, and risk untested changes, which is less-risky for us since we don't run stable branch IPAs) | 18:50 |
*** jcoufal has quit IRC | 18:51 | |
mat128 | JayF: if you use docker_build.bash you'll get it for free | 18:51 |
mat128 | unsure where you hook yourself to the build process | 18:51 |
JayF | mat128: we literally build outta a completely separate repo, and clone IPA in and build docker via that | 18:51 |
mat128 | ah, just read last part | 18:51 |
mat128 | got it | 18:52 |
jcook | I grokish | 18:52 |
JayF | mat128: pretty much because that repo is full of vendor tools and such | 18:52 |
jcook | ish | 18:52 |
mat128 | we have something similar | 18:52 |
JayF | Are you all doing more for cleaning now? | 18:52 |
*** jcoufal has joined #openstack-ironic | 18:52 | |
mat128 | I wish | 18:52 |
mat128 | but we still have a downstream repo that handles the build + publish as an artifact part | 18:53 |
mat128 | and local patches | 18:53 |
mat128 | tl;dr http://paste.openstack.org/show/564637/ | 18:53 |
mat128 | we build both for now | 18:53 |
mat128 | we are soon starting work on the proper cleaning steps | 18:54 |
mat128 | or so says my product owner :) | 18:54 |
anteaya | mat128: hey, I'll just share that I consider the term 'guys' to exclude me and perfer the terms that are more inclusive, others have found success with folks group people or team | 18:54 |
*** praneshp has quit IRC | 18:56 | |
mat128 | anteaya: really meant it in it's inclusive form (#2 from http://www.dictionary.com/browse/guy?s=t), and I certainly didn't want to offend | 18:57 |
*** jjohnson2 has joined #openstack-ironic | 18:58 | |
mat128 | anteaya: noted. | 18:58 |
anteaya | it isn't inclusive and thanks for listening | 18:58 |
rloo | vdrok, devananda (and others): min 1 year for osc deprecations: http://lists.openstack.org/pipermail/openstack-dev/2016-August/102467.html | 18:58 |
openstackgerrit | Merged openstack/ironic: IPMINative: Check the boot mode when setting the boot device https://review.openstack.org/354187 | 19:05 |
*** jcoufal has quit IRC | 19:06 | |
*** jcoufal has joined #openstack-ironic | 19:07 | |
*** praneshp has joined #openstack-ironic | 19:13 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/python-ironicclient: Add OSProfiler support to Ironicclient https://review.openstack.org/362331 | 19:15 |
*** zhugaoxiao has quit IRC | 19:16 | |
*** causten has quit IRC | 19:16 | |
*** zhugaoxiao has joined #openstack-ironic | 19:16 | |
*** gfhellma has joined #openstack-ironic | 19:20 | |
*** causten has joined #openstack-ironic | 19:26 | |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Adds node boot device & passthru OSC commands https://review.openstack.org/357400 | 19:28 |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Add node validate OSC command https://review.openstack.org/359322 | 19:31 |
*** causten has quit IRC | 19:33 | |
thiagop | jroll: good news from jenkins on 355591 | 19:36 |
*** fragatina has quit IRC | 19:39 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add base cinder common interface https://review.openstack.org/355625 | 19:39 |
*** fragatina has joined #openstack-ironic | 19:39 | |
openstackgerrit | Merged openstack/ironic: Option to enroll nodes with drac driver https://review.openstack.org/362267 | 19:43 |
rajinir | ifarkas_afk: Need help with python-dracclient installation | 19:43 |
jlvillal | pas-ha: FYI: ilyashakhat figured out the issue. There was a stuck Stackalytics worker. So it had be restarted. And now my script works again :) Thanks for your help. | 19:43 |
*** sabeen has joined #openstack-ironic | 19:45 | |
jroll | thiagop: nice! | 19:45 |
*** mrtenio has joined #openstack-ironic | 19:45 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add tempest tests for node Adoption https://review.openstack.org/344975 | 19:46 |
*** ijw has joined #openstack-ironic | 19:47 | |
*** sdake has quit IRC | 19:48 | |
*** aswadr_ has quit IRC | 19:51 | |
*** iurygregory has joined #openstack-ironic | 19:56 | |
openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 19:58 |
*** derekh has quit IRC | 20:03 | |
*** jhesketh has quit IRC | 20:05 | |
openstackgerrit | Oleksandr Berezovskyi proposed openstack/ironic-inspector: [WIP] Implement rules for ports https://review.openstack.org/360106 | 20:05 |
*** e0ne has quit IRC | 20:06 | |
*** causten has joined #openstack-ironic | 20:10 | |
*** jhesketh has joined #openstack-ironic | 20:12 | |
*** causten has quit IRC | 20:15 | |
*** trown is now known as trown|outtypewww | 20:16 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Fix BMC compatibility with Python 2.6 https://review.openstack.org/362396 | 20:16 |
*** srobert has joined #openstack-ironic | 20:19 | |
*** Sukhdev has joined #openstack-ironic | 20:20 | |
openstackgerrit | Caio Oliveira proposed openstack/python-oneviewclient: [WIP] Creating certificate manager for OneView https://review.openstack.org/334992 | 20:20 |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Reusing oneview_client when possible https://review.openstack.org/358041 | 20:21 |
openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 20:21 |
openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 20:21 |
openstackgerrit | Doug Hellmann proposed openstack/ironic-lib: Update .gitreview for stable/newton https://review.openstack.org/362407 | 20:21 |
openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 20:21 |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Fix BMC compatibility with Python 2.6 https://review.openstack.org/362396 | 20:21 |
*** milan has quit IRC | 20:26 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Fix BMC compatibility with Python 2.6 https://review.openstack.org/362396 | 20:27 |
jlvillal | Who uses Python 2.6? | 20:27 |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Fix BMC compatibility with Python 2.6 https://review.openstack.org/362396 | 20:27 |
mat128 | jlvillal: jjohnson2 mentioned having to support older operating systems | 20:28 |
jlvillal | Oh, that's no fun for them. | 20:29 |
* mat128 just realized this isn't listed anywhere | 20:29 | |
jjohnson2 | jlvillal, yeah, lot's of centos6/rhel6 users | 20:29 |
mat128 | jjohnson2: I wonder if we should add a python26 gate so we don't break 2.6 compat again | 20:30 |
jjohnson2 | not openstack users | 20:30 |
jjohnson2 | mat128, well, this code might not have ever been 2.6 friendly, it's the BMC side code | 20:30 |
jjohnson2 | mat128, 99.9% of users are using it as an ipmi client only | 20:30 |
jroll | mat128: infra doesn't support 2.6 anyway | 20:30 |
mat128 | at all | 20:30 |
mat128 | ? | 20:30 |
jroll | they don't build any images that have it | 20:30 |
jjohnson2 | today might have been the first time anyone wanted to play with a fake bmc in python | 20:30 |
openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 20:31 |
mat128 | Ahh | 20:31 |
jroll | in theory, you could add some PPA and install 2.6, or whatever | 20:31 |
jjohnson2 | fyi, prior to me tagging a release, it goes through a test suite in our environment | 20:31 |
openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 20:31 |
jjohnson2 | (which has not historically included *being* a bmc..) | 20:31 |
mat128 | ah | 20:32 |
mat128 | jjohnson2: so that test environment runs 2.6? | 20:32 |
jjohnson2 | mat128, yeah, we do centos6, sles11, sles12, centos7 | 20:33 |
jlvillal | mat128: No on adding Python 2.6 gate. We removed it on purpose :) | 20:33 |
jjohnson2 | it's part of our HPC management stack, which has some things that care more than others about the distro | 20:33 |
*** rbrud has quit IRC | 20:33 | |
*** rbrud has joined #openstack-ironic | 20:34 | |
mat128 | jlvillal: if infra slaves still supported running 2.6, I'd vote for bringing it back | 20:34 |
jjohnson2 | we are trying to use as much common pieces as we can between openstack, hpc, et al | 20:34 |
mat128 | considering it's important that the project works on it | 20:34 |
jjohnson2 | mat128, well, *pyghmi* specifically, but I'll say that pyghmi has some extra coverage outside of infra | 20:34 |
jlvillal | mat128: Not me :) I like to use Python 2.7 things in the code. | 20:34 |
jjohnson2 | since pyghmi is an odd 'openstack' project that hasa non-openstack users | 20:35 |
jlvillal | mat128: So instead of: "{0} {1}".format(a, b) in Python 2.7 we can do: "{} {}".format(a, b) | 20:35 |
jjohnson2 | with the openstack glue manifested as an ironic driver | 20:35 |
jjohnson2 | jlvillal, also, being able to toss a bytearray at struct.unpack without having to make it a 'bytes' first... | 20:36 |
jlvillal | jjohnson2: :) | 20:36 |
jjohnson2 | (a particular tedious frustration today...) | 20:36 |
mat128 | jlvillal: yes, all the great new things :) | 20:36 |
jjohnson2 | what's *really* fun is doing decent TLS security in python 2.6.... that is tricky | 20:37 |
jjohnson2 | anyway, sorry about the git review spam | 20:38 |
jjohnson2 | I forgot it would bug the hell out of the channel... | 20:38 |
jroll | I for one, am never upset about people writing code :) | 20:39 |
jjohnson2 | well, I could have waiting like 1 minute to amend my commit | 20:39 |
jroll | nah, not a big deal | 20:39 |
jjohnson2 | instead of amend, git review, oh whoops missed one, amend, review... then check, whoops... | 20:39 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Update IPA info in troubleshooting.rst https://review.openstack.org/362428 | 20:39 |
mat128 | on the bright side, we get to see commits | 20:39 |
jroll | it's not like you interrupted anything :) | 20:39 |
jroll | I work the same way | 20:39 |
*** Sukhdev_ has joined #openstack-ironic | 20:40 | |
*** Sukhdev has quit IRC | 20:40 | |
jjohnson2 | I've finally got our new team in Brazil doing things through openstack process now, and hopefully soon the guys in China will in turn bring things in, so there might be more reviews over time | 20:41 |
*** rama_y has quit IRC | 20:41 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add storage_interface DB field and object https://review.openstack.org/348005 | 20:42 |
openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add storage_interface to base driver class https://review.openstack.org/348006 | 20:42 |
thiagop | jjohnson2: where is located your team here in Brazil? | 20:44 |
*** Sukhdev_ has quit IRC | 20:49 | |
openstackgerrit | Merged openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/358855 | 20:49 |
*** dprince has quit IRC | 20:49 | |
*** rama_y has joined #openstack-ironic | 20:49 | |
jlvillal | mat128: ^^ Woot! :) | 20:51 |
mat128 | :D | 20:51 |
TheJulia | mat128: fwiw, The only time I've had to mess with anything python 2.6 related was for centos6 based stuff, and that was about a year ago. | 20:51 |
jlvillal | thiagop: Buenos Aires? | 20:51 |
jlvillal | thiagop: :P | 20:51 |
*** causten has joined #openstack-ironic | 20:52 | |
*** causten has quit IRC | 20:52 | |
*** sdake has joined #openstack-ironic | 20:52 | |
*** causten has joined #openstack-ironic | 20:52 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add base cinder common interface https://review.openstack.org/355625 | 20:52 |
thiagop | jlvillal: if I were a soccer fan (as 99.9% of brazillians), I would be pissed with this comment | 20:53 |
thiagop | lol | 20:53 |
jlvillal | thiagop: heh | 20:53 |
* mat128 understands you are part of the 0.1% | 20:53 | |
jlvillal | thiagop: I was born in Buenos Aires. But I was only there until 3 years old. | 20:54 |
thiagop | mat128: I'm the kind of guy that thinks "if the soccer selection wins, ok. If it loses, ok too" | 20:54 |
gabriel-bezerra | Buenos Aires has the beautiful tango | 20:54 |
thiagop | I'm the typical brazillian weirdo | 20:54 |
jroll | mat128: +2 on https://review.openstack.org/#/c/360795 - devananda mind approving that? | 20:55 |
jroll | mat128: and you've the wrong commit sha here on liberty https://review.openstack.org/#/c/360793/3 | 20:56 |
thiagop | jlvillal: Buenos Aires seems like a nice (and cheap for us) place to go spend some vacation days | 20:56 |
mat128 | I need to update the liberty one | 20:56 |
mat128 | yes | 20:56 |
jroll | heh | 20:56 |
jroll | cool | 20:56 |
gabriel-bezerra | let's not talk about soccer | 20:56 |
gabriel-bezerra | :P | 20:56 |
thiagop | ^agree | 20:57 |
openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/360793 | 20:58 |
mat128 | jroll: liberty updated. commit message was wrong (validated content manually) | 20:58 |
jlvillal | What would people think about having patchbot in this channel? | 20:59 |
mat128 | patchbot? | 20:59 |
wajdi | ^^ | 20:59 |
jlvillal | They use it in #openstack-swift | 20:59 |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Add --node to 'openstack baremetal port list' https://review.openstack.org/362445 | 20:59 |
jlvillal | If you say something like: i submitted https://review.openstack.org/#/c/360135/ to make the tests actually pass | 21:00 |
jlvillal | It replies: <patchbot> timburke: patch 360135 - python-swiftclient - Make functests py3-compatible | 21:00 |
mat128 | ah, just like bugs | 21:00 |
thiagop | jlvillal: very like the bugs one | 21:00 |
mat128 | yeah that'd be great | 21:00 |
jlvillal | mat128: Right! | 21:00 |
thiagop | +1 | 21:01 |
* jroll isn't opinionated | 21:01 | |
jjohnson2 | thiagop, I need to ask | 21:01 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Support to wait for node deployment to complete https://review.openstack.org/254245 | 21:01 |
jjohnson2 | for me it's a good day when I remember what country people are in | 21:01 |
thiagop | lol | 21:02 |
jjohnson2 | there's for me two geographies: central North Carolina and everywhere else | 21:02 |
thiagop | jjohnson2: I think lenovo begun a new project with our university here @ Campina Grande, just checkin' | 21:02 |
devananda | jroll: don't mind at all. | 21:02 |
JayF | it's pretty eaqsy to /ignore the bot if you dislike it too | 21:02 |
jroll | thanks! | 21:02 |
JayF | not like we're making a bot that does lots of things do more things | 21:02 |
JayF | new bot = easy to ignore/block | 21:03 |
mat128 | and old bot is at maxchan limit | 21:03 |
mat128 | :) | 21:03 |
thiagop | #pixiesay --angry WHAT? | 21:03 |
thiagop | pixie is not on the house... | 21:03 |
TheJulia | jjohnson2: But... there is the coast, the piedmont, and the mountains... those are different areas ;) | 21:04 |
jjohnson2 | TheJulia, coast and mountains are rest of world | 21:04 |
jjohnson2 | aka, people who I see in person, versus people I only ever see online | 21:05 |
jjohnson2 | TheJulia, Campinas and Fortaleza | 21:05 |
jjohnson2 | thiagop, err Campinas and Fortaleza | 21:05 |
TheJulia | jjohnson2: is North of I-540 the rest of the world as well ? :) | 21:05 |
thiagop | jjohnson2: cool | 21:05 |
jjohnson2 | TheJulia, depends on if the ydrive to Morrisville and work in one of these two buildings ;) | 21:06 |
*** causten has quit IRC | 21:07 | |
TheJulia | jjohnson2: no, the drive from 401 to 540 makes me go crazy in morning traffic. :) | 21:07 |
jlvillal | jjohnson2: You reminded me of this :) https://s-media-cache-ak0.pinimg.com/564x/96/00/1b/96001b9c6594229fb7468303c0e26157.jpg | 21:07 |
jjohnson2 | TheJulia, then rest of world | 21:07 |
*** causten has joined #openstack-ironic | 21:07 | |
* TheJulia informs her fiancee that they live in the rest of the world | 21:08 | |
jjohnson2 | jlvillal, yeah, that's about right | 21:08 |
*** Goneri has quit IRC | 21:09 | |
jjohnson2 | well, I'm going home | 21:09 |
jjohnson2 | jlvillal, well, adjusted for my locale at least | 21:09 |
jlvillal | jjohnson2: heh | 21:10 |
jjohnson2 | I will say that Brazil is easier than China, close enough time zone versus the worst possible time zone difference | 21:10 |
*** vishwanathj has quit IRC | 21:11 | |
*** vishwanathj has joined #openstack-ironic | 21:11 | |
jjohnson2 | I'm always impressed when there's an afternoon meeting and the stray person from China or New Zealand bothers to attend. No way I would return that favor... I'm not nearly so dedicated | 21:11 |
*** daemontool has quit IRC | 21:11 | |
jjohnson2 | well good evening all | 21:11 |
thiagop | jjohnson2: lol | 21:12 |
thiagop | good evening | 21:12 |
gabriel-bezerra | good evening | 21:12 |
*** wajdi_ has joined #openstack-ironic | 21:13 | |
*** daemontool has joined #openstack-ironic | 21:13 | |
openstackgerrit | Oleksandr Berezovskyi proposed openstack/ironic-inspector: [WIP] Implement rules for ports https://review.openstack.org/360106 | 21:15 |
*** wajdi has quit IRC | 21:17 | |
*** wajdi_ has quit IRC | 21:18 | |
*** srobert has quit IRC | 21:19 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add tempest tests for node Adoption https://review.openstack.org/344975 | 21:22 |
*** jcoufal has quit IRC | 21:24 | |
*** jcoufal has joined #openstack-ironic | 21:24 | |
*** ashestakov has quit IRC | 21:30 | |
*** sdake has quit IRC | 21:33 | |
*** ijw has quit IRC | 21:43 | |
*** aNupoisc has quit IRC | 21:44 | |
*** aNupoisc has joined #openstack-ironic | 21:44 | |
*** causten has quit IRC | 21:45 | |
openstackgerrit | Merged openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/360795 | 21:45 |
*** causten has joined #openstack-ironic | 21:50 | |
openstackgerrit | Merged openstack/python-ironicclient: Add openstack baremetal chassis commands https://review.openstack.org/345815 | 21:53 |
*** lucas-dinner has quit IRC | 21:56 | |
*** sai has quit IRC | 21:58 | |
TheJulia | thiagop: w/r/t your review on the tempest test, I'm going to change the logged message since that makes the most sense since it exclusively exercises ironic | 21:58 |
thiagop | TheJulia: fine to me | 21:58 |
*** rook has quit IRC | 21:58 | |
*** devananda has quit IRC | 21:59 | |
*** sai has joined #openstack-ironic | 21:59 | |
*** lucasagomes has joined #openstack-ironic | 21:59 | |
*** devananda has joined #openstack-ironic | 22:00 | |
*** rook has joined #openstack-ironic | 22:00 | |
*** rook is now known as Guest64782 | 22:00 | |
*** devananda has quit IRC | 22:01 | |
*** devananda has joined #openstack-ironic | 22:02 | |
*** gfhellma1 has joined #openstack-ironic | 22:03 | |
*** gfhellma has quit IRC | 22:03 | |
mrda | Morning Ironic | 22:09 |
thiagop | morning mrda , night Ironicers! | 22:11 |
thiagop | see ya | 22:11 |
*** thiagop has quit IRC | 22:11 | |
mrda | o/ | 22:15 |
*** gfhellma has joined #openstack-ironic | 22:19 | |
*** gfhellma1 has quit IRC | 22:20 | |
*** ijw has joined #openstack-ironic | 22:22 | |
*** bdeschenes has quit IRC | 22:23 | |
*** michauds has quit IRC | 22:23 | |
*** ijw has quit IRC | 22:23 | |
*** rbrud has quit IRC | 22:24 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add tempest tests for node Adoption https://review.openstack.org/344975 | 22:26 |
*** fragatin_ has joined #openstack-ironic | 22:35 | |
*** fragatina has quit IRC | 22:38 | |
openstackgerrit | Merged openstack/ironic: Fix for check if dynamic allocation model is enabled https://review.openstack.org/357425 | 22:43 |
*** Goneri has joined #openstack-ironic | 22:43 | |
*** daemontool has quit IRC | 22:49 | |
*** daemontool has joined #openstack-ironic | 22:50 | |
*** patchbot has joined #openstack-ironic | 22:50 | |
jlvillal | Testing this: https://review.openstack.org/357425 | 22:51 |
*** notmyname has joined #openstack-ironic | 22:51 | |
notmyname | jlvillal: hi | 22:51 |
notmyname | p 112344 | 22:52 |
jlvillal | notmyname: Hi :) | 22:52 |
patchbot | https://review.openstack.org/#/c/112344/ - tuskar - Remove Exception ProcessExecutionError (MERGED) | 22:52 |
notmyname | jlvillal: so I've got one bug in patchbot that I'll be fixing soon | 22:52 |
jlvillal | notmyname: I thought it handled the URLs too | 22:52 |
notmyname | patch 134234 will report something twice | 22:52 |
patchbot | (patch <patch number>) -- Generates a review.openstack.org URL to <patch number>. | 22:52 |
patchbot | https://review.openstack.org/#/c/134234/ - magnum - Incorporate feedback from the Initial commit review (MERGED) | 22:52 |
notmyname | yeah, it does | 22:52 |
notmyname | https://review.openstack.org/#/c/112344/ | 22:52 |
patchbot | patch 112344 - tuskar - Remove Exception ProcessExecutionError (MERGED) | 22:52 |
jlvillal | Testing this: https://review.openstack.org/357425 | 22:52 |
jlvillal | notmyname: But not that way? | 22:52 |
*** sabeen has quit IRC | 22:55 | |
notmyname | jlvillal: no, not right now. | 22:56 |
jlvillal | notmyname: Okay. Thank you :) | 22:57 |
*** mmedvede has quit IRC | 23:02 | |
*** rajinir has quit IRC | 23:05 | |
*** mpsairam has joined #openstack-ironic | 23:06 | |
*** mmedvede has joined #openstack-ironic | 23:06 | |
*** bdeschenes has joined #openstack-ironic | 23:10 | |
*** mari0jv has quit IRC | 23:13 | |
*** praneshp has quit IRC | 23:14 | |
*** vishwanathj has quit IRC | 23:18 | |
*** vishwanathj has joined #openstack-ironic | 23:18 | |
*** rpioso has quit IRC | 23:18 | |
*** adu has joined #openstack-ironic | 23:23 | |
*** rama_y has quit IRC | 23:28 | |
*** vishwanathj has quit IRC | 23:29 | |
*** vishwanathj has joined #openstack-ironic | 23:29 | |
*** vishwanathj has quit IRC | 23:31 | |
*** vishwanathj has joined #openstack-ironic | 23:31 | |
*** sabeen has joined #openstack-ironic | 23:33 | |
*** Guest64782 is now known as rook | 23:34 | |
*** vishwanathj has quit IRC | 23:37 | |
*** vishwanathj has joined #openstack-ironic | 23:37 | |
*** praneshp has joined #openstack-ironic | 23:38 | |
*** vishwanathj has quit IRC | 23:47 | |
*** vishwanathj has joined #openstack-ironic | 23:47 | |
*** anush__ has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!