Tuesday, 2017-01-03

*** rbudden has quit IRC00:00
*** zackf has quit IRC00:00
*** adreznec has joined #openstack-ironic00:01
*** zackf has joined #openstack-ironic00:05
*** chas__ has quit IRC00:08
*** chas has joined #openstack-ironic00:08
*** zackf has quit IRC00:09
*** chas has quit IRC00:13
*** rbudden has joined #openstack-ironic00:18
*** rbudden has quit IRC00:22
*** rbudden has joined #openstack-ironic00:36
*** rbudden has quit IRC00:44
*** hoangcx has joined #openstack-ironic00:45
*** zackf has joined #openstack-ironic00:47
*** david-lyle has joined #openstack-ironic00:51
*** zackf has quit IRC00:51
*** SerenaFeng has joined #openstack-ironic00:54
*** david-lyle has quit IRC00:55
*** zackf has joined #openstack-ironic01:02
*** phuongnh has joined #openstack-ironic01:05
*** zackf has quit IRC01:07
*** zackf has joined #openstack-ironic01:34
*** zackf has quit IRC01:39
*** zhangjl has joined #openstack-ironic01:41
*** zackf has joined #openstack-ironic01:44
*** zackf has quit IRC01:48
*** yufei has joined #openstack-ironic02:00
*** zhs_ has joined #openstack-ironic02:14
*** yufei has quit IRC02:22
*** yufei has joined #openstack-ironic02:24
openstackgerritlicanwei proposed openstack/ironic: remove redundant character  https://review.openstack.org/41611102:30
*** MattMan_1 has quit IRC02:31
*** MattMan_1 has joined #openstack-ironic02:31
*** gcb has joined #openstack-ironic02:36
*** zhs__ has joined #openstack-ironic02:48
*** zhs_ has quit IRC02:51
*** david-lyle has joined #openstack-ironic02:52
*** david-lyle has quit IRC02:57
*** chas has joined #openstack-ironic03:10
*** chas has quit IRC03:15
*** tuanluong has joined #openstack-ironic03:25
*** links has joined #openstack-ironic03:28
*** vikrant has joined #openstack-ironic03:28
*** hw_wutianwei has joined #openstack-ironic03:33
*** pmannidi has quit IRC03:33
*** links has quit IRC03:34
*** hoangcx has quit IRC03:34
*** adreznec has quit IRC03:34
*** zhs__ has quit IRC03:34
*** vikrant has quit IRC03:34
*** yufei has quit IRC03:34
*** phuongnh has quit IRC03:34
*** MattMan_1 has quit IRC03:34
*** zhs__ has joined #openstack-ironic03:34
*** vikrant has joined #openstack-ironic03:35
*** yufei has joined #openstack-ironic03:35
*** phuongnh has joined #openstack-ironic03:35
*** links has joined #openstack-ironic03:35
*** hoangcx has joined #openstack-ironic03:35
*** adreznec has joined #openstack-ironic03:35
*** MattMan_1 has joined #openstack-ironic03:35
*** baoli has quit IRC03:38
*** baoli has joined #openstack-ironic03:39
*** yufei has quit IRC03:48
*** yufei has joined #openstack-ironic03:51
*** g3ek has quit IRC03:54
*** pmannidi has joined #openstack-ironic03:57
*** trananhkma has joined #openstack-ironic03:58
*** baoli_ has joined #openstack-ironic04:00
*** baoli has quit IRC04:02
*** g3ek has joined #openstack-ironic04:02
*** baoli_ has quit IRC04:06
*** baoli has joined #openstack-ironic04:07
*** sacharya has quit IRC04:15
*** sacharya has joined #openstack-ironic04:16
*** SerenaFeng has quit IRC04:17
*** sacharya has quit IRC04:21
*** rbudden has joined #openstack-ironic04:21
*** baoli has quit IRC04:28
*** yufei has quit IRC04:35
*** yufei has joined #openstack-ironic04:37
*** SerenaFeng has joined #openstack-ironic04:43
*** SerenaFeng has quit IRC04:49
*** yufei has quit IRC04:57
*** yufei has joined #openstack-ironic04:58
*** chas has joined #openstack-ironic05:11
*** chas has quit IRC05:16
*** sacharya has joined #openstack-ironic05:17
*** yufei__ has joined #openstack-ironic05:21
*** yufei has quit IRC05:21
*** yufei__ is now known as yufei05:21
*** sacharya has quit IRC05:22
*** yufei has quit IRC05:27
*** yufei has joined #openstack-ironic05:38
*** SerenaFeng has joined #openstack-ironic05:41
*** zackf has joined #openstack-ironic05:45
*** jkilpatr has quit IRC05:45
*** jkilpatr has joined #openstack-ironic05:49
*** zackf has quit IRC05:50
*** SerenaFeng has quit IRC06:06
*** SerenaFeng has joined #openstack-ironic06:09
openstackgerritAparna proposed openstack/proliantutils: Update hpssacli to ssacli  https://review.openstack.org/39650906:13
*** zackf has joined #openstack-ironic06:21
*** zhs_ has joined #openstack-ironic06:23
*** zackf has quit IRC06:25
*** zhs__ has quit IRC06:26
*** rbudden has quit IRC06:27
*** hw_wutianwei1 has joined #openstack-ironic06:30
*** hw_wutianwei has quit IRC06:31
*** hw_wutianwei1 is now known as hw_wutianwei06:31
openstackgerritAparna proposed openstack/proliantutils: Update hpssacli to ssacli  https://review.openstack.org/39650906:45
openstackgerritHironori Shiina proposed openstack/ironic: Add REST API for volume connector and volume target operation  https://review.openstack.org/21458606:56
*** yufei has quit IRC07:00
*** zackf has joined #openstack-ironic07:02
*** mjura has joined #openstack-ironic07:02
*** zackf has quit IRC07:07
*** mjura has quit IRC07:09
*** mjura has joined #openstack-ironic07:10
*** chas has joined #openstack-ironic07:12
openstackgerritAparna proposed openstack/proliantutils: Add Sanitize disk erase support using SSA  https://review.openstack.org/39820207:13
*** ccamacho has quit IRC07:15
*** yufei has joined #openstack-ironic07:16
*** chas has quit IRC07:17
*** sacharya has joined #openstack-ironic07:18
*** e0ne has joined #openstack-ironic07:18
*** ChubYann has quit IRC07:19
*** SerenaFeng has quit IRC07:21
*** SerenaFeng has joined #openstack-ironic07:21
*** sacharya has quit IRC07:22
openstackgerritAparna proposed openstack/proliantutils: Add Sanitize disk erase support using SSA  https://review.openstack.org/39820207:24
*** chas has joined #openstack-ironic07:25
*** e0ne has quit IRC07:27
*** tesseract has joined #openstack-ironic07:30
*** e0ne has joined #openstack-ironic07:31
*** moshele has joined #openstack-ironic07:33
mosheleaarefiev_afk: hi, I update https://review.openstack.org/#/c/264257/ according to your comments07:34
patchbot_patch 264257 - ironic-inspector - Adding InfiniBand Support07:34
*** e0ne has quit IRC07:36
*** pmannidi has quit IRC07:37
*** e0ne has joined #openstack-ironic07:40
*** yufei has quit IRC07:40
*** pcaruana has joined #openstack-ironic07:41
*** yufei has joined #openstack-ironic07:41
*** e0ne has quit IRC07:43
*** e0ne has joined #openstack-ironic07:47
*** SerenaFeng has quit IRC07:52
*** SerenaFeng has joined #openstack-ironic07:53
*** e0ne has quit IRC07:54
*** dmellado has joined #openstack-ironic07:57
*** e0ne has joined #openstack-ironic07:58
*** e0ne has quit IRC08:00
*** rcernin has joined #openstack-ironic08:03
*** e0ne has joined #openstack-ironic08:05
*** aparnav has joined #openstack-ironic08:05
*** e0ne has quit IRC08:10
*** e0ne has joined #openstack-ironic08:13
*** e0ne has quit IRC08:25
*** e0ne has joined #openstack-ironic08:29
*** e0ne has quit IRC08:31
*** ccamacho has joined #openstack-ironic08:40
*** yufei has quit IRC08:42
*** yufei has joined #openstack-ironic08:51
*** david-lyle has joined #openstack-ironic08:57
*** zzzeek has quit IRC09:00
*** david-lyle has quit IRC09:01
*** zzzeek has joined #openstack-ironic09:02
*** zhangjl has quit IRC09:05
*** absubram has joined #openstack-ironic09:05
*** snecklifter_ has quit IRC09:08
*** milan has quit IRC09:09
lucasagomesmorning all09:16
*** jtomasek has quit IRC09:17
*** zhugaoxiao has joined #openstack-ironic09:17
*** sacharya has joined #openstack-ironic09:19
*** zackf has joined #openstack-ironic09:19
*** jrcloud has joined #openstack-ironic09:22
*** zackf has quit IRC09:23
*** sacharya has quit IRC09:23
*** jrcloud has quit IRC09:26
*** links has quit IRC09:29
*** snecklifter has joined #openstack-ironic09:29
*** derekh has joined #openstack-ironic09:30
yuriyzmorning ironic lucasagomes09:31
lucasagomeso/09:32
*** ohamada has joined #openstack-ironic09:34
*** e0ne has joined #openstack-ironic09:57
*** milan has joined #openstack-ironic09:57
*** david-lyle has joined #openstack-ironic09:58
*** milan has quit IRC10:01
*** milan has joined #openstack-ironic10:01
*** david-lyle has quit IRC10:02
*** hoangcx has quit IRC10:05
openstackgerritJoanna Taryma proposed openstack/ironic-specs: Clean invalid conflict response usages and unneccessary retries  https://review.openstack.org/41419410:06
*** athomas has joined #openstack-ironic10:09
*** SerenaFeng has quit IRC10:13
*** SerenaFeng has joined #openstack-ironic10:17
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent  https://review.openstack.org/40742910:25
*** sambetts_ is now known as sambetts10:27
sambettsMorning all10:28
*** snecklifter has quit IRC10:29
lucasagomessambetts, o/10:29
sambettsHey lucasagomes! How was your holiday??10:32
sambettsmuch happen while I was away? ?10:32
openstackgerritJoanna Taryma proposed openstack/ironic: Fixed allowing non-uuids as node UUID for port and portgroup.  https://review.openstack.org/40391010:32
lucasagomesapparently the python35 job is broken due a version of WebOb. Updating it to 1.7.0 fix the issues10:33
lucasagomessambetts, it was great! I think I really needed some time off10:33
lucasagomessambetts, ah, nothing really excited happened. Just a lot of drinking and eating due the xmas and NYE10:34
lucasagomessambetts, what about you ?10:34
sambettslucasagomes: same, first time taking a significant time away from work for a long time, so I both feel refreshed and a little out of the loop :-P10:35
lucasagomesoh yeah, I still catching up with emails and stuff10:36
*** dtantsur|afk is now known as dtantsur10:38
dtantsurmorning ironic, morning and happy new year, sambetts, lucasagomes10:38
sambettshey dtantsur o/ happy new year!10:38
dtantsuroh wonderful, the patch supposed to fix inspector grenade, broke all jobs10:40
lucasagomesdtantsur, happy new year!10:40
lucasagomesdtantsur, :-(10:40
sambetts\o/10:40
lucasagomesthis should fix the python35 job: https://review.openstack.org/#/c/416202/10:40
patchbot_patch 416202 - requirements - Require WebOb>=1.7.010:40
* dtantsur makes a patch10:42
*** livelace has joined #openstack-ironic10:43
*** snecklifter has joined #openstack-ironic10:48
sambettslucasagomes: I'm surprised that keystone middleware commit could land, I guess they aren't testing with upper constraints set?10:51
*** yufei has quit IRC10:51
lucasagomessambetts, that's the only thing I can think of, you can see in the comments from that patch that their were talking about webob 1.7.010:53
lucasagomesbut the upper-constraints in gate is still pinning at 1.6.310:54
lucasagomessambetts, tho: http://logs.openstack.org/02/415502/1/check/gate-keystonemiddleware-python35/b3ca351/console.html.gz#_2016-12-28_15_55_25_45899210:54
*** yufei has joined #openstack-ironic10:56
*** yufei has quit IRC10:57
vdrokgood morning all! lucasagomes yuriyz sambetts and dtantsur !11:01
sambettslucasagomes: hmm weird it does seem to set an upper constraints file but it doesn't use it :/11:02
lucasagomessambetts, yeah we are uncovering a lot of problems there11:04
* lucasagomes checks keystone middleware11:04
lucasagomessambetts, yeah, the install_command is not using upper constraint urgh11:06
* lucasagomes put a patch up for it11:06
sambettsah ... yeah that doesn't help us :-P11:06
*** jtomasek has joined #openstack-ironic11:11
sergekMorning All! vdrok lucasagomes yuriyz sambetts dtantsur11:13
lucasagomeso/11:13
vdrokmorning sergek11:14
sergeko/11:14
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Py3 unittests fail  https://review.openstack.org/41620911:15
*** athomas has quit IRC11:18
dtantsurmorning sergek11:18
*** phuongnh has quit IRC11:19
*** sacharya has joined #openstack-ironic11:20
*** sacharya has quit IRC11:24
*** galyna has joined #openstack-ironic11:25
*** athomas has joined #openstack-ironic11:26
openstackgerritAparna proposed openstack/ironic-specs: In-band hpsum firmware update for iLO drivers  https://review.openstack.org/41016111:26
*** ptx0 has quit IRC11:29
sambettshey sergek11:30
sambettso/ vdrok11:30
*** konetzed has quit IRC11:37
*** konetzed has joined #openstack-ironic11:37
*** jkilpatr has quit IRC11:39
*** gcb has quit IRC11:39
*** gcb has joined #openstack-ironic11:41
*** snecklifter has quit IRC11:45
*** zackf has joined #openstack-ironic11:49
*** zackf has quit IRC11:54
jrollmorning y'all \o/11:56
sambettshey jroll!!11:57
lucasagomesjroll, hi there11:57
lucasagomessambetts, this is what I'm getting when trying to set the constraints to keystonemiddleware: http://paste.openstack.org/show/593778/11:58
*** david-lyle has joined #openstack-ironic11:59
sambettsah thats because keystonemiddleware is in upper-constraints itself, there is a trick we use to handle that for ironicclient I think12:00
dtantsurmorning jroll12:00
sambettslucasagomes: https://github.com/openstack/python-ironicclient/blob/master/tools/tox_install.sh#L5212:01
*** jkilpatr has joined #openstack-ironic12:03
*** david-lyle has quit IRC12:04
vdrokgood morning jroll12:04
*** snecklifter has joined #openstack-ironic12:04
*** tuanluong has quit IRC12:04
*** TheJulia has quit IRC12:19
*** fyxim has quit IRC12:19
*** serverascode has quit IRC12:19
*** rm_work has quit IRC12:19
*** pas-ha has quit IRC12:19
*** izaakk has quit IRC12:19
*** rm_work has joined #openstack-ironic12:20
lucasagomessambetts, https://review.openstack.org/#/c/416198/ should be compatible with webob 1.6.3 and 1.7.012:21
patchbot_patch 416198 - keystonemiddleware - Auth token, set the correct charset12:21
lucasagomesat least on my local tests here12:21
*** izaakk has joined #openstack-ironic12:21
*** fyxim has joined #openstack-ironic12:22
sambettslucasagomes: seems like a safer way to do it if it works than needing a version bump12:22
*** pas-ha has joined #openstack-ironic12:22
lucasagomesyeah, waiting for the tests12:22
*** serverascode has joined #openstack-ironic12:24
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent  https://review.openstack.org/40742912:25
*** TheJulia has joined #openstack-ironic12:25
* lucasagomes go grab some food in the meantime12:25
*** lucasagomes is now known as lucas-hungry12:25
*** SerenaFeng has quit IRC12:25
*** sergek has quit IRC12:30
*** sergek has joined #openstack-ironic12:32
*** trananhkma has quit IRC12:33
*** snecklifter has quit IRC12:39
*** hw_wutianwei has quit IRC12:42
*** hwoarang_ is now known as hwoarang12:43
stevemaris lucas-hungry the same as lucasagomes? :)12:57
*** snecklifter has joined #openstack-ironic12:57
sambettsstevemar: yes, he's AFK for food right now though12:58
stevemarsambetts: i can see that ;)12:58
stevemarsambetts: i'll comment on his patches for now12:58
stevemarlucas-hungry: we have a patch for following upper-constraints support: https://review.openstack.org/#/c/413288/ -- but i think you're saying that https://review.openstack.org/#/c/415502/ only works with webob 1.7.0 ?13:00
patchbot_patch 413288 - keystonemiddleware - Add Constraints support13:00
patchbot_patch 415502 - keystonemiddleware - auth_token: set correct charset when replying with... (MERGED)13:00
openstackgerritDmitry Tantsur proposed openstack/ironic: Deprecated [inspector]enabled option in favor of enabled_inspect_interfaces  https://review.openstack.org/41623213:01
openstackgerritHironori Shiina proposed openstack/ironic: Add REST API for volume connector and volume target operation  https://review.openstack.org/21458613:01
*** jcoufal has joined #openstack-ironic13:04
stevemarlucas-hungry: so, all caught up now, i think we should revert 415502 (since it only works with 1.7.0 AFAICT) and merge the UC support patch13:06
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Create a table and dbapi calls to track loaded interfaces  https://review.openstack.org/40981213:15
*** ccamacho is now known as ccamacho|lunch13:15
dtantsurjroll, ugh, sorry, I didn't notice https://review.openstack.org/#/c/412631/ :( I guess we're getting some duplication now13:16
patchbot_patch 412631 - ironic - WIP: Register/unregister hardware interfaces for c...13:16
dtantsurI thought you planned to update the same patch13:17
*** dprince has joined #openstack-ironic13:18
*** baoli has joined #openstack-ironic13:20
*** trown has joined #openstack-ironic13:20
*** sacharya has joined #openstack-ironic13:20
*** vikrant has quit IRC13:20
*** baoli has quit IRC13:21
*** baoli has joined #openstack-ironic13:22
*** sacharya has quit IRC13:25
*** aparnav has quit IRC13:31
*** lucas-hungry is now known as lucasagomes13:31
lucasagomesstevemar, hi there, hah yes lucas-hungry is my "lunch" handle :-P13:32
stevemar:)13:32
lucasagomesstevemar, yeah exactly, apparenty #413288 only works for 1.7.013:32
lucasagomesstevemar, sorry I mean 41550213:32
lucasagomesstevemar, https://review.openstack.org/#/c/416198/ this works for both versions of WebOb13:33
patchbot_patch 416198 - keystonemiddleware - Auth token, set the correct charset13:33
*** jheroux has joined #openstack-ironic13:34
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Remove trailing backtick  https://review.openstack.org/41611113:38
* lucasagomes just updated the commit message from someone's patch ^13:38
TheJuliaGood morning13:38
sambettsHey TheJulia13:39
lucasagomesTheJulia, morning13:43
dtantsurmorning TheJulia13:44
*** milan has quit IRC13:44
*** yee37913 has joined #openstack-ironic13:46
*** yee379 has quit IRC13:46
*** ildikov has quit IRC13:47
stevemarlucasagomes: hmm, it does eh? i didn't see charset in the signature13:48
* lucasagomes checks13:49
stevemarlucasagomes: i thought https://github.com/Pylons/webob/commit/e810667e57750ed4b0f269fbed15a27cdd8aebf9 removed kwargs support13:50
TheJuliaSo... the hates us again this morning :(13:50
TheJuliaer,r the gate hates13:50
* TheJulia just needs to drink more coffee13:50
lucasagomesTheJulia, yeah it's broken13:50
*** ildikov has joined #openstack-ironic13:50
*** lindycoder has joined #openstack-ironic13:50
TheJulia:(13:53
*** lhinds has quit IRC13:54
*** lhinds has joined #openstack-ironic13:55
*** Marx314 has quit IRC13:56
*** Marx314 has joined #openstack-ironic13:56
vdrokmorning TheJulia13:57
*** snecklifter has quit IRC13:57
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Detailed sequence-diagram for deploy  https://review.openstack.org/41451213:58
*** zhugaoxiao has quit IRC14:00
*** david-lyle has joined #openstack-ironic14:01
*** ccamacho|lunch is now known as ccamacho14:01
*** zhugaoxiao has joined #openstack-ironic14:01
lucasagomesstevemar, yeah I think you are correct, probably webob is just falling back to the default charset (which is utf-8)14:04
*** trown|brb has joined #openstack-ironic14:04
stevemarlucasagomes: well, we can go with yours cause right now we can't adhere to upper-constaints :(14:05
lucasagomesstevemar, the problem is, the upper-contrainst will fail at gate14:05
lucasagomesand the revert as well14:05
*** trown has quit IRC14:05
stevemaryep14:05
lucasagomesstevemar, yeah exactly14:05
lucasagomesprobably merge my patch, then UC, then we can revert14:05
*** sasha21 has joined #openstack-ironic14:05
*** trown|brb is now known as trown14:05
*** david-lyle has quit IRC14:05
stevemaryeah, something nasty like that :(14:06
*** dyasny has joined #openstack-ironic14:07
lucasagomesyeah, that tox.ini from keystone middleware really need some work14:07
lucasagomesUC(almost there), py35, os-testr14:07
jrolldtantsur: oh noooo14:17
dtantsurjroll, I can revert to a version without this bit, or I can merge your patch in and walk from there - depending on what you'd prefer14:18
jrolldtantsur: either way works, I just had realized we would need those things to get to the api14:18
jroll(iirc)14:19
*** jjohnson2 has quit IRC14:19
dtantsurjroll, I think these can go in parallel, modulo it's hard to test API without loading hardware types14:19
jrolldtantsur: dunno, most (all?) of the api changes here don't make sense without hw types loaded: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/driver-composition-reform.html#rest-api-impact14:21
jrollor are weird to implement14:21
*** milan has joined #openstack-ironic14:21
jrollI could add `type` to GET /v1/drivers, but would need another patch later to return dynamic anyway14:22
*** snecklifter has joined #openstack-ironic14:23
dtantsuryeah, agreed14:27
dtantsurjroll, so I guess it boils down to whether you have time for this patch this week, or should I overtake it14:27
jrolldtantsur: yeah, I should be fine, I'll start with a rebase on patch 40981214:28
patchbot_https://review.openstack.org/#/c/409812/ - ironic - [WIP] Create a table and dbapi calls to track load...14:28
dtantsurjroll, so, do you want me to drop the dbapi bit from my patch? it's not unit testable without later changes in your patch14:30
jrolldtantsur: oh, that works too, I quite like that method so I'll pull it into mine14:30
dtantsurjroll, ack, will update the patch in a few minutes14:31
openstackgerritDmitry Tantsur proposed openstack/ironic: Create a table to track loaded interfaces  https://review.openstack.org/40981214:32
dtantsurjroll, back to table-only version ^^^14:32
jrolldtantsur: only question, should the fields in that new table be NOT NULL?14:35
dtantsurjroll, probably? we don't usually use NOT NULL, so I left it out as well14:35
*** ppiela has joined #openstack-ironic14:36
jrolldtantsur: yeah, I think that's typically due to API/wsme crap for some reason, iirc14:36
dtantsurwsme crap \o/14:36
jrollleft a comment, I think those should be null and then I'm +214:37
dtantsurlemme change it14:37
jroller, not null :P14:38
dtantsurTOO LATE14:38
dtantsur:-P14:38
jroll:D14:39
openstackgerritDmitry Tantsur proposed openstack/ironic: Create a table to track loaded interfaces  https://review.openstack.org/40981214:39
dtantsurjroll, done ^^^14:39
jrollthanks14:39
jroll+214:40
lucasagomesstevemar, btw, when that patch merge we probably will need a release of keystonemiddleware to unblock the ironic gate14:40
*** absubram has quit IRC14:43
jrolldtantsur: what did we need this bit for? https://review.openstack.org/#/c/409812/7/ironic/db/sqlalchemy/api.py14:47
patchbot_patch 409812 - ironic - Create a table to track loaded interfaces14:47
openstackgerritOksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV  https://review.openstack.org/39712614:49
dtantsurjroll, for issuing a warning on start up14:50
*** zackf has joined #openstack-ironic14:51
openstackgerritSzymon Borkowski proposed openstack/ironic: [PoC] Add fake fields in Node object and RPC  https://review.openstack.org/41546214:53
jrolldtantsur: okay, so we'll need to add a count(number of conductors) or something14:54
jrollsorry, thought I was missing something, was a silly question14:54
*** chlong has joined #openstack-ironic14:55
*** zackf has quit IRC14:55
dtantsurjroll, dunno, there are quite a few ways of implementing it.. actually it's not the highest priority, maybe we need to solve only loading the hardware types first14:56
jrolldtantsur: yeah, I may leave it as a todo for a moment14:58
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: [WIP] ETAGs for node in ironicclient  https://review.openstack.org/40033514:58
dtantsurjroll, sure.. then we don't even need to populate this table14:58
dtantsur... except for populating "conductors" table with new driver names (from hardware types), I guess14:59
*** moshele has quit IRC15:00
*** moshele has joined #openstack-ironic15:01
sambettsjroll: I caught up on the TC meeting that you mentioned me in, thanks for supporting me :)15:01
*** ElCoyote_ has joined #openstack-ironic15:01
*** david-lyle has joined #openstack-ironic15:01
jrollsambetts: :)15:01
stevemarlucasagomes: roger that15:02
lucasagomesstevemar, thanks a lot!15:02
*** snecklifter has quit IRC15:03
openstackgerritSam Betts proposed openstack/ironic: Fix invalid chassis/nodes/chassis_uuid API url  https://review.openstack.org/31614915:05
openstackgerritSam Betts proposed openstack/ironic: Fix all possible node subresources routing collisions  https://review.openstack.org/31576615:05
*** david-lyle has quit IRC15:06
openstackgerritAndrey Shestakov proposed openstack/bifrost: Fix condition for include ssh_public_key_path  https://review.openstack.org/41626515:13
*** srobert has joined #openstack-ironic15:19
*** sacharya has joined #openstack-ironic15:21
jlvillalGood morning Ironic!  I hope everyone had a great holiday :)15:25
lucasagomesjlvillal, morning!15:25
jlvillal:)15:26
*** sacharya has quit IRC15:26
jlvillalI learned about using letsencrypt with dns-01 verification. So I could setup SSL certificates for my systems internal to my home network.15:26
jlvillalAnd how to use Cloudflare as my DNS provider and make REST API calls to it.15:27
JayFNice15:27
jlvillalPlus setting up some extra Ubiquiti Unifi access points :)15:27
JayFmyself, mariojv, jcook, and some other rackers used our Rackspace provided voluinteer time to work on certbot, the EFF client to letsencrypt15:27
JayFit's a solid idea and the folks working on it are very nice15:27
*** srobert has quit IRC15:27
jlvillalJayF: Oh sweet. It would be awesome if it supported dns-01. Last I checked it supported http-01.15:28
jlvillalJayF: So I used this thing called 'dehydrated'15:28
JayFthey accept pull requests and give good feedback and have quick review times :P15:28
*** zackf has joined #openstack-ironic15:28
jrolljlvillal: https://github.com/certbot/certbot/issues/223615:28
jrolltl;dr certbot certonly --manual --preferred-challenges dns15:28
jlvillalJayF: :)15:28
jrollstill need to run the dns-01 hook yourself15:29
mariojvyep, the acme protocol is interesting as well. there are quite a few other clients out there to help with that, not sure if they also support auto-configuring some web servers like certbot15:29
mariojv"that" meaning acme protocol15:29
*** srobert has joined #openstack-ironic15:29
jlvillaljroll: Yeah. The 'dehyrdated' client does it. Seems to work fine. I also found one called 'acme.sh' which also supports it.15:30
jrolljlvillal: or better yet, https://github.com/certbot/certbot/pull/206115:30
jroll:P15:30
jrollyeah :)15:31
jlvillaljroll: I think major wrote a blog about dehyrdated and using it with Rackspace DNS15:31
jrollof course he did15:31
jlvillalheh15:31
jrollI have yet to find something major has not blogged about15:31
* jlvillal thinks he should try to write a blog post some day.15:31
jrolljlvillal: write one about using letsencrypt with dns-01 verification15:31
jroll:)15:31
jlvillalheh :)15:32
jrollbaby steps15:32
jlvillalFirst step, write one about setting up a blog. Since I still need to set that up.15:32
jrollha15:32
jlvillalI had been using StartSSL which gave one year certificates and worked fine. Until they got banned by Firefox and Chrome :(15:32
jrollwatch out, people get paralyzed because they overthink the setup itself (because that's the fun part, right?)15:33
jrollouch15:33
*** zackf has quit IRC15:33
jlvillalI think I'm in the "paralyzed" stage.15:33
*** Nisha_Agarwal has joined #openstack-ironic15:33
JayFI think I'm going to go the route of a youtube vlog15:33
JayFinstead of a blog15:33
JayFbecause I'm lazy, and recording video is easy15:33
JayFlol15:33
jlvillalJayF: Oh, now I was thinking about that! :)15:33
TheJuliaI can't do a vlog, but I'm trying to blog more15:34
jlvillalJayF: I had been researching audio recording gear and lighting.15:34
TheJuliajlvillal: lighting gear is relatively inexpensive15:34
*** fritzstauffacher has joined #openstack-ironic15:34
* TheJulia feels kind of sad she gave away her lighting gear15:35
fritzstauffacherI can’t seem to find any documentation on enabling uefi in ironic-inspector. Can anyone point me in the right direction?15:35
jlvillalTheJulia: Yeah. I was thinking about using a couple of the four foot LED lights I picked up.15:36
*** absubram has joined #openstack-ironic15:37
TheJuliajlvillal: just make sure your using a decent camera that doesn't freak with an odd temperature of light.  I think the lights in this room are 3000k white LEDs, they seem to work okay, but I think my lighting kit was 5500k CFLs and it was a bit too hot if that makes sense15:37
*** rbudden has joined #openstack-ironic15:39
*** chlong has quit IRC15:41
fritzstauffacherhmm looks like ipxe might support. Will give it a shot.15:42
TheJuliafritzstauffacher: I suspect that may be your only option15:43
*** bfournie has joined #openstack-ironic15:44
*** Goneri has joined #openstack-ironic15:44
JayFTheJulia: is that why my green shirts always look blue in video meetings? lol15:44
fritzstauffacherThanks. Looks like it.15:44
sambettsTheJulia, JayF: main thing for vlogs is a decent MIC, people can often deal with non-so-good video if the audio is good15:44
TheJuliaJayF: Pretty much :)15:44
JayFTheJulia: it's pretty annoying. it's a nice shade of green and nobody but vanessa gets to see it. lol15:45
jlvillalTheJulia: Thanks. Yeah, I would probably try using my DSLR. Which I have yet to use for video. See what happens.15:45
sambettsDSLRs are normally pretty good for video, although you have to watch out for short record times and overheating15:46
TheJuliajlvillal: what sambetts said, also if you have a canon DSLR, you might want to checkout ?magiclantern? which is geared more-so for video than the stock firmware15:46
TheJuliaThe only video I've recorded with my DSLR was with a shotgun mic to get audio of waves.  Was kind of fun, although I never did anything with it besides splitting the audio/video apart.15:47
jlvillalTheJulia: I use Nikon but I think it should work fine too.15:49
openstackgerritJim Rollenhagen proposed openstack/ironic: Register/unregister hardware interfaces for conductors  https://review.openstack.org/41263115:50
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Detailed sequence-diagram for deploy  https://review.openstack.org/41451215:50
*** Syed__ has joined #openstack-ironic15:52
*** chlong has joined #openstack-ironic15:54
*** strigazi_ is now known as strigazi_AFK15:54
*** athomas has quit IRC15:59
*** zackf has joined #openstack-ironic16:00
*** moshele has quit IRC16:01
*** d0ugal has joined #openstack-ironic16:03
*** zhugaoxiao has quit IRC16:03
*** zackf has quit IRC16:04
*** zhugaoxiao has joined #openstack-ironic16:04
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM Testing tempest "smoke" 2  https://review.openstack.org/40725816:08
*** Goneri has quit IRC16:08
*** mjura has quit IRC16:08
*** zackf has joined #openstack-ironic16:10
*** Goneri has joined #openstack-ironic16:11
*** alex_xu has quit IRC16:11
*** rama_y has joined #openstack-ironic16:12
*** rama_y_ has joined #openstack-ironic16:12
*** rama_y_ has quit IRC16:12
*** awiddersheim has quit IRC16:13
openstackgerritOksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV  https://review.openstack.org/39712616:15
*** athomas has joined #openstack-ironic16:16
*** awiddersheim has joined #openstack-ironic16:17
mariojvsomeone with write access to the Ocata Priorities trello board may want to move notifications to "Done" column16:20
mariojvyuriy finished the ironic side and says the searchlight plugin works now16:20
mariojvnot sure if searchlight plugin is merged, but that's not something we need to track on the ironic side i think16:20
mariojvunless we wait until the release to move things in trello?16:21
*** alex_xu has joined #openstack-ironic16:22
* jroll grabs it16:23
jrollthanks16:23
jrollI probably need to do a run-through of trello at some point16:24
mariojvty16:24
mariojvJayF: one of our rescue dependencies (fix to flat tenant network config) merged \o/ https://review.openstack.org/#/c/327046/37/ironic/drivers/modules/network/flat.py16:25
patchbot_patch 327046 - ironic - Add Virtual Network Interface Driver APIs (MERGED)16:25
JayFnice16:25
*** rcernin has quit IRC16:25
JayFI'm probably going to focus on reviews/reading code today16:25
JayFto get my brain warmed back up from the holidays16:25
mariojvsame16:25
JayFbut if you wanted to work on rescue tomorrow I'm game16:25
mariojvsounds good. i'm gone in the morning for an appointment but should be back around 1-2pm eastern16:25
mariojvmay or may not stay late to make up the hours, not sure yet16:26
mariojv(vs using pto)16:26
*** ccamacho has quit IRC16:26
*** jjohnson2 has joined #openstack-ironic16:27
*** dmellado_ has joined #openstack-ironic16:28
*** baoli has quit IRC16:31
*** tesseract has quit IRC16:31
*** chlong has quit IRC16:31
openstackgerritVladyslav Drok proposed openstack/ironic-specs: Tenant-aware baremetal nodes  https://review.openstack.org/41551216:32
jlvillalvdrok: I thought you were on holiday? ;)16:35
*** ptx0 has joined #openstack-ironic16:35
vdrokjlvillal: heh, no, saving some pto days for later :)16:35
vdrokmorning jlvillal JayF and mariojv16:35
jlvillalvdrok: Ah, good plan :D16:36
vdrokjlvillal: the office is almost empty, so it's like having another pto anyway :D16:36
jlvillalheh16:36
JayFvdrok: I replied to your email, found the bug I was trying to find the other day when we were chatting in irc16:38
vdrokJayF: yes, saw that, thank you. I'm digging into nova now to see at which point exactly that happens, maybe it can be fixed in nova itself16:39
*** jvgrant has quit IRC16:41
mariojvhi vdrok16:43
*** e0ne has quit IRC16:45
*** rama_y has quit IRC16:46
*** chlong has joined #openstack-ironic16:47
*** jvgrant has joined #openstack-ironic16:51
* jroll runs to the store and also eats lunch, bbl16:51
vdrokJayF: the thing I can not get yet is, if there would be an elif here https://github.com/openstack/nova/blob/1506c36b4446f6ba1487a2d68e4b23cb3fca44cb/nova/virt/ironic/driver.py#L321, we'll be reporting used==available for nodes not in usable state. how could we schedule something on a node that has instance_uuid on it?16:52
*** dmellado has quit IRC16:53
*** jvgrant has quit IRC16:54
*** Goneri has quit IRC16:55
*** jvgrant has joined #openstack-ironic16:55
openstackgerritPeter Piela proposed openstack/ironic-ui: Changes required for updated Bootstrap version  https://review.openstack.org/41600016:56
JayFevdI don't understand your question?16:56
JayFvdrok: ^16:56
vdrokJayF: what is proposed here https://review.openstack.org/#/c/391415/9/nova/virt/ironic/driver.py is to bring back elif on that line. the bug that you're referencing says: "Once the resource tracker calls _update_usage_from_instance, it will not find an instance associated with the node from nova's point of view and will return all of the memory as available16:59
vdrokinstead", but that won't happen, as used==available will be reported in such case16:59
patchbot_Error: No closing quotation16:59
patchbot_patch 391415 - nova - Fix hypervisor-stats show invalid info of baremeta...16:59
openstackgerritSam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation  https://review.openstack.org/39295916:59
vdrokJayF: this check here https://github.com/openstack/nova/blob/1506c36b4446f6ba1487a2d68e4b23cb3fca44cb/nova/virt/ironic/driver.py#L305 just verifies that node.instance_uuid is set, if it is, the all node's resources are used17:00
JayFvdrok: so basically, 2/2 is OK as long as nova knows about the instance on the node, 0/2 is OK as long as the node is provisionable in ironic, I think the case you're talking about would show up as 2/0 which is also OK17:01
JayFjcook: ^ this is relevant to that bug you fixed, btw17:01
vdrokJayF: yeah, tho why not report 2/2 in case of 2/0?17:02
vdrokJayF: if nova does not know about the instance on the node, are some failures happening?17:03
*** david-lyle has joined #openstack-ironic17:03
jcookI'm not sure what the 0/2 2/2 2/0 is being referred to here17:03
vdrokjcook: I think it's 2 used out of 2 available == 2/217:03
vdrokfor any of our resources, cpus, memory_mb etc17:04
JayFjcook: used/available resources in nova17:04
JayFvdrok: If nova does not know about the instance on the node, it'll reset 2/2 to 0/217:04
JayFvdrok: and in the maint'd, but instance, case, when nova deletes the node, it'll reset that 2/2 to 0/2 -- but if it's 2/0, it goes to 0/0, which is OK17:04
JayFvdrok: so basically there's just a short race in that case, does that makes sense now why we remove resources from nodes in maintenance (unavailable) but with instance?17:05
vdrokJayF: hmm, ok, yeah17:06
JayFthis is all stuff only seen at high build rates with lots of nodes17:07
JayFwe found most of these out the hard way :)17:07
*** david-lyle has quit IRC17:08
vdrokJayF: the problem is, with the switch to resource providers, this may change. want to make sure that we don't keep the not so obvious code if it's not needed :)17:08
JayFyeah, I agree17:08
JayFI don't mind it changing at all17:08
JayFbut the edges are not obvious17:08
*** chas has quit IRC17:09
*** chas has joined #openstack-ironic17:09
sambettsJayF, jcook: I've also seen this cause an issue with populating nova with ironic resources https://review.openstack.org/#/c/308348/17:10
patchbot_patch 308348 - nova - Eliminate resource tracker to host manager races (ABANDONED)17:10
sambettsJayF, jcook: I had this patch out for a while and have spoken to Sylvain about it but never resolved the issue17:11
*** chas has quit IRC17:14
*** trown is now known as trown|lunch17:15
jcookJayF was _node_resources_used changed in another patch?17:20
*** vinsh has joined #openstack-ironic17:20
JayFjcook: unsure?17:22
jcookI'm looking at this patch and I'm not sure what we're trying to accomplish with the change17:22
*** dtantsur is now known as dtantsur|afk17:23
dtantsur|afko/17:23
*** jaosorior has joined #openstack-ironic17:26
*** moshele has joined #openstack-ironic17:30
*** baoli has joined #openstack-ironic17:31
*** sambetts is now known as sambetts|afk17:38
sambetts|afko/17:38
*** srobert has quit IRC17:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM Testing tempest "smoke" 2  https://review.openstack.org/40725817:40
*** lucasagomes is now known as lucas-afk17:41
lucas-afkhave a great evening all!17:42
JayFo/17:42
*** jaosorior has quit IRC17:43
*** derekh has quit IRC17:47
jlvillaljroll: JayF: Since after the new year. The proposed project-config change for the grenade job. https://review.openstack.org/#/c/411956/17:49
patchbot_patch 411956 - openstack-infra/project-config - ironic: Have the grenade builder use devstack-virt...17:49
*** dsneddon_afk is now known as dsneddon17:52
JayFjlvillal: until the gate is cleaned up, I don't think we should land anything like that :/17:54
jlvillalJayF: Oh sorry, I guess I haven't been staying caught up. I thought the gate was in good shape right now.17:54
jlvillalJayF: With the work-around. While waiting for the Nova patch.17:54
jlvillalJayF: But no hurry for sure.17:55
JayFThe whiteboard says it's not, and I know lucas didn't land something earlier with a comment that the gate was unhappy17:55
JayFI saw several failures the other day when I worked a half day17:55
mariojvTheJulia: cinder storage interface will be enabled by default? re. https://review.openstack.org/#/c/366197/12/ironic/conf/default.py17:55
patchbot_patch 366197 - ironic - Add Cinder storage driver17:55
mariojvi suppose that's OK, given that it's not the default storage interface used when enrolling new nodes17:56
jlvillalJayF: My test patch did fine today. I know vsaienk0 landed a work-around a few days ago. https://review.openstack.org/#/c/415403/  28-Dec-201617:56
patchbot_patch 415403 - ironic - Disable placement-api by default (MERGED)17:56
jlvillalJayF: But sure, we can wait. More of a cleanup action.17:57
*** pcaruana has quit IRC17:58
*** Nisha_Agarwal has quit IRC18:01
*** david-lyle has joined #openstack-ironic18:03
*** ohamada has quit IRC18:05
*** david-lyle has quit IRC18:08
*** chlong has quit IRC18:10
*** athomas has quit IRC18:11
vdrokgood night everyone!18:22
TheJuliamariojv: That was what I was thinking would work since noop is the default and is routed to pass18:30
mariojvsounds fine. I'm wondering what service startup behavior will be if cinder is an enabled interface and the cinder config options aren't set, though18:31
jcookJayF I'm trying to re-understand what information we do and do not have in the ironic driver18:32
jcookwe have a node object with an instance uuid18:32
mariojvruntime error when trying to use cinder storage with a node, maybe?18:32
jcookdoes nova set that or ironic?18:32
mariojvjcook: nova18:32
TheJuliamariojv: driver task creation will fail, just like with network interfaces18:32
mariojvjcook: i'm pretty sure18:32
mariojvTheJulia: alright, thanks18:32
mariojvmodeling the behavior off of network interfaces seems like a good match18:33
mariojvplease disregard any comments about service startup in my review, then18:33
JayFjcook: nova sets the instance uuid, and the ironic db also holds it18:33
TheJuliamariojv: 10-418:33
jcookk, another related question, does the ironic driver not know what the resource tracker thinks about the instances?18:33
TheJuliamariojv: thank you :)18:33
JayFjcook: I don't believe so18:33
mariojvi thought there was something like "vcpus" in instance_info that nova uses JayF jcook18:34
mariojvi haven't examined that clearly, though18:34
openstackgerritBob Fournier proposed openstack/ironic-inspector: Add plugin to process basic management LLDP TLVs  https://review.openstack.org/40649618:34
JayFmariojv: yeah, but that's just the amount of resources we have that's read into the resource tracker18:34
JayFmariojv: we tell nova that, not the other way around18:34
JayFmariojv: and the concept of that goes away with the new resource tracker18:34
mariojvohhh, i see. i saw "ironic" instead of "ironic driver" in jcook's question about what the resource tracker thinks18:35
*** milan has quit IRC18:39
*** trown|lunch is now known as trown18:39
cvstealt1hi sorry if this should be posted to the nova channel but we are trying to add ironic to a current OS deployment that uses nova/libvirt.  The question that I have is for the nova scheduler do we run 1 nova scheduler for ironic and another one for libvirt requests or does the compute driver nova.virt.ironic.IronicDriver handle both baremetal and VM instances?18:41
*** chlong has joined #openstack-ironic18:41
*** david-lyle has joined #openstack-ironic18:42
*** moshele has quit IRC18:42
*** milan has joined #openstack-ironic18:44
*** rama_y has joined #openstack-ironic18:44
jcookcvstealt1 I kind of know the answer to that, but I'll defer to someone that can answer you better like JayF or jroll :)18:45
jcookJayF I updated this https://review.openstack.org/#/c/391415/9 . Hope it's clear it's been a long holiday and having a hard time focusing18:45
patchbot_patch 391415 - nova - Fix hypervisor-stats show invalid info of baremeta...18:45
cvstealt1jcook: thx, couldn't easily find the answer with docs/googling18:46
JayFcvstealt1: I know what I've been told is you have to setup a separate nova region or use host aggregates18:46
JayFcvstealt1: or use cells18:47
JayFcvstealt1: we used cells in our production deployment, so I don't have much help for you ://18:47
JayFcvstealt1: good question for the mailing list, and I'll add it to my list of docs improvements we need18:47
jcookand cells v1 is not recommended and cells v2 is not quite there for this18:47
jcookcvstealt1 at rackspace we had some downstream patches around resource classes and such that helped. I think, sadly, right now two regions is the best bet. This is something we'd like to fix soon^TM18:48
cvstealt1JayF: that's what I was fearing as I got more into it where it needed it's own set of nova/neutron api endpoints18:48
*** chlong has quit IRC18:48
JayFif cloudnull was here, I know he did some digging on host aggregates18:49
JayFbut he's not so :/18:49
cvstealt1jcook: bad assumption to say that keystone/glance can be used across those separate regions?18:50
JayFno, that's a perfectly fine assumption18:51
JayFand I think same neutron could be, too, but imbw18:51
jcookyeah, that should work I think, but I have no ops experience with that18:52
jcooknot sure why it wouldn't work though off the top of my head18:52
JayFcvstealt1: for bonus points, when/if you figure it out, make some notes and I can use it to improve docs18:52
JayFcvstealt1: also I'd ask on the list, lots of people using ironic that don't irc18:52
*** moshele has joined #openstack-ironic18:55
*** milan has quit IRC18:55
*** chlong has joined #openstack-ironic19:02
*** milan has joined #openstack-ironic19:04
jrollinspector folks, https://review.openstack.org/#/c/416205/ merged which (should) unblock CI19:06
patchbot_patch 416205 - openstack-infra/project-config - Fix a typo blocking ironic-inspector dsvm jobs (MERGED)19:06
cvstealt1JayF: thx I'll keep digging through it and let you know, appreciate the feedback and will probably just go with the additional region19:06
JayFcvstealt1: cool, good luck!19:06
jlvillalFYI: I'm asking over in #openstack-keystone to push out a new release of keystonemiddleware to fix the Ironic gate.19:09
*** milan has quit IRC19:09
jlvillalThis patch fixed the issue: https://review.openstack.org/#/c/416198/   But needs to be released.19:09
patchbot_patch 416198 - keystonemiddleware - Auth token, set the correct charset (MERGED)19:09
JayFjlvillal: is that bug on our whiteboard?19:09
jlvillalJayF: Yeah19:10
JayFman, it's a mess19:10
JayFsom uch broken :(/19:10
mariojvJayF: are security groups something we want support for in rescue? seems like cleaning and provisioning networks support it, so i don't see why we shouldn't19:10
*** chas has joined #openstack-ironic19:10
JayFmariojv: sure19:10
jlvillalJayF: Line 4219:10
mariojvack, thx19:10
jlvillalJayF: Yep :(19:10
JayFjlvillal: yeah, I see it, just messy because so much stuff is busted19:10
jlvillalagreed19:10
*** chas has quit IRC19:15
*** milan has joined #openstack-ironic19:16
openstackgerritJesse J. Cook proposed openstack/ironic-specs: Update rescue spec  https://review.openstack.org/40723619:18
jcookJayF mariojv put a couple of questions on that after I updated it ^19:21
JayFjcook: replied19:22
mariojvwill take a look in a bit19:23
mariojvdo we need to update the spec in light of driver comp reform?19:23
stevemarjlvillal: lucas-afk fyi  https://review.openstack.org/41632719:23
patchbot_patch 416327 - releases - release keystonemiddleware 4.13.119:23
jlvillalstevemar: Thank you :)19:23
jlvillalstevemar: Two releases in one day :D19:23
stevemarjlvillal: we're that good19:24
jlvillalheh19:24
JayFmariojv: ooooh.19:24
jrollI wanna do two releases a day.19:27
JayFyou have to catch a couple of bugs first, It hink19:27
openstackgerritJesse J. Cook proposed openstack/ironic-specs: Update rescue spec  https://review.openstack.org/40723619:27
jcookJayF thanks and updated19:27
jcookmariojv ^19:27
jrollJayF: or land features :)19:30
*** moshele has quit IRC19:30
*** moshele has joined #openstack-ironic19:31
*** ChubYann has joined #openstack-ironic19:32
*** fungi has quit IRC19:35
*** TheJulia has quit IRC19:35
*** serverascode has quit IRC19:35
*** rm_work has quit IRC19:35
*** vdrok has quit IRC19:35
*** igordcard has quit IRC19:35
*** zer0c00l has quit IRC19:35
*** mariojv has quit IRC19:35
*** aaa801 has quit IRC19:35
*** gmmaha has quit IRC19:35
*** keekz has quit IRC19:35
*** dtantsur|afk has quit IRC19:35
*** lennyb has quit IRC19:35
*** zer0c00l has joined #openstack-ironic19:35
*** keekz has joined #openstack-ironic19:35
*** dtantsur has joined #openstack-ironic19:35
*** igordcard has joined #openstack-ironic19:35
*** lennyb has joined #openstack-ironic19:36
*** mariojv has joined #openstack-ironic19:36
*** gmmaha has joined #openstack-ironic19:36
*** gmmaha has quit IRC19:36
*** gmmaha has joined #openstack-ironic19:36
*** rm_work has joined #openstack-ironic19:36
*** fungi has joined #openstack-ironic19:36
*** harlowja has joined #openstack-ironic19:38
*** gmmaha has quit IRC19:38
*** vdrok has joined #openstack-ironic19:39
*** milan has quit IRC19:40
*** aaa801 has joined #openstack-ironic19:40
*** gmmaha has joined #openstack-ironic19:40
*** milan has joined #openstack-ironic19:41
*** jkilpatr has quit IRC19:42
*** TheJulia has joined #openstack-ironic19:43
*** Goneri has joined #openstack-ironic19:44
*** rbudden has quit IRC19:46
*** yee37913 has quit IRC19:48
*** yee379 has joined #openstack-ironic19:49
*** serverascode has joined #openstack-ironic19:51
*** jkilpatr has joined #openstack-ironic19:55
stevemarjlvillal: patch has merged, gotta wait for the post processing to happen, but the new lib should be out shortly19:58
jrollstevemar: nice, thank you!19:58
stevemarjroll: least i can do after breaking you guys ;)19:58
jrollheh19:58
jrollstevemar: you're supposed to tell us we're doing it wrong19:59
stevemarjroll: oh right, umm... stop using the latest libraries, test only with old stuff :P19:59
jrollgot it19:59
* jroll installs rhel 419:59
jlvillalstevemar: Thanks20:00
*** openstackgerrit has quit IRC20:03
*** milan has quit IRC20:09
*** e0ne has joined #openstack-ironic20:09
*** milan has joined #openstack-ironic20:11
*** moshele has quit IRC20:22
*** moshele has joined #openstack-ironic20:23
mariojvdoes anyone remember on which patch it was where rloo pointed out we don't have a metrics deprecation policy?20:29
mariojvit was the week before christmas, can look in irc logs if no one knows20:29
*** moshele has quit IRC20:29
mariojvfound it - luckily irc was quiet for those few days https://review.openstack.org/#/c/41233920:32
patchbot_patch 412339 - ironic - Decorator METRICS.timer parameter is inconsistent ...20:32
mariojvabout that particular issue regarding deprecating metric names; it looks like swift explicitly doesn't guarantee anything about metric names or semantics20:38
mariojvhttp://docs.openstack.org/developer/swift/admin_guide.html#reporting-metrics-to-statsd20:38
*** milan has quit IRC20:39
mariojvdoesn't mean we shouldn't, but it's a good data point20:40
*** milan has joined #openstack-ironic20:41
*** absubram has quit IRC20:56
*** athomas has joined #openstack-ironic20:59
*** milan has quit IRC21:09
*** chas has joined #openstack-ironic21:11
*** milan has joined #openstack-ironic21:12
*** jheroux has quit IRC21:14
*** jheroux has joined #openstack-ironic21:15
*** chas has quit IRC21:15
mrdaMorning Ironic21:17
* jlvillal is thinking that upper-constraints is not bringing in latest version of keystonemiddleware :(21:19
jrolljlvillal: https://review.openstack.org/#/c/416338/21:19
patchbot_patch 416338 - requirements - update constraint for keystonemiddleware to new re...21:19
jlvillaljroll: Yeah, I just did a recheck on it21:20
jlvillalThat's cool that it automatically updates it though :)21:20
*** jjohnson2 has quit IRC21:24
mariojvmorning mrda21:25
mrdaMorning mariojv21:26
*** openstackgerrit has joined #openstack-ironic21:26
openstackgerritBob Fournier proposed openstack/ironic-inspector: Add plugin to process basic management LLDP TLVs  https://review.openstack.org/40649621:26
mrdajlvillal: you can probably summon tonyb if you need to21:26
jlvillalmrda: Well first need to have the job pass tests :(21:27
jlvillalmrda: Happy New Year!21:27
mrdajlvillal: ahh, trivial details :) And HNYTY!21:27
tonybYeah looks like a setup race to me .... http://logs.openstack.org/38/416338/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/16a1344/logs/grenade.sh.txt.gz#_2017-01-03_20_30_29_65721:29
*** jkilpatr has quit IRC21:29
*** e0ne has quit IRC21:29
*** Goneri has quit IRC21:29
*** jcoufal has quit IRC21:30
*** rbudden has joined #openstack-ironic21:30
*** athomas has quit IRC21:30
*** dsneddon is now known as dsneddon_afk21:33
*** dyasny has quit IRC21:36
*** milan has quit IRC21:39
*** ijw has joined #openstack-ironic21:47
*** dyasny has joined #openstack-ironic21:50
*** chas has joined #openstack-ironic21:51
*** milan has joined #openstack-ironic21:51
*** milan has quit IRC21:55
*** chas has quit IRC21:56
*** dprince has quit IRC21:59
*** Goneri has joined #openstack-ironic22:03
*** jkilpatr has joined #openstack-ironic22:05
*** Sukhdev has joined #openstack-ironic22:06
*** dyasny has quit IRC22:08
*** chlong has quit IRC22:10
*** ijw has quit IRC22:11
*** milan has joined #openstack-ironic22:11
*** chas has joined #openstack-ironic22:12
*** ijw has joined #openstack-ironic22:13
jlvillaltonyb: Passed in the recheck, as I'm watching zuul. But other jobs are still running.22:14
tonybjlvillal: cool.22:15
tonybjlvillal: I'll keep an eye on that review22:15
*** chas has quit IRC22:16
jlvillaltonyb: Thanks. Yeah that is blocking the Ironic gate at the moment. Well that is the belief :)22:16
tonybjlvillal: can you run up a DNM chnage in ironic to verify that?22:16
jlvillaltonyb: I just tested it locally and it works.22:20
jlvillaltonyb: It breaks the py35 unit test job, so easy for me to test.22:21
tonybjlvillal: okay.22:23
*** Sukhdev has quit IRC22:24
*** milan has quit IRC22:25
*** milan has joined #openstack-ironic22:31
*** baoli has quit IRC22:33
*** jralph_ has quit IRC22:37
*** milan has quit IRC22:39
*** jralph has joined #openstack-ironic22:41
*** jheroux has quit IRC22:42
*** lindycoder has quit IRC22:43
*** Nisha_Agarwal has joined #openstack-ironic22:45
*** rbudden has quit IRC22:45
*** pmannidi has joined #openstack-ironic22:48
*** milan has joined #openstack-ironic22:51
*** milan has quit IRC22:55
*** ppiela has quit IRC22:59
*** fritzstauffacher has quit IRC23:04
*** milan has joined #openstack-ironic23:12
*** openstack has joined #openstack-ironic23:15
*** milan has quit IRC23:25
openstackgerritNisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers  https://review.openstack.org/37242823:25
*** vinsh has quit IRC23:28
*** milan has joined #openstack-ironic23:31
*** Fdaisuke has joined #openstack-ironic23:38
*** baoli has joined #openstack-ironic23:39
*** milan has quit IRC23:44
*** Nisha_Agarwal has quit IRC23:45
*** milan has joined #openstack-ironic23:52
*** milan has quit IRC23:55
jlvillalFYI: The gate should be working again. The py35 job is passing locally for me now.23:55

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