| *** robcresswell has quit IRC | 00:04 | |
| *** ijw has joined #openstack-ironic | 00:04 | |
| *** amotoki has joined #openstack-ironic | 00:08 | |
| *** ijw has quit IRC | 00:16 | |
| *** causten has joined #openstack-ironic | 00:17 | |
| *** ijw has joined #openstack-ironic | 00:18 | |
| *** baoli has joined #openstack-ironic | 00:19 | |
| *** vsaienko has quit IRC | 00:22 | |
| *** mtanino has quit IRC | 00:23 | |
| *** ijw has quit IRC | 00:24 | |
| *** baoli has quit IRC | 00:25 | |
| *** amotoki_ has joined #openstack-ironic | 00:30 | |
| *** rloo has quit IRC | 00:31 | |
| *** amotoki has quit IRC | 00:34 | |
| *** hoangcx has joined #openstack-ironic | 00:37 | |
| *** rama_y has quit IRC | 00:44 | |
| *** rama_y has joined #openstack-ironic | 00:45 | |
| *** phuongnh has joined #openstack-ironic | 00:48 | |
| *** causten has quit IRC | 00:58 | |
| *** baoli has joined #openstack-ironic | 01:10 | |
| *** aNuposic has quit IRC | 01:31 | |
| *** tuanluong has joined #openstack-ironic | 01:55 | |
| *** Sukhdev has quit IRC | 01:58 | |
| *** tuanluong has quit IRC | 01:59 | |
| *** sabeen1 has quit IRC | 02:05 | |
| *** sabeen has joined #openstack-ironic | 02:09 | |
| openstackgerrit | gengchc2 proposed openstack/ironic: Fix a typo in policy.py,etc https://review.openstack.org/378487 | 02:21 |
|---|---|---|
| *** syed_ has quit IRC | 02:24 | |
| *** baoli has quit IRC | 02:25 | |
| *** baoli has joined #openstack-ironic | 02:25 | |
| *** yuanying has quit IRC | 02:47 | |
| *** bdeschenes has quit IRC | 02:51 | |
| *** yuanying has joined #openstack-ironic | 02:55 | |
| *** gcb has joined #openstack-ironic | 02:59 | |
| *** baoli has quit IRC | 03:00 | |
| *** david-lyle has quit IRC | 03:04 | |
| *** amotoki_ has quit IRC | 03:15 | |
| *** priteau has joined #openstack-ironic | 03:16 | |
| *** priteau has quit IRC | 03:21 | |
| *** aswadr_ has joined #openstack-ironic | 03:24 | |
| *** vikrant has joined #openstack-ironic | 03:34 | |
| *** yuanying has quit IRC | 03:38 | |
| *** yuanying has joined #openstack-ironic | 03:49 | |
| *** appu103_ has joined #openstack-ironic | 03:50 | |
| openstackgerrit | Jamie Lennox proposed openstack/ironic: Use oslo.context features https://review.openstack.org/379919 | 03:52 |
| *** amotoki has joined #openstack-ironic | 03:53 | |
| openstackgerrit | Jamie Lennox proposed openstack/ironic: Use oslo.context features https://review.openstack.org/379919 | 03:56 |
| *** amotoki has quit IRC | 03:57 | |
| *** vmud213 has joined #openstack-ironic | 03:57 | |
| *** links has joined #openstack-ironic | 03:58 | |
| *** Sukhdev has joined #openstack-ironic | 04:02 | |
| *** appu103_ is now known as aparna | 04:04 | |
| *** amotoki has joined #openstack-ironic | 04:10 | |
| *** sdake_ has joined #openstack-ironic | 04:42 | |
| *** sdake has quit IRC | 04:46 | |
| *** vmud213 has quit IRC | 04:56 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 05:11 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Fail test if excepted error was not raised https://review.openstack.org/379360 | 05:12 |
| *** vmud213 has joined #openstack-ironic | 05:12 | |
| *** sdake_ has quit IRC | 05:19 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic: Invalid URL and Typo in enrollment.rst https://review.openstack.org/379289 | 05:29 |
| *** jaosorior has joined #openstack-ironic | 05:40 | |
| *** xuao has quit IRC | 05:50 | |
| *** bks has joined #openstack-ironic | 05:53 | |
| *** bjolo has joined #openstack-ironic | 05:53 | |
| *** sdake has joined #openstack-ironic | 05:59 | |
| *** amotoki has quit IRC | 06:01 | |
| *** rcernin has joined #openstack-ironic | 06:07 | |
| *** mjura has joined #openstack-ironic | 06:20 | |
| openstackgerrit | Zhou Ya proposed openstack/ironic-python-agent: Get numa_node info when collecting pci devices info https://review.openstack.org/369245 | 06:22 |
| *** mjura has quit IRC | 06:27 | |
| *** mjura has joined #openstack-ironic | 06:27 | |
| *** phuongnh has quit IRC | 06:28 | |
| *** phuongnh has joined #openstack-ironic | 06:29 | |
| *** sdake has quit IRC | 06:35 | |
| *** sdake has joined #openstack-ironic | 06:35 | |
| *** pcaruana has joined #openstack-ironic | 06:39 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: doc: Add oslo.i18n usage link https://review.openstack.org/379956 | 06:40 |
| *** jaosorior has quit IRC | 06:40 | |
| *** amotoki has joined #openstack-ironic | 06:51 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Fix grenade jobs https://review.openstack.org/377653 | 06:52 |
| *** phuongnh has quit IRC | 06:56 | |
| *** Sukhdev has quit IRC | 07:02 | |
| *** amotoki has quit IRC | 07:04 | |
| *** e0ne has joined #openstack-ironic | 07:13 | |
| *** pmannidi is now known as pmannidi|Gone | 07:28 | |
| *** pmannidi|Gone has quit IRC | 07:28 | |
| *** e0ne has quit IRC | 07:29 | |
| *** sdake_ has joined #openstack-ironic | 07:34 | |
| *** sdake has quit IRC | 07:37 | |
| *** coolias has joined #openstack-ironic | 07:37 | |
| openstackgerrit | Zhou Ya proposed openstack/ironic-python-agent: Get numa_node info when collecting pci devices info https://review.openstack.org/369245 | 07:43 |
| zigo | JayF: Sphinx docs are also packaged. | 07:45 |
| zigo | JayF: Well, when there's a bit more than just an index.rst, sometimes, it's really not worth... | 07:46 |
| *** amotoki has joined #openstack-ironic | 07:46 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Fail test if excepted error was not raised https://review.openstack.org/379360 | 07:48 |
| *** coolias has quit IRC | 07:49 | |
| *** jpich has joined #openstack-ironic | 07:53 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch error when wrong temp_url were generated https://review.openstack.org/370875 | 07:53 |
| *** sabeen has quit IRC | 07:56 | |
| *** Romanenko_K has joined #openstack-ironic | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Use assertRaises() instead of fail() https://review.openstack.org/379988 | 08:06 |
| *** bdeschenes has joined #openstack-ironic | 08:10 | |
| *** dtantsur has joined #openstack-ironic | 08:11 | |
| dtantsur | Morning Ironic | 08:14 |
| dtantsur | my IRC bouncer died again, so please repeat any pings :) | 08:14 |
| *** coolias has joined #openstack-ironic | 08:15 | |
| *** priteau has joined #openstack-ironic | 08:20 | |
| *** derekh has joined #openstack-ironic | 08:20 | |
| *** sdake_ has quit IRC | 08:23 | |
| *** vsaienko has joined #openstack-ironic | 08:26 | |
| *** amotoki_ has joined #openstack-ironic | 08:27 | |
| *** amotoki has quit IRC | 08:29 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Fix grenade jobs https://review.openstack.org/377653 | 08:40 |
| *** ss7pro has quit IRC | 08:40 | |
| *** e0ne has joined #openstack-ironic | 08:48 | |
| *** vsaienko has quit IRC | 08:52 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: Add a missing whitespace to an error message https://review.openstack.org/380024 | 08:52 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic: Add a missing whitespace to an error message https://review.openstack.org/380024 | 08:54 |
| *** phuongnh has joined #openstack-ironic | 08:54 | |
| *** vsaienko has joined #openstack-ironic | 08:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/380027 | 08:56 |
| openstackgerrit | Zhou Ya proposed openstack/ironic-python-agent: Get numa_node info when collecting pci devices info https://review.openstack.org/369245 | 08:56 |
| *** bdeschenes has quit IRC | 08:57 | |
| *** vsaienko has quit IRC | 09:01 | |
| tangchen_ | morning dtantsur | 09:03 |
| *** yibo has quit IRC | 09:04 | |
| tangchen_ | dtantsur: Would you please help to approve three simple patches ? https://review.openstack.org/#/c/379360/ https://review.openstack.org/#/c/379415/ https://review.openstack.org/#/c/379988/ | 09:04 |
| patchbot | patch 379360 - ironic - Fail test if excepted error was not raised | 09:04 |
| patchbot | patch 379415 - ironic - Do not use mutable object as func default param | 09:04 |
| patchbot | patch 379988 - ironic - Use assertRaises() instead of fail() | 09:04 |
| openstackgerrit | Zhou Ya proposed openstack/ironic-python-agent: Get numa_node info when collecting pci devices info https://review.openstack.org/369245 | 09:04 |
| dtantsur | tangchen_, hi, I was mostly out for 3 days, so it'll take me some time to get to patches, except as far as low priority ones are concerned | 09:05 |
| zigo | betherly: Will you soon release the new ironic-ui? | 09:06 |
| tangchen_ | dtantsur: Oh, I see. Please get to the patches when you have time. :) Thank you very much | 09:06 |
| betherly | zigo: its already released | 09:07 |
| zigo | Oh, it was done yesterday! :P | 09:07 |
| betherly | :) | 09:07 |
| zigo | betherly: FYI, I'm currently testing it... | 09:07 |
| zigo | betherly: I mean, fixing the package so that it really works. | 09:07 |
| zigo | :P | 09:07 |
| betherly | zigo: cool thanks :) | 09:07 |
| betherly | did the horizon patch deal with the issues you were having? | 09:07 |
| zigo | betherly: Yeah, it did quite well! :P | 09:11 |
| zigo | betherly: https://review.openstack.org/379375 | 09:11 |
| patchbot | patch 379375 - horizon - Using /tmp for SECRET_KEY in tests | 09:11 |
| *** tosky has joined #openstack-ironic | 09:11 | |
| betherly | thanks | 09:12 |
| zigo | betherly: Where should I put the content folder? | 09:12 |
| zigo | ie: ironic_ui/content | 09:12 |
| tosky | vsaienk0: sooo, about https://review.openstack.org/#/c/380006/ and https://review.openstack.org/#/c/379952/ | 09:13 |
| patchbot | patch 380006 - openstack-dev/devstack - Set fixed_network_name always when ironic is enabled. | 09:13 |
| patchbot | patch 379952 - openstack-dev/devstack - Revert "Fix the logic of if used with is_service_e... | 09:13 |
| zigo | betherly: Is there a howto on how to install files in production? | 09:14 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry to connect to RPC if failed https://review.openstack.org/376462 | 09:14 |
| zigo | As often with plugins, files are kind of shuffled around... :/ | 09:14 |
| tosky | vsaienk0: what I was trying to say is that fixed_network_name has a specific meaning, linked to nova-network | 09:14 |
| vsaienk0 | tosky: according to option description it is not | 09:15 |
| betherly | zigo: ye things are moved around according to how horizon accesses them | 09:15 |
| vsaienk0 | tosky: https://github.com/openstack/tempest/blob/1d8b2834e944866eb39d617d375579352e40fa74/tempest/config.py#L243-L249 | 09:15 |
| betherly | theres a pretty specific folder structure it requires | 09:15 |
| tosky | vsaienk0: seen that, but there is not only the description | 09:15 |
| tosky | vsaienk0: this is when the broken if was introduced, check the commit message: https://review.openstack.org/#/c/183601/ | 09:15 |
| patchbot | patch 183601 - openstack-dev/devstack - Don't set tempest fixed_network_name with neutron (MERGED) | 09:15 |
| zigo | betherly: I basically have to install each file or folder manually in /usr/share/openstack-dashboard/openstack_dashboard | 09:16 |
| zigo | :/ | 09:16 |
| betherly | zigo: there are docs for installing onto horizon | 09:16 |
| tosky | vsaienk0: the "new" behavior was the intended behavior already one year ago | 09:16 |
| zigo | betherly: Where? | 09:16 |
| betherly | http://docs.openstack.org/developer/ironic-ui/ | 09:16 |
| tosky | vsaienk0: but the condition was incorrect, so it was working so far just by accident | 09:16 |
| vsaienk0 | tosky: yes, the reason that condition was incorrect and It worked with Ironic | 09:17 |
| zigo | betherly: This is totally useless for me, it tells about how to run devstack for developers. | 09:17 |
| vsaienk0 | tosky: it takes a lot of time to add a new variable to tempest, and start using it, while our voting job is broken... | 09:18 |
| tosky | vsaienk0: so the solution is to use a ironic-specific variable, I would say, and we can check with tempest developers, because that specific variable has a history of being tied to nova-network | 09:18 |
| tosky | vsaienk0: imho it takes even time to add something to devstack | 09:18 |
| betherly | zigo: ok sorrry. ive never run for production so i couldnt write further documentation than what ive done | 09:18 |
| tosky | vsaienk0: I mean, the time spent to fix devstack and tempest is almost the same | 09:18 |
| tosky | vsaienk0: is that failing test in tempest or in the ironic repository? | 09:18 |
| sambetts | Morning all | 09:19 |
| vsaienk0 | tosky: if leads to ironic-grenade job failure | 09:19 |
| vsaienk0 | *it | 09:19 |
| zigo | betherly: Where should ironic_ui/static/dashboard/admin/ironic/ go? Here? | 09:19 |
| zigo | # ls /usr/share/openstack-dashboard/openstack_dashboard/static/dashboard/ | 09:19 |
| zigo | img scss | 09:19 |
| sambetts | vsaienk0: did you see the bug that jlvillal linked? | 09:19 |
| sambetts | vsaienk0: I can't believe how annoying that is ... | 09:20 |
| tosky | vsaienk0: I see the failures, trying to understand how to fix it in a way that would not break again if something happens again to fixed_network_name, as you would be using it with a different meaning | 09:20 |
| vsaienk0 | sambetts: you are talking about subnetpools and 10.0.0.0/8 ? | 09:20 |
| sambetts | vsaienk0: yeah | 09:20 |
| vsaienk0 | sambetts: yeah I saw it, but didn't have time to check it | 09:21 |
| vsaienk0 | sambetts: it is related to multinode right? | 09:21 |
| zigo | betherly: What about ironic_ui/locale/ ? | 09:21 |
| *** tangchen_ has quit IRC | 09:21 | |
| zigo | betherly: Also, how do you compile these .po ? Is there a facility availble in ironic-ui? And how would that be merged into the horizon folder? | 09:22 |
| sambetts | vsaienk0: not just multinode, anything use the 10.0.0.0/8 it seems because it adds routes, so in my third party CI it broke the jenkins to slave network connection because we use a 10.x.x.x network for that | 09:22 |
| vsaienk0 | sambetts: not quite understand how it may break single node, hence on it we have more specific routes ieg: 10.1.0.0/20 and 10.2.0.0/20 | 09:23 |
| vsaienk0 | tosky: what is the deprecation policy for tempest? | 09:23 |
| tosky | vsaienk0: maybe I have another solution, let me try... | 09:24 |
| vsaienk0 | tosky: lets add a deprecation warning when fixed_network_name is used, and introduce new variable for Ironic needs | 09:24 |
| vsaienk0 | because just drop fixed_network_name without any notification is not right approach imo | 09:25 |
| sambetts | vsaienk0: my third party CI has a different network setup to upstream CI because of the intergration with real hardware, both networking hardware and ironic nodes, and thats why its causing issues on our end :( | 09:25 |
| *** robcresswell has joined #openstack-ironic | 09:25 | |
| vsaienk0 | sambetts: ah... I see, so it may be easily fixed by adding more specific routes right? | 09:26 |
| betherly | zigo: in production i dont know. i could guess about as well as you because i dont know how production stuff works if its totally different to the docs on merging into the horizon folder as per the dev docs | 09:26 |
| tosky | vsaienk0: the problem is that the test (which is a compute test) tries to create a server due to multiple nics, so maybe it can be fixed to find the right nick instead of hardcoding one which could not exist | 09:26 |
| betherly | zigo: i could ask the horizon team and find out more? | 09:26 |
| sambetts | vsaienk0: yeah I hope the patch proposed will help | 09:26 |
| vsaienk0 | tosky: on ironic-grenade all compute tests that boot instance are failing | 09:27 |
| tosky | vsaienk0: yes, I see that, and it's looks like after the upgrade the project used to run the tests has multiple nics | 09:27 |
| vsaienk0 | tosky: for example here http://logs.openstack.org/53/377653/17/check/gate-grenade-dsvm-ironic/24f5672/console.html | 09:27 |
| tosky | vsaienk0: but there is a function to get the proper nick, and given that the failing test is in tempest, it can use the private interface | 09:27 |
| zigo | betherly: This would be this afternoon then, because they aren't awake yet! :P | 09:28 |
| betherly | zigo: sorry about that | 09:28 |
| zigo | betherly: You're Elizabeth Elwell, right? | 09:29 |
| vsaienk0 | tosky: I'm not quite deeply familiar with tempest tests, do you have any other options how to fix it if do not set fixed_network_name? | 09:29 |
| betherly | zigo: yes :) | 09:29 |
| zigo | :) | 09:29 |
| betherly | why do you ask? | 09:29 |
| tosky | vsaienk0: that's what I'm trying to do :) | 09:29 |
| zigo | betherly: Just checking against top commiter in ironic-ui ... | 09:29 |
| betherly | ahh right :) | 09:29 |
| zigo | betherly: What I can do, is attempt to just put files where I think they belong, and resolve the 404 ... | 09:30 |
| betherly | zigo: sorry not to know more re production etc. started on openstack last august and immediately started on ironic-ui. been a matter of learning as i go and i just havent learnt about production stuff yet. | 09:30 |
| betherly | zigo: ok sounds good, please do let me know what you find | 09:30 |
| vsaienk0 | tosky: thanks for you help on this | 09:31 |
| zigo | betherly: What concerns me the most is the locales thing, though if it's not there, it will continue to work. In English only though, but not a big deal, I guess. | 09:32 |
| *** ralonsoh has joined #openstack-ironic | 09:32 | |
| zigo | betherly: What I'm trying to write is this: https://review.openstack.org/#/c/380049/2/debian/rules | 09:33 |
| patchbot | patch 380049 - deb-ironic-ui (debian/newton) - Merge tag '2.1.0' into debian/newton | 09:33 |
| zigo | betherly: I still don't know where to put the ironic_ui/content folder... | 09:33 |
| zigo | Hopefully, it's correct for the static folder... | 09:34 |
| zigo | Time for me to leave for lunch. TTYL | 09:34 |
| betherly | zigo: right ok. will take a look | 09:34 |
| *** athomas has joined #openstack-ironic | 09:35 | |
| *** gcb has quit IRC | 09:41 | |
| *** vsaienko has joined #openstack-ironic | 09:42 | |
| *** vsaienko has quit IRC | 09:50 | |
| *** vsaienko has joined #openstack-ironic | 09:54 | |
| *** electrofelix has joined #openstack-ironic | 09:55 | |
| *** vsaienko has quit IRC | 10:01 | |
| dtantsur | morning vsaienk0, sambetts, betherly, tosky! enjoy your meals, zigo | 10:05 |
| tosky | hi dtantsur | 10:06 |
| tosky | dtantsur: apparently a fix that I pushed to tempest triggered some code path which leads to failure in ironic/granade jobs; I'm trying to investigate | 10:06 |
| dtantsur | oops.. thanks for looking into it! | 10:07 |
| dtantsur | ironic-inspector grenade job is also broken, probably for the same reason, as we share most of the code with ironic | 10:07 |
| tosky | one of the basic compute tests from tempest fails because it find multiple networks with the same name and it can't choose | 10:08 |
| tosky | previously the network used was (incorrectly) always the same, from a tempest.conf variable which should not have been set for neutron jobs | 10:08 |
| tosky | the interesting thing is that the failing compute test already calls a function which should find the proper project(tenant) network to use for the test | 10:09 |
| tosky | so I'm a bit puzzled | 10:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Fix grenade jobs https://review.openstack.org/377653 | 10:10 |
| vsaienk0 | morning dtantsur | 10:10 |
| vdrok | good morning ironic! | 10:11 |
| dtantsur | hey vdrok | 10:11 |
| vdrok | morning dtantsur :) | 10:11 |
| tosky | vsaienk0: about that review (https://review.openstack.org/377653), did you say that you need to run instances inside that special network? | 10:14 |
| patchbot | patch 377653 - ironic - Fix grenade jobs | 10:14 |
| vsaienk0 | tosky: yes | 10:14 |
| vsaienk0 | tosky: I don't know it there is other way than use fixed_network_name | 10:15 |
| tosky | vsaienk0: so basically you ensure that that specific private network is available in the project used for tests? | 10:16 |
| vsaienk0 | tosky: yes | 10:16 |
| vsaienk0 | tosky: we need this, because we need to access from ironic api/conductor to instances in this network | 10:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/377443 | 10:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/377445 | 10:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/377446 | 10:17 |
| tosky | vsaienk0: is this network shared amongst all projects, or do you specify a certain project? Bcause the user and the project used for the tests are created dynamically afaik, as there is use_dynamic_credentials | 10:17 |
| jroll | zigo: to pile on jay's "do you package docs" question, I think his next question is "we just backported a bunch of docs stuff to newton, should we release again?" :) | 10:18 |
| jroll | morning all | 10:18 |
| vsaienk0 | morning jroll | 10:18 |
| vsaienk0 | tosky: yes the network is shared among all tenants | 10:18 |
| zigo | jroll: I'd answer: "yes if you really want this doc to reach Debian Stretch". | 10:19 |
| jroll | zigo: cool, thanks | 10:20 |
| zigo | (ie: the next Debian Stable will be frozen at the end of 2016) | 10:20 |
| jroll | ah, so there's some time | 10:20 |
| jroll | dtantsur: could we land https://review.openstack.org/#/c/374863/ please? | 10:20 |
| zigo | Well, release early, release often ... :P | 10:20 |
| patchbot | patch 374863 - ironic (stable/newton) - Update devstack IPA_DOWNLOAD_BRANCH to stable/newton | 10:20 |
| jroll | zigo: I agree, more like should I release now or next week :) | 10:21 |
| zigo | That's early enough indeed. | 10:21 |
| dtantsur | jroll, +A (and good morning) | 10:22 |
| jroll | morning dtantsur, thank you :) | 10:22 |
| *** coolias has quit IRC | 10:23 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/376136 | 10:23 |
| *** coolias has joined #openstack-ironic | 10:24 | |
| tosky | vsaienk0: ok, so I think the only move is to ask mtreinish when he is up, as he knows more than me the logic of the usage of fixed_network_name | 10:24 |
| *** hoangcx has quit IRC | 10:25 | |
| vsaienk0 | tosky: ok, thanks I will contact with mtreinish | 10:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/virtualbmc: Updated from global requirements https://review.openstack.org/373687 | 10:25 |
| tosky | vsaienk0: I guess he will read here | 10:26 |
| vsaienk0 | jroll: we have a problem with grenade, yesterday patch to devstack was merged https://review.openstack.org/#/c/374311/. The patch update logic of setting fixed_network_name tempest config variable. This variable is not set when neutron is enabled (our case). But ironic explicitly requires it: https://github.com/openstack/tempest/blob/master/tempest/scenario/manager.py#L1161 I'm not sure what is the right way here probably mtreinish cat put some | 10:30 |
| patchbot | patch 374311 - openstack-dev/devstack - Fix the logic of if used with is_service_enabled (MERGED) | 10:30 |
| jroll | vsaienk0: yeah, been trying to follow. I'll jump in after more coffee | 10:31 |
| vsaienk0 | jroll: I will be out of office in next hour or two, might be vdrok may help on this also | 10:33 |
| *** aparna has quit IRC | 10:33 | |
| jroll | vsaienk0: okay, I can push on it too | 10:33 |
| vsaienk0 | jroll: thanks! | 10:34 |
| jroll | :) | 10:35 |
| *** phuongnh has quit IRC | 10:45 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/378886 | 10:48 |
| *** mgould|afk is now known as mgould | 10:48 | |
| mgould | morning Ironic | 10:48 |
| vdrok | morning jroll and mgould | 10:53 |
| mgould | morning vdrok jroll vsaienk0 tosky zigo dtantsur | 10:54 |
| tosky | hi mgould | 10:54 |
| *** coolias has quit IRC | 10:55 | |
| TheJulia | Good Morning everyone | 11:08 |
| jroll | \o | 11:08 |
| jroll | vsaienk0: so if you're still around, our options are: 1) revert that. 2) fix our code in tempest to use something else, without deprecation. 3) in our devstack plugin, set fixed_network_name. yeah? | 11:10 |
| jroll | I think I like (3) and it gives us a path to (2) with deprecation | 11:10 |
| jroll | or 4) hack tempest devstack plugin short term to also set fixed_network_name if ironic is enabled | 11:12 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Set fixed_network_name in our devstack plugin https://review.openstack.org/380170 | 11:15 |
| jroll | vsaienk0: ^ hopefully that unbreaks us for now | 11:16 |
| jroll | tosky: ^ curious what you think about that too, I think it's okay for a short-term hack | 11:17 |
| mgould | morning TheJulia | 11:23 |
| mat128 | o/ TheJulia and everyone else | 11:25 |
| dtantsur | morning TheJulia, mat128 | 11:25 |
| *** bjolo has quit IRC | 11:27 | |
| *** bjolo has joined #openstack-ironic | 11:27 | |
| vdrok | morning TheJulia and mat128 | 11:28 |
| vdrok | jroll: that might work, configure_tempest from devstack is called at the very end of the upgrade, and it does not update the tempest.conf value in our case https://github.com/openstack-dev/devstack/blob/e43cca1adf6709ac273958a7877d2a60658e1225/lib/tempest#L320 | 11:30 |
| *** lucas-afk is now known as lucasagomes | 11:31 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: LLC Hook: Do not assume interfaces are added to Ironic https://review.openstack.org/379614 | 11:31 |
| vdrok | jroll: but we need to call ironic_configure_tempest during upgrade for it to take effect | 11:33 |
| jroll | vdrok: ah, hmmm | 11:33 |
| vdrok | otherwise I think it will be set only on /old | 11:33 |
| mgould | morning mat128 | 11:34 |
| lucasagomes | morning all | 11:36 |
| tosky | jroll: 3 probably makes more sense, and iirc it's what you do in https://review.openstack.org/377653 | 11:37 |
| patchbot | patch 377653 - ironic - Fix grenade jobs | 11:37 |
| tosky | jroll: the code that fails in tempest is not ironic code, is the generic "create server" test | 11:37 |
| tosky | so 2) is not really feasible | 11:37 |
| jroll | tosky: oh, vsaienk0 added that, I missed it | 11:37 |
| jroll | tosky: yeah, but it's an "if baremetal" block in that test | 11:37 |
| jroll | so it's "our" code | 11:37 |
| tosky | jroll: it fails in the resource_setup of ServersTestJSON | 11:38 |
| jroll | tosky: yes, due to this block https://github.com/openstack/tempest/blob/master/tempest/scenario/manager.py#L1161 | 11:39 |
| tosky | jroll: oh, interesting note there | 11:39 |
| *** dprince has joined #openstack-ironic | 11:40 | |
| *** ccamacho has quit IRC | 11:40 | |
| jroll | and now I see https://review.openstack.org/#/c/380006/ | 11:40 |
| patchbot | patch 380006 - openstack-dev/devstack - Set fixed_network_name always when ironic is enabled. | 11:40 |
| tosky | I think that the long term strategy should be: find a way to remove that if baremetal | 11:40 |
| jroll | whee. | 11:40 |
| * jroll was not fully caught up, vsaienk0 rocks | 11:40 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow management of vlans when specifying mac https://review.openstack.org/380183 | 11:41 |
| jroll | tosky: yeah, we're right in the middle of executing on that | 11:41 |
| tosky | yeah, my objection with 380006 is about the scope of fixed_network_name, as it seems to linked mostly to nova-network test; if it can be used also for neutron testing in the specific case of a shared private network for a project, well | 11:41 |
| tosky | then it would be fine, but that's up to tempest developers | 11:42 |
| tosky | or better, core tempest developers | 11:42 |
| jroll | right, it wasn't intended to be for neutron it seems | 11:42 |
| jroll | yep | 11:42 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix grenade jobs https://review.openstack.org/377653 | 11:42 |
| bfournie | hi sambetts, I'm going to create bugs and a patch for these 2 llc introspection issues - http://paste.openstack.org/show/583572/, let me know if you think there's a better way to fix it. thx. | 11:46 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow management of vlans when specifying mac https://review.openstack.org/380183 | 11:47 |
| *** ccamacho has joined #openstack-ironic | 11:51 | |
| *** baoli has joined #openstack-ironic | 11:51 | |
| *** baoli_ has joined #openstack-ironic | 11:51 | |
| *** baoli has quit IRC | 11:55 | |
| openstackgerrit | Merged openstack/ironic: Update devstack IPA_DOWNLOAD_BRANCH to stable/newton https://review.openstack.org/374863 | 11:55 |
| *** trown|outtypewww is now known as trown | 12:12 | |
| sambetts | bfournie: all of these bugs serves me right for developing this hook while all the Ironic code was still in development too :-P | 12:13 |
| *** wajdi has joined #openstack-ironic | 12:13 | |
| sambetts | bfournie: I've rolled the mac format etc issues into the same patch | 12:14 |
| sambetts | bfournie: might as well fix it all at once | 12:14 |
| sambetts | bfournie: again thanks for testing this all out :D | 12:15 |
| *** baoli_ has quit IRC | 12:17 | |
| bfournie | sambetts: hey no problem! been a good learning experience about plugins for me. Yeah I agree should roll into same patch to keep it together. I'll still create bugs for these ok? | 12:17 |
| *** baoli has joined #openstack-ironic | 12:18 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add user and project domains to ironic context https://review.openstack.org/295371 | 12:18 |
| sambetts | bfournie: sure | 12:18 |
| *** wajdi has quit IRC | 12:19 | |
| bfournie | sambetts: btw, one more, hopefully last question about plugins. I've been enabling the llc plugin directly in inspector.conf, but how do we recommend that users enable it? via the conf file, or in instack-undercloud/puppet-stack-config/puppet-stack-config.yaml.template, or elsewhere? | 12:21 |
| sambetts | bfournie: Thats a question for triple-o/instack I'm afraid | 12:22 |
| bfournie | sambetts: ok sure, I'll take it up there. thx. | 12:22 |
| sambetts | bfournie: Ironic inspector just documents how to configure it via its own config file, how that config file is populated is based on how inspector is installed | 12:23 |
| *** athomas has quit IRC | 12:24 | |
| bfournie | sambetts: sounds good, thank you | 12:25 |
| *** ccamacho is now known as ccamacho|lunch | 12:25 | |
| sambetts | bfournie: once you have created the bugs can you link me to them so I can update the commit message with the ids | 12:25 |
| *** gcb has joined #openstack-ironic | 12:26 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add user and project domains to ironic context https://review.openstack.org/295371 | 12:26 |
| vdrok | lucasagomes: morning, could you please take a look at https://review.openstack.org/295371 ? It seems to work fine, just added unittests today. There is also another related patch here - https://review.openstack.org/379919, it's purpose is a bit different, but it does almost the same thing | 12:27 |
| patchbot | patch 295371 - ironic - Add user and project domains to ironic context | 12:27 |
| patchbot | patch 379919 - ironic - Use oslo.context features | 12:27 |
| bfournie | sambetts: ok, I have subscribed you to the two bugs I created | 12:30 |
| *** xavierr|afk is now known as xavierr | 12:30 | |
| sambetts | bfournie: awesome thanks a bunch | 12:32 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Correct several issues with the LLC hook https://review.openstack.org/379614 | 12:34 |
| sambetts | bfournie: update the patch with all the fixes :) | 12:35 |
| sambetts | I've updated* | 12:35 |
| xavierr | # NOTE(xavierr) good morning ironic | 12:35 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Update playbooks to support os_client_config https://review.openstack.org/380224 | 12:36 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone installation playbook https://review.openstack.org/380225 | 12:36 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost https://review.openstack.org/380226 | 12:36 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 12:36 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone install and clouds.yaml to testing https://review.openstack.org/380228 | 12:36 |
| bfournie | sambetts: excellent! | 12:36 |
| openstackgerrit | M V P Nitesh proposed openstack/python-ironicclient: Fixed wrap from taking negative values https://review.openstack.org/379436 | 12:44 |
| *** jheroux has joined #openstack-ironic | 12:51 | |
| vdrok | jroll: https://review.openstack.org/377653 grenade passed | 12:52 |
| patchbot | patch 377653 - ironic - Fix grenade jobs | 12:52 |
| *** links has quit IRC | 12:56 | |
| *** david-lyle has joined #openstack-ironic | 12:57 | |
| mat128 | xavierr: missing a : | 12:58 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add user and project domains to ironic context https://review.openstack.org/295371 | 12:58 |
| mat128 | :) | 12:58 |
| mat128 | xavierr: good morning | 12:58 |
| *** cdearborn has joined #openstack-ironic | 12:59 | |
| vsaienk0 | vdok thanks for driving this patch :) o/ | 13:00 |
| *** vikrant has quit IRC | 13:01 | |
| *** mjura has quit IRC | 13:01 | |
| lucasagomes | vdrok, sorry for the delay. Was hooked on something else | 13:03 |
| lucasagomes | vdrok, will take a look | 13:03 |
| *** sdake has joined #openstack-ironic | 13:04 | |
| jroll | vsaienk0: vdrok: thank you for keeping on that | 13:04 |
| jroll | +A'd | 13:04 |
| vdrok | morning xavierr | 13:07 |
| vdrok | lucasagomes: thank you | 13:07 |
| vdrok | jroll: vsaienk0 \o/ | 13:07 |
| *** mtanino has joined #openstack-ironic | 13:08 | |
| *** Goneri has joined #openstack-ironic | 13:08 | |
| jroll | releasing newton already :D https://review.openstack.org/380275 | 13:11 |
| patchbot | patch 380275 - releases - ironic 6.2.1 | 13:11 |
| jroll | er, from newton* | 13:11 |
| sambetts | :-P | 13:14 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow management of vlans when specifying mac https://review.openstack.org/380183 | 13:16 |
| *** syed_ has joined #openstack-ironic | 13:20 | |
| *** ccamacho|lunch is now known as ccamacho | 13:21 | |
| *** vsaienko has joined #openstack-ironic | 13:22 | |
| *** mag009 has quit IRC | 13:23 | |
| *** mag009 has joined #openstack-ironic | 13:23 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Move add_node_capability to tests folder https://review.openstack.org/380290 | 13:28 |
| *** alaski is now known as lascii | 13:29 | |
| *** vmud213 has quit IRC | 13:32 | |
| *** bcornec2 has quit IRC | 13:33 | |
| *** joprovost has joined #openstack-ironic | 13:36 | |
| lucasagomes | jroll, o/ btw, any news on the ironic-lib release ? | 13:36 |
| *** hshiina has joined #openstack-ironic | 13:38 | |
| *** joprovost1 has joined #openstack-ironic | 13:38 | |
| *** joprovost has quit IRC | 13:40 | |
| *** joprovost1 is now known as joprovost | 13:40 | |
| *** bks has quit IRC | 13:41 | |
| jroll | lucasagomes: let me check, just a minute | 13:41 |
| jroll | this is neat https://keepingitclassless.net/2016/03/test-driven-network-automation/ | 13:41 |
| jroll | sambetts: ^^ | 13:41 |
| xavierr | mat128: hahaha :P | 13:46 |
| xavierr | morning vdrok :) | 13:47 |
| jroll | lucasagomes: oh, now that I think about it, dims asked me to wait for next week | 13:47 |
| lucasagomes | jroll, oh, well next week is almost here already | 13:48 |
| lucasagomes | so it's all good :D | 13:48 |
| jroll | yeah :) | 13:48 |
| *** dansmith is now known as superdan | 13:49 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Cleanup generate_uid() function https://review.openstack.org/380311 | 13:50 |
| *** vsaienko has quit IRC | 13:52 | |
| *** bfournie has quit IRC | 13:54 | |
| sambetts | jroll: ooo interesting thanks :D | 13:57 |
| * sambetts was reading up on petiboot yesterday, its pretty coo | 13:58 | |
| *** e0ne has quit IRC | 13:59 | |
| *** e0ne has joined #openstack-ironic | 14:00 | |
| *** vsaienko has joined #openstack-ironic | 14:01 | |
| *** bfournie has joined #openstack-ironic | 14:03 | |
| *** gcb has quit IRC | 14:04 | |
| *** amotoki_ has quit IRC | 14:04 | |
| *** gcb has joined #openstack-ironic | 14:05 | |
| *** joprovost1 has joined #openstack-ironic | 14:06 | |
| openstackgerrit | Merged openstack/ironic: Fix grenade jobs https://review.openstack.org/377653 | 14:06 |
| *** joprovost has quit IRC | 14:08 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 14:08 |
| vsaienk0 | fyi: grenade patch was just merged, gates should be unblocked | 14:10 |
| *** joprovost1 has quit IRC | 14:11 | |
| jroll | \o/ | 14:11 |
| jroll | thanks vsaienk0 | 14:11 |
| *** joprovost has joined #openstack-ironic | 14:12 | |
| *** vsaienko has quit IRC | 14:12 | |
| *** hshiina has quit IRC | 14:18 | |
| *** vishwanathj has joined #openstack-ironic | 14:19 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: doc: Add oslo.i18n usage link https://review.openstack.org/379956 | 14:20 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Trivial: Fix some typos in comments and docstring https://review.openstack.org/379081 | 14:21 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Use method delete_if_exists from oslo.utils https://review.openstack.org/379159 | 14:21 |
| *** bjolo has quit IRC | 14:21 | |
| *** amotoki has joined #openstack-ironic | 14:24 | |
| jroll | hey, we added a cacert thing to IPA recently, yeah? does anyone want to take on passing devstack's (self-signed) cert through there so we can turn on tls in devstack? | 14:26 |
| jroll | oh maybe we didn't land that yet | 14:26 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Cleanup get_ilo_license() https://review.openstack.org/380341 | 14:28 |
| jroll | holy IPA failures | 14:29 |
| jroll | https://review.openstack.org/#/c/377446/ | 14:29 |
| patchbot | patch 377446 - ironic-python-agent - Updated from global requirements | 14:29 |
| *** lucasagomes is now known as lucas-afk | 14:31 | |
| lucas-afk | have a great weekend all (I'm off this afternoon) | 14:32 |
| *** bks has joined #openstack-ironic | 14:32 | |
| xavierr | have a nice weekend lucas-afk :) | 14:32 |
| lucas-afk | you too ! See ya next week | 14:33 |
| sambetts | jroll: Get https://index.docker.io/v1/repositories/library/debian/images: dial tcp 52.73.159.23:443: connection timed out | 14:36 |
| patchbot | https://review.openstack.org/#/c/52/ - keystone - Changes to make cache time configurable. (ABANDONED) | 14:36 |
| sambetts | jroll: seems like there is an issues downloading the docker base image | 14:37 |
| jroll | sambetts: yeah talking in infra | 14:37 |
| *** david-lyle has quit IRC | 14:37 | |
| *** david-lyle has joined #openstack-ironic | 14:38 | |
| *** kumarb has joined #openstack-ironic | 14:38 | |
| *** bks has quit IRC | 14:38 | |
| *** amotoki has quit IRC | 14:39 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: DNM: testing 380023 https://review.openstack.org/380351 | 14:40 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Remove unused methods https://review.openstack.org/379364 | 14:41 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Enable DeprecationWarning in test environments https://review.openstack.org/379581 | 14:44 |
| *** amotoki has joined #openstack-ironic | 14:46 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Use assertRaises() instead of fail() https://review.openstack.org/379988 | 14:47 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Fix import of ironicclient and reformat docstring https://review.openstack.org/378526 | 14:50 |
| *** mkam has joined #openstack-ironic | 14:52 | |
| *** rbrud has joined #openstack-ironic | 14:52 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Correct several issues with the LLC hook https://review.openstack.org/379614 | 14:57 |
| *** mkam_ has joined #openstack-ironic | 14:57 | |
| *** mkam has quit IRC | 14:58 | |
| *** ijw has joined #openstack-ironic | 15:00 | |
| *** ppiela has joined #openstack-ironic | 15:01 | |
| *** ijw has quit IRC | 15:01 | |
| *** ijw has joined #openstack-ironic | 15:01 | |
| JayF | zigo: including install guide stuff? Mainly wondering if we need a "point release" of our newton line when all we added was docs | 15:03 |
| jroll | JayF: https://review.openstack.org/#/c/380275/ | 15:04 |
| jroll | :) | 15:04 |
| patchbot | patch 380275 - releases - ironic 6.2.1 | 15:04 |
| jroll | can't hurt amirit | 15:04 |
| dtantsur | jroll, I'd avoid using patch versions, they'll clash with stable/newton versioning | 15:04 |
| dtantsur | e.g. in this case we'll need to make 6.2.2 from stable/newton which does not contain 6.2.1, which is.. weird | 15:05 |
| jroll | dtantsur: that's a newton release | 15:05 |
| dtantsur | ah, ok, overlooked then. | 15:05 |
| jroll | :) | 15:05 |
| jroll | /b 336 | 15:05 |
| jroll | oops | 15:05 |
| dtantsur | :) | 15:05 |
| jroll | no, no I don't have too many irc channels | 15:05 |
| * jroll runs away | 15:05 | |
| dtantsur | jroll, do we want to get https://review.openstack.org/375055 in? | 15:06 |
| patchbot | patch 375055 - ironic (stable/newton) - Change the logic of selecting image for tests | 15:06 |
| dtantsur | jroll, also please release ironic-lib newton after https://review.openstack.org/375320 lands | 15:07 |
| patchbot | patch 375320 - ironic-lib (stable/newton) - Fix check for GPT partioned device | 15:07 |
| *** amotoki has quit IRC | 15:07 | |
| jroll | dtantsur: sure, and noted | 15:07 |
| jroll | +A on both | 15:09 |
| dtantsur | thnx | 15:09 |
| *** gcb has quit IRC | 15:14 | |
| sambetts | dtantsur: btw I'd like to backport that LLC hook bug fixes to newton if possible | 15:15 |
| *** rama_y has quit IRC | 15:15 | |
| dtantsur | sambetts, definitely | 15:15 |
| sambetts | dtantsur: Are you happy about us fixing all three bugs in the same patch? | 15:15 |
| sambetts | dtantsur: it seemed odd to not fix it all together | 15:16 |
| dtantsur | it depends on how big all three are, I guess, and how related | 15:16 |
| * dtantsur opens the patch | 15:16 | |
| *** sabeen has joined #openstack-ironic | 15:16 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 15:16 |
| dtantsur | sambetts, I'd fix the MAC and the remaining bits separately | 15:17 |
| dtantsur | both patches would be backportable IMO | 15:17 |
| sambetts | dtantsur: so 2 patches or 3 patches? | 15:18 |
| dtantsur | 2 patches: MAC format and everything around updating ports | 15:19 |
| *** bnemec is now known as beekneemech | 15:20 | |
| *** sabeen has quit IRC | 15:21 | |
| *** amotoki has joined #openstack-ironic | 15:24 | |
| *** ccamacho has quit IRC | 15:29 | |
| openstackgerrit | Merged openstack/ironic-lib: Fix check for GPT partioned device https://review.openstack.org/375320 | 15:33 |
| *** zhenguo has quit IRC | 15:43 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: LLC Hook: Do not assume interfaces are added to Ironic https://review.openstack.org/379614 | 15:43 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: LLC Hook: Fix patching Ironic ports https://review.openstack.org/380394 | 15:43 |
| sambetts | bfournie, dtantsur: split ^ | 15:43 |
| dtantsur | awesome | 15:43 |
| dtantsur | inspector grenade has recovered as well \o/ | 15:44 |
| bfournie | sambetts: looks great! | 15:46 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: LLC Hook: Fix patching Ironic ports https://review.openstack.org/380400 | 15:47 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: LLC Hook: Do not assume interfaces are added to Ironic https://review.openstack.org/380401 | 15:47 |
| jlvillal | Good morning Ironic :) | 15:47 |
| sambetts | dtantsur: backports ^ | 15:47 |
| jlvillal | vsaienk0: Thanks so much for figuring out Grenade :) | 15:47 |
| * sambetts -> home | 15:48 | |
| dtantsur | sambetts, usually it's better to submit backports after the master patch is merged, or in case of any changes you'll have to update the "cherry-picked" link | 15:48 |
| jlvillal | sambetts: Enjoy your weekend :) | 15:48 |
| dtantsur | see you on Monday, sambetts and everyone | 15:48 |
| dtantsur | (also morning jlvillal) | 15:48 |
| jlvillal | dtantsur: Good night! Enjoy your weekend too :) | 15:48 |
| sambetts | dtantsur: oh good point :-P | 15:48 |
| sambetts | cy'all monday | 15:48 |
| sambetts | o/ | 15:48 |
| jlvillal | dtantsur: sambetts: I would say do a depends-on but since the backport has the same Change-Id usually .... | 15:49 |
| sambetts | jlvillal: yup | 15:49 |
| *** sambetts is now known as sambetts|afk | 15:49 | |
| dtantsur | jlvillal, yep, that's not quite an option.. | 15:49 |
| jlvillal | Yeah :( | 15:49 |
| *** dtantsur is now known as dtantsur|afk | 15:50 | |
| *** aNuposic has joined #openstack-ironic | 15:50 | |
| *** e0ne has quit IRC | 15:55 | |
| *** zhenguo has joined #openstack-ironic | 15:59 | |
| *** ccamacho has joined #openstack-ironic | 16:03 | |
| *** krtaylor has joined #openstack-ironic | 16:06 | |
| *** rcernin has quit IRC | 16:08 | |
| *** mkam_ has quit IRC | 16:08 | |
| *** Romanenko_K has quit IRC | 16:08 | |
| *** Romanenko_K has joined #openstack-ironic | 16:08 | |
| openstackgerrit | Merged openstack/ironic: Change the logic of selecting image for tests https://review.openstack.org/375055 | 16:09 |
| *** hpe-hj has joined #openstack-ironic | 16:10 | |
| openstackgerrit | Merged openstack/ironic: Use assertRaises() instead of fail() https://review.openstack.org/379988 | 16:10 |
| *** awidders_ has joined #openstack-ironic | 16:11 | |
| *** hj-hpe has quit IRC | 16:12 | |
| *** awiddersheim has quit IRC | 16:12 | |
| *** awidders_ has quit IRC | 16:13 | |
| *** awiddersheim has joined #openstack-ironic | 16:13 | |
| *** awiddersheim has quit IRC | 16:16 | |
| zigo | JayF: I started packaging the install-guide but it is full of non-free stuff. I will attempt again to do it (after the FTP master reject). | 16:16 |
| *** awiddersheim has joined #openstack-ironic | 16:16 | |
| *** awiddersheim has quit IRC | 16:19 | |
| *** awiddersheim has joined #openstack-ironic | 16:19 | |
| *** tosky has quit IRC | 16:21 | |
| *** awiddersheim has quit IRC | 16:22 | |
| *** awiddersheim has joined #openstack-ironic | 16:22 | |
| *** awiddersheim has quit IRC | 16:25 | |
| *** priteau has quit IRC | 16:26 | |
| *** priteau has joined #openstack-ironic | 16:26 | |
| *** rbrud has quit IRC | 16:28 | |
| *** awiddersheim has joined #openstack-ironic | 16:29 | |
| *** kumarb has quit IRC | 16:29 | |
| *** mkam has joined #openstack-ironic | 16:30 | |
| *** ralonsoh has quit IRC | 16:30 | |
| *** bks has joined #openstack-ironic | 16:31 | |
| *** jpich has quit IRC | 16:40 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic: Add description for vendor passthru methods https://review.openstack.org/370469 | 16:41 |
| *** electrofelix has quit IRC | 16:47 | |
| *** ipukha has quit IRC | 16:47 | |
| *** hshiina has joined #openstack-ironic | 16:52 | |
| *** ijw has quit IRC | 16:56 | |
| *** derekh has quit IRC | 16:59 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic: Add description for vendor passthru methods https://review.openstack.org/370469 | 16:59 |
| *** ijw has joined #openstack-ironic | 17:02 | |
| *** hshiina has quit IRC | 17:04 | |
| *** e0ne has joined #openstack-ironic | 17:05 | |
| *** ijw has quit IRC | 17:16 | |
| *** krtaylor has quit IRC | 17:17 | |
| *** jheroux has quit IRC | 17:21 | |
| *** MattMan has quit IRC | 17:22 | |
| *** MattMan has joined #openstack-ironic | 17:22 | |
| *** absubram has joined #openstack-ironic | 17:26 | |
| openstackgerrit | Merged openstack/ironic: Remove unused method is_valid_ipv6_cidr https://review.openstack.org/376214 | 17:28 |
| *** joprovost has quit IRC | 17:34 | |
| *** rama_y has joined #openstack-ironic | 17:34 | |
| *** joprovost has joined #openstack-ironic | 17:36 | |
| *** trown is now known as trown|lunch | 17:43 | |
| *** krtaylor has joined #openstack-ironic | 17:57 | |
| NobodyCam | morning Ironic'ers | 17:59 |
| *** joprovost has quit IRC | 18:07 | |
| xavierr | hey NobodyCam, morning :) | 18:08 |
| xavierr | TGIF | 18:08 |
| *** ccamacho has quit IRC | 18:08 | |
| *** vishwanathj has quit IRC | 18:11 | |
| NobodyCam | morning xavierr :) and for sure TGIF :) | 18:16 |
| *** zhenguo has quit IRC | 18:23 | |
| *** e0ne has quit IRC | 18:29 | |
| *** cdearborn has quit IRC | 18:31 | |
| mat128 | zigo: do you have links to the non-free parts? It seems this is all original content, no? | 18:33 |
| *** cdearborn has joined #openstack-ironic | 18:39 | |
| cdearborn | Hey folks, I would like to store some additional identifying information on ironic nodes (service tag and model). Was wondering what attribute it would be appropriate to put this info in? | 18:40 |
| mat128 | cdearborn: I would put those in properties | 18:41 |
| mat128 | you can add any key:value in there | 18:41 |
| cdearborn | mat128: thx much! | 18:42 |
| *** trown|lunch is now known as trown | 18:47 | |
| *** aswadr_ has quit IRC | 18:52 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 18:53 |
| *** Sukhdev has joined #openstack-ironic | 18:53 | |
| *** ijw has joined #openstack-ironic | 18:54 | |
| *** bks has quit IRC | 18:54 | |
| *** bks has joined #openstack-ironic | 18:55 | |
| *** ijw has quit IRC | 18:56 | |
| zigo | mat128: I don't remember the details, probably there was some non-free minified javascript without source ... | 18:57 |
| *** ijw_ has joined #openstack-ironic | 18:57 | |
| *** ijw_ has quit IRC | 18:57 | |
| *** ijw has joined #openstack-ironic | 18:58 | |
| zigo | This happens very often in the OpenStack world. Unfortunately a lot of people don't realize that these BLOBS are like a .o without a .c file... | 18:58 |
| mat128 | zigo: I can assure you there isnt any in the new install-guide | 18:59 |
| mat128 | zigo: maybe in the developer docs | 19:00 |
| zigo | mat128: As I wrote, I should re-attempt to package it. | 19:00 |
| mat128 | zigo: let me know, the install guide (under "install-guide/") is all new and mostly my commits | 19:00 |
| mat128 | and I would never commit a minified .js :) | 19:00 |
| mat128 | can't find any .js file in doc/source either | 19:01 |
| mat128 | maybe they are brought in by the openstackdocstheme? | 19:01 |
| *** hamzy has quit IRC | 19:02 | |
| zigo | mat128: openstack-manual contains: www/static/common/js/bootstrap.js | 19:02 |
| zigo | It shouldn't... | 19:02 |
| zigo | That's a library. | 19:02 |
| mat128 | zigo: oh, that's in openstack-manual | 19:02 |
| zigo | mat128: Or are you talking about something else? | 19:02 |
| mat128 | zigo: I thought you said we committed that to ironic directly | 19:02 |
| zigo | mat128: Are you talking about this? https://packages.debian.org/ironic-doc | 19:03 |
| mat128 | zigo: JayF was asking if you packaged the install docs, and mentioned that we had just backported our new install guide to the newton branch. | 19:04 |
| mat128 | zigo: I thought you mentioned we had committed a minified js | 19:04 |
| zigo | mat128: I'm packaging whatever is in Ironic's doc/source sphinx doc ... :P | 19:04 |
| mat128 | given I made almost all commits to that new install guide, I was surprised | 19:04 |
| zigo | mat128: Is that part of it? | 19:04 |
| mat128 | doc/source is the developer guide, our install guide used to be in there | 19:04 |
| mat128 | but was moved to install-guide/source | 19:05 |
| *** wajdi has joined #openstack-ironic | 19:05 | |
| zigo | mat128: Oh, ok, got you then! :) | 19:05 |
| zigo | mat128: https://review.openstack.org/380504 <--- This will add your guide, thanks for the hint, I very much appreciate it. | 19:08 |
| patchbot | patch 380504 - deb-ironic (debian/newton) - Also package Ironic's install-guide. | 19:08 |
| zigo | Packaging patches are welcome also, btw ... :P | 19:08 |
| mat128 | zigo: will keep that in mind whenever we change a dependency and/or change our layout | 19:12 |
| zigo | :) | 19:12 |
| *** openstackgerrit has quit IRC | 19:18 | |
| *** openstackgerrit has joined #openstack-ironic | 19:19 | |
| *** Sukhdev has quit IRC | 19:22 | |
| *** joprovost has joined #openstack-ironic | 19:24 | |
| *** Sukhdev has joined #openstack-ironic | 19:35 | |
| *** hamzy has joined #openstack-ironic | 19:39 | |
| *** bks has quit IRC | 19:41 | |
| JayF | https://bugs.launchpad.net/ironic/+bug/1626977 anyone know what "CNR" means in the context of that last comment? | 19:42 |
| openstack | Launchpad bug 1626977 in Ironic "Ironic should raise an error when updating unsupported field" [High,Triaged] - Assigned to Vadim Hmyrov (vhmyrov) | 19:42 |
| mat128 | JayF: Could not reproduce | 19:42 |
| JayF | perfect, ty | 19:43 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost: Updated from global requirements https://review.openstack.org/377425 | 19:57 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/377443 | 19:59 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/377446 | 19:59 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/376136 | 20:05 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/virtualbmc: Updated from global requirements https://review.openstack.org/373687 | 20:07 |
| *** dprince has quit IRC | 20:13 | |
| *** e0ne has joined #openstack-ironic | 20:20 | |
| openstackgerrit | Merged openstack/bifrost: Updated from global requirements https://review.openstack.org/377425 | 20:29 |
| *** ijw has quit IRC | 20:30 | |
| *** rama_y has quit IRC | 20:34 | |
| *** baoli has quit IRC | 20:37 | |
| *** baoli has joined #openstack-ironic | 20:38 | |
| *** ijw has joined #openstack-ironic | 20:40 | |
| *** absubram has quit IRC | 20:40 | |
| *** Goneri has quit IRC | 20:44 | |
| JayF | https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L183 we can remove support for NOSTATE here now, right? | 20:48 |
| *** wajdi_ has joined #openstack-ironic | 20:50 | |
| xavierr | have a nice weekend all :) | 20:52 |
| *** xavierr is now known as xavierr|afk | 20:53 | |
| jroll | JayF: believe so yeah | 20:53 |
| *** wajdi has quit IRC | 20:54 | |
| *** wajdi has joined #openstack-ironic | 20:55 | |
| *** joprovost has quit IRC | 20:56 | |
| *** wajdi_ has quit IRC | 20:58 | |
| *** wajdi has quit IRC | 20:59 | |
| *** trown is now known as trown|outtypewww | 21:12 | |
| *** sdake has quit IRC | 21:19 | |
| aNuposic | mariojv: Hi mario so i was able to deploy the node using Ironic where nova sets flag "is_whole_disk_image" to false | 21:21 |
| *** lascii is now known as alaski | 21:22 | |
| *** e0ne has quit IRC | 21:23 | |
| *** e0ne has joined #openstack-ironic | 21:24 | |
| *** e0ne has quit IRC | 21:24 | |
| *** e0ne has joined #openstack-ironic | 21:25 | |
| *** e0ne has quit IRC | 21:25 | |
| *** sdake has joined #openstack-ironic | 21:25 | |
| *** e0ne has joined #openstack-ironic | 21:25 | |
| *** e0ne has quit IRC | 21:26 | |
| *** e0ne has joined #openstack-ironic | 21:26 | |
| *** e0ne has quit IRC | 21:27 | |
| *** e0ne has joined #openstack-ironic | 21:27 | |
| *** e0ne has quit IRC | 21:27 | |
| *** cdearborn has quit IRC | 21:28 | |
| *** e0ne has joined #openstack-ironic | 21:28 | |
| *** e0ne has quit IRC | 21:28 | |
| *** Goneri has joined #openstack-ironic | 21:33 | |
| *** gcb has joined #openstack-ironic | 21:37 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Use method delete_if_exists from oslo.utils https://review.openstack.org/379159 | 21:41 |
| *** sdake has quit IRC | 21:45 | |
| *** sdake has joined #openstack-ironic | 21:46 | |
| *** mkam has quit IRC | 21:47 | |
| *** ijw has quit IRC | 21:49 | |
| *** jtomasek has quit IRC | 21:50 | |
| *** priteau has quit IRC | 21:51 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Remove unused methods in common/utils.py https://review.openstack.org/379364 | 21:55 |
| *** sdake has quit IRC | 21:59 | |
| *** devananda is now known as devananda|away | 22:08 | |
| *** ijw has joined #openstack-ironic | 22:09 | |
| *** ijw has quit IRC | 22:14 | |
| openstackgerrit | Merged openstack/ironic: Fail test if excepted error was not raised https://review.openstack.org/379360 | 22:20 |
| openstackgerrit | Merged openstack/python-ironicclient: Add upper-constraints for stable/newton to tox.ini https://review.openstack.org/365147 | 22:23 |
| *** ijw has joined #openstack-ironic | 22:31 | |
| *** gcb has quit IRC | 22:35 | |
| *** ijw has quit IRC | 22:35 | |
| *** Sukhdev has quit IRC | 22:39 | |
| *** rama_y has joined #openstack-ironic | 22:41 | |
| *** aNuposic has quit IRC | 22:50 | |
| *** ijw has joined #openstack-ironic | 22:52 | |
| *** ijw has quit IRC | 22:56 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 23:08 |
| *** ijw has joined #openstack-ironic | 23:12 | |
| *** ijw has quit IRC | 23:19 | |
| *** Goneri has quit IRC | 23:20 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/377443 | 23:21 |
| *** sdake has joined #openstack-ironic | 23:29 | |
| *** ppiela has quit IRC | 23:30 | |
| *** ijw has joined #openstack-ironic | 23:35 | |
| openstackgerrit | haris proposed openstack/python-ironicclient: This fix enables ironic list commands to use fields and labels in the arguments Ironic list commands (such as node list, port list etc) did not take resource label as input. After the fix, in addition to providing resource.fields and resource.labels in th https://review.openstack.org/380671 | 23:41 |
| *** rama_y has quit IRC | 23:50 | |
| *** absubram has joined #openstack-ironic | 23:54 | |
| *** baoli has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!