Wednesday, 2017-01-25

*** awiddersheim has joined #openstack-ironic00:05
*** alexpilotti has joined #openstack-ironic00:09
*** alexpilotti has quit IRC00:15
*** Guest9757 is now known as rook00:21
openstackgerritJoanna Taryma proposed openstack/ironic: WIP host ipxe static assets in devstack on :80  https://review.openstack.org/42437400:24
openstackgerritSolio Sarabia proposed openstack/ironic-tempest-plugin: WIP/DNM: Migrate ironic-tempest-plugin to new repo  https://review.openstack.org/41960200:26
*** mtanino has joined #openstack-ironic00:26
*** rpioso has quit IRC00:27
*** mtanino_ has quit IRC00:28
*** mtanino has quit IRC00:32
TheJuliaJayF: I just +2+A'ed the IPA rev00:32
*** aNuposic has quit IRC00:33
*** hoangcx has joined #openstack-ironic00:39
*** alexpilotti has joined #openstack-ironic00:41
openstackgerritSolio Sarabia proposed openstack/ironic-tempest-plugin: WIP/DNM: Migrate ironic-tempest-plugin to new repo  https://review.openstack.org/41960200:42
*** hw_wutianwei has joined #openstack-ironic00:43
*** yolanda has quit IRC00:45
TheJuliavinsh: I used to just comment out the continue_deploy final step in the conductor code when I was troubleshooting a odd deployment failure.00:49
*** amotoki has joined #openstack-ironic00:49
*** yolanda has joined #openstack-ironic00:51
*** yolanda has quit IRC00:53
*** aNuposic has joined #openstack-ironic00:53
vinshI turned on that log gathering for failed deploys, good feature00:54
vinshI'm running the api/conductor in a container so I can keep closer to master then ubuntu packages.   It would be do-able to modify the conductor code like you suggest if it comes down to it :)00:55
vinshI just realized that ironic sort of assumes that your glance is backed by swift.. or if its not it glance backed by rados00:56
vinshwe have glance backed by rbd00:56
vinshso. back to fix that up.00:56
TheJuliafun :\  Yeah, agent drivers do expect swift backed00:58
*** yolanda has joined #openstack-ironic01:00
*** SerenaFeng has joined #openstack-ironic01:01
*** phuongnh has joined #openstack-ironic01:01
tovin07JayF, hello01:03
tovin07rloo, hello01:04
*** yolanda has quit IRC01:04
*** vinsh has quit IRC01:06
*** rajinir has quit IRC01:06
*** ChubYann has quit IRC01:13
*** Syed__ has quit IRC01:15
openstackgerritMerged openstack/ironic-python-agent: Support psutil 5.0.1  https://review.openstack.org/42489501:21
*** rama_y has quit IRC01:22
*** mtanino has joined #openstack-ironic01:25
*** ChubYann has joined #openstack-ironic01:25
TheJuliaUpdated the etherpad01:36
*** sayantani01 has joined #openstack-ironic01:37
*** sayantani01 has quit IRC01:37
*** sayantani01 has joined #openstack-ironic01:37
*** aNuposic has quit IRC01:44
*** Sukhdev has quit IRC01:45
*** rloo has quit IRC01:47
*** mtanino has quit IRC02:12
*** sacharya has joined #openstack-ironic02:15
*** SerenaFeng has quit IRC02:54
*** SerenaFeng has joined #openstack-ironic02:54
*** ijw has quit IRC03:21
*** nmathew has joined #openstack-ironic03:23
openstackgerritShivanand Tendulker proposed openstack/ironic: Update proliantutils version required for Ocata release  https://review.openstack.org/42459303:27
*** vikrant has joined #openstack-ironic03:32
*** udesale has joined #openstack-ironic03:33
openstackgerritNisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers  https://review.openstack.org/37242803:38
*** gcb has joined #openstack-ironic03:38
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914103:39
openstackgerritDao Cong Tien proposed openstack/ironic: Adds clean step 'restore_bios_config' to iRMC drivers  https://review.openstack.org/41640303:40
*** ijw has joined #openstack-ironic03:42
*** vinsh has joined #openstack-ironic03:42
*** vinsh has quit IRC03:43
openstackgerritZhenguo Niu proposed openstack/ironic: Add nodes tagging support - objs  https://review.openstack.org/23335703:44
*** bfournie has quit IRC03:45
openstackgerritZhenguo Niu proposed openstack/ironic: Add DB layer support for node tags filter  https://review.openstack.org/27262403:46
openstackgerritZhenguo Niu proposed openstack/ironic: Add nodes tagging support - API  https://review.openstack.org/25047803:46
*** ijw has quit IRC03:46
*** SerenaFeng has quit IRC03:57
*** tovin07 has quit IRC04:02
*** bfournie has joined #openstack-ironic04:18
*** bfournie has quit IRC04:20
*** Sukhdev has joined #openstack-ironic04:37
*** sasha2 has quit IRC04:42
*** gcb has quit IRC04:43
*** gcb has joined #openstack-ironic04:44
*** rbudden has quit IRC04:58
*** udesale__ has joined #openstack-ironic05:17
*** udesale has quit IRC05:17
*** udesale has joined #openstack-ironic05:19
*** udesale__ has quit IRC05:21
openstackgerritMerged openstack/python-ironicclient: Strip endpoint version in OSC plugin  https://review.openstack.org/36483305:31
*** zackf has quit IRC05:37
*** pester has quit IRC05:45
openstackgerritMerged openstack/python-ironicclient: Add --wait to OSC provisioning commands  https://review.openstack.org/35480605:50
openstackgerritMerged openstack/python-ironicclient: Extend OSC "node list" cmd to fetch nodes without instance UUID  https://review.openstack.org/36927905:50
*** sacharya has quit IRC05:51
*** SerenaFeng has joined #openstack-ironic05:52
openstackgerritMerged openstack/bifrost: Fix endpoint service name for inspector  https://review.openstack.org/42422706:00
openstackgerritKarthik S proposed openstack/ironic-python-agent: WIP: NUMA-topology collector  https://review.openstack.org/42472906:13
*** slagle has quit IRC06:15
*** SerenaFeng has quit IRC06:15
*** xavierr has quit IRC06:15
*** xavierr has joined #openstack-ironic06:16
*** slagle has joined #openstack-ironic06:17
openstackgerritDao Cong Tien proposed openstack/ironic: Adds clean step 'restore_bios_config' to iRMC drivers  https://review.openstack.org/41640306:28
*** zackf has joined #openstack-ironic06:28
*** dsneddon has quit IRC06:31
*** zackf has quit IRC06:32
*** noama has joined #openstack-ironic06:37
*** Sukhdev has quit IRC06:39
*** ChubYann has quit IRC06:47
*** mjura has joined #openstack-ironic06:48
*** Nisha_Agarwal has joined #openstack-ironic06:50
*** yolanda has joined #openstack-ironic06:51
*** makowals_ has quit IRC06:53
*** tovin07 has joined #openstack-ironic06:54
*** udesale__ has joined #openstack-ironic06:56
*** hoangcx_ has joined #openstack-ironic06:57
*** hoangcx has quit IRC07:00
*** makowals has joined #openstack-ironic07:06
*** aparnav has joined #openstack-ironic07:07
*** nmathew- has joined #openstack-ironic07:09
aparnavianw, Can you review the patch https://review.openstack.org/#/c/396504/ ? It is there for a long time.07:11
patchbotpatch 396504 - diskimage-builder - Update hpssacli to ssacli in proliant-tools element07:11
*** nmathew has quit IRC07:11
*** nmathew has joined #openstack-ironic07:12
*** nmathew- has quit IRC07:14
*** pcaruana has joined #openstack-ironic07:28
openstackgerritjiang wei proposed openstack/python-ironicclient: Support i18n for baremetal node cmds  https://review.openstack.org/35763907:29
*** tovin07 has quit IRC07:32
*** nmathew- has joined #openstack-ironic07:33
*** tesseract has joined #openstack-ironic07:34
*** nmathew has quit IRC07:35
*** sayantan_ has joined #openstack-ironic07:36
*** sayantani01 has quit IRC07:38
*** mgoddard has joined #openstack-ironic07:39
*** tovin07 has joined #openstack-ironic07:45
*** hoangcx has joined #openstack-ironic07:46
openstackgerritjiang wei proposed openstack/python-ironicclient: Support i18n for baremetal node cmds  https://review.openstack.org/35763907:46
*** moshele has joined #openstack-ironic07:48
*** e0ne has joined #openstack-ironic07:48
*** hoangcx_ has quit IRC07:48
*** lennyb has joined #openstack-ironic07:53
*** e0ne has quit IRC07:57
*** lennyb has quit IRC07:57
*** e0ne has joined #openstack-ironic08:01
openstackgerritMerged openstack/bifrost: Add wait_for_node_deploy parameter to deploy task with custom instance_info  https://review.openstack.org/42424408:05
openstackgerritZhenguo Niu proposed openstack/ironic-specs: Dynamically allocate console ports  https://review.openstack.org/24987608:06
*** lennyb_ has joined #openstack-ironic08:09
*** lennyb_ has quit IRC08:09
*** lennyb has joined #openstack-ironic08:10
*** yonglihe has joined #openstack-ironic08:11
*** tovin07 has quit IRC08:13
vdrokgood morning ironic!08:21
*** ralonsoh has joined #openstack-ironic08:24
*** rbudden has joined #openstack-ironic08:25
*** tovin07 has joined #openstack-ironic08:29
*** e0ne has quit IRC08:30
*** amarao has joined #openstack-ironic08:33
openstackgerritGalyna Zholtkevych proposed openstack/ironic: [WIP] Migrate to session for glance client  https://review.openstack.org/41076708:34
*** moshele has quit IRC08:34
*** moshele has joined #openstack-ironic08:34
openstackgerritAnh Tran proposed openstack/bifrost: Typo fix: insufficent = insufficient  https://review.openstack.org/42505308:36
*** amotoki has quit IRC08:47
*** bfournie has joined #openstack-ironic08:54
*** rbartal has joined #openstack-ironic08:56
*** bcornec1 has joined #openstack-ironic08:59
*** hoangcx has quit IRC09:00
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-ironic09:00
*** tovin07 has quit IRC09:00
*** dtantsur|afk is now known as dtantsur09:02
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Make grenade actually upgrade our source code  https://review.openstack.org/42506609:03
dtantsurmorning Ironic09:04
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui: Imported Translations from Zanata  https://review.openstack.org/42506709:05
vdrokgood morning dtantsur09:07
*** sayantan_ has quit IRC09:07
*** newmember has joined #openstack-ironic09:08
openstackgerritGalyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2  https://review.openstack.org/42506809:09
*** rbudden has quit IRC09:11
*** pmannidi has quit IRC09:14
*** tovin07 has joined #openstack-ironic09:15
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Change (started|finished)_at column type  https://review.openstack.org/39763609:23
*** bcornec1 has quit IRC09:24
*** alineb_ has joined #openstack-ironic09:26
*** lucas-afk is now known as lucasagomes09:27
lucasagomesmorning all09:27
vdrokmorning lucasagomes09:30
*** bcornec1 has joined #openstack-ironic09:33
openstackgerritAparna proposed openstack/ironic: Doc for sanitize erase support in iLO drivers  https://review.openstack.org/40844109:38
*** tovin07 has quit IRC09:41
openstackgerritShivanand Tendulker proposed openstack/ironic: Update proliantutils version required for Ocata release  https://review.openstack.org/42459309:42
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off  https://review.openstack.org/21674309:43
dtantsursambetts|afk, mind reviewing https://review.openstack.org/#/c/417871/ please? seems like folks wait for someone knowing about inspector to approve :)09:47
patchbotpatch 417871 - ironic-python-agent - Remove compatibility with old bash-based introspec...09:47
*** udesale__ has quit IRC09:47
dtantsurmorning lucasagomes, vdrok09:47
lucasagomeso/09:47
*** udesale has quit IRC09:47
*** derekh has joined #openstack-ironic09:47
*** strigazi_AFK is now known as strigazi09:50
*** e0ne has joined #openstack-ironic09:52
*** athomas has joined #openstack-ironic09:53
*** MattMan has quit IRC09:54
*** MattMan has joined #openstack-ironic09:55
*** Nisha_Agarwal has quit IRC09:55
*** mgoddard has quit IRC10:03
pas-hamorning all :)10:08
vdrokmorning pas-ha10:10
dtantsurmorning pas-ha10:12
pas-hadtantsur: sorry to miss your pings, I will propose backports for PXEBoot take-over now10:12
dtantsurthanks!10:12
dtantsurand no worries - it's busy time now :)10:12
*** sambetts|afk is now known as sambetts10:13
sambettsdtantsur: taking a look now10:14
*** deray has joined #openstack-ironic10:15
sambettsdtantsur: when did we deprecate the bash ramdisk, 1 or 2 cycles ago?10:15
derayhi Ironicers! g'morning10:16
deraydtantsur, sorry to bother you .. but could you bless this? https://review.openstack.org/#/c/418612/10:17
patchbotpatch 418612 - futurist - Introducing NeverAgain functionality for periodics10:17
*** phuongnh has quit IRC10:17
*** openstackgerrit has quit IRC10:17
* sambetts is kind of excited about that ^ 10:18
*** amotoki has joined #openstack-ironic10:19
*** mgoddard has joined #openstack-ironic10:19
deraysambetts, hey .. please relook (if you have time tho :-) )10:19
*** nmathew has joined #openstack-ironic10:19
*** Nisha_ has joined #openstack-ironic10:20
*** openstackgerrit has joined #openstack-ironic10:21
openstackgerritMerged openstack/bifrost: Add socat package  https://review.openstack.org/42425510:21
*** nmathew- has quit IRC10:23
*** nmathew- has joined #openstack-ironic10:24
*** nmathew- has quit IRC10:24
openstackgerritAndrey Shestakov proposed openstack/bifrost: Add support of collecting logs from baremetal nodes  https://review.openstack.org/41550310:27
openstackgerritMarkos Chandras proposed openstack/bifrost: roles: bifrost-ironic-install: SUSE/Debian: Add gcc and python-pip packages  https://review.openstack.org/42485310:28
vdrokmorning sambetts and deray10:28
sambettshey vdrok10:28
*** Nisha_ has quit IRC10:29
*** aarefiev_afk is now known as aarefiev10:32
aarefievmorning all!10:32
vdrokaarefiev: morning10:32
sambettso/ aarefiev10:32
pas-habtw, seems stable/newton functional gates for ironicclient are broken https://review.openstack.org/#/c/424779/10:34
patchbotpatch 424779 - python-ironicclient (stable/newton) - DNM test stable/newton gates10:34
pas-haalthough there's not much activity in this branch, still should be fixed IMO10:34
*** Nisha_ has joined #openstack-ironic10:35
openstackgerritVasyl Saienko proposed openstack/ironic: [Devstack] Allow add several ports to node  https://review.openstack.org/42511210:38
derayvdrok, morning o/10:44
derayaarefiev, o/10:44
dtantsursambetts, I think we don't run its jobs on newton10:48
dtantsurderay, will get to it10:48
deraydtantsur, cool10:48
deraydtantsur, tyot10:48
openstackgerritAndrey Shestakov proposed openstack/bifrost: Allow enroll-dynamic on remote host  https://review.openstack.org/41129910:49
openstackgerritAndrey Shestakov proposed openstack/bifrost: Allow deploy-dynamic on remote host  https://review.openstack.org/41130410:49
deraysambetts, yeah true .. second dtantsur.. we have moved on to ``futurist`` from Mitaka I suppose10:50
sambettsderay: I knew I'd see that sort of logic before hence knowing it was good and solved the problem, and it finally dawned on me this morning that it is because we use loopingcall in our nova driver for polling Ironic10:52
openstackgerritGalyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2  https://review.openstack.org/42506810:53
*** Nisha_ has quit IRC10:53
deraysambetts, okay10:56
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Grenade: don't enable tempest plugins in settings  https://review.openstack.org/42512210:57
openstackgerritVasyl Saienko proposed openstack/ironic: DNR/DNM Testing interface hotplug functionallity  https://review.openstack.org/42512310:59
*** newmember has quit IRC11:00
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix take over for ACTIVE nodes in PXEBoot  https://review.openstack.org/42512411:02
pas-hadtantsur: ^11:02
pas-hawould you like to have the agent driver fix re takeover backported too?11:03
dtantsurI guess it's reasonable11:03
dtantsuralso thanks11:03
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests  https://review.openstack.org/42355611:05
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914111:07
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to attach/detach VIFs to active ironic nodes  https://review.openstack.org/42472311:07
openstackgerritVladyslav Drok proposed openstack/ironic: Allow setting dhcp_provider in devstack  https://review.openstack.org/42512711:08
vdrokdtantsur: jroll pas-ha https://review.openstack.org/425127 fix for the stable/newton functional tests in client11:08
patchbotpatch 425127 - ironic (stable/newton) - Allow setting dhcp_provider in devstack11:08
dtantsurk11:09
*** deray has quit IRC11:09
openstackgerritVladyslav Drok proposed openstack/python-ironicclient: DNM test stable/newton gates  https://review.openstack.org/42477911:10
vdrokouch, one extra line :)11:10
vsaienk0morning Ironicers!11:10
vsaienk0dtantsur: not sure if you already saw https://review.openstack.org/#/c/423556 I hope it will help us to decrease number of CI jobs11:10
patchbotpatch 423556 - ironic - WIP: ironic standalone tests11:10
openstackgerritVladyslav Drok proposed openstack/ironic: Allow setting dhcp_provider in devstack  https://review.openstack.org/42512711:11
dtantsurvsaienk0, morning! thanks, I'll take a look when I have a minute.11:19
vsaienk0dtantsur: thanks!11:20
* dtantsur thinks he finally has to repair inspector's stable/newton, sigh.. :)11:24
*** dtantsur is now known as dtantsur|lunch11:27
*** jkilpatr_ has quit IRC11:37
*** aparnav has quit IRC11:37
*** mgoddard has quit IRC11:45
jrollmorning y'all11:49
jrollthanks for driver composition reviews, fixing those up now11:49
openstackgerritDao Cong Tien proposed openstack/python-ironicclient: Add args to CLI 'node-create' for selecting hardware interfaces  https://review.openstack.org/41835911:49
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-specs: Amend dynamic iPXE configuration spec  https://review.openstack.org/39229011:50
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests  https://review.openstack.org/42355611:51
*** jkilpatr has joined #openstack-ironic11:54
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: Support --os-baremetal-api-version latest  https://review.openstack.org/42065911:57
dtantsur|lunchmorning jroll11:58
*** dtantsur|lunch is now known as dtantsur11:58
openstackgerritMerged openstack/ironic: Use global vars for storing image deploy path's  https://review.openstack.org/42072412:01
*** lucasagomes is now known as lucas-hungry12:01
openstackgerritGalyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2  https://review.openstack.org/42506812:04
*** moshele has quit IRC12:05
*** moshele has joined #openstack-ironic12:05
dtantsurjroll, so, what's the release plan for today?12:08
jrolldtantsur: I haven't looked at the queue yet today, how are things looking?12:08
jrollI'd also like to clean up release notes12:08
dtantsurjroll, ironicclient looks good, I wonder if we want driver comp bits in (blocked by API)12:09
dtantsurin ironic-inspector-client we have an outstanding patch for LLDP, which is blocked on ironic-inspector part as well12:09
dtantsurdunno if we want both in12:09
*** d0ugal has quit IRC12:10
jrolldtantsur: well, it would be nice, are both of those ready to land?12:10
dtantsurthe former rather yes, the latter not quite12:11
jrollah12:11
jrolldtantsur: I should have new API patches in about 30 minutes12:12
jrollso if we get those approved before you leave today, I say we wait for driver comp bits12:12
jrolland go ahead with inspector without lldp12:12
*** charlle-afk has quit IRC12:12
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Cherry-pick fixes for Grenade  https://review.openstack.org/42196512:17
dtantsurjroll, ack12:17
*** d0ugal has joined #openstack-ironic12:18
vsaienk0jroll, sambetts: I'm wondered if we can land this in Ocata https://review.openstack.org/#/c/419975/12:24
patchbotpatch 419975 - nova - Implement interface attach/detach in ironic virt d...12:24
jrollvsaienk0: tomorrow is nova's feature freeze and we don't even have a blueprint describing this12:25
jrolland it's in merge conflict12:26
*** cdaniel has joined #openstack-ironic12:27
* dtantsur for some reason assumed that it was part of portgroup work..12:27
jrolldtantsur: well, it doesn't always "just work", e.g. environments without dhcp12:28
dtantsurah, ok12:28
jrolland in the vlan-aware world it will need instance config12:28
cdanielmorning all12:28
dtantsurdamn gerrit Y U SO SLOW12:28
jroll++12:28
vdrokmorning jroll12:29
vdrokand morning cdaniel12:29
jrollmorning cdaniel, vdrok12:29
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Allow diskless nodes introspection  https://review.openstack.org/32662012:31
vsaienk0jroll: I thought that it might be related to attach/detach work that was done12:31
jrollvsaienk0: it's related, but the blueprint didn't specify this12:32
*** ccamacho is now known as ccamacho|lunch12:33
cdanielmorning vdrok, jroll12:33
jrollvsaienk0: I'm not going to ask nova to accept it, one day before FF, without having it in the BP12:33
vsaienk0jroll: yeah, I understood, hope we can merge that in Pike, I'm going to create a bp12:33
jrollvsaienk0: agree, thanks12:34
vsaienk0jroll: found already existed bp https://blueprints.launchpad.net/nova/+spec/ironic-attach-detach-interface but it is not approved12:35
*** vikrant has quit IRC12:37
openstackgerritMerged openstack/ironic-inspector: Change (started|finished)_at column type  https://review.openstack.org/39763612:38
sambettsjroll, vsaienk0: I personally think that needs an RFE on the Ironic side too, we need to add code to deal with what happens if someone calls vif_attach when a node is in active state12:40
sambettsvsaienk0: I think you had one right? is it approved?12:40
vsaienk0sambetts: I have a code already https://review.openstack.org/#/c/424723/ but I didn't open RFE12:41
patchbotpatch 424723 - ironic - Allow to attach/detach VIFs to active ironic nodes12:41
*** rbartal has quit IRC12:41
vsaienk0sambetts: going to create RFE for this on ironic side12:41
jroll+112:43
sambettsvsaienk0: I think there is a bug up regarding it, there were attempts before interface attach/detach became a thing to implement the functions on the nova side12:43
mat128sambetts, vsaienk0: if attach/detach isnt to be used on active nodes, what is it for?12:45
sambettsmat128: so the idea is that post-deploy a user could call nova interface-attach and attach a new neutron port12:46
-openstackstatus- NOTICE: Gerrit is going to be restarted due to slow performance12:46
sambettsmat128: its a supported feature for VMs12:46
vsaienk0mat128 it is possible to attach VIF to active node at the moment, but it will not trigger neutron port update, it will not be plugged to tenant network. So might be we can treat it as a missing functionality or bug?12:47
mat128sambetts: I know this (we have it already on BM), but what is the purpose of attaching other than that?12:47
mat128vsaienk0: ahh, got it12:47
vsaienk0sambetts: it shouldn't be tied to nova as we may support it in standalone.12:47
*** nmathew- has joined #openstack-ironic12:48
sambettsvsaienk0: sure, but nova will reject the request right now becuase the nova driver says unsupported, and also the we haven't written any code to handle it in Ironic so I think it should be treated as missing functionality not a bug12:49
jrollsambetts: +112:49
-openstackstatus- NOTICE: Gerrit has been successfully restarted12:49
vsaienk0sambetts: it is not supported by neutron driver, for flat case we don't need changes. But I agree it is to late to try to push it in Nova12:51
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API  https://review.openstack.org/42472012:51
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API  https://review.openstack.org/41797012:51
jrollvdrok: dtantsur: addressed comments, let's get this done ^^12:51
dtantsurok, jumping on a meeting now, will check afterwards12:51
*** nmathew has quit IRC12:51
sambettsvsaienk0: for flat case we'll need to reject the request, so we'll still need to handle what happens if the node is in active12:52
jrollcool12:52
vsaienk0sambetts, jroll: so question do we still need a separate RFE or bug in ironic, or we may just use existed https://bugs.launchpad.net/ironic/+bug/1582188 ?12:52
openstackLaunchpad bug 1582188 in Ironic "[RFE] Add interface attach API" [Wishlist,Fix released] - Assigned to Vasyl Saienko (vsaienko)12:52
* jroll gets breakfast and such12:52
jrollvsaienk0: a new one, please12:52
jrollthat one is approved and completed already12:52
*** nmathew- has quit IRC12:55
*** rbartal has joined #openstack-ironic12:56
*** moshele has quit IRC12:57
openstackgerritNoam Angel proposed openstack/ironic-inspector: [devstack] Remove bridge only if it's not OVB_PHYSICAL_BRIDGE  https://review.openstack.org/42375813:00
*** lucas-hungry is now known as lucasagomes13:00
vsaienk0jroll, sambetts: https://bugs.launchpad.net/ironic/+bug/165928213:02
openstackLaunchpad bug 1659282 in Ironic "[RFE] Allow to attach/detach VIF to active Ironic nodes." [Undecided,New]13:02
vsaienk0sambetts: so do you think we should block vif attach/detach to/from active ironic nodes at the moment?13:02
*** hw_wutianwei has quit IRC13:03
sambettsvsaienk0: Nova already does it for us in the OpenStack case, in the standalone case I'm unsure because we don't know what third party network interfaces are capable of, they might already have added support for that13:04
sambetts;)13:04
vsaienk0sambetts: nova just check when call attach, detach will trigger ironic vif detach.13:05
sambetts?13:05
sambettswe block people calling attach/detach via nova on an active instance via https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L13313:06
vsaienk0sambetts: I found it when worked on nova patch. So nova checks that virt driver support interface attach when doing attach https://review.openstack.org/#/c/419975/6/nova/virt/ironic/driver.py@133 . But it doesn't check it when do detach13:07
patchbotpatch 419975 - nova - Implement interface attach/detach in ironic virt d...13:07
vsaienk0sambetts: but anyway detach_interface() is not implemented so nothing worry about :)13:08
sambettsyeah there is also that :-P13:08
ricardoasmorning ironic13:16
*** ccamacho|lunch is now known as ccamacho13:19
*** vsaienko has joined #openstack-ironic13:27
*** rbudden has joined #openstack-ironic13:28
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests  https://review.openstack.org/42355613:31
*** moshele has joined #openstack-ironic13:33
*** udesale has joined #openstack-ironic13:33
openstackgerritGalyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2  https://review.openstack.org/42506813:41
*** vsaienko has quit IRC13:47
*** hw_wutianwei has joined #openstack-ironic13:48
dtantsur"The neutron CLI is now deprecated. This is the signal that it is time to start using the openstack CLI. "13:49
*** vsaienko has joined #openstack-ironic13:51
*** baoli has joined #openstack-ironic13:58
dtantsurjroll, something potentially nice to get in the ironicclient release https://review.openstack.org/#/c/420659/13:59
patchbotpatch 420659 - python-ironicclient - Support --os-baremetal-api-version latest13:59
*** baoli_ has joined #openstack-ironic14:00
*** rloo has joined #openstack-ironic14:03
*** jcoufal has joined #openstack-ironic14:03
*** baoli has quit IRC14:03
*** vsaienko has quit IRC14:04
*** srobert has joined #openstack-ironic14:05
*** srobert has quit IRC14:05
*** bfournie has quit IRC14:05
*** srobert has joined #openstack-ironic14:06
*** vsaienko has joined #openstack-ironic14:07
*** vsaienko has quit IRC14:10
hwoarangHello. I recently started noticing this problem on centos7 http://paste.opensuse.org/45288127  The reason for this appears to be that the six module from the eventlet package is used instead of the one coming from the six package itself. Has anyone noticed that? Importing six before eventlet fixes the problem for me14:12
*** zzzeek has quit IRC14:12
*** Goneri has joined #openstack-ironic14:13
*** zzzeek has joined #openstack-ironic14:14
TheJuliaGood morning everyone14:20
*** dprince has joined #openstack-ironic14:21
dtantsurmorning TheJulia14:21
openstackgerritVladyslav Drok proposed openstack/python-ironicclient: Allow creating portgroups via create commands  https://review.openstack.org/42519914:22
vdrokmorning TheJulia14:22
openstackgerritMerged openstack/ironic: Allow setting dhcp_provider in devstack  https://review.openstack.org/42512714:22
dtantsurhwoarang, I haven't seen it, maybe it worth chatting with Neutron folks (the failure seems to happen in neutronclient)14:23
lucasagomesTheJulia, hi there14:23
hwoarangdtantsur: will do thanks14:23
vdrokdtantsur: jroll rloo ^^ completely forgot about doing https://review.openstack.org/425199. don't particularly care if it gets in or not, but maybe worth adding. If you have time to review that :)14:23
patchbotpatch 425199 - python-ironicclient - Allow creating portgroups via create commands14:23
* dtantsur adds to his review list14:24
rloovdrok: thx for that; i was wondering the other day. will look today.14:24
rloohi and good morning vdrok, dtantsur, TheJulia, lucasagomes14:24
vdrokthanks! and sorry for doing it this late14:24
lucasagomeso/14:24
dtantsurmorning rloo14:24
rlooTheJulia: thx for reviewing and landing those client patches last night!14:24
rloovdrok: better late than never ;)14:25
TheJuliarloo: no problem, wish I could have looked at them during the day but all day meetings yesterday.14:26
* TheJulia somehow attempts to summon the energy not to do the zombie shuffle into the office this morning.14:27
rlooTheJulia: no worries, as long as we get the ones I want! landed by today :)14:27
TheJuliarloo:  :)14:27
* jroll is back14:29
TheJuliaWhat14:30
TheJuliaErr, wb14:30
vdrokjroll: have you seen my comments here - https://review.openstack.org/#/c/417970/18/ironic/conductor/manager.py ? Am I missing something?14:33
patchbotpatch 417970 - ironic - Add dynamic driver functionality to REST API14:33
jrollvdrok: yeah, I, uhhh14:35
jrollthought I responded14:35
jrollwe have a vendor attribute on the hardware types returned there14:35
* jroll grabs laptop and publishes comments14:35
*** udesale has quit IRC14:36
jrollvdrok: and, published14:36
jrollI can walk you through the code if you like14:36
dtantsurplease remove the fake bit from generic.py, then it lgtm14:37
jrolldid I not? oops14:37
jrolloh wait, I wonder if vdrok is right O_o14:38
vdrokjroll: yeah, build_driver_for_task is not called AFAI can see14:39
jrollhrm14:39
*** vsaienko has joined #openstack-ironic14:40
*** bfournie has joined #openstack-ironic14:40
jrolloh, seems I'm testing that14:40
jrollwhomp14:40
jrollok I'll fix that up, thanks vdrok14:41
vdroknp :)14:41
* jroll wonders why we mock the vendor interface out in driver passthru tests, that's annoying14:42
*** gcb has quit IRC14:42
ricardoashello everyone!14:43
vdrokmorning ricardoas14:43
ricardoasI got a question about third party ci... is there any recommendation for running backports too?14:43
jrollhi ricardoas :)14:43
lucasagomesjlvillal, hi there, around ? Quick q: Is there anyone looking at why the grenade jobs fails with the *_ipmitool drivers ?14:43
*** cdearborn has joined #openstack-ironic14:43
jrollricardoas: I don't think we've talked about it much, we don't require it afaik, but you're welcome to do so. does that help?14:44
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests  https://review.openstack.org/42355614:44
jlvillallucasagomes, I don't think so. Also I'm not sure about "jobs". It was the one job I am working on.14:44
ricardoasi'm one of those responsible for ufcg-oneview-ci, and i see there are two patches fixing validation issues at oneview driver currently being backported for stable/newton14:45
lucasagomesjlvillal, right one, do we have a bug for it or something ? I'm kinda interested to see what is actually failing14:45
*** mrtenio has joined #openstack-ironic14:46
jlvillallucasagomes, No bug. This is still an under development gate job for multi-tenant + grenade. And then adding multi-node in the future.14:46
ricardoasbut they are not passing as currently configuration is not compatible with devstack for stable/newton, jroll14:46
jrollricardoas: file a bug?14:47
lucasagomesjlvillal, right on14:47
jrollricardoas: or you mean, your CI config is not compatible14:47
ricardoasjroll, yes! that's it... we stack master but we're not able to stack newton with the same ci configuration...14:48
jrollricardoas: should be a ZUUL_BRANCH variable you can use for config differences, or something like that14:48
*** hw_wutianwei has quit IRC14:50
*** mrtenio has quit IRC14:50
ricardoasjroll, ok... i'll take a look! The solution then is to maintain a specific configuration for every ZUUL_BRANCH, right?14:50
jrollricardoas: I'd probably just have if statements for particular pieces, lemme see if I can find an example14:51
ricardoasjroll thanks! that'd help a lot14:51
jrollricardoas: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L175-L18214:52
*** udesale has joined #openstack-ironic15:00
lucasagomesjroll, question re storing images for gate testing. We need an image with grub2 installed to be able to test the partition image deployment in gate. Neither CirrOS or Ubuntu partition images include it so, we probably need to create/customize one and store it somewhere for us to use15:01
lucasagomesjroll, do you know if -infra can store it for us ? (/me probably should have asked at -infra...)15:02
jrolllucasagomes: yeah, I'd recommend we just store it on tarballs.o.o, like we do for agents15:02
jrollput the builder code in the ironic repo (or maybe elsewhere, dunno), and build/publish if it changes15:03
jrollas a post job15:03
lucasagomesjroll, right on... but to get it there ? We should have something like apost job irght ?15:03
lucasagomesright15:03
jrollyep15:03
* lucasagomes looks into it15:03
jrolllucasagomes: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic-python-agent-jobs.yaml15:03
sambettsjroll, lucasagomes: at the summit I was dicussing with the Neutron team about setting up a repo with build scripts so that we could use a post-job to publish them to tarballs15:03
jrollis how we do IPA15:03
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support  https://review.openstack.org/26425715:03
jrollsambetts: yeah, we should do that thing15:04
*** dtantsur is now known as dtantsur|brb15:04
openstackgerritXavier proposed openstack/ironic: Validation before perform node deallocation  https://review.openstack.org/38601415:04
lucasagomessambetts, right on, yeah it would make sense to have a separated repo to keep it organized15:04
jrollI wonder if we should just have a openstack/test-images repo or something15:05
pas-hajroll: +15:05
lucasagomessambetts, any progress on it from the neutron part ? Or perhaps we should just take the first step ?15:05
lucasagomesjroll, yeah, reads good15:05
sambettslucasagomes: nope no progress only an idea15:07
*** sayantani01 has joined #openstack-ironic15:07
*** dyasny has quit IRC15:09
*** zackf has joined #openstack-ironic15:09
*** jaosorior has joined #openstack-ironic15:10
*** yufei has joined #openstack-ironic15:17
*** sasha2 has joined #openstack-ironic15:18
*** yufei_ has joined #openstack-ironic15:21
*** dyasny has joined #openstack-ironic15:21
*** yufei has quit IRC15:25
*** rajinir has joined #openstack-ironic15:27
*** Syed__ has joined #openstack-ironic15:27
*** baoli_ has quit IRC15:29
openstackgerritGalyna Zholtkevych proposed openstack/ironic: ETAGs maintenance for all api node's requests  https://review.openstack.org/39221315:29
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: WIP: IPA rescue extension  https://review.openstack.org/42352115:30
*** mtanino has joined #openstack-ironic15:30
*** vinsh has joined #openstack-ironic15:31
*** rpioso has joined #openstack-ironic15:31
openstackgerritMerged openstack/ironic: iRMC power driver for soft reboot and soft power off  https://review.openstack.org/21674315:32
mjturekmariojv: probably need tests for the checks you added to the rescue/unrescue CLI patch, would you want me to handle that?15:33
mariojvmjturek: go ahead - thanks !15:34
mjtureknp mariojv :)15:34
mariojvwe're starting to test rescue end-to-end and are finding a couple things that need fixing15:35
mjturekyeah passing the value to set_provision_state is probably important! lol, good catch15:36
*** dyasny has quit IRC15:38
rlooif anyone feels like reviewing a client patch, this is fairly easy: https://review.openstack.org/#/c/373096/15:38
patchbotpatch 373096 - python-ironicclient - Add --no-maintenance to OSC 'baremetal node list'15:38
*** dyasny has joined #openstack-ironic15:40
*** jschwarz is now known as jschwarz|brb15:41
openstackgerritLucas Alvares Gomes proposed openstack/ironic: DevStack: Fix cleaning up nodes with NVRAM (UEFI)  https://review.openstack.org/42524215:41
*** sasha2 has quit IRC15:42
*** sasha2 has joined #openstack-ironic15:44
*** awiddersheim has quit IRC15:45
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API  https://review.openstack.org/42472015:46
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API  https://review.openstack.org/41797015:46
jrollvdrok: dtantsur|brb ^ should be good now15:46
*** jschwarz|brb is now known as jschwarz15:48
*** Sukhdev has joined #openstack-ironic15:51
jlvillalsoliosg, I saw the test patches putting the ironic tempest tests into their own repo worked. Congrats :)15:51
*** awiddersheim has joined #openstack-ironic15:54
*** dtantsur|brb is now known as dtantsur15:56
*** milan has joined #openstack-ironic15:56
vdrokmorning jlvillal15:56
jlvillalHi vdrok :)15:56
milanmorning Ironic! :)15:56
jlvillalAnd hello everyone else :)15:56
vdrokjroll: wdyt about vsaienk0's comment on detail parameter?15:56
milanmorning jlvillal, vdrok! :)15:56
dtantsurmorning jlvillal, milan15:56
vdrokmorning milan :)15:57
milanmorning dtantsur! :)15:57
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: WIP: IPA rescue extension  https://review.openstack.org/42352115:57
jrollvdrok: that's a good catch15:58
vdrokjroll: also, generic hw still has that fake thing :D15:58
jrollwat15:59
jrolloh I think I removed it in the wrong patch15:59
jrollugh15:59
jlvillalsoliosg, A git-filter-branch thing I wrote: http://paste.openstack.org/show/596475/16:00
jlvillaljroll, On the new ironic tempest plugin repo. We have about 36 commits to recreate the history of what is in openstack/ironic/ironic_tempest_plugin/16:01
aslezildtantsur: (https://review.openstack.org/#/c/404372/ )  I found a nit in commit msg so uploaded a new patch. I lost your +2 on it. Could you please review.16:01
patchbotpatch 404372 - ironic-inspector-specs - Collect NIC name given by BIOS16:01
jlvillaljroll, Should we submit 36 patches to openstack/ironic_tempest_plugin/ to try to recreate the history? Or just one big patch?16:01
dtantsuraslezil, will do after the current meeting16:01
* jlvillal is leaning towards the more work of 36 patches...16:01
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add test to create a port with specific port group UUID  https://review.openstack.org/41182616:01
aslezilthanks dtantsur:16:01
JayFjlvillal: another option is tearing down the repo and having infra recreate it from an upstream, like you all were talkign about yesterday, right?16:02
jrolljlvillal: I'm fine with the former, history would be great16:02
jrollyeah or that16:02
jrollthat seems more painful though16:02
jlvillalJayF, They didn't sound too excited about that option...16:02
*** dsneddon_afk is now known as dsneddon16:02
jrollthere's no CI on that repo yet, right?16:02
jlvillaljroll, A pep8 one.16:02
jrollok, yeah not a big deal16:03
jlvillalI think. Not sure.16:03
jrollmaybe double check with infra because there's a chance it could overload zuul16:03
jlvillalOne job is what I remember. Looking...16:03
jrollbut I think it should be fine16:03
jlvillalOkay. I will work with soliosg on it. Thanks.16:03
jlvillaljroll, He did get to passing with the plugin last night :)16:04
jrolljlvillal: awesome16:04
jlvillalYeah, pep8 job. Takes less than 2 minutes to run.16:04
jrolljlvillal: soliosg: please be sure to document the steps here :)16:04
*** derekh has quit IRC16:05
jlvillalsoliosg, Also should update mtreinish etherpad to mention that people may want to use an "upstream" repository when creating the new repository for their plugin. If not arleady documented.16:05
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for port groups in ironicclient  https://review.openstack.org/29772716:05
jlvillalsoliosg, Plus could give a pointer to the script I did to filter-branch just the tempest plugin directory.16:06
*** rbartal has quit IRC16:11
jrolljlvillal: I think matt had that already in that etherpad16:12
jlvillaljroll, Ah good.16:12
openstackgerritGalyna Zholtkevych proposed openstack/ironic: ETAGs for subresource requests  https://review.openstack.org/42526016:13
soliosgjlvillal: ok, will continue to refine the patch sets, document and update docs as needed16:13
soliosgjlvillal: something important is to plan the transition from ironic to the new repo, since at some point there will be patches submitted to both repos16:14
soliosgjlvillal: we will need some kind of 'code freeze' for tempest patches submitted to ironic16:15
*** mgould|afk is now known as mgould16:15
*** rushil has joined #openstack-ironic16:16
soliosgjlvillal: *submitted to ironic that update ironic/ironic_tempest_plugin to be more specific16:17
jrollsuspect right after ocata release is a good time for that16:18
*** vsaienko has quit IRC16:18
jrollsoliosg: want to add that to the next meeting agenda and we can confirm? it shouldn't take long to transition it16:18
soliosgjroll, jlvillal: ok, it's good idea to bring it up, so everyone has it in mind16:21
jlvillalsoliosg, jroll: I think it will be one change to project-config. Once we have the plugin repository ready. Maybe a change to ironic too.16:21
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API  https://review.openstack.org/42472016:21
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API  https://review.openstack.org/41797016:21
jrolldtantsur: vdrok: let's do this16:22
dtantsur:)16:22
soliosgjroll, jlvillal: search filter for ongoing patches that touch ironic/ironic_tempest_plugin, https://review.openstack.org/#/q/project:openstack/ironic+status:open+file:%22%255Eironic_tempest_plugin/.*%2216:22
rloojroll and gang: as far as client patches go, these are the ones I'd like to see get in:16:22
rloohttps://review.openstack.org/#/c/373096/, needs review16:22
patchbotpatch 373096 - python-ironicclient - Add --no-maintenance to OSC 'baremetal node list'16:22
vdrokheh, will look in a minute16:22
jrolljlvillal: yeah, but we'll have to sync it between landing stuff in the plugin repo and whenever we convert it16:23
jlvillalsoliosg, Cool gerrit search :)16:23
rloohttps://review.openstack.org/#/c/420659/ needs to be updated and reviewed so not sure it will make it16:23
patchbotpatch 420659 - python-ironicclient - Support --os-baremetal-api-version latest16:23
rloohttps://review.openstack.org/#/c/425199/ needs to be updated and reviewed so not sure it will make it16:23
patchbotpatch 425199 - python-ironicclient - Allow creating portgroups via create commands16:23
rlooother than those, i may look at some other patches but there aren't any high bugs outstanding.16:24
*** sacharya has joined #openstack-ironic16:24
*** sacharya has quit IRC16:24
rlooif anyone else has any client patches they want to land today, please mention!16:24
jrollrloo: nice, thanks16:24
rloojroll: oh yeah, and your relnotes patch that doesn't exist yet :)16:24
jrollrloo: indeed16:25
rloojroll: let me/us know when you want to pull the trigger16:25
*** amarao has quit IRC16:25
*** rushil has quit IRC16:25
*** rushil has joined #openstack-ironic16:26
*** udesale has quit IRC16:26
vdrokrloo: do you have a preference in doing continue vs checking with if in the create patch?16:27
vdrokthere should not be a case when we have an error and portgroup_uuid at the same time16:27
jrollrloo: I'm slowly convincing myself that cleaning up release notes isn't worth it16:29
jrollrloo: so unless updates come in for patches #2 and #3 on your list, I'm ready when --no-maintenance lands16:30
dtantsurjroll, please check a small patch https://review.openstack.org/#/c/421965/ to unbreak inspector newton CI, if you have a second16:30
patchbotpatch 421965 - ironic-inspector (stable/newton) - Cherry-pick fixes for Grenade16:30
jrollsure16:31
*** mjura has quit IRC16:32
rloovdrok: no preference16:33
rloojroll: ok16:33
jrolldtantsur: +A16:33
dtantsurthnx16:33
rloovdrok, galyna ^^, let me know when/if you update your client patches soon/today16:33
vdrokjroll: one really tiny thing in the first patch16:34
vdrokrloo: I will do now16:34
dtantsurrloo, +A on --no-maintenance16:34
rloothx dtantsur!16:34
* jroll shakes fist at vdrok16:35
vdrokawww :'(16:35
vdrok:D16:35
* rloo thanks vdrok!16:35
*** AlexeyAbashkin has quit IRC16:35
*** morgabra has quit IRC16:36
openstackgerritVarun Gadiraju proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job  https://review.openstack.org/40389516:37
jrollvdrok: "Could be None by default, so that we could check if it was set either to True or False, we raise 406" I don't quite get this, it's a boolean type, it should be fine as is?16:38
openstackgerritVarun Gadiraju proposed openstack/ironic: WIP/DNM: Testing grenade multi-tenant experimental job  https://review.openstack.org/40389516:38
vdrokjroll: I mean, if we set detail=False, we'd ideally fail too. our boolean type also accepts None, so it will work16:38
vdrokif we set detail=False on old api version16:39
jrollvdrok: urgh, I see what you mean, sigh16:39
* jroll fixes16:41
*** moshele has quit IRC16:41
*** bcornec1 has quit IRC16:42
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API  https://review.openstack.org/42472016:42
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API  https://review.openstack.org/41797016:42
jrolldone vdrok16:42
vdrokthanks :)16:43
jrollnp, thank you16:44
jrolldtantsur: are you waiting on anything for inspector client release?16:44
mgouldhow do I install optional driver dependencies on devstack?16:45
mgouldspecifically, python-dracclient16:45
jrollmgould: pip install? :)16:46
jrollrather sudo pip install I believe16:47
jrolland restart conductor16:47
mgouldyeah, I guess so16:48
mgouldI was wondering if there was a way to make it happen as part of stack.sh16:48
jrollI feel like their might be a thing to add packages, not sure16:48
* jroll looks16:48
dtantsurjroll, lemme check16:49
mgouldjroll: thanks16:50
dtantsurwell, we only have the LLDP thing, and it does not look like we're merging it in the coming hours, right, bfournie?16:50
*** [1]rpioso has joined #openstack-ironic16:50
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Fix help message for the node-vif-attach command  https://review.openstack.org/42528116:51
JayFmariojv just rescued an ironic node in devstack \o/16:52
dtantsur\o/16:52
mariojvwoohoo \o/16:52
*** rpioso has quit IRC16:52
bfourniedtantsur: if needs to happen by EOD today probably not likely, waiting on reviews for plugin patch and am submitting updated patch for inspector-client which will need reviews16:53
jrollmgould: I'm not seeing anything, but you might try -qa channel, I'd also welcome a patch to our devstack plugin to install optional deps (e.g. if drac in enabled_drivers: pip_install python-dracclient)16:53
dtantsurbfournie, thanks16:53
mgouldjroll: cool, thanks16:53
dtantsurjroll, ironic-inspector-client seems ready for the release16:53
openstackgerritVladyslav Drok proposed openstack/python-ironicclient: Allow creating portgroups via create commands  https://review.openstack.org/42519916:53
jrolldtantsur: awesome, thanks!16:53
vdrokrloo: all done, I hope ^^16:54
jrollI see one ironicclient patch is approved16:54
rloovdrok: thx. looking...16:54
rloojroll: yup. give us 4 more hours? :)16:54
jrollgalyna: can you update this patch in the next few hours? https://review.openstack.org/#/c/420659/16:54
patchbotpatch 420659 - python-ironicclient - Support --os-baremetal-api-version latest16:54
jrollrloo: yep, sounds good16:54
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Add tests+support for psutil 1-5  https://review.openstack.org/42528416:55
JayFrloo: ^ that should cover all the feedback you had on the gate-fixin' patch :)16:55
dtantsurjroll, so, we're not including ironicclient --boot-interface etc change, right?16:55
rlooJayF: thx. may look later :)16:55
NobodyCamGood Morning Ironic'ers16:55
dtantsurmorning NobodyCam16:56
NobodyCam:) hey hey dtantsur :)16:56
rloomorning NobodyCam16:56
jrolldtantsur: we could, if we're merging the API today16:56
NobodyCamGood Morning role :)16:56
jrollI would like to16:56
* jroll will review that16:56
vdrokmorning NobodyCam16:57
jrolljlvillal: so I can abandon my *-tempest-plugin cookiecutter things, yes?16:57
NobodyCamgah.. Good Morning rloo... New computer have to train the autocrat all over again16:57
NobodyCamMorning vdrok :)16:57
vdrokanother tiny client patch, might be worth too https://review.openstack.org/42528116:57
patchbotpatch 425281 - python-ironicclient - Fix help message for the node-vif-attach command16:57
jlvillaljroll, Uh. Not sure.16:57
jrolljlvillal: ok, I'll look later16:57
jrollNobodyCam: ++ for "autocrat", lol16:58
dtantsurrloo or JayF or anyone, wanna check the driver comp API patches? vdrok and I have reviewed them several times already16:58
rloodtantsur: i haven't paid attention to the driver comp stuff this week. i'm hesitant to get the client part out today unless you all are really sure about it.16:58
NobodyCamlol ++++16:59
dtantsurrloo, well, the client part is trivial16:59
rloodtantsur: if the driver comp stuff is done in ocata, it is simple to push out a new client right after that.16:59
JayFdtantsur: I'm always willing to take a look, but it's not neccessarily in my area of expertise, so I'm not sure I'll catch much more than you all :)16:59
rloodtantsur: yeah, the client is trivial, the api part might not be so trivial. i'd feel more comfortable if more people tried out the api...16:59
*** athomas has quit IRC16:59
rloodtantsur: but i'll try to take a look in an hour or so...16:59
dtantsurrloo, we're severely limiting number of people trying the API by not providing a CLI for it, so dunno..17:00
jrolldtantsur: the client stuff is missing --detail, at least, so it isn't ready17:00
dtantsurmm, what about --detail?17:01
jrollGET /v1/drivers?detail=True <- to return all fields in driver-list17:01
*** retr0h has joined #openstack-ironic17:01
dtantsurright, I have to admin I've only reviewed the client patch with --XXX-interface, not one with driver changes17:01
dtantsurI'm much more excited about the former, as it would enable people to play with dynamic drivers17:02
jrollah, true17:02
jrollthough the other helps, too17:02
dtantsurI don't disagree with that, just trying to prioritize my reviewing efforts :)17:02
jrollfair17:02
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Add tests+support for psutil 1-5  https://review.openstack.org/42528417:03
jrolldtantsur: I'm +2 on the --foo-interface patch17:05
*** Nisha_Agarwal has joined #openstack-ironic17:06
dtantsurcool17:06
openstackgerritMerged openstack/python-ironicclient: Add --no-maintenance to OSC 'baremetal node list'  https://review.openstack.org/37309617:07
dtantsurrloo, do we also have --no-associated for OSC pending?17:07
rloodtantsur: unassociated. got merged i believe. let me see.17:07
JayFif it didn't, I can take a look at it now17:08
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914117:08
*** rama_y has joined #openstack-ironic17:10
* dtantsur has to go afk for roughly 20 minutes17:10
rloodtantsur, JayF: https://review.openstack.org/#/c/369279/17:10
patchbotpatch 369279 - python-ironicclient - Extend OSC "node list" cmd to fetch nodes without ... (MERGED)17:10
dtantsurcool!17:10
JayF--awesome --not-bad17:10
JayFlol17:10
rloonot a great title but ...17:10
rlooso we almost have feature parity with ironic cli & osc17:10
JayF\o/17:11
* dtantsur doesn't like --unassociated vs --no-maintenance too much, but I guess it's better English this way17:11
dtantsurrloo, JayF, have you seen that they've deprecated neutron CLI in favor of OSC?17:11
rloodtantsur: JayF and I discussed it. we prefer instance/no-instance, but decided it was too late for ocata to make the change.17:11
JayFdtantsur: re: associated, I suggested if enough people dislike it, going to --instance --no-instance might be clearer than --associated --unassociated17:11
dtantsurfine17:11
jroll--unassociated and --no-maintenance are different, right17:11
rloodtantsur: i'd like to deprecate our ironic cli too17:11
dtantsurrloo, goal for Pike? :)17:12
rloodtantsur: i would like.17:12
*** newmember has joined #openstack-ironic17:13
rloodtantsur: deprecate. not sure how long we have to leave before we delete17:13
dtantsurwe don't have 'driver properties list', do we?17:13
rloodtantsur: nope. that's one of the ones we are missing.17:13
rloodtantsur: and raid i think.17:13
dtantsurI see17:13
* dtantsur goes afk for real now, sorry17:13
jrollspeaking of goals for pike, py35 support (including dsvm tests) was selected, as well as deploy-api-via-wsgi, so we'll have that work to do17:14
rloodtantsur: https://bugs.launchpad.net/python-ironicclient/+bug/1619052 and https://bugs.launchpad.net/python-ironicclient/+bug/161905317:14
openstackLaunchpad bug 1619052 in python-ironicclient "[RFE] openstackclient cmd for 'ironic driver-raid-logical-disk-properties'" [Wishlist,In progress] - Assigned to Galyna Zholtkevych (gzholtkevych)17:14
openstackLaunchpad bug 1619053 in python-ironicclient "[RFE] openstackclient cmd for 'ironic driver-properties'" [Wishlist,In progress] - Assigned to Galyna Zholtkevych (gzholtkevych)17:14
rloodtantsur: cuz it wasn't obvious what the osc commands should be17:14
*** dtantsur is now known as dtantsur|brb17:14
rloobye dtantsur!17:14
dtantsur|brbhey, I'm gonna be back soon :)17:14
rloojroll: i thought we already support py35 thx to jlvillal ?17:15
rloojroll: don't answer. we can discuss later :)17:15
openstackgerritVladyslav Drok proposed openstack/python-ironicclient: Support --os-baremetal-api-version latest  https://review.openstack.org/42065917:15
vdrokjroll: rloo updated^, galyna is out for today already17:16
rloovdrok: thx. will look after i finish with your patch (almost done)17:16
jrollrloo: we don't have integration testing yet17:16
jrollvdrok: thanks17:16
*** alexpilotti has quit IRC17:17
JayFjroll: I think vgadiraj and maybe joanna have both looked at the deploy-api-via-wsgi stuff, I don't think we're far17:17
rloojroll: oh. well, we need stuff to do for pike anyway. ha ha.17:17
jrollJayF: awesome17:17
openstackgerritMerged openstack/python-ironicclient: Change os_tenant_name to os_project_name in tests configuration script  https://review.openstack.org/40823517:17
jrollrloo: yeah, we totally don't have enough work :)17:17
rloojroll: esp with you not being ptl, you're gonna get so much done we'll be trying to stay ahead of you to give you stuff to do17:18
*** yolanda has quit IRC17:18
jrollrloo: lol, I hope so17:18
*** alexpilotti has joined #openstack-ironic17:19
vgadirajJayF, jroll: joanna mentioned recently that she was looking at that17:19
JayFJim already shared his secret plan to sit back, drink beer, and cackle at us once he's free17:19
JayF:P17:19
*** ccamacho has quit IRC17:19
* jroll no comment17:20
TheJuliaWait? He is already not sitting back with beer?17:21
NobodyCamlol17:21
TheJuliaClearly if he is not, this needs to be corrected!17:22
*** jaosorior has quit IRC17:22
* TheJulia goes back to meetings17:22
*** alexpilotti has quit IRC17:23
joannaJayF, jroll, vgadiraj: yes, I actually do it now17:24
joannathe wsgi config for apache seems to be used only for hosting httpboot stuff17:24
*** jheroux has joined #openstack-ironic17:24
JayFjoanna: it's been added as a openstack-wide priority to get api services running in apache in devstack (via wsgi)17:25
JayFjoanna: which is sorta related to that, but different ... but def. in the same neighborhood, if you wanted to take a stab at it while you were there17:25
joannaJayF: not promising anything, but I can definitely try to do that and maybe even succeed :)17:26
joannais there any bug reference for that?17:26
JayFjoanna: I suspect there is a bug for it, but I don't know the number right off17:26
joannaok, I'll look into it17:27
JayFhttps://bugs.launchpad.net/ironic/+bug/151300517:27
openstackLaunchpad bug 1513005 in Ironic "Auto deploy devstack with ir-api under apache" [Wishlist,In progress] - Assigned to Anton Arefiev (aarefiev)17:27
joannaawesme, ty!17:27
JayFlooks like aarefiev already has something in flight that more or less got abandoned17:27
JayFmight be a decent starting poitn17:27
joannaI'll get done the httpboot hosting as wsgi, and then move to this one.17:28
*** alexpilotti has joined #openstack-ironic17:28
aarefievjoanna: yeah, there is patch on review which need to be updated, feel free to pick up it17:30
jlvillalrloo, jroll: Well mostly thanks to jroll for the py35 test job. Though last time I checked it failed due to swift. But I haven't had time to look into it.17:31
vdroktime to leave for me, seems I won't be able to look at the client side of driver comp stuff today :(17:31
joannaaarefiev: thank you! I will :)17:31
vdrokgood night everyone!17:31
rloojlvillal: no worries, there's always Pike!17:31
rloovdrok: night and thx for the client patches!17:31
jlvillalBye vdrok17:31
joannanight, vdrok :)17:31
*** sasha2 has quit IRC17:32
NobodyCamnight vdork17:32
*** jaosorior has joined #openstack-ironic17:32
*** alexpilotti has quit IRC17:32
*** dtantsur|brb is now known as dtantsur17:33
*** sdake_ has joined #openstack-ironic17:33
*** tesseract has quit IRC17:33
*** karts has quit IRC17:34
*** sasha2 has joined #openstack-ironic17:35
*** baoli has joined #openstack-ironic17:35
* jroll eats lunch17:36
*** sasha2 has quit IRC17:37
*** aarefiev is now known as aarefiev_afk17:37
*** yolanda has joined #openstack-ironic17:41
openstackgerritAndrea Frittoli proposed openstack/ironic: Fix broken link in the iLO driver docs  https://review.openstack.org/42531617:43
*** alexpilotti has joined #openstack-ironic17:44
*** [1]rpioso has quit IRC17:45
*** rushil has quit IRC17:45
*** alexpilotti has quit IRC17:46
*** alexpilotti has joined #openstack-ironic17:47
*** alineb_ has quit IRC17:48
rlooThanks for the +As TheJulia!17:50
*** aNuposic has joined #openstack-ironic17:51
*** dprince has quit IRC17:53
TheJuliaJust trying to review some stuff while I have some time between meeting subjects :)17:53
*** Nisha_Agarwal has quit IRC17:53
*** lucasagomes is now known as lucas-afk17:54
*** ralonsoh has quit IRC17:58
dtantsurmeh, zuul is slowly overflowing18:04
rloodtantsur: all those client patches waiting to get merged :)18:06
*** yolanda has quit IRC18:06
dtantsuryeah :)18:07
rloodtantsur: inspector-client is in good shape?18:07
dtantsurrloo, yep, we're ready (missed one feature, unfortunately)18:08
rloodtantsur: good (and bad)! am going to get some lunch, then look at driver comp...18:08
dtantsurjroll, vdrok and I are +2 on both driver comp patches, and I'm calling it a day now. Feel free to approve when they pass Jenkins or wait for more reviews.18:08
dtantsuryep, this :)18:08
jlvillalI wonder if there is a way to turn our ironic-bm-logs into an animated gif...18:08
dtantsurjlvillal+++18:09
jlvillalInstead of this: http://logs.openstack.org/49/422149/6/experimental/gate-grenade-dsvm-ironic-multitenant-ubuntu-xenial-nv/7a38d72/logs/old/ironic-bm-logs/node-6_console_24-01-2017-18:55:25.txt.gz18:09
dtantsursee you all :)18:09
jlvillaldtantsur, :)18:09
jlvillaldtantsur, Ciao18:09
*** dtantsur is now known as dtantsur|afk18:09
rloogood night dtantsur!18:09
JayFhttps://review.openstack.org/#/c/425316/ super duper simple doc fix if someone wants to land it18:09
patchbotpatch 425316 - ironic - Fix broken link in the iLO driver docs18:09
JayFnn dtantsur|afk o/18:09
TheJuliajlvillal: server config files :)18:13
jlvillalTheJulia, The "serial console" output that we get from the gate jobs. Very difficult to read for failures.18:14
jlvillalTheJulia, I'm not sure what "server config files" means.18:14
*** e0ne has quit IRC18:18
openstackgerritMerged openstack/ironic: Update proliantutils version required for Ocata release  https://review.openstack.org/42459318:20
openstackgerritMiles Gould proposed openstack/ironic: [WIP] install python-dracclient if drac enabled  https://review.openstack.org/42532618:22
TheJuliajlvillal: I wonder if we could somehow translate that...., but server config files was more in that we could likely update the server config so the content returned is re-processed or a replaced reply18:24
jlvillalTheJulia, On the translating. That was were I was thinking something that could take as input the text file and output an animated gif :)18:25
jlvillalBut I spent five minutes googling and no joy. And don't think it is worth my time at the moment to develop something.18:25
TheJuliaIf we could grab a screenshot via vnc....18:25
*** rushil has joined #openstack-ironic18:25
*** vsaienko has joined #openstack-ironic18:27
TheJuliajlvillal: so, it is partially the result of the serial graphics adapter driver.  We would likely have to reverse engineer the control characters out of existence minus backspace when it is rendered....18:27
*** ChubYann has joined #openstack-ironic18:28
jlvillalTheJulia, Yeah. It would be nice to have the result :)18:28
*** [1]rpioso has joined #openstack-ironic18:28
jlvillalAt the moment I use this.18:28
TheJuliascreenshots would be better, maybe virtual bmc could snap one before power-off ?18:28
jlvillal$ cat LOGFILE | pv -q -L 240018:28
jlvillalTheJulia, Actually the a final screen shot would be nice :)18:29
jlvillals/the a/a/18:29
TheJuliawell, prior to any power-off would be key.  Retries would make other things disappear.18:29
*** sayantani01 has quit IRC18:30
*** sambetts is now known as sambetts|afk18:34
sambetts|afknight all18:34
JayFI just ignore the control characters in the node consoles, honestly18:35
JayFif you read bottom-up like most of those devstack files, you don't see most of them18:35
* JayF wonders if perhaps it's worse on different os/browser combos18:35
*** Sukhdev has quit IRC18:35
*** Sukhdev has joined #openstack-ironic18:36
jlvillalJayF, For me this isn't that easy to read: http://logs.openstack.org/49/422149/6/experimental/gate-grenade-dsvm-ironic-multitenant-ubuntu-xenial-nv/7a38d72/logs/old/ironic-bm-logs/node-6_console_24-01-2017-18:55:25.txt.gz18:36
jlvillals/isn't that easy/is very difficult/18:37
*** trown is now known as trown|lunch18:37
JayFah, honestly I haven't had to trobuleshoot no-pxe that much with the console18:37
JayFand TIL the output I've been mostly-ignoring in those are the ipxe handshake18:37
JayFlol18:37
jlvillal:)18:38
*** sdake_ has quit IRC18:39
*** sdake has joined #openstack-ironic18:40
*** yolanda has joined #openstack-ironic18:40
*** krtaylor has quit IRC18:40
rloojlvillal: don't know if this helps or not, but yesterday mat128 mentioned that he does curl <url> to get a better view of that console stuff18:41
*** sasha2 has joined #openstack-ironic18:41
rloojlvillal: is that what you want? (I skimmed the above)18:41
jlvillalrloo, Yeah. Similar thing. Need to get it into a terminal to parse the escape codes18:41
*** dprince has joined #openstack-ironic18:41
TheJuliaso virsh does have the ability to do a screenshot...18:44
* JayF suspects a client-side greasemonkey script or extension might be the 90% solution for 10% effort for this problem18:45
JayFI've found a few js libs that'd render ansi inline, but that'd require modifying the logs before uploading (and having a js file hosted somewhere), which I suspect is unlikely to garner much support18:46
TheJuliaI typically just use curl or cat the content in a terminal. The annoying thing is when some of the data is not translated at all18:46
*** vsaienko has quit IRC18:46
JayFTheJulia: do you have an example of a log where that happens?18:47
JayFTheJulia: I'm really curious how much distro/shell/etc matters on stuff like that18:47
* JayF has been down this road, in a more painful way, with actual Serial-over-lan consoles that were awful :(18:48
JayFfor instance, on the hardware serial consoles I'm talking about, if you were in screen, it wouldn't render properly, but if you were not in screen it would18:49
*** rushil has quit IRC18:51
mariojvTheJulia: i had some comments on aNuposic's ceph patch that i think might clarify the various use-cases https://review.openstack.org/#/c/407639/6/ironic/conf/conductor.py18:52
patchbotpatch 407639 - ironic - Config drive support for Ceph18:52
mariojvit's really entirely about the auth mechanism, since both ceph support and swift support for configdrives are using the swift client18:52
*** pcaruana has quit IRC18:53
mariojvi think changing those config option names will make it a bit clearer to operators, but curious about what you think too18:53
TheJuliaSorry, had to step away18:53
TheJuliaJayF: specifically, prior to the kernel, I've seen examples where the data just looks like control characters, but when I actually watch like a vnc console is ipxe failing to boot.  The only thing that stands out in my mind is the screen starts with CGA level graphics and then goes to something like 640x480 80 columns by 25 rows with ipxe trying to do it's18:56
TheJuliathing.18:56
*** vsaienko has joined #openstack-ironic18:57
JayFTheJulia: interesting. Yeah, ipxe serial was an issue on our physical boxes too18:57
JayFTheJulia: and you have to recompile ipxe to change any of the settings w/r/t serial console18:57
openstackgerritAndrey Shestakov proposed openstack/bifrost: Allow deploy-dynamic on remote host  https://review.openstack.org/41130418:59
TheJuliaJayF: Typically I've seen that on ubuntu, I wonder if others see it less with different distributions18:59
aNuposicTheJulia: Hi Julia, wrt https://review.openstack.org/#/c/407639/6/ironic/conductor/manager.py@2587, have mentioned in RFE https://bugs.launchpad.net/ironic/+bug/1642719 that it is "yet another way" to talk with radosgw. Yes as mariojv said both swift and radosgw support use swift api and its all about auth mechanism19:00
openstackLaunchpad bug 1642719 in Ironic "[RFE] RADOSGW authentication mechanism for storing config drive in Ceph" [Wishlist,In progress] - Assigned to Anup (anup-d-navare)19:00
patchbotpatch 407639 - ironic - Config drive support for Ceph19:00
JayFaNuposic: mariojv: fwiw I'm in agreement with mariojv's suggestion about config names in your patch19:00
*** alexpilotti has quit IRC19:00
aNuposicJayF: Yup, will update those to make it more clear19:01
TheJuliaaNuposic: I'll take a look either later today or early tomorrow.  I'm in meetings this week19:01
aNuposicJayF: seems those are making a bit confusing to understand19:01
aNuposicTheJulia: Sure19:01
aNuposicTheJulia: thx :)19:02
*** rushil has joined #openstack-ironic19:04
*** hecliunyx has quit IRC19:05
*** ijw has joined #openstack-ironic19:08
openstackgerritMerged openstack/ironic: Onetime boot when set_boot_device isn't persistent  https://review.openstack.org/34059619:08
*** ijw has quit IRC19:09
*** ijw has joined #openstack-ironic19:09
openstackgerritMerged openstack/ironic: Fix broken link in the iLO driver docs  https://review.openstack.org/42531619:10
*** moshele has joined #openstack-ironic19:11
*** hecliunyx has joined #openstack-ironic19:13
*** Sukhdev has quit IRC19:13
*** alexpilotti has joined #openstack-ironic19:15
rloojroll: wrt the driver comp stuff, i see two patches for REST API, but no client ones. we don't need to get the REST API patches merged today?19:17
jrollrloo: we do, so that we can merge the client stuff19:17
rloojroll: where are the client patches?19:17
jrollrloo: https://review.openstack.org/#/q/status:open++branch:master+topic:bug/152474519:17
*** moshele has quit IRC19:18
jrollI'd like to at least get that first one in19:18
jrollif we don't, it's fine19:18
jrollbut will mean we wait a few more weeks before folks can kick the tires on it19:18
*** krtaylor has joined #openstack-ironic19:18
jroll(including things like tripleo)19:18
rloojroll: oh. (i was using the driver comp link in etherpad/priorities)19:19
jrollah, sorry19:19
*** alexpilotti has quit IRC19:19
*** jaosorior has quit IRC19:21
rloojroll: ok, will see what i can do. that's 3 patches that need to be reviewed/merged today...19:22
openstackgerritMerged openstack/python-ironicclient: Fix help message for the node-vif-attach command  https://review.openstack.org/42528119:22
jrollrloo: yeah, if there's more important things, do those19:23
* jroll wants to just +A his API patches, they have 2+219:23
rloojroll: all the client patches i wanted to get in have been +A'd. so I'll look at these.19:24
jrollrloo: ok, thank you19:25
openstackgerritSam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation  https://review.openstack.org/39295919:26
*** mgoddard has joined #openstack-ironic19:29
*** alexpilotti has joined #openstack-ironic19:30
*** trown|lunch is now known as trown19:33
*** alexpilotti has quit IRC19:35
*** e0ne has joined #openstack-ironic19:39
*** [1]rpioso has quit IRC19:40
openstackgerritAndrey Shestakov proposed openstack/bifrost: Allow deploy-dynamic on remote host  https://review.openstack.org/41130419:42
*** mitchjameson has joined #openstack-ironic19:42
openstackgerritMerged openstack/python-ironicclient: Support --os-baremetal-api-version latest  https://review.openstack.org/42065919:43
*** sayantani01 has joined #openstack-ironic19:43
*** alexpilotti has joined #openstack-ironic19:47
*** rushil has quit IRC19:48
*** dprince has quit IRC19:48
*** rushil has joined #openstack-ironic19:48
aNuposicmariojv: hey wrt https://review.openstack.org/#/c/407639/6/install-guide/source/configdrive.rst@95 agent_* drivers require you to use swift_temp_url and so the images are not stored in glance but in whatever object store when you are using agent_* drivers19:49
patchbotpatch 407639 - ironic - Config drive support for Ceph19:49
*** dprince has joined #openstack-ironic19:49
openstackgerritMerged openstack/bifrost: Allow enroll node with empty properties.  https://review.openstack.org/42424919:51
*** alexpilotti has quit IRC19:52
*** Sukhdev has joined #openstack-ironic19:52
*** e0ne has quit IRC19:55
*** hecliunyx has quit IRC20:01
*** Sukhdev has quit IRC20:05
*** alexpilotti has joined #openstack-ironic20:07
*** morgabra has joined #openstack-ironic20:08
*** hecliunyx has joined #openstack-ironic20:09
*** jaosorior has joined #openstack-ironic20:12
*** jaosorior has quit IRC20:17
*** e0ne has joined #openstack-ironic20:17
*** moshele has joined #openstack-ironic20:23
*** jaosorior has joined #openstack-ironic20:24
*** e0ne has quit IRC20:28
*** Goneri has quit IRC20:29
*** e0ne has joined #openstack-ironic20:32
openstackgerritMerged openstack/python-ironicclient: Allow creating portgroups via create commands  https://review.openstack.org/42519920:38
openstackgerritVasyl Saienko proposed openstack/ironic: [Devstack] Allow add several ports to node  https://review.openstack.org/42511220:40
*** e0ne has quit IRC20:40
openstackgerritVasyl Saienko proposed openstack/ironic: DNR/DNM Testing interface hotplug functionallity  https://review.openstack.org/42512320:40
openstackgerritVasyl Saienko proposed openstack/ironic: DNR/DNM Testing interface hotplug functionallity  https://review.openstack.org/42512320:41
openstackgerritMichael Turek proposed openstack/python-ironicclient: Add support for RESCUE and UNRESCUE provision states  https://review.openstack.org/40834120:42
*** dprince has quit IRC20:48
*** dprince has joined #openstack-ironic20:48
*** alexpilotti has quit IRC20:58
*** aNuposic has quit IRC21:00
rloojroll: i reviewed https://review.openstack.org/#/c/417970/. I'm not sure how serious any of those are. the API itself (syntax) is fine so could merge and address later.21:00
patchbotpatch 417970 - ironic - Add dynamic driver functionality to REST API21:00
jrollrloo: O_O21:01
*** alexpilotti has joined #openstack-ironic21:02
jrollrloo: some good catches in there, though, will need to re-spin21:02
rloojroll: ok, i'll keep an eye out for the spinned21:03
jrollthere's a point in time where you're staring at the code for so long, you can't see what it does and doesn't do21:03
jrollrloo: I'm gonna just ship the clients then, if there's nothing else to land21:03
rloojroll: yup. i feel like we should have some rule, no more than 10 patch sets or something...21:03
jrolland we can ship this first thing in pike21:03
rloojroll: I'm good with that.21:03
jrollrloo: I mean as an author, but yeah :)21:03
*** [1]rpioso has joined #openstack-ironic21:04
rloojroll: it is hard to get eerything right the first time, and as an author it is hard to work on the same code again and again, but i wonder whether we should all slow down as authors and not push something up until we're close to darn sure...21:04
* rloo not an agile person21:05
jrollrloo: well, this one I couldn't keep in my head because it's so much in one patch21:05
jrollprobably should have just eaten a few API version bumps21:05
jrollrather than having it all in one21:05
jrolllooks like there's nothing else ready for ironicclient, so I'm releasing those now, yay21:06
*** alexpilotti has quit IRC21:07
rloojroll: maybe split the vendor out but it was fine.21:07
jrollrloo: just hard to tell if the patch does everything it should, when it's that big (for me)21:07
rloojroll: in that case, then better to split up :)21:08
jrollindeed. too late now, will be incredibly hard to untangle21:08
rloojroll: oh yeah, do NOT separate it now.21:09
*** aNuposic has joined #openstack-ironic21:16
*** Goneri has joined #openstack-ironic21:17
*** rushil has quit IRC21:19
openstackgerritMerged openstack/ironic-inspector: Cherry-pick fixes for Grenade  https://review.openstack.org/42196521:20
*** dsneddon is now known as dsneddon_afk21:21
*** hamzy has quit IRC21:22
*** dprince has quit IRC21:23
*** dsneddon has joined #openstack-ironic21:23
*** sayantani01 has quit IRC21:23
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests  https://review.openstack.org/42355621:27
*** jaosorior has quit IRC21:31
openstackgerritMario Villaplana proposed openstack/ironic: Agent driver rescue implementation  https://review.openstack.org/40043721:32
openstackgerritMario Villaplana proposed openstack/ironic: Add API and conductor methods for [un]rescue  https://review.openstack.org/35083121:32
*** sayantani01 has joined #openstack-ironic21:36
*** aNuposic has quit IRC21:36
*** aNuposic has joined #openstack-ironic21:37
*** rushil has joined #openstack-ironic21:41
openstackgerritJohn L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs  https://review.openstack.org/42542621:44
jlvillalTheJulia, JayF ^^^  Sort of a hacky way. Just strip out the ansi escape sequences. The file becomes readable at least.21:46
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP/DNM: Testing multinode job  https://review.openstack.org/42214921:46
TheJuliajlvillal: ohhh, I've added it to my queue so I can take a look once the jobs are done21:47
jlvillalTheJulia, Thanks :)21:47
openstackgerritJohn L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs  https://review.openstack.org/42542621:51
*** milan has quit IRC21:52
*** alexpilotti has joined #openstack-ironic21:59
*** alexpilotti has quit IRC22:03
*** moshele has quit IRC22:16
*** bfournie has left #openstack-ironic22:26
*** srobert_ has joined #openstack-ironic22:28
jlvillalIt's beautiful :) http://logs.openstack.org/26/425426/2/check/gate-tempest-dsvm-ironic-ipa-partition-agent_ipmitool-tinyipa-ubuntu-xenial/7f6547b/logs/ironic-bm-logs/node-0_no_ansi_2017-01-25-22:18:12.txt.gz22:29
jlvillalTheJulia, JayF jroll rloo: ^^^22:29
jlvillalAnd http://logs.openstack.org/26/425426/2/check/gate-tempest-dsvm-ironic-ipa-partition-agent_ipmitool-tinyipa-ubuntu-xenial/7f6547b/logs/ironic-bm-logs/22:29
* jlvillal has to fix the pep8 failure. Already has the fix but wants to wait for job to finish22:30
*** bfournie has joined #openstack-ironic22:30
*** srobert has quit IRC22:31
*** baoli has quit IRC22:31
*** srobert_ has quit IRC22:32
rloojlvillal: a work of art!22:32
jlvillalrloo, heh :)22:33
jlvillalrloo, It isn't perfect but it does make the log readable. Especially for the failure cases. And I think that is good enough22:33
*** jheroux has quit IRC22:33
rloojlvillal: ++22:33
*** sayantani01 has quit IRC22:40
*** sayantani01 has joined #openstack-ironic22:41
*** Sukhdev has joined #openstack-ironic22:41
*** cdearborn has quit IRC22:45
*** mgoddard has quit IRC22:51
*** bfournie has quit IRC22:51
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: Rescue extension for CoreOS with DHCP tenant networks  https://review.openstack.org/42352122:53
*** jkilpatr has quit IRC23:00
*** Goneri has quit IRC23:07
*** rushil has quit IRC23:07
openstackgerritJohn L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs  https://review.openstack.org/42542623:09
*** hecliunyx has quit IRC23:10
*** jcoufal has quit IRC23:15
*** krtaylor has quit IRC23:18
*** hamzy has joined #openstack-ironic23:18
*** hecliunyx has joined #openstack-ironic23:19
*** rushil has joined #openstack-ironic23:19
*** vsaienko has quit IRC23:20
openstackgerritJohn L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs  https://review.openstack.org/42545723:21
*** alexpilotti has joined #openstack-ironic23:23
*** alexpilotti has quit IRC23:23
*** newmember has quit IRC23:25
*** newmember has joined #openstack-ironic23:25
*** vsaienko has joined #openstack-ironic23:30
openstackgerritAnup Navare proposed openstack/ironic: Config drive support for Ceph  https://review.openstack.org/40763923:33
*** rushil has quit IRC23:33
soliosgmariojv: regarding the last comment in https://review.openstack.org/#/c/400437/1623:39
patchbotpatch 400437 - ironic - Agent driver rescue implementation23:39
soliosgmariojv: looks like the node is not transitioning to one of the states in _LOOKUP_ALLOWED_STATES23:40
*** jkilpatr has joined #openstack-ironic23:41
soliosgmariojv: https://review.openstack.org/#/c/350831/26/ironic/api/controllers/v1/ramdisk.py@4223:41
patchbotpatch 350831 - ironic - Add API and conductor methods for [un]rescue23:41
openstackgerritAnup Navare proposed openstack/ironic: Config drive support for Ceph  https://review.openstack.org/40763923:42
*** sayantani01 has quit IRC23:45
*** sayantani01 has joined #openstack-ironic23:45
*** baoli has joined #openstack-ironic23:47
*** rushil has joined #openstack-ironic23:53
openstackgerritJoanna Taryma proposed openstack/ironic: WIP host ipxe static assets in devstack on :80  https://review.openstack.org/42437423:57

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