Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-30
20:04:49 mriedem artom: exactly
20:05:09 mriedem artom: the tempest guys stopped doing the deepcopy i think
20:05:10 mriedem for this very reason
20:05:23 mriedem oomichi: right? ^
20:06:52 artom Maybe I'll remove the network_type refactoring then and stick to copy-pasting
20:08:00 oomichi mriedem: yeah, you are right. gmann takes care of it on Tempest side
20:09:01 oomichi mriedem: that requrest came from sdague and mtreinish because the deepcopy of schema seems unreadable to know whole schema on each microversion
20:17:17 mriedem hmm, what's a better place for the nova-status CLI in the release notes, features or upgrades?
20:17:31 mriedem i assume the upgrades section gets more reads
20:18:46 dansmith yeah, probably
20:18:48 dansmith hopefully anyway
20:19:11 dansmith the placement patches got reset so they're still in the running
20:19:24 dansmith I'm going to wander off for a bit and will come back to check on them
20:20:10 openstackgerrit Artom Lifshitz proposed openstack/nova: Pass APIVersionRequest to extensions https://review.openstack.org/425876
20:20:11 openstackgerrit Artom Lifshitz proposed openstack/nova: Fix tag attribute disappearing in 2.33 and 2.37 https://review.openstack.org/424759
20:24:10 edleafe jaypipes: Found a test failure in https://review.openstack.org/#/c/404472/ due to an incomplete test I added. Do you want me to remove it, or are you working on anything else with that patch?
20:29:33 bauzas mriedem: I wrote about the nova-status in the prelude
20:30:00 bauzas mriedem: https://review.openstack.org/#/c/426864/
20:30:12 mriedem was coincidentally reading that now
20:30:28 jaypipes edleafe: please go ahead!
20:31:23 bauzas mriedem: about the nova-status CLI, I think we should provide a 'feature' section
20:31:49 bauzas mriedem: "upgrade" section is just for discussing about what to need to modify before upgrading
20:32:04 bauzas mriedem: but you could still add a separate section if you want too
20:32:19 mriedem bauzas: i assume operators are more likely to read the upgrades release notes first,
20:32:26 mriedem and we want them using nova-status prior to upgrading
20:32:34 mriedem so seems best to just have it in the upgrade section
20:32:38 bauzas mriedem: sure, but the upgrades section will be huge
20:32:42 mriedem i didn't really want to mention the exact same thing in both ugprades and features
20:32:47 bauzas mriedem: so that's why I provided the prelude
20:32:48 mriedem it already is...
20:33:03 bauzas mriedem: so I think operators would see that by the prelude
20:33:57 dtp anyone need anything? cup of coffee, etc
20:34:05 mriedem just had one thanks
20:34:14 mriedem bauzas: ok i guess i can rebase my reno on top of yours
20:34:51 mriedem bauzas: and your reno needs a rebase too because your change and dan's change below it are both adding something to the prelude about cells v2
20:35:23 bauzas mriedem: fine by me, which rebase?
20:35:58 mriedem your change isn't on top of the latest version of https://review.openstack.org/#/c/424737/
20:36:53 bauzas mriedem: okay
20:38:16 dtp ok. let me know if there's anything i can do
20:38:54 openstackgerrit Matt Riedemann proposed openstack/nova: Add release note for nova-status upgrade check CLI https://review.openstack.org/426911
20:43:56 mriedem bauzas: i suppose we still want to get something into nova-status for warning if CoreFilter/RamFilter/DiskFilter aren't in filter_scheduler.enabled_filters...
20:44:31 bauzas mriedem: mmm, it could be nice, agreed
20:44:40 bauzas mriedem: I can write that if you want
20:45:09 bauzas mriedem: you mean, adverting to modify the ratios then ?
20:45:24 mriedem yeah something along those lines,
20:45:31 mriedem (1) is the filter_scheduler being used,
20:45:43 mriedem (2) are any of those things NOT in enabled_filters
20:46:07 mriedem (3) for each filter that's not enabled, check it's related allocation ratio and see if it's the default value and if so, issue a warning
20:46:08 mriedem something likethat
20:46:22 bauzas yeah, like I wrote in the note
20:46:33 bauzas very good point
20:46:52 bauzas mriedem: just one point, where operators are running the CLI ?
20:47:14 mriedem we don't know yet
20:47:23 bauzas mriedem: they need to do that in not in a computer, right? rather close to the other services, I guess
20:47:26 mriedem as we haven't integrated nova-status into the CI system yet
20:47:40 mriedem well, it's really, give me your full config
20:47:57 mriedem so api db, main db, compute config, scheduler config, all of it
20:48:02 mriedem and probably run it in a venv or container
20:48:23 mriedem so you can install and run it in isolation from the newton deployment code (which doesn't have the CLI)
20:49:16 bauzas ok
20:49:19 mriedem i'm thinking about writing a wip patch to grenade to run nova-status in the from-netwon script, after we'd done our necessary setup for ocata
20:50:30 dansmith mriedem: did you abandon that devstack patch that sets the db host to the primary?
20:50:39 mriedem yeah
20:50:42 mriedem i can find it
20:51:06 mriedem dansmith: this? https://review.openstack.org/#/c/421399/
20:51:48 dansmith mriedem: yeah, can you un-abandon? I'll need that to start up a second cell on the subnode
20:52:48 mriedem done
20:53:03 dansmith thanks
20:53:05 mriedem this was my patch for creating a 2nd cell https://review.openstack.org/#/c/420976/
20:53:08 mriedem which is out of date now
20:53:24 mriedem which also needed https://review.openstack.org/#/c/420973/
20:53:50 openstackgerrit Ed Leafe proposed openstack/nova: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472
20:53:57 dansmith ah, okay, I hadn't seen that one
20:54:18 edleafe jaypipes: cdent: johnthetubaguy: ^^ fixed the functional test
20:55:05 jaypipes edleafe: danke
20:55:11 bauzas mriedem: dansmith: FYI, merging notes with both prelude sections are most of the time very difficult because they could be misgenerated
20:55:43 bauzas that's why we add the ">" section header
20:55:50 mriedem bauzas: so i guess take the prelude out of dan's and move it into yours
20:56:04 dansmith bauzas: okay, I added prelude text at mriedem's request
20:56:09 dansmith but yeah, feel free to take it
20:56:23 bauzas okay, just wanted to ask you folks before
20:56:34 bauzas and explain why reno is giving you -1 on its job
20:56:46 bauzas (well, guessing why)
20:57:15 bauzas yeah that http://logs.openstack.org/37/424737/10/check/gate-nova-releasenotes/20a3d9a/console.html#_2017-01-30_18_28_37_031390
20:57:50 dansmith ack
20:59:13 bauzas dansmith: I'll push a new rev by deleting your prelude section and putting it in my patch
20:59:49 dansmith bauzas: s/deleting/stealing/
20:59:50 dansmith sure
21:00:12 dansmith bauzas: you keep stealing american work and trump will put france on "the list"
21:00:30 bauzas dansmith: yeah, I'm stealing a job
21:00:44 bauzas but
21:01:06 bauzas our French beauty queen won a Trump meetup, so...
21:01:12 dansmith and it's after 1700 in france, so you're a criminal too!
21:01:22 bauzas poor me
21:02:05 cburgess lol
21:06:09 diana_clarke mriedem: Sorry, I meant linking to http://docs.openstack.org/developer/nova/cells.html for more information. And my internet keeps dropping on me, so I may have missed your response.
21:11:38 openstackgerrit Sylvain Bauza proposed openstack/nova: Collected release notes for Ocata CellsV2 https://review.openstack.org/424737
21:11:39 openstackgerrit Sylvain Bauza proposed openstack/nova: Add prelude section for Ocata https://review.openstack.org/426864
21:12:14 bauzas there
21:12:16 mriedem diana_clarke: sorry i'm not following

Earlier   Later