Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-05-07
10:31:27 bauzas gibi: worth thinking about RC3 for this one ? :(
10:31:50 bauzas or, maybe we just release and then backport later
10:31:59 bauzas the issue isn't that bad
10:32:12 jazeltq hello gibi
10:32:41 jazeltq I have one question about nova-scheduler
10:35:56 openstackgerrit Sylvain Bauza proposed openstack/nova master: Wait for all servers to be active when testing vGPUs https://review.opendev.org/726087
10:36:19 bauzas gibi: ^
10:37:01 bauzas gibi: FYI (functional) [sbauza@sbauza nova]$ stestr --test-path=./nova/tests/functional run --until-failure nova.tests.functional.libvirt.test_vgpu.VGPUTests.test_multiple_instance_create
10:42:21 stephenfin melwitt: When you're about, can you take a look at https://review.opendev.org/#/c/677444/ please?
10:48:39 gibi bauzas: I don't think we need an RC for a racy func test. It is OK to merge it after GA
12:00:36 bauzas gibi: okay, after 4560 seconds, no errors were found
12:00:57 gibi bauzas: no error seen in 412 runs
12:01:02 gibi so I agree you fixed it!
12:01:28 gibi nice work
12:27:54 efried stephenfin: go ahead and update that commit message. I'm here to re+1, for all the good it'll do ya.
12:35:25 openstackgerrit Stephen Finucane proposed openstack/nova master: config: Explicitly register 'remote_debug' CLI opts https://review.opendev.org/677444
12:35:31 openstack Launchpad bug 1877281 in OpenStack Compute (nova) "vGPU multiple instance creation test is racey" [High,In progress] - Assigned to Sylvain Bauza (sylvain-bauza)
12:35:31 bauzas stephenfin: https://review.opendev.org/#/c/726087/1 closing https://bugs.launchpad.net/nova/+bug/1877281
12:35:32 stephenfin efried: ack, thanks
12:35:39 stephenfin bauzas++
12:37:55 efried stephenfin: done (with bonus)
12:39:48 stephenfin bauzas: I have an ask on https://review.opendev.org/#/c/726087/
12:40:01 stephenfin if you don't want to, I can live with it as-is
12:43:59 bauzas stephenfin: no worries, I can write a new rev
12:44:06 bauzas there is no urgency to fix this race
12:44:18 bauzas we only get it for around 0.5%
12:45:55 openstackgerrit Sylvain Bauza proposed openstack/nova master: Wait for all servers to be active when testing vGPUs https://review.opendev.org/726087
12:46:10 bauzas stephenfin: fer your plizure ^
12:46:47 stephenfin +"
12:46:49 stephenfin *2
13:18:57 francoisp bauzas, salut, when you have time: https://review.opendev.org/#/c/725272/ (this time for Rocky) - thanks
14:27:17 bauzas francoisp: ack, will try to do it asap
14:27:43 francoisp bauzas, thank you!
14:32:53 openstackgerrit Jiri Suchomel proposed openstack/nova-specs master: Add spec for downloading images via RBD https://review.opendev.org/572805
14:36:19 openstackgerrit Takashi Kajinami proposed openstack/nova master: Check py3 fix in python-barbicanclient https://review.opendev.org/726164
14:57:54 openstackgerrit Sasha Andonov proposed openstack/nova master: rbd_utils: increase _destroy_volume timeout https://review.opendev.org/705764
15:45:49 gibi nova meeting starts in 15 minutes on #openstack-meeting-3
16:15:01 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Use proper headers in nova-manage man page https://review.opendev.org/726191
16:15:01 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rewrite 'nova-manage db' docs to use proper directives https://review.opendev.org/677509
16:15:02 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rewrite 'nova-manage placement' docs https://review.opendev.org/726193
16:15:02 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rewrite 'nova-manage api_db', 'cell_v2' docs https://review.opendev.org/726192
16:15:03 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Document options for nova-*proxy https://review.opendev.org/726195
16:15:03 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Only use section headers for HTML https://review.opendev.org/726194
16:44:23 openstackgerrit Shi Yan proposed openstack/nova master: Fix aggregate placement sync issue https://review.opendev.org/726201
16:47:07 openstackgerrit Merged openstack/os-vif master: Remove six https://review.opendev.org/724966
16:47:08 openstackgerrit Merged openstack/os-vif master: Update master for stable/ussuri https://review.opendev.org/724714
16:47:09 openstackgerrit Merged openstack/os-vif master: Add Python3 victoria unit tests https://review.opendev.org/724715
17:01:56 artom gmann, continuing from #-meeting, yeah, sounds like more discussion would have been nice
17:02:26 gmann stephenfin: gibi py3.8 functional job as voting on check and gate pipeline - https://review.opendev.org/#/c/725372/
17:02:27 artom I just... I don't get the opposition. Like, yeah, the name could have been better chosen, and yeah, we may want to improve it in the future
17:02:56 artom But as long as we document what it does for the former, and make it evolve-able for the latter, why not do it?
17:03:14 stephenfin gmann: Oh, it works now? I thought there were issues with mock from Python 3.8
17:03:50 melwitt artom: well, I think we need to know whether it's extendable right? we don't fully understand that yet, do we?
17:04:07 gmann artom: i will say if old things have limit and need more clarity then documenting those is fine but any future thing and we can better design so better design is my preference than going with document-limits
17:04:44 artom gmann, agreed, but we were pre-empted by projects with lesser standards than ours ;)
17:05:08 gmann stephenfin: i think it worked. let me again check log that it is correctly doing py3.8
17:05:15 melwitt like can we build a health data cache in that middleware or is there something about it that would preclude doing that? I don't think basic investigation has been done
17:05:31 artom melwitt, so, I'd be OK with just 2 URLs (and I know dansmith will hate me for that) - /healthcheck for unauthed, unversioned HTTP alice checks, and something like /status that's authed and versioned and we can evolve
17:05:59 melwitt that's backward, /status should be the ping check
17:06:18 gmann artom: that is something to fix since starting of openstack :), making uniform things across all projects is still an issue :).
17:06:23 melwitt and so to me seems worse to do that, expose two things that are doing the opposite of their names
17:07:09 gmann stephenfin: it does and worked fine - https://zuul.opendev.org/t/openstack/build/bc80a0fe21354e10b81b1e4a4db35356/log/job-output.txt#1172
17:07:55 artom melwitt, so find a way for /healthcheck to do both?
17:08:48 melwitt yeah, at least do basic due diligence on it. look at the middleware enough to determine whether we could extend the health check to actually be a health check. or if we would need to expose it some other way in order to be able to extend it. that's mho
17:09:47 stephenfin gmann: okay. comments left
17:10:17 gmann let me do some poc for extend things.but may be next week i will be able to do. so that we can check if current proposed things can be done in compatible way with extendable.
17:10:39 stephenfin who's actually going to work on this extended health check?
17:11:05 stephenfin I mean, without someone signed up to do that work, isn't this all effectively bikeshedding?
17:11:34 artom bauzas, 💣
17:11:34 artom bauzas, 💣
17:11:38 bauzas \o/
17:12:05 artom stephenfin, look like gmann wil try it? I kinda wanna have a whack at it too
17:13:46 melwitt I'm also willing to help work on it (if need be) since I know at least some operators would want it to be a real health check
17:14:11 dansmith me too, I have some ideas on how to make it useful without requiring auth
17:14:20 bauzas again, a good healthcheck is something about L7 checks
17:14:21 bnemec Not a tldr, but most of the previous discussions are linked on https://storyboard.openstack.org/#!/story/2001439
17:14:22 gmann +1
17:14:26 bauzas what dansmith said
17:14:39 zigo There's already some extra parameters to give to /healthcheck with the Accept: header, so we could extend that and add more headers to do deeper checks...
17:15:12 bnemec There's also the detailed opt in the middleware that already allows operators to choose how much they want checked.
17:15:21 zigo X-OpenStack-Check-Type: full-check
17:15:22 artom So we're all just going to have a massive healthcheck party? :)
17:15:30 bauzas this being said, hitting the WSGI nova v2 app could be sufficient IMHO
17:15:51 bauzas and AFAIK, we can hit it without being auth'd
17:15:51 artom bauzas, that's a '300 multiple choices', not '200 OK', and is different from other projects
17:16:19 bauzas artom: I'm just talking of the healthcheck being part of the WSGI app
17:16:24 bauzas and returning 200
17:16:35 bauzas not operators using the existing app call
17:16:44 artom bauzas, you mean not using the middleware at all?
17:16:53 artom It'd be easier to make it extensible, yah
17:17:02 bauzas the middleware just verifies that the WSGI server runs, honestly
17:17:15 bauzas and even worst, it's now a new app with another endpoint
17:17:39 bauzas which means that the wsgi main app can silently drop and the healthcheck would still tell 'heh, fine enough'
17:18:09 bauzas having it as a separate app or a middleware just means (again) that you verify uswgi, that's it
17:25:11 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Moving functional jobs to Victoria testing runtime https://review.opendev.org/725372
17:26:49 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Moving functional jobs to Victoria testing runtime https://review.opendev.org/725372
17:32:07 bauzas gibi: won't be there tomorrow, yet again bank holiday
17:49:58 sean-k-mooney artom: some service allwo the middelware to be customisable other hardcode it
17:50:02 sean-k-mooney im not sure what nova does
17:50:16 artom sean-k-mooney, nova doesn't do anything yet.

Earlier   Later