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

Run e2e test on BTP Manager installed from template.yaml #135

Closed
6 tasks done
PK85 opened this issue Jan 27, 2023 · 5 comments
Closed
6 tasks done

Run e2e test on BTP Manager installed from template.yaml #135

PK85 opened this issue Jan 27, 2023 · 5 comments
Assignees
Labels
area/tests Issues or PRs related to tests kind/feature Categorizes issue or PR as related to a new feature. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Comments

@PK85
Copy link
Contributor

PK85 commented Jan 27, 2023

Description

This ticket is about changing the way how to install BTP-Manager in our: https://github.com/kyma-project/btp-manager/blob/main/testing/go/e2e_test.go. Instead of make deploy we must install BTP Manager in the same way how Lifecycle Manager does.

Of course, testing Lifecycle Manager is not in our scope, which means we need to reuse ~~our generated template.yaml~ OCI image and out of that install BTP-Manager for testing purposes.

AC

  • 1) Generate module for PRs that results with template.yaml OCI image
    • 1.1) find a way where to store template.yaml for PR pipeline, that will be required for next steps this is not longer necessary since OCI image is stored in registry
  • 2) Adjust release pipeline to use the same way of storing template.yaml as in PR (Still keep template.yaml in the release itself) not valid any more, we are not storing template.yaml in PR
  • 3) download OCI image, unpack it, install
  • 4) describe in our documentation how and what is tested in our e2e tests
  • 5) try store OCI images and templates in the local registry (we use k3d for testing)
@PK85 PK85 added kind/feature Categorizes issue or PR as related to a new feature. 2023-Q1 area/tests Issues or PRs related to tests size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jan 27, 2023
@jaroslaw-pieszka jaroslaw-pieszka self-assigned this Feb 6, 2023
@jaroslaw-pieszka
Copy link
Contributor

During discussions with @PK85 and @piotrmiskiewicz we agreed to use OCI image only, not to use template.yaml

@jaroslaw-pieszka
Copy link
Contributor

jaroslaw-pieszka commented Feb 16, 2023

Prow job - creating OCI module image for PR (presubmit job) - merged

@jaroslaw-pieszka
Copy link
Contributor

jaroslaw-pieszka commented Feb 16, 2023

Initial version - stub of script creating the module image (via make invocation) - merged

@jaroslaw-pieszka
Copy link
Contributor

jaroslaw-pieszka commented Feb 16, 2023

PR

  • waiting for module image being pushed to registry in the pipeline
  • testing helm install
  • testing helm uninstall
  • applying prerequisites (namespace, priority class, sample operator)

@jaroslaw-pieszka
Copy link
Contributor

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tests Issues or PRs related to tests kind/feature Categorizes issue or PR as related to a new feature. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

No branches or pull requests

2 participants