Skip to content

Fixes #28131: Rudder server Instance ID does not show up in the node API#6851

Merged
fanf merged 1 commit intoNormation:branches/rudder/8.3from
fanf:enh_28131/rudder_server_instance_id_does_not_show_up_in_the_node_api
Feb 2, 2026
Merged

Fixes #28131: Rudder server Instance ID does not show up in the node API#6851
fanf merged 1 commit intoNormation:branches/rudder/8.3from
fanf:enh_28131/rudder_server_instance_id_does_not_show_up_in_the_node_api

Conversation

@fanf
Copy link
Copy Markdown
Member

@fanf fanf commented Jan 15, 2026

https://issues.rudder.io/issues/28131

Mostly uninteresting. Just adding the instanceId at full level from the service, and updating test and api doc.

@fanf fanf requested a review from clarktsiory January 15, 2026 16:29
Comment thread webapp/sources/api-doc/components/schemas/node-full.yml Outdated
@fanf
Copy link
Copy Markdown
Member Author

fanf commented Jan 19, 2026

PR updated with a new commit

@fanf fanf requested review from skaerg and removed request for clarktsiory January 21, 2026 13:30
Copy link
Copy Markdown
Contributor

@skaerg skaerg left a comment

Choose a reason for hiding this comment

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

this looks good to me.

@fanf fanf force-pushed the enh_28131/rudder_server_instance_id_does_not_show_up_in_the_node_api branch from f166dbd to 01ce417 Compare January 23, 2026 19:20
@Normation-Quality-Assistant
Copy link
Copy Markdown
Contributor

This PR is not mergeable to upper versions.
Since it is "Ready for merge" you must merge it by yourself using the following command:
rudder-dev merge https://github.com/Normation/rudder/pull/6851
-- Your faithful QA
Kant merge: "Live your life as though your every act were to become a universal law."
(https://ci.normation.com/jenkins/job/merge-accepted-pr/113412/console)

@fanf
Copy link
Copy Markdown
Member Author

fanf commented Feb 2, 2026

OK, merging this PR

@fanf fanf merged commit 0db3b3a into Normation:branches/rudder/8.3 Feb 2, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants