HDFS-16757. Add a new method copyBlockCrossNamespace to DataNode

Description of PR
As described in https://issues.apache.org/jira/browse/HDFS-16757
This is a new version which based on    5719
How was this patch tested?
Add test case for it.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
17m 13s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
buf
0m  1s

buf was not available.

+0 :ok:
buf
0m  1s

buf was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 35s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
38m 52s

trunk passed

+1 :green_heart:
compile
7m 28s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
compile
7m  8s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
checkstyle
1m 36s

trunk passed

+1 :green_heart:
mvnsite
2m 20s

trunk passed

+1 :green_heart:
javadoc
1m 58s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
2m 24s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
6m  2s

trunk passed

+1 :green_heart:
shadedclient
41m 49s

branch has no errors when building and testing our client artifacts.

-0 :warning:
patch
42m 13s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 34s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m  0s

the patch passed

+1 :green_heart:
compile
5m 53s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
cc
5m 53s

the patch passed

+1 :green_heart:
javac
5m 53s

the patch passed

+1 :green_heart:
compile
5m 47s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
cc
5m 47s

the patch passed

+1 :green_heart:
javac
5m 47s

the patch passed

-1 :x:
blanks
0m  0s
/blanks-eol.txt
The patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

-0 :warning:
checkstyle
1m 24s
/results-checkstyle-hadoop-hdfs-project.txt
hadoop-hdfs-project: The patch generated 35 new + 808 unchanged - 3 fixed = 843 total (was 811)

+1 :green_heart:
mvnsite
2m  6s

the patch passed

+1 :green_heart:
javadoc
1m 35s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
2m 15s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

-1 :x:
spotbugs
3m 23s
/new-spotbugs-hadoop-hdfs-project_hadoop-hdfs.html
hadoop-hdfs-project/hadoop-hdfs generated 2 new + 0 unchanged - 0 fixed = 2 total (was 0)

+1 :green_heart:
shadedclient
42m  6s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
2m 24s

hadoop-hdfs-client in the patch passed.

-1 :x:
unit
260m 54s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 46s

The patch does not generate ASF License warnings.

473m 50s

Reason
Tests

SpotBugs
module:hadoop-hdfs-project/hadoop-hdfs

Unread field:DataNode.java:[line 4468]

Unread field:DataNode.java:[line 4469]

Failed junit tests
hadoop.hdfs.server.datanode.fsdataset.impl.TestFsDatasetImpl

hadoop.tools.TestHdfsConfigFields

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6882

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets cc buflint bufcompat xmllint

uname
Linux ab11dda4e4d8 5.15.0-107-generic #117-Ubuntu SMP Fri Apr 26 12:26:49 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 87a59c95e2d6ec479f4fc90632ab5204de6d4c73

Default Java
Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/1/testReport/

Max. process+thread count
2787 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 22s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
buf
0m  0s

buf was not available.

+0 :ok:
buf
0m  0s

buf was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 17s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
26m 48s

trunk passed

+1 :green_heart:
compile
3m 55s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
compile
3m 36s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
checkstyle
0m 54s

trunk passed

+1 :green_heart:
mvnsite
1m 25s

trunk passed

+1 :green_heart:
javadoc
1m 12s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
1m 43s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
3m 44s

trunk passed

-1 :x:
shadedclient
23m 38s

branch has errors when building and testing our client artifacts.

-0 :warning:
patch
24m  4s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 18s

Maven dependency ordering for patch

-1 :x:
mvninstall
0m 21s
/patch-mvninstall-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

-1 :x:
mvninstall
0m 22s
/patch-mvninstall-hadoop-hdfs-project_hadoop-hdfs-client.txt
hadoop-hdfs-client in the patch failed.

-1 :x:
compile
0m 21s
/patch-compile-hadoop-hdfs-project-jdkUbuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2.txt
hadoop-hdfs-project in the patch failed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2.

-1 :x:
cc
0m 21s
/patch-compile-hadoop-hdfs-project-jdkUbuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2.txt
hadoop-hdfs-project in the patch failed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2.

-1 :x:
javac
0m 21s
/patch-compile-hadoop-hdfs-project-jdkUbuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2.txt
hadoop-hdfs-project in the patch failed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2.

-1 :x:
compile
0m 21s
/patch-compile-hadoop-hdfs-project-jdkPrivateBuild-1.8.0_412-8u412-ga-1~20.04.1-b08.txt
hadoop-hdfs-project in the patch failed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08.

-1 :x:
cc
0m 21s
/patch-compile-hadoop-hdfs-project-jdkPrivateBuild-1.8.0_412-8u412-ga-1~20.04.1-b08.txt
hadoop-hdfs-project in the patch failed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08.

-1 :x:
javac
0m 21s
/patch-compile-hadoop-hdfs-project-jdkPrivateBuild-1.8.0_412-8u412-ga-1~20.04.1-b08.txt
hadoop-hdfs-project in the patch failed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08.

-1 :x:
blanks
0m  0s
/blanks-eol.txt
The patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

-0 :warning:
checkstyle
0m 20s
/buildtool-patch-checkstyle-hadoop-hdfs-project.txt
The patch fails to run checkstyle in hadoop-hdfs-project

-1 :x:
mvnsite
0m 22s
/patch-mvnsite-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

-1 :x:
mvnsite
0m 25s
/patch-mvnsite-hadoop-hdfs-project_hadoop-hdfs-client.txt
hadoop-hdfs-client in the patch failed.

-1 :x:
javadoc
0m 22s
/patch-javadoc-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2.txt
hadoop-hdfs in the patch failed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2.

-1 :x:
javadoc
0m 22s
/patch-javadoc-hadoop-hdfs-project_hadoop-hdfs-client-jdkUbuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2.txt
hadoop-hdfs-client in the patch failed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2.

-1 :x:
javadoc
0m 22s
/patch-javadoc-hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_412-8u412-ga-1~20.04.1-b08.txt
hadoop-hdfs in the patch failed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08.

-1 :x:
spotbugs
0m 21s
/patch-spotbugs-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

-1 :x:
spotbugs
0m 21s
/patch-spotbugs-hadoop-hdfs-project_hadoop-hdfs-client.txt
hadoop-hdfs-client in the patch failed.

+1 :green_heart:
shadedclient
6m 14s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
0m 22s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

-1 :x:
unit
0m 21s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs-client.txt
hadoop-hdfs-client in the patch failed.

+0 :ok:
asflicense
0m 22s

ASF License check generated no output?

90m  8s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6882

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets cc buflint bufcompat xmllint

uname
Linux 25ea6f5295d2 5.15.0-106-generic #116-Ubuntu SMP Wed Apr 17 09:17:56 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 83bc3038bf7cc5c720022ba1c0feb2d38c377459

Default Java
Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/4/testReport/

Max. process+thread count
181 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs hadoop-hdfs-project/hadoop-hdfs-client U: hadoop-hdfs-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/4/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 19s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
buf
0m  1s

buf was not available.

+0 :ok:
buf
0m  1s

buf was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 12s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
25m 47s

trunk passed

+1 :green_heart:
compile
3m 32s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
compile
3m 26s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
checkstyle
1m  6s

trunk passed

+1 :green_heart:
mvnsite
1m 34s

trunk passed

+1 :green_heart:
javadoc
1m 16s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
2m  6s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
3m 41s

trunk passed

-1 :x:
shadedclient
3m  9s

branch has errors when building and testing our client artifacts.

-0 :warning:
patch
3m 22s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 22s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 13s

the patch passed

+1 :green_heart:
compile
3m 42s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
cc
3m 42s

the patch passed

+1 :green_heart:
javac
3m 42s

the patch passed

+1 :green_heart:
compile
3m 16s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
cc
3m 16s

the patch passed

+1 :green_heart:
javac
3m 16s

the patch passed

-1 :x:
blanks
0m  0s
/blanks-eol.txt
The patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

-0 :warning:
checkstyle
0m 46s
/results-checkstyle-hadoop-hdfs-project.txt
hadoop-hdfs-project: The patch generated 5 new + 808 unchanged - 4 fixed = 813 total (was 812)

+1 :green_heart:
mvnsite
1m  5s

the patch passed

+1 :green_heart:
javadoc
0m 56s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
1m 23s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
3m 13s

the patch passed

-1 :x:
shadedclient
23m 20s

patch has errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
0m 25s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs-client.txt
hadoop-hdfs-client in the patch failed.

-1 :x:
unit
0m 25s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

+0 :ok:
asflicense
0m 26s

ASF License check generated no output?

99m 41s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6882

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets cc buflint bufcompat xmllint

uname
Linux 57c40507d12c 5.15.0-106-generic #116-Ubuntu SMP Wed Apr 17 09:17:56 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / e87ec3168afcef813091eae80c98336b7318872b

Default Java
Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/3/testReport/

Max. process+thread count
189 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 20s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
buf
0m  0s

buf was not available.

+0 :ok:
buf
0m  0s

buf was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 16s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
19m 59s

trunk passed

+1 :green_heart:
compile
2m 49s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
compile
2m 40s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
checkstyle
0m 48s

trunk passed

+1 :green_heart:
mvnsite
1m 16s

trunk passed

+1 :green_heart:
javadoc
1m  7s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
1m 31s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
2m 59s

trunk passed

+1 :green_heart:
shadedclient
21m 20s

branch has no errors when building and testing our client artifacts.

-0 :warning:
patch
21m 33s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 22s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m  2s

the patch passed

+1 :green_heart:
compile
2m 43s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
cc
2m 43s

the patch passed

+1 :green_heart:
javac
2m 43s

the patch passed

+1 :green_heart:
compile
2m 40s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
cc
2m 40s

the patch passed

+1 :green_heart:
javac
2m 40s

the patch passed

-1 :x:
blanks
0m  0s
/blanks-eol.txt
The patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

-0 :warning:
checkstyle
0m 40s
/results-checkstyle-hadoop-hdfs-project.txt
hadoop-hdfs-project: The patch generated 4 new + 808 unchanged - 4 fixed = 812 total (was 812)

+1 :green_heart:
mvnsite
1m  8s

the patch passed

+1 :green_heart:
javadoc
0m 49s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
1m 22s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
3m  1s

the patch passed

+1 :green_heart:
shadedclient
21m  4s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
1m 47s

hadoop-hdfs-client in the patch passed.

-1 :x:
unit
204m 22s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 22s

The patch does not generate ASF License warnings.

310m 24s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestDistributedFileSystemWithECFileWithRandomECPolicy

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/5/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6882

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets cc buflint bufcompat xmllint

uname
Linux da2dee73b151 5.15.0-106-generic #116-Ubuntu SMP Wed Apr 17 09:17:56 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 2c653c93086168af62655c155b4c6bca08c0b8fa

Default Java
Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/5/testReport/

Max. process+thread count
4554 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/5/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 46s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
buf
0m  1s

buf was not available.

+0 :ok:
buf
0m  1s

buf was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 42s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
36m 46s

trunk passed

+1 :green_heart:
compile
6m  8s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
compile
6m  0s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
checkstyle
1m 36s

trunk passed

+1 :green_heart:
mvnsite
2m 25s

trunk passed

+1 :green_heart:
javadoc
1m 55s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
2m 27s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
6m  2s

trunk passed

+1 :green_heart:
shadedclient
41m 44s

branch has no errors when building and testing our client artifacts.

-0 :warning:
patch
42m  7s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 32s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m  4s

the patch passed

+1 :green_heart:
compile
5m 59s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
cc
5m 59s

the patch passed

+1 :green_heart:
javac
5m 59s

the patch passed

+1 :green_heart:
compile
5m 49s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
cc
5m 49s

the patch passed

+1 :green_heart:
javac
5m 49s

the patch passed

-1 :x:
blanks
0m  0s
/blanks-eol.txt
The patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

-0 :warning:
checkstyle
1m 25s
/results-checkstyle-hadoop-hdfs-project.txt
hadoop-hdfs-project: The patch generated 5 new + 807 unchanged - 4 fixed = 812 total (was 811)

+1 :green_heart:
mvnsite
2m  4s

the patch passed

+1 :green_heart:
javadoc
1m 37s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
2m 16s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
5m 59s

the patch passed

+1 :green_heart:
shadedclient
41m 31s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
2m 24s

hadoop-hdfs-client in the patch passed.

+1 :green_heart:
unit
269m 28s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 48s

The patch does not generate ASF License warnings.

460m 56s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6882

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets cc buflint bufcompat xmllint

uname
Linux 53de2e819488 5.15.0-107-generic #117-Ubuntu SMP Fri Apr 26 12:26:49 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / e87ec3168afcef813091eae80c98336b7318872b

Default Java
Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/2/testReport/

Max. process+thread count
2996 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 20s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
buf
0m  0s

buf was not available.

+0 :ok:
buf
0m  0s

buf was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 23s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
20m  1s

trunk passed

+1 :green_heart:
compile
2m 51s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
compile
2m 48s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
checkstyle
0m 50s

trunk passed

+1 :green_heart:
mvnsite
1m 20s

trunk passed

+1 :green_heart:
javadoc
1m 10s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
1m 38s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
3m 14s

trunk passed

+1 :green_heart:
shadedclient
21m 27s

branch has no errors when building and testing our client artifacts.

-0 :warning:
patch
21m 40s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 22s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m  4s

the patch passed

+1 :green_heart:
compile
2m 50s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
cc
2m 50s

the patch passed

+1 :green_heart:
javac
2m 50s

the patch passed

+1 :green_heart:
compile
2m 44s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
cc
2m 44s

the patch passed

+1 :green_heart:
javac
2m 44s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 39s
/results-checkstyle-hadoop-hdfs-project.txt
hadoop-hdfs-project: The patch generated 1 new + 808 unchanged - 4 fixed = 809 total (was 812)

+1 :green_heart:
mvnsite
1m  7s

the patch passed

+1 :green_heart:
javadoc
0m 49s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
1m 32s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
3m 11s

the patch passed

+1 :green_heart:
shadedclient
20m 57s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
1m 51s

hadoop-hdfs-client in the patch passed.

+1 :green_heart:
unit
196m 33s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 29s

The patch does not generate ASF License warnings.

303m 56s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/6/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6882

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets cc buflint bufcompat xmllint

uname
Linux c49935153449 5.15.0-106-generic #116-Ubuntu SMP Wed Apr 17 09:17:56 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / e856506e337391a0b7b2cb4cec23073453f5576b

Default Java
Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/6/testReport/

Max. process+thread count
4273 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/6/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@haiyang1987   Hello ,sir . This is based on your version  , and resolve lease and  changed some something else .   @ZanderXu @haiyang1987  Could you please review this, Thanks.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 19s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
buf
0m  0s

buf was not available.

+0 :ok:
buf
0m  0s

buf was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m  5s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
20m  7s

trunk passed

+1 :green_heart:
compile
2m 49s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
compile
2m 41s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
checkstyle
0m 49s

trunk passed

+1 :green_heart:
mvnsite
1m 20s

trunk passed

+1 :green_heart:
javadoc
1m  6s

trunk passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
1m 36s

trunk passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
2m 57s

trunk passed

+1 :green_heart:
shadedclient
21m 13s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 22s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m  8s

the patch passed

+1 :green_heart:
compile
2m 44s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
cc
2m 44s

the patch passed

+1 :green_heart:
javac
2m 44s

the patch passed

+1 :green_heart:
compile
2m 37s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
cc
2m 37s

the patch passed

+1 :green_heart:
javac
2m 37s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 42s
/results-checkstyle-hadoop-hdfs-project.txt
hadoop-hdfs-project: The patch generated 1 new + 808 unchanged - 4 fixed = 809 total (was 812)

+1 :green_heart:
mvnsite
1m  1s

the patch passed

+1 :green_heart:
javadoc
0m 53s

the patch passed with JDK Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2

+1 :green_heart:
javadoc
1m 28s

the patch passed with JDK Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

+1 :green_heart:
spotbugs
3m  5s

the patch passed

+1 :green_heart:
shadedclient
21m  8s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
1m 49s

hadoop-hdfs-client in the patch passed.

-1 :x:
unit
203m 40s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 31s

The patch does not generate ASF License warnings.

310m  4s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.blockmanagement.TestDatanodeManager

Subsystem
Report/Notes

Docker
ClientAPI=1.46 ServerAPI=1.46 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/7/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6882

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets cc buflint bufcompat xmllint

uname
Linux c890db85f5f0 5.15.0-106-generic #116-Ubuntu SMP Wed Apr 17 09:17:56 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 0f8d5184a630adc1f4a42da5fe9dc7c861b4f2b5

Default Java
Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.23+9-post-Ubuntu-1ubuntu120.04.2 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_412-8u412-ga-1~20.04.1-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/7/testReport/

Max. process+thread count
4158 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6882/7/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


[FEATURE] Add Google Cloud Cloud Functions as an output type

Motivation
Cloud Functions is a scalable pay-as-you-go function as a service (FaaS) to run your code with zero server management. In Falcosidekick, there is support available for some Google Cloud services like Storage, Pub/Sub, etc. So, it would be nice to have if we add another Google Cloud service named Cloud Function as an output for the Falcosidekick.
Feature
As I mentioned above, we have support for some Google Cloud services in Falcosidekick, we can add the code for Cloud Functions support to the available code used for implementing other Google Cloud services.
Alternatives
Alternatively, there is no support for Cloud Run which is another FaaS solution in Google Cloud in Falcosidekick, maybe we can add that later.
Additional context

cc: @cartyc @Issif

1 for cloud run.


[BEAM-14247] [Website] add image

[BEAM-14247]

add compressed ipng
add image to content page

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

LGTM

@bullet03 can you please align the graph to center?


Java SDK: Rewrap using KAS

Set up the SDK communications with KAS and correctly create the rewrap requests.

completed as part of PR #52


Unity Ethereum Support

There is no plan to support this at this time.

@mmcs85 Sorry to hear. Can you tell why? Ethereum is still the most popular blockchain for dapps


Can't update records which have datafeeds set (server error: invalid feed pointer)

import ns1
api = ns1.NS1(apiKey='[…]')
record = api.loadRecord('cloud.na3.pcigdc.com', 'A')
record.update(ttl=3)

results into
Traceback (most recent call last):
  File "x.py", line 6, in <module>
    record.update(ttl=3)
  File "/usr/lib/python3.7/site-packages/ns1/records.py", line 109, in update
    callback=success, errback=errback, **kwargs)
  File "/usr/lib/python3.7/site-packages/ns1/rest/records.py", line 136, in update
    errback=errback)
  File "/usr/lib/python3.7/site-packages/ns1/rest/resource.py", line 73, in _make_request
    return self._transport.send(type, self._make_url(path), **kwargs)
  File "/usr/lib/python3.7/site-packages/ns1/rest/transport/requests.py", line 60, in send
    resp.text)
ns1.rest.errors.ResourceException: server error: invalid feed pointer

This is happening with 0.9.19 release.

I'm having trouble recreating this error in python3.7 using the requests transport.  Can you enable verbose logging and print the output (example below)?
import logging
import ns1
logging.basicConfig(level=logging.DEBUG)
config = ns1.Config()
config.createFromAPIKey('[...]')
config['verbosity'] = 5
config['transport'] = 'basic'
api = ns1.NS1(config=config)
rec = api.loadRecord('cloud.na3.pcigdc.com', 'A')
rec.update(ttl=3)

Also, can you confirm the configuration on the NS1 side?  Is your top level zone pcigdc.com or na3.pcigdc.com?

DEBUG:ns1.rest.transport.basic:{'User-Agent': 'ns1-python 0.9.19 python 0x50791618 linux', 'X-NSONE-Key': '<redacted>'}
DEBUG:ns1.rest.transport.basic:GET https://api.nsone.net/v1/zones/na3.pcigdc.com/cloud.na3.pcigdc.com/A None
DEBUG:ns1.rest.transport.basic:{'User-Agent': 'ns1-python 0.9.19 python 0x50791618 linux', 'X-NSONE-Key': '<redacted>'}
DEBUG:ns1.rest.transport.basic:POST https://api.nsone.net/v1/zones/na3.pcigdc.com/cloud.na3.pcigdc.com/A {"zone": "na3.pcigdc.com", "domain": "cloud.na3.pcigdc.com", "type": "A", "ttl": 3}
Traceback (most recent call last):
  File "/usr/lib/python3.7/site-packages/ns1/rest/transport/basic.py", line 82, in send
    resp = opener.open(request, timeout=self._timeout)
  File "/usr/lib64/python3.7/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/usr/lib64/python3.7/urllib/request.py", line 641, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib64/python3.7/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/usr/lib64/python3.7/urllib/request.py", line 503, in _call_chain
    result = func(*args)
  File "/usr/lib64/python3.7/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 400: Bad Request

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "x.py", line 10, in <module>
    rec.update(ttl=3)
  File "/usr/lib/python3.7/site-packages/ns1/records.py", line 109, in update
    callback=success, errback=errback, **kwargs)
  File "/usr/lib/python3.7/site-packages/ns1/rest/records.py", line 136, in update
    errback=errback)
  File "/usr/lib/python3.7/site-packages/ns1/rest/resource.py", line 73, in _make_request
    return self._transport.send(type, self._make_url(path), **kwargs)
  File "/usr/lib/python3.7/site-packages/ns1/rest/transport/basic.py", line 88, in send
    handleProblem(resp.code, resp, body)
  File "/usr/lib/python3.7/site-packages/ns1/rest/transport/basic.py", line 77, in handleProblem
    body=msg)
ns1.rest.errors.ResourceException: server error, status code: 400: invalid feed pointer

Yes, changing configuration using web interface works.

Trouble with recreating was on my end, sorry about that.
I can recreate the issue via curl, so it's looking like this is not a bug with the python module, but with the API itself.  This issue occurs when a POST call is made whose body does not explicitly include the answer, meta and full value of the defined metadata type for any answer that has a datafeed set.
I've created a bug internally so NS1's backend team can take a look at this.  Will follow up when I have more info on progress there.

As an update here, this has been accepted as a bug with the API by NS1's backend team and has been added to their backlog.  I'll close this issue out once that has been completed.
In the mean time, the record can still be updated as long as the full answer that has the datafeed defined is included with the update.  So the workaround here would be to include the answers returned by the loadRecord as a parameter toupdate.  I.e:
import logging
import ns1
logging.basicConfig(level=logging.DEBUG)
config = ns1.Config()
config.createFromAPIKey('[...]')
config['verbosity'] = 5
api = ns1.NS1(config=config)
rec = api.loadRecord('cloud.na3.pcigdc.com', 'A')
rec.update(ttl=3, answers=rec.data['answers'])


Improved Flee Mortals Compatibility

Sorry to be a pain, but there are some things still not working as expected with the latest update
First, and weirdest, the Senses parsing seems to only work the first time an actor is imported, then doesnt work until you refresh
https://github.com/jbhaywood/5e-statblock-importer/assets/86370342/67f50f92-88b3-400c-b8ca-1c8257d2bc09
Abilities
Some FM actors (Goblin Assassin for example) seem to have their Abilities in a different format, some are on separate lines, and those import fine, some on one line and those dont import ability scores at all
 works (fire Giant Red Fist)
 does not work (Goblin Assassin)
Damage Immunities/Resistances/Vulnerablities and Condition Immunities

Custom options - Some FM creatures are immune to the Flanking Condition (Gibbering Mouther),  Some have conditions on their immunities (Mimic is immune to bludgeoning from falling, Hobgoblin Smokebinder is immune to BPS mundane damage, and Grapple condition in smoke form)
When FM has these custom options (and for BPS from mundane), they are usually prefixed with a ; or sometimes in parenthesis (Hobgoblin Smokebinder)

When these special characters/custom items are present on the actor, they are imported somewhere in the data, but not quite accurately as you do not see them in the config

Armors
Some armors are not being picked up, mostly an issue if they also have a shield, seems to be due to missing the word armor on these, I've noticed Padded, Studded Leather, Plate, Half Plate, Splint

Action Types
It looks like all Actions are assumed to be Weapon Attacks, but several monsters (including official monsters, particularly in Mordenkainen's) have actions that are Melee or Ranged Spell Attacks
 (Human Storm Wizard)

That senses thing is really weird. The regex is failing to match anything on the second time through. The only reason I can think of is because it's using the "global" flag. But other things are using it, like speed, and they don't have the same problem. Luckily you don't really need to import twice in a row though, since you can just copy the actor or the token once they're in Foundry.
I did make a minor fix so that it at least picks up any special sense text.
If you want to tackle those other issues, go for it. Hopefully it's a bit easier to work on now. I'm going to take a Foundry module break and work on my campaign. Of course the party zigged when I thought they would zag and now I have a bunch of planning to do. 😅
If not, I'll get to it eventually.

I have been trying to install 5e Statblock Importer and keep getting this message: "ERROR: Unable to Load valid module manifest data from: "HTTPS://raw.githubusercontent.com/jbhaywood/5e-statblock-importer/main/module.json" Cannot read properties of (undefined reading '0) - I am using the latest version of Foundry 11 Build 314. This module previously worked in another world, and was a life saver - is there a fix to the above, I would love to know how to fix it.

I have been trying to install 5e Statblock Importer and keep getting this message: "ERROR: Unable to Load valid module manifest data from: "HTTPS://raw.githubusercontent.com/jbhaywood/5e-statblock-importer/main/module.json" Cannot read properties of (undefined reading '0) - I am using the latest version of Foundry 11 Build 314. This module previously worked in another world, and was a life saver - is there a fix to the above, I would love to know how to fix it.

Can you open a new issue on this? It sounds like a more serious issue that what's being reported here. About the actual error, I just updated to the same Foundry build and aren't seeing any module.json issues. I'm not sure what's going on there.

I am also seeing the error where spell attacks are being interpreted as weapon attacks, and it changes the attack/damage values despite what is in the imported text.

Text:
Bugbear Channeler
Medium Humanoid (Bugbear), Any Alignment
Armor Class 14 (Studded Leather)
Hit Points 102 (12d8 + 48)
Speed 30 ft.
STR 10 (+0)
DEX 15 (+2)
CON 18 (+4)
INT 15 (+2)
WIS 16 (+3)
CHA 18 (+4)
Saving Throws Wis +6, Cha +7
Skills Arcana +5, Perception +6, Stealth +5, Survival +6
Senses Darkvision 60 ft., passive Perception 16
Languages Common, Goblin
Proficiency Bonus +3
Challenge 7 (2,900 XP)
Bugbear’s Inspiration. Allied Humanoids within 15 feet of the channeler who can see or hear them have advantage on Wisdom and Charisma saving throws.
ACTIONS
Multiattack. The channeler makes two Channeling Staff attacks.
Channeling Staff. Melee Spell Attack: +7 to hit, reach 5 ft., one target. Hit: 8 (1d8 + 4) bludgeoning damage plus 9 (2d8) lightning damage. If the target is a Large or smaller creature, the channeler can move them up to 10 feet horizontally.
Vine Eruption (3/Day; 4th-Level Spell). The channeler attempts to use primal magic to overload a creature they can see within 60 feet of them. The target must make a DC 15 Constitution saving throw as thorny vines erupt from their body and attempt to envelop them. On a failed save, the target takes 27 (5d10) piercing damage and is restrained until the vines are destroyed. On a successful save, the target takes half as much damage and isn’t restrained. A creature restrained in this way takes 5 (1d10) piercing damage at the start of each of their turns. The vines have AC 15, 10 hit points, and immunity to psychic damage.
Ball Lightning (1/Day; 7th-Level Spell; Concentration). The channeler creates a 20-foot-radius sphere of electricity centered on a point they can see within 120 feet of them. The sphere lasts for 1 minute. When an enemy starts their turn in that area, they must succeed on a DC 15 Dexterity saving throw or take 27 (6d8) lightning damage. As a bonus action, the channeler can move the sphere up to 20 feet.
REACTIONS
Shared Shield (2/Day; 2nd-Level Spell). When a creature the channeler can see within 30 feet of them is hit with an attack, the channeler can create a barrier of magical force around that target. Until the start of the channeler’s next turn, the target gains a +5 bonus to AC, including against the triggering attack.
UTILITY SPELLS
In addition to any other spells in this stat block, the channeler can cast the following spells, using Charisma as the spellcasting ability (spell save DC 15):
At will: light A, message A
1/day each: detect magic A, disguise self A, identify +

Actually, it's not just spell attacks:

Here's a basic greatsword attack it's just decided to deduct 2 from.
Bugbear Commander
Medium Humanoid (Bugbear), Any Alignment
Armor Class 18 (Plate)
Hit Points 97 (15d8 + 30)
Speed 30 ft.
STR 18 (+4)
DEX 15 (+2)
CON 15 (+2)
INT 17 (+3)
WIS 14 (+2)
CHA 15 (+2)
Saving Throws Wis +5, Cha +5
Skills Intimidation +5, Persuasion +5, Stealth +5, Survival +5
Senses Darkvision 60 ft., passive Perception 12
Languages Common, Goblin
Proficiency Bonus +3
Challenge 5 (1,800 XP)
Commander’s Inspiration. Allied Humanoids within 30 feet of the commander who can see or hear them have advantage on Wisdom and Charisma saving throws and gain a +1 bonus to weapon attack and damage rolls.
ACTIONS
Multiattack. The commander makes two Greatsword or two Longbow attacks.
Greatsword. Melee Weapon Attack: +7 to hit, reach 5 ft., one target. Hit: 11 (2d6 + 4) slashing damage, and one ally within 30 feet of the target who can see the commander has advantage on their next melee attack roll made against the target before the start of the commander’s next turn.
Longbow. Ranged Weapon Attack: +5 to hit, range 150/600 ft., one target. Hit: 6 (1d8 + 2) piercing damage.
Bark Orders (Recharge 6). Each ally within 60 feet of the commander who can hear and understand them can use a reaction to move up to their speed and make a weapon attack.
REACTIONS
Bolstering Pride. When another creature within 60 feet of the commander who can hear and understand them makes a saving throw, the commander can give that creature advantage on the saving throw.

Yea, there are likely several things that need to be updated for the new 2.4 version of DnD, the above seems to be caused by all actions defaulting to Dexterity. The Human Storm Wizard may be a good outlier example as well since the Arcane Staff should be set to Int

I've been importing from a variety of sources and am consistently seeing the problems that I suspect are 2.4 issues.  Maybe that should be a separate issue, since it's not just a Flee, Mortals problem.

These new issues should be fixed in v1.81.6. It would be useful to open new issues for any remaining problems, with one issue per problem. That would make them easier to track.

Sorry about the large issue, thought it made sense to keep everything in one since it was related to an individual source, but I see how that makes it challenging to track and handle. It looks like a lot of the items I brought up in the original comment have been fixed and I appreciate that a lot.
I was finally able to find some time to come back around to this and there are still a few things I've noticed, but I will create new issues for the individual items, and I have some PRs ready too.

Thanks for looking into those other issues! I super appreciate it. It can be tough to make sure every edge case is handled correctly. Of course, eventually ChatGPT will replace all of this hand made parsing code. 😅


Feat/plugin dep resolvers

Description
closes #535
Breaking Changes
I have tested my changes on:
ReNative project directly:

[ ] ios simulator
[ ] ios device
[ ] android simulator
[ ] android device
[ ] web browser
[ ] web -e next browser
[ ] tvos simulator
[ ] tvos device
[ ] androidtv simulator
[ ] androidtv device
[ ] androidwear simulator
[ ] androidwear device
[ ] tizen simulator
[ ] tizen device
[ ] tizenmobile simulator
[ ] tizenwatch device
[ ] webos simulator
[ ] webos device
[ ] macos
[ ] windows
[ ] chromecast device

New project:

[ ] ios simulator
[ ] ios device
[ ] android simulator
[ ] android device
[ ] web browser
[ ] web -e next browser
[ ] tvos simulator
[ ] tvos device
[ ] androidtv simulator
[ ] androidtv device
[ ] androidwear simulator
[ ] androidwear device
[ ] tizen simulator
[ ] tizen device
[ ] tizenmobile simulator
[ ] tizenwatch device
[ ] webos simulator
[ ] webos device
[ ] macos
[ ] windows
[ ] chromecast device

Existing Project created with previous version of renative:

[ ] ios simulator
[ ] ios device
[ ] android simulator
[ ] android device
[ ] web browser
[ ] web -e next browser
[ ] tvos simulator
[ ] tvos device
[ ] androidtv simulator
[ ] androidtv device
[ ] androidwear simulator
[ ] androidwear device
[ ] tizen simulator
[ ] tizen device
[ ] tizenmobile simulator
[ ] tizenwatch device
[ ] webos simulator
[ ] webos device
[ ] macos
[ ] windows
[ ] chromecast device

Codecov Report

Merging #543 into master will increase coverage by 0.28%.
The diff coverage is 20.39%.

@@            Coverage Diff             @@
##           master     #543      +/-   ##
==========================================
+ Coverage   19.28%   19.56%   +0.28%
==========================================
  Files          64       69       +5
  Lines        6814     7217     +403
  Branches     1464     1522      +58
==========================================
+ Hits         1314     1412      +98
- Misses       4187     4436     +249
- Partials     1313     1369      +56

Flag
Coverage Δ

#rnvunittests
19.56% <20.39%> (+0.28%)
:arrow_up:

Impacted Files
Coverage Δ

packages/rnv/src/common.js
24.56% <0.00%> (-0.44%)
:arrow_down:

packages/rnv/src/config.js
8.97% <0.00%> (ø)

packages/rnv/src/platformTools/android/index.js
4.06% <0.00%> (-0.02%)
:arrow_down:

packages/rnv/src/platformTools/apple/index.js
4.50% <0.00%> (-0.02%)
:arrow_down:

...ackages/rnv/src/platformTools/apple/swiftParser.js
3.03% <0.00%> (-0.15%)
:arrow_down:

packages/rnv/src/platformTools/chromecast/index.js
25.00% <0.00%> (-2.28%)
:arrow_down:

packages/rnv/src/platformTools/electron/index.js
8.18% <0.00%> (-0.08%)
:arrow_down:

packages/rnv/src/platformTools/firefox/index.js
12.76% <0.00%> (-0.28%)
:arrow_down:

packages/rnv/src/platformTools/tizen/index.js
9.33% <0.00%> (-0.05%)
:arrow_down:

packages/rnv/src/platformTools/web/index.js
7.57% <0.00%> (-0.24%)
:arrow_down:

... and 25 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2e798f9...6c33bbb. Read the comment docs.


feat: add user to all-contributors

PR Checklist

[x] Addresses an existing open issue: fixes #156
[x] That issue was marked as status: accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview
I added some logic to add the running user to the all-contributors list (according to the authenticated GitHub CLI account).
At first, I tried only to use the owner variable that the user prompted, but it failed the tests, where a fabricated owner name was provided; if you prefer to only use the provided owner name without reaching out to gh CLI, we might need to adjust the tests a little bit.

Thanks, I applied these changes


Promise.All should not use value returning catch

Now that we have IPromise Catch(Action on IPromise the static All method should us the non-value version of Catch to avoid the ugly return default(PromisedT)

Good point. I think this would work by just deleting that return default(PromisedT); line. Can't see how that would break any existing code, so unless there's anything else you think would be worth changing I'll switch it over. 👍

Yes, I can't see how it would break anything either.

I've fixed this now.


Please invite me to the GitHub Community Organization

Name
Rot
Discord Username (if applicable)
Rot#0959
Additional Context
Hi, I'd like to join your organization.
Thanks.

It's great having you contribute to this project
If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord Server and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.

Invitation sent for the GitHub Organisation. Welcome to the community 🎉#
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/jaidevstudio/people
Like this

Tips for practising:
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps

Please check your email or organisation page 🙏


Add pre-publish script

It should do a couple things, suggested by @danielmewes in https://github.com/rethinkdb/horizon/pull/430#issuecomment-220206951
It should:

[ ] Check for uncommitted files in the current directory

Some other nice to haves might be automatically tagging in git with the version number, and maybe some other checklist items

My original suggestion was actually just for the first part. A positive list always comes with the risk of forgetting files in the package, which also sucks.
We can use git status | grep -q "working directory clean" to check for any uncommitted files (though in the whole repository, not just the current directory).

Ok, removed the second checkbox

npm version might save some work. It checks for uncommitted files in the working directory before updating the package.json, committing, and tagging the commit.

This is done in 2.0.0. It handles changing deps


Selenide waitWhile() and waitUntil() functions ignore custom timeout if value exceeds certain number

The problem
Methods waitWhile() and waitUntil() do not wait full custom timeout but rather wait the global one if for example timeout value exceeds 10k of miliseconds
Details
Tested on values such as 8000, 9000, 12000 and 50000. First two worked, the other ones did not work (selenide was waiting 9 sec at max). Condition used was "Visible".
Tell us about your environment

Selenide Version: 5.11.1
Chrome\Firefox\IE Version:  Chrome version 80.0.39
Browser Driver Version:
Selenium Version: selenium 3 latest
OS Version: Windows 10

Code To Reproduce Issue [ Good To Have ]
element.waitUntil(Condition.visible, 50000);
element.waitWhile(Condition.visible, 50000);
I also tested with Long.valueOf() function.

okay it seems 2 things i did bad:

using raw integer as value instead of using Long.valueOf() but maybe it should be converted inside selenide
using wrong condition to task. I wanted to wait until element dissapears so i did wrong wait method (should be waitWhile() instead of waitUntil(), my bad)

@asolntsev what do you think about first point?

@SeleniumTestAB Can you show the code? I cannot reproduce the problem.


Support Laravel 9

Reason For Change
Fixes issue when used with Laravel 9.
Resolves
Issue #84.

Hello @usernotnull
I have a question about this PR. Why just not use share alias instead of Share::class? I mean, it is bound to the share alias in the ShareServiceProvider, so we can just use it. I would like to understand the logic behind your decision, so could you please explain it a little bit?

@kudashevs sure.
In the upgrade guide to L9, it mentions that:
The getFacadeAccessor method must always return a container binding key. In previous releases of Laravel, this method could return an object instance; however, this behavior is no longer supported. If you have written your own facades, you should ensure that this method returns a container binding string.
The lib was working until I upgraded to L9. After changing the facade accessor to a string key as in this PR, I got it to work.

The lib was working until I upgraded to L9. After changing the facade accessor to a string key as in this PR, I got it to work.

I had the same problem. However, I changed it to return 'share'; because it is the container binding key in this case (in ShareServiceProvider.php we have $this->app->bind('share', function () ...). That's why I asked you about your decision :)
Thanks you @usernotnull for your answer, I appreciate it :thumbsup:

Would you like me to update this branch with 'share' key?

PS. I'm not sure which way is better, btw. I see advantages and disadvantages in both of them.

The framework uses both.
IMHO the word 'share' is too broad, and an application may use that same key somewhere else and clash.
So it's probably safer to stick to the class name.

Thank you @usernotnull for your work on this! I appreciate that you took the time to fix!
I agree that it's better to stick to the class name as well 👍
I'm happy to merge this and tag a new release as soon as all workflows pass.

@jorenvh done.
I added the test cache to gitignore, and removed windows from the workflow (I aimed to keep older versions of PHP and laravel, but too many excludes to add in the list of os, so instead I just stuck to ubuntu, that's how spatie packages have it in any case :) ).
Also to note, I modified Share class to reset the $html value. This makes the tests pass (which was not an issue before changing the key of the facade accessor 🤷🏼). In any case, all tests green now 👍🏼

@usernotnull thank you very much sir!
I haven't had time to look at the exact changes yet. Are there any breaking changes and do I need to tag a new major release?

@jorenvh you're welcome!
No breaking changes. A minor release tag is enough 👍🏼

Thanks again for you work on this! 👍 I will take a closer look & tag a new release later today or during the weekend

@usernotnull if just tagged & released version 4.2.0 https://github.com/jorenvh/laravel-share/releases/tag/4.2.0

@jorenvh Great. Have a great weekend :)


MAINT: mypy: ignore mpmath imports in mypy.ini

Reference issue
N/A
What does this implement/fix?
As discussed in https://github.com/scipy/scipy/pull/13721#discussion_r598389079 and also suggested on the mailing list: https://mail.python.org/pipermail/scipy-dev/2021-March/024635.html, this adds the mpmath import to mypy.ini file.
mpmath should be ignored in the mypy.ini file so that future PRs don't have to use the type: ignore[import] directive every time mpmath is imported.
Additional information

Oops -- A lot of Azure failures on windows! Are any of them related to these changes?

Oops -- A lot of Azure failures on windows! Are any of them related to these changes?
Edit: All of them seem like spurious failures:
Failures
 - llvm (exited 1) - llvm not installed. An error occurred during installation:
 The remote server returned an error: (503) Server Unavailable. Service Unavailable

yeah I’m seeing the same it looks like in a previous step the step completed but the llvm install didn’t happen.
I restarted those builds, hopefully just a transient issue.

Everything is green so merges to resolve issue, thanks all.


Reserved CPUs in shared pool are allocated performance profile

I am pasting the powerworkloads configuration currently running in my machine.
apiVersion: v1
items:
- apiVersion: power.intel.com/v1
  kind: PowerWorkload
  metadata:
    annotations:
      kubectl.kubernetes.io/last-applied-configuration: |
        {"apiVersion":"power.intel.com/v1","kind":"PowerWorkload","metadata":{"annotations":{},"name":"shared-nuc-workload","namespace":"intel-power"},"spec":{"allCores":true,"name":"shared-nuc-workload","powerNodeSelector":{"kubernetes.io/hostname":"nuc"},"powerProfile":"shared","reservedCPUs":[0,1]}}
    creationTimestamp: "2022-08-23T10:37:00Z"
    generation: 1
    name: shared-nuc-workload
    namespace: intel-power
    resourceVersion: "1392"
    uid: 7797932b-742f-45e3-bd54-21bbc95e871e
  spec:
    allCores: true
    name: shared-nuc-workload
    powerNodeSelector:
      kubernetes.io/hostname: nuc
    powerProfile: shared
    reservedCPUs:
    - 0
    - 1
- apiVersion: power.intel.com/v1
  kind: PowerWorkload
  metadata:
    creationTimestamp: "2022-08-23T10:30:53Z"
    generation: 13
    name: performance-nuc
    namespace: intel-power
    resourceVersion: "3471"
    uid: 50ffde92-20b5-432c-88dd-320bc6012718
  spec:
    name: performance-nuc
    powerProfile: performance
    workloadNodes:
      containers:
      - exclusiveCpus:
        - 0
        id: containerd://74fe58c7227fd87f03c700df1c1c69b1e16a711ad568fbb56bb8d04b44de3526
        name: example-power-container-cmk
        pod: example-power-pod
        powerProfile: performance
      - exclusiveCpus:
        - 0
        id: containerd://74fe58c7227fd87f03c700df1c1c69b1e16a711ad568fbb56bb8d04b44de3526
        name: example-power-container-cmk
        pod: example-power-pod
        powerProfile: performance
      cpuIds:
      - 0
      name: nuc
kind: List
metadata:
  resourceVersion: ""

The CPU 0 is getting allocated performance profile even after shared pool reserved that CPU. am I missing something? Please help!
Below is the powernode config. The CPU 0 was present before in unaffectedcores, however after I scheduled the workload, it vanished!!
apiVersion: v1
items:
- apiVersion: power.intel.com/v1
  kind: PowerNode
  metadata:
    creationTimestamp: "2022-08-23T10:30:51Z"
    generation: 199
    name: nuc
    namespace: intel-power
    resourceVersion: "3727"
    uid: af459fcc-86a9-48d5-b9f6-5939bda1fab0
  spec:
    nodeName: nuc
    powerProfiles:
    - 'performance: 4700000 || 4500000 || performance'
    - 'shared: 1500000 || 1000000 || power'
    sharedPool: shared || 1500000 || 1000000 || 7,2-6
    unaffectedCores: "1"
kind: List
metadata:
  resourceVersion: ""

As per the docs, when I say CPU 0,1 are reserved both in Kubelet config and in shared workload. It will not be touched by power manager and all other CPUs will have "shared" level of frequency before I request performance on them. And when I do the request, one of the CPUs from 2-7 must have switched from "shared" frequency level to "performance" level. Instead, CPU 0 is being selected by the scheduler. I am using supported k3s version. And, my kubelet args look like below
kubelet-arg:
  - "reserved-cpus=0-1"

The Problem is solved after adding static policy in K3S kubelet args in this file: /etc/rancher/k3s/config.yaml
kubelet-arg:
  - "reserved-cpus=0-1"
  - "cpu-manager-policy=static"

After the edit, Remove this below file.
sudo rm /var/lib/kubelet/cpu_manager_state
And then restart k3s.
sudo systemctl daemon-reload
sudo systemctl restart k3s

Then, re-deploy power manager related pods by simply deleting powerconfig.
kubectl delete powerconfig power-config -n intel-power
Now, re-apply power-config, shared-profile and shared-workload configs.


Allow Android to use timerfd

Hey there! I'm from the Android Bluetooth team, and we're using timerfd for our timers (even in our old C/C++ code) since they can wake the device. Here's a small patch that opens up timerfd to the Android config too.

Looks good, assuming it works (we don't run Android tests in CI).  But could you also please add a CHANGELOG entry?  And since you obviously have an Android environment, could you please try running the test suite and report what happens?

Superseded by #1509.


Some small README improvements

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
None

License
Apache 2.0

What's in this PR?

Why?

Additional context

Checklist

[ ] Implementation tested
[ ] Error handling code meets the guideline
[ ] Logging code meets the guideline
[ ] User guide and development docs updated (if needed)

To Do

[ ] If the PR is not complete but you want to discuss the approach, list what remains to be done here

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Implemented UPDATE

Take a look at this. I have yet to implement all of the tests, as this is mainly to check the logic to make sure that I'm on the correct track and all. First time working with their pipeline stuff, it's kind of interesting.

Alright, this is now ready for the final review pass. That ORDER BY was a great idea to add to the tests because I found a bug through it. Was a pain to debug though (as is anything in the pipeline).


Browser history mode not working properly

In producction, page navigation across browser's history is not working properly, sometimes user gets 404 error

I suposedly requires serverver configuration, based on the oficial vue-router webpage
https://v3.router.vuejs.org/guide/essentials/history-mode.html

With the configuration added to the nginx server, the history mode and web search is working properly


When importing from Figma, simplifyStroke in svgOptions does not work

Looks like Figma has updated the default setting for simplifyStroke to be true. importFromFigma assumes that the default is false, and does not set svg_simplify_stroke=false in the query, which breaks this feature.

Fixed in 4.0.7.
Thanks a lot for reporting!


Documentation request - document how to override CSS elements

Looking here: https://mkdocstrings.github.io/python/customization/ it describes the CSS classes which are used with mkdocstrings-python.  Unfortunelty some of us are coming into this cold and so it's much less obvious how to customise them.  Despite giving example CSS rules for Material and ReadTheDocs, there's no hint of where to copy-paste this code to.
I understand the basic principle of CSS and HTML, but have no idea about the architecture that would allow me to add additional CSS files.
Would it be possible to document how to override CSS elements (on the page linked above) or otherwise cross link the relevant documentation (say on Mkdocs?).  At minimum, it would be helpful if those code examples for themes could be completed with instructions on what to do with them.

I'm hitting this because I personally find the grey styled code headings very distracting.  All features other than the background are fine, but the grey box is tripping my eye into missing that it's a heading.

Thanks for the feedback, this is always appreciated :slightly_smiling_face:
I'll label this as "docs improvements" for when I have time!


fix: int overflow in get_jiffies

The number in the results could be larger than the maximum int value. This is fixed by using long.

Thank you for your contribution and sorry for the delayed reply! Just checked it and it seems that the jiffies variable is unsigned long in the linux kernel... Could you change the type to uint64_t? Thanks!

good~


Implement Neg for PgMoney

This is a preatty easy one, it just implements the Neg trait for PgMoney

@Aandreba the Postgres driver was moved into its own crate, can you rebase?

Closing due to inactivity. Feel free to reopen but please rebase first.


Add bootstrap terraform

Add terraform to create a node to serve as a bootstrap node for the cluster.

@mjturek can you squash those commits? having fewer will make rebasing, etc., easier later!

And also can you add a note in the providers file that we'll need to delete it later?

@clnperez done & done. Thanks!

LGTM, thanks!


Add Location Strategy Keyword

The first selenium library had the Add Location Strategy keyword. It essentially allowed for instances like this which is exactly what I happen to need at the moment.
Is there a particular reason why it isn't present? I found the java implementation of this keyword.

We were actually having a discussion within #333 about allowing user specified locator strategies. Just curious what type of element are you trying to locate that is already satisfied by the current strategies?

I want to locate using ExtJs' component query (see the this link above. What I have in mind is basically a javascript locator. Already working on a PR.

Looking at that example what I see is this is a javascript locator. Just wandering if there is other locator type that we can implement with a very similar keyword/argument syntax that is not javascript based. The reason I ask is to see if we need to add an argument that states which method the locator code implements. Something like
Open Page
  # 'Add Location Strategy'  | strategy name | strategy method |  stategy implementation "code"
  Add Location Strategy extJs  javascript return window.document.getElementById((Ext.ComponentQuery.query(arguments[0])[0]).getId());

Better to add it now then trying to patch in later.

Well, I suppose that's entirely possible. You could just provide it with a python function to run. Then it could theoretically do both rather simply.

See #365 for progress on this issue.


fix: releaseNotes.type is allowed in Release and RP CRs

/test release-pipelines-e2e-suite

/test release-pipelines-e2e-suite


[SPARK-11652] [CORE] Remote code execution with InvokerTransformer

Update to Commons Collections 3.2.2 to avoid any potential remote code execution vulnerability

Test build #45991 has started for PR 9731 at commit 0828753.

Test build #45991 has finished for PR 9731 at commit 0828753.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class JSONOptions(\n

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/45991/
Test FAILed.

Merged build finished. Test FAILed.

Test build #2064 has started for PR 9731 at commit 0828753.

Test build #2064 has finished for PR 9731 at commit 0828753.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

I'm going to go ahead and merge this, as it's a bug fix update anyway, passes, and should make sure there's no exploit of this form. We may not be alone in getting some alarmed customer questions about this, even though I suspect there is no actual exploit in Spark.

Merged to master/1.6/1.5/1.4

@srowen I can't find this jar file, can you give me a download url?

@srowen I only find below commons-collection file:
<groupId>commons-collections</groupId>
  <artifactId>commons-collections</artifactId>
  <version>3.2.2</version>

@XuTingjun commons-collections? just search Maven Central
http://search.maven.org/#search|gav|1|g%3A"commons-collections" AND a%3A"commons-collections"

I think the groupId should be "commons-collections", not "org.apache.commons", right?

Oh dang it, yes the group is only org.apache.commons in version 4. Right now this does nothing. PR coming ...

ok, please fix it as soon as possible, thanks.

Seehttps://github.com/apache/spark/pull/10198


sshd_config_subsystem does not allow for manging the same subsystem name in multiple targets

Hi,
It seems that sshd_config_subsystem does not allow one to manage the parameter as a resource in multiple files. For instance, this gives a duplicate resource error:
sshd_config_subsystem { "sftp":
target => '/etc/ssh/sshd_config',
command => '/bin/false',
}
sshd_config_subsystem { "sftp":
target => '/etc/ssh/sshd_config_chroot',
command => 'internal-sftp',
}
This isn't really show-stopper for us since I think by the time we need it, we'll have our SFTP service migrated to a newer version of OpenSSH where we can have get the access control that we need in a single SSH service, but I was testing an implementation with CentOS 6 and thought I would pass along the problem.
Thanks!
Skylar

That's correct. The types in this module don't have support for composite namevars at the moment. It shouldn't be too hard to add though, so I'd welcome a PR adding title_patterns to these types.

Cool, thanks for confirming that I was on the right track. I don't know that I have the Puppet experience to add that functionality now, but I'll definitely have it as a goal at some point.


How to set Sec-WebSocket-Protocol on websocket client

REF 6455 defines Sec-WebSocket-Protocol.
https://tools.ietf.org/html/rfc6455#section-11.3.4
I'm using MQTT broker that supports Websocket. It requires upgrade request from client need to have Sec-WebSocket-Protocol and that value is mqtt.
I think that the timing of sending upgrade request is async_handshake.
http://vinniefalco.github.io/beast/beast/ref/websocket__stream/async_handshake.html
However, it seems that the interface doesn't support adding additional field such as Sec-WebSocket-Protocol.
Is there any good way to do that?

Yes, see:
https://github.com/vinniefalco/Beast/issues/70

Thank you! I replaced headers with fields in your set_subprotocols, then it works fine :)


pull request for e2e test at 01/05/2017 07:26:08

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


regarding DET and VID datasets

can you please tell us what version of DET and VID are you using?
It would be also very useful if you can provide a link to their source to download them please
Also when you mentioned that data should be in form of:
|--ILSVRC2015
|----Annotations
|------DET
|--------train
|--------val
|------VID
|--------train
|--------val
|----Data
|------DET
|--------train
|--------val
|------VID
|--------train
|--------val
|----ImageSets
|------DET
|------VID

can you clarify how they should be inside train and val?
I mean should it be like folders for each video or all the frames? and the same for DET.
Thanks :)

Thank you so much!

Thanks a lot!

can i have a follow up question?
When you use DET, do you just consider the classes that are in VID, OR you use all 1000 classes in DET and use the one that are not in the VID as background?
Thanks :)

I just used the intersection of the classes in DET and VID.

Here's a link to the train/val/test splits in ILSVRC2015 that I used: here


Clobber all registers in context switching for compiler compatibility

Re-introduced the register clobbering in the x86-64 implementation of context switching as clang is less conservative than gcc.

Merged. Thanks!


Support multiple IHostLifetime in Host class

Currently in Host.cs the constructor injection only allow one single IHostLifetime to be used.
https://github.com/aspnet/Hosting/blob/edada4acc744c6257e48b4e474ee2371fa0260db/src/Microsoft.Extensions.Hosting/Internal/Host.cs#L24
The request is to change it to IEnumerable so that multiple IHostLifetime could be registered each one dedicated to one function for host lifetime.

We considered a similar item (https://github.com/aspnet/Hosting/issues/1221) but decided against it. There weren't enough examples of different lifetimes that you would want to use together.
Which types of lifetime would you want to use together?

Here is the scenario for Microsoft Intune that needs multiple lifetime:

GenevaInitializationHostLifetime: to initialize Geneva MDS logging and MDM metrics like account and namespace etc, this needs to happen first before any production logging could be sent to Geneva.
ServiceCrashHostLifetime: to register TaskScheduler.UnobservedTaskException and AppDomain.CurrentDomain.UnhandledException and report service crash to metrics and logs for trouble-shooting
ServiceFabricContainerAssemblyResolverHostLifetime: this is needed to support container in Service Fabric, to register AppDomain.CurrentDomain.AssemblyResolve, as some assembly are not in current directory. See How to containerize your Service Fabric Reliable Services and Reliable Actors and SFBinaryLoader(https://github.com/Azure/service-fabric-scripts-and-templates/blob/master/code/SFBinaryLoaderForContainers/SFBinaryLoader.cs)

Would you always be using all of these, or would you be dynamically deciding which ones to use at startup?

above 1, 2 we always needs to use, 3 we only use in case it's running inside Service Fabric container, and in that case we need 1, 2, and 3.
Currently we haven't used generic host since it haven't officially released, we're running with asp.net + Service Fabric. but I'm working on switching to generic host. above are scenario are from our current production service. If I switch to generic host, I will need to have this multiple IHostLifetime support.

That's a code organization question, you're not functionally blocked here. If you always need scenario 1 an 2 then functionally they can be included in the same lifetime. For 3, it could be added via a derived implementation.

that's a hack workaround and violate Single Responsibility Principle, I want to avoid it if possible.
And you could argue the same for IHostedService, why you will need multiple IHostedService, why not combine them together and in IHost only support one single IHostedService.
for IApplicationLifetime, just one instance is enough as the CancellationToken.Register could be called in multiple places, thus IApplicationLifetime does not needs multiple instance. But that's not the case for IHostLifetime without compromise the design principle.

Those examples aren't very convincing TBH, can you explain why those are examples of host lifetime implementations?

do you have any suggested generic host interface to wrap those implementation but not use IHostedLifetime?

IHostedService has a StartAsync and a StopAsync and it sounds like what you want.

That's not really a hosted service, because it's not constantly running after host start. If you compare it with the ConsoleHostLifetime, it is just similar to that, to register some global event or do initialization.

IHostedService isn't about constantly running things. It's about hooking into the existing lifetime of the host.

If the suggestion is to use IHostedService, why ConsoleHostLifetime is not implemented as a ConsoleHostedService?

HostLifetime controls the lifetime of the host as in it decides why when the host starts up and when the host shuts down.
There are 2 extremely different concerns:

I want to know when the host starts and when it stops - IHostedService
I want to decide when the host starts and when it stops - IHostLifetime

I tried further and still see I could not use the IHostedService for this purpose.
For production, I'm running with Service Fabric with ServiceFabricHost that replaced regular Host (https://github.com/Azure/service-fabric-aspnetcore/issues/48), but for dev testing I use regular Host, I want to share the same logic for logging initialization and service crash handling.
for Service Fabric, if I model such logic as IHostedService, that mean there could be multiple service instances running in the same process, but I don't need multiple service crash handler, I only need one, and in such case modeling it with IHostLifetime make sense to me from logic perspective.

That's a code organization question, you're not functionally blocked here. If you always need scenario 1 an 2 then functionally they can be included in the same lifetime
@Tratcher, if I include 1 and 2, then I lost the flexibility in DI, as 1 and 2 has additional parameter coming from DI constructor injection, with including them both I now have to merge these unrelated parameters.

besides the 1, 2, and 3 mentioned, I'd also want to have the built-in ConsoleLifetime experience, if I merge them I now also have a hard dependency on ConsoleLifetime which is against the dependency injection principle.

here is my attempt to including them all, which really seems ugly IMHO;
/// <summary>
    /// Ugly workaround needed to support multiple IHostLifetime in Host class
    /// see issue https://github.com/aspnet/Hosting/issues/1401
    /// </summary>
    public class UglyDumpGroundLifetime : IHostLifetime
    {
        private ConsoleLifetime consoleLifetime;
        private ServiceCrashLifetime serviceCrash;
        private UnifiedApiLifetime genevaLogging;

        public UglyDumpGroundLifetime
            (
            ILoggerContext logger, // used for ServiceCrashLifetime
            ICloudEnvironment cloud, IServiceIdentity service, // used for UnifiedApiLifetime
            IOptions<ConsoleLifetimeOptions> options, IHostingEnvironment environment, IApplicationLifetime applicationLifetime // used for ConsoleLifetime
            )
        {
            this.consoleLifetime = new ConsoleLifetime(options, environment, applicationLifetime); // anti-pattern, direct type coupling
            this.serviceCrash = new ServiceCrashLifetime(logger);
            this.genevaLogging = new UnifiedApiLifetime(cloud, service);
        }

        public async Task StopAsync(CancellationToken cancellationToken)
        {
            await this.consoleLifetime.StopAsync(cancellationToken);
            await this.serviceCrash.StopAsync(cancellationToken);
            await this.genevaLogging.StopAsync(cancellationToken);
        }

        public async Task WaitForStartAsync(CancellationToken cancellationToken)
        {
            await this.consoleLifetime.WaitForStartAsync(cancellationToken);
            await this.serviceCrash.WaitForStartAsync(cancellationToken);
            await this.genevaLogging.WaitForStartAsync(cancellationToken);
        }
    }

for Service Fabric, if I model such logic as IHostedService, that mean there could be multiple service instances running in the same process, but I don't need multiple service crash handler, I only need one, and in such case modeling it with IHostLifetime make sense to me from logic perspective.

I don't understand. Everything you described was about getting notifications, it wasn't about driving the lifetime so IHostedService is a fit for this. Even your code sample kinda shows this.

That's not really a hosted service, because it's not constantly running after host start. If you compare it with the ConsoleHostLifetime, it is just similar to that, to register some global event or do initialization.

You're looking at the wrong aspect of the code, the console lifetime is controlling when the host stops. Your IHostedService will get notified on start and stop. You can wire up an event in Start and unhook it in Stop.
I'm just not seeing the problem.

thanks @davidfowl , that help me understand better of the IHostLifetime vs. IHostedService, and after further thinking I come up with below solution that does not require multiple IHostLifetime but still meet my scenario requirement. Can you help to take a look if that's right thing to do, and whether that's general concept that would benefit generic host.

Where?

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


AWS lambda Nodejs 22 runtime is missing

Describe what happened
AWS added it to go SDK in https://github.com/aws/aws-sdk-go-v2/blob/release-2024-11-20/service/lambda/CHANGELOG.md#v1670-2024-11-20
That was bumped in the TF provider on https://github.com/hashicorp/terraform-provider-aws/commit/5698f695d8f3163dd4173ef5ee9455f06f07ff22 and released with 5.77
It doesn't look like the bridge is regenerating this (and other related) file(s)
https://github.com/pulumi/pulumi-aws/blob/master/sdk/nodejs/types/enums/lambda/index.ts
https://github.com/pulumi/pulumi-aws/blob/master/sdk/java/src/main/java/com/pulumi/aws/lambda/enums/Runtime.java
This was present in 6.62 prior to retraction as well.
Sample program
aws.lambda.Runtime.NodeJS22dX
Log output
No response
Affected Resource(s)
No response
Output of pulumi about
CLI
Version      3.142.0
Go Version   go1.23.3
Go Compiler  gc
Plugins
KIND      NAME    VERSION
resource  aws     6.62.0
resource  aws     6.61.0
resource  awsx    2.19.0
resource  docker  4.5.7
resource  docker  3.6.1
language  nodejs  unknown
Host
OS       ubuntu
Version  22.04
Arch     x86_64
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hey @curtdept, the provider does not automatically pull in new enum values for the lambda runtimes. This is maintained manually at the moment: https://github.com/pulumi/pulumi-aws/blob/384111d99980bd10f9df4c96197ab98c1e2fa1d3/provider/types.go#L227-L243
I checked 6.62.0 to be sure and it wasn't part of that release: https://github.com/pulumi/pulumi-aws/blob/9d0a03f14938f4aeb4e83c97597799c8cdcd644e/sdk/nodejs/types/enums/lambda/index.ts#L6-L22
I'm gonna track this as an enhancement and create a PR right away to add this new runtime

@flostadler thank you :)
The 6.62 part was ambiguously worded on my part. I mean the runtime values were also missing in that release.


do you have any examples around combining cursor and touch position?

would be nice to support mobile and desktop in the same component.

@chrisdrackett - Thanks very much for your inquiry! I think you raise a great use case! I will create an example of react-cursor-position with react-touch-position and post back here. In the meantime I think a feature detection strategy is one approach that could be taken. It might look something like:
function getIsTouchEnv() {
    return ('ontouchstart' in window);
}

export default () => {
    return getIsTouchEnv()
    ? (
        <ReactTouchPosition {...{
            mapPropNames: ({  touchPosition }) => {  cursorPosition: touchPosition }
        }}>
            <YourComponent />
        </ReactTouchPosition>
    )
    : (
        <ReactCursorPosition>
            <YourComponent />
        </ReactCursorPosition>
    );

@chrisdrackett - Your issue has raised my awareness to the value of a solution that unifies touch and pointer environments without a conditional render. I will prioritize creating this and post back here when it is released. In the meantime, if your use case is a better fit for a unified touch and pointer solution, consider something like the example in this gist. I really appreciate you opening this issue. Thanks!

no problem, and thanks for this component, it let me simplify our code while adding functionality. Never a bad thing!

@chrisdrackett - I'm happy to inform you the latest version of react-cursor-position supports touch screen environments. The release notes contain an overview and cover breaking changes. As always, if you have questions or concerns, please post here or create a new issue.
Thanks again for your input. I think it helped make the project better!

Hey @chrisdrackett - I'm closing this due to inactivity. If you have any questions or comments please let me know. Thanks again for opening this issue!


Fix reloading of application when using geocoder in url

Title says it all

Interesting. Why does it work here: https://github.com/terrestris/anol/blob/master/src/modules/permalink/permalink-service.js#L477 ?

Interesting. Why does it work here: https://github.com/terrestris/anol/blob/master/src/modules/permalink/permalink-service.js#L477 ?

I just checked the angular code and at the end of the day, the window.location.replace method is being called. This not simply replaces the URL, but navigates to it (see mdn docs).
The reason why it works at other parts of the code, is possibly due to the $digest of $location not being finished. But I'm not sure on that.


Create jsolution

Because

This PR

Issue

Closes #XXXXX
Additional Information

Pull Request Requirements

[ ] I have thoroughly read and understand The Odin Project Contributing Guide
[ ] The title of this PR follows the location of change: brief description of change format, e.g. 01-flex-center: Update self check
[ ] The Because section summarizes the reason for this PR
[ ] The This PR section has a bullet point list describing the changes in this PR
[ ] If this PR addresses an open issue, it is linked in the Issue section
[ ] If applicable, I have ensured that the TOP solution files match the Desired Outcome image

we don't accept personal solution PR's to this repo. Keep up the good work with your exercises


fix: to trigger analysis only for prod in scarf and smartlook

Added env variable for production

Could we add the condition in the docusaurus.config.js instead?
Like this -
...
    scripts: [
		process.env.VERCEL_ENV === "production" && {
			src: "/scripts/intercomSettings.js",
			async: false,
		},
		process.env.VERCEL_ENV === "production" && {
			src: "/scripts/smartlook.js",
			async: false,
		},
		process.env.VERCEL_ENV === "production" && {
			src: "/scripts/analyticsEvents.js",
			defer: true,
		},
	],
...

Because now we're getting an error in the console in the deploy preview since process.env.VERCEL_ENV is only available during build step.

Done.


Fix TCP Connection data receive race condition

Prior to this commit, when we _accepted a new connection
or completed a connection via _event_notify there was a race
condition around reading data that the other party sent to us.
This existed because we didn't check to see if any data was
waiting on the socket for us already and instead only subscribed
to ASIO events for new read notifications. If the other party sent
us data after we signed up for ASIO events, we would successfully
receive notification and read the data. However, if the other
party sent us data before we signed up for ASIO events, we would
not read the data already on the socket and would wait for an ASIO
event telling us that we have data to read.
This commit fixes this race condition by ensuring that we try to
read from the socket whenever we _accept or complete a connection
via _event_notify prior to signing up for ASIO events. This
ensures we will not accidentally ignore any data that may already
be on the socket by the time we're ready to handle the connection.

I forgot to mention that this change makes the TCPMute, TCPUnmute and TCPThrottle test more reliable. They would sometimes previously fail because of the race condition around receiving data resolved by this commit.

The one failure in the CI matrix was a Travis blip, and unrelated to these changes.  I'm merging.