Skip to content

feat: add updateApplication and updateJob mutations#443

Merged
thokra-nav merged 4 commits into
mainfrom
feat/update-application-job
May 19, 2026
Merged

feat: add updateApplication and updateJob mutations#443
thokra-nav merged 4 commits into
mainfrom
feat/update-application-job

Conversation

@thokra-nav
Copy link
Copy Markdown
Contributor

@thokra-nav thokra-nav commented May 19, 2026

Implement GraphQL mutations for ad-hoc updates to Application (env + replicas) and Naisjob (env) resources. Includes per-field activity log tracking with old/new values, no-op detection, and integration tests.


Vi har gjort en bevisst breaking change, siden UpdateApplication og UpdateJob gir ikke mening å returnere som en "Generic kubernetes"-greie. Vi gjør heller at Apply også resulterer i en ApplicationUpdate-data struktur fremfor generic kubernetes for disse to ressursene.

del av https://github.com/nais/system/pull/529

Implement GraphQL mutations for ad-hoc updates to Application (env + replicas)
and Naisjob (env) resources. Includes per-field activity log tracking with
old/new values, no-op detection, and integration tests.

Co-authored-by: Thomas Kraggerud <thomas.kraggerud@nav.no>
Comment thread internal/graph/schema/workloads.graphqls Outdated

This comment was marked as outdated.

Remove GENERIC_KUBERNETES_RESOURCE_UPDATED activity type

Sort env vars by name in MergeEnvVars

Fix activity log changedFields for replicas to handle nil values

Update integration tests for new activity log types and env var order

Co-authored-by: Frode Sundby <frode.sundby@nav.no>

This comment was marked as outdated.

Co-authored-by: Frode Sundby <frode.sundby@nav.no>
@thokra-nav thokra-nav marked this pull request as ready for review May 19, 2026 11:34
@thokra-nav thokra-nav requested a review from a team as a code owner May 19, 2026 11:34
Update GraphQL schema, Go models, and integration tests to use
environmentVariables instead of env for updating environment variables
on
applications and jobs. Update input types and related logic accordingly.

Co-authored-by: Frode Sundby <frode.sundby@nav.no>
Copy link
Copy Markdown
Member

@christeredvartsen christeredvartsen left a comment

Choose a reason for hiding this comment

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

:shipit:

@thokra-nav thokra-nav merged commit b01080d into main May 19, 2026
11 checks passed
@thokra-nav thokra-nav deleted the feat/update-application-job branch May 19, 2026 13:00
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.

4 participants