Skip to content

internal/plugins: add manifests and scorecard plugin keys to all projects#4419

Merged
estroz merged 2 commits intooperator-framework:masterfrom
estroz:chore/version-phase2-plugins
Feb 2, 2021
Merged

internal/plugins: add manifests and scorecard plugin keys to all projects#4419
estroz merged 2 commits intooperator-framework:masterfrom
estroz:chore/version-phase2-plugins

Conversation

@estroz
Copy link
Copy Markdown
Member

@estroz estroz commented Jan 21, 2021

Description of the change:

  • internal/plugins: add manifests and scorecard plugin keys to ansible/v1, helm/v1, and go/v2 projects on init, and consider these keys when running create api

Motivation for the change: All phase 2 plugins should have keys in the plugins config field. This (backwards-compatible) change makes it so.

Signed-off-by: Eric Stroczynski ericstroczynski@gmail.com

Checklist

If the pull request includes user-facing changes, extra documentation is required:

@estroz estroz force-pushed the chore/version-phase2-plugins branch from 0b5d3e1 to 36cc6f3 Compare January 21, 2021 23:37
Comment thread changelog/fragments/phase-2-plugin-keys.yaml Outdated
Comment thread changelog/fragments/phase-2-plugin-keys.yaml Outdated
Comment thread changelog/fragments/phase-2-plugin-keys.yaml Outdated
Comment thread changelog/fragments/phase-2-plugin-keys.yaml Outdated
Copy link
Copy Markdown
Contributor

@camilamacedo86 camilamacedo86 left a comment

Choose a reason for hiding this comment

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

Just a few nits to make the changelog follow up its convention/standard and make it more user friendly and comprehensive to the end-users. Otherwise, It shows great.

@estroz estroz force-pushed the chore/version-phase2-plugins branch from 36cc6f3 to 8efd265 Compare January 25, 2021 21:02
@estroz
Copy link
Copy Markdown
Member Author

estroz commented Jan 26, 2021

/assign @theishshah

Comment thread testdata/go/v2/memcached-operator/PROJECT
Copy link
Copy Markdown
Contributor

@camilamacedo86 camilamacedo86 left a comment

Choose a reason for hiding this comment

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

In POV the changelog fragment could be a little more straightforward for end-users and then, shows that for go/v2 it is a change not additional. However, I am fine with this one. Shows a good idea to make all follow the same standard.

/lgtm

Eric Stroczynski added 2 commits February 2, 2021 11:53
to ansible/v1, helm/v1, and go/v2 projects on init, and consider
these keys when running `create api`

Signed-off-by: Eric Stroczynski <ericstroczynski@gmail.com>
Signed-off-by: Eric Stroczynski <ericstroczynski@gmail.com>
@estroz estroz force-pushed the chore/version-phase2-plugins branch from 8efd265 to 22cd1d7 Compare February 2, 2021 19:53
@estroz estroz merged commit 427cf04 into operator-framework:master Feb 2, 2021
@estroz estroz deleted the chore/version-phase2-plugins branch February 2, 2021 20:11
reinvantveer pushed a commit to reinvantveer/operator-sdk that referenced this pull request Feb 4, 2021
…projects (operator-framework#4419)

Signed-off-by: Rein van 't Veer <reinvantveer@gmail.com>
reinvantveer pushed a commit to reinvantveer/operator-sdk that referenced this pull request Feb 4, 2021
…projects (operator-framework#4419)

Signed-off-by: Rein van 't Veer <reinvantveer@gmail.com>
reinvantveer pushed a commit to reinvantveer/operator-sdk that referenced this pull request Feb 5, 2021
…projects (operator-framework#4419)

Signed-off-by: Rein van 't Veer <reinvantveer@gmail.com>
reinvantveer pushed a commit to reinvantveer/operator-sdk that referenced this pull request Feb 5, 2021
…projects (operator-framework#4419)

Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
reinvantveer pushed a commit to reinvantveer/operator-sdk that referenced this pull request Feb 5, 2021
…projects (operator-framework#4419)

Signed-off-by: Rein van 't Veer <reinvantveer@gmail.com>
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
waynesun09 added a commit to waynesun09/reportportal-operator that referenced this pull request Apr 21, 2021
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.

3 participants