Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-26
19:19:36 mriedem dhellmann's whereto thing
19:19:43 mriedem http://git.openstack.org/cgit/openstack/whereto/tree/requirements.txt#n6
19:20:35 sean-k-mooney so it got pulled in form https://github.com/openstack/nova/commit/cec0faa7111ce93b5f99622b8e6174d66b25b6d0
19:20:41 Roamer` mriedem, yeah, yesterday something started depending on python-pcre and it caused some problems for most third-party CI systems that didn't have libpcre3-dev installed (although it had been in requirements/bindep.txt for quite some time), but now it's really being used
19:21:23 mriedem yes it's that whereto change
19:21:27 mriedem asking doug in -release
19:21:39 Roamer` right, I saw your links, but I'd already started writing that :) sorry
19:21:58 Roamer` but still your problem looks..... very funny
19:22:41 sean-k-mooney well looking at https://pypi.python.org/pypi/python-pcre it has not had a release since 2015 so it looks pretty dead
19:23:11 sean-k-mooney its also not tested with python 3.5
19:25:11 sean-k-mooney i belived python had pretty good regex support natively does where to really need PCRE
19:26:50 jroll I was wondering that too, but the commit message makes sense: https://github.com/openstack/whereto/commit/797cb67a37d67e41b2caa57aeeb6307b75b66bc8
19:27:42 mriedem i commented out whereto, rebuilt and still hitting this overflow error...
19:28:32 jroll well, that enum certainly is bigger than a C long (32 bits) https://github.com/openstack/nova/blob/master/nova/console/rfb/auth.py#L40
19:28:39 jroll but that code is 6 months old
19:28:57 sean-k-mooney jroll: weel a long is not normally 32bit
19:29:08 sean-k-mooney and it is
19:29:17 mriedem jroll: that code just merged within the last 2 weeks,
19:29:21 sean-k-mooney but now a long is typically 128bits
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

Earlier   Later