Tuesday, 2016-05-31

*** xhku_ has joined #openstack-ironic00:09
*** xhku has quit IRC00:09
*** mat128 has quit IRC00:10
*** mat128 has joined #openstack-ironic00:10
*** amotoki has joined #openstack-ironic00:19
*** rama_y has joined #openstack-ironic00:24
*** _vsaienko has quit IRC00:28
*** _vs has quit IRC00:28
*** _vsaienko has joined #openstack-ironic00:31
*** _vs has joined #openstack-ironic00:31
*** amotoki has quit IRC00:32
*** _vsaienko has quit IRC00:35
*** _vs has quit IRC00:35
*** amotoki has joined #openstack-ironic00:39
*** _vsaienko has joined #openstack-ironic00:40
*** _vs has joined #openstack-ironic00:40
*** hoangcx has joined #openstack-ironic00:45
*** YiboCai has joined #openstack-ironic00:46
*** hoangcx has quit IRC00:47
*** ijw has joined #openstack-ironic00:48
*** hoangcx has joined #openstack-ironic00:52
*** phuongnh has joined #openstack-ironic00:53
*** _vs has quit IRC00:53
*** _vsaienko has quit IRC00:53
*** rloo has quit IRC00:53
*** ijw has quit IRC00:54
*** _vsaienko has joined #openstack-ironic00:56
*** _vs has joined #openstack-ironic00:56
*** hoangcx has quit IRC00:56
*** hoangcx has joined #openstack-ironic00:57
*** sacharya has joined #openstack-ironic01:12
*** baoli has joined #openstack-ironic01:13
*** sacharya has quit IRC01:17
*** baoli has quit IRC01:18
*** wajdi has joined #openstack-ironic01:24
*** Sukhdev has joined #openstack-ironic01:29
*** ijw has joined #openstack-ironic01:31
*** mtanino has quit IRC01:49
*** Goneri has joined #openstack-ironic01:49
openstackgerritRuby Loo proposed openstack/ironic-python-agent: Remove unused SUPPORTED_ROOT_DEVICE_HINTS  https://review.openstack.org/32305001:52
*** wajdi has quit IRC01:53
*** praneshp has joined #openstack-ironic01:53
*** wajdi has joined #openstack-ironic01:53
openstackgerritYibo Cai proposed openstack/ironic: Support multi arch deployment  https://review.openstack.org/32263201:53
*** wajdi_ has joined #openstack-ironic01:58
*** sacharya has joined #openstack-ironic01:59
*** wajdi has quit IRC02:01
*** _vsaienko has quit IRC02:04
*** _vs has quit IRC02:05
*** _vsaienko has joined #openstack-ironic02:11
*** _vs has joined #openstack-ironic02:11
*** _vs has quit IRC02:16
*** _vsaienko has quit IRC02:16
*** yuikotakadamori has joined #openstack-ironic02:17
*** Sukhdev has quit IRC02:19
*** _vsaienko has joined #openstack-ironic02:19
*** _vs has joined #openstack-ironic02:19
*** Sukhdev has joined #openstack-ironic02:28
*** baoli has joined #openstack-ironic02:30
*** Sukhdev has quit IRC02:32
*** _vs has quit IRC02:33
*** _vsaienko has quit IRC02:33
*** _vsaienko has joined #openstack-ironic02:33
*** _vs has joined #openstack-ironic02:33
*** wajdi_ has quit IRC02:37
*** fragatin_ has joined #openstack-ironic02:41
*** fragatin_ has quit IRC02:44
*** fragatin_ has joined #openstack-ironic02:45
*** fragatina has quit IRC02:45
*** baoli has quit IRC02:45
*** sdake_ has joined #openstack-ironic02:46
*** yuanying has quit IRC02:47
*** fragatin_ has quit IRC02:49
*** sdake has quit IRC02:50
*** yuanying has joined #openstack-ironic02:52
*** _vs has quit IRC02:52
*** _vsaienko has quit IRC02:53
*** Goneri has quit IRC02:54
*** _vsaienko has joined #openstack-ironic02:56
*** _vs has joined #openstack-ironic02:56
*** fragatina has joined #openstack-ironic02:57
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/32298603:00
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/32298703:01
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/32306603:01
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/32306703:01
*** fragatina has quit IRC03:02
*** Haomeng has joined #openstack-ironic03:03
*** _vs has quit IRC03:04
*** _vsaienko has quit IRC03:05
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/32300203:06
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/32300303:06
*** _vsaienko has joined #openstack-ironic03:06
*** _vs has joined #openstack-ironic03:06
*** sdake_ has quit IRC03:07
*** _vsaienko has quit IRC03:17
*** _vs has quit IRC03:18
*** _vsaienko has joined #openstack-ironic03:18
*** _vs has joined #openstack-ironic03:18
*** _vsaienko has quit IRC03:24
*** _vs has quit IRC03:24
*** _vsaienko has joined #openstack-ironic03:28
*** _vs has joined #openstack-ironic03:28
openstackgerritTan Lin proposed openstack/ironic: Add one use case for configdrive  https://review.openstack.org/32309903:30
*** _vsaienko has quit IRC03:35
*** _vs has quit IRC03:35
*** _vsaienko has joined #openstack-ironic03:38
*** _vs has joined #openstack-ironic03:38
*** flwang1 has joined #openstack-ironic03:38
*** yuanying has quit IRC03:43
*** _vsaienko has quit IRC03:44
*** haomeng__ has joined #openstack-ironic03:44
haomeng__flwang1, hi03:45
*** _vs has quit IRC03:46
*** yuanying has joined #openstack-ironic03:47
*** fragatina has joined #openstack-ironic03:48
*** rcernin has joined #openstack-ironic03:49
*** links has joined #openstack-ironic03:50
*** _vsaienko has joined #openstack-ironic03:50
*** _vs has joined #openstack-ironic03:51
*** _vsaienko has quit IRC03:54
*** appu103_ has joined #openstack-ironic03:54
*** _vs has quit IRC03:55
*** Sukhdev has joined #openstack-ironic03:55
*** _vsaienko has joined #openstack-ironic03:58
*** _vs has joined #openstack-ironic03:58
*** Sukhdev has quit IRC03:59
*** _vsaienko has quit IRC04:04
*** _vs has quit IRC04:04
*** _vsaienko has joined #openstack-ironic04:10
*** _vs has joined #openstack-ironic04:10
*** vmud213 has joined #openstack-ironic04:10
*** _vsaienko has quit IRC04:17
*** _vs has quit IRC04:17
*** sacharya has quit IRC04:19
*** sacharya has joined #openstack-ironic04:20
*** sacharya has quit IRC04:21
*** sacharya_ has joined #openstack-ironic04:21
*** _vsaienko has joined #openstack-ironic04:23
*** _vs has joined #openstack-ironic04:23
*** _vsaienko has quit IRC04:29
*** _vs has quit IRC04:29
*** _vsaienko has joined #openstack-ironic04:33
*** _vs has joined #openstack-ironic04:33
*** rama_y has quit IRC04:38
*** _vs has quit IRC04:42
*** _vsaienko has quit IRC04:43
*** _vsaienko has joined #openstack-ironic04:44
*** _vs has joined #openstack-ironic04:44
*** rcernin has quit IRC04:52
*** _vsaienko has quit IRC04:52
*** _vs has quit IRC04:53
*** chlong has quit IRC04:54
*** _vsaienko has joined #openstack-ironic04:56
*** _vs has joined #openstack-ironic04:57
*** baoli has joined #openstack-ironic04:57
*** baoli has quit IRC05:02
*** _vs has quit IRC05:04
*** _vsaienko has quit IRC05:04
*** _vsaienko has joined #openstack-ironic05:05
*** _vs has joined #openstack-ironic05:05
openstackgerritZhenguo Niu proposed openstack/ironic: Clear target_power_state on conductor startup  https://review.openstack.org/31009005:06
*** ChubYann has quit IRC05:08
*** sacharya_ has quit IRC05:09
*** _vs has quit IRC05:10
*** _vsaienko has quit IRC05:11
*** chlong has joined #openstack-ironic05:12
*** _vsaienko has joined #openstack-ironic05:15
*** _vs has joined #openstack-ironic05:15
*** _vs has quit IRC05:24
*** _vsaienko has quit IRC05:24
*** rama_y has joined #openstack-ironic05:27
*** Sukhdev has joined #openstack-ironic05:28
*** tyagi has joined #openstack-ironic05:32
*** tyagi has quit IRC05:32
*** vishwanathj has joined #openstack-ironic05:33
*** ishant has joined #openstack-ironic05:33
openstackgerritYuiko Takada Mori proposed openstack/ironic-specs: Nova compatible serial console support  https://review.openstack.org/31950505:34
*** praneshp has quit IRC05:34
*** ishant_ has joined #openstack-ironic05:35
*** ishant_ has left #openstack-ironic05:35
*** ishant_ has quit IRC05:35
*** hoangcx has quit IRC05:55
*** fragatina has quit IRC06:08
*** mjura has joined #openstack-ironic06:10
*** rama_y has quit IRC06:12
*** mjura has quit IRC06:15
*** mjura has joined #openstack-ironic06:15
*** moshele has joined #openstack-ironic06:22
*** parlos_ has joined #openstack-ironic06:50
*** parlos has quit IRC06:51
*** e0ne has joined #openstack-ironic06:56
*** tesseract has joined #openstack-ironic06:57
*** rcernin has joined #openstack-ironic07:02
*** yolanda has quit IRC07:05
*** yolanda has joined #openstack-ironic07:05
openstackgerritTan Lin proposed openstack/ironic: Add a doc about appending kernel parameters to boot instances  https://review.openstack.org/32084807:09
*** sacharya has joined #openstack-ironic07:09
*** hoangcx has joined #openstack-ironic07:12
*** sacharya has quit IRC07:14
*** praneshp has joined #openstack-ironic07:14
*** Sukhdev has quit IRC07:18
*** Sukhdev has joined #openstack-ironic07:19
*** parlos_ has quit IRC07:23
*** haomeng__ has quit IRC07:28
*** ijw has quit IRC07:32
*** ifarkas has joined #openstack-ironic07:34
*** sdake has joined #openstack-ironic07:34
*** stendulker has joined #openstack-ironic07:35
*** vishwanathj has quit IRC07:37
*** praneshp has quit IRC07:49
lucasagomesmorning all07:50
stendulkermorning lucasagomes :)07:52
*** yolanda has quit IRC07:54
*** yolanda has joined #openstack-ironic07:55
*** parlos has joined #openstack-ironic07:56
*** yolanda has quit IRC07:56
*** yolanda_ has joined #openstack-ironic07:56
*** Sukhdev has quit IRC07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:01
*** dtantsur|afk is now known as dtantsur08:01
dtantsurMorning Ironic08:01
*** hoangcx has quit IRC08:07
stendulkermorning dtantsur :)08:08
vsaienkomorning dtantsur stendulker, lucasagomes and all Ironic'ers!08:09
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: move create-node.sh to create_vms func  https://review.openstack.org/32285008:09
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Refactor enroll-node code  https://review.openstack.org/32319008:09
stendulkermorning vsaienko :)08:09
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
dtantsurhi stendulker, vsaienko08:10
*** sacharya has joined #openstack-ironic08:11
*** ohamada has joined #openstack-ironic08:11
*** fragatina has joined #openstack-ironic08:12
*** vishwanathj has joined #openstack-ironic08:13
*** vishwanathj has quit IRC08:15
*** sacharya has quit IRC08:16
*** vishwanathj has joined #openstack-ironic08:16
*** hoangcx has joined #openstack-ironic08:25
*** dmk0202 has joined #openstack-ironic08:26
*** MattMan has quit IRC08:30
*** MattMan has joined #openstack-ironic08:31
*** parlos has quit IRC08:31
*** vishwanathj is now known as vishwanathj_08:32
*** vishwanathj_ is now known as vishwanathj08:32
*** sdake_ has joined #openstack-ironic08:32
*** _vsaienko has joined #openstack-ironic08:33
*** _vs has joined #openstack-ironic08:33
openstackgerritMerged openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/32300208:34
*** sdake has quit IRC08:35
*** vdrok has joined #openstack-ironic08:37
vdrokmorning ironic!08:39
dtantsurmorning vdrok08:39
vdrokmorning dtantsur08:39
*** _vsaienko has quit IRC08:43
*** _vs has quit IRC08:43
*** _vsaienko has joined #openstack-ironic08:43
*** _vs has joined #openstack-ironic08:44
vdrokdtantsur, if you have a bit of time, very easy change - https://review.openstack.org/32295808:44
dtantsurwill take a look a bit later, thanks08:45
*** _vsaienko has quit IRC08:45
*** _vs has quit IRC08:45
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Refactor enroll-node code  https://review.openstack.org/32319008:45
vdroksure, thank you :)08:46
stendulkermorning vdrok :)08:51
*** milan has quit IRC08:51
*** milan has joined #openstack-ironic08:52
vdrokmorning stendulker !08:55
openstackgerritMerged openstack/ironic: Remove support for 'hexraw' iPXE type  https://review.openstack.org/32287508:58
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/32298608:59
*** kromanenko__ has joined #openstack-ironic09:00
*** Romanenko_K has joined #openstack-ironic09:00
*** kromanenko_ has joined #openstack-ironic09:00
*** electrofelix has joined #openstack-ironic09:01
*** ijw has joined #openstack-ironic09:02
*** derekh has joined #openstack-ironic09:03
*** ijw has quit IRC09:09
openstackgerritImre Farkas proposed openstack/ironic: [WIP] DRAC RAID configuration  https://review.openstack.org/17631909:10
openstackgerritVladyslav Drok proposed openstack/ironic: Revert "iscsi: wipe the disk before deployment"  https://review.openstack.org/32293209:20
*** daemontool has joined #openstack-ironic09:22
openstackgerritVladyslav Drok proposed openstack/ironic: Gracefully degrade start_iscsi_target for Mitaka ramdisk  https://review.openstack.org/31918309:24
openstackgerritVladyslav Drok proposed openstack/ironic: Revert "iscsi: wipe the disk before deployment"  https://review.openstack.org/32293209:27
*** chlong has quit IRC09:29
sambetts|afkMorning all09:36
*** sambetts|afk is now known as sambetts09:36
dtantsurhey sambetts09:38
milanmorning Ironic!09:38
milansambetts, dtantsur o/ :)09:38
dtantsurmorning milan :)09:38
sambettsMorning milan, dtantsur, did much happen yesterday while i was on holiday?09:38
dtantsurnext to nothing :) one Monday gate breakage that was quickly fixed09:39
milansambetts, not much with milan --- just created and destroyed a vm for grenade stuff ;)09:39
dtantsurI'm still fighting with my new laptop :(09:39
sambetts:( whats the problem?09:39
dtantsursambetts, it's Thinkpad T460s and it's not fully supported by *any* released Linux09:40
milan:-/09:40
dtantsurand has serious bugs with betas (e.g. suspend does not work for me on F24)09:40
sambetts:( oh man that sucks ... I thought the thinkpads were normally really good with linux support09:41
dtantsura lot of fun..... I miss my good old T530 whose only downside was its weight :)09:41
dtantsursambetts, intel did not do great job this time pushing drivers upstream09:41
sambettsah :(09:41
sambettsI was thinking about getting myself a ThinkPad X1 for personal use, I like how thin it is09:42
dtantsurif you use linux, don't take gen 4 (this year) for now09:43
dtantsurthe previous generation seems fine09:43
milandtantsur, you're in the office?09:43
sambettsthey using the same chips as in your new one?09:43
* milan hesitates to go or not to09:43
dtantsurmilan, no.. I planned on, but got distracted by $errands09:43
dtantsursambetts, yep09:44
milandtantsur, ok09:44
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/32298709:45
*** chlong has joined #openstack-ironic09:47
*** phuongnh has quit IRC09:47
*** phuongnh has joined #openstack-ironic09:49
openstackgerritMerged openstack/ironic: Use messaging notifications transport instead of default  https://review.openstack.org/32263909:49
milansambetts, I was wondering about this patch https://review.openstack.org/#/c/308379/ before the rebase you wanted to merge the processing and reapply functions09:49
milansambetts, should we make such a refactor a separate patch?09:50
sambettsmilan: right, I think there is a lot of shared logic there, and IMO passing data in via the unprocessed API should be more or less the same as the ramdisk sending some data to inspector (with some minor changes of course, e.g. no power off at the end)09:51
sambettsmilan: it may be quite a large refactor of the processing logic, so I wouldn't be too concerned about making it a separate patch I guess09:54
milansambetts, +109:54
milanwe could merge this change as it is now before it needs rebasing again?09:54
sambettsmilan: I also have some ideas about how to prevent us having to pass through things like power_off=False everywhere, e.g. checking Ironic to see if its already off09:54
sambettsmilan: Sure, give me a couple of minutes to finish up something and I'll go through it and see if it can be merged09:56
milanright, atm it's rather about stating an intention: in reapply I intend not to power off at all (no matter the state) in process I intend power off (if it is off, no need to do anything)09:56
milansambetts, sure09:56
milanthx09:56
vsaienkomorning sambetts, milan09:56
sambettsmilan: yeah good point09:56
sambettsvsaienko: Hi!09:56
milanmorning vsaienko :)09:56
*** sdake_ is now known as sdake09:57
vsaienkosambetts where I can check cisco CI status for https://review.openstack.org/#/c/32319009:57
sambettsvsaienko: It should have left a comment ... :/09:58
vdrokgood morning milan and sambetts09:58
sambettsvsaienko: All our CI stats are here: http://192.133.158.2:8080/view/openstack%20ironic/09:58
sambettsvsaienko: it seems like it still running through devstack on the Cisco CI09:59
sambettshttp://192.133.158.2:8080/view/openstack%20ironic/job/tempest-dsvm-ironic-pxe_ucs/1488/console09:59
sambettsfor that ^ job anyway09:59
vsaienkosambetts, that was exactly I looking for, thanks10:00
milanmorning vdrok :)10:00
sergekmorning vdrok, milan, vsaienko and all :)10:01
vdroko/10:01
*** _vsaienko has joined #openstack-ironic10:03
*** _vs has joined #openstack-ironic10:03
vsaienkosergek o/10:04
*** mgoddard has joined #openstack-ironic10:04
*** _vsaienko has quit IRC10:04
*** ijw has joined #openstack-ironic10:05
*** _vs has quit IRC10:05
*** _vs has joined #openstack-ironic10:06
*** YiboCai has quit IRC10:07
ishantsambetts, Hi10:08
sambettsMorning ishant10:09
ishantMorning sambetts10:09
ishantsambetts, I was going through https://review.openstack.org/#/c/277853 . I want to know how you will make macs_for_instance a no op .10:10
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844010:10
*** ijw has quit IRC10:10
*** pcaruana has joined #openstack-ironic10:11
ishantsambetts, A non admin tenant wont be able to update the mac of the port, so in non admin user context how ironic will change the mac of port?10:12
*** sacharya has joined #openstack-ironic10:12
ishantor am I missing something here10:12
dtantsurishant, what's the use case for non-admin to change a MAC?10:12
sambettsishant: We would simply not implement macs_for_instance in the Ironic virt driver, the default implmentation for that returns None10:12
*** _vs has quit IRC10:13
sambettsishant: The Ironic service user is an admin user, and can change things like the mac address of a port10:13
dtantsurah, this spec. got it.10:13
sambettsdtantsur: you say it like its a bad thing ;)10:13
ishantsambetts, ohh so ironic wont work on user context? It always works on admin context?10:14
dtantsur;)10:14
dtantsuryes. ironic is admin-only10:14
dtantsurnova is our non-privileged frontend10:14
ishantdtantsur, sambetts , thanks . I thought the user context gets passed to ironic from nova10:15
*** chlong has quit IRC10:15
ishantand then ironic uses that context10:15
*** _vsaienko has joined #openstack-ironic10:15
dtantsurno, nova has ironic admin credentials configured in nova.conf10:15
*** _vs has joined #openstack-ironic10:15
sambettsIronic has its own user for interservice communication and that should be an admin level user10:16
*** sacharya has quit IRC10:16
sambettsalso what dtantsur said :)10:16
ishantsambetts, dtantsur : thanks :) didnt know this10:17
dtantsursambetts, offtopic: do you know a good book to learn all this modern networking stuff (vlans, sdn and other buzzwords ;)10:17
* dtantsur is unacceptably clueless10:18
*** mgould|afk is now known as mgould10:18
vdrokI'd read that too :)10:19
sambettsdtantsur: https://www.packtpub.com/virtualization-and-cloud/learning-openstack-networking-neutron <- I have this on my desk at all times, it really helped me understand neutron better10:21
dtantsuroh, this is one we got on the vancouver summit, right?10:22
* dtantsur has it somewhere then10:22
sambettsRight, I picked it up in paris, and I tried to get as many people to get it in vancouver as possible because it was super useful10:23
dtantsurcool10:24
dtantsurI see the same author has something even newer: https://www.safaribooksonline.com/library/view/openstack-networking-essentials/9781785283277/10:24
sambettsooooo10:24
sambettsI wondered if there was a newer one, I've been looking out for it at the last 2 summit s10:25
* dtantsur adds both to his read list10:25
*** _vs has quit IRC10:25
*** _vsaienko has quit IRC10:25
dtantsursambetts, also there is the 2nd edition of your book from autumn 201510:25
dtantsurhttps://www.safaribooksonline.com/library/view/learning-openstack-networking/9781785287725/10:26
*** yolanda has joined #openstack-ironic10:27
*** yolanda_ has quit IRC10:27
*** chlong has joined #openstack-ironic10:27
* sambetts should update to one or other of the above 10:28
*** _vsaienko has joined #openstack-ironic10:29
*** _vs has joined #openstack-ironic10:29
* dtantsur got a one year safari subscription from RH, so he's like "READ ALL DA BOOKZZ!!1"10:32
sambettsLOL! I think I can get safari too, I need to look into that10:32
ishantsambetts, I like the idea of interface-attach and iterface-detach in nova ironic driver.10:32
sambettsishant: Thank you :) it'd really help with allow more customisation on the ironic networking side10:33
sambettsat a driver level10:34
ishantbut you should have specified this in VLAN-aware-baremetal spec also, because making macs_for_instance no-op seems incomplete10:34
sambettsishant: I'm not sure what you mean, they can both work independantly10:34
ishantsambetts, yes driver should have control over its nodes mac's10:34
*** _vsaienko has quit IRC10:35
*** _vsaienko has joined #openstack-ironic10:36
*** _vs has quit IRC10:37
*** _vs has joined #openstack-ironic10:37
*** _vs has quit IRC10:39
*** _vsaienko has quit IRC10:39
ishantsambetts, if you make macs_for_instance a no-op then how will ironic know about the neutron port which is created with neutron generated mac10:41
ishantI thought interface-attach will pass the info to ironic10:41
sambettsishant: making macs_for_instance a no-op doesn't change the fact we do plug_vifs10:42
sambettsmacs_for_instance doesn't do any Ironic port to neutron port mapping, it simply returns a list of mac addresses, plug_vifs does that mapping logic10:43
sambettsand in the VLAN aware BM spec that is modified very slightly, according to the spec10:43
*** _vsaienko has joined #openstack-ironic10:43
*** _vs has joined #openstack-ironic10:43
sambettsthe interface attach / detach spec, independant of the VLAN aware BM spec, is designed to moved the plug_vifs logic out of the nova driver, so that it can be customised by the Ironic network drivers10:44
ishantsambetts, okay got you , thanks10:46
sambetts:D10:46
sambettsishant: thanks for reviewing the specs10:46
sambettsthe more eyes the better10:47
ishantsambetts, so if I want to use virtual mac (neutron generated) I need the interface-attach and detach api's which passes vif details to then ironic driver and then driver can create its own ironic ports10:50
ishantsambetts, am i correct in my understanding :D10:50
ishantsambetts: brb10:56
*** stendulker has quit IRC10:58
jrolldtantsur: https://review.openstack.org/#/c/321491/2/doc/source/deploy/install-guide.rst@692 <- should we remove that or? maybe someone from tripleo can follow up with another patch since I don't know much about it? :)10:58
sambettsishant: Right, thats one way to do it, using that spec, because there is a direct mapping 1-1 of Ironic port to neutron port so you have to create a new Ironic port per virtual port (thats actually very similar to how we've implemented it for our card), once VLAN aware BMs merges, the model changes slightly because you can then assign multiple neutron ports to 1 ironic port, so the Ironic port10:59
sambettscan represent the physical nic on your flex card with its physical mac address, and then you can just map multiple neutron ports to that Ironic port, then at configure tenant networking time you can process that however your driver wants, e.g. creating new virtual nics or creating a neutron trunk or both, once both specs are merged all the doors open for us10:59
openstackgerritJim Rollenhagen proposed openstack/ironic: Remove links to github for OpenStack things  https://review.openstack.org/32149111:00
*** ishant has quit IRC11:01
dtantsurjroll, diskimage-builder should be installed via pip imo. I never tried what's written there.11:02
dtantsuralso morning jroll11:02
jrollmorning :)11:03
sambettsMorning jroll, dtantsur I've always installed DIB from pip too11:03
jrollwell, this is about system deps right?11:04
dtantsurjroll, I can only say that I never did that and it worked for me11:04
dtantsur"This script should die" (c) https://github.com/openstack/tripleo-incubator/blob/master/scripts/install-dependencies11:05
jrolldtantsur: and last time I ran DIB I need to install a bunch of things :P11:05
dtantsuralso this script does A LOT of things unrelated to DIB, like generation SSH keys and configuring libvirt11:06
jrollyeah11:06
dtantsurand would advise against using it11:06
dtantsurs/and/I/11:06
jrollI'm fine with just deleting that note11:06
dtantsuryeah, lets convert it to pip install for greater good11:06
*** ijw has joined #openstack-ironic11:06
jrollit already is :P11:07
jrollthat's just a note about deps11:07
dtantsurok :)11:07
*** phuongnh has quit IRC11:07
jrollhttp://docs.openstack.org/developer/ironic/deploy/install-guide.html#image-requirements11:07
*** dmk0202 has quit IRC11:07
* mgould feels the correct place to document dependencies for DIB is in DIB itself11:07
dtantsurmgould++11:07
dtantsurjroll, an unrelated question: what can cause the 1st cleaning to fail with "Block device /dev/sda is frozen and cannot be erased"?11:08
sambettsIs there a python-disk-image-builder system package we can point people at for those without pip/I guess it should also install all the right system deps too11:08
sambettsHi mgould11:08
mgouldhi sambetts11:08
jrolldtantsur: a locked SSD11:08
dtantsursambetts, it's called diskimage-builder on Fedora11:09
dtantsurjroll, do you know how to unlock such?11:09
mgouldsambetts: looks like there's a diskimage-builder RPM package11:09
jrolldtantsur: not off the top of my head, um11:10
openstackgerritJim Rollenhagen proposed openstack/ironic: Remove links to github for OpenStack things  https://review.openstack.org/32149111:10
*** vishwanathj has quit IRC11:10
*** vishwanathj has joined #openstack-ironic11:10
sambettsmgould, dtantsur: I guess that makes sense if RDO uses it :-P11:10
*** _vs has quit IRC11:10
*** _vsaienko has quit IRC11:10
jrollI'm not documenting how to install DIB without pip in this patch, y'all can put up another if you wish11:10
sambetts:-P11:11
sambettsjust my playing devils advocate again heh11:11
*** ijw has quit IRC11:12
jrolldtantsur: something like https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L71511:14
jrollbut with the right password, I guess, if this isn't working11:14
dtantsurmeh..11:14
dtantsurthanks jroll11:15
jrollnp11:15
jrollmight be able to do it in bios somehow11:15
dtantsuryeah.. I've dumped this conversation to the guy11:15
*** vishwanathj has quit IRC11:16
* dtantsur thinks their QE folk should work upstream more actively11:16
*** vishwanathj has joined #openstack-ironic11:16
milanmorning mgould, jroll11:16
*** hoangcx has quit IRC11:17
mgouldmorning milan11:18
*** divya has joined #openstack-ironic11:23
divya ConnectFailure: Unable to establish connection to http://172.26.24.110:6385/v1/nodes/detail11:23
divyanova-comp not starting11:23
divyawhat it could be11:23
dtantsurjroll, seems like this affects mitaka too: https://review.openstack.org/#/c/323301/11:34
dtantsurdivya, is ironic api even running?11:34
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Refactor enroll-node code  https://review.openstack.org/32319011:35
jrolldtantsur: waiting on CI but will +A when done11:35
mat128dtantsur, jroll: a typical BIOS will lock ATA devices on boot11:36
dtantsuraha11:36
mat128typical workarounds: go to sleep and come back, often unlocks the device (unfeasible in an automation/datacenter context)11:36
mat128look for a BIOS setting to disable that automatic lock on boot11:36
dtantsuryeah, not something you do on a server.11:36
dtantsurmat128, thanks!11:36
mat128or c) work with your provider to get that option11:36
jrollmat128: nice, thanks11:36
mat128for some reason this is not the kind of option they build into every bios11:37
mat128now, if you wan to fall back to doing shred instead of ATA secure erase, there's this work in progress by Julia: https://review.openstack.org/#/c/302819/11:38
mat128it's already in IPA, just missing the Ironic config part11:38
mat128so with a custom driver you can already use it11:38
divyaTimed out waiting for a reply to message ID--error observed in ironic-api11:39
divyadtantsur->nova compute, conductor not starting11:39
divyawhat could be the issue11:39
divyair-api is started dtantsur11:41
mat128divya: ir-api talks to ir-cond over bus, so I would look at that11:41
mat128divya: if nova-conductor is not starting, I would look at it's logs11:42
dtantsuryeah, the conductor should be started11:42
*** dmk0202 has joined #openstack-ironic11:42
*** vishwanathj has quit IRC11:46
divyamat128-n-cond logs placed in http://paste.openstack.org/show/506550/11:46
*** baoli has joined #openstack-ironic11:47
mat128n-cond couldn't reach ir-api11:47
mat128divya: shouldn't prevent n-cond from starting though11:47
mat128divya: from the box running n-cond, can you curl http://172.26.24.110:6385/11:47
divya curl http://172.26.24.110:6385/ {"default_version": {"status": "CURRENT", "min_version": "1.1", "version": "1.18", "id": "v1", "links": [{"href": "None/v1/", "rel": "self"}]}, "versions": [{"status": "CURRENT", "min_version": "1.1", "version": "1.18", "id": "v1", "links": [{"href": "None/v1/", "rel": "self"}]}], "name": "OpenStack Ironic API", "description": "Ironic is an OpenStack project which aims to provision baremetal11:48
divyathis is the output11:48
divyair-api logs are copied to http://paste.openstack.org/show/506551/11:49
mat128divya: your ir-api logs even show a successful /v1/nodes call11:51
mat128divya: I think your n-cond tracebacks might be outdated11:51
mat128divya: is ir-cond running?11:51
*** baoli has quit IRC11:52
divyair-cond is running mat12811:53
divyai am able to give ironic node-list11:53
divya ironic node-list +--------------------------------------+-------+---------------+-------------+--------------------+-------------+ | UUID                                 | Name  | Instance UUID | Power State | Provisioning State | Maintenance | +--------------------------------------+-------+---------------+-------------+--------------------+-------------+ | ea0c712b-ae13-425e-b81d-0f2e0ef1e49c | test1 | None          | Non11:53
sambettsthat means that ironic api is running11:53
sambettsnot the conductor11:53
divyawhat command for ironic conductor11:53
mat128changing a state would force it11:54
mat128ironic node-set-power-state test1 on11:54
divyaironic node-list +--------------------------------------+-------+---------------+-------------+--------------------+-------------+ | UUID                                 | Name  | Instance UUID | Power State | Provisioning State | Maintenance | +--------------------------------------+-------+---------------+-------------+--------------------+-------------+ | ea0c712b-ae13-425e-b81d-0f2e0ef1e49c | test1 | None          | powe11:55
divyayea11:55
divyai changed the power state to off11:55
divyai was changed11:55
mat128if it worked, that means ir-api -> ir-cond seems fine11:55
mat128because the conductor is doing the work in that case11:55
divyaok..11:55
mat128so besides logs, what's your original concern?11:55
divyabut why is my n-comp not starting11:55
mat128I believe it is starting but at some point it wasnt able to reach ir-api11:56
*** thrash|g0ne is now known as thrash11:56
mat128do you see your node when you do `nova hypervisor-list`11:56
divyano11:57
vsaienkomorning mat128, jroll, divya11:57
divya nova hypervisor-list +----+--------------------------------------+-------+---------+ | ID | Hypervisor hostname                  | State | Status  | +----+--------------------------------------+-------+---------+ | 22 | ea0c712b-ae13-425e-b81d-0f2e0ef1e49c | up    | enabled | +----+--------------------------------------+-------+---------+11:57
vsaienkojroll what do you think about https://review.openstack.org/#/c/322932/11:57
divyayes i see that11:57
mat128o/ Vasyl11:57
divyagood morning vsaienko11:57
*** baoli_ has joined #openstack-ironic11:58
mat128divya: that would be a yes ;)11:58
mat128divya: as far as I'm concerned, I don't see any problem in your stack11:59
mat128divya: your n-cond logs could have been when ir-api was not up yet11:59
mat128divya: communication is clearly working now because nodes are listed in your hypervisor-list11:59
*** dprince has joined #openstack-ironic12:00
*** mat128 is now known as mat128|brb12:00
nicodemosgood morning, ironic12:02
*** vishwanathj has joined #openstack-ironic12:02
vdrokmorning jroll mat128|brb nicodemos12:02
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add a log extension  https://review.openstack.org/24883212:07
*** _vsaienko has joined #openstack-ironic12:07
*** lucasagomes is now known as lucas-hungry12:07
* lucas-hungry brb lunch12:07
*** _vs has joined #openstack-ironic12:07
lucas-hungryjroll, morning, btw you and/or JayF have some time: https://bugs.launchpad.net/ironic/+bug/158714312:08
openstackLaunchpad bug 1587143 in Ironic "[RFE] Collect logs from IPA on deploy failure" [Wishlist,New] - Assigned to Lucas Alvares Gomes (lucasagomes)12:08
*** mgoddard has quit IRC12:08
*** ijw has joined #openstack-ironic12:08
*** mgoddard has joined #openstack-ironic12:08
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands  https://review.openstack.org/17251712:11
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands  https://review.openstack.org/21672712:11
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create  https://review.openstack.org/17246112:11
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Bring OSC plugin inline with approved spec  https://review.openstack.org/28416012:11
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add provision state commands  https://review.openstack.org/20611912:11
*** sacharya has joined #openstack-ironic12:13
*** ijw has quit IRC12:13
*** mat128|brb is now known as mat12812:15
*** sacharya has quit IRC12:17
*** xavierr_afk is now known as xavierr12:19
xavierrgood morning guys :)12:20
mat128morning vdrok xavierr12:23
vdrokmorning xavierr12:25
*** sdake has quit IRC12:27
*** trown|outtypewww is now known as trown12:27
xavierrhey mat128 vdrok o/12:32
*** appu103_ has quit IRC12:33
*** dtantsur is now known as dtantsur|bbl12:35
*** rbudden has joined #openstack-ironic12:36
*** watanabe_isao has joined #openstack-ironic12:36
TheJuliaGood morning12:44
*** parlos has joined #openstack-ironic12:46
*** parlos_ has joined #openstack-ironic12:46
mat128morning TheJulia12:49
*** parlos has quit IRC12:50
*** parlos_ has quit IRC12:50
*** ppiela has joined #openstack-ironic12:53
openstackgerritHugo Nicodemos proposed openstack/python-oneviewclient: Adding dynamic allocation operations  https://review.openstack.org/25869912:54
*** absubram has joined #openstack-ironic12:54
*** absubram_ has joined #openstack-ironic12:55
vdrokTheJulia, good morning12:58
*** lucas-hungry is now known as lucasagomes12:59
*** absubram has quit IRC12:59
*** absubram_ is now known as absubram12:59
*** Goneri has joined #openstack-ironic13:02
lucasagomesmat128, thanks for the review on the RFE, replied to your comment13:03
*** vmud213 has quit IRC13:03
mat128lucasagomes: oh great, thanks :)13:04
mat128I'd love to have that feature right now in production :)13:04
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Refactor enroll-node code  https://review.openstack.org/32319013:04
*** lindycoder has joined #openstack-ironic13:05
*** fragatina has quit IRC13:09
*** ijw has joined #openstack-ironic13:09
lucasagomesmat128, o/13:10
*** fragatina has joined #openstack-ironic13:10
*** wajdi has joined #openstack-ironic13:11
*** fragatina has quit IRC13:11
*** _vs has quit IRC13:11
*** _vs has joined #openstack-ironic13:12
*** rloo has joined #openstack-ironic13:12
*** wajdi_ has joined #openstack-ironic13:13
NobodyCamGood Morning Ironicers Hope everyone had a good weekend13:15
*** ijw has quit IRC13:15
*** fragatina has joined #openstack-ironic13:15
*** _vsaienko has quit IRC13:15
*** _vs has quit IRC13:15
*** _vsaienko has joined #openstack-ironic13:16
*** wajdi has quit IRC13:16
*** _vs has joined #openstack-ironic13:16
*** rbartal has joined #openstack-ironic13:21
*** ishant has joined #openstack-ironic13:21
rbartallucasagomes, ping13:21
lucasagomesrbartal, hi there13:21
*** _vs has quit IRC13:21
*** _vsaienko has quit IRC13:21
*** ayoung has joined #openstack-ironic13:22
*** _vsaienko has joined #openstack-ironic13:22
*** _vs has joined #openstack-ironic13:22
vdrokmorning NobodyCam !13:24
NobodyCamGood morning vdrok :)13:24
lucasagomesNobodyCam, vdrok good morning :-)13:25
NobodyCammorning lucasagomes :)13:25
vdrokgood afternoon Lucas :)13:25
*** Goneri has quit IRC13:26
*** ishant has quit IRC13:26
*** absubram has quit IRC13:33
TheJuliagood morning vdrok13:33
TheJuliagood morning NobodyCam13:34
NobodyCammorning TheJulia13:34
*** ChrisAusten has joined #openstack-ironic13:36
rloomorning vdrok, NobodyCam, TheJulia, lucasagomes13:38
vdrokmorning rloo13:38
lucasagomesrloo, hi there!13:38
NobodyCamgood morning rloo13:38
*** watanabe_isao has quit IRC13:38
*** electrofelix has quit IRC13:39
*** mtanino has joined #openstack-ironic13:39
*** yolanda_ has joined #openstack-ironic13:43
*** rloo has quit IRC13:43
*** Goneri has joined #openstack-ironic13:43
*** rloo has joined #openstack-ironic13:43
*** yolanda has quit IRC13:44
*** mgoddard_ has joined #openstack-ironic13:45
*** mgoddard has quit IRC13:45
*** thiagop has joined #openstack-ironic13:46
sambettso/ rloo TheJulia NobodyCam xavierr13:47
NobodyCammorning sambetts :)13:47
thiagopGood morning Ironic13:48
xavierrhey sambetts \o\13:48
NobodyCammorning thiagop and xavierr13:48
thiagopNobodyCam: o/13:49
xavierrmorning NobodyCam and thiagop13:49
NobodyCam:)13:49
*** KennethWilke has left #openstack-ironic13:49
thiagopTheJulia: after a few days out of the air to concentrated, the tests are now running on the new CI infra13:49
thiagopto concentrate*13:50
*** electrofelix has joined #openstack-ironic13:51
*** mtanino has quit IRC13:52
milanmorning NobodyCam rloo TheJulia thiagop  xavierr and all I've forgotten to mention :)13:53
thiagopmorning milan13:53
NobodyCammorning milan :)13:54
rloogood afternoon milan, sambetts, xavierr thiagop13:55
*** mgoddard__ has joined #openstack-ironic13:55
*** ametts has joined #openstack-ironic13:56
xavierrgood afternoon rloo :)13:56
*** ohamada has quit IRC13:58
*** mgoddard_ has quit IRC13:58
*** amotoki has quit IRC14:08
*** _vsaienko has quit IRC14:08
*** _vs has quit IRC14:09
*** ijw has joined #openstack-ironic14:11
*** wajdi_ has quit IRC14:12
*** d0ugal has quit IRC14:15
*** _vsaienko has joined #openstack-ironic14:16
*** _vs has joined #openstack-ironic14:16
*** d0ugal has joined #openstack-ironic14:16
*** amotoki has joined #openstack-ironic14:17
*** ijw has quit IRC14:17
TheJuliathiagop: \o/14:18
TheJuliagood morning rloo sambetts xavierr thiagop14:19
*** lucasagomes is now known as lucas-afk14:19
*** piet has joined #openstack-ironic14:22
xavierrmorning TheJulia :)14:22
*** parlos has joined #openstack-ironic14:31
mgouldmorning TheJulia14:32
xavierrhey mgould o/14:32
*** ohamada has joined #openstack-ironic14:33
*** ishant has joined #openstack-ironic14:34
*** vishwanathj has quit IRC14:38
*** amotoki has quit IRC14:38
*** mgoddard__ has quit IRC14:39
*** mgoddard has joined #openstack-ironic14:39
*** joprovost has joined #openstack-ironic14:40
mgouldxavierr: hi!14:40
vsaienkosambetts: does cisco-ci supports depends-on to devstack patches?14:42
sambettsvsaienko: it should do yes14:42
*** _vsaienko has quit IRC14:43
vsaienkosambetts: http://192.133.158.2:8080/view/openstack%20ironic/job/tempest-dsvm-ironic-pxe_ucs/1495/consoleFull14:43
*** _vs has quit IRC14:43
*** _vsaienko has joined #openstack-ironic14:45
*** _vs has joined #openstack-ironic14:45
sambettsvsaienko: the error I'm seeing there has nothing to do with the patch that triggered the change or the one it depends on :/14:46
sambettsoh actually hold on :/14:46
vsaienkosambetts: [localhost] out: 2016-05-31 14:24:45.388 | +++/opt/stack/ironic/devstack/lib/ironic:enroll_nodes:952 [m iniget_subsections /opt/stack/data/ironic/ironic_hw_nodes 'node-0\properties' [localhost] out: 2016-05-31 14:24:45.389 | /opt/stack/ironic/devstack/lib/ironic: line 952: iniget_subsections: command not found14:47
sambettsyeah, I just spotted that as I walked up the logs14:47
*** rbartal has quit IRC14:47
vsaienkosambetts: I can prepare temp patch to ironic that will add this function to ironic tree to tests that change14:48
sambettsvsaienko: something in my gut tells me that Depends-On on openstack/devstack might be a special edge case14:50
*** ayoung has quit IRC14:51
sambettsvsaienko: looking into it now, thanks for bringing it to my attention14:51
*** wajdi has joined #openstack-ironic14:52
vsaienkosambetts: thanks for help14:52
*** mtanino has joined #openstack-ironic14:55
*** _vs has quit IRC14:55
*** _vsaienko has quit IRC14:55
*** vishwanathj has joined #openstack-ironic14:58
*** rama_y has joined #openstack-ironic14:59
*** _vsaienko has joined #openstack-ironic15:04
*** _vs has joined #openstack-ironic15:04
thiagopkrtaylor: around?15:05
*** _vsaienko has quit IRC15:06
*** _vs has quit IRC15:06
*** rcernin has quit IRC15:07
*** daemontool_ has joined #openstack-ironic15:09
*** mjura has quit IRC15:09
krtaylorthiagop, yes, sup?15:09
thiagopkrtaylor: do we have the requirements for third party ci documented (specially that thing about 45 patches/day) somewhere?15:10
*** daemontool has quit IRC15:10
*** daemontool_ has quit IRC15:11
*** daemontool__ has joined #openstack-ironic15:11
*** moshele has quit IRC15:12
openstackgerritVladyslav Drok proposed openstack/ironic: Update the api-ref documentation for Ports  https://review.openstack.org/31318715:12
*** ijw has joined #openstack-ironic15:13
*** links has quit IRC15:15
krtaylorthiagop, we have the spec, but it does not document test sizing info, that would change over time so I'd think we wouldn't want to15:18
*** ijw has quit IRC15:18
thiagopkrtaylor: I'd rather have it at the spec and update it when the community agree to change the requirement that have it only in "our minds"15:20
thiagopkrtaylor: but, thanks for the info anyway ;)15:21
krtaylorthiagop, that works for me, I'll add it to the list of stuff to document15:21
*** rbudden has quit IRC15:23
JayFlucas-afk: curious if 1587143 might need a spec given the back and forth already in the bug. I'm with mat128 that we should not pass additional credentials to the agent.15:23
*** jjohnson2 has joined #openstack-ironic15:24
krtaylorthiagop, you have seen this?  https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/third-party-ci.html15:24
*** dprince has quit IRC15:25
vsaienkolucasagomes, dtantsur: please have look once you have a time: https://review.openstack.org/#/c/299318/. The patch adds a nv job to ironic-staging-drivers15:27
thiagopkrtaylor: I myself don't, but some people on my team did some time ago. I'm kinda taking it over.15:27
krtaylorthiagop, gotcha, that would be a good thing to read, and the pointers to the infra docs15:28
thiagopkrtaylor: will do. Today is documentation day. I'm writing how to reproduce the CI environment in cause of flodding #kidding15:29
thiagopin case*15:29
*** openstackgerrit has quit IRC15:33
*** openstackgerrit has joined #openstack-ironic15:33
JayFjroll: wdyt about backporting jcook and my changes for nova driver into nova stable? I think it's probably justified and a significant enough fix15:37
jrollJayF: sure? I don't have any say in what gets backported to nova :)15:37
JayFjroll: mikal suggested when he landed mine that we do a stable backport; and my fix depends on jcook's fix really15:37
jrollJayF: sure15:37
*** sacharya has joined #openstack-ironic15:38
jrollJayF: just saying, I don't think I'm the right person to ask about nova backports :)15:38
JayFFor something like that, would I cherry pick both patches and squash them into one, or would it be two stable fixes?15:38
jrollif there's a bug in stable we should fix it, in general :)15:38
JayFJust generally asking your opinion on if it's buggy enough to backport15:38
JayFnot an analysis on nova backport policies15:38
JayFlol15:38
jrollif the first is broken without the second, it's squashed, otherwise two, afaik15:38
jrollwell15:38
*** Sukhdev has joined #openstack-ironic15:38
JayFfirst is a good fix without the second, but incomplete15:38
jrollI tend to backport small bug fixes anyway15:39
jrollI'd just upload both and ask mriedem if he thinks you should squash15:39
*** vishwanathj has quit IRC15:40
sambettso/ JayF15:41
JayFo/15:41
*** dmk0202 has quit IRC15:42
*** ifarkas has quit IRC15:43
*** Sukhdev has quit IRC15:44
*** sabeen has joined #openstack-ironic15:44
mat128hey everyone :)15:45
TheJuliakrtaylor: thiagop: my $0.02 is documentation of sizing context is vital, and I think as long as any update details, in essence states "Hey, this may change at a later point in time, it is based on this xyz things as of blah", we will be best off in the long run.15:45
*** ayoung has joined #openstack-ironic15:45
mat128JayF: lucas-afk confirmed Ironic would do the upload to swift so that alleviates my concern15:45
krtaylorTheJulia, noted, and actually said that way, I agree completely15:50
*** lucas-afk is now known as lucasagomes15:52
lucasagomesJayF, hi there, I can make a spec for it if needed. But about the credentials specifically, as mat128 pointed out, I won't be passing it to IPA at all15:53
*** tesseract has quit IRC15:54
JayFYeah, I'll take a closer look and see15:54
lucasagomesJayF, I will get the logs gzipped and base64 enconded as a string from the result of the collect_logs_command() via the IPA API, and then Ironic itself will upload it to swift15:54
JayFif I like it and don't think it needs a spec, what's the process?15:54
JayFrfc -> rfc-approved?15:54
* JayF doesn't like an "approval process" that doesn't have voting :(15:54
mat128lucasagomes: I'm personally fine without a spec, but I think integration tests that work against all images is required15:54
lucasagomesJayF, yeah, if cores are OK with it we just change the tag15:54
mat128JayF: maybe we can introduce lightweight specs ;)15:55
lucasagomesmat128, yeah, so I'm now testing with DIB, I've tested with coreos and tinyipa (the ipa code is here: https://review.openstack.org/#/c/248832/)15:55
*** _vsaienko has joined #openstack-ironic15:55
*** _vs has joined #openstack-ironic15:55
mat128yeah I've seen the usage of dmesg + getting /var/log15:56
mat128what would be a good integration test for this? Is there anything that can be asserted?15:56
mat128(logs contained "Linux" somewhere? lol)15:56
lucasagomesmat128, yeah, for systems with systemd is easier because we just get the journal log which contains everything15:57
lucasagomesmat128, for systems withtout it I'm gzipping the whole /var/log + dmesg, we could extend it as needed15:57
JayFlucasagomes: if only we had some kind of voting and approval system to see if cores are OK with something :(15:57
*** vishwanathj has joined #openstack-ironic15:57
lucasagomesJayF, I iwll put a spec up15:58
lucasagomesJayF, I think that will be better for getting a consensus15:58
JayFI don't want you to have to write a spec just because I dislike the system though15:58
JayFI just never like ambiguity.15:58
JayFWhich means I'll just never approve a non-spec'd RFE bug15:58
JayFsince there's no reliable process to know when it's the right thing15:58
mat128JayF: wasnt there a document on how we do RFE different than blueprint that explained it?15:58
lucasagomesJayF, yeah, I hear that :-/ in fairness the RFE already looks like a spec15:58
*** vmud213 has joined #openstack-ironic15:58
lucasagomesso it won't be much work15:58
lucasagomesmat128, yeah, 1 sec15:58
lucasagomesmat128, http://docs.openstack.org/developer/ironic/dev/code-contribution-guide.html15:59
jrollJayF: if you aren't comfortable with 2-3 cores saying "yeah, spec-less is fine here" then ask for a spec15:59
*** pcaruana has quit IRC15:59
mat128JayF: easy way out: you change to rfe-approved and add a comment with "x, y and z were fine with this"16:00
*** dprince has joined #openstack-ironic16:01
vmud213TheJulia: Would you mind taking a look at https://review.openstack.org/#/c/302143/ ? It's already a +2 and pending for long time16:04
*** _vsaienko has quit IRC16:06
*** _vs has quit IRC16:06
*** rbudden has joined #openstack-ironic16:10
*** fragatina has quit IRC16:10
*** moshele has joined #openstack-ironic16:11
*** ijw has joined #openstack-ironic16:14
*** _vsaienko has joined #openstack-ironic16:15
*** _vs has joined #openstack-ironic16:15
JayFBackports for Ironic driver: https://review.openstack.org/323477 Unavailable hosts have no resources for use + https://review.openstack.org/323196 Ironic nodes with instance_uuid are not available are up for review against the stable/mitaka branch16:17
JayFjcook: ^ your change backport, fyi16:17
rlooJayF: I understand what you mean about specless rfe's. what jroll says though. even after that, i may still ask questions as you've noticed :)16:17
rlooJayF: as long as the info is somewhere, in spec or in the bug/RFE itself, with enough detail to know what/how/etc.16:17
lucasagomesrloo, can you take a look at that one specifically when you have time and say whether it needs a spec or not?16:18
* lucasagomes is really fine writing a spec for it if needed16:18
rloolucasagomes: darn, i shouldn't have said anything. give me a few minutes.16:19
JayFdtantsur|bbl: apparently I just pushed ^ one of those patches over your cherry pick. Didn't mean to stomp on you but both changes are needed anyway, so GMTA?16:19
lucasagomesrloo, hah no worries, only when you get some time really16:19
*** fellypefca has joined #openstack-ironic16:19
JayFlucasagomes: honestly, adding an extension to IPA should be treated like a new API being added16:19
JayFlucasagomes: which is something we call out as needing a spec typically16:19
lucasagomesJayF, yeah it does add a new command16:19
lucasagomesJayF, cool, fair enough I will put a spec up16:19
JayFso we need to have a spec16:19
JayFThanks!16:20
lucasagomesty for looking into it16:20
*** ijw has quit IRC16:20
rlooJayF, lucasagomes: OH, yes, if it affects the API, we should have a spec.16:20
lucasagomesrloo, API for IPA only16:21
lucasagomesbut still16:21
*** sabeen has quit IRC16:21
JayFI think it's easy to forget that adding a command/extension to IPA is an API change16:21
rloolucasagomes: Just skimmed the rfe, a spec would be good.16:21
lucasagomesrloo, ty will do it16:21
rloothx lucasagomes16:23
TheJuliavmud213: it is on my list, thank you16:23
vmud213TheJulia:nice.Thanks.16:24
*** ayoung has quit IRC16:25
JayFalineb: around? If so can you look at 321153 and read dtantsur|bbl's comment? I think he has the best idea of any of us re: naming but I didn't want to update the spec until you had looked at that too16:29
alinebJayF: yes, i forgot it was possible, but i'm totally ok with it16:31
JayFAwesome! I'll update the spec to reflect that16:32
JayFrloo: ^ that should make you happy re: metrics and naming. Thanks for pushing to reopen that b/c it will be much better than what we had before16:32
JayFI will push an updated spec with that design today16:33
*** _vsaienko has quit IRC16:33
*** _vs has quit IRC16:34
*** derekh has quit IRC16:34
*** garthb has joined #openstack-ironic16:35
devanandamorning, all16:36
*** morgabra has quit IRC16:36
*** morgabra has joined #openstack-ironic16:36
devanandakrtaylor, thiagop: generally, a spec doesn't get updated after it is approved / completed. it's meant more as a point-in-time reference than a living document. SO. If you add words about how many patches/day we expect, I would suggest leaving it open-ended and _not_ updating it16:38
*** e0ne has quit IRC16:38
*** sabeen has joined #openstack-ironic16:41
*** ayoung has joined #openstack-ironic16:42
*** fellypefca has quit IRC16:45
*** dprince has quit IRC16:51
*** dprince has joined #openstack-ironic16:51
NobodyCammorning JayF devananda mat128 jroll16:51
krtaylordevananda, agreed, thats what TheJulia was referring to, basically making the documentation say about sizing for test systems "as of this release there were xyz..."16:51
jrollhey NobodyCam16:52
NobodyCam:)16:52
*** trown is now known as trown|lunch16:52
*** spandhe has joined #openstack-ironic16:53
JayFgm16:55
*** links has joined #openstack-ironic17:00
*** clenimar has joined #openstack-ironic17:00
*** alex_xu has quit IRC17:00
*** rcernin has joined #openstack-ironic17:01
*** kromanenko_ has quit IRC17:02
*** kromanenko__ has quit IRC17:02
*** Romanenko_K has quit IRC17:02
thiagopmorning devananda, ok then :)17:02
*** alex_xu has joined #openstack-ironic17:02
*** vishwanathj has quit IRC17:10
*** links has quit IRC17:11
vdrokgood night!17:12
*** vdrok has quit IRC17:12
*** sabeen has quit IRC17:13
*** dtantsur|bbl is now known as dtantsur17:17
dtantsurJayF, morning. no worries as soon as we do get it backported :)17:18
dtantsuralso morning NobodyCam, devananda, TheJulia, xavierr, rloo and everyone else17:18
JayFgm17:19
NobodyCamhey hey dtantsur Good morning... just doing the catch-up thing for being off yesterday17:19
*** parlos has quit IRC17:19
JayFdtantsur: I also backported my second half of fixing that kind of case17:20
sambettsNight all, I'm heading off17:20
*** ijw has joined #openstack-ironic17:20
NobodyCamnight sambetts17:20
sambettscya NobodyCam17:20
*** sambetts is now known as sambetts|afk17:20
dtantsurJayF, awesome! Actually I should have waited for you to appear, but I wanted to get this patch out of my head :)17:20
dtantsurg'night sambetts|afk17:20
dtantsurjroll, hey, we finally got broken by testing ssh internals in mitaka too :) please review https://review.openstack.org/#/c/323506/17:22
dtantsurdevananda, ^^^17:22
devanandadtantsur: heh. looking17:22
*** penick has joined #openstack-ironic17:23
rloohi dtantsur17:24
NobodyCamahh the logs droped off the bug on that17:24
*** ijw has quit IRC17:25
devanandadtantsur: curious,what changed upstream?17:27
dtantsurdevananda, we're checking how one 3rd party function calls another. now they're calling them with a bit different signature17:28
dtantsurok, the cherry-pick passes the gate. please review17:31
*** ijw has joined #openstack-ironic17:31
*** mgould is now known as mgould|afk17:33
jcookJayF sweet!17:34
jcookJayF I saw the PR for the add on fix Friday, but I was done at that point. Glad to see it got merged and backported.17:34
devanandajust added ironic here: https://bugs.launchpad.net/ironic/+bug/158697917:35
openstackLaunchpad bug 1586979 in OpenStack Compute (nova) "AMQP 2.0 prevents services from starting" [Undecided,New]17:35
devanandabecause that has broken me locally and several folks downstream today17:35
dtantsurdevananda, I suggest it rather go to oslo.messaging17:36
dtantsurat first glance it does not like something we can fix17:36
dtantsurbut thanks for tracking it!17:36
devanandadtantsur: agreed17:37
dtantsurdevananda, btw http://docs.openstack.org/project-team-guide/stable-branches.html#review-guidelines says you can approve a backported provided by me (if you feel like ofc)17:38
*** e0ne has joined #openstack-ironic17:39
devanandahttps://bugs.launchpad.net/oslo.messaging/+bug/1586840  << oslo bug17:39
openstackLaunchpad bug 1586840 in oslo.messaging "amqp 2.0 breaks the kombu driver" [Undecided,In progress] - Assigned to Kirill Bespalov (k-besplv)17:39
*** jtomasek has quit IRC17:39
*** ohamada has quit IRC17:39
dtantsurdevananda, should we maybe mark our bug as duplicate? or do you want to keep it for tracking?17:39
devanandadepending on how long the oslo bug takes to fix, if we need to land a pin, it might be good17:40
devananda*to track it17:40
dtantsurgot it17:40
*** ishant has quit IRC17:40
dtantsurdevananda, (I know I ping you way too often, sorry :) I'm collecting feedback on the driver composition spec https://review.openstack.org/#/c/188370/ could you please at least take a look at API and upgrade parts which are totally new?17:41
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Collect system logs from IPA upon a deploy failure  https://review.openstack.org/32351117:41
lucasagomesmat128, JayF ^17:41
lucasagomeswhen you guys have some time17:42
lucasagomeswith that, I will call it a day :-)17:43
devanandaI wonder why https://bugs.launchpad.net/oslo.messaging/+bug/1586840 hasn't broken the gates yet17:43
openstackLaunchpad bug 1586840 in oslo.messaging "amqp 2.0 breaks the kombu driver" [Undecided,In progress] - Assigned to Kirill Bespalov (k-besplv)17:43
lucasagomeshave a great evening all, see you all tomorrow17:43
devanandalucasagomes: g'night!17:43
devanandadtantsur: ack, adding to my queue17:43
lucasagomesdevananda, oh morning!17:43
*** praneshp has joined #openstack-ironic17:43
*** lucasagomes is now known as lucas-dinner17:43
thiagopsee ya lucas-dinner17:43
dtantsurdevananda, because https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L60 and it's unlikely to get promoted17:43
dtantsurdevananda, remember, nova is in the constraints gate (unlike us)17:44
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Collect system logs from IPA upon a deploy failure  https://review.openstack.org/32351117:44
lucas-dinnerthiagop, see ya17:44
* lucas-dinner fixes a typo and go17:44
dtantsurg'night, lucas-dinner17:46
NobodyCamnight lucas-dinner17:47
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Collect system logs from IPA upon a deploy failure  https://review.openstack.org/32351117:48
* dtantsur likes the summary of ^^^ already. it's a big pain in the neck we can't do it now (compared to e.g. inspection)17:49
dtantsurjroll, btw I've stepped on your shoes a bit and proposed an ironicclient mitaka release (https://review.openstack.org/#/c/322891/). people were complaining we are not releasing an important bug fix17:51
*** vmud213 has quit IRC17:52
*** penick_ has joined #openstack-ironic17:52
*** praneshp has quit IRC17:52
*** Sukhdev has joined #openstack-ironic17:52
*** penick has quit IRC17:52
*** penick_ is now known as penick17:52
*** praneshp has joined #openstack-ironic17:53
devanandadtantsur: ah. I see. our requirements doesn't specify amqp, and oslo doesn't pin it: https://github.com/openstack/oslo.messaging/blob/master/requirements.txt#L3617:54
devanandaso this will actually break in a bifrost deploy17:54
devanandawhich is where it got reported downstream here17:54
dtantsurdevananda, correct. everything not using upper-constraints will probably get broken17:54
*** daemontool__ has quit IRC17:54
TheJulia:(17:55
*** daemontool has joined #openstack-ironic17:55
* devananda wonders if bifrost' gate is broken now17:55
dtantsurTheJulia, does it make sense for bifrost to use upper-constraints?17:55
TheJuliait actually does17:56
TheJuliawe just don't define the upper consstraints presently, if my memory is serving me on how it works17:58
devanandadtantsur: nova's requirements.txt doesn't pin it either, so anyone downstream who is installing nova from source // relying on "pip install -r requirements.txt" is going to be broken17:58
devanandaah, I see. the gate isn't breaking because of u-c, but downstream is17:59
dtantsuryes. only people using upper-constraints (like our gates) are saved for now17:59
dtantsurcorrect17:59
*** moshele has quit IRC18:01
*** fragatina has joined #openstack-ironic18:03
*** trown|lunch is now known as trown18:04
*** ChubYann has joined #openstack-ironic18:08
*** ChubYann has quit IRC18:09
*** ChubYann has joined #openstack-ironic18:09
*** sabeen has joined #openstack-ironic18:11
*** electrofelix has quit IRC18:12
openstackgerritGanesh Maharaj Mahalingam proposed openstack/bifrost: Install dependencies inside vagrant VM  https://review.openstack.org/32352318:21
gmmahaTheJulia: cinerama: trying to bring up the bifrost Vm with stable/mitaka and we found that the setup.sh fails with cryptography needing these packages18:21
gmmahathink we have fixed it the right way. your thoughts on that would help. https://review.openstack.org/32352318:21
*** moshele has joined #openstack-ironic18:24
cineramahey gmmaha we have a couple patches open in master which address this18:25
gmmahacinerama: aaah.. :) sorry should have looked for open patches18:25
cineramagmmaha, it's all good, thanks for the contribution18:25
gmmahacinerama: wold you like me to merge my patch with the other one thats open.. i saw that they havent updated the packages for yum based distros18:26
cineramagmmaha, we approved the other patches (there are two) which duplicate your change18:27
cineramagmmaha, if you need that backported to the mitaka branch that is something that needs to be done18:27
TheJuliagmmaha: it happens :)18:27
gmmahacinerama: yes definitely a backport would help.. is that a patch that i can push?18:27
cineramagmmaha, yes18:28
gmmahacinerama: cool.. will get them up asap..18:28
* gmmaha goes to find the other two patches18:28
gmmahaTheJulia: :)18:28
*** baoli_ has quit IRC18:29
openstackgerritRuby Loo proposed openstack/ironic: NOTIFICATION_TRANSPORT should be global  https://review.openstack.org/32352618:30
gmmahacinerama: i found one patch that you have approved, https://review.openstack.org/318943 but didnt find another that touches the yum packages18:31
gmmahaany chance you have them handy? I can keep hunting if you dont18:31
openstackgerritMerged openstack/bifrost: Document that ssh_public_key_path must be set  https://review.openstack.org/31278218:33
gmmahacinerama: nevermind.. found nit18:40
gmmahas/nit/it/18:40
openstackgerritMerged openstack/ironic: Remove links to github for OpenStack things  https://review.openstack.org/32149118:44
*** fragatina has quit IRC18:51
*** fragatina has joined #openstack-ironic18:52
openstackgerritMerged openstack/ironic: Remove hard-coded keystone version from setup  https://review.openstack.org/32302618:56
openstackgerritMerged openstack/ironic: Use a single uuid parameter in api-ref  https://review.openstack.org/32295818:59
gmmahacinerama: TheJulia: https://review.openstack.org/323535 && https://review.openstack.org/32353819:01
*** cinerama has left #openstack-ironic19:01
jrolldtantsur: ack on that stable patch + ironicclient release19:04
dtantsurthnx19:04
TheJuliagmmaha: Thank you!19:05
*** moshele has quit IRC19:05
*** e0ne has quit IRC19:07
*** thrash is now known as thrash|biab19:08
*** PollyZ has joined #openstack-ironic19:10
*** adu has joined #openstack-ironic19:12
*** dtantsur is now known as dtantsur|afk19:19
*** e0ne has joined #openstack-ironic19:20
*** e0ne has quit IRC19:28
*** _vsaienko has joined #openstack-ironic19:35
*** _vs has joined #openstack-ironic19:35
*** rama_y has quit IRC19:38
*** daemontool has quit IRC19:38
*** sdake has joined #openstack-ironic19:45
*** adu has quit IRC19:46
*** piet has quit IRC19:47
*** rbudden has quit IRC19:48
*** piet has joined #openstack-ironic19:48
*** sdake_ has joined #openstack-ironic19:49
*** thrash|biab is now known as thrash19:50
*** sdake has quit IRC19:51
*** moshele has joined #openstack-ironic19:51
*** ayoung has quit IRC19:54
*** Goneri has quit IRC19:55
*** ayoung has joined #openstack-ironic19:55
*** lucas-dinner has quit IRC19:59
*** rama_y has joined #openstack-ironic20:01
TheJuliarloo: you around?20:02
rlooin meeting, what's up?20:02
*** moshele has quit IRC20:03
TheJulialooking at manual cleaning, with an agent driver, and see https://github.com/openstack/ironic/blame/master/ironic/conductor/manager.py#L915 in the path.   Looks like witht he agent driver, 905 is always going to be met if there is not a failure with the driver, which makes me think that we never get to 917 properly20:03
*** lucasagomes has joined #openstack-ironic20:05
*** jcoufal has joined #openstack-ironic20:05
*** dmk0202 has joined #openstack-ironic20:05
rlooTheJulia: sorry, I have to listen20:05
TheJulia:)20:05
TheJuliaok20:05
*** rbudden has joined #openstack-ironic20:08
*** garthb_ has joined #openstack-ironic20:09
*** garthb has quit IRC20:10
*** rbudden has quit IRC20:10
TheJuliarloo: I have someone in an environment making a minor tweak and retrying20:13
*** daemontool has joined #openstack-ironic20:15
*** clenimar has quit IRC20:24
*** ayoung has quit IRC20:27
*** dmk0202 has quit IRC20:28
*** daemontool_ has joined #openstack-ironic20:31
*** clenimar has joined #openstack-ironic20:32
*** daemontool has quit IRC20:33
*** fragatina has quit IRC20:34
*** dprince has quit IRC20:35
*** baoli has joined #openstack-ironic20:35
*** dprince has joined #openstack-ironic20:37
*** _vsaienko has quit IRC20:41
*** _vs has quit IRC20:41
nicodemosgood night, everyone20:44
*** PollyZ__ has joined #openstack-ironic20:44
*** nicodemos is now known as nico-afk20:44
*** dmk0202 has joined #openstack-ironic20:44
*** PollyZ has quit IRC20:45
*** PollyZ__ is now known as PollyZ20:45
gmmahaTheJulia: the bifrost patches seem to be failing for the pycrypto package.. it fails even for the patches that fix the issue..20:47
gmmahai saw this in the logs, mkdir: cannot create directory ‘/opt/stack’: Permission denied20:47
*** moshele has joined #openstack-ironic20:49
*** Sukhdev has quit IRC20:50
*** xavierr is now known as xavierr_afk20:50
*** Sukhdev has joined #openstack-ironic20:51
*** dprince has quit IRC20:53
*** dprince has joined #openstack-ironic20:53
* TheJulia sighs20:53
gmmahaTheJulia: sorry, wish i could be of more help against just pointing out the obvious20:54
gmmahadont know enough about these tests to jump in a fix them.. atleast not right away20:54
TheJuliagmmaha: apparently while I was out on medical leave, pycrypto disappeared from the base images.  That fix got merged by putting it in requirements.txt, although appraently spawned an argument of how installation of ansible is occuring :\20:54
gmmahaohhhh20:56
*** wajdi_ has joined #openstack-ironic20:57
*** joprovost has quit IRC20:57
*** jcoufal has quit IRC20:57
gmmahaTheJulia: the ansible package itself? and guessing folks rather use the ppa piece?20:58
*** trown is now known as trown|outtypewww20:58
TheJuliasome folks would rather the requirement be managed as part of requirements.txt it's self, the problem is forcing that could result in some very bad things occuring20:59
*** wajdi has quit IRC21:00
TheJuliaOSA uses 1.9.4 afaik... installed from their git repo, although apparently the direct invokation of "Hey, we need pycrypto" is missing in the entire path somewhere, at least that was my takeaway from the whirlwind that was last week.  Truthfully it is a post-it note on my wall21:00
*** wajdi_ has quit IRC21:01
gmmahaTheJulia: :)21:01
*** daemontool_ has quit IRC21:02
*** rama_y has quit IRC21:03
*** mjturek1 has quit IRC21:10
*** garthb_ has quit IRC21:12
*** garthb_ has joined #openstack-ironic21:14
*** jjohnson2 has quit IRC21:19
PollyZwhen I was running “git review -s”, it gives me this error “raceback (most recent call last):21:19
PollyZ  File "/usr/bin/git-review", line 10, in <module>21:19
PollyZ    sys.exit(main())21:19
PollyZ  File "/usr/lib/python2.7/dist-packages/git_review/cmd.py", line 1157, in main21:19
PollyZ    raise no_git_dir21:19
PollyZgit_review.cmd.GitDirectoriesException: Cannot determine where .git directory is.21:19
PollyZThe following command failed with exit code 12821:19
PollyZ    "git rev-parse --show-toplevel --git-dir"21:19
PollyZfatal: Not a git repository (or any of the parent directories): .git21:19
PollyZ21:19
PollyZhow can i fix it? thanks21:19
mat128you're not in a git repo21:21
mat128PollyZ: try `git status`21:21
openstackgerritJay Faulkner proposed openstack/ironic-specs: Update metrics spec to reflect review comments  https://review.openstack.org/32115321:22
*** causten_ has joined #openstack-ironic21:22
PollyZi went to the ironic folder, and run the “git review -s”, it told me “Fetching gerrit      fatal: Project not found: openstack/project       fatal: Could not read from remote repository. ”21:23
*** dprince has quit IRC21:23
JayFPollyZ: what is the output of `git status` on that dir?21:23
*** ayoung has joined #openstack-ironic21:23
JayFAlso, everyone say hi to PollyZ, she's interning with Rackspace this summer on the OnMetal team21:23
*** clenimar has quit IRC21:24
PollyZ“On branch TOPIC-BRANCH21:25
PollyZUntracked files:21:25
PollyZ  (use "git add <file>..." to include in what will be committed)21:25
PollyZetc/ironic/ironic.conf.local21:25
PollyZnothing added to commit but untracked files present (use "git add" to track)21:25
PollyZ21:25
JayFwhen you need to paste stuff in that's more than 2 lines, you can go to gist.github.com and paste it in there, submit it, and send a link into here21:25
PollyZthx Jay21:25
JayFPollyZ: what about `git remote -v` on that dir?21:25
*** ChrisAusten has quit IRC21:26
PollyZhttps://gist.github.com/PollyZ/f74c1ccfbdc9746a464e6b5df9fc0bb721:27
PollyZlike this?21:27
JayFand from that dir, you ran git review -s and it gave that error?21:28
jrollgerrit ssh://PollyZhou@review.openstack.org:29418/openstack/project.git21:28
*** praneshp has quit IRC21:28
jrollopenstack/project is the problem there21:28
JayFoh good eye21:29
JayFPollyZ: git remote rm gerrit # should remove the bad remote; then try git review -s again21:29
gmmahaTheJulia: anyway i can help, please do let me know..21:31
PollyZi entered “yes”, it gave me this https://gist.github.com/PollyZ/f74c1ccfbdc9746a464e6b5df9fc0bb721:32
JayFAh, good, that's a better error21:33
JayFI think you missed a step in the docs, let me find it21:33
*** piet has quit IRC21:33
*** rama_y has joined #openstack-ironic21:33
*** piet has joined #openstack-ironic21:34
PollyZAt first I entered “openstack/project”, later on, I caught that, so I modified to “/ironic”, but it didnt work21:34
JayFPollyZ: http://docs.openstack.org/infra/manual/developers.html#account-setup at the very end of this section, just before "installing git-review", there's a paragraph starting "You'll also want to upload an SSH key to Gerrit"21:35
JayFthat step was either missed or done incorrectly21:35
JayFbecause gerrit doesn't know who you are.21:35
PollyZActually I have done that, and it seems to be right. https://gist.github.com/PollyZ/f74c1ccfbdc9746a464e6b5df9fc0bb721:37
*** dmk0202 has quit IRC21:38
JayFNo, for that part, you actually will have to login to review.openstack.org with the account you created21:39
JayFand upload a copy of your public key21:39
JayFit's not a local config that's missing21:39
PollyZyou mean the ssh key right21:40
JayFyes, your ssh public key21:40
mat128gerrit normally uses a "gerrit" remote in your repo21:40
jrollhere, specifically: https://review.openstack.org/#/settings/ssh-keys21:40
mat128that it sets up when you do git review -s21:40
JayFmat128: if you see two gists ago, we're to git-review throwing ssh auth errors21:40
PollyZdid that.21:40
mat128oops21:41
mat128JayF: lost track, my bad21:41
mat128Ahh21:41
mat128JayF, PollyZ: editing the same gist21:41
mat128confusing :)21:41
JayFI didn't even notice, I just click, read, close21:42
*** rama_y has quit IRC21:42
rlooTheJulia: sorry, meetings are over.21:42
mat128alright, signing off21:42
rlooTheJulia: yes that is right, it doesn't get to 917.21:43
JayFPollyZ: if you're continuing to have trouble, I can jump on a vidyo and pair to help you get this setup for about 30m or so.21:43
*** rbudden has joined #openstack-ironic21:43
TheJuliarloo: so.... sounds like we have a bug then :(21:44
rlooTheJulia: should it? the agent always has to prepare cleaning which is async.21:44
rlooTheJulia: what is the bug?21:44
PollyZYes please Jay!21:44
PollyZhow to tag someone btw?21:44
rlooTheJulia: if i recall correctly the agent heartbeats, and there is code that ends up calling the conductor's _do_next_clean_step.21:44
rlooTheJulia: do you want me to check?21:45
jrollPollyZ: in irc? just say their nickname like I just did to you :)21:45
* jroll says JayF for another example21:45
PollyZjroll: like that21:45
jrollyep21:45
PollyZcool. Thanks21:45
jrollnp21:46
jrollyour client should also be able to tab complete names so you don't need to type them all out21:46
TheJuliarloo: Oddly I didn't see that, but I also found out that my user might have some unknown code, they pieced together their isntall quite a bit by hand :(   If you can quickly verify that heartbeat does the right thing, then It might be a good starting point for backtracking if their install from current known-working source code goes sideways21:47
*** piet has quit IRC21:47
*** baoli has quit IRC21:47
*** krtaylor has quit IRC21:47
rlooTheJulia: looking ...21:47
*** baoli has joined #openstack-ironic21:48
rlooTheJulia: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L49421:49
TheJuliathanks21:49
TheJuliaI just found that myself, so it must have been some vendor package issue or something.  ERRINSUFFICENTINFO at the moment21:50
rlooTheJulia: phew :)21:50
PollyZJayF: I am on my vidyo room now21:50
rlooTheJulia: I mean, phew, the problem seems to be at your end :)21:50
TheJulialol21:51
*** fragatina has joined #openstack-ironic21:52
TheJuliaYeah, I have an open ticket to get access to the system in question to investigate further since, something is not right21:52
*** rbudden has quit IRC21:55
NobodyCamTheJulia: I expect ticket will be worked on until tomorrow thou :(22:01
TheJuliaNobodyCam: I'm going to call it a night then22:02
openstackgerritMerged openstack/ironic-python-agent: Consolidate IPA configuration into a config module  https://review.openstack.org/32086722:03
NobodyCam++22:03
*** moshele has quit IRC22:07
*** baoli has quit IRC22:08
*** baoli_ has joined #openstack-ironic22:08
*** sdake has joined #openstack-ironic22:10
*** sdake_ has quit IRC22:12
*** ametts has quit IRC22:14
*** clenimar has joined #openstack-ironic22:22
*** rama_y has joined #openstack-ironic22:23
*** praneshp has joined #openstack-ironic22:30
*** baoli_ has quit IRC22:40
*** lindycoder has quit IRC22:43
*** causten_ has quit IRC22:49
*** PollyZ has quit IRC23:08
*** piet has joined #openstack-ironic23:11
JayFjroll: https://review.openstack.org/#/c/318188/ has 4x+2 2x+1 but I figure it's the sort of think you should land, if you get a moment in the morning23:11
JayFcan someone with stable perms land this? been outstanding a long time https://review.openstack.org/#/c/311373/ (IPA requirements bump)23:22
*** clenimar has quit IRC23:27
*** piet has quit IRC23:27
JayFlucasagomes: around?23:29
mrdaJayF: wrong time of day :)23:29
JayFthat's what I thought, but usually the /nick would be -afk or something23:30
JayFlucasagomes: just -1'd a bugfix you probably care about it; I can talk about it a little in the morning if that's the absolute last resort (re: https://review.openstack.org/#/c/302143)23:34
JayFThere are a very large amount of outstanding, 1x+2 patches to devstack/lib/ironic -- I've reviewed almost all of them personally too, and they seem to be important fixes for one of our 3rd party CI folks if someone wants to review and potentially approve them23:39
*** sabeen has quit IRC23:39
openstackgerritMerged openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/32306723:52
*** chlong has quit IRC23:53
*** wajdi has joined #openstack-ironic23:57

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