Tuesday, 2015-10-06

devanandajroll: ah - the ilo driver doesn't assert that clean_up_ramdisk is not called00:00
*** alexpilotti has quit IRC00:00
devanandathis is complicated indeed00:00
jrolldevananda: yeah, indeed00:01
*** naohirot has joined #openstack-ironic00:02
devanandajroll: I feel like we should change the ilo driver (it shouldn't be a big change) since it is in tree and can be changed00:02
devanandajroll: so that we can preserve compat for out of tree drivers00:02
devanandaI'd also like ramesh to weigh in00:02
jrolldevananda: probably, yeah, I'd like to see that change00:03
devanandayea, looking at what it'll take now, but i'm not as familiar with the ilo driver code and tests, so it's a bit slower00:03
devanandaalso it's 5pm and I need to catch a bus soon, so I may not get it today00:03
jrollright on00:03
jrollmaybe send an email so ramesh can chip in overnight?00:03
*** garthb has quit IRC00:04
jlvillallifeless: Yeah, I looked around but nothing is jumping out at me why it doesn't fail :(00:06
Sukhdevjroll: That was the issue - it is now working fine after I made that change -00:06
jrollSukhdev: makes sense00:06
Sukhdevjroll: I post a comment on the patch so that it get fixed correctly00:07
jrollok00:07
crinklecould someone help point me in the right direction for this pxe issue? http://paste.openstack.org/show/475387/00:08
jrollcrinkle: ipxe.exe != ipxe.pxe00:10
crinklejroll: thanks, though same issue when i correct it00:10
jrollcrinkle: tftp 10.10.16.131 -c get ipxe.pxe fails?00:11
crinklejroll: yes00:11
jrollhrm00:11
jrollpermissions maybe?00:12
jlvillalcrinkle: are you using --secure?00:14
jlvillalcrinkle: What tftpd server are you using?00:14
crinklejroll: looks readable to me http://paste.openstack.org/show/475388/00:14
jlvillalI'm curious how you started the tftpd server00:14
crinklejlvillal: just dnsmasq, this is from a bifrost install00:14
jrollcrinkle: this is beyond my knowledge then00:15
*** alexpilotti has joined #openstack-ironic00:15
jlvillalcrinkle: Can you figure out the command line arguments?  'ps axww | grep tftpd'00:15
crinklejlvillal: in.tftpd -v -v -v -v -s --map-file /tftpboot/map-file /tftpboot00:16
* jlvillal wonders what -s does00:16
*** tsekiyama has quit IRC00:17
crinkleman in.tftpd says it's --secure00:17
jlvillalcrinkle: I think -s is --secure00:17
jlvillalWhich means it chroots into /tftpboot/00:17
jlvillalSo the tftpd daemon only sees /tftproot/ as /00:18
jlvillalTry this00:18
jlvillalcrinkle: ln -s /tftproot/tftproot /tftproo00:18
jlvillalcrinkle: ln -s /tftproot/tftproot /tftproot00:18
jlvillalI wonder if that will work00:18
*** tsekiyama has joined #openstack-ironic00:18
jlvillalI think the map file needs to be fixed.00:18
*** mitchjameson has quit IRC00:18
jlvillalcrinkle: I haven't played with bifrost yet (this week is the plan). So I'm kind of guessing here as I don't know what magic bifrost is doing00:19
jlvillalTheJulia: cinerama ^^^^  Any ideas?00:19
*** alexpilotti has quit IRC00:20
crinklejlvillal: you mean tftpboot not tftproot, yes?00:20
jlvillalcrinkle: Yes, sorry!00:21
crinklejlvillal: that link doesn't work but this gives me a really good direction to look in00:21
crinklejlvillal: thanks!00:21
*** achanda has joined #openstack-ironic00:21
jlvillalcrinkle: Good luck.00:22
*** shadower has quit IRC00:23
*** shadower has joined #openstack-ironic00:23
*** penick has quit IRC00:23
cineramahaving a look now00:23
devanandajlvillal: you have a NUC, right?00:26
jlvillaldevananda: Yep00:26
devanandajlvillal: https://github.com/devananda/bifrost/tree/nuc-demo-branch + https://etherpad.openstack.org/p/isnt-it-ironic-walkthrough00:27
cineramacrinkle, silly question here, what distro are you using?00:27
jlvillaldevananda: Thanks! I will check it out tomorrow. I have to leave in about 10 minutes.00:27
devanandaif you are going to play with it anyway ... that may be both a good jumping off point for you AND help us streamline the walkthrough / demo00:27
jlvillalmrda: ^^^ devananda's link00:27
jlvillaldevananda: mrda also has an AMT NUC00:27
*** jobewan has joined #openstack-ironic00:28
crinklecinerama: ubuntu trusty00:29
mrdathanks00:29
*** mtanino has quit IRC00:29
cineramacrinkle, ah ok00:29
devanandaI need to run as well -- see ya'll tmw00:29
jlvillaldevananda: Ciao00:29
*** penick has joined #openstack-ironic00:32
crinklejlvillal: cinerama changing the map-file to map to / instead of /tftpboot seems to make it work00:33
jlvillalcrinkle: Cool :)00:35
*** alexpilotti has joined #openstack-ironic00:35
*** tsekiyama has quit IRC00:36
*** alexpilotti has quit IRC00:39
* TheJulia reads00:40
TheJuliacrinkle: try "/ipxe.pxe"00:41
TheJuliahmm...  permissions?00:41
*** davideagnello has quit IRC00:42
jrollTheJulia: 00:33:15         crinkle | jlvillal: cinerama changing the map-file to map to / instead of /tftpboot seems to make it work00:42
crinkleTheJulia: i think /ipxe.pxe would have worked00:43
TheJuliajroll: thx, didn't see that00:43
crinkleTheJulia: i don't think it was a permission issue, you can verify http://paste.openstack.org/show/475388/00:43
TheJulianah, its definitly the rewrite + with the chroot00:43
crinkleyep00:44
*** davideagnello has joined #openstack-ironic00:44
TheJuliaits I thought I fixed that... oh00:44
TheJuliayou know what.... the test vms boot ipxe directly00:44
TheJuliavia http00:44
TheJuliaso this actually doesn't get tested in the gate since the dhcp response actually includes enough info to say "go over there"00:45
crinklehmm so why am i hitting it then?00:45
TheJuliainfracloud ?00:45
crinkleyep00:46
TheJuliaso that is actually physical gear that wants to start with PXE00:46
*** davideagnello has quit IRC00:46
TheJuliathe VMs that we fire up in the gate directly go to iPXE if the dhcp server crafts the replies00:47
TheJulias/crafts/has specific replies00:47
*** davideagnello has joined #openstack-ironic00:47
*** Sukhdev has quit IRC00:47
TheJuliaalthough, I could have sworn that I tested with physical gear, specifically my supermicro box after mucking with the tftp/pxe stuff last00:48
greghayneswe also have the funky mellanox chainloading00:49
greghaynesnot sjre if relavent00:49
TheJuliashouldn't be00:49
TheJuliabut yeah00:49
TheJuliathat was a fun one :(00:49
crinkleit's possible i need to update ironic/bifrost to latest00:49
*** dims_ has joined #openstack-ironic00:50
TheJuliacrinkle: what commit are you at?00:50
*** saripurigopi has joined #openstack-ironic00:51
*** mgoddard_ has joined #openstack-ironic00:52
crinklechecking...ironic is on latest, bifrost is patch 216431 but looks like it needed a rebase against master00:53
*** dims_ has quit IRC00:55
*** mgoddard__ has quit IRC00:55
*** alexpilotti has joined #openstack-ironic00:56
*** mgoddard__ has joined #openstack-ironic00:56
TheJuliacrinkle: http://git.openstack.org/cgit/openstack/bifrost/tree/playbooks/roles/bifrost-ironic-install/templates/dnsmasq.conf.j2#n43900:57
*** mgoddard_ has quit IRC00:57
*** mgoddard_ has joined #openstack-ironic00:57
*** dims_ has joined #openstack-ironic00:58
crinkleTheJulia: that must be it, i have dhcp-boot=tag:gpxe,ipxe.pxe00:59
crinkleTheJulia: i'll just update, thanks!01:00
*** alexpilotti has quit IRC01:00
*** mgoddard__ has quit IRC01:00
*** baoli has joined #openstack-ironic01:06
*** penick has quit IRC01:12
*** alexpilotti has joined #openstack-ironic01:16
*** baoli has quit IRC01:19
*** baoli has joined #openstack-ironic01:20
*** alexpilotti has quit IRC01:20
*** dims_ has quit IRC01:30
*** baoli has quit IRC01:34
*** praneshp has quit IRC01:34
*** alexpilotti has joined #openstack-ironic01:36
*** tsekiyama has joined #openstack-ironic01:37
*** baoli has joined #openstack-ironic01:37
*** alexpilotti has quit IRC01:41
*** fandi has quit IRC01:41
*** tsekiyama has quit IRC01:42
*** baoli has quit IRC01:48
*** baoli has joined #openstack-ironic01:48
*** alexpilotti has joined #openstack-ironic01:56
*** alexpilotti has quit IRC02:00
*** mgoddard__ has joined #openstack-ironic02:03
*** mgoddard_ has quit IRC02:04
*** mgoddard_ has joined #openstack-ironic02:04
*** jobewan has quit IRC02:07
*** baoli has quit IRC02:08
*** mgoddard__ has quit IRC02:08
*** smoriya has joined #openstack-ironic02:12
*** alexpilotti has joined #openstack-ironic02:16
*** saripurigopi has quit IRC02:18
*** alexpilotti has quit IRC02:20
*** jobewan has joined #openstack-ironic02:22
openstackgerritRuby Loo proposed openstack/ironic-specs: Manual cleaning  https://review.openstack.org/23128802:30
*** alexpilotti has joined #openstack-ironic02:36
*** alexpilotti has quit IRC02:41
*** sdake has joined #openstack-ironic02:49
*** achanda has quit IRC02:50
*** jamielennox has quit IRC02:51
*** rloo has quit IRC02:54
*** alexpilotti has joined #openstack-ironic02:57
*** alexpilotti has quit IRC03:01
*** Nisha has joined #openstack-ironic03:04
*** sdake has quit IRC03:09
*** sdake has joined #openstack-ironic03:10
*** fandi has joined #openstack-ironic03:10
*** jamielennox has joined #openstack-ironic03:13
*** Nisha has quit IRC03:13
*** sdake has quit IRC03:13
*** lazy_prince has joined #openstack-ironic03:15
*** alexpilotti has joined #openstack-ironic03:17
*** alexpilotti has quit IRC03:21
*** killer_prince has joined #openstack-ironic03:28
*** lazy_prince has quit IRC03:31
*** Nisha has joined #openstack-ironic03:32
*** saripurigopi has joined #openstack-ironic03:32
saripurigopiMorning Ironic03:32
*** sdake has joined #openstack-ironic03:33
*** alexpilotti has joined #openstack-ironic03:37
*** ijw has quit IRC03:39
*** alexpilotti has quit IRC03:42
*** links has joined #openstack-ironic03:48
*** praneshp has joined #openstack-ironic03:48
*** rameshg87 has joined #openstack-ironic03:53
*** VikasC has joined #openstack-ironic03:55
*** pradipta has joined #openstack-ironic03:56
*** alexpilotti has joined #openstack-ironic03:57
*** alexpilotti has quit IRC04:01
*** lazy_prince has joined #openstack-ironic04:04
*** puranamr has joined #openstack-ironic04:07
*** killer_prince has quit IRC04:08
*** puranamr has quit IRC04:12
*** alexpilotti has joined #openstack-ironic04:17
*** alexpilotti has quit IRC04:22
*** Nisha has quit IRC04:23
*** Sukhdev has joined #openstack-ironic04:23
*** Nisha has joined #openstack-ironic04:23
*** Marga__ has quit IRC04:26
*** praneshp_ has joined #openstack-ironic04:27
*** jobewan has quit IRC04:29
*** praneshp has quit IRC04:30
*** praneshp_ is now known as praneshp04:30
jlvillalmrda: Since you know more details than me. Any chance you can comment in: https://bugs.launchpad.net/ironic/+bug/1499489  ?04:32
openstackLaunchpad bug 1499489 in Ironic "AMT is not supported with Ironic-Python-Agent" [Undecided,Incomplete] - Assigned to John L. Villalovos (happycamp)04:32
mrdathanks jlvillal04:35
*** alexpilotti has joined #openstack-ironic04:37
*** amotoki has joined #openstack-ironic04:38
*** achanda has joined #openstack-ironic04:42
* mrda tries and remember which vm he did the amt work in :-S04:42
*** alexpilotti has quit IRC04:43
*** Nisha has quit IRC04:43
*** Nisha has joined #openstack-ironic04:43
rameshg87mrda: jlvillal: request your review and help on https://review.openstack.org/#/c/211101/104:46
*** dims_ has joined #openstack-ironic04:46
rameshg87mrda: jlvillal: this is for support in nova for booting bare metals from cinder volume04:46
jlvillalrameshg87, Sounds cool :) Not sure I will look tonight though. Already in bed...04:47
rameshg87jlvillal: sure, have a good sleep instead of thinking about cinder volumes ;-)04:47
rameshg87good night04:47
jlvillalrameshg87, Thanks!04:48
*** jobewan has joined #openstack-ironic04:49
mrdarameshg87: I'll take a look04:51
rameshg87thanks mrda04:52
mrdajlvillal: commented on this.  please let me know if it isn't enough.04:57
*** alexpilotti has joined #openstack-ironic04:58
jlvillalrameshg87, Left basically grammar comments...04:58
rameshg87jlvillal: sure, will have a look at it, thanks04:59
jlvillalmrda, Thanks!05:00
mrdajlvillal: np05:00
jlvillalOkay, I'm calling my work day/night over ;)05:00
jlvillalGood night05:00
* mrda is quite capable using others' work05:00
mrdaNight jlvillal - have a good one!05:00
jlvillalmrda, Thanks. Have a great day.05:00
*** _fortis has quit IRC05:02
*** alexpilotti has quit IRC05:02
*** morgan has quit IRC05:02
*** coreycb has quit IRC05:03
*** devananda has quit IRC05:03
*** saripurigopi1 has joined #openstack-ironic05:03
*** saripurigopi has quit IRC05:03
*** tristanC has quit IRC05:04
*** tristanC has joined #openstack-ironic05:05
*** morgan has joined #openstack-ironic05:06
*** coreycb has joined #openstack-ironic05:07
openstackgerritgreghaynes proposed openstack/bifrost: Fix fail if network_interface contains '-'  https://review.openstack.org/23077905:09
*** devananda has joined #openstack-ironic05:09
*** sdake_ has joined #openstack-ironic05:09
*** sdake has quit IRC05:13
*** _fortis has joined #openstack-ironic05:16
*** alexpilotti has joined #openstack-ironic05:18
*** Nisha has quit IRC05:19
*** Nisha has joined #openstack-ironic05:20
*** alexpilotti has quit IRC05:22
*** davideagnello has quit IRC05:30
*** davideagnello has joined #openstack-ironic05:30
*** pradipta has quit IRC05:31
*** Marga_ has joined #openstack-ironic05:32
openstackgerritRamakrishnan G proposed openstack/ironic: Refactor deploy_utils methods  https://review.openstack.org/22226405:33
*** Marga_ has quit IRC05:36
*** Marga_ has joined #openstack-ironic05:36
*** sdake_ has quit IRC05:38
*** alexpilotti has joined #openstack-ironic05:38
openstackgerritNaohiro Tamura proposed openstack/ironic: Add hardware inspection module for iRMC driver  https://review.openstack.org/19648005:42
*** alexpilotti has quit IRC05:43
*** aswadr has joined #openstack-ironic05:52
*** jobewan has quit IRC05:54
*** alexpilotti has joined #openstack-ironic05:58
openstackgerritNaohiro Tamura proposed openstack/ironic: Refactor IRMCVirtualMediaIscsiDeploy by applying new BootInterface  https://review.openstack.org/22137106:02
openstackgerritNaohiro Tamura proposed openstack/ironic: Refactor IRMCVirtualMediaAgentDeploy by applying new BootInterface  https://review.openstack.org/22157706:03
*** alexpilotti has quit IRC06:03
*** Nisha has quit IRC06:07
*** ukalifon has joined #openstack-ironic06:08
*** stendulker has joined #openstack-ironic06:14
*** alexpilotti has joined #openstack-ironic06:18
*** e0ne has joined #openstack-ironic06:22
*** alexpilotti has quit IRC06:23
*** Marga_ has quit IRC06:29
*** Marga_ has joined #openstack-ironic06:30
*** e0ne has quit IRC06:30
*** lazy_prince has quit IRC06:32
*** lazy_prince has joined #openstack-ironic06:32
*** Sukhdev has quit IRC06:33
*** alexpilotti has joined #openstack-ironic06:39
*** jamielennox is now known as jamielennox|away06:41
*** x3k has quit IRC06:42
*** killer_prince has joined #openstack-ironic06:43
*** alexpilotti has quit IRC06:43
*** ifarkas has joined #openstack-ironic06:46
*** lazy_prince has quit IRC06:46
mrdarameshg87: I'm not going to finish the spec review today, but I'll pick it up tomorrow (I'm going through the code reviews you refer to as part of the review)06:55
rameshg87mrda: sure, thanks06:56
*** ekarlso has quit IRC06:58
*** alexpilotti has joined #openstack-ironic06:59
*** alexpilotti has quit IRC07:04
*** jcoufal has joined #openstack-ironic07:17
*** alexpilotti has joined #openstack-ironic07:18
*** achanda has quit IRC07:18
*** jcoufal has quit IRC07:19
*** jcoufal has joined #openstack-ironic07:19
*** achanda has joined #openstack-ironic07:22
*** alexpilotti has quit IRC07:24
*** ekarlso has joined #openstack-ironic07:25
*** rameshg87 has quit IRC07:25
*** rameshg87 has joined #openstack-ironic07:26
*** praneshp has quit IRC07:34
*** praneshp has joined #openstack-ironic07:35
*** praneshp has quit IRC07:46
*** praneshp has joined #openstack-ironic07:47
*** priteau has joined #openstack-ironic07:48
*** rameshg87 has quit IRC07:51
*** rameshg871 has joined #openstack-ironic07:52
*** xek_ is now known as xek07:53
*** MattMan has quit IRC07:56
*** MattMan has joined #openstack-ironic07:56
*** achanda has quit IRC07:57
*** ionutbalutoiu has joined #openstack-ironic07:59
*** mkovacik has joined #openstack-ironic08:02
*** achanda has joined #openstack-ironic08:05
*** romainh has joined #openstack-ironic08:11
*** alexpilotti has joined #openstack-ironic08:13
*** praneshp has quit IRC08:15
*** achanda has quit IRC08:16
*** lucas-dinner has joined #openstack-ironic08:17
*** lucasagomes has joined #openstack-ironic08:18
*** smoriya has quit IRC08:24
lucasagomesmordred, alive?08:25
*** jistr has joined #openstack-ironic08:25
*** dtantsur|afk is now known as dtantsur08:34
dtantsurMorning Ironic08:34
*** achanda has joined #openstack-ironic08:39
*** alexpilotti has quit IRC08:47
*** achanda has quit IRC08:50
lucasagomesdtantsur, morning08:50
dtantsurrameshg871, hi there! do you know who is working on ironic-lib transition? do we have someone assigned (except for me :)?08:58
rameshg871dtantsur: rloo and jlvillal queried same thing yesterday08:59
dtantsurand what was the answer? :)08:59
rameshg871dtantsur: let me just check with faizan imeediately08:59
rameshg871hold on08:59
openstackgerritNaohiro Tamura proposed openstack/ironic: Use standard locale when executing 'parted' command  https://review.openstack.org/23136708:59
rameshg871dtantsur: he hasn't replied to their email yet09:00
rameshg871dtantsur: he is in a meeting, can I get back to you in half an hour09:02
dtantsuryes please09:02
rameshg871dtantsur: I will give the absolute status :)09:02
dtantsurI'll bump the g-r for ironic-lib in the meanwhile09:02
rameshg871okay09:02
*** alexpilotti has joined #openstack-ironic09:02
*** pelix has joined #openstack-ironic09:03
openstackgerritNaohiro Tamura proposed openstack/ironic-lib: Use standard locale when executing 'parted' command  https://review.openstack.org/23136909:04
vdrokmorning ironic09:04
vdrokmorning mrda, rameshg871, dtantsur, lucasagomes and lucas-dinner :)09:05
rameshg871vdrok: o/09:05
dtantsurmorning vdrok09:06
dtantsuroh I like that "morning lucasagomes and lucas-dinner" :D09:06
dtantsurvdrok, btw, where are you based?09:07
vdrokdtantsur, kharkiv, ukraine09:07
*** mbound has joined #openstack-ironic09:07
dtantsurvdrok, I see :) did we meet in Paris?09:07
* dtantsur has bad memory on names and faces09:07
*** alexpilotti has quit IRC09:07
vdrokdtantsur, nope, I didn't visit any of summits :)09:07
vdrokdtantsur, actually all of the team is here - yuriyz, aarefiev, pas-ha09:08
dtantsuryeah, I've met some folks from kharkiv back then in Paris, that's why I ask09:09
yuriyzmorning Ironic vdrok09:09
dtantsurmorning yuriyz :)09:09
vdrokmorning yuriyz09:09
*** lucas-dinner has quit IRC09:10
lucasagomesvdrok, morning09:11
*** romcheg has joined #openstack-ironic09:13
openstackgerritBob Ball proposed openstack/ironic: XenAPI: Add support for XenServer VMs  https://review.openstack.org/23138109:16
openstackgerritAnton Arefiev proposed openstack/ironic: Use oslo_config choices support  https://review.openstack.org/22620109:17
*** alexpilotti has joined #openstack-ironic09:19
rameshg871dtantsur: I have dropped a message to faizan as well, he will get back to you. I am stepping out for a while :)09:27
rameshg871sorry for the delay in ironic-lib stuff :(09:27
*** rameshg871 is now known as rameshg87-afk09:27
*** Nisha has joined #openstack-ironic09:33
sambettsMorning all o/09:35
*** Nisha has quit IRC09:36
*** Nisha has joined #openstack-ironic09:36
dtantsurmorning sambetts09:38
*** alexpilotti has quit IRC09:43
*** Nisha has quit IRC09:44
*** rameshg87-afk has quit IRC09:44
*** alexpilotti has joined #openstack-ironic09:47
mordredlucasagomes: alive and at the airport even :)09:48
lucasagomesmordred, hah good man!09:48
lucasagomesmordred, have a safe flight back home09:49
mordredthanks!09:49
*** alexpilotti has quit IRC09:51
sambettslucasagomes: Did you have a good meetup after linuxcon?09:52
lucasagomessambetts, morning! Yeah it was good09:52
sambetts:D awesome, I wish I could have been there09:52
lucasagomesfood and beer (-:09:53
lucasagomessambetts, yeah man, let's go to fosdem on january!09:53
lucasagomess/on/in09:53
sambettsI will definatly if it doesn't conflict with anything :D09:53
lucasagomes++09:53
*** naohirot has quit IRC09:57
*** e0ne has joined #openstack-ironic10:03
*** alexpilotti has joined #openstack-ironic10:07
*** alexpilotti has quit IRC10:11
*** jamielennox|away is now known as jamielennox10:14
*** romcheg1 has joined #openstack-ironic10:17
*** romcheg has quit IRC10:17
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: Introduce tempest-lib to functional tests  https://review.openstack.org/21670110:25
*** alexpilotti has joined #openstack-ironic10:27
*** alexpilotti has quit IRC10:31
*** romainh has left #openstack-ironic10:38
*** alexpilotti has joined #openstack-ironic10:47
*** stendulker has quit IRC10:47
*** romcheg1 has quit IRC10:49
*** romcheg has joined #openstack-ironic10:49
openstackgerritBob Ball proposed openstack/ironic: XenAPI: Add support for XenServer VMs  https://review.openstack.org/23138110:51
*** alexpilotti has quit IRC10:52
*** romcheg has quit IRC10:52
*** sdake has joined #openstack-ironic10:52
openstackgerritAnton Arefiev proposed openstack/ironic: Use oslo_config choices support  https://review.openstack.org/22620110:52
*** romcheg has joined #openstack-ironic10:53
*** trown|outttypeww is now known as trown10:56
*** baoli has joined #openstack-ironic10:57
openstackgerritAnton Arefiev proposed openstack/ironic: Add .eggs to gitignore  https://review.openstack.org/23141410:59
dtantsurlucasagomes, didn't jroll say that we already released ironic-lib 0.3.0?11:01
* dtantsur probably confused something...11:01
* lucasagomes looks11:01
lucasagomesI think we did just to have it in the g-r right?11:01
lucasagomeswe did something there11:01
*** saripurigopi1 has quit IRC11:02
lucasagomesdtantsur, https://github.com/openstack/requirements/commit/10d646aab603711c02aa9098b31209440f56338411:02
dtantsuryeah, but we need one more release then11:03
*** saripurigopi has joined #openstack-ironic11:03
dtantsurprobably I'm too sleepy today :) but we can't move to ironic-lib until we release what's in master11:03
dtantsurjroll, ^^11:03
*** alexpilotti has joined #openstack-ironic11:06
*** athomas has quit IRC11:06
*** baoli_ has joined #openstack-ironic11:07
*** athomas has joined #openstack-ironic11:09
lucasagomesdtantsur, right, we don't use ironic-lib yet11:10
*** baoli has quit IRC11:10
lucasagomesso... we have to keep thins in master until the release11:10
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: Introduce tempest-lib to functional tests  https://review.openstack.org/21670111:15
dtantsurlucasagomes, I think we're talking about different things :) I'm saying we can't switch to ironic-lib until one more ironic-lib release is done11:17
lucasagomesoh11:17
lucasagomesright, yeah I think that's true too11:17
lucasagomesAFAIUI, the first release was just to be able to add it to the g-r11:18
lucasagomesprior to the freeze11:18
openstackgerritMerged openstack/ironic-webclient: Automatic library concatenation  https://review.openstack.org/22863011:18
dtantsuryeah.. we had ironic-lib 0.2 a few days ago, but now we landed a breaking (IMO) change in ironic-lib master, so we have to release it11:18
lucasagomes:-(11:19
*** amotoki has quit IRC11:24
openstackgerritMerged openstack/ironic: Add notes to functions which are in ironic-lib  https://review.openstack.org/23109211:24
*** dtantsur is now known as dtantsur|bbl11:24
*** alexpilotti has quit IRC11:26
*** thrash|g0ne is now known as thrash11:33
*** gabriel-bezerra is now known as gabriel11:43
*** saripurigopi has quit IRC11:44
*** lucasagomes is now known as lucas-hungry11:44
*** mgoddard__ has joined #openstack-ironic11:50
*** mgoddard__ has quit IRC11:52
*** mgoddard__ has joined #openstack-ironic11:52
*** mgoddard_ has quit IRC11:53
*** mgoddard_ has joined #openstack-ironic11:54
*** mgoddard__ has quit IRC11:54
*** mgoddard__ has joined #openstack-ironic11:56
TheJuliaGood morning11:56
*** mgoddard_ has quit IRC11:56
*** mgoddard__ has quit IRC11:58
*** mgoddard__ has joined #openstack-ironic11:58
*** thrash has quit IRC11:59
*** mgoddard_ has joined #openstack-ironic12:00
*** mgoddard__ has quit IRC12:01
*** thrash has joined #openstack-ironic12:01
*** thrash has quit IRC12:01
*** thrash has joined #openstack-ironic12:01
vdrokmorning TheJulia12:01
sambettso/ TheJulia12:02
*** mgoddard_ has quit IRC12:02
*** mgoddard_ has joined #openstack-ironic12:02
*** mgoddard__ has joined #openstack-ironic12:04
*** mgoddard_ has quit IRC12:04
*** romainh has joined #openstack-ironic12:05
*** mgoddard_ has joined #openstack-ironic12:06
*** mgoddard__ has quit IRC12:06
*** mgoddard_ has quit IRC12:08
*** mgoddard__ has joined #openstack-ironic12:08
openstackgerritJulia Kreger proposed openstack/bifrost: Install ironicclient after installing the shade library  https://review.openstack.org/22819512:09
*** sdake has quit IRC12:09
*** sdake has joined #openstack-ironic12:10
*** mgoddard_ has joined #openstack-ironic12:10
*** mgoddard__ has quit IRC12:10
*** mgoddard_ has quit IRC12:12
*** mgoddard_ has joined #openstack-ironic12:12
openstackgerritJulia Kreger proposed openstack/bifrost: Add static DHCP for dynamic inventory  https://review.openstack.org/21737412:13
*** rvasilets___ has joined #openstack-ironic12:13
*** mgoddard__ has joined #openstack-ironic12:14
*** mgoddard_ has quit IRC12:15
*** mgoddard__ has quit IRC12:16
*** mgoddard__ has joined #openstack-ironic12:16
*** athomas has quit IRC12:17
*** Marga_ has quit IRC12:18
*** mgoddard_ has joined #openstack-ironic12:18
*** mgoddard__ has quit IRC12:18
*** Marga_ has joined #openstack-ironic12:19
*** mgoddard_ has quit IRC12:20
*** mgoddard_ has joined #openstack-ironic12:20
*** alexpilotti has joined #openstack-ironic12:21
*** athomas has joined #openstack-ironic12:22
*** mgoddard__ has joined #openstack-ironic12:22
*** mgoddard_ has quit IRC12:22
*** mgoddard__ has quit IRC12:24
*** mgoddard__ has joined #openstack-ironic12:24
*** alexpilotti has quit IRC12:25
*** mgoddard__ has quit IRC12:26
*** mgoddard__ has joined #openstack-ironic12:26
*** mgoddard__ has quit IRC12:28
*** mgoddard__ has joined #openstack-ironic12:28
*** jcoufal has quit IRC12:28
*** jcoufal has joined #openstack-ironic12:29
*** mgoddard_ has joined #openstack-ironic12:30
*** VikasC has quit IRC12:30
*** mgoddard__ has quit IRC12:30
*** mgoddard__ has joined #openstack-ironic12:32
*** mgoddard_ has quit IRC12:32
*** mgoddard__ has quit IRC12:34
*** mgoddard__ has joined #openstack-ironic12:34
*** amotoki has joined #openstack-ironic12:35
*** mgoddard__ has quit IRC12:36
*** mgoddard__ has joined #openstack-ironic12:36
*** ekarlso has quit IRC12:38
*** mgoddard__ has quit IRC12:38
*** mgoddard_ has joined #openstack-ironic12:38
*** mgoddard__ has joined #openstack-ironic12:40
*** mgoddard_ has quit IRC12:40
*** ionutbalutoiu has quit IRC12:42
*** mgoddard_ has joined #openstack-ironic12:42
*** mgoddard__ has quit IRC12:42
*** mgoddard_ has quit IRC12:44
*** mgoddard_ has joined #openstack-ironic12:44
*** lucas-hungry is now known as lucasagomes12:44
lucasagomesTheJulia, vdrok morning!12:45
*** mgoddard_ has quit IRC12:46
*** mgoddard__ has joined #openstack-ironic12:46
*** mgoddard_ has joined #openstack-ironic12:48
*** mgoddard__ has quit IRC12:48
*** sambetts is now known as sambetts-hungry12:50
*** mgoddard__ has joined #openstack-ironic12:50
*** mgoddard_ has quit IRC12:50
*** Marga_ has quit IRC12:50
*** Marga_ has joined #openstack-ironic12:51
*** mgoddard_ has joined #openstack-ironic12:52
*** mgoddard__ has quit IRC12:52
* TheJulia wonders if someone forgot to give the internet coffee this morning12:53
*** mgoddard__ has joined #openstack-ironic12:54
*** mgoddard_ has quit IRC12:54
*** notq has quit IRC12:55
*** notq has joined #openstack-ironic12:55
*** ekarlso has joined #openstack-ironic12:56
*** mgoddard_ has joined #openstack-ironic12:56
*** mgoddard__ has quit IRC12:56
*** mgoddard__ has joined #openstack-ironic12:58
*** mgoddard_ has quit IRC12:58
*** mgoddard_ has joined #openstack-ironic13:00
*** mgoddard__ has quit IRC13:00
*** mgoddard_ has quit IRC13:02
*** mgoddard__ has joined #openstack-ironic13:02
*** ukalifon has quit IRC13:03
*** mgoddard_ has joined #openstack-ironic13:04
*** mgoddard_ has quit IRC13:06
*** rloo has joined #openstack-ironic13:06
*** amotoki has quit IRC13:06
*** mgoddard_ has joined #openstack-ironic13:06
*** mgoddard__ has quit IRC13:07
*** mgoddard__ has joined #openstack-ironic13:08
*** mgoddard_ has quit IRC13:08
*** killer_prince has quit IRC13:09
*** sdake has quit IRC13:09
*** sdake has joined #openstack-ironic13:10
*** mgoddard_ has joined #openstack-ironic13:10
*** mgoddard__ has quit IRC13:10
*** mgoddard_ has quit IRC13:12
*** mgoddard_ has joined #openstack-ironic13:12
*** sdake_ has joined #openstack-ironic13:13
*** mgoddard_ has quit IRC13:14
*** mgoddard_ has joined #openstack-ironic13:14
BadCubMOrning Ironic13:14
*** sdake has quit IRC13:14
lucasagomesBadCub, morning13:15
BadCubmorning lucasagomes13:16
*** mgoddard__ has joined #openstack-ironic13:16
vdrokmorning BadCub13:16
BadCubmorning vdrok13:16
*** mgoddard_ has quit IRC13:16
*** mgoddard_ has joined #openstack-ironic13:18
*** mgoddard__ has quit IRC13:18
*** mgoddard__ has joined #openstack-ironic13:20
*** mgoddard_ has quit IRC13:20
*** mgoddard_ has joined #openstack-ironic13:22
*** mgoddard__ has quit IRC13:22
*** Marga_ has quit IRC13:23
*** Marga_ has joined #openstack-ironic13:23
*** mgoddard__ has joined #openstack-ironic13:24
*** mgoddard_ has quit IRC13:24
openstackgerritBob Ball proposed openstack/ironic: XenAPI: Add support for XenServer VMs  https://review.openstack.org/23138113:25
*** romainh has left #openstack-ironic13:25
*** mgoddard__ has quit IRC13:26
*** mgoddard_ has joined #openstack-ironic13:26
*** thiagop has joined #openstack-ironic13:27
*** saripurigopi has joined #openstack-ironic13:27
*** mgoddard_ has quit IRC13:28
*** mgoddard_ has joined #openstack-ironic13:28
*** amotoki has joined #openstack-ironic13:29
thiagopGood morning Ironicers13:30
*** mgoddard_ has quit IRC13:30
*** sambetts-hungry is now known as sambetts13:30
*** mgoddard_ has joined #openstack-ironic13:30
sambettso/ BadCub13:31
vdrokmorning sambetts, thiagop13:31
thiagoplucasagomes: Thanks for the review. I'm fixing things right now13:31
*** mgoddard__ has joined #openstack-ironic13:32
lucasagomesthiagop, morning! np!13:32
*** mgoddard_ has quit IRC13:32
*** amotoki has quit IRC13:33
*** mgoddard__ has quit IRC13:34
*** mgoddard__ has joined #openstack-ironic13:34
openstackgerritBob Ball proposed openstack/ironic: XenAPI: Add support for XenServer VMs  https://review.openstack.org/23138113:35
thiagopvdrok: o/13:35
*** mgoddard_ has joined #openstack-ironic13:36
*** mgoddard__ has quit IRC13:36
vdroklucasagomes, dtantsur|bbl, question, is it possible to build ipa ramdisk on top of busybox?13:37
vdrokand how much ram can it save you think?13:38
*** mgoddard_ has quit IRC13:38
*** mgoddard__ has joined #openstack-ironic13:38
*** amotoki has joined #openstack-ironic13:39
lucasagomesvdrok, hi there! I believe possible it is, but the problem would be to get a real init system with it as well13:40
* lucasagomes tho someone may already have hacked busybox + systemd together 13:40
*** mgoddard_ has joined #openstack-ironic13:40
openstackgerritBob Ball proposed openstack/ironic: XenAPI: Add support for XenServer VMs  https://review.openstack.org/23138113:40
*** mgoddard__ has quit IRC13:40
*** mgoddard_ has quit IRC13:42
*** mgoddard__ has joined #openstack-ironic13:42
lucasagomesvdrok, anyway it's something to think about. Tho, to be very honest if we can make the ramdisk boot with 1GB or slightly more it should be fine13:42
lucasagomesperhaps we wouldn't write IPA in python (since we have to carry the interpreter with it) if the ramdisk size was really limited13:43
vdroklucasagomes, yup, but it's what people suggest in qa channel to make full tempest run in some reasonable time13:43
lucasagomesoh13:44
*** mgoddard__ has quit IRC13:44
vdrokso that we could create at least 3 nodes13:44
*** mgoddard__ has joined #openstack-ironic13:44
vdrokand run with some concurrency13:44
lucasagomesvdrok, right yeah that's a problem13:44
*** lazy_prince has joined #openstack-ironic13:45
lucasagomesI'm sure we could just have a DIB element that creates the ramdisk with busybox and use a custom init script to start it (as PID 1)13:45
lucasagomesthat may save some time13:45
lucasagomesbut I wouldn't recommend that to run in production, that is hell to debug if something fails13:45
*** mgoddard_ has joined #openstack-ironic13:46
*** mgoddard__ has quit IRC13:46
lucasagomessave some space*13:47
sambettsI was interested in building a tinycore linux based IPA ramdisk :/ I didn't get that far with it though13:47
*** mgoddard__ has joined #openstack-ironic13:48
*** enikanorov has joined #openstack-ironic13:48
*** mgoddard_ has quit IRC13:48
*** amotoki has quit IRC13:49
jlvillalGood morning Ironic13:49
vdrokmorning jlvillal13:50
*** mgoddard_ has joined #openstack-ironic13:50
jlvillalvdrok, hi :)13:50
*** mgoddard__ has quit IRC13:50
*** mgoddard__ has joined #openstack-ironic13:52
*** mgoddard_ has quit IRC13:52
rloohi jlvillal, vdrok, sambetts, lucasagomes, BadCub, thiagop13:52
lucasagomesjlvillal, rloo morning13:53
sambettso/ rloo13:53
thiagopmorning jlvillal rloo13:53
rloovdrok: wrt https://review.openstack.org/#/c/231156/, I don't really want to show --serial or --concurrency. should i just delete the --parallel example?13:53
jlvillalvdrok, Do you know Pavlo S? If so, do you know what time zone he is in and an IRC nick?13:53
*** mgoddard_ has joined #openstack-ironic13:54
rloovdrok: I think the idea is the doc is a quickstart, not a detailed description of how tox works. I'm guessing default wasn't parallel before so someone put that instruction there (although i thought it was default in testr but)13:54
*** mgoddard__ has quit IRC13:54
*** mgoddard__ has joined #openstack-ironic13:56
*** mgoddard_ has quit IRC13:56
vdrokmorning rloo, yes, I think it can be just deleted, passing -r with regexp is already an example of positional args13:56
rloovdrok: good. I'll just reword that par then. thx.13:57
* lucasagomes likes parallel13:57
*** mgoddard_ has joined #openstack-ironic13:58
*** mgoddard__ has quit IRC13:58
vdrokjlvillal, he in eest, i think it's what our timezone is :)13:58
vdrok*he is13:58
vdroknick is pas-ha or pshchelo13:59
*** Sukhdev has joined #openstack-ironic14:00
*** mgoddard__ has joined #openstack-ironic14:00
jlvillalvdrok, Thanks :)14:00
*** amotoki has joined #openstack-ironic14:00
*** mgoddard_ has quit IRC14:00
*** pas-ha has joined #openstack-ironic14:01
*** derekh has joined #openstack-ironic14:01
jlvillallifeless, ^^^ Info on Pavlo S14:01
pas-hai'me here14:01
openstackgerritRuby Loo proposed openstack/ironic: To run a specific unit test with ostestr use -r  https://review.openstack.org/23115614:02
jlvillalpas-ha, Hi. But lifeless is not, I think.14:02
*** mgoddard__ has quit IRC14:02
*** mgoddard__ has joined #openstack-ironic14:02
jlvillalHe had some questions on the remove trivial assertions patch of yours.14:02
pas-hayes, I kind of suspected that14:02
jlvillal:)14:03
jlvillalpas-ha, He is on New Zealand/Australia time I think.14:03
pas-hathat might be non trivial than, I'm in UTC+3 now14:03
pas-hawe'll meet in Tokyo in three weeks I guess :)14:04
devanandag'morning, all14:04
*** mgoddard_ has joined #openstack-ironic14:04
*** mgoddard__ has quit IRC14:04
jlvillalpas-ha, Maybe he overlaps with you near your morning time?14:04
jlvillaldevananda, Good morning14:04
pas-habasing on asalkeld's timing - probably14:05
rloomorning devananda14:05
vdrokmorning devananda14:05
jlvillallucasagomes, If you ever get bored :)  Maybe you might want to look at: https://review.openstack.org/#/c/215362/14:06
lucasagomesjlvillal, will do !14:06
*** mgoddard__ has joined #openstack-ironic14:06
*** lazy_prince has quit IRC14:06
jlvillalThanks. And anyone else can feel free to look at it too :)14:06
*** mgoddard_ has quit IRC14:06
*** romcheg has quit IRC14:08
*** mgoddard_ has joined #openstack-ironic14:08
*** mgoddard__ has quit IRC14:08
*** mgoddard__ has joined #openstack-ironic14:10
*** mgoddard_ has quit IRC14:10
*** lazy_prince has joined #openstack-ironic14:10
openstackgerritKurt Taylor proposed openstack/ironic-inspector: Add coverage target to tox.ini  https://review.openstack.org/23149514:11
*** mgoddard_ has joined #openstack-ironic14:12
*** mgoddard__ has quit IRC14:12
*** mgoddard__ has joined #openstack-ironic14:14
krtaylor^^^, needs more debugging, --coverage is not being recognized for some reason, have to move on and didn't want to lose my place14:14
*** davideagnello has quit IRC14:14
*** mgoddard_ has quit IRC14:15
*** davideagnello has joined #openstack-ironic14:15
*** mgoddard__ has quit IRC14:16
*** mgoddard__ has joined #openstack-ironic14:16
*** romcheg has joined #openstack-ironic14:17
jlvillalvdrok, rloo: Actually that --serial option might be useful for my testing an issue I'm working on. But I think my case is unusual.14:17
jlvillalGlad I learned about it :)14:17
rloojlvillal: I'm not saying that --serial isn't useful, but that dev-quickstart is meant to be a 'quickstart'.14:18
*** mgoddard__ has quit IRC14:18
*** mgoddard__ has joined #openstack-ironic14:18
jlvillalrloo, Agreed!  I was just saying I'm glad I learned about it.14:18
rloojlvillal: :)14:19
*** mgoddard__ has quit IRC14:20
*** alexpilotti has joined #openstack-ironic14:20
*** mgoddard__ has joined #openstack-ironic14:20
*** mgoddard_ has joined #openstack-ironic14:22
*** mgoddard_ has quit IRC14:24
*** alexpilotti has quit IRC14:24
*** alexpilotti has joined #openstack-ironic14:25
*** mgoddard__ has quit IRC14:25
NobodyCamgood morning Ironicers14:25
jlvillalNobodyCam, morning14:26
NobodyCammorning jlvillal :)14:26
NobodyCammorning rloo dtantsur|bbl devananda  jroll jvmario sambetts14:26
NobodyCamand anyone not listed here14:27
lucasagomesNobodyCam, morning14:27
NobodyCamlike lucasagomes :)14:27
NobodyCammorning lucasagomes :)14:27
rloomorning NobodyCam14:29
*** achanda has joined #openstack-ironic14:29
*** alexpilotti has quit IRC14:30
NobodyCam:)14:30
*** alexpilotti has joined #openstack-ironic14:31
devanandajroll: on the topic of how to track small feature work, https://review.openstack.org/#/c/231381/5 is a good example. It would be worth mentioning in release notes, but it's trivially small at the same time.14:34
*** jcoufal has quit IRC14:35
*** achanda has quit IRC14:37
*** alexpilotti has quit IRC14:38
*** alexpilotti has joined #openstack-ironic14:38
jrollmorning everyone!14:38
jlvillaljroll, morning14:39
jrolllucasagomes: dtantsur|bbl https://review.openstack.org/#/c/231185/14:39
jrollironic-lib14:39
lucasagomesa-ha!14:39
lucasagomesjroll, devananda morning14:39
jrolldevananda: right, so I would have him file a blueprint with a sentence or two, and "approval" would be trivial14:39
devanandajroll: with no spec. gotcha14:40
jrolldevananda: which I'm tempted to do now, however 1) I'd like team buyoff, and 2) I have yet to clean up launchpad :P14:40
jrollyeah14:40
devanandayea, heh14:40
devanandaI think that approach can work -- IFF someone is diliently keeping LP organized14:41
rloojroll, devananda: devananda beat me to it, i had the same question :) There is no LP or bug associated with that patch which doesn't seem 'right' to me.14:42
devanandarloo: I agree - there needs to be something associated with it.14:43
* devananda comments on the patch14:43
*** dtantsur|bbl is now known as dtantsur14:43
dtantsurmorning TheJulia, BadCub, thiagop, jlvillal, rloo, devananda, NobodyCam, jroll14:43
dtantsurso many of us :)14:43
NobodyCam:)14:43
*** killer_prince has joined #openstack-ironic14:44
rloodtantsur: it's a party! :D14:44
jrolldevananda: ask him to file a BP then, I will take care of launchpad before tokyo at the latest14:44
devanandajroll: ack14:44
jrollthanks14:45
devanandaalso, in looking at the docs, I don't see any good location to suggest he add docs about ssh driver support xenserver14:45
*** saripurigopi has quit IRC14:45
*** Sukhdev has quit IRC14:45
*** lazy_prince has quit IRC14:45
devanandalike, ew should have a /drivers/ssh.rst file, but do not14:45
jlvillaldtantsur, I'm still trying to figure out why I can't get the mal-formed log.warning() call to trigger an error during unit tests. Not sure if it is an Ironic thing.14:46
devanandajroll: wdyt about using DocImpact flag to at least track changes that we need to go back and update documentation?14:47
dtantsurjlvillal, isn't it because the fake log just eats up all the exceptions?14:47
jrolldevananda: I'd rather people update docs in the patch14:47
devananda*nod*14:47
jrolldevananda: I do like UpgradeImpact :)14:48
jlvillaldtantsur, I submitted a patch to fix that. And still don't get an error during unit testing14:48
*** daemontool_ has quit IRC14:49
jlvillaldtantsur, https://github.com/JohnVillalovos/fixtures14:49
jlvillaldtantsur, Maybe it is a concurrency issue? Going to try running unit tests with -- --serial14:49
dtantsurhmm, maybe14:50
jrolllucasagomes: you might be interested in the conversation in -qa about ramdisk size14:52
lucasagomesjroll, joining14:53
*** alexpilotti has quit IRC14:54
jrolllucasagomes: dang, was hoping you were there already, lemme link you logs14:54
*** alexpilotti has joined #openstack-ironic14:55
*** alexpilotti has quit IRC14:55
*** alexpilotti has joined #openstack-ironic14:55
lucasagomesjroll, reading the logs14:55
*** achanda has joined #openstack-ironic14:57
*** alexpilotti has quit IRC14:58
*** derekh has quit IRC15:01
*** lazy_prince has joined #openstack-ironic15:01
*** achanda has quit IRC15:01
sambettsAnyone ever seen PBR throw a "Versioning for this project requires ether an sdist tarball or access to an upstream repo" exception when trying to install it via the tarball?15:01
dtantsursambetts, tarball must be generated by 'sdist' command, all other ways won't work15:02
dtantsuri.e. downloading tarball from github is known to be broken15:02
sambettsdtantsur: This is the tarball from tarball.openstack.org, is that known to be broken?15:02
dtantsurhmmmmmm15:02
*** alexpilotti has joined #openstack-ironic15:03
dtantsursambetts, no, it should not. which project?15:03
sambettsironic-python-agent15:03
jrollO_o15:04
*** killer_prince has quit IRC15:04
*** penick has joined #openstack-ironic15:05
dtantsureeeek15:05
*** davideagnello has quit IRC15:05
sambettstrying to determine if its a environment issue or an upstream issue, I'm trying to install in an env without git installed for the first time15:06
jrolloh heh15:06
jrollpbr doesn't like not having git installed15:06
*** david-ly_ is now known as david-lyle15:06
jrollif the shell out to git fails, pbr explodes in a ball of fire, last I knew15:06
dtantsurjroll, setup.py sdist must work too15:06
dtantsurat least it always did...15:07
sambettsjroll: but I should be able to pip install from a tar without git installed :/15:07
jrollsure, except with lifeless telling me setup.py is broken and I should never ever use it15:07
jrollsambetts: you would think...15:07
jrollsambetts: I could be wrong15:07
sambettsjroll: I'm going to build a new env and try again, I need to confirm its not a problem with my env15:09
dtantsurjroll, well 'setup.py sdist' is used by our publish jobs IIRC15:09
jrollcool15:09
openstackgerritMatt Keenan proposed openstack/ironic-specs: Add Solaris IPMItool driver  https://review.openstack.org/23153615:10
*** davideagnello has joined #openstack-ironic15:10
*** davideagnello has quit IRC15:12
*** davideagnello has joined #openstack-ironic15:13
*** rameshg87 has joined #openstack-ironic15:15
rameshg87jlvillal: hi15:15
* jlvillal learns that McDonald's will now serve all-day breakfast. And he does admit he likes the breakfast there, even if it is bad for him :)15:15
jlvillalrameshg87, Hey15:16
rameshg87jlvillal: I haven't heard from faizan as well. may be you can start working on his patch in ironic-lib if you are interested to take it up - https://review.openstack.org/#/c/184443/15:16
rameshg87jlvillal: he might be busy15:16
dtantsur++ for someone doing it15:16
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: [WIP]Use requests lib in HTTPClient  https://review.openstack.org/22928615:16
*** pas-ha has quit IRC15:17
jlvillaldtantsur, Did you say you wanted to work on it?15:17
dtantsurjlvillal, if you were not planning to, I can take it :)15:18
jlvillaldtantsur, If you can do it that would be great. I'm trying to track down some testing things.15:18
dtantsurack15:19
rameshg87jroll: devananda: any thoughts on the mail thread on broken out-of-tree driver ?15:19
jlvillaldtantsur, Thanks. Glad to help too.15:19
rameshg87jroll: devananda: was wondering what guarantees do we really give to our out-of-tree drivers ? may be we need to define it15:19
jrollrameshg87: I'm about to step away, need to think about it more. I don't like breaking out of tree drivers, but like you say we don't have a good definition for "break"15:20
rameshg87jroll: okay15:20
devanandarameshg87: the change to update my AMTTool driver to liberty is pretty trivial, but yah, what exactly is our driver API ?15:20
* jroll bbiab15:20
devanandathat we broke a very simple power driver is what concerns me15:20
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: [WIP]Use requests lib in HTTPClient  https://review.openstack.org/22928615:20
devanandait's not that AMTTool depends on some behavior of PXEBoot -- it depends on the BaseDriver class15:21
devanandaand *that* changed15:21
rameshg87devananda: BaseDriver call didn't change infact15:21
devanandaso, in effect, we broke the definition of what a driver is15:21
rameshg87devananda: the in-tree implementations used in self.deploy and self.vendor of AMTTool driver changed15:21
dtantsurwas someone doing something with etherpad right now? rloo?15:21
rloodtantsur: not me. i did yesterday.15:22
rameshg87devananda: do we gurantee that in-tree implementations will have same behaviour ? it's not part of our API anyway15:22
dtantsurweird... some text just disappeared in front of me15:22
dtantsurand I wonder who is to blame15:22
* rameshg87 is not on etherpad anyway15:22
devanandarameshg87: right. AMTTool isn't a "clean" out of tree driver -- it's a mix of an out of tree power + mgmt interface, with an in-tree deploy + vendor interface15:23
*** sdake_ is now known as sdake15:23
sambettsdtantsur: Does it not have a colour trace?15:23
devanandais that something we want to support? I dunno.15:23
dtantsurrloo, was subteam whiteboard empty last time you saw it?15:23
devanandarameshg87: fwiw, I can see your point, and I think we need to define what, exactly, the supported mechanism is for out of tree drivers, then document it, and add a test case15:24
rloodtantsur: i really don't like etherpad. sec, let me see what it looks like now15:24
rameshg87devananda: honestly I don't think we should be doing it. it's true that hasn't changed a lot over the past (AgentDeploy has been doing the same thing), but we don't have it in our API15:24
rameshg87devananda: in the other case we broke PXEDeploy into PXEBoot + ISCSIDeploy15:24
sambettsdtantsur, jroll: in a fresh ubuntu-server install installing from the tar works :(15:24
rameshg87devananda: a class that people were using might have disappeared :(15:24
* sambetts wonders whats broken in his other env 15:24
rloodtantsur: yeah, the subteam stuff looks like what it was yesterday after i cleared out the reports15:24
* rameshg87 wonders why sambetts is sad for something that works15:25
dtantsursambetts, old pbr?15:25
devanandarameshg87: the fix for AMTTool is one line -- https://github.com/devananda/ironic/commit/f05b237bb4d2b7afde139cdeeb84462a0cf316ef15:25
rameshg87devananda: yeah I just saw that too :)15:25
dtantsurrloo, ack, thanks!15:25
rloodtantsur: i always clear out the reports after i send out email with the reports15:25
sambettsdtantsur: the env thats broken was fresh 30 mins ago ... :(15:25
devanandarameshg87: so as far as personal impact to me -- this is easier than the patch I wrote for upstream. but I'm concerned about anyone else who built a power+mgmt driver in Kilo15:25
devananda(as an aside, i really wish that the AMT driver worked so I didn't have to maintain this thing out of tree)15:26
devanandalintan__: ^ :)15:26
rameshg87devananda: but I agree, it is very much a valid use case that people might have just implemented their own power+mgmt15:26
*** dims_ has quit IRC15:27
devanandarameshg87: even if ^ isn't defined today as part of the driver API, I think this is the question: we now know that Liberty will break any downstream driver that mixes interfaces in this way. Do we want to break them or not?15:29
devanandaI think it's important that we don't break potential users any time we can avoid breaking them -- and especially without a deprecation period15:30
*** romcheg has quit IRC15:30
dtantsurdevananda, what about bringing it in-tree? ;) after all, we already have to IPMI implementations15:30
rameshg87devananda: ++15:30
* rameshg87 brb15:31
devanandadtantsur: hrm?15:31
*** rameshg87 is now known as rameshg87-brb15:31
dtantsurdevananda, your driver15:31
devanandaoh15:32
dtantsurdevananda, re breaking drivers: we definitely need to 1. define a stable API, 2. provide an automatic way to ensure it is not broken15:32
dtantsurdevananda, maybe we need to somehow use N-1 version of pxe_ipmitool to check that N version of Ironic does not break it?15:32
*** dims_ has joined #openstack-ironic15:32
dtantsuri.e. create such a gate job..15:32
sambettsdtantsur, jroll: Solved it... https://bugs.launchpad.net/python-swiftclient/+bug/1379579 caused by old setuptools :(15:34
openstackLaunchpad bug 1379579 in PBR "unhandled PBR Exception "Versioning for this project requires either an sdist tarball" with older (<0.7.2) setuptools" [High,Confirmed]15:34
devanandadtantsur: I would actually like the in-tree one to work, but I haven't put in the time to try to fix it15:34
rloodevananda: wrt your question. I think no, we don't want to break them 'if we can avoid breaking them in a reasonable way'.15:34
dtantsurdevananda, ok, so what about we checkout any driver, say, from Liberty tree, and use it to check that we didn't break driver interface in Mitaka? of course we have to make sure that chosen driver does not use any non-public API...15:35
krotscheckbrains15:35
rloodevananda: and yeah, when i thought of out-of-tree drivers, i just thought of those drivers using everything out of tree.15:35
devanandadtantsur: on the surface, that sounds reasonable. whether kilo-driver + liberty release, or liberty driver + mitaka release, etc15:35
devanandarloo: yea, so far, I had done that as well15:36
*** davidlenwell has quit IRC15:36
trowndtantsur: sambetts, is running the inspector dbsync more than once supposed to work?15:36
rloodevananda: i still think it is a reasonable assumption. we could say that we don't guarantee that mixin in/out tree driver interfaces are guaranteed to work.15:36
devanandarloo: I don't think any of us were reviewing the boot/deploy split with this use case in mind, but in retrospect, I think it may be common. just look at how many in-tree drivers reuse PXEDeploy || AgentDeploy15:36
dtantsurtrown, yes, if you run it correctly15:37
dtantsurtrown, (i.e. upgrade command is a noop the second time)15:37
*** davidlenwell has joined #openstack-ironic15:37
devanandakrotscheck: toast!15:37
*** krtaylor has quit IRC15:38
trowndtantsur: in my environment if I run it a second time, then inspector can no longer write to it15:38
dtantsurtrown, sqlite? something with access rights?15:38
krotscheckdevananda: yeaaahsomethinglikethat15:39
dtantsurtrown, you have to run dbsync and inspector itself from the same user15:39
trowndtantsur: hmm maybe that is it, odd that it would work running once as root, but not twice15:42
*** links has quit IRC15:42
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: Adds more functional tests for commands  https://review.openstack.org/22424715:43
*** saripurigopi has joined #openstack-ironic15:43
*** penick has quit IRC15:43
*** VikasC has joined #openstack-ironic15:44
* jlvillal has to admit the Microsoft Surface Book looks pretty cool15:45
devanandaohhey food has arrived! /bbiab15:45
thiagop(late) morning dtantsur15:46
*** dtantsur_ has joined #openstack-ironic15:48
*** mkovacik_ has joined #openstack-ironic15:48
*** jistr_ has joined #openstack-ironic15:48
*** lsmola_ has joined #openstack-ironic15:49
*** mkovacik has quit IRC15:49
*** dtantsur has quit IRC15:49
*** jistr has quit IRC15:50
krotscheckSo, to get driver properties, I have to go to /driver/properties?driver_name=foo rather than /driver/foo/properties?15:50
*** ionutbalutoiu has joined #openstack-ironic15:50
*** mkovacik__ has joined #openstack-ironic15:51
*** krtaylor has joined #openstack-ironic15:51
*** lsmola has quit IRC15:52
mkovacik__guys, I'm running Devstack in VirtualBox instance and would like to add the VirtualBox host as ironic node but have been failing to accomplish this: http://pastebin.test.redhat.com/317777 (although i've set enabled_drivers to contain this driver type and run the ironic-api process again) any hints appreciated.15:52
*** mkovacik_ has quit IRC15:53
*** dtantsur_ has quit IRC15:53
*** lsmola_ has quit IRC15:53
*** jistr_ has quit IRC15:53
sambettstrown, dtantsur_ : Hmm, I wonder if we need to mention something about the user in the README?15:54
*** romcheg has joined #openstack-ironic15:54
trownsambetts: ya we definitely need to update the README, I am still messing with it15:54
trownfwiw, this is not how ironic does it... the dbsync command is run as root15:55
*** lazy_prince has quit IRC15:55
sambettstrown: They use mysql with its own user permissions, sqlite based DB is just a file so if its created by root its likely to only be accessible by root15:57
* mkovacik__ is away: I'm busy15:57
*** dtantsur has joined #openstack-ironic15:58
trownsambetts: ah good point. odd thing is it works if created by root as long as you do not run dbsync twice15:58
*** mtanino has joined #openstack-ironic15:59
trownsambetts: but I just confirmed that if we run the dbsync as the same user as inspector it can be run multiple times15:59
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/23122816:00
*** garthb has joined #openstack-ironic16:00
sambettstrown: Yeah, thats really weird :/16:00
*** penick has joined #openstack-ironic16:03
*** ijw has joined #openstack-ironic16:04
* rameshg87-brb is done for the day16:05
rameshg87-brbgood night folks16:05
*** rameshg87-brb has quit IRC16:05
*** alexpilo_ has joined #openstack-ironic16:05
*** lsmola_ has joined #openstack-ironic16:06
*** Marga_ has quit IRC16:06
*** lazy_prince has joined #openstack-ironic16:06
*** jistr_ has joined #openstack-ironic16:06
openstackgerritMerged openstack/ironic-lib: Use standard locale when executing 'parted' command  https://review.openstack.org/23136916:07
*** ifarkas has quit IRC16:07
*** saripurigopi has quit IRC16:07
*** e0ne has quit IRC16:08
dhellmannjroll: ironic-python-agent 1.0.0 is released and the stable branch has been created. Since it's a server project, I'm not sure how you want to handle the release announcement, so let me know.16:08
*** x3k has joined #openstack-ironic16:09
*** alexpilotti has quit IRC16:09
*** nicodemos has joined #openstack-ironic16:14
*** ionutbalutoiu has quit IRC16:14
*** alexpilotti has joined #openstack-ironic16:14
openstackgerritLucas Alvares Gomes proposed openstack/ironic: PXE: Support Extra DHCP Options for IPv6  https://review.openstack.org/23157316:15
*** alexpilotti has quit IRC16:16
*** mbound has quit IRC16:16
lucasagomesfolks interested in IPv6 + Ironic, I've started a wiki page to put some thoughts about it https://wiki.openstack.org/wiki/Ironic/IPv616:17
TheJuliaInteresting...16:18
*** alexpilo_ has quit IRC16:18
*** amotoki has quit IRC16:20
*** rvasilets___ has quit IRC16:21
*** penick has quit IRC16:22
*** links has joined #openstack-ironic16:23
*** jistr_ has quit IRC16:23
*** tsekiyama has joined #openstack-ironic16:29
*** ijw has quit IRC16:32
*** alexpilotti has joined #openstack-ironic16:32
*** alexpilotti has quit IRC16:37
*** dims_ has quit IRC16:37
*** ijw has joined #openstack-ironic16:39
jlvillalkrotscheck: I could see it be useful to just be /driver/properties/foo16:39
*** dims_ has joined #openstack-ironic16:41
*** dims_ has quit IRC16:41
krotscheckjlvillal: Wouldn't /drivers/foo/properties make more sense? Type/ID/subtype?16:41
jlvillalkrotscheck: Seems more difficult to code that as 'foo' changes.16:42
*** dims_ has joined #openstack-ironic16:42
jlvillalkrotscheck: And also would break backward compatibility, I think.16:42
jlvillalIf foo=properties ??16:43
*** dims__ has joined #openstack-ironic16:44
jlvillaldtantsur: FYI: I un-duplicate'd the logging bug.16:44
krotscheckjlvillal: Sorry, did you disagree with my comment? What makes more sense != whether we should build it.16:44
dtantsurjlvillal, found something to fix in ironic?16:45
jlvillaldtantsur: No, but I don't think the bug is in fixtures.16:45
dtantsurjlvillal, I just can't see how this bug could be in ironic16:45
*** dims_ has quit IRC16:45
jlvillalkrotscheck: Oh, I'm unsure if it makes more sense or not :)16:45
jlvillaldtantsur: Well until we figure out who is responsible, I think Ironic gets the blame. Or at least the bug stays with Ironic.16:46
dtantsurjlvillal, I would say "the bug stays with fixtures"16:46
jlvillaldtantsur: I don't think it is fixtures. I have a patch that runs and works.16:46
dtantsurI'm not found of bugs left on ironic "just because nobody knows". nobody will know, unless someone triages it...16:46
*** x3k has quit IRC16:47
*** romcheg has quit IRC16:47
jlvillaldtantsur: I'm investigating it now. But yes I should add myself as owning it.16:47
jlvillaldone16:47
krotscheckjlvillal: Right. I can kibitz about what the API should be all day long :D16:48
dtantsurjlvillal, ok please do :) what I'm trying to avoid is to having this bug hanging around indefinite time without clear idea if it's even related to us. Thanks!16:48
jlvillalkrotscheck: My main experience with REST API is the Gerrit API.16:48
krotscheckjlvillal: My condolences16:48
jlvillalkrotscheck: So my experience is limited :)16:48
jlvillalkrotscheck: And Artifactory...16:48
jlvillaldtantsur: Assigned. And I will be investigating it today.16:49
jrolldhellmann: it's ~kind of~ a server project, it's weird :) but yes, I'm excited about the initial release and will send the announcement, thanks!16:49
dtantsurwoohooo!16:49
jlvillaljroll: If you haven't already. Okay if I updated g-r with ironic-lib 0.3.016:49
dtantsurjlvillal, https://review.openstack.org/#/c/231368/16:49
NobodyCamTheJulia: are you around?16:50
jlvillalThe Ironic team is fast! ;)16:50
krotscheckjlvillal: You sound like you do java!16:50
jlvillalkrotscheck: Nope. Just Python interacting with Java based stuff.16:50
* krotscheck likes jlvillal a little more now :)16:50
jlvillalkrotscheck: I do have one patch in Gerrit. And I don't know any Java. Luckily the held my hand and walked me through the process ;)16:51
jlvillals/the held/they held/16:51
jlvillalOne patch in Gerrit = patch to the actual Gerrit source code16:51
jrolldtantsur: trown: \o/ https://pypi.python.org/pypi/ironic-python-agent16:51
dtantsurawesome!16:51
dtantsurjroll, I wonder why "False" is displayed instead of README though :)16:52
jrolldtantsur: O_o good question16:52
jlvillaldtantsur: Darn, still breaking for g-r patch.  Looking at Zuul16:52
trownwoot woot16:53
TheJuliaNobodyCam: semi-around, whats up?16:53
*** Sukhdev has joined #openstack-ironic16:54
lucasagomesIPA released? Nice!16:54
openstackgerritMerged openstack/bifrost: Update deprecated options for Mitaka release  https://review.openstack.org/22826716:55
*** alexpilotti has joined #openstack-ironic16:56
*** romcheg has joined #openstack-ironic16:59
*** ukalifon1 has joined #openstack-ironic17:02
*** enikanorov has quit IRC17:03
*** sdake has quit IRC17:03
openstackgerritMerged openstack/bifrost: Ansible 2.0 compatibility  https://review.openstack.org/19683217:04
*** romcheg1 has joined #openstack-ironic17:04
*** VikasC has quit IRC17:04
*** sdake has joined #openstack-ironic17:05
*** romcheg has quit IRC17:05
*** alexpilotti has quit IRC17:06
*** alexpilotti has joined #openstack-ironic17:06
*** x3k has joined #openstack-ironic17:07
*** romcheg has joined #openstack-ironic17:08
*** romcheg1 has quit IRC17:09
*** trown is now known as trown|lunch17:10
*** Marga_ has joined #openstack-ironic17:12
*** alexpilotti has quit IRC17:12
*** ijw has quit IRC17:16
*** e0ne has joined #openstack-ironic17:16
*** mkovacik__ has quit IRC17:16
*** alexpilotti has joined #openstack-ironic17:20
*** albertoffb has joined #openstack-ironic17:20
*** david-ly_ has joined #openstack-ironic17:21
*** david-lyle has quit IRC17:21
*** david-ly_ is now known as david-lyle17:22
*** ukalifon1 has quit IRC17:23
*** lazy_prince has quit IRC17:23
*** achanda has joined #openstack-ironic17:24
*** harlowja has quit IRC17:25
dtantsursambetts, https://bugs.launchpad.net/ironic-inspector/+bug/1503377 :(17:26
openstackLaunchpad bug 1503377 in Ironic Inspector "Functional tests are broken after some requirements update" [Critical,Confirmed]17:26
sambettsdtantsur: nooo :'(17:26
lucasagomesfolks I'm going to call it a day17:26
*** romcheg has quit IRC17:26
lucasagomeshave a great night everyone!17:26
dtantsurg'night, lucasagomes17:26
rloonight lucasagomes17:26
sambettsnight lucasagomes17:26
*** ijw has joined #openstack-ironic17:29
*** dims_ has joined #openstack-ironic17:29
*** lucasagomes is now known as lucas-dinner17:32
sambettsalso heading off, going to the gym o/17:32
*** dims__ has quit IRC17:32
jlvillalsambetts: Stay healthy!17:33
*** ijw has quit IRC17:33
*** Nisha has joined #openstack-ironic17:34
*** harlowja has joined #openstack-ironic17:35
thiagopnight lucas-dinner17:36
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Use auth_strategy=noauth in functional tests  https://review.openstack.org/23162517:36
dtantsurno inspector cores to review gate fix :(17:36
jrolldtantsur: maybe if you included ironic-core on that you might get a review :P17:37
dtantsurjroll, if I do, will you review? ;)17:37
jrolldtantsur: I +1'd17:37
rloojroll: i dunno. getting an invite at the last minute...17:37
*** x3k has quit IRC17:38
jrollrloo: ?17:38
dtantsurjroll, ironic-core included17:38
rloojroll: invite to be core on inspector i mean17:38
openstackgerritMerged openstack/bifrost: Stop using distro setuptools and easy_install  https://review.openstack.org/23082017:39
jrolldtantsur: hah, I was joking x.x17:39
jrollbut that's fine17:39
jrollrloo: don't feel obligated to review inspector stuff17:39
jrolldtantsur: +217:39
dtantsurjroll, yeah, I guess folks won't approve stuff they don't understand; and it's fair as we're under ironic unbrella17:40
jrolldtantsur: yeah, that's my thought17:40
rloojroll: nope, no obligation at all. there's only so much I can handle/get up to speed on!17:40
jrollyep :)17:40
openstackgerritDoug Hellmann proposed openstack/ironic-python-agent: Open mitaka development  https://review.openstack.org/23163017:42
*** dprince has joined #openstack-ironic17:42
*** praneshp has joined #openstack-ironic17:42
rloodtantsur: having said that, do you really want 231625 +2'd?17:43
dprincedtantsur: hey, so yeah per here I thought all the IPA bugs were just tagged https://wiki.openstack.org/wiki/Ironic#ironic-python-agent17:43
dprincedtantsur: perhaps I should update that?17:43
dtantsurdprince, that's a recent change. please update :)17:43
dprincedtantsur: cool.17:43
dtantsurrloo, yep, if you don't mind reviewing it17:43
dtantsur(I don't force anyone to review inspector changes, but I'm very happy when you do :)17:44
dprincedtantsur: so yeah I've found another Logging issue17:44
dtantsurdprince, yeah, great catch17:44
*** degorenko is now known as degorenko_away17:44
dtantsurdhellmann, jroll, what about switching to postversioning for IPA?17:44
jrolldtantsur: yeah, I asked a similar question in the relmgr channel17:44
rloodtantsur: +2'd.17:45
dtantsurthnx17:45
rloodtantsur: I'll +A if jenkins is happy17:45
dtantsurawesome, thank a ton! that means I can probably leave for a day (it's nearly 8pm after all)17:46
*** penick has joined #openstack-ironic17:47
NobodyCamhave a good night dtantsur17:47
NobodyCam:)17:47
NobodyCambrb17:47
*** baoli_ has quit IRC17:47
openstackgerritDan Prince proposed openstack/ironic-python-agent: Fix log formatting error in iscsi.py  https://review.openstack.org/23163417:47
*** dtantsur is now known as dtantsur|afk17:48
thiagopnight dtantsur|afk17:48
openstackgerritDan Prince proposed openstack/ironic-python-agent: Update the launchpad link for IPA  https://review.openstack.org/23163717:50
jrolldprince: thanks for that17:51
dprincejroll: np17:51
* dprince is trying to convert to use IPA this week17:52
*** dims_ has quit IRC17:53
jrolldprince: awesome :D little thing on the log patch17:53
openstackgerritDan Prince proposed openstack/ironic-python-agent: Fix log formatting error in iscsi.py  https://review.openstack.org/23163417:54
thiagopyuriyz: Hey, hi!17:56
jrollwoo17:56
*** vishwanathj has quit IRC17:56
* jroll goes heads down on some specs work17:56
jrollping me if you need me17:56
thiagopyuriyz: I just reveived your commend on the OneView driver. I'm working on it right now, but would like your opinion: the validate_oneview_resources_compatibility doesn't raises any exception17:57
JayFdprince: I just landed that, thanks for the patch17:57
thiagopyuriyz: it's but a proxy to call the oneview_client validations. Do you think we should document an exception that is not thrown by the method itself?17:58
yuriyzhi thiagop, IMO this should be mentioned in the docstring17:59
NobodyCambrb18:00
NobodyCamgah18:00
NobodyCamtoo many keyboards on desk18:01
dprinceJayF: np, thanks18:01
thiagopyuriyz: right, so I'll mention that the validation will fail due to oneview_client exceptions without the :raises: tag, that will enlighten things to you?18:01
*** dims_ has joined #openstack-ironic18:01
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Introduced Root State  https://review.openstack.org/23164118:02
yuriyzthiagop ok18:02
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Moved node list to default application state  https://review.openstack.org/23164218:02
thiagopyuriyz: awesome, thanks for the review btw ;)18:02
openstackgerritMerged openstack/ironic-inspector: Use auth_strategy=noauth in functional tests  https://review.openstack.org/23162518:02
*** ijw has joined #openstack-ironic18:05
*** dims_ has quit IRC18:05
*** dims_ has joined #openstack-ironic18:06
*** alexpilotti has quit IRC18:10
openstackgerritThiago Paiva Brito proposed openstack/ironic: OneView Driver for Ironic  https://review.openstack.org/19182218:11
*** alexpilotti has joined #openstack-ironic18:11
*** links has quit IRC18:13
*** athomas has quit IRC18:13
*** trown|lunch is now known as trown18:17
*** Nisha has quit IRC18:23
lifelessjroll: sambetts: huh? you can pip install from a tar without git installed18:24
*** harshs has joined #openstack-ironic18:24
jrolllifeless: I was talking about 18 months ago18:25
lifelesssambetts: what you can't do is pip install from a tar made by git export18:25
jrolland may or may not be correct18:25
jrollmemory doesn't go back that far18:25
lifelessjroll: pbr has been like this since nearly forever :)18:25
lifelessjroll: setup.py sdist bundles version metadata into the tarball18:25
jrolllifeless: yeah, I'm probably thinking of pip install .18:25
*** Nisha has joined #openstack-ironic18:25
jrollit was just an idea18:25
jrollignore me18:25
lifelessjroll: heh, never :). But I don't have full context on the discussion18:26
jrolllifeless: context is random flailing around a pip install thing.tar not working :)18:26
lifelessah :)18:26
*** sambetts has quit IRC18:26
lifelessjroll: so yeah, two possibilities18:26
lifelessjroll: there was a buggy point release of pbr that needed git always. LONG fixed.18:27
jrolllifeless: he got it working in a new environment btw18:27
lifelessjroll: and some folk try to pip install git exports of the source tree (vs sdists made by setup.py)18:27
lifelessjroll: anything else and I'd be looking for a bug report since its not expected ;)18:27
lifelessok cool18:28
jrolllifeless: my first patch ever to ironic was to add apt-get install git to dev docs, because tox failed hard without it :P18:28
jrollwas a fun one to find18:28
*** olaph has quit IRC18:29
*** olaph has joined #openstack-ironic18:30
*** sambetts has joined #openstack-ironic18:32
openstackgerritLilia Sampaio proposed openstack/python-oneviewclient: Creating OneViewException class  https://review.openstack.org/23165518:34
jlvillallifeless: Awake so early! :)18:37
jlvillallifeless: So I'm still trying to figure out the logging issue and not failing the unit tests.18:38
jlvillallifeless: Also, please let me know if you want any changes to the pull request I submitted for fixtures.18:38
jlvillaldprince: Thanks for finding those log issues. Though it has sent me off in search of why our unit tests aren't catching the errors.18:41
dprincejlvillal: yeah, I meant to follow up on that too. thanks18:44
openstackgerritRuby Loo proposed openstack/ironic-specs: Manual cleaning  https://review.openstack.org/23128818:53
*** jobewan has joined #openstack-ironic18:53
*** aswadr has quit IRC18:54
openstackgerritDan Prince proposed openstack/ironic-python-agent: Use mount -t sysfs to avoid host /sys dependencies  https://review.openstack.org/23165818:55
*** dprince has quit IRC18:55
*** Sukhdev has quit IRC18:56
*** albertoffb has quit IRC18:57
*** nicodemos has quit IRC19:02
*** penick has quit IRC19:03
*** nicodemos has joined #openstack-ironic19:04
*** pelix has quit IRC19:04
jlvillaljroll: So when the announcement said that IPA supports every Ironic driver. Does that include AMT? ;)19:10
jroll.......19:11
jlvillal:D19:11
jrolldoes AMT have a boot interface yet19:11
jlvillaljroll: I got this: https://bugs.launchpad.net/ironic/+bug/149948919:11
openstackLaunchpad bug 1499489 in Ironic "AMT is not supported with Ironic-Python-Agent" [Medium,Triaged] - Assigned to John L. Villalovos (happycamp)19:11
jlvillalSo it is on my list of things to do...19:12
jrolloh19:12
* jlvillal wonders if devananda wants to do it ;)19:12
*** nicodemos has quit IRC19:13
jrolljlvillal: though, now that I think about it, IPA works with every driver19:13
jrolllike pxe_amt19:13
jrollthere's just no AgentDeploy driver for AMT19:13
jrollso yes, it includes AMT19:13
jrollbut it's iscsi based19:13
*** Nisha has quit IRC19:13
jrollman, I need a NUC...19:15
openstackgerritJim Rollenhagen proposed openstack/ironic: Add agent_amt driver  https://review.openstack.org/23166519:16
jroll^ who wants to test19:16
jlvillaljroll: I recommend a NUC :)19:18
jrolljlvillal: grab me one from the supply closet pls? :)19:18
*** nicodemos has joined #openstack-ironic19:18
jlvillaljroll: I wish. We have to beg to get them here too...19:19
openstackgerritJim Rollenhagen proposed openstack/ironic: Add agent_amt driver  https://review.openstack.org/23166519:19
jrolljust updated commit message there19:19
jrollyeah, I know :(19:19
jlvillalmrda: ^^^^19:19
jrolldevananda: ^ wanna test that? :D19:20
jrolljlvillal: do all models have AMT support btw?19:21
jlvillaljroll: NO!!!!!!!19:21
jrolllolz19:21
jlvillaljroll: The ones with E at the end usually do.  E = Enterprise I think.19:21
jrollalright, thanks19:22
jlvillaljroll: http://ark.intel.com/products/series/70407/Intel-NUC-Boards-and-Kits  The ones with E at the end should be AMT19:22
jrollty19:23
*** e0ne has quit IRC19:23
jrolljlvillal: also does this have a serial console of some sort?19:24
mtreinishjroll, jlvillal: its any of them that have cpus with vpro19:24
jrollmtreinish: perfect, ty19:24
jlvillaljroll: Not really. You can use a USB to serial converter.19:24
jrollmehhh.19:24
jrollalright.19:24
jlvillaljroll: There is in AMT supposed to be Serial-Over-LAN. But I have not tried it.19:24
mtreinishamt has console support, but I think you need a monitor plugged in or something to enable it19:24
mtreinish(on nucs)19:25
mtreinishI don't have a nuc and it's been a long time since I've played with amt19:25
jrollhm, ok19:25
mtreinishbut you can ask sdague, he's got a small 4 node nuc cloud running19:26
mtreinishand uses amt to manage the nucs19:26
jrollyeah, I thought asking the intel guy would be enough :P19:26
TheJuliaamt9 has serial over lan if memory serves, at least that is what the desktop i just put on the network says19:29
* jroll ponders this over lunch19:29
*** e0ne has joined #openstack-ironic19:32
jlvillaljroll: heh19:33
lifelessjlvillal: that was 0730 local time :)19:36
lifelessjlvillal: I'll hopefully review the fixtures patch today19:36
jlvillallifeless: Oh, I thought you were off in Australia or something like that.19:36
lifelessjlvillal: I'm hoping to get my profiles patch for testrepository up in a PR today19:36
lifelessjlvillal: I'm in NZ19:37
jlvillalAh. Sorry! :)19:37
lifelessjlvillal: DST started here a couple weeks back19:37
lifelessjlvillal: and I believe finished for you? So we've moved 2hrs closer on this side of the day19:37
* TheJulia 3019:37
jlvillallifeless: We have like two more weeks or so19:37
TheJuliadoh19:37
jlvillallifeless: I updated the bug on unit tests not catching the log errors. Added you to it.19:37
jlvillallifeless: It is strange I can see two calls to 'emit' but they seem to come from alembic. But no calls from the ironic code under test. I'll keep looking at it.19:38
jlvillallifeless: Basically /me is confused at this point.  I'll work on it some more after lunch.19:39
lifelessjlvillal: confusion is good19:45
lifelessjlvillal: it means one is about to learn something19:45
jlvillalYep19:45
jlvillalOkay, time for lunch19:45
*** baoli has joined #openstack-ironic19:49
*** tsekiyama has quit IRC19:51
*** vdrok has quit IRC19:52
NobodyCamenjoy :)19:54
*** Sukhdev has joined #openstack-ironic19:55
*** baoli has quit IRC19:57
*** tsekiyama has joined #openstack-ironic19:57
*** nicodemos has quit IRC19:58
openstackgerritMerged openstack/ironic-python-agent: Open mitaka development  https://review.openstack.org/23163020:04
*** tsekiyama has quit IRC20:08
*** tsekiyama has joined #openstack-ironic20:09
*** x3k has joined #openstack-ironic20:14
openstackgerritSinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Refactoring python-oneviewclient to use domain models of OneView as objects  https://review.openstack.org/23098820:16
openstackgerritRuby Loo proposed openstack/ironic-specs: Organize Liberty specs into implemented versions  https://review.openstack.org/23169520:21
*** alexpilotti has quit IRC20:21
rloojroll, devananda: wrt the locations of our specs, let me know what you think of ^^, I used symbolic links to group the implemented specs20:22
rloojroll, devananda: I think we could move all the specs to one directory, and just create subdirs/add symbolic links to the actual specs. then no need to move them.20:23
*** sdake has quit IRC20:27
*** sdake has joined #openstack-ironic20:28
openstackgerritMichael Krotscheck proposed openstack/ironic-webclient: Fixed colspan in node list  https://review.openstack.org/23170120:31
*** alexpilotti has joined #openstack-ironic20:33
*** baoli has joined #openstack-ironic20:37
*** alexpilotti has quit IRC20:38
*** mtaylor has joined #openstack-ironic20:38
*** mordred has quit IRC20:42
*** mtaylor is now known as mordred20:42
*** Sukhdev has quit IRC20:42
*** dtantsur|afk has quit IRC20:43
*** ijw has quit IRC20:43
*** davidlenwell has quit IRC20:43
*** jamielennox has quit IRC20:43
*** zigo has quit IRC20:43
*** aarefiev has quit IRC20:43
*** amit213 has quit IRC20:43
*** tonyb has quit IRC20:43
*** krotscheck has quit IRC20:43
*** eglute has quit IRC20:43
*** jroll has quit IRC20:43
*** natorious has quit IRC20:43
*** comstud has quit IRC20:43
*** mgagne has quit IRC20:43
*** TheJulia has quit IRC20:43
*** liliars has quit IRC20:45
*** dtantsur|afk has joined #openstack-ironic20:45
*** ijw has joined #openstack-ironic20:46
*** davidlenwell has joined #openstack-ironic20:46
*** jamielennox has joined #openstack-ironic20:46
*** zigo has joined #openstack-ironic20:46
*** aarefiev has joined #openstack-ironic20:46
*** amit213 has joined #openstack-ironic20:46
*** tonyb has joined #openstack-ironic20:46
*** krotscheck has joined #openstack-ironic20:46
*** eglute has joined #openstack-ironic20:46
*** jroll has joined #openstack-ironic20:46
*** natorious has joined #openstack-ironic20:46
*** comstud has joined #openstack-ironic20:46
*** mgagne has joined #openstack-ironic20:46
*** TheJulia has joined #openstack-ironic20:46
* devananda returns from lunch20:47
*** pleia2 has quit IRC20:47
*** Sukhdev has joined #openstack-ironic20:51
*** morgan has quit IRC20:52
*** morgan has joined #openstack-ironic20:52
*** sdake has quit IRC20:53
*** Sukhdev has quit IRC20:55
devanandajroll: will test tomorrow. I'm not at home right now, and won't be until late20:56
jrollrloo: will look, ty20:56
jrolldevananda: no worries20:56
*** pleia2 has joined #openstack-ironic20:56
devanandajroll: oh - i can already tell you it wont work20:56
devanandathe in-tree AMT driver fails to manage my NUC20:56
jrolldevananda: ......20:57
jrollsigh.20:57
devanandalike - seriously. ironic-conductor starts locking up immediately and misses heartbeats20:57
jrollwe need to fix that20:57
devanandauh huh20:57
jrollsdague has code20:57
devanandaI filed that bug ages ago20:57
jrollwe were waiting on g-r freeze to pull it in20:57
devanandagreat. let's do it20:57
devanandahave a link handy?20:58
jrollhttps://github.com/sdague/amt/20:58
jrollit's factored into a library-like thing20:58
*** sdake has joined #openstack-ironic20:58
rloojroll, devananda: I thought mrda or jlvillal was looking into that?20:58
jrollrloo: maybe, I haven't seen any action20:58
rloojroll, devananda: or maybe they were just using it. I didn't pay much attention.20:58
mrdaMorning all.20:59
jlvillalrloo: jroll: We were looking into it, but then other things got in the way...20:59
jlvillalRight mrda?20:59
*** jcoufal has joined #openstack-ironic20:59
mrdarloo: I started looking, but didn't get far because of the already pre-existing efforts that were underway20:59
jlvillalMorning mrda20:59
jlvillalI did submit a pull request to the amt library of sdague's that got merged in.20:59
jlvillalAnd there is an ugly WIP patch of mine.21:00
TheJuliajroll: I'm actually working on sdague's code now to get tests working21:00
jrolllooks like lintan__ is also working on it a bit21:00
TheJuliabecause amt921:00
devanandajroll: see https://github.com/sdague/amt#amtctrl ?21:00
jrolloh wow21:00
devanandathis is a bit, well, different21:00
jrolldevananda: yes, why?21:00
jrolldevananda: this is a library and also a cli21:00
devanandait sounds like you enroll machines with amtctl to use them ?21:00
jrollok let's back up21:01
devanandaoh - gotcha. there's *also* a CLI21:01
jrollsean made a thing to manage his NUCs21:01
jrollhe wrote code to talk to amt21:01
*** x3k has quit IRC21:01
jrollbut also stuff around it for management21:01
mrda(which is pure python)21:01
jrollthe library is here: https://github.com/sdague/amt/blob/master/amt/client.py21:01
devanandayah - I read more code and understand it now21:01
* jroll is organizing launchpad tomorrow because this is like the tenth person I've explained this to and it should all be on a blueprint21:02
mrda:)21:02
TheJuliahttps://review.openstack.org/#/c/222682/ is sdague's rev21:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP: AMT stuff  https://review.openstack.org/22758121:02
jrollyeah, I believe that was just a wip for others to take over21:03
jlvillaljroll, devananda: https://review.openstack.org/#/c/227581/    AMT WIP patch. Very ugly.21:03
jlvillalBut does get rid of pywsman21:03
jlvillalFor AMT21:03
jrollnow we have two patches \o/21:03
jroll\o/ \o/ \o/21:03
jlvillaljroll: Mine runs :D21:03
jrollI mean, you could have updated the other one, but it's fine21:04
devanandajlvillal: if yours is meant to supercede sdague's patch, perhaps leave a comment there to that effect ...21:04
devanandaor what jroll said21:04
jrolljlvillal: you sure it runs? it has a merge conflict21:04
jlvillalOh, it used to run... :(21:04
devanandabut yea, this is a pretty significant change over sdague's patch, so I'd prefer a separate one (not just an update of the same rev)21:04
*** trown is now known as trown|outttypeww21:05
devanandaheh - well, also, we can't dothis in requirements.txt ... -e git+https://github.com/JohnVillalovos/amt.git#egg=amt21:06
TheJuliaamt is in pip now21:06
devanandajlvillal: is sdague ... nvm ^ :)21:06
TheJuliaso driver-requirements.txt21:06
devanandaTheJulia: awesome21:06
jlvillaldevananda: True! But I think my patch is now in sdague's amt library.21:07
* devananda is very confused21:07
TheJuliajlvillal: the pull req was merged21:07
jlvillalYes21:07
TheJuliajlvillal: I think he cut a release, looking21:07
*** harshs has quit IRC21:07
krotscheckNobodyCam: You around?21:07
jlvillalNot sure about a release21:07
TheJulia26 days, your request was merged after21:08
*** harshs has joined #openstack-ironic21:08
TheJuliaso it would need to be released again to get your pull req in21:08
*** jcoufal has quit IRC21:09
*** penick has joined #openstack-ironic21:09
NobodyCammaybe21:09
NobodyCamkrotscheck: whats up21:09
TheJuliamaybe we need to just need to setup a specific time to try and all get on the same page and a direction forward on the amt driver?21:10
jlvillalTheJulia: I pinged sdague to ask him to release a new version21:10
krotscheckNobodyCam: Refresh my memory. To enroll node it's: Get driver, get driver properties, ask user to fillin values for those properties, POST to create new instance, and then it'll be in an 'enroll' state?21:10
NobodyCamkrotscheck: instance may not be the correct word here/ state will depend on what version your on21:12
TheJuliakrotscheck: this may help https://github.com/openstack/ironic-specs/blob/master/specs/kilo-implemented/new-ironic-state-machine.rst as well as confuse21:12
*** baoli has quit IRC21:12
* TheJulia ducks21:12
krotscheckTheJulia: I'm going off this -> http://docs.openstack.org/developer/ironic/_images/states.svg21:12
krotscheckNobodyCam: Yes yes semantics. Overall it's basically the right steps?21:13
rlookrotscheck: that's right. your steps i mean.21:13
krotscheckDanke21:13
NobodyCamyep21:13
NobodyCamhey rloo :)21:13
*** baoli has joined #openstack-ironic21:14
rlooNobodyCam: :)21:14
jrollTheJulia: I think we just need to document what we're doing/planning better21:14
*** alexpilotti has joined #openstack-ironic21:14
rloojroll: you mean make use of LP/bugs better? etherpad? mail list?21:15
jrollrloo: yes.21:15
rloojroll: sounds good to me :)21:16
jroll:)21:16
mrda\o/21:16
TheJuliaso is it a bug or a feature that the pywsman based implementation of amt support only supports up through AMT7?21:18
*** alexpilotti has quit IRC21:19
NobodyCammornig mrda :)21:19
devanandakrotscheck: is webclient passing a version header?21:19
krotscheckdevananda: Nope. It's assuming latest.21:19
devanandakrotscheck: I'm grepping in webclient but do not see the strings I would expenct21:19
devanandakrotscheck: "assuming latest" ?21:19
devanandakrotscheck: ironic-api defaults to 1.0 if no header is passed21:19
mrdahey NobodyCam ;)21:19
devanandabecause backwards compat21:20
openstackgerritMerged openstack/python-oneviewclient: Creating OneViewException class  https://review.openstack.org/23165521:20
krotscheckGreat, so I have to write version selection logic now too.21:20
krotscheckBecause why would we ever want to have an API be easy.21:20
* jlvillal uses pdb for the first time...21:20
krotscheckAlso, I was under the impression that if I just go to / (without /v1/ or a header) it would default to the most recent version.21:22
devanandakrotscheck: no. without any header, all endpoints will default to the _oldest_ version21:23
devanandaso that we do not break older clients that did not send a header21:23
NobodyCambrb21:24
*** e0ne has quit IRC21:24
devanandakrotscheck: if you want the initial state to be ENROLL, you should request a version >= 1.1121:24
*** baoli has quit IRC21:25
*** rvasilets___ has joined #openstack-ironic21:26
jrollTheJulia: I have no clue so I'm going to call it a bug, however I'm inclined to just delete the entire AMT driver and start over as feature work, since I haven't heard anything other than "this doesn't work"21:26
*** baoli has joined #openstack-ironic21:26
*** baoli has quit IRC21:27
mrdajroll: the current amt driver does work, for some subset of use cases21:27
* jroll wtb third party CI on all the drivers21:27
devanandajroll: TheJulia: intel changed the AMT protocol (as in, complete rewrite) at v7 and again at v1021:27
jrollmrda: which use cases does it work for?21:27
mrdaI can do a ramdisk-based deploy acceptably21:27
devanandathere is no single client library which supports all THREE formats21:27
devanandaat least as far as I know21:27
jrollmrda: from what you've told me you need to play with sockets for that to work reliably?21:28
devanandamrda: I can not get it to work :(21:28
*** baoli has joined #openstack-ironic21:28
mrda...for my NUC which is a DC53427HYE21:28
jrolls/play/do weird magic/21:28
*** baoli has joined #openstack-ironic21:28
devanandamrda: same model # as mine21:28
*** jcoufal has joined #openstack-ironic21:29
mrdaI documented what I did here: http://lifelog.michaeldavies.org/2015/09/ironic-on-nuc-part-1-setting-things-up.html and http://lifelog.michaeldavies.org/2015/09/ironic-on-nuc-part-2-running-services.html21:29
*** Sukhdev has joined #openstack-ironic21:29
*** baoli has quit IRC21:29
jrollmrda: didn't you have to do something weird or connections would fail often?21:30
mrdakey point is to "nc -zv host port" just priori to setting the provisioning state to active21:31
*** baoli has joined #openstack-ironic21:31
jrollmrda: yeah, that's terribly broken IMO21:31
mrdain fact, I have a script that works 100% of the time now :) https://github.com/mrda/junkcode/blob/master/deploy-nuc.sh21:32
jroll:((((((((((((((((((((((((((21:32
jrollis what that does to me21:32
TheJulia:(21:32
mrdaI didn't say that the new amt driver work isn't incredibly  important21:32
mrdaJust that it can be made to work21:32
openstackgerritafazekas proposed openstack/ironic-python-agent: Not depending on bash  https://review.openstack.org/23172321:33
mrdaNotably I can't get a ipa deploy to work though21:33
mrdathat trips up the threading issue 100% of the time21:33
jrollit works if you mash the AMT with connections. that's so broken21:33
jrollinteresting21:33
mrdaI think what I'm saying is that I agree with you21:33
*** alexpilotti has joined #openstack-ironic21:33
jrollhah21:33
mrdaexcept that the current driver isn't 100% broken, only mostly so21:34
jrollthat sounds like 100% broken to me :(21:34
rloo'mostly broken' doesn't seem very good to me21:34
jrollwould it work for you if I put the existing one on github as something that could be pip install'd?21:34
rlooi would agree that we should pull the driver out in that case21:34
jrollwhile I delete it from our tree?21:34
*** baoli has quit IRC21:34
mrdaI wouldn't be happy with that until the new driver work was done21:35
mrdabecause at least now I have something that can use to test ironic on real hardware21:35
jrollwould it work for you if I put the existing one on github as something that could be pip install'd?21:35
jrollpip install [...]jimrollenhagen/ironic-broken-amt21:35
mrdaI guess so, but I think adding the new driver first would be a better solution21:36
jrollI can't describe how much I hate that we're shipping a broken driver.21:36
mrdawell, let's get the new driver in then!21:36
*** baoli has joined #openstack-ironic21:36
mrdaI'm happy to help, of course21:37
jrollit's going to be a 90% rewrite, I'm not sure I see the benefit of keeping the existing one in tree21:37
jrollbut /shrug, it's up to all of us, not just me21:37
rloohmm, jroll etc. do we have a bug about AMT driver? I just looked at HIGH bugs and I don't see anything about AMT.21:38
mrdaIf that's the consensus I'll just grab the current driver and patch when I want to test on real hardware21:38
jrollrloo: no clue21:38
*** alexpilotti has quit IRC21:38
* mrda shrugs21:38
rloojroll: this is the first I've heard that the driver is so bad.21:38
TheJuliarloo: There is one for pywsman treading issues, but nothing on compatability issues21:38
TheJuliarloo: at least as of 20 minutes ago when I looked21:38
jrollrloo: I've heard "it isn't good at concurrency", not "it breaks every time"21:39
mrdaYeah, I think the issues with amt are bundled up with the openwsman issue21:39
mrdajroll: It doesn't break *everytime*21:39
rloojroll: honestly. w/o a high or critical bug that the driver is bad, i don't see why we'd pull it out.21:39
mrdajust "often"21:39
jrollmrda: sigh21:40
mrdamy little hack just balances the terms of engagement.  Without it, it still works "often", at least for me21:40
devanandamrda: you can use my AMTTool driver (from github) right now. it works 100% of the time for me.21:40
mrda\o/21:40
devanandamrda: without having to do any side-bar mashing of connections21:40
devanandawith IPA21:40
devanandaand it has worked since Vancouver21:40
mrdadevananda: I can give that a go then21:41
devanandarloo: I filed a bug about how broken it is moooonths ago21:41
* devananda looks21:41
*** baoli has quit IRC21:41
devanandarloo: https://bugs.launchpad.net/ironic/+bug/145449221:41
openstackLaunchpad bug 1454492 in Ironic "openwsman library is not greenthread-aware, leads to driver lockup with both DRAC and AMT drivers" [Medium,Triaged] - Assigned to Imre Farkas (ifarkas)21:41
devanandaalso - as that bug says - the issue also affects the DRAC driver21:41
devanandathough DRAC hardware is, I hope, less prone to dropping network packets21:42
* mrda clones into a directory called "do-not-delete-amt-driver-works-for-me-mostly"21:42
jrollhttps://blueprints.launchpad.net/ironic/+spec/rewrite-amt-driver21:42
krotscheckIs it feasible to improve the data returned from /driver/properties? It'd be neat if there was a machine-readable way of determining defaults, required/optional, data types, and/or regex input ranges.21:42
jroll^ let's call that BP the bug21:42
rloodevananda: oh. yuck. so it isn't high. is that the only reason AMT drivers are bad?21:42
*** bodgix has joined #openstack-ironic21:42
*** baoli has joined #openstack-ironic21:42
krotscheckhttp://imgur.com/FasgoeD <-- Not human friendly21:42
devanandarloo: using openwsman/pywsman is the main reason the in-tree driver is bad, yes21:43
mrdarloo: I think so - that's the root cause21:43
*** baoli has quit IRC21:43
devanandakrotscheck: welp. we thought about it :)21:43
TheJuliarloo: There are also the protocol differences between some of the versions that are out there where it is not backwards compatible, that doesn't help htings21:43
TheJuliathings21:43
krotscheckdevananda: Thinking is good!21:43
*** baoli has joined #openstack-ironic21:44
devanandarloo: as I commented a bit ago, there are 3 different protocos for AMT-capable hardware21:44
devanandathe driver in my github repo supports only v7 - v921:44
jrollkrotscheck: it's totally possible. might be hard. you're welcome to send ideas/patches21:44
devanandasdague's NUC is v10, fwiw ....21:44
rloodevananda, TheJulia: sigh. that should be another bug then.21:44
jrollrloo: TheJulia devananda mrda can we collect thoughts/ideas/plans/etc here? (and escalate to a spec if we think that's necessary) https://blueprints.launchpad.net/ironic/+spec/rewrite-amt-driver21:45
devanandakrotscheck: we would need to use some sort of machine // human readable language to describe complex logic21:45
mrdajroll: sounds good21:45
devanandaetpad might be easier for first pass21:45
devanandahttps://etherpad.openstack.org/p/new-ironic-amt-driver21:46
TheJuliajroll: I was also going to say a etherpad for a first pass, but sounds good21:46
*** harlowja has quit IRC21:46
* devananda sets the BP spec URL to 'pad21:46
jrolldevananda: link the etherpad there please :) when yet-another-person decides they hate this driver, I want something to point them to21:46
jrollthanks21:46
rlooetherpad is good, but LP is placeholder so folks can see that something might be in the works :)21:46
jroll++21:46
*** baoli has quit IRC21:47
* TheJulia watches devananda type21:47
*** baoli has joined #openstack-ironic21:47
mrdaTheJulia: you have strange hobbies ;)21:48
*** baoli has quit IRC21:48
krotscheckjroll, devananda: Yeah, that's outside of my area of caring at the moment. I have zero energy to go through the inevitable design arguments.21:49
*** baoli has joined #openstack-ironic21:49
devanandakrotscheck: fair 'nuf. I recall discussing it with rloo when we implemented that API, but we settled on "make it human readable text, because metalanguages are even harder"21:49
rlookrotscheck: yeah. like he said. there were some that are sometimes required or something like that.21:51
devanandalots of conditionals and "if you specify X, then you also specify either Y or Z"21:52
devanandasorts of things21:52
*** harshs has quit IRC21:53
rlookrotscheck: eg 'node's UDP port to connect to. Only required for console access'21:53
devanandamrda, TheJulia: I am done typing. please carry on with the things I don't know21:54
mrdadevananda: shall do21:55
*** baoli has quit IRC21:56
*** baoli has joined #openstack-ironic21:56
*** harlowja has joined #openstack-ironic21:57
*** jobewan has quit IRC22:03
*** priteau has quit IRC22:04
*** priteau has joined #openstack-ironic22:04
*** baoli has quit IRC22:08
*** baoli has joined #openstack-ironic22:09
*** tombee has quit IRC22:09
*** boris-42 has quit IRC22:09
*** davideagnello has quit IRC22:10
*** baoli has quit IRC22:13
*** baoli has joined #openstack-ironic22:14
*** ijw has quit IRC22:14
*** alexpilotti has joined #openstack-ironic22:15
* jlvillal missed a lot of AMT discussion while he was trying to do a pdb debug session....22:15
*** baoli has quit IRC22:16
*** baoli has joined #openstack-ironic22:19
*** alexpilotti has quit IRC22:19
harlowjadtantsur|afk https://review.openstack.org/#/c/231721/22:20
harlowjadtantsur|afk think that will enable ironic and stuffs22:20
*** aweeks has quit IRC22:22
*** Ng has quit IRC22:22
*** JoshNang has quit IRC22:22
*** cppforlife_ has quit IRC22:22
*** lekha has quit IRC22:22
*** BadCub has quit IRC22:22
*** baoli has quit IRC22:22
*** baoli has joined #openstack-ironic22:23
openstackgerritKurt Taylor proposed openstack/ironic-inspector: Add coverage target to tox.ini  https://review.openstack.org/23149522:23
*** tombee has joined #openstack-ironic22:24
openstackgerritKurt Taylor proposed openstack/ironic-inspector: Add coverage target to tox.ini  https://review.openstack.org/23149522:24
*** baoli has quit IRC22:28
*** baoli has joined #openstack-ironic22:28
*** priteau has quit IRC22:28
*** priteau has joined #openstack-ironic22:32
*** alexpilotti has joined #openstack-ironic22:32
*** priteau has quit IRC22:32
* mrda finishes updating https://etherpad.openstack.org/p/new-ironic-amt-driver22:32
*** Marga_ has quit IRC22:33
*** priteau has joined #openstack-ironic22:33
*** priteau has quit IRC22:34
* mrda finishes updating https://etherpad.openstack.org/p/new-ironic-amt-driver22:34
mrdasorry22:34
*** alexpilotti has quit IRC22:36
*** Marga_ has joined #openstack-ironic22:37
*** jcoufal has quit IRC22:37
*** priteau has joined #openstack-ironic22:42
JayFjroll: I think you're the only person who can land this: https://review.openstack.org/#/c/231566/222:44
*** VikasC has joined #openstack-ironic22:44
JayFalso I'm like 80% sure the IPA gate is broken22:45
JayFwith the setup.cfg change that got merged22:45
*** priteau has quit IRC22:46
jrollJayF: I think patches before that setup.cfg change were broken22:46
jrollcouldn't have merged it if it broke the gate....22:46
JayFso hit the rebase button and it should fix for any changes with all the jobs busted?22:47
JayFI would've changed the order I landed things this morning had I known that :(22:47
jrollit should yeah22:47
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Fix log formatting error in iscsi.py  https://review.openstack.org/23163422:47
jrollalso, I believe this was the person talking about trying to slim the heck out of IPA this morning in -qa https://review.openstack.org/#/c/231723/22:48
jrollfor context22:48
*** priteau has joined #openstack-ironic22:48
JayFThat's fine; but it should still be documented with a bug22:48
JayFor bp22:48
jrolloh I know22:48
jrolljust providing context22:48
jrollI'm still not sure that's the right thing to do, but I'm curious22:48
JayFIs that also why Dan is working on the DIB image?22:49
jlvillallifeless: I'm thinking Nova ran into this issue before with logging: https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L7822:49
JayFHe's had a few patches in that direction recently22:49
jrollJayF: dan prince?22:49
jrollnot that I know of22:49
JayFyeah22:49
jrollnah22:49
jrollhe's trying to switch to IPA :P22:49
*** priteau has quit IRC22:49
*** boris-42 has joined #openstack-ironic22:53
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/23123022:53
lifelessjlvillal: maybe, can't tell from that22:55
*** priteau has joined #openstack-ironic22:55
*** davideagnello has joined #openstack-ironic22:56
*** priteau has quit IRC22:59
*** chlong has quit IRC23:06
*** baoli has quit IRC23:07
*** igordcard_ has joined #openstack-ironic23:08
*** igordcard_ has quit IRC23:08
*** igordcard_ has joined #openstack-ironic23:08
*** alexpilotti has joined #openstack-ironic23:13
*** alexpilotti has quit IRC23:18
*** Sukhdev has quit IRC23:20
*** igordcard_ has quit IRC23:20
*** harshs has joined #openstack-ironic23:21
*** igordcard_ has joined #openstack-ironic23:22
*** igordcard_ has quit IRC23:29
*** rvasilets___ has quit IRC23:31
*** tsekiyama has quit IRC23:35
*** igordcard_ has joined #openstack-ironic23:36
*** aweeks has joined #openstack-ironic23:36
*** achanda_ has joined #openstack-ironic23:36
*** lekha has joined #openstack-ironic23:36
*** Ng has joined #openstack-ironic23:38
jlvillalrloo: Would how to use pdb with tox be a 'quick start' entry?23:39
jlvillaldeveloper quick start example, that is23:39
*** BadCub has joined #openstack-ironic23:39
*** achanda has quit IRC23:40
*** cppforlife_ has joined #openstack-ironic23:40
*** lucas-dinner has quit IRC23:41
*** achanda_ has quit IRC23:41
*** sdake_ has joined #openstack-ironic23:41
*** sdake has quit IRC23:43
*** tsekiyama has joined #openstack-ironic23:44
* jlvillal wonders when Nova will release Liberty23:49
jrolljlvillal: next week along with everyone else: https://wiki.openstack.org/wiki/Liberty_Release_Schedule23:50
jlvillaljroll: Ah. ironic released before because on new release cycle. Thanks23:51
jrolljlvillal: ironic 4.2.0 is roughly equivalent to nova rc123:52
*** sdake_ has quit IRC23:52
jlvillalOkay23:52
jrollfor the purposes of timing anyway23:54
*** tsekiyama has quit IRC23:54
*** alexpilotti has joined #openstack-ironic23:55
*** mestery has quit IRC23:56
*** tsekiyama has joined #openstack-ironic23:56
*** mestery has joined #openstack-ironic23:56
JayFany objections to me landing this spec? https://review.openstack.org/#/c/228829/ cc: jroll23:57
JayFspec landing in 2 patchsets might be a record23:57
jrollI haven't looked, but you're core because we trust you23:58
*** alexpilotti has quit IRC23:59

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