Certificate issues

BUG REPORT
Environment:

vsphere

OS: Ubuntu 16.04

Version of Ansible: 2.6

Kubespray version: 846c7a26e81340055c659ea0b895b74f4706d8c0
Network plugin used: weave
Copy of your inventory file:
[all]

predev-k8s-1-12-etcd-01    ansible_host=<IP_ADDRESS> etcd_member_name=etcd1
predev-k8s-1-12-etcd-02    ansible_host=<IP_ADDRESS> etcd_member_name=etcd2
predev-k8s-1-12-etcd-03    ansible_host=<IP_ADDRESS> etcd_member_name=etcd3

predev-k8s-1-12-master-01  ansible_host=<IP_ADDRESS>
predev-k8s-1-12-master-02  ansible_host=<IP_ADDRESS>

predev-k8s-1-12-node-01  ansible_host=<IP_ADDRESS>
predev-k8s-1-12-node-02  ansible_host=<IP_ADDRESS>
predev-k8s-1-12-node-03  ansible_host=<IP_ADDRESS>
predev-k8s-1-12-node-04  ansible_host=<IP_ADDRESS>

# ## configure a bastion host if your nodes are not directly reachable
# bastion ansible_host=x.x.x.x ansible_user=some_user

[kube-master]
predev-k8s-1-12-master-01
predev-k8s-1-12-master-02

[etcd]
predev-k8s-1-12-etcd-01
predev-k8s-1-12-etcd-02
predev-k8s-1-12-etcd-03

[kube-node]
predev-k8s-1-12-node-01
predev-k8s-1-12-node-02
predev-k8s-1-12-node-03
predev-k8s-1-12-node-04

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

[calico-rr]

[vault]
predev-k8s-1-12-etcd-01
predev-k8s-1-12-etcd-02
predev-k8s-1-12-etcd-03

[centreon-clients:children]
kube-node
kube-master
etcd

[all:vars]
ansible_user=root
env_name=predev
k8s_default_namespace=predev
internal_domain_name=mydomain.net

ansible_ssh_common_args="-o PubkeyAuthentication=no -o ControlMaster=auto -o ControlPersist=30m"%

Command used to invoke ansible: ansible-playbook -i myfile.cfg kubespray/cluster.yml
Output of ansible run:
Firstly I was getting this :
TASK [kubernetes-apps/cluster_roles : Kubernetes Apps | Wait for kube-apiserver] **********************************************************************************************************************************
skipping: [predev-k8s-1-12-master-02]
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (10 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (9 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (8 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (7 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (6 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (5 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (4 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (3 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (2 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (1 retries left).
fatal: [predev-k8s-1-12-master-01]: FAILED! => {"attempts": 10, "changed": false, "connection": "close", "content": "{\"kind\":\"Status\",\"apiVersion\":\"v1\",\"metadata\":{},\"status\":\"Failure\",\"message\":\"Unauthorized\",\"reason\":\"Unauthorized\",\"code\":401}\n", "content_length": "129", "content_type": "application/json", "date": "Wed, 14 Nov 2018 13:36:44 GMT", "json": {"apiVersion": "v1", "code": 401, "kind": "Status", "message": "Unauthorized", "metadata": {}, "reason": "Unauthorized", "status": "Failure"}, "msg": "Status code was 401 and not [200]: HTTP Error 401: Unauthorized", "redirected": false, "status": 401, "url": "https://<IP_ADDRESS>:6443/healthz"}

Going to the url https://<IP_ADDRESS>:6443/healthz giving me an HTTP 200 (by disabling certificate check), i've tried to comment out the certificate related lines on the task :
- name: Kubernetes Apps | Wait for kube-apiserver
  uri:
    url: "{{ kube_apiserver_endpoint }}/healthz"
    validate_certs: no
    #    client_cert: "{{ kube_apiserver_client_cert }}"
    #    client_key: "{{ kube_apiserver_client_key }}"
  register: result
  until: result.status == 200
  retries: 10
  delay: 6
  when: inventory_hostname == groups['kube-master'][0]

By doing this, i got this one :
TASK [kubernetes-apps/cluster_roles : PriorityClass | Create k8s-cluster-critical] ********************************************************************************************************************************
skipping: [predev-k8s-1-12-master-02]
fatal: [predev-k8s-1-12-master-01]: FAILED! => {"changed": false, "msg": "error running kubectl (/usr/local/bin/kubectl apply --force --filename=/etc/kubernetes/k8s-cluster-critical-pc.yml) command (rc=1), out='', err='error: unable to recognize \"/etc/kubernetes/k8s-cluster-critical-pc.yml\": Get https://<IP_ADDRESS>:6443/api?timeout=32s: x509: certificate signed by unknown authority (possibly because of \"crypto/rsa: verification error\" while trying to verify candidate authority certificate \"kube-ca\")\n'"}

So my guess is that there is a problem with the certificate but i can't see what.

Could you try revert this and see if this works:
https://github.com/kubernetes-incubator/kubespray/pull/3698

Heya.
Just tried and same problem :
TASK [kubernetes-apps/cluster_roles : Kubernetes Apps | Wait for kube-apiserver] **********************************************************************************************************************************
skipping: [predev-k8s-1-12-master-02]
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (10 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (9 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (8 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (7 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (6 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (5 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (4 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (3 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (2 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (1 retries left).
fatal: [predev-k8s-1-12-master-01]: FAILED! => {"attempts": 10, "changed": false, "connection": "close", "content": "{\"kind\":\"Status\",\"apiVersion\":\"v1\",\"metadata\":{},\"status\":\"Failure\",\"message\":\"Unauthorized\",\"reason\":\"Unauthorized\",\"code\":401}\n", "content_length": "129", "content_type": "application/json", "date": "Wed, 14 Nov 2018 14:50:26 GMT", "json": {"apiVersion": "v1", "code": 401, "kind": "Status", "message": "Unauthorized", "metadata": {}, "reason": "Unauthorized", "status": "Failure"}, "msg": "Status code was 401 and not [200]: HTTP Error 401: Unauthorized", "redirected": false, "status": 401, "url": "https://<IP_ADDRESS>:6443/healthz"}

same error here, possibly related, though I don't see it working on https://github.com/kubernetes-incubator/kubespray/commit/b997912ebeb69e5e4f60c2c3f8cf7f5520bb2141 nor the 2.7.0 release, nor master.
my hosts file:
[all]
kubespraymaster1 ip=<IP_ADDRESS> etcd_member_name=kubespraymaster1 ansible_ssh_host=<IP_ADDRESS> ansible_ssh_port=22 ansible_user=root
kubespraymaster2 ip=<IP_ADDRESS> etcd_member_name=kubespraymaster2 ansible_ssh_host=<IP_ADDRESS> ansible_ssh_port=22 ansible_user=root
kubespraymaster3 ip=<IP_ADDRESS> etcd_member_name=kubespraymaster3 ansible_ssh_host=<IP_ADDRESS> ansible_ssh_port=22 ansible_user=root
kubesprayetcd1 ip=<IP_ADDRESS> etcd_member_name=kubesprayetcd1 ansible_ssh_host=<IP_ADDRESS> ansible_ssh_port=22 ansible_user=root
kubesprayetcd2 ip=<IP_ADDRESS> etcd_member_name=kubesprayetcd2 ansible_ssh_host=<IP_ADDRESS> ansible_ssh_port=22 ansible_user=root
kubesprayetcd3 ip=<IP_ADDRESS> etcd_member_name=kubesprayetcd3 ansible_ssh_host=<IP_ADDRESS> ansible_ssh_port=22 ansible_user=root
kubespraynode1 ip=<IP_ADDRESS> etcd_member_name=kubespraynode1 ansible_ssh_host=<IP_ADDRESS> ansible_ssh_port=22 ansible_user=root
kubespraynode2 ip=<IP_ADDRESS> etcd_member_name=kubespraynode2 ansible_ssh_host=<IP_ADDRESS> ansible_ssh_port=22 ansible_user=root
kubespraynode3 ip=<IP_ADDRESS> etcd_member_name=kubespraynode3 ansible_ssh_host=<IP_ADDRESS> ansible_ssh_port=22 ansible_user=root
kubesprayingress1 ip=<IP_ADDRESS> etcd_member_name=kubesprayingress1 ansible_ssh_host=<IP_ADDRESS> ansible_ssh_port=22 ansible_user=root

[kube-node]
kubespraynode1
kubespraynode2
kubespraynode3

[kube-node:vars]
ansible_ssh_extra_args="-o StrictHostKeyChecking=no"

[calico-rr]
kubespraymaster1
kubespraymaster2
kubespraymaster3

[kube-master]
kubespraymaster1
kubespraymaster2
kubespraymaster3

[kube-master:vars]
ansible_ssh_extra_args="-o StrictHostKeyChecking=no"

[etcd]
kubesprayetcd1
kubesprayetcd2
kubesprayetcd3

[vault]
kubesprayetcd1
kubesprayetcd2
kubesprayetcd3

[ingress]
kubesprayingress1

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

any update here, got into same situation

@mobilitie can you share the error log or hosts file?

I just tried on centos 7.5 with similar results, kubelet appears to be left in its default [broken] state:
-- The start-up result is done.
Nov 18 05:19:01 kubespraymaster1 kubelet[29997]: F1118 05:19:01.879461   29997 server.go:190] failed to load Kubelet config file /var/lib/kubelet/config.yaml, error failed to read kubelet config file "/var/lib/kubelet/config.yaml", error: open /var/lib/kubelet/config.yam
Nov 18 05:19:01 kubespraymaster1 systemd[1]: kubelet.service: Main process exited, code=exited, status=255/n/a
Nov 18 05:19:01 kubespraymaster1 systemd[1]: kubelet.service: Unit entered failed state.
Nov 18 05:19:01 kubespraymaster1 systemd[1]: kubelet.service: Failed with result 'exit-code'.
lines 1654-1724/1724 (END)

i have 3 masters with 3etcd, and 3 nodes
Ansible-play book fails at below task and errors
TASK [kubernetes-apps/cluster_roles : Kubernetes Apps | Wait for kube-apiserver] ********************************
Sunday 18 November 2018  05:50:29 +0000 (0:00:00.050)       0:07:38.740 *******
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (10 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (9 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (8 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (7 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (6 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (5 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (4 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (3 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (2 retries left).
FAILED - RETRYING: Kubernetes Apps | Wait for kube-apiserver (1 retries left).
fatal: [mgm-k8s-master-1]: FAILED! => {"attempts": 10, "changed": false, "connection": "close", "content": "{"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"Unauthorized","reason":"Unauthorized","code":401}\n", "content_length": "129", "content_type": "application/json", "date": "Sun, 18 Nov 2018 05:51:34 GMT", "json": {"apiVersion": "v1", "code": 401, "kind": "Status", "message": "Unauthorized", "metadata": {}, "reason": "Unauthorized", "status": "Failure"}, "msg": "Status code was 401 and not [200]: HTTP Error 401: Unauthorized", "redirected": false, "status": 401, "url": "https://<IP_ADDRESS>:6443/healthz"}
NO MORE HOSTS LEFT **********************************************************************************************
to retry, use: --limit @/home/centos/kubespray.working/cluster.retry
PLAY RECAP ******************************************************************************************************
localhost                  : ok=1    changed=0    unreachable=0    failed=0
mgm-bastion-1              : ok=4    changed=0    unreachable=0    failed=0
mgm-k8s-master-1           : ok=290  changed=18   unreachable=0    failed=1
mgm-k8s-master-nf-1        : ok=265  changed=17   unreachable=0    failed=0
mgm-k8s-master-nf-2        : ok=265  changed=17   unreachable=0    failed=0
mgm-k8s-node-nf-1          : ok=213  changed=11   unreachable=0    failed=0
mgm-k8s-node-nf-2          : ok=213  changed=11   unreachable=0    failed=0
mgm-k8s-node-nf-3          : ok=213  changed=11   unreachable=0    failed=0
When i logged into master and look at docker logs , i see certificate issues
[centos@mgm-k8s-master-1 ~]$ sudo docker logs --tail="5" k8s_kube-proxy_kube-proxy-mgm-k8s-master-1_kube-system_6921672a2a5f052e97051751ce351ac3_56
E1118 05:54:37.118581       1 reflector.go:134] k8s.io/client-go/informers/factory.go:131: Failed to list *v1.Endpoints: Get https://<IP_ADDRESS>:6443/api/v1/endpoints?limit=500&resourceVersion=0: x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "kube-ca")
E1118 05:54:38.095811       1 reflector.go:134] k8s.io/client-go/informers/factory.go:131: Failed to list *v1.Service: Get https://<IP_ADDRESS>:6443/api/v1/services?limit=500&resourceVersion=0: x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "kube-ca")
E1118 05:54:38.123691       1 reflector.go:134] k8s.io/client-go/informers/factory.go:131: Failed to list *v1.Endpoints: Get https://<IP_ADDRESS>:6443/api/v1/endpoints?limit=500&resourceVersion=0: x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "kube-ca")
E1118 05:54:39.101472       1 reflector.go:134] k8s.io/client-go/informers/factory.go:131: Failed to list *v1.Service: Get https://<IP_ADDRESS>:6443/api/v1/services?limit=500&resourceVersion=0: x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "kube-ca")
E1118 05:54:39.128721       1 reflector.go:134] k8s.io/client-go/informers/factory.go:131: Failed to list *v1.Endpoints: Get https://<IP_ADDRESS>:6443/api/v1/endpoints?limit=500&resourceVersion=0: x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "kube-ca")
[centos@mgm-k8s-master-1 ~]$

Nodes are  CentOS Linux release 7.5.1804 (Core) version, Using kubespray master
I also uncommented " cert_management: script " on group_vars/all.yml

how about journalctl -xeu kubelet?

in loop with below errors
Nov 18 06:02:16 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:16.773362   16695 reflector.go:134] k8s.io/kubernetes/pkg/kubelet/kubelet.go:442: Failed t
Nov 18 06:02:16 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:16.779673   16695 reflector.go:134] k8s.io/kubernetes/pkg/kubelet/config/apiserver.go:47:
Nov 18 06:02:16 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:16.786306   16695 reflector.go:134] k8s.io/kubernetes/pkg/kubelet/kubelet.go:451: Failed t
Nov 18 06:02:16 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:16.816515   16695 kubelet.go:2236] node "mgm-k8s-master-1" not found
Nov 18 06:02:16 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:16.916712   16695 kubelet.go:2236] node "mgm-k8s-master-1" not found
Nov 18 06:02:17 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:17.016914   16695 kubelet.go:2236] node "mgm-k8s-master-1" not found
Nov 18 06:02:17 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:17.117627   16695 kubelet.go:2236] node "mgm-k8s-master-1" not found
Nov 18 06:02:17 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:17.217802   16695 kubelet.go:2236] node "mgm-k8s-master-1" not found
Nov 18 06:02:17 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:17.317985   16695 kubelet.go:2236] node "mgm-k8s-master-1" not found
Nov 18 06:02:17 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:17.418120   16695 kubelet.go:2236] node "mgm-k8s-master-1" not found
Nov 18 06:02:17 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:17.518301   16695 kubelet.go:2236] node "mgm-k8s-master-1" not found
Nov 18 06:02:17 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:17.618488   16695 kubelet.go:2236] node "mgm-k8s-master-1" not found
Nov 18 06:02:17 mgm-k8s-master-1 kubelet[16695]: E1118 06:02:17.718632   16695 kubelet.go:2236] node "mgm-k8s-master-1" not found
lines 958-1001/1001 (END)

Same issues here ... :( it used to work on a different cluster (same host OS Ubuntu 16.04). Pulled latest master. It used to work on 3fafa583d1297c8cd7b7dadc102a185deb76d045

@mattymo Have you seen this before, where it throws this error:
Failed to list *v1.Node: Get https://<IP_ADDRESS>:6443/api/v1/nodes?fieldSelector=metadata.name%3Dmgm-k8s-master-1&limit=500&resourceVersion=0: x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "kube-ca"
Like the CA is buggy?

I got it running now specifying the version in the ansible-playbook command ...
ansible-playbook --flush-cache -i inventory/hosts.ini --become --user=ubuntu cluster.yml -e kube_version=v1.12.2
Take about 5x longer to finish (not sure why), but at least it starts the API server(s)

on centos 7 adding the -e kube_version=v1.12.2 results in:
RUNNING HANDLER [container-engine/docker : Docker | reload docker] ****************************************************************************
fatal: [kubesprayetcd1]: FAILED! => {"changed": false, "msg": "Unable to restart service docker: Job for docker.service failed because the control process exited with error code. See \"systemctl status docker.service\" and \"journalctl -xe\" for details.\n"}
fatal: [kubespraymaster1]: FAILED! => {"changed": false, "msg": "Unable to restart service docker: Job for docker.service failed because the control process exited with error code. See \"systemctl status docker.service\" and \"journalctl -xe\" for details.\n"}
fatal: [kubespraynode3]: FAILED! => {"changed": false, "msg": "Unable to restart service docker: Job for docker.service failed because the control process exited with error code. See \"systemctl status docker.service\" and \"journalctl -xe\" for details.\n"}
fatal: [kubespraynode1]: FAILED! => {"changed": false, "msg": "Unable to restart service docker: Job for docker.service failed because the control process exited with error code. See \"systemctl status docker.service\" and \"journalctl -xe\" for details.\n"}
fatal: [kubespraynode2]: FAILED! => {"changed": false, "msg": "Unable to restart service docker: Job for docker.service failed because the control process exited with error code. See \"systemctl status docker.service\" and \"journalctl -xe\" for details.\n"}
fatal: [kubesprayetcd2]: FAILED! => {"changed": false, "msg": "Unable to restart service docker: Job for docker.service failed because the control process exited with error code. See \"systemctl status docker.service\" and \"journalctl -xe\" for details.\n"}
fatal: [kubesprayetcd3]: FAILED! => {"changed": false, "msg": "Unable to restart service docker: Job for docker.service failed because the control process exited with error code. See \"systemctl status docker.service\" and \"journalctl -xe\" for details.\n"}

RUNNING HANDLER [container-engine/docker : Docker | pause while Docker restarts] **************************************************************

RUNNING HANDLER [container-engine/docker : Docker | wait for docker] **************************************************************************

NO MORE HOSTS LEFT ****************************************************************************************************************************
        to retry, use: --limit @/home/thoth/.kubash/submodules/kubespray/cluster.retry

PLAY RECAP ************************************************************************************************************************************
kubesprayetcd1             : ok=88   changed=21   unreachable=0    failed=1
kubesprayetcd2             : ok=87   changed=21   unreachable=0    failed=1
kubesprayetcd3             : ok=87   changed=21   unreachable=0    failed=1
kubespraymaster1           : ok=90   changed=23   unreachable=0    failed=1
kubespraymaster2           : ok=23   changed=3    unreachable=0    failed=1
kubespraymaster3           : ok=23   changed=3    unreachable=0    failed=1
kubespraynode1             : ok=104  changed=24   unreachable=0    failed=1
kubespraynode2             : ok=91   changed=23   unreachable=0    failed=1
kubespraynode3             : ok=91   changed=23   unreachable=0    failed=1
localhost                  : ok=1    changed=0    unreachable=0    failed=0

I'll try with xenial next

on xenial I still get a failing kubelet:
Nov 18 15:20:29 kubespraymaster1 systemd[1]: Starting Kubernetes Kubelet Server...
-- Subject: Unit kubelet.service has begun start-up
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit kubelet.service has begun starting up.
Nov 18 15:20:29 kubespraymaster1 systemd[1]: Started Kubernetes Kubelet Server.
-- Subject: Unit kubelet.service has finished start-up
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit kubelet.service has finished starting up.
--
-- The start-up result is done.
Nov 18 15:20:29 kubespraymaster1 kubelet[30202]: F1118 15:20:29.713949   30202 server.go:190] failed to load Kubelet config file /var/lib/kubelet/config.yaml, error failed to read kubelet config file "/var/lib/kubelet/config.yaml", error: open /var/lib/kubelet/config.yaml: no such file or directory
Nov 18 15:20:29 kubespraymaster1 systemd[1]: kubelet.service: Main process exited, code=exited, status=255/n/a
Nov 18 15:20:29 kubespraymaster1 systemd[1]: kubelet.service: Unit entered failed state.
Nov 18 15:20:29 kubespraymaster1 systemd[1]: kubelet.service: Failed with result 'exit-code'.

on debugging this, it appears that the external etcd cluster started successfully but how do I test it?
The official docs give this:
docker run --rm -it \
--net host \
-v /etc/kubernetes:/etc/kubernetes quay.io/coreos/etcd:v3.2.18 etcdctl \
--cert-file /etc/kubernetes/pki/etcd/peer.crt \
--key-file /etc/kubernetes/pki/etcd/peer.key \
--ca-file /etc/kubernetes/pki/etcd/ca.crt \
--endpoints https://${HOST0}:2379 cluster-health

but where are the certs in regards to kubespray?
Digest: sha256:1f433f387d7d0ff283e9abfdd493a461175bee0c372deca68c5633eb7ce7ee56
Status: Downloaded newer image for quay.io/coreos/etcd:v3.2.18
Error:  open /etc/kubernetes/pki/etcd/peer.crt: no such file or directory

@joshuacox did you reset the cluster first using reset.yml ? Not sure why this works for me... :(

I think I have found the issue. I will submit a PR later.

hmm how to run the reset.yml in a script? piping yes yes into ansible-playbook seems to be the wrong idea.

@joshuacox I've asked myself the same question, same for removeing nodes

@woopstar Do tell ? 👍


Added go program to find sum of GP series

Close: #3636

@AshitaSingamsetty Thanks for contributing 🎉
Feedback: Next time onwards try adding close issue number in description 🙂


Option to not add files to recent history for files outside of the opened folder

Somewhat related to this feature: #84271
Sometimes I'm working on a folder (Open vscode > Open Folder...) or several and I want to open a file with vscode that doesn't belong to any of the opened folders. This file gets added to the list of recently opened files.
I would like that it doesn't. The reason is because when I'm working on a folder, I would like to use Ctrl+E to search only for files contained in that folder. External files add noise that I have to clear manually.
I understand that other people might prefer differently from me, so it should be an option.
Thanks in advance.

Heads up that @GuiRitter has started a forum thread to crowdsource upvotes.
https://www.linuxquestions.org/questions/general-10/please-vote-on-my-vscode-feature-request-4175677418/

Heads up that @GuiRitter has started a forum thread to crowdsource upvotes.
https://www.linuxquestions.org/questions/general-10/please-vote-on-my-vscode-feature-request-4175677418/

Is that a bad thing?

Another forum thread intended to round up upvotes:
https://cplusplus.com/forum/lounge/271356/
That said, quoting the issues triaging page (emphasis mine):

If you feel your feature request deserves to stay open, improve your use case and ping us or gather more up-votes.

Another forum thread intended to round up upvotes:
https://cplusplus.com/forum/lounge/271348/
That said, quoting the issues triaging page (emphasis mine):

If you feel your feature request deserves to stay open, improve your use case and ping us or gather more up-votes.

Yeah, you're gonna find several...
I got the message. I'll take 'em down. I'm sorry, I wasn't aware.

All right, I think I got all of them. On 3 I couldn't edit nor remove the post. In these cases, I reported myself, asking for moderation to remove my posts.
(I'm appalled by how only 1 or 2 allowed me to delete the thread)
Sorry again for the inconvenience. I honestly didn't imagine something like this would be frowned upon and I didn't mean to cause trouble.


⚠ Processors has degraded performance

In 6409905, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 128 ms

Resolved: Processors performance has improved in c9e5423.


Break long lines

Makes it easier for syntax highlighters / linters to parse those lines.

Thanks, will check it out when I return from vacation

Hi, so I tried this but on my side it is more distracting to see the long lines, the thing is that this bloat appears as just one line on my editor and therefore I gracefully skip it.
Can you describe the issue you see on your side?
I really hope that https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2022/p1061r2.html gets in sometime and then we probably won't need those long lines.

Can you describe the issue you see on your side?

Hi. The issue is not serious. I just made the change so that Visual Studio and Visual Studio Code can handle them more easily. Right now they both cause stuttering around those lines, and also cannot display the whole line (they just terminate with ...):

I figured it could help users and contributors using the same editors so that they could fully read those lines. But if it's not a concern feel free to close the issue.
Unfortunately Visual Studio's linter still cannot handle the header and any file that includes the header ends up broken.
I made an issue on VS Dev community and it's currently "under consideration".

I think there is no right and wrong answer to this currently, since the long lines are kind of manual expansion of pack they seem to me not interesting for the reader - not worth expanding it into many lines so I vote to keep it the way it is for now. As for contributions, I believe it will require anyway change of the whole line so to not hurt the consistency so I assume the contributor will anyway erase the whole line. Thanks for putting up a PR and caring.


Unable to deploy drivers

Deployment fails in applying driver package
Trying debug mode but gets the same error
C:\temp\Invoke-CMApplyDriverPackage.ps1 : InnerTerminatingFailure
At line:1 char:1

.\Invoke-CMApplyDriverPackage.ps1 -DebugMode -Endpoint "svekmn-memcm. ...

  + CategoryInfo          : NotImplemented: (:String) [Invoke-CMApplyDriverPackage.ps1], RuntimeException
  + FullyQualifiedErrorId : Invoke-CMApplyDriverPackage.ps1

ApplyDriverPackage.log
smsts.log

Log shows it's unable to find a match on 'Drivers' in the package name.
what is the actual name of the driver package in the console for that HP ProBook 445?

The name is given automatically through Driver Automation Tool:
"Drivers - HP ProBook 445 14 inch G9 Notebook PC - Windows 11 23H2 x64"


Refactor OAuth setup

This is a refactoring to move oauth configuration into the core.common module to use it in the account setup.
This also fixes ProvidersXmlDiscovery depending on app.core and app.testing

I see that using an enum is not ideal. The dynamic OAuth provider could be added by extending the enum with a Custom type, but I removed them anyway.


Allowing Server Moderators to moderate help threads

This can help server moderators as we know that some of them can not manage help threads if they do not have the Help Mod role as of yet.
It is also a suggestion that Koala9712 made in the Nextcord server.

I have also just added periods(.) to the README.md page because yes.
Added SERVER_MOD_ROLE_ID as a .env, for the Nextcord server's case, you would obviously get the Server Mod role ID.

We give all server mods the help mod role so this is not necessary and just adds more complexity to previous.

We give all server mods the help mod role so this is not necessary and just adds more complexity to previous.

fair.


added some css to make it a little bit more organized and stylish. Th…

…en I also added the modals to the exercise hadnlerbars so the workouts could be diplayed and the user could interact with it

No thumbs up emoji :(


Add signatureHelp/resolve support, because signature help takes too long with multiple overloads

After we added "doc comment" support for signature help to our C/C++ extension, getting the results for 20+ overloads takes too long, so we want to send the documentation as part of a signatureHelp/resolve, just like in the completionItem/resolve case. We would get the signatureHelp/resolve message when the user uses the up/down arrow to change which signature is shown.
This would also allow us to reset the active signature after the user types, because currently if the user selects the 20th overload, it gets reset back to 1 after typing because we never get a "resolve" message that identifies the overload the user has switched to.

The active signature problem is tracked here https://github.com/Microsoft/vscode/issues/33413 with a solution to add this to the context.

For the lazy resolve I opened https://github.com/Microsoft/vscode/issues/65003


Add credentials :: each Source Type is listed twice

Is pretty evident that there is a bug:

Looking in some of your old video guide from 2019 it looks like the first set of logins was for normal logins while the second set was for domains login.

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

ID: 60d3e532-e3e5-219a-ce0c-502b48faaebb
Version Independent ID: 19765a75-4ee0-7ce7-5d34-1cac117cf80c
Content: Discover physical servers with Azure Migrate Discovery and assessment - Azure Migrate
Content Source: articles/migrate/tutorial-discover-physical.md
Service: azure-migrate
GitHub Login: @vineetvikram
Microsoft Alias: vivikram

I tested today and I can confirm you that you have the same bug for both drop down menu in:

Add credentials
Add discovery source

I tested and the second one works with domain credentials while the first fails

@francesco1119 - Are you noticing this bug in tool or in documentation? I could not find this error in documentation. Request you to help me with navigation path if this is a documentation bug.
If have self hosted the scenario and it is working fine for me. For any bug related to tool/ code, please open a support case to get it rectified.
Regards
Vineet

Yes, the bug is not in the documentation but in the tool itself.
As you can see from the picture I pasted on my first post you have each single login listed twice. This because one was for the normal login and the other for the DOMAIN\ login.
Nevermind, I will open a ticket form the Azure portal dashboard.
...what's the point to use GitHub for the documentation and not for tool issues... :|


Fix whitespace from original xauth project.

Sad but necessary.  Glad to be creating this blackhole right after the initial commit, at least.


Tab and paging active class collision

If I have a tab container with a paging control inside the first tab, the first switch back to the first tab will display both tabs. Subsequent tab switches display correctly, but lose some classes and may no longer animate the fade.
Example: https://codepen.io/ohammersmith/pen/jGaxNM

Refresh the page
Click the second tab
Click the first tab
See both tab contents

The issue seems to be that the active class on the nav-link gets that element picked up by the tab switching. You can see that visually... we lose the current page style on the paging button.
Note: further switches are displayed correctly, but the show and fade classes are lost. I'm not entirely sure why both tabs don't end up hidden without the show class.
No patch because the couple simple fixes I tried weren't complete fixes. I maybe able to contribute a patch later.
Chrome: Version 61.0.3163.100 (Official Build) (64-bit)
OS X: 10.12.6

I believe this is fixed in v4-dev. It must be https://github.com/twbs/bootstrap/pull/23087.
/CC @Johann-S

Yep already fixed by #23087

Awesome, thanks! I missed it when I searched.


Temporarily disable Push

Our current Push (IMAP IDLE) implementation isn't working reliably with the power saving measures introduced by modern Android versions. Fixing this requires a lot of internal changes, hence time. To not hold up a (beta) release we'll temporarily disable the Push functionality.
The "Folder poll frequency" will be set to the lowest value (15 Minutes) if "Push folders" was set to anything but "None". Otherwise the old value is kept.
The "Poll folders" setting will be changed based on the combined values of "Poll folders" and "Push folders".

Poll folders
Push folders
New Poll folders

All
All
All

All
Only 1st Class folders
All

All
1st and 2nd Class folders
All

All
All except 2nd Class folders
All

All
None
All

Only 1st Class folders
All
All

Only 1st Class folders
Only 1st Class folders
Only 1st Class folders

Only 1st Class folders
1st and 2nd Class folders
1st and 2nd Class folders

Only 1st Class folders
All except 2nd Class folders
All except 2nd Class folders

Only 1st Class folders
None
Only 1st Class folders

1st and 2nd Class folders
All
All

1st and 2nd Class folders
Only 1st Class folders
1st and 2nd Class folders

1st and 2nd Class folders
1st and 2nd Class folders
1st and 2nd Class folders

1st and 2nd Class folders
All except 2nd Class folders
All

1st and 2nd Class folders
None
1st and 2nd Class folders

All except 2nd Class folders
All
All

All except 2nd Class folders
Only 1st Class folders
All except 2nd Class folders

All except 2nd Class folders
1st and 2nd Class folders
All

All except 2nd Class folders
All except 2nd Class folders
All except 2nd Class folders

All except 2nd Class folders
None
All except 2nd Class folders

None
All
All

None
Only 1st Class folders
Only 1st Class folders

None
1st and 2nd Class folders
1st and 2nd Class folders

None
All except 2nd Class folders
All except 2nd Class folders

None
None
None

Just to be sure: This disable all automatic email check in the background?

No. "Push" is keeping a connection to the IMAP server open to be immediately notified about new messages. Periodically checking for messages (poll) will still work.


feat: OAuth Property Supplier for IAS

Context

SAP/cloud-sdk-java-backlog#391.
This PR adds a new OAuth2PropertySupplier implementation that is capable of reading IAS service bindings.
With that, it should - in theory - be possible to fetch tokens for App-to-Service communication scenarios.

Manual E2E Test
The implementation has been manually E2E tested by extending our scp-cf-spring-ias-java-17 test with the following service:
@RestController
@RequestMapping("/ias")
public class IasSpringController {
    private static final Logger logger = LoggerFactory.getLogger(IasSpringController.class);

    @RequestMapping( method = RequestMethod.GET, path = "/token" )
    public ResponseEntity<?> getToken()
    {
        try {
            logger.info("Requesting IAS token on behalf of '{}'", TenantAccessor.getCurrentTenant());

            final ServiceBindingDestinationOptions options =
                    ServiceBindingDestinationOptions.forService(ServiceIdentifier.of("identity")).build();
            final HttpDestination destination =
                    ServiceBindingDestinationLoader.defaultLoaderChain().getDestination(options);

            final String authorizationHeaderValue =
                    destination
                            .getHeaders()
                            .stream()
                            .filter(h -> HttpHeaders.AUTHORIZATION.equalsIgnoreCase(h.getName()))
                            .map(Header::getValue)
                            .findFirst()
                            .get();

            return ResponseEntity
                    .ok(authorizationHeaderValue.substring(0, Math.min(8, authorizationHeaderValue.length() - 8)) + "****");
        }
        catch( final Exception e ) {
            logger.error("Exception while fetching IAS token.", e);
            return ResponseEntity.internalServerError().build();
        }
    }
}

cf.logs.zip

Feature scope:

[x] Implement new OAuth2PropertySupplier for IAS
[x] Refactor logic of how the OAuth2TokenEndpointsProvider is created

This was needed as the IAS token endpoints are slightly different from the XSUAA ones

Definition of Done

[x] Functionality scope stated & covered
[x] Tests cover the scope above
[x] Error handling created / updated & covered by the tests above
[x] ~Documentation updated~ (not meant for the public audience just yet)
[x] ~Release notes updated~ (not meant for the public audience just yet)

Continued in #278


Optional Language Detect

Implementing optional language detection and allow the creation of custom language detection libraries by users.
This work is currently in progress and this PR is created for the monitoring purpose by the maintainers to suggest progress.
This PR is currently not complete.
TODOS:

[X] Restructuring protect to pass function for language detection.
[X] Managing HTTP exception for fasttext model download.
[x] Updating setup.py for optional language detection.
[x] Mock test of parse, search_dates and DateDataParser.
[x] Unit tests for language detect functions.
[x] Functionality analysis of the implemented language detection.
[x] Testing optional language detection settings.
[x] Make apply_setting independent functions.
[x] Fasttext model download manager for models.
[x] Documentation optional language detection settings.
[x] Language Mapping (CLDR & ISO 639).
[x] Check working with other settings.
[x] Removing unsupported locale from language detecting
[x] Improving docs.
[x] Setting fasttext default language.
[x] Removing langauge_map.json and sinking to language_data.py
[x] langdetect set default DetectorFactory without changing global state.
[x] dateparser-download set default caching folder.
[x] Documenting dateparser-download.
[x] Changes for preventing breaking changes.
[x] detect_languages_func -> detect_languages_function

regarding adding dateparser/custom_lang_detect/data/lid.176.ftz - I'm not sure we should package this in dateparser even though it's not too big, but this is a minor thing which can be dealt with later

HI @lopuhin, I am actually still developing and testing so I have not deleted it but will do that but in the end, I will clean codes and remove them.

Hi @lopuhin and @noviluni, I have made some changes can you suggest. I have also tried to cache the model without the global keyword instead used a class for the same.
I have started writing additional tests will try to complete them by Friday.

@gavishpoddar I saw you resolved some conversations, but I don't think you pushed anything, perhaps you forgot to push?

Hey should I start writing tests for the additional language detection library or wait till the internal API is complete

Hey should I start writing tests for the additional language detection library or wait till the internal API is complete

That's a good question - maybe @noviluni can give more solid advice here. At first I was thinking it would be good to start with tests which check the public API, but I'm not sure how to exactly test it properly - I mean how to ensure that language detection is called and it's results are used, without too much hacks or mocking.

Hi @kishan3, I am making changes to the structuring so this will be fixed with it :)

Hi, Can you please review the PR?
Docs are WIP.

Trivial point: @noviluni @lopuhin @gavishpoddar Should we have full name detect_languages_function instead of detect_languages_func?

Should we have full name detect_languages_function instead of detect_languages_func?

detect_languages_fn could be another one if we strive for brevity. I personally don't have a big preference, but I like detect_languages_function a bit better.

Hi @gavishpoddar, I enabled the pipelines. The tests are failing because there's an error: Fatal Python error: Segmentation fault. I don't know the reasons, but it is produced by test_custom_language_detect_fast_text(), so you can either try to fix it or disable it temporarily (just comment) to be able to run the pipelines and see if the other parts are working as expected.

You have here two options:

Put this outside the class (after the import statements):

# here the imports

detect_languages = Mock()
detect_languages.return_value = ["en"]

# here the `CustomLangDetectParser` class (BTW I would call it `CustomLangDetectParserTest`, with the "test" word)

Add this as a property in the setUp() method and access them using self:

    def setUp(self) -> None:
        self.detect_languages = Mock()
        self.detect_languages.return_value = ["en"]
        super().setUp()

# and then when needing `detect_languages` use `self.detect_languages`.

Let me know if it's not clear :)

Hi @gavishpoddar, you uncommented the test_custom_language_detect_fast_text test. Could you comment it again? (better with # than with """)

We need to (not necessarily in this order):

Document new settings in the settings docs
Fix langdetect test
Improve coverage by adding tests to get the same or better coverage than previous one.
Do as much as use of this feature, trying different things to fix them and submit the patches.

Hi @lopuhin, @noviluni, @kishan3,
I have -

Document new settings in the settings docs.
Fix language detection tests (fasttext segment faults).
Improve coverage.

I am unable to increase coverage anymore it will be great if you can suggest how to do that I am left with 4 new lines of code that can't be covered with the tests.
And, It will be really great to receive as many reviews as possible.

Hi @gavishpoddar!
I added some comments and suggestions. I miss at least two things before merging:

Update the check_settings function (in dateparser/conf.py) with DEFAULT_LANGUAGES and LANGUAGE_DETECTION_CONFIDENCE_TRESHOLD to ensure that they only takes correct values.
Add tests for this two settings in tests/test_settings.py.

Let me know if you need help with this :)

I have added new settings in dateparser/conf.py and created tests in test_settings.py I am not sure about them and if they are correct please suggest

Thanks, @noviluni, for the review.
I have made some changes to DEFAULT_LANGUAGES.
Pl suggest is it fine or should I remove this from this PR.

Thanks, @noviluni, for the review.
I have made some changes to DEFAULT_LANGUAGES.
Pl suggest is it fine or should I remove this from this PR.

Ok! Good job @gavishpoddar, I've been resolving some conversations.
I have finished reviewing 22/26 files. I will try to find a moment (maybe tomorrow) to review the other 4 and will provide some feedback :)

Hi tests are failing for some reason, I am not sure why but looks like some issue here
Pl suggest

Hi @gavishpoddar, I'm not sure why is the error happening, I will investigate it. However, you have to remove this change: https://github.com/scrapinghub/dateparser/pull/932/commits/a2c999fdd95985e72a3be30bde55d6c11f938cb1
Typing is only available from Python 3.6, Python 3.5 will break with that.

Hi Gavish, I found why the pipelines are failing and submitted a PR: https://github.com/scrapinghub/dateparser/pull/966
Once it is merged (is not merged yet) the pipelines will work again :)

@gavishpoddar congrats on the merge, great work :+1:
@noviluni sorry that I didn't ask before merging, I hope a merge commit is fine, or would you prefer a squash and merge?

Thanks for the merge :)

Hi @lopuhin, sorry for the late answer. I prefer to squash because in that way it is easier for me to build the changelog when releasing a new version, but no problem, it is fine :)
(Congrats Gavish! :muscle: )


Update snakeyaml to 1.32

Updates org.yaml:snakeyaml from 1.28 to 1.32.
Release Notes - Release Notes - Release Notes - Release Notes - Release Notes - Release Notes - Release Notes - Release Notes - Release Notes - Release Notes - Release Notes - Release Notes - Changelog - Changelog - Changelog - Changelog - Changelog - Changelog - Changelog - Changelog - Changelog - Changelog - Changelog - Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.yaml", artifactId = "snakeyaml" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.yaml", artifactId = "snakeyaml" }
}]

labels: library-update, commit-count:1

would be great to merge it to fix the following issue:
https://nvd.nist.gov/vuln/detail/CVE-2022-25857


[monitor-opentelemetry] Live Metrics: Perf Counter update

Packages impacted by this PR
[monitor-opentelemetry]
Describe the problem that is addressed by this PR
See: https://github.com/aep-health-and-standards/Telemetry-Collection-Spec/blob/main/ApplicationInsights/livemetrics.md#process-metrics
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
For CPU, looked at .net & java implementations, along with this stack overflow. They all seem to be following the same rough calculation & multiplying by 100 in the end for a percentage.
Are there test cases added in this PR? (If not, why?)
Modified an existing test to assert that the body of the post request includes old/new counter names with the process counter values.
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[x] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


Changes for showing pebblo server and client version on DAXA UI.

Sending ai_apps data including pebbloServerVersion and pebbloClientVersion in v1/app/discover API
Updated UTs as well

Hold until after 0.1.14 release


tooltip html entity

How can I use a SafeHtml in a tooltip attribute?
Right now it always shows the html entity instead of the actual character.
Like ©
With attributes it is not working, only works with innerHTML.
:(

We do not support SafeHtml in the tooltip attribute. It should be sufficient to put the actual character into the message string, e.g. https://plnkr.co/edit/DxM7r7BkLGusYTdLLDXb?p=preview

It would be nice!
For example: we show a tooltip for a quick help, and add a html link in it to redirect with a QA, more explanation... Etc.
And for it, the tooltip should be "fixed" when the mouse is over :D
Thanks for your work,


fix: tweak events + don't trigger GA on dev

What it solves
@francovenica noticed some inconsistencies in a few events. These have been fixed/tweaked and the GTM-GA link removed on dev. environments.
How this PR fixes it
It is possible to have production only triggers in GA but they cost with every event. Instead, the triggers have been adjusted to fire when the built-in debug mode variable (true by default in the development environment) is false.
The following events were tweaked:

[x] 1. Duplicate page views fixed.
[x] 2. Add "Execute transaction" checkbox tracking.
[x] 3. Add use of spending limit tracking.
[x] 4. Only track number of added Safes in sidebar.
[x] 5. Parse all labels (numbers, booleans, etc. passed as strings).
[x] 6. Track number of NFTs, not collections.

How to test it
Preview the local branch/PR in GTM and observe no GA tags firing in GTM.

Repeatedly click on a transaction tab and observe only one 'pageview' event.
(Un-)check the "Execute transaction" checkbox and observe a new 'cusomClick' event.
Pay for a transaction from a spending limit and observe a new 'metadata' event.
Open the sidebar and observe a 'metadata' event for the number of Safes on the network. Navigating to the settings should not trigger this.
Add/remove an owner. Observe the owner number 'metadata' be a number, not a string.
Open Collectibles and observe the total number of NFTs being tracked in a 'metadata' event, not the number of collections.

Analytics changes
GA events should not trigger locally or in PRs.

Looks good:
Visiting the Queue/History tab won't register pageviews if clicked again and again
Clicking the execute checkbox creates an event with true/false for checked/unchecked
Submiting a tx with the spending limit creates a metadata log with "spending limit" tag
Opening the sidebar now shows a metadata with the amount of safes you have for the network you are currently in
The label in the "policies" metadata is now a number
NFT's are now counted individually


Support a timeout on docker client

This gets us slightly closer to being able to remove E501 (ignore line length) from flake8 ignore list

support a timeout environment variable for the docker-py client
sort DOCKER_CONFIG_KEYS and make them one-per-line (helps avoid merge conflicts when new keys are added, also just easier to read)
only include python versions in tox that actually work with the current version

Is DOCKER_CLIENT_TIMEOUT the right name for this environment variable? I'm not sure inventing new DOCKER_* vars that aren't supported by the docker client is a good idea.

It seems like the most logical name for it (especially with the rebrand).
Maybe the docker cli should support it?  I wasn't able to find an existing environment variable at the time (but it's possible I missed it).


Some questions for gradcam

If I want to do visualization, do I need to train a new  model for classification?

Updated usage!


Create schemas for generated Parquet files

Create schemas for generated Parquet files

Implemented there:
https://github.com/RedHatInsights/insights-data-schemas/blob/master/schemas/parquet_output_rule_hits.py
https://github.com/RedHatInsights/insights-data-schemas/blob/master/schemas/parquet_output_thanos_info.py
https://github.com/RedHatInsights/insights-data-schemas/blob/master/schemas/platform_upload_buckit_messages.py


WebSearch: Incorrect parsing of user input

Originally on 2010-12-03
following lines in search_engine.py look more than suspicious.
elif p.startswith("cocitedwith:"):  #WAS EXPERIMENTAL
(...)
recID = p[12:]
For example in the case of having some logical operator after this, Invenio crashes.
example:
http://inspirebeta.net/search?ln=en&p=cocitedwith%3A561945+and+doesnotwork&f=&action_search=Search&sf=&so=d&rm=&rg=25&sc=0&of=hb
RT 106598

FIxed with new search unit (see https://github.com/inveniosoftware/invenio/commit/4b95199889cbf814a2ef134cb149946f44a58d1e#diff-be41293cb5b20239b11f6b00a3405eec).


generator-m default is not working on Windows

Hi,
I followed the instructions to get started, but I cannot get a basic sample running on Windows 7.
Node is running in MINGW32.
There were some errors reported after yo-m.
npm WARN optional dep failed, continuing<EMAIL_ADDRESS>npm ERR! Error: EPERM, open 'C:\Users\crowne\AppData\Roaming\npm-cache\b21bf682-cache-inherits-2-0-1-package-tgz.lock'
I reran npm install and got Error: Cannot find module 'readable-stream'
gulp watch gives Error: Cannot find module 'wiredep'
What can I do to resolve these errors?
Regards,
Neil

Hi Neil,
running the generator on windows usually isn't a problem. What you could do is:

delete the node_modules folder in the generated project and rerun npm install

if that doesn't work:

install node.js for windows https://nodejs.org/download/, without MINGW32.

That's how we successfully run generator-m on windows.
Please let us know whether this was of any help!

Thanks Jonathon.
I had to upgrade npm from 1.4.28 to latest (2.7.5).
I had to npm clean cache.
I had to reboot windows, after clean cache to eventually get an install working.
But I now have the scaffold running.


Feature/jel 296 order items retirement

Implements retirement for RDS, S3, and EC2

 Code Climate has analyzed this pull request.


now supports fabrication note path/text color prop

@seveibar what's with the size check?

No idea, but this pr shouldnt change thr size anyway. It seems like it could be an issue with the workflow not supporting whatever we're supplying as the GITHUB_TOKEN
Its worth fixing this in the future but we can merge this one for now.


slow tests on windows

Several tests seem to be flaking as timeouts on windows. Here is a list of tests I've marked as such:
# suite dart2js_extra
# in chrome:
class_test: Pass, Slow
consistent_codeUnitAt_error_test: Pass, Slow
closure_capture3_test: Pass, Slow
deferred_split_test: Pass, Slow
closure_capture5_test: Pass, Slow
conditional_test: Pass, Slow
constant_javascript_semantics2_test: Pass, Slow

# in ff:
consistent_index_error_string_test: Pass, Slow

# suite co19 in #ff
Language/Classes/Getters/type_object_t02: RuntimeError, Slow
Language/Classes/Abstract_Instance_Members/override_no_named_parameters_t06: Pass, Slow
Language/Classes/Constructors/Factories/return_type_t03: Pass, Slow
Language/Classes/Constructors/Factories/return_wrong_type_t02: Pass, Slow
Language/Classes/Constructors/Factories/return_type_t05: Pass, Slow

On linux, dart2js_extra/mirror_printer_test takes just about 1 minute, so it is marked as slow.

If the type_object_t02.dart test timed out, it is probably flaky. It doesn't have any loop, or dependency on the browser.
Did you (@sigmundch) check that the tests were slow in previous runs?

I don't recall, it's quite possible that I didn't.


Broken @import errors

Since upgrading from 0.1.23 to 0.1.25  'Broken @import declaration of" when bundling.
I had to go back to 0.1.23.

Can you share a test case? Are you setting the rootURL?
On Wed, 27 Jul 2016 at 17:52, Alex<EMAIL_ADDRESS>wrote:

Since upgrading from 0.1.23 to 0.1.25 'Broken @import
https://github.com/import declaration of" when bundling.
I had to go back to 0.1.23.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/systemjs/plugin-css/issues/96, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkiyjn5sqGywzMJ1NRJEXlmTE0EPf4Mks5qZ37LgaJpZM4JWV0a
.

Sure.
This is how the bundle task looks:
`   gulp.task('bundle', ['ensure-bundle'], function (cb) {
     var builder = new Builder();
     builder.loadConfig(options.config)
        .then(function() {
           builder.config({
              buildCSS: true,
              separateCSS: true
           });
           return builder.buildStatic(options.entryJs, options.outputJs);
        })
        .then(function() {
           gutil.log("Output " + options.outputJs + ", " + options.outputCss);
           cb();
        })
        .catch(function(err) {
           gutil.log('Build error:');
           gutil.log(err);
           cb(err);
        });
  });`

I'm not setting rootURL.
Then I got  in every module(folder) of the project an index.css file with all the css imports like for example ui/controls/src/styles/index.css or ui/crm/src/styles/index.css :
@import "styles/component/event-description.css";
@import "styles/module/recent-events-widget.css";
When bundling using version 0.1.25 I get the following errors:  'Broken @import declaration of "crm/src/styles/component/event-description.css"',
'Broken @import declaration of "crm/src/styles/module/recent-events-widget.css"',
Do I need to set the rootURL at all?
Thanks
Alex

Thanks, sure I'll test this out as soon as I can.
So it sounds like it is normalizing to crm/src/styles/module/recent-events-widget.css which for some reason is invalid? What do you think the correct resolved path should be?

I've just posted out an update in 0.1.26, including some normalization adjustments. If that doesn't fix it for you please let me know, and I'll have another look at this.


IronicStandalone really needs init-Pod(ipa-downloader)?

Some times we can build a http server out of cluster and we can also ues it to be as a image-server. And we can upload kernal and ramdisk into it. So I think we may don't need the init-Pod in Ironic's Pod.

It is a  required option in our project. So I think we should provide users with a configurable item.

/triage accepted
I concur. We also don't use IPA-downloader in OpenShift. I'm not sure what the best API for that is, maybe as simple as a cacheAgentImage boolean flag.

In fact I would like to update official-docs to tell user. You should get and upload IPA to the HTTP server. But if one want to use it. we can define a acheAgentImage boolean flag.

@dtantsur could you assign it to me? I want to realize it.

/assign

@dtantsur @Rozzii Hi friends, I  find we remove some ironc's configuration  https://github.com/metal3-io/ironic-standalone-operator/pull/47. If someone want to upgrade the ironic we have to modiry the controller's deployment. IMO we set image through controller's env isn't a good idea.

As a controller, we should not intrude our code logic too much into CRD.

Hi! Could you open a new discussion here or on the public slack?
The removal was to follow the best practices of operators, as I got told several times by operator sdk folks.

I find we remove some ironc's configuration https://github.com/metal3-io/ironic-standalone-operator/pull/47. If someone want to upgrade the ironic we have to modiry the controller's deployment.

Unrelated to the bug, but just as a heads-up: we're going to reconsider this decision based on https://github.com/metal3-io/ironic-standalone-operator/discussions/64. Please stay tuned.


PR-less runs can't report comments

Hi, I have a scheduled test run
which executes this workflow that uses your action.
For example see https://github.com/TWiStErRob/net.twisterrob.healthcheck/actions/runs/1832934306.
I don't understand why it doesn't happen every time, I didn't change anything for a while.
I just noticed that there are some issues:

log
Run EnricoMi/publish-unit-test-result-action@v1
/usr/bin/docker run --name ghcrioenricomipublishunittestresultactionv128_821dbf --label 7db2d7 --workdir /github/workspace --rm -e INPUT_CHECK_NAME -e INPUT_COMMENT_TITLE -e INPUT_REPORT_INDIVIDUAL_RUNS -e INPUT_TEST_CHANGES_LIMIT -e INPUT_FILES -e INPUT_GITHUB_TOKEN -e INPUT_GITHUB_RETRIES -e INPUT_COMMIT -e INPUT_FAIL_ON -e INPUT_TIME_UNIT -e INPUT_DEDUPLICATE_CLASSES_BY_FILE_NAME -e INPUT_HIDE_COMMENTS -e INPUT_COMMENT_ON_PR -e INPUT_COMMENT_MODE -e INPUT_COMPARE_TO_EARLIER_COMMIT -e INPUT_PULL_REQUEST_BUILD -e INPUT_EVENT_FILE -e INPUT_EVENT_NAME -e INPUT_CHECK_RUN_ANNOTATIONS -e INPUT_CHECK_RUN_ANNOTATIONS_BRANCH -e INPUT_SECONDS_BETWEEN_GITHUB_READS -e INPUT_SECONDS_BETWEEN_GITHUB_WRITES -e HOME -e GITHUB_JOB -e GITHUB_REF -e GITHUB_SHA -e GITHUB_REPOSITORY -e GITHUB_REPOSITORY_OWNER -e GITHUB_RUN_ID -e GITHUB_RUN_NUMBER -e GITHUB_RETENTION_DAYS -e GITHUB_RUN_ATTEMPT -e GITHUB_ACTOR -e GITHUB_WORKFLOW -e GITHUB_HEAD_REF -e GITHUB_BASE_REF -e GITHUB_EVENT_NAME -e GITHUB_SERVER_URL -e GITHUB_API_URL -e GITHUB_GRAPHQL_URL -e GITHUB_REF_NAME -e GITHUB_REF_PROTECTED -e GITHUB_REF_TYPE -e GITHUB_WORKSPACE -e GITHUB_ACTION -e GITHUB_EVENT_PATH -e GITHUB_ACTION_REPOSITORY -e GITHUB_ACTION_REF -e GITHUB_PATH -e GITHUB_ENV -e RUNNER_OS -e RUNNER_ARCH -e RUNNER_NAME -e RUNNER_TOOL_CACHE -e RUNNER_TEMP -e RUNNER_WORKSPACE -e ACTIONS_RUNTIME_URL -e ACTIONS_RUNTIME_TOKEN -e ACTIONS_CACHE_URL -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v "/home/runner/work/net.twisterrob.healthcheck/net.twisterrob.healthcheck":"/github/workspace" ghcr.io/enricomi/publish-unit-test-result-action:v1.28
[20](https://github.com/TWiStErRob/net.twisterrob.healthcheck/runs/5165723653?check_suite_focus=true#step:8:20)[22](https://github.com/TWiStErRob/net.twisterrob.healthcheck/runs/5165723653?check_suite_focus=true#step:8:22)-02-12 06:04:30 +0000 - publish-unit-test-results -  INFO - reading /github/workspace/build/test-results/*/TEST-*.xml
2022-02-12 06:04:30 +0000 - publish -  INFO - publishing success results for commit b03644b6a69fd6c83789676fddb57ad1474aa610
2022-02-12 06:04:31 +0000 - publish -  INFO - created check https://github.com/TWiStErRob/net.twisterrob.healthcheck/runs/5165733182
2022-02-12 06:04:[31](https://github.com/TWiStErRob/net.twisterrob.healthcheck/runs/5165723653?check_suite_focus=true#step:8:31) +0000 - publish-unit-test-results - WARNING - Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed: Forbidden
2022-02-12 06:04:31 +0000 - publish - WARNING - Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed with 403: Forbidden
2022-02-12 06:04:31 +0000 - publish-unit-test-results -  INFO - There is no Retry-After given in the response header
2022-02-12 06:04:31 +0000 - publish-unit-test-results -  INFO - Response body indicates retry-able error
2022-02-12 06:04:31 +0000 - publish-unit-test-results - WARNING - Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed: Forbidden
2022-02-12 06:04:31 +0000 - publish - WARNING - Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed with 403: Forbidden
2022-02-12 06:04:31 +0000 - publish-unit-test-results -  INFO - There is no Retry-After given in the response header
2022-02-12 06:04:31 +0000 - publish-unit-test-results -  INFO - Response body indicates retry-able error
2022-02-12 06:04:[35](https://github.com/TWiStErRob/net.twisterrob.healthcheck/runs/5165723653?check_suite_focus=true#step:8:35) +0000 - publish-unit-test-results - WARNING - Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b0[36](https://github.com/TWiStErRob/net.twisterrob.healthcheck/runs/5165723653?check_suite_focus=true#step:8:36)44b6a69fd6c8[37](https://github.com/TWiStErRob/net.twisterrob.healthcheck/runs/5165723653?check_suite_focus=true#step:8:37)89676fddb57ad1474aa610&per_page=100 failed: Forbidden
2022-02-12 06:04:35 +0000 - publish - WARNING - Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed with [40](https://github.com/TWiStErRob/net.twisterrob.healthcheck/runs/5165723653?check_suite_focus=true#step:8:40)3: Forbidden
2022-02-12 06:04:35 +0000 - publish-unit-test-results -  INFO - There is no Retry-After given in the response header
2022-02-12 06:04:35 +0000 - publish-unit-test-results -  INFO - Response body indicates retry-able error
2022-02-12 06:04:[44](https://github.com/TWiStErRob/net.twisterrob.healthcheck/runs/5165723653?check_suite_focus=true#step:8:44) +0000 - publish-unit-test-results - WARNING - Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1[47](https://github.com/TWiStErRob/net.twisterrob.healthcheck/runs/5165723653?check_suite_focus=true#step:8:47)4aa610&per_page=100 failed: Forbidden
2022-02-12 06:04:44 +0000 - publish - WARNING - Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed with 403: Forbidden
2022-02-12 06:04:44 +0000 - publish-unit-test-results -  INFO - There is no Retry-After given in the response header
2022-02-12 06:04:44 +0000 - publish-unit-test-results -  INFO - Response body indicates retry-able error
2022-02-12 06:05:00 +0000 - publish-unit-test-results - WARNING - Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed: Forbidden
2022-02-12 06:05:00 +0000 - publish - WARNING - Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed with 403: Forbidden
2022-02-12 06:05:00 +0000 - publish-unit-test-results -  INFO - There is no Retry-After given in the response header
2022-02-12 06:05:00 +0000 - publish-unit-test-results -  INFO - Response body indicates retry-able error
Warning: Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed with 403: Forbidden
Warning: Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed with 403: Forbidden
Warning: Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed with 403: Forbidden
Warning: Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed with 403: Forbidden
Warning: Request GET /search/issues?q=type%3Apr+repo%3A%22TWiStErRob%2Fnet.twisterrob.healthcheck%22+b03644b6a69fd6c83789676fddb57ad1474aa610&per_page=100 failed with 403: Forbidden

This happens regardless of failure and it gets reported as annotations:

I think the problem is that this scheduled run is happening on master which doesn't have a PR. Would it be possible to handle this case better?

recognize this and recommend some actions rather than retrying 5 times.
an option for changing the target where the comment goes.
put the comment into the commit rather than the PR?
i.e. open https://github.com/TWiStErRob/net.twisterrob.healthcheck/commit/b03644b6a69fd6c83789676fddb57ad1474aa610 and see the "comment section" at the bottom.

As a workaround for now I'll just disable with comment_mode: off to prevent this error, because there will never be a PR for master.

The problem is not that there is no PR in this case, but that the action cannot search for a PR (that is Forbidden). Even if there would be a PR, the same error would occur.
The question is: why is the search request failing with Forbidden? Also the Response body indicates retry-able error looks suspicious.
Could you please run the action with:
with:
  root_log_level: DEBUG
  log_level: DEBUG

Can you also provide a link to the failing workflow run? There might be other insights from the logs.

Looking into the code, the log message Response body indicates retry-able error only occurs when the response contains either of the following strings:

api rate limit exceeded
please wait a few minutes before you try again.

This indicates that the cause of the Forbidden response is that the rate limit for the search API is exceeded. That limit is very low, only 30 requests per minute. So there might be other actions in your workflow that use the search API extensively and leave no requests for this action.
Can you please run the following action? This shows the full response message:
uses: uses: EnricoMi/publish-unit-test-result-action/composite@master

I was able to reproduce the behaviour of the action with search rate limit exceeded. In #223 the behaviour is fixes in the sense that it waits until the rate limit is reset (>60s), so that only a single retry occurs. This will be merged into master soon. Release will happen later this month. Thanks for reporting this!

Hmm, didn't have time to try yet, sorry. Do you still need me to run debug or master? Based on what you said it might've been a one off. Which is weird because it runs every single day for months now. I'll try to look what triggered that many requests around the same time.


1.8: Moved 'make safety' to the end of the test workflow

Rolls back PR #464 into 1.8.1.

coverage: 33.944%. first build when pulling 3dae5b3aa47e6e1ed5584f7807608cb90bb9a9f6 on 1.8/andy/safety-end into b8dded4079521fd425b2580d16f6f86cca5a4a15 on stable_1.8.


Delegate and EWS

[Enter feedback here]
After contacting Microsoft support has given us the solution. Adding the user agent : SchedulingService in the EWSAllowList solve the problem for delegation.

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

ID: 37e10f52-d726-d29a-e0a8-93729a32dfae
Version Independent ID: 83555f80-a8ca-ee85-c1fc-e39891029758
Content: Use the Microsoft Teams Meeting add-in in Outlook - Microsoft Teams
Content Source: Teams/Teams-add-in-for-Outlook.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

must be added like this : SchedulingService

should be prefix and sufix with the "*" symbol

"SchedulingService"

@RomanNG Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@RomanNG thank you for pointing this out and helping improve MS Docs. It seems the doc has been updated/fixed in the interim. Please can you confirm.

@RomanNG  We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Change title of  "Connect Wallet" to "Claim"

Hi, can I work on this?

i would love to be a path of issue@https://github.com/maciejka

i would love to be a path of issue@https://github.com/maciejka

Hi, unfortunately this issue is already resolved.


Stil not updated on pub.dev, Please update on pub.dev to really use it easily and please use flutter 3.0.5

searched here for the latest fork
https://techgaun.github.io/active-forks/index.html#justkawal/excel
this looks good
https://github.com/justkawal/excel/compare/null-safety...dukefirehawk:excel:master

please tell how to use this?? extension/fork in my current project
On Wed, Sep 21, 2022 at 1:12 PM Ronjack @.***> wrote:

searched here for the latest fork
https://techgaun.github.io/active-forks/index.html#justkawal/excel
this looks good
null-safety...dukefirehawk:excel:master
https://github.com/justkawal/excel/compare/null-safety...dukefirehawk:excel:master
[image: image]
https://user-images.githubusercontent.com/35878088/186659801-0251ad20-d00a-4922-a15f-99417b703889.png
—
Reply to this email directly, view it on GitHub
https://github.com/justkawal/excel/issues/176#issuecomment-1253357933,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRXJSF5Z2LDRRPPROHTOTDV7K7PHANCNFSM57S25ODA
.
You are receiving this because you authored the thread.Message ID:
@.***>

any updateS???

on pub.dev

???

Can anyone reply

Omg thanks would help us a lot
On Fri, Dec 16, 2022, 10:29 PM Kawaljeet Singh @.***>
wrote:

Closed #176 https://github.com/justkawal/excel/issues/176 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/justkawal/excel/issues/176#event-8058618261, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRXJSA2TVJG4GZSLA5QDUDWNSRJFANCNFSM57S25ODA
.
You are receiving this because you authored the thread.Message ID:
@.***>

still not showing on pub.dev though

??

The latest release on pub.dev is 2.0.1 (https://pub.dev/packages/excel) and implements the features you wanted.
What version are you seeing?


GPS / Geo-Location data lost on edit

Reproduce

select Draw for an image with GPS data
enable Keep EXIF option so that metadata is kept
draw on image
tap disc icon to save image
see confirmation dialog

Currently
When looking at the edited image, the GPS data is gone.
Expected
All metadata should be kept.

Thanks for the response. When I open Settings > Image source I see Photo picker selected. Since that option mentions Has issues receiving EXIF metadata I switched to Gallery and repeated above steps but saved image still has no GPS data.

Hm, even with get content?

Third option works and keeps the GPS metadata 🎉
This is very helpful, been wondering about this for a long time.
Would it make sense to remember the EXIF setting last used? My personal use-case requires this to be on, but it is always off when I go into editing.

Hm, i can add an additional setting if you want :)

Not sure how this is best handled. Follow last used state avoids an additional settings option and my be sufficient. Certainly a setting is also possible - not sure which section would fit and adding yet another section to settings would make them even more crowded so not a fan. But if you have an existing category in mind sure.

In the additional exif category, i prefer this way because it's much easier to implement 👀

Sure, you're the boss and since I can't code happy either way 🚀
Do you want a separate issue for that feature request?

Yup, that's would be great 💯

Persisting in 2.6.0a3. Could you re-open this issue? Sadly none of the three image source options allows to keep the GPS information in the image when saving a draw edit. The timestamp was kept with all three options.
This feels like a problem in Android behavior.
Found a similar issue filed against Nextcloud and fm-sys pointed out that Stripping EXIF data is part of the android Q privacy feature set. Starting with Android 10 (or 11?), apps need to explicitly specify that they want to receive Metadata..
If you'd be willing to take a look how to specifically request to receive EXIF data when opening an image, that might solve this issue.
In comment https://github.com/nextcloud/android/issues/6248#issuecomment-1147167524 there are also three links about the behavior which are:

https://stackoverflow.com/questions/68442139/exif-location-data-removed-when-sharing-photo-via-intent
https://developer.android.com/reference/android/provider/MediaStore#EXTRA_ACCEPT_ORIGINAL_MEDIA_FORMAT
https://issuetracker.google.com/issues/128675076

Thanks, i will check

I did this i think, checked with latest image source and location not lost on android 14 😄


Global context not available in a loop?

If I have the following context:
{{model: {username: 'sontek'}, timezones: ['a', 'b', 'c']}

and I do:
{{#each timezones }}
     {{model.username}}
{{/each}}

model doesn't exist so I can't access the username.

I found I can use ../model.username but can't find docs on that, I just found it in some bug reports.... Is that documented somewhere?

It's in the docs under "Paths"

Nested handlebars paths can also include ../ segments, which evaluate their paths against a parent context.

Comments

  {{#each comments}}

{{title}}

{{body}}

  {{/each}}

Even though the link is printed while in the context of a comment, it can still go back to the main context (the post) to retrieve its permalink.
The ../ path segment references the parent template scope, not one level up in the context. This is because block helpers can invoke a block with any context, so the notion of "one level up" isn't particularly meaningful except as a reference to the parent template scope.


Moving to a proper logging system

http://wiki.ros.org/roscpp/Overview/Logging
http://wiki.ros.org/rospy/Overview/Logging
ROS has inbuilt support for Logging. We just have to use it.
We will have to port all of our code from existing cout, std::cout and print statements to ROS_INFO, ROS_DEBUG, etc.
It would be helpful if everyone quickly edited their projects and made these changes, and opened a PR. I would be able to merge them in fast.
Also, I would urge whoever is writing code from here on out to strictly use these instead of using stray cout and print statements in C++ and Python, respectively. (cc @kalyan-kumar , Barno and Priyank)
Opinions? @prudhvid @nevinvalsaraj @VishnuDuttSharma
cc @auviitkgp/developers @auviitkgp/embedded-team

Packages yet to port over to logging system

[ ] NqDVL /home/siddharth/ros_workspace/kraken_3.0/sensor_stack/NqDVL
[x] Scripts /home/siddharth/ros_workspace/kraken_3.0/Scripts
[ ] SerialPort /home/siddharth/ros_workspace/kraken_3.0/hardware_stack/SerialPort
[ ] Telemetry /home/siddharth/ros_workspace/kraken_3.0/gui_stack/Telemetry
[ ] absolute_rpy_publisher /home/siddharth/ros_workspace/kraken_3.0/control_system_stack/absolute_rpy_publisher
[ ] actionmsg /home/siddharth/ros_workspace/kraken_3.0/vision_stack/actionmsg
[ ] blob /home/siddharth/ros_workspace/kraken_3.0/vision_stack/blob
[ ] bottomcam /home/siddharth/ros_workspace/kraken_3.0/vision_stack/camera/bottomcam
[ ] control_server /home/siddharth/ros_workspace/kraken_3.0/control_system_stack/control_server
[ ] controller_basic /home/siddharth/ros_workspace/kraken_3.0/control_system_stack/controller_basic
[ ] depth_sensor /home/siddharth/ros_workspace/kraken_3.0/sensor_stack/depth_sensor
[ ] frontcam /home/siddharth/ros_workspace/kraken_3.0/vision_stack/camera/frontcam
[ ] gui_control /home/siddharth/ros_workspace/kraken_3.0/gui_stack/gui_control
[ ] gui_controller_loader /home/siddharth/ros_workspace/kraken_3.0/gui_stack/gui_controller_loader
[ ] gui_template /home/siddharth/ros_workspace/kraken_3.0/gui_stack/gui_template
[ ] ip_msgs /home/siddharth/ros_workspace/kraken_3.0/msgs_stack/ip_msgs
[ ] joystick_control /home/siddharth/ros_workspace/kraken_3.0/simulator_stack/joystick_control
[ ] keyboard_control /home/siddharth/ros_workspace/kraken_3.0/control_system_stack/keyboard_control
[ ] kraken_msgs /home/siddharth/ros_workspace/kraken_3.0/msgs_stack/kraken_msgs
[ ] kraken_visualization /home/siddharth/ros_workspace/kraken_3.0/simulator_stack/kraken_visualization
[ ] manual_control /home/siddharth/ros_workspace/kraken_3.0/control_system_stack/manual_control
[ ] mission_planner /home/siddharth/ros_workspace/kraken_3.0/mission_planner_stack/mission_planner
[ ] nqDVL /home/siddharth/ros_workspace/kraken_3.0/sensor_stack/nqDVL
[ ] pose_server /home/siddharth/ros_workspace/kraken_3.0/mission_planner_stack/pose_server
[ ] pose_server_python /home/siddharth/ros_workspace/kraken_3.0/mission_planner_stack/pose_server_python
[ ] premap /home/siddharth/ros_workspace/kraken_3.0/mission_planner_stack/premap
[ ] resources /home/siddharth/ros_workspace/kraken_3.0/resources
[ ] seabotix /home/siddharth/ros_workspace/kraken_3.0/sensor_stack/seabotix
[ ] simulator_physics_model /home/siddharth/ros_workspace/kraken_3.0/simulator_stack/simulator_physics_model
[ ] simulator_to_kmsgs /home/siddharth/ros_workspace/kraken_3.0/simulator_stack/simulator_to_kmsgs
[ ] simulator_to_navpose /home/siddharth/ros_workspace/kraken_3.0/simulator_stack/simulator_to_navpose
[ ] sparton /home/siddharth/ros_workspace/kraken_3.0/sensor_stack/sparton
[ ] task_buoy /home/siddharth/ros_workspace/kraken_3.0/vision_stack/task_buoy
[ ] task_marker /home/siddharth/ros_workspace/kraken_3.0/vision_stack/task_marker
[ ] thruster_converter /home/siddharth/ros_workspace/kraken_3.0/control_system_stack/thruster_converter
[ ] thruster_force_converter /home/siddharth/ros_workspace/kraken_3.0/simulator_stack/thruster_force_converter
[ ] tracks_imu /home/siddharth/ros_workspace/kraken_3.0/sensor_stack/tracks_imu
[ ] trax /home/siddharth/ros_workspace/kraken_3.0/sensor_stack/trax
[ ] vgateclient /home/siddharth/ros_workspace/kraken_3.0/vision_stack/validationgate/vgateclient
[ ] vgatedetect /home/siddharth/ros_workspace/kraken_3.0/vision_stack/validationgate/vgatedetect
[ ] vgateserver /home/siddharth/ros_workspace/kraken_3.0/vision_stack/validationgate/vgateserver
[ ] videoread /home/siddharth/ros_workspace/kraken_3.0/vision_stack/camera/videoread
[ ] vision_gui /home/siddharth/ros_workspace/kraken_3.0/vision_stack/vision_gui

update regarding log levels

Print all debug messages using the rospy.logdebug or the ROS_DEBUG function.
Also, if the CLI argument --debug or --verbose is passed to the executable call statement, then initialise the node with loglevel as DEBUG, so that the /rosout topic will become the central place to recieve all log information.

cc @auviitkgp/developers @auviitkgp/embedded-team @auviitkgp/maintainers

I have abstracted the verbose tag code as a function in the resources package, under the tools namespace for CPP and under resource.tools for Python.
Refer to https://github.com/icyflame/kraken_3.0/commit/857f5ab9052491525dd85577a068103018bdc384 for CPP.
Refer to https://github.com/icyflame/kraken_3.0/commit/168940aac3e9099a758b3530ff23bc3d00286163 for Python.
cc @auviitkgp/developers @auviitkgp/embedded-team

@icyflame :+1: nice work. And also assign this task to others. You don't do the shift all alone.
@kalyan-kumar @ManviG @pranaypratyush  you take some of the other packages and shift them to the new logging system.

References for C++ and Python

C++ - Line number 20 and Line numbers 100-103

Python - Line number 29 and Line number 68


Handle exceeding the localStorage quotum

Out of curiosity, does this library handle the (unlikely) event of a user exceeding the localStorage quotum in a session? From what I understand (http://stackoverflow.com/a/14191200/5918874) in that case an exception will be thrown and no new data will be written.
Theoretically it could be possible since users can set the maximum themselves, or the device could be limited in available space.

On last version It fails completely when quotum is exceeded.
It's a real-world problem because some browsers have no quotum (private navigation in Safari for ex.)

For anyone having the same problem, here is my dirty solution :
import _ from "lodash"
import persistState, { mergePersistedState } from 'redux-localstorage'
import adapter from 'redux-localstorage/lib/adapters/localStorage';

const storage = window.localStorage

...

const isStorageOk = () => {
  try {
    storage.setItem("test", "1");
    storage.removeItem("test");
    return true
  }
  catch(e) {
    console.warn("No local storage", e)
  }
}

const storageAdapter = adapter(storage)

const enhancers = _.compact([
  applyMiddleware(...middlewares),
  isStorageOk() && persistState(storageAdapter, storageKey)
])

const enhancer = composeEnhancers(...enhancers)

export const store = createStore(reducer, enhancer)


Feature Request: Unofficial builds

@ljharb We recently talked about how it makes no sense to have a POSIX compliant installer, since on embedded systems you most likely need to compile node yourself, so the benefit of not installing bash is neglectable.
The reason why nvm doesn't work on Alpine Linux, is because the official builds are compiled against glibc, but there are unofficial builds for musl that should work.
It would be beneficial to have a way to define a different source for node versions, so that I can use nvm on Alpine in the first place.
node releases on the Alpine package manager are tied to repository branches, meaning accessing an older node version means to run an ancient OS as well. I'd love if you could look into it.
Related: https://github.com/nodejs/build/issues/1140

Note: I put "PR wanted" on this, but I'm still very unclear about whether this can be added cleanly. I'd love to see a suggested implementation.

That would be also very useful for RISC-V machines, I see that files like node-v20.5.1-linux-riscv64.tar.xz are available for download, but nvm still compiles everything from sources, which takes quite a lot of time.
Is there anything that can be done to speed up installs on RISC-V machines?

FWIW, the override in https://github.com/nvm-sh/nvm/pull/3251 (which we're applying to our local install of nvm as a patch) is working well for us on CentOS 7, allowing us to install unofficial builds, check for updates etc. Without being able to add the linux-x64-glibc-217 suffix, it would not work at all.
Obviously this (being stuck on CentOS 7) is not a situation we want to be in long term, but for now this solution is preventing major dependency update headaches.


Fixing deprecation of contains

#fix#
Changelog

Use includes method over contains to fix deprecation in Ember 2.8

This change is

Awesome, thanks!

Approved

Changes Unknown when pulling 23fe76242a6308912657c2fb03ab26d0c2033b70 on thoov:patch-1 into ** on ciena-blueplanet:master**.

This will break < stable, so should either be versioned as a breaking change or do one of the following:

instead check for the presence of includes before invoking it and otherwise falling back to contains.
include the ember-cli addon for the includes polyfill

I will add the polyfill

Changes Unknown when pulling 37d35ba8a8b914a55723a85eb3bdd002edbdab35 on thoov:patch-1 into ** on ciena-blueplanet:master**.

Changes Unknown when pulling 5f7d3edc0e78e76ff5ac9a158090ae2b247e2e3d on thoov:patch-1 into ** on ciena-blueplanet:master**.

@jasonmit good catch - much prefer to not do a major until absolutely necessary

Approved


Upgrade the swagger-tools version to 0.10.3 for async controllers

Current behaviour
See this stackoverflow thread : here
What I have done to make the async controller working
As you can see in my answer in the previous stackoverflow thread, upgrading to<EMAIL_ADDRESS>fixed the error and now all is passing and working: I can successfully use an async controller. Do you think that upgrading to this version will break something elsewhere?

Sorry, I have no idea. I'd consider upgrading to the latest release of this library that uses the new Sway library instead.


Please redact age report URL from error output

Hello, age author here. A few of your users are getting errors which include a https://filippo.io/age link, and they are following that link seeking support on my issue tracker.
https://github.com/FiloSottile/age/issues/544
I obviously can't help them. Would you mind redacting those links?

I apologize for this mistake. We'll fix it. Thank you!

Just chiming in to say that it might be nice of you to consider including github.com/FiloSottile/age in the eligible repos since you use it for the drop

The age link is still displayed (see https://github.com/FiloSottile/age/issues/553), and we're still getting reports, which is frankly getting annoying.

@elshan-eth I opened the latest issue what is the workaround this ?
this claiming process is def annoying is there a way to do a github auth on the ui and claim tokens that way ?

I am claiming my rewards only because it's @FiloSottile's code, definitely should be included!


GEODE-8250: Create new custom log config acceptance tests

Resubmit new acceptance tests after reverting initial commit.
ServerWithCustomLogConfigAcceptanceTest was failing on Windows because it was attempting to use locators="" in the command to launch the Server. Empty string is the default so explicitly specifying this is not necessary. I deleted it and the test started passing on Windows.

AcceptanceTest job hit a SIGABRT for invalid pointer. StatusLocatorExitCodeAcceptanceTest also failed -- this PR doesn't change that test.

:geode-assembly:docker SIGABRT. I think we just need to rerun tests again and hopefully Docker will behave.


imon-percent widget

A widget that displays percentage imon-data in the form of icons (units), similar to other widgets like percent-online.
The user can:

Choose any country/region with any indicator that has "%" in the `displaySuffix' field and whose max is no more than 100. If the chosen combination does not have any data, an error is shown in the settings.
Resize the widget without distorting it (and increasing/decreasing the size of the icon/value as a result).
Choose a color for the "filled" icons/the icons that represent the value.

Unresolved issues:

The api does not have a flag/indicate what type of percentage it is, so for now any indicator that could use another icon that the default 'user' icon is in Settings.icons.

Looks great, Mariam!


input-class-name with object not working

:input-class-name="{'error' : true}"

input-class-name accepts only string for now. I don't think there will be support for a different type anytime soon.


vivo.sx new/obfuscated InitializeStream()

[x] I've verified and I assure that I'm running youtube-dl 2018.11.07

[x] At least skimmed through the README, most notably the FAQ and BUGS sections

[x] Searched the bugtracker for similar issues including closed ones

[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'http://vivo.sx/d7ddda0e78']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.11.07
[debug] Python version 2.7.15rc1 (CPython) - Linux-4.15.0-36-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.4-0ubuntu<IP_ADDRESS>, ffprobe 3.4.4-0ubuntu<IP_ADDRESS>, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[Vivo] d7ddda0e78: Downloading webpage
ERROR: Unable to extract stream; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 792, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 508, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/shared.py", line 22, in _real_extract
    video_url = self._extract_video_url(webpage, video_id, url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/shared.py", line 92, in _extract_video_url
    webpage, 'stream', group='url'),
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 983, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract stream; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

It's not released.


PLAT-49774: Add wrap prop to VirtualList

Issue Resolved / Feature Added
For Easy TV, VirtualList and VirtualGridList are required to support wrap-around navigation by 5-way keys.
In this PR, VirtualListEasy and VirtualGridListEasy are added to moonstone. The new prop wrap is added. The acceptable value should be one of true, false, and 'noAnimation'

false (default): the spotlight deos not move to the opposite side.
true: the spotlight moves to the opposite side with an animation when it reaches the starting point or the ending point.
'noAnimation': a list is not animated but it jumps when the spotlight moves in wraparound manner.

Links
PLAT-49774
Comments
Enact-DCO-1.0-Signed-off-by: Seungcheon Baek<EMAIL_ADDRESS>

LGTM

@0x64
I forgot that we needed to update CHANGELOG.md. Could you do it?

LGTM

LGTM


Don't lose 'focus'

When I have duplicated one or more blocks, I have lost focus, i.e., it exits the Journal and I would like to continue editing on the recently duplicated blocks. I waste time re-entering to edit the desired block.
(Or something I making bad)

hi, sorry for the late response! i've added an option for this in v.1.2.1. please update the plugin and enable this feature in the settings


luci-app-statistics: nut: Add additional stats some UPSes report

Different UPSes report differents stats; here we add graphing
of some additional statistics supports by some models.
Signed-off-by: Daniel Dickinson<EMAIL_ADDRESS>

any comments?

LGTM, merged.


Create datamodel base classes based on openbisschema

Closes #3

Everything seems correct, missing the types that I will automate using scripts. I would like to review the tests together in the future, but the branch can be merged.

Ok, thanks for the review!
I think everything would make more sense with some documentation. I can dedicate myself to do this later this week, so next year will be, hopefully, ready.


Update spinner

Update the spinner styles to match design:
https://www.figma.com/file/xZ0IW0MJO0vnFicmrHiKaY/Components-Post?type=design&node-id=16527-53294&mode=design&t=87EdEEuV6V0XjvFY-0
@rouvenpost created a dynamic spinner image that can be implemented.
Files: SwissPostSpinner.zip
Antialiasing on the apng and gif assets is really horrible, try to implement the svg variant.

Should solve https://github.com/swisspost/design-system/issues/226
What about https://github.com/swisspost/design-system/issues/2439 ?


Create alternative Crossref DOI deposit which excludes chapters

The assumption in the existing Crossref DOI deposit logic is that if a publisher's bothered to enter chapters for a work at all, they must be wanting to register them with Crossref individually - so a lack of a chapter DOI is interpreted as an accidental omission.
Some publishers might want to both record their chapter data (without DOIs) and output chapter-less Crossref XML - but publishers who do want to assign DOIs to all their chapters would still want to see an error message if they'd missed one by accident.
Perhaps what we can do is to create a new output, “Crossref DOI deposit minimised”, that only outputs the book record.

good idea - maybe we can call it explicitly "CrossRef DOI deposit without
chapters"
also something to think about for Thoth+  Some publishers may wish to have
chapter level DOIs only for edited works but not for monographs - so
different deposits for different types of works. (I think punctum works
like that).
R
On Thu, Feb 8, 2024 at 10:56 AM rhigman @.***> wrote:

The assumption in the existing Crossref DOI deposit logic is that if a
publisher's bothered to enter chapters for a work at all, they must be
wanting to register them with Crossref individually - so a lack of a
chapter DOI is interpreted as an accidental omission.
Some publishers might want to both record their chapter data (without
DOIs) and output chapter-less Crossref XML - but publishers who do want to
assign DOIs to all their chapters would still want to see an error message
if they'd missed one by accident.
Perhaps what we can do is to create a new output, “Crossref DOI deposit
minimised”, that only outputs the book record.
—
Reply to this email directly, view it on GitHub
https://github.com/thoth-pub/thoth/issues/551, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFECW6IOOWK6FLFCWKMTN3YSSVOJAVCNFSM6AAAAABC7QPIMOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEZDIOBYGUZDENY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Dr Rupert Gatti
Director
Open Book Publishers
tel: +44 1223 331484
skype: jrupertjg
www.openbookpublishers.com

Related but different use case raised by @amandasramalho : a work might contain some chapters which need to have DOIs assigned, and others which don't. The preferred output would be a Crossref record containing only those chapters for which a DOI had been entered in Thoth (the current output fails assuming that the empty DOIs were errors, and the proposed new output would omit chapters entirely).

Perhaps it'd be better to simply exclude chapters that don't have a DOI and not assume the omission is an error altogether, instead of creating a separate deposit file


Change location of c.cs that is referenced in docs

New location of C# API is https://github.com/KxSystems/csharpkdb
This doc https://code.kx.com/q/interfaces/csharp-client-for-q/ currently points to https://github.com/KxSystems/kdb/blob/master/c/c.cs in the first sentance. Should be changed to https://github.com/KxSystems/csharpkdb
Thanks

Fixed – thanks for catch.

Fixed – thanks for catch.