Skip to content

Commit

Permalink
* Split the "ngf" tilt-resource into two resources, with new "ngf-ear…
Browse files Browse the repository at this point in the history
…ly" including the service-account and cluster-role[+binding]. This resolved a tilt-up error I was getting (ngf pod creation failed due to service-account not existing, even after resource redeploy), and also may fix the general issue (#309) of tilt-up stalling on the ngf resource. (we'll see)
  • Loading branch information
Venryx committed May 16, 2024
1 parent 3d6dfa8 commit d423e93
Showing 1 changed file with 10 additions and 3 deletions.
13 changes: 10 additions & 3 deletions Tilt/NGINX_Gateway.star
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,16 @@ def Start_NGINXGateway(g):
#"service.externalIPs={15.204.30.179}",
],
)
NEXT_k8s_resource_batch(g, [
{
"new_name": "ngf-early", "labels": ["gateway"],
"objects": [
"ngf-nginx-gateway-fabric:serviceaccount",
"ngf-nginx-gateway-fabric:clusterrole",
"ngf-nginx-gateway-fabric:clusterrolebinding",
],
},
])
NEXT_k8s_resource_batch(g, [
# we unfortunately can't group these atm; they are added at end of code in helm_remote.star
#{"workload": "nginx-gateway-fabric-crds-install", "labels": ["gateway"]},
Expand All @@ -37,9 +47,6 @@ def Start_NGINXGateway(g):
# commented; had two matches (with same exact id string!) on linux/rancher-desktop (2024-04-27)
#"nginxgateways.gateway.nginx.org:customresourcedefinition",
"nginx:gatewayclass",
"ngf-nginx-gateway-fabric:serviceaccount",
"ngf-nginx-gateway-fabric:clusterrole",
"ngf-nginx-gateway-fabric:clusterrolebinding",
"ngf-config:nginxgateway",
],
# This port-forward is only really necessary on Linux, when using docker-engine. (in other cases, k8s auto-creates PF for the load-balancer -- when accessing on the same machine anyway)
Expand Down

0 comments on commit d423e93

Please sign in to comment.