Thursday, 2016-01-07

*** baoli has quit IRC00:04
*** dims_ has quit IRC00:07
*** sacharya has joined #openstack-ironic00:16
*** absubram has quit IRC00:17
*** Sukhdev has joined #openstack-ironic00:17
Sukhdevjroll: that was kind of fresh install - did not clean up the pip packages though :-):-)00:19
*** sacharya has quit IRC00:21
jrollSukhdev: run unstack.sh and stack.sh again, if you didn't already00:21
Sukhdevjroll: did it couple of times already - now getting ready to clean up all pip packages - just incase some old residual mismatched version of any package is causing the issue and then will do unstack/stack again00:23
jrollSukhdev: yeah, like I said I always just reinstall the OS with things like this00:24
Sukhdevjroll : makes sense - let me give it a shot -00:25
*** baoli has joined #openstack-ironic00:25
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Allow hardware managers to override clean step priority  https://review.openstack.org/18996900:31
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Allow hardware managers to override clean step priority  https://review.openstack.org/18996900:37
*** hoangcx_ has joined #openstack-ironic00:39
*** yolanda has quit IRC00:50
*** dims has joined #openstack-ironic00:56
*** baoli has quit IRC00:57
*** [1]rpioso has quit IRC00:58
*** jaybeale has quit IRC00:59
*** jaypipes has quit IRC00:59
jxiaobinhi folks: if a node is in "clean failed" state, how to make it available again?01:09
openstackgerritStephanie Miller proposed openstack/bifrost: Use "state=touch" instead of running touch  https://review.openstack.org/26449801:13
*** boris-42 has quit IRC01:13
*** _zouyee has joined #openstack-ironic01:14
openstackgerritMerged openstack/ironic: Remove release differences from flavor creation docs  https://review.openstack.org/26429001:16
openstackgerritNaohiro Tamura proposed openstack/ironic: Add hardware inspection module for iRMC driver  https://review.openstack.org/19648001:19
*** praneshp has quit IRC01:22
*** Sukhdev has quit IRC01:25
lintanhi jxiaobin, ironic node-set-provision-state node manage01:28
lintanthen run 'ironic node-set-provision-state node provide' to make run auto clean again and become available and ready to deploy01:30
jxiaobinlintan: thanks! just figured it out!01:32
openstackgerritYuiko Takada proposed openstack/ironic: Change default IRONIC_VM_SPECS_RAM to 1024  https://review.openstack.org/26450601:37
*** Sukhdev has joined #openstack-ironic01:38
*** verdurin has quit IRC01:38
*** _zouyee has quit IRC01:41
*** absubram has joined #openstack-ironic01:42
*** Sukhdev has quit IRC01:43
*** absubram_ has joined #openstack-ironic01:43
*** garthb_ has quit IRC01:45
*** absubram has quit IRC01:47
*** absubram_ is now known as absubram01:47
*** mtanino has quit IRC01:47
*** verdurin has joined #openstack-ironic01:48
JayFHow do you run a single test?01:50
JayFlike if I have a single test failing and want to try it over and over without running the whole world01:50
JayF(in IPA, but I don't think it matters)01:50
jrolltox -e py27 -- --regex ironic.tests.unit....01:51
jrollend with .ClassName.test_name01:51
JayFthat doesn't work in ipa :(01:52
jrollwtf01:52
JayFI'll dig into it later01:52
jrollnah01:52
jrollI thought we switched to ostestr there01:52
JayFI got a usage for setup.py01:52
jrollwith just testr it's tox -e py27 ironic.tests.unit....ClassName.test_name01:52
jrollshould work for ipa ^ JayF01:53
JayFtrying01:53
jrollfwiw the extra stuff is the {posargs} part of tox.ini01:53
*** kan_ has joined #openstack-ironic01:54
JayFmorgabra helped me do the deduplicate stuff more pleasantly, but tests fail on py34 but not py2701:54
jrollorly01:55
openstackgerritZhenguo Niu proposed openstack/ironic-specs: Use cached copy of hash ring in api service  https://review.openstack.org/25069201:55
* jroll is curious now01:55
JayFjroll: onmetal engy room if you wanna see01:55
jrollokayyyyy01:55
*** praneshp has joined #openstack-ironic01:56
*** praneshp_ has joined #openstack-ironic01:59
*** praneshp has quit IRC02:00
*** praneshp_ is now known as praneshp02:00
jrollJayF: https://www.destroyallsoftware.com/talks/wat02:00
openstackgerritZhenguo Niu proposed openstack/ironic-specs: Console ports allocation management  https://review.openstack.org/24987602:01
*** harshs has quit IRC02:01
*** davideagnello has quit IRC02:07
openstackgerritZhenguo Niu proposed openstack/ironic: Start dead consoles on conductor startup  https://review.openstack.org/25725202:07
*** s-moriya has quit IRC02:13
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Allow hardware managers to override clean step priority  https://review.openstack.org/18996902:16
JayFrloo_: ^ morgabra helped me perform functional-surgery on that, tests are passing including a new one I added, would love a re-review02:18
JayFmariojv: ^ same02:18
rloo_JayF: tomorrow. I am trying to finish up reviewing your reboot-requested one right now.02:19
JayFThat's fine :) I'm just glad these are not awful02:19
*** piet has joined #openstack-ironic02:20
JayFthat code was awful and it's not that bad02:20
rloo_JayF: awful is in the eye of the beholder :D02:21
JayFwell I beheld that code a lot02:23
JayFand I declared it awful02:24
*** andrewxu has joined #openstack-ironic02:28
*** absubram has quit IRC02:28
rloo_JayF: you're allowed to say that then :)02:32
*** rloo_ has quit IRC02:32
*** piet has quit IRC02:35
*** harshs has joined #openstack-ironic02:36
*** deray has joined #openstack-ironic02:38
*** deray has quit IRC02:39
openstackgerritKan proposed openstack/ironic-inspector: Use assertTrue/False instead of assertEqual(T/F)  https://review.openstack.org/26452202:42
*** baoli has joined #openstack-ironic02:42
*** baoli has quit IRC02:46
*** baoli has joined #openstack-ironic02:47
*** Fdaisuke has joined #openstack-ironic02:48
*** baoli has quit IRC02:48
*** harshs has quit IRC02:51
*** baoli has joined #openstack-ironic02:54
*** baoli has quit IRC02:54
*** baoli has joined #openstack-ironic02:55
*** dims has quit IRC02:58
openstackgerritKan proposed openstack/python-ironicclient: Format the valid fields for list commands  https://review.openstack.org/24554103:00
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: continue to delete next node if failed with previous one  https://review.openstack.org/26239303:02
*** boris-42 has joined #openstack-ironic03:02
openstackgerritReedip proposed openstack/pyghmi: Fix py34 DB issue  https://review.openstack.org/26051703:03
*** links has joined #openstack-ironic03:10
*** links has quit IRC03:10
openstackgerritKan proposed openstack/ironic: Remove the duplicate target power state check  https://review.openstack.org/26452703:11
*** baoli has quit IRC03:12
*** links has joined #openstack-ironic03:12
*** sacharya has joined #openstack-ironic03:14
*** jaybeale has joined #openstack-ironic03:23
*** sacharya has quit IRC03:23
*** sacharya has joined #openstack-ironic03:34
*** sacharya has quit IRC03:40
*** ChubYann has quit IRC03:42
*** thrash is now known as thrash|g0ne03:42
*** raddaoui has joined #openstack-ironic03:45
*** jaybeale has quit IRC03:48
*** vishwanathj has joined #openstack-ironic03:49
*** vishwana_ has quit IRC03:51
*** sacharya has joined #openstack-ironic04:02
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: devstack 'cleanup-node' script should delete OVS bridges  https://review.openstack.org/26350804:15
*** coolsvap|away is now known as coolsvap04:21
*** _zouyee has joined #openstack-ironic04:28
*** piet has joined #openstack-ironic04:29
openstackgerritYatin Kumbhare proposed openstack/ironic: Use assertTrue/False instead of assertEqual(T/F)  https://review.openstack.org/26415604:39
*** harshs has joined #openstack-ironic04:42
*** piet has quit IRC04:47
openstackgerritYuiko Takada proposed openstack/ironic-specs: Add allowed transitions to API  https://review.openstack.org/22402204:50
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/26442204:53
*** andrewxu has quit IRC04:59
openstackgerritReedip proposed openstack/pyghmi: Remove support for py33/py26  https://review.openstack.org/26457005:00
*** Nisha has joined #openstack-ironic05:02
*** andrewxu has joined #openstack-ironic05:04
*** absubram has joined #openstack-ironic05:06
*** vinm213 has joined #openstack-ironic05:09
*** smoriya_ has joined #openstack-ironic05:13
*** teju has joined #openstack-ironic05:13
*** kevinbenton has quit IRC05:14
*** kevinbenton has joined #openstack-ironic05:14
*** andrewxu has quit IRC05:16
*** andrewxu has joined #openstack-ironic05:19
*** baoli has joined #openstack-ironic05:23
openstackgerritShivanand Tendulker proposed openstack/ironic-specs: Baremetal provisioning in UEFI secure boot mode  https://review.openstack.org/23027405:24
*** baoli has quit IRC05:28
*** kevinbenton has quit IRC05:29
*** piet has joined #openstack-ironic05:31
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib: WIP: Testing  https://review.openstack.org/26457705:41
openstackgerritVaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic  https://review.openstack.org/26457905:42
*** piet has quit IRC05:43
*** jaybeale has joined #openstack-ironic05:50
*** harshs has quit IRC05:54
*** rajinir has quit IRC05:55
openstackgerritVaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic  https://review.openstack.org/26457905:56
*** appu103_ has joined #openstack-ironic05:56
openstackgerritVaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic  https://review.openstack.org/26457906:00
*** vinaym213 has joined #openstack-ironic06:01
*** vinm213 has quit IRC06:05
*** absubram has quit IRC06:05
*** jaybeale has quit IRC06:05
*** piet has joined #openstack-ironic06:08
*** sacharya has quit IRC06:08
openstackgerritMerged openstack/ironic-inspector: Use assertTrue/False instead of assertEqual(T/F)  https://review.openstack.org/26452206:09
openstackgerritVaishnavi proposed openstack/ironic: Add support for building ISO for deploy ramdisk  https://review.openstack.org/26459006:19
*** vinay50muddu has joined #openstack-ironic06:23
*** piet has quit IRC06:24
*** vinaym213 has quit IRC06:27
*** sacharya has joined #openstack-ironic06:30
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support  https://review.openstack.org/26425706:34
*** sacharya has quit IRC06:38
*** moshele has joined #openstack-ironic06:44
*** deray has joined #openstack-ironic06:48
openstackgerritMoshe Levi proposed openstack/ironic: Adding InfiniBand Support  https://review.openstack.org/26426306:49
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/26442306:54
*** coolsvap is now known as coolsvap|away06:56
*** hoangcx_ has quit IRC07:00
openstackgerritKan proposed openstack/python-ironicclient: Add CLI to list nodes using the same driver  https://review.openstack.org/26400707:04
openstackgerritKan proposed openstack/ironic: API to list nodes using the same driver  https://review.openstack.org/26400507:08
*** yolanda has joined #openstack-ironic07:12
openstackgerritKan proposed openstack/ironic: API to list nodes using the same driver  https://review.openstack.org/26400507:16
openstackgerritKan proposed openstack/ironic: API to list nodes using the same driver  https://review.openstack.org/26400507:18
openstackgerritNaohiro Tamura proposed openstack/ironic: Add hardware inspection module for iRMC driver  https://review.openstack.org/19648007:21
*** sirmonkey has quit IRC07:24
*** praneshp has quit IRC07:28
openstackgerritChangBo Guo(gcb) proposed openstack/ironic: test:clean up wrong value in method CONF.set_override  https://review.openstack.org/24343007:29
*** ukalifon1 has joined #openstack-ironic07:36
*** appu103_ has quit IRC07:36
*** appu103___ has joined #openstack-ironic07:36
openstackgerritLiuNanke proposed openstack/python-ironicclient: Remove openstack-common.conf  https://review.openstack.org/26464807:38
*** sacharya has joined #openstack-ironic07:38
*** coolsvap|away is now known as coolsvap07:41
*** sacharya has quit IRC07:43
*** raddaoui has quit IRC07:50
*** stendulker has joined #openstack-ironic08:16
openstackgerritKan proposed openstack/ironic: Remove the duplicate target power state check  https://review.openstack.org/26452708:19
*** ifarkas has joined #openstack-ironic08:40
openstackgerritMerged openstack/ironic: Use assertTrue/False instead of assertEqual(T/F)  https://review.openstack.org/26415608:44
*** ijw has quit IRC08:49
openstackgerritVaishnavi proposed openstack/ironic: Add support for building ISO for deploy ramdisk  https://review.openstack.org/26459008:53
*** ionutbalutoiu has joined #openstack-ironic08:53
*** jcoufal has joined #openstack-ironic08:57
*** hoangcx has joined #openstack-ironic09:00
*** athomas has joined #openstack-ironic09:01
openstackgerritNisha Agarwal proposed openstack/ironic: Add support for providing custom Swift endpoint URLs  https://review.openstack.org/25127709:20
*** appu1034 has joined #openstack-ironic09:32
*** appu103___ has quit IRC09:32
*** daemontool has joined #openstack-ironic09:32
*** derekh has joined #openstack-ironic09:35
*** sacharya has joined #openstack-ironic09:40
*** jistr has joined #openstack-ironic09:41
*** sacharya has quit IRC09:44
*** chopmann has joined #openstack-ironic09:55
*** chopmann is now known as sirmonkey09:55
*** ndipanov has joined #openstack-ironic09:57
*** dtantsur|afk is now known as dtantsur10:03
*** mkovacik has joined #openstack-ironic10:04
dtantsurMorning Ironic10:04
*** Haomeng has left #openstack-ironic10:07
*** Haomeng has joined #openstack-ironic10:08
*** electrofelix has joined #openstack-ironic10:15
*** sambetts has joined #openstack-ironic10:15
sambettsMorning all o/10:16
*** appu103___ has joined #openstack-ironic10:18
dtantsurmorning sambetts10:21
*** appu1034 has quit IRC10:22
sambettso/ dtantsur10:25
stendulkermorning dtantsur sambetts10:27
*** jcoufal has quit IRC10:28
*** appu1034 has joined #openstack-ironic10:43
*** hoangcx has quit IRC10:43
*** Nisha has quit IRC10:43
*** appu103___ has quit IRC10:46
*** mgould has joined #openstack-ironic10:53
mgouldmorning ironic10:59
dtantsurmorning mgould11:02
mgoulddtantsur, morning!11:05
*** dims has joined #openstack-ironic11:10
*** coolsvap is now known as coolsvap|away11:11
*** pch has joined #openstack-ironic11:11
*** appu1034 has left #openstack-ironic11:14
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP][REALLY WIP] Switch to Futurist library for periodic tasks  https://review.openstack.org/26472011:14
*** stendulker has quit IRC11:18
*** lucas-dinner is now known as lucasagomes11:20
lucasagomesmorning all11:20
*** aparnav has joined #openstack-ironic11:20
*** andrewxu has quit IRC11:26
*** cdearborn has joined #openstack-ironic11:40
*** devananda has quit IRC11:54
*** devananda has joined #openstack-ironic11:55
*** kalpase has joined #openstack-ironic12:01
sambettslucasagomes: Morning o/12:05
*** thrash|g0ne is now known as thrash12:20
*** jcoufal has joined #openstack-ironic12:27
*** kalpase has quit IRC12:32
*** raildo-afk is now known as raildo12:34
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/26442212:37
TheJuliaGood morning everyone12:38
*** kalpase has joined #openstack-ironic12:39
*** kalpase has left #openstack-ironic12:39
*** kalpase has joined #openstack-ironic12:39
*** baoli has joined #openstack-ironic12:41
deraygood morning to all, good evening to me :)12:41
dtantsurg'evening deray, g'morning TheJulia, lucasagomes :)12:48
deraydtantsur, o/12:48
openstackgerritChangBo Guo(gcb) proposed openstack/ironic: test:clean up wrong value in method CONF.set_override  https://review.openstack.org/24343012:52
*** sirmonkey has quit IRC12:52
*** chopmann has joined #openstack-ironic12:52
*** chopmann is now known as sirmonkey12:52
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Switch to Futurist library for periodic tasks  https://review.openstack.org/26472012:53
dtantsuroh, this patch has 7 potential conflicts >_<12:54
*** smoriya_ has quit IRC12:59
*** aparnav has quit IRC13:00
*** dprince has joined #openstack-ironic13:07
*** links has quit IRC13:07
*** rbudden has joined #openstack-ironic13:09
openstackgerritMathieu Mitchell proposed openstack/ironic-python-agent: Mount bind /etc/resolv.conf in chroot  https://review.openstack.org/26476713:21
lucasagomesdtantsur, yeah I'm testing the IPA venv right now, I think that --install-option= is not working13:23
dtantsurgot it13:24
lucasagomesjust changed it to create a symlink13:24
lucasagomesas it was in patch-set #1013:24
*** sacharya has joined #openstack-ironic13:24
lucasagomescause I recall that working fine13:24
*** sacharya has quit IRC13:29
openstackgerritMoshe Levi proposed openstack/ironic: Adding InfiniBand Support  https://review.openstack.org/26426313:29
*** harshs has joined #openstack-ironic13:35
*** trown|outttypeww is now known as trown13:35
lucasagomesdtantsur, just uploaded a new version creating the symlink, that worked here locally13:36
lucasagomes| f109461b-c652-4356-b24a-545a94bf95e3 | testing | ACTIVE | -          | Running     | private=10.1.0.8, fd4d:7512:12c1:0:5054:ff:fe31:1138 |13:36
dtantsurcool!13:36
lucasagomesnot sure why --install-option doesn't work, I will investigate but for now let's just keep the symlink13:36
*** ndipanov has quit IRC13:36
*** kan_ has quit IRC13:40
*** sirmonkey has quit IRC13:44
*** rloo has joined #openstack-ironic13:46
*** chopmann has joined #openstack-ironic13:49
*** chopmann is now known as sirmonkey13:49
*** absubram has joined #openstack-ironic13:57
*** _zouyee has quit IRC13:57
*** absubram_ has joined #openstack-ironic13:58
derayhi all, it would be nice if you can spare some time to review this: https://review.openstack.org/#/c/21101913:59
*** absubram has quit IRC14:01
*** absubram_ is now known as absubram14:01
dtantsurjroll, reno jobs added to IPA, wanna make a release soonish?14:03
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Switch to Futurist library for periodic tasks  https://review.openstack.org/26472014:05
dtantsurrloo, if you feel like providing an early feedback ^^14:06
*** intr1nsic has quit IRC14:06
rloodtantsur: not really :) How WIP is it?14:06
dtantsurrloo, "I suspect it might work, but never tried, and unit tests do not pass" kind of WIP :)14:07
rloohi and good morning everyone, dtantsur, deray14:07
dtantsurand g'morning rloo :)14:07
rloodtantsur: ha ha. after unit tests pass, I'll look.14:07
derayrloo, o/14:07
dtantsurok, no problem at all14:07
*** intr1nsic has joined #openstack-ironic14:09
rloolucasagomes, jroll: what more needs to be done wrt https://bugs.launchpad.net/ironic/+bug/1524041?14:10
openstackLaunchpad bug 1524041 in oslo.messaging "oslo.messaging 3.1.0 breaking ironic stable releases unit tests" [High,In progress] - Assigned to Jim Rollenhagen (jim-rollenhagen)14:10
jrolldtantsur: yeah, it's on my todo list for today14:10
dtantsurawesome14:10
jrollrloo: oh, that's done14:11
rloojroll: thx, i updated it.14:12
jrollty14:13
*** harshs has quit IRC14:13
jrolljust gonna leave this here https://pbs.twimg.com/media/CYDtDRQWsAARkfW.jpg:large14:13
dtantsurLOOOL14:14
rlooI don't expect to see that in our channel :D14:15
jroll\m/ ʕ•͡ᴥ•ʔ \m/14:16
jroll^ bear metal14:16
* dtantsur saves :D14:16
mgould:-)14:16
*** intr1nsic has quit IRC14:18
rlooI added that to pixie boots on our main page14:19
* mgould tries to interpret \m/ >_< \m/ as valid Perl14:20
mgouldI don't *think* it can be done, but I'm not sure :-)14:20
dtantsurI think it can be valid in Scala14:20
dtantsurtheir operator overloading is pretty cool14:20
lucasagomesjroll, hah nice14:21
jrollrfe docs, anyone? https://review.openstack.org/#/c/259055/14:21
lucasagomesrloo, checking14:21
lucasagomesyeah seems done14:21
rloojroll: I started to look at that, but was interrupted by a rockin' bear14:21
*** [1]cdearborn has joined #openstack-ironic14:22
jrollheh14:22
*** ndipanov has joined #openstack-ironic14:26
openstackgerritMoshe Levi proposed openstack/ironic: Adding InfiniBand Support  https://review.openstack.org/26426314:32
*** lucasagomes is now known as lucas-hungry14:32
*** daemontool_ has joined #openstack-ironic14:33
*** daemontool has quit IRC14:34
*** cdearborn has quit IRC14:37
NobodyCamgood morning Ironicers14:37
dtantsurmorning NobodyCam14:38
openstackgerritZhenguo Niu proposed openstack/ironic: Add a bitmap to manage console ports  https://review.openstack.org/25778914:38
NobodyCammorning dtantsur, jroll lucas-hungry, rloo, vdrok, sambetts jlvillal, JayF, mgould, TheJulia, and all others not directly listed here14:40
sambettso/ NobodyCam14:40
rloogood morning NobodyCam14:40
NobodyCam:)14:42
jrollheya NobodyCam14:43
NobodyCam:)14:45
mgouldNobodyCam, morning!14:45
*** jcoufal_ has joined #openstack-ironic14:50
*** jcoufal has quit IRC14:53
*** vinay50muddu has quit IRC14:54
jlvillalGood morning NobodyCam sambetts rloo jroll mgould lucas-hungry dtantsur deray and everyone else14:54
derayjlvillal, o/14:54
* jlvillal is lazy and only looks at current page of IRC log for names :)14:55
rloomorning jlvillal!14:55
jlvillal:)14:56
TheJuliagood morning jlvillal14:56
jlvillalTheJulia, good morning. I missed you in my log reading!14:56
* deray filling up school admission form for my kid and gettign asked questions like "why this school", "describe your son (in 100 words)" etc14:56
*** piet has joined #openstack-ironic14:57
deraywill log off now.. time for dinner. bye!14:57
*** ionutbalutoiu has quit IRC14:57
*** baoli has quit IRC14:58
jlvillalderay, ciao!14:58
*** ionutbalutoiu has joined #openstack-ironic14:58
*** dprince has quit IRC14:58
*** dprince has joined #openstack-ironic14:59
dtantsurmorning jlvillal14:59
NobodyCam:)15:00
*** deray has quit IRC15:00
jlvillal:)15:01
*** baoli has joined #openstack-ironic15:01
dtantsurjroll, if releasing IPA liberty (not sure if you plan), please include the just merged commit15:04
*** teju has quit IRC15:09
*** pch has quit IRC15:12
*** absubram has quit IRC15:12
dtantsurwow, dsvm jobs are passing on the futurist patch! time to fix unit tests :)15:13
*** jaypipes has joined #openstack-ironic15:15
*** Nisha has joined #openstack-ironic15:17
*** mtanino has joined #openstack-ironic15:19
*** sirmonkey has quit IRC15:20
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Switch to Futurist library for periodic tasks  https://review.openstack.org/26472015:27
jrolldtantsur: will do15:28
devanandag'morning, all15:30
NobodyCamgood moorning devananda15:30
jlvillalgood morning devananda15:32
*** dprince has quit IRC15:32
*** dprince_ has joined #openstack-ironic15:32
jlvillalJayF, lekha +2 and +A for mimic in global requirements :)15:33
openstackgerritJulia Kreger proposed openstack/bifrost: DO NOT MERGE: test commit to debug CI failure  https://review.openstack.org/26483515:33
*** lucas-hungry is now known as lucasagomes15:34
lucasagomesNobodyCam, derekh jlvillal morning15:34
jlvillal:)15:34
*** dims_ has joined #openstack-ironic15:35
NobodyCammorning lucasagomes15:35
NobodyCam:)15:35
*** dims has quit IRC15:36
derekhlucasagomes: howdy15:38
lucasagomesderekh, o/ hi there!15:39
*** garthb has joined #openstack-ironic15:40
*** cdearborn has joined #openstack-ironic15:44
rloojroll, lucasagomes, dtantsur, devananda, jlvillal and everyone that might have an opinion. Now that we approved the OSC spec, should we ask for the OSC CLI + ironic CLI in specs?15:44
*** moshele has quit IRC15:45
rlooI am referring to this OSC spec: http://specs.openstack.org/openstack/ironic-specs/specs/approved/ironicclient-osc-plugin.html15:45
dtantsursounds reasonable..15:45
openstackgerritJulia Kreger proposed openstack/bifrost: DO NOT MERGE: test commit to debug CI failure  https://review.openstack.org/26483515:48
lucasagomesrloo, do we intent to keep 2 CLIs in the future?15:50
rloolucasagomes: i thought that openstackclient was going to take over the world.15:50
rloolucasagomes: but I don't know for sure.15:50
*** electrofelix has quit IRC15:50
lucasagomesI wish we could have only 1, if OS wants to make OSC the de-facto CLI I think it's resonable to ask it in the spec15:50
rloolucasagomes: i think it makes sense to deprecate the ironic CLI for the OSC one. but until that happens, we'll need both.15:51
*** dprince_ has quit IRC15:51
*** dprince has joined #openstack-ironic15:51
*** electrofelix has joined #openstack-ironic15:51
lucasagomesrloo, yeah I agree with that, but before we ask for both let's make sure we are for sure going towards deprecating the ironic CLI15:51
rloolucasagomes: I suppose the earliest we can decide/mention a deprecation period, is when we actually have OSC equivalents for ironic CLI.15:52
* lucasagomes don't want to have to maintain 2 CLIs15:52
lucasagomesrloo, ++15:52
rloolucasagomes: we cannot deprecate if we don't have OSC up-to-date, because why would someone use osc otherwise.15:52
lucasagomesyeah, totally we need pair functionality before depracting the ironic CLI15:53
lucasagomesbut I meant more about the idea of OSC being the CLI in the future15:53
lucasagomesthe spec says "The OpenStackClient is becoming the defacto cli client for OpenStack" but there's no references for it15:54
* lucasagomes looks for references15:54
rloolucasagomes: I think OSC should be the client since it makes more sense for our (openstack) users15:54
lucasagomesrloo, I agree15:54
rloolucasagomes: let's see what our PTL says but I think that approving the OSC spec means that we've committed to supporting it (regardless of whether we deprecate ironicclient) and 'supporting' it means it should be complete.15:57
*** sacharya has joined #openstack-ironic15:58
rloolucasagomes: I think the answer to my original question is yes then, we should probably update our spec template to indicate that osc cli needs to be mentioned too.15:58
lucasagomesrloo, yeah that's a good point. We can add two sub sections under Client (CLI) impact15:59
lucasagomes(or two sections one for ironic and another for osc)15:59
*** [1]cdearborn has quit IRC16:00
jrollrloo: lucasagomes: I agree with what y'all have said here16:00
jrollthere may be some prior art on the deprecation thing, I think I've seen something16:00
*** sacharya has quit IRC16:02
rloojroll, lucasagomes, dtantsur: ok, I'll update the spec template now...16:02
lucasagomesrloo, ty!16:02
dtantsurrloo, lucasagomes, jroll, keystone has deprecated their CLI tool in favor of OSC16:03
rloolucasagomes: so many things to think about/do to get a feature in!16:03
jrolldtantsur: oh neat16:03
NobodyCamoh wow16:04
rloojroll, dtantsur: so one difference is that keystone has real OSC or whatever it is called. ironic is a plugin to osc. our osc plugin is in python-ironicclient package.16:07
rlooi don't even know how to use osc.16:07
dtantsuralso FYU inspector client never had a non-OSC CLI16:08
dtantsur* FYI16:08
jrollrloo: that's fine, we'd deprecate the CLI in ironicclient, not the entire client16:09
jrollrloo: pip install openstackclient python-ironicclient && openstack baremetal node list, AIUI16:09
rloodtantsur: you're so in vogue with the latest and greatest :)16:09
dtantsur:)16:09
lucasagomesyeah we can keep the libraries in Ironic16:09
lucasagomesand OSC just use then16:09
lucasagomesthem*16:09
dtantsurwell, python-keystoneclient also still exists as a python library16:09
dtantsurso no big deal here16:10
lucasagomes++16:10
lucasagomesat the end we will end up killing the shell.py module from ironicclient16:10
lucasagomesin favor of the OSC plugin there16:10
*** raddaoui has joined #openstack-ironic16:10
rlooif we don't have instructions in our dev guide for using osc, someone should add it!16:11
NobodyCamwe also have a strong standalone use case16:12
cinerama++16:12
rlooNobodyCam, cinerama: for the standalone use case, does it matter if you use 'ironic xx' or 'openstack baremetal xx'?16:13
*** jaybeale has joined #openstack-ironic16:14
jrollit shouldn't16:14
NobodyCamjust revamp existing tooling16:14
rlooNobodyCam: OK. so the same issue as everyone else that is currently using 'ironic ...'.16:15
jrollright, that's why we have deprecation periods16:16
lucasagomesyeah we can keep the ironiccli for idk 2 cycles of deprecation if needed16:16
openstackgerritJulia Kreger proposed openstack/bifrost: DO NOT MERGE: test commit to debug CI failure  https://review.openstack.org/26483516:16
lucasagomesit should be more than enough time for migration16:16
*** absubram has joined #openstack-ironic16:17
rloojust wanted to make sure there was no problem using openstackclient with the extra 'baggage'? of non-baremetal commands included with that16:17
TheJuliabaggage of all of the other client libraries... :\16:18
TheJuliaand their dependencies16:18
TheJuliaalthough, osc uses os-client-config, so auth parameters should be far more flexible for both use cases in terms of using a cli16:19
jrolloooo. https://github.com/openstack/python-openstackclient/blob/master/requirements.txt16:19
rlooTheJulia: so you aren't too thrilled? :) I guess the deprecation, if any, needs to be discussed.16:19
TheJuliarloo: not exactly thrilled, I like the consistency desire, I like the fact that it likely will force conformity across the entire openstack project, just as long as we remember that we have API users, not just people typing commands in a shell16:20
rloojroll: sigh. maybe they can make them all plugins or something16:21
TheJuliajroll: I think https://github.com/openstack-infra/shade/blob/master/requirements.txt is worse ;)16:21
jrollTheJulia: "we remember that we have API users, not just people typing commands in a shell" +1000 but not sure if you're saying we'll forget ironicclient or forget the people not using a client at all16:21
jrollheh16:21
TheJuliawe mentally seem to focus on people typing commands, just an observation16:22
TheJulianot a bad thing though, its a nice consistant surface16:22
TheJuliahonestly, the duality is going to cause confusion until things go back to one client, I think its unavoidable tbh16:22
jrollyeah, I don't especially like that16:22
jrollfwiw we have code downstream that uses the api directly16:23
TheJuliaI almost started doing direct rest calls in shade... ''<---- this close16:23
jrollheh16:23
jrollwe have a node.js app that uses the api16:24
jrollafaik there's no node.js client yet, but lucasagomes wanted to build one16:24
jroll;D16:24
lucasagomeshah doing in cobol first16:24
TheJulialol16:24
jroll:P16:24
jrollstepping way for a bit16:27
*** harshs has joined #openstack-ironic16:28
openstackgerritJulia Kreger proposed openstack/bifrost: DO NOT MERGE: test commit to debug CI failure  https://review.openstack.org/26483516:32
*** harshs has quit IRC16:33
*** ifarkas has quit IRC16:33
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: [WIP] Switch to IPA as a primary ramdisk  https://review.openstack.org/26373016:33
*** sacharya has joined #openstack-ironic16:34
*** sacharya has quit IRC16:34
*** sacharya has joined #openstack-ironic16:35
*** piet has quit IRC16:38
openstackgerritRuby Loo proposed openstack/ironic-specs: Update template to include openstackclient  https://review.openstack.org/26485716:38
*** garthb has quit IRC16:39
*** Sukhdev has joined #openstack-ironic16:41
*** jaybeale has quit IRC16:41
*** piet has joined #openstack-ironic16:43
*** thiagop has quit IRC16:44
*** thiagop has joined #openstack-ironic16:45
*** mgoddard__ has joined #openstack-ironic16:46
*** baoli has quit IRC16:46
*** mgoddard_ has quit IRC16:49
mgouldwhat's the relationship between python-ironicclient and python-tripleoclient?16:49
TheJuliawell, python-ironicclient drives the ironic service which provisions baremetal16:51
mgouldOK16:51
lucasagomesrloo, lgtm, but I wonder about openstack vs OpenStack16:51
TheJuliaI'm guessing the tripleoclient is more oriented for managing tripleo based clouds16:51
mgouldso python-tripleoclient loads python-ironicclient to do baremetally things?16:52
TheJulialooks like it does16:52
mgouldcool, thanks16:52
*** cdearborn has quit IRC16:52
TheJuliahttps://github.com/openstack/python-tripleoclient/blob/master/requirements.txt#L1316:52
rloolucasagomes: Oh. that was cuz the command is 'openstack baremetal'.16:52
mgouldah, of course16:52
rloolucasagomes: I could put it in quotes?16:53
*** cdearborn has joined #openstack-ironic16:53
lucasagomesrloo, right, but the title of the section is "openstack client" (w/o quotes)16:53
rloolucasagomes: or 'OpenStackclient plugin CLI' but I don't think people will understand it16:53
lucasagomesopenstack baremetal*16:53
rloolucasagomes: yeah, that's what I mean. Should I put the title in quotes?16:53
lucasagomesrloo, I've no idea... (trademarks are hard to get...)16:54
lucasagomeslemme investigate16:54
TheJuliamgould: looks like baremetal inventory loading and introspecting stuffs16:54
rloolucasagomes: if it is in quotes, it should be ok. cuz the command itself is 'openstack' not 'OpenStack'16:54
lucasagomesyeah16:54
lucasagomesor OpenStackClient16:54
lucasagomes"OpenStackClient (aka OSC) is a command-line client for OpenStack that" ref http://docs.openstack.org/developer/python-openstackclient/16:55
rloolucasagomes: I can do 'OpenStackClient', but if I do that, then the 'ironic CLI' isn't consistent.16:55
rloolucasagomes: and again, I don't know if people will understand 'OpenStackClient' as easily as seeing 'openstack baremetal'.16:55
lucasagomesrloo, oh... well let's leave as is, if someone complain they can put a patch up for fixing it16:56
* lucasagomes tries to find about trademarks anyway16:56
rloolucasagomes: ha ha. i'll just change the titles to have quotes.16:56
lucasagomesrloo, ack16:56
lucasagomesyeah or quote it to avoid problems16:56
*** kalpase has left #openstack-ironic16:56
*** dims has joined #openstack-ironic16:59
*** mgoddard__ has quit IRC17:00
*** mgoddard has joined #openstack-ironic17:00
*** yolanda has quit IRC17:00
openstackgerritRuby Loo proposed openstack/ironic-specs: Update template to include openstackclient  https://review.openstack.org/26485717:01
*** dims_ has quit IRC17:02
*** ukalifon1 has quit IRC17:03
*** daemontool_ has quit IRC17:06
*** baoli has joined #openstack-ironic17:07
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: Change polling give up policy for OneView client  https://review.openstack.org/26487417:09
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: Change polling give up policy for OneView client  https://review.openstack.org/26487417:11
*** Sukhdev has quit IRC17:12
rloolucasagomes: I have a dumb question wrt https://review.openstack.org/#/c/264164/17:12
openstackgerritThiago Paiva Brito proposed openstack/ironic: Change polling give up policy for OneView driver  https://review.openstack.org/26487617:13
rloolucasagomes: if we don't care and you don't plan to address dmitry's nits, I'll approve it17:13
lucasagomesrloo, oh it's a totally valid code...17:13
lucasagomesvalid question*17:13
*** Sukhdev has joined #openstack-ironic17:13
*** trown is now known as trown|lunch17:13
lucasagomesrloo, lemme see if I can find something about deprecating variables in devstack17:14
rloolucasagomes: ok. or maybe if someone specified it, use it and assume the directory exists?17:14
openstackgerritJim Rollenhagen proposed openstack/ironic: Document the process of proposing new features  https://review.openstack.org/25905517:15
*** davideagnello has joined #openstack-ironic17:15
dtantsursee you tomorrow!17:15
NobodyCamnight dtantsur17:15
*** dtantsur is now known as dtantsur|afk17:15
lucasagomesrloo, yeah if someone specified it before the directory had to exist or the devstack run would fail17:16
* lucasagomes feels a bit guilty when introduced that before because it was rushy and we wanted to find out what was going on in our gate17:16
lucasagomesso I only tested on ubuntu cause that's what we run in gate :-(17:16
rloolucasagomes: oh. how long have we had that qemu variable?17:16
lucasagomesidk 2 weeks?17:17
lucasagomeswas before xmas when we had a lot of gate problems17:17
*** slagle has joined #openstack-ironic17:17
rloolucasagomes: i think it is fine to change it then.17:17
lucasagomesyeah17:17
*** praneshp has joined #openstack-ironic17:17
rloolucasagomes: although if you consider it code, you might not think it is fine. didn't we have a discussion about that before? :)17:17
lucasagomeswe did, we should actually deprecate (if devstack has a guideline for that)17:18
* lucasagomes is trying to find it17:18
jrollthe right thing to do is deprecate it17:19
jrollit's probably fine not to17:19
*** piet has quit IRC17:19
*** raddaoui has quit IRC17:19
*** piet has joined #openstack-ironic17:19
jrollI'd probably jfdi with a quick note to openstack-dev17:20
Nishai have one silly ques on reno (release notes)17:20
NishaEverything wiped offin vacation :(17:20
jrollrealistically nobody is using that, it only makes sense with agent_ssh17:20
jrollNisha: what's your question?17:20
Nishahow do we update a release note in the patch17:20
Nishaits already existing17:21
openstackgerritMerged openstack/ironic-specs: Update template to include openstackclient  https://review.openstack.org/26485717:21
Nishai have to address the comment on the renotes17:21
*** yolanda has joined #openstack-ironic17:21
*** raddaoui has joined #openstack-ironic17:21
Nishabut using vi, gate fails for release notes17:21
NobodyCamNisha: what patch?17:21
Nishahttps://review.openstack.org/#/c/251277/17:22
NishaI updated the release notes using vi, and gate started failing17:22
rlooNisha: I doubt it had anything to do with vi, vs what your update was...17:23
Nisharloo gate for release notes only fail....you mean the contents are added wrongly?17:23
jrollNisha: http://docs.openstack.org/developer/reno/usage.html#formatting17:24
jrollit's a formatting issue17:24
rlooNisha: that would be my guess17:24
*** garthb has joined #openstack-ironic17:24
Nishajroll, thanks17:24
jrollnp17:24
NobodyCamya: http://logs.openstack.org/77/251277/5/check/gate-ironic-releasenotes/f2fc3a2/console.html#_2016-01-07_09_29_34_42817:24
lucasagomesrloo, jroll can't find anything relevant about deprecation of those variables... I can come up with a way to deprecate that if needed tho17:27
jrolllucasagomes: don't worry about it, just email the list17:27
lucasagomesyeah17:27
lucasagomeswill ask at -qa too17:27
jrollwith a link to the patch17:28
jrolland then people can -1 if they're using it17:28
jrollI wouldn't worry too much though17:28
lucasagomesack, ty17:28
rloolucasagomes: yeah, don't worry about it. like jroll sez. I would have just +A'd but jroll is more cautious :)17:30
jrollI just haven't reviewed yet :P17:30
lucasagomesrloo, asking on -qa if no asnwer I will put an email to the ML17:30
rloojroll: ha ha.17:30
lucasagomesand we can wait for few days17:30
rloojroll: question wrt rfe process. Here's an example: https://bugs.launchpad.net/ironic/+bug/152631917:31
openstackLaunchpad bug 1526319 in Ironic "[RFE] Add api support to query nodes by name filters" [Wishlist,In progress] - Assigned to Haomeng,Wang (whaom)17:31
lucasagomesjroll, lol, fair enuff17:31
openstackgerritNisha Agarwal proposed openstack/ironic: Add support for providing custom Swift endpoint URLs  https://review.openstack.org/25127717:31
jrolllucasagomes: I'd rather just put an email out "heads up this changed"17:31
rloojroll: so it seems fine to me. although the description isn't the greatest, what kind of regex are we talking about, what about the CLI. So if they added more info in the bug, it doesn't need a spec, right?17:31
lucasagomesjroll, yeah or that, lemme do it17:31
jrolllucasagomes: or if you don't want to change the variable name, you could leave it as the file name and use readlink or something to find the dir17:32
jrollbut then again if 'qemu' is the only file name that works, might be best to keep that hardcoded17:33
JayFrloo: re: why does IPA allow things to have the same support level; that's how the API was designed more or less ... we have GENERIC, MAINLINE, and SERVICE_PROVIDER (1,2,3) that are provided in IPA17:33
jrolllucasagomes: +A'd17:34
JayFMaking it exclusive would preclude being able to have pluggable generic hardware managers17:34
JayFi.e. HP ships proliantutils with a mainline level hardware manager17:34
lucasagomesjroll, yeah qemu is the only name that works http://libvirt.org/hooks.html#names17:34
*** Sukhdev has quit IRC17:34
jrollright17:35
rlooJayF: OK. Just thought we could simplify things :)17:35
JayFrloo: I mean, there are def places I wish we had designed diff w/r/t that after dealing with our huge mix of hardware managers downstream (we have probably 10 different ones, and up to 6 mixing for a node)17:35
JayFrloo: but it works reasonably well once all the patches I'm working on get in (I picked up joshs' old dedupe steps patch after writing a downstream test and finding a bunch of edge cases unhandled)17:36
rlooJayF: it is just that the code will always have to deal with breaking ties for managers with the same support level.17:36
sambettsNight all o/17:37
NobodyCamnight sambetts17:37
sambettso/ NobodyCam17:37
*** sambetts has quit IRC17:37
jrollrloo: I think that RFE needs a spec, honestly. it's an API change (which we should always tread carefully with), and I'm hesitant on that feature as is17:37
jrollmostly this https://review.openstack.org/#/c/250112/14/ironic/db/sqlalchemy/api.py17:38
JayFrloo: yep, and I think I figured out that in some cases, the behavior while consistent is not deterministic17:38
rloojroll: oh, ok. devil is in the details.17:38
JayFrloo: i.e. if you dispatch_to_managers() and have 2 hwm "tied" at support level, right now the first one loaded would be the first one run17:38
jrollrloo: yep17:39
JayFrloo: rather than something more ... predictable17:39
rlooJayF: yeah, see. ties are bad!17:39
jrollrloo: tempted to say "all api changes need a spec"17:39
JayFbut in order to prevent ties, you have to enforce uniqueness17:39
JayFwhich is probably even harder when talking about plugins17:39
JayFesp since hwms are dynamically loaded based on hw on th enode17:39
openstackgerritMerged openstack/python-ironicclient: Remove openstack-common.conf  https://review.openstack.org/26464817:41
openstackgerritMerged openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/26442517:41
rloojroll: it seems to me that someone had mentioned that all api changes need a spec, but not sure who/where.17:44
*** jcoufal_ has quit IRC17:44
jrollrloo: nova does, other than that idk17:44
rloojroll: oh, maybe that was where i saw it.17:44
*** deray has joined #openstack-ironic17:45
*** jistr has quit IRC17:45
*** mkovacik has quit IRC17:46
JayFlekha: mimic -> global-requirements is on it's way to a merge, was approved earlier17:49
lekhaJayF: yay!17:49
lekhaI will rebase the ironic python client PR once its merged17:50
*** athomas has quit IRC17:51
*** yarkot has joined #openstack-ironic17:53
openstackgerritMerged openstack/ironic: Change default IRONIC_VM_SPECS_RAM to 1024  https://review.openstack.org/26450617:55
*** rebase has joined #openstack-ironic17:55
*** praneshp has quit IRC18:01
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/26442418:02
* mgould -> home; good night!18:08
*** jaybeale has joined #openstack-ironic18:08
*** mgould has quit IRC18:08
*** penick has quit IRC18:08
*** trown|lunch is now known as trown18:08
NobodyCamnight mgould18:09
*** derekh has quit IRC18:11
*** toure has quit IRC18:14
*** garthb_ has joined #openstack-ironic18:14
*** garthb has quit IRC18:15
*** trown has quit IRC18:15
*** trown|brb has joined #openstack-ironic18:16
*** raddaoui has quit IRC18:16
*** toure has joined #openstack-ironic18:16
*** trown|brb is now known as trown18:21
*** ndipanov has quit IRC18:22
*** e0ne has joined #openstack-ironic18:23
* lucasagomes sent the heads up email18:25
NobodyCamTy lucasagomes :)18:28
openstackgerritMerged openstack/ironic: Devstack: Make sure libvirt's hooks directory exists  https://review.openstack.org/26416418:30
*** praneshp has joined #openstack-ironic18:33
*** harlowja has quit IRC18:35
*** harlowja has joined #openstack-ironic18:36
openstackgerritJohn L. Villalovos proposed openstack/ironic-lib: WIP: Testing  https://review.openstack.org/26457718:47
*** ionutbalutoiu has quit IRC18:56
*** electrofelix has quit IRC18:58
*** moshele has joined #openstack-ironic19:04
*** rebase has quit IRC19:06
*** ijw has joined #openstack-ironic19:07
*** e0ne has quit IRC19:08
*** moshele has quit IRC19:10
jrollJayF: lekha: you may want to be in on this thread folks asked me to start http://lists.openstack.org/pipermail/openstack-dev/2016-January/083510.html19:10
*** Sukhdev_ has joined #openstack-ironic19:13
*** Sukhdev_ has quit IRC19:14
*** Sukhdev_ has joined #openstack-ironic19:14
lekhajroll: thanks jroll. Just to be clear, mimic uses twisted plugin architecture, but for the most part (over 95%) of the code is just good old python. But if things are failing cause of twisted itself, I agree its a whole other ball game.19:14
*** baoli has quit IRC19:14
jrolllekha: yeah, I know19:15
*** ijw has quit IRC19:17
*** penick has joined #openstack-ironic19:21
*** Nisha has quit IRC19:28
*** harshs has joined #openstack-ironic19:29
*** alexpilotti has joined #openstack-ironic19:29
*** garthb__ has joined #openstack-ironic19:31
*** yarkot has quit IRC19:31
*** yarkot has joined #openstack-ironic19:31
*** tangchen__ has joined #openstack-ironic19:32
*** dims_ has joined #openstack-ironic19:33
*** chihhsin_ has joined #openstack-ironic19:33
*** marios_ has joined #openstack-ironic19:34
*** dims has quit IRC19:34
*** yonglihe_ has joined #openstack-ironic19:35
*** MattMan_ has joined #openstack-ironic19:35
*** bradjones_ has joined #openstack-ironic19:37
*** bradjones_ has quit IRC19:37
*** bradjones_ has joined #openstack-ironic19:37
*** garthb_ has quit IRC19:38
*** MattMan has quit IRC19:38
*** bradjones has quit IRC19:38
*** alexpilo_ has quit IRC19:38
*** kbyrne has quit IRC19:38
*** yuriyz has quit IRC19:38
*** marios has quit IRC19:38
*** tangchen_ has quit IRC19:38
*** yonglihe has quit IRC19:38
*** chihhsin has quit IRC19:38
*** vdrok- has quit IRC19:38
*** bradjones_ is now known as bradjones19:38
*** yonglihe_ is now known as yonglihe19:39
*** ijw has joined #openstack-ironic19:40
*** ChubYann has joined #openstack-ironic19:40
*** yuriyz has joined #openstack-ironic19:42
*** kbyrne has joined #openstack-ironic19:42
*** gabriel has joined #openstack-ironic19:43
*** harshs has quit IRC19:44
rloojroll, devananda: where are we at with the node claims/filter API? https://review.openstack.org/#/c/204641/19:46
*** vdrok- has joined #openstack-ironic19:46
*** harshs has joined #openstack-ironic19:46
jrollrloo: we kind of decided to focus on getting the other priorities done19:46
jrollI don't think we can finish that before nova FF19:46
jrollI do need to push a new iteration of that spec but it's low prio on my todo list19:47
rloojroll: Oh. I agree, we can't finish/get the nova part done before nova FF. So yeah, I can see it being low priority.19:47
rloojroll: the problem? being that I'm trying to get downstream folks to do things the way the upstream is going to flow, and it isn't clear what the flow is.19:48
*** Sukhdev_ has quit IRC19:48
lucasagomesfolks, I will call it a day19:48
rloonight lucasagomes19:48
jrollrloo: the plan laid out has not changed, does that help? :)19:48
lucasagomeshave a great night every one!19:48
jrollnight lucasagomes19:48
lucasagomessee y'all!19:49
*** lucasagomes is now known as lucas-dinner19:49
rloojroll: they/we want to separate the claims from the filtering. which makes sense to me, but it isn't explicitly mentioned in the spec.19:49
jrollrloo: there's some details needed yet, but the overall architecture is there19:49
jrollum19:49
jrolloh, we were breaking the filter api to a separate spec19:50
* jroll looks around19:50
jroller no19:50
jrollline 14719:50
rloojroll: so a filtering that returns nodes but doesn't claim, and a claim API to add a claim given a list of nodes.19:50
jrolllines 147-157 is the filter api19:50
jrollI don't see the use in making claim only accept a list of nodes19:51
rloojroll: that's the filtering w/o claims part.19:51
jrollso19:51
rloojroll: if you/we provide an API that filters -> list of nodes, why wouldn't you allow a claim to take a list of nodes.19:51
jroll1) filter api should be a thing, without claims. that's lines 147-15719:51
jroll2) claims api should accept node uuid(s), but should also accept other properties similar to the filter api19:52
jrolldoes that make sense19:52
jrollI don't see the use case in restricting the claims API to *only* node uuid(s)19:52
rloojroll: yup, that's what I wanted to see/hear. Sorry, I didn't mean for it to *only* take node uuids, that wouldn't make sense :)19:52
jrollrloo: okay cool19:53
rloojroll: ok, i'll add a comment to the spec with a link to this discussion :)19:53
JayFWhat's the right way to multiline something if it's "var = [such a long string comprehension it barely fits on a line]"19:53
jrollrloo: I just added a +1 to that comment19:53
*** ijw has quit IRC19:53
JayFvar = \n+4spaces [] gives a syntax error19:53
jrollJayF: var = [thing for thing\n in things]19:54
rloothx jroll19:54
jrollI line up the second line one space past the [ but I'm not sure if pep8 flags those at all19:54
jrollnp rloo :)19:54
JayFjroll: the predicate is too long to wrap it to there :)19:54
JayFbut I'll do it with a shorter indent19:55
JayFand then that'll pass syntax and pep8 can yell if it wants19:55
JayFlol19:55
jrollJayF: I mean, wrap anywhere within the []19:55
jrollJayF: or var = (\n[...]) or var = [\n...]19:55
JayFwell that's easy then19:55
JayFgotcha19:55
jrolljust needs some sort of bracket for python19:55
jrollor \19:55
jrollbut \ is ew19:55
JayFgotcha19:56
JayFyeah I forget I can use () in python even though you almost never need it19:56
JayFbut wrapping it here looks better so works for me19:56
rlooi think () or \. I don't think we've been using []19:56
JayFrloo: it's a string comprehention :) trying to replace the filter() as you suggested in that patch19:56
* JayF about to see if it passes tests19:56
rlooJayF: Oh. blame it on me then :)19:57
JayFhaha19:57
JayFwhy did we use filter? because we didn'tt hink of doing it a better way19:57
JayFthat is the honest answer to that, lol19:57
JayFand your suggestion passes tests \o/19:57
JayFand woudl've saved us 45m at WTF'ing at py3 differences in filter()19:57
rlooJayF: ok, i thought i was missing something.19:58
rlooJayF: now you know for next time :)19:58
*** dims_ has quit IRC20:00
*** meghal has joined #openstack-ironic20:00
*** harshs has quit IRC20:04
*** chopmann has joined #openstack-ironic20:04
*** chopmann is now known as sirmonkey20:04
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Allow hardware managers to override clean step priority  https://review.openstack.org/18996920:07
*** dhellmann has quit IRC20:11
*** dims has joined #openstack-ironic20:12
*** dims has quit IRC20:19
*** dims has joined #openstack-ironic20:22
*** openstackgerrit has quit IRC20:23
*** dims has quit IRC20:23
*** MattMan_ has quit IRC20:24
*** MattMan has joined #openstack-ironic20:24
*** openstackgerrit has joined #openstack-ironic20:25
*** deray has quit IRC20:26
rlooJayF: did you see this? (I'm guessing not): https://review.openstack.org/#/c/169504/20:29
JayFI didn't see it until my version was up, no20:29
JayFbut it's based on the same downstream patch20:29
jrolllol20:30
JayFI just was two patchsets into the new one before I realized there was an old one20:30
rlooJayF: so I'll abandon it then :)20:30
JayFlol20:30
JayFyeah I was going to do the same but I don't have core on ironic20:30
JayFrloo: that review is an interesting/fun one, espeically around naming. Sorta funny from my perspective b/c until I saw that downstream patch 2wks ago I thought this was upstream already, lol20:30
JayFrloo: including references in docstrings to reboot_requested even though it wasn't implemented upstream, hooray20:31
* JayF not okay with downstream patches and trying to work them out20:31
rlooJayF: yeah. we need to be faster at getting things IN and/or abandoning them.20:31
rlooJayF: otherwise, we are spending extra time (submitter, reviewers) that we shouldn't need to do.20:32
JayFyep, I 100% agree20:32
JayFand add to that downstream time spent rebasing stuff and it gets even more crazy20:32
rlooJayF: oh, I don't care about your downstream times :)20:33
JayFlike I don't think we'll ever be able to run OnMetal on Ironic master, just because there's always going to be a new thing or whatever ... but stuff like reboot_requested, and the clean step ordering ... it's just absolutely nuts20:33
JayFrloo: I'm just making the observation that the time incentive does work both ways20:33
*** dprince has quit IRC20:33
rlooJayF: yeah, I know. I'm just joking.20:33
JayFrloo: so when "we" (talking specifically about OM team) don't upstream stuff we hurt ourselves too20:33
rlooJayF: we have similar issues at Yahoo20:34
*** Marga_ has joined #openstack-ironic20:35
*** Marga_ has quit IRC20:35
*** Marga_ has joined #openstack-ironic20:35
rlooJayF: quick question. Are you using manual cleaning downstream? And what about the GET /nodes/<uuid>/cleaning/steps API (to get list of all clean steps)?20:37
JayFwe don't use manual cleaning downstream at all, and it's unlikely we will20:37
rlooJayF: I have been dragging my feet about those; Josh has some patches up and I was wondering if they worked.20:37
JayFthe ironic api to get clean steps, I don't think we have that downstream either20:38
rlooJayF: ok. that's good to know.20:38
JayFThat API call is less helpful to us generally, because we have homogenous hardware20:38
JayFin fact that new test I added in deduplicate_steps, where it merges all the managers and verifies steps, we have one test like that for each hardware config we use20:38
JayFso we have a place, test-verified, to see what all steps would run on a given node anyway20:38
*** dims has joined #openstack-ironic20:39
rlooJayF: i don't like that it that to implement that API, one needs to query the agent on each node, to get the clean steps. And caching it all seems yucky to me.20:40
JayFI mean, a lot of stuff around the agent gets yukkier than other places because the agent gets to define what happens20:40
JayFsame deal with that reboot_requested patch20:40
JayFIPA is the only thing that'd need to *request* a reboot during cleaning because all other clean steps are directly in the driver, and can reboot themselves at will20:40
rlooJayF: why can't the agent cause itself/node to reboot?20:43
JayFrloo: the agent has no place to persistently store information. If an agent reboots itself as a part of a cleaning step, how does it let the conductor know it's done?20:44
JayFrloo: and when the agent comes back up, how would it know it's post reboot?20:44
JayFIronic has to coordinate anything IPA does that involves a reboot, because Ironic is the only persistent data store IPA has20:44
JayFand when you're talking power actions, and you already have to hit Ironic, why wouldn't Ironic manage that?20:44
TheJuliaJayF: I was having a very deep discussion with NobodyCam a few weeks ago about that and it became very clear during that discussion that we needed something external telling the agent what to do instead of the agent determining it, but that would be a bit of a massive shift20:45
JayFTheJulia: I disagree vehemently. We designed the agent to run the show for specific reasons20:45
jrollJayF: rloo: also that firmware updates often require a hard reboot as opposed to in-band20:45
JayFah that's true, in-band reboots wouldn't even work for all of what we're doing today20:45
*** ionutbalutoiu has joined #openstack-ironic20:46
rloowell, I think it is interesting that TheJulia and NobodyCam think otherwise.20:46
TheJuliaJayF: Oh, there are definitely pluses in that nature as well, just... I think it shifts some of the complexity and tracking elsewhere in the management of the nodes with diverse fleets20:47
TheJuliaI just wish I remember what spurred that entire chain of thought20:47
TheJulia:\20:47
JayFThe use case matters a lot, I think20:47
TheJuliayeah20:47
JayFWithout it it's hard to have a good conversation about it20:48
TheJuliaindeed20:48
NobodyCamhardware managers20:48
*** raddaoui has joined #openstack-ironic20:48
*** piet has quit IRC20:48
NobodyCamhaving multiable HW managers20:48
JayFNobodyCam: I'm running environments that load up to 8 hardware managers at the same time :)20:49
NobodyCam:)20:49
JayFNobodyCam: and am working now on a set of patches to make that work better, b/c I discovered some of the fixup code for those cases were not merged20:49
TheJuliaJayF: awesome20:49
JayFNobodyCam: https://review.openstack.org/#/c/18996920:49
NobodyCamI was tring to remember what start the convo with TheJulia20:49
TheJuliaNobodyCam: it was a firmware upgrade question, but it was a very very diverse hardware situation if memory serves, like wanting different firmware on the same model chassis for some $reason20:50
TheJuliaand then being able to orchustrate that20:50
JayFAah. That's a use case I'd almost have explicitly coded to prevent. In my world, identical hardware runinng non-identical firmwares is a pretty bad failure state20:51
JayFyou can, however, modify what a clean step does based on information in the node, we do it today20:52
TheJuliayeah20:52
JayFso you could do something like node.extra{'load_firmware': 'old'}20:52
JayFb/c we pass the node into clean steps when run20:52
* jroll smells pets20:52
jroll:P20:52
TheJuliabut you'd need that pre-built, which has pluses, and minuses20:52
TheJuliajroll: many petcattle20:53
JayFTheJulia: yeah, but if it's in Ironic, you'd need it prebuilt too, right?20:53
jrollhehe20:53
JayFTheJulia: unless you're going to start sending firmware images over the IPA http api20:53
JayFwhich would be kinda gross20:53
JayFalthough I guess you could ship a url to ipa and have it pull it down, which is not evil20:54
TheJuliaJayF: that is kind of the point where we got to "wow, this is a can of worms"20:54
TheJuliabut yeah20:54
JayFyeah that's kinda why we ended up making it the way we did20:54
JayFkeep all the worms in the agent20:54
JayFand most of them in hardware managers20:54
JayFcontain the hardware nastiness since we can't get rid of it, lol20:54
TheJuliaheh20:55
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: Change polling give up policy for OneView client  https://review.openstack.org/26487420:55
* TheJulia hears a catastrophy of multiple scrambling cats and wonders.... what just happened.20:59
*** raildo is now known as raildo-afk21:04
* TheJulia steps away for a little while21:06
* NobodyCam hopes the catastrophy was not to bad21:07
* jlvillal seems to have been able to simulate devstack-gate locally and run the ironic tempest job. Next step ironic Grenade tempest job :)21:10
TheJulianah, need to go to an appointment21:10
NobodyCamjlvillal: w00t :)21:10
NobodyCam:)21:10
jlvillalNobodyCam: I'll admit it doesn't sound like much. But it has taken me up to now to get that done. Been working on it since Monday, with some meetings getting in the way.21:11
jlvillalI know have a nice automated way to spin up a VM and put the correct stuff on and have it work. And all behind a proxy server, which has caused some pain along the way.21:11
jlvillals/I know/I now/21:12
*** sirmonkey has quit IRC21:14
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: Change polling give up policy for OneView client  https://review.openstack.org/26487421:17
*** rajinir has joined #openstack-ironic21:18
*** piet has joined #openstack-ironic21:18
*** penick has quit IRC21:20
*** harshs has joined #openstack-ironic21:24
*** baoli has joined #openstack-ironic21:25
*** baoli_ has joined #openstack-ironic21:26
*** baoli has quit IRC21:30
*** penick has joined #openstack-ironic21:34
*** ijw has joined #openstack-ironic21:34
*** baoli_ has quit IRC21:36
*** baoli has joined #openstack-ironic21:37
jlvillallifeless: Or anyone else. Do you know if people would be amenable to running OpenStack projects using 'pypy', if it works?  Just trying to find out if there had been any done in the past and people objected to using pypy.21:37
*** Sukhdev has joined #openstack-ironic21:37
jlvillalI'm in a meeting where they are discussing it here :)21:38
lifelessjlvillal: we have some jobs that test pypy compat21:38
lifelessjlvillal: openstack doesn't specify which VM you want to use, but we can only state compat with the ones we test21:38
jlvillallifeless: I have noticed that. Internally they have found some projects can see 2x improvement using pypy21:38
lifelessjlvillal: if they want great pypy support, step up and bugfix things21:39
jlvillallifeless: Great. Thanks.21:39
*** meghal has quit IRC21:39
*** baoli has quit IRC21:40
*** baoli has joined #openstack-ironic21:40
*** baoli has quit IRC21:45
*** baoli has joined #openstack-ironic21:46
*** slagle has quit IRC21:50
*** jaypipes has quit IRC21:50
*** rbudden has quit IRC21:50
*** boris-42 has quit IRC21:50
*** dtantsur|afk has quit IRC21:50
*** slagle has joined #openstack-ironic21:51
*** jaypipes has joined #openstack-ironic21:51
*** rbudden has joined #openstack-ironic21:51
*** boris-42 has joined #openstack-ironic21:51
*** dtantsur|afk has joined #openstack-ironic21:51
*** harshs has quit IRC21:58
*** trown is now known as trown|outttypeww22:04
*** baoli has quit IRC22:05
*** baoli has joined #openstack-ironic22:06
*** meghal has joined #openstack-ironic22:10
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: Change polling give up policy for OneView client  https://review.openstack.org/26487422:10
*** Marga_ has quit IRC22:10
*** meghal has quit IRC22:14
*** penick has quit IRC22:15
*** baoli has quit IRC22:17
* jlvillal is sad that apt-get hangs when switching to grenade ironic job from non-grenade ironic job.22:18
* jlvillal has no clue why apt-get is hanging...22:18
*** baoli has joined #openstack-ironic22:18
jlvillalJust sits here: http://paste.openstack.org/show/483271/22:19
openstackgerritJonathan Provost proposed openstack/ironic: Adjust ipminative._reboot to comply with pyghmi contract  https://review.openstack.org/26499022:20
NobodyCamhumm22:20
*** raddaoui has quit IRC22:21
*** raddaoui has joined #openstack-ironic22:22
openstackgerritMerged openstack/python-oneviewclient: Change polling give up policy for OneView client  https://review.openstack.org/26487422:23
*** absubram has quit IRC22:26
lifelessjlvillal: whats the process state?22:27
lifelessjlvillal: whats its stdout / stderr - could they be full pipes?22:27
lifelessjlvillal: does it have any sockets open?22:27
jlvillallifeless: I see 16829 pts/0    T      0:00 apt-get --option Dpkg::Options::=--  when doing 'ps axww'22:28
jlvillallifeless: I have this: https://review.openstack.org/26435422:28
jlvillallifeless: And the last lines of the grenade log are here: http://paste.openstack.org/show/483273/22:29
*** baoli has quit IRC22:29
*** baoli has joined #openstack-ironic22:30
*** vishwanathj has quit IRC22:30
jlvillallifeless: So I guess the process is stopped. But why?22:31
*** penick has joined #openstack-ironic22:32
*** [1]cdearborn has joined #openstack-ironic22:34
* jlvillal wonders if he should send the CONT signal22:34
* jlvillal discovers the CONT signal didn't seem to do anything.22:36
*** gmmaha has joined #openstack-ironic22:38
thiagopPheww, couple of headaches here to put the CI back online22:40
thiagopdamn power outages...22:40
thiagopcalling it a day, see ya next week folks22:40
*** thiagop has quit IRC22:40
*** baoli has quit IRC22:41
*** baoli has joined #openstack-ironic22:41
*** thrash is now known as thrash|g0ne22:45
*** cdearborn has quit IRC22:48
*** ijw has quit IRC22:50
*** harshs has joined #openstack-ironic22:52
*** baoli has quit IRC22:53
*** baoli has joined #openstack-ironic22:54
*** jaybeale has quit IRC22:57
*** dims_ has joined #openstack-ironic23:00
*** dims has quit IRC23:01
*** sacharya has quit IRC23:02
*** sacharya has joined #openstack-ironic23:03
*** meghal has joined #openstack-ironic23:04
*** mkovacik has joined #openstack-ironic23:05
*** baoli has quit IRC23:05
*** baoli has joined #openstack-ironic23:06
*** sacharya has quit IRC23:07
*** meghal has quit IRC23:09
*** mkovacik has quit IRC23:12
*** ijw has joined #openstack-ironic23:15
*** baoli has quit IRC23:17
*** baoli has joined #openstack-ironic23:18
*** ionutbalutoiu has quit IRC23:19
*** ionutbalutoiu has joined #openstack-ironic23:20
*** [1]cdearborn has quit IRC23:21
*** piet has quit IRC23:28
*** ionutbalutoiu has quit IRC23:28
*** piet has joined #openstack-ironic23:28
*** baoli has quit IRC23:29
*** baoli has joined #openstack-ironic23:30
*** mkovacik has joined #openstack-ironic23:30
*** dims_ has quit IRC23:39
*** baoli has quit IRC23:44
*** dims_ has joined #openstack-ironic23:45
*** sacharya has joined #openstack-ironic23:47
*** dims_ has quit IRC23:50
*** sacharya has quit IRC23:51
*** raddaoui has quit IRC23:52
*** piet has quit IRC23:56
*** garthb__ has quit IRC23:57
*** garthb__ has joined #openstack-ironic23:57
*** meghal has joined #openstack-ironic23:58
*** jaybeale has joined #openstack-ironic23:59

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