Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-05
21:18:55 dansmith opens a DoS though
21:19:05 mriedem true
21:19:24 dansmith I would make it static and then if people need cuteness, we can ask for safe suggestions
21:20:01 mriedem alright, now how to track this - could do it against the blueprint if someone is going to work on it now, else i can open a bug since it's kind of a bug on the nova side that we don't limit
21:20:11 mriedem i'm sure no one cares about that tracking business except me
21:20:24 dansmith I can cook up a patch while you decide how to appease your paperwork fetish
21:20:37 mriedem cool, because i'm already getting rashy
21:21:58 edmondsw the placement api-ref isn't linked from https://developer.openstack.org/api-guide/quick-start/ ... is that intentional or an oversight?
21:27:37 fried_rice mriedem leakypipes dansmith FYI using GET /allocation_candidates limiting is on the sched mtg agenda for Monday.
21:27:57 leakypipes cool, thanks
21:28:21 leakypipes edmondsw: good question...
21:28:32 mriedem edmondsw: likely an oversight
21:28:56 mriedem edmondsw: you should totally add that
21:29:05 edmondsw mriedem leakypipes alright, will do
21:35:25 dansmith I look forward to the rash of "you didn't write this conf option help text right" comments
21:36:40 leakypipes :w
21:40:38 openstackgerrit Merged openstack/nova master: VMware: add support for different firmwares https://review.openstack.org/448034
21:52:57 openstackgerrit Merged openstack/nova stable/pike: Retry _trait_sync on deadlock https://review.openstack.org/528094
22:12:20 mriedem stvnoyes: tempest patch updated with the detach logic and todos for more tests https://review.openstack.org/266605
22:13:59 dansmith hmm, between 1.10 and 1.16 /allocation_candidates result changed ?
22:18:05 figleaf dansmith: yep - https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/microversion.py#L52-L55
22:18:33 dansmith yeah
22:18:56 dansmith figleaf: have links to what they looked like before/after?
22:19:00 dansmith I would go for api_samples normally
22:26:08 figleaf dansmith: IIRC, they were identical, with only the addition of 'project_id' and 'user_id'
22:26:22 figleaf dansmith: but no, I don't have such links handy
22:26:34 dansmith figleaf: no i think the "format now matches that of get" was more of a change
22:26:59 dansmith I'm getting some issues that are related to trying to treat an array like a dict
22:28:05 fried_rice leakypipes Looks like today we don't handle RP generation when we update aggregates.
22:28:30 fried_rice leakypipes At least according to the API ref as it stands. I think that's a problem.
22:30:05 openstackgerrit Jay Pipes proposed openstack/nova master: placement: _get_trees_matching_all() https://review.openstack.org/531512
22:30:16 leakypipes fried_rice: add it as an item on Monday?
22:30:27 fried_rice leakypipes Ight.
22:30:54 fried_rice leakypipes It comes into play in the discussion around coordinating shared providers from multiple compute nodes
22:31:03 leakypipes ack
22:31:15 fried_rice Unless the agg UUIDs are deterministic, we absolutely need generation handling there.
22:32:39 leakypipes fried_rice: to be discussed :) (I am heading off for dinner...)
22:32:46 fried_rice leakypipes Enjoy
22:34:37 mriedem dansmith: https://developer.openstack.org/api-ref/placement/#list-allocation-candidates
22:35:06 dansmith oh I didn't realize both versions were in there
22:35:08 dansmith awesome
22:38:50 mriedem lbragstad: looks like your patch fixes the issue http://logs.openstack.org/97/531497/2/check/legacy-tempest-dsvm-neutron-src/a196420/logs/screen-n-cpu.txt.gz?level=WARNING
22:39:02 mriedem lbragstad: my question was really, let's say nova has 10 deprecated policies in code,
22:39:14 mriedem and i've overridden one of those deprecated policy rules in my policy.yaml,
22:39:17 mriedem but not the other 9
22:39:28 mriedem will i see warnings about all 10, or just the 1 that overrode the default?
22:39:41 lbragstad you will see a warning for the one you overrode
22:40:00 mriedem ok i figured self.file_rules was just checking if there are any
22:40:03 mriedem but not specific ones
22:40:16 lbragstad that's what https://review.openstack.org/#/c/531497/2/oslo_policy/tests/test_policy.py@1004 tests for
22:40:39 lbragstad since that specific test isn't writing a temp file for policy rules (or overrides)
22:41:03 mriedem default.name in self.file_rules
22:41:03 mriedem oh duh
22:41:09 mriedem i didn't realize it was looping over the rules
22:41:10 lbragstad yup
22:41:19 lbragstad yeah - it goes through everything
22:41:54 mriedem +1
22:41:55 mriedem thanks
22:41:57 lbragstad and logs a single warning (so warnings aren't constantly emitted every time a poilcy is run through enforce())
22:42:06 lbragstad no problem - thanks for raising the issue
22:42:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Pass multiattach flag to reserve_block_device_name https://review.openstack.org/531244
22:42:20 openstackgerrit Matt Riedemann proposed openstack/nova master: [libvirt] Allow multiple volume attachments https://review.openstack.org/267587
22:42:21 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: [api] Allow multi-attach in compute api https://review.openstack.org/271047
22:48:08 dansmith bah, have to figure that format thing out on monday
22:48:10 openstackgerrit Dan Smith proposed openstack/nova master: WIP: Pass limit to /allocation_requests https://review.openstack.org/531517
#openstack-nova - 2018-01-06
00:05:24 fried_rice dansmith FYI, I added ^ under the appropriate discussion point for the sched meeting.
00:06:04 fried_rice So you better be there, or we'll make decisions like you have to follow every comma with a newline.
00:08:22 dansmith it's usually too early for me, fwiw
00:12:23 efried Oh, yeah, west coast nah? Oh well. Should be an interesting one.
00:40:34 openstackgerrit Jie Li proposed openstack/nova master: Support volume-backed server rescue https://review.openstack.org/531524
01:58:04 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Fix hypervisor_type in /os-hypervisors/details https://review.openstack.org/531328
01:58:04 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [libvirt] Filter hypervisor_type by virt_type https://review.openstack.org/531347
02:01:44 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: [api] Allow multi-attach in compute api https://review.openstack.org/271047
02:04:00 openstackgerrit melanie witt proposed openstack/nova master: Add index on token_hash and instance_uuid for console_auth_tokens https://review.openstack.org/531477
02:04:00 openstackgerrit melanie witt proposed openstack/nova master: Add access_url_base to console_auth_tokens table https://review.openstack.org/334614
02:04:01 openstackgerrit melanie witt proposed openstack/nova master: Add ConsoleAuthToken object https://review.openstack.org/320063
02:04:01 openstackgerrit melanie witt proposed openstack/nova master: Optionalize instance_uuid in console_auth_token_get_valid() https://review.openstack.org/481700
02:04:02 openstackgerrit melanie witt proposed openstack/nova master: Use ConsoleAuthToken object to generate authorizations https://review.openstack.org/325414
02:04:02 openstackgerrit melanie witt proposed openstack/nova master: Add periodic task to clean expired console tokens https://review.openstack.org/325381
02:04:03 openstackgerrit melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990
12:51:48 openstackgerrit Merged openstack/nova master: manager: more detailed info of unsupported compute driver https://review.openstack.org/530701
14:44:46 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
15:06:07 openstackgerrit Hongbin Lu proposed openstack/nova master: Update FAQs about listing hosts in cellv2 https://review.openstack.org/531559
15:08:29 openstackgerrit Hongbin Lu proposed openstack/nova master: Update FAQs about listing hosts in cellv2 https://review.openstack.org/531559
15:40:48 openstackgerrit Merged openstack/nova master: Add access_url_base to console_auth_tokens table https://review.openstack.org/334614
19:49:53 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408
19:56:46 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
#openstack-nova - 2018-01-07
04:22:32 ninja1WHXFQ ▄▄▄▄▄▄▄▄▄▄▄▄▄▄ LRH IS LIVE NOW!! TODAYS EDITION SLIMER GETS FUCKED IN VEGAS!! https://www.youtube.com/user/l0de/live CALL 315-505-4666 tignww: purplerbot alex_xu_ sapcc-bot ▄▄▄▄▄▄▄▄▄▄
04:22:32 ninja1WHXFQ ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ LRH IS LIVE NOW!! TODAYS EDITION SLIMER GETS FUCKED IN VEGAS!! https://www.youtube.com/user/l0de/live CALL 315-505-4666 ruyjey: frickler rm_work purplerbot ▄▄▄▄▄▄▄▄▄▄▄▄▄▄
10:59:46 openstackgerrit Jie Li proposed openstack/nova master: Support volume-backed server rescue https://review.openstack.org/531524
11:39:50 openstackgerrit Andreas Jaeger proposed openstack/nova master: Fix OpenStack capitalization https://review.openstack.org/531593
13:32:10 openstackgerrit Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900
13:43:08 Rambo Good mornig,everyone, Can you help me to review it ?The link is https://review.openstack.org/#/c/531524/
13:43:35 Rambo Another is :https://review.openstack.org/#/c/528740/
13:43:50 Rambo Thank you very much
15:08:44 mriedem ildikov: http://logs.openstack.org/86/531386/5/check/tempest-full/9531761/job-output.txt.gz#_2018-01-06_19_38_04_914985
15:15:25 ildikov mriedem: you mean no more errors anywhere? :)
15:16:21 ildikov mriedem: and thank you of course!
15:18:01 mriedem didn't check the n-cpu logs but the test passed
15:26:07 ildikov mriedem: this looks a little scary: Do NOT duplicate iSCSI InitiatorNames

Earlier   Later