Wednesday, 2016-09-28

*** davidlenwell has quit IRC00:11
*** ijw has quit IRC00:14
*** ijw has joined #openstack-ironic00:19
*** davidlenwell has joined #openstack-ironic00:25
openstackgerritYibo Cai proposed openstack/ironic: Support multi arch deployment  https://review.openstack.org/32263200:25
*** ijw has quit IRC00:45
*** hoangcx has joined #openstack-ironic00:50
*** Goneri has quit IRC01:04
*** ijw has joined #openstack-ironic01:05
*** ijw_ has joined #openstack-ironic01:10
*** ijw has quit IRC01:11
*** ijw_ has quit IRC01:15
*** jvgrant has quit IRC01:22
*** jvgrant has joined #openstack-ironic01:22
openstackgerritAnh Tran proposed openstack/ironic-python-agent: Fixed typo in doc file  https://review.openstack.org/37813701:45
openstackgerritRuby Loo proposed openstack/ironic: Update multitenancy documentation  https://review.openstack.org/37813801:46
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off  https://review.openstack.org/21673001:47
openstackgerritTang Chen proposed openstack/python-ironicclient: Add missing unit tests for provision state commands  https://review.openstack.org/37760701:49
openstackgerritTang Chen proposed openstack/python-ironicclient: Trivial: Rename and reorder deploy baremetal test  https://review.openstack.org/37814001:49
*** thrash is now known as thrash|g0ne01:54
openstackgerritXu Ao proposed openstack/ironic: Fix the mistakes in Installation Guide  https://review.openstack.org/35322401:58
openstackgerritXu Ao proposed openstack/ironic: Fix the mistakes in Installation Guide  https://review.openstack.org/35322402:00
*** phuongnh has joined #openstack-ironic02:00
*** baoli has quit IRC02:05
*** baoli has joined #openstack-ironic02:11
*** ppiela has joined #openstack-ironic02:17
*** ijw has joined #openstack-ironic02:19
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off  https://review.openstack.org/21673802:21
*** ppiela has quit IRC02:22
*** ijw has quit IRC02:23
*** awiddersheim has quit IRC02:23
*** ijw has joined #openstack-ironic02:24
*** awiddersheim has joined #openstack-ironic02:28
*** ijw has quit IRC02:29
*** ijw has joined #openstack-ironic02:31
*** ijw has quit IRC02:44
*** baoli has quit IRC02:52
*** xuao has joined #openstack-ironic02:52
*** Sukhdev has joined #openstack-ironic02:59
*** tuanla has joined #openstack-ironic03:00
*** Sukhdev has quit IRC03:01
openstackgerritHa Van Tu proposed openstack/ironic: Fix typos in docstring/comments  https://review.openstack.org/37726103:01
*** david-lyle has quit IRC03:04
*** sdake_ has quit IRC03:17
*** ppiela has joined #openstack-ironic03:18
*** aswadr_ has joined #openstack-ironic03:21
*** ppiela has quit IRC03:22
*** vikrant has joined #openstack-ironic03:26
*** rajinir has quit IRC03:35
*** sdake has joined #openstack-ironic03:39
*** ppiela has joined #openstack-ironic03:43
*** sdake_ has joined #openstack-ironic03:50
*** Sukhdev has joined #openstack-ironic03:50
*** sdake has quit IRC03:51
*** sai has quit IRC04:22
*** sai has joined #openstack-ironic04:23
*** amotoki has joined #openstack-ironic04:28
*** tangchen_ has quit IRC04:29
*** tangchen_ has joined #openstack-ironic04:29
*** ppiela has quit IRC04:36
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off  https://review.openstack.org/21673004:51
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off  https://review.openstack.org/21674305:04
*** syed_ has joined #openstack-ironic05:05
*** bks has joined #openstack-ironic05:07
*** Sukhdev has quit IRC05:08
*** links has joined #openstack-ironic05:09
*** links has quit IRC05:15
*** links has joined #openstack-ironic05:17
*** ppiela has joined #openstack-ironic05:19
*** ppiela has quit IRC05:24
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off  https://review.openstack.org/21673805:27
*** sdake_ has quit IRC05:30
*** jaosorior has joined #openstack-ironic05:46
openstackgerritTuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work  https://review.openstack.org/37632205:54
*** vmud213 has joined #openstack-ironic05:55
openstackgerritgengchc2 proposed openstack/bifrost: Update flake8 ignore list  https://review.openstack.org/37825705:57
mikalAnyone home?06:08
*** mjura has joined #openstack-ironic06:09
*** rcernin has joined #openstack-ironic06:14
*** mjura has quit IRC06:16
*** mjura has joined #openstack-ironic06:16
*** rbartal has joined #openstack-ironic06:23
*** ppiela has joined #openstack-ironic06:25
*** ppiela has quit IRC06:30
*** crinkle_ is now known as crinkle06:33
*** pcaruana has joined #openstack-ironic06:37
*** jrist has joined #openstack-ironic06:54
*** marlinc has joined #openstack-ironic06:59
*** tuanla has quit IRC07:01
openstackgerritDao Cong Tien proposed openstack/ironic: Fix inexact config option name in multitenancy.rst  https://review.openstack.org/37829107:01
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url  https://review.openstack.org/35228907:02
*** jrist has quit IRC07:04
*** ralonsoh has joined #openstack-ironic07:04
*** ralonsoh_ has joined #openstack-ironic07:04
*** ralonsoh has quit IRC07:04
*** ralonsoh_ has quit IRC07:04
*** vmud213 has quit IRC07:04
*** ralonsoh has joined #openstack-ironic07:04
*** ralonsoh_ has joined #openstack-ironic07:04
*** ralonsoh has quit IRC07:04
*** ralonsoh_ has quit IRC07:04
*** ralonsoh has joined #openstack-ironic07:05
*** vmud213 has joined #openstack-ironic07:07
*** syed_ has quit IRC07:14
*** links has quit IRC07:16
openstackgerritzhangyanxian proposed openstack/bifrost: Fix typo in bifrost-role-auth-support-ea6b5571cd339aa2.yaml  https://review.openstack.org/37829607:20
openstackgerritzhangyanxian proposed openstack/bifrost: Fix typo in bifrost-role-auth-support-ea6b5571cd339aa2.yaml  https://review.openstack.org/37829607:21
*** marlinc has quit IRC07:21
*** marlinc has joined #openstack-ironic07:27
*** jpich has joined #openstack-ironic07:35
openstackgerritZhenguo Niu proposed openstack/ironic-python-agent: Parallel erase disk devices  https://review.openstack.org/34672307:42
*** links has joined #openstack-ironic07:45
pesterhi 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 strigazi07:49
pesteroh "ERROR nova.compute.manager PortBindingFailed: Binding failed for port" why nova compute tries to bind port ?... :(07:56
*** marlinc has quit IRC07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:01
*** ijw has joined #openstack-ironic08:03
pas-hamorning Ironic08:07
pas-hapester: that's why https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L1221-L122708:07
pas-hait seems currently for that to work in flat network case, you have to run neutron-agent on the node where ironic-conductor is08:09
*** ijw has quit IRC08:11
openstackgerritZhenguo Niu proposed openstack/ironic-specs: Check duplicate nodes enrollment/update  https://review.openstack.org/29635708:11
*** links has quit IRC08:21
*** Romanenko_K has joined #openstack-ironic08:24
*** marlinc has joined #openstack-ironic08:24
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url  https://review.openstack.org/35228908:26
openstackgerritZhenguo Niu proposed openstack/ironic-specs: Dynamically allocate console ports  https://review.openstack.org/24987608:36
*** links has joined #openstack-ironic08:37
*** sdake has joined #openstack-ironic08:41
*** derekh has joined #openstack-ironic08:41
*** sdake has quit IRC08:42
*** priteau has joined #openstack-ironic08:48
*** openstackgerrit has quit IRC08:48
*** openstackgerrit has joined #openstack-ironic08:49
*** moshele has joined #openstack-ironic08:50
*** mgould|afk is now known as mgould08:57
mgouldmorning Ironic08:57
tangchen_morning mgould09: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
patchbotpatch 377607 - python-ironicclient - Add missing unit tests for provision state commands09:04
*** sambetts_ is now known as sambetts09:12
sambettsMorning all09:12
*** e0ne has joined #openstack-ironic09:14
*** priteau has quit IRC09:15
*** priteau has joined #openstack-ironic09:17
openstackgerritNam Nguyen Hoai proposed openstack/ironic-lib: Fix typo in docstring  https://review.openstack.org/37847009:35
openstackgerritJulia Kreger proposed openstack/bifrost: WIP: Update Ansible version to stable-2.1  https://review.openstack.org/37804109:36
TheJuliaGood morning09:38
*** electrofelix has joined #openstack-ironic09:39
*** tuanla has joined #openstack-ironic09:40
openstackgerritTang Chen proposed openstack/python-ironicclient: Make console commands support multiple node operation  https://review.openstack.org/37847709:41
*** aswadr_ has quit IRC09:42
*** electrofelix has quit IRC09:45
*** electrofelix has joined #openstack-ironic09:46
sambettsMorning TheJulia09:46
openstackgerritgengchc2 proposed openstack/ironic: Fix a typo in policy.py,etc  https://review.openstack.org/37848709:53
*** ralonsoh_ has joined #openstack-ironic09:58
*** ralonsoh_ has quit IRC09:58
*** ralonsoh has quit IRC09:59
*** ralonsoh has joined #openstack-ironic09:59
*** xuao has quit IRC10:12
openstackgerrityolanda.robla proposed openstack/bifrost: Add read and execute perms to httpboot and tftpboot folder  https://review.openstack.org/37401510:14
*** wajdi has quit IRC10:19
mgouldmorning TheJulia10:22
mgouldtangchen_: thanks!10:22
mgouldtangchen_: both look good to me, +1x210:28
*** hoangcx has quit IRC10:29
openstackgerritJulia Kreger proposed openstack/bifrost: Update Ansible version to stable-2.1  https://review.openstack.org/37804110:32
openstackgerritBharath kumar proposed openstack/ironic: Adding a ipxe_boot_sleep parameter  https://review.openstack.org/37319410:34
openstackgerritVasyl Saienko proposed openstack/ironic: Update configure_tenant_networks  https://review.openstack.org/35646010:35
*** tuanla has quit IRC10:39
*** awiddersheim has quit IRC10:39
*** awiddersheim has joined #openstack-ironic10:44
*** links has quit IRC10:53
*** priteau has quit IRC10:53
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Reformat docstring and remove unused argument  https://review.openstack.org/37852610:54
*** phuongnh has quit IRC10:58
*** dprince has joined #openstack-ironic11:02
*** links has joined #openstack-ironic11:02
*** moshele has quit IRC11:09
pas-hadtantsur: about bug https://bugs.launchpad.net/ironic/+bug/1512898 - could you please check my last comment?11:10
openstackLaunchpad bug 1512898 in Ironic "Ironic grenade job should use IPA" [High,Triaged] - Assigned to Pavlo Shchelokovskyy (pshchelo)11:10
*** moshele has joined #openstack-ironic11:10
*** hshiina has joined #openstack-ironic11:13
*** awiddersheim has quit IRC11:15
*** appu103_ has joined #openstack-ironic11:24
*** thrash|g0ne is now known as thrash11:25
mat128morning sambetts, TheJulia, mgould11:25
mgouldmorning mat128 sambetts11:26
*** lucas-afk is now known as lucasagomes11:29
lucasagomesmorning all11:29
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for portgroups in ironicclient  https://review.openstack.org/29772711:30
*** ccamacho is now known as ccamacho|lunch11:36
*** bfournie has quit IRC11:36
mgouldlucasagomes: morning!11:39
lucasagomeso/11:39
*** thrash has quit IRC11:40
TheJuliao/11:47
*** thrash has joined #openstack-ironic11:48
*** thrash has quit IRC11:48
*** thrash has joined #openstack-ironic11:48
*** jrist has joined #openstack-ironic11:49
mikalIs tinyipa considered production ready?11:49
mikalI'm liking it in my testing, but the docs still describe it as experimental11:49
TheJuliamikal: 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
lucasagomesTheJulia, ++11:52
*** derekh has quit IRC11:52
sambettsIt is no longer experimental though, so the docs probably need updating now its throughly tested in CI11:52
TheJuliamikal: 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
TheJuliamikal: mainly in the sense of using VMs to test, where nested virt can get.... rather... slow.11:54
mikalSo, tinyipa is working fine on these here supermicros. I was going to test HP tomorrow.11:54
mikalBut yeah ok, it sounds like we might need to consider different IPA images for different types of hardware11:54
mikalWhich was something I was braced for, but hadn't actually needed to do yet11:55
sambettsI use it successfully for Cisco HW too, its all down to the drivers included in the base OS11:55
sambettsin this case TinyCoreLinux11:55
TheJuliamikal: You'll likely want to use Fedora for the HP gear11:55
mikalTheJulia: coreos has been working so far on the HP Cloudlines at least11:56
TheJuliamikal: Also, there is a special driver you'll need to install if you have a dynamic smart array.11:56
mikalBut yeah, I can test those tomorrow11:56
mikalI am close to no longer caring for today11:56
TheJuliamikal: okay, awesome!11:56
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Fix import of ironicclient and reformat docstring  https://review.openstack.org/37852611:57
*** baoli has joined #openstack-ironic11:58
*** lucasagomes is now known as lucas-bbl12:01
xavierrgood morning Ironic12:02
*** sdake has joined #openstack-ironic12:02
TheJuliagood morning xavierr12:05
mat128mikal: it should be possible to build one image that boots on almost all machines12:08
mat128mikal: the tiny core we build has a new enough kernel to support most of the stuff out there12:08
TheJuliamat128: does tinyipa build in any hardware managers?12:09
sambettsTheJulia: only whats included in the IPA repo12:10
sambettsTheJulia: but thats just because the build process doesn't have a way to inject other ones12:11
sambettsyet12:11
sambetts;)12:11
sambettscould be a good RFE12:11
sambettstbh I've always wondered about how your supposed to do that with the coreos image too12:12
*** mtanino has joined #openstack-ironic12:12
mgouldmorning xavierr12:12
TheJuliasambetts: very carefully? :)12:13
TheJuliasambetts: that or rip it apart, chroot into it, and inject it12:13
*** wajdi has joined #openstack-ironic12:15
*** baoli has quit IRC12:16
*** wajdi_ has joined #openstack-ironic12:16
mat128TheJulia: same as CoreOS, we build from soruce12:17
mat128gotta go12:17
*** mat128 is now known as mat128|afk12:17
TheJuliawell, out of tree hardware managers like the one provided by proliantutils12:18
TheJuliaBut even then, like the dib element installs some other stuff that is required to control the array12:18
TheJuliaStepping away for a little bit myself12:19
sambettsso we need a way to specify addition packages to install during the build process12:19
sambettsmakes sense to me12:19
sambettscurrently all the build process just uses requirements.txt from IPAs repo12:20
*** wajdi has quit IRC12:20
sambettsso I guess you could append proliantutils to the end of that, but I don't have much experience with custom IPAs12:20
TheJuliayeah, binary deps though :)12:20
TheJuliaThat is a whole larger ball of wax that12:21
sambettsyeah :/12:21
sambettsthere are examples in the tinyipa build process for adding binarys from source that aren't in the tinyipa repos12:22
sambettsso perhaps we can make some generic hook for doing that wizardry12:22
*** vikrant has quit IRC12:22
TheJuliaI 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 time12:23
sambettsYeah I looked into turning my tinyipa scripts into DIB elements, but didn't get very far12:23
TheJuliaanyway, I need to step away12:24
sambettso/12:25
mat128|afksambetts: there was some work done on bringing tinycore to dib12:25
mat128|afkcant find it atm12:25
mat128|afkhave to go12:25
sambettso/12:25
pas-habtw, I have a patch to ipa docs with general info on tinyipa Ic6cfe64255aba0460cc960a98b18580b7798b204 (gerrit is down for me :( )12:27
pas-hahttps://review.openstack.org/#/c/374714/12:28
patchbotpatch 374714 - ironic-python-agent - Add info on tinyipa to docs12:28
*** bfournie has joined #openstack-ironic12:28
sambettspas-ha: ++ nice! I'll review asap12:29
pas-hathanks12:36
*** bks has quit IRC12:38
*** vmud213 has quit IRC12:40
*** e0ne has quit IRC12:40
*** yuriyz has quit IRC12:41
*** ppiela has joined #openstack-ironic12:41
*** yuriyz has joined #openstack-ironic12:42
*** Romanenko_K has quit IRC12:42
*** ipukha has quit IRC12:42
*** aarefiev has quit IRC12:42
*** vnogin has quit IRC12:42
*** Romanenko_K has joined #openstack-ironic12:43
*** e0ne has joined #openstack-ironic12:43
*** ipukha has joined #openstack-ironic12:43
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to keystone session for generating temp_url  https://review.openstack.org/35228912:44
*** vnogin has joined #openstack-ironic12:44
*** aarefiev has joined #openstack-ironic12:44
*** rbrud has joined #openstack-ironic12:45
openstackgerrityolanda.robla proposed openstack/bifrost: Add read and execute perms to httpboot and tftpboot folder  https://review.openstack.org/37401512:45
*** trown|outtypewww is now known as trown12:48
*** Goneri has joined #openstack-ironic12:57
*** david-lyle has joined #openstack-ironic12:57
*** rloo has joined #openstack-ironic12:57
*** vishwanathj has joined #openstack-ironic12:58
*** baoli has joined #openstack-ironic12:59
*** baoli_ has joined #openstack-ironic13:02
betherlyjroll: we are on track to release ironic ui today!13:02
jrollbetherly: awesome \o/13:03
jrollmorning everyone13:03
jrollmikal: 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 lucasagomes13:05
jrollsambetts: 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 lines13:06
*** baoli has quit IRC13:06
sambettsjroll: ah cool, yeah I think you could achieve the same thing if you added custom lines to the IPA requirements.txt for tinyipa13:06
sambettsjroll: but thats a little hacky ;)13:07
jrollsambetts: yeah, that's another route, it just means carrying patches13:07
sambettsyeah13:07
*** ccamacho|lunch is now known as ccamacho13:08
sambettsI wonder about adding a ADDITIONAL_REQUIREMENTS_FILES= env variable for the tinyipa build that you could point at or something13:09
sambetts:/13:09
*** mtanino has quit IRC13:09
*** hshiina has quit IRC13:09
sambettsbut as TheJulia said, that doesn't help the binary deps you might have13:10
jrollyeah13:12
openstackgerritMerged openstack/ironic-ui: Added support for editing Ironic nodes  https://review.openstack.org/37605213:12
jrollsambetts: bindep might be an option there13:12
*** appu103_ has quit IRC13:18
*** aswadr_ has joined #openstack-ironic13:18
openstackgerritAnton Arefiev proposed openstack/ironic-specs: Torrent-based provisioning  https://review.openstack.org/31109113:20
openstackgerritYushiro FURUKAWA proposed openstack/ironic-lib: Add prefix "$" for command examples  https://review.openstack.org/37861113:20
mariojvmorning ironic o/13:20
jrollo/13:20
jroll\o13:20
jroll\o/13:20
*** jaosorior has quit IRC13:20
*** jaosorior has joined #openstack-ironic13:21
*** wajdi_ has quit IRC13:24
*** mat128|afk is now known as mat12813:27
sambettsjroll: 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
mat128jroll: 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/161227813:28
xavierrmorning jroll13:28
mat128sambetts: another option is making your own python package with a dependency on ipa13:28
jrollmat128: not yet but it is on my list for today13:28
mat128sambetts: IPAs entrypoint will be installed in the virtualenv and everything will just work (tm), but it is more work13:28
*** cdearborn has joined #openstack-ironic13:30
sambettshmm 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
mat128sambetts: interesting..13:31
mat128sambetts: I think wrapping it with more pip installs is probably the best13:31
mat128sambetts: maybe the user could provide a script with "final steps"13:32
mat128deployers could patch that in themselves pretty easily, but offering it to them is probably easier13:32
sambetts++ some helper scripts for building TinyCore packages etc would help too13:33
pesterhi 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
sambettspester: you meed to move the node to manageable, then set state provide13:36
mat128pester: http://docs.openstack.org/developer/ironic/_images/states.svg13:36
mat128^this13:36
pestermat128: thx!13:37
*** links has quit IRC13:38
*** athomas has joined #openstack-ironic13:38
*** moshele has quit IRC13:39
*** rbartal has quit IRC13:39
pesteryet, node go again to - "provision_state        | clean failed" I use fake_pxe driver, I don`t need cleaning...13:42
sambettspester: You can disable cleaning via the ironic.conf file13:42
pesterautomated_clean ?13:43
sambettsyup13:43
openstackgerritBob Fournier proposed openstack/ironic-inspector-specs: Display LLDP information via new commands  https://review.openstack.org/37438113:46
*** vnogin has quit IRC13:47
pesterlooks like it worked13:48
*** bdeschenes has joined #openstack-ironic13:48
*** narasimha_SV has joined #openstack-ironic13:49
narasimha_SVhttp://paste.openstack.org/show/583301/ this is file for the server which it is getting generated for the server I am doing pxe booting13:50
narasimha_SVhttp://paste.openstack.org/show/583302/ contents inside the id shown in above link13:52
narasimha_SVam I missing anything here13:52
narasimha_SVwhich is making my pxe boot stop after ramdisk installation13:52
narasimha_SVhttp://paste.openstack.org/show/583303/ console output where it is getting stopped13:53
*** dprince has quit IRC13:53
*** dprince has joined #openstack-ironic13:54
jrollsambetts: hey, is UCS hw inspection or OOB RAID config done?13:57
* jroll going through rfe-approved looking for finished things13:57
sambettsjroll: no, we have inspection with inspector but not OOB yet13:58
jrollsambetts: cool thanks13:58
openstackgerritRuby Loo proposed openstack/ironic: Change links to point to new install guide  https://review.openstack.org/37864013:59
rloomat128: ^^ we missed those13:59
*** mtanino has joined #openstack-ironic13:59
openstackgerritYushiro FURUKAWA proposed openstack/ironic-lib: Use 'code-block' for pieces of code  https://review.openstack.org/37864314:00
*** anush has joined #openstack-ironic14:01
*** hoangcx has joined #openstack-ironic14:01
*** mtanino has quit IRC14:04
*** joprovost has joined #openstack-ironic14:07
openstackgerritKyrylo Romanenko proposed openstack/ironic: Add headers and extra_headers to arguments in tempest plugin  https://review.openstack.org/37557014:08
xavierrI 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
patchbotpatch 366150 - ironic - Fixes parameters validation in SSH power manager.14:09
xavierrrloo: any opinion on that? ^^^14:10
xavierr:)14:10
*** anush has quit IRC14:10
*** narasimha_SV has quit IRC14:10
openstackgerritVasyl Saienko proposed openstack/ironic: Testing broken grenade job  https://review.openstack.org/37765314:11
rlooxavierr: dunno, are we not making changes to unsupported drivers? i suppose not? guess we should decide/make it explicit...14:11
rloojroll: ^^ do we stop making changes to unsupported/deprecated drivers? (I guess the question is if they are unsupported or not)14:11
openstackgerritJim Rollenhagen proposed openstack/ironic-specs: Mark specs implemented in Newton  https://review.openstack.org/37865714:11
jrolleasy peasy maybe review ^14:11
xavierrrloo: I agree with you14:12
jrollrloo: I'm not opposed to bug fixes, features on the other hand I'd rather not bother with14:12
bfourniemorning 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
bfourniein 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-ironic14:12
rloojroll: ok. should we send out email or indicate that somewhere for all to know?14:13
jrollrloo: probably :)14:13
rloothx jroll, for volunteering14:13
jroll:D14:13
* jroll adds to todo list for this week14:13
xavierrhahaha14:14
openstackgerritZhenguo Niu proposed openstack/ironic: Move flavor create under 'VIRT_DRIVER == ironic'  https://review.openstack.org/37866114:14
*** mkam has joined #openstack-ironic14:17
*** anush has joined #openstack-ironic14:17
xavierrrloo, jroll: that email that you are talking about will be send to openstack mail list or ironic has one mail list?14:17
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Use correct option value for standalone install  https://review.openstack.org/37866414:18
jrollxavierr: we use the openstack-dev mailing list with the tag [ironic]14:18
*** mkam has quit IRC14:18
sambettsbfournie: You might have to do ironic port-show with the right version of the Ironic API specified otherwise Ironic hides the field14:19
*** mkam has joined #openstack-ironic14:19
openstackgerritMichael Turek proposed openstack/ironic: Enable PXE for systems using petitboot  https://review.openstack.org/18598714:19
* jroll echo 'OS_IRONIC_API_VERSION=latest' >> .zshrc14:19
xavierrjroll: thanks, I'll subscribe to that mailing list :)14:19
jrollxavierr: I'll also be writing it down in our docs, fwiw :)14:20
bfourniesambetts: I see the field, just no value14:20
bfournie[stack@host01 etc]$ ironic port-show 62096bb4-6f3d-4359-8bdb-632bb68c88ae14: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
jrollbfournie: please use paste.openstack.org in the future :)14:20
bfourniejroll: oops sorry, will do14:20
xavierrbradjones: paste.openstack.org +1 :)14:21
jrollbfournie: 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-632bb68c88ae14:22
xavierrbradjones: sorry... it was bfournie :)14:22
jrollbfournie: for reference, http://docs.openstack.org/developer/ironic/dev/webapi-version-history.html14:22
jrollanything 1.19 or greater should work14:22
*** wajdi has joined #openstack-ironic14:22
*** hamzy has quit IRC14:23
bfourniejroll: ok, thanks, still see an empty list - http://paste.openstack.org/show/583311/  - although it seems to be recognized now14:25
sambettsbfournie: 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-ironic14:27
bfourniesambetts: 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 IRC14:28
sambettsbfournie: no, the config files are only re-read on reboot of the service14:28
*** mkam has joined #openstack-ironic14:29
openstackgerritBeth Elwell proposed openstack/ironic-ui: Added release notes for 2.1.0 release  https://review.openstack.org/37866814:29
sambettsbfournie: some OpenStack services will re-read config files on SIGHUP, but none I know of will automatically detect config changes14:29
bfourniesambetts: ok will do that, sorry for the noise. Can I just HUP ironic-inspector?14:30
sambettsbfournie: I don't think we've implmented it yet :(14:30
*** baoli_ has quit IRC14:31
bfourniesambetts: ok14:31
*** baoli has joined #openstack-ironic14:31
pas-habfournie: AFAIK most services handle SIGHUP only for logging configuration reload (e.g. to enable debug w/o restarting service)14:32
bfourniepas-ha: ah, OK, thanks14:33
rloojroll: 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.html14:33
*** lennyb has quit IRC14:34
jrollrloo: hrm, that is odd, looks like they split API config and the rest14:36
rloojroll: 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
jrollrloo: yeah, something's odd, JayF would you have time to look at that today?14:38
rloojroll: in the/some past release, i looked/modified the config docs, but i didn't think to do it this time.14:38
jrollJayF: or like, this week I guess14:38
*** AlexeyAbashkin has joined #openstack-ironic14:38
jrollrloo: so I think oslo config generator will generate that stuff now14:38
jrollbut am not 100% sure14:38
*** baoli has quit IRC14:38
narasimha_SVxavierr:   http://paste.openstack.org/show/583301/  is this file generated for the server is correctly generated or not14:38
rloojroll: 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 all14:39
jrollrloo: hm, idk, hoping jay can bug the docs team about it14:39
rloojroll: me too (cuz I ain't volunteerin')14:40
jrollheh14:40
xavierrhey narasimha_SV14:41
xavierrnarasimha_SV: which file is that?14:41
*** hshiina has joined #openstack-ironic14:42
*** aNuposic has joined #openstack-ironic14:42
*** rpioso has joined #openstack-ironic14:43
rloohi 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
patchbotpatch 372682 - ironic-specs - Amend ipxe-swift-tempurls spec14:48
openstackgerritVasyl Saienko proposed openstack/ironic: Testing broken grenade job  https://review.openstack.org/37765314:48
lucasagomesrloo, hi there, totally! Added here I will look at it today14:48
rloolucasagomes: 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
openstackgerritMerged openstack/ironic-specs: Mark specs implemented in Newton  https://review.openstack.org/37865714:49
*** vsaienk0 has joined #openstack-ironic14:49
lucasagomes6.2.0 is the lastest release14:50
rloolucasagomes: right, just can't remember when the actual fix went in. aug 18 it sez.14:50
vsaienk0good morning Ironic'ers, lucasagomes, sambetts, dtantsur, vdrok, jroll, aarefiev and all!14:50
openstackgerritMerged openstack/ironic: Use correct option value for standalone install  https://review.openstack.org/37866414:51
rloolucasagomes: in 6.2, i looked at release notes :)14:51
vsaienk0I'm wondered if anyone else looking into grenade job failure or have any thoughts?14:51
lucasagomesvsaienk0, i there14:51
lucasagomesrloo, cool!14:52
vsaienk0lucasagomes: cool, I'm thinking that it is due to slow neutron14:52
rloovsaienk0: not me. I was hoping/assuming you and vdrok were...14:52
jrollvsaienk0: I've been trying but haven't had much time :(14:52
rloovsaienk0: is it timing out? (shows how much i know of the problem. nothin')14:53
jrollrloo: instances built before upgrade not pinging after upgrade14:53
xavierrjroll: 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
rloojroll: oh. i guess no one has been able to replicate it in their own dev env?14:54
vsaienk0have 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 address14:54
jrollxavierr: you need to fix it in master, and then backport it: http://docs.openstack.org/project-team-guide/stable-branches.html14:54
vsaienk0and VM try only once to get PXE, that it just stuck14:55
narasimha_SVxavierr: that is the file genereated in side pxelinux.cfg folder for the server which I am doing pxe boot14:55
rloovsaienk0: is there any way to put a wait before that? to test your theory of neutron being slow?14:56
rloovsaienk0: the grenade test fails all the time, regardless of whether it is osic cloud or not?14:56
*** hamzy has joined #openstack-ironic14:57
xavierrjroll: thanks. and in launchpad I register the bug only for master branch or should I tell there that it will be backported?15:00
xavierrnarasimha_SV: I'll check mine15:00
*** syed_ has joined #openstack-ironic15:00
vsaienk0rloo: all the time, I'm trying to increase timeout here: https://review.openstack.org/#/c/377653/15:01
patchbotpatch 377653 - ironic - Testing broken grenade job15:01
jrollxavierr: master is fine, you can register the branches if you like15:02
*** mjura has quit IRC15:03
*** lennyb has joined #openstack-ironic15:03
rloovsaienk0: 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_SVxavierr: the uuid showing in that file is the folder id created inside pxelinux.cfg folder15:04
*** rcernin has quit IRC15:04
vsaienk0rloo: 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 know15:05
vdrokRloo ipa-wholedisk should be fixed after ironic lib release bumped in constraints Iirc15:09
xavierrnarasimha_SV: I dont have that file because I'm using iPXE15:09
vdrokI'm on sick leave today, so sorry, no progress on anything :(15:09
xavierrhey vdrok o/15:09
rloovdrok: no worries. get better!!15:10
rloovsaienk0: oh, right, I recall the frenzy of activity about that :)15:10
rloovsaienk0: wrt your patch, are you incrementing the value each time, and nothing works yet?15:11
JayFjroll: rloo: yeah; it's kinda ugly. Do we have an example of a "good" version of this from another project?15:15
rloovsaienk0: my only contribution is to add to the etherpad, the link to your patch and that you are looking at it15:15
JayFjroll: 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 sick15:15
rlooJayF: glad you're better!15:15
JayFSo it's just weird that API is split out15:15
rlooJayF: if you look at the 'mitaka' version it is a bit different15:16
mat128JayF: welcome back!15:16
JayFthe new/updated/deprecation options for mitaka looks like it needs to be migrated to a newton section?15:16
rlooJayF: I haven't looked at any other project's configs for newton.15:16
*** amotoki has quit IRC15:16
rlooJayF: 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
JayFyeah; I'll drop an email to someone I know who works on this15:17
JayFand try to determine wtf is the right thing to do15:17
*** absubram has joined #openstack-ironic15:18
mat128rloo: good catch https://review.openstack.org/#/c/378640/15:18
patchbotpatch 378640 - ironic - Change links to point to new install guide15:18
xavierrnarasimha_SV: tell me the commands you are using to register the images in glance. look in your history15:18
JayFhey mat128, nice work on the install guide, I looked at a rendered version live yesterday and it looked great!15:20
rloomat128: I was reading the multitenancy doc and noticed it. so hard to remember 'everything'...15:20
JayFmat128: do I understand correctly that we're backporting it to newton? If so, we need to have a party \o/15:20
mat128rloo: agreed15:20
mat128JayF: thanks :) and yes we're backporting it15:20
mat128Andreas asked for it on the ML15:20
JayFI saw that15:21
rlooJayF: yup. supposedly on jroll's list to +2. I guess we need two people to get them +A'd/backported :)15:21
mat128rloo: concerning your last concern, do you think we should change them to "ref:" in master?15:21
JayFtonyb 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
rloomat128: the security page? yes.15:21
*** lucasagomes is now known as lucas-hungry15:22
rloomat128: and maybe just check that there aren't other cases like that.15:22
mat128they're hard to find15:22
mat128next time someone commits a link we should ask ourselves if a ref: would have been possible15:22
rloomat128: 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
rloomat128: yup, but... so hard to remember everything when reviewing :-(15:23
JayFI'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-Docs15:23
*** xek__ has joined #openstack-ironic15:23
mat128JayF: good idea ^15:25
*** xek_ has quit IRC15:25
jrollJayF: depending how many things you get in there, it might be worth a summit session?15:28
JayFjroll: 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
JayFjroll: someone else would have to lead it, and it'd suck to have those priorities made without my input :(15:29
jrollJayF: true :(15:29
openstackgerritVasyl Saienko proposed openstack/ironic: Testing broken grenade job  https://review.openstack.org/37765315:30
*** baoli has joined #openstack-ironic15:31
*** rajinir has joined #openstack-ironic15:32
rajinir<jlvillal>: Probably yes, Is it broke on subnet creation?15:32
zigoironic-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_99113715:35
zigono, no, ... no no no ... you can't write in /usr !!! :)15:35
JayF/o\15:36
zigoWho wrote ironic-ui ?15:36
zigoHow can I change this path?15:36
JayFkrotschek (sp?) isn't here15:37
JayFbetherly: cinerama: I think one of you work on this? ^15:37
betherlyye i work on it. one sec let me look15:37
zigoI think I know how.15:38
zigoironic-ui is simply importing openstack_dashboard/test/settings.py who has:15:38
zigoTEST_DIR = os.path.dirname(os.path.abspath(__file__))15:39
zigoSECRET_KEY = secret_key.generate_or_read_from_file(15:39
zigo    os.path.join(TEST_DIR, '.secret_key_store'))15:39
*** bks has quit IRC15:39
JayFif it's in openstack_dashboard; I wonder why you haven't seen it in the other horizon packages15:39
*** bks has joined #openstack-ironic15:39
betherlyzigo: thats part of the horizon package if im reading right?15:39
betherlyis that ok or what do i need to change?15:40
betherlygetting ready  to release v2.1.0 but dont want to release something with an error im not aware of15:40
zigobetherly: 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
zigobetherly: 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 IRC15:43
betherlyzigo: riiiight....ok....not entirely sure how to do that lol....15:43
jrollthis does seem like an openstack-dashboard bug, not ironic-ui15:45
zigobetherly: I'm attempting to patch.15:45
betherlyzigo: thanks :)15:45
jrollSECRET_KEY is a django thing, I doubt ironic-ui does anything special here15:45
betherlyjroll: thanks for the clarification!15:46
jrollbetherly: 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 there15:47
jrollor just set openstack_dashboard.test.settings.SECRET_KEY = 'foo'15:47
jrollor something15:47
zigojroll: 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
zigojroll: For horizon, that's fine, as it's not installed when building the package.15:47
zigoFor ironic-ui, that's a no-go ...15:48
zackfTrying 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-ironic15:48
*** bks has quit IRC15:48
zigoSo, while re-using the test-settings from Horizon is a good idea, this type of path should be overwriten.15:48
jrollzigo: let me put up a patch and see if it works for you15:48
zackfAnyone run into this kind of thing in their experiences?15:48
zigojroll: I'm attempting to do this: https://review.openstack.org/#/c/378694/3/debian/patches/fix-secret-key-path.patch15:49
patchbotpatch 378694 - deb-ironic-ui (debian/newton) - Merge tag '2.0.0' into debian/newton15:49
zigoLet's see if that builds ...15:49
jrollzigo: I think just making the import will break you15:49
* jroll tries alternate way just in case15:49
zigoAh...15:49
zigoMaybe, if it tries to write directly at the import time.15:50
*** aNuposic has quit IRC15:50
zigoOh, right, it's doing secret_key.generate_or_read_from_file() ... :(15:50
zigojroll: What are you attempting?15:50
*** kumarb has quit IRC15:50
jrollzigo: some mock magic15:51
zigoMaybe out of my skills then! :)15:51
*** joprovost1 has joined #openstack-ironic15:51
*** bks has joined #openstack-ironic15:51
zigobetherly: jroll: Please ping me when you have some news, I'm attempting to package another Horizon plugin now.15:51
*** joprovost has quit IRC15:53
*** joprovost1 is now known as joprovost15:53
jrollwill do15:53
*** electrofelix has quit IRC15:56
*** rbrud has quit IRC15:59
jrollzigo: betherly: https://review.openstack.org/#/c/378744/15:59
patchbotpatch 378744 - ironic-ui - Mock openstack_dashboard secret key in unit tests15:59
jrolltests pass on my system, I *think* it does what you need16:00
*** krtaylor has quit IRC16:01
betherlyjroll: awesome thanks so much for sorting that!16:01
jrollbetherly: np, let's see if it works for zigo before merging, but we should get that in the release16:01
jlvillalGood morning everyone :)16:01
betherlyjroll: for sur16:01
betherly*sure16:01
zigobetherly: jroll: I'm trying to apply this patch to the latest release of ironic-ui...16:01
jrollthanks16:02
*** moshele has joined #openstack-ironic16:02
*** mtanino has joined #openstack-ironic16:04
zigoPackage is building...16:05
*** tuan_luong has joined #openstack-ironic16:06
*** tuan_luong has quit IRC16:06
*** ijw has joined #openstack-ironic16:07
*** moshele has quit IRC16:11
zigoManila-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_40500816:11
*** vgadiraj has joined #openstack-ironic16:15
*** Romanenko_K has quit IRC16:16
*** jpich has quit IRC16:16
openstackgerritCharlle Daniel proposed openstack/ironic: Ensures that OneView nodes are free for use by Ironic  https://review.openstack.org/37276216:16
*** lucas-hungry is now known as lucasagomes16:20
*** strigazi is now known as strigazi_AFK16:21
*** Guest46101 is now known as mgagne16:21
*** mgagne has quit IRC16:21
*** mgagne has joined #openstack-ironic16:21
betherlyzigo: is the patch passing the tests yet?16:27
*** jaosorior has quit IRC16:27
*** athomas has quit IRC16:28
*** jrist has quit IRC16:28
jlvillalvsaienk0: 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
patchbotpatch 356026 - openstack-dev/devstack - Make the Neutron l3 plugin use the subnetpools (MERGED)16:33
*** baoli has quit IRC16:34
*** baoli has joined #openstack-ironic16:35
*** aNuposic has joined #openstack-ironic16:35
*** thrash is now known as thrash|biab16:36
*** sabeen has joined #openstack-ironic16:40
*** frontrunner has quit IRC16:41
NobodyCamGood morning Ironic'ers16:47
*** krtaylor has joined #openstack-ironic16:50
lucasagomesNobodyCam, hi there16:51
NobodyCammorning lucasagomes :)16:51
jrollmorning NobodyCam, lucasagomes :)16:51
NobodyCammorning jroll :)16:51
zigojroll: 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_75486316:51
zigoI have to go for dinner unfortunately.16:51
*** e0ne has quit IRC16:52
jrolllame :(16:52
zigoI'll be there tonight.16:52
jrollI think I know what I did wrong16:52
betherlyugh :/16:52
zigojroll: You could take over my packaging patch and try there if you'd like: https://review.openstack.org/37869416:52
patchbotpatch 378694 - deb-ironic-ui (debian/newton) - Merge tag '2.0.0' into debian/newton16:52
zigoThe way it works is that you add the patch in debian/patches16:53
zigoGot to go now, TTYL16:53
jrollzigo: got it, thanks16:53
lucasagomesdid 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 passed17:00
*** rama_y has joined #openstack-ironic17:00
lucasagomesreading around it seems that dnsmasq lacks DHCPv6 extensions for tftp booting ?17:00
jrollzigo: I've no clue how to do this, giving me branch errors and such :(17:00
jrollzigo: I updated my patch, not sure it'll work nor the best way to test this17:00
sambettsjlvillal: qa meeting today?17:01
jrollguess not!17:02
jlvillalsambetts: jroll We are all in the meeting.17:02
jlvillalWhere are you all?17:02
jrolloh wow17:02
sambettsjlvillal: literally must have started just as I switched back from the channel17:03
jrollyep17:03
*** thrash|biab is now known as thrash17:03
devanandag'morning, all17:12
betherlyo/ devananda :)17:12
*** bks has quit IRC17:16
*** MattMan has quit IRC17:21
*** MattMan has joined #openstack-ironic17:21
sambettso/ devananda17:22
openstackgerritJohn L. Villalovos proposed openstack/ironic: Testing a revert to see if fixes Grenade  https://review.openstack.org/37892817:23
*** praneshp has joined #openstack-ironic17:25
mariojvrloo: yuriyz: hey, regarding the comments on https://review.openstack.org/#/c/321865/33/ironic/conductor/manager.py17:25
patchbotpatch 321865 - ironic - Add power state change notifications17:25
mariojvdo you think switching the power sync notification on L2470 to "baremetal.node.power_sync.error" event type would be clearer?17:26
lucasagomesdevananda, betherly jlvillal o/ morning17:26
mariojvi thought the docs were OK in describing what the notification means, but maybe not17:26
mariojvi'm open to changing the event type string to something like "baremetal.node.power_state_changed.success" or something17:27
mariojvif that's clearer17:27
rloomariojv: 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
rloomariojv: what exactly are you/we trying to notify about?17:27
vdrokjlvillal: grenade job stopped working after the base branch changed to stable Newton in devstack gate17:27
rloomariojv: 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
mariojvrloo: any time ironic detects that the actual hardware power state changed from what ironic has in the DB for that node17:28
mariojvrloo: correct17:28
rloomariojv: so how could the notification be an .error?17:28
jlvillalvdrok: Ah17:28
rloomariojv: there was no error in setting node.power_state == actual_state17:28
mariojvrloo: that's why i put it as success - the error had nothing to do with ... exactly17:28
mariojvso i think baremetal.node.power_sync.success is ok17:29
mariojvbut i'm open to changing the action part of the event type ("power_sync") to something else, if it's clearer17:29
mariojvit's sort of a hard thing to concisely describe17:29
rloomariojv: but yuriyz thinks it should be .error. I'm wondering what he thinks the notification is for.17:29
* sambetts -> home17:29
sambettsnight all17:29
rloomariojv: yeah, easy to describe but not concisely.17:29
*** sambetts is now known as sambetts|afk17:29
rloonight sambetts|afk17:29
jlvillalsambetts|afk: Good night17:30
rloomariojv: power_reset? I forgot what i had proposed before.17:30
vdrokJlvillal So I think it can be anything merged in devstack since mitaka :(17:30
mariojvrloo: 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 state17:30
JayFI 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 erorr17:30
JayFso if you run power sync and it's out of sync, I could see someone feeling like that was an error17:30
mariojvrloo: i think you proposed power_sync, but i don't remember17:30
mat128JayF: ^we use "sync power state = off" and may control it ourselves outside of Ironic17:30
JayFmat128: 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 error17:31
mariojvyeah, i don't think it's an error either17:31
rlooJayF: that is a valid point. which goes back to my question, *what* are we (theRoyalWe) trying to convey in the notification?17:31
JayFmat128: but I think it's a point where other folks COULD think that17:31
mariojvironic did nothing wrong, so if nothing else it's not an ironic error17:31
jlvillalmorning lucasagomes17:31
mat128JayF: so you mean, if Ironic has to act on the device because it thought otherwise?17:31
mat128JayF: I could see a notification for this, but not of type "error"17:31
mariojvrloo: 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 driver17:31
mariojvrloo: maybe i should guard that piece in an if statement, i think right now it notifies whether it changed it or not, on L247017:32
JayFmat128: ++ I agree. We do not disagree. lol17:32
rloomariojv: ok, that is settled then. how to convey that.17:32
mariojvpower_sync is fine imo17:32
mariojvas long as the docs are accurate17:32
rlooi 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
rloopower_set_to_hw?17:33
mariojvpower_state_modified?17:33
JayFhow about power_synced_to_db17:33
JayFvs power_synced_to_hw17:33
mariojvi'm fine with power_state_modified or power_synced_to_db17:33
rlooJayF: can you think of another word than 'sync'?17:33
mariojvpower_set_to_hw sounds like it could be the same as power_set17:33
rloomariojv, JayF: power_adjust? no.17:34
rloopower_match?17:34
rloopower_equal?17:35
JayFI actually like power_state_modified, or similar17:35
mat128changed17:35
JayFif I could think of a term that would be used if we modified the actual power of the node instead17:35
mariojvwhat'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 imbw17:35
mariojvooh17:36
mariojvpower_state_corrected?17:36
JayFoh, I like that17:36
rloopower_state_adjusted?17:36
JayFmariojv: how would you indicate corrected via ipmi or corrected via db?17:36
*** ralonsoh has quit IRC17:36
mariojvpower_state_corrected_in_db?17:36
mariojvJayF: if there's a driver (aka ipmi) call to change it, that sends a power_set notification17:37
mariojvthis is exclusively for when ironic changes its db representation of the node to the hardware power state17:37
mariojvthis being what's currently named power_sync17:37
JayFI like power_state_corrected for that17:37
rlooJayF: 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 that17:37
JayFor power_state_matched17:37
mat128they are 2 separate actions17:37
mariojvi'm fine with power_state_matched, too17:38
mariojvadjusted to me might indicate that ironic adjusted the hw power state, which isn't the case here17:38
JayFyeah, I don't like adjusted17:38
rloohey, i'm still looking through the thesaurus, might come up with some more :)17:38
JayFfor that reason17:38
mariojvhttp://www.thesaurus.com/browse/sync?s=t17:38
mariojvpower_state_harmonized :P17:38
JayFrloo: me too :P and I have a nice ad for Ford cars w/Sync technology as a result, lol17:38
rloopower_state_reconcile :)17:38
mariojvhaha. power_state_reconciled is fine with me too17:38
JayFI think jibe would be my favorite joke suggestion17:39
*** baoli has quit IRC17:39
*** baoli has joined #openstack-ironic17:39
mariojvso 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
rlooJayF: i like jibe but ...17:39
*** baoli has quit IRC17:40
mat128mariojv: power_state_corrected, as long as I can know which of them was corrected17:40
rloorestore?17:40
rloono, forget restore17:40
*** baoli has joined #openstack-ironic17:41
mariojvmat128: yeah, it will be clear in the payload17:41
rloomariojv: i like corrected & reconciled17:41
mat128mariojv: will there be a different notification if it ends up calling the power driver?17:41
rloomat128: yup. that's a power_set notification17:41
mariojvmat128: to change the power, yes17:41
JayFmat128: it would also send a power_set notification17:41
mariojvyup, what rloo said17:41
mat128thats perfect then17:42
mariojvrloo: ok, if no vehement disagreement, i'll go with power_state_corrected for the next patch set17:42
mariojvthanks!17:42
*** aswadr_ has quit IRC17:42
rloomariojv: for now anyway :)17:42
mariojv:(17:42
rloomariojv: 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
rloomariojv: but i think we should be good as long as the documentation is clear...17:43
mariojvok, i'll re-read it and try to read from the perspective of someone who has no idea what this is all supposed to do17:43
mariojvmaybe 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 overkill17:44
*** e0ne has joined #openstack-ironic17:44
rloomariojv: nah. just clearly describe them.17:44
rloomariojv: and don't use 'sync' anywhere when describing power_state_corrected17:45
mariojvack, thx17:46
rlooJayF: a really easy +2/+A. I verified it :) https://review.openstack.org/#/c/378291/17:46
patchbotpatch 378291 - ironic - Fix inexact config option name in multitenancy.rst17:46
rlooor NobodyCam ^^17:47
JayFrloo: I got it17:47
NobodyCamhey hey morning rloo17:47
NobodyCammorning JayF17:47
NobodyCamand thank you :)17:47
JayFrloo: I just was checking to see who we were going to make rebase; and it was you, so did it ;P17:47
rloohi NobodyCam, you're too late :)17:47
NobodyCamlol17:47
rlooJayF: yeah, I have to redo mine anyway, so thought might as well get that in first.17:47
*** hshiina has quit IRC17:48
betherlyzigo: jroll: thanks so much for trying to sort this out! so appreciated! please do ping me and keep me posted17:52
openstackgerritRuby Loo proposed openstack/ironic-specs: Amend ipxe-swift-tempurls spec  https://review.openstack.org/37268217:56
*** trown is now known as trown|lunch17:57
openstackgerritMerged openstack/ironic: Fix inexact config option name in multitenancy.rst  https://review.openstack.org/37829118:00
*** sabeen has quit IRC18:02
openstackgerritRuby Loo proposed openstack/ironic: Update multitenancy documentation  https://review.openstack.org/37813818:16
*** ijw has quit IRC18:16
*** jheroux has joined #openstack-ironic18:18
*** rbrud has joined #openstack-ironic18:21
narasimha_SVxavierr: http://paste.openstack.org/show/583358/ these are the commands. for the kernel and ramdisk details , I am replacing the variables with the IDs18:27
*** baoli has quit IRC18:28
*** krtaylor has quit IRC18:32
*** vgadiraj has quit IRC18:34
xavierrnarasimha_SV: just a minute :)18:35
*** rpioso has quit IRC18:41
*** ijw has joined #openstack-ironic18:41
*** narasimha_SV has quit IRC18:42
*** vsaienko has joined #openstack-ironic18:45
*** sdake has quit IRC18:46
*** ijw has quit IRC18:46
xavierrmorning NobodyCam :)18:47
NobodyCammorning xavierr :)18:47
*** krtaylor has joined #openstack-ironic18:48
*** rpioso has joined #openstack-ironic18:55
*** rbrud has quit IRC18:55
*** baoli has joined #openstack-ironic18:56
*** hogepodge has quit IRC19:01
*** trown|lunch is now known as trown19:03
*** baoli has quit IRC19:05
*** baoli has joined #openstack-ironic19:06
*** hoonetorg has quit IRC19:14
*** anush has quit IRC19:17
*** anush has joined #openstack-ironic19:17
*** rpioso has quit IRC19:20
*** vsaienko has quit IRC19:24
*** vsaienko has joined #openstack-ironic19:25
*** vsaienko has quit IRC19:28
*** vsaienko has joined #openstack-ironic19:29
*** sabeen has joined #openstack-ironic19:31
*** hoonetorg has joined #openstack-ironic19:31
openstackgerritStenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation  https://review.openstack.org/37707319:32
*** sdake has joined #openstack-ironic19:34
*** rpioso has joined #openstack-ironic19:34
*** tangchen_ has quit IRC19:45
*** tangchen_ has joined #openstack-ironic19:45
*** vsaienko has quit IRC19:50
*** vsaienko has joined #openstack-ironic19:51
*** ppiela has quit IRC19:57
*** vsaienko has quit IRC19:59
lucasagomescalling it a day! Talk you tomorrow20:00
*** lucasagomes is now known as lucas-afk20:04
rloobye lucas-afk20:04
xavierrnight lucas-afk20:06
rlooxavierr: 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
patchbotpatch 374989 - ironic - Avoid name errors in oneview periodics20:07
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Fix import of ironicclient and reformat docstring  https://review.openstack.org/37852620:12
openstackgerritMathieu Mitchell proposed openstack/ironic: Update external links to developer documentation  https://review.openstack.org/37899020:15
openstackgerritMathieu Mitchell proposed openstack/ironic: Update configuration reference link to latest draft  https://review.openstack.org/37899120:22
openstackgerritMathieu Mitchell proposed openstack/ironic: Point configuration reference and sample config to Newton  https://review.openstack.org/37899420:23
*** Goneri has quit IRC20:27
mat128rloo: addressed your points in https://review.openstack.org/#/c/378037/ via new commits ^20:28
patchbotpatch 378037 - ironic (stable/newton) - Point documentation links to newton equivalents20:28
rloomat128: thx.20:28
*** baoli has quit IRC20:29
rloomat128: this conflicts with your other patches: https://review.openstack.org/#/c/378994/20:32
patchbotpatch 378994 - ironic (stable/newton) - Point configuration reference and sample config to...20:32
mat128oh wow20:33
mat128it conflicts because it touches the same file20:33
mat128I can put it at the end of the patch chain20:33
rloomat128: i've seen where it doesn't conflict if the same file, it depends on the lines that are touched.20:33
rloomat128: yeah, if you put it at the end, that should work.20:33
rloomat128: i would have put it with the 'Point documentation links to newton equivalents' patch myself, instead of a new patch.20:34
mat128riight20:34
*** rpioso has quit IRC20:34
mat128my only concern was the manual edits vs commands20:34
mat128I guess I can merge and note it in the commit message20:34
mat128the goal is the same20:34
rloomat128: oh, just note it in the commit msg :)20:35
rloomat128: or add new seds to do that :D20:35
mat128lol20:35
*** wajdi_ has joined #openstack-ironic20:35
mikaljroll: 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 opportunities20:35
openstackgerritMathieu Mitchell proposed openstack/ironic: Point documentation links to newton equivalents  https://review.openstack.org/37803720:36
mat128rloo: squashed + abandoned the other one20:36
rloomat128: thx20:36
* mat128 is getting better and better with git20:36
*** wajdi__ has joined #openstack-ironic20:37
mat128mikal: both ramdisks should have the same features. Did you file a bug?20:38
*** wajdi has quit IRC20:39
*** wajdi_ has quit IRC20:39
mikalmat128: yes, I filed a bug for the coreos problem I found20:41
mikalmat128: it turns out that calling shutdown inside a docker container doesn't do what the author had hoped it did20:41
mat128mikal: it's running from a chroot now, we used to issue reboot via sysrq exactly for this20:42
mat128let me look at the report :)20:42
jrollmikal: morning :)20:42
mat128ouch20:42
mat128"Running in chroot, ignoring request"20:42
mikalhttps://bugs.launchpad.net/ironic-python-agent/+bug/162836720:42
openstackLaunchpad bug 1628367 in ironic-python-agent "shutdown.sh fails with coreos IPA image" [Undecided,New]20:42
*** wajdi__ has quit IRC20:43
jrollthat's a fun one20:43
*** rpioso has joined #openstack-ironic20:43
mikalSo, its not terminal, but it does slow down my deply by like 30 seconds20:43
mat128https://github.com/openstack/ironic-python-agent/commit/cbd90c1c14e083c5cc563cfd94f55266d28f77be20:43
mikalI'm confused about why the agent is in a container / chroot at all to be honest20:43
mikalGiven its a single purpose ram disk20:43
openstackgerritMarc Aubry proposed openstack/python-ironicclient: Make error work with client in python3  https://review.openstack.org/37805020:46
JayFmikal: 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 ci20:46
JayFmikal: at least that's what I'll suggest :)20:47
mat128JayF: as long as we have the agent element in-tree :)20:47
JayFmat128: that's part of what we're waiting for20:47
mat128mikal: in the meantime, I guess you could revert https://github.com/openstack/ironic-python-agent/commit/cbd90c1c14e083c5cc563cfd94f55266d28f77be20:48
mikalDoes the coreos image use containers / chroots because that's how coreos works?20:48
JayFmikal: coreos doesn't ship with python.20:48
mikalI don't see why dib would build a different ram disk than coreos20:48
JayFmikal: the coreos "way" is to ship containers, it stopped being a container and started being a chroot because of technical reasons20:48
mat128mikal: dib could (can) build Ubuntu/Fedora/whatever ramdisks20:48
mikalIs the dib method documented anywhere? I see docs for coreos and tinyipa, but I don't think I noticed dib docs20:49
mikal(But I might have glossed over them)20:49
mat128oh yes it is20:49
* mat128 has just migrated all of the docs :)20:49
mat128http://docs.openstack.org/project-install-guide/baremetal/draft/deploy-ramdisk.html20:49
mikalmat128: oh, I've been looking at the readmes in the agent source20:50
mikalWhich only cover the other two20:50
mikalAnd don't point to the web docs20:50
* mikal might tweak that today20:50
JayFmikal: feel free to add me as a reviewer to any docs or ipa patches you post20:50
mikalJayF: sir yes sir20:50
*** bfournie has quit IRC20:51
mikalAt 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
mikalSo yeah, not sure where this adventure will take me, but I'll let you know what I learn along the way.20:51
JayFI'd strongly suggest building a dib image20:52
JayFand seeing if those issues just go poof20:52
mikalSo far we've saved 2 minutes of that by moving to tinyipa20:52
mikalOk, I can try that today now that I know its a thing20:52
mat128mikal: how big is the image you are deploying?20:53
mikalJayF: 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 customer20:53
mikalmat128: a few hundred mb IIRC?20:53
*** joprovost has quit IRC20:53
mikali.e. not huge20:53
mat128mikal: our deployments are much faster than that, even when cold booting20:53
mat128mikal: on supermicro too20:53
mikalHmmm, interesting20:54
mat128mikal: there is one more thing that might slow you down20:54
mikalThese supermicros a very slow to boot / POST to be honest20:54
mat128yes20:54
mat128unavoidable except if you start deploying from a warm machine20:54
JayFmikal: I don't really have the time to look over them, but if you have an inexplicable log snippet you can toss it my way20:54
mikalJayF: oh, I wouldn't recommend them unless you have insomnia20:54
mat128mikal: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/write_image.sh#L4220:55
mikalJayF: 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
mikalmat128: well, the image has to be synced before reboot anyway though right?20:56
mikalmat128: so that's just forcing that earlier in the chain20:56
mat128mikal: 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 faster20:56
mikalmat128: oh, interesting20:56
mat128mikal: if your guest image is very sparse, you might be doing a lot of random io20:56
mikalI think you also raised something else interesting there...20:57
mat128where as other buffering modes might make it more linear20:57
mat128experiment with it :)20:57
mikalIronic is powering off the machines when they're not in use20:57
mat128so far we have a patch that simply removes "-t directsync"20:57
mikalI'd be ok with them sitting warm instead if it meant a faster boot20:57
mikalHow do I convince ironic to do that thing?20:57
mat128mikal: thats not a built in feature right now20:57
mikalmat128: I shall try that too, thanks20:57
mat128mikal: 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 deployment20:58
mat128let me find that line :)20:59
*** ijw has joined #openstack-ironic21:00
mat128mikal: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L46221:00
mat128turns off the node when preparing deployment21:00
openstackgerritMerged openstack/ironic-python-agent: Add info on tinyipa to docs  https://review.openstack.org/37471421:00
*** Marx314 has joined #openstack-ironic21:00
mat128mikal: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L41721:00
*** ijw has quit IRC21:01
*** trown is now known as trown|outtypewww21:01
mat128mikal: so yeah, a few things to patch out if you want to start deploying for a machine that is already on21:01
* mikal wonders if he can trick his coworkers into having a go at doing that thing21:01
*** ijw has joined #openstack-ironic21:01
*** ijw has quit IRC21:03
mat128Gotta go, good luck mikal :)21:04
*** ijw has joined #openstack-ironic21:05
*** ijw has quit IRC21:05
*** ijw has joined #openstack-ironic21:05
openstackgerritMerged openstack/ironic-specs: Amend ipxe-swift-tempurls spec  https://review.openstack.org/37268221:05
jrollmikal: I proposed that spec for juno and got... busy21:06
jrollheh21:06
*** baoli has joined #openstack-ironic21:06
jrollmat128: I approved your backports, hopefully devananda or dtantsur can pick them up21:07
mat128yay :)21:07
mat128jroll: that one is also a pure backport https://review.openstack.org/#/c/377988/21:07
patchbotpatch 377988 - ironic (stable/newton) - Fix absolute links to install-guide.rst in develop...21:07
*** e0ne has quit IRC21:07
jrollmat128: oh, you caught me in a lie21:08
JayFmat128: can you put a list of those backport patches in an ehterpad or something?21:08
jrollI thought I finished, two left to go21:08
xavierrty 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
jrollJayF: they're all in the same tree21:08
mat128JayF: https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:stable/newton+topic:bug/161227821:08
JayFmat128: I can ping tonyb about them when he gets online from australia21:08
jrolltonyb's boss is here in channel21:08
mikalYes, please hassle tonyb. He loves that stuff.21:08
JayFtonyb: 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, PTL21:08
mat128Hahaha21:08
JayFmikal: 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 does21:08
mikalAlthough, I think he'll just yell at me about nova stable reviews for revenge21:09
mikalJayF: no, he does. I'm just teasing him in absentia.21:09
JayFIs mikal able to +2 things in any stable repo?21:09
mikalrloo: I'm not stable core21:09
jrollpro tip: if mikal is speaking it is snark21:09
mikalrloo: I'm only nova-stable core21:09
rloomikal: but you're a boss! :)21:09
mikalSo I think that means I'm just a bystander here?21:09
mikalrloo: I try very hard to not tell people how to vote in gerrit during meetings21:09
mikalrloo: cause I don't want to be a bad person21:10
rloomikal: ha ha...21:10
*** mat128 is now known as mat128|afk21:11
rloomikal: in all fairness, for those that don't know, I am just joking with you. You are trusted!21:11
JayFjroll: you missed https://review.openstack.org/#/c/378037/21:11
patchbotpatch 378037 - ironic (stable/newton) - Point documentation links to newton equivalents21:11
mat128|afkalright, bye everyone21:11
jrollJayF: no, I'm reviewing it21:11
JayFah, cool21:11
jrollI said two left 3 minutes ago, dang21:11
JayFthat's at the top of the chain, right?21:11
* jroll hands JayF his old management whip21:11
jrollyes21:11
JayFor am I reversed in gerrit again /o\21:11
mat128|afkJayF: depends what your "top" is, but yes21:11
jrolltop meaning HEAD21:11
JayFit's the foundation for all the other commits21:11
JayFno, it's the HEAD?21:12
mat128|afkJayF: the other way around21:12
JayFman, I hate gerrit ui for this21:12
mat128|afkit's the leaf in the graph21:12
rlooc'mon, stop interrupting jroll so he can finish the reviews. i can't wait for them to be over...21:12
mat128|afkJayF: my personal trick is "It reads like git log --oneline"21:12
mat128|afkhahaha21:12
jrollit's done21:12
mat128|afk:D21:12
JayFmat128|afk: that's ridiculously helpful, tyvm21:12
rlooyay. now we need one other person with powers...21:12
mat128|afkgotta go21:12
mat128|afkfor real this time21:12
mat128|afkJayF: np :)21:13
*** dprince has quit IRC21:14
*** dprince has joined #openstack-ironic21:17
*** krtaylor has quit IRC21:21
zigojroll: I'm back and trying the last version of your patch.21:21
*** dprince has quit IRC21:25
jrollzigo: I'm just heading off, but let me know21:27
jrollI'll check back soonish21:27
zigook21:27
*** Goneri has joined #openstack-ironic21:28
*** hogepodge has joined #openstack-ironic21:28
*** sabeen has quit IRC21:28
*** sabeen has joined #openstack-ironic21:30
mikalrloo: awww, that's the nicest thing which has been said to me today21:31
mikalrloo: mostly its been dogs barking to be honest21:31
*** Goneri has quit IRC21:33
rloomikal: 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 IRC21:36
betherlyzigo: thanks :) keep me posted how it goes!21:36
mikalLOL21:36
mikal"On a scale of hugs through stabby, how would you rate your urge to murder mikal?"21:37
zigojroll: 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_94973021:37
betherly:/21:37
openstackgerritJohn L. Villalovos proposed openstack/ironic: Update iptables rules and services IPs for multinode  https://review.openstack.org/36864821:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: Fix setting custom IRONIC_VM_NETWORK_BRIDGE  https://review.openstack.org/36511621:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: Add devstack setup_vxlan_network()  https://review.openstack.org/36864721:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: Skip some steps for multinode case  https://review.openstack.org/36864621:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: Update ironic node names for multinode case  https://review.openstack.org/36864521:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: Update devstack provision net config for multihost  https://review.openstack.org/36864421:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: Source openrc on subnode in multinode topology  https://review.openstack.org/35330221:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: Skip db configuration on subnodes  https://review.openstack.org/35330321:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817321:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: Ignore required_services for multinode topology  https://review.openstack.org/35279321:38
openstackgerritStenio Araujo proposed openstack/python-oneviewclient: Adds attribute mac_type to a SPT and a validation  https://review.openstack.org/37705421:39
*** Goneri has joined #openstack-ironic21:41
*** absubram has quit IRC21:42
openstackgerritMarc Aubry proposed openstack/python-ironicclient: Make error work with client in python3  https://review.openstack.org/37805021:43
rloomikal: i hadn't thought of phrasing it that way, but it'll do!21:44
rloojlvillal: ^^ what's that about?21:45
jlvillalrloo: I'm just super productive ;)21:45
jlvillalrloo: I did some changes to the multi-node job in the devstack-gate section.21:45
rloojlvillal: I'm impressed!21:45
jlvillalrloo: So I had to make some changes in the Ironic code.21:46
jlvillalrloo: s/MULTI_HOST/IRONIC_MULTI_NODE/g21:46
jlvillalAnd then push the new patches :)21:46
rloojlvillal: ahhh. I'd like to take a look at those patches. one day...21:46
jlvillalrloo: Basically Nova is already using MULTI_HOST and might be confusing. So I changed the devstack variable to IRONIC_MULTI_NODE21:46
rloojlvillal: i think you should ask nova to change theirs to NOVA_MULTI_HOST :)21:47
JayFI think nova wins all games of review-chicken21:47
jlvillalYES!!!! :)21:47
*** Goneri has quit IRC21:53
*** baoli has quit IRC21:53
*** rajinir has quit IRC21:55
*** baoli_ has joined #openstack-ironic21:58
mrdaMorning Ironic22:00
*** baoli_ has quit IRC22:04
*** krtaylor has joined #openstack-ironic22:10
*** mkam has quit IRC22:11
*** Goneri has joined #openstack-ironic22:11
*** praneshp has quit IRC22:11
tonybholy 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
JayFtonyb: I'm the docs liason for Ironic, and I approved this message(tm)22:13
*** sabeen1 has joined #openstack-ironic22:13
*** baoli has joined #openstack-ironic22:13
JayFtonyb: 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 job22:14
jrollbetherly: let's just go ahead and release as-is and we can backport the fix when we come up with it22:14
jrollbetherly: let me know when you un-wip and I'll handle the rest :)22:15
tonybJayF: 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
JayFtonyb: 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 IRC22:15
JayFtonyb: but if you're uncomfortable we can get someone on ironic stable maint to push the button22:16
*** baoli has quit IRC22:16
tonybJayF: I'm not super-uncomfortable but I just want to check.22:16
*** cdearborn has quit IRC22:18
rlootonyb: 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.html22:19
rlootonyb: he even asked nicely :)22:20
tonybrloo: He's a bit of both :)22:20
*** bdeschenes has quit IRC22:20
*** praneshp has joined #openstack-ironic22:28
betherlyjroll: will that not mess up the gates?22:29
*** vishwanathj has quit IRC22:29
*** bdeschenes has joined #openstack-ironic22:31
betherlyjroll: 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 out22:31
patchbotpatch 378668 - ironic-ui - Added release notes for 2.1.0 release22:31
jrollbetherly: right on, and no, zigo's bug is in the previous release22:36
jrollso shouldn't bother any CI or anything, except for his, which is already bothered22:38
betherlyjroll: 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-ironic22:41
jrollbetherly: oof, yeah, it didn't get in the pipeline22:41
jrolldone22:41
betherlythanks so much jroll22:42
betherlyI'll look first thing and sort the release22:42
jrollnp, thank you!22:42
betherlyHave a good evening!22:42
jrollit's you that's here way too late, not me22:42
jrollyou too :)22:42
*** awiddersheim has joined #openstack-ironic22:42
*** joprovost has quit IRC22:46
*** ijw has quit IRC22:46
openstackgerritMerged openstack/ironic-ui: Added release notes for 2.1.0 release  https://review.openstack.org/37866822:54
*** sdake has quit IRC22:56
*** ijw has joined #openstack-ironic22:58
*** rpioso has quit IRC22:59
*** ijw has quit IRC23:03
*** ijw has joined #openstack-ironic23:19
*** Goneri has quit IRC23:22
*** ijw has quit IRC23:24
*** ijw has joined #openstack-ironic23:29
openstackgerritBruno Cornec proposed openstack/ironic-specs: Propose Redfish support into Ironic  https://review.openstack.org/18465323:31
*** sdake has joined #openstack-ironic23:32
*** ijw has quit IRC23:34
*** sdake_ has joined #openstack-ironic23:35
*** sdake has quit IRC23:38
xavierrmorning mrda23:39
mrdahey xavierr23:45
*** ijw has joined #openstack-ironic23:47
openstackgerritXavier proposed openstack/ironic: Add inspection feature for the OneView drivers  https://review.openstack.org/36706523:50
*** rama_y has quit IRC23:50
openstackgerritBruno Cornec proposed openstack/ironic-specs: Propose Redfish support into Ironic  https://review.openstack.org/18465323:51
*** ijw has quit IRC23:53
xavierrhave a good evening all23:54
*** xavierr is now known as xavierr|afk23:54

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