SM4 AESE optimization for ARMv8

This patch optimizes SM4 for ARM processor using ASIMD and AES HW instruction, which is an optional feature of
crypto extension for ARMv8-A.
This optimization is based on existing ASIMD implementation (i.e., vpsm4-armv8.pl). Similarly, it also improve performance if both of following conditions are met:

Input data equal to or more than 4 blocks;
Cipher mode allows parallelism, including ECB, CTR, GCM or CBC decryption

For more details about the theory, see sm4ni, whose trick is to "use affine transforms to emulate the SM4 S-Box with the AES S-Box". The constants used in subroutine load_sbox() are from this blog. We've done some further optimizations so the constants don't look the same.
The performance data on Kunpeng-920 2.6GHz hardware is as follows. The vpsm4_ex-armv8.pl is improved by about 16% compared with vpsm4-armv8.pl and 140%~160% compared with sm4.c. As for single block input data, performance could drop about 50%.
sm4.c
type             16 bytes     64 bytes    256 bytes   1024 bytes   8192 bytes  16384 bytes
SM4-ECB          80407.14k    85300.06k    85868.24k    86763.97k    86715.05k    86775.13k
SM4-CBC          77752.78k    80713.71k    81698.56k    82142.27k    82004.65k    82056.53k
SM4-CBC          76679.58k    82742.53k    84521.56k    85360.16k    85281.45k    85536.54k (decrypt)
SM4-CTR          75415.10k    78762.37k    79654.66k    80073.03k    79943.00k    80210.37k
SM4-XTS          39961.43k    64757.93k    76031.74k    79360.34k    80618.60k    80303.45k
SM4-CFB          77561.46k    80984.51k    81551.03k    81826.47k    82374.81k    82029.59k
SM4-OFB          77704.14k    81598.04k    82886.89k    82812.59k    83613.38k    82935.81k
vpsm4-armv8.pl
type             16 bytes     64 bytes    256 bytes   1024 bytes   8192 bytes  16384 bytes
SM4-ECB          35425.95k   122996.12k   175135.34k   177286.49k   178211.50k   178266.72k
SM4-CBC          35538.75k    36535.89k    36425.55k    36748.93k    36541.78k    36484.64k
SM4-CBC          35657.63k   122078.85k   174190.76k   176717.06k   176949.93k   177308.85k (decrypt)
SM4-CTR          35683.95k   120279.59k   176395.86k   177032.08k   177991.06k   177629.87k
SM4-XTS          17801.13k    28802.15k    33827.48k    35345.12k    35919.19k    35935.57k
SM4-CFB          35478.46k    35681.64k    36060.73k    35748.18k    35806.44k    35655.75k
SM4-OFB          35562.41k    35804.29k    35923.59k    36177.24k    36324.26k    36164.95k
vpsm4_ex-armv8.pl
type             16 bytes     64 bytes    256 bytes   1024 bytes   8192 bytes  16384 bytes
SM4-ECB          32092.50k   116268.76k   205334.59k   207700.42k   207833.77k   208569.96k
SM4-CBC          32175.81k    32750.21k    32736.51k    32762.88k    32863.89k    32619.55k
SM4-CBC          31912.62k   115407.54k   202925.74k   205583.36k   207293.22k   206684.16k (decrypt)
SM4-CTR          32049.50k   114071.89k   205088.28k   206120.62k   206561.28k   207457.61k
SM4-XTS          15978.03k    60165.70k   147150.85k   188738.22k   205923.32k   206597.31k
SM4-CFB          31816.47k    32057.11k    32239.30k    32148.48k    32288.54k    32178.18k
SM4-OFB          31876.35k    32269.23k    32510.29k    32431.10k    32456.70k    32565.25k
Signed-off-by: Xu Yizhou<EMAIL_ADDRESS>

(also on our radar - adding @zorrorffm)

the same issue, could optimization be disabled in SM4-CBC encrypt、SM4-CFB and SM4-OFB mode since the  performance is dropped.

the same issue, could optimization be disabled in SM4-CBC encrypt、SM4-CFB and SM4-OFB mode since the performance is dropped.

C implementation is faster on these modes, but it's not secure, the look-up table is side-channel attack prone because it's not constant-time code.   So this optimization is a perf and security tradeoff, and it's only for 3 architectures (N1, V1 and Kunpeng-920) that are used in infrastructure world where security is more considered.

I test this code in kirin990, the perfromace is slower than  vpsm4-armv8.pl.

I test this code in kirin990, the perfromace is slower than vpsm4-armv8.pl.

Sorry, I'm a little bit confused. If you just applied this patch and didn't modify any code, the code in vpsm4-armv8.pl would not get chance to be run when you did the test on kirin990 , how did you benchmark vspsm4-armv8.pl on kirin990?  Could you describe your question in details? ( How did you do the test? What's your expectation?  Which mode's performance is slower than vpsm4-armv8.pl?) ...

I test this code in kirin990, the perfromace is slower than vpsm4-armv8.pl.

Sorry, I'm a little bit confused. If you just applied this patch and didn't modify any code, the code in vpsm4-armv8.pl would not get chance to be run when you did the test on kirin990 , how did you benchmark vspsm4-armv8.pl on kirin990? Could you describe your question in details? ( How did you do the test? What's your expectation? Which mode's performance is slower than vpsm4-armv8.pl?) ...

https://github.com/openssl/openssl/pull/19547#issuecomment-1316417677,I changed code to run in kirin990.
the benchmark data:
when the input data is 16384bytes
SM4.c vpsm4-armv8.pl vpsm4_ex-armv8.pl
SM4-GCM     109.6M          246.9M     228.4M
SM4-CBC decrypt 110.5M  257M        238.1M
in other words, the vpsm4_ex-armv8.pl may slower than vpsm4-armv8.pl in some platforms.
I don't have any N1/V1 device. so I don't have benchmark data in these two platforms.
My question is, whether the vpsm4_ex-armv8.pl is faster or slower than vpsm4-armv8.pl in N1/V1 platform.

https://github.com/openssl/openssl/pull/19547 changed code to run in kirin990.

I wonder why it works on kirin990, since I think it will work only after adding MIDR_IS_CPU_MODEL(OPENSSL_arm_midr, HISI_CPU_IMP, ARM_CPU_PART_V1)) in vpsm4_capable() here,
https://github.com/openssl/openssl/blob/f5b06306b7727f830407c639f8a7bf1698f8aaa5/include/crypto/sm4_platform.h#L19-L25

#19547 changed code to run in kirin990.

I wonder why it works on kirin990, since I think it will work only after adding MIDR_IS_CPU_MODEL(OPENSSL_arm_midr, HISI_CPU_IMP, ARM_CPU_PART_V1)) in vpsm4_capable() here,
https://github.com/openssl/openssl/blob/f5b06306b7727f830407c639f8a7bf1698f8aaa5/include/crypto/sm4_platform.h#L19-L25

yes，you are right, I add this code to run in kirin990. the final code is like
return (OPENSSL_armcap_P & ARMV8_CPUID) &&
(MIDR_IS_CPU_MODEL(OPENSSL_arm_midr, ARM_CPU_IMP_ARM,
ARM_CPU_PART_V1) ||
MIDR_IS_CPU_MODEL(OPENSSL_arm_midr, ARM_CPU_IMP_ARM,
ARM_CPU_PART_N1) ||
MIDR_IS_CPU_MODEL(OPENSSL_arm_midr, HISI_CPU_IMP,
HISI_CPU_PART_KP920) ||
MIDR_IS_CPU_MODEL(OPENSSL_arm_midr, HISI_CPU_IMP, ARM_CPU_PART_V1));

@tom-cosgrove-arm @zorrorffm, what about the performance of vpsm4_ex-armv8.pl in N1/V1 platform? Is it slower than vpsm4-armv8.pl?

@tom-cosgrove-arm @zorrorffm, what about the performance of vpsm4_ex-armv8.pl in N1/V1 platform? Is it slower than vpsm4-armv8.pl?

@xu-yi-zhou , I‘ll run the test and let you know the result.

I do observe the performance regression for SM4-ECB, CBC, CTR, CFB and OFB on N1 and V1 when applying this patch.    Only For SM4-XTS, the performance is uplift.

@zorrorffm, thank you very much for your test.
If possible, could please test the SM4-XTS in vpsm4-armv8.pl? I add vpsm4_xts_encrypt_xx in vpsm4-armv8.txt, just change the .txt to '.pl', and replace vpsm4-armv8.pl.
If vpsm4_xts_encrypt_xx is faster thanvpsm4_ex_xts_encrypt_xx, I will add it in vpsm4-armv8.pl, and define the macro like this:
extern unsigned int OPENSSL_arm_midr;
static inline int vpsm4_capable(void)
{
    return (OPENSSL_armcap_P & ARMV8_CPUID) &&
            (MIDR_IS_CPU_MODEL(OPENSSL_arm_midr, ARM_CPU_IMP_ARM, ARM_CPU_PART_V1) ||
             MIDR_IS_CPU_MODEL(OPENSSL_arm_midr, ARM_CPU_IMP_ARM, ARM_CPU_PART_N1));
}
static inline int vpsm4_ex_capable(void)
{
    return (OPENSSL_armcap_P & ARMV8_CPUID) &&
            (MIDR_IS_CPU_MODEL(OPENSSL_arm_midr, HISI_CPU_IMP, HISI_CPU_PART_KP920));
}
#    if defined(VPSM4_ASM)
#     define VPSM4_CAPABLE vpsm4_capable()
#     define VPSM4_EX_CAPABLE vpsm4_ex_capable()
#    endif

If not, let's talk about then.

@xu-yi-zhou , I did the perf test based on your vpsm4-armv8.txt, sm4-xts performance is better than  vpsm4_ex-armv8.pl.  The changes I did for the test is following:
1, change vpsm4-armv8.txt to vpsm4-armv8.pl and use it to replace the original one.
2, add the declarations of vpsm4_xts_encrypt and vpsm4_xts_encrypt_gb in sm4_platform.h
3, set sm4-xts encryption functions to be vpsm4_xts_encrypt and vpsm4_xts_encrypt_gb in cipher_sm4_xts_hw.c

looking forward to next step

Hi all，
Since the aboved perf test, I added a new function vpsm4_ex_capable() to ensure that only Kunpeng-920 will use the SM4 AESE optimization in vpsm4_ex-armv8.pl, other platform will still use the original SM4 optimization in vpsm4-armv8.pl.
Also, I added SM4 optimization for XTS mode in vpsm4-armv8.pl.
Please review it again.

Looks good to me.

Ping for the second review!

after this performance improvement，could you port  performance improvement for SM3 like https://git.gnupg.org/cgi-bin/gitweb.cgi?p=libgcrypt.git;a=blob;f=cipher/sm3-aarch64.S;h=77dba2ba8265bb276629c9486055da14fd2d9b43;hb=f664333a4749eab03c6b19bae2f28863b8501a24
it could improve about 5% on kunpeng 920 and 40% on kirin 990 and about 20% on other armv8 platform like ft-d2000.

The Libgcrypt code as it is can't be incorporated into OpenSSL as it is licensed under LGPL

The Libgcrypt code as it is can't be incorporated into OpenSSL as it is licensed under LGPL

maybe someone could rewrite optimze code using ARM NEON.

Only whitespace changes, so still good from our side

Ping for the second review

This pull request is ready to merge

Merged, thanks for the improvement.

This seems to have caused a build failure on our Mac Mini M1:
https://ci.buildbot.openssl.org/#/builders/54/builds/621

I will take a look and fix it

@levitte what about this [vpsm4-armv8.txt]? I use a new Lxts_magic label(https://github.com/openssl/openssl/files/10567388/vpsm4-armv8.txt)
.Lxts_magic:
	.dword 0x0101010101010187,0x0101010101010101

ldr  @qtmp[0], .Lxts_magic

That looks like a plausible fix.  PR welcome! :wink:

There seems to be similarly large constants in crypto/sm4/asm/vpsm4_ex-armv8.pl as well, don't forget those


Update to MoltenVK 1.0.38

Release notes

Seems like we need to upgrade travis or switch to actions.

This looks like issue #4 of potentially that the new MoltenVK doesn't support older Xcode 10.3 (which we use and still need to use). Did retry building MoltenVK locally now again with Xcode 10.3 and get the same issue, will file a issue with them and ask

Turns out MoltenVK indeed requires Xcode 11 now, I'll upgrade the CI in the PR here

Hmm no compile errors now, but some fairly cryptic link errors instead.
 = note: Undefined symbols for architecture x86_64:

            "_OBJC_CLASS_$_CAEDRMetadata", referenced from:

                objc-class-ref in libMoltenVK.a(libMoltenVK.a-x86_64-master.o)

This is defined in core animation https://developer.apple.com/documentation/quartzcore/caedrmetadata?language=objc we just need to link with it.

Wait quartzcore includes core animation which we already link with.

Hmm I got a different build error when building this branch with Xcode 11 locally:
xcodebuild -quiet -project "MoltenVKPackaging.xcodeproj" -scheme "MoltenVK Package (macOS only)" build
still building MoltenVK
clang: error: no such file or directory: '11.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk'
clang: error: no such file or directory: '11.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk/System/Library/Frameworks'
clang: error: no such file or directory: '11.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk/usr/lib'
clang: warning: no such sysroot directory: '/Applications/Xcode' [-Wmissing-sysroot]
note: Using new build systemnote: Planning buildnote: Constructing build description

This builds locally for me with Xcode 11.1. Did notice that Travis was building on an older Mac OS 10.14.4 but with bleeding edge Xcode 11.2 beta so that could be the cause of the problem and linking error potentially.
Will switch it to Xcode 11.1

Nope some error still with Xcode 11.1 on Travis

Can't fail on travis if you remove travis!

Oh noes failed on GitHub Actions also, well at least it is failing faster now. in 15 min instead of 37. so that is some type of progress.
GitHub Actions is also building with Mac OS 10.14 (Mojave) and not 10.15 (Catalina), no option to change or configure.

GitHub plans to roll out Mac 10.15 (Catalina) within 2 weeks: https://github.com/actions/virtual-environments/issues/53
So if that is the error then this could be resolved then soon. And right now we don't have an urgent need for this and it is critical that CI works so can just wait.

Yay now compiles with Mac OS 10.15 (Catalina) that GitHub Actions just switched to!


no pip

pip, pip3, python -m pip, python3 -m -pip does not work in appstore app.

I have no problems with pip, python -m pip install *module* works for me.


DOC-2890 CI Run step, background step - Shell, Command fields

Add examples for each Shell type.
Make Shell & Command more cohesive.
Run step settings preview:
https://64372278be7d633d2d2b6ae5--harness-developer.netlify.app/docs/continuous-integration/ci-technical-reference/run-step-settings#shell-and-command
Background step settings page preview:
https://64372278be7d633d2d2b6ae5--harness-developer.netlify.app/docs/continuous-integration/ci-technical-reference/background-step-settings#shell-entry-point-and-command

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Change common-utils ref from 2.x to 2.3

Description
Change common-utils ref from 2.x to 2.3
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2573 (b1b28d4) into main (625bae5) will increase coverage by 5.58%.
The diff coverage is n/a.

:exclamation: Current head b1b28d4 differs from pull request most recent head 1ddb1d4. Consider uploading reports for the commit 1ddb1d4 to get more accurate results

@@              Coverage Diff              @@
##               main     #2573      +/-   ##
=============================================
+ Coverage     94.41%   100.00%   +5.58%
=============================================
  Files           219         6     -213
  Lines          4462       105    -4357
  Branches         29        19      -10
=============================================
- Hits           4213       105    -4108
+ Misses          243         0     -243
+ Partials          6         0       -6

Impacted Files
Coverage Δ

tests/jenkins/jobs/BuildFailureMessage_Jenkinsfile

...ts_workflow/component_opensearch_dashboards_min.py

tests/jenkins/jobs/PromoteYumRepos_Jenkinsfile

src/assemble_workflow/bundle_url_location.py

src/test_workflow/bwc_test/bwc_test_runners.py

tests/jenkins/jobs/SignArtifacts_Jenkinsfile

src/sign_workflow/signer.py

...c/test_workflow/dependency_installer_opensearch.py

src/assemble_workflow/dists.py

tests/jenkins/jobs/CreateGithubIssue_Jenkinsfile

... and 203 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Hi @getsaurabh02 I dont see 2.3 branch in common-utils repo due to which CI is failing. Can you cut the branch first?
Thanks!

Hi @gaiksaya I just created 2.3 from 2.x. Can we retry the workflow. Thanks.

Tagging 2.3 release issue from common utils


Stop kismatic-inspector even on failure

Does the ansible version that we use run the "rescue" block in case of failure? The ansible team mentioned that they had a bug that was fixed in 2.2.0

I tested with the version we have and it worked

Cool. Sounds like we can merge this, and close #142 and #105.

Fixes #142 and Fixes #105


[MINOR][TEST]Off -head memory leaks for unit tests

What changes were proposed in this pull request?
Free off -head memory .
I have checked all the unit tests.
How was this patch tested?
N/A

Test build #81206 has started for PR 19075 at commit d40bdd2.

Can you fix "off-head" to "off-heap"?
This looks reasonable. Are there other tests where memory isn't freed though?

@srowen  Thanks, I have checked all the unit tests,there are no other tests.

Test build #81206 has finished for PR 19075 at commit d40bdd2.

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

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81206/
Test PASSed.

Merged build finished. Test PASSed.

Merged to master


feat: add looking for contributings

@thuongtruong1009 We don't support this but I will share your work with others..


Yahoo Weather Api to be retired Feb 1, 2021

Yahoo Weather API to be retired Feb 1, 2021
https://developer.yahoo.com/weather/

they extended the expiration time of free usage to 1. March
As of Monday, March 1, 2021, the free Yahoo Weather API will be retired.


[dashboard][native-filter]Filter indicator overlay in Dashboard on scroll

Screenshot

there is an open issue for this one you may have to dig it out. and this is not easy to fix AFAIK.


[BUG] check script reports false negative on SLES15-SP2

Describe the bug
Check Script provides a false positive on SLES15-SP2 + k3s
Longhorn does install properly
To Reproduce
Steps to reproduce the behavior:
run the Check Script for Longhorn
Expected behavior
Check Script should be satisfied
Log
node : MountPropagation DISABLED
MountPropagation is disabled on at least one node.
As a result, CSI driver and Base image cannot be supported.
cleaning up...
cat /proc/self/mountinfo | grep "/ / rw"
99 1 259:2 / / rw,relatime shared:1 - xfs /dev/nvme0n1p2 rw,attr2,inode64,logbufs=8,logbsize=32k,noquota
Environment:

Longhorn version: 1.0.1
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: k3s
Node config

OS type and version: SLES15-SP2 on XFS + k3s
Disk type(e.g. SSD/NVMe): NVMe

Additional context
Add any other context about the problem here.

@c3y1huang please help with this. Thanks.

@c3y1huang please help with this. Thanks.

Hi @abonillabeeche I've got no luck trying to reproduce this.
ip-172-30-0-189:~ # cat /etc/os-release
NAME="SLES"
VERSION="15-SP2"
VERSION_ID="15.2"
PRETTY_NAME="SUSE Linux Enterprise Server 15 SP2"
ID="sles"
ID_LIKE="suse"
ANSI_COLOR="0;32"
CPE_NAME="cpe:/o:suse:sles:15:sp2"

ip-172-30-0-189:~ # kubectl get nodes
NAME              STATUS   ROLES                  AGE     VERSION
ip-172-30-0-189   Ready    control-plane,master   6m38s   v1.20.0+k3s2
ip-172-30-0-189:~ # cat /proc/self/mountinfo | grep "/ / rw"
97 1 259:3 / / rw,relatime shared:1 - xfs /dev/nvme0n1p3 rw,attr2,inode64,logbufs=8,logbsize=32k,noquota

ip-172-30-0-189:~ # curl -sSfL https://raw.githubusercontent.com/longhorn/longhorn/master/scripts/environment_check.sh | bash
daemonset.apps/longhorn-environment-check created
waiting for pods to become ready (0/1)
all pods ready (1/1)

  MountPropagation is enabled!

cleaning up...
daemonset.apps "longhorn-environment-check" deleted
clean up complete

ip-172-30-0-189:~ # kubectl -n longhorn-system get pod
NAME                                       READY   STATUS    RESTARTS   AGE
longhorn-ui-849c455d79-n9jd5               1/1     Running   0          3m42s
instance-manager-r-df5b3514                1/1     Running   0          3m16s
instance-manager-e-4d342415                1/1     Running   0          3m17s
engine-image-ei-cf743f9c-2n6dg             1/1     Running   0          3m17s
longhorn-manager-qgnzg                     1/1     Running   0          3m42s
longhorn-driver-deployer-cdb7464c6-svqpz   1/1     Running   0          3m42s
csi-provisioner-5c9dfb6446-4xvmj           1/1     Running   0          2m35s
csi-snapshotter-96bfff7c9-v8g9f            1/1     Running   0          2m34s
csi-resizer-54d484bf8-p2gc4                1/1     Running   0          2m34s
csi-attacher-5dcdcd5984-2vwpk              1/1     Running   0          2m35s
csi-attacher-5dcdcd5984-sgprb              1/1     Running   0          2m35s
csi-provisioner-5c9dfb6446-s8bmd           1/1     Running   0          2m35s
csi-snapshotter-96bfff7c9-wpmgl            1/1     Running   0          2m34s
csi-provisioner-5c9dfb6446-lp7q8           1/1     Running   0          2m35s
csi-snapshotter-96bfff7c9-kp55b            1/1     Running   0          2m34s
csi-resizer-54d484bf8-r9clb                1/1     Running   0          2m34s
csi-resizer-54d484bf8-7279f                1/1     Running   0          2m34s
longhorn-csi-plugin-7zbq4                  2/2     Running   0          2m34s
csi-attacher-5dcdcd5984-vt2gn              1/1     Running   0          2m35s

If you could run kubectl -l app=longhorn-environment-check describe pod while the check script is failing might give us better hint on the issue.

Hi @abonillabeeche I've got no luck trying to reproduce this.
ip-172-30-0-189:~ # cat /etc/os-release
NAME="SLES"
VERSION="15-SP2"
VERSION_ID="15.2"
PRETTY_NAME="SUSE Linux Enterprise Server 15 SP2"
ID="sles"
ID_LIKE="suse"
ANSI_COLOR="0;32"
CPE_NAME="cpe:/o:suse:sles:15:sp2"

ip-172-30-0-189:~ # kubectl get nodes
NAME              STATUS   ROLES                  AGE     VERSION
ip-172-30-0-189   Ready    control-plane,master   6m38s   v1.20.0+k3s2
ip-172-30-0-189:~ # cat /proc/self/mountinfo | grep "/ / rw"
97 1 259:3 / / rw,relatime shared:1 - xfs /dev/nvme0n1p3 rw,attr2,inode64,logbufs=8,logbsize=32k,noquota

ip-172-30-0-189:~ # curl -sSfL https://raw.githubusercontent.com/longhorn/longhorn/master/scripts/environment_check.sh | bash
daemonset.apps/longhorn-environment-check created
waiting for pods to become ready (0/1)
all pods ready (1/1)

  MountPropagation is enabled!

cleaning up...
daemonset.apps "longhorn-environment-check" deleted
clean up complete

ip-172-30-0-189:~ # kubectl -n longhorn-system get pod
NAME                                       READY   STATUS    RESTARTS   AGE
longhorn-ui-849c455d79-n9jd5               1/1     Running   0          3m42s
instance-manager-r-df5b3514                1/1     Running   0          3m16s
instance-manager-e-4d342415                1/1     Running   0          3m17s
engine-image-ei-cf743f9c-2n6dg             1/1     Running   0          3m17s
longhorn-manager-qgnzg                     1/1     Running   0          3m42s
longhorn-driver-deployer-cdb7464c6-svqpz   1/1     Running   0          3m42s
csi-provisioner-5c9dfb6446-4xvmj           1/1     Running   0          2m35s
csi-snapshotter-96bfff7c9-v8g9f            1/1     Running   0          2m34s
csi-resizer-54d484bf8-p2gc4                1/1     Running   0          2m34s
csi-attacher-5dcdcd5984-2vwpk              1/1     Running   0          2m35s
csi-attacher-5dcdcd5984-sgprb              1/1     Running   0          2m35s
csi-provisioner-5c9dfb6446-s8bmd           1/1     Running   0          2m35s
csi-snapshotter-96bfff7c9-wpmgl            1/1     Running   0          2m34s
csi-provisioner-5c9dfb6446-lp7q8           1/1     Running   0          2m35s
csi-snapshotter-96bfff7c9-kp55b            1/1     Running   0          2m34s
csi-resizer-54d484bf8-r9clb                1/1     Running   0          2m34s
csi-resizer-54d484bf8-7279f                1/1     Running   0          2m34s
longhorn-csi-plugin-7zbq4                  2/2     Running   0          2m34s
csi-attacher-5dcdcd5984-vt2gn              1/1     Running   0          2m35s

If you could run kubectl -l app=longhorn-environment-check describe pod while the check script is failing might give us better hint on the issue.

Lower the priority due to unable to reproduce.

Lower the priority due to unable to reproduce.


Proposal - Data Virtualization using datavirt

Currently the VDC scenario worked on in Egeria makes use of IBM GaianDB - https://github.com/gaiandb/gaiandb
This technology is

lightweight
peer to peer/distributed (load, IoT)
supports virtualisation (mapping, renaming, federation/joins, views)
suppports plugin auth (ie with our ranger plugin)

However this project

does not have an active vibrant community
is based on an old (2011-2015) version of Apache Derby
: no current security fixes
: no ongoing sql updates
: Derby community is also quiet now

As such we need to consider
a) Whether we might adopt Gaian & bring it up to appropriate levels, provide ongoing development
b) Look at other technologies
Here I propose as part of b) that we look at RedHat/JBoss 'datavirt'
See https://developers.redhat.com/products/datavirt/overview for product information
Datavirt is a combination of a few products that ships within the Redhat Stack.
The core virtualization component is 'teiid' https://github.com/teiid/teiid and is able to be embedded into other applications. The broader datavirt product includes more support for configuration, containers etc
Opening to track further investigations
See also #1843 for a postgresql proposal

@danielaotelea are you currently looking at data virtualization?


config: qualify the name of the version field

https://github.com/opencontainers/specs/issues/110
Signed-off-by: Vincent Batts<EMAIL_ADDRESS>

Closes #110

LGTM

LGTM

updated to ociVersion. PTAL

+1

re-LGTM

ociVersion is better, but we may end up with multiple OCI specs.
We're calling this spec “Open Container Specifications” 1, so why
not “ocsVersion”?

Not to take us down too much of a tangent, but do we have one spec or multiple in this project? If we have multiple then "ocs" doesn't help since having "s=spec" when there are multiple is kind of ambiguous. Perhaps "Open Container Runtime (ocr)" for the core spec we've been talking about.   But if we view this all as one big spec (which might be nice to avoid each one having a different version number), then just "oci" would work.

On Fri, Jan 15, 2016 at 10:35:34AM -0800, Doug Davis wrote:

Not to take us down too much of a tangent, but do we have one spec
or multiple in this project?

I would love to have this question answered, since I'm frequently
suggesting we handle orthogonal issues in separate OCI specs
(e.g. 1).  But to stay on topic here, maybe we should move this
discussion to the list?
 Subject: Re: appc + oci harmonization progress
 Date: Thu, 13 Aug 2015 11:29:08 -0700
 Message-ID<EMAIL_ADDRESS>

We should leave the discussion about the needs of multiple OCI spec to the list.
LGTM


Prepare for v1.18 development cycle

Compared with da63c10, this just seems to be missing the backport labels updater. I see that this was updated for v1.17 in c56ece3, which seems intended - that is to say the workflow runs in context of the stable branch, so the stable branch needs the v1.17 branch to be mentioned. Since that change is already in the tree, it will naturally flow into v1.17 without further action.
That makes it correct for now but we may want to consider adding v1.18 into the tag (but maybe we delay that until we prepare the stable branch for v1.18, ie, in six months' time).

@joestringer I got a little bit confused. :sweat_smile: the workflow is different now and doesn't require to be updated. It was simplified in https://github.com/cilium/cilium/commit/e91378b34f9451c2f9016333efc65b3af8c57cf4


Add Codeowners

Signed-off-by: Matt Delacour<EMAIL_ADDRESS>Which issue(s) this PR fixes:

Fixes # https://github.com/Shopify/feast-trino/issues/7
What this PR does / why we need it:
Add Codeowners so that we can ask for at least 1 approval before merging
Does this PR introduce a user-facing change?:

NONE

@feast-dev/maintainers what do you think about the following change ?


Fixed LinkedIn authorization and token endpoints

Changed LinkedInAuthenticationHandler to use the endpoints described at the documentation.

Sorry for leaving this for so long...life and what not @_@

Thanks! That's alright, I could workaround it overwriting the authorization endpoint using the provider's OnApplyRedirect.


Add codegen for Functions which are not part of impl

As far as I can see, there was no way to print a function at the top level, so I added that, and one test.

I don't have time to maintain this crate anymore. I would be happy to hand it off to whoever. Sorry :(


feat: FlownodeClient

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
added FlownodeClient which can send requests to Flownode in distributed mode
Please explain IN DETAIL what the changes are in this PR and why they are needed:

rename DatanodeClients to NodeClients
impl NodeManager::flownode() for NodeClients which return corrsponding FlownodeClient
added FlownodeClient and impl Flownode for it

Checklist

[ ] I have written the necessary rustdoc comments.
[ ] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

Summary by CodeRabbit

New Features

Introduced FlowRequester for handling flow requests and inserts.

Improvements

Renamed DatanodeClients to NodeClients for consistency across the codebase.
Enhanced error handling with the new FlowServer variant in error reporting.

Refactor

Updated method implementations and references to reflect the renaming from DatanodeClients to NodeClients.

Do we really want to rename DatanodeClient to NodeClient? Datanode is a term in our database while Node sounds like more general purpose


Clarified Package management description

Tried to better explain the purpose and advantage of package management.
Description

Urgency

[ ] Blocker
[ ] High
[ ] Medium

Requirements

Test all commands and procedures.
Build content locally and test for formatting/links.
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, and 1.9).
See the contribution guidelines.

Can one of the admins verify this patch?

thank you, @ethanport !


Disable PerfDisableSharedMem in jvm.options instead of cassandra-env

Tests for CASSANDRA-10494

Won't this break on every version without the jvm options file?

Looking at cassci, it did. I will fix it.

In fact, it looks like this is breaking on trunk as well? Oh I expect that is because 10494 wasn't merged yet. I'm going to push my version gating fix, and trust in the original code that it will still work when 10494 is merged.


Implement control over migration transaction

Is your feature request related to a problem? Please describe.
I have a multiple tenant database structure where I would like to sync up migrations across tenant databases. For example, if I have two databases, A and B, this requires that if database A's migrations succeed, but database B's migrations fail, both database A and B's transactions rollback.
Describe the solution you'd like
In my scenario, I would have a MikroORM instance for each tenant database where I'd like to control commiting and rollback of the transactions.
The migration methods accept a transaction as a option:
migrator.up({ transaction: trx });

A way to pull a transaction from the orm. Currently I do a hacky workaround by doing:
const connection = orm.em.getDriver().getConnection();
// @ts-ignore
const trx = await connection.client.transaction();

This way, I can wait for all the migrations in the different databases to finish and commit the transactions only if all migration instances succeed.

In v4 there is em.getConnection().begin() that will give you the tx context, so the latter should be fine already.

await orm.em.transactional(async em => {
  await migrator.up({ transaction: em.getTransactionContext() });
});


Panels in minimized rows not detected

I've been using detect-angular-dashboards against an on-premise installation of Grafana Enterprise on 10.4.
Going through the list of detections, I noticed that panels in a row that is minimized / not expanded are not detected. I confirmed this on a few different dashboards, and I noticed that when I migrated visible panels, which were detected, but left Angular panels in a minimized row, the dashboard was fully removed.
Panels hidden like this are one of by biggest concerns, so this seems like a major gap in the tool.

I noticed https://github.com/grafana/detect-angular-dashboards/pull/18 after opening this issue


Switching between release types

Would it be possible to switch between releases? Maybe even link it to the Hass.io stable/beta channel?
At the moment there is no option (that I know of) to clarify whether a user wants to receive the stable or development builds.
Currently there is also the issue of #163. If the appropriate releases could be linked to the stable and beta release version files, users that installed the development build (2.1) won't see an option to 'update' to 1.9.

Just noticed that the 'join beta channel' option does indeed state users will receive beta updates for the host system as well, however, this is currently not the case.

Looks like the beta channel only includes RC-releases for HassOS.
The other releases I meant are on dev, however there is no option to switch to that.
https://s3.amazonaws.com/hassio-version/dev.json


colormap for labels, tests ok fo python3.12

tests for python 3.10,3.11 and 3.12 seemed good for my ubuntu 24.04 laptop.

Thank you for the PR, @paapu88.
I made some minor changes to the CI and the Python versions.
Once it passes I'll merge it.

It seems there's an open issue the segment-anything package, https://github.com/facebookresearch/segment-anything/issues/477
We could set numpy<2 what do you think? @paapu88

@JoOkuma  I personally don't like to keep numpy<2
But it may be the only option. I'm not so familiar with facebookresearch/segment-anything
but often this kind of big-company-packages are not maintained unless somebody else makes a fork and starts taking care...
Sorry for not being too helpful.


Not getting the latest version when installing through composer.Installing version(0.1.3)and current version is 0.1.4.

Please convey these sought of things on the other channels as this is not the issue to be filed here.
And yes very shortly this will get fixed when the new release comes.


Parameters incorrect for getTimeChartIntent

R&D feature + probably only affecting TDH = don't care about replicating this on the old ui

http://manage.dimagi.com/default.asp?181403


bug or docs: useHeadSafe and useServerHeadSafe strips href on link

Environment
Using the bundled version that comes with Nuxt 3.5.1
Reproduction
Add the following to app.vue inside the script setup (<script lang="ts" setup>):
useServerHeadSafe({
  link: [
    {
      href: "/favicon.svg", // this file exists inside the `/public` directory
      rel: "icon",
      type: "image/svg+xml",
    },
  ],
});

When you open the page in a browser, the rel and type attributes are present, but the href attribute is dropped.

Note: The same happens when using useHeadSafe

Describe the bug
According to the documentation, the href property on link is in the allowed list.
https://unhead.harlanzw.com/guide/composables/use-head-safe#logic
As mentioned above, when loaded in the browser, the href attribute is not present on the link element.
Additional context
No response
Logs
No response

This is a bug, it should be implemented to allow href on a subset of rels.
Will try get to it when I have a chance, PRs welcome 🙂

This should be fixed in v1.1.28, let me know if you have any issues :)


All roles assume v13

Despite being able to specify the installation archive which could be any version, the role assumes v13.0 when invoking the installers.

fixed in dccc939


Unable to run certain images - issue with supervisord

BUG REPORT INFORMATION
docker version:
Client version: 1.5.0
Client API version: 1.17
Go version (client): go1.4.1
Git commit (client): a8a31ef
OS/Arch (client): linux/amd64
Server version: 1.5.0
Server API version: 1.17
Go version (server): go1.4.1
Git commit (server): a8a31ef
docker info:
Containers: 9
Images: 230
Storage Driver: overlay
Backing Filesystem: extfs
Execution Driver: native-0.2
Kernel Version: 3.19.3-1.el7.elrepo.x86_64
Operating System: CentOS Linux 7 (Core)
CPUs: 1
Total Memory: 3.41 GiB
Name: ip-10-89-12-242.eu-west-1.compute.internal
ID: 4PJ3:U5FE:Q3ZK:7LVO:KCJU:63AF:ARY2:PXTM:ZFR7:5DVK:HKPW:DRJ5
uname -a:
Linux  3.19.3-1.el7.elrepo.x86_64 #1 SMP Thu Mar 26 10:57:36 EDT 2015 x86_64 x86_64 x86_64 GNU/Linux
Provide additional environment details (AWS, VirtualBox, physical, etc.):
AWS
List the steps to reproduce the issue:

docker run siomiz/chrome

Describe the results you received:
exec: "/usr/bin/supervisord": stat /usr/bin/supervisord: no such file or directoryFATA[0000] Error response from daemon: Cannot start container 71c44656160c5ed757a0aaa9aede5e613ff530b5a99a458aeb47a68939d5f620: exec: "/usr/bin/supervisord": stat /usr/bin/supervisord: no such file or directory
Describe the results you expected:
Container runs
Provide additional info you think is important:
I am running into issues with supervisord, shows up a  few different images, an example below is from the registry called siomiz/chrome
[root@steve]# docker run siomiz/chrome
exec: "/usr/bin/supervisord": stat /usr/bin/supervisord: no such file or directoryFATA[0000] Error response from daemon: Cannot start container 71c44656160c5ed757a0aaa9aede5e613ff530b5a99a458aeb47a68939d5f620: exec: "/usr/bin/supervisord": stat /usr/bin/supervisord: no such file or directory
[root@steve]# docker run siomiz/chrome

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

I was unable to reproduce this on 1.7.1, using the AUFS graph driver on Ubuntu. It's possible that this relates to overlay driver being used (which has some known issues around unix sockets)
Are you still able to reproduce this on an up to date docker release (currently, 1.7.1)?

Or 1.8.
There was an issue with being able to remove layers that should not be which is fixed in 1.8 as well.
Closing as this seems to have gone cold and is likely resolved already.
If you still have an issue, please ping.


J01: Add MeterValues Response event for 201

I would remove the debug logging if I were you.

Whoops, this is a duplicate of #77 which just wasn't merged yet


[Candidate 3v3 Cursed Hollows ZH v1] Some unusable houses still show on radar

Observed behavior
Some unusable houses still show on radar.
Expected behavior
All unusable houses do not show on radar.
Screenshots

fixed


Can I add content about Azure Spring Apps under TOC node: Azure Categories / Web

Hi, there.
This is Rujun from Azure Spring team.
Now I want to add a reference architecture article about best practice of building web applications using Spring framework and deploy to Azure Spring Apps. Here is related issue: https://github.com/Azure/azure-sdk-for-java/issues/32637
In current TOC, I found this: Azure Categories / Web.
I think it's a good place to add my article. But I found that all articles are about Azure App Service. My concern is that maybe it's not a good place to put my article here.
Here are my question:

Do you have any concerns if I add article about Azure Spring Apps under Azure Categories / Web.
If Azure Categories / Web is not a good place, do you have any better suggestions? For example: https://learn.microsoft.com/en-us/azure/architecture/microservices/

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

ID: 2fad1a93-de94-0030-0733-3c2441d25c3c
Version Independent ID: 2fad1a93-de94-0030-0733-3c2441d25c3c
Content: Web architecture design - Azure Architecture Center
Content Source: docs/guide/web/web-start-here.md
Service: architecture-center
Sub-service: azure-guide
GitHub Login: @martinekuan
Microsoft Alias: architectures

I found this: https://learn.microsoft.com/en-us/azure/architecture/example-scenario/blue-green-spring/blue-green-spring
So I think it's OK to add article about Azure Spring Apps under Azure Categories / Web.
Closing this issue.
Tell me if you have any concerns.


Use of SystemInformation.MouseWheelScrollDelta within MouseWheel event for Zooming

Hi,
I want to bring attention to the usage of SystemInformation.MouseWheelScrollDelta which cannot be relied upon.
It causes an issue where cannot Zoom without scrolling the mouse wheel a lot because there are mice that have a different delta constant to what is given by SystemInformation.MouseWheelScrollDelta.
The mouse I have right now has a delta of 30 but SystemInformation.MouseWheelScrollDelta is 120.
The code where issue is:
       int spins;

        // The MouseWheel event can contain multiple "spins" of the wheel so we need to adjust accordingly
        spins = Math.Abs(e.Delta / SystemInformation.MouseWheelScrollDelta);

        // TODO: Really should update the source method to handle multiple increments rather than calling it multiple times
        for (int i = 0; i < spins; i++)
        {
          this.ProcessMouseZoom(e.Delta > 0, e.Location);
        }

One way to fix the issue is to keep a remainder of delta / SystemInformation.MouseWheelScrollDelta and make sure to reset the remainder to 0 if scroll direction changes or scrolling stops (after a period of time has passed).
I have put together code changes for the above fix. The code has been tested and the fix is working.

    private const int MouseWheelScrollDeltaRemainderResetDelay = 250;
    private int _mouseWheelScrollDeltaRemainder;
    private int _mouseWheelScrollDeltaUpdateTime;

    protected override void OnMouseWheel(MouseEventArgs e)
    {
      base.OnMouseWheel(e);

      if (this.AllowZoom && this.SizeMode == ImageBoxSizeMode.Normal)
      {
        // A remainder is kept so can support mice that use a  delta value that isn't SystemInformation.MouseWheelScrollDelta
        // for the mouse wheel.
        //
        // The remainder will also be reset to 0 after a time delay to prevent it affecting future scrolling where user stops
        // scrolling and then comes back and starts scrolling again. In that case we want the remainder to start at 0 again.
        //
        // To detect the switching of scroll direction will simply just check the sign of delta and delta remainder.
        // Then can reset the remainder to 0 because do not want the scrolling in different direction to affect the current direction.
        int delta = e.Delta + this._mouseWheelScrollDeltaRemainder;

        if (Environment.TickCount - this._mouseWheelScrollDeltaUpdateTime > MouseWheelScrollDeltaRemainderResetDelay ||
            (delta < 0) != (this._mouseWheelScrollDeltaRemainder < 0))
        {
          this._mouseWheelScrollDeltaRemainder = 0;
        }

        this._mouseWheelScrollDeltaRemainder = delta % SystemInformation.MouseWheelScrollDelta;
        this._mouseWheelScrollDeltaUpdateTime = Environment.TickCount;

        int spins;

        // The MouseWheel event can contain multiple "spins" of the wheel so we need to adjust accordingly
        spins = Math.Abs(delta / SystemInformation.MouseWheelScrollDelta);

        // TODO: Really should update the source method to handle multiple increments rather than calling it multiple times
        for (int i = 0; i < spins; i++)
        {
          this.ProcessMouseZoom(e.Delta > 0, e.Location);
        }
      }
    }

Hello,
Thanks for the bug report, that's something I haven't come across before.
What mouse are you using? I don't really want to add random fixes without being able to reproduce it so I'm slightly wary of this code at present. The MSDN page for WM_MOUSEWHEEL proposes a slightly different solution to what you have implemented, but regardless of the final solution I'd still like to be able to repro it first. I use similar code to the (original) delta code in other controls without custom wheel processing too.
Many thanks for taking the time to log the issue and propose a solution.
Regards;
Richard Moss

Hi,
I had never come across it before either until I started using this mouse. The mouse I am using right now is a Microsoft IntelliMouse Explorer (Model: 1004) (Was previously using a Logitech Performance MX but I need to get a replacement) but as far as I know it is any mouse that has a high resolution/smooth scroll. I have found doing a search on Google for "High-resolution mouse wheel" brings up information.
That fix is for when cannot do partial or per-pixel scrolling. So another way could be to change ProcessMouseZoom so it can do smaller increments using the delta.
A crude way to reproduce the problem for testing would be to change the incoming delta values to a factor of 120 (will need to handle multiples aswell). This will simulate the problem for the mouse you are using.
This will work (I have tested it with my Logitech Performance MX and it reproduces the problem):
int delta = 30 * (e.delta / SystemInformation.MouseWheelScrollDelta);
spins = Math.Abs(delta / SystemInformation.MouseWheelScrollDelta);

Regards,
Charles

Hello,
Thanks for the follow up. I used to love my IntelliMouse Explorers, they were good mice and I still have a couple of receivers still floating around after the base units themselves gave up the ghost. I don't even think you can get them now, just as difficult getting Comfort Curve 2000 replacements.
I'll take a look at this as soon as I can - I've been working on and off on a newer version of the control for some time now, should probably do some more work on that!
Regards;
Richard Moss


Markdown prevails on Katex when parsing a formula

Issue description
Issue description
When using katex to write math formulaes, it is first processed in markdown then in katex, resulting in many broken formulaes without any feedback from katex, as it didn't understand it was supposed to be its.
Minimum reproducible examples
$$

$$

The newline creates two empty paragraphs, instead of a katex block in a single paragraph.
$$ {a}\_1 + a\_{1} $$

The _ are not escaped, resulting in emphasis in markdown, instead of subscripts for katex.
$$
f(x) = x
* y
* z
$$

The * create a list in markdown, instead of a multiplication in katex.
...and so on.
Possible solution
For my website, I wrote two simple shortcodes to solve the issue:
<!-- texblock -->
$${{.Inner}}$$

and
<!-- texline -->
\({{.Inner}}\)

Then in markdown files, I first use {{< katex >}} to include katex, then either {{< texblock >}} or {{< texline >}} to write formulaes.
I'm new to hugo and congo, so this might not be the best solution or most elegant one (is it possible to automatically include katex if any tex shortcode is found?), but it does the job. I'm more than willing to do a PR if you want.
Note: bug previously mentioned in #586
Theme version
v2.6.1
Hugo version
v0.117.0
Which browser rendering engines are you seeing the problem on?
Chromium (Google Chrome, Microsoft Edge, Brave, Vivaldi, Opera, etc.), Firefox (Mozilla Firefox)
URL to sample repository or website
No response
Hugo output or build error messages
No response

So after a quick look at congo's codebase, I went ahead and made the PR, as it was simple enough for me to have a go. :)


Add LED Support (Take two)

Add LED support along with proper requirements.txt

Second time's the charm! Merging.


"stylix.targets.wezterm.enable" does not exist

I think there may be an issue with the wezterm module. In my config I have stylix.targets.wezterm.enable = false and it the following error error: The option "stylix.targets.wezterm" does not exist. Definition values: ┃        - In /nix/store/xvl26ggs70i7kgpik34xbywrn2flsy0q-source/modules/stylix.nix':
{
┃              enable = false;
┃            }I checked the wiki and the spelling is correct as well as the wezterm module in stylix and it should be this option. I don't know if this is releated, but the reason I was trying to disable it was because wezterm was reporting the following issue18:56:01.040  ERROR  mux::connui > Configuration Error: runtime error: [string "/home/neon/.config/wezterm/wezterm.lua"]:5: attempt to index a nil value (global 'config')
stack traceback:
[string "/home/neon/.config/wezterm/wezterm.lua"]:5: in local 'stylix_wrapped_config'
[string "/home/neon/.config/wezterm/wezterm.lua"]:10: in main chunkline in question-- Generated by Stylix
local wezterm = require("wezterm")
wezterm.add_to_config_reload_watch_list(wezterm.config_dir)
local function stylix_wrapped_config()
config.front_end = "WebGpu"
config.enable_wayland = false
end
local stylix_base_config = wezterm.config_builder()
local stylix_user_config = stylix_wrapped_config()
stylix_base_config = {
color_scheme = "stylix",
font = wezterm.font_with_fallback {
"FiraCode Nerd Font Propo",
"Openmoji Color",
},`

AFAIK, the option should be available:
https://github.com/danth/stylix/blob/eccb9f2d63f4582b1c1ffe97d806156147aeee5f/modules/wezterm/hm.nix#L5-L6
How and where are you declaring this?

I have it in /etc/nixos/modules/stylix.nix (Where I put all of my stylix stuff)

WezTerm is configured in Home Manager, so you also need to set Stylix options for it there.
See https://github.com/danth/stylix/issues/554#issuecomment-2363421028 for an example if you're using Home Manager combined with NixOS.

WezTerm is configured in Home Manager, so you also need to set Stylix options for it there.
See #554 (comment) for an example if you're using Home Manager combined with NixOS.

Thank you, this was what I was missing.


Why are there 2 different result orders?

How are they ordered by defaut?

The result is no guarantee of order for now, because the nebula is a distributed system. I will close this issue first util you run into some troubles. And as an alternative way, you could sort the result first if you need an order. And please let us know the details if you truly being troubled with this issue.


Feature/ios add debounce for map move event

Implement calling map efent with debounce effect to prevent unexpected map issues and crashes

Please create a pull request to the multiple_maps branch, not master as described

Also, the code of Android and iOS must be synchronized as much as possible.
Please write the code for Android as well.

Since the original issue has been solved, this PR is closed


ISO-198 smart contract and contract deploy script gptreview

The commit you've shared is a script for deploying a smart contract and interacting with it. Here's a critical review of the code:

Hardcoded values: The application name 'Dog Sensor' is hardcoded. It would be better to pass this as an argument or environment variable to make the script more flexible.
const APP_NAME = 'Dog Sensor';

Error handling: The script lacks specific error handling. While there is a catch block at the end of the script, it would be beneficial to have more granular error handling, especially around contract interactions which could fail for a variety of reasons.
main().catch((error) => {
    console.error(error);
    process.exitCode = 1;
});

Logging: The script logs the entire transaction receipt for each operation. This could be overwhelming and make it hard to find specific information. Consider logging only the necessary details.
console.log("Create Application Log: ", createApplicationTxnReceipt);

Code comments: The script could benefit from more detailed comments explaining what each section of the code does. This would make it easier for other developers to understand the script.

Use of console.log: While it's fine for a development script, in a production environment, you might want to consider a more robust logging solution that can handle different log levels (info, debug, error, etc.) and that can output logs to various transports (console, file, remote server, etc.).

No use of TypeScript: While this is not a problem per se, using TypeScript could help catch errors at compile time, provide better autocompletion and make the code more self-documenting.

No tests: This script doesn't include any tests. It would be beneficial to have tests that verify the contract's functionality and the deployment script itself.

No validation of contract state: After operations like changing the owner of the contract, it would be good to have assertions that validate the state of the contract. This would help catch any issues where the contract's state doesn't change as expected.
const changeOwnerTxn = await isoApplication.changeOwner(otherAccount.address);

No use of environment variables: The script doesn't use environment variables for any configuration. It would be beneficial to use environment variables for things like the Ethereum network to use, private keys, and other configuration options.

No cleanup or teardown: After the script runs, it leaves a deployed contract on the network. It would be good to have a cleanup or teardown step that undoes the changes made by the script, especially if you're using a shared test network.

Please note that some of these points might not apply depending on the context in which this script is used. For example, if this is just a development script used by the author, then some of the points about error handling, logging, and cleanup might not be relevant.


Fix navigation menu

closes https://github.com/covidwatchorg/portal/issues/479
See the bug in this video and see that this updated design fixes it. FTR, a similar bug could be triggered by clicking the image avatar to navigate to the settings page.

Seems to fix the bug, but I'm wondering if it would be better to just use Link redux components instead for the menu items (or wrap the menu items in Link components)? Might be better to just convert it over now to save future band-aiding.

In principle makes sense to just make these "Link" components, though we're not using redux so I don't think those specifically would work.


bash completion for dm.min_free_space

Ref: #20786
Also fixes two broken examples in the docs.
ping @sdurrheimer for zsh completion

LGTM!
thanks for the catch in the documentation, looks like it went in before docs review

LGTM


Add startsWith predicate

Some users have requested prefix matching or startsWith.
This issue was migrated from https://github.com/Netflix/iceberg/issues/49.

The Netflix repository had an incomplete PR that is a good starting point: https://github.com/Netflix/iceberg/pull/78

#327 was merged. Should we close this?

Feel free to close issues like this. It's easy to reopen them if we need to.

I only have comment privileges :)

I didn't realize that was locked down. Good to know, thanks!


Improve dtab ergonomics for dropping path segments

/http/1.1 => /method ;
/method => /$/io.buoyant.http.anyMethodPfx/host ;
/host => /$/io.buoyant.http.anyHostPfx/local ;
/local => /$/inet/127.1/8080 ;

This is verbose, hard to understand, and hard to maintain.  It would be great if we had some kind of syntactic sugar to make this easier.
/http/1.1/*/* => /$/inet/127.1/8080

A first attempt of this posted at https://github.com/twitter/finagle/pull/472


Add support for zstd dictionaries

zstd dictionaries allow you to train on similar data to achieve better compression ratios on small files. This PR provides no way for people to train their own data, just use pre-trained dictionaries for decoding and encoding.

@robjtede Can you review this please?
Since this involves changes to macro_rules, I'd like to review from another maintainer.

@robjtede Can you review this and get this into main before the next minor release?

Reviewing this was very helpful in gaining a better understanding of how the macros work. So, I've made some simplifications that reduce noise in the diff. While giving the same API diff:
 Documenting async-compression v0.4.0 (/Users/rob/Development/forks/async-compression)
    Finished dev [unoptimized + debuginfo] target(s) in 0.48s
    Blocking waiting for file lock on package cache
 Documenting async-compression v0.4.0 (/Users/rob/Development/forks/async-compression)
    Finished dev [unoptimized + debuginfo] target(s) in 0.53s
Removed items from the public API
=================================
(none)

Changed items in the public API
===============================
(none)

Added items to the public API
=============================
+pub fn async_compression::futures::bufread::ZstdDecoder<R>::with_dict(inner: R, dictionary: &[u8]) -> std::io::error::Result<Self>
+pub fn async_compression::futures::bufread::ZstdEncoder<R>::with_dict(inner: R, level: async_compression::Level, dictionary: &[u8]) -> std::io::error::Result<Self>
+pub fn async_compression::futures::write::ZstdDecoder<W>::with_dict(inner: W, dictionary: &[u8]) -> std::io::error::Result<Self>
+pub fn async_compression::futures::write::ZstdEncoder<W>::with_dict(inner: W, level: async_compression::Level, dictionary: &[u8]) -> std::io::error::Result<Self>
+pub fn async_compression::tokio::bufread::ZstdDecoder<R>::with_dict(inner: R, dictionary: &[u8]) -> std::io::error::Result<Self>
+pub fn async_compression::tokio::bufread::ZstdEncoder<R>::with_dict(inner: R, level: async_compression::Level, dictionary: &[u8]) -> std::io::error::Result<Self>
+pub fn async_compression::tokio::write::ZstdDecoder<W>::with_dict(inner: W, dictionary: &[u8]) -> std::io::error::Result<Self>
+pub fn async_compression::tokio::write::ZstdEncoder<W>::with_dict(inner: W, level: async_compression::Level, dictionary: &[u8]) -> std::io::error::Result<Self>

Note on simplification:
The actual problem to be solved was adding custom constructors to the decoders as well as encoders. So instead of poluting the type parameters at callsites, I've just added a way in the macro to do so, with clear markers.
Future work: the runtime encoder/decoder macros know what their generics should be. It it probbaly possible to omit the $inner param from the algos! macro.

Added some testing for this feature.
@NobodyXu if you're happy with my changes we can merge this.

Oh great... Windows-only test failures.

Thanks @robjtede, sorry I couldn't help I've been competing in the GMTK game jam

P.S. @robjtede I think using squash merge is better than merge commits, especially in this PR where I merge changes in main back to this PR.
With squash merge, the commit history would be much more straight-forward.
If you agree, I will disable merge commit in setting and only allow squash merging.

I've disabled merge commit and rebase merge, allowing only squash merge.


[5.3][CodeCompletion] Suggest 'macOS' instead of 'OSX'

Cherry-pick of #32440 into release/5.3

Explanation: for @available attribute and #available statement condition, macOS is preferred over OSX. Since OSX is deprecated, suggest macOS instead of OSX.
Scope: Code completion inside @available and #available
Risk: Very low. The change is simple and scope is limited
Testing: Passed updated regression test cases
Issue: rdar://problem/64379019
Reviewer: Argyrios Kyrtzidis (@akyrtzi)

@swift-ci Please test

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 98298f53d6e0f7d23716ce5f1432eded6a60b9d5

Windows: https://ci-external.swift.org/job/swift-PR-windows/4291/

Build failed
Swift Test OS X Platform
Git Sha - 98298f53d6e0f7d23716ce5f1432eded6a60b9d5

@swift-ci Please nominate


Update channel-and-group-conversations.md

[x] C# samples
[x] TS samples
[ ] Final sample needs example JSON
[ ] context reviewed.

Docs Build status updates of commit c96d53d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/channel-and-group-conversations.md
:warning:Warning

Details

msteams-platform/includes/bots/teams-conversation-bot-sample.md
:warning:Warning

Details

msteams-platform/bots/how-to/conversations/channel-and-group-conversations.md

Line 26: [Warning] Invalid file link:(~/send-proactive-messages.md#creating-a-channel-conversation).
Line 216: [Warning] Invalid file link:(~/bots/how-to/send-proactive-messages.md#welcome-messages).

msteams-platform/includes/bots/teams-conversation-bot-sample.md

Line 3: [Warning] Invalid file link:(~/foo.md).
Line 3: [Warning] Invalid file link:(~/foo.md).
Line 3: [Warning] Invalid file link:(~/foo.md).
Line 3: [Warning] Invalid file link:(~/foo.md).
Line 3: [Warning] Invalid file link:(~/foo.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit b4a0ec1:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/channel-and-group-conversations.md
:warning:Warning

Details

msteams-platform/includes/bots/teams-conversation-bot-sample.md
:warning:Warning

Details

msteams-platform/bots/how-to/conversations/channel-and-group-conversations.md

Line 26: [Warning] Invalid file link:(~/send-proactive-messages.md#creating-a-channel-conversation).
Line 216: [Warning] Invalid file link:(~/bots/how-to/send-proactive-messages.md#welcome-messages).

msteams-platform/includes/bots/teams-conversation-bot-sample.md

Line 3: [Warning] Invalid file link:(~/foo.md).
Line 3: [Warning] Invalid file link:(~/foo.md).
Line 3: [Warning] Invalid file link:(~/foo.md).
Line 3: [Warning] Invalid file link:(~/foo.md).
Line 3: [Warning] Invalid file link:(~/foo.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@WashingtonKayaker Is this one good to merge, or do you need to add another sample?


Update Windows config for error reporting

Original request
When something crashes on a hosted Windows agent, Windows hangs waiting for consent to send telemetry. Consider either disable error reporting, or offer a way to auto-consent.
Area for Triage:

Question, Bug, or Feature?:
Feature
Virtual environments affected

[ ] macOS 10.15
[ ] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[X] Windows Server 2016 R2
[X] Windows Server 2019

@jessehouwing

I consider it a bug ;).

closing in lieu of #806 thanks @vtbassmatt and @jessehouwing !


Tests for concoction database refreshing

This is a learning experience.
I have two fights saved:

A Fruit Golem dropping three consumables
A BASIC Elemental dropping a GOTO

I intend to use the response txts - two per fight: the start and and the finish - to check how many refreshes are done.
Three ways:

GenericRequest - raw request/response
KoLmafia.makeRequest with a single iteration - AdventureRequest
KoLmafia.makeRequest with multiple iterations

So far, I have only the first (for the two monsters).
Discoveries:

My response text had a large number of combat items. KoLmafia compares the "Use Item" dropdowns on a fight page with inventory - and resynchs inventory, which can cause a refresh, if the items are concoction ingredients, say.
My fix was to empty out the combat item dropdowns in my saved HTML.
After we finish a fight, we call api.php. I'm using an empty response, but a real one would show a turn being used - and that would expect a refresh, since an "Adventure" is a virtual ingredient for mixing methods that take turns.

I'm going to have to think about how to simulate that when I start testing automation.

I'm not feeling it any more. I don't see the point.

I'm not feeling it any more. I don't see the point.

Fair enough!


feat: add date format to the email subject

SUMMARY

Problem
If the subject remains the same, all scheduled reports will be delivered in the same email thread, requiring the recipient to scroll back to the original thread to find the latest email.
However, if a date is included in the email subject, the emails will not be grouped in the same thread. To address this, I propose adding a feature that allows users to specify a date format in the email subject, enabling the email client to correctly thread the emails and display them in a more organized manner.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before

After

How to use it

Set the DATE_FORMAT_IN_EMAIL_SUBJECT=True to allow users to include a date format in the email subject, which will automatically be replaced with the corresponding actual date.
available date format  https://strftime.org/
Example
In this example i defined Year-Day-Minutes
Year - %Y
Day - %D
Minutes - %M

Output

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Required feature flags: DATE_FORMAT_IN_EMAIL_SUBJECT (by default is disabled)
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@US579 can you also add some documentation on how this is used? It won't be clear for users who don't know about this feature that they can start doing this post merge.

Could you point me where i could add this doc, currently i just added MR link (which has description for how to use it)  next to the FF

@US579 can you also add some documentation on how this is used? It won't be clear for users who don't know about this feature that they can start doing this post merge.

hey @sadpandajoe Could you point me where is the bast place to add this doc, currently i just added here

@US579 great question. I would say you can either put it in the alerts and reports sections: https://github.com/apache/superset/blob/4ff9aac1fae2d88fd7667317c706256d3cfcb4f4/docs/docs/configuration/alerts-reports.mdx or in the faq https://github.com/apache/superset/blob/4ff9aac1fae2d88fd7667317c706256d3cfcb4f4/docs/docs/faq.mdx

Totally get the use case. Hey I'm wondering if it may make sense to support more templating in emails (subject and body) in general to tackle this use case but similar ones as well. I know jinja isn't safe for untrusted (user-provided) content, maybe we could use some pybars or something untrusted safe to allow for bringing arbitrary context in arbitrary places in the subject/body.


[FEATURE] Add support for cerberus and other credential stores in snowflake operator

Is your feature request related to a problem? Please describe.
Current implementation of snowflake operator supports databricks secrets only. We have a need to use other credential stores like cerberus.
Cloud Information

[ ] AWS
[ ] Azure
[ ] GCP
[ ] Other

Describe the solution you'd like
Enhance the snowflake operator logic to support other credential stores along with databricks secrets.

Cerberus is not opensource anymore, hence it has to be an internal implementation!


fix(@ngtools/webpack): correct path to linker

Fixes https://github.com/angular/angular-cli/issues/4346

I am not sure where the errors come from, as there are no tests for the code I touched, that I know of.


Issue-360. Exception appeared on shutdown

A possible solution to the latest comment could be to base64 encode the chat name(afair all base64 characters can be used in a file path)


not able to use with symfony/console v5+

Working on it, started a little while ago but did not finish it, it'll be soon now


net_download fail

I'm currently trying to try out the net_download command, but I am getting the following error.
Accessing: https://cmiclab.cs.ucl.ac.uk/CMIC/NiftyNetExampleServer
Warning: updating the examples file from the server caused an error: [Errno socket error] [SSL: UNKNOWN_PROTOCOL] unknown protocol (_ssl.c:661)
highres3dnet_brain_parcellation_model_zoo: FAIL.
Cannot download the examples configuration file. Is the server down?

What can I do?

I also encountered the same problem when using net_download, are you Chinese?  I think it may be because of our network.

try to climb over the wall

Create discussion groups from Chinese niftyNet users, please join QQ:684027141


Galaxy brain multipliedReportingOverflow(by:)

From https://github.com/oscbyspro/Numberick/issues/6

It turns out that multipliedReportingOverflow(by:) can be written simply and efficiently with the overflow indicator returned by formTwosComplementSubsequence(true). I recently realized that you can use it to check if unsigned integers equal zero, and if signed integers equal their minimum representable value. [...].

This requires (#121).


security configuration issue with opensearch multi-node cluster setup (in k8s env)

Hi,
I am spending some time these days to prepare a helm chart for setting up a complete (scalable) opensearch logging stack in a kubernetes env. The AWS Opendistro for Elasticsearch setup works ok and now the effort is dedicated to get the helm chart supporting also opensearch
Here is the current state for the opensearch case:
When a single node setup is used then the cluster is initialized properly. Single node type can be successfully scaled up and everything is functioning as expected.
However the chart also supports separating node types (coordination, data and master) and allows scaling up those independently.  In this case setting up the security configuration although applied successfully doesn't bring the cluster in the expected functional state.
The same config works just fine when all nodes share same (all) types. I'd like to ask for a help to get the multi type setup also in a correct functional state. Please drop me an email and we can organise a remote session where I can go into details of the setup and show where I see the problems. Meanwhile I will continue trying to find a solution.
Will follow up if successful.
Thanks
Niki Dokovski

@nickytd Thanks for reporting this. It would be great if you could try to post a way to reproduce this here, rather than trying to get someone to do remote sessions. It's more work, but this way we can build something repeatable once we identify the problem. Let's do remote as a last resort?

Yes sure. The helm charts has examples folder which contains configurations for the various setup modes.
So in short to test the different modes.

get a minikube and start it with 2 nodes (minikube start --nodes=2)
checkout the helm chart and switch to -with-opensearch branch
Make sure current kubeconfig targets the minikube cluster and run the setup.sh in the examples folder.
This brings the cluster setup with a single opensearch node, fluent bit and opensearch-dashboard. The scenario works as expected. You can authenticate in the opensearch-dashboards, check dashboards, switch tenants etc.

To simulate the multi node setup, get a clean minikube cluster, (with 2 nodes) and execute this time the setup.sh --scaled from the examples folder.  This time however the operational state of cluster is not correct.
The main branch of the repo contains the current related version of the helm chart based on AWS OpenDistro for Elasticsearch. And the goal here is to release it with opensearch 1.0 GA.
Meanwhile I also continue looking into the issue and will update in case of a progress. Any other feedback is of course more than welcome
Thanks
Niki Dokovski

The issue goes down to a problem with the index mappings.
There are predefined templates which are uploaded to the cluster during the setup phase.
However when fluentd starts to put (_bulk) logs to the opensearch the data node rejects those with
[2021-07-23T07:40:52,547][INFO ][o.o.a.b.TransportShardBulkAction] [efk-data-0] [containers-2021.07.23][0] mapping update rejected by primary
java.lang.IllegalArgumentException: mapper [kubernetes.pod_name] cannot be changed from type [keyword] to [text]
at org.opensearch.index.mapper.ParametrizedFieldMapper.merge(ParametrizedFieldMapper.java:103) ~[opensearch-1.0.0.jar:1.0.0]
And when looking into the index settings the template mapping is indeed not there.
{
"systemd-2021.07.23": {
"settings": {
"index": {
"codec": "best_compression",
"mapping": {
"total_fields": {
"limit": "2000"
}
},
The index templates are uploaded and present in the cluster (at least from the maser node logs) long before any indexing starts and at least the master node acknowledges this and we can see this in the logs
[2021-07-23T07:40:01,370][INFO ][o.o.c.m.MetadataCreateIndexService] [efk-master-0] [containers-2021.07.23] creating index, cause [auto(bulk api)], templates [containers], shards [1]/[0]
In summary:
Master Node: creates index with a template
Data Node: rejects indexing as it doesn't see the index template
Thanks
Niki Dokovski
PS / this very same configuration is fully functional when all nodes share same types and there are no separation on coordination, master and data nodes /


Improve/document test experience

Currently we have to run all tests for it to be a pleasant experience to run tests, via the tests.js script.
I propose we split it up into 2 scripts - one called prepare.js that does all the preparation, and one file that actually runs the tests. This will allow us to run the tests e.g. from an IDE.
We should also look to symlink our dist folder, so we don't need to copy all the files every run.

Haha, I never saw this issue, but it is actually more or less what is done in beta.
I'm going to close this since I don't think it's worth spending some time on implementing it for current version while it exists already in beta. @GeeWee please re-open if you think this shouldn't be closed


Edits to OAuth example and moving the content to the cookbook

Closing.
See: https://github.com/mulesoft/docs-munit/pull/49/files
And https://github.com/mulesoft/docs-mule-sdk/pull/46


NLogTraceListener - Reduce code complexity of ProcessLogEventInfo

Hi, is this refactored by a tool, or do I need to double-check it? (E.g. diff)

It was done with tool, but with a small adjustment afterwards to avoid returning updated loggerName
Sent from my Sony Xperia
---- Julian Verdurmen wrote ----
Hi, is this refactored by a tool, or do I need to double-check it? (E.g. diff)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/NLog/NLog/pull/3202#issuecomment-477025694, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK-fnOWZgb1F5SNhHMrNLJh_qm9L21eAks5vayX0gaJpZM4cMnhR.

travis seems to be down for now

travis seems to be down for now

No your dependabot[bot] has just made a huge queue of meaningless builds, which it is spinning up and down build-server to handle.

ah thanks!

travis should not build closed prs. That should be fixed now

@304NotModified Looks like your "fix" have now stopped travis from building anything :)

@304NotModified But it seems to build nice for NLog.Extension.Logging: https://travis-ci.org/NLog

@304NotModified Now it is building again: https://travis-ci.org/NLog/NLog/builds/512191462

Build successfully, merged, done! Thanks!


Live Preview Brackets Issue

My bracket does not auto update while editing the HTML, so I enabled experimental live preview, but now it tells me i have to load an HTML file even though thats the one I want to open. Could someone help me out?
The picture says: 'Load an HTML file to start Live Preview.'

Start by reading the Live Preview section of the How To Use Brackets doc and the Live Preview Isn't Working section of the Brackets Troubleshooting guide.


Id is used instead of positiveInt

Hello,
Thank you for sharing these awesome packages.
I'm currently using them as a part of a flutter project and came across some issues regarding serialization.
looking at the implementation https://github.com/akdu12/fhir/blob/main/fhir/lib/stu3/resource_types/financial/general/general.dart careTeamLinkId , procedureLinkId , diagnosisLinkId  and informationLinkId are considered as a list of id while they should be a positiveInt http://hl7.org/fhir/STU3/explanationofbenefit.html

#14

Thanks for the PR!


UE5.2.1 Packaging error:Unknown Cook Failure .Could I trouble you to help?Thanks.

I am still in process in UE5 learning,and I just instaled the Visual Studio 2022.
However,when I was packaging it as the course of the study,it errored.
Could I trouble you to help?Thanks again.
This is the ending of the log:
UATHelper: 打包 (Windows): LogShaderCompilers: Display: === FShaderJobCache stats ===
UATHelper: 打包 (Windows): LogShaderCompilers: Display: Total job queries 0, among them cache hits 0 (0.00%)
UATHelper: 打包 (Windows): LogShaderCompilers: Display: Tracking 0 distinct input hashes that result in 0 distinct outputs (0.00%)
UATHelper: 打包 (Windows): LogShaderCompilers: Display: RAM used: 240.00 B of 819.20 MiB budget. Usage: 0.00%
UATHelper: 打包 (Windows): LogShaderCompilers: Display: === Shader Compilation stats ===
UATHelper: 打包 (Windows): LogShaderCompilers: Display: Shaders Compiled: 0
UATHelper: 打包 (Windows): LogShaderCompilers: Display: Jobs assigned 0, completed 0 (0.00%)
UATHelper: 打包 (Windows): LogShaderCompilers: Display: Time at least one job was in flight (either pending or executed): 0.00 s
UATHelper: 打包 (Windows): LogShaderCompilers: Display: ================================================
UATHelper: 打包 (Windows): Took 18.1252s to run UnrealEditor-Cmd.exe, ExitCode=1
UATHelper: 打包 (Windows): ERROR: Cook failed.
UATHelper: 打包 (Windows):        (see C:\Users\ZryYanc_\AppData\Roaming\Unreal Engine\AutomationTool\Logs\C+Program+Files+Epic+Games+UE_5.2\Log.txt for full exception trace)
UATHelper: 打包 (Windows): AutomationTool executed for 0h 0m 18s
UATHelper: 打包 (Windows): AutomationTool exiting with ExitCode=25 (Error_UnknownCookFailure)
UATHelper: 打包 (Windows): BUILD FAILED
PackagingResults: Error: Unknown Cook Failure
This is the former log：
C:\GameExploitation\UEProject\Demo2_zhangliang002_2024.8\Demo2\Content\Resources\Effects_Particles\FXVarietyPack\EpicContent\Materials\M_DemoWallPattern.uasset: Failed to compile Material for platform PCD3D_SM6, Default Material will be used in game.
UATHelper: 打包 (Windows):   (Node TextureSample) Texture Sample> Missing input texture
UATHelper: 打包 (Windows):   (Node TextureSample) Texture Sample> Missing input texture
UATHelper: 打包 (Windows): LogInit: Display: LogMaterial: Warning: [AssetLog] C:\GameExploitation\UEProject\Demo2_zhangliang002_2024.8\Demo2\Content\Resources\Effects_Particles\FXVarietyPack\EpicContent\Materials\M_DemoWallPattern.uasset: Failed to compile Material for platform PCD3D_SM5, Default Material will be used in game.
UATHelper: 打包 (Windows):   (Node TextureSample) Texture Sample> Missing input texture
UATHelper: 打包 (Windows):   (Node TextureSample) Texture Sample> Missing input texture
UATHelper: 打包 (Windows): LogInit: Display: LogMaterial: Warning: [AssetLog] C:\GameExploitation\UEProject\Demo2_zhangliang002_2024.8\Demo2\Content\Resources\Effects_Particles\FXVarietyPack\EpicContent\Materials\M_DemoRoomTiles.uasset: Failed to compile Material for platform PCD3D_SM6, Default Material will be used in game.
UATHelper: 打包 (Windows):   (Node TextureSample) Texture Sample> Missing input texture
UATHelper: 打包 (Windows):   (Node TextureSample) Texture Sample> Missing input texture
UATHelper: 打包 (Windows): LogInit: Display: LogMaterial: Warning: [AssetLog] C:\GameExploitation\UEProject\Demo2_zhangliang002_2024.8\Demo2\Content\Resources\Effects_Particles\FXVarietyPack\EpicContent\Materials\M_DemoRoomTiles.uasset: Failed to compile Material for platform PCD3D_SM5, Default Material will be used in game.
UATHelper: 打包 (Windows):   (Node TextureSample) Texture Sample> Missing input texture
UATHelper: 打包 (Windows):   (Node TextureSample) Texture Sample> Missing input texture
UATHelper: 打包 (Windows): LogInit: Display: LogMaterial: Warning: Cooking a material resource (in M_DemoWallPattern hierarchy) that doesn't have a valid ShaderMap! Shadermap pointer is null.
UATHelper: 打包 (Windows): LogInit: Display: NOTE: Only first 50 warnings displayed.
UATHelper: 打包 (Windows): LogInit: Display:
UATHelper: 打包 (Windows): LogInit: Display: Failure - 9 error(s), 129 warning(s)
UATHelper: 打包 (Windows): LogInit: Display:
UATHelper: 打包 (Windows): Execution of commandlet took:  9.27 seconds
UATHelper: 打包 (Windows): LogShaderCompilers: Display: ================================================
UATHelper: 打包 (Windows): LogShaderCompilers: Display: === FShaderJobCache stats ===
UATHelper: 打包 (Windows): LogShaderCompilers: Display: Total job queries 0, among them cache hits 0 (0.00%)
UATHelper: 打包 (Windows): LogShaderCompilers: Display: Tracking 0 distinct input hashes that result in 0 distinct outputs (0.00%)
UATHelper: 打包 (Windows): LogShaderCompilers: Display: RAM used: 240.00 B of 819.20 MiB budget. Usage: 0.00%
UATHelper: 打包 (Windows): LogShaderCompilers: Display: === Shader Compilation stats ===
UATHelper: 打包 (Windows): LogShaderCompilers: Display: Shaders Compiled: 0
UATHelper: 打包 (Windows): LogShaderCompilers: Display: Jobs assigned 0, completed 0 (0.00%)
UATHelper: 打包 (Windows): LogShaderCompilers: Display: Time at least one job was in flight (either pending or executed): 0.00 s
UATHelper: 打包 (Windows): LogShaderCompilers: Display: ================================================
UATHelper: 打包 (Windows): Took 18.1252s to run UnrealEditor-Cmd.exe, ExitCode=1
UATHelper: 打包 (Windows): ERROR: Cook failed.
UATHelper: 打包 (Windows):        (see C:\Users\ZryYanc_\AppData\Roaming\Unreal Engine\AutomationTool\Logs\C+Program+Files+Epic+Games+UE_5.2\Log.txt for full exception trace)
UATHelper: 打包 (Windows): AutomationTool executed for 0h 0m 18s
UATHelper: 打包 (Windows): AutomationTool exiting with ExitCode=25 (Error_UnknownCookFailure)
UATHelper: 打包 (Windows): BUILD FAILED
PackagingResults: Error: Unknown Cook Failure

This might have nothing to do with Cesium; you should carefully review the log files to look for other causes.

Yeah @yanccee from the log it looks like a problem with some materials in a plugin called FXVarietyPack, nothing obvious to do with Cesium. So I'm going to close this issue, but if you do think it's Cesium related, please start a thread on the community forum instead of making a GitHub issue, and include as much detail as you can, particularly around what makes you think it's Cesium-related.


Portuguese version

Hi. I'd like to translate your content about markdown to my primary language. It is any possible? If yes, how do it do it?

Hi! It is possible, but there is not a whole lot of documentation.
You must change all of the same files in this PR: https://github.com/gjtorikian/markdowntutorial.com/pull/79/files
Just imitate the changes there in a pt folder and it should be good. Also is this Brazillian Portuguese or from Portugal or elsewhere>

Is Brazillian Portuguese. I'll do it here


Not creating chroot environment after debootstrap update - package install failure

sudo edit-chroot -all
name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
Creating hwaudio group with GID 18...
Changing audio GID from 29 to 600...
Changing cdrom GID from 24 to 19...
Creating crouton group with GID 1001...
Creating devbroker-access group with GID 403...
Changing floppy GID from 25 to 11...
Creating i2c group with GID 404...
Creating input group with GID 222...
Creating serial group with GID 402...
Changing plugdev GID from 46 to 85...
Changing uucp GID from 10 to 14...
Changing video GID from 44 to 27...
Moving sudo GID from 27 to 28...
Creating wayland group with GID 601...
crouton: version unknown
release: unknown
architecture: unknown
targets: cli-extra
host: version 10575.58.0 (Official Build) stable-channel candy
kernel: Linux localhost 4.4.127-13890-g6c206f1711d4 #1 SMP PREEMPT Fri Jun 22 17:23:19 PDT 2018 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...
Please describe your issue:
Some base packages not installing, failing to set up chroot environment:
W: Failure while installing base packages.  This will be re-attempted up to five times.
W: See //debootstrap/debootstrap.log for details (possibly the package /var/cache/apt/archives/kbd_1.15.5-1ubuntu4_amd64.deb is at fault)
I: Unpacking kbd...
W: Failure while installing base packages.  This will be re-attempted up to five times.
W: See //debootstrap/debootstrap.log for details (possibly the package /var/cache/apt/archives/kbd_1.15.5-1ubuntu4_amd64.deb is at fault)
W: Failure while installing base packages.  This will be re-attempted up to five times.
W: See //debootstrap/debootstrap.log for details (possibly the package /var/cache/apt/archives/kbd_1.15.5-1ubuntu4_amd64.deb is at fault)
W: Failure while installing base packages.  This will be re-attempted up to five times.
W: See //debootstrap/debootstrap.log for details (possibly the package /var/cache/apt/archives/kbd_1.15.5-1ubuntu4_amd64.deb is at fault)
W: Failure while installing base packages.  This will be re-attempted up to five times.
W: See //debootstrap/debootstrap.log for details (possibly the package /var/cache/apt/archives/kbd_1.15.5-1ubuntu4_amd64.deb is at fault)
Failed to complete chroot setup.
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...
If known, describe the steps to reproduce the issue:
sudo sh crouton -t cli-extra

yea.. seems to be due to the fact that the debootstrap URL has died or something, he is fixing it rn

This happened after the URL was fixed...
Here are the last few lines of debootstrap.log:
Processing triggers for libc-bin (2.23-0ubuntu3) ...
Errors were encountered while processing:
/var/cache/apt/archives/kbd_1.15.5-1ubuntu4_amd64.deb
/var/cache/apt/archives/libpng12-0_1.2.54-1ubuntu1_amd64.deb
dpkg: warning: parsing file '/var/lib/dpkg/status' near line 4297 package 'dpkg':
missing description
dpkg: warning: parsing file '/var/lib/dpkg/status' near line 4297 package 'dpkg':
missing architecture
dpkg: error: parsing file '/var/lib/dpkg/status' near line 4297 package 'dpkg':
multiple non-coinstallable package instances present; most probably due to an upgrade from an unofficial dpkg

David,
Just letting you know that it worked with version 101. Thanks for your help!

Awesome.  Thanks for the feedback!


[Null-aware elements] Language Tests in tests/language

Write up language tests for the tests/language repository.
The example is pulled from the spec.
void printThree(String? a, String? b, String? c) {
  print([?a, ?b, ?c].join(' '));
}

main() {
  printThree('first', null, 'last');
}

The current direction is to convert the emergent unit tests into language tests and to get @lrhn's help to generate particularly tricky examples.


Kapp-controller will not work properly in Kubernetes 1.24.

I raised this in a private Slack channel discussion but since not seen any issue created here by others since that discussion, creating one.
What steps did you take:
Install kapp-controller in Kubernetes 1.24.
Try installing a package. Installation fails with the error:
kctrl: Error: packageinstall/xxx (packaging.carvel.dev/v1alpha1) namespace: xxx:
  Reconciling:
    Preparing kapp:
      Expected to find one service account token secret, but found none. Reconcile failed:
        Error (see .status.usefulErrorMessage for details)

What happened:
The problem relates to:

https://www.padok.fr/en/blog/new-kubernetes-changes#No_Secret_by_default_for_service_account_tokens
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.24.md#no-really-you-must-read-this-before-you-upgrade-1
https://github.com/kubernetes/kubernetes/pull/108309

That is, in Kubernetes 1.24, an API token secret is no longer automatically created and listed as a secret against a service account. For in-cluster API tokens for pods a new mechanism is now used to allow token refreshes every hours.
A service account therefore now looks like:
apiVersion: v1
kind: ServiceAccount
metadata:
  creationTimestamp: "2022-05-13T03:34:02Z"
  name: default
  namespace: lab-k8s-fundamentals-w01-s001
  resourceVersion: "1744"
  uid: ff853727-1989-4f49-b5ab-a0e1ec51944c

Note how there is no secrets property like before.
It seems kapp-controller would look up the API token secret name in secrets of the service account and use the token in the linked secret to do stuff. Since this no longer exists, kapp-controller fails with the error shown above.
What did you expect:
It works.
Anything else you would like to add:
This will likely have implications for where a distinct service account is provided by a user when installing a package, rather than the default one used. For the default, kapp-controller can do something else whereby it has created an API token secret of its own to use and linked it up to the default service account used.
For a user providing a service account that will no longer be sufficient, unless you tell users to also create a separate API token secret linked to the service account. What then needs to happen is that kapp-controller needs to search for any secret in the same namespace with the kubernetes.io/service-account.name annotation giving the name of the service account it is for.
To avoid that, it may or may not be possible to grant RBAC to kapp-controller to create API token secrets and create them for the user. The question is whether that is a good idea and raises security concerns and thus whether force it on the users to create the API secret for the service account they nominate.
So basically it is going to be messy. :-(
Environment:

Kubernetes version (use kubectl version)

1.24

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

This is how we solve this by impersonating ServiceAccounts /w Flux via User Impersonation

https://github.com/fluxcd/kustomize-controller/blob/v0.25.0/controllers/kustomization_impersonation.go#L106-L115
https://github.com/fluxcd/helm-controller/blob/main/internal/kube/impersonate.go#L36-L47

This means that you don't even need a BearerToken for the account as long as the kapp-controller ServiceAccount has ClusterRoleBinding to a ClusterRole /w the impersonate verb for SA's
As long as we have a way to change the client configuration via Go, this is easy.
This needs to be done for every tool that kapp-controller uses that might talk to the k8s cluster.
If we need to do this by exec'ing a CLI that is using client-go, there is a bug that I've been meaning to fix for years with the heuristic in-cluster config that we can prioritize. I have a POC fix, but the code-path in client-go is confusing/hard-to-patch.
A workaround by writing a kube-config within the controller-filesystem to circumvent the heuristics is also perfectly usable without the bug fix.

This should be fixed as part of https://github.com/vmware-tanzu/carvel-kapp-controller/pull/695
We opted for a TokenRequest by KC instead of user impersonation.


Add go.mod at root

Codecov Report

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

@@            Coverage Diff             @@
##           master     #302      +/-   ##
==========================================
+ Coverage   99.11%   99.81%   +0.69%
==========================================
  Files          69       19      -50
  Lines        3153     1053    -2100
==========================================
- Hits         3125     1051    -2074
+ Misses         17        1      -16
+ Partials       11        1      -10

Flag
Coverage Δ

#dot
99.81% <ø> (+0.69%)
:arrow_up:

#dot_changes
100% <ø> (ø)
:arrow_up:

#dot_changes_diff
100% <ø> (ø)
:arrow_up:

#dot_changes_run
100% <ø> (ø)
:arrow_up:

#dot_changes_table
100% <ø> (ø)
:arrow_up:

#dot_changes_types
100% <ø> (ø)
:arrow_up:

#dot_example
100% <ø> (ø)
:arrow_up:

#dot_log
?

#dot_ops
?

#dot_ops_bolt
?

#dot_ops_nw
?

#dot_ops_pg
?

#dot_ops_sjson
?

#dot_ops_sync
?

#dot_refs
?

#dot_streams
?

#dot_streams_undo
?

#dot_test_seqtest
?

#dot_test_testops
?

#dot_x_dotc
?

#dot_x_fold
?

#dot_x_heap
?

#dot_x_meta
?

#dot_x_meta_snapshot
?

Impacted Files
Coverage Δ

streams/undo/stream.go

ops/nw/request.go

x/dotc/structs.go

x/dotc/union.go

x/dotc/slice.go

x/meta/snapshot/snapshot.go

refs/caret.go

ops/nw/server.go

x/dotc/slices.go

x/dotc/unions.go

... and 40 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 011d8cf...62df016. Read the comment docs.


apk add failed

$ apk add vim
ERROR: Unable to lock database: Read-only file system
ERROR: Failed to open apk database: Read-only file system
but by mount get that  /  mount point is "rw" mounted.

I think this is due to /usr being mounted as read only and /bin & /sbin being symlinked to /usr/bin & /usr/sbin, respectively
attempting to change the root directory in which to install packages while using apk add {package} produces a similar error, but says "no such file or directory"

Why not setup apk properly ? Alpine support lbu, seems a good fit for this.

I can only guess as to the developers intentions. I don't know if they plan to have packages managed by the abstraction layer they may put in or if the idea is to make a system that cannot have "proper" package management. Of course none of this makes it more secure, a piece of malware just has to be compiled prior to saving to a directory that can be written to, or be in the class of "in memory". I would be interested in an official position on all of this as well.

I hit this issue. Is there any workaround to get rid off this?

From the README.md (emphasis mine):

k3OS is a Linux distribution designed to remove as much as possible OS maintenance in a Kubernetes cluster. It is specifically designed to only have what is need to run k3s. Additionally the OS is designed to be managed by kubectl once a cluster is bootstrapped. Nodes only need to join a cluster and then all aspects of the OS can be managed from Kubernetes. Both k3OS and k3s upgrades are handled by the k3OS operator.

Software installation via Alpine's apk is not supported.


pickadate version

The datepicker version used is more than a year old: https://github.com/Dogfalo/materialize/blob/master/js/date_picker/picker.js
Are there plans to upgrade to current version 3.5.6 and add timepicker functionality ?

This is a lower priority for us, upgrading the version takes some time as we made a lot of modifications to it. Maybe when the major version is updated we will upgrade.


minibuffer-depth-indicate-mode has display issues with recursive minibuffers

The depth indication switches positions and flickers.

Should be fixed.

Can confirm, it's fixed. As an related idea: It would be nice if the "Quit" message wouldn't "take over" the minibuffer when quitting a recursive level but I'm not sure how to fix that.

Here is a hacky way to do it:
(defun selectrum-abort-recursive-edit ()
  (interactive)
  (let ((om (get 'quit 'error-message))
        (wh (window-height)))
    (put 'quit 'error-message "")
    (run-at-time 0 nil
                 (lambda ()
                   (put 'quit 'error-message om)
                   (enlarge-window (1- wh))))
    (abort-recursive-edit)))

What version of Emacs are you using? In release 26 and previous, the way messages are displayed while the minibuffer is active is unworkably bad. In release 27 and subsequent, it looks like this:

Where the [Quit] disappears after two seconds or when input is received.

Ah, nice thanks! I'm using 26 but will switch soon


Add database table 'delete' functionality

To Do:

Create a generic function within getstreamy.db that removes all records from all tables in a database that contain some query-index where the query-index is in some list of query-index-values.

Refer to getstreamy.vault.delete_account as a reference for the pattern

Create additional functions that handle special deletion scenarios for when Users, Sessions and Data are removed

Deletion scenarios:
Users
1. Retrieve all tables in {USERS_DB_NAME} that contain {SESSIONS_QUERY_INDEX},
    - For all tables,
      - Get a list of {SESSIONS_DB_QUERY_INDEX} values associated only with the {USERS_DB_QUERY_INDEX} value to be deleted (these are sessions only associated with that particular user that will become "orphaned" once the user is deleted)
2. Retrieve all tables in {SESSIONS_DB_QUERY_INDEX} that contain {DATA_DB_QUERY_INDEX},
    - For all tables,
      - Get a list of {DATA_DB_QUERY_INDEX} values associated with all {SESSIONS_DB_QUERY_INDEX} values
3. Run the generic function to delete the selected {USERS_DB_QUERY_INDEX} value from all tables in {USERS_DB_NAME}
4. Run the generic function to delete the orphaned {SESSIONS_DB_QUERY_INDEX} values from all tables in {SESSIONS_DB_NAME}
5. Run the generic function to delete the orphaned {DATA_DB_QUERY_INDEX} values from all tables in {DATA_DB_NAME}
6. Drop all tables in {DATA_DB_NAME} for each orphaned {DATA_DB_QUERY_INDEX} value

Sessions
Run (2.), (4.)-(6.).

Data
Run (5.)-(6.).

Bug issue - Investigate and resolve an issue regarding deleting sessions where after deletion, the session-id is re-added to the database with empty values for all other fields