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

Fix CD for devel and feature branch #3835

Merged
merged 1 commit into from
Sep 23, 2023

Conversation

BacLuc
Copy link
Contributor

@BacLuc BacLuc commented Sep 23, 2023

This was not changed when switching from a scheduled job on devel
to a job which is executed when the label is set.

Before this was not a problem, but with #3803 this is leads to an error.

Successful deployment can be seen here: BacLuc#206
Successful deployment of devel can be seen here: https://github.com/BacLuc/ecamp3/actions/runs/6284222267
(the deployment failed because it conflicted with a deployment of ecamp/ecamp3, but the job id could be retrieved)

@BacLuc BacLuc added the deploy! Creates a feature branch deployment for this PR label Sep 23, 2023
@github-actions
Copy link

github-actions bot commented Sep 23, 2023

💥 Feature branch deployment failed

Name Link
🕒 Last attempted at Sat Sep 23 2023 16:05:20 GMT+0200
🔨 Latest commit e22704cacb9e000fcbf635e34910064bd291cc27
🔍 Latest deploy log

@BacLuc BacLuc removed the deploy! Creates a feature branch deployment for this PR label Sep 23, 2023
This was not changed when switching from a scheduled job on devel
to a job which is executed when the label is set.
@BacLuc BacLuc changed the title Fix ci for devel Fix CD for devel and feature branch Sep 23, 2023
@manuelmeister manuelmeister added the deploy! Creates a feature branch deployment for this PR label Sep 23, 2023
@@ -33,7 +33,7 @@ jobs:
id: job-url
with:
github_token: ${{ secrets.GITHUB_TOKEN }}
job_name: Upgrade or install deployment (${{ inputs.name }}, ${{ inputs.sha }}, ${{ inputs.pr_number }}) / Deploy to Kubernetes
job_name: Upgrade or install deployment / Deploy to Kubernetes
Copy link
Member

Choose a reason for hiding this comment

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

Why is this info not available anymore?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is available, but not used anymore for the job_name

@BacLuc BacLuc removed the deploy! Creates a feature branch deployment for this PR label Sep 23, 2023
@BacLuc
Copy link
Contributor Author

BacLuc commented Sep 23, 2023

feature branch deployment won't work until this is merged

@manuelmeister manuelmeister added this pull request to the merge queue Sep 23, 2023
Merged via the queue into ecamp:devel with commit a20d87c Sep 23, 2023
29 of 30 checks passed
@BacLuc BacLuc deleted the fix-ci-for-devel branch September 23, 2023 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants