Skip to content

ci: upgrade pr deployments workflow #8924

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

Merged
merged 75 commits into from
Aug 11, 2023
Merged
Changes from 1 commit
Commits
Show all changes
75 commits
Select commit Hold shift + click to select a range
2d92a15
create-pr
matifali Aug 6, 2023
3ffb0c8
add permission to create serviceaccounts
matifali Aug 6, 2023
24f78d5
refactor
matifali Aug 6, 2023
52faafc
lint
matifali Aug 6, 2023
8b6ba6d
only build if needed
matifali Aug 6, 2023
f8121dd
Discard changes to helm/templates/rbac.yaml
matifali Aug 6, 2023
c633fec
Discard changes to helm/values.yaml
matifali Aug 6, 2023
51413cc
refactor
matifali Aug 6, 2023
4d370f7
wip
matifali Aug 6, 2023
65f1e08
update
matifali Aug 7, 2023
b8e6eab
add volume
matifali Aug 7, 2023
b0aaa69
set KUBECONFIG env variable
matifali Aug 7, 2023
098e5e6
Merge branch 'main' into matifali/pr-deployment-namespace-access
matifali Aug 8, 2023
cc65580
remove `--skip-build` and add `--force`
matifali Aug 8, 2023
003caf0
typo
matifali Aug 8, 2023
24dc6ca
test
matifali Aug 8, 2023
3e95af7
use a custom template
matifali Aug 8, 2023
4ff0966
finalize
matifali Aug 8, 2023
5923471
Merge branch 'main' into matifali/pr-deployment-namespace-access
matifali Aug 8, 2023
c64a7b4
use `set -euo pipefail`
matifali Aug 8, 2023
3f8599b
use `envsubst`
matifali Aug 8, 2023
9fa3d67
fmt
matifali Aug 8, 2023
3403530
fix typos
matifali Aug 9, 2023
fdbdf20
add template
matifali Aug 9, 2023
df4d829
use the included template
matifali Aug 9, 2023
27c175c
update
matifali Aug 9, 2023
0ec9c8a
separate build and deploy
matifali Aug 9, 2023
219a44b
do not force certificate creation
matifali Aug 9, 2023
b0afe7d
Merge branch 'main' into matifali/pr-deployment-namespace-access
matifali Aug 9, 2023
c5cbd16
update docs
matifali Aug 9, 2023
2ac3378
fix template
matifali Aug 9, 2023
1f3c2e2
use HOSTNAME
matifali Aug 9, 2023
d3f57b0
use better condition
matifali Aug 9, 2023
296d352
test `service_account_name`
matifali Aug 9, 2023
de998be
make fmt
matifali Aug 9, 2023
0a9eeeb
remove volumes
matifali Aug 9, 2023
f637501
fix renaming
matifali Aug 9, 2023
a1f2b8e
remove comments
matifali Aug 9, 2023
55524c6
add prefix
matifali Aug 9, 2023
4835b29
`terraform init`
matifali Aug 9, 2023
9576a3f
remove comments
matifali Aug 9, 2023
56d582c
change username to coder
matifali Aug 9, 2023
459b898
update docs
matifali Aug 9, 2023
188dfc9
Merge branch 'main' into matifali/pr-deployment-namespace-access
matifali Aug 9, 2023
7c71058
test parameter fro CLI
matifali Aug 9, 2023
4c8ab01
update condition
matifali Aug 9, 2023
96888a2
update comment
matifali Aug 9, 2023
b198d33
remove comments
matifali Aug 9, 2023
780f51c
use tailscale derpmap
matifali Aug 10, 2023
e3dac1c
use updated path of helm chart
matifali Aug 10, 2023
832e237
refactor to run on push
matifali Aug 10, 2023
76dbe1b
fix
matifali Aug 10, 2023
2062915
update permissions
matifali Aug 10, 2023
5be2f02
update
matifali Aug 10, 2023
4f5abfd
test
matifali Aug 10, 2023
62bcb8e
remove BRANCH NAME
matifali Aug 10, 2023
8a5f3f8
fix permission
matifali Aug 10, 2023
667acef
update checkout action
matifali Aug 10, 2023
40c3b74
update conditions
matifali Aug 10, 2023
fb29a3c
refactor
matifali Aug 10, 2023
232bed4
fix commenting
matifali Aug 10, 2023
e18a2c7
fix tag lookup
matifali Aug 10, 2023
57567e4
fixup!
matifali Aug 10, 2023
1767a70
Merge branch 'main' into matifali/pr-deployment-namespace-access
matifali Aug 10, 2023
1f5a5be
remove extra condition
matifali Aug 10, 2023
1a94dbb
debug
matifali Aug 10, 2023
f3b56a3
Update pr-deploy.yaml
matifali Aug 10, 2023
0bc368a
use PR_HOSTNAME
matifali Aug 11, 2023
b69af83
quoting
matifali Aug 11, 2023
ab7be03
Merge branch 'main' into matifali/pr-deployment-namespace-access
matifali Aug 11, 2023
ef13b94
run when build is true
matifali Aug 11, 2023
5906e8e
set BUILD to true if forced
matifali Aug 11, 2023
49f79f3
simplify condition
matifali Aug 11, 2023
bb6cd55
update deploy conditions
matifali Aug 11, 2023
d3a4c98
do a force deploy
matifali Aug 11, 2023
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
remove volumes
  • Loading branch information
matifali committed Aug 9, 2023
commit 0a9eeebf4c911b5dc3b6d1fd13dd9e412a58d20c
24 changes: 12 additions & 12 deletions .github/pr-deployments/template/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -299,12 +299,12 @@ resource "kubernetes_deployment" "main" {
name = "home"
read_only = false
}
volume_mount {
mount_path = "/tmp/config"
sub_path = "kubeconfig"
name = "kubeconfig"
read_only = true
}
# volume_mount {
# mount_path = "/tmp/config"
# sub_path = "kubeconfig"
# name = "kubeconfig"
# read_only = true
# }
}

volume {
Expand All @@ -316,12 +316,12 @@ resource "kubernetes_deployment" "main" {
}

# mount kubeconfig secret as a volume to /home/coder/.kube/config
volume {
name = "kubeconfig"
secret {
secret_name = "coder-namespace-kubeconfig"
}
}
# volume {
# name = "kubeconfig"
# secret {
# secret_name = "coder-namespace-kubeconfig"
# }
# }

affinity {
// This affinity attempts to spread out all workspace pods evenly across
Expand Down