Thursday, 2017-03-09

*** david-lyle has joined #openstack-ironic00:08
*** srart has quit IRC00:09
*** gfhellma has joined #openstack-ironic00:12
*** david-lyle has quit IRC00:15
*** gfhellma has quit IRC00:20
joannaTheJulia: does it start tomorrow?00:24
*** goldyfruit has quit IRC00:24
*** priteau has joined #openstack-ironic00:24
*** priteau has quit IRC00:29
*** srart has joined #openstack-ironic00:32
*** baha has quit IRC00:36
*** vills has quit IRC00:40
*** zhurong has joined #openstack-ironic00:46
*** Lambert has joined #openstack-ironic00:51
*** david-lyle has joined #openstack-ironic00:54
*** rloo has quit IRC00:54
*** david-lyle has quit IRC00:54
*** Syed__ has quit IRC00:55
*** hw_wutianwei has joined #openstack-ironic00:55
*** SerenaFeng has joined #openstack-ironic00:56
mat128aNuposic, JayF: I just noticed the ping and cant spend too much time here, but send me the link to the review where the error occurs and I'll look at it tomorrow morning01:06
*** Goneri has joined #openstack-ironic01:07
aNuposicmat128: cool, i will put in gerrit the error and will update you with the link01:08
mat128ty01:09
*** rama_y has quit IRC01:12
TheJuliajoanna: at this point, likely not until next week. :(01:14
*** jjohnson2 has joined #openstack-ironic01:19
*** zhangjl has joined #openstack-ironic01:30
*** pmannidi has quit IRC01:30
*** Goneri has quit IRC01:31
*** mtanino has quit IRC01:35
*** tuanluong has joined #openstack-ironic01:40
openstackgerritzack chen proposed openstack/ironic-python-agent master: collected logs is 'bytes' type.  https://review.openstack.org/43890101:51
*** david-lyle has joined #openstack-ironic01:52
*** jkilpatr has joined #openstack-ironic02:05
*** ijw has quit IRC02:06
*** ijw has joined #openstack-ironic02:07
*** ijw has quit IRC02:09
*** jkilpatr has quit IRC02:10
*** zhangjl has left #openstack-ironic02:14
*** kjw3 has joined #openstack-ironic02:23
*** phuongnh has joined #openstack-ironic02:45
*** rbudden has quit IRC02:50
*** rbudden has joined #openstack-ironic02:53
*** amotoki has joined #openstack-ironic02:57
*** amotoki has quit IRC03:09
*** bfournie has joined #openstack-ironic03:12
*** kjw3 has quit IRC03:12
*** bfournie has quit IRC03:12
*** tommylikehu has joined #openstack-ironic03:13
*** frontrunner has joined #openstack-ironic03:15
*** jjohnson2 has quit IRC03:17
*** SerenaFeng has quit IRC03:21
*** jcoufal has joined #openstack-ironic03:23
*** amotoki has joined #openstack-ironic03:32
*** jcoufal has quit IRC03:35
*** karthiks has quit IRC03:36
*** aparnav has joined #openstack-ironic03:43
*** amotoki has quit IRC03:46
*** fragatina has quit IRC03:48
*** karthiks has joined #openstack-ironic03:49
*** sayantan_ has quit IRC03:58
*** g3ek has quit IRC03:58
*** udesale has joined #openstack-ironic03:59
*** sayantan_ has joined #openstack-ironic03:59
*** amotoki has joined #openstack-ironic04:02
*** frontrunner has quit IRC04:05
*** ijw has joined #openstack-ironic04:07
*** g3ek has joined #openstack-ironic04:08
*** amotoki has quit IRC04:10
*** SerenaFeng has joined #openstack-ironic04:12
*** Nisha_Agarwal has joined #openstack-ironic04:14
*** links has joined #openstack-ironic04:17
*** crushil has joined #openstack-ironic04:20
*** sasha21 has quit IRC04:22
*** amotoki has joined #openstack-ironic04:24
*** jaganathan has joined #openstack-ironic04:24
*** priteau has joined #openstack-ironic04:26
*** priteau has quit IRC04:30
*** rbudden has quit IRC04:31
*** moshele has joined #openstack-ironic04:32
*** moshele has quit IRC04:42
*** Nisha_Agarwal has quit IRC04:45
*** SerenaFeng has quit IRC04:53
*** jaganathan has quit IRC04:58
*** ijw has quit IRC05:08
*** fragatina has joined #openstack-ironic05:18
*** fragatin_ has joined #openstack-ironic05:20
*** fragatina has quit IRC05:22
*** fragatin_ has quit IRC05:25
*** fragatina has joined #openstack-ironic05:25
*** mvpnitesh has joined #openstack-ironic05:27
*** fragatina has quit IRC05:29
*** fragatina has joined #openstack-ironic05:32
*** fragatina has quit IRC05:33
*** vikrant has joined #openstack-ironic05:35
*** SerenaFeng has joined #openstack-ironic05:36
*** fragatina has joined #openstack-ironic05:36
openstackgerritkavitha h r proposed openstack/ironic-lib master: Python 3.5 is added  https://review.openstack.org/44342105:37
mvpniteshhi , i'm trying to install Devstack setup of ironic , I'm getting  Internal Server Error (HTTP 500) due to "s-proxy failed to start" , s-object failed to start" , "s-account failed to start05:39
patchbotError: No closing quotation05:39
*** yolanda has quit IRC05:41
mvpniteshShould i install swift for Ironic devstack setup ??05:41
*** fragatina has quit IRC05:41
*** fragatina has joined #openstack-ironic05:42
*** venki has joined #openstack-ironic06:03
*** links has quit IRC06:04
*** stendulker has joined #openstack-ironic06:05
*** crushil has quit IRC06:10
*** karthiks has quit IRC06:14
*** links has joined #openstack-ironic06:16
*** jaganathan has joined #openstack-ironic06:24
*** priteau has joined #openstack-ironic06:27
*** moshele has joined #openstack-ironic06:27
*** venki has quit IRC06:29
*** priteau has quit IRC06:32
*** venki1 has joined #openstack-ironic06:34
*** venki1 has left #openstack-ironic06:34
*** rrr has joined #openstack-ironic06:35
*** venki1 has joined #openstack-ironic06:35
*** venki1 has left #openstack-ironic06:42
*** rrr has quit IRC06:42
*** kkk has joined #openstack-ironic06:42
*** mjura has joined #openstack-ironic06:44
*** rcernin has joined #openstack-ironic06:44
*** kkk has quit IRC06:46
*** e0ne has joined #openstack-ironic06:54
*** links has quit IRC06:58
*** amotoki_ has joined #openstack-ironic07:01
*** amotoki has quit IRC07:03
*** e0ne has quit IRC07:04
*** sayantan_ has quit IRC07:09
*** sayantan_ has joined #openstack-ironic07:09
*** venki has joined #openstack-ironic07:09
venkihi guys, can we use IRONIC_HWINFO_FILE as mentioned in https://software.intel.com/en-us/articles/physical-server-provisioning-with-openstack ?07:11
venkifor baremetal provisioning ....07:11
pas-hamorning ironic :)07:15
*** links has joined #openstack-ironic07:15
pas-hamvpnitesh: you need swift if you plan to use/test agent-* drivers (agent on the node downloads the target user image from swift tempurl)07:16
pas-hamvpnitesh: I have a patch around for our docs describing how to set it yourself (though devstack should've handled that for you if using it) https://review.openstack.org/#/c/379422/07:17
patchbotpatch 379422 - ironic - [install-guide] add section on Glance+Swift config07:17
pas-havenki: afaik IRONIC_HWINFO_FILE is used by devstack, and is used by it to enroll real HW nodes into devstack. not sure how applicable it to real openstack setup07:19
mvpnitesh@pas-ha: thanks07:19
*** dsneddon has quit IRC07:21
*** tesseract has joined #openstack-ironic07:22
*** karthiks has joined #openstack-ironic07:27
*** vmud213 has joined #openstack-ironic07:31
*** rbartal has joined #openstack-ironic07:31
*** zhangjl has joined #openstack-ironic07:34
*** rbartal has quit IRC07:37
*** venki has quit IRC07:42
*** rbartal has joined #openstack-ironic07:49
*** SerenaFeng has quit IRC07:53
*** SerenaFeng has joined #openstack-ironic07:53
*** ralonsoh has joined #openstack-ironic07:54
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Serve boot configs from Ironic API  https://review.openstack.org/36943808:03
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Implement Driver API for dynamic boot config  https://review.openstack.org/39439808:03
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add RPC API to get the boot config  https://review.openstack.org/39439908:03
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Allow not writing pxe configs to disk  https://review.openstack.org/39439708:03
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: DNM test enabled dynamic iPXE  https://review.openstack.org/39194308:03
*** makowals has quit IRC08:06
openstackgerritTuan Luong-Anh proposed openstack/bifrost master: Indicating the location tests directory in oslo_debug_helper  https://review.openstack.org/44346308:07
*** makowals has joined #openstack-ironic08:07
*** venki has joined #openstack-ironic08:08
*** mgoddard has joined #openstack-ironic08:11
*** milan has joined #openstack-ironic08:13
milanmorning Ironic! :)08:14
openstackgerritOksana Voshchana proposed openstack/ironic master: Add wsgi handling to ironic-api in devstack  https://review.openstack.org/43085108:16
openstackgerritTuan Luong-Anh proposed openstack/ironic-ui master: Indicating the location tests directory in oslo_debug_helper  https://review.openstack.org/44347208:17
*** fragatina has quit IRC08:22
Lambertmorning!08:23
*** mvpnitesh has quit IRC08:24
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355608:26
*** ccamacho has joined #openstack-ironic08:26
*** priteau has joined #openstack-ironic08:27
*** mgoddard has quit IRC08:28
*** jaosorior has joined #openstack-ironic08:28
openstackgerritVasyl Saienko proposed openstack/ironic master: Add ironic standlaone test with ipmi dynamic driver  https://review.openstack.org/43912608:28
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729908:29
*** priteau has quit IRC08:32
openstackgerritVasyl Saienko proposed openstack/ironic master: Add standalone tests using direct HTTP links  https://review.openstack.org/43754908:38
*** mgoddard has joined #openstack-ironic08:40
*** amarao has joined #openstack-ironic08:41
*** e0ne has joined #openstack-ironic08:44
openstackgerritOksana Voshchana proposed openstack/ironic master: Add wsgi handling to ironic-api in devstack  https://review.openstack.org/43085108:48
openstackgerritOksana Voshchana proposed openstack/ironic master: Add wsgi handling to ironic-api in devstack  https://review.openstack.org/43085108:50
vdrokgood morning ironic, milan and Lambert08:51
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729908:51
milanmorning Lambert, vdrok! :)08:52
Lambertmorning Milan!08:52
*** racedo has joined #openstack-ironic08:55
*** aolwas has joined #openstack-ironic08:57
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-ironic09:02
vmud213Morning ironic09:05
milanmorning vmud213! :)09:06
vmud213milan o/09:06
*** amoralej|off is now known as amoralej09:07
*** athomas has joined #openstack-ironic09:08
*** e0ne has quit IRC09:08
*** e0ne has joined #openstack-ironic09:08
*** lucas-afk is now known as lucasagomes09:09
lucasagomesmorning all09:09
*** priteau has joined #openstack-ironic09:12
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: [POC] multiple PXE white--blacklist backends  https://review.openstack.org/43960309:13
milanmorning lucasagomes! :)09:13
*** hshiina has joined #openstack-ironic09:18
*** alexpilotti has quit IRC09:19
*** alexpilotti has joined #openstack-ironic09:20
openstackgerritHironori Shiina proposed openstack/ironic master: DNM/POC Workaround for boot from Cinder volumae  https://review.openstack.org/44194409:22
openstackgerritHironori Shiina proposed openstack/ironic master: DNM/POC Workaround for boot from Cinder volume  https://review.openstack.org/44194409:23
*** srart has quit IRC09:24
*** g3ek has quit IRC09:28
*** milan has quit IRC09:35
*** g3ek has joined #openstack-ironic09:38
vdrokmorning vmud213 and lucasagomes !09:45
lucasagomeso/09:45
vmud213morning vdrok,lucasagomes09:45
*** mvpnitesh has joined #openstack-ironic09:58
*** e0ne has quit IRC09:58
openstackgerritGalyna Zholtkevych proposed openstack/ironic-python-agent master: Correct failure message output when downloading  https://review.openstack.org/44107210:01
openstackgerritVasyl Saienko proposed openstack/ironic master: Fix updating node.driver to classic  https://review.openstack.org/43902610:02
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs master: Follow-up patch for the Redfish spec  https://review.openstack.org/44317710:03
*** MattMan has quit IRC10:03
*** MattMan has joined #openstack-ironic10:04
openstackgerritAparna proposed openstack/proliantutils master: Support to perform erase on SSD drives  https://review.openstack.org/44354210:06
*** SerenaFeng has quit IRC10:06
*** vills has joined #openstack-ironic10:06
openstackgerritVasyl Saienko proposed openstack/ironic master: Fix updating node.driver to classic  https://review.openstack.org/43902610:08
*** SerenaFeng has joined #openstack-ironic10:08
openstackgerritTao Li proposed openstack/ironic master: Address a shell sytax mistake  https://review.openstack.org/44354610:09
*** sayantan_ has quit IRC10:09
*** venki1 has joined #openstack-ironic10:10
*** hshiina has quit IRC10:10
*** tovin07 has quit IRC10:11
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib master: Add a debug log when comparing root device hints  https://review.openstack.org/44315910:13
*** tuanluong has quit IRC10:18
*** milan has joined #openstack-ironic10:32
*** udesale has quit IRC10:33
*** sambetts|afk is now known as sambetts10:35
*** dtantsur|afk is now known as dtantsur10:39
dtantsurmorning ironic10:39
vdrokmorning dtantsur !10:40
dtantsurTheJulia, hey, no worries. thanks for creating it!10:40
vsaienk0morning Ironic'ers!10:41
dtantsurhey vdrok, vsaienk0, lucasagomes, vmud213, milan10:41
*** Lambert has quit IRC10:42
vmud213morning dtantsur10:42
*** Lambert has joined #openstack-ironic10:42
sambettsJayF, aNuposic: your trying to install glean into tinyipa? I guess there would be a couple of steps, first getting the python package installed, and that is done in two steps, first a build here https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/tinyipa/build-tinyipa.sh#L101 and then an install here10:42
lucasagomeso/10:42
sambettshttps://github.com/openstack/ironic-python-agent/blob/master/imagebuild/tinyipa/finalise-tinyipa.sh#L133 and then you have to actually run glean on start up, there are a couple of ways we could do this, the simplest is probably to run it before we run IPA here: https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/tinyipa/build_files/bootlocal.sh10:42
dtantsurTheJulia, sambetts, vsaienk0, vdrok, yuriyz_, your votes missing on https://review.openstack.org/#/c/439710/ ;)10:44
patchbotpatch 439710 - ironic-specs - Add priorities for the Pike cycle10:44
vdrokoh, yup, looking10:44
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib master: Add a debug log when comparing root device hints  https://review.openstack.org/44315910:46
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib master: Improve logs for the root device hints mechanism  https://review.openstack.org/44315910:46
milanmorning dtantsur, sambetts! :)10:47
milanand morning vsaienk0! :)10:48
*** Lambert has quit IRC10:49
*** Lambert has joined #openstack-ironic10:49
milandtantsur, sambetts, guys, w/r https://review.openstack.org/#/c/439862/1/specs/multiple-pxe-filtering-backends.rst@105 I'm wondering how to deprecate stuff; IIRC we want to a) introduce the driver b) deprecate current firewall settings c) deprecate iptables (firewall) altogether10:50
patchbotpatch 439862 - ironic-inspector-specs - Multiple PXE filtering backends10:50
milanso I'm wondering how to best address this10:51
milandtantsur, sambetts let10:51
milan's have the pxe_driver default firewall or noop w/ firewall.manage_firewall overriding it?10:52
*** derekh has joined #openstack-ironic10:52
dtantsur1. introduce drivers; 2. set iptables as a default; 3. issue a warning about changing defaults; 4. change defaults; 5. deprecate iptables driver10:52
dtantsuror am I missing something?10:52
sambettsmilan: just posted a comment about the default10:52
sambettsIMO default should be iptables, and only the iptables filter should care about the manage_firewall option10:53
sambettsso that if someone changes the pxe_filter to something other than iptables manage_firewall does nothing10:53
sambettsdtantsur: sounds right to me10:54
milandtantsur, sambetts, ack, thanks, lemme update the spec10:54
*** stendulker has quit IRC10:54
sambetts:) thanks!"10:54
milansambetts, ;) qq: is firewall an OK driver name? cause we've got the firewall everywhere already and I'm lazy replacing it w/ iptables O:-)10:56
openstackgerritOksana Voshchana proposed openstack/ironic master: Add wsgi handling to ironic-api in devstack  https://review.openstack.org/43085110:56
sambetts"firewall" for the iptables driver?10:56
milansambetts, yeah as a driver name10:57
sambettsmight get a little confusing if someone writes a driver for similar driver but for a different firewall system10:57
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729910:57
*** vills_ has joined #openstack-ironic10:58
sambettsmilan: what would you need to replace?10:58
milansambetts, well, maybe not much :D /me checks10:58
milansambetts, the module name is firewall and the config options are under firewall section10:59
* milan wonders if it's not confusing to configure an iptables driver thru the firewall config options11:00
milanw/r another firewall driver implementation, if someone writes pf_driver they don't have to prefix it w/ firewall maybe?11:01
sambettsmaybe pxe_filter = firewall_iptables, and pxe_filter = firewall_pf?11:02
sambetts:/11:02
dtantsurI'd go with just "iptables"11:03
sambettsyeah I prefer that, maybe we should also migrate the iptables specific configuration option to an [iptables] config section11:03
*** amarao has quit IRC11:03
sambetts?11:03
sambettsobviously we'd need deprecation there11:03
dtantsurwe can, it's not too hard thanks to oslo.config11:03
sambetts+10011:04
milansambetts, we're deprecating it, I'd rather avoid it11:04
milancause we're deprecating the whole thing or?11:04
*** amarao has joined #openstack-ironic11:04
milandtantsur, sambetts, in case of the rename, the spec should mention that too, correct?11:05
dtantsurit should, yeah, though it's not something I care about wrt this spec11:05
sambettswe don't have firewall drivers right now so what is there to rename from?11:06
*** zhangjl has left #openstack-ironic11:06
milansambetts, the config section in the deployers impact paragraph11:07
*** mgould|afk is now known as mgould11:07
mgouldmorning Ironic!11:07
milanmorning mgould! :)11:07
sambettsmilan: oh yeah config option renames totally need mentioning11:07
mgouldmorning milan dtantsur sambetts vdrok vsaienk0 vmud213 lucasagomes11:08
milanalso, maybe I'm over engineering this ;) but should we have both supported for some period?11:08
milansambetts, ^11:08
sambettsthe config options? Yes, and oslo.config has the deprecation options to deal with that for us11:09
openstackgerritOksana Voshchana proposed openstack/ironic master: Add wsgi handling to ironic-api in devstack  https://review.openstack.org/43085111:09
milansambetts, dtantsur, ack, thanks! Let's do it then: iptables as the name for the driver; rename the firewall config option to iptables and deprecate stuff as dtantsur mentioned above11:10
dtantsuryep11:11
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729911:11
*** athomas has quit IRC11:13
*** rbartal has quit IRC11:18
*** rbartal has joined #openstack-ironic11:18
*** athomas has joined #openstack-ironic11:20
*** hshiina has joined #openstack-ironic11:21
vmud213morning mgould11:24
lucasagomesmorning mgould11:27
*** vishwanathj has quit IRC11:29
*** vishwanathj has joined #openstack-ironic11:29
*** Guest25991 has quit IRC11:30
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib master: Add .eggs to the .gitignore file  https://review.openstack.org/44359211:32
*** galyna has left #openstack-ironic11:32
*** aarefiev_afk has quit IRC11:32
*** vsaienk0 has quit IRC11:35
*** dnikishov has joined #openstack-ironic11:37
*** dnikishov is now known as Guest512111:37
*** e0ne has joined #openstack-ironic11:37
*** SerenaFeng has quit IRC11:46
*** yuriyz_ has quit IRC11:46
*** athomas has quit IRC11:51
*** Nisha_Agarwal has joined #openstack-ironic11:56
*** Nisha_ has joined #openstack-ironic12:00
*** Nisha_Agarwal has quit IRC12:00
*** moshele has quit IRC12:01
*** athomas has joined #openstack-ironic12:01
*** moshele has joined #openstack-ironic12:01
*** Nisha_ has quit IRC12:01
*** Nisha_ has joined #openstack-ironic12:02
*** SerenaFeng has joined #openstack-ironic12:05
*** tuanluong has joined #openstack-ironic12:06
*** amotoki_ has quit IRC12:07
*** zhurong has quit IRC12:08
*** amotoki has joined #openstack-ironic12:08
*** aarefiev_afk has joined #openstack-ironic12:12
*** lucasagomes is now known as lucas-hungry12:12
*** amotoki has quit IRC12:13
vdrokgood morning mgould12:16
*** ricardoas has joined #openstack-ironic12:17
*** rbudden has joined #openstack-ironic12:17
*** g3ek has quit IRC12:21
*** tuanluong has quit IRC12:21
*** amarao has quit IRC12:26
*** amotoki has joined #openstack-ironic12:26
openstackgerritHao Li proposed openstack/ironic-lib master: Retry when execute partprobe cmd failed  https://review.openstack.org/44360412:29
*** g3ek has joined #openstack-ironic12:30
*** Nisha_ has quit IRC12:31
*** Nisha_Agarwal has joined #openstack-ironic12:31
*** Lambert has quit IRC12:33
*** SerenaFeng has quit IRC12:38
*** moshele has quit IRC12:38
*** moshele has joined #openstack-ironic12:38
*** e0ne_ has joined #openstack-ironic12:39
*** phuongnh has quit IRC12:40
*** vills__ has joined #openstack-ironic12:40
*** vills_ has quit IRC12:41
*** e0ne has quit IRC12:42
*** phuongnh has joined #openstack-ironic12:43
*** lucas-hungry is now known as lucasagomes12:48
*** Nisha_ has joined #openstack-ironic12:49
*** Nisha_Agarwal has quit IRC12:49
*** amarao has joined #openstack-ironic12:53
*** vills has quit IRC12:54
*** vills__ has quit IRC12:54
*** phuongnh has quit IRC12:54
*** jkilpatr has joined #openstack-ironic12:56
*** sasha21 has joined #openstack-ironic12:59
*** mvpnitesh has quit IRC12:59
*** mvpnitesh has joined #openstack-ironic12:59
*** Nisha_ has quit IRC13:02
*** jaosorior is now known as jaosorior_brb13:03
openstackgerritAparna proposed openstack/proliantutils master: Performs IPA erase_devices after Sanitize disk erase  https://review.openstack.org/43645013:03
*** jkilpatr has quit IRC13:04
*** jkilpatr has joined #openstack-ironic13:07
*** aparnav has quit IRC13:10
*** mvpnitesh has quit IRC13:10
*** mvpnitesh has joined #openstack-ironic13:11
*** jkilpatr has quit IRC13:12
*** vmud213 has quit IRC13:14
*** bfournie has joined #openstack-ironic13:15
*** jkilpatr has joined #openstack-ironic13:16
*** frontrunner has joined #openstack-ironic13:20
*** vills has joined #openstack-ironic13:20
*** jkilpatr has quit IRC13:22
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362813:24
openstackgerritMilan Kováčik proposed openstack/ironic-inspector-specs master: Multiple PXE filtering backends  https://review.openstack.org/43986213:24
*** vsaienko has joined #openstack-ironic13:32
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362813:32
*** chlong_ has joined #openstack-ironic13:33
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362813:39
*** dtantsur is now known as dtantsur|bbl13:40
*** vills_ has joined #openstack-ironic13:44
*** jjohnson2 has joined #openstack-ironic13:44
sambettslucasagomes: do we document anywhere the system package requirements for the pxe_ipmitool driver???13:45
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362813:45
openstackgerritVladyslav Drok proposed openstack/ironic master: Add functional api tests for node resource class  https://review.openstack.org/43796313:45
sambettslucasagomes: and also do you remember if the agent_ipmitool driver can work without swift??13:45
*** Goneri has joined #openstack-ironic13:45
*** goldyfruit has joined #openstack-ironic13:45
*** cdearborn has joined #openstack-ironic13:46
lucasagomessambetts, hmm system package requirements like the version of ipmitool it supports ? Not sure we document it, but there's some stuff done in the driver's code to find out the features ipmitool supports13:46
lucasagomeslike dual bridge and things like that, there's a check13:47
sambettslucasagomes: things like the requirements on the kernel modules / packages to be able to do iscsi?13:47
lucasagomessambetts, agent_ipmitool, AFAICT, you could use it in the standlone way with your own HTTP server13:47
sambettsyeah, I'm was hoping to use it with glance without swift13:48
lucasagomessambetts, yeah I don't think we document that, for now, there's a fallback if linux-io is not supported to tgtd13:48
sambettslucasagomes: thats in IPA right? I'm talking about on the conductor13:48
lucasagomessambetts, will take a look in the code 1sec, but off the top of my head, if you have glance you will need swift13:48
lucasagomesto create the tempurl13:48
*** vills_ has quit IRC13:49
lucasagomessambetts, oh right, yeah that's IPA13:49
lucasagomesbut the same, we don't document anything for the conductor either... Not that I know of13:49
* lucasagomes looks at the docs13:49
sambetts:/ hmmm13:50
pas-hasambetts: agent_* drivers do not work w/o swift or radosgw13:52
sambetts:'(13:52
pas-hathey we need tempurls13:52
pas-hahttps://review.openstack.org/#/c/379422/13:53
patchbotpatch 379422 - ironic - [install-guide] add section on Glance+Swift config13:53
pas-hasambetts: ^13:53
*** mvpnitesh has quit IRC13:53
*** skazi has quit IRC13:53
pas-hawe need glance images stored in swift so we can construct a tempurl for those. otherwise we need some way to pass a keystone token to IPA so that it could download the image directly thru glance API13:54
*** jcoufal has joined #openstack-ironic13:54
*** aparnav has joined #openstack-ironic13:56
sambetts:( shame we can't serve them from the conductor13:56
*** SerenaFeng has joined #openstack-ironic13:57
*** skazi has joined #openstack-ironic13:58
*** SerenaFe_ has joined #openstack-ironic13:59
*** SerenaFe_ has quit IRC14:00
pas-haimo that was one of the key points to agent drivers at all so we do not have to...14:00
*** SerenaFeng has quit IRC14:00
*** vills_ has joined #openstack-ironic14:01
pas-haalternatively, there is this for ansible-deploy :) https://github.com/pshchelo/ironic-staging-drivers/commit/1efdc3520fc0b3fafe35d32fb2b371793215d4d514:01
*** SerenaFeng has joined #openstack-ironic14:02
pas-hawhich passes in a token (kind of securely as we communicate with the node over encrypted SSH channel)14:02
milanmorning pas-ha! :)14:03
pas-hamorning milan :)14:03
milandtantsur|bbl, sambetts https://review.openstack.org/#/c/439862/2/specs/multiple-pxe-filtering-backends.rst@102 makes sense?14:03
patchbotpatch 439862 - ironic-inspector-specs - Multiple PXE filtering backends14:03
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent master: Change the _wait_for_disks behavior to wait for a specific disk  https://review.openstack.org/44364914:04
*** SerenaFeng has quit IRC14:05
*** udesale has joined #openstack-ironic14:06
*** SerenaFeng has joined #openstack-ironic14:07
*** rloo has joined #openstack-ironic14:08
*** amotoki has quit IRC14:08
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Serve boot configs from Ironic API  https://review.openstack.org/36943814:15
*** jaganathan has quit IRC14:16
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: DNM test enabled dynamic iPXE  https://review.openstack.org/39194314:17
TheJuliaGood morning everyone14:22
rlooGood morning TheJulia, pas-ha, and everyone else14:26
milanmorning rloo, TheJulia! :)14:28
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Don't 'enable_plugin ironic' in grenade settings  https://review.openstack.org/44215414:28
*** vills_ has quit IRC14:30
mariojvgood morning ironic14:32
mariojvaNuposic: feel free to propose that topic. i only read ones that have been proposed 24 hours in advance there personally and prefer reading about upstream priorities :)14:32
mariojvbut i already have context on your patch. others may read it since it's early in the day, too14:33
*** vikrant has quit IRC14:34
*** links has quit IRC14:35
*** baoli has joined #openstack-ironic14:39
*** venki1 has left #openstack-ironic14:39
*** venki1 has joined #openstack-ironic14:41
*** venki1 has left #openstack-ironic14:41
*** venki1 has joined #openstack-ironic14:41
*** jaosorior_brb is now known as jaosorior14:43
*** strigazi has quit IRC14:44
TheJuliagood morning rloo, milan14:44
TheJuliagood morning mariojv14:44
milanmorning mariojv! :)14:44
mariojvhi TheJulia milan everyone else14:46
*** aparnav has quit IRC14:47
rloohi milan, morning mariojv!14:48
mariojvhi rloo14:48
*** crushil has joined #openstack-ironic14:49
*** Goneri has quit IRC14:51
rlooTheJulia, mariojv, dtantsur|bbl, anyone. Do you recall what we decided on Friday at PTG?14:51
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent master: Change the _wait_for_disks behavior to wait for a specific disk  https://review.openstack.org/44364914:51
mariojvrloo: on which topic? we didn't talk about a lot then14:52
mariojvi remember us being more OK with informal online events and stuff14:52
rlooTheJulia, mariojv, dtantsur|bbl, lucasagomes. I was just reading dmitry's ptg report and noticed that it didn't mention anything from friday.14:52
rloomariojv: i don't recall, that's why i'm asking :)14:52
lucasagomesrloo, morning o/14:52
rloomariojv: there was something about bug triaging, and we had some discussion about reviews or something like that?14:52
mariojvah, yes14:52
mariojvbug triaging will be done by volunteers each week14:53
rloomariojv: just wondering if there was anything to report to the community, that's all14:53
mariojvinstead of just 1 or 2 people responsible for hte full cycle14:53
mariojvyeah, thx for bringing it up14:53
*** xnox has joined #openstack-ironic14:53
rloomariojv: also... do you or anyone remember about ipminative. i thought we had decided to kick it out but maybe i was wrong. lucasagomes, do you remember?14:53
xnoxIt would seem that iRST is broken completely in both Debian and Ubuntu14:53
xnoxjudring by all of the Teeth udev rules overrides14:54
xnoxin the base-image-template repository14:54
mariojvi do not remember14:54
lucasagomesrloo, hmm I remember that yeah, we decided to remove it from tree14:54
lucasagomesrloo, actually14:54
lucasagomesrloo, we would ask ppl in IBM for 3rd party CI on it because, since virtualbmc uses pyghmi it would make little sense to test pyghmi against pyghmi14:55
lucasagomesit wouldn't be a real test14:55
lucasagomesand if there's no interest in 3rd part ci it would then be removed from tree14:55
rloolucasagomes: ahhh, maybe that was it.14:55
lucasagomesthat's what I remember14:55
lucasagomesJayF, ^ please correct me if I'm wrong14:55
openstackgerritOksana Voshchana proposed openstack/ironic master: Add wsgi handling to ironic-api in devstack  https://review.openstack.org/43085114:55
lucasagomesI think you brought up the topic about not testing pyghmi against pyghmi itself14:55
rloolucasagomes: we should make sure that someone does that then...14:56
lucasagomesrloo, right14:56
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729914:56
rloodtantsur|bbl: ^^ about pyghmi testing14:56
lucasagomesrloo,  idk who to ping tho14:56
rloolucasagomes: me neither so i pinged the ptl ;)14:56
*** jaganathan has joined #openstack-ironic15:01
*** strigazi has joined #openstack-ironic15:04
sambettsHas anyone ever seen a Operation not permitted error on the DD command that pxe_ipmitool tries to run??/15:10
vdrokmorning TheJulia mariojv and rloo15:12
*** jjohnson2_ has joined #openstack-ironic15:12
mariojvhi vdrok15:12
lucasagomesrloo, lol15:13
rloolucasagomes: /me can guess :)15:13
*** jjohnson2 has quit IRC15:13
*** jaganathan has quit IRC15:13
*** hshiina has quit IRC15:13
*** chlong_ has quit IRC15:17
*** bcornec2 has joined #openstack-ironic15:18
mjturekTheJulia: was a time decided on for the BFV meeting?? Might've missed if it was announced on ML15:20
*** SerenaFeng has quit IRC15:21
TheJuliamjturek: the consensus of availability centered around 1400 UTC, that is what I put the meeting in for.  Which is in... 39 minutes15:21
TheJuliasambetts: I have seen it when the iscsi connection has been interrupted.15:22
TheJuliaCrazy silly question... Why do we not support alternatively streaming through something like netcat off of the conductor.15:23
*** Syed__ has joined #openstack-ironic15:23
TheJuliarloo: all I remember of note that had real community wide [insert word such as importance] was bug triaging.  There were some various smaller discussions, but community wide I'm not sure they are really relevant15:26
* TheJulia needs to pack a blanket on her next business trip15:26
rlooTheJulia: Ok, maybe that was all. dtantsur|bbl has already 'instituted' that in the weekly meetings, so probably no need to mention in email list.15:27
rlooTheJulia: is it to keep you warm? where'd you go, Alaska?15:27
TheJuliaKeep warm and for comfort in general.  I feel super run down from this trip15:28
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: [POC] multiple PXE white--blacklist backends  https://review.openstack.org/43960315:28
rlooTheJulia: :-(15:28
TheJuliaI'm in San Francisco, so it is also a little cold in the convention space where I've been at during the days.  It is unheated as far as I can tell. :(15:29
mariojvTheJulia: that'd be 1600 UTC15:29
mariojvin 31 minutes now15:29
TheJuliamariojv: yes, that!15:29
mariojvah ok15:29
* TheJulia ponders just going back to sleep15:30
TheJuliaOf course, I've reached the giggling phase of exhaustion!15:30
* rloo thinks the BFV meeting might be loads of fun with TheJulia there15:31
mjturekTheJulia okay awesome! thanks15:31
*** baha has joined #openstack-ironic15:32
*** rbartal has quit IRC15:36
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362815:39
JayFlucasagomes: rloo: That reflects my rememberance: ipminative is broken, we know this. Testing it against itself is unlikely to suss out problems. We should remove it from tree.15:39
rlooJayF: so we weren't going to ask for some 3rd party CI for that?15:41
*** moshele has quit IRC15:42
*** sayantan_ has joined #openstack-ironic15:42
JayFrloo: We can always ask, but bluntly, I feel a bit of urgency around removing it, because we have had more than a couple people use that driver and get burned because it doesn't work well15:43
JayFrloo: Just seems to be one big blemish when all our other shipped drivers work well :/15:43
rlooJayF: ok. would you take responsibility for that? eg, check with dtantsur|bbl, has it been deprecated yet, do we need to send out email, ..., do the deed, etc. i don't recall the process.15:44
rlooJayF: probably open a bug for it too.15:44
JayFHmm. Yeah, I can own that.15:44
JayFI'll file a big now15:44
JayF*bug15:44
rlooJayF: thx.15:44
*** ccamacho has quit IRC15:45
rlooJayF: someone should inform the pyghmi people too...15:45
mgouldTheJulia: I've been trying to get an ironic-ui development environment up15:47
openstackgerritVasyl Saienko proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job  https://review.openstack.org/43695715:47
mgouldam I right in thinking that Horizon really really wants to be installed on a machine that already has the rest of OpenStack running?15:47
JayFrloo: https://bugs.launchpad.net/ironic/+bug/167153215:47
openstackLaunchpad bug 1671532 in Ironic "ipminative driver does not meet project standards" [Undecided,New]15:47
rloothx JayF. So I guess we need to first confirm that no one is going to do 3rd party CI for it?15:49
*** ccamacho has joined #openstack-ironic15:49
robcresswellmgould: No, not at all15:49
ppielamgould: I run Horizon on my laptop that points to a devstack vm15:49
robcresswellmgould: Just needs a keystone endpoint15:49
JayFrloo: I'll send an email out to the list calling for "saviours" :)15:49
rlooJayF: :)15:50
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362815:52
ppielamgould: You should be able to set the OPENSTACK_HOST in horizon/openstack_dashboard/local/local_settings.py15:53
mgouldppiela: yeah, I did that15:53
mgouldit's complaining that ceilometerclient isn't installed15:54
robcresswellppiela: Yeah, we use the pythonclients for interaction. Although that must be Newton or earlier.15:55
openstackgerritGalyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers  https://review.openstack.org/38199115:55
ppielamgould: How are you starting Horizon15:55
robcresswellppiela: What are you trying to do? Dev work?15:55
mgouldrobcresswell: yes15:55
mgouldppiela: tox -e runserver15:56
mgouldI'm trying to follow the instructions at https://docs.openstack.org/developer/horizon/quickstart.html15:56
ppielarobcresswell: I am developing the Ironic-UI plugin15:56
mgouldrobcresswell: there was a stale .pyc file in my tree, I've deleted it15:56
mgouldwoo!15:57
*** vsaienko has quit IRC15:57
mgouldI can now see a login page15:57
robcresswellmgould: You can use `make clean` to get rid of those.15:57
robcresswellppiela: Sorry, I meant to ping mgould with that question, got my names mixed up :)15:57
ppielarobcresswell: np15:58
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362815:58
*** rcernin has quit IRC15:58
* mgould notes that the Makefile claims "make test" will run tests, but it doesn't15:59
*** jaosorior has quit IRC15:59
robcresswellmgould: Yeah, its pretty old, but clean is sometimes useful15:59
robcresswellTheJulia: I've got the ironic-ui meeting on my calendar btw, so should be able to help out there16:00
* mgould opens a review to remove it16:00
joannamorning rloo, JayF, mgould, TheJulia, mariojv, mjturek :)16:02
JayFo/16:02
*** Nisha_Agarwal has joined #openstack-ironic16:02
*** mjturek has quit IRC16:02
robcresswellmgould: Congrats, your first merge in Horizon?16:03
*** mjturek has joined #openstack-ironic16:03
mjturekgm joanna :)16:03
mat128JayF: "and cannot upstream as virtualbmc is built using pyghmi"16:03
mat128what do you mean?16:03
JayFmat128: We talked about this some at the PTG; mainly the assumption is that testing pyghmi against pyghmi (via vbmc) is not going to be a valid test16:04
rloomorning joanna!16:05
mgouldrobcresswell: I believe so :-)16:06
*** Goneri has joined #openstack-ironic16:06
mgouldwait, it hasn't merged yet16:06
mgouldah, but you've +2ed it - thanks!16:07
robcresswellmgould: :)16:07
mgouldsee also https://review.openstack.org/#/c/443716/, which fixes another problem I ran into16:07
patchbotpatch 443716 - horizon - Clarify message if key file has wrong permissions16:07
mgouldor at least makes it easier to fix manually :-)16:07
*** links has joined #openstack-ironic16:07
rloomariojv: wrt your weekly review party, i was wondering if it would be good to put a link to it/your etherpad, either in our main etherpad, or even in our meetings wiki.16:12
mariojvrloo: great idea! I didn't want to give the idea that this is an official thing, though16:13
mariojvbut I think it's OK if we put a disclaimer saying that ^16:13
rloomariojv: yeah, that's why i'm not sure about putting in our meetings wiki although i am fine if we put it down as unofficial.16:13
mariojvdtantsur|bbl: would you be ok with that ^ ? (putting info about my unofficial review party on etherpad/wiki)16:13
mariojvs/my//16:13
openstackgerritMerged openstack/ironic-lib master: Python 3.5 is added  https://review.openstack.org/44342116:14
mariojvalso, i will likely have to move the meeting with daylight savings16:15
rloodtantsur|bbl: when you have some time, I updated the OSC driver-properties spec. Was wondering if you or jroll had any thoughts about how these cmds would work with dynamic drivers, in particular hardware types + non-default interfaces. See my ramblings from L139: https://review.openstack.org/#/c/439907/3/specs/approved/osc-driver-property-commands.rst16:17
patchbotpatch 439907 - ironic-specs - OSC commands for ironic driver-related commands16:17
*** jjohnson2_ has quit IRC16:18
rloodtantsur|bbl: I didn't see anything in the driver comp spec about it. But we probably need to update our REST API etc to address it.16:18
*** jaosorior has joined #openstack-ironic16:19
*** rpioso has joined #openstack-ironic16:22
TheJuliarobcresswell: awesome! :)16:24
soliosgmgould: your patch to clarify the key file permissions (horizon) is useful; i've cr+116:24
mgouldsoliosg: thanks!16:24
mgouldI ran into that problem trying to get my dev env up, and that was the easiest way to debug it...16:25
soliosgmgould: ack16:25
NobodyCamGood Morning Ironic'ers :)16:27
openstackgerritMichael Turek proposed openstack/ironic master: OpenBMC Driver  https://review.openstack.org/40383516:27
openstackgerritMerged openstack/bifrost master: playbooks: roles: bifrost-ironic-install: Verify checksum for IPA files  https://review.openstack.org/43993816:33
openstackgerritMerged openstack/bifrost master: playbooks: roles: bifrost-ironic-install: Retry failed PXE downloads  https://review.openstack.org/44222416:33
openstackgerritMerged openstack/bifrost master: Use JSON baremetal data in testing  https://review.openstack.org/43089816:33
openstackgerritMerged openstack/bifrost master: Use virtualbmc for virtual BM nodes  https://review.openstack.org/42737616:33
*** jjohnson2 has joined #openstack-ironic16:35
*** vills has quit IRC16:35
*** vills has joined #openstack-ironic16:36
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362816:38
mat128JayF: got it, makes sense16:38
*** bcornec2 has quit IRC16:38
openstackgerritNisha Agarwal proposed openstack/proliantutils master: Vendor gpu count  https://review.openstack.org/42448916:40
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Allow to delete nodes in states in which node deletion is allowed  https://review.openstack.org/44374416:41
*** srart has joined #openstack-ironic16:41
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Allow to delete nodes in states in which node deletion is allowed  https://review.openstack.org/44374416:43
mgouldmorning NobodyCam16:44
NobodyCamMorning mgould :)16:44
aNuposicGm mat128: JayF: I pushed glean inside tinyipa https://review.openstack.org/#/c/443341/ but when i am trying to run glean inside tinyipa i am getting this error http://paste.openstack.org/show/602031/16:47
patchbotpatch 443341 - ironic-python-agent - Add glean package in requirements.txt16:47
mat128aNuposic: good morning. What are you trying to achieve with glean inside TinyIPA?16:49
pas-haaNuposic: that's because in ipa by default all Python stuff is compiled to *.pyo and original *.py are removed16:49
mat128Is it related to the rescue mode?16:49
*** udesale has quit IRC16:49
aNuposicpass-ha JayF mat128: i passed BUILD_AND_INSTALL_TINYIPA=true flag and so tinyipa is built with python16:49
JayFmat128: yep, this is all for testing with rescue mode16:49
mat128aNuposic: I don't see any error besides the global-requirements job failing16:49
aNuposicmat128: yes related to rescue patch to support static network configuration for tinyipa16:50
JayFmat128: look at the paste error -- it builds in but won't run16:50
mat128Oh16:50
* mat128 completely missed the paste link :)16:50
pas-hatry this one too export PYOPTIMIZE_TINYIPA=false16:50
mat128+1 ^16:50
mat128if it works with it, we can find exactly what is missing16:50
aNuposicpass-ha but that will just give me .py files correct instead of .pyo? But i don't see that flag will add some packages?16:51
pas-haalso check what I'm doing when rebuilding tinyipa for ansible-deploy - https://git.openstack.org/cgit/openstack/ironic-staging-drivers/tree/imagebuild/tinyipa-ansible/rebuild-tinyipa.sh#n15416:51
pas-hano, that flag just won't compile all the stuff to pyo and remove originals16:52
pas-hathe thing is, the ipa itself is being run from the startup shell script where PYTHONOPTIMIZE is set to 116:53
pas-haso depending on how you start/use glean, you'd need that too16:54
pas-haif original *.py files are removed16:54
pas-haaNuposic: https://github.com/openstack/ironic-python-agent/tree/master/imagebuild/tinyipa/build_files16:56
aNuposicpass-ha: I see, okay let me build the agent with that flag set16:56
pas-hathat ^^ is where you add packages that will be backed into tinyipa16:56
pas-hameaning the system ones16:57
pas-hadoes glean requires anything to be c-compiled while installing from pypi?16:57
pas-hado you need packages with some dev headers for it?16:58
*** racedo has quit IRC16:58
*** racedo has joined #openstack-ironic16:58
joannamorning NobodyCam, pas-ha16:59
pas-hamorning joanna16:59
*** racedo has quit IRC17:00
aNuposicpass-ha: I don't see any requirements or dependencies specified for glean17:00
pas-haaNuposic: checked glean, seems not to require anything, is already a pure python wheel on PyPI17:00
pas-hayep17:00
NobodyCammorning joanna :)17:00
aNuposicpass-ha: so if i set the flag it will basically precompile python https://github.com/openstack/ironic-python-agent/blob/a7808d0f4db4b1e76901757d07da027344c92dc6/imagebuild/tinyipa/finalise-tinyipa.sh#L156?17:02
*** gfhellma has joined #openstack-ironic17:03
pas-haaNuposic: set it to false (lower-case) which is a command that always evaluates to 'False' (/bin/false)17:04
pas-hahttps://github.com/openstack/ironic-python-agent/blob/a7808d0f4db4b1e76901757d07da027344c92dc6/imagebuild/tinyipa/README.rst#deoptimizing-the-image17:04
openstackgerritFellype Cavalcante proposed openstack/ironic master: Validation before perform node deallocation  https://review.openstack.org/38601417:05
*** e0ne_ has quit IRC17:05
aNuposicpass-ha: okay, let me try that and see17:05
aNuposicpass-ha: may be that does the magic :)(17:06
aNuposic:)17:06
sambettsmat128, JayF, aNuposic: did you see my messages from this morning in the backscroll??17:07
*** baha has quit IRC17:07
aNuposicsambetts, nope, i missed let me check17:09
openstackgerritMerged openstack/ironic-lib master: Add .eggs to the .gitignore file  https://review.openstack.org/44359217:10
JayFsambetts: looking17:10
JayFsambetts: yeah, I see it -- we just want the command line in, so we can use it to reconfigure networks in rescue17:11
JayFsambetts: we got it installed via requirements.txt, but then it wouldn't run -- I think aNuposic is going to try mat128's suggestion of disabling the pyoptimize stuff, and if that works, see what we need to do to keep it working17:11
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362817:11
*** dsneddon has joined #openstack-ironic17:11
*** vills has quit IRC17:12
*** Madasi has joined #openstack-ironic17:14
aNuposicsambetts, JayF : yes that's what i thought and tried yesterday by setting BUILD_AND_INSTALL_TINYIPA=true17:15
aNuposicyup let me try as pas-ha, mat128 suggested to disable pyoptimize stuff17:16
openstackgerritMiles Gould proposed openstack/ironic-ui master: Fix docstring for node_validate  https://review.openstack.org/44376017:18
*** vsaienko has joined #openstack-ironic17:20
*** jaganathan has joined #openstack-ironic17:22
openstackgerritMiles Gould proposed openstack/ironic-ui master: Fix typos in docstrings  https://review.openstack.org/44376217:22
*** Nisha_Agarwal has quit IRC17:23
*** Nisha_Agarwal has joined #openstack-ironic17:23
*** moshele has joined #openstack-ironic17:24
*** ccamacho has quit IRC17:25
*** jaosorior has quit IRC17:29
*** athomas has quit IRC17:30
*** amarao has quit IRC17:30
openstackgerritMiles Gould proposed openstack/ironic-ui master: Fix typos in docstrings  https://review.openstack.org/44376217:30
*** amarao has joined #openstack-ironic17:30
openstackgerritMiles Gould proposed openstack/ironic-ui master: Fix docstrings for node validation  https://review.openstack.org/44376017:30
* pas-ha signs off, have a good evening / rest of the day all :)17:31
mgouldpas-ha: good night!17:32
*** mtanino has joined #openstack-ironic17:32
*** venki has quit IRC17:36
joannapas-ha: night!17:36
mgouldso, Horizon: we have a JavaScript front-end that makes a bunch of AJAX calls to a Django app which makes a bunch of AJAX calls to the OpenStack services, right?17:37
mgouldthat seems... redundant17:37
openstackgerritMilan Kováčik proposed openstack/ironic-inspector-specs master: Split service into API and Conductor  https://review.openstack.org/43551717:38
* mgould wonders if it has some security benefit17:38
robcresswellmgould: Sort of17:38
robcresswellProxying is a pretty common UI workflow17:38
mgouldsure17:39
robcresswellI'd say its much more common than CORS17:39
*** yolanda has joined #openstack-ironic17:39
robcresswellwhich is the alternative17:39
* mgould googles CORS17:39
mgouldright, makes sense17:39
robcresswellIts security++ supposedly, but it also lets us do data cleaning and works without extra config17:39
robcresswellCORS requires extra config in every service :)17:40
mgouldI guess all Web 2.0 CRUD apps basically follow this pattern :-)17:40
mgouldaaargh17:40
mgouldwe already have a lot of config, so I can see the point of that :-)17:40
robcresswellTripleO uses CORS17:40
robcresswellmgould: Ironic already has the cors config baked in (I think everyone does, via the oslo cors middleware)17:40
mgouldah, OK17:40
robcresswellBut its still a pain to enable it for every service17:40
*** dtantsur|bbl is now known as dtantsur17:41
robcresswellMy general experience is that backend engineers do not like taking requests from front end people17:41
robcresswell:)17:41
robcresswellBut yeah, the proxy part in our case is 9/10 times just a wrapper that calls the pythonclient17:41
* mgould nods17:42
*** tesseract has quit IRC17:42
dtantsurrloo, lucasagomes, re pyghmi: we agreed to call for volunteers to maintain it (with all the consequences). if we don't get them, we kick it out.17:42
mgouldrobcresswell: it's a common point of friction, certainly17:42
robcresswellSometimes it does some handling of data, like replacing ':' with '__' because it messes up django templates iirc.17:42
rloodtantsur: thx, JayF ^^, ask for volunteers, etc, first.17:42
aNuposicgnight pas-ha17:42
dtantsurJayF, I agree with removing it quite soon, but I'd prefer to get people the last chance to shout NOOOOO17:43
dtantsurmariojv, ack re review party17:43
JayFdtantsur: it's on my short-term-todo (probably today, maybe tomorrow) to email about it17:43
JayFdtantsur: to the list, and put some kind of time limit (2 weeks?)17:43
lucasagomesdtantsur, right yeah, I remember talking about that indeed17:43
*** sayantan_ has quit IRC17:43
mariojvthanks dtantsur17:43
mariojvwill add that today17:44
dtantsurrloo, will check re OSC driver-properties. Still, I remember we explicitly left it out from the driver comp. we may get back to it (= ask jroll to get back to it ;)17:44
*** sayantan_ has joined #openstack-ironic17:44
dtantsurJayF, 2 weeks sounds good17:44
dtantsuralso g'morning to everyone who I missed while being out17:44
lucasagomes:D17:44
NobodyCamMorning dtantsur :)17:45
lucasagomesit's a g'night for me now, calling it a day17:45
lucasagomesNobodyCam, morning17:45
rloodtantsur: ha ha, pass the buck! I'll ask jroll next week. I want to make sure we know how we want to handle that, before we decide on the osc command for 'ironic driver-properties'17:45
NobodyCamnight lucasagomes :)17:45
rloobye lucasagomes!17:45
lucasagomeshave a great evening all! "See u all" tomorrow17:45
dtantsurlucasagomes, o/17:45
*** lucasagomes is now known as lucas-afk17:45
dtantsurrloo, so, there will always be some mode of operation which is the same for dynamic and classic drivers17:45
dtantsurrloo, i.e. if 'ironic driver-properties pxe_ipmitool' works, ''ironic driver-properties ipmi' should also produce some sane results17:46
dtantsurthis comes from the requirements to support new-style drivers in old API versions17:46
rlooJayF: fwiw, we already indicated (can't remember when) that ipminative wasn't supported. so this is a last-call, 2 weeks seems plenty of time.17:46
rloodtantsur: it does produce sane results, just that for a hardware type, it produces results for default interfaces.17:46
rloodtantsur: the problem is if operators are using this to know what to set in node.driver_info[], it may not produce the right list.17:47
*** derekh has quit IRC17:47
dtantsurthis is correct, and quite unfortunate. we can't fix it in old API versions, can we?17:48
rloodtantsur: eg, ipmi has no-console, so it won't show the ipmi-whatever-port that a node with ipmi & serialconsolewhatever console interface needs to set. i described it better in the bug.17:48
* mgould -> home; good night!17:48
*** mgould is now known as mgould|afk17:48
dtantsurc u mgould|afk17:48
NobodyCamnight mgould|afk :)17:48
rloodtantsur: i don't think we can, i am guessing we need a microversion bump and update the API to show that info. we need to decide what that API would be though.17:48
rloodtantsur: i almost feel like the request is something like (in Englisheze) 'Give me the driver-related properties that I need to set for this node <node>'17:49
rloodtantsur: versus 'give me the driver-related properties for <driver> with boot-interface=b, console-interface=c, ... Or both? Maybe I'm crazy...17:50
dtantsurrloo, yeah.. maybe we should go away from the notion of "driver properties" and get to "node driver-specific properties"17:50
rloodtantsur: now that i think of it, we might want both :-(17:51
dtantsurI guess we have a similar, though very unlikely, problem with RAID properties, right?17:51
*** jaganathan has quit IRC17:51
rloodtantsur: yeah, if you specify a hardware type, unless the default raid interface is something useful, you won't get any RAID properties.17:51
rloodtantsur: makes me think that RAID properties might only be useful if you ask for the properties wrt a particular RAID interface, not driver.17:52
dtantsurand the same for driver properties, right?17:52
dtantsurno properties are attached to a classic driver17:53
dtantsurdamn, this is a can of worms17:53
rloodtantsur: classic drivers have properties -- their interfaces are set/static. so it depends on the classic driver.17:53
rloodtantsur: heh, except for network and storage. i wonder if any of those interfaces will have properties associated with them :)17:54
dtantsurthey may17:54
*** moshele has quit IRC17:54
*** milan has quit IRC17:54
*** amarao has quit IRC17:55
*** moshele has joined #openstack-ironic17:55
*** moshele has quit IRC18:00
dtantsurrloo, I started thought dump at https://etherpad.openstack.org/p/ironic-driver-properties-reform, I cannot reason about API on IRC18:00
rloodtantsur: thought dump is good!18:00
*** lhinds is now known as lhinds|afk18:00
*** ralonsoh has quit IRC18:01
*** Nisha_Agarwal has quit IRC18:02
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362818:02
*** Nisha_Agarwal has joined #openstack-ironic18:04
*** links has quit IRC18:04
sambettsTheJulia: I worked out my operation not permitted issue, it was all to do with LXC permissions because I was running the conductor in docker, all fixed now :)18:08
*** jaganathan has joined #openstack-ironic18:09
*** Nisha_Agarwal has quit IRC18:09
*** ChubYann has joined #openstack-ironic18:11
*** Nisha_Agarwal has joined #openstack-ironic18:11
*** Nisha_Agarwal has quit IRC18:12
*** Nisha_Agarwal has joined #openstack-ironic18:12
*** _vsaienko has joined #openstack-ironic18:15
*** vsaienko has quit IRC18:15
*** Nisha_Agarwal has quit IRC18:17
*** Nisha_Agarwal has joined #openstack-ironic18:18
JayFrloo: dtantsur: Email sent to list re: ipminative driver, an ack (specifically from PTL) would be appreciated so folks know I'm not going off the deep end ;)18:19
*** bcornec2 has joined #openstack-ironic18:20
*** baha has joined #openstack-ironic18:21
JayFdtantsur: "Actually, I have only one strong feeling about the Forum. I don't want to go there, if it involves more than 3 hours of flight time." <-- very yes18:23
dtantsur:)18:23
dtantsuralso thanks for the email, I'll respond18:24
rloodtantsur: i added thoughts to your etherpad, and linked it from the spec & rfe bug. will see what jroll or others think.18:26
openstackgerritRamamani Yeleswarapu proposed openstack/ironic master: Enable OSProfiler support in Ironic  https://review.openstack.org/35814218:28
*** jaganathan has quit IRC18:29
*** rama has joined #openstack-ironic18:31
*** rama is now known as Guest6137518:31
*** amoralej is now known as amoralej|off18:31
*** Guest61375 is now known as rama_y18:40
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362818:45
*** _vsaienko has quit IRC18:46
*** baha has quit IRC18:46
*** baha has joined #openstack-ironic18:46
*** fragatina has joined #openstack-ironic18:50
*** fragatina has quit IRC18:50
*** fragatina has joined #openstack-ironic18:51
*** sambetts is now known as sambetts|afk18:52
sambetts|afknight all!!18:52
*** vsaienko has joined #openstack-ironic18:53
vdrokgood night sambetts|afk and all!18:56
*** yogi has quit IRC19:02
dtantsurTheJulia, mind linking to the BFV meeting wiki from "Related meetings" in https://wiki.openstack.org/wiki/Meetings/Ironic please?19:07
openstackgerritRamamani Yeleswarapu proposed openstack/ironic master: Update the Ironic Upgrade guide  https://review.openstack.org/43919619:09
rloojoanna: fyi, I plan to discuss your oslo patch https://review.openstack.org/#/c/442106/ in monday's oslo meeting.19:14
patchbotpatch 442106 - oslo.db - Decrease number of retries on deadlock19:14
*** david-lyle has quit IRC19:14
joannarloo: awesome19:14
*** ElCoyote_ has joined #openstack-ironic19:16
*** sayantan_ has quit IRC19:24
*** sayantan_ has joined #openstack-ironic19:24
*** bcornec2 has quit IRC19:31
*** vsaienko has quit IRC19:33
*** vsaienko has joined #openstack-ironic19:34
*** gfhellma has quit IRC19:38
*** vills has joined #openstack-ironic19:45
*** sayanta__ has joined #openstack-ironic19:47
*** sayantan_ has quit IRC19:50
*** bcornec2 has joined #openstack-ironic19:50
*** amarao has joined #openstack-ironic20:01
*** mtanino has quit IRC20:02
*** gfhellma has joined #openstack-ironic20:11
*** vsaienko has quit IRC20:11
*** vsaienko has joined #openstack-ironic20:14
*** Nisha_Agarwal has quit IRC20:19
rloojoanna: michael and clint commented on your db patch. looks like 20 is a good default to use.20:21
*** dtantsur is now known as dtantsur|afk20:28
*** mgoddard has quit IRC20:29
joannarloo: good for them or good for us?20:29
rloojoanna: ? good for us. take a look at their comments.20:30
*** mjura has quit IRC20:34
*** fragatina has quit IRC20:36
openstackgerritMerged openstack/ironic-ui master: Updated from global requirements  https://review.openstack.org/44005320:38
*** e0ne has joined #openstack-ironic20:44
*** chlong_ has joined #openstack-ironic20:47
*** bcornec2 has quit IRC20:50
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job  https://review.openstack.org/42214920:51
*** priteau has quit IRC20:54
mariojvreview party in 3 minutes for anyone attending20:55
mariojvhttps://etherpad.openstack.org/p/ironic-review-party20:56
mariojvmight be 2 mins late20:56
jlvillalhttps://plus.google.com/hangouts/_/calendar/amltcm9sbGVuaGFnZW5AZ21haWwuY29t.s3ntmquk4omj7jvncki7q7rc3c20:56
jlvillalhttps://www.youtube.com/watch?v=njsF1jWSWeM20:56
*** vsaienko has quit IRC20:57
*** _vsaienko has joined #openstack-ironic20:57
*** frontrunner has quit IRC20:57
mariojvthanks jlvillal20:58
jlvillalmariojv: So I am trying to join but it says I am waiting...20:58
jlvillalOh now I got iin20:58
joannarloo: I will then upload the update with fixes for comments + bug reference. Would you mind me abandoning oslo patch? I see some unanswered questions there21:00
rloojoanna: yes, i think your oslo patch should be abandoned. not sure about the unanswered questions there too.21:00
rloojoanna: wrt your ironic patch, yes, please update the bug associated with it and the patch.21:01
joannarloo: I don't want to abandod it if it will cause these questions being unanswered forever21:02
joanna:)21:02
rloojoanna, i don't recall what those unanswered questions are. use your judgement?21:02
*** gfhellma has quit IRC21:03
rloojoanna: oh, do you mean the question i had about documentation?21:03
joannathey were about documentation of retries & deadlock21:03
*** fragatina has joined #openstack-ironic21:03
joannarloo: yes21:03
rloojoanna: well, i am curious to know, whether you abandon or not, they could answer. up to you whether to abandon now, or wait til they answer. it is possible that if you abandon, they won't reply. or they might not reply at all. dunno.21:04
joannaI will give it one day... I'll abandon it tomorrow eod21:05
*** gfhellma has joined #openstack-ironic21:07
rloojoanna: ok21:09
*** mjturek1 has joined #openstack-ironic21:14
*** mjturek has quit IRC21:14
*** baha1 has joined #openstack-ironic21:14
*** baha has quit IRC21:16
*** jaosorior has joined #openstack-ironic21:25
*** jaosorior has quit IRC21:33
*** amarao has quit IRC21:38
openstackgerritJoanna Taryma proposed openstack/ironic master: Remove overwriting the default value of db_max_retries.  https://review.openstack.org/44086521:40
*** _vsaienko has quit IRC21:40
*** krtaylor has quit IRC21:42
*** vsaienko has joined #openstack-ironic21:44
*** hamzy has quit IRC21:48
*** e0ne has quit IRC21:48
*** mjturek1 has quit IRC21:51
*** mjturek has joined #openstack-ironic21:51
*** baha1 is now known as baha21:55
*** e0ne has joined #openstack-ironic21:57
openstackgerritJohn L. Villalovos proposed openstack/ironic stable/newton: WIP/DNM NEWTON: Testing multi-tenant multi-node grenade job  https://review.openstack.org/44397821:57
*** crushil has quit IRC21:59
*** e0ne has quit IRC22:01
openstackgerritJohn L. Villalovos proposed openstack/ironic stable/newton: Don't 'enable_plugin ironic' in grenade settings  https://review.openstack.org/44398122:15
mariojvwhat are "IRONIC_VM_SPECS_DISK" and "IRONIC_VM_EPHEMERAL_DISK" on this patch? https://review.openstack.org/#/c/423556/75/devstack/lib/ironic22:20
patchbotpatch 423556 - ironic - Add Ironic standalone tests22:20
mariojvi guess ephemeral can disappear so we don't surface it to nova?22:21
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Don't 'enable_plugin ironic' in grenade settings  https://review.openstack.org/44215422:22
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Don't 'enable_plugin ironic' in grenade settings  https://review.openstack.org/44215422:25
*** jjohnson2 has quit IRC22:32
*** goldyfruit has quit IRC22:33
*** vsaienko has quit IRC22:34
*** vsaienko has joined #openstack-ironic22:37
aNuposichey mariojv so have you started anything at dynamic network support for tinyipa?22:39
aNuposicJayF ^22:39
*** gfhellma has quit IRC22:40
*** gfhellma_ has joined #openstack-ironic22:40
aNuposicI am just taking a look at patch to support in coreos https://review.openstack.org/#/c/423521/ and thinking to add that support first before moving to static network support for tinyipa22:40
patchbotpatch 423521 - ironic-python-agent - Rescue extension for CoreOS with DHCP tenant networks22:40
*** jcoufal has quit IRC22:41
mariojvnot yet aNuposic22:42
mariojvi will soon-ish22:42
mariojvaNuposic: i'm fine with it if you get to it first :)22:42
*** jtomasek has quit IRC22:44
JayFmariojv: I was just talking to aNuposic, and suggested he take that first22:44
JayFmariojv: because if we get DHCP working on both images, then aslezil_'s work on tempest for rescue should pass across the board22:44
JayFmariojv: meaning we can hopefully merge w/limitation for DHCP-supported-networks only, get the nova in while we work on static22:45
mariojvis something wrong with gerrit? i keep getting signed out22:45
mariojvlike i said, that's fine if he gets to it first :)22:45
JayFyou think that's a sensible approach?22:45
mariojvsure22:45
aNuposicmariojv, JayF cool22:45
mariojvi'm out half next week anyway22:45
aNuposicJayF, mariojv also i have added some notes in the rescue etherpad about patch inside glean22:46
JayFI'm out from *next* friday until the following Wednesday22:46
mariojvthanks aNuposic22:47
*** jtomasek has joined #openstack-ironic22:47
*** baha has quit IRC22:48
mariojvdoes someone else want to lead review party next week while i'm away?22:48
mariojvvsaienko: you might want to reconsider +2 on https://review.openstack.org/#/c/443159/, found a small bug22:53
patchbotpatch 443159 - ironic-lib - Improve logs for the root device hints mechanism22:53
mariojvuseful patch though lucas-afk, thanks22:55
*** priteau has joined #openstack-ironic22:55
vsaienkomariojv that part was just copied, the patch is about improving log, but if there is a bug in the code it is up to Lucas fix it in current patch, or in separate22:56
mariojvoh, i didn't realize that was in the code already22:56
mariojvi'll ping him tomorrow and see what he thinks22:56
mariojvthx for clarifying22:56
openstackgerritMerged openstack/ironic-inspector master: Reapply doesn't update started_at time  https://review.openstack.org/43105322:58
*** sayanta__ has quit IRC23:00
*** priteau has quit IRC23:00
*** cdearborn has quit IRC23:01
*** Syed__ has quit IRC23:05
*** baoli has quit IRC23:20
*** chlong_ has quit IRC23:21
*** baoli has joined #openstack-ironic23:21
*** baoli has quit IRC23:26
aNuposicmariojv, JayF can similar cloud-config be used for tinyipa?23:30
aNuposicright now there is no cloud-config for tinyipa23:31
*** vills has quit IRC23:33
*** pmannidi has joined #openstack-ironic23:44
*** vsaienko has quit IRC23:44
*** dsneddon has quit IRC23:53

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