| karimb | jroll, devananda thanks buddies, driver loaded :) | 00:02 |
|---|---|---|
| NobodyCam | karimb: awesome! | 00:02 |
| jroll | karimb: woot, you're welcome :) | 00:02 |
| NobodyCam | looking forward for the review of it | 00:02 |
| NobodyCam | :) | 00:02 |
| *** verdurin has quit IRC | 00:03 | |
| *** amotoki has joined #openstack-ironic | 00:03 | |
| NobodyCam | s/for/to/ | 00:03 |
| karimb | NobodyCam, yeah, time to sleep, tomorrow i ll test it. quite confident on the ovirt code, no clue on ironic stuff, just looked at pxe_vbox and adapted :) | 00:04 |
| NobodyCam | karimb: awesome... we'll be here tormorrow if you have questions :) | 00:05 |
| * NobodyCam will be devananda will be on a plane | 00:05 | |
| NobodyCam | :-p | 00:05 |
| *** verdurin has joined #openstack-ironic | 00:06 | |
| *** Haomeng|2 has joined #openstack-ironic | 00:07 | |
| *** karimb has quit IRC | 00:08 | |
| *** amotoki has quit IRC | 00:08 | |
| *** Haomeng has quit IRC | 00:10 | |
| *** Kennan2 is now known as Kennan | 00:11 | |
| *** naohirot has joined #openstack-ironic | 00:14 | |
| *** early has quit IRC | 00:26 | |
| *** puranamr has quit IRC | 00:40 | |
| *** mtanino has quit IRC | 00:40 | |
| harlowja_ | something u guys might be interseted in (ironic has experience here that i think nova folks can benefit from) | 00:47 |
| harlowja_ | https://review.openstack.org/#/c/197668/ | 00:47 |
| *** puranamr has joined #openstack-ironic | 00:47 | |
| harlowja_ | 'Introduce a simple state machine module' (into nova) | 00:47 |
| harlowja_ | if u guys are interested, i think the ironic experience could be valueable for nova to learn from | 00:48 |
| *** h00327910_ has joined #openstack-ironic | 00:49 | |
| * harlowja_ is trying to convince nikola there to not use auto-generated transition names but invest the time in figuring out the ING transition names | 00:51 | |
| * harlowja_ i may or may not convince him :-P | 00:51 | |
| *** Kennan has quit IRC | 00:54 | |
| *** Haomeng|2 has quit IRC | 00:54 | |
| *** Haomeng|2 has joined #openstack-ironic | 00:55 | |
| *** Kennan has joined #openstack-ironic | 00:58 | |
| *** yuanying_ has joined #openstack-ironic | 01:02 | |
| *** amotoki has joined #openstack-ironic | 01:04 | |
| *** yuanying has quit IRC | 01:05 | |
| *** shalq has quit IRC | 01:07 | |
| *** yuanying_ has quit IRC | 01:07 | |
| *** amotoki has quit IRC | 01:09 | |
| *** kkoski has joined #openstack-ironic | 01:12 | |
| *** kkoski has joined #openstack-ironic | 01:12 | |
| *** chlong has joined #openstack-ironic | 01:20 | |
| *** kkoski has quit IRC | 01:21 | |
| *** rbrooker has quit IRC | 01:22 | |
| *** kkoski has joined #openstack-ironic | 01:22 | |
| *** yuanying has joined #openstack-ironic | 01:24 | |
| *** kkoski_ has joined #openstack-ironic | 01:25 | |
| *** kkoski__ has joined #openstack-ironic | 01:27 | |
| *** kkoski has quit IRC | 01:27 | |
| *** rloo has quit IRC | 01:28 | |
| *** chenglch has joined #openstack-ironic | 01:28 | |
| *** yuanying has quit IRC | 01:28 | |
| *** kkoski has joined #openstack-ironic | 01:29 | |
| *** kkoski_ has quit IRC | 01:29 | |
| *** yuanying has joined #openstack-ironic | 01:29 | |
| *** yuanying has quit IRC | 01:30 | |
| zer0c00l1 | I am trying to setup devstack/ironic | 01:31 |
| zer0c00l1 | my setup dies aty | 01:31 |
| zer0c00l1 | [ERROR] /opt/stack/devstack/lib/ironic:704 server didn't become ssh-able! | 01:31 |
| *** kkoski__ has quit IRC | 01:31 | |
| zer0c00l1 | When i try to manually ssh to the host , it works | 01:32 |
| zer0c00l1 | but the script seems to be dying with 'killed by SIGNAL 15' | 01:32 |
| *** kkoski_ has joined #openstack-ironic | 01:32 | |
| *** yuanying has joined #openstack-ironic | 01:33 | |
| *** kkoski has quit IRC | 01:33 | |
| *** kkoski has joined #openstack-ironic | 01:37 | |
| *** kkoski_ has quit IRC | 01:37 | |
| *** Sukhdev has quit IRC | 01:56 | |
| Haomeng|2 | zer0c00l1: can you try with the command "ssh -i ..." in the log by manually again, to see if the ssh command works or not | 01:57 |
| *** Haomeng|2 is now known as Haomeng | 01:58 | |
| Haomeng | zer0c00l1: if it works, just commen line 704 out or change 'die' to 'echo | 01:59 |
| Haomeng | zer0c00l1: that should be a workaround | 01:59 |
| Haomeng | zer0c00l1: the command should be - "timeout 15 sh -c 'while ! ssh -p 22 -o StrictHostKeyChecking=no -i /opt/stack/data/ironic/ssh_keys/ironic_key stack@$YOUR_DEVSTACK_IP echo success; do sleep 1; done'" | 02:04 |
| *** kan__ has joined #openstack-ironic | 02:05 | |
| *** amotoki has joined #openstack-ironic | 02:05 | |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/ironic: Add port creation passing the name of the node instead of the UUID of the node https://review.openstack.org/193439 | 02:08 |
| *** amotoki has quit IRC | 02:09 | |
| Haomeng | zer0c00l1: I guess $YOUR_DEVSTACK_IP host does not support ssh key, just supports password, so default devstack script not working without PubkeyAuthentication enabled | 02:11 |
| zer0c00l1 | i see | 02:13 |
| zer0c00l1 | Haomeng: i am able to run the same command as stack user | 02:13 |
| zer0c00l1 | and it works | 02:13 |
| zer0c00l1 | i worked around it by commenting out that ssh check | 02:13 |
| zer0c00l1 | Haomeng: Thanks! | 02:13 |
| Haomeng | zer0c00l1: welcome:) | 02:15 |
| *** puranamr has quit IRC | 02:19 | |
| *** gzhai has quit IRC | 02:19 | |
| *** achanda has joined #openstack-ironic | 02:23 | |
| openstackgerrit | Merged openstack/bifrost: Move diskimage-builder and bootable image creation to roles https://review.openstack.org/199329 | 02:38 |
| *** kkoski has quit IRC | 02:39 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add CLEANWAIT state - DO NOT MERGE https://review.openstack.org/200152 | 02:42 |
| *** achanda has quit IRC | 02:42 | |
| jroll | harlowja_: interesting | 02:42 |
| *** lintan has joined #openstack-ironic | 02:48 | |
| *** amotoki has joined #openstack-ironic | 03:05 | |
| *** Kennan has quit IRC | 03:07 | |
| *** Kennan has joined #openstack-ironic | 03:07 | |
| *** amotoki has quit IRC | 03:10 | |
| openstackgerrit | Merged openstack/ironic: Migrate RPC objects to oslo.versionedobjects Fields https://review.openstack.org/198755 | 03:10 |
| openstackgerrit | Zhenguo Niu proposed openstack/python-ironicclient: Filtering nodes by provision state https://review.openstack.org/197012 | 03:11 |
| *** rwsu has quit IRC | 03:11 | |
| *** Nisha has joined #openstack-ironic | 03:25 | |
| *** Nisha has quit IRC | 03:29 | |
| *** zz_natorious is now known as natorious | 03:36 | |
| *** Nisha has joined #openstack-ironic | 03:36 | |
| *** early has joined #openstack-ironic | 03:39 | |
| *** Sukhdev has joined #openstack-ironic | 04:06 | |
| *** amotoki has joined #openstack-ironic | 04:06 | |
| *** amotoki has quit IRC | 04:11 | |
| *** achanda has joined #openstack-ironic | 04:20 | |
| *** vishwanathj has quit IRC | 04:56 | |
| *** vishwana_ has joined #openstack-ironic | 04:59 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: Periodically check nodes' existance https://review.openstack.org/197106 | 05:04 |
| *** amotoki has joined #openstack-ironic | 05:07 | |
| openstackgerrit | Syed Ismail Faizan Barmawer proposed openstack/ironic-lib: Merge latest disk partitioning code from ironic https://review.openstack.org/162162 | 05:09 |
| *** amotoki has quit IRC | 05:12 | |
| *** Marga_ has joined #openstack-ironic | 05:23 | |
| *** Marga_ has quit IRC | 05:24 | |
| *** Marga_ has joined #openstack-ironic | 05:25 | |
| *** natorious is now known as zz_natorious | 05:34 | |
| *** david-ly_ has joined #openstack-ironic | 05:37 | |
| *** david-lyle has quit IRC | 05:40 | |
| *** ig0r__ has joined #openstack-ironic | 05:52 | |
| *** lazy_prince has joined #openstack-ironic | 05:54 | |
| *** Sukhdev has quit IRC | 05:54 | |
| *** ig0r_ has quit IRC | 05:55 | |
| lazy_prince | good morning ironickers.... | 05:58 |
| *** Kennan has quit IRC | 06:02 | |
| *** Kennan has joined #openstack-ironic | 06:03 | |
| *** Kennan2 has joined #openstack-ironic | 06:06 | |
| *** radek__ has joined #openstack-ironic | 06:06 | |
| *** Kennan has quit IRC | 06:07 | |
| *** amotoki has joined #openstack-ironic | 06:08 | |
| *** achanda has quit IRC | 06:11 | |
| Nisha | lazy_prince, GM | 06:12 |
| *** amotoki has quit IRC | 06:12 | |
| lazy_prince | Nisha: gm.. | 06:13 |
| *** dtantsur|afk is now known as dtantsur | 06:13 | |
| dtantsur | Morning folks, happy Friday :) | 06:13 |
| lazy_prince | dtantsur: GM | 06:14 |
| Nisha | dtantsur, gm... | 06:14 |
| *** vishwana_ has quit IRC | 06:19 | |
| *** vishwanathj has joined #openstack-ironic | 06:19 | |
| *** vishwana_ has joined #openstack-ironic | 06:22 | |
| *** subscope has quit IRC | 06:25 | |
| *** vishwanathj has quit IRC | 06:25 | |
| *** subscope has joined #openstack-ironic | 06:37 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Fixes a testcase ralated to trusted boot in UEFI boot mode. https://review.openstack.org/200398 | 06:56 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Fixes a testcase related to trusted boot in UEFI boot mode https://review.openstack.org/200398 | 06:59 |
| *** yog__ has joined #openstack-ironic | 07:02 | |
| *** jcoufal has joined #openstack-ironic | 07:07 | |
| *** ifarkas has joined #openstack-ironic | 07:08 | |
| *** davideag_ has joined #openstack-ironic | 07:16 | |
| *** jistr has joined #openstack-ironic | 07:16 | |
| *** jamielennox is now known as jamielennox|away | 07:17 | |
| *** davideagnello has quit IRC | 07:18 | |
| openstackgerrit | Merged openstack/ironic-specs: Make ilo drivers standalone work without swift https://review.openstack.org/193478 | 07:18 |
| dtantsur | Nisha, some good news ^^ | 07:21 |
| Nisha | dtantsur, oh wow | 07:22 |
| Nisha | :) | 07:22 |
| Nisha | thanks | 07:22 |
| Nisha | will repost the patches | 07:22 |
| Nisha | dtantsur, wanted to ask on switch id and switch details inspection which was planned to be included in inspector....is that done or planned for liberty? | 07:23 |
| dtantsur | Nisha, you mean LLDP? | 07:24 |
| Nisha | yes | 07:24 |
| dtantsur | planned for liberty | 07:24 |
| dtantsur | with switch to IPA as a base for ramdisk | 07:24 |
| Nisha | oh so inspector ramdisk will go off? | 07:24 |
| dtantsur | Nisha, it will be based on IPA to be precise :) but yeah, essentially it's going to be one ramdisk | 07:25 |
| Nisha | dtantsur, hmmm:) ok | 07:25 |
| *** eglute has quit IRC | 07:30 | |
| *** davideag_ has quit IRC | 07:31 | |
| *** yuikotakada has joined #openstack-ironic | 07:35 | |
| *** kan__ has quit IRC | 07:52 | |
| *** dtantsur is now known as dtantsur|brb | 07:53 | |
| *** ralfvand0__ has quit IRC | 07:54 | |
| *** yuikotakada has quit IRC | 07:56 | |
| *** amotoki has joined #openstack-ironic | 08:09 | |
| *** chlong has quit IRC | 08:10 | |
| *** yuanying has quit IRC | 08:10 | |
| *** amotoki has quit IRC | 08:14 | |
| *** lucasagomes has joined #openstack-ironic | 08:16 | |
| *** Nisha has quit IRC | 08:17 | |
| lucasagomes | jroll, awesome! Thanks for the update on the CLEANWAIT patch | 08:18 |
| *** dlpartain has joined #openstack-ironic | 08:19 | |
| *** dlpartain has quit IRC | 08:24 | |
| *** dlpartain has joined #openstack-ironic | 08:24 | |
| *** dlpartain has joined #openstack-ironic | 08:26 | |
| *** coolsvap|afk is now known as coolsvap|away | 08:26 | |
| *** dlpartain has quit IRC | 08:26 | |
| *** coolsvap|away is now known as coolsvap | 08:28 | |
| *** romcheg has joined #openstack-ironic | 08:31 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add CLEANWAIT state https://review.openstack.org/200152 | 08:34 |
| *** Marga__ has joined #openstack-ironic | 08:44 | |
| *** Marga_ has quit IRC | 08:45 | |
| *** yuikotakada has joined #openstack-ironic | 08:49 | |
| *** jistr has quit IRC | 09:00 | |
| *** pelix has joined #openstack-ironic | 09:06 | |
| *** stendulker has joined #openstack-ironic | 09:07 | |
| *** amotoki has joined #openstack-ironic | 09:10 | |
| *** athomas has joined #openstack-ironic | 09:12 | |
| *** amotoki has quit IRC | 09:14 | |
| *** jistr has joined #openstack-ironic | 09:16 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add db migration for tags and node_tags table https://review.openstack.org/192576 | 09:27 |
| *** Marga__ has quit IRC | 09:32 | |
| *** Marga_ has joined #openstack-ironic | 09:32 | |
| *** ndipanov has joined #openstack-ironic | 09:33 | |
| *** radek_ has joined #openstack-ironic | 09:46 | |
| *** max_lobur has joined #openstack-ironic | 09:46 | |
| *** radek__ has quit IRC | 09:47 | |
| *** radek__ has joined #openstack-ironic | 09:50 | |
| *** e0ne has joined #openstack-ironic | 09:51 | |
| *** radek_ has quit IRC | 09:53 | |
| yuikotakada | Hi, Ironic | 09:53 |
| *** naohirot has quit IRC | 10:00 | |
| lucasagomes | yuikotakada, good morning | 10:01 |
| lucasagomes | ugt* | 10:01 |
| yuikotakada | lucasagomes, good ugt morning :) | 10:01 |
| *** chenglch has quit IRC | 10:01 | |
| yuikotakada | lucasagomes, is ironic also be sufferd from latest mock? http://lists.openstack.org/pipermail/openstack-dev/2015-July/069156.html | 10:06 |
| lucasagomes | yuikotakada, apparently yeah, our gate seems broken | 10:06 |
| lucasagomes | I'm looking at it see if I can find the problem | 10:06 |
| *** radek_ has joined #openstack-ironic | 10:07 | |
| yuikotakada | lucasagomes, oh >_< we need to fix some wrong usage of mock | 10:07 |
| lucasagomes | yup | 10:07 |
| lucasagomes | soonish!! | 10:08 |
| lucasagomes | that's a good improvement of mock actually, it was annoying that you could call it with whatever | 10:08 |
| yuikotakada | lucasagomes, yup, it is unfortunate that today is friday | 10:09 |
| *** radek__ has quit IRC | 10:09 | |
| *** romcheg has quit IRC | 10:09 | |
| *** romcheg has joined #openstack-ironic | 10:10 | |
| lucasagomes | lol true! | 10:10 |
| *** smoriya has quit IRC | 10:11 | |
| *** yog__ has quit IRC | 10:25 | |
| *** cascone has joined #openstack-ironic | 10:30 | |
| *** amotoki has joined #openstack-ironic | 10:31 | |
| *** dtantsur|brb is now known as dtantsur | 10:40 | |
| *** zhenguo has quit IRC | 10:41 | |
| dtantsur | hey lucasagomes, yuikotakada! | 10:41 |
| dtantsur | yuikotakada, are we broken as well? | 10:41 |
| yuikotakada | dtantsur, o/ | 10:42 |
| lucasagomes | dtantsur, yup | 10:42 |
| lucasagomes | patch coming | 10:42 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix unittests due mock 1.1.0 release https://review.openstack.org/200466 | 10:43 |
| *** coolsvap is now known as coolsvap|away | 10:43 | |
| lucasagomes | dtantsur, ^ | 10:43 |
| yuikotakada | dtantsur, yep, please see https://bugs.launchpad.net/ironic-inspector/+bug/1473381 | 10:43 |
| openstack | Launchpad bug 1473381 in Ironic Inspector "mock usage is wrong" [Undecided,New] | 10:43 |
| dtantsur | yuikotakada, omg, how came it broke so many tests... are you working on a fix or should I? | 10:44 |
| lucasagomes | dtantsur, mostly must be due side_effect | 10:44 |
| lucasagomes | not being iter | 10:45 |
| yuikotakada | lucasagomes, dtantsur , yes, I think so too and I'm trying to inspector now | 10:46 |
| lucasagomes | yuikotakada, cool! thanks for looking into it | 10:46 |
| dtantsur | can someone educate these mock devs about semver, backward compatibility and all other weird things that the industry does? | 10:47 |
| *** e0ne is now known as e0ne_ | 10:49 | |
| dtantsur | lucasagomes, I guess we (openstack) should consider alternatives to mock. writing iter([Exception(..)]) is insane | 10:51 |
| lucasagomes | dtantsur, right... well yeah probably we will need a big discussion about it before | 10:52 |
| lucasagomes | worth sending it to the list ideas, about alternatives and all | 10:52 |
| * lucasagomes doesn't know much about alternatives | 10:52 | |
| *** eglute has joined #openstack-ironic | 10:53 | |
| *** jamielennox|away is now known as jamielennox | 10:54 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix unittests due mock 1.1.0 release https://review.openstack.org/200466 | 10:56 |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: Fix usage of mock due for the latest 1.1.0 mock https://review.openstack.org/200471 | 10:57 |
| *** stendulker has quit IRC | 10:59 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Use DEPLOYWAIT while waiting for agent to write image https://review.openstack.org/200153 | 11:01 |
| *** e0ne_ is now known as e0ne | 11:01 | |
| * dtantsur sent a email | 11:04 | |
| dtantsur | yuikotakada, one small request on the patch | 11:07 |
| *** romcheg has quit IRC | 11:10 | |
| yuikotakada | dtantsur, thank you for reviewing, you mean fix 2.1.0 -> 2.0.1 in [metadata] in setup.cfg? | 11:10 |
| TheJulia | Good morning! | 11:11 |
| yuikotakada | TheJulia, good morning :) | 11:11 |
| lucasagomes | TheJulia, good ugt morning | 11:14 |
| *** lucasagomes is now known as lucas-hungry | 11:15 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: Fix usage of mock due for the latest 1.1.0 mock https://review.openstack.org/200471 | 11:21 |
| lifeless | dtantsur: you know, thats actually pretty insulting | 11:21 |
| lifeless | dtantsur: calling out the 'mock devs' like that | 11:22 |
| *** dlpartain has joined #openstack-ironic | 11:22 | |
| dtantsur | yuikotakada, yep | 11:26 |
| yuikotakada | dtantsur, done | 11:26 |
| dtantsur | lifeless, I'm sorry. Please tell me how you react when people release a minor version purposely breaking tons of code? | 11:27 |
| lifeless | dtantsur: I try to understand whats going on before accussing people of being either uneducated or malicious | 11:27 |
| lifeless | dtantsur: I feel like you don't understand; to whit: there was no purpose breaking of code. | 11:27 |
| dtantsur | lifeless, I didn't call anyone uneducated. I only stated that public API was broken. | 11:28 |
| lifeless | 22:47 < dtantsur> can someone educate these mock devs about semver, backward compatibility and all other weird things that the industry does? | 11:28 |
| openstackgerrit | Merged openstack/bifrost: Make inventory plugin errors visible https://review.openstack.org/200284 | 11:28 |
| dtantsur | lifeless, ok, I'm sorry for that one, that was inconstructive rant, which I tried not to bring to the ML | 11:29 |
| lifeless | ok | 11:29 |
| dtantsur | still, that was a breakage IMO and did deserve major version bump and proper communication | 11:29 |
| lazy_prince | TheJulia: Good morning.. | 11:31 |
| lazy_prince | TheJulia: I was able to bring up IPA with debian. but for some reason, its not working with ubuntu. | 11:31 |
| lazy_prince | The IP address itself is not getting assigned to the interfaces. | 11:32 |
| *** e0ne is now known as e0ne_ | 11:35 | |
| *** dprince has joined #openstack-ironic | 11:35 | |
| TheJulia | lazy_prince: ubuntu or ubuntu-minimal? | 11:36 |
| dtantsur | lifeless, do you know why http://www.voidspace.org.uk/python/mock/mock.html#mock.Mock.side_effect still mentions that you can assign exception instance to side_effect? | 11:36 |
| lazy_prince | TheJulia: ubuntu did not try with ubuntu-minimal | 11:37 |
| lazy_prince | TheJulia: i mean i tried with ubuntu | 11:37 |
| lifeless | dtantsur: thats the 1.0.1 docs | 11:37 |
| TheJulia | so ubuntu has cloud-init burned into the image if memory serves, ubuntu-minimal may be less problematic | 11:37 |
| lifeless | dtantsur: https://docs.python.org/dev/library/unittest.mock.html | 11:37 |
| lazy_prince | k.. let me try with ubuntu-minimal | 11:38 |
| dtantsur | lifeless, "Alternatively side_effect can be an exception class or instance" | 11:38 |
| *** romcheg has joined #openstack-ironic | 11:38 | |
| dtantsur | that's from the latter link | 11:39 |
| dtantsur | https://docs.python.org/dev/library/unittest.mock.html#unittest.mock.Mock.side_effect also has an example for using Exception | 11:39 |
| lifeless | yup | 11:40 |
| lifeless | it should work | 11:40 |
| *** lintan has quit IRC | 11:41 | |
| dtantsur | lifeless, well, at least ironic and ironic inspector gates were broken due to it | 11:41 |
| dtantsur | patches https://review.openstack.org/#/c/200471 and https://review.openstack.org/200466 are fixing it | 11:42 |
| dtantsur | I wonder if these 2 projects are any different from other ones.. | 11:42 |
| *** lintan has joined #openstack-ironic | 11:43 | |
| *** slagle has joined #openstack-ironic | 11:45 | |
| lifeless | dtantsur: so, side_effect being an exception works for me here | 11:50 |
| lifeless | dtantsur: but | 11:50 |
| lifeless | dtantsur: there seems to be a potential bug in autospecs | 11:50 |
| lifeless | dtantsur: you might try changing from autospec=True to spec=True and seeing if that works | 11:50 |
| dtantsur | lifeless, won't it be a regression in our test coverage? | 11:50 |
| lifeless | dtantsur: how so ? | 11:51 |
| dtantsur | lifeless, "spec: This can be either a list of strings or an existing object" will it actually work? | 11:51 |
| dtantsur | i.e. we should provide some real object to 'spec', not just 'True' | 11:52 |
| lifeless | huh? no | 11:53 |
| lifeless | https://review.openstack.org/#/c/200466/2/ironic/tests/drivers/ilo/test_deploy.py | 11:53 |
| lifeless | line 1051 | 11:53 |
| lifeless | change autospec to spec | 11:53 |
| lifeless | thats all | 11:53 |
| lifeless | I have to go sleep | 11:54 |
| dtantsur | lifeless, according to the documentation, after that mock will be build from a boolean object, no? | 11:54 |
| TheJulia | goodnight lifeless | 11:54 |
| lifeless | but the list thing is definitely wonky | 11:54 |
| lifeless | dtantsur: I can't stay to convince you to do a 30 second experiment, sorry. | 11:54 |
| * dtantsur tries | 11:54 | |
| lifeless | dtantsur: it worked for nova, it worked for neutron, and it going to be looked into to see what the cause is | 11:54 |
| lifeless | I suspect a bug in funcsigs | 11:55 |
| lifeless | night TheJulia | 11:55 |
| dtantsur | lifeless, ok thanks, I'll report results to the ML | 11:55 |
| *** romcheg has quit IRC | 12:00 | |
| *** romcheg has joined #openstack-ironic | 12:00 | |
| *** absubram has joined #openstack-ironic | 12:08 | |
| *** trown|outttypeww is now known as trown | 12:08 | |
| *** absubram has quit IRC | 12:10 | |
| *** absubram has joined #openstack-ironic | 12:10 | |
| *** thrash|g0ne is now known as thrash | 12:12 | |
| *** yuikotakada is now known as yuikot_go_home | 12:13 | |
| *** yuikot_go_home has quit IRC | 12:13 | |
| openstackgerrit | Satoru Moriya proposed openstack/ironic-specs: Add attributes about volume conneciton into nodes table https://review.openstack.org/200496 | 12:15 |
| *** ig0r__ has quit IRC | 12:17 | |
| *** e0ne_ is now known as e0ne | 12:21 | |
| *** ig0r_ has joined #openstack-ironic | 12:23 | |
| *** dlpartain has left #openstack-ironic | 12:23 | |
| openstackgerrit | Satoru Moriya proposed openstack/ironic-specs: Add attributes about volume conneciton into nodes table https://review.openstack.org/200496 | 12:27 |
| *** david-lyle has joined #openstack-ironic | 12:28 | |
| *** chlong has joined #openstack-ironic | 12:28 | |
| *** david-ly_ has quit IRC | 12:30 | |
| *** lucas-hungry is now known as lucasagomes | 12:34 | |
| *** absubram has quit IRC | 12:43 | |
| openstackgerrit | Merged openstack/ironic-inspector: Fix usage of mock due for the latest 1.1.0 mock https://review.openstack.org/200471 | 12:43 |
| *** e0ne is now known as e0ne_ | 12:46 | |
| *** coolsvap|away has quit IRC | 12:47 | |
| *** kbyrne has quit IRC | 12:47 | |
| *** e0ne_ is now known as e0ne | 12:47 | |
| *** coolsvap|away has joined #openstack-ironic | 12:51 | |
| *** coolsvap|away is now known as coolsvap | 12:53 | |
| *** derekh has joined #openstack-ironic | 13:03 | |
| jroll | lucasagomes: you're welcome sir :) | 13:03 |
| lucasagomes | jroll, thanks | 13:03 |
| lucasagomes | jroll, and good morning! | 13:03 |
| dtantsur | jroll, morning! | 13:03 |
| jroll | morning lucas, dtantsur :) | 13:04 |
| jroll | looking at gate fix now | 13:04 |
| lucasagomes | jroll, btw, if you have time please take a look at https://review.openstack.org/200466 the gate is broken and that's fixing it | 13:04 |
| lucasagomes | oh ok | 13:04 |
| lucasagomes | hah | 13:04 |
| lucasagomes | thanks again! | 13:04 |
| *** cdearborn has joined #openstack-ironic | 13:04 | |
| jroll | yeah just saw scrollback :P | 13:04 |
| jroll | lucasagomes: as far as the cleaning thing I think the patch I pushed fixes any compatibility issues but we should think about it hard | 13:04 |
| jroll | and idk if we want to add a deprecation message or anything | 13:05 |
| lucasagomes | jroll, oh that's a good point | 13:05 |
| lucasagomes | yuiko also pointed me to update the state machine spec | 13:05 |
| lucasagomes | that's what I'm doing now | 13:05 |
| jroll | oh cool | 13:05 |
| lucasagomes | jroll, I will add some deprecated messages | 13:05 |
| jroll | really that should be in our dev docs | 13:05 |
| lucasagomes | I've tested the patch this morning and it works | 13:05 |
| jroll | \o/ | 13:05 |
| lucasagomes | but cleaning takes shit loads of time! | 13:05 |
| jroll | no | 13:05 |
| jroll | shred does ;D | 13:05 |
| lucasagomes | only now I realised that I could create a 1GB disk for the vm to test | 13:06 |
| lucasagomes | that's way faster | 13:06 |
| lucasagomes | jroll, oh yeah | 13:06 |
| jroll | if you have a drive that supports ATA secure erase it's super fast | 13:06 |
| *** bizarrochristy has joined #openstack-ironic | 13:06 | |
| * jroll hugs SSDs | 13:06 | |
| lucasagomes | right yeah, I wish | 13:08 |
| lucasagomes | actually I was thinking about getting a ssd for my laptop | 13:08 |
| jroll | oh god, yes do it | 13:08 |
| jroll | it's like the best upgrade you can get for a computer these days | 13:09 |
| jroll | or most impactful I should say | 13:09 |
| *** jamielennox is now known as jamielennox|away | 13:09 | |
| lucasagomes | yeah it def will boost the speed here | 13:10 |
| * lucasagomes looks at amazon | 13:10 | |
| lucasagomes | jroll, what you recommend? Samsung? I haven't looked much into it | 13:11 |
| jroll | lucasagomes: yeah, samsung 850 seems to be the recommended these days | 13:11 |
| jroll | er, 850 pro | 13:11 |
| jroll | http://www.amazon.com/Samsung-256GB-2-5-Inch-Internal-MZ-7KE256BW/dp/B00LMXBOP4 | 13:11 |
| lucasagomes | cool | 13:12 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Use DEPLOYWAIT while waiting for agent to write image https://review.openstack.org/200153 | 13:13 |
| jroll | just rebased this on top of the gate fix | 13:13 |
| *** athomas has quit IRC | 13:13 | |
| lucasagomes | cool | 13:14 |
| jroll | oh heh switching to deploy_has_started unbroke pxe/ipa | 13:14 |
| jroll | I didn't think about that case | 13:14 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Update the new state machine spec with CLEANWAIT https://review.openstack.org/200521 | 13:16 |
| lucasagomes | yeah and there's a patch to make that job voting | 13:17 |
| *** athomas has joined #openstack-ironic | 13:20 | |
| lucasagomes | dtantsur, re https://review.openstack.org/#/c/150488/ we agreed that for vendor passthru methods we don't need a spec | 13:21 |
| dtantsur | lucasagomes, then we should not reference an existing one, no? | 13:22 |
| lucasagomes | yeah... | 13:22 |
| lucasagomes | just update the commit message maybe? | 13:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Switch to pbr postversioning https://review.openstack.org/200527 | 13:24 |
| dtantsur | lucasagomes, I think so. I abandon the blueprint | 13:25 |
| lucasagomes | ack | 13:26 |
| dtantsur | s/I/and/ because I doubt I have rights to abandon blueprints | 13:26 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add DRAC BIOS config vendor passthru API https://review.openstack.org/150488 | 13:27 |
| *** cdearborn has quit IRC | 13:31 | |
| *** kkoski has joined #openstack-ironic | 13:32 | |
| NobodyCam | good morning Ironicers | 13:33 |
| NobodyCam | and .... | 13:33 |
| NobodyCam | T G I F | 13:33 |
| dtantsur | NobodyCam, hey, morning :) TGIF really | 13:33 |
| jroll | good morning NobodyCam, TGIF! :) | 13:33 |
| lucasagomes | NobodyCam, good ugt morning TGIF! | 13:33 |
| lazy_prince | TheJulia: looks like curl is missing from ubuntu-minimal. so it fails when downloading get-pip.py. trying with wget now.. | 13:33 |
| dtantsur | we celebrated this Friday with broken gates :) | 13:34 |
| NobodyCam | morning dtantsur jroll lucasagomes and lazy_prince | 13:34 |
| NobodyCam | oh joy | 13:34 |
| lazy_prince | GM NobodyCam | 13:34 |
| NobodyCam | :) | 13:34 |
| lazy_prince | is the broken gate issue resolved..? | 13:35 |
| dtantsur | lazy_prince, patch for ironic is on its way | 13:35 |
| dtantsur | I guess Nova is still investigating their side | 13:35 |
| NobodyCam | is it onn the whiteboard? | 13:37 |
| dtantsur | not yet | 13:37 |
| *** cdearborn has joined #openstack-ironic | 13:38 | |
| NobodyCam | ack :) | 13:40 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add CLEANWAIT state https://review.openstack.org/200152 | 13:50 |
| * lucasagomes added deprecated messages | 13:51 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove deprecated code for driver vendor passthru https://review.openstack.org/200547 | 13:57 |
| openstackgerrit | Merged openstack/ironic: Fix unittests due mock 1.1.0 release https://review.openstack.org/200466 | 13:58 |
| *** dlpartain has joined #openstack-ironic | 14:11 | |
| *** cdearborn has quit IRC | 14:11 | |
| *** dlpartain has left #openstack-ironic | 14:12 | |
| *** kbyrne has joined #openstack-ironic | 14:12 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Periodically checks the status of nodes in DEPLOYING state https://review.openstack.org/197141 | 14:18 |
| lucasagomes | btw jroll ^ with those fixes you may want to lift the vote | 14:19 |
| lucasagomes | actaully i will rebase on top of ur fix | 14:19 |
| jroll | lucasagomes: cool, ty sir, I'll come back to it in a bit | 14:19 |
| lucasagomes | enjoy | 14:20 |
| jroll | and yay for gate no longer being broken | 14:20 |
| *** jistr is now known as jistr|mtg | 14:22 | |
| NobodyCam | ++ | 14:22 |
| lucasagomes | w00t | 14:23 |
| *** mtanino has joined #openstack-ironic | 14:23 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Periodically checks the status of nodes in DEPLOYING state https://review.openstack.org/197141 | 14:23 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Disable cleaning by default in bifrost https://review.openstack.org/200566 | 14:35 |
| *** kbyrne has quit IRC | 14:36 | |
| BadCub | hiya folks | 14:41 |
| TheJulia | Good morning BadCub | 14:43 |
| BadCub | Morning TheJulia :) | 14:43 |
| *** viktors is now known as viktors|weekend | 14:44 | |
| *** zz_natorious is now known as natorious | 14:50 | |
| *** coolsvap is now known as coolsvap|away | 14:50 | |
| dtantsur | BadCub, o/ | 14:50 |
| * lazy_prince signing out for the day.. | 14:51 | |
| openstackgerrit | Merged openstack/ironic-inspector: Switch to pbr postversioning https://review.openstack.org/200527 | 14:52 |
| TheJulia | goodnight lazy_prince | 14:53 |
| *** chlong is now known as chlong-weekend | 14:54 | |
| lucasagomes | BadCub, morning | 14:55 |
| BadCub | happy friday dtantsur and lucasagomes :) | 14:57 |
| dtantsur | :) | 14:57 |
| NobodyCam | have a good night lazy_prince | 14:58 |
| *** lazy_prince has quit IRC | 14:59 | |
| *** natorious is now known as zz_natorious | 15:03 | |
| *** alex_xu has quit IRC | 15:08 | |
| *** Sukhdev has joined #openstack-ironic | 15:08 | |
| *** amotoki has quit IRC | 15:09 | |
| *** alex_xu has joined #openstack-ironic | 15:10 | |
| *** mgoddard has quit IRC | 15:11 | |
| *** mgoddard has joined #openstack-ironic | 15:12 | |
| *** mgoddard has quit IRC | 15:12 | |
| NobodyCam | jroll: one question on 185171 before I +A it | 15:13 |
| *** mgoddard has joined #openstack-ironic | 15:13 | |
| NobodyCam | on the Data model impact section you have NOne. is there anything needed to support the switch to symversioning? | 15:14 |
| *** mgoddard has quit IRC | 15:14 | |
| *** mgoddard has joined #openstack-ironic | 15:15 | |
| *** r-daneel has joined #openstack-ironic | 15:16 | |
| *** max_lobur has quit IRC | 15:17 | |
| *** ig0r__ has joined #openstack-ironic | 15:19 | |
| *** romcheg has quit IRC | 15:22 | |
| *** kbyrne has joined #openstack-ironic | 15:22 | |
| *** ig0r_ has quit IRC | 15:22 | |
| *** chlong-weekend has quit IRC | 15:26 | |
| *** e0ne is now known as e0ne_ | 15:28 | |
| *** absubram has joined #openstack-ironic | 15:28 | |
| *** e0ne_ is now known as e0ne | 15:29 | |
| *** amotoki has joined #openstack-ironic | 15:29 | |
| *** jistr|mtg is now known as jistr | 15:30 | |
| *** puranamr has joined #openstack-ironic | 15:42 | |
| jroll | NobodyCam: not in the database or whatever no | 15:42 |
| openstackgerrit | Merged openstack/ironic-specs: Change release model to independent releases https://review.openstack.org/185171 | 15:42 |
| openstackgerrit | Merged openstack/ironic-specs: Change release model to independent releases https://review.openstack.org/185171 | 15:42 |
| *** puranamr has quit IRC | 15:42 | |
| NobodyCam | jroll: ^^^^^^^^^ | 15:43 |
| jroll | \o/ | 15:43 |
| NobodyCam | hummm two merges | 15:43 |
| jroll | so can we do a release yet? | 15:43 |
| jroll | yeah that's weird | 15:43 |
| NobodyCam | lol | 15:43 |
| *** puranamr has joined #openstack-ironic | 15:43 | |
| TheJulia | I've seen the bot do that a few times | 15:48 |
| NobodyCam | :-p | 15:51 |
| *** rwsu has joined #openstack-ironic | 15:52 | |
| *** radek__ has joined #openstack-ironic | 15:53 | |
| *** achanda has joined #openstack-ironic | 15:54 | |
| *** jistr has quit IRC | 15:55 | |
| *** radek_ has quit IRC | 15:55 | |
| *** ijw has joined #openstack-ironic | 15:59 | |
| lucasagomes | http://redsymbol.net/articles/unofficial-bash-strict-mode/ if I knew that before I would like bash a little more | 16:03 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Update the new state machine spec with CLEANWAIT https://review.openstack.org/200521 | 16:08 |
| jroll | lucasagomes: neat | 16:09 |
| lucasagomes | w00t good catch on waiting the ramdisk | 16:09 |
| *** mgarza has joined #openstack-ironic | 16:09 | |
| dtantsur | lucasagomes, I'm usually avoiding -o pipefile, and often add -x :) | 16:09 |
| lucasagomes | dtantsur, I see, I saw padraig comment about bash and SIGPIPE in the original post http://www.pixelbeat.org/programming/sigpipe_handling.html | 16:10 |
| *** radek__ has quit IRC | 16:11 | |
| lucasagomes | bash seems to not handle it correctly | 16:11 |
| *** e0ne has quit IRC | 16:11 | |
| lucasagomes | anyway, none of that makes me want to program in bash again. But if I knew that few years ago it would definetly be at least a bit more pleasant | 16:11 |
| jroll | lucasagomes: I meant the bash stuff was neat :P | 16:13 |
| jroll | but yeah | 16:13 |
| jroll | I also +A'd your periodic task thing | 16:13 |
| lucasagomes | oh thanks! | 16:13 |
| jroll | also... | 16:13 |
| jroll | you've an extra file in this specs patch | 16:13 |
| dtantsur | Ok, I'll be going, see you next week :) | 16:15 |
| *** dtantsur is now known as dtantsur|afk | 16:15 | |
| lucasagomes | jroll, oh damn git add . | 16:16 |
| jroll | hah | 16:16 |
| *** ijw has quit IRC | 16:16 | |
| jroll | have a good weekend dtantsur|afk :) | 16:16 |
| lucasagomes | I was writing that passthru for all interfaces after the summit | 16:16 |
| lucasagomes | dtantsur|afk, see ya enjoy the weekend | 16:16 |
| NobodyCam | have a good weekend dtantsur|afk :) | 16:16 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Update the new state machine spec with CLEANWAIT https://review.openstack.org/200521 | 16:17 |
| *** max_lobur has joined #openstack-ironic | 16:17 | |
| jroll | man gerrit is slow today | 16:19 |
| lucasagomes | yeah, odd because it's friday | 16:20 |
| *** Marga_ has quit IRC | 16:21 | |
| *** ijw has joined #openstack-ironic | 16:22 | |
| *** ijw has quit IRC | 16:22 | |
| jroll | lucasagomes: thinking about https://review.openstack.org/#/c/197141/6/ironic/conductor/manager.py more | 16:24 |
| jroll | if we hit that race it's going to explode | 16:24 |
| lucasagomes | release model has been merged w00t! | 16:24 |
| jroll | because conductor will come up, release lock | 16:24 |
| jroll | we try to release it and it isn't locked | 16:25 |
| lucasagomes | right, if it tries and it's not locked nothing actually happens | 16:25 |
| lucasagomes | it will set reservation = None again in the db | 16:25 |
| * lucasagomes tests | 16:25 | |
| jroll | no, it raises NodeNotLocked | 16:25 |
| lucasagomes | Oh | 16:26 |
| *** Marga_ has joined #openstack-ironic | 16:26 | |
| * lucasagomes investigates more | 16:26 | |
| jroll | https://github.com/openstack/ironic/blob/master/ironic/db/sqlalchemy/api.py#L248 | 16:26 |
| jroll | added a -1 there | 16:26 |
| jroll | it depends on that other patch so not a big deal | 16:27 |
| lucasagomes | cool yeah I will handle it | 16:27 |
| jroll | thanks! | 16:27 |
| lucasagomes | thanks for the review | 16:27 |
| jroll | yeah np | 16:27 |
| trown | lucasagomes: that bash doc is pretty awesome...would have helped to read that before hacking on DIB | 16:33 |
| lucasagomes | trown, ++ | 16:33 |
| lucasagomes | exactly | 16:33 |
| lucasagomes | they should enforce it in OOO | 16:34 |
| trown | most of that stuff is in the DIB scripts, I just had to figure it out | 16:34 |
| trown | like this bar=${foo:-alpha} | 16:34 |
| trown | we use that alot | 16:35 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic: Vagrant configuration generation now uses pymysql. https://review.openstack.org/199767 | 16:35 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic: Added CORS support middleware to Ironic https://review.openstack.org/199769 | 16:35 |
| lucasagomes | oh that will be a huge refactor then :-/ | 16:37 |
| *** max_lobur has quit IRC | 16:43 | |
| *** romcheg has joined #openstack-ironic | 16:45 | |
| devananda | morning, all | 16:49 |
| *** jxiaobin has joined #openstack-ironic | 16:50 | |
| lucasagomes | devananda, good ugt morning | 16:50 |
| *** derekh has quit IRC | 16:52 | |
| *** bitblt has joined #openstack-ironic | 17:01 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Fix using non-existent mock method https://review.openstack.org/200628 | 17:05 |
| jroll | heya devananda | 17:07 |
| jroll | release model spec landed, we should do a release soonish | 17:07 |
| jroll | maybe after this stuff around aborting cleaning and deploywait etc land | 17:07 |
| *** Sukhdev has quit IRC | 17:08 | |
| *** davideagnello has joined #openstack-ironic | 17:08 | |
| *** Sukhdev has joined #openstack-ironic | 17:09 | |
| lucasagomes | ++ | 17:17 |
| lucasagomes | devananda, also it would be good if we released a client version once https://review.openstack.org/#/c/198021/ lands | 17:18 |
| devananda | jroll: \o/ | 17:18 |
| lucasagomes | I will then update the ironic nova driver to clean up the logs which is now logging the whole node representation | 17:18 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Periodically checks the status of nodes in DEPLOYING state https://review.openstack.org/197141 | 17:19 |
| lucasagomes | jroll, ^ | 17:19 |
| jroll | woo | 17:19 |
| * NobodyCam gets more coffee | 17:20 | |
| *** trown is now known as trown|lunch | 17:23 | |
| *** kozhukalov_ has joined #openstack-ironic | 17:24 | |
| lucasagomes | and folks I will call it a day | 17:24 |
| lucasagomes | have a great evening! Enjoy the weekend | 17:24 |
| NobodyCam | have a great weekend lucasagomes | 17:25 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/python-ironicclient: Allow specifying a set of fields of the Port and Chasis resources https://review.openstack.org/198345 | 17:26 |
| lucasagomes | last update :-) | 17:26 |
| lucasagomes | NobodyCam, you too! Have a pint for me today | 17:26 |
| lucasagomes | see y'all! | 17:27 |
| jroll | see ya lucasagomes :) | 17:27 |
| NobodyCam | will do :) after 5 here | 17:27 |
| lucasagomes | jroll, see you, enjoy the weekend! | 17:27 |
| *** lucasagomes is now known as lucas-beer | 17:27 | |
| *** e0ne has joined #openstack-ironic | 17:28 | |
| openstackgerrit | Merged openstack/ironic: Add DRAC BIOS config vendor passthru API https://review.openstack.org/150488 | 17:28 |
| *** cdearborn has joined #openstack-ironic | 17:33 | |
| *** Nisha has joined #openstack-ironic | 17:42 | |
| *** e0ne has quit IRC | 17:52 | |
| *** e0ne has joined #openstack-ironic | 17:54 | |
| *** puranamr has quit IRC | 17:54 | |
| *** e0ne has quit IRC | 17:58 | |
| *** puranamr has joined #openstack-ironic | 17:59 | |
| *** [1]cdearborn has joined #openstack-ironic | 17:59 | |
| *** bizarrochristy has quit IRC | 18:01 | |
| *** bizarrochristy has joined #openstack-ironic | 18:01 | |
| *** romcheg has quit IRC | 18:05 | |
| *** romcheg has joined #openstack-ironic | 18:06 | |
| SpamapS | TheJulia: so, I bifrosted a second box and got this again: | 18:06 |
| SpamapS | grub-probe: error: unknown filesystem. | 18:06 |
| *** bizarrochristy has quit IRC | 18:06 | |
| NobodyCam | SpamapS: partition based image? | 18:09 |
| SpamapS | NobodyCam: err, not sure | 18:09 |
| SpamapS | built with the vm element | 18:09 |
| SpamapS | I always forget which is which | 18:10 |
| NobodyCam | vm is whole dosk | 18:10 |
| *** trown|lunch is now known as trown | 18:10 | |
| NobodyCam | disk even | 18:10 |
| NobodyCam | brb | 18:11 |
| *** davideagnello has quit IRC | 18:12 | |
| *** boris-42 has quit IRC | 18:12 | |
| *** coolsvap|away is now known as coolsvap | 18:13 | |
| *** cdearborn has quit IRC | 18:14 | |
| *** vishwana_ has quit IRC | 18:17 | |
| *** mgoddard has left #openstack-ironic | 18:18 | |
| *** vishwana_ has joined #openstack-ironic | 18:19 | |
| *** pelix has quit IRC | 18:19 | |
| *** davideagnello has joined #openstack-ironic | 18:23 | |
| *** ndipanov has quit IRC | 18:25 | |
| *** ndipanov has joined #openstack-ironic | 18:25 | |
| *** Nisha has quit IRC | 18:26 | |
| *** zz_natorious is now known as natorious | 18:29 | |
| *** bizarrochristy has joined #openstack-ironic | 18:29 | |
| *** Marga_ has quit IRC | 18:32 | |
| *** Marga_ has joined #openstack-ironic | 18:32 | |
| *** jcoufal has quit IRC | 18:40 | |
| *** harlowja_ has quit IRC | 18:46 | |
| *** ndipanov has quit IRC | 18:48 | |
| *** harlowja has joined #openstack-ironic | 18:50 | |
| *** logan2 has quit IRC | 18:51 | |
| *** athomas has quit IRC | 18:51 | |
| *** logan2 has joined #openstack-ironic | 18:52 | |
| *** athomas has joined #openstack-ironic | 18:56 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Fix using non-existent mock method https://review.openstack.org/200628 | 18:56 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/200670 | 18:59 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/200671 | 19:00 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/200674 | 19:00 |
| *** igordcard has joined #openstack-ironic | 19:06 | |
| *** Sukhdev has quit IRC | 19:09 | |
| *** Sukhdev has joined #openstack-ironic | 19:13 | |
| *** achanda has quit IRC | 19:16 | |
| *** ig0r__ has quit IRC | 19:19 | |
| SpamapS | hm | 19:19 |
| SpamapS | so it's not the configdrive | 19:19 |
| SpamapS | I deleted that and grub-probe still complains | 19:19 |
| *** ig0r_ has joined #openstack-ironic | 19:19 | |
| TheJulia | SpamapS: did you update the code? | 19:20 |
| SpamapS | TheJulia: update what code? | 19:22 |
| SpamapS | I think, its possible, the partition is too big | 19:23 |
| TheJulia | eek... :( | 19:24 |
| SpamapS | ergo /boot on most systems ;) | 19:24 |
| TheJulia | SpamapS: weve landed a few updates to the code, kind of wondering what path you hit, but you were already installed so likely just enroll/deploy | 19:24 |
| TheJulia | yeah :( | 19:25 |
| SpamapS | I have a 2TB root disk | 19:25 |
| TheJulia | hmm | 19:25 |
| JayF | using the agent? | 19:25 |
| JayF | I suspect there are bugs using teh agent on boxes that size, but imbw | 19:25 |
| JayF | SpamapS: first of all, using an agent_* driver? Secondly, do you know if your image is full disk? if so, MBR or GPT? If GPT and you don't already have a partition labelled config-2 in the image, agent will barf on configdrive | 19:26 |
| JayF | unless some patch went in I missed | 19:26 |
| JayF | tl;dr: Agent doesn't support creating configdrive partition on GPT, only MBR | 19:27 |
| SpamapS | yes agent driver | 19:27 |
| JayF | and given you have a 2T root disk; I assume you might be looking at GPT not MBR | 19:27 |
| SpamapS | So, it's all bifrost. :) | 19:27 |
| TheJulia | yeah, work is required there | 19:29 |
| SpamapS | pretty common thing.. :-P | 19:29 |
| TheJulia | yeah... guess i need to get a bigger disk ;) | 19:30 |
| SpamapS | JayF: so everything worked and booted fine | 19:31 |
| SpamapS | but grub is refusing to update itself | 19:31 |
| SpamapS | I have a configdrive | 19:32 |
| SpamapS | In fact, 2TB is the exact maximum of an MBR | 19:32 |
| SpamapS | [ 36.581852] sd 4:0:0:0: [sda] 3906963632 512-byte logical blocks: (2.00 TB/1.81 TiB) | 19:34 |
| SpamapS | thats interesting | 19:34 |
| SpamapS | I wonder if it is intentionally _exactly_ 2TB | 19:34 |
| SpamapS | solely to not require any newfangled MBR tricks | 19:34 |
| SpamapS | JayF: so I think the real problem is that grub just doesn't know how to figure the disk out.. note the output of grub-probe -v -d /dev/sda1 ... http://paste.openstack.org/show/362708/ | 19:36 |
| SpamapS | Likely need to make a /boot so grub doesn't freak out, or figure out how to boot with UEFI | 19:37 |
| SpamapS | also it is actually not "2TiB", but "1.8TiB" , so that seems like even more evidence it should work fine with all traditional methods. | 19:40 |
| *** mtanino has quit IRC | 19:42 | |
| SpamapS | so it appears grub wants its install partitions to be 1TiB or less | 19:49 |
| SpamapS | This is one reason /boot partitions are still used | 19:50 |
| SpamapS | any thoughts on how we might get around this? | 19:50 |
| * SpamapS will ponder over lunch | 19:50 | |
| lifeless | SpamapS: the 2TB was just the # of bits they had | 19:54 |
| lifeless | SpamapS: oh, you mean the manufacturer's drive size? could be | 19:54 |
| *** kkoski has quit IRC | 19:56 | |
| *** kkoski has joined #openstack-ironic | 19:57 | |
| *** rwsu has quit IRC | 19:58 | |
| *** kkoski has quit IRC | 20:02 | |
| *** harlowja has quit IRC | 20:09 | |
| *** Marga__ has joined #openstack-ironic | 20:09 | |
| *** Marga__ has quit IRC | 20:09 | |
| *** harlowja has joined #openstack-ironic | 20:09 | |
| *** Marga__ has joined #openstack-ironic | 20:10 | |
| *** Marga_ has quit IRC | 20:10 | |
| *** puranamr has quit IRC | 20:11 | |
| *** romcheg has quit IRC | 20:12 | |
| *** romcheg has joined #openstack-ironic | 20:12 | |
| *** bitblt has quit IRC | 20:14 | |
| *** Marga_ has joined #openstack-ironic | 20:19 | |
| *** Marga_ has quit IRC | 20:19 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/200670 | 20:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/200671 | 20:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/200674 | 20:20 |
| *** Marga_ has joined #openstack-ironic | 20:20 | |
| *** Marga__ has quit IRC | 20:22 | |
| *** Marga__ has joined #openstack-ironic | 20:24 | |
| *** Marga_ has quit IRC | 20:26 | |
| *** kkoski has joined #openstack-ironic | 20:27 | |
| *** kozhukalov_ has quit IRC | 20:37 | |
| *** coolsvap is now known as coolsvap|away | 20:39 | |
| SpamapS | lifeless: grub is unable to recognize the ext4 partition we have that is 2TB | 20:40 |
| SpamapS | lifeless: or we could just make a 500MB /boot and dodge the bullet. | 20:41 |
| SpamapS | I want to say UEFI is easy | 20:42 |
| SpamapS | but I still have yet to see it actually be | 20:42 |
| * SpamapS still hasn't gotten lunch tho..so.. more pondering | 20:42 | |
| * TheJulia ponders as well | 20:42 | |
| *** achanda has joined #openstack-ironic | 20:43 | |
| *** puranamr has joined #openstack-ironic | 20:44 | |
| *** mtanino has joined #openstack-ironic | 20:44 | |
| *** kkoski has quit IRC | 20:50 | |
| *** davideagnello has quit IRC | 20:56 | |
| -openstackstatus- NOTICE: Gerrit will be unavailable from 22:00 to 22:30 UTC for project renames | 21:02 | |
| *** cdearborn has joined #openstack-ironic | 21:07 | |
| *** dprince has quit IRC | 21:09 | |
| *** gabriel-bezerra has quit IRC | 21:10 | |
| *** ig0r_ has quit IRC | 21:10 | |
| SpamapS | uefi doesn't matter here actually | 21:12 |
| *** Sukhdev has quit IRC | 21:13 | |
| *** [1]cdearborn has quit IRC | 21:14 | |
| *** achanda has quit IRC | 21:14 | |
| SpamapS | even grub-efi can't seem to grok this massive / | 21:15 |
| *** Sukhdev has joined #openstack-ironic | 21:20 | |
| TheJulia | did the ubuntu grub issues get resolved in the base images? | 21:24 |
| *** achanda has joined #openstack-ironic | 21:26 | |
| SpamapS | TheJulia: don't know what they are, haven't experienced. | 21:28 |
| SpamapS | So one thing we could do, possibly, is tell growroot not to go beyond 1TB. | 21:28 |
| SpamapS | oh.. weird.. thats not installed? | 21:30 |
| *** Marga__ has quit IRC | 21:31 | |
| *** Marga_ has joined #openstack-ironic | 21:31 | |
| SpamapS | JayF: does the agent actually do the filesystem growing? | 21:32 |
| JayF | SpamapS: no; we expect cloud-init or another in-instance agent to do that | 21:32 |
| SpamapS | hm, weird that cloud-initramfs-growroot isn't installed... | 21:33 |
| SpamapS | wait that just does the parttable | 21:33 |
| SpamapS | the FS is done by cloud-init | 21:33 |
| SpamapS | mordred: ^ does glean do filesystem resizing? | 21:33 |
| mordred | no - cloud-initramfs-growroot does | 21:34 |
| SpamapS | it does partition resizing.. | 21:34 |
| mordred | k | 21:34 |
| * SpamapS reads it again | 21:34 | |
| mordred | SpamapS: you're probably right - it's entirely possible it's a thing we just havent' it on our simple-init images in the gate | 21:35 |
| SpamapS | pull-lp-source: Error: The source package 'cloud-initramfs-growroot' does not exist in the Ubuntu primary archive in wily, wily-security, wily-updates or wily-proposed | 21:35 |
| mordred | although - if you don't grow the fs - there woudl be no space on / right? | 21:35 |
| mordred | because there is not extra space on the image, yeah? | 21:35 |
| SpamapS | no there will be some | 21:35 |
| mordred | enough to run devstack? | 21:35 |
| SpamapS | it's not shrunk to fit | 21:35 |
| SpamapS | it is grown to a reasonably good size for building most things | 21:36 |
| SpamapS | or you can force a size | 21:36 |
| mordred | gotcha | 21:36 |
| mordred | so we _cluld_ be having the issue in the gate and just ahven't noticed because accident | 21:36 |
| mordred | or - the filesystem could be resizing and we haven't noticed because we havent' noticed | 21:36 |
| *** trown is now known as trown|outttypeww | 21:39 | |
| SpamapS | ok looks like what we do is du the build dir, and make the image used-space +40% | 21:39 |
| *** rwsu has joined #openstack-ironic | 21:43 | |
| *** karimb has joined #openstack-ironic | 21:47 | |
| SpamapS | JayF: ok, so resize fs agent does not do, but it does size the partitions so it can fit config drive in there right? | 21:49 |
| JayF | yes | 21:50 |
| *** achanda has quit IRC | 21:50 | |
| JayF | well | 21:50 |
| JayF | I don't know about the partial disk image support for agent | 21:50 |
| JayF | if you're using full disk images it uses this | 21:50 |
| JayF | https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/copy_configdrive_to_disk.sh | 21:51 |
| *** Marga_ has quit IRC | 21:51 | |
| *** gabriel-bezerra has joined #openstack-ironic | 21:51 | |
| SpamapS | JayF: yeah, so you'd have to do that _after_ the partition was sized to get the effect I see. | 21:51 |
| SpamapS | Otherwise we wouldn't have a 1.8TiB sda1 | 21:52 |
| *** achanda has joined #openstack-ironic | 21:54 | |
| -openstackstatus- NOTICE: Gerrit is unavailable from approximately 22:00 to 22:30 UTC for project renames | 21:58 | |
| *** ChanServ changes topic to "Gerrit is unavailable from approximately 22:00 to 22:30 UTC for project renames" | 21:58 | |
| *** karimb has quit IRC | 21:58 | |
| *** karimb has joined #openstack-ironic | 21:59 | |
| *** karimb has quit IRC | 22:05 | |
| jxiaobin | had anybody tried spawning multiple physical instances concurrently? | 22:08 |
| jroll | yes | 22:09 |
| *** harlowja_ has joined #openstack-ironic | 22:09 | |
| jxiaobin | hi jroll | 22:09 |
| jroll | jxiaobin: having problems or just curious? :) | 22:09 |
| jroll | and hello! | 22:09 |
| jxiaobin | problems | 22:09 |
| jroll | what are you seeing? | 22:09 |
| *** harlowja has quit IRC | 22:10 | |
| jxiaobin | so I got 20 ironic nodes, and tried booting them all one by one in a loop | 22:10 |
| *** karimb has joined #openstack-ironic | 22:10 | |
| jxiaobin | most of them failed | 22:10 |
| jxiaobin | btw, I have 3 nova-scheduler services running | 22:11 |
| jroll | what driver? how many conductors? how much cpu/ram on the conductors? how many nova-compute services? what errors are you seeing? | 22:11 |
| jxiaobin | they failed at scheduling phase | 22:11 |
| jroll | oh, odd | 22:12 |
| jroll | what's the error? | 22:12 |
| jxiaobin | for my deployment, I think there is race condition | 22:12 |
| jroll | what sort of race condition? | 22:12 |
| jxiaobin | for each instance, schedulers might select the same ironic hv to provision | 22:12 |
| jroll | yeah, I was thinking that | 22:13 |
| jroll | we only run one nova-scheduler | 22:13 |
| jxiaobin | when creating ports, the api call to neutron failed due to duplicate mac | 22:13 |
| jxiaobin | even retry failed after 3 times | 22:13 |
| jroll | yep, that'll do it. that's in the compute, how many computes are you running? | 22:13 |
| jroll | the compute should lock the node unless you're running more than one | 22:13 |
| jxiaobin | it might sound odd, we are running 1 compute, but 3 nova scheduler/conductor | 22:14 |
| *** karimb has quit IRC | 22:15 | |
| jroll | hmmm | 22:15 |
| *** boris-42 has joined #openstack-ironic | 22:15 | |
| jxiaobin | lock on the nova-compute might be too late | 22:16 |
| jroll | the scheduler will also claim it but I don't know how that interation works with multiple schedulers | 22:16 |
| jxiaobin | I think they interact through db, by reducing used ram/disk | 22:18 |
| jxiaobin | but the duration is too long, another scheduler pick the same hv before reducing ram/disk/cores | 22:19 |
| jroll | a periodic task in the nova-compute process does that bit | 22:19 |
| jroll | so yeah probably races there | 22:19 |
| jroll | I'd just roll with one scheduler | 22:19 |
| jxiaobin | so you do run one scheduler ? | 22:19 |
| jroll | jxiaobin: yes | 22:22 |
| jxiaobin | jroll: how about ha? | 22:23 |
| *** mgarza has quit IRC | 22:23 | |
| jroll | jxiaobin: active/passive as it just subscribes to rabbit | 22:24 |
| *** karimb has joined #openstack-ironic | 22:24 | |
| karimb | hello buddies, is it valid nost-show information ? | 22:25 |
| karimb | http://pastebin.com/H7QytGT5 | 22:25 |
| *** jhova has quit IRC | 22:27 | |
| karimb | anyone? :=) | 22:29 |
| *** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 22:30 | |
| jroll | karimb: seems reasonable | 22:30 |
| jroll | karimb: of course I don't know what driver_info fields are required for ovirt :) | 22:31 |
| *** davideagnello has joined #openstack-ironic | 22:31 | |
| karimb | jroll, i just put the vmname, all other fields are from ironic.conf [ovirt] section | 22:31 |
| jroll | karimb: cool, seems fine then :) | 22:32 |
| karimb | i am able to change power state with an ironic api script againt this node so i think i m fine :) | 22:32 |
| karimb | thanks :) | 22:32 |
| jroll | \o/ | 22:33 |
| jroll | np | 22:34 |
| devananda | jxiaobin: i believe we've never tested (upstream) having >1 nova-scheduler, but as i understand the nova arch, i'm not surprised that you're having that problem | 22:39 |
| devananda | jxiaobin: the ironic node isn't locked until the nova-compute passes control down to the driver | 22:40 |
| devananda | jxiaobin: so if multiple schedulers pick the same ironic node, and that gets serialized at some layer within nova-compute, yea, it's definitely going to collide | 22:41 |
| jxiaobin | devananda: thanks for the confirmation. any plan to address this? | 22:43 |
| *** alex_xu has quit IRC | 22:43 | |
| *** alex_xu has joined #openstack-ironic | 22:43 | |
| devananda | jroll: do you think the changes that you've spec'd out with Nova would address this ? | 22:44 |
| devananda | jroll: matching the failure domains and running >1 nova=compute | 22:44 |
| jroll | it... might, actually | 22:44 |
| jroll | if we're scheduling only by (host,) | 22:44 |
| jroll | the compute would know it's already claimed a thing | 22:44 |
| jroll | it's going to be slightly racey but retries should handle things | 22:44 |
| jroll | devananda: jxiaobin: ^ | 22:45 |
| devananda | awesome | 22:45 |
| jroll | I'm curious if it will make it this cycle; I certainly hope so | 22:45 |
| devananda | that's much better than it is today | 22:45 |
| jroll | yeah | 22:45 |
| jroll | nova folks from rackspace are also working on scheduler fixes to make host selection better and not overload a single host | 22:46 |
| jroll | which may help some too | 22:46 |
| jxiaobin | thanks jroll and devananda | 22:46 |
| jroll | np | 22:47 |
| *** purp has quit IRC | 22:53 | |
| *** thrash is now known as thrash|wknd | 22:53 | |
| *** purp has joined #openstack-ironic | 22:53 | |
| *** sbadia has quit IRC | 22:53 | |
| *** dhellmann has quit IRC | 22:54 | |
| *** JoshNang has quit IRC | 22:54 | |
| *** dhellmann has joined #openstack-ironic | 22:56 | |
| *** JoshNang has joined #openstack-ironic | 22:57 | |
| *** sbadia has joined #openstack-ironic | 22:57 | |
| *** natorious is now known as zz_natorious | 23:16 | |
| *** Sukhdev has quit IRC | 23:20 | |
| *** puranamr has quit IRC | 23:21 | |
| *** puranamr has joined #openstack-ironic | 23:22 | |
| *** lucas-beer has quit IRC | 23:23 | |
| *** openstackgerrit has quit IRC | 23:39 | |
| karimb | ok, so discovery fails with No filename or root path specified .... workaround ? | 23:39 |
| *** openstackgerrit has joined #openstack-ironic | 23:39 | |
| karimb | where does ironic get the tftp file associated to a given node ? | 23:53 |
| *** rwsu has quit IRC | 23:54 | |
| *** r-daneel has quit IRC | 23:55 | |
| *** karimb has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!