Skip to content

Comments

feat: consume environment services from confimap#321

Merged
shreddedbacon merged 6 commits intomainfrom
services-from-configmap
Feb 19, 2026
Merged

feat: consume environment services from confimap#321
shreddedbacon merged 6 commits intomainfrom
services-from-configmap

Conversation

@shreddedbacon
Copy link
Member

@shreddedbacon shreddedbacon commented Sep 7, 2025

Checklist

  • Affected Issues have been mentioned in the Closing issues section
  • Documentation has been written/updated
  • PR title is ready for changelog and subsystem label(s) applied

Changes the remote-controller to retrieve information about the services in an environment from a saved configmap. This will require changes for a few components before it can be rolled out

@shreddedbacon shreddedbacon force-pushed the services-from-configmap branch from a382111 to 21adb49 Compare February 9, 2026 20:44
@shreddedbacon
Copy link
Member Author

shreddedbacon commented Feb 9, 2026

I'm going to change this to be backwards compatible too. Will change to prefer configmap if present, and if not, fall back to previous behaviour.

@shreddedbacon shreddedbacon force-pushed the services-from-configmap branch from 32a9098 to 3e31b18 Compare February 11, 2026 03:25
@shreddedbacon
Copy link
Member Author

I'll merge uselagoon/machinery#99 and do a machinery release before merging this PR, so that it isn't using a commit hash 😄

@shreddedbacon shreddedbacon force-pushed the services-from-configmap branch from af53dee to 3428f56 Compare February 18, 2026 01:12
@shreddedbacon shreddedbacon force-pushed the services-from-configmap branch from 3428f56 to 7b0d727 Compare February 18, 2026 01:17
@shreddedbacon shreddedbacon merged commit dea58dd into main Feb 19, 2026
12 of 13 checks passed
@shreddedbacon shreddedbacon deleted the services-from-configmap branch February 19, 2026 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants