Tuesday, 2017-07-04

*** rpioso has quit IRC00:02
*** mat128 has joined #openstack-ironic00:13
*** pmannidi has quit IRC00:21
openstackgerritJulia Kreger proposed openstack/ironic-ui master: Moving ironic-ui install content to install folder  https://review.openstack.org/47998000:30
openstackgerritJulia Kreger proposed openstack/ironic-ui master: Move ironic-ui contributor docs  https://review.openstack.org/47998100:30
*** pmannidi has joined #openstack-ironic00:46
*** ijw has joined #openstack-ironic00:48
*** mat128 has quit IRC00:50
*** hw_wutianwei has joined #openstack-ironic00:51
*** litao__ has joined #openstack-ironic01:02
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Add volume connector support to Python API  https://review.openstack.org/21478601:02
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Add volume target support to Python API  https://review.openstack.org/28523301:08
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Add volume connector support to Python API  https://review.openstack.org/21478601:09
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Add OSC commands for volume connector  https://review.openstack.org/42705301:11
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Switch from oslosphinx to openstackdocstheme  https://review.openstack.org/47998801:12
*** mat128 has joined #openstack-ironic01:16
* TheJulia thinks it is time to call it a night01:17
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Add Ironic CLI commands for volume connector  https://review.openstack.org/47984001:19
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Add OSC commands for volume target  https://review.openstack.org/42773801:19
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Add volume target support to Python API  https://review.openstack.org/28523301:19
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Add Ironic CLI commands for volume target  https://review.openstack.org/47984801:19
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Add OSC commands for volume connector  https://review.openstack.org/42705301:19
TheJuliahshiina: Thanks :)01:19
hshiinaTheJulia: thanks for review01:20
*** mat128 has quit IRC01:22
TheJuliaI'm going to take care of some stuff around the house and go to sleep.  I'll likely be on in the morning even though it is a holiday01:22
TheJuliagoodnight everyone01:22
*** jaypipes has quit IRC01:22
*** shuyingya has joined #openstack-ironic01:27
*** jkilpatr has quit IRC01:32
openstackgerritTuan Luong-Anh proposed openstack/python-ironicclient master: switch from oslosphinx to openstackdocstheme  https://review.openstack.org/47999901:51
*** rbudden has quit IRC02:01
*** joe817 has quit IRC02:07
*** skazi has quit IRC02:25
*** shuyingya has quit IRC02:31
*** shuyingya has joined #openstack-ironic02:31
*** skazi has joined #openstack-ironic02:33
*** shuyingy_ has joined #openstack-ironic02:34
*** shuyingya has quit IRC02:34
*** crushil_ has joined #openstack-ironic02:36
*** ijw has quit IRC02:39
openstackgerritMerged openstack/ironic master: Add node power state validation to volume resource update/deletion  https://review.openstack.org/47914202:44
*** crushil_ has quit IRC02:44
*** baoli has quit IRC02:46
*** zhugaoxiao has quit IRC02:52
*** zhugaoxiao has joined #openstack-ironic02:52
*** ijw_ has joined #openstack-ironic02:53
*** baoli has joined #openstack-ironic02:53
*** ijw_ has quit IRC02:57
*** gcb has joined #openstack-ironic02:59
*** dsneddon has quit IRC03:03
*** dsneddon has joined #openstack-ironic03:03
*** dsneddon_ has joined #openstack-ironic03:07
*** dsneddon_ has quit IRC03:07
*** dsneddon has quit IRC03:11
*** ChubYann has quit IRC03:14
*** dsneddon has joined #openstack-ironic03:16
*** jaganathan has joined #openstack-ironic03:19
*** ChubYann has joined #openstack-ironic03:28
openstackgerritTuan Luong-Anh proposed openstack/python-ironicclient master: Rearrange existing documentation to fit the new standard layout  https://review.openstack.org/48001303:29
*** rbudden has joined #openstack-ironic03:31
*** hshiina has quit IRC03:38
*** rbudden has quit IRC03:39
*** kiennt has joined #openstack-ironic03:46
*** yee37925 has joined #openstack-ironic03:47
*** yee379 has quit IRC03:47
*** tuanluong has joined #openstack-ironic03:50
kiennthi everyone, i just read about multinode grenade CI in Rolling upgrade spec: https://github.com/openstack/ironic-specs/blob/master/specs/approved/support-rolling-upgrade.rst#testing.03:53
kienntCan anyone give more details about it: Progress, idea or the difficulties encountered?03:53
kienntthank you :)03:53
*** hshiina has joined #openstack-ironic04:04
openstackgerritTuan Luong-Anh proposed openstack/python-ironicclient master: Rearrange existing documentation to fit the new standard layout  https://review.openstack.org/48001304:04
*** crushil_ has joined #openstack-ironic04:07
*** links has joined #openstack-ironic04:11
*** udesale has joined #openstack-ironic04:15
*** crushil_ has quit IRC04:22
*** crushil_ has joined #openstack-ironic04:26
*** crushil_ has quit IRC04:31
*** jaganathan has quit IRC04:53
*** baoli has quit IRC04:55
*** mariojv has quit IRC05:16
*** eglute has quit IRC05:16
*** eglute_s has joined #openstack-ironic05:16
openstackgerritAparna proposed openstack/ironic master: Add manual clean step ironic standalone test  https://review.openstack.org/42977005:17
*** kiennt has quit IRC05:22
*** crushil_ has joined #openstack-ironic05:23
*** dsneddon has quit IRC05:26
*** crushil_ has quit IRC05:28
*** crushil_ has joined #openstack-ironic05:30
*** mvpnitesh has joined #openstack-ironic05:32
*** joe817 has joined #openstack-ironic05:32
*** joe817 has quit IRC05:36
*** kiennt has joined #openstack-ironic05:41
*** gdony has joined #openstack-ironic05:41
*** ChubYann has quit IRC05:46
*** crushil_ has quit IRC05:47
*** mjura has joined #openstack-ironic05:50
*** kiennt has quit IRC06:00
*** crushil_ has joined #openstack-ironic06:02
*** rcernin has joined #openstack-ironic06:04
*** crushil_ has quit IRC06:23
*** hjensas has quit IRC06:25
*** rcernin has quit IRC06:31
*** rcernin has joined #openstack-ironic06:34
*** rcernin has quit IRC06:34
*** rcernin has joined #openstack-ironic06:34
*** marios has joined #openstack-ironic06:34
*** crushil_ has joined #openstack-ironic06:38
openstackgerritAparna proposed openstack/proliantutils master: Redfish: Adds manager support for redfish API's.  https://review.openstack.org/47371906:45
*** crushil_ has quit IRC06:48
*** pcaruana has joined #openstack-ironic06:55
*** kiennt has joined #openstack-ironic06:56
openstackgerritAparna proposed openstack/proliantutils master: Redfish: Add support for Virtual Media Operations  https://review.openstack.org/47379807:02
*** mvpnitesh has quit IRC07:07
*** mvpnitesh has joined #openstack-ironic07:07
*** joe817 has joined #openstack-ironic07:11
*** jaganathan has joined #openstack-ironic07:12
*** shuyingy_ has quit IRC07:13
*** ccamacho has joined #openstack-ironic07:13
*** tesseract has joined #openstack-ironic07:14
openstackgerritmelissaml proposed openstack/proliantutils master: Using assertIsNone() instead of assertEqual(None)  https://review.openstack.org/48003807:14
*** joe817 has quit IRC07:18
openstackgerritHironori Shiina proposed openstack/ironic master: Add API for volume resources to api-ref  https://review.openstack.org/47879807:22
openstackgerritWang KaiFeng proposed openstack/ironic master: Make IP address of socat console configurable  https://review.openstack.org/47320107:27
*** ccamacho has quit IRC07:35
*** ccamacho has joined #openstack-ironic07:35
openstackgerritWang KaiFeng proposed openstack/ironic master: Make IP address of socat console configurable  https://review.openstack.org/47320107:38
*** etingof has joined #openstack-ironic07:42
*** joe817 has joined #openstack-ironic07:44
vdrokgood morning ironic!07:46
*** mgoddard_ has joined #openstack-ironic07:47
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Serve boot configs from Ironic API  https://review.openstack.org/36943807:50
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Implement Driver API for dynamic boot config  https://review.openstack.org/39439807:50
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add RPC API to get the boot config  https://review.openstack.org/39439907:50
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: DNM test enabled dynamic iPXE  https://review.openstack.org/39194307:50
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Allow not writing pxe configs to disk  https://review.openstack.org/39439707:50
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Convert iPXE boot config to proper Jinja template  https://review.openstack.org/39346207:50
*** ralonsoh has joined #openstack-ironic07:52
*** joe817 has quit IRC07:53
*** racedo has joined #openstack-ironic07:56
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** makowals has joined #openstack-ironic08:06
*** skazi__ has joined #openstack-ironic08:07
*** mariojv has joined #openstack-ironic08:07
*** makowals_ has quit IRC08:09
*** joe817 has joined #openstack-ironic08:10
*** joe817 has quit IRC08:15
*** skazi__ has quit IRC08:15
*** mvpnitesh has quit IRC08:21
*** mvpnitesh has joined #openstack-ironic08:21
*** joe817 has joined #openstack-ironic08:22
*** joe817 has quit IRC08:27
*** mvpnitesh has quit IRC08:28
*** mvpnitesh has joined #openstack-ironic08:34
*** mvpnitesh has quit IRC08:40
*** mvpnitesh has joined #openstack-ironic08:40
*** gdony has quit IRC08:43
*** mvpnitesh has quit IRC08:44
*** mvpnitesh has joined #openstack-ironic08:44
*** lucas-afk is now known as lucasagomes08:47
*** joe817 has joined #openstack-ironic08:54
*** hshiina has quit IRC08:59
*** e0ne has joined #openstack-ironic08:59
*** joe817 has quit IRC09:01
*** racedo has quit IRC09:09
*** racedo has joined #openstack-ironic09:11
*** milan has joined #openstack-ironic09:12
*** hjensas has joined #openstack-ironic09:15
*** mrchao has joined #openstack-ironic09:17
*** joe817 has joined #openstack-ironic09:19
*** joe817 has quit IRC09:24
*** joe817 has joined #openstack-ironic09:30
*** amotoki_away is now known as amotoki09:31
*** joe817 has quit IRC09:37
pas-hamorning vdrok and all ironic :)09:39
vdrokmorning pas-ha !09:39
milanmorning Ironic! :)09:39
milanmorning pas-ha, vdrok and all! :)09:40
vdrokMorning milan :)09:42
openstackgerritArundhati Surpur proposed openstack/ironic-python-agent master: Removes unnecessary utf-8 encoding  https://review.openstack.org/48009909:42
milan :)09:42
*** amotoki is now known as amotoki_away09:43
*** etingof has quit IRC09:43
*** sambetts|afk is now known as sambetts09:45
sambettsMorning all09:45
pas-hamorning milan sambetts09:46
vdrokmorning sambetts09:51
*** joe817 has joined #openstack-ironic09:54
milanmorning sambetts! :)09:55
*** etingof has joined #openstack-ironic09:56
openstackgerritMark Goddard proposed openstack/ironic master: Physical network aware VIF attachment  https://review.openstack.org/47040609:58
*** kiennt has quit IRC09:58
mgoddard_morning all09:58
mgoddard_sambetts: https://review.openstack.org/470406, now with added messiness09:59
patchbotpatch 470406 - ironic - Physical network aware VIF attachment09:59
mgoddard_(again) :)09:59
*** joe817 has quit IRC09:59
sambettsheh thanks mgoddard_10:02
*** livelace has joined #openstack-ironic10:03
*** joe817 has joined #openstack-ironic10:04
vdrokmorning mgoddard_10:04
*** tuanluong has quit IRC10:05
mgoddard_sambetts: I took a less efficient route with the code which makes it a lot less messy than before, at the cost of an extra neutron port lookup. Gives us some incentive to push on with the follow on changes :)10:05
mgoddard_morning vdrok10:05
sambettsmgoddard_: sounds good10:06
*** MattMan_1 has quit IRC10:08
*** MattMan_1 has joined #openstack-ironic10:08
mgoddard_sambetts: I went back over the thought process behind the long dependency chain, and I think it all comes back to the tempest tests. They're the only things that still need the network interface to silently fail without neutron10:09
mgoddard_sambetts: we decided we couldn't switch to the noop interface in the tempest VIF tests because the current tests would fail due to the VIF list not returning previously attached VIFs10:10
*** joe817 has quit IRC10:11
mgoddard_sambetts: but given that tempest is only testing the API, perhaps that doesn't matter, and those test cases should be rewritten to handle the noop driver?10:11
sambettsmgoddard_: yeah, we could definatly change the test case, I think the idea of adding a fake network interface to exercise the API is the right way to do things though even if it just returns hard coded values of vif list etc, its how we test the rest of the APIs set power state, uses the fake power driver, that just accepts any valid power state and sets the nodes state to that value10:14
mgoddard_sambetts: on a related note, this whole issue leads me to think that VIFs should be a first class object in the DB rather than tagged on ports. Then the VIF API would be less driver dependent, and a lot of logic could be pushed up into the conductor from the drivers, and VIF list would work for noop - the VIFs would exist but would just not be mapped to a port10:14
sambettsmgoddard_: the reason why the logic is pushed into the drivers is because some drivers don't map the vifs to ports, because magic hardware can make ports out of vifs at deploy time10:17
sambettsmgoddard_: mapping vifs to ports is just how the existing interfaces work, but is by no means the only way it can be implemented10:18
sambettsmgoddard_: the interfaces I'm refering too just collect the attached vifs in a list of vifs in the driver internal info10:19
sambettsmgoddard_: and during connect tenent networks, contact the hardware and make new NICs10:19
*** amarao has joined #openstack-ironic10:20
mgoddard_sambetts: fair point, but a first class VIF object would avoid each driver needing to store the VIFs itself10:20
sambettsmgoddard_: yeah, a driver would still have to store the mapping or not mapping in their driver speicifc way though right?10:23
*** fellypefca has quit IRC10:25
mgoddard_sambetts: the common path maps VIFs to ports, and that could be supported via DB relations. Magic drivers would just not use that mechanism10:25
mgoddard_sambetts: it's not a huge issue, the current design works well enough and is quite lightweight. Just a thought I had as I was working through it all10:27
*** ricardoas has quit IRC10:27
sambettsmgoddard_: worth a RFE/spec and conversation at the PTG I think10:27
*** xavierr has quit IRC10:28
*** joe817 has joined #openstack-ironic10:28
mgoddard_sambetts: I guess in these lean days it has to be weighed up against other features10:29
mgoddard_sambetts: sadly I won't make this PTG, I'm getting married a week or so after so I'll be busy preparing/fretting10:30
sambettsmgoddard_: unfortunatly yes, although I'd like to think making ironic network intergration work as well as possible and be as clean as possble would be relatively high priority10:30
sambettsmgoddard_: OH! Congrats!10:31
mgoddard_sambetts: thanks - although it's not over 'til the fat lady sings ( / slim lady rings?)10:32
sambettsheh10:32
vdrokyeah, making vif a ovo will also enable us to version the payloads sent from conductor to api10:32
vdrokcongrats mgoddard_ :)10:32
mgoddard_thanks vdrok10:33
*** joe817 has quit IRC10:33
*** ricardoas has joined #openstack-ironic10:34
*** xavierr has joined #openstack-ironic10:34
openstackgerritAnshul Jain proposed openstack/proliantutils master: Redfish: Adds 'update_firmware' APIs support through redfish  https://review.openstack.org/47253210:41
*** ricardoas has quit IRC10:45
*** ricardoas has joined #openstack-ironic10:48
*** mrchao has quit IRC10:48
mgoddard_third party CI is a joke10:50
*** aszeszo has joined #openstack-ironic10:52
*** dmellado is now known as dmellado_10:57
*** dmellado_ is now known as dmellado10:58
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/48014010:58
*** mat128 has joined #openstack-ironic10:59
*** shvepsy has joined #openstack-ironic11:04
*** xavierr has quit IRC11:05
*** ricardoas has quit IRC11:05
*** lucasagomes is now known as lucas-hungry11:06
shvepsyHello there! Do you know why ironic-inspector documentation no more available?11:06
shvepsyOh rework, sorry11:08
openstackgerritbhavani proposed openstack/ironic-specs master: Remove pbr warnerrors in favor of sphinx check  https://review.openstack.org/47887911:18
*** xavierr has joined #openstack-ironic11:22
*** Guest47284 has quit IRC11:29
*** rook has joined #openstack-ironic11:30
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: WIP: Add support for storage_interface to node and driver CLI  https://review.openstack.org/46618611:30
*** rook is now known as Guest713011:31
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: WIP: Add support for storage_interface to node and driver CLI  https://review.openstack.org/46618611:35
*** fellypefca has joined #openstack-ironic11:42
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Serve boot configs from Ironic API  https://review.openstack.org/36943811:43
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Implement Driver API for dynamic boot config  https://review.openstack.org/39439811:43
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add RPC API to get the boot config  https://review.openstack.org/39439911:43
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: DNM test enabled dynamic iPXE  https://review.openstack.org/39194311:43
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Allow not writing pxe configs to disk  https://review.openstack.org/39439711:43
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Convert iPXE boot config to proper Jinja template  https://review.openstack.org/39346211:43
*** afazekas is now known as afazekas|away11:43
*** ccamacho is now known as ccamacho|lunch11:45
openstackgerritAparna proposed openstack/proliantutils master: Redfish: Add support for Virtual Media Operations  https://review.openstack.org/47379811:49
*** joe817 has joined #openstack-ironic11:50
fellypefcaGood morning all :)11:51
*** links has quit IRC11:53
*** joe817 has quit IRC11:54
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Skip warning when changing target_raid_config  https://review.openstack.org/48016112:03
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode  https://review.openstack.org/46798212:04
*** jkilpatr has joined #openstack-ironic12:05
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode  https://review.openstack.org/46798212:05
*** links has joined #openstack-ironic12:09
*** lucas-hungry is now known as lucasagomes12:13
TheJuliaGood morning everyone12:21
*** links has quit IRC12:21
sambettshey TheJulia12:24
*** e0ne has quit IRC12:24
TheJuliasambetts: Oh the fun of this doc stuff :(12:25
sambettsyup...12:25
TheJuliaI see you chatted with Andreas this morning12:25
TheJuliaHey, Review... I know it is a holiday here, but come on12:25
*** e0ne has joined #openstack-ironic12:25
sambettsTheJulia: yeah it looks like the api-ref might not be moving initially :/12:26
TheJuliayeah, and we need the test on the ocata install guide. I had a feeling that might be the case, but couldn't see how they were building it out of the pipeline.12:27
*** mrchao has joined #openstack-ironic12:29
sambettsTheJulia: yeah thats all in place now :D Andreas quick merged it to unblock us12:29
TheJuliaYeah, I saw it merged.  I can't actually load the change it seems :(12:30
*** rbudden has joined #openstack-ironic12:37
sambettsTheJulia: I'm just reordering my patches to reflect this mornings discussion so we can start landing stuff12:37
*** links has joined #openstack-ironic12:40
TheJuliaCool, I have a couple questions for Doug, but figure they will have to wait until tomorrow12:42
*** amotoki_away is now known as amotoki12:43
*** sasha2 has joined #openstack-ironic12:45
TheJuliaI guess once we have some more clarity, it shouldn't be too difficult for me to reshuffle ipa's docs.12:49
*** links has quit IRC12:52
openstackgerritJulia Kreger proposed openstack/ironic-ui master: Move ironic-ui contributor docs  https://review.openstack.org/47998112:52
openstackgerritJulia Kreger proposed openstack/ironic-ui master: Set warning-is-error for documentation builds  https://review.openstack.org/48018012:54
openstackgerritJulia Kreger proposed openstack/ironic-ui master: Moving ironic-ui install content to install folder  https://review.openstack.org/47998012:58
openstackgerritJulia Kreger proposed openstack/ironic-ui master: Move ironic-ui contributor docs  https://review.openstack.org/47998112:58
milansambetts, hey, I'm copying your docs patches into inspector, wondering why openstackdocstheme ( https://review.openstack.org/#/c/479871/3/doc/source/conf.py ) isn't added in test requirements12:58
patchbotpatch 479871 - ironic - switch from oslosphinx to openstackdocstheme12:58
sambettsmilan: it was already in there12:58
*** pcaruana has quit IRC12:59
milanheh :)12:59
sambettsmilan: I think it was in there from where we added in-tree api-ref and install guide12:59
milanI see just the line above what you're removing ;)12:59
milansambetts, thx13:00
sambettsno problem :)13:00
milansambetts +1 ;)13:01
*** rbartal has joined #openstack-ironic13:02
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Skip warning when changing target_raid_config  https://review.openstack.org/48016113:04
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: switch from oslosphinx to openstackdocstheme  https://review.openstack.org/48018513:07
milan... the first one ^13:07
milanbugs reset the branch?!?13:08
*** ccamacho|lunch is now known as ccamacho13:08
* milan updates commit message13:08
*** joe817 has joined #openstack-ironic13:12
*** amotoki is now known as amotoki_away13:12
*** amotoki_away is now known as amotoki13:15
milansambetts, TheJulia I'm adding you guys to the inspector review list for the docs13:15
milanoh and morning TheJulia! :)13:15
*** joe817 has quit IRC13:16
*** amotoki is now known as amotoki_away13:16
*** mat128 has quit IRC13:17
*** mat128 has joined #openstack-ironic13:18
*** jtomasek has joined #openstack-ironic13:19
openstackgerritM V P Nitesh proposed openstack/python-ironicclient master: Openstack baremetal node maintenance  https://review.openstack.org/48018813:20
*** mkrai has quit IRC13:20
*** retr0h has quit IRC13:21
*** amotoki_away is now known as amotoki13:24
*** jtomasek has quit IRC13:24
*** rama_y has quit IRC13:25
openstackgerritM V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info  https://review.openstack.org/46056413:26
*** mrtenio has joined #openstack-ironic13:26
*** amotoki is now known as amotoki_away13:29
*** ralonsoh has quit IRC13:31
*** ralonsoh has joined #openstack-ironic13:31
*** pcaruana has joined #openstack-ironic13:36
*** links has joined #openstack-ironic13:36
*** baoli has joined #openstack-ironic13:40
*** amotoki_away is now known as amotoki13:45
*** pcaruana has quit IRC13:45
TheJuliaIs anyone actively reviewing the volume target/connector info API?  It presently has 2x+213:49
*** pcaruana has joined #openstack-ironic13:50
openstackgerritPeter Piela proposed openstack/ironic-ui master: Introduce form-field directive  https://review.openstack.org/48020113:51
sambettsTheJulia: I've not taken a look yet13:52
*** retr0h has joined #openstack-ironic13:53
TheJuliaruby gave it a +1 with some changes fixes requested, dmitry gave it a good review over the weekend and the requested changes were addressed.  If you really want to dig through it, it is likely an all day review13:53
TheJuliaunless you've reviewed it recently that is13:54
mrtenioGood Morning Ironic'ers.13:57
TheJuliaGood morning13:57
TheJuliappiela: I saw the change above, you working today?13:57
fellypefcaMorning mrtenio :)13:58
sambettsTheJulia: sounds like its more or less ready to land, just took an incredibly quick flick through and there is nothing obvious to me13:58
TheJuliasambetts: That is exactly why I asked :)13:58
ppielaHi TheJulia: on and off13:58
TheJuliappiela: I wasn't really thinking of trying to hold the meeting today since I think 85% of the attendees are based in north america, an likely have today off.13:59
*** retr0h has quit IRC14:00
ppielaTheJulia: Yeah, I was not planning on a meeting today. I dont see an burning issues, things are moving along.14:01
TheJuliaThe only thing I could note is the docs rework means that the UI docs are not present on docs.openstack.org right now.  I've got a few patches up there, working on a patch to improve it further at the moment right now.  I don't think there is any content that needs to be pulled in from elsewhere. I'm sure there will be more to discuss on that front next week.14:03
TheJulia:33714:03
TheJuliadoh14:03
*** dyasny has joined #openstack-ironic14:03
*** aignatov has quit IRC14:03
*** aignatov has joined #openstack-ironic14:06
*** retr0h has joined #openstack-ironic14:08
mrtenioI am having a problem with Ironic today, I have installed it from master, and it is failing to get the ironic API URL from keystone, showing me the error: "Keystone request failed: An auth plugin is required to determine endpoint URL: MissingAuthPlugin: An auth plugin is required to determine endpoint URL". I was able to work around this problem by setting the api_url parameter under conductor within ironic.conf. Any tip on why? Thanks.14:08
TheJuliamrtenio: We are working to retrieve URLs from the service catalog, and some of that involves changes in keystoneauth calls14:10
TheJuliaI'm wondering if we're missing a default or if the keystoneauth1 library is out of date, or... I don't know.  I've always found those to be not fun problems to troubleshoot.14:10
mrtenioTheJulia, hmm, I am gonna take a look to the library. For now, api_url set manually is helping :)14:12
TheJuliamrtenio: you may want to check the current sample.conf and see if there is a user definable auth plugin setting.  It should be "password" if memory serves14:13
TheJuliaor password should work.14:13
TheJuliaTruthfully, I need quite a bit of liquor whenever I have to dig into such issues.14:14
mrtenioTheJulia haha, I wouldn't mind some of that :P14:14
mrtenioI looked that too, auth_type is set to password.14:14
TheJuliaso auth_type is not the same as plugin14:15
mrteniohmm, isn't auth_type deprecated?14:15
*** fellypefca has quit IRC14:15
TheJuliamrtenio: I do believe it was in favor of plugins a long time ago14:16
TheJuliaat least with os-client-config and keystoneauth114:16
*** litao__ has quit IRC14:17
TheJuliamrtenio: if you file a bug due to this, please include a sanitized version of your ironic.conf file and a sanitized listing of your services in your keystone service catalog so we can mentally piece together context14:17
mrtenioTheJulia, ok, Thank you again :)14:18
TheJuliamrtenio: for what its worth, a good chunk of this is should be updated prior to the end of the cycle.  At least, I'm hoping :)14:18
TheJuliaWe had a heard break due to a glance change the week before last that required us to merge an intermediate fix.  I suspect that is where your running into trouble14:19
*** r-daneel has joined #openstack-ironic14:20
*** shvepsy has quit IRC14:20
mrtenioYeah, it is sad it needed to happen :/ When I am filling up the bug I will look it deeper.14:21
TheJuliamilan: sambetts: I updated the links on the priorities list to include the topic queue links for the docs stuffs on each queue14:22
milanTheJulia, makes sense, thanks14:22
*** aszeszo has quit IRC14:22
* milan still fights with nesting index.rst 14:23
TheJuliamilan: fun! for the UI I figured "Oh, I might as well turn on autodoc for the python parts.... oh what a mistake.14:24
milanheh :)14:24
milanyeah14:24
openstackgerritSam Betts proposed openstack/ironic master: Update README to point at new doc location  https://review.openstack.org/47987714:24
openstackgerritSam Betts proposed openstack/ironic master: Move ironic dbsync tool docs into doc/source/cli  https://review.openstack.org/47987614:24
openstackgerritSam Betts proposed openstack/ironic master: Move doc/source/dev to doc/source/contributor  https://review.openstack.org/47987514:24
openstackgerritSam Betts proposed openstack/ironic master: Move operator docs into into doc/source/admin  https://review.openstack.org/47987414:24
openstackgerritSam Betts proposed openstack/ironic master: Move install guide into new doc/source/install location  https://review.openstack.org/47987314:24
openstackgerritSam Betts proposed openstack/ironic master: Move api-ref into new doc/source/api location  https://review.openstack.org/47987214:24
sambettsalllllll the patches ^14:24
milanboom :D14:24
*** ricardoas has joined #openstack-ironic14:25
TheJulia\o/14:26
milanguys, based on https://specs.openstack.org/openstack/docs-specs/specs/pike/os-manuals-migration.html#migration-process the contributor should have an index.rst that should be included in the top-level doc/source/index.rst14:32
milanbut I can't make that work14:32
milan:-/14:32
openstackgerritSam Betts proposed openstack/ironic master: Update README to point at new doc location  https://review.openstack.org/47987714:32
openstackgerritSam Betts proposed openstack/ironic master: Move ironic dbsync tool docs into doc/source/cli  https://review.openstack.org/47987614:32
openstackgerritSam Betts proposed openstack/ironic master: Move doc/source/dev to doc/source/contributor  https://review.openstack.org/47987514:32
openstackgerritSam Betts proposed openstack/ironic master: Move operator docs into into doc/source/admin  https://review.openstack.org/47987414:32
openstackgerritSam Betts proposed openstack/ironic master: Move install guide into new doc/source/install location  https://review.openstack.org/47987314:32
openstackgerritSam Betts proposed openstack/ironic master: switch from oslosphinx to openstackdocstheme  https://review.openstack.org/47987114:32
openstackgerritSam Betts proposed openstack/ironic master: Move api-ref into new doc/source/api location  https://review.openstack.org/47987214:32
sambettsrebase :( ^14:32
sambettsmilan: what error are you seeing??14:33
milandoc/source/contributor/index.rst:1:toctree contains reference to nonexisting document 'developer'14:33
TheJuliamilan: post what you have and I bet sam or I could spot it immediately14:33
milanTheJulia, sambetts thx14:33
* milan posts14:33
sambettsmilan: I add my version of that here: https://review.openstack.org/#/c/479875/6/doc/source/index.rst14:33
patchbotpatch 479875 - ironic - Move doc/source/dev to doc/source/contributor14:33
*** joe817 has joined #openstack-ironic14:33
milansambetts, yeah that way it works for me too14:34
milanbut it "bypasses" the requirement of the sub-index.rst, no?14:34
sambettssub-index.rst14:34
sambetts?14:34
TheJuliayour still supposed to put the index.rst in he other folder and load it14:34
TheJuliabut you can make it a hidden toc14:34
TheJuliaso it still gets loaded/parsed all together14:34
milanhttps://specs.openstack.org/openstack/docs-specs/specs/pike/os-manuals-migration.html#migration-process see bullet 514:35
milanTheJulia, so what exactly to put in the sub-index?14:35
*** rbartal has quit IRC14:35
TheJuliamilan: one moment14:35
TheJuliahttps://review.openstack.org/#/c/479981/3/doc/source/contributor/index.rst14:36
patchbotpatch 479981 - ironic-ui - Move ironic-ui contributor docs14:36
sambettsmilan: bullet 5 just refers to having a link in the doc/source/index.rst to the subdirectories index.rst14:36
TheJuliahttps://review.openstack.org/#/c/479981/3/doc/source/index.rst14:36
patchbotpatch 479981 - ironic-ui - Move ironic-ui contributor docs14:36
*** udesale has quit IRC14:36
milanhmmm14:37
*** joe817 has quit IRC14:37
sambettsmilan: the idea is that the top level index should saying something like "Contributor Documentation here" with a link to contrbutor/index.rst and then contributor/index.rst has all the contributor documentation links14:37
milansambetts, that makes sense to me, just unsure how to implement ;)14:38
TheJuliasambetts: I actually suspect the top level index is the subfolder.rst, but without the linking plan being visible to us, I'm not sure :\14:38
TheJuliaerr, that doesn't make sense14:38
TheJuliaI mean, the descriptive headers14:39
sambettsso the top level index.rst literally becomes like 5 links to api, admin, contributor, install etc14:39
TheJuliaYup14:39
milansambetts yeah just not through the ..include:: directive I guess as that doesn't work for me14:39
sambettsmilan: no, doc/source/index should be a toctree and doc/source/contributor/index should also be a toctree14:40
milansambetts, correct14:41
milanhow to make it step-wise for inspector though: https://raw.githubusercontent.com/openstack/ironic-inspector/master/doc/source/index.rst14:41
*** mkrai has joined #openstack-ironic14:42
TheJuliamilan: contributor/index14:43
* milan tries14:44
sambettsmilan: rename contributing.rst to contributor/index.rst14:45
TheJuliaThat is also a required step :)14:45
sambettsmilan: then in doc/source/index change it to contributor/index14:45
sambettsmilan: then you'll have to add an additonall ../ to the include line in contributor/index14:46
* milan applied & fingers crossed compiling docs14:47
*** karthiks is now known as karthiks_afk14:47
milanok no more errors, that's cool :D14:48
milanthis is the magic I was missing .. toctree:: inside of a .. toctree::14:49
milanimplicit14:50
*** jaganathan has quit IRC14:50
* milan fine-tunes :maxdepth:14:50
*** links has quit IRC14:59
TheJuliawoot, closer with autodoc!15:00
openstackgerritSam Betts proposed openstack/ironic master: Update README to point at new doc location  https://review.openstack.org/47987715:02
openstackgerritSam Betts proposed openstack/ironic master: Move ironic dbsync tool docs into doc/source/cli  https://review.openstack.org/47987615:02
openstackgerritSam Betts proposed openstack/ironic master: Move doc/source/dev to doc/source/contributor  https://review.openstack.org/47987515:02
openstackgerritSam Betts proposed openstack/ironic master: Move operator docs into into doc/source/admin  https://review.openstack.org/47987415:02
openstackgerritSam Betts proposed openstack/ironic master: Move install guide into new doc/source/install location  https://review.openstack.org/47987315:02
openstackgerritSam Betts proposed openstack/ironic master: Move api-ref into new doc/source/api location  https://review.openstack.org/47987215:02
sambetts^ had to fix a pep8 issue :(15:03
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Introducing the contributor content  https://review.openstack.org/48022215:03
TheJuliadoh15:03
* milan still has issue with an extra bullet but I guess it will go away once the struct of the https://review.openstack.org/#/c/480222/1/doc/source/index.rst changes into a single toc 15:04
patchbotpatch 480222 - ironic-inspector - Introducing the contributor content15:04
sambettsmilan: extra bullet?15:05
milansambetts, contributor.contributor.developer15:05
milanand contributor.contributor.python_api15:05
milanbecause of how it's nested :-/15:05
*** baoli has quit IRC15:05
sambettsmilan: contributor/developer.rst should be contributor/index.rst15:06
milansambetts I thought that developer.rst should just link to ../../../CONTRIBUTING.rst and do the magic in index.rst15:06
*** bfournie has quit IRC15:06
milansuch as further bullets, no?15:07
*** bfournie has joined #openstack-ironic15:07
*** mrchao has quit IRC15:07
*** mjura has quit IRC15:07
milanoops, missed the index15:08
*** baoli has joined #openstack-ironic15:08
*** skazi__ has joined #openstack-ironic15:09
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Introducing the contributor content  https://review.openstack.org/48022215:09
milansambetts, sorry, I missed the index in the commit15:09
*** bfournie has quit IRC15:09
*** links has joined #openstack-ironic15:11
*** skazi__ has quit IRC15:12
*** sambetts has quit IRC15:14
*** jaypipes has joined #openstack-ironic15:17
*** joe817 has joined #openstack-ironic15:17
*** sambetts has joined #openstack-ironic15:19
*** crushil_ has joined #openstack-ironic15:21
*** rama_y has joined #openstack-ironic15:22
*** aszeszo has joined #openstack-ironic15:22
*** joe817 has quit IRC15:23
*** dyasny has quit IRC15:28
*** dyasny has joined #openstack-ironic15:29
*** ricardoas has quit IRC15:32
*** xavierr has quit IRC15:32
*** mrtenio has quit IRC15:32
sambettsmilan: http://paste.openstack.org/show/614399/ <----- I would do something like  this15:32
milansambetts also an option15:33
* milan going to post a top-level patch to flip the main index into a single toc15:33
milanto see how it works15:33
*** mrtenio has joined #openstack-ironic15:34
milancan rebase though15:34
sambettsmilan: the resulting page is much nicer with that diff because you end up with the "How to Contribute" header from the CONTRIBUTING.rst file becoming the main header in doc/source/contrbutor/index.rst15:35
milanhmmmmm15:35
milanok, will update, anyways I guess I have to switch the main index to a single toc, no?15:37
sambettsmilan: unsure what the final form of the top level will be, but possibly :/15:37
milanyeah15:38
sambettsas long as you can get to all the docs from the top level then I think we're ok for now15:38
*** joe817 has joined #openstack-ironic15:39
*** xavierr has joined #openstack-ironic15:39
*** crushil_ has quit IRC15:40
openstackgerritJulia Kreger proposed openstack/ironic-ui master: Move ironic-ui contributor docs  https://review.openstack.org/47998115:43
openstackgerritJulia Kreger proposed openstack/ironic-ui master: Add python api docs to contributor docs  https://review.openstack.org/48024015:43
*** joe817 has quit IRC15:43
*** ricardoas has joined #openstack-ironic15:44
* vdrok tries to get into the docs changes15:45
*** d0ugal has quit IRC15:46
vdroksambetts: shouldn't openstackdocstheme be added into extensions instead of importing? https://docs.openstack.org/openstackdocstheme/latest/15:47
TheJuliavdrok: I think it is a relic of prior to this week. It has been done both ways in the various changes that have been pushed out for theme changes15:48
*** chlong_ has quit IRC15:48
vdrokI mean, I suppose we can get rid of all those ifs? at least the glance change does not have any -- https://review.openstack.org/#/c/477001/4/doc/source/conf.py15:49
patchbotpatch 477001 - glance - use openstackdocstheme html context (MERGED)15:49
*** marios has quit IRC15:49
vdroksambetts: TheJulia ^15:50
TheJuliavdrok: The patches I've posted are without the ifs for a few of the repos. O15:51
TheJuliaerr15:51
TheJuliaI'm in support of it, but I don't really see a difference at this point15:52
vdrokOK. let's just not forget about it afterwards :)15:52
TheJuliaIt is in the fun place everyone forgets to update.  I had to go update ironic-ui's doc/source/conf.py with quite a bit of code from the cookie cutter15:53
sambettsyeah and looking through the patches pushed up for doc-migration some do it with the if and some without15:53
sambettsso I'm not sure what the final form should be15:54
vdrokI hope the one without the ifs :) glance change depended on  https://review.openstack.org/#/c/476997/5 which is merged15:55
patchbotpatch 476997 - openstackdocstheme - automate some of the extension setup (MERGED)15:55
*** lucasagomes is now known as lucas-afk15:56
TheJuliastepping away for a little while15:56
*** rcernin has quit IRC15:58
milansambetts so it didn't play nicely :-/ the .. toctree:: doesn't "import" the formatting of the headlines15:59
milanI'll push the whole chain but I guess it's a [WIP]16:00
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Introducing the installation content  https://review.openstack.org/48024916:03
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Introducing the user content  https://review.openstack.org/48025016:03
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Docs migration conclusion  https://review.openstack.org/48025116:03
milanTheJulia, sambetts thanks for help on this, unfortunately I gotta be running for my PTO till Monday :-/16:05
sambettsmilan: no problem o/16:07
milannight Ironic! :)16:07
*** milan has quit IRC16:07
*** e0ne has quit IRC16:08
openstackgerritSam Betts proposed openstack/ironic master: Update README to point at new doc location  https://review.openstack.org/47987716:13
openstackgerritSam Betts proposed openstack/ironic master: Move ironic dbsync tool docs into doc/source/cli  https://review.openstack.org/47987616:13
openstackgerritSam Betts proposed openstack/ironic master: Move api-ref into new doc/source/api location  https://review.openstack.org/47987216:13
openstackgerritSam Betts proposed openstack/ironic master: Move python API reference into contributor/api  https://review.openstack.org/48025316:13
* sambetts --> home 16:13
sambettsnight all o/16:13
*** sambetts is now known as sambetts|afk16:13
*** alex_xu has quit IRC16:14
*** alex_xu has joined #openstack-ironic16:17
vdrokgood night milan and sambetts|afk16:24
*** makowals has quit IRC16:25
*** racedo has quit IRC16:30
*** mgoddard__ has joined #openstack-ironic16:30
*** racedo has joined #openstack-ironic16:30
*** racedo has quit IRC16:30
*** mgoddard_ has quit IRC16:32
*** mgoddard_ has joined #openstack-ironic16:33
*** makowals has joined #openstack-ironic16:34
*** ralonsoh has quit IRC16:35
*** mgoddard__ has quit IRC16:36
openstackgerritRodrigo Farias de Macedo proposed openstack/ironic master: Migrate oneview driver to use python-hpOneView  https://review.openstack.org/46687216:36
*** fellypefca has joined #openstack-ironic16:37
*** mgoddard__ has joined #openstack-ironic16:40
openstackgerritRodrigo Farias de Macedo proposed openstack/ironic master: Migrate oneview driver to use python-hpOneView  https://review.openstack.org/46687216:41
*** mgoddard_ has quit IRC16:42
*** mgoddard_ has joined #openstack-ironic16:44
*** mgoddard__ has quit IRC16:46
*** jaganathan has joined #openstack-ironic16:49
*** amarao has quit IRC16:51
*** links has quit IRC16:54
*** joe817 has joined #openstack-ironic17:00
*** aszeszo has quit IRC17:00
*** aszeszo1 has joined #openstack-ironic17:00
*** arahal_ has joined #openstack-ironic17:00
*** e0ne has joined #openstack-ironic17:01
*** r-daneel has quit IRC17:04
*** joe817 has quit IRC17:04
*** jaganathan has quit IRC17:08
* vdrok leaves too17:08
vdrokgood night!17:08
*** tesseract has quit IRC17:17
*** zhugaoxiao has quit IRC17:18
*** zhugaoxiao has joined #openstack-ironic17:19
openstackgerritIlya Etingof proposed openstack/ironic master: Removed nonexistent option from quickstart snippet  https://review.openstack.org/48027517:33
openstackgerritHugo Nicodemos proposed openstack/ironic master: Migrate oneview driver to use python-hpOneView  https://review.openstack.org/46687217:48
*** ChubYann has joined #openstack-ironic17:49
*** arahal_ has quit IRC17:55
openstackgerritHugo Nicodemos proposed openstack/ironic master: Migrate python-oneviewclient validations to Ironic OneView drivers  https://review.openstack.org/46842817:59
openstackgerritHugo Nicodemos proposed openstack/ironic master: Remove python-oneviewclient from Ironic OneView drivers  https://review.openstack.org/48028417:59
*** yolanda has quit IRC18:05
*** fellypefca has quit IRC18:18
*** joe817 has joined #openstack-ironic18:22
*** joe817 has quit IRC18:26
*** mgoddard_ has quit IRC18:48
*** e0ne_ has joined #openstack-ironic18:51
*** joe817 has joined #openstack-ironic18:53
*** e0ne_ has quit IRC18:53
*** e0ne has quit IRC18:53
*** e0ne has joined #openstack-ironic18:54
*** livelace has quit IRC18:57
*** e0ne has quit IRC19:00
*** rbudden has quit IRC19:04
openstackgerritHugo Nicodemos proposed openstack/ironic master: Migrate python-oneviewclient validations to Ironic OneView drivers  https://review.openstack.org/46842819:04
*** e0ne has joined #openstack-ironic19:06
*** jaypipes has quit IRC19:14
*** etingof has quit IRC19:19
*** baoli has quit IRC19:21
*** baoli has joined #openstack-ironic19:22
*** aszeszo has joined #openstack-ironic19:25
*** aszeszo1 has quit IRC19:25
*** e0ne has quit IRC19:33
*** mgoddard_ has joined #openstack-ironic19:38
*** etingof has joined #openstack-ironic19:41
*** rama_y has quit IRC19:49
*** zhugaoxiao has quit IRC19:59
*** hw_wutianwei has quit IRC20:04
*** hw_wutianwei has joined #openstack-ironic20:04
*** openstackgerrit has quit IRC20:18
*** mvpnitesh has quit IRC20:36
*** mvpnitesh has joined #openstack-ironic20:45
*** mvpnitesh has quit IRC20:53
*** mvpnitesh has joined #openstack-ironic20:53
*** rbudden has joined #openstack-ironic20:57
*** mvpnitesh has quit IRC20:57
*** mvpnitesh has joined #openstack-ironic20:58
*** mvpnitesh has quit IRC21:02
*** mvpnitesh has joined #openstack-ironic21:02
*** openstackgerrit has joined #openstack-ironic21:07
openstackgerritStenio Araujo proposed openstack/ironic master: [WIP] Using non-persistent boot in PXE interface  https://review.openstack.org/47992721:07
*** mvpnitesh has quit IRC21:09
*** aszeszo has quit IRC21:12
*** aszeszo1 has joined #openstack-ironic21:12
*** mrtenio is now known as mrtenio|afk21:19
*** racedo has joined #openstack-ironic21:40
*** joe817 has quit IRC21:44
*** joe817 has joined #openstack-ironic21:45
*** e0ne has joined #openstack-ironic21:46
*** e0ne_ has joined #openstack-ironic21:52
*** racedo has quit IRC21:54
*** e0ne has quit IRC21:55
*** mgoddard_ has quit IRC21:56
*** e0ne_ has quit IRC21:58
*** e0ne has joined #openstack-ironic21:59
*** hjensas has quit IRC22:00
*** e0ne has quit IRC22:02
*** e0ne has joined #openstack-ironic22:04
*** aszeszo1 has quit IRC22:19
*** xnox has quit IRC22:20
*** xnox has joined #openstack-ironic22:20
*** adisky__ has quit IRC22:20
*** xnox has quit IRC22:21
*** xnox has joined #openstack-ironic22:21
openstackgerritMerged openstack/ironic master: Add REST API for volume connector and volume target operation  https://review.openstack.org/21458622:33
*** rbudden has quit IRC22:59
*** e0ne has quit IRC23:09
*** e0ne has joined #openstack-ironic23:10
*** e0ne has quit IRC23:10
*** e0ne has joined #openstack-ironic23:11
*** e0ne has quit IRC23:11
*** e0ne has joined #openstack-ironic23:12
*** e0ne has quit IRC23:12
*** e0ne has joined #openstack-ironic23:12
*** e0ne has quit IRC23:13
*** e0ne has joined #openstack-ironic23:13
*** e0ne has quit IRC23:14
*** joe817 has quit IRC23:26
*** Nisha_Agarwal has joined #openstack-ironic23:35
*** gcb has quit IRC23:39
*** sasha2 has quit IRC23:44
*** mjturek has joined #openstack-ironic23:58

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