| *** karthiks has joined #openstack-ironic | 00:01 | |
| *** karthiks has quit IRC | 00:09 | |
| *** cjloader has joined #openstack-ironic | 00:19 | |
| *** karthiks has joined #openstack-ironic | 00:22 | |
| *** cjloader has quit IRC | 00:24 | |
| *** liuzz has joined #openstack-ironic | 00:48 | |
| *** hshiina has joined #openstack-ironic | 01:03 | |
| *** phuongnh has joined #openstack-ironic | 01:06 | |
| *** hieulq has joined #openstack-ironic | 01:12 | |
| *** tiendc has joined #openstack-ironic | 01:14 | |
| *** yangyapeng has joined #openstack-ironic | 01:15 | |
| *** cjloader has joined #openstack-ironic | 01:21 | |
| *** openstackgerrit has joined #openstack-ironic | 01:23 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: devstack: add support for rescue mode https://review.openstack.org/524118 | 01:23 |
|---|---|---|
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: CI job for rescue mode https://review.openstack.org/528704 | 01:23 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: DNM: test rescue mode https://review.openstack.org/533187 | 01:24 |
| *** tuanla____ has joined #openstack-ironic | 01:24 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: DNM: standalone test rescue mode https://review.openstack.org/534751 | 01:24 |
| *** cjloader has quit IRC | 01:25 | |
| *** yolanda has quit IRC | 01:44 | |
| *** rbudden has joined #openstack-ironic | 02:01 | |
| *** fragatina has quit IRC | 02:05 | |
| *** gcb has joined #openstack-ironic | 02:09 | |
| *** crushil_ has joined #openstack-ironic | 02:11 | |
| *** weshay has quit IRC | 02:14 | |
| *** weshay has joined #openstack-ironic | 02:15 | |
| *** rbudden has quit IRC | 02:24 | |
| *** crushil_ has quit IRC | 02:53 | |
| *** jaganathan has joined #openstack-ironic | 03:16 | |
| *** karthiks has quit IRC | 03:18 | |
| *** cjloader has joined #openstack-ironic | 03:21 | |
| *** cjloader has quit IRC | 03:25 | |
| *** karthiks has joined #openstack-ironic | 03:31 | |
| *** rbudden has joined #openstack-ironic | 03:34 | |
| *** hshiina has quit IRC | 03:43 | |
| *** hshiina has joined #openstack-ironic | 03:47 | |
| *** yangyapeng has quit IRC | 03:55 | |
| *** vcgi has joined #openstack-ironic | 03:56 | |
| *** links has joined #openstack-ironic | 04:00 | |
| *** crushil_ has joined #openstack-ironic | 04:10 | |
| *** sapd has quit IRC | 04:14 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:15 | |
| *** Nisha_Agarwal has quit IRC | 04:16 | |
| *** sapd has joined #openstack-ironic | 04:28 | |
| *** yangyapeng has joined #openstack-ironic | 04:34 | |
| *** crushil_ has quit IRC | 04:36 | |
| *** jaganathan has quit IRC | 04:49 | |
| *** rbudden has quit IRC | 04:54 | |
| *** karthiks has quit IRC | 04:58 | |
| *** jaganathan has joined #openstack-ironic | 05:03 | |
| *** dpawar has joined #openstack-ironic | 05:12 | |
| *** yangyapeng has quit IRC | 05:16 | |
| *** jaganathan has quit IRC | 05:17 | |
| *** cjloader has joined #openstack-ironic | 05:21 | |
| *** jaganathan has joined #openstack-ironic | 05:23 | |
| *** cjloader has quit IRC | 05:25 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 05:32 |
| *** jaganathan has quit IRC | 05:35 | |
| *** threestrands has quit IRC | 05:40 | |
| *** yangyapeng has joined #openstack-ironic | 05:41 | |
| *** yangyapeng has quit IRC | 05:45 | |
| *** sapd has quit IRC | 05:51 | |
| *** oanson has quit IRC | 05:55 | |
| *** sapd has joined #openstack-ironic | 05:56 | |
| *** hshiina has quit IRC | 05:56 | |
| *** yangyapeng has joined #openstack-ironic | 05:59 | |
| *** dpawar has quit IRC | 06:00 | |
| *** jaganathan has joined #openstack-ironic | 06:03 | |
| *** dpawar has joined #openstack-ironic | 06:06 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add documentation for rescue interface https://review.openstack.org/419606 | 06:07 |
| *** jaganathan has quit IRC | 06:24 | |
| *** vcgi has quit IRC | 06:24 | |
| *** sapd has quit IRC | 06:24 | |
| *** sapd has joined #openstack-ironic | 06:24 | |
| *** jaganathan has joined #openstack-ironic | 06:25 | |
| *** jaganathan has quit IRC | 06:33 | |
| *** rbartal has joined #openstack-ironic | 06:33 | |
| *** jaganathan has joined #openstack-ironic | 06:39 | |
| *** makowals has joined #openstack-ironic | 06:40 | |
| *** hieulq has quit IRC | 06:45 | |
| *** marios has joined #openstack-ironic | 06:49 | |
| *** mjura has joined #openstack-ironic | 06:52 | |
| pas-ha | morning ironic :) | 06:58 |
| *** hieulq has joined #openstack-ironic | 07:00 | |
| *** oanson has joined #openstack-ironic | 07:02 | |
| *** qsyqian has joined #openstack-ironic | 07:15 | |
| *** rcernin has quit IRC | 07:17 | |
| *** cjloader has joined #openstack-ironic | 07:21 | |
| *** cjloader has quit IRC | 07:25 | |
| *** fragatina has joined #openstack-ironic | 07:36 | |
| *** dpawar has quit IRC | 07:37 | |
| *** sapd_ has joined #openstack-ironic | 07:48 | |
| *** sapd has quit IRC | 07:48 | |
| *** jtomasek has joined #openstack-ironic | 07:49 | |
| *** pcaruana has joined #openstack-ironic | 07:52 | |
| *** hjensas has joined #openstack-ironic | 07:53 | |
| *** hjensas has quit IRC | 07:53 | |
| *** hjensas has joined #openstack-ironic | 07:53 | |
| *** dpawar has joined #openstack-ironic | 07:54 | |
| openstackgerrit | Vasyl Saienko proposed openstack/networking-generic-switch master: DNR/DNM test change https://review.openstack.org/536242 | 07:58 |
| *** jtomasek has quit IRC | 08:04 | |
| *** jtomasek has joined #openstack-ironic | 08:04 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 08:08 |
| *** qsyqian has quit IRC | 08:14 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 08:15 | |
| *** tesseract has joined #openstack-ironic | 08:19 | |
| *** cjloader has joined #openstack-ironic | 08:21 | |
| *** fragatina has quit IRC | 08:22 | |
| *** cjloader has quit IRC | 08:25 | |
| *** racedo has joined #openstack-ironic | 09:06 | |
| *** milan_ has joined #openstack-ironic | 09:12 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Update proliantutils version for Queens release https://review.openstack.org/535895 | 09:16 |
| openstackgerrit | paresh sao proposed openstack/ironic master: Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/468288 | 09:16 |
| *** mgoddard_ has joined #openstack-ironic | 09:16 | |
| *** derekh has joined #openstack-ironic | 09:25 | |
| *** etingof has quit IRC | 09:30 | |
| *** athomas has joined #openstack-ironic | 09:31 | |
| *** gcb has quit IRC | 09:32 | |
| *** stendulker has joined #openstack-ironic | 09:51 | |
| *** e0ne has joined #openstack-ironic | 09:55 | |
| *** dtantsur|afk is now known as dtantsur | 10:05 | |
| dtantsur | morning ironic | 10:05 |
| *** jaganathan has quit IRC | 10:05 | |
| stendulker | Good morning dtantsur :) | 10:11 |
| dtantsur | hey stendulker | 10:11 |
| *** etingof has joined #openstack-ironic | 10:11 | |
| dtantsur | stendulker: if you have time today: https://review.openstack.org/535772 this is a part of the classic driver deprecation priority | 10:11 |
| patchbot | patch 535772 - ironic - Allow data migrations to accept options | 10:11 |
| stendulker | dtantsur: Sure | 10:12 |
| *** mgoddard_ has quit IRC | 10:13 | |
| *** jaganathan has joined #openstack-ironic | 10:14 | |
| *** yolanda has joined #openstack-ironic | 10:17 | |
| *** yangyapeng has quit IRC | 10:18 | |
| *** e0ne has quit IRC | 10:21 | |
| *** mgoddard_ has joined #openstack-ironic | 10:25 | |
| *** e0ne has joined #openstack-ironic | 10:25 | |
| *** amarao has joined #openstack-ironic | 10:27 | |
| *** liuzz has quit IRC | 10:30 | |
| *** tuanla____ has quit IRC | 10:30 | |
| *** jaganathan has quit IRC | 10:31 | |
| *** c3-oscar has joined #openstack-ironic | 10:32 | |
| *** jaganathan has joined #openstack-ironic | 10:34 | |
| *** d0ugal has quit IRC | 10:37 | |
| *** yangyapeng has joined #openstack-ironic | 10:38 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs master: Future of classic drivers: update the migration proposal https://review.openstack.org/536298 | 10:40 |
| *** yangyapeng has quit IRC | 10:44 | |
| *** yangyapeng has joined #openstack-ironic | 10:49 | |
| mgoddard_ | morning dtantsur | 10:49 |
| dtantsur | hi mgoddard_ | 10:50 |
| mgoddard_ | just a very slow reply, seems there's a lot of lag on the line :) | 10:51 |
| dtantsur | heh, I see | 10:52 |
| gmonteiro | Good Morning everyone o/ | 10:52 |
| *** yangyapeng has quit IRC | 10:53 | |
| mgoddard_ | dtantsur: qq about API backwards compatibility. Should we expect the same behaviour from an API endpoint introduced in queens, whether using pike or an insufficient microversion in queens? | 10:55 |
| *** stendulker has quit IRC | 10:55 | |
| dtantsur | mgoddard_: ideally, yes | 10:55 |
| mgoddard_ | dtansur: and if we've already released code that does not do this, should we fix it or leave it? | 10:55 |
| *** d0ugal has joined #openstack-ironic | 10:57 | |
| dtantsur | mgoddard_: unless it results in a nasty bug, we should probably just leave it alone | 11:03 |
| mgoddard_ | dtantsur: ok | 11:04 |
| *** yangyapeng has joined #openstack-ironic | 11:17 | |
| *** yolanda has quit IRC | 11:17 | |
| *** cjloader has joined #openstack-ironic | 11:21 | |
| *** links has quit IRC | 11:23 | |
| *** cjloader has quit IRC | 11:26 | |
| *** AlexeyAbashkin has quit IRC | 11:32 | |
| *** openstackgerrit has quit IRC | 11:33 | |
| *** vmud213 has joined #openstack-ironic | 11:34 | |
| *** links has joined #openstack-ironic | 11:36 | |
| *** yangyapeng has quit IRC | 11:37 | |
| *** yangyapeng has joined #openstack-ironic | 11:37 | |
| vmud213 | Morning Ironic | 11:40 |
| gmonteiro | o/ vmud213 | 11:40 |
| *** AlexeyAbashkin has joined #openstack-ironic | 11:41 | |
| vmud213 | gmonteiro o/ | 11:42 |
| vmud213 | dtantsur: https://review.openstack.org/#/c/535895/ is due for workflow. | 11:42 |
| patchbot | patch 535895 - ironic - Update proliantutils version for Queens release | 11:43 |
| vmud213 | dtantsur: It's just the requirements update in the driver-requirements. | 11:43 |
| dtantsur | vmud213: so, question inline. if it's required for SUM, why wasn't the version bump done in the SUM patch? | 11:45 |
| *** jkilpatr has joined #openstack-ironic | 11:47 | |
| johnthetubaguy | dtantsur: I am curious about the policy.sample file, do we still need that now post policy in code? | 11:48 |
| vmud213 | dtantsur: just a mnt | 11:48 |
| dtantsur | johnthetubaguy: I guess we still need to ship a sample for a reference, no? | 11:48 |
| johnthetubaguy | dtantsur: I didn't think you had to, as you can just generate it, like the config | 11:48 |
| dtantsur | johnthetubaguy: well, we ship the config for the same reason | 11:49 |
| johnthetubaguy | dtantsur: ah, OK, well that is consistent then | 11:49 |
| dtantsur | not everyone who just want to check a few options want a full development environment | 11:49 |
| dtantsur | well | 11:49 |
| dtantsur | now that we generate the documentation for both config and policy we can revisit this decision | 11:49 |
| johnthetubaguy | dtantsur: yeah, it is at least available in raw form where needed now | 11:50 |
| dtantsur | I mean, https://docs.openstack.org/ironic/latest/configuration/sample-config.html and https://docs.openstack.org/ironic/latest/configuration/sample-policy.html | 11:50 |
| dtantsur | johnthetubaguy: wanna start a ML thread on deleting both examples now that we have ^^^? | 11:50 |
| johnthetubaguy | dtantsur: sure, good plan, was just reviewing the traits stuff and it peaked my interest | 11:50 |
| dtantsur | johnthetubaguy: thanks! and ditto for ironic-inspector: https://docs.openstack.org/ironic-inspector/latest/configuration/sample-config.html and https://docs.openstack.org/ironic-inspector/latest/configuration/sample-policy.html | 11:51 |
| johnthetubaguy | dtantsur: ah, right, cool | 11:51 |
| vmud213 | dtantsur:That's because from the ironic i tried to include whatever made sense. | 11:52 |
| dtantsur | and of course https://docs.openstack.org/ironic-inspector/latest/configuration/ironic-inspector.html is even nicer | 11:52 |
| dtantsur | vmud213: you shouldn't raise the version unless you really need to | 11:52 |
| vmud213 | from pike to queens, w.r.t ironic SUM is the feature that the user sees. | 11:52 |
| dtantsur | raising a version is a MUST for downstreams | 11:53 |
| dtantsur | while just releasing 2.5.0 is a SHOULD for downstreams | 11:53 |
| dtantsur | vmud213: SUM was released in 2.4.0 according to your changelog | 11:53 |
| vmud213 | Even though the changelog on proliantutils says, iSCSI related changes, the corresponding patch on ironic that enables BFV is not merged yet | 11:53 |
| dtantsur | right, and that's why I think we should not bump the version | 11:53 |
| *** e0ne has quit IRC | 11:54 | |
| dtantsur | there is nothing that we must have there right here and now | 11:54 |
| *** tesseract-RH has joined #openstack-ironic | 11:55 | |
| *** jkilpatr has quit IRC | 11:55 | |
| *** tesseract has quit IRC | 11:57 | |
| vmud213 | dtantsur: The Pike release was tagged with proliantutils version 2.4.0 | 12:05 |
| dtantsur | yep | 12:06 |
| vmud213 | There was only a limites support for hpsum based Firmware upgrade. | 12:06 |
| vmud213 | The complete features were part of 2.4.1 which is after Pike release | 12:07 |
| vmud213 | And SUM support is added after 2.4.1 | 12:07 |
| dtantsur | to be honest, I don't see anything about SUM in https://github.com/openstack/proliantutils/compare/2.4.1...2.5.0 and your changelog talks about 2.4.1 | 12:07 |
| dtantsur | (sorry, I said 2.4.0 above, I meant 2.4.1) | 12:08 |
| vmud213 | Also do you think we should also include Information pertaining to iSCSI settings in the patch set? | 12:08 |
| *** openstackgerrit has joined #openstack-ironic | 12:08 | |
| openstackgerrit | John Garbutt proposed openstack/ironic master: Remove sample policy and config files https://review.openstack.org/536349 | 12:08 |
| dtantsur | vmud213: only if ironic can use it already | 12:08 |
| *** jkilpatr has joined #openstack-ironic | 12:09 | |
| vmud213 | Yes dtantsur. I agree. | 12:10 |
| vmud213 | But From last release (pike) to now this is what is new. | 12:10 |
| vmud213 | Even though they are added in the intermedate releases, tagging to Queens release this is an added feature | 12:11 |
| vmud213 | WDYT? | 12:11 |
| vmud213 | So you think, we can not include iSCSI settings related changes because the BFV patch that support these changes hasn't gone in. Did i get it correctly? | 12:13 |
| dtantsur | vmud213: correct. you should update the version to 2.5.0 in the BFV patch, because it will actually use the changes from 2.5.0 | 12:15 |
| dtantsur | since ironic only uses features from 2.4.1 (if I understood it right), it should stay on 2.4.1 for now | 12:15 |
| vmud213 | But how about some bug fixes that are part of 2.5.0 release? | 12:16 |
| vmud213 | i mean retrying of power operation etc | 12:17 |
| dtantsur | vmud213: requirements are not bumped for bug fixes per openstack policies | 12:18 |
| vmud213 | I got your point but, by downgrading the version in the driver-requirement file will also lose some bug fixes. | 12:18 |
| vmud213 | ok.hmm. | 12:18 |
| *** krtaylor has joined #openstack-ironic | 12:19 | |
| vmud213 | dtantsur: One more thing, Sushy backward compatibility is also part of this fix | 12:22 |
| vmud213 | Which we have not mentioned in the ChangeLog | 12:22 |
| dtantsur | vmud213: what exactly backward compatibility do you mean and how does it affect ironic? | 12:22 |
| *** bfournie has quit IRC | 12:27 | |
| *** bfournie has joined #openstack-ironic | 12:27 | |
| *** bfournie has quit IRC | 12:32 | |
| vmud213 | Ok. So there's some logic related to "refresh" that needed to be changed in the proliantutils to comply with sushy library | 12:37 |
| vmud213 | here https://review.openstack.org/#/c/531295/ | 12:37 |
| patchbot | patch 531295 - proliantutils - Changes to support Sushy 1.3.0 (MERGED) | 12:37 |
| vmud213 | dtantsur: So i am trying to understand. Do you say that the proliantutils release version sshould have been 2.4.2 instead of 2.5.0? | 12:38 |
| *** phuongnh has quit IRC | 12:39 | |
| *** phuongnh has joined #openstack-ironic | 12:40 | |
| *** rbartal has quit IRC | 12:40 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Migrate oneview hardware type to use python-hpOneView https://review.openstack.org/524310 | 12:48 |
| *** phuongnh has quit IRC | 12:49 | |
| openstackgerrit | Merged openstack/ironic master: Add a version argument to traits DB API https://review.openstack.org/535482 | 12:49 |
| *** rbartal has joined #openstack-ironic | 12:52 | |
| *** jaganathan has quit IRC | 12:57 | |
| vmud213 | dtantsur: I agree we should keep the version that includes the feature. how to deal with the Sushy change for the refined refresh? | 12:58 |
| vmud213 | It certainly is needed and including the 2.4.1 breaks it. | 12:59 |
| *** jlabarre has joined #openstack-ironic | 13:00 | |
| TheJulia | good morning everyone | 13:07 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: [docs] mention new nova scheduler option https://review.openstack.org/534401 | 13:07 |
| *** bfournie has joined #openstack-ironic | 13:07 | |
| TheJulia | Greetings from Southern California | 13:07 |
| *** tiendc has quit IRC | 13:08 | |
| *** jaganathan has joined #openstack-ironic | 13:10 | |
| *** crushil_ has joined #openstack-ironic | 13:15 | |
| dtantsur | mornign TheJulia, is it sunny there? | 13:15 |
| nicodemos | morning TheJulia and ironic. | 13:15 |
| dtantsur | vmud213: yeah, the sushy issue is unclear. but recent versions of proliantutils work with recent versions of sushy, no? | 13:17 |
| dtantsur | also, I'm not sure how sushy release breaks you, given that the behavior did not change between 1.2.0 and 1.3.0 | 13:17 |
| *** e0ne has joined #openstack-ironic | 13:18 | |
| *** vmud213 has quit IRC | 13:18 | |
| *** rbartal has quit IRC | 13:24 | |
| *** MattMan has quit IRC | 13:25 | |
| *** jcoufal has joined #openstack-ironic | 13:25 | |
| *** MattMan has joined #openstack-ironic | 13:25 | |
| *** trown|outtypewww is now known as trown | 13:26 | |
| *** trown is now known as trown|ruck | 13:26 | |
| TheJulia | dtantsur: the sun has not even come up yet | 13:29 |
| TheJulia | dtantsur: maybe vmud213 is unaware of the last minute changes | 13:30 |
| * TheJulia starts drinking coffee | 13:31 | |
| dtantsur | yeah, maybe | 13:33 |
| dtantsur | also we have a regression in one of my patches, sigh.. :( | 13:33 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Rework upgrade to upgrade nova/ironic https://review.openstack.org/535596 | 13:36 |
| *** dpawar has quit IRC | 13:37 | |
| TheJulia | ugh | 13:37 |
| *** dprince has joined #openstack-ironic | 13:38 | |
| *** gcb has joined #openstack-ironic | 13:39 | |
| openstackgerrit | Rushil Chugh proposed openstack/ironic master: Add XClarity Driver https://review.openstack.org/519427 | 13:39 |
| TheJulia | I think our gate failure simply boils down to during the upgrade, some of the linked c libraries that were in the main process were deleted, so I bet depending on exactly how nova-conductor spawns, that could cause major problems. | 13:50 |
| *** rloo has joined #openstack-ironic | 13:50 | |
| TheJulia | good morning rloo | 13:51 |
| rloo | Good Monday Morning TheJulia and everyone else! | 13:52 |
| jroll | morning dtantsur, rloo, TheJulia, everyone else :) | 13:53 |
| rloo | morning jroll | 13:53 |
| jroll | TheJulia: that's... fun | 13:53 |
| TheJulia | jroll: yeah :( | 13:53 |
| TheJulia | now to just get two patches merged so we can make the tests voting again! | 13:53 |
| jroll | whee | 13:54 |
| *** cdearborn has joined #openstack-ironic | 13:54 | |
| jroll | anything I can help with atm? | 13:54 |
| rloo | ++ TheJulia, do you need help with whatever two patches mentioned above? | 13:55 |
| *** vmud213 has joined #openstack-ironic | 13:55 | |
| dtantsur | morning jroll, rloo | 13:55 |
| rloo | good morning dtantsur | 13:55 |
| dtantsur | jroll: hunt nova cores? | 13:55 |
| * jroll puts on his camo | 13:55 | |
| * rloo looks at whiteboard for status | 13:55 | |
| dtantsur | rloo: the drivers migration change is ready to review, when you have some (quite some) time | 13:56 |
| rloo | dtantsur: yes, on my list this week. need to catch up on what's been going on, then figure out priorities for the week... | 13:56 |
| TheJulia | https://review.openstack.org/#/c/535786/4 Yeah, pretty much hunt nova cores down | 13:56 |
| patchbot | patch 535786 - nova - Ironic: negotiate microversion to allow downgrade ... | 13:56 |
| rloo | TheJulia: ahhh. | 13:56 |
| jroll | trying to think of a good/simple way to explain why we need this method of negotiating | 13:57 |
| *** sasha2 has joined #openstack-ironic | 13:57 | |
| dtantsur | folks, if you want to discuss a potential FFE before filing it, add to the agenda https://wiki.openstack.org/wiki/Meetings/Ironic | 13:57 |
| TheJulia | rloo: https://review.openstack.org/535596 is our pach | 13:57 |
| patchbot | patch 535596 - ironic - Rework upgrade to upgrade nova/ironic | 13:57 |
| jroll | "you can go tracking down segfaults or merge this" :P | 13:57 |
| TheJulia | jroll: pretty much | 13:57 |
| dtantsur | jroll: this patch is a good thing regardless of segfaults | 13:57 |
| *** yangyapeng has quit IRC | 13:58 | |
| dtantsur | it removes a hard dependency on ironic queens | 13:58 |
| jroll | dtantsur: I agree, but it isn't the "right" way that we wanted to do it (that we specified in launchpad) | 13:58 |
| rloo | i'm still trying to catch up but i wonder if the segfaults are due to eg dynamic linking to new/incompatible libraries, old nova + new whatever | 13:58 |
| TheJulia | rloo: current status, fyi, is we marked the grenade tests non-voting for now | 13:58 |
| TheJulia | well, grenade multinode | 13:58 |
| *** crushil has quit IRC | 13:58 | |
| * johnthetubaguy heard something about nova-cores | 13:58 | |
| rloo | TheJulia: ok. but we haven't changed that grenade multinode test? | 13:58 |
| TheJulia | rloo: basically, during the upgrade, a bunch of pip libraries with their chared c code are getting deleted | 13:59 |
| TheJulia | or moved | 13:59 |
| TheJulia | and then deleted | 13:59 |
| TheJulia | so.... bad things happen afterwards | 13:59 |
| *** jaganathan has quit IRC | 13:59 | |
| rloo | TheJulia: ok. SOOO... if I understand it, at the end of the day, with your proposed patches, etc, we will not be able to test rolling upgrades, right? | 13:59 |
| TheJulia | rloo: We can, we only upgrade nova and allow it to handle the older version | 14:00 |
| jroll | johnthetubaguy: *cough* https://review.openstack.org/#/c/535786/ | 14:00 |
| patchbot | patch 535786 - nova - Ironic: negotiate microversion to allow downgrade ... | 14:00 |
| jroll | and good morning :) | 14:00 |
| *** weshay is now known as weshay|rover | 14:00 | |
| johnthetubaguy | yeah, just peaking at that now | 14:01 |
| jroll | thanks | 14:01 |
| dtantsur | jroll: "not the right way" works as a workaround :) | 14:01 |
| * jroll rechecks | 14:01 | |
| dtantsur | I mean, it's a negotiation, just done inside of nova instead of inside ironicclient | 14:01 |
| rloo | TheJulia: and not upgrade ironic at all? Well, let me review the patches that need to land. we can discuss later. | 14:01 |
| *** baha has joined #openstack-ironic | 14:01 | |
| TheJulia | We will make it better, just need if sooner rather than later | 14:01 |
| jroll | dtantsur: you don't have to convince me it's a good thing, you have to convince nova folks that it's worth merging last minute :P | 14:02 |
| jroll | I think I phrased it nicely | 14:02 |
| TheJulia | rloo: we upgrade the ironic conductor, but the usable api endpoing is still the unupgaded subnode | 14:02 |
| rloo | jroll: it isn't last minute. last minute would be errr in 2 weeks time? :) | 14:02 |
| dtantsur | yeah, it's not that bad | 14:02 |
| jroll | rloo: sometimes feels like nova last-minute begins at milestone 1 | 14:02 |
| dtantsur | I remember merging some deprecations after nova rc1 ;) | 14:03 |
| rloo | TheJulia: besides the nova patch, what's the other one? | 14:04 |
| rloo | TheJulia: to fix the grenade test. Am updating whiteboard with the links. | 14:04 |
| TheJulia | rloo: just turns the job back to voting and restarts nova stuffs | 14:04 |
| TheJulia | rloo: it has a depends-on flag for the nova patch | 14:04 |
| rloo | TheJulia: ok, that'll be an easy patch to review :) | 14:04 |
| TheJulia | rloo: thanks! I completely forgot to update the whiteboard on friday since I had started my trek to southern california | 14:05 |
| TheJulia | it is soooo dark outside.... | 14:05 |
| *** c3-oscar has quit IRC | 14:06 | |
| rloo | TheJulia: many thanks for working on this! | 14:07 |
| *** mjturek has joined #openstack-ironic | 14:08 | |
| johnthetubaguy | jroll: been wondering about the API version we request (mostly for the traits work) | 14:08 |
| johnthetubaguy | jroll: does that ironic client not have a max API version it will request? | 14:08 |
| jroll | johnthetubaguy: it does, let me find it | 14:09 |
| johnthetubaguy | mgoddard_: ^ | 14:09 |
| jroll | johnthetubaguy: oh, maybe only for osc https://review.openstack.org/#/c/532622/5/ironicclient/osc/plugin.py | 14:10 |
| patchbot | patch 532622 - python-ironicclient - Traits support | 14:10 |
| johnthetubaguy | jroll: ah, curious | 14:11 |
| * TheJulia goes back to the patches that changes some of that up | 14:14 | |
| *** jaganathan has joined #openstack-ironic | 14:16 | |
| *** rbudden has joined #openstack-ironic | 14:23 | |
| *** vmud213 has quit IRC | 14:23 | |
| *** makowals has quit IRC | 14:27 | |
| *** Goneri has joined #openstack-ironic | 14:28 | |
| rloo | jroll: are you sure 535786 works? maybe I'm still asleep... | 14:29 |
| *** Guest86649 is now known as dansmith | 14:29 | |
| jroll | rloo: I played with it in devstack and it seemed to, why | 14:29 |
| jroll | I didn't actually do a rebuild though | 14:29 |
| rloo | jroll: i'm asleep. | 14:29 |
| *** rbartal has joined #openstack-ironic | 14:29 | |
| jroll | k :) | 14:30 |
| *** makowals has joined #openstack-ironic | 14:30 | |
| johnthetubaguy | jroll: so seems legit, found a minor nit on the reported hypervisor version that is probably worth fixing up | 14:31 |
| *** yangyapeng has joined #openstack-ironic | 14:32 | |
| johnthetubaguy | jroll: FWIW, I like the static two option negotiation, so much easier to test, but clearly would be good to let the client do the choosing logic | 14:32 |
| *** jjohnson2 has joined #openstack-ironic | 14:33 | |
| jroll | johnthetubaguy: good point on the sighup thing, with the hyp version, that only uses the major version, not minor | 14:33 |
| jroll | (which makes it totally useless, but ya know | 14:34 |
| *** crushil has joined #openstack-ironic | 14:34 | |
| *** lucasagomes is now known as lucas-hungry | 14:34 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: API: Node Traits API https://review.openstack.org/532269 | 14:36 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add node traits to API reference https://review.openstack.org/536384 | 14:36 |
| openstackgerrit | Mark Goddard proposed openstack/ironic-tempest-plugin master: Add node trait tests https://review.openstack.org/535908 | 14:36 |
| jroll | johnthetubaguy: do you know if there's something in nova I should tie into for that sighup hook, or just use the normal python stuff? | 14:40 |
| johnthetubaguy | jroll: so compute manager already has a handler, so I think you could make it call a virt driver method, if there isn't one already, will see if I can see it | 14:40 |
| jroll | ah, I can look | 14:41 |
| johnthetubaguy | jroll: I think it is this thing: https://github.com/openstack/nova/blob/a40c00957ef4552681c172913bae1135a1614bbc/nova/compute/manager.py#L543 | 14:41 |
| jroll | hm | 14:42 |
| johnthetubaguy | jroll: its a bit clearer here: https://github.com/openstack/nova/blob/a40c00957ef4552681c172913bae1135a1614bbc/nova/manager.py#L143 | 14:42 |
| jroll | right right | 14:42 |
| jroll | johnthetubaguy: that doesn't touch the virt driver at all, though :/ | 14:43 |
| johnthetubaguy | yeah, not yet... | 14:43 |
| johnthetubaguy | new virt driver method I guess | 14:43 |
| *** makowals has quit IRC | 14:44 | |
| johnthetubaguy | jroll: yeah, sorry about the false alarm on reporting the hypervisor version, that's not really a problem, as you point out | 14:46 |
| jroll | yeah, no worries | 14:46 |
| jroll | johnthetubaguy: think I should inject that new method now, or down the road? | 14:46 |
| *** makowals has joined #openstack-ironic | 14:47 | |
| johnthetubaguy | so the current upgrade method includes a SIG_HUP of nova-compute once everything is upgraded | 14:47 |
| johnthetubaguy | that triggers the min_compute_version to get re-calculated and bumps the RPC automagically | 14:47 |
| johnthetubaguy | ... I am tempted to piggy back that | 14:47 |
| johnthetubaguy | mostly because the graceful shutdown is so bad in Nova... :( | 14:48 |
| johnthetubaguy | tl;dr I would do it now | 14:49 |
| *** rpioso|afk is now known as rpioso | 14:50 | |
| rpioso | Good morning | 14:50 |
| jroll | johnthetubaguy: yeah, that's a fair point, can do | 14:50 |
| johnthetubaguy | jroll: so I just though about testing this, I guess we don't have a job with old ironic and new nova? | 14:52 |
| jroll | johnthetubaguy: well, the impetus for doing this quickly, now, is because we need to turn our multinode grenade job into exactly that | 14:52 |
| johnthetubaguy | jroll: ah, sweet | 14:53 |
| jroll | johnthetubaguy: https://review.openstack.org/535596 | 14:53 |
| patchbot | patch 535596 - ironic - Rework upgrade to upgrade nova/ironic | 14:53 |
| rloo | jroll: question about 535786. is the intent then, that with every named release of nova, we update that PREVIOUS_IRONIC_API_VERSION (regardless of if IRONIC_API_VERSION was up'd?) and we use the previous named release of ironic (as opposed to a release that happened during a dev cycle)? | 14:53 |
| jroll | rloo: I guess so, yeah. I think the plan for next cycle is to replace it with the client-driven negotiation anyway | 14:54 |
| jroll | s/think/hope/ | 14:54 |
| rloo | jroll: ok. not sure if worth mentioning. in case someone asks. | 14:55 |
| openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: Raise exception for unsupported microversion https://review.openstack.org/532979 | 14:56 |
| openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: Minor changes to version negotiation logic https://review.openstack.org/536390 | 14:56 |
| johnthetubaguy | I still think the static version are nicer, just need to ask the client if the higher version is available I think | 14:56 |
| * jroll needs to re-read the RFE for the future plan | 14:57 | |
| * johnthetubaguy probably should read it first... | 14:57 | |
| *** jlabarre has quit IRC | 14:58 | |
| *** jlabarre has joined #openstack-ironic | 14:59 | |
| TheJulia | jroll: johnthetubaguy Preferably go ahead and do it this cycle, but if we can't, then we can't. some of the patches are already up on python-ironicclient. I need to add min/max behavior soon | 15:01 |
| TheJulia | rloo: a list will most likely be maintained since it is the most static, and we can track capabilities by that or behavior differences based upon the actual negotiated version | 15:03 |
| johnthetubaguy | TheJulia: ++ | 15:03 |
| rloo | TheJulia: ok, lets see how that shakes out. (And we can discuss at PTG if we need to, but I suspect we won't.) | 15:04 |
| johnthetubaguy | You probably saw this already, but this is the cinder stuff that landed relatively recently: https://github.com/openstack/nova/blob/a40c00957ef4552681c172913bae1135a1614bbc/nova/volume/cinder.py#L60 | 15:04 |
| rloo | johnthetubaguy: I didn't see it (no surprise). Good to know; we should do something similar for ironicclient then. | 15:05 |
| *** namnh has joined #openstack-ironic | 15:05 | |
| johnthetubaguy | it seems to work at least :) | 15:05 |
| johnthetubaguy | I always seem to forget you need to worry about all three of: the caller, the server and the installed client code | 15:06 |
| rloo | johnthetubaguy: don't say that. lots of things 'seem to work' but aren't good enuf :D | 15:06 |
| johnthetubaguy | heh, true :) | 15:06 |
| TheJulia | rloo: johnthetubaguy: we have all of that burried in ironicclient/common/http.py | 15:06 |
| rloo | johnthetubaguy: i like the 'consistency' approach/reason for doing things :) | 15:07 |
| * johnthetubaguy that swaddling looks good... (turns around for 10 seconds).. wait how did you get your arms out and scratch your face already? | 15:07 | |
| rloo | brings back memories to /me | 15:08 |
| *** r-daneel has joined #openstack-ironic | 15:09 | |
| *** crushil_ has quit IRC | 15:11 | |
| *** rbartal has quit IRC | 15:12 | |
| *** cjloader has joined #openstack-ironic | 15:12 | |
| *** cjloader has quit IRC | 15:13 | |
| *** cjloader has joined #openstack-ironic | 15:16 | |
| *** cjloader has quit IRC | 15:16 | |
| *** cjloader has joined #openstack-ironic | 15:16 | |
| rloo | TheJulia, dtantsur: with multinode grenade being non-voting, what patches are OK to approve? Are we using our own judgement? | 15:18 |
| dtantsur | rloo: yes. something not affecting rolling upgrades | 15:18 |
| dtantsur | so I'd refrain from RPC changes for now.. | 15:18 |
| rloo | dtantsur: ok -- worth mentioning in our meeting today. or sending email to cores? | 15:19 |
| rloo | dtantsur: or devlist. | 15:19 |
| dtantsur | yeah | 15:19 |
| rloo | dtantsur: or -2ing those patches | 15:19 |
| dtantsur | -2 EVERYTHING \o/ | 15:20 |
| rloo | dtantsur: NOOOOOOOO | 15:20 |
| dtantsur | no changes - no regressions, right? | 15:20 |
| rloo | dtantsur: true. no need to review too. PROFIT! | 15:20 |
| TheJulia | but then no progress :( | 15:22 |
| dtantsur | this is called stability :) | 15:22 |
| TheJulia | ugh | 15:22 |
| *** lucas-hungry is now known as lucasagomes | 15:23 | |
| TheJulia | I think we need to focus on client/driver patches, and just make sure we get everythiny to rock and roll w/r/t rpc and above changes once we have grenade sorted | 15:23 |
| TheJulia | _OR_ we could chain stuff for testing, but I'd prefer to avoid that path since it creates extra work | 15:24 |
| dtantsur | yeah | 15:25 |
| dtantsur | I mean, "yeah for option #1" | 15:25 |
| dtantsur | and I can offer you my data migration patches for review ;) | 15:26 |
| dtantsur | we don't run online_data_migrations in the multinode grenade, do we? | 15:26 |
| * TheJulia thinks | 15:26 | |
| TheJulia | not 100% sure, but I think we do not | 15:27 |
| *** trown|ruck is now known as trown|brb | 15:28 | |
| dtantsur | because we don't finish upgrade there | 15:28 |
| TheJulia | yeah | 15:29 |
| *** Goneri has quit IRC | 15:29 | |
| *** rbartal has joined #openstack-ironic | 15:30 | |
| * rloo wonders if dtantsur and TheJulia are impartial in this matter :D | 15:30 | |
| TheJulia | yeah, we pin on multinode, and do not run online_data_migrations | 15:30 |
| TheJulia | rloo: about what? | 15:31 |
| rloo | TheJulia: what to focus on now ;) | 15:31 |
| TheJulia | heh for the good of the project | 15:31 |
| dtantsur | I have a dream to not do the driver composition reform any more | 15:31 |
| dtantsur | the deprecation (and later removal) of classic drivers is between me and this dream | 15:32 |
| dtantsur | :) | 15:32 |
| *** cjloader has quit IRC | 15:32 | |
| rloo | dtantsur: dreams are GOOD; nightmares are bad :) | 15:32 |
| *** cjloader has joined #openstack-ironic | 15:33 | |
| rloo | dtantsur: i should dream about not reviewing driver composition patches :) | 15:33 |
| * TheJulia wonders if dmitry wishes to apply a sed statement to his last comments ;) | 15:33 | |
| *** trown|brb is now known as trown|ruck | 15:36 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/534934 | 15:37 |
| *** namnh has quit IRC | 15:38 | |
| *** namnh has joined #openstack-ironic | 15:39 | |
| *** namnh has quit IRC | 15:39 | |
| *** namnh has joined #openstack-ironic | 15:39 | |
| *** namnh has quit IRC | 15:39 | |
| *** namnh has joined #openstack-ironic | 15:40 | |
| *** namnh has quit IRC | 15:40 | |
| *** namnh has joined #openstack-ironic | 15:40 | |
| *** mgkwill has quit IRC | 15:40 | |
| *** portdirect has quit IRC | 15:40 | |
| *** BadCub has quit IRC | 15:40 | |
| *** mgkwill has joined #openstack-ironic | 15:41 | |
| *** BadCub has joined #openstack-ironic | 15:41 | |
| *** portdirect has joined #openstack-ironic | 15:41 | |
| *** Goneri has joined #openstack-ironic | 15:44 | |
| *** hw_wutianwei has joined #openstack-ironic | 15:45 | |
| mgoddard_ | will the next client release be 2.2.0? Is there a requirements bump patch for it? | 15:46 |
| dtantsur | TheJulia: not sure :) sometimes I do apply Russian grammar and way to express things to English | 15:47 |
| dtantsur | mgoddard_: 1. I think so, 2. not that I'm aware of | 15:47 |
| TheJulia | we actually need to review/land on the client | 15:47 |
| mgoddard_ | dtantsur: just looking for a patch to depend on for my nova virt driver change. | 15:50 |
| mgoddard_ | does the requirements bump change normally come after the release? | 15:50 |
| dtantsur | mgoddard_: you mean, automatically? no, somebody has to do it | 15:50 |
| mgoddard_ | dtantsur: I just mean in terms of ordering - is the release required before the requirements change? | 15:51 |
| dtantsur | mgoddard_: yes | 15:51 |
| mgoddard_ | dtantsur: ok, thanks | 15:52 |
| TheJulia | mgoddard_: I'd depend on the python-ironicclient change its self, not perfect, and when we update releases, we that depends-on can be updated | 15:53 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add baremetal neutron agent https://review.openstack.org/456235 | 15:53 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: start_flag = True, only first time, or conf change https://review.openstack.org/533707 | 15:53 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Switch from MechanismDriver to SimpleAgentMechanismDriverBase https://review.openstack.org/521838 | 15:53 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Devstack - Add ironic-neutron-agent https://review.openstack.org/530117 | 15:53 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add support to bind type vlan networks https://review.openstack.org/532349 | 15:53 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add dsvm job https://review.openstack.org/530409 | 15:53 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Docs - Add multi-tenant devstack - n-g-s vlans https://review.openstack.org/534459 | 15:53 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications https://review.openstack.org/524709 | 15:53 |
| mgoddard_ | TheJulia: will do | 15:53 |
| *** derekh has quit IRC | 15:56 | |
| *** derekh has joined #openstack-ironic | 15:56 | |
| *** ijw has joined #openstack-ironic | 15:57 | |
| dtantsur | folks, looking for a 2nd +2 to close the keystoneauth topic https://review.openstack.org/#/c/515787 | 16:01 |
| patchbot | patch 515787 - ironic-inspector - Add keystoneauth adapters | 16:01 |
| openstackgerrit | Merged openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/535117 | 16:01 |
| *** makowals has quit IRC | 16:02 | |
| *** stendulker has joined #openstack-ironic | 16:06 | |
| *** mjura has quit IRC | 16:08 | |
| jjohnson2 | So, So... haing a problem with 'tempest-devstack-ironic-pxe_ipmitool-pyghmi-src', it always fails now... | 16:08 |
| dtantsur | jjohnson2: mmm, we might have missed an update to switch to an out-of-tree tempest plugin | 16:11 |
| dtantsur | jjohnson2: this has to be fixed, lemme show how https://github.com/openstack/pyghmi/blob/master/playbooks/legacy/tempest-devstack-ironic-pxe_ipmitool-pyghmi-src/run.yaml#L75-L81 | 16:12 |
| dtantsur | to just https://github.com/openstack/ironic/blob/master/playbooks/legacy/tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3/run.yaml#L66 | 16:12 |
| jjohnson2 | ok, trying it out | 16:16 |
| jjohnson2 | dtantsur, https://review.openstack.org/#/c/536401/ | 16:18 |
| patchbot | patch 536401 - pyghmi - Fix the tempest job | 16:18 |
| *** rbartal has quit IRC | 16:19 | |
| *** stendulker has quit IRC | 16:20 | |
| *** jaganathan has quit IRC | 16:21 | |
| *** hshiina has joined #openstack-ironic | 16:23 | |
| *** namnh has quit IRC | 16:24 | |
| *** namnh has joined #openstack-ironic | 16:25 | |
| *** AlexeyAbashkin has quit IRC | 16:27 | |
| *** crushil has quit IRC | 16:28 | |
| *** namnh has quit IRC | 16:30 | |
| *** namnh has joined #openstack-ironic | 16:35 | |
| NobodyCam | Good Morning Ironic'ers | 16:36 |
| *** namnh has quit IRC | 16:39 | |
| *** penick has joined #openstack-ironic | 16:46 | |
| *** stendulker has joined #openstack-ironic | 16:47 | |
| TheJulia | good morning NobodyCam | 16:51 |
| *** tesseract-RH has quit IRC | 16:51 | |
| jroll | morning NobodyCam | 16:51 |
| NobodyCam | :) Good morning TheJulia and jroll :) | 16:52 |
| openstackgerrit | Merged openstack/ironic-inspector master: Imported Translations from Zanata https://review.openstack.org/533100 | 16:52 |
| openstackgerrit | Merged openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/534931 | 16:52 |
| dtantsur | morning NobodyCam | 16:54 |
| NobodyCam | :) morning dtantsur :) | 16:54 |
| * jroll anxiously awaits the meeting | 16:58 | |
| dtantsur | :) | 16:58 |
| jroll | oh wow, forgot to upload a new nova patch | 17:00 |
| TheJulia | doh! | 17:00 |
| dtantsur | #startmeeting ironic | 17:00 |
| openstack | Meeting started Mon Jan 22 17:00:33 2018 UTC and is due to finish in 60 minutes. The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot. | 17:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 17:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 17:00 | |
| TheJulia | o/ | 17:00 |
| openstack | The meeting name has been set to 'ironic' | 17:00 |
| dtantsur | hi all :) | 17:00 |
| NobodyCam | howdy | 17:00 |
| dtantsur | #link https://wiki.openstack.org/wiki/Meetings/Ironic | 17:00 |
| pas-ha | o/ | 17:00 |
| rpioso | o/ | 17:00 |
| *** derekh_ has joined #openstack-ironic | 17:00 | |
| jroll | \o | 17:01 |
| rloo | o/ | 17:01 |
| hjensas | o/ | 17:01 |
| hshiina | o/ | 17:01 |
| bfournie | o/ | 17:01 |
| mgoddard_ | \o/ | 17:01 |
| stendulker | o/ | 17:01 |
| dtantsur | #topic Announcements / Reminder | 17:01 |
| *** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)" | 17:01 | |
| dtantsur | We have a few | 17:01 |
| dtantsur | #info Jan 25 - Client release deadline and Feature Freeze | 17:02 |
| mjturek | o/ | 17:02 |
| dtantsur | this is Thursday | 17:02 |
| dtantsur | now, related: | 17:02 |
| dtantsur | #info dtantsur semi-unavailable Thu-Mon due to devconf.cz | 17:02 |
| jlvillal | o/ | 17:02 |
| dtantsur | so TheJulia, if you're available, you'll have to request the clients releases please | 17:02 |
| TheJulia | dtantsur: acknowledged, will do | 17:03 |
| dtantsur | I'll try to be online as much as it's possible | 17:03 |
| dtantsur | but on Thu I'll be in a train somewhere between Germany and Czechia | 17:03 |
| rloo | TheJulia will run next Monday's meeting too? | 17:03 |
| *** derekh has quit IRC | 17:03 | |
| TheJulia | I can | 17:03 |
| *** e0ne has quit IRC | 17:03 | |
| dtantsur | rloo: I should be here by that time, but in case I won't - yes | 17:03 |
| rloo | dtantsur: ok, Action Item - dtantsur to be back for meeting on Monday :) | 17:04 |
| dtantsur | got it :) | 17:04 |
| dtantsur | so, lemme repeat probably: | 17:04 |
| dtantsur | #info the feature freeze is coming, and the good time to request an exception, if you need it, is right now | 17:04 |
| dtantsur | I have a few potential exceptions to discuss for later today | 17:05 |
| dtantsur | but in the end all requests have to happen on the ML | 17:05 |
| rloo | dtantsur: how do we decide whether to grant a FFE? vote? or 2 cores agree or ?? | 17:05 |
| dtantsur | meh, I knew somebody would ask :D | 17:05 |
| dtantsur | 2 cores agree and no cores -2? | 17:05 |
| rloo | dtantsur: ok, w/i some time period i guess. | 17:05 |
| dtantsur | yep. for requests posted today we can give time till Thursday for cores to object | 17:06 |
| dtantsur | if you post a request on Thu, it may be delayed, so plan ahead | 17:06 |
| dtantsur | any more questions on the feature freeze? | 17:06 |
| rloo | dtantsur: and what's the deadline, if we grant FFE, to get that feature in? | 17:07 |
| *** marios has quit IRC | 17:07 | |
| dtantsur | so | 17:07 |
| dtantsur | lemme double-check the schedule | 17:07 |
| *** crushil has joined #openstack-ironic | 17:07 | |
| johnthetubaguy | so does traits need to request an exception now? | 17:07 |
| rloo | maybe RC1 target week? feb 5-9? | 17:07 |
| dtantsur | johnthetubaguy: tl;dr yes | 17:07 |
| *** fragatina has joined #openstack-ironic | 17:07 | |
| rloo | johnthetubaguy: we're going to get that in this week, right? :) | 17:07 |
| dtantsur | the final release will happen on Thu, Feb 8th | 17:07 |
| dtantsur | I don't want last-minute features, I've screwed up sushy enough :-/ | 17:08 |
| TheJulia | so even if we grant an ffe for traits, we can't land client support.... | 17:08 |
| rloo | dtantsur: the final RC is week of feb 19 | 17:08 |
| jroll | yeah, not worth ffe'ing traits, just land it this week :) | 17:08 |
| dtantsur | rloo: rc1 is when we branch | 17:08 |
| TheJulia | jroll: +++++ | 17:08 |
| * jlvillal thinks we don't want any raw features getting into sushy... | 17:08 | |
| johnthetubaguy | jroll: ++ | 17:08 |
| rloo | TheJulia: true | 17:08 |
| dtantsur | so rc1 is the final queens feature release | 17:09 |
| dtantsur | then stable/queens gets created, and master switches to rocky | 17:09 |
| dtantsur | so | 17:09 |
| rloo | dtantsur: OH. then def week of Feb 5 is it. wonder if we should make it the week before, but ... | 17:09 |
| dtantsur | #info Feb 8th is the final feature release, features that get an exception must land by Friday, Feb 2nd | 17:09 |
| dtantsur | sounds reasonable? | 17:10 |
| TheJulia | dtantsur: yes | 17:10 |
| rloo | works for me. thx. | 17:10 |
| jroll | ++ | 17:10 |
| *** Nisha_Agarwal has joined #openstack-ironic | 17:10 | |
| dtantsur | more questions? | 17:10 |
| rloo | johnthetubaguy: we'll get traits in if you get that nova patch in :) | 17:10 |
| johnthetubaguy | rloo: trying :) | 17:10 |
| *** pcaruana has quit IRC | 17:11 | |
| rloo | johnthetubaguy: we actually can't get traits in until we fix that grenade test | 17:11 |
| dtantsur | this ^^^ | 17:11 |
| * jroll is surprised mentioning segfaults on upgrade in irc doesn't gather much attention | 17:11 | |
| * johnthetubaguy nods | 17:11 | |
| dtantsur | I have a topic for a potential FFE for traits, let's wait for it please | 17:11 |
| dtantsur | #info PTG planning has started: https://etherpad.openstack.org/p/ironic-rocky-ptg | 17:11 |
| dtantsur | please dump your ideas :) | 17:11 |
| TheJulia | jroll: I entirely expect it could have been the messenger or otherwise focus at the time, multiple variables | 17:12 |
| dtantsur | #info stable/queens created for sushy and ironic-lib | 17:12 |
| rloo | yay | 17:12 |
| dtantsur | sushy got screwed up, sorry for that. we'll have an emergency fix release soon :( | 17:12 |
| rloo | dtantsur: boo | 17:12 |
| TheJulia | dtantsur: I thought we took care of that.... | 17:13 |
| rloo | TheJulia: i suspect he is referring to https://review.openstack.org/#/c/535910/ | 17:13 |
| patchbot | patch 535910 - sushy - Restores sushy session functionality. | 17:13 |
| dtantsur | TheJulia: there is one small place where an argument is passed as positional, and adding back username and password broke it | 17:14 |
| dtantsur | now, the patch does a lot of unnecessary changes, we need to clean it up | 17:14 |
| TheJulia | argh | 17:14 |
| dtantsur | the essence of the fix is one line | 17:14 |
| TheJulia | okay, lets move on | 17:14 |
| * TheJulia appologies for slight derail | 17:14 | |
| dtantsur | #topic Review action items from previous meeting | 17:15 |
| *** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)" | 17:15 | |
| dtantsur | #link http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-15-17.00.html | 17:15 |
| dtantsur | so, I posted patches enabling automatic migration to hardware types - see the whiteboard | 17:15 |
| dtantsur | I need a patch to move the CI to hardware types, then we can deprecate | 17:16 |
| *** hieulq has quit IRC | 17:16 | |
| dtantsur | this will be finished probably tomorrow | 17:16 |
| *** etingof has quit IRC | 17:16 | |
| dtantsur | jlvillal did announce the tempest plugin removal from tree | 17:16 |
| dtantsur | mjturek: how was the bug triage? | 17:17 |
| jlvillal | Yes, I did send out an email | 17:17 |
| rloo | http://lists.openstack.org/pipermail/openstack-dev/2018-January/126232.html email about removal | 17:17 |
| rloo | (and we broke pyghmi but no big deal :)) | 17:17 |
| dtantsur | yeah, we have a patch to fix pyghmi already | 17:18 |
| dtantsur | any comments? | 17:18 |
| rloo | not wrt AIs. should we mention grenade test breakage? | 17:19 |
| dtantsur | well, let's do it | 17:19 |
| dtantsur | #info the multinode grenade was broken and we disabled it, please be careful when approving patches affecting upgrade | 17:20 |
| dtantsur | #topic Review subteam status reports (capped at ten minutes) | 17:20 |
| *** openstack changes topic to "Review subteam status reports (capped at ten minutes) (Meeting topic: ironic)" | 17:20 | |
| dtantsur | #link https://etherpad.openstack.org/p/IronicWhiteBoard line 182 | 17:20 |
| rloo | wrt grenade tests, see whiteboard for more info :) | 17:20 |
| dtantsur | sorry, I totally forgot about bugs. partly because my dashboard was sent to /dev/null :( | 17:20 |
| dtantsur | I'll do something about it soon(ish) | 17:21 |
| rloo | I haven't looked at the ironic client API version negotiation patches; how are they? close? | 17:22 |
| TheJulia | I suspect not many teams made much progress due to the grenade issues last week. I for one, spent the majority of the week focused on that instead of reviewing, so i suspect there is just not much in the way of updates. :\ | 17:22 |
| *** stendulker has quit IRC | 17:22 | |
| dtantsur | rloo: they have my +2 | 17:22 |
| *** milan_ has quit IRC | 17:22 | |
| rloo | dtantsur: i'd say close then, thx! | 17:22 |
| rloo | johnthetubaguy: even if we manage to land the traits stuff this week, what about the nova part? ff is this week for nova. | 17:23 |
| dtantsur | any work that has "RPC" in it has to wait until the grenade job is fixed.. | 17:23 |
| jroll | oh, nova's FF is this week? sigh | 17:23 |
| TheJulia | dtantsur: I'll try to hammer out min/max version in the client sometime today, although no promises | 17:23 |
| dtantsur | TheJulia: awesome! | 17:24 |
| rloo | jroll: yup :-( johnthetubaguy, can we (you) ask for FFE in nova? | 17:24 |
| dtantsur | jroll: everyone's FF is M3 | 17:24 |
| *** trown|ruck is now known as trown|lunch | 17:24 | |
| johnthetubaguy | not sure how the nova FFE will work, normally PTL approved | 17:24 |
| johnthetubaguy | its normally needed only after thursday though | 17:24 |
| rloo | johnthetubaguy: ok, guess we'll see how things go this week | 17:24 |
| rloo | it is the same problem with rescue work. | 17:25 |
| jroll | dtantsur: ah right, got my dates mixed up with the RC1 cut | 17:25 |
| rloo | is shiv here? i don't see him. | 17:25 |
| dtantsur | yeah, it's complex.. RC1 is stable branching | 17:25 |
| *** baha has quit IRC | 17:25 | |
| dtantsur | rloo: bad time for east asia.. | 17:25 |
| *** baha has joined #openstack-ironic | 17:25 | |
| *** yshaban has joined #openstack-ironic | 17:26 | |
| rloo | dtantsur: yeah, was wondering what the status was wrt CI for that. the patches are up, but given that traits is a higher priority, i am 99% sure rescue won't land this week. | 17:26 |
| * rloo hopes that she is proven wrong | 17:26 | |
| dtantsur | it all boils down the grenade recovering.. | 17:27 |
| dtantsur | * boils down to | 17:27 |
| dtantsur | both patches have RPC bits that need testing by it | 17:27 |
| rloo | even with grenade working, so many things to review this week... :-( | 17:27 |
| rloo | and i'm sure that gate will be slow (er than usual) | 17:27 |
| rloo | ok, i'm done with status (and trying to remain hopeful) | 17:28 |
| * TheJulia slihtly mentally steps away for a couple minutes | 17:28 | |
| dtantsur | moving on then? | 17:28 |
| dtantsur | #topic Deciding on priorities for the coming week | 17:28 |
| *** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 17:28 | |
| dtantsur | so | 17:28 |
| rloo | dtantsur: we need to land the grenade fix, after nova lands the patch: https://review.openstack.org/535596 | 17:29 |
| patchbot | patch 535596 - ironic - Rework upgrade to upgrade nova/ironic | 17:29 |
| dtantsur | I'd leave the 4 topics we discussed as important, with ironicclient gettign the highest priority | 17:30 |
| dtantsur | because we have a rigid deadline for clients | 17:30 |
| rloo | dtantsur: classic driver migration can slip to next week? | 17:31 |
| dtantsur | rloo: I'd prefer it does not | 17:31 |
| dtantsur | it's a big change, I'd like to have as much as possible before FF | 17:31 |
| rloo | dtantsur: i'd prefer not either, but i prefer more that traits lands | 17:31 |
| Nisha_Agarwal | dtantsur, ironic stable branch cut will be done this week, right? | 17:31 |
| dtantsur | or rather: I'd like to have the database migration before FF; the actual deprecation, CI, docs can wait | 17:32 |
| dtantsur | Nisha_Agarwal: no, on Feb 8th | 17:32 |
| *** dprince has quit IRC | 17:32 | |
| dtantsur | Nisha_Agarwal: but feature freeze is this Thu | 17:32 |
| rloo | dtantsur: ok, i'll see what I can do :) | 17:32 |
| Nisha_Agarwal | dtantsur, ok. | 17:32 |
| dtantsur | rloo: the deprecation work is not blocked by the multinode CI, so we can do it around today.. | 17:32 |
| Nisha_Agarwal | so next week high priority bugs and high priority left over work from features will be merged in, right? | 17:33 |
| rloo | ok, i'm good with the priorities, except we need to fix grenade job. but we all know that. | 17:33 |
| dtantsur | Nisha_Agarwal: any bugs and features that get an exception | 17:34 |
| TheJulia | w/r/t priorities, I'm good with what is there, we might want to add a note that the list may be updated as the week goes on if we land traits/rescue stuff and manage to get to be able to expose their apis | 17:34 |
| dtantsur | I mean: 1. any bugs, 2. features that can an exception | 17:34 |
| dtantsur | we don't need exceptions for bugs :) | 17:34 |
| Nisha_Agarwal | dtantsur, yes :) | 17:34 |
| dtantsur | TheJulia: yeah, this is always implied | 17:34 |
| rloo | Nisha_Agarwal: no guarantee of 'merged in'. just that people might have time to focus on those other things. | 17:34 |
| Nisha_Agarwal | rloo, yeah i know | 17:35 |
| Nisha_Agarwal | rloo, i just asked because i the stable branch gets cut with FF | 17:35 |
| Nisha_Agarwal | :) | 17:35 |
| rloo | Nisha_Agarwal: good, wanted to clarify cuz you said 'will be merged in' above :) | 17:35 |
| rloo | Nisha_Agarwal: and again, stable branch does not get cut with FF | 17:35 |
| Nisha_Agarwal | s/i/i thought | 17:36 |
| dtantsur | #topic Appointing a bug triaging lead for the coming week | 17:36 |
| *** openstack changes topic to "Appointing a bug triaging lead for the coming week (Meeting topic: ironic)" | 17:36 | |
| dtantsur | I'm not sure anyone will have time for it, but just in case you want to.. | 17:36 |
| TheJulia | mjturek: do you by chance have time again this week? | 17:36 |
| mjturek | sure! | 17:37 |
| TheJulia | thanks! | 17:37 |
| dtantsur | thanks mjturek | 17:37 |
| dtantsur | #action mjturek to continue leading the bug triage this week | 17:37 |
| dtantsur | okay, let's discuss some FFEs | 17:37 |
| dtantsur | #topic FFE: Routed network support | 17:37 |
| *** openstack changes topic to "FFE: Routed network support (Meeting topic: ironic)" | 17:37 | |
| dtantsur | #link http://lists.openstack.org/pipermail/openstack-dev/2018-January/126296.html | 17:37 |
| dtantsur | this got 2 cores backing it, and 2 +1s from other cores. no objections so far. | 17:38 |
| dtantsur | approving the FFE in 3... | 17:38 |
| dtantsur | 2... | 17:38 |
| dtantsur | 1... | 17:39 |
| rloo | dtantsur: i thought we couldn't approve til Thurs | 17:39 |
| dtantsur | rloo: this was requested earlier | 17:39 |
| dtantsur | I was talking about things that can requested today or tomorrow morning | 17:39 |
| rloo | dtantsur: heh, ok then. | 17:39 |
| dtantsur | people had time to object, the last chance is right now :) | 17:39 |
| dtantsur | #agreed the routed network support work gets an FFE | 17:39 |
| dtantsur | hjensas: ^^^ | 17:39 |
| hjensas | \o/ | 17:40 |
| jlvillal | Do any of the patches actually touch Ironic at this time? Seems low risk to approve. | 17:40 |
| dtantsur | jlvillal: only CI, I think | 17:40 |
| hjensas | jlvillal: 1 patch + CI. | 17:40 |
| jlvillal | Okay | 17:40 |
| dtantsur | #topic Potential FFE: classic drivers deprecation | 17:40 |
| *** openstack changes topic to "Potential FFE: classic drivers deprecation (Meeting topic: ironic)" | 17:40 | |
| jlvillal | No objection at this point from me. | 17:40 |
| dtantsur | this was not formally requested, just collecting early feedback | 17:40 |
| rloo | dtantsur: put me down as core reviewer for deprecation | 17:40 |
| dtantsur | thanks rloo! | 17:40 |
| rloo | dtantsur: cuz you're going to make me do it anyway :D | 17:41 |
| dtantsur | you know me :) | 17:41 |
| dtantsur | any early objections / questions? | 17:41 |
| dtantsur | okay, I'll propose it formally later today | 17:42 |
| dtantsur | #topic Potential FFE: rescue mode | 17:42 |
| *** openstack changes topic to "Potential FFE: rescue mode (Meeting topic: ironic)" | 17:42 | |
| rloo | that won't work, it needs client update | 17:42 |
| dtantsur | well, we can land the ironic part, and at least get rid of it | 17:43 |
| rloo | are you good with it being the ironic code (and ipa code)? | 17:43 |
| TheJulia | I think the ipa code already landed | 17:43 |
| rloo | TheJulia: even better! | 17:43 |
| dtantsur | I think there is a tinyipa patch? the coreos one landed | 17:43 |
| TheJulia | even then, it is not api related, it is minor | 17:43 |
| dtantsur | yep | 17:44 |
| TheJulia | I'm +2 to granting an ffe for rescue and landing what we can | 17:44 |
| rloo | dtantsur: so i'm good with it and can be core reviewer. unless we have a FFE for traits too. then my loyalties may change. | 17:44 |
| dtantsur | this is my next topic :) | 17:44 |
| dtantsur | but I can be a core for rescue | 17:44 |
| dtantsur | however, I'm not leading the rescue work, so I won't request this FFE myself | 17:44 |
| dtantsur | Nisha_Agarwal: can you please talk to Shiv, if he wants to do ^^^? | 17:45 |
| Nisha_Agarwal | dtantsur, sure | 17:45 |
| dtantsur | thanks! | 17:45 |
| dtantsur | okay, this was just to collect initial feedback, you'll be able to agree/object on the ML | 17:45 |
| *** penick has quit IRC | 17:45 | |
| dtantsur | #topic Potential FFE: traits API | 17:46 |
| *** openstack changes topic to "Potential FFE: traits API (Meeting topic: ironic)" | 17:46 | |
| dtantsur | this last one I have | 17:46 |
| rloo | ditto :) | 17:46 |
| dtantsur | from the project's point of view, this is higher than rescue | 17:46 |
| dtantsur | from the point of view of not annoying contributors, rescue is higher than this :) | 17:46 |
| rloo | ditto == what i said above in rescue, applies here. | 17:46 |
| * TheJulia feels there is consensus | 17:46 | |
| dtantsur | yes, the scope of the FFE will be only ironic parts for both | 17:46 |
| dtantsur | anyone here wants to request an FFE for traits? anyone objects to it? | 17:47 |
| mgoddard_ | I would like to request one :) | 17:47 |
| *** dprince has joined #openstack-ironic | 17:47 | |
| dtantsur | mgoddard_: please do then (on the ML, with pros/cons/risks and backing cores) | 17:47 |
| rloo | we need another core reviewer to commit to traits | 17:48 |
| mgoddard_ | dtantsur: will do | 17:48 |
| dtantsur | I'll be busy with drivers and rescue.. | 17:48 |
| johnthetubaguy | having the API landed, but no client and nova bits is not ideal, but way better than not having it at all | 17:48 |
| rloo | mgoddard_: guess you can ask later for another core to help out | 17:49 |
| TheJulia | I can, I'd prefer to try and get everything pushed through prior and get the client bits too, but... grenade | 17:49 |
| * jroll side note: TheJulia: I'm discussing the gate breakage thing in -nova, fyi | 17:49 | |
| rloo | thx TheJulia | 17:49 |
| dtantsur | johnthetubaguy, mgoddard_, not all cores are here or active right now: please find someone to back your FFE | 17:49 |
| dtantsur | oh, thanks TheJulia | 17:49 |
| rloo | anyway, traits will land this week. ha ha. | 17:49 |
| dtantsur | ideally! | 17:49 |
| mgoddard_ | thanks TheJulia! | 17:49 |
| dtantsur | #topic Open discussion | 17:49 |
| *** openstack changes topic to "Open discussion (Meeting topic: ironic)" | 17:49 | |
| dtantsur | if you want to quickly chat about your FFE, this is about the good time | 17:50 |
| *** hoonetorg has quit IRC | 17:50 | |
| TheJulia | mgoddard_: Idea: test patch everything together and I guess we could consider landing because then we would be able to see grenade passing | 17:50 |
| dtantsur | curl https://example.com/crickets | 17:50 |
| rloo | crickets | 17:51 |
| * TheJulia thinks she needs to register a domain | 17:51 | |
| dtantsur | thanks all! 9 minutes of life are yours again | 17:51 |
| dtantsur | #endmeeting | 17:51 |
| *** openstack 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" | 17:51 | |
| openstack | Meeting ended Mon Jan 22 17:51:37 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:51 |
| TheJulia | \o/ | 17:51 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-22-17.00.html | 17:51 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-22-17.00.txt | 17:51 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-01-22-17.00.log.html | 17:51 |
| dtantsur | TheJulia++ | 17:51 |
| mgoddard_ | TheJulia: what do you mean by test patch everything together? | 17:52 |
| Nisha_Agarwal | dtantsur, vendor features could be proposed for FFE? | 17:52 |
| Nisha_Agarwal | above whatever discussed are nothing specific to any vendor | 17:52 |
| *** yshaban has quit IRC | 17:53 | |
| dtantsur | Nisha_Agarwal: yes, they can. I'll send a email outlining the process in a few minutes | 17:53 |
| Nisha_Agarwal | ok. I was thinking BFV for ilo drivers is a small one, if cores find time to look into it in RC cycle | 17:54 |
| *** AlexeyAbashkin has joined #openstack-ironic | 17:54 | |
| *** AlexeyAbashkin has quit IRC | 17:56 | |
| *** srart has joined #openstack-ironic | 17:56 | |
| *** derekh_ has quit IRC | 18:00 | |
| dtantsur | can someone remind me, what did we agree on? having minimum two cores to back an FFE or one? Or one if you're a core yourself? | 18:00 |
| dtantsur | rloo, TheJulia ^^^ | 18:00 |
| rloo | dtantsur: two cores | 18:01 |
| rloo | dtantsur: we didn't discuss what to do if you're a core yourself | 18:01 |
| dtantsur | then I need someone else for the driver composition | 18:01 |
| dtantsur | anyone? :) | 18:02 |
| * dtantsur sees people hiding | 18:02 | |
| rloo | dtantsur: we need two cores to approve anyway | 18:02 |
| *** penick has joined #openstack-ironic | 18:03 | |
| dtantsur | TheJulia, jlvillal, vdrok, anyone interested to help moving the classic drivers deprecation forward during the feature freeze? | 18:03 |
| * dtantsur checks who's online now | 18:03 | |
| *** baha has quit IRC | 18:04 | |
| *** rmcall has joined #openstack-ironic | 18:05 | |
| *** mgoddard_ has quit IRC | 18:05 | |
| *** gmonteiro has quit IRC | 18:06 | |
| TheJulia | I feel like I'm going to need a list of things to review | 18:06 |
| *** hoonetorg has joined #openstack-ironic | 18:07 | |
| dtantsur | heh, I'll see if I can talk someone in the EU time to do it :) | 18:08 |
| *** hshiina has quit IRC | 18:08 | |
| *** dtantsur is now known as dtantsur|afk | 18:09 | |
| dtantsur|afk | o/ | 18:09 |
| TheJulia | woot \o/ offer accepted | 18:12 |
| * TheJulia dances | 18:12 | |
| NobodyCam | w00t | 18:12 |
| *** e0ne has joined #openstack-ironic | 18:12 | |
| vdrok | dtantsur|afk: heya, probably not me | 18:15 |
| vdrok | good morning all | 18:15 |
| NobodyCam | morning vdork | 18:16 |
| NobodyCam | gah *vdrok | 18:16 |
| vdrok | :D | 18:16 |
| vdrok | morning NobodyCam | 18:16 |
| *** e0ne has quit IRC | 18:16 | |
| NobodyCam | :) | 18:16 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: Restores sushy session functionality. https://review.openstack.org/535910 | 18:19 |
| *** trown|lunch is now known as trown | 18:21 | |
| *** trown is now known as trown|ruck | 18:22 | |
| *** pcaruana has joined #openstack-ironic | 18:32 | |
| openstackgerrit | Merged openstack/virtualbmc master: Updated from global requirements https://review.openstack.org/534073 | 18:34 |
| *** Nisha_Agarwal has quit IRC | 18:34 | |
| *** dsneddon_ has joined #openstack-ironic | 18:35 | |
| *** e0ne has joined #openstack-ironic | 18:36 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:38 | |
| * TheJulia finally goes and eats something for breakfast | 18:39 | |
| *** jaganathan has joined #openstack-ironic | 18:40 | |
| *** AlexeyAbashkin has quit IRC | 18:42 | |
| *** rloo has quit IRC | 18:43 | |
| *** dsneddon_ has quit IRC | 18:44 | |
| *** dsneddon has joined #openstack-ironic | 18:44 | |
| *** dsneddon has quit IRC | 18:45 | |
| *** dsneddon has joined #openstack-ironic | 18:46 | |
| *** dsneddon has quit IRC | 18:46 | |
| *** rloo has joined #openstack-ironic | 18:46 | |
| *** dsneddon has joined #openstack-ironic | 18:49 | |
| *** mrtenio has quit IRC | 18:49 | |
| *** links has quit IRC | 18:55 | |
| *** baha has joined #openstack-ironic | 18:57 | |
| *** fragatina has quit IRC | 18:58 | |
| *** jkilpatr has quit IRC | 19:02 | |
| *** lucasagomes is now known as lucas-afk | 19:04 | |
| *** penick has quit IRC | 19:05 | |
| *** actatux has quit IRC | 19:08 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 19:08 | |
| *** Goneri has quit IRC | 19:09 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add RPC object for traits https://review.openstack.org/532268 | 19:09 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add RPC API and conductor manager for traits https://review.openstack.org/535296 | 19:09 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: API: Node Traits API https://review.openstack.org/532269 | 19:09 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add node traits to API reference https://review.openstack.org/536384 | 19:09 |
| *** jkilpatr has joined #openstack-ironic | 19:16 | |
| *** AlexeyAbashkin has quit IRC | 19:20 | |
| *** penick has joined #openstack-ironic | 19:26 | |
| *** actatux has joined #openstack-ironic | 19:31 | |
| *** mgoddard has quit IRC | 19:36 | |
| *** mgoddard has joined #openstack-ironic | 19:36 | |
| *** pcaruana has quit IRC | 19:43 | |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: Restores sushy session functionality. https://review.openstack.org/535910 | 19:46 |
| *** fragatina has joined #openstack-ironic | 19:47 | |
| rloo | if we had 100% coverage, would it make it easier to review patches, to see if they are missing any unit tests? (cuz I hate reviewing tests more than I hate reviewing code...) | 19:55 |
| openstackgerrit | Merged openstack/ironic-inspector master: Add keystoneauth adapters https://review.openstack.org/515787 | 19:55 |
| *** dsneddon has quit IRC | 19:56 | |
| *** namnh has joined #openstack-ironic | 20:02 | |
| *** dsneddon has joined #openstack-ironic | 20:03 | |
| *** namnh has quit IRC | 20:03 | |
| *** namnh has joined #openstack-ironic | 20:03 | |
| *** harlowja has joined #openstack-ironic | 20:04 | |
| *** namnh has quit IRC | 20:07 | |
| openstackgerrit | Michael Turek proposed openstack/ironic-python-agent master: Use lshw in place of dmidecode for the default hardware manager https://review.openstack.org/501799 | 20:09 |
| *** Goneri has joined #openstack-ironic | 20:20 | |
| jroll | TheJulia: I'm increasingly suspicious of this apt-get install about 3 minutes before the segfaults start: http://logs.openstack.org/36/509336/31/check/ironic-grenade-dsvm-multinode-multitenant/6da9163/logs/grenade.sh.txt.gz#_2018-01-18_05_46_59_521 | 20:20 |
| *** jkilpatr has quit IRC | 20:25 | |
| *** bfournie has quit IRC | 20:26 | |
| TheJulia | did the python-libvirt package get updated? | 20:28 |
| *** racedo has quit IRC | 20:28 | |
| *** mjturek_ has joined #openstack-ironic | 20:31 | |
| *** hw_wutianwei has quit IRC | 20:31 | |
| *** hw_wutianwei has joined #openstack-ironic | 20:31 | |
| *** penick has quit IRC | 20:32 | |
| *** namnh_ has joined #openstack-ironic | 20:32 | |
| *** mjturek has quit IRC | 20:33 | |
| *** namnh_ has quit IRC | 20:33 | |
| *** namnh_ has joined #openstack-ironic | 20:33 | |
| *** johnthetubaguy has quit IRC | 20:33 | |
| *** crushil has quit IRC | 20:36 | |
| *** athomas has quit IRC | 20:39 | |
| *** johnthetubaguy has joined #openstack-ironic | 20:40 | |
| *** mjturek has joined #openstack-ironic | 20:41 | |
| *** mjturek has quit IRC | 20:41 | |
| *** mjturek_ has quit IRC | 20:41 | |
| *** ijw has quit IRC | 20:41 | |
| *** ijw has joined #openstack-ironic | 20:42 | |
| *** ijw has quit IRC | 20:42 | |
| *** ijw_ has joined #openstack-ironic | 20:43 | |
| *** namnh_ has quit IRC | 20:45 | |
| *** r-daneel has quit IRC | 20:46 | |
| *** ijw_ has quit IRC | 20:47 | |
| jroll | TheJulia: yep, among others | 20:49 |
| *** crushil has joined #openstack-ironic | 20:51 | |
| *** jkilpatr has joined #openstack-ironic | 20:53 | |
| *** etingof has joined #openstack-ironic | 20:54 | |
| TheJulia | well, seems were at distribution whim at that point and should route around such that were providing the most flexibility | 21:00 |
| *** r-daneel has joined #openstack-ironic | 21:04 | |
| *** baha has quit IRC | 21:04 | |
| *** penick has joined #openstack-ironic | 21:05 | |
| *** e0ne has quit IRC | 21:22 | |
| *** jcoufal has quit IRC | 21:23 | |
| *** bfournie has joined #openstack-ironic | 21:32 | |
| *** jjohnson2 has quit IRC | 21:39 | |
| *** trown|ruck is now known as trown|outtypewww | 22:02 | |
| *** jtomasek has quit IRC | 22:03 | |
| *** jlabarre has quit IRC | 22:12 | |
| *** openstackgerrit has quit IRC | 22:18 | |
| *** rcernin has joined #openstack-ironic | 22:19 | |
| *** openstackgerrit has joined #openstack-ironic | 22:20 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/networking-generic-switch master: Stop using populate_ml2_config in devstack plugin https://review.openstack.org/536610 | 22:20 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/ironic master: DNM testing whether lib/neutron switch breaks this repo https://review.openstack.org/535928 | 22:20 |
| *** crushil_ has joined #openstack-ironic | 22:24 | |
| *** crushil has quit IRC | 22:28 | |
| *** ijw has joined #openstack-ironic | 22:33 | |
| openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: WIP: Min/max version logic https://review.openstack.org/536617 | 22:44 |
| * TheJulia goes and puts brain on to other things for a little while | 22:46 | |
| *** cdearborn has quit IRC | 22:59 | |
| *** pmannidi has joined #openstack-ironic | 23:09 | |
| *** dprince has quit IRC | 23:15 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 23:18 | |
| *** AlexeyAbashkin has quit IRC | 23:23 | |
| *** kiseok7_ has quit IRC | 23:26 | |
| *** patchbot has quit IRC | 23:26 | |
| *** kiseok7_ has joined #openstack-ironic | 23:26 | |
| *** betherly has quit IRC | 23:26 | |
| *** patchbot has joined #openstack-ironic | 23:26 | |
| *** rajinir has quit IRC | 23:26 | |
| *** gregwork has quit IRC | 23:27 | |
| *** betherly has joined #openstack-ironic | 23:27 | |
| *** rajinir has joined #openstack-ironic | 23:27 | |
| *** gregwork has joined #openstack-ironic | 23:27 | |
| *** marlinc has quit IRC | 23:27 | |
| *** lennyb has quit IRC | 23:27 | |
| *** lennyb has joined #openstack-ironic | 23:28 | |
| *** marlinc has joined #openstack-ironic | 23:28 | |
| openstackgerrit | Merged openstack/ironic master: Add XClarity Driver https://review.openstack.org/519427 | 23:30 |
| openstackgerrit | Tony Breeds proposed openstack/ironic-python-agent master: Use lshw in place of dmidecode for the default hardware manager https://review.openstack.org/501799 | 23:42 |
| *** penick has quit IRC | 23:46 | |
| *** srart has quit IRC | 23:49 | |
| *** openstack has quit IRC | 23:55 | |
| *** openstack has joined #openstack-ironic | 23:57 | |
| *** ChanServ sets mode: +o openstack | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!