Skip to content
This repository has been archived by the owner on Aug 19, 2024. It is now read-only.

Make default operator configuration configurable not hardcoded #49

Merged
merged 8 commits into from
Nov 30, 2023

Conversation

gazarenkov
Copy link
Member

@gazarenkov gazarenkov commented Nov 28, 2023

Description

This PR allows Backstage operator to be configured with plain yaml files under config managed by Kustomize, instead of hardcoded yamls.
These yamls eventually become ConfigMap objects and mounted to the Controller's container.
This allow to:

  • simplify default configuration (yaml files are more readable) and config/ folder is more predictable place to configure
  • be able to change Backstage default configuration in cluster scope (by admin) changing the content of ConfigMap
  • potentially have more than one overlays to choose, like bare backstage vs janus showcase, OCP Route vs Ingress etc.

Which issue(s) does this PR fix or relate to

#6
and
#51

How to test changes / Special notes to the reviewer

This PR must not change existing behavior, so make test, make run, make deploy should work as usual

@gazarenkov gazarenkov changed the title [DRAFT] [DRAFT] Make default operator configuration configurable instead of hardcoded Nov 28, 2023
@gazarenkov gazarenkov changed the title [DRAFT] Make default operator configuration configurable instead of hardcoded Make default operator configuration configurable instead of hardcoded Nov 28, 2023
@gazarenkov gazarenkov changed the title Make default operator configuration configurable instead of hardcoded Make default operator configuration configurable not hardcoded Nov 28, 2023
Copy link
Member

@rm3l rm3l left a comment

Choose a reason for hiding this comment

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

Also, could you rebase and fix the conflicts? We can work together on some of those conflicts due to #27..

config/manager/kustomization.yaml Outdated Show resolved Hide resolved
controllers/backstage_controller.go Show resolved Hide resolved
controllers/backstage_controller.go Show resolved Hide resolved
config/rbac/kustomization.yaml Outdated Show resolved Hide resolved
# Conflicts:
#	.gitignore
#	controllers/backstage_controller.go
#	controllers/backstage_controller_test.go
#	controllers/backstage_deployment.go
#	controllers/backstage_service.go
#	controllers/local_db_deployment.go
#	controllers/local_db_storage.go
@gazarenkov gazarenkov merged commit 4ecaa15 into janus-idp:main Nov 30, 2023
3 checks passed
nickboldt added a commit that referenced this pull request Dec 5, 2023
Signed-off-by: nickboldt <nickboldt@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants