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
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
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
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
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
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
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 {
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)
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
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
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
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
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
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
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
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
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?
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
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
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
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
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
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
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
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.
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
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
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
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
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.
-
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
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
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
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
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
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
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
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.
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
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
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
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.
