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

[KOGITO-1396] Adjustments for 0.8.0 release #205

Merged
merged 1 commit into from Mar 12, 2020

Conversation

ricardozanini
Copy link
Member

@ricardozanini ricardozanini commented Mar 10, 2020

See:
https://issues.redhat.com/browse/KOGITO-1396

This commit contains small adjustments for the OperatorHub PR.

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 needs review 🔍 Pull Request that needs reviewers label Mar 10, 2020
@ricardozanini ricardozanini added this to the v0.8.0 milestone Mar 10, 2020
Copy link
Contributor

@radtriste radtriste left a comment

Choose a reason for hiding this comment

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

One comment though

# httpPort: 8080
# details about the kafka connection
kafka:
# let's leave this burden to KogitoInfra CR to deploy a new Kafka instance for us
useKogitoInfra: true
useKogitoInfra: false
Copy link
Contributor

Choose a reason for hiding this comment

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

why did you switch to false here ?
dataindex will need kafka and infinispan or ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Because of the CI on OperatorHub. Since they don't implement the dependency management there, the Data Index will fail to deploy and will fail the PR

Copy link
Contributor

Choose a reason for hiding this comment

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

oki

Copy link
Member Author

Choose a reason for hiding this comment

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

Although I'll work on this for the next version. The Operator will handle with this kind of failures and will have a nice status message.

@ricardozanini
Copy link
Member Author

OperatorHub PR: operator-framework/community-operators#1325

Copy link
Member

@MarianMacik MarianMacik 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 question about CRDs in the new scorecard script. Otherwise looks good.

hack/scorecard.sh Show resolved Hide resolved
Signed-off-by: Ricardo Zanini <zanini@redhat.com>
@ricardozanini ricardozanini changed the title [KOGITO-1386] Adjustments for 0.8.0 release [KOGITO-1396] Adjustments for 0.8.0 release Mar 11, 2020
Copy link
Member

@MarianMacik MarianMacik left a comment

Choose a reason for hiding this comment

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

@ricardozanini thank you!

@ricardozanini ricardozanini merged commit b033028 into apache:master Mar 12, 2020
@ricardozanini
Copy link
Member Author

OperatorHub PR merged.

@ricardozanini ricardozanini deleted the kogito-1396 branch September 6, 2021 14:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs review 🔍 Pull Request that needs reviewers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants