[GitHub] [ozone-site] flirmnave opened a new pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-05 Thread GitBox
flirmnave opened a new pull request #1: URL: https://github.com/apache/ozone-site/pull/1 ## What changes were proposed in this pull request? Add **Community** menu. This page contains Ozone mailing list, Ozone slack, Ozone community call. But there is a question: Should I ad

[GitHub] [ozone-site] cxorm commented on a change in pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-05 Thread GitBox
cxorm commented on a change in pull request #1: URL: https://github.com/apache/ozone-site/pull/1#discussion_r518122171 ## File path: content/community.md ## @@ -0,0 +1,63 @@ +--- +title: Ozone Community +--- + +## Ozone Mailing Lists + +### Developers + +If you’d like to contri

[GitHub] [ozone-site] flirmnave commented on pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-06 Thread GitBox
flirmnave commented on pull request #1: URL: https://github.com/apache/ozone-site/pull/1#issuecomment-722985571 Thanks @cxorm for the review and comment. I update the ozone community information. ![image](https://user-images.githubusercontent.com/2565172/98351842-37c80b80-2058-11eb-9d

[GitHub] [ozone-site] cxorm commented on a change in pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-07 Thread GitBox
cxorm commented on a change in pull request #1: URL: https://github.com/apache/ozone-site/pull/1#discussion_r519263085 ## File path: content/community.md ## @@ -0,0 +1,43 @@ +--- +title: Ozone Community +--- + +## Ozone Mailing Lists + +### Developers + +If you’d like to contri

[GitHub] [ozone-site] cxorm commented on a change in pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-07 Thread GitBox
cxorm commented on a change in pull request #1: URL: https://github.com/apache/ozone-site/pull/1#discussion_r519263118 ## File path: content/community.md ## @@ -0,0 +1,43 @@ +--- +title: Ozone Community +--- + +## Ozone Mailing Lists + +### Developers + +If you’d like to contri

[GitHub] [ozone-site] adoroszlai commented on a change in pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-08 Thread GitBox
adoroszlai commented on a change in pull request #1: URL: https://github.com/apache/ozone-site/pull/1#discussion_r519594558 ## File path: content/community.md ## @@ -0,0 +1,43 @@ +--- +title: Ozone Community +--- + +## Ozone Mailing Lists + +### Developers + +If you’d like to c

[GitHub] [ozone-site] flirmnave commented on pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-09 Thread GitBox
flirmnave commented on pull request #1: URL: https://github.com/apache/ozone-site/pull/1#issuecomment-723851498 Thanks @adoroszlai for review and comment. I fix it. This is an automated message from the Apache Git Service.

[GitHub] [ozone-site] smengcl commented on a change in pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-09 Thread GitBox
smengcl commented on a change in pull request #1: URL: https://github.com/apache/ozone-site/pull/1#discussion_r519903672 ## File path: content/community.md ## @@ -0,0 +1,43 @@ +--- +title: Ozone Community +--- + +## Ozone Mailing Lists + +### Developers + +If you’d like to cont

[GitHub] [ozone-site] cxorm commented on pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-09 Thread GitBox
cxorm commented on pull request #1: URL: https://github.com/apache/ozone-site/pull/1#issuecomment-724111822 > Thanks @flirmnave for working on this. Looks good, but the email address links need `mailto:` protocol. Otherwise they are treated as web links and result in 404. Thanks @ad

[GitHub] [ozone-site] cku328 commented on a change in pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-09 Thread GitBox
cku328 commented on a change in pull request #1: URL: https://github.com/apache/ozone-site/pull/1#discussion_r519944036 ## File path: content/community.md ## @@ -0,0 +1,43 @@ +--- +title: Ozone Community +--- + +## Ozone Mailing Lists + +### Developers + +If you’d like to contr

[GitHub] [ozone-site] adoroszlai commented on a change in pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-09 Thread GitBox
adoroszlai commented on a change in pull request #1: URL: https://github.com/apache/ozone-site/pull/1#discussion_r519951691 ## File path: content/community.md ## @@ -0,0 +1,43 @@ +--- +title: Ozone Community +--- + +## Ozone Mailing Lists + +### Developers + +If you’d like to c

[GitHub] [ozone-site] cku328 commented on a change in pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-09 Thread GitBox
cku328 commented on a change in pull request #1: URL: https://github.com/apache/ozone-site/pull/1#discussion_r519944036 ## File path: content/community.md ## @@ -0,0 +1,43 @@ +--- +title: Ozone Community +--- + +## Ozone Mailing Lists + +### Developers + +If you’d like to contr

[GitHub] [ozone-site] flirmnave commented on pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-09 Thread GitBox
flirmnave commented on pull request #1: URL: https://github.com/apache/ozone-site/pull/1#issuecomment-724465389 Thanks @cxorm @adoroszlai @smengcl @cku328 for review and comment. I update it. And add a section named **Older Archives**. Can you please share your suggestions? T

[GitHub] [ozone-site] cxorm merged pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-12 Thread GitBox
cxorm merged pull request #1: URL: https://github.com/apache/ozone-site/pull/1 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 s

[GitHub] [ozone-site] cxorm commented on pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-12 Thread GitBox
cxorm commented on pull request #1: URL: https://github.com/apache/ozone-site/pull/1#issuecomment-725938310 Thank you all for the careful reviews, @adoroszlai , @cku328 and @smengcl . Thank you @flirmnave for working on this. I have committed it. -

[GitHub] [ozone-site] flirmnave commented on pull request #1: HDDS-4415. Add mail lists page on ozone website

2020-11-12 Thread GitBox
flirmnave commented on pull request #1: URL: https://github.com/apache/ozone-site/pull/1#issuecomment-725946399 Thanks @cxorm for commit and @adoroszlai @smengcl @cku328 for review. :smiley: This is an automated message fro

[GitHub] [ozone-docker-runner] elek opened a new pull request #1: HDDS-4795. Separate source of Ozone container images to different repositories

2021-02-05 Thread GitBox
elek opened a new pull request #1: URL: https://github.com/apache/ozone-docker-runner/pull/1 This PR contains basic cleanup after moving out the source of `apache/ozone-runner` to this repository. Basic Github action workflow + standard descriptors (PR template, notification) + READ

[GitHub] [ozone-docker-runner] adoroszlai commented on a change in pull request #1: HDDS-4795. Separate source of Ozone container images to different repositories

2021-02-06 Thread GitBox
adoroszlai commented on a change in pull request #1: URL: https://github.com/apache/ozone-docker-runner/pull/1#discussion_r571382666 ## File path: README.md ## @@ -15,15 +15,35 @@ limitations under the License. --> -# Apache Ozone runner base image +# Apache Ozone **runne

[GitHub] [ozone-docker-runner] elek commented on pull request #1: HDDS-4795. Separate source of Ozone container images to different repositories

2021-02-08 Thread GitBox
elek commented on pull request #1: URL: https://github.com/apache/ozone-docker-runner/pull/1#issuecomment-775080667 Thanks for the review @adoroszlai. I applied the suggestions. After merging it, I am planning to do the following next steps: 1. Create a similar initial commit

[GitHub] [ozone-docker-builder] elek opened a new pull request #1: HDDS-4795. Separate source of Ozone container images to different repositories

2021-02-25 Thread GitBox
elek opened a new pull request #1: URL: https://github.com/apache/ozone-docker-builder/pull/1 Similar to fd796d64ce6e387ad7b93e88605ed97cc755ef85, this pull request contains initial README/builds/github actions after the separation.

[GitHub] [ozone-docker-builder] adoroszlai commented on a change in pull request #1: HDDS-4795. Separate source of Ozone container images to different repositories

2021-02-26 Thread GitBox
adoroszlai commented on a change in pull request #1: URL: https://github.com/apache/ozone-docker-builder/pull/1#discussion_r583521241 ## File path: SECURITY.md ## @@ -0,0 +1,5 @@ +# Security Policy + +This container image (`apache/ozone-runner`) is indented to be used only in

[GitHub] [ozone-docker-testkrb5] elek opened a new pull request #1: Provide testkrb5 image for faster ozonesecure tests

2021-03-09 Thread GitBox
elek opened a new pull request #1: URL: https://github.com/apache/ozone-docker-testkrb5/pull/1 Today ozonesecure compose clusters (and ozonesecure-ha and ozonesecure-mr) use an adhoc keytab issuer. The issuer is download during the [image creation](https://github.com/apache/ozone/blob/mast

[GitHub] [ozone-docker-testkrb5] elek commented on pull request #1: HDDS-4938. Provide testkrb5 image for faster ozonesecure tests

2021-03-09 Thread GitBox
elek commented on pull request #1: URL: https://github.com/apache/ozone-docker-testkrb5/pull/1#issuecomment-793989075 Tested on a [forked branch](https://github.com/elek/ozone/tree/docker-secure), and finally [it's passed](https://github.com/elek/ozone/runs/2056855876) (= only an intermit

[GitHub] [ozone-docker-testkrb5] adoroszlai commented on a change in pull request #1: HDDS-4938. Provide testkrb5 image for faster ozonesecure tests

2021-03-09 Thread GitBox
adoroszlai commented on a change in pull request #1: URL: https://github.com/apache/ozone-docker-testkrb5/pull/1#discussion_r590497494 ## File path: .github/workflows/build.yaml ## @@ -0,0 +1,27 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contri

[GitHub] [ozone-docker-testkrb5] elek commented on a change in pull request #1: HDDS-4938. Provide testkrb5 image for faster ozonesecure tests

2021-03-09 Thread GitBox
elek commented on a change in pull request #1: URL: https://github.com/apache/ozone-docker-testkrb5/pull/1#discussion_r590533575 ## File path: .github/workflows/build.yaml ## @@ -0,0 +1,27 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor

[GitHub] [ozone-docker-testkrb5] adoroszlai commented on a change in pull request #1: HDDS-4938. Provide testkrb5 image for faster ozonesecure tests

2021-03-09 Thread GitBox
adoroszlai commented on a change in pull request #1: URL: https://github.com/apache/ozone-docker-testkrb5/pull/1#discussion_r590537066 ## File path: .github/workflows/build.yaml ## @@ -0,0 +1,27 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contri

[GitHub] [ozone-docker-testkrb5] elek commented on a change in pull request #1: HDDS-4938. Provide testkrb5 image for faster ozonesecure tests

2021-03-10 Thread GitBox
elek commented on a change in pull request #1: URL: https://github.com/apache/ozone-docker-testkrb5/pull/1#discussion_r591223319 ## File path: .github/workflows/build.yaml ## @@ -0,0 +1,27 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor

[GitHub] [ozone-docker-testkrb5] adoroszlai commented on pull request #1: HDDS-4938. Provide testkrb5 image for faster ozonesecure tests

2021-03-10 Thread GitBox
adoroszlai commented on pull request #1: URL: https://github.com/apache/ozone-docker-testkrb5/pull/1#issuecomment-795234468 Thanks @elek for updating the patch. Do you have any hint on the problem with the image built locally?

[GitHub] [ozone-docker-testkrb5] elek commented on pull request #1: HDDS-4938. Provide testkrb5 image for faster ozonesecure tests

2021-03-11 Thread GitBox
elek commented on pull request #1: URL: https://github.com/apache/ozone-docker-testkrb5/pull/1#issuecomment-796569758 > Shouldn't it be possible to seamlessly use a different image? How will image update work (eg. when a new principal is needed)? Are there any steps I missed? Good

[GitHub] [ozone-docker-testkrb5] adoroszlai commented on pull request #1: HDDS-4938. Provide testkrb5 image for faster ozonesecure tests

2021-03-11 Thread GitBox
adoroszlai commented on pull request #1: URL: https://github.com/apache/ozone-docker-testkrb5/pull/1#issuecomment-796573963 > the export should happen once when the docker image version is updated (and exported keytabs are committed) > please execute the `generate-keytab` command from t

[GitHub] [ozone-docker-testkrb5] elek commented on pull request #1: HDDS-4938. Provide testkrb5 image for faster ozonesecure tests

2021-03-11 Thread GitBox
elek commented on pull request #1: URL: https://github.com/apache/ozone-docker-testkrb5/pull/1#issuecomment-796672157 > Thanks for the explanation. Where do I find this command? It's here: https://github.com/elek/ozone/tree/docker-secure/hadoop-ozone/dist/src/main/keytabs

[GitHub] [ozone-docker-testkrb5] elek commented on pull request #1: HDDS-4938. Provide testkrb5 image for faster ozonesecure tests

2021-03-19 Thread GitBox
elek commented on pull request #1: URL: https://github.com/apache/ozone-docker-testkrb5/pull/1#issuecomment-802680678 Thanks the review, merging it now... -- 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

[GitHub] [ozone-site] adoroszlai opened a new pull request #2: HDDS-5034. Update wiki links in website

2021-03-26 Thread GitBox
adoroszlai opened a new pull request #2: URL: https://github.com/apache/ozone-site/pull/2 ## What changes were proposed in this pull request? Update references to Hadoop wiki. https://issues.apache.org/jira/browse/HDDS-5034 ## How was this patch tested? ``` hug

[GitHub] [ozone-site] adoroszlai opened a new pull request #3: HDDS-5036. Apply merge/notification settings to ozone-site repo

2021-03-26 Thread GitBox
adoroszlai opened a new pull request #3: URL: https://github.com/apache/ozone-site/pull/3 ## What changes were proposed in this pull request? * Only allow squash merge. * Send issue/PR updates to `issues@`, not `dev@`. * Add PR link and label in Jira. https://issues.ap

[GitHub] [ozone-site] elek merged pull request #3: HDDS-5036. Apply merge/notification settings to ozone-site repo

2021-03-28 Thread GitBox
elek merged pull request #3: URL: https://github.com/apache/ozone-site/pull/3 -- 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. For queries about this service, please conta

[GitHub] [ozone] rakeshadr commented on a diff in pull request #3175: HDDS-6414. Ozone doesn’t reclaim space after deletion of volume via ofs

2022-04-03 Thread GitBox
rakeshadr commented on code in PR #3175: URL: https://github.com/apache/ozone/pull/3175#discussion_r841389614 ## hadoop-ozone/ozonefs-common/src/main/java/org/apache/hadoop/fs/ozone/BasicRootedOzoneFileSystem.java: ## @@ -567,7 +626,12 @@ public boolean delete(Path f, boolean re

[GitHub] [ozone] rakeshadr commented on a diff in pull request #3175: HDDS-6414. Ozone doesn’t reclaim space after deletion of volume via ofs

2022-04-03 Thread GitBox
rakeshadr commented on code in PR #3175: URL: https://github.com/apache/ozone/pull/3175#discussion_r841390333 ## hadoop-ozone/ozonefs-common/src/main/java/org/apache/hadoop/fs/ozone/BasicRootedOzoneFileSystem.java: ## @@ -468,6 +468,42 @@ boolean processKeyPath(List keyPathList)

[GitHub] [ozone] rakeshadr commented on a diff in pull request #3175: HDDS-6414. Ozone doesn’t reclaim space after deletion of volume via ofs

2022-04-03 Thread GitBox
rakeshadr commented on code in PR #3175: URL: https://github.com/apache/ozone/pull/3175#discussion_r841391130 ## hadoop-ozone/ozonefs-common/src/main/java/org/apache/hadoop/fs/ozone/BasicRootedOzoneFileSystem.java: ## @@ -986,15 +1076,15 @@ boolean iterate() throws IOException {

[GitHub] [ozone] rakeshadr commented on a diff in pull request #3175: HDDS-6414. Ozone doesn’t reclaim space after deletion of volume via ofs

2022-04-03 Thread GitBox
rakeshadr commented on code in PR #3175: URL: https://github.com/apache/ozone/pull/3175#discussion_r841394899 ## hadoop-ozone/ozonefs-common/src/main/java/org/apache/hadoop/fs/ozone/BasicRootedOzoneFileSystem.java: ## @@ -489,6 +532,29 @@ private boolean innerDelete(Path f, bool

[GitHub] [ozone] rakeshadr commented on a diff in pull request #3175: HDDS-6414. Ozone doesn’t reclaim space after deletion of volume via ofs

2022-04-03 Thread GitBox
rakeshadr commented on code in PR #3175: URL: https://github.com/apache/ozone/pull/3175#discussion_r841400022 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/key/OmKeysDeleteRequestWithFSO.java: ## @@ -0,0 +1,266 @@ +/** + * Licensed to the Apache

[GitHub] [ozone] sadanand48 commented on a diff in pull request #3175: HDDS-6414. Ozone doesn’t reclaim space after deletion of volume via ofs

2022-04-04 Thread GitBox
sadanand48 commented on code in PR #3175: URL: https://github.com/apache/ozone/pull/3175#discussion_r841483449 ## hadoop-ozone/ozonefs-common/src/main/java/org/apache/hadoop/fs/ozone/BasicRootedOzoneFileSystem.java: ## @@ -986,15 +1076,15 @@ boolean iterate() throws IOException

[GitHub] [ozone] sadanand48 commented on a diff in pull request #3175: HDDS-6414. Ozone doesn’t reclaim space after deletion of volume via ofs

2022-04-04 Thread GitBox
sadanand48 commented on code in PR #3175: URL: https://github.com/apache/ozone/pull/3175#discussion_r841486761 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/key/OmKeysDeleteRequestWithFSO.java: ## @@ -0,0 +1,266 @@ +/** + * Licensed to the Apache

[GitHub] [ozone] sadanand48 commented on a diff in pull request #3175: HDDS-6414. Ozone doesn’t reclaim space after deletion of volume via ofs

2022-04-04 Thread GitBox
sadanand48 commented on code in PR #3175: URL: https://github.com/apache/ozone/pull/3175#discussion_r841486761 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/key/OmKeysDeleteRequestWithFSO.java: ## @@ -0,0 +1,266 @@ +/** + * Licensed to the Apache

[GitHub] [ozone] sadanand48 commented on a diff in pull request #3175: HDDS-6414. Ozone doesn’t reclaim space after deletion of volume via ofs

2022-04-04 Thread GitBox
sadanand48 commented on code in PR #3175: URL: https://github.com/apache/ozone/pull/3175#discussion_r841489692 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/OMBucketCreateRequest.java: ## @@ -152,6 +152,9 @@ public OMClientResponse validat