| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-10 | |||
| 15:06:08 | ioni | no warning, nothing related to that | |
| 15:06:42 | mriedem | this doesn't work for me either https://gist.github.com/mriedem/d20978819dbbc28c77f89e118fd4f52f | |
| 15:06:52 | mriedem | i get oslo_config.cfg.NoSuchOptError: no such option notify_on_state_change in group [DEFAULT] | |
| 15:06:55 | sfinucan | mriedem: AFAIK the deprecated_group and deprecated_name properties only affects reading from config files. When accessing it programatically (cfg.CONF) you need to use the current group and name | |
| 15:06:57 | mriedem | might be a problem in the unit tests | |
| 15:07:04 | mriedem | sfinucan: ah | |
| 15:07:29 | sfinucan | which, imo, is the correct behavior | |
| 15:07:32 | mriedem | ioni: are you able to change nova/conf/notifications.py and change deprecated_group='DEFAULT', restart and see if that works? | |
| 15:07:48 | mriedem | i don't have a devstack in front of me to recreate | |
| 15:08:06 | ioni | ok, wait | |
| 15:08:21 | dims_ | mriedem : sfinucan : we just merged something that may help in the future - https://review.openstack.org/#/c/357987/ | |
| 15:10:38 | ioni | mriedem, deprecated_group='default' this line? | |
| 15:10:48 | mriedem | ioni: yeah | |
| 15:10:51 | ioni | right? | |
| 15:10:51 | ioni | with capital DEFAULT | |
| 15:10:53 | mriedem | yes | |
| 15:11:37 | ioni | 2017-03-10 17:11:26.849 14528 WARNING oslo_config.cfg [req-13263cac-0dca-42bd-9c0a-d2e3cbab71a6 - - - - -] Option "notify_on_state_change" from group "DEFAULT" is deprecated. Use option "notify_on_state_change" from group "notifications". | |
| 15:11:39 | ioni | now it works | |
| 15:12:15 | mriedem | ioni: ok, crap, sorry - can you report a bug against nova? | |
| 15:12:20 | ioni | should i submit a patch ? | |
| 15:12:24 | mriedem | https://bugs.launchpad.net/nova | |
| 15:12:40 | mriedem | ioni: sure, i see we have the same mistake for some console options | |
| 15:12:54 | mriedem | ioni: report a bug, submit a patch and then we can work on reviewing and backporting to ocata | |
| 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 | ioni | mriedem, report here: https://bugs.launchpad.net/nova/+bug/1671847 | |
| 15:17:27 | openstack | Launchpad bug 1671847 in OpenStack Compute (nova) "Incorect set deprecated flag for notify_on_state_change" [Undecided,New] | |
| 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 | 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:20 | ioni | mriedem, i just saw your comment | |
| 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 | https://review.openstack.org/#/c/436474/ | |
| 15:30:20 | sfinucan | lyarwood: The tools are found in openstack-docs-tools, but the output is kept in openstack-manuals | |
| 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 ;) | |