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.

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

Update regarding adding a DMA to a DMS #3709

Merged
merged 5 commits into from
Sep 18, 2024

Conversation

XanderWA
Copy link
Contributor

No description provided.

@XanderWA XanderWA requested a review from a team as a code owner September 18, 2024 10:00
@MariekeGO
Copy link
Member

Is this a recent change, or has it been like this for quite some time (at least since 10.1.0)? Are there cases where using that Join cluster button is in fact still needed?

@MariekeGO MariekeGO self-assigned this Sep 18, 2024
@bertbsky
Copy link
Contributor

The Join cluster button is only needed when initially going from a standalone DMA to a DMS (cluster of one).
Cube has been sending the SetDmsClusterMessage(clustername) message to the DMA being added to a cluster since DataMiner 8.5 (RN7748).

@MariekeGO
Copy link
Member

The Join cluster button is only needed when initially going from a standalone DMA to a DMS (cluster of one). Cube has been sending the SetDmsClusterMessage(clustername) message to the DMA being added to a cluster since DataMiner 8.5 (RN7748).

In that case, it might even be better to not add the note about the cluster name, as this could cause confusion (i.e. people might wonder in which case you do have to set the cluster name, while they actually don't have to do it at all).

@XanderWA
Copy link
Contributor Author

My reasoning behind putting the note there is that people could be confused as they remember the procedure being different in the past. However, I agree with your reasoning for not putting it there too.

…MA/Adding_a_regular_DataMiner_Agent.md

Co-authored-by: Marieke Goethals <[email protected]>
…_a_new_DMS/Setting_up_a_new_DMS_in_DataMiner_Cube.md

Co-authored-by: Marieke Goethals <[email protected]>
@MariekeGO MariekeGO merged commit 87ea312 into SkylineCommunications:main Sep 18, 2024
1 check passed
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.

3 participants