Friday, 2016-04-01

*** suro-patz has quit IRC00:04
*** garthb_ has quit IRC00:09
*** mtanino has quit IRC00:21
*** smoriya_ has joined #openstack-ironic00:25
*** gmmaha has quit IRC00:25
*** gmmaha has joined #openstack-ironic00:27
jrollstable IPA stuff landed, devstack changes still under test, will resume tomorrow00:31
jrollnight y'all00:31
*** thrash is now known as thrash|g0ne00:46
*** ChrisAusten has joined #openstack-ironic00:48
*** hoangcx has joined #openstack-ironic00:51
*** jaybeale has quit IRC00:55
openstackgerritMerged openstack/ironic-python-agent: Add docstrings to all functions in Agent standby extension  https://review.openstack.org/28863200:57
*** baoli has joined #openstack-ironic01:00
*** spandhe has quit IRC01:20
*** mtanino has joined #openstack-ironic01:28
clif_hwoot01:30
*** krtaylor has joined #openstack-ironic01:35
*** suro-patz has joined #openstack-ironic01:39
*** rbudden has joined #openstack-ironic01:45
*** suro-patz has quit IRC01:53
*** baoli has quit IRC01:58
*** baoli has joined #openstack-ironic01:58
*** Sukhdev has quit IRC01:59
*** achanda has joined #openstack-ironic02:14
*** Nisha has joined #openstack-ironic02:15
*** baoli has quit IRC02:25
*** rbudden has quit IRC02:34
*** baoli has joined #openstack-ironic02:42
*** sivaramakrishna has joined #openstack-ironic02:42
*** baoli has quit IRC02:45
*** yuanying has quit IRC02:51
*** Nisha_away has joined #openstack-ironic02:59
*** Nisha has quit IRC03:01
*** jjohnson2 has joined #openstack-ironic03:09
*** krotscheck is now known as krotscheck_vaca03:20
openstackgerritTan Lin proposed openstack/ironic: Add nodeinfo_list() to Node Object  https://review.openstack.org/29031303:24
*** spandhe has joined #openstack-ironic03:30
*** spandhe_ has joined #openstack-ironic03:33
*** spandhe has quit IRC03:34
*** spandhe_ is now known as spandhe03:34
*** harshs has joined #openstack-ironic03:38
*** links has joined #openstack-ironic03:41
*** yuanying has joined #openstack-ironic03:49
*** jjohnson2 has quit IRC03:52
*** achanda has quit IRC04:04
*** alexpilotti has quit IRC04:06
*** achanda has joined #openstack-ironic04:08
*** achanda has quit IRC04:13
*** alexpilotti has joined #openstack-ironic04:17
*** harlowja_at_home has joined #openstack-ironic04:19
*** alexpilotti has quit IRC04:21
*** Sukhdev has joined #openstack-ironic04:25
*** ndipanov has quit IRC04:29
*** suro-patz has joined #openstack-ironic04:30
*** vmud213 has joined #openstack-ironic04:34
*** mtreinish has quit IRC04:44
*** suro-patz has quit IRC04:45
*** pradipta has joined #openstack-ironic04:49
*** pradipta has quit IRC04:50
*** hoangcx has quit IRC04:54
*** rcernin has joined #openstack-ironic04:57
*** alexpilotti has joined #openstack-ironic04:59
*** achanda has joined #openstack-ironic05:02
*** stevemar_ has quit IRC05:02
*** stevemar has joined #openstack-ironic05:02
*** alexpilotti has quit IRC05:03
*** achanda has quit IRC05:10
*** vinaym213 has joined #openstack-ironic05:13
*** vmud213 has quit IRC05:16
*** alexpilotti has joined #openstack-ironic05:17
*** harlowja_at_home has quit IRC05:19
*** alexpilotti has quit IRC05:24
*** davideag_ has joined #openstack-ironic05:24
*** ChrisAusten has quit IRC05:25
*** davideagnello has quit IRC05:26
*** suro-patz has joined #openstack-ironic05:31
*** yolanda has quit IRC05:39
*** davideag_ has quit IRC05:42
*** ChubYann has quit IRC05:43
*** harshs_ has joined #openstack-ironic05:49
*** harshs has quit IRC05:51
*** mtreinish has joined #openstack-ironic05:52
*** ifarkas has quit IRC05:54
*** harshs_ has quit IRC05:55
*** harshs has joined #openstack-ironic05:58
*** hoangcx has joined #openstack-ironic06:03
*** harshs has quit IRC06:03
*** spandhe has quit IRC06:04
*** Marga_ has quit IRC06:06
*** spandhe has joined #openstack-ironic06:10
*** Nisha_away has quit IRC06:12
*** yolanda has joined #openstack-ironic06:15
*** spandhe_ has joined #openstack-ironic06:16
*** spandhe has quit IRC06:17
*** spandhe_ is now known as spandhe06:17
*** harshs has joined #openstack-ironic06:17
*** spandhe_ has joined #openstack-ironic06:21
*** spandhe has quit IRC06:21
*** spandhe_ is now known as spandhe06:21
*** suro-patz has quit IRC06:22
*** Sukhdev has quit IRC06:24
*** Nisha_away has joined #openstack-ironic06:25
*** Marga_ has joined #openstack-ironic06:25
*** Marga_ has quit IRC06:29
*** achanda has joined #openstack-ironic06:30
*** Marga_ has joined #openstack-ironic06:38
*** yushiro has joined #openstack-ironic06:39
yushiroHi all,06:41
*** Marga_ has quit IRC06:43
yushiroI'd like to know about cleaning for the ironic node with agent_xxxx driver.06:44
yushiroWhen timeout(clean_callback_timeout in ironic.conf) has occurred for cleaning, provision state has changed into "clean failed".06:48
yushiroHowever, deploy image keeps on running and verifying heartbeat forever.06:49
yushiroI think it should be executed power off and unmounted some virtual media.06:53
*** alexpilotti has joined #openstack-ironic06:56
*** ohamada has joined #openstack-ironic06:56
openstackgerritTan Lin proposed openstack/ironic: Adopt Ironic's own context  https://review.openstack.org/29566306:57
yushiroI'd like to post the bug-report but I'm not sure it's a bug or not. Therefore, would you please tell me your opinion?06:57
*** alexpilotti has quit IRC07:00
*** spandhe has quit IRC07:04
*** e0ne has joined #openstack-ironic07:07
*** mgoddard has joined #openstack-ironic07:14
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Tempest plugin initial commit  https://review.openstack.org/29392507:17
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: [WIP] Tempest: add basic test  https://review.openstack.org/29392607:17
*** arturas has joined #openstack-ironic07:19
*** pcaruana has joined #openstack-ironic07:25
*** harshs has quit IRC07:25
*** e0ne has quit IRC07:35
*** e0ne has joined #openstack-ironic07:35
*** dtantsur|afk is now known as dtantsur07:37
dtantsurMorning Ironic07:37
*** Nisha_away has quit IRC07:38
*** jcoufal has joined #openstack-ironic07:39
*** e0ne has quit IRC07:40
*** e0ne has joined #openstack-ironic07:45
*** amotoki has joined #openstack-ironic07:52
*** e0ne has quit IRC07:54
*** Nisha has joined #openstack-ironic07:55
*** tesseract has joined #openstack-ironic07:56
*** tesseract is now known as Guest9087707:56
*** e0ne has joined #openstack-ironic07:57
*** dtantsur is now known as dtantsur|bbl07:59
*** ifarkas has joined #openstack-ironic08:02
*** yolanda has quit IRC08:04
*** ndipanov has joined #openstack-ironic08:06
*** jistr has joined #openstack-ironic08:07
*** e0ne has quit IRC08:10
*** mbound has quit IRC08:12
*** e0ne has joined #openstack-ironic08:13
aarefievmorning!08:15
*** baoli has joined #openstack-ironic08:16
*** e0ne has quit IRC08:16
*** mkovacik has quit IRC08:17
*** alexpilotti has joined #openstack-ironic08:17
*** e0ne has joined #openstack-ironic08:20
*** baoli has quit IRC08:20
*** alexpilotti has quit IRC08:22
*** achanda has quit IRC08:22
*** achanda has joined #openstack-ironic08:25
*** e0ne has quit IRC08:26
*** e0ne has joined #openstack-ironic08:29
*** e0ne_ has joined #openstack-ironic08:31
*** kromanenko has joined #openstack-ironic08:32
*** kromanenko_ has joined #openstack-ironic08:32
*** e0ne has quit IRC08:32
*** achanda has quit IRC08:32
*** e0ne_ has quit IRC08:33
*** achanda has joined #openstack-ironic08:38
*** alexpilotti has joined #openstack-ironic08:40
*** alexpilotti has quit IRC08:45
*** mbound has joined #openstack-ironic08:45
*** Nisha has quit IRC08:46
*** e0ne has joined #openstack-ironic08:48
openstackgerritZhenguo Niu proposed openstack/ironic: Add test to verify ironic multitenancy  https://review.openstack.org/26915708:49
openstackgerritZhenguo Niu proposed openstack/ironic: Allow to build user image with DIB  https://review.openstack.org/25636308:49
openstackgerritZhenguo Niu proposed openstack/ironic: Add Link-Local-Connection info to ironic port  https://review.openstack.org/25636508:49
openstackgerritZhenguo Niu proposed openstack/ironic: Added operator documentation for ironic portgroups  https://review.openstack.org/22849608:49
openstackgerritZhenguo Niu proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/20624408:49
openstackgerritZhenguo Niu proposed openstack/ironic: refactor ironic enroll-node code  https://review.openstack.org/25636408:49
openstackgerritZhenguo Niu proposed openstack/ironic: Add configure_provision_network function  https://review.openstack.org/25636708:49
openstackgerritZhenguo Niu proposed openstack/ironic: Update Ironic VM network connection  https://review.openstack.org/25636608:49
openstackgerritZhenguo Niu proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643208:49
openstackgerritZhenguo Niu proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326208:49
openstackgerritZhenguo Niu proposed openstack/ironic: Add network drivers  https://review.openstack.org/28585208:49
openstackgerritZhenguo Niu proposed openstack/ironic: Add Ironic/Neutron integration documentation  https://review.openstack.org/25859608:49
openstackgerritZhenguo Niu proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack  https://review.openstack.org/29352008:49
*** amit213 has quit IRC08:49
*** thrash|g0ne has quit IRC08:51
*** thrash has joined #openstack-ironic08:51
*** thrash has quit IRC08:51
*** thrash has joined #openstack-ironic08:51
*** e0ne has quit IRC08:53
*** yushiro has quit IRC08:54
*** achanda has quit IRC08:57
openstackgerritSofiia Andriichenko proposed openstack/python-ironicclient: Add test for 'node-show --field' command  https://review.openstack.org/30006408:58
*** e0ne has joined #openstack-ironic08:58
*** alexpilotti has joined #openstack-ironic09:02
*** alexpilotti has quit IRC09:02
*** mkovacik has joined #openstack-ironic09:04
vdrokmorning ironic, dtantsur|bbl, aarefiev09:05
aarefievvdrok: morning09:05
vinaym213morning vdrok,aarefiev,dtantsur and other ironicers not listed here09:06
*** kbyrne has quit IRC09:06
aarefievvinaym213 morning09:07
*** kbyrne has joined #openstack-ironic09:11
mkovacikmorning all!09:13
mkovacikdtantsur|bbl, ifarkas, sambetts|afk aarefiev, I've pushed new revision of https://review.openstack.org/#/c/267637/ based on yesterday's discussion, please, take a look when you've some time. Thanks!09:16
*** chihhsin has joined #openstack-ironic09:16
ifarkasmkovacik, sure, will take a look09:17
*** alexpilotti has joined #openstack-ironic09:18
vdrokmorning vinaym213 mkovacik and ifarkas09:19
mkovacikifarkas, thanks!09:19
mkovacikvdrok, o/09:19
*** arturas has quit IRC09:23
*** alexpilotti has quit IRC09:23
*** amotoki has quit IRC09:27
*** alexpilotti has joined #openstack-ironic09:33
*** e0ne has quit IRC09:36
*** amotoki has joined #openstack-ironic09:42
*** alexpilotti has quit IRC09:42
*** jcoufal has quit IRC09:42
*** amotoki has quit IRC09:44
*** amotoki has joined #openstack-ironic09:46
*** arturas has joined #openstack-ironic09:46
*** e0ne has joined #openstack-ironic09:47
*** mgould has joined #openstack-ironic09:50
*** amotoki has quit IRC09:51
*** amotoki has joined #openstack-ironic09:56
*** arturas has quit IRC09:56
*** amotoki has quit IRC09:56
mgouldmorning Ironicers!09:57
*** daemontool has joined #openstack-ironic09:57
*** achanda has joined #openstack-ironic09:58
*** amotoki has joined #openstack-ironic09:59
*** achanda has quit IRC10:02
*** e0ne has quit IRC10:08
*** e0ne has joined #openstack-ironic10:12
*** amotoki has quit IRC10:16
*** hoangcx has quit IRC10:16
*** daemontool_ has joined #openstack-ironic10:24
*** jcoufal has joined #openstack-ironic10:25
*** ohamada_ has joined #openstack-ironic10:27
*** ohamada has quit IRC10:27
*** daemontool has quit IRC10:27
*** alaski is now known as lascii10:38
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command  https://review.openstack.org/25145810:39
*** alexpilotti has joined #openstack-ironic10:40
*** dtantsur|bbl is now known as dtantsur10:40
dtantsurmkovacik, with pleasure. only one note: due to suddenly worsen weather, I'll only put -2 on reviews today.10:41
dtantsurthanks for your understanding10:41
*** alexpilotti has quit IRC10:44
*** amotoki has joined #openstack-ironic10:45
*** e0ne has quit IRC10:47
*** sivaramakrishna has quit IRC10:51
*** smoriya_ has quit IRC10:51
*** e0ne has joined #openstack-ironic10:52
sambetts|afkMorning all10:58
*** sambetts|afk is now known as sambetts10:59
openstackgerritKarim Boumedhel proposed openstack/ironic-specs: Adds a Blueprint for ovirt support in Ironic  https://review.openstack.org/30041010:59
*** trown|outtypewww is now known as trown10:59
openstackgerritKarim Boumedhel proposed openstack/ironic-specs: Add cloudinit support to ovirt.py module  https://review.openstack.org/30041111:00
*** e0ne has quit IRC11:04
mkovacikdtantsur, yeah, but since you're staying in bed the whole day could be just -1 ;)11:04
dtantsur:)11:06
*** Marga_ has joined #openstack-ironic11:07
openstackgerritKarim Boumedhel proposed openstack/ironic-specs: Adds a Blueprint for ovirt support in Ironic  https://review.openstack.org/30041811:07
*** e0ne has joined #openstack-ironic11:08
*** e0ne has quit IRC11:11
*** Marga_ has quit IRC11:11
*** baoli has joined #openstack-ironic11:12
jrollgooood morning, happy friday11:14
dtantsurmorning jroll11:15
jrollhey dtantsur11:15
jrollfriendly reminder not to believe anything you read today11:15
jrollI see neutron folks are being properly trolled on the ML11:16
mkovacikjroll, o/11:16
*** baoli has quit IRC11:16
dtantsuryep :)11:25
dtantsurI forgot to put a spec to turn Ironic into a proper CMDB.. again..11:25
dtantsur:)11:25
jrollheh11:26
trownhttps://review.openstack.org/#/c/300272/ is my favorite11:26
jrollI'm going to build a PTLaaS one year11:26
jrolltrown: idk, I think http://lists.openstack.org/pipermail/openstack-dev/2016-April/091120.html11:27
jrolloh my, https://review.openstack.org/#/c/300274/11:28
trownhehe11:29
trownshould post a follow up adding Ironic back in to... could be Notronic11:30
dtantsurLOL11:30
*** achanda has joined #openstack-ironic11:30
*** e0ne has joined #openstack-ironic11:33
jrollomg yes11:33
* jroll does it11:33
trown:)11:33
*** amotoki has quit IRC11:34
dtantsursambetts, please see if your concerns are resolved on the reapply patch11:35
*** achanda has quit IRC11:35
sambettsdtantsur: sure :)11:35
jrollhttps://review.openstack.org/#/c/300428/11:37
*** daemontool_ is now known as daemontool11:37
dtantsur+111:40
*** alexpilotti has joined #openstack-ironic11:40
*** baoli has joined #openstack-ironic11:42
*** vinay50muddu has joined #openstack-ironic11:43
*** vinay50muddu has quit IRC11:43
mkovacikdtantsur, thanks for the sprinkling of my patch with the Holy Ironic Bear pee ;)11:44
*** vmud213 has joined #openstack-ironic11:44
dtantsurlol wut?? :D11:44
jrollO_o11:45
dtantsurman, we should rename our +W gerrit text to "Approved by the Holy Ironic Bear pee"11:45
*** alexpilotti has quit IRC11:45
*** baoli_ has joined #openstack-ironic11:45
jrolllol11:45
*** vinaym213 has quit IRC11:47
*** baoli has quit IRC11:48
*** e0ne has quit IRC11:50
sambettsdtantsur, mkovacik: left a comment on that patch, but everything else looks good to me11:50
*** Goneri has quit IRC11:50
jrollwelp.11:50
jroll$ find . -type f -exec sed -i -e 's/ironic/neutronic/g' {} \;11:50
jroll$ git st11:50
jrollerror: bad index file sha1 signature11:50
jrollfatal: index file corrupt11:50
jrollmistakes were made.11:50
sambettsohhhh god ... .11:51
TheJuliagood morning11:51
sambettso/ TheJulia11:51
dtantsuroh yeah, "find ." was not the best idea, I'm afraid11:51
dtantsurmorning TheJulia11:51
jrollyeah :D11:51
jrollthat's one way to clean local branches out11:51
dtantsuralso imagine how many things it screwed in .tox :)11:51
sambettsjroll: http://logs.openstack.org/29/299529/4/gate/gate-tempest-dsvm-ironic-agent_ssh-src/32a27ac/ I hope this isn't an april fool11:52
dtantsursambetts, tripleo CI is/was widely affected by this11:52
jrollsambetts: nope, that was a thing yesterday11:52
dtantsurbut I hope it got fixed this night11:52
sambettsLast 2 gate runs blew up here because it this morning https://review.openstack.org/#/c/299529/11:53
dtantsurfolks, please check my memory: we try to NOT do network requests in the driver validate() call, right?11:53
dtantsurjroll, ^^11:53
jrolldtantsur: I believe so as that's synchronous11:53
TheJuliadtantsur: re renaming +W.... omg yes11:53
openstackgerritDebayan Ray proposed openstack/ironic: Fix NamedTemporaryFile() OSError Exception  https://review.openstack.org/29376211:53
sambettsdtantsur: its a sync method so I believe so11:53
dtantsur:)11:53
*** e0ne has joined #openstack-ironic11:54
sambettsdtantsur: thats why I prompted the OneView guys to make their thing a periodic, and have the checks they want in validate to just be checking node properties11:54
sambettsdtantsur: originally they were going to call out to OneView in the validate function and I was like probably not a good idea :-p11:55
dtantsuryeah. I'm reviewing the submission of the libvirt driver to ironic-staging-drivers, and they do the same11:55
*** e0ne has quit IRC11:57
*** dprince has joined #openstack-ironic11:57
openstackgerritJim Rollenhagen proposed openstack/ironic: Rename ironic to neutronic  https://review.openstack.org/30043411:58
jrollwheeeee11:58
sambettswow thats a huge patch :-P11:59
* sambetts things we should replace everywhere in the code that says Ironic with a {project-name} indicator so that we can rename the whole project on the fly by just changing a variable 12:00
sambettss/things/thinks12:00
jrollheh12:01
trownconflicts with 11512:02
jrolldtantsur: btw, https://review.openstack.org/#/c/300146/12:02
jroll(this is not a joke patch)12:03
trownsambetts: that traceroute issue was infra, and should be fixed12:03
*** alexpilotti has joined #openstack-ironic12:03
sambettsdtantsur: my concern is that if someone didn't pay attention to the api docs and sent a body to the POST endpoint, right now it wouldn't care and it would just do a reapply, but in the future if we start doing things with the body then it would change the functionality of their call12:03
sambettstrown: yeah, it must be taking some time for the images to propergate or something12:03
*** jjohnson2 has joined #openstack-ironic12:06
dtantsursambetts, I'm not against starting such checks, just pointing our that we never did it before12:07
openstackgerritJim Rollenhagen proposed openstack/ironic: Make tox respect upper-constraints.txt  https://review.openstack.org/30014612:07
dtantsurmkovacik, ^^12:07
jrolldtantsur: good points, fixed12:07
dtantsur+@12:08
dtantsur+212:08
sambettsdtantsur: Does it fit into any of our other API endpoints?12:08
dtantsursambetts, you mean, such check? well, we could check absence of additional arguments in all other endpoints..12:09
dtantsurjroll, and then we have to backport the same thing to liberty and mitaka stable?12:09
*** e0ne has joined #openstack-ironic12:09
sambettsdtantsur: Oh yeah I guess :/12:10
jrolldtantsur: yeah, probably should12:10
jrollalso. my branch-based ipa builds aren't working >.>12:11
dtantsursambetts, well, anyway, since we plan on using body later on, we should probably check it for now. mkovacik, objections?12:11
jrollhttp://logs.openstack.org/23/237b76e0d532ff5c6bf45fefc89c81d8f89a3d49/post/ironic-python-agent-buildimage-coreos/8a37d7f/console.html#_2016-04-01_00_24_55_78412:11
dtantsurjroll, ouch :(12:11
jrollಠ_ಠ12:11
jrollis it because I didn't export BRANCH_PATH, I assume?12:11
dtantsurprobably12:11
jrollgrumble12:12
dtantsurnow to something completely different. jroll, do you have any ideas about https://bugzilla.redhat.com/show_bug.cgi?id=1317731 ? (looks like the bug is public, so you should be able to see it)12:12
openstackbugzilla.redhat.com bug 1317731 in openstack-ironic-python-agent "[RFE]: IPA needs to support Multipathed SAN LUNs for root disk" [High,New] - Assigned to dtantsur12:12
dtantsurdunno what it implies for upstream, maybe you've heard about someone doing that?12:13
jrollhttps://review.openstack.org/30044112:14
* TheJulia takes a look at the ipa bug since that has been a bit of a major issue in testing12:15
jrolldtantsur: so they want boot from volume, essentially :)12:15
jrolldtantsur: though, I suspect we could be missing something in the image to detect that, and/or they're missing a root device hint12:16
dtantsuryeah, but some simple variant, as I understood it.. i.e. assume the disk is already magically connected12:16
jrollright12:16
dtantsurjroll, missing root device hint is my current guess12:16
TheJuliajroll: nah, its not boot from volume, it is missing root device hint12:16
jrollya12:16
jrollTheJulia: they did mention iscsi :)12:16
TheJuliatrue12:16
TheJuliasame thing, path deduplication12:17
jrollyeah12:17
dtantsurwe do miss a couple of things on an image as well12:17
TheJuliabottom line it could be a feature request although really we just need to get mpath tools running12:17
TheJuliabad side of that is... san venders have variety in the proper way to identify alternate paths12:17
* TheJulia twitches has flashbacks12:18
jrollheh12:18
TheJuliaso tooling to pass in a specific multipath config file would be needed in dib12:18
TheJuliadoes coreos even support multipathing?12:18
jrolldoubt it, they don't even support mdraid12:19
TheJuliai remember searching their docs for the word a few months ago and got 0 results12:19
dtantsuroh hmm, number of proposed virt drivers increase: https://bugs.launchpad.net/ironic/+bug/156484112:20
openstackLaunchpad bug 1564841 in Ironic "[RFE] Add a ovirt driver to Ironic" [Undecided,Incomplete] - Assigned to Karim Boumedhel (karmab)12:20
jrollwhy do people want all these virt drivers.12:21
sambettsjroll: why can't they use nova?12:21
sambettssurely thats the question :-P12:21
jrollheh12:21
dtantsurI suspect that's for testing on something weird (again)12:22
jrollit is, it calls out testing tripleo12:22
dtantsurI remember a guy trying to run tripleo on ovirt nodes12:22
dtantsurdidn't quite work :)12:22
* jroll is reminded to do an rfe sweep today12:22
* dtantsur is going through bugs as well12:23
dtantsurso we can sync our efforts/have quick discussions, etc12:23
jrolldtantsur: if you want to triage rfes to just unconfirmed, I can take them from there12:23
TheJuliadtantsur: w/r/t multipath, I kind of think it could be a simple addition to IPA to test for/handle, and try to identified the path that is not locked, or to assert a device lock on a particular path, which would kind of allow it to work in mpath environments without trully consolidating the path down to a single device where the OS is magically doing MPIO under the hood.... but in order to devel12:28
TheJuliaop that I think one of is is goign to need access to a san system that is either very buggy or very mean with lock management12:28
*** wuhg has quit IRC12:29
TheJuliawhat we've seen is one controller rejects IO on that path because persumably the lun has a lock on the other path, and saldy ipa has chosen the wrong path to try and write the disk out to12:30
dtantsurjroll, re https://bugs.launchpad.net/python-ironicclient/+bug/1564484 I don't think we should treat writing tests as rfe12:30
openstackLaunchpad bug 1564484 in python-ironicclient "[RFE] Write tests tests for testing node-delete command for Ironic Client " [Undecided,In progress] - Assigned to Sergii Turivnyi (sturivnyi)12:30
jrollgeez, https://review.openstack.org/#/c/297479/ finally passed CI12:30
jrolldtantsur: +112:30
dtantsurTheJulia, sounds complex :(12:30
TheJuliadtantsur: very :(12:30
jrollforgot about client rfes :/12:31
*** achanda has joined #openstack-ironic12:32
jrolldtantsur: you clearing these test RFEs out or shall I?12:34
* jroll jfdi12:34
dtantsurjroll, go ahead, I got dragged into another discussion12:35
jrollyeah no worries12:35
jrolldtantsur: any problem with me approving https://bugs.launchpad.net/ironic-python-agent/+bug/1552090 ?12:37
openstackLaunchpad bug 1552090 in ironic-python-agent "IPA cannot create UEFI bootable iso for coreOS " [Wishlist,In progress] - Assigned to Shivanand Tendulker (stendulker)12:37
*** achanda has quit IRC12:37
dtantsurjroll, no problem12:38
jrollthanks12:38
*** devananda has quit IRC12:43
*** keedya has joined #openstack-ironic12:43
*** devananda has joined #openstack-ironic12:44
openstackgerritVasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy  https://review.openstack.org/26915712:44
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to build user image with DIB  https://review.openstack.org/25636312:44
openstackgerritVasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port  https://review.openstack.org/25636512:44
openstackgerritVasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups  https://review.openstack.org/22849612:44
openstackgerritVasyl Saienko proposed openstack/ironic: refactor ironic enroll-node code  https://review.openstack.org/25636412:44
openstackgerritVasyl Saienko proposed openstack/ironic: Add configure_provision_network function  https://review.openstack.org/25636712:44
openstackgerritVasyl Saienko proposed openstack/ironic: Update Ironic VM network connection  https://review.openstack.org/25636612:44
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643212:44
openstackgerritVasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326212:44
openstackgerritVasyl Saienko proposed openstack/ironic: Add network drivers  https://review.openstack.org/28585212:44
openstackgerritVasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation  https://review.openstack.org/25859612:44
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack  https://review.openstack.org/29352012:44
mkovacikjroll, dtantsur w/r the reapply, I think it's better if we avoid splitting manual from api, it doesn't make much sense; instead if we chose one  day to have the ramdisk posting the data to /uuid/data/unprocessed we should have put over the same path for reapply; i still think that the idempotence isn't affected as resulting processed data is going to be manipulated so if you GET /uuid/data/unprocessed you'll have the same thing12:49
mkovacikyou put there and if you put 100 times, the data there will remain the same; what will change is the result data at /uuid/data12:49
*** dims_ has quit IRC12:49
*** dims has joined #openstack-ironic12:49
* jroll has no context here12:49
dtantsurmkovacik, " we chose one  day to have the ramdisk posting the data to /uuid/data/unprocessed" this day is not very close ;)12:49
dtantsurjroll, I think he meant to include sambetts :)12:49
jrollprobably :)12:50
mkovacikthen, sambetts, why the -112:50
mkovacikyeah sorry jroll12:50
* dtantsur feels like he also has no context12:50
dtantsurmkovacik, I think sambetts' -1 is about enforcing that the new API does not accept a body, until we implement it12:50
jrollheh no worries12:51
mkovacikdtantsur, sambetts, renaming api_data_unprocessed  won't help anything; I'm cool with the 400 over non-empty body12:51
mkovacikbut we may one day ;) want to change that ;)12:52
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with port.  https://review.openstack.org/25935412:53
dtantsurwe probably will, even though it's unlikely to come from a ramdisk12:53
dtantsurmaybe more in line with what nailgun folks are doing: periodic resubmitting of introspection data12:53
dtantsurthis has interesting implications though12:53
mkovacikdtantsur, so it would make sense to have POST trigger regular processing and PUT trigger reapply over such an endpoint12:54
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with port.  https://review.openstack.org/25935412:54
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for the Ironic CLI commands which checks port-create, port-show, and port-delete using them either without arguments or with incorrect arguments.  https://review.openstack.org/25935412:55
*** vinaym213 has joined #openstack-ironic12:55
mkovacikdtantsur, sambetts so lemme hack in the body size check with some reasonable text: "body processing not implemented yet" and keep it POST @ /uuid/data/unprocessed ?12:56
*** e0ne has quit IRC12:57
dtantsurI hope we already agreed on the endpoint itself ;) yeah12:58
sambettsyup12:58
*** vmud213 has quit IRC12:58
mkovacikdtantsur, sambetts, ack :)12:59
*** mtanino has quit IRC13:01
mkovaciksambetts, dtantsur I've just realised we won't be able to deprecate it later if we chose to have ramdisk post there; it will be just 400 to empty bodies, possibly breaking users; or you think we could redirect then somehow?13:03
sambetts?13:04
sambettswhen we enable receiving a body we'll just remove the 40013:05
sambettsnow: empty body = reapply, populated body = 400, the future: empty body = reapply, populated body = set and apply13:06
dtantsursambetts++13:06
*** amotoki has joined #openstack-ironic13:06
mkovaciksambetts, put -> reapply?13:07
mkovacikbut OK13:07
sambettsno, there is no PUT, we established ysterday we can't use PUT13:07
mkovacikOK13:08
*** mbound has quit IRC13:08
*** mbound has joined #openstack-ironic13:08
*** links has quit IRC13:12
*** mkovacik has quit IRC13:14
*** Goneri has joined #openstack-ironic13:14
*** e0ne has joined #openstack-ironic13:15
*** [1]cdearborn has joined #openstack-ironic13:15
*** jjohnson2 has quit IRC13:16
*** jjohnson2 has joined #openstack-ironic13:16
jrollhm, need a simple, backportable, IPA job to trigger post jobs on all the things13:23
*** dansmith is now known as bizarroman13:25
*** bizarroman is now known as bizarrodan13:25
*** dims_ has joined #openstack-ironic13:27
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: Correct link to enabling agent drivers  https://review.openstack.org/30048513:28
jrollthere we go.13:28
*** e0ne has quit IRC13:29
*** jistr has quit IRC13:32
*** openstackgerrit has quit IRC13:33
*** rbudden has joined #openstack-ironic13:34
*** openstackgerrit has joined #openstack-ironic13:34
*** achanda has joined #openstack-ironic13:34
*** krtaylor has quit IRC13:35
jrollcould use reviews on that so we can land/backport once project-config change has been applied ^13:37
jrollthe intent is to re-kick post jobs on master and stable branches13:38
*** achanda has quit IRC13:39
*** electrofelix has joined #openstack-ironic13:41
dtantsurjroll, +213:43
*** thiagop-afk has joined #openstack-ironic13:43
*** thiagop-afk is now known as thiagop13:43
thiagopgood morning folks13:43
openstackgerritThiago Paiva Brito proposed openstack/ironic-specs: Dynamic allocation of nodes on OneView drivers  https://review.openstack.org/27572613:44
dtantsurmorning thiagop13:44
thiagophiya dtantsur13:45
dtantsurugh, there is a race between DHCP and IPA in the DIB build. looks like it results in inspection sometimes not returning any IP addresses13:45
dtantsurI wonder if we should have a code waiting for IP addressed to get assigned..13:46
*** e0ne has joined #openstack-ironic13:51
*** mtanino has joined #openstack-ironic13:51
*** lascii is now known as lutf-813:52
NobodyCamgood morning Ironicers and of course ....13:52
NobodyCamTGIF!13:52
dtantsurmorning NobodyCam, really TGIF :)13:52
sambettsdtantsur: I think it should be possible to make the .service file wait until after dhcp as got an IP13:54
dtantsursambetts, well, I thought I did it by making it depend on network-online target13:54
dtantsurtoday, however, I see it didn't happen: https://bugzilla.redhat.com/show_bug.cgi?id=132289213:55
openstackbugzilla.redhat.com bug 1322892 in openstack-ironic-discoverd "No valid interfaces found during introspection" [Unspecified,New] - Assigned to dtantsur13:58
thiagopmorning TGIF NobodyCam13:58
sambettsdtantsur: it should do looking at the docs for it, although maybe we need to do this? systemctl enable systemd-networkd-wait-online.service or systemctl enable NetworkManager-wait-online.service13:58
sambettshttps://www.freedesktop.org/wiki/Software/systemd/NetworkTarget/13:58
dtantsurthere is no network manager on the ramdisk13:58
NobodyCam:)13:58
sambettsdtantsur: networkd?13:58
dtantsurdunno, checking13:58
sambettsdtantsur: it seems like without at least one of those, the network targets won't work13:58
sambettsHey NobodyCam TGIF :)13:58
dtantsurwell, I have neither, but it did solve a similar problem some time ago13:58
dtantsurat least for our downstream RHEL builds13:58
jrollfolks, don't forget to vote for the TC, deadline is sometime next week!13:58
* dtantsur already did13:58
* sambetts did too 13:58
* jroll just finished13:58
*** openstack has quit IRC13:58
*** openstack has joined #openstack-ironic14:00
*** daemontool_ is now known as daemontool14:01
jrollsomeone mind landing https://review.openstack.org/#/c/300485/1 ?14:02
jrollneed to backport that through stable to kick off some post jobs14:02
jrollwtf, docs job fail?14:03
dtantsursambetts, maybe the problem is in the weird way DIB DHCP's things: https://github.com/openstack/diskimage-builder/tree/master/elements/dhcp-all-interfaces14:03
sambettsjroll: I've seen that a lot recently14:04
jrolltimeout, /me rechecks14:04
jrollyeah14:04
dtantsurme too14:04
jrollthis was vexxhost, maybe should start keeping track14:04
openstackgerritSofiia Andriichenko proposed openstack/python-ironicclient: Add test for chassis-node-list  https://review.openstack.org/29948214:04
sambettsdtantsur: yeah :/ maybe we need to make sure that the ipa.service file is after the dhcp-all-interfaces.service file?14:06
openstackgerritSofiia Andriichenko proposed openstack/python-ironicclient: Add test for chassis-node-list  https://review.openstack.org/29948214:06
dtantsursambetts, maybe, but there's no single service, rather a family of them...14:07
*** e0ne has quit IRC14:07
dtantsurso I'm starting to think that we should ensure it on IPA side rather than requiring downstream to figure out all this craziness14:07
sambettsI guess we could just add a peice of code to IPA that does, if no route to to Ironic IP blow up, and then add the retry= properties to the ipa.service14:08
thiagopNobodyCam: have you seen the changes on the spec of dyn allocation wrt the use of periodic tasks?14:12
thiagopNobodyCam: L88-L95 and L158-17914:13
*** stendulker has joined #openstack-ironic14:14
dtantsursambetts, we do have retry on detecting route to Ironic, it should rather be a retry on at least one IP address appearing in inventory... but I'd ask someone knowing about dhcp-all-interfaces thing first14:14
* dtantsur moves this to #tripleo14:15
sambettsdtantsur: maybe retry on route to inspector too for that case14:18
stendulkerdtantsur: Hi14:18
dtantsursambetts, maybe14:18
dtantsurstendulker, hi14:18
stendulkerdtantsur: this review https://review.openstack.org/#/c/299573/ is based on a new feature code https://review.openstack.org/#/c/27576614:18
stendulkerdtantsur: Is that ok? It has 2 +2s14:18
openstackgerritMerged openstack/ironic-python-agent: Correct link to enabling agent drivers  https://review.openstack.org/30048514:18
dtantsurstendulker, well, this fix is independent of that feature, so I'm fine with landing it. It's a question for TheJulia what it was done this way14:18
stendulkerdtantsur: yes fix is independent14:18
stendulkerdtantsur: is basing code like this is correct for landing?14:18
dtantsurstendulker, well, no. it won't land until the parent lands. TheJulia probably needs to rebase it on top of master14:18
stendulkerdtantsur: ok. i will -1 it with the comment14:18
stendulkerdtantsur: thank you.14:19
TheJuliaI'll rebase it off of master a little later then14:22
stendulkerTheJulia: thank you14:22
TheJuliathe previous patch really hits the issue hard, so the dependency has to be flipped in that case since ANC won't work otherwise14:22
TheJuliawell, it will work, and ilo users will find their hardware powered off14:23
stendulkerTheJulia: yes, that would be better14:23
thiagophiya TheJulia14:25
jrolldtantsur: when you have a moment, liberty https://review.openstack.org/#/c/300516/ and mitaka https://review.openstack.org/#/c/300515/14:25
jrollno rush, going to take a break for a bit14:25
*** harshs has joined #openstack-ironic14:27
TheJuliawell, easier rebase than I thought14:27
*** stendulker_ has joined #openstack-ironic14:28
*** stendulker has quit IRC14:31
openstackgerritJulia Kreger proposed openstack/ironic: Don't power off non-deploying iLO nodes in takeover  https://review.openstack.org/29957314:34
jlvillalGood morning Ironic14:37
*** vishwanathj has joined #openstack-ironic14:37
*** stendulker_ has quit IRC14:38
*** vinay50muddu has joined #openstack-ironic14:39
*** ametts has joined #openstack-ironic14:42
*** vinaym213 has quit IRC14:43
*** harshs_ has joined #openstack-ironic14:44
dtantsurmorning jlvillal14:44
*** vinay50muddu has quit IRC14:44
dtantsursambetts, I'm looking at our downstream ramdisk code (which is a bash hell), I see things like 'sleep 10' to make sure all interfaces did receive DHCP >_<14:44
*** harshs has quit IRC14:45
*** harshs_ is now known as harshs14:45
dtantsursambetts, so next idea: loop until all interfaces get their IP's with some reasonable deadline (30 seconds?), then proceed with inspection14:45
dtantsurwdyt?14:45
*** jistr has joined #openstack-ironic14:46
sambettsdtantsur: thats definatly one way to do it14:50
dtantsurlemme try to put together a patch..14:54
*** garthb_ has joined #openstack-ironic14:54
*** Marga_ has joined #openstack-ironic14:55
*** e0ne has joined #openstack-ironic14:57
*** Marga_ has quit IRC14:57
*** Marga_ has joined #openstack-ironic14:58
sambettsjroll: https://review.openstack.org/#/c/299485/3/jenkins/jobs/ironic-python-agent-jobs.yaml, AJaeger seems to want us to standardise the post jobs, that would involve standardising the build scripts right?14:59
*** Marga_ has quit IRC14:59
*** Marga_ has joined #openstack-ironic15:00
jrollsambetts: yeah, I guess merging them is an option15:01
jrollwe don't have to standardize any scripts, just run both, then upload all the things15:02
jrollthis also means a bug in either script will break uploading of both images, though15:02
jroll\o jlvillal15:02
sambettsjroll: I didn't want to do that so that one wouldn't affect the other15:02
jrollyeah, that's my thought15:02
*** electrofelix has quit IRC15:03
sambettsI messaged AJaeger on #infra about it and he seemed to want us to make the scripts both dump into a common UPLOAD folder so that we could have generic scp code etc15:04
*** mdorman has joined #openstack-ironic15:05
jrollsambetts: I'm kinda meh on that, it's also going to break devstack if we move them somewhere else15:06
openstackgerritKarim Boumedhel proposed openstack/ironic-specs: Adds a Blueprint for ovirt support in Ironic  https://review.openstack.org/30041815:06
sambettshaving something like job: ironic-python-agent-buildimage-{image-type} then using the {image-type} to select what get built15:07
jrolloh, a template, that's fine15:07
jrollmaybe just template out the scp source path for now too15:07
*** links has joined #openstack-ironic15:07
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Wait for the interfaces to get IP addresses before inspection  https://review.openstack.org/30054815:13
dtantsursambetts, ^^^15:13
jroll2016-04-01 15:18:08.986 | + imagebuild/coreos/full_trusty_build.sh15:18
jroll2016-04-01 15:18:08.986 | + BRANCH_PATH=stable-mitaka15:18
jrollwoooooo. finally.15:18
dtantsur\o/15:20
NobodyCamwow :)15:20
jrollnext step is https://review.openstack.org/29747915:21
jrolland then all the backport craziness for that15:21
dtantsurjroll, really dunno how to act upon https://bugs.launchpad.net/ironic/+bug/1564841. redirect to ironic-staging-drivers?15:21
openstackLaunchpad bug 1564841 in Ironic "[RFE] Add a ovirt driver to Ironic" [Undecided,In progress] - Assigned to Karim Boumedhel (karmab)15:21
jrolldtantsur: yeah, I think so15:21
jrollhttp://tarballs.openstack.org/ironic-python-agent/coreos/files/ \o/15:24
*** harshs has quit IRC15:30
NobodyCamjroll: am I missing reason there is a  need for the 'tr' command?15:30
dtantsurNobodyCam, replace / with - for use in the file name15:31
NobodyCampreventing folks from sub-pathing15:31
*** e0ne has quit IRC15:31
dtantsuraka stable/mitaka -> stable-mitaka15:31
NobodyCamah15:31
NobodyCamTY15:31
NobodyCam+2 with a minor nit15:36
NobodyCam:p15:36
*** achanda has joined #openstack-ironic15:37
*** suro-patz has joined #openstack-ironic15:37
*** achanda has quit IRC15:42
dtantsursambetts, do we expect cisco CI to pass on https://review.openstack.org/#/c/297479/ ?15:43
*** jaybeale has joined #openstack-ironic15:44
*** krtaylor has joined #openstack-ironic15:44
*** Guest90877 has quit IRC15:44
openstackgerritZhenguo Niu proposed openstack/ironic: Correct api version check conditional for node.name  https://review.openstack.org/29926415:45
NobodyCamdoh TheJulia I just saw that I made a comment on pre existing section of a doc that was not part of the change you are working on... sorry :p15:49
sambettsdtantsur: CIMC will pass UCSM will fail15:49
dtantsurok, so it's safe to approve, right?15:49
sambetts+215:50
*** yolanda has joined #openstack-ironic15:53
*** links has quit IRC15:55
openstackgerritMerged openstack/ironic-python-agent: Enable branch tagging during tinyipa build  https://review.openstack.org/29952915:55
jrollwoohoo, thanks for +W dtantsur15:57
*** penick has joined #openstack-ironic15:57
dtantsur:)15:58
* jlvillal feels somewhat vindicated in his belief that checking log messages in unit tests can have value. https://bugs.launchpad.net/ironic/+bug/156453115:58
openstackLaunchpad bug 1564531 in Ironic "Unit test: ironic.tests.unit.conductor.test_manager.DoNodeCleanTestCase.test__do_next_clean_step_manual_fail_in_tear_down_cleaning incorrect" [Undecided,In progress] - Assigned to John L. Villalovos (happycamp)15:58
dtantsurjlvillal, I personally prefer to only check if a substantial part of a message is present (e.g. node UUID). I also tend to never check log messages, only user-visible ones.15:59
jrollwhatever happened to oneview ci?15:59
dtantsursambetts, could you please make the ucsm job non-voting until it's ready to vote?16:00
jlvillaldtantsur: Understood. But only because I checked the log message did I discover that the unit test was wrong.16:00
dtantsursambetts, otherwise it's confusing for reviewers whether we should pay attention to Cisco CI or not16:00
dtantsurjlvillal, sigh. it happens :(16:01
dtantsuralso, today I've fixed one more bug with incorrect formatting in log messages...16:01
jlvillaldtantsur: Oh :(  Sad face for bug with log messages.16:01
jlvillaldtantsur: :) happy face for you finding and fixing it!16:01
dtantsurjlvillal, btw please review it: https://review.openstack.org/29988716:02
dtantsura really small one16:02
jlvillaldtantsur: will do!16:02
jroll+2'd16:02
jlvillaldtantsur: And if you care to look at this: https://review.openstack.org/#/c/300204/   Though not really small one16:02
jlvillal+2 +A16:04
*** amotoki has quit IRC16:04
dtantsurthnx16:05
* jlvillal is that grumpy guy who doesn't find most of the April Fools jokes funny :P16:05
jlvillalSometimes there are good ones, but most of them are meh...16:06
dtantsurjlvillal, my wife said the same, except that she likes neither16:06
jlvillal:D16:06
*** trown is now known as trown|lunch16:06
jlvillalI have learned to stay away from Slashdot on April 1st...16:06
jrolllol, dtantsur and I had the same comment16:08
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Wait for the interfaces to get IP addresses before inspection  https://review.openstack.org/30054816:09
dtantsurjust added a release note ^^^16:09
dtantsurjroll, really lol16:09
jrollyep :P16:09
*** garthb_ has quit IRC16:10
dtantsurifarkas, hey! mind reviewing https://review.openstack.org/#/c/294467/ please? it's been around for some time already16:10
*** zzzeek has quit IRC16:13
sambettsdtantsur: I've made the UCS job voting: false, so any new runs should be marked non-voting from now on16:13
dtantsurthnx16:13
*** MattMan has quit IRC16:13
*** zzzeek has joined #openstack-ironic16:14
*** jcoufal has quit IRC16:15
*** suro-patz has quit IRC16:16
*** jjohnson2 has quit IRC16:16
*** jjohnson2 has joined #openstack-ironic16:17
*** mbound has quit IRC16:19
*** yolanda has quit IRC16:19
*** karimb has joined #openstack-ironic16:21
*** ohamada_ has quit IRC16:23
*** e0ne has joined #openstack-ironic16:24
jlvillaldtantsur: jroll: Thanks for helping me learn something new on the asserts :)16:25
jroll:D16:25
dtantsur:)16:25
dtantsurthe mock library has a lot of funny stuff, I also learn something new from time to time16:26
jlvillalYeah, trying to teach the mock stuff to my new co-worker. I can seem them struggling a bit to understand it. I know when I first learned it I was going "huh????"16:27
*** suro-patz has joined #openstack-ironic16:27
sambetts:-P its a weird concept to learn :-P16:27
jlvillalsambetts: +116:28
*** e0ne has quit IRC16:29
jlvillalAlso some of them have never done unit testing before. And then you add in mock.16:29
*** e0ne has joined #openstack-ironic16:29
*** Marga_ has quit IRC16:30
*** yolanda has joined #openstack-ironic16:30
*** openstackgerrit has quit IRC16:31
*** rcernin has quit IRC16:32
sambettsbeen through a similar thing with a co-worker16:32
*** achanda has joined #openstack-ironic16:38
* jlvillal waits patiently for bot to announce patch he just pushed...16:38
*** e0ne has quit IRC16:39
jlvillalJohn Villalovos proposed openstack/ironic: Fix _do_next_clean_step_fail_in_tear_down_cleaning() https://review.openstack.org/#/c/300204/16:39
TheJuliaheh16:39
jlvillalI was tired of waiting. It had been a whole two minutes16:40
TheJuliawell, the bot did timeout about 01 minutes ago16:40
TheJulia1016:40
jlvillalAh.16:40
* jlvillal disables seeing join/quit messages16:40
*** kromanenko has quit IRC16:41
TheJuliaPosted new revision of https://review.openstack.org/#/c/275766 -16:41
TheJuliaActive Node Creation via adopt state16:41
*** kromanenko_ has quit IRC16:41
* TheJulia goes and digs into new bug16:41
jrolldtantsur: mitaka devstack changes are passing now https://review.openstack.org/#/c/299317/16:43
NobodyCamjlvillal: I did that a year ago :)16:44
*** pcaruana has quit IRC16:44
* dtantsur clicks16:44
jlvillalNobodyCam: :)  It does help the signal to noise ratio for me.16:45
*** achanda has quit IRC16:45
*** openstackgerrit has joined #openstack-ironic16:46
dtantsurjroll, +2. looks like it even works with the inspector gate, good16:47
dtantsurand now I have to run, see you next week and have a great weekend16:47
jrolldtantsur: thanks! have a good weekend16:47
*** dtantsur is now known as dtantsur|afk16:47
jrolloh dtantsur, before you go16:47
mgoulddtantsur, good night!16:47
jrolldtantsur|afk: ^16:47
dtantsur|afkyep?16:47
jlvillaldtantsur: One good thing about April Fools day, both Apress and PragPub are having 40% of ebooks :)16:47
jrolldtantsur|afk: you want a change on liberty inspector to test the devstack change on inspector?16:47
jrollI can push one up16:48
dtantsur|afkjroll, not sure I got the question, but please do :)16:48
dtantsur|afkprobably both mitaka and liberty, in this case16:48
jrolldtantsur|afk: you don't have a test on devstack changes, so a testpatch on inspector liberty against that devstack change16:48
jrollok16:48
jrollthere's no kilo, right?16:48
dtantsur|afknope16:49
jrollk16:49
jrollthanks!16:49
dtantsur|afkthank you16:49
* dtantsur|afk is now really afk16:49
sambettsgoodnight dtantsur|afk16:54
*** e0ne has joined #openstack-ironic16:55
*** Marga_ has joined #openstack-ironic17:02
openstackgerritMerged openstack/ironic: Fix _do_next_clean_step_fail_in_tear_down_cleaning()  https://review.openstack.org/30020417:04
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Tests for testing node-create command  https://review.openstack.org/26205517:08
*** trown|lunch is now known as trown17:09
*** mgoddard has quit IRC17:14
*** ndipanov has quit IRC17:14
*** e0ne has quit IRC17:15
*** Marga_ has quit IRC17:17
*** Marga_ has joined #openstack-ironic17:17
*** yolanda has quit IRC17:19
*** davideagnello has joined #openstack-ironic17:19
*** suro-patz has quit IRC17:21
*** sambetts is now known as sambetts|afk17:26
sambetts|afkNight all17:26
jlvillalsambetts|afk: Ciao17:26
openstackgerritxiexs proposed openstack/python-ironicclient: Add multi nodes support for "baremetal delete" command  https://review.openstack.org/28967617:28
NobodyCamnight sambetts|afk17:31
NobodyCamnight dtantsur|afk17:31
openstackgerritZhenguo Niu proposed openstack/ironic: Correct api version check conditional for node.name  https://review.openstack.org/29926417:39
*** vishwanathj has quit IRC17:40
openstackgerritMerged openstack/ironic-python-agent: Fix incorrect formatting in the iSCSI extension  https://review.openstack.org/29988717:44
*** suro-patz has joined #openstack-ironic17:46
*** cdearborn has joined #openstack-ironic17:52
*** mgould has quit IRC17:54
*** [3]cdearborn has joined #openstack-ironic17:55
*** vishwanathj has joined #openstack-ironic17:57
*** jistr has quit IRC17:58
*** suro-patz has quit IRC18:02
jrolldevananda: you'll be interested in https://review.openstack.org/#/c/245459/18:02
jroll(TIL)18:02
*** harlowja has quit IRC18:02
*** suro-patz has joined #openstack-ironic18:03
*** piet has joined #openstack-ironic18:05
* devananda looks18:05
jrollI haven't reviewed yet, but didn't realize someone was working on this18:05
devanandaoh. hah. nice18:05
*** [1]cdearborn has quit IRC18:06
jrolldevananda: would also like your eyes on https://review.openstack.org/#/c/299317/18:07
jrollthat's the last thing to make mitaka test mitaka ipa18:07
*** cdearborn has quit IRC18:10
*** piet has quit IRC18:15
*** daemontool has quit IRC18:18
jrolldtantsur|afk: liberty inspector stuff looks okay in terms of devstack/ipa changes, but py27 looks busted :( https://review.openstack.org/#/c/300581/18:20
jrollanyway, not going to block my stuff on that18:20
jrollI'm taking off, might check back in later but otherwise have a good weekend :)18:21
*** baoli_ has quit IRC18:22
*** garthb has joined #openstack-ironic18:22
*** vishwanathj has quit IRC18:22
NobodyCamhave a great weekend jroll18:24
*** piet has joined #openstack-ironic18:30
*** baoli has joined #openstack-ironic18:35
*** jaybeale has quit IRC18:39
*** ifarkas has quit IRC18:43
*** ChubYann has joined #openstack-ironic18:43
*** keedya has quit IRC18:45
*** Sukhdev has joined #openstack-ironic18:53
*** david-lyle has quit IRC18:53
*** piet has quit IRC18:56
*** piet has joined #openstack-ironic18:58
*** david-lyle has joined #openstack-ironic18:59
*** Goneri has quit IRC18:59
*** Sukhdev has quit IRC19:00
*** suro-patz has quit IRC19:03
*** piet has quit IRC19:03
*** suro-patz has joined #openstack-ironic19:11
*** mbound has joined #openstack-ironic19:11
*** jaybeale has joined #openstack-ironic19:18
*** baoli has quit IRC19:19
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/30062519:29
*** baoli has joined #openstack-ironic19:29
*** harshs has joined #openstack-ironic19:36
*** harlowja has joined #openstack-ironic19:37
*** baoli has quit IRC19:43
TheJuliaanyone know who maintains the irmc driver?19:45
*** Sukhdev has joined #openstack-ironic19:52
*** baoli has joined #openstack-ironic19:55
*** garthb_ has joined #openstack-ironic19:57
*** garthb has quit IRC20:00
*** Sukhdev has quit IRC20:08
*** mbound has quit IRC20:08
*** UForgotten has quit IRC20:12
JayFTheJulia: might wanna read mailing list20:27
JayFTheJulia: tl;dr: complaints about bifrost not making build artifacts20:27
*** dprince has quit IRC20:28
*** jayc has quit IRC20:32
*** raildo is now known as raildo-afk20:42
* jlvillal assumes TheJulia saw the email about Bifrost build artifacts20:48
* jlvillal also sees JayF is much faster than him...20:48
JayFwhen I'm working I tend to react to email boops very quickly, for better or worse20:49
* JayF inbox zero 4 lyfe20:49
jlvillal:)20:49
*** alexpilotti has quit IRC20:54
TheJuliaugh21:05
TheJuliaoh lovely21:08
TheJuliaI must be missing something deep inside the CI config that is just not clearly documented on project creation :\21:10
*** rbudden has quit IRC21:11
openstackgerritJarrod Johnson proposed openstack/pyghmi: Discard reference when no longer needed  https://review.openstack.org/30066021:11
*** trown is now known as trown|outtypewww21:11
*** devananda has quit IRC21:12
*** PhilipTW has quit IRC21:12
*** devananda has joined #openstack-ironic21:13
*** alexpilotti has joined #openstack-ironic21:15
*** mdorman has quit IRC21:16
*** jjohnson2 has quit IRC21:19
*** alexpilotti has quit IRC21:19
TheJuliaJayF: jlvillal thank you both21:20
JayFnp21:20
*** Sukhdev has joined #openstack-ironic21:29
*** Sukhdev has quit IRC21:31
*** Goneri has joined #openstack-ironic21:31
*** baoli has quit IRC21:36
*** daemontool has joined #openstack-ironic21:44
*** jayc has joined #openstack-ironic21:51
*** Sukhdev has joined #openstack-ironic21:52
openstackgerritJulia Kreger proposed openstack/ironic: deployment vmedia ops should not be run when not deploying  https://review.openstack.org/30067421:53
*** Sukhdev has quit IRC21:55
*** Sukhdev has joined #openstack-ironic22:00
*** daemontool has quit IRC22:03
*** Sukhdev has quit IRC22:07
*** rbrud has quit IRC22:12
*** lutf-8 is now known as alaski22:12
*** alexpilotti has joined #openstack-ironic22:17
*** alexpilotti has quit IRC22:22
*** mbound has joined #openstack-ironic22:24
*** Sukhdev has joined #openstack-ironic22:25
*** mbound has quit IRC22:25
*** mbound has joined #openstack-ironic22:25
openstackgerritJulia Kreger proposed openstack/ironic: Active Node Creation via adopt state  https://review.openstack.org/27576622:28
*** ametts has quit IRC22:28
*** achanda has joined #openstack-ironic22:29
*** suro-patz has quit IRC22:29
*** karimb has quit IRC22:32
* jlvillal only now sees jroll's April Fools patch :)22:39
*** Marga_ has quit IRC22:39
*** krtaylor has quit IRC22:44
JayFjlvillal: link?22:46
jlvillalJayF: https://review.openstack.org/30043422:47
*** ppiela_ has quit IRC22:49
*** daemontool has joined #openstack-ironic22:51
*** krtaylor has joined #openstack-ironic22:56
*** keedya has joined #openstack-ironic22:58
*** Marga_ has joined #openstack-ironic23:01
*** mgoddard has joined #openstack-ironic23:01
TheJulia2016-04-01 22:29:59.230 | /tmp/hudson1946662699213468843.sh: line 99: syntax error: unexpected end of file \o/23:01
jlvillalTheJulia: What do you mean?23:03
TheJuliaour gate is failing23:04
TheJulialooking at recent changes in project config right now23:04
JayFremember lib/ironic in devstack comes from us23:05
JayFso it could be something from ironic, although we'd have hoped that'd fail gate I guess23:05
*** Marga_ has quit IRC23:06
*** [1]cdearborn has joined #openstack-ironic23:06
TheJuliayeah, 99 makes me think it is coming out of something from jjb23:09
openstackgerritClint 'SpamapS' Byrum proposed openstack/bifrost: Add playbook to transfer nodes to another Ironic  https://review.openstack.org/29966023:09
TheJuliawell, more so the hudson and then line 9923:09
TheJuliahudson part23:09
JayFoh, gotcha23:11
JayFthat's the jeknins runner script23:11
*** mgoddard has quit IRC23:11
TheJuliayup, "./safe-devstack-vm-gate-wrap.sh" is apparently line 99 in the corresponding devstack-gate job that is running23:12
*** alexpilotti has joined #openstack-ironic23:13
*** mtanino has quit IRC23:13
*** Marga_ has joined #openstack-ironic23:14
*** alexpilotti has quit IRC23:16
*** alexpilotti has joined #openstack-ironic23:16
*** daemontool has quit IRC23:17
*** daemontool has joined #openstack-ironic23:18
*** [3]cdearborn has quit IRC23:19
TheJuliahmm.. nope... off by two lines :\23:20
*** trown|outtypewww is now known as trown23:21
*** alexpilotti has quit IRC23:21
*** dkehn has joined #openstack-ironic23:23
*** Sukhdev has quit IRC23:23
TheJuliahmm... can't run the extracted output...... same error23:23
TheJuliai see it23:30
*** harlowja has quit IRC23:36
*** daemontool has quit IRC23:38
TheJuliahttps://review.openstack.org/#/c/300694/ should fix the gate, etherpad updated.23:39
*** harshs has quit IRC23:40
*** jayc has quit IRC23:44
TheJuliaLooks like the fix is going to land in a little bit.  Would it be wrong to update the etherpad with "Everything is Awesome!"23:56
TheJulia?23:56
TheJulia:)23:56
JayFI'd just link to the patch and say "if this has landed; gate should be fixed"23:58
JayFand let someone mark it as happy once it actually is :P23:58
TheJuliaThen my work here is done for the night23:59
TheJulia:)23:59
JayFTheJulia: you'll appreciate this I think, if you wanna see something awesome23:59
TheJuliaoh?23:59
JayFTheJulia: I just replaced my keycaps on my das ultimate with this23:59

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