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.

Sign up for GitHub

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

Add service and ingress resources #33

Closed
wants to merge 3 commits into from
Closed

Add service and ingress resources #33

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6fff8d2
Adding the definition about the Service and Ingress kubernetes resour…
cmoulliard Oct 17, 2023
d6b26df
Changing localhost host name to https://backstage.idpbuilder.cnoe.io.…
cmoulliard Oct 17, 2023
b906954
Switching the DNS name from cnoe.io.local to cnoe.io.localtest.me
cmoulliard Oct 18, 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
44 changes: 40 additions & 4 deletions pkg/apps/srv/backstage/install.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -98,13 +98,49 @@ spec:
image: public.ecr.aws/cnoe-io/backstage:v0.0.3
imagePullPolicy: IfNotPresent
ports:
- name: http
- name: backend
containerPort: 7007
volumeMounts:
- mountPath: /app/config
name: backstage-config
readOnly: true
---
---
apiVersion: v1
kind: Service
metadata:
name: backstage
namespace: backstage
spec:
ports:
- name: http-backend
port: 7007
protocol: TCP
targetPort: backend
selector:
app: backstage
sessionAffinity: None
type: ClusterIP
---
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
name: backstage
namespace: backstage
spec:
ingressClassName: nginx
rules:
- host: backstage.idpbuilder.cnoe.io.local
http:
paths:
- backend:
service:
name: backstage
port:
number: 7007
path: /
pathType: Prefix
---
apiVersion: v1
kind: ConfigMap
metadata:
Expand All @@ -116,17 +152,17 @@ data:
app-config.yaml: |
app:
title: CNOE
baseUrl: http://localhost:3000
baseUrl: "https://backstage.idpbuilder.cnoe.io.local:8443"
Copy link
Collaborator

Choose a reason for hiding this comment

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

These needs to be either local.me or local.cnoe.io, depending on which one we use in #29

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I suggest to change all the URLs (including what we have here + documentation when a PR will be created to fix issue 29

Copy link
Contributor

Choose a reason for hiding this comment

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

Yep - we're going to hopefully merge #29 asap which does this, we should change it here assuming we merge that one first.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I updated the DNS name to use cnoe.io.localtest.me @greghaynes

organization:
name: CNOE
backend:
baseUrl: http://localhost:7007
baseUrl: "https://backstage.idpbuilder.cnoe.io.local:8443"
listen:
port: 7007
csp:
connect-src: ["'self'", 'http:', 'https:']
cors:
origin: http://localhost:3000
origin: "https://backstage.idpbuilder.cnoe.io.local:8443"
methods: [GET, HEAD, PATCH, POST, PUT, DELETE]
credentials: true
database:
Expand Down
Loading