NobodyCam | ugg | 00:00 |
---|---|---|
NobodyCam | night jlvillal | 00:00 |
*** Marga_ has joined #openstack-ironic | 00:02 | |
*** DCWillia_ has quit IRC | 00:02 | |
*** saripurigopi_ has joined #openstack-ironic | 00:06 | |
*** naohirot has joined #openstack-ironic | 00:06 | |
*** saripurigopi has quit IRC | 00:08 | |
*** romcheg has quit IRC | 00:08 | |
*** saripurigopi_ is now known as saripurigopi | 00:08 | |
*** Marga_ has quit IRC | 00:17 | |
*** Marga_ has joined #openstack-ironic | 00:17 | |
*** igordcard has quit IRC | 00:28 | |
*** zhenguo has joined #openstack-ironic | 00:28 | |
*** cui has joined #openstack-ironic | 00:36 | |
*** cui has quit IRC | 00:38 | |
*** cui has joined #openstack-ironic | 00:39 | |
*** DCWilliams_VA has joined #openstack-ironic | 00:40 | |
openstackgerrit | Merged openstack/ironic: Drop use of 'oslo' namespace package https://review.openstack.org/178356 | 00:40 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Fix accumulated nits in iRMC power, mgmt, and deploy specs https://review.openstack.org/177065 | 00:46 |
*** lazy_prince has joined #openstack-ironic | 00:49 | |
*** saripurigopi_ has joined #openstack-ironic | 00:51 | |
*** saripurigopi has quit IRC | 00:51 | |
*** saripurigopi_ is now known as saripurigopi | 00:51 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Update Ironic spec URL refs to specs.openstack.org https://review.openstack.org/177632 | 00:56 |
*** shakamunyi has quit IRC | 01:02 | |
*** DCWilliams_VA has quit IRC | 01:12 | |
*** achanda has quit IRC | 01:29 | |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Consistent and more valid strings for Booleans https://review.openstack.org/170230 | 01:30 |
*** Marga_ has quit IRC | 01:42 | |
*** Marga_ has joined #openstack-ironic | 01:43 | |
*** kan_ has joined #openstack-ironic | 01:54 | |
*** saripurigopi has quit IRC | 01:59 | |
*** lazy_prince has quit IRC | 02:07 | |
*** cui has quit IRC | 02:08 | |
*** cui has joined #openstack-ironic | 02:08 | |
*** shakamunyi has joined #openstack-ironic | 02:10 | |
*** cui has quit IRC | 02:10 | |
*** achanda has joined #openstack-ironic | 02:29 | |
*** mtanino has quit IRC | 02:32 | |
*** saripurigopi has joined #openstack-ironic | 02:35 | |
*** saripurigopi has quit IRC | 02:44 | |
*** Marga_ has quit IRC | 02:45 | |
*** achanda has quit IRC | 02:46 | |
*** zz_natorious is now known as natorious | 02:47 | |
*** achanda has joined #openstack-ironic | 02:48 | |
*** r-daneel has quit IRC | 02:49 | |
*** praneshp has joined #openstack-ironic | 02:53 | |
praneshp | hello ironic | 02:53 |
praneshp | when I do an ironic node-create and pass some properties (using the -p option), I don’t see them show up on the nova compute side | 02:53 |
Haomeng|2 | praneshp: morning | 02:58 |
praneshp | Haomeng|2: hello! | 02:58 |
Haomeng|2 | praneshp: that is used by Ironic I think | 02:59 |
Haomeng|2 | praneshp: so what is your question | 02:59 |
praneshp | so ironic help node-create says this | 02:59 |
praneshp | -p <key=value>, --properties <key=value> | 02:59 |
praneshp | Key/value pairs describing the physical | 02:59 |
praneshp | characteristics of the node. This is exported to Nova | 02:59 |
praneshp | and used by the scheduler. Can be specified multiple | 02:59 |
praneshp | times | 02:59 |
praneshp | i expected that the propertis are somehow transferred to nova, and if I set extra specs in the nova flavor, I could use that as a way to schedule | 03:00 |
praneshp | but I took a look at nova hypervisor-list, the information passed to —properrties doesn’t seem to transfer over | 03:00 |
jroll | praneshp: you want -p capabilities=something | 03:01 |
jroll | example, capabilities=boot_mode:uefi,cap2:value2 | 03:01 |
jroll | those should match against extra_specs | 03:01 |
praneshp | jroll ah, ok. can you explain the difference? | 03:01 |
jroll | praneshp: no difference, just that the capabilities key is what we use to schedule against extra_specs | 03:02 |
praneshp | jroll: ah, got it. Thanks a lot. trying out now | 03:02 |
jroll | praneshp: and for example we schedule -p cpus=2 against the scheduler filter for number of CPUs | 03:02 |
praneshp | jroll: that worked. thanks! | 03:03 |
*** vishwanathj has joined #openstack-ironic | 03:03 | |
praneshp | jroll: got it. the nova side looks for specific keywords, and you can pass those via -p | 03:03 |
jroll | correct :) | 03:03 |
praneshp | you could write your own filter to look for ‘foo’ and pass -p foo=bar | 03:03 |
jroll | indeed | 03:03 |
praneshp | thanks! | 03:03 |
jroll | no problem :) | 03:03 |
openstackgerrit | Merged openstack/ironic: Fix Python 3.4 test failure https://review.openstack.org/180763 | 03:04 |
openstackgerrit | Merged openstack/ironic: Add section header to state machines page https://review.openstack.org/180204 | 03:04 |
openstackgerrit | Merged openstack/ironic: Remove unneeded usage of '# noqa' https://review.openstack.org/180767 | 03:04 |
*** praneshp has quit IRC | 03:06 | |
*** achanda has quit IRC | 03:28 | |
*** Nisha has joined #openstack-ironic | 03:35 | |
*** Nisha has quit IRC | 03:41 | |
*** Nisha has joined #openstack-ironic | 03:41 | |
*** jrist has quit IRC | 03:43 | |
*** pcrews has quit IRC | 03:43 | |
*** praneshp has joined #openstack-ironic | 03:43 | |
*** links has joined #openstack-ironic | 03:45 | |
*** jrist has joined #openstack-ironic | 03:55 | |
*** achanda has joined #openstack-ironic | 04:05 | |
*** kalpase has joined #openstack-ironic | 04:09 | |
*** saripurigopi has joined #openstack-ironic | 04:14 | |
*** spandhe has quit IRC | 04:14 | |
*** ramineni has joined #openstack-ironic | 04:20 | |
*** lazy_prince has joined #openstack-ironic | 04:21 | |
openstackgerrit | Anusha Ramineni proposed openstack/ironic-python-agent: Fix inband disk erase using agent_ilo driver https://review.openstack.org/178925 | 04:21 |
ramineni | vdrok: hi | 04:30 |
*** pcrews has joined #openstack-ironic | 04:41 | |
jlvillal | devananda, Maybe you could give feedback on patch to add checking of Python 3.4 in the gate: https://review.openstack.org/#/c/180599/ | 05:03 |
*** coolsvap|afk is now known as coolsvap | 05:03 | |
jlvillal | devananda, I think we are good to go as 'tox -epy34' is now passing :) | 05:04 |
*** praneshp has quit IRC | 05:06 | |
openstackgerrit | Gopi Krishna S proposed openstack/ironic: Add pxe_ucs and agent_ucs drivers to manage Cisco UCS servers https://review.openstack.org/159734 | 05:10 |
*** kalpase has quit IRC | 05:15 | |
*** rameshg87 has joined #openstack-ironic | 05:24 | |
rameshg87 | good morning ironic | 05:25 |
*** mdbooth has quit IRC | 05:27 | |
mrda | Morning rameshg87 | 05:28 |
rameshg87 | morning mrda | 05:28 |
*** mdbooth has joined #openstack-ironic | 05:31 | |
*** killer_prince has joined #openstack-ironic | 05:33 | |
*** lazy_prince has quit IRC | 05:36 | |
*** kalpase has joined #openstack-ironic | 05:43 | |
*** killer_prince has quit IRC | 05:43 | |
*** lazy_prince has joined #openstack-ironic | 05:43 | |
*** ukalifon1 has joined #openstack-ironic | 06:02 | |
*** lazy_prince has quit IRC | 06:06 | |
*** ukalifon1 has quit IRC | 06:11 | |
*** dlpartain has joined #openstack-ironic | 06:12 | |
*** natorious is now known as zz_natorious | 06:17 | |
*** spandhe has joined #openstack-ironic | 06:22 | |
*** sandhya has joined #openstack-ironic | 06:22 | |
*** spandhe_ has joined #openstack-ironic | 06:23 | |
sandhya | dtantsur: Hi! Can you give your further comments on - https://review.openstack.org/#/c/134866/, after the recent discussions? It will help me proceed... | 06:24 |
*** dlpartain has left #openstack-ironic | 06:25 | |
*** spandhe has quit IRC | 06:27 | |
*** spandhe_ is now known as spandhe | 06:27 | |
*** jcoufal has joined #openstack-ironic | 06:28 | |
*** lazy_prince has joined #openstack-ironic | 06:32 | |
*** killer_prince has joined #openstack-ironic | 06:34 | |
openstackgerrit | Ghe Rivero proposed openstack/python-ironicclient: Sync oslo.incubator https://review.openstack.org/162806 | 06:37 |
*** lazy_prince has quit IRC | 06:37 | |
openstackgerrit | Merged openstack/ironic: Remove cleanfail->cleaning from state diagram https://review.openstack.org/179900 | 06:48 |
*** bkero has quit IRC | 06:50 | |
*** bkero has joined #openstack-ironic | 06:50 | |
*** dlpartain has joined #openstack-ironic | 07:02 | |
*** dlpartain has left #openstack-ironic | 07:02 | |
zhenguo | rameshg87, are you around there? | 07:05 |
*** viktors|afk is now known as viktors | 07:18 | |
*** ifarkas has joined #openstack-ironic | 07:34 | |
rameshg87 | zhenguo: hi | 07:47 |
rameshg87 | zhenguo: sorry, i was away. i am going away again | 07:47 |
rameshg87 | zhenguo: shall i ping you back ? | 07:47 |
*** rameshg87 is now known as rameshg87-lunch | 07:48 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Improve strictness of iRMC test cases error checking https://review.openstack.org/177696 | 07:49 |
zhenguo | rameshg87: want to discuss about https://review.openstack.org/#/c/180014/ | 07:50 |
zhenguo | rameshg87: do you mean only check ephemeral disk is enough? | 07:51 |
zhenguo | ramesh87: o, wait for you back :) | 07:52 |
*** jistr has joined #openstack-ironic | 07:54 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Improve strictness of iLO test cases error checking https://review.openstack.org/177700 | 07:54 |
*** yog__ has joined #openstack-ironic | 07:55 | |
*** achanda has quit IRC | 07:56 | |
*** athomas has joined #openstack-ironic | 07:57 | |
kejiahu | morning Ironic | 07:58 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Improve strictness of AMT test cases error checking https://review.openstack.org/177698 | 07:59 |
*** jcoufal_ has joined #openstack-ironic | 08:03 | |
*** kalpase has quit IRC | 08:07 | |
*** jcoufal has quit IRC | 08:07 | |
*** kalpase has joined #openstack-ironic | 08:13 | |
*** MattMan has joined #openstack-ironic | 08:15 | |
*** e0ne has joined #openstack-ironic | 08:23 | |
*** kalpase has quit IRC | 08:25 | |
*** kalpase has joined #openstack-ironic | 08:25 | |
*** ifarkas has quit IRC | 08:27 | |
*** saripurigopi has quit IRC | 08:27 | |
*** lucasagomes has joined #openstack-ironic | 08:28 | |
*** chenglch has joined #openstack-ironic | 08:29 | |
openstackgerrit | chenglch proposed openstack/ironic-specs: Add petitboot deploy driver support https://review.openstack.org/177641 | 08:30 |
*** ifarkas has joined #openstack-ironic | 08:36 | |
*** ifarkas has quit IRC | 08:42 | |
*** ifarkas_ has joined #openstack-ironic | 08:42 | |
*** saripurigopi has joined #openstack-ironic | 08:50 | |
*** kalpase has quit IRC | 08:51 | |
*** kalpase has joined #openstack-ironic | 08:52 | |
*** jcoufal_ has quit IRC | 08:55 | |
*** jcoufal has joined #openstack-ironic | 08:56 | |
*** tiagogomes_ has joined #openstack-ironic | 08:56 | |
*** e0ne is now known as e0ne_ | 09:03 | |
*** athomas has quit IRC | 09:11 | |
*** derekh has joined #openstack-ironic | 09:16 | |
*** athomas has joined #openstack-ironic | 09:17 | |
* rameshg87-lunch rameshg87 | 09:17 | |
*** rameshg87-lunch is now known as rameshg87 | 09:18 | |
rameshg87 | zhenguo: hi | 09:18 |
rameshg87 | zhenguo: yeah i meant checking only if ephemeral disk is enough | 09:18 |
*** ramineni has quit IRC | 09:20 | |
zhenguo | rameshg87: if rebuild with preserve_ephemeral, root and swap can be changed but total not, right? | 09:20 |
*** ramineni has joined #openstack-ironic | 09:20 | |
rameshg87 | zhenguo: if we don't allow root to be changed, how can people deploy images larger than their current root_gb ? | 09:21 |
rameshg87 | zhenguo: why can't we change total ? | 09:21 |
zhenguo | we should keep ephemeral disk layout data, if total size changed, can we still preserve it. | 09:23 |
rameshg87 | zhenguo: i don't see why we cannot preserve it | 09:25 |
rameshg87 | zhenguo: the layout of the partition table is like first ephemeral, second swap, third root | 09:25 |
*** pelix has joined #openstack-ironic | 09:26 | |
zhenguo | rameshg87: I find the bug description said the only exception is root got bigger | 09:26 |
rameshg87 | zhenguo: if ephemeral_gb changes and preserve_ephemeral is set, we cannot match the file system that's there already | 09:26 |
rameshg87 | zhenguo: but if ephemeral_gb is same, and root_gb is different, the partition table will use the same size for ephemeral and it can be preserved | 09:27 |
* rameshg87 checks bug | 09:27 | |
rameshg87 | zhenguo: i don't find a reason why it won't work if root_gb changed | 09:29 |
rameshg87 | lucasagomes: hi | 09:29 |
ramineni | lucasagomes: hi | 09:29 |
rameshg87 | lucasagomes: any thoughts on above ^^^^ and https://review.openstack.org/#/c/180014/ ? | 09:29 |
*** Nisha has quit IRC | 09:30 | |
zhenguo | rameshg87: after think about more, if we first create ephemeral partition, seems the ephemeral disk can be reserved make sense. | 09:31 |
rameshg87 | zhenguo: yeah, i think root_gb and swap_mb may be changed and it doesn't affect | 09:33 |
*** e0ne_ is now known as e0ne | 09:33 | |
zhenguo | rameshg87: ok, I think you are right, will confirm with the bug reporter about the root_gb reference exception ;) | 09:35 |
rameshg87 | zhenguo: okay.. | 09:36 |
rameshg87 | zhenguo: i think you could try out as well if it really works :) | 09:36 |
rameshg87 | zhenguo: i mean i still don't know if i am missing something | 09:36 |
*** spandhe has quit IRC | 09:37 | |
zhenguo | rameshg87: ok, thanks a lot for the explanation, I will try it :) | 09:37 |
*** kan_ has quit IRC | 09:44 | |
*** lucasagomes_ has joined #openstack-ironic | 10:00 | |
*** lucasagomes has quit IRC | 10:02 | |
*** romcheg has joined #openstack-ironic | 10:04 | |
*** lucasagomes_ is now known as lucasagomes | 10:08 | |
*** romcheg has quit IRC | 10:15 | |
*** romcheg has joined #openstack-ironic | 10:15 | |
*** romcheg1 has joined #openstack-ironic | 10:19 | |
*** romcheg has quit IRC | 10:22 | |
*** chenglch has quit IRC | 10:25 | |
*** thrash|g0ne is now known as thrash | 10:29 | |
*** ifarkas_ has quit IRC | 10:29 | |
*** sandhya has quit IRC | 10:39 | |
* rameshg87 goes home | 10:40 | |
*** rameshg87 has quit IRC | 10:40 | |
*** coolsvap is now known as coolsvap|afk | 10:43 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Wake-On-Lan Power interface https://review.openstack.org/179078 | 10:46 |
*** alexpilotti has joined #openstack-ironic | 10:51 | |
*** ifarkas has joined #openstack-ironic | 10:52 | |
*** achanda has joined #openstack-ironic | 11:02 | |
*** achanda has quit IRC | 11:07 | |
*** romcheg1 has quit IRC | 11:15 | |
*** naohirot has quit IRC | 11:17 | |
*** e0ne is now known as e0ne_ | 11:27 | |
*** links has quit IRC | 11:31 | |
*** links has joined #openstack-ironic | 11:32 | |
*** romcheg has joined #openstack-ironic | 11:43 | |
*** kan_ has joined #openstack-ironic | 11:43 | |
*** e0ne_ is now known as e0ne | 11:51 | |
*** openstackgerrit has quit IRC | 11:52 | |
lucasagomes | ramineni, hi there | 11:53 |
*** openstackgerrit has joined #openstack-ironic | 11:53 | |
lucasagomes | ramesh<tab> | 11:53 |
lucasagomes | http://news.softpedia.com/news/Mark-Shuttleworth-Asks-Devs-from-Different-Desktop-Environments-to-Work-Together-480336.shtml | 11:53 |
lucasagomes | gosh is it a joke ^!? | 11:53 |
ekarlso | by god, god forbid 1 desktop or fewer environments .. :p | 11:57 |
*** trown|outttypeww is now known as trown | 12:05 | |
*** saripurigopi has quit IRC | 12:07 | |
*** dmellado has quit IRC | 12:09 | |
*** dmellado has joined #openstack-ironic | 12:11 | |
*** dprince has joined #openstack-ironic | 12:16 | |
*** dlpartain has joined #openstack-ironic | 12:40 | |
*** romcheg has quit IRC | 12:40 | |
afaranha | Hey guys, I'm reading a post from the last year relating some issues in running devstack with real machines ( https://answers.launchpad.net/devstack/+question/247992 ), and maybe some changes need to be made in create_bridge_and_vms() and enroll_vms() methods in devstack/lib/ironic. Does anyone knows if we still need no change these methods? | 12:41 |
*** dlpartain has left #openstack-ironic | 12:43 | |
*** kkoski has joined #openstack-ironic | 12:45 | |
lucasagomes | afaranha, haven't tried :-( | 12:46 |
*** rloo has joined #openstack-ironic | 12:47 | |
*** romcheg has joined #openstack-ironic | 12:51 | |
rloo | hello ironickers :) | 13:01 |
afaranha | lucasagomes, I think it's already working, just asked to be sure, thanks :) | 13:02 |
lucasagomes | rloo, morning | 13:02 |
lucasagomes | afaranha, cool | 13:02 |
rloo | viktors: about object.__new__() (https://review.openstack.org/#/c/156192/) that's yours, right? | 13:02 |
rloo | viktors: I don't understand your comment about behaviour being deprecated since python 2.6 | 13:02 |
*** yog__ has quit IRC | 13:10 | |
*** kalpase has quit IRC | 13:14 | |
*** kalpase1 has joined #openstack-ironic | 13:14 | |
*** david-lyle has quit IRC | 13:19 | |
jroll | lucasagomes: shuttleworth means that everyone should work together on wayland :P | 13:19 |
jroll | (don't actually know what he means, to be clear) | 13:19 |
*** yog__ has joined #openstack-ironic | 13:22 | |
*** yog__ has quit IRC | 13:23 | |
trown | jroll: lucasagomes, I am sure he would prefer we all abandon Ironic and work on MaaS as well | 13:24 |
*** openstackstatus has quit IRC | 13:25 | |
jroll | trown: if I'm abandoning ironic it's going to be to rewrite it in go :P | 13:25 |
*** openstackstatus has joined #openstack-ironic | 13:26 | |
*** ChanServ sets mode: +v openstackstatus | 13:26 | |
trown | jroll, why go? just for coroutines? | 13:27 |
lucasagomes | jroll, heh what happened with MIR ? | 13:28 |
trown | for good types why not just go all the way to haskell | 13:28 |
lucasagomes | trown, hah yeah he def would love that | 13:28 |
jroll | trown: that, ease of deployment, typing, etc | 13:28 |
lucasagomes | C folks C | 13:28 |
jroll | lucasagomes: oh yeah, I meant mir :P | 13:28 |
lucasagomes | (not kidding) | 13:28 |
jroll | :| | 13:28 |
lucasagomes | wayland FTW MIR FTL | 13:29 |
jroll | I see very few reasons to write C these days | 13:29 |
lucasagomes | oh noes | 13:29 |
lucasagomes | C ftw man | 13:29 |
jroll | it has it's place, and something like this is not it | 13:29 |
*** ramineni has quit IRC | 13:29 | |
jroll | imagine writing a drac driver in c | 13:29 |
NobodyCam | good morning Ironic-ers | 13:33 |
jroll | heyheyhey | 13:34 |
NobodyCam | morning jroll lucasagomes trown and rloo | 13:34 |
*** e0ne is now known as e0ne_ | 13:34 | |
lucasagomes | jroll, heh yeah XML and C is a problem | 13:35 |
lucasagomes | jroll, well XML with any language is a problem :-) | 13:35 |
lucasagomes | NobodyCam, morning | 13:35 |
jroll | lucasagomes: :P | 13:36 |
trown | good morning NobodyCam | 13:36 |
rloo | morning NobodyCam, jroll, lucasagomes, trown | 13:36 |
*** e0ne_ is now known as e0ne | 13:37 | |
jroll | hiya rloo :) | 13:37 |
NobodyCam | :) | 13:37 |
trown | o/ | 13:38 |
viktors | hi Ironic, rloo | 13:39 |
*** chlong has joined #openstack-ironic | 13:39 | |
rloo | hey viktors, did you see my question ^^ | 13:40 |
viktors | rloo: object.__new__() accepts args and kwargs in python 2.6 and 2,7 but ignores them and raises Deprecation warning | 13:40 |
NobodyCam | morning viktors | 13:40 |
viktors | morning NobodyCam | 13:40 |
rloo | viktors: so why do the if/else stuff? | 13:40 |
rloo | viktors: the super_new(cls, *args, **kwargs) line. When will that ever be used? | 13:41 |
openstackgerrit | Merged openstack/python-ironicclient: Sync oslo.incubator https://review.openstack.org/162806 | 13:42 |
viktors | rloo: because after multiple inherit we can get super_new as __new__ from another class, not object | 13:42 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Virtual Media Deploy module for iRMC Driver https://review.openstack.org/151958 | 13:42 |
viktors | rloo: this idea was taken from http://blog.jaraco.com/2014/05/how-to-safely-override-init-or-new-in.html | 13:44 |
rloo | viktors: ahh (I think). So line 136: https://review.openstack.org/#/c/156192/11/ironic/drivers/base.py | 13:45 |
rloo | viktors: that will always be object.__new__? | 13:45 |
viktors | rloo: I'm not very familiar with Ironic codebase, so I made an universal variant | 13:48 |
*** r-daneel has joined #openstack-ironic | 13:49 | |
rloo | viktors: and I'm not a python expert :) The change looks confusing to me, I'm trying to understand it. BaseInterface is a subclass of object, so super(BaseInterface, cls).__new__ will be object.__new__? | 13:50 |
viktors | rloo: let me try to make some snipped to demonstrate this | 13:51 |
*** jamielennox is now known as jamielennox|away | 13:51 | |
*** links has quit IRC | 13:54 | |
tiagogomes_ | Hi, is there a plan for creating 2014.2.3 tag (Juno releases) | 13:56 |
BadCub | morning folks | 13:56 |
NobodyCam | morning BadCub | 13:57 |
*** SpamapS has quit IRC | 13:58 | |
BadCub | morning NobodyCam | 13:59 |
viktors | rloo: please see http://paste.openstack.org/show/216195/ | 14:03 |
viktors | morning BadCub | 14:03 |
*** achanda has joined #openstack-ironic | 14:06 | |
jlvillal | jroll, lucasagomes: A PyCon talk about using Rust with Python: http://www.youtube.com/watch?v=3CwJ0MH-4MA Since you were talking about C before. | 14:07 |
jroll | O.o | 14:07 |
rloo | viktors: thx! I will expand my knowledge of python... | 14:09 |
lucasagomes | jlvillal, cool | 14:09 |
* lucasagomes never played with rust tho | 14:09 | |
*** rameshg87 has joined #openstack-ironic | 14:10 | |
rameshg87 | morning folks :) | 14:10 |
NobodyCam | hey hey lucasagomes just added comments to 179078, note that I have only had ~ one cup of coffee so take the naming suggestion with that in mind! | 14:11 |
NobodyCam | morning rameshg87 | 14:11 |
*** achanda has quit IRC | 14:11 | |
jlvillal | lucasagomes, Me either. But the little I have learned about Rust it looks interesting. I also would like to learn Go. Go is probably the first language I would like to learn about. | 14:12 |
lucasagomes | NobodyCam, ah nice! thanks for reviewing it | 14:12 |
viktors | rloo: thanks for your help with it :) | 14:12 |
BadCub | morning viktors :) | 14:12 |
rloo | hi tiagogomes_ I don't know. Are there bugs that need to be fixed in Juno? I don't know the actual dates, but this might help: https://wiki.openstack.org/wiki/StableBranch#Support_phases | 14:12 |
viktors | morning rameshg87 | 14:12 |
lucasagomes | jlvillal, yeah, I saw a presentantion with rob pike about go | 14:12 |
lucasagomes | pretty cool | 14:12 |
rameshg87 | NobodyCam: viktors: o/ | 14:13 |
NobodyCam | :) | 14:13 |
tiagogomes_ | ta rloo | 14:13 |
* lucasagomes is not really enthusiastic about learning new high level languages... Would like to try a diff paradigm lang tho | 14:14 | |
*** jcoufal has quit IRC | 14:14 | |
*** jrist has quit IRC | 14:14 | |
*** absubram has joined #openstack-ironic | 14:16 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Add generic RAID configuration spec for liberty https://review.openstack.org/173214 | 14:25 |
*** jrist has joined #openstack-ironic | 14:26 | |
*** jcoufal has joined #openstack-ironic | 14:27 | |
*** achanda has joined #openstack-ironic | 14:29 | |
*** achanda has quit IRC | 14:31 | |
*** saripurigopi has joined #openstack-ironic | 14:32 | |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Break out of FRU read if zero data returned https://review.openstack.org/181019 | 14:34 |
BadCub | hey folks.. I put up a question about an Ironic Dinner in Vancouver on the Planning Pad & Meeting Agenda. | 14:35 |
NobodyCam | mmmm food | 14:36 |
NobodyCam | :) | 14:36 |
NobodyCam | brb | 14:39 |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Break out of FRU read if zero data returned https://review.openstack.org/181019 | 14:47 |
*** jasonamyers has joined #openstack-ironic | 14:51 | |
*** hemnafk is now known as hemna | 14:52 | |
*** saripurigopi has quit IRC | 14:53 | |
*** achanda has joined #openstack-ironic | 14:56 | |
*** rameshg87 is now known as rameshg87-away | 14:56 | |
*** jcoufal has quit IRC | 15:02 | |
*** achanda has quit IRC | 15:03 | |
*** zz_natorious is now known as natorious | 15:10 | |
*** ramineni has joined #openstack-ironic | 15:10 | |
JayF | BadCub: https://review.openstack.org/#/c/176395/ has been advertised in nova several times. We can't get anyone to core-review it. Mind taking a shot? It's got +10 in +1s and no core reviews. | 15:11 |
BadCub | JayF: sure! | 15:12 |
lucasagomes | JayF, yeah nova takes a while to review stuff... It seems that only a niche of cores in nova actually looks at the ironic patches | 15:12 |
lucasagomes | maybe pinging them directly? | 15:13 |
BadCub | I just pinged in channel | 15:14 |
JayF | lucasagomes: I mean; I don't know who they are or really have time to do that today. That should /not/ be a requirement to get code reviewed though, especially with such a simple patch and 30% of theIronic core team +1'ing it | 15:14 |
JayF | lucasagomes: I also know mikal and dansmith was pointed directly at this patch and didn't put a vote on it | 15:14 |
lucasagomes | JayF, right... yeah I just advertised it again in the nova channel | 15:15 |
lucasagomes | if not I will see if I can ping some red hat nova people to take a look at it | 15:15 |
lucasagomes | it's +3,-2LOC | 15:15 |
lucasagomes | takes 1 min to review it | 15:15 |
JayF | haha, that's 3 times in one IRC scroll in #openstack-nova | 15:15 |
JayF | I <3 you guys | 15:15 |
jlvillal | Got one +2 now :) | 15:15 |
BadCub | hehehehe | 15:16 |
lucasagomes | :D | 15:16 |
rloo | JayF: you'd hate me if I pointed out something in that nova patch, right? | 15:16 |
jroll | rloo: is it a docstring thing | 15:16 |
jroll | look at the "needed by" :) | 15:16 |
*** saripurigopi has joined #openstack-ironic | 15:16 | |
rloo | jroll, JayF: yeah, docstring. can be updated later. | 15:16 |
JayF | rloo: followup patch is already there with 3x+1 | 15:16 |
JayF | :D | 15:17 |
jroll | :P | 15:17 |
rloo | JayF: ok, line 644? | 15:17 |
rloo | back in a bit | 15:17 |
ramineni | lucasagomes: JayF: please have a look at https://review.openstack.org/#/c/177141/ and let me know your opinion | 15:17 |
lucasagomes | ramineni, ack | 15:18 |
jlvillal | rloo: Yes, 644 and 647. https://review.openstack.org/#/c/180632/1/nova/virt/ironic/driver.py | 15:18 |
ramineni | lucasagomes: tnx | 15:18 |
JayF | ramineni: I pretty much agree with everything lucasagomes said in that patch + I have a couple more things | 15:20 |
JayF | ramineni: basically you're breaking the hardwaremanager API with that change | 15:20 |
JayF | ramineni: it's *supposed* to encapsulate all errors in errors.IncompatibleHardwareMethodError because that's what dispatch_to_managers() expects | 15:21 |
lucasagomes | ramineni, look at L434. I'm ok with raising an exception, but then we need to handle that properly in the places that the method is invoked | 15:21 |
JayF | lucasagomes: I'm basically thinking I'm OK with the whole patch, if the except ... : block continues to return False rather than raising | 15:22 |
lucasagomes | JayF, yeah that would be simpler indeed | 15:23 |
JayF | commented + -1'd | 15:24 |
*** rameshg87-away is now known as rameshg87 | 15:25 | |
rameshg87 | JayF: one question in raminenis patch | 15:29 |
rameshg87 | JayF: the point of argument was shred is not supposed to fail for any device for any *real* reason, so how do we know about those (unless we log it and someone checks IPA logs) | 15:30 |
JayF | rameshg87: Because by that shred method returning False, it ends up raising IncompatibleHardwareMethodError in the parent method (the one actually called in cleaning) which indicates that it failed | 15:32 |
JayF | rameshg87: the problem with raising in that method is that it prevents someone from being able to do something like, add a method that dd's zeros and have that tried if shred fails | 15:32 |
*** athomas has quit IRC | 15:32 | |
JayF | rameshg87: or other similar fallback stuff. Look at erase_devices() method that calls the shred method in question | 15:32 |
*** killer_prince has quit IRC | 15:32 | |
rameshg87 | JayF: hmm okay. i understand the concern now. | 15:34 |
rameshg87 | JayF: but may be some means to propagate the error might be useful | 15:34 |
JayF | rameshg87: I'd suggest logging there; but the idea is that failure is OK | 15:35 |
JayF | rameshg87: for instance, should we propogate the error that secure erase wasn't possible? | 15:35 |
rameshg87 | JayF: may be when there is a real way to see ipa logs from outside, it will be helpful | 15:35 |
*** Marga_ has joined #openstack-ironic | 15:35 | |
JayF | rameshg87: There are ways, in fact in the gate you can see the logs they just aren't persisted, which was being worked on for a while but never got completed | 15:35 |
jlvillal | I have uploaded a patch to make the 'gate-ironic-python34' job be a voting job for 'master' branch. https://review.openstack.org/#/c/181034/ If anyone wants to look. | 15:35 |
rameshg87 | JayF: yeah i agree with trying one-after-the-other logic | 15:35 |
*** sdake_ has joined #openstack-ironic | 15:36 | |
*** Marga_ has quit IRC | 15:36 | |
jlvillal | viktors: ^^^^ my previous message. | 15:36 |
*** Marga_ has joined #openstack-ironic | 15:36 | |
rameshg87 | JayF: i think our concern was when someone internally reported this error, we had no way of finding out why it failed unless we went into their system and debugged :D | 15:36 |
rameshg87 | JayF: may be with logging should helpful as you said | 15:37 |
JayF | rameshg87: makes sense. I'm just curious how an agent image got built without shred. | 15:37 |
rameshg87 | JayF: it wasn't shred not existing. | 15:37 |
rameshg87 | JayF: it was another issue - https://review.openstack.org/#/c/178925/ | 15:38 |
viktors | jlvillal: I'n not an expert in this config, but shouldn't your patch be similar to https://review.openstack.org/#/c/176356/1/zuul/layout.yaml ? | 15:38 |
rameshg87 | JayF: it was trying to erase the virtual media device | 15:38 |
JayF | hah! Oh no! | 15:38 |
*** kozhukalov has joined #openstack-ironic | 15:39 | |
JayF | rameshg87: landed it | 15:39 |
rameshg87 | JayF: great .. thanks :) | 15:40 |
jlvillal | viktors: I think I have to make it look like clarkb suggested. | 15:40 |
rameshg87 | \o/ +1 from rloo on raid spec :) | 15:40 |
JayF | I gotta make sure to look at that spec, we'll end up using it | 15:40 |
rameshg87 | but thanks a lot rloo for detailed review. it really helped | 15:41 |
rameshg87 | JayF: please .. | 15:41 |
viktors | jlvillal: ok, I don't mind :) | 15:41 |
*** natorious is now known as zz_natorious | 15:41 | |
jlvillal | viktors: I'm fumbling around with layout.yaml as I've never made changes before... Luckily clarkb has been giving very helpful info :) | 15:41 |
*** sdake_ has quit IRC | 15:42 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add disk layout check on re-provisioning https://review.openstack.org/180014 | 15:43 |
viktors | jlvillal: yes, this looks weird at first look | 15:44 |
jlvillal | viktors: :) | 15:44 |
*** zhenguo has quit IRC | 15:44 | |
jlvillal | viktors: And hopefully you are Victor Sergeyev. Otherwise you would be confused about what I am talking about. | 15:47 |
viktors | jlvillal: yes, that's me :) | 15:47 |
jlvillal | viktors: Great. And thanks for all the Python 3.4 work! :) | 15:47 |
*** romcheg has quit IRC | 15:48 | |
jlvillal | viktors: Is there a benefit to using args[0] for 'IOError.message' vs using 'IOError.__str__()'. I used 'args[0]' like you did. | 15:48 |
jlvillal | viktors: But I was wondering if '__str__()' might be a closer match to '.message'??? | 15:49 |
jlvillal | But I don't know. | 15:49 |
*** mtanino has joined #openstack-ironic | 15:50 | |
viktors | jlvillal: actually, I'm not sure as well. I don't remember, where I found this recommendation at first. But it used widely, see for example http://python3porting.com/noconv.html#handling-exceptions | 15:51 |
NobodyCam | morning jlvillal | 15:51 |
jlvillal | NobodyCam: Good morning. | 15:51 |
NobodyCam | :) | 15:51 |
jlvillal | viktors: Looking... | 15:51 |
*** mtanino_ has joined #openstack-ironic | 15:52 | |
jlvillal | viktors: Works for me :) | 15:53 |
rameshg87 | jlvillal: debian folks filed a bug against proliantutils tests not working on python3. i might end up following your path soon :) | 15:53 |
viktors | :) | 15:53 |
jlvillal | rameshg87: Great. viktors did 99% of the work I think! | 15:54 |
rameshg87 | thanks viktors :) | 15:54 |
*** mtanino has quit IRC | 15:55 | |
viktors | rameshg87: glad to help :) | 15:56 |
*** mtanino_ has quit IRC | 15:59 | |
*** saripurigopi has quit IRC | 16:03 | |
*** zz_natorious is now known as natorious | 16:04 | |
*** trown is now known as trown|lunch | 16:05 | |
openstackgerrit | Kan proposed openstack/python-ironicclient: Disable meaningless sort keys in list command https://review.openstack.org/177578 | 16:06 |
*** kalpase1 has left #openstack-ironic | 16:06 | |
*** e0ne is now known as e0ne_ | 16:08 | |
*** praneshp has joined #openstack-ironic | 16:10 | |
*** viktors is now known as viktors|afk | 16:14 | |
*** e0ne_ is now known as e0ne | 16:17 | |
*** SpamapS has joined #openstack-ironic | 16:18 | |
*** SpamapS has quit IRC | 16:18 | |
*** SpamapS has joined #openstack-ironic | 16:18 | |
*** derekh has quit IRC | 16:19 | |
*** gridinv has joined #openstack-ironic | 16:20 | |
*** Marga_ has quit IRC | 16:24 | |
*** mtanino has joined #openstack-ironic | 16:25 | |
openstackgerrit | Merged openstack/ironic-python-agent: Fix inband disk erase using agent_ilo driver https://review.openstack.org/178925 | 16:26 |
*** gridinv has quit IRC | 16:26 | |
*** shakamunyi has quit IRC | 16:29 | |
lucasagomes | folks I will call it a day | 16:31 |
lucasagomes | came to the office and will catch the train back home yet | 16:31 |
lucasagomes | have a good night everyone! | 16:31 |
NobodyCam | have a good night lucasagomes | 16:31 |
BadCub | g'night lucasagomes :) | 16:31 |
*** Marga_ has joined #openstack-ironic | 16:31 | |
lucasagomes | see y'all | 16:33 |
*** lucasagomes has quit IRC | 16:33 | |
*** tiagogomes_ has quit IRC | 16:41 | |
NobodyCam | brb | 16:41 |
*** ramineni has quit IRC | 16:45 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Python 3 compatibility https://review.openstack.org/181072 | 16:49 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Add new deploy driver for diskless nodes https://review.openstack.org/181074 | 16:50 |
*** ramineni has joined #openstack-ironic | 16:54 | |
*** Nisha has joined #openstack-ironic | 16:55 | |
*** Marga_ has quit IRC | 17:00 | |
*** ramineni has quit IRC | 17:06 | |
*** e0ne has quit IRC | 17:11 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is usable for ipmitool driver https://review.openstack.org/160383 | 17:14 |
*** jistr has quit IRC | 17:17 | |
*** Marga_ has joined #openstack-ironic | 17:25 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Fixes some docstring warnings https://review.openstack.org/180339 | 17:26 |
*** natorious is now known as zz_natorious | 17:29 | |
*** Marga_ has quit IRC | 17:30 | |
*** praneshp has left #openstack-ironic | 17:30 | |
*** Marga_ has joined #openstack-ironic | 17:30 | |
*** Marga_ has quit IRC | 17:31 | |
*** pelix has quit IRC | 17:31 | |
*** Marga_ has joined #openstack-ironic | 17:35 | |
*** achanda has joined #openstack-ironic | 17:43 | |
*** anteaya has quit IRC | 17:45 | |
*** trown|lunch is now known as trown | 17:48 | |
NobodyCam | jroll: got a second to take a look at https://review.openstack.org/#/c/173218 | 17:48 |
*** jistr has joined #openstack-ironic | 17:49 | |
jlvillal | mariojv: What tool did you use to check docstrings? https://review.openstack.org/180339 | 17:54 |
mariojv | jlvillal: i just looked at the warnings generated by "python setup.py build_sphinx" | 17:54 |
jlvillal | mariojv: ah, thanks :) | 17:55 |
mariojv | np | 17:55 |
mariojv | jlvillal: i think the gate jobs will generate draft docs we can use for sanity checks - see the comments in patch set 2 | 17:56 |
rameshg87 | good night all | 17:57 |
*** rameshg87 has quit IRC | 17:58 | |
*** derekh has joined #openstack-ironic | 17:59 | |
jlvillal | mariojv: Did you get DuplicateOptError when doing the build_sphinx? I was trying with 'tox -v -v -egendocs' | 18:02 |
mariojv | i don't think so - double-checking | 18:03 |
mariojv | is the "-v -v" doing that? | 18:03 |
mariojv | jlvillal: ^ | 18:04 |
jlvillal | mariojv: Not sure. | 18:04 |
jlvillal | mariojv: Trying without -v | 18:04 |
jlvillal | mariojv: I still see the messages on my system. But maybe it is only my system? | 18:05 |
mariojv | i'll check on mine | 18:05 |
*** afaranha has quit IRC | 18:06 | |
mariojv | jlvillal: i don't see that error | 18:06 |
jlvillal | mariojv: Okay, good :) | 18:07 |
*** e0ne has joined #openstack-ironic | 18:07 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Python 3 compatibility https://review.openstack.org/181072 | 18:10 |
*** edcragg has quit IRC | 18:17 | |
*** anteaya has joined #openstack-ironic | 18:18 | |
*** derekh has quit IRC | 18:18 | |
*** zz_natorious is now known as natorious | 18:23 | |
*** kan_ has quit IRC | 18:23 | |
*** kan_ has joined #openstack-ironic | 18:23 | |
jlvillal | jroll: Woo hoo https://review.openstack.org/#/c/176395/ 2 +2s and a +A :) | 18:25 |
*** penick has joined #openstack-ironic | 18:25 | |
NobodyCam | oh wow... BadCub must have magic core powers! | 18:25 |
jroll | I mean, four people pinged nova on that :P | 18:26 |
NobodyCam | magic core SUMMONING powers | 18:26 |
NobodyCam | lol for like a week I thought | 18:26 |
NobodyCam | :? | 18:26 |
* BadCub did what? | 18:29 | |
NobodyCam | lol this is just wrong: https://www.youtube.com/watch?v=XOmcew2DtTE | 18:29 |
*** Marga_ has quit IRC | 18:32 | |
Shrews | NobodyCam: please never tell me what you were googling to find such a video | 18:34 |
NobodyCam | lol | 18:34 |
Shrews | :-P | 18:34 |
Shrews | that guy has waaaay too much time on his hands | 18:35 |
* BadCub watches video and weeps | 18:35 | |
NobodyCam | ++++ | 18:35 |
NobodyCam | thou it is a instersting use of celery | 18:35 |
* BadCub will never eat celery again | 18:36 | |
*** Marga_ has joined #openstack-ironic | 18:36 | |
*** penick has quit IRC | 18:41 | |
*** penick has joined #openstack-ironic | 18:41 | |
*** jistr has quit IRC | 18:51 | |
*** romcheg has joined #openstack-ironic | 18:53 | |
NobodyCam | brb | 18:54 |
*** kan_ has quit IRC | 18:58 | |
*** Sukhdev has joined #openstack-ironic | 18:58 | |
jroll | NobodyCam: amazing | 19:00 |
Sukhdev | devananda: Ping | 19:04 |
*** romcheg has quit IRC | 19:05 | |
Nisha | is anyone aware of this error from latest ironic code http://paste.openstack.org/show/216387/ | 19:06 |
Nisha | I am constantly getting this error on ironic-api and ironic-conductor services | 19:07 |
jroll | Nisha: I saw someone mention it but don't remember the cause, I think it was a client library issue? | 19:08 |
jroll | what is api.py, line 211? | 19:08 |
Nisha | jroll, i am getting this on just starting the services | 19:08 |
jroll | what is api.py, line 211? | 19:09 |
rloo | we recently changed our logging to use oslo_log. | 19:10 |
Nisha | no idea...may be from oslo package | 19:10 |
Nisha | rloo, i reinstalled oslo_log also | 19:11 |
rloo | did the configs change because of that? I don't recall. | 19:11 |
Nisha | rloo, configs of? | 19:11 |
dhellmann | that looks like a problem because the ironic context doesn't have "user_name" defined but the format string for log messages is expecting it | 19:12 |
rloo | logging stuff | 19:12 |
dhellmann | there was a patch to devstack recently for a similar issue | 19:12 |
*** romcheg has joined #openstack-ironic | 19:12 | |
dhellmann | https://review.openstack.org/#/c/180051/ | 19:13 |
dhellmann | rloo, Nisha : ^^ | 19:14 |
rloo | dhellmann: thx | 19:15 |
Nisha | dhellmann, thanks... | 19:16 |
Nisha | dhellmann, but i havent restacked....my stack was already up... | 19:19 |
Nisha | with the fix proposed in ^^^^ i will need to restack??? | 19:19 |
dhellmann | Nisha: I don't know if that fix is actually correct. My suggestion was for the sahara team to fix their context object to work with oslo.log | 19:20 |
*** kozhukalov has quit IRC | 19:20 | |
NobodyCam | Sukhdev: devananda is Down Under in AUS atm. so I expect he won't be around for a few hours yet | 19:22 |
*** Nisha has quit IRC | 19:22 | |
*** Nisha has joined #openstack-ironic | 19:22 | |
Sukhdev | NobodyCam: Oh I see - thanks for the update - I just wanted to check with him if had put together the schedule for the design sessions. I do not see it here - https://wiki.openstack.org/wiki/Summit/Liberty/Etherpads#Neutron | 19:23 |
NobodyCam | Sukhdev: so far the draft is on the top of: https://etherpad.openstack.org/p/liberty-ironic-design-summit-ideas | 19:24 |
Sukhdev | NobodyCam: got it - thanks. I was looking at the wrong place :-) | 19:25 |
NobodyCam | :) | 19:25 |
Nisha | is there anyway i can workaround this issue in ironic...any idea | 19:27 |
*** Marga_ has quit IRC | 19:28 | |
Nisha | dhellmann, ^^^ | 19:29 |
dhellmann | Nisha: the issue is likely that the attributes of the context don't match the values expected in the logging format string, so you could add the missing value to the context | 19:29 |
rloo | hey, who knew about this documentation for ironic CLI: http://docs.openstack.org/cli-reference/content/ironicclient_commands.html | 19:59 |
rloo | NobodyCam, jroll, whoever. Is it true (cuz I don't really grok what impact maintenance=true has) that we don't want to show nodes-in-maintance in a nova hypervisor-stats? https://bugs.launchpad.net/ironic/+bug/1435153 | 20:02 |
openstack | Launchpad bug 1435153 in OpenStack Compute (nova) "ironic hypervisor is still available when the node is in maintenance status." [Undecided,In progress] - Assigned to Zhenguo Niu (niu-zglinux) | 20:02 |
*** achanda has quit IRC | 20:04 | |
*** achanda has joined #openstack-ironic | 20:04 | |
NobodyCam | rloo: that was my understanding. I'll have to look in to it.. on somethign else atm. but ya a node is maintenace should not be seen by nova... though with out looking I could see it taking a minute to update the status of a node | 20:04 |
rloo | NobodyCam: ok. nodes can go in maintenance after they are active though. i'm a bit concerned about hiding that stuff. i suspect the fix for it (if that's what we want) isn't correct. | 20:06 |
*** alexpilotti has quit IRC | 20:08 | |
*** achanda has quit IRC | 20:08 | |
*** alexpilotti has joined #openstack-ironic | 20:10 | |
*** alexpilotti has quit IRC | 20:16 | |
*** Marga_ has joined #openstack-ironic | 20:23 | |
*** Marga_ has quit IRC | 20:23 | |
*** Marga_ has joined #openstack-ironic | 20:24 | |
*** gridinv has joined #openstack-ironic | 20:29 | |
*** gridinv has quit IRC | 20:35 | |
*** bnemec has quit IRC | 20:36 | |
*** Marga_ has quit IRC | 20:41 | |
*** Marga_ has joined #openstack-ironic | 20:42 | |
*** achanda has joined #openstack-ironic | 20:44 | |
*** absubram has quit IRC | 20:49 | |
*** e0ne has quit IRC | 20:51 | |
*** bnemec has joined #openstack-ironic | 20:51 | |
*** ifarkas has quit IRC | 20:57 | |
*** Marga_ has quit IRC | 20:58 | |
*** Sukhdev has quit IRC | 20:58 | |
*** Marga_ has joined #openstack-ironic | 20:58 | |
*** EmilienM is now known as EmilienM|afk | 21:16 | |
*** Nisha has quit IRC | 21:19 | |
*** dprince has quit IRC | 21:20 | |
*** bnemec has quit IRC | 21:25 | |
*** kkoski has quit IRC | 21:31 | |
*** ijw has joined #openstack-ironic | 21:32 | |
*** Sukhdev has joined #openstack-ironic | 21:32 | |
*** trown is now known as trown|outttypeww | 21:48 | |
*** Marga_ has quit IRC | 21:50 | |
*** david-lyle has joined #openstack-ironic | 21:56 | |
*** chlong has quit IRC | 21:59 | |
NobodyCam | humm something up with either devstack or my test env | 22:00 |
NobodyCam | rloo: quick question on 160383 (the temp dir check patch): are you thinking that the message should be replaced or appended to? | 22:03 |
jlvillal | NobodyCam: I wonder if want to create a new exception? | 22:07 |
jlvillal | NobodyCam: Or is that overkill? | 22:07 |
*** Marga_ has joined #openstack-ironic | 22:08 | |
NobodyCam | jlvillal: that patch add two as it is. each one for the different errors | 22:08 |
NobodyCam | https://review.openstack.org/#/c/160383/30/ironic/common/exception.py | 22:08 |
NobodyCam | looking to address ruby comment on https://review.openstack.org/#/c/160383/30/ironic/drivers/modules/ipmitool.py | 22:09 |
jlvillal | NobodyCam: Yeah, I'm not sure best how to do what rloo asked. Sounds like she liked the error message you created. | 22:09 |
NobodyCam | but If i replace the message I will lose some detail | 22:09 |
NobodyCam | or will I? | 22:10 |
NobodyCam | oh I create the new str with the old error txt and then replace... never mind | 22:10 |
NobodyCam | I think I can do it | 22:11 |
NobodyCam | but first a quick smoke | 22:11 |
jlvillal | NobodyCam: I'm not sure. But I think you have a plan! | 22:12 |
jlvillal | NobodyCam: You should buy a bottle of champagne and open it when this patch finally gets merged ;) | 22:13 |
NobodyCam | lol it started as such a easy concept | 22:19 |
NobodyCam | but its no where near the best one.. I think I had one hit 80 or 90 revs | 22:19 |
NobodyCam | folks had bets on if it would hit 100 | 22:20 |
NobodyCam | :-p | 22:20 |
jlvillal | NobodyCam: I don't think you are going to get close on this one. And yes for such a simple concept this has really grown.... | 22:20 |
BadCub | I put a venue/day/time idea on the Summit Pad for an Ironic Group Dinner... Very few places to get [large party] reservations | 22:22 |
jlvillal | JayF: JoshNang: Any idea why this requirements update to IPA is consistently failing? https://review.openstack.org/#/c/176832/ | 22:23 |
*** Marga_ has quit IRC | 22:25 | |
*** Marga_ has joined #openstack-ironic | 22:26 | |
JoshNang | hmmm http://logs.openstack.org/32/176832/4/check/gate-tempest-dsvm-ironic-agent_ssh-src/e156a5c/logs/devstacklog.txt.gz#_2015-05-07_22_21_52_048 | 22:26 |
JoshNang | looks like something broke in the ipa builder | 22:27 |
openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is usable for ipmitool driver https://review.openstack.org/160383 | 22:27 |
NobodyCam | jlvillal: I think that will do it | 22:28 |
*** Marga_ has quit IRC | 22:29 | |
*** Marga_ has joined #openstack-ironic | 22:29 | |
NobodyCam | anyone seeing this from devstack? http://paste.openstack.org/show/Uk8jE6v49kSY2MSqDJn1 | 22:35 |
jlvillal | NobodyCam: Not sure that e.message will work with py34. Maybe e.args[0] | 22:36 |
NobodyCam | jlvillal: huuummmmm | 22:37 |
jlvillal | NobodyCam: I only know that from my recent history.... | 22:38 |
*** david-lyle has quit IRC | 22:38 | |
NobodyCam | jlvillal: I think it should work as all the errors it is re rasing are custom and should have a .message prop | 22:43 |
NobodyCam | jlvillal: do you have a handy test env? | 22:44 |
jlvillal | NobodyCam: Okay, as the 'message' attribute was removed in Python 3.3 I think. | 22:44 |
jlvillal | NobodyCam: Let me run a test.... | 22:44 |
NobodyCam | jlvillal: from built in error. everything that will re reaise is custom from our execptions file | 22:45 |
* NobodyCam dbl checks that | 22:45 | |
jlvillal | NobodyCam: Works. I will remove my -1 :) | 22:46 |
*** ijw has quit IRC | 22:46 | |
NobodyCam | yep! | 22:46 |
NobodyCam | :) | 22:46 |
jlvillal | NobodyCam: +1 :) | 22:47 |
NobodyCam | woo hoo :) | 22:47 |
* BadCub thinks he is done for the day... Have a good night folks!!! | 22:54 | |
*** BadCub is now known as BadCub_Hiding | 22:55 | |
jroll | have we looked at automatic raid provisioning at all? | 22:56 |
jroll | or only operator-initiated through zapping | 22:56 |
jroll | NobodyCam: ^^ | 22:58 |
jroll | like, what if someone wants RAID 1 as the default deploy scheme | 22:58 |
NobodyCam | jroll: I thing the spec calls out that it can be a cleaning step | 23:00 |
jroll | oh? | 23:00 |
NobodyCam | s/thing/think/ | 23:00 |
jroll | not the raid spec afaict | 23:00 |
jroll | or maybe any zap step can be a clean step? | 23:00 |
* jroll wants to see the code, this is when specs are not as useful :/ | 23:00 | |
JayF | any zap step can be a clean step | 23:01 |
JayF | that's a really good point | 23:01 |
jroll | yeah | 23:01 |
jroll | just set the prio | 23:01 |
JayF | although without a way to get that turned on via Ironic | 23:01 |
JayF | it'd require a custom hw manager in the agent to enable that clean step | 23:01 |
jroll | it's via config | 23:01 |
jroll | we can do that in ironic | 23:01 |
jroll | same way we disable erase_devices for the agent | 23:02 |
JayF | That's a one-off priority change | 23:02 |
jroll | or set agent erase_devices in ironic, I should say | 23:02 |
JayF | we don't support that for arbitrary agent steps | 23:02 |
jroll | well, then we should figure that out :) | 23:02 |
jroll | or something, idk | 23:02 |
JayF | I think we should land this as-is, but we need more than just this to do automatic raid config | 23:02 |
jroll | maybe an agent config | 23:02 |
jroll | which is kernel cmd line | 23:02 |
jroll | which is easy | 23:02 |
JayF | but there's no need to hold up this spec for that use case anyway | 23:02 |
jroll | +1 | 23:03 |
jroll | land it with wild abandon | 23:03 |
jlvillal | jroll: Your Nova patch is so close to getting merged..... | 23:15 |
jroll | jlvillal: and? | 23:17 |
*** hemna is now known as hemnafk | 23:17 | |
jroll | I already have it downstream, don't mind too much how long upstream takes :) | 23:17 |
jlvillal | :) | 23:17 |
*** romcheg has quit IRC | 23:19 | |
*** chlong has joined #openstack-ironic | 23:24 | |
jlvillal | JoshNang, JayF: What version of pip is being used in the IPA CoreOS part? | 23:25 |
JayF | jlvillal: pip never runs inside coreos; we build a Docker container based on Ubuntu Trusty, then export that FS and use it as the base for the container | 23:26 |
jlvillal | Looking at: http://logs.openstack.org/32/176832/4/check/gate-tempest-dsvm-ironic-agent_ssh-src/e156a5c/logs/devstacklog.txt.gz#_2015-05-07_22_21_51_926 | 23:26 |
jlvillal | Not sure if python-requests being updated is causing issues with it? Seems to be pip 1.5.2, which seems old. | 23:27 |
jlvillal | But I don't know enough about what is going on there. | 23:27 |
JayF | IncompleteRead? Weird | 23:27 |
JayF | jlvillal: go tup about 10 lines | 23:28 |
jlvillal | JayF: https://review.openstack.org/#/c/176832/4/requirements.txt the update | 23:28 |
*** Marga_ has quit IRC | 23:28 | |
JayF | jlvillal: there's a syntax error | 23:28 |
JayF | http://logs.openstack.org/32/176832/4/check/gate-tempest-dsvm-ironic-agent_ssh-src/e156a5c/logs/devstacklog.txt.gz#_2015-05-07_22_21_49_843 | 23:28 |
JayF | in one of the packages pip is trying to install | 23:28 |
JayF | which is... concerning | 23:28 |
jroll | uh | 23:28 |
jlvillal | JayF: clarkb seemed to think the linecache2 was just line noise. | 23:28 |
jroll | what's the chance 15.04 shipped py3 by default | 23:28 |
JayF | jroll: Why does 15.04 matter? | 23:29 |
jroll | but yeah, might be noise | 23:29 |
JayF | jroll: our container wouldn't magically update itself :) | 23:29 |
jroll | JayF: because I'm not sure if our docker file is FROM trusty or FROM ubuntu | 23:29 |
JayF | https://github.com/openstack/ironic-python-agent/blob/master/Dockerfile | 23:29 |
JayF | FROM ubuntu:trusty | 23:29 |
jroll | k | 23:29 |
jlvillal | JayF: If you look over in #openstack-infra you can see conversation I was having. | 23:29 |
jlvillal | JayF: Suspect is pip 1.5.2 is too old? | 23:30 |
JayF | I'm... suspicious | 23:30 |
JayF | /win 15 | 23:30 |
jroll | could be | 23:30 |
jroll | we should probably pip install -U pip in the dockerfile though | 23:30 |
* jroll tries soemthing | 23:30 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/181204 | 23:31 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/176831 | 23:31 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/176832 | 23:31 |
* jlvillal wonders what kind of mad powers jroll has over OpenStack Proposal Bot! ;) | 23:32 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Use latest pip in Docker build https://review.openstack.org/181206 | 23:32 |
jroll | this "should" fix it I think | 23:32 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/176832 | 23:32 |
*** Sukhdev has quit IRC | 23:32 | |
jroll | ^ makes it a dependency | 23:32 |
jroll | let's see how that gopes | 23:32 |
jroll | goes, even | 23:32 |
jlvillal | jroll, JayF: Thanks for your help! | 23:33 |
* jroll shrugs | 23:33 | |
jroll | no idea if this is helpful :) | 23:33 |
jlvillal | It makes sense to me. See what the test does :) | 23:33 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/181234 | 23:37 |
*** Marga_ has joined #openstack-ironic | 23:42 | |
NobodyCam | lol so it appears that my devstack issue was bifrost started nginx on port 8080 and this prevented swift-proxy from starting which ofcourse caused stack to blow up | 23:58 |
NobodyCam | s/stack/stack.sh/ | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!