Wednesday, 2016-09-07

*** [1]rpioso has quit IRC00:10
TheJuliasahil: See http://docs.openstack.org/developer/ironic/_images/states.svg  (You can send "abort" if your still in clean-wait (or you can just wait for cleaning to timeout), and then send it to manageable state using "manage")00:15
*** mmedvede has quit IRC00:19
*** hoangcx has joined #openstack-ironic00:28
*** syed_ has quit IRC00:32
*** thrash is now known as thrash|g0ne00:40
*** wajdi has joined #openstack-ironic00:53
*** mmedvede has joined #openstack-ironic00:54
sahilThank you TheJulia00:55
sahilso I ironed out some issues that I figured out00:55
sahilone was incorrect gateway00:55
*** Sukhdev has joined #openstack-ironic00:56
sahilso here is the confusion00:56
sahilmy openstack VM has ip address x.x.x.1700:56
sahiland gateway is configured on x.x.x.1100:56
sahilx.x.x.11 is a bridge configured on the host00:57
sahilvia which the DEVSTACK VM is able to ping the bare-metal node00:57
sahilI configured the baremetal-node (installed OS manually) and pinged it from the devstack VM00:57
sahilit worked00:57
sahilbut after restacking it no longer can ping to the node00:58
sahilon checking I realized that devstack has used the gateway ip x.x.x.11 as one of the interface in the qrouter-<id>00:58
sahilis that correct configuration ?00:59
TheJuliasahil: so, as I understand it, restackinf is destructive, but I'm not a frequent devstack uset00:59
TheJuliaThat sounds correct routing wise with the use of neutron namespaces01:00
TheJuliaWell, really neutron using Linux kernel namespaces01:00
*** Sukhdev has quit IRC01:01
sahilif that sounds correct, then it is weird01:01
sahilbecause devstack used the ip address of an existing gateway to configure an interface on its router01:02
*** rama_y has joined #openstack-ironic01:08
sahilhere is my confusion01:10
sahilI am trying to setup devstack with flat network01:10
sahilwhere my VM and the baremetal I wish to boot using ironic are on the same vlan01:11
sahilso towards that end, I am using the following network parameters in the local.conf01:11
sahilHOST_IP=x.x.x.1701:11
sahilwhich is one of the ip on the VM01:12
sahilcorresponding interface is01:12
*** phuongnh has joined #openstack-ironic01:12
sahilPUBLIC_INTERFACE=eth001:12
sahilPHYSICAL_NETWORK=private PROVIDER_NETWORK_TYPE="flat"01:12
sahilNETWORK_GATEWAY=x.x.x.1101:13
*** xuao has joined #openstack-ironic01:13
sahilALLOCATION_POOL=start=x.x.x.21,end=x.x.x.3001:13
sahilwhat is wrong in this01:13
*** bfournie has joined #openstack-ironic01:16
*** sahil has quit IRC01:27
*** amotoki has joined #openstack-ironic01:29
*** amotoki has quit IRC01:33
openstackgerritTang Chen proposed openstack/python-ironicclient: Support i18n for osc plugin help msg  https://review.openstack.org/35763901:41
*** baoli has joined #openstack-ironic01:43
*** amotoki has joined #openstack-ironic01:46
*** jjohnson2 has joined #openstack-ironic01:49
*** ijw has joined #openstack-ironic01:58
*** ijw has quit IRC01:58
*** ijw has joined #openstack-ironic01:59
*** amotoki has quit IRC02:05
*** xuao has quit IRC02:07
*** ijw has quit IRC02:09
*** amotoki has joined #openstack-ironic02:20
*** baoli has quit IRC02:21
*** priteau has joined #openstack-ironic02:22
*** baoli has joined #openstack-ironic02:26
*** priteau has quit IRC02:27
*** amotoki has quit IRC02:28
*** hoangcx has quit IRC02:29
*** baoli has quit IRC02:35
*** amotoki has joined #openstack-ironic02:41
*** sabeen has quit IRC02:42
*** sabeen has joined #openstack-ironic02:45
*** amotoki has quit IRC02:48
*** hoangcx has joined #openstack-ironic02:53
*** vishwanathj has quit IRC02:54
*** rbudden has quit IRC02:57
*** Romanenko_K has joined #openstack-ironic03:12
*** vikrant has joined #openstack-ironic03:31
*** wuhg has joined #openstack-ironic03:31
*** sdake_ has joined #openstack-ironic03:32
*** sdake has quit IRC03:35
openstackgerritZhenguo Niu proposed openstack/ironic: Clean nodes stuck in CLEANING state when ir-cond restarts  https://review.openstack.org/34997103:41
*** fragatin_ has joined #openstack-ironic03:44
*** links has joined #openstack-ironic03:45
*** fragatin_ has quit IRC03:46
*** fragatina has quit IRC03:46
*** fragatina has joined #openstack-ironic03:53
*** fragatina has quit IRC03:57
*** appu103_ has joined #openstack-ironic03:58
*** appu103_ is now known as aparnav03:58
openstackgerritNguyen Phuong An proposed openstack/ironic: [api-ref] Stop supporting os-api-ref 1.0.0  https://review.openstack.org/36647304:01
*** vmud213 has joined #openstack-ironic04:07
*** sabeen has quit IRC04:11
*** rama_y has quit IRC04:12
*** amotoki has joined #openstack-ironic04:20
openstackgerritAnkit Kumar proposed openstack/proliantutils: Fix Inspection failure in Gen7  https://review.openstack.org/35713404:35
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643205:07
*** moshele has joined #openstack-ironic05:09
*** ChubYann has quit IRC05:12
*** jaosorior has joined #openstack-ironic05:12
*** rbartal has joined #openstack-ironic05:13
*** harlowja_at_home has joined #openstack-ironic05:18
*** openstackgerrit has quit IRC05:18
*** openstackgerrit has joined #openstack-ironic05:19
*** hoangcx has quit IRC05:20
*** hoangcx has joined #openstack-ironic05:25
*** Sukhdev has joined #openstack-ironic05:29
*** wuhg has quit IRC05:41
*** sdake_ is now known as sdake05:46
*** Sukhdev has quit IRC05:51
*** e0ne has joined #openstack-ironic05:52
*** harlowja_at_home has quit IRC05:54
*** mjura has joined #openstack-ironic06:01
*** rajinir has quit IRC06:05
*** sdake has quit IRC06:18
*** praneshp has quit IRC06:30
*** pcaruana has joined #openstack-ironic06:34
openstackgerritTuan Luong-Anh proposed openstack/ironic: Change $LOGFILE path in configuration devstack  https://review.openstack.org/36651806:39
*** stevemar has joined #openstack-ironic06:44
*** tangchen has joined #openstack-ironic06:52
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643206:54
*** devananda|travel has quit IRC07:00
*** fxpester has joined #openstack-ironic07:00
*** e0ne has quit IRC07:01
*** tesseract- has joined #openstack-ironic07:02
*** sdake has joined #openstack-ironic07:03
*** ohamada has joined #openstack-ironic07:10
openstackgerritYibo Cai proposed openstack/ironic: Support multi arch deployment  https://review.openstack.org/32263207:12
*** hshiina has joined #openstack-ironic07:22
*** jpich has joined #openstack-ironic07:24
*** bcornec has quit IRC07:28
openstackgerritVasyl Saienko proposed openstack/ironic: Update configure_tenant_networks  https://review.openstack.org/35646007:33
*** pas-ha has quit IRC07:37
*** pas-ha has joined #openstack-ironic07:37
pas-hamorning Ironicers :)07:38
*** ifarkas_afk is now known as ifarkas07:42
*** mbound has joined #openstack-ironic07:51
*** zzzeek has quit IRC08:00
*** sborkows has joined #openstack-ironic08:00
*** zzzeek has joined #openstack-ironic08:00
*** moshele has quit IRC08:01
*** moshele has joined #openstack-ironic08:01
tangchenmorning, all08:04
sborkowsmorning all08:04
tangchenmorning, pas-ha08:04
openstackgerritTang Chen proposed openstack/python-ironicclient: Support i18n for osc plugin help msg  https://review.openstack.org/35763908:08
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
xekmorning08:19
vmud213morning pas-ha,tangchen,sborkows,xek and everyone else08:21
*** lucas-dinner is now known as lucasagomes08:22
*** bcornec has joined #openstack-ironic08:22
openstackgerritNguyen Phuong An proposed openstack/ironic: [api-ref] Remove temporary block in conf.py  https://review.openstack.org/36647308:24
*** priteau has joined #openstack-ironic08:27
*** openstackgerrit has quit IRC08:34
*** openstackgerrit has joined #openstack-ironic08:34
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643208:38
*** derekh has joined #openstack-ironic08:40
*** e0ne has joined #openstack-ironic08:41
*** athomas has joined #openstack-ironic08:41
*** dtantsur|afk is now known as dtantsur08:46
dtantsurMorning Ironic08:46
pas-hamorning dtantsur08:46
*** hshiina has quit IRC08:47
*** links has quit IRC08:53
*** vikrant has quit IRC08:53
*** vikrant has joined #openstack-ironic08:54
*** priteau has quit IRC08:55
*** links has joined #openstack-ironic08:55
*** mbound has quit IRC08:56
aarefievmorning!08:58
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Test grenade  https://review.openstack.org/36660208:58
dtantsurmorning aarefiev08:59
aarefievmorning dtantsur, I'm trying  to play with timeouts, I still can't reproduce issue09:01
dtantsurugh09:02
Romanenko_Khi!09:05
lucasagomesdtantsur, pas-ha aarefiev Romanenko_K morning09:05
dtantsuro/09:06
aarefievmorning lucasagomes09:06
Romanenko_KPlease triage these two bugs about OSC plugin CLI: https://bugs.launchpad.net/python-ironicclient/+bug/1620737    https://bugs.launchpad.net/python-ironicclient/+bug/1620749  Do i have correct expectations regarding to CLI behavior here?09:10
openstackLaunchpad bug 1620737 in python-ironicclient "Malformed ports field in 'node create' and 'node show' OSC subcommands" [Undecided,New]09:10
openstackLaunchpad bug 1620749 in python-ironicclient "Lack of chassis_uuid field in OSC plugin baremetal node show output" [Undecided,New]09:10
dtantsurRomanenko_K, lemme check09:11
Romanenko_Kthnks!09:13
*** moshele has quit IRC09:21
*** moshele has joined #openstack-ironic09:21
*** moshele has quit IRC09:23
*** moshele has joined #openstack-ironic09:24
*** sborkows has quit IRC09:27
*** sborkows has joined #openstack-ironic09:27
jrollmorning y'all09:31
openstackgerritVasyl Saienko proposed openstack/ironic: Update configure_tenant_networks  https://review.openstack.org/35646009:33
lucasagomesjroll, morning09:34
lucasagomesthat's quite early dude :-)09:34
jrollyeah, woke up too early :P09:34
*** wznoinsk has quit IRC09:35
*** tuanla_ has joined #openstack-ironic09:36
tuanla_Hello ironic team09:40
dtantsurmorning jroll, tuanla_09:41
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Update openstack baremetal driver commands  https://review.openstack.org/36204709:44
*** sdake has quit IRC09:47
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Test grenade  https://review.openstack.org/36660209:47
*** wuhg has joined #openstack-ironic09:49
openstackgerritJim Rollenhagen proposed openstack/ironic: Log a warning for unsupported drivers and interfaces  https://review.openstack.org/36639909:52
jrolldo folks think ^ should have a release note, or the followup patches that deprecate drivers? (or both?)09:53
* jroll steps away for a bit09:54
*** hoonetorg has quit IRC09:58
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Update openstack baremetal driver commands  https://review.openstack.org/36204709:59
dtantsurjroll, I thought about that, but decided that we only need release notes for things that are actually visible10:00
dtantsuri.e. when we deprecate actual drivers10:01
lucasagomesjroll, I'm good with it w/o a release note10:10
*** phuongnh has quit IRC10:11
*** hoangcx has quit IRC10:13
*** athomas has quit IRC10:14
*** ohamada has quit IRC10:20
*** ohamada has joined #openstack-ironic10:20
*** mgould has joined #openstack-ironic10:22
*** athomas has joined #openstack-ironic10:23
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis.  https://review.openstack.org/34379610:26
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with chassis-update  https://review.openstack.org/34471510:26
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with node-list command  https://review.openstack.org/36054910:28
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with node-port-list.  https://review.openstack.org/36063610:29
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with node-get command  https://review.openstack.org/34206010:31
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Tests for testing port-create command  https://review.openstack.org/29180210:32
dtantsurlucasagomes, just got hit by https://bugs.launchpad.net/ironic/+bug/1621006 wdyt?10:38
openstackLaunchpad bug 1621006 in Ironic "Cleaning must not be allowed for nodes in maintenance" [Low,Triaged] - Assigned to Dmitry Tantsur (divius)10:38
lucasagomesdtantsur, :-O +2!10:38
lucasagomesIMHO ironic should not touch such nodes10:39
dtantsuryeah... it happened just now: I haven't noticed that my nodes are in maintenance and was waiting for cleaning to finish...10:39
dtantsurwill fix today10:39
mgouldshould we disallow all state transitions for nodes in maintenance?10:40
mgouldalso, morning everyone!10:40
dtantsurmgould, this is exactly what I was going to ask10:40
jrollhm, there was a reason we did it that way... I believe it was so we could boot into an agent so ops could investigate and then kick out of maintenance if fixed up10:40
dtantsuri.e. deploy will fail the same manner10:40
jrollbut I'm not 100% sure that was why10:41
dtantsurjroll, rescue? because deploy won't work as well10:41
jrollwe don't allow a transition to deploy in maintenance10:41
jrolldtantsur: what rescue? :)10:41
jrollrescue is only valid for ACTIVE anyway10:41
dtantsurthe very rescue we're going to have soon and *somebody* already has downstream ;)10:41
jroll:P10:41
dtantsurhmm, right10:42
dtantsuranyway, the way we have it now, cleaning won't work, so I'm inclined to ban at least "provide" and "clean" transitions10:42
jrollpossibly10:43
jrollmight be worth talking to deva or JayF10:43
jrollsee if they can recall10:43
jrollor even ruby10:43
dtantsurI'll put up a patch and we'll see...10:43
jrollfair enough10:44
jrollplease mark it in upgrade notes so I notice we should bump major semver version, it's a breaking change10:44
dtantsurjroll, why do you think it is breaking? cleaning already does not work in maintenance, we'll just make it explicit10:45
jrolldtantsur: the transition does work, I can tell you that's something we use while operating downstream10:46
jrollit boots IPA and waits for intervention10:47
jrollelse you must boot it manually, which is somewhat painful (because networking/dhcp/etc)10:47
dtantsurjroll, hmm, I guess we need to expose the boot interface in API...10:47
jrollit's breaking because set-provision-state provide|clean used to be a 202, now is a 400 or whatever (no we should probably not microversion this if we change it)10:48
jrollheh, maybe10:48
dtantsurjroll, btw do you use "provide" or "clean" downstream?10:49
jrolldtantsur: provide, we don't typically do manual cleaning steps except in dev/test10:49
dtantsurI guess "clean" might make sense, OOB cleaning will work10:50
dtantsurah, provide...10:50
jrollwe're at the point where when something goes wrong that isn't handled automatically, we want to get on the box and investigate10:50
dtantsurjroll, I wonder if we need to allow rescue in all states10:51
dtantsur* future rescue :)10:51
jrollmmmmm10:51
jrollthat'd be an interestign conversation10:52
jrollI'd almost rather have: openstack baremetal node boot-me-an-agent10:52
dtantsur++ I would have it too10:52
dtantsurwhich is nearly in-band introspection btw ;)10:53
*** sambetts has joined #openstack-ironic10:53
sambettsMorning everyone10:53
jrollheh10:53
jrollhey sambetts :)10:53
dtantsurmorning sambetts10:54
sambettsif anyone's pinged me you'll need to repeat please because I'm on webchat as my gfx card on my PC has blown up :(10:54
jroll:/10:55
sambettshey jroll, dtantsur10:55
sambettsjroll: yeah... no idea what happened, its just freaked out10:56
sambettsits an old card though10:56
dtantsuraarefiev, can our grenade problems be related to some networking conflict with infra?10:59
dtantsurthis is the only reason I can think of why ironic itself is not affected and it can't be reproduced locally...10:59
mgouldmorning sambetts!11:01
sambettshey mgould11:01
mgouldanything exciting happen while I was gone?11:03
aarefievdtantsur: yeah, it's strange, it fails on ssh timeout when trying to ping gateway from instance11:04
openstackgerritTuan Luong-Anh proposed openstack/ironic: Change $LOGFILE path in configuration devstack  https://review.openstack.org/36651811:05
dtantsurmgould, largely, a few releases. I've also finished bringing overcloud ironic to tripleo.11:05
dtantsurmgould, now ironic-inspector is close to finishing the feature work, but we got our grenade broken in a strange way11:06
*** slagle has joined #openstack-ironic11:06
*** amotoki has quit IRC11:06
mgoulddtantsur: 1) yay! 2) yay! 3) yay! 4) d'oh!11:06
mgouldso grenade is currently broken?11:06
dtantsuryep (for inspector only, apparently)11:07
lucasagomesjroll, dtantsur boot-me-an-agent would be super handy11:07
dtantsursambetts, we've been discussing https://bugs.launchpad.net/ironic/+bug/162100611:08
openstackLaunchpad bug 1621006 in Ironic "Cleaning must not be allowed for nodes in maintenance" [Low,Triaged] - Assigned to Dmitry Tantsur (divius)11:08
lucasagomesand using with nova we could just use the same mechanism for rescue11:08
lucasagomeswhen the instance is active11:08
jrollwhen is grenade supposed to run tempest after the upgrade? I'm not seeing it in ironic's grenade job... http://logs.openstack.org/99/366399/2/check/gate-grenade-dsvm-ironic/a637018/logs/grenade.sh.txt.gz11:08
mgouldjroll: I think it gets run elsewhere in the grenade script11:09
dtantsurjroll, it's in main console.log11:09
jrollahhhhh, ok11:09
jrollthanks11:09
*** tuanla_ has quit IRC11:11
sambettsthe bug makes sense to me, and I like the idea of being able to boot the node with a agent while in maintenance for all sorts of debugging and manual maintence stuff11:12
mgouldjroll: looks like the gate script runs tempest after grenade.sh has finished: https://review.openstack.org/#/c/346923/1/grenade.sh11:14
patchbotpatch 346923 - openstack-dev/grenade - Document TARGET_RUN_SMOKE flag11:14
jrollmgould: right, which is why it's on console.log11:15
mgouldjroll: ah, sorry11:15
jrollmgould: no, thank you for pointing it out :)11:16
*** baoli has joined #openstack-ironic11:17
*** lucasagomes is now known as lucas-hungry11:20
*** thrash|g0ne is now known as thrash11:25
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643211:26
*** sambetts has quit IRC11:32
*** skazi has quit IRC11:32
*** sambetts|afk is now known as sambetts11:32
sambettsmanaged to squeeze a little bit of life out of my gfx card while I wait for my new one to arrive tomorrow11:33
mat128hi group!11:34
jrollmorning mat128 :)11:35
mgouldmorning mat128!11:36
*** amotoki has joined #openstack-ironic11:37
sambettshey mat12811:39
TheJuliaGood morning11:47
ricardoasmorning!11:48
dtantsurmorning mat128, TheJulia, ricardoas11:48
openstackgerritSam Betts proposed openstack/ironic-python-agent: Include IPA Version during heartbeat  https://review.openstack.org/36483411:49
*** amotoki has quit IRC11:50
ricardoasmorning dtantsur :)11:50
*** bfournie has quit IRC11:54
*** skazi has joined #openstack-ironic11:54
*** baoli_ has joined #openstack-ironic11:54
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Hide 'ports' field from node OSC subcommands output  https://review.openstack.org/36670311:57
*** baoli has quit IRC11:57
*** amotoki has joined #openstack-ironic11:58
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Update openstack baremetal driver commands  https://review.openstack.org/36204712:01
*** trown|outtypewww is now known as trown12:02
*** wajdi has quit IRC12:04
*** wajdi has joined #openstack-ironic12:04
*** links has quit IRC12:04
*** wajdi has joined #openstack-ironic12:05
mgouldmorning TheJulia, ricardoas12:05
*** chlong has joined #openstack-ironic12:05
*** devananda has joined #openstack-ironic12:06
*** devananda is now known as devananda|travel12:06
*** amotoki has quit IRC12:07
vsaienk0jroll: https://review.openstack.org/#/c/366399/2/ironic/common/driver_factory.py this will generate a warning for drivers from ironic-staging-drivers, I think we should skip them.12:07
patchbotpatch 366399 - ironic - Log a warning for unsupported drivers and interfaces12:07
ricardoasmorning mgould12:08
jrollvsaienk0: why would it? the default in the base class is supported=True12:08
jrollvsaienk0: I made True the default, because we can't control out-of-tree things, so no reason to warn for them12:08
vsaienk0jroll: right! I've missed that supported is True by default12:10
jroll:)12:10
sambettsjroll: I like the idea of warning if someone tries to load any unoffical driver, although I guess they can just set supported=True if they really wanted :/12:12
jrollsambetts: yeah, like I said, no control12:12
jrollpresumably if they've installed and used an unofficial driver, they know what that means (or their vendor does)12:13
sambettsyeah12:13
jrollbbiab12:15
*** links has joined #openstack-ironic12:16
*** hoonetorg has joined #openstack-ironic12:23
*** lucas-hungry is now known as lucasagomes12:23
openstackgerritSam Betts proposed openstack/ironic: Receive and store agent version on heartbeat  https://review.openstack.org/36486112:23
*** bfournie has joined #openstack-ironic12:27
*** dprince has joined #openstack-ironic12:27
*** rbudden has joined #openstack-ironic12:30
*** amotoki has joined #openstack-ironic12:33
lucasagomesno open patches in ironic-lib :-)12:36
*** mmedvede has quit IRC12:36
lucasagomesjroll, ^ btw, can we get a release of it soon-ish ?12:36
*** wajdi has quit IRC12:42
sambettslucasagomes: Do you know if I want to merge a change into nova, that depends on an ironic client change, if we have to release Ironic client or if I can use depends-on?12:43
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Hide 'nodes' field from chassis OSC subcommands output  https://review.openstack.org/36673112:43
lucasagomessambetts, you likely to need a release, unless the code in nova knows how to handle old and new versions of the client lib12:44
*** vikrant has quit IRC12:47
lucasagomes(even tho nova is not sctrictly dependent on the ironicclient, but I think we should be able to say: "the nova ironic driver now depends on client verison X.Y.Z"12:47
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Test grenade  https://review.openstack.org/36660212:47
lucasagomes)*12:47
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643212:47
*** mgoddard_ has joined #openstack-ironic12:48
*** jaosorior has quit IRC12:49
*** jaosorior has joined #openstack-ironic12:50
*** Goneri has joined #openstack-ironic12:51
*** wuhg has quit IRC12:51
*** mgoddard has quit IRC12:52
*** mjturek has joined #openstack-ironic12:52
sambettslucasagomes: :( its annoying because I can't use the CI to test my changes because the CI doesn't read the depends-on :(12:59
lucasagomessambetts, ouch :-/13:01
lucasagomesthat can be a PITA indeed13:01
*** mmedvede has joined #openstack-ironic13:02
lucasagomessambetts, so the change in the client is already merged in master ?13:03
lucasagomesand we just need a release ?13:03
sambettslucasagomes: no, its not merged yet, I have a change that requires changes in nova, client, and ironic itself, and I wanted to run an end to end test in the CI to ensure it all worked when put together in the CI13:05
sambettsso I've pushed commits with the nova patch depends on the client patch and the client patch depends on my Ironic change13:05
*** leitan has joined #openstack-ironic13:06
openstackgerritSam Betts proposed openstack/ironic: Add Network Interface Attach and Detach APIs  https://review.openstack.org/32704613:06
sambettslucasagomes: I think if we had ironicclient from source enabled in the CI it would work, but I assume we don't do that for a reason13:08
lucasagomessambetts, yeah, the only place we should test ironicclient from source is in the ironicclient gate itself13:09
lucasagomesthat said, I'm in a similar situation but less complex waiting a release from ironic-lib to make changes in IPA and ironic to use the root device hints thingy (which I'm code atm)13:10
sambettslucasagomes: I guess I could put up a fake patch into ironicclient to test it, make that fake patch depend on the nova, client and ironic changes, then the client from  source test should test it all together right?13:11
*** vishwanathj has joined #openstack-ironic13:11
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add support for root device hints with operators  https://review.openstack.org/36674213:12
*** links has quit IRC13:12
lucasagomessambetts, it should pull the depends-on changes yes13:12
openstackgerritSam Betts proposed openstack/python-ironicclient: Add python API for node interface attach/detach  https://review.openstack.org/36442013:13
*** vmud213 has quit IRC13:14
openstackgerritSam Betts proposed openstack/python-ironicclient: DO NOT MERGE Testing Client Changes with Nova  https://review.openstack.org/36674713:16
*** trown is now known as trown|brb13:17
*** aparnav has quit IRC13:18
openstackgerritSam Betts proposed openstack/ironic: Receive and store agent version on heartbeat  https://review.openstack.org/36486113:19
*** hoangcx has joined #openstack-ironic13:19
*** rloo has joined #openstack-ironic13:24
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add storage_interface DB field and object  https://review.openstack.org/34800513:26
*** ohamada has quit IRC13:27
*** trown|brb is now known as trown13:28
*** jcoufal has joined #openstack-ironic13:28
*** mgoddard_ has quit IRC13:30
*** mgoddard has joined #openstack-ironic13:30
jrolllucasagomes: yeah, I can post one - I'm not sure when ocata opens for library releases but I can put the request up13:33
*** rloo has quit IRC13:33
jrollsambetts: vsaienk0 did an interesting thing with a dummy patch to devstack-gate to pull in an ironicclient patch, you might ask him13:33
*** hamzy has quit IRC13:34
lucasagomesjroll, right on, thanks a lot13:34
openstackgerritSam Betts proposed openstack/python-ironicclient: DO NOT MERGE Testing Client Changes with Nova  https://review.openstack.org/36674713:35
sambettsjroll: hoping this will work ^13:35
lucasagomesin any case I will just put up the patches for IPA (already up) and Ironic as well13:35
vsaienk0jroll, sambetts: the patch actually was to ironic https://review.openstack.org/#/c/296432/65/devstack/lib/ironic13:35
patchbotpatch 296432 - ironic - DO NOT REVIEW13:35
lucasagomesso we can get the feature as soon as ocata opens13:35
sambettsvsaienk0: ah I see, if my ironicclient patch doesn't work I'll try that method instead13:35
jrolllucasagomes: we just need a y bump in x.y.z for ironic-lib, yes? didn't break anything?13:36
jrollvsaienk0: ah, cool13:36
jrollthanks13:36
lucasagomesjroll, yeah just a minor13:36
jrollk13:36
* lucasagomes added x.y.z in the patch because he didn't remember the current version :D13:36
jrolllucasagomes: eh? which patch?13:37
lucasagomesjroll, https://review.openstack.org/36674213:38
patchbotpatch 366742 - ironic-python-agent - Add support for root device hints with operators13:38
jrollah, lol13:38
lucasagomesI thought you looked at it heh13:38
jrollit'll be 2.2.0 btw13:38
lucasagomesfair, I will update the patch after the release, don't need to waste more gate resources cause I know it will fail now (the function is not present in the utils.py module from ironic-lib)13:39
* lucasagomes updates locally13:39
jrolllucasagomes: I'm going to -2 that so we don't accidentally merge in newton, remind me to undo it when ocata is open :)13:41
lucasagomesjroll, ++, want me to -2  it ?13:41
jrolloh yeah, go for it13:41
jrollthen I won't be blocking you13:41
lucasagomesyeah13:41
lucasagomesdone13:41
jrollthanks13:42
*** sabeen has joined #openstack-ironic13:42
*** hoangcx has quit IRC13:42
*** skazi has quit IRC13:42
*** krtaylor has joined #openstack-ironic13:42
*** jvgrant_ has joined #openstack-ironic13:43
devananda|travelmorning, all13:44
jrollohai devananda|travel13:45
*** hoangcx has joined #openstack-ironic13:45
*** sabeen1 has joined #openstack-ironic13:46
lucasagomesdevananda|travel, morning ;-)13:46
*** joprovost has joined #openstack-ironic13:46
*** sabeen has quit IRC13:48
jrolllucasagomes: apparently ocata releases will open after g-r gets branched, around sept 26 or maybe a bit later depending on things13:49
dtantsurmorning devananda|travel13:51
*** michauds has joined #openstack-ironic13:52
*** rloo has joined #openstack-ironic13:54
lucasagomesjroll, thanks for the info! It's a bit far tho, but ¯\_(ツ)_/¯13:55
jrollrloo: since I'm bad at writing things, do you mind reviewing at least the warning log here? https://review.openstack.org/#/c/366399/2/ironic/common/driver_factory.py13:56
patchbotpatch 366399 - ironic - Log a warning for unsupported drivers and interfaces13:56
jrolllucasagomes: heh, yeah you can hack our devstack plugin to pull in master13:56
*** skazi has joined #openstack-ironic13:56
rloojroll: sure. but i want you to know, that I find it painful to write English.13:56
lucasagomesjroll, yeah, that's how I test it :-)13:56
lucasagomesno rush13:56
jrolllucasagomes: cool13:57
lucasagomesty13:57
*** hj-hpe has joined #openstack-ironic13:57
*** rcernin has joined #openstack-ironic13:57
jrollrloo: you're very good at it, so I'm grateful when you do. you don't have to on this patch if you don't want to :)13:57
rloojroll: do we have an idea as to when we might remove them?13:58
rloojroll: or at least, the earliest time frame that we might do so?13:58
*** wajdi has joined #openstack-ironic13:59
rloojroll: i'm actually wondering if we can put some value in the supported flag, to indicate the earliest that it might be deleted13:59
lucasagomesrloo, I would prefer to leave it open without a hard date13:59
jrollrloo: well, 3 months from landing the deprecation, at a minimum. but, they might not be removed (if someone adds CI), and this is meant to be continuous.13:59
jrollI'd also prefer to leave it open, I think it will vary by driver14:00
mat128rloo: then people will set the date back on their computers to run unsupported drivers :)14:00
lucasagomesit may happen that somoene that is depending on that driver see the message and propose {him, her}self to maintain it14:00
rloojroll: i was thinking 'the earliest that this may be removed is ...'.14:00
*** anush has joined #openstack-ironic14:00
rloowhat are operators meant to do when they see this warning?14:00
rlooam also wondering how we will deal with this with the new driver composition, but we can deal with that later.14:01
jrollpanic and flail wildly14:01
jrollI put it on interfaces as well for that reason14:01
rloojroll: no worries, we can handle it :)14:01
*** joprovost1 has joined #openstack-ironic14:02
*** joprovost has quit IRC14:02
*** joprovost1 is now known as joprovost14:02
*** narasimha_SV has joined #openstack-ironic14:03
rloojroll: commented.14:04
rloojroll: are we allowed to deprecate AND unsupport? I mean, is that ok with the 'follows deprecation' tag or whatever it is.14:04
dtantsurmorning rloo14:04
jrollthanks rloo14:04
rloohi and good morning dtantsur, jroll, mat128, lucasagomes and everyone else!14:05
mat128o/14:05
jrollrloo: yes, 'follows deprecation' implies that a valid config for cycle X is also valid for X+1, if there's no deprecation warnings14:05
lucasagomesrloo, morning14:05
rloojroll: ok, nothing about having to support it?14:05
jrollrloo: nope14:06
rloojroll: good!14:06
jroll:)14:06
*** cdearborn has joined #openstack-ironic14:06
mgouldmorning rloo14:08
rloohi mgould14:08
narasimha_SVwhile using pxe_ipmitool , after creating a node when I check hepervisor-stats it is showing 1 node but I am not able to see all the other hardware details ?14:09
narasimha_SVis this expected result ??14:10
narasimha_SVhttp://paste.openstack.org/show/567476/14:11
lucasagomesnarasimha_SV, how many nodes do you have ? And what's the value of "count" in the hypervisor-stats command ?14:11
* lucasagomes looks14:11
narasimha_SVi have added all the details about his in above link14:12
narasimha_SVhttp://paste.openstack.org/show/567476/14:12
lucasagomesnarasimha_SV, does your node have all the properties/ set ?14:12
lucasagomescpus, local_gb, memory_mb and cpu_arch14:12
jrollnarasimha_SV: depending on the state of the node, there might be no resources presented - your node looks ok, but if you've just changed power or provision state (or ironic just changed it), you may need to wait a few minutes for nova to pick up the resources14:13
lucasagomes^ yeah it's also not instanteneous, the changes may take a while to be propagated from ironic to nova14:13
narasimha_SVjroll: its been more than 2 hours14:13
jrollok14:13
* jlvillal wonders if this really is opensource video conferencing: https://meet.jit.si/ 14:13
jrollnarasimha_SV: so yeah, can you show a full node-show?14:14
mat128jlvillal: it is14:14
jlvillalThinking about next virtual mid-cycle14:14
mat128jlvillal: I tried it, you can self-host by simply running their app14:14
mat128it's very similar to appear.in, but open source14:14
openstackgerritJim Rollenhagen proposed openstack/ironic: Log a warning for unsupported drivers and interfaces  https://review.openstack.org/36639914:14
jlvillalmat128, Cool :)  And you are fast in responding!!!14:14
jrollrloo: dtantsur: lucasagomes: jlvillal: ^ updated log message14:14
mat128jlvillal: I was looking for video conferencing products for my team14:14
jrolljlvillal: I use jitsi to connect to pbx.o.o, it isn't terrible14:14
mat128jlvillal: current solutions all peg my CPU and drain my laptop battery14:14
mat128jroll: did you get video to work with jitsi?14:15
jlvillaljroll, Right. This is a video version of it. I guess14:15
mat128I mean, the fat app14:15
mat128jlvillal: the jitsi app supposedly allows video conferencing on jingle-compatible XMPP servers14:15
jrollmat128: not sure pbx.o.o supports it, I haven't tried though14:15
jrolljlvillal: yeah, seems neat14:15
mat128jlvillal: but I have never got it to work14:15
narasimha_SVhttp://paste.openstack.org/show/567477/14:15
mat128meet.jit.si is WebRTC so that works for most people, as long as you can access the page14:16
lucasagomesjroll, thanks14:16
mat128jlvillal: https://github.com/jitsi/jitsi-meet14:16
lucasagomesnarasimha_SV, | properties             | {u'cpu_arch': u'x86_64'}14:16
rloojroll: sorry, I'm actually looking at the patch now. can we distinguish interfaces from drivers?14:16
lucasagomesnarasimha_SV, that's the only property you have14:16
jrollnarasimha_SV: ok right, so you need cpus, memory_mb, and local_gb in properties, that's what tells nova what resources are available14:17
rloojroll: or is it obvious when the 'driver' name is logged, that it is a driver or an interface?14:17
lucasagomesnarasimha_SV, you need to specify the number of cpus (cpus), memory_mb, local_gb (disk size in GB)14:17
jrollrloo: we can, in the subclasses, I could split that out if you like. I hate the name interfaces personally but ¯\_(ツ)_/¯14:17
lucasagomesthen it will appear in the nova command14:17
narasimha_SVok thanks14:17
narasimha_SVi will try14:17
*** hoangcx has quit IRC14:17
rloojroll: well, we should have agreed on the naming before. or now. at least before the driver composition reform is done. or maybe it is too late. dunno.14:18
jrollrloo: I mean, it's fine, both called drivers would be too confusing :)14:18
rloojroll: so i'll leave it up to you to change that log msg to distinguish drivers from interfaces. sorry, i only looked at the string before.14:19
jrolldtantsur: mr. driver composition, what do you think? ^14:19
*** hoangcx has joined #openstack-ironic14:20
*** nicodemos has quit IRC14:20
*** cinerama has quit IRC14:20
*** yibo has quit IRC14:20
*** anteaya has quit IRC14:20
*** sambetts has quit IRC14:20
*** stevebaker has quit IRC14:20
*** Guest40699 has quit IRC14:20
* dtantsur runs away14:20
dtantsursorry, I don't quite get the question14:21
*** sabeen has joined #openstack-ironic14:21
*** sabeen1 has quit IRC14:21
*** aslezil has joined #openstack-ironic14:21
TheJuliaslightly more context might help14:21
jrolldtantsur: that driver deprecation log, should we have different messages for drivers vs. interfaces?14:22
narasimha_SVjroll,lucasagomes: if I wont add those details will there be an issue ??14:22
narasimha_SVlike scheduler wont be able to get this node added ?14:22
lucasagomesnarasimha_SV, yes, you won't be able to deploy the node :-)14:22
jrollnarasimha_SV: that's how nova knows what resources the node has, and it is how it matches a flavor to a node14:22
dtantsurjroll, we deprecate drivers, no?14:22
jrollso if your flavor has cpus:0 and memory_mb:0 and local_gb:0, maybe it will work :)14:22
narasimha_SVok14:22
jrolldtantsur: I think one day we may deprecate interfaces, though (or maybe just hardware types?)14:23
lucasagomesjroll, I think we use things like local_gb when partitioning the disk14:23
jrollah, true14:23
lucasagomesworth trying, but I don't think it will work14:23
lucasagomeswe check if the image fits the disk and things like that14:23
lucasagomesif the disk is 0 it will fial14:23
lucasagomesfail*14:23
jrollyep14:23
openstackgerritImre Farkas proposed openstack/ironic: Fix nits on DRAC OOB inspection patch  https://review.openstack.org/36678514:24
dtantsurjroll, let's start with deprecating drivers :) right now we don't have interfaces independent of drivers14:24
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643214:25
jrolldtantsur: well, driver factory handles both, but I can make it just do drivers? or leave as is and promise not to mark interfaces unsupported14:25
rloodtantsur, jroll: we actually have those network interfaces that are independent of drivers. I commented on that; not sure how your patch deals with those.14:25
*** fxpester has quit IRC14:26
jrolltbh I was going to mark noop network as unsupported, given we don't test it (but probably not remove it anytime soon)14:26
dtantsurhuh14:27
dtantsurI'd wait for driver composition before that, just to avoid too much confusion14:27
jrollsure14:27
*** lucasagomes is now known as lucas-afk14:29
TheJuliaThis makes me think we really ought to have some sort of testing wrapped around noop... but it is such a foreign thing that we might as well just trigger bifrost's ci job at that point.14:30
jrollI'd be +1 on a bifrost job on ironic14:31
*** syed_ has joined #openstack-ironic14:32
dtantsurmoar jobs \o/14:32
* dtantsur pushes for a lightweight tripleo job for quite some time as well14:33
TheJuliadtantsur: What do you mean by lightweight?  (Admittedly, I'm totally out of touch with tripleo's current... well... mostly everything since it has changed so much in the last two years.)14:35
dtantsurTheJulia, I mean, avoid doing the whole 1.5 hrs deployment :) e.g. they have multinode jobs now, which are substantially faster14:37
TheJuliaThe only concern that I would truly have is just memory limitations in a single CI node.14:38
*** rpioso has joined #openstack-ironic14:38
*** nicodemos has joined #openstack-ironic14:42
*** cinerama has joined #openstack-ironic14:42
*** yibo has joined #openstack-ironic14:42
*** anteaya has joined #openstack-ironic14:42
*** sambetts has joined #openstack-ironic14:42
*** stevebaker has joined #openstack-ironic14:42
*** Guest40699 has joined #openstack-ironic14:42
*** hoonetorg has quit IRC14:43
TheJuliadtantsur: but ultimately, moar is better :)14:43
dtantsuryeah :) we'll see14:44
TheJulia:)14:45
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Drastically reduce list of actions allowed for nodes in maintenance  https://review.openstack.org/36679314:45
dtantsurjroll, JayF, ^^^14:45
openstackgerritJim Rollenhagen proposed openstack/ironic: Log a warning for unsupported drivers and interfaces  https://review.openstack.org/36639914:46
*** hamzy has joined #openstack-ironic14:46
dtantsurTheJulia, re https://review.openstack.org/366793: not sure if we need to allow adopt there...14:46
patchbotpatch 366793 - ironic - [WIP] Drastically reduce list of actions allowed f...14:46
TheJuliadtantsur: wow that is a context switch for my brain14:47
jrollrloo: not sure what you mean by "Did you test to see what happens if 'flat' can't be loaded?" do you want me to add a test there? did you mean if 'flat' is unsupported? is there any part of that code you'd like me to explain (how interfaces are loaded etc)14:47
*** hoangcx has quit IRC14:48
rloojroll: sorry, i added another comment later. initially, i wasn't quite sure if it worked with interfaces.14:48
dtantsurTheJulia, sorry :)14:48
rloojroll: although i guess we should add a test for interface too14:48
*** moshele has quit IRC14:49
jrollrloo: I'll just make sure _warn_if_unsupported is called for the base case there, cool?14:50
rloojroll: yeah, that should do it14:50
*** mkam has joined #openstack-ironic14:51
*** sborkows has quit IRC14:51
TheJuliadtantsur: I'm honestly not sure.  Off hand, I think allowing retries makes sense, but at the same time I'm struggling to think of a case where a user would be stuck in that kind of situation.  But perhaps, the answer is to process the heartbeats.  Maintenance state and delete a node kind of makes sense, and seems like something a user might end up getting14:52
TheJuliathemselves into kind of easily.14:52
jrolloh yeah, we definitely need to allow the delete verb14:53
dtantsurjroll, TheJulia, yeah, but deleting a node implies cleaning14:53
dtantsurwhich does not work :)14:53
jrollelse a bmc going out means the user is billed until the operator fixes it14:53
TheJuliadtantsur: not if cleaning is disabled :)14:53
dtantsurTheJulia, well, yes, but it's enabled by default14:53
jrollI'm absolutely -1 on not allowing delete14:54
*** mbound has joined #openstack-ironic14:54
dtantsurjroll, then how do you suggest to make it work?14:54
jrollnova users shouldn't need to care if a node is in maintenance14:54
dtantsurbecause you're not -1 on not allowing it, you're +1 on fixing it ;)14:54
TheJuliajroll: ++14:54
dtantsuror should we get back heartbeats processing?14:55
jrolldtantsur: erm, I'm -1 on breaking delete verb while in maintenance14:55
dtantsurjroll, nobody breaks it, it's broken. now.14:55
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643214:55
dtantsurjroll, unless you disable the whole cleaning, which we don't recommend that users do.14:55
jrolldtantsur: from the nova user perspective, it isn't broken - your patch would break it14:55
*** amotoki has quit IRC14:56
jroll"why'd my nova delete fail?" "the bmc is broken" "why do I care? I'm not paying for something I don't want"14:56
dtantsurjroll, well, fair. if we're ok with these nodes hanging in clean wait forever...14:56
dtantsurso, now I think we should just allow heartbeats and make cleaning (try to) work..14:56
TheJuliaor just let cleaning fail14:56
jrolldtantsur: well, no, we shouldn't do any actions on the machine while in maintenance14:57
*** hoonetorg has joined #openstack-ironic14:57
TheJuliaif a node is in maintenance, there should be an alarm or an indicator in an operator's monitoring system14:57
JayFdtantsur: wdyt about allowing delete in maintenance14:57
jrolldidn't I mention why we chose to do this earlier today? :)14:57
JayFdtantsur: imo we have to allow deleteing in maint14:57
*** hoangcx has joined #openstack-ironic14:57
dtantsurJayF, I'm fine with that, as soon as we're fine with it resulting on clean fail after the timeout14:57
JayFah, I see jroll in scrollback now, lol14:57
JayFdtantsur: very fine with it14:57
JayFdtantsur: as you can tell, it already happens today :)14:57
dtantsurokie, you've convinced me14:58
*** rbrud has joined #openstack-ironic14:58
jrollwell, it doesn't fail today14:58
jrollit waits14:58
jroll10:40:49            jroll | hm, there was a reason we did it that way... I believe it was so we could boot into an agent so ops could investigate and then kick out of maintenance if fixed up14:58
JayFand then it times out14:58
jrollI still believe that's useful14:58
jrollit does not14:58
*** mkam has quit IRC14:58
JayFoh? interesting14:58
jrollthe heartbeat touches the updated at or whatever14:58
JayFEither way, the use case is: I have a semi-broken node, that's got an instance -> I want to fix it, but can't disrupt the customer -> on customer delete, troubleshoot14:59
JayFlike we've used a maint-while-active workflow for nodes that had reported hardware failures from a customer14:59
jrollright14:59
*** mjura has quit IRC14:59
jrollJayF: so wdyt is more useful while a thing is in maint, booting into an agent and heartbeating while we wait, or failing cleaning altogether14:59
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Drastically reduce list of actions allowed for nodes in maintenance  https://review.openstack.org/36679315:00
*** hoangcx has quit IRC15:01
dtantsurjroll, wdym "it doesn't fail"? cleaning won't happen, so it will timeout eventually...15:02
jrolldtantsur: no, the heartbeat updates the field that the timeout checks15:02
dtantsurjroll, I mean cleaning timeout15:02
jrollI do too15:02
dtantsurhmm, ok, so it hangs forever :)15:02
jrollunless something has changed15:02
jrollyeah, looks like it should work15:03
dtantsurwell, it's even worse then, we're stuck in "clean wait"15:03
jrollI guess I don't understand why that's so bad15:04
jrollmaybe a bit confusing15:04
*** hoangcx has joined #openstack-ironic15:04
openstackgerritCaio Oliveira proposed openstack/ironic: [WIP] Custom power sync  https://review.openstack.org/33297815:05
dtantsurjroll, 1. start cleaning for node in maintenance, 2. it gets in "clean wait", 3. and stays there forever15:05
dtantsurjroll, because of https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L474-L47815:05
jrolldtantsur: I understand what happens now, and why15:05
jrolldtantsur: why is booting an agent and waiting, worse than failing altogether?15:05
dtantsurconfusing for users?15:06
jrollright, so it's a bit confusing, we can probably fix that15:06
jrollother than that it seems fine to me, but I may be missing something15:06
openstackgerritJim Rollenhagen proposed openstack/ironic: Log a warning for unsupported drivers and interfaces  https://review.openstack.org/36639915:08
dtantsurjroll, another approach will be in https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L474-L478 to fail cleaning15:08
dtantsur(ditto for deploying)15:08
jrolldtantsur: yes, I understand the options, we just talked about that15:09
jrolllike I said, there's a valid use case for getting an agent booted and waiting15:09
jrollso it's a matter of if that use case is worth making this less confusing, or if we should just kill that use case15:09
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Test grenade  https://review.openstack.org/36660215:09
dtantsuryes, but it's not what we call cleaning. unless we document this behavior and state that it's desired15:10
*** thrash is now known as thrash|biab15:10
jrollright, we can make it less confusing15:10
dtantsurthe reason I don't like it is because maintenance happen automatically15:10
dtantsurso today I just didn't notice that I got into it. I spent some time wondering why cleaning hangs.15:10
jrollsure15:11
jrollthere's a log there, did that not help?15:11
*** mkam has joined #openstack-ironic15:11
dtantsurI didn't get to logs at that moment15:12
dtantsuranyway, it's a bit15:12
dtantsur* a bit cruel IMO15:12
dtantsurbut we can document it as a feature and leave it like that15:13
dtantsur"Cleaning works as cleaning for nodes not in maintenance, and just boots IPA and waits for nodes in maintenance"15:13
jrollI mean, I'm not trying to tell you you're wrong, I'm just saying that we originally did it this way for a reason, and we can either document it correctly or kill that use case :/15:14
dtantsurthat's the question :) wanna me to start a ML thread/bring it to the meeting?15:15
*** narasimha_SV has quit IRC15:15
jrolldtantsur: sure, if you like, I can dig up old conversation or whatever15:16
*** rloo has quit IRC15:16
dtantsuryes please15:16
* jroll wishes eavesdrop.o.o was in elastic search with a retention time of forever15:17
sambetts++15:17
mgould++15:18
*** praneshp has joined #openstack-ironic15:23
TheJulia++15:24
*** lucas-afk is now known as lucasagomes15:24
*** sdake has joined #openstack-ironic15:24
dtantsurjroll, I've send a email, and I think option #2 there actually satisfies both your and my requests. tl;dr we can fail cleaning on receiving the 1st heartbeat, but not power off the node.15:26
jrolldtantsur: https://review.openstack.org/#/c/167801/ and http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2015-03-26.log.html#t2015-03-26T11:48:1415:26
patchbotpatch 167801 - ironic - Agent driver: no-op heartbeat for maintenanced node (MERGED)15:26
jrollseems like the most interesting history15:26
*** ifarkas is now known as ifarkas_afk15:27
jrolldtantsur: interesting, will take a look shortly15:27
jrolldtantsur: it looks like this patch was possibly to solve a race when maintenance is set during agent boot for cleaning15:28
jrolls/possibly/partially15:28
dtantsuryeah, it's pretty clear, and it makes sense15:30
*** rbartal has quit IRC15:31
dtantsurexcept for it's not necessary a race: we perfectly allow cleaning in maintenance..15:32
*** jvgrant_ has quit IRC15:34
*** baoli_ has quit IRC15:34
*** rama_y has joined #openstack-ironic15:35
sambettsdtantsur: is there a possible 1.5 there which bans the provision verbs from working in maintenance, and if a node is in maintenance doesn't start cleaning on delete?15:35
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Abort cleaning and deployment if a node ends up in maintenance  https://review.openstack.org/36682815:35
*** krtaylor has quit IRC15:35
dtantsurjroll, here is how it would look like ^^^15:35
dtantsursambetts, I'm thinking about that too.15:35
dtantsurhmm, not I need to make sure it does not power off though15:36
jrolldtantsur: well, the race was that it would actually clean, which we definitely didn't want to do :P15:36
dtantsurheh15:36
sambettsdtantsur: so on delete if in maintenance, remove instance_info move to availiable or managable? and then leave the node alone15:37
*** thrash|biab is now known as thrash15:37
*** hoonetorg has quit IRC15:38
dtantsursambetts, I was thinking active -> deleted -> cleanfail15:38
*** Sukhdev has joined #openstack-ironic15:38
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Abort cleaning and deployment if a node ends up in maintenance  https://review.openstack.org/36682815:38
dtantsurjroll, sambetts, now without power off ^^^15:38
openstackgerritMarkos Chandras proposed openstack/bifrost: Fix unbound variable error in scripts/collect-test-info.sh  https://review.openstack.org/36683515:39
* jroll in a meeting, this is way too fast for me to follow :P15:39
sambettsdtantsur: I'm thinking if in maintenance active -> deleted -> managable else active -> deleted -> cleaning15:40
JayFdtantsur: ++15:40
dtantsurheh sorry man, just wanted to put up the patches before I leave for a day15:40
JayFdtantsur: re: the state flow15:40
JayFdtantsur: Because we wanna explicitly indicate this node is "unclean"15:40
dtantsursambetts, I don't want to pretend like we passed cleaning15:40
dtantsurJayF++ we still have tenant data and whatever15:41
*** Romanenko_K has quit IRC15:41
JayFexactly15:41
*** chlong has quit IRC15:41
*** moshele has joined #openstack-ironic15:41
sambettsdtantsur: I move to managable because you have to go through cleaning to get back to avaible and I wouldn't even start cleaning to prevent a node reboot15:41
dtantsursambetts, we can't report success for something that failed..15:42
*** hoonetorg has joined #openstack-ironic15:42
*** praneshp has quit IRC15:43
dtantsur*FAIL state will make it clear that something didn't work. also we don't have a transition CLEANWAIT -> MANAGEABLE, it will have to go trough something else anyway15:43
JayFcleanfail -> managable is a transition though15:44
JayFthe state machine itself is sorta indicating what's the "right" next step15:44
JayFactive -> deleted -> cleanfail -(by operator)-> managable15:44
* jroll finds lunch quickly15:44
sambetts(just play devils advacate here) In my case I wouldn't go cleanwait to manageable, I would just go DELETED -> MANAGEABLE, I wouldn't even try cleaning if I detected I was in maintenance15:45
dtantsursambetts, this will give an admin a big WTF, I'm afraid :)15:45
dtantsurimagine, a user tears down an ACTIVE instance, and it silently goes to MANAGEABLE15:45
*** MattMan has quit IRC15:46
*** rloo has joined #openstack-ironic15:46
sambettswhat happens in cases where cleaning is disabled? we just go straight to availiable and leave the node in maintenance?15:46
*** MattMan has joined #openstack-ironic15:46
dtantsursambetts, when cleaning is disabled, tear down succeeds15:46
dtantsurso yes15:47
JayFavailable+maintenance would be what makes sense to me there too15:47
*** joprovost1 has joined #openstack-ironic15:47
*** joprovost has quit IRC15:48
*** joprovost1 is now known as joprovost15:48
sambettsok, clean fail makes sense to me, are you going into clean fail based on a heartbeat or just adding it as a conditional in the statemachine transition?15:48
JayFyou'd always have to do it on heartbeat, at least15:49
JayFto handle the maintenanced-while-cleaning-is-starting (or has started) case15:49
sambettsI would have something that simply does if event == CLEAN AND node.maintance: then CLEANFAIL15:49
sambettsJayF: oh right yeah15:49
sambettshmm '15:49
JayFyou can shortcut it like that, which is fine too, but you'll have to get the heartbeat or else you get a race15:49
dtantsur+115:50
*** rajinir has joined #openstack-ironic15:51
sambettsI'd shortcut it before we power cycle the node, so in the case that the node is in maintenace before delete, then it wouldn't even get a chance to boot the ramdisk15:52
*** praneshp has joined #openstack-ironic15:53
sambettsif it goes into maintenance during the cleaning process then we need to catch the heartbeat and then move to cleanfailed at the end when we try to perform the power action or something right15:54
*** moshele has quit IRC15:55
dtantsuranyway, let's continue on the ML, I have to bail out already15:56
sambettscool :)15:56
dtantsurI think jroll has a use case for actually trying cleaning, soooo...15:56
sambettsyeah that might be interesting15:57
dtantsursee you all tomorrow15:57
*** dtantsur is now known as dtantsur|afk15:57
sambettscya later dtantsur|afk15:57
*** praneshp has quit IRC15:57
*** pcaruana has quit IRC16:01
*** mgoddard_ has joined #openstack-ironic16:05
*** rama_y has quit IRC16:07
*** rama_y has joined #openstack-ironic16:07
*** mgoddard has quit IRC16:08
*** tesseract- has quit IRC16:09
*** rbrud has quit IRC16:10
*** rpioso has quit IRC16:15
*** wznoinsk has joined #openstack-ironic16:15
jrolldtantsur|afk: no, again, I have a use case for *booting to an agent* and waiting16:19
*** rpioso has joined #openstack-ironic16:19
*** mgoddard has joined #openstack-ironic16:21
JayFjroll: honestly, isn't that something that my general long running ramdisk spec should cover?16:21
JayFjroll: we can still power on a machine if we want an agent booted on it16:21
*** mgoddard_ has quit IRC16:21
jrollJayF: sure, agree, how do you propose we do it in the meantime?16:21
mgagneso using haproxy, I see a lot of connection reset by the server (ironic-api). Lets say I don't run Mitaka, what can I do to improve my life? Could those errors be related to lack of workers in the API? Too many connections?16:22
jrollmgagne: I haven't personally seen that, but it does sound like not enough workers, any logs around it?16:22
JayFjroll: like I said ^ before, we have to catch the heartbeat and go to cleanfail. the question would be if we should preempt that by going straight to cleanfail on a delete-while-maint16:23
mgagnejroll: only logs I have is on the client side (nova) where the response is "empty", no HTTP status at all so it crashes with BadStatusLine exception16:23
jrollJayF: I'd think cleanfail would/should shutdown the node, but I haven't checked16:24
*** praneshp has joined #openstack-ironic16:24
JayFjroll: and I'm saying that's OK. For our downstream use case we can explicitly power on an agent if we wish16:24
jrollmgagne: no access to ironic-api logs I guess? I'd think there would be a traceback there16:24
mgagnejroll: nothing, even with DEBUG level16:24
jrollJayF: curious if other people use that16:24
jrollmgagne: :/16:24
jrollmgagne: might be worth pummeling a dev server to see if that's what happened16:25
mgagneso it looks like something is dropped before logging happens, clearly between haproxy backend and ironic-api. trying to see if there is any knob I can tweak on the api side first16:25
*** krtaylor has joined #openstack-ironic16:26
mgagnejroll: issue is in prod. curl api manually and no issue of course =)16:27
mgagnejroll: could try to run mod_wsgi and see if it changes anything16:29
jrollmgagne: yeah, I mean spin up a dev server and see if you can hit it hard enough with concurrent requests to make it do the same16:29
*** baoli has joined #openstack-ironic16:29
jrollhaproxy is just seeing a connection reset? just to be sure, there's nothing between haproxy and ironic-api?16:30
mgagnewe are hitting ceilometer-api much harder without issue but it's running in mod_wsgi :P16:30
jrollright16:30
jrollwouldn't be surprised if apache hangs on to connections and waits, while wsgiref just drops them16:30
mgagnesame L2 network16:30
jrollno firewall or anything though?16:30
mgagneno firewall, straight pipe16:30
jrollhrm16:31
mgagneI guess my only option is mod_wsgi for now16:31
mgagneor upgrade to ironic mitaka16:31
*** jpich has quit IRC16:31
mgagneso I can use api_workers16:31
JayFmgagne: we run a bunch of ironic behind haproxy, and I don't think I've *ever* seen a connection reset from ironic-api16:31
JayFmgagne: I'd be very, very suspicious of environmental things, especially if it's new16:31
JayFmgagne: like basic-linux-network-troubleshooting on the api nodes and haproxy16:32
JayFi.e. an iptables state table that's full or something16:32
mgagneJayF: lot of nodes? like, lets say, over 500?16:32
JayFwould cause this16:32
mgagnewe don't have iptables, we purge it16:32
mgagneoh16:32
mgagnehahaha16:32
mgagneTCP: TCP: Possible SYN flooding on port 6385. Sending cookies.  Check SNMP counters.16:32
jrollo_o16:33
JayFmgagne: >1k nodes, with hundreds heartbeating simultaneously, backed by 6 ironic api nodes and a haproxy16:33
JayFmgagne: told you :D16:33
JayF\o/16:33
mgagnebut16:33
mgagneit's not every day in the logs16:33
mgagnewe see connection drops every day/minutes16:33
*** jaosorior has quit IRC16:33
mgagnewill check on that side first16:33
JayFah16:34
TheJuliamgagne: I feel like I've seen that where haproxy is trying to keep a connection ready/open for requests16:34
JayFlooks like you can get that message in some cases16:34
JayFwhen the backend isn't selecting packets off the interface fast enough16:34
mgagnewill tweak tcp settings and see if it changes anything16:35
mgagneJayF: thanks for the hint ;)16:35
jrollcan always just spin up more api hosts :)16:35
jrollmgagne: how many apis/nodes are you running?16:36
mgagnewe have 2 already :O16:36
mgagneand as I said, we are hitting ceilometer-api MUCH harder16:36
jrolljust curious16:36
JayFmgagne: we have *6* for ~1k servers16:36
jrollI think we run 6 in our larger environments? we also keep agents up heartbeating though16:36
mgagnebut could also be that ironic driver in nova-compute hits ironic-api in parallel?16:36
jrolland may also be over-scaled16:36
jrollit does16:37
JayFyeah16:37
jroller16:37
jrollit may*16:37
jrollthere's multiple threads and such :)16:37
mgagne:D16:37
*** praneshp_ has joined #openstack-ironic16:37
NobodyCamGood Morning Ironic'ers :)16:38
JayFmgagne: have you checked your haproxy admin panel? anything interesting16:38
mgagnebut could it be also caused by api not accepting the connection fast enough?16:38
mgagneJayF: yea, I'm in the stats panel, that's where I found the connection reset by server16:38
jrollok this may sound crazy16:38
mgagnedug my way up from nova-compute.log16:38
jrollbut is dns working well on your api hosts?16:38
*** praneshp has quit IRC16:38
*** praneshp_ is now known as praneshp16:38
*** mbound has quit IRC16:39
mgagnewe have dnsmasq installed, dns works fine, I mean, I can resolve entries16:39
JayFmgagne: here's the haproxy config we're using, I doubt it's helpful but thought it'd at least give you a comparison point https://gist.github.com/jayofdoom/aea5c419b537441536101cd9d5917cc716:39
jrollmgagne: okay, if it's slow or being throttled or whatever it can cause problems16:40
jroll... because wsgiref does an rDNS query for each request16:40
TheJuliaouch16:40
mgagneat this point, the problem would show on all services? or is it new in Liberty or deps in it?16:40
jrollno, it's part of wsgiref, which we've used since the beginning (but I think dropped in mitaka)16:41
mgagneI think we are reaching a limit somewhere, max session is 24416:41
jrollwhich is stdlib16:41
mgagnelooks similar to a kernel limit16:41
TheJuliamgould: did you check the open file limit?16:41
jrollTheJulia: you say ouch - don't ask me how I learned about this :)16:42
JayFI think jroll and I are less helpful, because when we saw this stuff blowing up in the inital onmetal deployment, our reaction was to spin up more apis, and now we're overscaled and never see these problems ;P16:42
mgagneJayF: hehe, don't mind the input. this might also be our solution =)16:42
*** ChubYann has joined #openstack-ironic16:42
TheJuliajroll: I'm not sure I want to know how :)16:42
JayFTheJulia: lets just say don't bite the hand that sits on the dns-rate-limiting lever16:43
JayFlol16:43
jrollheh16:43
TheJulialol16:43
openstackgerritMiles Gould proposed openstack/python-ironicclient: Document updating nested node attributes with CLI  https://review.openstack.org/35474216:43
jrollJayF: you know, this did present as abnormal levels of TCP RST16:43
JayFI honestly don't remember16:44
JayFI believe you though :)16:44
JayFI block out bad memories :P16:44
jrollthen noticed tcp queues filled up16:44
jrollmgagne: another thing to check, I guess16:44
mgagnejroll: well now my keyboard stopped working on those nodes. time to blame network team16:44
jrollit's always the network \o/16:45
mgagne:D16:45
jrollmgagne: maybe your tcp queues are so full ssh stopped working :P16:45
mgagne:D16:45
mgagnedropped load-balancer too and coworkers too16:45
lucasagomescalling it a day, have a great evening all!16:45
JayFall computer problems have two root causes: networking, dns, or off-by-one16:45
*** lucasagomes is now known as lucas-dinner16:45
mgagneanyway will tweak tcp settings and try mod_wsgi16:45
TheJulialol16:45
mgagnecan confirm :D16:46
*** rbrud has joined #openstack-ironic16:46
*** praneshp has quit IRC16:48
*** rama_y has quit IRC16:48
jlvillalJayF: :)16:49
mgagneok, lot of connections in TIME_WAIT16:49
jlvillalJayF: And I was having networking problems yesterday while trying to setup port forwarding in anticipation of moving away from my expensive static IP internet service to much more reasonably priced DHCP internet service.16:50
*** trown is now known as trown|lunch16:50
*** jvgrant_ has joined #openstack-ironic16:50
TheJuliamgagne: between what parts?16:52
mgagneon ironic-api when nova-compute triggers power sync periodic tasks16:52
mgagnewe have over 600 nodes and I see a max session of ~240. So I think some are getting dropped. maybe time to add api workers16:53
mgagneok so will add new api workers16:54
*** xavierr_afk is now known as xavierr16:54
mgagnethrowing resources at the problem :D16:54
*** rbudden has quit IRC16:58
TheJuliamgagne: I'm fairly sure you can turn down the time that the kernel waits to close the socket out in TIME_WAIT state.  I had a database years ago that would end up running out of open connections because it didn't close it's self out so things were held open until the socket was fully closed out.16:59
mgagneyea at this point, I don't think that's a real issue. TIME_WAIT closes in less than ~2s17:00
*** anush has quit IRC17:00
TheJuliaAhhh17:00
mgagneasked someone to spawn more api =)17:00
TheJuliayeah, I was seeing the full socket timeout occur :(17:00
*** Sukhdev has quit IRC17:01
TheJuliawhich, is far more than 2 seconds17:01
*** watanabe_isao has joined #openstack-ironic17:01
JayFman I've troubleshooted some crazy bugs before17:02
JayFone bug, IPVS working badly with iptables on the same IPs, you'd have iptables block incoming FIN17:02
JayFthis was IMAP, so if you did a server quit, we sent the FIN and it worked no problem17:02
JayFbut if you had a connection drop unexpectedly, the FIN would be blocked by iptables and the connection would hold open through the ESTABLISHED timeout, which is ~5 days by default in linux17:03
mgagneok, "fixed" the issue. I limited the backend connection to 200 instead of 800017:05
mgagneonly got 1 error instead of hundreds17:05
*** derekh is now known as derekh_ark17:05
*** derekh_ark is now known as derekh_afk17:05
*** fragatina has joined #openstack-ironic17:08
krtaylorjlvillal, ironic-qa meeting?17:08
TheJuliamgould: artificial throttling I guess :)17:08
krtaylorjlvillal, just curious, not that I had anything17:09
openstackgerritJulia Kreger proposed openstack/ironic: Add volume_connectors and volume_targets to task  https://review.openstack.org/36619617:10
*** jcoufal has quit IRC17:13
watanabe_isaokrtaylor, good morning. About the qa meeting, I have one question to ask. If the meeting is not held today I would like to ask it here.17:13
*** e0ne has quit IRC17:14
*** Sukhdev has joined #openstack-ironic17:14
Sukhdevjroll: Ping17:14
*** links has joined #openstack-ironic17:15
mjturekwatanabe_isao: yep looks like it's not happening17:15
*** krtaylor has quit IRC17:16
*** krtaylor has joined #openstack-ironic17:16
watanabe_isaoI17:17
rlookrtaylor, mjturek, watanabe_isao: John sent out email this morning, cancelling the qa meeting17:17
krtaylorrloo, thanks, just checking that...17:17
krtaylorwatanabe_isao, ask away17:18
mjturekrloo: whoops! I see it now :) thanks17:18
watanabe_isaokrtaylor, yes sir. It is about your spec of 3rd party ci <https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/third-party-ci.html#problem-description>17:19
watanabe_isaoI think for us, 3rd party mentioners, we need to return the test result within 8 hours, since the patch set has been sent, is it right?17:20
krtaylorwatanabe_isao, initially (Newton) yes17:21
krtaylorwatanabe_isao, the goal is to get to 4hrs17:21
krtaylorbut we'll have a chance to revisit that requirement at summit17:22
krtaylorwatanabe_isao, is that doable for your test sytem?17:22
krtaylorsystem17:22
watanabe_isaokrtaylor, yes. I just want to know, how can we self check we are returning all the tests within that time.17:24
krtaylorah, sure, just look at the patch set submission time and compare it with when you post results17:25
krtaylorwe haven't implemented any kind of strict time interval check17:25
krtaylorit is just so that a developer can expect results from all the test systems in a reasonable amount of time17:26
watanabe_isaokrtaylor, I mean do we have a tool like ci-watch, that can automatically check all the tests?17:26
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add storage_interface DB field and object  https://review.openstack.org/34800517:27
krtaylorwatanabe_isao, I dont believe that is an option in ci-watch, but it sounds like a good enhancement  :)17:27
krtaylormmedvede, ^^^17:27
mmedvededid not have much time to work on ci-watch lately :(17:28
mmedvedekrtaylor: what is the enhancement, in a couple of words?17:28
*** links has quit IRC17:29
krtaylormmedvede, to check time it took for a system to return results on a patchset17:29
watanabe_isaokrtaylor, so does it mean, currently we ask 3rd party CIs return their result within 8(4) hours. But wo can only take a random sampling to check for that?17:29
krtaylorwatanabe_isao, all of these requirements are not strictly policed, by any project that I know of17:31
krtaylorwatanabe_isao, if a system was reporting late over a period of weeks, then some action might be taken, but that's not really my call to make17:32
krtaylorwatanabe_isao, I think it would be something that was looked at only if the system was causing some other kind of issue, reporting incorrectly, etc17:33
krtaylorthere may be people that watch that info very closely, but not me :)17:34
watanabe_isaokrtaylor, I see. I just noticed that jroll has mailed to warn some drivers which do not have a CI with it. And our CI currently has a performance issue with it, and is stopped. I just want to make sure that our CI will not in the next list.:)17:35
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add storage_interface to base driver class  https://review.openstack.org/34800617:35
*** Sukhdev has quit IRC17:36
watanabe_isaokrtaylor, do you know who may be the time keeper, please?17:38
*** jvgrant_ has quit IRC17:38
*** jvgrant_ has joined #openstack-ironic17:39
krtaylorwatanabe_isao, you can ask jroll, but I don't think we are watching that (time) too closely right now17:39
watanabe_isaokrtaylor, I got it. Thanks very much.17:40
*** Sukhdev has joined #openstack-ironic17:41
watanabe_isaojroll, sir, may I have your one minute, please?17:41
jrollSukhdev: hey, what's up?17:41
jrollwatanabe_isao: one moment, reading back17:42
jrollwatanabe_isao: ok, what's your question? who is watching the time?17:42
Sukhdevjroll : QQ - the work that we are doing for ironic-neutron, should it not qualify for ATC code for Summit?17:42
watanabe_isaojroll, yes.17:43
jrollSukhdev: depends what "we" means - typically, code commits are what qualify17:43
rajinirhttps://www.irccloud.com/pastebin/fM64KYd0/cleaning17:44
*** jvgrant_ has quit IRC17:44
jrollwatanabe_isao: it won't be super strict, more about if we notice a CI system is consistently slower than the goal17:44
rajinirShouldn't there be default cleaning steps? Any pointers?17:44
mgagneok so issue is still present. we bypassed the load-balancer to make sure it wasn't the problem. :-/17:45
watanabe_isaojroll, sir. Do we have any tool or something to confirm that (consistently slower than the goal)? I want to use it for a self check.17:46
TheJuliarajinir: Curious, it depends on how the node has reached cleaning17:46
*** harlowja has quit IRC17:46
rajinirTheJulia: Its CI, part of tempest17:46
rajinirtest17:46
*** causten has joined #openstack-ironic17:46
jrollwatanabe_isao: I'm not aware of any toold to make that easy, sorry17:47
TheJuliarajinir: got a link to a CI test failure where we can see this?17:47
rajinirTheJulia: https://stash.opencrowbar.org/logs/61/364861/3/check/dell-hw-tempest-dsvm-ironic-pxe_drac/1c2339b/17:48
*** sahil has joined #openstack-ironic17:48
sahilHello17:48
*** harlowja has joined #openstack-ironic17:49
watanabe_isaojroll, I see. One last question, please. More like a require.17:50
jrollwatanabe_isao: no need to ask to ask, just ask :)17:51
TheJuliarajinir: that link just hangs for me right now :\17:51
*** jralph has quit IRC17:51
rajinirThejulia: hmm. I can see clean_steps are missing17:52
*** mbound has joined #openstack-ironic17:52
TheJuliahello sahil, sorry I didn't reply again last night.  For some reason I didn't get any notifications17:52
rajinirTheJulia: Is there a way to set?17:52
*** fragatina has quit IRC17:52
sahilhi17:52
sahilI wanted to talk to someone from yesterday17:52
*** fragatina has joined #openstack-ironic17:53
jrollhi sahil17:53
sahilHI TheJulia17:53
sahilHi Jroll17:53
TheJuliarajinir: they should be created as part of a node going into cleaning from being deleted if memory serves.  There is no way to set that field afaik, nor should it be since it is for the driver's use17:53
sahilso it may be a small issue17:53
sahilbut I guess I need help17:53
rajinirTheJulia: Probably the node didn't crated properly17:53
sahilso with the help of some others here I was able to setup devstack with ironic that can talk to a baremetal node17:54
sahilon a fixed vlan17:54
sahilfor that i had to configure a ovs bridge and add the interface that was connected to that vlan17:54
*** hoangcx has quit IRC17:54
TheJuliarajinir: more likely there is some issue along the way :\17:54
sahilthe issue with the gateway17:54
sahilbefore running stack.sh17:55
rajinirTheJulia: ok17:55
sahilI could ping the node from my devstack vm17:55
TheJuliarajinir: By the way, the server finally replied indicating that permission to access is denied17:55
sahilafter stacking I could no17:55
sahilnot*17:55
sahilas a consequence, when I try to provision a node it boots but hangs at dhcp17:55
*** sambetts is now known as sambetts|afk17:55
sahilas it does not get any info from ironic/neutron17:55
watanabe_isaojroll, Our CI (irmc CI) has a performance issue right now, our test is a little bit slow. We are stopping the CI and improving it. I noticed your mail of warning non-CI drivers. If their is any schedule about the next check, I would like to know, and will do our best to recover our CI before that time.17:56
sahilI checked the qrouter-<id> namespace17:56
*** trown|lunch is now known as trown17:56
sahiland found that the gateway IP address that I had passed in local.conf was configured as an interface on the qrouter-xx17:57
sahilthat messed up things17:57
*** hoangcx has joined #openstack-ironic17:57
sahilso I need help to fix my devstack17:57
jrollwatanabe_isao: I'm planning to do a run through today and see what's going on, and mark some drivers untested - I will keep your comment in mind and leave irmc marked supported17:57
sahilsuch that it can talk to baremetal node can recieve dhcp response from dhcp server running under neutron17:57
sahilI am done taling17:58
sahiltalking17:58
sahilwaiting for response/suggestions17:58
watanabe_isaojroll, thank you very much. I will go back to my work, and return irmc CI ASAP.17:59
*** Sukhdev has quit IRC17:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Allow using TempURLs for deploy images  https://review.openstack.org/34416017:59
mgouldsahil: so the gateway was added to the wrong namespace?18:00
sahilI guess so yes18:00
sahilin the qrouter namespace18:00
sahilas an interface18:00
sahilI suppose that should not be the case as the gateway IP is actually of the bridge on the host18:01
sahilthat makes it possible for the DEVSTACK VM to talk to the baremetal node18:01
mgouldhow did you specify it in local.conf18:01
mgould?18:01
TheJuliasahil: can you share your local.conf file?18:01
sahilnew to irc18:01
sahilhow do I share the file18:01
TheJuliasahil: Use http://paste.openstack.org/18:01
mgouldsahil: http://paste.openstack.org18:01
sahilgot it18:01
TheJuliamake sure nothing sensitive to your environment is present, and then share the link once posted. :)18:02
rajinirThejulia: Working with the IT folks to get the log link working back again :(18:02
jrollwatanabe_isao: you're welcome, thanks for the chat18:02
*** jvgrant_ has joined #openstack-ironic18:02
TheJuliarajinir: It happens :)18:03
sahilhere it is18:03
*** fragatina has quit IRC18:03
sahilhttp://paste.openstack.org/show/567586/18:03
*** fragatina has joined #openstack-ironic18:04
TheJuliarajinir: so clean_steps should be getting created upon the agent heartbeating https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L443 lines 493-502 would be pertinent18:08
*** rbudden has joined #openstack-ironic18:08
rajinirTheJulia: The node doesn't have clean_step config set18:09
*** jcoufal has joined #openstack-ironic18:10
*** jvgrant_ has quit IRC18:11
TheJuliarajinir: So I think that would be the first entry into cleaning then... Are you building an IPA image?18:13
sahilany ideas TheJulia18:15
TheJuliasahil: pondering18:16
*** watanabe_isao has quit IRC18:16
*** jvgrant_ has joined #openstack-ironic18:18
mgouldsahil: no idea, sorry18:19
* mgould -> home; good night everyone!18:19
sahilhmm, ok thanks for the time18:19
*** e0ne has joined #openstack-ironic18:19
*** mgould is now known as mgould|afk18:19
TheJuliasahil: so you have PHYSICAL_NETWORK set to "private", is that the actual label in ovs?18:22
sahilyup18:22
sahilthis is the exact local.conf I am using18:22
sahilit creates private network attaches subnet with network range reserved for my vlan18:23
*** hoangcx has quit IRC18:24
sahilwait, what do you mean "label in ovs"18:24
sahilit makes a network with name "private"18:24
sahilI am guessing for flat network I do not need a router18:25
sahilwhat happens if I set Q_USE_PROVIDER_NETWORKING=True to 'false' instead18:26
TheJuliasahil: I honestly don't know neutorn well enough to say, but we don't indicate use of that in the developer quickstart documentation18:30
TheJuliasahil: I was trying to piece together context with the question while I skimmed through the ironic devstack plugin.  With your config, everything is expecting things to be hooked in advance to your OVS_PHYSICAL_BRIDGE.  The only thing neutron really should end up doing then is updating the configuration dnsmasq has, so it does seem like somehow extra18:32
TheJuliaconfiguration is taking place18:32
sahilI did create that br-ironic bridge and added the interface to it before stacking18:32
sahilso you think I should comment out certain directives18:33
TheJulialikely, neutron related ones we don't explicitly indicate would be a good start :)18:39
*** mkam has quit IRC18:39
*** goneri_ has joined #openstack-ironic18:39
*** goneri_ has quit IRC18:41
*** jcoufal_ has joined #openstack-ironic18:44
*** jcoufal has quit IRC18:45
*** wajdi_ has joined #openstack-ironic18:47
*** joprovost has quit IRC18:47
*** mkam has joined #openstack-ironic18:48
*** wajdi__ has joined #openstack-ironic18:49
*** joprovost has joined #openstack-ironic18:50
*** wajdi has quit IRC18:50
*** wajdi_ has quit IRC18:51
*** jvgrant_ has quit IRC18:53
*** causten has quit IRC18:54
*** jvgrant_ has joined #openstack-ironic18:54
*** hoangcx has joined #openstack-ironic18:55
*** athomas has quit IRC18:56
*** sukhdev has joined #openstack-ironic18:57
*** jvgrant_ has quit IRC18:58
*** causten has joined #openstack-ironic18:59
*** sahil has quit IRC19:01
*** sdake has quit IRC19:02
*** mkam has quit IRC19:04
openstackgerritMathieu Mitchell proposed openstack/ironic: Add a basic install guide  https://review.openstack.org/36329419:06
openstackgerritMathieu Mitchell proposed openstack/ironic: Import Bare Metal service overview into install-guide  https://review.openstack.org/36632919:06
*** clenimar has quit IRC19:10
*** sdake has joined #openstack-ironic19:10
*** joprovost has quit IRC19:11
*** narasimha_SV has joined #openstack-ironic19:12
narasimha_SVnova boot --flavor ironic-test-3 --image test-image instance-119:12
narasimha_SVthis will be the command for provisioning an ironic node right19:13
narasimha_SVso how it takes IP from this ?19:13
*** derekh_afk is now known as derekh19:13
narasimha_SVcan anyone please explain ?19:13
narasimha_SVI am confised at this point19:13
narasimha_SV*confused19:13
*** lucas-dinner has quit IRC19:13
mat128nova will take an IP from the network you have in your tenant19:15
mat128note that this command wont work if you have multiple networks19:15
*** lucasagomes has joined #openstack-ironic19:15
*** causten has quit IRC19:16
*** mkam has joined #openstack-ironic19:17
*** Sukhdev_ has joined #openstack-ironic19:17
*** fragatin_ has joined #openstack-ironic19:17
*** mkam has joined #openstack-ironic19:17
*** causten_ has joined #openstack-ironic19:19
*** fragatina has quit IRC19:21
*** rcernin has quit IRC19:21
*** jvgrant_ has joined #openstack-ironic19:22
*** joprovost has joined #openstack-ironic19:28
*** jvgrant has quit IRC19:29
*** causten_ has quit IRC19:31
*** causten has joined #openstack-ironic19:31
*** moshele has joined #openstack-ironic19:37
*** jjohnson2 has quit IRC19:41
*** hoangcx has quit IRC19:43
*** rama_y has joined #openstack-ironic19:43
*** wajdi__ has quit IRC19:48
*** fragatin_ has quit IRC19:53
*** fragatina has joined #openstack-ironic19:54
*** wajdi_ has joined #openstack-ironic19:56
*** wajdi__ has joined #openstack-ironic19:57
*** wajdi_ has quit IRC19:57
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643219:57
*** wajdi_ has joined #openstack-ironic19:58
*** joprovost has quit IRC19:58
* rloo loves the DoNotReview patches19:58
*** wajdi__ has quit IRC20:01
*** dprince has quit IRC20:01
*** rama_y has quit IRC20:02
*** causten has quit IRC20:04
*** sahil has joined #openstack-ironic20:04
*** openstackgerrit has quit IRC20:04
*** sabeen has quit IRC20:04
*** mbound has quit IRC20:04
*** openstackgerrit has joined #openstack-ironic20:04
*** causten has joined #openstack-ironic20:06
*** e0ne has quit IRC20:22
*** moshele has quit IRC20:23
*** anush has joined #openstack-ironic20:25
*** narasimha_SV has quit IRC20:25
*** derekh has quit IRC20:25
*** Goneri has quit IRC20:30
JayFjroll: how about: good new contributor documentation20:31
JayFjroll: I think we need some of that20:31
JayFlol20:31
jrollJayF: heh, I've never seen a new contributor doc that couldn't be improved :)20:32
*** rama_y has joined #openstack-ironic20:32
jrollrunning to the store, back later20:32
jlvillalkrtaylor: Sorry. I sent out an email about no QA meeting. Plus I had a meeting conflict :(20:33
krtaylorjlvillal, no worries, my bad for not checking email first20:35
*** jabroney has joined #openstack-ironic20:36
openstackgerritGabriel Bezerra proposed openstack/python-oneviewclient: Add auditing instrumentation for python-oneviewclient  https://review.openstack.org/33728120:39
*** snizar has joined #openstack-ironic20:41
*** snizar_ has joined #openstack-ironic20:41
*** michauds has quit IRC20:43
*** wajdi_ has quit IRC20:45
*** wajdi has joined #openstack-ironic20:45
*** jcoufal_ has quit IRC20:48
*** vishwana_ has joined #openstack-ironic20:51
*** vishwanathj has quit IRC20:52
*** jcoufal has joined #openstack-ironic20:54
*** sahil has quit IRC20:54
*** jvgrant_ has quit IRC20:55
*** jvgrant has joined #openstack-ironic20:55
*** sabeen has joined #openstack-ironic20:56
*** trown is now known as trown|outtypewww20:57
rloojlvillal: before i forget (again), what was your question about https://review.openstack.org/#/c/364440/20:57
patchbotpatch 364440 - python-ironicclient - osc node power on & off commands20:57
jlvillalrloo: Oh. Let me get back to you. Kind of in the middle of something.20:57
*** mkam has quit IRC20:57
jlvillalrloo: Thanks for following up!20:57
rloojlvillal: sure. no worries.20:57
*** jvgrant has quit IRC21:00
openstackgerritJay Faulkner proposed openstack/ironic: Add metrics to administrator guide  https://review.openstack.org/35681121:01
*** sukhdev has quit IRC21:04
*** mbound has joined #openstack-ironic21:05
openstackgerritDoug Hellmann proposed openstack/python-ironicclient: standardize release note page names and ordering  https://review.openstack.org/36698021:06
openstackgerritJulia Kreger proposed openstack/ironic: Add volume_connectors and volume_targets to task  https://review.openstack.org/36619621:06
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add storage_interface DB field and object  https://review.openstack.org/34800521:07
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add storage_interface to base driver class  https://review.openstack.org/34800621:07
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add base cinder common interface  https://review.openstack.org/35562521:07
openstackgerritJulia Kreger proposed openstack/ironic: [WIP] Add Cinder storage driver  https://review.openstack.org/36619721:08
*** mbound has quit IRC21:10
openstackgerritMerged openstack/python-oneviewclient: Add auditing instrumentation for python-oneviewclient  https://review.openstack.org/33728121:10
*** krtaylor has quit IRC21:11
*** mmedvede has quit IRC21:15
*** mkam has joined #openstack-ironic21:17
openstackgerritMerged openstack/python-oneviewclient: Remove discover from test-requirements  https://review.openstack.org/34577321:25
*** jheroux has quit IRC21:27
*** mkam has quit IRC21:27
*** jvgrant has joined #openstack-ironic21:35
*** mkam has joined #openstack-ironic21:40
*** mmedvede has joined #openstack-ironic21:41
*** snizar_ has quit IRC21:45
*** snizar has quit IRC21:45
*** jmccrory has quit IRC21:53
*** jmccrory has joined #openstack-ironic21:53
*** krtaylor has joined #openstack-ironic21:53
*** causten has quit IRC21:54
*** jmccrory has quit IRC21:55
*** jmccrory has joined #openstack-ironic21:55
*** jmccrory has quit IRC21:57
*** causten has joined #openstack-ironic21:57
*** jmccrory has joined #openstack-ironic21:57
mrdaMorning Ironic22:02
*** mkam has quit IRC22:02
*** rama_y has quit IRC22:04
*** Goneri has joined #openstack-ironic22:05
*** jvgrant has quit IRC22:07
*** jvgrant has joined #openstack-ironic22:07
*** bfournie has quit IRC22:08
*** rama_y has joined #openstack-ironic22:10
*** jvgrant has quit IRC22:10
*** chlong has joined #openstack-ironic22:12
*** snizar has joined #openstack-ironic22:13
*** snizar_ has joined #openstack-ironic22:13
*** cdearborn has quit IRC22:16
*** jvgrant has joined #openstack-ironic22:17
openstackgerritRuby Loo proposed openstack/python-ironicclient: Adds --chassis-uuid to osc 'baremetal node set'  https://review.openstack.org/36399922:17
*** baoli_ has joined #openstack-ironic22:17
*** baoli has quit IRC22:18
*** jvgrant has quit IRC22:19
*** jralph has joined #openstack-ironic22:21
*** rama_y has quit IRC22:22
*** sdake has quit IRC22:24
*** rloo has quit IRC22:26
*** rloo has joined #openstack-ironic22:27
*** rbudden has quit IRC22:28
*** jvgrant has joined #openstack-ironic22:32
*** rbrud has quit IRC22:32
*** fragatin_ has joined #openstack-ironic22:34
*** jvgrant has quit IRC22:36
*** fragatina has quit IRC22:37
*** rpioso has quit IRC22:46
openstackgerritMerged openstack/ironic-inspector: TrivialFix: Remove cfg import unused  https://review.openstack.org/36590222:50
*** baoli_ has quit IRC22:54
*** fragatin_ has quit IRC22:57
*** harlowja has quit IRC23:03
*** Goneri has quit IRC23:09
*** rajinir has quit IRC23:15
*** wajdi_ has joined #openstack-ironic23:19
*** sabeen has quit IRC23:19
*** wajdi has quit IRC23:22
*** jabroney1 has joined #openstack-ironic23:24
NobodyCammorning mrda23:25
*** anush has quit IRC23:26
*** wajdi_ has quit IRC23:26
*** jabroney has quit IRC23:26
*** rbudden has joined #openstack-ironic23:28
mrdahey NobodyCam23:29
NobodyCam:)23:29
*** leitan has quit IRC23:39
*** rloo has quit IRC23:39

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