Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-01-08
17:05:52 openstackgerrit Merged openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/554212
17:07:01 sean-k-mooney jaypipes: ya, im so familar with the tuneing options at this point that i pick an chooes the ones that make sense when i need them
17:07:34 sean-k-mooney i normally have at least one of the numa feature enabled even if it is just hw:numa_nodes=1
17:08:37 sean-k-mooney oh the pending state spec has merged cool :)
17:09:00 sean-k-mooney the rest of the premptable instance work is in the reaper project right?
17:15:16 jaypipes yonglihe: k, reviewed the NUMA spec.
17:51:01 openstackgerrit Merged openstack/nova-specs master: Flavor Extra Spec and Image Properties Validation https://review.openstack.org/618542
18:07:26 openstackgerrit Chris Dent proposed openstack/nova master: Remove placement from nova install docs https://review.openstack.org/628184
18:07:26 openstackgerrit Chris Dent proposed openstack/nova master: Remove link to placement configuration from nova config docs https://review.openstack.org/628185
18:07:27 openstackgerrit Chris Dent proposed openstack/nova master: Remove placement from contributor doc https://review.openstack.org/628186
18:07:45 openstackgerrit Chris Dent proposed openstack/nova master: Switch to using os-resource-classes https://review.openstack.org/628278
18:24:04 openstackgerrit Stephen Finucane proposed openstack/nova master: Fix overcommit for NUMA-based instances https://review.openstack.org/629281
18:24:13 stephenfin sean-k-mooney, gibi, jaypipes: Ta-da ^
18:24:22 stephenfin Makes sense to me, anyway. Lemme know
18:34:13 sean-k-mooney stephenfin: its almost right comment inline. you messed the case fo explcit small pages e.g. hw:mem_page_size=small|4|4k
18:40:31 stephenfin sean-k-mooney: replied
19:20:51 lbragstad johnthetubaguy jaypipes o/ curious if either of you have thoughts on this since the last time we discussed it - http://lists.openstack.org/pipermail/openstack-discuss/2019-January/001518.html
19:52:27 openstackgerrit Paul-Emile Element proposed openstack/os-traits master: Add new traits for TPM support https://review.openstack.org/629299
20:19:20 mnaser oh yikes
20:19:34 mnaser how safe is it to change the admin_or_owner rule
20:19:59 mnaser to say, something like: "is_admin:True or (role:compute and project_id:%(project_id)s)"
20:20:04 openstackgerrit Jack Ding proposed openstack/nova master: Improve libvirt image and snapshot handling https://review.openstack.org/616692
20:51:58 openstackgerrit Jack Ding proposed openstack/nova-specs master: Fix format flavor-extra-spec-image-property-validation https://review.openstack.org/629311
22:07:16 yonglihe jay, thanks.
22:27:17 yonglihe jaypipes, sean-k-mooney do we need aligned about sean-k-mooney's suggestion? move some stuff out of the 'nodes' and a more granular policy rule.
22:35:29 openstackgerrit Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256
22:37:38 yonglihe jaypipes, update per your comments. i working on a more specific policy rule, that's fine to me.
23:15:13 openstackgerrit Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256
23:15:19 jaypipes yonglihe: I actually think no, it's best just left defaulted to admin-only policy.
23:15:37 jaypipes yonglihe: but I won't hold up the spec for that reason or anything
23:16:05 yonglihe ok, i need revert it. already add a fine control
23:16:44 jaypipes yonglihe: don't revert :)
23:16:51 jaypipes yonglihe: it's not a big deal
23:16:54 yonglihe ok
23:18:04 yonglihe nice to know your idea about which item cause you add -1
23:18:32 yonglihe really nice hint for review.
23:40:33 openstackgerrit Merged openstack/python-novaclient stable/rocky: Fix up userdata argument to rebuild. https://review.openstack.org/613057
23:45:03 openstackgerrit Merged openstack/python-novaclient stable/rocky: Follow up "Fix up userdata argument to rebuild" https://review.openstack.org/613086
23:45:51 openstackgerrit Hongbin Lu proposed openstack/nova-specs master: Scheduling support for SR-IOV bond https://review.openstack.org/626055
#openstack-nova - 2019-01-09
00:03:54 openstackgerrit Hongbin Lu proposed openstack/nova-specs master: Scheduling support for SR-IOV bond https://review.openstack.org/626055
00:36:31 openstackgerrit Merged openstack/nova stable/rocky: Fail to live migration if instance has a NUMA topology https://review.openstack.org/625880
02:06:03 openstackgerrit Merged openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/628109
02:06:09 openstackgerrit Merged openstack/nova stable/rocky: Fix a broken link https://review.openstack.org/626798
02:06:15 openstackgerrit Merged openstack/nova stable/rocky: Removing pip-missing-reqs from default tox jobs https://review.openstack.org/628952
02:06:24 openstackgerrit Merged openstack/nova stable/rocky: Fix destination_type attribute in the bdm_v2 documentation https://review.openstack.org/626874
02:25:59 takashin
04:07:05 openstackgerrit Takashi NATSUME proposed openstack/nova stable/queens: Add description of custom resource classes https://review.openstack.org/619125
04:07:10 openstackgerrit Takashi NATSUME proposed openstack/nova stable/queens: Add description of custom resource classes https://review.openstack.org/619125
07:38:32 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Amend the detach-boot-volume design https://review.openstack.org/619161
08:33:46 openstackgerrit Yikun Jiang proposed openstack/nova master: Per aggregate scheduling weight https://review.openstack.org/628163
08:58:43 gibi stephenfin: left a question in https://review.openstack.org/#/c/629281
09:23:51 sean-k-mooney gibi: i replied to your question. i think is a safe assumtion to make if they have enabled ram oversubcription that the smallest pagesize on the host is swappable and therefor can be over subsribed
09:24:34 sean-k-mooney the linux kernel does not allow you to reserve all memory on a node for hugepages so the smallest pagesize will always be the system default small page size
09:40:20 openstackgerrit Maciej Jozefczyk proposed openstack/nova master: Add fill_virtual_interface_list online_data_migration script https://review.openstack.org/614167
09:40:21 openstackgerrit Maciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal https://review.openstack.org/591607
09:41:20 maciejjozefczyk sean-k-mooney: I found a bug in https://review.openstack.org/#/c/614167
09:42:09 gibi sean-k-mooney: thanks for the explanation
09:48:19 openstackgerrit Jan Gutter proposed openstack/nova master: Convert vrouter legacy plugging to os-vif https://review.openstack.org/571325
09:51:29 kashyap Is this too verbose a message?
09:51:29 kashyap msg = _("Setting both 'live_migration_tunnelled' and"
09:51:30 kashyap "'live_migration_with_native_tls' at the same"
09:51:30 kashyap "time is invalid. If you have the relevant"
09:51:30 kashyap "libvirt and QEMU versions, and have TLS"
09:51:32 kashyap "configured in your environment, pick"
09:51:35 kashyap "'live_migration_with_native_tls'.")
09:53:02 kashyap stephenfin: ^
09:54:22 kashyap (Or anyone interested in wordsmithing)
09:55:37 gibi kashyap: I like this message but I'm not a native English speaker
09:56:18 kashyap I consider myself an "English geek" (although it's my 3rd language). I went for maximum clarity for non-native speakers.
09:56:28 kashyap Because: "explicit is better than implicit".
09:56:49 kashyap And I don't like error messages that tells me what is not valid (that I already see from the error); give me something actionable.
09:57:27 kashyap I'll go with the above (with the second occurrence of "have" removed), and will see if anyone can come up w/ something better.
09:59:18 sean-k-mooney kashyap: it reads fine to me. its hard to be too verbose when explaining that you have conflicting config options set
09:59:37 sean-k-mooney you even provided a recommendation on what option to prefer so i looks good to me too
09:59:49 kashyap Okido; thanks for the review.
09:59:54 sean-k-mooney maciejjozefczyk: was it related to the socket timeout or unrelated.
10:00:03 kashyap As I noted, my bar for error messages is: "give me something actionable"
10:00:19 kashyap It is really frustrating to just see "error: please figure it out"
10:00:42 sean-k-mooney you mean like "no valid host"
10:00:46 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Support native TLS for migration and disks over NBD https://review.openstack.org/625216
10:00:47 kashyap sean-k-mooney: Heh
10:01:06 sean-k-mooney its my favorite nova error message
10:01:17 kashyap Yeah, it's the most awful message.
10:01:25 kashyap One day, if I feel angry enough, I might clean them all up
10:03:00 sean-k-mooney well its availd error message but the issue is 90% of the time it because you hit the retry filter limit and you hit the retry filter because or a different error which was lost in the logs
10:03:22 kashyap Yeah, I recall that it's "best we could do"
10:35:43 maciejjozefczyk sean-k-mooney: nope, different topic
10:35:48 maciejjozefczyk already fixed in new patch
10:36:22 sean-k-mooney cool i had only tried the followon patch as i was testing on master so did not need the db change
10:38:50 maciejjozefczyk for me I needed to update about 7k vifs only for one db
11:08:54 nehaalhat gibi: Hi, in case of functional test cases, how nova sync with placement service? Do you any idea about this?
11:11:32 gibi nehaalhat: in functional test nova imports placement code via https://github.com/openstack/nova/blob/e3eb5f916580a9bab8f67b0fd685c6b3b23a97b7/nova/tests/functional/fixtures.py#L41
11:12:42 nehaalhat like aggregates created in nova, how they are synced in placement service?
11:22:28 nehaalhat gibi: and do you have idea how to mock a method from placement in nova functional test cases where placementfixture is used like: https://github.com/openstack/nova/blob/e3eb5f916580a9bab8f67b0fd685c6b3b23a97b7/nova/tests/functional/test_aggregates.py#L385
11:31:47 gibi nehaalhat: do you have a patch with a functional test which doesn't work without mocking something in placement?
11:32:08 gibi nehaalhat: in general mocking in functional test is discuraged
11:32:26 gibi nehaalhat: but I can look at your patch to understand more
11:34:21 nehaalhat gibi: No, i dont have a patch on community gerrit right now
11:35:02 gibi nehaalhat: let me try to dig out where nova sync the aggregates to placement...
11:35:18 nehaalhat ok

Earlier   Later