Monday, 2016-10-10

« Sunday, 2016-10-09 Index Tuesday, 2016-10-11 »
*** vsaienko has quit IRC00:06
*** hoangcx has joined #openstack-ironic00:45
*** hoangcx has quit IRC00:46
*** hoangcx has joined #openstack-ironic00:47
*** xuao has joined #openstack-ironic01:03
*** david-lyle_ has joined #openstack-ironic01:04
*** david-lyle has quit IRC01:08
*** david-lyle_ is now known as david-lyle01:10
*** tuanluong has joined #openstack-ironic01:14
*** vishwanathj has joined #openstack-ironic01:20
*** mandre has quit IRC01:26
*** mandre has joined #openstack-ironic01:26
*** xek_ has joined #openstack-ironic01:28
*** mtreinish has quit IRC01:28
*** jistr has quit IRC01:29
*** xek has quit IRC01:29
*** jtomasek has quit IRC01:29
*** charlledaniel has quit IRC01:30
*** charlledaniel has joined #openstack-ironic01:30
*** ChubYann has quit IRC01:30
*** jtomasek has joined #openstack-ironic01:34
*** jistr has joined #openstack-ironic01:35
*** mtreinish has joined #openstack-ironic01:35
*** awidders_ has joined #openstack-ironic01:37
*** awiddersheim has quit IRC01:37
*** ChubYann has joined #openstack-ironic01:38
*** baoli has joined #openstack-ironic01:46
*** yibo has quit IRC02:17
*** tiendc has joined #openstack-ironic02:42
*** heyongli has joined #openstack-ironic02:45
*** sdake has joined #openstack-ironic02:46
*** MattMan_ has quit IRC02:52
*** MattMan has joined #openstack-ironic02:52
*** yonglihe has joined #openstack-ironic03:02
*** david-lyle has quit IRC03:09
*** baoli has quit IRC03:31
*** vikrant has joined #openstack-ironic03:38
*** sdake has quit IRC03:41
*** bfournie has quit IRC03:54
*** pmannidi_ is now known as pmannidi04:04
*** phuongnh has joined #openstack-ironic04:07
*** hamzy_ has quit IRC04:10
*** moshele has joined #openstack-ironic04:47
*** jaosorior has joined #openstack-ironic05:07
*** jaosorior has quit IRC05:10
*** jaosorior has joined #openstack-ironic05:11
*** e0ne has joined #openstack-ironic05:29
*** e0ne has quit IRC05:31
*** lazy_prince has quit IRC05:32
*** ishant|away has quit IRC05:32
*** kirankv has quit IRC05:35
*** e0ne has joined #openstack-ironic05:35
*** yonglihe has quit IRC05:36
*** e0ne has quit IRC05:39
*** baoli has joined #openstack-ironic05:43
*** baoli has quit IRC05:47
openstackgerritlinbing proposed openstack/ironic-python-agent: Update .gitignore to ignore .idea of PyCharm  https://review.openstack.org/38429205:59
openstackgerritlinbing proposed openstack/python-ironicclient: Update .gitignore to ignore .idea of PyCharm  https://review.openstack.org/38429306:01
*** kuntumtarm has joined #openstack-ironic06:07
*** vsaienko has joined #openstack-ironic06:08
*** vsaienko has quit IRC06:10
*** rbartal has joined #openstack-ironic06:13
pas-hamorning Ironic :)06:13
tuanluongmorning ironic06:21
*** mjura has joined #openstack-ironic06:32
*** kuntumtarm has quit IRC06:34
*** mjura has quit IRC06:39
*** mjura has joined #openstack-ironic06:39
*** duonghq has joined #openstack-ironic06:40
*** rcernin has joined #openstack-ironic06:48
*** vsaienko has joined #openstack-ironic06:50
*** pcaruana has joined #openstack-ironic07:03
*** tesseract has joined #openstack-ironic07:06
*** tesseract is now known as Guest4972307:06
*** vsaienko has quit IRC07:10
openstackgerritqinchunhua proposed openstack/python-ironicclient: Replace 'MagicMock' with 'Mock'  https://review.openstack.org/38398507:13
*** yibo has joined #openstack-ironic07:15
openstackgerritqinchunhua proposed openstack/python-ironicclient: Replace 'MagicMock' with 'Mock'  https://review.openstack.org/38398507:16
*** snecklifter has joined #openstack-ironic07:18
*** vsaienko has joined #openstack-ironic07:19
*** hshiina has joined #openstack-ironic07:23
*** tiendc has quit IRC07:27
*** jpich has joined #openstack-ironic07:38
*** ccamacho has joined #openstack-ironic07:38
*** ccamacho is now known as ccamacho|mtg07:39
*** hoangcx has quit IRC07:46
openstackgerritXu Ao proposed openstack/ironic: Make ironic console supporting the nokeepalive mode  https://review.openstack.org/37283007:47
openstackgerritZhou Ya proposed openstack/ironic-python-agent: Get numa_node info when collecting pci devices info  https://review.openstack.org/36924507:52
*** vsaienko has quit IRC07:53
*** athomas has joined #openstack-ironic07:53
*** athomas has quit IRC07:53
*** vsaienko has joined #openstack-ironic07:53
*** athomas has joined #openstack-ironic07:54
*** strigazi1AFK is now known as strigazi07:56
*** zzzeek has quit IRC08:00
*** bjolo_ has joined #openstack-ironic08:00
*** zzzeek has joined #openstack-ironic08:01
*** e0ne has joined #openstack-ironic08:01
*** makowals has quit IRC08:12
*** makowals has joined #openstack-ironic08:13
*** milan has joined #openstack-ironic08:16
*** zhangjl has joined #openstack-ironic08:16
milanmorning Ironic! :)08:17
*** ccamacho|mtg is now known as ccamacho08:21
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Fix OSC port create and port set commands  https://review.openstack.org/38434508:22
*** Romanenko_K has joined #openstack-ironic08:23
openstackgerritNishant Kumar proposed openstack/ironic-inspector: Stop adding ServiceAvailable group option  https://review.openstack.org/38434608:24
xuaomorning milan~morning ironic~08:24
milanmorning xuao! :)08:25
*** ohamada has joined #openstack-ironic08:29
tuanluonghi milan , xuao08:31
milanhi tuanluong! :)08:31
tuanluongay milan, I have successed deploy ironic node but  u'ipmi_force_boot_device' didn't have in my bm node08:32
*** dtantsur|afk is now known as dtantsur08:32
dtantsurMorning Ironic08:32
milanmorning dtantsur! :)08:32
tuanluongMorning dtantsur08:32
tuanluongI have tried to investigate in https://bugs.launchpad.net/ironic/+bug/162645308:33
openstackLaunchpad bug 1626453 in Ironic "Setting persistent boot device does not work with ipmi_force_boot_device on" [High,In progress] - Assigned to Tuan (tuanla)08:33
tuanluonghello dtantsur, when I test im my node (after i reinstall devstack and deploy ironic node) my bm node didn't have 'ipmi_force_boot_device'08:35
milantuanluong, so you hit the same issue as the bug mentions?08:36
tuanluongmilan, no, I am trying to produce this bug08:36
milanah08:37
tuanluongI have been change some equiment in my lab,08:37
aarefievmorning!08:38
tuanluongHow can i set manualy 'ipmi_force_boot_device' in driver_info08:39
milanmorning aarefiev!08:39
aarefievmorning milan08:39
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Catch error when wrong temp_url were generated  https://review.openstack.org/37087508:40
milantuanluong, not sure, sry, I'd guess through some CLI switch but I'm just guessing08:42
*** sdake has joined #openstack-ironic08:43
tuanluongmilan, yest when i test node-set-boot-device it show that work. http://paste.openstack.org/show/585139/08:44
milanhmmm08:45
*** jralph__ has joined #openstack-ironic08:45
milantuanluong, the bug mentions a second reboot, would you mind trying again?08:45
*** vishwana_ has joined #openstack-ironic08:46
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902108:46
tuanluongmilan, yes08:46
*** early` has joined #openstack-ironic08:47
*** strigazi_ has joined #openstack-ironic08:49
*** ptx0_ has joined #openstack-ironic08:49
*** pleia2_ has joined #openstack-ironic08:50
*** yibo_ has joined #openstack-ironic08:50
tuanluongmilan, dtantsur : yes, it work http://paste.openstack.org/show/585146/08:50
*** dansmith_ has joined #openstack-ironic08:53
milantuanluong, do the ironic versions match with the bug? are you using an ipmi driver to control your bm?08:53
*** chihhsin_ has joined #openstack-ironic08:54
tuanluonghttp://paste.openstack.org/show/585147/08:54
*** athomas has quit IRC08:54
*** yibo has quit IRC08:54
*** pcaruana has quit IRC08:54
*** vishwanathj has quit IRC08:54
*** xuao has quit IRC08:54
*** xavierr has quit IRC08:54
*** caiobo has quit IRC08:54
*** ricardoas has quit IRC08:54
*** strigazi has quit IRC08:54
*** chihhsin has quit IRC08:54
*** pleia2 has quit IRC08:54
*** early has quit IRC08:54
*** jralph has quit IRC08:54
*** vsaienk0 has quit IRC08:54
*** ptx0 has quit IRC08:54
*** dansmith has quit IRC08:54
*** aarefiev has quit IRC08:54
*** pshige_________ has quit IRC08:54
*** aslezil has quit IRC08:54
*** vgadiraj has quit IRC08:54
*** rybridges2 has quit IRC08:54
*** jcook has quit IRC08:54
*** hoonetorg has quit IRC08:54
*** dansmith_ is now known as dansmith08:54
tuanluongyes milan because i don't have information version. not sure but i guess someone fix that08:54
*** dansmith is now known as Guest1232908:55
milantuanluong, OK08:55
milantuanluong, sorry I can't help more, just if you wan't to confirm it's fixed now, please update the bug with your logs and versions in which it works for you08:56
*** aslezil has joined #openstack-ironic08:57
*** vgadiraj has joined #openstack-ironic08:58
dtantsurtuanluong, note that node-get-boot-device only returns whatever is cached in database in this case. it says nothing about whether it worked with real hardware.08:58
tuanluongyes dtantsur, I will looking deeply in my hardware but because i using Fujisu Server for test Not sure that the same with orther08:59
tuanluongthanks dtantsur , milan .09:00
*** caiobo has joined #openstack-ironic09:00
*** pcaruana has joined #openstack-ironic09:00
*** jcook has joined #openstack-ironic09:01
*** athomas has joined #openstack-ironic09:01
*** xavierr has joined #openstack-ironic09:01
*** hoonetorg has joined #openstack-ironic09:01
*** pshige_________ has joined #openstack-ironic09:01
milandtantsur, tuanluong it would be good to mention in the bug how to check it works w/ reproducer steps e.g that you need to check X,Y,Z to confirm the bug (fixed)09:01
*** aarefiev has joined #openstack-ironic09:01
*** rybridges2 has joined #openstack-ironic09:02
*** ricardoas has joined #openstack-ironic09:02
*** vsaienk0 has joined #openstack-ironic09:03
dtantsurjroll, JayF, xenial issues tracker: https://bugs.launchpad.net/ironic/+bug/163187509:03
openstackLaunchpad bug 1631875 in Ironic "iSCSI CI jobs don't work on Xenial" [Critical,Confirmed]09:03
sergekmorning all09:16
*** lazy_prince has joined #openstack-ironic09:21
*** kirankv has joined #openstack-ironic09:21
*** ishant|away has joined #openstack-ironic09:21
*** sambetts|afk is now known as sambetts09:23
sambettsMorning all09:23
dtantsurmorning sambetts09:23
sambettso/ dtantsur09:23
sergeko/ sambetts09:23
sambettsjroll: just saw your email wrt the neutron meeting, I should be able to chair it :)09:23
*** Guest10079 is now known as zigo09:23
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Fix OSC port create and port set commands  https://review.openstack.org/38434509:24
*** vsaienko has quit IRC09:24
*** mgould|afk is now known as mgould09:26
mgouldmorning tuanluong milan dtantsur sergek sambetts and everyone else!09:26
*** vsaienko has joined #openstack-ironic09:27
milanmorning mgould, sambetts, sergek! :)09:27
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902109:27
sambettso/ sergek milan mgould09:27
*** snecklifter has quit IRC09:28
*** ishant|away has quit IRC09:29
sergeko/ sambetts09:30
*** lazy_prince has quit IRC09:30
*** kirankv has quit IRC09:30
*** hamzy has quit IRC09:35
*** vsaienko has quit IRC09:35
*** derekh has joined #openstack-ironic09:36
milanaarefiev, dtantsur, sambetts just an FYI: the API WG suggestions as a patch to the API spec: https://review.openstack.org/#/c/383862/ ; see the thread: http://lists.openstack.org/pipermail/openstack-dev/2016-October/105180.html09:37
patchbotpatch 383862 - api-wg - Specify time intervals based filtering queries09:37
*** jaosorior has quit IRC09:38
*** jaosorior has joined #openstack-ironic09:38
openstackgerritNishant Kumar proposed openstack/ironic-inspector: Stop adding ServiceAvailable group option  https://review.openstack.org/38434609:40
tuanluonghello09:42
dtantsurmilan++ good job09:42
milandtantsur, thx!09:42
*** hoangcx has joined #openstack-ironic09:42
dtantsursambetts, wanna check the LLDP spec https://review.openstack.org/#/c/374381/ ? it's all RH folks for now09:43
patchbotpatch 374381 - ironic-inspector-specs - Display LLDP information via new commands09:43
sambettsdtantsur: sure :)09:44
sambettsmilan: just caught up on the WG thread, it sounds good to me :D09:44
milansambetts, thanks :)09:45
*** e0ne has quit IRC09:45
dtantsuractually, I've -1ed it, as new CLI design requires to not list things for all nodes09:47
dtantsurbut please review anyway09:47
sambettsmilan: WRT the proposal, I wonder if you should specify the "between" operator separatly from the ISODate use case09:47
*** hamzy has joined #openstack-ironic09:47
sambettsmilan: because it might be possible to use it outside of that use case09:47
mgouldmilan++09:47
milansambetts, yeah, lemme check the spec cause I think that one has already been suggested09:49
milansambetts, so you're right, it wasn't09:50
milansambetts, mind a review comment? ;)09:50
sambettsmilan: done :)09:51
milanmgould, thx :)09:51
milancool09:51
milanI'll be posting a fix soon09:52
vdrokgood morning everyone, dtantsur tuanluong milan sambetts09:53
milanmorning vdrok! :)09:54
mgouldmorning vdrok09:54
vdrokand morning mgould and sergek :)09:54
*** duonghq has quit IRC09:55
tuanluonghi milan, dtantsur, I have been understand more about the bug, sometime deplay in database but the problem also happend when we didn't set ipmi_force_boot_device. http://paste.openstack.org/show/585158/10:00
milantuanluong, you might want to add this as a comment to the bug so people can consider it when fixing/reviewing10:02
*** jistr is now known as jistr|mtg10:03
tuanluongMilan, It also bugs. tomorrow i will investigate more and debug10:03
milantuanluong, thanks10:04
*** tangchen has joined #openstack-ironic10:09
*** hoangcx has quit IRC10:11
sambettsdtantsur: do you not think that you could run "openstack baremetal introspection interface list" without a node to list all nodes lldp info?10:11
*** tuanluong has quit IRC10:12
dtantsursambetts, the command to list all nodes is "openstack baremetal introspection list" I suggested --interfaces flag to it, but seems like it was rejected10:12
dtantsurwe should avoid having two commands do roughly the same with different output10:13
*** zhangjl has left #openstack-ironic10:14
mgoulddtantsur: +110:14
mgouldthat's a recipe for having two commands, both of which do *part* of what you need10:14
dtantsurexactly10:15
sergeko/ vdrok10:17
*** bfournie has joined #openstack-ironic10:18
sambettsdtantsur: so I'd have to do introspection list followed by for node in <result>: introspection interface list <node>10:20
sambettsinstead of just introspection interfaces list10:21
dtantsurright10:21
dtantsuror plan on extending introspection list10:21
dtantsurwhich I'm not sure is a great idea, to be honest10:21
dtantsurbecause each node has a few interfaces, and the whole resulting table looks a bit ugly and against OSC ideas10:22
dtantsurnote that OSC operates with objects and verbs, in similar fashion to RESTful API10:22
dtantsurso if we have "interface" object, "interface list" lists such objects (not nodes!)10:23
sambetts++ interfaces belong to nodes therefore we should always specify a node10:23
sambettsgot it just wanted to check that our thought process was the same :)10:24
dtantsurcool10:24
dtantsurjroll, TIL earliest-version flag in http://docs.openstack.org/developer/reno/sphinxext.html#directive-release-notes10:25
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902110:25
sambettsdtantsur: I experimented with that but didn't have much luck, it would be cool if it worked though10:25
dtantsurah, I see10:26
* mgould would expect `... interfaces list` to show all interfaces by default, and to need something like `... interfaces list --node=<uuid>` to show interfaces for a specific node10:28
mgouldbut that might disagree with CLI guidelines, I'm not sure10:28
sambettsmgould: thats the current suggestion, but "interfaces" aren't a top level resource inspector provides10:29
mgouldah, OK, I see the problem now10:29
sambettsdtantsur: based on this dicussion wrt to interfaces, what are your thoughts on the "introspection vlan list" ? It feels like it falls into the same catagory to me10:30
sambettsdtantsur: I also think the output is quite messy10:30
dtantsursambetts, yeah, maybe10:31
dtantsurmgould, in addition to what sambetts said, if "interface list" without a node is a thing, then "interface show eth0" without a node should also make some sense10:31
*** Guest2483 is now known as rook10:31
sambettswhich just doesn't work :-P10:32
openstackgerritMerged openstack/ironic-python-agent: [tinyipa] remove trailing slash from TC repo URL  https://review.openstack.org/38222710:32
dtantsuryep10:32
openstackgerritMerged openstack/ironic-python-agent: [tinyipa] allow skipping python compileall  https://review.openstack.org/38126010:32
mgoulddtantsur: hmmm, yeah10:37
mgouldwe'd need UUIDs for interfaces, which seems like work for no real benefit10:37
sambettsI guess the difference between interfaces and vlans is that vlans already have a unique number (although with hierarcical port binding that might not be the case)10:41
sambettse.g. you can have two different VXLAN networks that then have the same vlans underneith them, but from the LLDP perspective you would just see VLAN 101 without knowing its on different VXLANs10:44
*** mmedvede has joined #openstack-ironic10:45
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: iPXE allow skip caching in standalone mode  https://review.openstack.org/36732110:48
*** phuongnh has quit IRC10:49
caioboGood morning Ironic :)10:54
mgouldmorning caiobo10:54
caiobomorning mgould10:54
bfournieGood morning!10:55
dtantsurmorning bfournie10:55
bfourniesambetts, dtantsur, mgould: catching up on lldp interface discussion, missed first part as was disconnected10:56
tangchenmorning guys10:56
dtantsur<dtantsur> note that OSC operates with objects and verbs, in similar fashion to RESTful API10:57
dtantsur<dtantsur> so if we have "interface" object, "interface list" lists such objects (not nodes!)10:57
dtantsur* mgould would expect `... interfaces list` to show all interfaces by default, and to need something like `... interfaces list --node=<uuid>` to show interfaces for a specific node10:57
dtantsur<mgould> but that might disagree with CLI guidelines, I'm not sure10:57
dtantsur<sambetts> mgould: thats the current suggestion, but "interfaces" aren't a top level resource inspector provides10:57
dtantsur<dtantsur> mgould, in addition to what sambetts said, if "interface list" without a node is a thing, then "interface show eth0" without a node should also make some sense10:57
dtantsurbfournie, tl;dr ^^^10:57
bfourniedtantsur: thanks, got it.10:58
mgouldmorning bfournie10:59
bfourniemorning dtantsur, mgould! so is recommendation to switch to 'interfaces'? or rework command to use other name?11:03
caiobodtantsur, is this a feature for the OSC - bare metal for the next release?11:03
dtantsurbfournie, very tl;dr - node should be mandatory for "interface list" command11:04
bfourniedtantsur: ok11:05
sambettsI'm still unsure on the vlan CLI for the same reasons as the interfaces one11:05
dtantsurwell, I guess "vlan" can be a top level object, modulo some really tricky use cases?11:07
*** lazy_prince has joined #openstack-ironic11:11
*** kirankv has joined #openstack-ironic11:11
*** ishant|away has joined #openstack-ironic11:11
sambettsI'm just not sure on the suggested output format with the VLAN to node: interfaces dict/list, I think it'll probably result in a very messy looking output when for example all nodes ever introspected are on a single inspection VLAN so you end up with 1 giant row11:13
dtantsurhmm, I forgot to voice the same concern... maybe we should split "vlan list" (only IDs and names, maybe also switches) and "vlan show" everything, including nodes? or even "vlan nodes list"11:13
dtantsuryeah, I guess "vlan show" may not be interesting, so maybe "vlan list" and "vlan node list <VLAN ID>"11:15
sambettsI was wondering if we should do it on a query, something like "introspection list --vlans=['1', '2', '3']" ?11:15
dtantsurnice idea (but in OSC it will look like "--vlan 1 --vlan 2" or --vlan 1,2)11:17
dtantsurbfournie, wdyt ^^^11:17
* dtantsur grabs food real quick11:17
bfourniesambetts, dtantsur: vlan command is really just reformatting of info in the interface list command for the 'switch_port_vlans' field so yeah maybe can work it into list command.  What it does is provide a view across all nodes, but if 'interface list' is across all nodes could just use the "—field switch_port_vlans" parameter.11:21
bfournieI like "—vlans", I think we can make that work11:21
bfourniealthough would still end up with giant row when all nodes use same vlan11:23
mgould"giant rows" are a common hazard of openstack commands, IME11:24
*** Alexey_Abashkin_ has joined #openstack-ironic11:24
sambettsso I was thinking, introspection list --vlan=1 --vlan=2 would result in a list of nodes that are on those vlans11:24
*** ralonsoh has joined #openstack-ironic11:25
sambettsthen you can do introspection interface list <node id> to get full info on a node11:25
*** awidders_ has quit IRC11:25
sambettsand you could even add a filter on vlan there11:25
bfourniesambetts: oh, I see. yeah, that's good, that would avoid the messy dict list for the node/interfaces11:26
sambettsintrospection interface list --vlan=111:26
sambettsintrospection interface list <node id> --vlan=111:26
bfourniesambetts: but in the vlan case node would not be required?11:26
sambettsit is, that was my mistake :-P11:26
*** Alexey_Abashkin has quit IRC11:27
*** awiddersheim has joined #openstack-ironic11:27
sambettsSo the commands would be "introspection list --vlan=1 --vlan=2" followed by "introspection interface list <node id> --vlan=1 --vlan=2"11:27
sambettsdtantsur: WDYT ^11:27
*** awiddersheim has quit IRC11:28
*** awiddersheim has joined #openstack-ironic11:28
openstackgerritKyrylo Romanenko proposed openstack/ironic: Add headers and extra_headers to arguments in tempest plugin  https://review.openstack.org/37557011:28
dtantsursambetts, so the latter command will list interfaces on that VLANs? +1 to both, if so11:29
sambettsdtantsur: yeah, so first command gets you the nodes on that vlan, then you can dig into it with the second command and see which interfaces on that vlan11:29
dtantsuragreed, great idea11:30
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902111:30
bfourniesambetts: looks good!11:30
dtantsurmgould, oh, so true (re "giant rows" are a common hazard of openstack commands")11:30
sambettsmgould: ++ but is it the framework or the command designers fault that is the question ;)11:31
milanmgould, sry I've just noticed wrong copy&paste block I used O:-)11:31
mgouldsambetts: good question11:31
mgouldpossibly the framework could make it easier for command designers to avoid the problem :-)11:32
milanmorning bfournie! :)11:32
bfourniemilan: good morning!11:33
aarefievso if we list interfaces on some vlan, it is possible to mesh interfaces from other one11:34
aarefievsambetts ^11:34
*** jistr|mtg is now known as jistr11:34
openstackgerritStenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation  https://review.openstack.org/37707311:34
aarefievand morning dtantsur, sambetts, mgould, bfournie11:35
sambettsaarefiev: I'm not sure I understand :/11:35
aarefievsambetts; I'm reading your example about vxlan's11:35
aarefievsame vlan id on diff vxlans11:37
*** ccamacho is now known as ccamacho|lunch11:37
sambettsso with my suggested improvments to the CLI I think it avoids this case because it doesn't group VLANs together anymore11:38
aarefievsambetts: how inspector will distinguish that this is diff vlans11:39
aarefievis there some unique id11:39
sambettshowever there is still the problem that if you filter the nodes by VLAN, that even though its 1 vlan id they might not be in the same LAN, but I don't think this is a problem we can solve without also integrating with neutron so we can know the heirarchy11:40
dtantsuryeah, let's postpone integrating inspector with neutron :)11:40
sambettsaarefiev: some vendors might have a way to expose this via the LLDP, but its not a standard thing11:40
sambettsdtantsur: ++11:40
aarefievaha, I see11:40
aarefievthanks sambetts11:41
*** sdake has quit IRC11:41
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: [EXPERIMENTAL] Get rid of provisioning states rollback  https://review.openstack.org/38373811:45
*** lucas-afk is now known as lucasagomes11:45
lucasagomesmorning all11:45
mgouldmorning lucasagomes11:46
caiobomorning lucasagomes11:46
aarefievmorning lucasagomes11:46
*** dprince has joined #openstack-ironic11:49
dtantsurmorning lucasagomes11:51
*** eset has quit IRC11:53
milanmorning caiobo, lucasagomes! :)11:53
caiobomorning milan :)11:54
sambettshey lucasagomes11:55
lucasagomeso/11:58
bfourniemorning lucasagomes11:59
*** baoli has joined #openstack-ironic12:00
bfourniesambetts: so we currently don't have an 'introspection list' command, would it only be added for the —vlan?12:00
bfourniesambetts: or are there other uses for this command?12:01
dtantsurbfournie, plenty; milan is working on it12:02
dtantsurat least we need to give a user an overview over recent introspections12:02
bfourniedtantsur: ah, cool12:02
milandtantsur, ++12:03
milantoo much bike-shedding though ;)12:03
dtantsurheh12:03
dtantsurmilan, well, the command itself can proceed now :)12:03
milanyeah12:03
sambettsbfournie: For reference https://review.openstack.org/#/c/375045/7/specs/list-introspection-statuses.rst12:04
patchbotpatch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses12:04
milanhttps://review.openstack.org/#/c/383862/4/guidelines/pagination_filter_sort.rst12:05
patchbotpatch 383862 - api-wg - Specify time intervals based filtering queries12:05
bfourniesambetts: thanks!  dtantsur: we could add back in '—interfaces' that we had discussed originally12:05
milanhttps://review.openstack.org/#/c/344921/12:05
patchbotpatch 344921 - ironic-inspector - Add API for listing all introspection statuses12:05
*** baoli has quit IRC12:05
milanbfournie, ^ & ^^ too12:05
tangchenlucasagomes, dtantsur, mgould, milan: morning guys. :)  would you please help to review these two client patches ? https://review.openstack.org/#/c/377607/   https://review.openstack.org/#/c/378140/12:09
patchbotpatch 377607 - python-ironicclient - Add missing unit tests for provision state commands12:09
patchbotpatch 378140 - python-ironicclient - Trivial: Rename and reorder deploy baremetal test12:09
mgouldmorning tangchen12:09
bfourniethanks milan. in the spec above though, CLI section has "introspection statuses" command not "introspection list", unless I am missing it…12:10
milanmorning tangchen, I'll put in todo ;)12:10
milanbfournie, yeah, I think dtantsur mentioned it too, but should be the same thing12:11
milanI'll correct it once the API WG patch lands12:12
bfourniemilan: ok, so it will change to "introspection list"? thanks12:12
milanbfournie, right12:13
bfourniemilan: great12:13
vdrokmorning lucasagomes caiobo and bfournie12:14
caiobomorning vdrok :)12:15
*** ccamacho|lunch is now known as ccamacho12:15
*** baoli has joined #openstack-ironic12:16
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902112:17
*** baoli_ has joined #openstack-ironic12:18
*** tangchen has quit IRC12:18
*** tangchen has joined #openstack-ironic12:21
*** baoli has quit IRC12:21
openstackgerritVasyl Saienko proposed openstack/ironic-specs: Update collect-logs-from-agent-ramdisk spec  https://review.openstack.org/38447012:27
*** vikrant has quit IRC12:30
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/37744612:32
bfourniemorning vrdok12:37
*** moshele has quit IRC12:38
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal driver commands  https://review.openstack.org/36735912:39
TheJuliagood morning12:41
mgouldmorning TheJulia12:42
dtantsurmorning TheJulia12:42
vsaienk0morning TheJulia12:44
*** trown|outtypewww is now known as trown12:44
lucasagomesTheJulia, hi there12:44
jrollmorning y'all12:46
sambettso/ jroll12:46
mgouldmorning jroll12:47
aarefievmorning TheJulia, jroll12:47
jrollvsaienk0: just to make sure I understand correctly, from your email, networking-generic-switch is meant for production yes? and "neutron port in down state" is only a testing issue, right?12:47
dtantsurmorning jroll. do you plan to propose ironic-lib newton release with the configdrive fixes? or should I?12:48
jrolldtantsur: please do12:48
dtantsurk12:48
jrolloh cool, ocata ironic-lib is unblocked12:49
vsaienk0jroll: neutron port is always remain in down state when flat network is used since nobody bind the port in flat network. For multitenancy case neutron port become ACTIVE12:49
jrolland also ironic 6.2.1 for newton12:50
jrollvsaienk0: ah, right ok12:50
sambettsvsaienk0: I thought it got bound onto the host where the Ironic nova compute is running?12:50
vsaienk0sambetts: there is no agent that can bound ironic instance port12:53
vsaienk0sambetts: ovs agent monitors a new interfaces on compute, when kvm VM is created nova adds a new interface q-xxxx, ovs agent see this new interface and bound it. When we talking about ironic instance, port is not created on compute host, as result ovs agent doesn't know anything about it12:55
sambettsah yup got it12:55
vsaienk0sambetts: that is why there is no need to run neutron-agent on compute with ironic virt-driver12:56
sambetts++12:56
*** amotoki has joined #openstack-ironic12:56
sambettsvsaienk0: I'm still +1 on adding a magic Ml2 driver for "binding" baremetal ports on flat networks #12:56
vsaienk0jroll: I meant that we may use networking-generic-switch in production to cover mentioned bugs, bind port in flat network and send notifications to Ironic. And use vendors ML2 drivers to perform actual port binding on the switch12:57
vsaienk0jroll: port can be bound by networking-cisco or arista for example, while notification will be sent by networking-generic-switch12:58
vsaienk0jroll: sambetts: I've build cirros locally with bond support, and checked nova patch https://review.openstack.org/#/c/206163/ it seems to be working12:59
patchbotpatch 206163 - nova - Ironic: use portgroups12:59
jrollvsaienk0: cool, makes sense, let me think about it and I'll reply to your email today13:00
*** ralonsoh_ has joined #openstack-ironic13:02
*** cdearborn has joined #openstack-ironic13:02
*** rbudden has joined #openstack-ironic13:03
*** rbudden has quit IRC13:03
*** ralonsoh has quit IRC13:05
*** ralonsoh_ is now known as ralonsoh13:05
*** e0ne has joined #openstack-ironic13:06
*** ptx0_ has quit IRC13:07
*** ptx0 has joined #openstack-ironic13:09
*** ptx0 has joined #openstack-ironic13:09
openstackgerritMerged openstack/ironic-python-agent: Update .gitignore to ignore .idea of PyCharm  https://review.openstack.org/38429213:09
*** ptx0 has quit IRC13:13
*** ptx0 has joined #openstack-ironic13:13
milanmorning jroll, TheJulia, vsaienk0! :)13:15
vsaienk0o/ milan13:15
TheJuliagood morning milan13:16
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Fix import of ironicclient and reformat docstring  https://review.openstack.org/37852613:17
vdrokmorning TheJulia and jroll13:21
mariojv_good morning ironic o/13:26
*** mariojv_ is now known as mariojv13:26
openstackgerritVasyl Saienko proposed openstack/python-ironicclient: Add portgroup support to osc plugin  https://review.openstack.org/36213013:27
dtantsurlucasagomes, ironic-lib released, you can continue your root device hints work13:33
lucasagomesdtantsur, yup! Just got the email o/13:33
lucasagomesthat's great13:33
*** mtanino has joined #openstack-ironic13:33
* lucasagomes will do it after dealing with RPMs :-(13:33
*** rbudden has joined #openstack-ironic13:36
*** moshele has joined #openstack-ironic13:36
vsaienk0dtantsur: could you please check https://bugs.launchpad.net/ironic/+bug/1531149 I think we can close that bug13:40
openstackLaunchpad bug 1531149 in ironic-python-agent "Add a DSVM job covering instance local boot" [Wishlist,Triaged] - Assigned to Vasyl Saienko (vsaienko)13:40
dtantsurvsaienk0, local boot is about installing boot loader. I don't think we test it now for partition images outside of tripleo CI13:41
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Do not break sync power states on validation error  https://review.openstack.org/38451313:44
vsaienk0dtantsur: gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-tinyipa uses partitioned image13:45
*** milan has quit IRC13:45
dtantsurvsaienk0, yes, but does it do local or netboot?13:45
*** milan has joined #openstack-ironic13:46
vsaienk0dtantsur: oops, it uses netboot13:46
dtantsuryeah, this is something we miss13:46
sambettsthe only job I know that we have right now that should be testing localboot is the multitenancy one13:46
sambettsbecause you can't netboot in that mode13:46
vsaienk0sambetts: we using fill disk image there13:47
dtantsur:(13:47
vsaienk0*full13:47
sambettsah :(13:47
*** Guest12329 is now known as dansmith13:47
sambettsvsaienk0: it would be good to have one for each13:47
dtantsurI still plan on booting several instances per job, so we can test both in one CI run13:47
sambettsdtantsur: ++, we boot several instances as part of the tempest/grenade jobs so we could just configure the nodes in Ironic in different ways13:48
vsaienk0sambetts, dtantsur: we may add a partitioned image without kernel_id and ramdisk_id and it should do localboot then13:49
*** sdake has joined #openstack-ironic13:49
*** bjolo_ has quit IRC13:53
dtantsurvsaienk0, then it will be treated as whole disk image13:54
*** sdake has quit IRC13:54
vsaienk0dtantsur: yes, we still need to specify boot_option=local explicitly https://github.com/openstack/ironic/blob/659f951d72e96f39bb967455a6855682e517ca43/ironic/drivers/modules/deploy_utils.py#L92013:54
*** sdake has joined #openstack-ironic13:55
dtantsurwell, the point is to test local boot with partition images with both drivers13:55
*** cdearborn has quit IRC13:55
*** moshele has quit IRC13:56
sambettsif the grenade job boots 3 instances, then you can have 1 of each configuration, whole + local, partition + net, partition + local13:57
sambettswe just need to configure the 3 nodes in Ironic to be those 3 different configurations13:58
dtantsurbetter 6, one for each driver13:58
dtantsurthis is the final goal13:58
sambettsalthough you can only use 1 image in tempest oof...13:58
dtantsurwe can have a whole disk job with 2 drivers, and then a partitions job with 2 drivers and 2 boot options13:59
sambetts++13:59
*** anush_ has joined #openstack-ironic14:00
vsaienk0sambetts: dtantsur: we run smoke tests on grenade job14:04
vsaienk0sambetts, dtantsur: if we enroll nodes with different agents, boot_options there will be no guarantee that we test all configurations during test run, nova will randomly pick the hypervisor14:06
mgouldvsaienk0: that's better than deterministically only testing one configuration, iyswim14:08
vsaienk0mgould or use availability zone filter in tests: http://paste.openstack.org/show/550163/14:10
* mgould may have misunderstood the problem14:11
mgoulddo we have the capacity to test every combination we care about?14:11
*** anush_ has quit IRC14:11
mgouldand we need to ensure that we do actually test with those combinations?14:11
mgoulds/combination/configuration/g14:12
vsaienk0mgould: I think that we should find a way to test everything we support14:13
mgouldideally, yes14:13
mgouldbut if that space is too large, exploring it randomly is better than deterministically testing only a small part of it14:14
*** bnemec has joined #openstack-ironic14:15
mgouldbut it sounds like that's not the problem you were trying to solve :-)14:15
openstackgerritNishant Kumar proposed openstack/ironic-inspector: Stop adding ServiceAvailable group option  https://review.openstack.org/38434614:18
sambettsI personally think that we just need to rework our CI jobs to test the different drivers in standalone mode so as to reduce the spin up time for the jobs, and then have many, smaller jobs than few long running ones14:22
dtantsurvsaienk0, well, we just have to launch even instances to cover all nodes14:24
sambettsor the same number of instances as Ironic nodes, before we start shutting instances down14:25
dtantsuryes, this14:25
sambettsI guess tempest could do that, but custom test code like our bm basic ops code could#14:26
*** narasimha_SV has joined #openstack-ironic14:32
narasimha_SVhttp://paste.openstack.org/show/585195/14:32
narasimha_SVgetting this issue while doing pxe boot for a server14:32
*** rbrud has joined #openstack-ironic14:36
*** openstack has joined #openstack-ironic18:24
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Fix broken xenial job  https://review.openstack.org/38465218:27
*** openstack has joined #openstack-ironic19:06
*** yarkot has joined #openstack-ironic19:07
*** bfournie has joined #openstack-ironic19:08
*** sabeen has joined #openstack-ironic19:10
*** yarkot has quit IRC19:10
*** bjolo_ has quit IRC19:14
lucasagomesdevananda, btw, happy birthday :-)19:15
*** dprince has joined #openstack-ironic19:15
*** krtaylor has quit IRC19:17
*** yarkot has joined #openstack-ironic19:21
*** vsaienko has joined #openstack-ironic19:23
*** portdirect has quit IRC19:28
*** aslezil_ has joined #openstack-ironic19:29
*** yarkot has quit IRC19:30
vsaienkolucasagomes have you tried to set mtu to 1450 on br-ex? https://review.openstack.org/#/c/384651/19:32
patchbotpatch 384651 - openstack-infra/devstack-gate - Set PUBLIC_BRIDGE_MTU for Ironic case.19:32
vsaienkolucasagomes: xenial job passed http://logs.openstack.org/52/384652/1/check/gate-tempest-dsvm-ironic-pxe_ipa-postgres-ubuntu-xenial-nv/7ed6cbb/19:34
*** mkam has quit IRC19:34
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Fix broken xenial job  https://review.openstack.org/38465219:34
lucasagomesvsaienk0, currently running ./stack.sh with the patch applied19:34
lucasagomesvsaienk0, awesome!19:34
openstackgerritJohn L. Villalovos proposed openstack/ironic: devstack: Use $HOME/.ssh/id_rsa for SSH private key location  https://review.openstack.org/38348719:35
lucasagomesvsaienk0, deploying19:38
jlvillalmariojv: I hope https://review.openstack.org/383487  has a better commit message now.19:39
patchbotpatch 383487 - ironic - devstack: Use $HOME/.ssh/id_rsa for SSH private ke...19:39
mariojvlooking19:39
mariojvjlvillal: yup, looks good! thanks19:40
jlvillalmariojv: Thanks for the review!19:40
lucasagomesvsaienk0, 2016-10-10 20:42:13.011 DEBUG ironic.common.states [req-2b5c0a23-1b54-42b9-9305-443842357742 None None] Entering new state 'active' in response to event 'done' from (pid=31991) on_enter /opt/stack/ironic/ironic/common/states.py:22519:42
lucasagomeso/19:42
vsaienkolucasagomes o/19:42
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Fix broken xenial job  https://review.openstack.org/38465219:46
openstackgerritVasyl Saienko proposed openstack/ironic: Fix broken xenial job  https://review.openstack.org/38465219:47
vsaienkolucasagomes: could you please vote one more time https://review.openstack.org/#/c/384651/ and update commit message (removed WIP)https://review.openstack.org/#/c/384652/19:47
patchbotpatch 384651 - openstack-infra/devstack-gate - Set PUBLIC_BRIDGE_MTU for Ironic case.19:47
patchbotpatch 384652 - ironic - Fix broken xenial job19:47
vsaienkojroll ^ 2 patches to fix broken xenial jobs one to ironic one to devstack-gatse19:48
*** mmedvede has quit IRC19:48
lucasagomesvsaienk0, sure thing!19:49
*** rama_y has quit IRC19:50
*** anush has quit IRC19:52
narasimha_SVwhen I am using agent_ipmitool19:53
narasimha_SVpxe request is failing at nova-scheduler itself19:53
narasimha_SVthe same which is working with pxe_ipmitool19:54
narasimha_SVhttp://paste.openstack.org/show/585242/19:54
*** mpotdar has quit IRC19:54
*** anush has joined #openstack-ironic19:55
*** mpotdar has joined #openstack-ironic19:55
mariojvnot sure if check experimental was necessary for non ironic lib patches, but reviewed vsaienko. thanks!19:55
*** caiobo is now known as caiobo-afk19:55
*** caiobo-afk has quit IRC19:55
jlvillalvsaienko: Do you have links to the patches?  In the commit message it says: Changed by [0] but I don't see [0]19:57
vsaienkojlvillal https://bugs.launchpad.net/ironic/+bug/1631875/comments/220:02
openstackLaunchpad bug 1631875 in Ironic "iSCSI CI jobs don't work on Xenial" [Critical,In progress] - Assigned to Vasyl Saienko (vsaienko)20:02
devanandalucasagomes: thanks! :)20:03
jlvillalvsaienko: Oh, I thought your reference was referring to a commit that was merged into Neutron.20:03
mariojvcommit for those who don't want to click through: https://github.com/openstack/neutron/commit/51a697817da849c8f9dae9651f17cd863e170fdc20:03
jlvillalThat is what I was thinking20:03
mariojvjlvillal: it was, the commit's above20:03
mariojvunless [0] is supposed to be the bug, but it sounds like it's referencing that commit message20:04
mariojvs/commit message/commit20:04
jlvillalyeah.20:04
mariojvdoes this affect users too, or just the gate?20:05
*** mkam has joined #openstack-ironic20:06
jrollvsaienko: lucasagomes: nice find20:07
*** dprince has quit IRC20:16
*** vsaienko has quit IRC20:17
lucasagomesfolks, I'm calling it a day!20:18
*** rama_y has joined #openstack-ironic20:18
lucasagomeshave a great evening all20:18
*** vsaienko has joined #openstack-ironic20:19
lucasagomesvsaienk0, currently setting up on a fresh xenial install just to make sure it's all good. I will try to deploy a vm on it later20:19
*** lucasagomes is now known as lucas-afk20:19
*** vsaienko has quit IRC20:20
narasimha_SVdoes agent_ipmitool driver requires any additonal parameter or property ?20:21
narasimha_SVwhich is not used in pxe_ipmitool20:21
narasimha_SVsame node pxe booting works for pxe_ipmitool driver but fails at nova-scheduler with agent_ipmitool20:23
mariojvare we ready to +W yuriy's CRUD notification spec? has 2x+2 https://review.openstack.org/#/c/347242/20:31
patchbotpatch 347242 - ironic-specs - Add CRUD and node states notifications to ironic20:31
jrollnice20:33
* jroll reviews20:33
mariojvnarasimha_SV: you can check that via API: http://developer.openstack.org/api-ref/baremetal/?expanded=show-driver-properties-detail20:33
mariojvthanks jim20:33
mrdaMorning Ironic20:38
jroll\o mrda20:39
jlvillalHi mrda20:39
*** vsaienko has joined #openstack-ironic20:39
mariojvmorning mrda20:39
jrollmariojv: +W20:39
mariojv\o/ yuriyz congrats20:39
NobodyCammorning mrda20:40
narasimha_SVmariojv: i dont see much difference with respect to peoperties20:40
narasimha_SVeven node-validate is also showing no issues20:40
mrdao/20:40
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/37744620:41
openstackgerritMerged openstack/ironic-specs: Add CRUD and node states notifications to ironic  https://review.openstack.org/34724220:46
*** trown is now known as trown|outtypewww21:00
*** vsaienko has quit IRC21:06
*** hoonetorg has quit IRC21:15
*** rbrud has joined #openstack-ironic21:15
*** jheroux has quit IRC21:26
*** hoonetorg has joined #openstack-ironic21:27
jlvillalvsaienk0: Are you still around?  https://review.openstack.org/#/c/384651/21:28
patchbotpatch 384651 - openstack-infra/devstack-gate - Set PUBLIC_BRIDGE_MTU for Ironic case.21:28
jlvillalvsaienk0: If you are not around. I was going to push an update to fix the minor issues.21:29
*** hoonetorg has quit IRC21:36
*** narasimha_SV has quit IRC21:38
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib: WIP/DNM: Testing Xenial fix  https://review.openstack.org/38472121:41
*** diablo_rojo has quit IRC21:43
*** rbrud has quit IRC21:49
*** hoonetorg has joined #openstack-ironic21:53
*** absubram has quit IRC22:07
*** mkam has quit IRC22:10
*** cdearborn has quit IRC22:11
*** sabeen has quit IRC22:15
*** arahal_ has quit IRC22:18
*** absubram has joined #openstack-ironic22:19
*** portdirect has joined #openstack-ironic22:20
*** absubram has quit IRC22:22
*** sdake has quit IRC22:44
*** wrouesnel_ has joined #openstack-ironic22:55
openstackgerritJohn L. Villalovos proposed openstack/ironic: Fix broken xenial job  https://review.openstack.org/38465222:56
*** wrouesnel_ has quit IRC22:56
*** IridiumScaffold has joined #openstack-ironic22:56
*** IridiumScaffold is now known as wrouesnel_22:57
*** wrouesnel_ is now known as IridiumScaffold22:57
*** IridiumScaffold has quit IRC22:58
*** aslezil has quit IRC22:58
*** anush has quit IRC22:59
*** mpotdar has quit IRC22:59
*** vgadiraj has quit IRC22:59
*** sabeen has joined #openstack-ironic23:01
*** caiobo has joined #openstack-ironic23:01
*** aslezil_ has quit IRC23:06
*** aslezil has joined #openstack-ironic23:10
*** sdake has joined #openstack-ironic23:12
*** aslezil_ has joined #openstack-ironic23:13
*** aslezil has quit IRC23:14
*** sdake_ has joined #openstack-ironic23:17
*** hoonetorg has quit IRC23:20
*** sdake has quit IRC23:21
*** aslezil_ has quit IRC23:23
*** baoli_ has quit IRC23:27
*** sambetts|afk has quit IRC23:29
*** HenryG has quit IRC23:32
*** HenryG has joined #openstack-ironic23:32
*** rbudden has quit IRC23:33
*** hoonetorg has joined #openstack-ironic23:35
*** rama_y has quit IRC23:45
openstackgerritBob Fournier proposed openstack/ironic-inspector-specs: Add LLDP processing hook and new CLI commands  https://review.openstack.org/37438123:53
« Sunday, 2016-10-09 Index Tuesday, 2016-10-11 »

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