Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-10
05:12:38 jianghuaw the database configuration in nova.conf is:
05:12:39 jianghuaw [database]
05:12:39 jianghuaw connection = mysql+pymysql://root:citrix@127.0.0.1/nova_cell0?charset=utf8
05:12:39 jianghuaw [api_database]
05:12:39 jianghuaw connection = mysql+pymysql://root:citrix@127.0.0.1/nova_api?charset=utf8
05:14:32 jianghuaw mriedem, the nova_api database connection settings seems correct.
05:24:32 openstackgerrit Alex Xu proposed openstack/nova master: placement: ensure RP maps to those RPs that share with it https://review.openstack.org/480379
06:22:19 openstackgerrit Alex Xu proposed openstack/nova master: placement: ensure RP maps to those RPs that share with it https://review.openstack.org/480379
06:43:31 openstackgerrit Alex Xu proposed openstack/nova master: placement: ensure RP maps to those RPs that share with it https://review.openstack.org/480379
06:43:31 openstackgerrit Alex Xu proposed openstack/nova master: placement: avoid to return duplicated alloc_reqs when no sharing rp https://review.openstack.org/492395
06:58:58 openstackgerrit Merged openstack/nova master: [placement] Add api-ref for usages https://review.openstack.org/480563
07:03:47 openstackgerrit Merged openstack/nova master: Raise NoValidHost if no allocation candidates https://review.openstack.org/491491
07:04:33 openstackgerrit Alex Xu proposed openstack/nova master: placement: ensure RP maps to those RPs that share with it https://review.openstack.org/480379
07:14:33 openstackgerrit Alex Xu proposed openstack/nova master: placement: ensure RP maps to those RPs that share with it https://review.openstack.org/480379
07:14:46 alex_xu emm...finally
07:20:07 openstackgerrit jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482013
08:13:34 openstackgerrit Balazs Gibizer proposed openstack/nova master: replace chance with filter scheduler in func tests https://review.openstack.org/491529
08:19:01 gibi good morning
08:20:15 maciejjozefczyk hello
08:40:00 bauzas morning Novaists
09:05:37 ratailor alex_xu, you around ?
09:05:44 alex_xu ratailor: yea
09:06:17 ratailor alex_xu, regarding this bug https://bugs.launchpad.net/nova/+bug/1709260
09:06:20 openstack Launchpad bug 1709260 in OpenStack Compute (nova) "Addition of host to host-aggregate should be case -sensitive" [Low,Confirmed] - Assigned to Rajesh Tailor (ratailor)
09:06:52 ratailor alex_xu, I have done something like this.. https://github.com/openstack/nova/commit/402b3abf990d08d2af8331079d36a92d84d84b80#diff-342854d9b83486a3db6189bc21be6266L1026
09:07:24 ratailor alex_xu, for host_mappings table in api_db.
09:08:03 ratailor alex_xu, I was able to pass through this code block. https://github.com/openstack/nova/blob/master/nova/objects/host_mapping.py#L91-L93
09:09:21 alex_xu ratailor: not sure your first link is correct, that is pointed to a empty migration check
09:09:47 alex_xu ratailor: oh, sorry, I saw the entire patch now
09:10:19 alex_xu ratailor: I'm not sure we need to convert the table to case-sensitive
09:10:24 ratailor alex_xu, but again I got stuck in this code block. https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L571-L575
09:10:29 alex_xu at least, the hostname should be case-insensitive
09:11:40 ratailor alex_xu, in that case, do I need to look for alternative approach if possible ?
09:14:04 alex_xu ratailor: I guess it won't work after you change the collation of the table? since the in-memory copy of host<->aggregation mapping in the scheduler is still case-sensitive
09:16:03 alex_xu ratailor: ah, I see, you want to stop the api user input a hostname which differnet with the Db one
09:16:20 ratailor alex_xu, I also applied the collation logic to service table in nova_cell1 db, but here, case-insensitive check is getting passed.
09:16:47 alex_xu but the hostname should be case-insensitive? whatever the user input COMPUTE0.example.com or compute0.example.com, it should works
09:17:01 ratailor alex_xu, yes, I wan't to prevent from adding the case-insensitive host to host-aggregate.
09:17:42 ratailor but aggregates store the hostname as COMPUTE0.example.com, whereas in db its compute0.example.com
09:18:16 ratailor alex_xu, so that is what might be confusing scheduler.
09:20:05 alex_xu ratailor: there is a related bug https://bugs.launchpad.net/nova/+bug/1538011
09:20:07 openstack Launchpad bug 1538011 in OpenStack Compute (nova) "Metadata keys are not case sensitive" [Medium,In progress] - Assigned to Augustina Ragwitz (auggy)
09:20:53 ratailor alex_xu, Thanks! let me check it.
09:21:09 alex_xu ratailor: it would be great to check those two bugs, see the best way to fix
09:21:22 ratailor alex_xu, sure.
09:22:33 alex_xu ratailor: and there is a spec for that bug https://review.openstack.org/#/c/350843/, but didn't get merged
09:37:28 openstackgerrit Vladyslav Drok proposed openstack/nova master: [placement] Add api-ref for RP usages https://review.openstack.org/450105
09:51:35 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Add datapath type information to OVS vif objects https://review.openstack.org/474892
09:57:11 openstackgerrit Vladyslav Drok proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112
09:57:11 openstackgerrit Vladyslav Drok proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924
10:01:34 alex_xu cdent: bauzas this is ready for review https://review.openstack.org/#/q/topic:bug/1702420
10:01:58 cdent alex_xu: thanks alex, I’ve got that in my queue to look at soon. Was it hard to resolve?
10:02:14 bauzas alex_xu: thanks, will look to it asap
10:02:21 bauzas just have some internal paperwork to do
10:02:39 alex_xu cdent: after I pull some code into a sub-method, then thing gets a little better
10:02:55 cdent ah, yeah, good idea
10:04:07 maciejjozefczyk cdent: please add also this to your queue https://review.openstack.org/#/c/491808/
10:04:28 cdent maciejjozefczyk: already there
10:04:36 maciejjozefczyk cdent: :) thx
10:42:19 openstackgerrit Sean Dague proposed openstack/nova master: Clarify that vlan feature means nova-network support https://review.openstack.org/478551
10:54:00 openstackgerrit Merged openstack/nova master: Remove ram/disk sched filters from default list https://review.openstack.org/491854
10:57:37 openstackgerrit Merged openstack/nova master: Mark Chance and Caching schedulers as deprecated https://review.openstack.org/492210
11:10:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: replace chance with filter scheduler in func tests https://review.openstack.org/491529
11:12:57 openstackgerrit Merged openstack/python-novaclient master: Remove substitutions for command error msg https://review.openstack.org/490705
11:26:39 openstackgerrit Merged openstack/nova master: [placement] Avoid error log on 405 response https://review.openstack.org/490021
11:26:40 openstackgerrit Vladyslav Drok proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112
11:27:14 openstackgerrit Vladyslav Drok proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924
11:27:41 openstackgerrit Merged openstack/nova master: api-ref: fix security_groups response parameter in os-security-groups https://review.openstack.org/489274
11:28:19 openstackgerrit Merged openstack/nova master: api-ref: requested security groups are not applied to pre-existing ports https://review.openstack.org/489275
11:34:37 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091
11:39:04 openstackgerrit Merged openstack/nova master: Remove translation of log messages https://review.openstack.org/466637
11:39:48 maciejjozefczyk sdague: Could you check this one? https://review.openstack.org/#/c/491808/ please?
11:39:49 openstackgerrit Merged openstack/nova master: remove mox from unit/virt/vmwareapi/test_driver_api.py https://review.openstack.org/452128
11:41:22 sdague maciejjozefczyk: how does this handle the synchronization problem that now computes might be trying to delete the same instances at the same time?
11:41:39 sdague previously, by being host scoped, this was a sharded problem
11:46:29 openstackgerrit Merged openstack/nova master: imagebackend: cleanup constructor args to Rbd https://review.openstack.org/490499
11:47:13 openstackgerrit Merged openstack/nova master: Add policy granularity to the Flavors API https://review.openstack.org/449288
11:49:29 sdague ah, I see now
11:49:47 maciejjozefczyk sdague: The problem is about instance (which is deleted from nova side) is still running on compute A, but nova says that its deleted and it was on host B
11:50:00 sdague maciejjozefczyk: yeh, I get the problem
11:50:19 sdague I was just trying to make sure that this didn't make it so that multiple computes were trying to delete the same instance
11:50:48 sdague but _get_instances_on_driver should keep us sharded
11:51:06 maciejjozefczyk yes
11:52:13 maciejjozefczyk I'm going to work on patch to rollback migration if deletion of instance will be triggered, in near future
11:52:22 sdague cool
11:52:49 maciejjozefczyk but this fix for already 'lost' and 'working' zombiee instances i think should be in nova
11:53:30 maciejjozefczyk in my installation I have hundreds of them
11:59:23 sdague maciejjozefczyk: yep, +2 on this fix
12:00:34 maciejjozefczyk sdague: thx
12:06:29 openstackgerrit Alex Xu proposed openstack/nova master: placement: avoid returning duplicated alloc_reqs when no sharing rp https://review.openstack.org/492395
12:06:30 openstackgerrit Alex Xu proposed openstack/nova master: placement: ensure RP maps to those RPs that share with it https://review.openstack.org/480379
12:06:36 alex_xu cdent: thanks
12:06:46 openstackgerrit Ilya Popov proposed openstack/nova master: Tests: Add cleanup of 'instances' directory https://review.openstack.org/491589
12:08:42 sdague alex_xu: can I tempt you with doc patches? :)
12:08:56 sdague mostly I'd like to get the manuals stuff merged before I go on vacation next week
12:09:30 sdague https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/doc-migration
12:10:09 alex_xu sdague: yea, let me try
12:13:03 sdague alex_xu: thank you
12:20:24 openstackgerrit Chris Dent proposed openstack/nova master: placement: ensure RP maps to those RPs that share with it https://review.openstack.org/480379
12:54:14 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Reconnect volumes and encryptors during a hard reboot https://review.openstack.org/400384

Earlier   Later