Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

[KOGITO-148] - Implementation of deploy command to CLI #35

Merged
merged 1 commit into from Aug 21, 2019

Conversation

ricardozanini
Copy link
Member

@ricardozanini ricardozanini commented Aug 19, 2019

See: https://issues.jboss.org/browse/KOGITO-148

This PR adds the option deploy to CLI, for example:

kogito deploy example-drools https://github.com/kiegroup/kogito-examples --context drools-quarkus-example --app kogito

The command above will deploy a new Kogito Service from https://github.com/kiegroup/kogito-examples in the context directory drools-quarkus-example. The OpenShift application will be named as example-drools and deployed in the namespace kogito.

Many thanks for submiting your Pull Request ❤️!

Please make sure that your PR meets the following requirements:

  • You have read the contributors guide
  • Pull Request title is properly formatted: [KOGITO-XYZ] Subject
  • Pull Request contains link to the JIRA issue
  • Pull Request contains description of the issue
  • Pull Request does not include fixes for issues other than the main ticket
  • Your feature/bug fix has a unit test that verifies it
  • You've tested the new feature/bug fix in an actual OpenShift cluster

@ricardozanini ricardozanini added the WIP 👷‍♂️ Pull Request is on work in progress label Aug 19, 2019
@ricardozanini ricardozanini marked this pull request as ready for review August 20, 2019 21:02
@ricardozanini
Copy link
Member Author

@bmozaffa @xiezhang7 @sutaakar could you guys please review this PR? This should be the last feature added to the operator this week. We're now focusing on testing, bug fixing, build binaries and images for 0.3 release.

Thanks!!

@ricardozanini ricardozanini added needs review 🔍 Pull Request that needs reviewers and removed WIP 👷‍♂️ Pull Request is on work in progress labels Aug 20, 2019
README.md Show resolved Hide resolved
Copy link
Contributor

@xieshenzh xieshenzh left a comment

Choose a reason for hiding this comment

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

@ricardozanini looks good. Thanks.

README.md Outdated Show resolved Hide resolved
@ricardozanini ricardozanini added ready 🚀 PR is ready to be merged and removed needs review 🔍 Pull Request that needs reviewers labels Aug 21, 2019
@ricardozanini
Copy link
Member Author

Merging since we're about to launch 0.3

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready 🚀 PR is ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants