Skip to content

[doc] Adjust the logic before and after, add a small optimization when installing manager#13994

Merged
Anonymitaet merged 3 commits intoapache:masterfrom
zhaoyajun2009:deploy-manager1
Feb 15, 2022
Merged

[doc] Adjust the logic before and after, add a small optimization when installing manager#13994
Anonymitaet merged 3 commits intoapache:masterfrom
zhaoyajun2009:deploy-manager1

Conversation

@zhaoyajun2009
Copy link
Contributor

@zhaoyajun2009 zhaoyajun2009 commented Jan 28, 2022 •
edited
Loading

Motivation

Adjust the logic before and after, add a small optimization

Fixes #13922

Modifications

1）add 1 notes
2）add method2 of deploy manager
3) Adjust ‘Set administrator account and password’ to the back

Does this pull request potentially affect one of the following parts:

  • Anything that affects deployment: (yes )

Documentation

  • doc

@github-actions
Copy link

github-actions bot commented Jan 28, 2022

@zhaoyajun2009:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@zhaoyajun2009 zhaoyajun2009 changed the title Adjust the logic before and after, add a small optimization Adjust the logic before and after, add a small optimization when installing manager Jan 28, 2022
@zhaoyajun2009 zhaoyajun2009 changed the title Adjust the logic before and after, add a small optimization when installing manager [doc] Adjust the logic before and after, add a small optimization when installing manager Jan 28, 2022
@github-actions
Copy link

github-actions bot commented Jan 28, 2022

@zhaoyajun2009:Thanks for providing doc info!

@github-actions github-actions bot added doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. and removed doc-label-missing labels Jan 28, 2022
@github-actions
Copy link

github-actions bot commented Jan 28, 2022

@zhaoyajun2009:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions github-actions bot added doc-label-missing and removed doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. labels Jan 28, 2022
@github-actions
Copy link

github-actions bot commented Jan 28, 2022

@zhaoyajun2009:Thanks for providing doc info!

@github-actions github-actions bot added doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. and removed doc-label-missing labels Jan 28, 2022
@zhaoyajun2009
Copy link
Contributor Author

zhaoyajun2009 commented Jan 28, 2022

Please @tuteng help review , thanks.

@Anonymitaet
Copy link
Member

Anonymitaet commented Feb 7, 2022

Hi @tuteng can you review this PR from the technical perspective? Then writers can review it from the tech writing perspective, thanks.

@Anonymitaet Anonymitaet requested a review from tuteng February 14, 2022 03:32
Anonymitaet
Anonymitaet reviewed Feb 14, 2022
View reviewed changes
@Anonymitaet Anonymitaet added this to the 2.10.0 milestone Feb 15, 2022
@Anonymitaet Anonymitaet merged commit 9e73f9c into apache:master Feb 15, 2022
@lhotari lhotari mentioned this pull request Feb 15, 2022
Closed
Nicklee007 pushed a commit to Nicklee007/pulsar that referenced this pull request Apr 20, 2022
@zhaoyajun2009
[doc] Adjust the logic before and after, add a small optimization whe…
9fcdb47 
…n installing manager (apache#13994)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Anonymitaet Anonymitaet Anonymitaet left review comments

@tuteng tuteng tuteng approved these changes

Assignees

@zhaoyajun2009 zhaoyajun2009

Labels

doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.

Projects

None yet

Milestone

2.10.0

Development

Successfully merging this pull request may close these issues.

[Doc] This part of the document needs to be optimized when installing manager

3 participants

@zhaoyajun2009 @Anonymitaet @tuteng

Comments