Thursday, 2017-10-19

openstackgerritMathieu Gagné proposed openstack/ironic master: Add ability to provide configdrive when rebuilding  https://review.openstack.org/50176100:10
*** crushil has quit IRC00:12
*** jkilpatr has quit IRC00:18
*** yangyapeng has quit IRC00:21
*** yangyapeng has joined #openstack-ironic00:22
*** ijw has joined #openstack-ironic00:23
*** yangyapeng has quit IRC00:27
*** rwsu has quit IRC00:30
*** ijw has quit IRC00:31
openstackgerritMerged openstack/ironic-python-agent master: Remove python-subunit in test-requirements  https://review.openstack.org/51002800:34
*** rama_y has quit IRC00:39
*** crushil has joined #openstack-ironic00:43
*** rwsu has joined #openstack-ironic00:51
*** masuberu has joined #openstack-ironic01:00
*** masber has quit IRC01:04
*** phuongnh has joined #openstack-ironic01:12
*** zhangfei has joined #openstack-ironic01:15
*** yangyape_ has joined #openstack-ironic01:21
*** namnh has joined #openstack-ironic01:23
*** zhangfei has quit IRC01:24
*** baoli has joined #openstack-ironic01:25
*** ijw has joined #openstack-ironic01:27
*** litao__ has joined #openstack-ironic01:31
*** ijw has quit IRC01:32
*** zhangfei has joined #openstack-ironic01:37
*** masber has joined #openstack-ironic01:44
*** masuberu has quit IRC01:47
*** baoli has quit IRC01:55
*** ijw has joined #openstack-ironic01:56
*** ijw has quit IRC02:01
*** ijw has joined #openstack-ironic02:02
*** baoli has joined #openstack-ironic02:03
*** zshi__ has joined #openstack-ironic02:06
*** ijw has quit IRC02:06
*** hw_wutianwei has joined #openstack-ironic02:17
*** masber has quit IRC02:19
*** baoli has quit IRC02:20
*** rwsu has quit IRC02:24
*** ijw has joined #openstack-ironic02:35
*** ijw has quit IRC02:39
*** hshiina has joined #openstack-ironic02:46
*** karthiks has quit IRC02:48
openstackgerritMerged openstack/ironic master: Fix missing logging format error  https://review.openstack.org/51253502:58
openstackgerritNguyen Van Trung proposed openstack/ironic master: Update validating node information docs  https://review.openstack.org/51323102:59
*** TuanLA has joined #openstack-ironic03:03
*** aolwas has quit IRC03:11
*** aolwas has joined #openstack-ironic03:13
*** TuanLA has quit IRC03:25
*** trungnv has quit IRC03:25
*** namnh has quit IRC03:25
*** phuongnh has quit IRC03:25
*** trungnv has joined #openstack-ironic03:26
*** namnh has joined #openstack-ironic03:26
*** TuanLA has joined #openstack-ironic03:26
*** phuongnh has joined #openstack-ironic03:26
*** hshiina has quit IRC03:33
*** hshiina has joined #openstack-ironic04:01
*** arnewiebalck has quit IRC04:14
*** namnh has quit IRC04:21
*** TuanLA has quit IRC04:21
*** phuongnh has quit IRC04:21
*** trungnv has quit IRC04:21
*** namnh has joined #openstack-ironic04:21
*** TuanLA has joined #openstack-ironic04:21
*** phuongnh has joined #openstack-ironic04:21
*** trungnv has joined #openstack-ironic04:21
*** ijw has joined #openstack-ironic04:23
*** ijw has quit IRC04:28
openstackgerritfupingxie proposed openstack/ironic master: stop console when unnecessary  https://review.openstack.org/51142204:49
openstackgerritMerged openstack/ironic stable/pike: Reduce complexity of node_power_action() function  https://review.openstack.org/50832104:56
openstackgerritMerged openstack/ironic stable/pike: ipmitool: reboot: Don't power off node if already off  https://review.openstack.org/50832204:56
*** marios has joined #openstack-ironic05:13
*** Goneri has joined #openstack-ironic05:15
*** masber has joined #openstack-ironic05:15
*** ChubYann has quit IRC05:15
*** masuberu has joined #openstack-ironic05:17
*** masber has quit IRC05:19
*** zshi__ has quit IRC05:33
*** spectr has joined #openstack-ironic05:48
*** ijw has joined #openstack-ironic05:50
*** alex_xu has quit IRC05:54
*** alex_xu has joined #openstack-ironic06:00
*** mjura has joined #openstack-ironic06:08
*** hoonetorg has quit IRC06:09
*** parlos has joined #openstack-ironic06:13
*** yolanda has quit IRC06:15
*** e0ne has joined #openstack-ironic06:23
openstackgerritMerged openstack/ironic master: Secure boot support for irmc-virtual-media driver  https://review.openstack.org/50778006:25
*** hoonetorg has joined #openstack-ironic06:25
*** vsaienk0 has joined #openstack-ironic06:36
*** jtomasek has joined #openstack-ironic06:42
*** jtomasek has quit IRC06:43
*** jtomasek has joined #openstack-ironic06:43
*** ijw has quit IRC06:48
openstackgerritAparna proposed openstack/ironic master: Adds RPC calls for rescue interface  https://review.openstack.org/50933606:53
openstackgerritMerged openstack/ironic master: Move ironic legacy jobs into the ironic tree  https://review.openstack.org/51126706:53
*** vishwana_ has joined #openstack-ironic06:54
*** vishwanathj has quit IRC06:54
*** pcaruana has joined #openstack-ironic06:55
*** vishwanathj has joined #openstack-ironic06:58
*** vishwana_ has quit IRC06:59
*** hshiina has quit IRC07:03
*** zhangfei has quit IRC07:09
*** karimb has joined #openstack-ironic07:10
openstackgerritAparna proposed openstack/ironic master: Adds rescue_interface to base driver class  https://review.openstack.org/50933507:10
openstackgerritAparna proposed openstack/ironic master: Adds RPC calls for rescue interface  https://review.openstack.org/50933607:13
openstackgerritAparna proposed openstack/ironic master: Add rescuewait timeout periodic task  https://review.openstack.org/35315607:15
openstackgerritAparna proposed openstack/ironic master: Add rescue related methods to network interface  https://review.openstack.org/50934207:17
*** vmud213 has joined #openstack-ironic07:17
*** skazi has quit IRC07:18
*** crushil has quit IRC07:19
*** tesseract has joined #openstack-ironic07:21
openstackgerritAparna proposed openstack/ironic master: Agent driver rescue implementation  https://review.openstack.org/40043707:21
*** zhangfei has joined #openstack-ironic07:22
openstackgerritAparna proposed openstack/ironic master: Add API methods for [un]rescue  https://review.openstack.org/35083107:23
*** skazi has joined #openstack-ironic07:31
*** e0ne has quit IRC07:33
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector master: WiP add zuul config  https://review.openstack.org/51329407:36
pas-hamorning ironic :)07:42
pas-hayay! zuul3 jobs in-tree!07:42
pas-haalthough there are definitely some quirks to decide on, like the inspector job o_O07:43
*** ktibi has joined #openstack-ironic07:45
*** parlos has quit IRC07:51
*** gcb has quit IRC07:55
*** ccamacho has joined #openstack-ironic07:57
*** ralonsoh has joined #openstack-ironic08:03
*** parlos has joined #openstack-ironic08:05
*** vsaienk0 has quit IRC08:06
*** _milan_ has joined #openstack-ironic08:10
*** lucas-afk is now known as lucasagomes08:16
*** mgoddard_ has joined #openstack-ironic08:17
*** vsaienk0 has joined #openstack-ironic08:22
*** AlexeyAbashkin has joined #openstack-ironic08:28
*** gcb has joined #openstack-ironic08:28
*** dtantsur|afk is now known as dtantsur08:32
dtantsurmorning ironic08:32
Tenguhello dtantsur :)08:32
*** yolanda has joined #openstack-ironic08:34
*** derekh has joined #openstack-ironic08:46
*** spectr has quit IRC08:47
*** hjensas has quit IRC08:48
*** spectr has joined #openstack-ironic08:48
*** spectr has quit IRC08:48
*** spectr has joined #openstack-ironic08:50
*** parlos has quit IRC09:06
*** yolanda has quit IRC09:16
*** TuanLA has quit IRC09:16
*** trungnv has quit IRC09:16
*** phuongnh has quit IRC09:16
*** namnh has quit IRC09:16
*** TuanLA has joined #openstack-ironic09:17
*** phuongnh has joined #openstack-ironic09:17
*** trungnv has joined #openstack-ironic09:17
*** namnh has joined #openstack-ironic09:17
*** yolanda has joined #openstack-ironic09:18
*** e0ne has joined #openstack-ironic09:20
*** parlos has joined #openstack-ironic09:24
*** sambetts|afk is now known as sambetts09:24
sambettspas-ha: I think we can make all jobs live in the ironic tree, and then our sub-projects can use them from there09:25
pas-hasambetts: yes, but they have to be updated to use the new names09:26
sambettsyup09:26
pas-haalready have some patches up, for inspector and n-g-s09:26
dtantsuris inspector broken or just behaving weirdly?09:26
sambettspas-ha: what I'm saying is that the job definition can't live in the inspector tree unfortunatly09:26
pas-hawhy?09:27
sambettsdtantsur: currently not brekon, but its configured to use the old legacy job defintion which is renamed09:27
sambettspas-ha: because there is a cross dependecy, ironic depends on inspector job and inspector depends in an ironic job, and job loading happends in order ironic then inspector or vs versa09:28
sambettsso if the job definition moved into inspector tree, ironic would break because the jobs from inspector aren't loaded yet09:28
pas-hagot you.09:29
sambettsor if we changed the order of the projects, inspector then ironic, then inspector would break because the ironic jobs aren't loaded yet09:29
vdrokgood morning ironic!09:29
*** skazi has quit IRC09:29
sambettspas-ha: definatly should be updated to use the new names though, and load them from ironic, it gives us the best control then09:30
dtantsurugh09:32
sambetts?09:33
*** e0ne has quit IRC09:33
sambettspas-ha: we need to make sure that ironic's jobs are loaded before neutron/devstack etc too to make sure they can use our jobs09:34
pas-haAfaik they are loaded in alphanum order09:35
sambettspas-ha: they are be default, but in special cases we're allowed to move projects that need to be loaded early09:36
sambettsfor example openstack-zuul-jobs is loaded out of order09:36
*** e0ne has joined #openstack-ironic09:43
*** yolanda has quit IRC09:51
*** skazi has joined #openstack-ironic09:51
*** hw_wutianwei has quit IRC09:55
*** zshi__ has joined #openstack-ironic09:56
*** _milan_ has quit IRC09:57
*** TuanLA has quit IRC10:02
*** namnh has quit IRC10:08
*** namnh has joined #openstack-ironic10:09
*** marios has quit IRC10:12
*** marios has joined #openstack-ironic10:12
*** lihi has quit IRC10:20
*** lihi has joined #openstack-ironic10:20
*** rbartal has joined #openstack-ironic10:32
johnthetubaguyso got great spec comments from rloo, but I am wondering if traits should get updated via the node PATCH interface?10:35
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Allow re-negotiation of the latest version supplied by CLI  https://review.openstack.org/51298610:37
vdrokjohnthetubaguy: in the tags spec, we had a separate endpoint for updating tags, and doing it through node was forbidden10:40
vdrokthe spec is not yet implemented tho :)10:40
*** namnh has quit IRC10:40
vdrokbut is merged10:40
johnthetubaguyvdrok: yeah, that's what I went for too, but looking at the CLI, its a bit funky10:40
johnthetubaguyI guess I could go either way, just going to add a note in the alternatives section, and note the CLI limitations10:41
vdrokjohnthetubaguy: you mean that the command would look like 'node tags set'? seems not that bad10:42
dtantsurjohnthetubaguy: I'd prefer one way of updating things10:42
johnthetubaguydtantsur: you mean reusing the patch API?10:42
johnthetubaguywell, extending the patch API10:42
dtantsurjohnthetubaguy: if we don't create separate CRUD for traits - then yes10:43
dtantsurbut I think you're planning on new API, so I'd prefer we don't update things via node then10:43
vdrok++10:43
johnthetubaguycurrently its separate CRUD10:43
johnthetubaguyyeah, totally should not happen both ways, that would be nuts10:43
johnthetubaguyjust on the CLI we could go for:10:43
johnthetubaguyopenstack baremetal node set --trait CUSTOM_FOO <node-uuid>10:44
johnthetubaguyor10:44
johnthetubaguyopenstack baremetal node trait add <node-uuid> "CUSTOM_FOO"10:44
johnthetubaguyor10:44
johnthetubaguyopenstack baremetal node add trait <node-uuid> "CUSTOM_FOO"10:44
*** _milan_ has joined #openstack-ironic10:44
vdrokI'd vote for #2 :)10:44
johnthetubaguyso #2 was plan A, its just #1 is more consistent with what other APIs have10:45
johnthetubaguyI think the CLI should probably follow the API though10:45
_milan_morning Ironic! :)11:00
_milan_morning dtantsur johnthetubaguy pas-ha sambetts vdrok and all!11:01
vdrokgood morning _milan_ :)11:01
* johnthetubaguy waves11:01
*** yangyape_ has quit IRC11:01
dtantsurjohnthetubaguy: I vote for #2 as well, and #3 is quite anti-OSC11:02
dtantsurhey _milan_11:02
*** phuongnh has quit IRC11:02
johnthetubaguydtantsur: I wasn't sure where OSC had landed on the verb / object order, I certainly prefer #2, so that is all good :)11:02
dtantsurjohnthetubaguy: I think it's "object [subobject ..] verb"11:03
johnthetubaguydtantsur: perfect11:03
_milan_so what's up today? :)11:03
johnthetubaguyas you can tell, Nova hasn't been a great OSC citizen so far11:03
openstackgerritJohn Garbutt proposed openstack/ironic-specs master: Traits on Ironic Nodes  https://review.openstack.org/50453111:06
johnthetubaguyI was mostly chatting about the above spec: ^11:06
*** zhangfei has quit IRC11:07
johnthetubaguyall this spec work has made me hungry though, I should go fix that...11:07
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Set the default API version of OSC CLI to "latest"  https://review.openstack.org/51298911:08
johnthetubaguydtantsur: in that patch above, are you requesting "latest" from the API? That sounds incorrect11:11
dtantsurjohnthetubaguy: nope11:12
*** zshi__ has quit IRC11:12
*** amarao has joined #openstack-ironic11:13
johnthetubaguydtantsur: so the Nova CLI requests the highest version it knows it works with and the API supports, to avoid the case where it gets pointed a cloud that is too new and breaks horribly11:13
johnthetubaguy(like when we remove the ability to set something in the API)11:13
dtantsurjohnthetubaguy: that's precisely what these changes do11:14
dtantsur:)11:14
johnthetubaguyah, I just read more carefully "latest" doesn't mean version: latest11:14
johnthetubaguylooks good then :)11:15
*** parlos has quit IRC11:15
dtantsuryep, it's confusing, but OSC --os-baremetal-api-version=latest never passes "latest" to the API11:15
johnthetubaguyyeah, there is no better word to use there, without going to crazy land11:16
*** vmud213 has quit IRC11:20
*** fellypefca has joined #openstack-ironic11:20
*** lucasagomes is now known as lucas-hungry11:22
*** AlexeyAbashkin has quit IRC11:24
*** AlexeyAbashkin has joined #openstack-ironic11:26
openstackgerritJohn Garbutt proposed openstack/ironic-specs master: Config Template traits  https://review.openstack.org/50495211:28
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Set the default API version of OSC CLI to "latest"  https://review.openstack.org/51298911:29
dtantsurthis seems to work ^^^ proved harder than I expected..11:35
*** parlos has joined #openstack-ironic11:43
dtantsursambetts: could you please have another review pass on https://review.openstack.org/#/c/511240/ ?11:43
patchbotpatch 511240 - ironic - Finish the guide on upgrading to hardware types11:43
sambettsdtantsur: sure :)11:46
*** bfournie has quit IRC11:47
*** bfournie has joined #openstack-ironic11:48
*** AlexeyAbashkin has quit IRC11:51
*** bfournie has quit IRC11:52
*** AlexeyAbashkin has joined #openstack-ironic11:55
openstackgerritVladyslav Drok proposed openstack/ironic master: Revise deploy process documentation  https://review.openstack.org/45886711:57
openstackgerritVladyslav Drok proposed openstack/ironic master: Workflow of iPXE usage in Deploy Process  https://review.openstack.org/45537711:58
*** rcernin has joined #openstack-ironic11:58
fellypefcaMorning :)11:59
vdrokgood morning fellypefca11:59
fellypefcavdrok :D12:00
*** litao__ has quit IRC12:10
*** ktibi has quit IRC12:14
*** dprince has joined #openstack-ironic12:14
*** jlabarre has joined #openstack-ironic12:16
*** vsaienk0 has quit IRC12:21
*** jtomasek has quit IRC12:22
*** jtomasek has joined #openstack-ironic12:22
*** bfournie has joined #openstack-ironic12:25
*** trown|outtypewww is now known as trown12:26
*** dtantsur is now known as dtantsur|brb12:26
*** catintheroof has joined #openstack-ironic12:27
*** yangyapeng has joined #openstack-ironic12:29
*** e0ne_ has joined #openstack-ironic12:34
*** e0ne has quit IRC12:37
*** vsaienk0 has joined #openstack-ironic12:41
*** yolanda has joined #openstack-ironic12:41
*** catintheroof has quit IRC12:42
*** lucas-hungry is now known as lucasagomes12:43
*** jcoufal has joined #openstack-ironic12:46
*** rbudden has joined #openstack-ironic12:50
*** catintheroof has joined #openstack-ironic12:51
*** catintheroof has quit IRC12:53
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector master: Add zuul3 jobs in-tree  https://review.openstack.org/51329413:08
*** rloo has joined #openstack-ironic13:08
*** baoli has joined #openstack-ironic13:11
*** baoli has quit IRC13:12
*** yolanda has quit IRC13:13
*** mjturek has joined #openstack-ironic13:14
openstackgerritVasyl Saienko proposed openstack/ironic-python-agent master: WIP: Move legacy jobs into tree  https://review.openstack.org/51338013:16
*** crushil has joined #openstack-ironic13:18
*** baoli has joined #openstack-ironic13:19
*** rcernin has quit IRC13:21
*** rpioso has joined #openstack-ironic13:24
openstackgerritVasyl Saienko proposed openstack/ironic-python-agent master: WIP: Move legacy jobs into tree  https://review.openstack.org/51338013:28
openstackgerritVasyl Saienko proposed openstack/ironic-python-agent master: WIP: Move legacy jobs into tree  https://review.openstack.org/51338013:30
*** rcernin has joined #openstack-ironic13:35
*** rcernin has quit IRC13:40
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector master: Add zuul3 jobs in-tree  https://review.openstack.org/51329413:42
*** zshi__ has joined #openstack-ironic13:42
*** jrist_ has joined #openstack-ironic13:43
*** jrist has quit IRC13:43
*** MattMan has quit IRC13:44
*** MattMan has joined #openstack-ironic13:44
*** hshiina has joined #openstack-ironic13:59
*** zshi__ has quit IRC14:00
*** zshi__ has joined #openstack-ironic14:01
TheJuliadtantsur|brb: I'm in no shape to run a bfv meeting this week. Hopefully at the end of this cold.14:04
rloohowdy and good morning all, dtantsur|brb, vsaienk0, TheJulia14:07
rlooTheJulia: good bye, go back to bed and get better soon!14:07
TheJuliarloo: already there14:07
rlooTheJulia: sweet dreams :)14:08
*** baoli has quit IRC14:09
*** baoli has joined #openstack-ironic14:09
*** baoli has quit IRC14:09
openstackgerritRuby Loo proposed openstack/ironic master: Finish the guide on upgrading to hardware types  https://review.openstack.org/51124014:10
*** baoli has joined #openstack-ironic14:11
vdrokgood morning rloo !14:17
rloovdrok: hi!14:17
*** rbartal has quit IRC14:17
*** hshiina has left #openstack-ironic14:18
_milan_morning rloo, TheJulia and all! :)14:19
rloovdrok: wrt https://review.openstack.org/#/c/458867/, did you go through all the steps in the examples to make sure they were correct? or did you update the parts where you knew it was wrong?14:19
patchbotpatch 458867 - ironic - Revise deploy process documentation14:19
rloohi _milan_14:19
_milan_TheJulia, I've got a question for you here: https://review.openstack.org/#/c/471831/32/ironic_inspector/conf.py14:19
patchbotpatch 471831 - ironic-inspector - Refactoring the firewall14:19
_milan_TheJulia, thing is I'm removing the manage_firewall config option and dtantsur|brb was worried about breaking bifrost14:20
_milan_TheJulia, wdyt about it?14:20
vdrokrloo: I was trying to make sure that the steps in agent_ diagram are correct (was reviewing galyna's patch for quite a while), with pxe_ I just copy-pasted the stuff excluding the actual provision steps, as these should be the same. I can check once again14:21
* _milan_ most likely going to give a follow-up patch to handle the config option in a custom way 14:21
vdroks/these/they14:21
TheJulia_milan_: we disable the firewall it because we don't explicitly want iptable changes really occurring and the chian loading that we have in bifrost is such that we end up falling back to inspection if the conductor didn't perform the deploy.14:22
_milan_TheJulia, the change would introduce a different config option [pxe_filter]driver=noop ; would that work for you? Or should we have some transition period with accepting the manage_firewall==False as driver==noop?14:23
rloovdrok: thx, i commented, cuz i am not convinced the steps are in the right order in example 1.14:24
TheJulia_milan_: realistically users need a graceful transition period anyway through deprecation14:24
_milan_makes sense14:25
TheJulia_milan_: I'm also okay if you treat it as breaking behavior change, we can always update the config file that is generated14:25
TheJuliaso kind of a pick you poison scenario :)14:25
_milan_lol14:26
_milan_aarefiev, hey, wdyt about all this ^14:27
TheJulia_milan_: I'm going to go back to resting now14:27
_milan_TheJulia, sure, thanks!14:27
*** mat128 has joined #openstack-ironic14:28
*** trown is now known as trown|brb14:38
*** spectr has quit IRC14:38
*** zshi__ has quit IRC14:42
*** trown|brb is now known as trown14:46
*** e0ne_ has quit IRC14:56
jlvillalsambetts, I think you had suggested this before: https://review.openstack.org/51290014:57
patchbotpatch 512900 - ironic-python-agent - tox: Use the default version of Python 3 for tox t...14:57
*** parlos has quit IRC14:57
*** e0ne has joined #openstack-ironic15:00
*** mjura has quit IRC15:02
*** amarao has quit IRC15:03
*** jrist_ is now known as jrist15:03
*** jrist has quit IRC15:03
*** jrist has joined #openstack-ironic15:03
*** parlos has joined #openstack-ironic15:07
*** baoli has quit IRC15:07
*** baoli has joined #openstack-ironic15:08
*** baoli has quit IRC15:08
*** dtantsur|brb is now known as dtantsur15:08
dtantsurmorning TheJulia, get better! should I run it or let's just skip?15:09
dtantsurmorning rloo, jlvillal15:09
*** baoli has joined #openstack-ironic15:11
dtantsurrloo: thanks for updating the docs patch!15:11
jlvillalHi dtantsur :)15:11
* jlvillal goes to prepare breakfast15:11
rloodtantsur: you're welcome. glad that it is done! :)15:12
rloojlvillal: good morning!15:12
dtantsuryeah, /me too15:12
dtantsurnow I need to transition tripleo to hardware types, which is fun (no, it's not)15:12
rloodtantsur: nice that you're dogfooding (https://newrepublic.com/article/115349/dogfooding-tech-slang-working-out-glitches)15:13
dtantsurheh, yeah15:14
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Refactoring the firewall  https://review.openstack.org/47183115:15
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver  https://review.openstack.org/46644815:15
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Allow concurrect updating of dnsmasq configuration  https://review.openstack.org/50443815:15
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack  https://review.openstack.org/46871215:15
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Support manage_firewall during deprecation period  https://review.openstack.org/51341515:15
*** penick has joined #openstack-ironic15:15
openstackgerritVasyl Saienko proposed openstack/ironic-python-agent master: Move legacy jobs into tree  https://review.openstack.org/51338015:16
_milan_dtantsur, w/r the deprecation of manage_firewall; is this the right direction: https://review.openstack.org/#/c/513415/1/ironic_inspector/test/unit/test_pxe_filter.py15:18
patchbotpatch 513415 - ironic-inspector - Support manage_firewall during deprecation period15:18
dtantsur_milan_: yeah, seems okay at first glance15:21
TheJuliadtantsur: I'm thinking skip because there are likely to be minimal status updates.15:21
_milan_dtantsur, ack, thx15:21
dtantsurTheJulia: okie. I'll show up on the meeting channel to announce that the meeting is canceled. please go back to bed :)15:22
*** penick has quit IRC15:23
dtantsurrloo, vdrok, could you please review https://review.openstack.org/#/c/512986/ ?15:25
patchbotpatch 512986 - python-ironicclient - Allow re-negotiation of the latest version supplie...15:25
dtantsurI know it's a bit confusing, but that's an important step towards switching to "latest" :)15:25
* dtantsur fixes vdrok's comments on the next patch15:25
vdrokcurrently in a meeting, but will try to do today15:25
rloodtantsur: oh, did you need it this week? i'm reviewing the traits spec and was hoping not to do any difficult reviewing after that :)15:25
* rloo wonders what's difficult about coming up with APIs... :-(15:26
*** penick has joined #openstack-ironic15:26
dtantsurheh15:26
dtantsurrloo: no big pressure. I'd like an ironicclient release next week though15:26
rloodtantsur: ah. ok, i'll see how i feel :)15:26
openstackgerritVasyl Saienko proposed openstack/ironic-python-agent master: Move legacy jobs into tree  https://review.openstack.org/51338015:27
dtantsuraarefiev: hey, up for quick review of https://review.openstack.org/#/c/513415/ ? ideally, it should merge soon after the firewall patch15:31
patchbotpatch 513415 - ironic-inspector - Support manage_firewall during deprecation period15:31
_milan_dtantsur, wow, iptables refactor merging <315:31
dtantsur:)15:31
*** crushil has quit IRC15:32
*** mjturek has quit IRC15:34
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Set the default API version of OSC CLI to "latest"  https://review.openstack.org/51298915:36
sambettsdtantsur: do you remember how to use the openstack client no auth?? I can't remmeber the flag required15:39
NobodyCamGood Morning Ironicer's15:39
*** parlos has quit IRC15:41
_milan_morning NobodyCam! :)15:42
*** vsaienk0 has quit IRC15:42
NobodyCamGood Morning _milan_ :)15:42
*** rloo has quit IRC15:42
*** rloo has joined #openstack-ironic15:43
*** parlos has joined #openstack-ironic15:43
rloomornin' NobodyCam15:44
NobodyCamGood Morning rloo :)15:44
rloosambetts: I thought i had documented it cuz i can never remember, let me see if i can find...15:44
rlooexport OS_TOKEN=fake-token15:45
rlooexport OS_URL=http://localhost:6385/15:45
rloosambetts: does that work? ^^15:45
rloosambetts: from https://docs.openstack.org/ironic/latest/contributor/dev-quickstart.html15:45
dtantsursambetts: sorry, I think this ^^ is right15:48
dtantsurmorning NobodyCam15:48
NobodyCamGood Morning dtantsur ... it's almost that day :)15:49
sambettsrloo: forwarding on to my co-worker, hopfully it works, I thought we documented it in the python-ironicclient docs but I couldn't find it15:50
rloosambetts: oh, that's a good point; we should document it there. i recently looked at those docs and don't recall seeing it there.15:51
rloosambetts: well, maybe we don't want to document it in user-facing docs vs for developers only. not sure.15:52
sambettsrloo: the use case is for bifrost15:52
sambettsrloo: seeing as ironic client is going away15:53
sambettswe need to be able to use openstack baremetal for standalone15:53
rloosambetts: oh. yes. hmm.15:53
sambettsnot just for dev15:53
dtantsurright15:53
dtantsurhell, somebody should go and finally implement --auth-type none..15:54
sambettsI think we need something here: https://docs.openstack.org/python-ironicclient/latest/cli/osc_plugin_cli.html15:54
rloosambetts, dtantsur: ok, let's update the client docs.15:54
rloosambetts: yeah, there.15:54
rloosambetts: i don't even know how accurate that doc is, with existing environ vars :)15:55
rloosambetts: you want to do it, or shall i? (if me, later today maybe)15:55
sambettsrloo: my co-worker just tried that out and it didn't work, so I don't know how to do it (so I can't write it)15:56
rloosambetts: damn.15:56
rloosambetts: ok, i'll look into it. does your co-worker need it now?15:56
sambettsit would be useful, because inspector client is only openstack client, and we need to talk to the inspector in a bifrost deploy to abort an inspection15:57
vdrokdtantsur: see ML post :)15:58
sambettsbut we can wait it out for timeout15:58
dtantsurvdrok: still unread in my inbox, yeah :)15:58
dtantsurETOOMANYTHINGSTODO15:58
rloosambetts: ok, let me try/test it out...15:59
sambettsrloo: thanks a ton!16:00
*** trown is now known as trown|lunch16:00
*** marios has quit IRC16:05
*** rpioso is now known as rpioso|afk16:05
*** ianychoi has quit IRC16:06
rloosambetts: so that ^^ worked for me. what versions of ironic/client are you using?16:07
*** frontrunner has joined #openstack-ironic16:07
*** ianychoi has joined #openstack-ironic16:07
rloosambetts: and in config, 'auth_strategy = noauth'?16:07
sambettsrloo: do you have no keystone running?16:09
rloosambetts: oh, wait, i am testing in an env that does have keystone. sec.16:09
*** lucasagomes is now known as lucas-afk16:09
*** rama_y has joined #openstack-ironic16:13
rloosambetts: it still works for me. i killed all the services, only have ironic api & conductor running.16:13
sambettshuh ok... :/ I'll see if he can give me an error message or something16:14
pas-haFYI, made a small etherpad to track zuul-jobs-in-tree migration effort https://etherpad.openstack.org/p/ironic-zuulv3-intree-tracking so if you are working on that please update it with status/patches etc16:16
*** penick has quit IRC16:17
*** pcaruana has quit IRC16:18
* _milan_ relocates16:19
* _milan_ should check pas-ha 's patches ;)16:19
*** e0ne has quit IRC16:19
*** _milan_ has quit IRC16:21
*** penick has joined #openstack-ironic16:21
*** rpioso|afk has quit IRC16:23
*** crushil has joined #openstack-ironic16:30
*** penick_ has joined #openstack-ironic16:30
*** ralonsoh has quit IRC16:33
rloopas-ha: thx for the etherpad! wow, there's still quite a bit to do.16:33
*** penick has quit IRC16:33
sambettsrloo: we have a lot of sub-projects apprently :-P16:34
dtantsurheh16:35
* sambetts gazing off into the distances, remembers the days when ironic was just one/two repos16:35
sambetts:-P16:36
dtantsurno IPA, no inspector, yeah16:36
dtantsurfun old times of the bash ramdisk and barely working deployment :D16:36
sambetts:D16:38
*** crushil has quit IRC16:38
* dtantsur joined around Icehouse16:38
*** bnemec has quit IRC16:45
*** bnemec has joined #openstack-ironic16:46
*** AlexeyAbashkin has quit IRC16:46
*** mgoddard_ has quit IRC16:47
sambettsI started in OpenStack around grizly, it at the summit in hong kong where I first heard about ironic16:50
sambettswhich was just after the havana release I think16:50
*** ijw has joined #openstack-ironic16:54
rlooHK was the start of the icehouse cycle, so sambetts, yeah.16:55
*** srart has quit IRC16:55
*** dsneddon has joined #openstack-ironic16:58
* dtantsur starts moving towards beers, see you tomorrow16:59
*** ijw has quit IRC17:00
*** ccamacho has quit IRC17:00
*** derekh has quit IRC17:00
*** trown|lunch is now known as trown17:00
sambettsrloo: ah that makes sense then17:00
*** ijw has joined #openstack-ironic17:02
rloosambetts: i was trying to resist, but i took a look at icehouse release notes. that was the first release of ironic, if i recall, they were trying to get it be accepted as an openstack project (before big tent thing happened).17:02
rloosambetts: ah, the good old daze :)17:03
sambettshaha yeah :-P17:03
* sambetts is also heading toward beers 17:03
sambettsnight all o/17:04
*** dtantsur is now known as dtantsur|afk17:04
dtantsur|afknight sambetts17:04
*** sambetts is now known as sambetts|afk17:04
sambetts|afko/ dtantsur|afk17:04
*** parlos has quit IRC17:04
*** tesseract has quit IRC17:04
openstackgerritMathieu Gagné proposed openstack/ironic master: Add ability to provide configdrive when rebuilding  https://review.openstack.org/50176117:07
*** srart has joined #openstack-ironic17:10
*** rpioso|afk has joined #openstack-ironic17:10
*** jralph has quit IRC17:12
*** jralph has joined #openstack-ironic17:17
*** fragatina has quit IRC17:23
*** dprince has quit IRC17:25
radezTheJulia: ping, hey are you around?17:29
radezI found that error message again and I'm having trouble understanding why I'm getting it17:29
radezhttps://paste.fedoraproject.org/paste/7~0LhJTxWioH3IuSgoA-MQ17:29
*** mgoddard_ has joined #openstack-ironic17:36
openstackgerritRuby Loo proposed openstack/ironic master: [WIP] Copy port[group] VIF info from extra to internal_info  https://review.openstack.org/51163617:38
*** csmart has quit IRC17:43
*** rama_y has quit IRC17:43
*** rama_y has joined #openstack-ironic17:44
*** mkrai has quit IRC17:44
*** kzaitsev_pi has quit IRC17:44
*** retr0h has quit IRC17:45
*** kzaitsev_pi has joined #openstack-ironic17:45
*** dprince has joined #openstack-ironic17:49
rlooradez: Julia is sick, I don't know when she'll reply to you17:54
*** baoli has quit IRC17:58
openstackgerritMerged openstack/ironic master: [bfv] Set the correct iqn for pxe  https://review.openstack.org/51266218:02
openstackgerritMerged openstack/ironic master: Finish the guide on upgrading to hardware types  https://review.openstack.org/51124018:02
*** baoli has joined #openstack-ironic18:07
*** rpioso|afk is now known as rpioso18:18
*** AlexeyAbashkin has joined #openstack-ironic18:19
*** ChubYann has joined #openstack-ironic18:19
*** AlexeyAbashkin has quit IRC18:23
*** Nisha_Agarwal has joined #openstack-ironic18:28
*** baoli has quit IRC18:37
*** Nisha_Agarwal has quit IRC18:39
*** baoli has joined #openstack-ironic18:48
*** jcoufal has quit IRC18:48
*** e0ne has joined #openstack-ironic18:51
openstackgerritVladyslav Drok proposed openstack/ironic master: Revise deploy process documentation  https://review.openstack.org/45886718:53
vdrokrloo: updated18:54
vdrokgood night everyone!18:54
rloovdrok: thx and have a great evening!18:54
NobodyCamnight vdrok18:55
*** fellypefca has quit IRC19:02
*** e0ne has quit IRC19:11
*** AlexeyAbashkin has joined #openstack-ironic19:17
mrtenioHi rloo are you still available? I have a question about https://review.openstack.org/#/c/510644/19:19
patchbotpatch 510644 - ironic - Get a new OneView client when needed19:19
*** AlexeyAbashkin has quit IRC19:21
*** ijw has quit IRC19:29
rloomrtenio: what's the question?19:31
mrtenioI am answering you in the patch, sorry.19:31
ricardoashey rloo mrtenio - do we really need to open a bug on launchpad for a thing introduced in master code and not released on previous version? shouldn´t it be a rfe? - talking specifically about https://bugs.launchpad.net/ironic/+bug/172226819:34
openstackLaunchpad bug 1722268 in Ironic "OneView client sessions expires after 24 hours" [High,Invalid] - Assigned to Stenio Araujo (stenioaraujo)19:34
rlooricardoas, mrtenio: as far as i am concerned, it isn't a bug because it doesn't happen using master code19:35
ricardoasrloo: mrtenio: this does not affects stable/pike because the client change is still in progress (from python-openstackclient to hpOneView)19:35
rlooricardoas, mrtenio: I think you can figure it out :)19:36
ricardoasrloo: cool! should we change it to a rfe? I don´t know the protocol for this...19:36
rlooricardoas: as far as i am concerned, it is somewhat like 'code cleanup'.19:37
rlooricardoas: however, how are you tracking this switch to a new client. is there an rfe for that?19:37
mrteniorloo, ricardoas: Would it be a good approach to link this patch to the bug https://bugs.launchpad.net/ironic/+bug/1693788?19:37
openstackLaunchpad bug 1693788 in Ironic "Switch the OneView driver to hpOneView and ilorest libraries" [Medium,In progress] - Assigned to Hugo Nicodemos (hugonicodemos)19:37
rloomrtenio, ricardoas: yes, the 24-hr fix should be part of ^^19:38
rloomrtenio, ricardoas: I think i alluded to that, by rebasing/reordering your patches to address that ^^. which seems to be an rfe but whatever.19:39
ricardoasrloo: i think it is... we´ll change that then19:39
rloomrtenio, ricardoas: you probably don't want me to check that bug, cuz if it is an rfe and it hasn't been triaged as such...19:39
rlooi'd block you and you probably don't want that... so i'm going to just ignore that i mentioned any of this.19:39
rloodarn, i can't just ignore it. gonna look...19:40
ricardoasrloo: not sure who did it... but we were suggested to open this as a bug to trace this migration19:40
rlooricardoas: what's the reason for making the switch. it isn't mentioned in the bug.19:41
ricardoasrloo: just to track the status as no new functionality is being introduced...19:41
rlooricardoas: so why make the switch?19:41
rlooricardoas: it is a change as far as the user is concerned.19:41
ricardoasrloo: the new client is being actively maintained by hpe folks and provides a lot more features than the old one19:42
ricardoasrloo: features we plan to leverage in a new future19:42
ricardoasrloo: near*19:43
ricardoasrloo: it is already being used in our ml2 plugin for neutron for example...19:43
rlooricardoas: i marked it as an rfe. please update the description to mention why you are making the change. (^^ that stuff)19:44
ricardoasrloo: I guess python-oneviewclient was a fast line solution while this official client was being developed... not sure as I was not in the team19:44
rlooricardoas: after updating the desc, i'm ok approving it but i can't remember if we need two people19:44
ricardoasrloo: ok, I´ll update it! Thank you very much!19:45
ricardoasrloo: I guess we also need to put on the meeting agenda... right?19:46
rlooricardoas: so, looking at the bug, dtantsur|afk triaged it, so i hope i can assume that he's good with it.19:47
rlooricardoas: so after you update the description, ping me, if i'm happy with it, i'll approve it.19:47
*** dprince has quit IRC19:49
ricardoasrloo: sure! thanks again...19:49
mrteniorloo, ricardoas: thank you.19:49
mrteniorloo, I am sorry for the are you available type of question. I only figured it when I sent it :P19:50
*** ijw has joined #openstack-ironic19:52
rloomrtenio: no worries.19:54
*** ijw has quit IRC19:57
*** penick_ is now known as penick20:02
*** AlexeyAbashkin has joined #openstack-ironic20:17
*** AlexeyAbashkin has quit IRC20:21
*** ijw has joined #openstack-ironic20:23
*** ijw has quit IRC20:27
*** AlexeyAbashkin has joined #openstack-ironic20:38
*** AlexeyAbashkin has quit IRC20:42
*** mat128 has quit IRC20:48
*** ijw has joined #openstack-ironic20:56
*** AlexeyAbashkin has joined #openstack-ironic20:59
*** trown is now known as trown|outtypewww21:02
*** mgoddard_ has quit IRC21:03
*** AlexeyAbashkin has quit IRC21:03
openstackgerritMerged openstack/ironic-python-agent master: Release notes clean up for the upcoming release  https://review.openstack.org/51262421:16
*** Goneri has quit IRC21:17
openstackgerritDerek Higgins proposed openstack/ironic stable/pike: [bfv] Set the correct iqn for pxe  https://review.openstack.org/51351621:17
*** Goneri has joined #openstack-ironic21:23
*** penick has quit IRC21:30
*** aimeeu has left #openstack-ironic21:31
*** jlabarre has quit IRC21:36
*** baoli has quit IRC21:40
openstackgerritMerged openstack/ironic-inspector master: Add request context and policy enforcement  https://review.openstack.org/50782621:40
openstackgerritMerged openstack/ironic-inspector master: Refactoring the firewall  https://review.openstack.org/47183121:40
openstackgerritMatt Keenan proposed openstack/python-ironicclient master: Synchronize ironic and ironicclients list of boot devices  https://review.openstack.org/51352821:41
*** penick has joined #openstack-ironic21:41
*** penick has quit IRC21:51
*** bfournie has quit IRC21:54
*** mgoddard_ has joined #openstack-ironic21:59
*** csmart has joined #openstack-ironic22:02
*** jrist has quit IRC22:03
*** rwsu has joined #openstack-ironic22:08
*** frontrunner has quit IRC22:16
*** mgoddard_ has quit IRC22:19
*** jrist has joined #openstack-ironic22:19
*** fragatina has joined #openstack-ironic22:23
*** hjensas has joined #openstack-ironic22:35
*** hjensas has joined #openstack-ironic22:35
*** mkrai has joined #openstack-ironic22:38
*** retr0h has joined #openstack-ironic22:38
*** ijw has quit IRC22:58
*** baoli has joined #openstack-ironic23:06
*** ijw has joined #openstack-ironic23:12
*** baoli has quit IRC23:12
*** ijw has quit IRC23:13
*** ijw has joined #openstack-ironic23:13
*** AlexeyAbashkin has joined #openstack-ironic23:16
*** AlexeyAbashkin has quit IRC23:21
*** rpioso has quit IRC23:34
*** AlexeyAbashkin has joined #openstack-ironic23:58

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