[BUG] Multiserver cluster creation fail (x509 cert not valid for the domain )

What did you do

How was the cluster created?

k3d cluster create cluster-mulriserver --servers 3 --no-rollback

What did you expect to happen
Cluster should be created
Screenshots or terminal output
INFO[0000] Created network 'k3d-cluster-mulriserver'
INFO[0000] Created volume 'k3d-cluster-mulriserver-images'
INFO[0000] Creating initializing server node
INFO[0000] Creating node 'k3d-cluster-mulriserver-server-0'
INFO[0005] Creating node 'k3d-cluster-mulriserver-server-1'
INFO[0007] Creating node 'k3d-cluster-mulriserver-server-2'
ERRO[0007] Failed waiting for log message 'Wrote kubeconfig' from node 'k3d-cluster-mulriserver-server-1'
ERRO[0007] Failed to start container
ERRO[0007] Failed to create node 'k3d-cluster-mulriserver-server-2'
ERRO[0007] Failed to create node
ERRO[0007] error during connect: Post "http://%2Fvar%2Frun%2Fdocker.sock/v1.40/containers/1ef9cbbe4088c9fb328cb8ee1efcc2497db06cbea73451bf1595bc40f68537ac/start": context canceled
FATA[0007] Cluster creation FAILED, rollback deactivated.

Logs of container server-1 and server-2
time="2020-10-03T16:56:28.351711193Z" level=info msg="Starting k3s v1.19.2+k3s1 (d38505b1)"
time="2020-10-03T16:56:28.548194125Z" level=fatal msg="starting kubernetes: preparing server: server https://k3d-cluster-mulriserver-server-0:6443/cacerts is not trusted: Get \"https://k3d-cluster-mulriserver-server-0:6443/cacerts\": x509: certificate is valid for kubernetes, kubernetes.default, kubernetes.default.svc.cluster.local, localhost, not k3d-cluster-mulriserver-server-0"

Which OS & Architecture

macOS Catalina

Which version of k3d
k3d version v3.0.2
k3s version latest (default)

Which version of docker
Client: Docker Engine - Community
 Cloud integration  0.1.18
 Version:           19.03.13
 API version:       1.40
 Go version:        go1.13.15
 Git commit:        4484c46d9d
 Built:             Wed Sep 16 16:58:31 2020
 OS/Arch:           darwin/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.13
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.13.15
  Git commit:       4484c46d9d
  Built:            Wed Sep 16 17:07:04 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          v1.3.7
  GitCommit:        8fba4e9a7d01810a393d5d25a3621dc101981175
 runc:
  Version:          1.0.0-rc10
  GitCommit:        dc9208a3303feef5b3839f4323d9beb36df0a9dd
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

docker info
Client:
 Debug Mode: false

Server:
 Containers: 8
  Running: 3
  Paused: 0
  Stopped: 5
 Images: 21
 Server Version: 19.03.13
 Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Native Overlay Diff: true
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: inactive
 Runtimes: runc
 Default Runtime: runc
 Init Binary: docker-init
 containerd version: 8fba4e9a7d01810a393d5d25a3621dc101981175
 runc version: dc9208a3303feef5b3839f4323d9beb36df0a9dd
 init version: fec3683
 Security Options:
  seccomp
   Profile: default
 Kernel Version: 4.19.76-linuxkit
 Operating System: Docker Desktop
 OSType: linux
 Architecture: x86_64
 CPUs: 4
 Total Memory: 1.944GiB
 Name: docker-desktop
 ID: PMEC:N4W4:PM6J:RIY3:L6ZT:GV5G:V3CG:AZNL:7RBA:7BX6:FWYY:FEPA
 Docker Root Dir: /var/lib/docker
 Debug Mode: true
  File Descriptors: 72
  Goroutines: 74
  System Time: 2020-10-03T17:00:45.176503025Z
  EventsListeners: 5
 HTTP Proxy: gateway.docker.internal:3128
 HTTPS Proxy: gateway.docker.internal:3129
 Registry: https://index.docker.io/v1/
 Labels:
 Experimental: false
 Insecure Registries:
  <IP_ADDRESS>/8
 Live Restore Enabled: false
 Product License: Community Engine

Hi @Xenon341 and thanks for opening this issue! :)
That exact error message I managed to fix pretty easily.. however I've run into subsequent issues that I have to check with the k3s maintainers :+1:

As an update: it's currently not clear if this is going to be fixed in k3d or in k3s itself. Anyway it should be fixed soon :+1:
Option 1)

fix automatic add of the TLS SAN in k3s when first request to /cacerts is being made
retry joining etcd if there are already too many members learning (1)

Option 2)

add tls SAN in k3d (one line to add)
wait for preceding server to finish learning (watch log messages) and then start the next server (prolonging the time-to-cluster)

Hey 👋
This should be fixed in the new v3.1.3 release, please give it a try 🙂

Seems to be good ;)
I got another ERROR message when creating a cluster but not sure if it's related
INFO[0061] (Optional) Trying to get IP of the docker host and inject it into the cluster as 'host.k3d.internal' for easy access
ERRO[0062] Logs from failed access process:
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>:53

** server can't find host.docker.internal: NXDOMAIN

Non-authoritative answer:
Name:	host.docker.internal
Address: <IP_ADDRESS>

WARN[0062] Failed to get HostIP: %+v Failed to read address for 'host.docker.internal' from nslookup response
INFO[0062] Cluster 'cluster-mulriserver' created successfully!

Should I open a new issue ?
Thanks !

Nice, thanks for the feedback :)
Nope, that new error is a known issue with that k3s version (see release notes https://github.com/rancher/k3d/releases/tag/v3.1.3) on MacOS. We're working on that :+1:


Battery Sensor not showing

Battery % and Battery V not available since update.
Indego is online.
Version: 0.9
Indigo S+350

Indego 350 Connect is showing. Anything in the logs?

Wait a Moment. That was a problem with my mower. Took it out of the station, shut it off and put it back in. Worked after turning it in again.


New version: astral-sh.ruff version 0.4.8

Checklist for Pull Requests

[X] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-06-06 02:22:32.566 [FAIL] Installer failed security check. Url: https://github.com/astral-sh/ruff/releases/download/v0.4.8/ruff-0.4.8-x86_64-pc-windows-msvc.zip Result: 0x80004005
2024-06-06 01:21:28.676 [FAIL] Installer failed security check. Url: https://github.com/astral-sh/ruff/releases/download/v0.4.8/ruff-0.4.8-i686-pc-windows-msvc.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
(Automated response - build 883.)

@wingetbot run

@wingetbot run

@wingetbot run

Automatic Validation ended with:

Installation failed with exit code -1978335141
2024-06-09 23:54:16.825 [CLI ] Unable to locate nested installer at: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\astral-sh.ruff.0.4.8\extracted\astral-sh.ruff

Automated error analysis suggests -1978335141 may mean Unable to locate nested installer. Does the RelativeFilePath have the previous version number in it?
(Automated response - build 884.)


I want to implement the following view

I want to achieve the following view by reducing the width of the indicator or of the individual tabs. Please help me out.

@monowar1993 This is unfortunately not possible with the library as it is right now. Sorry!


Setting Zip File Compression Level

Is there a way to set the compression level for zip files?

I was digging through the code and it looks like it is already possible. For anyone wondering the same thing:
var zip = ZipFileSystem.CreateZipFile(file, FileSystemOptions.Default.AddVariables(new { ZipCompressionLevel = "0" }))

Well, I'm reopening this.
using (ZipFileSystem zfs = ZipFileSystem.CreateZipFile(file, FileSystemOptions.Default.AddVariables(new { ZipCompressionLevel = "0" })))

...
This does not seem to work. It is still compressing the file

I think the problem is that ZipFileSystem's constructor does not know about compression level. CreatZipFile is returning a new ZipFileSystem instance, so these option values are not checked in there.


⚠ AlphaRatio has degraded performance

In d458bbd, AlphaRatio ($AR) experienced degraded performance:

HTTP code: 200
Response time: 1244 ms

Resolved: AlphaRatio performance has improved in 1d98657 after 6 minutes.


export validation

Pull requests into Cypress require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:

[ ] This pull request describes why these changes were made.
[ ] Internal ticket for this PR:
[ ] Internal ticket links to this PR
[ ] Code diff has been done and been reviewed
[ ] Tests are included and test edge cases
[ ] Tests have been run locally and pass

Reviewer 1:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Reviewer 2:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Example report here
newnew_6053ac10c1c38832fde56d40_report (65).zip

Codecov Report

Merging #14 (ea522a9) into master (a06028d) will decrease coverage by 4.47%.
The diff coverage is 52.38%.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
- Coverage   91.16%   86.69%   -4.48%
==========================================
  Files          52       53       +1
  Lines        1336     1368      +32
==========================================
- Hits         1218     1186      -32
- Misses        118      182      +64

Impacted Files
Coverage Δ

app/controllers/vendors/records_controller.rb
76.92% <25.00%> (-5.94%)
:arrow_down:

lib/cypress/create_download_zip.rb
40.00% <40.00%> (ø)

lib/ext/fhirbundle.rb
98.86% <85.71%> (-1.14%)
:arrow_down:

app/controllers/sessions_controller.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/controllers/registrations_controller.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

lib/hash.rb
0.00% <0.00%> (-88.89%)
:arrow_down:

app/controllers/records_controller.rb
77.14% <0.00%> (-10.00%)
:arrow_down:

app/helpers/records_helper.rb
88.23% <0.00%> (-5.89%)
:arrow_down:

app/controllers/vendors_controller.rb
75.75% <0.00%> (-4.55%)
:arrow_down:

app/controllers/application_controller.rb
90.47% <0.00%> (-1.59%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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

new_product_report.zip


fix(bootstrap): v10.2.0

Runs bootstrap v10.2.0.

Release Notes

10.2.0 (2022-07-26)
Features

add annotation to have albs target specific nodes (#1313) (c0d3fd8)
add support for otel tracing (#1277) (ea38fe3)
clerkgen: Don't run clerkgen in bootstrap (#1281) (d01a514)
devbase to latest to bring clerkgen with protooptions (#1275) (a565222)
DP-3722: update devbase to include clerkgen updates (#1319) (284ba02)
migrate clerk templates to stencil-clerk (#1307) (0e41397)
migrate tollgate ingress to alb (#1305) (e67e633)
pass metrics flag to stencil (#1263) (279c83a)
stencil-smartstore templates (#1303) (4f91a3a)

Bug Fixes

add descriptive error for manual otel upgrade (#1290) (9cd0f55)
bump Smartstore version (#1324) (75e7e9f)
bump stencil (#1340) (3184f32)
clerk: Add clerkcommons to dependencies (#1310) (e5ca147)
clerkcommons: Update clerkcommons version (#1293) (f2b45f5)
clerk: update clerk to 3.13.0 (#1289) (11154b1)
clerk: Update clerkcommons to 1.19.1 (#1308) (748ae84)
delete integ test protos (#1309) (177f198)
devbase: 2.1.0-rc.3 (#1286) (e5e010e)
devbase: Update devbase version (#1271) (e347c88)
devbase: Update devbase version (#1328) (bbfe9e6)
devbase: v2.0.5 (#1284) (bfece4e)
DP-2294: pick up clerkcommons workaround to support one schema for multiple pipelines (#1327) (7b8666a)
DTSS-2002: Correct handling of hyphens in service names (#1304) (fc58a4e)
enable e2e job to configure resource_class (#1299) (1a7cb1b)
generating redundant segments (#1274) (e42722e)
move cli apps to otel (#1317) (85a1f08)
setup_repo release branch (#1285) (6482d69)
smartstore to v1.46.2 (#1282) (ace7220)
smartstore: Initialize all smartstore extensions before starting UC. (#1302) (0cc1d7a)
smartstore: Update smartstore's version (#1300) (a535ba2)
stencil: Update stencil version (#1315) (bd09a7d)
support stencil-kaia (#1311) (f4f6b2e)
Temporal 1.16.2 must have metrics in global (#1331) (24f4b3f)
tollgate alb ingress backend protocol needs to be grpc (#1314) (5ee3a46)
tollgate alb needs tls (#1312) (6e91475)
Update clerk's version (#1336) (421086e)
Update clerk's version (#1337) (6923b8b)
Update clerk's version to v3.16.1 (#1323) (7253ce3)
Update clerkcommons (#1318) (dda1298)
Update clerkcommons version to v1.21.0 (#1325) (467dc48)
Update clerkcommons version to v1.28.0 (#1332) (cc27eeb)
update devbase to include fixes (region, protoc-gen-validate) (#1320) (16166a3)
Update devbase to v2.2.0 (#1341) (c53742b)
Update to smartstore version 1.51.0 (#1334) (fd85175)
Update versions.yaml (#1329) (07216fc)
yaml indentation for pg schemas use json (#1278) (5333449), closes #1271 #1275

Feedback? Run into bugs? Let us know at #dev-tooling-support!

Pull Request Test Coverage Report for Build d9505cb7-4bfc-4356-86e6-5b81b3e25331

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 87.879%

Totals

Change from base Build b1d73fca-812c-449e-a68d-931bae6e5f08:
0.7%

Covered Lines:
261

Relevant Lines:
297

💛  - Coveralls


Add a gestureRecognizers parameter to GoogleMap.

These gesture recognizers participates in gesture arenas on behalf of
the map.
This allows fine grain control over which gestures are dispatched to
the map.

Thanks!
I ended up adding EagerGestureRecognizers in the framework (flutter/flutter#21407),
I'll wait for that PR to land first, and switch the example to use the framework's EagerGestureRecognizer before landing.


IntersectionCallback needs better explaination or naming

It's not quite obvious from the README that this IntersectionCallback will actually be fired whenever the intersection condition changed.
For example, if threshold is 0.3, this callback will be called for both intersection part becomes above and below 0.3.
It might be easily interpolated as IntersectionCallback will only be called if the threshold condition is met. Or at least there's no way to confirm what the exact behavior is except experimenting around with it.
IntersectionCallback is better called IntersectionChangeCallback or we can add better documentation for this part.
Just my two cents.

I'm open to ways we can explain this better; please let me know how we can do that.
Currently, the explainer doc says:
"The callback includes change records for all elements that have crossed the threshold of the root element since the last callback. Conceptually, this gives you a rectangle (based at the root element) that calls a callback whenever a given point in each element crosses the threshold."
The spec says:
"Callback will be invoked when intersectionRect’s area changes from greater than or equal to any threshold to less than that threshold, and vice versa."
... and the text of  then "Run the Update Intersection Observations Steps" is very explicit about how thresholds are treated (see steps 7-12):
https://wicg.github.io/IntersectionObserver/#update-intersection-observations-algo

Personally I feel all elements that have crossed the threshold is a bit subtle here.
While "Callback will be invoked when intersectionRect’s area changes from greater than or equal to any threshold to less than that threshold, and vice versa." is actually much clearer


Add RLPSwift

Project URL
https://github.com/bitfwdcommunity/RLPSwift
Category
Other Parsing
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          2 Warnings

:warning:
Too many changes (when adding, please keep it to one project per Pull Request)

:warning:
Found 2 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

5
Dupe
https://gitter.im/vsouza/awesome-ios

2705
503
https://boombox.io/

  Generated by :no_entry_sign: Danger


chore(main): release 0.14.1

:robot: I have created a release beep boop
0.14.1 (2024-01-16)
Reverts

revert search bar code to previous version to undo recent changes (272ed02)

Bug Fixes

专栏页面侧边栏目录按钮垂直居中 (e2f4427)
从稍后再看打开视频时自动跳转 #180 (1edf382)

Miscellaneous

adjust regex (781b2c0)
update version number (d459603)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/hakadao/BewlyBewly/releases/tag/v0.14.1 :sunflower:


I've created some issue templates. You can stumble into them if you try to submit a new issue.
I also created this video to help explain how to create/edit them.
I don't like how the template kinda sorta blurs instruction with the actual submission but don't know what to do about that either.
Feel free to close this issue if after review the templates and/or the video, you have sufficient info to proceed.

So, I already see something that doesn't work too well with the curated content template. The checkboxes work fine...as long as they are not deleted as part of the submission. Maybe Mike's idea of using the thumbs up emoticon (restricted to the original issue submission and not burried down in the follow-on conversation) would work better. One thing I do like about the check-box approach is that the status of those check-marks appears in the issue as (0 of 2 tasks completed) so its easy to review a page full of issues and see which have already recieved two favorable reviews.

BTW...sorry if the video quality isn't great. I chose to store in 480p due to size. I think its sufficient to see your way around the repo.

Nice video Mark, but I don’t have access to settings anymore. I did figure out how to unassign myself by editing the template file directly.
From: "Mark C. Miller"<EMAIL_ADDRESS>Reply-To: "betterscientificsoftware/betterscientificsoftware.github.io"<EMAIL_ADDRESS>Date: Thursday, April 18, 2019 at 1:31 PM
To: "betterscientificsoftware/betterscientificsoftware.github.io"<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [betterscientificsoftware/betterscientificsoftware.github.io] Create some issue templates; hand off to Pat G. (#342)
I've created some issue templates. You can stumble into them if you try to submit a new issue.
I also created this videohttps://youtu.be/Dy7isHImrAM to help explain how to create/edit them.
I don't like how the template kinda sorta blurs instruction with the actual submission but don't know what to do about that either.
Feel free to close this issue if after review the templates and/or the video, you have sufficient info to proceed.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/betterscientificsoftware/betterscientificsoftware.github.io/issues/342, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAS3ZUZ7YJVKEBVHOHIHPNDPRDD7TANCNFSM4HG7XTCA.

Nice video Mark, but I don’t have access to settings anymore. I did figure out how to unassign myself by editing the template file directly.

Sorry. Hadn't realized I was utilizing magic powers to go through the GitHub web interface to edit the templates. Hmmm... Sounds like you did find how to do it directly by adjusting YAML front-matter in the TEMPLATE files in the repo. That works too. In fact, going through GitHub web interface does the same thing in terms of doing git operations on the repo.

I really think I was able to do it earlier, but I won’t guarantee my memory is correct.
From: "Mark C. Miller"<EMAIL_ADDRESS>Reply-To: "betterscientificsoftware/betterscientificsoftware.github.io"<EMAIL_ADDRESS>Date: Wednesday, May 15, 2019 at 10:37 AM
To: "betterscientificsoftware/betterscientificsoftware.github.io"<EMAIL_ADDRESS>Cc: "Grubel, Patricia A"<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [betterscientificsoftware/betterscientificsoftware.github.io] Create some issue templates; hand off to Pat G. (#342)
Nice video Mark, but I don’t have access to settings anymore. I did figure out how to unassign myself by editing the template file directly.
Sorry. Hadn't realized I was utilizing magic powers to go through the GitHub web interface to edit the templates. Hmmm... Sounds like you did find how to do it directly by adjusting YAML front-matter in the TEMPLATE files in the repo. That works too. In fact, going through GitHub web interface does the same thing in terms of doing git operations on the repo.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/betterscientificsoftware/betterscientificsoftware.github.io/issues/342?email_source=notifications&email_token=AAS3ZUY3N5BSVKPAA6DVL5LPVQ32JA5CNFSM4HG7XTCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVPHLHY#issuecomment-492729759, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAS3ZUZ6OHO4KK6Y6J46TITPVQ32JANCNFSM4HG7XTCA.


Added Kadane's Algorithm

@tarun26091999  Please attend to following issues in your contribution :

Remove the .exe file .
Keep the complexity to O(n), which is not true due to line no. 66 in Code/C++/kadane_algo.cpp
Program should expect only one test case each time. You don't need to input number of test cases.
Provide two sample inputs.

@tarun26091999  Please attend to following issues in your contribution :

Remove the .exe file .
Keep the complexity to O(n), which is not true due to line no. 66 in Code/C++/kadane_algo.cpp
Program should expect only one test case each time. You don't need to input number of test cases.
Provide two sample inputs.

@nakul-19 all changes done.

@nakul-19 all changes done.


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add xo from template
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

Codecov Report

Merging #437 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #437   +/-   ##
=======================================
  Coverage   73.94%   73.94%
=======================================
  Files           2        2
  Lines         476      476
=======================================
  Hits          352      352
  Misses        124      124

Continue to review full report at Codecov.

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

Coverage remained the same at 71.626% when pulling 67a4789527e3be4f545755efd77eb74daf92f2e0 on template-sync-1 into e120d8800662a57a84e4548a01755e69fb747f6f on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(docker-compose): bash not found in the blockscan container

What this PR does / why we need it?
This PR:

fixes https://github.com/axonweb3/axon/issues/1524

What is the impact of this PR?
No Breaking Change
CI Settings

CI Usage
Tip: Check the CI you want to run below, and then comment /run-ci.
CI Switch

[x] Web3 Compatible Tests
[x] OCT 1-5 And 12-15
[x] OCT 6-10
[x] OCT 11
[x] OCT 16-19
[x] v3 Core Tests

CI Description

CI Name
Description

Web3 Compatible Test
Test the Web3 compatibility of Axon

v3 Core Test
Run the compatibility tests provided by Uniswap V3

OCT 1-5 | 6-10 | 11 | 12-15 | 16-19
Run the compatibility tests provided by OpenZeppelin

/run-ci


[MINOR] fix: Add  method close for ApplicationManager

What changes were proposed in this pull request?
Add  method close for ApplicationManager
Why are the changes needed?
Clean resources while testing. It will improve the stability of our test.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
GA passed

But this doesn't seem to be the real reason why this ut ShuffleServerFaultToleranceTest#testReadFaultTolerance failed. Right?

Yes. But we should reduce the influence of other factors.

Merged. thanks @smallzhongfeng @zuston


Current time feature js

Adding in current time feature IN COMMENTS. After an input is submitted via the front end for a time slot, back end can now process the time which is a "word", e.g. "12:45". Function can now turn that word into a number, e.g. 1245. This can now be quantitatively evaluated within an if statement as a bound.
E.g. if (current_time > rise_time && current_time < fall_time){do something}

tried to merge with main by accident AGAIN


About supporting the creation of multiple entities

Opening a discussion about pros / cons of this template allowing the user to create more than just 1 entity

Some pros I found:

Project generation from 70% to maybe 90% in a single run
Allow the dev to think about the architecture while using this tool (and not just boilerplate code generation)

Some cons on the other hand:

May be harder to use (and even to develop & maintain)
Maybe the result is limited and then the dev still needs to refactor code (spending the same time than developing the actual code)

Maybe it would be convenient to specify entities + relationships in a config file (e.g. YAML file), so the prompt will ask for that config file to generate the template based on that. Otherwise, it is not nice to specify everything in a prompt (as a developer, I like to have everything well defined in a config file)

I was thinking a lot about this, and my conclusion was the next one: Adding the ability to automatically generate more entities without taking into account their relationship doesn't make any sense, it adds overall complexity and little to no value. Maybe we can consider making another script to generate these isolated entities from the CLI (after the cookie-cutter was executed and the architecture was all set up) with something like ./scripts/generate_entity.py --user and this creates the same structure as the original first entity.
That said, the best of all worlds would be generating a whole architecture from an entity-relationship diagram so we have there all the models, the entities, and how their relationships are, but I can't think of a solution to generate all this structure at once.

I guess we should start sharing the template and see what the community says about it and what they suggest to evolve this

@leferrad suggested checking https://github.com/koxudaxi/fastapi-code-generator it seems promising


[WIP] Fix gateway datapath disrupt on update

Reimplementing the vswitchd reload using separate start and stop scripts so that it can be executed partially between consecutives instances of the pod.
TODO:

[x] Semaphore
[ ] Kuttl testing

This will need a kuttl test that would validate that flows injected by OVN are retained after ovs pod restart.

/ok-to-test

New version includes the reviews from @ralonsoh. I have tested this in my env and it works better now. The flows are still not correctly written on the DB with the start script, though.

I have removed the sleep on the ovsdb-server preStop. We still need the kuttl testing

/hold

@elvgarrui @ralonsoh I worked on the patch today, here are the changes I made: https://github.com/openstack-k8s-operators/ovn-operator/pull/301/commits/0a185c58cc1bd55080040f20ccf83a87057024e5

removed net_setup.sh - now it's a single startup script.
introduced a bunch of variables / functions in functions common module.
pass TMPDIR to save script so that it creates all files in the expected location - /var/lib/openswitch, after that I was able to remove some scary code that was modifying the script in-place with sed etc.
added some protective code that makes sure flow scripts are cleaned up after they were restored. This is so that in case later vswitchd crashes, it doesn't suddenly pick up old flow restore files.
some other random bug fixes.
lots of comments.

I haven't found time to write kuttl scenario. Please work on this when you have time.

recheck

/test ovn-operator-build-deploy-kuttl

@ralonsoh can you rebase and fix finalizers in assert file as per https://github.com/openstack-k8s-operators/ovn-operator/commit/c2b1320e94c23bf940e6288bd03553e20b9fc9cc

@ralonsoh can you rebase and fix finalizers in assert file as per c2b1320

Right now, actually I'll also squash the 3 commits in this PR into one.

Please also update the commit message:

remove WIP
add some details as to how this patch avoids disruption (mention that flows are restored via ovs-save.)

Hi folks, I think this patch is ready. I don't know (I think I can't) change the PR name (and remove the WIP tag) but is removed from the commit message now. Don't hesitate to ping me here, by mail or in IRC. Thanks!

/lgtm

/hold off

/lgtm

/approve
There's only so much yak to shave. Some improvements to kuttl may still be useful, but I don't think we should wait for it addressed here.


NixIO ChannelIndex Reading channel_names as empty

Minimal Example
Create ChannelIndex for single Block and save as .h5.
from neo.core import (Block, Segment, ChannelIndex)
from neo.io import NixIO

bl = Block()

chx = ChannelIndex(name="Array", index=[0,1,2], channel_names=['ch0', 'ch1', 'ch2'])
bl.channel_indexes.append(chx)

with NixIO(filename="/tmp/chx.h5", mode='rw') as writer:
    writer.write_block(bl)

>>> bl.channel_indexes[0].channel_names
array(['ch0', 'ch1', 'ch2'], dtype='<U3')

But when .h5 file was loaded the channel names array was empty  (note: array originally encoded as unicode but read as str):
>>> with NixIO(filename="/tmp/chx.h5", mode='ro') as reader:
...     blk = reader.read_block()
...
>>> blk.channel_indexes[0].channel_names
array([], dtype='|S1')
>>> blk.channel_indexes[0].index
array([0, 1, 2])

HDF Viewer suggests channel names were stored in .h5 file:

Setup
Python 3.5.2
h5py==2.10.0
neo==0.8.0
nixio==1.5.0b3
numpy==1.17.3
ubuntu 16.04

Thanks.  I found the issue.  It was introduced in #695 (as a fix for #694).
Fix coming in a bit.


deforests, frotteurs

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[d,f]

/trunk merge


PN 4.x: Buffer distance doesn't update in list label when changed

To repro:

Click Add button to start a notification list
Select Search tab and search for an address
Enter a buffer distance of 100 feet and click Done
The label correctly shows the searched address and the buffer distance
Edit the list and change the buffer distance to 500 feet, click Done
The label still lists 100 feet.

Verified at the component level


Automated cherry pick of #5061: Use OpenFlow group for Network Policy logging

Cherry pick of #5061 on release-1.11.
#5061: Use OpenFlow group for Network Policy logging
For details on the cherry pick process, see the cherry pick requests page.

@antoninbas I want to abandon this cherry-pick to release v1.11 since the code is not compatible after switching packetIn to packetIn2. Any thoughts?

@antoninbas I want to abandon this cherry-pick to release v1.11 since the code is not compatible after switching packetIn to packetIn2. Any thoughts?

ok


fixed build issues

@inoxx03 see if this works for you locally.

Thank you @rhoads-zach. it works:)
merged into RHOARDOC-1133: b2b001decae6753ced56694ecdcf98cc52a46021


Add namespace declaration to manifests

What this PR does / why we need it:

best practice
prevent argocd error like "Namespace for $name networking.k8s.io/v1, Kind=Ingress is missing."

Hey, I do not agree that this is good practice to do. Even more, my understanding of how Helm handles the rendered manifest's metadata.namespace is as follows:

•	When you deploy a Helm chart with helm install or helm upgrade, you can specify a namespace using the --namespace flag.
•	If you do not specify the --namespace flag, Helm defaults to the default namespace unless overridden in the chart configuration.

The error you are seeing must be originated by anything else in your ArgoCD usage. I am not an ArgoCD expert, as I use with Flux and Fleet in daily manner, but I did deploy several Helm Charts with ArgoCD (specifically developed ones for a customer application, but also an older version of this one) and neither my own ones nor this Zabbix server one did ever need the changes you are suggesting with this PR.

@aeciopires your thoughts?

Hi, @fibbs!
I think @mustdiechik is right. The PR is not setting the namespace name, it continues to behave as you said, only the name of the chosen namespace is being registered in the YAML.
I'm going to test it here using kind and argocd+github+helm to validate the need for this, but at first it seems to make sense.
I looked at the Prometheus helm chart (which I believe is also widely used with ArgoCD)... and there is this namespace line in the helm chart template files.
References:

https://artifacthub.io/packages/helm/prometheus-community/prometheus?modal=template&template=cm.yaml
https://artifacthub.io/packages/helm/prometheus-community/prometheus?modal=template&template=deploy.yaml
https://artifacthub.io/packages/helm/prometheus-community/prometheus?modal=template&template=headless-svc.yaml
https://artifacthub.io/packages/helm/prometheus-community/prometheus?modal=template&template=ingress.yaml

Thank you @mustdiechik for your contribution.
Can you please remove the changes in the Chart.yaml file?
Only maintainers should change this file just before releasing a release in Helm Chart... We recently changed the process to allow the accumulation of changes and pull requests in the same version.
Changes in Chart.yaml and README.md files in different PRs were causing a lot of conflict issues in the Git repository.
See new process in CONTRIBUTING.md

I am still not convinced this is a good idea. I have read at several places that it is "bad practice" to define the manifest's namespace within the templates and that it would not be necessary, as Helm would take care of that by itself.
I have myself several deployments running in ArgoCD where namespace is not explicitly set in any of the templates. Could I see the Application definition from ArgoCD you are using @mustdiechik?

I have spun up a testcluster, did an installation of ArgoCD and will do some tests, also for #139

I am still not convinced this is a good idea. I have read at several places that it is "bad practice" to define the manifest's namespace within the templates and that it would not be necessary, as Helm would take care of that by itself. I have myself several deployments running in ArgoCD where namespace is not explicitly set in any of the templates. Could I see the Application definition from ArgoCD you are using @mustdiechik?

Hi @fibbs.
It is really bad practice to leave the namespace value fixed. You are right on that point.
But what I understand in the PR is that the value of the namespace field is dynamic depending on whether Helm and ArgoCD are using it at the installation.
I ran Helm with dry-run manually, and Helm is not really rendering the namespace in the resulting YAML files... I think that is what ArgoCD is complaining about...
I am finishing configuring ArgoCD and I will test it with and without the suggested change... then we will evaluate it with more information.

apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  creationTimestamp: "2024-12-30T15:33:01Z"
  generation: 49
  name: zabbix1
  namespace: argocd
  resourceVersion: "1631027"
  uid: 0e313ecc-d9b5-47c8-8938-7dfeecf8768f
spec:
  destination:
    namespace: zabbix1
    server: https://kubernetes.default.svc
  project: default
  source:
    chart: zabbix
    helm:
      parameters:
      - name: ingress.enabled
        value: "true"
      - name: ingress.hosts[0].host
        value: zabbix1.local
      - name: ingress.hosts[0].paths[0].paths
        value: /
      - name: ingress.hosts[0].paths[0].pathType
        value: ImplementationSpecific
      - name: zabbixServer.zabbixServerHA.enabled
        value: "false"
    repoURL: https://zabbix-community.github.io/helm-zabbix
    targetRevision: 6.1.2

@fibbs I had tha same result.
My application.yaml of the ArgoCD is:
# References:
#
# https://blog.argoproj.io/draft-argo-cd-v2-6-release-candidate-ced1853bbfdb
# https://github.com/argoproj/argo-cd/issues/2531
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: zabbix
  namespace: argocd
  annotations:
    # https://argo-cd.readthedocs.io/en/stable/user-guide/sync-waves/
    argocd.argoproj.io/sync-wave: "6"
  finalizers:
    # https://argo-cd.readthedocs.io/en/stable/user-guide/app_deletion/
    - resources-finalizer.argocd.argoproj.io
# https://argo-cd.readthedocs.io/en/stable/operator-manual/declarative-setup/
spec:
  destination:
    server: https://kubernetes.default.svc
    namespace: monitoring
  # https://argo-cd.readthedocs.io/en/release-2.6/user-guide/projects/
  project: default
  # https://argo-cd.readthedocs.io/en/release-2.6/user-guide/multiple_sources/
  sources:
  - repoURL: https://zabbix-community.github.io/helm-zabbix
    chart: zabbix
    targetRevision: 6.1.2
    helm:
      valueFiles:
      - $values/argocd/zabbix/zabbix_values.yaml
  - repoURL: https://gitlab.com/aeciopires/repo-test.git
    # Branch or tag name
    targetRevision: main
    ref: values
  # https://argo-cd.readthedocs.io/en/stable/user-guide/auto_sync/
  syncPolicy:
    automated:
      prune: true
      selfHeal: true
    # https://argo-cd.readthedocs.io/en/stable/user-guide/sync-options/
    syncOptions:
    - CreateNamespace=true

I create a git repository for values. The principal diference is:
I don't used ingress. I test with MetalLB using the LoadBalancer service type:
[...]

zabbixWeb:
  enabled: true
  ZBX_SERVER_HOST: zabbix-zabbix-server
  ZBX_SERVER_PORT: 10051
  service:
    # Using MetalLB
    type: LoadBalancer
    externalTrafficPolicy: Cluster
    nodePort: 31080
    port: 80
[...]

ingress:
  enabled: false
[...]

@mustdiechik can you share more information about your installation?


add ogre

macosx 上报错找不到符号sysctlbyname，我这没法调试

Since this package is not compatible with ogre1, shouldn't we call it ogre2 or ogre-next?

macosx 上报错找不到符号sysctlbyname，我这没法调试

#include <sys/sysctl.h> 没加，master 版本似乎加上了，可以打个 patch，参考

https://github.com/OGRECave/ogre-next/blob/4b997cc4963f02ed12d694dfa87306bf8bfdc7e8/OgreMain/src/OgrePlatformInformation.cpp#L49-L52

Since this package is not compatible with ogre1, shouldn't we call it ogre2 or ogre-next?

@xq114 It might be more appropriate to change to ogre-next, keep it consistent with the name on the github repo

Since this package is not compatible with ogre1, shouldn't we call it ogre2 or ogre-next?

Yes, ogre-next is a more appropriate name, since there is version 13.0.0 for ogre(1?). Mixing ogre and ogre-next can be confusing. I've renamed the package


Fix user facing encoding wrt #2023

So I kept the legacy parsing for both json and the query params to not break API for existing users.

There are a few small issues with coding style. It's worth taking a look at our coding standards. These should hopefully be easy to fix. +1

Could you be more specific?

Could you be more specific?

Sure! I was referring to the comments on the review. :+1:
AFAIK, all the issues are mentioned in these comments.

Wrt the build failure... I cannot reproduce it.

1. As mentioned in the [previous review](https://github.com/input-output-hk/cardano-wallet/pull/2055#pullrequestreview-473111599), I couldn't find a roundtrip property test for `decodePoolIdBech32` and `encodePoolIdBech32`. Would it be okay to add one?

That has been done: https://github.com/input-output-hk/cardano-wallet/pull/2055/commits/b52ee3ffd88efcd72a6e695d2c303998e9db8a65

bors r+

Can't reproduce the build failure, even with the given seed...

bors r+

a) hex-encoded pool ids are still accepted in path parameters

These are the tests that fail if hex encoding for path is disabled:
Failing tests

Failures:

  test/unit/Cardano/Wallet/ApiSpec.hs:137:42:
  1) Cardano.Wallet.Api, Malformed PathParam, (ApiT WalletId) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/śśśśśśśśśśśśśśśśśśśśśśśśśśśśśśśśśśśśśśśś
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected response body \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"wallet id should be a hex-encoded string of 40 characters\\\"}\", but received \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Invalid stake pool id: expecting a Bech32 encoded value with human readable part of 'pool'.\\\"}\""

  To rerun use: --match "/Cardano.Wallet.Api/Malformed PathParam/(ApiT WalletId) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347\347/"

  test/unit/Cardano/Wallet/ApiSpec.hs:137:42:
  2) Cardano.Wallet.Api, Malformed PathParam, (ApiT WalletId) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/111111111111111111111111111111111111111
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected response body \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"wallet id should be a hex-encoded string of 40 characters\\\"}\", but received \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Invalid stake pool id: expecting a Bech32 encoded value with human readable part of 'pool'.\\\"}\""

  To rerun use: --match "/Cardano.Wallet.Api/Malformed PathParam/(ApiT WalletId) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/111111111111111111111111111111111111111/"

  test/unit/Cardano/Wallet/ApiSpec.hs:137:42:
  3) Cardano.Wallet.Api, Malformed PathParam, (ApiT WalletId) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/11111111111111111111111111111111111111111
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected response body \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"wallet id should be a hex-encoded string of 40 characters\\\"}\", but received \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Invalid stake pool id: expecting a Bech32 encoded value with human readable part of 'pool'.\\\"}\""

  To rerun use: --match "/Cardano.Wallet.Api/Malformed PathParam/(ApiT WalletId) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/11111111111111111111111111111111111111111/"

  test/unit/Cardano/Wallet/ApiSpec.hs:142:42:
  4) Cardano.Wallet.Api, Malformed BodyParam, (ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected response body \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Error in $.passphrase: passphrase is too long: expected at most 255 characters\\\"}\", but received \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Invalid stake pool id: expecting a Bech32 encoded value with human readable part of 'pool'.\\\"}\""

  To rerun use: --match "/Cardano.Wallet.Api/Malformed BodyParam/(ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:142:42:
  5) Cardano.Wallet.Api, Malformed BodyParam, (ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected response body \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Error in $.passphrase: parsing Text failed, expected String, but encountered Number\\\"}\", but received \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Invalid stake pool id: expecting a Bech32 encoded value with human readable part of 'pool'.\\\"}\""

  To rerun use: --match "/Cardano.Wallet.Api/Malformed BodyParam/(ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:142:42:
  6) Cardano.Wallet.Api, Malformed BodyParam, (ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected response body \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Error in $.passphrase: parsing Text failed, expected String, but encountered Array\\\"}\", but received \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Invalid stake pool id: expecting a Bech32 encoded value with human readable part of 'pool'.\\\"}\""

  To rerun use: --match "/Cardano.Wallet.Api/Malformed BodyParam/(ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:142:42:
  7) Cardano.Wallet.Api, Malformed BodyParam, (ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected response body \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Error in $.passphrase: parsing Text failed, expected String, but encountered Number\\\"}\", but received \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Invalid stake pool id: expecting a Bech32 encoded value with human readable part of 'pool'.\\\"}\""

  To rerun use: --match "/Cardano.Wallet.Api/Malformed BodyParam/(ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:142:42:
  8) Cardano.Wallet.Api, Malformed BodyParam, (ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected response body \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Error in $: parsing Cardano.Wallet.Api.Types.ApiWalletPassphrase(ApiWalletPassphrase) failed, expected Object, but encountered Number\\\"}\", but received \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Invalid stake pool id: expecting a Bech32 encoded value with human readable part of 'pool'.\\\"}\""

  To rerun use: --match "/Cardano.Wallet.Api/Malformed BodyParam/(ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:142:42:
  9) Cardano.Wallet.Api, Malformed BodyParam, (ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected response body \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Error in $: parsing Cardano.Wallet.Api.Types.ApiWalletPassphrase(ApiWalletPassphrase) failed, expected Object, but encountered String\\\"}\", but received \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Invalid stake pool id: expecting a Bech32 encoded value with human readable part of 'pool'.\\\"}\""

  To rerun use: --match "/Cardano.Wallet.Api/Malformed BodyParam/(ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:142:42:
  10) Cardano.Wallet.Api, Malformed BodyParam, (ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected response body \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"trailing junk after valid JSON: endOfInput\\\"}\", but received \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Invalid stake pool id: expecting a Bech32 encoded value with human readable part of 'pool'.\\\"}\""

  To rerun use: --match "/Cardano.Wallet.Api/Malformed BodyParam/(ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:142:42:
  11) Cardano.Wallet.Api, Malformed BodyParam, (ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected response body \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"I couldn't understand the content of your message. If your message is intended to be in JSON format, please check that the JSON is valid.\\\"}\", but received \"{\\\"code\\\":\\\"bad_request\\\",\\\"message\\\":\\\"Invalid stake pool id: expecting a Bech32 encoded value with human readable part of 'pool'.\\\"}\""

  To rerun use: --match "/Cardano.Wallet.Api/Malformed BodyParam/(ApiWalletPassphrase) PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:148:46:
  12) Cardano.Wallet.Api, Malformed Headers, ("Accept") PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected status code 406, but received 400"

  To rerun use: --match "/Cardano.Wallet.Api/Malformed Headers/(\"Accept\") PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:155:46:
  13) Cardano.Wallet.Api, Malformed Headers, ("Content-Type") PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected status code 415, but received 400"

  To rerun use: --match "/Cardano.Wallet.Api/Malformed Headers/(\"Content-Type\") PUT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:162:42:
  14) Cardano.Wallet.Api, Not Allowed Methods,  GET /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected status code 405, but received 400"

  To rerun use: --match "/Cardano.Wallet.Api/Not Allowed Methods/ GET /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:162:42:
  15) Cardano.Wallet.Api, Not Allowed Methods,  POST /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected status code 405, but received 400"

  To rerun use: --match "/Cardano.Wallet.Api/Not Allowed Methods/ POST /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:162:42:
  16) Cardano.Wallet.Api, Not Allowed Methods,  PATCH /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected status code 405, but received 400"

  To rerun use: --match "/Cardano.Wallet.Api/Not Allowed Methods/ PATCH /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:162:42:
  17) Cardano.Wallet.Api, Not Allowed Methods,  DELETE /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected status code 405, but received 400"

  To rerun use: --match "/Cardano.Wallet.Api/Not Allowed Methods/ DELETE /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:162:42:
  18) Cardano.Wallet.Api, Not Allowed Methods,  CONNECT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected status code 405, but received 400"

  To rerun use: --match "/Cardano.Wallet.Api/Not Allowed Methods/ CONNECT /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:162:42:
  19) Cardano.Wallet.Api, Not Allowed Methods,  TRACE /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected status code 405, but received 400"

  To rerun use: --match "/Cardano.Wallet.Api/Not Allowed Methods/ TRACE /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

  test/unit/Cardano/Wallet/ApiSpec.hs:162:42:
  20) Cardano.Wallet.Api, Not Allowed Methods,  OPTIONS /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000
       uncaught exception: WaiTestFailure
       WaiTestFailure "Expected status code 405, but received 400"

  To rerun use: --match "/Cardano.Wallet.Api/Not Allowed Methods/ OPTIONS /stake-pools/0000000000000000000000000000000000000000000000000000000000000000/wallets/0000000000000000000000000000000000000000/"

b) bech32 encoded pool ids are accepted in path parameters

I believe we just have to adjust this:
instance Wellformed (PathParam ApiPoolId) where
    wellformed = PathParam $ T.replicate 64 "0"

And add a bech32 param there.

bors r+


Create an updated release

There has been a few commits lately and no release. Latest release doesn't even support tls certificates for the amqp alive test.

Version 1.2.0 has been released.


[FLINK-28447] GitHub Pull Request template

I have started from the template in the main Flink repo and made some simplification given the smaller scope of the project.
Let me know if you prefer to make the guide more concise.

Thanks, @gyfora. Addressed your comments.


Lf 1524 extend flows for claim page

Success State:

Error State:

Pending State:

Claiming State:
...
Not Qualified State:


Visual simulation step-by-step

A terminal cipher simulator/demonstration showing the encryption/decryption step-by-step.
It should look something like this (where the @ sign is the chess piece as an example), for example the English 6x6 version would look like
#@ _  2  3  4  5
6  7  8  9  a  b
c  d  e  f  g  h
i  j  k  l  m  n
o  p  q  r  s  t
u  v  w  x  y  z

@DonaldTsang, thanks for the nice idea.
This is implemented.
When calling encrypt or decrypt, there is now an optional verbose option that will result in a step-by-step algorithm visualization. The @ sign shows the current location, and the green highlighting shows the row and column that have shifted.
Here's a screenshot that captures the first few rows of example output:

Nice!


React native Unit testing

HI all
This is not a issue but information sharing kindof thing
I have a basic class which extends react native component
_export default class testProject extends Component {
sum(a, b) {
return a + b;
}
render() {
return (
Business Finance

);
}
}
How can I write unit tests for both UI and Business logic :)

You can read this articles then you can write test :)
https://facebook.github.io/jest/docs/en/getting-started.html
https://blog.callstack.io/unit-testing-react-native-with-the-new-jest-i-snapshots-come-into-play-68ba19b1b9fe
https://blog.callstack.io/unit-testing-react-native-with-the-new-jest-ii-redux-snapshots-for-your-actions-and-reducers-8559f6f8050b
https://medium.com/@romarioraff/testing-react-apps-with-jest-and-chai-enzyme-for-beginners-baa5f57cec0b
https://medium.com/@dschmidt1992/jest-snapshot-testing-3ef9fa1222bb

@facebook-github-bot answered

after a day of reading documents a stright answer for the question
For UI -> Enzyme and Jest example :->
import Index from '../index.android.js';
const expectedText = 'Business Finance';
let wrapper=shallow(<Index />);
expect(wrapper.find('Text').children().text()).toEqual(expectedText);

For Businesslogic testing ->Enzyme
expect(wrapper.instance()._sum(2,3)).toBe(5);


Set-slots implementation

Particles that consume set-slots need to:

have template contain just the single slot’s template
have model contain list of items, each item needs to have a unique subId identifier that will be used to match the item to its proper slot.
use single “annotation” slot for multiple particles

On the slot-composer side:

Refactored dom-slot’s context to a separate classes, now supporting either singleton or set slot.
added observer to monitor ongoing changes and update inner slots on the fly (this functionality was lost when migrated to new slot-composer implementation)

Next steps:

update strategies to match provided-consumed set-of vs singleton slots
update remote-slots strategy and slot-composer to support multiple particle providing slot with the same name (Chooser to also provide annotation for arrival info)
migrate the demo to schema.org based schemas and use proper “identifier” field.

Scott, as discussed, i reverted all changes to xen-* files and made the changes we agreed on. PTAL

\o/


Fix CI

Update Redox compiler
Fix "unknown lint clippy::if_then_panic"

The dfly failure is due to a change I submitted to libc. The libc policy on breaking changes is to deprecate the item in a release prior to actually making the change (and removing the deprecation).
It's listed as a tier 3 target, so I think the correct move is for the failure to be ignored at the build level, but we could also trivially disable deprecation warnings on dfly until the change to libc is submitted.

Yeah, since there isn't a non-deprecated alternative, I think we should silence those deprecation warnings for now.

bors r+


Error with Calico wait for etcd

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
Bug report
Environment:

Cloud provider or hardware configuration:
3 bare metal nodes

OS (printf "$(uname -srm)\n$(cat /etc/os-release)\n"):

Linux 4.4.0-103-generic x86_64
NAME="Ubuntu"
VERSION="16.04.3 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.3 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

Version of Ansible (ansible --version):

ansible-playbook <IP_ADDRESS>
  config file = /Users/focaaby/Project/kubespray/ansible.cfg
  configured module search path = [u'/Users/focaaby/Project/kubespray/library']
  ansible python module location = /usr/local/Cellar/ansible/<IP_ADDRESS>_1/libexec/lib/python2.7/site-packages/ansible
  executable location = /usr/local/bin/ansible-playbook
  python version = 2.7.14 (default, Sep 25 2017, 09:54:19) [GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.37)]

Kubespray version (commit) (git rev-parse --short HEAD):
79417e07
Network plugin used:
Calico
Copy of your inventory file:
[all]
node1 ansible_host=<IP_ADDRESS> ip=<IP_ADDRESS>
node2 ansible_host=<IP_ADDRESS> ip=<IP_ADDRESS>
node3 ansible_host=<IP_ADDRESS> ip=<IP_ADDRESS>

[kube-master]
node1
node2

[kube-node]
node1
node2
node3

[etcd]
node1
node2
node3

[k8s-cluster:children]
kube-node
kube-master

[calico-rr]

[vault]
node1
node2
node3

Command used to invoke ansible:
ansible-playbook --flush-cache -i inventory/inventory.cfg cluster.yml -b -vvvv -u prlab -K
Output of ansible run:
https://gist.githubusercontent.com/focaaby/fca02a271c6c63176b486e07908851ed/raw/722c3526d83f9c790bc8be1bc73187e97a5c0735/gistfile1.txt
I had found similar issue  https://github.com/kubernetes-incubator/kubespray/issues/1466, after I tried --flush-cache flag and removed all files  /tmp directory, it still same error.

I have the same issue, how did you solve it?

Hello,
I am also facing this issue. Is there any work around to avoid this issue? Or any work is going on?
Please update.

It seems you have non zero swap on node1 and node2:
fatal: [node1]: FAILED! => {
    "assertion": "ansible_swaptotal_mb == 0",
    "changed": false,
    "evaluated_to": false
}
fatal: [node2]: FAILED! => {
    "assertion": "ansible_swaptotal_mb == 0",
    "changed": false,
    "evaluated_to": false
}


Featured: Is it add long text for message ?

Hello thanks for your great plugin
Could you add more long text for message maybe around 3 lines for that ?
thanks

see: https://github.com/dkcas11/CRNotifications/issues/6


1.9.4 support

It currently doesn't support 1.9.4. A protocol bump and some changed packets.

I made some progress in
finalchild/1.9.4

DONE.


Eclipse Angus support

Hi guys,
Similiar to the Jakarta Mail update in #832 I would now like to ask, if you plan to add support for Eclipse Angus?
According to https://eclipse-ee4j.github.io/angus-mail/ Eclipse Angus is the successor of Jakarta Mail as denoted on https://github.com/jakartaee/mail-api
Edit: see https://eclipse-ee4j.github.io/angus-mail/#Download_Angus_Mail_Release for a comparison of artefacts
Do you have plans to support this version besides javax and jakarta as well? It's a big version mess....
Thanks and BR,
Philip

Forget it - my bad. I think I just had a weird combination of dependencies...


Is it possible to add a VPN client to the raspiblitz firmware?

Hello, TOR is blocked in Russia, can you create a VPN server outside of Russia, and install a VPN client on raspiblitz to bypass the TOR blocking? Otherwise, even the blockchain has not been synchronized for six months. I asked people in our chat (Raspiblitz_RU) in a telegram, they say that this is happening because of DPI and we need to pack everything into the VPN tunnel. Can you help with solving this problem?

Iam using OpenVPN to a VPS as OpenVPN Server

# install openvpn and setup client
sudo apt install openvpn
sudo cp /mnt/hdd/yourclient.ovpn /etc/openvpn/client.conf
sudo systemctl start openvpn@client
sudo systemctl enable openvpn@client
# set firewall
# need to check own public IP (UPDATE  EXTERNAL IP HERE replace <IP_ADDRESS>)
sudo ufw allow out on eth0 to <IP_ADDRESS>/32
# allow OpenVPN Client connection
sudo ufw allow out 1194/udp
# allow traffic via openvpn tunnel only
sudo ufw allow out on tun0 from any to any
sudo ufw allow in on tun0 from any to any

And a script to check connectivity

#!/bin/sh
ping -q -c 4 <IP_ADDRESS> > /dev/null
if [ $? -eq 0 ]; then
echo "date: Connectivity check successful"
else
echo "date: Connectivity check failed"
dhclient -r; dhclient > /dev/null
sleep 15
systemctl restart openvpn@client
sleep 15
systemctl restart nginx
echo "date: Connectivity repaired... hopefully"
exit
fi

an open VPN will not cope, because VPN traffic is detected by the provider and blocked in Russia. We need a VPN that could mask VPN traffic, accessing the server and establishing a connection should take place using an unencrypted protocol, after that the server should receive a certain command to open a certain port and only after that the VPN should be turned on. There is a description of the problem and ways to solve it, can you do as recommended in the new raspiblitz build? I installed version 1.9, but it also cannot establish a connection with knives due to VPN blocking. https://habr.com/ru/post/415977/


(Consider how to) add spack

How will spack work?

Dependencies required by the core and one or more satellite repos exist centrally (i.e. here?)
Dependencies unique to any one repo exists in just that repo?
What about a dep required by two or more satellite repo - which one "owns" that dep?
Keep all spack config for everything in an entirely separate repo? How do we go about updating two things at once? Can it be automated via Actions?

@will-saunders-ukaea had some thoughts on this.

And we'll need GH actions to make use of a docker container with all the spack deps in, otherwise it'll spend forever just building sycl.

Creating a custom docker image with spack is probably the best bet: https://spack.readthedocs.io/en/latest/containers.html

Is there anything here that wasn't covered by PR #93 ? If so, perhaps we could create a more focussed issue and close this one?


Feature Request: Do not allow image width that is lesser than page width to fill/fit the remaining page width

Currently the image smaller than the page width will always fill the full page width, which cause some less wide image to zoomed in.
Describe the solution you'd like
Maybe using javascript to render the image, like Lightbox with magnifier ?
Additional context
Notice how the second image is zoomed in because its width is less than the page width.

Sound reasonable. Unfortunately, I am little busy now. So, it may take some time to implement.


Created signin.css

Have created signin.css file from scratch, for Already present file signin.html (Also created by me)
@arghadipmanna101 I request you to please merge this as without css , signin.html file is looking ugly
to the issue #131
Have created Pr already related to this issue but it  doesnt merge till now , So I have created this new Pr . Thankyou
#254 , #337

@arghadipmanna101 sir , have a look on this PR thanks.

@arghadipmanna101 please label as gssoc . Thanks


docker: simplify non-root user setup

It turns out that our base container image already has a "nonroot" user, with uid/gid 65532.
The previous setup worked more of less out of the box when running as uid 1000 (the default uid for the initial user on most linux setups). Since we changed this default we need to document how to specify the required user.
Followup to #302.

@mmikitka: does this still work for you?

@mostynb Sorry for not catching that in my PR, but yes this works for me. Thanks for asking.

No problem, I didn't catch it either. That dive tool is pretty nice.


on() event listener is not called if MetaMask is disabled

Hey team, we discovered an issue in Web3Modal that on listener of getWallets not getting triggered depending on the available extension options.
In example, I have MetaMask, Phantom, Trust enabled;

Case 1: If all of them enabled, the event listener getting triggered and returns these three wallets as list. And get() returns three wallets as expected
Case 2 If MetaMask disabled and others enabled, the listener is not getting triggered. But get() returns three wallets as expected.

As I did different combinations, it seems like if MetaMask is not enabled or not installed, it's not triggering the listeners. In any case, the get() function returns the wallets as expected. This is only related with on() listener.
Here is the simplified code sample that we created our watch method;
const { get, on } = getWallets()

let standardAdapters: StandardWalletAdapter[] = [...get()]

export function watchStandard(callback: (arg: StandardWalletAdapter[]) => void) {
  return () => [
    on('register', (...wallets) => {
      standardAdapters = [...standardAdapters, ...wallets]
      callback(standardAdapters)
    }),
    on('unregister', (...wallets) => {
      ...
      callback(standardAdapters)
    })
  ].forEach(off => off())
}

// Registering event listeners
watchStandard(standardAdapters => {
  // Works in Case 1, but not in Case 2
  console.log("Standard adapters have changed", standardAdapters)
})

Simply adding a initial callback trigger resolving the problem. But this seems an issue to me and wanted to double check with you.
export function watchStandard(callback: (arg: StandardWalletAdapter[]) => void) {
+ callback(standardAdapters)
  return () => [
    on('register', (...wallets) => {
      standardAdapters = [...standardAdapters, ...wallets]
      callback(standardAdapters)
    }),
    ...
  ].forEach(off => off())
}


Any possible support nested bucket?

Currently, use Upsert or Insert  to insert or update value is convenience without create bucket, but in some cases need nested bucket that bolthold not currently supported.
Is there any plane to support nested bucket?

It might be possible.  Right now you have direct access to the underlying bolt DB, so you can use nested buckets, just not with the querying interface.
What is the scenario where this would be useful?  You can insert and query nested structs in Bolthold without needing nested buckets.

Sorry for late response.
I have some nested data like this:
|- project1(bucket)
|    |__ device_type_1(bucket)
|    |        |__ device_1(kv)
|    |        |__ device_2(kv)
|    |
|    |__ device_type_2(bucket)
|    |        |__ device_1
|    |        |__ device_2
|    |
|- project2(bucket)
|
store same type data in same bucket, so can quickly locate the target bucket with some specific role(like name + id), also update or insert item in same bucket.
This is my scenario. Yes, with direct access to the underlying bolt DB, I can create nested nested buckets, but it will lose all of the benifit with bolthold, can't use exists nested bucket  and query with bolthold.
Any alternative way to achieve this?

So do you actually need querying at the project or device_type levels?  If not, then you could simply put them in different database files.  It would also allow for better performance since you won't have to wait on writes.

I think I've got this sorted out, just need to write some tests and I'll merge the changes.

I have already know the project bucket name, and need query device_type. All about this is need a way to setup the custom bucket name as the operation bucket, then update or insert item on it.
Different database files is the alternative way, but adds administrative overhead, need a way to join queries from different databases.

Nested Bucket support Added in #72

pushed to master, open up a new issue if you run into anything.
Thanks,


[Snyk] Fix for 2 vulnerabilities

Snyk has created this PR to fix 2 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

docs/package.json
docs/package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Regular Expression Denial of Service (ReDoS) SNYK-JS-PATHTOREGEXP-7925106
  666

Cross-site Scripting (XSS) SNYK-JS-SERIALIZEJAVASCRIPT-6147607
  626

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)
🦉 Cross-site Scripting (XSS)

Right, 👍


Fix formatting of final example in Getting Started: Themeing

Matches the previous examples to be in a syntax highlighter block

Thanks for this @cameronbaney!


python311Packages.pymc: 5.10.0 -> 5.10.1, python311Packages.pytensor: 2.18.1 -> 2.18.2

Description of changes
Changelog pymc: https://github.com/pymc-devs/pymc/releases/tag/v5.10.1
Changelog pytensor: https://github.com/pymc-devs/pytensor/releases/tag/v2.18.2
cc @nidabdella @bcdarwin

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 273331 run on x86_64-linux 1

12 packages built:

python310Packages.bambi
python310Packages.bambi.dist
python310Packages.pymc
python310Packages.pymc.dist
python310Packages.pytensor
python310Packages.pytensor.dist
python311Packages.bambi
python311Packages.bambi.dist
python311Packages.pymc
python311Packages.pymc.dist
python311Packages.pytensor
python311Packages.pytensor.dist

Result of nixpkgs-review pr 273331 run on aarch64-linux 1

12 packages marked as broken and skipped:

python310Packages.bambi
python310Packages.bambi.dist
python310Packages.pymc
python310Packages.pymc.dist
python310Packages.pytensor
python310Packages.pytensor.dist
python311Packages.bambi
python311Packages.bambi.dist
python311Packages.pymc
python311Packages.pymc.dist
python311Packages.pytensor
python311Packages.pytensor.dist


:checkered_flag: Enabling building and testing under Windows

In order to build and run all tests under Windows I did the following:

Get rid of all bash related commands in package.json and replaced them with pure node solutions (e.g. del-cli and ncp instead of "rm -rf" and "cp")

Replaced test-integration.sh with corresponding test-integration.js to enable cross-platform execution of docker commands. Also built in check whether Docker is installed. The integration test will pass if docker is not installed though, because an integration test is not possible to do then.

Hello, thanks for your contribution ;)
I've totally forgotten about testing on Windows environment so far, I appreciate you for bringing this up!
It would be great to have a Windows CI environment but it seems like Travis doesn't support it yet. I cannot find any mentions of the Windows environment in https://docs.travis-ci.com/user/multi-os/ and found some old Github issue on this topic https://github.com/trailsjs/trails/issues/110, which concludes by suggesting to use https://www.appveyor.com/.
I think we should continue maintaining the util scripts to be cross-platform as your PR suggests, and I'm glad to learn something new thanks to your PR.
I'm happy to take care of the CI stuff unless you want to give it a try =) I think you can set up a personal Appveyor project and add a config file for testing, then commit that config file in the end with some instruction on how I can set up the Appveyor project from my end.
I personally do not like to manage too many different CIs at once for a project, so if Appveyor works well, then I think we should switch to it. Moreover, it seems like Appveyor is also Free for open-source projects just like Travis =)
Let me know your thought,
Cheers

Hello, thanks for your contribution ;)
I've totally forgotten about testing on Windows environment so far, I appreciate you for bringing this up!
It would be great to have a Windows CI environment but it seems like Travis doesn't support it yet. I cannot find any mentions of the Windows environment in https://docs.travis-ci.com/user/multi-os/ and found an old Github issue on this topic trailsjs/trails#110, which concludes by suggesting to use https://www.appveyor.com/.
I think we should continue maintaining the util scripts to be cross-platform as your PR suggests, and I'm glad to learn something new thanks to your PR.
I'm happy to take care of the CI stuff unless you want to give it a try =) I think you can set up a personal Appveyor project and add a config file for testing, then commit that config file in the end with some instruction on how I can set up the Appveyor project from my end.
I personally do not like to manage too many different CIs at once for a project, so if Appveyor works well, then I think we should switch to it. Moreover, it seems like Appveyor is also Free for open-source projects just like Travis =)
Let me know your thoughts,
Cheers

The usage of Appveyour seems like a good move if you plan to support Windows in the future. As you are the owner of this repository I guess you need to set this one up. There's also a Github Marketplace link for that: https://github.com/marketplace/appveyor/
My goal with this PR was just to make it compile under Windows again to start working on the real features :)

Hey, if you would like to work on the new features, you can just branch off from develop, which contains updates from this PR ;)
I will document this in the CONTRIBUTION.md


Clarify how  kubernetes_sd_configs works for scraping pods, replicasets, etc.

It was really non obvious to me how I would scrape a replica set where the targets are dynamic pods.  i.e. how does Prom know how/where to contact the dynamic pods?
The blog that connected the dots for me was:
http://marselester.com/prometheus-on-kubernetes.html
 source_labels: [__meta_kubernetes_pod_label_app]
        regex: hello-server

Means "look for a pod that has meta data app=hello-server".
A short example snippet connecting these concepts together would really help.

I still don’t understand how Prometheus discovers the right pods after reading the blog post above.  E.g., how does it know to look at “hello:8000/metrics” and not, for example, “foo:8088/management/prometheus”; and how does it know to look for pods with the label “app: hello-server” and not, for example, “app: bar”?

@Typraeurion Hope below snippet helps you
scrape_configs:
  - job_name: 'activemq monitoring'
    # metrics_path defaults to '/metrics'
    # scheme defaults to 'http'.
    # The HTTP resource path on which to fetch metrics from targets.
    metrics_path: '/activemq-monitoring-app/metrics'


allow bulkPut to accept object with keys as primary key

I'd like to load https://www.floatrates.com/daily/usd.json into a dexie table, but the data is returned as an object, with each key of the object being the primary key of the object.

  "aud": {
    "code": "AUD",
    "alphaCode": "AUD",
    "numericCode": "036",
    "name": "Australian Dollar",
    "rate": 1.5228242038472,
    "date": "Fri, 15 Mar 2024 23:55:10 GMT",
    "inverseRate": 0.65667461646174
  },
  "cad": {
    "code": "CAD",
    "alphaCode": "CAD",
    "numericCode": "124",
    "name": "Canadian Dollar",
    "rate": 1.3531700713821,
    "date": "Fri, 15 Mar 2024 23:55:10 GMT",
    "inverseRate": 0.7390054074863
  },

There is probably some slick javascript way to process these, but it'd be a cool feature if bulkAdd detected this format and treated it accordingly.  Obviously, it wouldn't work for compound keys, but I have to imagine that this formatted is somewhat common.
In the meantime, I'll use this:
const sendUrl = "https://www.floatrates.com/daily/usd.json";
                const sendType = "GET";
                fetch(sendUrl, {
                    method: sendType,
                    headers: {
                        'Accept': 'application/json',
                    },
                })
                    .then(data => {
                        let arrayValues=[];
                        Object.keys(data).forEach(key => {
                            const value = data[key];
                            value['lcode'] = key;
                            console.log(value);
                            arrayValues.push(value);
                        });
                        console.log(arrayValues, db);
                        return db.currency.bulkAdd(arrayValues);
                    });

Thanks for your consideration.

I try to avoid special sugar unless they are extremely obvious or expected. Docs of the API would have to grow and with it, the time spent for users to understand it. In this case it's easy to implement once it's clear how bulkAdd works. Having this option would require some special options parameter or another bulk method specific to this. Closing this ticket. Feel free to reopen of there's something clearly convincing to implement it.


Can't --require custom UI and use it

In the Third party UIs wiki page, it is said that one can --require a module which enriches the mocha.interfaces object bag with a custom UI and directly use it with the --ui option.
This doesn't seem to work.
Looking at the latest bin/_mocha, I can see why: the UI lookup happens before all the requires are required.
Is this a regression?

@joaomoreno don't know if it's a regression or not, but this seems like a bug.

@joaomoreno what 3p UI are you using?

@boneskull I'm not using one 3p UI. I followed the aforementioned guide, in which you defined a UI in a JS module, --require it and refer to it using the --ui option. Just follow the guide using the latest version and you'll see what I mean.

@boneskull Please check joaomoreno/mocha@102e1402fcb61f772940c085517352198885f163. This adds an integration test which tests for the exact same example that is explained in the Wiki. Run it with:
mocha test/integration/regression.js

@joaomoreno It's been a bit since I wrote the page, and I'll give it an update tomorrow. But try running npm install mocha. Prior to doing so, I got:
$ mocha --require ./simple-ui.js --ui simple-ui test.js
/usr/local/lib/node_modules/mocha/lib/mocha.js:174
  if (!this._ui) throw new Error('invalid interface "' + name + '"');
                       ^
Error: invalid interface "simple-ui"
    at Mocha.ui (/usr/local/lib/node_modules/mocha/lib/mocha.js:174:24)
    at Object.<anonymous> (/usr/local/lib/node_modules/mocha/bin/_mocha:210:7)
    at Module._compile (module.js:410:26)
    at Object.Module._extensions..js (module.js:428:10)
    at Module.load (module.js:335:32)
    at Function.Module._load (module.js:290:12)
    at Function.Module.runMain (module.js:451:10)
    at startup (node.js:123:18)
    at node.js:868:3

But after:
$ cd Desktop/mocha-test/
dstjules:~/Desktop/mocha-test

$ cat test.js
test('pass', function() {
  // pass
});

test('fail', function() {
  throw new Error('oops!');
});

dstjules:~/Desktop/mocha-test
$ cat simple-ui.js
var Mocha = require('mocha');
    Suite = require('mocha/lib/suite'),
    Test  = require('mocha/lib/test');

/**
 * A simple UI that only exposes a single function: test
 */
module.exports = Mocha.interfaces['simple-ui'] = function(suite) {
  suite.on('pre-require', function(context, file, mocha) {
    var common = require('mocha/lib/interfaces/common')([suite], context);

    context.run = mocha.options.delay && common.runWithSuite(suite);

    /**
     * Describes a specification or test-case with the given `title`
     * and callback `fn` acting as a thunk.
     */
    context.test = function(title, fn) {
      var test = new Test(title, fn);
      test.file = file;
      suite.addTest(test);

      return test;
    };
  });
};

dstjules:~/Desktop/mocha-test
$ mocha --version
2.2.5

dstjules:~/Desktop/mocha-test
$ mocha --require ./simple-ui.js --ui simple-ui test.js

  ✓ pass
  1) fail

  1 passing (12ms)
  1 failing

  1)  fail:
     Error: oops!
      at Context.<anonymous> (test.js:6:9)

Your answer led me to the real issue: it fails when simple-ui.js lives in a child directory. Try the following:
mkdir foo
mv simple-ui.js foo
mocha --require ./foo/simple-ui.js --ui simple-ui test.js

it fails when simple-ui.js lives in a child directory

Looks like it fails when the ui lives anywhere outside of the cwd, instead? Should be easier to hunt down what to change :)

Yep sounds like it!

OK, so the sample works not because you provide --require to mocha, but because of this require call. It will also work if you simply run
mocha --ui simple-ui test.js

This makes sense, given the fact that requiring the modules defined via --require will happen after the ui resolution.
Knowing this, a potential workaround is to provide the full path to the UI module:
mocha --ui /full/path/to/simple-ui.js test.js

This actually works, but doesn't help me in my use case: I need to add the --ui parameter in a mocha.opts file, which knows nothing about its own full path, much less simple-ui.js's.

Is anyone working on a fix for this yet? If not, I'll try to find some time to work on it.

I came up with a finding which at least works for me.
For any Custom UI it is not necessary or even required to add it to --require, just adding it to --ui will load it automatically.
Other point is --ui does not accept the name of ui which is defined in Mocha.interfaces['tags'], it's the name of the npm package/module in which that ui resides.
So if we have a ui my-ui in a package called my-style-ui
module.exports = Mocha.interfaces['my-ui'] = function(suite) {

For running it we need to specify --ui my-style-ui not my-ui

@nazarhussain That's true for UI plugins published on NPM, but if you implemented an UI plugin in your project, you shoud use both --require and --ui. Of course, this is a rare case, but I encountered this problem when I developed a plugin.

Since we had identified the issue... so we should add a note to readme and close this issue.

@joaomoreno Fixed via https://github.com/mochajs/mocha/commit/a61b1f3287b436522545e5458d071d7e791c15d0 Included your test case, and updated it to work with local mocha. https://github.com/mochajs/mocha/commits/master

So will remain that a ui required via npm will need to use -ui npm-module-name and not the ui name?


Error running in Jenkins

This is working fine in my development environment.
When I run from Jenkins - I get the following errors.

However - it looks like possibly it is using the local jenkins ruby to run in the jenkins environment.

<plugin>
	<groupId>nl.geodienstencentrum.maven</groupId>
	<artifactId>sass-maven-plugin</artifactId>
	<version>2.25</version>
	<executions>
		<execution>
			<id>compile</id>
			<goals>
				<goal>update-stylesheets</goal>
			</goals>
			<phase>compile</phase>
			<configuration>
				<sassSourceDirectory>${basedir}/src/versioned/sass</sassSourceDirectory>
				<destination>${project.build.directory}/compiled-sass/css</destination>
			</configuration>
		</execution>
	</executions>

</plugin>

Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=256m; support was removed in 8.0
<===[JENKINS REMOTING CAPACITY]===>channel started
Executing Maven:  -B -f /var/lib/jenkins/jobs/Build Common Assets/workspace/pom.xml clean install -X -U
Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5; 2015-11-11T05:41:47+13:00)
Maven home: /var/lib/jenkins/tools/hudson.tasks.Maven_MavenInstallation/Maven_3.3.9
Java version: 1.8.0_72, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-8-oracle/jre
Default locale: en_NZ, platform encoding: UTF-8
OS name: "linux", version: "3.13.0-125-generic", arch: "amd64", family: "unix"

[SNIP]

[INFO] --- sass-maven-plugin:2.25:update-stylesheets (compile) @ common-assets ---
[INFO] Compiling Sass templates
[INFO] No resource element was specified, using short configuration.
[DEBUG] Setting source directory: /var/lib/jenkins/jobs/Build Common Assets/workspace/src/versioned/sass
[DEBUG] Setting includes: [**/*.scss]
[INFO] Queueing Sass template for compile: /var/lib/jenkins/jobs/Build Common Assets/workspace/src/versioned/sass => /var/lib/jenkins/jobs/Build Common Assets/workspace/target/compiled-sass/css
[DEBUG] Execute Sass Ruby script:

require 'rubygems'
env = { 'GEM_PATH' => [
  '/var/lib/jenkins/jobs/Build Common Assets/workspace/target/rubygems',
  'file',
  '/var/lib/jenkins/plugins/ruby-runtime/WEB-INF/lib/stapler-jruby-1.209.jar!/gem'
].uniq.join(File::PATH_SEPARATOR) }
Gem.paths = env
require 'sass/plugin'
require 'java'
Sass::Plugin.options.merge!(
  :css_location => '/var/lib/jenkins/jobs/Build Common Assets/workspace/target/compiled-sass/css',
  :cache => true,
  :always_update => true,
  :template_location => '/var/lib/jenkins/jobs/Build Common Assets/workspace/src/versioned/sass',
  :cache_location => '/var/lib/jenkins/jobs/Build Common Assets/workspace/target/sass_cache',
  :unix_newlines => true,
  :style => :expanded
)
Sass::Plugin.on_compilation_error {|error, template, css| $compiler_callback.compilationError(error.message, template, css) }
Sass::Plugin.on_updated_stylesheet {|template, css| $compiler_callback.updatedStylesheeet(template, css) }
Sass::Plugin.on_template_modified {|template| $compiler_callback.templateModified(template) }
Sass::Plugin.on_template_created {|template| $compiler_callback.templateCreated(template) }
Sass::Plugin.on_template_deleted {|template| $compiler_callback.templateDeleted(template) }
require 'pp'
pp Sass::Plugin.options
Sass::Plugin.update_stylesheets

Sass is in the process of being separated from Haml,
and will no longer be bundled at all in Haml 3.2.0.
Please install the 'sass' gem if you want to use Sass.
NoMethodError: undefined method `on_updated_stylesheet' for Sass::Plugin:Module
Did you mean?  update_stylesheets
method_missing at org/jruby/RubyBasicObject.java:1657
method_missing at /var/lib/jenkins/plugins/ruby-runtime/WEB-INF/lib/stapler-jruby-1.209.jar!/gem/gems/haml-3.1.1/vendor/sass/lib/sass/plugin.rb:113
        <main> at <script>:20

nb. If SSH into the jenkins box directly, and run mvn clean install -X directly there - it doesn't have a problem, and that ruby line looks like:
require 'rubygems'
env = { 'GEM_PATH' => [
    '/var/lib/jenkins/jobs/Build Common Assets/workspace/target/rubygems'
].uniq.join(File::PATH_SEPARATOR) }

This suggests that Jenkins itself if somehow modifying how the plugin is being run.

Ok, I think I've found the issue.
In AbstractSassMojo.java you have
`		sassScript.append("require 'rubygems'\n");
if (this.gemPaths.length > 0) {
sassScript.append("env = { 'GEM_PATH' => [\n");
for (final String gemPath : this.gemPaths) {
sassScript.append("    '").append(gemPath).append("',\n");
}
		final String gemPath = System.getenv("GEM_PATH");
		if (gemPath != null) {
			for (final String p : gemPath.split(File.pathSeparator)) {
				sassScript.append("    '").append(p).append("',\n");
			}
		}
		/* remove trailing comma+\n */
		sassScript.setLength(sassScript.length() - 2);
		// TODO
		// quick fix for the deprecation message coming from Gem.paths, this should be cleaned up;
		// there's a round trip of splitting into array in java and then unsplitting the array in ruby...
		// see #118
		sassScript.append("\n].uniq.join(File::PATH_SEPARATOR) }\n");
		sassScript.append("Gem.paths = env\n");
	}`

What this is going to do is add the GEM_PATH environment variable to the ruby script if it exists.
This environment variable exists on my jenkins build server, but not in my development environment.
For now - to solve this issue I'll see if I can run the the jenkins job unsetting the environment variable. But I think there's a question here about whether that it's required to get that environment variable, or the option to skip it should be allowed.

To solve via Jenkins:
In your Jenkins build configuration:

Check prepare an environment for the run
In properties content add

GEM_PATH=

You appear to have a jenkins plugin with stapler and/or ruby-runtime, this is messing things up for you.
This sets up an incompatible environment.

It might be incompatible but is it possible for your code to be slightly more defensive to prevent this?  Even installing the Travis-CI plugin will cause this because it also has a dependency on that out-dated ruby-runtime plugin.   You and I both know the real culprit is the ruby-runtime plugin, but is there a way for this to at least side-step the issue until the ruby plugin gets its act together?

picking up the gempath is a feature of the plugin, it is useful on the desktop. You can use the workaround shown above.


Implementation of .NET packages.config extractor and fixes

Pull request for the .NET packages.config extractor.
Implemented:
Extractor implementation for package.config;
Unit test implementation for extractor:
Integration of the extractor in the main tool;
Fixes from the .NET PR #355.

re binary/proto/proto.go (idk how to respond to my own comment above)
So you either add a metadata proto for .net. Then you need to update proto.go and scanresult proto. This is used in some programs to differentiate different result types. It's also recommended to have all necessary information in metadata for matching.
Or you don't, then that change is not required.


[SPARK-11242][SQL] In conf/spark-env.sh.template SPARK_DRIVER_MEMORY is documented incorrectly

Minor fix on the comment

Can one of the admins verify this patch?

Right, let me change that too. Thx Sean!

Test build #1939 has started for PR 9201 at commit 5a11872.

Test build #1939 has finished for PR 9201 at commit 5a11872.

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


.list-group-flush within .card causes double border

Saw this locally when I upgraded to alpha 3, confirmed in the public documentation:

Seeing this in Chrome 51, Firefox 46, and Safari 9.1.2.

+1 For me as well when migrating.
Also occurring in the plain list-in-a-card example.

Potentially this commit removed the different border when list groups are flush.
https://github.com/twbs/bootstrap/commit/955e0405ea04e8bce3fa8395cd5e1c4030c4947a#diff-cd8ec88dc7823a5c78f7111e37a78adbL37

I thought that mdo fixed these with #19861 and #19885, but I guess he never fixed/reverted all of the changes he made.

Also occurring for me:

Happy Friday @mdo / @cvrebert! Since this seems to be a common problem and since the fix seems pretty benign, can this be added to the alpha.3.1 patch release?

Hi @mdo, just wanted to follow up on this since its still present in the alpha.4 (http://v4-alpha.getbootstrap.com/components/card/#content-types)
Any chance this fix can catch the alpha.5 milestone?

Temporary fix / monkey patch:
// Documentation example: Remove double borders
.card > .list-group > .list-group-item {
  border-left: 0;
  border-right: 0;
}

// Plain list card example: Remove also first + last border
.card > .list-group > .list-group-item {
  border-left: 0;
  border-right: 0;
  &:first-of-type {
    border-top: 0;
  }
  &:last-of-type {
    border-bottom: 0;
  }
}

The above fix works when the .list-group is the only element within the .card. The following version allows for the .list-group to be combined with other elements within .card
.card > .list-group {
  > .list-group-item {
    border-left: 0;
    border-right: 0;
  }
  &:first-child .list-group-item:first-of-type {
    border-top: 0;
  }

  &:last-child .list-group-item:last-of-type {
    border-bottom: 0;
  }
}

Also, when rounded borders are enabled you also need to add
border-top-left-radius: 0;
border-top-right-radius: 0;


No error if JSON file other than dim.json is specified.

It is necessary to check for fileVersion / lockFileVersion in constructor.
Similar to the following issue.
https://github.com/c-3lab/dim/issues/54

Proposals:
First draft
Check fileVersion / lockFileVersion in the constructor.
Second draft
Check in constructor that JSON entry in addition to fileVersion / lockFileVersion is identical to dim.json.


Missing 'revisions' annotation in some classes

In the documentation of a couple of models and packages, there is a revisions annotation containing some kind of 'copyright':

 Developed 2010-2014 at the DLR Institute of System Dynamics and Control

On the other side, there exist models which do not contain such annotation anyway.
There should be a decission if that annotation is still reasonable.

I think it is a good thing when the authorship of components is marked. This also includes components contributed by other parties.

Well then this part of documentation should be upgraded to '2015', I think.
And what about classes (mostly packages) which don't contain this annotation?

There are currently lots of classes concerned, so their revisions shall be updated each year. This is error-prone and time consuming. Therefore, the last year will be removed from revisions and just year of development retained, e.g.

Developed 2010-2020 at the DLR Institute of System Dynamics and Control

will be chaned to

Developed 2010 at the DLR Institute of System Dynamics and Control


Game value to query tax paid / payment received

The query would be similar to query custom stat in that you can specify the time period to query in, or leave it at 0 to query lifetime. This would be helpful for accounting and refunding purposes (e.g. issue an exec action to refund everyone for all taxes of "Sales Tax" code in the last 24hrs, but not other tax codes).
Possibly also another set of game values for the government side, e.g. query how much revenue a particular tax has generated for a given gov account.
Maybe we want to add some view of this data to the tax card or tax log too so the player can view their aggregate stats at a glance?

Added in #21