Wednesday, 2016-06-29

*** sacharya_ has quit IRC00:00
*** piet has quit IRC00:03
*** piet has joined #openstack-ironic00:12
*** jaybeale has joined #openstack-ironic00:19
*** sabeen has joined #openstack-ironic00:25
*** rama_y has joined #openstack-ironic00:29
*** sabeen has quit IRC00:30
*** jaybeale has joined #openstack-ironic00:33
*** rajinir has quit IRC00:34
*** hoangcx has joined #openstack-ironic00:39
*** gfhellma has quit IRC00:40
*** mbound has joined #openstack-ironic00:44
*** mbound has quit IRC00:49
*** sabeen has joined #openstack-ironic00:50
*** rama_y has quit IRC00:51
*** klindgren has joined #openstack-ironic00:52
*** jaybeale has quit IRC01:06
*** sacharya has joined #openstack-ironic01:11
*** Sukhdev has quit IRC01:18
*** gabriel-bezerra has quit IRC01:18
*** xavierr has quit IRC01:18
*** nicodemos has quit IRC01:18
*** liliars has quit IRC01:18
*** clenimar has quit IRC01:18
*** caiobo-afk has quit IRC01:18
*** openstack has joined #openstack-ironic01:24
*** raildo is now known as raildo-afk01:27
*** raildo-afk is now known as raildo01:27
*** xavierr has joined #openstack-ironic01:30
*** clenimar has joined #openstack-ironic01:30
*** PollyZ has joined #openstack-ironic01:31
*** gabriel-bezerra has joined #openstack-ironic01:33
*** liliars has joined #openstack-ironic01:33
*** caiobo-afk has joined #openstack-ironic01:33
*** nicodemos has joined #openstack-ironic01:35
*** piet has quit IRC01:37
*** PollyZ_ has joined #openstack-ironic01:40
*** PollyZ has quit IRC01:42
*** PollyZ_ has quit IRC01:45
*** PollyZ has joined #openstack-ironic01:50
*** PollyZ has quit IRC02:01
*** klindgren has quit IRC02:01
*** thrash is now known as thrash|g0ne02:05
openstackgerritRuby Loo proposed openstack/python-ironicclient: Add invalid attribute names to exception  https://review.openstack.org/33528102:22
*** baoli_ has quit IRC02:25
*** sacharya_ has joined #openstack-ironic02:26
*** sacharya has quit IRC02:28
*** rloo has quit IRC02:43
*** PollyZ has joined #openstack-ironic02:46
*** kylin-chou has joined #openstack-ironic02:50
*** yuanying has quit IRC02:50
*** kylin-chou has left #openstack-ironic02:55
*** fragatina has quit IRC02:55
*** fragatina has joined #openstack-ironic02:55
*** PollyZ has quit IRC02:57
*** PollyZ has joined #openstack-ironic02:57
openstackgerritMichael Davies proposed openstack/python-ironicclient: Add env var for version cache timeout  https://review.openstack.org/33438703:14
mrdaThis time for sure, Rocky!03:15
*** krtaylor has joined #openstack-ironic03:18
*** PollyZ has quit IRC03:19
*** PollyZ has joined #openstack-ironic03:20
*** krtaylor has quit IRC03:20
*** sacharya_ has quit IRC03:20
*** sacharya has joined #openstack-ironic03:20
*** rbudden has quit IRC03:27
*** PollyZ has quit IRC03:28
*** PollyZ has joined #openstack-ironic03:33
*** MattMan has quit IRC03:40
*** MattMan has joined #openstack-ironic03:41
*** yuanying has joined #openstack-ironic03:47
*** sacharya has quit IRC03:49
*** sacharya has joined #openstack-ironic03:49
*** appu103_ has joined #openstack-ironic03:56
*** PollyZ has quit IRC04:02
*** PollyZ has joined #openstack-ironic04:07
*** joprovost has quit IRC04:18
*** links has joined #openstack-ironic04:22
mrdajlvillal: thanks for the review.  Comment already addressed :)04:30
*** Sukhdev has joined #openstack-ironic04:34
openstackgerritDao Cong Tien proposed openstack/ironic: Nova-compatible serial console: socat console_utils  https://review.openstack.org/32816804:38
*** shirly has quit IRC04:43
*** sabeen has quit IRC04:45
*** rcernin has joined #openstack-ironic04:53
*** pcaruana has quit IRC04:58
*** ChubYann has quit IRC04:59
*** sacharya has quit IRC05:00
*** PollyZ has quit IRC05:01
*** sabeen has joined #openstack-ironic05:01
*** Goneri has joined #openstack-ironic05:05
*** M00nr41n has quit IRC05:06
jlvillalmrda, Thanks. But it doesn't seem like the os.environ.get() is checked.05:11
mrdajlvillal: just realised my mistake.  New version about to be uploaded05:11
jlvillalmrda, Thanks05:12
openstackgerritMichael Davies proposed openstack/python-ironicclient: Add env var for version cache timeout  https://review.openstack.org/33438705:12
mrdajlvillal: I misread your original comment - I thought you wanted a check on the warn log only. My apoligies.05:12
jlvillalmrda, I misread comments all the time!05:14
jlvillalmrda, Have a great day. I'm off to bed.05:14
mrdajlvillal: have a good night!05:15
jlvillalmrda, Why MOCK.ANY instead of DEFAULT_EXPIRY?05:16
jlvillalJust curious05:16
mrdaoh, because I'm not testing whether the call was with a defaulted value05:17
*** sabeen has quit IRC05:17
mrdarather that the env var is referred to05:17
mrdaan equivalent implementation would be to use the stadard get(one param) function, in a try catch, and grab the key error.05:17
mrdathat's implementation detail.05:17
jlvillalmrda, Okay. works for me :)05:17
mrdait would have been less efficient, but might have been done for reasons05:18
jlvillalmrda, +205:18
mrdahope that makes sense :)05:18
mrda\o/05:18
*** sabeen has joined #openstack-ironic05:19
*** moshele has joined #openstack-ironic05:23
*** rcernin has quit IRC05:33
*** dtantsur|afk is now known as dtantsur05:40
dtantsurMorning Ironic05:40
mrdahey dtantsur05:46
moshelemorning dtantsur, mrda05:53
*** edand has joined #openstack-ironic05:54
*** mjura has joined #openstack-ironic05:55
*** Goneri has quit IRC05:57
*** mjura has quit IRC06:00
*** mjura has joined #openstack-ironic06:00
*** edand has quit IRC06:01
*** praneshp has joined #openstack-ironic06:02
*** edand has joined #openstack-ironic06:03
*** rcernin has joined #openstack-ironic06:06
*** M00nr41n has joined #openstack-ironic06:08
*** sabeen has quit IRC06:10
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Use run_process in the devstack plugin  https://review.openstack.org/33532106:14
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Introducing Inspector Grenade testing  https://review.openstack.org/32766706:15
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Use run_process in the devstack plugin  https://review.openstack.org/33532106:15
*** pcaruana has joined #openstack-ironic06:16
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Introducing Inspector Grenade testing  https://review.openstack.org/32766706:17
*** jtomasek_ has joined #openstack-ironic06:19
*** pcaruana is now known as pcaruana|afk|06:19
*** M00nr41n has quit IRC06:22
*** M00nr41n has joined #openstack-ironic06:23
*** Sukhdev has quit IRC06:24
*** _vsaienko has joined #openstack-ironic06:25
*** fragatina has quit IRC06:29
*** M00nr41n has quit IRC06:33
*** M00nr41n has joined #openstack-ironic06:34
*** amotoki has joined #openstack-ironic06:39
*** _vsaienko has quit IRC06:40
*** rbartal has joined #openstack-ironic06:41
*** ccamacho has quit IRC06:41
*** praneshp has quit IRC06:41
*** tiendc has joined #openstack-ironic06:41
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [WIP] Use new agent API if available  https://review.openstack.org/33452306:42
tiendclucasagomes: rloo:  Hi, could you help review the nova patch for serial console? Link https://review.openstack.org/#/c/32815706:44
*** phuongnh has joined #openstack-ironic06:45
*** ccamacho has joined #openstack-ironic06:46
*** pcaruana|afk| is now known as pcaruana06:49
*** itamarl has joined #openstack-ironic06:55
*** piet has joined #openstack-ironic07:02
*** appu103_ is now known as aparnav07:07
pas-hamorning Ironic07:08
*** tesseract- has joined #openstack-ironic07:09
dtantsurmorning pas-ha07:11
pas-hamorning dtantsur07:21
*** e0ne has joined #openstack-ironic07:25
*** e0ne has quit IRC07:28
*** piet has quit IRC07:29
*** ifarkas has joined #openstack-ironic07:30
*** amotoki has quit IRC07:37
*** amotoki has joined #openstack-ironic07:43
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [WIP] Use new agent API if available  https://review.openstack.org/33452307:44
dtantsurifarkas, morning! please review a tiny patch https://review.openstack.org/#/c/335321/ it's blocking grenade work07:46
*** jtomasek has joined #openstack-ironic07:52
ifarkasmorning dtantsur and all07:55
ifarkasdtantsur, will do07:55
*** jtomasek_ has quit IRC07:55
*** hshiina has joined #openstack-ironic07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** yogi has joined #openstack-ironic08:06
*** ohamada has joined #openstack-ironic08:11
openstackgerritMerged openstack/ironic: Fix typo in inspection.rst  https://review.openstack.org/33521908:13
*** ohamada has quit IRC08:14
*** ohamada has joined #openstack-ironic08:15
lucasagomestiendc, hi there, sure will do08:27
openstackgerritSaravanan KR proposed openstack/ironic-python-agent: Set the kernel args to the nodes at bootloader instal  https://review.openstack.org/33505208:29
openstackgerritMerged openstack/ironic-inspector: Use run_process in the devstack plugin  https://review.openstack.org/33532108:29
dtantsurmorning lucasagomes08:29
lucasagomesdtantsur, hi there08:29
lucasagomesmorning all08:29
tiendclucasagomes: thanks08:29
*** klindgren has joined #openstack-ironic08:29
gbraadhi all08:29
dtantsurhi gbraad, g'afternoon I guess?08:30
*** electrofelix|afk is now known as electrofelix08:31
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Use run_process in the devstack plugin  https://review.openstack.org/33536808:31
dtantsurmitaka backport will hopefully fix inspector grenade ^^^08:32
gbraaddtantsur, nearly time to go home 1630 now :-P08:33
dtantsurgood for you :) I had to wake up at 8am, which is night for me08:34
gbraadLOL, I used to work from home too... until they changed regulations here. very counterproductive in a way.08:35
*** stendulker has joined #openstack-ironic08:36
openstackgerritYuiko Takada Mori proposed openstack/ironic-specs: Follow-up patch of 75290c72f7c3138bfdab151728718f771dfb9685  https://review.openstack.org/33537808:39
*** dmk0202 has joined #openstack-ironic08:39
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [WIP] Use new agent API if available  https://review.openstack.org/33452308:49
*** derekh has joined #openstack-ironic08:52
*** klindgren has quit IRC08:59
*** athomas has joined #openstack-ironic09:01
*** chlong has quit IRC09:04
*** pece has joined #openstack-ironic09:08
moshelehi, can ironic-cores review my Adding InfiniBand Support to ironic spec  - https://review.openstack.org/#/c/265635/09:12
openstackgerritDavid Edery proposed openstack/ironic: Fix non-deployed nodes due to short nodes.instance_info  https://review.openstack.org/33496709:15
*** chlong has joined #openstack-ironic09:17
*** mgoddard has joined #openstack-ironic09:17
sergekmorning pas-ha , dtantsur , moshele , mrda09:18
openstackgerritMerged openstack/ironic: Follow-up patch of 9a1aeb76da2ed53e042a94ead8640af9374a10bf  https://review.openstack.org/33473109:31
mosheledtantsur, lucasagomes: can you review the infiniband spec https://review.openstack.org/#/c/265635/09:32
dtantsursorry, don't have time right now. will try to get to it later09:32
*** amotoki has quit IRC09:37
*** yuriy_n17 has joined #openstack-ironic09:38
*** hshiina has quit IRC09:38
*** yuriy_n17 has left #openstack-ironic09:40
*** amotoki has joined #openstack-ironic09:50
lucasagomesmoshele, question, what you mean by "In the PXE boot process the device driver truncate the GUID to fit the Ethernet MAC address (48 bits) according to a vendor specific way" ?09:54
lucasagomesthat's in the hardware inspection section09:54
vdrokmorning ironic!09:54
*** e0ne has joined #openstack-ironic09:58
yibovdrok, would you have a look at https://review.openstack.org/#/c/322632. sorry for bothering in the morning :)09:59
moshelelucasagomes:  in infiniband  GUID has similar concept of MAC in Ethernet, but to support PXE over inifinband you have to covert the GUID to "inifinband MAC" to support create the pxe file09:59
mosheleucasagomes:  converting GUID to  "inifinband MAC" is vendor specific10:00
lucasagomesmoshele, right, that part is fine... but what you mean by the "device driver" ?10:00
openstackgerritMerged openstack/ironic-inspector: Use run_process in the devstack plugin  https://review.openstack.org/33536810:00
vdrokyibo, np, will do today10:00
lucasagomese.g, inspector will setup it's own DHCP server right? With a wildcard to boot uknown MAC addresses10:00
lucasagomesnow a machine with inifiniband is booting, how the GUID gets converted to a MAC address?10:01
lucasagomesit's done automatically but the PXE image?10:01
moshelelucasagomes: you need to burn infiniband pxe  Firmware on the physical server before you can do inspection10:03
moshelelucasagomes: the you can see in the bios that you are able to boot from Infiniband NIC10:03
lucasagomesmoshele, a-ha, right so you have to flash the nic ROM with a specific image10:03
moshelelucasagomes: yes10:05
lucasagomesmoshele, and for deployment ? same history ?10:05
moshelelucasagomes: you just burn the Firmware once before you start using the  physical server for inspection and deployment.10:07
lucasagomesmoshele, ok, would be good to have that info in the spec... cause it talks about converting the GUID to the MAC and it's confusing to understand how that actually happens10:08
moshelelucasagomes: ok I will update the spec10:10
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844010:10
*** athomas has quit IRC10:11
lucasagomesmoshele, other than that it LGTM10:11
mgouldmorning Ironic10:12
lucasagomesmoshele, it doesn't seem to do much in Ironic, the only thing that I can see as a change in the ironic project itself is that we need to pass the client id to neutron extra_dhcp_opts10:12
lucasagomesmoshele, correct?10:12
*** e0ne has quit IRC10:12
*** hoangcx has quit IRC10:13
openstackgerritMerged openstack/ironic: Introduce new driver call and RPC for heartbeat  https://review.openstack.org/33289010:16
*** dedery has joined #openstack-ironic10:16
*** athomas has joined #openstack-ironic10:17
dederyvsaienko: hi, thank you for the review10:17
moshelelucasagomes: yes, here is an example of the ironic change https://review.openstack.org/#/c/264263/10:22
moshelelucasagomes: The new design move most of the logic to the IPA10:23
*** amotoki has quit IRC10:23
dederymoshele: hi :) first time in #ironic, nice to see you here10:23
moshelelucasagomes: you can see it here https://review.openstack.org/#/c/334816/10:23
moshelededery: you too :)10:24
*** jtomasek has quit IRC10:27
lucasagomesmoshele, cool stuff10:32
*** thrash|g0ne is now known as thrash10:32
*** amotoki has joined #openstack-ironic10:33
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic driver-list command  https://review.openstack.org/33215010:34
*** stendulker has quit IRC10:35
*** sambetts|afk is now known as sambetts|deadlin10:36
*** sambetts|deadlin is now known as sambetts|halfher10:38
*** sambetts|halfher is now known as sambetts|HAK10:38
*** sambetts|HAK is now known as sambetts|HFK10:38
sambetts|HFKthats better :-P10:38
sambetts|HFKMorning all10:38
sambetts|HFKnot really here again today10:39
lucasagomessambetts|HFK, morning10:40
sambetts|HFKtrying to think if a better postfix to discribe my situation of may or may not respond :-P10:40
openstackgerritMerged openstack/ironic: Add missing translation marker to clear_node_target_power_state  https://review.openstack.org/33496910:41
lucasagomessambetts|HFK: "sam|schrodinger": here, not here or both :-)10:41
openstackgerritMerged openstack/ironic: Improve tests for driver's parse_driver_info()  https://review.openstack.org/33466610:41
sambetts|HFKlucasagomes: haha if that fits I'm using it10:41
lucasagomeshah10:41
*** sambetts|HFK is now known as sam|schrodinger10:41
lucasagomeshere you go10:41
sam|schrodinger\o/10:41
sam|schrodingerlucasagomes: perhaps |CAT should indicate half here half not10:42
lucasagomesyeah, that would do it too10:42
*** sam|schrodinger is now known as sambetts|cat10:42
sambetts|cathaha :-P inside jokes10:42
lucasagomeslol the best ones10:43
vdrokmorning sambetts|cat and lucasagomes :)10:44
lucasagomesvdrok, morning :D10:44
*** amotoki has quit IRC10:44
*** tiendc has quit IRC10:46
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Implement new heartbeat for AgentDeploy  https://review.openstack.org/33396410:49
openstackgerritDmitry Tantsur proposed openstack/ironic: Promote agent vendor passthru to core API  https://review.openstack.org/33001810:49
*** _vsaienko has joined #openstack-ironic10:50
*** ccamacho is now known as ccamacho|lunch10:58
*** amotoki has joined #openstack-ironic11:02
*** jtomasek has joined #openstack-ironic11:08
mgouldsambetts|cat: it's even better in my client, because you render as <sambetts|cat>, which is actual quantum physics notation11:12
mgouldhttps://en.wikipedia.org/wiki/Bra%E2%80%93ket_notation11:12
*** _vsaienko has quit IRC11:12
*** _vsaienko has joined #openstack-ironic11:12
*** lucasagomes is now known as lucas-hungry11:13
lucas-hungrymgould, lol11:13
mgouldnot sure what physical interpretation we could give to that quantity, though :-)11:14
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body  https://review.openstack.org/29123911:16
*** jcoufal has quit IRC11:19
*** _vsaienko has quit IRC11:24
*** livelace has joined #openstack-ironic11:25
*** _vsaienko has joined #openstack-ironic11:30
sean-k-mooneyhi11:33
*** _vsaienko has quit IRC11:33
sean-k-mooneyam i was wondering if there was a recommended way to use ironic for dev without baremetal nodes e.g. using vms?11:34
dtantsursean-k-mooney, you mean for testing?11:34
sean-k-mooneyyes11:34
dtantsursean-k-mooney, yes, there are drivers called pxe_ssh and agent_ssh which ssh into a hypervisor to manager your virtual nodes11:35
dtantsurlemme check if we have any docs for it..11:35
sean-k-mooneydtantsur: yes i was trying to use the agent_ssh driver with vms created by bifrost create_vms script11:35
dtantsursean-k-mooney, sounds correct11:36
sean-k-mooneydtantsur: i was able to enrol the vms and start the deploy but the callback was never triggered11:36
sean-k-mooneythere is very little docs e.g. none on the agent_ssh driver so i was not really sure how to proceed11:36
dtantsursean-k-mooney, you may want to wait for TheJulia for bifrost question.. but I would check if network is set correct. It might be worth connecting to your nodes' virtual consoles and seeing what's going on there11:37
*** _vsaienko has joined #openstack-ironic11:37
sean-k-mooneyi have acess to baremetal server that i can use if i use my home lab but at work i dont have an easy way to use phyical server currently so was trying to use vms11:37
sean-k-mooneydtantsur:  ya the virtual consoles were blank when i checked the logs from libvirt.11:38
sean-k-mooneyim going to do a little more debuging but i may revirt to using my home lab if there is no standard way to do this for dev though i assumed that how ironic is tested in the gate e.g. with vms instead of physical servers11:39
sean-k-mooneyits probaly something simple but just taught i would ask. dtantsur i assume you use phyical server for developing11:40
*** _vsaienko has quit IRC11:41
mgouldsean-k-mooney: AIUI a lot of us use VM-only setups for developing11:42
openstackgerritMoshe Levi proposed openstack/ironic-specs: Adding InfiniBand Support to ironic  https://review.openstack.org/26563511:43
*** ayoung has quit IRC11:43
sean-k-mooneymgould: do you have any recomended drivers or guides that you would suggest?11:43
*** amotoki has quit IRC11:44
mgouldsean-k-mooney: there's http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html11:45
*** _vsaienko has joined #openstack-ironic11:45
mgouldwhich I personally found pretty confusing when I started :-(11:45
mgouldthe bit about Vagrant is surprising: it installs Ironic's dependencies in a Vagrant box but Ironic itself runs on the host machine11:46
openstackgerritMerged openstack/python-ironicclient: Add test for ironic driver-list command  https://review.openstack.org/33215011:47
mgouldhttp://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack gives instructions for setting up a small OpenStack cloud in a VM which includes Ironic11:47
mgouldcreate a VM with libvirt/virtualbox/etc, then follow the instructions from within the VM11:48
sean-k-mooneymgould:  yes i read that it did not seam to discribe how to create vms for ironic to manage11:48
mgouldyour "baremetal" nodes are then nested virtual nodes within your devstack VM11:48
mgoulddevstack does that for you11:48
sean-k-mooneyso waht im actully trying to do is intergrate ironc/biforts with kolla so if i can avoid devstack that would be ideal. i can use bifrst create vms script to create the vms and that works11:49
mgouldyou can ssh into your devstack VM and run `sudo virsh list` and it will have a bunch of entries describing your "baremetal" VMs11:49
mgouldcool11:49
mgouldthat sounds like a better idea11:49
mgouldI've never used Bifrost or Kolla, I'm afraid11:50
sean-k-mooneyim currently trying to figure out why the the agent_ssh driver is not able to clean down the registered vm11:50
mgouldthe lead developer of Bifrost is TheJulia, she should be here in a couple of hours11:50
sean-k-mooneyya i was talking to them yesterday and a few weeks ago11:51
mgouldsean-k-mooney: can you paste your ironic-conductor logs somewhere?11:51
sean-k-mooneyyes give me one sec to get them11:51
mgouldnp11:51
sean-k-mooneyon well that explaines it Stderr: u"error: failed to connect to the hypervisor\nerror: Failed to connect socket to '/var/run/libvirt/libvirt-sock': No such file or directory\n".11:52
mgouldah11:53
sean-k-mooneyi am runing ironic from withing a container11:53
mgouldstart libvirtd and try again :-)11:53
sean-k-mooneythe vms are running on the host11:53
mgouldOK11:53
sean-k-mooneyi have not passed the libvirt socket into the contianer11:53
sean-k-mooneythat is why it cant connect11:53
mgouldso you have (host (container ironic) (libvirt vm1 vm2...))?11:53
*** _vsaienko has quit IRC11:53
sean-k-mooneyyes11:54
mgouldgotcha11:54
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [WIP] Use new agent API if available  https://review.openstack.org/33452311:54
sean-k-mooneythough i would have taught the agent_ssh dirver should not need to talk to the libvirt socket11:54
* mgould doesn't know how agent_ssh works, sorry11:55
sean-k-mooneyno worries ill try and allow the contaienr to use the host libvirt socket so it can control the vms directly its not too hard11:55
mgouldgood luck!11:56
* mgould likes the use of Lisp notation for nested virtualisation, I'm totally going to use that in future11:57
*** sean-k-mooney has quit IRC11:57
*** _vsaienko has joined #openstack-ironic12:00
*** phuongnh has quit IRC12:02
*** pece has quit IRC12:05
*** _vsaienko has quit IRC12:09
*** e0ne has joined #openstack-ironic12:10
*** ccamacho|lunch is now known as ccamacho12:10
*** baoli has joined #openstack-ironic12:10
*** yogi has quit IRC12:10
*** mgoddard has quit IRC12:10
*** vishwanathj has joined #openstack-ironic12:13
sambetts|catmgould: heh awesome :D12:16
*** mgoddard has joined #openstack-ironic12:21
*** lucas-hungry is now known as lucasagomes12:22
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for openstackclient plugin  https://review.openstack.org/30168712:26
*** baoli has quit IRC12:26
*** jjohnson2 has joined #openstack-ironic12:27
openstackgerritThiago Paiva Brito proposed openstack/ironic: [WIP] Custom power sync  https://review.openstack.org/33297812:28
*** trown|outtypewww is now known as trown12:28
TheJuliaGood morning12:29
*** baoli has joined #openstack-ironic12:30
*** mgould is now known as mgould|lunch12:31
mgould|lunchmorning TheJulia12:31
*** thiagop has joined #openstack-ironic12:33
thiagopGood morning, Ironic12:33
*** baoli has quit IRC12:35
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Use new agent API if available  https://review.openstack.org/33452312:35
*** vishwanathj has quit IRC12:37
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Implement new heartbeat for AgentDeploy  https://review.openstack.org/33396412:41
openstackgerritDmitry Tantsur proposed openstack/ironic: Promote agent vendor passthru to core API  https://review.openstack.org/33001812:41
*** rloo has joined #openstack-ironic12:43
lucasagomesif you guys have some time, mind taking a quick peek at https://review.openstack.org/#/c/248832/ ?12:49
dtantsurlucasagomes, +2. but why is it partial-bug? do you need more patches to IPA for https://bugs.launchpad.net/ironic/+bug/1587143 ?12:51
openstackLaunchpad bug 1587143 in Ironic "[RFE] Collect deployment logs from IPA" [Wishlist,Confirmed] - Assigned to Lucas Alvares Gomes (lucasagomes)12:51
lucasagomesdtantsur, we still need the ironic part12:51
lucasagomesonce that's merged the IPA bits are completed12:51
lucasagomesmaybe I should add ironic-pyhton-agent to that bug and actually mark as close12:52
lucasagomesclosed*12:52
dtantsurlucasagomes, yes, but that does not matter here12:52
dtantsurironic parts are tracked on ironic side of bug12:52
lucasagomesdtantsur, yeah, see it now12:52
lucasagomeslemme add ipa to that bug12:52
dtantsurso if you don't need any IPA changes, you should Closes-Bug even if other projects might require changes12:52
dtantsurthanks12:52
lucasagomesdone12:52
gbraadTheJulia, got your email. so it means bifrost's test script is not side-effects free...12:54
gbraadany idea why my dhcp could fail?12:55
TheJuliano, the test script is intended to be run inside CI, it just luckilly works if your sitting there and want to run it12:55
TheJuliaprevious dhcp configuration?12:55
gbraadthe same setup worked before for TripleO12:56
TheJuliawas the machine clean, or did it have something like devstack running on it12:56
*** rbudden has joined #openstack-ironic12:56
gbraadthe install was clean12:56
TheJuliahmm12:56
gbraadno devstack ever12:56
*** sean-k-mooney has joined #openstack-ironic12:57
TheJulia14.04, no changes, just ran the install right?12:57
gbraadcentos 7.212:57
TheJuliaoh12:57
TheJuliaI've not tested on centos 7.2 :\12:57
*** jcoufal has joined #openstack-ironic12:59
TheJulianetwork_interface was set to the same network that your nodes are atached to?12:59
gbraadthat might explain it. i'll spend some more time on it tomorrow13:00
gbraadyes, all nodes use em2 for ipmi/pxe13:00
gbraad(separate broadcast domain), same setup worked before with TripleO install.13:01
*** rook_ has joined #openstack-ironic13:02
gbraadwith tcpdump i can see the packets arriving on the node. but sems dnsmasq does nothing13:03
*** Romanenko_K has joined #openstack-ironic13:07
*** cdearborn has joined #openstack-ironic13:08
*** rook_ has quit IRC13:11
*** M00nr41n has quit IRC13:11
*** rook_ has joined #openstack-ironic13:12
*** sdake has joined #openstack-ironic13:17
vdrokmorning TheJulia and thiagop13:18
thiagophi vdrok13:18
*** sdake_ has joined #openstack-ironic13:22
*** sdake has quit IRC13:22
*** sdake has joined #openstack-ironic13:26
*** sdake_ has quit IRC13:27
*** rook has quit IRC13:27
*** mbound has joined #openstack-ironic13:28
*** rook_ is now known as rook13:29
*** aparnav has quit IRC13:32
*** edand has quit IRC13:32
TheJuliagbraad: so, thinking about it while I was taking a shower, testing resets the default configuration for where dnsmasq listens to virbr0 if it is not overridden.  You may just want to re-run installation and that will overwrite parameters that were testing centric13:33
TheJuliagood morning vdrok, thiagop13:34
thiagopTheJulia: o/13:34
vdrokdtantsur: so if I understand your concerns correctly, you want to remove the default choosing logic from dbapi, make network_interface nullable and move this logic here https://review.openstack.org/#/c/285852/60/ironic/common/driver_factory.py@9413:34
vdrokre multitenancy of course13:35
thiagopJayF: I looked into the problem in your patch. Seems like someone deleted a network on my OneView for testing, but I fixed it yesterday and your test passed now. :)13:35
thiagopjlvillal: I'll see to change that message of recheck, but not today (end of sprint around here, lots of meetings)13:35
*** links has quit IRC13:36
thiagopjlvillal: If you find some recommendation about the message format, please, ping me13:36
*** mgould|lunch is now known as mgould13:36
jlvillalthiagop, sambetts|cat might know.  And thanks13:36
vdrokgood morning jlvillal13:36
jlvillalvdrok, good morning :)13:37
TheJuliavdrok: that is my my perception as well13:37
TheJuliawhee lag13:37
dtantsurvdrok, I think so (sorry, I'm too sleepy to think). this is to make it more in line with driver composition where interfaces are nullable13:37
vdrokyep, that should work OK I think13:38
dtantsurmorning jlvillal, thiagop and everyone I could miss while sleeping in front of my laptop13:38
gbraadTheJulia; I had actually removed the dnsmasq config, reinstalled the RPM, and re-ran the install...13:38
jlvillal:)13:38
mgouldmorning jlvillal thiagop gbraad13:38
gbraadanyways, I will sepnd some time on this tomorrow...13:38
TheJuliagbraad: :(  I'm fairly sure we overwrite the config file, but yeah :(13:39
dtantsurvdrok, if we don't make it nullable, we have to update the driver composition with the conditions and when and how the interfaces can be mandatory. I'd prefer to avoid that.13:41
thiagopmorning dtantsur, that happens... ;)13:41
dtantsurthiagop, had to wake up at 7am, which is deep night for me..13:41
dtantsurwhat's even worse, I'll have to do the same tomorrow...13:41
rloohi and morning all, TheJulia, gbraad, mgould, jlvillal, vdrok, dtantsur (why so sleepy?), thiagop, lucasagomes13:41
dtantsurmorning rloo, see above13:41
thiagopmorning rloo13:41
rloodtantsur: ohhh. :)13:41
thiagop7am UTC or local time?13:42
TheJuliagood morning rloo13:42
*** ayoung has joined #openstack-ironic13:42
lucasagomesrloo, hi there13:42
*** ayoung has quit IRC13:42
dtantsurthiagop, local13:42
gbraaddtantsur if it any comfort, the second time is actually easier... ;-)13:42
*** ayoung has joined #openstack-ironic13:42
dtantsurgbraad, not for me :) I'm a real night owl13:42
thiagopdtantsur: o'rly?13:43
* thiagop couldn't miss the joke13:43
dtantsur:)13:43
vdrokmorning rloo !13:43
*** appu103_ has joined #openstack-ironic13:43
*** jiangfei has quit IRC13:44
rloodtantsur: wrt ironic-lib. I have no idea if we are going to use reno or not for it, but someone already put in a reno. Maybe we should wait for jroll to decide. I don't care either way. (And would prefer not so it is easier for us).13:45
*** [1]cdearborn has joined #openstack-ironic13:45
dtantsurrloo, maybe. as I've mentioned on the patch, openstack officials think that reno is for deployers and end users13:45
gbraadsame here... but since having a kid, time is not that easy to schedule according to my wishes13:45
dtantsurI'm fine either way, but we have to be consistent13:45
dtantsurright now we don't have anything to build renos there. and I know that oslo does not use reno13:46
gbraadthiagop: OoO, LoL13:46
rloodtantsur: exactly. so we either remove that note, or we add reno.13:46
rloodtantsur: oh, if oslo doesn't use reno, then we shouldn't either.13:46
*** mgoddard_ has joined #openstack-ironic13:46
dtantsurthat's my point13:46
rloodtantsur: now you are going to make me do some digging to see why the original reno is there.13:46
dtantsurheh13:47
mgouldmorning rloo13:47
rloodtantsur: figured it out. you approved the patch https://review.openstack.org/#/c/311613/ but Shiv asked for the reno.13:49
*** mgoddard has quit IRC13:50
rloodtantsur: am going to ask harlowja about oslo & reno, just to confirm.13:50
dtantsuryeah, my bad13:50
dtantsurit's hard to remember where we have it and where we don't..13:51
rlooharlowja: do you know if oslo plans on using reno at all for their libraries?13:51
rloodtantsur: no worries. easy to miss. too bad the author didn't mention that it wasn't set up for release notes.13:51
rloodtantsur: to be honest, i don't see why renos can't be used for libraries. seems like it would be useful. but anyway.13:52
dtantsurI fully agree13:52
*** wajdi has joined #openstack-ironic13:56
*** ametts has joined #openstack-ironic13:56
*** cdearborn has quit IRC13:58
*** wajdi_ has joined #openstack-ironic13:58
*** wajdi_ has quit IRC13:58
*** wajdi_ has joined #openstack-ironic13:59
*** wajdi has quit IRC14:00
*** rook- has joined #openstack-ironic14:00
*** pece has joined #openstack-ironic14:01
openstackgerritJulia Kreger proposed openstack/bifrost: Do Not Merge: another test for ironic networking  https://review.openstack.org/33021014:02
*** ohamada has quit IRC14:02
*** mgoddard_ has quit IRC14:03
*** sabeen has joined #openstack-ironic14:03
openstackgerritAndrey Shestakov proposed openstack/ironic: [WIP] [POC] Torrent based image provisioning  https://review.openstack.org/27846914:06
*** cdearborn has joined #openstack-ironic14:06
rloohi lucasagomes, are you OK with the plan to get nova's extra-specs-ops into oslo.utils?14:07
lucasagomesrloo, totally, that was the initial plan14:07
rloolucasagomes: or err, an improved version of that code :)14:08
rloolucasagomes: ok, guess i better review their beautiful version then. there's already one +2 for the original nova one.14:08
rloolucasagomes: do you have any idea how robust the unit tests are?14:08
lucasagomesrloo, I'm not against it... but, does we need it? Does nova also plan to use it14:08
lucasagomesraises questions14:08
rloolucasagomes: what do you mean, do we need it?14:09
rloolucasagomes: i thought you needed it.14:09
lucasagomesrloo, for the original version seems alright14:09
lucasagomesrloo, right I mean I need the original version, but I don't mind the "improved" one14:09
rloolucasagomes: oh. if the code for the original version is really subpar, then yes, i would agree with them that it shouldn't be put in a library.14:09
rloolucasagomes: it would be somewhat embarassing, to put lousy code in a library...14:10
lucasagomesrloo, right, that's where it's stuck in oslo right? Whether we should just add the original version of it or an enhanced version14:11
rloolucasagomes: the agreement was that they would put both in. i wanted the original there for historical reasons, but the one to-be-used is the enhanced version.14:12
lucasagomesrloo, cool14:12
rloolucasagomes: which means i should review it :-(14:12
lucasagomesrloo, the enhanced version has to be backward compatible with the original right? If so, it's all good14:12
rloolucasagomes: right.14:12
rloolucasagomes: hence, i want to make sure the unit tests are complete.14:13
rloolucasagomes: did you / jim just copy nova's tests?14:13
lucasagomesrloo, they seem alright, they cover all operators14:13
lucasagomesrloo, yes14:13
rloolucasagomes: ok14:13
lucasagomesit's a c&p from the code in nova14:13
*** mbound has quit IRC14:13
* mgould votes for "direct copy of nova's code, followed by replacement with something equivalent but nicer", FWIW14:14
mgouldfull disclosure: I spent quite a lot of time reviewing the "something nicer" :-)14:14
*** dedery has quit IRC14:14
*** PollyZ has joined #openstack-ironic14:15
mgouldand if we're planning to support the same syntax as nova, we should definitely be calling it from a library14:15
*** sdake has quit IRC14:16
lucasagomesmgould, how is the "something nicer", nice?14:16
lucasagomes:-)14:16
mgouldparser library + trivial evaluator, instead of hand-rolled parser/evaluator mixture14:17
mgouldalso slightly shorter, IIRC14:17
mgouldmuch easier to see what language is accepted by reading the code14:17
*** [1]cdearborn has quit IRC14:18
mgouldeasier to extend in future14:18
rloomgould: thank you for reviewing the nicer one!14:19
mgouldit's less of a win than it could be, because the original grammar is crazy *because* it made the hand-rolled implementation easier14:19
mgouldrloo: np, glad to help14:19
*** sdake has joined #openstack-ironic14:19
mgouldbut IMHO it's still a win14:20
*** mgoddard has joined #openstack-ironic14:20
dtantsursorry folks, feeling too bad to work :( see you tomorrow14:21
openstackgerritAndrey Shestakov proposed openstack/ironic-python-agent: [WIP] [POC] Torrent based image provisioning  https://review.openstack.org/27846814:21
mgoulddtantsur: feel better soon!14:21
*** dtantsur is now known as dtantsur|afk14:21
*** ohamada has joined #openstack-ironic14:22
*** amotoki has joined #openstack-ironic14:31
*** dedery has joined #openstack-ironic14:34
*** itamarl has quit IRC14:35
dederyyuriyz: hi, thank you for the review.14:35
mariojvthanks for comments on the notifications patch dmitry, i'll try to respond today14:36
mariojvfeel better soon14:37
*** joprovost has joined #openstack-ironic14:37
*** jaybeale has joined #openstack-ironic14:38
*** amotoki has quit IRC14:38
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body  https://review.openstack.org/29123914:40
*** amotoki has joined #openstack-ironic14:46
gbraaddtantsur: take care14:48
rloohi lucasagomes, do you have any idea what state the console stuff is wrt nova? https://review.openstack.org/#/c/328157/14:49
rloolucasagomes: i was going to ask at the meeting on monday and i forgot. nova's non-priority feature freeze is today (according to subteam report). does that mean if it doesn't get approved today, it doesn't get in this cycle?14:51
rloolucasagomes: oops, off by one day, june 30 apparently, so tomorrow.14:52
sambetts|catrloo: mrda said as log as there are patches in flight already then it should be ok, but I don't know if there are for the console stuff14:58
openstackgerritMathieu Mitchell proposed openstack/ironic: [WIP DO NOT MERGE] Force devstack VM to 1536M  https://review.openstack.org/33554314:58
rloosambetts|cat: oh, good to know. there is a patch: https://review.openstack.org/#/c/328157/14:58
rloosambetts|cat: i was ping'd to review it. was wondering if it was worth doing if it wouldn't get in. also, the nova-ironic-driver isn't my 'thing'.14:59
openstackgerritMerged openstack/ironic: Centralize config options - [keystone]  https://review.openstack.org/30309314:59
*** piet has joined #openstack-ironic15:00
sambetts|catrloo: ah right, yeah, hopefully it'll be OK then15:01
openstackgerritVladyslav Drok proposed openstack/ironic: Add multitenancy-related fields to port API object  https://review.openstack.org/20624415:06
openstackgerritVladyslav Drok proposed openstack/ironic: Add network_interface node field to DB and object  https://review.openstack.org/31739115:06
openstackgerritVladyslav Drok proposed openstack/ironic: Add 'neutron' network driver  https://review.openstack.org/31739315:06
openstackgerritVladyslav Drok proposed openstack/ironic: Expose node's network_interface field in API  https://review.openstack.org/31739215:06
openstackgerritVladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326215:06
openstackgerritVladyslav Drok proposed openstack/ironic: Add network interface to base driver class  https://review.openstack.org/28585215:06
lucasagomesrloo, hmm lemme check15:08
*** sacharya has joined #openstack-ironic15:09
rloolucasagomes: in particular (i'm looking at the patch now), do we need a new API version for the serial console work? if so, the nova code won't work w/o a newer microversion.15:09
*** tesseract- has quit IRC15:12
*** rajinir has joined #openstack-ironic15:12
lucasagomesrloo, AFAIUI, we don't need a new microversion (will look at the patch in ironic)... because right now even with shellinbox it returns a dictionary and one of the fields is the URL15:12
*** dedery has quit IRC15:13
lucasagomesI expect the socat console to use that same field to pass the tcp:// (or ssl in the future)15:13
lucasagomesconnection there15:13
rloolucasagomes: ok. i couldn't remember if we have to do a bump for eg new drivers.15:13
lucasagomesrloo, we do not no, unless we change the structure of the api15:14
lucasagomesadding new fields, different return value15:14
rloolucasagomes: ok, then we are good, no version bump.15:14
*** rcernin has quit IRC15:14
lucasagomesyeah looking at the code https://review.openstack.org/#/c/328168/6/ironic/drivers/modules/console_utils.py15:14
lucasagomesL254, I think they will just reuse the url field15:15
rloolucasagomes: just reading the spec again, and yeah :)15:15
lucasagomeswhich is the same structure as used by shellinabox15:15
lucasagomescool15:15
lucasagomeslemme see the nova thing15:15
*** sacharya_ has joined #openstack-ironic15:16
*** piet has quit IRC15:16
NobodyCamGood morning Ironicers15:16
*** catintheroof has joined #openstack-ironic15:16
*** pcaruana has quit IRC15:17
lucasagomesNobodyCam, hi there15:17
NobodyCamgood morning lucasagomes rloo15:17
NobodyCam:)15:18
lucasagomesrloo, seems fine, looking at the blueprint it seems that it was agreed in being a specless blueprint15:18
lucasagomesand is approved already15:18
*** sacharya has quit IRC15:18
lucasagomeshttps://blueprints.launchpad.net/nova/+spec/ironic-serial-console-support15:18
lucasagomesaccepted for newton15:18
*** rbartal has quit IRC15:18
apughazhis there a special local.conf file for setting up devstack on bare metal?15:21
rloolucasagomes: right, it is specless and approved in nova, although the non-prior feature freeze is tomorrow, but it looks like we are ok wrt to that.15:25
rloolucasagomes: not sure if they will approve if the ironic code isn't in though. but as long as we have time...15:25
*** Sukhdev has joined #openstack-ironic15:27
*** rcernin has joined #openstack-ironic15:31
*** amotoki has quit IRC15:31
mgouldapughazh: here's the one sambetts uses: http://paste.openstack.org/show/516174/15:32
*** pcaruana has joined #openstack-ironic15:32
apughazhthanks!15:33
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Add Ironic specs process to the code contribution guide  https://review.openstack.org/33016715:35
*** Romanenko_K has quit IRC15:37
apughazhwhere would15:38
apughazhI find the information to fill in the brackets in that config file?15:38
mgoulddunno, sorry15:39
sambetts|catapughazh: thats specfic to your environment15:39
openstackgerritMichael Turek proposed openstack/ironic: Devstack: When appropriate, Skip setting cleaning_network_uuid  https://review.openstack.org/33556715:39
apughazhok cool15:39
NobodyCammorning mgould sambetts|cat15:39
sambetts|catMorning NobodyCam15:39
NobodyCam:)15:39
mgouldmorning NobodyCam15:40
NobodyCam:)15:40
*** Nakato has quit IRC15:43
*** Nakato has joined #openstack-ironic15:45
*** piet has joined #openstack-ironic15:45
*** amotoki has joined #openstack-ironic15:45
*** blakec has joined #openstack-ironic15:46
*** david-lyle has quit IRC15:46
*** klindgren has joined #openstack-ironic15:48
*** jaybeale has quit IRC15:48
lucasagomesrloo, yeah, neither they should approve that patch while it's not in15:50
lucasagomesthey def should wait IMO15:50
rloolucasagomes: SOOO MUCH to review! :)15:50
lucasagomeshah :-/ yeah15:51
*** klindgren_ has joined #openstack-ironic15:52
*** klindgren has quit IRC15:53
jcookstackalytics broken for Ironic or do I have wrong url: http://stackalytics.com/report/contribution/ironic/9015:58
jcooknova seems to work15:58
jcooksimilarly nova-group works but not ironic-group15:58
*** moshele has quit IRC15:59
*** amotoki has quit IRC15:59
openstackgerritMerged openstack/bifrost: Disable flat network driver  https://review.openstack.org/33506016:00
mgouldcurses, all that time gaming the stackalytics stats down the drain :-)16:00
jcookthat's pessimistic =/16:00
*** praneshp has joined #openstack-ironic16:01
* mgould wasn't actually being serious :-)16:01
jcook^_^16:02
*** ifarkas has quit IRC16:06
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/33442016:06
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to use FLAT network setup  https://review.openstack.org/33438216:06
vsaienkohey Ironic'ers!16:07
*** Goneri has joined #openstack-ironic16:08
*** athomas has quit IRC16:10
rloojcook: that's correct. nothing has been doing in ironic for the past 90 days. we're on holiday. (just kidding). If you need it, please poke the stackalytics folks16:13
*** HenryG has quit IRC16:16
*** HenryG has joined #openstack-ironic16:16
jcookrloo they got their own chanel? like #openstack-stackalytics?16:17
jcookchannel*16:17
rloojcook: i have no idea16:17
jcookkk16:18
*** piet has quit IRC16:21
*** raildo is now known as raildo-afk16:23
*** raildo-afk is now known as raildo16:26
*** praneshp has quit IRC16:27
*** dmk0202 has quit IRC16:31
*** jaybeale has joined #openstack-ironic16:34
*** piet has joined #openstack-ironic16:35
*** pcaruana has quit IRC16:36
*** sabeen has quit IRC16:39
*** ChubYann has joined #openstack-ironic16:42
*** rcernin has quit IRC16:45
*** jaybeale has quit IRC16:47
openstackgerritStephanie Miller proposed openstack/ironic-python-agent: Move diskimage-builder IPA element to IPA tree  https://review.openstack.org/33558316:48
*** gfhellma has joined #openstack-ironic16:57
*** e0ne has quit IRC16:59
*** jjohnson2 has quit IRC16:59
*** jjohnson2 has joined #openstack-ironic16:59
*** rpioso has joined #openstack-ironic17:00
*** krotscheck is now known as krotscheck_vaca17:01
*** krotscheck_vaca is now known as krot_vaca_jul1917:01
*** Goneri has quit IRC17:05
jlvillalFYI: Ironic QA meeting going on now in #openstack-meeting17:05
sambetts|catjlvillal: schrodinger's I'm both here and not here17:07
sambetts|cattoday17:07
sambetts|catrloo: ^17:07
rloosambetts|cat: h aha17:07
jlvillalsambetts|cat, :)17:07
*** sdake_ has joined #openstack-ironic17:08
*** spandhe has joined #openstack-ironic17:08
*** trown is now known as trown|lunch17:08
*** Sukhdev has quit IRC17:09
*** sdake has quit IRC17:10
*** Goneri has joined #openstack-ironic17:12
*** ccamacho is now known as ccamacho|away17:14
*** dmk0202 has joined #openstack-ironic17:16
lucasagomescalling it a day, have a great evening all17:17
*** lucasagomes is now known as lucas-afk17:17
*** praneshp has joined #openstack-ironic17:19
*** piet has quit IRC17:19
sambetts|catcya lucas-afk17:20
* jlvillal forgot to ask lucas-afk if KerryGold butter is popular in Ireland. I buy it here in the US :)17:21
openstackgerritNisha Agarwal proposed openstack/ironic: Add support for building ISO for deploy ramdisk  https://review.openstack.org/26459017:30
jlvillaljcook, This works for me: http://stackalytics.com/?module=ironic17:30
*** sabeen has joined #openstack-ironic17:31
openstackgerritNisha Agarwal proposed openstack/ironic: Config variable to configure [glance] section  https://review.openstack.org/26680317:33
*** ohamada has quit IRC17:33
harlowjarloo we started using reno for libraries that use oslo.config17:37
harlowjaones that don't we aren't17:37
rlooharlowja: why?17:38
rlooI mean, why just for libraries that use oslo.config?17:38
harlowjaso that we can let ops know about release things that affect config17:38
rlooharlowja: is it a good idea to use reno for libraries? we have an internal library and wondering whether we should use it or not.17:38
*** piet has joined #openstack-ironic17:38
rlooharlowja: so is reno meant for users (operators), not developers?17:39
harlowjaya afaik17:39
harlowjahttps://specs.openstack.org/openstack/oslo-specs/specs/policy/release-note-management.html#proposed-policy17:39
harlowja'Release notes for developers, whether part of the OpenStack community or not, are handled in the developer documentation published for the libraries. This ensures that all developer-focused information is in one place.'17:39
harlowja'Release notes for deployers should be managed through reno and published under http://docs.openstack.org/releasenotes/. Each library with configuration options or other deployer-facing interfaces should set up reno to publish release notes. Not all Oslo libraries will need to do that.'17:39
harlowjaso deployer stuff (config things) --> reno17:39
rlooharlowja: ok, that answers that question then. thx!17:40
harlowjanp17:40
harlowjaits a weird gray area imho17:40
harlowjain that some libraries probably don't need to use oslo.config but they do anyway17:40
rlooharlowja: i don't really care much, my preference is whatever is *less* work for me :D17:40
harlowjak17:40
*** electrofelix has quit IRC17:41
rlooharlowja: oh. wait. oslo.config. our library uses it. but i don't see the need to use reno cuz of that.17:41
*** [1]cdearborn has joined #openstack-ironic17:41
harlowjak17:41
*** cdearborn has quit IRC17:41
rlooharlowja: ironic-lib is consumed by ironic-related projects, and those projects have reno so we are ok.17:42
harlowjaok, depends on if stuff in ironic-lib changes configuration options17:42
harlowjaand that change then by side-effect affects ironic-releated projects17:42
harlowjathats why imho oslo.config has always been weird17:42
harlowjait allows for action at a distance17:42
*** mgould is now known as mgould|afk17:42
harlowjain that its basically the equivalent of passing a void pointer around to objects17:43
sambetts|catZOMG this is awesome, infra now gives you a telnet connection to the slave running the jobs in the gate so you can watch them in real time!17:43
rlooharlowja: oh. so *if* ironic-lib did change a config option and *if* ironic-lib had reno, would that mean that if ironic consumed ironic-lib, the change would show up in ironic's release notes?17:43
harlowjarloo unsure about that :-P17:43
harlowjadhellmann might know17:43
rlooharlowja: then what's the use of having renos at the lower level. the operator won't see it.17:44
harlowjai summoned doug17:44
harlowjalol17:44
*** rcernin has joined #openstack-ironic17:44
jcookjlvillal I ended up using that to pull data17:45
jcookthx17:45
*** ayoung has quit IRC17:47
*** ayoung has joined #openstack-ironic17:47
jlvillaljcook, They also have an API17:49
jcookneat! only needed one of data though, doing OKRs and encouraging capturing good metrics for measuring upstream contributions by team over next quarter ;)17:49
rloojlvillal: do you know how to notify the stackalytics folks that the link that jcook used, didn't work?17:50
jcookdata once *17:50
*** derekh is now known as derekh_afk17:50
rloojcook, you can ask us about your team. They are GREAT! :D17:50
jcook:D17:50
jcookI added myself too, I need to contribute more upstream >_>;17:51
rloojcook: it isn't the quantity, it is the quality. but you know that, right? ;)17:51
jlvillalrloo, Not exactly. I think they maintainer is in the Ukraine.17:51
jlvillalrloo, Probably vdrok or vsaienko now the Stackalytics maintainer :)17:51
*** thrash is now known as thrash|biab17:51
jlvillals/now/know/17:51
rloojlvillal: oh. yeah, one of the mirantis folks might know.17:51
*** chlong has quit IRC17:52
jcookrloo yep, I'm using -1 reviews and bugs filed / fixed as well as participating in substantial conversations here as metrics17:52
jcookany metric can be gamed but trying to design them to be meaninful vs say pure review count17:52
rloojcook: i think would would be more useful, is getting the actual comments and skimming those.17:53
jcookyeah, that's not a bad idea17:53
rloojcook: i was thinking that, when i was trying to figure out if someone was worthy of being a core :)17:54
jcookyeah, were focusing on alignment, so reducing technical debt is in there too, not nearly as bad as in nova, but we have a bit we could kill17:54
*** pece has quit IRC17:54
jcookwe're*17:55
*** sdake_ has quit IRC17:55
jcook-1 reviews seem like a relativistic measure of meaningful review comments though. I mean someone could be an arse and -1 all the things, but I'm not really concerned about that happening here :D17:56
jcookbut I might incorporate comments bit in there too as it's not hard to skim through in stackalytics17:56
rloojcook: there once was someone that -1'd for typos only. somewhat amusing, somewhat annoying.17:57
jcook:D17:57
jcookwell at least the code had less typos?17:57
jcooklol17:57
rlooor the person's job was to find typos. dunno...17:58
jcookmaybe it was a docs person :)17:58
vdrokjcook: the maintainer of stackalytics is this guy - https://launchpad.net/~shakhat17:59
vdrokbut it's probably too late in his TZ already, 10 pm17:59
*** Goneri has quit IRC18:04
mariojvhow do i edit an existing release note? referring to https://review.openstack.org/#/c/298461/14/releasenotes/notes/add-notifications-97b6c79c18b48073.yaml18:06
mariojvi'm guessing there's something i need other than editing directly based on the hash at the end18:06
*** blakec has quit IRC18:07
*** jjohnson2 has quit IRC18:07
rloomariojv: can't you just clikc on the edit button?18:08
rloomariojv: you want me to do it for you?18:08
mariojvrloo: yes, just wasn't sure if file name needs an edit18:08
mariojvi've got it if that's all it needs, thanks though18:09
rloomariojv: you want to rename the file or just add a dot in the file?18:09
mariojvrloo: just add a dot18:09
mariojvi wasn't sure if the -97b... at the end was a hash of the file or what though18:09
rloomariojv: just add the dot then. that filename was generated; i'm guessing the blahblah is just to make it unique or something.18:09
mariojvor commit reference maybe18:09
mariojvok, thanks!18:09
*** livelace has quit IRC18:10
rloomariojv: fwiw, i'm not going to comment, at least not now, but it should say more, like how to enalbe it, or point to documentation for more info.18:10
mariojvwill do18:11
rloomariojv: the operators thank you :)18:11
*** livelace has joined #openstack-ironic18:12
openstackgerritJulia Kreger proposed openstack/bifrost: Do Not Merge: Canery test for ironic networking changes  https://review.openstack.org/33013818:15
*** blakec has joined #openstack-ironic18:16
*** gfhellma has quit IRC18:18
*** [1]cdearborn has quit IRC18:20
*** [1]cdearborn has joined #openstack-ironic18:20
*** blakec has quit IRC18:21
*** appu103_ has quit IRC18:22
jcookvdrok right on thanks18:23
*** dmk0202 has quit IRC18:32
vdrokgood night everyone!18:32
*** sambetts|cat is now known as sambetts|afk18:33
*** dmk0202 has joined #openstack-ironic18:34
jcooksent a mail about stackalytics18:36
*** trown|lunch is now known as trown18:36
*** dmk0202 has quit IRC18:37
openstackgerritMario Villaplana proposed openstack/ironic: Add power state change notifications  https://review.openstack.org/32186518:40
openstackgerritMario Villaplana proposed openstack/ironic: Add notification base classes and docs  https://review.openstack.org/29846118:40
*** amotoki has joined #openstack-ironic18:41
*** ayoung has quit IRC18:44
*** ayoung has joined #openstack-ironic18:44
*** thrash|biab is now known as thrash18:44
*** fragatina has joined #openstack-ironic18:44
*** fragatina has quit IRC18:44
*** fragatina has joined #openstack-ironic18:45
mariojvrloo: you were right about the suffix of the release note file just being a unique identifier, i think. i just added a new one for the power state patch and it doesn't correspond to the commit hash18:46
rloomariojv: :)18:46
*** amotoki has quit IRC18:47
rloomariojv: all i know, is we've been modifying those files and not had to rename the file.18:47
*** derekh_afk is now known as derekh_afk_ish18:47
*** fragatina has quit IRC18:56
openstackgerritMario Villaplana proposed openstack/ironic: Add power state change notifications  https://review.openstack.org/32186518:57
*** Sukhdev has joined #openstack-ironic19:01
*** stevemar has quit IRC19:02
*** sambetts|afk has quit IRC19:02
*** stevemar has joined #openstack-ironic19:03
*** sambetts has joined #openstack-ironic19:04
*** sdake has joined #openstack-ironic19:14
*** spandhe has quit IRC19:15
*** derekh_afk_ish has quit IRC19:16
mariojvJayF: when you're back, if there's a way to do it in gerrit feel free to make yourself owner of https://review.openstack.org/#/c/263842/, since you've done most of the work on it and it's changed significantly since the first patch set19:16
mat128JayF: Victory! 1.5G worked https://review.openstack.org/#/c/327891/19:26
mat128(so you have 2 pings hwen you get back :)19:26
*** sabeen has quit IRC19:27
*** sabeen has joined #openstack-ironic19:34
*** piet has quit IRC19:38
*** catintheroof has quit IRC19:43
*** joprovost has quit IRC19:46
*** fragatina has joined #openstack-ironic19:47
*** piet has joined #openstack-ironic19:49
*** fragatina has quit IRC19:51
*** fragatina has joined #openstack-ironic19:55
*** Guest20454 is now known as mgagne19:58
*** mgagne has joined #openstack-ironic19:58
*** livelace has quit IRC20:00
*** amotoki has joined #openstack-ironic20:02
*** stevebaker has quit IRC20:02
*** marios has quit IRC20:03
*** slagle has quit IRC20:04
*** rbudden has quit IRC20:04
*** amotoki has quit IRC20:06
*** sabeen has quit IRC20:08
*** praneshp has quit IRC20:19
openstackgerritStephanie Miller proposed openstack/ironic-python-agent: Move diskimage-builder IPA element to IPA tree  https://review.openstack.org/33558320:27
*** PollyZ has quit IRC20:33
zigoCould you guys try to *not* be smart when generating the ironic config? :)20:34
zigoIt's currently quite painful at the packaging level.20:34
zigoI tried to redo the oslo-config-generator call, like I do for all other packages, and I just couldn't.20:34
zigoHow is it calculated?20:34
rloozigo: I just do 'tox -egenconfig'.20:36
zigorloo: Naaah... that can't happen in the packaging world.20:36
rloozigo which does 'oslo-config-generator --config-file=tools/config/ironic-config-generator.conf'20:36
zigoNo tox, no pip.20:36
rloozigo: will that work? ^^20:36
zigorloo: That's the point, there's no such thing such as ironic-config-generator.conf20:36
rloozigo: OH.20:37
zigoThat would have been too easy! :)20:37
rloohow do the other projects do it then?20:37
*** PollyZ has joined #openstack-ironic20:37
zigoThey *DO* have a oslo-generator config file.20:37
rloozigo: *where* ? :)20:37
zigoI ended up doing:20:37
zigo        bash tools/config/generate_sample.sh -b . -p ironic -o $(CURDIR)/debian/ironic-common/usr/share/ironic-common20:37
zigo        mv  $(CURDIR)/debian/ironic-common/usr/share/ironic-common/ironic.conf.sample $(CURDIR)/debian/ironic-common/usr/share/ironic-common/ironic.conf20:37
zigoBut I would have prefer to call oslo-config-generator myself.20:38
jlvillalzigo, Which release are you doing?20:38
zigo5.1.220:38
*** sabeen has joined #openstack-ironic20:38
*** praneshp has joined #openstack-ironic20:39
jlvillalzigo, Named release?20:39
zigoMitaka20:39
jlvillalIn the Newton cycle we have moved to oslo-config-generator20:39
rloojlvillal: 5.1.2 is latest mitaka.20:39
zigojlvillal: Mitaka also is using oslo-config-generator somehow.20:39
zigoBut not in an easy way for package maintainers like me... :P20:39
jlvillalzigo, I don't think so.20:40
rloozigo, how did you do the previous ones, eg 5.1.1. should be similar process.20:40
*** fragatina has quit IRC20:40
rloozigo: we just changed things recently (in master).20:40
zigoI'm fixing it because the git's sample file doesn't contain all the keystonemiddleware config directives, which I fixed in the Debian's keystonemiddleware package (ie: so that it can provide older types of directives even if we're moving away from them: it breaks some users if we don't have them)20:40
*** [1]cdearborn has quit IRC20:40
zigorloo: Cool, I'll have a look.20:40
rloozigo: but master won't help you with mitaka. on the other hand, if we changed it in master and you don't like it, let us know soon :)20:41
*** [1]cdearborn has joined #openstack-ironic20:41
zigoSure, thanks ! :)20:41
zigoLooking right away now.20:41
*** PollyZ has quit IRC20:42
jlvillalzigo, So Newton cycle is now and also is master... :)20:43
zigojlvillal: I'll package 6.0.0 to Debian Experimental, so that it joins all the rest of Newton! :020:43
*** piet has quit IRC20:44
rloooh, one problem ? might be that we haven't had a release of ironic since newton cycle started.20:44
zigoAh, indeed, the way to generate the config file is a way more reasonable in Newton.20:45
zigorloo: Isn't 6.0.0 for Newton?20:46
rloozigo: i am not sure. we're doing semver and i forget how it works. I'm just a developer not a release person.20:46
zigoI really wish Ironic was release-managed, that would be a lot simpler.20:47
zigorloo: http://releases.openstack.org/newton/index.html20:47
zigoNewton it is.20:47
rloozigo: OH. well there you go. I must have forgotten that we did a release!20:49
zigo:)20:49
rloozigo: wish it'd show the dates. hopefully the config generator changes are in 6.0.020:49
zigoThey are.20:49
zigoI already merged it in the debian/newton branch for ironic in my git, and I checked.20:50
rloozigo: ah, now i remember. we released cuz of the security issue, 1-2 weeks ago. good.20:50
zigoOh !20:50
zigoI also remember updating the Sid package because of it... :P20:50
*** PollyZ has joined #openstack-ironic20:50
rloozigo, thx for doing the packaging!20:51
zigoMy pleasure.20:51
*** dmk0202 has joined #openstack-ironic20:54
*** PollyZ has quit IRC20:55
zigoUnfortunately, it looks like I wont be able to upload Ironic until Newton b2, as every other packages are saying fixtures <2.0, and ironic has fixtures >3.020:58
zigoThat's a very good example why it's very annoying for package maintainers if projects aren't in sync ...20:58
rloozigo. oh. wonder why we are out of sync with other projects.21:00
zigorloo: It was decided a few cycles ago.21:00
rloozigo: it was that Bot that changed that.21:00
rloozigo: go complain to the OpenStack Proposal Bot! Maybe we're too fast in approving those changes. We can wait a few months first if you like :)21:01
zigorloo: If the project is out of sync, then IMO, it should care having very loose coupling with the versions of libs its using.21:02
zigoThat's what swift does, and it works very well.21:02
*** raildo is now known as raildo-afk21:03
* zigo has a break and goes talk to other people at debconf 16 (cape town)21:03
*** rbudden has joined #openstack-ironic21:06
openstackgerritRuby Loo proposed openstack/ironic-lib: Remove releasenotes/*  https://review.openstack.org/33566821:07
openstackgerritRuby Loo proposed openstack/ironic-lib: Add keyword arg 'log_stdout' to utils.execute()  https://review.openstack.org/33521121:09
*** trown is now known as trown|outtypewww21:11
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Add Ironic specs process to the code contribution guide  https://review.openstack.org/33016721:12
*** jcoufal has quit IRC21:12
*** PollyZ has joined #openstack-ironic21:18
*** wajdi has joined #openstack-ironic21:18
*** ayoung has quit IRC21:20
*** wajdi_ has quit IRC21:21
*** amotoki has joined #openstack-ironic21:22
*** wajdi_ has joined #openstack-ironic21:22
*** PollyZ has quit IRC21:23
*** baoli has joined #openstack-ironic21:24
*** wajdi has quit IRC21:24
*** PollyZ has joined #openstack-ironic21:25
*** amotoki has quit IRC21:26
*** wajdi_ has quit IRC21:27
*** derekh has joined #openstack-ironic21:27
*** derekh has quit IRC21:31
*** sabeen has quit IRC21:36
*** baoli has quit IRC21:39
*** spandhe has joined #openstack-ironic21:43
*** PollyZ_ has joined #openstack-ironic21:45
*** PollyZ has quit IRC21:48
mat128zigo: all projects are tested using the upper-constraints file21:49
mat128https://github.com/openstack/requirements/blob/stable/mitaka/upper-constraints.txt21:49
mat128https://github.com/openstack/requirements/blob/master/upper-constraints.txt21:49
mat128sorry21:49
mat128in theory that set of versions works with everything in master21:50
*** mbound has joined #openstack-ironic21:51
*** baoli has joined #openstack-ironic21:52
*** mbound_ has joined #openstack-ironic21:53
*** baoli has quit IRC21:54
*** rcernin has quit IRC21:54
*** [2]cdearborn has joined #openstack-ironic21:54
openstackgerritStephanie Miller proposed openstack/ironic-python-agent: Move diskimage-builder IPA element to IPA tree  https://review.openstack.org/33558321:55
*** mbound has quit IRC21:57
*** piet has joined #openstack-ironic21:57
*** PollyZ has joined #openstack-ironic21:59
*** PollyZ has quit IRC21:59
*** PollyZ_ has quit IRC22:02
mrdaMorning Ironic22:03
*** PollyZ has joined #openstack-ironic22:04
mrdasambetts: actually, it's not in flight, it's landed.  My mistake.  Exceptions might be had for things that are very close, but officially, it's landed22:05
*** baoli has joined #openstack-ironic22:05
mariojvmorning mrda22:05
*** sabeen has joined #openstack-ironic22:05
mariojvwith this patch: https://review.openstack.org/#/c/327891/ does the commit message indicate that CoreOS is now running gate tests to make sure IPA still works before each CoreOS release?22:06
*** baoli has quit IRC22:06
*** mbound_ has quit IRC22:06
mrdahey mariojv22:06
*** mbound has joined #openstack-ironic22:06
*** [1]cdearborn has quit IRC22:06
*** baoli has joined #openstack-ironic22:08
*** dmk0202 has quit IRC22:08
*** baoli has quit IRC22:08
*** mbound has quit IRC22:08
*** mbound has joined #openstack-ironic22:08
*** ametts has quit IRC22:16
*** baoli has joined #openstack-ironic22:17
*** chlong has joined #openstack-ironic22:17
*** baoli has quit IRC22:18
*** baoli has joined #openstack-ironic22:21
*** rama_y has joined #openstack-ironic22:25
*** ayoung has joined #openstack-ironic22:27
*** baoli has quit IRC22:28
*** [2]cdearborn has quit IRC22:29
*** thiagop has quit IRC22:31
*** baoli has joined #openstack-ironic22:34
*** baoli has quit IRC22:35
*** sdake_ has joined #openstack-ironic22:42
*** amotoki has joined #openstack-ironic22:42
jlvillalGood morning mrda22:43
*** thrash is now known as thrash|g0ne22:43
*** sdake has quit IRC22:45
*** amotoki has quit IRC22:47
*** baoli has joined #openstack-ironic22:48
*** baoli has quit IRC22:49
*** sdake_ has quit IRC22:49
*** baoli has joined #openstack-ironic22:55
*** baoli has quit IRC22:56
*** baoli has joined #openstack-ironic22:58
*** baoli has quit IRC22:59
mrdahey jlvillal - just sent you some email :)23:00
jlvillalmrda, Thanks23:00
*** sdake has joined #openstack-ironic23:01
*** sdake has quit IRC23:04
*** baoli has joined #openstack-ironic23:11
*** piet has quit IRC23:11
*** baoli has quit IRC23:12
*** piet has joined #openstack-ironic23:18
*** stevebak` has joined #openstack-ironic23:19
*** stevebak` is now known as stevebaker23:19
*** PollyZ has quit IRC23:25
*** baoli has joined #openstack-ironic23:28
*** baoli has quit IRC23:29
*** slagle has joined #openstack-ironic23:33
*** mbound has quit IRC23:48
*** rbudden has quit IRC23:50
*** rpioso has quit IRC23:55
*** sacharya_ has quit IRC23:56

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