Monday, 2016-06-27

*** PollyZ has quit IRC00:11
*** piet has quit IRC00:17
*** rook has joined #openstack-ironic00:26
*** piet has joined #openstack-ironic00:33
*** PollyZ has joined #openstack-ironic00:35
*** klindgren_ has quit IRC00:44
*** hoangcx has joined #openstack-ironic00:54
*** yibo has joined #openstack-ironic00:54
*** piet has quit IRC00:58
*** caoshufeng has joined #openstack-ironic01:01
caoshufengHi, all, I have a question. Why ironic-api doesn't ask for a token when user access the vendor_passthru api?01:10
*** phuongnh has joined #openstack-ironic01:29
*** amotoki has joined #openstack-ironic01:34
*** baoli has joined #openstack-ironic01:36
*** baoli has quit IRC02:14
*** baoli has joined #openstack-ironic02:22
*** sdake_ has quit IRC02:23
*** yibo has quit IRC02:33
*** baoli has quit IRC02:36
*** yibo has joined #openstack-ironic02:39
*** amit213 has joined #openstack-ironic02:39
*** baoli has joined #openstack-ironic02:46
*** yuanying has quit IRC02:49
*** piet has joined #openstack-ironic03:03
*** baoli has quit IRC03:03
*** Sukhdev has joined #openstack-ironic03:10
openstackgerritTang Chen proposed openstack/python-ironicclient: Use osc-lib instead of openstackclient  https://review.openstack.org/33331103:10
*** pleia2_ is now known as pleia203:15
*** _milan_ has joined #openstack-ironic03:19
*** milan has quit IRC03:21
*** robcresswell has quit IRC03:24
*** lynxman has quit IRC03:26
*** robcresswell has joined #openstack-ironic03:27
*** lynxman has joined #openstack-ironic03:34
*** MattMan has quit IRC03:40
*** MattMan has joined #openstack-ironic03:40
*** sacharya has joined #openstack-ironic03:41
*** yuanying has joined #openstack-ironic03:47
*** links has joined #openstack-ironic04:15
openstackgerritTang Chen proposed openstack/python-ironic-inspector-client: Use osc-lib instead of openstackclient  https://review.openstack.org/33371504:19
openstackgerritTang Chen proposed openstack/python-ironicclient: Use osc-lib instead of openstackclient  https://review.openstack.org/33331104:26
*** amotoki has quit IRC04:27
openstackgerritDao Cong Tien proposed openstack/ironic: [WIP] Nova-compatible serial console: socat console_utils  https://review.openstack.org/32816804:36
*** amotoki has joined #openstack-ironic04:36
*** amotoki has quit IRC04:44
*** amit213 has quit IRC04:46
Sukhdevjroll: Hi Jim, if you are online, can you ping me?04:57
*** chihhsin_ has quit IRC04:57
*** chihhsin has joined #openstack-ironic04:57
*** amotoki has joined #openstack-ironic05:02
*** ChubYann has quit IRC05:13
*** appu103_ has joined #openstack-ironic05:14
*** appu103_ is now known as aparnav05:15
*** skramaja has joined #openstack-ironic05:17
*** PollyZ has quit IRC05:25
openstackgerritDebayan Ray proposed openstack/proliantutils: Change log added for 2.1.9 release  https://review.openstack.org/33428905:35
*** aparnav has quit IRC05:36
*** fragatina has joined #openstack-ironic05:40
*** fragatina has quit IRC05:40
*** fragatina has joined #openstack-ironic05:41
*** sabeen has quit IRC05:48
*** dtantsur|afk is now known as dtantsur05:51
dtantsurMorning Ironic05:51
openstackgerritDebayan Ray proposed openstack/proliantutils: Change log added for 2.1.9 release  https://review.openstack.org/33428905:51
*** jcoufal has joined #openstack-ironic05:57
*** jcoufal has quit IRC05:58
openstackgerritMerged openstack/proliantutils: Change log added for 2.1.9 release  https://review.openstack.org/33428906:06
*** amotoki has quit IRC06:10
*** rbartal has joined #openstack-ironic06:11
*** amotoki has joined #openstack-ironic06:13
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: Enhance Power Interface for Soft Power Off and Inject NMI  https://review.openstack.org/18670006:13
*** mjura has joined #openstack-ironic06:21
caoshufengHi, all, I have a question. Why ironic-api doesn't ask for a token when user access the vendor_passthru api?06:25
*** Sukhdev has quit IRC06:25
dtantsurcaoshufeng, hi! there are a couple of vendor passthru APIs that are not authenticated, the others should be. which one bothers you?06:26
caoshufengdtantsur, such as this: curl -X POST http://10.167.133.33:6385/v1/drivers/pxe_ssh/vendor_passthru/lookup06:27
dtantsurcaoshufeng, yeah, these are not authenticated by design, as they're used from the deployment ramdisk06:27
*** rcernin has joined #openstack-ironic06:27
*** mjura has quit IRC06:28
caoshufengdtantsur, you mean deploy_reamdisk will access this api?06:28
dtantsuryep06:28
caoshufengdtantsur, and deploy_ramdisk don't have anyway to get token?06:29
dtantsurcaoshufeng, doesn't have a *secure* way, at least not with generic drivers (like pxe_ipmitool)06:29
caoshufengdtantsur, thanks for teaching, I will reseach it!06:30
dtantsuryou're welcome06:30
*** M00nr41n has joined #openstack-ironic06:33
*** pcaruana has joined #openstack-ironic06:41
*** mjura has joined #openstack-ironic06:41
*** appu103_ has joined #openstack-ironic06:42
*** moshele has joined #openstack-ironic06:42
openstackgerritCao ShuFeng proposed openstack/ironic-python-agent: Update doc about lookup action  https://review.openstack.org/33430406:45
*** appu103_ has quit IRC06:46
caoshufengdtantsur, Can you help me review a related change?06:48
caoshufengdtantsur, https://review.openstack.org/33430406:48
dtantsur+2, thnx06:48
*** vmud213 has joined #openstack-ironic06:51
*** sacharya has quit IRC06:55
*** ccamacho has joined #openstack-ironic06:57
*** d0ugal has joined #openstack-ironic06:58
*** amotoki has quit IRC06:58
*** d0ugal has quit IRC06:59
*** tesseract- has joined #openstack-ironic06:59
*** d0ugal has joined #openstack-ironic07:00
*** d0ugal has quit IRC07:00
*** d0ugal has joined #openstack-ironic07:00
*** e0ne has joined #openstack-ironic07:01
*** appu103_ has joined #openstack-ironic07:02
*** persia has quit IRC07:08
*** oberezovskyi has left #openstack-ironic07:10
*** jtomasek has joined #openstack-ironic07:11
*** persia has joined #openstack-ironic07:11
*** e0ne has quit IRC07:16
*** e0ne has joined #openstack-ironic07:16
*** amotoki has joined #openstack-ironic07:22
*** ifarkas has joined #openstack-ironic07:28
*** amotoki has quit IRC07:28
openstackgerritDmitry Tantsur proposed openstack/ironic: Drop references to RPC calls from user-visible errors  https://review.openstack.org/33431807:31
*** dmk0202 has joined #openstack-ironic07:31
openstackgerritDmitry Tantsur proposed openstack/ironic: Add dbapi and objects functions to get a node by associated MAC addresses  https://review.openstack.org/33093807:34
openstackgerritDmitry Tantsur proposed openstack/ironic: Introduce new driver call and RPC for heartbeat  https://review.openstack.org/33289007:35
openstackgerritDmitry Tantsur proposed openstack/ironic: Promote agent vendor passthru to core API  https://review.openstack.org/33001807:36
*** amotoki has joined #openstack-ironic07:39
openstackgerritDmitry Tantsur proposed openstack/ironic: [EARLY WIP] Implement new heartbeat for AgentDeploy  https://review.openstack.org/33396407:44
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: [WIP]Introducing Inspector Grenade testing  https://review.openstack.org/32766707:47
*** sacharya has joined #openstack-ironic07:56
*** zzzeek has quit IRC08:00
*** sacharya has quit IRC08:01
*** zzzeek has joined #openstack-ironic08:02
*** lucas-afk is now known as lucasagomes08:02
lucasagomesmorning all08:02
*** athomas has joined #openstack-ironic08:03
ifarkasmorning lucasagomes and all08:03
alinebmorning all08:12
aarefievmorning!08:13
*** itamarl has joined #openstack-ironic08:17
vdrokgood morning ironic!08:18
dtantsurmorning lucasagomes, ifarkas, alineb, aarefiev, vdrok08:19
vdrokmorning dtantsur lucasagomes ifarkas and alineb :)08:19
aarefievmorning  dtantsur vdrok alineb lucasagomes ifarkas08:21
vdrokaarefiev: o/ :)08:21
alinebmorning aarefiev vdrok dtantsur ifarkas lucasagomes08:22
aarefievdtantsur: hey, pushed infra patch for discovery, I thought to do this via rootwrap, but it requires separate bin, which I'd like to avoid08:24
dtantsuraarefiev, not sure what you mean by "it", maybe I'm out of context after the weekend :) what are you doing with a separate bin?08:24
sean-k-mooneydevananda: thanks for the heads up regarding the agent_ssh driver. i was planning on using it to develop the biforst intergration with kolla.08:25
aarefievdtantsur: I'm about discovery tempest which require sudo for virsh and iptables08:25
*** yogi has joined #openstack-ironic08:25
aarefievI hope to do this via rootwrap08:26
dtantsuraarefiev, I agree with https://review.openstack.org/#/c/334337/ fwiw08:26
dtantsurwe do the same in the usual discovery job08:26
sean-k-mooneydevananda: i can use physical server as i do at home but vms are simpler at work.08:26
aarefievdtantsur: yeah, looks like for now it's only valid option08:27
openstackgerritMerged openstack/ironic-python-agent: Update doc about lookup action  https://review.openstack.org/33430408:29
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add a log extension  https://review.openstack.org/24883208:30
*** electrofelix has joined #openstack-ironic08:32
*** phuongnh has quit IRC08:34
*** derekh has joined #openstack-ironic08:36
openstackgerritAline Bousquet proposed openstack/python-ironicclient: Fix error returned by ironic --json node-validate  https://review.openstack.org/30396408:42
*** gbraad has joined #openstack-ironic08:44
gbraadhi, currently trying to dynamicly enroll nodes using bifrost, but it fails without a clear indication: https://gist.github.com/gbraad/1e5a17fd05cd660855fe04c6c753419a08:46
gbraadenrolling nodes from the command line with ironic node-create will work and allows me to ue IPMI08:47
gbraadjust wondering how I can debug bifrost08:47
*** aparnav has joined #openstack-ironic08:48
openstackgerritMoshe Levi proposed openstack/ironic: Adding InfiniBand Support  https://review.openstack.org/26426308:50
openstackgerritlokesh s proposed openstack/ironic: Add support for the audit middleware  https://review.openstack.org/27265808:53
openstackgerritYuriy Zveryanskyy proposed openstack/bifrost: Increase timeout for downloading IPA ramdisk and kernel  https://review.openstack.org/33435608:53
openstackgerritDmitry Tantsur proposed openstack/ironic: [EARLY WIP] Implement new heartbeat for AgentDeploy  https://review.openstack.org/33396408:57
*** sacharya has joined #openstack-ironic08:57
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add vendor id, vendor name and product_id to interface information  https://review.openstack.org/27994008:57
*** hoangcx has quit IRC09:00
*** sacharya has quit IRC09:01
*** hshiina has joined #openstack-ironic09:02
*** mgould|afk is now known as mgould09:04
mgouldmorning Ironic!09:04
dtantsurmorning mgould09:06
mgouldmorning dtantsur09:07
dtantsurgbraad, hi! I don't know bifrost very well, but are you sure about "power" subkey in "driver_info"? ironic itself does not use it, the parameters go directly under driver_info09:07
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Implement new heartbeat for AgentDeploy  https://review.openstack.org/33396409:08
gbraaddtantsur, I'll have a look. but this is modified from their included examples09:12
dtantsurinteresting09:12
dtantsurgbraad, do you have a link to the examples handy?09:12
gbraadI also did it with a conversion from CSV, but that didn't succeed either09:12
sambetts|afkMorning all09:13
*** sambetts|afk is now known as sambetts09:13
dtantsurmorning sambetts09:13
sambettsHey dtantsur09:14
openstackgerritMerged openstack/python-ironic-inspector-client: Use osc-lib instead of openstackclient  https://review.openstack.org/33371509:15
gbraaddtantsur, https://github.com/openstack/bifrost/blob/master/playbooks/inventory/baremetal.json.example09:17
dtantsurgbraad, hmm, I see. then sorry, I can't help much, please wait for TheJulia to wake up09:20
gbraadOK09:21
mgouldmorning sambetts gbraad09:21
gbraadthe way tripleo does it seems much leaner.09:21
*** _milan_ has quit IRC09:22
vsaienkoMorning Ironic'ers!09:23
*** amotoki has quit IRC09:24
*** amotoki has joined #openstack-ironic09:25
dtantsurmorning vsaienko09:25
*** amotoki has quit IRC09:25
vdrokmorning mgould and sambetts09:26
mgouldmorning vsaienko vdrok09:26
*** uggla_ has joined #openstack-ironic09:32
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add power_transition_started_at to the nodes table  https://review.openstack.org/32764209:37
openstackgerritMerged openstack/ironic: Centralize config options - [iboot]  https://review.openstack.org/30185609:57
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Use FLAT network  https://review.openstack.org/33438209:59
*** yibo has quit IRC10:03
*** amotoki has joined #openstack-ironic10:06
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844010:10
openstackgerritMichael Davies proposed openstack/python-ironicclient: Add env var for version cache timeout  https://review.openstack.org/33438710:11
*** dims has quit IRC10:14
*** dims has joined #openstack-ironic10:20
*** Goneri has joined #openstack-ironic10:39
*** links has quit IRC10:43
*** fragatina has quit IRC10:48
*** fragatina has joined #openstack-ironic10:49
*** daemontool has joined #openstack-ironic10:50
*** links has joined #openstack-ironic10:55
*** sacharya has joined #openstack-ironic10:58
*** Goneri has quit IRC10:58
*** _vsaienko has joined #openstack-ironic11:00
openstackgerritSam Betts proposed openstack/ironic: Fail early if ramdisk type is dib, and not building  https://review.openstack.org/32824611:01
*** caoshufeng has quit IRC11:02
*** sacharya has quit IRC11:02
*** _milan_ has joined #openstack-ironic11:03
*** ccamacho is now known as ccamacho|lunch11:03
*** baoli has joined #openstack-ironic11:07
*** baoli_ has joined #openstack-ironic11:08
*** Goneri has joined #openstack-ironic11:11
*** baoli has quit IRC11:12
sambettsdtantsur: Re: your comment about overriding the node_info ironic client, the functions in node_cache.py that aren't in the node_info class all take an ironic parameter, but I don't see any functions inside the NodeInfo class that take an ironic parameter, they all use the self.ironic object11:12
dtantsursambetts, I see. still, we should not change a private attribute. maybe we need a proper setter for 'ironic' then? though it will affect all requests in the processing chain, so I'm not sure it's a good idea11:13
sambettsdtantsur: yeah, I wasn't sure either, it seems like the only way to do it without setting it then reverting the change after11:14
sambettsdtantsur: unless we make every function in the NodeInfo class take an optional ironic kw param and it uses that or falls back to self.ironic11:16
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Create devstack/example.local.conf and include it in the docs  https://review.openstack.org/33441611:16
*** Goneri has quit IRC11:18
openstackgerritVasyl Saienko proposed openstack/ironic: Use FLAT network setup  https://review.openstack.org/33438211:20
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/33442011:20
_milan_#pixiesay morning Ironic! -m flexing11:25
PixieBootsᕙʕ⇀ᴥ⇀ʔᕗ: morning Ironic!11:25
dtantsurmorning _milan_ :)11:25
_milan_dtantsur, morning :)11:25
_milan_guys, I'm wondering why I see this in devstack (old, mitaka)  creating nodes; I've asked for 5 nodes, but  see this: grep 'ironic port-create' /opt/stack/logs/grenade.sh.txt | wc -l11:28
_milan_1111:28
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands  https://review.openstack.org/21672711:28
_milan_of course the result is a failure due to a port with an already existing MAC11:28
mgould_milan_: are some of those ports temporary, perhaps?11:28
mgouldwhat's the output of the grep command without wc -l?11:29
_milan_mgould, not sure11:29
* _milan_ pastes11:29
_milan_http://paste.openstack.org/show/523531/11:29
_milan_mgould, ^11:29
* mgould takes a look11:30
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands  https://review.openstack.org/21672711:30
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add provision state commands  https://review.openstack.org/20611911:30
_milan_they're reused afaictl (dunno why tho) but why 11?11:30
mgould_milan_: what commit of ironic is in /devstack/old?11:31
*** _vsaienko has quit IRC11:31
* mgould is trying to work out what is on line 77711:31
mgould_milan_: it fails half-way through, right?11:31
mgouldso the final number should be larger than that11:32
_milan_mgould, 3aac4cc632a660cef668cf3d711fa419dbe0699711:32
* mgould is guessing it would have been 15, but failed on the (2*5 + 1)th allocation11:32
_milan_mgould, yeah, it fails on the last node create11:32
_milan_well, after11:33
dtantsuryuriyz, any reason not to approve https://review.openstack.org/#/c/334318/ ?11:33
_milan_but why the hack it's 11 times?!?!?! :D11:33
mgouldunknown revision :-(11:33
* _milan_ links github (hopefully)11:33
yuriyzhi dtantsur no reason will do11:33
dtantsurthnx11:33
dtantsurand g'morning11:34
mgouldmorning yuriyz11:34
_milan_mgould https://github.com/openstack/ironic/commit/aca99298f9014c5b4542de2bd113a185748dd91c11:35
mgould_milan_: thanks11:35
mgouldgot it11:36
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands  https://review.openstack.org/21672711:36
*** _vsaienko has joined #openstack-ironic11:37
mgould_milan_: how many lines are in your hwinfo file?11:38
_milan_mgould, I cannot find that file (I'm looking into /opt/stack/data/ironic11:39
_milan_)11:39
mgouldright11:39
mgouldcan you add debugging info to that script, or is it checked out for you by grenade?11:39
mgouldlooking at the timestamps, I see 5 invocations 2s apart11:40
mgouldthen a 10s gap11:40
mgouldthen 6 invocations 2s apart11:41
_milan_it's grenade checkout but possibly I can hack it11:41
mgouldthe first 5 could come from a single invocation of enroll_nodes11:41
*** _vsaienko has quit IRC11:41
mgouldbut I'd expect a similar gap after the next 5 if my theory was correct11:41
mgould_milan_: good luck :-(11:41
mgould*were correct11:42
_milan_mgould, but why is it done 2 times in first place?11:42
mgoulddunno11:42
_milan_or even better, why the second run does 6 times port create if I ask for 5 only :)11:43
mgouldthat just feels like the kind of thing that could go wrong, you know?11:43
mgouldyeah11:43
_milan_right11:43
mgouldthere's only one place enroll_nodes is called from11:43
mgouldbut grenade might be invoking the whole script twice, or something11:43
*** vmud213 has quit IRC11:44
*** _vsaienko has joined #openstack-ironic11:45
_milan_this happens in stack.sh in old11:46
_milan_maybe both inspector and ironic trying to create the nodes?11:46
mgould...maybe?11:49
*** aparnav has quit IRC11:49
*** appu103_ has quit IRC11:49
mgould_milan_: it's possible to roll your own stack-tracer in bash: http://wiki.bash-hackers.org/commands/builtin/caller11:52
mgouldthat assumes you can inject code into the script, of course11:52
* mgould hates bash11:52
_milan_right. I'll try again unstack and grenade it first without that to avoid interference with new bugs ;)11:54
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Move nova extra_specs_ops to ironic-lib  https://review.openstack.org/33443111:54
*** raildo-a` is now known as raildo11:56
lucasagomesjroll, ^11:58
*** lucasagomes is now known as lucas-hungry11:59
*** trown is now known as trown|outtypewww12:04
*** daemontool has quit IRC12:05
*** daemontool has joined #openstack-ironic12:05
*** jjohnson2 has joined #openstack-ironic12:06
*** amotoki has quit IRC12:08
*** jcoufal has joined #openstack-ironic12:08
*** amotoki has joined #openstack-ironic12:09
*** ccamacho|lunch is now known as ccamacho12:10
_milan_mgould, 2016-06-27 12:07:16.635 | ++ /opt/stack/old/ironic/devstack/lib/ironic:enroll_nodes:696 :   local ironic_hwinfo_file=/opt/stack/data/ironic/ironic_macs.csv ; and that file has 10 entries for me, 5 unique12:13
openstackgerritMerged openstack/python-ironicclient: Fix error returned by ironic --json node-validate  https://review.openstack.org/30396412:15
*** krtaylor has quit IRC12:18
_milan_mgould, I dunno  why but the enrol gets called second time :-/12:24
mgould_milan_: so the first ten entries are from the first loop through the file, but then the 11th is from the second invocation?12:28
mgouldweeeeeeiiiiiiirrrrrrrrdddddd12:28
_milan_mgould, seems like that :-/12:28
*** derekh is now known as derekh_afk12:29
mgould_milan_: enroll_nodes is only called from prepare_baremetal_basic_ops12:30
mgouldwhich is called once, directly from plugin.sh12:30
_milan_mgould, yup that seems correct12:31
mgouldhow many times does the string "Starting Ironic" occur in your logs?12:31
mgould[you know what's great? idempotent build steps...]12:32
_milan_mgould, 4 times12:32
_milan_mgould, that would be sweet12:32
mgouldweeeeiiiiiirrrrrrd12:32
mgould_milan_: can you paste the output of grep  'ironic port-create' /opt/stack/logs/grenade.sh.txt12:33
mgouldbah, stupid copy-and-paste12:33
mgould_milan_: can you paste the output of grep  'ironic port-create\|Starting Ironic' /opt/stack/logs/grenade.sh.txt12:33
mgouldor the whole log, possibly12:34
_milan_mgould, I'll share the whole log12:34
mgouldcool12:35
_milan_btw this is the last line before a 3rd call of enrol /opt/stack/old/ironic/devstack/lib/ironic:create_ovs_taps:608 :   neutron net-update 1904e6ae-f40a-4799-b812:35
_milan_06-1fe27a1c5ab4 --shared true12:35
*** snecklifter has joined #openstack-ironic12:36
mgouldright, that's called from create_bridge_and_vms, which is called before enroll_nodes12:37
vdrokmorning _milan_12:38
mgouldmorning snecklifter vdrok12:38
_milan_morning vdrok12:38
_milan_https://drive.google.com/open?id=0B4h8f9zDnf67dkZvU0xSX0dxVFU12:38
sneckliftermgould: morning12:38
_milan_mgould, ^12:39
*** amotoki has quit IRC12:39
mgould_milan_: got it12:39
*** _vsaienko has quit IRC12:39
TheJuliagood morning12:39
mgouldmorning TheJulia12:40
_milan_vdrok, I've update the pull request of the inspector grenade job ; now it does contain both the ironic and inspector grenade plugins12:40
_milan_jlvillal, ^12:40
dtantsurmorning TheJulia12:40
_milan_morning TheJulia12:40
*** _vsaienko has joined #openstack-ironic12:40
mgould_milan_: I have a theory12:42
mgouldhwinfo is being appended to the hwinfo file12:42
vdrok_milan_: maybe you enable ironic twice? :)12:42
vdrokor something like that12:42
mgouldfirst run: file is empty, details of five nodes go into the file12:43
mgouldsecond run: file is already full12:43
mgouldports have been torn down, or something12:43
mgouldhwinfo gets added to the hwinfo file again, making it 2n lines long12:43
mgouldfirst n lines, no problem12:43
_milan_vdrok, yeah could be that12:43
mgouldline n+1, "oh dear that MAC address is taken"12:43
_milan_mgould, I'd say I'm calling it twice somehow (through the grenade plugin most likely) :-/12:45
mgouldyep12:45
mgouldso there are two bugs here:12:45
mgould1) we're setting up ironic twice12:45
mgould2) we're >>ing hwinfo, not >ing it12:45
_milan_mgould, http://paste.openstack.org/show/523540/12:46
mgouldeither on its own would not be fatal12:46
mgouldyep12:46
_milan_mgould, right, that might be it12:46
mgouldthe first call to enroll_nodes creates five ports, the second call creates six ports12:47
mgould_milan_: yep, we're >>ing to that .csv file on line 65212:49
*** rbudden has joined #openstack-ironic12:50
_milan_mgould, correct12:50
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Implement new heartbeat for AgentDeploy  https://review.openstack.org/33396412:50
openstackgerritDmitry Tantsur proposed openstack/ironic: Promote agent vendor passthru to core API  https://review.openstack.org/33001812:50
mgouldso an `rm $IRONIC_VM_MACS_CSV_FILE on line 648 would fix the immediate problem12:50
mgould`12:51
*** rloo has joined #openstack-ironic12:51
*** moshele has quit IRC12:52
*** moshele has joined #openstack-ironic12:52
*** _vsaienko has quit IRC12:53
_milan_mgould, vdrok the issue is I'm trying to "stack" inspector plugin on ironic plugin12:55
_milan_http://paste.openstack.org/show/523541/12:55
dtantsursambetts, one more thing about the lldp inspector extension: we should probably give it a less generic name12:55
moshelejroll, sambetts: can you review again the infiniband spec https://review.openstack.org/#/c/265635/ the ironic change will be simple something like this  https://review.openstack.org/#/c/264263/12:56
dtantsursambetts, maybe 'link-information' or whatever saying what exactly information we update in ironic...12:56
_milan_mgould, vdrok http://paste.openstack.org/show/523544/12:57
mat128good morning Ironic12:57
_milan_morning mat12812:57
mgouldmat128: morning12:58
*** _vsaienko has joined #openstack-ironic12:58
_milan_mgould, vdrok so the enable_plugin ironic is called twice12:58
mgould_milan_: add a "have I already been installed?" check to ironic plugin.sh?12:58
_milan_mgould, that could work yeah12:58
mgouldor a "has ironic already been installed?" check to inspector plugin, but I think that's less robust12:58
_milan_mgould, vdrok this happens because of these 2 files interaction: https://review.openstack.org/#/c/331001/6/jenkins/jobs/devstack-gate.yaml12:59
mgouldkinda like #ifdef MYCOOLLIBRARY_H checks in C12:59
_milan_the GRENADE_PLUGINRC12:59
_milan_var12:59
_milan_and12:59
_milan_https://review.openstack.org/#/c/327667/12/devstack/upgrade/settings13:00
sambettsdtantsur: ok sure :)13:00
_milan_mgould, I'm going to fork devstack in Make&preprocessor13:00
_milan_m413:00
_milan_since although ugly, it works (tm) :D13:00
vdrok_milan_: I think you can drop it from the ironic-inspector plugin settings13:01
*** derekh_afk is now known as derekh13:01
* mgould tries to decide if he hates Make more or less than bash13:01
_milan_vdrok, I'll try that13:01
mgouldprobably more, because Make rules get template-preprocessed to bash13:01
_milan_vdrok, I've been trying to fix my friday issue with grenade not starting ironic13:01
mgouldso all my hatred of bash applies to Make by transitivity13:01
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add vendor id, vendor name and product_id to interface information  https://review.openstack.org/27994013:02
openstackgerritMerged openstack/ironic: Drop references to RPC calls from user-visible errors  https://review.openstack.org/33431813:02
_milan_mgould, not all make is processed by bash, I hope :D13:02
mgouldhttps://www.gnu.org/software/make/manual/html_node/Choosing-the-Shell.html13:04
openstackgerritJulia Kreger proposed openstack/bifrost: Fix testing script permission settings for logs  https://review.openstack.org/33446613:04
mgould/bin/sh by default13:04
_milan_#pixiesay shell everywhere -m dead13:05
PixieBootsʕxᴥxʔ: shell everywhere13:05
mgouldwhich is probably a symlink to bash13:05
_milan_mgould, true, unless one runs it on a wifi router ;)13:06
mgouldcan you use Bashisms if the shell is invoked as /bin/sh?13:06
mgouldif you're building code on a router, you probably have bigger problems :-)13:06
*** _vsaienko has quit IRC13:07
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: [WIP]Introducing Inspector Grenade testing  https://review.openstack.org/32766713:07
_milan_atm my biggest is to tell grenade to use inspector and ironic in a sane and consistent way. Next step: make it work on my router :D13:08
*** _vsaienko has joined #openstack-ironic13:08
mgouldhttp://www.strangehorizons.com/2004/20040405/badger.shtml13:09
vdroklol13:09
*** lucas-hungry is now known as lucasagomes13:09
* mgould recommends Charlie Stross' Laundry series to anyone who hasn't read it13:09
*** sdake has joined #openstack-ironic13:10
mgouldLovecraftian spy fiction, in a world where magic works like computer science13:10
* _milan_ googles cyberspiritual controller program13:10
*** krtaylor has joined #openstack-ironic13:12
*** [1]cdearborn has joined #openstack-ironic13:13
_milan_mgould, that sounds worth reading indeed :)13:14
* _milan_ sudo -H -u stack stdbuf -oL -eL ./grenade.sh && fingers-cross 13:15
mgould:-)13:15
krtaylorsambetts, thiagop - could I beg a follow-up review? this is ready to merge unless you all have concerns ->  https://review.openstack.org/#/c/304683/13:18
*** e0ne has quit IRC13:18
*** e0ne has joined #openstack-ironic13:18
sambettskrtaylor: sure I'll take a look :)13:18
krtaylorsambetts, thanks a bunch13:19
*** sdake_ has joined #openstack-ironic13:20
*** sdake has quit IRC13:21
openstackgerritedan david proposed openstack/ironic: Remove unnecessary calls to dict.keys()  https://review.openstack.org/33447613:27
*** skramaja has quit IRC13:29
*** e0ne has quit IRC13:29
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Implement new heartbeat for AgentDeploy  https://review.openstack.org/33396413:30
openstackgerritDmitry Tantsur proposed openstack/ironic: Introduce new driver call and RPC for heartbeat  https://review.openstack.org/33289013:30
openstackgerritDmitry Tantsur proposed openstack/ironic: Promote agent vendor passthru to core API  https://review.openstack.org/33001813:30
*** e0ne has joined #openstack-ironic13:30
_milan_#pxiesay banana time, _milan_! -m happy13:31
_milan_#pixiesay banana time, _milan_! -m happy13:31
PixieBootsʕ^ᴥ^ʔ: banana time, _milan_!13:31
* _milan_ 's grenade moved past the plugin stacking issue :D13:31
*** wajdi has joined #openstack-ironic13:36
openstackgerritMerged openstack/ironic: Add dbapi and objects functions to get a node by associated MAC addresses  https://review.openstack.org/33093813:37
vdrokwooo another multitenancy merge conflict \o/13:38
mgouldyay13:39
sambetts\o/13:39
TheJuliaugh13:39
openstackgerritSam Betts proposed openstack/ironic-inspector: Add GenericLocalLinkConnectionHook processing hook  https://review.openstack.org/32108213:43
*** skramaja has joined #openstack-ironic13:43
dtantsuroh, and it's me to blame \o/13:44
dtantsursorry vdrok13:44
vdrokdtantsur: np, there were some comments to resolve anyway :)13:44
vdrokwill try to do today13:44
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/33442013:47
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to use FLAT network setup  https://review.openstack.org/33438213:47
*** _vsaienko has quit IRC13:49
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Move nova extra_specs_ops to ironic-lib  https://review.openstack.org/33443113:50
*** amotoki has joined #openstack-ironic13:53
*** skramaja has quit IRC13:55
lucasagomesanyone wants to take a final look at the nova compatible serial console spec (https://review.openstack.org/#/c/319505/) ? I think it's alright now (not colliding with the driver composition reform spec anymore)13:55
*** M00nr41n has quit IRC13:56
*** ametts has joined #openstack-ironic14:00
openstackgerritMerged openstack/python-ironicclient: Add provision state commands  https://review.openstack.org/20611914:02
*** appu103_ has joined #openstack-ironic14:02
*** amotoki has quit IRC14:05
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: [WIP]Introducing Inspector Grenade testing  https://review.openstack.org/32766714:08
openstackgerritMerged openstack/python-ironicclient: Add maintenance mode commands  https://review.openstack.org/21672714:10
*** skramaja has joined #openstack-ironic14:10
*** amotoki has joined #openstack-ironic14:10
*** amotoki has quit IRC14:11
*** mjura has quit IRC14:15
*** appu103_ has quit IRC14:16
*** appu103_ has joined #openstack-ironic14:16
*** sabeen has joined #openstack-ironic14:17
*** rajinir has joined #openstack-ironic14:20
*** PollyZ has joined #openstack-ironic14:20
*** appu103_ has quit IRC14:22
*** ayoung has joined #openstack-ironic14:22
*** hshiina has quit IRC14:22
*** ppiela has joined #openstack-ironic14:31
rloohi and good morning ironic'ers.14:31
*** jjohnson2 has quit IRC14:31
*** jjohnson2 has joined #openstack-ironic14:32
*** rook_ has joined #openstack-ironic14:32
rloodtantsur: question about osc-lib: https://review.openstack.org/#/c/333311/. if we do this change, it means that in order to use openstackclient14:32
xavierrhey rloo good morning :)14:32
rloodtantsur: the user is going to have to install python-openstackclient14:32
xavierrmorning all14:32
vdrokmorning rloo and xavierr14:32
xavierrvdrok, hi there o/14:33
rloodtantsur: is that considered a breaking change? Or is it sufficient to put in reno, that they will need to install that package?14:33
dtantsurrloo, I was wondering the same essentially. Seems like it's an OSC policy, so I thought it's fine.14:33
lucasagomesrloo, xavierr morning14:33
dtantsuranyway reno is the best way to signal such changes14:33
dtantsuralso morning rloo, xavierr14:34
rloodtantsur: the problem is that w/o that change, if you install python-ironicclient, osc commands work.14:34
rloodtantsur: same thing we discussed with the other patch. I'm not sure how to handle it.14:34
dtantsurI also don't like it.. but OSC is pretty heavy, so I understand people not wanting to install it to e.g. use python SDK14:34
rloohi lucasagomes, dtantsur, xavierr, vdrok!14:34
rloodtantsur: i agree with you. i just don't know how to handle it breaking users.14:35
rloodtantsur: or do we release the client with a new major number?14:35
xavierr;)14:35
dtantsurrloo, major version + reno, yeah14:35
*** PollyZ has quit IRC14:36
openstackgerritSaravanan KR proposed openstack/ironic-specs: Provide a mechanism to configure kernel boot parameters  https://review.openstack.org/33156414:36
rloodtantsur: ok. wonder if we should bring it up in the meeting or email list before it lands.14:36
dtantsurrloo, ML, I guess. the meeting is not as popular among our end users as it could be14:37
rloodtantsur: ok14:37
*** PollyZ has joined #openstack-ironic14:42
*** amit213 has joined #openstack-ironic14:43
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add vendor id, vendor name and product_id to interface information  https://review.openstack.org/27994014:44
NobodyCamGood Morning Ironicers says the man just back from 5 days of vacation14:44
vdrokmorning NobodyCam, welcome back :)14:45
NobodyCammorning vdrok :)14:46
xavierrhappy new year NobodyCam 5 days...14:46
xavierr;)14:46
NobodyCam:) and I feel another year older. :p14:46
xavierrhahaha14:47
NobodyCammorning xavierr :)14:47
xavierrmorning and welcome back!14:47
thiagopGood morning, Ironic14:48
*** ayoung_ has joined #openstack-ironic14:48
*** joprovost has joined #openstack-ironic14:48
*** david-lyle_ has joined #openstack-ironic14:51
*** vmud213 has joined #openstack-ironic14:55
dtantsurmorning NobodyCam, thiagop14:55
*** david-lyle has quit IRC14:55
*** crinkle_ is now known as crinkle14:55
openstackgerritRuby Loo proposed openstack/ironic: Replace rootwrap_config with root_helper config  https://review.openstack.org/33452214:56
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [WIP] Use new agent API if available  https://review.openstack.org/33452314:56
NobodyCamMorning thiagop and dtantsur :)14:57
*** amotoki has joined #openstack-ironic14:57
*** sacharya has joined #openstack-ironic14:57
*** itamarl has quit IRC14:58
*** sdake has joined #openstack-ironic15:01
*** jistr is now known as jistr|mtg15:01
*** sdake_ has quit IRC15:03
JayFSo no Deva, no Jim, who is running the meeting today?15:04
*** krotscheck is now known as krotscheck_dcm15:05
JayFI also will have to miss the last half of the meeting15:07
*** skramaja has quit IRC15:09
NobodyCammorning JayF15:09
JayFgm15:09
mgouldmorning JayF NobodyCam thiagop15:09
NobodyCammorning mgould :)15:09
sambettsMorning JayF, NobodyCam15:09
NobodyCammorning sambetts15:09
*** david-lyle_ is now known as david-lyle15:09
thiagopI think dtantsur was the appointed one, but don't know if he did accepted15:09
dtantsurJayF, rloo and I15:09
JayFawesome15:09
dtantsuranyone is free to help ofc :)15:09
rlooyeah, I think I appointed dtantsur :)15:09
openstackgerritVladyslav Drok proposed openstack/python-ironicclient: Add create command to ironic client  https://review.openstack.org/32895515:09
rloomorning JayF, NobodyCam, thiagop15:09
NobodyCamgood morning rloo :)15:09
*** e0ne has quit IRC15:09
*** ayoung has quit IRC15:09
*** ayoung_ is now known as ayoung15:09
openstackgerritVladyslav Drok proposed openstack/python-ironicclient: Add create command to ironic client  https://review.openstack.org/32895515:11
openstackgerritRuby Loo proposed openstack/ironic: Replace rootwrap_config with root_helper config  https://review.openstack.org/33452215:11
mat128hey guys :)15:11
mat128sambetts: any update on multinode on your side?15:11
vdrokmorning JayF and mat12815:12
* vdrok leaves early today, good night!15:12
JayFjfyi today/tomorrow are my last full days for a couple of weeks, as I'm moving15:12
JayFif there's anything you want me to look at before then, please send it to me asap15:12
rloobye vdrok15:12
JayFnn vdrok15:12
mat128o/ vdrok15:13
rloothx for the heads up JayF. Enjoy your move (where?)15:13
JayFTacoma, WA15:13
JayFtl;dr: lots cheaper cost of living, still in the pacific northwest and near the water15:13
sambettsmat128: I've not had a lot of time to work on it since thursday :( got downstream deadlines for the end of the month we're working to meet so its all hands on deck15:14
mat128sambetts: not asking for justification, just dont want to duplicate work already done15:14
mat128=)15:14
rlooJayF: nice!15:14
mat128sambetts: alright, I'll check the neutron integration etherpad15:15
sambettsmat128: :) yeah, just kinda sucks that its taking my time away15:15
JayFmat128: were you able to make any progress on the coreos stable upgrade stuff? I see the patches haven't moved but was curious if any of your downstream testing was fruitful15:15
mat128sambetts: dont worry about it :) I've got a thousand priorities downstream too, just have to juggle with them15:15
*** jistr|mtg is now known as jistr15:16
mat128JayF: introducing your current patches passes all internal tests downstream, still waiting for a second +2 +A (yeah we're on gerrit and friends) before it lands in master15:16
mat128then it's off to real hardware15:16
mat128in staging and prod15:16
mat128I have a task on my todo list for the min ram stuff15:16
*** fragatina has quit IRC15:17
*** fragatina has joined #openstack-ironic15:17
rlooJayF: you haven't commented so maybe it isn't of interest to you, but it would be nice to get these specs approved soon, so if you have any comments: https://review.openstack.org/#/c/319505/, https://review.openstack.org/#/c/186700/, https://review.openstack.org/#/c/294995/15:18
JayFI've stayed mostly out of the BfV stuff because storage tech is probably my weakest spot overall; I'm not familiar with even a single technology much less the whole matrix15:19
mat128rloo: you were right on https://review.openstack.org/#/c/332614/ about s/Partial/Closes/ but it was accepted before I could see it. I adjusted the bug report so it says "Fix Committed"15:19
rlooJayF: that's fine.15:19
JayFI'll read over the serial console support one; any reason I shouldn't land it if I'm +1?15:19
rlooJayF: i say go ahead and land it if it meets your approval!15:20
JayFkk15:20
TheJuliaJayF: Well, if you review and have questions, I'm more than happy to answer them or explain technologies on a call or something to help provide context, and you never know, you might raise some really good questions doing so :)15:20
mat128oh hi Julia :P15:21
JayFTheJulia: I have been what I call stealth-reviewing the code15:21
JayFTheJulia: i.e. reading every few revisions or so15:21
JayFTheJulia: if I didn't think you had it under control I'd say something :P15:21
rloomat128: oh yeah. i think once upon a time (or maybe still), folks might -1 you for that, but you can see how much I care. Updating the bug itself is fine. I mean, if someone looks at the commit msg, they might think there are other patches but anyway.15:21
TheJuliaSpeaking of which, I'll post the BfV revision based on existing feedback in a little bit, meeting permitting15:21
TheJuliaJayF: :)15:21
mat128rloo: I totally agree to fixing it, but I have seen your reply too late15:22
rloomat128: no worries! go do some more useful work instead :)15:22
mat128rloo: haha15:22
mat128yup15:22
dtantsura friendly reminder to update the whiteboard with your statuses15:23
dtantsuralso, any items for the agenda?15:24
*** dmk0202 has quit IRC15:30
JayFmariojv: I'm going to update the whiteboard status for notifications subtask since I know you're downstream-busy this morning15:30
*** harlowja_at_home has joined #openstack-ironic15:31
mariojvthanks JayF15:31
mariojvno updates, plan to rebase later today15:31
*** derekh is now known as derekh_afk15:31
JayFmariojv: https://review.openstack.org/#/c/298461/ is going to need a rebase :(15:31
JayFmariojv: ah, okay. I'll still ask for reviews though since it's not gotten a lot of non-rackspace attention15:32
*** moshele has quit IRC15:33
mariojvcool15:33
openstackgerritMerged openstack/ironic: Add validation of 'ilo_deploy_iso' in deploy.validate()  https://review.openstack.org/32983715:35
JayFBTW for the software metrics support; Blocked on reviews: ironic-lib code up with no negative feedback since 6/8: https://review.openstack.org/#/c/301526/15:37
openstackgerritMerged openstack/ironic: Clear target_power_state on conductor startup  https://review.openstack.org/31009015:37
JayFSo if anyone has extra review bandwidth, alineb and I would appreciate eyes on that15:37
*** pcaruana has quit IRC15:38
*** tesseract- has quit IRC15:40
*** M00nr41n has joined #openstack-ironic15:41
*** bnemec has quit IRC15:43
*** uggla_ has quit IRC15:43
*** bnemec has joined #openstack-ironic15:43
*** daemontool has quit IRC15:44
*** PollyZ has quit IRC15:46
*** bnemec has quit IRC15:50
*** bnemec has joined #openstack-ironic15:52
*** links has quit IRC15:52
dtantsurJayF, reviewed15:52
JayFty15:52
*** sukhdev has joined #openstack-ironic15:52
*** baoli_ has quit IRC15:54
openstackgerritMerged openstack/bifrost: Updated from global requirements  https://review.openstack.org/33227515:54
*** hshiina has joined #openstack-ironic15:54
*** sukhdev has quit IRC15:54
*** sukhdev has joined #openstack-ironic15:54
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/33442015:56
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to use FLAT network setup  https://review.openstack.org/33438215:56
*** PollyZ has joined #openstack-ironic16:02
*** jaybeale has joined #openstack-ironic16:03
sambettsHi is there anyone around who can chair the Ironic Neutron meeting? Or perhaps instruct me how too?16:03
sambettssukdev and jroll are both away16:04
JayFrloo: https://review.openstack.org/#/c/319505/ I looked at that (the serial console support); and am wondering if it needs to be updated for / dependant on the driver comp changes?16:04
mat128JayF: in the midcycle the question came up16:05
rlooJayF: no, I didn't want it to be dependent.16:05
mat128and we ended up choosing that no16:05
rlooJayF: I think it is OK for them to add 2 new drivers.16:05
mat128^this16:05
JayFhm. ok16:05
JayFand you're OK landing it even with the litany of minor comments you have in there?16:06
rlooJayF: I think we can handle a limited number of new drivers.16:06
rlooJayF: I'm OK cuz I don't think we can expect nonEnglish writers to write fluent English.16:06
JayFkk16:06
rlooJayF: I am fine if someone wants to clean up with a followup but it seems like too much effort to ask them to fix. My criteria is 'as long as i can understand it'.16:06
JayFthat's totally fine, I'm a-ok with lower standards for that sort of thing16:07
JayFpatches not being landed, especially in specs, is just being in the way of someone else trying to do good work :)16:07
rlooJayF: my standards are high, but I think I have to lower them sometimes!16:07
JayFI don't wanna be in someone's way just because me and you might be better at scrabble :P16:07
rlooJayF: in user-facing documentation and actual code, I am a bit more strict :)16:08
JayFabsolutely16:08
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Follow-up patch of 8068e1f6d92d22bf25a56b8327c1abd0821b1bcd  https://review.openstack.org/33120016:08
*** rbartal has quit IRC16:08
*** sacharya has quit IRC16:09
mat128That would make a great quote, JayF16:09
mat128"I don't wanna be in someone's way just because me and you might be better at scrabble :P"16:09
JayFHey, I met my wife by challenging her to a scrabble game on an online dating site like six years ago. Scrabble is life skills :P16:10
mat128wow nice16:10
rlooJayF: that's great. Remind me never to play scrabble with you :) [I'm a bananagrams person myself even though I don't like bananas.]16:11
JayFlol. We don't play much anymore just because we're so equally matched it's like all up to the random pull of the letters who wins.16:12
*** sacharya has joined #openstack-ironic16:12
*** rbudden has quit IRC16:12
jlvillalJayF, My wife and I do joint Sodoku recently. We aren't very good at it :)16:12
openstackgerritMerged openstack/ironic-specs: Nova compatible serial console support  https://review.openstack.org/31950516:12
JayFhaha. Crosswords is what we picked up now. At least one a day typically16:13
JayFkeeps the brain shark16:13
JayF*sharp16:13
rlooJayF: my neighbours are > 70 years old. They play scrabble with each other almost every day (with wine). They have the scrabble dictionary and look up things. It is so sweet.16:13
jlvillalJayF, Those are fun to do together.16:13
*** uggla_ has joined #openstack-ironic16:14
jlvillalsambetts, I added an agenda item to make the IPA functional tests functional. I hope nobody objects :)16:14
* jlvillal brews another cup of Earl Grey16:14
jlvillalsambetts, I added an agenda item to make the IPA functional tests VOTING. I hope nobody objects :)16:15
*** ayoung has quit IRC16:16
*** klindgren has joined #openstack-ironic16:16
sambettsjlvillal: oh cool, no objection from me as long as they've proven themselves as stable16:16
openstackgerritMerged openstack/ironic-python-agent: Documentation follow-up to the LLDP patch  https://review.openstack.org/33407616:17
openstackgerritvinay kumar muddu proposed openstack/proliantutils: Removes duplicated code and reduces code bloat  https://review.openstack.org/33455316:19
jlvillalsambetts, I was thinking you had done a fix recently to fix the functional tests.16:19
jlvillalLooking at recent patches that have merged. Everything is looking green.16:20
sambettsjlvillal: yeah, I think JayF had concerns that it might not be stable, although that might have been for in the stable/branches16:20
JayFsambetts: I mean, anything that's not in CI is unstable until proven otherwise. By being run in CI.16:21
JayFsambetts: altohugh I don't remember commenting on that specifically16:21
JayFsambetts: just a matter of if it's not tested, it doesn't work. (unless you're lucky and it does)16:21
jlvillalJayF, It has been a non-voting job for quite some time16:23
*** klindgren has quit IRC16:23
JayFso that patch that got landed to break the functional tests, it failed and we still merged? :(16:23
JayFboo us16:23
sambettsJayF: oooh, it was dtantsur not you that left the comment16:23
sambettshttps://review.openstack.org/#/c/332766/16:23
*** _vsaienko has joined #openstack-ironic16:24
sambettswe only added the job 4 days ago16:24
JayFokay that more fits with my recollection16:24
jlvillalsambetts, Oh I thought it had been awhile16:24
jlvillalsambetts, My bad16:24
jlvillalI remember Josh and mariojv doing the functional tests almost a year ago.16:25
JayFI honestly don't think they've been added to at all much since then16:25
sambettsjlvillal: func tests as far as I know have been working for a while, func tests got broken at the end of last week due to a bad merge, which lucasagomes fixed16:25
jlvillalSeattle mid-cycle16:25
sambettsjlvillal: the we decided we needed to add a job to prevent that happening again16:25
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection  https://review.openstack.org/20190416:26
lucasagomesthe job is currently non voting16:26
lucasagomesbut we cna change it soon16:26
*** klindgren has joined #openstack-ironic16:26
*** klindgren has quit IRC16:26
rloolucasagomes: you forgot jlvillal's nit: https://review.openstack.org/#/c/331200/2..3/specs/approved/collect-system-logs-from-ipa.rst. jlvillal, do you care?16:26
lucasagomesrloo, oh, lemme put a new patch-set16:27
lucasagomeswill take 2 sec16:27
*** rpioso has joined #openstack-ironic16:27
* jlvillal looks16:28
lucasagomesrloo, jlvillal done ^ :-)16:28
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Follow-up patch of 8068e1f6d92d22bf25a56b8327c1abd0821b1bcd  https://review.openstack.org/33120016:28
lucasagomeswell ^16:28
NobodyCammorning lucasagomes16:28
lucasagomesNobodyCam, hi there16:28
NobodyCam:)16:28
rloothx lucasagomes16:29
jlvillalJayF, Want to do a quick re-review of https://review.openstack.org/#/c/331200/  lucasagomes did a minor update16:29
*** _vsaienko has quit IRC16:30
NobodyCammorning jlvillal16:30
*** ayoung has joined #openstack-ironic16:31
mgouldmorning jlvillal16:31
*** moshele has joined #openstack-ironic16:31
*** klindgren has joined #openstack-ironic16:34
*** Sukhdev_ has joined #openstack-ironic16:34
lucasagomeswe are having the meeting today right?16:34
jlvillalGood morning NobodyCam mgould rloo lucasagomes dtantsur sambetts JayF vdrok vsaienko TheJulia cinerama and everyone else :)16:34
jlvillallucasagomes, In 25 minutes16:35
lucasagomesjlvillal, morning :-)16:35
NobodyCam:)16:35
lucasagomescool, ok, lemme get a quick bite before joining then16:35
jlvillallucasagomes, Run by acting supreme leader dtantsur16:35
* lucasagomes brb16:35
*** yogi has quit IRC16:35
dtantsurlol16:35
* dtantsur thinks he should learn the commands we use during the meeting16:36
*** amotoki has quit IRC16:37
*** harlowja_at_home has quit IRC16:37
openstackgerritMerged openstack/bifrost: Fix testing script permission settings for logs  https://review.openstack.org/33446616:38
dtantsurthis is the final call for updating the whiteboard and adding your items to the agenda16:39
dtantsurplease do!16:39
jlvillaldtantsur, #startmeeting, #topic, #info, #endmeeting16:39
dtantsurthnx16:39
jlvillalTakes care of pretty much everything for my meetings :)16:39
*** vmud213 has quit IRC16:40
rloojlvillal: maybe we should vote on something, just to add more commands :)16:41
*** jvgrant_ has quit IRC16:42
*** jvgrant_ has joined #openstack-ironic16:42
openstackgerritImre Farkas proposed openstack/ironic: DRAC RAID configuration  https://review.openstack.org/17631916:42
vsaienkogood morning jlvillal16:42
dtantsuroh, DRAC RAID, cool16:43
jlvillalrloo, :P16:43
*** rama_y has joined #openstack-ironic16:44
* jlvillal read that as "Oh draT RAID"16:45
*** e0ne has joined #openstack-ironic16:46
rloolucasagomes: any idea what the issue is wrt it not getting in oslo.utils? https://review.openstack.org/#/c/334431/16:46
dtantsurlol. I think ifarkas might agree with you16:46
*** amit213 has quit IRC16:46
*** athomas has quit IRC16:46
jlvillal:)16:46
TheJuliagood morning jlvillal16:47
ifarkasdtantsur, sorry, I am bit out of context16:47
dtantsurifarkas: * jlvillal read that as "Oh draT RAID"16:47
dtantsurjust kidding about your patch :)16:47
ifarkaslol, ok16:47
ifarkasI missed that comment16:47
jlvillalifarkas, Just becase DRAC and drat are very similar in spelling :)16:48
ifarkasjlvillal, gotcha ;-)16:48
*** swatson has joined #openstack-ironic16:51
JayFdtantsur: I'm going to be travelling into the city for most of the meeting time; but I made sure subteam reports for IPA/metrics/rescue/policy are all up to date16:51
devanandamorning, all.16:51
dtantsurawesome, thanks16:51
NobodyCamgood morning devananda16:51
dtantsuroh we got someone to chair the meeting, morning devananda ;)16:51
* devananda is fighting off a cold bug (still, again) this week and moving real slow16:51
*** aNupoisc has joined #openstack-ironic16:51
devanandaoh? no jroll today?16:51
dtantsurdevananda, no jroll till somewhere in july iirc16:52
TheJuliagood morning devananda16:52
jlvillalI think he will be back for next meeting. As I think we are cancelled for July 4th (US Holiday)16:52
*** sabeen has quit IRC16:53
devanandaah, I see16:53
* NobodyCam has to reboot .... brb16:53
dtantsurjlvillal, devananda, I can show up for July 4th if people want, but I don't insist, as I'll be on PTO :)16:53
jlvillaldtantsur, I'm not insisting. Since I will not be here :)16:53
devanandadtantsur: I will probably not be here for july 4th16:53
dtantsurok, so I'll ask around that day16:54
JayFI think it's very sensible to cancel the July 4 meeting16:55
_milan_vdrok, mgould so with the last revision of the inspector grenade plugin I made it work up to detecting service running after upgrade. Seems grenade uses ps | grep service-name that doesn't work with ironic-inspector-dhcp (dnsmasq) https://review.openstack.org/#/c/334564/ could use a+1 ;)16:55
sambettsI'll also cancel the Ironic neutron meeting if the Ironic one is canceled for the 4th of July16:55
devanandasambetts: ++16:57
dtantsurdevananda, just noticed you're not feeling well, do you want me to chair the meeting?16:57
devanandadtantsur: if you could, that would be much appreciated16:58
dtantsurno problem, will do16:58
devanandaI can chair it, just don't have a lot of brain-energy to track things16:58
devanandathanks :)16:58
dtantsur1.5 minutes before the meeting16:58
devanandaagenda looks light, too16:59
dtantsurtrue16:59
lucasagomesdevananda, hope you get better soon17:00
devanandathanks17:00
*** jaybeale_ has joined #openstack-ironic17:01
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add vendor id, vendor name and product_id to interface information  https://review.openstack.org/27994017:02
*** jaybeale has quit IRC17:03
*** Sukhdev_ has quit IRC17:05
*** ayoung has quit IRC17:06
*** jaybeale_ has quit IRC17:09
*** stendulker has joined #openstack-ironic17:11
*** xhku has joined #openstack-ironic17:12
*** xhku has quit IRC17:12
*** jaybeale has joined #openstack-ironic17:12
*** xhku has joined #openstack-ironic17:13
*** Sukhdev_ has joined #openstack-ironic17:13
*** xhku_ has quit IRC17:15
* JayF getting offline, if anyone has any action for me post-meeting just leave it here and I'll see it later :)17:16
*** rbudden has joined #openstack-ironic17:16
mat128JayF: so far latest CoreOS boots on 1G :)17:18
mat128lookup + heartbeat, no image deploy yet17:18
*** ayoung has joined #openstack-ironic17:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [inspector]  https://review.openstack.org/30195517:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [irmc]  https://review.openstack.org/30247617:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [iscsi]  https://review.openstack.org/30961517:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe]  https://review.openstack.org/30961217:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [seamicro]  https://review.openstack.org/30311317:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [agent]  https://review.openstack.org/30960317:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [neutron]  https://review.openstack.org/30483817:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [ssh]  https://review.openstack.org/30364917:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [deploy]  https://review.openstack.org/30920617:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize Config Options - patch merge, cleanup  https://review.openstack.org/33016017:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [ipmi]  https://review.openstack.org/30196717:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [swift]  https://review.openstack.org/30365317:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [virtualbox]  https://review.openstack.org/30365517:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [oneview]  https://review.openstack.org/30310317:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [glance]  https://review.openstack.org/30482917:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [ilo]  https://review.openstack.org/30194717:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [snmp]  https://review.openstack.org/30364417:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [keystone]  https://review.openstack.org/30309317:22
openstackgerritJulia Kreger proposed openstack/ironic-specs: Boot from Volume - Reference Drivers  https://review.openstack.org/29499517:23
*** penick has joined #openstack-ironic17:24
*** krtaylor has quit IRC17:24
*** harlowja has quit IRC17:25
*** piet has quit IRC17:26
*** jjohnson2 has quit IRC17:27
*** jjohnson2 has joined #openstack-ironic17:28
*** harlowja has joined #openstack-ironic17:33
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [seamicro]  https://review.openstack.org/30311317:33
*** penick_ has joined #openstack-ironic17:33
*** Sukhdev_ has quit IRC17:34
*** penick has quit IRC17:35
*** penick_ is now known as penick17:35
NobodyCamsorry was on an internal call missed most if the meeting17:35
*** amotoki has joined #openstack-ironic17:37
*** xhku has quit IRC17:38
*** xhku has joined #openstack-ironic17:39
*** amotoki has quit IRC17:42
*** PollyZ has quit IRC17:42
*** xhku has quit IRC17:42
*** jjohnson2_ has joined #openstack-ironic17:43
*** xhku has joined #openstack-ironic17:44
*** moshele has quit IRC17:44
*** _vsaienko has joined #openstack-ironic17:44
*** jjohnson2 has quit IRC17:45
*** piet has joined #openstack-ironic17:45
*** ifarkas has quit IRC17:48
*** fragatina has quit IRC17:49
*** jaybeale has quit IRC17:50
*** xhku has quit IRC17:56
*** xhku has joined #openstack-ironic17:56
*** PollyZ has joined #openstack-ironic17:58
lucasagomesalright folks, I'm calling it a day18:00
lucasagomeshave a great evening all18:00
dtantsurme too18:00
lucasagomessee you all tomorrow18:00
dtantsurg'night lucasagomes, good evening everyone18:00
rloobye lucasagomes, dtantsur18:00
devanandag'night!18:00
lucasagomesdtantsur, night!18:00
*** lucasagomes is now known as lucas-afk18:01
devanandaalso, i'm off to the airport - see ya'll online intermittently over the next few days18:01
rloohave a safe trip devananda!18:01
*** xhku_ has joined #openstack-ironic18:01
*** xhku_ has quit IRC18:01
rlooI'm staying and working :)18:01
*** xhku_ has joined #openstack-ironic18:02
*** dtantsur is now known as dtantsur|afk18:02
*** xhku_ has quit IRC18:02
*** xhku has quit IRC18:03
*** xhku has joined #openstack-ironic18:05
mat128staying and getting into a meeting :(18:06
*** xhku has quit IRC18:06
*** xhku has joined #openstack-ironic18:06
*** _milan_ has quit IRC18:07
*** xhku has quit IRC18:09
*** stendulker has quit IRC18:10
JayFmat128: it certainly doesn't in the gate; you can see that on the patch I have up18:10
*** blakec has joined #openstack-ironic18:12
*** Sukhdev_ has joined #openstack-ironic18:16
openstackgerritJarrod Johnson proposed openstack/pyghmi: Add support for PyCA cryptography  https://review.openstack.org/33182818:18
*** xhku has joined #openstack-ironic18:21
*** hshiina has quit IRC18:22
*** sukhdev has quit IRC18:23
*** xhku has quit IRC18:26
*** aNupoisc has quit IRC18:26
*** ayoung has quit IRC18:27
*** aNupoisc has joined #openstack-ironic18:32
*** Sukhdev_ has quit IRC18:33
*** rcernin has quit IRC18:34
*** blakec1 has joined #openstack-ironic18:46
*** caldw__ has joined #openstack-ironic18:48
*** blakec has quit IRC18:49
*** fellypefca has joined #openstack-ironic18:50
*** blakec1 has quit IRC18:51
*** ChubYann has joined #openstack-ironic18:51
*** livelace has quit IRC18:53
*** caldw__ has quit IRC18:58
*** vgadiraj has joined #openstack-ironic19:01
*** sabeen has joined #openstack-ironic19:02
openstackgerritMario Villaplana proposed openstack/ironic: Add power state change notifications  https://review.openstack.org/32186519:02
openstackgerritMario Villaplana proposed openstack/ironic: Add notification base classes and docs  https://review.openstack.org/29846119:02
*** rloo has quit IRC19:04
*** rcernin has joined #openstack-ironic19:04
*** rloo has joined #openstack-ironic19:04
*** amit213 has joined #openstack-ironic19:04
*** sacharya_ has joined #openstack-ironic19:04
*** klindgren has quit IRC19:05
*** sacharya has quit IRC19:07
*** vgadiraj has quit IRC19:08
*** e0ne has quit IRC19:10
*** e0ne has joined #openstack-ironic19:11
*** adu has joined #openstack-ironic19:12
*** e0ne_ has joined #openstack-ironic19:13
*** jcoufal has quit IRC19:14
*** e0ne has quit IRC19:17
*** sukhdev has joined #openstack-ironic19:19
*** Sukhdev_ has joined #openstack-ironic19:20
*** sacharya has joined #openstack-ironic19:24
*** piet has quit IRC19:25
*** sacharya_ has quit IRC19:28
*** moshele has joined #openstack-ironic19:29
*** uggla_ has quit IRC19:32
*** xhku has joined #openstack-ironic19:34
*** xhku has quit IRC19:36
*** harlowja has quit IRC19:36
*** xhku has joined #openstack-ironic19:37
*** sdake has quit IRC19:38
*** amotoki has joined #openstack-ironic19:39
*** sdake has joined #openstack-ironic19:40
*** ayoung has joined #openstack-ironic19:42
*** e0ne_ has quit IRC19:43
*** amotoki has quit IRC19:44
*** rajinir has quit IRC19:44
*** derekh_afk is now known as derekh19:48
*** fellypefca has quit IRC19:50
*** harlowja has joined #openstack-ironic19:52
*** piet has joined #openstack-ironic19:53
*** moshele has quit IRC19:56
sambettsnight all19:57
*** sambetts is now known as sambetts|afk19:57
*** klindgren has joined #openstack-ironic20:01
mat128cya20:01
*** aNupoisc has quit IRC20:01
*** Sukhdev_ has quit IRC20:04
*** vgadiraj has joined #openstack-ironic20:15
*** sdake_ has joined #openstack-ironic20:19
*** sdake has quit IRC20:22
openstackgerritMerged openstack/ironic-specs: Follow-up patch of 8068e1f6d92d22bf25a56b8327c1abd0821b1bcd  https://review.openstack.org/33120020:24
*** aNupoisc has joined #openstack-ironic20:30
*** sabeen has quit IRC20:32
*** adu has quit IRC20:32
*** derekh has quit IRC20:33
*** sabeen has joined #openstack-ironic20:38
*** joprovost has quit IRC20:41
*** vgadiraj has quit IRC20:43
*** sdake has joined #openstack-ironic20:49
*** sdake_ has quit IRC20:53
*** catintheroof has joined #openstack-ironic20:55
*** wajdi_ has joined #openstack-ironic20:57
*** wajdi has quit IRC21:00
*** wajdi_ has quit IRC21:02
*** milan has joined #openstack-ironic21:03
*** sinh has joined #openstack-ironic21:05
*** lucas-afk has quit IRC21:06
*** raildo is now known as raildo-afk21:07
*** sdake_ has joined #openstack-ironic21:12
*** jjohnson2_ has quit IRC21:12
*** lucasagomes has joined #openstack-ironic21:13
*** sdake has quit IRC21:13
openstackgerritPolly Zhou proposed openstack/ironic: Improve tests for driver's parse_driver_info()  https://review.openstack.org/33466621:15
*** sdake has joined #openstack-ironic21:16
*** sdake_ has quit IRC21:17
*** klindgren has quit IRC21:18
*** klindgren has joined #openstack-ironic21:19
*** lucasagomes has quit IRC21:19
*** lucasagomes has joined #openstack-ironic21:27
*** xhku has quit IRC21:34
*** xhku has joined #openstack-ironic21:34
*** klindgren_ has joined #openstack-ironic21:37
*** klindgren has quit IRC21:39
*** PollyZ has quit IRC21:42
*** jaybeale has joined #openstack-ironic21:46
*** PollyZ has joined #openstack-ironic21:50
*** catintheroof has quit IRC21:55
*** jjohnson2 has joined #openstack-ironic22:02
*** [1]cdearborn has quit IRC22:08
*** terje has joined #openstack-ironic22:13
*** sabeen has quit IRC22:19
*** sabeen has joined #openstack-ironic22:19
*** aNupoisc has quit IRC22:22
*** sukhdev has quit IRC22:23
*** ametts has quit IRC22:28
*** aNupoisc has joined #openstack-ironic22:33
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Add Ironic specs process to the code contribution guide  https://review.openstack.org/33016722:36
*** ametts has joined #openstack-ironic22:41
*** jaybeale has quit IRC22:43
*** aNupoisc has quit IRC22:45
*** ametts has quit IRC22:48
openstackgerritVladyslav Drok proposed openstack/ironic: Added operator documentation for ironic portgroups  https://review.openstack.org/22849622:51
openstackgerritVladyslav Drok proposed openstack/ironic: Add multitenancy-related fields to port API object  https://review.openstack.org/20624422:51
openstackgerritVladyslav Drok proposed openstack/ironic: Add network_interface node field to DB and object  https://review.openstack.org/31739122:51
openstackgerritVladyslav Drok proposed openstack/ironic: Add 'neutron' network driver  https://review.openstack.org/31739322:51
openstackgerritVladyslav Drok proposed openstack/ironic: Expose node's network_interface field in API  https://review.openstack.org/31739222:51
openstackgerritVladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326222:51
openstackgerritVladyslav Drok proposed openstack/ironic: Add network interface to base driver class  https://review.openstack.org/28585222:51
openstackgerritVladyslav Drok proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/33217722:51
openstackgerritVladyslav Drok proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643222:53
*** rpioso has quit IRC23:00
*** gfhellma has joined #openstack-ironic23:05
*** PollyZ has quit IRC23:06
*** _vsaienko has quit IRC23:06
*** _vsaienko has joined #openstack-ironic23:06
jlvillalNobodyCam, Not sure if you want to look at: https://review.openstack.org/#/c/272658/  Release notes look wonky23:12
jlvillalNobodyCam, I noticed you were a co-author23:13
jlvillalNobodyCam, Never mind. Looks like something else broke the releasenotes23:15
*** rajinir has joined #openstack-ironic23:15
openstackgerritMerged openstack/bifrost: Increase timeout for downloading IPA ramdisk and kernel  https://review.openstack.org/33435623:16
*** aNupoisc has joined #openstack-ironic23:17
openstackgerritJohn L. Villalovos proposed openstack/ironic: Fix releasenotes formatting error  https://review.openstack.org/33471223:28
*** _vsaienko has quit IRC23:28
openstackgerritMichael Davies proposed openstack/python-ironicclient: Add env var for version cache timeout  https://review.openstack.org/33438723:28
*** _vsaienko has joined #openstack-ironic23:33
jlvillalGood morning mrda23:33
*** thrash is now known as thrash|g0ne23:33
mrdahey jlvillal23:34
jlvillalmrda, I already reviewed! :)23:34
mrda\o/23:34
*** piet has quit IRC23:38
*** sacharya has quit IRC23:40
*** piet has joined #openstack-ironic23:40
*** amotoki has joined #openstack-ironic23:41
*** _vsaienko has quit IRC23:44
*** amotoki has quit IRC23:45
*** jaybeale has joined #openstack-ironic23:54
*** gfhellma has quit IRC23:56
*** wajdi has joined #openstack-ironic23:58

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