HBASE-22271 Implement grant/revoke/delete table acls/delete namespace acls in Procedure

Use UpdatePermissionProcedure to implement grant/revoke and RemovePermissionProcedure to implement delete namespace or table acls when delete namespace or table.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
39
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

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

+1
test4tests
0
The patch appears to include 7 new or modified test files.

_ master Compile Tests _

0
mvndep
33
Maven dependency ordering for branch

+1
mvninstall
300
master passed

+1
compile
113
master passed

+1
checkstyle
91
master passed

+1
shadedjars
370
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
623
master passed

+1
javadoc
63
master passed

_ Patch Compile Tests _

0
mvndep
15
Maven dependency ordering for patch

+1
mvninstall
302
the patch passed

+1
compile
124
the patch passed

+1
cc
124
the patch passed

+1
javac
124
the patch passed

-1
checkstyle
79
hbase-server: The patch generated 4 new + 297 unchanged - 4 fixed = 301 total (was 301)

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

+1
shadedjars
349
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
654
Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.

+1
hbaseprotoc
118
the patch passed

+1
findbugs
506
the patch passed

+1
javadoc
62
the patch passed

_ Other Tests _

+1
unit
47
hbase-protocol-shaded in the patch passed.

-1
unit
20430
hbase-server in the patch failed.

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

24502

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestAsyncTableRegionReplicasGet

hadoop.hbase.TestIOFencing

hadoop.hbase.master.balancer.TestStochasticLoadBalancerRegionReplicaLargeCluster

hadoop.hbase.client.TestScannersFromClientSide2

hadoop.hbase.filter.TestFuzzyRowFilterEndToEnd

hadoop.hbase.master.balancer.TestStochasticLoadBalancerRegionReplicaSameHosts

hadoop.hbase.coprocessor.TestMetaTableMetrics

hadoop.hbase.client.TestAsyncTableRegionReplicasScan

hadoop.hbase.client.TestMobSnapshotCloneIndependence

hadoop.hbase.regionserver.TestEncryptionKeyRotation

hadoop.hbase.master.balancer.TestStochasticLoadBalancerRegionReplicaHighReplication

hadoop.hbase.replication.regionserver.TestWALEntryStream

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/168

Optional Tests
dupname  asflicense  cc  unit  hbaseprotoc  javac  javadoc  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 3dbf645df018 4.4.0-131-generic #157-Ubuntu SMP Thu Jul 12 15:51:36 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 268bcce76f

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/1/artifact/out/diff-checkstyle-hbase-server.txt

whitespace
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/1/artifact/out/whitespace-eol.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/1/testReport/

Max. process+thread count
4356 (vs. ulimit of 10000)

modules
C: hbase-protocol-shaded hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/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
20
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

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

+1
test4tests
0
The patch appears to include 7 new or modified test files.

_ master Compile Tests _

0
mvndep
26
Maven dependency ordering for branch

+1
mvninstall
294
master passed

+1
compile
104
master passed

+1
checkstyle
95
master passed

+1
shadedjars
302
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
420
master passed

+1
javadoc
47
master passed

_ Patch Compile Tests _

0
mvndep
15
Maven dependency ordering for patch

+1
mvninstall
260
the patch passed

+1
compile
93
the patch passed

+1
cc
93
the patch passed

+1
javac
93
the patch passed

-1
checkstyle
79
hbase-server: The patch generated 4 new + 297 unchanged - 4 fixed = 301 total (was 301)

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

+1
shadedjars
285
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
625
Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.

+1
hbaseprotoc
82
the patch passed

+1
findbugs
417
the patch passed

+1
javadoc
46
the patch passed

_ Other Tests _

+1
unit
38
hbase-protocol-shaded in the patch passed.

-1
unit
7654
hbase-server in the patch failed.

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

11043

Reason
Tests

Failed junit tests
hadoop.hbase.master.replication.TestRegisterPeerWorkerWhenRestarting

hadoop.hbase.wal.TestWALOpenAfterDNRollingStart

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/168

Optional Tests
dupname  asflicense  cc  unit  hbaseprotoc  javac  javadoc  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux bca70a72b7b5 4.4.0-139-generic #165~14.04.1-Ubuntu SMP Wed Oct 31 10:55:11 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / f30d6c958a

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/2/artifact/out/diff-checkstyle-hbase-server.txt

whitespace
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/2/artifact/out/whitespace-eol.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/2/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/2/testReport/

Max. process+thread count
4316 (vs. ulimit of 10000)

modules
C: hbase-protocol-shaded hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/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
39
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

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

+1
test4tests
0
The patch appears to include 7 new or modified test files.

_ master Compile Tests _

0
mvndep
26
Maven dependency ordering for branch

+1
mvninstall
258
master passed

+1
compile
92
master passed

+1
checkstyle
88
master passed

+1
shadedjars
270
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
368
master passed

+1
javadoc
46
master passed

_ Patch Compile Tests _

0
mvndep
17
Maven dependency ordering for patch

+1
mvninstall
256
the patch passed

+1
compile
92
the patch passed

+1
cc
92
the patch passed

+1
javac
92
the patch passed

+1
checkstyle
10
The patch passed checkstyle in hbase-protocol-shaded

+1
checkstyle
80
hbase-server: The patch generated 0 new + 297 unchanged - 4 fixed = 297 total (was 301)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
275
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
532
Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.

+1
hbaseprotoc
89
the patch passed

+1
findbugs
427
the patch passed

+1
javadoc
53
the patch passed

_ Other Tests _

+1
unit
40
hbase-protocol-shaded in the patch passed.

-1
unit
14672
hbase-server in the patch failed.

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

18230

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestAdmin1

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/168

Optional Tests
dupname  asflicense  cc  unit  hbaseprotoc  javac  javadoc  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux ce0cec451353 3.13.0-153-generic #203-Ubuntu SMP Thu Jun 14 08:52:28 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 67c937fa92

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/3/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/3/testReport/

Max. process+thread count
4601 (vs. ulimit of 10000)

modules
C: hbase-protocol-shaded hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/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
22
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

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

+1
test4tests
0
The patch appears to include 8 new or modified test files.

_ master Compile Tests _

0
mvndep
16
Maven dependency ordering for branch

+1
mvninstall
241
master passed

+1
compile
144
master passed

+1
checkstyle
146
master passed

+1
shadedjars
254
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
468
master passed

+1
javadoc
107
master passed

_ Patch Compile Tests _

0
mvndep
13
Maven dependency ordering for patch

+1
mvninstall
226
the patch passed

+1
compile
145
the patch passed

+1
cc
145
the patch passed

+1
javac
145
the patch passed

-1
checkstyle
31
hbase-client: The patch generated 3 new + 105 unchanged - 0 fixed = 108 total (was 105)

-1
checkstyle
64
hbase-server: The patch generated 3 new + 300 unchanged - 4 fixed = 303 total (was 304)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
260
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
969
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.0.3 3.1.2.

+1
hbaseprotoc
167
the patch passed

+1
findbugs
536
the patch passed

+1
javadoc
104
the patch passed

_ Other Tests _

+1
unit
31
hbase-protocol-shaded in the patch passed.

+1
unit
194
hbase-client in the patch passed.

+1
unit
204
hbase-procedure in the patch passed.

-1
unit
12586
hbase-server in the patch failed.

+1
unit
308
hbase-thrift in the patch passed.

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

17800

Reason
Tests

Failed junit tests
hadoop.hbase.coprocessor.TestMetaTableMetrics

hadoop.hbase.client.TestAsyncTableRegionReplicasScan

hadoop.hbase.regionserver.TestCompactingToCellFlatMapMemStore

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/168

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile  cc  hbaseprotoc

uname
Linux 4255f4f8d1d3 4.4.0-145-generic #171-Ubuntu SMP Tue Mar 26 12:43:40 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 1c1638f698

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/4/artifact/out/diff-checkstyle-hbase-client.txt

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/4/artifact/out/diff-checkstyle-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/4/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/4/testReport/

Max. process+thread count
3873 (vs. ulimit of 10000)

modules
C: hbase-protocol-shaded hbase-client hbase-procedure hbase-server hbase-thrift U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/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
92
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

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

+1
test4tests
0
The patch appears to include 9 new or modified test files.

_ master Compile Tests _

0
mvndep
28
Maven dependency ordering for branch

+1
mvninstall
271
master passed

+1
compile
209
master passed

+1
checkstyle
167
master passed

+1
shadedjars
270
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
518
master passed

+1
javadoc
120
master passed

_ Patch Compile Tests _

0
mvndep
11
Maven dependency ordering for patch

-1
mvninstall
211
root in the patch failed.

-1
compile
22
hbase-backup in the patch failed.

-1
cc
22
hbase-backup in the patch failed.

-1
javac
22
hbase-backup in the patch failed.

+1
checkstyle
12
The patch passed checkstyle in hbase-protocol-shaded

-1
checkstyle
31
hbase-client: The patch generated 1 new + 105 unchanged - 0 fixed = 106 total (was 105)

+1
checkstyle
12
The patch passed checkstyle in hbase-procedure

+1
checkstyle
67
hbase-server: The patch generated 0 new + 302 unchanged - 4 fixed = 302 total (was 306)

+1
checkstyle
33
The patch passed checkstyle in hbase-thrift

+1
checkstyle
13
The patch passed checkstyle in hbase-backup

+1
whitespace
1
The patch has no whitespace issues.

+1
shadedjars
264
patch has no errors when building our shaded downstream artifacts.

-1
hadoopcheck
151
The patch causes 16 errors with Hadoop v2.8.5.

-1
hadoopcheck
298
The patch causes 16 errors with Hadoop v2.9.2.

-1
hadoopcheck
477
The patch causes 16 errors with Hadoop v3.1.2.

-1
hbaseprotoc
29
hbase-backup in the patch failed.

-1
findbugs
25
hbase-backup in the patch failed.

+1
javadoc
139
the patch passed

_ Other Tests _

+1
unit
41
hbase-protocol-shaded in the patch passed.

+1
unit
228
hbase-client in the patch passed.

+1
unit
207
hbase-procedure in the patch passed.

+1
unit
11404
hbase-server in the patch passed.

-1
unit
500
hbase-thrift in the patch failed.

-1
unit
34
hbase-backup in the patch failed.

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

16764

Reason
Tests

Failed junit tests
hadoop.hbase.thrift.TestThriftServer

hadoop.hbase.thrift2.TestThrift2HttpServer

hadoop.hbase.thrift2.TestThriftHBaseServiceHandlerWithLabels

hadoop.hbase.thrift2.TestThriftHBaseServiceHandlerWithReadOnly

hadoop.hbase.thrift.TestThriftHttpServer

hadoop.hbase.thrift.TestThriftSpnegoHttpServer

hadoop.hbase.thrift2.TestThriftHBaseServiceHandler

hadoop.hbase.thrift2.TestThriftConnection

hadoop.hbase.thrift.TestThriftServerCmdLine

hadoop.hbase.thrift2.TestThrift2ServerCmdLine

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/168

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile  cc  hbaseprotoc

uname
Linux 79d38d32469a 4.4.0-145-generic #171-Ubuntu SMP Tue Mar 26 12:43:40 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / aacacc5cce

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

mvninstall
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-mvninstall-root.txt

compile
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-compile-hbase-backup.txt

cc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-compile-hbase-backup.txt

javac
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-compile-hbase-backup.txt

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/diff-checkstyle-hbase-client.txt

hadoopcheck
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-javac-2.8.5.txt

hadoopcheck
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-javac-2.9.2.txt

hadoopcheck
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-javac-3.1.2.txt

hbaseprotoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-hbaseprotoc-hbase-backup.txt

findbugs
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-findbugs-hbase-backup.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-unit-hbase-thrift.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/artifact/out/patch-unit-hbase-backup.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/testReport/

Max. process+thread count
3952 (vs. ulimit of 10000)

modules
C: hbase-protocol-shaded hbase-client hbase-procedure hbase-server hbase-thrift hbase-backup U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/5/console

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
27
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

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

+1
test4tests
0
The patch appears to include 9 new or modified test files.

_ master Compile Tests _

0
mvndep
23
Maven dependency ordering for branch

+1
mvninstall
241
master passed

+1
compile
161
master passed

+1
checkstyle
145
master passed

+1
shadedjars
257
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
510
master passed

+1
javadoc
104
master passed

_ Patch Compile Tests _

0
mvndep
13
Maven dependency ordering for patch

+1
mvninstall
223
the patch passed

+1
compile
158
the patch passed

+1
cc
158
the patch passed

+1
javac
158
the patch passed

+1
checkstyle
8
The patch passed checkstyle in hbase-protocol-shaded

+1
checkstyle
31
The patch passed checkstyle in hbase-client

+1
checkstyle
12
The patch passed checkstyle in hbase-procedure

+1
checkstyle
70
hbase-server: The patch generated 0 new + 302 unchanged - 4 fixed = 302 total (was 306)

+1
checkstyle
28
The patch passed checkstyle in hbase-thrift

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
259
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
704
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
hbaseprotoc
134
the patch passed

+1
findbugs
534
the patch passed

+1
javadoc
104
the patch passed

_ Other Tests _

+1
unit
34
hbase-protocol-shaded in the patch passed.

+1
unit
130
hbase-client in the patch passed.

+1
unit
200
hbase-procedure in the patch passed.

+1
unit
8263
hbase-server in the patch passed.

+1
unit
214
hbase-thrift in the patch passed.

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

13062

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/6/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/168

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile  cc  hbaseprotoc

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

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 9b23e1d00c

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/6/testReport/

Max. process+thread count
4583 (vs. ulimit of 10000)

modules
C: hbase-protocol-shaded hbase-client hbase-procedure hbase-server hbase-thrift U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/6/console

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

This message was automatically generated.

@mymeiyi  any update? Thanks.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
178
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

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

+1
test4tests
0
The patch appears to include 9 new or modified test files.

_ master Compile Tests _

0
mvndep
25
Maven dependency ordering for branch

+1
mvninstall
250
master passed

+1
compile
153
master passed

+1
checkstyle
157
master passed

+1
shadedjars
262
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
512
master passed

+1
javadoc
112
master passed

_ Patch Compile Tests _

0
mvndep
15
Maven dependency ordering for patch

+1
mvninstall
235
the patch passed

+1
compile
158
the patch passed

+1
cc
158
the patch passed

+1
javac
158
the patch passed

+1
checkstyle
9
The patch passed checkstyle in hbase-protocol-shaded

+1
checkstyle
32
The patch passed checkstyle in hbase-client

+1
checkstyle
11
The patch passed checkstyle in hbase-procedure

+1
checkstyle
72
hbase-server: The patch generated 0 new + 302 unchanged - 4 fixed = 302 total (was 306)

+1
checkstyle
32
The patch passed checkstyle in hbase-thrift

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
260
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
722
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
hbaseprotoc
139
the patch passed

+1
findbugs
544
the patch passed

+1
javadoc
110
the patch passed

_ Other Tests _

+1
unit
35
hbase-protocol-shaded in the patch passed.

+1
unit
198
hbase-client in the patch passed.

+1
unit
215
hbase-procedure in the patch passed.

-1
unit
17926
hbase-server in the patch failed.

+1
unit
354
hbase-thrift in the patch passed.

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

23214

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas

hadoop.hbase.client.TestFromClientSide

hadoop.hbase.master.TestSplitWALManager

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/7/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/168

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile  cc  hbaseprotoc

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

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / ada772a1d3

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/7/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/7/testReport/

Max. process+thread count
4994 (vs. ulimit of 10000)

modules
C: hbase-protocol-shaded hbase-client hbase-procedure hbase-server hbase-thrift U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/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
30
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

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

+1
test4tests
0
The patch appears to include 9 new or modified test files.

_ master Compile Tests _

0
mvndep
15
Maven dependency ordering for branch

+1
mvninstall
237
master passed

+1
compile
142
master passed

+1
checkstyle
148
master passed

+1
shadedjars
261
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
514
master passed

+1
javadoc
108
master passed

_ Patch Compile Tests _

0
mvndep
11
Maven dependency ordering for patch

+1
mvninstall
247
the patch passed

+1
compile
158
the patch passed

+1
cc
158
the patch passed

+1
javac
158
the patch passed

+1
checkstyle
10
The patch passed checkstyle in hbase-protocol-shaded

+1
checkstyle
30
The patch passed checkstyle in hbase-client

+1
checkstyle
14
The patch passed checkstyle in hbase-procedure

+1
checkstyle
66
hbase-server: The patch generated 0 new + 302 unchanged - 4 fixed = 302 total (was 306)

+1
checkstyle
31
The patch passed checkstyle in hbase-thrift

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
268
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
707
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
hbaseprotoc
143
the patch passed

+1
findbugs
519
the patch passed

+1
javadoc
103
the patch passed

_ Other Tests _

+1
unit
32
hbase-protocol-shaded in the patch passed.

+1
unit
201
hbase-client in the patch passed.

+1
unit
207
hbase-procedure in the patch passed.

-1
unit
10818
hbase-server in the patch failed.

+1
unit
228
hbase-thrift in the patch passed.

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

15734

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestFastFail

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/8/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/168

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile  cc  hbaseprotoc

uname
Linux 630b17db8fd4 4.4.0-145-generic #171-Ubuntu SMP Tue Mar 26 12:43:40 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / ada772a1d3

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/8/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/8/testReport/

Max. process+thread count
4460 (vs. ulimit of 10000)

modules
C: hbase-protocol-shaded hbase-client hbase-procedure hbase-server hbase-thrift U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-168/8/console

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

This message was automatically generated.

Will close this PR in next week or so unless it comes back to life. Thanks.

Close this PR and will split this task into several minor tasks. Thanks for all comments.


Consolidate KV watches for RunTime config

Currently the KV watches are spread throughout the server setup code, as well as various other packages. They should probably all be in one place.

Also make the KV watch for LRU size work


After building example Ubuntu wic.img it appears that it has no kernel image in it

Hello, I checked out to jan/ubuntu branch and tried to create the Example Ubuntu image with the command below
./kas-docker build kas-iot2050-example.yml:kas/opt/ubuntu.yml
In the end I got a wic.img. After opening that archive the root filesystem was okay but couldn't see the kernel image(vmlinuz) and initrd. Here is what the archive looks like :

I cannot boot this image for IOT2050 Advanced I changed the boot order of uboot and gave priority to usb.  What might cause that?
Best regards.
Ömer

The kernel is under /boot and will be picked up by the boot loader (U-Boot) from there.
I didn't try to boot the generated image myself recently, maybe something broke. Do you have a boot log taken from the debug UART?

Since I don't have uart connection cables yet I dont't have the logs. But  as soon as I acquire the required equipments I will share here.
Thanks for the quick response.


togglePanel + panelPosition keybinding

It would be great to have keybinding that helps to not only switch the position of the panels (Left/Bottom/Right) as workbench.action.positionPanel$Position doing, but also toggle its visibility.
How I see it's can be implemented:

Triggering workbench.action.togglePanel$Position -> Open/Close panel at position, Close it at previous position if was Opened at different $Position.

Adding when settings that tell about  isWorkbenchPanelVisible (or something like that)

P/S/
As extension API doesn't provide access to DOM or any other way to get panels position and visibility status of the panels, it makes it impossible to implement it by myself, not via monkey patching.

@butuzov can you explain exactly what the keybinding does? I am not sure I understand

@sbatten I was talking about toggling panel ("View: Toggle Panel") and Changing Position Combination ("View: move panel to Right/Left/Bottom"). So, for example, you can have shortcut to TogglePanelToBottom and another one for TogglePanelToRight.
I think vscode already had such functionality, but it was removed for some reason.

When the panel is moved with e.g. Move Panel Right, the panel is made visible even if it is not already. What would this new command do differently?

It will hide it. In this case,  user will need to remember a keyboard shortcut less.

Actually even adding to vscode when the condition that indicates that panel is open will help.
In this case its will possible to set same keybinding for let's say Move Panel Right and Toggle Panel, but run Toggle Panel with crated when condition.
e.g.
 {
        "key": "cmd+right",
        "command": "workbench.action.positionPanelRight"
},
{
        "key": "cmd+right",
        "command": "workbench.action.togglePanel",
        "when": "positionPanel == 'right' && panelIsVissible"
},
{
        "key": "cmd+j",
        "command": "-workbench.action.togglePanel"
},

Damn. Guys, you really need to write more docs. I did some source digging.
    {
        "key": "cmd+right",
        "command": "workbench.action.positionPanelRight"
    },
    {
        "key": "cmd-right",
        "command": "workbench.action.togglePanel",
        "when" : "activePanel != '' && panelPosition == 'right'"
    },
    {
        "key": "cmd+down",
        "command": "workbench.action.positionPanelBottom"
    },
    {
        "key": "cmd-down",
        "command": "workbench.action.togglePanel",
        "when" : "activePanel != '' && panelPosition == 'bottom'"
    },

P/S/
Closing as solved.


axiom id correspondence between FOL and OWL

From<EMAIL_ADDRESS>on June 28, 2012 11:53:30
Sometimes an axiom in FOL turns into several axioms in OWL, or to a non-equivalent but derivable axiom in OWL (e.g. at-all-times relationship versions).
Thus far I am making the axiom id for all the OWL axioms be the same as the FOL axiom id.
The question is should I do that? Or should there be an additional relation to record this and a distinct id for the OWL axiom?
Original issue: http://code.google.com/p/bfo/issues/detail?id=77

From<EMAIL_ADDRESS>on June 29, 2012 13:15:02
Stefan (and alan agrees) that current method is adequate. Let Chris review and close or send mail if he doesn't like it.
Status: Started

From<EMAIL_ADDRESS>on July 10, 2012 10:25:58
Status: Done

From<EMAIL_ADDRESS>on July 10, 2012 10:27:22
We will follow this policy


UI: DataOptions form layout

Position the Snapshot properties section at the top of the DataOptions step. (from meeting feedback)

close duplicate


Optical flow: compute velocity from corrected flow data for logging

This is really useful to align the flow with the IMU data and verify that the compensation is properly done
update to https://github.com/PX4/ecl/pull/920
includes:

logging of compensated optical flow data converted to velocity measurement in body and local frames
logging of un/compensated optical flow data for rate motion and synchronized gyro data
adjust scaling value of pmw3901 data (was off by 30%)
adjust delay parameter for pmw3901 for perfect synchronization with gyro data

Bench tested with pmw3901 optical flow sensor and Lanbao PSK-CM8JL65 range finder:
Note that I recently renamed the fields in optical_flow_vel so the signals in the screenshot below don't have the correct names

@dagar I addressed your comments, any other remarks?

@dagar I addressed your comments, any other remarks?

Thanks, good to go from my perspective.

unrelated SITL failure

@dagar I just squashed two commits together to make the history cleaner, could you approve again please?


Try changing the name to the name of an existing class, or creating a class with the name 'GeneratedIntColumn'.dartundefined_class

Getting below error
Flutter 2.4.0-1.0.pre.68 • channel master • https://github.com/flutter/flutter.git
Framework • revision c8884b53c6 (2 hours ago) • 2021-07-03 21:16:02 -0400
Engine • revision 5a4b9bacd5
Tools • Dart 2.14.0 (build 2.14.0-276.0.dev)

Whne you're using moor version 4.4.0, please upgrade to moor_generator version 4.4.0 and run the build again. Sorry for the inconvenience!


Make the WebApps logger configurable

Right now, in it's rough implementation, it only accepts a parameter. It should be configurable through the configuration system (options?) and you should be able to set an environment variable to configure the max file size, for example.
cc @pakrym

@victorhurdugaci Let's have a look when you're back.

I think pakrym fixed this...

@pakrym

@pakrym remember fixing this?

@pakrym If this isn't already done, we can move this out.

Done as part of https://github.com/aspnet/Logging/pull/477
See https://github.com/aspnet/Logging/blob/dev/src/Microsoft.Extensions.Logging.AzureAppServices/AzureAppServicesDiagnosticsSettings.cs


Realm RealmMapView.toObject — unsafeBitCast returns error — swift 3.0.2

I've been trying the Building an iOS Clustered Map View for Swift, and every time I get the following error from the code below:
fatal error: can't unsafeBitCast between types of different sizes
The error is being thrown in RealmMapView.toObject(), invoked as shown (copy/paste from the web page, with the addition of ".self" to get rid of a warning):
extension ViewController: MKMapViewDelegate {
  func mapView(mapView: MKMapView, didSelectAnnotationView view: MKAnnotationView) {
    if let safeObjects = ABFClusterAnnotationView.safeObjectsForClusterAnnotationView(view) {
      if let firstObjectName = safeObjects.first?.toObject(ABFRestaurantObject.self).name {
        print("First Object: \(firstObjectName)")
      }
    }
  }
}

I've never managed to get around this error. It seems like the framework is trying to convert an annotation to an ABFRestaurantObject, but maybe I'm confused. Is there a workaround for this, or am I just missing something?
The Podfile just references RealmMapView and RealmSwiftSFRestaurantData.
Here is the effect of a pod install:
Using ABFRealmMapView (2.0)
Using RBQFetchedResultsController (5.0.2)
Using RBQSafeRealmObject (1.1)
Using Realm (2.4.4)
Using RealmMapView (2.0)
Using RealmSwift (2.4.4)
Using RealmSwiftSFRestaurantData (0.5)
Using RealmUtilities (0.4)
Using SafeRealmObject (1.1)
Using SwiftFetchedResultsController (5.0.2)

@DallasLyon I'm not sure if this is the correct solution but this has worked for me:
if let firstObject = safeObjects.first?.rlmObject() as? ABFRestaurantObject {         print("First Object: \(firstObject.name)")       }
The maintainer could probably confirm whether or not this is the intended behavior due to some API changes or if I just read their source code and hacked my own solution! Not sure the rlmObject() returned from this method.
See this code in the OBJ-C example:
https://github.com/bigfish24/ABFRealmMapView/blob/46ccd004368302d90d49e8b959a12ebd2fcf9399/ABFRealmMapViewExample/ABFRealmMapViewExample/ViewController.m#L95

Excellent, @jimjeffers. I had sort of gone down the path toward using rlmObject(), but apparently saw something shiny before getting to it. That works. Great.


nixos/gdm: add banner option

Description of changes
This exposes the banner message option in GDM. Some computing
environments have compliance requirements which include displaying a
message to the user before logon.
This option matches the banner for SSH at services.openssh.banner, and as well as the banner at console login settable via services.getty.helpLine.
Example:

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@danielfullmer rebased, please test.

banner = ''
  something
  blah
  yeah
'';


No way to set value for property with [SupplyParameterFromQuery] but not [Parameter]

If I have a component with a property decorated with [SupplyParameterFromQuery]  without [Parameter]:
FromQuery.razor:
Search = @Search

@code {
    [SupplyParameterFromQuery]
    public string? Search { get; set; }
}

Blazor will set the property correctly from the query string (e.g. ?search=test), but there is currently no way to set the value in bUnit.
Using NavigationManager.NavigateTo() has no effect:
using TestContext testContext = new();

IRenderedComponent<FromQuery> renderedComponent = testContext.RenderComponent<FromQuery>();

NavigationManager navigationManager = testContext.Services.GetRequiredService<NavigationManager>();
string uri = navigationManager.GetUriWithQueryParameter("search", "test");

nvaigationManager.NavigateTo(uri);

// `renderedComponent.Instance.Search` is `null` here.

Setting the property directly as parameter causes ArgumentException:
using TestContext testContext = new();

testContext.RenderComponent<FromQuery>(
    parameters => parameters.Add(component => component.Search, "test"));

System.ArgumentException: The parameter selector 'component => component.Search' does not resolve to a public property on the component `FromQuery` with a [Parameter] or [CascadingParameter] attribute.

Version info:

bUnit version: 1.26.64
.NET Runtime and Blazor version: 8.0.1

Hey @aayjaychan and thanks for reporting the issue.
Indeed that might be a bug since .NET 8. Before .NET 8 the SupplyParameterFromQueryAttribute only inherited from System.Attribute and users should add the ParameterAttribute as well:
[Parameter]
[SupplyParameterFromQuery]
public int QuerySize { get; set; }

Since the latest installment, the SupplyParameterFromQueryAttribute directly inherits from CascadingParameterAttributeBase making the additional Parameter attribute obsolete.
So we have to adopt our logic for .NET 8 here.

This also applies to SupplParameterFromForm.

For SupplyParameterFromForm we might want to add new overloads to the ComponentParameterCollectionBuilder as it allows targeting specific forms (by name).

Some more context: https://github.com/dotnet/aspnetcore/pull/48554/files#diff-329ab76a7fca1f1325c9c8a21d79c9d7731b5a2163b81b7797a86291b53e0153

@aayjaychan We have an idea of how to tackle that. Unfortunately, it isn't as straightforward and easy as we thought.
So a proper fix will take a bit more time.
For the time being you can set the value on the instance a trigger a new render cycle if you are stuck otherwise:
cut.Instance.Search = "My Value";
cut.Render();

@aayjaychan
Thanks for staying with us. @egil  and I pushed a changed, that should enable your scenario with the latest preview package.
NavigationManager navigationManager = testContext.Services.GetRequiredService<NavigationManager>();
string uri = navigationManager.GetUriWithQueryParameter("search", "test");

nvaigationManager.NavigateTo(uri);

This should now do the trick for you. The 1.27.12-preview version is available via NuGet. It would be awesome to get your feedback on this one.
We will also adopt the docs to guide others a bit better in this case.


Keep the original worker id after master fail-over or worker re-register

What changes are proposed in this pull request?
Please outline the changes and how this PR fixes the issue.
Why are the changes needed?
Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

@beinan Fix the failed test?


System.Data.SqlClient.SqlException (0x80131904): Connection Timeout Expired.

SQL Operations Studio Version: sqlops 0.23.6 (8a3011756318a2ded1e6e84332230a2b2bf9c544, 2017-11-05T22:46:14.815Z)
OS Version: Linux x64 4.8.0-54-generic

Steps to Reproduce:
System.Data.SqlClient.SqlException (0x80131904): Connection Timeout Expired.  The timeout period elapsed during the post-login phase.  The connection could have timed out while waiting for server to complete the login process and respond; Or it could have timed out while attempting to create multiple active connections.  The duration spent while attempting to connect to this server was - [Pre-Login] initialization=128; handshake=267; [Login] initialization=4; authentication=25; [Post-Login] complete=14115;  ---> System.ComponentModel.Win32Exception (0x80004005): Unknown error 258
at System.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, Object providerInfo, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling)
at System.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
at System.Data.ProviderBase.DbConnectionFactory.CreateNonPooledConnection(DbConnection owningConnection, DbConnectionPoolGroup poolGroup, DbConnectionOptions userOptions)
at System.Data.ProviderBase.DbConnectionFactory.<>c__DisplayClass22_0.b__0(Task1 _)    at System.Threading.Tasks.ContinuationResultTaskFromResultTask2.InnerInvoke()
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot)
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.SqlTools.ServiceLayer.Connection.ReliableConnection.ReliableSqlConnection.<>c__DisplayClass28_0.<b__0>d.MoveNext() in C:\J\jobs\sqltoolsservice_master_win\workspace\src\Microsoft.SqlTools.ServiceLayer\Connection\ReliableConnection\ReliableSqlConnection.cs:line 298
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.SqlTools.ServiceLayer.Connection.ConnectionService.d__46.MoveNext() in C:\J\jobs\sqltoolsservice_master_win\workspace\src\Microsoft.SqlTools.ServiceLayer\Connection\ConnectionService.cs:line 542
ClientConnectionId:dbfc1573-ada2-43f1-bd49-ded8ec967736
Error Number:-2,State:0,Class:11

I've seen this type of connection failure in 2 cases:

You've just booted up a docker instance and SQL Server is still initializing. Try connecting again after waiting 30seconds
I think this might be the case when connecting to SQL Server 2008 R2 and where SP3 or higher isn't installed. There's a mandatory security update in SP3, and the .Net Core driver won't connect to versions without this fix.

Either way, this would be a connectivity issue with the server. In the next release, we've moved the large stacktrace out of the connection dialog so it's clearer that it's a connection problem. I'm going to close this with the recommendations above since any solution would be on the server / setup side.

when will the next version come out ?.
please let me know if it's out.
Thank you very much.


Dockerfile fails to build

This is the error currently faced with the latest version of Ubuntu.
CMake Error at /usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
Could NOT find OpenSSL, try to set the path to OpenSSL root folder in the
system variable OPENSSL_ROOT_DIR (missing: OPENSSL_CRYPTO_LIBRARY) (found
suitable version "3.0.13", minimum required is "1.1.1")
Call Stack (most recent call first):
/usr/share/cmake-3.28/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
/usr/share/cmake-3.28/Modules/FindOpenSSL.cmake:668 (find_package_handle_standard_args)
CMakeLists.txt:135 (find_package)
-- Configuring incomplete, errors occurred!
The command '/bin/sh -c cmake -S liboqs -B liboqs/build -DBUILD_SHARED_LIBS=ON &&     cmake --build liboqs/build --parallel 4 &&     cmake --build liboqs/build --target install' returned a non-zero code: 1

Thanks for this bug report, @loganaden . It reminds me of a problem I think @SWilson4 worked around somewhere else, so asking for his input/suggestions here.

@SWilson4 any suggestions ?

Ahhh--I think I rememver it has to do with a bug in cmake. Can you try building&using a later Version than the one in Ubuntu by default @loganaden ?

This is an issue we've encountered before with newer versions of CMake. See this comment for details. To get the command to work as is, I had to downgrade CMake. An easier workaround is to explicitly set OPENSSL_ROOT_DIR, as described in that comment. I haven't tested newer versions of CMake to see if they fix the issue.
Come to think of it, it would probably be a good idea to file a bug report with CMake... I'll get on that.

I have a (hopeful) fix up: see https://github.com/open-quantum-safe/liboqs/issues/1748#issuecomment-2256628489 and https://github.com/open-quantum-safe/liboqs/pull/1873.

Successfully built liboqs-python Installing collected packages: nose2, liboqs-python Successfully installed liboqs-python-0.10.0 nose2-0.15.1 Removing intermediate container 7563a817b7d0  ---> 73e2688dcb34 Successfully built 73e2688dcb34 Successfully tagged oqs-python:latest
`liboqs version: 0.10.2-dev
liboqs-python version: 0.10.0
Enabled KEM mechanisms:
['BIKE-L1', 'BIKE-L3', 'BIKE-L5', 'Classic-McEliece-348864',
'Classic-McEliece-348864f', 'Classic-McEliece-460896',
'Classic-McEliece-460896f', 'Classic-McEliece-6688128',
'Classic-McEliece-6688128f', 'Classic-McEliece-6960119',
'Classic-McEliece-6960119f', 'Classic-McEliece-8192128',
'Classic-McEliece-8192128f', 'HQC-128', 'HQC-192', 'HQC-256', 'Kyber512',
'Kyber768', 'Kyber1024', 'ML-KEM-512-ipd', 'ML-KEM-512', 'ML-KEM-768-ipd',
'ML-KEM-768', 'ML-KEM-1024-ipd', 'ML-KEM-1024', 'sntrup761',
'FrodoKEM-640-AES', 'FrodoKEM-640-SHAKE', 'FrodoKEM-976-AES',
'FrodoKEM-976-SHAKE', 'FrodoKEM-1344-AES', 'FrodoKEM-1344-SHAKE']
Key encapsulation details:
{'claimed_nist_level': 1,
'is_ind_cca': True,
'length_ciphertext': 768,
'length_public_key': 800,
'length_secret_key': 1632,
'length_shared_secret': 32,
'name': 'Kyber512',
'version': 'https://github.com/pq-crystals/kyber/commit/74cad307858b61e434490c75f812cb9b9ef7279b'}
Shared secretes coincide: True

Closing as fixed by open-quantum-safe/liboqs#1873.


RGB format convertion on Jetson Nano

Required Info

Camera Model
D415

Firmware Version
<IP_ADDRESS>.00

Operating System & Version
UBuntu 18.04 for Tegra

Kernel Version (Linux Only)
4.9.140-tegra

Platform
NVIDIA Jetson Nano

SDK Version
2.23.1

Language
C++/opencv

Issue Description

When using Intel RealSense Viewer, the color of YUYV stream more accurate than RGB stream. the images of RGB stream look redder. how the Intel RealSense Viewer convert YUYV stream

After getting YUYV stream using C++, call yuy_decoder.process(), it also gives a redder image and the convertion cost about 15ms, but if  calling cv::cvtColor of opencv, it gives a accurate color image, just like using RealSense Viewer with YUYV stream, and the convertion costs 5 to 8ms. The convertion to RGB with yuy_decoder should using CUDA, but it is slower than convertion with opencv, which only use CPU. Maybe the memory transfer between host and device leads to this?

The documentation for yuy_decoder.process() says that it automatically uses CUDA "where available".  I wonder if this is dependent on Librealsense having been built with CUDA support.
https://intelrealsense.github.io/librealsense/doxygen/classrs2_1_1yuy__decoder.html#a2754499ae0f8fa1892ef33fe6bdc4313
If you are installing Librealsense using the Debian packages, CUDA support should already be included.  Otherwise, Librealsense can be built with CMake to have CUDA support by including the BUILD_WITH_CUDA flag in the CMake build instruction.
https://github.com/IntelRealSense/librealsense/issues/4816#issuecomment-529195322

Hi @7oud
Thank you for the feedback. Interesting...
Could you please share more info about item 1. You mean YUY render in the Viewer vs RGB in the Viewer? Same exact hardware config? Perhaps there is inconsistency between CUDA and GLSL implementations. Please share what you see.
Regarding second item, GPU is not guarantied to be always faster than CPU. The reason we use CUDA on Jetson, is mainly because our CPU implementation on Arm is not optimized. OpenCV clearly is. We decided not to add opencv as a dependency to simplify installation.
The team did some work on this problem, and our plan is to use libyuv to get good CPU performance on Arm.

@dorodnic
yes, I mean [YUY] render in the viewer vs RGB.

The two images(cropped) are captured with viewer,  both resolution 1920x1080, 30FPS, without Auto Exposure, without Auto White Balance, without Post Processing, the only difference is stream, one is YUYV, the other is RGB8.
If save the YUYV stream .bag, and convert to RGB with yuy_decoder or opencv, it gives similar result(RGB8 stream renders redder).
Some HW info, Jetson Nano, anything else?

Do you require any further assistance with this case or can it be closed, please?   Thanks!

@MartyG-RealSense You can close this issue, it is not serious problem and may be fixed in future update.

Okay, thanks very much!


Plays 'this is what slow speed sounds like' when translator is called

When  or useTranslator() is used, after the result is returned, an audio is played automatically saying 'this is what slow speed sounds like. How to solve this issue. Any help is much appreciated.
Also mentioned in this issue,
also the "eu policy" setInterval causes a sound in the "enter url" page
it should check the url in injecetedscript
Originally posted by @soroshzzz26 in https://github.com/KoreanThinker/react-native-translator/issues/18#issuecomment-1703905022
version "react-native-translator": "^1.1.6",

Same issue. This package makes my app speaking.

I think this package  doesent maintained anymore but i forked the package and fixed the bugs for my project
https://github.com/soroshzzz26/react-native-translator
you can check the commits

Hi @soroshzzz26 ! This looks interesting. But have you fixed all the bugs ? I also tested this package on another project and the application crashed...


passwords.expired missing from Devise-I18n

Devise Views gem had this:
    passwords:
      expired:
        renew_your_password: Renew your password
        change_required: Your password is expired. Please change your password.
        change_required_days: Your password is older than %{days} days. Please change your password.

Any chance we can add this back to this gem?

I don't see that in https://github.com/plataformatec/devise/blob/master/config/locales/en.yml or https://github.com/mcasimir/devise-i18n-views/blob/master/locales/en.yml (which is what I assume you mean by "Devise Views gem".
What would require those strings?

Closing per above.


Update datasets dependency to 2.11.0 version

Close #1009.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Codecov Report
Patch coverage has no change and project coverage change: +1.79 :tada:

Comparison is base (2021c40) 89.50% compared to head (41bc40c) 91.29%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1010      +/-   ##
==========================================
+ Coverage   89.50%   91.29%   +1.79%
==========================================
  Files         149       55      -94
  Lines        8128     3998    -4130
==========================================
- Hits         7275     3650    -3625
+ Misses        853      348     -505

Flag
Coverage Δ

jobs_cache_refresh
?

jobs_mongodb_migration
?

libs_libcommon
?

services_admin
?

services_api
?

services_worker
91.29% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 94 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


blurry text if  is set under 500px height

#vuex #vue
Expected Behavior
Clear Text on the tooltips and text itself

Current Behavior
blurry text on the tooltips and text itself. it happens when the canvas element is set below 500px. i should set it under 500px though so i set the max-height: [below 500px]

Environment

Chart.js version: v 2.7.3
Browser name and version: Chrome v 70.0.3538.110 (Official Build) (64-bit) & Firefox Quantum v64.0b14 (32-bit)
Code:

<template>
    <div>
		<canvas ref="revLineChart"/>
	</div>
</template>

<script>
import { mapGetters, mapActions } from 'vuex';

export default {
	name: 'RevenueLineChart',
};

</script>

<style scoped>
    canvas {
        max-height: 350px;
    }

    #revLineChart {
        -moz-font-smoothing: antialiased;
        -webkit-font-smoothing: antialiased;
    }

</style>

Please provide a jsfiddle or similar that reproduces your issue.

Please provide a jsfiddle or similar that reproduces your issue.

https://codepen.io/dhika345/pen/yQKmmB

We don't support (min|max)-(height|width) style on the canvas when responsive: true, so when the canvas should be bigger than 450px, we still resize it to the parent container size, which one is not constrained. The blurry issue comes from that the canvas render size is different from the canvas display size.

okay. Never realized there's doc for it. thank you so much. everything's okay now


Madara update

madara:

fix search 404 at the end
getCheerio instead of calling fetch everytime
dragon tea translator
fix riwyat search (closes #919)
fix riwyat random text (closes #912)
sources cleanup
daonovel new icon
icons cleanup
filters for everyone
better get_filters

lightnovelwp:

better get_filters

should this close https://github.com/LNReader/lnreader-plugins/issues/874 too?

Dragon Tea still
chapter path: novel/the-necromancer-in-the-novel-picks-up-bones/chapter-2/

Btw I think we can keep the down sources in madara instead of removing them. They can be back some day

Btw I think we can keep the down sources in madara instead of removing them. They can be back some day

They've been gone for 2 months at least, and if someone really wanted them he would just create a PR, it takes 5s to add them back

Btw I think we can keep the down sources in madara instead of removing them. They can be back some day

They've been gone for 2 months at least, and if someone really wanted them he would just create a PR, it takes 5s to add them back

Ye, but if someone changes the pluginId, the old novels aded by that plugin will be ghost novels


flannel_use_upstart = "False" instead of false on ansible version 1.9.4

In Ansible 1.9.4, [flannel_use_upstart] (https://github.com/kubernetes/contrib/blob/master/ansible/roles/flannel/tasks/client.yml#L17) is set to "False"
TASK: [flannel | set_fact ] ***************************************************
<<IP_ADDRESS>> ESTABLISH CONNECTION FOR USER: core
ok: [<IP_ADDRESS>] => {"ansible_facts": {"flannel_use_upstart": "False"}}
<<IP_ADDRESS>> ESTABLISH CONNECTION FOR USER: core
ok: [<IP_ADDRESS>] => {"ansible_facts": {"flannel_use_upstart": "False"}}
<<IP_ADDRESS>> ESTABLISH CONNECTION FOR USER: core
ok: [<IP_ADDRESS>] => {"ansible_facts": {"flannel_use_upstart": "False"}}

In Ansible 2.0.1 [flannel_use_upstart] (https://github.com/kubernetes/contrib/blob/master/ansible/roles/flannel/tasks/client.yml#L17) is set to false
TASK [flannel : set_fact] ******************************************************
task path: /Users/stelouie/Documents/RIO/contrib/ansible/roles/flannel/tasks/client.yml:16
ok: [kube-master] => {"ansible_facts": {"flannel_use_upstart": false}, "changed": false, "invocation": {"module_args": {"flannel_use_upstart": false}, "module_name": "set_fact"}}
ok: [kube-node-1] => {"ansible_facts": {"flannel_use_upstart": false}, "changed": false, "invocation": {"module_args": {"flannel_use_upstart": false}, "module_name": "set_fact"}}
ok: [kube-node-2] => {"ansible_facts": {"flannel_use_upstart": false}, "changed": false, "invocation": {"module_args": {"flannel_use_upstart": false}, "module_name": "set_fact"}}

This causes a CoreOS install using ansible 1.9.4 to skip [systemd-service.yml] (https://github.com/kubernetes/contrib/blob/master/ansible/roles/flannel/tasks/systemd-service.yml) instead of running it like it should.

Possible Fix

Require ansible 2.0.1
Sloppily check for the string values [here] (https://github.com/kubernetes/contrib/blob/master/ansible/roles/flannel/tasks/github-release.yml#L39)

cc @rutsky @danehans
Edit
It attempts both flannel_use_upstart and not flannel_use_upstart. I don't imagine that this is desired. Maybe connected to this https://github.com/ansible/ansible/issues/8629
[systemd-service.yml] (https://github.com/kubernetes/contrib/blob/master/ansible/roles/flannel/tasks/systemd-service.yml)
[upstart-service.yml] (https://github.com/kubernetes/contrib/blob/master/ansible/roles/flannel/tasks/upstart-service.yml)
TASK: [flannel | assert ] *****************************************************
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]

TASK: [flannel | Add Flanneld Systemd Unit File] ******************************
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]

TASK: [flannel | Create Docker systemd dropin directory] **********************
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]

TASK: [flannel | Add Docker drop-in with dependency on Flannel] ***************
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]

TASK: [flannel | assert ] *****************************************************
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]

TASK: [flannel | Create Upstart Script] ***************************************
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]
skipping: [<IP_ADDRESS>]

Looks like we are hitting the upstream issue. The issue was fixed here: https://github.com/ansible/ansible/commit/a571fd4efecf0b7a870f40e9b5433559def4244e
From reading the upstream issue details, it seems as though ansible/roles/flannel/tasks/client.yml should be reverted in: https://github.com/stephenrlouie/contrib/commit/86c3df1ffcd76496218f60049aecddef878f0858
Can you try reverting ^ and test both ansible versions?


[Bug]: I can't make a sharepoint report

Hi, thakns for this tool !
I would like audit my tenant but, audit of sharepoint doesn't works.
I have create a new online user and adding these roles :

Global Reader and Security Reader
Sharepoint Administrator

Steps to reproduce the behavior:

What command are you running?

I have used the commands same as the documentation :
$param = @{
Instance = 'Microsoft365';
Collect = 'ExchangeOnline','MicrosoftTeams','Purview','SharePointOnline';
PromptBehavior = 'SelectAccount';
IncludeEntraID = $true;
ExportTo = 'HTML';
}
$assets = Invoke-Monkey365 @param

See error

$assets = Invoke-Monkey365 @param
AVERTISSEMENT : [13:22:25:031] - [Connect-MonkeyM365] - Not connected to SharePointOnline - warning - DESKTOP-XXX -
Monkey365SharePointError
AVERTISSEMENT : [13:22:25:039] - [Connect-MonkeyM365] - Unable to connect SharePoint online. No valid host was found -
warning - DESKTOP-XXX - Monkey365SharePointError
From where are you running Monkey365?
Powershel v5.1 on Windows 11
Thank you for support !

Hi @impalaa,
Glad you like it! :D
According to the error message It seems that monkey365 is unable to get information from Microsoft Graph regarding available SharePoint domains.
As a workaround, you can include the -ScanSites param

$param = @{
    Instance = 'Microsoft365';
    Collect = 'ExchangeOnline','MicrosoftTeams','Purview','SharePointOnline';
    PromptBehavior = 'SelectAccount';
    IncludeEntraID = $true;
    ExportTo = 'HTML';
    ScanSites = https://**yourdomain**.sharepoint.com;
}
Invoke-Monkey365 @param

That flag is not documented yet, I'll add this to my TODO list and will try to better document the tool.
Thanks,

Thanks for your fast answer.
I have tried like that :
$param = @{     Instance = 'Microsoft365';     Collect = 'ExchangeOnline','MicrosoftTeams','Purview','SharePointOnline';     PromptBehavior = 'SelectAccount';     IncludeEntraID = $true;     ExportTo = 'HTML';     ScanSites = https://domain-admin.sharepoint.com; }
I have an error :
The term “https://domain-admin.sharepoint.com” is not recognized as a command applet name.
Do you have an idea ?
Thanks again :)

Hi,
This is because you're copying and pasting the whole snippet. Try with this one (Please, note the quotes between domain name):

$param = @{
    Instance = 'Microsoft365';
    Collect = 'ExchangeOnline','MicrosoftTeams','Purview','SharePointOnline';
    PromptBehavior = 'SelectAccount';
    IncludeEntraID = $true;
    ExportTo = 'HTML';
    ScanSites = "https://domain.sharepoint.com";
}
Invoke-Monkey365 @param

Please, also note that you must use the regular non-admin SharePoint url, and not the SharePoint admin url (i.e. yourdomain-admin.sharepoint.com is not allowed for ScanSites param). Once the default url is set, Monkey365 will attempt to authenticate to the url passed through the ScanSites parameter, and if the authentication is done, then will try to locate and authenticate as well to the admin blade.
Hope that helps!

Hello,
Not better :(
I don't have any report for sharepoint, all the rest works.

Hi,
Could you please add -Verbose -Debug and -InformationAction Continue to the execution command? Those flags will show you info from console and hopefully you will see what's happening.


Can't set env on broken deployment

I've push a deployment but have some wrong env variable which are causing the app not to work but using timed out waiting for deployment completion give an error of

timed out waiting for deployment completion

before I had pushed the app I could set envs without any issue

Would you please provide detailed reproduction steps with an open source example app? I am so far unable to reproduce this.

Closing for lack of detail, please open a new issue with reproduction steps.


Error when there is a mismatch on named vector format

When using a named vector format, if there is a mismatch, causing error.  Should just be flowing through.

NA in stub column was causing trouble.  Made it so it does not crash.  Still produces report.


Can not delete s3 path, Directory not empty

In mars project, delete a s3 path:
https://github.com/deepeth/mars/blob/abb7563bb90f7987927a86577802a1e44f32c924/ethetl/src/exporters/worker.rs#L65-L67
will return error:
Remove "/tmp/eth/14111000_14111099" error:Custom { kind: DirectoryNotEmpty, error: ObjectError { op: "delete", path: "/home/bohu/github/deepeth/mars/_datas/tmp/eth/14111000_14111099", source: Directory not empty (os error 39) } }

Please use remove_all
op.batch().remove_all("path/to/dir").await

Use this function in cautions to avoid unexpected data loss.


Issue when using a roda prefix when using the same prefix as a namespace in Rails

In rails, I'm using the following routes to scope my Admins controller:
namespace :admins do
  resources :questions

  root to: 'dashboard#index'
end

Which creates the routes:
admins_root      GET  /admins(.:format)                      admins/dashboard#index
admins_questions GET  /admins/questions(.:format)            admins/questions#index

Then I'm configuration rodauth with:
configure(:admin) do
    enable :login, :logout, :remember
    rails_controller { Admins::RodauthController }
    prefix "/admins"
end

route do |r|
    rodauth.load_memory # autologin remembered users

    r.on "admins" do
      r.rodauth(:admin)

       if !rodauth(:admin).logged_in?
         rodauth(:admin).require_authentication
       end
    end
end

The login is working, but once I'm logged, I can not access the Rails routes and I got a blank page.
One solution would be to use an other Roda prefix (such as admin-authentication), but maybe there's a way to fallthrough Rails routes ?
I've a branch wich reproduce this issue https://github.com/nicolas-besnard/rodauth-rails/commit/ddf5da42d27c325f72324dec363c2b0d73448770

Thanks again for the report. This is caused by how Roda routing works; if a r.on block reaches the end, the request is considered as handled, and in this case Roda middleware will not call the Rails app.
One workaround is to load the pass Roda plugin and call r.pass at the end of the r.on block:
class RodauthApp < Rodauth::Rails::App
  # ...
  plugin :pass

  route do |r|
    # ...
    r.on "admins" do
      r.rodauth(:admin)

      if !rodauth(:admin).logged_in?
        rodauth(:admin).require_authentication
      end

      r.pass # this breaks out of the block
    end
    # ...
  end
end

@jeremyevans do you know of a better way to have custom routes accessible that have the same prefix as the Rodauth prefix when the Roda app is used as a middleware?

Using r.pass is fine. You can also use throw :next, true at any point to directly pass control to the next middleware.  Maybe we could add a Roda method for that to the middleware plugin (e.g. r.next_middleware).

I'm not sure if there are any downsides to this, but I have been just using break which seems to accomplish it too:
admin = rodauth(:admin)
r.on(admin.prefix.delete_prefix("/")) do
  r.rodauth(:admin)
  admin.require_authentication
  break # IMPORTANT: allow routing to continue to Rails
end

Should I switch to using throw :next or r.pass?

Wow, thank you for this discovery, break seems to work for me too, it's definitely preferred 👍🏻

So break > r.pass?  If break works, what is the purpose of the pass plugin sugar @jeremyevans ?  Are we missing out on something if we simply use break?

The pass plugin was designed for users familiar with the same feature in Sinatra. Agreed that break seems like a better way to handle things in Roda. I'll have to do some testing to see if there are any corner cases (maybe around hash_routes/multi_route?).

I found that break doesn't produce correct behavior, because it doesn't reset the matched segment from the block we're skipping. So, the following:
r.on "auth" do
  r.rodauth
  break
end

r.on "admin" do
  r.rodauth(:admin)
end

would in addition to /admin/login also route /auth/admin/login, assuming the :admin Rodauth configuration loads the login feature.
As of latest rodauth-rails version, you shouldn't need break or r.pass anymore (unless you're defining custom routes in the Roda app itself), because it activates the next_if_not_found: true on Roda's middleware plugin, which will automatically forward any unmatched requests to the Rails router.

Since break inside routing blocks doesn't work by default for the reasons @janko mentioned, I added a break plugin to Roda to support it: https://github.com/jeremyevans/roda/commit/bc1fefea1d583883b9c3d4501f6b45eb19003751


bgp community members with bracket syntax flagged as invalid

Hi,
First, thanks for the great work !
Just testing junoser vs. a couple of configs at hand and found that it doesn't like specifying community members as a list in a single line statement.
The following line triggers an invalid syntax error :
set policy-options community COMM_TEST members [ 64496:1 64496:2 ]

Any hint ? Thx !

HI, this is a junoser bug. (Juniper's actually)
set policy-options community COMM_TEST members one_word

# or

set policy-options community COMM_TEST members "quoted words"

The syntax above is valid according to the Juniper .xsd.
          <xsd:element name="members" minOccurs="0" maxOccurs="unbounded" type="xsd:string">
            <xsd:annotation>
              <xsd:documentation>Community members</xsd:documentation>
              <xsd:appinfo>
                <flag>mustquote</flag>
                <flag>twig-dynamic-db-ok</flag>
                <flag>current-product-support</flag>
              </xsd:appinfo>
            </xsd:annotation>
          </xsd:element>

I can fix it anyways. Thanks for your report.

Hi,
I've pushed fixes to this repo. Will publish it as a gem in a week.
In the meantime, you can try GitHub based gem:
gem install specific_install
gem specific_install https://github.com/codeout/junoser

I just tested and the error is not present anymore 👍 Thanks for the fix !

Thanks for your testing! v0.4.1 has been just published.


Allow bundling for browser without having to set PLATFORM_BROWSER: true with webpack

Currently to use in the browser you have to either load the minified bundle, which includes (and duplicates) dependencies that I may already have in my bundle, or bundle with webpack and set PLATFORM_BROWSER to true. Parcel doesn't have an option to inject these global variables or tree-shake with them, so importing lightstep-tracer and bundling with parcel will always end up bundling both platform implementations and running the Node implementation.

Does this also happen if you import lightstep-tracer/browser? Not super familiar with Parcel.

Ahh, ok, huh. Yeah, I think this is a pretty sane request. I'm not really sure what the right way to go about doing this would be though - split the package into a browser and non-browser version (pre-minification)?

package.json supports a browser field that is like a main field used when bundling for browsers. But it would require a bit of restructuring of the current code base with the platform abstraction.


Refactor quant_llm to work with affine quantized tensor

Summary:
We want to add quant_llm to affine quantized tensor as a general fp2-fp7 dtype, before that we need to refactor the current implementation to work with AffineQuantizedTensor first
Test Plan:
python test/prototype/test_quant_llm.py

# perf
# in torchao/_models/llama
python generate.py --compile --quantization fp6

# accuracy
python eval.py --quantization fp6 --compile

tok/s is matching the original now, fp6 is:
==========
Average tokens/sec: 115.62
Average Bandwidth: 573.39 GB/s
Peak Memory Usage: 6.97 GB
Model Size: 4.96 GB

perplexity with eval in _models/llama/ is:
wikitext: {'word_perplexity,none': 12.360869670148428, 'word_perplexity_stderr,none': 'N/A', 'byte_perplexity,none': 1.6003634603882877, 'byte_perplexity_stderr,none': 'N/A', 'bits_per_byte,none': 0.6783995944569804, 'bits_per_byte_stderr,none': 'N/A', 'alias': 'wikitext'}

matching previous result:
FP6 E3M2 | 12.34 | 106.76
Reviewers:
Subscribers:
Tasks:
Tags:

Since you're alraedy running this, what was the tok/s you were getting and was the output coherent?

Since you're already running this, what was the tok/s you were getting and was the output coherent?

I'm only getting 20 tokens/sec right now, still trying to see what is the problem

Still digesting the change. Just some questions about high-level design:

From what I understand, when we construct an AQT with FPx, AQT object will contain FpxAQTLayout, and then FpxAQTLayout will contain FpxTensor. It's a bit confusing. Do we need FpxTensor?
I don't really like the fact that things are scattered in many files now. Make it hard to follow. Is it possible to make AQT expose a mechanism to register custom layout, so that everything related to a particular layout can stay in a single, standalone file? (maybe in the future, seems like a big design change)
About the naming, should we keep some of the names somewhat related to the original work? i.e. Quant-LLM / FP6-LLM

From what I understand, when we construct an AQT with FPx, AQT object will contain FpxAQTLayout, and then FpxAQTLayout will contain FpxTensor. It's a bit confusing. Do we need FpxTensor?

yeah that's correct, FpxTensor is supposed to mimic the native dtype Tensor like uint4 tensor, if we add fpx dtypes (e.g. fp6e3m2 etc.) in core in the future, this tensor subclass will be used to back up these dtypes. it's similar to the UintxTensor

I don't really like the fact that things are scattered in many files now. Make it hard to follow. Is it possible to make AQT expose a mechanism to register custom layout, so that everything related to a particular layout can stay in a single, standalone file? (maybe in the future, seems like a big design change)

oh it's intermediate step, I'm planning to move quant_llm out of prototype in the next PR to reduce the size of the PR
yeah I think custom layout can already be registered separately, e.g. https://github.com/pytorch/ao/blob/2c8e3f368613f896fde14cc24b9268c9e125575e/torchao/dtypes/uintx/Uintx.py#L152 we 'll put fpx in a separate folder as well

About the naming, should we keep some of the names somewhat related to the original work? i.e. Quant-LLM / FP6-LLM
maybe this can live in kernel name? I'm trying to add this as a "standard" fpx dtype actually

Since you're already running this, what was the tok/s you were getting and was the output coherent?

updated summary

please take a look again @msaroufim @gau-nernst @jcaip

Just rmb about this. We have to update the benchmark script too 😅
https://github.com/pytorch/ao/blob/main/benchmarks/benchmark_fp6_llm.py


Allow task types to be "disabled"

There are some cases where a task type is not useful. In this case, the task type shouldn't be surfaced anywhere. One way to accomplish this would be to add a when property to task definitions. We can also add three built in contexts: ShellExecutionTasksEnabled, ProcessExecutionTasksEnabled, and CustomExecutionTasksEnabled.

Example of the current contribution point:
    "taskDefinitions": [
      {
        "type": "typescript",
        "required": [
          "tsconfig"
        ],
        "properties": {
          "tsconfig": {
            "type": "string",
            "description": "%taskDefinition.tsconfig.description%"
          },
          "option": {
            "type": "string"
          }
        }
      }
    ],

Proposal is simply to add a when:
    "taskDefinitions": [
      {
        "type": "typescript",
        "required": [
          "tsconfig"
        ],
        "properties": {
          "tsconfig": {
            "type": "string",
            "description": "%taskDefinition.tsconfig.description%"
          },
          "option": {
            "type": "string"
          }
        }
      "when": "ShellExecutionTasksSupported"
      }
    ],

This is now finalized.


fix: multiple small bug fixes

I hope all fixes are ok :-)
Sorry for adding more commits in one PR. In my project I had to to many changes to make it work and I hope it is not big deal in project of this state.
I am open to specific feedback and will improve it ASAP, so they can be merged soon. You can also cherry pick some commits if it makes sense.

Yes, I haven't did it just because I does not use that components, so I can not test it.If there were working examples in docs it would be easier to test it all. What plans you have about it?Honza Pobořil💬Signal30. 9. 2022 v 8:09, Vinayak Kulkarni @.***>:
@vinayakkulkarni requested changes on this pull request.
In src/markers/VMarker.vue:

 map.value.on('style.load', () => {

   // https://github.com/mapbox/mapbox-gl-js/issues/2268#issuecomment-401979967

   const styleTimeout = () => {

     if (!map.value.isStyleLoaded()) {

       loaded.value = false;

       setTimeout(styleTimeout, 200);

     } else {

       loaded.value = true;

     }

   };

   styleTimeout();

 });

If you've added the loaded check in map, maybe we can refactor this code from all the other files ?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I would like to use this library but it still needs more work and more reactivity. If I pass new coordinates to Marker then it should move on the map accordingly. I could make a PR with some improvements but first I would like to see this PR merged.


Change Language from Fragment

Hi
how to call setLanguage function from fragment for change language?
i have FragmentStatePagerAdapter for intro pages and i need to change language from child fragments
thanks.

Call getActivity().setLanguage(lang)

You can call
(getActivity() as? LocalizationActivity)?.setLanguage(language)


#1095 - Tag Creator process for MCP (previously ACS Tools Tag Maker)

Coverage increased (+0.02%) to 29.388% when pulling 08333e5760b56f89be3a635324230ff99824d798 on davidjgonzalez:feature/1095 into c98fb5e010c59af1d0c8cf0c25d1aa0f83be7dec on Adobe-Consulting-Services:master.

Coverage increased (+0.02%) to 29.388% when pulling 08333e5760b56f89be3a635324230ff99824d798 on davidjgonzalez:feature/1095 into c98fb5e010c59af1d0c8cf0c25d1aa0f83be7dec on Adobe-Consulting-Services:master.

Coverage increased (+0.02%) to 29.388% when pulling 08333e5760b56f89be3a635324230ff99824d798 on davidjgonzalez:feature/1095 into c98fb5e010c59af1d0c8cf0c25d1aa0f83be7dec on Adobe-Consulting-Services:master.

Coverage increased (+0.02%) to 29.388% when pulling 08333e5760b56f89be3a635324230ff99824d798 on davidjgonzalez:feature/1095 into c98fb5e010c59af1d0c8cf0c25d1aa0f83be7dec on Adobe-Consulting-Services:master.

Coverage increased (+0.02%) to 29.388% when pulling 08333e5760b56f89be3a635324230ff99824d798 on davidjgonzalez:feature/1095 into c98fb5e010c59af1d0c8cf0c25d1aa0f83be7dec on Adobe-Consulting-Services:master.

Looks great and I like that you added even more form components for folks to use!  Very cool.  Note that there's no need for NAME to be a static field on the process definition.  I was able to re-work things such that only the factory's getName() method has to return that value so it's not needed in more than one spot.

Changes Unknown when pulling 781e1ef77ec16c2b1961ec6b518c69d065a8ef8f on davidjgonzalez:feature/1095 into ** on Adobe-Consulting-Services:master**.

It all looks good to me but I don't have the final say to accept and merge the PR.


Fix applications count in workshop card

I see two major changes in code - adding deletion of child applications and changed WorkshopProviderViewCard.AmountOfPendingApplications mapping profile. Could they both be tested? Could other uncovered lines be tested as well?
Also, please see minor code formatting issue in separate comment.

Fixed


Use official windows docker image of dapper

https://github.com/rancher/rancher/blob/master/.drone.yml#L334
https://github.com/rancher/rancher/blob/master/.drone.yml#L347
We need to create/publish and start using rancher/dapper:windows-1809 instead of maiwj/dapper:windows-1809

Appears to have been fixed.


tests: filterpair test

Codecov Report

Merging #8 into master will increase coverage by 6.34%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   42.91%   49.25%   +6.34%
==========================================
  Files          10       10
  Lines         268      268
==========================================
+ Hits          115      132      +17
+ Misses        146      129      -17
  Partials        7        7

Impacted Files
Coverage Δ

internal/filter.go
53.12% <0%> (+53.12%)
:arrow_up:

Continue to review full report at Codecov.

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


feat: 日历组件

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Docs
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

您好，您的邮件我已收到。我将尽快给您回复，谢谢您的来件。


Georef subscene origin sync

Let any changes to CesiumGeoreference origin propagate to the active subscene, if exists.
Refactored CesiumSubScene to register itself as a listener to any parent CesiumGeoreference changes (changed  delegate). Similar to the pattern that CesiumGlobeAnchor uses.
A special OnParentGeorefChanged function ensures that parent changes don't propagate even more parent changes.
Added 2 new unit tests.
Resolves Issue #175

Done and done, @kring . Thanks for the review.

Thanks @csciguy8!


Error parsing email: Could not parse From  address header

Hey!
I ran into this project while researching ARM supported alternatives to Mailhog, it looks great! I'm all setup, but can't get Mailcrab to receive e-mails yet..
I'm on a docker setup, and when I use docker exec -it mailcrab sh to get into the container, and then run something like:
echo "Subject: hello" | sendmail -S <IP_ADDRESS>:1025 -v<EMAIL_ADDRESS>< mail.txt -f<EMAIL_ADDRESS>
The response I get is:
sendmail: recv:'220 mailcrab-backend ESMTP'
sendmail: send:'EHLO f3f6ce0b2354'
sendmail: recv:'250-server offers extensions:'
sendmail: recv:'250 8BITMIME'
sendmail: send:'MAIL<EMAIL_ADDRESS>sendmail: recv:'250 OK'
sendmail: send:'RCPT<EMAIL_ADDRESS>sendmail: recv:'250 OK'
sendmail: send:'DATA'
sendmail: recv:'354 Start mail input; end with <CRLF>.<CRLF>'
sendmail: send:'To<EMAIL_ADDRESS>sendmail: send:'test'
sendmail: send:'.'
sendmail: recv:'250 OK'
sendmail: send:'QUIT'
sendmail: recv:'221 Goodbye'

The mailhog docker logs show:
mailcrab  | 2023-05-08T12:19:10.465302Z  INFO mailcrab_backend::mail_server: New email on f3f6ce0b2354 from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>mailcrab  | 2023-05-08T12:19:10.465468Z  WARN mailcrab_backend::mail_server: Error parsing email: Could not parse From  address header

This is my configuration:
docker-compose.yml
  mailcrab:
    container_name: mailcrab
    image: marlonb/mailcrab:latest
    ports:
      - '1080:1080'
      - '1025:1025'
    labels:
      - "traefik.enable=true"
      - "traefik.http.routers.mail.rule=Host(`mail.c1`)"
      - "traefik.http.services.mail.loadbalancer.server.port=1080"
      - "traefik.http.routers.mail.tls=true"

I'm not sure how to further debug this issue, and I really don't get why Mailcrab is not able to parse the address header.
I'd appreciate any insight on this!
/ Marinus

Alright, so it does work using:
/usr/sbin/sendmail -S mailcrab:1025 -i -t << MESSAGE_END
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>
Hi, this is my message,
and I'm sending it to you!
MESSAGE_END

So I'll probably figure it out, share the solution and close the issue soon :)

As I see it, is the original

I'm not sure how to further debug this issue, and I really don't get why Mailcrab is not able to parse the address header.

still valid.
Having the WARN mailcrab_backend::mail_server: Error parsing email: Could not parse From  address header in only the logging after a 250 OK in response to 354 Start mail input; end with <CRLF>.<CRLF> feels wrong.

Makes sense - do you have
To reproduce this:

Launch docker container (marlonb/mailcrab:latest)
SSH into docker container (ex: docker exec -it mailcrab sh)
Run echo "Subject: hello" | sendmail -S <IP_ADDRESS>:1025 -v<EMAIL_ADDRESS>-f<EMAIL_ADDRESS>
Command response:
sendmail: recv:'220 mailcrab-backend ESMTP'
sendmail: send:'EHLO f3f6ce0b2354'
sendmail: recv:'250-server offers extensions:'
sendmail: recv:'250 8BITMIME'
sendmail: send:'MAIL<EMAIL_ADDRESS>sendmail: recv:'250 OK'
sendmail: send:'RCPT<EMAIL_ADDRESS>sendmail: recv:'250 OK'
sendmail: send:'DATA'
sendmail: recv:'354 Start mail input; end with <CRLF>.<CRLF>'
sendmail: send:'Subject: hello'
sendmail: send:'To<EMAIL_ADDRESS>sendmail: send:''
sendmail: send:'.'
sendmail: recv:'250 OK'
sendmail: send:'QUIT'
sendmail: recv:'221 Goodbye'

Mailcrab response:
mailcrab  | 2023-05-09T10:49:08.043132Z  INFO mailcrab_backend::mail_server: New email on f3f6ce0b2354 from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>mailcrab  | 2023-05-09T10:49:08.043532Z  WARN mailcrab_backend::mail_server: Error parsing email: Could not parse From  address header

It does work with a command like:
/usr/sbin/sendmail -S mailcrab:1025 -i -t << MESSAGE_END
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>
Hi, this is my message,
and I'm sending it to you!
MESSAGE_END

Even though my issue was resolved by adding -i and -t to my sendmail_path, adding these in the command line like below does not work:
echo "Subject: hello" | sendmail -i -t -S <IP_ADDRESS>:1025 -v<EMAIL_ADDRESS>-f<EMAIL_ADDRESS>
I'll leave it to maintainers/contributors to decide if this needs re-opening or not

As I see it, there is a source code change like
--- a/backend/src/types.rs
+++ b/backend/src/types.rs
@@ -162,7 +162,7 @@ impl TryFrom<mail_parser::Message<'_>> for MailMessage {
     fn try_from(message: mail_parser::Message) -> Result<Self, Self::Error> {
         let from = match message.from() {
             mail_parser::HeaderValue::Address(addr) => addr.into(),
-            _ => return Err("Could not parse From  address header"),
+            _ => addr.HeaderValue::Address(Addr::new(None, "not@input")),
         };

         let to = match message.to() {

needed. So not returning an error, but some thing that expresses not at input.
And I  think that it is better to not expect, to not insist, on From:  header being present.

I like the idea of just filling in a placeholder address when none is found - so we can proceed with the message.
This is changed in https://github.com/tweedegolf/mailcrab/commit/8677d4f4f82951e1916cbfba0491cc8e49247d0e (which I accidently pushed to main instead of making a PR). I also return an error now when there is a hard error parsing the message.

Beautiful - thanks!

Beautiful - thanks!

Ah yeah :blush:  :blush:,  I did understood the problem, but not yet the https://github.com/tweedegolf/mailcrab/issues/59#issuecomment-1538313334
sendmail_path = /usr/sbin/sendmail -t -i -S mailcrab:1025

I think the sendmail_path is an environment variable, but for which environment?
But is it an environment variable and all?

No worries, my issue has been resolved!
The sendmail_path is a PHP config setting :)


3D Object Detection Code

I appreciate the method in ponderv2 very much, and any plan to  release the code and the model weight  about 3D object detection?

Hi, currently we do not have detection code in our code base. You can load our pre-trained weight to other detection codebase that support sparse convnet.


[FLINK-11519][table] Add function related catalog APIs

What is the purpose of the change
As a followup for FLINK-11474 and its PR#8007, this PR adds additional catalog APIs for functions (UDFs)
Brief change log

Added APIs in ReadableCatalog and ReadableWritableCatalog
Provided an implementation of the APIs for GenericInMemoryCatalog
Supplied with corresponding tests

Verifying this change
Manually tested and provided UTs
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (JavaDocs)

This is to replace PR https://github.com/apache/flink/pull/8212. cc: @JingsongLi @KurtYoung

Thanks for Kurt's detailed review. I have updated the PR to reflect Kurt's suggestions. Please take another look and let me know if additional changes are needed. Thanks.


Changes are not applied after adding tags to PVC

What steps will reproduce the bug?
Hi! It looks like, that we found some issue after adding tags to storageConfigs.metadata.annotations field in nifi yaml configuration. Changes are not applied until the PVC is manually recreated. We are using EBS CSI GP3. Can you please look for solution to this issue?
What is the expected behavior?
Applying tags changes without manual intervention in the PVC.
What do you see instead?
No changes applied in tags until the PVC is manually recreated.
Possible solution
No response
NiFiKop version
v1.5.0-release
Golang version

Kubernetes version
Client Version: v1.28.2
Server Version: v1.28.2-eks
NiFi version
1.23.2
Additional context
No response

Can you please share the NifiCluster configuration you're using?

I think i've verified this as a bug, so working on a fix.

Do you still need any information from us?


Revert "Issue #1367: Build arm64/v2 (aarch64) skupper-router docker image"

Since we've decided we don't want to cross-compile with zig for arm64, let's again remove the code added to support it.
Zig compilation fails with libunwind 1.8.0 as whitnessed by @ganeshmurthy .

Reverts skupperproject/skupper-router#1100

There is a crash in system tests,

https://github.com/skupperproject/skupper-router/issues/1397


Ex4 - TDD Assignment

@ItamarShDev

please submit the form here
also as written in the instructions you need to do at least 7 tests

:+1:


Edge not getting generated when multiple  tasks in pipeline

Affected module
UI
Describe the bug

The issue is we were only checking if the task is a downstream task or not, we were missing the check for if the task has a downstream task.
Will push the fix for the same.


Convert from Flow to TypeScript

Adds source linter and auto-formatter (on build).
Renames the dist/ files for clarity.
Resolves #11

@glebm Thanks! Do you mind if I add you to collaborators of this repo?

Sure! By the way, textoverlay is now used here https://github.com/orlandohill/waxeye/pull/83, it's a perfect fit and a brilliant idea, thank you!

Yey 🎉 Feel free to add your name to package.json.
Do you need ownership of npm package too?

Sure, if that's OK with you, thanks!

Done 🕺


update @mocha/contributors to v1.1.2

that's about it.

Coverage remained the same at 92.361% when pulling 48740ad3f17dd3a75289ebcfe9df486a25f30d04 on boneskull/update-contributors into d02a0966d03b0945c77412bee5ea56b48ca26cf2 on master.


Enforce code style more strictly

This issue is imported from pivotal - Originaly created at Oct 15, 2019 by Michiel Kodde
The last couple of PR reviews, we had some discussion about enforcing thecode style. See some tasks below.

For typehint annotations we could use:
https://github.com/slevomat/coding-standard (bstrooband - Oct 15, 2019)

I think we should take into account that the typehints could be needed by external libraries (i.e. doctrine) (bstrooband - Oct 15, 2019)

@bstrooband I think Doctrine only relies on it\'s annotations if you choose to use Doctrine with annotations. Or do they rely on other type hints? If so I\'m most interested in a link to the documentation. (Michiel Kodde - Oct 15, 2019)

Experimented with Slevomat CS rules. I added these rules to the phpcs.xml
    <rule ref="SlevomatCodingStandard.TypeHints.TypeHintDeclaration" />
    <rule ref="SlevomatCodingStandard.TypeHints.UselessConstantTypeHint" />
    <rule ref="SlevomatCodingStandard.TypeHints.DeclareStrictTypes" />
    <rule ref="SlevomatCodingStandard.TypeHints.NullableTypeForNullDefaultValue" />

That resulted in a nice set of required changes in the src folder. Many of these can be addressed automatically via phpcbf. But this results in a non-functional set of changes. As some type hints are not correctly set on interfaces/implementations of interfaces.
Bottom line is: in order to use PHP 7 type hints on the existing code. We should update the existing code to use type hints correctly. (Michiel Kodde - Oct 21, 2019)

Created a branch that addresses some of the tasks of this story
https://github.com/OpenConext/OpenConext-engineblock/tree/feature/improve-cs
The todo on this branch is to implement PHP 7 strict types and type definitions on all classes. This is a change that should not be taken lightly. As increasing the strictness of the application might lead to unexpected errors/issues.  (Michiel Kodde - Oct 21, 2019)


Try to fix Moon icon on Yandex.Pogoda

Bad/With fix/Original

Hi.
Common selectors were changed, please look if still you see an issue.
The contribution process was also changed https://github.com/alexanderby/darkreader#how-to-contribute


Test labeling an issue using comments

Describe the issue you're reporting
Testing adding labels through comments.

Normal comment.

/label pkg/doesntexist exporter/doesntexist help-wanted

/label -pkg/doesntexist -help-wanted -no-such-label

/label pkg/doesntexist help-wanted

/label pkg/reallydoesntexist

/label pkg/doesntexist exporter/doesntexist help-wanted

/label -pkg/doesntexist -exporter/doesntexist -help-wanted

/label -pkg/doesntexist -exporter/doesntexist -help-wanted

/label pkg/doesntexist exporter/zipkin


Test Boxen with Beaker

So Virtualbox has made a lot of strides with OSX host support and @AndrewDryga made an easy to use Mavericks image: https://github.com/AndrewDryga/vagrant-box-osx-mavericks
I've got a proof of concept acceptance test working with manual intervention, but with a bit of work I can get it automated with beaker
This is kinda related to https://github.com/boxen/our-boxen/issues/401, but TravisCI seems way too hard to set this up. Probably easier to run on an OSX machine with Vagrant.

Here's some notes I'm making along the way:
https://gist.github.com/petems/74456ea4bc6bcbe0903f

I can try to update that image to latest version, also I can reduce box size and there is one known bug (happens sometimes) that I can fix in .box file. Do you need it? :)

@AndrewDryga that would be fantastic!
I was just doing the following on the vagrant box now for repackaging:

Accept XCode liscense
Update XCode (there's an Xcode issue with the boxen stuff when I ran it)
Run security unlock-keychain for Vagrant user (but I think I'll have to run that every time...)

If you could do that and host it somewhere you would be a :star: and I would owe you :beers:  :smile:

@petems sorry for delay, I don't have enough time lately but I didn't forget about this task

No problem, there's no rush :+1:

@petems I can agree license for you, but good thing is to use
sudo xcrun cc

to agree it in CLI mode

@petems which one keychain do you want to unlock? Login? I've tryed to make it unlocked by-default, but can't give promises because Mac OS being too unpredictable with keychains :)

@petems everything is done. Check it out: https://github.com/AndrewDryga/vagrant-box-osx

@AndrewDryga awesome! :+1:

https://github.com/boxen/our-boxen/pull/720

Hi,
Will you support Travis or CircleCI?
Thanks

@pelegb I haven't been involved in the above testing but my preference is via Travis CI because that is where most of the puppet stuff is also run so it would be good to use the one provider. I've got this on my to-do list to look into since Travis CI does have OSX images able to run tests on but I'll open other issues or pull requests when I get around to it.

Closing due to this issue being stale. If you can still reproduce or would like to continue this discussion, please let me know and I'll reopen this.
There are plans for a better testing suite in the future so I'll reference this (and other PR's) as it comes up.