Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-26
19:29:25 mriedem the commit was written 6 months ago
19:29:32 jroll ah
19:29:40 jroll sean-k-mooney: oh right, 32 bits is the minimum in the spec
19:30:23 sean-k-mooney jroll: on a 32 bit arch yes on 64bit a long is at least the size of an int so minium would be 64bit
19:31:17 jroll nod
19:32:46 mriedem this is a 32 bit vm i guess, so that's my problem
19:32:55 jroll woo
19:34:00 jroll :P
19:35:08 Roamer` sean-k-mooney, actually it depends on the architecture, there are some really crazy ones, like e.g. x86_64, where a long is 64 bits, but an int is still 32 bits :)
19:36:47 Roamer` (this one really caught me off-guard some years ago when I was writing something quite close to the hardware and didn't even think to check sizeof(int) until a coworker pointed it out to me)
19:41:51 openstackgerrit Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/537171
20:08:36 openstackgerrit Sen Yang proposed openstack/python-novaclient master: Implement hypervisor hostname exact patten match for server cold migrate https://review.openstack.org/520187
20:17:53 chris___ Hey Guys, does anyone know how to disable non-admin users from taking snapshots via horizon in openstack juno? I'm looking in the /etc/nova/policy.json file, but I don't see anything related to compute:snapshots, only compute:volume_snapshot_create
20:23:08 jgwentworth chris___: I think it's called create_image
20:25:23 sean-k-mooney Roamer`: actully yes but x86_64 even depends on the os i was reading this a while back. https://wiki.osdev.org/X86-64#Data_Types so linux should have a 64 bit long on x86_64. if you are on power or arm totally different. that is why c++ 11 and c 11 standards introduced the fixed with integer types http://en.cppreference.com/w/cpp/types/integer
20:25:24 jgwentworth hm, looks like back in juno it wasn't controllable via policy
20:26:13 sean-k-mooney Roamer`: anyway time to head home enjoy your weekend.
20:28:21 chris___ jgwentworth: Thanks for the response, but i'm not seeing that parameter inside of the nova policy.json file
20:28:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle network-changed event for a specific port https://review.openstack.org/465787
20:28:31 mriedem cfriesen: finally added tests for this ^
20:28:57 jgwentworth chris___: yeah, I posted another message saying back in juno it looks like it wasn't controllable via policy. I see it was possible starting in liberty
20:30:08 sean-k-mooney chris___: since it apprently was not supported. if you run nova's api under appache or have a loadplancer in front of it you might need to add a redirect rule that will block the api endpoint form public ips.
20:31:10 sean-k-mooney chris___: you should be able to match on the url and return a 404 or other message at the webserver level before it hits the openstack api as a workaround
20:34:16 chris___ jgwentworth: sean-k-mooney: Thanks for the replies, i might look into that for the time being
20:44:45 jgwentworth mriedem: I've been noticing frequent timeouts on the functional test job. I was thinking, I wonder if we should split the api samples into their own job? I already feel like they should be removed from a normal 'tox -e functional' run considering they have their own tox target 'api-samples'
20:45:28 mriedem https://review.openstack.org/#/c/537933/
20:46:01 mriedem ^ increases functional job timeout, but doesn't split them up
20:47:43 jgwentworth mriedem: yeah, we should definitely do that. I was just wondering if it might help to split them up longer term
20:50:34 mriedem yeah maybe,
20:50:45 mriedem i know i don't run the full functional very often b/c of the api-samples tests
20:53:42 jgwentworth same
20:54:38 jgwentworth I was just thinking we could a) reduce developer pain for running non-api-sample functional tests b) keep the timeouts more sane in the future
22:25:01 jgwentworth the 'increase functional test job timeout' change timed out
22:25:11 jgwentworth on the py35 functional test job
22:28:04 jgwentworth and there are about a billion policy deprecated messages in the job output
22:34:08 jgwentworth I wonder if the deprecation messages are related to the timeouts
22:53:41 mriedem superdan has a patch to fix that
22:53:49 mriedem er superlance
22:53:56 mriedem https://review.openstack.org/#/c/537603/
22:54:25 jgwentworth already sent them toward the gate
22:55:01 mriedem i've been busy getting a new dev vm setup this afternoon...
22:55:16 mriedem after all the 32-bit shame from earlier
22:55:33 jgwentworth haha
23:07:53 penick Someone took my junoman nick, dang
23:16:08 mriedem welp, looks like most of the approved stuff is all happily waiting or chugging through the gate now
#openstack-nova - 2018-01-27
00:02:01 openstackgerrit melanie witt proposed openstack/nova master: Reduce policy deprecation warnings in test runs https://review.openstack.org/537603
00:02:02 openstackgerrit melanie witt proposed openstack/nova master: Remove deprecated policy items from fake_policy https://review.openstack.org/537600
00:15:51 openstackgerrit Merged openstack/nova master: Deprecate allowed_direct_url_schemes and nova.image.download.modules https://review.openstack.org/510195
00:16:10 openstackgerrit Merged openstack/nova master: Only call numa_fit_instance_to_host if necessary https://review.openstack.org/530792
00:58:59 openstackgerrit melanie witt proposed openstack/nova master: Bumping functional test job timeouts https://review.openstack.org/537933
01:20:06 Spazmotic Morning
01:22:46 Spaz-Work Alllllright time to register so I can ghost these nicks and quick messing about I suppose
01:36:44 openstackgerrit Merged openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
01:49:23 openstackgerrit Brooks Kaminski proposed openstack/nova master: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/533168
03:34:13 openstackgerrit Merged openstack/nova master: placement: support traits in allocation candidates API https://review.openstack.org/535642
06:31:57 openstackgerrit Merged openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436
09:40:06 openstackgerrit Brooks Kaminski proposed openstack/nova master: XenAPI: XenServer 7 Live Migration do not assert_can_migrate on source. https://review.openstack.org/538415
09:40:46 openstackgerrit Brooks Kaminski proposed openstack/nova master: XenAPI: XenServer7 Live Migration do not assert_can_migrate https://review.openstack.org/538415
09:41:13 Spazmotic Woops :0
09:41:24 Spazmotic made it a little long on the firstp ass heh
09:41:33 Spazmotic erm... pass.. i'll just..
10:26:38 openstackgerrit Merged openstack/nova master: Initialize osprofiler in WSGI application https://review.openstack.org/519664
10:36:45 openstackgerrit Merged openstack/nova master: Fix functional tests for USE_NEUTRON https://review.openstack.org/529456
10:36:59 openstackgerrit Merged openstack/nova master: Expand on when you might want to set --max-count for map_instances https://review.openstack.org/536959
10:37:14 openstackgerrit Merged openstack/nova master: Fix the incorrect RST convention https://review.openstack.org/538208
11:32:02 openstackgerrit Merged openstack/nova master: correct referenced url in comments https://review.openstack.org/534987
11:32:11 openstackgerrit Merged openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref https://review.openstack.org/522918
11:32:22 openstackgerrit Merged openstack/nova master: Add exception to no-upcall note of cells doc https://review.openstack.org/532491
11:36:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix a comment in a notification functional test https://review.openstack.org/536262
12:01:47 openstackgerrit Merged openstack/nova master: Fix missing marker functions https://review.openstack.org/514579
12:02:03 openstackgerrit Merged openstack/nova master: Extract instance allocation removal code https://review.openstack.org/513041
12:44:41 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add server filters whitelist in server api-ref https://review.openstack.org/527042
14:13:31 openstackgerrit Merged openstack/nova master: Reduce complexity of _from_db_object https://review.openstack.org/537728
15:28:44 openstackgerrit Merged openstack/nova master: Move remaining uses of parted to privsep. https://review.openstack.org/519483
15:28:53 openstackgerrit Merged openstack/nova master: Merge ResourceV21 obj into Resource obj https://review.openstack.org/529217
15:29:17 openstackgerrit Merged openstack/nova master: Handle images with no data https://review.openstack.org/526329
15:29:26 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/537093
15:29:44 openstackgerrit Merged openstack/nova master: libvirt: pass the mdevs when rebooting the guest https://review.openstack.org/533818
15:48:30 Spazmotic Whew. Full and sleepy
15:48:37 Spazmotic Fuleepy
17:16:15 openstackgerrit Merged openstack/nova master: Make sure that functional test triggered on sample changes https://review.openstack.org/533210
17:16:31 openstackgerrit Merged openstack/nova master: Stop globally caching host states in scheduler HostManager https://review.openstack.org/533371
17:46:30 efried Any cores around to do trivial re-+Ws on patches with commit message updates for Depends-On brokenness?
18:05:35 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/538473
18:21:35 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/533994
18:21:56 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918
19:06:06 efried mriedem Willing to +W some already-approved patches if I update the commit messages for the Depends-On thing (see ML)
19:15:35 mriedem see what in the ML? depends-on with the URL isn't working?
19:17:34 efried mriedem The #/c form isn't working, needs to be the permalink for now.
19:18:23 efried mriedem So I was gonna change https://review.openstack.org/#/c/532288/ but it'll need a re-+W
19:18:37 efried ...and the guy on top of it, when rebased.
19:20:26 mriedem ok
19:20:33 mriedem do it and send me the list
19:22:24 openstackgerrit Eric Fried proposed openstack/nova master: Implement get_traits() for the ironic virt driver https://review.openstack.org/532288
19:22:25 openstackgerrit Eric Fried proposed openstack/nova master: Add functional tests for virt driver get_traits() method https://review.openstack.org/538047
19:22:26 efried mriedem ^
19:22:52 efried both a doze
19:23:37 efried Hm, the top one kept its +W, so just https://review.openstack.org/#/c/532288/
19:24:20 mriedem done
19:24:21 mriedem thanks

Earlier   Later