| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-18 | |||
| 22:06:52 | sean-k-mooney | edleafe: ofcouse efried just said so above :) | |
| 22:07:18 | sean-k-mooney | it would be eaiser to do some of this on the placement side however | |
| 22:07:19 | efried | oh, edleafe, is this *your* goat? | |
| 22:07:51 | mriedem | oh nice a stack overflow in unit tests http://logs.openstack.org/45/649345/7/check/openstack-tox-py36/ba15c17/job-output.txt.gz#_2019-04-18_18_10_53_423952 | |
| 22:07:55 | edleafe | Weighers in Placement - yay! | |
| 22:09:06 | sean-k-mooney | mriedem: so there is definetly an infinity loop between _get and __getattr__ in tha tpatch | |
| 22:09:25 | efried | but only in py36 | |
| 22:09:53 | mriedem | sean-k-mooney: i've seen this TestRPC class intermittently failing unit tests all over the gate today | |
| 22:09:56 | mriedem | it's not this patch | |
| 22:10:31 | sean-k-mooney | with the same oslo config issue? | |
| 22:10:48 | mriedem | maybe? | |
| 22:10:51 | mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22%7C%20Fatal%20Python%20error%3A%20Cannot%20recover%20from%20stack%20overflow.%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d | |
| 22:11:35 | mriedem | what did we merge on april 15? | |
| 22:12:32 | sean-k-mooney | in oslo config. apparently droping python 35 testing | |
| 22:12:46 | mriedem | ... | |
| 22:12:52 | mriedem | why would that matter in a nova py36 job? | |
| 22:13:02 | mriedem | especially if an oslo.config change is unreleased for us to use it | |
| 22:13:06 | sean-k-mooney | i assume this is a version bump in upper constratins or a new release? | |
| 22:13:25 | mriedem | https://github.com/openstack/requirements/commit/a96ee0e258aafb2880149b3e25dd5959f7b37c09 | |
| 22:13:31 | efried | sean-k-mooney: are you at the PTG through Saturday? | |
| 22:14:09 | sean-k-mooney | im flying there the friday before and leaving the Starday afternoon so yes | |
| 22:14:50 | sean-k-mooney | mriedem: oslo.config 4.11.2 came out 2 days ago | |
| 22:15:07 | efried | sean-k-mooney: I'm going to assert that the CI topic is one we don't need jaypipes for; and dtroyer is booked Th+Fr. So I'll set the CI topic for Saturday a.m., cool? | |
| 22:15:17 | sean-k-mooney | oh never mind | |
| 22:15:22 | sean-k-mooney | that is old | |
| 22:15:40 | sean-k-mooney | efried: ya that sound fine | |
| 22:15:46 | mriedem | we're using 6.8.1 from 2 months ago | |
| 22:16:14 | mriedem | i also started seeing seg faults in pip 3.6 installs yesterday, so maybe something in py36 on bionic nodes since april 15 | |
| 22:18:39 | mriedem | https://bugs.launchpad.net/nova/+bug/1825435 | |
| 22:18:40 | openstack | Launchpad bug 1825435 in OpenStack Compute (nova) "TestRPC unit tests intermittently fail with "'>' not supported between instances of 'NoneType' and 'datetime.datetime'"" [Undecided,Confirmed] | |
| 22:19:06 | aspiers | sean-k-mooney: I forgot to get your take on whether hugepages are still helpful for SEV, in terms of accounting | |
| 22:19:32 | sean-k-mooney | am i think they are | |
| 22:19:34 | aspiers | sean-k-mooney: I'm a little confused by recent discussions but I got the impression they can only be used for accounting guest RAM | |
| 22:19:42 | sean-k-mooney | mainly because you still have to lock the memory | |
| 22:19:48 | aspiers | and QEMU allocates a whole bunch of other stuff | |
| 22:20:13 | sean-k-mooney | yes it does for mmio/dma | |
| 22:20:42 | sean-k-mooney | with SEV instace there will be no oversubsription of memory | |
| 22:20:48 | sean-k-mooney | that is also true of hugepages | |
| 22:20:53 | aspiers | is it useful to only track part of what is allocated though? | |
| 22:21:10 | sean-k-mooney | yes which will be most of it | |
| 22:21:28 | sean-k-mooney | the qemu overhead is ment to be tracked using the resved meory config option | |
| 22:22:36 | aspiers | Yeah, that's the bit which danpb is suggesting should have an rlimit at the top /machine.slice cgroup I think? | |
| 22:22:37 | sean-k-mooney | or on a hugepage host its assumed you leave enough free non hugepage memeroy on the host for the os and any qemu memroy required outside of the guest ram | |
| 22:22:48 | sean-k-mooney | ya | |
| 22:22:50 | aspiers | i.e. reserving some memory for the host | |
| 22:23:01 | aspiers | I added some paras in the spec about this - please could you check them if you didn't already? | |
| 22:23:12 | sean-k-mooney | so my suggetion is still use the locked element but dont set the hard_limit | |
| 22:23:21 | aspiers | Yes, I already changed the spec to propose that | |
| 22:23:25 | sean-k-mooney | and preferably use hugepages | |
| 22:23:40 | aspiers | it pretty much says that too | |
| 22:23:50 | aspiers | Most of my recent edits were just copying your advice ;) | |
| 22:23:58 | sean-k-mooney | if you dont use hugepages we need to tell the operators to set the ram_allocatoin_raito to 1.0 | |
| 22:24:00 | aspiers | but I'm not 100% sure I got it all accurate | |
| 22:24:20 | sean-k-mooney | ill take a look at it before the ptg | |
| 22:24:25 | aspiers | cool thanks | |
| 22:27:56 | mriedem | "'>' not supported between instances of 'NoneType' and 'datetime.datetime'" seems to be showing up in stephen's cells v1 removal series first...i'm not sure what could have introduced that | |
| 22:28:40 | sean-k-mooney | did we merge any of that yet | |
| 22:28:46 | mriedem | yes | |
| 22:32:41 | sean-k-mooney | maybe the instance list change | |
| 22:34:30 | sean-k-mooney | maybe not im wondering if we change how we are handeling the sorted list | |
| 22:35:25 | sean-k-mooney | we used to call self._get_instances_by_filters and now we call instance_list.get_instance_objects_sorted | |
| 22:39:24 | sean-k-mooney | acutly no _get_instances_by_filters just calls instance_list.get_instance_objects_sorted | |
| 22:46:00 | sean-k-mooney | just skimmed through the few that have merged but nothing obvious jumped out at me | |
| 22:47:34 | efried | sean-k-mooney: Are you particularly motivated to discuss provider yaml at the PTG? | |
| 22:47:43 | sean-k-mooney | no | |
| 22:48:03 | sean-k-mooney | i mean we can but i didnt think many people were interested | |
| 22:48:17 | sean-k-mooney | and im not planning to work on it in train | |
| 22:48:18 | efried | Right, lack of interest had put it way on the back burner for me | |
| 22:48:27 | sean-k-mooney | ya same | |
| 22:48:47 | efried | It's not something we can discuss without jaypipes, so putting it at the back of the agenda is kinda pointless. I'm just going to scrub it, k? | |
| 22:49:32 | sean-k-mooney | sure | |
| 22:49:47 | sean-k-mooney | i think we have more pressing things for Train | |
| 22:49:57 | efried | right | |
| 23:00:08 | sean-k-mooney | looking at the etherpads i dont really have much propsoed anymore | |
| 23:00:26 | efried | sean-k-mooney: It's not a requirement for you to have stuff on there :) | |
| 23:00:35 | sean-k-mooney | ya i know | |
| 23:00:36 | efried | besides, I think the CI topic is plenty, don't you? | |
| 23:01:08 | sean-k-mooney | well the ci topic was not originally my idea | |
| 23:01:55 | sean-k-mooney | but i ment it more as what i will like be working in trains has shifited some what so i have not really need to propose much to discuss | |
| 23:03:53 | sean-k-mooney | thats actully a good thing as i need to still figure out what cyborg/plaement topics i shoudl attend | |
| 23:08:24 | efried | sean-k-mooney: It's not a bad thing if the stuff you will be working on is well enough understood that we don't need PTG time for it. | |
| 23:08:31 | efried | If we could do that for everything, we wouldn't need a PTG :) | |
| 23:08:37 | efried | gotta run o/ | |
| 23:08:49 | sean-k-mooney | o/ | |
| 23:20:42 | openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Re-approve AMD SEV support for Train https://review.openstack.org/641994 | |
| 23:55:30 | openstackgerrit | melanie witt proposed openstack/nova master: Create request spec, build request and mappings in one transaction https://review.openstack.org/586742 | |
| #openstack-nova - 2019-04-19 | |||
| 00:47:40 | openstackgerrit | Merged openstack/nova stable/queens: Update instance.availability_zone on revertResize https://review.openstack.org/648415 | |
| 01:00:43 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix ProviderUsageBaseTestCase._run_periodics for multi-cell https://review.openstack.org/641179 | |
| 01:00:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Improve CinderFixtureNewAttachFlow https://review.openstack.org/639382 | |
| 01:00:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1818914 https://review.openstack.org/641521 | |
| 01:00:46 | openstack | bug 1818914 in OpenStack Compute (nova) "Hypervisor resource usage on source still shows old flavor usage after resize confirm until update_available_resource periodic runs" [Low,In progress] https://launchpad.net/bugs/1818914 - Assigned to Matt Riedemann (mriedem) | |
| 01:00:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove unused context parameter from RT._get_instance_type https://review.openstack.org/641792 | |
| 01:00:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Update usage in RT.drop_move_claim during confirm resize https://review.openstack.org/641806 | |
| 01:00:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid https://review.openstack.org/614012 | |
| 01:00:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.openstack.org/614036 | |
| 01:00:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Add instance hard delete https://review.openstack.org/650984 | |
| 01:00:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add Instance.hidden field https://review.openstack.org/631123 | |
| 01:00:48 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add TargetDBSetupTask https://review.openstack.org/627892 | |
| 01:00:48 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask https://review.openstack.org/631581 | |
| 01:00:49 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Execute TargetDBSetupTask https://review.openstack.org/633853 | |