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

feat: Add combiner rhel for edge #3526

Merged
merged 10 commits into from
Sep 22, 2022
Merged

feat: Add combiner rhel for edge #3526

merged 10 commits into from
Sep 22, 2022

Conversation

wushiqinlou
Copy link
Contributor

@wushiqinlou wushiqinlou commented Sep 16, 2022

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:

Create a combiner to determine if it is an edge computing system.

@wushiqinlou wushiqinlou changed the title Add combiner rhel for edge Feat: Add combiner rhel for edge Sep 16, 2022
@wushiqinlou wushiqinlou changed the title Feat: Add combiner rhel for edge feat: Add combiner rhel for edge Sep 16, 2022
Signed-off-by: jiazhang <[email protected]>
insights/combiners/rhel_for_edge.py Outdated Show resolved Hide resolved
insights/combiners/rhel_for_edge.py Outdated Show resolved Hide resolved
Signed-off-by: jiazhang <[email protected]>
Signed-off-by: jiazhang <[email protected]>
Copy link
Contributor

@jobselko jobselko left a comment

Choose a reason for hiding this comment

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

Hi @wushiqinlou, I have a few comments, but probably nothing important. You can ignore them and resolve these discussions if you think they are unnecessary.

In the end, I am not the one who approves this PR :)

insights/combiners/rhel_for_edge.py Show resolved Hide resolved
insights/combiners/rhel_for_edge.py Outdated Show resolved Hide resolved
@skontar skontar self-requested a review September 22, 2022 06:52
@psachin psachin merged commit db2b6bc into RedHatInsights:master Sep 22, 2022
psachin pushed a commit that referenced this pull request Sep 22, 2022
* Add combiner rhel_for_edge

Signed-off-by: jiazhang <[email protected]>

* Update test

Signed-off-by: jiazhang <[email protected]>

* Update return format

Signed-off-by: jiazhang <[email protected]>

* Remove SkipComponent

Signed-off-by: jiazhang <[email protected]>

* Update description

Signed-off-by: jiazhang <[email protected]>

* Update description format

Signed-off-by: jiazhang <[email protected]>

* Update description format

Signed-off-by: jiazhang <[email protected]>

* Add note

Signed-off-by: jiazhang <[email protected]>

* Update long condition

Signed-off-by: jiazhang <[email protected]>

* Update condition format

Signed-off-by: jiazhang <[email protected]>

Signed-off-by: jiazhang <[email protected]>
(cherry picked from commit db2b6bc)
xiangce pushed a commit that referenced this pull request Sep 6, 2024
* Add combiner rhel_for_edge

Signed-off-by: jiazhang <[email protected]>

* Update test

Signed-off-by: jiazhang <[email protected]>

* Update return format

Signed-off-by: jiazhang <[email protected]>

* Remove SkipComponent

Signed-off-by: jiazhang <[email protected]>

* Update description

Signed-off-by: jiazhang <[email protected]>

* Update description format

Signed-off-by: jiazhang <[email protected]>

* Update description format

Signed-off-by: jiazhang <[email protected]>

* Add note

Signed-off-by: jiazhang <[email protected]>

* Update long condition

Signed-off-by: jiazhang <[email protected]>

* Update condition format

Signed-off-by: jiazhang <[email protected]>

Signed-off-by: jiazhang <[email protected]>
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.

6 participants