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

HDDS-11667. Validating DatanodeID on any request to the datanode #5

Merged
merged 8 commits into from
Nov 18, 2024

Conversation

swamirishi
Copy link
Owner

What changes were proposed in this pull request?

Provide a one-liner summary of the changes in the PR Title field above.
It should be in the form of HDDS-1234. Short summary of the change.

Please describe your PR in detail:

  • What changes are proposed in the PR? and Why? It would be better if it is written from third person's
    perspective not just for the reviewer.
  • Provide as much context and rationale for the pull request as possible. It could be copy-paste from
    the Jira's description if the jira is well defined.
  • If it is complex code, describe the approach used to solve the issue. If possible attach design doc,
    issue investigation, github discussion, etc.

Examples of well-written pull requests:

What is the link to the Apache JIRA

Please create an issue in ASF JIRA before opening a pull request, and you need to set the title of the pull
request which starts with the corresponding JIRA issue number. (e.g. HDDS-XXXX. Fix a typo in YYY.)

(Please replace this section with the link to the Apache JIRA)

How was this patch tested?

(Please explain how this patch was tested. Ex: unit tests, manual tests, workflow run on the fork git repo.)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this.)

Change-Id: I3a4604cedd9dc1857fa139dbcb53dcaa184ad9b1
Change-Id: Ia33fbb2c8eb755d77b966275b2f03f1a72a569c6
Change-Id: Ided9ba2da3bcc907fa4e25644def6ef16b3f8cc6
Change-Id: Id81b19200015d5b7d5c578a0314d570bb63755de
Change-Id: Iaaa88f6a53cce8448c6095d994cafbdb26d6eebc
Change-Id: I833ebfa62089da43eaa72266da72962f94aba92b
Change-Id: If5b9c305b8367054e301192dc1a2aeae9b22d33f
Change-Id: I2a0eb6e418c15159715c0f133ce515af0f6cd0bf
@swamirishi swamirishi merged commit 2945d32 into master Nov 18, 2024
43 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.

1 participant