*** awiddersheim has joined #openstack-ironic | 00:05 | |
*** alexpilotti has joined #openstack-ironic | 00:09 | |
*** alexpilotti has quit IRC | 00:15 | |
*** Guest9757 is now known as rook | 00:21 | |
openstackgerrit | Joanna Taryma proposed openstack/ironic: WIP host ipxe static assets in devstack on :80 https://review.openstack.org/424374 | 00:24 |
---|---|---|
openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: WIP/DNM: Migrate ironic-tempest-plugin to new repo https://review.openstack.org/419602 | 00:26 |
*** mtanino has joined #openstack-ironic | 00:26 | |
*** rpioso has quit IRC | 00:27 | |
*** mtanino_ has quit IRC | 00:28 | |
*** mtanino has quit IRC | 00:32 | |
TheJulia | JayF: I just +2+A'ed the IPA rev | 00:32 |
*** aNuposic has quit IRC | 00:33 | |
*** hoangcx has joined #openstack-ironic | 00:39 | |
*** alexpilotti has joined #openstack-ironic | 00:41 | |
openstackgerrit | Solio Sarabia proposed openstack/ironic-tempest-plugin: WIP/DNM: Migrate ironic-tempest-plugin to new repo https://review.openstack.org/419602 | 00:42 |
*** hw_wutianwei has joined #openstack-ironic | 00:43 | |
*** yolanda has quit IRC | 00:45 | |
TheJulia | vinsh: 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-ironic | 00:49 | |
*** yolanda has joined #openstack-ironic | 00:51 | |
*** yolanda has quit IRC | 00:53 | |
*** aNuposic has joined #openstack-ironic | 00:53 | |
vinsh | I turned on that log gathering for failed deploys, good feature | 00:54 |
vinsh | I'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 |
vinsh | I just realized that ironic sort of assumes that your glance is backed by swift.. or if its not it glance backed by rados | 00:56 |
vinsh | we have glance backed by rbd | 00:56 |
vinsh | so. back to fix that up. | 00:56 |
TheJulia | fun :\ Yeah, agent drivers do expect swift backed | 00:58 |
*** yolanda has joined #openstack-ironic | 01:00 | |
*** SerenaFeng has joined #openstack-ironic | 01:01 | |
*** phuongnh has joined #openstack-ironic | 01:01 | |
tovin07 | JayF, hello | 01:03 |
tovin07 | rloo, hello | 01:04 |
*** yolanda has quit IRC | 01:04 | |
*** vinsh has quit IRC | 01:06 | |
*** rajinir has quit IRC | 01:06 | |
*** ChubYann has quit IRC | 01:13 | |
*** Syed__ has quit IRC | 01:15 | |
openstackgerrit | Merged openstack/ironic-python-agent: Support psutil 5.0.1 https://review.openstack.org/424895 | 01:21 |
*** rama_y has quit IRC | 01:22 | |
*** mtanino has joined #openstack-ironic | 01:25 | |
*** ChubYann has joined #openstack-ironic | 01:25 | |
TheJulia | Updated the etherpad | 01:36 |
*** sayantani01 has joined #openstack-ironic | 01:37 | |
*** sayantani01 has quit IRC | 01:37 | |
*** sayantani01 has joined #openstack-ironic | 01:37 | |
*** aNuposic has quit IRC | 01:44 | |
*** Sukhdev has quit IRC | 01:45 | |
*** rloo has quit IRC | 01:47 | |
*** mtanino has quit IRC | 02:12 | |
*** sacharya has joined #openstack-ironic | 02:15 | |
*** SerenaFeng has quit IRC | 02:54 | |
*** SerenaFeng has joined #openstack-ironic | 02:54 | |
*** ijw has quit IRC | 03:21 | |
*** nmathew has joined #openstack-ironic | 03:23 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Update proliantutils version required for Ocata release https://review.openstack.org/424593 | 03:27 |
*** vikrant has joined #openstack-ironic | 03:32 | |
*** udesale has joined #openstack-ironic | 03:33 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 03:38 |
*** gcb has joined #openstack-ironic | 03:38 | |
openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 03:39 |
openstackgerrit | Dao Cong Tien proposed openstack/ironic: Adds clean step 'restore_bios_config' to iRMC drivers https://review.openstack.org/416403 | 03:40 |
*** ijw has joined #openstack-ironic | 03:42 | |
*** vinsh has joined #openstack-ironic | 03:42 | |
*** vinsh has quit IRC | 03:43 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add nodes tagging support - objs https://review.openstack.org/233357 | 03:44 |
*** bfournie has quit IRC | 03:45 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add DB layer support for node tags filter https://review.openstack.org/272624 | 03:46 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add nodes tagging support - API https://review.openstack.org/250478 | 03:46 |
*** ijw has quit IRC | 03:46 | |
*** SerenaFeng has quit IRC | 03:57 | |
*** tovin07 has quit IRC | 04:02 | |
*** bfournie has joined #openstack-ironic | 04:18 | |
*** bfournie has quit IRC | 04:20 | |
*** Sukhdev has joined #openstack-ironic | 04:37 | |
*** sasha2 has quit IRC | 04:42 | |
*** gcb has quit IRC | 04:43 | |
*** gcb has joined #openstack-ironic | 04:44 | |
*** rbudden has quit IRC | 04:58 | |
*** udesale__ has joined #openstack-ironic | 05:17 | |
*** udesale has quit IRC | 05:17 | |
*** udesale has joined #openstack-ironic | 05:19 | |
*** udesale__ has quit IRC | 05:21 | |
openstackgerrit | Merged openstack/python-ironicclient: Strip endpoint version in OSC plugin https://review.openstack.org/364833 | 05:31 |
*** zackf has quit IRC | 05:37 | |
*** pester has quit IRC | 05:45 | |
openstackgerrit | Merged openstack/python-ironicclient: Add --wait to OSC provisioning commands https://review.openstack.org/354806 | 05:50 |
openstackgerrit | Merged openstack/python-ironicclient: Extend OSC "node list" cmd to fetch nodes without instance UUID https://review.openstack.org/369279 | 05:50 |
*** sacharya has quit IRC | 05:51 | |
*** SerenaFeng has joined #openstack-ironic | 05:52 | |
openstackgerrit | Merged openstack/bifrost: Fix endpoint service name for inspector https://review.openstack.org/424227 | 06:00 |
openstackgerrit | Karthik S proposed openstack/ironic-python-agent: WIP: NUMA-topology collector https://review.openstack.org/424729 | 06:13 |
*** slagle has quit IRC | 06:15 | |
*** SerenaFeng has quit IRC | 06:15 | |
*** xavierr has quit IRC | 06:15 | |
*** xavierr has joined #openstack-ironic | 06:16 | |
*** slagle has joined #openstack-ironic | 06:17 | |
openstackgerrit | Dao Cong Tien proposed openstack/ironic: Adds clean step 'restore_bios_config' to iRMC drivers https://review.openstack.org/416403 | 06:28 |
*** zackf has joined #openstack-ironic | 06:28 | |
*** dsneddon has quit IRC | 06:31 | |
*** zackf has quit IRC | 06:32 | |
*** noama has joined #openstack-ironic | 06:37 | |
*** Sukhdev has quit IRC | 06:39 | |
*** ChubYann has quit IRC | 06:47 | |
*** mjura has joined #openstack-ironic | 06:48 | |
*** Nisha_Agarwal has joined #openstack-ironic | 06:50 | |
*** yolanda has joined #openstack-ironic | 06:51 | |
*** makowals_ has quit IRC | 06:53 | |
*** tovin07 has joined #openstack-ironic | 06:54 | |
*** udesale__ has joined #openstack-ironic | 06:56 | |
*** hoangcx_ has joined #openstack-ironic | 06:57 | |
*** hoangcx has quit IRC | 07:00 | |
*** makowals has joined #openstack-ironic | 07:06 | |
*** aparnav has joined #openstack-ironic | 07:07 | |
*** nmathew- has joined #openstack-ironic | 07:09 | |
aparnav | ianw, Can you review the patch https://review.openstack.org/#/c/396504/ ? It is there for a long time. | 07:11 |
patchbot | patch 396504 - diskimage-builder - Update hpssacli to ssacli in proliant-tools element | 07:11 |
*** nmathew has quit IRC | 07:11 | |
*** nmathew has joined #openstack-ironic | 07:12 | |
*** nmathew- has quit IRC | 07:14 | |
*** pcaruana has joined #openstack-ironic | 07:28 | |
openstackgerrit | jiang wei proposed openstack/python-ironicclient: Support i18n for baremetal node cmds https://review.openstack.org/357639 | 07:29 |
*** tovin07 has quit IRC | 07:32 | |
*** nmathew- has joined #openstack-ironic | 07:33 | |
*** tesseract has joined #openstack-ironic | 07:34 | |
*** nmathew has quit IRC | 07:35 | |
*** sayantan_ has joined #openstack-ironic | 07:36 | |
*** sayantani01 has quit IRC | 07:38 | |
*** mgoddard has joined #openstack-ironic | 07:39 | |
*** tovin07 has joined #openstack-ironic | 07:45 | |
*** hoangcx has joined #openstack-ironic | 07:46 | |
openstackgerrit | jiang wei proposed openstack/python-ironicclient: Support i18n for baremetal node cmds https://review.openstack.org/357639 | 07:46 |
*** moshele has joined #openstack-ironic | 07:48 | |
*** e0ne has joined #openstack-ironic | 07:48 | |
*** hoangcx_ has quit IRC | 07:48 | |
*** lennyb has joined #openstack-ironic | 07:53 | |
*** e0ne has quit IRC | 07:57 | |
*** lennyb has quit IRC | 07:57 | |
*** e0ne has joined #openstack-ironic | 08:01 | |
openstackgerrit | Merged openstack/bifrost: Add wait_for_node_deploy parameter to deploy task with custom instance_info https://review.openstack.org/424244 | 08:05 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-specs: Dynamically allocate console ports https://review.openstack.org/249876 | 08:06 |
*** lennyb_ has joined #openstack-ironic | 08:09 | |
*** lennyb_ has quit IRC | 08:09 | |
*** lennyb has joined #openstack-ironic | 08:10 | |
*** yonglihe has joined #openstack-ironic | 08:11 | |
*** tovin07 has quit IRC | 08:13 | |
vdrok | good morning ironic! | 08:21 |
*** ralonsoh has joined #openstack-ironic | 08:24 | |
*** rbudden has joined #openstack-ironic | 08:25 | |
*** tovin07 has joined #openstack-ironic | 08:29 | |
*** e0ne has quit IRC | 08:30 | |
*** amarao has joined #openstack-ironic | 08:33 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: [WIP] Migrate to session for glance client https://review.openstack.org/410767 | 08:34 |
*** moshele has quit IRC | 08:34 | |
*** moshele has joined #openstack-ironic | 08:34 | |
openstackgerrit | Anh Tran proposed openstack/bifrost: Typo fix: insufficent = insufficient https://review.openstack.org/425053 | 08:36 |
*** amotoki has quit IRC | 08:47 | |
*** bfournie has joined #openstack-ironic | 08:54 | |
*** rbartal has joined #openstack-ironic | 08:56 | |
*** bcornec1 has joined #openstack-ironic | 08:59 | |
*** hoangcx has quit IRC | 09:00 | |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #openstack-ironic | 09:00 | |
*** tovin07 has quit IRC | 09:00 | |
*** dtantsur|afk is now known as dtantsur | 09:02 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Make grenade actually upgrade our source code https://review.openstack.org/425066 | 09:03 |
dtantsur | morning Ironic | 09:04 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/425067 | 09:05 |
vdrok | good morning dtantsur | 09:07 |
*** sayantan_ has quit IRC | 09:07 | |
*** newmember has joined #openstack-ironic | 09:08 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2 https://review.openstack.org/425068 | 09:09 |
*** rbudden has quit IRC | 09:11 | |
*** pmannidi has quit IRC | 09:14 | |
*** tovin07 has joined #openstack-ironic | 09:15 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Change (started|finished)_at column type https://review.openstack.org/397636 | 09:23 |
*** bcornec1 has quit IRC | 09:24 | |
*** alineb_ has joined #openstack-ironic | 09:26 | |
*** lucas-afk is now known as lucasagomes | 09:27 | |
lucasagomes | morning all | 09:27 |
vdrok | morning lucasagomes | 09:30 |
*** bcornec1 has joined #openstack-ironic | 09:33 | |
openstackgerrit | Aparna proposed openstack/ironic: Doc for sanitize erase support in iLO drivers https://review.openstack.org/408441 | 09:38 |
*** tovin07 has quit IRC | 09:41 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Update proliantutils version required for Ocata release https://review.openstack.org/424593 | 09:42 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off https://review.openstack.org/216743 | 09:43 |
dtantsur | sambetts|afk, mind reviewing https://review.openstack.org/#/c/417871/ please? seems like folks wait for someone knowing about inspector to approve :) | 09:47 |
patchbot | patch 417871 - ironic-python-agent - Remove compatibility with old bash-based introspec... | 09:47 |
*** udesale__ has quit IRC | 09:47 | |
dtantsur | morning lucasagomes, vdrok | 09:47 |
lucasagomes | o/ | 09:47 |
*** udesale has quit IRC | 09:47 | |
*** derekh has joined #openstack-ironic | 09:47 | |
*** strigazi_AFK is now known as strigazi | 09:50 | |
*** e0ne has joined #openstack-ironic | 09:52 | |
*** athomas has joined #openstack-ironic | 09:53 | |
*** MattMan has quit IRC | 09:54 | |
*** MattMan has joined #openstack-ironic | 09:55 | |
*** Nisha_Agarwal has quit IRC | 09:55 | |
*** mgoddard has quit IRC | 10:03 | |
pas-ha | morning all :) | 10:08 |
vdrok | morning pas-ha | 10:10 |
dtantsur | morning pas-ha | 10:12 |
pas-ha | dtantsur: sorry to miss your pings, I will propose backports for PXEBoot take-over now | 10:12 |
dtantsur | thanks! | 10:12 |
dtantsur | and no worries - it's busy time now :) | 10:12 |
*** sambetts|afk is now known as sambetts | 10:13 | |
sambetts | dtantsur: taking a look now | 10:14 |
*** deray has joined #openstack-ironic | 10:15 | |
sambetts | dtantsur: when did we deprecate the bash ramdisk, 1 or 2 cycles ago? | 10:15 |
deray | hi Ironicers! g'morning | 10:16 |
deray | dtantsur, sorry to bother you .. but could you bless this? https://review.openstack.org/#/c/418612/ | 10:17 |
patchbot | patch 418612 - futurist - Introducing NeverAgain functionality for periodics | 10:17 |
*** phuongnh has quit IRC | 10:17 | |
*** openstackgerrit has quit IRC | 10:17 | |
* sambetts is kind of excited about that ^ | 10:18 | |
*** amotoki has joined #openstack-ironic | 10:19 | |
*** mgoddard has joined #openstack-ironic | 10:19 | |
deray | sambetts, hey .. please relook (if you have time tho :-) ) | 10:19 |
*** nmathew has joined #openstack-ironic | 10:19 | |
*** Nisha_ has joined #openstack-ironic | 10:20 | |
*** openstackgerrit has joined #openstack-ironic | 10:21 | |
openstackgerrit | Merged openstack/bifrost: Add socat package https://review.openstack.org/424255 | 10:21 |
*** nmathew- has quit IRC | 10:23 | |
*** nmathew- has joined #openstack-ironic | 10:24 | |
*** nmathew- has quit IRC | 10:24 | |
openstackgerrit | Andrey Shestakov proposed openstack/bifrost: Add support of collecting logs from baremetal nodes https://review.openstack.org/415503 | 10:27 |
openstackgerrit | Markos Chandras proposed openstack/bifrost: roles: bifrost-ironic-install: SUSE/Debian: Add gcc and python-pip packages https://review.openstack.org/424853 | 10:28 |
vdrok | morning sambetts and deray | 10:28 |
sambetts | hey vdrok | 10:28 |
*** Nisha_ has quit IRC | 10:29 | |
*** aarefiev_afk is now known as aarefiev | 10:32 | |
aarefiev | morning all! | 10:32 |
vdrok | aarefiev: morning | 10:32 |
sambetts | o/ aarefiev | 10:32 |
pas-ha | btw, seems stable/newton functional gates for ironicclient are broken https://review.openstack.org/#/c/424779/ | 10:34 |
patchbot | patch 424779 - python-ironicclient (stable/newton) - DNM test stable/newton gates | 10:34 |
pas-ha | although there's not much activity in this branch, still should be fixed IMO | 10:34 |
*** Nisha_ has joined #openstack-ironic | 10:35 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: [Devstack] Allow add several ports to node https://review.openstack.org/425112 | 10:38 |
deray | vdrok, morning o/ | 10:44 |
deray | aarefiev, o/ | 10:44 |
dtantsur | sambetts, I think we don't run its jobs on newton | 10:48 |
dtantsur | deray, will get to it | 10:48 |
deray | dtantsur, cool | 10:48 |
deray | dtantsur, tyot | 10:48 |
openstackgerrit | Andrey Shestakov proposed openstack/bifrost: Allow enroll-dynamic on remote host https://review.openstack.org/411299 | 10:49 |
openstackgerrit | Andrey Shestakov proposed openstack/bifrost: Allow deploy-dynamic on remote host https://review.openstack.org/411304 | 10:49 |
deray | sambetts, yeah true .. second dtantsur.. we have moved on to ``futurist`` from Mitaka I suppose | 10:50 |
sambetts | deray: 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 Ironic | 10:52 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2 https://review.openstack.org/425068 | 10:53 |
*** Nisha_ has quit IRC | 10:53 | |
deray | sambetts, okay | 10:56 |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Grenade: don't enable tempest plugins in settings https://review.openstack.org/425122 | 10:57 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 10:59 |
*** newmember has quit IRC | 11:00 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix take over for ACTIVE nodes in PXEBoot https://review.openstack.org/425124 | 11:02 |
pas-ha | dtantsur: ^ | 11:02 |
pas-ha | would you like to have the agent driver fix re takeover backported too? | 11:03 |
dtantsur | I guess it's reasonable | 11:03 |
dtantsur | also thanks | 11:03 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 11:05 |
openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 11:07 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to attach/detach VIFs to active ironic nodes https://review.openstack.org/424723 | 11:07 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Allow setting dhcp_provider in devstack https://review.openstack.org/425127 | 11:08 |
vdrok | dtantsur: jroll pas-ha https://review.openstack.org/425127 fix for the stable/newton functional tests in client | 11:08 |
patchbot | patch 425127 - ironic (stable/newton) - Allow setting dhcp_provider in devstack | 11:08 |
dtantsur | k | 11:09 |
*** deray has quit IRC | 11:09 | |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: DNM test stable/newton gates https://review.openstack.org/424779 | 11:10 |
vdrok | ouch, one extra line :) | 11:10 |
vsaienk0 | morning Ironicers! | 11:10 |
vsaienk0 | dtantsur: not sure if you already saw https://review.openstack.org/#/c/423556 I hope it will help us to decrease number of CI jobs | 11:10 |
patchbot | patch 423556 - ironic - WIP: ironic standalone tests | 11:10 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Allow setting dhcp_provider in devstack https://review.openstack.org/425127 | 11:11 |
dtantsur | vsaienk0, morning! thanks, I'll take a look when I have a minute. | 11:19 |
vsaienk0 | dtantsur: thanks! | 11:20 |
* dtantsur thinks he finally has to repair inspector's stable/newton, sigh.. :) | 11:24 | |
*** dtantsur is now known as dtantsur|lunch | 11:27 | |
*** jkilpatr_ has quit IRC | 11:37 | |
*** aparnav has quit IRC | 11:37 | |
*** mgoddard has quit IRC | 11:45 | |
jroll | morning y'all | 11:49 |
jroll | thanks for driver composition reviews, fixing those up now | 11:49 |
openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient: Add args to CLI 'node-create' for selecting hardware interfaces https://review.openstack.org/418359 | 11:49 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-specs: Amend dynamic iPXE configuration spec https://review.openstack.org/392290 | 11:50 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 11:51 |
*** jkilpatr has joined #openstack-ironic | 11:54 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Support --os-baremetal-api-version latest https://review.openstack.org/420659 | 11:57 |
dtantsur|lunch | morning jroll | 11:58 |
*** dtantsur|lunch is now known as dtantsur | 11:58 | |
openstackgerrit | Merged openstack/ironic: Use global vars for storing image deploy path's https://review.openstack.org/420724 | 12:01 |
*** lucasagomes is now known as lucas-hungry | 12:01 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2 https://review.openstack.org/425068 | 12:04 |
*** moshele has quit IRC | 12:05 | |
*** moshele has joined #openstack-ironic | 12:05 | |
dtantsur | jroll, so, what's the release plan for today? | 12:08 |
jroll | dtantsur: I haven't looked at the queue yet today, how are things looking? | 12:08 |
jroll | I'd also like to clean up release notes | 12:08 |
dtantsur | jroll, ironicclient looks good, I wonder if we want driver comp bits in (blocked by API) | 12:09 |
dtantsur | in ironic-inspector-client we have an outstanding patch for LLDP, which is blocked on ironic-inspector part as well | 12:09 |
dtantsur | dunno if we want both in | 12:09 |
*** d0ugal has quit IRC | 12:10 | |
jroll | dtantsur: well, it would be nice, are both of those ready to land? | 12:10 |
dtantsur | the former rather yes, the latter not quite | 12:11 |
jroll | ah | 12:11 |
jroll | dtantsur: I should have new API patches in about 30 minutes | 12:12 |
jroll | so if we get those approved before you leave today, I say we wait for driver comp bits | 12:12 |
jroll | and go ahead with inspector without lldp | 12:12 |
*** charlle-afk has quit IRC | 12:12 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Cherry-pick fixes for Grenade https://review.openstack.org/421965 | 12:17 |
dtantsur | jroll, ack | 12:17 |
*** d0ugal has joined #openstack-ironic | 12:18 | |
vsaienk0 | jroll, sambetts: I'm wondered if we can land this in Ocata https://review.openstack.org/#/c/419975/ | 12:24 |
patchbot | patch 419975 - nova - Implement interface attach/detach in ironic virt d... | 12:24 |
jroll | vsaienk0: tomorrow is nova's feature freeze and we don't even have a blueprint describing this | 12:25 |
jroll | and it's in merge conflict | 12:26 |
*** cdaniel has joined #openstack-ironic | 12:27 | |
* dtantsur for some reason assumed that it was part of portgroup work.. | 12:27 | |
jroll | dtantsur: well, it doesn't always "just work", e.g. environments without dhcp | 12:28 |
dtantsur | ah, ok | 12:28 |
jroll | and in the vlan-aware world it will need instance config | 12:28 |
cdaniel | morning all | 12:28 |
dtantsur | damn gerrit Y U SO SLOW | 12:28 |
jroll | ++ | 12:28 |
vdrok | morning jroll | 12:29 |
vdrok | and morning cdaniel | 12:29 |
jroll | morning cdaniel, vdrok | 12:29 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Allow diskless nodes introspection https://review.openstack.org/326620 | 12:31 |
vsaienk0 | jroll: I thought that it might be related to attach/detach work that was done | 12:31 |
jroll | vsaienk0: it's related, but the blueprint didn't specify this | 12:32 |
*** ccamacho is now known as ccamacho|lunch | 12:33 | |
cdaniel | morning vdrok, jroll | 12:33 |
jroll | vsaienk0: I'm not going to ask nova to accept it, one day before FF, without having it in the BP | 12:33 |
vsaienk0 | jroll: yeah, I understood, hope we can merge that in Pike, I'm going to create a bp | 12:33 |
jroll | vsaienk0: agree, thanks | 12:34 |
vsaienk0 | jroll: found already existed bp https://blueprints.launchpad.net/nova/+spec/ironic-attach-detach-interface but it is not approved | 12:35 |
*** vikrant has quit IRC | 12:37 | |
openstackgerrit | Merged openstack/ironic-inspector: Change (started|finished)_at column type https://review.openstack.org/397636 | 12:38 |
sambetts | jroll, 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 state | 12:40 |
sambetts | vsaienk0: I think you had one right? is it approved? | 12:40 |
vsaienk0 | sambetts: I have a code already https://review.openstack.org/#/c/424723/ but I didn't open RFE | 12:41 |
patchbot | patch 424723 - ironic - Allow to attach/detach VIFs to active ironic nodes | 12:41 |
*** rbartal has quit IRC | 12:41 | |
vsaienk0 | sambetts: going to create RFE for this on ironic side | 12:41 |
jroll | +1 | 12:43 |
sambetts | vsaienk0: 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 side | 12:43 |
mat128 | sambetts, vsaienk0: if attach/detach isnt to be used on active nodes, what is it for? | 12:45 |
sambetts | mat128: so the idea is that post-deploy a user could call nova interface-attach and attach a new neutron port | 12:46 |
-openstackstatus- NOTICE: Gerrit is going to be restarted due to slow performance | 12:46 | |
sambetts | mat128: its a supported feature for VMs | 12:46 |
vsaienk0 | mat128 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 |
mat128 | sambetts: I know this (we have it already on BM), but what is the purpose of attaching other than that? | 12:47 |
mat128 | vsaienk0: ahh, got it | 12:47 |
vsaienk0 | sambetts: it shouldn't be tied to nova as we may support it in standalone. | 12:47 |
*** nmathew- has joined #openstack-ironic | 12:48 | |
sambetts | vsaienk0: 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 bug | 12:49 |
jroll | sambetts: +1 | 12:49 |
-openstackstatus- NOTICE: Gerrit has been successfully restarted | 12:49 | |
vsaienk0 | sambetts: 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 Nova | 12:51 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API https://review.openstack.org/424720 | 12:51 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API https://review.openstack.org/417970 | 12:51 |
jroll | vdrok: dtantsur: addressed comments, let's get this done ^^ | 12:51 |
dtantsur | ok, jumping on a meeting now, will check afterwards | 12:51 |
*** nmathew has quit IRC | 12:51 | |
sambetts | vsaienk0: for flat case we'll need to reject the request, so we'll still need to handle what happens if the node is in active | 12:52 |
jroll | cool | 12:52 |
vsaienk0 | sambetts, 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 |
openstack | Launchpad bug 1582188 in Ironic "[RFE] Add interface attach API" [Wishlist,Fix released] - Assigned to Vasyl Saienko (vsaienko) | 12:52 |
* jroll gets breakfast and such | 12:52 | |
jroll | vsaienk0: a new one, please | 12:52 |
jroll | that one is approved and completed already | 12:52 |
*** nmathew- has quit IRC | 12:55 | |
*** rbartal has joined #openstack-ironic | 12:56 | |
*** moshele has quit IRC | 12:57 | |
openstackgerrit | Noam Angel proposed openstack/ironic-inspector: [devstack] Remove bridge only if it's not OVB_PHYSICAL_BRIDGE https://review.openstack.org/423758 | 13:00 |
*** lucas-hungry is now known as lucasagomes | 13:00 | |
vsaienk0 | jroll, sambetts: https://bugs.launchpad.net/ironic/+bug/1659282 | 13:02 |
openstack | Launchpad bug 1659282 in Ironic "[RFE] Allow to attach/detach VIF to active Ironic nodes." [Undecided,New] | 13:02 |
vsaienk0 | sambetts: so do you think we should block vif attach/detach to/from active ironic nodes at the moment? | 13:02 |
*** hw_wutianwei has quit IRC | 13:03 | |
sambetts | vsaienk0: 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 that | 13:04 |
sambetts | ;) | 13:04 |
vsaienk0 | sambetts: nova just check when call attach, detach will trigger ironic vif detach. | 13:05 |
sambetts | ? | 13:05 |
sambetts | we block people calling attach/detach via nova on an active instance via https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L133 | 13:06 |
vsaienk0 | sambetts: 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 detach | 13:07 |
patchbot | patch 419975 - nova - Implement interface attach/detach in ironic virt d... | 13:07 |
vsaienk0 | sambetts: but anyway detach_interface() is not implemented so nothing worry about :) | 13:08 |
sambetts | yeah there is also that :-P | 13:08 |
ricardoas | morning ironic | 13:16 |
*** ccamacho|lunch is now known as ccamacho | 13:19 | |
*** vsaienko has joined #openstack-ironic | 13:27 | |
*** rbudden has joined #openstack-ironic | 13:28 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 13:31 |
*** moshele has joined #openstack-ironic | 13:33 | |
*** udesale has joined #openstack-ironic | 13:33 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2 https://review.openstack.org/425068 | 13:41 |
*** vsaienko has quit IRC | 13:47 | |
*** hw_wutianwei has joined #openstack-ironic | 13: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-ironic | 13:51 | |
*** baoli has joined #openstack-ironic | 13:58 | |
dtantsur | jroll, something potentially nice to get in the ironicclient release https://review.openstack.org/#/c/420659/ | 13:59 |
patchbot | patch 420659 - python-ironicclient - Support --os-baremetal-api-version latest | 13:59 |
*** baoli_ has joined #openstack-ironic | 14:00 | |
*** rloo has joined #openstack-ironic | 14:03 | |
*** jcoufal has joined #openstack-ironic | 14:03 | |
*** baoli has quit IRC | 14:03 | |
*** vsaienko has quit IRC | 14:04 | |
*** srobert has joined #openstack-ironic | 14:05 | |
*** srobert has quit IRC | 14:05 | |
*** bfournie has quit IRC | 14:05 | |
*** srobert has joined #openstack-ironic | 14:06 | |
*** vsaienko has joined #openstack-ironic | 14:07 | |
*** vsaienko has quit IRC | 14:10 | |
hwoarang | Hello. 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 me | 14:12 |
*** zzzeek has quit IRC | 14:12 | |
*** Goneri has joined #openstack-ironic | 14:13 | |
*** zzzeek has joined #openstack-ironic | 14:14 | |
TheJulia | Good morning everyone | 14:20 |
*** dprince has joined #openstack-ironic | 14:21 | |
dtantsur | morning TheJulia | 14:21 |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Allow creating portgroups via create commands https://review.openstack.org/425199 | 14:22 |
vdrok | morning TheJulia | 14:22 |
openstackgerrit | Merged openstack/ironic: Allow setting dhcp_provider in devstack https://review.openstack.org/425127 | 14:22 |
dtantsur | hwoarang, I haven't seen it, maybe it worth chatting with Neutron folks (the failure seems to happen in neutronclient) | 14:23 |
lucasagomes | TheJulia, hi there | 14:23 |
hwoarang | dtantsur: will do thanks | 14:23 |
vdrok | dtantsur: 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 |
patchbot | patch 425199 - python-ironicclient - Allow creating portgroups via create commands | 14:23 |
* dtantsur adds to his review list | 14:24 | |
rloo | vdrok: thx for that; i was wondering the other day. will look today. | 14:24 |
rloo | hi and good morning vdrok, dtantsur, TheJulia, lucasagomes | 14:24 |
vdrok | thanks! and sorry for doing it this late | 14:24 |
lucasagomes | o/ | 14:24 |
dtantsur | morning rloo | 14:24 |
rloo | TheJulia: thx for reviewing and landing those client patches last night! | 14:24 |
rloo | vdrok: better late than never ;) | 14:25 |
TheJulia | rloo: 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 | |
rloo | TheJulia: no worries, as long as we get the ones I want! landed by today :) | 14:27 |
TheJulia | rloo: :) | 14:27 |
* jroll is back | 14:29 | |
TheJulia | What | 14:30 |
TheJulia | Err, wb | 14:30 |
vdrok | jroll: have you seen my comments here - https://review.openstack.org/#/c/417970/18/ironic/conductor/manager.py ? Am I missing something? | 14:33 |
patchbot | patch 417970 - ironic - Add dynamic driver functionality to REST API | 14:33 |
jroll | vdrok: yeah, I, uhhh | 14:35 |
jroll | thought I responded | 14:35 |
jroll | we have a vendor attribute on the hardware types returned there | 14:35 |
* jroll grabs laptop and publishes comments | 14:35 | |
*** udesale has quit IRC | 14:36 | |
jroll | vdrok: and, published | 14:36 |
jroll | I can walk you through the code if you like | 14:36 |
dtantsur | please remove the fake bit from generic.py, then it lgtm | 14:37 |
jroll | did I not? oops | 14:37 |
jroll | oh wait, I wonder if vdrok is right O_o | 14:38 |
vdrok | jroll: yeah, build_driver_for_task is not called AFAI can see | 14:39 |
jroll | hrm | 14:39 |
*** vsaienko has joined #openstack-ironic | 14:40 | |
*** bfournie has joined #openstack-ironic | 14:40 | |
jroll | oh, seems I'm testing that | 14:40 |
jroll | whomp | 14:40 |
jroll | ok I'll fix that up, thanks vdrok | 14:41 |
vdrok | np :) | 14:41 |
* jroll wonders why we mock the vendor interface out in driver passthru tests, that's annoying | 14:42 | |
*** gcb has quit IRC | 14:42 | |
ricardoas | hello everyone! | 14:43 |
vdrok | morning ricardoas | 14:43 |
ricardoas | I got a question about third party ci... is there any recommendation for running backports too? | 14:43 |
jroll | hi ricardoas :) | 14:43 |
lucasagomes | jlvillal, 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-ironic | 14:43 | |
jroll | ricardoas: 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 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 14:44 |
jlvillal | lucasagomes, I don't think so. Also I'm not sure about "jobs". It was the one job I am working on. | 14:44 |
ricardoas | i'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/newton | 14:45 |
lucasagomes | jlvillal, right one, do we have a bug for it or something ? I'm kinda interested to see what is actually failing | 14:45 |
*** mrtenio has joined #openstack-ironic | 14:46 | |
jlvillal | lucasagomes, No bug. This is still an under development gate job for multi-tenant + grenade. And then adding multi-node in the future. | 14:46 |
ricardoas | but they are not passing as currently configuration is not compatible with devstack for stable/newton, jroll | 14:46 |
jroll | ricardoas: file a bug? | 14:47 |
lucasagomes | jlvillal, right on | 14:47 |
jroll | ricardoas: or you mean, your CI config is not compatible | 14:47 |
ricardoas | jroll, yes! that's it... we stack master but we're not able to stack newton with the same ci configuration... | 14:48 |
jroll | ricardoas: should be a ZUUL_BRANCH variable you can use for config differences, or something like that | 14:48 |
*** hw_wutianwei has quit IRC | 14:50 | |
*** mrtenio has quit IRC | 14:50 | |
ricardoas | jroll, ok... i'll take a look! The solution then is to maintain a specific configuration for every ZUUL_BRANCH, right? | 14:50 |
jroll | ricardoas: I'd probably just have if statements for particular pieces, lemme see if I can find an example | 14:51 |
ricardoas | jroll thanks! that'd help a lot | 14:51 |
jroll | ricardoas: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L175-L182 | 14:52 |
*** udesale has joined #openstack-ironic | 15:00 | |
lucasagomes | jroll, 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 use | 15:01 |
lucasagomes | jroll, do you know if -infra can store it for us ? (/me probably should have asked at -infra...) | 15:02 |
jroll | lucasagomes: yeah, I'd recommend we just store it on tarballs.o.o, like we do for agents | 15:02 |
jroll | put the builder code in the ironic repo (or maybe elsewhere, dunno), and build/publish if it changes | 15:03 |
jroll | as a post job | 15:03 |
lucasagomes | jroll, right on... but to get it there ? We should have something like apost job irght ? | 15:03 |
lucasagomes | right | 15:03 |
jroll | yep | 15:03 |
* lucasagomes looks into it | 15:03 | |
jroll | lucasagomes: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic-python-agent-jobs.yaml | 15:03 |
sambetts | jroll, 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 tarballs | 15:03 |
jroll | is how we do IPA | 15:03 |
openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support https://review.openstack.org/264257 | 15:03 |
jroll | sambetts: yeah, we should do that thing | 15:04 |
*** dtantsur is now known as dtantsur|brb | 15:04 | |
openstackgerrit | Xavier proposed openstack/ironic: Validation before perform node deallocation https://review.openstack.org/386014 | 15:04 |
lucasagomes | sambetts, right on, yeah it would make sense to have a separated repo to keep it organized | 15:04 |
jroll | I wonder if we should just have a openstack/test-images repo or something | 15:05 |
pas-ha | jroll: + | 15:05 |
lucasagomes | sambetts, any progress on it from the neutron part ? Or perhaps we should just take the first step ? | 15:05 |
lucasagomes | jroll, yeah, reads good | 15:05 |
sambetts | lucasagomes: nope no progress only an idea | 15:07 |
*** sayantani01 has joined #openstack-ironic | 15:07 | |
*** dyasny has quit IRC | 15:09 | |
*** zackf has joined #openstack-ironic | 15:09 | |
*** jaosorior has joined #openstack-ironic | 15:10 | |
*** yufei has joined #openstack-ironic | 15:17 | |
*** sasha2 has joined #openstack-ironic | 15:18 | |
*** yufei_ has joined #openstack-ironic | 15:21 | |
*** dyasny has joined #openstack-ironic | 15:21 | |
*** yufei has quit IRC | 15:25 | |
*** rajinir has joined #openstack-ironic | 15:27 | |
*** Syed__ has joined #openstack-ironic | 15:27 | |
*** baoli_ has quit IRC | 15:29 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: ETAGs maintenance for all api node's requests https://review.openstack.org/392213 | 15:29 |
openstackgerrit | Mario Villaplana proposed openstack/ironic-python-agent: WIP: IPA rescue extension https://review.openstack.org/423521 | 15:30 |
*** mtanino has joined #openstack-ironic | 15:30 | |
*** vinsh has joined #openstack-ironic | 15:31 | |
*** rpioso has joined #openstack-ironic | 15:31 | |
openstackgerrit | Merged openstack/ironic: iRMC power driver for soft reboot and soft power off https://review.openstack.org/216743 | 15:32 |
mjturek | mariojv: probably need tests for the checks you added to the rescue/unrescue CLI patch, would you want me to handle that? | 15:33 |
mariojv | mjturek: go ahead - thanks ! | 15:34 |
mjturek | np mariojv :) | 15:34 |
mariojv | we're starting to test rescue end-to-end and are finding a couple things that need fixing | 15:35 |
mjturek | yeah passing the value to set_provision_state is probably important! lol, good catch | 15:36 |
*** dyasny has quit IRC | 15:38 | |
rloo | if anyone feels like reviewing a client patch, this is fairly easy: https://review.openstack.org/#/c/373096/ | 15:38 |
patchbot | patch 373096 - python-ironicclient - Add --no-maintenance to OSC 'baremetal node list' | 15:38 |
*** dyasny has joined #openstack-ironic | 15:40 | |
*** jschwarz is now known as jschwarz|brb | 15:41 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Fix cleaning up nodes with NVRAM (UEFI) https://review.openstack.org/425242 | 15:41 |
*** sasha2 has quit IRC | 15:42 | |
*** sasha2 has joined #openstack-ironic | 15:44 | |
*** awiddersheim has quit IRC | 15:45 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API https://review.openstack.org/424720 | 15:46 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API https://review.openstack.org/417970 | 15:46 |
jroll | vdrok: dtantsur|brb ^ should be good now | 15:46 |
*** jschwarz|brb is now known as jschwarz | 15:48 | |
*** Sukhdev has joined #openstack-ironic | 15:51 | |
jlvillal | soliosg, I saw the test patches putting the ironic tempest tests into their own repo worked. Congrats :) | 15:51 |
*** awiddersheim has joined #openstack-ironic | 15:54 | |
*** dtantsur|brb is now known as dtantsur | 15:56 | |
*** milan has joined #openstack-ironic | 15:56 | |
vdrok | morning jlvillal | 15:56 |
jlvillal | Hi vdrok :) | 15:56 |
milan | morning Ironic! :) | 15:56 |
jlvillal | And hello everyone else :) | 15:56 |
vdrok | jroll: wdyt about vsaienk0's comment on detail parameter? | 15:56 |
milan | morning jlvillal, vdrok! :) | 15:56 |
dtantsur | morning jlvillal, milan | 15:56 |
vdrok | morning milan :) | 15:57 |
milan | morning dtantsur! :) | 15:57 |
openstackgerrit | Mario Villaplana proposed openstack/ironic-python-agent: WIP: IPA rescue extension https://review.openstack.org/423521 | 15:57 |
jroll | vdrok: that's a good catch | 15:58 |
vdrok | jroll: also, generic hw still has that fake thing :D | 15:58 |
jroll | wat | 15:59 |
jroll | oh I think I removed it in the wrong patch | 15:59 |
jroll | ugh | 15:59 |
jlvillal | soliosg, A git-filter-branch thing I wrote: http://paste.openstack.org/show/596475/ | 16:00 |
jlvillal | jroll, 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 |
aslezil | dtantsur: (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 |
patchbot | patch 404372 - ironic-inspector-specs - Collect NIC name given by BIOS | 16:01 |
jlvillal | jroll, Should we submit 36 patches to openstack/ironic_tempest_plugin/ to try to recreate the history? Or just one big patch? | 16:01 |
dtantsur | aslezil, will do after the current meeting | 16:01 |
* jlvillal is leaning towards the more work of 36 patches... | 16:01 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test to create a port with specific port group UUID https://review.openstack.org/411826 | 16:01 |
aslezil | thanks dtantsur: | 16:01 |
JayF | jlvillal: 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 |
jroll | jlvillal: I'm fine with the former, history would be great | 16:02 |
jroll | yeah or that | 16:02 |
jroll | that seems more painful though | 16:02 |
jlvillal | JayF, They didn't sound too excited about that option... | 16:02 |
*** dsneddon_afk is now known as dsneddon | 16:02 | |
jroll | there's no CI on that repo yet, right? | 16:02 |
jlvillal | jroll, A pep8 one. | 16:02 |
jroll | ok, yeah not a big deal | 16:03 |
jlvillal | I think. Not sure. | 16:03 |
jroll | maybe double check with infra because there's a chance it could overload zuul | 16:03 |
jlvillal | One job is what I remember. Looking... | 16:03 |
jroll | but I think it should be fine | 16:03 |
jlvillal | Okay. I will work with soliosg on it. Thanks. | 16:03 |
jlvillal | jroll, He did get to passing with the plugin last night :) | 16:04 |
jroll | jlvillal: awesome | 16:04 |
jlvillal | Yeah, pep8 job. Takes less than 2 minutes to run. | 16:04 |
jroll | jlvillal: soliosg: please be sure to document the steps here :) | 16:04 |
*** derekh has quit IRC | 16:05 | |
jlvillal | soliosg, 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 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for port groups in ironicclient https://review.openstack.org/297727 | 16:05 |
jlvillal | soliosg, Plus could give a pointer to the script I did to filter-branch just the tempest plugin directory. | 16:06 |
*** rbartal has quit IRC | 16:11 | |
jroll | jlvillal: I think matt had that already in that etherpad | 16:12 |
jlvillal | jroll, Ah good. | 16:12 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: ETAGs for subresource requests https://review.openstack.org/425260 | 16:13 |
soliosg | jlvillal: ok, will continue to refine the patch sets, document and update docs as needed | 16:13 |
soliosg | jlvillal: something important is to plan the transition from ironic to the new repo, since at some point there will be patches submitted to both repos | 16:14 |
soliosg | jlvillal: we will need some kind of 'code freeze' for tempest patches submitted to ironic | 16:15 |
*** mgould|afk is now known as mgould | 16:15 | |
*** rushil has joined #openstack-ironic | 16:16 | |
soliosg | jlvillal: *submitted to ironic that update ironic/ironic_tempest_plugin to be more specific | 16:17 |
jroll | suspect right after ocata release is a good time for that | 16:18 |
*** vsaienko has quit IRC | 16:18 | |
jroll | soliosg: want to add that to the next meeting agenda and we can confirm? it shouldn't take long to transition it | 16:18 |
soliosg | jroll, jlvillal: ok, it's good idea to bring it up, so everyone has it in mind | 16:21 |
jlvillal | soliosg, 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 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API https://review.openstack.org/424720 | 16:21 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API https://review.openstack.org/417970 | 16:21 |
jroll | dtantsur: vdrok: let's do this | 16:22 |
dtantsur | :) | 16:22 |
soliosg | jroll, 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/.*%22 | 16:22 |
rloo | jroll and gang: as far as client patches go, these are the ones I'd like to see get in: | 16:22 |
rloo | https://review.openstack.org/#/c/373096/, needs review | 16:22 |
patchbot | patch 373096 - python-ironicclient - Add --no-maintenance to OSC 'baremetal node list' | 16:22 |
vdrok | heh, will look in a minute | 16:22 |
jroll | jlvillal: yeah, but we'll have to sync it between landing stuff in the plugin repo and whenever we convert it | 16:23 |
jlvillal | soliosg, Cool gerrit search :) | 16:23 |
rloo | https://review.openstack.org/#/c/420659/ needs to be updated and reviewed so not sure it will make it | 16:23 |
patchbot | patch 420659 - python-ironicclient - Support --os-baremetal-api-version latest | 16:23 |
rloo | https://review.openstack.org/#/c/425199/ needs to be updated and reviewed so not sure it will make it | 16:23 |
patchbot | patch 425199 - python-ironicclient - Allow creating portgroups via create commands | 16:23 |
rloo | other than those, i may look at some other patches but there aren't any high bugs outstanding. | 16:24 |
*** sacharya has joined #openstack-ironic | 16:24 | |
*** sacharya has quit IRC | 16:24 | |
rloo | if anyone else has any client patches they want to land today, please mention! | 16:24 |
jroll | rloo: nice, thanks | 16:24 |
rloo | jroll: oh yeah, and your relnotes patch that doesn't exist yet :) | 16:24 |
jroll | rloo: indeed | 16:25 |
rloo | jroll: let me/us know when you want to pull the trigger | 16:25 |
*** amarao has quit IRC | 16:25 | |
*** rushil has quit IRC | 16:25 | |
*** rushil has joined #openstack-ironic | 16:26 | |
*** udesale has quit IRC | 16:26 | |
vdrok | rloo: do you have a preference in doing continue vs checking with if in the create patch? | 16:27 |
vdrok | there should not be a case when we have an error and portgroup_uuid at the same time | 16:27 |
jroll | rloo: I'm slowly convincing myself that cleaning up release notes isn't worth it | 16:29 |
jroll | rloo: so unless updates come in for patches #2 and #3 on your list, I'm ready when --no-maintenance lands | 16:30 |
dtantsur | jroll, please check a small patch https://review.openstack.org/#/c/421965/ to unbreak inspector newton CI, if you have a second | 16:30 |
patchbot | patch 421965 - ironic-inspector (stable/newton) - Cherry-pick fixes for Grenade | 16:30 |
jroll | sure | 16:31 |
*** mjura has quit IRC | 16:32 | |
rloo | vdrok: no preference | 16:33 |
rloo | jroll: ok | 16:33 |
jroll | dtantsur: +A | 16:33 |
dtantsur | thnx | 16:33 |
rloo | vdrok, galyna ^^, let me know when/if you update your client patches soon/today | 16:33 |
vdrok | jroll: one really tiny thing in the first patch | 16:34 |
vdrok | rloo: I will do now | 16:34 |
dtantsur | rloo, +A on --no-maintenance | 16:34 |
rloo | thx dtantsur! | 16:34 |
* jroll shakes fist at vdrok | 16:35 | |
vdrok | awww :'( | 16:35 |
vdrok | :D | 16:35 |
* rloo thanks vdrok! | 16:35 | |
*** AlexeyAbashkin has quit IRC | 16:35 | |
*** morgabra has quit IRC | 16:36 | |
openstackgerrit | Varun Gadiraju proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job https://review.openstack.org/403895 | 16:37 |
jroll | vdrok: "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 |
openstackgerrit | Varun Gadiraju proposed openstack/ironic: WIP/DNM: Testing grenade multi-tenant experimental job https://review.openstack.org/403895 | 16:38 |
vdrok | jroll: I mean, if we set detail=False, we'd ideally fail too. our boolean type also accepts None, so it will work | 16:38 |
vdrok | if we set detail=False on old api version | 16:39 |
jroll | vdrok: urgh, I see what you mean, sigh | 16:39 |
* jroll fixes | 16:41 | |
*** moshele has quit IRC | 16:41 | |
*** bcornec1 has quit IRC | 16:42 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API https://review.openstack.org/424720 | 16:42 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API https://review.openstack.org/417970 | 16:42 |
jroll | done vdrok | 16:42 |
vdrok | thanks :) | 16:43 |
jroll | np, thank you | 16:44 |
jroll | dtantsur: are you waiting on anything for inspector client release? | 16:44 |
mgould | how do I install optional driver dependencies on devstack? | 16:45 |
mgould | specifically, python-dracclient | 16:45 |
jroll | mgould: pip install? :) | 16:46 |
jroll | rather sudo pip install I believe | 16:47 |
jroll | and restart conductor | 16:47 |
mgould | yeah, I guess so | 16:48 |
mgould | I was wondering if there was a way to make it happen as part of stack.sh | 16:48 |
jroll | I feel like their might be a thing to add packages, not sure | 16:48 |
* jroll looks | 16:48 | |
dtantsur | jroll, lemme check | 16:49 |
mgould | jroll: thanks | 16:50 |
dtantsur | well, 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-ironic | 16:50 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Fix help message for the node-vif-attach command https://review.openstack.org/425281 | 16:51 |
JayF | mariojv just rescued an ironic node in devstack \o/ | 16:52 |
dtantsur | \o/ | 16:52 |
mariojv | woohoo \o/ | 16:52 |
*** rpioso has quit IRC | 16:52 | |
bfournie | dtantsur: 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 reviews | 16:53 |
jroll | mgould: 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 |
dtantsur | bfournie, thanks | 16:53 |
mgould | jroll: cool, thanks | 16:53 |
dtantsur | jroll, ironic-inspector-client seems ready for the release | 16:53 |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Allow creating portgroups via create commands https://review.openstack.org/425199 | 16:53 |
jroll | dtantsur: awesome, thanks! | 16:53 |
vdrok | rloo: all done, I hope ^^ | 16:54 |
jroll | I see one ironicclient patch is approved | 16:54 |
rloo | vdrok: thx. looking... | 16:54 |
rloo | jroll: yup. give us 4 more hours? :) | 16:54 |
jroll | galyna: can you update this patch in the next few hours? https://review.openstack.org/#/c/420659/ | 16:54 |
patchbot | patch 420659 - python-ironicclient - Support --os-baremetal-api-version latest | 16:54 |
jroll | rloo: yep, sounds good | 16:54 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Add tests+support for psutil 1-5 https://review.openstack.org/425284 | 16:55 |
JayF | rloo: ^ that should cover all the feedback you had on the gate-fixin' patch :) | 16:55 |
dtantsur | jroll, so, we're not including ironicclient --boot-interface etc change, right? | 16:55 |
rloo | JayF: thx. may look later :) | 16:55 |
NobodyCam | Good Morning Ironic'ers | 16:55 |
dtantsur | morning NobodyCam | 16:56 |
NobodyCam | :) hey hey dtantsur :) | 16:56 |
rloo | morning NobodyCam | 16:56 |
jroll | dtantsur: we could, if we're merging the API today | 16:56 |
NobodyCam | Good Morning role :) | 16:56 |
jroll | I would like to | 16:56 |
* jroll will review that | 16:56 | |
vdrok | morning NobodyCam | 16:57 |
jroll | jlvillal: so I can abandon my *-tempest-plugin cookiecutter things, yes? | 16:57 |
NobodyCam | gah.. Good Morning rloo... New computer have to train the autocrat all over again | 16:57 |
NobodyCam | Morning vdrok :) | 16:57 |
vdrok | another tiny client patch, might be worth too https://review.openstack.org/425281 | 16:57 |
patchbot | patch 425281 - python-ironicclient - Fix help message for the node-vif-attach command | 16:57 |
jlvillal | jroll, Uh. Not sure. | 16:57 |
jroll | jlvillal: ok, I'll look later | 16:57 |
jroll | NobodyCam: ++ for "autocrat", lol | 16:58 |
dtantsur | rloo or JayF or anyone, wanna check the driver comp API patches? vdrok and I have reviewed them several times already | 16:58 |
rloo | dtantsur: 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 |
NobodyCam | lol ++++ | 16:59 |
dtantsur | rloo, well, the client part is trivial | 16:59 |
rloo | dtantsur: if the driver comp stuff is done in ocata, it is simple to push out a new client right after that. | 16:59 |
JayF | dtantsur: 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 |
rloo | dtantsur: 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 IRC | 16:59 | |
rloo | dtantsur: but i'll try to take a look in an hour or so... | 16:59 |
dtantsur | rloo, we're severely limiting number of people trying the API by not providing a CLI for it, so dunno.. | 17:00 |
jroll | dtantsur: the client stuff is missing --detail, at least, so it isn't ready | 17:00 |
dtantsur | mm, what about --detail? | 17:01 |
jroll | GET /v1/drivers?detail=True <- to return all fields in driver-list | 17:01 |
*** retr0h has joined #openstack-ironic | 17:01 | |
dtantsur | right, I have to admin I've only reviewed the client patch with --XXX-interface, not one with driver changes | 17:01 |
dtantsur | I'm much more excited about the former, as it would enable people to play with dynamic drivers | 17:02 |
jroll | ah, true | 17:02 |
jroll | though the other helps, too | 17:02 |
dtantsur | I don't disagree with that, just trying to prioritize my reviewing efforts :) | 17:02 |
jroll | fair | 17:02 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Add tests+support for psutil 1-5 https://review.openstack.org/425284 | 17:03 |
jroll | dtantsur: I'm +2 on the --foo-interface patch | 17:05 |
*** Nisha_Agarwal has joined #openstack-ironic | 17:06 | |
dtantsur | cool | 17:06 |
openstackgerrit | Merged openstack/python-ironicclient: Add --no-maintenance to OSC 'baremetal node list' https://review.openstack.org/373096 | 17:07 |
dtantsur | rloo, do we also have --no-associated for OSC pending? | 17:07 |
rloo | dtantsur: unassociated. got merged i believe. let me see. | 17:07 |
JayF | if it didn't, I can take a look at it now | 17:08 |
openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 17:08 |
*** rama_y has joined #openstack-ironic | 17:10 | |
* dtantsur has to go afk for roughly 20 minutes | 17:10 | |
rloo | dtantsur, JayF: https://review.openstack.org/#/c/369279/ | 17:10 |
patchbot | patch 369279 - python-ironicclient - Extend OSC "node list" cmd to fetch nodes without ... (MERGED) | 17:10 |
dtantsur | cool! | 17:10 |
JayF | --awesome --not-bad | 17:10 |
JayF | lol | 17:10 |
rloo | not a great title but ... | 17:10 |
rloo | so we almost have feature parity with ironic cli & osc | 17:10 |
JayF | \o/ | 17:11 |
* dtantsur doesn't like --unassociated vs --no-maintenance too much, but I guess it's better English this way | 17:11 | |
dtantsur | rloo, JayF, have you seen that they've deprecated neutron CLI in favor of OSC? | 17:11 |
rloo | dtantsur: JayF and I discussed it. we prefer instance/no-instance, but decided it was too late for ocata to make the change. | 17:11 |
JayF | dtantsur: re: associated, I suggested if enough people dislike it, going to --instance --no-instance might be clearer than --associated --unassociated | 17:11 |
dtantsur | fine | 17:11 |
jroll | --unassociated and --no-maintenance are different, right | 17:11 |
rloo | dtantsur: i'd like to deprecate our ironic cli too | 17:11 |
dtantsur | rloo, goal for Pike? :) | 17:12 |
rloo | dtantsur: i would like. | 17:12 |
*** newmember has joined #openstack-ironic | 17:13 | |
rloo | dtantsur: deprecate. not sure how long we have to leave before we delete | 17:13 |
dtantsur | we don't have 'driver properties list', do we? | 17:13 |
rloo | dtantsur: nope. that's one of the ones we are missing. | 17:13 |
rloo | dtantsur: and raid i think. | 17:13 |
dtantsur | I see | 17:13 |
* dtantsur goes afk for real now, sorry | 17:13 | |
jroll | speaking 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 do | 17:14 |
rloo | dtantsur: https://bugs.launchpad.net/python-ironicclient/+bug/1619052 and https://bugs.launchpad.net/python-ironicclient/+bug/1619053 | 17:14 |
openstack | Launchpad 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 |
openstack | Launchpad bug 1619053 in python-ironicclient "[RFE] openstackclient cmd for 'ironic driver-properties'" [Wishlist,In progress] - Assigned to Galyna Zholtkevych (gzholtkevych) | 17:14 |
rloo | dtantsur: cuz it wasn't obvious what the osc commands should be | 17:14 |
*** dtantsur is now known as dtantsur|brb | 17:14 | |
rloo | bye dtantsur! | 17:14 |
dtantsur|brb | hey, I'm gonna be back soon :) | 17:14 |
rloo | jroll: i thought we already support py35 thx to jlvillal ? | 17:15 |
rloo | jroll: don't answer. we can discuss later :) | 17:15 |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Support --os-baremetal-api-version latest https://review.openstack.org/420659 | 17:15 |
vdrok | jroll: rloo updated^, galyna is out for today already | 17:16 |
rloo | vdrok: thx. will look after i finish with your patch (almost done) | 17:16 |
jroll | rloo: we don't have integration testing yet | 17:16 |
jroll | vdrok: thanks | 17:16 |
*** alexpilotti has quit IRC | 17:17 | |
JayF | jroll: I think vgadiraj and maybe joanna have both looked at the deploy-api-via-wsgi stuff, I don't think we're far | 17:17 |
rloo | jroll: oh. well, we need stuff to do for pike anyway. ha ha. | 17:17 |
jroll | JayF: awesome | 17:17 |
openstackgerrit | Merged openstack/python-ironicclient: Change os_tenant_name to os_project_name in tests configuration script https://review.openstack.org/408235 | 17:17 |
jroll | rloo: yeah, we totally don't have enough work :) | 17:17 |
rloo | jroll: 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 do | 17:18 |
*** yolanda has quit IRC | 17:18 | |
jroll | rloo: lol, I hope so | 17:18 |
*** alexpilotti has joined #openstack-ironic | 17:19 | |
vgadiraj | JayF, jroll: joanna mentioned recently that she was looking at that | 17:19 |
JayF | Jim already shared his secret plan to sit back, drink beer, and cackle at us once he's free | 17:19 |
JayF | :P | 17:19 |
*** ccamacho has quit IRC | 17:19 | |
* jroll no comment | 17:20 | |
TheJulia | Wait? He is already not sitting back with beer? | 17:21 |
NobodyCam | lol | 17:21 |
TheJulia | Clearly if he is not, this needs to be corrected! | 17:22 |
*** jaosorior has quit IRC | 17:22 | |
* TheJulia goes back to meetings | 17:22 | |
*** alexpilotti has quit IRC | 17:23 | |
joanna | JayF, jroll, vgadiraj: yes, I actually do it now | 17:24 |
joanna | the wsgi config for apache seems to be used only for hosting httpboot stuff | 17:24 |
*** jheroux has joined #openstack-ironic | 17:24 | |
JayF | joanna: it's been added as a openstack-wide priority to get api services running in apache in devstack (via wsgi) | 17:25 |
JayF | joanna: 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 there | 17:25 |
joanna | JayF: not promising anything, but I can definitely try to do that and maybe even succeed :) | 17:26 |
joanna | is there any bug reference for that? | 17:26 |
JayF | joanna: I suspect there is a bug for it, but I don't know the number right off | 17:26 |
joanna | ok, I'll look into it | 17:27 |
JayF | https://bugs.launchpad.net/ironic/+bug/1513005 | 17:27 |
openstack | Launchpad bug 1513005 in Ironic "Auto deploy devstack with ir-api under apache" [Wishlist,In progress] - Assigned to Anton Arefiev (aarefiev) | 17:27 |
joanna | awesme, ty! | 17:27 |
JayF | looks like aarefiev already has something in flight that more or less got abandoned | 17:27 |
JayF | might be a decent starting poitn | 17:27 |
joanna | I'll get done the httpboot hosting as wsgi, and then move to this one. | 17:28 |
*** alexpilotti has joined #openstack-ironic | 17:28 | |
aarefiev | joanna: yeah, there is patch on review which need to be updated, feel free to pick up it | 17:30 |
jlvillal | rloo, 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 |
vdrok | time to leave for me, seems I won't be able to look at the client side of driver comp stuff today :( | 17:31 |
joanna | aarefiev: thank you! I will :) | 17:31 |
vdrok | good night everyone! | 17:31 |
rloo | jlvillal: no worries, there's always Pike! | 17:31 |
rloo | vdrok: night and thx for the client patches! | 17:31 |
jlvillal | Bye vdrok | 17:31 |
joanna | night, vdrok :) | 17:31 |
*** sasha2 has quit IRC | 17:32 | |
NobodyCam | night vdork | 17:32 |
*** jaosorior has joined #openstack-ironic | 17:32 | |
*** alexpilotti has quit IRC | 17:32 | |
*** dtantsur|brb is now known as dtantsur | 17:33 | |
*** sdake_ has joined #openstack-ironic | 17:33 | |
*** tesseract has quit IRC | 17:33 | |
*** karts has quit IRC | 17:34 | |
*** sasha2 has joined #openstack-ironic | 17:35 | |
*** baoli has joined #openstack-ironic | 17:35 | |
* jroll eats lunch | 17:36 | |
*** sasha2 has quit IRC | 17:37 | |
*** aarefiev is now known as aarefiev_afk | 17:37 | |
*** yolanda has joined #openstack-ironic | 17:41 | |
openstackgerrit | Andrea Frittoli proposed openstack/ironic: Fix broken link in the iLO driver docs https://review.openstack.org/425316 | 17:43 |
*** alexpilotti has joined #openstack-ironic | 17:44 | |
*** [1]rpioso has quit IRC | 17:45 | |
*** rushil has quit IRC | 17:45 | |
*** alexpilotti has quit IRC | 17:46 | |
*** alexpilotti has joined #openstack-ironic | 17:47 | |
*** alineb_ has quit IRC | 17:48 | |
rloo | Thanks for the +As TheJulia! | 17:50 |
*** aNuposic has joined #openstack-ironic | 17:51 | |
*** dprince has quit IRC | 17:53 | |
TheJulia | Just trying to review some stuff while I have some time between meeting subjects :) | 17:53 |
*** Nisha_Agarwal has quit IRC | 17:53 | |
*** lucasagomes is now known as lucas-afk | 17:54 | |
*** ralonsoh has quit IRC | 17:58 | |
dtantsur | meh, zuul is slowly overflowing | 18:04 |
rloo | dtantsur: all those client patches waiting to get merged :) | 18:06 |
*** yolanda has quit IRC | 18:06 | |
dtantsur | yeah :) | 18:07 |
rloo | dtantsur: inspector-client is in good shape? | 18:07 |
dtantsur | rloo, yep, we're ready (missed one feature, unfortunately) | 18:08 |
rloo | dtantsur: good (and bad)! am going to get some lunch, then look at driver comp... | 18:08 |
dtantsur | jroll, 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 |
dtantsur | yep, this :) | 18:08 |
jlvillal | I wonder if there is a way to turn our ironic-bm-logs into an animated gif... | 18:08 |
dtantsur | jlvillal+++ | 18:09 |
jlvillal | Instead 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.gz | 18:09 |
dtantsur | see you all :) | 18:09 |
jlvillal | dtantsur, :) | 18:09 |
jlvillal | dtantsur, Ciao | 18:09 |
*** dtantsur is now known as dtantsur|afk | 18:09 | |
rloo | good night dtantsur! | 18:09 |
JayF | https://review.openstack.org/#/c/425316/ super duper simple doc fix if someone wants to land it | 18:09 |
patchbot | patch 425316 - ironic - Fix broken link in the iLO driver docs | 18:09 |
JayF | nn dtantsur|afk o/ | 18:09 |
TheJulia | jlvillal: server config files :) | 18:13 |
jlvillal | TheJulia, The "serial console" output that we get from the gate jobs. Very difficult to read for failures. | 18:14 |
jlvillal | TheJulia, I'm not sure what "server config files" means. | 18:14 |
*** e0ne has quit IRC | 18:18 | |
openstackgerrit | Merged openstack/ironic: Update proliantutils version required for Ocata release https://review.openstack.org/424593 | 18:20 |
openstackgerrit | Miles Gould proposed openstack/ironic: [WIP] install python-dracclient if drac enabled https://review.openstack.org/425326 | 18:22 |
TheJulia | jlvillal: 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 reply | 18:24 |
jlvillal | TheJulia, 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 |
jlvillal | But 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 |
TheJulia | If we could grab a screenshot via vnc.... | 18:25 |
*** rushil has joined #openstack-ironic | 18:25 | |
*** vsaienko has joined #openstack-ironic | 18:27 | |
TheJulia | jlvillal: 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-ironic | 18:28 | |
jlvillal | TheJulia, Yeah. It would be nice to have the result :) | 18:28 |
*** [1]rpioso has joined #openstack-ironic | 18:28 | |
jlvillal | At the moment I use this. | 18:28 |
TheJulia | screenshots would be better, maybe virtual bmc could snap one before power-off ? | 18:28 |
jlvillal | $ cat LOGFILE | pv -q -L 2400 | 18:28 |
jlvillal | TheJulia, Actually the a final screen shot would be nice :) | 18:29 |
jlvillal | s/the a/a/ | 18:29 |
TheJulia | well, prior to any power-off would be key. Retries would make other things disappear. | 18:29 |
*** sayantani01 has quit IRC | 18:30 | |
*** sambetts is now known as sambetts|afk | 18:34 | |
sambetts|afk | night all | 18:34 |
JayF | I just ignore the control characters in the node consoles, honestly | 18:35 |
JayF | if you read bottom-up like most of those devstack files, you don't see most of them | 18:35 |
* JayF wonders if perhaps it's worse on different os/browser combos | 18:35 | |
*** Sukhdev has quit IRC | 18:35 | |
*** Sukhdev has joined #openstack-ironic | 18:36 | |
jlvillal | JayF, 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.gz | 18:36 |
jlvillal | s/isn't that easy/is very difficult/ | 18:37 |
*** trown is now known as trown|lunch | 18:37 | |
JayF | ah, honestly I haven't had to trobuleshoot no-pxe that much with the console | 18:37 |
JayF | and TIL the output I've been mostly-ignoring in those are the ipxe handshake | 18:37 |
JayF | lol | 18:37 |
jlvillal | :) | 18:38 |
*** sdake_ has quit IRC | 18:39 | |
*** sdake has joined #openstack-ironic | 18:40 | |
*** yolanda has joined #openstack-ironic | 18:40 | |
*** krtaylor has quit IRC | 18:40 | |
rloo | jlvillal: 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 stuff | 18:41 |
*** sasha2 has joined #openstack-ironic | 18:41 | |
rloo | jlvillal: is that what you want? (I skimmed the above) | 18:41 |
jlvillal | rloo, Yeah. Similar thing. Need to get it into a terminal to parse the escape codes | 18:41 |
*** dprince has joined #openstack-ironic | 18:41 | |
TheJulia | so 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 problem | 18:45 | |
JayF | I'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 support | 18:46 |
TheJulia | I typically just use curl or cat the content in a terminal. The annoying thing is when some of the data is not translated at all | 18:46 |
*** vsaienko has quit IRC | 18:46 | |
JayF | TheJulia: do you have an example of a log where that happens? | 18:47 |
JayF | TheJulia: I'm really curious how much distro/shell/etc matters on stuff like that | 18:47 |
* JayF has been down this road, in a more painful way, with actual Serial-over-lan consoles that were awful :( | 18:48 | |
JayF | for 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 would | 18:49 |
*** rushil has quit IRC | 18:51 | |
mariojv | TheJulia: 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.py | 18:52 |
patchbot | patch 407639 - ironic - Config drive support for Ceph | 18:52 |
mariojv | it's really entirely about the auth mechanism, since both ceph support and swift support for configdrives are using the swift client | 18:52 |
*** pcaruana has quit IRC | 18:53 | |
mariojv | i think changing those config option names will make it a bit clearer to operators, but curious about what you think too | 18:53 |
TheJulia | Sorry, had to step away | 18:53 |
TheJulia | JayF: 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's | 18:56 |
TheJulia | thing. | 18:56 |
*** vsaienko has joined #openstack-ironic | 18:57 | |
JayF | TheJulia: interesting. Yeah, ipxe serial was an issue on our physical boxes too | 18:57 |
JayF | TheJulia: and you have to recompile ipxe to change any of the settings w/r/t serial console | 18:57 |
openstackgerrit | Andrey Shestakov proposed openstack/bifrost: Allow deploy-dynamic on remote host https://review.openstack.org/411304 | 18:59 |
TheJulia | JayF: Typically I've seen that on ubuntu, I wonder if others see it less with different distributions | 18:59 |
aNuposic | TheJulia: 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 mechanism | 19:00 |
openstack | Launchpad 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 |
patchbot | patch 407639 - ironic - Config drive support for Ceph | 19:00 |
JayF | aNuposic: mariojv: fwiw I'm in agreement with mariojv's suggestion about config names in your patch | 19:00 |
*** alexpilotti has quit IRC | 19:00 | |
aNuposic | JayF: Yup, will update those to make it more clear | 19:01 |
TheJulia | aNuposic: I'll take a look either later today or early tomorrow. I'm in meetings this week | 19:01 |
aNuposic | JayF: seems those are making a bit confusing to understand | 19:01 |
aNuposic | TheJulia: Sure | 19:01 |
aNuposic | TheJulia: thx :) | 19:02 |
*** rushil has joined #openstack-ironic | 19:04 | |
*** hecliunyx has quit IRC | 19:05 | |
*** ijw has joined #openstack-ironic | 19:08 | |
openstackgerrit | Merged openstack/ironic: Onetime boot when set_boot_device isn't persistent https://review.openstack.org/340596 | 19:08 |
*** ijw has quit IRC | 19:09 | |
*** ijw has joined #openstack-ironic | 19:09 | |
openstackgerrit | Merged openstack/ironic: Fix broken link in the iLO driver docs https://review.openstack.org/425316 | 19:10 |
*** moshele has joined #openstack-ironic | 19:11 | |
*** hecliunyx has joined #openstack-ironic | 19:13 | |
*** Sukhdev has quit IRC | 19:13 | |
*** alexpilotti has joined #openstack-ironic | 19:15 | |
rloo | jroll: 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 |
jroll | rloo: we do, so that we can merge the client stuff | 19:17 |
rloo | jroll: where are the client patches? | 19:17 |
jroll | rloo: https://review.openstack.org/#/q/status:open++branch:master+topic:bug/1524745 | 19:17 |
*** moshele has quit IRC | 19:18 | |
jroll | I'd like to at least get that first one in | 19:18 |
jroll | if we don't, it's fine | 19:18 |
jroll | but will mean we wait a few more weeks before folks can kick the tires on it | 19:18 |
*** krtaylor has joined #openstack-ironic | 19:18 | |
jroll | (including things like tripleo) | 19:18 |
rloo | jroll: oh. (i was using the driver comp link in etherpad/priorities) | 19:19 |
jroll | ah, sorry | 19:19 |
*** alexpilotti has quit IRC | 19:19 | |
*** jaosorior has quit IRC | 19:21 | |
rloo | jroll: ok, will see what i can do. that's 3 patches that need to be reviewed/merged today... | 19:22 |
openstackgerrit | Merged openstack/python-ironicclient: Fix help message for the node-vif-attach command https://review.openstack.org/425281 | 19:22 |
jroll | rloo: yeah, if there's more important things, do those | 19:23 |
* jroll wants to just +A his API patches, they have 2+2 | 19:23 | |
rloo | jroll: all the client patches i wanted to get in have been +A'd. so I'll look at these. | 19:24 |
jroll | rloo: ok, thank you | 19:25 |
openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 19:26 |
*** mgoddard has joined #openstack-ironic | 19:29 | |
*** alexpilotti has joined #openstack-ironic | 19:30 | |
*** trown|lunch is now known as trown | 19:33 | |
*** alexpilotti has quit IRC | 19:35 | |
*** e0ne has joined #openstack-ironic | 19:39 | |
*** [1]rpioso has quit IRC | 19:40 | |
openstackgerrit | Andrey Shestakov proposed openstack/bifrost: Allow deploy-dynamic on remote host https://review.openstack.org/411304 | 19:42 |
*** mitchjameson has joined #openstack-ironic | 19:42 | |
openstackgerrit | Merged openstack/python-ironicclient: Support --os-baremetal-api-version latest https://review.openstack.org/420659 | 19:43 |
*** sayantani01 has joined #openstack-ironic | 19:43 | |
*** alexpilotti has joined #openstack-ironic | 19:47 | |
*** rushil has quit IRC | 19:48 | |
*** dprince has quit IRC | 19:48 | |
*** rushil has joined #openstack-ironic | 19:48 | |
aNuposic | mariojv: 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_* drivers | 19:49 |
patchbot | patch 407639 - ironic - Config drive support for Ceph | 19:49 |
*** dprince has joined #openstack-ironic | 19:49 | |
openstackgerrit | Merged openstack/bifrost: Allow enroll node with empty properties. https://review.openstack.org/424249 | 19:51 |
*** alexpilotti has quit IRC | 19:52 | |
*** Sukhdev has joined #openstack-ironic | 19:52 | |
*** e0ne has quit IRC | 19:55 | |
*** hecliunyx has quit IRC | 20:01 | |
*** Sukhdev has quit IRC | 20:05 | |
*** alexpilotti has joined #openstack-ironic | 20:07 | |
*** morgabra has joined #openstack-ironic | 20:08 | |
*** hecliunyx has joined #openstack-ironic | 20:09 | |
*** jaosorior has joined #openstack-ironic | 20:12 | |
*** jaosorior has quit IRC | 20:17 | |
*** e0ne has joined #openstack-ironic | 20:17 | |
*** moshele has joined #openstack-ironic | 20:23 | |
*** jaosorior has joined #openstack-ironic | 20:24 | |
*** e0ne has quit IRC | 20:28 | |
*** Goneri has quit IRC | 20:29 | |
*** e0ne has joined #openstack-ironic | 20:32 | |
openstackgerrit | Merged openstack/python-ironicclient: Allow creating portgroups via create commands https://review.openstack.org/425199 | 20:38 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: [Devstack] Allow add several ports to node https://review.openstack.org/425112 | 20:40 |
*** e0ne has quit IRC | 20:40 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 20:40 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 20:41 |
openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Add support for RESCUE and UNRESCUE provision states https://review.openstack.org/408341 | 20:42 |
*** dprince has quit IRC | 20:48 | |
*** dprince has joined #openstack-ironic | 20:48 | |
*** alexpilotti has quit IRC | 20:58 | |
*** aNuposic has quit IRC | 21:00 | |
rloo | jroll: 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 |
patchbot | patch 417970 - ironic - Add dynamic driver functionality to REST API | 21:00 |
jroll | rloo: O_O | 21:01 |
*** alexpilotti has joined #openstack-ironic | 21:02 | |
jroll | rloo: some good catches in there, though, will need to re-spin | 21:02 |
rloo | jroll: ok, i'll keep an eye out for the spinned | 21:03 |
jroll | there'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 do | 21:03 |
jroll | rloo: I'm gonna just ship the clients then, if there's nothing else to land | 21:03 |
rloo | jroll: yup. i feel like we should have some rule, no more than 10 patch sets or something... | 21:03 |
jroll | and we can ship this first thing in pike | 21:03 |
rloo | jroll: I'm good with that. | 21:03 |
jroll | rloo: I mean as an author, but yeah :) | 21:03 |
*** [1]rpioso has joined #openstack-ironic | 21:04 | |
rloo | jroll: 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 person | 21:05 | |
jroll | rloo: well, this one I couldn't keep in my head because it's so much in one patch | 21:05 |
jroll | probably should have just eaten a few API version bumps | 21:05 |
jroll | rather than having it all in one | 21:05 |
jroll | looks like there's nothing else ready for ironicclient, so I'm releasing those now, yay | 21:06 |
*** alexpilotti has quit IRC | 21:07 | |
rloo | jroll: maybe split the vendor out but it was fine. | 21:07 |
jroll | rloo: just hard to tell if the patch does everything it should, when it's that big (for me) | 21:07 |
rloo | jroll: in that case, then better to split up :) | 21:08 |
jroll | indeed. too late now, will be incredibly hard to untangle | 21:08 |
rloo | jroll: oh yeah, do NOT separate it now. | 21:09 |
*** aNuposic has joined #openstack-ironic | 21:16 | |
*** Goneri has joined #openstack-ironic | 21:17 | |
*** rushil has quit IRC | 21:19 | |
openstackgerrit | Merged openstack/ironic-inspector: Cherry-pick fixes for Grenade https://review.openstack.org/421965 | 21:20 |
*** dsneddon is now known as dsneddon_afk | 21:21 | |
*** hamzy has quit IRC | 21:22 | |
*** dprince has quit IRC | 21:23 | |
*** dsneddon has joined #openstack-ironic | 21:23 | |
*** sayantani01 has quit IRC | 21:23 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests https://review.openstack.org/423556 | 21:27 |
*** jaosorior has quit IRC | 21:31 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Agent driver rescue implementation https://review.openstack.org/400437 | 21:32 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 21:32 |
*** sayantani01 has joined #openstack-ironic | 21:36 | |
*** aNuposic has quit IRC | 21:36 | |
*** aNuposic has joined #openstack-ironic | 21:37 | |
*** rushil has joined #openstack-ironic | 21:41 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs https://review.openstack.org/425426 | 21:44 |
jlvillal | TheJulia, JayF ^^^ Sort of a hacky way. Just strip out the ansi escape sequences. The file becomes readable at least. | 21:46 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Testing multinode job https://review.openstack.org/422149 | 21:46 |
TheJulia | jlvillal: ohhh, I've added it to my queue so I can take a look once the jobs are done | 21:47 |
jlvillal | TheJulia, Thanks :) | 21:47 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs https://review.openstack.org/425426 | 21:51 |
*** milan has quit IRC | 21:52 | |
*** alexpilotti has joined #openstack-ironic | 21:59 | |
*** alexpilotti has quit IRC | 22:03 | |
*** moshele has quit IRC | 22:16 | |
*** bfournie has left #openstack-ironic | 22:26 | |
*** srobert_ has joined #openstack-ironic | 22:28 | |
jlvillal | It'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.gz | 22:29 |
jlvillal | TheJulia, JayF jroll rloo: ^^^ | 22:29 |
jlvillal | And 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 finish | 22:30 | |
*** bfournie has joined #openstack-ironic | 22:30 | |
*** srobert has quit IRC | 22:31 | |
*** baoli has quit IRC | 22:31 | |
*** srobert_ has quit IRC | 22:32 | |
rloo | jlvillal: a work of art! | 22:32 |
jlvillal | rloo, heh :) | 22:33 |
jlvillal | rloo, It isn't perfect but it does make the log readable. Especially for the failure cases. And I think that is good enough | 22:33 |
*** jheroux has quit IRC | 22:33 | |
rloo | jlvillal: ++ | 22:33 |
*** sayantani01 has quit IRC | 22:40 | |
*** sayantani01 has joined #openstack-ironic | 22:41 | |
*** Sukhdev has joined #openstack-ironic | 22:41 | |
*** cdearborn has quit IRC | 22:45 | |
*** mgoddard has quit IRC | 22:51 | |
*** bfournie has quit IRC | 22:51 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic-python-agent: Rescue extension for CoreOS with DHCP tenant networks https://review.openstack.org/423521 | 22:53 |
*** jkilpatr has quit IRC | 23:00 | |
*** Goneri has quit IRC | 23:07 | |
*** rushil has quit IRC | 23:07 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs https://review.openstack.org/425426 | 23:09 |
*** hecliunyx has quit IRC | 23:10 | |
*** jcoufal has quit IRC | 23:15 | |
*** krtaylor has quit IRC | 23:18 | |
*** hamzy has joined #openstack-ironic | 23:18 | |
*** hecliunyx has joined #openstack-ironic | 23:19 | |
*** rushil has joined #openstack-ironic | 23:19 | |
*** vsaienko has quit IRC | 23:20 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs https://review.openstack.org/425457 | 23:21 |
*** alexpilotti has joined #openstack-ironic | 23:23 | |
*** alexpilotti has quit IRC | 23:23 | |
*** newmember has quit IRC | 23:25 | |
*** newmember has joined #openstack-ironic | 23:25 | |
*** vsaienko has joined #openstack-ironic | 23:30 | |
openstackgerrit | Anup Navare proposed openstack/ironic: Config drive support for Ceph https://review.openstack.org/407639 | 23:33 |
*** rushil has quit IRC | 23:33 | |
soliosg | mariojv: regarding the last comment in https://review.openstack.org/#/c/400437/16 | 23:39 |
patchbot | patch 400437 - ironic - Agent driver rescue implementation | 23:39 |
soliosg | mariojv: looks like the node is not transitioning to one of the states in _LOOKUP_ALLOWED_STATES | 23:40 |
*** jkilpatr has joined #openstack-ironic | 23:41 | |
soliosg | mariojv: https://review.openstack.org/#/c/350831/26/ironic/api/controllers/v1/ramdisk.py@42 | 23:41 |
patchbot | patch 350831 - ironic - Add API and conductor methods for [un]rescue | 23:41 |
openstackgerrit | Anup Navare proposed openstack/ironic: Config drive support for Ceph https://review.openstack.org/407639 | 23:42 |
*** sayantani01 has quit IRC | 23:45 | |
*** sayantani01 has joined #openstack-ironic | 23:45 | |
*** baoli has joined #openstack-ironic | 23:47 | |
*** rushil has joined #openstack-ironic | 23:53 | |
openstackgerrit | Joanna Taryma proposed openstack/ironic: WIP host ipxe static assets in devstack on :80 https://review.openstack.org/424374 | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!