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

docs: add script to allow users to join upstream clusters to fleet hub #187

Merged
merged 12 commits into from
Aug 2, 2024

Conversation

Arvindthiru
Copy link
Contributor

@Arvindthiru Arvindthiru commented Jul 31, 2024

What type of PR is this?
/kind documentation

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Requirements:

How has this code been tested

Manually ran the script to join 2 member clusters to fleet hub

Note: This script can only be run after running script in Azure/fleet#894, this will be mentioned in the written doc for joining on prem cluster to fleet hub in the fleet repo

Special notes for your reviewer

@Arvindthiru Arvindthiru marked this pull request as ready for review August 1, 2024 00:09
@Arvindthiru Arvindthiru requested a review from michaelawyu August 1, 2024 00:13
@zhiying-lin
Copy link
Contributor

can we add a readme to include this information "This script can only be run after running script in Azure/fleet#894, this will be mentioned in the written doc for joining on prem cluster to fleet hub in the fleet repo"?

eg, what's the purpose of this script? how can we use?

@zhiying-lin zhiying-lin changed the title feat: add script to allow users to join upstream clusters to fleet hub docs: add script to allow users to join upstream clusters to fleet hub Aug 1, 2024
Copy link
Contributor

@ryanzhang-oss ryanzhang-oss left a comment

Choose a reason for hiding this comment

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

How is this script supposed to be used? What about Readme?

@Arvindthiru
Copy link
Contributor Author

Arvindthiru commented Aug 1, 2024

The readme will be initially added in the fleet repo for running both scripts since there is a sequence Azure/fleet#895. Updated comment on the script indicating sequence of operations

@Arvindthiru Arvindthiru merged commit 65dc872 into Azure:main Aug 2, 2024
8 checks 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