Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-07-25
23:05:13 mriedem rpc could have also fallen over
23:05:15 eandersson Then no
23:05:31 sean-k-mooney mriedem: right that is the only thing between those two lines that could fail
23:05:33 mriedem in that case you'd probably have MessagingTimeouts for the db save rpc calls
23:06:52 eandersson I have 1-2 MySQL server has gone away in the logs, but nothing near the time that happened
23:07:18 eandersson (also those only failed on select 1)
23:07:22 sean-k-mooney well i was wondering if the notifier = rpc.get_notifier(service) line is where it stoped
23:07:34 sean-k-mooney so it might not be related to the db
23:07:36 sean-k-mooney but to rabbit
23:09:11 eandersson hmm does placement do rpc?
23:09:20 eandersson or would this be within nova only?
23:09:25 sean-k-mooney this is in nova
23:09:30 sean-k-mooney in the conductor
23:09:42 sean-k-mooney and no placment does not do any rpc as far as i am aware
23:10:14 sean-k-mooney its just a wsgi app in front of a db
23:10:42 eandersson One thing I don't like with oslo.messaging is that it ack's the message before it gets processed
23:14:24 eandersson oh
23:14:26 eandersson > Exception during message handling
23:14:46 eandersson > Exception during message handling: MaxRetriesExceeded: Exceeded maximum number of retries.
23:15:02 eandersson That is the error I posted above
23:15:08 eandersson And this is from a normal failure
23:15:11 eandersson > Setting instance to ERROR state.: MaxRetriesExceeded: Exceeded maximum number of retries.
23:17:00 mriedem yeah that's this https://opendev.org/openstack/nova/src/branch/master/nova/scheduler/utils.py#L730
23:17:10 mriedem called from https://github.com/openstack/nova/blob/stable/rocky/nova/conductor/manager.py#L619
23:17:22 mriedem the instance change should be saved here https://opendev.org/openstack/nova/src/branch/master/nova/scheduler/utils.py#L736
23:19:32 sean-k-mooney right but in the case where it remains in building we get Exception during message handling:... instead of Setting instance to ERROR state.:...
23:19:50 sean-k-mooney eandersson: is ^ correct
23:19:54 eandersson https://opendev.org/openstack/oslo.messaging/src/branch/master/oslo_messaging/rpc/server.py#L174
23:20:03 eandersson This could be anything :'(
23:23:28 sean-k-mooney eandersson: it might be good to check your rabbitmq server logs and see if there are any errors
23:23:46 sean-k-mooney although i honestly dont really know how that code works
23:24:04 eandersson http://paste.openstack.org/show/754875/
23:24:15 eandersson sean-k-mooney, I honestly don't think it's a rmq issue directly
23:25:00 sean-k-mooney ok its unlikely that its related to that placement error
23:25:17 sean-k-mooney that should happen before we try to build the instace
23:25:29 eandersson That log was from the same milisecond
23:25:58 eandersson I found two Messaging errors and both are the same issue
23:26:00 sean-k-mooney its posible that we only have 1 candiate host
23:26:06 sean-k-mooney and that we raced
23:26:10 eandersson Other than that no other oslo messaging issues
23:26:27 sean-k-mooney as a result of the fact overcommit is not working
23:26:41 eandersson If there was a rmq issue I am sure other services or request would have failed
23:26:56 sean-k-mooney neutron would be the first
23:26:57 eandersson We have a pretty massive deployment so always a lot of things going on
23:27:18 sean-k-mooney althogh they have redused the rpc traffic a lot in the last releae or two
23:29:59 sean-k-mooney oh the same message is also logged form here https://github.com/openstack/nova/blob/stable/rocky/nova/conductor/manager.py#L676
23:30:04 artom Heh, for what it's worth, that per-compute libvirt connection mocking has issues:
23:30:08 artom " 2019-07-25 19:29:16,889 ERROR [nova.virt.libvirt.host] Hostname has changed from test_compute0 to test_compute1. A restart is required to take effect."
23:30:10 sean-k-mooney the other place was teh cellv1 version
23:31:18 sean-k-mooney actully never bind its the same function
23:33:17 sean-k-mooney so if this is a retry then we are executing https://github.com/openstack/nova/blob/stable/rocky/nova/conductor/manager.py#L642-L676
23:34:08 sean-k-mooney and on line 655 we are trying to calim the alternate hosts which is failing with http://paste.openstack.org/show/754875/ because overcommit is not working
23:34:54 eandersson Yea pretty sure the error is within oslo.messaging
23:35:16 sean-k-mooney it might not be
23:35:25 sean-k-mooney when we rais the excetip form here https://github.com/openstack/nova/blob/stable/rocky/nova/conductor/manager.py#L676
23:35:33 sean-k-mooney it is not caut locally in this funciton
23:35:33 eandersson oh
23:39:17 sean-k-mooney i should proably go to bed but there is likely a bug there but im to tired to track it this evening
23:40:04 mnaser efried: i think you were working on moving nova to use openstacksdk ? do you have some of the commit you did in nova that did that? trying to do something similar for another project
23:42:19 eandersson Thanks for the help sean-k-mooney
23:42:24 eandersson I'll create a bug report
23:42:46 gmann johnthetubaguy: melwitt sean-k-mooney efried added my comment/query on unified limit spec. I am not very clear about how we will handle GET for limits which are not going to move to new unified limits(for example server_groups).
23:43:29 gmann I am almost ok for proxy APIs (as per operators interest) if HTTPGone on those APIs is not acceptable.
23:45:35 melwitt gmann: thanks for reviewing. I am not 100% operators would be opposed to having to use an older microversion to use the APIs *but* I think the thing that makes it weird is that unified limits would be opt-in. so that is where I'm uneasy with removing proxy API in new microversion. wanted your opinion on that aspect as well
23:50:17 gmann melwitt: 'removing proxy API in new microversion' and keep them working for older one seems no benefit and even more maintenance. I was thinking we say those APIs are gone (410 response code HTTPGone) because nova quota system is gone and without microversion bump. similar approach as nova-network & n-cert case.
23:51:16 gmann and before we do that we can trigger the notification to users via deprecating those APIs
23:51:29 melwitt gmann: oh, sorry, I didn't know HTTPGone is different. my bad
23:52:07 melwitt yeah, I'd like to be able to do that but wasn't sure about the API perspective of deprecating something when the new thing is opt-in and not on by default
23:52:41 gmann melwitt: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/floating_ip_dns.py#L25
23:53:02 openstack Launchpad bug 1837955 in OpenStack Compute (nova) "MaxRetriesExceeded sometime fails with messaging exception" [Undecided,New]
23:53:02 eandersson sean-k-mooney, https://bugs.launchpad.net/nova/+bug/1837955
23:54:42 gmann melwitt: the only things make me feel uncomfortable to do that was the comment johnthetubaguy added in alternate section about Forum discussion with operators about keeping the old tooling. But i hope that is only for transition period not permanently
23:57:27 gmann melwitt: and second point is about GET quotas API to get the limits which are not going on new system. keep existing GET quotas APIs for them ?
23:57:52 melwitt limits mode is on by default
23:57:52 melwitt gmann: ok. the treatment of /os-quota-sets and /os-quota-class-sets is definitely temporary for the transition. the /limits API we're not completely sure because if we HTTPGone that one, there will be no more ability for users to show limits + usage in one API. but if community could be OK with having to go to two different API (keystone for limits and placement for usage) then we could 410 /limits too in the future when unified
23:59:10 gmann melwitt: +1 we can decide the limit thing later once hierarchy unified limits are there.
#openstack-nova - 2019-07-26
00:01:06 gmann melwitt: but can we query on placement for nova owned limits usage (server groups, injected_files etc) ? also not about showing the limit from keystone so we need to keep the GET /os-quota-sets for those limits al least.
00:01:45 melwitt gmann: I'm not sure I understand the second question about the existing GET quotas API for limits. we have to keep that as long as old quotas are the default, I think
00:02:11 melwitt gmann: we cannot, no. so that's another thing
00:03:17 gmann yeah.
00:04:35 melwitt gmann: it's a little unclear because there's overlap with /limits and GET /os-quota-sets. I'm not sure we need the latter if we have the former (once unified limits is default down the road)
00:05:17 melwitt the (server groups, injected_files etc) limits are shown from /limits and will only be modifiable via the config files. no more API access for those
00:05:35 melwitt *no more API access to update those
00:06:13 gmann yeah, if we keep /limits then it is enough. But we need one API to show those limits and usage.
00:06:41 melwitt yeah, agreed
00:12:18 openstackgerrit melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
00:20:43 openstackgerrit melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
00:22:03 openstackgerrit melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
03:00:44 openstackgerrit karim proposed openstack/nova-specs master: Support VM memory encryption using Intel's MKTME https://review.opendev.org/666769
03:22:36 openstackgerrit Erik Olof Gunnar Andersson proposed openstack/nova master: Always cleanup when hitting MaxRetriesExceeded https://review.opendev.org/672855
03:23:42 openstackgerrit pengyuesheng proposed openstack/os-vif master: Blacklist sphinx 2.1.0 (autodoc bug) https://review.opendev.org/672856
03:25:24 eandersson sean-k-mooney, no clue if that is the fix, but looks reasonable to me ^
03:26:08 openstackgerrit pengyuesheng proposed openstack/os-vif master: Bump the openstackdocstheme extension to 1.20 https://review.opendev.org/672857
03:27:17 openstackgerrit pengyuesheng proposed openstack/os-vif master: Bump the openstackdocstheme extension to 1.20 https://review.opendev.org/672857
03:46:35 openstackgerrit Erik Olof Gunnar Andersson proposed openstack/nova master: Always cleanup when hitting MaxRetriesExceeded https://review.opendev.org/672855
03:49:14 openstackgerrit Erik Olof Gunnar Andersson proposed openstack/nova master: Always cleanup when hitting MaxRetriesExceeded https://review.opendev.org/672855
04:25:41 openstackgerrit Merged openstack/nova-specs master: Add user_id to the migrations https://review.opendev.org/667894
04:31:37 alex_xu sean-k-mooney: stephenfin the real_time_mask indeed neeed the '^', you can check my reply https://review.opendev.org/#/c/668656/4/specs/train/approved/use-pcpu-vcpu-in-one-instance.rst@84
04:49:20 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Functional test for NUMA live migration https://review.opendev.org/672595
07:02:41 openstackgerrit Huachang Wang proposed openstack/os-traits master: CPU: add a trait for AVX512-VNNI support https://review.opendev.org/672888

Earlier   Later