| *** Guest2352 is now known as prometheanfire | 00:34 | |
| *** prometheanfire is now known as Guest2640 | 00:35 | |
| *** Guest2640 is now known as prometheanfire | 00:36 | |
| *** rpittau|afk is now known as rpittau | 07:10 | |
| opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Reproducer unit test for bug 1860312 https://review.opendev.org/c/openstack/nova/+/802840 | 08:45 |
|---|---|---|
| opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Allow deletion of compute service with no compute nodes https://review.opendev.org/c/openstack/nova/+/802841 | 08:46 |
| opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: Reproducer unit test for bug 1860312 https://review.opendev.org/c/openstack/nova/+/802842 | 08:49 |
| opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: Allow deletion of compute service with no compute nodes https://review.opendev.org/c/openstack/nova/+/802843 | 08:49 |
| opendevreview | Artom Lifshitz proposed openstack/nova stable/ussuri: Reproducer unit test for bug 1860312 https://review.opendev.org/c/openstack/nova/+/802846 | 08:57 |
| opendevreview | Artom Lifshitz proposed openstack/nova stable/ussuri: Allow deletion of compute service with no compute nodes https://review.opendev.org/c/openstack/nova/+/802847 | 08:57 |
| opendevreview | Artom Lifshitz proposed openstack/nova stable/train: Reproducer unit test for bug 1860312 https://review.opendev.org/c/openstack/nova/+/802848 | 09:01 |
| opendevreview | Artom Lifshitz proposed openstack/nova stable/train: Allow deletion of compute service with no compute nodes https://review.opendev.org/c/openstack/nova/+/802849 | 09:01 |
| opendevreview | Qiu Fossen proposed openstack/nova master: Allow migrating PMEM's data https://review.opendev.org/c/openstack/nova/+/802225 | 09:56 |
| opendevreview | Merged openstack/placement master: Fix SQL query counting the number of individual consumers having allocations by only selecting the aggregated consumer_id column. https://review.opendev.org/c/openstack/placement/+/801419 | 11:07 |
| stephenfin | bauzas: lyarwood: Dead simple typo fix here if you have 5 seconds https://review.opendev.org/c/openstack/nova/+/800855 | 11:26 |
| bauzas | ack | 11:27 |
| bauzas | stephenfin: we need a test, right? | 11:27 |
| stephenfin | eh, I thought about it. didn't seem worth the effort | 11:28 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Remove 'stub_out_trusted_certs' https://review.opendev.org/c/openstack/nova/+/765432 | 11:47 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service https://review.opendev.org/c/openstack/nova/+/778696 | 11:48 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service https://review.opendev.org/c/openstack/nova/+/778696 | 11:50 |
| sean-k-mooney | ah they used & assuming the were appendign to an existing query string effectivly | 11:56 |
| sean-k-mooney | ya that fix is correct | 11:56 |
| sean-k-mooney | a unit test would be nice but im kindof with stephen. in this cae im not sure its required | 11:58 |
| sean-k-mooney | it should not be hard to add as a follow up if ye want i woudl be tempetd to say though a better follow up would be to extra this funciton form manage.py and put the resouce provider query into the SchedulerReportClient and then just use it form nova manage | 12:02 |
| sean-k-mooney | *extract | 12:03 |
| sean-k-mooney | nova manage should not really be wriging its own functions for interacting with placment it should just use common ones and focus on the cli aspect and the business logic of the thing its tryign to do | 12:04 |
| lyarwood | stephenfin: sorry was afk looks like it's already in | 12:14 |
| * lyarwood tries to look at the nova-lvm failures before his next call | 13:06 | |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Add test for bug #1936278 https://review.opendev.org/c/openstack/nova/+/802893 | 13:54 |
| stephenfin | bauzas: FYI ^ | 13:54 |
| bauzas | ta | 13:59 |
| stephenfin | bauzas: lyarwood: You think you folks could look at https://review.opendev.org/c/openstack/nova/+/773643 and the patch above it to get those nice docs fixes in? Trying to close out what I can :) | 14:16 |
| lyarwood | stephenfin: ack | 14:39 |
| opendevreview | Stephen Finucane proposed openstack/nova master: manager: Reduce unnecessary calls https://review.opendev.org/c/openstack/nova/+/795027 | 14:56 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Moved all calls from _ENFORCER.authorize to a separate _authorize method https://review.opendev.org/c/openstack/nova/+/749080 | 15:46 |
| *** rpittau is now known as rpittau|afk | 16:03 | |
| bauzas | sean-k-mooney: still around ? can't find where we uppercase the RC names in https://github.com/openstack/placement/blob/3c51c6413bfc316cad8bb9d50b6fbffc83069b79/placement/objects/resource_class.py#L33 | 16:38 |
| bauzas | the API accepts lowercasing for the name but I'm almost sure we do create the RC object with uppercase name | 16:39 |
| bauzas | just can't find where | 16:39 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Rename vgpu options to mdev https://review.opendev.org/c/openstack/nova/+/801607 | 16:56 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: DNM (yet) : Expose the mdev class https://review.opendev.org/c/openstack/nova/+/801743 | 16:56 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: WIP Provide the mdev class for every PCI device https://review.opendev.org/c/openstack/nova/+/802918 | 16:56 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: WIP Provide the mdev class for every PCI device https://review.opendev.org/c/openstack/nova/+/802918 | 17:00 |
| melwitt | bauzas: you likely already found it but it doesn't look like lowercase is allowed https://github.com/openstack/placement/blob/3c51c6413bfc316cad8bb9d50b6fbffc83069b79/placement/schemas/common.py#L17-L21 | 17:01 |
| sean-k-mooney | bauzas: sorry yes im still here | 17:21 |
| sean-k-mooney | bauzas: i think there is a normalise fucntion | 17:22 |
| sean-k-mooney | maybe this one https://github.com/openstack/placement/blob/3c51c6413bfc316cad8bb9d50b6fbffc83069b79/placement/util.py#L233 | 17:23 |
| sean-k-mooney | well that is just checkign the query stiring but i think there is a helper util for it | 17:23 |
| sean-k-mooney | bauzas: the api should not accpet lowercase | 17:25 |
| sean-k-mooney | bauzas: here |https://github.com/openstack/os-resource-classes/blob/145ed791d050725394dbd786a5d45600da656812/os_resource_classes/__init__.py#L94-L113 | 17:26 |
| sean-k-mooney | bauzas: its in os-resouce-classes | 17:26 |
| sean-k-mooney | that used to also be in placment https://github.com/openstack/placement/blob/1257198effa3b41d206d18356dded543810da6fd/nova/rc_fields.py#L58-L66 | 17:28 |
| sean-k-mooney | bauzas i think melwitt is right that the scema will rejct it if its not uppercase | 17:38 |
| sean-k-mooney | bauzas: and we will not do the conversion automatically | 17:38 |
| sean-k-mooney | but os-resource-class does provide teh function to do the normalisation which you should use | 17:38 |
| sean-k-mooney | we do use it in nova i belive | 17:38 |
| sean-k-mooney | yep we do https://github.com/openstack/nova/blob/d64edd3da2336a5c7c8f69cced45272cbaf638a9/nova/tests/functional/compute/test_resource_tracker.py#L529 | 17:39 |
| sean-k-mooney | well actully here https://github.com/openstack/nova/blob/052cf963583ab7c6bbe4fcbf7bfe69f8f6733bdb/nova/scheduler/utils.py#L304 | 17:40 |
| sean-k-mooney | but anytime we constoct a custom RC name we should use it really | 17:41 |
| opendevreview | Samuel proposed openstack/nova-specs master: Migrate Instance Between Projects https://review.opendev.org/c/openstack/nova-specs/+/802034 | 20:52 |
| opendevreview | Samuel proposed openstack/nova-specs master: Migrate Instance Between Projects https://review.opendev.org/c/openstack/nova-specs/+/802034 | 20:57 |
| *** amodi_ is now known as amodi | 21:31 | |
| opendevreview | Vlad Gusev proposed openstack/nova stable/train: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791807 | 22:01 |
| opendevreview | Vlad Gusev proposed openstack/nova stable/stein: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/802935 | 22:03 |
| opendevreview | Samuel proposed openstack/nova-specs master: Migrate Instance Between Projects https://review.opendev.org/c/openstack/nova-specs/+/802034 | 22:04 |
| opendevreview | melanie witt proposed openstack/nova master: Add stub unified limits driver https://review.opendev.org/c/openstack/nova/+/712137 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Assert quota related API behavior when noop https://review.opendev.org/c/openstack/nova/+/712140 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Make unified limits APIs return reserved of 0 https://review.opendev.org/c/openstack/nova/+/712141 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/c/openstack/nova/+/712139 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/c/openstack/nova/+/712143 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to unified limits https://review.opendev.org/c/openstack/nova/+/713498 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | 22:26 |
| opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | 22:26 |
| opendevreview | Vlad Gusev proposed openstack/nova stable/stein: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/802935 | 23:01 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!