Monday, 2016-08-29

*** sabeen has quit IRC00:02
*** baoli has joined #openstack-ironic00:22
*** hoangcx2 has quit IRC00:22
*** morgabra has quit IRC00:41
*** morgabra has joined #openstack-ironic00:41
openstackgerritTuan Luong-Anh proposed openstack/ironic-inspector: Fix formatting strings in LOG.error  https://review.openstack.org/36114800:42
*** fragatina has joined #openstack-ironic00:48
*** hoangcx has joined #openstack-ironic00:48
*** Goneri has joined #openstack-ironic01:00
*** bfournie has joined #openstack-ironic01:08
*** vdrok has quit IRC01:09
*** serverascode has quit IRC01:10
*** TheJulia has quit IRC01:10
*** tdurakov has quit IRC01:10
*** betherly has quit IRC01:11
*** mtanino has joined #openstack-ironic01:16
*** TheJulia has joined #openstack-ironic01:19
*** vdrok has joined #openstack-ironic01:25
*** sdake_ has quit IRC01:25
*** tdurakov has joined #openstack-ironic01:26
*** serverascode has joined #openstack-ironic01:26
*** betherly has joined #openstack-ironic01:27
*** sdake has joined #openstack-ironic01:30
*** baoli has quit IRC01:36
*** baoli has joined #openstack-ironic01:44
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent: Use namedtuple to improve code readability  https://review.openstack.org/36178801:44
*** Goneri has quit IRC01:46
*** baoli has quit IRC01:49
*** Goneri has joined #openstack-ironic01:52
*** baoli has joined #openstack-ironic01:56
*** vishwanathj has joined #openstack-ironic02:01
*** annp has joined #openstack-ironic02:10
*** vishwanathj is now known as vishwanathj_zzz02:10
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent: WIP: DO NOT REVIEW (namedtuple playground)  https://review.openstack.org/36179302:17
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent: WIP: DO NOT REVIEW (namedtuple playground)  https://review.openstack.org/36179302:19
*** amotoki has joined #openstack-ironic02:24
*** Goneri has quit IRC02:30
*** vishwanathj_zzz has quit IRC02:33
*** vishwanathj_zzz has joined #openstack-ironic02:33
*** vishwanathj_zzz has quit IRC02:41
*** vishwanathj_zzz has joined #openstack-ironic02:41
*** vishwanathj_zzz has quit IRC02:44
*** vishwanathj_zzz has joined #openstack-ironic02:44
*** yuanying has quit IRC02:51
*** yuanying has joined #openstack-ironic02:52
*** vishwanathj_zzz has quit IRC02:56
*** vishwanathj_zzz has joined #openstack-ironic02:56
*** tiendc has joined #openstack-ironic02:59
*** tuanla_ has joined #openstack-ironic02:59
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off  https://review.openstack.org/21673003:01
openstackgerritTuan Luong-Anh proposed openstack/ironic-python-agent: Using assertIsNone() is preferred over assertEqual()  https://review.openstack.org/36181103:10
*** amotoki has quit IRC03:21
*** amotoki has joined #openstack-ironic03:40
*** vikrant has joined #openstack-ironic03:41
*** yuanying has quit IRC03:44
*** aswadr_ has joined #openstack-ironic03:45
*** vikrant is now known as vikrant|brb03:58
*** chlong has quit IRC04:12
*** vikrant|brb is now known as vikrant04:14
*** amotoki has quit IRC04:21
*** chlong has joined #openstack-ironic04:26
*** links has joined #openstack-ironic04:33
openstackgerritTuan Luong-Anh proposed openstack/ironic: Fix incorrect order of params of assertEqual()  https://review.openstack.org/35958004:33
*** amotoki has joined #openstack-ironic04:42
*** fragatin_ has joined #openstack-ironic04:45
*** fragati__ has joined #openstack-ironic04:48
*** fragatina has quit IRC04:48
*** amotoki has quit IRC04:48
*** fragatin_ has quit IRC04:50
*** jaosorior has joined #openstack-ironic04:52
*** fragati__ has quit IRC04:53
*** vishwanathj_zzz has quit IRC05:01
*** vishwanathj_zzz has joined #openstack-ironic05:01
*** amotoki has joined #openstack-ironic05:03
*** yuanying has joined #openstack-ironic05:03
*** sdake_ has joined #openstack-ironic05:10
openstackgerritTuan Luong-Anh proposed openstack/ironic-python-agent: Using assertIsNone() is preferred over assertEqual()  https://review.openstack.org/36181105:10
*** sdake_ is now known as dake05:13
*** sdake has quit IRC05:13
*** dake is now known as sdake05:13
*** mtanino has quit IRC05:15
openstackgerritTuan Luong-Anh proposed openstack/ironic-python-agent: Using assertIsNone() is preferred over assertEqual()  https://review.openstack.org/36181105:22
*** sdake has quit IRC05:26
openstackgerritAparna proposed openstack/proliantutils: Raise exception when HPSSA controller not available  https://review.openstack.org/34779105:54
*** ChubYann has quit IRC05:57
*** zhangjl has joined #openstack-ironic06:02
*** watanabe_isao has joined #openstack-ironic06:06
*** mjura has joined #openstack-ironic06:06
*** moshele has joined #openstack-ironic06:06
*** mjura has quit IRC06:12
*** mjura has joined #openstack-ironic06:12
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643206:24
*** fragatina has joined #openstack-ironic06:25
*** fragatina has quit IRC06:26
*** fragatina has joined #openstack-ironic06:26
*** yuanying has quit IRC06:29
*** amotoki has quit IRC06:29
*** praneshp has quit IRC06:38
*** rcernin has joined #openstack-ironic06:47
*** amotoki has joined #openstack-ironic06:50
*** amotoki has quit IRC06:58
*** pcaruana has joined #openstack-ironic07:00
*** tesseract- has joined #openstack-ironic07:01
*** aswadr_ has quit IRC07:01
openstackgerritVasyl Saienko proposed openstack/ironic: Remove neutron client workarounds  https://review.openstack.org/35458807:08
*** ohamada has joined #openstack-ironic07:09
*** MattMan has quit IRC07:12
*** MattMan has joined #openstack-ironic07:12
*** ifarkas_afk is now known as ifarkas07:16
*** amotoki has joined #openstack-ironic07:19
*** amotoki has quit IRC07:36
*** jpich has joined #openstack-ironic07:37
*** tiendc has quit IRC07:38
*** tiendc has joined #openstack-ironic07:40
*** amotoki has joined #openstack-ironic07:45
*** Sukhdev has quit IRC07:46
*** yuanying has joined #openstack-ironic07:51
*** vishwanathj_zzz has quit IRC07:53
*** vishwanathj_zzz has joined #openstack-ironic07:53
*** zzzeek has quit IRC08:00
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643208:00
*** zzzeek has joined #openstack-ironic08:01
*** mpsairam has quit IRC08:03
*** fxpester has quit IRC08:04
*** e0ne has joined #openstack-ironic08:05
*** links has quit IRC08:06
*** derekh has joined #openstack-ironic08:12
*** greghaynes has quit IRC08:13
*** lucas-afk is now known as lucasagomes08:21
lucasagomesmorning all08:21
*** cmyster has joined #openstack-ironic08:21
pas-hamorning lucasagomes and all :)08:23
tangchenmorning, all08:23
cmystermorning08:23
vdrokmorning lucasagomes pas-ha tangchen and cmyster08:33
lucasagomeshi there08:34
tangchenmorning, vdrok08:34
cmysterhi08:34
vsaienk0Morning lucasagomes, pas-ha, vdrok and all Ironic'ers!08:38
lucasagomeso/08:38
vdrokhey vsaienk008:38
*** ashestakov has joined #openstack-ironic08:39
*** greghaynes has joined #openstack-ironic08:40
tangchenlucasagomes, vdrok: I have updated functional tests patches. Start from here: https://review.openstack.org/#/c/349421/08:41
tangchenlucasagomes, vdrok: Please help to review when you are free. :)08:41
*** MattMan_ has joined #openstack-ironic08:41
vdroktangchen: ok will try to08:41
tangchenthx08:42
*** MattMan_ has quit IRC08:42
*** MattMan_ has joined #openstack-ironic08:42
*** MattMan_ has quit IRC08:42
*** MattMan_ has joined #openstack-ironic08:42
*** MattMan_ has quit IRC08:44
*** MattMan has quit IRC08:44
*** MattMan has joined #openstack-ironic08:44
*** MattMan has quit IRC08:46
*** MattMan has joined #openstack-ironic08:51
ifarkasmorning all!08:52
*** watanabe_isao has quit IRC08:53
*** electrofelix has joined #openstack-ironic09:00
vdrokmorning ifarkas09:03
*** MattMan has quit IRC09:08
*** wznoinsk_ is now known as wznoinsk09:09
*** MattMan has joined #openstack-ironic09:09
*** MattMan has quit IRC09:10
*** MattMan has joined #openstack-ironic09:11
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Fix pep8 on Python3.5  https://review.openstack.org/36197109:43
*** dtantsur|afk is now known as dtantsur09:44
dtantsurMorning Ironic, morning pas-ha, lucasagomes, tangchen, vdrok, vsaienk0, ifarkas09:44
lucasagomesdtantsur, hi there09:44
*** xuao has joined #openstack-ironic09:46
*** xuao has quit IRC09:47
*** xuao has joined #openstack-ironic09:47
openstackgerritLucas Alvares Gomes proposed openstack/ironic: DevStack: Configure retrieving logs from the deploy ramdisk  https://review.openstack.org/36065809:49
*** MattMan has quit IRC09:52
*** MattMan has joined #openstack-ironic09:54
tangchenmorning, dtantsur09:55
*** zigo_ is now known as zigo09:56
*** vishwanathj_zzz has quit IRC09:58
*** vishwanathj_zzz has joined #openstack-ironic09:58
vdrokdtantsur: morning!10:21
*** tiendc has quit IRC10:27
*** hoangcx has quit IRC10:30
*** e0ne has quit IRC10:32
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/36088910:41
*** amotoki has quit IRC10:41
*** amotoki has joined #openstack-ironic10:46
*** amotoki has quit IRC10:46
*** amotoki has joined #openstack-ironic10:50
*** tiendc has joined #openstack-ironic10:59
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643211:00
*** amotoki has quit IRC11:02
*** tuanla_ has quit IRC11:05
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Fix pep8 on Python3.5  https://review.openstack.org/36197111:07
*** ashestakov_ has joined #openstack-ironic11:10
*** ashestakov has quit IRC11:11
*** ashestakov_ has left #openstack-ironic11:16
*** ashestakov has joined #openstack-ironic11:16
*** thrash|wknd is now known as thrash11:23
*** tiendc has quit IRC11:40
*** annp has quit IRC11:44
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Wish to help with rebasing this patch  https://review.openstack.org/36204711:54
*** lucasagomes is now known as lucas-hungry11:55
openstackgerritMerged openstack/ironic: Fix formatting strings in LOG.error  https://review.openstack.org/36112711:58
openstackgerritMerged openstack/ironic: Update documentation for keystone policy support  https://review.openstack.org/34555611:58
openstackgerritMerged openstack/ironic: Add multi-tenancy section to security doc  https://review.openstack.org/34555711:58
*** dprince has joined #openstack-ironic12:00
*** eset- has quit IRC12:01
*** jaosorior has quit IRC12:02
*** jaosorior has joined #openstack-ironic12:03
*** eset has joined #openstack-ironic12:03
openstackgerritVasyl Saienko proposed openstack/ironic: Rely on portgroup individual_mode_supported  https://review.openstack.org/36064712:06
*** eset has quit IRC12:06
*** eset has joined #openstack-ironic12:06
*** vishwanathj_zzz has quit IRC12:08
*** vishwanathj_zzz has joined #openstack-ironic12:08
*** aswadr_ has joined #openstack-ironic12:08
mat128morning Ironic12:12
*** amotoki has joined #openstack-ironic12:17
*** amotoki has quit IRC12:17
*** baoli_ has joined #openstack-ironic12:18
vdrokmorning mat12812:20
*** baoli has quit IRC12:21
*** vikrant has quit IRC12:28
openstackgerritVasyl Saienko proposed openstack/ironic: Rely on portgroup individual_mode_supported  https://review.openstack.org/36064712:31
*** appu103_ has joined #openstack-ironic12:32
*** appu103_ is now known as aparnav12:32
*** appu1034 has joined #openstack-ironic12:36
*** aparnav has quit IRC12:36
*** nicodemos has joined #openstack-ironic12:36
openstackgerritAnkit Kumar proposed openstack/proliantutils: Fix log message for exception in Gen7  https://review.openstack.org/35386012:38
*** milan has joined #openstack-ironic12:40
milanmorning Ironic! :)12:40
jaosoriordtantsur: When are RFEs approved? Wandering about the status of https://review.openstack.org/#/c/358457/12:46
*** Goneri has joined #openstack-ironic12:50
*** xavierr has joined #openstack-ironic12:50
*** ricardoas has joined #openstack-ironic12:51
*** amotoki has joined #openstack-ironic12:51
jrolljaosorior: 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 bit12:53
jrollmorning everyone12:53
jaosoriorjroll, JayF, dtantsur: well, if you have time, can you take a look at https://review.openstack.org/#/c/358457/ ?12:54
jrolljaosorior: yeah, I'll try12:56
milanmorning jroll! :)12:56
*** lucas-hungry is now known as lucasagomes12:56
jrolldtantsur: JayF: the RFE is here, btw, I'm okay with it https://bugs.launchpad.net/ironic-python-agent/+bug/161649512:56
openstackLaunchpad 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
dtantsurjaosorior, what's the RFE bug number? we're approving intentions first :)12:56
* jroll ninja's dtantsur12:57
dtantsurmorning jroll, milan, mat12812:57
dtantsur:)12:57
jrollmorning :)12:57
milanmorning dtantsur :)12:57
lucasagomesmorning all12:57
dtantsurjroll, +1 for this RFE. I did have some questions about the implementation, but the intention looks clear.12:58
jrolldtantsur: cool12:58
* jroll approves12:58
jrollJayF: when you're around, you can remove your -2 on https://review.openstack.org/#/c/358457/ :)12:58
xavierrmorning jroll jaosorior dtantsur lucasagomes12:59
lucasagomeshi there12:59
jaosoriorxavierr: morning (afternoon in my case :P )12:59
*** rloo has joined #openstack-ironic13:00
jaosoriordtantsur, jroll thanks for checking it out13:00
jrollyeah no problem :)13:00
*** pcaruana has quit IRC13:02
xavierrlucasagomes: I was reading pixie boot bot code this night :)13:03
*** sdake has joined #openstack-ironic13:03
xavierrlucasagomes: I was making some magic with RabbitMQ and irc lib for python :D13:03
dtantsurmorning xavierr13:03
dtantsurbtw, where is Pixie?13:03
*** strigazi_AFK is now known as strigazi13:05
*** daemontool has joined #openstack-ironic13:06
*** Goneri has quit IRC13:06
*** ricardoas has quit IRC13:06
*** appu1034 has quit IRC13:08
*** nicodemos has quit IRC13:08
*** xavierr has quit IRC13:08
*** ashestakov has quit IRC13:11
milanaarefiev, 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 IRC13:13
* dtantsur is still unsure about landing that before feature freeze... opinions?13:13
dtantsurs/before/so close to/13:14
lucasagomesdtantsur, the VM was shutdown :-(13:14
jrolldtantsur: 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
lucasagomesdtantsur, have to find another place for i13:14
lucasagomesit*13:14
dtantsurjroll, it's just rewriting half of ironic-inspector, nothing special :D13:15
jrolldtantsur: O_O13:15
dtantsurlucasagomes, :( our poor bear is looking for a home13:15
lucasagomesyeah13:15
milandtantsur, I'd like to get feedback sooner, from Newton ideally13:15
dtantsurmilan, yeah. on the other hand, I don't want to break tripleo, for example, so close to their FF13:15
milanbefore other HA bits land13:15
jrolldtantsur: this one? https://review.openstack.org/#/c/348943/13:15
dtantsurjroll, yes13:16
milanjroll, yup13:16
dtantsur+1071, -126 sweet :D13:16
* milan hides ;)13:16
milanmy only excuse is that majority of the change are test cases O:-)13:17
*** pcaruana has joined #openstack-ironic13:17
dtantsurIt'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
jrollcode doesn't look that insane, honestly13:18
jrollbut, it is ripe for bugs13:18
jrollespecially in systems interacting with inspector13:18
jrolldoes tripleo etc use inspector's REST API? or only ironic?13:19
dtantsurjroll, it does13:19
dtantsur(use inspector's own API pretty extensively)13:19
jrollyeah, I'd be worried then :)13:19
* milan worried :D13:19
milanfeedback from TripleO will be substantial, something tells me ;)13:20
jrollit always is :)13:20
dtantsurI 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 fallouts13:20
jrollif this was ironic I'd probably wait13:20
jrolldtantsur: ++ for calling that out13:20
*** mjturek has joined #openstack-ironic13:22
*** sdake_ has joined #openstack-ironic13:23
milanjroll, dtantsur shall we possibly bring this up on the upstream meeting?13:23
jrollmilan: you can bring it up in open discussion if you like, can't hurt13:24
dtantsurmilan, I'd like sambetts|afk and aarefiev's opinions first of all (unfortunately, it's a holiday in UK)13:24
milandtantsur, my intention was to get their opinion there13:25
* milan didn't know about the holiday13:25
*** sdake has quit IRC13:25
dtantsurhell, I need to rebase https://review.openstack.org/#/c/354806/ for ironicclient release... anyone wants some free work? :)13:26
mjturekdtantsur: I can rebase it for you if you'd like!13:27
dtantsurmjturek, would be awesome! in addition to rebases it also needs unit tests though..13:27
mjturekdtantsur: sure :) I'll work on it!13:27
jrolldang, mjturek got tricked13:27
mjturekhahaha13:28
dtantsurmjturek, thanks! :)13:28
dtantsurlol13:28
*** skazi has quit IRC13:32
*** hoonetorg has quit IRC13:33
*** itooon has joined #openstack-ironic13:37
*** itooon has quit IRC13:38
rlooHappy Monday ironic'ers. This includes you dtantsur, mjturek, milan, jroll, lucasagomes :)13:38
lucasagomesrloo, hello there13:38
milanmorning rloo! :) Happy Monday to you too!13:38
*** hshiina has joined #openstack-ironic13:39
dtantsurmorning rloo13:39
*** rloo has quit IRC13:40
*** nicodemos has joined #openstack-ironic13:40
mat128o/ rloo13:41
*** rloo has joined #openstack-ironic13:41
mjturekgood morning rloo :)13:41
*** ricardoas has joined #openstack-ironic13:42
ricardoasmorning13:42
*** xavierr has joined #openstack-ironic13:43
*** rbrud has joined #openstack-ironic13:47
*** sdake_ has quit IRC13:50
*** sdake has joined #openstack-ironic13:51
hshiinahi folks, patches for soft reboot and inject nmi have been posted.13:54
*** xuao has quit IRC13:54
hshiinacould you please review soft reboot changes of power interface and ipmitool driver firstly?13:54
hshiinapower interface: https://review.openstack.org/#/c/216730/13:54
hshiinaipmitool driver: https://review.openstack.org/#/c/216738/13:54
openstackgerritVasyl Saienko proposed openstack/python-ironicclient: Add portgroup support to osc plugin  https://review.openstack.org/36213013:55
*** pcaruana has quit IRC13:55
*** causten has joined #openstack-ironic13:56
rloodtantsur: 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
dtantsurrloo, maybe just "RAID configuration is updated separately from the remaining fields"?13:58
*** sabeen has joined #openstack-ironic13:58
rloodtantsur: 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
rloodtantsur: 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
dtantsurI'm fine with not mentioning it for now14:01
jrollmorning rloo :)14:02
rloodtantsur: ok14:02
openstackgerritMerged openstack/ironic: Mask instance secrets in API responses  https://review.openstack.org/32676814:05
*** mkoderer has quit IRC14:06
openstackgerritMerged openstack/ironic: Fix pep8 on Python3.5  https://review.openstack.org/36197114:07
*** mtanino has joined #openstack-ironic14:07
*** syed_ has joined #openstack-ironic14:08
*** mkoderer has joined #openstack-ironic14:08
*** pcaruana has joined #openstack-ironic14:09
* lucasagomes won't see those annoying pep8 errors anymore ^ yay...14:10
openstackgerritImre Farkas proposed openstack/ironic: DRAC RAID configuration  https://review.openstack.org/17631914:11
openstackgerritRuby Loo proposed openstack/python-ironicclient: Set/unset node's target RAID config via OSC command  https://review.openstack.org/36078714:12
*** thiagop has joined #openstack-ironic14:13
*** Goneri has joined #openstack-ironic14:14
thiagopGood morning, Ironicers14:14
*** ksumit has joined #openstack-ironic14:15
rloomorning thiagop14:15
thiagoprloo: o/14:17
*** moshele has quit IRC14:18
rloodtantsur: 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
rloodtantsur: some people wanted 'passthru', and others wanted 'passthrough' (more English).14:19
dtantsurthis does not change my position, though I guess I'm overvoted14:19
rloodtantsur: since osc is meant to be more English, i was going to change it to passthrough.14:19
dtantsurhaving two names for the same command does not sound right to me14:19
rloodtantsur: i am happy to change it again. would like to make you happy :)14:20
*** vishwanathj_zzz is now known as vishwanathj14:20
vdrokmorning milan jroll xavierr thiagop rloo14:21
thiagopvdrok: o/14:21
xavierrgood morning vdrok :D14:22
dtantsurrloo, 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" situation14:22
rloodtantsur: sure, could do that.14:22
*** jaosorior is now known as jaosorior_away14:25
TheJuliaGood morning everyone14:27
openstackgerritMerged openstack/ironic: Add metrics for the ipminative driver  https://review.openstack.org/35282914:27
vdrokmorning TheJulia14:29
*** jtomasek|afk has quit IRC14:30
thiagopGood morning TheJulia14:30
milanmorning vdrok :)14:32
*** clenimar has joined #openstack-ironic14:34
*** ksumit has quit IRC14:34
*** appu103_ has joined #openstack-ironic14:37
xavierrdtantsur: is there a way inspector don't include hyperthreads in the total number of cpus?14:38
dtantsurxavierr, not currently14:39
mat128xavierr, dtantsur: please use 2 separate fields if you don't want it14:39
mat128downstream, we've used names likes "number of threads" instead of "cores" to distinguish14:39
xavierrmat128: oh, good to know14:40
dtantsuryeah, we can make IPA return two fields, then have an option on ironic-inspector side to pick one for properties/cpus14:40
*** appu103_ has quit IRC14:41
mat128xavierr: exerpt from downstream flavor14:41
mat128http://paste.openstack.org/show/564588/14:41
*** appu103_ has joined #openstack-ironic14:41
mat128vcpus is "total threads presented"14:41
mat128then deeper you have actual processor count14:41
mat128we have plenty more data in the ironic node (not on vpn, cant show right now), but flavor only "filters" on those14:42
xavierrmat128: I see. I need to do that!14:43
jrollneat14:43
*** ashestakov has joined #openstack-ironic14:44
gabriel-bezerrasomething to observe: the cpu capabilities could also show the hyperthreading14:46
TheJulia++ since some applications run poorly on hyperthreads, so it might be useful to know in advance14:47
gabriel-bezerracould be "ht": "cpu_smt" in the https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/plugins/capabilities.py#L2314:47
JayFjroll: jaosorior_away: I'll pull my -214:49
jrollthanks14:50
*** e0ne has joined #openstack-ironic14:51
lucasagomesrloo, commented on the ML. tl;dr I don't think we should keep 2 commands for the same functionality14:52
lucasagomesIMO we should deprecate the "passthrough" one14:52
lucasagomesbecause we use passthru in the API, that's the only reason14:53
*** appu103_ has quit IRC14:53
*** jtomasek has joined #openstack-ironic14:53
*** anush__ has joined #openstack-ironic14:54
JayFlucasagomes: fwiw there's no "deprecate". passthrough doesn't exist yet14:56
*** mari0jv has joined #openstack-ironic14:56
lucasagomesJayF, oh I thought it was in osc already14:57
lucasagomesthanks for clarifying, so yeah I think we should only add passthru then14:57
rloolucasagomes: 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
rloolucasagomes: having said that, passthru might be an acceptable english term.15:02
thiagopdtantsur: answered your comments on 358041, thank you. Wdyt I should do about that decorator?15:02
lucasagomesrloo, gotcha. Is it a queens english vs american english thing ?15:02
rloolucasagomes: and yeah, am asking now before we push out something that we don't want to do :) so thx for your comment.15:03
rloolucasagomes: more like proper English :)15:03
JayFI mean, thru was in every dictionary I found :)15:03
lucasagomesheh I see15:03
JayFthrough is the base word for thru15:03
JayFbut it's not like it's improper slang or something15:03
rlooJayF: ++. 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_Thttp://grammarist.com/spelling/through-thru/15:05
Mr_Thttp://www.merriam-webster.com/words-at-play/how-thru-turned-into-through15:05
*** mjura has quit IRC15:05
Mr_Tthis is (almost) literally the definition of bike-shedding, though, so I'll back away now15:05
mat128lmao15:06
rlooMr_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 IRC15:07
rloohonestly, i would have left it as passthru, but i saw dean troyer mention it as passthrough :)15:07
Mr_Tafter reading those articles, I vote "drive-thru"15:08
JayFrloo: if you haven't met him, Mr_T is the downstream product manager for OnMetal15:09
* Mr_T waves15:09
rloohi Mr_T! thx for your vote.15:09
Mr_TJayF 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-ironic15:09
rloooperator and user feedback is more appreciated than developer feedback wrt 'words' :)15:09
JayFMr_T: on either Thursday or Friday I broke out the OED and looked up "thru", lol15:10
*** jaosorior_away is now known as jaosorior15:10
jaosoriorJayF: thanks15:10
Mr_TJayF - as long as you go with a respected descriptivist dictionary, it sounds like you're safe :-)15:10
JayFMr_T: *blink*15:11
JayFMr_T: I know those words, just not what they mean all together15:11
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Allow suppressing ramdisk logs collection  https://review.openstack.org/35938115:11
mat128:P15:11
*** dprince has quit IRC15:11
jrollrloo: 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_TJayF - descriptivists call the language as they see it - prescriptionists set the language in stone15:12
rloojroll: 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
jrollrloo: it's possible, even if we don't know the process15:12
JayFMr_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_Tprecisely!15:13
*** hshiina has quit IRC15:13
*** persia has quit IRC15:13
jlvillalGood morning Ironic15:14
thiagopmorning jlvillal15:14
*** michauds has joined #openstack-ironic15:15
*** persia has joined #openstack-ironic15:16
lucasagomesjlvillal, thiagop morning15:16
*** praneshp has joined #openstack-ironic15:16
jlvillal:)15:16
thiagoplucasagomes: o/15:16
*** hoonetorg has joined #openstack-ironic15:16
rloomorning jlvillal15:17
jrollTheJulia: hey, can we do a bifrost release this week?15:17
*** sdake_ has joined #openstack-ironic15:18
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command  https://review.openstack.org/25145815:18
thiagopFolks, 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
vdrokmorning jlvillal15:19
thiagopbtw, it's failing one of our jobs due to a timeout downloading IPA images, but I did rechecked it15:19
thiagopI did recheck*15:19
*** sdake has quit IRC15:20
openstackgerritThiago Paiva Brito proposed openstack/ironic: Bring configurations from tempest to ironic_tempest_plugin  https://review.openstack.org/35559115:24
*** sdake has joined #openstack-ironic15:24
*** sdake_ has quit IRC15:26
*** Sukhdev has joined #openstack-ironic15:31
*** ChubYann has joined #openstack-ironic15:31
*** harlowja_at_home has joined #openstack-ironic15:32
TheJuliajroll: I don't see why not, it wouldn't be like a 2.0 from a functionality standpoint not much has really changed this cycle15:33
*** thrash is now known as thrash|biab15:33
*** rpioso has joined #openstack-ironic15:36
*** xuao has joined #openstack-ironic15:38
*** tesseract- has quit IRC15:42
*** pcaruana has quit IRC15:43
*** rcernin has quit IRC15:44
jrollTheJulia: okay, release team would like one, just to be sure there will be one this cycle (email from doug on -dev)15:45
TheJuliak, 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
jrollheh15:47
jrollyeah no rush, just this week sometime15:47
jrollironic/neutron meeting in 10, I'm running it so this is more reminding myself than anyone else :P15:50
*** wajdi has joined #openstack-ironic15:53
*** amotoki has quit IRC15:55
dtantsurthiagop, sorry, on the meeings now, will answer in a few16:00
*** thrash|biab is now known as thrash16:01
* jlvillal wonders what Ilya Shakhat's IRC nick is. To ask about Stackalytics REST API strangeness16:01
*** david-lyle_ is now known as david-lyle16:01
jrolljlvillal: https://launchpad.net/~shakhat16:02
jlvillaljroll: Thanks. I'm guessing he is offline or that is not his IRC nick.16:02
dtantsurmorning jlvillal, JayF and everyone I missed while on a meeting16:03
JayFo/16:03
vsaienk0jroll: 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
jrollvsaienk0: in ironic/neutron meeting now16:04
devananda|OPSmorning, all16:04
*** devananda|OPS is now known as devananda16:05
dtantsurmorning devananda16:05
openstackgerritMerged openstack/python-ironicclient: Add baremetal port list command to OSC plugin  https://review.openstack.org/34672216:05
pas-hajlvillal: Ilya's nick is "ilyashakhat"16:06
pas-hahe seems busy ATM though16:07
openstackgerritSukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers  https://review.openstack.org/36145116:07
jlvillalpas-ha: Thanks!16:08
*** ifarkas is now known as ifarkas_afk16:11
*** e0ne has quit IRC16:14
*** ppiela has joined #openstack-ironic16:18
*** sdake has quit IRC16:19
*** ohamada has quit IRC16:19
*** rama_y has joined #openstack-ironic16:20
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Update openstack baremetal driver commands  https://review.openstack.org/36204716:20
*** dprince has joined #openstack-ironic16:21
*** e0ne has joined #openstack-ironic16:23
*** jpich has quit IRC16:25
NobodyCamGood Morning Ironic'ers :)16:26
*** sdake has joined #openstack-ironic16:30
lucasagomesNobodyCam, hi there16:33
lucasagomesdevananda, morning16:33
dtantsurmorning NobodyCam16:33
openstackgerritRajini Ram proposed openstack/ironic: Option to enroll nodes with drac driver  https://review.openstack.org/36226716:33
openstackgerritRajini Ram proposed openstack/ironic: Option to enroll nodes with drac driver  https://review.openstack.org/36226716:34
NobodyCamMorning lucasagomes, dtantsur16:34
*** aNupoisc has joined #openstack-ironic16:36
*** Guest35918 is now known as mgagne16:39
*** mgagne has quit IRC16:39
*** mgagne has joined #openstack-ironic16:39
*** harlowja_at_home has quit IRC16:39
openstackgerritRodion Promyshlennikov proposed openstack/python-ironicclient: Add openstack baremetal driver commands  https://review.openstack.org/35005016:40
openstackgerritRodion Promyshlennikov proposed openstack/python-ironicclient: Add openstack baremetal driver commands  https://review.openstack.org/35005016:44
*** lucasagomes is now known as lucas-afk16:45
* lucas-afk will be back for the meeting16:45
*** jcoufal has joined #openstack-ironic16:46
jlvillalpas-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
thiagopjroll: 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
jlvillalpas-ha: Regarding stackalytics REST API16:47
jrollthiagop: security groups do work16:47
jroll(or should?)16:47
thiagopjroll: so not-running the security group tests is the real problem. Will remove the check then.16:49
jrollthiagop: sounds like it16:49
*** jtomasek has quit IRC16:50
*** rbrud has quit IRC16:51
*** gfhellma has joined #openstack-ironic16:54
vdrokmorning NobodyCam16:55
NobodyCammorning vdrok :)16:55
*** lucas-afk is now known as lucasagomes17:00
*** jaosorior has quit IRC17:00
vdrokjroll: meeting time :)17:00
jrollheh17:00
* jroll easily distracted17:00
*** jtomasek has joined #openstack-ironic17:10
JayFmat128: I'll be honest, part of why I haven't reviewed that upper-constraints is because it's a little hard to grok17:23
JayFmat128: I presume you built this and checked pip freeze inside it?17:23
mat128You can see the pip install process installing the right versions17:23
mat128I can pinpoint lines17:23
mat1281 sec17:23
JayFthanks; I was about to look for that myself \o/17:23
JayFI can land this before the end of the meeting once you show me that17:23
JayFI've looked over the code17:23
mat128I think I liked it in the comments17:24
JayFjcook: jroll ^ we should make sure to implement the upper-constraints stuff in our downstream ipa build repo17:24
mat128stable/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_65817:24
*** sabeen has quit IRC17:25
mat128Pint 0.617:25
mat128https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h=stable/liberty17:25
mat128otherwise would be 0.7.2 (https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt)17:25
mat128or even later17:25
JayFah17:25
JayFI'll review the code again and land it then17:25
mat128it 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_file17:27
JayFmat128: landing17:31
*** trown is now known as trown|lunch17:32
mat128:D17:36
mat128In that case, backports are already based on the right commit (let me double check)17:36
*** electrofelix has quit IRC17:37
*** praefect has joined #openstack-ironic17:40
*** fragatina has quit IRC17:44
*** Sukhdev has quit IRC17:45
*** praefect has quit IRC17:50
*** bdeschenes has joined #openstack-ironic17:52
*** xuao has quit IRC17:54
*** rajinir has joined #openstack-ironic17:55
rloothx devananda18:00
jrolldevananda: feel free to add that dash to the whiteboard18:00
devanandajroll: I did18:00
jrollwoo18:00
dtantsurdevananda, "small changes" is a good idea. gertty doesn't support it?18:00
NobodyCamthank you :)18:00
devanandajroll: question is - should it replace our official dashboard?18:00
devanandadtantsur: nope18:01
vdroktime to leave for me, good night everyone!18:01
dtantsurwe should ask them to support it for sure..18:01
dtantsurg'night vdrok18:01
jrolldevananda: define official :)18:01
rloodevananda: are 'small things' also included in 'recently proposed to master'?18:01
* dtantsur is leaving as well, g'night18:01
lucasagomesdevananda, unfortunately replacing the module in sys.module is causing other tests to fail when runnning in parallel18:01
jrolldevananda: honestly, I think we should merge them, both have some useful categories18:01
devanandajroll: https://github.com/openstack/gerrit-dash-creator/blob/master/dashboards/ironic.dash18:01
*** Sukhdev has joined #openstack-ironic18:01
devanandaofficial :-D18:01
devanandalucasagomes: yep18:01
devanandarloo: the queries are separate. it's possible for there to be overlap.18:02
*** moshele has joined #openstack-ironic18:02
devanandarloo: something could be: starred; small; recent. and appear in all three.18:02
*** fragatina has joined #openstack-ironic18:02
mat128devananda: I'd love your new categories to appear in gerrit-dash-reator18:02
jrolldevananda: yeah, I'd suggest adding some of those categories to the 'official' one18:02
*** dtantsur is now known as dtantsur|afk18:02
devanandak18:02
mat128rloo: you can always click on a sub-category to see the query18:03
devanandajroll: I'm particularly interested to share with you the "starred by PTL" list, and how Swift is using that18:03
rloodevananda: 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
mat128Small things will tell you "delta"18:03
*** sabeen has joined #openstack-ironic18:03
devanandarloo: easy enough :)18:03
jrolldevananda: I'm not good enough at keeping my stars up to date to do that right now :)18:03
rloomat128: i didn't know they were clickable. but still, makes it easier for my poor brain.18:04
rloojroll: what about a starred by core? ha ha.18:04
mat128rloo: maybe you can do reviewers.contains(one_of(ironic_cores()))18:05
jrollrloo: I mean, I think we could make some good dashboards if we paid attention to how we star things and such18:05
rloojroll: 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" category18:06
mat128if people only look at those18:06
rloojroll: 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
jrollrloo: yeah, I hear ya, but I guess it depends what you look at18:07
jrollrloo: if the first thing you do is think "what do I feel like looking at", I can tell you which ones will get looked at18:07
devanandaswift 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
mat128rloo: on the bright side, small might be easier to go through18:07
jrollrloo: (small, needs approval, cores care)18:07
rloojroll: right. so those dashboards are useful/sort of indicate/importance based on order of the categories and where they are situated :)18:07
jrollrloo: I think so, yeah18:08
devanandajust a guess, but perhaps something like that could help us to focus on patches we collectively feel are important18:08
jrollyeah, I agree18:09
jrollinstead of asking people in irc18:09
mat128devananda: https://github.com/openstack/gerrit-dash-creator/blob/master/dashboards/swift.dash18:09
rlooto date, i try to follow (apart from intel priorities) what jroll/PTL sez is important. so upstream priorities, blah blah.18:09
devanandaI 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 on18:09
jroll*if* we're diligent about maintaining stars18:09
devanandarloo: 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 plane18:10
rloodevananda: oh, yes, that's true.18:10
devanandathat's my own failing at tracking them, in a sense, so I've started using stars to keep track18:10
mat128devananda: do you have a link to the swift ranking? Can't find it in gerrit-dash-creator18:10
rloodevananda: 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 IRC18:11
*** trown|lunch is now known as trown18:11
*** jcoufal has quit IRC18:11
devanandamat128: searching, one sec18:11
*** vishwanathj has joined #openstack-ironic18:11
devanandamat128: http://d.not.mn/swift_community_dashboard.html18:12
devanandamat128: dash on the right, "community starred patches"18:12
openstackgerritJay Faulkner proposed openstack/ironic: Metrics for ConductorManager  https://review.openstack.org/35264818:13
mat128thank you18:13
mat128interesting18:13
openstackgerritThiago Paiva Brito proposed openstack/ironic: Bring configurations from tempest to ironic_tempest_plugin  https://review.openstack.org/35559118:13
*** rbrud has joined #openstack-ironic18:14
devanandajroll, rloo: which of the sections in the current dash do you find valuable / want to keep?18:16
rloodevananda: comparing...18:18
JayFdevananda: needs approval is my favorite section, but I think your new dash has that18:18
*** jvgrant__ has quit IRC18:18
*** jvgrant has joined #openstack-ironic18:18
devanandaJayF: yep18:18
jrolldevananda: "Needs Reverify" is useful, and "5 Days Without Feedback" *can* be useful but often isn't18:18
devanandajroll: keeping reverify, I agree that's useful18:19
JayF5d without feedback could be named "unpopular/hard things to review"18:19
*** moshele has quit IRC18:19
JayFthat's almost always what's in the list, stuff that's driver specific or extra sized18:19
jrollyeah18:19
*** jvgrant_ has joined #openstack-ironic18:19
rloodevananda: the ones that aren't in your proposed: needs reverify, my patches requiring attention (only to guilt me)18:20
devanandais "open backports" going to be useful to enough folks to put on the dash?18:20
JayFhonestly, backports in the review dash are frustrating18:20
JayFbecause I can't land them18:20
rloodevananda: 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
devanandaJayF: k, that's what I thought.18:21
rlooopen 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
devanandaso, it sounds like ya'll would prefer age:5d to be removed18:21
*** sabeen has quit IRC18:22
jrollI would want backports in review dash because he always forgets them18:22
jrolls/he/I/18:22
devanandaJayF: would your frustration at seeing it inspire you to nag me or jroll? :)18:22
jrollheh18:23
JayFI 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 :P18:23
jroll++18:23
*** jvgrant has quit IRC18:23
JayFOr I could just volunteer to be a stable maintainer for IPA...18:23
rloowhy 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! this18:23
mat128lol18:23
devanandarloo: I wish :)18:24
mat128#gerrit18:24
lucasagomesdevananda, thinking of a way to do what you want, I think we could mock the "wsexpose" method from wsme and reload the module18:25
mat128what world is this18:25
lucasagomesdevananda, then check the call_list()18:25
mat128gerrit project not even using gerrit18:25
jrollmat128: wat18:25
*** jcoufal has joined #openstack-ironic18:25
mat128jroll: was looking for gerrit's source code18:25
mat128jroll: https://gerrit.googlesource.com/gerrit/18:26
mat128all I can find ..18:26
JayFyep and https://www.gerritcodereview.com/ is the official site18:26
mat128yes18:26
jrollhttps://gerrit-review.googlesource.com/#/projects/gerrit,dashboards/default18:26
mat128they want you to use git push bleh:/refs/for/18:27
jrolleven on their homepage in the 'reviews' link18:27
*** moshele has joined #openstack-ironic18:27
mat128oh wow18:28
* mat128 fails18:28
jroll:P18:28
jrollit is weird they don't recommend git-review though18:28
devanandaJayF: does IPA have stable branches?18:28
mat128jroll: git-review https://pypi.python.org/pypi/git-review/1.25.018:28
JayFdevananda: absolutely18:29
mat128openstack-led project18:29
jrollmat128: heh, go figure18:29
lucasagomesfolks calling it a day18:29
lucasagomeshave a great evening all18:29
JayFo/18:29
mat128cya18:29
lucasagomesdevananda, left a comment, it's late here so I haven't tested18:29
devanandalucasagomes: 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 approach18:29
lucasagomesbut I think the idea might work18:29
devanandalucasagomes: thanks much. I'll give it a shot18:29
lucasagomesdevananda, even if we reload the module?18:29
*** moshele has quit IRC18:30
lucasagomesdevananda, http://paste.openstack.org/show/564635/18:31
devanandalucasagomes: yea. I tried replacing wsexpose in another attempt, which I didn't post to gerrit18:32
devanandabecause it had the same problem18:32
lucasagomesdevananda, I can see the functions there, probably we need to sanitize the list of calls and check if they are functions or not18:32
lucasagomesdevananda, I se e:-(18:32
lucasagomesI will try more tomorrow if you don't find a way tonight18:32
devanandaI think it stems from the way I'm overloading sys.modules entries18:32
devanandanew dashboard combination: http://goo.gl/dg2lDE18:32
devanandaif ya'll like, I'll propose that to gerrit-dash-creator18:33
devanandalucasagomes: thanks. have a good night!18:33
lucasagomesyou too18:33
lucasagomessee yall18:33
*** lucasagomes is now known as lucas-dinner18:33
*** causten has quit IRC18:34
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Add metrics for IPA extensions  https://review.openstack.org/36011118:35
devanandajroll, rloo: new dash: http://goo.gl/dg2lDE . feedback requested before I propose it.18:35
*** jcoufal has quit IRC18:36
jrolldevananda: can we add not jenkins -1 to the "Older (>1wk) Open Patches" ?18:36
devanandajroll: ah. yes :)18:36
jrollnice18:36
jrollother than that lgtm18:37
* jroll runs an errand18:37
*** jcoufal has joined #openstack-ironic18:37
*** jvgrant_ has quit IRC18:38
*** gfhellma has quit IRC18:39
*** jvgrant_ has joined #openstack-ironic18:39
*** causten has joined #openstack-ironic18:40
openstackgerritJay Faulkner proposed openstack/ironic: Add metrics to administrator guide  https://review.openstack.org/35681118:40
JayF^ should be a quick review if anyone has a moment :)18:41
rloodevananda: the newer ones -- is that a complete list or does it take the top X? similar question for older patches.18:41
rloodevananda: otherwise, looks good to me, thx! (wrt the other question, cuz you had 'limit 10' for small which i like)18:42
openstackgerritMerged openstack/ironic-python-agent: Changed an assert to more specific assert method  https://review.openstack.org/36212418:42
openstackgerritMerged openstack/ironic-python-agent: Using assertIsNone() is preferred over assertEqual()  https://review.openstack.org/36181118:43
*** Sukhdev has quit IRC18:46
jcookJayF upper constraints stuff, sorry, need a tl;dr18:49
JayFjcook: 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 ramdisk18:50
JayFjcook: 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 IRC18:51
mat128JayF: if you use docker_build.bash you'll get it for free18:51
mat128unsure where you hook yourself to the build process18:51
JayFmat128: we literally build outta a completely separate repo, and clone IPA in and build docker via that18:51
mat128ah, just read last part18:51
mat128got it18:52
jcookI grokish18:52
JayFmat128: pretty much because that repo is full of vendor tools and such18:52
jcookish18:52
mat128we have something similar18:52
JayFAre you all doing more for cleaning now?18:52
*** jcoufal has joined #openstack-ironic18:52
mat128I wish18:52
mat128but we still have a downstream repo that handles the build + publish as an artifact part18:53
mat128and local patches18:53
mat128tl;dr http://paste.openstack.org/show/564637/18:53
mat128we build both for now18:53
mat128we are soon starting work on the proper cleaning steps18:54
mat128or so says my product owner :)18:54
anteayamat128: 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 team18:54
*** praneshp has quit IRC18:56
mat128anteaya: 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 offend18:57
*** jjohnson2 has joined #openstack-ironic18:58
mat128anteaya: noted.18:58
anteayait isn't inclusive and thanks for listening18:58
rloovdrok, devananda (and others): min 1 year for osc deprecations: http://lists.openstack.org/pipermail/openstack-dev/2016-August/102467.html18:58
openstackgerritMerged openstack/ironic: IPMINative: Check the boot mode when setting the boot device  https://review.openstack.org/35418719:05
*** jcoufal has quit IRC19:06
*** jcoufal has joined #openstack-ironic19:07
*** praneshp has joined #openstack-ironic19:13
openstackgerritRamamani Yeleswarapu proposed openstack/python-ironicclient: Add OSProfiler support to Ironicclient  https://review.openstack.org/36233119:15
*** zhugaoxiao has quit IRC19:16
*** causten has quit IRC19:16
*** zhugaoxiao has joined #openstack-ironic19:16
*** gfhellma has joined #openstack-ironic19:20
*** causten has joined #openstack-ironic19:26
openstackgerritRuby Loo proposed openstack/python-ironicclient: Adds node boot device & passthru OSC commands  https://review.openstack.org/35740019:28
openstackgerritRuby Loo proposed openstack/python-ironicclient: Add node validate OSC command  https://review.openstack.org/35932219:31
*** causten has quit IRC19:33
thiagopjroll: good news from jenkins on 35559119:36
*** fragatina has quit IRC19:39
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add base cinder common interface  https://review.openstack.org/35562519:39
*** fragatina has joined #openstack-ironic19:39
openstackgerritMerged openstack/ironic: Option to enroll nodes with drac driver  https://review.openstack.org/36226719:43
rajinirifarkas_afk: Need help with python-dracclient installation19:43
jlvillalpas-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-ironic19:45
jrollthiagop: nice!19:45
*** mrtenio has joined #openstack-ironic19:45
openstackgerritJulia Kreger proposed openstack/ironic: Add tempest tests for node Adoption  https://review.openstack.org/34497519:46
*** ijw has joined #openstack-ironic19:47
*** sdake has quit IRC19:48
*** aswadr_ has quit IRC19:51
*** iurygregory has joined #openstack-ironic19:56
openstackgerritStephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information  https://review.openstack.org/20098319:58
*** derekh has quit IRC20:03
*** jhesketh has quit IRC20:05
openstackgerritOleksandr Berezovskyi proposed openstack/ironic-inspector: [WIP] Implement rules for ports  https://review.openstack.org/36010620:05
*** e0ne has quit IRC20:06
*** causten has joined #openstack-ironic20:10
*** jhesketh has joined #openstack-ironic20:12
*** causten has quit IRC20:15
*** trown is now known as trown|outtypewww20:16
openstackgerritJarrod Johnson proposed openstack/pyghmi: Fix BMC compatibility with Python 2.6  https://review.openstack.org/36239620:16
*** srobert has joined #openstack-ironic20:19
*** Sukhdev has joined #openstack-ironic20:20
openstackgerritCaio Oliveira proposed openstack/python-oneviewclient: [WIP] Creating certificate manager for OneView  https://review.openstack.org/33499220:20
openstackgerritThiago Paiva Brito proposed openstack/ironic: Reusing oneview_client when possible  https://review.openstack.org/35804120:21
openstackgerritStephanie Miller proposed openstack/ironic: Introduce VolumeConnector object  https://review.openstack.org/21458420:21
openstackgerritStephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume connector operation  https://review.openstack.org/21458520:21
openstackgerritDoug Hellmann proposed openstack/ironic-lib: Update .gitreview for stable/newton  https://review.openstack.org/36240720:21
openstackgerritStephanie Miller proposed openstack/ironic: Add volume_targets table to save target information  https://review.openstack.org/28521820:21
openstackgerritJarrod Johnson proposed openstack/pyghmi: Fix BMC compatibility with Python 2.6  https://review.openstack.org/36239620:21
*** milan has quit IRC20:26
openstackgerritJarrod Johnson proposed openstack/pyghmi: Fix BMC compatibility with Python 2.6  https://review.openstack.org/36239620:27
jlvillalWho uses Python 2.6?20:27
openstackgerritJarrod Johnson proposed openstack/pyghmi: Fix BMC compatibility with Python 2.6  https://review.openstack.org/36239620:27
mat128jlvillal: jjohnson2 mentioned having to support older operating systems20:28
jlvillalOh, that's no fun for them.20:29
* mat128 just realized this isn't listed anywhere20:29
jjohnson2jlvillal, yeah, lot's of centos6/rhel6 users20:29
mat128jjohnson2: I wonder if we should add a python26 gate so we don't break 2.6 compat again20:30
jjohnson2not openstack users20:30
jjohnson2mat128, well, this code might not have ever been 2.6 friendly, it's the BMC side code20:30
jjohnson2mat128, 99.9% of users are using it as an ipmi client only20:30
jrollmat128: infra doesn't support 2.6 anyway20:30
mat128at all20:30
mat128?20:30
jrollthey don't build any images that have it20:30
jjohnson2today might have been the first time anyone wanted to play with a fake bmc in python20:30
openstackgerritStephanie Miller proposed openstack/ironic: Introduce VolumeTarget object  https://review.openstack.org/28521920:31
mat128Ahh20:31
jrollin theory, you could add some PPA and install 2.6, or whatever20:31
jjohnson2fyi, prior to me tagging a release, it goes through a test suite in our environment20:31
openstackgerritStephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations  https://review.openstack.org/28522020:31
jjohnson2(which has not historically included *being* a bmc..)20:31
mat128ah20:32
mat128jjohnson2: so that test environment runs 2.6?20:32
jjohnson2mat128, yeah, we do centos6, sles11, sles12, centos720:33
jlvillalmat128: No on adding Python 2.6 gate. We removed it on purpose :)20:33
jjohnson2it's part of our HPC management stack, which has some things that care more than others about the distro20:33
*** rbrud has quit IRC20:33
*** rbrud has joined #openstack-ironic20:34
mat128jlvillal: if infra slaves still supported running 2.6, I'd vote for bringing it back20:34
jjohnson2we are trying to use as much common pieces as we can between openstack, hpc, et al20:34
mat128considering it's important that the project works on it20:34
jjohnson2mat128, well, *pyghmi* specifically, but I'll say that pyghmi has some extra coverage outside of infra20:34
jlvillalmat128: Not me :) I like to use Python 2.7 things in the code.20:34
jjohnson2since pyghmi is an odd 'openstack' project that hasa non-openstack users20:35
jlvillalmat128: So instead of:  "{0} {1}".format(a, b) in Python 2.7 we can do: "{} {}".format(a, b)20:35
jjohnson2with the openstack glue manifested as an ironic driver20:35
jjohnson2jlvillal, also, being able to toss a bytearray at struct.unpack without having to make it a 'bytes' first...20:36
jlvillaljjohnson2: :)20:36
jjohnson2(a particular tedious frustration today...)20:36
mat128jlvillal: yes, all the great new things :)20:36
jjohnson2what's *really* fun is doing decent TLS security in python 2.6.... that is tricky20:37
jjohnson2anyway, sorry about the git review spam20:38
jjohnson2I forgot it would bug the hell out of the channel...20:38
jrollI for one, am never upset about people writing code :)20:39
jjohnson2well, I could have waiting like 1 minute to amend my commit20:39
jrollnah, not a big deal20:39
jjohnson2instead of amend, git review, oh whoops missed one, amend, review... then check, whoops...20:39
openstackgerritJulia Kreger proposed openstack/bifrost: Update IPA info in troubleshooting.rst  https://review.openstack.org/36242820:39
mat128on the bright side, we get to see commits20:39
jrollit's not like you interrupted anything :)20:39
jrollI work the same way20:39
*** Sukhdev_ has joined #openstack-ironic20:40
*** Sukhdev has quit IRC20:40
jjohnson2I'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 time20:41
*** rama_y has quit IRC20:41
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add storage_interface DB field and object  https://review.openstack.org/34800520:42
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add storage_interface to base driver class  https://review.openstack.org/34800620:42
thiagopjjohnson2: where is located your team here in Brazil?20:44
*** Sukhdev_ has quit IRC20:49
openstackgerritMerged openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks  https://review.openstack.org/35885520:49
*** dprince has quit IRC20:49
*** rama_y has joined #openstack-ironic20:49
jlvillalmat128: ^^  Woot! :)20:51
mat128:D20:51
TheJuliamat128: 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
jlvillalthiagop: Buenos Aires?20:51
jlvillalthiagop: :P20:51
*** causten has joined #openstack-ironic20:52
*** causten has quit IRC20:52
*** sdake has joined #openstack-ironic20:52
*** causten has joined #openstack-ironic20:52
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add base cinder common interface  https://review.openstack.org/35562520:52
thiagopjlvillal: if I were a soccer fan (as 99.9% of brazillians), I would be pissed with this comment20:53
thiagoplol20:53
jlvillalthiagop: heh20:53
* mat128 understands you are part of the 0.1%20:53
jlvillalthiagop: I was born in Buenos Aires. But I was only there until 3 years old.20:54
thiagopmat128: I'm the kind of guy that thinks "if the soccer selection wins, ok. If it loses, ok too"20:54
gabriel-bezerraBuenos Aires has the beautiful tango20:54
thiagopI'm the typical brazillian weirdo20:54
jrollmat128: +2 on https://review.openstack.org/#/c/360795 - devananda mind approving that?20:55
jrollmat128: and you've the wrong commit sha here on liberty https://review.openstack.org/#/c/360793/320:56
thiagopjlvillal: Buenos Aires seems like a nice (and cheap for us) place to go spend some vacation days20:56
mat128I need to update the liberty one20:56
mat128yes20:56
jrollheh20:56
jrollcool20:56
gabriel-bezerralet's not talk about soccer20:56
gabriel-bezerra:P20:56
thiagop^agree20:57
openstackgerritMathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks  https://review.openstack.org/36079320:58
mat128jroll: liberty updated. commit message was wrong (validated content manually)20:58
jlvillalWhat would people think about having patchbot in this channel?20:59
mat128patchbot?20:59
wajdi^^20:59
jlvillalThey use it in #openstack-swift20:59
openstackgerritRuby Loo proposed openstack/python-ironicclient: Add --node to 'openstack baremetal port list'  https://review.openstack.org/36244520:59
jlvillalIf you say something like:  i submitted https://review.openstack.org/#/c/360135/ to make the tests actually pass21:00
jlvillalIt replies: <patchbot> timburke: patch 360135 - python-swiftclient - Make functests py3-compatible21:00
mat128ah, just like bugs21:00
thiagopjlvillal: very like the bugs one21:00
mat128yeah that'd be great21:00
jlvillalmat128: Right!21:00
thiagop+121:01
* jroll isn't opinionated21:01
jjohnson2thiagop, I need to ask21:01
openstackgerritJulia Kreger proposed openstack/bifrost: Support to wait for node deployment to complete  https://review.openstack.org/25424521:01
jjohnson2for me it's a good day when I remember what country people are in21:01
thiagoplol21:02
jjohnson2there's for me two geographies: central North Carolina and everywhere else21:02
thiagopjjohnson2: I think lenovo begun a new project with our university here @ Campina Grande, just checkin'21:02
devanandajroll: don't mind at all.21:02
JayFit's pretty eaqsy to /ignore the bot if you dislike it too21:02
jrollthanks!21:02
JayFnot like we're making a bot that does lots of things do more things21:02
JayFnew bot = easy to ignore/block21:03
mat128and old bot is at maxchan limit21:03
mat128:)21:03
thiagop#pixiesay --angry WHAT?21:03
thiagoppixie is not on the house...21:03
TheJuliajjohnson2: But... there is the coast, the piedmont, and the mountains... those are different areas ;)21:04
jjohnson2TheJulia, coast and mountains are rest of world21:04
jjohnson2aka, people who I see in person, versus people I only ever see online21:05
jjohnson2TheJulia, Campinas and Fortaleza21:05
jjohnson2thiagop, err Campinas and Fortaleza21:05
TheJuliajjohnson2: is North of I-540 the rest of the world as well ?  :)21:05
thiagopjjohnson2: cool21:05
jjohnson2TheJulia, depends on if the ydrive to Morrisville and work in one of these two buildings ;)21:06
*** causten has quit IRC21:07
TheJuliajjohnson2: no, the drive from 401 to 540 makes me go crazy in morning traffic. :)21:07
jlvillaljjohnson2: You reminded me of this :)  https://s-media-cache-ak0.pinimg.com/564x/96/00/1b/96001b9c6594229fb7468303c0e26157.jpg21:07
jjohnson2TheJulia, then rest of world21:07
*** causten has joined #openstack-ironic21:07
* TheJulia informs her fiancee that they live in the rest of the world21:08
jjohnson2jlvillal, yeah, that's about right21:08
*** Goneri has quit IRC21:09
jjohnson2well, I'm going home21:09
jjohnson2jlvillal, well, adjusted for my locale at least21:09
jlvillaljjohnson2: heh21:10
jjohnson2I will say that Brazil is easier than China, close enough time zone versus the worst possible time zone difference21:10
*** vishwanathj has quit IRC21:11
*** vishwanathj has joined #openstack-ironic21:11
jjohnson2I'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 dedicated21:11
*** daemontool has quit IRC21:11
jjohnson2well good evening all21:11
thiagopjjohnson2: lol21:12
thiagopgood evening21:12
gabriel-bezerragood evening21:12
*** wajdi_ has joined #openstack-ironic21:13
*** daemontool has joined #openstack-ironic21:13
openstackgerritOleksandr Berezovskyi proposed openstack/ironic-inspector: [WIP] Implement rules for ports  https://review.openstack.org/36010621:15
*** wajdi has quit IRC21:17
*** wajdi_ has quit IRC21:18
*** srobert has quit IRC21:19
openstackgerritJulia Kreger proposed openstack/ironic: Add tempest tests for node Adoption  https://review.openstack.org/34497521:22
*** jcoufal has quit IRC21:24
*** jcoufal has joined #openstack-ironic21:24
*** ashestakov has quit IRC21:30
*** sdake has quit IRC21:33
*** ijw has quit IRC21:43
*** aNupoisc has quit IRC21:44
*** aNupoisc has joined #openstack-ironic21:44
*** causten has quit IRC21:45
openstackgerritMerged openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks  https://review.openstack.org/36079521:45
*** causten has joined #openstack-ironic21:50
openstackgerritMerged openstack/python-ironicclient: Add openstack baremetal chassis commands  https://review.openstack.org/34581521:53
*** lucas-dinner has quit IRC21:56
*** sai has quit IRC21:58
TheJuliathiagop: 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 ironic21:58
thiagopTheJulia: fine to me21:58
*** rook has quit IRC21:58
*** devananda has quit IRC21:59
*** sai has joined #openstack-ironic21:59
*** lucasagomes has joined #openstack-ironic21:59
*** devananda has joined #openstack-ironic22:00
*** rook has joined #openstack-ironic22:00
*** rook is now known as Guest6478222:00
*** devananda has quit IRC22:01
*** devananda has joined #openstack-ironic22:02
*** gfhellma1 has joined #openstack-ironic22:03
*** gfhellma has quit IRC22:03
mrdaMorning Ironic22:09
thiagopmorning mrda , night Ironicers!22:11
thiagopsee ya22:11
*** thiagop has quit IRC22:11
mrdao/22:15
*** gfhellma has joined #openstack-ironic22:19
*** gfhellma1 has quit IRC22:20
*** ijw has joined #openstack-ironic22:22
*** bdeschenes has quit IRC22:23
*** michauds has quit IRC22:23
*** ijw has quit IRC22:23
*** rbrud has quit IRC22:24
openstackgerritJulia Kreger proposed openstack/ironic: Add tempest tests for node Adoption  https://review.openstack.org/34497522:26
*** fragatin_ has joined #openstack-ironic22:35
*** fragatina has quit IRC22:38
openstackgerritMerged openstack/ironic: Fix for check if dynamic allocation model is enabled  https://review.openstack.org/35742522:43
*** Goneri has joined #openstack-ironic22:43
*** daemontool has quit IRC22:49
*** daemontool has joined #openstack-ironic22:50
*** patchbot has joined #openstack-ironic22:50
jlvillalTesting this: https://review.openstack.org/35742522:51
*** notmyname has joined #openstack-ironic22:51
notmynamejlvillal: hi22:51
notmynamep 11234422:52
jlvillalnotmyname: Hi :)22:52
patchbothttps://review.openstack.org/#/c/112344/ - tuskar - Remove Exception ProcessExecutionError (MERGED)22:52
notmynamejlvillal: so I've got one bug in patchbot that I'll be fixing soon22:52
jlvillalnotmyname: I thought it handled the URLs too22:52
notmynamepatch 134234 will report something twice22:52
patchbot(patch <patch number>) -- Generates a review.openstack.org URL to <patch number>.22:52
patchbothttps://review.openstack.org/#/c/134234/ - magnum - Incorporate feedback from the Initial commit review (MERGED)22:52
notmynameyeah, it does22:52
notmynamehttps://review.openstack.org/#/c/112344/22:52
patchbotpatch 112344 - tuskar - Remove Exception ProcessExecutionError (MERGED)22:52
jlvillalTesting this: https://review.openstack.org/35742522:52
jlvillalnotmyname: But not that way?22:52
*** sabeen has quit IRC22:55
notmynamejlvillal: no, not right now.22:56
jlvillalnotmyname: Okay. Thank you :)22:57
*** mmedvede has quit IRC23:02
*** rajinir has quit IRC23:05
*** mpsairam has joined #openstack-ironic23:06
*** mmedvede has joined #openstack-ironic23:06
*** bdeschenes has joined #openstack-ironic23:10
*** mari0jv has quit IRC23:13
*** praneshp has quit IRC23:14
*** vishwanathj has quit IRC23:18
*** vishwanathj has joined #openstack-ironic23:18
*** rpioso has quit IRC23:18
*** adu has joined #openstack-ironic23:23
*** rama_y has quit IRC23:28
*** vishwanathj has quit IRC23:29
*** vishwanathj has joined #openstack-ironic23:29
*** vishwanathj has quit IRC23:31
*** vishwanathj has joined #openstack-ironic23:31
*** sabeen has joined #openstack-ironic23:33
*** Guest64782 is now known as rook23:34
*** vishwanathj has quit IRC23:37
*** vishwanathj has joined #openstack-ironic23:37
*** praneshp has joined #openstack-ironic23:38
*** vishwanathj has quit IRC23:47
*** vishwanathj has joined #openstack-ironic23:47
*** anush__ has quit IRC23:58

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!