Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-07
21:14:42 openstackgerrit Matt Riedemann proposed openstack/nova master: Drop support for the Cinder v2 API https://review.openstack.org/501874
21:15:10 smcginnis Woot! ^
22:05:01 openstackgerrit Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247
22:05:02 openstackgerrit Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137
22:09:09 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323
22:09:10 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
22:09:11 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
22:19:04 openstackgerrit Merged openstack/nova master: Transform keypair.delete notification https://review.openstack.org/463946
22:26:04 openstackgerrit Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247
22:27:59 cdent efried: I don’t think you are being sufficiently critical of yourself in your comments
22:28:32 efried cdent I know what I mean, and I'm VICIOUS.
22:28:42 cdent :)
22:38:28 openstackgerrit Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
22:39:35 openstackgerrit Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057
22:51:09 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323
22:51:10 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
22:51:10 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
#openstack-nova - 2017-09-08
00:46:09 openstackgerrit Merged openstack/nova stable/pike: Allow setting up multiple cells in the base TestCase https://review.openstack.org/501818
00:48:57 openstackgerrit Merged openstack/nova stable/pike: Make ConductorTaskTestCase run with 2 cells https://review.openstack.org/501819
00:50:05 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/489031
00:52:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
00:52:52 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - List/Show all server migration types https://review.openstack.org/430839
01:53:21 openstackgerrit liusheng proposed openstack/nova master: Correct the out-of-date flavor doc reference https://review.openstack.org/501921
02:17:19 openstackgerrit Merged openstack/nova master: Fix broken URLs https://review.openstack.org/501402
02:35:39 openstackgerrit Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023
02:52:20 openstackgerrit Philip Choi proposed openstack/nova master: Add warning message for cache_utils connection failure https://review.openstack.org/501928
02:56:56 openstackgerrit Philip Choi proposed openstack/nova master: Add warning message for cache_utils connection failure https://review.openstack.org/501928
03:04:17 openstackgerrit kangyufei proposed openstack/nova master: Typo error about help resource_classes.inc https://review.openstack.org/501930
03:25:49 mnaser https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/pike+topic:bug/1715462 -- everything is finally passing, appreciate a review from anyone on the stable team to get this pike bug fixed :>
03:54:17 openstackgerrit Michael Still proposed openstack/nova master: Remove two testing stubs which aren't really needed. https://review.openstack.org/501939
03:57:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
04:31:27 openstackgerrit Michael Still proposed openstack/nova master: Cleanup unused get_iscsi_initiator. https://review.openstack.org/501944
05:34:12 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
05:59:46 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - Enable cold migration with target host https://review.openstack.org/406707
06:33:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
06:49:34 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
08:19:06 jsheeren hi all, i'm curious as to why no special characaters like a dot (.) are allowed for the ssh keypair names
08:19:24 jsheeren if i check the code: safe_chars = "_- " + string.digits + string.ascii_letters
08:19:46 jsheeren does anyone have an idea as to why?
08:29:37 openstackgerrit Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
08:34:33 bauwser jsheeren: here, I'm just guessing, but if you look at https://github.com/openstack/nova/commit/ec0a65d81fd11d26be77b45827a4dd7c77711675 it seems that's because AWS was only accepting that
08:35:11 bauwser since we did cut the EC2 API previously, I don't think it's still a problem
08:49:43 ygl hi all
08:49:51 ygl i need some help
08:49:53 openstackgerrit Rick van de Loo proposed openstack/nova master: Allow dots in keypair filenames https://review.openstack.org/502000
08:59:24 jsheeren bauwser: thanks, Rick proposed a patch that fixes our issue. :-)
08:59:53 bauwser jsheeren: you can't do that directly, you'll require an API microversion
09:00:22 jsheeren ah
09:00:50 bauwser jsheeren: I'll provide some comments to that, but the fix will require a Nova spec that will desribe the API change as we need to keep a consistent behaviour for our API
09:01:01 jsheeren ok, thanks
09:35:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix the ocata config-reference URLs https://review.openstack.org/502017
10:43:40 stephenfin bauwser, gibi: Could you folks look at https://review.openstack.org/#/c/501342/, please?
10:54:41 gibi stephenfin: looking...
11:06:00 openstackgerrit Chenghui Yu proposed openstack/nova-specs master: Fix usage of method isotime https://review.openstack.org/502043
11:37:12 gibi stephenfin: I left some comments in https://review.openstack.org/#/c/501342/
11:43:32 openstackgerrit Balazs Gibizer proposed openstack/nova stable/pike: reno: note that custom resources are not supported https://review.openstack.org/500521
12:15:46 openstackgerrit Elod Illes proposed openstack/nova master: Transform aggregate.add_host notification https://review.openstack.org/396210
12:23:38 openstackgerrit Lajos Katona proposed openstack/nova master: Extend ServerMovingTests with custom resources https://review.openstack.org/497399
12:58:35 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transform servergroup.delete notification https://review.openstack.org/483902
13:01:21 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transform servergroup.delete notification https://review.openstack.org/483902
13:09:08 mriedem sdague: you want to take a gander at this osc-placement plugin post_test_hook change to run functional dsvm tests? https://review.openstack.org/#/c/452122/ - it's modeled after the one in novaclient, with a couple of odd differences for env vars which i don't quite follow, but don't seem to be a problem either
13:09:57 sdague mriedem: +A
13:10:06 mriedem are you core on project-config?
13:10:08 sdague yes
13:10:15 mriedem well do i have a deal for you
13:10:22 sdague :)
13:10:24 mriedem https://review.openstack.org/#/c/501884/
13:10:31 mriedem moves that functional job from experimental to non-voting check
13:10:51 sdague cool
13:10:59 sdague now, we just need zuul to come back online
13:11:15 openstackgerrit Matthew Booth proposed openstack/nova master: fake_notifier: Refactor wait_for_versioned_notification https://review.openstack.org/489637
13:16:58 openstackgerrit Matt Riedemann proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882
13:20:01 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721
13:20:01 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Split flavors docs into admin and user guides https://review.openstack.org/501342
13:20:02 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Add documentation for cpu_realtime, cpu_realtime_mask https://review.openstack.org/502056
13:21:23 stephenfin gibi: All good point. Addressed above ^
13:22:00 stephenfin tbh, that whole section needs a thorough looking over. I'll do it after the PTG but this is a good start
13:27:59 gibi stephenfin: totally agree. this is a good start so I'm +2
13:29:00 stephenfin Excellent, thanks gibi :)
13:37:09 mriedem johnthetubaguy: policy question on scope checking,
13:37:24 mriedem this is something that continues to confuse me about whether or not non-admins can see extra specs for a flavor
13:37:46 mriedem default policy is admin or owner
13:37:46 mriedem "os_compute_api:os-flavor-extra-specs:index": "rule:admin_or_owner"
13:38:09 mriedem but when we check the scope, we don't pass the flavor in, and the flavor doesn't have a user_id/project_id (owner) anyway
13:38:38 mriedem https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/flavors_extraspecs.py#L53
13:38:57 mriedem so it's unclear to me if a non-admin can by default view flavor extra specs
13:43:01 johnthetubaguy mriedem: just reading back your note
13:44:24 johnthetubaguy mriedem: we have a default target of {project_id: context.project_id, user_id:context.user_id}, basically making the scope check a no op
13:45:15 johnthetubaguy mriedem: well my no-op I mean always returns true
13:45:21 johnthetubaguy s/my/by/
13:45:41 mriedem ok so basically anyone can view flavor extra specs by default
13:47:21 johnthetubaguy mriedem: it seems that way, I am just checking the DB code
13:48:42 johnthetubaguy mriedem: it looks like we join extra_specs for everyone here: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L5017
13:49:15 mriedem yup
13:49:26 mriedem well, if the flavor is in a cell still, but yeah
13:49:44 mriedem https://github.com/openstack/nova/blob/master/nova/objects/flavor.py#L295 is the new hotness
13:50:23 johnthetubaguy mriedem: oh right, we only hit the other for the unmigrated (probably no one) case
13:50:52 mriedem yeah we can remove that old compat now https://review.openstack.org/#/c/460377/
13:50:58 mriedem but anyway, that answers my question
13:51:08 johnthetubaguy I see what you did there

Earlier   Later