cloudfoundry / cloudfoundry/cloud_controller_ng
Inconsistent value for missing instance ports in process stats
Nobody has claimed this yet.
- Dominant language
- Ruby
- Stars
- 207
- Forks
- 373
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 56
Description
GET /v3/processes/:guid/stats returns inconsistent values for non-existing instance_ports:
- missing
instance_ports.internalandinstance_ports.externalare reported as0 - missing
instance_ports.internal_tls_proxy_portandinstance_ports.external_tls_proxy_portare reported asnull
capi-release v1.210.0 changed the values for a missing instance_ports.internal and instance_ports.external to null as a side effect of updating protobufs (#4359).
This (unintended) change broke cf-java-client because it checks for non-null instance_ports.internal and instance_ports.external.
#4401 fixes/changes missing instance_ports values back to the in-consistent values described above. Released with capi-release v1.211.0.
This issue is to find a long-term solution that returns consistent values for missing ports.
Proposal is to consolidate on null eventually. Can be done only when cf-java-client allows null values, cf-java-client #1284.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the GET /v3/processes/:guid/stats endpoint and the changes described in #4401, then review cf-java-client #1284 for the compatibility dependency. Done means the project has an agreed consistent representation for missing instance ports and the affected client behavior is accounted for.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java, ruby
- Domain
- api, backend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100