svclb pods not deleted when services are deleted

Following the merge of https://github.com/k3s-io/k3s/pull/5657, deleting LoadBalancer Services leaves orphan svclb pods that block reuse of ports by subsequent services.
Steps to reproduce:
brandond@dev01:~$ kubectl apply -f - <<EOF
apiVersion: v1
kind: Service
metadata:
  name: test
  namespace: default
spec:
  type: LoadBalancer
  selector:
    app.kubernetes.io/name: test
  ports:
  - name: web
    port: 8080
    protocol: TCP
    targetPort: 8080
EOF
service/test created

brandond@dev01:~$ kubectl get pod -n kube-system -l svccontroller.k3s.cattle.io/svcnamespace=default,svccontroller.k3s.cattle.io/svcname=test
NAME                        READY   STATUS    RESTARTS   AGE
svclb-test-31792a59-p4mw2   1/1     Running   0          26s

brandond@dev01:~$ kubectl delete service -n default test
service "test" deleted

brandond@dev01:~$ kubectl get service -n default test
Error from server (NotFound): services "test" not found

brandond@dev01:~$ kubectl get pod -n kube-system -l svccontroller.k3s.cattle.io/svcnamespace=default,svccontroller.k3s.cattle.io/svcname=test
NAME                        READY   STATUS    RESTARTS   AGE
svclb-test-31792a59-p4mw2   1/1     Running   0          59s

This was reported on Rancher Users slack: https://rancher-users.slack.com/archives/C3ASABBD1/p1657304386040799

We are using the KlipperLB load balancer and are finding that the svclb services are getting created in the kube-system namespaces.  In addition, we can not delete the pods in that namespace - the initial pod is deleted, but then it gets recreated, so it won't die.
Initial pods in kube-system namespace:

k get pods -n kube-system
NAME                                                        READY   STATUS    RESTARTS       AGE
coredns-d76bd69b-bnx9m                                      1/1     Running   3 (9d ago)     9d
metrics-server-7cd5fcb6b7-gflch                             1/1     Running   3 (9d ago)     9d
local-path-provisioner-6c79684f77-hlwsg                     1/1     Running   5 (9d ago)     9d
svclb-rook-ceph-mgr-dashboard-loadbalancer-1364e2b9-k7wsp   1/1     Running   0              7d23h
svclb-rook-ceph-mgr-dashboard-loadbalancer-1364e2b9-df8cj   1/1     Running   0              7d23h
svclb-rook-ceph-mgr-dashboard-loadbalancer-1364e2b9-sxb88   1/1     Running   0              7d23h
svclb-rook-ceph-mgr-dashboard-loadbalancer-1364e2b9-7nqq6   1/1     Running   1 (7d6h ago)   7d23h
svclb-istio-ingressgateway-c9776a98-m6hzn                   3/3     Running   0              3d2h
svclb-istio-ingressgateway-c9776a98-kwvr7                   3/3     Running   0              3d2h
svclb-istio-ingressgateway-c9776a98-4jn4m                   3/3     Running   0              3d2h
svclb-istio-ingressgateway-c9776a98-hd4jw                   3/3     Running   0              3d2h
svclb-keycloak-fb0704bb-pnq85                               1/1     Running   0              2d23h
svclb-keycloak-fb0704bb-xfh2p                               1/1     Running   0              2d23h
svclb-keycloak-fb0704bb-llc4r                               1/1     Running   0              2d23h
svclb-keycloak-fb0704bb-cjx4v                               1/1     Running   0              2d23h
svclb-postgres-db-40a02dd4-7h7q7                            1/1     Running   0              2d22h
svclb-postgres-db-40a02dd4-svf68                            1/1     Running   0              2d22h
svclb-postgres-db-40a02dd4-cn25l                            1/1     Running   0              2d22h
svclb-postgres-db-40a02dd4-tskt4                            1/1     Running   0              2d22h
svclb-stackgres-restapi-e9fc8149-xqhgw                      0/1     Pending   0              23h
svclb-stackgres-restapi-e9fc8149-jp4cx                      0/1     Pending   0              23h
svclb-stackgres-restapi-e9fc8149-444fj                      0/1     Pending   0              23h
svclb-stackgres-restapi-e9fc8149-jxgjm                      0/1     Pending   0              23h
svclb-nginx-service-2fcd8743-zbxv6                          1/1     Running   0              3h9m
svclb-nginx-service-2fcd8743-dpw6q                          1/1     Running   0              3h9m
svclb-nginx-service-2fcd8743-hj4gk                          1/1     Running   0              3h9m
svclb-nginx-service-2fcd8743-dt5pn                          1/1     Running   0              3h9m
svclb-nginx-b1379761-dhw6w                                  1/1     Running   0              3h27m
svclb-nginx-b1379761-zcgnd                                  1/1     Running   0              3h27m
svclb-nginx-b1379761-czflw                                  1/1     Running   0              3h27m
svclb-nginx-b1379761-jbpmv                                  1/1     Running   0              3h27m
svclb-west2-service-9d3852cf-rjhfw                          1/1     Running   0              64m
svclb-west2-service-9d3852cf-9p29w                          1/1     Running   0              64m
svclb-west2-service-9d3852cf-t2s5h                          1/1     Running   0              64m
svclb-west2-service-9d3852cf-7z2xw                          1/1     Running   0              64m
svclb-test-service-ac1dfd76-6qj28                           1/1     Running   0              47m
svclb-test-service-ac1dfd76-ckfdz                           1/1     Running   0              47m
svclb-test-service-ac1dfd76-cmdgz                           1/1     Running   0              47m
svclb-test-service-ac1dfd76-47j66                           1/1     Running   0              47m
svclb-test-f79880dc-gxrcx                                   0/2     Pending   0              4m35s
svclb-test-f79880dc-l9b9k                                   0/2     Pending   0              4m35s
svclb-test-f79880dc-x2bcz                                   0/2     Pending   0              4m35s
svclb-test-f79880dc-zkwdz                                   0/2     Pending   0              4m35s
svclb-test-replicas-2fef0816-l2xfv                          0/2     Pending   0              4m35s
svclb-test-replicas-2fef0816-zctcp                          0/2     Pending   0              4m35s
svclb-test-replicas-2fef0816-lgj2v                          0/2     Pending   0              4m35s
svclb-test-replicas-2fef0816-gjgwv                          0/2     Pending   0              4m35s

After deleting a couple of pods:
k delete pods -n kube-system svclb-test-f79880dc-gxrcx svclb-test-f79880dc-l9b9k
pod "svclb-test-f79880dc-gxrcx" deleted
pod "svclb-test-f79880dc-l9b9k" deleted
(envgen)$ k get pods -n kube-system
NAME                                                        READY   STATUS    RESTARTS       AGE
coredns-d76bd69b-bnx9m                                      1/1     Running   3 (9d ago)     9d
metrics-server-7cd5fcb6b7-gflch                             1/1     Running   3 (9d ago)     9d
local-path-provisioner-6c79684f77-hlwsg                     1/1     Running   5 (9d ago)     9d
svclb-rook-ceph-mgr-dashboard-loadbalancer-1364e2b9-k7wsp   1/1     Running   0              7d23h
svclb-rook-ceph-mgr-dashboard-loadbalancer-1364e2b9-df8cj   1/1     Running   0              7d23h
svclb-rook-ceph-mgr-dashboard-loadbalancer-1364e2b9-sxb88   1/1     Running   0              7d23h
svclb-rook-ceph-mgr-dashboard-loadbalancer-1364e2b9-7nqq6   1/1     Running   1 (7d6h ago)   7d23h
svclb-istio-ingressgateway-c9776a98-m6hzn                   3/3     Running   0              3d2h
svclb-istio-ingressgateway-c9776a98-kwvr7                   3/3     Running   0              3d2h
svclb-istio-ingressgateway-c9776a98-4jn4m                   3/3     Running   0              3d2h
svclb-istio-ingressgateway-c9776a98-hd4jw                   3/3     Running   0              3d2h
svclb-keycloak-fb0704bb-pnq85                               1/1     Running   0              2d23h
svclb-keycloak-fb0704bb-xfh2p                               1/1     Running   0              2d23h
svclb-keycloak-fb0704bb-llc4r                               1/1     Running   0              2d23h
svclb-keycloak-fb0704bb-cjx4v                               1/1     Running   0              2d23h
svclb-postgres-db-40a02dd4-7h7q7                            1/1     Running   0              2d22h
svclb-postgres-db-40a02dd4-svf68                            1/1     Running   0              2d22h
svclb-postgres-db-40a02dd4-cn25l                            1/1     Running   0              2d22h
svclb-postgres-db-40a02dd4-tskt4                            1/1     Running   0              2d22h
svclb-stackgres-restapi-e9fc8149-xqhgw                      0/1     Pending   0              23h
svclb-stackgres-restapi-e9fc8149-jp4cx                      0/1     Pending   0              23h
svclb-stackgres-restapi-e9fc8149-444fj                      0/1     Pending   0              23h
svclb-stackgres-restapi-e9fc8149-jxgjm                      0/1     Pending   0              23h
svclb-nginx-service-2fcd8743-zbxv6                          1/1     Running   0              3h11m
svclb-nginx-service-2fcd8743-dpw6q                          1/1     Running   0              3h11m
svclb-nginx-service-2fcd8743-hj4gk                          1/1     Running   0              3h11m
svclb-nginx-service-2fcd8743-dt5pn                          1/1     Running   0              3h11m
svclb-nginx-b1379761-dhw6w                                  1/1     Running   0              3h29m
svclb-nginx-b1379761-zcgnd                                  1/1     Running   0              3h29m
svclb-nginx-b1379761-czflw                                  1/1     Running   0              3h29m
svclb-nginx-b1379761-jbpmv                                  1/1     Running   0              3h29m
svclb-west2-service-9d3852cf-rjhfw                          1/1     Running   0              66m
svclb-west2-service-9d3852cf-9p29w                          1/1     Running   0              66m
svclb-west2-service-9d3852cf-t2s5h                          1/1     Running   0              66m
svclb-west2-service-9d3852cf-7z2xw                          1/1     Running   0              66m
svclb-test-service-ac1dfd76-6qj28                           1/1     Running   0              48m
svclb-test-service-ac1dfd76-ckfdz                           1/1     Running   0              48m
svclb-test-service-ac1dfd76-cmdgz                           1/1     Running   0              48m
svclb-test-service-ac1dfd76-47j66                           1/1     Running   0              48m
svclb-test-f79880dc-x2bcz                                   0/2     Pending   0              6m23s
svclb-test-f79880dc-zkwdz                                   0/2     Pending   0              6m23s
svclb-test-replicas-2fef0816-l2xfv                          0/2     Pending   0              6m23s
svclb-test-replicas-2fef0816-zctcp                          0/2     Pending   0              6m23s
svclb-test-replicas-2fef0816-lgj2v                          0/2     Pending   0              6m23s
svclb-test-replicas-2fef0816-gjgwv                          0/2     Pending   0              6m23s
svclb-test-f79880dc-bfhcc                                   0/2     Pending   0              3s
svclb-test-f79880dc-vz52q                                   0/2     Pending   0              3s

See also same in KlipperLB github:  https://github.com/k3s-io/klipper-lb/issues/38
We tore down our entire cluster and reinstalled (1 master and 3 worker nodes).  Then we started nginx with the following configuration:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx
spec:
  selector:
    matchLabels:
      app: nginx
  replicas: 1
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - name: nginx
        image: nginx
        ports:
        - containerPort: 80
        volumeMounts:
            - name: nginx-conf
              mountPath: /etc/nginx/nginx.conf
              subPath: nginx.conf
              readOnly: true
            - name: nginx-media
              mountPath: /usr/share/nginx/media
              readOnly: false
      volumes:
      - name: nginx-conf
        configMap:
          name: nginx-conf
          items:
            - key: nginx.conf
              path: nginx.conf
      - name: nginx-media
        persistentVolumeClaim:
          claimName: nginx-pv-claim

---
apiVersion: v1
kind: Service
metadata:
  name: nginx
spec:
  type: LoadBalancer
  ports:
  - port: 9080
    protocol: TCP
    targetPort: 80
  selector:
    app: nginx

Review the different namespaces, we saw the following (as expected):

Then we executed a "k delete -f nginx-test.yaml" with the following results:
kubectl delete -f nginx-test.yaml
deployment.apps "nginx" deleted
service "nginx" deleted
kubectl get pods
No resources found in default namespace.
[root@sdie-2-pm-4 nginx-test]# kubectl get services
NAME         TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)   AGE
kubernetes   ClusterIP   <IP_ADDRESS>    <none>        443/TCP   164m
[root@sdie-2-pm-4 nginx-test]# kubectl get pods -n kube-system
NAME                                      READY   STATUS    RESTARTS   AGE
local-path-provisioner-6c79684f77-m8sfx   1/1     Running   0          164m
coredns-d76bd69b-47x8m                    1/1     Running   0          164m
metrics-server-7cd5fcb6b7-r7p8d           1/1     Running   0          164m
**svclb-nginx-8acfcf71-5f565                1/1     Running   0          110s
svclb-nginx-8acfcf71-hqmf6                1/1     Running   0          110s
svclb-nginx-8acfcf71-jhgpp                1/1     Running   0          110s
svclb-nginx-8acfcf71-q4z4q                1/1     Running   0          110s**

As a work-around, you can roll back your cluster to v1.24.1, which does not include the above-linked PR.

According to K3S software, we are using version 1.23.8
k3s --version
k3s version v1.23.8+k3s1 (53f2d4e7)
go version go1.17.5

Also, 1.24.1 is not available on github.

@brandond - I am able to download the 1.23.6_k3s1 by clicking on download tarball or zip icon.  I am not able to figure out the combination of how to install a specific tagged version.  I have tried setting the INSTALL_K3S_VERSION and that seems to be trying to get that version.  However, it is not found in the releases area.  I have also tried to work with INSTALL_K3S_CHANNEL_URL and INSTALL_K3S_SKIP_ENABLE without getting the software to install.
Please provide details on how to install tagged version that is not a release.

INSTALL_K3S_VERSION=v1.23.6+k3s1

Validated on master branch with commitid ffe72eecc4cb058ba8d4635450f99c88fd71b71c
Environment Details

Infrastructure

[x] Cloud (AWS)
[ ] Hosted

Node(s) CPU architecture, OS, and Version:

$ uname -a
Linux ip-172-31-46-97 5.15.0-1011-aws #14-Ubuntu SMP Wed Jun 1 20:54:22 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

Cluster Configuration:

1 server

Config.yaml:

N/A
Additional files

N/A
Testing Steps

Install k3s, I used the following EXEC arg during install: INSTALL_K3S_EXEC="server --write-kubeconfig-mode 644 --cluster-init --token secret"
After node is Ready and pods are all Running, deploy service:

kubectl apply -f - <<EOF
apiVersion: v1
kind: Service
metadata:
  name: test
  namespace: default
spec:
  type: LoadBalancer
  selector:
    app.kubernetes.io/name: test
  ports:
  - name: web
    port: 8080
    protocol: TCP
    targetPort: 8080
EOF

Ensure it is created and svclb pod is created and running:

$ kubectl get pod -n kube-system -l svccontroller.k3s.cattle.io/svcnamespace=default,svccontroller.k3s.cattle.io/svcname=test

$ kubectl get service -n default test

Delete service: kubectl delete service -n default test

Replication Results:

k3s version used for replication:

curl -sfL https://get.k3s.io | INSTALL_K3S_VERSION=v1.24.2+k3s2 INSTALL_K3S_EXEC="server --write-kubeconfig-mode 644 --cluster-init --token secret" sh -

# After deleting the service:
$ kubectl get service -n default test
Error from server (NotFound): services "test" not found

$ kubectl get pod -n kube-system -l svccontroller.k3s.cattle.io/svcnamespace=default,svccontroller.k3s.cattle.io/svcname=test
NAME                        READY   STATUS    RESTARTS   AGE
svclb-test-2d2d1b5e-msd8n   1/1     Running   0          57s

Validation Results:

k3s version used for validation:

curl -sfL https://get.k3s.io | INSTALL_K3S_COMMIT=ffe72eecc4cb058ba8d4635450f99c88fd71b71c INSTALL_K3S_EXEC="server --write-kubeconfig-mode 644 --cluster-init --token secret" sh -

# After deleting the service
$ kubectl get service -n default test
Error from server (NotFound): services "test" not found

$ kubectl get pod -n kube-system -l svccontroller.k3s.cattle.io/svcnamespace=default,svccontroller.k3s.cattle.io/svcname=test
No resources found in kube-system namespace.

As can be seen, the svclb pod is deleted now correctly.

I'm seeing this issue with k3s version v1.25.6+k3s1

@vxav please don't "me too" on old closed issues. Open a new issue and fill out the issue template and provide steps to reproduce.


throw error for failed use space execution

Codecov Report
Base: 74.09% // Head: 50.72% // Decreases project coverage by -23.36% :warning:

Coverage data is based on head (855f8b4) compared to base (f78db4c).
Patch coverage: 80.46% of modified lines in pull request are covered.

:exclamation: Current head 855f8b4 differs from pull request most recent head a08bdb0. Consider uploading reports for the commit a08bdb0 to get more accurate results

Additional details and impacted files
@@              Coverage Diff              @@
##             master      #63       +/-   ##
=============================================
- Coverage     74.09%   50.72%   -23.37%
+ Complexity      153      122       -31
=============================================
  Files            24       26        +2
  Lines          1594     1451      -143
  Branches        256      240       -16
=============================================
- Hits           1181      736      -445
- Misses          266      639      +373
+ Partials        147       76       -71

Impacted Files
Coverage Δ

...ala/com/vesoft/nebula/connector/NebulaConfig.scala
56.89% <ø> (ø)

...scala/com/vesoft/nebula/connector/NebulaEnum.scala
78.57% <ø> (ø)

...la/com/vesoft/nebula/connector/NebulaOptions.scala
0.00% <ø> (ø)

...cala/com/vesoft/nebula/connector/NebulaUtils.scala
59.70% <ø> (ø)

...a/com/vesoft/nebula/connector/PartitionUtils.scala
100.00% <ø> (ø)

...n/scala/com/vesoft/nebula/connector/Template.scala
0.00% <ø> (ø)

.../vesoft/nebula/connector/exception/Exception.scala
0.00% <ø> (ø)

...cala/com/vesoft/nebula/connector/ssl/SSLEnum.scala
100.00% <ø> (ø)

...om/vesoft/nebula/connector/ssl/SSLSignParams.scala
0.00% <ø> (ø)

...esoft/nebula/connector/writer/NebulaExecutor.scala
0.00% <ø> (ø)

... and 24 more

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

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


"react-native" in path, causing failed tests

I was the only one in the whole team who got failed tests. My project path was: /Users/matzie/Gimi/react-native/projects/veckopengen-app
Changed it to:
/Users/matzie/Gimi/vp
fixed it.
Just so you guys know :)

Which version of Jest was that?

@cpojer Oh sorry forgot to tell, it was v18.1.0

@Matzielab would you mind providing a repro or a PR with failing test?


Play console and dev console links broken

In the "Google Login test" scenario, step 31, Try clicking on the ‘open cloud console’ and ‘open play console’ links and make sure they work, does not work. Looks like the URLs have changed.
Both end up with
URL not found
We couldn't find what you were looking for. Try one of the links below.
Google Developers Console Home
Google Cloud Platform
Google Developers

at
https://console.developers.google.com/accountsettings?pli=1
https://console.developers.google.com/accountsettings

The play console link seems to still work.


Creating new table in dektrium/yii2-user

i need to create a separate table and want to add the needed fields in that table. how far it is possible? kindly help in this issue.

You can use migrations. https://github.com/yiisoft/yii2/blob/master/docs/guide/db-migrations.md


Add the two-level header to Manage (v2)

Context
This PR implements the lower-level grey nav bar in the prototype.
Changes proposed in this pull request

Add moj-frontend bits
Parameterize the PhaseBanner component so it can have its border removed when stacked on top of another element
Refactor/add NavigationItems#for_provider_account_nav and NavigationItems#for_primary_account_nav
touch it in a system spec

Link to Trello card
https://trello.com/c/OTojnjhu/2440-implement-the-new-two-level-header-for-manage
Things to check

[X] This code doesn't rely on migrations in the same Pull Request
[X] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[X] API release notes have been updated if necessary
[X] New environment variables have been added to the Azure config

Looks great. I would just question the ‘stacked’ class name and param name. If it's literally about removing the border then I think we can be on the nose about it. ‘app-phase-banner--noborder`.
If we want to be more semantic and generic then provider-phase-banner.

Great — thanks. I've made it app-phase-banner--no-border.

Just noticed the underline is required when we're on a non-primary nav page. Going to fix


Helm values not persisting correctly to sub-charts in charts directory

Output of helm version:
Client: &version.Version{SemVer:"v2.10.0", GitCommit:"9ad53aac42165a5fadc6c87be0dea6b115f93090", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.10.0", GitCommit:"9ad53aac42165a5fadc6c87be0dea6b115f93090", GitTreeState:"clean"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.1", GitCommit:"d4ab47518836c750f9949b9e0d387f20fb92260b", GitTreeState:"clean", BuildDate:"2018-04-13T22:29:03Z", GoVersion:"go1.9.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.10", GitCommit:"098570796b32895c38a9a1c9286425fb1ececa18", GitTreeState:"clean", BuildDate:"2018-08-02T17:11:51Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.): AWS
So I have 2 cron jobs, with init containers to delay the start of the job by 5 seconds. I'm trying to put in resource requests/limits.
job1initContainer(kube-logging/charts/es-delete-indices/templates/cronjob.yaml)
          initContainers:
          - name: {{ template "name" . }}-es-delete-init
            image: alpine:3.6
            resources:
{{ toYaml .Values.initContainer.need | indent 14 }}
            securityContext:
              runAsUser: 500
              allowPrivilegeEscalation: false
            command: ["sleep", "5"]

job2initContainer(kube-logging/charts/es-snapshot/templates/cronjob.yaml)
          initContainers:
          - name: {{ template "name" . }}-es-snapshot-init
            image: alpine:3.6
            resources:
{{ toYaml .Values.initContainer.need | indent 14 }}
            securityContext:
              runAsUser: 500
              allowPrivilegeEscalation: false
            command: ["sleep", "5"]

I have a values file that exists outside of the charts directory, and should be taken in by all applications and jobs within.
initContainer:
  need:
    limits:
      cpu: 25m
      memory: 10Mi
    requests:
      cpu: 10m
      memory: 10Mi

job1/job2 initcontainers come up with null resources.
Error:
Error: UPGRADE FAILED: render error in "kube-logging/charts/es-delete-indices/templates/cronjob.yaml": template: kube-logging/charts/es-delete-indices/templates/cronjob.yaml:31:17: executing "kube-logging/charts/es-delete-indices/templates/cronjob.yaml" at <.Values.initContaine...>: can't evaluate field need in type interface {}
However, if I put this into job1's values file, it works fine:
initContainer:
  resources:
    limits:
      cpu: 25m
      memory: 10Mi
    requests:
      cpu: 10m
      memory: 10Mi

I was tinkering with the names of the attributes because there are "resources" defined, but don't have the parent attribute of initContainer in those values files. Note how the attributes put into job1's values file don't match up with the actual attributes, yet this passes through fine. I don't believe this should even be able to be run however, because the sub charts name aren't designated in the parent charts values file.

Delete jobs dry run: (job2 doesn't get deployed at all)
spec:
initContainers:
- name: es-delete-indices-es-delete-init
image: alpine:3.6
resources:
null
        securityContext:
          runAsUser: 500
          allowPrivilegeEscalation: false
        command: ["sleep", "5"]

Closing this as I see it is now a design decision behind helm


IPROTO-26 Add support for java.time.Instant

https://issues.jboss.org/browse/IPROTO-26
https://issues.jboss.org/browse/IPROTO-23
https://issues.jboss.org/browse/IPROTO-24
https://issues.jboss.org/browse/IPROTO-25
https://issues.jboss.org/browse/IPROTO-27

Integrated in master.


Implementing Add to Cart functionality

@khushi-joshi-05 I would like to work on the following tasks as part of GSSOC'24.

Implement cart.html (which is currently empty) to display menu items that are added to the cart
Ensure that when the "Order Now" button is pressed on the menu item card, the item gets added to the cart.
Include remove from cart option in the cart page
Display bill total
Place order button

@A-b-i-r-a-m-i-G-S I have assigned you the issue. Go ahead . All the best.


power rename can`t work

Microsoft PowerToys version
0.76.2
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
PowerRename
Steps to reproduce
Before it can`t work , it flashback
Now power rename disappered from the right click menu
Have checked the integration shell on the powertoys and PC options
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/31547


Not getting QR code to scan in extension

Browser - Chrome - Version 68.0.3440.106 (Official Build) (64-bit)
Attached screenshot for reference
Just a small rectangle opens when I click the extension icon. Also, not able to enable notifications by clicking the blue button on the page

Hello @pranay01, this probably happened because you blocked notifications the first time you clicked on the blue button. Please go to chrome://settings/content/notifications and check if the Safe Browser Extension is blacklisted. If that happens, remove it from there. Now open the extension again, click the blue button and accept notifications permission.
I hope after these steps you can start using the extension.
Please, tell me if this works for you. I will update this flow to make it more intuitive.

Don't see Safe browser extension in the blocked section in notifications section.

Have you closed a couple of times the popup that appears after clicking the blue button?

The popup never appeared for me. I guess that may be because I have some ad blockers like Ghostery and AdBlock plus

Hi @pranay01, could you clear you browser data and try again? You can do it here: chrome://settings/clearBrowserData.
Thank you


Foxx route tagging

Is it possible to organise endpoints in FOXX service API tab?
I have more then 50 endpoints in one service and it will be cool to separate endpoints by categories.
For example:
Default
GET /schemes_list
GET /schemes_list2
...
...
...
Schemes
DELETE /schemes/{name}
GET /schemes/{name}
POST /schemes/{name}
....
...
Nodes
GET /db/list
GET /db/node/{id}
POST /db/node/{id}
....
...
Other category
....
...
...

Hm, doesn't sound like a Microservice anymore? ;-)
If you have a look at the arangodb swagger documentation, these are also combined by topic, so its generally possible.

I have about 10 collections and I don't think that it it too many :)
And for every collection I need CRUD - 4 operations(endpoints). And 10*4=40 endpoints and it is not a big huge service, it is Microservice I think))
I don't think that it is good idea to separate this to 10 different microservices because collections is interrelated.
I tried to find at the arangodb swagger documentation something like topics but found only .summury(), .description() and etc. It is doesn't solve my problem.
P.S. Sorry for my bad english(

n/p - not a native speaker neither.
the ArangoDB API Documentation uses the 'tags' feature to group the different topics to sub topics:
 "/_admin/shutdown": {
            "delete": {
                "description": "\n\nThis call initiates a clean shutdown sequence.\n\n",
                "parameters": [],
                "responses": {
                    "200": {
                        "description": "is returned in all cases.\n\n\n"
                    }
                },
                "summary": " Initiate shutdown sequence",
                "tags": [
                    "Administration"
                ],
                "x-examples": [],

@pluma is there a way to achieve this using foxx?

Give the 3.2 alpha a spin. Foxx now supports the .tag method on endpoints. I think we sneakily added the support for this in one of the recent 3.1 releases but it's officially supported in 3.2.

Oops, just realised this isn't documented yet.

Wow) So fast response and solution.
I go to try install 3.2.alpha2 )))

@caracal7 sorry for the confusion.
The feature is available in 3.1.10 and later. So you don't need to upgrade to the 3.2 alpha just to use it.
You can try out the alpha if you want but beware that alpha releases are expected to be unstable and break in various exciting ways.

@pluma Thanks a lot
I have setup new server with 3.1.12. It working exactly as I need)))


Accordion.Title index prop type

https://github.com/Semantic-Org/Semantic-UI-React/blob/a740b36998401a7bdb6f86979443c022343411ee/src/modules/Accordion/AccordionTitle.js#L37
The Accordion seems to function perfectly well if the index given is a string. We use uuids on the back-end for everything, so I used a uuid as the index without thinking about it. Is there anything wrong with changing the PropType to number or string?
index: PropTypes.oneOfType([
    PropTypes.string,
    PropTypes.number,
]),

I think changing to string or number is acceptable.  In the end, the string will be coerced to a number though, since it is actually an array index that is needed.  Strings are common the UI anyway as HTML control values are string based.
PRs welcome.

Great. I would have submitted a PR, but didn't know if there were any other changes necessary. Glad to see @bharatzvm got to it so quickly.


[fix][client] Fix conversion of TimestampMillisConversion has no effect when Jsr310Conversion enabled

Motivation
Fixes #15858
The conversion that is registered first is a higher priority than the registered later, so TimestampMillisConversion should not be registered after TimestampMicrosConversion.
Modifications
Improve avro conversion order of registration.
Verifying this change

[x] Make sure that the change passes the CI checks.

This change added tests and can be verified
Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

good work! can you change the title explaining the issue ? improve avro conversion order of registration seems an improvement and it doesn't talk about the timestamp problem

ok, I changed the title


Project Updates

Summary of Changes

Added .gitignore to exclude unnecessary files from version control. Closes #1
Added documentation files: CONTRIBUTING.md, CONTRIBUTORS.md, and CODE_OF_CONDUCT.md. Closes #2
Added type hints to SQLAxe and logging functions for better type safety and readability. Closes #3
Fixed syntax issue in test_grep to ensure all tests pass. Closes #4
Introduced an events log file to the logger module for better logging. Closes #5
Created a data structure to hold raw SQL files for testing purposes. Closes #6
Updated setup.py and setup.cfg to reflect new dependencies and configuration. Closes #7
Updated requirements.txt with the latest dependencies. Closes #8
Updated README.md to include recent changes and new instructions. Closes #9

Looks good.


The kafka/kineses sampler route should have the ability to list topics/streams

Description
Right now in the console when selecting Kafka ingestion you then need to enter the bootstrap servers and type in the topic.

It would be really cool if the user could enter the bootstrap servers and get a listing of topic on those servers saving them from having to type it in.
Motivation
This would remove a keyboard step from the console which is potentially a source of mistakes (typo in topic name) or slowdown (forgetting the topic name and having to look it up).

Hi @vogievetsky i would like to work on this feature. Can i start working on it?


Test Bookshelf queries

Write tests for the Bookshelf class.
It should pass the following:

return the right title searched for
return results by author
return results by keyword and perhaps phrase
return null if nothing is found.

@Metalaxe1 mentioning you for discussion.


The /burp/scanner/status API still returns "scanPercentage": 0 after the scans are complete

Describe the bug
The /burp/scanner/status API still returns "scanPercentage": 0 after the scans are complete
To Reproduce
Steps to reproduce the behavior:

Use the "/burp/scanner/scans/active" API to trigger the scan on target url
Check the UI to see if the scans are finished
Once the active scan and passive scan is complete, use the "/burp/scanner/status" api
The response from status API is still "scanPercentage": 0

Expected behavior
The scan status API should return "scanPercentage": 100
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows
Burp Suite Professional  v2020.2
Extension burp-rest-api 2.0.1

Additional context
Add any other context about the problem here.

Adding screenshots

This looks like a bug. We're not actively supporting this extension at this time, but will try to take a look at some point

getPercentageComplete() in IScanQueueItem has been deprecated by Burp Suite.
We've pushed a change to simulate the 100% once the scan is either canceled, finished or abandoned. This should at least avoid breaks in CI/CD pipelines.


Rubocop anime controller

This PR add some changes to rubocop rules and refactor the Anime Controller.
Rubocop rules removed:

Rails/HasAndBelongsToMany because we use habtm associations a lot
Rails/TimeZone to stop receiving errors about time.zone, since we are all on
Some rules were removed because we use Code Climate to handle that:

Metrics/CyclomaticComplexity
Metrics/PerceivedComplexity
Metrics/AbcSize
Metrics/MethodLength

Done!

'oldest' string fixed and commits squashed :)


[stable/filebeat] Allow to add more secrets

What this PR does / why we need it:
Allow to add an then mount secrets
For example certficates
Checklist

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @sstarcher

/ok-to-test

/lgtm


[Backport 2.16] Implement write and read flow for shard diff file.

Description
[Backport] Implement write and read flow for shard diff file #14684
Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[ ] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

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.

Flaky test: https://github.com/opensearch-project/OpenSearch/issues/1703


Test also OCaml 4.07 and 4.08 versions

Related to https://github.com/alavrik/piqi/issues/62

Ok. now it is green. Please merge.

merged. thank you!


Flutter Web

Does this support Flutter Web or just any web app ? Not clear from readme

It should work, it’s just plain Dart. There’s no Flutter-specific support, but since it’s generic, I don’t really think it needs it.


.github: Run tests in fedora 37 as well

And remove soon to be deprecated fedora 35

Created new one since I don't have permission to rebase the one by @jjardon


Brightness drops a little bit when refresh rate switches.

When the refresh rate switches to 60 hz the brightness of the display lowers just slightly but still noticeable and when it's at 120 brightness looks normal. This is most noticeable at lower brightness and on a black background.

New feature added in dynamic_fps: Disable if the screen brightness is low


Unable to read back bits set in pilosa

I 'm running a pilosa docker container: https://hub.docker.com/u/pilosa/ on my OSX host.
When running the code on the your github page from the quick overview I do not retrieve any results back.
However, I can see in the web-ui of pilosa that the bit 42 is set on row 5 of myindex, myframe.
The code itself is not capable of reading the bit back out. It does not return an error but rather empty result arrays:
go run cmd/test2/main.go
Got bits:  []
[]
[]

I have the same behaviour in the real application i'm developing. I can set bits, retrieve them from the webui, but simple Bitmap queries from code do not work.

@maartenheremans Were you using the Go client from the master branch? There's a pretty big change coming to the Pilosa server, and master of Go client was modified to work with that. We've just merged a change to master that makes the Go client work with both the current Pilosa and the new. Can you give it a try with the latest Go client on master?

@maartenheremans Thank you for the report.
It appears that gopilosa master is not compatible with v0.8.8 of Pilosa (which you get with the docker image). This unfortunately happens occasionally as we evolve APIs in our pre-1.0 state.
You have two options to work around this:

Use the master branch of Pilosa (building from source is fairly straightforward), and running the built binary is as easy as pilosa server.
checkout version 0.8.0 of go-pilosa and use that. git checkout v0.8.0. Depending on what you're using for dependency management, you may wish to pin go-pilosa to this revision in Gopkg.toml or similar.

If you're just experimenting with Pilosa and want to track the latest and greatest features, I recommend route 1. If you need to depend on Pilosa in production, I recommend the second route, and you can wait to upgrade until we have tagged 0.9 releases of both Pilosa and the clients.

Ah I didn't see @yuce's message before I posted. You're welcome to try pulling the latest go-pilosa and using that, but I would actually recommend you follow one of the two paths I outlined as go-pilosa:master and pilosa:v0.8 (or whatever the last release is) are not guaranteed to stay compatible in general.

Thanks for the quick reply.
Since we are building production code I assume path 2 is the best. Our build pipeline however relies on 'go get'. Is there a versioned package available via gopkg.in ?

We'd recommend using dep, but assuming that's not possible we can look into supporting gopkg.in.

@yuce
It now gives a proper error indicating a version mismatch.
I believe this is a good solution.
2018/02/27 21:21:40 Pilosa server's version is 0.8.8, does not meet the minimum required for this version of the client: >=0.9.0.

@maartenheremans We probably need to change/remove that message. It just tells that the client is now working in "legacy" mode (Pilosa <= 0.9). But everything should still work. You shouldn't be getting empty results now.

@maartenheremans Does using the master branch of the client solve the empty results problem? Can we close this issue?

@yuce
No it does not solve the problem.
The following is with the 0.8.8 pilosa version:
2018/02/28 21:12:08 Pilosa server's version is 0.8.8, does not meet the minimum required for this version of the client: >=0.9.0.
Got bits:  []
[]
[]

However, with the latest build of pilosa (build manually, ignore the 0.0.0 version):
2018/02/28 21:14:02 Pilosa server's version is 0.0.0, does not meet the minimum required for this version of the client: >=0.9.0.
Got bits:  [42]
[42]
[]

That log is just a warning, it doesn't affect your results which I believe are correct. @yuce we should probably remove that log now that both versions are supported, right?

@maartenheremans The protobuf definitions in Pilosa server changed after v0.8.8 was released so current versions of Pilosa clients are not compatible with that version. We test our clients against both Pilosa master and Pilosa cluster-resize branch (which is going to become v0.9) and missed testing against v0.8.8 which as I mentoned has different protobuf definitions. The web UI uses JSON, so that change doesn't affect it.
As you mentioned, Go client on master is compatible with Pilosa on master. So you can use those together; or use Go client v0.8.0 and Pilosa v0.8.8 together. In that case a small change is required in your code: you will need to change result.Bitmap() to result.Bitmap. Please refer to the README of the Go client on v0.8.0 tag.
Thank you for bringing this issue to our attention and I am sorry for this error. I will remove Pilosa v0.8 compatibility badge and restore the original "incompatibility" message for the Go client.

@yuce Thanks again for picking this up so quickly. I can indeed continue for now with both a latest build of pilosa and the go-client.


Fixing sentiment analysis baseline

Fixes issue #9

thank you for your work. I manually addressed some part and merged. Feel free to leave your though if you still have concern. Thanks!


Minimize models without evaluating them

It occurred to me while fixing #26 that one reason evaluation is slow is that the models are large, and this is because they aren't yet minimized. However, minimization only requires knowing the universe cardinalities and not constructing the full Interpretations, and we can extract this very efficiently with some basic sexp parsing.
After minimization model evaluation should generally be much faster; for example the slow test we currently have involves a node sort with a universe of 78 elements, and evaluating a function of two nodes on about 6,000 inputs. Probably that universe could be of size 3 or 4.

Here's an alternative trick you can use to avoid needing to know the initial cardinalities: "minimize up" rather than "minimizing down". To minimize, you start by asking the solver if the formula is satisfiable if all universes have size 1. If not, try with all universes having size 2, and so on. Once you find a size that works, you can then "minimize down" to finish minimizing in a way that allows the universes to have different sizes from each other. This algorithm doesn't even need to call (get-model) much less parse the output. You could also speed it up by using "upward binary search" by doubling the size each time instead of incrementing by 1.


Issue UDP port loadbalancer

Hello!
I'm trying to install stunner on my kubernetes but i seeing that UPD service on my loadbalancer no found , Did anyone have this problem, how did you solve it?
kubectl describe svc udp-gateway -n stunner-system
Name:                     udp-gateway
Namespace:                stunner-system
Labels:                   stunner.l7mp.io/owned-by=stunner
                          stunner.l7mp.io/related-gateway-name=udp-gateway
                          stunner.l7mp.io/related-gateway-namespace=stunner-system
Annotations:              stunner.l7mp.io/related-gateway-name: stunner-system/udp-gateway
Selector:                 app=stunner,stunner.l7mp.io/related-gateway-name=udp-gateway,stunner.l7mp.io/related-gateway-namespace=stunner-system
Type:                     LoadBalancer
IP Family Policy:         SingleStack
IP Families:              IPv4
IP:                       <IP_ADDRESS>
IPs:                      <IP_ADDRESS>
Port:                     udp-listener  3478/UDP
TargetPort:               3478/UDP
NodePort:                 udp-listener  31325/UDP
Endpoints:                <IP_ADDRESS>:3478
Session Affinity:         None
External Traffic Policy:  Cluster
Events:
  Type     Reason                  Age                   From                Message
  ----     ------                  ----                  ----                -------
  Warning  SyncLoadBalancerFailed  22m (x19 over 88m)    service-controller  Error syncing load balancer: failed to ensure load balancer: only TCP protocol is supported, got: "UDP"
  Normal   EnsuringLoadBalancer    2m47s (x23 over 88m)  service-controller  Ensuring load balancer

Unfortunately, this sometimes happens with certain cloud providers. Which provider are you using? Note that STUNner can fall back to NodePort if UDP load-balancing is not available, did you try this?

Unfortunately, this sometimes happens with certain cloud providers. Which provider are you using? Note that STUNner can fall back to NodePort if UDP load-balancing is not available, did you try this?

Thanks for you response mate @levaitamas, i don't try because stunner make a work by himself create service, you can help me with some guide about this... Thank you in advance for your assistance

Check the generated services. LoadBalancer services are exposed via a NodePort (tipically in the range of 30000-32767) too. You can use this port to connect. Here is a short summary on how to get the nodeport of a lb service: https://sysdig.com/blog/kubernetes-services-clusterip-nodeport-loadbalancer/
Still interested in that which cloud provider are you using? Some providers require some extra annotations for UDP-based LB services. Hopefully, this trick can work in your setup too.

Check the generated services. LoadBalancer services are exposed via a NodePort (tipically in the range of 30000-32767) too. You can use this port to connect. Here is a short summary on how to get the nodeport of a lb service: https://sysdig.com/blog/kubernetes-services-clusterip-nodeport-loadbalancer/
Still interested in that which cloud provider are you using? Some providers require some extra annotations for UDP-based LB services. Hopefully, this trick can work in your setup too.

Thank you @levaitamas i check and tell you ...

I have solved that on provider which did not support udp for mnaged loadbalancers but allowed that for manually added loadbalancers (openstack based provider), used such config
---
apiVersion: gateway.networking.k8s.io/v1
kind: Gateway
metadata:
  name: media-gateway
  namespace: {{ .Release.Namespace }}
  annotations:
    stunner.l7mp.io/service-type: NodePort
spec:
  gatewayClassName: stunner-gatewayclass
  addresses:
    - type: IPAddress
      value: **MANUAL.LB.ADDRESS.X**
  listeners:
    - name: udp-listener
      port: 3478
      protocol: TURN-UDP
---
apiVersion: v1
kind: Service
metadata:
  name: media-gateway
  namespace: {{ .Release.Namespace }}
spec:
  selector:
    app: stunner
  ports:
    - port: 3478
      nodePort: 30478
      protocol: UDP
      targetPort: 3478
      name: udp-listener
  type: NodePort

Then i have used some terraform to spin up manual loadbalancer pointing such UDP node service (External LoadBalancer => <nodeIp>:<nodePort>), uggly but works

I'm also facing the same issue on AWS. Have not yet tried @gtokarz approach, hoping that works.

Closing this for now. Feel free to reopen is something comes up.


Expanding to support other languages

I looked into making Prism work in a generic way in a similar plugin. The primary problem is shown below:
> Object.keys(prism.languages)
[ 'extend',
  'insertBefore',
  'DFS',
  'markup',
  'css',
  'clike',
  'javascript' ]

Prism actually supports a ton of languages. Unfortunately I cannot see an easy way to access them through Node. I am not entirely sure why the entry point looks like that. It's possible I'm missing something here.
One way to "solve" this would be to set up a package that contains the language definitions in an easy to consume format. Of course there would be extra maintenance as it's easy to get out of sync with the main project.
Ideas are welcome as I would love to solve this somehow.

I made a little package to work around the issue. It simply gives you a an object with the language definitions. As a bonus I included a JSON pattern as that's sort of important for web stuff. Later on that can be removed as the functionality gets merged to Prism.

Awesome! Want to send a PR to make it work? I'd imagine it's possible to extract lang from <code> element's class.

Yeah, it's a trivial PR now. I'll do that later once I have a bit of time to focus. :+1:

Should be fixed in 1.0.0.


Added ViolenceAgainstWomenData

Description
Added the data for violence against women and girls in different African, Asian, and South American countries, exploring the attitudes and perceived justifications given for committing acts of violence against women. The data also explores different sociodemographic groups that the respondents belong to, including: Education Level, Marital status, Employment, and Age group.

Issue Ticket Number
Fixes #174

Checklist:

[x] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[x] I have cross-checked that the dataset I'm uploading is not present in the project
[x] I have cross-checked that the dataset is present in the correct folder

Good one @yashg-116
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow these steps.

Star this repository. ⭐
SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .
Go to the profile section from the top right-hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard!
And don’t forget to share your achievement with your friends.

@nikhil25803
https://www.clueless.tech/profile/admin/110786907949178549507


Use labels instead of numbers in i18n

I changed so that we now can refer to inputFields as textual labels in the translation strings instead of positional numbers. In the last commit you can see some translations changed as well.

Sorry for the late response, I took a few days off from work.

How hard do you estimate it to be to link them together? I could do the corresponding changes so that the argument names also is part of the translation strings of the call-blocks. I'll try to read the code myself also.

I don't remember why I didn't do it when I linked the procedure/function names. I vaguely remember some issue with the Blockly version that was fixed in a later version but I can't tell exactly what the problem was. Anyway, I think it could be relatively easy to fix, although it might be a little boring. You should probably start by changing the block definitions to use a Blockly.FieldLabel instead of a hardcoded string for each arg label in the calling blocks. This would allow you to reference the labels by name and update their value whenever the corresponding argument is renamed. Then you should do something like this: https://github.com/GIRA/UziScript/blob/ed5ac2415d79dc7000452e489f7f326357883e50/web/ide/blocks.js#L2163-L2182
This function runs whenever the blockly workspace is updated. If the event corresponds to the renaming of a procedure, it will walk over the blocks looking for the fields that need to be renamed and it will update their value to the new name. I think you should probably write a function like this for the argument names. I don't think anything else is needed but I might be forgetting something.

What would your approach to this be?

In the smalltalk code there is a class called UziParseTreeValidator that validates that each node in the tree is valid. For instance, for the "variable node" it checks that the variable is defined in the scope. If an invalid node is found it throws a syntax error. I could specify that return nodes are only allowed inside procedures and return-with-value is only allowed inside functions. Of course, this will be an incomplete solution because we should also change the UI to forbid this invalid programs in the first place. I don't know how to do that using blockly yet, though, but I assume it won't be very hard.

I have been thinking on the blocks' types as a way to prevent illegal combinations. Block types could also be reflected with the block colors. I have noticed the Boolean type blocks under Mathematics are correctly typed, but the color suggests they still are of type Number. Another mismatch seems to be with Pins, they seem to be Numbers which kind of makes sense for experienced users, but for someone new to the game it might be hard to understand what is the difference between number 1 and D1 and A1.

Yes, PLEASE review the block colors and fix them as you see fit. Thanks!


Add more section to make it more attractive #5

added FAQS on the page
https://github.com/user-attachments/assets/6ba5bfa9-e6a0-493b-a3ec-735e116ac55c

tell me what change i need to make

Please commit the change

Just commit suggestion the changes i want will be added to the pull request

@sahhoArjun097 you dont have to do anything just press on commit suggestion

laptop is dead that why i take time  to commit.


Invalid AMP due to broken style tag

Hello, from the WP you cannot access the code to modify parameters or metadata.
When sending the stories to Google Search Control, it tells me that they are not validated for AMP.
The problem is:
Only "style" tags with "amp-boilerplate" and "amp-custom" are allowed, and can only be placed in the document header.
Thank you very much, greetings

Hi @MeryEU, please share the URL to the story you've created so we can better assist you. It's likely that another plugin (e.g. a caching or performance plugin) of yours is interfering with Web Stories.

Yes,
https://www.cronicasdelaenfermedad.com/stories/cronicasdelaenfermedad/
I have plugin sg optimizer, asset cleanup
Thanks, this is my first story, there are more what the same problem.

Hi there
Please update to version 5.5.8 of SG Optimizer if you haven't already. That version fixes an incompatibility with Web Stories.
Looking at your story I don't see any AMP validation errors now, so it seems like you already updated and the issue is resolved!

Yes it true, thanks so much, is so cool stories


push pm5 code to poggit

push the pm5 code to poggit so the poggit ci can inject the virions this plugin needs to run

just wait.

just wait.

on what, its already updated to pm5 lol

You can use this link to download the latest PM5.0 phar https://github.com/KingOfTurkey38/Voting38/releases/tag/pm5


Current demo validates schema.org Event example differently between shex and shacl

Relaying from Aaron:

I pasted the first http://schema.org/Event example into the tool and it passed as error free with ShEx validation, but generated 4 errors with SHACL...

(I have verified this -- @danbri)
{
    "@context": "https://schema.org",
    "@type": "MusicGroup",
    "event": [
        {
            "@type": "Event",
            "location": "Memphis, TN, US",
            "offers": "ticketmaster.com/foofighters/may20-2011",
            "startDate": "2011-05-20",
            "url": "foo-fighters-may20-fedexforum"
        },
        {
            "@type": "Event",
            "location": "Council Bluffs, IA, US",
            "offers": "ticketmaster.com/foofighters/may23-2011",
            "startDate": "2011-05-23",
            "url": "foo-fighters-may23-midamericacenter"
        }
    ],
    "image": [
        "foofighters-1.jpg",
        "foofighters-2.jpg",
        "foofighters-3.jpg"
    ],
    "name": "Foo Fighters",
    "track": [
        {
            "@type": "MusicRecording",
            "audio": "foo-fighters-rope-play.html",
            "duration": "PT4M5S",
            "inAlbum": "foo-fighters-wasting-light.html",
            "interactionStatistic": {
                "@type": "InteractionCounter",
                "interactionType": "https://schema.org/ListenAction",
                "userInteractionCount": "14300"
            },
            "name": "Rope",
            "offers": "foo-fighters-rope-buy.html",
            "url": "foo-fighters-rope.html"
        },
        {
            "@type": "MusicRecording",
            "audio": "foo-fighters-everlong-play.html",
            "duration": "PT6M33S",
            "inAlbum": "foo-fighters-color-and-shape.html",
            "name": "Everlong",
            "interactionStatistic": {
                "@type": "InteractionCounter",
                "interactionType": "https://schema.org/ListenAction",
                "userInteractionCount": "11700"
            },
            "offers": "foo-fighters-everlong-buy.html",
            "url": "foo-fighters-everlong.html"
        }
    ],
    "subjectOf": {
        "@type": "VideoObject",
        "description": "Catch this exclusive interview with Dave Grohl and the Foo Fighters about their new album, Rope.",
        "duration": "PT1M33S",
        "name": "Interview with the Foo Fighters",
        "thumbnail": "foo-fighters-interview-thumb.jpg",
        "interactionStatistic": {
            "@type": "InteractionCounter",
            "interactionType": "https://schema.org/CommentAction",
            "userInteractionCount": "18"
        }
    }
}

.

@Gnomus042 any idea what is happening here?

@ericprud and I took a look. There is a similar issue for 1st sample too (recipe).
SHACL validation finds the interactionType property's value (http://schema.org/Comment) to fail.
This fixes the example:
  "interactionStatistic": {
    "@type": "InteractionCounter",
    "interactionType": { "@type": "ConsumeAction" },
    "userInteractionCount": "140"
  }, // ...etc

By contrast, ShEx validation doesn't complain.
This specific issue seems to trace to the ShEx-generation code that populates validation/shex/raw_shapes directory.
The SHACL and ShEx files generated from Schema.org's own definitions are organized in a type-centric way, so we find the generated rules for interactionType by looking for the type(s) that are schema:domainIncludes of this property.
In fact, this is also puzzling. The only type where interactionType is expected is Action, and yet neither SHACL nor ShEx complain, despite it being used on InteractionCounter, which isn't an Action subtype. This needs investigation.
Meanwhile, the values of interactionType: SHACL complains, ShEx doesn't.
Hypothesis...
In Action.shex in shex/raw_shapes we have this:
<#ValidSchemaAction> @<#ValidSchemaThing> AND EXTRA a {
	a [schema:MarryAction schema:IgnoreAction schema:EatAction schema:ConsumeAction schema:AssignAction schema:UseAction schema:ApplyAction schema:EndorseAction schema:ShareAction schema:FilmAction schema:ControlAction schema:CheckAction schema:DisagreeAction schema:TakeAction schema:DonateAction schema:CreateAction schema:ScheduleAction schema:ReactAction schema:InteractAction schema:ArriveAction schema:DeactivateAction schema:ReserveAction schema:PlanAction schema:BookmarkAction schema:ReviewAction schema:AcceptAction schema:CancelAction schema:BorrowAction schema:MoveAction schema:PaintAction schema:AddAction schema:ChooseAction schema:DiscoverAction schema:ViewAction schema:ExerciseAction schema:DrawAction schema:OrganizeAction schema:ReplaceAction schema:LikeAction schema:VoteAction schema:AchieveAction schema:InviteAction schema:CheckOutAction schema:PerformAction schema:ActivateAction schema:DrinkAction schema:DownloadAction schema:FollowAction schema:RegisterAction schema:TieAction schema:TravelAction schema:UnRegisterAction schema:DislikeAction schema:FindAction schema:SellAction schema:AskAction schema:ReceiveAction schema:AllocateAction schema:AssessAction schema:RejectAction schema:RentAction schema:RsvpAction schema:PlayAction schema:LoseAction schema:ResumeAction schema:TrackAction schema:ConfirmAction schema:SubscribeAction schema:WinAction schema:InformAction schema:WatchAction schema:UpdateAction schema:LendAction schema:OrderAction schema:ReturnAction schema:SendAction schema:AgreeAction schema:TipAction schema:LeaveAction schema:PrependAction schema:GiveAction schema:TransferAction schema:ListenAction schema:QuoteAction schema:JoinAction schema:PhotographAction schema:DepartAction schema:CommentAction schema:PreOrderAction schema:SuspendAction schema:DeleteAction schema:WantAction schema:AppendAction schema:InsertAction schema:CommunicateAction schema:WriteAction schema:BefriendAction schema:ReadAction schema:CookAction schema:SearchAction schema:AuthorizeAction schema:WearAction schema:CheckInAction schema:PayAction schema:ReplyAction schema:TradeAction schema:BuyAction schema:InstallAction schema:Action] * ;

If we replace that final '*' with a '+' it may fix this for ShEx.
Next steps

We need better unit tests that compare the engines

This could be in core/ rather than demo/ as it is foundational and UI-agnostic; but currently all knowledge of Schema.org is in demo, so we should probably handle testing the engine comparison there too.

The set of examples in the UI should be copied, snapshotted and frozen for unit testing. Perhaps simplified, and with variations that fix or break different things.
We should document some legitimate samples where the two validators behave differently, and have tests that will catch other regressions.


Breaking API Change

https://github.com/satori/go.uuid/commit/0ef6afb2f6cdd6cdaeee3885a95099c63f18fc8c breaks the API that's been stable for years. Why is this important enough to change with no warning, despite the community standard to be avoiding breaking API changes without some kind of versioning system? The original issue indicates some sort of deprecation warning was considered, but seems to never have been implemented. If users are going to have to rug pulled out from under them with no warning, why should we use this library?

@satori or anyone else, will you please cut a new tag for the master branch? Please! I’m sick of having to tell everyone that tries to use a package manager that they need to pin to the master branch. Cut a new tag and then close this issue. Code changes, we get it, we’ve all updated, but you’re still causing a world of hurt because no one can take 10 seconds and cut a new tag. Add me to the repo and I’ll do it, if no one else can be bothered with it. This ticket has been open for a month now. Cut a new release so the community can move on.

I was bitten by this issue today as well. Please add a 2.0.0 tag to the current master as @markbates suggests.

Like so:
[[override]]
  name = "github.com/satori/go.uuid"
  revision = "master"

Buullllllshhiiiiiitttttttt

@mattwilliamson that's not a helpful way to express frustration

@mattwilliamson that's not a helpful way to express frustration

For what it's worth -- when you have a project which people rely on - breaking the API in this manner warrants that level of response. It's egregious.

It's not like we are owed anything by @satori; as unpleasant as it is to break large parts of the ecosystem, there are ways around it.

It's not like we are owed anything by @satori; as unpleasant as it is to break large parts of the ecosystem, there are ways around the breakage.

very true -- but you need to follow semantic versioning at the very least. When you have users relying on your library -- you should at least consider that.

but you need to follow semantic versioning at the very least

It might be polite, but is not required, nor are we (as non-paying non-customers) entitled to it :-)

you should at least consider that

It's nice and useful, but again, adhering to it is merely politeness.

I'm very for this change. Took me less than 10 minutes to fix all of my projects with simple find and replace. I'm now no longer using recover() for these functions.

This comment is for those who are just starting out with the package and have not used it before.. when using the NewV4(), use it like so, to get rid of the "multiple-value uuid.NewV4() in single-value context"
id := uuid.Must(uuid.NewV4()).String()

It could have easily been done by adding a new function instead of breaking tons and tons of projects. I, myself have about 20 that were. Switching to github.com/google/uuid was faster and easier than implementing error checking.
 It's just a stinking random byte array. Fail to get MAC address? Fail to get timestamp? At this point I would rather have the application panic and dump a trace, because something is catastrophically wrong at that point.

In rare cases, you can't read random bytes from crypto/rand.Reader.

Retry or return nil? Backwards compatibility is important stuff. I don't know why you would downplay it.

I'm not downplaying it. See earlier in the thread where I described how I maintain a fork now. You asked why it could return an error and I told you.

Some members of the Go community are now maintaining a fork of this repo. We've just cut a v2.0.0 which is for the breaking change. Our fork keeps v1.2.0 available, if you still need it:

https://github.com/gofrs/uuid


Add Ukrainian translation

Add Ukrainian translation

@all-contributors please add @captain-yossarian for translation.


Regarding ngx-quill: 'formcontrol' is not defined in FieldQuillType

Hi,
I have integrated the ngx-quill into the project, but I'm unable to get the form control value. I'm getting formcontrol is not defined in FieldQuillType.ts.
https://stackblitz.com/edit/angular-elwtix-tmme97?file=src%2Fapp%2Fapp.module.ts  is the code which I have integrated into the project. I'm using angular 9 version. Please, help me out!!
Thanks.

A reproduction example is needed here also make sure you're using the latest version of Formly!

I'm using the latest version of formly too. The thing is, it is working in stackblitz but when integrated in project I'm getting the following error,
TypeError: Cannot read property 'formControl' of undefined at FieldQuillType.get [as formControl]

Well, I think you're trying to access to the field too early (constructor), where the field is not defined yet.
make sure the field is defined before calling formControl


Update pkg/term to 1.2.0

This fixes build errors on FreeBSD. Ran go mod tidy to clean up unused dependencies.
Term package has fixed FreeBSD build errors recently https://github.com/pkg/term/pull/61

Do you plan on making a new release soonish?

Sorry for the delayed response. I've just pushed v0.2.6 tag.

Great! Thanks a lot!

Like!


Add TTL support to mongodb state store

Description
Adds TTL support to the mongodb state store using the first class TTL document index functionality.
Adds appropriate tests to cover the TTL. FYI, mongodb runs a TTL garbage collector every 60s, so a value may live past its TTL for a maximum of 60 seconds.
Part of https://github.com/dapr/dapr/issues/5899 and https://github.com/dapr/components-contrib/issues/2288
Here is the commit to add to the docs, just need to wait for the v1.11 branch to become available first.

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Thanks for this PR!
Left some comments on the code. As for this:

FYI, mongodb runs a TTL garbage collector every 60s, so a value may live past its TTL for a maximum of 60 seconds.

I think we should make sure that when the TTL is passed, the value is not returned to the app. How other components do it, after they Get a value, they check the TTL and if it's past expiration, they just return a "not found" to the caller.
(And in this case too we should use the clock of the DB server and not the app server)=

@ItalyPaleAle Although I have resolved or said done to the above, I haven't pushed yet.
I agree having the TTL check on gets is a good idea to get around the 60s problem.. I am just currently pulling my hair out trying to get $$NOW to work in the new filter :sweat_smile:

@ItalyPaleAle what a headache. Got there in the end :slightly_smiling_face:
I have updated the test to ensure that 1. state it not returned soon after TTL, and 2. the data is actually deleted from the underlying state store to prevent regression.
PTAL!

@JoshVanL please fix the linter issues: https://github.com/dapr/components-contrib/actions/runs/4206220094/jobs/7299567920

@ItalyPaleAle done

/ok-to-test

Good catch!

/ok-to-test

FYI - this PR seems to have create an issue when using TTL with Mongo against Azure CosmosDB.


[codebuild] use dockerhub login

Motivation
Prevent codebuid from breaking due to dockerhub api requests.
Have you read the [Contributing Guidelines on pull requests]
Yes
Test Plan
CI
Related PRs
None

/canary

:broken_heart: Test Failed - Build images and run cluster test

/canary

:broken_heart: Test Failed - Build images and run cluster test

/canary

:sunny: Canary successful

/land


Create ioz.txt

增加中国科学院大学动物研究所

@goodluckxll I am sorry, JetBrains does not generally grant free educational licenses to organizations which are not educational, but belong to Academies of sciences. If you are a PhD student there, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).


Web app refresh 🍉

This is a complete rewrite of the provisioning web app for jambonz. The best place right now to grock the scope is the new contributor readme on this refresh branch here:
https://github.com/jambonz/jambonz-webapp/blob/refresh/docs/contributors.md
List of issues to complete functional parity on this branch:
https://github.com/jambonz/jambonz-webapp/issues?q=is%3Aissue+is%3Aopen+Refresh
Final checklist before merge to main:

[ ] Update pr-checks workflow branches
[ ] Update branch protection for main to require the pr-checks

Hey, I have been looking around the thing and I really like it. Lots of things got chopped down to smaller pieces so it is easier to look around. Lots of (self) documentation too so that will help me a lot with on boarding this.

Hey, I have been looking around the thing and I really like it. Lots of things got chopped down to smaller pieces so it is easier to look around. Lots of (self) documentation too so that will help me a lot with on boarding this.

Awesome! Glad to hear that. I definitely have put a lot into making sure this is the case! I considered jsdocs as a requirement via linting but decided against it since it’s a burden on developers and ideally we write the code well enough it can be understood easily. I think typescript is also helping in this sense to articulate code clarity. I think longer term having test coverage would be good but also 🤔. For next steps I’m going to create issues on this repo for the remaining work with acceptance criteria based on parity of function. From there we can work out where it makes the most sense to start for you but also feel free to just dive in where you want once we have issues to tackle 🤘

@khanghugo I was just playing around and found a runtime error on the applications view when you don't yet have any accounts for the current service provider. I tested on the old webapp and they just redirect you to accounts with a toast when you try to create an application but there are no accounts. I thought we could handle that redirect scenario as well as a last ditch fallback but also provide more up front UI to help the user understand they need to first create an account. This way you are not surprised by a redirect because the applications view provides messaging directing you to go create an account. Commit here: https://github.com/jambonz/jambonz-webapp/pull/64/commits/2201644df11e20ba34abc175196c27d480fa62bc

Woo-hoo


api name must only contain alphanumeric and '., -, _, ~' characters

I have a problem about the "Adding your API" section in the reference documentation.
I follow the steps using "curl" command to add my API.
I name my API with other language characters.
But I get the response as"name must only contain alphanumeric and '., -, _, ~' characters".
Can this function be able to handle non-alphanumeric characters such as Chinese characters?

@Deamrase , no, API name can not be Chinese characters.

I confirm @chenyukang 's answer. The api name is limited to ascii alphanumerics and the ones mentioned in the error message.


required(:title) presence check does not work as advertised

According to your API, the following should validate the presence of an object:
However, I get the following error:
module MyModule
  class CreateParamsValidator < ::MyCustomBaseParamsValidator
    define do
      required(:title).filled(:str?)

 #<Dry::Schema::Result{} errors={:title=>["is missing"]}>

I'm curious, has the API changed but not the documentation?

Could you please share a full example to reproduce this issue? It’s hard to ascertain where a fault may lie from your short snippets. Thanks!

Please provide a full reproducible example.

Hello, I was able to figure it out. Problem was I needed to do to_h. Thank you params.require("data").permit!.to_h


Bug: empty string serde and Kotlin in 2.8.2

I believe there is an issue in 2.8.2 when serializing an empty string to a non-null field in Kotlin, which is reproduced by this PR.
Instead of the expected validation error fruitCommand.name: must not be empty, with Kotlin we get:
Parameter specified as non-null is null: method example.FruitCommand.<init>, parameter name

This works in 2.8.1, and was exposed by the eclipsestore guide https://github.com/micronaut-projects/micronaut-guides/pull/1443#commits-pushed-0276ab1

Actually, I'm not sure if this reproducer is correct...
Going back to earlier commits, the Kotlin tests in this PR still seem to fail, and I'm not sure why...
However in the EclipseStore guide (check out micronaut-guides, and run ./gradlew mEPBuild), adding
configurations.all {
    resolutionStrategy.eachDependency { details ->
        if (details.requested.group == "io.micronaut.serde") {
            details.useVersion("2.8.1")
        }
    }
}

To the build of the kotlin guide does make it pass...

It looks correct to me; the default inclusion setting is NON_EMPTY. So, an empty string is omitted.
For Java we can initialize beans even with non-null but not for Kotlin.

@dstepanov did something change to make the old way invalid?  As far as I can see, this used to work

Not sure, maybe the configuration of the inclusion wasn't properly delegated?


Allow property overrides to configure etcd

Currently, the other etcd properties can be overridden in property_overrides. We need to also be able to override properties.etcd.machines and properties.diego.bbs.etcd.machines in the same way.

Thanks, @cholick! Are these overrides related to the consul datacenter override PR you also submitted?
Best,
Eric

They're related in that it was a set of changes to add enough flexibility to get the scripts to work with our installation, but the actual commit should be independent.
Feel free to close this one if you don't want to take it, as we ended up scrapping one approach and of the three pull requests, this is the one change we didn't need for the final working manifest. (Though someone else might need it)

Thanks for the extra context, @cholick. If you won't currently find this change valuable, we'll go ahead and close this PR out.


fix: removed unused files after nemesis block is generated.

This PR removes the nemesis block temp files after generation. We can then have deterministic nemesis seed folder generation and compare folders in unit tests (Future PR).

This PR removes the nemesis block temp files after generation.

Removes from where? Local machine or a deployed server?

Hi @Jaguar0625 , the issue description wasn't the best.
When user runs -p bootstrap, the nemgen creates all these files:

This PR is to clean up the files the catapult client doesn't require saving space and helping me with the unit test suite that it's coming

why are you not running nemgen with useTemporaryCacheDatabase option?

why are you not running nemgen with useTemporaryCacheDatabase option?

Thanks! I didn't know the option.


Ability to run executables

Please describe the problem you're trying to solve
I want to use yazi like I do File Explorer on Windows. It's almost perfect in that regard, but I can't run executables, trying to open them causes my web browser to open and try to download a file even though yazi recognizes that it's an ELF file. I'd probably add this myself as an opener, but there's no standardized extension for ELF files nor mimetype.
Would you be willing to contribute this feature?

[ ] Yes, I'll give it a shot

Describe the solution you'd like
Ability to run executables like an any other file.
Additional context
The closest function is "run a shell command" function but I'd like to just press enter or o and have it run the executable.

Hi, what's your Yazi version? Just tried in my Windows VM, with the default "Open" opener, I can get it to open the executable file:
https://github.com/sxyazi/yazi/blob/b10f2de16d46df3ed7f6efe99ac966fd49d6e919/yazi-config/preset/yazi.toml#L36
https://github.com/sxyazi/yazi/assets/17523360/28e663b5-8439-419a-88e3-84e7c3fb2722
My Yazi version:
yazi 0.2.4 (b10f2de 2024-03-11)

Hi, what's your Yazi version? Just tried in my Windows VM, with the default "Open" opener, I can get it to open the executable file:
https://github.com/sxyazi/yazi/blob/b10f2de16d46df3ed7f6efe99ac966fd49d6e919/yazi-config/preset/yazi.toml#L36
windows.open.mp4
My Yazi version:
yazi 0.2.4 (b10f2de 2024-03-11)

Hi, it's
yazi 0.2.4 (Arch Linux 2024-03-10)

Are you on Windows right? Why is Arch Linux included in the version?

It works, but with run in opener set to '"$@"', without xdg-open

Yeah that's how it works, just like when you directly execute it in the shell.
Nice to hear it's working; I think we can close this issue now.


dynamic-graph-python: 4.0.2-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository dynamic-graph-python to 4.0.2-1:

upstream repository: https://github.com/stack-of-tasks/dynamic-graph-python.git
release repository: https://github.com/stack-of-tasks/dynamic-graph-python-ros-release.git
distro file: noetic/distribution.yaml
bloom version: 0.10.1
previous version for package: 4.0.1-1

putting me as maintainer in package.xml, so I can receive errors from buildfarm.


[RPA-1668] Add sending decrypted credentials

Description

Motivation and Context

Types of changes

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

I agree to the terms of the RunBotics Contributor License Agreement.

todo:

add credentialId in the list
check if credential is primary based on the order in db
make credentials list required


Controls should not have superfluous help text in their accessible labels

Issue Type: Bug
CC: @isidorn

Multiple tree views such as the explorer view, the tree view rendering the results of a find in project etc all have this help text as part of the tree view's accessible label: "Use the navigation keys to navigate.".

Although not a bug strictly speaking, it doesn't really convey any additional useful information (the tree view role is sufficient). More significantly, the label is read in its entirety before NVDA starts to read the items inside the tree view when it gains focus, which is slightly annoying.
Hence, I would like to propose removing it.
If there're no objections, I'd like to take this opportunity to submit my first PR to Vs Code to fix this :)
VS Code version: Code - Insiders 1.46.0-insider (288852d8c26d797974bfb8aed06e1bff9b809223, 2020-05-12T05:43:31.363Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz (8 x 1992)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.86GB (3.68GB free)

Process Argv

Screen Reader
yes

VM
33%

Extensions (13)

Extension
Author (truncated)
Version

notes
can
1.0.0

vscode-markdownlint
Dav
0.35.1

vscode-eslint
dba
2.1.5

prettier-vscode
esb
4.7.0

rest-client
hum
0.23.2

python
ms-
2020.5.78807

remote-wsl
ms-
0.44.2

cpptools
ms-
0.28.0

Go
ms-
0.14.2

vsnotes
pat
0.7.1

rust
rus
0.7.8

markdown-preview-enhanced
shd
0.5.3

vscode-webhint
web
1.5.5

This is also applicable to the quick picker as well: "Quick Input. Use the navigation keys to navigate.  list." would read better if it just announced "Quick Input list"
Probably best to remove all references to this text project wide.

Great proposal but we actually already removed it, more details here https://github.com/microsoft/vscode/issues/97510
Thus closing as duplicate and if you update to latest insiders should be fixed.
As for the first PR, we can find some other area. Maybe the references list for enter and space moving focus / not moving focus. Let me know what you are interested in and I can provide pointers.
Also fyi we have channel for accessibility so consider joining it https://gitter.im/Microsoft/vscode-a11y

Wow, glad that I filed an issue first as I wouldn't have realized this was already worked on.

Maybe the references list for enter and space moving focus
Sure, that sounds like a manageable first PR :)


feature: import_files.sh should adaptive to the newest minio client

as title display, the import_files.sh should adaptive to the new minio client.

/kind feature

fix in #241


docker-compose up -d postgres raise error:Version in "./docker-compose.yml" is unsupported.

docker-compose up -d postgres
ERROR: Version in "./docker-compose.yml" is unsupported. Either specify a version of "2" (or "2.0") and place your service definitions under the `services` key, or omit the `version` key and place your service definitions at the root of the file to use version 1.
For more on the Compose file format versions, see https://docs.docker.com/compose/compose-file/

my env:
Docker version 17.03.0-ce, build 60ccb22
docker-compose version 1.7.0, build 0d7bf73

You need a newer version of docker-compose, at least 1.13.0.
Please refer(https://docs.docker.com/compose/install/ .


chore: Update Cloud Pak for Data to release 4.6.2

Is your feature request related to a problem? Please describe.
Cloud Pak for Data 4.6.2 was released, so we need to update this repository.
https://www.ibm.com/docs/en/cloud-paks/cp-data/4.6.x?topic=overview-whats-new#whats-new__refresh-2
Describe the solution you'd like
Update the CRs and jobs to reflect the updated installation procedures.
Describe alternatives you've considered
None, the requirement is pretty narrow.
Additional context
CP4D installation procedure shifted towards a procedural approach in release 4.5.x. See #173 for reference and corresponding PR with the changes to understand the nature of that change.

:tada: This issue has been resolved in version 0.17.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fix: add solidity ^0.5.0 support

UGH I only just noticed your PR after spending time and making it compatible with latest truffle/OZ/solidity. Thanks, closing to merge my PR which has some slight differences...


⚠ ClamAV has degraded performance

In 43c28bd, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 201 ms

Resolved: ClamAV performance has improved in 31c97ac.


AWS Glue Data Catalog Support

https://github.com/datafusion-contrib/datafusion-catalogprovider-glue


fix: filename with comma

Fix #658

@rofe does this also cover filenames with a . (dot) in it?

@rofe does this also cover filenames with a . (dot) in it?

@felix-staud Good catch! Fixed and added a test for that as well.

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

GitHub release
v6.40.3

Your semantic-release bot :package::rocket:


Error profiling Dev15 RC

This is a day or two old build of Dev15 RC with updated PTVS build (today's) using our install script.
I created a bottle app
I selected virtual env option (based on 3.5)
App works when I launch with F5
Launch Python profiling
Result in command prompt:
Traceback (most recent call last):
  File "C:\Program Files (x86)\Microsoft Visual Studio\Enterprise\Common7\IDE\Extensions\Microsoft\Python\Profiling\proflaun.py", line 35, in <module>
    vspyprof.profile(__file__, globals(), locals(), profdll)
  File "C:\Program Files (x86)\Microsoft Visual Studio\Enterprise\Common7\IDE\Extensions\Microsoft\Python\Profiling\vspyprof.py", line 65, in profile
    pyprofdll = ctypes.PyDLL(profdll)
  File "C:\Program Files\Python35\lib\ctypes\__init__.py", line 347, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: [WinError 126] The specified module could not be found
Press any key to continue . . .

If this is because I'm missing some VS component (when I installed VS i only picked Python workload), then we need to check for it early in the command handler, tell the user if it's missing, and I assume use IVsProjectAcquisitionSetupDriver  like done in https://github.com/Microsoft/PTVS/pull/1769.

No, that error means that our DLL is not there. Can you post a file listing of the Extensions\Microsoft\Python\Profiling directory?

extension.vsixmanifest
Microsoft.PythonTools.Profiling.dll
Microsoft.PythonTools.Profiling.pdb
Microsoft.PythonTools.Profiling.pkgdef
proflaun.py
Python Tools for VS License.rtf
PythonProject.ico
PythonProjectBig.ico
VsPyProf.dll
VsPyProf.pdb
vspyprof.py
VsPyProfX86.dll
VsPyProfX86.pdb

Profiling works fine on d15prerel build 25828.02 with data science workload on anaconda.


Add documentation

Using the wiki pages on GitHub, create documentation/user guide to explain how to use the language. Use little to no jargon, include examples, and keep it short and simple.

Refer to book for this.


SAK-45109: Webcomponents: Problem with save permissions

Explanation: If we have a role that contains, for example, a symbol sum. Ex: instructor+ when save permission an error is produced.


[MODORDERS-1026] - Test for api which execute template-request in mod…

…-template-engine
Purpose
https://folio-org.atlassian.net/browse/MODORDERS-1026
Approach


[api-minor] Reduce postMessage overhead, in PartialEvaluator.getTextContent, by sending text chunks in batches (issue 13962)

Following the STR in the issue, this patch reduces the number of PartialEvaluator.getTextContent-related postMessage-calls by approximately 65 percent.[1]
Note that by enforcing a relatively low value when batching text chunks, we should thus improve worst-case scenarios while not negatively affect all textLayer building.
While working on these changes I noticed, thanks to our unit-tests, that the implementation of the appendEOL function unfortunately means that the number and contents of the textItems could actually be affected by the particular chunking used.
That seems extremely unfortunate, since in practice this means that the particular chunking used is thus observable through the API. Obviously that should be a completely internal implementation detail, which is why this patch also modifies appendEOL to mitigate that.[2]
Given that this patch adds a minimum batch size in enqueueChunk, there's obviously nothing preventing it from becoming a lot larger then the limit (depending e.g. on the PDF structure and the CPU load/speed).
While sending more text chunks at once isn't an issue in itself, it could become problematic at the main-thread during textLayer building. Note how both the PartialEvaluator and CanvasGraphics implementations utilize Date.now()-checks, to prevent long-running parsing/rendering from "hanging" the respective thread. In the textLayer building we don't utilize such a construction[3], and streaming of textContent is thus essentially acting as a simple stand-in for that functionality.
Hence why we want to avoid choosing a too large minimum batch size, since that could thus indirectly affect main-thread performance negatively.
Improves and/or fixes #13962

[1] While it'd be possible to go even lower, that'd likely require more invasive re-factoring/changes to the PartialEvaluator.getTextContent-code to ensure that the batches don't become too large.
[2] This should also, as far as I can tell, explain some of the regressions observed in the "enhance" text-selection tests back in PR #13257.
Looking closer at the appendEOL function it should potentially be changed even more, however that should probably not be done here.
[3] I'd really like to avoid implementing something like that for the textLayer building as well, given that it'd require adding a fair bit of complexity.

/botio-linux preview

/botio test

/botio-linux preview

Nice work!

@timvandermeij As always, thanks a lot for reviews :-)

*Off-topic: How do you feel about a new release soon, since there's been a fair number of api-minor changes recently and also new features such as the just landed /Interpolate-support (fixing many old issues)?


[WIP] Assert integer ranges

This is a proof-of-concept to add methods for integer ranges:
$variable = type($variable)->asPositiveInt();

First of all, do we want this? I'm thinking about methods like asNegativeInt(), asPositiveFloat(), asIntBetween(50, 100), etc.

I am not fan of those tbh. I would rather keep this only for native types.


Bug in KES period usage

The following test fails:
#[test]
    fn evolved_4() {
        let (mut skey, pkey) = Sum4Kes::keygen(&mut [0u8; 32]);
        let dummy_message = b"tilin";
        let sigma = skey.sign(12, dummy_message).expect("Period is on-limits");

        assert!(sigma.verify(12, &pkey, dummy_message).is_ok());
    }

while the following passes:
#[test]
    fn evolved_4() {
        let (mut skey, pkey) = Sum4Kes::keygen(&mut [0u8; 32]);
        let dummy_message = b"tilin";
        let sigma = skey.sign(12, dummy_message).expect("Period is on-limits");

        assert!(sigma.verify(0, &pkey, dummy_message).is_ok());
    }

The period in verify matches with the number of evolutions, and the kes period in signing seems useless. E.g., the following passes:
#[test]
    fn buff_4() {
        let (mut skey, pkey) = Sum4Kes::keygen(&mut [0u8; 32]);
        let dummy_message = b"tilin";

        // Key can be updated 2^4 - 1 times
        for period in 0..15 {
            assert!(skey.update(period).is_ok());
        }

        let sigma = skey.sign(11, dummy_message).expect("Period is on-limits");
        sigma.verify(15, &pkey, dummy_message).unwrap();
    }

I'm closing this, as this seems to match the cardano node implementation. We will modify the API in accordance to the progress made there


New feature: aspect ratio

This feature needs to be implemented on the 2D projection of a morphology and it will describe the minimum over the maximum dimension in the projected plane i.e. x-y. We will therefore need to compute this in all three projections at the morphology level.
To extract the projection points from a morphology:
def get_points_projection(neurite, projection1='X', projection2='Y'):
    proj1 = neurite.points[:, getattr(COLS, projection1)]
    proj2 = neurite.points[:, getattr(COLS, projection2)]
    return np.array([proj1, proj2]).transpose()

Please find some sample code for this feature:
def aspect_ratio(pts):
    """Computes min dimension / max dimension"""
    max_dim = np.max(nm.morphmath.principal_direction_extent(pts))
    min_dim = np.min(nm.morphmath.principal_direction_extent(pts))
    return min_dim / max_dim

For testing purposes here are a few shapes:
TestData.zip
The expected results for these shapes:
aspect_ratio(circle) == 0.9838348383830106
aspect_ratio(square) == 0.9799193845949096
aspect_ratio(rectangle) == 0.33658158126782983
aspect_ratio(oval) == 0.49083100011838016

Using the convex hull points to avoid the point distribution to affect the principal directions, I get the following results:
In [4]: aspect_ratio(circle)
Out[4]: 0.9975026440270615

In [5]: aspect_ratio(square)
Out[5]: 0.9829836870510568

In [6]: aspect_ratio(rectangle)
Out[6]: 0.4130280313071019

In [7]: aspect_ratio(oval)
Out[7]: 0.493502615108013


Timeline Plot

Greetings!
I watched the episode about Gonum, but I don't understand how to create a time-based plot ..
I need to plot it:

2020/09/11,1336.44
2020/10/20,1936.44
2020/11/01,2005.11
2020/11/01,2005.24
2020/11/25,1994.25
2020/12/04,2594.25
2020/12/04,3194.25
2020/12/14,3185.49
2020/12/21,3172.35
2020/12/22,3342.35
2020/12/22,3172.35
2020/12/30,3161.34
2020/12/31,3011.34
2021/01/01,3078.34
2021/01/20,3038.34
2021/01/22,3028.34
2021/01/25,3328.34
2021/01/25,3628.34
2021/02/01,3758.34
2021/02/02,3733.26
2021/02/16,3709.38
2021/02/16,3859.38
2021/02/16,3709.38
2021/02/22,3673.38
2021/02/24,3619.38

I'm trying:
func readData(path string) (plotter.XYs, error) {
	f, err := os.Open(path)
	if err != nil {
		return nil, err
	}
	defer f.Close()

	var xys plotter.XYs
	s := bufio.NewScanner(f)
	for s.Scan() {
		var y float64
		var x float64

		_, err := fmt.Sscanf(s.Text(), "%f,%f", &x, &y)
		if err != nil {
			log.Printf("discarding bad data point %q: %v", s.Text(), err)
			continue
		}
		s := fmt.Sprintf("%f", x)
		date1, err := time.Parse("2006/02/01", s)
		date2, _ := strconv.Atoi(date1.String())
		x = float64(date2)
		xys = append(xys, struct{ X, Y float64 }{x, y})

		if err != nil {
			fmt.Println(err)
		}

	}
	if err := s.Err(); err != nil {
		return nil, fmt.Errorf("could not scan: %v", err)
	}
	return xys, nil
}

What I'm doing wrong?
Thanks in advance!

Wow, I'm sorry! This was sent three times I don't know why!


Search Indexes are not being copied for pre-parsed repos

When someone tries to parse a pre-processed repos, search indexes are not being copied.

Sure you can pick this up


is there a way to permit arguments as in gotty --permit-arguments

No, this option is not supported in ttyd.

Added support on master (--url-arg).


X509Certificate2 DNSNameList is not language independent, don't work on non-english clients

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
In any version of powershell create a X509Certificate2 object by using:
$certificate = New-Object System.Security.Cryptography.X509Certificates.X509Certificate2($path, $pwd)
Then query for the $certificate.DNSNameList property.
If you do the action on an english client you obtain the list of the SubjectAlternativeNames, if you do from another language (in my case, Italian client) the property is empty.
If you look at source file:

src/Microsoft.PowerShell.Security/security/CertificateProvider.cs

You can see that the property has a FIXED dns name prefix in english language
    public sealed class DnsNameProperty
    {
        private readonly List<DnsNameRepresentation> _dnsList = new();
        private readonly System.Globalization.IdnMapping idnMapping = new();

        private const string dnsNamePrefix = "DNS Name=";

and this prefix is used in the DNSNameList property to discard all that is not a DNS name
            foreach (X509Extension extension in cert.Extensions)
            {
                // Filter to the OID for Subject Alternative Name
                if (extension.Oid.Value == "<IP_ADDRESS>")
                {
                    string[] names = extension.Format(true).Split(Environment.NewLine);
                    foreach (string nameLine in names)
                    {
                        // Get the part after 'DNS Name='
                        if (nameLine.StartsWith(dnsNamePrefix, System.StringComparison.InvariantCultureIgnoreCase))
                        {

But in Italian language the prefix code you obtais is not 'DNS Name=" but "Nome DNS=" and so they are all discarded, resulting in an empty DNSNameList property.
Expected behavior
The expected behaviour is that the DNSNameList is populated the same way if the client is english or not english language.

Actual behavior
The actual behaviour is that the DNSNameList property is empty on non-english clients (for example, Italian client)

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.4.6
PSEdition                      Core
GitCommitId                    7.4.6
OS                             Microsoft Windows 10.0.26100
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Using EnumerateDnsNames does sound like the right fix.
Today, PowerShell is relying on parsing the extension as a string. On Windows by default, that string comes from the CngAsnFormatter, which itself the text from Windows' Crypt32 format function, and in Windows, IDS_DNS_NAME is a localized string which has a different value on a non-English system. And it's hypothesized above that this might even fail on an English system if OpenSslAsnFormatter.cs does not format the text in the same way as Crypt32 does.

Right. Parsing the output of the extension as a string is prone to failure. It depends on the OS locale, and other operating systems (Linux / OpenSSL, macOS) are going to output different things.
The EnumerateDnsNames was added exactly so that folks would not try to parse the output from stringifying the extension since it is brittle.
https://github.com/dotnet/runtime/issues/22699

Currently applications make programmatic decisions for it via ToString() which has a couple of problems:

It's localized on Windows
It's not the same on Windows and !Windows.

I guess usage would simply turn into:
var subjectAlternativeNameExtension = new X509SubjectAlternativeNameExtension(cert.RawData);

foreach (string dnsNameEntry in subjectAlternativeNameExtension.EnumerateDnsNames())
{
    // Add entries to DnsNameList property
}

Which looks pretty simple instead of going through all the extensions and doing DNS name prefix + OID SAN checking.

@ArmaanMcleod Yes, it is right way.

I guess workaround for now could be to just do:
($cert.Extensions | Where-Object {
    $_ -is [System.Security.Cryptography.X509Certificates.X509SubjectAlternativeNameExtension]
}).EnumerateDnsNames()

I'll see if I can create a PR soon to get the DnsNameList property populated.