Skip to content

Commit

Permalink
fix: correct cloudflare namespace & container (#215)
Browse files Browse the repository at this point in the history
#### Motivation

cloudflared was being deployed into the wrong namespace and using a
invalid container link

#### Modification

corrects the namespace and container url.

#### Checklist

_If not applicable, provide explanation of why._

- [ ] Tests updated
- [ ] Docs updated
- [ ] Issue linked in Title
  • Loading branch information
blacha authored Oct 31, 2023
1 parent bd529aa commit 1817138
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions infra/cdk8s.ts
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ async function main(): Promise<void> {
});

new Cloudflared(app, 'cloudflared', {
namespace: 'cloudflared',
tunnelId: ssmConfig.tunnelId,
tunnelSecret: ssmConfig.tunnelSecret,
tunnelName: ssmConfig.tunnelName,
Expand Down
2 changes: 1 addition & 1 deletion infra/charts/cloudflared.ts
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ export class Cloudflared extends Chart {
containers: [
{
name: 'cloudflared',
image: props.accountId + '.dkr.ecr.ap-southeast-2.amazonaws.com/eks:cloudflared-2023.8.2',
image: '019359803926.dkr.ecr.ap-southeast-2.amazonaws.com/cloudflared:2023.8.2',
args: ['tunnel', '--loglevel', 'trace', '--config', '/etc/cloudflared/config/config.yaml', 'run'],
volumeMounts: [
{ volume: kplus.Volume.fromConfigMap(this, 'mount-config', cm), path: '/etc/cloudflared/config' },
Expand Down

0 comments on commit 1817138

Please sign in to comment.