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

ENTESB-12330: Allow for specifying a custom resource on install #7410

Merged
merged 2 commits into from Dec 16, 2019

Conversation

phantomjinx
Copy link
Contributor

No description provided.

* install.go
 * Moves the custom-resource switch to the `install app` command

* install_app.go
 * Handles a custom resource being specified.
 * Loads it, checks its a syndesis schema, applies it

* Provides tests for providing a custom resource
…ripts

* Allows custom resource to be specified on the command line then pushed
  down for processing by the operator
@phantomjinx phantomjinx self-assigned this Dec 13, 2019
@pure-bot pure-bot bot added the pr/review-requested Use this if you want to have a review. pure-bot will prevent merging if set and no review given label Dec 13, 2019
@syndesisio-bot
Copy link

@phantomjinx The bot could not transition the ticket automatically, please update this Jira ticket manually: https://issues.redhat.com/browse/ENTESB-12330

@pull-request-size pull-request-size bot added the size/l Large label Dec 13, 2019
@@ -99,6 +99,7 @@ func New(parent *internal.Options) *cobra.Command {
},
}
app.PersistentFlags().StringVarP(&o.addons, "addons", "", "", "a comma separated list of addons that should be enabled")
Copy link
Contributor

Choose a reason for hiding this comment

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

Can't we remove the addons option now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Probably. I'll put it in on the TODO list.

@pure-bot
Copy link
Contributor

pure-bot bot commented Dec 16, 2019

Pull request approved by @lgarciaaco - applying pr/approved label

1 similar comment
@pure-bot
Copy link
Contributor

pure-bot bot commented Dec 16, 2019

Pull request approved by @lgarciaaco - applying pr/approved label

@pure-bot pure-bot bot added the pr/approved Applied by pure-bot when a review is approved label Dec 16, 2019
@pure-bot pure-bot bot merged commit fc58f4c into syndesisio:master Dec 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 1.9.x pr/approved Applied by pure-bot when a review is approved pr/review-requested Use this if you want to have a review. pure-bot will prevent merging if set and no review given size/l Large
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants