| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-31 | |||
| 12:57:54 | bauzas | so in theory, we should accept to migrate to foo or FOO | |
| 12:58:04 | bauzas | but yeah, I guess it's because mysql | |
| 12:58:42 | bauzas | ratailor: around ? | |
| 12:58:50 | ratailor | bauzas, yep | |
| 12:58:56 | bauzas | ratailor: I feel I badly triaged your bug | |
| 12:59:12 | bauzas | ratailor: since DNS is case-insensitive, hostnames should be too | |
| 12:59:45 | ratailor | bauzas, I reproduced it, and found that mysql doesn't support case-sensitivity by-default. So I had to change the collation on related tables. | |
| 12:59:50 | ratailor | bauzas, which seems to work. | |
| 13:00:26 | mriedem | you know what is case sensitive by default (i think)? | |
| 13:00:29 | mriedem | POSTGRESQL! | |
| 13:01:23 | mriedem | sort of https://stackoverflow.com/questions/21796446/postgres-case-sensitivity | |
| 13:01:23 | ratailor | I was discussing it with alex_xu some days ago and he mentioned that mysql is case-insensitive by-default. | |
| 13:01:24 | bauzas | ratailor: tbh, I just feel that we should allow HoStNaME1 as a possible value for a host to be added in an aggregate | |
| 13:01:25 | mriedem | depends on quotes | |
| 13:01:39 | bauzas | ratailor: and rather fix the filter | |
| 13:01:44 | mriedem | ratailor: yeah, the mysql case issue is a known one | |
| 13:01:48 | mriedem | that breaks several things | |
| 13:01:48 | bauzas | yup | |
| 13:02:05 | ratailor | mriedem, the host-aggregate metadata keys as well, | |
| 13:02:10 | bauzas | ratailor: I'll rephrase your bug report if you agree | |
| 13:02:12 | mriedem | ratailor: https://specs.openstack.org/openstack/nova-specs/specs/newton/approved/lowercase-metadata-keys.html | |
| 13:02:23 | ratailor | bauzas, no problem. | |
| 13:02:52 | bauzas | mriedem: well, hostnames can be FQDNs | |
| 13:03:04 | bauzas | mriedem: if so, those have to be case-insensitive | |
| 13:04:03 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Create allocations against forced dest host during evacuate https://review.openstack.org/499399 | |
| 13:05:32 | ratailor | bauzas, I don't understand you mentioning HoStNaME1 as possible value for host to add in aggregate. | |
| 13:05:34 | bauzas | mriedem: about your point above with providing the same host for the target and source, if we accept to verify that by the API, we also need to add this to the live-migration one | |
| 13:06:02 | ratailor | bauzas, As of now, if hostname is set as "compute0.example.com" (in lower case) | |
| 13:06:02 | ratailor | and user tries to add this host to host-aggregate but by-mistake types | |
| 13:06:02 | ratailor | "COMPUTE0.example.com" (in capital case), then instead of throwing | |
| 13:06:02 | ratailor | HostNotFound error, that host is successfully added to host-aggregate | |
| 13:06:02 | ratailor | as "COMPUTE0.example.com". And after that if instance creation request | |
| 13:06:02 | ratailor | with flavor matching metadata as host-aggregate metadata comes, this | |
| 13:06:04 | ratailor | host is not filtered by scheduler, since there is no host with hostname | |
| 13:06:06 | ratailor | COMPUTE0.example.com, as added in host-aggregate | |
| 13:09:11 | bauzas | ratailor: I got the problem | |
| 13:09:43 | ratailor | bauzas, cool, | |
| 13:09:46 | bauzas | ratailor: what I feel is that we somehow should still accept COMPUTE0 as a possible value for the host to be added in the aggregate | |
| 13:09:57 | bauzas | from an API perspective | |
| 13:10:08 | bauzas | if we want to follow the DNS RFC | |
| 13:10:51 | bauzas | ratailor: what we could do tho is to lowercase that string before amending the aggregate | |
| 13:11:39 | bauzas | and that wouldn't trample https://specs.openstack.org/openstack/nova-specs/specs/newton/approved/lowercase-metadata-keys.html | |
| 13:11:52 | ratailor | bauzas, But that can happen other way round as well, hostname can be COMPUTE0 and user tries to add compute0 which should fail, as there is no hostname with compute0. | |
| 13:12:56 | bauzas | then the filter has to be fixed too | |
| 13:13:19 | bauzas | because it fails due to the filtter, right? | |
| 13:13:26 | ratailor | bauzas, I think, that's separate bug, which is only concerned about metadata keys. Is it somehow related to hostname. | |
| 13:14:29 | ratailor | bauzas, I think, the issue is with mysql only, | |
| 13:14:31 | ratailor | bauzas, https://github.com/openstack/nova/commit/402b3abf990d08d2af8331079d36a92d84d84b80 | |
| 13:15:27 | ratailor | bauzas, similar problem was there, and he has changed collation type for mysql backend. | |
| 13:18:59 | ratailor | bauzas, you want to discuss anything else, it EOD here. Can we continue tomorrow, if anything is left ? | |
| 13:20:13 | bauzas | ratailor: for sure | |
| 13:20:30 | bauzas | ratailor: have a good evening | |
| 13:20:40 | ratailor | bauzas, same to you! | |
| 13:23:50 | gibi | mriedem: I reverted https://review.openstack.org/#/c/491808 top of https://review.openstack.org/#/c/498482 and I still see that the recovered source compute tries to delete the instance | |
| 13:25:18 | gibi | mriedem: so I think setting the migration to failed is the solution we need. Do you suggest to add an extra check to the destroy_evacuated_instances about that instance.host != CONF.host as well? | |
| 13:26:49 | gibi | hm, the bug disappeared from launchpad https://bugs.launchpad.net/nova/+bug/1713783 | |
| 13:26:51 | openstack | gibi: Error: malone bug 1713783 not found | |
| 13:29:49 | openstackgerrit | Bob Ball proposed openstack/nova master: XenAPI: Unit tests must mock os_xenapi calls https://review.openstack.org/499573 | |
| 13:30:02 | gibi | mriedem: but you were able to just update that bug. that is weird | |
| 13:31:02 | hrw | morning | |
| 13:31:56 | hrw | can someone tell me where nova defines default VM scheme? As 'hi libvirt, here I have some VM definition for you' way | |
| 13:35:48 | gibi | hrw: hi, I think this is the place https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L5170 | |
| 13:35:58 | hrw | gibi: thanks | |
| 13:56:22 | mriedem | gibi: one thing on the fix for that evacuate issue is, i'd like to backport that further than just pike, | |
| 13:56:30 | mriedem | but we don't have the ServerMovingTests stuff before pike | |
| 13:56:40 | mriedem | so backporting to ocata and newton will be difficult if we don't decouple those | |
| 13:56:50 | mriedem | it would be nice if the code fix patch is self-contained | |
| 14:01:49 | gibi | mriedem: would you like to have an independent regression test for that bug under tests/functional/regressions ? | |
| 14:01:59 | gibi | mriedem: that could be backportable | |
| 14:02:37 | mriedem | it's not really a regression | |
| 14:02:46 | mriedem | but yes a separate standalone test would be nice | |
| 14:02:59 | mriedem | it's just a latent bug from what i can tell | |
| 14:03:19 | mriedem | well, it was maybe regressed when the migration record stuff was changed to track evacuations, but that's pretty old now (liberty or mitaka i think) | |
| 14:03:46 | mriedem | so you could put it in tests/functional/regressions if you wanted, that's fine | |
| 14:03:50 | mriedem | just decouple from ServerMovingTests | |
| 14:06:33 | gibi | mriedem: OK, cool | |
| 14:08:03 | mriedem | i also need to look at splitting https://review.openstack.org/#/c/499399/ up a bit later today | |
| 14:08:08 | mriedem | since it's a pretty large change atm | |
| 14:10:50 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove deprecated 'null_kernel' opt https://review.openstack.org/499611 | |
| 14:10:51 | openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP! conf: Remove deprecated 'multi_instance_display_name_template' opt https://review.openstack.org/499612 | |
| 14:17:14 | efried | where da source for the placement API reference doc? https://developer.openstack.org/api-ref/placement/ | |
| 14:17:24 | efried | found typo, wanna fix. | |
| 14:17:54 | sdague | new ironic issue - https://bugs.launchpad.net/nova/+bug/1714248 ? | |
| 14:17:55 | openstack | Launchpad bug 1714248 in OpenStack Compute (nova) "Compute node HA for ironic doesn't work due to the name duplication of Resource Provider " [Undecided,New] | |
| 14:18:59 | efried | nova project, placement-api-ref/source/parameters.yaml? | |
| 14:19:09 | sdague | efried: yep | |
| 14:19:14 | efried | thx | |
| 14:19:25 | efried | starts_with => startswith, according to my playing around. | |
| 14:19:50 | efried | ...for GET /traits | |
| 14:21:24 | sdague | efried: cool | |
| 14:21:45 | sdague | the names of parameters is in the rst, so depending you might only need to change that side, not that parameters.yaml | |
| 14:22:23 | efried | sdague rgrep showed starts_with only in that yaml file, so... | |
| 14:23:00 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: [placement] Add test for empty resources in allocation https://review.openstack.org/499615 | |
| 14:23:01 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: [placement] Require at least one resource class in allocation https://review.openstack.org/499616 | |
| 14:23:21 | bauzas | mriedem: I'm still not at full steam, but how can I help by reviewing changes ? | |
| 14:23:38 | bauzas | mriedem_away: the above gibi's changes ? | |
| 14:24:23 | gibi | bauzas: that change need some work based on the above discussion to make it backportable | |
| 14:24:32 | bauzas | k | |
| 14:26:12 | efried | The list of links returned by GET /resource_providers doesn't have one for .../allocations. I think it should (because that URI responds). LP bug? | |
| 14:30:30 | gibi | bauzas: mriedem_away has a refactor here https://review.openstack.org/#/c/498596/ that needs a +2 | |