| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-30 | |||
| 17:23:53 | dansmith | did we fix the leak due to evacuation or are we saying the fix for that is just to have the audit command? | |
| 17:24:05 | dansmith | obviously for things that happened before such a fix, audit could help | |
| 17:26:35 | bauzas | dansmith: nope, we haven't fixed it yet | |
| 17:26:50 | dansmith | okay I would be in favor of backporting *that* :) | |
| 17:26:53 | bauzas | the audit command just cleans the mess | |
| 17:28:00 | bauzas | anyway, I need a beer | |
| 17:49:36 | mriedem | https://review.opendev.org/#/c/678100/ | |
| 17:57:25 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] testing with force_legacy_port_binding workaround https://review.opendev.org/724387 | |
| 18:03:19 | melwitt | mriedem: want an e-r stat? https://review.opendev.org/724719 | |
| 18:04:18 | mriedem | bauzas: no just linking in bugs that are assigned to me | |
| 18:08:41 | mriedem | melwitt: commented | |
| 18:08:59 | mriedem | it's weird but the before and after on that query are on the same change, and in the new query they are all coming from the same file (n-cpu in the subnode) | |
| 18:09:02 | mriedem | is that expected? | |
| 18:09:15 | mriedem | was qemu or libvirt bumped in devstack on master? | |
| 18:11:17 | melwitt | mriedem: ah dangit ... I focused on the error log message and didn't notice the warning log is what's in the e-r query. so the query doesn't need an update. sorry :( | |
| 18:13:49 | openstackgerrit | sean mooney proposed openstack/nova master: silence amqp heartbeat warning https://review.opendev.org/724188 | |
| 18:13:50 | openstackgerrit | sean mooney proposed openstack/nova master: remove support of oslo.messagiang 9.8.0 warning message https://review.opendev.org/724725 | |
| 18:13:59 | melwitt | and since the bug was greyed out on http://status.openstack.org/elastic-recheck/#1840159 at the time, that reinforced my thinking that the query didn't pick it up. sigh | |
| 18:21:40 | mriedem | \o/ | |
| 18:33:54 | gmann | dansmith: gibi this is ready - https://review.opendev.org/#/c/723645/ | |
| 18:49:04 | dansmith | gmann: a couple more wording clarification tings | |
| 18:49:06 | dansmith | *things | |
| 19:18:18 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/723645 | |
| 19:18:21 | gmann | dansmith: updated ^^ | |
| 19:19:43 | dansmith | gmann: okay lemme just fix another thing in-line to avoid you having to do it | |
| 19:20:00 | gmann | dansmith: sure, thanks | |
| 19:20:56 | dansmith | gmann: is it "recommended" or "required" to enable enforce_scope if you have scope checking rules? | |
| 19:21:02 | dansmith | I thought required | |
| 19:21:22 | gmann | dansmith: this is separate flaf not scope one - enforce_new_defaults | |
| 19:21:24 | gmann | flag | |
| 19:21:42 | gmann | to switch to new default without overwriting the file | |
| 19:22:01 | dansmith | gmann: right, but if you have scope-checking rules and want to keep them, then you need enforce_scope=True right? | |
| 19:22:14 | dansmith | you're checking enforce_scope, but if that's disabled, you tell them to enable enforce_new_defaults | |
| 19:23:07 | gmann | dansmith: ok, so enforce_scope=True is required and (enforce_new_defaults=True OR file overwrite) to move to new things | |
| 19:23:24 | dansmith | right, I think you're telling them the wrong thing | |
| 19:23:33 | dansmith | let me comment for context and you can see if I'm right | |
| 19:23:39 | gmann | in that sentence i was targeting only file overwrite alternate via flag | |
| 19:24:00 | gmann | but i agree we can add enforce_scope=True also for full context | |
| 19:25:19 | dansmith | but the sentence is about keeping the new format | |
| 19:25:24 | dansmith | U cinnebted | |
| 19:25:26 | dansmith | whoa | |
| 19:25:28 | dansmith | "I commented" | |
| 19:33:26 | gmann | dansmith: replied, https://review.opendev.org/#/c/723645/11/nova/cmd/status.py@358 | |
| 19:33:38 | gmann | added scope and new flag way. | |
| 19:34:24 | dansmith | gmann: ++ | |
| 19:34:28 | gmann | my intention is they are aware of new flag enforce_new_defaults t o avoid legacy way of overwriting the file to new default | |
| 19:34:30 | dansmith | gmann: I will try really hard to +2 the next one | |
| 19:34:39 | gmann | ok, updating. | |
| 19:34:53 | dansmith | gmann: ack, I just think the message should include the error, and "how to fix the error" | |
| 19:40:54 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/723645 | |
| 19:41:17 | gmann | dansmith: ^^ updated | |
| 19:44:49 | dansmith | gmann: I think you forgot to remove the old sentence | |
| 19:45:03 | gmann | ohh :( | |
| 19:45:08 | dansmith | otherwise that looks good to me | |
| 19:46:05 | dansmith | melwitt: you around to help me put gmann out of his misery? | |
| 19:47:29 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add nova-status upgrade check and reno for policy new defaults https://review.opendev.org/723645 | |
| 19:47:52 | gmann | dansmith: done, sorry for that. | |
| 19:47:56 | dansmith | gmann: thanks for your patience with me, +2d | |
| 19:47:57 | dansmith | melwitt: ^ | |
| 19:48:10 | dansmith | I gotta stretch my legs, bbl | |
| 19:49:48 | gmann | dansmith: thanks | |
| 19:51:31 | melwitt | yeah I'll get it | |
| 20:13:27 | melwitt | gmann: I found some misspellings and grammar things and what seem to be a few bugs, but I also suggested some additional info. let me know what you think. if you agree with them, I can update and approve if you want. I know you have been through a lot of updates with the policy docs things | |
| 20:13:56 | gmann | melwitt: ok, checking.. | |
| 20:15:13 | melwitt | the main thing I'm worried about is if operators just enable enforce_scope = True when they see the status check, without realizing that means end users will have to request scoped tokens from keystone in order to access the formerly admin, now system scoped admin APIs. if I understand correctly | |
| 20:33:58 | gmann | melwitt: thanks. replied. i linked the new doc there which i can move on those line so that they have complete steps info. | |
| 20:34:24 | gmann | mentioning those in upgrade checks is too much as they need to do read-only roles things also. | |
| 20:34:39 | gmann | is that fine ? | |
| 20:35:20 | melwitt | I dunno ... will depend on how carefully the operators regard that message | |
| 20:36:04 | melwitt | we can try it, just let's be ready for when/if people miss the "you have to request scoped tokens" part | |
| 20:37:05 | melwitt | like blarg admin APIs no longer work! | |
| 20:37:33 | gmann | ok, let me add those lines and then doc link. | |
| 20:38:24 | melwitt | I could be wrong. let's just say I've been burned by not 100% overly clear reno/tool output ... nova-consoleauth *cough cough* | |
| 21:22:57 | gmann | melwitt: ^^ updated, please check if looks fine. linked the main doc as 'educate' in reno but in upgrade check msg i need to paste the complete link - https://review.opendev.org/#/c/723645/15 | |
| 21:26:52 | melwitt | gmann: thanks. weird it didn't post an update in channel | |
| 21:26:56 | melwitt | oh our gerritbot left | |
| 21:28:06 | gmann | melwitt: yeah i was also surprised initially and rechecking my local branch that i did submit or not :) | |
| 21:28:17 | melwitt | haha yeah | |
| 21:28:32 | melwitt | I mentioned it in #opendev | |
| 21:31:31 | melwitt | gmann: the updates look good. I will watch in zuul so I can sanity check the generated doc previews and then I'll approve it after that | |
| 21:31:52 | gmann | melwitt: sure, thanks | |
| 21:40:16 | openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: [Community goal] Update contributor documentation https://review.opendev.org/722392 | |
| 21:41:09 | gmann | working now \o/ | |
| 21:41:34 | gmann | sean-k-mooney: ^^ updated as per your comment on 722392 | |
| 21:41:49 | sean-k-mooney | yep looking now | |
| 21:42:20 | sean-k-mooney | omg you have a white space how coudl you :P https://review.opendev.org/#/c/722392/3/doc/source/contributor/contributing.rst@29 | |
| 21:46:07 | sean-k-mooney | gmann: if you have time to fix https://review.opendev.org/#/c/722392/3/doc/source/contributor/contributing.rst those nits im happy with the content | |
| 21:46:39 | openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: [Community goal] Update contributor documentation https://review.opendev.org/722392 | |
| 21:46:54 | gmann | sean-k-mooney: done | |
| 21:47:03 | sean-k-mooney | awsome | |
| 21:47:47 | sean-k-mooney | ah i see you have already rechecked the other patch | |
| 21:47:57 | sean-k-mooney | i proably got lost with the zuul restart | |
| 21:48:28 | gmann | yeah, i thought that was merged | |
| 21:48:42 | sean-k-mooney | ya same | |
| 21:48:51 | sean-k-mooney | i guess not but it should merge soon | |
| 21:49:18 | sean-k-mooney | then we can merge stephens cahnge which is blocked by that | |
| 21:51:14 | gmann | sean-k-mooney: seems we need to backport that on ussuri brach also - https://review.opendev.org/#/c/724712/ | |
| 21:52:14 | sean-k-mooney | ya that makes sense i dont think we cap the sphinx version in the upperconstratit to prevent the issue | |
| 21:53:35 | gmann | we missed to recheck the fix and ussuri release one merged first - https://review.opendev.org/#/c/723687/ | |
| 21:55:22 | sean-k-mooney | ya one it merges on master we can cherry pick it and fast appove | |