8000 feat: ci to build new registry on push to main by bcpeinhardt · Pull Request #363 · coder/modules · GitHub
[go: up one dir, main page]
Skip to content
This repository was archived by the owner on May 15, 2025. It is now read-only.

feat: ci to build new registry on push to main #363

Merged
merged 21 commits into from
Dec 19, 2024
Merged

feat: ci to build new registry on push to main #363

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
f52b8ca
add ci to trigger build of registry-v2 init commit
bcpeinhardt Dec 11, 2024
3e2128c
fix mixedup scripts
bcpeinhardt Dec 11, 2024
df92e31
add current branch to branch triggers to test action
bcpeinhardt Dec 11, 2024
233775e
make deploy script executable. Action should still fail with secrets …
bcpeinhardt Dec 11, 2024
ca69c68
test with secrets as nonsense values to make sure they aren't logged …
bcpeinhardt Dec 11, 2024
1d9288a
alright let's try this out
bcpeinhardt Dec 11, 2024
0b73692
remove test branch from trigger branches list
bcpeinhardt Dec 11, 2024
374e8de
warnings
bcpeinhardt Dec 11, 2024
ee83bb2
add the fail flag to hide the body as well in case it contains the UR…
bcpeinhardt Dec 11, 2024
58ab394
use github action instead of script to run new builds
bcpeinhardt Dec 13, 2024
e86e50c
add this branch to workflow to trigger build
bcpeinhardt Dec 13, 2024
d9622e5
try id-token perm
bcpeinhardt Dec 13, 2024
c262bec
maybe this will do it
bcpeinhardt Dec 13, 2024
e8461fd
take that
bcpeinhardt Dec 13, 2024
e36e66b
I will best you google cloud
bcpeinhardt Dec 13, 2024
3b04b61
cmon you can do it
bcpeinhardt Dec 13, 2024
48b9b5b
maybe now
bcpeinhardt Dec 16, 2024
236439c
set project id
bcpeinhardt Dec 16, 2024
41a89e2
how bout now
bcpeinhardt Dec 16, 2024
86d144e
remove testing branch
bcpeinhardt Dec 19, 2024
019c690
Merge branch 'main' into bcpeinhardt/ci-to-build-new-registry-on-push…
bcpeinhardt Dec 19, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
add current branch to branch triggers to test action
  • Loading branch information
@bcpeinhardt
bcpeinhardt committed Dec 11, 2024
commit df92e319e9799084721cd3de7927c6dd0cd6bbab
1 change: 1 addition & 0 deletions .github/workflows/deploy-registry.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ on:
push:
branches:
- main
- bcpeinhardt/ci-to-build-new-registry-on-push-to-main
Copy link
Member

Choose a reason for hiding this comment

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

don't forget to take this out.

Suggested change
- bcpeinhardt/ci-to-build-new-registry-on-push-to-main

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah I'm still testing haha but thank you 🙏 putting this in draft until it's ready.


pull_request:
Copy link
Member

Choose a reason for hiding this comment

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

Do we want to deploy on each PR too?

Copy link
Collaborator Author
@bcpeinhardt bcpeinhardt Dec 12, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

No I should pull that out, was moving too quickly, good catch 🙏

workflow_dispatch:
Expand Down
0