Monday, 2016-11-07

*** e0ne_ has joined #openstack-ironic00:05
*** e0ne has quit IRC00:05
*** e0ne_ has quit IRC00:28
*** jkilpatr has quit IRC00:36
*** rbudden has joined #openstack-ironic00:46
*** tuanluong has joined #openstack-ironic00:56
*** causten has quit IRC01:04
*** causten has joined #openstack-ironic01:05
*** causten_ has joined #openstack-ironic01:09
*** causten has quit IRC01:09
*** hoangcx has joined #openstack-ironic01:18
*** amotoki has joined #openstack-ironic01:25
*** hoangcx has quit IRC01:32
*** amotoki has quit IRC01:34
*** hoangcx has joined #openstack-ironic01:34
*** amotoki has joined #openstack-ironic01:35
*** amotoki has quit IRC01:47
*** amotoki has joined #openstack-ironic01:51
*** amotoki has quit IRC01:53
*** amotoki has joined #openstack-ironic01:55
*** amotoki has quit IRC02:03
*** sdake has quit IRC02:08
*** wlfightup has joined #openstack-ironic02:10
*** sdake has joined #openstack-ironic02:12
wlfightuphello02:20
*** shuquan has joined #openstack-ironic02:22
*** shobhar_ has joined #openstack-ironic02:23
*** amotoki has joined #openstack-ironic02:23
*** baoli has quit IRC02:25
*** shobhar_ has quit IRC02:25
*** shobhar_ has joined #openstack-ironic02:25
*** hw_wutianwei has joined #openstack-ironic02:26
*** baoli has joined #openstack-ironic02:26
*** baoli has quit IRC02:28
*** baoli has joined #openstack-ironic02:28
*** causten_ has quit IRC02:33
*** shobhar__ has joined #openstack-ironic02:33
*** baoli has quit IRC02:37
*** shobhar_ has quit IRC02:37
*** baoli has joined #openstack-ironic02:39
*** sabeen has joined #openstack-ironic02:39
*** zhangjl has joined #openstack-ironic02:52
*** shobhar__ has quit IRC02:52
*** shobhar__ has joined #openstack-ironic02:52
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off  https://review.openstack.org/21673002:54
*** shobhar__ has quit IRC02:55
*** baoli has quit IRC03:01
*** shuquan__ has joined #openstack-ironic03:06
*** shuquan has quit IRC03:07
*** gcb has joined #openstack-ironic03:18
*** ElCoyote_ has quit IRC03:21
*** amotoki has quit IRC03:30
*** Nisha has joined #openstack-ironic03:34
*** Nisha is now known as Guest9171003:34
*** amotoki has joined #openstack-ironic03:40
*** Guest91710 is now known as Nisha103:48
*** links has joined #openstack-ironic03:55
*** bks has joined #openstack-ironic04:17
*** sdake has quit IRC04:21
*** aparnav has joined #openstack-ironic04:22
*** baoli has joined #openstack-ironic04:25
*** vmud213 has joined #openstack-ironic04:28
*** baoli has quit IRC04:30
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off  https://review.openstack.org/21673804:34
*** bks has quit IRC04:38
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off  https://review.openstack.org/21674304:42
*** chlong has joined #openstack-ironic04:47
*** wlfightup has quit IRC04:56
*** rbrud has quit IRC05:04
openstackgerritNaohiro Tamura proposed openstack/ironic: Update the existing APIs due to adding get_supported_power_states  https://review.openstack.org/38219405:17
*** hshiina has joined #openstack-ironic05:17
*** wlfightup has joined #openstack-ironic05:18
hshiinamorning ironic05:22
*** shuquan__ has quit IRC05:47
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI  https://review.openstack.org/34819105:47
*** shuquan has joined #openstack-ironic05:49
*** aparnav has quit IRC05:56
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI  https://review.openstack.org/34872105:57
*** xuao has joined #openstack-ironic06:02
*** rcernin has joined #openstack-ironic06:03
*** amotoki has quit IRC06:03
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC management driver for Inject NMI  https://review.openstack.org/34873206:06
*** xuao has quit IRC06:06
*** sabeen has quit IRC06:11
openstackgerritNguyen Phuong An proposed openstack/ironic: Fix typo  https://review.openstack.org/37726706:34
*** pmannidi_ has quit IRC06:34
*** vmud213 has quit IRC06:37
*** aparnav has joined #openstack-ironic06:41
*** bfournie1 has quit IRC06:41
*** pmannidi_ has joined #openstack-ironic06:50
openstackgerritNguyen Phuong An proposed openstack/ironic: Fix typo of 'authenticaiton'  https://review.openstack.org/37726706:50
*** vmud213 has joined #openstack-ironic06:52
*** amotoki has joined #openstack-ironic06:53
*** rbudden has quit IRC06:58
*** shuquan__ has joined #openstack-ironic07:04
*** shuquan has quit IRC07:04
*** tesseract has joined #openstack-ironic07:06
*** tesseract is now known as Guest7131007:06
*** amotoki has quit IRC07:08
*** ChubYann has quit IRC07:16
*** pcaruana has joined #openstack-ironic07:17
*** pmannidi_ has quit IRC07:18
*** Nisha1 has quit IRC07:21
*** amotoki has joined #openstack-ironic07:21
*** Nisha1 has joined #openstack-ironic07:21
*** amotoki has quit IRC07:22
*** amotoki has joined #openstack-ironic07:22
pas-hamorning Ironic :)07:23
tuanluongmorning ironic07:25
tuanluongmorning pas-ha07:25
*** rbartal has joined #openstack-ironic07:26
*** Nisha has joined #openstack-ironic07:31
*** Nisha1 has quit IRC07:31
*** Nisha is now known as Guest6128507:32
*** Guest61285 has quit IRC07:37
*** khamtamtun has joined #openstack-ironic07:41
*** khamtamtun has quit IRC07:43
*** Guest61285 has joined #openstack-ironic07:46
*** moshele has joined #openstack-ironic07:51
pas-hamorning tuanluong :)07:52
*** athomas has joined #openstack-ironic07:53
*** aNuposic has joined #openstack-ironic07:55
*** ccamacho has joined #openstack-ironic08:10
*** aparnav has quit IRC08:17
*** baoli has joined #openstack-ironic08:18
*** aparnav has joined #openstack-ironic08:20
*** baoli has quit IRC08:23
*** shuquan_1 has joined #openstack-ironic08:24
*** shuquan__ has quit IRC08:24
*** vmud213 has quit IRC08:28
*** vmud213 has joined #openstack-ironic08:29
*** pmannidi has joined #openstack-ironic08:29
*** alex_xu has quit IRC08:30
*** snozhka has joined #openstack-ironic08:31
openstackgerritVasyl Saienko proposed openstack/ironic-specs: Update ironic-ml2-integration spec  https://review.openstack.org/39366008:33
openstackgerritVasyl Saienko proposed openstack/ironic-specs: Update ironic-ml2-integration spec  https://review.openstack.org/39366008:33
*** betherly has joined #openstack-ironic08:35
*** Guest61285 has quit IRC08:36
*** alex_xu has joined #openstack-ironic08:37
*** chlong has quit IRC08:41
-openstackstatus- NOTICE: Gerrit is going to be restarted due to slowness and proxy errors08:46
*** ohamada has joined #openstack-ironic08:46
*** openstackgerrit has quit IRC08:48
*** jpich has joined #openstack-ironic08:48
*** fxpester has joined #openstack-ironic08:48
*** openstackgerrit has joined #openstack-ironic08:48
*** milan has joined #openstack-ironic08:49
milanmorning Ironic! :)08:50
vmud213morning milan08:52
milanmorning vmud213! :)08:52
vmud213morning ironic08:52
*** aNuposic has quit IRC08:54
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-ironic09:00
*** betherly_ has joined #openstack-ironic09:01
*** rbartal has quit IRC09:07
*** rbartal has joined #openstack-ironic09:09
openstackgerritXu Ao proposed openstack/ironic: Syncing the power status immediately as the node is updated  https://review.openstack.org/39433209:11
*** ralonsoh has joined #openstack-ironic09:15
*** lucas-afk is now known as lucasagomes09:18
lucasagomesmorning all09:18
openstackgerritVasyl Saienko proposed openstack/ironic: DNM: Test patch to verify bond  https://review.openstack.org/38866009:19
openstackgerritVasyl Saienko proposed openstack/ironic: Ironic devstack portgroup support  https://review.openstack.org/38174309:19
openstackgerritVasyl Saienko proposed openstack/ironic: Tempest tests for portgroups  https://review.openstack.org/38247609:19
moshelelucasagomes: morning09:19
milanmorning lucasagomes, moshele! :)09:19
lucasagomeso/09:20
moshelelucasagomes: I have question regarding infiniband IPA changes  in https://review.openstack.org/#/c/334816/09:20
patchbotpatch 334816 - ironic-python-agent - InfiniBand Support for IPA09:20
moshelelucasagomes:  should it all be in our vendor repository? I wasn't able to understand how to build IPA with a vendor hardware manager09:22
*** rbartal has quit IRC09:26
*** dtantsur|afk is now known as dtantsur09:28
dtantsurMorning Ironic09:28
*** karthiks has joined #openstack-ironic09:29
dtantsurmoshele, this might help as an example: https://review.openstack.org/39183609:30
patchbotpatch 391836 - ironic-python-agent - Add a new Hardware Manager for X710 network card09:30
vmud213morning dtantsur, lucasagomes09:30
milanmorning dtantsur! :)09:30
dtantsurhey vmud213, milan, lucasagomes, pas-ha :)09:30
lucasagomesdtantsur, vmud213 hi there09:31
mosheledtantsur: thanks I will take a look09:31
lucasagomesmoshele, I don't think it should be a different repo, because we added support for infiniband in ironic itself. So let's keep the IPA part inside the IPA repository09:31
moshelelucasagomes: ok, got it09:32
*** xek has joined #openstack-ironic09:34
xekmorning!09:34
openstackgerrityolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled  https://review.openstack.org/39374509:38
*** mgoddard has joined #openstack-ironic09:39
*** betherly has quit IRC09:44
*** betherly_ is now known as betherly09:44
*** derekh has joined #openstack-ironic09:44
milanmorning xek! :)09:45
openstackgerrityolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled  https://review.openstack.org/39374509:48
*** Guest61285 has joined #openstack-ironic09:52
*** dnikishov has joined #openstack-ironic09:52
*** dnikishov is now known as Guest7607909:53
*** Guest76079 has quit IRC09:53
*** dnikisho1 has joined #openstack-ironic09:53
*** rbartal has joined #openstack-ironic09:53
*** sdake has joined #openstack-ironic09:55
openstackgerritSergii Nozhka proposed openstack/ironic-inspector: Add a test for introspection abort action verification.  https://review.openstack.org/39382409:58
*** betherly has quit IRC09:59
*** betherly has joined #openstack-ironic09:59
*** sdake_ has joined #openstack-ironic10:00
openstackgerrityolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled  https://review.openstack.org/39374510:00
*** e0ne has joined #openstack-ironic10:02
*** e0ne has quit IRC10:02
*** sdake has quit IRC10:03
*** sambetts|afk is now known as sambetts10:12
sambettsMorning all10:12
dtantsurmorning sambetts10:12
milanmorning sambetts! :)10:12
sambettshey dtantsur, milan o/10:12
lucasagomessambetts, morning10:14
milanaarefiev, dtantsur, sambetts guys, please consider reviewing https://review.openstack.org/#/c/344921/ once you've got some time, thanks!10:15
patchbotpatch 344921 - ironic-inspector - Add API for listing all introspection statuses10:15
*** hoangcx has quit IRC10:15
* milan should actually start hacking client code for this 10:16
milanand for the new GET introspection status fields, too10:16
dtantsurwill try to get to these today, thanks for patience :)10:17
*** vnogin has joined #openstack-ironic10:18
sambettshey lucasagomes10:19
openstackgerritVasyl Saienko proposed openstack/ironic: Add portgroup to api-ref  https://review.openstack.org/39383410:22
vsaienk0morning sambetts, lucasagomes, dtantsur, milan and all Ironic'ers!10:23
*** zhangjl has left #openstack-ironic10:23
dtantsurhey vsaienk010:24
*** sdake_ is now known as sdake10:25
openstackgerrityolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled  https://review.openstack.org/39374510:27
lucasagomeshi vsaienk0 !10:27
milanmorning vsaienk0! :)10:29
* milan relocates10:29
*** Guest61285 has quit IRC10:30
*** Guest61285 has joined #openstack-ironic10:31
*** milan has quit IRC10:32
*** jaosorior has joined #openstack-ironic10:38
sambettso/ vsaienk010:41
*** tuanluong has quit IRC10:42
*** bks has joined #openstack-ironic10:42
dtantsurmgould|afk (when you're here) or lucasagomes: could you please fast-forward https://review.openstack.org/#/c/394353/? I'll request a dracclient release then. Please do double-check me on correctness ;)10:43
patchbotpatch 394353 - python-dracclient - Manually sync requirements (compatible with Newton...10:43
lucasagomesdtantsur, looking10:43
dtantsuroh, wait, why do we have a requirements job there? Oo10:44
lucasagomesdtantsur, jenkin's -1'd it with no message10:44
dtantsurlucasagomes, this is a requirements job, see its comments. now I wonder why we even run it, we don't seem to sync requirements...10:45
* lucasagomes CI wasn't toggled10:45
openstackgerritSam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation  https://review.openstack.org/39295910:46
openstackgerrityolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled  https://review.openstack.org/39374510:46
dtantsurlucasagomes, now something to think about: should we sync requirements for dracclient? no other vendor clients seem to do that...10:47
vdrokgood morning ironic and all!10:47
lucasagomesdtantsur, I recently did that for virtualbmc10:48
dtantsurlucasagomes, but we don't have requirements incompatible with g-r on the other hand..10:48
lucasagomesdtantsur, true... /me thinks10:48
lucasagomesdtantsur, perhaps we should sync the (test-)requirements.txt but for something external we could add a "extra-requirements.txt"10:50
lucasagomestho extra sounds like "not required"10:51
lucasagomes:-/10:51
dtantsurlucasagomes, we don't have external stuff.. so the question is whether we should be in projects.txt10:51
dtantsurlucasagomes, the only problem I see is that we're branchless... e.g. in this particular case I'd prefer to leave minimum requirements at newton level10:52
sambettsdtantsur: you were experimenting with Ironic support in the overcloud right?10:54
dtantsursambetts, if by "experimenting" you mean "implemented in a simple form and ready to ship to customers", then yes10:54
dtantsur:)10:54
dtantsurby simple form I mean no multitenancy mostly10:55
openstackgerrityolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled  https://review.openstack.org/39374510:55
sambettsdtantsur: Oh awesome! so I had a question about it, do you run a separate Ironic service for the overcloud or one Ironic service to serve both?10:57
dtantsursambetts, I run a separate service. It would be very interesting to use one ironic, but it would break separation of undercloud in overcloud.10:58
dtantsursambetts, we expect the undercloud to come and go, while overcloud sees no interruption10:58
vdrokdtantsur: lucasagomes we also don't sync requirements in staging drivers, but still have check-requirements job10:58
sambettsdtantsur: yeah, I thought that might be the case10:58
lucasagomesvdrok, true... we are probably facing the same problem as dracclient10:59
vdrokand we don't use constraints in staging, which recently caused gate break for a day or so10:59
sambettsdtantsur: do you know anyone that uses a single Ironic to manage equipment for OpenStack and not-OpenStack at the same time? e.g. like operating a standalone and non-standalone ironic at the same time11:01
dtantsurvdrok, this is wrong, we need to do either both or none11:02
dtantsursambetts, not really. I was thinking about it just this morning though, so good timing ;)11:02
sambettsdtantsur: great minds and all that ;)11:03
dtantsurtrue :)11:03
openstackgerritMerged openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/39412211:06
aarefievmorning !11:06
dtantsurlucasagomes, vdrok, I'm inclined to just remove the job from dracclient. wdyt?11:06
dtantsurmorning aarefiev11:07
vdrokdtantsur: I'm OK with it, maybe do the same with staging drivers for now, until we figure out if we need it or not11:07
dtantsurvdrok, if staging drivers are to join ironic umbrella, I'd prefer them to have a job actually...11:08
vdrokdtantsur: oh, are they? I missed the last meeting11:08
dtantsurthe reason is that dracclient is independent of openstack, while ironic-staging-drivers are tied to ironic11:08
dtantsurvdrok, this was discussed on the summit, no official consensus yet11:08
vdrokdtantsur: aha, OK. but it is also branchless, do we need to add branches then to correspond to ironic ones?11:09
*** g3ek has quit IRC11:09
dtantsurvdrok, if we want to backport fixes, yes11:10
lucasagomesdtantsur, good with me removing it for drac11:12
dtantsurk11:12
dtantsurI've proposed https://review.openstack.org/39436511:12
patchbotpatch 394365 - openstack-infra/project-config - Remove check-requirements job for python-dracclient11:12
* dtantsur -> lunch11:13
*** bks has quit IRC11:14
*** chlong has joined #openstack-ironic11:16
*** g3ek has joined #openstack-ironic11:18
*** pc_m has joined #openstack-ironic11:24
*** e0ne has joined #openstack-ironic11:24
*** athomas has quit IRC11:25
*** hw_wutianwei has quit IRC11:30
*** athomas has joined #openstack-ironic11:32
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/39273111:34
openstackgerrityolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled  https://review.openstack.org/39374511:42
*** bfournie has joined #openstack-ironic11:46
*** e0ne has quit IRC11:47
*** milan has joined #openstack-ironic11:48
*** vmud213 has quit IRC11:51
*** dprince has joined #openstack-ironic11:57
*** jkilpatr has joined #openstack-ironic11:59
*** aparnav has quit IRC12:01
*** shuquan_1 has quit IRC12:03
*** shuquan_ has joined #openstack-ironic12:05
*** Guest61285 has quit IRC12:06
openstackgerrityolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled  https://review.openstack.org/39374512:13
*** shuquan_ has quit IRC12:15
*** lucasagomes is now known as lucas-hungry12:15
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Serve iPXE configs from Ironic API  https://review.openstack.org/36943812:16
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot config to proper Jinja template  https://review.openstack.org/39346212:16
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot script to Jinja template  https://review.openstack.org/39439612:16
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Allow not writing pxe configs to disk  https://review.openstack.org/39439712:16
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic iPXE config  https://review.openstack.org/39439812:16
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get iPXE template and options  https://review.openstack.org/39439912:16
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE  https://review.openstack.org/39194312:17
aarefievdtantsur: about abort operation, do you think we can analyze  on ironic side  that  introspection was aborted by operator ?12:18
aarefievand not just fail12:18
dtantsuraarefiev, what instead?12:18
aarefievwe have abort in ironic, right?12:18
aarefievthe same for introspection12:19
dtantsuraarefiev, well, abort also ends up in failed state12:19
dtantsurwhen we implement abort for introspection on ironic side, the end result will still be "inspect failed"12:19
aarefievyeah, hmm, really?12:20
dtantsuraarefiev, e.g. https://github.com/openstack/ironic/blob/master/ironic/common/states.py#L31512:20
dtantsuranyway, do you think we have something to fix wrt https://review.openstack.org/393824?12:21
patchbotpatch 393824 - ironic-inspector - Add a test for introspection abort action verifica...12:21
aarefievhm, I see clean failed, why 'failed'12:21
aarefievif it was aborted12:21
dtantsuraarefiev, what else? we can't move it to successful path either12:21
aarefievsomething like aborted12:22
mat128good morning12:22
aarefievmorning mat12812:22
dtantsuraarefiev, well, yes, but probably folks thought that it will unnecessary complicate the state machine without too much benefit?12:22
dtantsurmorning mat12812:23
aarefievI'm trying to understand if something happened - node in fail state,  and if operator manually aborted operation it's also in fail state.12:23
dtantsuraarefiev, I think the reasoning is: an operation can either succeed or fail. there is no really other states from the operation point of view.12:24
dtantsurso we record the operation state, not the cause of this state12:24
aarefievok, it was always confusing for me12:25
aarefievdtantsur: should we then test this behavior in abort test?12:26
dtantsuraarefiev, moving to INSPECTFAIL? yeah, good idea12:26
aarefievI don't like it but it's expected12:26
dtantsurplease comment there12:26
dtantsursnozhka, ^^^12:26
aarefievsure12:26
*** mgould|afk is now known as mgould12:28
mgouldmorning dtantsur mat128 aarefiev and everyone else12:29
dtantsuro/12:29
mat128aarefiev, dtantsur: iirc this covers some of your questions - https://review.openstack.org/#/c/334113/12:30
patchbotpatch 334113 - ironic-specs - Add spec for specific faults support12:30
jrollmorning yo12:31
cdanielmorning all12:32
aarefievmorning mgould,  jroll, cdaniel12:32
nicodemosmorning, jroll, cdaniel, aarefiev, mgould, dtantsur and all. o/12:33
aarefievmat128: thanks, I didn't have the  chance to take a look on it12:33
aarefievmorning nicodemos12:33
cdanielmorning nicodemos12:35
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal node commands  https://review.openstack.org/30168712:39
openstackgerritMerged openstack/bifrost: Updated from global requirements  https://review.openstack.org/38987612:39
*** sdake has quit IRC12:40
*** sdake has joined #openstack-ironic12:40
aarefievdtantsur: btw, do we need spec for abort inspection in ironic ?12:43
dtantsuraarefiev, maybe an RFE would be enough (we might even have one)12:43
aarefievyeah, we have one12:43
*** rcernin has quit IRC12:43
mgouldmorning cdaniel jroll nicodemos12:43
*** rcernin has joined #openstack-ironic12:44
openstackgerrityolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled  https://review.openstack.org/39374512:45
openstackgerritAline Bousquet proposed openstack/ironic: Add possibility to remove chassis_uuid from a node  https://review.openstack.org/39055812:48
milanmorning cdaniel, jroll, mat128, mgould, nicodemos! :)12:48
mgouldmorning milan12:48
openstackgerritSergii Nozhka proposed openstack/ironic-inspector: Add a test for introspection abort action verification.  https://review.openstack.org/39382412:48
*** bfournie has quit IRC12:50
nicodemosmorning, milan.12:54
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add support for DBDeadlock handling  https://review.openstack.org/39441312:57
*** ohamada has quit IRC12:59
openstackgerritMerged openstack/ironic-inspector: Bump hacking to 0.11.0 in test-requirements  https://review.openstack.org/38933113:00
jrolldevananda|away: jlvill-travel: sambetts: yuriyz: still need your review on this, please :) https://review.openstack.org/#/c/392685/213:01
patchbotpatch 392685 - ironic - Clarify ironic governance requirements and process13:01
dtantsurmorning jroll13:03
jroll\o13:03
openstackgerritJim Rollenhagen proposed openstack/ironic-specs: Add ocata priorities  https://review.openstack.org/39375313:05
vdrokmorning mgould nicodemos milan and jroll13:05
jrollmorning :)13:05
milanmorning vdrok! :)13:05
nicodemoshey, vdrok13:06
*** gcb has quit IRC13:07
openstackgerritSergii Nozhka proposed openstack/ironic-inspector: Add a test for introspection abort action verification.  https://review.openstack.org/39382413:08
*** ccamacho is now known as ccamacho|lunch13:09
mgouldmorning vdrok13:14
wlfightupmorning13:16
wlfightupi want ask question for ironic13:16
sambettsjroll: LGTM :)13:16
wlfightuphow can modify resetapi return create_at date, the date have timzezone+0, but My timezone don't +013:17
wlfightupI don't want time zone13:17
jrollsambetts: thanks13:17
jrollwlfightup: don't get what you mean, you want a different time zone in the API or you want it formatted without time zone?13:17
wlfightupApi time to bring back the time zone, but my time zone is not 0 time zone, so I do not want this time zone13:18
vdrokwlfightup: I think all time values returned from api are utc13:20
jrollwell, I think they are server time zone13:21
dtantsurmgould, do you want to do releases of dracclient yourself? if no, please add me to https://review.openstack.org/#/admin/groups/1033,members13:21
*** bfournie has joined #openstack-ironic13:21
*** lucas-hungry is now known as lucasagomes13:21
wlfightuphow to set timezone?13:22
*** jheroux has joined #openstack-ironic13:22
jrolldepends on the linux distribution, really13:22
jrollit's easy to find that via google13:22
mgoulddtantsur: done13:22
dtantsurmgould, thanks!13:23
mgoulddtantsur: any reason I shouldn't also add cdearborn?13:23
wlfightupbut openstack get time by utcnow() function, this is 0 timezone13:23
dtantsurmgould, I'd prefer we have less people doing releases :) not sure if he actually wants to bother with that13:23
jrollwlfightup: ah, then it's probably always UTC13:23
jrollwlfightup: I'm not sure if it's UTC or server time zone, but that's easy to test, and it's easy to convert to your time zone in your API client13:24
mgoulddtantsur: OK, I'll add him if he asks but otherwise not bother13:24
dtantsurmgould, yeah.. I wonder if we should kick Imre from the release group (I'm fine with leaving him as a core for now)13:24
mgouldwlfightup: "always return times from your API in UTC and convert client-side if needed" is AIUI considered an API-design best practice: http://apiux.com/2013/03/20/5-laws-api-dates-and-times/13:25
mgouldso, consider if there's another way of achieving your goal :-)13:26
mgoulddtantsur: have removed Imre from the release group13:28
dtantsurthanks!13:28
wlfightupBut this article did not say how to set up13:28
dtantsurmgould, I'm going to tag the release as 1.0.0 just to signify the expected level of support and stability promise13:30
mgouldOK13:31
jrollwlfightup: your API client code needs to convert the timezone, how to do that depends on the language you are using (for example in python use datetime)13:31
mgouldwlfightup: the article I linked? No, it's meant to be very general advice, not connected to any specific service or technology13:32
mgouldtl;dr do not try to change the timezone returned by the server, that way lies only pain13:33
dtantsurlucasagomes, mgould, another quick review please https://review.openstack.org/39443113:37
patchbotpatch 394431 - python-dracclient - Update setup.cfg13:37
wlfightupBut the 0 time zone in here we see the wrong time13:38
wlfightupcostomer see a wrong time will anger13:38
lucasagomesdtantsur, will take a look13:40
*** links has quit IRC13:40
lucasagomesdtantsur, is dracclient already out of the ironic's umbrealla ?13:40
*** jcoufal has joined #openstack-ironic13:41
TheJuliaGood morning!13:41
dtantsurlucasagomes, not yet, but pending essentially13:41
mgouldmorning TheJulia13:42
dtantsurTheJulia, o/13:42
* dtantsur writes a email about wsmanclient future13:43
lucasagomesdtantsur, right... well, should be fine then. As a note, I don't see maintainer or maintainer_email's on other p13:43
lucasagomesprojects*13:43
mgouldwlfightup: how are your users talking to Ironic?13:44
vdrokmorning TheJulia13:44
dtantsurlucasagomes, on some projects it's set to "OpenStack" and the openstack-dev ML13:44
vdrokwlfightup: time zone +0 means UTC time, it is a correct UTC time, just clarify this to your users13:44
lucasagomesdtantsur, right13:45
mgouldvdrok: I think wlfightup wants to return non-UTC times from the API13:46
vdrokmgould: I get that, but I think it's not possible in an easy way right now13:46
vdrokyou'll have to modify dbapi I think13:47
mgouldvdrok: indeed, and I don't think it's a good idea even if it were possible13:47
vdrok++13:47
dtantsurlucasagomes, mgould, is it fair to speak on your behalf that you don't want to support separate wsmanclient?13:47
*** sdake has quit IRC13:48
mgoulddtantsur: I know almost nothing about the codebase13:48
lucasagomesdtantsur, the idea was to dracclient to use wsmeclient, right ?13:48
dtantsurlucasagomes, yes, but it was never done13:48
lucasagomesdtantsur, right, and the folks working in the AMT driver (if there's one) don't want to maintain wsmeclient either /13:49
lucasagomes?*13:49
lucasagomesif not, yeah, I think we can drop it13:49
* lucasagomes wants less things to maintain13:49
dtantsurlucasagomes, this is what I'm going to ask on the ML13:49
*** hshiina has quit IRC13:49
dtantsurI just want to know your opinion, so that I can speak on our behalf, not only myself13:49
lucasagomesdtantsur, ok so, if it's drac exclusive +1 to drop it13:49
dtantsurk13:50
wlfightupyes, I wants to return non-UTC times from the API13:51
wlfightuphow to do this13:51
lucasagomeswlfightup, create a layer to convert this time for you ? e.g, dashboard, rest client etc13:52
mgouldwlfightup: it looks like you can set per-user timezone preferences in Horizon, so dates are displayed in the user's timezone13:56
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Serve iPXE configs from Ironic API  https://review.openstack.org/36943813:56
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic iPXE config  https://review.openstack.org/39439813:56
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get iPXE template and options  https://review.openstack.org/39439913:56
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE  https://review.openstack.org/39194313:56
*** Goneri has joined #openstack-ironic13:56
mgouldif they're using the command-line clients, they're probably technically sophisticated enough to want UTC anyway13:56
*** amotoki has quit IRC13:58
*** Guest71310 has quit IRC13:58
dtantsurmgould, lucasagomes, the requirements patch is finally passing, please review: https://review.openstack.org/#/c/394353/13:59
patchbotpatch 394353 - python-dracclient - Manually sync requirements (compatible with Newton...13:59
robcresswellYeah, Horizon does it nicely for the user.13:59
robcresswellCLI should probably be UTC or everything will get difficult.13:59
mgoulddtantsur: +214:01
dtantsurmgould, please see my comment on https://review.openstack.org/#/c/394431/ I'm not really sure how pypi will render a link there..14:03
patchbotpatch 394431 - python-dracclient - Update setup.cfg14:03
*** baoli has joined #openstack-ironic14:03
mgoulddtantsur: OK, let's leave it out14:06
*** rloo has joined #openstack-ironic14:06
dtantsurthanks! now I'll do a short local testing with one of our dell machines, then do the release14:07
*** rbudden has joined #openstack-ironic14:08
*** amotoki has joined #openstack-ironic14:09
*** beekneemech is now known as bnemec14:10
*** Guest71310 has joined #openstack-ironic14:12
*** amotoki has quit IRC14:13
*** syed_ has joined #openstack-ironic14:14
*** yufei__ has joined #openstack-ironic14:16
openstackgerritSergii Nozhka proposed openstack/ironic-inspector: Add a test for introspection abort action verification.  https://review.openstack.org/39382414:17
*** Saulo_Aislan has joined #openstack-ironic14:18
Saulo_AislanHow install patch in openstack ironic?14:18
*** Guest71310 has quit IRC14:18
*** tesseract has joined #openstack-ironic14:19
*** tesseract is now known as Guest4111714:19
mgouldSaulo_Aislan: do you mean that you have a code patch, and you want to deploy an Ironic service with that patch applied?14:19
mgouldSaulo_Aislan: the answer depends on how you're deploying Ironic14:21
*** mgoddard has quit IRC14:22
mgouldfor devstack, see http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack and search for "To check out an in-progress patch"14:22
*** Guest41117 has quit IRC14:22
Saulo_Aislanmgould: I want to deploy a patch in Ironic service14:22
Saulo_AislanI didn't install devstack14:24
mgoulda patch to the code of Ironic?14:24
Saulo_AislanYes! https://review.openstack.org/#/c/345963/14:24
patchbotpatch 345963 - ironic - Neutron notifications14:24
mgouldSaulo_Aislan: do you already have an Ironic server deployed which you want to apply the patch to?14:25
Saulo_AislanYes I have it running14:26
mgouldhow did you deploy it?14:26
*** mgoddard has joined #openstack-ironic14:27
*** tesseract- has joined #openstack-ironic14:27
*** makowals has quit IRC14:29
Saulo_Aislanmgould: I installed using apt-get install14:30
*** ohamada has joined #openstack-ironic14:30
mgouldOK14:30
mgould^^^ suggestions anyone?14:31
*** jaosorior has quit IRC14:32
*** makowals has joined #openstack-ironic14:32
mgould"git-clone the Ironic tree, copy it over the version apt-get installed, restart service" sounds... risky14:32
*** jaosorior has joined #openstack-ironic14:33
dtantsurmgould, meh, found a few more bugs in dracclient >_< will fix first14:33
*** chlong has quit IRC14:34
sambettsmgould: you might not have to copy it over it, you should be able to git clone ironic, and then just "pip install" the new one, however you'd have to be careful if any DB changes have landed that would need migrating etc14:34
Saulo_Aislanmgould: thanks, I will try!!!14:34
mgouldSaulo_Aislan: try sambetts' suggestion first ;)14:35
Saulo_Aislanmgould: Ok14:35
mgouldSaulo_Aislan: oh, and run "git fetch ssh://pozorvlak@review.openstack.org:29418/openstack/ironic refs/changes/63/345963/4 && git checkout FETCH_HEAD" before you do the pip install or copy, so you get the patch you want14:36
*** yufei__ has quit IRC14:38
*** rbrud has joined #openstack-ironic14:38
Saulo_Aislanmgould: sambetts: Thks14:40
mgouldSaulo_Aislan: np, good luck!14:41
mgouldwait, shit, that command will only work for me14:41
sambettsXD14:41
*** makowals has quit IRC14:42
mgouldon https://review.openstack.org/#/c/345963/, click the "download" dropdown, and copy the "checkout" command14:42
patchbotpatch 345963 - ironic - Neutron notifications14:42
*** ccamacho|lunch is now known as ccamacho14:42
Saulo_AislanGot it, I modifed it !!!14:42
mgould\o/14:43
Saulo_AislanXD14:43
*** jlvill-travel is now known as jlvillal14:43
*** wajdi has joined #openstack-ironic14:45
openstackgerritSergii Nozhka proposed openstack/ironic-inspector: Add a test for introspection abort action verification.  https://review.openstack.org/39382414:46
*** wlfightup has quit IRC14:50
jlvillaljroll: I voted and left some comments on https://review.openstack.org/#/c/392685/  Thanks.14:50
patchbotpatch 392685 - ironic - Clarify ironic governance requirements and process14:50
jrollthanks14:51
jrollwelcome back jlvillal14:51
jlvillalThanks :D14:51
jlvillalI would say it is great to be back from vacation but .... ;)14:51
mgouldhi jlvillal, welcome back14:52
*** causten has joined #openstack-ironic14:53
jlvillalThanks mgould. Sorry to hear about your Airbnb woes at the beginning of the summit.14:53
mgouldjlvillal: thanks! we got it sorted in the end, though14:54
jlvillalGood to hear.14:54
vdrokmorning jlvillal !14:56
mosheleJayF: around?14:57
jlvillalvdrok: Hey! You superstar OpenStack Summit presenter :D14:58
*** makowals has joined #openstack-ironic14:58
vdrokheh14:59
jlvillalvdrok: Any idea how many people came to see the presentation? They are asking me.14:59
jlvillalIt didn't seem like that many. but the room was HUGE!!!!14:59
jlvillalSo it just seemed very empty.15:00
vdrokjlvillal: no idea, around 30 maybe?15:01
jlvillalvdrok: Sounds about right. In a room that could probably hold 400...15:02
mariojvwelcome back jlvillal15:03
jlvillalThanks mariojv :)15:03
* mgould apologises for missing it; something urgent came up and I'd already seen the rehearsal :-)15:03
mariojv+1, was a great presentation, saw the recording earlier :)15:03
jlvillalmariojv: Thanks. Did you hear me say, "That was fast" after I finished the first part.15:03
* jlvillal laughs at himself.15:03
* jlvillal goes to make some coffee15:04
JayFmoshele: what's up?15:05
mosheleJayF: I want to move the code in https://review.openstack.org/#/c/334816/ to use MellanoxDeviceHardwareManager which I am working on15:08
patchbotpatch 334816 - ironic-python-agent - InfiniBand Support for IPA15:08
*** rbartal has quit IRC15:09
JayFmoshele: we have some decent docs in IPA (IPA dev docs) about hardware managers, and I have some sample code here: https://github.com/jayofdoom/ipa-example-hardware-managers15:09
*** jaosorior is now known as jaosorior_sick15:09
JayFmoshele: IDK what you're looking for, but that should help?15:09
mosheleJayF: I read those and it was very helpful15:09
*** amotoki has joined #openstack-ironic15:10
mosheleJayF: I need to overwrite the list_network_interfaces and I want the manager dispatcher to merge it with the generic hardware manager result15:11
mosheleJayF: and not return only mellanox vendor result15:12
*** tuan_luong has joined #openstack-ironic15:12
mosheleJayF: see this https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L102215:13
devananda|awayjroll: commented. +2 on what's there, -1 because it lacks details.15:13
JayFmoshele: hmm. There are two approaches you could take to this:15:13
JayFmoshele: 1) rewrite list_network_interfaces in GenericHardwareManager to send each interface through a dipatch call, something logically like `for NIC in nics: dispatch_to_managers(get_interface_information, NIC)`15:15
jrolldevananda|away: wdyt about landing it as-is and following up with details (though, I'm not sure I want to get into where these things should be hosted, etc)15:15
JayFmoshele: then obviously implement get_interface_information in both generic and mellanox managers15:15
jrolldevananda|away: (which is why I'd prefer to get through details in a separate patch, that could get more controversial and you're the only core not +2 on this right now)15:15
devananda|awaythat's fine with me15:16
JayFmoshele: the other option is to subclass generichardwaremanager, and call super() after you're done working on mellanox cards, but I don't like that15:16
jrolldevananda|away: cool, want to change your vote? :)15:16
tuan_luonghello yuriyz, dtantsur15:18
tuan_luongcan we discuss about https://review.openstack.org/#/c/376322/15:18
patchbotpatch 376322 - ironic - Fix setting persistent boot device does not work15:18
*** sabeen has joined #openstack-ironic15:18
tuan_luongI thinks the problem hard because it depend on hardware and environment for test15:19
tuan_luonghus, Can we discuss how to test bugs and solutions for this bugs.15:20
openstackgerritMichael Turek proposed openstack/ironic: Enable PXE for systems using petitboot  https://review.openstack.org/18598715:20
mosheleJayF: I will take a look on the first option thanks15:20
JayFmoshele: look at how erase_block_devices handles it15:21
JayFmoshele: that's basically the pattern I'm suggesting you reproduce15:21
mosheleJayF: I will take a look15:21
*** rbartal has joined #openstack-ironic15:21
*** sabeen1 has joined #openstack-ironic15:21
devananda|awayjroll: found a factual error just now15:22
devananda|awayoh -- never mind15:23
jrolldevananda|away: that governance site is generated from your link15:23
devananda|awaythat's odd15:23
*** sabeen has quit IRC15:23
devananda|awaythere's a file in github at that same path, which is out of date15:23
devananda|awayso yea, the link is correct15:23
rloohello Ironic'ers and Happy Monday!15:32
JayFo/15:33
rloojlvillal: I counted about 45 bodies at the BlowingUpGrenade talk15:33
jlvillalrloo: Oh thanks :)15:33
*** jjohnson2 has quit IRC15:33
sambettsjlvillal: I thought it was awesome :) it made me realise how incredibly hacky our networking simulation is so I've created a patch to fix it15:34
jrolldevananda|away: thanks, I will follow up on that patch with more things15:35
*** links has joined #openstack-ironic15:35
*** absubram has joined #openstack-ironic15:36
jlvillalsambetts: Oh cool. I'll try to find it today when I get a chance to do some reviews15:37
rloojroll: wrt the governance patch, seems like either a followup, or just revise the existing and get 2 +2/+A on it.15:37
sambettsjlvillal: https://review.openstack.org/#/c/392959/ although its failing a few of the tests because the devstack variables have changed15:37
patchbotpatch 392959 - ironic - Rework Ironic devstack baremetal network simulation15:37
jlvillalvsaienk0: vdrok ^^  FYI: sambetts patch, if you didn't see it15:38
jrollrloo: yeah, those seem equivalent to me, I'm going to land it now as I'm the only core without a vote recorded15:38
rloojroll: ok, whatever works for you15:38
rloojroll: you wrote it, i hope that means you're +2 on it ;)15:39
JayFNobodyCam: iirc,at one point you had a testing environment where you used the devstack vm to reach back and make the ironic-node-vms on the same kvm hyp that hosted devstack -- do you have any details on this/15:39
*** wajdi_ has joined #openstack-ironic15:40
jrollJayF: ssh driver makes that easy - give devstack ssh access to the hyp15:40
JayFjroll: I suspect it got a lot more complicated if I want network flipping to work15:41
jrollJayF: hm, shouldn't?15:41
*** harlowja_at_home has joined #openstack-ironic15:41
JayFjroll: but I was mainly fishin for if we had any support in devstack for doing it rather than hand-crafting it15:41
*** serverascode has quit IRC15:41
TheJuliaJayF: I highly highly suspect NobodyCam is traveling for family reasons for the next few days15:41
jrollbecause that's control plane -> some hyp15:41
JayFjroll: well, I know devstack sets up a lot of that infrastructure --- for the vms to be on different networks15:41
*** izaakk has quit IRC15:42
JayFjroll: so I'd have to do that bit manually on the hyp host, since the whole point is to isolate devstack mess inside a vm15:42
jrollJayF: oh, yeah, you might be right hm15:42
*** vdrok has quit IRC15:42
*** sergek_ has quit IRC15:42
*** mat128_ has quit IRC15:42
*** cppforlife_ has quit IRC15:42
*** mgkwill has quit IRC15:42
*** BadCub has quit IRC15:42
*** francoblanco has quit IRC15:42
*** kozhukalov has quit IRC15:42
*** amit213 has quit IRC15:42
*** wajdi has quit IRC15:42
*** romcheg has quit IRC15:42
*** zhenguo has quit IRC15:42
*** raginbaj- is now known as raginbajin15:42
*** Ng has quit IRC15:43
*** clif_h has quit IRC15:43
*** madorn has quit IRC15:43
*** mariojv has quit IRC15:43
*** fungi has quit IRC15:43
*** francoblanco has joined #openstack-ironic15:44
*** mat128_ has joined #openstack-ironic15:44
*** sergek_ has joined #openstack-ironic15:44
*** romcheg has joined #openstack-ironic15:44
*** izaakk has joined #openstack-ironic15:45
*** cppforlife_ has joined #openstack-ironic15:46
*** BadCub has joined #openstack-ironic15:46
*** Ng has joined #openstack-ironic15:46
*** mgkwill has joined #openstack-ironic15:46
*** zhenguo has joined #openstack-ironic15:46
*** vdrok has joined #openstack-ironic15:46
jlvillalsambetts: On that patch. You might want to add a Co-Authored... for yourself.15:46
*** amit213 has joined #openstack-ironic15:47
*** gcb has joined #openstack-ironic15:47
vdrokmorning rloo, jlvillal will take a look some time :)15:47
openstackgerritMerged openstack/ironic: Clarify ironic governance requirements and process  https://review.openstack.org/39268515:47
*** serverascode has joined #openstack-ironic15:47
*** kozhukalov has joined #openstack-ironic15:48
sambettsjlvillal: oh... thats weird why did it use Vasyl as the Author... XXD15:48
jlvillalsambetts: I added a comment about it. /me thinks he is funny :)15:48
rloohi vdrok15:49
jlvillalsambetts: Maybe you had started with a patch from Vasyl??15:49
sambettsjlvillal: I started off by experimenting with vasyls flat network patch, then it sort of evolved XD15:50
sambettsjroll: are you leading the ir/neutron meeting today?15:51
*** makowals has quit IRC15:51
jrollsambetts: not that I know of? :)15:51
jrollI can15:51
sambettsjroll: oh my mistake I thought sukdev was out today, but that is for the wrong date15:52
jrollsambetts: heh, there's an agenda, so he must be back15:53
sambetts:-P15:53
*** rama_y has joined #openstack-ironic15:54
*** clif_h has joined #openstack-ironic15:54
*** rcernin has quit IRC15:55
*** mariojv has joined #openstack-ironic15:55
*** fungi has joined #openstack-ironic15:55
*** madorn has joined #openstack-ironic15:56
*** hshiina has joined #openstack-ironic15:57
*** tuan_luong has quit IRC15:58
*** g3ek has quit IRC15:58
*** Sukhdev has joined #openstack-ironic15:59
*** mgoddard_ has joined #openstack-ironic16:01
*** g3ek has joined #openstack-ironic16:03
*** makowals has joined #openstack-ironic16:03
*** ppiela has joined #openstack-ironic16:04
*** mgoddard has quit IRC16:05
*** rbartal has quit IRC16:08
dtantsurmgould, this patch chain fixes OOB inspection for me: https://review.openstack.org/#/c/394510/16:09
patchbotpatch 394510 - python-dracclient - Virtual disk properties can be nullable16:09
*** moshele has quit IRC16:11
*** pcaruana has quit IRC16:12
*** romcheg has quit IRC16:15
*** sergek_ has quit IRC16:15
mgoulddtantsur: excellent, reviewing now16:15
dtantsurthnx16:15
*** rloo has quit IRC16:17
*** rloo has joined #openstack-ironic16:17
*** BadCub has quit IRC16:18
*** BadCub has joined #openstack-ironic16:18
*** rbrud has quit IRC16:18
*** sergek_ has joined #openstack-ironic16:19
*** romcheg has joined #openstack-ironic16:20
*** rbartal has joined #openstack-ironic16:22
*** sdake has joined #openstack-ironic16:25
*** tesseract- has quit IRC16:31
*** krtaylor has quit IRC16:33
*** godp1301 has joined #openstack-ironic16:33
*** causten has quit IRC16:34
*** fxpester has quit IRC16:34
*** causten has joined #openstack-ironic16:35
*** vishwanathj has joined #openstack-ironic16:40
mgoulddtantsur: done, looks fine16:45
dtantsurthnx!16:48
*** jkilpatr_ has joined #openstack-ironic16:52
*** godp1301 has quit IRC16:54
*** jkilpatr has quit IRC16:55
*** godp1301 has joined #openstack-ironic16:55
*** sdake has quit IRC16:56
*** stendulker has joined #openstack-ironic16:56
*** Guest61285 has joined #openstack-ironic16:56
*** mat128_ has quit IRC16:58
*** fragatina has joined #openstack-ironic16:59
rlooironic meeting in a minute16:59
*** srobert has joined #openstack-ironic16:59
*** harlowja_at_home has quit IRC17:00
*** rbartal has quit IRC17:00
openstackgerritSam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation  https://review.openstack.org/39295917:00
rloojroll: ironic meeting17:00
*** mgoddard_ has quit IRC17:00
*** mgoddard has joined #openstack-ironic17:00
sambettsjlvillal: fixed the author/commit information on that patch :)17:01
*** fragatina has quit IRC17:01
*** fragatina has joined #openstack-ironic17:01
*** Guest61285 is now known as Nisha117:02
jlvillalsambetts: :)17:02
*** strigazi is now known as strigazi_AFK17:02
*** vishwanathj has quit IRC17:03
*** krtaylor has joined #openstack-ironic17:03
sambettsjlvillal: I hope that with that patch we can remove a lot hacky stuff from the grenade plugin around the neutron resources because neutron sees our simulated environment exactly like a physical network17:04
*** krtaylor has quit IRC17:05
jlvillalsambetts: Hopefully I have time to look today. First day back after about two weeks away from normal work-day activities. So ....17:05
sambettsjlvillal: heh spin up time is always hard17:05
*** gcb has quit IRC17:08
*** krtaylor has joined #openstack-ironic17:11
*** _mbelanger has joined #openstack-ironic17:16
*** links has quit IRC17:20
milanguys, I'm wondering about socat exec parameter; I cannot make it work w/ ironic's command line: http://paste.openstack.org/show/588292/17:24
milanhowever, I'm once I remove the EXEC:/:987:983:HOME: all just works17:25
kvcobbGood Morning, Folks. I'm having an issue with spinning up an Ironic instance. Wondering if someone has seen this before: I spin up an instance with my BM flavor selected. The server powers down, gets set for PXE, powers back up, sees PXE and gets the deploy image and ramdisk, shortly after it fails with these errors in the conductor logs: http://paste.openstack.org/show/587940/ Anyone see that before? This a problem with the images17:25
kvcobbI created (I built my image with the partition image instructions rather than whole disk)?17:25
dtantsurmilan, is it related to https://review.openstack.org/393746?17:25
patchbotpatch 393746 - ironic (stable/newton) - IPMI command should depend on console type17:25
JayFkvcobb: you're going to need logs from the ironic side to troubleshoot that17:26
JayFkvcobb: that's just nova driver telling you "yep, it never changed to active state"17:26
JayFkvcobb: the ironic conductor logs are more likely to contain the real error, or the node.last_error on the node you were deploying to17:26
milandtantsur, yeah, maybe that will fix it17:26
* milan applies patch17:26
kvcobbThanks Jay, I thought that log I was looking at was the conductor log. My apologies, just learning Ironic.17:27
*** fragatina has quit IRC17:28
kvcobbI'm only aware of the two sets of logs for Ironic - Conductor and API, is there another set of conductor logs?17:28
JayFkvcobb: nope; the logs you gave are from the nova compute process though17:29
*** Sukhdev has quit IRC17:30
kvcobbOK, I'll go back and look at it again.17:30
*** trown is now known as trown|lunch17:31
*** stendulker has quit IRC17:35
openstackgerritSergii Nozhka proposed openstack/ironic-inspector: Add a test for introspection abort action verification.  https://review.openstack.org/39382417:36
jlvillalFYI: I think you will be able to sign up for the PTG for $100. And then if need be can transfer it to someone else later. If you can't go.17:37
*** causten has quit IRC17:38
sambetts++17:39
gabriel-bezerraThere won't be ATC passes on PTG, then?17:39
JayFBasically it's $100 to go to PTG17:39
vdrokgabriel-bezerra: seems not17:39
*** causten has joined #openstack-ironic17:40
JayFand people who attend PTG get into the forum (new summit) for free17:40
dtantsurwell, if you go to the forum, that is..17:40
persiaOr, significant discount (can be free if combined with early registration)17:40
lucasagomesdtantsur, lol right17:40
vdrokalso I've asked about travel support for ptg, no answer yet17:41
vdrokasked if there is one I mean17:41
*** jjohnson2 has joined #openstack-ironic17:41
jrollkeep in mind only PTG *attendees* get the forum discount, you can't get it if you sign up and not attend17:41
JayFSupposedly I get to find out about PTG by the end of the week.17:41
jrollvdrok: I hear they are working on it17:41
mgouldkvcobb: under devstack you need to look at logs/ir-cond.log17:41
persiaI have been told there will br travel support for PTG, but not heaps and heaps of it.  My sources are those who ought be correct.17:41
mgouldunder tripleo it'll be /var/log/ironic/ironic-conductor.log17:42
sambettswhen you sign up for PTG it asks you to optionally contribute to travel support17:42
mgouldetc17:42
mgouldtake the file you're looking at and s/nova/ironic/g :-)17:42
vdrokjroll: sambetts aha, thanks17:43
kvcobbGot it, was on the wrong page in screen :) Here's the correct conductor log for my issue: http://paste.openstack.org/show/588296/17:43
sambettsvdrok: not sure how to apply for it though17:43
*** causten has quit IRC17:44
vdroksambetts: me neither, at least there is a refund option for the ticket, so I'll get one and see if i get travel support or maybe find a super cheap flight :)17:44
JayFkvcobb: that's incredibly strange, are you sure you'r econfigured correctly? Any proxies between the conducotr and the agents?17:45
lucasagomesfolks I'm calling it a day17:45
lucasagomeshave a great evening all!17:45
vdroknight lucasagomes17:45
lucasagomeso/17:45
*** lucasagomes is now known as lucas-afk17:45
mgouldlucas-afk: o/17:46
*** jpich has quit IRC17:47
kvcobbWell here's a little background: I need to get a BM provisioner up and running with only 2 servers attached (this is for QA at OSIC). As I only have 2 servers the simplest cloud config seemed like the best call and as an All in One devstack deployment I can use snapshots on a VM to rollback as needed if anything gets configured improperly.17:47
dtantsurgoing as well, see you17:47
*** dtantsur is now known as dtantsur|afk17:47
mgoulddtantsur|afk: o/17:47
*** athomas has quit IRC17:47
JayFkvcobb: I'm just saying, you're getting an html "cannot connect" response which would not be returned by *any* tools ironic hsips17:48
kvcobbI am on company intranet provided via NAT on the uplonk port of a little linksys router17:48
JayFkvcobb: so something very strange is going on in your local/environmental configuration that you're going to have to sort out :/17:48
kvcobbuplink*17:48
mgouldkvcobb: so your conductor is running on a machine behind your linksys uplink, but the BM nodes are somewhere else on your corporate VPN?17:49
kvcobbhahh, alright then. I will say this hasn't been easy at all so far so I'm not real surprised.17:49
mgouldthat could well be some "helpful" web proxying done by your ISP17:49
kvcobbno conductor and nodes are all on the linksys router17:49
mgouldOK17:50
* jroll -> lunch17:50
vdrokgood night dtantsur|afk17:50
kvcobbI just provide internet to the network via uplink from company intranet which does have a proxy17:50
kvcobbI use port forwarding on the router to get to horizon, SSH on devstack, etc17:50
mgould[conductor, BM1, BM2] -> [linksys uplink] -> [company intranet] -> [internet]?17:51
kvcobbI had to use the router so that iPXE would not conflict with intranet PXE service17:51
mgouldOK17:51
kvcobbyes thats correct mgould17:51
mgouldis the router running a DHCP server?17:52
kvcobbno dhcp is disabled17:52
mgouldOK17:52
mgouldtry `curl http://192.168.1.126:9999/v1/commands` and `traceroute 192.168.1.126`17:52
kvcobbI have static address assigned to the conductor and it is able to successfully lease address and transfer images to the nodes17:53
kvcobbtrying now17:53
*** rama_y has quit IRC17:54
kvcobb`traceroute 192.168.1.12617:56
milandtantsur|afk, that helped (socat thing)! Good night!17:56
kvcobboops - lets try that again: http://paste.openstack.org/show/588299/17:56
*** shuquan has joined #openstack-ironic17:57
kvcobb192.168.1.126 isn't my node right? Because it looks like ironic powered down my node.17:57
*** jkilpatr_ has quit IRC17:58
anishdtantsur|afk: curious, what bugs were hit ?17:58
*** ralonsoh has quit IRC17:58
mgouldkvcobb: yes, that's what it looks like to me17:59
*** aNuposic has joined #openstack-ironic18:00
kvcobbthat could be trickie then because when the node comes up via pxe its only alive for a short time before ironic power cycles it to retry18:00
*** shuquan has quit IRC18:01
kvcobbcould you give me a high level of where this webserver is running on 192.168.1.126 and what its function is?18:01
anishdtantsur|afk: nm, I see it. more 'nullable'18:02
mgouldkvcobb: AIUI, we boot a ramdisk on the node (the "agent")18:02
mgouldthat ramdisk then runs a little web application18:02
kvcobbmy conductor is 192.168.1.102 and I would expect the node to be the first acvailbel DHCP address (192.168.1.120)18:02
mgouldIronic talks to the agent via HTTP18:02
*** derekh has quit IRC18:03
mgouldbut yeah, that web server should be running on the node to be booted18:03
kvcobbahh got ya, just elarning this part because getting the PXE service working over the physical network with devstack was really tricky18:03
mgouldyep18:03
mgouldnot something I've ever tried, because I assumed it would be :-(18:03
sambettsworking on it ;)18:03
mgouldsambetts: is http://paste.openstack.org/show/516174/ still usable?18:04
*** fragatina has joined #openstack-ironic18:04
kvcobbwell everything is possible :) I plan to publish a guide once  get it working - and I have to be close.18:04
mgould^^^ the local.conf you shared a while back for working with actual baremetal18:04
kvcobbit helps to understand whats happening there with the AIUI piece. I'll read up on it18:04
sambettsmgould: should be pretty close to working18:05
sambettsmgould: I don't think I've changed much in my CI since I created that lcoal.conf18:05
mgouldcool18:05
mgouldkvcobb: so traceroute is failing to connect to that IP18:06
sambettsmgould: I hope the options I introduce in this patch will let us optionally configure Ironic to use a real network interface too https://review.openstack.org/#/c/392959/18:06
patchbotpatch 392959 - ironic - Rework Ironic devstack baremetal network simulation18:06
mgouldso *something* is generating that "Network Error" page18:06
*** rama_y has joined #openstack-ironic18:06
mgouldmy guess would be your router18:06
mgouldsambetts: awesome18:07
*** jkilpatr_ has joined #openstack-ironic18:08
kvcobbThis is the local.conf that I used to get this far: http://paste.openstack.org/show/588301/18:08
kvcobbthe critical pieces were the ones for IRONIC_PROVISION*18:08
sambettskvcobb: are you using vasyls patch for that?18:08
mgouldkvcobb: you could maybe try deploying again and trying those curl/traceroute commands while the node is up18:09
kvcobbsambetts: No I stumbled through that with help of Matt Dorn18:09
mgouldkvcobb: can you run `ironic node-show [name of node you're trying to boot]`?18:10
kvcobbthis is the first I'm hearing of the patch you mentioned18:10
*** jcoufal_ has joined #openstack-ironic18:10
sambettskvcobb: the one I linked too is pretty new, I'm still working on it too18:10
sambettskvcobb: I've never used the IRONIC_PROVISION_* options for real hardware networks18:11
kvcobbhttp://paste.openstack.org/show/588303/18:12
kvcobbtheres the node-show18:12
mgouldthanks18:12
mgouldwhat's the difference between agent_ipmitool and pxe_ipmitool?18:12
sambettsthey were added to support ironic multitenancy support in the CI18:12
mat128mgould: deploy driver18:12
kvcobbnp - I appreciate the help :)18:12
*** jcoufal has quit IRC18:13
sambettsmgould: agent_ipmitool deploys to the ramdisk using HTTP via swift tempurls, and pxe_ipmitool deploys using iscsi18:13
sambettsmgould: they are terribly named...18:13
kvcobbwell until I made those changes to local.conf. My node couldn't see the PXE server18:13
*** jkilpatr_ has quit IRC18:13
sambettskvcobb: huh ok, thats good to know :)18:14
*** jaosorior_sick has quit IRC18:14
kvcobbGoing to try and rerun now - and issue that curl command while its powered up18:14
mgouldidiot question: we use neutron's DHCP server, right?18:15
sambettskvcobb: qq, do you have a HTTP_PROXY environment variable set up on your Ironic conductor node??18:15
*** hshiina has quit IRC18:15
mgoulddo we use that to serve PXE as well?18:15
sambettsmgould: ironic hosts the TFTP server, but we use the neutron DHCP18:16
kvcobbmgould: that's correct - DHCP is working over that ironicnet mentioned in local.conf18:16
kvcobbyup18:16
mgouldOK18:17
kvcobbsambetts: no I don't would that need to be somehow slipstreamed into the deploy ramdisk?18:17
mgouldand is Neutron's DHCP server connected to that network?18:17
kvcobbyeah its dnsmasq running on devstack hosted to that 192.x.x.x network18:19
sambettskvcobb: no, so the v1/commands API is ramdisks API the Ironic conductor uses that to talk to the booted ramdisk to install the image on the node, if there is a http_proxy on the conductor node, it might be trying to go through the proxy when it doesn't need to, I have had that happen before, so I had to setup a no_proxy environment variable for all the IPs I was using for my baremetal18:19
openstackgerritMerged openstack/ironic: Add a unit test for microversion validation V1.22  https://review.openstack.org/39402518:21
kvcobbI think I just realized an interesting related behavior of my environment. I have horizon installed on devstack at 192.168.1.102. If I try to access horizon from another machine on my router network I get that same network connection refused error18:21
kvcobbthe only way to pull up horizon is via the uplink IP coming from my intranet and assigned to the router18:22
mgouldriiiiight18:23
kvcobband that only loads because portforwarding on the router sends those to my 102 address18:23
mgouldsounds like your router is trying to be helpful and failing18:23
kvcobbagreed18:23
*** ChubYann has joined #openstack-ironic18:23
kvcobbI think if I solve that problem this this issue with the node goes away18:23
mgouldpoke around in its settings and turn off anything that looks like an HTTP proxy18:23
mgouldyeah, I think so too18:24
mgouldgood luck!18:24
kvcobbWill do - very helpful talking this though, guys - thanks again.18:24
mgouldnp18:24
sambetts:) np18:24
*** Nisha1 has quit IRC18:25
*** jkilpatr_ has joined #openstack-ironic18:26
*** rbrud has joined #openstack-ironic18:29
*** fragatina has quit IRC18:30
*** fragatina has joined #openstack-ironic18:31
*** ohamada has quit IRC18:31
*** mgould is now known as mgould|afk18:36
* milan gone18:36
milangood night Ironic!18:36
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/39408718:39
*** milan has quit IRC18:42
*** Romanenko_K has joined #openstack-ironic18:47
*** leduardo has quit IRC18:49
*** godp1301 has quit IRC18:52
*** Sukhdev has joined #openstack-ironic18:55
*** Romanenko_K has quit IRC18:57
* sambetts -> home18:59
sambettsnight everyone18:59
*** sambetts is now known as sambetts|afk18:59
jlvillalsambetts|afk: Good night19:06
*** ppiela has quit IRC19:08
*** trown|lunch is now known as trown19:10
*** causten has joined #openstack-ironic19:18
*** ppiela has joined #openstack-ironic19:20
openstackgerritMerged openstack/ironic: Clean up unit test of API root test  https://review.openstack.org/39402219:21
*** rpioso has joined #openstack-ironic19:29
*** d0ugal has quit IRC19:30
*** d0ugal has joined #openstack-ironic19:31
*** d0ugal has quit IRC19:31
*** d0ugal has joined #openstack-ironic19:31
*** ppiela has quit IRC19:31
openstackgerritMerged openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/39412319:39
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/39408819:40
*** awidders_ has joined #openstack-ironic19:43
*** awiddersheim has quit IRC19:45
*** godp1301 has joined #openstack-ironic19:56
*** godp1301 has quit IRC19:56
*** godp1301 has joined #openstack-ironic19:57
*** dprince has quit IRC20:00
openstackgerritJohn L. Villalovos proposed openstack/ironic: Add PS4 for better logfile information of devstack runs  https://review.openstack.org/39460820:05
*** rbrud has quit IRC20:06
*** godp1301 has quit IRC20:11
*** shuquan has joined #openstack-ironic20:11
TheJuliabetherly: I sent Peter and yourself the feature comparison with notes and suggestions. Please don't hesitate to make edits if I missed something in the UI20:11
*** shuquan has quit IRC20:16
dimsrloo : "nonchalant" :)20:20
rloodims: you read it :)20:20
dimshaha, yes20:21
*** rbrud has joined #openstack-ironic20:21
*** jmccrory has quit IRC20:24
*** jmccrory has joined #openstack-ironic20:25
rloojroll, devananda|away: do we still want to hold the ironic-api meetings?20:25
*** d0ugal has quit IRC20:26
jrollrloo: when I asked about biweekly shortly before summit, deva said he hoped to have the content of those meetings get more interesting as we start working on things20:26
rloojroll: we went over the specs at the summit; just wondering if we can move things to the specs instead.20:27
jrollrloo: I would think so as well, just echoing from before summit20:27
*** d0ugal has joined #openstack-ironic20:27
*** d0ugal has quit IRC20:27
*** d0ugal has joined #openstack-ironic20:27
jrollI'd love to get that meeting off my list20:27
rloojroll: ok, let's see what devananda|away thinks20:27
rloojroll, devananda|away: not that I hate talking to you two ;)20:28
jroll:)20:28
*** causten has quit IRC20:30
*** causten has joined #openstack-ironic20:32
openstackgerritSukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers  https://review.openstack.org/36145120:34
jlvillaljroll: Is this you? Or was you?  http://jroll.ghost.io/20:40
jlvillalIt says the "free trial is over"20:40
jrolljlvillal: yeah, I decided not to renew, why?20:40
jlvillalI was going to read your blog :)20:40
jrollI do have all the things that were there20:40
jrollthere was two posts20:40
* jlvillal was hoping to see if you had written a write-up on the summit already.20:41
jrollmitaka summit summary, and newton summit summary, both of which I copy/pasted to the ML20:41
jrollno :(20:41
jlvillaljroll: Thanks!20:41
jrollnp20:41
*** godp1301 has joined #openstack-ironic20:46
*** rbrud has quit IRC20:53
*** e0ne has joined #openstack-ironic20:53
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications  https://review.openstack.org/35654120:55
*** Goneri has quit IRC20:57
*** causten has quit IRC20:58
*** e0ne has quit IRC20:59
*** causten has joined #openstack-ironic20:59
*** e0ne has joined #openstack-ironic21:00
*** godp1301 has quit IRC21:03
*** godp1301 has joined #openstack-ironic21:04
mariojvyuriyz: is that one ready for review ? ^21:05
*** ijw has joined #openstack-ironic21:08
*** ppiela has joined #openstack-ironic21:09
*** moshele has joined #openstack-ironic21:10
*** godp1301 has quit IRC21:14
*** godp1301 has joined #openstack-ironic21:15
*** moshele has quit IRC21:17
*** jkilpatr_ has quit IRC21:27
*** _mbelanger has quit IRC21:29
*** e0ne has quit IRC21:30
*** e0ne has joined #openstack-ironic21:38
*** ijw has quit IRC21:38
*** jcoufal_ has quit IRC21:41
*** trown is now known as trown|outtypewww21:44
*** baoli has quit IRC21:45
*** jheroux has quit IRC21:55
betherlyTheJulia: thanks so much for compiling that! i will take a look!21:55
*** jkilpatr has joined #openstack-ironic21:58
*** ijw has joined #openstack-ironic21:58
*** shuquan_ has joined #openstack-ironic22:01
*** ijw has quit IRC22:03
*** shuquan_ has quit IRC22:05
*** e0ne has quit IRC22:07
*** betherly has quit IRC22:08
*** srobert has quit IRC22:16
*** ijw has joined #openstack-ironic22:20
*** krtaylor has quit IRC22:22
*** ijw has quit IRC22:24
*** absubram has quit IRC22:26
*** fragatin_ has joined #openstack-ironic22:37
*** fragatina has quit IRC22:40
*** ijw has joined #openstack-ironic22:40
*** ijw has quit IRC22:48
*** bfournie has quit IRC22:51
*** baoli has joined #openstack-ironic23:11
*** ijw has joined #openstack-ironic23:12
*** ijw has quit IRC23:19
*** ijw has joined #openstack-ironic23:19
*** godp1301 has quit IRC23:24
rpiosoI'm submitting a bug and would like to include about 95 lines of logs in the report.  May that be directly copied into the 'Bug Description' field?  If not, what's the recommended approach?23:27
*** rbudden has quit IRC23:28
*** bfournie has joined #openstack-ironic23:38
*** causten has quit IRC23:50
*** godp1301 has joined #openstack-ironic23:51
*** godp1301 has quit IRC23:51

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