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

ansible and helm packages should be useful libraries #1307

Closed
mhrivnak opened this issue Apr 9, 2019 · 3 comments
Closed

ansible and helm packages should be useful libraries #1307

mhrivnak opened this issue Apr 9, 2019 · 3 comments
Labels
design kind/feature Categorizes issue or PR as related to a new feature. language/ansible Issue is related to an Ansible operator project language/helm Issue is related to a Helm operator project lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.
Milestone

Comments

@mhrivnak
Copy link
Member

mhrivnak commented Apr 9, 2019

Feature Request

Relevant use cases as an operator author:

  • I want to augment reconciliation logic with go code that runs before or after my ansible or helm based reconciliation logic.
  • I want to trigger reconciliation from a goroutine, such as from a Channel source.

These use cases target someone who has created a go-based operator and wants to add reconciliation behavior by using ansible or helm.

Describe the solution you'd like
Proposals need to be made to turn the ansible and helm packages into first-class libraries, with documentation, that can be used to add behavior to a go-based controller.

Open Questions
The existing Helm (and possibly Ansible) libraries export most of their types and functions. As we approach the SDK 1.0.0 release:

  • Which of those types and functions should we unexport to give us more flexibility to make changes in the future?
  • Which do we need to continue to export to support these use cases?
    • Answer: We should expose pretty much everything except the reconciler.
@lilic lilic added language/ansible Issue is related to an Ansible operator project language/helm Issue is related to a Helm operator project kind/feature Categorizes issue or PR as related to a new feature. labels Apr 12, 2019
@joelanford joelanford added this to the 1.0.0 milestone Apr 23, 2019
@openshift-bot
Copy link

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@openshift-ci-robot openshift-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jul 22, 2019
@hasbro17
Copy link
Contributor

/lifecycle frozen

@openshift-ci-robot openshift-ci-robot added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Jul 22, 2019
@joelanford joelanford removed this from the 1.0.0 milestone Jan 10, 2020
@estroz estroz added this to the Next milestone Mar 2, 2020
@theishshah theishshah added the priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. label Jan 12, 2022
@varshaprasad96
Copy link
Member

We have released SDK 1.0 a while back, and also have respective code-bases split into useful libraries. Closing this issue since this has been open for a while.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design kind/feature Categorizes issue or PR as related to a new feature. language/ansible Issue is related to an Ansible operator project language/helm Issue is related to a Helm operator project lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.
Projects
None yet
Development

No branches or pull requests

10 participants