| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-09-06 | |||
| 18:30:19 | mriedem | nicolasbock: w/o knowing your full situation, that's my first guess - let me know if that doesn't align with what you're dealing with | |
| 18:30:54 | nicolasbock | mriedem, Thanks. Is there a command line client for placement? | |
| 18:31:04 | nicolasbock | Or is that done directly through the API via curl? | |
| 18:31:23 | mriedem | that's the osc-placement plugin | |
| 18:31:26 | mriedem | in the docs i linked | |
| 18:31:37 | mriedem | https://docs.openstack.org/osc-placement/latest/ | |
| 18:31:54 | mriedem | https://pypi.org/project/osc-placement/ | |
| 18:32:52 | nicolasbock | Thanks! | |
| 19:02:47 | mriedem | so grenade hasn't been working in stein yet just fyi https://review.openstack.org/#/c/591594/ | |
| 19:03:04 | mriedem | good news is you can skip level from queens to stein! | |
| 19:03:10 | mriedem | FFU MISSION ACCOMPLISHED! | |
| 19:03:20 | jroll | congrats | |
| 19:15:53 | mriedem | mnaser: you've been summoned to https://review.openstack.org/#/c/600157/ | |
| 19:16:03 | mriedem | discussion about how extracted placement is going to deal with config | |
| 19:16:21 | mnaser | hi | |
| 19:16:29 | mriedem | right now, as written, deployers would need to copy/rename nova.conf to placement.conf *or* write a fresh placement.conf with the needed stuff that was in nova.conf before | |
| 19:16:43 | mriedem | there is no "if nova.conf: use it; else placement.conf" | |
| 19:17:00 | mriedem | we'll likely be doing the latter in grenade | |
| 19:18:43 | mnaser | well | |
| 19:18:49 | mnaser | we've always deprecated things | |
| 19:19:14 | mnaser | so i don't see why this would be any different | |
| 19:19:33 | mnaser | if anything this is way more critical than any other config disappearing | |
| 19:21:16 | mriedem | you mean as in deprecated but still rely on nova.conf in extracted placement, and drop support for placement using nova.conf in T? | |
| 19:22:00 | mnaser | mriedem: load in order of /etc/placement.conf => /etc/nova/nova.conf in S with a giant warning | |
| 19:22:01 | mnaser | drop that in T | |
| 19:22:33 | mriedem | i believe that's what jaypipes asked for on the review | |
| 19:23:36 | jaypipes | correct. | |
| 19:24:13 | mnaser | i really hope to try and loop in other deployment tools into this | |
| 19:24:15 | mnaser | i.e. tripleo | |
| 19:24:35 | mriedem | beyond config, the other major thing is obviously the db | |
| 19:25:08 | mriedem | if placement does use nova.conf in stein (use/supports as a bridge), it would rely on the [api_database]/connection config | |
| 19:25:57 | mnaser | yeah and also | |
| 19:26:00 | mriedem | looks like that code is still there | |
| 19:26:01 | mnaser | wasnt placement_database a thing | |
| 19:26:07 | mnaser | but i think that was removed cause it wasn't wired | |
| 19:26:09 | mriedem | in openstack/placement, to support that | |
| 19:26:11 | mnaser | i cant remember if it was fixed | |
| 19:26:22 | mriedem | CONF.placement_database is a thing in rocky | |
| 19:26:29 | mriedem | optional thing | |
| 19:26:31 | mnaser | ok so that's back | |
| 19:26:37 | mriedem | yes | |
| 19:26:42 | mnaser | so the logic to determine the db source would be useful there | |
| 19:26:48 | mriedem | if not set, it still uses the nova_api db | |
| 19:27:05 | mnaser | i remember some deployment tools actually creating a placement db | |
| 19:27:08 | mnaser | and pointing that stuff to it | |
| 19:27:10 | mriedem | https://github.com/openstack/placement/blob/master/placement/db_api.py#L27 | |
| 19:27:24 | mriedem | yeah i think i remember that, maybe kolla? | |
| 19:27:44 | mriedem | i do remember it triggering that we needed to remove some shit from the nova config that was missed when that was originally reverted | |
| 19:28:52 | mnaser | mriedem: i added a comment with my pov | |
| 19:31:59 | jaypipes | mriedem: again, my primary concern is the use of default values if /etc/placement.conf doesn't exist and how that would be quite difficult to diagnose if there were changed values in the /etc/nova.conf for placement-related stuffs. | |
| 19:32:50 | mnaser | yeah i can imagine that being so confusing | |
| 19:33:55 | mriedem | there's not actually a ton of configuration options for placement, mostly policy, db, and some other small things, | |
| 19:34:06 | mriedem | but yeah if your db is pointing at the wrong thing, i.e. nothing, you're f'ed | |
| 20:18:54 | efried | edmondsw: You'll be interested in this discussion from the nova mtg this morning: http://eavesdrop.openstack.org/meetings/nova/2018/nova.2018-09-06-14.00.log.html#l-120 | |
| 20:20:09 | efried | edmondsw: TL;DR: we should proceed as if cyborg ain't a thing, and worry about integrating with cyborg later as it matures. | |
| 20:20:41 | efried | edmondsw: Which means pretty much sticking with the direction of the spec(s) as written. | |
| 20:20:50 | efried | which is good timeline-wise :) | |
| 20:21:08 | edmondsw | that sounds good to me, assuming we'll be able to squeeze cyborg in when the time comes without reinventing too much | |
| 20:21:19 | edmondsw | s/assuming/hoping/ ? | |
| 20:21:53 | efried | yeah, that was my concern, but it's between that rock and the hard place of trying to anticipate $world-with-cyborg before that's a thing. | |
| 20:22:07 | edmondsw | yep | |
| 20:22:20 | edmondsw | I see many interesting PTG conversations in your future... | |
| 20:22:43 | edmondsw | wish I could be there | |
| 21:01:01 | gryf | +++++++++++++++++ | |
| 21:32:59 | mnaser | fun fact | |
| 21:33:02 | mnaser | console logs disappear on live migration | |
| 21:33:19 | mnaser | themoreyouknow.gif | |
| 21:47:37 | mriedem | you're welcome | |
| 21:53:59 | mriedem | so i thought artom changed direction on numa-aware live migration https://review.openstack.org/#/c/599587/ ? | |
| 21:54:04 | mriedem | because the re-proposed spec is the same old thing | |
| 21:54:21 | mriedem | stephenfin: ^ | |
| 21:54:31 | mriedem | btw, GET /artom is a 404 | |
| 21:54:33 | mriedem | wtf? | |
| 22:05:09 | openstackgerrit | Merged openstack/nova-specs master: Add serial numbers for local disks https://review.openstack.org/595247 | |
| #openstack-nova - 2018-09-07 | |||
| 00:32:25 | alex_xu | good morning nova | |
| 01:07:04 | openstackgerrit | Elancheran S proposed openstack/nova stable/pike: Add apply_cells to nova-manage https://review.openstack.org/599050 | |
| 01:18:24 | openstackgerrit | fupingxie proposed openstack/nova master: Add an example to add more pci devices in nova.conf https://review.openstack.org/592243 | |
| 02:08:16 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add live-migration-force-after-timeout-stein spec https://review.openstack.org/600608 | |
| 02:18:53 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Spec to add per instance timeout for LM - Stein https://review.openstack.org/600613 | |
| 03:01:26 | openstackgerrit | Merged openstack/nova stable/rocky: Fix a failure to format config sample https://review.openstack.org/600336 | |
| 03:09:08 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Detach and attach boot volumes - Stein https://review.openstack.org/600628 | |
| 03:30:30 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add changes-before filter https://review.openstack.org/591976 | |
| 05:38:46 | openstackgerrit | Merged openstack/nova master: Fix TypeError in nova-manage cell_v2 list_cells https://review.openstack.org/599861 | |
| 05:46:59 | openstackgerrit | fupingxie proposed openstack/nova master: Add an example to add more pci devices in nova.conf https://review.openstack.org/592243 | |
| 06:26:36 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Detach and attach boot volumes - Stein https://review.openstack.org/600628 | |
| 06:28:31 | openstackgerrit | fupingxie proposed openstack/nova master: Delete allocations for instances that have been moved to another node https://review.openstack.org/582899 | |
| 06:43:58 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add live-migration-force-after-timeout-stein spec https://review.openstack.org/600608 | |
| 06:44:13 | openstackgerrit | fupingxie proposed openstack/nova master: Delete allocations for instances that have been moved to another node https://review.openstack.org/582899 | |
| 06:53:20 | openstackgerrit | fupingxie proposed openstack/nova master: Delete allocations for instances that is not in this node https://review.openstack.org/582899 | |
| 06:55:06 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Spec to add per instance timeout for LM - Stein https://review.openstack.org/600613 | |
| 07:03:54 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Do not dump all instances in the scheduler https://review.openstack.org/598611 | |
| 07:04:48 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Do not dump all instances in the scheduler https://review.openstack.org/598611 | |
| 07:08:44 | artom | Actually, sahid, if you're awake already, you might know this. Can NUMA resources be claimed without using a MoveClaim context? | |
| 07:08:51 | openstackgerrit | Chen proposed openstack/nova master: doc: update info for hypervisors https://review.openstack.org/599554 | |
| 07:10:29 | sahid | artom: not sure what you mean? i thnk we have two ways to claim resources but you can implement a new one if necessary | |
| 07:11:20 | artom | sahid, so, for cold migration for instance, there's code in the conductor that essentially says, "with rt.cold_migration_claim: <do cold migration>" | |
| 07:11:35 | artom | And IIUC this claim context is how resources are claimed on the destination node | |
| 07:12:01 | artom | Is there a less opaque way of doing, for example by calling something like rt.claim_these_resources_on_the_compute_host? | |
| 07:12:28 | artom | I'm essentially trying to avoid having to understand claim contexts :D | |
| 07:15:00 | sahid | artom: well... i did not have yet finish my coffe but i would say, you will have to reuse moveclaim that to avoid rewriting everything, | |