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

fix: Update the spec "du_dirs" to filterable #3384

Merged
merged 3 commits into from
Apr 18, 2022
Merged

fix: Update the spec "du_dirs" to filterable #3384

merged 3 commits into from
Apr 18, 2022

Conversation

huali027
Copy link
Contributor

@huali027 huali027 commented Apr 13, 2022

Signed-off-by: Huanhuan Li [email protected]

All Pull Requests:

Check all that apply:

  • Have you followed the guidelines in our Contributing document, including the instructions about commit messages?
  • Is this PR to correct an issue?
  • Is this PR an enhancement?

Complete Description of Additions/Changes:

fixes: #3383

@huali027
Copy link
Contributor Author

@xiangce, please help to review, thanks.

insights/specs/default.py Outdated Show resolved Hide resolved
@xiangce xiangce added the SPEC Change needs SPEC approval label Apr 13, 2022
Copy link
Contributor

@xiangce xiangce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And better rename the get_dirs to a prop-noun like dir_list.

insights/specs/datasources/get_dirs.py Outdated Show resolved Hide resolved
* Also raise SkipComponent if there are no filters

Signed-off-by: Huanhuan Li <[email protected]>
@xiangce xiangce merged commit 7d78fc5 into RedHatInsights:master Apr 18, 2022
xiangce pushed a commit that referenced this pull request Apr 18, 2022
* fix: Update the spec "du_dirs" to filterable

Signed-off-by: Huanhuan Li <[email protected]>

* Move the "du_dirs_list" datasource to the datasource directory

Signed-off-by: Huanhuan Li <[email protected]>

* Rename to "dir_list"

* Also raise SkipComponent if there are no filters

Signed-off-by: Huanhuan Li <[email protected]>
(cherry picked from commit 7d78fc5)
xiangce pushed a commit that referenced this pull request Sep 6, 2024
* fix: Update the spec "du_dirs" to filterable

Signed-off-by: Huanhuan Li <[email protected]>

* Move the "du_dirs_list" datasource to the datasource directory

Signed-off-by: Huanhuan Li <[email protected]>

* Rename to "dir_list"

* Also raise SkipComponent if there are no filters

Signed-off-by: Huanhuan Li <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SPEC Change needs SPEC approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update the spec "du_dirs" to filterable
2 participants