YARN-11472. Normalizes log records on the Yarn client

YARN-11472. Normalizes log records on the Yarn client,On the basis of YARN-11465

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 47s

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.

+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 1 new or modified test files.

_ trunk Compile Tests _

-1 :x:
mvninstall
0m 10s
/branch-mvninstall-root.txt
root in trunk failed.

-1 :x:
compile
0m  9s
/branch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-yarn-client in trunk failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

-1 :x:
compile
0m  9s
/branch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-yarn-client in trunk failed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.

-0 :warning:
checkstyle
0m  8s
/buildtool-branch-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
The patch fails to run checkstyle in hadoop-yarn-client

-1 :x:
mvnsite
0m 10s
/branch-mvnsite-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
hadoop-yarn-client in trunk failed.

-1 :x:
javadoc
0m 10s
/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-yarn-client in trunk failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

-1 :x:
javadoc
0m  9s
/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-yarn-client in trunk failed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.

-1 :x:
spotbugs
0m 10s
/branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
hadoop-yarn-client in trunk failed.

-1 :x:
shadedclient
1m  9s

branch has errors when building and testing our client artifacts.

_ Patch Compile Tests _

-1 :x:
mvninstall
0m 10s
/patch-mvninstall-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
hadoop-yarn-client in the patch failed.

-1 :x:
compile
0m 10s
/patch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-yarn-client in the patch failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

-1 :x:
javac
0m 10s
/patch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-yarn-client in the patch failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

-1 :x:
compile
0m  9s
/patch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-yarn-client in the patch failed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.

-1 :x:
javac
0m  9s
/patch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-yarn-client in the patch failed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m  8s
/buildtool-patch-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
The patch fails to run checkstyle in hadoop-yarn-client

-1 :x:
mvnsite
0m 10s
/patch-mvnsite-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
hadoop-yarn-client in the patch failed.

-1 :x:
javadoc
0m 10s
/patch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-yarn-client in the patch failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

-1 :x:
javadoc
0m  9s
/patch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-yarn-client in the patch failed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.

-1 :x:
spotbugs
0m 10s
/patch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
hadoop-yarn-client in the patch failed.

-1 :x:
shadedclient
1m 47s

patch has errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
0m 10s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
hadoop-yarn-client in the patch failed.

+0 :ok:
asflicense
0m 11s

ASF License check generated no output?

7m  9s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 5016ee89f4ae 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / bdb956c016528326d3bac43c8f3b2bfb57d7a56e

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

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

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

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

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

versions
git=2.25.1 maven=3.6.3

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 48s

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.

+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 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
43m 46s

trunk passed

+1 :green_heart:
compile
0m 32s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
0m 28s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
0m 30s

trunk passed

+1 :green_heart:
mvnsite
0m 37s

trunk passed

+1 :green_heart:
javadoc
0m 36s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 24s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m 13s

trunk passed

+1 :green_heart:
shadedclient
24m 24s

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

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 27s

the patch passed

+1 :green_heart:
compile
0m 31s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
0m 31s

the patch passed

+1 :green_heart:
compile
0m 26s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
0m 26s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 18s
/results-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client: The patch generated 1 new + 59 unchanged - 0 fixed = 60 total (was 59)

+1 :green_heart:
mvnsite
0m 31s

the patch passed

+1 :green_heart:
javadoc
0m 22s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 20s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m  7s

the patch passed

+1 :green_heart:
shadedclient
23m 26s

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

_ Other Tests _

+1 :green_heart:
unit
28m  9s

hadoop-yarn-client in the patch passed.

+1 :green_heart:
asflicense
0m 37s

The patch does not generate ASF License warnings.

130m 56s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux dbf57bbcc5ee 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / bdb956c016528326d3bac43c8f3b2bfb57d7a56e

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

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

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

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

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5575/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 49s

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.

+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 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
43m 54s

trunk passed

+1 :green_heart:
compile
0m 36s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
0m 30s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
0m 29s

trunk passed

+1 :green_heart:
mvnsite
0m 38s

trunk passed

+1 :green_heart:
javadoc
0m 38s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 25s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m 10s

trunk passed

+1 :green_heart:
shadedclient
23m 21s

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

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 29s

the patch passed

+1 :green_heart:
compile
0m 31s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
0m 31s

the patch passed

+1 :green_heart:
compile
0m 27s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
0m 27s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 17s
/results-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client: The patch generated 1 new + 59 unchanged - 0 fixed = 60 total (was 59)

+1 :green_heart:
mvnsite
0m 31s

the patch passed

+1 :green_heart:
javadoc
0m 22s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 21s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m  4s

the patch passed

+1 :green_heart:
shadedclient
23m 29s

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

_ Other Tests _

+1 :green_heart:
unit
28m  1s

hadoop-yarn-client in the patch passed.

+1 :green_heart:
asflicense
0m 37s

The patch does not generate ASF License warnings.

130m 15s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 0cdd62a22116 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / bdb956c016528326d3bac43c8f3b2bfb57d7a56e

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

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

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

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

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5575/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.

@ayushtkn that build seemed to go wrong due to Unknown host repo.maven.apache.org
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5575/1/artifact/out/branch-mvninstall-root.txt
Maybe there was some temporary connectivity issue.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 53s

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.

+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 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
44m 15s

trunk passed

+1 :green_heart:
compile
0m 36s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
0m 32s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
0m 29s

trunk passed

+1 :green_heart:
mvnsite
0m 35s

trunk passed

+1 :green_heart:
javadoc
0m 35s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 25s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m 13s

trunk passed

+1 :green_heart:
shadedclient
23m 47s

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

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 30s

the patch passed

+1 :green_heart:
compile
0m 32s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
0m 32s

the patch passed

+1 :green_heart:
compile
0m 28s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
0m 28s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 17s

the patch passed

+1 :green_heart:
mvnsite
0m 29s

the patch passed

+1 :green_heart:
javadoc
0m 22s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 20s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m  9s

the patch passed

+1 :green_heart:
shadedclient
23m 39s

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

_ Other Tests _

+1 :green_heart:
unit
27m 59s

hadoop-yarn-client in the patch passed.

+1 :green_heart:
asflicense
0m 36s

The patch does not generate ASF License warnings.

131m 22s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 042bd280feed 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / f261178e63a8697f8a0b9da2c39e7e6bbfb03e3a

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

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

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

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

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5575/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.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 47s

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.

+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 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
44m 13s

trunk passed

+1 :green_heart:
compile
0m 36s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
0m 31s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
0m 29s

trunk passed

+1 :green_heart:
mvnsite
0m 35s

trunk passed

+1 :green_heart:
javadoc
0m 37s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 26s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m 10s

trunk passed

+1 :green_heart:
shadedclient
23m 37s

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

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 29s

the patch passed

+1 :green_heart:
compile
0m 32s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
0m 32s

the patch passed

+1 :green_heart:
compile
0m 26s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
0m 26s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 17s

the patch passed

+1 :green_heart:
mvnsite
0m 29s

the patch passed

+1 :green_heart:
javadoc
0m 22s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 20s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
1m  8s

the patch passed

+1 :green_heart:
shadedclient
23m 52s

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

_ Other Tests _

+1 :green_heart:
unit
27m 56s

hadoop-yarn-client in the patch passed.

+1 :green_heart:
asflicense
0m 37s

The patch does not generate ASF License warnings.

131m 22s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 7d7b9b076c7b 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / f261178e63a8697f8a0b9da2c39e7e6bbfb03e3a

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

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

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

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

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5575/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.


Refactor mil.dds.anet.test.resources.utils.GraphQlHelper

Technical debt description
The GraphQlHelper in the Java tests unmarshalls the GraphQL responses into ANET beans. However, this uses a JSON mapper, and not the GraphQL accessors on the beans. A consequence of this is that the beans have additional methods that are only necessary for the tests (and thus perhaps our GraphQL schema shows more fields than it should). Also several methods in the beans need to be annotated with @JsonIgnore to stop the tests from (un)marshalling too many fields.
Proposed solution
It would be better if the GraphQlHelper was ANET bean-agnostic, and just returned a Map<String, Object> holding the GraphQL response. This means refactoring all Java tests that do GraphQL requests to use map lookups; e.g. a current returned result like:
final AnetBeanList<Position> result = graphQLHelper.searchObjects(…, new TypeReference<GraphQlResponse<AnetBeanList<Position>>>() {});
final List<Position> positions = result.getList();
final String uuid = positions.get(0).getUuid();

would become something like:
final Map<String, Object> result = graphQLHelper.searchObjects(…);
final List<Map<String, Object>> positions = (List<Map<String, Object>>) result.get("list");
final String uuid = (String) positions.get(0).get("uuid");

Once all the tests have been refactored, the excess methods and annotations should be removed.

I've chosen a different approach based on https://github.com/graphql-java-generator/graphql-gradle-plugin-project


Glam 0.28 update - adopted

Basically it's https://github.com/bevyengine/bevy/pull/13792 with the bumped versions of encase and hexasphere.

@alice-i-cecile @superdump I just did the last required action to finish #13792. I hope this was ok? It was really just the version bump and everything else seems to work now out of the box.


EndsWith should use rfind()

https://github.com/philsquared/Catch/blob/7424b23bfb4423fd1213141633844f09faaff8bc/single_include/catch.hpp#L1089
.find() is used.

Wow that's terrible code! Did I write that? (checks... yes I did. I am ashamed).
You are quite right that, given the approach being used it should be using rfind().
But the approach itself seems like an awkward way to do it. I can only imagine that I started with Contains() and made the minimum changes necessary to morph it into StartsWith(), then EndsWith().
But given that I already had the functions startsWith() and endsWith() (https://github.com/philsquared/Catch/blob/master/include/internal/catch_common.hpp#L19) I don't understand why I didn't use them.
So, thanks for bringing this up! I'll take a closer look later and clean things up a bit (fixing the latent bug in the process).

This has been since fixed, albeit it would probably be better if every endsWith and startsWith calls didn't allocate a brand new string.


#127 - Fixes local dates for Safari, Safari has known issues with Dat…

This PR fixes the one day off error when working with Safari browsers, which today are the only desktop browsers that do not have input date support anyways

Coverage decreased (-1.06%) to 45.865% when pulling e0e9c07e47ae2ad480cd0a082527577baeb72246 on fstani:fix-safari-dates into d1d935047430e44bf23e125e2ce5723ccffb05d7 on chemerisuk:master.

@chemerisuk seems like coverage is not passing, any hints on what can I do to help get this merged into the main branch?

@fstani thanks, I'll take a look asap.

@fstani slightly reworked your changes at https://github.com/chemerisuk/better-dateinput-polyfill/commit/b72e139f6eb57a3f56e9073141b0cfd39e76dcea. Let me know if it works for you.


Fix nil dereference in pull_request.rb

Should fix the undefined method find for nil:NilClass (NoMethodError) error we're seeing on recent workflow runs: https://github.com/alphagov/govuk-dependabot-merger/actions/runs/7797524406/job/21264292221

Superseded by #44.


adds new automated tests for admin search references

Jira link

HOTT-4867
HOTT-4868

What?
I have added/removed/altered:

Added new automated tests for admin search references import and export functionality.
Added new commands for admin search references for import and export to verify the different text and buttons etc.

Why?
I am doing this because:

To ensure that we have enough test coverage for import and export search references functionality.

Change looks good


Added node-email-verification to email section

Please read and adhere to the contributing guidelines.

@sindresorhus done


How can I compile multiple units to one executable?

In all the compilation units but one, define TEST_NO_MAIN prior including acutest.h.
(Or alternatively, propagate it from your build system via compiler's command line option, typically -DTEST_NO_MAIN, and #undef it in exactly one compilation unit.)


feat(framework-lib): made selection controllable from outside

createableSelectDropdown is now controllable from the outside using its value prop.
closes DEV-12261

@SebCodesTheWeb take a lead :)


Flyway migration introduces duplicate creation of sequences

Description

Flyway migration introduces duplicate creation of sequences for notification tables.
We became aware of this because we have to insert or remove test data in the table for the local execution via script and have to adapt the sequence accordingly.
Current situation
Sequences for alerts and investigations are created twice:

first explicit creation at
https://github.com/eclipse-tractusx/traceability-foss/blob/c656f301c74515cbe692dd81c7c4cf2627d26121/tx-backend/src/main/resources/db/migration/V1__base_sql_configuration.sql#L3
https://github.com/eclipse-tractusx/traceability-foss/blob/c656f301c74515cbe692dd81c7c4cf2627d26121/tx-backend/src/main/resources/db/migration/V1__base_sql_configuration.sql#L12

second implicit creation with the related tables
https://github.com/eclipse-tractusx/traceability-foss/blob/c656f301c74515cbe692dd81c7c4cf2627d26121/tx-backend/src/main/resources/db/migration/V1__base_sql_configuration.sql#L36
https://github.com/eclipse-tractusx/traceability-foss/blob/c656f301c74515cbe692dd81c7c4cf2627d26121/tx-backend/src/main/resources/db/migration/V1__base_sql_configuration.sql#L219
One of the above creation is superfluous.

Expected behavior

Steps to reproduce the Bug

Thanks for raising this bug issue. We will estimate and prioritize this according to our iteration plan.

All unused Sequences have been removed:

All unused Sequences have been removed:


Taruti/tlf self first

Could use some eyes.

@taruti should I review this again or are you still making changes?  (I see the tests don't pass.)

@strib, yes, except the login/logout stuff.

Jenkins complains about https://github.com/keybase/kbfs/pull/427 and kbfs-server fails as is common these days. Not rebasing to make CR easier.

Now with the renames too, ping @strib

Rebasing due to conflict.

For the kbfs-server branches see the PR https://github.com/keybase/kbfs-server/pull/150 (which has this branch vendored in and will be committed with proper vendored changes later).


Metadata Summary Update by Transfers and Deletions

What does this PR do and why?
Note: Dependent and branched off of PR 339
This PR is the secondary part of the metadata_summary update and includes updating based on group/project/sample transfers and deletion.
When transferred or deleted, all namespaces  will be updated accordingly.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other pull requests.
How to set up and validate locally
In the Rails console, set up your metadata as described in PR 339
Using the website UI, transfer groups, projects, and samples and subsequently check in the console that both old and new parents/ancestors are updated accordingly. Ie: if a sample with metadata = {'metadatafield1' => 'value1'} is transferred, ensure its old parents have metadata_summary['metadatafield1'] -= 1 and the new parents have metadata_summary['metadatafield1'] += 1
Using the website UI, delete groups, projects, and samples and subsequently check in the console that itsparents/ancestors are updated accordingly. Ie: if a sample with metadata = {'metadatafield1' => 'value1'} is deleted, ensure its parents have metadata_summary['metadatafield1'] -= 1
PR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

[x] I have evaluated the PR acceptance checklist for this PR.

Hi Chris, just wanted to let you know I'm still looking it over. I haven't tested every single scenario yet, but this looks awesome!


Develop

Pattilla

Bro 🤫🤫🤫🤫🤫🤫 don't message me
On Mon, 6 Dec 2021, 2:22 pm Muhammed, @.***> wrote:

Closed #5 https://github.com/PR0FESS0R-99/LuciferMoringstar_Robot/pull/5
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PR0FESS0R-99/LuciferMoringstar_Robot/pull/5#event-5719004665,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWFJJL2PD2C5NXFDGI6LEFLUPR2T5ANCNFSM5IOZH5MQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Bumps gh cli in builder base to 2.21.1

Issue #, if available:
Description of changes:
We use the gh cli in a few places across the stack mainly for creating PRs via the bot in postsubmits.  We are on an older version which does not support the -H flag which I introduced usage of during the distro-base yaml tag update flow.
This has limited scope across since its not used in too many places. I would say there is some risk of issues, but should be super low.  The major version change according to the gh changelog does not look like breaking changes that will affect us.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve


Limited customization of header

Header should be a fully customisable view

I've done a simple private enhancement to OCBorghettiView to add attributed string support for section titles, so that things like FontAwesome can make for more interesting section headers with graphics/etc.
Is it worth sending this as a pull request?
Or are you planning a more significant re-work of the header view, in which case I'll hold off?

(Y) just send him a pull request

Yes, please ;-) Thank you @alexshepard and @yasirmturk!

you are welcome @otaviocc :) and thanks @alexshepard


A few huge improvement suggestions (easy to implement)

FIRST OFF... Great program, but here's some easy improvements i see...

1-move 'delete' icon to left of LINK icon (so less easy to hit by accident when trying to hit the "download" icon on a small screen (eg phone)
2-move ALL iconset to 'left of filename' (so alway close to filename, even when zoomed)
3-prompt for deletion (too easy to hit delete and BOOM... GONE...)
4-setting to disable the "view" when you touch a file (often just mean to "select" it and
when i touch it on a slow/remote link, it wastes a few minutes of showing a PDF etc
These would be easy to implement and add HUGE usability.
Thanks! Fred

Oh, i added a 4th item... below (not included on) the screenshot... also found would be huge addition... to be able to disable the "view" when touching a file

I'm still having files deleted by accident... too darn easy!
Any possibility you can implement recommended changes??

Fantastic usability ideas. +1

+1 For all of these ideas. Would improve usability greatly.


Remove version: Jackett.Jackett version 0.20.2405

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.20.2405/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


”Jurilovca” is listed in the drop-down menu

Description:
"Jurilovca" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Jurilovca" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Jurilovca, TULCEA".


Can I change the firebase sdk paths safety?

Hello, I have tried the firebask sdk for unity examples and they all worked well by default.
But, in my actual project, I want to manage my project directory structure, so I moved Firebase folder to a git submodule folder.
For example, the origin path of sdk is "Assets/Firebase/...",now is "Assets/SDKModule/Firebase/...".
Then errors occured that the console told me "Unable to find command line tool D:/UnityProject/XXX/Assets..\Assets\Firebase\Editor\generate_xml_from_google_services_json.exe required for Firebase Android resource generation."
So...if it is possible to fix this? Or can the sdk  only be used in the default folder path?

btw, I found every plugin have a config file such as FirebaseRemoteConfig_XXXXX.txt, there're many path config inside. I modified them to what I need but nothing happened even I have reimported everything.

So we could not change SDK path?

@ldk18501 Unity - especially across different versions - is very particular about where things live in the directory structure.  In order to have Unity build components for each platform we needed to lay things out in the current structure.  Worse, since older versions of Unity don't have a package manager we need to make sure shared components (e.g Parse, PlayServicesResolver etc.) reside in known locations.  Ideally this could be resolved with a package manager that works across all Unity versions, alas we haven't found something reasonable to use yet and our implementation is far from complete.
FYI: The manifest files distributed with the plugin are simply there to assist with upgrades, we use the manifest to calculate the difference in files when importing new versions of the plugin which allows us to clean up stale components.

@ldk18501 @stewartmiles
Same issue. I just do not want  so many folders with no-uniform name just used by SDK  directly to be put  in unity project asset root directory.  It's hard to maintain without extra wiki or other document to explain them.
I used most Unity third-pard SDK which is just in their own folder.  So maybe firebase SDK can also do the same thing.
Thanks.

@fogmoon Firebase is a little more complicated than many SDKs, in addition we support many versions of Unity that most plugin developers don't care about.  So I'm afraid not, we need our current layout.  Have you considered using source control to handle rollbacks / upgrades?

So this can't be done? Firebase and all it's Folders need to be in the root Asset folder of Unity?

@ldk18501 @stewartmiles
Same issue. I just do not want so many folders with no-uniform name just used by SDK directly to be put in unity project asset root directory. It's hard to maintain without extra wiki or other document to explain them.
yes, absolutely agree.Firebase messup our project  structure and make our version sharing cubersome.

Anything new on this issue? I got my whole project structure messed up because of using the SDK, it makes it hard to track changes in the project.
We usually have a thirdparty directory that includes all the SDKs and other thirdparty tools and using Firebase messes up the whole structure, there has to be something we can do about it?

Please, at least make it one neat folder "Firebase/" in the root Assets directory. Right now it's hard to find stuff and my project's folder structure doesn't look clean.

@stewartmiles Could we reopen this? I'd like to be able to change the structure of the original Firebase path like many others here. Is there a way to make it possible right now or in the future?
Thank you

So I kind of got this partly working. I have most of firebase in a submodule shared across a few different unity apps. What I have to do is inside each project when I add the googleservices files for that project, I also have to copy into the project another folder from the submodule. In this folder there's no CS files or DLLs but the py and exe files from the firebase/editor folder, and also the m2repository folder. Using this method I can share and update firebase across a bunch of apps, and also have my shared tools dependent on the firebase API.
I think that the SDK should support this more seamlessly. simply by using a unity editor call the SDK can find out where it is located in a project and run your import scripts against that. There's no technical reason why the SDK should force itself to be located in a specific folder.
This issue was closed even though a number of users agreed with it and it was not fixed. That itself is the worst thing about this problem. Ignoring your users is very annoying for them. Currently firebase needs to be used to leverage google ads properly, so you guys are extremely privileged. Forcing users into clunky pointless workflows through monopolies is not cool.

@ChrisZul glad you have something working.  We do support moving components around in our plugins to a certain extent as we fallback to searching for components using asset GUIDs, if you blow away the asset metadata though everything will break.  If you have a particular problem you would like to see addressed, please open an issue describing exactly what isn't working for you and we can take a look.

Thanks for the response @stewartmiles. I would vote for re-opening this current issue, as it already perfectly describes the issue and has a range of interested users already participating in it who would probably be interested in any fixes.
But just to be extra clear, from my experience with what I have to do to get good iOS and Android builds using my submodule, the files which are part of the firebase unity SDK and are not currently happy about being moved around into new paths are as follows:
Firebase/Editor/AnalyticsDependencies.xml
Firebase/Editor/AppDependencies.xml
Firebase/Editor/generate_xml_from_google_services_json.py
Firebase/Editor/generate_xml_from_google_services_json.exe
Firebase/Editor/network_request.py
Firebase/Editor/network_request.exe
Firebase/m2repository/**
Firebase/Plugins/Android/Firebase/AndroidManifest.xml
Firebase/Plugins/Android/Firebase/project.properties
It's possible that not all of these need copying however I copy them into each project from my submodule and then firebase seems to work for me. I am not sure what you would have to do to fix this problem, but basically the way most unity packages work these days is that everything is contained in a folder. Users can move that folder around within their project, and the package continues to work. In a new project Asset GUIDs are all completely recreated so it is not a good way to keep track of things.

The following should already work:

*Dependencies.xml files really don't care where they live, they must exist under an Editor/ folder.  See https://github.com/googlesamples/unity-jar-resolver#android-resolver-usage
generate_xml_from_google_services_json.{py,exe} is found via asset GUID and only searched via path as a fallback.  If the asset metadata isn't changed, this should work fine to move anywhere.
Firebase/m2repository/** should be found if path relative to Assets is preserved.  e.g it can be moved into any subfolder and found via https://github.com/googlesamples/unity-jar-resolver/blob/56d476e23464f797d8c90b5fac0958af2e0887db/source/PlayServicesResolver/src/GradleResolver.cs#L134

I don't think moving the following works:

The project template in Plugins/Android/Firebase is primarily due to this being treated as a build artifact.  We populate this with the config generated google-services.json.  I guess it's possible for us to again find the output directory by asset GUID based upon say the AndroidManifest.xml.

w.r.t replacing asset GUIDs / asset metadata, you're going to be totally out of luck, AFAIK, there are a few ways to uniquely identify a file in a Unity project:

Asset GUIDs
Path relative to Assets/
File checksum (very slow, very small chance of hash collisions)

We mostly do (1), with a bit of (2) when looking for maven repos using the subpath as a way to identify a directory, we do not do (3) as things are slow enough.

I have a problem.
Crashlytics library is hard coded as "Asset/Firebase/Editor" path in the VersionInfo.GetEditorGvhVersion() function.

Hi folks, it has been awhile.
I think you can change most of the Firebase library location as @stewartmiles mentioned.
Be careful with Crashlytics.
Another option for you now is to install Firebase through UPM so that your Assets folder can be clean and shiny.
Instructions
https://firebase.google.com/docs/unity/setup#add-sdks-upm
Known Issues
https://firebase.google.com/docs/unity/setup#known-issues
I will close this one for now.


Teaching Page Updates

issue: https://github.com/participedia/api/issues/677

todo:

[ ] wrap strings for translations
[ ] run new translations through PhraseApp auto-pilot

@ascott this looks great. The thing i'm noticing is that the hierarchy for the headings and sub headings is off a bit. Can we fix so that it is the following hierarchy (I realize this differs from my mockup, but looking at the page here I think it's easiest to digest this way):
H1 Teaching & Learning Resource Centre
H2 What are Teaching & Learning Resources?
H2 Discussions & Questions About Our Resources
H2 Share Your Resources with the Community
H2 Participedia in the Classroom

@jesicarson thanks jesi, i'll make these changes.

@jesicarson updated with the new heading sizes


Add template preview

https://github.com/aws-cloudformation/cfn-python-lint/pull/1411
Description of changes:

Try it out by pulling this PR, running npm install && npm run compile, open this PR in VS Code and then launching the extension. (Ensure you have run pip3 install cfn-lint pydot --upgrade too)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Launch extension seems to work for me:
git clone https://github.com/aws-cloudformation/aws-cfn-lint-visual-studio-code
gh pr checkout 106
code aws-cfn-lint-visual-studio-code/

but doesn't seem to work when building and installing .vsix: 🤔
npm install -g node@latest
npm install -g npm@latest
npm install -g vscode@latest
npm install -g vsce@latest
npm install
vsce package

@miparnisari not sure how this is working currently, but it'd be nice to eventually allow the resource dependency graph preview to occupy the entire side panel. It seems to currently be limited height-wise at least:

it'd be nice to eventually allow the resource dependency graph preview to occupy the entire side panel

Try again :)

it'd be nice to eventually allow the resource dependency graph preview to occupy the entire side panel

Try again :)

not seeing the preview button anymore after re-launching after pulling in these commits:
$ git pull --rebase
Updating adb3ea0..317ea85
$ code .

@kddejong @PatMyron if you review this i'll buy you a beer (or whatever drink you like)

working incredibly well in dark mode, light mode and online 🎉

sometimes the preview button disappears when switching back to a CloudFormation template from editing a non-CloudFormation JSON/YAML file until the CloudFormation template is re-edited, re-saved, or re-opened again, but just something to keep in mind

really excited for this to launch 🥳
only thing left is a mention of pydot in the Requirements section of the README
we should also update the README screenshot eventually to a better one including this 🎉

awesome new feature!


Add new Rails/Pick cop

pick(...) is a shorthand for pluck(...).first.
https://api.rubyonrails.org/classes/ActiveRecord/Calculations.html#method-i-pick
This cop is unsafe because pluck is defined on both ActiveRecord::Relation and Enumerable, whereas pick is only defined on ActiveRecord::Relation in Rails 6.0. This will be addressed in Rails 6.1 via https://github.com/rails/rails/pull/38760, at which point the cop will be safe. I don't know if there's currently a way to apply a different default configuration based on the Rails version, but I'd be happy to investigate that once Rails 6.1 is released.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Thanks!


FIX: Derive field from SyN displacements using EPI affine

Theoretically, a fieldmap in Hz is the number of voxels/second that signal is shifted from the true location. SyN-SDC calculates the shift in mm from the true location. In order to find the equivalent fieldmap, we need the size of the voxels, readout time and phase-encoding direction of the EPI image that is to be corrected.
Previously, we were deriving the fieldmap from the readout time and phase-encoding direction of the EPI image, but the voxel size of the displacement field. This causes an overestimate of the number of voxels to be shifted, and thus exaggerated correction.
Original EPI:

Current unwarp:

New unwarp:

@mgxd @oesteban This is ready to go.

👍 definitely an improvement, although the current unwarping in the images you shared looks much better than in https://github.com/nipreps/fmriprep/issues/3158

Agreed. Running tests on one known to produce extreme distortions.


Fix missing gradlew in generated project

This just makes MANIFEST.in permissive enough to include gradlew (and all files that don't have a . at the end of their name).

/azp run


Issue with Serverless offline plugin

from mangum import Mangum
from fastapi import FastAPI

app = FastAPI()

@app.post("/items/")
def create_item(item_id: int):
    return {"id": item_id}

@app.get("/items/")
def list_items():
    items = [{"id": i} for i in range(10)]
    return items

@app.get("/")
def read_root():
    return {"Hello": "World!"}

handler = Mangum(app)

I'm using the above code when specified in main.py and it's configured with a catch-all route in serverless.yml. I want to test it locally using the serverless-offline plugin but when I run the offline plugin using sls offline, I don't get any response in the browser for any routes. It just says, localhost didn’t send any data.
What could I be doing wrong? The offline plugin is listening on port 3000 by default.

@anil-grexit can you post your serverless.yml?
I tried to re-create your issue using the example here: https://github.com/chgangaraju/fastapi-mangum-example (NOTE: I changed the Mangum and FastAPI requirements to be the latest versions and Python 3.7), however I did not have any issues using the serverless-offline plugin with the following serverless.yml:
service: fastapi-mangum-example

provider:
  name: aws
  runtime: python3.7
  region: ${opt:region, 'us-east-1'}
  stage: ${opt:stage, 'test'}
  environment:
    REGION: ${self:provider.region}
    LOG_LEVEL: ${opt:loglevel, 'INFO'}

custom:
  pythonRequirements:
    dockerizePip: non-linux

package:
  exclude:
    - node_modules/**
    - venv/**
    - .direnv/**

functions:
  app:
    handler: app.main.handler
    events:
      - http: ANY /
      - http:
          path: /{proxy+}
          method: ANY

plugins:
  - serverless-python-requirements
  - serverless-offline

Hi @erm
I found the issue in my setup. Basically, it was a networking issue. I'm working with serverless inside a docker image and by default, serverless-offline or uvicorn bind to localhost. I only needed to bind it to <IP_ADDRESS> and voila! It started working. Thank you so much for your help. Really appreciate it.


Forgetting to add #[template_child] to TemplateChild members leads to runtime crashes when accessing the member

When I have a struct like:
    #[derive(Debug, CompositeTemplate)]
    #[template(resource = "/dev/Cogitri/test/ui/my_window.ui")]
    pub struct MyWindow {
        #[template_child]
        pub add_data_button: TemplateChild<gtk::Button>,
    }

And init it like so:
        fn new() -> Self {
            Self {
                add_data_button: TemplateChild::default(),
            }
       }

I don't get any error message during compilation or when creating the object. However, once I access add_data_button, the app crashes:
thread 'main' panicked at 'assertion failed: !self.ptr.is_null()', /var/home/rasmus/Projects/Jinsei/build/target/cargo-home/git/checkouts/gtk4-rs-e74ad56283dfeb5e/64d0e1c/gtk4/src/subclass/widget.rs:1000:13
stack backtrace:
   0: std::panicking::begin_panic
             at /var/home/rasmus/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/panicking.rs:505
   1: <gtk4::subclass::widget::TemplateChild<T> as core::ops::deref::Deref>::deref
             at ./build/target/cargo-home/git/checkouts/gtk4-rs-e74ad56283dfeb5e/64d0e1c/gtk4/src/subclass/widget.rs:1000
   2: health::windows::window::imp::HealthWindow::connect_handlers
             at ./src/windows/window.rs:158
   3: <health::windows::window::imp::HealthWindow as glib::subclass::object::ObjectImpl>::constructed
             at ./src/windows/window.rs:129
   4: glib::subclass::object::constructed
             at ./build/target/cargo-home/git/checkouts/gtk-rs-48ef14c1f17c79fb/c58fab4/glib/src/subclass/object.rs:109
   5: <unknown>
   6: g_object_newv
   7: glib::object::Object::new_internal
             at ./build/target/cargo-home/git/checkouts/gtk-rs-48ef14c1f17c79fb/c58fab4/glib/src/object.rs:1165
   8: glib::object::Object::with_type
             at ./build/target/cargo-home/git/checkouts/gtk-rs-48ef14c1f17c79fb/c58fab4/glib/src/object.rs:1105
   9: glib::object::Object::new
             at ./build/target/cargo-home/git/checkouts/gtk-rs-48ef14c1f17c79fb/c58fab4/glib/src/object.rs:1074
  10: health::windows::window::HealthWindow::new
             at ./src/windows/window.rs:213
  11: <health::core::application::imp::HealthApplication as gio::subclass::application::ApplicationImpl>::activate
             at ./src/core/application.rs:57
  12: gio::subclass::application::application_activate
             at ./build/target/cargo-home/git/checkouts/gtk-rs-48ef14c1f17c79fb/c58fab4/gio/src/subclass/application.rs:368
  13: g_signal_emit_valist
  14: g_signal_emit
  15: <unknown>
  16: <T as gio::subclass::application::ApplicationImplExt>::parent_local_command_line
             at ./build/target/cargo-home/git/checkouts/gtk-rs-48ef14c1f17c79fb/c58fab4/gio/src/subclass/application.rs:232
  17: gio::subclass::application::ApplicationImpl::local_command_line
             at ./build/target/cargo-home/git/checkouts/gtk-rs-48ef14c1f17c79fb/c58fab4/gio/src/subclass/application.rs:99
  18: gio::subclass::application::application_local_command_line
             at ./build/target/cargo-home/git/checkouts/gtk-rs-48ef14c1f17c79fb/c58fab4/gio/src/subclass/application.rs:411
  19: g_application_run
  20: <O as gio::application::ApplicationExtManual>::run
             at ./build/target/cargo-home/git/checkouts/gtk-rs-48ef14c1f17c79fb/c58fab4/gio/src/application.rs:23
  21: health::main
             at ./src/main.rs:27
  22: core::ops::function::FnOnce::call_once
             at /var/home/rasmus/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/ops/function.rs:

Which makes sense, but it'd be nice if it either crashed during construction with a more helpful error message (or maybe even when compiling).
Tested w/ e0cbea57

(or maybe even when compiling)

I suppose since the struct is being processed by the CompositeTemplate proc macro, this would indeed be possible.
It would also be possible to make it less verbose by either having #[template_child] automatically wrap a member with TemplateChild, or removing  #[template_child] and auto-detecting TemplateChild members. The former seems too magical and unclear, the latter might be alright.

The issue with the second suggestion is that you can no longer set a different id for the type or mark it as internal-child.

The first suggestion cannot be done without making it an attribute macro. At bare minimum we probably want to use the better error message from TemplateChild::get inside that Deref impl.
The other things we do can are:

Try to parse the XML inside the macro and check for the existence of the ids. This should work but cannot  be done with resource templates. We can only do null checking here, not type checking.
Add another trait method for some null + type checking during init_template. This should work in all cases.


Generic SSO option

Describe the feat you'd like
Hello,
It could be really great if authentication can  be used with SSO portal for example Azure AD
Additional context
We found that you do a great jobs already! Thanks a lots :).
I think there is a piece of config available here :
https://django-auth-adfs.readthedocs.io/en/latest/oauth2_explained.html

Any news here


--force option doesn't like trailing / on output path

Ran into an interesting testcase where --force isn't doing it's thing on Debian 9.x x64 due to the trailing / in the path:
./tippecanoe -f -zg -e /dev/shm/ -l test test.geojson
Works, no problem.
Run it again though:
test.geojson:0: feature without coordinates array
In JSON object {"type":"Feature","geometry":{"type":"LineString","coordinaUes":[[102.0,0.0],[103.0,1.0],[104.0,0.0],[105.0,1.0]]},"properties":{"prop0":"value0","prop1":0.0}}
2 features, 67 bytes of geometry, 6 bytes of separate metadata, 39 bytes of string pool
Choosing a maxzoom of -z15 for features about 14 feet (5 meters) apart
Can't write tile to already existing /dev/shm//0/0/0.pbf

Seems like the trailing / on the path throws it for a loop. FYI, -F doesn't care about the trailing /.

Thanks for the report. I'll take a look.

I wonder if this is behavior is specific to your OS and its /dev/shm, because it isn't reproducible for me (on either Linux or MacOS). Does it happen for you with ordinary directories or only with this special file system?

Writing to /tmp/1/ over and over results in no error so maybe it is just a quirk of that particular path. I'm going to close this.


HDMI output fails after suspend/hibernate using bumblebee and intel-virtual-output

Issue description
HDMI output on MSI GS65 Stealth Thin laptop does not work after suspend or hibernate.
I have an Optimus laptop with bumblebee installed. I use "intel-virtual-output" from the xorg.xf86videointel package to enable HDMI out. This works fine until I suspend or hibernate once. Afterwards HDMI output will not work. Even if I restart, it still will not output anything. Even if I restart, and go into windows, windows will also not be able to display anything through HDMI out.
The only way I can again be able to display to the HDMI out after suspending is if I shutdown the machine and start it using the power button.
I also tried updating the bios, to no avail.
Here are the things I tried (between each one, I fully shutdown, and started the machine using the power button):
1. start intel-virtual-output, and xrandr -- HDMI out works
2. (hibernate or suspend)
3. kill intel-virtual-output, and xrandr
4. start intel-virtual-output, and xrandr -- HDMI out doesn't work

1. start intel-virtual-output, and xrandr -- HDMI out works
2. kill intel-virtual-output, and xrandr
3. (hibernate or suspend)
4. start intel-virtual-output, and xrandr -- HDMI out doesn't work

1. start intel-virtual-output, and xrandr -- HDMI out works
2. hibernate (system turns completely off)
3. go into windows
-- HDMI output doesn't work
4. shutdown windows
5. start windows
-- HDMI output works
6. resume from hibernate
7. kill and restart intel-virtual-output, and xrandr -- HDMI out works

1. start intel-virtual-output, and xrandr -- HDMI out works
2. hibernate (system turns completely off)
3. go into windows
-- HDMI output doesn't work
4. restart windows
-- HDMI output doesn't work
5. shutdown windows
6. start windows
-- HDMI output works
7. resume from hibernate
8. kill and restart intel-virtual-output, and xrandr -- HDMI out works

This last one is very strange to me. Even if I go into windows, restart and go back into windows, the HDMI output will still not work. Only if I shutdown in windows will it work again. And then it will even work after I resume from hibernate within linux (and this also works in linux, if I shutdown completely in linux, when I restart, HDMI output works again). It seems there is something special about shutting down that even hibernate isn't doing.
Steps to reproduce
Run:
$ intel-virtual-output
$ xrandr --output VIRTUAL1 --mode VIRTUAL1.447-2560x1440
$ systemctl suspend
HDMI output will then not send a signal after this no matter what I do until I fully shutdown, for example:
$ killall intel-virtual-output
$ intel-virtual-output
$ xrandr --output VIRTUAL1 --mode VIRTUAL1.447-2560x1440
will do nothing. Also see other cases listed above
Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.
[tim@yellowhat:~]$ nix-shell -p nix-info --run "nix-info -m"

system: "x86_64-linux"
host os: Linux 4.14.53, NixOS, 18.03.132821.56fad146a12 (Impala)
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.0.4
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

Here are some journal logs of various points in this process. Note that I don't think this line is a problem:
Jul 08 18:59:08 yellowhat bumblebeed[839]: [XORG] (EE) /dev/dri/card0: failed to set DRM interface version 1.4: Permission denied
because it even occurs when I can successfully display from the HDMI output.
journal_after_hdmi_output.txt
journal_after_hdmi_output_kill_ivo_restart_ivo_xrandr__works__.txt
journal_after_hibernate_and_restart_ivo_xrandr.txt
journal_after_resume_from_suspend.txt
journal_after_suspend_then_kill_and_restart_intel-vout_and_xrandr.txt

I found a workaround, in case this is affecting anyone else. When you want to wake up the machine, start it, shut it down at the grub menu by pressing the power button, and restart it. Then HDMI output will function correctly again.

Hey! I seem to have somewhat the same problem. My computer hibernated while outputting HDMI using Intel-virtual-output but the difference is I haven't been able to get it working again neither on Linux nor windows. Any ideas?


prometheus api and +/-Inf values

System info: [Include InfluxDB version, operating system name, and other relevant details
InfluxDB v1.7.0 (git: 1.7 dac4c6f571662c63dc0d73346787b8c7f113222a)
Steps to reproduce:
Send +/-Inf values with prometheus' remote_write.
Expected behavior: [What you expected to happen]
Ignore +/-Inf values
Actual behavior: [What actually happened]
The whole batch fails.
Additional info: [Include gist of relevant config, logs, etc.]
Here is an example setup with docker-compose:
https://gist.github.com/datamuc/d6085cc4c4190619796f0ea5a8661dce
influxdb_1       | [httpd] <IP_ADDRESS> - admin [16/Nov/2018:16:34:55 +0000] "POST /api/v1/prom/write?db=prom HTTP/1.1" 400 59 "-" "Go-http-client/1.1" 8ce96825-e9bd-11e8-8001-0242c0a8d002 88584
prometheus_1     | level=warn ts=2018-11-16T16:34:55.788646973Z caller=queue_manager.go:531 component=remote queue="0:http://influxdb:8086/api/v1/prom/write?db=prom" msg="Error sending samples to remote storage" count=375 err="server returned HTTP status 400 Bad Request: {\"error\":\"+/-Inf is an unsupported value for field value\"}"

All metrics sent in this POST request are lost because of a single +Inf value.

QuickFix, at least identifying the culprit:
diff --git a/prometheus/converters.go b/prometheus/converters.go
index f3f438587..3cb9a5064 100644
--- a/prometheus/converters.go
+++ b/prometheus/converters.go
@@ -57,6 +57,7 @@ func WriteRequestToPoints(req *remote.WriteRequest) ([]models.Point, error) {
 		for _, s := range ts.Samples {
 			// skip NaN values, which are valid in Prometheus
 			if math.IsNaN(s.Value) {
+				// FIXME: Add some logging here?
 				droppedNaN = ErrNaNDropped
 				continue
 			}
@@ -66,7 +67,8 @@ func WriteRequestToPoints(req *remote.WriteRequest) ([]models.Point, error) {
 			fields := map[string]interface{}{fieldName: s.Value}
 			p, err := models.NewPoint(measurement, models.NewTags(tags), fields, t)
 			if err != nil {
-				return nil, err
+				// FIXME: Add some logging here?
+				continue
 			}

 			points = append(points, p)

This is tripping me up in 1.7.4 as well

Fixed via #12813


Update cats-core to 2.0.0

Updates org.typelevel:cats-core from 2.0.0-RC2 to 2.0.0.
Release Notes/Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-core" } ]

labels: semver-pre-release

Codecov Report

Merging #139 into series/0.7.x will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           series/0.7.x     #139   +/-   ##
=============================================
  Coverage         61.18%   61.18%
=============================================
  Files                11       11
  Lines               304      304
  Branches             11       11
=============================================
  Hits                186      186
  Misses              118      118

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 d5124b6...213a983. Read the comment docs.


Runs unlimited

Hey there, i tried the script and it ends in a never ending "red" error messages ... I did the test with the small racing.txt.
Result in the eventloop looks like: el_add=639846445
el_gamemode=racing
el_laps=0
el_add=gravel1_main_loop
el_gamemode=racing
el_laps=6
el_add=mixed3_short_loop_rev
el_gamemode=racing
el_laps=6
el_add=mixed4_main_circuit
el_gamemode=racing
el_laps=6
el_add=sandpit1_short_loop
el_gamemode=racing
el_laps=6
el_add=mixed7_r1_rev
el_gamemode=racing
el_laps=6
el_add=gravel1_main_loop_rev
el_gamemode=racing
el_laps=6
el_add=mixed3_short_loop
el_gamemode=racing
el_laps=6
el_add=tarmac1_short_circuit_rev
el_gamemode=racing
el_laps=6
el_add=sandpit1_short_loop_rev
el_gamemode=racing
el_laps=6
el_add=tarmac1_short_circuit
el_gamemode=racing
el_laps=6
el_add=mixed4_main_circuit_rev
el_gamemode=racing
el_laps=6
el_add=mixed8_r1
el_gamemode=racing
el_laps=6
el_add=mixed8_r1
el_gamemode=racing
el_laps=6
el_add=mixed8_r1
el_gamemode=racing
el_laps=6
and then midex8 over and over and over ...
As my powershell is German here is the error:
Get-Random : Der Wert "mixed3_short_loop_rev" kann nicht in den Typ "System.Int32" konvertiert werden. Fehler: "Die
Eingabezeichenfolge hat das falsche Format."
In C:\XXX\XXXX\XXXXX\Wreckfest-eventloop-randomizer-main\script\WreckFest_ERS.ps1:64 Zeichen:19

    $rt_map = Get-Random $($types[$rt].Maps) # pick a random map  ...

              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidArgument: (:) [Get-Random], PSInvalidCastException
FullyQualifiedErrorId : InvalidCastFromStringToInteger,Microsoft.PowerShell.Commands.GetRandomCommand

Thank you

already got that... fixed it with https://github.com/Hedrauta/Wreckfest-eventloop-randomizer/commit/fcb07fb71c113a605c0386b9a53850468a611d3e
Something about the -Maximum in the Get-Random had changed.... it should work fine now ;) sorry that slow respond :D had some issues with my PC ^.^

already got that... fixed it with https://github.com/Hedrauta/Wreckfest-eventloop-randomizer/commit/fcb07fb71c113a605c0386b9a53850468a611d3e
Something about the -Maximum in the Get-Random had changed.... it should work fine now ;) sorry that slow respond :D had some issues with my PC ^.^


Cli: Container exec command to handle whitespace and strings

See spec for example.

Nice fix, thanks!


Updated gemspec and changelog + remove 1.9 support leftovers

See individual commit messages for more details.
Thanks for working on this tiny gem! I make use of in in my own backtracie and it's been a great way to get access to things I shouldn't mess with ;)

Hey @hurricup just a tiny ping in case you've missed this PR :)
Let me know if I can help in any other way ^^

Sorry, had no time to check this yet.

Cool cool cool, no hurries!

Just saw there was a new release, would be nice to get this PR in as well!

😭

Awesome, thanks for adding this in! :)


Clean up the update/verify script

What type of PR is this?

/kind cleanup
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

NONE

/lgtm


FR - New strings

Big doubt on settingsPanelToggles, I'm not so happy with current translation but can't find anything better in French.

Big doubt on settingsPanelToggles, I'm not so happy with current translation but can't find anything better in French.

This was a problem for me as well when creating the Japanese translation. I used the word equivalent to others which is closest to this meaning in Japanese. In Japanese, this translation doesn't seem to be misleading.

@cyril-amar
I approve this pull request, so if you're using deck.blue and come up with a better word, please create another pull request!


Create 02_appbar_example.py

Add new Appbar example

 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.


issue 10 solved

Please, go through these steps before you submit a PR.

[ ] My Pod Leader knows I'm working on this Pull Request
[ ] I've explained what the Pull Request is adding.
[ ] I've explained why this is important.

This PR would partially resolve #10
Added details for three fellows in this PR. Once this PR gets approved, @norachams would be adding in the location details for other fellows.

This only shows three fellows. Please, take a look. I expect to see more fellows

Yes, can you please merge the PR so i can pull the code and add the rest of the fellows.

This only shows three fellows. Please, take a look. I expect to see more fellows
Me and Nora did pair programming on this issue. So we decided to resolve the issue in  2 PRs. Once, this one gets approved, she would be adding in the other fellow details in her PR.


Draft: Container plugin improvements

Separates build and runtime steps
Final container size reduced

The 'app-binary-name' string needs to be replaced by the project's app binary name


imports don't seem to be sorted

current code:
https://github.com/OmniSharp/omnisharp-roslyn/blob/47bebf9eb1d2fd45710ff58640e9aaf96d66f29d/src/OmniSharp.Roslyn.CSharp/Workers/Refactoring/FixUsingsWorker.cs#L56-L58
code when fixusing is first implemented:
https://github.com/OmniSharp/omnisharp-roslyn/blob/eeb76b426c47252f96f2c66b1a88fa0fd3f8b43c/src/OmniSharp/Workers/Refactoring/FixUsings.cs#L31-L34
was trying to figure out how to turn on the sorting of imports. couldnt find a flag to turn it on, i found that sort function was left out of the current fixusing function, is it implemented elsewhere, and if so, how do we trigger it?

It was moved to formatting https://github.com/OmniSharp/omnisharp-roslyn/blob/696cf2159f89a6c9e7bbc296e6eb95e76ac916ac/src/OmniSharp.Roslyn.CSharp/Workers/Formatting/FormattingWorker.cs#L109-L115
when enabled, imports are organized as part of regular code formatting.

thank u very much, i managed to find the corresponding flag :+1:


update participant import test based on front end change

Browser test for participant import needs to be updated as the front end changed.
ref: https://app.asana.com/0/8755179914462/235819452188789

@Drew, reminder that , as the front end change is still in develop branch,
the test will fail if running against coinstraining.
On Thu, Jan 5, 2017 at 5:09 PM, Drew Landis<EMAIL_ADDRESS>wrote:

Merged #11 https://github.com/MRN-Code/coins-browser-test/pull/11.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MRN-Code/coins-browser-test/pull/11#event-913454679,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFFaGWRnrMjXkZiPlQahxje9O6C6WTbVks5rPYYogaJpZM4La5Qr
.


do not swallow exception when processing system messages

re-throw exception when processing system messages rather than swallowing
test to prove supervision works when failure/exception processing Started message
Fixes https://github.com/AsynkronIT/protoactor-dotnet/issues/196

👍


Checklist:

[x] Securely signed commits

travis build green, on php73:sqlite build is unrelated.


CI:  runtime.RuntimePolicyImportTests checks policy trace output

runtime.RuntimePolicyImportTests checks policy trace output

Failing for the past 2 builds (Since Failed#2033 )
Took 14 sec.
Stacktrace
/home/jenkins/workspace/cilium-ginkgo_cilium_master/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:383
Expected allowed identities [1 31542], but instead got [1 29722 31542]
Expected
    <[]int64 | len:2, cap:2>: [1, 31542]
to equal
    <[]int64 | len:3, cap:4>: [1, 29722, 31542]
/home/jenkins/workspace/cilium-ginkgo_cilium_master/src/github.com/cilium/cilium/test/runtime/Policies.go:1933
Standard Output
Number of "context deadline exceeded" in logs: 0
Number of "level=error" in logs: 0
Number of "level=warning" in logs: 0
Number of "Cilium API handler panicked" in logs: 0
Number of "Goroutine took lock for more than" in logs: 0

Standard Error
STEP: Setting PolicyEnforcement=default
STEP: Checking policy trace by labels
STEP: Importing policy that allows ingress to "id.httpd1" from the host and "id.httpd2"
STEP: Setting up policy: /home/vagrant/go/src/github.com/cilium/cilium/test/policy_f9ea8b5e.json
STEP: Verifying that trace says that "id.httpd2" can reach "id.httpd1"
STEP: Getting models of endpoints to access policy-related metadata
STEP: Getting all identities which contain label container:id.httpd2
STEP: Verifying allowed identities for ingress traffic to "httpd1"
=== Test Finished at 2018-12-12T17:49:57Z====
===================== TEST FAILED =====================
cmd: sudo cilium endpoint list
Exitcode: 0
Stdout:
 	 ENDPOINT   POLICY (ingress)   POLICY (egress)   IDENTITY   LABELS (source:key[=value])   IPv6                 IPv4            STATUS
	            ENFORCEMENT        ENFORCEMENT
	 9659       Disabled           Disabled          41193      container:id.httpd3           f00d::a0f:0:0:3472   <IP_ADDRESS>    ready
	                                                            container:id.service1
	 41358      Disabled           Disabled          44400      container:id.app2             f00d::a0f:0:0:9bfc   <IP_ADDRESS>    ready
	 42001      Disabled           Disabled          43950      container:id.app3             f00d::a0f:0:0:c7a2   <IP_ADDRESS>    ready
	 43741      Disabled           Disabled          4          reserved:health               f00d::a0f:0:0:be36   <IP_ADDRESS>   ready
	 44492      Enabled            Disabled          44465      container:id.httpd1           f00d::a0f:0:0:5e65   <IP_ADDRESS>   ready
	                                                            container:id.service1
	 47374      Disabled           Disabled          31542      container:id.httpd2           f00d::a0f:0:0:913f   <IP_ADDRESS>     ready
	                                                            container:id.service1
	 63409      Disabled           Disabled          13975      container:id.app1             f00d::a0f:0:0:6653   <IP_ADDRESS>    ready

Stderr:
 	 level=info msg="Detected MTU 1500" subsys=mtu

cmd: sudo cilium policy get
Exitcode: 0
Stdout:
 	 [
	   {
	     "endpointSelector": {
	       "matchLabels": {
	         "any:id.httpd1": ""
	       }
	     },
	     "ingress": [
	       {
	         "fromEndpoints": [
	           {
	             "matchLabels": {
	               "reserved:host": ""
	             }
	           },
	           {
	             "matchLabels": {
	               "any:id.httpd2": ""
	             }
	           }
	         ]
	       }
	     ]
	   }
	 ]
	 Revision: 35

Stderr:
 	 level=info msg="Detected MTU 1500" subsys=mtu

===================== Existing AfterFailed =====================

[[ATTACHMENT|184607d6_RuntimePolicyImportTests_checks_policy_trace_output.zip]]

Link: https://jenkins.cilium.io/job/cilium-ginkgo/job/cilium/job/master/2034/testReport/(root)/runtime/RuntimePolicyImportTests_checks_policy_trace_output/
Zip:
184607d6_RuntimePolicyImportTests_checks_policy_trace_output.zip

@ianvernon potentially related to https://github.com/cilium/cilium/pull/6375 ?
https://jenkins.cilium.io/job/cilium-ginkgo/job/cilium/job/master/2031/

Duplicate of #6347.


raise ValueError("First layer should be input")

Why first layer type have to be 'input'? There is no such data layer in deploy.prototxt.

please delete this restrain,and the tools can work normally


Short name for analytics changes

The suggested name will be more convenient :)

I agree about more convenient, but we'll have a problem with handle_text() function in the main module, because it has faculty handler, which expects 5 symbols (not 6 like in your PR), line 183 in main.py. So we need to refactor this function as well before this before changing the faculty key.


No response when using

No response when using and can't update version 4.3

Bundle Identifier: com.bohemiancoding.sketch3
Build Version: 72.4
Build Number: 124482
OS Version: 11.5.2
CPU Type: X86
Rosetta: No

No response when using，and can't update version 4.3

Bundle Identifier: com.bohemiancoding.sketch3
Build Version: 72.4
Build Number: 124482
OS Version: 11.5.2
CPU Type: X86
Rosetta: No

Hello @seobaby ! Please use the latest version of Chart plugin: https://chartplugin.com/


EAM: Suggest to add App Instance Name

Problem description
No human-readable identifier for application instances. Very difficult differentiate between application instances only via UUID.
Possible evolution
Suggest to add a user-assigned name string field to application instances, much like AppManifest has a user-defined name field. This will allow users to easily distinguish between the different application instances they have deployed.
Alternative solution
None.
Additional context
None.

The UUIDs of the application instances are generated by the Edge Platform depending on the Edge Cloud Zones, so it is transparent for the Application Provider and this new parameter should follow the same approach, if we want to add this new parameter we should agree on a data model name for it as well (e.g. AppName-EdgeCloudZoneName).
Some doubts here:

For one application can you deploy one app instance per EdgeCloudZone or several app instances per EdgeCloudZone? if multiple instances are allowed then we should add numbering to the data model name.
The multiple app instances option is based on auto scaling or is something that the app provider can define?
An application could be composed of multiple microservices, how can we define a naming for this?

My intent was the name field be user-specified, not server generated. This is the same as AppManifest.Name. The user can choose any name they want, because that name will help them identify the application instance, so there is no specific format beyond being a string.

Regarding your doubts:

We allow multiple instances of the same app per edge-cloud-site, but it's really irrelevant here. If the user chooses a name that already exists (and here we need to define the scope, i.e. the name must be unique per edge-cloud-site and per AppProvider, but not necessarily unique globally), then the API fails and the user must choose a non-conflicting name.
Typically autoscaling is left to Kubernetes, and so all pods that are scaled are still part of a single app instance from the platform's point of view. In any case the name is for the app instance, not individual pods or VMs that may be scaled as part of the single app instance.
Similarly, the name is for the app instance, not the individual microservices that may be part of the instance, so it doesn't matter how many individual services an app instance may be composed of.

My intent was the name field be user-specified, not server generated. This is the same as AppManifest.Name. The user can choose any name they want, because that name will help them identify the application instance, so there is no specific format beyond being a string.

In that case, developers should define one name per instance, I think it could be complex when instantiating several instances, imagining the case for 50 instances developers have to think of 50 names. And what happens when the instantiation is triggered by an user request, dynamic instantiation, the developer receives a notification to add the name for the instance?

In that case, developers should define one name per instance, I think it could be complex when instantiating several instances, imagining the case for 50 instances developers have to think of 50 names. And what happens when the instantiation is triggered by an user request, dynamic instantiation, the developer receives a notification to add the name for the instance?

That's a fair point, I would suggest that we allow the user to leave the name blank, in which case the system will either leave it blank, or perhaps fill it in with the appInstanceId.

To address the issue with mass creation of objects (AppInstances), https://github.com/camaraproject/EdgeCloud/issues/256 proposes that the API only targets a single AppInstance due to other issues. Additionally, https://github.com/camaraproject/EdgeCloud/issues/312 proposes that names are scoped to the tenant (Provider), so as long as the user is generating names that don't conflict with themselves, there should be no conflicts.


 This technique on carousels or sliders load images only when the image is next in the slide (to the right) BUT even the first image still loads EVEN IF the slider is lower in the page "below the fold".  This is the technique slider pro uses and i guess pretty much every carousel or slider uses.  The problem is that pages that have slider pro lower in the page "below the fold" the first image for sure still loads and if the carousel is set to layout option 2 is has to load EVERY image that is visible on that initial view (AND preview images if those layout options are used with low res preview images).  If the slider is set to Auto Play the carousel is moving and loading images even if the carousel is "below the fold" and not visible.   This is a MAJOR hit now with Google's mobile first update hitting in June/July this year.
 This technique ONLY loads images within a "threshold" distance, usually 100 pixels away so the image or images only load as the screen is visible or within that "threshold" distance.  Something like B-Lazy image loading.  Carousel sliders not visible lower in the page or "below the fold" that are set to auto play may still move BUT if they are using inline base64 place holder images with the built in left to right lazy load as mentioned in method 1 above then none of the main images are actually loaded yet so no bandwidth hit and no extra http requests for images and for mobile devices so it greatly speeds up page load times and helps with Google's mobile first scoring when using their page speed insights test page.
So my question - is there a way to convert slider pro to the 2nd technique so that slider pro images will NOT load unless either visible on screen or within a threshold distance like 100 pixels from the visible screen size so it begins to download images ONLY if you scroll down far enough on the page?  This speeds up mobile loading times and comes into compliance with the new google standards on page scoring?

Hi,
It's possible to delay the instantiation of the slider instead of instantiating it when the page loads. You can use the 'scroll' event to check if the slider comes into view while the page is scrolled:
$( document ).on( 'scroll.sliderPro', function( event ) {
	if ( $( window ).scrollTop() > $( '#my-slider' ).position().top - $( window ).height() - 100 ) {
		$( document ).off( 'scroll.sliderPro' );
		$( '#my-slider' ).sliderPro({ width: 900, height: 500, arrows: true });
	}
} );

Cheers!

in a preliminary test it works with 1 issue.  You did what i mentioned with the threshold but unlike a dedicated script such as B-Lazy if for some reason the slider pro gallery is slightly visible within the screen or i guess even the 100 pixel threshold you programmed in the entire slider does NOT show and you have to scroll down or up out of view then scroll back into view to make it work.
so your version seems perfect if the slider pro carousel is definitely out of view but if it is close to being in view it does not work as planned and requires a scrolling out of view then back into view.  is there a way to add a trigger that if it is already in view or within the threshold amount you programmed in that it loads?  other true lazy loads are able to calculate that somehow.
you are on the right track.  although not an exact lazy load it accomplishes the same thing but we need to figure out how to trigger it for carousels that may start out in view or within the threshold limit.

David.....tip.....you are probably the most active carousel developer responding on github to your script and is one of the reasons i chose to use slider pro.  so many other carousels and other js have little to no support.
you have so many updates you have posted on github that you should show some of these on your main slider pro page in the form of extra examples.  the more examples you show of variations of the main 5 examples the more people will see how powerful slider pro is.  i guess the best at doing this is the fred carousel that shows something like 78 completely different examples but it helps show so many different uses for it.
i think that might be helpful to grow slider pro.....if that is something you want to do.

Hi. Thank you for the suggestion! I will consider that.
Regarding the custom JS, it can be adjusted to check if the slider is in view when the page loads:
function checkSlider() {
	if ( $( window ).scrollTop() > $( '#my-slider' ).position().top - $( window ).height() - 100 ) {
		$( document ).off( 'scroll.sliderPro' );
		$( '#my-slider' ).sliderPro({ width: 900, height: 500 });
	}
}

$( document ).on( 'scroll.sliderPro', checkSlider );
checkSlider();

Cheers!


Autowiring and autoconfiguration

Please describe the feature you would like to see implemented.
Please make use of Symfony's autowiring and autoconfiguration.
It would make platform and bundle development way easier and custom tags are also supported.

You can do that alreadyhttps://github.com/FriendsOfShopware/FroshDevelopmentHelper/blob/0.3/src/Resources/config/services.xml#L13

You can autowire most of your own services but there are issues when it comes to repository injections & automatic tagging. For example when trying to auto-tag an entity definition.

Repositories work already https://github.com/shopware/platform/commit/7bbf8da091bbc053e666ad10b109cc2ff31c0635
Feel free to create a pull request to add more auto-tagging.

didn't know but looks great! thanks!


Select documentacion

Cambios  / 13/01/21

Agregue documentacion a Select()
Converti en modulo Utils.jl
Algunos cambios de compatibilidad a Make.jl

Codecov Report

Merging #17 (4c296ec) into master (1eb9465) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master     #17   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files           3       3
  Lines          47      48    +1
======================================
- Misses         47      48    +1

Impacted Files
Coverage Δ

src/Operations.jl
0.00% <0.00%> (ø)

src/Stats.jl
0.00% <ø> (ø)

src/Utils.jl
0.00% <ø> (ø)

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 1eb9465...4c296ec. Read the comment docs.

Codecov Report

Merging #17 (4c296ec) into master (1eb9465) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master     #17   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files           3       3
  Lines          47      48    +1
======================================
- Misses         47      48    +1

Impacted Files
Coverage Δ

src/Operations.jl
0.00% <0.00%> (ø)

src/Stats.jl
0.00% <ø> (ø)

src/Utils.jl
0.00% <ø> (ø)

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 1eb9465...4c296ec. Read the comment docs.


Fix SNBT generating nothing for empty string values

Prior to this fix,
quartz_nbt::snbt::parse(&quartz_nbt::compound!{"abc":""}.to_string())

would give
Err(UnexpectedToken { index: 5, expected: "value" })

This fix simply edits the should_quote function to return true if the string is empty.
This pull request also includes a test for that so that it does not break in the future.

Thanks for this fix!


Very confusing for how to use it

From a user's PoV, the user guide is very confusion, e.g. if I launch the Android app on mobile device, is it possible to show available devices(when connected through USB), if not, how do I confugure VSCode for debugging? How do I know the Android player processs? ...

Hi @clifflinmao - I'm sorry to hear you're finding the debugger confusing to use.
The debugger is set up in a similar way to other debuggers in VSCode - using launch.json and a set of custom properties used to configure the debugger.
The debugger is designed to work with the most common case - a developer connecting a single device (or emulator) and launching the app via the VSCode Start Debug command. The extension deploys the app to the device and launches it with the debugger connected.
In the case that multiple devices are connected, an error is shown in the console which describes how to configure the debugger to launch on a particular device.
You can also use the property-completion command in launch.json to show the list of valid properties and how they work:

How do I know the Android player processs

I'm not sure what you mean by this. There's currently no option in the debugger to show a list of running android processes or connect to an already-running app. You could use logcat or adb to show the IDs of running processes.
If you would like to improve the extension or documentation, I'm happy to accept PRs.

Hi,
Thanks for your reply. My problem is not about debugging on multiple devices. It's about debugging a Unity 'exported' android project with VSCode, which is a bit different from debugging normal Android project with VSCode.
I understand that I need to add configuration in launch.json and I did as below. It can install and launch the APK in my phone successfully, but it does not stop at the breakpoint. I guess that is because my Android proejct is 'exported' from Unity, which has only 1 java file(which is the main activity), all my project code(that I need to debug) are not visible as Java file. So in this case, is it possible to debug with VSCode? Any example? Thanks.
       {             "type": "android",             "request": "launch",             "name": "Launch Android App2",             "appSrcRoot": "C:/workspaces/unity/AR4FM/P17-AzureAnchorTest3/Apps-Android/P17S5_K_2_Exported/AR4FM1_K_2T/src/main",             "apkFile": "C:/workspaces/unity/AR4FM/P17-AzureAnchorTest3/Apps-Android/P17S5_K_2_Exported/AR4FM1_K_2T/build/apk/AR4FM1_K_2T.apk",             "adbPort": 5037         },

No, it's not possible to debug other frameworks using this debugger.
The debugger only recognises java and kotlin files and uses the builtin Java Debug Wire Protocol to configure breakpoints, stepping etc for the Java runtime. If your project uses other types of files or a different runtime (e.g native code or a JS engine), it is unlikely to work.


Can't access array after parse

There is an API that responds with JSON arrays. The response can be an empty array [] or [{...}, {...}, {...}] an array with one or more objects. Here it its one example of a response:
[{
  "entityId": "74c975351-3fe7-4a5c-91d6-15b73e1304cd",
  "entityCoords": {
    "x": 100,
    "y": 825,
    "offsetX": 160,
    "offsetY": 215
  }
}]

As I receive the response from the API, I try to parse the string so I can verify how many objects are there, if any, and, for each one, get the information I need:
  Set JSONResponse = New JSONObject
  JSONResponse.Parse(APIResponse)

There is no error with that, but I can't access the JSONResponse object, no matter how I try, but if I write it down, I can see its contents, so JSONResponse.Write gives:
{"data":[{"entityId": "74c975351-3fe7-4a5c-91d6-15b73e1304cd","entityCoords": {"x": 100,"y": 825,"offsetX": 160,"offsetY": 215}}]}
But JSONResponse("data") is null/empty, JSONResponse.length or JSONResponse.items raises errors.
I figured out that if I parse the response in a temporary JSONObject and then, parse its serialization:
  Set JSONResponse = New JSONObject
  Set JSONTmp = New JSONObject
  JSONTmp.parse(APIResponse)
  JSONResponse.Parse(JSONTmp.serialize())

JSONResponse("data") become accessible as well I can get JSONRespose("data").length and iterate over the array, if any data is present.
Is this behaviour ok?! Or did I do something wrong in the first place?
Thanks!

Hi @mvlaran,
That behavior is not expected. You should be able to access the data property on the object, so I need to check if there is a bug there.
Usually, when parsing arrays, the Parse method returns a JSONArray object that enables you to access and iterate the items. This should work without problems.
You can see some examples in the README.

Hi @rcdmk! Thanks for your reply.
It really appears to be a bug, since it only works by doing this workaround I mentioned...

Hi,
The same thing happens to me, thanks for the trick to avoid it
Grettings.

Hi all,
A fix for this is on development branch (4b5fef0). Please feel free to test that out.
Best,
Ricardo


fix(bump): Fix incorrect paths when bumping release groups

When bumping release groups, the spawned subprocesses were using the repo root as the working directory, which meant that the command did not work properly for most release groups.
I fixed the paths and did some other refactoring to make the code a bit cleaner.

Thanks for the review, @Josmithr!


paper-tabs selected issue

hi the community
here is a coepen to illustrate my issue :+1: http://codepen.io/burno75/pen/LGbLoM
working : remove jack paper-tab -> annick selected and content 2 displayed OK
not working : remove annick  paper-tab -> KO jack paper-tab not selected (and  his iron-page content1 not visible)
Any idea ?
is there any compenent like chrome tabs ?
best regards
Bruno

This issue was moved to PolymerElements/paper-tabs#118


Responses are deleted when added to a story by md

Describe the bug
responses created in the bot responses page are sometimes deleted when added to a story via md.
To Reproduce
Steps to reproduce the behavior:
note: the behaviour is inconsistent, sometimes the response is deleted, other-times it works normally.

create a response via the bot responses page
use the md editor to add the response to a story
toggle to the visual editor
result:  the response shows utter_response_name, and does not exist in the bot responses page

Expected behavior
the response created in the bot responses page is added to the story.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: IOS
Browser chrome
76

Additional context
possibly related to responses being deleted when they do not exist in a story.

@kenrilea Is this still valid?


MaskRCNN + DynamicHead

We implement your Dynamichead with MaskRCNN-r50 on mmdetection.  results are worse than maskrcnn, and it took almost twice as long.

我按照作者的思路进行了尝试，发现在YOLO上很难收敛，效果很差。


[BUG]:  CSI PODMON is tainting the worker node

Bug Description
We are observing that in the CSI version v2.5.0-1-7e431865, Podmon is randomly applying taint to the worker nodes. The tainted node's loss of connectivity is reported in the log. However, there is no disconnection of connectivity found when we examine the storage network's connectivity. This problem began with CSI version 2.5.0 and is affecting all of our critical applications.
Logs
logs.zip
Screenshots
Some more details on what is observed on the logs when issue is observed.
	For example, Wn029, has the taint.
(poaccdcaf01)[ccdadmin@POAcCCDADM01 poaccdcaf01]$ kubectl describe node caf-pool06-poaccdcaf01-wn029 | grep Taint
Taints:             vxflexos.podmon.storage.dell.com:NoSchedule
	Error messages are filled with,
"csi-vxflexos.dellemc.com":"578CD2B2-EFCB-4332-B7C3-320CB50B6055","topolvm.cybozu.com":"caf-pool06-poaccdcaf01-wn029"}"
time="2023-04-15T03:58:36Z" level=info msg="ValidateVolumeHostConnectivity Node caf-pool06-poaccdcaf01-wn029 NodeId 578CD2B2-EFCB-4332-B7C3-320CB50B6055 Connected false"
time="2023-04-15T03:58:36Z" level=info msg="Pod polcafgp2/eric-data-object-storage-mn-mgt-595f7d74c4-skw9x node caf-pool06-poaccdcaf01-wn029 has no connectivity to arrayID default"
time="2023-04-15T03:58:37Z" level=info msg="Tainting node caf-pool06-poaccdcaf01-wn029 because of connectivity loss"
time="2023-04-15T03:58:37Z" level=info msg="Calling to tainting  caf-pool06-poaccdcaf01-wn029 with vxflexos.podmon.storage.dell.com NoSchedule (remove = false)"
time="2023-04-15T03:58:37Z" level=info msg="TaintAlreadyExists : vxflexos.podmon.storage.dell.com on node caf-pool06-poaccdcaf01-wn029"
	When we got o the node there is no connectivity issue. (Validate by pinging mdm and gateway IP)
eccd@caf-pool06-poaccdcaf01-wn029:~> ip a s ccd_stgfe
26: ccd_stgfe@bond_ccdstgfe: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 9000 qdisc noqueue state UP group default qlen 1000
link/ether 5e:a7:f6:b0:00:3f brd ff:ff:ff:ff:ff:ff
inet <IP_ADDRESS>/23 brd x.x.x.x scope global noprefixroute ccd_stgfe
valid_lft forever preferred_lft forever
eccd@caf-pool06-poaccdcaf01-wn029:~> ping x.x.x.x
PING <IP_ADDRESS> (x.x.x.x) 56(84) bytes of data.
64 bytes from x.x.x.x: icmp_seq=1 ttl=64 time=0.215 ms
^C
--- <IP_ADDRESS> ping statistics ---
1 packets transmitted, 1 received, 0% packet loss, time 0ms
rtt min/avg/max/mdev = 0.215/0.215/0.215/0.000 ms
eccd@caf-pool06-poaccdcaf01-wn029:~> ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=64 time=0.125 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=64 time=0.122 ms
^C
--- <IP_ADDRESS> ping statistics ---
2 packets transmitted, 2 received, 0% packet loss, time 1011ms
rtt min/avg/max/mdev = 0.122/0.123/0.125/0.001 ms
caf-pool06-poaccdcaf01-wn029:~ # /bin/emc/scaleio/drv_cfg --query_mdms
Retrieved 1 mdm(s)
MDM-ID 31f837752eaaa40f SDC ID 8dfe8e6700000031 INSTALLATION ID<PHONE_NUMBER>a7f62c IPs [0]-<IP_ADDRESS>
	Removing the taint manually doesn’t bring it back.
	Sometimes we also notice below errors (should be part of full logs provided earlier).
time="2023-04-13T17:19:41Z" level=error msg="could not unmarshal driver path key from nodeid annotation {"topolvm.cybozu.com":"pool08-riccdcaf01-wn042"}: to json: unexpected end of JSON
input"
time="2023-04-13T17:19:44Z" level=error msg="could not unmarshal driver path key from nodeid annotation {"topolvm.cybozu.com":"-ricaf01-wn038"}: to json: unexpected end of JSON
input"
time="2023-04-13T17:19:47Z" level=error msg="could not unmarshal driver path key from nodeid annotation {"topolvm.cybozu.com":"-pool09-ricaf01-wn047"}: to json: unexpected end of JSON
input"
Additional Environment Information
No response
Steps to Reproduce
a.	Drain and multiple reboots.
b.	Sometimes restarting csi pods also caused it.
Expected Behavior
CSI driver shouldn’t taint the node if it is healthy.
CSM Driver(s)
mdm_config:v1.2.0-19-98abcff9
podmon:v1.4.0-4-7e431865
csi-attacher:v3.5.0-50-98abcff9
csi-provisioner:v3.2.1-50-98abcff9
csi-snapshotter:v6.0.1-50-98abcff9
csi-resizer:v1.5.0-50-98abcff9
csi-vxflexos:v2.5.0-1-7e431865
Installation Type
Operator method, (not helm)
Container Storage Modules Enabled
Podmon : v1.4.0-4-7e431865) is enabled
Container Orchestrator
Ericsson Cloud Container Distribution (ECCD), 2.24.1
Operating System
SUSE Linux Enterprise Server 15 SP4

Closing issue as a fix has been provided. Please reopen if the issue persists.