Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-06-23
06:11:17 artom alex_xu, how *are* schemas tested?
06:11:50 alex_xu artom: np, some unittest ensure tag are accepted, and some invalid tag can be rejected
06:13:47 artom alex_xu, ah, so it'd be an API test where we mock out essentially everything, and just do a couple of requests with valid and invalid tags, and ensure it either accepts or fails schema validation
06:14:08 alex_xu artom: yea
06:14:09 artom I had understood you as a specific unit test for the schema
06:14:24 artom As in, from schemas import create_v249; assert some stuff
06:14:32 alex_xu artom: sorry, I didn't describe very clearly
06:14:44 artom alex_xu, no worries - I tend to interpret things too literally :)
06:15:20 alex_xu artom: :)
06:18:52 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
06:57:25 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225
07:16:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
07:56:02 openstackgerrit Jiri Suchomel proposed openstack/nova master: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928
08:01:15 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568
08:02:53 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
08:09:49 openstackgerrit Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
09:33:39 openstackgerrit Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
09:44:43 artom alex_xu, still around?
09:46:10 artom Actually, wait
09:46:23 artom Typing out my message to you made me think of something else I can try
09:53:08 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
09:53:22 openstackgerrit Andrey Volkov proposed openstack/nova master: PoC: Select PCI devices with distinct tag values https://review.openstack.org/448008
09:55:23 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225
10:05:12 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622
10:05:15 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387
10:05:24 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082
10:05:40 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941
10:05:42 openstackgerrit Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
10:17:00 openstackgerrit falseuser proposed openstack/nova master: Add img_linked_clone support in libvirt https://review.openstack.org/473362
11:08:48 openstackgerrit Maciej Kucia proposed openstack/nova master: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/382853
12:39:51 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285
13:10:45 openstackgerrit Sylvain Bauza proposed openstack/nova master: Destroy the ReqSpec object when deleting the instance https://review.openstack.org/391060
13:21:59 openstackgerrit Pavel Gluschak (scsnow) proposed openstack/nova master: VStorage: changed default log path https://review.openstack.org/458557
13:27:39 leakypipes dansmith: easy one for you: https://review.openstack.org/#/c/419114/
13:30:04 bauzas snap, holy Friday
13:30:19 bauwser leakypipes: so, I'm back
13:31:04 bauwser leakypipes: tbc, I'm a bit concerned by all the things we have in plate for placement
13:31:09 bauwser for Pike I mean
13:32:01 bauwser leakypipes: so, I just want to make sure I'm not blocking you with my -1 for a few points with https://review.openstack.org/#/c/471927/
13:32:53 leakypipes bauwser: that's just the spec...
13:33:09 bauwser of course
13:33:16 leakypipes bauwser: I'm mostly focused on the code in the placement-allocation-requests patch series.
13:33:32 bauwser that's something I'm reviewing now
13:34:04 leakypipes cool, thank you sir
13:34:29 bauwser edleafe: so I guess you'll rebase placement-claims on top of leakypipes's branch ?
13:35:03 bauwser leakypipes: FWIW, saw 2 patches merged by June 20, so I'm looking at them first
13:56:52 openstackgerrit melanie witt proposed openstack/nova master: Claim and report zero root disk for boot-from-volume instances https://review.openstack.org/428505
13:56:52 openstackgerrit melanie witt proposed openstack/nova master: Request zero root disk for boot-from-volume instances https://review.openstack.org/428481
14:48:32 openstackgerrit Jay Pipes proposed openstack/nova master: move resources_from_request_spec() to utils https://review.openstack.org/476629
14:48:33 openstackgerrit Jay Pipes proposed openstack/nova master: Add uuid to scheduler's HostState object https://review.openstack.org/476630
14:48:33 openstackgerrit Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/476631
14:48:34 openstackgerrit Jay Pipes proposed openstack/nova master: WIP - placement: claim resources in scheduler https://review.openstack.org/476632
14:48:34 openstackgerrit Jay Pipes proposed openstack/nova master: placement: support GET /allocation_candidates https://review.openstack.org/475448
14:49:15 openstack Launchpad bug 1688673 in OpenStack Compute (nova) "cpu_realtime_mask handling is not intuitive" [Undecided,New]
14:49:15 cfriesen_ sdague: I added a review to bug https://bugs.launchpad.net/nova/+bug/1688673, not sure why the link wasn't automatically added.
14:50:39 sdague cfriesen_: I think the fact that you didn't add closes-bug on the first iteration
14:50:51 cfriesen_ sdague: seems likely
14:51:07 sdague cfriesen_: ok, make sure to mark in progress as well
14:51:29 sdague because I'm building another script that's going to unassign all the not in progress bugs
14:51:30 cfriesen_ sdague: done
14:53:39 cfriesen_ anyone know if sahid is on holidays or something?
14:54:34 leakypipes cfriesen_: sahid doesn't hang out on public IRC any more. only RH internal IRC.
14:54:43 leakypipes cfriesen_: he responds to review comments, though.
14:55:35 cfriesen_ leakypipes: ah, okay. thanks
14:55:44 leakypipes cfriesen_: or email of course
15:08:07 bauwser leakypipes: shit, your gerrit topic name is different from one series to another
15:08:11 bauwser that confused me
15:08:41 bauwser bp/placement-allocation-request vs. bp/placement-allocation-request*s*
15:09:22 bauwser leakypipes: you should probably upload again that branch https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/placement-allocation-request and change the topic
15:09:37 bauwser so we could just look over at https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/placement-allocation-requests
15:10:34 bauwser and FWIW I thank matt and dan for having reviewed https://review.openstack.org/#/c/473377/9/nova/objects/resource_provider.py that :p
15:12:35 openstackgerrit Eric Fried proposed openstack/nova master: Adopt new pypowervm power_off APIs https://review.openstack.org/476945
15:20:26 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453
15:33:44 openstackgerrit Jay Pipes proposed openstack/nova master: Remove code that produces warning in modern Python https://review.openstack.org/476952
15:51:39 dtantsur edleafe: hi! is https://review.openstack.org/#/c/473627/ enough for this feature to work? I may try to get a patch against ironic with depends-on to try it.
15:51:50 dtantsur (though probably on Monday, as it's nearly EOD here)
16:24:58 openstackgerrit Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/476631
16:24:59 openstackgerrit Jay Pipes proposed openstack/nova master: placement: support GET /allocation_candidates https://review.openstack.org/475448
16:24:59 openstackgerrit Jay Pipes proposed openstack/nova master: Add uuid to scheduler's HostState object https://review.openstack.org/476630
16:25:00 openstackgerrit Jay Pipes proposed openstack/nova master: WIP - placement: claim resources in scheduler https://review.openstack.org/476632
16:27:08 leakypipes efried, edleafe: addressed your comments (thanks!) from the reviews above.
16:27:22 efried ack
16:27:32 efried leakypipes FYI, still working my way through the series.
16:27:56 leakypipes efried: you definitely need a casual Friday nick.
16:28:09 leakypipes and with a name like Eric Fried, you have many options.
16:28:13 efried leakypipes My name anagrams to fried_rice
16:28:20 leakypipes :)
16:28:23 leakypipes perfect.
16:32:09 dtp lol
16:33:59 openstackgerrit Maciej Kucia proposed openstack/nova master: WIP Introduced class PciDeviceAddress https://review.openstack.org/476972
16:34:00 openstackgerrit Maciej Kucia proposed openstack/nova master: WIP Introduced PciDeviceAddressPattern class https://review.openstack.org/476973
16:36:57 cfriesen_ I've got a proposed change (https://review.openstack.org/#/c/461456) to treat "hw:cpu_realtime_mask" in the flavor extra-specs as a bitmask rather than an implied exclusion mask. If anyone is opinionated on this please feel free to review it. (Personally I think the implicit "all cpus are realtime" is confusing and illogical.)
16:40:17 mriedem edleafe: DictOfStringsField does not validate the value is not null
16:40:18 mriedem http://paste.openstack.org/show/613526/
16:41:04 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285
16:43:47 dtp mriedem i see your -2 on https://review.openstack.org/#/c/464280/; should i still look it over?
16:44:59 mriedem dtp: i think you already looked at the os-services one once
16:45:11 mriedem dtp: the -2 is procedural
16:45:33 mriedem once i'm done with the os-hypervisors change on top and it's generally ok with reviewers, then we can merge them in order or squash them
16:45:50 mriedem i just didn't want another microversion slipping in between

Earlier   Later