Caddy 0.10.11 builds fails with status code 500

Builds for caddy 0.10.11—with many but not all plugins—fail with status code 500:
How to reproduce:
$ curl 'https://caddyserver.com/download/linux/386?plugins=http.hugo%2Chttp.upload%2Ctls.dns.cloudflare%2Chttp.awses%2Chttp.jekyll%2Chttp.mailout%2Ctls.dns.otc%2Ctls.dns.ns1%2Chttp.webdav%2Chttp.locale%2Chttp.minify%2Chttp.expires%2Ctls.dns.azure%2Ctls.dns.dnsmadeeasy%2Ctls.dns.cloudxns%2Chttp.restic%2Chttp.realip%2Cdns%2Chttp.datadog%2Ctls.dns.auroradns%2Ctls.dns.exoscale%2Ctls.dns.gandi%2Ctls.dns.googlecloud%2Ctls.dns.dnsimple%2Ctls.dns.route53%2Ctls.dns.lightsail%2Chttp.cors%2Ctls.dns.linode%2Cnet%2Ctls.dns.gandiv5%2Ctls.dns.godaddy%2Ctls.dns.namecheap%2Ctls.dns.vultr%2Chttp.nobots%2Ctls.dns.dnspod%2Chttp.awslambda%2Ctls.dns.rfc2136%2Cdyndns%2Chttp.git%2Chttp.authz%2Chttp.login%2Chttp.cgi%2Ctls.dns.digitalocean%2Chttp.gopkg%2Ctls.dns.dyn%2Ctls.dns.powerdns%2Chttp.jwt%2Chttp.filter%2Chttp.proxyprotocol%2Chttp.ratelimit%2Chttp.cache%2Chttp.prometheus%2Ctls.dns.rackspace%2Chttp.ipfilter%2Ctls.dns.ovh%2Chttp.reauth%2Chttp.filemanager%2Chttp.forwardproxy'
provisioning build environment: git fetch {github.com/caddyserver/dnsproviders/cloudxns v0.1.2 https://github.com/caddyserver/dnsproviders.git}: fork/exec /usr/bin/git: no such file or directory

It does not matter which architecture you use, i.e. amd64 and arm builds fail with them same message.

A link to the broken CI run where I build caddy for the AUR package caddy-full-bin.

It seems some plugins are currently incompatible with this version of Caddy; I'll reach out to the devs and see if I can get them to release an update, since I think most of them have already pushed the necessary changes. They just need to go to the site. Thanks!

Unfortunately, it is still broken https://circleci.com/gh/klingtnet/caddy-release-downloader/71. Do you have any idea which plugin could be the problematic one?

I would if I ssh'ed in to check the logs. However I am currently mobile with limited connectivity so I am not able to do that at the moment, sorry. I'll try to do so as soon as I'm back on my home continent!

Any news on this? The problem still persits.

It's the DNS plugins. I had the same issue.

@geek1011 Thank you very much, I can confirm that the dns plugin (server type) breaks the build.

@geek1011 No problem! It took me quite a while to test everything. Maybe if @mholt made the build server automatically test a build with each plugin and showed something saying it's broken on the download page?

I have to omit all plugins starting with tls.dns and the dns plugin to build caddy.

It only fails if the dns plugin is selected, as far as I can tell. @miekg just has to do a new release sometime.

[ Quoting<EMAIL_ADDRESS>in "Re: [mholt/caddy] Caddy 0.10.11 bui..." ]

It only fails if the dns plugin is selected, as far as I can tell. @miekg just has to do a new release sometime.

I tried, it failed :-( I gave up after that.

I would love to help you fix it, rather than just leaving it broken. :)

Hmm. I did some more testing and found something unusual: If you select  tls.dns.rfc2136 along with either tls.dns.route53 or tls.dns.vultr, the build breaks. Maybe conflicting vendored dependencies?

I will look into it again by checking the logs after the Caddy release tomorrow. (Plugins can be released whenever.)


HDDS-1260. Create Recon Server lifecycle integration with Ozone.

Create the lifecycle scripts (start/stop) for Recon Server along with Shell interface like the other components.
Verify configurations are being picked up by Recon Server on startup.

@avijayanhwx @bharatviswa504 @swagle Please review when you find time

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
27
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

0
mvndep
59
Maven dependency ordering for branch

+1
mvninstall
996
trunk passed

+1
compile
107
trunk passed

+1
checkstyle
35
trunk passed

+1
mvnsite
150
trunk passed

+1
shadedclient
667
branch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
141
trunk passed

+1
javadoc
119
trunk passed

_ Patch Compile Tests _

0
mvndep
16
Maven dependency ordering for patch

-1
mvninstall
21
dist in the patch failed.

+1
compile
96
the patch passed

+1
javac
96
the patch passed

+1
checkstyle
25
the patch passed

+1
mvnsite
112
the patch passed

+1
shellcheck
25
There were no new shellcheck issues.

+1
shelldocs
18
There were no new shelldocs issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
4
The patch has no ill-formed XML file.

+1
shadedclient
762
patch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
160
the patch passed

+1
javadoc
104
the patch passed

_ Other Tests _

+1
unit
36
common in the patch passed.

+1
unit
45
ozone-manager in the patch passed.

+1
unit
36
ozone-recon in the patch passed.

+1
unit
25
dist in the patch passed.

+1
asflicense
33
The patch does not generate ASF License warnings.

4087

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-643/2/artifact/out/Dockerfile

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

Optional Tests
dupname  asflicense  mvnsite  unit  shellcheck  shelldocs  compile  javac  javadoc  mvninstall  shadedclient  xml  findbugs  checkstyle

uname
Linux 703a0a4ae7c5 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / c99b107

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

shellcheck
v0.4.6

findbugs
v3.1.0-RC1

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/2/artifact/out/patch-mvninstall-hadoop-ozone_dist.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/2/testReport/

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

modules
C: hadoop-ozone/common hadoop-ozone/ozone-manager hadoop-ozone/ozone-recon hadoop-ozone/dist U: hadoop-ozone

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/2/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
28
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

0
mvndep
63
Maven dependency ordering for branch

+1
mvninstall
1031
trunk passed

+1
compile
121
trunk passed

+1
checkstyle
27
trunk passed

+1
mvnsite
132
trunk passed

+1
shadedclient
639
branch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
134
trunk passed

+1
javadoc
95
trunk passed

_ Patch Compile Tests _

0
mvndep
13
Maven dependency ordering for patch

-1
mvninstall
18
dist in the patch failed.

+1
compile
92
the patch passed

+1
javac
92
the patch passed

+1
checkstyle
23
the patch passed

+1
mvnsite
98
the patch passed

+1
shellcheck
24
There were no new shellcheck issues.

+1
shelldocs
15
There were no new shelldocs issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
4
The patch has no ill-formed XML file.

+1
shadedclient
719
patch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
155
the patch passed

+1
javadoc
91
the patch passed

_ Other Tests _

+1
unit
36
common in the patch passed.

-1
unit
43
ozone-manager in the patch failed.

+1
unit
34
ozone-recon in the patch passed.

+1
unit
21
dist in the patch passed.

+1
asflicense
28
The patch does not generate ASF License warnings.

3921

Reason
Tests

Failed junit tests
hadoop.ozone.security.TestOzoneDelegationTokenSecretManager

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-643/3/artifact/out/Dockerfile

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

Optional Tests
dupname  asflicense  mvnsite  unit  shellcheck  shelldocs  compile  javac  javadoc  mvninstall  shadedclient  xml  findbugs  checkstyle

uname
Linux 65b968f0f31e 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / c99b107

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

shellcheck
v0.4.6

findbugs
v3.1.0-RC1

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/3/artifact/out/patch-mvninstall-hadoop-ozone_dist.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/3/artifact/out/patch-unit-hadoop-ozone_ozone-manager.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/3/testReport/

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

modules
C: hadoop-ozone/common hadoop-ozone/ozone-manager hadoop-ozone/ozone-recon hadoop-ozone/dist U: hadoop-ozone

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/3/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
511
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

0
mvndep
56
Maven dependency ordering for branch

+1
mvninstall
1106
trunk passed

+1
compile
111
trunk passed

+1
checkstyle
29
trunk passed

+1
mvnsite
136
trunk passed

+1
shadedclient
623
branch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
151
trunk passed

+1
javadoc
103
trunk passed

_ Patch Compile Tests _

0
mvndep
13
Maven dependency ordering for patch

-1
mvninstall
21
dist in the patch failed.

+1
compile
101
the patch passed

+1
javac
101
the patch passed

+1
checkstyle
26
the patch passed

+1
mvnsite
106
the patch passed

+1
shellcheck
27
There were no new shellcheck issues.

+1
shelldocs
12
There were no new shelldocs issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
4
The patch has no ill-formed XML file.

+1
shadedclient
742
patch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
162
the patch passed

+1
javadoc
86
the patch passed

_ Other Tests _

+1
unit
36
common in the patch passed.

+1
unit
44
ozone-manager in the patch passed.

+1
unit
34
ozone-recon in the patch passed.

+1
unit
21
dist in the patch passed.

+1
asflicense
31
The patch does not generate ASF License warnings.

4539

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-643/1/artifact/out/Dockerfile

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

Optional Tests
dupname  asflicense  mvnsite  unit  shellcheck  shelldocs  compile  javac  javadoc  mvninstall  shadedclient  xml  findbugs  checkstyle

uname
Linux 00d8f7473f37 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / c99b107

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

shellcheck
v0.4.6

findbugs
v3.1.0-RC1

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/1/artifact/out/patch-mvninstall-hadoop-ozone_dist.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/1/testReport/

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

modules
C: hadoop-ozone/common hadoop-ozone/ozone-manager hadoop-ozone/ozone-recon hadoop-ozone/dist U: hadoop-ozone

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
25
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

0
mvndep
31
Maven dependency ordering for branch

+1
mvninstall
1066
trunk passed

+1
compile
104
trunk passed

+1
checkstyle
29
trunk passed

-1
mvnsite
28
ozone-manager in trunk failed.

+1
shadedclient
648
branch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

-1
findbugs
27
ozone-manager in trunk failed.

+1
javadoc
93
trunk passed

_ Patch Compile Tests _

0
mvndep
12
Maven dependency ordering for patch

-1
mvninstall
19
dist in the patch failed.

+1
compile
99
the patch passed

+1
javac
99
the patch passed

+1
checkstyle
24
the patch passed

+1
mvnsite
102
the patch passed

+1
shellcheck
25
There were no new shellcheck issues.

+1
shelldocs
13
There were no new shelldocs issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
5
The patch has no ill-formed XML file.

+1
shadedclient
739
patch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
156
the patch passed

+1
javadoc
86
the patch passed

_ Other Tests _

+1
unit
35
common in the patch passed.

+1
unit
40
ozone-manager in the patch passed.

+1
unit
37
ozone-recon in the patch passed.

+1
unit
21
dist in the patch passed.

+1
asflicense
27
The patch does not generate ASF License warnings.

3921

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-643/4/artifact/out/Dockerfile

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

Optional Tests
dupname  asflicense  mvnsite  unit  shellcheck  shelldocs  compile  javac  javadoc  mvninstall  shadedclient  xml  findbugs  checkstyle

uname
Linux 1a4c3caa7d66 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / c99b107

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

mvnsite
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/4/artifact/out/branch-mvnsite-hadoop-ozone_ozone-manager.txt

shellcheck
v0.4.6

findbugs
v3.1.0-RC1

findbugs
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/4/artifact/out/branch-findbugs-hadoop-ozone_ozone-manager.txt

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/4/artifact/out/patch-mvninstall-hadoop-ozone_dist.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/4/testReport/

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

modules
C: hadoop-ozone/common hadoop-ozone/ozone-manager hadoop-ozone/ozone-recon hadoop-ozone/dist U: hadoop-ozone

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/4/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
27
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

0
mvndep
65
Maven dependency ordering for branch

+1
mvninstall
1065
trunk passed

+1
compile
109
trunk passed

+1
checkstyle
31
trunk passed

+1
mvnsite
133
trunk passed

+1
shadedclient
633
branch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
146
trunk passed

+1
javadoc
107
trunk passed

_ Patch Compile Tests _

0
mvndep
14
Maven dependency ordering for patch

-1
mvninstall
23
dist in the patch failed.

+1
compile
111
the patch passed

+1
javac
111
the patch passed

+1
checkstyle
25
the patch passed

+1
mvnsite
111
the patch passed

+1
shellcheck
27
There were no new shellcheck issues.

+1
shelldocs
13
There were no new shelldocs issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
3
The patch has no ill-formed XML file.

+1
shadedclient
728
patch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
162
the patch passed

+1
javadoc
108
the patch passed

_ Other Tests _

+1
unit
39
common in the patch passed.

+1
unit
45
ozone-manager in the patch passed.

+1
unit
37
ozone-recon in the patch passed.

+1
unit
26
dist in the patch passed.

+1
asflicense
29
The patch does not generate ASF License warnings.

4090

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-643/5/artifact/out/Dockerfile

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

Optional Tests
dupname  asflicense  mvnsite  unit  shellcheck  shelldocs  compile  javac  javadoc  mvninstall  shadedclient  xml  findbugs  checkstyle

uname
Linux 048df944be83 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / c99b107

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

shellcheck
v0.4.6

findbugs
v3.1.0-RC1

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/5/artifact/out/patch-mvninstall-hadoop-ozone_dist.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/5/testReport/

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

modules
C: hadoop-ozone/common hadoop-ozone/ozone-manager hadoop-ozone/ozone-recon hadoop-ozone/dist U: hadoop-ozone

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/5/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
511
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

0
mvndep
62
Maven dependency ordering for branch

+1
mvninstall
1081
trunk passed

+1
compile
103
trunk passed

+1
checkstyle
26
trunk passed

+1
mvnsite
130
trunk passed

-1
shadedclient
1044
branch has errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
145
trunk passed

+1
javadoc
105
trunk passed

_ Patch Compile Tests _

0
mvndep
16
Maven dependency ordering for patch

-1
mvninstall
22
dist in the patch failed.

+1
compile
96
the patch passed

+1
javac
96
the patch passed

+1
checkstyle
24
the patch passed

+1
mvnsite
107
the patch passed

+1
shellcheck
24
There were no new shellcheck issues.

+1
shelldocs
15
There were no new shelldocs issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
4
The patch has no ill-formed XML file.

+1
shadedclient
744
patch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
159
the patch passed

+1
javadoc
105
the patch passed

_ Other Tests _

+1
unit
38
common in the patch passed.

+1
unit
46
ozone-manager in the patch passed.

+1
unit
35
ozone-recon in the patch passed.

+1
unit
24
dist in the patch passed.

+1
asflicense
32
The patch does not generate ASF License warnings.

4961

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-643/6/artifact/out/Dockerfile

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

Optional Tests
dupname  asflicense  mvnsite  unit  shellcheck  shelldocs  compile  javac  javadoc  mvninstall  shadedclient  xml  findbugs  checkstyle

uname
Linux c580f94eed50 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / c99b107

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

shellcheck
v0.4.6

findbugs
v3.1.0-RC1

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/6/artifact/out/patch-mvninstall-hadoop-ozone_dist.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/6/testReport/

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

modules
C: hadoop-ozone/common hadoop-ozone/ozone-manager hadoop-ozone/ozone-recon hadoop-ozone/dist U: hadoop-ozone

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/6/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

LGTM +1. We can build a proper UI to serve up container-key mapping in HDDS-1335

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
0
Docker mode activated.

-1
patch
7
https://github.com/apache/hadoop/pull/643 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.

Subsystem
Report/Notes

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

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/7/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
27
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

0
mvndep
47
Maven dependency ordering for branch

+1
mvninstall
970
trunk passed

+1
compile
103
trunk passed

+1
checkstyle
26
trunk passed

+1
mvnsite
95
trunk passed

+1
shadedclient
625
branch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
93
trunk passed

+1
javadoc
77
trunk passed

_ Patch Compile Tests _

0
mvndep
15
Maven dependency ordering for patch

-1
mvninstall
18
dist in the patch failed.

+1
compile
90
the patch passed

+1
javac
90
the patch passed

+1
checkstyle
22
the patch passed

+1
mvnsite
72
the patch passed

+1
shellcheck
24
There were no new shellcheck issues.

+1
shelldocs
11
There were no new shelldocs issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
2
The patch has no ill-formed XML file.

+1
shadedclient
695
patch has no errors when building and testing our client artifacts.

0
findbugs
0
Skipped patched modules with no Java source: hadoop-ozone/dist

+1
findbugs
106
the patch passed

+1
javadoc
68
the patch passed

_ Other Tests _

+1
unit
34
common in the patch passed.

+1
unit
32
ozone-recon in the patch passed.

+1
unit
20
dist in the patch passed.

+1
asflicense
29
The patch does not generate ASF License warnings.

3521

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-643/8/artifact/out/Dockerfile

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

Optional Tests
dupname  asflicense  mvnsite  unit  shellcheck  shelldocs  compile  javac  javadoc  mvninstall  shadedclient  xml  findbugs  checkstyle

uname
Linux 69eb151f6f53 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 56f1e13

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

shellcheck
v0.4.6

findbugs
v3.1.0-RC1

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/8/artifact/out/patch-mvninstall-hadoop-ozone_dist.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/8/testReport/

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

modules
C: hadoop-ozone/common hadoop-ozone/ozone-recon hadoop-ozone/dist U: hadoop-ozone

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-643/8/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

Robot tests will be added later as part of - https://issues.apache.org/jira/browse/HDDS-1261
Tested build manually and it succeeds.


Use "prerelease" GHA workflow with quoting fixes

Use my balena-yocto-scripts branch that includes some fixes.

LGTM


How to remove darkmode toggle and side panel

Any ideas how to remove the side panel and dark mode toggle?

You can add custom CSS (see docs) with display: none; for these elements.


Service stuck in 'Stopping' state when HostControl.Stop is called

I have a TopShelf (3.1.3) service that hangs in a 'Stopping' state when HostControl.Stop is invoked.
What is the recommended approach for killing a service if something fatal occurs? I do not wish to simply swallow/log the exception and continue, nor do I want my processing thread to die without killing the service. I would like for the service to die, and ideally for the service recovery actions to be run.
Thoughts?
HostFactory.Run(o =>
{
    o.Service<TaskRunner>();
    o.RunAsLocalSystem();
    o.SetServiceName("MyService");
    o.SetDisplayName("MyService");
    o.SetDescription("MyService");
    o.EnableServiceRecovery(r => r.RunProgram(1, "notepad.exe"));
});

public class TaskRunner : ServiceControl
{
    private CancellationTokenSource cancellationTokenSource;
    private Task mainTask;

    public bool Start(HostControl hostControl)
    {
        var cancellationToken = cancellationTokenSource.Token;
        this.mainTask = Task.Factory.StartNew(() =>
            {
                try
                {
                    while (!this.cancellationTokenSource.IsCancellationRequested)
                    {
                        // ... service logic ...
                        throw new Exception("oops!");
                    }
                }
                catch (Exception)
                {
                    hostControl.Stop();
                }
            });
        return true;
    }

    public bool Stop(HostControl control)
    {
        this.cancellationTokenSource.Cancel();
        this.mainTask.Wait();
        return true;
    }
}

Return false or just throw the exception out so that Topshelf will recognize that your service didn't start and handle it appropriately. By catching the exception yourself, Topshelf is unable to detect that your service failed to start. So, let it throw.


Gauge server is crashing when in step definition > or < is missing in variable name

Getting error "The Gauge server crashed 5 times in the last 3 minutes." if in step definition > or < is missing in variable name.
example:
[Step("Append two strings <Text1> and <Text2 ")]
Expected: It should provide the error information so that user can correct the mistake.
Gauge (Output of gauge -v)
Gauge version: 1.1.1
Commit Hash: 28617ea
Plugins
dotnet (0.1.4)
Operating System information
Windows 10
IDE information
VScode

The plugin did not crash for me, instead it gave an error stating step implementation missing and in the implementation it showed 0 references. Please do check in the lsp.logs to see if there is any information regarding the crash.

Closing due to inactivity.


[windows][USM] converge etw consumer

What does this PR do?
Converges the USM code to use the more general ETW component, instead of its own standalone implementation.
Motivation
Not having duplicated code.
Additional Notes
Possible Drawbacks / Trade-offs
Describe how to test/QA your changes
There are automated tests.
However, install agent with USM enabled.  Generate traffic to IIS site(s).  Ensure those appear as services on back-end.
Reviewer's Checklist

[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] Changed code has automated tests for its functionality.
[x] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[x] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

p.s., you didn't set qa/done tag nor described qa steps in the description

/merge


Questions about Isolate Groups and "move" operator

(Context: I've learned a bit about parallelism in general and in Dart but don't have much hands-on experience)
isolate_agents looks like an awesome implementation of the Actor pattern! Thanks for implementing and sharing it, especially the nice write up and example use-cases in your Medium post and this repo.
I had a few thoughts / questions:

You mentioned that controlling the lifecycle of isolates is a benefit of isolate_agents compared to Flutter's .compute(). How do Agents relate to isolate groups? I remember hearing exciting stuff ~18 months ago about how much isolate groups would reduce the cost of spinning up new isolates. With Agents, would you create a pool of long-running agents and send work to them?

I don't even know enough about this to know what I'm asking, but it seems like there could be some interesting interactions between agents and isolate groups.

Another performance consideration you mentioned is the need for a "move" operator to allow constant time communication between isolates. Again I'm not familiar with how this all works, but I thought you might be interested in this Rust crate that generates FFI bindings and allows "zero copy returns over the FFI boundary". It would be awesome to use Dart everywhere (with agents?) instead of Rust for high-performance parallelism. Just thought I'd share in case that approach is of interest. (I love how you took inspiration from Clojure for isolate_agents)

Thanks again!

Thanks Kyle.

You mentioned that controlling the lifecycle of isolates is a benefit of isolate_agents compared to Flutter's .compute(). How do Agents relate to isolate groups? I remember hearing exciting stuff ~18 months ago about how much https://github.com/dart-lang/sdk/issues/46754 would reduce the cost of spinning up new isolates. With Agents, would you create a pool of long-running agents and send work to them?

When running on a version of Dart with isolate groups, agents will be spawned in the isolate group.  Isolate groups make the cost of spinning up isolates much cheaper, but they still aren't free.  With agents you have the ability to create long running agents if you want.  Unfortunately it's a complicated question to answer if that is the best way to do things because of the optimization that happens when you kill an isolate.  It does constant time returning values in that case.  So if you are returning a large value from an isolate it might actually be cheaper to spawn and kill a new isolate for each calculation (i.e. time_to_spin_up_isolate - time_to_transmit_output < 0).

Another performance consideration you mentioned is the need for a "move" operator to allow constant time communication between isolates. Again I'm not familiar with how this all works, but I thought you might be interested in this Rust crate that generates FFI bindings and allows "zero copy returns over the FFI boundary". It would be awesome to use Dart everywhere (with agents?) instead of Rust for high-performance parallelism. Just thought I'd share in case that approach is of interest. (I love how you took inspiration from Clojure for isolate_agents)

Thanks for sharing Membrane.  I dig it.  You can simulate constant time communication if the data you are transmitting is held in a buffer outside of the Dart VM.  However it isn't really constant-time since it requires extra work reading and writing from that buffer since it isn't in the dart heap in the memory format of dart objects.


[CCI] Create advanced index actions guide

Description

[x] Create advanced index actions guide

Issues Resolved

closes #470

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] Linter check was successfull - yarn run lint doesn't show any errors
[x] Commits are signed per the DCO using --signoff
[ ] Changelog was updated.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@Nicksqain Would you mind refreshing your fork then rebase this commit to see if the link-checker error is resolved afterward?

@nhtruong I have updated my fork and the commit is up to date. Can I make a micro-change and do a push?


🐞 - <svg> inside tui-svg not displayed on iOS/macOS devices

Playground Link
https://stackblitz.com/edit/angular-p1ky3v-xb1md3?file=src%2Fapp%2Fapp.component.html
Description
Some svg-s are not visible, while there's no problem with others. I pinned different examples in the playground.
<svg>'s parent .t-src is a flex-container. If you change it to display: block, svg is displayed (with broken layout, obviously).
Setting <svg>'s min-width/width to some value (e.g. 100%) also helps, but I doubt it's a universal solution.
Angular version
16.2.7
Taiga UI version
3.59.0
Which browsers have you used?

[X] Chrome
[ ] Firefox
[X] Safari
[ ] Edge

Which operating systems have you used?

[X] macOS
[ ] Windows
[ ] Linux
[X] iOS
[ ] Android

Unfortunately that's just Safari being Safari:
https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/width#svg
Specs say missing width/height are interpreted as auto which should be treated as 100% which other browsers properly implement, but Safari does not. You need to add explicit width="100% height="100%" to your SVG source if you want them to be scalable, or a fixed size of you want fixed size.


Adding proof in schema doc

We shold attach a proof in the schema,
https://w3c-ccg.github.io/vc-json-schemas/v1/index.html#credential_schema_definition_proof
{
  "type": "https://w3c-ccg.github.io/vc-json-schemas/schema/1.0/schema.json",
  "modelVersion": "1.0",
  "id": "did:work:MDP8AsFhHzhwUvGNuYkX7T;id=06e126d1-fa44-4882-a243-1e326fbe21db;version=1.0",
  "name": "EmailCredentialSchema",
  "author": "did:work:MDP8AsFhHzhwUvGNuYkX7T",
  "authored": "2018-01-01T00:00:00+00:00",
  "schema": {
    "$schema": "http://json-schema.org/draft-07/schema#",
    "description": "Email",
    "type": "object",
    "properties": {
      "emailAddress": {
        "type": "string",
        "format": "email"
      }
    },
    "required": ["emailAddress"],
    "additionalProperties": false
  },
  "proof": {
      "created": "2019-09-27T06:26:11Z",
      "creator": "did:work:MDP8AsFhHzhwUvGNuYkX7T#key-1",
      "nonce": "0efba23d-2987-4441-998e-23a9d9af79f0",
      "signatureValue": "2A7ZF9f9TWMdtgn57Y6dP6RQGs52xg2QdjUESZUuf4J9BUnwwWFNL8vFshQAEQF6ZFBXjYLYNU4hzXNKc3R6y6re",
      "type": "Ed25519VerificationKey2018"
  }
}

only problem is nonce /  challenge. like who will give the challenge to the creator of scehma.  the node should ideally give and the creator should sign that challenge and the node should verify
but for that we need to write a rpc to create challenge ...
not if that is the right way

Add checks for authored and created dates


Any reason why branch 'filtering-improvements' was deleted?

The branch 'filtering-improvements' was deleted.
any reason why? (there was no merge etc)
It fixed some bugs that I was seeing in the main branch.

@nextpulse it was not deleted. It waits for merging PR - https://github.com/antillas21/ajax-datatables-rails/pull/123

@ajahongir the branch is not there and there are no obvious paths to pull this. Any advice?
prev:
gem 'ajax-datatables-rails', github: 'ajahongir/ajax-datatables-rails', branch: 'filtering-improvements'

here is branch - https://github.com/ajahongir/ajax-datatables-rails/tree/filtering-improvements

The filtering-improvements PR #123 has already been merged to the corresponding v-0-4-0 branch of this gem.


evaluation copy of Windows 10 Enterprise download link redirected

I use the "evaluation copy of Windows 10 Enterprise" to download enterprise evaluation image but failed, this link seems redirected.
https://www.microsoft.com/en-us/evalcenter/evaluate-windows-10-enterprise

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a1e7dc7e-3df6-2eaf-9c11-ed03c00719e0
Version Independent ID: ab5e43cf-c4db-cd8f-e92a-c33e4ebc2b68
Content: Overview of Windows 10 IoT Enterprise - Windows IoT
Content Source: windows-iotcore/windows-iot-enterprise.md
Product: windows-iot
GitHub Login: @saraclay
Microsoft Alias: saclayt

HI @luyaok, we are looking into this.

Hello again - the evaluation center would be putting back up the RS4 and 2016 LTSB bits soon (latest tomorrow). To learn more about the issue, you can read the blog here: https://blogs.windows.com/windowsexperience/2018/10/09/updated-version-of-windows-10-october-2018-update-released-to-windows-insiders/#KyvjHp6XBmGFdVrd.97
If you find that you still cannot get any bits after tomorrow, feel free to open a new issue.


Extracting marshaling code

Hey there, we're working with this library for @keep-network. @pdyraga submitted #2 earlier this week, and we're interested in continuing to upstream further changes (if any); however, one of them is that we don't need the marshaling/unmarshaling code for BSON and JSON (we may need protobuf). We're thinking of refactoring the marshaling so that it can live in its own package and consumers can import the marshaling they need. Is that a change you'd be interested in pulling back into this repo?

Totally, as a stand alone crypto package, the marshalling/unmarshalling
should be a separate issue.  Json might be ok to keep as it does not
require external dependancies but bson is a very odd dependency.  I would
totally accept any pull request in that direction.
Didier
On Thu, Jun 7, 2018, 10:18 AM Antonio Salazar Cardozo <
<EMAIL_ADDRESS>wrote:

Hey there, we're working with this library for @keep-network
https://github.com/keep-network. @pdyraga https://github.com/pdyraga
submitted #2 https://github.com/didiercrunch/paillier/pull/2 earlier
this week, and we're interested in continuing to upstream further changes
(if any); however, one of them is that we don't need the
marshaling/unmarshaling code for BSON and JSON (we may need protobuf).
We're thinking of refactoring the marshaling so that it can live in its own
package and consumers can import the marshaling they need. Is that a change
you'd be interested in pulling back into this repo?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/didiercrunch/paillier/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/AC4UlJvW80vMdRIgVQSvCzjSkdAoXVb6ks5t6TYjgaJpZM4UegCD
.

Good to hear! Hope to offer up a PR soon :)

#4 closed this!


[Bug]: Auto Attendant: Call Routing: text label cut off

What happened?
When selecting redirect: voicemail as call routing option the text label Call routing voice... is being seen which makes it hard to understand for users what this option does

What is the version of the App?
V1.0
What were the steps you followed?

Edit an Auto Attendant
Set the Call routing Options to redirect: voicemail
Text label is cut off

Area in which the issue occurred
PowerApp
Relevant log output
No response

Same happens for the call routing after business hours

Fixed as part of GUI design


Create a GitHub Action

Create a github action that people can use in their workflows that automatically installs wap. Or, more complexly, builds and publishes their package.

Completed, see docs at https://t-mart.github.io/wap/gh-action/


serializeIndex does not properly update nested array field names

When cloning an element like this with serializeIndex: true:

    <input name="first[0][second][0][field]">
    <input name="first[0][second][1][field]">

I would expect only the first-level array key to be updated to the new sequence - eg:

    <input name="first[0][second][0][field]">
    <input name="first[0][second][1][field]">

    <input name="first[1][second][0][field]">
    <input name="first[1][second][1][field]">

However, cloneya resets all the numeric keys in a field name in both the toClone and newClone elements - so the output is actually:

    <input name="first[0][second][0][field]">
    <input name="first[0][second][0][field]">

    <input name="first[1][second][1][field]">
    <input name="first[1][second][1][field]">

This then produces duplicate field names in each block, so the submitted data is missing expected values.
Also reproduced in this jsFiddle.
I appreciate this probably isn't as simple as just forcing it to only change the first key, since potentially in a nested set of clones you might actually want to end up with fields like:
<input name="person[0][telephone][0]">
<input name="person[0][telephone][1]">
<input name="person[1][telephone][0]">
<input name="person[1][telephone][1]">
<input name="person[1][telephone][2]">

where both the "person" block and the individual "telephone" rows could be cloned - so you'd have to track the depth of the clone and work out which index to change each time.
However, as-is the behaviour is very unexpected and doesn't really work for any usecase - so we should either try and fix it or at least strongly document that multi-level array field names are not supported.

One possible solution for all usecases might be to define a nameIndexPattern or similar option to define the regular expression that will be used to find indexes to replace (in place of the hardcoded /\[([^}]+)\]/)[https://github.com/yapapaya/jquery-cloneya/blob/stable/dist/jquery-cloneya.js#L418].
We could then do things like:
$('.clone-wrapper').cloneya({serializeIndex: true, nameIndexPattern: /^first\[(\d+)\]/});
$('.person-blocks).cloneya({serializeIndex: true, nameIndexPattern: /^person\[(\d+)\]/});
$('.nested-telephone-rows).cloneya({serializeIndex: true, nameIndexPattern: /^person\[\d+\]\[telephone\]\[(\d+)\]/});

or basically whatever pattern was appropriate for the usecase. Cloneya would continue to replace the content of any numeric capture group found by the regex.
The default could either be the current pattern, or a slightly safer pattern that only matches the first numeric key in a name.
If you think something like this would be OK I can work up a PR with tests.

@yapapaya bump - what do you think? Should I try to fix this with a namePattern option instead of the hardcoded one?

@acoulton I actually wanted to keep this open to plugging by the dev. So, the idea was to either to not support it at all and let the dev figure it out or handle just the first level.
The ideal thing to do is to set it to false and hook onto after_append.cloneya that is triggered just after the _redoIDs function is run to do things specific to one's use case. It'll be impossible to handle all possible use-cases. That is why I haven't documented this one and would possibly remove it altogether!

You can add new param 'nthMatch' for options and add function like this one: http://stackoverflow.com/questions/36183/replacing-the-nth-instance-of-a-regex-match-in-javascript#answer-7958627
for param what have to be increment.
if($this.config.nthMatch){
									name = replaceNthMatch(name, /(\d+)/, $this.config.nthMatch, function(val){
									  return i;
									});
								}else{

									name = [].map.call(st, function (s, n) {
										return (!isNaN(+s) && st[n - 1] === '[' && st[n + 1] === ']') ? i : s;
									}).join('');

								};

Was there any solutions for this?!  I'm having the same problem that @acoulton
Any ideas?!

I've just wrote own js/jquery script (no plugin) for cakephp+bootstrap structure - to dynamic change field ('name', 'id') and label ('for') - without cloneya.

See this related issue: https://github.com/yapapaya/jquery-cloneya/issues/53
I still think Indexing is not something that a generic plugin like cloneya can do. Each case is going to be different from the other. I'd rather have developers use the after_clone and before_append events to figure out how to do the indexing for form elements.

See event documentation: https://github.com/yapapaya/jquery-cloneya/wiki/Events#after_clone
at each clone event, one could easily hook in and manage the indexing on their own.


Route A - A with space dash space in the name

Same as #233.

Frontenex - Voie ferrée - Frontenex - Voie ferrée

Raised 4 facts:

With the implementation of release 4.0, I believe this situation is handled correctly now.

I still have an issue with this one, please have a look: https://knooppuntnet.nl/fr/analysis/route/13542311
For other cases the issue is fixed: https://knooppuntnet.nl/fr/analysis/route/12264240
Thanks for your kind help

In La Chapelle Saint-Pierre - La Chapelle Saint-Pierre the node name contain only - (dash)😉
In Les Choseaux - Verneys - Les Choseaux - Verneys the node name contain - (space dash space)

RouteNameAnalyzerTest.scala should also test "start - node - EndNode"
    val routeNameAnalysis = analyzeRouteName(Tags.from("name" -> "start - end-node"))

Issue is there ?
https://github.com/vmarc/knooppuntnet/blob/86a48b43de2ff12f99a7ae11d28c93c38fe3ebba/server/src/main/scala/kpn/server/analyzer/engine/analysis/route/analyzers/RouteNameAnalyzer.scala#L124-L142

Note: route Frontenex - Voie ferrée - Frontenex - Voie ferrée don't raise an error because the dash wash removed from the Node name.
See version 4

Note: In France, city name are "normalised".

Saint-Pierre is a city (Saint Peter's)
Saint - Pierre could be a route from Saint to Pierre or a place called Saint - Pierre
Saint-Maurice-Crillat is the merging of Saint-Maurice and Crillat
Saint-Maurice-Crillat - Saint-Maurice-Crillat could be a loop around this village (the case of this issue)


Spamming most messages in an hour record

Is there an existing issue for this?

[X] I have searched the existing issues and none does.

Describe the bug
Scradd counts itself in the most messages in an hour record and then it spams #general when the record is broken because of it
To reproduce

Break the record for most messages in an hour

Expected behavior
Scradd should have sent the message one time.
Relevant log output
No response
Anything else?
No response
Agree to our policies

[X] I read, understand, and agree to follow the Code of Conduct.
[X] I read, understand, and agree to follow the Contributing Guidelines.
[ ] I'm willing to submit a PR to fix this (optional).

Records have already been removed in main

aw man

Sure they're cool and all, but they lead to issues like this, as well as tons of extra DBs for tracking them, and massive code.


MH-12976, custom role patterns not working

The new chosen library we use introduced line breaks to chosen-type generated elements. These line breaks pick up as a textNodes; the DOM selection query in the custom function getMoreRoles thus returns the textNode instead of the input element.
The fix simply updates the DOM selection query to search for an input element with class chosen-search-input. Note that, if the chosenjs maintainers remove the class chosen-search-input from the input, this change will need to be updated again.

Normally I wouldn't take this since I was starting work on it, but I haven't actually done anything with it yet so I'll review since I'm familiar with the code already :)
+1 for pushing upstream, will test this afternoon.

Ok, still seeing issues.  Well, two really:

The default build does not ship with enough roles showing up in the ACL list to trigger the text input.  If you're seeing the text box then you're not looking at strictly default roles.  I have a patch set for this here
Once you do have enough roles, this does't work right (at least for me).  The input allows you to type, once you type something that matches the text you just typed disappears, and the dropdown reappears with the text you typed as a new item at the bottom.  With enough roles this new item may not be visible on screen, and even if it is it's very confusing.  This leads to be user confusion (what happened?) and bad data.

STR for bad data:
Target role is abc123_Learner, custom pattern is ^[0-9a-f-]+_(Learner|Instructor)$

Type asdbc123_Learner
Move cursor to beginning of text, remove s
New role is entered as adbc123_Learner

IMO, if the role does not match an existing role, and matches the custom pattern it should replace the no search results found text with something like "hit enter to create a new role", or something.

Thanks for first pass.

I'll cherry-pick your commits from there (i.e. 451ac10)
yes, that's irritating in the least. Most (if not all) of the code for this specific functionality is largely unchanged on our side, so I suspect there may be further changes on the chosen lib side which need to be adapted for our use case.
Give me a day or two to go thru the angular-chosen plumbing to make it work (at least) as it was in 3.x*. Thereafter, I can look at implementing your idea for explicitly opting to add the external role.

* 3.x had it that the new role automatically popped up in the dropdown as soon as it passed the regex match + without the text in the input disappearing.

Hi @gregorydlogan
I added an earlier customisation that was skipped during the angular-chosen update. The fix restores the functionality to how it was in 3.x/4.x, where the textbox is not cleared on a successful custom pattern match.
There's a slight discrepancy though in that the newly entered custom role is not always highlighted, once the list of roles is updated as a result of a successful search.
To see this discrepancy, try:

Enter "abc_Learner" (this will be highlighted: expected).
Move cursor to beginning of text, and add a single "a" to make the text read as "aabc_Learner".
Although "aabc_Learner" is the text in the textbox, the element in the dropdown list with role "aabc_Learner" is not highlighted (unexpected behaviour).

I'll resolve this discrepancy with a follow-on commit.

Hi @gregorydlogan
The latest commit highlights the newly entered custom role (see my previous comment).
Would you still prefer confirmation text for new roles, i.e. "hit enter to use role [new role here]"?

IMO, if the role does not match an existing role, and matches the custom pattern it should replace the no search results found text with something like "hit enter to create a new role", or something.

@gregorydlogan I'm not sure if this is relevant, but note that there might be other role provider (e.g. for Sakai or Moodle) that provide non-existing roles. Here the "matches the custom pattern"-check is not possible. Why do we need that check anyway? Additionally, these role provider could potentially return multiple roles. E.g. given the input 123 the Moodle role provider returns both 123_Learner and 123_Instructor. If I recall correctly, the Sakai role provider does the same.

@mtneug you bring up good points, and I was unable to actually look at the result when I made my previous comment.  Having built it and actually tested now, the current implementation is acceptable, if a little jarring.
@slampunk I'm going to merge this as is, thanks for the work!


Flipping doesn't work properly

https://jsfiddle.net/0ugrmce5/5/

Version with latest build:
https://jsfiddle.net/crn8Lko4/2/


Support for S3 storage provided by non-AWS (FreeNAS/TrueNAS/etc)

I'm learning Rancher, and I noticed there's only local storage or S3 storage for backups (amongst other things). But S3 storage can only seemingly be configured to interface with AWS, but not other S3 storage servers.
I run FreeNAS/TrueNAS for my central storage in all regards, as my whole setup is 100% on-prem, and it can serve S3 storage endpoints. As such, I was hoping that support for S3 storage endpoints that are not just AWS could be added to Rancher. Perhaps just enable the option to define the target server and connection details, since the protocol is supposed to be compliant anyways.
Any chance we can get this added as a feature, and this request added to a milestone? I have no interest in using AWS services whatsoever.


Could we get more detailed update examples?

I would like to use the API to change and escalation_rules target from a schedule_reference to a user_reference.
I am getting all the Escalation Policies like so
services = list(session.iter_all('escalation_policies', params={'query': 'myquery'}))
How could I go through them and update the escalation_rules target?

Hi @ajmcateer,
The escalation_rules property is an array (translates to a list type in Python); each entry should be an object (dict) with another list of dicts in targets property. In general, array -> list and object -> dict when decoded from JSON.
https://developer.pagerduty.com/api-reference/b3A6Mjc0ODEyOA-update-an-escalation-policy
So, for example, the first target on level 2 of an escalation policy (represented by an item yielded from the iterator session.iter_all('escalation_policies' ...)) would be ep['escalation_rules'][1]['targets'][0]. Wetting this to a new dict representing a user reference, i.e. ep['escalation_rules'][1]['targets'][0] = {'type': 'user_reference', 'id': '{user_id}'}, and then using session.rput(ep, json=ep) would update it accordingly.
The client doesn't have any logic that pertains specifically to any given resource, i.e. abstraction for escalation policies, and so one must work directly with API schemas. For that reason I've not really written documentation that pertains only to a given API resource as that would be redundant (there's already documentation for our schemas in the API reference).


https://login.growserver.app:8080: login.growserver.app’s server IP address could not be found.

<EMAIL_ADDRESS>dev
rimraf dist && rimraf assets/cache && eslint ./src && tsc && vite build --logLevel silent && node -r dotenv/config dist/app.js

[8/25/2024, 6:16:48 PM - INFO    ] Please wait extracting cache.zip
[8/25/2024, 6:16:48 PM - INFO    ] Connecting Discord bot...
[8/25/2024, 6:16:48 PM - READY   ] Successfully extracting cache.zip
[8/25/2024, 6:16:48 PM - INFO    ] Loaded custom items
[8/25/2024, 6:16:49 PM - READY   ] Items data ready!
[8/25/2024, 6:16:49 PM - INFO    ] Fetching latest Growtopia Cache
[8/25/2024, 6:16:50 PM - READY   ] Starting web server on: http://<IP_ADDRESS>:80
[8/25/2024, 6:16:50 PM - READY   ] Starting login server on: https://login.growserver.app:8080
[8/25/2024, 6:16:50 PM - EVENT   ] Loaded "disconnect" events
[8/25/2024, 6:16:50 PM - EVENT   ] Loaded "connect" events
[8/25/2024, 6:16:50 PM - EVENT   ] Loaded "raw" events
src/structures/Webserver.ts && dist/structures/Webserver.js
                 httpServer.listen(80);   85   │     httpsServer.listen(443);   86   │     httpsServerLogin.listen(8080);
https://login.growserver.app:8080: login.growserver.app’s server IP address could not be found.
http://http://<IP_ADDRESS>:80: works fine
installation
installed mkcert
npm install
npm run mkcert:setup
npm run dv
what am I doing wrong ?

did you set the hosts file correctly?
<IP_ADDRESS> www.growtopia1.com
<IP_ADDRESS> www.growtopia2.com
<IP_ADDRESS> login.growserver.app # New login system for development purposes

Sorry about that I didn't understand this part, can you walk me through it please, appreciate the help.
(I am hosting it on linux)

I managed to do it, everything is working fine now, thank you.

Alright if you have any issue, freely open a new issue or reopen this issue


no build folder/please provide final js and css

The example index.html, for example, references the js and css in the build folder, which is no longer in the repo.
Firefox and Thunderbird addons need js and css locally, no CDN. So it would be good if both are provided in the repo ... you need to build them anyway to put them into the CDN, so please also maintain them here.
Yes, I could download them directly from the CDN, but the link given in getting-started.md, for example, is broken, probably outdated. And downloading from CDN also feels a bit silly given that you have the js and css and upload them to the CDNs.
So please include it in the repo as well.
many thanks,
And: THANKS for the good editor.
Klaus

the examples are not working because they need the build folder - at least 2 of them: index.html and "another", forgot which.

the examples are not working because they need the build folder - at least 2 of them: index.html and "another", forgot which.

What examples are we talking about? Can you provide a link?

this for example: the result box should be a jodit editor?
https://xdsoft.net/jodit/examples/sizes/fixed_height.html
also, download code from github and open
file:///E://jodit/examples/index.html
there is a textarea but no editor.


Search field and illustrated message tokens

Description
Added search field and illustrated message tokens
Related Issue

DNA-1247 New search field token
DNA-1248 New illustrated message tokens

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

GitHub release
npm package (@next-major dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 12.0.0 :tada:
The release is available on:

GitHub release
npm package (@next-major dist-tag)

Your semantic-release bot :package::rocket:


HTTP Connection Pipelining Support

Howdy Developers, I've got a quandary/question and wondering what is possible.
I'm developing an application for bulk uploading of large media files into Google Cloud Storage. However, my upload source is on the far end of a "long fat pipe" (high bandwidth, high latency). I am using this library's excellent support for simply writing objects into my bucket.
However, what I am finding is that I need to increase the concurrency of my upload workers to a very high amount in order to fully fill my bandwidth. Lots of time is wasted on many round-trips to establish connections.
In some manual testing of crafted upload requests, it seems that the API frontends support processing multiple requests through the same TLS/TCP/HTTP connection/transport. So, I'm trying to research how possible this would be using this library.
So far, I don't see any connection pooling logic or obvious ways to craft raw requests for use in other libraries.
There was some previous discussion in #218 about switching to using google-auth instead of oauth2client, which would enable the use of requests and urllib3's connection pooling.
I'm starting to wonder if I will need to switch my project to craft multipart requests manually and wrap them with google-auth's google.auth.transport.requests.AuthorizedSession requests wrapper.
Is there some other way to accomplish HTTP pipelining with this library?
Is there a better way to accomplish this?

You should look into using the newer google-cloud-storage library. The next release will use requests as the http transport and will have connection pooling by default.

Oh - and the shorter answer - this library is inextricably tied to httplib2. There's no way we can support connection pooling and pipelining without breaking all of our users here. If you must use this library, I'd suggest checking out httplib2shim.

Thank you for the lightning-quick reply @jonparrott
google-cloud-storage looks a lot more like what I want!
Unfortunately, there doesn't seem to be clear pipelining support inside of google-cloud-storage either.
For future potential searchers, this is pretty verbose and feels a bit ugly, but here's an example of how I forced this to work for me:
#!/usr/bin/env python
from email.mime.multipart import MIMEMultipart
from email.mime.nonmultipart import MIMENonMultipart
try:
    from email.generator import BytesGenerator
except ImportError:
    from email.generator import Generator as BytesGenerator
import json
from six import BytesIO
import google.auth
from google.oauth2 import service_account
from google.auth.transport.requests import AuthorizedSession

class _BytesGenerator(BytesGenerator):
    _write_lines = BytesGenerator.write

def upload_blob(session, bucket, name, data):
    metadata = {
        "name": name
    }
    multipart_message = MIMEMultipart('related')
    setattr(multipart_message, '_write_headers', lambda self: None)
    metadata_message = MIMENonMultipart('application', 'json')
    metadata_message.set_charset('UTF-8')
    metadata_message.replace_header('Content-Transfer-Encoding', 'binary')
    metadata_message.set_payload(json.dumps(metadata), 'UTF-8')
    multipart_message.attach(metadata_message)
    content_message = MIMENonMultipart('application', 'octet-stream')
    content_message.add_header('Content-Transfer-Encoding', 'binary')
    content_message.set_payload(data)
    multipart_message.attach(content_message)
    fp = BytesIO()
    g = _BytesGenerator(fp, mangle_from_=False)
    g.flatten(multipart_message, unixfrom=False)
    body = fp.getvalue()
    multipart_boundary = multipart_message.get_boundary()
    headers = {}
    headers['Content-Type'] = ('multipart/related; boundary="%s"') % multipart_boundary
    headers['Content-Length'] = str(len(body))
    params = {}
    params['uploadType'] = 'multipart'
    upload_endpoint = "https://www.googleapis.com/upload/storage/v1/b/{bucket}/o"
    upload_endpoint = upload_endpoint.format(bucket=bucket)
    response = session.post(upload_endpoint,
                            params=params,
                            headers=headers,
                            data=body)

credentials, project = google.auth.default()
authed_session = AuthorizedSession(credentials)
bucket = "mybucket"
with open('/dev/urandom', 'r') as fh:
    fake_data1 = fh.read(1024)
    fake_data2 = fh.read(1024)

upload_blob(authed_session, bucket, "testing/fake_data1", fake_data1)
upload_blob(authed_session, bucket, "testing/fake_data2", fake_data2)

When I do this, I can confirm with wireshark that this only uses a single connection via urllib3's connection pooling.


Investigate replacement of uWSGI for application server

Description/Context
The uWSGI project has been officially in maintenance mode for over a year now. In order to mitigate any risk from lack of support or feature development in a core part of our serving capability we need to identify and implement a viable replacement for this technology.
A common alternative to uWSGI is Gunicorn, but that is more limited in its capabilities (e.g. no serving of static assets). Alternatives to investigate:

gunicorn - https://gunicorn.org/
uvicorn - https://www.uvicorn.org/
Hypercorn - https://github.com/pgjones/hypercorn
Nginx Unit - https://unit.nginx.org/

Plan/Design
We want the replacement to be flexible and applicable to all of our current serving environments. It is also beneficial if we can use it for future applications without requiring substantial customization. Nginx Unit is the most akin to uWSGI in that it is able to serve multiple language runtimes and is not tightly coupled to Python.
After determining the likely candidate(s) we need to implement it in a test application to establish the necessary patterns for its use and itegration with our existing infrastructure.

Once we identify the target that we want to support, we will need to follow up with replacing uWSGI across our various applications. This includes (not an exhaustive list):

Open edX
MITx Online
xPRO
OCW Studio
Bootcamps
ODL Video


Create a command completer for LSP

Group all the commands together as arguments for the main command lsp. This approach prevents the editor's command bar from becoming overcrowded with individual commands, making it simpler for the user to access and utilize them.

I tested it, and it seems to be functioning well. The arguments used in some of the commands are correctly set but I will leave it to your experience regarding the plugin to confirm.

I don't like how you have moved the initialization stuff to the bottom of the file, IMO that should live at the top of the file as it's what happens first.

It generates runtime errors. The way gotoAction is implemented requires it to be defined before the plugin is loaded, as it is currently in a table. I believe you should handle this, as I prefer not to modify the code and risk making changes that you may not approve of.


New version: CamiXon v1.6.0

Registering package: CamiXon
Repository: https://github.com/walra356/CamiXon.jl
Created by: @walra356
Version: v1.6.0
Commit: 331d80426f6147cc3fb0bfcee72b315a8223b87a
Reviewed by: @walra356
Reference: https://github.com/walra356/CamiXon.jl/commit/331d80426f6147cc3fb0bfcee72b315a8223b87a#commitcomment-70034223
Description: A package for image analysis of backscattered light

The JuliaTagBot automerged CamiXon v.1.5.10 and v1.6.0. This correctly is reflected by the Package page on Github, the documentel but for an unkown reason the Pkg.update() sticks to v.1.5.9.I am not aware of changing any settings locally.
Can you advise me?


sort: possibility for deduplication

Hi I'm missing the possibility for sorting+dedup via std lib.
So I implemented a function DedupSorted(data Interface) (n int) for the sort package.
It rearranges different elements of data to the front of the Interface
preserving order and returns the number of different elements.
So for example if I want to sort+dedup a slice of strings:
strings :=[]string{"c", "a", "b", "a", "b"}
sort.Strings(strings)
n:=sort.DedupSorted(sort.StringSlice(strings))
strings=strings[:n]
fmt.Println(strings)

would give me
[a b c]

If there is already something like this I would be happy to know and I'm sorry for filing this issue.
Comments are welcome.
Marvin

A general advice: when you file an API addition issue, before
sending CL for review, please at least wait for some consensus
on the issue tracker.
As Burcu mentioned in the code review, I don't think this functions
belongs to the sort package. If we have a sortutil package, perhaps.

If we have a sortutil package, perhaps.

sortutil.Dedupe ;-)

Generally, it is better to propose new APIs on golang-dev mailing list first where more contributors are actively watching each discussion. We need to discuss this function's value with other contributors to reason why it should or should not be merged into the standard library and where it would fit.

Generally, it is better to propose new APIs on golang-dev mailing list first where more contributors are actively watching each discussion.

Actually the correct place to propose new API additions and other changes is through the issue tracker.
https://golang.org/s/proposal-process

I don't think this needs to live in the standard library, here is an
example of the functionality you want
http://godoc.org/github.com/xtgo/set#Uniq
On Mon, Sep 28, 2015 at 12:44 PM, Andrew Gerrand<EMAIL_ADDRESS>wrote:

Generally, it is better to propose new APIs on golang-dev mailing list
first where more contributors are actively watching each discussion.
Actually the correct place to propose new API additions and other changes
is through the issue tracker.
https://golang.org/s/proposal-process
—
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/12763#issuecomment-143626233.


added entries for definition and submission

@csuwildcat @glcohen @timcappalli  are there any other potentially top level claims that we need to register? Currently, I identified presentation_definition and presentation_submission but I guess input_descriptor and presentation_requirements might be top-level too?

Hi @awoie is this in a state to be merged?

Hi @awoie is this in a state to be merged?

Yes, the PR is fine if the only claims that we want to register are presentation_definition and presentation_submission. Isn't it also required to add input_descriptors and presentation_requirements?

@awoie I don't understand - input_descriptors and presentation_requirements are sub-objects within presentation_definition and presentation_submission, so don't they fall under those top-level registrations?

@awoie I don't understand - input_descriptors and presentation_requirements are sub-objects within presentation_definition and presentation_submission, so don't they fall under those top-level registrations?

then, we don't need to register them. feel free to merge the PR.


Added 3d effect to the title with some text shadow

Description
Added text shadow to enhance the title section and to give it a 3d effect
Do acknowledge and suggest if any changes required
Notes for Reviewers
now :

Thank you!!


Datasets availability

Hello,
Thank you for your great work! Are the two datasets (Ennedi 2019 and Delplanque et al. 2022) you mentioned in this repo available to download?  If so, how to download them? Thanks.

Hi @eebowen,
Thanks! Unfortunately, the Ennedi dataset can not be shared, but you can find and use the HerdNet's trained parameters in the README, the dataset of Delplanque et al. (2022) will soon be available again (we're migrating the dataset to another server).
I'll post the new link here and in the README once completed.
Thanks for waiting.
Best.

Hi @eebowen,
Here is the new link to the dataset: https://doi.org/10.58119/ULG/MIRUU5
Have fun!


Remove OpenGL compatibility header

I don't see the value of STPOpenGL.h anymore, it was added because I don't want to include GL header in the test application. Now it is just very cumbersome to maintain those typedef's.

Never mind, it is still useful and actually necessary when using with NVRTC.


Bf/raw id fields pages tree

There is a bug - when we want to user Page model in a raw_id_fields property, if we open up the dialog to choose a Page and click on any row, nothing happens because there is no JS handler from the "opener" object on Page rows.
That handler was in the menu_item.html file
https://github.com/divio/django-cms/commits/101fe24d7727756e80ebadbdd137400e022c9aaa/cms/templates/admin/cms/page/tree/menu_item.html
But after the Tree refactoring, that file was removed and it looks like that we forgot to add than handler in current version.

@vxsx could you review this? it's better to apply this manually to 3.2.x and then forward port it?

codewise LGTM
https://github.com/divio/django-cms/commit/6216bcc2182ef346f7f79032b4550270585b44a0 original commit (for reference)
maybe we should have a title tooltip as well, or even replace "view" icon with "pick" icon.
not really sure if it should be onclick on the row since we no longer use click on the page name as means to view it
wdyt @FinalAngel ?
@yakky for me no difference regarding forward-/back-porting, whatever would make most sense to you

@vxsx thanks. I'll when everything is ok here I'll merge manually in 3.2 branch
Thanks @Air-Mark for the fix

@yakky welcome. Are there any news about the merge?

@Air-Mark merged via #5125 thanks for your contribution


Adding SSR to skeleton-esnext-webpack 1.1.2

I'm submitting a question

Aurelia Skeleton Version
skeleton-esnext-webpack

Framework Version:
1.1.5

Please tell us about your environment:

Operating System:
OSX 10.x

Node Version:
7.8.0

NPM Version:
5.6.0

JSPM OR Webpack AND Version
3.4.1

Language:
ESNext

Current behavior:
I've currently got my production app https://isopro.solutions/about which was written in Aurelia. I've got to get SEO working and I'm wondering what the recommended approach is. Should I start with the newly created ssr template or is there an "upgrade" path explaining how to add ssr support to the skeleton-esnext-webpack 1.1.2

since it's not an issue. you are better off @ https://discourse.aurelia.io
I'd start with comparing the 2 skeletons and figuring the difference..
@rquast on discourse already ate a dog .. 2 maybe figuring out SSR + SEO + meta tags..

can be closed as not an issue.

Thanks for taking the time to respond!
—Steve

@sboyd no problem.. I'll meet you @ discourse


The files downloaded from the openclimatefix database are throwing a "blosc2" error.

The data downloaded from the database, https://huggingface.co/datasets/openclimatefix/eumetsat-rss/tree/main, cannot be read due to the error "ValueError: codec not available: blosc2."Even after reinstalling this module, the issue persists.

Hi, there needs to pip install ocf_blosc2 and then import ocf_blosc2 for ti to work.


build(deps): bumps @artsy/palette-mobile from 13.1.29 to 13.1.30

This PR resolves []
Description
bumps @artsy/palette-mobile from 13.1.29 to 13.1.30

#nochangelog

Warnings

:warning:

✅ No changelog changes

  Generated by :no_entry_sign: dangerJS against 997b46fc9d7e12ba254f0a3c07b4f621a476da93


fish 2.2 compatibility fixes

Greetings. Fish 2.2 is still in repositories of last versions of debian-based systems (Ubuntu 16, Debian 8), so it would be good to have compatibility with it.
Notes to patch:
history --null and builtin string is unavailable in fish2.2, so I removed related code, and it works the same way for me, but maybe I missed something?
Tnx in advance.

hi @vgavro,
I'm aware of that, but I still made the conscious decision to not support older versions of fish. Fish has changed quite a bit recently, and wish to utilitize the newer functionalities exposed.
Your changes will make it such that fzf can't read multi-line histories properly. There used to be a really convoluted way (https://github.com/fisherman/fzf/commit/c3defd4a922e97120503b45e26efa775bc672b50) that works with fish 2.2.0, but I'd really much rather use fish builtins from fish 2.4 and above.
That said, I'd be perfectly happy to keep a branch off this project that supports fish version 2.2.0, installable via fisher i fzf#2.2 for example. However, I won't be making changes to the branch, and you can expect it to diverge in many ways from master.

Yup, multiline - that's what i've missed. It's reasonable arguments, and I don't think we should have separate branch for it, on local machine it's easy to upgrade, and there may be fixes in user config for remote server (where you may not have root rights). My main goal is to be able to use same config on different machines without reinstalling shell, but it's ok to downgrade functionality in that case for me.
My solution then to redefine this functions in ~/.config/fish/config.fish in case of 2.2 version:
if echo $version | grep -q '^2.2'
    function __fzf_reverse_isearch
    function __fzf_find_and_execute
end

Thanks for quick response.

If you're using fisherman, I think a better way to go about this is to installl your fork of fzf: fisher i vgavro/fzf, and use this 2.2 compatible branch, instead of polluting your config.fish file.


多tag不能消费

The issue tracker is ONLY used for bug report and feature request.
Any question or RocketMQ proposal please use our mailing lists.
BUG REPORT

Please describe the issue you observed:

What did you do (The steps to reproduce)?
producer对一条message打了多个tag

What did you expect to see?
consumer订阅其中一个tag，期望正常消费

What did you see instead?
producer正常发送，但consumer一直没有消费到message

Please tell us about your environment:
RocketMQ4.1.0 jdk1.8

Hi, one message could only have one tag, and the consumer could subscribe with multiple tags.
Thanks and the issue tracker is ONLY used for bug report and feature request.

非常感谢回复@zhouxinyu ，看了代码知道一条message不支持打多个tag，但公司有这方面的业务需求，需要对一条message打多个tag，订阅消费，broker filter代码和client filter代码我做了修改，已经支持该功能并测试通过，可以将该功能提交到社区么。

Of course, any contribution is welcome.
As a new feature, we recommend discussing it in dev list first.
And please check out this feature filter-by-sql92  whether meets your demand.


[QA] Regression Testing Cycle - V0.4.0

QA -> develop branch

[X] 1. There seems to be a problem with the toggle column feature. When attempting to toggle the column, errors are encountered on the chrome://extension page. Additionally, when using the 'Toggle All' function to revert the changes, the columns do not render properly.

[!Important]
This issue is present on feat/column-reordering branch. Since it was checked out from dev, I checked on dev branch as well. The issue presents there as well.

Steps to replicate the issue -> https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/902c9c26-e72a-435e-b71e-2029ccde5478

Screenshot

chrome://extensions

UI glitch on PSAT

[x] 2. The cookie icon show with white color when clicked else where

https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/4797181c-f59d-4cf8-bc96-312da6ec5767

[x] 3. I need to click the 'Analyze this tab' button multiple times to retrieve the cookies, and occasionally, it requires more than two clicks. Additionally, there is a noticeable flickering effect that disrupts the basic functionality of the PSAT.

With older codebase
With latest code pull from dev branch (commit - 702e0699b2f37dae4f9b783c05cbb606541083d5)

https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/831f9db3-945f-44d8-aea2-65f84a038999
https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/e46b2694-a42c-42bb-9853-06880193108b

Errors on chrome://extensions page -

~4. Delete single cookie icon - it should only enabled after any row is selected~ Not in a scope for current release

https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/4b072411-a28a-4f53-8131-16a456cafd45

5. UI glitch on Toggle column Ref:comment

[x] 6. Warnings are not being shown for the cookies that are going to be affected

PSAT
Application Tab with console output showing Affected Cookies

[x] 7. Sidebar : Frame Selection Issue

Tab 1: The user navigates to any frame to view the table data.
Tab 2 : It will show Analyse this tab button. Clicking on this button it will load the cookies.
Tab 1: Return to Tab 1, where you will find the correct display of 'Analyse this tab.' Clicking on it successfully loads the cookie for the website. However, it is important to note that there is an issue in the sidebar – the 'Cookies Landing' frame should be selected, but currently, it is not reflecting this behavior.

Expected behaviour
Actual behaviour

https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/e39e74ff-bc97-4b86-88f9-18b88c13853d

8. As per the latest implementation the cookie jar should show 0 count in pop up - Invalid issue

[x] 9. The filter's basic functionality is not working as expected

filter disappears once you click on the filter option for which the cookie is not present

filter icon gets disabled

clearing the filter option from top bar, retains the filter but the selected options does nor come up

Please refer to the screencast below.
https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/c5a93043-c7dc-42f0-a0c9-f6ea02212f86

[x] 10. Filters: Missing padding from top and bottom when the single filter is applied.

[x] 11. Frame overlay icon is not visible.

[!Note]
The added fix is partially working. It does not show the frame overlay icon for the very first site. or until the user explicitly clicks on Analyse this tab button.

https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/2ade8e15-9aec-4ed5-a3c6-d8d4fef4b202

~12. Clicking X (single cookie delete icon) without selecting any row throws the error on chrome://extension page~ Not in scope of current release now.

Testing with latest commit - 702e0699b2f37dae4f9b783c05cbb606541083d5 from dev branch

[x] 13. URL is not working as expected. rewriting the URL does not load the correct set of cookies within some expected set of time and takes longer than expected.

https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/60979c31-552c-4406-9a4e-6fe2b59f49d4

~14. There seems to have an issue with Blocked Cookie. Either it shows very less cookie or zero cookie while console tabs there are lot more cookies to be blocked~ Being tracked separately via https://github.com/GoogleChromeLabs/ps-analysis-tool/issues/354

Test Site - https://www.aljazeera.com/

Attempt 1
Attempt 2

Test Site - https://www.livemint.com/

With latest code pull from dev branch

Tab
Screenshot

Network Tab

PSAT Table

Cookies Landing PAge

[x] 15. Size and priority are not visible under filter

[x] 16. There seems to have some jerk/flickering effect while opening any landing page or sub menus of it.

Visual
Link

Screencast 1
https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/9233d40a-0f45-471a-9933-1947ff578db1

Screencast 2
https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/a5a1c171-7e33-49a6-abd7-3824a118ba3c

While switching between the tabs it also flickers
https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/d63a4c94-7ee5-4f21-94a8-e35817eeca44

Error log

[x] 17. Please make the colors uniform for the links on landing pages

On both the themes, there is a difference in colors of link but for light mode it is not much noticeable.

[!Important]
Not a regression bug

Dark Mode
Light Mode

~18. Frame overlay: Selecting the filters from the dedicated sidebar does not reflect's in tooltip. If I select any filter for for the dedicated frame, then it should reflect the same in tooltip as well.~ Invalid Ref : comment

Please check the below visual where I have tried to showcase

How the current build is behaving and not changing the values
Also, showcased the same using  stable version how it should reflect

https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/027f9cad-72dd-4eec-b94c-79269ebe460f

[ ] CLI: In the CSV and sitemap, Cookies count are not visible in the report.

Warnings are not being shown for the cookies those going to be affected

Not able to reproduce this. It works fine on my system.

As per the latest implementation the cookie jar should show 0 count in pop up

Only for those tabs which are being analysed. If a tab is not being analysed then the count wont show.

Describe the bug
The cookies landing page that displays the number of blocked cookies randomly gets stuck at 0, despite the confirmation of blocked cookies in the Network Tab of Chrome's developer tools.
To Reproduce
Steps to reproduce the behavior:

Load the Privacy Sandbox Analysis Tool extension in Chrome.
Open a new instance and navigate to a website (e.g., example.com).
Close (if already open) and reopen the developer tools to ensure the Privacy Sandbox tab is present.
Observe the website, which likely involves third-party cookies, eg, https://edition.cnn.com .
Check the cookies landing page of the Privacy Sandbox Analysis Tool for the count of blocked cookies.
Navigate to the Network Tab and filter the results by blocked response cookies.
Notice the discrepancy between the actual blocked cookies in the Network Tab and the zero count displayed on the cookie's landing page.

Expected behavior
The expected behavior is that the cookies landing page counter of the Privacy Sandbox Analysis Tool accurately reflects the number of third-party cookies blocked by the browser, in accordance with the data shown in the Network Tab.
Screenshots

Screencast:
https://www.youtube.com/watch?v=cCHg9rT5uyQ
Desktop (please complete the following information):

OS: Pop!_OS 22.04 LTS x86_64
Chrome Version: Version 120.0.6099.109 (Official Build) (64-bit)

Additional context
This issue seems to occur randomly and poses a significant problem in accurately debugging and analyzing third-party cookie behavior in Chrome.

Item 8 in the first comment is an invalid issue, since it is expected that pop-up badge should not appear when a website is not analyzed.

Clicking X (single cookie delete icon) without selecting any row throws the error on chrome://extension page.

This will not be addressed since we are not releasing cookie modification and deletion functionalities in this release

Item 13 is also expected due to the usage of PROMISE QUEUE

Frame overlay icon will only be visible once user clicks analyse this tab button if tab is not being analysed it wont be visible.

Describe the bug
The cookies landing page that displays the number of blocked cookies randomly gets stuck at 0, despite the confirmation of blocked cookies in the Network Tab of Chrome's developer tools.

@gagan0123 Would you mind adding the information about any flags you have enabled ( third-party phaseout etc ) or you have blocked 3p cookies from the Chrome settings?
Also if this happens again please follow the following steps

Right click on the devtools panel and select inspect frame

After passing the sitemap file in the CLI. It is showing 0 URLs in the file even though there are URL's in the sitemap.xml file.

You are using the wrong argument its
npm run cli -- -p <path to xml>

A minor issue at the cookie table: when we right-click a cookie, the menu has . at the end of the Copy network filter string option:

@ayushnirwal Agree with @fellyph's suggestion. I have added the related issue here also under the subtask 3

A minor issue at the cookie table: when we right-click a cookie, the menu has . at the end of the Copy network filter string option

Removing the period
@kjnanda @fellyph

5: UI glitch on Toggle column

Will be updating the logic in a later release, not planning to push any changes for now.
cc @kjnanda

@Mayank Rana Size is visible in filter but can we please show it in ASC sorting order?

@ayushnirwal With the latest code pull from dev branch - Commit - 1d18c6d2035cec36ce690f63d401fa0efb70bf5e
The following issues are noticed:

[ ] Only Top frame shows the cookies. And rest frames are blank

https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/07cadda7-cd47-4d73-a5b3-855e242ed600

@ayushnirwal Below task needs attention from you.
Subtask 3, 9 (partially fixed - https://github.com/GoogleChromeLabs/ps-analysis-tool/issues/339#issuecomment-1863939211 ), 13

QA - branch fix/blocked-cookies-2

[ ] Invalid Domain option is missing under filters

[ ] Popup -> Clicking on "analyse this tab" button is not functioning as expected and throws error on chrome://extensions page

QA - fix/blocked-cookie-count

[ ] 1. Blocked cookie count is not visible for 2nd tab

Testing against fix/blocked-cookie-count branch

[ ] Cookie Blocked Reason : I see NotOnPath has 1 cookie as per cookies landing page. However, Could not see any cookies that falls under this category.

Test Site : https://edition.cnn.com/
https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/15173772/c617c598-1dd7-44f8-9a39-c35954af1694

CLI Dash board

@ayushnirwal Could you please look into this issue?

[ ] Horizontal matrix showing frames count is not visible on CLI

Closing this issue as it is addressed in V0.4.0 release


[Feature Request] Directly link to the remote community on the home instance

Hi,
First of, this script is pretty awesome! Reminds me of  (a lite version of) what RES is to Reddit!
However, one thing that I noticed was that when you're on a different instance and you click the "easy subscribe" button, it redirects you to the search page on your home instance. I'm not sure if this has any benefits of going through the search page first, but I thought it would be way smoother if the script redirected you directly to the actual community page via your own instance.
To do so, I changed line 722 from this
var subString = settings.theInstance + "/search?q=!" + community + "@" + site + "&type=All&listingType=All&page=1";
to this
var subString = settings.theInstance + "/c/" + community + "@" + site;
That works great for me so far, but if using the search page has any benefits, you could also add it as an extra button.
Additionally, it would be nice if you could optionally trigger this redirection automatically. So when the script notices that you're on an instance that is not your home instance while the option is enabled, it redirects you automatically.
Would love to hear your thoughts on this!
Edit: Another minor thing that is not worth opening a separate issue for:  I noticed is that the button that leads to Lemmyverse.net is called "Find Subs". This refers to subreddits, which is of course Reddit terminology. I suggest changing this to "Find Communities", which is the term that is used on Lemmy (Code is at line 630 ).

That is a good idea! My thinking is yes! This should definitely be added. When I started this I wrote it for myself on my personal instance so each new sub it was required to manually search to add it. However, if you are signed up on one of the big instances it likely already exists since someone else has already subscribed. So... for most people that is probably going to work but we would have to enable some sort of a check to see if it exits before the button would show up. In the future, there is going to be calls back to the homeInstance with these sorts of things which I haven't began to implement. However, I think it will be doable.
So yes, consider this added in the future if possible. I will update this ticket once it is added.
It's funny you mention the "Find Subs", It was "Find Comms" before this version but I changed it because I was thinking people wouldn't know what "Comms" were! So, I should of went with my gut and left it at comms. Will update in the next bug fix.
Thank you for your ticket. I'll update you asap.

Thank you! I admire your enthusiasm for your project! Having to search manually when your instance visits a different instance for the first time makes sense, if that is what you mean. You could of course simply make 2 separate buttons for both functions, makes it easier to implement as well. But the auto-check that you mentioned sounds like a really neat feature as well!
Also is there a reason why you want to shorten "Communities" to "Comms"? I've changed it to "communities" on my system and it doesn't seem too long to me, at least on my 1080p screen: Screenshot. I've also changed the text on the "Easy subscribe" button to "Browse community on home instance", as that seemed slightly more descriptive to me. However, these are just minor nitpicks based on my personal preference. When I label a button, I try to be descriptive as possible, so that the user knows exactly what to expect when they click on it.

I have added the "Browse/Sub on remote instance(s) that works correctly now. I'll leave this open as I want to make the auto-redirectr an option. I still plan to change back to comms :) I just keep forgetting. Thank you so much for your help.


Make gloss-juicy compile for my project.

I have mostly followed the type errors thrown at me by stack build --fast. The explicit imports make it easier to figure out which module exports what names. The style I use just happens to be easier for me to work with, but I wouldn't mind tweaking it into your own style. I've done away with gloss-juicy.cabal's version ranges, as I used stack to figure dependencies out on itself.
I've made it compile for yuvallanger/yasamsim@b07a13510b6eb8f79c768f041a3542af22837fe7, strictly using stackage, as described in stack.yaml.
I will push more commits and rebase it as you see fit, making this pull request suitable for a merge.
Thank you!

Oops. Two existing PRs do the exact same thing.


[Duplicate #5695]Morphological-operations-in-image-processing-using-Open CV

Article checklist
Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes
This closes #5435

@WanjaMIKE  i have made the changes requested, you can now proceed with the review thank you.

@hectorkambow  I have made grammatical  edits requested  looking forward for a review.

requesting for a review

@section-engineering-education/peer-reviewers is anyone available to do a prelim review (& tech review) on this. IF ANY more errors persist - we will close asap 👍


Enable comments only on certain notes

Is your feature request related to a problem? Please describe.
I don't want for the Giscus comments to appear on every notes, and also not on a folder view.
Describe the solution you'd like
It would be nice to be able to enable comments only for certain notes. Maybe by adding comments: true to the frontmatter.
Describe alternatives you've considered
Going in the opposite direction could also work, by having to explicitely disable comments on certain pages.
Additional context
This is in the case of making a blog-focused website.

I guess page specific enabling/disabling could be a configuration option for the Comments component.

maybe this solution will solve your problem click


USI I2C Slave @ 8MHz talking to Arduino Wire Library - Protocol Error

Thanks for the USI I2C library, great help. However, I found when in USI I2C Slave mode that if your Microcontroller is running at 8MHz or lower you will need to reduce the Arduino's Wire Library default (100kHz) I2C SCL frequency. Else the timing in the ACK section can be a bit dicey.
To change the Arduino's Wire Library default I2C SCL frequency you need to call Wire.setClock(<SCL_FREQ>) where SCL_FREQ is anywhere between 31000L to 400000L (31kHz to 400kHz). See begin section here .
This was an extremely annoying bug to hunt down.
Hope this makes sense and helps others.

Thank you for report. There are few problems with this. Some are global, some are local.
I incorrectly forked this git. So, I need to fix and fork correctly.
I tested this with AtTiny85. I didn't use arduino at all. Only pure C/C++. There was problems when tiny85 is working from 4 MHz, but 8 MHz worked fine with 100 kHz I2C clock. Seems that problem is with code in Interrupt. Interrupt response is too slow.
Because I am not planning to build any project with this library in near future, I will not fix this problem. So, try to contact with library author.