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

feat: Using OpenShift routes for integration tests #2042

Open
wants to merge 69 commits into
base: main
Choose a base branch
from

Conversation

hamza-m-masood
Copy link
Contributor

Which problem does the PR fix?

related issue: https://github.com/camunda/distribution/issues/254

What's in this PR?

Modifying integration tests to use OpenShift routes.

Checklist

Please make sure to follow our Contributing Guide.

Before opening the PR:

  • In the repo's root dir, run make go.update-golden-only.
  • There is no other open pull request for the same update/change.
  • Tests for charts are added (if needed).
  • In-repo documentation are updated (if needed).

After opening the PR:

  • Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
  • Did all checks/tests pass in the PR?

aabouzaid and others added 30 commits June 11, 2024 03:09
…#1978)

* chore: update chart versions for:

- camunda 8.5
- camunda 8.4
- camunda 8.3
- camunda 8.2

* chore: update release notes

* chore: fix auto-generated golden values

* chore: get the latest released chart by app version

* chore: update the helm cli annotation

* chore: update release workflow

---------

Signed-off-by: Ahmed AbouZaid <[email protected]>
Use gsed instead of sed when running the openshift post-render script on mac
- support multi-version in the release workflow
- customize generate-chart-matrix action
- set ci snapshot version based on the chart dir name
- rename chart dir camunda-platform to camunda-platform-latest
- update generate-release-notes.sh to work with multi-version

---------

Signed-off-by: Ahmed AbouZaid <[email protected]>
* simplify getting the previous chart version in workflow-vars

* update working dir in chart-release-snapshot.yaml

* use unit test template

---------

Signed-off-by: Ahmed AbouZaid <[email protected]>
* feat(console): add auth vars

* feat(console): add auth vars

* feat(console): add tests

* fix(console): configmap test

* fix(console): tests

* fix(console): tests

* fix(console): tests

* fix(console): tests

* update golden files

---------

Co-authored-by: Ahmed AbouZaid <[email protected]>
* chore(deps): update camunda/console docker tag to v8.5.52

* chore(deps): post upgrade tasks - update golden files

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* chore(deps): update camunda/web-modeler docker tag to v8.5.3

* chore(deps): post upgrade tasks - update golden files

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
….0-debian-12-r11 (#1967)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
#1935)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
#1965)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* chore: bump camunda-platform chart version to 10.1.1

* chore: add generated files for camunda-platform 10.1.1

* chore: update values in values-latest.yaml

* chore: Generate version matrix

* chore: fix default chart name in generate-version-matrix.sh

---------

Signed-off-by: Ahmed AbouZaid <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
#1990)

* fix: unauthenticated external elasticsearch no longer forces passwords to be configured

* fix: fixup boolean expression

* test: adds operate tests for elasticsearch auth disabled
* chore: added alpha chart
* add snapshot

---------

Signed-off-by: Ahmed AbouZaid <[email protected]>
@hamza-m-masood hamza-m-masood marked this pull request as ready for review June 27, 2024 19:17
@hamza-m-masood
Copy link
Contributor Author

@aabouzaid I am finished with this values.yaml for OpenShift but I don't know how to incorporate it in our CI. There are three problems:

  1. OpenShift routes operator is not currently working in our CI
  2. I have hardcoded the namespace in the broker contact-point env vars. The name of the hardcoded namespace is test-ns
  3. Within the controller config, there is a secret referenced called keystore. I have to run a number of manual tasks in order to create this secret, listed here: https://github.com/camunda/distribution/issues/254#issuecomment-2192839213

Once these 3 problems are solved, we can incorporate routes in our CI.

Copy link
Member

@aabouzaid aabouzaid left a comment

Choose a reason for hiding this comment

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

@hamza-m-masood Let's sync about this PR to ensure it's in the correct shape.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants