| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-24 | |||
| 13:02:15 | ralonsoh | alex_xu: I meant in https://review.openstack.org/#/c/435954/11/specs/pike/approved/enable-sriov-nic-features.rst@121, instead of writing low case value | |
| 13:02:32 | alex_xu | ralonsoh: yea | |
| 13:02:39 | ralonsoh | alex_xu: we need to write the normalized strings (defined in os-traits) | |
| 13:02:41 | ralonsoh | perfect! | |
| 13:02:52 | ralonsoh | alex_xu: I'll update the spec today | |
| 13:02:53 | alex_xu | ralonsoh: that is what i mean | |
| 13:02:58 | ralonsoh | and thanks! | |
| 13:02:58 | alex_xu | ralonsoh: thanks | |
| 13:09:52 | mriedem | bauwser: you might want to take a look at this https://bugs.launchpad.net/nova/+bug/1675607 | |
| 13:09:52 | openstack | Launchpad bug 1675607 in OpenStack Compute (nova) "test_list_migrations_in_flavor_resize_situation fails with NoValidHost - AvailabilityZoneFilter returned 0 hosts" [Undecided,Confirmed] | |
| 13:10:02 | mriedem | ES is backed up so i'm not sure if it's a recent regression or not | |
| 13:10:04 | bauwser | mriedem: yup, saw you adding me | |
| 13:10:19 | bauwser | it's in my pipe | |
| 13:10:39 | mriedem | well smoke'em if you got'em | |
| 13:11:22 | bauwser | :p | |
| 13:18:30 | markus_z | trivial fix: https://review.openstack.org/#/c/449543/1 | |
| 13:19:33 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: WIP: Add policy-remove-scope-checks spec https://review.openstack.org/433037 | |
| 13:20:09 | johnthetubaguy | sdague: attempt to apply the first few of your comments, and I am heading for lunch now, but I am curious if that looks better or worse ^ | |
| 13:21:19 | sdague | ok, I'll check post breakfast | |
| 13:23:21 | mriedem | johnthetubaguy: i think sdague summed up my issues nicely :) | |
| 13:49:11 | dansmith | mriedem: so that az bug | |
| 13:49:17 | mriedem | yes superdan | |
| 13:49:20 | dansmith | heh | |
| 13:49:35 | superdan | mriedem: after my change to persist the az properly in the instance, | |
| 13:49:54 | superdan | the scheduler will start keeping an instance in the same az it was booted in, | |
| 13:50:00 | superdan | even if they didn't ask for one | |
| 13:50:12 | superdan | that's because of the cruft still left to cleanup with reqspec | |
| 13:50:29 | superdan | reqspec has what they requested, but instance stores what it was booted in, and we're not restoring the reqspec at the right place yet | |
| 13:50:43 | superdan | now, I think in practice this is fine, and we even discussed it prior to merging | |
| 13:50:51 | superdan | however, I think what is happening is, | |
| 13:51:11 | superdan | tempest creates a test az and adds the one compute to it in another test, | |
| 13:51:36 | superdan | and if that instance happens to land there while the az is in place, and is then removed, the instance won't schedule anywhere because we have now told nova that the az it wants doesn't exist anymore | |
| 13:51:38 | superdan | we == tempest | |
| 13:51:54 | superdan | if the tests run such that we don't get lucky there, then we're fine | |
| 13:52:11 | superdan | bauwser: ^ | |
| 13:52:22 | mriedem | let me see what the test does | |
| 13:52:25 | bauwser | sorry was distracted by some internal bug triaging :) | |
| 13:52:27 | mriedem | the az is random btw | |
| 13:52:38 | superdan | mriedem: right | |
| 13:52:51 | superdan | mriedem: the az comes from compute/admin/test_aggregates | |
| 13:53:09 | mriedem | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/admin/test_migrations.py#n42 | |
| 13:53:38 | superdan | right, no az stuff there | |
| 13:54:58 | mriedem | superdan: oh so the instance lands on the single compute which, because of a concurrently running test, is now in an AZ, so the instance in the test that fails is now in that temp AZ which on cleanup of the other unrelated test is removed | |
| 13:55:00 | bauwser | mriedem: superdan: so if there is no AZ asked when booting, the default is not set | |
| 13:55:09 | mriedem | so when we move the instance, it's in an AZ that is now gone b/c of the tempest cleanup? | |
| 13:55:17 | superdan | mriedem: that's what I'm thinking yeah | |
| 13:55:23 | bauwser | mriedem: that could be a reason yes | |
| 13:55:52 | mriedem | look here http://logs.openstack.org/62/449362/1/check/gate-tempest-dsvm-py35-ubuntu-xenial/ec959b4/console.html#_2017-03-24_00_00_55_563868 | |
| 13:55:58 | mriedem | there is an AZ test right at the same time | |
| 13:56:24 | superdan | we need this cleanup to be done in order to ask for what they actually wanted: https://github.com/openstack/nova/blob/master/nova/conductor/tasks/migrate.py#L49-L55 | |
| 13:56:37 | bauwser | I was planning to do so next week | |
| 13:57:00 | bauwser | -ish | |
| 13:57:42 | bauwser | I could try to tackle that now | |
| 13:58:15 | superdan | hmm, | |
| 13:58:26 | superdan | looks like we got request_spec at init and we just overwrite it from_compontents? | |
| 13:58:29 | superdan | is that right? | |
| 13:58:39 | sdague | mriedem or others, anyone want to merge in i18n deletes. I'm trying to keep no more than 2 open at a time just to not waste time on rebasing - https://review.openstack.org/#/c/448082/ | |
| 13:59:23 | mriedem | superdan: i tried to summarize in the bug | |
| 13:59:26 | superdan | yeah, I think we should have that right there | |
| 13:59:32 | superdan | mriedem: okay I was going to, sorry | |
| 13:59:51 | bauwser | superdan: yup, I actually think we can directly use self.request_spec without needing to rehydrate | |
| 14:00:06 | superdan | I'll try | |
| 14:00:06 | bauwser | because it's useless to regeneate ity | |
| 14:00:11 | bauwser | lemme fire the drill | |
| 14:00:28 | bauwser | because some other move operations can have the same stuff | |
| 14:00:36 | bauwser | and here, it's resize | |
| 14:00:42 | bauwser | not live-mig | |
| 14:00:46 | bauwser | superdan: ^ | |
| 14:01:02 | bauwser | superdan: I'm cooking the patch | |
| 14:01:11 | superdan | we get it from the same place in both cases | |
| 14:01:12 | superdan | bauwser: about to submit | |
| 14:01:36 | bauwser | superdan: okay, so if you cook, don't forget to add the resize case | |
| 14:01:47 | superdan | eh? | |
| 14:02:00 | superdan | it's in migratetask, same for both | |
| 14:02:56 | bauwser | oh snpa | |
| 14:03:09 | bauwser | I was thinking we were loooking at the *live* migrate Task | |
| 14:03:24 | bauwser | nvm then | |
| 14:03:39 | bauwser | but I think we probably have the same for live-migration and rebuild | |
| 14:04:09 | bauwser | I'm just looking at that | |
| 14:04:10 | superdan | let me put this up to see if it breaks anything and/or appears to fix | |
| 14:05:08 | bauwser | okay | |
| 14:05:41 | bauwser | FWIW, live-migration is not having the same useless rehydratation | |
| 14:05:50 | bauwser | only if we have a not found spec | |
| 14:05:57 | bauwser | but we can remove the conditional | |
| 14:06:10 | superdan | yeah | |
| 14:06:15 | bauwser | so, I'm gonna fire a patch for that given we now have 100% of chances to have it | |
| 14:06:22 | superdan | alright | |
| 14:06:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: remove i18n log markers from nova.api.* https://review.openstack.org/448082 | |
| 14:07:22 | mriedem | yay my sump pump is running every other minute | |
| 14:07:24 | mriedem | weeee | |
| 14:07:55 | superdan | mriedem: http://imgur.com/DWMGBJW | |
| 14:08:27 | mriedem | which is which? | |
| 14:08:29 | mriedem | blue is rain? | |
| 14:08:34 | superdan | seconds on the left, inches on the right | |
| 14:08:44 | superdan | runtime seconds per hour, that is | |
| 14:09:11 | superdan | oh and green is rain | |
| 14:09:13 | bauwser | interesting | |
| 14:09:19 | bauwser | we don't build houses this way | |
| 14:09:30 | bauwser | is that for collecting rain ? | |
| 14:10:10 | bauwser | because I have a drain running all around my house but the gravity does the rest :) | |