| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-16 | |||
| 17:42:28 | bauzas | from what I understood from the very-long-standing battle of allocation ratios is that some operators do care about having a grouping system for managing their fleet of computes | |
| 17:42:41 | sean-k-mooney | this one woudl requirte us to creat a sharing resouce provider per host aggreate | |
| 17:42:49 | bauzas | (even if that can be done programmatically by something else) | |
| 17:43:18 | bauzas | my old grandma' was sayin' : "if that works, don't touch it" | |
| 17:43:37 | bauzas | and loooots of ops do manage aggregates thru this filter | |
| 17:43:57 | sean-k-mooney | we have had custoemr bitten by this in the past as an fyi. specifcly the conflict betwwen the capablity filter and aggreate one | |
| 17:44:11 | bauzas | so unless we come up with a solid upgrade plan for replacing it with very simple abstractions, don't touch it | |
| 17:44:26 | sean-k-mooney | bauzas: yep agree | |
| 17:44:29 | bauzas | sean-k-mooney: we resolved it with namespaces, right? | |
| 17:44:33 | sean-k-mooney | yes | |
| 17:44:42 | sean-k-mooney | basicaly they were adding pinned=true | |
| 17:44:47 | bauzas | problem solved. | |
| 17:44:58 | sean-k-mooney | they just namespaced it | |
| 17:45:06 | bauzas | ++ | |
| 17:51:32 | dansmith | man, so busy this morning I missed out on 50% of my usual coffee consumption.. it must be TEOTWAWKI | |
| 17:53:23 | sean-k-mooney | i try to some degree contol my caffein intake including normaly not drinking coffee at the weekends but i can totally feel teh difference when i dont have any | |
| 17:53:55 | sean-k-mooney | given i only drink 1-2 cups a day i dont know if it woudl be more noticable if i drank more or less | |
| 17:55:32 | sean-k-mooney | if i drank more i think it would have less of an effect when i drank it but likely more of an effect when i didnt which is why i reduced my cafee intake in the first place | |
| 19:55:33 | openstackgerrit | melanie witt proposed openstack/nova stable/stein: Reset the cell cache for database access in Service https://review.opendev.org/720587 | |
| 20:39:53 | openstackgerrit | melanie witt proposed openstack/nova stable/rocky: Reset the cell cache for database access in Service https://review.opendev.org/720592 | |
| 20:56:27 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: Reset the cell cache for database access in Service https://review.opendev.org/720596 | |
| 20:56:57 | openstackgerrit | melanie witt proposed openstack/nova stable/rocky: Reset the cell cache for database access in Service https://review.opendev.org/720592 | |
| 21:40:58 | openstackgerrit | Merged openstack/nova master: libvirt: Remove VIR_DOMAIN_BLOCK_REBASE_RELATIVE flag check https://review.opendev.org/702021 | |
| 22:37:19 | openstackgerrit | Merged openstack/nova master: images: Make JSON the default output format of calls to qemu-img info https://review.opendev.org/711679 | |
| #openstack-nova - 2020-04-17 | |||
| 00:37:15 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add docs and releasenotes for BP policy-defaults-refresh https://review.opendev.org/720129 | |
| 00:40:10 | gmann | lbragstad: can you verify the steps for migration to new policy. I cannot find the keystone exact doc please let me know if there is any so that i can link that in my doc - https://review.opendev.org/#/c/720129/4..5/doc/source/configuration/policy-new-defaults.rst@227 | |
| 00:51:33 | openstackgerrit | melanie witt proposed openstack/nova master: Enable test_volume_backed_live_migration in tempest https://review.opendev.org/528104 | |
| 00:53:12 | lbragstad | gmann sounds good - i'm about to pick up the review i started earlier for https://review.opendev.org/#/c/720129/4/doc/source/configuration/policy-new-defaults.rst | |
| 00:53:39 | lbragstad | i'll take a look at the new patch set | |
| 01:01:59 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage db purge to nova-audit https://review.opendev.org/708783 | |
| 01:02:01 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage cell_v2 discover_hosts to nova-manage https://review.opendev.org/708785 | |
| 01:02:01 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage db archive_deleted_rows to nova-audit https://review.opendev.org/708784 | |
| 01:02:02 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage placement sync_aggregates to nova-audit https://review.opendev.org/708787 | |
| 01:02:02 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage cell_v2 map_instances to nova-audit https://review.opendev.org/708786 | |
| 01:02:03 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage placement heal_allocations to nova-audit https://review.opendev.org/708788 | |
| 01:20:21 | openstackgerrit | Merged openstack/nova-specs master: Cleanup py27 support https://review.opendev.org/718368 | |
| 02:03:08 | openstackgerrit | Brin Zhang proposed openstack/nova master: Accurately clean up ARQs resources during build_instances in conductor https://review.opendev.org/720439 | |
| 02:07:11 | openstackgerrit | Brin Zhang proposed openstack/nova master: Accurately clean up ARQs resources during build_instances in conductor https://review.opendev.org/720439 | |
| 02:23:46 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Re-proposes 'Proposal for a safer remote console with password authentication' https://review.opendev.org/719720 | |
| 02:31:52 | openstack | Launchpad bug 1841932 in OpenStack Compute (nova) "hide_hypervisor_id extra_specs in nova flavor cannot pass AggregateInstanceExtraSpecsFilter" [Undecided,New] | |
| 02:31:52 | rambo_li | sean-k-mooney: maybe their is an error, for the bug :https://bugs.launchpad.net/nova/+bug/1841932. when we set the metadata "hide_hypervisor_id=true"in aggreate,so we will go to the https://github.com/openstack/nova/blob/46a3bcd80b41e99ec4923c7cf3d0f8dd8505e97c/nova/scheduler/filters/aggregate_instance_extra_specs.py#L69-L71, but this should not break, because the aggregate_val doesn't match the req. | |
| 02:48:16 | rambo_li | maybe we should delete the for loop in https://github.com/openstack/nova/blob/46a3bcd80b41e99ec4923c7cf3d0f8dd8505e97c/nova/scheduler/filters/aggregate_instance_extra_specs.py#L69 | |
| 02:51:32 | openstackgerrit | Merged openstack/os-traits master: Cleanup py27 support https://review.opendev.org/719350 | |
| 03:06:34 | openstackgerrit | Brin Zhang proposed openstack/nova master: Accurately clean up ARQs resources during build_instances in conductor https://review.opendev.org/720439 | |
| 06:36:27 | openstack | Launchpad bug 1873387 in OpenStack Compute (nova) "Unable to delete instance because cyborg.get_client() failed" [Medium,Confirmed] - Assigned to Brin Zhang (zhangbailin) | |
| 06:36:27 | brinzhang | gibi, bauzas: I open a new bug https://bugs.launchpad.net/nova/+bug/1873387, please check while you are wake up. | |
| 06:45:37 | gibi | brinzhang: thanks for finding that bug | |
| 06:46:27 | gibi | brinzhang: I marked it as ussuri-rc-potential as it is a new regression in Ussuri | |
| 06:51:08 | brinzhang | gibi: Got it | |
| 06:51:11 | brinzhang | thanks | |
| 07:29:59 | openstack | bug 1872934 in OpenStack Compute (nova) "Accurately clean up ARQs resources during build_instances() in conductor" [Low,In progress] https://launchpad.net/bugs/1872934 - Assigned to Brin Zhang (zhangbailin) | |
| 07:29:59 | brinzhang | gibi: for bug 1872934 we pushed the fix patch https://review.opendev.org/#/c/720439/, please review if you are free | |
| 09:17:44 | openstackgerrit | Wenping Song proposed openstack/nova master: handle to get cyborg client exception when delete instance https://review.opendev.org/720670 | |
| 10:05:47 | hrw | lyarwood: replied to your comment ;D | |
| 10:11:07 | openstackgerrit | Wenping Song proposed openstack/nova master: Handle to get cyborg client exception when delete instance https://review.opendev.org/720670 | |
| 10:15:19 | lyarwood | hrw: /me rolls eyes | |
| 10:15:42 | lyarwood | hrw: just a suggestion | |
| 10:19:16 | hrw | lyarwood: a bit of pure sarcasm in a morning | |
| 10:19:42 | lyarwood | :) I've not had enough coffee for that yet | |
| 10:19:55 | hrw | I just finished second | |
| 10:26:33 | lyarwood | just starting mine, tgif++ | |
| 10:26:49 | lyarwood | https://review.opendev.org/#/c/669674/ - could use a non-RH core on this given it has an impact across virt drivers. | |
| 11:52:37 | averi | hi, anyone has an idea on why code changes are not picked up by nova_compute containers (OSP 13)? | |
| 11:53:06 | averi | even when spinning up a new image based on the nova-compute one, it doesn't work, code changes are there but they're like ignored all together | |
| 11:53:32 | averi | even killing one of the library files results in the binary not to fail like the file was still around | |
| 12:47:55 | gibi | dansmith, sean-k-mooney, brinzhang: I need a second opinion in https://review.opendev.org/#/c/720670 | |
| 12:51:26 | sean-k-mooney | gibi: well my view is we should have been storing the acell profile in the request spec or in the instnace like we cache the flavor so we dont have this proble but ill go read what the patch acatully does now | |
| 12:52:27 | gibi | sean-k-mooney: it is like create an instance with accel, then remove cyborg service from deployment, then try to delete the instance which will fail | |
| 12:52:47 | sean-k-mooney | gibi: im kind of felling like we should do what ever we would do if cinder or neutron went away | |
| 12:53:01 | sean-k-mooney | i would assume we would fail to delete but i confess i have never tried it | |
| 12:53:20 | gibi | I can try that | |
| 12:54:27 | sean-k-mooney | cool you should jsut need to delete the entry in the service in keystone to emulate that right | |
| 12:55:19 | gibi | I'm not sure that a) this is a valid scenarios that cyborg service is removed from the deployment b) if yes then which is more important in general keeping cyborg - nova in sync or allowing to delete an instance during a temporary failure of cyborg | |
| 12:56:15 | sean-k-mooney | gibi: i am leaning towords this si operator error for removing a service before its consumer are removed | |
| 12:56:33 | sean-k-mooney | that said i dont think they should have to do db surgury to remove the instance | |
| 13:01:59 | gibi | sean-k-mooney: without neutron endpoint in keystone the instance delete fails with HTTP 500 | |
| 13:02:02 | gibi | http://paste.openstack.org/show/792326/ | |
| 13:02:18 | sean-k-mooney | ack | |
| 13:02:27 | sean-k-mooney | ya that is baskcially what i expected | |
| 13:02:43 | sean-k-mooney | and i would expect the same for cyborg | |
| 13:02:45 | gibi | so this also points towards that the scenario of removing cyborg is invalid | |
| 13:03:08 | sean-k-mooney | yes i was going to comment with a proceedure to do that in the bug | |
| 13:03:15 | sean-k-mooney | but just removing it i think is invalid | |
| 13:03:20 | sean-k-mooney | *patch | |
| 13:05:07 | openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM - Test stable device rescue tests with BFV instances https://review.opendev.org/710050 | |
| 13:06:17 | sean-k-mooney | gibi: left a comment with my toughts | |
| 13:06:21 | gibi | sean-k-mooney: thanks | |
| 13:38:25 | gibi | sean-k-mooney, dansmith thanks for the reviews on the https://review.opendev.org/#/c/720670 I marked the bug invalid | |
| 13:38:50 | dansmith | cool | |
| 13:48:21 | sean-k-mooney | dansmith: gibi cross cell migrate was complete this cycle right it was not supported in train | |
| 13:48:42 | sean-k-mooney | well cross cell resize. same thing | |
| 13:48:46 | averi | dansmith, sean-k-mooney: sorry for the direct poke, got any pointer on the question above? | |
| 13:49:15 | sean-k-mooney | i did not see the quest can you repeat it. i can check scollback | |
| 13:49:20 | averi | ah sure, sorry | |
| 13:49:26 | averi | <averi> even killing one of the library files results in the binary not to fail like the file was still around | |
| 13:49:26 | averi | <averi> even when spinning up a new image based on the nova-compute one, it doesn't work, code changes are there but they're like ignored all together | |
| 13:49:26 | averi | <averi> hi, anyone has an idea on why code changes are not picked up by nova_compute containers (OSP 13)? | |
| 13:49:51 | averi | I cannot troubleshooting anything as soon as oslo service comes into play | |
| 13:50:35 | sean-k-mooney | how are you modifying the code and restarting thet container | |
| 13:50:40 | sean-k-mooney | are you using docker directly | |