Thursday, 2015-06-18

*** naohirot has joined #openstack-ironic00:01
*** Marga_ has quit IRC00:02
*** yuikotakada has joined #openstack-ironic00:03
*** spandhe has quit IRC00:06
*** yuikotakada has quit IRC00:08
*** Marga_ has joined #openstack-ironic00:11
*** rloo has quit IRC00:16
*** penick has quit IRC00:16
*** smoriya has joined #openstack-ironic00:20
*** jamielennox|away is now known as jamielennox00:35
*** mtanino has quit IRC00:56
*** praneshp has joined #openstack-ironic00:56
*** praneshp has quit IRC00:56
*** jamielennox is now known as jamielennox|away01:00
*** jamielennox|away is now known as jamielennox01:08
*** mitchjameson has quit IRC01:21
*** Haomeng|2 has quit IRC01:23
*** Marga_ has quit IRC01:29
*** Marga_ has joined #openstack-ironic01:30
*** rbudden has quit IRC01:39
*** rbudden has joined #openstack-ironic01:39
*** rloo has joined #openstack-ironic01:52
*** rloo_ has joined #openstack-ironic01:54
*** rloo has quit IRC01:54
openstackgerritYuiko Takada proposed openstack/ironic-inspector: Load authenticate token from HTTP header  https://review.openstack.org/17423102:03
*** yuikotakada has joined #openstack-ironic02:03
openstackgerritJamie Lennox proposed openstack/python-ironicclient: Cleanup session creation  https://review.openstack.org/18585002:06
*** mtanino has joined #openstack-ironic02:07
*** rloo_ has quit IRC02:14
*** jamielennox is now known as jamielennox|away02:20
openstackgerritPeter Martini proposed openstack/bifrost: Move explicit URLs in ironic-install to defaults  https://review.openstack.org/19258002:23
*** Shrews has quit IRC02:39
*** Shrews has joined #openstack-ironic02:39
*** ramineni has joined #openstack-ironic02:42
openstackgerritMerged openstack/bifrost: Initial changes for testing VM creation for CentOS  https://review.openstack.org/19150402:51
*** Marga_ has quit IRC02:51
*** Marga_ has joined #openstack-ironic02:52
*** spandhe has joined #openstack-ironic02:53
openstackgerritTan Lin proposed openstack/ironic: Add a new boot section 'trusted_boot' for PXE  https://review.openstack.org/19166102:53
*** Marga_ has quit IRC03:01
*** jamielennox|away is now known as jamielennox03:03
*** jamielennox is now known as jamielennox|away03:15
*** lazy_prince has joined #openstack-ironic03:22
*** rbudden has quit IRC03:22
*** rbudden has joined #openstack-ironic03:23
*** jamielennox|away is now known as jamielennox03:25
*** Marga_ has joined #openstack-ironic03:31
*** Marga_ has quit IRC03:33
*** Marga__ has joined #openstack-ironic03:33
*** rbudden has quit IRC03:39
*** rbudden has joined #openstack-ironic03:39
*** penick has joined #openstack-ironic03:43
*** spandhe has quit IRC03:43
*** mtanino has quit IRC03:54
*** achanda has joined #openstack-ironic04:07
*** Marga__ has quit IRC04:10
*** korean101 has quit IRC04:14
*** achanda has quit IRC04:23
*** coolsvap|away is now known as coolsvap04:30
*** achanda has joined #openstack-ironic04:33
*** rbudden has quit IRC04:33
*** rbudden has joined #openstack-ironic04:33
*** achanda_ has joined #openstack-ironic04:39
*** achanda_ has quit IRC04:40
*** pradipta has joined #openstack-ironic04:40
*** achanda has quit IRC04:41
*** zz_natorious is now known as natorious04:43
*** boris-42 has joined #openstack-ironic04:44
*** natorious is now known as zz_natorious04:48
*** ramineni has quit IRC04:53
*** Marga_ has joined #openstack-ironic05:11
*** Marga_ has quit IRC05:16
*** jamielennox is now known as jamielennox|away05:23
*** jamielennox|away is now known as jamielennox05:26
*** coolsvap is now known as coolsvap|away05:28
*** coolsvap|away is now known as coolsvap05:36
*** chlong has quit IRC05:38
*** chlong has joined #openstack-ironic05:53
*** radek_ has joined #openstack-ironic05:56
*** Nisha has joined #openstack-ironic06:06
*** Marga_ has joined #openstack-ironic06:12
*** dtantsur|sick is now known as dtantsur06:13
dtantsurMorning Ironic!06:14
*** Marga_ has quit IRC06:16
*** spandhe has joined #openstack-ironic06:19
*** rameshg87 has joined #openstack-ironic06:20
rameshg87good morning ironic06:21
*** pradipta has quit IRC06:23
*** spandhe has quit IRC06:24
*** spandhe_ has joined #openstack-ironic06:24
dtantsurrameshg87, o/06:27
yuikotakadadtantsur, rameshg87: o/06:28
dtantsurhey yuikotakada! :)06:28
rameshg87dtantsur: yuikotakada: o/06:28
dtantsuryuikotakada, we got https://github.com/openstack/python-ironic-inspector-client yesterday, please keep an eye on it from now on as well :)06:29
*** mitchjameson has joined #openstack-ironic06:37
yuikotakadadtantsur, OK, I have not catch up with it at all. I'll check it from now :)06:38
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Add gitreview file  https://review.openstack.org/19298806:41
yuikotakadadtantsur, checking... It seems np, well, how did you decide port no? It's too dififcult to review lol06:43
dtantsuryuikotakada, you mean this patc above? I've copy-pasted ironic-inspector .gitreview and fixed the repo name :)06:45
dtantsurno magic actually06:45
yuikotakadadtantsur, yep, I mean so. Ah, that's true. I didn't know all projects have same port no (now I checked Nova's one also)06:48
openstackgerritMerged openstack/python-ironic-inspector-client: Add gitreview file  https://review.openstack.org/19298806:48
yuikotakadadtantsur, BTW, are you OK? You were sick yesterday, right?06:49
*** subscope has joined #openstack-ironic06:50
*** mmorais_ has quit IRC06:51
dtantsuryuikotakada, yeah thanks, I'm better now :)06:52
*** coolsvap is now known as coolsvap|away06:53
*** mmorais has joined #openstack-ironic06:53
yuikotakadadtantsur, fine :) take care.06:54
dtantsurthanks06:54
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Sync remaining changes from ironic-inspector tree  https://review.openstack.org/19298906:55
*** saripurigopi has joined #openstack-ironic07:01
*** coolsvap|away is now known as coolsvap07:01
*** spandhe_ has quit IRC07:03
*** e0ne has joined #openstack-ironic07:04
*** subscope has quit IRC07:05
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: After-moving code reorganization  https://review.openstack.org/19299307:05
*** penick has quit IRC07:07
yuikotakadadtantsur, I'm checking https://review.openstack.org/#/c/192989 , it seems np. then, how to handle https://review.openstack.org/#/c/192190/ ?07:07
*** mitchjameson has quit IRC07:07
dtantsuryuikotakada, ah good question, for some reason I thought we approved it already :) I'm going to approve them, and then drop client code in a separate patch. WDYT?07:08
*** jcoufal has joined #openstack-ironic07:09
yuikotakadadtantsur, yeah, I'm OK. just confused07:10
*** lazy_prince has quit IRC07:10
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: After-moving code reorganization  https://review.openstack.org/19299307:11
*** Marga_ has joined #openstack-ironic07:12
*** Marga_ has quit IRC07:17
*** chlong has quit IRC07:18
openstackgerritMerged openstack/ironic-inspector: Change the way error are returned from API to JSON  https://review.openstack.org/19219007:21
*** dasm|afk is now known as dasm07:23
*** ifarkas has joined #openstack-ironic07:23
*** e0ne has quit IRC07:23
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Basic support for API versions  https://review.openstack.org/19299607:28
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: After-moving code reorganization  https://review.openstack.org/19299307:30
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Basic support for API versions  https://review.openstack.org/19299607:30
*** subscope has joined #openstack-ironic07:30
openstackgerritMerged openstack/python-ironic-inspector-client: Sync remaining changes from ironic-inspector tree  https://review.openstack.org/19298907:30
*** coolsvap is now known as coolsvap|away07:34
*** jlvillal has quit IRC07:37
*** jrist has quit IRC07:44
*** jrist has joined #openstack-ironic07:44
*** Haomeng has joined #openstack-ironic07:45
*** mmorais has quit IRC07:47
*** mmorais has joined #openstack-ironic07:48
*** dlpartain has joined #openstack-ironic07:48
*** stendulker has joined #openstack-ironic07:48
*** boris-42 has quit IRC07:52
*** rbudden has quit IRC07:52
*** rbudden has joined #openstack-ironic07:52
*** yog_ has joined #openstack-ironic07:53
*** jlvillal has joined #openstack-ironic07:54
*** ramineni has joined #openstack-ironic07:54
*** athomas has joined #openstack-ironic07:58
*** gzhai has joined #openstack-ironic08:00
gzhaidtantsur:hello08:08
*** Nisha has quit IRC08:13
*** Marga_ has joined #openstack-ironic08:13
gzhaidtantsur: Do you have time to review my BP after modification? @  https://review.openstack.org/#/c/176109/08:13
*** dlpartain has left #openstack-ironic08:14
*** dguerri` is now known as dguerri08:15
dtantsurgzhai, hi, will do (a bit later today)08:16
*** Marga_ has quit IRC08:18
gzhaidtantsur: Thanks!08:18
*** sambetts_ has joined #openstack-ironic08:21
*** romainh has joined #openstack-ironic08:22
sambetts_Morning Ironic o/08:23
yuikotakadasambetts, o/08:24
*** MattMan has joined #openstack-ironic08:26
*** ndipanov has joined #openstack-ironic08:27
*** mdbooth has joined #openstack-ironic08:28
*** viktors|afk is now known as viktors08:28
*** lucas-dinner has joined #openstack-ironic08:28
dtantsursambetts_, morning!08:30
dtantsursambetts_, also FYI: we have https://github.com/openstack/python-ironic-inspector-client now \o/08:31
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Virtual Media Deploy module for iRMC Driver  https://review.openstack.org/15195808:31
openstackgerritMerged openstack/python-ironic-inspector-client: After-moving code reorganization  https://review.openstack.org/19299308:32
*** subscope has quit IRC08:32
*** Nisha has joined #openstack-ironic08:33
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Refactor node's and driver's vendor passthru to a common place  https://review.openstack.org/19012108:33
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Allow vendor methods to serve static files  https://review.openstack.org/18971608:33
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Change return value of [driver_]vendor_passthru to dict  https://review.openstack.org/19012008:33
*** lucas-dinner is now known as lucasagomes08:33
dtantsurlucasagomes, it was a long dinner, wasn't it? :D08:33
lucasagomesdtantsur, haha good morning08:34
lucasagomesit was indeed08:34
dtantsurmorning :)08:34
yuikotakadalucasagomes, g'morning08:34
lucasagomesyuikotakada, good ugt morning08:36
*** mdbooth has quit IRC08:39
*** derekh has joined #openstack-ironic08:40
*** mdbooth has joined #openstack-ironic08:40
*** yuikotakada is now known as yuikot_dinner08:40
*** yuikot_dinner has quit IRC08:40
*** zhenguo has quit IRC08:44
*** mdbooth has quit IRC08:45
*** sambetts_ has quit IRC08:46
*** e0ne has joined #openstack-ironic08:48
*** subscope has joined #openstack-ironic08:49
*** sambetts_ has joined #openstack-ironic08:53
*** mdbooth has joined #openstack-ironic08:53
openstackgerritNaohiro Tamura proposed openstack/ironic: Add localboot support for iscsi_irmc driver  https://review.openstack.org/16373808:54
openstackgerritAnusha Ramineni proposed openstack/ironic: Add option to configure passes in erase_devices  https://review.openstack.org/19175008:57
*** e0ne is now known as e0ne_08:58
*** pelix has joined #openstack-ironic09:00
*** e0ne_ is now known as e0ne09:00
sambetts_dtantsur: thats awesome we've got a client :D09:00
*** coolsvap|away is now known as coolsvap09:02
*** oomichi has quit IRC09:05
*** dtantsur is now known as dtantsur|brb09:05
openstackgerritNaohiro Tamura proposed openstack/ironic: Add whole disk image support for iscsi_irmc driver  https://review.openstack.org/18177109:07
*** romcheg has joined #openstack-ironic09:09
*** Marga_ has joined #openstack-ironic09:14
openstackgerritNaohiro Tamura proposed openstack/ironic: Add IPA support for iscsi_irmc driver  https://review.openstack.org/18180709:17
*** Marga_ has quit IRC09:19
openstackgerritAnusha Ramineni proposed openstack/ironic-python-agent: IPA:'shred' utility to use configured iterations  https://review.openstack.org/19175109:25
openstackgerritAnusha Ramineni proposed openstack/ironic-python-agent: IPA:'shred' utility to use configured iterations  https://review.openstack.org/19175109:26
openstackgerritAnusha Ramineni proposed openstack/ironic: Add option to configure passes in erase_devices  https://review.openstack.org/19175009:27
lucasagomesbtw gate is broken https://bugs.launchpad.net/ironic/+bug/146640509:35
openstackLaunchpad bug 1466405 in Ironic "broken ACL tests " [Undecided,New] - Assigned to Lucas Alvares Gomes (lucasagomes)09:35
lucasagomessomething changed in the dependecies I think and broke us09:35
*** Marga_ has joined #openstack-ironic09:39
*** Marga_ has quit IRC09:43
*** Marga_ has joined #openstack-ironic09:44
rameshg87lucasagomes: sambetts: morning09:46
rameshg87yeah gate seems broken :(09:46
lucasagomesrameshg87, it's the new version of keystonemiddleware that was release yesterday09:47
lucasagomes:-/09:47
* lucasagomes fixing09:47
openstackgerritNaohiro Tamura proposed openstack/ironic: Rename ilo_power._attach_boot_iso to improve readability  https://review.openstack.org/19304209:53
*** e0ne is now known as e0ne_09:57
*** naohirot has quit IRC09:59
*** e0ne_ is now known as e0ne09:59
*** jcoufal has quit IRC10:02
*** dtantsur|brb is now known as dtantsur10:08
*** yuanying has quit IRC10:10
dtantsurlucasagomes, so, keystonemiddleware.. I wonder if it broke inspector too...10:11
lucasagomesdtantsur, right idk10:11
lucasagomesI don't know what changed that broke ironic yet10:11
lucasagomesI suspect https://review.openstack.org/#/c/17419610:11
openstackgerritMerged openstack/python-ironic-inspector-client: Basic support for API versions  https://review.openstack.org/19299610:15
*** martini has joined #openstack-ironic10:16
martiniGood morning Ironic10:16
*** Nisha has quit IRC10:17
*** romainh has quit IRC10:18
*** achanda has joined #openstack-ironic10:20
MattManmorning ironic10:27
MattMandtantsur: I have a question related to inspection... specifically the naming of discovered properties10:29
MattManEssential properties that I can see are memory_gb, local_gb, cpus and cpu_arch right ?10:30
lucasagomesok found the problem in gate... fixing10:32
dtantsurMattMan, hi! yes (just memory_mb not memory_gb)10:33
MattManah ok thank10:33
MattManA driver can add any other properties it needs as well I presume10:34
dtantsurMattMan, that's properties Nova needs from us. We also find out MAC addresses (and probably LLDP info in the future)10:34
MattManFor instance SPARC hardware may have a small number of CPU's but it's the number of cores per CPU that makes the real difference, so we would need "cpu_cores"10:35
dtantsurMattMan, if your driver depends on something introspectable, of course you can do it10:35
MattMangreat10:35
dtantsurMattMan, note that cpus is used by Nova, not Ironic10:35
MattMannod10:35
*** saripurigopi has quit IRC10:36
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Switch to pbr  https://review.openstack.org/19305610:36
MattManNow where does capabilities fit into this /10:36
dtantsurMattMan, depending. you can introspect capabilities like ilo does: http://specs.openstack.org/openstack/ironic-specs/specs/kilo/ilo-properties-capabilities-discovery.html10:37
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Fix broken ACL tests  https://review.openstack.org/19305710:38
lucasagomes^ this is fixing gate10:38
lucasagomes(hopefully) so please cores take a look10:38
MattMandtantsur, is there a predefined list of capabilities as well ?10:38
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Switch to pbr  https://review.openstack.org/19305610:38
dtantsurMattMan, only what ilo folks decided for them10:39
*** saripurigopi has joined #openstack-ironic10:39
MattManok, from what I have been reading they can be set on the flavor and passed down to ironic in that manner.. I'll look at ilo driver to see if what they do makes sense for us as well... thanks for your help10:40
dtantsurlucasagomes, ouch, I believe we should not use it in tests10:40
lucasagomesdtantsur, remove those tests?10:40
lucasagomesdtantsur, I mean... we had it before, I'm just fixing it to be compatible with the new version10:40
dtantsurlucasagomes, remove anything touching keystonemiddleware internals10:40
dtantsurwhy do we even have them?10:40
lucasagomesdtantsur, I can't recall10:41
lucasagomesit seems to be old stuff10:41
dtantsurgood time to drop it ;)10:41
*** achanda has quit IRC10:41
lucasagomes71c2525d (Roman Prykhodchenko    2013-07-15 20:29:19 +0300  34) class FakeMemcache(object):10:41
lucasagomes201310:41
lucasagomes:-)10:41
lucasagomesdtantsur, right... I mean, let's fix it. And then we can discuss whether drop it or not10:42
dtantsurlucasagomes, fine10:43
dtantsurlemme know when tests pass :)10:43
lucasagomesdtantsur, ack, just as a FYI I'm having this same discussion in the keystone channel10:44
lucasagomes<samueldmq> lucasagomes, and yes, fixing that is fair enough for now10:44
lucasagomes<lucasagomes> samueldmq, yup10:44
lucasagomes<samueldmq> lucasagomes, nice10:44
lucasagomes<lucasagomes> yeah I will fix it and then bring it up to the community whether we should drop it or not10:44
lucasagomes<lucasagomes> I don't wanna fix the gate by deleting the tests, I think it's unfair10:44
lucasagomes<samueldmq> lucasagomes, yes it is, fix that to have the gate working asap, then revisit if they're really needed10:44
lucasagomes<lucasagomes> samueldmq, yes, I think it sounds like a good approach10:44
lucasagomes<lucasagomes> samueldmq, thanks much!10:44
dtantsurack10:44
*** athomas has quit IRC10:47
openstackgerritMerged openstack/python-ironic-inspector-client: Switch to pbr  https://review.openstack.org/19305610:47
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Manual update from global requirements  https://review.openstack.org/19305910:49
*** e0ne is now known as e0ne_10:50
*** coolsvap is now known as coolsvap|away10:56
*** athomas has joined #openstack-ironic10:56
*** Haomeng|2 has joined #openstack-ironic10:57
openstackgerritMerged openstack/python-ironic-inspector-client: Manual update from global requirements  https://review.openstack.org/19305910:57
dasmo/10:58
dasmi have an question about ironic baremetal provisioning. using rdo, i've encountered a problem with provisioning hosts.10:59
dasmin heat stack-show i see failure with traceback: "No valid host was found. Exceeded max scheduling "10:59
*** Haomeng has quit IRC10:59
*** Marga_ has quit IRC11:00
dasmin the mean time, nova-scheduler shows me sth like:11:00
*** e0ne_ has quit IRC11:00
dasm Selected host: WeighedHost [host: (gklab-44-126, 04672bd8-5e8e-41e9-be5d-8b89b4c13c8b) ram:262144 disk:1142784 io_ops:0 instances:0, weight: 0.666666666667] _schedule /usr/lib/python2.7/site-packages/nova/scheduler/filter_scheduler.py:15811:00
dasm2015-06-17 17:58:37.129 61895 WARNING nova.scheduler.host_manager [req-45297851-3cec-4052-84a5-7ca0d2e2c4ab 6fcd049dbb6c4eb4b6e30a1b58656c27 58d12f25b9664589875c097668963a54 - - -] Host 04672bd8-5e8e-41e9-be5d-8b89b4c13c8b has more disk space than database expected (1116gb > 1076gb)11:00
*** ramineni has quit IRC11:00
dasmis it connected, each other or just notification?11:00
dtantsurlucasagomes, cool, I've set https://launchpad.net/python-ironic-inspector-client maintainers to ironic-drivers by mistake and it locked me out of maitaining it11:01
dtantsurlucasagomes, as you're ironic-drivers member, could you revert it back to me?11:01
dasmhow ironic supposed to work, when starting baremetal provisioning? could anyone try to explain it in short words?11:01
lucasagomesdtantsur, hah sure11:01
* lucasagomes reverts11:01
lucasagomesdtantsur, you want me to set it to you directly? Or a specific group?11:02
dtantsurlucasagomes, to me for now11:02
lucasagomesthere's a inspected-members or something?11:02
lucasagomesack11:02
lucasagomesdtantsur, done11:02
dtantsurlucasagomes, thanks a lot!11:02
lucasagomesno problem11:02
*** Marga_ has joined #openstack-ironic11:03
lucasagomesdtantsur, the beauty of launchpad :D11:03
dtantsurlucasagomes, I will form a group and invite you there just in case, ok?11:03
lucasagomesdtantsur, ack11:03
* rameshg87 goes home11:06
*** rameshg87 has quit IRC11:06
*** Marga_ has quit IRC11:08
*** Marga_ has joined #openstack-ironic11:08
*** stendulker has quit IRC11:09
* dtantsur makes a release11:12
*** saripurigopi has quit IRC11:17
yuriyzmorning Ironic11:24
dtantsuryuriyz, morning11:24
yuriyzlucasagomes thanks for the unit tests fix but it's py3 incompatible11:25
*** sambetts_ has quit IRC11:28
*** jlvillal has quit IRC11:30
lucasagomesyuriyz, oh :-( fixing that too11:31
*** jlvillal has joined #openstack-ironic11:31
lucasagomesyuriyz, thanks for the review11:32
*** amotoki_ has joined #openstack-ironic11:33
yuriyzlucasagomes maybe you should set up local py34 env :)11:33
lucasagomesyeah I have too11:34
lucasagomesI'm using f20 yet, cause I'm a bit stuck so there's no py3411:34
lucasagomesplus unittests on py34 for newer versions of python34 are all broken11:34
lucasagomesdue a mock bug11:34
lucasagomessummarizing py34 is a piece of s*11:34
*** e0ne has joined #openstack-ironic11:36
openstackgerritMonty Taylor proposed openstack/python-ironic-inspector-client: Add venv environment for tox  https://review.openstack.org/19307811:36
*** romainh has joined #openstack-ironic11:46
TheJuliaHaomeng|2: By chance are you awake?11:49
*** Marga_ has quit IRC11:52
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Add venv environment for tox  https://review.openstack.org/19307811:55
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Use pbr without explicit version  https://review.openstack.org/19308711:55
*** jrist has quit IRC11:56
*** jrist has joined #openstack-ironic11:56
*** Marga_ has joined #openstack-ironic11:57
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Fix broken ACL tests  https://review.openstack.org/19305711:57
lucasagomesyuriyz, ^ thanks for the suggestions inline11:58
openstackgerritMerged openstack/python-ironic-inspector-client: Use pbr without explicit version  https://review.openstack.org/19308712:01
*** achanda has joined #openstack-ironic12:02
openstackgerritMerged openstack/python-ironic-inspector-client: Add venv environment for tox  https://review.openstack.org/19307812:02
openstackgerritMerged openstack/bifrost: Fix when logic for boolean handling  https://review.openstack.org/19197912:04
TheJuliaYay, merge conflicts :(12:05
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Explicitly depend on pbr  https://review.openstack.org/19309012:08
*** trown|outttypeww is now known as trown12:08
martinidang - I rebased ~14 hours ago (at least I think I did)12:08
*** dprince has joined #openstack-ironic12:09
martiniOh, I *definitely* need to add docs on how to clone ansible offline.12:09
*** yuikotakada has joined #openstack-ironic12:10
martiniIt's a bit tedious because of the submodules - git clone recursive will clone the top level locally, then try to clone the submodules from github.  What I end up doing each time I do this is: cd /opt/stack; git clone /vagrant/git/ansible; cd ansible; cp /vagrant/gitmodules .gitmodules; git submodule init; git submodule update; git checkout -- .gitmodules; then go back and env_setup works fine12:11
openstackgerritJulia Kreger proposed openstack/bifrost: Move explicit URLs in ironic-install to defaults  https://review.openstack.org/19258012:12
TheJuliamartini: I merged a fix for a nasty issue that had a neighboring line change12:12
TheJuliaso... merge conflcit :(12:12
martiniAh, such is timing :-)12:13
TheJuliaI've heard submodules are a nightmare at times, but have never really messed with them.12:13
TheJuliawell, one +2 :)12:15
*** Shrews has quit IRC12:17
openstackgerritMerged openstack/python-ironic-inspector-client: Explicitly depend on pbr  https://review.openstack.org/19309012:18
*** Shrews has joined #openstack-ironic12:18
*** jjohnson2 has joined #openstack-ironic12:25
*** Marga_ has quit IRC12:27
*** e0ne is now known as e0ne_12:32
openstackgerritZhenguo Niu proposed openstack/ironic: When boot option is not persisted, set boot on next power on  https://review.openstack.org/17764212:34
*** radek_ has quit IRC12:36
*** radek_ has joined #openstack-ironic12:37
*** e0ne_ is now known as e0ne12:39
openstackgerritJarrod Johnson proposed stackforge/pyghmi: Add monotonic time handling for Windows  https://review.openstack.org/19310812:50
openstackgerritJarrod Johnson proposed stackforge/pyghmi: Add monotonic time handling for Windows  https://review.openstack.org/19310812:51
openstackgerritMerged stackforge/pyghmi: Acknowledge PET alert on decode  https://review.openstack.org/18939212:52
*** rloo has joined #openstack-ironic12:57
*** smoriya has quit IRC13:01
openstackgerritJulia Kreger proposed openstack/bifrost: WIP: Change job logging such that successful jobs save logs  https://review.openstack.org/19311413:06
*** pradipta has joined #openstack-ironic13:11
*** achanda has quit IRC13:12
*** athomas_ has joined #openstack-ironic13:15
openstackgerritLucas Alvares Gomes proposed openstack/ironic: API: Get a subset of fields from Ports and Chassis  https://review.openstack.org/18354913:19
openstackgerritLucas Alvares Gomes proposed openstack/ironic: API: Get a subset of fields from Nodes  https://review.openstack.org/18349113:19
lucasagomesjroll, dtantsur rloo if you guys are around, please take a look at https://review.openstack.org/#/c/193057/13:20
openstackgerritChristopher Dearborn proposed openstack/ironic: Add DRAC BIOS config vendor passthru API  https://review.openstack.org/15048813:20
* dtantsur looks13:20
lucasagomesto unblock the gate, tests are passing13:20
rloolucasagomes, dtantsur: hi13:20
rloolucasagomes: you're too fast for me. I just replied to an older version of 183491!13:21
rloolucasagomes: will look13:21
lucasagomesrloo, oh hah ack13:21
dtantsurrloo, hi!13:21
rloolucasagomes: so for 193057, we keep supporting keystonemiddleware until the requirements change to >= 2.0.0?13:25
*** yuikotakada has quit IRC13:25
*** athomas_ has quit IRC13:26
rloolucasagomes: I mean keep supporting the non-hash-vesion13:26
lucasagomesrloo, yeah pretty much... currently the requirements.txt is >=1.5.013:26
lucasagomesso we gotta support the non-hash13:26
lucasagomesrloo, yeah13:26
rloolucasagomes: ok, might have been nice to have a comment about removing the other stuff later, but dtantsur has already approved :)13:26
lucasagomesI don't think people are pinning the global requirements to >=2.0.0 just yet tho, cause it doesn't break all projects. It broke Ironic and Ceilometer AFAICT13:27
lucasagomesbecause we do ACL tests13:27
lucasagomesapparently other projects doesn't bother on doing it13:27
NobodyCammorning from grants Pass Ironicers13:27
lucasagomesso it just works for them13:27
rloolucasagomes: yay for us! sort of ;)13:27
lucasagomesyeah13:27
rloomorning NobodyCam13:28
lucasagomesrloo, well we also could think about removing it tho, it's sorta of a functional test that we have there13:28
rloolucasagomes: we can't as long as it isn't pinned to >2.0.013:28
lucasagomesbut let's fix first to unblock the gate and then talk about whether we should remove it or not13:28
rloo>= I mean13:28
lucasagomesrloo, I mean removing the whole tests13:29
lucasagomesfor ACL13:29
rloolucasagomes: oh. hmm. hadn't thought of why we have the acl tests in the first place. if it is just testing keystone... yeah, do we need it.13:29
lucasagomesyeah it's old stuff from 201313:30
dtantsurNobodyCam, morning!13:30
lucasagomesI also don't rememeber the reasonings about having it13:30
NobodyCammorning rloo lucasagomes and dtantsur13:30
lucasagomesrloo, and tempest will test it for us, we probably don't need to bother having it on unittest13:31
lucasagomesbut anyway13:31
lucasagomes...13:31
lucasagomesNobodyCam, good ugt morning13:31
NobodyCam:)13:32
lucasagomesrloo, re 183491 replied to ur replies13:32
lucasagomesthanks13:32
rloolucasagomes: ok. yw :)13:32
lucasagomesI've most changed the names according to your suggestions before _DEFAULT_RETURN_FIELDS and check_allow_specify_fields()13:33
rloolucasagomes: btw, I think you mean 'polling', not 'pooling'13:33
lucasagomesand agreed with chassis_id being PITA13:33
lucasagomes:-( yup I do13:34
* lucasagomes fix the bug13:34
rloolucasagomes: i noticed in the nova bug but forgot to mention it to you13:34
*** Marga_ has joined #openstack-ironic13:34
lucasagomesrloo, thx, fixing both13:34
dtantsurlucasagomes, can you convince me not to -1 you for check_allow_specify_fields? ;)13:35
rloolucasagomes: ok, i'm fine with your comments. will wait for jenkins first, i haven't yet looked at the test changes13:36
* dtantsur is not fond of "let's blindly copy Nova" approach13:36
lucasagomesdtantsur, heh I probably can't... But so far I think people in the API-WG have being towards to agree that we should bump the micro version for new features too13:37
lucasagomesdtantsur, well yeah, but we should follow the API-WG it was created to make APIs more consistent13:37
dtantsurlucasagomes, I'm not asking about bumping the microversion13:37
lucasagomesno reason to diverge from it13:37
dtantsurlucasagomes, reason: it's insane and they ignore critics?13:37
rloodtantsur: i think until we come to an agreement, we should continue to do what we've been doing13:37
lucasagomesdtantsur, right yeah, but I will keep doing as we did before13:37
lucasagomesif turns out we should not do it we can remove13:38
dtantsur*facepalm*13:38
lucasagomesjust like we would remove it from name and so on13:38
dtantsurok, do whatever you want13:38
* dtantsur removes patch from his watch list13:38
lucasagomesheh fair enuff13:38
lucasagomesdtantsur, also jroll was talking about it yesterday, threating micro version just like releases of a library13:38
dtantsurit's clearly not even remotely the same13:39
lucasagomescause you don't bump a version of a lib and release it when things are not backward compat13:39
rloodtantsur: it seems like there are several 'issues'. 1. What the guidelines will be; 2. whether Ironic will follow all/some of those guidelines; 3. whether ironic agrees with you13:39
lucasagomesyeah well it's an analogy13:39
dtantsurlucasagomes, "bump a version of a lib and release it when things are not backward compat" is exactly what everyone does and will do13:40
lucasagomesdtantsur, they also bump the version and release when new features are added13:40
lucasagomesthat's the thinking13:40
dtantsurlucasagomes, we also bump version and release, but how is it related to API?13:41
dtantsurAPI is not library. it's rather like requiring #define API_VERSION 10 to use new features. it's done extremely rarely13:41
lucasagomesdtantsur, it was an analogy13:42
lucasagomesjroll, may be able to explain it better...13:42
dtantsurwell, analogies are nice, but I'm still waiting for at least one use case for that. without use case we're jsut doing additional work13:43
dtantsurlike rloo says: do everything, unless it requires me to do more work :)13:43
lucasagomesheh13:43
lucasagomesyeah I understand, we are in the middle of the discussions about when to bump the micro version etc... so for now I will continue to do what we did before for other things13:44
lucasagomesuntil we get a consensus13:44
rloodtantsur: ++. That's partly why I'm ignoring, not participating in the decision of what to do. that means having to think about it etc.13:44
dtantsuri.e. I'm not against us playing weird toys, I'm against it making Ironic harder for contributors and for reviewers :)13:44
rloodtantsur: sometimes I prefer to be a lamb.13:44
lucasagomesdtantsur, sure, I understand13:44
lucasagomesbut in doubt fall back to the current default13:45
lucasagomesso... that's what I did13:45
rloodtantsur: "just tell me what to do so we can get the code in and things are consistent" :)13:45
dtantsurthe current default, to be honest, was not properly decided and properly approved by the tean13:45
* lucasagomes brb coffee13:46
dtantsurok, the last thing that pisses me off is that people are talking about making developers happy. I am the developer for Ironic, but they ignore me :)13:46
dtantsurand enough with this, time to get some job done :)13:46
rloodtantsur: agreed (wrt lack of decision process)13:46
rloodtantsur: i didn't realize they wanted to make developers happy. if they wanted to do that, we should let everyone merge their code.13:47
rloodtantsur: except that might make me unhappy. hmm, guess we can't make everyone happy.13:48
trownlol13:48
trownindeed13:48
trowngood ugt moring Ironic13:48
dtantsurheh, merging random code won't make developers happy :)13:48
dtantsurtrown, morning :)13:48
lucasagomestrown, morning13:48
dtantsurdidn't I say g'morning to you already?13:48
NobodyCammorning (UGT) trown13:49
NobodyCamlol13:49
rloodtantsur: we should ask on the mailing list. if any non-cores reply, it might be interesting to read.13:49
trowndtantsur: ya, in #rdo :p13:49
rloohi trown13:49
trowngood morning rloo lucasagomes NobodyCam13:49
openstackgerritMerged openstack/bifrost: Fix DHCP conditionals  https://review.openstack.org/19217413:49
dtantsurrloo, if we make even majority of cores reply to anything on ML, I would call it a great success :D13:49
rloodtantsur: oh, I rarely reply unless I feel strongly about something and no one else has commented. I don't see it as a voting thing (others do I guess) and I feel like it adds more noise if I reply to say I agree, so I don't bother.13:51
*** achanda has joined #openstack-ironic13:52
openstackgerritMerged openstack/bifrost: Allow a user to define the OS for the Disk Image  https://review.openstack.org/19198013:52
lucasagomesdtantsur, ignoring ur comments... Have you posted it on the microversion review ?13:56
lucasagomesI think you will get some input from there instead of ML13:56
lucasagomes(just an idea, I haven't actually checked if you did or not)13:56
openstackgerritMerged openstack/ironic: Fix broken ACL tests  https://review.openstack.org/19305713:57
lucasagomesw00t gate is back13:57
*** Marga_ has quit IRC13:58
rloolucasagomes: wow, that was fast!13:59
lucasagomesrloo, yeah, well took me a couple of hours in the morning13:59
lucasagomesto actually find the problem13:59
rloolucasagomes: oh yeah, thank you for doing that :D14:00
lucasagomesthanks to some folks in the keystone channel helped with it14:00
NobodyCamlucasagomes: :) w00t14:01
* lucasagomes opens a beer14:01
lucasagomesjk :-( I wish14:01
*** achanda has quit IRC14:02
rloolucasagomes: fwiw, I was thinking that we should pay more attention to new library releases. so I looked at the email - release for keystonemiddleware, and I would not have known from that, if anything would break.14:02
*** achanda has joined #openstack-ironic14:03
*** Marga_ has joined #openstack-ironic14:03
*** achanda has quit IRC14:03
lucasagomesrloo, yeah, problem is to know when it will break u or not... I think the new library release email is good to check when things break14:04
rloolucasagomes: it would have been useful if there had been a 'non-backwards-compat' section in the email or somewhere...14:05
dtantsurlucasagomes, the fun thing is that nobody openly says about hiding the features. the guidelines do not contain it, it's considered to be implied :)14:05
lucasagomesrloo, that would be awesome!14:05
lucasagomesrloo, I'm not sure people are actually tracking it... perhaps once a change is merged which is non backward compat14:06
lucasagomesthey could send an email14:06
lucasagomessame for us14:06
dtantsurhmm, interesting14:06
dtantsurlucasagomes, the problem is, changing internals is not considered a breaking change14:06
lucasagomesyeah14:07
rloolucasagomes: i was thinking since dhellmann (or whoever) sends out an email about releases, that the info should be in that email or linked from it.14:07
lucasagomesfor this case14:07
lucasagomesdtantsur, +114:07
lucasagomesrloo, right could be, people would need to add this info somewhere as things are getting merged so they don't forget14:07
rloolucasagomes: true. but if you're going to release something with a major version bump...14:08
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Switch to pbr  https://review.openstack.org/19314514:10
dtantsurcan't believe I'm doing it ^^^14:10
dtantsur:)14:10
lucasagomesrloo, yeah... we could think about something along those lines for Ironic14:10
lucasagomesfor our new release model14:10
lucasagomeseverytime we release something, put up an email with the changes and so on14:10
rloolucasagomes: oh, yeah, whoever is handling the releases should let the rest of us know what they might want :)14:11
lucasagomeshah fair14:12
trowndtantsur: haha14:12
trowndtantsur: I tried the 'postversioning', and I like it better14:13
trownseems more intuitive to me14:13
dtantsurtrown, a bit annoying that your version will be patch one, until you tag14:14
dtantsuri.e. it won't understand that 2.1.0 will follow 2.0.0 on master14:15
dtantsurby default14:15
trownya I suppose for stuff that is > 1.0.0, it makes more sense for preversion...you know what you want in the next version14:17
trownI kept screwing up preversion when packaging though14:18
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Move client out of tree  https://review.openstack.org/19315014:18
*** bethelwell has joined #openstack-ironic14:19
dtantsursambetts, are we calling https://blueprints.launchpad.net/ironic-inspector/+spec/node-not-found-hook done?14:22
*** dasm is now known as dasm|afk14:23
* dtantsur assumes we do14:24
openstackgerritMerged openstack/ironic-inspector: Switch to pbr  https://review.openstack.org/19314514:26
*** rameshg87 has joined #openstack-ironic14:31
rameshg87morning folks14:32
thiagopmorning rameshg8714:32
thiagoprameshg87:  I did some rebase on your patch, have you seen it?14:33
rameshg87thiagop: o/14:33
* rameshg87 checks14:33
rameshg87thiagop: I saw that. looks like jenkins has failed.14:33
rameshg87thiagop: simple rebase might not help. more changes might be needed.14:33
thiagopYeah, some tests are failing because of the mocks on PXEDeploy14:34
rameshg87thiagop: I see all pxe_ssh tests also failed14:34
thiagoprameshg87: in fact, I think it needs a rework on tests to see how it'll work with the new interface14:34
rameshg87thiagop: AttributeError: 'PXEBoot' object has no attribute 'tear_down_deploy_ramdisk_boot'14:35
NobodyCammorning rameshg8714:35
rameshg87thiagop: yeah, the methods need to synced as per the spec14:35
rameshg87NobodyCam: o/14:35
thiagoprameshg87: This one I haven't seen...14:35
rameshg87thiagop: http://logs.openstack.org/13/166513/5/check/check-tempest-dsvm-ironic-pxe_ssh/d5bb66b/logs/screen-ir-cond.txt.gz14:35
thiagoptks14:36
rameshg87thiagop: I will plan to put up a new patch today or tomorrow for pxe driver14:37
*** e0ne is now known as e0ne_14:37
*** e0ne_ is now known as e0ne14:37
rameshg87thiagop: I see you are waiting for this ? is there some dependency on your work on this ?14:37
thiagoprameshg87: Nice. I have some internal issues to take care of today, but I can help as soon as it ends.14:38
thiagopYes, I intend to implement the OneView driver compliant to the new interface14:38
rameshg87thiagop: oh okay14:38
jrolldtantsur: I haven't read all of the scrollback, but I think when folks talk about developers and API versioning, they are talking about developers that are developing against the API, not ironic developers14:39
dtantsurjroll, I'm both :)14:39
jrolldtantsur: aha, true :)14:39
jrollfair enough.14:39
jrollalso good morning :)14:41
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Override PXE kernel cmdline via Glance property  https://review.openstack.org/17822914:42
lucasagomesjroll, good morning14:43
lucasagomesugt14:43
jroll\o14:43
rameshg87lucasagomes: hi there14:45
rameshg87lucasagomes: In https://review.openstack.org/#/c/178229/14/ironic/drivers/modules/pxe.py L35114:45
rameshg87lucasagomes: I meant someone could still give boot_option == 'local' and give a glance image that has custom kernel_cmdline14:46
lucasagomesrameshg87, hi there14:47
* lucasagomes checks14:47
lucasagomesrameshg87, right but that glance metadata will be copied to the instance_info right?14:47
*** romcheg has quit IRC14:48
rameshg87lucasagomes: but only after validate() right ?14:48
* rameshg87 checks14:48
* lucasagomes checks too14:49
*** mtanino has joined #openstack-ironic14:49
rameshg87lucasagomes: yeah it's in prepare14:49
rameshg87lucasagomes: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L39814:50
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Move client out of tree  https://review.openstack.org/19315014:50
lucasagomesrameshg87, oh right... I will fix that14:50
rameshg87okay14:50
*** zz_jgrimm is now known as jgrimm14:51
*** romcheg has joined #openstack-ironic14:51
*** romainh has left #openstack-ironic14:54
lucasagomesrameshg87,  _build_pxe_config_options() will be called after deploy() right?14:55
*** romainh has joined #openstack-ironic14:56
lucasagomesso the instance_info will be populated14:56
* lucasagomes looks14:56
*** rameshg87 has quit IRC14:57
lucasagomesrameshg87, problem with validate() it's not only called when the node is being dpeloyed14:57
lucasagomesit can be called any time, even on our API14:57
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Override PXE kernel cmdline via Glance property  https://review.openstack.org/17822915:00
lucasagomes^ just remove some unrelated changes from pxe.py15:00
NobodyCammorning jroll15:01
jrollheya NobodyCam15:01
NobodyCam:)15:01
jroll:D15:01
openstackgerritDmitry Tantsur proposed openstack/ironic-specs: Switch periodic tasks to the Futurist library  https://review.openstack.org/19171015:02
*** achanda has joined #openstack-ironic15:04
*** e0ne has quit IRC15:04
*** boris-42 has joined #openstack-ironic15:10
*** subscope has quit IRC15:11
*** achanda has quit IRC15:13
*** rwsu has joined #openstack-ironic15:18
*** subscope has joined #openstack-ironic15:26
openstackgerritDmitry Tantsur proposed openstack/ironic: Finish switch to inspector and inspector-client  https://review.openstack.org/19317615:26
dtantsurfolks, I will appreciate fast review of ^^^ very much, otherwise it's blocking getting client out of inspector itself (aka https://review.openstack.org/193150)15:27
openstackgerritMerged openstack/ironic: Add Wake-On-Lan driver documentation  https://review.openstack.org/19225215:28
trowndtantsur: I will give it a look shortly15:28
dtantsurthnx!15:28
*** spandhe has joined #openstack-ironic15:30
*** Marga_ has quit IRC15:31
*** achanda has joined #openstack-ironic15:39
*** mdbooth has quit IRC15:40
*** achanda_ has joined #openstack-ironic15:41
* dtantsur has new bass pickups to install, so he's heading home :)15:41
dtantsursee you tomorrow15:41
NobodyCamahhh15:42
NobodyCamdtantsur: i was just going to ask if you had a quick second to look at https://review.openstack.org/#/c/19110915:42
NobodyCamI ask because you have already lookd at it :)15:43
openstackgerritMerged openstack/bifrost: Move explicit URLs in ironic-install to defaults  https://review.openstack.org/19258015:43
*** achanda has quit IRC15:44
*** mdbooth has joined #openstack-ironic15:45
dtantsurNobodyCam, I'm not that good in bifrost anyway :)15:45
NobodyCam:) hehehe15:45
NobodyCamdtantsur: your just "good"15:46
NobodyCam:-p15:46
dtantsurlol true :D15:46
*** dtantsur is now known as dtantsur|afk15:46
*** ifarkas has quit IRC15:47
NobodyCamhave a goodnight dtantsur|afk :)15:47
*** achanda_ has quit IRC15:51
*** achanda has joined #openstack-ironic15:52
*** Marga_ has joined #openstack-ironic15:56
*** kozhukalov_ has joined #openstack-ironic15:57
*** victor_lowther has left #openstack-ironic15:59
devanandag'morning, all16:02
lucasagomesdevananda, morning16:02
NobodyCammornign devananda16:02
*** ukalifon has joined #openstack-ironic16:04
openstackgerritMerged openstack/ironic: Validate capability in properties and instance_info  https://review.openstack.org/17478516:07
*** zz_natorious is now known as natorious16:08
*** jjohnson2 has quit IRC16:13
openstackgerritJulia Kreger proposed openstack/bifrost: Change job logging such that successful jobs save logs  https://review.openstack.org/19311416:17
NobodyCambrb16:17
openstackgerritMerged openstack/bifrost: Support for AMT  https://review.openstack.org/19110916:20
*** ukalifon has quit IRC16:20
*** openstackgerrit has quit IRC16:22
*** openstackgerrit has joined #openstack-ironic16:23
*** ijw_ has joined #openstack-ironic16:25
*** ukalifon1 has joined #openstack-ironic16:25
*** ukalifon1 has quit IRC16:27
*** romcheg has quit IRC16:28
*** rbudden has quit IRC16:32
*** rbudden_ has joined #openstack-ironic16:32
*** rbudden_ has quit IRC16:33
*** Marga_ has quit IRC16:34
*** Marga_ has joined #openstack-ironic16:34
*** rbudden has joined #openstack-ironic16:37
*** yog_ has quit IRC16:39
*** ukalifon has joined #openstack-ironic16:41
*** teju has joined #openstack-ironic16:42
*** romcheg has joined #openstack-ironic16:43
*** Marga_ has quit IRC16:43
*** Marga_ has joined #openstack-ironic16:43
*** derekh has quit IRC16:48
*** teju has quit IRC16:49
jrolldevananda: welcome back? :)16:51
devanandajroll: thanks :)16:51
devanandait's nice to be back in seattle. I know I can eat the food here :)16:51
lucasagomesheh16:54
lucasagomesyeah that's important16:54
lucasagomesanyhoo, folks I'm calling it a day16:54
lucasagomescame to the office so gotta catch the train back home yet16:54
lucasagomeshave a good night everyone!16:54
jrolllater lucasagomes :)16:55
lucasagomessee y'all16:55
*** lucasagomes has quit IRC16:55
NobodyCamhave a good night lucas16:55
*** ijw_ has quit IRC17:00
*** bethelwell has quit IRC17:01
*** ndipanov has quit IRC17:01
openstackgerritVladyslav Drok proposed openstack/ironic: Add pluggable credentials storage  https://review.openstack.org/18507417:04
*** ijw_ has joined #openstack-ironic17:04
*** pradipta has quit IRC17:06
openstackgerritMerged openstack/ironic-specs: Minor grammar cleanup of enroll-node-state  https://review.openstack.org/19196717:06
*** achanda has quit IRC17:08
openstackgerritVladyslav Drok proposed openstack/ironic: Add credentials migration script.  https://review.openstack.org/19321817:08
*** mitchjameson has joined #openstack-ironic17:11
*** achanda has joined #openstack-ironic17:11
*** ijw_ has quit IRC17:13
*** ijw_ has joined #openstack-ironic17:14
*** krtaylor has quit IRC17:14
*** krtaylor has joined #openstack-ironic17:15
*** jjohnson2 has joined #openstack-ironic17:15
*** natorious is now known as zz_natorious17:16
*** amotoki_ has quit IRC17:18
openstackgerritVladyslav Drok proposed openstack/ironic-specs: Add pluggable credentials storage  https://review.openstack.org/18605617:18
*** achanda has quit IRC17:19
openstackgerritVladyslav Drok proposed openstack/ironic: Add credentials migration script  https://review.openstack.org/19321817:23
*** Marga_ has quit IRC17:23
*** achanda has joined #openstack-ironic17:24
*** dguerri is now known as dguerri`17:30
*** rbudden has quit IRC17:30
*** rbudden has joined #openstack-ironic17:30
martiniTheJulia: just realized I can post another commit to update the README for the ironic-install role ;-)17:34
martiniTheJulia: and I meant to ask, it looks like latest_os_ironic_url and latest_os_ironic_node_url aren't actually used anymore, just by a quick grep.  (I didn't do a run through to delete them).  Were those made irrelevant by some refactor, or am I missing something silly?17:35
*** trown is now known as trown|lunch17:38
TheJuliamartini: oh, please rip them out, they were made irrelevant and I keep forgetting17:38
*** zz_natorious is now known as natorious17:39
martiniCool, will do this evening17:39
TheJuliaawesome17:39
TheJuliaI have to step away, but I'll be back on this evening as well.17:39
*** athomas has quit IRC17:42
*** absubram has joined #openstack-ironic17:46
*** openstackgerrit has quit IRC17:50
*** openstackgerrit has joined #openstack-ironic17:51
*** marzif has joined #openstack-ironic17:52
*** Marga_ has joined #openstack-ironic17:54
*** Marga_ has quit IRC17:59
*** Marga_ has joined #openstack-ironic18:07
openstackgerritShilla Saebi proposed openstack/ironic: changes and edit of the ironic install guide  https://review.openstack.org/19190018:10
*** achanda has quit IRC18:15
*** zer0c00l has quit IRC18:18
*** zer0c00l has joined #openstack-ironic18:18
*** amotoki_ has joined #openstack-ironic18:18
*** zer0c00l has quit IRC18:18
*** zer0c00l has joined #openstack-ironic18:19
openstackgerritMerged openstack/ironic: Enable flake8 checking of ironic/nova/*  https://review.openstack.org/19118318:19
*** pelix has quit IRC18:23
*** amotoki_ has quit IRC18:23
openstackgerritRichard H proposed openstack/ironic: Copy editing of ironic docs  https://review.openstack.org/19288918:28
*** _gridinv has joined #openstack-ironic18:28
*** kozhukalov_ has quit IRC18:37
*** marzif has quit IRC18:37
*** marzif has joined #openstack-ironic18:40
*** marzif has quit IRC18:47
*** Sukhdev has joined #openstack-ironic18:49
openstackgerritJeremy Stanley proposed openstack/ironic: Switch from MySQL-python to PyMySQL  https://review.openstack.org/18437818:50
JayFjroll: ^ talk about a patch we'll want to test well before going for it18:52
jrollJayF: ohhhh ya.18:53
jrollJayF: that patch is just docs/tox though :)18:53
*** spandhe has quit IRC18:54
*** Marga_ has quit IRC18:55
*** Marga_ has joined #openstack-ironic18:55
*** ijw__ has joined #openstack-ironic18:57
*** ijw_ has quit IRC19:00
*** bethelwell has joined #openstack-ironic19:02
*** bethelwell has quit IRC19:07
*** rloo has quit IRC19:10
*** rloo has joined #openstack-ironic19:11
*** romcheg has quit IRC19:11
openstackgerritDevananda van der Veen proposed openstack/ironic-specs: Add spec for user-settable Node affinity  https://review.openstack.org/19326319:13
devanandaJayF: ^19:13
*** rloo_ has joined #openstack-ironic19:16
*** ukalifon has quit IRC19:18
*** rloo_ has quit IRC19:18
*** rloo_ has joined #openstack-ironic19:18
*** romcheg has joined #openstack-ironic19:18
*** romainh has quit IRC19:19
*** rloo has quit IRC19:20
*** ukalifon has joined #openstack-ironic19:20
*** _gridinv has quit IRC19:22
*** uschreiber_ has joined #openstack-ironic19:23
*** uschreiber_ has quit IRC19:25
*** lauramoore has joined #openstack-ironic19:26
*** ukalifon has quit IRC19:34
*** Marga_ has quit IRC19:37
*** dprince has quit IRC19:45
*** r-daneel has joined #openstack-ironic19:47
*** Marga_ has joined #openstack-ironic19:47
*** Marga_ has quit IRC19:53
*** trown|lunch is now known as trown19:53
*** achanda has joined #openstack-ironic19:54
openstackgerritMathieu Mitchell proposed openstack/ironic-python-agent: Use host_device mode for qemu-img  https://review.openstack.org/19328419:58
openstackgerritMathieu Mitchell proposed openstack/ironic-python-agent: Use host_device mode for qemu-img  https://review.openstack.org/19328420:00
*** jjohnson2 has quit IRC20:02
*** natorious is now known as zz_natorious20:07
*** ijw__ has quit IRC20:25
*** achanda has quit IRC20:30
*** thiagop has quit IRC20:32
openstackgerritDevananda van der Veen proposed openstack/ironic-specs: Update nodes-tagging spec  https://review.openstack.org/19293520:36
*** marzif has joined #openstack-ironic20:41
*** sambetts has quit IRC20:43
*** Marga_ has joined #openstack-ironic20:44
*** sambetts has joined #openstack-ironic20:46
mrdaMorning20:49
*** Sukhdev has quit IRC20:49
*** e0ne has joined #openstack-ironic20:59
openstackgerritJoshua Harlow proposed openstack/ironic: Transition state machine to use automaton oslo lib.  https://review.openstack.org/19198520:59
*** jamielennox is now known as jamielennox|away21:01
*** marzif has quit IRC21:03
rloo_morning mrda :)21:06
devanandamorning!21:06
mrdao/21:07
*** jamielennox|away is now known as jamielennox21:09
*** thiagop has joined #openstack-ironic21:16
*** Marga_ has quit IRC21:23
*** ijw_ has joined #openstack-ironic21:25
*** trown is now known as trown|outttypeww21:26
*** Marga_ has joined #openstack-ironic21:29
*** Marga_ has quit IRC21:34
*** Marga_ has joined #openstack-ironic21:34
rloo_so ironic's official name & capitalization is 'Bare metal service'. That seems odd to me. https://wiki.openstack.org/wiki/Documentation/Conventions#Service_and_project_names21:37
JayFYeah; shouldn't it be Bear?21:37
rloo_which means we'd see something like 'The Bare metal service is really useful' vs 'The Bare Metal service is even better'? Yeah, maybe we should change to Bear metal service.21:38
JayFI agree that the M should be capitalized as well21:38
*** Marga__ has joined #openstack-ironic21:39
rloo_just mentioning cuz there are some patches that are proposing that. https://review.openstack.org/#/c/191900/21:39
*** Marga_ has quit IRC21:42
rloo_devananda: ^^ do you know why it is 'Bare metal service' vs 'Bare Metal service'?21:44
* devananda checks something21:45
rloo_oh, its from http://git.openstack.org/cgit/openstack/governance/tree/reference/projects.yaml21:46
*** romcheg has quit IRC21:46
devanandayep21:46
devanandathat's what i just checked21:46
devanandaeasy to change that21:46
*** romcheg has joined #openstack-ironic21:46
rloo_sigh.21:46
devanandahowever, the entry in keystone catalog is what's critical21:46
rloo_thx devananda. if we have to review those patches, might as well do it just once.21:47
*** romcheg has quit IRC21:47
devanandarloo_: so it's registered in keystone as "baremetal"21:48
rloo_(would have been nice for some writer person to point out that it might not have been ideal...)21:48
devanandano spaces or caps21:48
rloo_devananda: so that 'string' is meant to be the same everywhere, eg in documentation and keystone catalog?21:48
devanandaliterally, the only thing that actually matters is the entry in keystone catalog21:48
devanandachanging that breaks users21:48
devanandachanging a doc string somewhere else is just noise in the review queue :)21:49
rloo_devananda: gotcha.21:49
rloo_devananda: yeah, i want to minimize the noise in the review queue.21:49
devanandarloo_: feel free to propose any changes to cApItAlIzation where you feel it's needed and I'll +[1,2]21:50
rloo_devananda: I think we need to change that govenance.../projects.yaml file.21:50
rloo_devananda: cuz ... umm, I think people are using that and may not deviate if you know what I mean21:50
devanandaexactly21:51
devananda"Baremetal" || "Bare Metal" ?21:51
rloo_devananda: I'd actually prefer 'Baremetal' but I don't know if there is an official/correct version.21:51
morganfainbergdevananda: i have a fix mostly rolled up to eliminate the subclassing of keystonemiddleware for you. I *think* it'll work. hope to have it posted this week.21:52
morganfainbergshould be a relatively small-ish change21:52
devanandamorganfainberg: \o/21:53
devanandarloo_: google tells me that "bare metal" is much more prevalent than "baremetal" in our community's usage21:53
devanandacaps notwithstandting21:53
devanandaalso - i need to catch a bus very soon21:53
devanandabbl21:53
rloo_devananda: yeah, unfortunately. no worries, no hurry on it :)21:53
*** jgrimm is now known as zz_jgrimm21:55
*** ijw_ has quit IRC21:55
*** lauramoore has quit IRC21:56
*** lauramoore has joined #openstack-ironic21:56
*** lauramoore has quit IRC21:56
*** Marga__ has quit IRC22:05
openstackgerritDoug Hellmann proposed openstack/ironic-lib: Drop use of 'oslo' namespace package  https://review.openstack.org/19333822:07
*** thiagop has quit IRC22:11
*** Marga_ has joined #openstack-ironic22:11
*** thrash is now known as thrash|g0ne22:13
*** ijw_ has joined #openstack-ironic22:28
*** e0ne has quit IRC22:31
*** radek_ has quit IRC22:32
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/19237222:34
openstackgerritMerged openstack/ironic: Added documentation to Vagrantfile  https://review.openstack.org/19191222:41
*** davidlenwell is now known as david_lenwell22:44
*** david_lenwell is now known as davidlenwell22:44
*** absubram has quit IRC22:47
*** absubram has joined #openstack-ironic22:48
*** kevinbenton has quit IRC22:48
*** mitchjameson has quit IRC22:49
*** ijw__ has joined #openstack-ironic22:50
*** TheJulia has quit IRC22:50
*** kevinbenton has joined #openstack-ironic22:51
*** boris-42 has quit IRC22:52
*** ijw_ has quit IRC22:53
* SpamapS gets bifrosty22:54
*** openstackgerrit has quit IRC22:55
*** vdrok has quit IRC22:55
*** EmilienM has quit IRC22:55
cineramadevananda, rloo_ : the openstack glossary suggests "bare metal" http://docs.openstack.org/glossary/content/glossary.html22:58
*** TheJulia has joined #openstack-ironic22:58
cineramawhich is harder for me to write without making a typo that references our lovable mascot :)22:59
*** ijw__ is now known as ijw_23:00
*** ijw_ has quit IRC23:00
*** Marga_ has quit IRC23:01
*** bethelwell has joined #openstack-ironic23:03
SpamapSugh23:04
SpamapSTheJulia: hey I'm playing with BiFrost23:04
SpamapSand the iPXE is failing23:04
SpamapSTheJulia: 10.10.16.131 - - [18/Jun/2015:16:03:07 -0700] "GET /boot.ipxe HTTP/1.0" 200 269 "-" "iPXE/1.0.0+"23:04
jrollSpamapS: always blame the network :P23:04
SpamapSTheJulia: its downloading, but something fails after that and then my ilo switches to booting from disk23:05
SpamapSI suppose I should see if I can make the ilo keep trying network only23:05
*** openstackgerrit has joined #openstack-ironic23:06
*** vdrok has joined #openstack-ironic23:06
*** EmilienM has joined #openstack-ironic23:06
SpamapSjroll: inorite23:06
jrollcan you get a console and see where it's failing?23:06
SpamapSjroll: I'm at the console, and no, currently the fail is wiped by "Attempting Boot From Hard Drive"23:06
SpamapSjroll: I'll see if I can et the BIOS to not do that.23:07
jrollgrr23:07
jrollsounds like maybe an image download thing23:07
jrollI assume apache serves images, do you see that happening?23:07
SpamapSnginx23:08
SpamapSand no23:08
jrollor whatever :P23:08
jrollhrm, can you paste boot.ipxe?23:08
*** bethelwell has quit IRC23:08
SpamapSjroll: http://git.openstack.org/cgit/openstack/bifrost/tree/playbooks/roles/ironic-install/files/boot.ipxe23:10
*** marzif has joined #openstack-ironic23:10
SpamapSfunny enough, these mellanox cards already have ipxe loaded on them23:11
SpamapSor something23:11
jrollhrm23:11
SpamapSI don't really understand how it all works23:11
jrollmaybe a lame version of ipxe that doesn't support ${mac:hexraw} ?23:11
jrollI've never heard good things about mellanox23:11
*** mitchjameson has joined #openstack-ironic23:11
SpamapSactually23:12
SpamapStheres nothing in my pxelinux.cfg dir23:12
jrolloh, that'll do it.23:12
martiniquit23:13
*** martini has quit IRC23:13
SpamapSunrecognized option --autofree23:13
SpamapSthats the error23:13
jroll:/23:14
jrollyeah, may be an ipxe version thing23:14
SpamapSbut I thought we were chain-loading a new one on? I'm confused now. :_P23:14
SpamapSit says iPXE 1.0.023:14
SpamapSanyway I suspect something has borked writing my pxelinux.cfg too23:14
jrollfull disclosure: I still haven't used bifrost :/23:15
SpamapSthats ok23:15
SpamapSyou've used ipxe23:15
jrollheh23:15
SpamapSlooks like --autofree is not part of 1.0.023:15
jrollyes, but very differently23:15
jrollright23:15
*** zhenguo has joined #openstack-ironic23:16
jrollso I don't think this is chainloading to a different ipxe but I may be wrong23:16
SpamapSREADME on bifrost claims it is23:16
jrollhuh.23:17
*** Marga_ has joined #openstack-ironic23:18
SpamapShttp://git.openstack.org/cgit/openstack/bifrost/tree/README.rst#n5523:18
jrollwell23:18
jrollso that's for normal pxe booting23:18
SpamapSdhcp-match=set:ipxe,175 # iPXE sends a 175 option.23:18
SpamapSdhcp-boot=tag:!ipxe,undionly.kpxe23:18
SpamapSdhcp-boot=http://10.10.16.130:8080/boot.ipxe23:18
jrollbut the response depends on the dhcp options in the request23:18
jrollso to the dhcp server it looks like it's already chain-loaded to ipxe23:18
*** alexpilotti has quit IRC23:19
SpamapSoh thats annoying23:19
*** Marga_ has quit IRC23:19
SpamapSTheJulia: have you run into this yet?23:20
jrollI suspect nobody has used bifrost yet with ipxe on the NIC ROM23:20
SpamapSentirely possible23:20
jrollit's something I've always wanted to do, though, just to skip a step23:21
SpamapSwell lets see what happens if I remove the tag from the undionly.kpxe bit23:22
*** Marga__ has joined #openstack-ironic23:23
* SpamapS wishes his hardware booted faster than 8 minutes23:23
*** Marga__ has quit IRC23:24
*** Marga_ has joined #openstack-ironic23:25
SpamapSit actually says it is 'iPXE 1.0.0+'23:27
SpamapSthe + is like, what, +mellanoxspecialsauce ?23:27
jrolllol23:27
jrollprobably23:27
*** boris-42 has joined #openstack-ironic23:28
SpamapSsame thing.. invalid argument23:28
*** Marga_ has quit IRC23:31
*** Marga_ has joined #openstack-ironic23:32
openstackgerritMichael Davies proposed openstack/python-ironicclient: Cache negotiated api microversion for server  https://review.openstack.org/17367423:35
TheJuliaSpamapS: hmmm23:36
TheJuliaSpamapS: Interesting......  can you packet capture everything for that mac address?  Kind of agree with jroll, I tested on a moonshot without any issues, but I don't remember if it was chain loading off-hand.23:38
TheJuliaSpamapS: oh, this is the mellonox nics?23:39
TheJuliaSpamapS: want to possibly try ilo virtual media? :)23:40
SpamapSTheJulia: I will try anything that might work.23:42
TheJuliaLooks like nobody in ironic-core has not +2+a'ed the ilo rev :)23:43
rloo_cinerama: thx. Yeah, I think we're going to go with 'bare metal' :)23:43
TheJuliahttps://review.openstack.org/#/c/191218/23:44
jrollTheJulia: boom23:44
TheJuliaSpamapS: From there, it would take using our bleeding edge dynamic inventory code to deploy.  I would still love a pcap of the exchange23:44
TheJuliajroll: awesomeness23:44
jrollI figure I should at least do one review today :P23:45
*** rloo_ has quit IRC23:45
*** yuanying has joined #openstack-ironic23:45
SpamapSTheJulia: dump in 8 minutes...23:45
TheJuliaMerci23:45
TheJuliaSpamapS: I guess your using a csv file?23:47
*** chlong has joined #openstack-ironic23:47
jrolldevananda: do you have time to look at https://review.openstack.org/#/c/185171/5/specs/liberty/feature-based-releases.rst23:48
jrolltomorrow is a fine answer23:48
jrollI had at least one question specifically for you there23:48
*** Haomeng has joined #openstack-ironic23:49
SpamapSTheJulia: I did use a csv file yes, the node is enrolled (there's just one) and claims to know that power is on/off23:49
*** absubram has quit IRC23:49
*** Haomeng|2 has quit IRC23:52
openstackgerritMerged openstack/bifrost: Enable agent_ilo driver  https://review.openstack.org/19121823:57

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