HBASE-22322 Use special pause for CallQueueTooBigException

Still have some problems on implementing UTs, will update the patch soon. And maybe we could change the config name? I think a more general reason to use a larger pause is that the region server is overloaded, CallQueueTooBigException is only one possible result of a overloaded region server. So maybe we could change the config and also the new methods in TableBuilder and AdminBuilder to something like pauseWhenOverloaded? What do you think @carp84 . Thanks.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
51
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 1 new or modified test files.

_ master Compile Tests _

0
mvndep
30
Maven dependency ordering for branch

+1
mvninstall
274
master passed

+1
compile
96
master passed

+1
checkstyle
122
master passed

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

+1
findbugs
296
master passed

+1
javadoc
54
master passed

_ Patch Compile Tests _

0
mvndep
15
Maven dependency ordering for patch

+1
mvninstall
248
the patch passed

+1
compile
76
the patch passed

+1
javac
76
the patch passed

+1
checkstyle
32
hbase-client: The patch generated 0 new + 21 unchanged - 9 fixed = 21 total (was 30)

+1
checkstyle
72
The patch passed checkstyle in hbase-server

+1
whitespace
0
The patch has no whitespace issues.

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

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

+1
findbugs
403
the patch passed

+1
javadoc
55
the patch passed

_ Other Tests _

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

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

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

15580

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-195/1/artifact/out/Dockerfile

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

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

uname
Linux 1a45d0f6bf0b 4.4.0-141-generic #167~14.04.1-Ubuntu SMP Mon Dec 10 13:20:24 UTC 2018 x86_64 GNU/Linux

Build tool
maven

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

git revision
master / 4477dd5de8

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-195/1/testReport/

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

modules
C: hbase-client hbase-server U: .

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

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

This message was automatically generated.

Maybe we could change the config name? I think a more general reason to use a larger pause is that the region server is overloaded, CallQueueTooBigException is only one possible result of an overloaded region server. So maybe we could change the config and also the new methods in TableBuilder and AdminBuilder to something like pauseWhenOverloaded?

Agreed. Actually the original config name proposed in HBASE-17114 is hbase.client.pause.special but was concerned in this comment thus changed to some more specific one to cqtbe.
Will be back and review the PR soon.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
295
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 1 new or modified test files.

_ master Compile Tests _

0
mvndep
26
Maven dependency ordering for branch

+1
mvninstall
296
master passed

+1
compile
99
master passed

+1
checkstyle
125
master passed

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

+1
findbugs
364
master passed

+1
javadoc
67
master passed

_ Patch Compile Tests _

0
mvndep
17
Maven dependency ordering for patch

+1
mvninstall
294
the patch passed

+1
compile
98
the patch passed

+1
javac
98
the patch passed

+1
checkstyle
38
hbase-client: The patch generated 0 new + 21 unchanged - 9 fixed = 21 total (was 30)

+1
checkstyle
77
The patch passed checkstyle in hbase-server

+1
whitespace
0
The patch has no whitespace issues.

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

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

+1
findbugs
382
the patch passed

+1
javadoc
66
the patch passed

_ Other Tests _

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

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

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

19747

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestAsyncTableAdminApi

hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas

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-195/2/artifact/out/Dockerfile

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

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

uname
Linux 6341fd104ff4 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 / 4477dd5de8

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-195/2/artifact/out/patch-unit-hbase-server.txt

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

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

modules
C: hbase-client hbase-server U: .

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

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

This message was automatically generated.

Agreed. Actually the original config name proposed in HBASE-17114 is hbase.client.pause.special but was concerned in this comment thus changed to some more specific one to cqtbe.

OK, can do this in another issue...


Update docstring for compute and persist

Updates the docstring for persist to mention that it is not altering the original object.
Adds a return value to the docstring for compute and persist on both Dataset and DataArray

[x] Closes #8901

@keewis maybe you can check the wording for the return values?
I am using persisted DaskArrays not sure that's a commonly used term? In the discussion #6837 you also say that it should cross-reference on the DaskArray docs so I added that as well


All infrastructure as code

Currently patroni scripts are not in the repo. We should have all infrastructure in the repo and continuously deployed and PRable

Database templates pueshed to repo.


Add ceedling webpage and other blog

TDD for embedded systems -> http://www.throwtheswitch.org/
Blog of one of the founders of throwtheswitch with TDD content and ceedling stuff -> http://vandervoord.net/

Hi @arbMerl!
Thanks for the suggestions.
If you want, you are of course welcome to create PR with your suggested changes, just let me know.

Resolved by #39.


UsageTracker

Should definitely be an extends FrayBaseVisitor<List<TrackedSymbol>>.
SymbolUsage.java:
public class SymbolUsage {
  public ParserRuleContext getSource();
}

TrackedSymbol.java:
public class TrackedSymbol {
  public String getName();
  public List<SymbolUsage> getUsages();
}

UsageTracker.java:
public class UsageTracker extends FrayBaseVisitor<List<TrackedSymbol>> {}

Ok, I lied.
But the tracker should use the above three classes.

Usage subclass include:

val read, write
method invocation
constructor invocation
interp counts as read
used as annotation
type referenced


SPM depends on a version tag without the "v"

It seems that SPM needs a version tag that isn't prefixed with "v" in order to work.
i.e. 4.0.0 not v4.0.0.  Luckily there is nothing that prevents git from having both tags if you don't want to just switch.  For the next release could you tag it this way?
swift build
Cloning https://github.com/jdhealy/PrettyColors.git
error: The dependency graph could not be satisfied. The package (https://github.com/jdhealy/PrettyColors.git) with version tag in range (4.0.0..<4.9223372036854775807.9223372036854775807) is not found. Found tags ([0.0.1, 0.0.2])

Good looking out, appreciate the reminder.
As the time of this writing, the development branch (currently on commit 6f54504)
is tracking the 3.0.X and SPM-conforming work. The release cut from that branch will have a version tag without the v.

The release cut from that branch will have a version tag without the v.

https://github.com/jdhealy/PrettyColors/releases/tag/5.0.0


Automatic Let's Encrypt TLS certificate automatic renewal

During docker compose up I received SSL certificate with the next message. The domain were replaced to sub.example.com for privacy reasons.
How can I make sure that it will be auto-renewed?
Should I add something to the docker-compose.yml or enable cronjob anywhere?
The next command shows:
sudo certbot renew --force-renewal --dry-run
No simulated renewals were attempted.

Generating a temporary self-signed certificate. This certificate will be used until a proper certificate is issued by the CA server.
plausible-ce-plausible-1            | 07:44:59.113 [notice] Use of --no-self-upgrade is deprecated.
plausible-ce-plausible-1            | Saving debug log to /var/lib/plausible/site_encrypt/certbot/acme-v02.api.letsencrypt.org/log/letsencrypt.log
plausible-ce-plausible-1            | Account registered.
plausible-ce-plausible-1            | Requesting a certificate for sub.example.com
plausible-ce-plausible-1            |
plausible-ce-plausible-1            | Successfully received certificate.
plausible-ce-plausible-1            | Certificate is saved at: /var/lib/plausible/site_encrypt/certbot/acme-v02.api.letsencrypt.org/config/live/sub.example.com/fullchain.pem
plausible-ce-plausible-1            | Key is saved at:         /var/lib/plausible/site_encrypt/certbot/acme-v02.api.letsencrypt.org/config/live/sub.example.com/privkey.pem
plausible-ce-plausible-1            | This certificate expires on 2025-01-22.
plausible-ce-plausible-1            | These files will be updated when the certificate renews.
plausible-ce-plausible-1            | NEXT STEPS:
plausible-ce-plausible-1            | - The certificate will need to be renewed before it expires. Certbot can automatically renew the certificate in the background, but you may need to take steps to enable that functionality.  It is valid until 2025-01-22. Next renewal is scheduled for 2024-12-23.

👋 @zuk22
It will be renewed automatically. The logs come from certbot that's called by Plausible on startup and 30 days before the active certificate expires. We might switch to a "native" ACME client in the future, but for now certbot seems to wirk fine.


Event merge id to xdm

Description
Supports promises being passed in options and moved eventMergeId to be part of the xdm block rather than a top-level option.
For a little background, we supported a promise being passed as an eventMergeId value. When a promise is passed, we were waiting for the promise to be resolved and then using the resolved value as the eventMergeId. Now that eventMergeId is no longer a top-level option, we didn't really want components to be digging through xdm blocks looking for eventMergeId. Preferably, components could be unaware of specific things inside xdm blocks.
I talked with @jfkhoury about this and he mentioned that he's heard customers sometimes want to pass promises for values in some of the things he's worked on. In other words, instead of just supporting promises for the eventMergeId value, why not support promises for any value inside xdm or data options? I kind of took it a step further, for better or worse, and proposed we support promises for any value in any of the options. Maybe this is a good idea, or maybe this is a really dumb idea. I did talk to Justin and he thought it was a good idea.
This code requires hitting the konductor-qe.apps-exp-edge-npe-va6.experience-edge.adobeinternal.net edge domain.

Related Issue
https://jira.corp.adobe.com/browse/CORE-33593

Motivation and Context
eventMergeId would be part of the scheme the user created, so it makes sense to pass it as part of the xdm option and not as a top-level option.

Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] All tests pass and I've made any necessary test changes.
[x] I have run the Sandbox successfully.

BTW @owenleej, I fully expect this to be a hit to performance that will show up in your reports. I think it's a reasonable benefit/cost tradeoff though.

Thanks @owenleej. I've added a test for the rejected promise.

For the logging that tells users we're waiting for promises, I'm going to push off the work to a separate PR.


Metrics not working for POJOs/Entities

It seems like the Annotations don't work for Classes, which aren't managed by Spring (like POJOs, or JPA Entities). Is this intended?

Correct, in order to instrument an object it must be a spring managed object.  Spring can do autowiring in non-managed beans by enabling Compile Time Weaving (CTW), but I don't think that Spring AOP is compatible with CTW (not positive about this).  A better option may be to use metrics-aspectj with AspectJ CTW, but again, I'm not 100% confident that they are compatible.


lk shortcut not working

When I enter lk in the browser address bar I get the Linkwarden icon and it is supposed to search within my collection, but it doesn't do anything.
Whatever I type after lk I get no result.

When I enter lk in the browser address bar I get the Linkwarden icon and it is supposed to search within my collection, but it doesn't do anything. Whatever I type after lk I get no result.

what browser are you using?

nevermind i fixed it in pr #86  so it's just waiting to merge

Still doesn't work.
Tested on firefox and chrome


Prevent dragging to current list (but not others)

Hello,
thanks for providing this library. I am trying to develop a frontend for a music player (mopidy), and using SortableJS (React component) for tracklist and library, so as to be able to reorder the tracklist, and drag items from the library into the tracklist.
Thus I was wondering if that was possible to allow dragging from the library into the tracklist, but not into the library ? (This is really a cosmetic issue, since it is easy to not have any callbacks when dragging happens inside the library).

This part of the docs is what you're looking for: https://github.com/SortableJS/Sortable#group-option
Play around with the group.pull and group.put options.
Libary options:
options = {
  group: {
    name: 'library',
    pull: [ 'tracklist' ],
    put: false
  }
}

tracklist options:
options = {
  group: {
    name: 'tracklist',
  }
}

This is really out of scope and should probably be asked on stack overflow in future.

Yes, I had read the documentation, and run with the following (almost same as you) setup:
<ReactSortable  # for the library
            group={{name: 'library', put: false, pull: ['tracklist']}}
            setList={() => console.log("This should not happen, should it ?")
...
>

<ReactSortable # for the tracklist
              group={{name: 'tracklist', pull: true, put: ['library']}}
...
>

But the sorting animation still takes place in the library, and setList is triggered all the same.

If you want to clone it, you should change group.pull to 'clone'.
Add a clone function to the props.
On Mon, 10 Feb. 2020, 9:40 pm sapristi<EMAIL_ADDRESS>wrote:

Yes, I had read the documentation, and run with the following (almost same
as you) setup:
<ReactSortable  # for the library
group={{name: 'library', put: false, pull: ['tracklist']}}
setList={() => console.log("This should not happen, should it ?")
...

<ReactSortable # for the tracklist
group={{name: 'tracklist', pull: true, put: ['library']}}
...

But the sorting animation still takes place in the library, and setList
is triggered all the same.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/SortableJS/react-sortablejs/issues/127?email_source=notifications&email_token=AHBYVFSZ427P24P6IRE24CDRCEVKNA5CNFSM4KSCGFMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELIA27I#issuecomment-584060285,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHBYVFXX33MR4RSSAAZ5MDTRCEVKNANCNFSM4KSCGFMA
.


the react-select is nice,how to convert to material-ui style

in my app the default select or autocomplete is not enough , the react-select is nice ,how to convert to material-ui style?

how to convert to material-ui style?

@iwangheng It seems like @JedWatson added some new APIs to allow customizability. I don't remember seen them some months ago.

arrowRenderer
clearRenderer
inputRenderer
menuRenderer
optionRenderer
valueRenderer

I think that we should give it a shot. We might be able to customize react-select with our component like we did with react-autosuggest and downshift. Do you want to work on a new documentation example?

i will try in my app,if it works,i would like to share


Adds ESC11, missing IF_ENFORCEENCRYPTICERTREQUEST flag

This adds a check over RRPor CSRA for the IF_ENFORCEENCRYPTICERTREQUEST flag.
See https://blog.compass-security.com/2022/11/relaying-to-ad-certificate-services-over-rpc/

@sploutchy - Thanks for this PR! It was used by my team on a previous engagement and worked perfectly.

Awesome. Thank you.


Notebook computes verification metrics through WPS on a climatology-based forecast and displays graphic results.

@huard first pass done but fails for strange reason in the WPS, as stated in #321 . Notebook is in branch "forecasting_notebook"


Clarify the map layer controls

Fix missing parens in landing page haml
Reposition the map controls
Update version and changelog

New layout of map controls:


shaderObject: Fix pipelines race condition

Closes #263

CI Vulkan-ExtensionLayer build queued with queue ID 3500.

CI Vulkan-ExtensionLayer build # 593 running.

CI Vulkan-ExtensionLayer build queued with queue ID 5263.

CI Vulkan-ExtensionLayer build # 603 running.

CI Vulkan-ExtensionLayer build # 603 failed.

CI Vulkan-ExtensionLayer build queued with queue ID 5966.

CI Vulkan-ExtensionLayer build # 621 running.

CI Vulkan-ExtensionLayer build # 621 passed.

CI Vulkan-ExtensionLayer build queued with queue ID 6060.

CI Vulkan-ExtensionLayer build # 622 running.

CI Vulkan-ExtensionLayer build # 622 passed.

CI Vulkan-ExtensionLayer build queued with queue ID 6684.

CI Vulkan-ExtensionLayer build # 628 running.

CI Vulkan-ExtensionLayer build # 628 passed.

CI Vulkan-ExtensionLayer build queued with queue ID 6768.

CI Vulkan-ExtensionLayer build # 629 running.

CI Vulkan-ExtensionLayer build # 629 passed.


is there any problem in future to use this library?

I guess not. This library is free to use. It just lacks some new features of newer version of official OPUS library.


Prevent showing 'Manage' button to the mentor in "MENTEE_APPLICATION" state

Describe the bug

MENTEE_APPLICATION period showing a manage button for mentors (in my programs)

To Reproduce

Add a condition to prevent showing the button.

Expected behaviour
A clear and concise description of what you expected to happen.

'Manage' button shouldn't appear to mentors in the "MENTEE_APPLICATION" state.

Desktop (please complete the following information):

OS: MacOS BigSur
Browser : Safari
Version : 14.0.1

@anjula-sack, can you assign me for this issue

Go ahead


no-floating-effects rule

What is the problem this feature would solve?
Background
typescript-eslint has a rule called no-floating-promises which requires all promises to be handled (either via return or await) or explicitly ignored (via void).
It would be nice to be able to get a lint warning when leaving an Effect hanging.
What is the feature you are proposing to solve the problem?
A new rule to detect floating Effects could warn on the following circumstances:
❌ Incorrect
Effect.gen(function* () {
  Console.log('foo');
//^ floating Effect
});

const program = pipe(
  Effect.succeed('foo'),
  Effect.tap(v => {
    Console.log(v);
//  ^ floating Effect
  }),
);

✅ Correct
Effect.gen(function* () {
  yield* Console.log('foo');
});

const program = pipe(
  Effect.succeed('foo'),
  Effect.tap(Console.log),
);

What alternatives have you considered?
No response

This would be fantastic, just wanted to enthusiastically voice my support! 🙂

I would love tho have this. But I understand it's tricky to implement


Added Real Python reference to Descriptors

Added Real Python reference to Descriptors in the sub-section "Commonly Used Knowledge"

Adds @YaelD as a contributor for content.

@all-contributors please add @YaelD for content


syscall: mksyscall_windows.go doesn't handle goroot correctly

What version of Go are you using (go version)?
go version devel +1d2ca9e Mon Jul 18 21:07:34 2016 +0000 windows/amd64

What operating system and processor architecture are you using (go env)?
windows/amd64

What did you do?
set GOROOT=c:\dev\go
set GOPATH=c:\dev\gopath
go generate with //go:generate go run $GOROOT/src/syscall/mksyscall_windows.go -output zsyscall_windows.go syscall_windows.go

What did you expect to see?
mksyscall_windows.go doesn't put import internal/syscall/windows because it's not a standard package, or x/sys package.

What did you see instead?
I can see import internal/syscall/windows in the code.

https://github.com/golang/go/blob/4cffe44e361deb39e3274774a7984ab78a5b3931/src/syscall/mksyscall_windows.go#L698-L712
goroot is checked without delimiter.

What did you do?
set GOROOT=c:\dev\go
set GOPATH=c:\dev\gopath
go generate with //go:generate go run $GOROOT/src/syscall/mksyscall_windows.go -output zsyscall_windows.go syscall_windows.go

What did you do? Did you run "go generate" command? Which directory?
I don't understand what the problem is. Please provide more details. Thank you.
Alex

Few days ago, I contributed some patches for https://github.com/tompng/pokemon-go .
I modified pokemon-go to use https://github.com/jteeuwen/go-bindata .
In my configuration, GOROOT and GOPATH have same prefix part C:\dev\go but not same directory as you can see in above. But mksyscall_windows.go handle those as same. So the code made by mksyscall_windows.go are generated as pkgStd because IsStdRepo doesn't check trailing path separator as delimiter.
If I have backslash in GOPATH and GOROOT, it will works fine. But official installation instruction doesn't have trailing slash/backslash in the environment variables.

Few days ago, I contributed some patches for https://github.com/tompng/pokemon-go .
I modified pokemon-go to use https://github.com/jteeuwen/go-bindata .

I checked github.com/tompng/pokemon-go package. It does not use mksyscall_windows.go. Why are you telling me about github.com/tompng/pokemon-go package?

In my configuration, GOROOT and GOPATH have same prefix part C:\dev\go but not same directory as you can see in above. But mksyscall_windows.go handle those as same. So the code made by mksyscall_windows.go are generated as pkgStd because IsStdRepo doesn't check trailing path separator as delimiter.

You keep repeating yourself. I understanding that you think there is a bug here. And I don't disagree with you. But I would like to see the bug by myself. Please provide some code or commands to demonstrate the bug. Show us some real computer output, and explain why that output is wrong in your opinion.
Thank you.
Alex

I checked github.com/tompng/pokemon-go package. It does not use mksyscall_windows.go. Why are you telling me about github.com/tompng/pokemon-go package?

Sorry much! I was confusing. True repository is my private for my job. Please check following code.
foo.go is below:
package foo

//go:generate go run $GOROOT/src/syscall/mksyscall_windows.go -output bar.go foo.go

//sys	GetLastError() (lasterr error)

C:\dev\godev\src\github.com\mattn\foo>go generate

and go generate. bar.go is following.
// MACHINE GENERATED BY 'go generate' COMMAND; DO NOT EDIT

package foo

import (
	"internal/syscall/windows/sysdll"
	"syscall"
	"unsafe"
)

var _ unsafe.Pointer

var (
	modkernel32 = syscall.NewLazyDLL(sysdll.Add("kernel32.dll"))

	procGetLastError = modkernel32.NewProc("GetLastError")
)

func GetLastError() (lasterr error) {
	r0, _, _ := syscall.Syscall(procGetLastError.Addr(), 0, 0, 0, 0)
	if r0 != 0 {
		lasterr = syscall.Errno(r0)
	}
	return
}

internal/syscall/windows/sysdll should not be generated.
P.S. I must say sorry again.

Please check following code.

Thanks for the test.
Alex


should ingress support wrr algorithm

In my case, I want to support canary release/gray release. Control client traffic to Version A and Version B accordingly.
I think currently nginx ingress do not support.

Closing. This is not part of the ingress spec (yet) https://github.com/kubernetes/kubernetes/issues/25485


WurstSetup Crash on First Use

Please report this crash with the following info:
Version: <IP_ADDRESS>-jenkins-WurstSetup-85
java.lang.ExceptionInInitializerError
	at ui.MainWindow$UI.createConfigTable(MainWindow.kt:315)
	at ui.MainWindow$UI.initComponents(MainWindow.kt:150)
	at ui.MainWindow.init(MainWindow.kt:68)
	at ui.UiManager$initUI$1.run(UiManager.kt:13)
	at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:313)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:770)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
	at java.base/java.security.AccessController.doPrivileged(Native Method)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:740)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)
Caused by: java.lang.RuntimeException: java.lang.NoSuchMethodException: java.util.prefs.WindowsPreferences.WindowsRegOpenKey(int, [B, int)
	at global.WinRegistry.<clinit>(WinRegistry.kt:51)
	... 17 more
Caused by: java.lang.NoSuchMethodException: java.util.prefs.WindowsPreferences.WindowsRegOpenKey(int, [B, int)
	at java.base/java.lang.Class.getDeclaredMethod(Class.java:2476)
	at global.WinRegistry.<clinit>(WinRegistry.kt:30)
	... 17 more

Tried in Downloads and Documents folders and still get this error. Uninstalled an old one since it's crashing with a different error as well. Java 1.8.0_172.

There are some issues with the old WinRegistry implementation with newer java versions/systems.
It's addressed by #26 which I will hopefully merge this weekend.

The same problem, but another message:
Version: <IP_ADDRESS>-jenkins-WurstSetup-91
java.lang.IllegalStateException: key.getValueByName("InstallPath") must not be null
	at ui.MainWindow$UI.createConfigTable(MainWindow.kt:312)
	at ui.MainWindow$UI.initComponents(MainWindow.kt:153)
	at ui.MainWindow.init(MainWindow.kt:70)
	at ui.UiManager$initUI$1.run(UiManager.kt:12)
	at java.awt.event.InvocationEvent.dispatch(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$500(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)```

Should be fixed, but I still wonder why you don't have registry entries. Did you not install wc3 conventionally?

Should be fixed, but I still wonder why you don't have registry entries. Did you not install wc3 conventionally?
At now it's ok. Of course wc3 installed correctly.


Hard to build from source

I was trying to build https://github.com/rakudo/rakudo/. The readme page says that I can download MoarVM and nqp myself.

I built moar easily.
Then I tried to build nqp. It failed at "cannot get MoarVM version".

Turns out moar --version says `This is MoarVM version  built with JIT support.

Fixed the version VERSION=(cat VERSION ) perl Configure.pl --prefix=xxxxx. However, NQP still refuse to build. (version too old, need 2022.06-2-gcc49dcf93)
Deleted the version check manually. (nqp/tools/lib/NQP/Config/NQP.pm line 393). Rakudo still can't find nqp.
Ran VERSION=(cat VERSION ) perl Configure.pl --prefix=xxxxxx --git-cache-dir=.. in Rakudo repo.  Says NQP has no version.
Gave NQP VERSION 2022.06-3 and git tag, Rakudo still refuse to build. Says the version isn't right.
Modified rakudo/tools/lib/NQP/Config/Rakudo.pm line 651. Rakudo still says it need git tag.
Ran git tag xxxx for Rakudo. Finally it build.

Request: Use feature detection, not version number.
Another pain point is that there is no easy --with-moar when building rakudo.

I want to contribute to Rakudo, but must I use Perl 5?

I'm not quite sure why you're having all this trouble. For me, the simplest way to build Rakudo from source is running perl Configure.pl --prefix=/my/raku/install/ --backend=moar --gen-moar --make-install in the Rakudo directory. If I want to build the three components separately, I run perl Configure.pl --prefix=/my/raku/install/ && make -j12 install in the MoarVM repo, then perl Configure.pl --prefix=/my/raku/install/ --backend=moar && make -j12 install in the NQP repo, and finally perl Configure.pl --prefix=/my/raku/install/ --backend=moar && make -j12 install in the Rakudo repo.

Another pain point is that there is no easy --with-moar when building rakudo.

Yeah, maybe that should be added. However, NQP does have that option, and you need an NQP install to build Rakudo, so I can see the logic of why it's left out for Rakudo.

I want to contribute to Rakudo, but must I use Perl 5?

Use as in write Perl 5? No, not unless you want to make any changes to the build process. Use as in have a Perl 5 binary installed? Yes, it's readily available for all the platforms Rakudo supports, so makes bootstrapping easy.

Why doesn't Rakudo use viv to translate to perl 5?

It's way too outdated. Also, I'm not sure why you'd want it? For the Rakudo build process? Or just in general?

I don't want to download the git repo multiple times. I can't use --gen-moar.
When building Rakudo, --git-depth=1 isn't fully respected. It fetches nqp bootstrapping repo, which isn't necessary.

The instructions for installing the Rakudo compiler from scratch on windows don't seem to work: There is no nqp directory as outlined with the instructions at https://rakudo.org/downloads/rakudo/source for the "Bash on Windows" section.

Those instructions are way overcomplicated. Looks like they rely on doing lots of steps manually that are already handled by Configure.pl. I don't use WSL, I use 'git bash', but I imagine they should be similar.
I'll put together a cleaned up version in a day or so and post it here, and we can decide what we want to be on the website.
Question: do you have a version of visual studio installed? git bash?

@coke OK, I don't remember any details of my experience getting it installed and I don't typically use Windows. But let me know if you want me to try out your new instructions to see if they work for me on a fresh install of Windows 11.

My initial take is that there are several flavors of possible installs here, including:

rakubrew
git & gcc
git & visual studio
tarballs & gcc
tarballs & visual studio

This becomes a very large exercise, might be worth having just the options on the main page and linking to each individual set of instructions.

https://gist.github.com/coke/f6488d106d4a580216809f3e69b553f5 with the git/gcc directions so far.
Feedback welcome, we can probably move these to the github repo's rakudo wiki.


[mips/mips64: msa] add add_a_b intrinsic

This PR adds a single mips::msa intrinsic (add_a_b: vector add absolute values).
It adds all the scaffolding for adding more mips/mips64 intrinsics "easily" in the future.

adds an arch::mips64 module
generates documentation for mips/mips64
adds run-time feature detection for mips/mips64 (auxiliary vectors only)
adds simd_test support for mips/mips64 (requires parsing instructions as expressions instead of identifiers because add_a.b is a valid MIPS64 instruction name)

The following issues are left for future PRs:

adding CI for MIPS: #364
adding synthetic tests using the auxval crate: #355
adding cpuinfo support: #354

@alexcrichton I cleaned up the run-time feature detection on linux a bit because it was getting out-of-hand. I have no idea whether this might break rustc builds or not. Is there a way for me to test this? If these changes break some build bots there, we should add these build bots to our CI as well.
cc @jcowgill

@alexcrichton So I think I managed to clean up cfg-hell at least a bit. The gist in a nutshell is:
stdsimd/detect/ now has two sub-directories:

stdsimd/detect/arch/{target_arch}.rs: define architecture features, that is, the Feature enum and the is_{target_arch}_feature_detected! macros, nothing more.
stdsimd/detect/os/{target_os}.rs: defines the check_for function, that is, how features are actually detected on the different platforms

I don't think this is great, but I think it is better than what we had. Now, cpuinfo, auxvec, and the cache, are only defined on the platforms that actually need them. For example, on non-x86 non-linux platforms, we don't need the cache, because we can't detect any features anyways.

Yes we will need to do so to add windows on arm support.
On Sat 10. Mar 2018 at 19:23, Alex Crichton<EMAIL_ADDRESS>wrote:

Alright looks good to me, thanks! We can always of course continue to
tweak this over time
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang-nursery/stdsimd/pull/365#issuecomment-372052322,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3Npo8ZPBn1KcLSM9jzJkSOAw6bNDumks5tdBoNgaJpZM4SkamW
.


support twikoo comment system

Hi @OnlyFor. The current version of this theme (v0.2.0) already contains your changes. 🙂


Old style exception --> new style for Python 2 & 3

Old style exceptions are a syntax error in Python 3 while new style exceptions work in both Python 2 and Python 3.

Thk U 👍


Capitalize Y in Yoga for pod file.

If I don't capitalize the Y in "yoga" in the Podfile I receive these warnings:
Fetching podspec for `yoga` from `../node_modules/react-native/ReactCommon/yoga`
[!] The name of the given podspec `Yoga` doesn't match the expected one `yoga`

I know that in the past the "Y" was capitalized, but somewhere along the way it looks like it got upcased. Ignore this if it is meant to be upcased moving forward.

This appears intentional: https://github.com/zibs/react-native/commit/b050b6906d6701ae26fe683d0e9451d0fb9c0543#diff-b43bc13efab326925ef51e0cd31e2f0e
Can you check with @rigdern first?

I can reproduce error on my pod update after upgrading RN to 0.49.1:
Analyzing dependencies
Fetching podspec for `RNFirebase` from `../node_modules/react-native-firebase`
Fetching podspec for `React` from `../node_modules/react-native`
Fetching podspec for `Yoga` from `../node_modules/react-native/ReactCommon/yoga`
[!] The name of the given podspec `yoga` doesn't match the expected one `Yoga`

And it quits here.

@zibs What version of React Native were you using when you saw this warning?

@rigdern I was using 0.48.3

Yoga was lowercased to "yoga" in RN 0.49. With 0.49, I believe "yoga" is the correct thing to specify in the Podfile and "Yoga" will cause an error.

I shall close then! Thanks y'all.


csvHeaders, domainClassess

could you send more information about your new proposal features?

@paulhodel I've been working with 2.1.0 recently and this seems to be an issue still. Sure I can add .jexcel to external modal, but as long as I use the default jexcel stylesheet, there will be tons of styles that are designed for jexcel but not really for the external modal, which results in a broken page. I can confirm that adding .jexcel to Bootstrap modals would not work. It should really just be a separate class that is not referred anywhere in the jexcel code besides that one line determining if the modal is within the domain.


Add zero check for string bytes

Add a fix for IntercomDDS sending an empty string with no null terminator in some cases.
Without this fix, an empty string will cause a panic

Thank you for the fix.
I reworded the patch in terms of std library function split_last.


docker update memory throws error for docker:dind with nested container

Expected:
When updating a privileged docker:dind container that has a nested container within it, the outer privileged docker:dind container should successfully change it's memory (and other allowed host config parameters).
Actual Behavior:
Docker update command throws the following error:
Error response from daemon: Cannot update container be875422d297286d9cb7c3f9febb96fcaf84e4fabb5b0d46e90afd2d70ec6fcf: runc did not terminate sucessfully: failed to write "a : rwm" to "/sys/fs/cgroup/devices/docker/be875422d297286d9cb7c3f9febb96fcaf84e4fabb5b0d46e90afd2d70ec6fcf/devices.allow": write /sys/fs/cgroup/devices/docker/be875422d297286d9cb7c3f9febb96fcaf84e4fabb5b0d46e90afd2d70ec6fcf/devices.allow: invalid argument
: unknown
Repro:
# start outer dind container
docker run -m 2G --name outerdind docker:dind

# go into the container
docker exec -it outerdind /bin/sh

# start a container in the background in an infinite loop
docker run -d alpine sh -c "while true; do sleep 1; done"

# Exit shell of container to go back to host shell
# ....
#

# Update the outer dind container's memory
docker update -m 1G outerdind

# The following error is thrown

Error response from daemon: Cannot update container be875422d297286d9cb7c3f9febb96fcaf84e4fabb5b0d46e90afd2d70ec6fcf: runc did not terminate sucessfully: failed to write "a *:* rwm" to "/sys/fs/cgroup/devices/docker/be875422d297286d9cb7c3f9febb96fcaf84e4fabb5b0d46e90afd2d70ec6fcf/devices.allow": write /sys/fs/cgroup/devices/docker/be875422d297286d9cb7c3f9febb96fcaf84e4fabb5b0d46e90afd2d70ec6fcf/devices.allow: invalid argument
: unknown

Troubleshooting and Attempts to Diagnose
What is very interesting is that the outer dind container successfully updates when you have not yet started an inner container within it (disappointingly, this does not solve my use-case because I need to have the inner container started when updating the outer container).
As a side note, I cannot determine at this point if this is a dind issue or a moby issue (so please let me know)
Docker Version info
Client: Docker Engine - Community
 Version:           19.03.8
 API version:       1.40
 Go version:        go1.12.17
 Git commit:        afacb8b
 Built:             Wed Mar 11 01:21:11 2020
 OS/Arch:           darwin/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.8
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.17
  Git commit:       afacb8b
  Built:            Wed Mar 11 01:29:16 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          v1.2.13
  GitCommit:        7ad184331fa3e55e52b890ea95e65ba581ae3429
 runc:
  Version:          1.0.0-rc10
  GitCommit:        dc9208a3303feef5b3839f4323d9beb36df0a9dd
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

I'm not positive, but I think what's happening here is that your inner container gets initialized with the memory limit of the Docker-in-Docker instance, and thus because that cgroup now has a child with that limit the limit of the parent can no longer be lowered.  Can you raise the limit?  Is the behavior any different if you start the Docker-in-Docker container with a limit that's smaller than the limit you want to update the outer container to?
(It sounds like this is likely a kernel cgroups implementation limitation, not really something we could fix in either Docker or the image.)

I'm not positive, but I think what's happening here is that your inner container gets initialized with the memory limit of the Docker-in-Docker instance, and thus because that cgroup now has a child with that limit the limit of the parent can no longer be lowered.  Can you raise the limit?  Is the behavior any different if you start the Docker-in-Docker container with a limit that's smaller than the limit you want to update the outer container to?
(It sounds like this is likely a kernel cgroups implementation limitation, not really something we could fix in either Docker or the image.)

Raising the limit yields the same error disappointingly. This issue also occurs when attempting to update any of the host config options: MemoryReservation, MemorySwap, NanoCPUs (for reference, I've tried them all individually, as well as different combinations, as well as increasing/decreasing values).
What's even more interesting, since I have the ability with my architecture to swap out dind with an equivalent feature-set using sysbox as the isolation method, sysbox actually throws a similar error:
ecs_manager_test_backend | Error: (HTTP code 500) server error - Cannot update container d97504278bad736f06a92ed1b265ffd901a137cef673cb95a05106ac63efbd0a: /usr/local/sbin/sysbox-runc did not terminate successfully: failed to write a : rwm to devices.deny: write /sys/fs/cgroup/devices/docker/d97504278bad736f06a92ed1b265ffd901a137cef673cb95a05106ac63efbd0a/devices.deny: invalid argument
I'm not quite sure what to try next - any suggestions you have, I will be most willing to try. It almost seems like a weird permissions issue, noting the devices.deny info from the above message. From a semantic standpoint, it seems like a common usage pattern to contain a set of containers within an outer container, allowing you to holistically limit the resources of all inner containers that an end-user would attempt to spawn within their inner playground. Thank you again for working with me on this!

Raising the limit yields the same error disappointingly. This issue also occurs when attempting to update any of the host config options: MemoryReservation, MemorySwap, NanoCPUs (for reference, I've tried them all individually, as well as different combinations, as well as increasing/decreasing values).
What's even more interesting, since I have the ability with my architecture to swap out dind with an equivalent feature-set using sysbox as the isolation method, sysbox actually throws a similar error:
ecs_manager_test_backend | Error: (HTTP code 500) server error - Cannot update container d97504278bad736f06a92ed1b265ffd901a137cef673cb95a05106ac63efbd0a: /usr/local/sbin/sysbox-runc did not terminate successfully: failed to write a : rwm to devices.deny: write /sys/fs/cgroup/devices/docker/d97504278bad736f06a92ed1b265ffd901a137cef673cb95a05106ac63efbd0a/devices.deny: invalid argument
I'm not quite sure what to try next - any suggestions you have, I will be most willing to try. It almost seems like a weird permissions issue, noting the devices.deny info from the above message. From a semantic standpoint, it seems like a common usage pattern to contain a set of containers within an outer container, allowing you to holistically limit the resources of all inner containers that an end-user would attempt to spawn within their inner playground. Thank you again for working with me on this!

On the one hand, yes, but on the other I'm actually really confused why you're creating this environment and then trying to decrease the memory available to it after-the-fact?  What's the use case for the docker update and not simply starting with the intended final limit?
(Even if this can be made to work, I think the behavior of it would be kind of sketchy/flaky - what does it do if processes are over the limit currently when you lower it?  It'd probably just fall back to a standard OOM kill which seems less than ideal?)

On the one hand, yes, but on the other I'm actually really confused why you're creating this environment and then trying to decrease the memory available to it after-the-fact?  What's the use case for the docker update and not simply starting with the intended final limit?
(Even if this can be made to work, I think the behavior of it would be kind of sketchy/flaky - what does it do if processes are over the limit currently when you lower it?  It'd probably just fall back to a standard OOM kill which seems less than ideal?)

I appreciate you asking the question - it probably would help to provide context.  I'm in the last phase of building an entire kubernetes-like architecture that has specific features that kubernetes does not support (the "updating container limits on-the-fly without killing/rotating/etc" is one of those features, which is a resulting feature-set to support the feature to preload containers into pools as a significant processing performance boost.)
A specific example would be a user wants to perform some sort of remote code execution, but the libraries that are required for their given code take 1 minute to load into memory - this can simply be solved with a pooling algorithm, but any type of pooling would require containers to be already running. Upon queuing a task to be executed, you as the administrator of the architecture can configurably provide outer container limits on demand to the given container that will immediately be picking up the task from the queue to execute. (there are a lot of other moving parts, but I've tried to keep it as to-the-point as possible). Let me know if that helps to understand the overall goal.

I appreciate you asking the question - it probably would help to provide context.  I'm in the last phase of building an entire kubernetes-like architecture that has specific features that kubernetes does not support (the "updating container limits on-the-fly without killing/rotating/etc" is one of those features, which is a resulting feature-set to support the feature to preload containers into pools as a significant processing performance boost.)
A specific example would be a user wants to perform some sort of remote code execution, but the libraries that are required for their given code take 1 minute to load into memory - this can simply be solved with a pooling algorithm, but any type of pooling would require containers to be already running. Upon queuing a task to be executed, you as the administrator of the architecture can configurably provide outer container limits on demand to the given container that will immediately be picking up the task from the queue to execute. (there are a lot of other moving parts, but I've tried to keep it as to-the-point as possible). Let me know if that helps to understand the overall goal.

Honestly, it sounds like you should probably be building on top of containerd and using containerd namespaces for logical separation/grouping -- Docker-in-Docker is really a hack (definitely not a security boundary), and containerd was purpose-designed to be the exact underlying container substrate for a larger orchestration system that you seem to be looking for.

Honestly, it sounds like you should probably be building on top of containerd and using containerd namespaces for logical separation/grouping -- Docker-in-Docker is really a hack (definitely not a security boundary), and containerd was purpose-designed to be the exact underlying container substrate for a larger orchestration system that you seem to be looking for.


READ-SUPPRESS.17

I believe the following should not fail:
(in-package :eclector.reader)
(multiple-value-list
              (WITH-STANDARD-IO-SYNTAX
                (LET ((*READ-SUPPRESS* T))
                  (READ-FROM-STRING "#garbage"))))
->
debugger invoked on a ECLECTOR.READTABLE:UNKNOWN-MACRO-SUB-CHARACTER in thread #<THREAD "main thread" RUNNING {10005205B3}>: g is not a sub-character of the dispatch macro character #.

I agree that ignoring an expression such as #garbage when *read-suppress* is true could be useful, but I see two problems:

#g could initiate an arbitrary syntax so it is not clear how skip over it.

The entry Entry for Variable *read-suppress* says

Dispatching macro characters (including sharpsign)
Dispatching macro characters continue to parse an infix numerical argument, and invoke the dispatch function. The standardized sharpsign reader macros do not enforce any constraints on either the presence of or the value of the numerical argument.

I take this to mean that the dispatch function is invoked and either calls a user-supplied function if the user has installed one in the readtable or calls a standard macro function or signals an error according to the table in 2.4.8 Sharpsign.

Did you find passages implying the behavior you propose?

From testing a few implementations in seems ABCL, CMUCL, SBCL skip over the token, and CCL, Clisp, ECL reject it.

See https://irclog.tymoon.eu/freenode/%23sicl?around=1551169235#1551169235


PeerTube icon

Thought I'd add another Fediverse Icon! :smile:

Looks good! Mind if we shrink the path data a little bit first though? Try this for the triangles:
<path fill="#211f20" d="M140.8,102.4V256Z"/><path fill="#737373" d="M140.8,256V409.6Z"/><path fill="#f1680d" d="M256,179.2V332.8Z"/>

I've removed spaces from the path definitions, but your suggestion won't work I think as it just draws a single vertical line for each triangle and then closes the path - it doesn't put the right most point in.

<path fill="#211f20" d="M140.8,102.4V256Z"/><path fill="#737373" d="M140.8,256V409.6Z"/><path fill="#f1680d" d="M256,179.2V332.8Z"/>


Camera/Video black screen

First of all, thx for updating the plugin. I wanted to ask if someone else have the issues with black screen video (when receive from a browser) and sending no video to other devices. I have this issue on both my iPhone 12 and Oneplus 9.

Hi @xtlHung.
I had the same problem. I solved it by forcing version 0.0.5 in pubspec.yaml.
jitsi_meet_wrapper: 0.0.5

Hey @FlavioCarnevale thanks for the info. But I need an updated version with somewhat newer JitsiMeetSDK. Im currently using another fork and it works fine.

I have the same problem with Jist_meet_wrapper:0.0.5
Please help me :)

@saibotma,  not solve , camera and screen sharing still showing black.  what is solution for it ?

Will take a look at this eventually, but currently I have no resources.

Hey there,
please try the newest release and create a reproducible example if it still does not work.


Final touches on glTF serializer rework

Just a couple of last things related to the recent glTF serializer rework.

Additional tests
Update index.ts to exclude internals (minus some functions from GLTFMaterialExporter-- need them for integration tests)

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15972/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15972/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15972/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15972/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15972/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15972/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15972/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15972/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15972/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15972/merge/testResults/webgpuplaywright/index.html

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15972/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15972/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.


Update twitter handle on README.md

Looks like you changed the Twitter handle and forgot to update it here :)

Note that the handles listed on the KeyboardKit organization and in KeyboardKitPro also need to be updated.

Thank you!


upd(discord-canary): 0.0.134 -> 0.0.135

Updated from 0.0.134 to 0.0.135 and add a repology variable.

Also the original maintainer, @aranym1 seems to have deleted his github account, so shall i take over this package?

Also the original maintainer, @aranym1 seems to have deleted his github account, so shall i take over this package?

If you want to, yes

Changed the maintainer.


Consider removing Send + Sync requirements.

All the data in the World currently requires Send + Sync bounds. This was done so that #4 could be implemented without a breaking change. However, I'm not sure I want this for a few reasons:

Performance: Users must use Arc and Mutex when Rc and RefCell is sufficient.
Send + Sync bounds make it difficult to work with frameworks like ggez and sdl2 that have !Send + !Sync requirements.
The multithreaded executor as currently implemented in bevy_ecs has very significant overhead, to the point where many games can improve their performance by disabling it. This may be sufficient evidence that inter-handler parallelism is not worth implementing.
Parallel iterators for Fetcher may be sufficient for most users.

I would rather not handicap this library indefinitely in order to support a feature which may not actually arrive.

https://github.com/kuviman/evenio/tree/non-send

I'm all for removing Send + Sync and then having a breaking change if needed. :)

@spikespaz fyi non-send needs to work with rayon feature


Add settings for limiting PostgreSQL MaxOpenConns and MaxIdleConns

Is this a bug or a feature request?
Feature request.
PoC here, tried this and it solved the problem: github/kimmokh/postgresql-connections-setting
What did you expect?
Our test-case running on local Docker-compose stack should be able add 200+ devices to lora-app-server without a problem.
What happened?
PostgreSQL ran out of connections during the test, most of the devices were not added.
PostgreSQL has by default 100 connections, whereas Go SQL connection pool has no upper limit (unless it is explicitly set) (see Go connection pool)
What version are you are using?
3.2.0
How can your issue be reproduced?
By adding 100+ devices in rapid succession.
PostgreSQL connections are depleted even faster if using Amazon RDS (default about 40 connections on a small EC2 instance) or when limiting the count using PgBouncer.
Could you share your log output?
...
lora-app-server       | time="2019-09-06T13:33:23Z" level=info msg="finished client unary call" grpc.code=OK grpc.method=CreateDevice grpc.service=ns.NetworkServerService grpc.time_ms=866.444 span.kind=client system=grpc
lora-app-server       | time="2019-09-06T13:33:23Z" level=info msg="device created" dev_eui=4846f19900201271
lora-app-server       | time="2019-09-06T13:33:23Z" level=error msg="sync remote multicast setup error" error="storage: begin transaction error: pq: remaining connection slots are reserved for non-replication superuser connections"
postgresql            | FATAL:  sorry, too many clients already
postgresql            | FATAL:  sorry, too many clients already
postgresql            | FATAL:  sorry, too many clients already
postgresql            | FATAL:  sorry, too many clients already
postgresql            | FATAL:  sorry, too many clients already
lora-app-server       | time="2019-09-06T13:33:23Z" level=info msg="finished unary call with code OK" grpc.code=OK grpc.method=Create grpc.service=api.DeviceService grpc.start_time="2019-09-06T13:33:23Z" grpc.time_ms=345.52 peer.address="<IP_ADDRESS>:46766" span.kind=server system=grpc
postgresql            | FATAL:  sorry, too many clients already
postgresql            | FATAL:  sorry, too many clients already
postgresql            | FATAL:  sorry, too many clients already
...

This has been merged.


AFP_NAME doesn't seem to work anymore

I upgraded to your latest recently (from last July) and my TM share name changed to simply 'tm'. I am setting AFP_NAME to 'TimeMachine'.

Here there is an error in fact. As I can see here and here, we are using AFP_NAME as the mountpoint, which should be an absolute path.
I'll fix this this week (it's a bit busy one). If you just avoid using env variables and just use the command line option, it should be fine, but the server name will appear as your user 🤔

Ok, thanks.  I'm using docker-compose, hence the need for ENVs.  Happy to test when you have a fix, and I appreciate the image.

Sorry for the delay. It has been a tough June.
The image has been updated so when using environment variables the mountpoint selected is always the root of the external volume. And now the AFP_NAME attribute do it's work as expected :)
Does it solve your issue?

Closing the issue. Please ping me if it's not solved.

Hi -- yes, the latest image seems to have fixed this.  I am able to backup to it just fine, thank you.


Question: Create Observable and corresponding Observer at the same time

I have a class that implements a delegate protocol for a custom DatePicker which gets called whenever the selected date changes.
Now I want to expose the current date as an Observable.
With ReactiveCocoa I created a Signal and Sink in the constructor, exposing the Signal to the outside and keeping the Sink private to publish new values.
The closest I came with RxSwift was creating a PublishSubject which I exposed as Observable and also kept internally to sendNext new values.
Is this the best approach? I would rather just expose a pure Observable, such that clients can not type-cast back to a PublishSubject and start sending values (though this is only a theoretical problem).

Hi,
regarding wrapping delegates, RXS has generic delegate wrapping functionality.
This is how we wrapped UISearchBar in RxCocoa
class RxSearchBarDelegateProxy : DelegateProxy
                               , UISearchBarDelegate
                               , DelegateProxyType {

    class func currentDelegateFor(object: AnyObject) -> AnyObject? {
        let searchBar: UISearchBar = castOrFatalError(object)
        return searchBar.delegate
    }

    class func setCurrentDelegate(delegate: AnyObject?, toObject object: AnyObject) {
        let searchBar: UISearchBar = castOrFatalError(object)
        searchBar.delegate = castOptionalOrFatalError(delegate)
    }
}

extension UISearchBar {

    public var rx_delegate: DelegateProxy {
        return proxyForObject(self) as RxSearchBarDelegateProxy
    }

    public var rx_searchText: Observable<String> {
        return defer { [weak self] in
            let text = self?.text ?? ""

            return self?.rx_delegate.observe("searchBar:textDidChange:") ?? empty()
                    >- map { a in
                        return a[1] as? String ?? ""
                    }
                    >- startWith(text)
        }
    }
}

If you want to hide type of any observable, you can use asObservable operator.
return original
     >- asObservable

Hope this helps.


Add float: footnote to CSS3

In http://www.w3.org/TR/css3-gcpm/ float:footnote is described. It would be cool to see which browser support this feature.

Subfeature of #1510.

Deep link is https://www.w3.org/TR/css-gcpm-3/#footnotes

+1


Add version filter to ATT&CK/CAPEC/CWE endpoints

It is not immediately clear what version of ATT&CK objects is returned (user can import different versions).
Should allow user to filter on version (stix2arango_note) property. e.g. v15.1.
Default should be is_latest objects
Endpoints affected:
/api/v1/attack-enterprise/
/api/v1/attack-enterprise/{stix_id}/
/api/v1/attack-ics/
/api/v1/attack-ics/{stix_id}/
/api/v1/attack-mobile/
/api/v1/attack-mobile/{stix_id}/
/api/v1/capec/
/api/v1/capec/{stix_id}/
/api/v1/cwe/
/api/v1/cwe/{stix_id}/

Parameter should have names attack_enterprise_version, attack_ics_version, attack_mobile_version, capec_version, cwe_version.
Description: Filter the results by the version of <NAME e.g. ATT&CK Enterprise>

Version filter still missing from
/api/v1/attack-enterprise/{stix_id}/
/api/v1/attack-ics/{stix_id}/
/api/v1/attack-mobile/{stix_id}/
/api/v1/capec/{stix_id}/
/api/v1/cwe/{stix_id}/


Weave NPC Container Crashes - set with the same name already exists

What you expected to happen?
Weave should not keep crashing
What happened?

➜  ~ kprod -n kube-system logs --tail 3 -p  weave-net-5z498 weave-npc
INFO: 2019/10/11 21:50:46.683769 EVENT AddNetworkPolicy {"metadata":{"creationTimestamp":"2019-10-11T21:46:21Z","generation":1,"labels":{"ship-update":"6242c5e66f4a3b052f0cd819deb3faca","shipcloud-role":"update"},"name":"shipupdate-6242c5e66f4a3b052f0cd819deb3faca","namespace":"shipupdate-6242c5e66f4a3b052f0cd819deb3faca","resourceVersion":"89569393","selfLink":"/apis/networking.k8s.io/v1/namespaces/shipupdate-6242c5e66f4a3b052f0cd819deb3faca/networkpolicies/shipupdate-6242c5e66f4a3b052f0cd819deb3faca","uid":"903ac49c-ec70-11e9-b528-0ea30c65b9c4"},"spec":{"egress":[{"to":[{"ipBlock":{"cidr":"<IP_ADDRESS>/0","except":["<IP_ADDRESS>/32"]}}]}],"podSelector":{},"policyTypes":["Egress"]}}
FATA: 2019/10/11 21:50:46.684610 ipset [create weave-JrM~x^cy3PiwSz=_FjxOZ4iPn hash:ip comment] failed: ipset v6.32: Set cannot be created: set with the same name already exists
: exit status 1

Weave daemonset with one pod crashing:
➜  ship-operator git:(master) ✗ kprod -n kube-system get pods | grep weave
weave-net-56pwp                              2/2     Running   2          9d
weave-net-5m9jk                              2/2     Running   2          9d
weave-net-5z498                              2/2     Running   26         3h15m
weave-net-787fr                              2/2     Running   2          9d
weave-net-hql7b                              2/2     Running   2          7d23h
weave-net-jfjzg                              2/2     Running   7          8d
weave-net-k9h7j                              2/2     Running   3          9d
weave-net-qckdk                              2/2     Running   2          9d
weave-net-tt6bm                              2/2     Running   2          9d

How to reproduce it?
I am not sure
Anything else we need to know?

Amazon EKS

This looks very similar to https://github.com/weaveworks/weave/issues/3247 but seems to be resolved in 2.2.1.
Please let me know what additional details would be helpful? Iptables and weave logs are huge and possibly provide more info about our cluster than I would like to share. I can provide them upon request.
Versions:

$ weave version
2.5.2
$ docker version
$ uname -a
$ kubectl version
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.6", GitCommit:"96fac5cd13a5dc064f7d9f4f23030a6aeface6cc", GitTreeState:"clean", BuildDate:"2019-08-19T11:05:16Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

Logs:
From the bad pod:
✗ kprod -n kube-system logs --tail 3 -p  weave-net-5z498 weave-npc
INFO: 2019/10/11 22:00:05.664100 EVENT AddNetworkPolicy {"metadata":{"creationTimestamp":"2019-10-11T21:55:49Z","generation":1,"labels":{"ship-update":"4ad1d6a4a20055fb4843b117d8c86959","shipcloud-role":"update"},"name":"shipupdate-4ad1d6a4a20055fb4843b117d8c86959","namespace":"shipupdate-4ad1d6a4a20055fb4843b117d8c86959","resourceVersion":"89571392","selfLink":"/apis/networking.k8s.io/v1/namespaces/shipupdate-4ad1d6a4a20055fb4843b117d8c86959/networkpolicies/shipupdate-4ad1d6a4a20055fb4843b117d8c86959","uid":"e315761c-ec71-11e9-b528-0ea30c65b9c4"},"spec":{"egress":[{"to":[{"ipBlock":{"cidr":"<IP_ADDRESS>/0","except":["<IP_ADDRESS>/32"]}}]}],"podSelector":{},"policyTypes":["Egress"]}}
FATA: 2019/10/11 22:00:05.667396 ipset [create weave-Pooy+}1bi7=ypAIJYYR)W8j$U hash:ip comment] failed: ipset v6.32: Set cannot be created: set with the same name already exists
: exit status 1

From a good pod:
✗ kprod -n kube-system logs weave-net-5m9jk weave-npc
...
INFO: 2019/10/11 21:55:49.790564 EVENT AddNetworkPolicy {"metadata":{"creationTimestamp":"2019-10-11T21:55:49Z","generation":1,"labels":{"ship-update":"4ad1d6a4a20055fb4843b117d8c86959","shipcloud-role":"update"},"name":"shipupdate-4ad1d6a4a20055fb4843b117d8c86959","namespace":"shipupdate-4ad1d6a4a20055fb4843b117d8c86959","resourceVersion":"89571392","selfLink":"/apis/networking.k8s.io/v1/namespaces/shipupdate-4ad1d6a4a20055fb4843b117d8c86959/networkpolicies/shipupdate-4ad1d6a4a20055fb4843b117d8c86959","uid":"e315761c-ec71-11e9-b528-0ea30c65b9c4"},"spec":{"egress":[{"to":[{"ipBlock":{"cidr":"<IP_ADDRESS>/0","except":["<IP_ADDRESS>/32"]}}]}],"podSelector":{},"policyTypes":["Egress"]}}
INFO: 2019/10/11 21:55:49.790640 creating ipset: &npc.ipBlockSpec{key:"<IP_ADDRESS>/32", ipsetName:"weave-W6_xc^aa3W)YQBVxUZP.{A~qv", ipBlock:(*v1.IPBlock)(0xc4220bd1d0), nsName:"shipupdate-4ad1d6a4a20055fb4843b117d8c86959"}
...

Network:

$ ip route
$ ip -4 -o addr
$ sudo iptables-save

@emosbaugh Please share full logs of weave-npc container that is crashing. On start weave-npc container always flushes all the ipsets. Does the weave-npc container consistently crashes after restart or its just crashed once?

@murali-reddy it consistently crashes. stays up for a few minutes before crashing. we think the crashes are related to the number of namespaces and network policies in our environment. logs to follow.

@emosbaugh from the logs you shared offline
it seems you created a network policy with same name as namespace name
/apis/networking.k8s.io/v1/namespaces/shipupdate-4d04ac1a88bfb439b3f7c2b0a2074d30/networkpolicies/shipupdate-4d04ac1a88bfb439b3f7c2b0a2074d30
I guess that is what is causing the problem. Need to test it out and confirm that. Weave-npc ipset name generation logic resulting in the conflict.

@emosbaugh are you still running into this issue? Do you any insight into pattern when this happens.
From the logs you shared it does not seems similar to #3247 as I dont see any deletion of namespace shipupdate-4d04ac1a88bfb439b3f7c2b0a2074d30 or at least its not there in the log snip you shared.
If you still happen to hit this issue, can you please share full activity for the ipset in the logs. In the logs you shared I need logs related to ipset getting created.
grep "weave-X^L;x4YH;_4CK:gyic_Smc}a]" weave5.log                                                                                ❯❯❯
INFO: 2019/10/11 22:48:25.716223 adding entry <IP_ADDRESS> to weave-X^L;x4YH;_4CK:gyic_Smc}a] of 36c8b98a-ec79-11e9-b528-0ea30c65b9c4
INFO: 2019/10/11 22:48:25.716237 added entry <IP_ADDRESS> to weave-X^L;x4YH;_4CK:gyic_Smc}a] of 36c8b98a-ec79-11e9-b528-0ea30c65b9c4
INFO: 2019/10/11 22:48:37.884908 deleting entry <IP_ADDRESS> from weave-X^L;x4YH;_4CK:gyic_Smc}a] of 36c8b98a-ec79-11e9-b528-0ea30c65b9c4
INFO: 2019/10/11 22:48:37.884919 deleted entry <IP_ADDRESS> from weave-X^L;x4YH;_4CK:gyic_Smc}a] of 36c8b98a-ec79-11e9-b528-0ea30c65b9c4
FATA: 2019/10/11 22:52:44.264531 ipset [create weave-X^L;x4YH;_4CK:gyic_Smc}a] hash:ip comment] failed: ipset v6.32: Set cannot be created: set with the same name already exists

We are no longer running into this issue after significantly reducing the number of namespaces and network policies. I will send the requested logs if the issue reoccurs.

thanks @emosbaugh for the update. I am closing the issue. Please reopen if you run into it again.


"Credential storing" functionality of iOS not working

Dear UI5-Team,
my application uses sap.m.Dialog and sap.m.Input to have the user enter their credentials.
On an iPhone 8 + Safari I tried to store the credentials with the help of the "credential storing" functionality of iOS. Both the username and the password are correctly stored; I checked it under settings -> accounts.
However on recalling my application and using this "credential storing" functionality, the password is  placed into the username field and the password field stays empty.
On macOS (Mojave) + Safari it works.
OpenUI5: 1.60.12

Hi @born2dev ,
This seems to me like issue in the iOS.
Could you please isolate it in jsBin.
It will be really useful if you make two examples:

one with openui5 controls
one with native html

King Regards,
Lidiya

Hi Lidiya,
i am running currently into same issues on iOS.
To be able to control the new iOS features, it it neccessary to set autocomplete attribute on sap.m.input! If not set, iOS tries to figure out username, password, etc. from naming, but this can go wrong!
For further info see:
https://developer.apple.com/documentation/security/password_autofill/enabling_password_autofill_on_an_html_input_element
Right know there is a autocomplete property only in conjunction with suggestions, but this will be missunderstood.
My current workaround is:
onInit: function () {
			// delegate input autocompletion for passwords
			var oPwd = this.byId("new-password"),
				oPwdConfirmation = this.byId("confirm-new-password");

			oPwd.addEventDelegate({
				onAfterRendering: function () {
					oPwd.$("inner").attr("autocomplete", "new-password");
					//oPwd.$("inner").attr("name", "abc");
				}
			});

			oPwdConfirmation.addEventDelegate({
				onAfterRendering: function () {
					oPwdConfirmation.$("inner").attr("autocomplete", "new-password");
				}
			});
		}

I am also using the sap.m.Wizard for an onboarding szenario which kills usability on iOS, if you use a suggested password. After that, you can not change the pw anymore, because the dialog is showing up, but directly hidden for scroll issues with wizard control. You can only see this, if you use an external keyboard on ipad that the ios helper is flickering around.
Maybe someone can put a password field into the wizard sample to test and fix this feature in conjunction with iOS password suggestions.
Right know, if you have choosen a suggested password, you are no more able to change it to a new value, because the corresponding dialog will always be hidden for the used eventing inside wizard (i think it is the scrolling).
Regards Holger

Hello @LidiaBobeva,
because the example consists of several files, I created it with Plunker:
https://plnkr.co/edit/yeT6W5VxhyYRdnS3kR1S?p=preview
The UI5 application is automatically loaded into the window at the right (takes a few seconds). Then press "login", please.

Dear UI5-Team,
is there any update?

Hi born2dev,
thank you for your patience and the detailed description of this issue. I've created an internal incident and forwarded it to the responsible development team. Please refer to the following incident ID for further details:
1970495903
The status of this GitHub issue will be updated by the responsible processor.
Regards Sebastian

Can we clarify the steps through https://jsbin.com/litiqeralo/edit?html,output (test URL http://output.jsbin.com/litiqeralo for mobile device)

on iPhone X I can type user/pass and request the OS to save the credentials for this site
then I can reload and start the dialog again
if I start typing in the username input, I can select the credentials stored for this site
---> username goes in the correct box after selecting credentials

Thanks

It's not working on iPhone 8

I tested the same steps with iPhone 8 and the username part is filled correctly. Have you tested the same example I posted: https://jsbin.com/litiqeralo/edit?html,output (save a new jsbin to get the output URL for a test)
Which iOS version is yours?

Because I myself don't have an iPhone I have to use an iPhone of one of my friends. Today I had the chance to test again. It's an iPhone 8 with iOS 13.1.2. I can hereby confirm that the "Credential storing" functionality works with both your jsbin sample and my Plunker sample.
However it still doesn't work with my real application. I need some time to deploy it and then I get back to you.

I need some more time so don't close this message, please.

Dear UI5-Team,
please try it with http://<IP_ADDRESS>:8080/ios-credentials-issue
Because I don't have an iPhone available at the moment I haven't tested it myself. So I carefully say the issue should occur.

With your app the issue is reproducible.
Without the custom code, the issue is not present: https://jsbin.com/coqemecoxa/edit?html,output
*Tested on iPhone X/Safari
Issues:

Home.view.xml --> do not place layout controls in a SimpleForm. You can run Support Assistant to get more detailed report on the issues in the layout: https://openui5.hana.ondemand.com/#/topic/57ccd7d7103640e3a187ed55e1d2c163

Test excluding the Library.js call on beforeOpen of the dialog, as it links RX.min.js file

Thanks for your suggestions. However, to be honest, I don't know what to do.
a)
What do you mean by "without the custom code the issue is not present"?
What is the "custom code"? Everything in my application is required.
b)
Home.view.xml: okay, I can remove the layout controls from within the SimpleForm.
But does this influence the dialog box which itself doesn't have a SimpleForm?
c)
Test excluding the Library.js call: I use this throttle functionality of the RX library
to avoid that ONE button press leads to SEVERAL event handler calls.
Is this issue already known to the UI5-Team? If there is a solution, I could get rid of the
RX library of my whole application.

Custom code is anything you added on top of the default functionality of controls, e.g. code in event handlers. By custom code here I refer to the external libraries which mess with controls.
In the current case, the test is if a form with user/pass inputs, without the external libraries, causes the same issue - password to appear in user input element. If the issue is not present, try to isolate the issue with rendered HTML and the external libraries in order to report the issue to the provider of the external libraries.
I am not sure what issue is resolved with rx library, provide a link to a discussion where it is described, including an example. Better in a separate thread
The layout issue in the XML is not the source of your issue, but it is better to resolve it.

I removed all fields except username and password. I also removed the Rx library. After I have deployed it, the issue is still there.
http://<IP_ADDRESS>:8080/ios-credentials-issue/

The app does not work - only the password is returned in the currently focused input.
autoFillControlsByID (Apple sources) has an object with Control ID and the password only, missing the Control ID with the username
The extracted HTML does not work https://jsbin.com/kuxawiguje/edit?html,output 😕
The extracted UI5 code works though, and in autoFillControlsByID I can see both user/pass data  https://jsbin.com/sozufanuvi/edit?html,output
The extracted HTML works as well https://jsbin.com/qavediyeqe/edit?html,output

The label "author action" is set. What do I have to do now? If the cause is in my UI5 or JavaScript source text, I need to know what I have to change.

The point of the thread is to investigate why autopopulation of user/pass on iPhone is not working.
In the best case, someone with experience will know the issue and solve the problem, in the standard case:
Taken steps are to isolate the issue to 2 HTML only examples

We excluded the interception by the RX libraries
We produced two HTML chunks produced by the same Open UI5 libraries, without SAPUI5 on the page
Test variations of the extracted HTML on different devices
Example1:
https://jsbin.com/wejuqayaso/edit?html,output
-- On iPhone 8 on the second attempt, only user is added
-- on iPhone 11 only password is added in the username place
Example2:
https://jsbin.com/xopemuvecu/edit?html,output
-- on iPhone8 credentials are added
-- on iPhone11 only password appears in username field

Example3: H2 is changed to H1  in the originally not working example
https://jsbin.com/hifolikufo/edit?html,output
-- iPhone11 works
-- iPhone8 adds only user name on second attempt
Example4:  https://jsbin.com/qavediyeqe/edit?html,output

Works on iPhone8
works on iPhone11

What is next? I ask in Apple community via forums https://forums.developer.apple.com/message/398626#398626
If you debug and see something else, post an update

In order that the users of my UI5 application accept the application, it is very important to get this "Credential storing" functionality of iOS working! Unfortunately there has been no response so far from Apple to the question you posted in the Apple community forum. What to do now?

For the time being, you can illustrate the issue with the isolated HTML examples to your end-users, and provide them the URL of the question. Test if for the list of desired devices+OS changing the title level in the logon helps (customHeader and sap.m.Title with level property set to H1) - https://sapui5.hana.ondemand.com/#/api/sap.m.Dialog%23aggregations
You can also run a parallel question for the same.

Since there is no response from the author for more than year I'm closing the ticket.


fix: extract path from nested arrays

Fixes #657
I added a few tests and used it on a local project to check if it was working

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.07%. Comparing base (17bf6b5) to head (049ee2b).
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #658      +/-   ##
==========================================
+ Coverage   88.95%   89.07%   +0.11%
==========================================
  Files          32       32
  Lines         851      851
  Branches      189      189
==========================================
+ Hits          757      758       +1
+ Misses         87       86       -1
  Partials        7        7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks for this!


truncated metric names in bosh System Overview

Commit a2d769c (just before the v23 release) truncated two metrics in the BOSH / BOSH: System Overview dahsboard.

Thanks @romain-dartigues!


upgrade hangs doing autocomplete step

$ brew upgrade heroku
==> Upgrading 1 outdated package, with result:
heroku/brew/heroku 7.6.0 -> 7.7.0
==> Upgrading heroku/brew/heroku
==> Downloading https://cli-assets.heroku.com/heroku-v7.7.0/heroku-v7.7.0.tar.xz
########################################################################     100.0%
==> /usr/local/Cellar/heroku/7.7.0/bin/heroku autocomplete --refresh-cache

then it just hangs.
This may be waiting for user input:
$ heroku autocomplete
heroku: Enter your login credentials
Building the autocomplete cache... ?
Email:

^I have been having the same issue with an install of a fresh heroku client

Confirmed. I am having the same problem

confirmed.  I'm having the same issue.  It times out.

Confirmed. I'm having the same issue. It times out.

==> /usr/local/Cellar/heroku/7.7.1/bin/heroku autocomplete --refresh-cache
Last 15 lines from /Users/vlad/Library/Logs/Homebrew/heroku/01.heroku:
Building the autocomplete cache... ?
heroku: Enter your login credentials
Email: Building the autocomplete cache... done
/usr/local/Cellar/heroku/7.7.1/libexec/node_modules/@oclif/errors/lib/index.js:29
throw err;
^
Error: timed out
at Object.error (/usr/local/Cellar/heroku/7.7.1/libexec/node_modules/@oclif/errors/lib/index.js:22:17)
at Timeout.setTimeout (/usr/local/Cellar/heroku/7.7.1/libexec/node_modules/@heroku-cli/command/lib/login.js:29:38)
at ontimeout (timers.js:427:11)
at tryOnTimeout (timers.js:289:5)
at unrefdHandle (timers.js:511:7)
at Timer.processTimers (timers.js:211:12)

Noting installation proceeds if you blindly provide your email, password and two-factor code (if enabled) when the installation hangs.

same thing happend

For people stuck, this method works:
curl https://cli-assets.heroku.com/install.sh | sh

I'd still like to continue using homebrew to manage this. I see there was 7.7.2 today, which doesn't fix this. @RasPhilCo any ideas?

I just pushed a fix, try that and let me know!

Also works for me. If you wanted to add it back in, would heroku auth:whoami be a good check to see if there's an active login?
def logged_in?
    system bin/"heroku", "auth:whoami"
     true
rescue
    false
end

if logged_in?
  system bin/"heroku", "autocomplete", "bash", "--refresh-cache"
end

@srfraser good idea but it's prob more trouble then it's worth in the long run. We were attempting to save users one command run but it probably was going to generate more confusion, so I'll leave it as is for now. Since you opened this ticket and are good to go now, I'll close it on your behalf. Thanks for reporting it.


'duration' property takes no effect, any idea?

Hi,
Any idea on why the 'duration' property takes no effect?
Any number I provide, the animations just flicks.
I would like to animate the arrow as a CTA.
     <VelocityComponent animation={{translateY:10}} duration={800} loop={true}>                    <Arrow className='arrow' width='100' height='100' onClick={()=>this.scrollTo('section2')}/>                </VelocityComponent>
Thanks.

Does leaving off the loop param change anything?

This has been solved with the runOnMount property.


Get-DbaRandomizedValue - Generate bigint values

Type of Change

[x] Bug fix (non-breaking change, fixes #7499 )

Screenshots

Heck yeah! Thanks Claudio 🥳  And thank you for the review, @andreasjordan! Will merge once all tests pass.
@sanderstad Claudio is working on the Masking chapter and may have some bug fixes in the future too 🤩


Marriage abroad: Outcome per path for Japan

Trello card
Motivation
Editing Marriage abroad questions and outcomes can be quite a complex process.
Marriage abroad is also one of the most frequently updated Smart Answer.
Here we have flattened all outcomes belonging to Japan, to enable content designers make updates quicker.
By providing one template per outcome for marriage abroad Japan content, content editing barriers are greatly reduced.
Factcheck
Preview link
GOV.UK
Expected changes

None

As this is a refactor of Japan flow, and not a change to the content or flow of the smart-answer, users should not experience any changes. The changes that have occurred to the code are:

Create an outcome per path for Japan
Change Japan flow to use outcome per path
Remove redundant Japanese guards in outcomes
Remove Japan from Country lists

Thanks @cbaines for reviewing this


Refine the tiflow master/executor address arguments name

Is your feature request related to a problem?
                        - "/tiflow"
                        - "master"
                        - "--config=/config.toml"
                        - "--log-level=debug"
                        - "--master-addr=<IP_ADDRESS>:10240"
                        - "--advertise-addr=server-master:10240"
                        - "--etcd-endpoints=etcd-standalone:2379"
                        - "--frame-meta-endpoints=mysql-standalone:3306"
                        - "--business-meta-endpoints=etcd-standalone:2379"

It's weird to use command line: tiflow master --master-addr because the meaning of address is obviously so it's better to use tiflow master --addr xxx
Describe the feature you'd like
Remove the duplicated part in the argument name.
Describe alternatives you've considered
No response
Teachability, Documentation, Adoption, Migration Strategy
No response

/assign @lonng


Linking to Patterns with Pattern Lab's Default link Variable via Json

Hey guys,
So I really need to use the Linking to Patterns with Pattern Lab's Default link Variable but via my json so that i can dynamically change a links url on a page to page basis.
I have tried setting my mustache template like so:
{{#subCategories}}

{{name}}

{{/subCategories}}

and my json like so:
"subCategories":[
	         {
	            "id":"47",
	            "name":"Books",
	            "uri":"link.pages-entertainment"
	         },

This seems to link.pages-entertainment as a string in the irl and doesnt convert the url to the correct path.
any ideas are very welcome on this.
thanks
@GUIDeveloper

Hi @guideveloper,
The following code should work:
"url" : "link.pages-entertainment"
Looking at your example, it doesn't look like there's anything wrong with how you've structured it. But try changed it to url instead of uri. I think @dmolsen may have hard-coded it to link up to url.

Nice one @bradfrost but this seems to just output the tag in the href instead of converting it to the actual link?
Visited retailers

Do you think this could be to do with that I'm using the php version?
thanks
@GUIDeveloper

Try:
Visited retailers

Maybe?

Yeah this works just fine @daigofuji ,  the issue is that i am loading my links via Json into my mustache template so really need the links populating from there.

@guideveloper try downloading a clean version of Pattern Lab and seeing if that works. Don't think I have any other ideas for you.

Ok mate cheers I'll give that a go
thanks
@bradfrost and @daigofuji

@guideveloper I encountered the same issue just now for what it's worth. Was working fine then a new entry into my json file had the function that creates links start bombing out. The only difference I can find is the url is burried three levels deep in my json vs two levels.
"news" : [
		{
			"widget-entry" : [
				{
					"url" : "link.pages-publications",

Is your snippet of code above two levels deep or deeper?

@Founts yeah mine is also three levels deep
"myAccountMenuItems":[
		{
	      "id":"458",
	      "name":"My stuff",
	      "uri":"my-stuff",
	      "subCategories":[
	         {
	            "id":"9017",
	            "name":"Favourite retailers",
	            "url":"link.pages-posters-flyers"
	         },

@Founts Don't suppose you found a fix for this did you?

@guideveloper nope. Thought about restructuring my code somehow but had other pressing items to tackle.

Same problem here.
The problem is that PatternLab suggests to use Pages as specific version of a Template. So this way you can include the partial template and give your site specific content via the JSON file. Well not all content, because internal patternlab links are not compiled in json files.
Every solution I can come up with throws away at least a couple advantages of PatternLab,
This is a real blocker imo.

@Stalski -
Are you on dev? I'm assuming so but wanted to double-check just in case I was forgetting something from master. The issue is the use of array_walk_recursive(). Have a fix but haven't fired up dev in ages and don't want to commit something that isn't helpful.

Can you post it here or by email? I would be glad to test it out.
@dmolsen

The issue is because of this array_walk_recursive() in DataMergeExporter. Turns out array_walk_recursive() doesn't go into nodes that contain arrays. Hence the depth issue.
The fix is to add the recursiveWalk() and compareReplaceListVars() functions to your Data.php class from PL core. Make the recursiveWalk() function public. Then replace the array_walk_recursive() call above with:
Data::$store = Data::recursiveWalk(Data::$store);

That should work since Data::$store is public in the version of PL you're using.

I was able to set it up (dev branch I mean)
It did not work by default, but after I saw in code that the folders were changed (cfr _meta , _annotations, ...) , and after fixing some bugs as well like arrays not being arrays , stuff like that.  If you want more info or a patch, let me know.
Also a problem then with the autoloader SPL classes which were not there but I commented this out for the moment.
After that it worked, and it is much more verbose, E.G. "this is preserved word" or "you probably made a typo in ..." . Nice features :)
I will now take a look at the piece of code you mentioned. Can you just give me the example on how the json should be defined? I mean, suppose the json variable is "link-url" : "link.page-one" or more like "link-url" : "{{link.page-one}} or some other method? In other words, what is the code expecting to find in the json?
thx in advance

Wow. Like I said I haven't been in there in a while so glad you were able to work through the problems :) Let's say this was a snippet from your mustache file and you wanted to link to the pages-homepage pattern:
Home

Then the JSON would be:
"homelink": "link.pages-homepage"

Make sense?

Hi,
it needed the getOptions method as well  btw ;)
So The string are correctly replaced, thx for the help!
I now get : The requested URL /05-pages-01-content-overview-05-news-archive/05-pages-01-content-overview-05-news-archive.html was not found on this server.
So I am investigating this for now. If you have any quick idea's , it would be helpful. I think the root app url is not correct or something.

This is really strange. I don't seem to find where the actual store is built with the correct URL (not path) to the patterns.
I know it does not take /patterns/ into account in the url, so I know what the replacement should have done, but I don't seem to find the correct way to fix the "base uri"
I tried to set the "patternPublicDir" in the config.ini but that did not do the trick.
Are there any changes that I am not aware of in the dev branch?

Nevermind ;) Found it
It needed a change in "DataLinkExporter":
Data::$store["link"][$patternStoreKey] = "../../patterns/".$patternStoreData["pathDash"]."/".$patternStoreData["pathDash"].".html";
instead of
Data::$store["link"][$patternStoreKey] = "../../".$patternStoreData["pathDash"]."/".$patternStoreData["pathDash"].".html";
Win! I am really looking forward to the new version (with yaml as in your commit)

I don't know if this will be useful but I'll add the patch in a zip here.
If you want me take appropriate action with a pull request, let me know. We are using this extensively for a big client, so it is very important for us to have this functionality into the stable release.
So If you need me, just call out to me.
patternlab-php-dev.patch.zip

FWIW, I'm probably not going to merge anything here to master because I believe most of that stuff is addressed in the true development edition of PL. Down to our last big feature to implement. It's still not stable but you can preview the development edition if you want.
Thanks again for getting that to work.

So basically at this time, there is no stable version yet where internal linking from a json file can be achieved?
I am not a fan of using the development edition which is not stable for a real project in production.
Can you give me some pointers on how I show see this? What to use for a project in production with the possibility to have internal linking?


procs.exe on windows throwing traceback error

This issue respects the following points:

[X] This is a bug, not a question or a setup/configuration issue.
[X] This issue is not already reported on Github (I've searched it).
[X] I use the latest release of the Monitoring Plugins (https://github.com/Linuxfabrik/monitoring-plugins/releases).
[X] I agree to follow Monitoring Plugins's Code of Conduct.

Which variant of the Monitoring Plugins do you use?

[X] Compiled, from repo
[ ] Source Code, from GitHub

Bug description
The latest windows procs.exe plugin is reporting the following error on all windows machines regardless of windows version:
Traceback (most recent call last):
File "C:\PROGRA~3\icinga2\usr\lib64
agios\plugins\procs3", line 429, in 'module'
File "C:\PROGRA~3\icinga2\usr\lib64
agios\plugins\procs3", line 252, in main
TypeError: can only join an iterable
Steps to reproduce - Plugin call
'C:\ProgramData\icinga2\usr\lib64\nagios\plugins\procs.exe' '--argument' 'icinga' '--critical-mem' '1288490189' '--warning-mem' '1073741824'
Steps to reproduce - Data
No response
Environment
Windows Server 2016
Plugin Version
latest.zip from repo
Python version
No response
List of Python modules
No response
Additional Information
No response

recompiled.


Fix: Failed to parse x '1463498940000' to Date object

when parameter:date is an Array with 1 element of type number

parseDate expect a single value to be coerced into a date.
If an array is provided at one point the issue is coming from the caller.


CHKT-1175 throw warning when decision handler is not called

Throw a console warning if decisionHanlder is not called within 5 sec

[question] Would it be better/easier to stop the timer in case there's a call to decisionHandler instead of using a decisionHandlerHasBeenCalled flag?