You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2022/11/04 10:14:23 UTC
[GitHub] [ozone] sumitagrawl opened a new pull request, #3934: HDDS-7454. OM to DN token verification should include Pipeline
sumitagrawl opened a new pull request, #3934:
URL: https://github.com/apache/ozone/pull/3934
## What changes were proposed in this pull request?
1. Pipeline verification in DN for write operation passed in token via OM
2. Issue where token verification is skipped when no container in DN for newly created container
## What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7454
## How was this patch tested?
1. UT cases with pipeline verification
3. docker secureozone environment verifying pipeline is passed and verified successfully for write operation, and pipeline empty for read operation
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] sumitagrawl commented on pull request #3934: HDDS-7454. OM to DN token verification should include Pipeline
Posted by GitBox <gi...@apache.org>.
sumitagrawl commented on PR #3934:
URL: https://github.com/apache/ozone/pull/3934#issuecomment-1316639335
@neils-dev @fapifta Plz review
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] fapifta commented on pull request #3934: HDDS-7454. OM to DN token verification should include Pipeline
Posted by GitBox <gi...@apache.org>.
fapifta commented on PR #3934:
URL: https://github.com/apache/ozone/pull/3934#issuecomment-1317097090
I left a comment in the JIRA to the discussion we have.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] sumitagrawl commented on pull request #3934: HDDS-7454. OM to DN token verification should include Pipeline
Posted by GitBox <gi...@apache.org>.
sumitagrawl commented on PR #3934:
URL: https://github.com/apache/ozone/pull/3934#issuecomment-1347773893
Will close this PR as per discussion that this change is not required as do not have much impact to system.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] kerneltime commented on pull request #3934: HDDS-7454. OM to DN token verification should include Pipeline
Posted by GitBox <gi...@apache.org>.
kerneltime commented on PR #3934:
URL: https://github.com/apache/ozone/pull/3934#issuecomment-1305924618
@neils-dev
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] kerneltime commented on pull request #3934: HDDS-7454. OM to DN token verification should include Pipeline
Posted by GitBox <gi...@apache.org>.
kerneltime commented on PR #3934:
URL: https://github.com/apache/ozone/pull/3934#issuecomment-1305923331
@duongkame
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] sumitagrawl closed pull request #3934: HDDS-7454. OM to DN token verification should include Pipeline
Posted by GitBox <gi...@apache.org>.
sumitagrawl closed pull request #3934: HDDS-7454. OM to DN token verification should include Pipeline
URL: https://github.com/apache/ozone/pull/3934
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] sumitagrawl commented on pull request #3934: HDDS-7454. OM to DN token verification should include Pipeline
Posted by GitBox <gi...@apache.org>.
sumitagrawl commented on PR #3934:
URL: https://github.com/apache/ozone/pull/3934#issuecomment-1310436298
@kerneltime @neils-dev @fapifta
All test case failure is fixed and EC part is also handled correcting pipeline information in request. One part is pending related to validation of pipeline in request and actual DN in participating to this pipeline, which will handle in next PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] neils-dev commented on pull request #3934: HDDS-7454. OM to DN token verification should include Pipeline
Posted by GitBox <gi...@apache.org>.
neils-dev commented on PR #3934:
URL: https://github.com/apache/ozone/pull/3934#issuecomment-1317606089
Thanks @sumitagrawl for bringing up this issue and discussion on the Jira. Following the Jira discussion.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org