Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-16
13:36:05 openstackgerrit Yikun Jiang proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904
13:36:05 openstackgerrit Yikun Jiang proposed openstack/nova master: Add migration_get_by_uuid in db api. https://review.openstack.org/511421
13:36:06 openstackgerrit Yikun Jiang proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils https://review.openstack.org/518871
13:36:06 fallenour Quick relevant dev question though, currently working with nova to build a self healing openstack, whats the best way recommended for integrating into the oslo system for custom rpc messages?
13:36:06 openstackgerrit Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644
13:36:07 openstackgerrit Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
13:36:07 openstackgerrit Yikun Jiang proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273
13:37:14 gibi fallenour: do you mean you would like to use oslo.messaging to send/receive messages ?
13:38:22 fallenour gibi: very much so
13:38:22 bauzas jianghuaw_: I'd love to look at https://review.openstack.org/#/c/512580 and relate it with https://review.openstack.org/#/c/514763/
13:38:32 bauzas s/relate/compare
13:39:08 fallenour gibi: btw, nova is absolutely mind shatteringly awesome, thanks for all the incredible work you guys do.
13:40:07 jianghuaw_ bauzas, thanks
13:40:49 gibi fallenour: there is a very simple messing ending example in vitrage: https://github.com/openstack/vitrage/blob/master/vitrage/datasources/zabbix/auxiliary/zabbix_vitrage.py I used this to learn how to send arbitrary stuff
13:41:13 gibi s/messing ending/message sending/.
13:41:29 fallenour gibi: If I wanted to study the messages, especially the ones generated by nova to the cloud controller, where would I go to read those/ how would i read those?
13:43:16 gibi fallenour: I think you can turn on logging in rabbit (or whatever message queue your deployment uses) But also I think oslo.messaging logs the message if on debug level
13:51:16 openstackgerrit Maciej Jozefczyk proposed openstack/nova master: Update resources once in update_available_resource https://review.openstack.org/520024
14:01:07 mriedem nova meeting happening in #openstack-meeting
14:01:18 mriedem surprise time changes
14:11:55 openstackgerrit Elod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification https://review.openstack.org/508506
14:12:21 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.56 - Add trusted_image_certificates https://review.openstack.org/500396
14:19:05 openstackgerrit Claudiu Belu proposed openstack/nova master: hyper-v: Do not allow instances with pinned CPUs to spawn https://review.openstack.org/424116
14:20:05 mriedem moshele: https://review.openstack.org/#/c/482640/ is failing tests
14:20:27 moshele mriedem: yes it need to os-brick changes
14:21:02 mriedem oh missing the actual initiator connector, yeah
14:21:06 mriedem i see the cinder change has a -2 on it?
14:22:21 ArchiFleKs Hi, I'm having some issue with the diskfilter scheduler, how does it work when using NFS shared storage for instances and should it be always at the end ?
14:22:41 mriedem ArchiFleKs: you can't use the DiskFilter with shared storage
14:23:02 openstackgerrit Claudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM https://review.openstack.org/478943
14:23:02 ArchiFleKs mriedem: neither the aggregatediskfilter ?
14:23:13 mriedem ArchiFleKs: i believe so
14:24:06 ArchiFleKs mriedem: thanks :)
14:24:50 mriedem bauzas: correct? ^
14:25:00 mriedem if so, we should add a warning to the docs https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html?highlight=diskfilter#diskfilter
14:25:13 mriedem i'm pretty sure DiskFilter doesn't work with shared storage pools though
14:25:21 bauzas mriedem: well, something like that
14:25:36 bauzas mriedem: I need to go find my daughters tho
14:25:42 mriedem ok
14:25:44 mriedem jaypipes: ^?
14:25:45 bauzas ArchiFleKs: I know you dude, so ping me off
14:25:50 mriedem johnthetubaguy: ^
14:25:57 moshele mriedem: thanks for approving the bp
14:26:05 tssurya jaypipes : would you know any tricks in SQL to get the resulting record list from the delete statement ? (https://review.openstack.org/#/c/515034/4/nova/db/sqlalchemy/api.py@6591)
14:26:49 johnthetubaguy mriedem: can I help?
14:27:29 jaypipes tssurya: there is no result from a DELETE statement. unless you're referring to the number of records deleted?
14:28:07 jaypipes mriedem: DiskFilter only looks at local_gb_used/local_gb AFAIK
14:28:13 openstackgerrit Claudiu Belu proposed openstack/nova master: doc: Adds Hyper-V PCI passthrough details https://review.openstack.org/510467
14:28:16 jaypipes mriedem: which obviously isn't placement...
14:28:25 openstackgerrit Hamdy Khader proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640
14:28:36 tssurya jaypipes : yes well we know that there is only the number of records that we get, but we were wondering if there was a way to get the records too ?
14:30:38 jaypipes tssurya: not unless you do a SELECT first, then DELETE.
14:30:49 jaypipes tssurya: in the same read-consistent transaction of course.
14:31:08 tssurya jaypipes : we have some integrity issues with the list of records deleted and inserted during archiving, so I was looking for an efficient way to do this.
14:31:17 tssurya jaypipes : ok! thanks :)
14:31:33 jaypipes tssurya: np :)
14:33:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (usage) https://review.openstack.org/520603
14:40:58 openstackgerrit Hamdy Khader proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640
14:43:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (trait) https://review.openstack.org/520605
14:47:44 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633
14:47:44 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs https://review.openstack.org/519617
14:47:45 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982
14:49:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (aggregate) https://review.openstack.org/520608
14:55:32 mriedem johnthetubaguy: jaypipes: my assertion was that the DiskFilter won't work with shared storage on the computes
14:55:39 mriedem so shouldn't be enabled if you're using NFS, Ceph, etc
14:56:00 johnthetubaguy its kinda useless in that case, yeah
14:56:12 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (resource_class) https://review.openstack.org/520611
15:04:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (inventory) https://review.openstack.org/520613
15:08:28 mriedem pretty simple novaclient change to add support for microversion 2.54 https://review.openstack.org/#/c/519572/
15:17:57 openstackgerrit Ed Leafe proposed openstack/nova master: Refactor the code to check for sufficient hosts https://review.openstack.org/520242
15:17:57 openstackgerrit Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239
15:17:58 openstackgerrit Ed Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts https://review.openstack.org/510159
15:17:58 openstackgerrit Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854
15:17:59 openstackgerrit Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357
15:17:59 openstackgerrit Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/516707
15:18:00 openstackgerrit Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358
15:21:38 mriedem yikun: https://review.openstack.org/#/c/515640/
15:22:39 jianghuaw_ johnthetubaguy, hi John. Could you help to look at this XenAPI fix? https://review.openstack.org/#/c/514902/
15:24:07 jianghuaw_ johnthetubaguy, I think that's a straightforward fix.
15:29:46 mriedem edleafe: are you going to make the changes in https://review.openstack.org/#/c/520242/ and fix the tests on the change after it? i'd like to hit those today while i'm still on the trail
15:29:52 mriedem the alternate hosts trail
15:29:53 mriedem of blood
15:33:50 edleafe mriedem: you mean the >= change?
15:34:17 edleafe if we ever select more hosts than requested, we have much bigger problems
15:35:11 mriedem ok so leave that,
15:35:15 mriedem but there are the typos and unrelated change
15:35:18 mriedem easy fixes
15:35:33 mriedem i figured you'd hit those while you cleanup the test failures in the next patch above
15:36:23 edleafe mriedem: you did see that I pushed an update to the series like 20 minutes ago?
15:37:15 mriedem orly
15:37:17 mriedem i didn't :)
15:37:51 edleafe :)
15:39:01 mriedem ok, one little thing in the test
15:42:04 johnthetubaguy jianghuaw_: been meaning to look at that, its on my list
15:43:07 openstackgerrit Ed Leafe proposed openstack/nova master: Refactor the code to check for sufficient hosts https://review.openstack.org/520242
15:43:15 edleafe mriedem: ^ added
15:44:42 mriedem thanks, lgtm
15:44:51 mriedem jaypipes: bauzas: you should review https://review.openstack.org/#/c/520242/ as well
15:44:58 edleafe mriedem: the series is blocked because you (plural) wanted to verify the whole process before beginning to commit any of the series. Is it possible to get such a review so that we can begin to merge some of these lower patches?
15:45:43 mriedem jaypipes: bauzas: review note, this change is because the next one in the series is going to be returning Selection objects from FilterScheduler._schedule to FilterScheduler.select_destinations, where that HostState.updated = None was done, which won't work with a Selection object

Earlier   Later