*** davidlenwell has quit IRC | 00:11 | |
*** ijw has quit IRC | 00:14 | |
*** ijw has joined #openstack-ironic | 00:19 | |
*** davidlenwell has joined #openstack-ironic | 00:25 | |
openstackgerrit | Yibo Cai proposed openstack/ironic: Support multi arch deployment https://review.openstack.org/322632 | 00:25 |
---|---|---|
*** ijw has quit IRC | 00:45 | |
*** hoangcx has joined #openstack-ironic | 00:50 | |
*** Goneri has quit IRC | 01:04 | |
*** ijw has joined #openstack-ironic | 01:05 | |
*** ijw_ has joined #openstack-ironic | 01:10 | |
*** ijw has quit IRC | 01:11 | |
*** ijw_ has quit IRC | 01:15 | |
*** jvgrant has quit IRC | 01:22 | |
*** jvgrant has joined #openstack-ironic | 01:22 | |
openstackgerrit | Anh Tran proposed openstack/ironic-python-agent: Fixed typo in doc file https://review.openstack.org/378137 | 01:45 |
openstackgerrit | Ruby Loo proposed openstack/ironic: Update multitenancy documentation https://review.openstack.org/378138 | 01:46 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 01:47 |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Add missing unit tests for provision state commands https://review.openstack.org/377607 | 01:49 |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Trivial: Rename and reorder deploy baremetal test https://review.openstack.org/378140 | 01:49 |
*** thrash is now known as thrash|g0ne | 01:54 | |
openstackgerrit | Xu Ao proposed openstack/ironic: Fix the mistakes in Installation Guide https://review.openstack.org/353224 | 01:58 |
openstackgerrit | Xu Ao proposed openstack/ironic: Fix the mistakes in Installation Guide https://review.openstack.org/353224 | 02:00 |
*** phuongnh has joined #openstack-ironic | 02:00 | |
*** baoli has quit IRC | 02:05 | |
*** baoli has joined #openstack-ironic | 02:11 | |
*** ppiela has joined #openstack-ironic | 02:17 | |
*** ijw has joined #openstack-ironic | 02:19 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off https://review.openstack.org/216738 | 02:21 |
*** ppiela has quit IRC | 02:22 | |
*** ijw has quit IRC | 02:23 | |
*** awiddersheim has quit IRC | 02:23 | |
*** ijw has joined #openstack-ironic | 02:24 | |
*** awiddersheim has joined #openstack-ironic | 02:28 | |
*** ijw has quit IRC | 02:29 | |
*** ijw has joined #openstack-ironic | 02:31 | |
*** ijw has quit IRC | 02:44 | |
*** baoli has quit IRC | 02:52 | |
*** xuao has joined #openstack-ironic | 02:52 | |
*** Sukhdev has joined #openstack-ironic | 02:59 | |
*** tuanla has joined #openstack-ironic | 03:00 | |
*** Sukhdev has quit IRC | 03:01 | |
openstackgerrit | Ha Van Tu proposed openstack/ironic: Fix typos in docstring/comments https://review.openstack.org/377261 | 03:01 |
*** david-lyle has quit IRC | 03:04 | |
*** sdake_ has quit IRC | 03:17 | |
*** ppiela has joined #openstack-ironic | 03:18 | |
*** aswadr_ has joined #openstack-ironic | 03:21 | |
*** ppiela has quit IRC | 03:22 | |
*** vikrant has joined #openstack-ironic | 03:26 | |
*** rajinir has quit IRC | 03:35 | |
*** sdake has joined #openstack-ironic | 03:39 | |
*** ppiela has joined #openstack-ironic | 03:43 | |
*** sdake_ has joined #openstack-ironic | 03:50 | |
*** Sukhdev has joined #openstack-ironic | 03:50 | |
*** sdake has quit IRC | 03:51 | |
*** sai has quit IRC | 04:22 | |
*** sai has joined #openstack-ironic | 04:23 | |
*** amotoki has joined #openstack-ironic | 04:28 | |
*** tangchen_ has quit IRC | 04:29 | |
*** tangchen_ has joined #openstack-ironic | 04:29 | |
*** ppiela has quit IRC | 04:36 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 04:51 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off https://review.openstack.org/216743 | 05:04 |
*** syed_ has joined #openstack-ironic | 05:05 | |
*** bks has joined #openstack-ironic | 05:07 | |
*** Sukhdev has quit IRC | 05:08 | |
*** links has joined #openstack-ironic | 05:09 | |
*** links has quit IRC | 05:15 | |
*** links has joined #openstack-ironic | 05:17 | |
*** ppiela has joined #openstack-ironic | 05:19 | |
*** ppiela has quit IRC | 05:24 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off https://review.openstack.org/216738 | 05:27 |
*** sdake_ has quit IRC | 05:30 | |
*** jaosorior has joined #openstack-ironic | 05:46 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work https://review.openstack.org/376322 | 05:54 |
*** vmud213 has joined #openstack-ironic | 05:55 | |
openstackgerrit | gengchc2 proposed openstack/bifrost: Update flake8 ignore list https://review.openstack.org/378257 | 05:57 |
mikal | Anyone home? | 06:08 |
*** mjura has joined #openstack-ironic | 06:09 | |
*** rcernin has joined #openstack-ironic | 06:14 | |
*** mjura has quit IRC | 06:16 | |
*** mjura has joined #openstack-ironic | 06:16 | |
*** rbartal has joined #openstack-ironic | 06:23 | |
*** ppiela has joined #openstack-ironic | 06:25 | |
*** ppiela has quit IRC | 06:30 | |
*** crinkle_ is now known as crinkle | 06:33 | |
*** pcaruana has joined #openstack-ironic | 06:37 | |
*** jrist has joined #openstack-ironic | 06:54 | |
*** marlinc has joined #openstack-ironic | 06:59 | |
*** tuanla has quit IRC | 07:01 | |
openstackgerrit | Dao Cong Tien proposed openstack/ironic: Fix inexact config option name in multitenancy.rst https://review.openstack.org/378291 | 07:01 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 07:02 |
*** jrist has quit IRC | 07:04 | |
*** ralonsoh has joined #openstack-ironic | 07:04 | |
*** ralonsoh_ has joined #openstack-ironic | 07:04 | |
*** ralonsoh has quit IRC | 07:04 | |
*** ralonsoh_ has quit IRC | 07:04 | |
*** vmud213 has quit IRC | 07:04 | |
*** ralonsoh has joined #openstack-ironic | 07:04 | |
*** ralonsoh_ has joined #openstack-ironic | 07:04 | |
*** ralonsoh has quit IRC | 07:04 | |
*** ralonsoh_ has quit IRC | 07:04 | |
*** ralonsoh has joined #openstack-ironic | 07:05 | |
*** vmud213 has joined #openstack-ironic | 07:07 | |
*** syed_ has quit IRC | 07:14 | |
*** links has quit IRC | 07:16 | |
openstackgerrit | zhangyanxian proposed openstack/bifrost: Fix typo in bifrost-role-auth-support-ea6b5571cd339aa2.yaml https://review.openstack.org/378296 | 07:20 |
openstackgerrit | zhangyanxian proposed openstack/bifrost: Fix typo in bifrost-role-auth-support-ea6b5571cd339aa2.yaml https://review.openstack.org/378296 | 07:21 |
*** marlinc has quit IRC | 07:21 | |
*** marlinc has joined #openstack-ironic | 07:27 | |
*** jpich has joined #openstack-ironic | 07:35 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-python-agent: Parallel erase disk devices https://review.openstack.org/346723 | 07:42 |
*** links has joined #openstack-ironic | 07:45 | |
pester | hi all I`m trying to boot node with fake_pxe driver, but nova-sheduler fails and in conductor logs I can see - "Deploy boot mode is None for" | 07:45 |
*** strigazi_AFK is now known as strigazi | 07:49 | |
pester | oh "ERROR nova.compute.manager PortBindingFailed: Binding failed for port" why nova compute tries to bind port ?... :( | 07:56 |
*** marlinc has quit IRC | 07:58 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:01 | |
*** ijw has joined #openstack-ironic | 08:03 | |
pas-ha | morning Ironic | 08:07 |
pas-ha | pester: that's why https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L1221-L1227 | 08:07 |
pas-ha | it seems currently for that to work in flat network case, you have to run neutron-agent on the node where ironic-conductor is | 08:09 |
*** ijw has quit IRC | 08:11 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-specs: Check duplicate nodes enrollment/update https://review.openstack.org/296357 | 08:11 |
*** links has quit IRC | 08:21 | |
*** Romanenko_K has joined #openstack-ironic | 08:24 | |
*** marlinc has joined #openstack-ironic | 08:24 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 08:26 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-specs: Dynamically allocate console ports https://review.openstack.org/249876 | 08:36 |
*** links has joined #openstack-ironic | 08:37 | |
*** sdake has joined #openstack-ironic | 08:41 | |
*** derekh has joined #openstack-ironic | 08:41 | |
*** sdake has quit IRC | 08:42 | |
*** priteau has joined #openstack-ironic | 08:48 | |
*** openstackgerrit has quit IRC | 08:48 | |
*** openstackgerrit has joined #openstack-ironic | 08:49 | |
*** moshele has joined #openstack-ironic | 08:50 | |
*** mgould|afk is now known as mgould | 08:57 | |
mgould | morning Ironic | 08:57 |
tangchen_ | morning mgould | 09:03 |
tangchen_ | mgould: I have updated this https://review.openstack.org/#/c/377607/, and added a followup patch. Please help to check when you are free. :) | 09:04 |
patchbot | patch 377607 - python-ironicclient - Add missing unit tests for provision state commands | 09:04 |
*** sambetts_ is now known as sambetts | 09:12 | |
sambetts | Morning all | 09:12 |
*** e0ne has joined #openstack-ironic | 09:14 | |
*** priteau has quit IRC | 09:15 | |
*** priteau has joined #openstack-ironic | 09:17 | |
openstackgerrit | Nam Nguyen Hoai proposed openstack/ironic-lib: Fix typo in docstring https://review.openstack.org/378470 | 09:35 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Update Ansible version to stable-2.1 https://review.openstack.org/378041 | 09:36 |
TheJulia | Good morning | 09:38 |
*** electrofelix has joined #openstack-ironic | 09:39 | |
*** tuanla has joined #openstack-ironic | 09:40 | |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Make console commands support multiple node operation https://review.openstack.org/378477 | 09:41 |
*** aswadr_ has quit IRC | 09:42 | |
*** electrofelix has quit IRC | 09:45 | |
*** electrofelix has joined #openstack-ironic | 09:46 | |
sambetts | Morning TheJulia | 09:46 |
openstackgerrit | gengchc2 proposed openstack/ironic: Fix a typo in policy.py,etc https://review.openstack.org/378487 | 09:53 |
*** ralonsoh_ has joined #openstack-ironic | 09:58 | |
*** ralonsoh_ has quit IRC | 09:58 | |
*** ralonsoh has quit IRC | 09:59 | |
*** ralonsoh has joined #openstack-ironic | 09:59 | |
*** xuao has quit IRC | 10:12 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add read and execute perms to httpboot and tftpboot folder https://review.openstack.org/374015 | 10:14 |
*** wajdi has quit IRC | 10:19 | |
mgould | morning TheJulia | 10:22 |
mgould | tangchen_: thanks! | 10:22 |
mgould | tangchen_: both look good to me, +1x2 | 10:28 |
*** hoangcx has quit IRC | 10:29 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Update Ansible version to stable-2.1 https://review.openstack.org/378041 | 10:32 |
openstackgerrit | Bharath kumar proposed openstack/ironic: Adding a ipxe_boot_sleep parameter https://review.openstack.org/373194 | 10:34 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update configure_tenant_networks https://review.openstack.org/356460 | 10:35 |
*** tuanla has quit IRC | 10:39 | |
*** awiddersheim has quit IRC | 10:39 | |
*** awiddersheim has joined #openstack-ironic | 10:44 | |
*** links has quit IRC | 10:53 | |
*** priteau has quit IRC | 10:53 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Reformat docstring and remove unused argument https://review.openstack.org/378526 | 10:54 |
*** phuongnh has quit IRC | 10:58 | |
*** dprince has joined #openstack-ironic | 11:02 | |
*** links has joined #openstack-ironic | 11:02 | |
*** moshele has quit IRC | 11:09 | |
pas-ha | dtantsur: about bug https://bugs.launchpad.net/ironic/+bug/1512898 - could you please check my last comment? | 11:10 |
openstack | Launchpad bug 1512898 in Ironic "Ironic grenade job should use IPA" [High,Triaged] - Assigned to Pavlo Shchelokovskyy (pshchelo) | 11:10 |
*** moshele has joined #openstack-ironic | 11:10 | |
*** hshiina has joined #openstack-ironic | 11:13 | |
*** awiddersheim has quit IRC | 11:15 | |
*** appu103_ has joined #openstack-ironic | 11:24 | |
*** thrash|g0ne is now known as thrash | 11:25 | |
mat128 | morning sambetts, TheJulia, mgould | 11:25 |
mgould | morning mat128 sambetts | 11:26 |
*** lucas-afk is now known as lucasagomes | 11:29 | |
lucasagomes | morning all | 11:29 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for portgroups in ironicclient https://review.openstack.org/297727 | 11:30 |
*** ccamacho is now known as ccamacho|lunch | 11:36 | |
*** bfournie has quit IRC | 11:36 | |
mgould | lucasagomes: morning! | 11:39 |
lucasagomes | o/ | 11:39 |
*** thrash has quit IRC | 11:40 | |
TheJulia | o/ | 11:47 |
*** thrash has joined #openstack-ironic | 11:48 | |
*** thrash has quit IRC | 11:48 | |
*** thrash has joined #openstack-ironic | 11:48 | |
*** jrist has joined #openstack-ironic | 11:49 | |
mikal | Is tinyipa considered production ready? | 11:49 |
mikal | I'm liking it in my testing, but the docs still describe it as experimental | 11:49 |
TheJulia | mikal: It has limited hardware support. Due to the nature of varying hardware platforms, we expect most deployments will need to build their own IPA deployment image that fits with-in their needs. As in some hardware behaves better with Fedora, where other hardware works just fine with CoreOS. | 11:51 |
lucasagomes | TheJulia, ++ | 11:52 |
*** derekh has quit IRC | 11:52 | |
sambetts | It is no longer experimental though, so the docs probably need updating now its throughly tested in CI | 11:52 |
TheJulia | mikal: tinyipa is (how does one describe best thing since sliced bread, but for testing?) totally awesome for testing though, and if you can get your hardware disk/network to function without issue under it, It is extremely fast at booting, which can save quite a bit of time depending on what your doing. | 11:53 |
TheJulia | mikal: mainly in the sense of using VMs to test, where nested virt can get.... rather... slow. | 11:54 |
mikal | So, tinyipa is working fine on these here supermicros. I was going to test HP tomorrow. | 11:54 |
mikal | But yeah ok, it sounds like we might need to consider different IPA images for different types of hardware | 11:54 |
mikal | Which was something I was braced for, but hadn't actually needed to do yet | 11:55 |
sambetts | I use it successfully for Cisco HW too, its all down to the drivers included in the base OS | 11:55 |
sambetts | in this case TinyCoreLinux | 11:55 |
TheJulia | mikal: You'll likely want to use Fedora for the HP gear | 11:55 |
mikal | TheJulia: coreos has been working so far on the HP Cloudlines at least | 11:56 |
TheJulia | mikal: Also, there is a special driver you'll need to install if you have a dynamic smart array. | 11:56 |
mikal | But yeah, I can test those tomorrow | 11:56 |
mikal | I am close to no longer caring for today | 11:56 |
TheJulia | mikal: okay, awesome! | 11:56 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Fix import of ironicclient and reformat docstring https://review.openstack.org/378526 | 11:57 |
*** baoli has joined #openstack-ironic | 11:58 | |
*** lucasagomes is now known as lucas-bbl | 12:01 | |
xavierr | good morning Ironic | 12:02 |
*** sdake has joined #openstack-ironic | 12:02 | |
TheJulia | good morning xavierr | 12:05 |
mat128 | mikal: it should be possible to build one image that boots on almost all machines | 12:08 |
mat128 | mikal: the tiny core we build has a new enough kernel to support most of the stuff out there | 12:08 |
TheJulia | mat128: does tinyipa build in any hardware managers? | 12:09 |
sambetts | TheJulia: only whats included in the IPA repo | 12:10 |
sambetts | TheJulia: but thats just because the build process doesn't have a way to inject other ones | 12:11 |
sambetts | yet | 12:11 |
sambetts | ;) | 12:11 |
sambetts | could be a good RFE | 12:11 |
sambetts | tbh I've always wondered about how your supposed to do that with the coreos image too | 12:12 |
*** mtanino has joined #openstack-ironic | 12:12 | |
mgould | morning xavierr | 12:12 |
TheJulia | sambetts: very carefully? :) | 12:13 |
TheJulia | sambetts: that or rip it apart, chroot into it, and inject it | 12:13 |
*** wajdi has joined #openstack-ironic | 12:15 | |
*** baoli has quit IRC | 12:16 | |
*** wajdi_ has joined #openstack-ironic | 12:16 | |
mat128 | TheJulia: same as CoreOS, we build from soruce | 12:17 |
mat128 | gotta go | 12:17 |
*** mat128 is now known as mat128|afk | 12:17 | |
TheJulia | well, out of tree hardware managers like the one provided by proliantutils | 12:18 |
TheJulia | But even then, like the dib element installs some other stuff that is required to control the array | 12:18 |
TheJulia | Stepping away for a little bit myself | 12:19 |
sambetts | so we need a way to specify addition packages to install during the build process | 12:19 |
sambetts | makes sense to me | 12:19 |
sambetts | currently all the build process just uses requirements.txt from IPAs repo | 12:20 |
*** wajdi has quit IRC | 12:20 | |
sambetts | so I guess you could append proliantutils to the end of that, but I don't have much experience with custom IPAs | 12:20 |
TheJulia | yeah, binary deps though :) | 12:20 |
TheJulia | That is a whole larger ball of wax that | 12:21 |
sambetts | yeah :/ | 12:21 |
sambetts | there are examples in the tinyipa build process for adding binarys from source that aren't in the tinyipa repos | 12:22 |
sambetts | so perhaps we can make some generic hook for doing that wizardry | 12:22 |
*** vikrant has quit IRC | 12:22 | |
TheJulia | I know there is a little interest in supporting busybox for dib and enabling tinyipa builds out of dib, but that is kind of crazy talk at the same time | 12:23 |
sambetts | Yeah I looked into turning my tinyipa scripts into DIB elements, but didn't get very far | 12:23 |
TheJulia | anyway, I need to step away | 12:24 |
sambetts | o/ | 12:25 |
mat128|afk | sambetts: there was some work done on bringing tinycore to dib | 12:25 |
mat128|afk | cant find it atm | 12:25 |
mat128|afk | have to go | 12:25 |
sambetts | o/ | 12:25 |
pas-ha | btw, I have a patch to ipa docs with general info on tinyipa Ic6cfe64255aba0460cc960a98b18580b7798b204 (gerrit is down for me :( ) | 12:27 |
pas-ha | https://review.openstack.org/#/c/374714/ | 12:28 |
patchbot | patch 374714 - ironic-python-agent - Add info on tinyipa to docs | 12:28 |
*** bfournie has joined #openstack-ironic | 12:28 | |
sambetts | pas-ha: ++ nice! I'll review asap | 12:29 |
pas-ha | thanks | 12:36 |
*** bks has quit IRC | 12:38 | |
*** vmud213 has quit IRC | 12:40 | |
*** e0ne has quit IRC | 12:40 | |
*** yuriyz has quit IRC | 12:41 | |
*** ppiela has joined #openstack-ironic | 12:41 | |
*** yuriyz has joined #openstack-ironic | 12:42 | |
*** Romanenko_K has quit IRC | 12:42 | |
*** ipukha has quit IRC | 12:42 | |
*** aarefiev has quit IRC | 12:42 | |
*** vnogin has quit IRC | 12:42 | |
*** Romanenko_K has joined #openstack-ironic | 12:43 | |
*** e0ne has joined #openstack-ironic | 12:43 | |
*** ipukha has joined #openstack-ironic | 12:43 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url https://review.openstack.org/352289 | 12:44 |
*** vnogin has joined #openstack-ironic | 12:44 | |
*** aarefiev has joined #openstack-ironic | 12:44 | |
*** rbrud has joined #openstack-ironic | 12:45 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add read and execute perms to httpboot and tftpboot folder https://review.openstack.org/374015 | 12:45 |
*** trown|outtypewww is now known as trown | 12:48 | |
*** Goneri has joined #openstack-ironic | 12:57 | |
*** david-lyle has joined #openstack-ironic | 12:57 | |
*** rloo has joined #openstack-ironic | 12:57 | |
*** vishwanathj has joined #openstack-ironic | 12:58 | |
*** baoli has joined #openstack-ironic | 12:59 | |
*** baoli_ has joined #openstack-ironic | 13:02 | |
betherly | jroll: we are on track to release ironic ui today! | 13:02 |
jroll | betherly: awesome \o/ | 13:03 |
jroll | morning everyone | 13:03 |
jroll | mikal: there's some really nice features that allow one agent to work across multiple hw platforms, we should talk more :) | 13:05 |
*** lucas-bbl is now known as lucasagomes | 13:05 | |
jroll | sambetts: if your curious how to add hw managers to coreos builder, we basically have a dockerfile that inherits the upstream one and adds a couple pip install lines | 13:06 |
*** baoli has quit IRC | 13:06 | |
sambetts | jroll: ah cool, yeah I think you could achieve the same thing if you added custom lines to the IPA requirements.txt for tinyipa | 13:06 |
sambetts | jroll: but thats a little hacky ;) | 13:07 |
jroll | sambetts: yeah, that's another route, it just means carrying patches | 13:07 |
sambetts | yeah | 13:07 |
*** ccamacho|lunch is now known as ccamacho | 13:08 | |
sambetts | I wonder about adding a ADDITIONAL_REQUIREMENTS_FILES= env variable for the tinyipa build that you could point at or something | 13:09 |
sambetts | :/ | 13:09 |
*** mtanino has quit IRC | 13:09 | |
*** hshiina has quit IRC | 13:09 | |
sambetts | but as TheJulia said, that doesn't help the binary deps you might have | 13:10 |
jroll | yeah | 13:12 |
openstackgerrit | Merged openstack/ironic-ui: Added support for editing Ironic nodes https://review.openstack.org/376052 | 13:12 |
jroll | sambetts: bindep might be an option there | 13:12 |
*** appu103_ has quit IRC | 13:18 | |
*** aswadr_ has joined #openstack-ironic | 13:18 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-specs: Torrent-based provisioning https://review.openstack.org/311091 | 13:20 |
openstackgerrit | Yushiro FURUKAWA proposed openstack/ironic-lib: Add prefix "$" for command examples https://review.openstack.org/378611 | 13:20 |
mariojv | morning ironic o/ | 13:20 |
jroll | o/ | 13:20 |
jroll | \o | 13:20 |
jroll | \o/ | 13:20 |
*** jaosorior has quit IRC | 13:20 | |
*** jaosorior has joined #openstack-ironic | 13:21 | |
*** wajdi_ has quit IRC | 13:24 | |
*** mat128|afk is now known as mat128 | 13:27 | |
sambetts | jroll: yeah, I wonder about making a generic way to inject dependecies from source, so they get built during the build process like with the other dependecies we have ' | 13:28 |
mat128 | jroll: have you had a chance to look at the docs backports? https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:stable/newton+topic:bug/1612278 | 13:28 |
xavierr | morning jroll | 13:28 |
mat128 | sambetts: another option is making your own python package with a dependency on ipa | 13:28 |
jroll | mat128: not yet but it is on my list for today | 13:28 |
mat128 | sambetts: IPAs entrypoint will be installed in the virtualenv and everything will just work (tm), but it is more work | 13:28 |
*** cdearborn has joined #openstack-ironic | 13:30 | |
sambetts | hmm interesting, I wonder how that would work because of how the tinyipa build script operates based on knowing its inside of the IPA repo directory :/ | 13:30 |
mat128 | sambetts: interesting.. | 13:31 |
mat128 | sambetts: I think wrapping it with more pip installs is probably the best | 13:31 |
mat128 | sambetts: maybe the user could provide a script with "final steps" | 13:32 |
mat128 | deployers could patch that in themselves pretty easily, but offering it to them is probably easier | 13:32 |
sambetts | ++ some helper scripts for building TinyCore packages etc would help too | 13:33 |
pester | hi all, i`m trying to boot node with fake_pxe driver, but my ironic-node stuck in "provision_state | clean failed" how to get it back to 'avaiable' ? | 13:35 |
sambetts | pester: you meed to move the node to manageable, then set state provide | 13:36 |
mat128 | pester: http://docs.openstack.org/developer/ironic/_images/states.svg | 13:36 |
mat128 | ^this | 13:36 |
pester | mat128: thx! | 13:37 |
*** links has quit IRC | 13:38 | |
*** athomas has joined #openstack-ironic | 13:38 | |
*** moshele has quit IRC | 13:39 | |
*** rbartal has quit IRC | 13:39 | |
pester | yet, node go again to - "provision_state | clean failed" I use fake_pxe driver, I don`t need cleaning... | 13:42 |
sambetts | pester: You can disable cleaning via the ironic.conf file | 13:42 |
pester | automated_clean ? | 13:43 |
sambetts | yup | 13:43 |
openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands https://review.openstack.org/374381 | 13:46 |
*** vnogin has quit IRC | 13:47 | |
pester | looks like it worked | 13:48 |
*** bdeschenes has joined #openstack-ironic | 13:48 | |
*** narasimha_SV has joined #openstack-ironic | 13:49 | |
narasimha_SV | http://paste.openstack.org/show/583301/ this is file for the server which it is getting generated for the server I am doing pxe booting | 13:50 |
narasimha_SV | http://paste.openstack.org/show/583302/ contents inside the id shown in above link | 13:52 |
narasimha_SV | am I missing anything here | 13:52 |
narasimha_SV | which is making my pxe boot stop after ramdisk installation | 13:52 |
narasimha_SV | http://paste.openstack.org/show/583303/ console output where it is getting stopped | 13:53 |
*** dprince has quit IRC | 13:53 | |
*** dprince has joined #openstack-ironic | 13:54 | |
jroll | sambetts: hey, is UCS hw inspection or OOB RAID config done? | 13:57 |
* jroll going through rfe-approved looking for finished things | 13:57 | |
sambetts | jroll: no, we have inspection with inspector but not OOB yet | 13:58 |
jroll | sambetts: cool thanks | 13:58 |
openstackgerrit | Ruby Loo proposed openstack/ironic: Change links to point to new install guide https://review.openstack.org/378640 | 13:59 |
rloo | mat128: ^^ we missed those | 13:59 |
*** mtanino has joined #openstack-ironic | 13:59 | |
openstackgerrit | Yushiro FURUKAWA proposed openstack/ironic-lib: Use 'code-block' for pieces of code https://review.openstack.org/378643 | 14:00 |
*** anush has joined #openstack-ironic | 14:01 | |
*** hoangcx has joined #openstack-ironic | 14:01 | |
*** mtanino has quit IRC | 14:04 | |
*** joprovost has joined #openstack-ironic | 14:07 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: Add headers and extra_headers to arguments in tempest plugin https://review.openstack.org/375570 | 14:08 |
xavierr | I saw this patch and I'm not sure it will be merged since it is for ssh driver. It already has +2. wdyt? https://review.openstack.org/#/c/366150/ | 14:09 |
patchbot | patch 366150 - ironic - Fixes parameters validation in SSH power manager. | 14:09 |
xavierr | rloo: any opinion on that? ^^^ | 14:10 |
xavierr | :) | 14:10 |
*** anush has quit IRC | 14:10 | |
*** narasimha_SV has quit IRC | 14:10 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 14:11 |
rloo | xavierr: dunno, are we not making changes to unsupported drivers? i suppose not? guess we should decide/make it explicit... | 14:11 |
rloo | jroll: ^^ do we stop making changes to unsupported/deprecated drivers? (I guess the question is if they are unsupported or not) | 14:11 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Mark specs implemented in Newton https://review.openstack.org/378657 | 14:11 |
jroll | easy peasy maybe review ^ | 14:11 |
xavierr | rloo: I agree with you | 14:12 |
jroll | rloo: I'm not opposed to bug fixes, features on the other hand I'd rather not bother with | 14:12 |
bfournie | morning ironic! sambetts: I have a question on local_link_connection. On a baremetal system I’m getting all lldp data including chassis-id (mac) and port-id. I added local_link_connection to ‘processing_hooks’ | 14:12 |
bfournie | in inspector.conf and then reran introspection, but don’t see local_link_connection populated in “ironic port-show”. Is there something else I need to do run GenericLocalLinkConnectionHook? | 14:12 |
*** narasimha_SV has joined #openstack-ironic | 14:12 | |
rloo | jroll: ok. should we send out email or indicate that somewhere for all to know? | 14:13 |
jroll | rloo: probably :) | 14:13 |
rloo | thx jroll, for volunteering | 14:13 |
jroll | :D | 14:13 |
* jroll adds to todo list for this week | 14:13 | |
xavierr | hahaha | 14:14 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Move flavor create under 'VIRT_DRIVER == ironic' https://review.openstack.org/378661 | 14:14 |
*** mkam has joined #openstack-ironic | 14:17 | |
*** anush has joined #openstack-ironic | 14:17 | |
xavierr | rloo, jroll: that email that you are talking about will be send to openstack mail list or ironic has one mail list? | 14:17 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Use correct option value for standalone install https://review.openstack.org/378664 | 14:18 |
jroll | xavierr: we use the openstack-dev mailing list with the tag [ironic] | 14:18 |
*** mkam has quit IRC | 14:18 | |
sambetts | bfournie: You might have to do ironic port-show with the right version of the Ironic API specified otherwise Ironic hides the field | 14:19 |
*** mkam has joined #openstack-ironic | 14:19 | |
openstackgerrit | Michael Turek proposed openstack/ironic: Enable PXE for systems using petitboot https://review.openstack.org/185987 | 14:19 |
* jroll echo 'OS_IRONIC_API_VERSION=latest' >> .zshrc | 14:19 | |
xavierr | jroll: thanks, I'll subscribe to that mailing list :) | 14:19 |
jroll | xavierr: I'll also be writing it down in our docs, fwiw :) | 14:20 |
bfournie | sambetts: I see the field, just no value | 14:20 |
bfournie | [stack@host01 etc]$ ironic port-show 62096bb4-6f3d-4359-8bdb-632bb68c88ae | 14:20 |
bfournie | +-----------------------+--------------------------------------+ | 14:20 |
bfournie | | Property | Value | | 14:20 |
bfournie | +-----------------------+--------------------------------------+ | 14:20 |
bfournie | | address | b0:83:fe:c6:63:86 | | 14:20 |
bfournie | | created_at | 2016-09-12T18:46:30+00:00 | | 14:20 |
bfournie | | extra | {} | | 14:20 |
bfournie | | internal_info | | | 14:20 |
bfournie | | local_link_connection | | | 14:20 |
bfournie | | node_uuid | 5f428939-698d-4942-b164-ff645a768e4a | | 14:20 |
bfournie | | pxe_enabled | | | 14:20 |
bfournie | | updated_at | None | | 14:20 |
bfournie | | uuid | 62096bb4-6f3d-4359-8bdb-632bb68c88ae | | 14:20 |
bfournie | +-----------------------+--------------------------------------+ | 14:20 |
jroll | bfournie: please use paste.openstack.org in the future :) | 14:20 |
bfournie | jroll: oops sorry, will do | 14:20 |
xavierr | bradjones: paste.openstack.org +1 :) | 14:21 |
jroll | bfournie: I don't believe we adjust the fields the client displays based on api version, though, try ironic --ironic-api-version 1.19 port-show 62096bb4-6f3d-4359-8bdb-632bb68c88ae | 14:22 |
xavierr | bradjones: sorry... it was bfournie :) | 14:22 |
jroll | bfournie: for reference, http://docs.openstack.org/developer/ironic/dev/webapi-version-history.html | 14:22 |
jroll | anything 1.19 or greater should work | 14:22 |
*** wajdi has joined #openstack-ironic | 14:22 | |
*** hamzy has quit IRC | 14:23 | |
bfournie | jroll: ok, thanks, still see an empty list - http://paste.openstack.org/show/583311/ - although it seems to be recognized now | 14:25 |
sambetts | bfournie: Can you see the debug logs in the ironic-inspector with regards to the processing of the LLDP packet? | 14:26 |
*** bks has joined #openstack-ironic | 14:27 | |
bfournie | sambetts: No I don't, actually I think I need to restart ironic-inspector to reread inspector.conf after adding local_link_connection to processing-hooks. I guess this is not done when I rerun introspection? | 14:28 |
*** mkam has quit IRC | 14:28 | |
sambetts | bfournie: no, the config files are only re-read on reboot of the service | 14:28 |
*** mkam has joined #openstack-ironic | 14:29 | |
openstackgerrit | Beth Elwell proposed openstack/ironic-ui: Added release notes for 2.1.0 release https://review.openstack.org/378668 | 14:29 |
sambetts | bfournie: some OpenStack services will re-read config files on SIGHUP, but none I know of will automatically detect config changes | 14:29 |
bfournie | sambetts: ok will do that, sorry for the noise. Can I just HUP ironic-inspector? | 14:30 |
sambetts | bfournie: I don't think we've implmented it yet :( | 14:30 |
*** baoli_ has quit IRC | 14:31 | |
bfournie | sambetts: ok | 14:31 |
*** baoli has joined #openstack-ironic | 14:31 | |
pas-ha | bfournie: AFAIK most services handle SIGHUP only for logging configuration reload (e.g. to enable debug w/o restarting service) | 14:32 |
bfournie | pas-ha: ah, OK, thanks | 14:33 |
rloo | jroll: we seem to have doc for config options in newton, although it seems a bit strange. maybe it isn't finished: http://docs.openstack.org/newton/config-reference/bare-metal.html | 14:33 |
*** lennyb has quit IRC | 14:34 | |
jroll | rloo: hrm, that is odd, looks like they split API config and the rest | 14:36 |
rloo | jroll: yeah, i didn't check but am not convinced all the config options are listed there, and the third one, about 'new..in Mitaka...'... | 14:37 |
jroll | rloo: yeah, something's odd, JayF would you have time to look at that today? | 14:38 |
rloo | jroll: in the/some past release, i looked/modified the config docs, but i didn't think to do it this time. | 14:38 |
jroll | JayF: or like, this week I guess | 14:38 |
*** AlexeyAbashkin has joined #openstack-ironic | 14:38 | |
jroll | rloo: so I think oslo config generator will generate that stuff now | 14:38 |
jroll | but am not 100% sure | 14:38 |
*** baoli has quit IRC | 14:38 | |
narasimha_SV | xavierr: http://paste.openstack.org/show/583301/ is this file generated for the server is correctly generated or not | 14:38 |
rloo | jroll: all i recall is that the way it was generated (whenever it was that I looked) was somewhat painful. you had to enter edit the section names and there was something odd about it all | 14:39 |
jroll | rloo: hm, idk, hoping jay can bug the docs team about it | 14:39 |
rloo | jroll: me too (cuz I ain't volunteerin') | 14:40 |
jroll | heh | 14:40 |
xavierr | hey narasimha_SV | 14:41 |
xavierr | narasimha_SV: which file is that? | 14:41 |
*** hshiina has joined #openstack-ironic | 14:42 | |
*** aNuposic has joined #openstack-ironic | 14:42 | |
*** rpioso has joined #openstack-ironic | 14:43 | |
rloo | hi lucasagomes, if you've got a few minutes (I hope it doesn't take more than that), could you take a look at the ipxe swift temp URL spec? Is it done now? https://review.openstack.org/#/c/372682/ | 14:48 |
patchbot | patch 372682 - ironic-specs - Amend ipxe-swift-tempurls spec | 14:48 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 14:48 |
lucasagomes | rloo, hi there, totally! Added here I will look at it today | 14:48 |
rloo | lucasagomes: good. when that spec is updated, we should move it to 6.2 (I think, need to confirm if 6.2 or 6.1) | 14:49 |
lucasagomes | ++ | 14:49 |
openstackgerrit | Merged openstack/ironic-specs: Mark specs implemented in Newton https://review.openstack.org/378657 | 14:49 |
*** vsaienk0 has joined #openstack-ironic | 14:49 | |
lucasagomes | 6.2.0 is the lastest release | 14:50 |
rloo | lucasagomes: right, just can't remember when the actual fix went in. aug 18 it sez. | 14:50 |
vsaienk0 | good morning Ironic'ers, lucasagomes, sambetts, dtantsur, vdrok, jroll, aarefiev and all! | 14:50 |
openstackgerrit | Merged openstack/ironic: Use correct option value for standalone install https://review.openstack.org/378664 | 14:51 |
rloo | lucasagomes: in 6.2, i looked at release notes :) | 14:51 |
vsaienk0 | I'm wondered if anyone else looking into grenade job failure or have any thoughts? | 14:51 |
lucasagomes | vsaienk0, i there | 14:51 |
lucasagomes | rloo, cool! | 14:52 |
vsaienk0 | lucasagomes: cool, I'm thinking that it is due to slow neutron | 14:52 |
rloo | vsaienk0: not me. I was hoping/assuming you and vdrok were... | 14:52 |
jroll | vsaienk0: I've been trying but haven't had much time :( | 14:52 |
rloo | vsaienk0: is it timing out? (shows how much i know of the problem. nothin') | 14:53 |
jroll | rloo: instances built before upgrade not pinging after upgrade | 14:53 |
xavierr | jroll: since we have our newton version tagged (6.2.0) wow should I proceed for solve a bug in newton release? how to indicate that my fix is for newton branch, for example? | 14:54 |
rloo | jroll: oh. i guess no one has been able to replicate it in their own dev env? | 14:54 |
vsaienk0 | have look here: http://logs.openstack.org/53/377653/1/check/gate-grenade-dsvm-ironic/a7488a5/logs/syslog.txt.gz#_Sep_28_12_26_12 in 12:26 during PXE boot VM didn't received an IP address | 14:54 |
jroll | xavierr: you need to fix it in master, and then backport it: http://docs.openstack.org/project-team-guide/stable-branches.html | 14:54 |
vsaienk0 | and VM try only once to get PXE, that it just stuck | 14:55 |
narasimha_SV | xavierr: that is the file genereated in side pxelinux.cfg folder for the server which I am doing pxe boot | 14:55 |
rloo | vsaienk0: is there any way to put a wait before that? to test your theory of neutron being slow? | 14:56 |
rloo | vsaienk0: the grenade test fails all the time, regardless of whether it is osic cloud or not? | 14:56 |
*** hamzy has joined #openstack-ironic | 14:57 | |
xavierr | jroll: thanks. and in launchpad I register the bug only for master branch or should I tell there that it will be backported? | 15:00 |
xavierr | narasimha_SV: I'll check mine | 15:00 |
*** syed_ has joined #openstack-ironic | 15:00 | |
vsaienk0 | rloo: all the time, I'm trying to increase timeout here: https://review.openstack.org/#/c/377653/ | 15:01 |
patchbot | patch 377653 - ironic - Testing broken grenade job | 15:01 |
jroll | xavierr: master is fine, you can register the branches if you like | 15:02 |
*** mjura has quit IRC | 15:03 | |
*** lennyb has joined #openstack-ironic | 15:03 | |
rloo | vsaienk0: so it isn't just grenade that is broken? I see gate-tempest-dsvm-ironic-ipa-wholedisk-pxe_ssh-tinyipa-nv failing too -- or is that a different issue? (I know, it is non-voting) | 15:03 |
narasimha_SV | xavierr: the uuid showing in that file is the folder id created inside pxelinux.cfg folder | 15:04 |
*** rcernin has quit IRC | 15:04 | |
vsaienk0 | rloo: wholedisk job failing due to another issue http://logs.openstack.org/70/375570/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-pxe_ssh-tinyipa-nv/999cce3/logs/screen-ir-cond.txt.gz?level=ERROR, dtantsur and vdrok working on it as far I know | 15:05 |
vdrok | Rloo ipa-wholedisk should be fixed after ironic lib release bumped in constraints Iirc | 15:09 |
xavierr | narasimha_SV: I dont have that file because I'm using iPXE | 15:09 |
vdrok | I'm on sick leave today, so sorry, no progress on anything :( | 15:09 |
xavierr | hey vdrok o/ | 15:09 |
rloo | vdrok: no worries. get better!! | 15:10 |
rloo | vsaienk0: oh, right, I recall the frenzy of activity about that :) | 15:10 |
rloo | vsaienk0: wrt your patch, are you incrementing the value each time, and nothing works yet? | 15:11 |
JayF | jroll: rloo: yeah; it's kinda ugly. Do we have an example of a "good" version of this from another project? | 15:15 |
rloo | vsaienk0: my only contribution is to add to the etherpad, the link to your patch and that you are looking at it | 15:15 |
JayF | jroll: rloo: I'm just not always the best at determining what's good in these kind of docs :) | 15:15 |
* JayF back to work after being out sick | 15:15 | |
rloo | JayF: glad you're better! | 15:15 |
JayF | So it's just weird that API is split out | 15:15 |
rloo | JayF: if you look at the 'mitaka' version it is a bit different | 15:16 |
mat128 | JayF: welcome back! | 15:16 |
JayF | the new/updated/deprecation options for mitaka looks like it needs to be migrated to a newton section? | 15:16 |
rloo | JayF: I haven't looked at any other project's configs for newton. | 15:16 |
*** amotoki has quit IRC | 15:16 | |
rloo | JayF: probably. I'm not quite sure what that section is meant to convey. 'what changed'? is it the option itself, or help string for option, or possible values for option... | 15:17 |
JayF | yeah; I'll drop an email to someone I know who works on this | 15:17 |
JayF | and try to determine wtf is the right thing to do | 15:17 |
*** absubram has joined #openstack-ironic | 15:18 | |
mat128 | rloo: good catch https://review.openstack.org/#/c/378640/ | 15:18 |
patchbot | patch 378640 - ironic - Change links to point to new install guide | 15:18 |
xavierr | narasimha_SV: tell me the commands you are using to register the images in glance. look in your history | 15:18 |
JayF | hey mat128, nice work on the install guide, I looked at a rendered version live yesterday and it looked great! | 15:20 |
rloo | mat128: I was reading the multitenancy doc and noticed it. so hard to remember 'everything'... | 15:20 |
JayF | mat128: do I understand correctly that we're backporting it to newton? If so, we need to have a party \o/ | 15:20 |
mat128 | rloo: agreed | 15:20 |
mat128 | JayF: thanks :) and yes we're backporting it | 15:20 |
mat128 | Andreas asked for it on the ML | 15:20 |
JayF | I saw that | 15:21 |
rloo | JayF: yup. supposedly on jroll's list to +2. I guess we need two people to get them +A'd/backported :) | 15:21 |
mat128 | rloo: concerning your last concern, do you think we should change them to "ref:" in master? | 15:21 |
JayF | tonyb has been willing to help us land that stuff in the past if deva or dmitry aren't around; we can ping him if it gets too late and it hasn't been fixed. | 15:21 |
rloo | mat128: the security page? yes. | 15:21 |
*** lucasagomes is now known as lucas-hungry | 15:22 | |
rloo | mat128: and maybe just check that there aren't other cases like that. | 15:22 |
mat128 | they're hard to find | 15:22 |
mat128 | next time someone commits a link we should ask ourselves if a ref: would have been possible | 15:22 |
rloo | mat128: if you would like to do that, it'd be great. given that it has been 'broken', i don't care too much if it is fixed in stable/newton or not. | 15:22 |
rloo | mat128: yup, but... so hard to remember everything when reviewing :-( | 15:23 |
JayF | I'm jotting down some items here, things we want to look into for improving in docs this cycle. feel free to add things if you wanthttps://etherpad.openstack.org/p/Ironic-Ocata-Docs | 15:23 |
*** xek__ has joined #openstack-ironic | 15:23 | |
mat128 | JayF: good idea ^ | 15:25 |
*** xek_ has quit IRC | 15:25 | |
jroll | JayF: depending how many things you get in there, it might be worth a summit session? | 15:28 |
JayF | jroll: I can't be at the summit, so whether or not it's a session is not something I think I can really decide? | 15:28 |
JayF | jroll: someone else would have to lead it, and it'd suck to have those priorities made without my input :( | 15:29 |
jroll | JayF: true :( | 15:29 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 15:30 |
*** baoli has joined #openstack-ironic | 15:31 | |
*** rajinir has joined #openstack-ironic | 15:32 | |
rajinir | <jlvillal>: Probably yes, Is it broke on subnet creation? | 15:32 |
zigo | ironic-ui is being very funny with its unit tests: http://logs.openstack.org/94/378694/2/check/gate-deb-ironic-ui-dpkg-debian-jessie/fae609b/console.html#_2016-09-28_15_24_42_991137 | 15:35 |
zigo | no, no, ... no no no ... you can't write in /usr !!! :) | 15:35 |
JayF | /o\ | 15:36 |
zigo | Who wrote ironic-ui ? | 15:36 |
zigo | How can I change this path? | 15:36 |
JayF | krotschek (sp?) isn't here | 15:37 |
JayF | betherly: cinerama: I think one of you work on this? ^ | 15:37 |
betherly | ye i work on it. one sec let me look | 15:37 |
zigo | I think I know how. | 15:38 |
zigo | ironic-ui is simply importing openstack_dashboard/test/settings.py who has: | 15:38 |
zigo | TEST_DIR = os.path.dirname(os.path.abspath(__file__)) | 15:39 |
zigo | SECRET_KEY = secret_key.generate_or_read_from_file( | 15:39 |
zigo | os.path.join(TEST_DIR, '.secret_key_store')) | 15:39 |
*** bks has quit IRC | 15:39 | |
JayF | if it's in openstack_dashboard; I wonder why you haven't seen it in the other horizon packages | 15:39 |
*** bks has joined #openstack-ironic | 15:39 | |
betherly | zigo: thats part of the horizon package if im reading right? | 15:39 |
betherly | is that ok or what do i need to change? | 15:40 |
betherly | getting ready to release v2.1.0 but dont want to release something with an error im not aware of | 15:40 |
zigo | betherly: Yes, but probably ironic-ui is assuming that the folder containing this file is writable, which it is *not* in the context of packaging. | 15:40 |
zigo | betherly: So ironic-ui needs to define a correct path for the secret key during tests, which is *not* something in the horizon package. | 15:42 |
zigo | (sorry for the *emphasys* but it's often easy to miss the *not* if it doesn't have stars...) | 15:42 |
*** hoangcx has quit IRC | 15:43 | |
betherly | zigo: riiiight....ok....not entirely sure how to do that lol.... | 15:43 |
jroll | this does seem like an openstack-dashboard bug, not ironic-ui | 15:45 |
zigo | betherly: I'm attempting to patch. | 15:45 |
betherly | zigo: thanks :) | 15:45 |
jroll | SECRET_KEY is a django thing, I doubt ironic-ui does anything special here | 15:45 |
betherly | jroll: thanks for the clarification! | 15:46 |
jroll | betherly: one way you could hack it for tests is something like... make a temp directory and override openstack_dashboard.test.settings.SECRET_KEY to point there | 15:47 |
jroll | or just set openstack_dashboard.test.settings.SECRET_KEY = 'foo' | 15:47 |
jroll | or something | 15:47 |
zigo | jroll: The only thing is that in the case of unit tests during package build, the secret key path is somewhere in horizon's Python module, so that's in /usr/lib/python2.7/dist-package. That's read only ... | 15:47 |
zigo | jroll: For horizon, that's fine, as it's not installed when building the package. | 15:47 |
zigo | For ironic-ui, that's a no-go ... | 15:48 |
zackf | Trying to provision a BM node and i'm hitting an error i can't get passed... It goes through scheduling finds my baremetal node, attempts to deploy and dies with "Failed to prepare to deploy. Error: `/httpboot/boot.ipxe` and `/httpboot/boot.ipxe` are the same file" | 15:48 |
*** kumarb has joined #openstack-ironic | 15:48 | |
*** bks has quit IRC | 15:48 | |
zigo | So, while re-using the test-settings from Horizon is a good idea, this type of path should be overwriten. | 15:48 |
jroll | zigo: let me put up a patch and see if it works for you | 15:48 |
zackf | Anyone run into this kind of thing in their experiences? | 15:48 |
zigo | jroll: I'm attempting to do this: https://review.openstack.org/#/c/378694/3/debian/patches/fix-secret-key-path.patch | 15:49 |
patchbot | patch 378694 - deb-ironic-ui (debian/newton) - Merge tag '2.0.0' into debian/newton | 15:49 |
zigo | Let's see if that builds ... | 15:49 |
jroll | zigo: I think just making the import will break you | 15:49 |
* jroll tries alternate way just in case | 15:49 | |
zigo | Ah... | 15:49 |
zigo | Maybe, if it tries to write directly at the import time. | 15:50 |
*** aNuposic has quit IRC | 15:50 | |
zigo | Oh, right, it's doing secret_key.generate_or_read_from_file() ... :( | 15:50 |
zigo | jroll: What are you attempting? | 15:50 |
*** kumarb has quit IRC | 15:50 | |
jroll | zigo: some mock magic | 15:51 |
zigo | Maybe out of my skills then! :) | 15:51 |
*** joprovost1 has joined #openstack-ironic | 15:51 | |
*** bks has joined #openstack-ironic | 15:51 | |
zigo | betherly: jroll: Please ping me when you have some news, I'm attempting to package another Horizon plugin now. | 15:51 |
*** joprovost has quit IRC | 15:53 | |
*** joprovost1 is now known as joprovost | 15:53 | |
jroll | will do | 15:53 |
*** electrofelix has quit IRC | 15:56 | |
*** rbrud has quit IRC | 15:59 | |
jroll | zigo: betherly: https://review.openstack.org/#/c/378744/ | 15:59 |
patchbot | patch 378744 - ironic-ui - Mock openstack_dashboard secret key in unit tests | 15:59 |
jroll | tests pass on my system, I *think* it does what you need | 16:00 |
*** krtaylor has quit IRC | 16:01 | |
betherly | jroll: awesome thanks so much for sorting that! | 16:01 |
jroll | betherly: np, let's see if it works for zigo before merging, but we should get that in the release | 16:01 |
jlvillal | Good morning everyone :) | 16:01 |
betherly | jroll: for sur | 16:01 |
betherly | *sure | 16:01 |
zigo | betherly: jroll: I'm trying to apply this patch to the latest release of ironic-ui... | 16:01 |
jroll | thanks | 16:02 |
*** moshele has joined #openstack-ironic | 16:02 | |
*** mtanino has joined #openstack-ironic | 16:04 | |
zigo | Package is building... | 16:05 |
*** tuan_luong has joined #openstack-ironic | 16:06 | |
*** tuan_luong has quit IRC | 16:06 | |
*** ijw has joined #openstack-ironic | 16:07 | |
*** moshele has quit IRC | 16:11 | |
zigo | Manila-ui has the exact same issue, lol ! http://logs.openstack.org/15/378715/2/check/gate-deb-manila-ui-dpkg-debian-jessie/71ac58e/console.html#_2016-09-28_16_04_47_405008 | 16:11 |
*** vgadiraj has joined #openstack-ironic | 16:15 | |
*** Romanenko_K has quit IRC | 16:16 | |
*** jpich has quit IRC | 16:16 | |
openstackgerrit | Charlle Daniel proposed openstack/ironic: Ensures that OneView nodes are free for use by Ironic https://review.openstack.org/372762 | 16:16 |
*** lucas-hungry is now known as lucasagomes | 16:20 | |
*** strigazi is now known as strigazi_AFK | 16:21 | |
*** Guest46101 is now known as mgagne | 16:21 | |
*** mgagne has quit IRC | 16:21 | |
*** mgagne has joined #openstack-ironic | 16:21 | |
betherly | zigo: is the patch passing the tests yet? | 16:27 |
*** jaosorior has quit IRC | 16:27 | |
*** athomas has quit IRC | 16:28 | |
*** jrist has quit IRC | 16:28 | |
jlvillal | vsaienk0: Any chance that the subnetpools could be related to the issues we are seeing? https://review.openstack.org/#/c/356026/ Though it didn't merge until Monday. Not sure when the Grenade job stopped working. | 16:33 |
patchbot | patch 356026 - openstack-dev/devstack - Make the Neutron l3 plugin use the subnetpools (MERGED) | 16:33 |
*** baoli has quit IRC | 16:34 | |
*** baoli has joined #openstack-ironic | 16:35 | |
*** aNuposic has joined #openstack-ironic | 16:35 | |
*** thrash is now known as thrash|biab | 16:36 | |
*** sabeen has joined #openstack-ironic | 16:40 | |
*** frontrunner has quit IRC | 16:41 | |
NobodyCam | Good morning Ironic'ers | 16:47 |
*** krtaylor has joined #openstack-ironic | 16:50 | |
lucasagomes | NobodyCam, hi there | 16:51 |
NobodyCam | morning lucasagomes :) | 16:51 |
jroll | morning NobodyCam, lucasagomes :) | 16:51 |
NobodyCam | morning jroll :) | 16:51 |
zigo | jroll: betherly: It didn't work: http://logs.openstack.org/94/378694/4/check/gate-deb-ironic-ui-dpkg-debian-jessie/a9a0c46/console.html#_2016-09-28_16_18_55_754863 | 16:51 |
zigo | I have to go for dinner unfortunately. | 16:51 |
*** e0ne has quit IRC | 16:52 | |
jroll | lame :( | 16:52 |
zigo | I'll be there tonight. | 16:52 |
jroll | I think I know what I did wrong | 16:52 |
betherly | ugh :/ | 16:52 |
zigo | jroll: You could take over my packaging patch and try there if you'd like: https://review.openstack.org/378694 | 16:52 |
patchbot | patch 378694 - deb-ironic-ui (debian/newton) - Merge tag '2.0.0' into debian/newton | 16:52 |
zigo | The way it works is that you add the patch in debian/patches | 16:53 |
zigo | Got to go now, TTYL | 16:53 |
jroll | zigo: got it, thanks | 16:53 |
lucasagomes | did someone try to PXE boot over IPv6 with dnsmasq (neutron) ? I'm getting a http://paste.openstack.org/show/583329 , so I get an IP but apparently the options (bootfile-name, next-server) are not being passed | 17:00 |
*** rama_y has joined #openstack-ironic | 17:00 | |
lucasagomes | reading around it seems that dnsmasq lacks DHCPv6 extensions for tftp booting ? | 17:00 |
jroll | zigo: I've no clue how to do this, giving me branch errors and such :( | 17:00 |
jroll | zigo: I updated my patch, not sure it'll work nor the best way to test this | 17:00 |
sambetts | jlvillal: qa meeting today? | 17:01 |
jroll | guess not! | 17:02 |
jlvillal | sambetts: jroll We are all in the meeting. | 17:02 |
jlvillal | Where are you all? | 17:02 |
jroll | oh wow | 17:02 |
sambetts | jlvillal: literally must have started just as I switched back from the channel | 17:03 |
jroll | yep | 17:03 |
*** thrash|biab is now known as thrash | 17:03 | |
devananda | g'morning, all | 17:12 |
betherly | o/ devananda :) | 17:12 |
*** bks has quit IRC | 17:16 | |
*** MattMan has quit IRC | 17:21 | |
*** MattMan has joined #openstack-ironic | 17:21 | |
sambetts | o/ devananda | 17:22 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing a revert to see if fixes Grenade https://review.openstack.org/378928 | 17:23 |
*** praneshp has joined #openstack-ironic | 17:25 | |
mariojv | rloo: yuriyz: hey, regarding the comments on https://review.openstack.org/#/c/321865/33/ironic/conductor/manager.py | 17:25 |
patchbot | patch 321865 - ironic - Add power state change notifications | 17:25 |
mariojv | do you think switching the power sync notification on L2470 to "baremetal.node.power_sync.error" event type would be clearer? | 17:26 |
lucasagomes | devananda, betherly jlvillal o/ morning | 17:26 |
mariojv | i thought the docs were OK in describing what the notification means, but maybe not | 17:26 |
mariojv | i'm open to changing the event type string to something like "baremetal.node.power_state_changed.success" or something | 17:27 |
mariojv | if that's clearer | 17:27 |
rloo | mariojv: i don't remember :-( I mean, we talked about it then I went along my merry way, then when I looked again, I became confused... | 17:27 |
rloo | mariojv: what exactly are you/we trying to notify about? | 17:27 |
vdrok | jlvillal: grenade job stopped working after the base branch changed to stable Newton in devstack gate | 17:27 |
rloo | mariojv: I *thought* it was that we sync'd the node (db) power state to be that of the actual hw node's power state. | 17:28 |
mariojv | rloo: any time ironic detects that the actual hardware power state changed from what ironic has in the DB for that node | 17:28 |
mariojv | rloo: correct | 17:28 |
rloo | mariojv: so how could the notification be an .error? | 17:28 |
jlvillal | vdrok: Ah | 17:28 |
rloo | mariojv: there was no error in setting node.power_state == actual_state | 17:28 |
mariojv | rloo: that's why i put it as success - the error had nothing to do with ... exactly | 17:28 |
mariojv | so i think baremetal.node.power_sync.success is ok | 17:29 |
mariojv | but i'm open to changing the action part of the event type ("power_sync") to something else, if it's clearer | 17:29 |
mariojv | it's sort of a hard thing to concisely describe | 17:29 |
rloo | mariojv: but yuriyz thinks it should be .error. I'm wondering what he thinks the notification is for. | 17:29 |
* sambetts -> home | 17:29 | |
sambetts | night all | 17:29 |
rloo | mariojv: yeah, easy to describe but not concisely. | 17:29 |
*** sambetts is now known as sambetts|afk | 17:29 | |
rloo | night sambetts|afk | 17:29 |
jlvillal | sambetts|afk: Good night | 17:30 |
rloo | mariojv: power_reset? I forgot what i had proposed before. | 17:30 |
vdrok | Jlvillal So I think it can be anything merged in devstack since mitaka :( | 17:30 |
mariojv | rloo: i think that was just because of a previous comment i made - there is an error logged, but it's unrelated to the operation of changing the db state to actual power state | 17:30 |
JayF | I think it's arguable (I don't feel this way) that a node's power status being modified by something other than ironic is an erorr | 17:30 |
JayF | so if you run power sync and it's out of sync, I could see someone feeling like that was an error | 17:30 |
mariojv | rloo: i think you proposed power_sync, but i don't remember | 17:30 |
mat128 | JayF: ^we use "sync power state = off" and may control it ourselves outside of Ironic | 17:30 |
JayF | mat128: we use the setting to "make the db match reality" because that's what we do too; that's why I don't think it's an error | 17:31 |
mariojv | yeah, i don't think it's an error either | 17:31 |
rloo | JayF: that is a valid point. which goes back to my question, *what* are we (theRoyalWe) trying to convey in the notification? | 17:31 |
JayF | mat128: but I think it's a point where other folks COULD think that | 17:31 |
mariojv | ironic did nothing wrong, so if nothing else it's not an ironic error | 17:31 |
jlvillal | morning lucasagomes | 17:31 |
mat128 | JayF: so you mean, if Ironic has to act on the device because it thought otherwise? | 17:31 |
mat128 | JayF: I could see a notification for this, but not of type "error" | 17:31 |
mariojv | rloo: i/we are trying to convey that the db state of the node was updated to the actual power state of the node as returned by the driver | 17:31 |
mariojv | rloo: maybe i should guard that piece in an if statement, i think right now it notifies whether it changed it or not, on L2470 | 17:32 |
JayF | mat128: ++ I agree. We do not disagree. lol | 17:32 |
rloo | mariojv: ok, that is settled then. how to convey that. | 17:32 |
mariojv | power_sync is fine imo | 17:32 |
mariojv | as long as the docs are accurate | 17:32 |
rloo | i think we should use something else besides power_sync. the more i think of it, the more it is going to confuse ironic developers. | 17:32 |
rloo | power_set_to_hw? | 17:33 |
mariojv | power_state_modified? | 17:33 |
JayF | how about power_synced_to_db | 17:33 |
JayF | vs power_synced_to_hw | 17:33 |
mariojv | i'm fine with power_state_modified or power_synced_to_db | 17:33 |
rloo | JayF: can you think of another word than 'sync'? | 17:33 |
mariojv | power_set_to_hw sounds like it could be the same as power_set | 17:33 |
rloo | mariojv, JayF: power_adjust? no. | 17:34 |
rloo | power_match? | 17:34 |
rloo | power_equal? | 17:35 |
JayF | I actually like power_state_modified, or similar | 17:35 |
mat128 | changed | 17:35 |
JayF | if I could think of a term that would be used if we modified the actual power of the node instead | 17:35 |
mariojv | what's wrong with power_sync_to_db? i think the "to_db" part conveys that this is separate from ironic's power sync loop, but imbw | 17:35 |
mariojv | ooh | 17:36 |
mariojv | power_state_corrected? | 17:36 |
JayF | oh, I like that | 17:36 |
rloo | power_state_adjusted? | 17:36 |
JayF | mariojv: how would you indicate corrected via ipmi or corrected via db? | 17:36 |
*** ralonsoh has quit IRC | 17:36 | |
mariojv | power_state_corrected_in_db? | 17:36 |
mariojv | JayF: if there's a driver (aka ipmi) call to change it, that sends a power_set notification | 17:37 |
mariojv | this is exclusively for when ironic changes its db representation of the node to the hardware power state | 17:37 |
mariojv | this being what's currently named power_sync | 17:37 |
JayF | I like power_state_corrected for that | 17:37 |
rloo | JayF: this notification is *only* used when changing the nodes' db power_state value to that of the actual hw's state. the hw node doesn't change at all. | 17:37 |
mat128 | ^i like that | 17:37 |
JayF | or power_state_matched | 17:37 |
mat128 | they are 2 separate actions | 17:37 |
mariojv | i'm fine with power_state_matched, too | 17:38 |
mariojv | adjusted to me might indicate that ironic adjusted the hw power state, which isn't the case here | 17:38 |
JayF | yeah, I don't like adjusted | 17:38 |
rloo | hey, i'm still looking through the thesaurus, might come up with some more :) | 17:38 |
JayF | for that reason | 17:38 |
mariojv | http://www.thesaurus.com/browse/sync?s=t | 17:38 |
mariojv | power_state_harmonized :P | 17:38 |
JayF | rloo: me too :P and I have a nice ad for Ford cars w/Sync technology as a result, lol | 17:38 |
rloo | power_state_reconcile :) | 17:38 |
mariojv | haha. power_state_reconciled is fine with me too | 17:38 |
JayF | I think jibe would be my favorite joke suggestion | 17:39 |
*** baoli has quit IRC | 17:39 | |
*** baoli has joined #openstack-ironic | 17:39 | |
mariojv | so besides jibed, which is by far the best choice, which do people prefer? 1. power_state_corrected 2. power_state_matched 3. power_state_reconciled 4. other ? | 17:39 |
rloo | JayF: i like jibe but ... | 17:39 |
*** baoli has quit IRC | 17:40 | |
mat128 | mariojv: power_state_corrected, as long as I can know which of them was corrected | 17:40 |
rloo | restore? | 17:40 |
rloo | no, forget restore | 17:40 |
*** baoli has joined #openstack-ironic | 17:41 | |
mariojv | mat128: yeah, it will be clear in the payload | 17:41 |
rloo | mariojv: i like corrected & reconciled | 17:41 |
mat128 | mariojv: will there be a different notification if it ends up calling the power driver? | 17:41 |
rloo | mat128: yup. that's a power_set notification | 17:41 |
mariojv | mat128: to change the power, yes | 17:41 |
JayF | mat128: it would also send a power_set notification | 17:41 |
mariojv | yup, what rloo said | 17:41 |
mat128 | thats perfect then | 17:42 |
mariojv | rloo: ok, if no vehement disagreement, i'll go with power_state_corrected for the next patch set | 17:42 |
mariojv | thanks! | 17:42 |
*** aswadr_ has quit IRC | 17:42 | |
rloo | mariojv: for now anyway :) | 17:42 |
mariojv | :( | 17:42 |
rloo | mariojv: i did mention, didn't i, that maybe that patch could be separated, to only power_set, then add the other notifcation in a separate patch. ha ha. | 17:43 |
rloo | mariojv: but i think we should be good as long as the documentation is clear... | 17:43 |
mariojv | ok, i'll re-read it and try to read from the perspective of someone who has no idea what this is all supposed to do | 17:43 |
mariojv | maybe it would help if i talked about an example of how someone would use this to track power state of a node over time, but that may be overkill | 17:44 |
*** e0ne has joined #openstack-ironic | 17:44 | |
rloo | mariojv: nah. just clearly describe them. | 17:44 |
rloo | mariojv: and don't use 'sync' anywhere when describing power_state_corrected | 17:45 |
mariojv | ack, thx | 17:46 |
rloo | JayF: a really easy +2/+A. I verified it :) https://review.openstack.org/#/c/378291/ | 17:46 |
patchbot | patch 378291 - ironic - Fix inexact config option name in multitenancy.rst | 17:46 |
rloo | or NobodyCam ^^ | 17:47 |
JayF | rloo: I got it | 17:47 |
NobodyCam | hey hey morning rloo | 17:47 |
NobodyCam | morning JayF | 17:47 |
NobodyCam | and thank you :) | 17:47 |
JayF | rloo: I just was checking to see who we were going to make rebase; and it was you, so did it ;P | 17:47 |
rloo | hi NobodyCam, you're too late :) | 17:47 |
NobodyCam | lol | 17:47 |
rloo | JayF: yeah, I have to redo mine anyway, so thought might as well get that in first. | 17:47 |
*** hshiina has quit IRC | 17:48 | |
betherly | zigo: jroll: thanks so much for trying to sort this out! so appreciated! please do ping me and keep me posted | 17:52 |
openstackgerrit | Ruby Loo proposed openstack/ironic-specs: Amend ipxe-swift-tempurls spec https://review.openstack.org/372682 | 17:56 |
*** trown is now known as trown|lunch | 17:57 | |
openstackgerrit | Merged openstack/ironic: Fix inexact config option name in multitenancy.rst https://review.openstack.org/378291 | 18:00 |
*** sabeen has quit IRC | 18:02 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Update multitenancy documentation https://review.openstack.org/378138 | 18:16 |
*** ijw has quit IRC | 18:16 | |
*** jheroux has joined #openstack-ironic | 18:18 | |
*** rbrud has joined #openstack-ironic | 18:21 | |
narasimha_SV | xavierr: http://paste.openstack.org/show/583358/ these are the commands. for the kernel and ramdisk details , I am replacing the variables with the IDs | 18:27 |
*** baoli has quit IRC | 18:28 | |
*** krtaylor has quit IRC | 18:32 | |
*** vgadiraj has quit IRC | 18:34 | |
xavierr | narasimha_SV: just a minute :) | 18:35 |
*** rpioso has quit IRC | 18:41 | |
*** ijw has joined #openstack-ironic | 18:41 | |
*** narasimha_SV has quit IRC | 18:42 | |
*** vsaienko has joined #openstack-ironic | 18:45 | |
*** sdake has quit IRC | 18:46 | |
*** ijw has quit IRC | 18:46 | |
xavierr | morning NobodyCam :) | 18:47 |
NobodyCam | morning xavierr :) | 18:47 |
*** krtaylor has joined #openstack-ironic | 18:48 | |
*** rpioso has joined #openstack-ironic | 18:55 | |
*** rbrud has quit IRC | 18:55 | |
*** baoli has joined #openstack-ironic | 18:56 | |
*** hogepodge has quit IRC | 19:01 | |
*** trown|lunch is now known as trown | 19:03 | |
*** baoli has quit IRC | 19:05 | |
*** baoli has joined #openstack-ironic | 19:06 | |
*** hoonetorg has quit IRC | 19:14 | |
*** anush has quit IRC | 19:17 | |
*** anush has joined #openstack-ironic | 19:17 | |
*** rpioso has quit IRC | 19:20 | |
*** vsaienko has quit IRC | 19:24 | |
*** vsaienko has joined #openstack-ironic | 19:25 | |
*** vsaienko has quit IRC | 19:28 | |
*** vsaienko has joined #openstack-ironic | 19:29 | |
*** sabeen has joined #openstack-ironic | 19:31 | |
*** hoonetorg has joined #openstack-ironic | 19:31 | |
openstackgerrit | Stenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation https://review.openstack.org/377073 | 19:32 |
*** sdake has joined #openstack-ironic | 19:34 | |
*** rpioso has joined #openstack-ironic | 19:34 | |
*** tangchen_ has quit IRC | 19:45 | |
*** tangchen_ has joined #openstack-ironic | 19:45 | |
*** vsaienko has quit IRC | 19:50 | |
*** vsaienko has joined #openstack-ironic | 19:51 | |
*** ppiela has quit IRC | 19:57 | |
*** vsaienko has quit IRC | 19:59 | |
lucasagomes | calling it a day! Talk you tomorrow | 20:00 |
*** lucasagomes is now known as lucas-afk | 20:04 | |
rloo | bye lucas-afk | 20:04 |
xavierr | night lucas-afk | 20:06 |
rloo | xavierr: you had already looked at this, but some questions came up. would be good if you or some other oneview folks looked: https://review.openstack.org/#/c/374989/ | 20:07 |
patchbot | patch 374989 - ironic - Avoid name errors in oneview periodics | 20:07 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Fix import of ironicclient and reformat docstring https://review.openstack.org/378526 | 20:12 |
openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Update external links to developer documentation https://review.openstack.org/378990 | 20:15 |
openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Update configuration reference link to latest draft https://review.openstack.org/378991 | 20:22 |
openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Point configuration reference and sample config to Newton https://review.openstack.org/378994 | 20:23 |
*** Goneri has quit IRC | 20:27 | |
mat128 | rloo: addressed your points in https://review.openstack.org/#/c/378037/ via new commits ^ | 20:28 |
patchbot | patch 378037 - ironic (stable/newton) - Point documentation links to newton equivalents | 20:28 |
rloo | mat128: thx. | 20:28 |
*** baoli has quit IRC | 20:29 | |
rloo | mat128: this conflicts with your other patches: https://review.openstack.org/#/c/378994/ | 20:32 |
patchbot | patch 378994 - ironic (stable/newton) - Point configuration reference and sample config to... | 20:32 |
mat128 | oh wow | 20:33 |
mat128 | it conflicts because it touches the same file | 20:33 |
mat128 | I can put it at the end of the patch chain | 20:33 |
rloo | mat128: i've seen where it doesn't conflict if the same file, it depends on the lines that are touched. | 20:33 |
rloo | mat128: yeah, if you put it at the end, that should work. | 20:33 |
rloo | mat128: i would have put it with the 'Point documentation links to newton equivalents' patch myself, instead of a new patch. | 20:34 |
mat128 | riight | 20:34 |
*** rpioso has quit IRC | 20:34 | |
mat128 | my only concern was the manual edits vs commands | 20:34 |
mat128 | I guess I can merge and note it in the commit message | 20:34 |
mat128 | the goal is the same | 20:34 |
rloo | mat128: oh, just note it in the commit msg :) | 20:35 |
rloo | mat128: or add new seds to do that :D | 20:35 |
mat128 | lol | 20:35 |
*** wajdi_ has joined #openstack-ironic | 20:35 | |
mikal | jroll: good morning. Yes, we should talk about the agent more. I've founda t least one bug in the coreos agent and am having some success with tinyipa, but this area looks like its got some interesting opportunities | 20:35 |
openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Point documentation links to newton equivalents https://review.openstack.org/378037 | 20:36 |
mat128 | rloo: squashed + abandoned the other one | 20:36 |
rloo | mat128: thx | 20:36 |
* mat128 is getting better and better with git | 20:36 | |
*** wajdi__ has joined #openstack-ironic | 20:37 | |
mat128 | mikal: both ramdisks should have the same features. Did you file a bug? | 20:38 |
*** wajdi has quit IRC | 20:39 | |
*** wajdi_ has quit IRC | 20:39 | |
mikal | mat128: yes, I filed a bug for the coreos problem I found | 20:41 |
mikal | mat128: it turns out that calling shutdown inside a docker container doesn't do what the author had hoped it did | 20:41 |
mat128 | mikal: it's running from a chroot now, we used to issue reboot via sysrq exactly for this | 20:42 |
mat128 | let me look at the report :) | 20:42 |
jroll | mikal: morning :) | 20:42 |
mat128 | ouch | 20:42 |
mat128 | "Running in chroot, ignoring request" | 20:42 |
mikal | https://bugs.launchpad.net/ironic-python-agent/+bug/1628367 | 20:42 |
openstack | Launchpad bug 1628367 in ironic-python-agent "shutdown.sh fails with coreos IPA image" [Undecided,New] | 20:42 |
*** wajdi__ has quit IRC | 20:43 | |
jroll | that's a fun one | 20:43 |
*** rpioso has joined #openstack-ironic | 20:43 | |
mikal | So, its not terminal, but it does slow down my deply by like 30 seconds | 20:43 |
mat128 | https://github.com/openstack/ironic-python-agent/commit/cbd90c1c14e083c5cc563cfd94f55266d28f77be | 20:43 |
mikal | I'm confused about why the agent is in a container / chroot at all to be honest | 20:43 |
mikal | Given its a single purpose ram disk | 20:43 |
openstackgerrit | Marc Aubry proposed openstack/python-ironicclient: Make error work with client in python3 https://review.openstack.org/378050 | 20:46 |
JayF | mikal: the coreos ramdisk is a little special. and probably will be put out to pasture when the work is done to make dib fully supported in ci | 20:46 |
JayF | mikal: at least that's what I'll suggest :) | 20:47 |
mat128 | JayF: as long as we have the agent element in-tree :) | 20:47 |
JayF | mat128: that's part of what we're waiting for | 20:47 |
mat128 | mikal: in the meantime, I guess you could revert https://github.com/openstack/ironic-python-agent/commit/cbd90c1c14e083c5cc563cfd94f55266d28f77be | 20:48 |
mikal | Does the coreos image use containers / chroots because that's how coreos works? | 20:48 |
JayF | mikal: coreos doesn't ship with python. | 20:48 |
mikal | I don't see why dib would build a different ram disk than coreos | 20:48 |
JayF | mikal: the coreos "way" is to ship containers, it stopped being a container and started being a chroot because of technical reasons | 20:48 |
mat128 | mikal: dib could (can) build Ubuntu/Fedora/whatever ramdisks | 20:48 |
mikal | Is the dib method documented anywhere? I see docs for coreos and tinyipa, but I don't think I noticed dib docs | 20:49 |
mikal | (But I might have glossed over them) | 20:49 |
mat128 | oh yes it is | 20:49 |
* mat128 has just migrated all of the docs :) | 20:49 | |
mat128 | http://docs.openstack.org/project-install-guide/baremetal/draft/deploy-ramdisk.html | 20:49 |
mikal | mat128: oh, I've been looking at the readmes in the agent source | 20:50 |
mikal | Which only cover the other two | 20:50 |
mikal | And don't point to the web docs | 20:50 |
* mikal might tweak that today | 20:50 | |
JayF | mikal: feel free to add me as a reviewer to any docs or ipa patches you post | 20:50 |
mikal | JayF: sir yes sir | 20:50 |
*** bfournie has quit IRC | 20:51 | |
mikal | At the moment what I am actually doing is trying to work out why supermicros in a customer environment take 15 minutes to image / boot. Along the way it became obvious that the coreos IPA was a big chunk of that time. | 20:51 |
mikal | So yeah, not sure where this adventure will take me, but I'll let you know what I learn along the way. | 20:51 |
JayF | I'd strongly suggest building a dib image | 20:52 |
JayF | and seeing if those issues just go poof | 20:52 |
mikal | So far we've saved 2 minutes of that by moving to tinyipa | 20:52 |
mikal | Ok, I can try that today now that I know its a thing | 20:52 |
mat128 | mikal: how big is the image you are deploying? | 20:53 |
mikal | JayF: I have a series of extremely soporific videos you can watch if you want, but they're internal only at the moment because they identify the customer | 20:53 |
mikal | mat128: a few hundred mb IIRC? | 20:53 |
*** joprovost has quit IRC | 20:53 | |
mikal | i.e. not huge | 20:53 |
mat128 | mikal: our deployments are much faster than that, even when cold booting | 20:53 |
mat128 | mikal: on supermicro too | 20:53 |
mikal | Hmmm, interesting | 20:54 |
mat128 | mikal: there is one more thing that might slow you down | 20:54 |
mikal | These supermicros a very slow to boot / POST to be honest | 20:54 |
mat128 | yes | 20:54 |
mat128 | unavoidable except if you start deploying from a warm machine | 20:54 |
JayF | mikal: I don't really have the time to look over them, but if you have an inexplicable log snippet you can toss it my way | 20:54 |
mikal | JayF: oh, I wouldn't recommend them unless you have insomnia | 20:54 |
mat128 | mikal: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/write_image.sh#L42 | 20:55 |
mikal | JayF: I was more threatening you with them. You know, be good to your mother or I'll strap your eye lids open and show you videos of machines booting over and over. | 20:55 |
mikal | mat128: well, the image has to be synced before reboot anyway though right? | 20:56 |
mikal | mat128: so that's just forcing that earlier in the chain | 20:56 |
mat128 | mikal: bdeschenes found out that for our Windows images not specifying the mode (aka falling back to something else, cant really remember right now) was much faster | 20:56 |
mikal | mat128: oh, interesting | 20:56 |
mat128 | mikal: if your guest image is very sparse, you might be doing a lot of random io | 20:56 |
mikal | I think you also raised something else interesting there... | 20:57 |
mat128 | where as other buffering modes might make it more linear | 20:57 |
mat128 | experiment with it :) | 20:57 |
mikal | Ironic is powering off the machines when they're not in use | 20:57 |
mat128 | so far we have a patch that simply removes "-t directsync" | 20:57 |
mikal | I'd be ok with them sitting warm instead if it meant a faster boot | 20:57 |
mikal | How do I convince ironic to do that thing? | 20:57 |
mat128 | mikal: thats not a built in feature right now | 20:57 |
mikal | mat128: I shall try that too, thanks | 20:57 |
mat128 | mikal: We have plans to bring it in Ironic, but right now it requires another service to turn on machines and patches to avoid rebooting when starting the deployment | 20:58 |
mat128 | let me find that line :) | 20:59 |
*** ijw has joined #openstack-ironic | 21:00 | |
mat128 | mikal: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L462 | 21:00 |
mat128 | turns off the node when preparing deployment | 21:00 |
openstackgerrit | Merged openstack/ironic-python-agent: Add info on tinyipa to docs https://review.openstack.org/374714 | 21:00 |
*** Marx314 has joined #openstack-ironic | 21:00 | |
mat128 | mikal: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L417 | 21:00 |
*** ijw has quit IRC | 21:01 | |
*** trown is now known as trown|outtypewww | 21:01 | |
mat128 | mikal: so yeah, a few things to patch out if you want to start deploying for a machine that is already on | 21:01 |
* mikal wonders if he can trick his coworkers into having a go at doing that thing | 21:01 | |
*** ijw has joined #openstack-ironic | 21:01 | |
*** ijw has quit IRC | 21:03 | |
mat128 | Gotta go, good luck mikal :) | 21:04 |
*** ijw has joined #openstack-ironic | 21:05 | |
*** ijw has quit IRC | 21:05 | |
*** ijw has joined #openstack-ironic | 21:05 | |
openstackgerrit | Merged openstack/ironic-specs: Amend ipxe-swift-tempurls spec https://review.openstack.org/372682 | 21:05 |
jroll | mikal: I proposed that spec for juno and got... busy | 21:06 |
jroll | heh | 21:06 |
*** baoli has joined #openstack-ironic | 21:06 | |
jroll | mat128: I approved your backports, hopefully devananda or dtantsur can pick them up | 21:07 |
mat128 | yay :) | 21:07 |
mat128 | jroll: that one is also a pure backport https://review.openstack.org/#/c/377988/ | 21:07 |
patchbot | patch 377988 - ironic (stable/newton) - Fix absolute links to install-guide.rst in develop... | 21:07 |
*** e0ne has quit IRC | 21:07 | |
jroll | mat128: oh, you caught me in a lie | 21:08 |
JayF | mat128: can you put a list of those backport patches in an ehterpad or something? | 21:08 |
jroll | I thought I finished, two left to go | 21:08 |
xavierr | ty for the review rloo, I'm working in other review for inspection interface but I'll take another look on that patch asap :) | 21:08 |
jroll | JayF: they're all in the same tree | 21:08 |
mat128 | JayF: https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:stable/newton+topic:bug/1612278 | 21:08 |
JayF | mat128: I can ping tonyb about them when he gets online from australia | 21:08 |
jroll | tonyb's boss is here in channel | 21:08 |
mikal | Yes, please hassle tonyb. He loves that stuff. | 21:08 |
JayF | tonyb: https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:stable/newton+topic:bug/1612278 -- your stable approval would be appreciated for our install guide backport. Already +2'd from jroll, PTL | 21:08 |
mat128 | Hahaha | 21:08 |
JayF | mikal: was that snark? I seriously thought he enjoyed helping with that stuff... | 21:08 |
* rloo wonders why mikal can't do it for tonyb... | 21:08 | |
* JayF does | 21:08 | |
mikal | Although, I think he'll just yell at me about nova stable reviews for revenge | 21:09 |
mikal | JayF: no, he does. I'm just teasing him in absentia. | 21:09 |
JayF | Is mikal able to +2 things in any stable repo? | 21:09 |
mikal | rloo: I'm not stable core | 21:09 |
jroll | pro tip: if mikal is speaking it is snark | 21:09 |
mikal | rloo: I'm only nova-stable core | 21:09 |
rloo | mikal: but you're a boss! :) | 21:09 |
mikal | So I think that means I'm just a bystander here? | 21:09 |
mikal | rloo: I try very hard to not tell people how to vote in gerrit during meetings | 21:09 |
mikal | rloo: cause I don't want to be a bad person | 21:10 |
rloo | mikal: ha ha... | 21:10 |
*** mat128 is now known as mat128|afk | 21:11 | |
rloo | mikal: in all fairness, for those that don't know, I am just joking with you. You are trusted! | 21:11 |
JayF | jroll: you missed https://review.openstack.org/#/c/378037/ | 21:11 |
patchbot | patch 378037 - ironic (stable/newton) - Point documentation links to newton equivalents | 21:11 |
mat128|afk | alright, bye everyone | 21:11 |
jroll | JayF: no, I'm reviewing it | 21:11 |
JayF | ah, cool | 21:11 |
jroll | I said two left 3 minutes ago, dang | 21:11 |
JayF | that's at the top of the chain, right? | 21:11 |
* jroll hands JayF his old management whip | 21:11 | |
jroll | yes | 21:11 |
JayF | or am I reversed in gerrit again /o\ | 21:11 |
mat128|afk | JayF: depends what your "top" is, but yes | 21:11 |
jroll | top meaning HEAD | 21:11 |
JayF | it's the foundation for all the other commits | 21:11 |
JayF | no, it's the HEAD? | 21:12 |
mat128|afk | JayF: the other way around | 21:12 |
JayF | man, I hate gerrit ui for this | 21:12 |
mat128|afk | it's the leaf in the graph | 21:12 |
rloo | c'mon, stop interrupting jroll so he can finish the reviews. i can't wait for them to be over... | 21:12 |
mat128|afk | JayF: my personal trick is "It reads like git log --oneline" | 21:12 |
mat128|afk | hahaha | 21:12 |
jroll | it's done | 21:12 |
mat128|afk | :D | 21:12 |
JayF | mat128|afk: that's ridiculously helpful, tyvm | 21:12 |
rloo | yay. now we need one other person with powers... | 21:12 |
mat128|afk | gotta go | 21:12 |
mat128|afk | for real this time | 21:12 |
mat128|afk | JayF: np :) | 21:13 |
*** dprince has quit IRC | 21:14 | |
*** dprince has joined #openstack-ironic | 21:17 | |
*** krtaylor has quit IRC | 21:21 | |
zigo | jroll: I'm back and trying the last version of your patch. | 21:21 |
*** dprince has quit IRC | 21:25 | |
jroll | zigo: I'm just heading off, but let me know | 21:27 |
jroll | I'll check back soonish | 21:27 |
zigo | ok | 21:27 |
*** Goneri has joined #openstack-ironic | 21:28 | |
*** hogepodge has joined #openstack-ironic | 21:28 | |
*** sabeen has quit IRC | 21:28 | |
*** sabeen has joined #openstack-ironic | 21:30 | |
mikal | rloo: awww, that's the nicest thing which has been said to me today | 21:31 |
mikal | rloo: mostly its been dogs barking to be honest | 21:31 |
*** Goneri has quit IRC | 21:33 | |
rloo | mikal: they say dogs are a man's best friend :) Anyway, I bet you're a great manager. I'll take a poll when I'm at the summit... :) | 21:35 |
*** jheroux has quit IRC | 21:36 | |
betherly | zigo: thanks :) keep me posted how it goes! | 21:36 |
mikal | LOL | 21:36 |
mikal | "On a scale of hugs through stabby, how would you rate your urge to murder mikal?" | 21:37 |
zigo | jroll: betherly: Not any better, still the same issue: http://logs.openstack.org/94/378694/5/check/gate-deb-ironic-ui-dpkg-debian-jessie/569858d/console.html#_2016-09-28_21_31_39_949730 | 21:37 |
betherly | :/ | 21:37 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Update iptables rules and services IPs for multinode https://review.openstack.org/368648 | 21:38 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix setting custom IRONIC_VM_NETWORK_BRIDGE https://review.openstack.org/365116 | 21:38 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Add devstack setup_vxlan_network() https://review.openstack.org/368647 | 21:38 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Skip some steps for multinode case https://review.openstack.org/368646 | 21:38 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Update ironic node names for multinode case https://review.openstack.org/368645 | 21:38 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Update devstack provision net config for multihost https://review.openstack.org/368644 | 21:38 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Source openrc on subnode in multinode topology https://review.openstack.org/353302 | 21:38 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Skip db configuration on subnodes https://review.openstack.org/353303 | 21:38 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 21:38 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Ignore required_services for multinode topology https://review.openstack.org/352793 | 21:38 |
openstackgerrit | Stenio Araujo proposed openstack/python-oneviewclient: Adds attribute mac_type to a SPT and a validation https://review.openstack.org/377054 | 21:39 |
*** Goneri has joined #openstack-ironic | 21:41 | |
*** absubram has quit IRC | 21:42 | |
openstackgerrit | Marc Aubry proposed openstack/python-ironicclient: Make error work with client in python3 https://review.openstack.org/378050 | 21:43 |
rloo | mikal: i hadn't thought of phrasing it that way, but it'll do! | 21:44 |
rloo | jlvillal: ^^ what's that about? | 21:45 |
jlvillal | rloo: I'm just super productive ;) | 21:45 |
jlvillal | rloo: I did some changes to the multi-node job in the devstack-gate section. | 21:45 |
rloo | jlvillal: I'm impressed! | 21:45 |
jlvillal | rloo: So I had to make some changes in the Ironic code. | 21:46 |
jlvillal | rloo: s/MULTI_HOST/IRONIC_MULTI_NODE/g | 21:46 |
jlvillal | And then push the new patches :) | 21:46 |
rloo | jlvillal: ahhh. I'd like to take a look at those patches. one day... | 21:46 |
jlvillal | rloo: Basically Nova is already using MULTI_HOST and might be confusing. So I changed the devstack variable to IRONIC_MULTI_NODE | 21:46 |
rloo | jlvillal: i think you should ask nova to change theirs to NOVA_MULTI_HOST :) | 21:47 |
JayF | I think nova wins all games of review-chicken | 21:47 |
jlvillal | YES!!!! :) | 21:47 |
*** Goneri has quit IRC | 21:53 | |
*** baoli has quit IRC | 21:53 | |
*** rajinir has quit IRC | 21:55 | |
*** baoli_ has joined #openstack-ironic | 21:58 | |
mrda | Morning Ironic | 22:00 |
*** baoli_ has quit IRC | 22:04 | |
*** krtaylor has joined #openstack-ironic | 22:10 | |
*** mkam has quit IRC | 22:11 | |
*** Goneri has joined #openstack-ironic | 22:11 | |
*** praneshp has quit IRC | 22:11 | |
tonyb | holy reviews batman! that's quite a few reviews to approve. | 22:11 |
* tonyb will check with docs to make sure they wont hate me if we land those things. | 22:13 | |
JayF | tonyb: I'm the docs liason for Ironic, and I approved this message(tm) | 22:13 |
*** sabeen1 has joined #openstack-ironic | 22:13 | |
*** baoli has joined #openstack-ironic | 22:13 | |
JayF | tonyb: we didn't originally plan to backport the install guide, we intentionally cut newton beforehand, but many folks asked us to backport it because mat128|afk did such a good job | 22:14 |
jroll | betherly: let's just go ahead and release as-is and we can backport the fix when we come up with it | 22:14 |
jroll | betherly: let me know when you un-wip and I'll handle the rest :) | 22:15 |
tonyb | JayF: okay that's good background. I'm still a little grey on how this all works. I'll either approve them or drop a comment explaining why today. | 22:15 |
JayF | tonyb: pretty much ironic docs are in-tree, and get built, then published alongside the official stuff. Frankly, I can't imagine a reason the docs team would generally care :) | 22:15 |
*** sabeen has quit IRC | 22:15 | |
JayF | tonyb: but if you're uncomfortable we can get someone on ironic stable maint to push the button | 22:16 |
*** baoli has quit IRC | 22:16 | |
tonyb | JayF: I'm not super-uncomfortable but I just want to check. | 22:16 |
*** cdearborn has quit IRC | 22:18 | |
rloo | tonyb: is Andreas Jaeger a docs or infra person? He wanted us to backport the install guide: http://lists.openstack.org/pipermail/openstack-dev/2016-September/104607.html | 22:19 |
rloo | tonyb: he even asked nicely :) | 22:20 |
tonyb | rloo: He's a bit of both :) | 22:20 |
*** bdeschenes has quit IRC | 22:20 | |
*** praneshp has joined #openstack-ironic | 22:28 | |
betherly | jroll: will that not mess up the gates? | 22:29 |
*** vishwanathj has quit IRC | 22:29 | |
*** bdeschenes has joined #openstack-ironic | 22:31 | |
betherly | jroll: dangggggg https://review.openstack.org/#/c/378668/ still hasn't merged into the repo! I thought that would be long done by now. Im heading to bed now but will check in the morning and sort the release patch out | 22:31 |
patchbot | patch 378668 - ironic-ui - Added release notes for 2.1.0 release | 22:31 |
jroll | betherly: right on, and no, zigo's bug is in the previous release | 22:36 |
jroll | so shouldn't bother any CI or anything, except for his, which is already bothered | 22:38 |
betherly | jroll: okydokes - any chance you could +2 workflow again that patch I mentioned above to see if that helps push it through? | 22:40 |
*** joprovost has joined #openstack-ironic | 22:41 | |
jroll | betherly: oof, yeah, it didn't get in the pipeline | 22:41 |
jroll | done | 22:41 |
betherly | thanks so much jroll | 22:42 |
betherly | I'll look first thing and sort the release | 22:42 |
jroll | np, thank you! | 22:42 |
betherly | Have a good evening! | 22:42 |
jroll | it's you that's here way too late, not me | 22:42 |
jroll | you too :) | 22:42 |
*** awiddersheim has joined #openstack-ironic | 22:42 | |
*** joprovost has quit IRC | 22:46 | |
*** ijw has quit IRC | 22:46 | |
openstackgerrit | Merged openstack/ironic-ui: Added release notes for 2.1.0 release https://review.openstack.org/378668 | 22:54 |
*** sdake has quit IRC | 22:56 | |
*** ijw has joined #openstack-ironic | 22:58 | |
*** rpioso has quit IRC | 22:59 | |
*** ijw has quit IRC | 23:03 | |
*** ijw has joined #openstack-ironic | 23:19 | |
*** Goneri has quit IRC | 23:22 | |
*** ijw has quit IRC | 23:24 | |
*** ijw has joined #openstack-ironic | 23:29 | |
openstackgerrit | Bruno Cornec proposed openstack/ironic-specs: Propose Redfish support into Ironic https://review.openstack.org/184653 | 23:31 |
*** sdake has joined #openstack-ironic | 23:32 | |
*** ijw has quit IRC | 23:34 | |
*** sdake_ has joined #openstack-ironic | 23:35 | |
*** sdake has quit IRC | 23:38 | |
xavierr | morning mrda | 23:39 |
mrda | hey xavierr | 23:45 |
*** ijw has joined #openstack-ironic | 23:47 | |
openstackgerrit | Xavier proposed openstack/ironic: Add inspection feature for the OneView drivers https://review.openstack.org/367065 | 23:50 |
*** rama_y has quit IRC | 23:50 | |
openstackgerrit | Bruno Cornec proposed openstack/ironic-specs: Propose Redfish support into Ironic https://review.openstack.org/184653 | 23:51 |
*** ijw has quit IRC | 23:53 | |
xavierr | have a good evening all | 23:54 |
*** xavierr is now known as xavierr|afk | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!