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

Init for backend switching action #388

Merged
merged 3 commits into from
Jan 12, 2024
Merged

Conversation

mfaizanse
Copy link
Member

@mfaizanse mfaizanse commented Jan 12, 2024

Description

Changes proposed in this pull request:

  • Added wait-until-build-succeeds action to centralize waiting for build prow job.
  • Added hello world for new backend switching job.

Related issue(s)

@mfaizanse mfaizanse requested review from the1bit and a team as code owners January 12, 2024 11:51
@kyma-bot kyma-bot added the area/ci Issues or PRs related to CI related topics label Jan 12, 2024
@mfaizanse mfaizanse added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. and removed area/ci Issues or PRs related to CI related topics labels Jan 12, 2024
@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jan 12, 2024
@kyma-bot kyma-bot added the area/ci Issues or PRs related to CI related topics label Jan 12, 2024
@mfaizanse mfaizanse closed this Jan 12, 2024
@mfaizanse mfaizanse reopened this Jan 12, 2024
@mfaizanse mfaizanse removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 12, 2024
@@ -0,0 +1,36 @@
name: e2e-on-k8s
Copy link
Contributor

Choose a reason for hiding this comment

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

why not just e2e?

Copy link
Member Author

@mfaizanse mfaizanse Jan 12, 2024

Choose a reason for hiding this comment

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

we have another workflow e2e-without-lifecycle-manager, which should be renamed to e2e IMO. Thats why I added the k8s to convey the msg that this workflow uses real k8s on gardener.

Copy link
Contributor

Choose a reason for hiding this comment

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

Is e2e-without-lifecycle-manager also running on gardener?

Copy link
Contributor

@marcobebway marcobebway Jan 12, 2024

Choose a reason for hiding this comment

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

How about we use the following prefixes when it make sense e2e-k3d e2e-gardener?

Copy link
Member Author

Choose a reason for hiding this comment

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

I will do it in next PR.

pull_request_target:
branches:
- main
- "release-*"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this understandable by gh-workflows? because it is not a valid regex.

Copy link
Member Author

Choose a reason for hiding this comment

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

I copied it from the other workflow. But I am sure atm, I will confirm this.

Copy link
Contributor

@marcobebway marcobebway left a comment

Choose a reason for hiding this comment

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

As this is an initial work for the coming PRs, we agreed to approve for now, and fix the review comments in the next PRs.

@kyma-bot kyma-bot added the lgtm Looks good to me! label Jan 12, 2024
@kyma-bot kyma-bot merged commit dc18409 into kyma-project:main Jan 12, 2024
15 checks passed
@grischperl grischperl modified the milestones: 1.0.3, 1.1.0 Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Issues or PRs related to CI related topics cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants