Re: [PR] Remove installation via snap package [couchdb]

2023-11-10 Thread via GitHub
nickva commented on PR #4826: URL: https://github.com/apache/couchdb/pull/4826#issuecomment-1806713224 Debian Bookworm packages for x86_64 should be available via the standard installation method

Re: [I] Debian Bookworm packages please [couchdb-pkg]

2023-11-10 Thread via GitHub
nickva commented on issue #112: URL: https://github.com/apache/couchdb-pkg/issues/112#issuecomment-1806711257 Bookworm packages are now available in the Apache CouchDB .deb repo So far it's for x86_64 arch only. -- This is an automated message from the Apache Git Service. To

Re: [I] Debian Bookworm packages please [couchdb-pkg]

2023-11-10 Thread via GitHub
nickva closed issue #112: Debian Bookworm packages please URL: https://github.com/apache/couchdb-pkg/issues/112 -- 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,

Re: [PR] Add Debian Bookworm for packaging [couchdb-pkg]

2023-11-10 Thread via GitHub
nickva merged PR #115: URL: https://github.com/apache/couchdb-pkg/pull/115 -- 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:

[PR] Bump axios from 0.27.2 to 1.6.0 [couchdb-fauxton]

2023-11-10 Thread via GitHub
dependabot[bot] opened a new pull request, #1419: URL: https://github.com/apache/couchdb-fauxton/pull/1419 Bumps [axios](https://github.com/axios/axios) from 0.27.2 to 1.6.0. Release notes Sourced from https://github.com/axios/axios/releases;>axios's releases. Release v1.6.0

Re: [PR] feat: add JDK 8 to the Docker images for running Clouseau [couchdb-ci]

2023-11-10 Thread via GitHub
pgj commented on PR #62: URL: https://github.com/apache/couchdb-ci/pull/62#issuecomment-1806617414 Yes, you are right. The compilation of Clouseau does not work with JDK 8. But the created JARs could be run with that version, I have tested it on multiple platforms. -- This is an

[Jenkins] FAILURE: CouchDB » Full Platform Builds » main #919

2023-11-10 Thread Apache Jenkins Server
Boo, we failed. https://ci-couchdb.apache.org/job/jenkins-cm1/job/FullPlatformMatrix/job/main/919/display/redirect

Re: [PR] Replace unique_integer with a ref in ets_lru [couchdb]

2023-11-10 Thread via GitHub
nickva merged PR #4839: URL: https://github.com/apache/couchdb/pull/4839 -- 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:

Re: [PR] feat: add JDK 8 to the Docker images for running Clouseau [couchdb-ci]

2023-11-10 Thread via GitHub
rnewson commented on PR #62: URL: https://github.com/apache/couchdb-ci/pull/62#issuecomment-1806517789 I thought the version of scala we need was incompatible with JDK 8, at least at compile time? -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] Replace unique_integer with a ref in ets_lru [couchdb]

2023-11-10 Thread via GitHub
rnewson commented on code in PR #4839: URL: https://github.com/apache/couchdb/pull/4839#discussion_r1389981849 ## src/ets_lru/src/ets_lru.erl: ## @@ -366,4 +366,4 @@ table_name(Name, Ext) -> -spec strict_monotonic_time(atom()) -> strict_monotonic_time().

Re: [PR] Replace unique_integer with a ref in ets_lru [couchdb]

2023-11-10 Thread via GitHub
nickva commented on code in PR #4839: URL: https://github.com/apache/couchdb/pull/4839#discussion_r1389955618 ## src/ets_lru/src/ets_lru.erl: ## @@ -366,4 +366,4 @@ table_name(Name, Ext) -> -spec strict_monotonic_time(atom()) -> strict_monotonic_time().

Re: [PR] Replace unique_integer with a ref in ets_lru [couchdb]

2023-11-10 Thread via GitHub
rnewson commented on code in PR #4839: URL: https://github.com/apache/couchdb/pull/4839#discussion_r1389933573 ## src/ets_lru/src/ets_lru.erl: ## @@ -366,4 +366,4 @@ table_name(Name, Ext) -> -spec strict_monotonic_time(atom()) -> strict_monotonic_time().

[PR] Replace unique_integer with a ref in ets_lru [couchdb]

2023-11-10 Thread via GitHub
nickva opened a new pull request, #4839: URL: https://github.com/apache/couchdb/pull/4839 We don't really need a strictly monotonic integer, just a unique key for each atime and ctime entry. `unique_integer([monotonic])` is also a concurrency bottleneck as indicated the warning in [1].

Re: [PR] Update node, elixir and improve building [couchdb-ci]

2023-11-10 Thread via GitHub
nickva merged PR #63: URL: https://github.com/apache/couchdb-ci/pull/63 -- 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:

[PR] Update node, elixir and improve building [couchdb-ci]

2023-11-10 Thread via GitHub
nickva opened a new pull request, #63: URL: https://github.com/apache/couchdb-ci/pull/63 * Most node versions are now at 18, except a few which needed to be held back. * Elixir is at 1.15 * Building is now a single command per erlang version: ```sh

Re: [PR] feat: add JDK 8 to the Docker images for running Clouseau [couchdb-ci]

2023-11-10 Thread via GitHub
pgj commented on PR #62: URL: https://github.com/apache/couchdb-ci/pull/62#issuecomment-1806372694 With the help of `eclipse-temurin` the change has become much smaller and it works for all the currently supported platforms. -- This is an automated message from the Apache Git Service. To

Re: [PR] Extend branch protection rules [couchdb-pkg]

2023-11-10 Thread via GitHub
nickva commented on PR #118: URL: https://github.com/apache/couchdb-pkg/pull/118#issuecomment-1806287591 Let's try it, but I think overall both here and on couchdb-ci protection doesn't make as much sense without a place to test and validate the artifacts before the pr is merged. The repo

Re: [PR] Extend branch protection rules [couchdb-pkg]

2023-11-10 Thread via GitHub
big-r81 merged PR #118: URL: https://github.com/apache/couchdb-pkg/pull/118 -- 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:

[Jenkins] FAILURE: CouchDB » Full Platform Builds » main #918

2023-11-10 Thread Apache Jenkins Server
Boo, we failed. https://ci-couchdb.apache.org/job/jenkins-cm1/job/FullPlatformMatrix/job/main/918/display/redirect

Re: [PR] add eviction for idleness to ets_lru [couchdb]

2023-11-10 Thread via GitHub
rnewson merged PR #4838: URL: https://github.com/apache/couchdb/pull/4838 -- 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:

Re: [PR] Remove empty line from debian control.in [couchdb-pkg]

2023-11-10 Thread via GitHub
nickva merged PR #121: URL: https://github.com/apache/couchdb-pkg/pull/121 -- 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:

[PR] Remove empty line from debian control.in [couchdb-pkg]

2023-11-10 Thread via GitHub
nickva opened a new pull request, #121: URL: https://github.com/apache/couchdb-pkg/pull/121 Attempt to fix: ``` [2023-11-10T17:11:57.735Z] cd /home/jenkins/workspace/kins-cm1_FullPlatformMatrix_main/bullseye/couchdb/apache-couchdb-3.3.2-5a21bb2 && dpkg-buildpackage -b -us -uc

[Jenkins] FAILURE: CouchDB » Full Platform Builds » main #917

2023-11-10 Thread Apache Jenkins Server
Boo, we failed. https://ci-couchdb.apache.org/job/jenkins-cm1/job/FullPlatformMatrix/job/main/917/display/redirect

[PR] add eviction for idleness to ets_lru [couchdb]

2023-11-10 Thread via GitHub
rnewson opened a new pull request, #4838: URL: https://github.com/apache/couchdb/pull/4838 ## Overview add `max_idle` option, which evicts items by last access time (as opposed to `max_lifetime` which evicts based on creation time). ## Testing recommendations N/A

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-11-10 Thread via GitHub
nickva commented on PR #4814: URL: https://github.com/apache/couchdb/pull/4814#issuecomment-1806072709 Makes sense that we can just ignore the conflict as they should be the same content. Even if a conflict is generated it should merge cleanly. > I can't trigger the conflict cases

Re: [PR] feat(`dev`): add Clouseau to the developer setup [couchdb]

2023-11-10 Thread via GitHub
pgj commented on code in PR #4835: URL: https://github.com/apache/couchdb/pull/4835#discussion_r1389623988 ## README-DEV.rst: ## @@ -272,6 +266,50 @@ to keep those databases around for further investigation:: MANGO_TESTS_KEEP_DBS=please \ make mango-test

Re: [PR] feat(`dev`): add Clouseau to the developer setup [couchdb]

2023-11-10 Thread via GitHub
pgj commented on code in PR #4835: URL: https://github.com/apache/couchdb/pull/4835#discussion_r1389623756 ## README-DEV.rst: ## @@ -272,6 +266,50 @@ to keep those databases around for further investigation:: MANGO_TESTS_KEEP_DBS=please \ make mango-test

Re: [PR] Remove python3-sphinx debian dependency [couchdb-pkg]

2023-11-10 Thread via GitHub
nickva merged PR #120: URL: https://github.com/apache/couchdb-pkg/pull/120 -- 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:

[PR] Remove python3-sphinx debian dependency [couchdb-pkg]

2023-11-10 Thread via GitHub
nickva opened a new pull request, #120: URL: https://github.com/apache/couchdb-pkg/pull/120 We use a venv for it. We just need python3 to be present. https://github.com/apache/couchdb/tree/main/src/docs#readme Hopefully it should fix this package build issue in CI: ```

[Jenkins] FAILURE: CouchDB » Full Platform Builds » jenkins-decouple_offline_hash_strength_from_online #5

2023-11-10 Thread Apache Jenkins Server
Boo, we failed. https://ci-couchdb.apache.org/job/jenkins-cm1/job/FullPlatformMatrix/job/jenkins-decouple_offline_hash_strength_from_online/5/display/redirect

Re: [PR] feat(`dev`): add Clouseau to the developer setup [couchdb]

2023-11-10 Thread via GitHub
big-r81 commented on code in PR #4835: URL: https://github.com/apache/couchdb/pull/4835#discussion_r1389554272 ## README-DEV.rst: ## @@ -272,6 +266,50 @@ to keep those databases around for further investigation:: MANGO_TESTS_KEEP_DBS=please \ make mango-test

Re: [PR] feat(`dev`): add Clouseau to the developer setup [couchdb]

2023-11-10 Thread via GitHub
big-r81 commented on code in PR #4835: URL: https://github.com/apache/couchdb/pull/4835#discussion_r1389553098 ## README-DEV.rst: ## @@ -272,6 +266,50 @@ to keep those databases around for further investigation:: MANGO_TESTS_KEEP_DBS=please \ make mango-test

[Jenkins] FAILURE: CouchDB » Full Platform Builds » jenkins-decouple_offline_hash_strength_from_online #6

2023-11-10 Thread Apache Jenkins Server
Boo, we failed. https://ci-couchdb.apache.org/job/jenkins-cm1/job/FullPlatformMatrix/job/jenkins-decouple_offline_hash_strength_from_online/6/display/redirect

Re: [PR] feat(`dev`): add Clouseau to the developer setup [couchdb]

2023-11-10 Thread via GitHub
big-r81 commented on code in PR #4835: URL: https://github.com/apache/couchdb/pull/4835#discussion_r1389552537 ## README-DEV.rst: ## @@ -272,6 +266,50 @@ to keep those databases around for further investigation:: MANGO_TESTS_KEEP_DBS=please \ make mango-test

Re: [PR] feat(`dev`): add Clouseau to the developer setup [couchdb]

2023-11-10 Thread via GitHub
pgj commented on code in PR #4835: URL: https://github.com/apache/couchdb/pull/4835#discussion_r1389490610 ## dev/run: ## @@ -504,6 +512,119 @@ def boot_nouveau(ctx): ) +CLOUSEAU_DIR = "clouseau" +JAVA_VERSION_RE = re.compile('"(\d+\.\d+).*"') + + +def

Re: [PR] feat(`dev`): add Clouseau to the developer setup [couchdb]

2023-11-10 Thread via GitHub
jiahuili430 commented on code in PR #4835: URL: https://github.com/apache/couchdb/pull/4835#discussion_r1389437909 ## dev/run: ## @@ -504,6 +512,119 @@ def boot_nouveau(ctx): ) +CLOUSEAU_DIR = "clouseau" +JAVA_VERSION_RE = re.compile('"(\d+\.\d+).*"') + + +def

Re: [PR] feat(`dev`): add Clouseau to the developer setup [couchdb]

2023-11-10 Thread via GitHub
jiahuili430 commented on PR #4835: URL: https://github.com/apache/couchdb/pull/4835#issuecomment-1805790263 The code is very clean and makes Clouseau easier to use. Thank you, Gábor. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-11-10 Thread via GitHub
rnewson commented on PR #4814: URL: https://github.com/apache/couchdb/pull/4814#issuecomment-1805656312 I can't trigger the conflict cases locally but I've added a `catch` on the front `couch_password_hasher`'s call to `AuthModule:update_user_creds`. -- This is an automated message from

Re: [PR] Decouple offline hash strength from online [couchdb]

2023-11-10 Thread via GitHub
rnewson commented on PR #4814: URL: https://github.com/apache/couchdb/pull/4814#issuecomment-1805625536 I'd like to suppress the conflict errors since those are benign. Each node might elect to hash and update the user doc when it gets a request. however, by design of this PR, they will