Thursday, 2016-08-11

*** aNupoisc has quit IRC00:03
*** wznoinsk has joined #openstack-ironic00:05
*** praneshp has joined #openstack-ironic00:10
*** wajdi has joined #openstack-ironic00:13
*** Sukhdev has quit IRC00:13
*** Sukhdev has joined #openstack-ironic00:13
wznoinskhi all, which deploy driver would you recommend as the least problematic?00:14
*** ksumit has quit IRC00:15
*** ksumit has joined #openstack-ironic00:15
*** ksumit has quit IRC00:23
*** ijw has joined #openstack-ironic00:32
*** tuanla_ has joined #openstack-ironic00:37
*** ksumit has joined #openstack-ironic00:39
*** ijw has quit IRC00:40
*** ijw has joined #openstack-ironic00:40
*** joprovost has joined #openstack-ironic00:41
*** ksumit has quit IRC00:48
*** praneshp has quit IRC00:49
*** rbudden has quit IRC00:52
zer0c00l_                            https://review.openstack.org/32676800:57
zer0c00l_oops00:58
*** sabeen has joined #openstack-ironic00:59
*** xuao has joined #openstack-ironic01:01
*** mtanino has joined #openstack-ironic01:02
*** tangchen has quit IRC01:04
*** penick has quit IRC01:07
*** hoangcx has joined #openstack-ironic01:09
*** praneshp has joined #openstack-ironic01:24
openstackgerritTeng Fei proposed openstack/ironic-inspector: Modify the SQL's string from double quote to single quote.  https://review.openstack.org/34779501:27
*** Sukhdev has quit IRC01:28
*** Sukhdev has joined #openstack-ironic01:30
*** mtanino has quit IRC01:41
*** rama_y has joined #openstack-ironic01:41
*** mtanino has joined #openstack-ironic01:51
*** rloo_ has joined #openstack-ironic01:53
openstackgerritMerged openstack/ironic: Clean up release notes for 6.1.0  https://review.openstack.org/35266001:53
*** rloo has quit IRC01:53
*** Sukhdev has quit IRC01:56
*** mtanino has quit IRC01:57
*** chlong|mtg has joined #openstack-ironic01:57
*** rloo_ has quit IRC02:00
openstackgerritZhenguo Niu proposed openstack/ironic-inspector: Add API for listing all introspection statuses  https://review.openstack.org/34492102:02
*** bfournie has joined #openstack-ironic02:04
openstackgerritMerged openstack/ironic-specs: Fix Nova-compatible Serial Console's Other deployer impact  https://review.openstack.org/33537802:04
*** ijw has quit IRC02:05
*** adu has joined #openstack-ironic02:07
openstackgerritMerged openstack/ironic: Refactor OneView dynamic allocation release notes  https://review.openstack.org/35304902:17
*** jcoufal has joined #openstack-ironic02:17
*** xuao has quit IRC02:22
*** penick has joined #openstack-ironic02:35
*** harlowja_at_home has joined #openstack-ironic02:39
*** penick_ has joined #openstack-ironic02:40
*** penick has quit IRC02:41
*** penick_ is now known as penick02:41
*** baoli has joined #openstack-ironic02:52
*** harlowja_at_home has quit IRC02:54
*** jcoufal has quit IRC02:55
*** aNupoisc has joined #openstack-ironic02:58
*** aNupoisc has quit IRC03:02
*** baoli has quit IRC03:03
*** zhangjl1 has joined #openstack-ironic03:05
*** baoli has joined #openstack-ironic03:09
openstackgerritZhenguo Niu proposed openstack/ironic-inspector: Allow using introspection data in condition's 'value' field  https://review.openstack.org/34553503:12
*** krotscheck_ has joined #openstack-ironic03:12
*** krotscheck has quit IRC03:13
*** baoli has quit IRC03:13
*** krotscheck_ is now known as krotscheck03:14
*** BarbaraLove16 has joined #openstack-ironic03:20
*** sabeen has quit IRC03:20
*** sabeen has joined #openstack-ironic03:23
*** BarbaraLove16 has quit IRC03:23
*** links has joined #openstack-ironic03:37
*** appu103_ has joined #openstack-ironic03:38
*** appu103_ is now known as aparnav03:38
*** amotoki has quit IRC03:39
*** vikrant has joined #openstack-ironic03:40
*** ljjjustin has quit IRC03:46
*** mgkwill has quit IRC03:47
*** mgkwill has joined #openstack-ironic03:48
*** rama_y has quit IRC03:48
*** jtomasek has quit IRC03:48
*** ljjjustin has joined #openstack-ironic03:49
*** praneshp has quit IRC03:50
*** praneshp has joined #openstack-ironic03:51
*** fragatina has quit IRC03:57
openstackgerritAparna proposed openstack/ironic: Update proliantutils version required for Newton release  https://review.openstack.org/35330103:58
*** fragatina has joined #openstack-ironic03:58
*** fragatina has quit IRC03:58
*** jtomasek has joined #openstack-ironic03:59
*** hj-hpe has quit IRC04:02
*** hjanssen-hpe has quit IRC04:02
*** vishwanathj has quit IRC04:20
*** vishwanathj has joined #openstack-ironic04:21
*** aparnav has quit IRC04:30
*** joprovost has quit IRC04:31
*** vmud213 has joined #openstack-ironic04:34
*** appu103_ has joined #openstack-ironic04:35
*** psachin has joined #openstack-ironic04:36
*** sabeen has quit IRC04:42
*** ijw has joined #openstack-ironic04:54
*** appu103_ is now known as aparnav04:57
*** ijw has quit IRC04:59
*** e0ne has joined #openstack-ironic05:17
*** sdake has quit IRC05:20
*** e0ne has quit IRC05:23
*** rcernin has joined #openstack-ironic05:29
*** ChubYann has quit IRC05:36
*** fxpester|2 has joined #openstack-ironic05:41
*** david-lyle_ has joined #openstack-ironic05:43
*** TheJulia_ has joined #openstack-ironic05:43
*** agordeev_ has joined #openstack-ironic05:43
*** serverascode_ has joined #openstack-ironic05:43
*** evgenyl_ has joined #openstack-ironic05:44
*** beekneemech has joined #openstack-ironic05:45
*** TheJulia has quit IRC05:48
*** fxpester has quit IRC05:48
*** evgenyl has quit IRC05:48
*** david-lyle has quit IRC05:48
*** agordeev has quit IRC05:48
*** bapalm has quit IRC05:48
*** serverascode has quit IRC05:48
*** bnemec has quit IRC05:48
*** bapalm_ has joined #openstack-ironic05:48
*** TheJulia_ is now known as TheJulia05:48
*** agordeev_ is now known as agordeev05:49
*** evgenyl_ is now known as evgenyl05:49
*** serverascode_ is now known as serverascode05:51
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add vendor id, vendor name and product_id to interface information  https://review.openstack.org/27994005:55
*** vishwanathj has quit IRC06:15
*** vishwanathj has joined #openstack-ironic06:15
*** adu has quit IRC06:17
openstackgerritlokesh s proposed openstack/ironic: Adding follow-up patch for audit middleware  https://review.openstack.org/35039906:30
*** pcaruana has joined #openstack-ironic06:45
*** moshele has joined #openstack-ironic06:47
*** xuao has joined #openstack-ironic06:49
*** rbartal has joined #openstack-ironic06:51
*** tesseract- has joined #openstack-ironic06:53
openstackgerritAnkit Kumar proposed openstack/proliantutils: Fix log message for exception in Gen7  https://review.openstack.org/35386006:59
*** Goneri has joined #openstack-ironic07:01
*** praneshp has quit IRC07:04
*** ijw has joined #openstack-ironic07:11
*** milan has joined #openstack-ironic07:11
*** xuao has quit IRC07:14
*** ijw has quit IRC07:15
*** xuao has joined #openstack-ironic07:17
*** ifarkas_afk is now known as ifarkas07:23
*** Lord_Phoenix has joined #openstack-ironic07:29
*** mbound has joined #openstack-ironic07:31
*** ohamada has joined #openstack-ironic07:33
*** aparnav has quit IRC07:36
pas-hamorning all07:36
*** xuao has quit IRC07:42
*** jpich has joined #openstack-ironic07:43
*** e0ne has joined #openstack-ironic07:45
*** chlong|mtg has quit IRC07:46
*** moshele has quit IRC07:55
*** xuao has joined #openstack-ironic07:59
xuaomorning ~~~~07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** jtomasek_ has joined #openstack-ironic08:02
*** moshele has joined #openstack-ironic08:07
*** appu103_ has joined #openstack-ironic08:09
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
*** amoralej|off is now known as amoralej08:12
*** rcarrillocruz has joined #openstack-ironic08:14
*** lucas-dinner is now known as lucasagomes08:16
lucasagomesmorning all08:16
xuaomorning~~~~~~08:16
zhangjl1hey all08:16
xuaohi lucasagomes, I am so sorry to bother you. But zhangjl1 and me have got into a new trouble..... : (08:18
xuaoAfter we `nova boot` the `user-image`. We found our physical machine boot into the `deploy-system` requiring the username/password.Meanwhile, the ironic node was waiting for call back and then error. We wonder what can we do next, or if we can sign in the system to have a debug~08:19
*** moshele has quit IRC08:21
lucasagomeszhangjl1, xuao 1 sec... I need to get my coffee first :D08:22
lucasagomess/get/drink08:22
xuaoOh I think I need one cup too~~~08:23
*** sdake has joined #openstack-ironic08:23
*** moshele has joined #openstack-ironic08:24
*** alineb has joined #openstack-ironic08:25
*** appu103_ has quit IRC08:25
*** appu103_ has joined #openstack-ironic08:25
alinebmorning all08:25
xuaomorning~08:26
moshelelucasagomes: kindly reminder for review the infiniband patches https://review.openstack.org/#/c/264263/ and https://review.openstack.org/#/c/264257/08:28
*** dtantsur|afk is now known as dtantsur08:29
dtantsurmorning Ironic08:32
*** dtantsur is now known as dtantsur|bbl08:32
*** appu103_ is now known as aparnav08:35
lucasagomesxuao, rigth so, it seems that the deployment ramdisk can't contact the Ironic node back (to move it from wait call-back to deploying state)08:36
lucasagomesxuao, that's tricky to debug, depends how you've built the deploy ramdisk08:36
*** e0ne has quit IRC08:37
lucasagomesxuao, did you download one upstream (coreos one ?) or did you build one with DIB ?08:37
*** bvandewa has joined #openstack-ironic08:37
zhangjl1we build deploy ramdisk with DIB08:37
zhangjl1with the command : disk-image-create ironic-agent fedora -o ironic-deploy08:38
lucasagomeszhangjl1, right, so by default it doesn't come with any user... You can build another one and use elements such as "devuser" (https://github.com/openstack/diskimage-builder/tree/master/elements/devuser) or "dynamic-login" (https://github.com/openstack/diskimage-builder/tree/master/elements/dynamic-login)08:39
lucasagomeszhangjl1, that will create a default user in the image so you will be able to log in the ramdisk08:39
lucasagomese.g, disk-image-create devuser ironic-agent fedora -o ironic-deploy08:40
lucasagomessee the options of that element in the README as well https://github.com/openstack/diskimage-builder/tree/master/elements/devuser#environment-variables08:40
xuaoget√ we'll have a try following it ~~08:41
*** electrofelix has joined #openstack-ironic08:41
zhangjl1your means, we need to login to the ramdisk, then , then ironic deploying would be continue?08:41
*** mbound has quit IRC08:43
*** e0ne has joined #openstack-ironic08:43
openstackgerritDao Cong Tien proposed openstack/ironic: Remove incorrect CONF.import_opt() from test_ipmitool.py  https://review.openstack.org/35390708:46
lucasagomeszhangjl1, no I mean, you need to login to debug it08:48
lucasagomeszhangjl1, usually the reason that happens is because the deploy ramdisk can't talk to the ironic api08:48
lucasagomesdue some network configuration issues08:48
zhangjl1ok!!! thanks for your advice08:49
lucasagomeszhangjl1, once you log in, see the /proc/cmdline it will contain the URL for the ironic api08:49
lucasagomestry to reach it08:49
zhangjl1i would try it now08:49
openstackgerritImre Farkas proposed openstack/ironic: DRAC RAID configuration  https://review.openstack.org/17631908:52
ifarkasmorning all!08:53
xuaomorning ~~~~08:53
zhangjl1hey all08:53
*** phuongnh has joined #openstack-ironic09:01
*** permalac has quit IRC09:02
openstackgerritlokesh s proposed openstack/ironic: Adding follow-up patch for audit middleware  https://review.openstack.org/35039909:02
openstackgerritDao Cong Tien proposed openstack/ironic: Rename some variables in test_ipminative.py  https://review.openstack.org/35391409:02
*** Romanenko_K has joined #openstack-ironic09:03
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643209:04
openstackgerritVasyl Saienko proposed openstack/ironic: Source openrc on subnode in multinode topology  https://review.openstack.org/35330209:04
openstackgerritVasyl Saienko proposed openstack/ironic: Skip db configuration on subnodes  https://review.openstack.org/35330309:04
openstackgerritVasyl Saienko proposed openstack/ironic: Ignore required_services for multinode topology  https://review.openstack.org/35279309:04
openstackgerritDao Cong Tien proposed openstack/ironic: Rename some variables in test_ipminative.py  https://review.openstack.org/35391409:08
*** bvandewa has quit IRC09:13
milanmorning Ironic! :)09:16
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Fix release notes formatting  https://review.openstack.org/35392709:19
aarefievmorning!09:19
*** xuao has quit IRC09:22
sambetts|afkMooooooorning everyone!09:23
*** sambetts|afk is now known as sambetts09:23
*** xuao has joined #openstack-ironic09:25
*** amitkqed has quit IRC09:27
*** amitkqed has joined #openstack-ironic09:27
xuaomorning ~~09:29
*** xuao has quit IRC09:37
*** xuao has joined #openstack-ironic09:37
lucasagomessambetts, morning09:38
sambettsHey lucasagomes09:38
openstackgerritDao Cong Tien proposed openstack/ironic: Remove incorrect CONF.import_opt() from test_ipmitool.py  https://review.openstack.org/35390709:38
openstackgerritMerged openstack/ironic: Updated tests for db migration scripts  https://review.openstack.org/34424609:40
*** xuao has quit IRC09:41
*** xuao has joined #openstack-ironic09:44
*** e0ne has quit IRC09:52
openstackgerritZhenguo Niu proposed openstack/ironic-inspector: Add API for listing all introspection statuses  https://review.openstack.org/34492109:52
*** e0ne has joined #openstack-ironic09:56
*** dtantsur|bbl is now known as dtantsur10:04
*** xuao has quit IRC10:06
*** xuao has joined #openstack-ironic10:09
*** hoangcx has quit IRC10:10
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844010:10
xuaolucasagomes, it seems that the ironic_python_agent raise an error 401.  And zhangjl1 is debuging now~ maybe difficult~10:16
lucasagomesxuao, oh right, that's good news actually. So you can ping the API10:17
lucasagomesxuao, can you check if the ironic-python-agent process is running in the ramdisk?10:17
*** phuongnh has quit IRC10:17
zhangjl1yes, i am sure is running10:17
lucasagomeszhangjl1, there's any logs you can see ?10:18
zhangjl1wait minutes10:18
*** xuao has quit IRC10:21
*** xuao has joined #openstack-ironic10:22
milansambetts, morning, I could use some more opinions on https://review.openstack.org/#/c/348943 would you mind giving it a review? ;)10:22
*** aparnav has quit IRC10:22
sambettsmilan: sure I'll try to take a look at some point today :)10:22
milandtantsur, morning, could you please take a look as well? ^^10:23
milansambetts, thanks!10:23
zhangjl1lucasagomes: the error log is here, http://paste.openstack.org/show/554108/10:23
*** eset has joined #openstack-ironic10:23
milanifarkas, hey, same begging I'm afraid ;) ^^^10:24
*** xuao has quit IRC10:25
*** xuao has joined #openstack-ironic10:26
*** mgould|afk is now known as mgould10:26
mgouldmorning Ironic!10:26
milanmorning mgould! :)10:26
mgouldmorning milan sambetts dtantsur zhangjl1 lucasagomes xuao10:26
xuaomorning~~~10:27
milanmorning aarefiev! :) thanks for the review yesterday, took me a bit to reply, though10:27
aarefievmilan: hey, I'm looking your replies, about sqlalchemy, it's bug fixed only in 1.1.010:29
lucasagomeszhangjl1, hmm are you deploying master right?10:29
lucasagomesdtantsur, I wonder if we need to update the dib element to use the new endpoints for lookup/heartbeat ?10:29
milanaarefiev, then it may get more worth using the enum.Enum10:30
ifarkasmilan, sure, on my list ;-)10:30
milanifarkas, thanks :)10:30
*** fragatina has joined #openstack-ironic10:31
aarefievmilan: it doesn't make sense to use value in enums, you should compare enum' members, not values10:32
zhangjl1my iroinc-api and ironic-conductor version is mitaka , and the ironic-python-agent is master branch with commit dfaf92b611d7c4abb98345fbcf21ef3ed4bb9d3b10:32
zhangjl1lucasagomes, is the version mismatch?10:32
lucasagomeszhangjl1, oh that might be it then, cause it's trying to talk to v1/lookup10:32
lucasagomesI don't think that endpoint existed in mitaka10:32
lucasagomesit used the vendor passthru before10:33
lucasagomeszhangjl1, apparently yes10:33
milanaarefiev, interesting10:33
lucasagomeszhangjl1, if you can build a ramdisk using the IPA from mitaka that would help10:33
* milan spins automaton with Enums10:33
*** fragatina has quit IRC10:33
*** Goneri has quit IRC10:33
*** mbound has joined #openstack-ironic10:33
*** fragatina has joined #openstack-ironic10:34
aarefievmilan: just trying with 1.1.0 version, it works fine, sqlalchemy problems (10:34
milanaarefiev, it didn't cross my mind I could use the attributes straight honestly10:35
zhangjl1lucasagomes: thanks a lot !!!10:35
*** amotoki has joined #openstack-ironic10:35
milanit just feels so odd you create a class with attributes of string values and once you access those they're changed behind the scenes10:37
aarefievmilan: maybe create our own class for states or use global var here, interesting how it will looks like10:38
aarefievyep10:38
milanaarefiev, that would be a way as well10:38
milanthe globals10:38
*** mbound has quit IRC10:38
milansince currently it's just the class10:38
aarefievtbt  it's looks hacky now10:40
aarefievit looks10:40
milanaarefiev, I know10:40
*** amotoki has quit IRC10:40
xuao😂lucasagomes,Still feel puzzled about how how to build the ramdisk with version specified😂10:42
lucasagomesxuao, I don't know much DIB to actually help with it :-/10:43
lucasagomesxuao, maybe asking at #tripleo ?10:43
xuaoThanks for your advice~~~~~~10:43
openstackgerritSam Betts proposed openstack/ironic-specs: Add spec for VLAN Aware Baremetal Instances  https://review.openstack.org/27785310:43
xuaoHelp a lotttttttttttttttttttttttttttttttttttttttttttttt~10:43
dtantsurlucasagomes, the DIB element is deprecated, why should we?10:44
lucasagomesxuao, you welcome10:44
lucasagomesdtantsur, yeah, forget what I said... I think we found the problem already10:45
xuaomemeda~10:45
dtantsurok :)10:45
openstackgerritSam Betts proposed openstack/ironic-specs: Add spec for VLAN Aware Baremetal Instances  https://review.openstack.org/27785310:47
*** Goneri has joined #openstack-ironic10:47
openstackgerritSam Betts proposed openstack/ironic-specs: Add spec for interface attach and detach API  https://review.openstack.org/31763610:48
openstackgerritSam Betts proposed openstack/ironic-specs: Add spec for interface attach and detach API  https://review.openstack.org/31763610:50
*** fragatin_ has joined #openstack-ironic10:50
milanaarefiev, yeah, the automaton doesn't mind using enum.Enum attributes as states and events: http://paste.openstack.org/show/554109/10:52
*** athomas has quit IRC10:53
aarefievmilan: yeah, that make sense10:54
*** fragatina has quit IRC10:54
openstackgerritDmitry Tantsur proposed openstack/ironic: Fix updating port MAC address for active nodes  https://review.openstack.org/35345010:56
milanaarefiev, on the plus side of the current hacky way, the change to Enum won't be too bad. But simpler still would be to use plain class (no need to remove the utils code&unit tests)10:58
milanjust the SA db column will be more verbose10:59
milanaarefiev, I'd vote for plain class for now rather than GLOBALs11:00
aarefievyep, I thought about some wrap to switch on enums later11:00
*** athomas has joined #openstack-ironic11:01
*** ijw has joined #openstack-ironic11:02
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Document retrieving logs from the deploy ramdisk  https://review.openstack.org/35248311:04
*** xuao has quit IRC11:06
*** ijw has quit IRC11:07
vsaienk0good day Ironic'ers, dtantsur, sambetts, jroll, milan, aarefiev, vdrok, lucasagomes11:15
lucasagomesvsaienk0, hello, good morning11:15
aarefievhey vsaienk011:15
milanvsaienk0, morning! :)11:16
sambettso/ vsaienk011:16
mat128good morning vsaienk0 and everyone else :)11:16
jrollmorning everyone!11:16
* jroll too many names to list :)11:16
mat128=)11:16
sambettshey jroll11:17
jrolllucasagomes: dtantsur: sambetts: have you seen anything in the last 12 hours that tells us we aren't ready to release?11:17
lucasagomesjroll, not that I can think of11:17
dtantsurmorning jroll, vsaienk011:17
dtantsurlemme think11:17
sambettsonly thing might be that, the IPA image we're testing with in the gate is super out of date, because of the post job breakages11:18
dtantsuroh yeah, this is bad11:18
lucasagomesjroll, there's a cool patch from devananda using the new policy mechanism to hide the configdrive content11:18
sambettsI'd like to see a couple of test runs with the latest IPA to make sure nothing is busted11:19
lucasagomesI've +2 it today, it may be a cool thing to include in the release cause it touches security11:19
lucasagomesfor ref: https://review.openstack.org/#/c/326768/11:19
jrollhm, we merged something in IPA very recently11:19
dtantsurpas-ha, hey! do you still plan on https://review.openstack.org/#/c/253054/?11:20
jrolllucasagomes: I'm ready to cut off new things, I wanted to release this tuesday but the reno updates took longer than I though11:20
jrollt11:20
sambettsjroll: I guess the src job on there should have caught most things right?11:20
lucasagomesjroll, fair enough, so I would say: go ahead and release it :D11:20
jrollsambetts: the one on IPA, yeah11:20
lucasagomeswe can include the rest in the next one11:21
dtantsurwe're going to release ironic-inspector too, but we need to fix our release notes11:21
dtantsuraarefiev is working on it11:21
*** appu103_ has joined #openstack-ironic11:21
jrollawesome11:22
sambettsjroll: unforunatly there hasn't been anything merged into IPA to trigger the post job since my fix for it merged ysterday11:22
dtantsurlet's merge something :) any typos?11:22
jroll^^11:22
jrolldoc updates always welcome ;)11:22
aarefievyep, I'm updating it now11:22
milanmorning jroll, mat128! :)11:23
jrollsambetts: I've got a thing11:23
openstackgerritRicardo Carrillo Cruz proposed openstack/bifrost: Default auth to empty dict instead of None  https://review.openstack.org/35399011:23
*** moshele has quit IRC11:24
sambettsjroll: oh nice! I was just looking at the docs and thinking we need to update the APIs in there based on dtantsur promotion patch11:24
lucasagomesjroll, we can merge https://review.openstack.org/#/c/284309/1 after you fix it11:24
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: Make code blocks real code blocks in metrics docs  https://review.openstack.org/35399211:24
jrolllucasagomes: oh, forgot about that one11:25
* jroll does it11:25
jrollthat docs patch will push through the gate quicker :)11:25
rcarrillocruzTheJulia: heya, i filed a bug and pushed change https://review.openstack.org/35399011:25
rcarrillocruzlet me know what you think when you're around11:25
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: Warn if extension fails to load  https://review.openstack.org/28430911:26
jrolllucasagomes: there you are, thanks for the reminder :)11:26
jrollshould I have tests for that, though?11:26
dtantsurjroll, sambetts, mind checking https://review.openstack.org/#/c/353450/ please? seems to work locally11:26
lucasagomesjroll, hmm would be good indeed :-/11:27
jrollwow, post queue at 4 hours today :/11:27
sambettswow...11:27
jrollsambetts: I say we just ship it, and have a 6.0.1 if something unexpected comes up11:27
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Fix release notes formatting  https://review.openstack.org/35392711:28
jrolland I'll wait to ship the agent until we get a proper build11:28
*** amitkqed has quit IRC11:28
jrolldtantsur: on my list11:28
sambettsmakes sense, to me, most things should have been covered by the -src job run on the IPA merge that happened ~43hrs ago11:28
*** amitkqed has joined #openstack-ironic11:28
jrollsambetts: indeed, I think we're fine11:28
jrollhttps://review.openstack.org/#/c/353995/ \o/11:30
*** moshele has joined #openstack-ironic11:30
aarefievdtantsur: would be nice to have this one https://review.openstack.org/#/c/347795/11:31
vdrokmorning11:31
aarefievmorning vdrok11:31
vdrokmorning aarefiev11:32
dtantsuraarefiev, yeah, if the author updates it today11:32
dtantsurmorning vdrok11:32
vdrokdtantsur: good morning11:32
moshelevdrok: hi, can you review again https://review.openstack.org/#/c/264263/11:35
vdrokmoshele: hello, will try to11:36
mosheledtantsur: I upload new patch for the vendor-id like JayF proposed https://review.openstack.org/#/c/279940/11:36
TheJuliaGood morning11:36
mosheledtantsur: also I hope I understood you comments  on https://review.openstack.org/#/c/264257/11:37
dtantsurmoshele, now you need your RFE https://bugs.launchpad.net/ironic-python-agent/+bug/1611856 approved11:37
openstackLaunchpad bug 1611856 in ironic-python-agent "[RFE] expose port vendor id and product_id" [Wishlist,In progress] - Assigned to Moshe Levi (moshele)11:37
dtantsurjroll, ^^^11:37
jrollwas this not in the spec?11:37
* jroll approves11:38
jroll¯\_(ツ)_/¯11:38
dtantsursambetts, you can remove your -2 now ^^^11:38
mosheledtantsur: so currently there is no usage for this, but maybe TripleO in the future11:38
jrollmuch process, wow11:38
TheJuliarcarrillocruz: I truly don't remember the set_fact bit, monty's review is right imho.  I think we split the roles and added omit after setting the fact stuffs11:38
sambettsdtantsur: done :)11:39
dtantsurthnx11:39
openstackgerritSzymon Borkowski proposed openstack/ironic-inspector: Add PCI devices plugin to inspector  https://review.openstack.org/35229511:40
sambettsdtantsur: code for the mac address update bug fix looks good for me, just waiting for Jenkins :)11:41
dtantsurcool11:42
*** chlong|mtg has joined #openstack-ironic11:43
lucasagomesdtantsur, can you take a look at https://review.openstack.org/#/c/354004/ ? ironic-staging-drivers gate is broken due that11:45
lucasagomesI mean, due one patch that landed in Ironic. That's fixing the tests in ironic-staging-drivers11:45
dtantsurLGTM, assuming it passes11:45
lucasagomesty11:52
*** lucasagomes is now known as lucas-hungry11:52
dtantsursambetts, could you please review https://review.openstack.org/#/c/353927/ ?11:53
dtantsurwe need it for the releaser11:53
dtantsur* release11:53
*** narasimha_SV has joined #openstack-ironic11:53
narasimha_SVSSHPowerDriver requires one and only one of password, key_contents and key_filename to be set.11:54
narasimha_SVI am using pxe_ssh driver for KVM11:54
*** xuao has joined #openstack-ironic11:54
narasimha_SVhttp://paste.openstack.org/show/554121/11:54
narasimha_SVwhen I try to validate the node I am getting this information11:55
narasimha_SVwhen I give ssh key path in the driver_info11:55
narasimha_SVthen http://paste.openstack.org/show/554122/11:55
narasimha_SVthis is what I am getting the issue11:56
*** xuao has quit IRC11:56
narasimha_SVplease help me in understanding what other values I missed here to get validation properly11:56
*** xuao has joined #openstack-ironic11:56
*** milan has quit IRC11:56
jrollnarasimha_SV: you had this problem earlier, it looks like a permissions issue, the user running the ironic-conductor service needs to be able to read the key11:57
narasimha_SVas part of connection i have ssh_address of the host , username and password of the host to connect11:57
sambettsdtantsur: sure looking now11:57
narasimha_SVjroll: is there any other way to do this11:58
narasimha_SVrather than giving the key file name11:58
jrollnarasimha_SV: username and password should work fine without setting key_filename11:58
jrollit's ssh_address, ssh_username, ssh_password11:58
narasimha_SVwhen I use username and password node is not getting validated properly11:58
narasimha_SVhttp://paste.openstack.org/show/554121/11:59
jrollare you using password or ssh_password? username or ssh_username?11:59
narasimha_SVyes ssh_username and ssh_password11:59
jrollare you sure there's no typos? please paste a node-show and censor the password12:00
narasimha_SVironic node-create -d pxe_ssh -i ssh_address='192.168.56.101' -i ssh_username='root' -i ssh_pasword='password'  -i ssh_virt_type='virsh' -n $NAME12:00
jrollssh_password, not ssh_pasword12:00
narasimha_SVok sorry12:00
rcarrillocruzTheJulia: yup, I think removing those facts will be functionally equivalent. I will test and push patchset if It works too12:00
jrollnarasimha_SV: no worries, you helped me find a bug :D12:01
openstackgerritJim Rollenhagen proposed openstack/ironic: Fix ssh credential validation message  https://review.openstack.org/35400812:01
jrollsuper easy review there :)12:01
jrollone liner12:01
narasimha_SVyes12:01
narasimha_SVthanks it worked12:01
jrollyou're welcome12:01
*** milan has joined #openstack-ironic12:03
TheJuliarcarrillocruz: awesome, thank you12:04
*** baoli has joined #openstack-ironic12:10
*** tuanla_ has quit IRC12:12
openstackgerritDao Cong Tien proposed openstack/ironic: Remove CONF.import_opt() from api/controllers/v1/node.py  https://review.openstack.org/35401212:15
*** hshiina has joined #openstack-ironic12:16
*** xuao has quit IRC12:16
*** xuao has joined #openstack-ironic12:18
openstackgerritMerged openstack/ironic-inspector: Fix release notes formatting  https://review.openstack.org/35392712:20
*** xuao has quit IRC12:20
*** xuao has joined #openstack-ironic12:21
*** MattMan_ has quit IRC12:22
*** MattMan has joined #openstack-ironic12:22
dtantsurjroll, as this ^^ landed, mind also proposing ironic-inspector release?12:24
*** rbartal has quit IRC12:24
*** dprince has joined #openstack-ironic12:27
jrolldtantsur: sure12:27
*** xuao has quit IRC12:27
jrolldtantsur: 4.1.0?12:27
* jroll looks at relnotes12:27
dtantsurjroll, yeah, nothing major I think12:28
* jroll is glad there's a formatting fix heh12:28
dtantsurwow folks, we've got an amazing Ironic release out this time. thanks all!12:29
* dtantsur has just taken a look at the Ironic release notes12:29
jroll+1, so much good stuff in this release12:29
sambetts\o/12:29
jrollthanks so much to everyone that contributed :)12:29
*** rbudden has joined #openstack-ironic12:30
*** aparnav has joined #openstack-ironic12:32
*** appu103_ has quit IRC12:32
*** amoralej is now known as amoralej|lunch12:35
*** ohamada has quit IRC12:35
*** ohamada has joined #openstack-ironic12:36
*** [1]cdearborn has joined #openstack-ironic12:39
xavierrGood morning Ironic 6.1.0 ;)12:40
nicodemos=D12:40
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Refactoring code of generating temp_url  https://review.openstack.org/35228912:41
jrollxavierr: :D12:42
*** sdake has quit IRC12:43
jrolldtantsur: oh, I forgot to link you to the release request https://review.openstack.org/#/c/354026/12:46
dtantsuryeah, but I've seen it :) thanks12:46
jrollah, cool12:46
*** baoli_ has joined #openstack-ironic12:47
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Tempest: increase ironic sync timeout  https://review.openstack.org/35049912:48
*** hamzy has quit IRC12:48
*** baoli has quit IRC12:50
*** aparnav has quit IRC12:51
openstackgerritJim Rollenhagen proposed openstack/ironic: Fix ssh credential validation message  https://review.openstack.org/35400812:51
*** zhangjl1 has left #openstack-ironic12:51
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command  https://review.openstack.org/25145812:52
*** lucas-hungry is now known as lucasagomes12:55
*** amoralej|lunch is now known as amoralej12:58
*** rloo has joined #openstack-ironic13:00
*** vmud213 has quit IRC13:00
*** appu103_ has joined #openstack-ironic13:06
lucasagomesvdrok, thanks for merging the gate fix ;D13:07
*** wajdi has quit IRC13:12
openstackgerritJim Rollenhagen proposed openstack/ironic: Add docs for node.resource_class and flavor creation  https://review.openstack.org/34508113:12
jrollrloo: I feel like we'll have a couple back-and-forths there :)13:13
jrollno rush on that patch though13:14
*** cdearborn has joined #openstack-ironic13:14
rloojroll: really? I wonder why :)13:14
jrollhehe13:14
rloojroll: I was just thinking that we really should get the docs done before we push a release (or say a feature is done)13:14
jrolljaypipes: ^ would like your review on that one too13:14
jrollrloo: if we want to do the former, we'll need to require docs with the code13:14
rloojroll: other random thought yesterday. given the amount of time/revisions it took to get the release notes done (I know, most of the delay was zuul), maybe we really ought to spend more time reviewing/cleaning up the release notes when they are originally proposed. (I really did want to punt that for the release person at the end but it does affect others too like me. ha ha.)13:16
*** z- has quit IRC13:16
jrollrloo: I completely agree, while we were going back and forth I was wishing the same thing, the reviews were really good13:17
rloojroll: i will keep that in mind for the future. I had let some things slide before.13:18
*** [1]cdearborn has quit IRC13:18
jrollrloo: I'm here to serve the team, so I don't mind doing the work to get them in order, but I think the reviews are the harder part of it so it doesn't really take any load off of people13:18
jrollrloo: just takes some iteration load off of the committer13:18
rloojroll: right.13:18
rloojroll: i mean, right, you're here to serve the team! :D13:18
jroll:)13:19
*** jcoufal has joined #openstack-ironic13:23
*** z- has joined #openstack-ironic13:24
jaypipesjroll: done.13:24
jrollthanks jaypipes13:25
jrollvsaienk0: our grenade job upgrades everything, right? e.g. both nova and ironic are updated?13:26
vsaienk0jroll: yes, it upgrades everything13:26
*** narasimha_SV has quit IRC13:26
jrollok thanks13:26
* jroll writing upgrade docs13:26
*** jistr has quit IRC13:28
lucasagomeshi all, can I get some eyes on https://review.openstack.org/#/c/352820/ ? It's fixing a hi-prio bug and is +2 already13:29
lucasagomesI also want to backport it to the mitaka branch once merged13:29
*** appu103_ has quit IRC13:29
*** ppiela has joined #openstack-ironic13:30
*** moshele has quit IRC13:30
openstackgerritCornelio Hopmann proposed openstack/ironic-specs: [RFE] Support network switches provisioning  https://review.openstack.org/35057013:30
*** jistr has joined #openstack-ironic13:30
openstackgerritJim Rollenhagen proposed openstack/ironic: Bring upgrade documentation up to date  https://review.openstack.org/35407513:33
jrollwhee13:33
*** ohamada has quit IRC13:33
*** ohamada has joined #openstack-ironic13:33
sambettslucasagomes: +A, see my comment though13:37
openstackgerritRicardo Carrillo Cruz proposed openstack/bifrost: Default auth to empty dict instead of None  https://review.openstack.org/35399013:40
openstackgerritJim Rollenhagen proposed openstack/ironic: Clean up release note ipmitool-bootdev-persistent-uefi  https://review.openstack.org/35407913:40
jrolllucasagomes: and a follow-up to clean up the release note ^13:40
jrollsambetts: if it's a problem in pyghmi, we should just fix it in pyghmi13:41
jrolllike later versions of ipmitool fixed it :)13:41
jrolljaypipes: oh wow, I got all that right?!13:41
* jroll mind blown13:42
sambettsjroll: I meant more like adding the logic to actually include the efiboot option based on the nodes boot mode into the IPMINative power interface13:42
sambettsno idea if pyghmi supported it :/13:42
lucasagomessambetts, jroll I was thinking, it would be better to fix it all in 1 commit13:43
lucasagomessambetts, jroll to facilitate the backport13:43
lucasagomessambetts, can you revoke that +a and I will update the patch?13:43
jrollsambetts: oh I see, you're right13:43
jrolllucasagomes: no, please fix separate drivers in separate backports13:43
lucasagomesre ipminative... I've no idea if that works, I have to check13:43
jrollunless you mean the release note, +1 on that13:43
*** daemontool has joined #openstack-ironic13:43
lucasagomesjroll, I mean, the followups13:43
jroll++, let's squash it13:43
jrolllucasagomes: you can -2 to stop the merge :)13:43
*** mgoddard_ has joined #openstack-ironic13:44
lucasagomesjroll, done :D13:44
jrollcool13:44
lucasagomesand I will test the ipminative driver in the env I have13:45
jrolllet me know when you squash it and I can approve and abandon my patch13:45
dtantsurjroll, lucasagomes, removing W+1 is enough13:45
sambettslucasagomes: awesome :), I've never used IPMINative13:45
jrolldtantsur: yeah, just telling lucas how to do it if sam disappeared or for the future13:45
dtantsuroh yeah13:45
* sambetts vanishes into the night like batman 13:46
*** sborkows has joined #openstack-ironic13:46
sambetts:-P13:46
* jroll always forgets this part13:47
*** sdake has joined #openstack-ironic13:47
jroll#success ironic 6.1.0 is released with multitenant networking, keystone policy, metrics, and more!13:47
*** mgoddard has quit IRC13:47
openstackstatusjroll: Added success to Success page13:47
lucasagomes * [new tag]         6.1.0      -> 6.1.013:48
lucasagomesw00t13:48
dtantsur\o/13:48
dtantsuralso the ironic-inspector release features LLDP discovery13:49
jrollawesome!13:49
*** hamzy has joined #openstack-ironic13:49
sambetts\o/13:49
*** thiagop has joined #openstack-ironic13:50
thiagopGood morning, ironic13:50
jrollhey thiagop :)13:52
openstackgerritLucas Alvares Gomes proposed openstack/ironic: IPMITool: Check the boot mode when setting the boot device  https://review.openstack.org/35282013:53
lucasagomessambetts, jroll ^ fixed the commit message as well13:53
lucasagomesthanks for the review13:53
*** abashmak has joined #openstack-ironic13:54
sambettslucasagomes: awesome :D13:54
*** hoangcx has joined #openstack-ironic13:54
*** Lord_Phoenix has quit IRC13:54
*** abashmak has left #openstack-ironic13:54
*** vikrant has quit IRC13:54
jrolllucasagomes: cool, thanks13:57
sambettslucasagomes: do you think we should add a comment to the raw_cmd as jroll suggested?13:58
* jroll +213:58
lucasagomessambetts, I can do that if needed yes13:58
jrollI'm not too worried about it13:58
lucasagomesyeah, there are other raw byte strings around the code without explanation as well13:59
sambettsme neither, if its all going to be removed, just wanted to check13:59
lucasagomesmaybe we can put a patch explaning 'em all13:59
lucasagomessambetts, yeah the idea is to remove that code once distros update their ipmitool package13:59
lucasagomesthing is, it may take some time13:59
*** anush_ has joined #openstack-ironic14:00
*** vikrant has joined #openstack-ironic14:01
*** vikrant is now known as vikrant|gone14:01
sambettsyeah, you refer to the ipmi spec further up anyway if people are interested14:01
*** baoli_ has quit IRC14:02
*** dprince_ has joined #openstack-ironic14:03
*** dprince has quit IRC14:04
*** sdake_ has joined #openstack-ironic14:07
*** sdake has quit IRC14:09
jrolloh, this can go now that we did the release https://review.openstack.org/#/c/352991/14:09
rloojroll: ? i thought that needed to wait til the newton release.14:09
rloojroll: you know that the next one will be the newton release?14:10
jrollrloo: I planned to have it in the final newton release14:10
rloojroll: so shouldn't we wait til you are about to do the final release?14:10
jrollrloo: yeah, maybe we should14:11
jrollor at least until we know there's nothing between14:11
*** [1]cdearborn has joined #openstack-ironic14:11
rloojroll: right, cuz we could do 4 releases between now and then :)14:11
jrollcould and will are different, but yeah maybe we can get a bunch of stuff done and have 6.2 in a few weeks14:11
rloojroll: yup14:12
jrollokay, I re-minus-2'd14:13
rloojroll: wrt upgrade documentation. do you know how the 'official' openstack doc handle that kind of documentation?14:13
jrollit could still use reviews though :D14:13
jrollrloo: there's an upgrades section in the ops guide or something14:13
rloojroll: yeah, i'll review, but figured i'd procrastinate on it. i was actually wondering if we should only do that in an ocata release but didn't want to think about that too much.14:14
jrollyeah, no worries14:14
jrollI don't want to block us from removing it in ocata14:14
jrollbut I think it will depend on how well the new thing works in prod :)14:14
rloojroll: any idea how we can get our upgrades in that ops guide?14:14
jrollrloo: I suspect they will 'big tent' it soon, like the api-ref and the install guide14:15
jrollso I'm kinda just waiting for that14:15
jrollI want to do the plugin thing for our install guide in ocata14:15
rloojroll: oh, so not there yet. ok. we *should* get our install guide there though.14:15
jroll+114:15
jrollrloo: I think after the install guide we can start asking about the upgrade guide14:16
rloohow much work is it to get the plugin? would be nice to have it in newton.14:16
jrollI'm not sure, it shouldn't be terrible14:16
*** jistr is now known as jistr|call14:16
jrollhere's the guide: http://docs.openstack.org/contributor-guide/project-install-guide.html14:16
rloowhere's JayF? Does he know?14:16
jrollsome project-config changes14:16
jrollsome in-repo changes14:16
rloojroll: am going to add a bug about that unless there is one already. i wonder if it is low-hanging-fruit...14:17
jrollmy worry toward end-of-cycle would be about conflicting with other install guide changes14:17
vdroklucasagomes: around?14:17
jrollrloo: I'm not sure if there is a bug or not yet14:17
vdrokmorning jroll  and rloo14:17
jrollhey vdrok :)14:17
rloojroll:  no bug. is it a bug or a RFE to move the install guide?14:18
rloohi vdrok14:18
jrollrloo: probably RFE, but I can approve it now if you file one :)14:18
lucasagomesvdrok, hi there14:19
vdroklucasagomes: I was wondering about enabling the iboot driver in the staging drivers gate job, I thought you said you'll do that, or do you want to do that in a separate change?14:19
lucasagomesvdrok, oh, I think I forgot to do that... but if we could do in a separated change it would be fine too14:20
lucasagomeswould save me some time actually14:20
vdroklucasagomes: yup, +2 then :)14:20
*** wajdi has joined #openstack-ironic14:20
lucasagomescool, I'm just testing the ipminative with persistent + uefi and I will see that right after14:20
* lucasagomes adds a todo14:20
*** wajdi has joined #openstack-ironic14:21
lucasagomesvdrok, thanks for the reminder14:22
rloojroll: here's the RFE: https://bugs.launchpad.net/ironic/+bug/161227814:22
openstackLaunchpad bug 1612278 in Ironic "[RFE] publish install guide on docs.openstack.org" [Wishlist,New]14:22
*** cdearborn has quit IRC14:22
*** beekneemech is now known as bnemec14:23
jrollthanks rloo, approved14:23
rloojroll: might be more than 'just' following those instructions. we've been talking about reorg'ing the install guide. something to keep in mind i guess. will think about it later :)14:23
jrollrloo: yeah, I think we should :)14:24
jrollor at least break it up a bit14:24
*** causten has joined #openstack-ironic14:24
vsaienk0jroll, sambetts: could you please have look https://review.openstack.org/#/c/353921/1/jenkins/jobs/ironic.yaml14:24
jrollmaybe JayF can architect how that's laid out :D14:24
*** joprovost has joined #openstack-ironic14:24
*** syed_ has joined #openstack-ironic14:25
rloojroll: heh, yeah, i was going to ping JayF about that :)14:25
vsaienk0jroll, sambetts: that patch switch multinode job to use multitenancy14:25
sambettsvsaienk0: making all jobs test multitenant?14:25
sambettsoooh right yes14:25
jrollvsaienk0: +114:25
vsaienk0sambetss: no only multinode jobs14:25
jrollrloo: I think his ops experience will be super helpful figuring out how to group that14:26
rloojroll: i totally agree! ;)14:26
rloojroll: mat128 might also be helpful there14:26
jroll+114:26
sambettsvsaienk0: can we get rid of other multitenancy job then?14:26
mat128reading scrollback :)14:27
*** joprovost has quit IRC14:27
vsaienk0jroll: once more, when you have a time https://review.openstack.org/#/c/340295/, we talked about multitest job.14:27
jrollmat128: tl;dr breaking up the install guide14:27
mat128oh14:27
mat128sure14:27
*** joprovost has joined #openstack-ironic14:27
jrollsambetts: oh, we probably could14:27
jrollgotta get multinode stable first14:27
rloomat128: in preparation for putting the install guide in the ops documentation14:27
vsaienk0sambetts: you mean when we have green multinode job with multitenancy remove current single node multitenancy job?14:27
sambettsyup14:28
vsaienk0sambetts: sure, I think this is our goal14:28
rloomat128: in fact, you interested in doing it? https://bugs.launchpad.net/ironic/+bug/161227814:28
openstackLaunchpad bug 1612278 in Ironic "[RFE] publish install guide on docs.openstack.org" [Wishlist,Confirmed]14:28
mat128yup sure14:28
rloomat128: awesome! and thank you!14:29
sambettsvsaienk0: I'm interested to what we are gaining by testing multutenantcy in the mutinode job though14:29
lucasagomesjroll, sambetts *sigh* ipminative does not check the boot mode either14:29
lucasagomesgood news is that pygmi itself works setting persistent + uefi14:29
* lucasagomes uploads a patch14:29
mat128rloo: assigned myself to the bug, added to my todo list14:30
mat128thanks for the link14:30
rloothank YOU mat128. I expect great things from you. Ha ha.14:30
mat128=)14:30
sambettslucasagomes: tbh I wonder how many other drivers don't take that into account? and if that sort of thing happens in non-ipmi cases14:30
vsaienk0sambetts: we will be able to add 7 VMs on primary node and 7 VMs on subnode, it will allow to increase tempest concurrency14:30
lucasagomessambetts, probably many others14:30
jrollvsaienk0: is there a patch to add exercise.sh yet?14:31
lucasagomessambetts, I think we know that drac doesn't work with efi either14:31
mat128sambetts, lucasagomes: wasnt it only an IPMI bug?14:31
mat128ah, that kind of drivers :)14:31
*** hjanssen-hpe has joined #openstack-ironic14:31
vsaienk0jroll: not yet, but I'm going to start working on it once we have a job :)14:31
*** hpe-hj has joined #openstack-ironic14:31
jrollsambetts: we're using multitenancy because our current "flat" doesn't work with multinode, we either need real flat networking or multitenant, and multitenant is the future :D14:31
*** amotoki has joined #openstack-ironic14:31
jrollvsaienk0: okay14:31
lucasagomessambetts, it would be good to have a uefi test in gate14:31
sambettslucasagomes: right14:31
lucasagomessambetts, we could configure qemu with uefi I think14:32
vsaienk0sambetts: btw I already prepared patches to work with real FLAT setup14:32
sambettslucasagomes: it would be good to have a tempest test case for an EFI node that can be run14:32
jrollvsaienk0: +114:32
lucasagomessambetts, ++14:32
* lucasagomes fix ipminative for now14:32
sambetts++14:32
sambettssigh deployment purmutation testing is a pain in the ass :-P14:33
*** chlong|mtg has quit IRC14:33
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for port-related commands  https://review.openstack.org/25935414:33
lucasagomessambetts, yeah, too many configs :-)14:33
jrollyeah, we really need to work on that next cycle14:33
lucasagomesjroll, re gate jobs... I want to use the wipe disk metadata cleaning step once merged14:34
lucasagomesto test the cleaning steps in gate14:34
*** michauds has joined #openstack-ironic14:34
lucasagomeswe have a bug open for that (I guess)14:34
jrolllucasagomes: +114:34
dtantsurgreat idea14:34
jrolllucasagomes: you plan to validate it was wiped as well?14:34
lucasagomeswiping the disk metadata is really fast, so I think it suits for gate tests14:34
sambettsI can imagine infra continuing to get annoyed at us for too many jobs ;)14:34
lucasagomesjroll, yeah, I would mount the device in a loop and see if the partition table has been wipes14:34
lucasagomeswiped14:34
lucasagomesafter the node is cleaned14:35
jrolllucasagomes: ++14:35
jrollawesome14:35
lucasagomesthe device I mean, disk device14:35
dtantsursambetts, I think nova/neutron still have more, but I may be wrong14:35
jrolldtantsur: seems about the same for nova https://review.openstack.org/#/c/354030/14:35
jrollcompared to https://review.openstack.org/#/c/352991/14:35
mat128we can always specify which jobs for which parts of the codebase changed14:37
mat128to reduce total jobs that run14:37
mat128lucasagomes: wow that'd be great14:37
jrollplease no, that'll get way too complex14:37
mat128hahaha14:37
lucasagomesmat128, yeah, shouldn't be difficult to write the tempest bits14:37
lucasagomesmat128, once the patch merges I will look at it14:37
jrollI just noticed sambetts' refactor accidentally(?) made our job on nova voting14:38
sambettsI'd really like us to do job that test parts of the deployment process, e.g. specifcally provisioning, specifically cleaning etc14:38
sambettsinstead of the whole process14:39
sambettsjroll: did i?!14:39
jrollheh, yeah14:39
sambettswhoops XD14:39
*** jtomasek_ has quit IRC14:40
sambettsI must have missed an -nv somewhere14:40
jrollwe might want to fix that before the end-of-release rush starts pounding on the gate14:40
jrollswitching that to ipmitool will make it worse, probably14:40
jrollgotta step away for a bit14:42
*** PollyZ has joined #openstack-ironic14:44
sambettsjroll:  https://review.openstack.org/35412414:44
*** chlong|mtg has joined #openstack-ironic14:46
jrollsambetts: thanks14:46
*** sdake has joined #openstack-ironic14:47
*** sdake_ has quit IRC14:47
openstackgerritAkilan Pughazhendi proposed openstack/ironic: Devstack setup fails on Fedora 23  https://review.openstack.org/35413614:52
*** sborkows has quit IRC14:52
*** rajinir has joined #openstack-ironic14:58
dtantsurwith our product we hit all kinds of strange hardware/drivers, and here is an example:14:59
dtantsurdmidecode does not report memory: https://bugzilla.redhat.com/show_bug.cgi?id=136630914:59
openstackbugzilla.redhat.com bug 1366309 in openstack-ironic-inspector "ironic introspection fails on GCE node" [Unspecified,New] - Assigned to dtantsur14:59
dtantsurany ideas what else to use/how to fix it?14:59
mat128dtantsur: No Module Installed15:00
mat128wow15:00
lucasagomesdtantsur, maybe fallback to /proc/meminfo for ram if it fails to get it from dmidecoce ?15:01
mat128dtantsur: maybe also a dmesg dump15:01
lucasagomesand yeah, that's an odd output there >.<15:01
mat128dtantsur: Ravello backed by something else doesnt do it?15:01
mat128according to reporter15:01
dtantsurlucasagomes, /proc/meminfo gives wrong data IIRC15:01
mat128ehhh15:02
mat128how much ram does it have?15:02
dtantsurdunno, just got this report15:02
mat128dtantsur: is it something like the xen memory balloning thats the issue?15:02
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command  https://review.openstack.org/25145815:02
mat128i understand it's not xen, but this bit me in the past :)15:02
lucasagomesdtantsur, really? It may be because some of the ram is used for hardware interruption15:02
lucasagomesbut I thought you had all information there15:03
dtantsurlucasagomes, yeah, it does not account for reserved memory15:03
lucasagomesyou may have to consider few other fields apart from TotalMem15:03
mgould`man free` suggests MemTotal + SwapTotal15:04
mat128free -m reports usable, not machine capacity :(15:04
mat128mgould: ^15:04
mat128http://paste.openstack.org/show/554193/15:05
mgouldmat128: so "capacity - space taken up by the kernel itself"?15:05
mat128or even this on a physical machine: http://paste.openstack.org/show/554194/15:05
mat128oh, journal dump has kernel logs15:06
mat128for anyone not wanting to download the file http://paste.openstack.org/show/554195/15:07
mat128L19 and 20, thats a kvm virtual machine15:07
mat128I wonder how they came up with no memory bank installed15:07
dtantsurthat's what I wonder too15:08
mat128is Ravello open source? guess not :(15:08
dtantsur"It is a Google Compute Engine VM" hmm, I wonder if we support that at all15:08
mat128can we have the qemu/kvm cmdline ?15:08
mat128look up Ravello15:09
mat128Aug 11 09:38:15 localhost.localdomain kernel: Memory: 16048252k/17825792k available (6442k kernel code, 1049096k absent, 728444k reserved, 4264k data, 1620k init)15:09
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command  https://review.openstack.org/25145815:09
dtantsurnevermind folks, I suspect it should be CLOSED WONTFIX15:10
mat128well15:10
mat128I guess we shouldn't be solely relying on dmidecode15:10
mat128but at the same time, I have never seen dmidecode report "no memory bank installed"15:10
dtantsuryeah, probably a limitation of GCE15:10
mat128well15:11
dtantsurI'd not rush changing how we detect memory based on it alone15:11
*** penick has quit IRC15:12
jrolllol, GCE VM?15:14
*** PollyZ has quit IRC15:15
mat128Ravello (i think) runs a vm on top of GCE15:15
mat128that apparently has only 1 memory slot with no memory bank installed15:15
jrollso much meh15:15
*** vikrant|gone has quit IRC15:17
*** hshiina has quit IRC15:17
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix Ubuntu 16.04 pxelinux path handling  https://review.openstack.org/32602415:17
mat128dtantsur: pretty sure they're using something like this https://github.com/qemu/qemu/blob/master/docs/memory-hotplug.txt15:18
mat128dtantsur: probably defining it with no memory and hotplugging some before starting the machine15:18
dtantsur>_<15:18
mat128I'd love to try it :)15:19
mat128maybe I can find smt15:19
dtantsurI'd love folks to stop using Ironic to manage VMs in production15:19
sambettsthis ^15:19
jroll+100015:19
openstackgerritLucas Alvares Gomes proposed openstack/ironic: IPMINative: Check the boot mode when setting the boot device  https://review.openstack.org/35415315:19
sambettsrm -rf VM Drivers15:19
jrollgotta wait a few more weeks, sambetts :P15:19
mat128Ravello is a lab thing, they might be doing exactly what we are doing with devstack15:19
lucasagomes^ patch fixing pygmi set_boot_device() to consider the boot mode15:19
mat128sambetts: you deleted all drivers and all VMs ;)15:20
sambettsmat128: yup :-P15:20
jrolllucasagomes: LGTM15:21
*** ifarkas is now known as ifarkas_afk15:21
lucasagomesjroll, that was quick... thanks :D15:21
jrollyeah np :)15:21
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix Ubuntu 16.04 pxelinux path handling  https://review.openstack.org/32602415:21
jrollit was a pretty straightforward patch :P15:21
*** jistr|call is now known as jistr15:21
lazy_princehas anyone tried using devuser element with ubuntu..? In my case, the user gets deleted after instance boot15:22
lucasagomesyeah, luckily pyghmi already had a parameter for it15:22
sambettslazy_prince: I've had it working with ubuntu15:22
*** tesseract- has quit IRC15:23
lucasagomesand the behavior described in the pyghmi docstring is exactly what we've experienced https://github.com/openstack/pyghmi/blob/657608ca605034245cc537022b414708f9d10ea0/pyghmi/ipmi/command.py#L332-L33615:23
sambettslucasagomes: Nice!15:24
lazy_princesambetts: can you share how you built your image.. just dib cmd and parameters and any environment vars that i need to export15:24
openstackgerritRicardo Carrillo Cruz proposed openstack/bifrost: Remove auth line to fallback on default(omit) behaviour  https://review.openstack.org/35399015:24
*** baoli has joined #openstack-ironic15:25
sambettslazy_prince: export DIB_DEV_USER_PASSWORD=password15:26
sambettsdisk-image-create ubuntu baremetal dhcp-all-interfaces grub2 devuser -o my-image15:26
sambettsthen I've logged in with devuser and password15:27
lazy_princeI did same but its not working for me..15:27
*** catintheroof has joined #openstack-ironic15:27
lazy_princeI can see the image has necessary info.. but after cloud-init is over, they disappear..15:28
sambettslazy_prince: I don't think cloud-init should touch the users built into the image unless you have a custom cloud config that is deleteing it :/15:30
lazy_princek.. let me try again...15:31
lucasagomeslazy_prince, you can try the dynamic-login element as well15:31
lucasagomeswell tho it's more for the ramdisk...15:31
* lazy_prince checking15:31
openstackgerritMerged openstack/ironic: Fix ssh credential validation message  https://review.openstack.org/35400815:32
mgoulddtantsur: people use Ironic for managing VMs in production? I thought those drivers were only used for testing...15:37
*** rbrud has quit IRC15:37
dtantsurboth are correct, unfortunately15:38
mat128mgould: I think it's their test environment15:38
mat128Ravello is a product specifically for that use case15:38
mgouldah, OK15:38
mat128but dtantsur is closer to the customer than I could be15:38
mat128so consider me wrong ;)15:38
mgouldCLOSED YOUHAVEMADEYOURBEDANDMUSTNOWLIEINIT15:38
rloohi dtantsur (and others). about https://bugs.launchpad.net/ironic/+bug/1569254, we allow updates to the node in lots of other situations (besides the console). eg, if node is active or available, i think we allow ipmi-related stuff to be modified. Maybe we need some better guidelines as to when things are allowed to be updated.15:39
openstackLaunchpad bug 1569254 in Ironic "It's possible to update node's console related properties while console is running" [Low,In progress] - Assigned to Pavlo Shchelokovskyy (pshchelo)15:39
mat128rloo: interesting bug15:39
dtantsuryeah, maybe, I haven't put much thought in it15:39
mat128rloo: preventing IPMI config changes while node is active or available might a problem though15:40
mat128what if you actually need to change the values?15:40
rajinirthiagop, sambetts: Is there a specific setting like port_enabled_security for tftp? The slave VM is not seeing the tftp traffic15:40
mat128rloo: console isnt too bad because you can just stop it, change and start it again15:40
rloodtantsur: me neither. but clearly, something we need to keep in mind when reviewing stuff...15:40
*** jcoufal has quit IRC15:40
sambettsrajinir: once port security is disabled all traffic should be allowed through15:40
mgouldmat128, rloo: can you prevent the user from calling ipmitool directly?15:41
mat128mgould: how could you?15:41
rloomat128: yeah, just wondering if we can come up with some guidelines to make it consistent/easy to know when things can be updated or not.15:41
mgouldmat128: that's what I thought15:41
rloomat128: anyway, just a thought. not going to think about it now :)15:41
mgouldso preventing IPMI changes at the Ironic level doesn't buy us much safety15:41
mat128mgould: the bug was about changing ipmi_address, user and pass15:42
rloomat128: otherwise, we are piecemeal addressing them as someone encounters an issue and they are coded/addressed in different ways.15:42
mat128or, more precisely, equivalents for the serial console15:42
lazy_princesambetts: figured out that the server has two disks with each one having "cloudimg-rootfs" partition... and post boot, it was mounting wrong partition15:42
mat128rloo: we need a guideline thats for sure15:42
mgouldoh, OK15:42
sambettslazy_prince: ah :( thats annoying15:42
mat128but if for some reason my bmc is at a different address now, i dont want Ironic to prevent me (and possibly keep acting on a machine thats not the right one)15:42
*** psachin has quit IRC15:42
*** links has quit IRC15:43
mgouldmat128: ah, gotcha15:43
sambettsmgould, mat128: tbh I think we should prevent all updates like that for nodes in active unless moved to maintenance15:44
mat128sambetts: makes sense15:44
mat128they could be validated when you get out of maintenance15:45
sambettsproblem is that, that then leads to API behavior changes :'(15:45
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add "erase_devices_metadata_priority" config option  https://review.openstack.org/34417915:45
mat128rloo: i think something like what sambetts is suggesting could make a lot of sense and would be a general approach15:45
mat128sambetts: what do you mean?15:45
rloomat128, sambetts: yup.15:46
sambettsmat128: something that was allowed before now is disallowed, so it technically breaks the API15:46
lucasagomesvdrok, updated the sample file (plenty of new inclusions there) ^15:46
mat128sambetts: argh... not again :)15:46
mat128microversion15:46
vdroklucasagomes: thx :)15:46
jrollthiagop: are you still working on dropping the tests in tempest tree?15:46
rloosambetts: interesting. does it break the API, or is it fixing a bug so it is OK.15:47
rloosambetts: eg, look at the actual bug that was mentioned. if we fix it, does it need a version bump/15:47
jrolllucasagomes: would you mind landing this so that sambetts can have a build going? https://review.openstack.org/#/c/353992/15:48
lucasagomesjroll, boom done :-)15:48
jrollthanks15:48
*** sdake_ has joined #openstack-ironic15:49
sambettsrloo: right, this is somewhat like what me and dtantsur did with the port mac address in a recent patch to prevent updates on an active node unless its in maintenance because it will result in network disruption15:49
*** mgoddard_ has quit IRC15:49
*** mgoddard has joined #openstack-ironic15:50
*** sdake has quit IRC15:51
*** milan has quit IRC15:51
jlvillaldtantsur: Do you customize your colors for gertty? I'm not sure I'm thrilled with the defaults but not sure if I can come up with better ones :)15:52
dtantsurjlvillal, no, I use the defaults15:52
jlvillaldtantsur: Thanks15:52
sambettslucasagomes: thanks, hopefully once that merges the post job will pass15:53
lucasagomessambetts, ++15:53
rajinirsambetts: Able to see dhcp traffic, and the node gets ip assigned, but times out on TFTP. Any ideas on how to debug this?15:54
mat128rajinir: tftp server not reachable?15:54
mat128rajinir: tftp service not running?15:54
rajinirmat128: TFTP open timeout15:55
sambettsrajinir: TFTP address in the Ironic configuration the correct one? based on the subnet the node is in15:55
mat128can you tftp get from another machine?15:55
lazy_princesambetts: probably, I should have disk cleaning enabled to avoid such issues..15:55
openstackgerritMerged openstack/ironic-python-agent: Make code blocks real code blocks in metrics docs  https://review.openstack.org/35399215:56
rajinirmat128, sambetts: where is this configuration? Is it in ironic.conf?15:57
mat128rajinir: in your DHCP traffic dump, is the "next-server" IP the right one?15:57
sambettsrajinir: you may need to set HOST_IP= the IP address of the slave VM in the network that the baremetal node is15:57
sambettsrajinir: yes ironic.conf15:57
mat128rajinir: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L244815:57
openstackgerritAkilan Pughazhendi proposed openstack/ironic: Fixes devstack setup failure on Fedora 23  https://review.openstack.org/35413615:58
*** praneshp has joined #openstack-ironic16:01
*** bnemec has quit IRC16:04
rajinirmat128>, sambetts: tftp_server set to the HOST_IP, looks right16:04
rajinirmat128, sambetts: I can see the host machine is forwarding the packets to the correct IP.16:05
openstackgerritLucas Alvares Gomes proposed openstack/ironic: IPMITool: Check the boot mode when setting the boot device  https://review.openstack.org/35418616:06
mat128rajinir: so the node is making tftp requests and the server is replying but they reply isn't getting through?16:06
rajinirsambetts: tftp is not running on the salve vm.16:06
lucasagomesbackport ^16:06
mat128rajinir: can you describe/draw your logical setup?16:06
mat128lucasagomes: are you submitting ipminative for backport too?16:06
lucasagomesmat128, yup doing it now16:06
sambettsrajinir: oh? have your not using a normal devstack deployment?16:06
*** sabeen has joined #openstack-ironic16:07
rajinirmat128: This is thirdparty CI. so the request flows from Hardwarenode->Devstack Host Node -> Slave Vm running ironic tests16:07
*** e0ne has quit IRC16:07
openstackgerritLucas Alvares Gomes proposed openstack/ironic: IPMINative: Check the boot mode when setting the boot device  https://review.openstack.org/35418716:07
lucasagomesmat128, there you go :D ^16:07
mat128rajinir: no problem, but where is the node?16:07
mat128within "Slave Vm running ironic tests" ?16:07
jlvillallucasagomes: Was there a bug in that patch?16:08
rajinirsambetts: I do use the devstack gate . Is there any configuration I'm missing?16:08
mat128rajinir: you mentioned the TFTP running somewhere else16:08
jlvillallucasagomes: Ah, back port! Nevermind16:08
mat128can you describe that?16:08
lucasagomesjlvillal, those are the backport :D16:08
sambettsrajinir: devstack should set up tftp for you on the jenkin slave16:08
rajinirmat128: tftp should be running on the slave vm's16:09
lucasagomesjlvillal, thanks for approving btw16:09
jlvillallucasagomes: Thanks for fixing it! :)16:09
lucasagomes:-)16:09
rajinirsambetts: no idea why tftp is not running, it is supposed to bet setup, agree16:11
*** bnemec has joined #openstack-ironic16:12
NobodyCamGood Morning Ironicers16:12
mat128hi NobodyCam16:12
NobodyCammorning mat128 :)16:13
*** rcernin has quit IRC16:15
*** baoli has quit IRC16:17
*** baoli has joined #openstack-ironic16:18
dtantsurmorning NobodyCam and good night to everyone else :)16:18
*** dtantsur is now known as dtantsur|afk16:18
sambettsNight dtantsur|afk16:20
sambettsrajinir: how are you checking that its running?16:20
rajinirsambetts: I don't see the service running. "service tftp status". There is no service installed.16:21
sambettsin the ironic devstack script we run TFTP using xinetd16:22
*** daemontool has quit IRC16:23
sambettsjroll: do you know how to check if TFTP is running the way we run in it our devstack script ?16:23
NobodyCamhave a good night dtantsur|afk16:24
rajinirsambetts: yeah, figured that out. But the file is setup right16:24
rajinirhttps://www.irccloud.com/pastebin/EVrUKBuL/tftp16:24
jrollsambetts: I've no clue, ps aux?16:25
sambettsmaybe netstat and look for the tftp port might be the best option then :-P16:25
jrollnetstat would be helpful, too, make sure it's running on the right interface16:25
jrollyeah16:25
sambettson your VM slave host machine, can you see the packets arriving from the baremetal node/16:26
sambetts?16:26
sambettsif you do a tcpdump on the network interface that goes into that network16:26
lucasagomesNobodyCam, hi there16:27
*** PollyZ has joined #openstack-ironic16:27
rajinirsambetts: I can see the packets on the  host machine and getting passed to the Slave VM ip/HOST_IP16:28
*** ijw has joined #openstack-ironic16:29
lucasagomessambetts, perhaps sending a "tftp get" (or another command) and check if it's running and listen ?16:29
*** sabeen1 has joined #openstack-ironic16:29
lucasagomesor check the tftp port via netstat idk16:30
NobodyCamhey hey lucasagomes Good morning to you :)16:30
NobodyCamalso Good morning jroll sambetts rloo thiagop jlvillal JayF TheJulia and everyone else not listed here16:31
jaypipesmornin16:31
rloohi, morning NobodyCam! which means lunchtime for me :)16:32
openstackgerritMichael Turek proposed openstack/ironic: Add classes for Portgroups API enablement  https://review.openstack.org/34754916:32
*** sabeen has quit IRC16:32
NobodyCam:)16:32
jlvillalHi NobodyCam :)16:33
NobodyCam:) hey jlvillal :)16:34
jlvillalHey jaypipes :) I vote for a Salmon as the Nova mascot ;)16:34
jrollmorning NobodyCam :)16:34
* jlvillal learned that a type of salmon is called Nova16:34
openstackgerritMichael Turek proposed openstack/ironic: Add classes for Portgroups API enablement  https://review.openstack.org/34754916:34
jlvillalOr maybe that is, cured and smoked salmon16:35
jlvillalmmmmm.  smoked salmon :)16:35
NobodyCamhey jroll :)16:35
lucasagomessambetts, http://paste.openstack.org/show/554225/16:35
lucasagomesrloo, hello too :-) didn't see you around16:35
rloolucasagomes: hi! no worries. i'm trying to be invisible :)16:36
lucasagomes:D16:36
*** jpich has quit IRC16:37
*** fxpester|2 has quit IRC16:40
*** sabeen1 has quit IRC16:40
TheJuliagood morning NobodyCam16:43
mgouldmorning NobodyCam16:44
*** aNupoisc has joined #openstack-ironic16:44
mgouldand hi to TheJulia jlvillal rloo jroll rajinir and anyone else who snuck in while I wasn't paying attention :-)16:45
jrollhey mgould :)16:45
* mgould is currently rather distracted by last-minute prep for https://twitter.com/AndyMilesExped16:45
sambettsmgould: oooh, nice!16:47
mgould:-)16:48
NobodyCammorning mgould , TheJulia16:49
mgouldhurrah, you fixed the autocorrect problem!16:49
*** Romanenko_K has quit IRC16:49
lucasagomesI'm calling it a day16:50
lucasagomeshave a great evening all16:50
TheJuliagoodnight lucasagomes16:50
mgouldlucasagomes: good night!16:50
lucasagomesTheJulia, oh morning16:50
*** lucasagomes is now known as lucas-dinner16:50
TheJulia:)16:50
NobodyCamnight lucas-dinner16:51
*** fragatina has joined #openstack-ironic16:51
*** rbudden has quit IRC16:52
*** fragatin_ has quit IRC16:54
*** rama_y has joined #openstack-ironic16:56
*** sdake_ has quit IRC16:57
*** anush_ has quit IRC17:00
*** vmud213 has joined #openstack-ironic17:02
* sambetts -> home 17:02
*** rcernin has joined #openstack-ironic17:02
sambettsHave a great night everyone!17:02
*** sambetts is now known as sambetts|afk17:03
*** wajdi has quit IRC17:03
*** rcernin has quit IRC17:05
*** vmud213 has quit IRC17:05
*** rcernin has joined #openstack-ironic17:05
NobodyCamnight sambetts|afk17:05
*** fragatina has quit IRC17:06
*** fragatina has joined #openstack-ironic17:07
*** vmud213 has joined #openstack-ironic17:07
jlvillalmgould: :)17:07
vdroknight lucas-dinner and sambetts|afk17:09
*** fragatina has quit IRC17:12
*** ksumit has joined #openstack-ironic17:12
*** rbrud has joined #openstack-ironic17:12
*** baoli has quit IRC17:13
*** e0ne has joined #openstack-ironic17:13
*** baoli has joined #openstack-ironic17:13
*** rbrud has quit IRC17:14
*** rbrud has joined #openstack-ironic17:14
*** rbrud has quit IRC17:15
*** rbrud has joined #openstack-ironic17:15
* mgould -> home; good night!17:17
*** mgould is now known as mgould|afk17:17
*** PollyZ has quit IRC17:18
*** abehl has joined #openstack-ironic17:20
vdrokgood night mgould|afk17:21
openstackgerritVasyl Saienko proposed openstack/ironic: Source openrc on subnode in multinode topology  https://review.openstack.org/35330217:25
openstackgerritVasyl Saienko proposed openstack/ironic: Skip db configuration on subnodes  https://review.openstack.org/35330317:25
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643217:25
*** fragatina has joined #openstack-ironic17:27
*** electrofelix has quit IRC17:28
*** Goneri has quit IRC17:28
jlvillalvsaienk0: I just learned about permalinks yesterday :)17:29
*** sdake has joined #openstack-ironic17:29
*** catintheroof has quit IRC17:30
*** ijw has quit IRC17:36
*** ijw has joined #openstack-ironic17:37
*** abehl has quit IRC17:41
*** ijw has quit IRC17:41
openstackgerritMerged openstack/ironic: Update proliantutils version required for Newton release  https://review.openstack.org/35330117:42
openstackgerritAkilan Pughazhendi proposed openstack/ironic: Doc: Recommend users to update their systems  https://review.openstack.org/35413617:42
openstackgerritMerged openstack/ironic: IPMITool: Check the boot mode when setting the boot device  https://review.openstack.org/35282017:50
*** ijw has joined #openstack-ironic17:51
*** rbrud has quit IRC17:52
*** ijw has quit IRC17:52
*** ijw has joined #openstack-ironic17:52
*** rbudden has joined #openstack-ironic17:54
*** ijw has quit IRC17:54
*** ijw has joined #openstack-ironic17:55
*** fragatina has quit IRC17:57
*** ohamada has quit IRC17:57
*** PollyZ has joined #openstack-ironic17:57
*** ametts has joined #openstack-ironic17:57
*** alineb has quit IRC18:00
openstackgerritMerged openstack/ironic: test_console_utils: using mock_open for builtin open()  https://review.openstack.org/35275618:02
openstackgerritVladyslav Drok proposed openstack/ironic: Remove unused [pxe]disk_devices option  https://review.openstack.org/35430818:13
vdrokrloo: btw, just noticed that ^^18:14
openstackgerritStephanie Miller proposed openstack/ironic: [WIP] Add storage_interface DB field and object  https://review.openstack.org/34800518:14
jrollvdrok: needs a sample config generation :)18:15
*** vmud213 has quit IRC18:15
*** amotoki has quit IRC18:15
vdrokjroll: ooo, right :D18:15
rloovdrok: thx for noticing :)18:15
*** ijw has quit IRC18:16
openstackgerritVladyslav Drok proposed openstack/ironic: Remove unused [pxe]disk_devices option  https://review.openstack.org/35430818:17
vdrokdone!18:17
vdrokrloo: this was used at some point, but I don't know what to write in the release note18:18
vdrokrloo: as we've had a couple of releases since it became unused18:18
rloovdrok: would be a good idea to spend a few minutes to find out what happened. in a meeting right now, can't look. did we mention in some release, that it was deprecated?18:19
vdrokor you mean to change the release note with the ramdisk deprecation?18:19
jrollrloo: it was part of dropping the bash ramdisk, see the commit message :)18:19
* jroll thinks it's fine without but welcomes disagreement18:19
vdroklemme find the old ramdisk deprecation note18:20
*** e0ne has quit IRC18:20
rloojroll, vdrok: so probably ok w/o relnote18:20
*** amotoki has joined #openstack-ironic18:22
*** bvandewa has joined #openstack-ironic18:26
vdrokrloo: I looked at the commit, we just forgot about that option. but as the old ramdisk was deprecated for a long time and this option was never used in ipa ramdisk, it might be ok without a separate note about for this option18:26
rloovdrok: should be fine w/o relno then. thx for looking.18:27
vdroknp :)18:27
*** ChubYann has joined #openstack-ironic18:33
*** catintheroof has joined #openstack-ironic18:34
*** fragatina has joined #openstack-ironic18:35
*** chlong|mtg has quit IRC18:37
vdrokgood night everyone!18:38
*** chlong|mtg has joined #openstack-ironic18:38
*** amoralej is now known as amoralej|off18:47
devanandatwo more patches for keystone policy support are ready to be landed18:49
devanandahttps://review.openstack.org/326768 and https://review.openstack.org/34555618:49
*** aarefiev_ has joined #openstack-ironic18:50
*** ijw has joined #openstack-ironic18:55
*** fragatin_ has joined #openstack-ironic19:00
*** fragatina has quit IRC19:03
*** athomas has quit IRC19:07
*** dprince_ has quit IRC19:08
*** sabeen has joined #openstack-ironic19:09
*** sabeen has quit IRC19:14
*** sabeen has joined #openstack-ironic19:16
*** aarefiev_ has quit IRC19:20
openstackgerritMichael Turek proposed openstack/ironic: Add classes for Portgroups API enablement  https://review.openstack.org/34754919:21
*** Sukhdev has joined #openstack-ironic19:22
*** bvandewa has quit IRC19:22
rloodevananda: just reviewed 326768. I could be wrong...19:27
*** catintheroof has quit IRC19:28
*** ijw has quit IRC19:41
openstackgerritMerged openstack/ironic: IPMINative: Check the boot mode when setting the boot device  https://review.openstack.org/35415319:43
*** PollyZ has quit IRC19:43
*** PollyZ has joined #openstack-ironic19:43
openstackgerritMerged openstack/ironic: Rename some variables in test_ipminative.py  https://review.openstack.org/35391419:46
rajinirsambetts,mat128: tftp is running , tested it19:48
rajinirsambetts,mat128: I can get the a file from /opt/stack/data/ironic/tftpboot folder from another machine.19:49
rajinirsambetts,mat128: so concluding tftp is running and setup on the slave vm. May be images are missing19:50
*** e0ne has joined #openstack-ironic19:55
rajinirsambetts: the pxelinux.o is missing from the  /opt/stack/data/ironic/tftpboot19:55
rajinirsambetts: could it be the issue?19:55
*** catintheroof has joined #openstack-ironic20:00
mat128sambetts is gone for the day and I'm about to leave20:03
mat128hmm20:03
mat128rajinir: ^20:03
mat128so are ytou able to dump network traffic on the VM tap?20:03
mat128is tftp responding with "file not found" or simply not responding anything20:04
rajinirmat128: I haven't done that, probably now20:04
*** e0ne has quit IRC20:08
*** ijw has joined #openstack-ironic20:09
*** ijw has quit IRC20:09
*** ijw has joined #openstack-ironic20:10
*** fragatin_ has quit IRC20:15
*** fragatina has joined #openstack-ironic20:16
*** amotoki has quit IRC20:20
rajinirmat128: Might have found the problem. The slave vm has two IPs, the tftp request is going to one of the IP and the tap is setup on the other. Not sure how to fix this20:20
*** ijw has quit IRC20:21
*** Goneri has joined #openstack-ironic20:22
*** ijw has joined #openstack-ironic20:22
*** e0ne has joined #openstack-ironic20:23
*** ijw has quit IRC20:23
*** ijw has joined #openstack-ironic20:23
mat128hmm20:25
mat128rajinir: I remember a devstack option for "which interface should we setup the tap on?"20:25
mat128vaguely20:25
mat128Look into the devstack plugin in the ironic tree20:26
rajinirmat128:ok20:27
rajinirmat128: thanks20:27
mat128I'll have to go, but let us know if you get something going20:27
mat128we'll all be here tomorrow :)20:27
rajinirmat128: I think I have specified the right interface for the tap, its the forwarding from the host machine. Sure, good night20:29
rajinirmat128: will update you tommorow, thanks20:29
*** hamzy has quit IRC20:30
*** aNupoisc has quit IRC20:30
*** e0ne has quit IRC20:32
*** [1]cdearborn has quit IRC20:36
*** ametts has quit IRC20:45
*** sdake has quit IRC20:48
*** PollyZ has quit IRC20:50
*** ametts has joined #openstack-ironic20:51
*** sabeen has quit IRC20:52
*** ametts has quit IRC20:56
*** bvandewa has joined #openstack-ironic20:59
jrollout for the night after a long day of meetings20:59
jrollI'm also out tomorrow20:59
jrollfeel free to ping me on hangouts if I'm needed though, I won't be far21:00
jrollhave a good weekend everyone :D21:00
TheJuliayou too jroll :)21:01
*** e0ne has joined #openstack-ironic21:02
*** e0ne has quit IRC21:05
*** causten has quit IRC21:08
*** causten has joined #openstack-ironic21:10
*** aNupoisc has joined #openstack-ironic21:11
mrdaMorning Ironic21:13
rloomorning mrda21:15
*** sdake has joined #openstack-ironic21:16
mrdao/21:17
*** PollyZ has joined #openstack-ironic21:19
*** baoli has quit IRC21:25
*** michauds has quit IRC21:25
*** thiagop has quit IRC21:28
*** Goneri has quit IRC21:28
*** causten has quit IRC21:32
*** sdake has quit IRC21:33
*** ijw has quit IRC21:34
*** causten has joined #openstack-ironic21:38
devanandarloo: thanks for the review! I'm reading .. and updating now21:40
*** ijw has joined #openstack-ironic21:40
rloodevananda: i'm around for another 20 min or so, ping me if you want me to re-review.21:40
*** ijw has joined #openstack-ironic21:40
*** Goneri has joined #openstack-ironic21:43
*** ksumit has quit IRC21:45
*** hjanssen-hpe has quit IRC21:45
*** hpe-hj has quit IRC21:45
openstackgerritRuby Loo proposed openstack/ironic: Update renos for fix to ipmi's set-boot-device  https://review.openstack.org/35441821:48
*** PollyZ has quit IRC21:49
*** mlupton has joined #openstack-ironic22:00
mluptonHey everyone, I'm having some networking trouble with an instance I've spawned on my overcloud.22:01
mluptonI created a security group that allows ssh and icmp, but I still can't ping the instance, even if I'm pinging it from the compute node that it sits on.22:01
*** causten has quit IRC22:02
*** PollyZ has joined #openstack-ironic22:03
*** rama_y has quit IRC22:10
*** rama_y has joined #openstack-ironic22:13
*** mlupton has quit IRC22:15
*** bvandewa has quit IRC22:20
*** Goneri has quit IRC22:21
*** bvandewa has joined #openstack-ironic22:26
*** bvandewa has quit IRC22:27
*** bvandewa has joined #openstack-ironic22:27
openstackgerritPolly Zhou proposed openstack/ironic: Use the new microversion-parse library to parse the version headers  https://review.openstack.org/35443222:33
*** bvandewa_ has joined #openstack-ironic22:34
*** bvandewa has quit IRC22:34
*** catintheroof has quit IRC22:35
openstackgerritDevananda van der Veen proposed openstack/ironic: Update documentation for keystone policy support  https://review.openstack.org/34555622:40
devanandarloo: took me an hour to update it ... E_DISTRACTIONS22:41
devanandaall good points. thanks again!22:41
*** rbudden has quit IRC22:50
*** PollyZ has quit IRC22:51
*** hoangcx2 has joined #openstack-ironic23:07
*** hoangcx has quit IRC23:08
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe]  https://review.openstack.org/30961223:09
*** sabeen has joined #openstack-ironic23:09
openstackgerritjxiaobin proposed openstack/ironic-python-agent: Deactivate LVM before write image to disk  https://review.openstack.org/35444023:12
*** ijw has quit IRC23:13
*** Sukhdev has quit IRC23:13
*** thrash is now known as thrash|g0ne23:15
openstackgerritjxiaobin proposed openstack/ironic-python-agent: Deactivate LVM before write image to disk  https://review.openstack.org/35444023:24
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [iscsi]  https://review.openstack.org/30961523:25
openstackgerritjxiaobin proposed openstack/ironic-python-agent: Deactivate LVM before write image to disk  https://review.openstack.org/35444023:26
openstackgerritjxiaobin proposed openstack/ironic-python-agent: Deactivate LVM before write image to disk  https://review.openstack.org/35444023:28
*** testing1234 has joined #openstack-ironic23:29
*** ijw has joined #openstack-ironic23:31
*** ijw has quit IRC23:35
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize Config Options - patch merge, cleanup  https://review.openstack.org/33016023:40
*** ijw has joined #openstack-ironic23:43
*** ijw has quit IRC23:45
*** ijw has joined #openstack-ironic23:45
*** rama_y has quit IRC23:47
*** jvgrant_ has joined #openstack-ironic23:57
*** rloo has quit IRC23:58
*** rloo has joined #openstack-ironic23:58

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