Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Track namespaces in options property #266

Merged
merged 1 commit into from
Apr 6, 2019

Conversation

rmweir
Copy link
Contributor

@rmweir rmweir commented Mar 28, 2019

Problem:
Requests associated with a particular project are not being filtered. Consequently, requests for projects that posses a fraction of total resources are taking longer than necessary to return.

Solution:
Norman now tracks and assigns project namespaces which are necessary for rancher to filter project requests.

Issues:
rancher/rancher#18870
rancher/rancher#18522
rancher/rancher#18295
rancher/rancher#16605

@rmweir rmweir force-pushed the bug/add-ns-filters-dep branch 4 times, most recently from d4ed930 to 2367690 Compare March 28, 2019 23:02
@ibuildthecloud
Copy link
Contributor

LGTM

@rmweir rmweir force-pushed the bug/add-ns-filters-dep branch 3 times, most recently from e4b7116 to 49a37e1 Compare April 1, 2019 19:14
types/server_types.go Outdated Show resolved Hide resolved
@rmweir rmweir force-pushed the bug/add-ns-filters-dep branch 2 times, most recently from 4e43e75 to fcbbc90 Compare April 1, 2019 23:04
Now, identifies and assigns namespaces. Prior, the
backend listed items belongng to all namespaces in
all projects no matter what because they were not
being tracked. This led to longer request times
for projects that contained a fraction of total
resources. Now, rancher can filter project
resource requests; times are improved and
largely dependent on resource amount for the given
project.
@rmweir rmweir force-pushed the bug/add-ns-filters-dep branch from 69862f5 to 4fcf8aa Compare April 1, 2019 23:22
@StrongMonkey
Copy link

LGTM

@cjellick cjellick merged commit c55b1ee into rancher:master Apr 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants