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

Rebase master #206

Closed
wants to merge 17 commits into from
Closed

Conversation

oceanc80
Copy link
Collaborator

Description of the change:
Rebase master

Motivation for the change:

Checklist

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

OchiengEd and others added 17 commits September 11, 2023 15:12
…mework#6546)

* Allow users to add annotation to set configure rollback

Allow users to add annotation, "helm.sdk.operatorframework.io/rollback-force", to set rollback
to false. Default value is true

Signed-off-by: Edmund Ochieng <ochienged@gmail.com>

* Update OPERATOR_SDK_VERSION in testdata Makefiles


* Add tests for function reading helm.sdk.operatorframework.io/rollback-force annotation


* Move rollback out of the UpgradeRelease method


* Print error whenever release upgrade fails

---------

Signed-off-by: Edmund Ochieng <ochienged@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ework/ansible-operator-plugins, remove ansible code (operator-framework#6583)

* swap to ansible-operator-plugins for the ansible plugin

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* remove ansible logic from the SDK repo

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* more ansible reference removals

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* go mod tidy

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* remove ansible specific steps from release guide

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

---------

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
…de affinity. (operator-framework#6592)

Signed-off-by: Jeremy Poulin <jaypoulz@gmail.com>
Signed-off-by: Catherine Chan-Tse <cchantse@redhat.com>
Bumping versions of dependencies for k8s 1.27

Signed-off-by: Varsha Prasad Narsing <varshaprasad96@gmail.com>
This commit fixes most of the OLM client and helm controller
code to accomodate the changes made in controller-runtime.

Signed-off-by: Varsha Prasad Narsing <varshaprasad96@gmail.com>
This commit fixes some of the dependency versions which need
to be pinned specifically in the project to be able to run
successfully. It also fixes the handler code base in ansible.

Signed-off-by: Varsha Prasad Narsing <varshaprasad96@gmail.com>
Making linter happy by replacing deprecated methods
whereever possible.

Signed-off-by: Varsha Prasad Narsing <varshaprasad96@gmail.com>
This commit re-generates the testdata based on the
changes made in respective plugins and adds changelog
for user facing changes.

Signed-off-by: Varsha Prasad Narsing <varshaprasad96@gmail.com>
Bump go version to 1.21 in CI to solve scaffolding issues.

Signed-off-by: Varsha Prasad Narsing <varshaprasad96@gmail.com>
Fix default plugins and OLM commands to include discovery errors.

Signed-off-by: Varsha Prasad Narsing <varshaprasad96@gmail.com>
Signed-off-by: Varsha Prasad Narsing <varshaprasad96@gmail.com>
* Trim deprecation warning before unmarshalling output.

Signed-off-by: Neo2308 <pradha.krishna.cse17@itbhu.ac.in>
Signed-off-by: Neo2308 <pradha.krishna.cse17@itbhu.ac.in>
@Neo2308
Copy link

Neo2308 commented Oct 30, 2023

@varshaprasad96 could you take a look at this?

@Neo2308
Copy link

Neo2308 commented Nov 17, 2023

@varshaprasad96 @oceanc80 what are the next steps here? Is there anything blocking this PR from getting merged?

@oceanc80 oceanc80 closed this Dec 14, 2023
@oceanc80 oceanc80 deleted the bump/k8s-1.27 branch December 14, 2023 19:01
@oceanc80
Copy link
Collaborator Author

@Neo2308 This work got redone/merged to the main SDK and ansible operator plugin repo as part of these PRs:

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.

None yet

8 participants