Monday, 2017-01-16

« Sunday, 2017-01-15 Index Tuesday, 2017-01-17 »
*** jistr has quit IRC00:03
*** vsaienko has quit IRC00:06
*** jistr has joined #openstack-ironic00:07
*** zackf has joined #openstack-ironic00:20
*** zackf has quit IRC00:36
*** tuanluong has joined #openstack-ironic00:42
*** hoangcx has joined #openstack-ironic00:50
*** SerenaFeng has joined #openstack-ironic00:58
*** phuongnh has joined #openstack-ironic01:03
*** hw_wutianwei has joined #openstack-ironic01:10
*** rbudden has quit IRC01:10
openstackgerritCao Xuan Hoang proposed openstack/ironic-python-agent: Removes unnecessary utf-8 encoding  https://review.openstack.org/41386601:27
*** sacharya has joined #openstack-ironic01:31
*** zhangjl has joined #openstack-ironic01:34
*** amotoki has joined #openstack-ironic01:42
*** ijw has joined #openstack-ironic01:43
*** ijw has quit IRC01:47
*** jkilpatr has quit IRC01:54
*** gcb has joined #openstack-ironic01:54
*** yonglihe has joined #openstack-ironic01:57
*** mrda is now known as mrda-lca02:00
*** mrda-lca is now known as mrda02:00
*** tiendc has joined #openstack-ironic02:06
*** stevemar has quit IRC02:21
*** stevemar has joined #openstack-ironic02:21
openstackgerritNaohiro Tamura proposed openstack/python-ironicclient: Add soft reboot/poweroff power states.  https://review.openstack.org/24790402:21
*** baoli has joined #openstack-ironic02:24
*** links has joined #openstack-ironic02:25
*** links has quit IRC02:38
*** links has joined #openstack-ironic02:46
openstackgerritCao Xuan Hoang proposed openstack/ironic-inspector: Fix test function name with three underscores to have only two  https://review.openstack.org/42047502:58
openstackgerritNaohiro Tamura proposed openstack/python-ironicclient: Support soft reboot and soft power off with timeout for OSC  https://review.openstack.org/35762703:01
*** moshele has joined #openstack-ironic03:03
*** ijw has joined #openstack-ironic03:53
*** SerenaFeng has quit IRC04:01
*** moshele has quit IRC04:10
*** ijw has quit IRC04:22
openstackgerritNaohiro Tamura proposed openstack/python-ironicclient: Add soft reboot/poweroff power states.  https://review.openstack.org/24790404:25
*** udesale has joined #openstack-ironic04:33
*** baoli has quit IRC04:36
*** Nisha_Agarwal has joined #openstack-ironic04:46
*** moshele has joined #openstack-ironic04:53
*** Sukhdev has joined #openstack-ironic04:59
*** amotoki_ has joined #openstack-ironic05:09
*** amotoki has quit IRC05:12
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off  https://review.openstack.org/21674305:14
*** tiendc_ has joined #openstack-ironic05:27
*** tiendc has quit IRC05:28
*** sacharya has quit IRC05:38
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support  https://review.openstack.org/26425705:38
*** sacharya has joined #openstack-ironic05:39
*** SerenaFeng has joined #openstack-ironic05:45
*** Sukhdev has quit IRC05:50
*** Nisha_ has joined #openstack-ironic05:53
*** Nisha_Agarwal has quit IRC05:55
*** pmannidi has quit IRC06:29
*** moshele has quit IRC06:30
*** pmannidi has joined #openstack-ironic06:32
*** pmannidi is now known as pmannidi|mtg06:32
*** ijw has joined #openstack-ironic06:34
*** ijw has quit IRC06:39
*** karthiks has joined #openstack-ironic06:47
*** greghaynes has quit IRC06:52
*** mnaser has quit IRC06:53
*** afazekas has quit IRC06:53
*** xhku_ has quit IRC06:53
*** xhku has joined #openstack-ironic06:55
*** afazekas has joined #openstack-ironic06:57
*** Nisha_ has quit IRC06:57
*** Nisha_Agarwal has joined #openstack-ironic06:57
*** greghaynes has joined #openstack-ironic06:58
*** pmannidi|mtg is now known as pmannidi07:00
*** e0ne has joined #openstack-ironic07:00
*** afazekas has quit IRC07:02
*** greghaynes has quit IRC07:04
*** pmannidi has quit IRC07:04
*** fxpester has joined #openstack-ironic07:06
*** xhku has quit IRC07:12
*** mnaser has joined #openstack-ironic07:12
*** afazekas has joined #openstack-ironic07:13
*** e0ne has quit IRC07:14
*** moshele has joined #openstack-ironic07:14
*** xhku has joined #openstack-ironic07:15
*** greghaynes has joined #openstack-ironic07:15
*** Nisha_Agarwal has quit IRC07:18
*** pmannidi has joined #openstack-ironic07:18
*** tesseract has joined #openstack-ironic07:21
*** ohamada has joined #openstack-ironic07:22
*** hogepodge_ has joined #openstack-ironic07:24
*** e0ne has joined #openstack-ironic07:33
*** pcaruana has joined #openstack-ironic07:34
*** hogepodge_ has quit IRC07:37
openstackgerritTuan Luong-Anh proposed openstack/python-ironicclient: Replace yaml.load() with yaml.safe_load()  https://review.openstack.org/42053707:38
*** rbartal has joined #openstack-ironic07:40
*** ralonsoh has joined #openstack-ironic07:43
*** gcb has quit IRC07:44
*** gcb has joined #openstack-ironic07:46
*** SerenaFeng has quit IRC08:05
*** amotoki has joined #openstack-ironic08:07
*** vsaienko has joined #openstack-ironic08:08
*** Nisha_Agarwal has joined #openstack-ironic08:11
*** amotoki_ has quit IRC08:11
*** mjura has joined #openstack-ironic08:12
*** e0ne has quit IRC08:14
*** ijw has joined #openstack-ironic08:22
*** ijw has quit IRC08:27
*** trananhkma has joined #openstack-ironic08:30
*** vsaienko has quit IRC08:37
*** amoralej|off is now known as amoralej08:37
openstackgerritMerged openstack/bifrost: Fix indentation of code in network_metadata.py  https://review.openstack.org/42003608:47
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-ironic09:00
*** SerenaFeng has joined #openstack-ironic09:01
*** bcornec1 has quit IRC09:07
*** e0ne has joined #openstack-ironic09:08
*** milan has joined #openstack-ironic09:10
*** bcornec1 has joined #openstack-ironic09:13
*** cdaniel has quit IRC09:16
milanmorning Ironic! :)09:17
*** bcornec1 has quit IRC09:22
*** e0ne has quit IRC09:25
*** e0ne has joined #openstack-ironic09:26
vsaienk0sambetts: yes, cloud-init supports network_data.json, vdrok patch updates it to support portgroups https://review.openstack.org/#/c/38875609:30
patchbotpatch 388756 - nova - Generate necessary network metadata for portgroups09:30
moshelemilan: moring09:34
milanmorning moshele! :)09:34
milanmorning vsaienk0! :)09:34
moshelemilan: can we talk about you comments  on https://review.openstack.org/#/c/264257/ ?09:34
patchbotpatch 264257 - ironic-inspector - Adding InfiniBand Support09:34
milanmoshele, yup09:35
* milan loads it up09:35
moshelemilan: why do you think the  EoIB is not part of dnsmasq interface ?09:37
milanmoshele, https://github.com/openstack/ironic-inspector/blob/master/example.conf#L419 vs https://review.openstack.org/#/c/264257/46/example.conf@42909:40
patchbotpatch 264257 - ironic-inspector - Adding InfiniBand Support09:40
*** derekh has joined #openstack-ironic09:40
milanmoshele, to me it seems for this to work if dnsmasq interface is different from ib interface, forwarding has to be enabled09:41
milanmoshele, or09:41
milanbetter maybe09:41
milanthe filtering should happen on the ib interfaces instead09:41
milanmoshele, so that the "ib rules" are set on the ib interfaces09:42
*** athomas has joined #openstack-ironic09:43
moshelemilan: but  iptables don't support filtering ib interfaces  like it mac filtering that why I am doing it  on the EoIB interface09:43
milanmoshele, i'm not 100% sure but I guess the kernel, when forwarding the packets between interfaces, replaces original macs09:43
milanmoshele, that's cool, but the rules are set on the dnsmasq interfaces, no?09:44
milanmoshele, isn't this the same chain https://review.openstack.org/#/c/264257/46/ironic_inspector/firewall.py@20309:46
patchbotpatch 264257 - ironic-inspector - Adding InfiniBand Support09:46
moshelemilan: yes, so in my setup the  dnsmasq is a bridge which one of it port is the EoIB09:46
milanmoshele, so it's not a generic config09:47
moshelemilan: you want me to block it on the EoIB interface chain?09:47
*** lucas-afk is now known as lucasagomes09:47
lucasagomesmorning all09:47
milanmoshele, wouldn't it be more straightforward if the dnsmasq listened on the Eoib interfaces?09:47
milanmorning lucasagomes09:47
milan:)09:47
milanmoshele, is it possible to block on the EoIB?09:48
milanmoshele, so no bringing or forwarding is required?09:49
sambetts|afkvsaienk0: thanks, it looks like its cloud-init v0.7.7 that supports network-data.json unfortunately the centos image I had, has a version of cloud-init that is too old 0.7.5 :(09:49
milanmorning sambetts|afk! :)09:50
vsaienk0morning milan, lucasagomes and all Ironic'ers!09:50
lucasagomeso/09:50
moshelemilan: no forwarding, I just create br-eth2.85 (to get the provison netwrork dhcp) and the eth2 (EoIB interface) is a port in br-eth209:51
vsaienk0sambetts: I've tested it with xenial cloudimg:  https://cloud-images.ubuntu.com/xenial/current/xenial-server-cloudimg-amd64-disk1.img09:51
*** bcornec1 has joined #openstack-ironic09:52
milanmoshele, I see, but some setups might want to keep those unbridged and rather configure their dnsmasq to listen on more interfaces I guess09:52
phuongnhgood morning lucasagomes09:53
sambetts|afkvsaienk0: cool, thanks unfortunatly I need to stick with centos for downstream reasons, but I've built my own image with simple-init (glean) instead which is working :)09:53
*** sambetts|afk is now known as sambetts09:53
*** dtantsur|afk is now known as dtantsur09:54
dtantsurmorning Ironic09:54
milanmorning dtantsur! :)09:54
phuongnhmorning dtantsur09:54
*** e0ne has quit IRC09:55
milanmoshele, I'd like to avoid stating that "to support InfiniBand, you have to bridge your dnsmasq interface w/ your IB interfaces" unless absolutely necessary09:57
*** ccamacho|lunch is now known as ccamacho09:57
moshelemilan: they can work also on eth2.85 and it also will work (but for the inspector it don't matter)09:57
*** e0ne has joined #openstack-ironic09:57
dtantsurjroll, sorry, was on sickday (lucasagomes or milan were supposed to know it from an internal ML ;)09:58
milandtantsur, sry, /me irregular ML poller O:-)09:58
dtantsurheh :) when I don't see someone online, I usually check with these MLs09:59
milanmoshele, I don't think I understand09:59
* milan just hopes it's Monday today :D09:59
*** amotoki has quit IRC10:01
* milan -> coffe10:03
phuongnhlucasagomes: as your suggestion in the last ironic weekly meeting, I have worked with Nisha_Agarwal and she updated the spec with additional capabilities properties for iRMC.10:03
milanbbib10:03
phuongnhlucasagomes: If you have time, please help us to review it. Thanks10:03
phuongnhhttps://review.openstack.org/#/c/338138/10:03
patchbotpatch 338138 - ironic-specs - Add new capabilities to ironic10:03
openstackgerritNisha Agarwal proposed openstack/ironic: Enhance pxe_ilo driver to do inband-inspection  https://review.openstack.org/34357710:06
*** tuanluong has quit IRC10:07
lucasagomesphuongnh, will do10:07
*** yonglihe has quit IRC10:07
*** amotoki has joined #openstack-ironic10:08
*** aparnav has joined #openstack-ironic10:08
phuongnhthanks10:09
*** bcornec1 has quit IRC10:10
*** phuongnh has quit IRC10:10
*** hoangcx has quit IRC10:14
aarefievmorning all!10:16
*** MattMan has quit IRC10:17
*** MattMan has joined #openstack-ironic10:17
pas-hamorning ironicers :)10:22
openstackgerritTaku Izumi proposed openstack/ironic-python-agent: Add support for finalize_rescue  https://review.openstack.org/41735110:24
*** amotoki has quit IRC10:24
hwoarangyolanda: TheJulia: good morning. Do you have any thoughts on https://review.openstack.org/#/c/419838/ ? bifrost will break when jinja2 >= 2.9.5 is released10:25
patchbotpatch 419838 - bifrost - requirements.txt: Restrict Jinja2 version to < 2.910:25
hwoarangright now it only builds because we restrict 2.9.0-2.9.4 in global requirements due to a jinja2 bug10:26
yolandahwoarang, so that is managed through global requirements10:26
yolandayou need to change the requirements project10:27
hwoarangthat means that all openstack projects will have to use jinja2 < 2.910:27
hwoarangbecause that's what ansible-2.1.X requires in bifrost10:27
yolandaor fix bifrost to use newer jinja10:27
tiendc_dtantsur, hi, can we finish the driver composition reform this cycle?10:27
yolandaif more projects are affected, jinja will need to be pinned for sure10:27
hwoarangyolanda: it's not a bifrost issue. it's ansible10:27
*** udesale has quit IRC10:28
yolandathen it will be needed in global requirements yes10:28
yolandahwoarang, i'd check with the requirements team10:28
jrollmorning y'all10:29
hwoarangthanks yolanda.10:29
jrolldtantsur: no worries, hope you're feeling well :)10:29
*** zhangjl has quit IRC10:29
tiendc_morning jroll10:29
dtantsurhi aarefiev, pas-ha, hi and thanks jroll :)10:29
jrolldtantsur: could I bug you to look at this and the api patch at some point today, to make sure it isn't insane? https://review.openstack.org/#/c/419669/10:31
patchbotpatch 419669 - ironic - WIP: Add get_hardware_interface_info RPC method10:31
dtantsursure, will check it (a bit later)10:31
jrollI'm wondering if this should be a real versioned object instead, for one, because upgrades10:31
jrollthanks10:31
*** alineb has quit IRC10:40
*** Nisha_Agarwal has quit IRC10:40
*** dtantsur is now known as dtantsur|bbl10:42
openstackgerritMerged openstack/python-ironic-inspector-client: List introspection statuses support  https://review.openstack.org/40811610:42
jrollthis looks like an easy merge: https://review.openstack.org/#/c/408404/10:43
patchbotpatch 408404 - ironic - Move baremetal tempest config setting from devstack10:43
vdrokgood morning vsaienk0 milan sambetts dtantsur|bbl lucasagomes pas-ha and jroll !10:51
vdrokand morning ironic!10:51
*** SerenaFeng has quit IRC10:52
*** SerenaFeng has joined #openstack-ironic10:52
lucasagomeso/10:52
*** moshele has quit IRC10:53
lucasagomesmorning vdrok, jroll, dtantsur|bbl, tiendc_, yolanda, all10:53
yolandahi, morning10:53
tiendc_morning vdrok, lucasagomes, all10:53
*** moshele has joined #openstack-ironic10:54
jrollgreat read: https://jvns.ca/blog/good-questions/10:57
mat128morning10:57
openstackgerritMerged openstack/ironic: Allow to set min,max API microversion in tempest  https://review.openstack.org/41943011:00
openstackgerritMerged openstack/ironic: Allow to set min,max API microversion in tempest  https://review.openstack.org/41951611:03
*** alineb has joined #openstack-ironic11:06
*** Nisha_Agarwal has joined #openstack-ironic11:06
*** SerenaFeng has quit IRC11:14
*** SerenaFeng has joined #openstack-ironic11:14
*** SerenaFeng has quit IRC11:18
*** hw_wutianwei has quit IRC11:19
*** tiendc_ has quit IRC11:20
milanmorning aarefiev jroll mat128 pas-ha vdrok! :)11:27
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/42020311:27
vdrokmorning mat128 and yolanda11:27
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Add `latest` to baremetal-api-version  https://review.openstack.org/42065911:29
vdrokjroll: re https://github.com/openstack/ironic/blob/9088891ce72081684761e5cf54d3b3eabab0ca37/ironic/drivers/modules/network/flat.py#L72 do you think we need to do it now? I thought we could deal with cleaning/provisioning stuff afterwards11:29
vdrokI'm just not sure what is going to do the configuration on the ramdisk side11:30
sambettsvdrok: I think we still need that because the neutron port still needs to be bound right?11:31
jrollvdrok: yeah, I tried to think about it in my head, I'm really not sure11:31
jrolland that's a good point11:31
vdroksambetts: ah, so only for binding?11:31
jrollonmetal just pxe boots a single port on provisioning/cleaning network11:31
sambettsvdrok: yes thats all it does, it sets the binding host ID for the port, which triggers neutron to bind it11:32
sambettsvdrok: that used to happen on the nova side, because nova would set the host id11:32
sambettsvdrok: but now it doesn't, so we handle it on the Ironic side11:32
jrollactually, pxe ROMs typically can't boot on a bond, right?11:33
sambettsjroll: right, but switches can be configured to fall back11:34
sambettsto single mode if no LLDP packets are recieved11:35
jrollthis is too much for my brain this early :|11:35
sambettsso if you have a statically configured bond that supports fall back then you can still define it as a portgroup in Ironic11:35
sambettsand use the flat net driver11:35
jrollso... we need that code in the attach/detach portgroups patch yes/11:36
sambettsI think so11:36
*** yolanda has quit IRC11:36
*** yolanda has joined #openstack-ironic11:36
*** amotoki has joined #openstack-ironic11:37
sambettsotherwise neutron ports assigned to a portgroup using the flat net driver won't even attempt to get bound in Neutron11:37
jrollyeah11:37
aarefievmorning dtantsur, milan, jroll, sambetts11:37
milanaarefiev, hi! :)11:38
vdroksambetts: jroll what about this place then, when we do add cleaning network ? https://github.com/openstack/ironic/blob/master/ironic/common/neutron.py#L20711:40
vdrokin case of flat we were not doing that previously11:40
vdrokI remember having issues without it11:41
*** alineb- has joined #openstack-ironic11:41
sambettsvdrok: notice that for flat you have to use the nova host ID https://github.com/openstack/ironic/blob/9088891ce72081684761e5cf54d3b3eabab0ca37/ironic/drivers/modules/network/flat.py#L6811:41
*** bcornec1 has joined #openstack-ironic11:42
sambettsvdrok: otherwise the port fails to bind11:42
*** alineb has quit IRC11:42
*** alineb- is now known as alineb11:42
sambettsbecause *vomit* it binds it as non-baremetal vnic type :/11:42
sambettsideally we need to crete the "fake" ML2 driver to handle baremetal vnic type ports assigned to statically configured networks11:43
vdroksambetts: so just to set to nova host id if present and still ignore this otherwise?11:44
sambettsvdrok: yup, that just keeps the behaviour the same as when Nova used to do it11:44
vdrokok, will update the patch then, thanks!11:44
sambettsvdrok: awesome thanks D:11:45
sambetts:D! !!!11:45
vdrok:O11:45
sambetts:-P11:45
jrollthanks for figuring that out, guys :)11:46
*** bcornec1 has quit IRC11:46
openstackgerritMerged openstack/python-ironicclient: Extend VIF attach commands  https://review.openstack.org/41889411:47
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: [WIP] Serving ipxe conf  https://review.openstack.org/42067611:57
*** jkilpatr has joined #openstack-ironic12:01
*** dtantsur|bbl is now known as dtantsur12:11
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to use no nova installation  https://review.openstack.org/42068812:13
*** Nisha_ has joined #openstack-ironic12:15
*** Nisha_Agarwal has quit IRC12:19
*** zackf has joined #openstack-ironic12:26
*** krtaylor has quit IRC12:29
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to use no nova installation  https://review.openstack.org/42068812:30
openstackgerritVasyl Saienko proposed openstack/ironic: Use add_image_link devstack function.  https://review.openstack.org/42069212:30
*** zackf has quit IRC12:31
*** Nisha_ has quit IRC12:38
openstackgerritMerged openstack/ironic: Ipmitool power driver for soft reboot and soft power off  https://review.openstack.org/21673812:41
*** lucasagomes is now known as lucas-hungry12:45
openstackgerritVasyl Saienko proposed openstack/ironic: DNM: Test patch to verify bond  https://review.openstack.org/38866012:50
openstackgerritVasyl Saienko proposed openstack/ironic: Ironic devstack portgroup support  https://review.openstack.org/38174312:50
openstackgerritVasyl Saienko proposed openstack/ironic: Tempest tests for portgroups  https://review.openstack.org/38247612:50
karthiksmilan,   Regarding https://review.openstack.org/#/c/396147/, Oliver Walsh has concerns on not adding memory available per node and the naming conventions (NUMA instead of socket).  We have proposed a data structure accordingly in the comments section. Can you please let us know your thoughts on this.12:58
patchbotpatch 396147 - ironic-inspector-specs - Spec for retrieving NUMA node information12:58
mat128can't help but think about this video when reading NUMA ^ https://www.youtube.com/watch?v=0HR4hp_-kSI12:59
sambettsmat128: /me doesn't even need to click the link ...12:59
*** gcb has quit IRC13:01
mat128:P13:02
*** moshele has quit IRC13:03
milankarthiks, yup, will look into that later today I hope13:03
karthiksmat128, Nice song :)13:03
*** moshele has joined #openstack-ironic13:04
karthiksmilan, Thanks13:05
openstackgerritAparna proposed openstack/ironic: Doc for sanitize erase support in iLO drivers  https://review.openstack.org/40844113:07
*** ccamacho is now known as ccamacho|lunch13:10
*** SerenaFeng has joined #openstack-ironic13:11
*** bfournie has quit IRC13:11
*** cdaniel has joined #openstack-ironic13:12
*** aparnav has quit IRC13:13
*** trown|outtypewww is now known as trown13:14
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add RPC and object version pinning  https://review.openstack.org/40749113:20
*** amoralej is now known as amoralej|lunch13:21
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add version column  https://review.openstack.org/41239713:21
openstackgerritMerged openstack/ironic: Move baremetal tempest config setting from devstack  https://review.openstack.org/40840413:24
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add version column  https://review.openstack.org/41239713:28
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command with first online data migration  https://review.openstack.org/40855613:28
*** krtaylor has joined #openstack-ironic13:30
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Rolling upgrades related dev documentation  https://review.openstack.org/41943913:33
*** vsaienko has joined #openstack-ironic13:36
*** vsaienko has quit IRC13:36
openstackgerritVladyslav Drok proposed openstack/ironic: Allow to attach/detach VIF to portgroup  https://review.openstack.org/40691513:38
*** moshele has quit IRC13:39
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Rolling upgrades related dev documentation  https://review.openstack.org/41943913:40
*** mgoddard has joined #openstack-ironic13:41
openstackgerritMiles Gould proposed openstack/ironic: Refactor DRAC driver boot-device tests  https://review.openstack.org/42010613:42
openstackgerritMiles Gould proposed openstack/ironic: Allow setting UEFI boot devices in DRAC driver  https://review.openstack.org/42010713:42
*** bfournie has joined #openstack-ironic13:42
dtantsuraarefiev, would be interesting to hear background behind https://review.openstack.org/#/c/420676/ :)13:44
patchbotpatch 420676 - ironic-inspector - [WIP] Serving ipxe conf13:44
aarefievdtantsur: sure, I'll file a bug13:44
*** baoli has joined #openstack-ironic13:45
aarefievdtantsur: or you have objections already :)13:48
dtantsurno, just wondering13:48
*** moshele has joined #openstack-ironic13:48
aarefievok :)13:49
*** srobert has joined #openstack-ironic13:49
openstackgerritAnton Arefiev proposed openstack/ironic: Use global vars for storing image deploy path's  https://review.openstack.org/42072413:51
*** sacharya has quit IRC13:52
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Devstack: don't rely on Ironic local vars  https://review.openstack.org/42072513:53
*** lucas-hungry is now known as lucasagomes13:55
*** ppiela has joined #openstack-ironic13:58
*** dprince has joined #openstack-ironic13:59
*** baoli has quit IRC13:59
*** sasha2 has joined #openstack-ironic14:02
*** jcoufal has joined #openstack-ironic14:02
*** sasha21 has joined #openstack-ironic14:02
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add compatibility with Newton when creating a node  https://review.openstack.org/42072814:04
*** srobert has quit IRC14:04
*** srobert has joined #openstack-ironic14:04
openstackgerritVasyl Saienko proposed openstack/ironic: Ironic devstack portgroup support  https://review.openstack.org/38174314:06
openstackgerritVasyl Saienko proposed openstack/ironic: Tempest tests for portgroups  https://review.openstack.org/38247614:06
openstackgerritVasyl Saienko proposed openstack/ironic: DNM: Test patch to verify bond  https://review.openstack.org/38866014:06
*** sasha2 has quit IRC14:06
*** baoli has joined #openstack-ironic14:09
*** rbartal has quit IRC14:09
*** rloo has joined #openstack-ironic14:10
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add compatibility with Newton when creating a node  https://review.openstack.org/42072814:13
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: [PoC] Add fake fields in Node object and RPC  https://review.openstack.org/41546214:14
*** vsaienko has joined #openstack-ironic14:15
*** jheroux has joined #openstack-ironic14:17
*** ccamacho|lunch is now known as ccamacho14:18
*** jcoufal_ has joined #openstack-ironic14:18
openstackgerritJeffrey Zhang proposed openstack/bifrost: Need install ansible in source case  https://review.openstack.org/42073714:19
*** jcoufal has quit IRC14:20
*** rbartal has joined #openstack-ironic14:22
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: [PoC] Add fake fields in Node object and RPC  https://review.openstack.org/41546214:29
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add compatibility with Newton when creating a node  https://review.openstack.org/42072814:29
*** links has quit IRC14:30
*** SerenaFeng has quit IRC14:31
openstackgerritVasyl Saienko proposed openstack/ironic: DNM: Test patch to verify bond  https://review.openstack.org/38866014:32
openstackgerritVasyl Saienko proposed openstack/ironic: Ironic devstack portgroup support  https://review.openstack.org/38174314:32
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to attach/detach VIF to portgroup  https://review.openstack.org/40691514:32
openstackgerritVasyl Saienko proposed openstack/ironic: Tempest tests for portgroups  https://review.openstack.org/38247614:32
*** cdearborn has joined #openstack-ironic14:33
*** jcoufal has joined #openstack-ironic14:38
*** jcoufal_ has quit IRC14:39
*** rloo has quit IRC14:40
*** rloo has joined #openstack-ironic14:40
*** baoli has quit IRC14:47
*** baoli has joined #openstack-ironic14:47
*** gcb has joined #openstack-ironic14:50
openstackgerritMiles Gould proposed openstack/ironic: Refactor DRAC driver boot-device tests  https://review.openstack.org/42010614:50
openstackgerritMiles Gould proposed openstack/ironic: Allow setting UEFI boot devices in DRAC driver  https://review.openstack.org/42010714:50
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Functional test node remove  https://review.openstack.org/42074714:52
milanaarefiev, dtantsur quick patch to make my life easier w/ the attribute lookup ^ wdyt?14:53
dtantsur+214:54
TheJuliaGood morning14:55
milandtantsur, thx14:55
milanmorning TheJulia! :)14:55
*** amarao has joined #openstack-ironic14:56
amaraoHello. I'm trying to bring up baremetal server with neutron (newton), and cloud-init inside instance fails with ''Unknown network_data link type: %s' % link['type'])#012ValueError: Unknown network_data link type: unbound'. Is someone know anything about this? (google is deaf).14:56
*** dyasny has quit IRC14:56
*** dyasny has joined #openstack-ironic14:57
TheJuliaInteresting...14:58
TheJuliaamarao: is nova and ironic the same release?14:58
amaraoYes, newton.14:58
sambettsamarao: are you using Ironic's multi-tenant networking logic? or flat networks?14:59
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/42020414:59
amaraomultitenant network via neutron.14:59
amaraoHere instance metadata: https://gist.github.com/amarao/47c2c5db245dce3557e49fbe24705c1314:59
amarao*network part14:59
amaraovlan networks15:00
sambettsamarao: which Ironic network driver you are using?15:00
*** e0ne has quit IRC15:00
amaraodefault_network_interface=neutron15:00
*** vsaienko has quit IRC15:01
aarefievmilan: why only for functional?15:01
sambettsamarao: ok, do you have neutron configured with a ML2 driver that supports your switches and has support for the baremetal VNIC type15:01
*** mgould has joined #openstack-ironic15:01
mgouldhi everyone!15:01
milanaarefiev, the database bothers me there15:01
*** mtanino has joined #openstack-ironic15:01
milanmorning mgould! :)15:02
amarao@sambetts: Yes, I use generic_switch driver for ML2, and it switches interface on hardware switch between vlans as expected.15:02
mgouldqq: how do I get a node out of provision-state "error"? ironic node-delete isn't working15:02
amaraomgould: nova delete15:02
sambettsamarao: I'm surprised that Nova is injecting network_data.json into the server, we havn't written that code yet...15:02
milanaarefiev, I can put it to more if you like15:02
sambettsmgould: set-provision-set-manage15:03
sambettsmgould: set-provision-state manage *15:03
amaraoI boot instance like this: nova boot good --config-drive true --flavor bare-1 --image ubuntu-custom-6-devuser --key-name keyname15:03
*** zackf has joined #openstack-ironic15:03
mgouldsambetts: 'The requested action "manage" can not be performed on node "1952dc66-8921-4141-85d2-91e03e046081" while it is in state "error"' :-(15:03
mgouldISTR aarefiev or vdrok telling me the answer to this, but I've forgotten :-(15:04
sambettsmgould: set-provision-state deleted?15:04
mgouldaha!15:04
mgouldtyvm15:04
sambetts:)15:04
aarefievmilan: I didn't take a look where we are creating db record, but it would be nice to do this in one base class15:04
*** fxpester has quit IRC15:04
amaraosambetts: Is this a bug? I mean, should I search fix for cloud-init, or I should dig nova? Or Ironic?15:05
mgouldand of course that's what the state machine diagram was telling me, but I was too blinded by my own preconceptions to see it :-/15:05
vdrokamarao: the types supported by cloud init are 'bridge', 'ethernet', 'hw_veb', 'hyperv', 'ovs', 'phy', 'tap', 'vhostuser', 'vif', 'bond', 'vlan'15:05
vdrokat least that's what in master15:05
vdrokseems to be a bug, in nova's generate metadata maybe?15:06
vdrokmorning TheJulia and mgould15:06
mgouldmorning vdrok TheJulia sambetts milan amarao and everyone else15:07
milanaarefiev, https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/test/base.py#L178 and https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/test/functional.py#L103 cause the functional tests use sqlite15:07
amaraoThank you. I'll report this bug now and I'll try to debug it around nova/ironic/neutron interaction.15:08
rlooHappy Martin Luther King Jr. Day (US) and good morning ironickers amarao, milan, mgould, vdrok, TheJulia, sambetts, milan, aarefiev, dtantsur, xek, vsaienk0 :)15:10
rlooTheJulia: qq, what's the status with boot from volume? Done or awaiting more patches? https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:bug/155969115:10
vdrokmorning rloo !15:11
dtantsurmorning rloo15:11
vsaienk0morning rloo15:11
aarefievmilan: ahh, ok, we need to refactor this a little bit, this hierarchy is confusing (15:11
aarefievmorning rloo15:11
milanmorning rloo! :)15:12
TheJuliahmm.. multiple kvm failures on ovh test nodes causing jobs to fail over the last couple days.  Perhaps we need to turn off KVM for our test VMs.15:12
milanaarefiev, I hope not in this patch though ;)15:12
dtantsuralso morning TheJulia15:12
*** rbartal has quit IRC15:13
aarefievmilan: w+1, thanks :)15:13
milanaarefiev, thanks! :)15:13
*** e0ne has joined #openstack-ironic15:14
TheJuliarloo: logically, ironicclient + nova + devstack are the next things for basic iscsi booting, Once that is in place we will want to add the remaining features into boot from volume for additional scenarios based on configuration, as well as get some data collection into inspector for HBAs.15:14
rlooTheJulia: nice. Is it time to review/merge the REST API for volume* ?15:15
TheJuliarloo: I had the rest api patch moved until after cinder storage and ipxe template changes so we can test everything together.  We should begin reviewing it, and the client patches are going to need osc plugins as well since they they were submitted originally a very long time ago.  I rebased them Friday and marked them -1 since there is no osc client plugin15:17
TheJuliasupport in them.15:17
openstackgerritMerged openstack/ironic-inspector: Functional test node remove  https://review.openstack.org/42074715:18
*** Nisha_ has joined #openstack-ironic15:18
rlooTheJulia: ok, will try to look at those this week. I am guessing the nova-related changes won't make it for ocata?15:19
rlooTheJulia: i just realized, i could wait to ask this in the ironic meeting.15:19
dtantsurthe nova team told me they won't15:20
rlooTheJulia: so no need to answer now if you don't want to :)15:20
TheJuliayeah, definitely not and was never really my plan.  My hope was to get the substrate done in ironic this cycle, and try and get nova next cycle.15:20
rloodtantsur, TheJulia: that's what I thought. Thx. So not a big hurry to get the REST+client stuff done this week.15:20
openstackgerritAndrey Shestakov proposed openstack/bifrost: Allow user to specify private SSH key  https://review.openstack.org/42021015:20
TheJuliarloo: exactly :)15:21
openstackgerritAnton Arefiev proposed openstack/ironic: Use global vars for storing image deploy path's  https://review.openstack.org/42072415:21
dtantsurrloo, well, it's still important for stand-alone use case15:21
dtantsurrloo, we can't call a feature really done if we don't expose it in API at least15:21
sambettsamarao: we havn't finished the code for updating the network_data.json file for baremetal, we have patches for that, I have a feeling that something is being parsed poorly because in Ironic we bind the port later in the deploy process, post-config-drive generation15:22
rloodtantsur: yeah, will get it done in Ocata, just trying to think of my priorities for this week, but am not sure why I am asking now instead of at the meeting :)15:22
amaraosambetts: Thanks. Here bugreport: https://bugs.launchpad.net/ironic/+bug/165685415:22
openstackLaunchpad bug 1656854 in Ironic "Incorrect metada in ConfigDrive when using barematal ports under neutron" [Undecided,New]15:22
sambettsvsaienk0, vdrok: ^^15:22
* dtantsur whispers "driver composition" to rloo :D15:23
sambettsvsaienk0: seems like nova is generating the vifs as "unbound" because we bind them later in Ironic15:23
sambettsvsaienk0: will our patch fix that?15:23
dtantsurmgould, jroll, I can't make up my mind if https://bugs.launchpad.net/ironic/+bug/1656841 is a bug report or a feature request15:24
openstackLaunchpad bug 1656841 in Ironic "pxe_drac driver fails to boot nodes in UEFI mode" [Medium,In progress] - Assigned to Miles Gould (mgould)15:24
vdroksambetts: in case of portgroups, I had no issue with that and neutron network interface, flat seems to work too15:24
sambettsvdrok: with or without the nova metadata patch?15:24
vdrokwith it15:24
sambettsah great! :D15:24
vsaienk0sambetts: checking15:24
*** jaosorior has joined #openstack-ironic15:25
amaraoHas someone link to this 'patch'? I can test it in those settings.15:25
jrolldtantsur: do we assume UEFI works for drac?15:25
dtantsurjroll, this is the question :)15:26
*** beekneemech is now known as bnemec15:26
sambettsamarao: https://review.openstack.org/#/c/388756/15:26
patchbotpatch 388756 - nova - Generate necessary network metadata for ironic por...15:26
mgouldjroll: https://blueprints.launchpad.net/ironic/+spec/uefi-boot-for-ironic is marked as "complete"15:26
dtantsurjroll, I guess it boils down to whether we assume UEFI to work for all drivers15:26
jrolldtantsur: I had thought it did support it15:26
amaraomgould, thanks! I will try to use is for 'non port-group' ports and see if it fixes metadata.15:26
dtantsurjroll, then it's a bug15:27
mgouldamarao: you're welcome, but I think you meant to thank someone else :-)15:27
sambettsamarao: you may need to be careful as that comes with a caveat that it is based on a new API we added in Ocata :/15:27
jrolldtantsur: right15:27
vdrokamarao: what's the image you're booting instance with?15:27
amaraoUbuntu 16.04 with cloud-init source = ConfigDrive and enabled config drive at the boot time (command line)15:28
amaraosambetts, thanks.15:28
vdrokyeah, that should be fine.15:28
*** mkrai has joined #openstack-ironic15:29
dtantsurhmm, I'm trying to understand https://bugs.launchpad.net/ironic/+bug/1653068.. do you think it's about ironic-neutron notifications?15:30
openstackLaunchpad bug 1653068 in Ironic " ironic-conductor is not capable of verifying that the node is associated with port or not." [Undecided,New]15:30
mgoulddtantsur, jroll: cool, thanks15:30
vdrokbrb15:30
jrolldtantsur: I don't think it's about neutron ports at all15:31
jrolldtantsur: seems like this person has two problems: an extra query when there are no ports, and no logs if there are ports15:31
dtantsurI don't get the "ironic-conductor is not capable of verifying that the node is associated with port or not" bit15:32
dtantsurfor us, "associated" means ports.node_id =<node_id>15:32
jrollyeah, idgi15:33
*** ijw has joined #openstack-ironic15:34
jrollTheJulia: wow, we got lots of BFV stuff done O_O15:36
jrollamazing15:36
NobodyCamGood Morning Ironic'ers15:36
jroll\o NobodyCam15:36
mgouldmorning NobodyCam15:37
NobodyCamo/15:37
TheJuliajroll: I don't feel like it, but I'll take your word for it ;)15:37
*** amoralej|lunch is now known as amoralej15:37
NobodyCammorning jroll mgould TheJulia :)15:37
jrollTheJulia: I see an empty gerrit topic :)15:37
* jroll finds the other one15:37
TheJuliaoh hell15:37
jrolloh, there's still this: https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:bug/152623115:37
jrollok, lots to do :)15:38
TheJuliayeah15:38
TheJuliatopic got reset by the api patch I guess15:38
* TheJulia goes and fixes the topic15:38
jrolloh it's fine15:38
jrollI know there's two topics15:38
*** ijw has quit IRC15:39
TheJuliaAll better now15:40
vsaienk0sambetts: yeah, nova generates configdrive data at time when tenant port is not bound. In ironic we will bound tenant port in the end of deployment. Our patches to nova with attach/detach https://review.openstack.org/#/c/388756/ https://review.openstack.org/#/c/364413/ will not fix that15:40
patchbotpatch 388756 - nova - Generate necessary network metadata for ironic por...15:40
patchbotpatch 364413 - nova - Support Ironic interface attach/detach in nova virt15:40
sambettsvsaienk0: we change the link type to ethernet though so it should be fine right??15:41
vsaienk0sambetts: you mean to phy? https://review.openstack.org/#/c/388756/11/nova/virt/ironic/driver.py@70915:43
patchbotpatch 388756 - nova - Generate necessary network metadata for ironic por...15:43
sambettsvsaienk0: yup thats what I meant15:43
sambettssorry15:43
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: [WIP] ETAGs for node in ironicclient  https://review.openstack.org/40033515:45
amaraosambetts: If I write a patch to fix this case, is any chance it will land in Newton?15:46
mgoulddtantsur, jroll: I think https://bugs.launchpad.net/ironic/+bug/1653068 is complaining that the DELETE query is issued even when there's nothing to delete15:46
openstackLaunchpad bug 1653068 in Ironic " ironic-conductor is not capable of verifying that the node is associated with port or not." [Undecided,Incomplete]15:46
mgouldto which the answer is "so what?"15:46
dtantsurmgould, hmm, not the biggest problem ever, to be honest15:46
mgouldyeah15:46
mgould"show me benchmarking info indicating that this is a real problem, then we'll think about fixing it"15:46
sambettsamarao: newton has already been released, and this fix would need to go into nova, I would check their backport policy15:47
amaraosambetts: And if you know, what kind of port should it be for 'normal baremetal port'? 'phy' or 'ethernet'?15:47
sambettsphy15:48
amaraoThnx.15:48
mgoulddtantsur: ah, I think I get it15:48
NobodyCammorning dtantsur :)15:49
mgouldKiran wants Ironic to (a) list all the ports on the node, (b) log that it's deleting them, (c) issue the DELETE query iff there's anything to delete15:49
mgould(b) might be useful, I guess15:49
mgouldIANAO15:49
vsaienk0sambetts: so nova will pick value from Neutron port https://github.com/openstack/nova/blob/4f91ed3a547965ed96a22520edcfb783e7936e95/nova/virt/netutils.py#L251 it network metadata is not passed to InstanceMetadata constructor https://review.openstack.org/#/c/364413/18/nova/virt/ironic/driver.py@71415:50
patchbotpatch 364413 - nova - Support Ironic interface attach/detach in nova virt15:50
sambettsmgould: surely we'll end up with 2 queries to the DB then? a SELECT then a DELETE ? instead of just a DELETE15:50
mgouldyep15:50
dtantsurvsaienk0, FYI https://bugs.launchpad.net/ironic/+bug/1655676 (I can fix it later if you're too busy)15:50
openstackLaunchpad bug 1655676 in Ironic "[stable only] TypeError: %o format: a number is required, not str in neutron errors handling" [Medium,Triaged]15:50
mgouldand it would introduce a race condition15:51
* mgould votes for WONTFIX15:51
sambettsvsaienk0: so once we add our patches all that gets resolved any way?15:51
dtantsurjroll, is https://bugs.launchpad.net/ironic/+bug/1655888 done now?15:52
openstackLaunchpad bug 1655888 in Ironic "tempest plugin runs tests for new API endpoints against stable releases" [High,In progress]15:52
jrolldtantsur: yes, I believe so15:52
jrollthat's why stable was broken15:52
* dtantsur marks as fix released15:53
openstackgerritMerged openstack/bifrost: Don't assume 'dhcp' if node_network_data set  https://review.openstack.org/42005915:54
*** rpioso has joined #openstack-ironic15:57
*** amotoki has quit IRC15:57
bfourniemorning ironic, hi sambetts - I know you’re probably swamped, but if you have any time to look at the lldp patches, plugin - https://review.openstack.org/#/c/406496/, and UI - https://review.openstack.org/#/c/419141, would appreciate it, thank you16:00
patchbotpatch 406496 - ironic-inspector - Add plugin to process basic management LLDP TLVs16:00
vsaienk0sambetts: so I've just tested we need to add type: phy here https://review.openstack.org/#/c/364413/18/nova/virt/ironic/driver.py@692 and it fixed bug https://bugs.launchpad.net/ironic/+bug/165685416:00
openstackLaunchpad bug 1656854 in OpenStack Compute (nova) "Incorrect metada in ConfigDrive when using barematal ports under neutron" [Undecided,New]16:00
patchbotpatch 419141 - python-ironic-inspector-client - Add new introspection commands for interface data ...16:00
patchbotpatch 364413 - nova - Support Ironic interface attach/detach in nova virt16:00
vsaienk0sambetts: going to reupload that patch16:00
sambettsvsaienk0: +116:00
sambettsvsaienk0: thanks :D16:01
TheJuliais review hanging for anyone else?16:05
TheJuliaheh, I ask and it finally loads after two minutes16:05
*** phuongnh has joined #openstack-ironic16:08
*** moshele has quit IRC16:08
openstackgerritVasyl Saienko proposed openstack/ironic: Trivial: fix warning message formatting  https://review.openstack.org/42078916:09
openstackgerritVasyl Saienko proposed openstack/ironic: Trivial: fix warning message formatting  https://review.openstack.org/42078916:09
vsaienk0dtantsur: https://review.openstack.org/#/c/420789/16:10
patchbotpatch 420789 - ironic (stable/newton) - Trivial: fix warning message formatting16:10
dtantsurthanks!16:10
openstackgerritMerged openstack/python-ironicclient: Replace yaml.load() with yaml.safe_load()  https://review.openstack.org/42053716:11
*** rama_y has joined #openstack-ironic16:13
*** rbudden has joined #openstack-ironic16:14
*** Nisha_ has quit IRC16:17
openstackgerritMerged openstack/ironic: Remove check for UEFI + Whole disk images  https://review.openstack.org/41078416:21
NobodyCamlooks like I have a conflict with this morning meeting :(16:27
vsaienk0sambetts, jroll: I've just uploaded a new version of https://review.openstack.org/#/c/364413/16:32
patchbotpatch 364413 - nova - Support Ironic interface attach/detach in nova virt16:32
rloovsaienk0: i've started reviewing ^^ 3? times now. Please let me know when you have a good revision up.16:33
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Add `latest` to baremetal-api-version  https://review.openstack.org/42065916:34
vsaienk0rloo: the latest version is good16:34
rloovsaienk0: thx. grumble, grumble... :)16:34
vsaienk0rloo: sorry for spam...16:35
*** ohamada has quit IRC16:35
openstackgerritVasyl Saienko proposed openstack/ironic: Tempest tests for portgroups  https://review.openstack.org/38247616:37
openstackgerritVasyl Saienko proposed openstack/ironic: DNM: Test patch to verify bond  https://review.openstack.org/38866016:37
jrollthanks vsaienk016:38
*** Nisha_ has joined #openstack-ironic16:40
openstackgerritOpenStack Proposal Bot proposed openstack/bifrost: Updated from global requirements  https://review.openstack.org/41933816:40
*** gcb has quit IRC16:42
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent  https://review.openstack.org/40742916:44
vsaienk0sambetts: I think we shouldn't call network_driver.validate() for vif_list, vif_attach, vif_detach https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L238016:44
*** phuongnh has quit IRC16:44
*** e0ne has quit IRC16:45
sambettsvsaienk0: why? We need to make sure we have the options we need in vif_attach/detach etc16:45
vsaienk0sambetts: User shouldn't receive configuration errors like provisioning network is not present when perform vif_* API call https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/network/neutron.py#L6316:46
vsaienk0sambetts: we can introduce a new method that will check that all options needed for attach/detach are set, also for neutron we seems don't have such options16:47
vsaienk0sambetts: I'm trying to solve local_link_info field validation before starting deploy and hit this issue: https://review.openstack.org/#/c/415881/16:47
patchbotpatch 415881 - ironic - Validate local_link_connection before start deploy16:47
sambettsvsaienk0: So too things, 1. I think if we are failing deploy because those are missing then we should also fail vif_attach if those are missing 2. I don't think we should be failing deploy or vif_attach if that information is missing on a port we should just ignore it on vif_attach16:51
sambettss/too/two16:51
sambettsjroll: WDYT? ^16:52
vsaienk0sambetts: that is fear but still do you think it is ok that user will get error that provision/cleaning networks are not configured when doing vif_* stuff?16:53
jrollsambetts: I'll come back to this after the meeting16:55
vsaienk0sambetts: it is good idea to ignore ports that are miss configured when attaching interfaces to them16:55
*** amotoki has joined #openstack-ironic16:57
*** hshiina has joined #openstack-ironic17:00
vdrokvsaienk0: otoh, if we don't have provisioning network, provisioning will fail anyway, why bother attaching? same for cleaning, if we won't able to do cleaning, maybe it's better to forbid attaching too?17:01
*** dprince has quit IRC17:04
*** dprince has joined #openstack-ironic17:05
*** jcoufal_ has joined #openstack-ironic17:07
vsaienk0vdrok: provisioning/cleaning is a global thing not related to a port. Why not allow to attach/detach/list vif, and fail when user start deployment?17:07
*** jcoufal has quit IRC17:09
vdrokvsaienk0: if it is considered as just node-update (or getting a specific node field, as in case of vif_list), as it is now, then yes, in more generic case I'd say the validation needs to happen17:15
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/42083517:19
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/42083617:19
vsaienk0jroll, sambetts: latest version of patches portgroup tests are passed: http://logs.openstack.org/60/388660/37/check/gate-tempest-dsvm-ironic-multitenant-network-ubuntu-xenial/aca83ca/console.html#_2017-01-16_17_13_15_18074817:26
jrollok17:26
* jroll in a meeting :P17:26
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/42085517:27
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/42085617:27
*** ricardoas has joined #openstack-ironic17:28
*** tesseract has quit IRC17:29
*** hshiina has quit IRC17:30
*** phuongnh has joined #openstack-ironic17:32
openstackgerritMiles Gould proposed openstack/ironic: Allow setting UEFI boot devices in DRAC driver  https://review.openstack.org/42010717:32
*** moshele has joined #openstack-ironic17:35
*** Michael-zte2 has joined #openstack-ironic17:38
* jroll goes to lunch17:38
* sambetts -> home 17:39
TheJuliaMichael-zte2: So you indicated specific vlans for deployment. Can you explain to us why you can't just use a single vlan for node deployment and then move the node to the tenant network?17:39
* sambetts hangs around for a moment to see17:39
TheJuliasambetts: sorry!17:39
sambetts;)17:40
* TheJulia feels bad, should have waited oh... 10 minutes17:40
*** phuongnh has quit IRC17:40
Michael-zte2<TheJulia>: Net flip is one possible solution to our problem17:40
TheJuliaMichael-zte2: So the neutron driver could fit the needs of your deployment?17:41
sambettsThis RFE also covers customisable provisoning vlans per node too https://bugs.launchpad.net/ironic/+bug/161487617:42
openstackLaunchpad bug 1614876 in Ironic "[RFE] Allow setting {provisioning,cleaning}_network_uuid in node driver_info" [Wishlist,Confirmed]17:42
TheJuliasambetts: thanks!17:42
lucasagomesfolks I'm calling it a day17:42
lucasagomeshave a great evening all!17:42
Michael-zte2There are not enough neutron drivers to support all the switcher now17:42
TheJuliagoodnight lucasagomes17:42
vdrokgood night lucasagomes17:43
lucasagomeso/17:43
lucasagomessee ya17:43
*** lucasagomes is now known as lucas-afk17:43
TheJuliaMichael-zte2: I'm interpreting "all the switcher" as "all of the switches", presently in your environment.  Or do you mean something else?17:43
*** athomas has quit IRC17:44
vdrokjroll: question on resource providers. was it decided how/if to register the resource provider in nova after node creation?17:44
Michael-zte2<TheJulia> : I typed wrongly17:45
Michael-zte2sorry17:45
vsaienk0Michael-zte2:  have you checked https://github.com/openstack/networking-generic-switch, it is based on netmiko and potentially may support any of https://github.com/ktbyers/netmiko#supports17:45
vdrokI mean, is it going to happen manually, or we'll add some hook in ironic, or resource tracker will do that? I see that resource classes are created by resource tracker17:45
jrollvdrok: resource tracker does it17:45
jrollbased on node.resource_class17:45
TheJuliaMichael-zte2: No worries, just trying to understand :)17:45
vdrokjroll: aha, ok17:45
jrollvdrok: which reminds me I need to bring this back up https://review.openstack.org/#/c/345081/17:46
patchbotpatch 345081 - ironic - Add docs for node.resource_class and flavor creation17:46
* jroll will do that this afternoon17:46
vdrokthanks!17:46
Michael-zte2<vsaienk0> : Our customers have more switches than the list17:47
Michael-zte2https://github.com/ktbyers/netmiko#supports17:48
*** derekh has quit IRC17:48
Michael-zte2But thank you for your infos  :)17:49
*** trown is now known as trown|lunch17:49
TheJuliaMichael-zte2: but the root of the need to so have isolated infrastructure for some number of conductors because those nodes will need to use a flat network since there may not be a neutron driver.17:49
Michael-zte2<TheJulia> : https://wiki.openstack.org/wiki/Meetings/Ironic-neutron17:52
Michael-zte2This program has a lot to do.17:52
TheJuliaMichael-zte2: so that meeting has been suspended17:52
TheJuliaMichael-zte2: largely because they view their work as done17:53
*** dprince has quit IRC17:53
Michael-zte2https://etherpad.openstack.org/p/YVR-neutron-ironic17:53
Michael-zte2<TheJulia>:17:53
Michael-zte2<TheJulia> : We try to be able to have a faster solution17:54
openstackgerritMerged openstack/bifrost: Updated from global requirements  https://review.openstack.org/41933817:54
Michael-zte2<TheJulia> : Thank you for your answer.17:56
*** dprince has joined #openstack-ironic17:56
sambettsMichael-zte2: we have landed the code to implement the network flip in Newton, the work to support portgroups is targets to land in Ocata and trunk support targeted at Pike17:57
*** e0ne has joined #openstack-ironic17:58
*** mgoddard has quit IRC17:58
Michael-zte2<TheJulia><sambetts> : It is  2:00 am here. I must go to bed now.  Thank you for your patience and your information. :)17:59
sambettsnp17:59
TheJuliaMichael-zte2: No problem, rest well17:59
* sambetts -> home now 17:59
sambettsNight all o/18:00
mgouldo/18:00
*** mgould is now known as mgould|afk18:00
*** sambetts is now known as sambetts|afk18:00
*** ralonsoh has quit IRC18:01
dtantsurg'night18:01
*** Michael-zte2 has quit IRC18:01
*** dtantsur is now known as dtantsur|afk18:01
*** rpioso is now known as rpioso|afk18:01
*** clenimar has joined #openstack-ironic18:02
*** mjura has quit IRC18:03
*** jcoufal_ has quit IRC18:03
*** jcoufal has joined #openstack-ironic18:05
*** rloo has quit IRC18:17
*** e0ne has quit IRC18:19
openstackgerritZhenguo Niu proposed openstack/ironic: Add nodes tagging support - objs  https://review.openstack.org/23335718:19
*** jkilpatr has quit IRC18:23
vdrokgood night sambetts|afk mgould|afk and dtantsur|afk18:24
*** rloo has joined #openstack-ironic18:25
*** milan has quit IRC18:28
*** g3ek has quit IRC18:33
*** mgoddard has joined #openstack-ironic18:34
cdanielGood night :)18:34
*** amoralej is now known as amoralej|off18:36
*** dprince has quit IRC18:36
*** jkilpatr has joined #openstack-ironic18:37
cdanielCould you review again this patch https://review.openstack.org/#/c/372762/ please? TheJulia vdrok Thank you.18:40
patchbotpatch 372762 - ironic - Ensures that OneView nodes are free for use by Ironic18:40
vdrokcdaniel: ok, will do18:40
TheJuliacdaniel: sure18:40
*** g3ek has joined #openstack-ironic18:42
mariojvxek: hey, i had 1 comment on the rolling upgrades patch set about how the change interacts with notifications: https://review.openstack.org/#/c/40749118:47
patchbotpatch 407491 - ironic - Add RPC and object version pinning18:47
*** e0ne has joined #openstack-ironic18:47
mariojvi think just removing them from release_mappings.py is OK, but i wanted to verify my understanding18:47
mariojvi'm heading out in a few minutes, but i'll be around tomorrow to discuss if needed18:47
*** sacharya has joined #openstack-ironic18:54
*** mrtenio has joined #openstack-ironic18:55
*** dprince has joined #openstack-ironic19:04
*** ijw has joined #openstack-ironic19:05
*** ijw has quit IRC19:05
*** ijw has joined #openstack-ironic19:05
*** moshele has quit IRC19:06
*** ijw_ has joined #openstack-ironic19:12
*** trown|lunch is now known as trown19:13
*** ijw has quit IRC19:15
*** joanna has joined #openstack-ironic19:19
openstackgerritMerged openstack/ironic: Refactor DRAC driver boot-device tests  https://review.openstack.org/42010619:23
cdanielThanks vdrok, TheJulia! : )19:25
*** ijw has joined #openstack-ironic19:27
*** mgoddard has quit IRC19:28
*** ijw__ has joined #openstack-ironic19:30
*** ijw_ has quit IRC19:30
openstackgerritJim Rollenhagen proposed openstack/ironic: Move to tooz hash ring implementation  https://review.openstack.org/39902819:30
*** ijw has quit IRC19:34
*** jkilpatr has quit IRC19:36
*** jkilpatr has joined #openstack-ironic19:37
openstackgerritMerged openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/42085619:37
*** moshele has joined #openstack-ironic19:44
*** Nisha_ has quit IRC19:44
*** dsneddon_ is now known as dsneddon19:45
vdrokgood night everyone!19:50
*** ElCoyote_ has quit IRC19:51
*** ElCoyote_ has joined #openstack-ironic19:51
joannanight, vdrok :)19:51
*** rpioso has joined #openstack-ironic19:54
*** rpioso|afk has quit IRC19:55
mrtenioGood night19:57
*** jerrygb has joined #openstack-ironic19:57
*** cfarquhar has joined #openstack-ironic19:58
*** cdaniel is now known as cdaniel-afk19:58
TheJuliagoodnight20:06
*** moshele has quit IRC20:13
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/42083520:16
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/42020620:16
*** jkilpatr has quit IRC20:18
*** Syed__ has joined #openstack-ironic20:29
*** jkilpatr has joined #openstack-ironic20:30
*** ijw has joined #openstack-ironic20:31
*** rbudden has quit IRC20:31
*** ijw__ has quit IRC20:34
*** jcoufal has quit IRC20:50
jrollrloo: hm, I wonder if we should really wait for the deadline to branch21:00
jroll* I will branch the requirements repository shortly after all of the cycle-with-milestone projects have branched. After the requirements repository is branched and the master requirements list is opened, projects that have not branched will be tested with Pike requirements as the requirements master branch advances and stable/ocata stays stable. Waiting too long to create the stable/ocata branch may21:00
jrollresult in broken CI jobs in either stable/ocata or master. Don't delay any further than necessary.21:00
jrollit might be fine but we might end up with broken stable/ocata CI for a bit21:01
rloojroll: sigh.21:01
jrollI suspect that will be tagged jan 30ish21:01
rloojroll: i don't think we want broken CI jobs21:01
jrollwell, it's a *might*21:02
jrolle.g. if a dependency is released, and our tests work with the new but not the old21:02
jrollor something21:02
rloojroll: well, much as i'd like to get 'everything' or 'as much as possible', maybe we should be pessimistic and set an earlier target date for ocata.21:02
jrollseems like it would be hard to get hit21:02
jrollrloo: yeah, let's see how we look on rc1 week21:02
rloojroll: i think we should mention to folks that there is a high probability that we cut a branch on rc1. otherwise, you know, procrastination...21:03
rloojroll: i'm worried now that rolling upgrades may not make it. or driver composition. will have a better idea at end of week i guess.21:04
jrollrloo: we shall see21:04
jrollI'll talk to dmitry about release date too, see what he thinks21:05
rloojroll: ok, thx.21:05
jrollnp21:05
TheJuliaFun :(21:05
rlooTheJulia: yeah. the same fun we have towards the end of every release :)21:06
TheJuliaTrue21:06
jrollreleasing is the worst21:06
TheJuliaGood scotch, tequila, or whisky?21:09
TheJuliaand ice cube molds to make death star like ice cubes?21:09
jrollI tend towards bourbon if I'm going to need more than a few21:09
jrollshots for releasing21:09
rloojroll: shhh, don't say that in public. Don't we want someone to volunteer to do releases? :)21:09
TheJulialol21:09
jrollgood scotch to celebrate when its over21:09
jrollrloo: dunno, it's the PTL job by default, so after this I am freeeee21:10
rloojroll: unless no one volunteers to be ptl :D21:10
jrollnot my problem!21:10
jroll:P21:10
TheJuliaheh21:11
rloojroll: i thought if no one ran, you were still stuck with it. but maybe it is more like 'disband project' if no one wants the job!21:11
jrollheh21:11
jrollrloo: I suspect it's more like, if nobody runs, someone will step up at the last minute that shouldn't be21:12
jrollso it's in our interests for someone to run :)21:12
rloojroll: ok. Folks, are you all listening? :D21:12
jroll:D21:12
* TheJulia hears loud and clear21:15
openstackgerritMerged openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/42085521:18
openstackgerritMerged openstack/ironic: Allow to attach/detach VIF to portgroup  https://review.openstack.org/40691521:18
rlooyay! ^^^21:21
jroll:)21:33
*** vsaienko has joined #openstack-ironic21:35
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/42083521:44
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914121:45
*** absubram_ has joined #openstack-ironic21:46
* TheJulia really wonders why proposal bot tries to rebase needlessly.....21:55
TheJuliaoh, that was a stupid thought21:56
*** trown is now known as trown|outtypewww21:56
*** dprince has quit IRC21:57
*** absubram_ has quit IRC22:00
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914122:07
*** [1]rpioso has joined #openstack-ironic22:10
*** rpioso has quit IRC22:14
*** bfournie has quit IRC22:22
*** jheroux has quit IRC22:22
openstackgerritMerged openstack/ironic: Ensures that OneView nodes are free for use by Ironic  https://review.openstack.org/37276222:32
*** wrouesnel has joined #openstack-ironic22:34
*** jerrygb has quit IRC22:34
*** srobert_ has joined #openstack-ironic22:39
*** jaosorior has quit IRC22:41
*** srobert has quit IRC22:43
*** srobert_ has quit IRC22:44
*** ppiela has quit IRC22:51
*** bfournie has joined #openstack-ironic22:52
*** bfournie has quit IRC22:53
*** bfournie has joined #openstack-ironic22:53
*** bfournie has quit IRC22:54
*** bfournie has joined #openstack-ironic22:55
*** jkilpatr has quit IRC22:57
*** bcornec1 has joined #openstack-ironic22:59
*** vsaienko has quit IRC23:04
*** krtaylor has quit IRC23:06
*** cdearborn has quit IRC23:07
*** jkilpatr has joined #openstack-ironic23:13
*** rloo has quit IRC23:19
*** jerrygb has joined #openstack-ironic23:22
*** jerrygb has quit IRC23:26
*** jerrygb has joined #openstack-ironic23:34
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/42083623:42
*** rbudden has joined #openstack-ironic23:43
*** cdearborn has joined #openstack-ironic23:55
« Sunday, 2017-01-15 Index Tuesday, 2017-01-17 »

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