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

Jump to bottom

Status must merge new and existing Conditions so data is not lost #558

Closed
ja20222 opened this issue Apr 5, 2023 · 0 comments · Fixed by #1220
Closed

Status must merge new and existing Conditions so data is not lost #558

ja20222 opened this issue Apr 5, 2023 · 0 comments · Fixed by #1220
Assignees
@kate-osborn
Labels
area/control-plane General control plane issues bug Something isn't working refined Requirements are refined and the issue is ready to be implemented. size/medium Estimated to be completed within a week
Milestone
v1.1.0

Comments

@ja20222
Copy link

ja20222 commented Apr 5, 2023 •
edited by kate-osborn
Loading

As a user of the Gateway API
I want to be able to attach my HTTPRoutes to multiple Gateways that belong to different GatewayClasses
and be able to see the statuses from all Gateways
So that I know the status of all attachments,

A/C:

  • When writing statuses to HTTPRoute resources, merge the conditions owned by other Gateway implementations with the new conditions.
  • NGF must not overwrite Conditions written by other Gateway implementations.
  • resolve FIXME
@ja20222 ja20222 added the bug Something isn't working label Apr 5, 2023
@pleshakov pleshakov added the area/control-plane General control plane issues label Apr 7, 2023
@pleshakov pleshakov added this to the v1.0.0 milestone Apr 7, 2023
@mpstefan mpstefan modified the milestones: v1.0.0, v1.0.1 Aug 11, 2023
@mpstefan mpstefan modified the milestones: v1.0.1, v1.1.0 Sep 22, 2023
@kate-osborn kate-osborn added refined Requirements are refined and the issue is ready to be implemented. size/medium Estimated to be completed within a week labels Oct 9, 2023
@kate-osborn kate-osborn self-assigned this Oct 31, 2023
@kate-osborn kate-osborn moved this from 🆕 New to 🏗 In Progress in NGINX Gateway Fabric Nov 2, 2023
@kate-osborn kate-osborn mentioned this issue Nov 3, 2023
Merged
6 tasks
@kate-osborn kate-osborn moved this from 🏗 In Progress to 👀 In Review in NGINX Gateway Fabric Nov 3, 2023
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in NGINX Gateway Fabric Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kate-osborn kate-osborn

Labels
area/control-plane General control plane issues bug Something isn't working refined Requirements are refined and the issue is ready to be implemented. size/medium Estimated to be completed within a week
Projects
NGINX Gateway Fabric
Archived in project
Milestone
v1.1.0
Development

Successfully merging a pull request may close this issue.

Merge HTTPRoute conditions kate-osborn/nginx-gateway-fabric
4 participants
@pleshakov @mpstefan @kate-osborn @ja20222