| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-10 | |||
| 15:13:00 | mriedem | i'm going to do a stable/ocata release next week, we can get this fix in there | |
| 15:13:21 | mriedem | dims_: ^ seems like a bug in oslo.config if you can specify a deprecated_group that doesn't exist | |
| 15:13:49 | mriedem | maybe that's a chicken and egg, | |
| 15:13:58 | mriedem | since the deprecated group is no longer registered (except the DEFAULT case) | |
| 15:14:57 | mriedem | johnthetubaguy: were you going to re-propose a spec for pike for this? https://review.openstack.org/#/c/393888/ | |
| 15:15:10 | dims_ | mriedem : a bit distracted at the moment. bug please? | |
| 15:15:35 | mriedem | forthcoming, i'll point it out when it's up | |
| 15:16:30 | sfinucan | mriedem: Yeah, I reviewed that patch and missed that. Sorry :( | |
| 15:16:44 | dims_ | thanks mriedem ! | |
| 15:16:51 | sfinucan | I'll try see if we can fix this in oslo.config, if that's ok with you dims_ | |
| 15:17:27 | openstack | Launchpad bug 1671847 in OpenStack Compute (nova) "Incorect set deprecated flag for notify_on_state_change" [Undecided,New] | |
| 15:17:27 | ioni | mriedem, report here: https://bugs.launchpad.net/nova/+bug/1671847 | |
| 15:17:32 | ioni | mriedem, patch coming up | |
| 15:18:08 | sfinucan | tonyb, lyarwood: Could ye stick this on your review queues? https://review.openstack.org/#/c/444306/ | |
| 15:18:41 | sfinucan | I'm seeing the issue in openstack-manuals for stable/ocata still. Not a huge problem but it's very annoying | |
| 15:18:55 | cdent | leakypipes: is https://review.openstack.org/#/c/442718/ -w cuz the tests aren't done, or are you reconsidering the idea? (I'm gathering things for this week's updated) | |
| 15:19:52 | cdent | counting number of available mempages seems ... odd? | |
| 15:19:55 | mriedem | ioni: thanks | |
| 15:21:01 | mriedem | cdent: also, someone needs to re-propose https://review.openstack.org/#/c/386710/ for pike and update it to mention the discussion at the ptg about not supporting multiple parents, | |
| 15:21:20 | cdent | mriedem: ✔ | |
| 15:21:21 | mriedem | doesn't need great detail, but that seemed like a long whiteboarding session and if that was the outcome we should probably capture in the spec with the reason why | |
| 15:22:07 | lyarwood | sfinucan: ack, looking | |
| 15:23:14 | sfinucan | lyarwood: (y) | |
| 15:25:05 | openstackgerrit | Ionuț Bîru proposed openstack/nova master: Fix notify_on_state_change deprecation https://review.openstack.org/444350 | |
| 15:25:22 | ioni | mriedem, ^ | |
| 15:27:26 | dims_ | ioni : does it work if you just delete the line 33? (https://review.openstack.org/#/c/444350/1/nova/conf/notifications.py) | |
| 15:28:05 | ioni | testing now | |
| 15:28:11 | dims_ | thanks! | |
| 15:28:27 | mriedem | dims_: removing the deprecated_group line would break the alias right? | |
| 15:28:51 | mriedem | ioni: we should fix the other similar issues | |
| 15:28:56 | dims_ | should end up as 'DEFAULT' i think mriedem (need to check) | |
| 15:28:59 | mriedem | i pointed those out in the review (as did sfinucan) | |
| 15:29:03 | ioni | dims_, no, it doesn't work | |
| 15:29:20 | ioni | mriedem, i just saw your comment | |
| 15:29:20 | mriedem | dims_: if you don't have the deprecated_group kwarg, oslo.config doesn't know it came from a deprecated group and won't alias | |
| 15:29:21 | lyarwood | sfinucan: stupid question but where is the stable/ocata openstack-docs-tools build listed? | |
| 15:29:27 | ioni | mriedem, i'll resend | |
| 15:29:51 | mriedem | lyarwood: there isn't a stable branch for that http://git.openstack.org/cgit/openstack/openstack-doc-tools/ | |
| 15:29:58 | dims_ | ioni : mriedem : ack thanks | |
| 15:30:20 | sfinucan | lyarwood: The tools are found in openstack-docs-tools, but the output is kept in openstack-manuals | |
| 15:30:20 | sfinucan | https://review.openstack.org/#/c/436474/ | |
| 15:30:25 | lyarwood | mriedem / sfinucan ; right, so do we need https://review.openstack.org/#/c/444306/ in stable/ocata ? I assume not | |
| 15:31:30 | sfinucan | lyarwood: That tool checks out the nova code and uses a combo of oslo.config and Jinja2 templates to build those table files | |
| 15:31:49 | lyarwood | sfinucan: ah gotcha | |
| 15:32:40 | sfinucan | then dumps it into openstack-manuals project, which is where the stable/ocata branch will be (not openstack-doc-tools) | |
| 15:32:53 | sfinucan | yes, it *is* a somewhat convoluted tool chain :) | |
| 15:34:04 | lyarwood | sfinucan: ha, I was just looking for proof that we needed it, I'll add a note in the review, LGTM otherwise. | |
| 15:36:11 | ioni | mriedem can you help me a bit? | |
| 15:37:10 | ioni | i don't know how to send a new patch in the same review :D | |
| 15:41:17 | openstackgerrit | Ionuț Bîru proposed openstack/nova master: Correctly set up deprecation warning https://review.openstack.org/444357 | |
| 15:42:04 | ioni | sorry for that :( | |
| 15:42:13 | ioni | i didn't know how to send a new patch set | |
| 15:42:55 | sfinucan | ioni: No problem :) If you keep the 'Change-ID' line the same, it will submit a new patchset | |
| 15:43:15 | ioni | damn it | |
| 15:43:25 | sfinucan | it's fine though. Don't worry about it | |
| 15:44:17 | siva_krishnan | ioni: git review --amend should help as well | |
| 15:44:22 | sfinucan | ioni: +2d | |
| 15:44:39 | siva_krishnan | sorry git commit --amend | |
| 15:44:55 | ioni | well, i did a rebase | |
| 15:45:01 | ioni | from two commits that i made locally | |
| 15:45:13 | ioni | and i messed up i guess | |
| 15:45:40 | mriedem1 | artom: replied in https://review.openstack.org/#/c/393930/ - i think we talked about that at one point | |
| 15:45:49 | mriedem1 | artom: basically the various interface APIs in nova are a mess | |
| 15:45:57 | mriedem1 | and that's why i omitted nic tags from the spec | |
| 15:46:11 | mriedem | that's lazy on my part, i just didn't want to sort out the interface APIs first | |
| 15:46:33 | mriedem | ioni: i do git commit -a --amend | |
| 15:46:49 | mriedem | then git review to push it up | |
| 15:46:56 | ioni | mriedem, cool. i'll keep in mind | |
| 15:47:43 | leakypipes | cdent: oh, that's -w because I still need to submit a mini-spec or blueprint. | |
| 15:48:22 | cdent | leakypipes: ended up leaving it off this week's update | |
| 15:48:33 | sfinucan | johnthetubaguy, mriedem: noddy stuff, but I'd like to fix this in stable/ocata before you make the point release next week https://review.openstack.org/#/c/443086/ | |
| 15:48:54 | sfinucan | (AJaeger asked me to do so for openstack-manuals) | |
| 15:48:57 | artom | mriedem, I don't remember talking about it, but OK, your explanation makes sense | |
| 15:49:56 | mriedem | sfinucan: can you show me where those are broken today? | |
| 15:50:01 | mriedem | a bug would have been nice | |
| 15:50:10 | mriedem | if we're backporting | |
| 15:50:13 | leakypipes | cdent: totally fine :) mostly was just pushing it off my lappie so I could pull to my workstation ;) | |
| 15:50:15 | mriedem | plus links to see what the bug is would be nice | |
| 15:50:51 | cdent | leakypipes: I know how that can be. I left a comment on it as a sort of note to self to remember my confusion and sadness | |
| 15:51:04 | leakypipes | :) | |
| 15:51:28 | cdent | and with that I'm going for a walk, biab | |
| 15:51:34 | sfinucan | mriedem: https://docs.openstack.org/newton/config-reference/compute/config-options.html look for 'flat_network_bridge' | |
| 15:51:48 | sfinucan | or any of the giant text with grey bars on the left | |
| 15:52:25 | sfinucan | I didn't think formatting issues like that warranted a bug but I'll draw up one shortly | |
| 15:52:25 | mriedem | ok i see | |
| 15:52:39 | mriedem | sfinucan: well i just didn't see any examples in the commit message | |
| 15:52:45 | mriedem | also, this doesn't seem like a huge issue | |
| 15:53:01 | mriedem | and it's been like this for awhile already, like all of ocata | |
| 15:53:42 | sfinucan | mriedem: I can't include images in commit messages, and the issues won't exist once they're fixed | |
| 15:54:17 | sfinucan | openstack-manuals hasn't released the ocata docs yet, hence why I'm eager to fix it now | |
| 15:54:22 | mriedem | ok i guess i just won't ask for any context or examples | |
| 15:54:31 | mriedem | anyway, +2 | |
| 15:55:40 | sfinucan | mriedem: ta | |
| 15:59:53 | mriedem | ioni: https://review.openstack.org/#/c/444374/ | |
| 16:01:27 | ioni | mriedem, do i have to do something? :D | |
| 16:02:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Always set MTU when plugging devices https://review.openstack.org/408781 | |
| 16:03:52 | mriedem | ioni: nope | |
| 16:07:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Flatten functions https://review.openstack.org/367470 | |
| 16:07:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Standardize '_get_XXX_constraints' functions https://review.openstack.org/385071 | |
| 16:07:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Rename '_numa_get_constraints_XXX' functions https://review.openstack.org/385072 | |