Thursday, 2015-11-12

*** baoli has quit IRC00:03
*** naohirot has joined #openstack-ironic00:04
*** jamielennox is now known as jamielennox|away00:08
*** sacharya has joined #openstack-ironic00:11
*** achanda has joined #openstack-ironic00:15
*** sacharya has quit IRC00:15
*** jamielennox|away is now known as jamielennox00:21
*** slappymcfry has joined #openstack-ironic00:22
openstackgerritRuby Loo proposed openstack/ironic-lib: Remove tempdir config, add tempdir parameter  https://review.openstack.org/24337800:30
*** achanda has quit IRC00:35
*** slappymcfry has quit IRC00:36
*** sinval_ has joined #openstack-ironic00:40
*** amotoki has joined #openstack-ironic00:41
zer0c00lAre there any plans to support 'partition' image  in agent mode?00:42
zer0c00lFrom what i have seen partition image was only supported part of the iscsi deploy00:43
lintanmorning Ironic00:51
mrdahi lintan00:58
openstackgerritJulia Kreger proposed openstack/bifrost: Additional inspection role and adding role to NV job script  https://review.openstack.org/23274600:58
lintanmorning mrda01:01
rloozer0c00l: I think yes: http://specs.openstack.org/openstack/ironic-specs/priorities/mitaka-priorities.html#ironic-lib-refactor01:04
*** garthb__ has quit IRC01:10
zer0c00li see01:18
*** zhenguo has joined #openstack-ironic01:18
*** Guest36170 has quit IRC01:21
*** mbound has quit IRC01:26
*** thrash is now known as thrash|g0ne01:26
*** Sukhdev has quit IRC01:41
*** mtanino has quit IRC01:45
*** sacharya has joined #openstack-ironic01:54
*** sacharya has quit IRC01:59
zer0c00lping lucas-dinner02:02
openstackgerritYuiko Takada proposed openstack/ironic-specs: Add allowed transitions to API  https://review.openstack.org/22402202:12
*** hemna has quit IRC02:13
*** ijw has quit IRC02:14
zer0c00llucas-dinner: rloo mentioned that you were proposing to use blivet on  IPA. Is that plan still on the table?02:14
*** MattMan has quit IRC02:15
*** baoli has joined #openstack-ironic02:15
*** MattMan has joined #openstack-ironic02:15
*** baoli has quit IRC02:17
*** baoli has joined #openstack-ironic02:17
*** baoli has quit IRC02:18
*** baoli has joined #openstack-ironic02:19
*** mbound has joined #openstack-ironic02:26
*** spandhe has quit IRC02:29
*** mbound has quit IRC02:31
*** sinval_ has quit IRC02:31
*** lnxnut has joined #openstack-ironic02:37
zer0c00ls/proposing/thinking02:37
openstackgerritMerged openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/24422802:38
*** rebase has quit IRC02:41
*** achanda has joined #openstack-ironic02:54
*** dims has quit IRC02:57
*** dims has joined #openstack-ironic03:07
*** dims has quit IRC03:14
*** dims has joined #openstack-ironic03:16
*** spandhe has joined #openstack-ironic03:18
*** dims has quit IRC03:25
*** spandhe has quit IRC03:37
*** caiobo has quit IRC03:52
*** rloo has quit IRC03:54
openstackgerritMerged openstack/ironic: Node's last_error to show the actual error from sync_power_state  https://review.openstack.org/23804604:03
*** Marga_ has quit IRC04:14
*** spandhe has joined #openstack-ironic04:15
*** kalpase has joined #openstack-ironic04:19
*** mbound has joined #openstack-ironic04:23
*** spandhe has quit IRC04:23
*** mbound has quit IRC04:27
*** rebase has joined #openstack-ironic04:50
*** achanda has quit IRC05:04
*** links has joined #openstack-ironic05:07
*** sacharya has joined #openstack-ironic05:19
*** baoli has quit IRC05:20
*** arturas has joined #openstack-ironic05:22
*** arturas has quit IRC05:26
*** arturas has joined #openstack-ironic05:29
*** arturas has quit IRC05:32
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/24445305:53
*** arturas has joined #openstack-ironic06:04
*** achanda has joined #openstack-ironic06:05
*** Marga_ has joined #openstack-ironic06:06
*** achanda has quit IRC06:10
*** jerrygb has quit IRC06:14
*** achanda has joined #openstack-ironic06:15
*** sc68cal has quit IRC06:45
*** links has quit IRC06:45
*** Nisha has joined #openstack-ironic06:49
*** sacharya has quit IRC06:54
*** baoli has joined #openstack-ironic06:55
*** baoli has quit IRC07:01
*** links has joined #openstack-ironic07:05
*** sirushti has quit IRC07:14
*** gridinv has quit IRC07:16
*** marios has quit IRC07:16
*** gridinv has joined #openstack-ironic07:17
*** sirushti has joined #openstack-ironic07:17
*** marios has joined #openstack-ironic07:17
*** achanda has quit IRC07:20
pas-hamorning all07:21
openstackgerritZhenguo Niu proposed openstack/ironic: Check shellinabox process during stopping console  https://review.openstack.org/24231907:23
lintanmoring psa-ha07:37
lintanmoring pas-ha07:37
*** Nisha has quit IRC07:39
openstackgerritTan Lin proposed openstack/python-ironicclient: Sync with oslo-incubator  https://review.openstack.org/24448207:46
*** ukalifon1 has joined #openstack-ironic07:51
*** sacharya has joined #openstack-ironic07:55
*** sacharya has quit IRC07:59
openstackgerritTan Lin proposed openstack/python-ironicclient: Use keystoneclient.exception directly  https://review.openstack.org/24449108:02
*** ifarkas has joined #openstack-ironic08:14
*** dtantsur|afk is now known as dtantsur08:17
dtantsurMorning Ironic08:17
*** jcoufal has joined #openstack-ironic08:21
*** rebase has quit IRC08:28
*** rebase has joined #openstack-ironic08:28
*** rebase has quit IRC08:29
*** sc68cal has joined #openstack-ironic08:31
*** praneshp has quit IRC08:34
*** praneshp_ has joined #openstack-ironic08:35
vdrokmorning dtantsur lintan pas-ha08:39
dtantsuro/08:39
lintanmorning vdrok and dtantsur :)08:39
*** amotoki has quit IRC08:40
*** Pablo|off| has joined #openstack-ironic08:40
openstackgerritTan Lin proposed openstack/python-ironicclient: Add a new cmd method node-get-vendor-passthru-methods  https://review.openstack.org/23744408:42
openstackgerritTan Lin proposed openstack/python-ironicclient: Sync with oslo-incubator  https://review.openstack.org/24448208:42
openstackgerritTan Lin proposed openstack/python-ironicclient: Make print_list accept a list of dict  https://review.openstack.org/24449508:42
*** achanda has joined #openstack-ironic08:47
*** mayflyman has joined #openstack-ironic08:48
*** Pablo|off| is now known as pcaruana08:48
*** mayflyman has quit IRC08:48
*** romainh has joined #openstack-ironic08:49
*** rcernin has joined #openstack-ironic08:50
*** Nisha has joined #openstack-ironic08:51
*** mbound has joined #openstack-ironic08:51
aarefievhi folks08:59
*** lucas-dinner is now known as lucasagomes09:00
lucasagomeszer0c00l, hi there! Yes I think blivet would be great09:00
*** mbound has quit IRC09:01
aarefievdtantsur: morning, have a min? could you help with inspector, please09:02
dtantsuraarefiev, morning! yeah, sure09:03
*** jistr has joined #openstack-ironic09:03
aarefievdtantsur: I'm trying to deploy it with devstack: http://paste.openstack.org/show/478635/09:03
aarefievdtantsur: but my dnsmasq goes down09:05
dtantsuraarefiev, ubuntu? I remember sambetts complaining about conflict between ubuntu default dnsmasq and inspector's one09:06
aarefievdtantsur: yep09:06
*** Nisha has quit IRC09:07
aarefievdo you have solution?09:07
dtantsuraarefiev, the idea is to disable ubuntu's behavior of using proxy DNS in dnsmasq. Not sure I remember how, you can try to google or just wait for sambetts to appear09:08
aarefievdtantsur: ok, thanks, I'll try09:08
*** yuikotakada has joined #openstack-ironic09:16
openstackgerritZhenguo Niu proposed openstack/ironic: Add console feature to ssh driver  https://review.openstack.org/24145309:18
*** electrofelix has joined #openstack-ironic09:23
*** walt has joined #openstack-ironic09:23
*** walt is now known as Guest6319109:24
yuikotakadagood morning, ironic09:27
lucasagomesyuikotakada, , dtantsur aarefiev morning09:32
aarefievlucasagomes: morning09:32
dtantsurmorning lucasagomes, yuikotakada09:33
dtantsur(UGT morning, of course :)09:33
yuikotakadalucasagomes, aarefiev, dtantsur, good ugt morning :)09:33
lucasagomes:-)09:33
aarefievyuikotakada, morning :)09:33
* lucasagomes still jet lagged anyway09:33
*** achanda has quit IRC09:34
yuikotakadalucasagomes, how was your trip to kyoto and osaka?09:34
lucasagomesyuikotakada, was fantastic! I ended up not going to osaka. Instead I went to Himeji and Nara09:34
lucasagomesbut Kyoto is amazing, loved it09:34
yuikotakadadtantsur, about tempest-lib(or tempest-external-plugin? I'm not sure, anyway), do I need to write a spec?09:35
yuikotakadalucasagomes, that's fine, what kind of things did you do? seeing temples or something like that?09:36
lucasagomesyuikotakada, yeah as well, loads of temples, shrines and bars :-P09:37
lucasagomesyuikotakada, I had a local to kinda show me around in Kyoto so I ended up going to some cool places09:37
lucasagomeslike a 200 yen beer bar09:38
lucasagomesand tried some diff stuff, like blue beer and pumpkin beer09:38
*** mbound has joined #openstack-ironic09:38
yuikotakadalucasagomes, haha, that's fun, did you eat something fantastic?09:39
lucasagomesyuikotakada, ah, I loved the food in general, it's almost impossible to get bad food over there09:39
dtantsuryuikotakada, I don't think a spec is required for such work09:39
lucasagomesI don't know how, all places I've ate were great09:39
lucasagomesyuikotakada, and I went to a sake place in Nara as well where you can try 6 diff types09:40
lucasagomesthey give you the cup at the end, pretty cool. I have it at home now09:40
yuikotakadalucasagomes, I'm relieved that you were not discouraged by japan :)09:41
lucasagomesyuikotakada, oh not at all!09:41
lucasagomesjapan +109:42
yuikotakadadtantsur, ok, I investigated some other projects' progress and have find that maybe we can try to do tempest test in our tree.09:42
dtantsurawesome!09:42
yuikotakadalucasagomes, yeah! :D09:42
*** mkovacik has joined #openstack-ironic09:44
yuikotakadadtantsur, then I will try to use it in my environment in the next week. BTW, I want to make alisting of TODO to somewhere and share it. where is suitable? irc meeting?09:44
dtantsuryuikotakada, what exactly do you mean here?09:45
*** openstackgerrit has quit IRC09:46
*** openstackgerrit has joined #openstack-ironic09:47
*** Guest63191 has quit IRC09:48
yuikotakadadtantsur, If I can, I'd like to share findings with other guys before posting some patches suddenly, don't I need to do it?09:49
openstackgerritDmitry Tantsur proposed openstack/ironic: Only mention IPA in the quick start and user guides for DevStack  https://review.openstack.org/23590709:49
dtantsurliliars, ^^09:49
dtantsuryuikotakada, ah, gotcha. I think the openstack-dev ML is a perfect place for it.09:49
yuikotakadadtantsur, ok, thanks :)09:50
dtantsuryuikotakada, btw are you following https://review.openstack.org/#/c/169126/ re microversion testing?09:50
yuikotakadadtantsur, yes09:52
dtantsurcool, thanks09:52
*** russell_ has joined #openstack-ironic09:54
*** russell_ has quit IRC09:55
*** mgould has joined #openstack-ironic09:57
*** naohirot has quit IRC09:57
*** mbound has quit IRC10:05
*** dtantsur is now known as dtantsur|brb10:05
*** mbound has joined #openstack-ironic10:07
*** walts has joined #openstack-ironic10:08
*** derekh has joined #openstack-ironic10:09
*** walts has left #openstack-ironic10:09
*** openstackgerrit has quit IRC10:16
*** openstackgerrit has joined #openstack-ironic10:17
openstackgerritLucas Alvares Gomes proposed openstack/ironic: iPXE: use set arithmetic to increment the value of a variable  https://review.openstack.org/24453010:18
*** sacharya has joined #openstack-ironic10:21
*** sturivnyi has joined #openstack-ironic10:21
sturivnyiHaomeng, jlvillal, sambetts : Hi! Could you please review patch `Documentation how to run tests` https://review.openstack.org/#/c/240866/. Thanks =)10:25
*** sacharya has quit IRC10:25
*** walts has joined #openstack-ironic10:26
*** yuikotakada has quit IRC10:29
*** arturas has quit IRC10:32
*** walts has quit IRC10:39
openstackgerritMerged openstack/ironic: Added unit tests for CORS middleware  https://review.openstack.org/24004810:39
sambettsHey dtantsur o/10:39
sambettsaarefiev: I actually just ended up manually modifing my /etc/resolv.conf file, changing it from 127.0.0.1 to my actual DNS server address10:41
sambettsdtantsur: aarefiev: this is the other reason I really want us to be able to use neutrons dhcp server10:42
openstackgerritMerged openstack/ironic: Add whole disk image creation command to Installation Guide  https://review.openstack.org/24381910:43
openstackgerritSyed Ismail Faizan Barmawer proposed openstack/ironic: Refactor disk partitioner code from ironic and use ironic-lib.  https://review.openstack.org/18444310:45
*** arturas has joined #openstack-ironic10:47
*** jistr_ has joined #openstack-ironic10:48
*** jistr has quit IRC10:49
*** rcernin has quit IRC10:49
*** rcernin has joined #openstack-ironic10:50
sambettsaarefiev: it works fine until you restart the machine and then restart devstack, and you have to remember to do it again, however I don't restart that often so its not a big problem10:50
*** Walts has joined #openstack-ironic10:51
*** faizan has joined #openstack-ironic10:52
*** jistr_ has quit IRC10:54
*** dtantsur|brb is now known as dtantsur10:57
dtantsursambetts, morning10:57
*** baoli has joined #openstack-ironic10:58
*** baoli has quit IRC11:03
sambettsdtantsur: do you think my OSC suggestions made sense?11:04
aarefievsambetts: hi, thanks, I'm trying now to deploy with suggestion, using neutrons dhcp server would be nice, are you working on this?11:04
aarefiev*with your suggestion:)11:04
sambettsaarefiev: We have a request for enhancement submitted to neutron to tackle this and there is some discussion going on around it :)11:05
*** jistr_ has joined #openstack-ironic11:05
dtantsursambetts, I don't really like adding "node" everywhere, otherwise looks good. anyway I guess thrash|g0ne is planning on a spec for it11:06
mgouldlucasagomes, so there are ROMs that support string concatenation but not increment? Bizarre11:06
aarefievsambetts: could you share link, please ?11:06
lucasagomesmgould, yeah... these are really old ROMs tho11:07
lucasagomesseems CentOS doesn't update the iPXE ROMs since 201311:07
lucasagomes:-/11:07
sambettsaarefiev: https://bugs.launchpad.net/neutron/+bug/1512666 leaving your experience as a data point on the RFE might help11:07
openstackLaunchpad bug 1512666 in neutron " Allow for per-subnet/network dhcp options" [Wishlist,Triaged] - Assigned to Sam Betts (sambetts)11:07
*** zhenguo has quit IRC11:07
sambettsdtantsur: how would you allow it to handle nodes and chassis etc?11:08
mgouldlucasagomes, I'm going to my local functional programming meetup tonight - I'll tell them you found a real-world use for Peano arithmetic :-)11:08
lucasagomesLOL11:08
lucasagomesyeah show'em the patch11:08
aarefievsambetts: very appreciate11:08
lucasagomesI want dprince to test it see if it works tho11:08
lucasagomesI tested locally, seems fine11:09
dtantsursambetts, "baremetal show" vs "baremetal port show"11:09
*** e0ne has joined #openstack-ironic11:09
sambettsdtantsur: oh ok, so no object defaults to a node11:09
dtantsurthat's what we already landed, btw11:09
sambettsright :) I wondered if you had any different thoughts11:10
sambettsI think I actually prefer including node, it just standardises the client and makes it explict instead of implict11:11
sambettslucasagomes: WDYT?11:11
* lucasagomes reads11:12
*** jamielennox is now known as jamielennox|away11:12
lucasagomessambetts, I don't have a very strong opnion on CLI syntax's, but I sympathize with the explicit is better than implicit11:13
*** dims_ has joined #openstack-ironic11:14
lucasagomesso I think having the resource (node, port, chassis) is better11:14
lucasagomesit's easier for someone that doesn't know the CLI to figure out what it's doing11:14
*** baoli has joined #openstack-ironic11:14
sambettsI agree11:14
*** jerrygb has joined #openstack-ironic11:15
*** romainh has left #openstack-ironic11:16
sambettswhat I thought was interesting on the ML disucssion about this was people that assumed that baremetal node inspect did the same thing as baremetal node show11:16
sambettsand that manage was the same as set11:16
dtantsurwell, that's prpobably the reason to go with 'baremetal provision --transition=manage' or something11:17
sambettsif you were going that route I would rather baremetal node transition manage11:19
dtantsurthis reads as you're managing a transition :)11:19
*** baoli has quit IRC11:19
*** mgoddard_ has quit IRC11:19
dtantsurthere's no such object as transition, so it's not really OSC-ish11:19
*** mgoddard_ has joined #openstack-ironic11:19
*** jerrygb has quit IRC11:19
sambettsin my mind provision doesn't fit there either though, tbh I think people would soon relise that baremetal node manage is something  different than show because we also have show and its the convention, so people would be use to typing show on a resource and unless that failed for some reason I don't know why they would go looking for another command11:23
sinvalgood morning folks11:26
mgouldsinval, morning!11:28
sinvalhey mgould11:29
*** trown|outttypeww is now known as trown11:33
*** lnxnut has quit IRC11:39
*** baoli has joined #openstack-ironic11:41
lucasagomessinval, morning11:43
sambettsmorning sinval mgould o/11:43
sinvalhey lucasagomes, sambetts11:44
mgouldsambetts, morning :-)11:44
*** baoli has quit IRC11:46
*** ndipanov has quit IRC11:50
*** Nisha has joined #openstack-ironic11:52
*** arturas has quit IRC12:00
*** arturas has joined #openstack-ironic12:01
WaltsHi all, I have been trying out ironic in a lab set up and I have a few questions if any one has the time12:07
lucasagomesWalts, hi there, just ask. Someone may answer it eventually12:08
*** marzif has joined #openstack-ironic12:08
Waltspresuming all the correct setup has been done and a node is visible as available, does this mean that I can then use the nova service to boot an image to it12:10
*** romainh has joined #openstack-ironic12:10
dtantsurWalts, should be fine12:11
Waltsand what does setting a node to active achieve "ironic node-set-provision-state $NODE_UUID active"12:11
dtantsurofc make sure that flavor matches what you have in node properties12:11
dtantsurWalts, thats what nova uses behind the scene when deploying an instance12:11
Waltsoh I see12:12
dtantsurif you use nova, you don't need it12:12
*** faizan has quit IRC12:12
Waltsso just to be clear the nodes just need to be in available state (with correct flavor to match hardware properties) then nova can be used to provision them12:13
lucasagomesWalts, correct12:14
lucasagomesWalts, you can also check nova hypervisor-stat to see if the node is already available for the nova scheduler12:14
dtantsurWalts, also make sure maintenance mode is off12:14
dtantsur++ fir hypervisor-stats12:14
Walts dtantsur: I was going to ask about the hypervisor stats12:15
*** kalpase has quit IRC12:15
*** kalpase has joined #openstack-ironic12:15
Waltswhat i run nova hypervisor-stats it displays the correct count but all other values are 012:16
Waltsis this normal12:16
dtantsurWalts, no. you probably have to wait up to 2 mins for nova to catch up12:16
lucasagomesWalts, those values comes from the node's properties, do you have cpus, memory_mb, local_gb and arch set correctly there?12:18
lucasagomesironic node-show (see properties there)12:18
openstackgerritMerged openstack/ironic-lib: Remove tempdir config, add tempdir parameter  https://review.openstack.org/24337812:18
Waltslucasagomes: I set the properties manually first using "ironic node-update $NODE_UUID add properties/etc"12:20
*** jistr_ is now known as jistr12:20
Waltsbut i think they where overwritten when i used inspect12:20
lucasagomesWalts, that's fine12:20
lucasagomesright, and the values are correct there?12:20
Waltsyes they are correct now, more correct then when I set manually12:20
Walts:)12:21
lucasagomesheh12:21
lucasagomesyeah so it should appear on nova hypervisor-stat12:21
WaltsI moved the ironic nodes to active almost immediately on first set-up would that effect the hypervisor-stats, its been about an hour since I set them up12:22
lucasagomesWalts, oh alright so there's something wrong with it12:23
lucasagomesWalts, what's the output of ironic node-validate <uuid>12:23
lucasagomes ?12:23
Waltslucasagomes: out put of validate is boot true, console true, deploy true, inspect true, management true, power true, raid none - not supported12:25
Walts*output12:25
lucasagomeshmm12:25
* lucasagomes thinks12:25
lucasagomesWalts, the node is deployed then? Cause the deploy interface usually validate to false12:26
lucasagomesuntil it has an image set and so on12:26
Waltssorry yes I was mistakenly using  the active command and not leaving the nodes in the available state12:28
WaltsI thought active was needed by nova to be able to boot12:28
lucasagomesright, yeah, nova will see the node as available to put an instance on it only if it's in "available" state in Ironic12:29
Waltsso in the active state the hypervisor stats will be all 0 and in the available they will be populated?12:30
WaltsI will delete the node and retry12:30
Waltsjust in case its important I do see this in the compute log  Hypervisor: VCPU information unavailable _report_hypervisor_resource_view /usr/lib/python2.7/site-packages/nova/compute/resource_tracker.py12:31
lucasagomesWalts, yes12:33
lucasagomesI will brb 1 sec12:33
*** baoli has joined #openstack-ironic12:39
*** Nisha has quit IRC12:41
*** thrash|g0ne is now known as thrash12:43
Waltslucasagomes, dtantsur: thanks for you help hypervisor stats is now populated12:43
aarefievdtantsur: could you explain your comment on https://review.openstack.org/#/c/219298/ , please12:43
*** baoli has quit IRC12:44
aarefievdtantsur: don't get it what you mean12:44
*** sacharya has joined #openstack-ironic12:45
openstackgerritJohn L. Villalovos proposed openstack/python-ironicclient: Add documentation on how to run tests  https://review.openstack.org/24086612:47
dtantsuraarefiev, see discussion here: https://review.openstack.org/#/c/244021/12:47
dtantsurwe either have to mute INFO messages from oslo.service, or convince them to change how they log12:48
*** sacharya has quit IRC12:49
*** Walts has quit IRC12:52
*** baoli has joined #openstack-ironic12:53
aarefievdtantsur: ok, now I get it, I think we can set default log level to warning for oslo.service12:57
*** kalpase has quit IRC12:58
dtantsurok, this solves it12:58
*** arif-ali has quit IRC13:01
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Allow several UUID's in 'introspection start'  https://review.openstack.org/24354113:02
dtantsursambetts, ^^13:02
ifarkasdtantsur, hi, could you please review https://review.openstack.org/#/c/230980/?13:07
dtantsursure13:07
ifarkasthanks!13:08
openstackgerritDevananda van der Veen proposed openstack/ironic-specs: Add node reservations spec  https://review.openstack.org/20464113:10
*** pas-ha has quit IRC13:11
*** dprince has joined #openstack-ironic13:15
*** jerrygb has joined #openstack-ironic13:15
*** jerrygb_ has joined #openstack-ironic13:18
*** liliars has quit IRC13:21
*** jerrygb has quit IRC13:22
*** arif-ali has joined #openstack-ironic13:24
*** dims_ has quit IRC13:24
*** dims has joined #openstack-ironic13:26
*** pas-ha has joined #openstack-ironic13:27
*** jerrygb_ has quit IRC13:39
*** boris-42 has quit IRC13:48
*** e0ne has quit IRC13:51
*** e0ne has joined #openstack-ironic13:52
*** BobBall has quit IRC13:54
*** pas-ha has quit IRC13:59
*** e0ne has quit IRC14:04
*** rloo has joined #openstack-ironic14:04
afaranhaHaomeng, lintan Hey, I read the feedback about tests in change https://review.openstack.org/#/c/242071/ , do you have any suggestion how can I mock a Keystone with SSL and accepting a specific ca_file?14:05
afaranhasorry lintan  , this was meant to kan :P14:07
*** pas-ha has joined #openstack-ironic14:12
*** e0ne has joined #openstack-ironic14:13
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/24422714:20
*** marzif has quit IRC14:20
*** jerrygb has joined #openstack-ironic14:22
*** itsuugo has joined #openstack-ironic14:30
*** arturas has quit IRC14:37
*** itsuugo has quit IRC14:43
*** arturas has joined #openstack-ironic14:48
*** dims has quit IRC15:01
openstackgerritMerged openstack/ironic: Check shellinabox process during stopping console  https://review.openstack.org/24231915:01
openstackgerritZhenguo Niu proposed openstack/ironic: Follow up patch for Ib8968418a1835a4131f2f22fb3e4df5ecb9b0dc5  https://review.openstack.org/24469915:03
*** dims has joined #openstack-ironic15:07
*** arturas has quit IRC15:13
*** mgould has quit IRC15:16
*** lnxnut has joined #openstack-ironic15:16
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Node  https://review.openstack.org/23950615:16
*** Claudio2 has joined #openstack-ironic15:19
*** marzif has joined #openstack-ironic15:21
*** Claudio2 has quit IRC15:21
openstackgerritAnton Arefiev proposed openstack/ironic: Fix oslo namespace in default log level  https://review.openstack.org/24470715:25
NobodyCamgood mornign Ironic folks15:27
sambettsMorning NobodyCam o/15:27
dtantsurmorning NobodyCam15:27
*** garthb has joined #openstack-ironic15:28
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add documentation on how to run tests  https://review.openstack.org/24086615:30
*** mgould has joined #openstack-ironic15:31
*** mtanino has joined #openstack-ironic15:31
mkovacikdtantsur, hi, w/r https://blueprints.launchpad.net/ironic-inspector/+spec/abort-introspection15:31
mkovacikdtantsur, I may have identified couple of "critical sections" where the abort should be avoided: Abort call re-entrance, Abort proceeding Introspect call, Abort proceeding Process call, Abort proceeding Periodic Cache Clean Up15:32
mkovacikhttp://www.fpaste.org/289668/47342049/15:32
mkovacikdtantsur, would you have some implementation hints?15:32
openstackgerritZhenguo Niu proposed openstack/ironic: Validate all tcp/udp port numbers  https://review.openstack.org/24471315:33
dtantsurmkovacik, that's why we need https://blueprints.launchpad.net/ironic-inspector/+spec/node-states ... I'll try to finish this one asap, otherwise I don't have any hints, we can land a naive implementation, then add locking as part of https://blueprints.launchpad.net/ironic-inspector/+spec/node-states15:34
* mkovacik looks15:34
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Make it explicit that finished is set to true on failures too  https://review.openstack.org/24471415:35
dtantsur(having own kind of state machine is the worst consequence of being a separate server)15:35
mkovacikdtantsur, ack.15:37
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add documentation on how to run tests  https://review.openstack.org/24086615:38
mkovacikdtantsur, w/r this naive solution, I can imagine some basic checks before NodeInfo.finished() and power-off are called: node exist, introspection start time present, finish time absent, no error but it's a race anyway. Do you have some other check tip? Or should we really postpone this till the Node states are implemented?15:40
openstackgerritMiles Gould proposed openstack/ironic: Get mandatory patch attrs from WSME properties  https://review.openstack.org/24020215:41
dtantsurmkovacik, yeah, maybe it's worth postponing15:41
mkovacikdtantsur, OK15:42
*** baoli has quit IRC15:44
openstackgerritMerged openstack/python-ironic-inspector-client: Allow several UUID's in 'introspection start'  https://review.openstack.org/24354115:44
*** baoli has joined #openstack-ironic15:46
*** baoli has quit IRC15:48
*** achanda has joined #openstack-ironic15:51
*** baoli has joined #openstack-ironic15:52
*** baoli has quit IRC15:58
*** baoli has joined #openstack-ironic16:00
*** achanda has quit IRC16:02
*** achanda has joined #openstack-ironic16:03
*** achanda has quit IRC16:04
*** baoli has quit IRC16:10
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: [WIP] Track introspection state for nodes  https://review.openstack.org/24473316:15
*** davideagnello has quit IRC16:17
*** davideagnello has joined #openstack-ironic16:17
*** links has quit IRC16:18
*** kbyrne has joined #openstack-ironic16:21
*** arturas has joined #openstack-ironic16:27
*** mgould has quit IRC16:34
*** hemnafk is now known as hemna16:36
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: [WIP] Lock nodes to avoid simultaneous introspection requests  https://review.openstack.org/24475016:38
dtantsurmkovacik, sambetts, that's ^^ how I see locking for now16:38
*** Walts has joined #openstack-ironic16:39
*** sacharya has joined #openstack-ironic16:39
gabrielHi folks, I'd love to have some eyes on: https://review.openstack.org/237122 . Thanks in advance.16:39
*** yonglihe has joined #openstack-ironic16:40
* dtantsur goes home16:40
*** dtantsur is now known as dtantsur|afk16:41
openstackgerritVladyslav Drok proposed openstack/ironic: Make task parameter mandatory in get_supported_boot_devices  https://review.openstack.org/24475816:41
*** romainh has left #openstack-ironic16:42
*** r-daneel has joined #openstack-ironic16:42
*** mgoddard__ has joined #openstack-ironic16:45
*** mgould has joined #openstack-ironic16:46
*** Konstantinos1 has joined #openstack-ironic16:47
*** mgoddard_ has quit IRC16:48
*** rcernin has quit IRC16:50
*** Konstantinos1 has left #openstack-ironic16:50
*** Konstantinos1 has joined #openstack-ironic16:50
*** Konstantinos1 has left #openstack-ironic16:50
*** caiobo has joined #openstack-ironic16:53
*** mgoddard has joined #openstack-ironic16:58
*** mgoddard__ has quit IRC16:58
*** marzif has quit IRC16:59
*** Nisha has joined #openstack-ironic16:59
*** marzif has joined #openstack-ironic17:00
*** e0ne has quit IRC17:02
yonglihejroll, are you there?17:03
*** kozhukalov_ has quit IRC17:04
*** garthb has quit IRC17:05
*** marzif has quit IRC17:05
*** marzif has joined #openstack-ironic17:06
*** pcaruana has quit IRC17:06
*** pas-ha has quit IRC17:06
*** baoli has joined #openstack-ironic17:07
*** baoli_ has joined #openstack-ironic17:09
*** baoli has quit IRC17:12
*** ifarkas has quit IRC17:14
*** sinval has quit IRC17:17
*** sinval has joined #openstack-ironic17:18
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Chassis  https://review.openstack.org/24056217:18
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Chassis  https://review.openstack.org/24056217:20
*** marzif has quit IRC17:20
*** sinval has left #openstack-ironic17:22
*** sinval has joined #openstack-ironic17:22
*** trown is now known as trown|lunch17:22
*** rcernin has joined #openstack-ironic17:24
* sambetts calls it a day 17:24
*** itsuugo has joined #openstack-ironic17:28
*** aojea_ has joined #openstack-ironic17:28
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Node  https://review.openstack.org/23950617:30
*** itsuugo has quit IRC17:31
*** aojea_ has quit IRC17:31
*** mbound has quit IRC17:38
*** spandhe has joined #openstack-ironic17:40
NobodyCamnight sambetts17:45
*** david-lyle has quit IRC17:46
*** praneshp_ has quit IRC17:48
*** e0ne has joined #openstack-ironic17:50
*** derekh has quit IRC17:54
*** ukalifon1 has quit IRC18:04
*** garthb has joined #openstack-ironic18:06
lucasagomesfolks I'm calling it a day as well18:10
lucasagomeshave a good night everyone18:10
*** lucasagomes is now known as lucas-afk18:10
*** arturas has quit IRC18:16
*** Nisha has quit IRC18:16
*** Nisha has joined #openstack-ironic18:16
*** arturas has joined #openstack-ironic18:17
mgouldme too, good night everyone!18:18
*** mgould has quit IRC18:18
devanandag'morning :)18:18
NobodyCamnight mgould18:19
NobodyCammorning devananda18:19
*** bnemec has quit IRC18:21
*** jistr has quit IRC18:22
*** Nisha has quit IRC18:24
*** Nisha has joined #openstack-ironic18:24
*** praneshp has joined #openstack-ironic18:24
*** achanda has joined #openstack-ironic18:26
Nishadevananda, g'morning18:27
Nishadevananda, wanted to know how to proceed on https://bugs.launchpad.net/ironic/+bug/151042518:27
openstackLaunchpad bug 1510425 in Ironic "the take_over for iscsi_ilo drivers doesnt recreate the boot iso" [Undecided,In progress] - Assigned to Nisha Agarwal (agarwalnisha1980)18:27
*** trown|lunch is now known as trown18:28
Nishadevananda, the fix for this bug is merged with https://review.openstack.org/#/c/216538/18:28
Nishain master branch18:28
Nishadevananda, the fix is critical and needed for Liberty branch18:28
NishaNobodyCam, ^^^^18:29
*** jpeeler has quit IRC18:30
devanandaNisha: when you say "critical" what do you mean?18:32
NishaNobodyCam, the fix which landed in master branch is a feature patch(boot and deploy seperation) for iscsi_ilo and has the fix for this issue already18:33
devanandaNisha: this is the definition of critical: https://wiki.openstack.org/wiki/BugTriage#Task_2:_Prioritize_confirmed_bugs_.28bug_supervisors.2918:33
devanandaNisha: it's highly unlikely we'll be able to accept a ~2k line backport. Is the fix that *entire* patch, or is it a subset of that patch?18:34
Nishadevananda, when a node is taken over by another conductor and is powered off it will not boot successfully when powered on due to failure to recreate the boot iso18:35
NishaThe required fix is https://review.openstack.org/23962718:35
NobodyCamthats just 73 lines18:36
Nishayes but the same code(functionality fix which is just 5-6 lines) already exists in the feature patch18:36
Nishaand thats merged in master18:37
*** itsuugo has joined #openstack-ironic18:37
devanandaNisha: oh. that's tricky.18:37
Nishadevananda, yes, but this is a required fix in liberty18:38
devanandaNisha: I would say, get everything fixed in master, if it isn't already, and then isolate the change needed to fix it in liberty from any other parts of the change18:38
devanandaNisha: it really should be "backport this <patch> to liberty" but it sounds like that's no longer possible because the fix is mixed into another much larger patch18:38
Nishadevananda, in master i guess only the test case is missing18:39
*** electrofelix has quit IRC18:39
devanandaNisha: also FWIW, I would categorize this as a "High" priority bug, not critical18:39
devanandaNisha: that's moot as we'll still do backports for liberty either way :)18:40
Nishadevananda, the only way i could think is to first back out the fix from master and then propose the fix for this bug18:40
Nishain master18:40
Nishaand then backport this fix to liberty18:40
devanandayea, that's one way18:41
devanandahow small is the change itself, if that were to be done?18:42
Nisha#link https://review.openstack.org/23962718:42
devanandaoooh18:42
devanandaI see18:42
devanandathat's why that patch is in merge conflict?18:42
*** EmilienM has quit IRC18:42
Nishahttps://review.openstack.org/#/c/239627/9/ironic/drivers/modules/ilo/deploy.py18:42
Nishayes18:42
devanandagot it18:44
*** EmilienM has joined #openstack-ironic18:45
Nishadevananda, so how should i go ahead on this18:46
Nishashould i propose a fix to first back out the fix and then propose the current patch18:46
Nisha?18:46
Nishadevananda, or any other way?18:46
*** e0ne has quit IRC18:47
NobodyCamthen propose the split patch again ninus the fix18:47
devanandaNisha: no, don't propose the revert18:48
devanandaNisha: I think you can propose the fix directly to stable/liberty, and in the commit message you should reference the patch that landed in master already18:48
devanandawith a detailed explanation of this situation -- the only part that needed to be backported is this small fix, here's why, etc18:48
Nishadevananda, oh if thats fine, i will do that18:49
Nishadevananda, thanks. i can do that18:49
NobodyCamthats the easiest way18:49
Nishadevananda, the test case is missing in master for this fix18:49
Nishathat should be fine while proposing the fix in liberty ?18:50
*** Marga_ has quit IRC18:50
devanandaNisha: no, that's not fine :)18:50
devanandaNisha: please propose the test case to master18:50
NobodyCamI would say add nessary test to the patch18:50
Nishadevananda, so i should add the test case as part of this bug or should i raise another bug for adding test case18:51
Nishain master18:51
devanandaNisha: #related-to: bug18:51
devanandause the same bug18:51
devanandajust don't use the #closes flag :)18:51
Nisha:) that eases the job18:51
NishaThanks18:52
NishaI will propose the fix to liberty for this then18:52
NishaThanks18:52
devanandawelcome18:53
devanandayou may also want to email the list to explain and get some more eyes on it, eg. from the stable-maint team18:53
Nishadevananda, sure so i should mail to stable-maint team or openstack-dev alias?18:54
devanandaI'm glad to see that the fix for liberty is pretty small -- that makes this much easier, even if it's not the ideal flow of just backporting a single commit from master18:54
Nishadevananda, yeah the fix is pretty small.but is needed for functionality to work18:55
devanandaNisha: [ironic] [stable]18:55
Nishaok :) got it. thanks18:55
devanandanp18:55
devanandatime for me to run - catching a cab to the airport18:55
Nisha:)18:56
*** Sukhdev has joined #openstack-ironic19:01
openstackgerritStephanie Miller proposed openstack/bifrost: ironic-inspector install doc polish  https://review.openstack.org/23481019:09
openstackgerritDavanum Srinivas (dims) proposed openstack/python-ironicclient: Last sync from oslo-incubator  https://review.openstack.org/24482419:12
*** achanda has quit IRC19:21
*** achanda has joined #openstack-ironic19:21
*** bnemec has joined #openstack-ironic19:33
zer0c00lhey lucas-afk, Great! May be we should propose something to use blivet to do partitioning ? Do some experiments?19:36
*** ukalifon1 has joined #openstack-ironic19:38
*** itsuugo has quit IRC19:38
*** ijw has joined #openstack-ironic19:39
*** ijw has quit IRC19:43
*** greghaynes_ has quit IRC19:47
openstackgerritRuby Loo proposed openstack/ironic: Update etc/ironic/ironic.conf.sample  https://review.openstack.org/24484119:47
*** greghaynes_ has joined #openstack-ironic19:50
*** linggao has joined #openstack-ironic19:54
*** thiagop has quit IRC20:04
*** gabriel has quit IRC20:04
*** sinval has quit IRC20:05
*** caiobo has quit IRC20:05
*** nicodemos has quit IRC20:05
*** afaranha has quit IRC20:05
*** ukalifon1 has quit IRC20:09
*** mbound has joined #openstack-ironic20:16
*** dprince has quit IRC20:24
*** ijw has joined #openstack-ironic20:33
*** arturas has quit IRC20:33
mrdaMorning Ironic20:34
*** ijw_ has joined #openstack-ironic20:36
*** ijw has quit IRC20:37
NobodyCammorning mrda ... :)20:37
mrdahi NobodyCam20:38
NobodyCam:)20:42
*** Sukhdev has quit IRC20:46
*** cdearborn has joined #openstack-ironic21:00
*** Marga_ has joined #openstack-ironic21:12
openstackgerritRuby Loo proposed openstack/ironic: Update etc/ironic/ironic.conf.sample  https://review.openstack.org/24484121:14
*** itsuugo has joined #openstack-ironic21:17
*** achanda has quit IRC21:23
*** achanda has joined #openstack-ironic21:26
*** achanda has quit IRC21:26
*** achanda has joined #openstack-ironic21:27
*** kozhukalov_ has joined #openstack-ironic21:37
*** kozhukalov_ has quit IRC21:42
*** achanda has quit IRC21:54
crinkleSpamapS: I'm wondering about https://review.openstack.org/#/c/217374/6/playbooks/roles/bifrost-deploy-nodes-dynamic/tasks/main.yml - why would we only want to setup DHCP if instance_info is defined, and then why would we not restart dnsmasq afterward?21:54
*** achanda has joined #openstack-ironic21:54
*** e0ne has joined #openstack-ironic21:54
crinklefrom http://git.openstack.org/cgit/openstack/bifrost/tree/playbooks/library/os_ironic_node.py#n63 i'm not sure how instance_info is related21:54
*** kozhukalov_ has joined #openstack-ironic21:54
SpamapScrinkle: me neither. Clearly a comment was needed there, as I've forgotten why I thought that might work.21:56
*** spandhe has quit IRC21:58
*** Marga_ has quit IRC21:58
crinkleSpamapS: fun :)21:58
jrolldevananda: thanks for updating my spec :D22:03
*** Marga_ has joined #openstack-ironic22:04
*** harshs has joined #openstack-ironic22:05
*** jamielennox|away is now known as jamielennox22:05
openstackgerritNisha Agarwal proposed openstack/ironic: Adds test case for the bug #1510425  https://review.openstack.org/23962722:06
openstackbug 1510425 in Ironic "the take_over for iscsi_ilo drivers doesnt recreate the boot iso" [Undecided,In progress] https://launchpad.net/bugs/1510425 - Assigned to Nisha Agarwal (agarwalnisha1980)22:06
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/24445322:08
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/24418622:08
*** Sukhdev has joined #openstack-ironic22:10
*** Marga_ has quit IRC22:11
*** itsuugo has quit IRC22:11
*** david-lyle has joined #openstack-ironic22:16
*** sacharya has quit IRC22:19
*** boris-42 has joined #openstack-ironic22:19
*** dims_ has joined #openstack-ironic22:23
crinkleTheJulia: do you know the answer to my question above ^ ? looks like you fixed it up in https://review.openstack.org/#/c/236001 but the logic is basically the same22:24
*** dims has quit IRC22:25
openstackgerritNisha Agarwal proposed openstack/ironic: Adds test case for the bug #1510425  https://review.openstack.org/23962722:32
openstackbug 1510425 in Ironic "the take_over for iscsi_ilo drivers doesnt recreate the boot iso" [Undecided,In progress] https://launchpad.net/bugs/1510425 - Assigned to Nisha Agarwal (agarwalnisha1980)22:32
*** Marga_ has joined #openstack-ironic22:32
*** lnxnut has quit IRC22:32
*** trown is now known as trown|outttypeww22:35
*** Nisha has quit IRC22:53
*** kozhukalov_ has quit IRC23:00
*** thrash is now known as thrash|g0ne23:03
*** kozhukalov_ has joined #openstack-ironic23:07
*** jamielennox is now known as jamielennox|away23:13
*** kozhukalov_ has quit IRC23:20
*** jamielennox|away is now known as jamielennox23:22
*** cdearborn has quit IRC23:35
*** lynxman has quit IRC23:52
*** lynxman has joined #openstack-ironic23:52
*** achanda has quit IRC23:55
*** harshs has quit IRC23:56
*** e0ne has quit IRC23:56

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