[BUG] System.Collections.Generic.KeyNotFoundException: The given key 'content-platform-dictionaries.zip' was not present in the dictionary.

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[X] I did not answer truthfully to all the above checkboxes.

Bloxstrap Version
v2.8.0
What problem did you encounter?
it aint startin gng fix it
Bloxstrap Log
2024-10-31T20:02:51Z [App::OnStartup] Starting Bloxstrap v4.2.0
2024-10-31T20:02:51Z [App::OnStartup] Compiled Thursday, 17 October 2024 at 3:46:36 AM from LUKECOMPUTER\minet
2024-10-31T20:02:51Z [App::OnStartup] Loaded from %UserProfile%\AppData\Local\Bloxstrap\Bloxstrap.exe
2024-10-31T20:02:51Z [App::OnStartup] Temp path is %UserProfile%\AppData\Local\Temp\Bloxstrap
2024-10-31T20:02:51Z [App::OnStartup] WindowsStartMenu path is %UserProfile%\AppData\Roaming\Microsoft\Windows\Start Menu\Programs
2024-10-31T20:02:51Z [Logger::Initialize] Initializing at %UserProfile%\AppData\Local\Bloxstrap\Logs\Bloxstrap_20241031T200251Z.log
2024-10-31T20:02:51Z [Logger::Initialize] Finished initializing!
2024-10-31T20:02:51Z [JsonManager<Settings>::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\Settings.json...
2024-10-31T20:02:51Z [JsonManager<Settings>::Load] Loaded successfully!
2024-10-31T20:02:51Z [JsonManager<State>::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\State.json...
2024-10-31T20:02:51Z [JsonManager<State>::Load] Loaded successfully!
2024-10-31T20:02:51Z [FastFlagManager::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\Modifications\ClientSettings\ClientAppSettings.json...
2024-10-31T20:02:51Z [FastFlagManager::Load] Loaded successfully!
2024-10-31T20:02:54Z [LaunchHandler::LaunchRoblox] Initializing bootstrapper
2024-10-31T20:02:54Z [LaunchHandler::LaunchRoblox] Initializing bootstrapper dialog
2024-10-31T20:02:54Z [Bootstrapper::Run] Running bootstrapper
2024-10-31T20:02:54Z [Bootstrapper::SetStatus] Connecting to Roblox...
2024-10-31T20:02:54Z [Deployment::TestConnection<https://setup.rbxcdn.com>] Connecting...
2024-10-31T20:02:54Z [HttpClientLoggingHandler::ProcessRequest] GET https://setup.rbxcdn.com/versionStudio
2024-10-31T20:02:54Z [Deployment::InitializeConnectivity] Testing connectivity...
2024-10-31T20:02:55Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://setup.rbxcdn.com/versionStudio
2024-10-31T20:02:55Z [Deployment::InitializeConnectivity] Got https://setup.rbxcdn.com as the optimal base URL
2024-10-31T20:02:55Z [Bootstrapper::Run] Connectivity check finished
2024-10-31T20:02:55Z [Bootstrapper::CheckForUpdates] Checking for updates...
2024-10-31T20:02:55Z [HttpClientLoggingHandler::ProcessRequest] GET https://api.github.com/repos/bloxstraplabs/bloxstrap/releases/latest
2024-10-31T20:02:56Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://api.github.com/repos/bloxstraplabs/bloxstrap/releases/latest
2024-10-31T20:02:56Z [Bootstrapper::CheckForUpdates] No updates found
2024-10-31T20:02:56Z [Bootstrapper::GetLatestVersionInfo] Got channel as production
2024-10-31T20:02:56Z [Deployment::GetInfo] Getting deploy info for channel production
2024-10-31T20:02:56Z [HttpClientLoggingHandler::ProcessRequest] GET https://clientsettingscdn.roblox.com/v2/client-version/WindowsPlayer
2024-10-31T20:02:56Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://clientsettingscdn.roblox.com/v2/client-version/WindowsPlayer
2024-10-31T20:02:56Z [RegistryKeyEx::SetValueSafe] Writing '' to HKEY_CURRENT_USER\SOFTWARE\ROBLOX Corporation\Environments\RobloxPlayer\Channel\www.roblox.com
2024-10-31T20:02:56Z [HttpClientLoggingHandler::ProcessRequest] GET https://setup.rbxcdn.com/version-b7eebc919e96477a-rbxPkgManifest.txt
2024-10-31T20:02:56Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://setup.rbxcdn.com/version-b7eebc919e96477a-rbxPkgManifest.txt
2024-10-31T20:02:56Z [Bootstrapper::SetStatus] Upgrading {product}...
2024-10-31T20:03:01Z [Bootstrapper::DownloadPackage.RobloxApp.zip] Package is already downloaded, skipping...
2024-10-31T20:03:01Z [Bootstrapper::ExtractPackage] Extracting RobloxApp.zip...
2024-10-31T20:03:01Z [Bootstrapper::DownloadPackage.content-avatar.zip] Package is already downloaded, skipping...
2024-10-31T20:03:01Z [Bootstrapper::DownloadPackage.content-configs.zip] Package is already downloaded, skipping...
2024-10-31T20:03:01Z [Bootstrapper::DownloadPackage.content-fonts.zip] Package is already downloaded, skipping...
2024-10-31T20:03:01Z [Bootstrapper::DownloadPackage.content-models.zip] Package is already downloaded, skipping...
2024-10-31T20:03:01Z [Bootstrapper::DownloadPackage.content-sky.zip] Package is already downloaded, skipping...
2024-10-31T20:03:01Z [Bootstrapper::DownloadPackage.content-sounds.zip] Package is already downloaded, skipping...
2024-10-31T20:03:01Z [Bootstrapper::ExtractPackage] Extracting content-avatar.zip...
2024-10-31T20:03:01Z [Bootstrapper::ExtractPackage] Extracting content-configs.zip...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.shaders.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.ssl.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.content-textures2.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.content-textures3.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.content-terrain.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.content-platform-fonts.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.content-platform-dictionaries.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.extracontent-places.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.extracontent-luapackages.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.extracontent-translations.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.extracontent-models.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.extracontent-textures.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.WebView2.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.WebView2RuntimeInstaller.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::DownloadPackage.redist.zip] Package is already downloaded, skipping...
2024-10-31T20:03:02Z [Bootstrapper::SetStatus] Configuring {product}...
2024-10-31T20:03:02Z [Bootstrapper::ExtractPackage] Finished extracting content-configs.zip
2024-10-31T20:03:02Z [Bootstrapper::ExtractPackage] Extracting content-fonts.zip...
2024-10-31T20:03:03Z [Bootstrapper::ExtractPackage] Finished extracting content-avatar.zip
2024-10-31T20:03:03Z [Bootstrapper::ExtractPackage] Extracting content-models.zip...
2024-10-31T20:03:03Z [Bootstrapper::ExtractPackage] Finished extracting content-models.zip
2024-10-31T20:03:03Z [Bootstrapper::ExtractPackage] Extracting content-sky.zip...
2024-10-31T20:03:04Z [Bootstrapper::ExtractPackage] Extracting content-sounds.zip...
2024-10-31T20:03:04Z [Bootstrapper::ExtractPackage] Finished extracting content-sky.zip
2024-10-31T20:03:04Z [Bootstrapper::ExtractPackage] Extracting shaders.zip...
2024-10-31T20:03:04Z [Bootstrapper::ExtractPackage] Finished extracting RobloxApp.zip
2024-10-31T20:03:04Z [Bootstrapper::ExtractPackage] Extracting ssl.zip...
2024-10-31T20:03:04Z [Bootstrapper::ExtractPackage] Finished extracting content-sounds.zip
2024-10-31T20:03:04Z [Bootstrapper::ExtractPackage] Extracting content-textures2.zip...
2024-10-31T20:03:04Z [Bootstrapper::ExtractPackage] Finished extracting ssl.zip
2024-10-31T20:03:04Z [Bootstrapper::ExtractPackage] Extracting content-textures3.zip...
2024-10-31T20:03:05Z [Bootstrapper::ExtractPackage] Extracting content-terrain.zip...
2024-10-31T20:03:05Z [Bootstrapper::ExtractPackage] Finished extracting content-terrain.zip
2024-10-31T20:03:05Z [Bootstrapper::ExtractPackage] Extracting content-platform-fonts.zip...
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Finished extracting content-fonts.zip
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Extracting extracontent-places.zip...
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Finished extracting shaders.zip
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Extracting extracontent-luapackages.zip...
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Finished extracting content-textures3.zip
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Extracting extracontent-translations.zip...
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Finished extracting content-platform-fonts.zip
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Finished extracting extracontent-translations.zip
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Extracting extracontent-models.zip...
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Finished extracting extracontent-places.zip
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Extracting extracontent-textures.zip...
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Finished extracting extracontent-models.zip
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Extracting WebView2.zip...
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Finished extracting WebView2.zip
2024-10-31T20:03:06Z [Bootstrapper::ExtractPackage] Extracting redist.zip...
2024-10-31T20:03:07Z [Bootstrapper::ExtractPackage] Finished extracting extracontent-luapackages.zip
2024-10-31T20:03:07Z [Bootstrapper::ExtractPackage] Finished extracting redist.zip
2024-10-31T20:03:27Z [Bootstrapper::ExtractPackage] Finished extracting extracontent-textures.zip
2024-10-31T20:03:30Z [Bootstrapper::ExtractPackage] Finished extracting content-textures2.zip
2024-10-31T20:03:30Z [LaunchHandler::LaunchRoblox] Bootstrapper task has finished
2024-10-31T20:03:30Z [LaunchHandler::LaunchRoblox] An exception occurred when running the bootstrapper
2024-10-31T20:03:30Z [App::FinalizeExceptionHandling] System.Collections.Generic.KeyNotFoundException: The given key 'content-platform-dictionaries.zip' was not present in the dictionary.
   at System.Collections.Generic.Dictionary`2.get_Item(TKey key)
   at Bloxstrap.Bootstrapper.ExtractPackage(Package package, List`1 files)
   at Bloxstrap.Bootstrapper.<>c__DisplayClass33_1.<UpgradeRoblox>b__6()
   at System.Threading.Tasks.Task.InnerInvoke()
   at System.Threading.Tasks.Task.<>c.<.cctor>b__272_0(Object obj)
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location ---
   at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot, Thread threadPoolThread)
--- End of stack trace from previous location ---
   at Bloxstrap.Bootstrapper.UpgradeRoblox()
   at Bloxstrap.Bootstrapper.Run()
   at Bloxstrap.Bootstrapper.Run()

update to 2.8.1


add support for usernames and passwords

@oferitz Changes have been made. However, I was unable to get an edited connection to save. I believe I have resolved the issue, but please review connectionStore.updateConnection

@andyburke The isExist statement in the update method is actually necessary. I guess you removed this because you weren't able to update username and password? The reason is that each connection has unique id which created from the name + address and since they didn't change and the if rule was resulted with negative condition. In order to solve it you need to add the user name and the password as params to the getHash function both on creating method and the update one. This is probably not an ideal solution as more and more params will be added in the future, but for now lets go with that solution, unless you got better idea.

Could you explain why !isExist would be the correct case for an update? I can understand that when you create a new connection, but if anything, it seems like if !isExist during an update should be an error. (That is, the connection should exist if you're trying to update it.)
If the uid is the id of the connection, why does there need to be an isExist and a gethash call at all?
Trying to understand.

@andyburke Sure! That's a valid question. The motivation behind this was to prevent the user from creating duplicate connections by editing connection with different attributes to be the same as already existing connection. For example lets say you have two connections:
A: {name: local: address: <IP_ADDRESS>}
B: {name: local1: address: <IP_ADDRESS>}
a user can edit local1 to be local and it will result two identical connections and that is what isExist is doing. In other words i want to help the user from making mistakes that might confuse him :)
I Agree that it might be over-engineering or superfluous and it's definitely not critical to allow duplicate connection. What do you think?

Although I understand what you're trying to prevent now, in my opinion that's actually confusing. It will just silently fail to update the connection they are trying to modify. Additionally, it means any time a connection is changed, you need to update the gathash() call as you outlined above. That seems like quite a bit of complexity and potential confusion to try to protect the user from themselves.
Personally, I would eliminate connectionId and rely on the connection id in the store.
Later, if there are bug reports about the use case you outline, I would add code to check if there are other similar connections and warn the user. But again, only if it became a widespread issue.
You're the contributor and get to decide. I would like to get some form of this PR in, though, as this currently blocks me from using my database where username/password are set. Let me know how you want to proceed.

@andyburke I'm fine with removing the isExist. Thanks for putting this together, hopping for more contributions in the future. Thanks!


Add 2 additional input fields so that foods that need amount and unit weight can be entered.

For foods like rice or quinoa, an amount and unit have to be considered. If no units are provided, the code will default to 100 g of that food item.

Done.


only cancel successive runs for pull requests

This should skip workflows only in pull requests, since the commit sha should be unique. In case the ci is run for the same commit sha (e.g. when creating new branches), I think it is fine to cancel the CI.
@da-woods this should help with https://github.com/cython/cython/pull/5006#issuecomment-1236300023

Thanks - let's give this a go. If this doesn't work I'll try the version given in the documentation https://docs.github.com/en/actions/using-jobs/using-concurrency#example-using-a-fallback-value

0.29.x 9b989f430


service type LoadBalancer does NOT work, when it does it's intermittent

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I've deployed Cilium on my k3s cluster like this:
helm upgrade --install cilium cilium/cilium --set operator.replicas=1 --set ipam.operator.clusterPoolIPv4PodCIDRList=<IP_ADDRESS>/16 --set ipv4NativeRoutingCIDR=<IP_ADDRESS>/16 --set ipv4.enabled=true --set loadBalancer.mode=dsr --set routingMode=native --set autoDirectNodeRoutes=true --set l2announcements.enabled=true --set kubeProxyReplacement=true --set k8sClientRateLimit.qps=50 --set k8sClientRateLimit.burst=100 --set k8sServiceHost=<IP_ADDRESS> --set k8sServicePort=6443 --set l2announcements.leaseDuration=3s --set l2announcements.leaseRenewDeadline=1s --set l2announcements.leaseRetryPeriod=200ms --set ingressController.Enabled=true --set enable-bgp-control-plane.enabled=true --namespace kube-system

Everything it's good:
root@raspberrypi4-5:~/cilium/manifests# cilium status
    /¯¯\
 /¯¯\__/¯¯\    Cilium:             OK
 \__/¯¯\__/    Operator:           OK
 /¯¯\__/¯¯\    Envoy DaemonSet:    disabled (using embedded mode)
 \__/¯¯\__/    Hubble Relay:       disabled
    \__/       ClusterMesh:        disabled

Deployment             cilium-operator    Desired: 1, Ready: 1/1, Available: 1/1
DaemonSet              cilium             Desired: 2, Ready: 2/2, Available: 2/2
Containers:            cilium             Running: 2
                       cilium-operator    Running: 1
Cluster Pods:          22/22 managed by Cilium
Helm chart version:
Image versions         cilium-operator    quay.io/cilium/operator-generic:v1.15.6@sha256:5789f0935eef96ad571e4f5565a8800d3a8fbb05265cf6909300cd82fd513c3d: 1
                       cilium             quay.io/cilium/cilium:v1.15.6@sha256:6aa840986a3a9722cd967ef63248d675a87add7e1704740902d5d3162f0c0def: 2

root@raspberrypi4-5:~# kubectl exec -it -n kube-system ds/cilium -- cilium-dbg status
Defaulted container "cilium-agent" out of: cilium-agent, config (init), mount-cgroup (init), apply-sysctl-overwrites (init), mount-bpf-fs (init), clean-cilium-state (init), install-cni-binaries (init)
KVStore:                 Ok   Disabled
Kubernetes:              Ok   1.29 (v1.29.6+k3s2) [linux/arm64]
Kubernetes APIs:         ["EndpointSliceOrEndpoint", "cilium/v2::CiliumClusterwideNetworkPolicy", "cilium/v2::CiliumEndpoint", "cilium/v2::CiliumNetworkPolicy", "cilium/v2::CiliumNode", "cilium/v2alpha1::CiliumCIDRGroup", "core/v1::Namespace", "core/v1::Pods", "core/v1::Service", "networking.k8s.io/v1::NetworkPolicy"]
KubeProxyReplacement:    True   [wlan0   <IP_ADDRESS> fe80::da3a:ddff:fe57:c42e (Direct Routing)]
Host firewall:           Disabled
SRv6:                    Disabled
CNI Chaining:            none
CNI Config file:         successfully wrote CNI configuration file to /host/etc/cni/net.d/05-cilium.conflist
Cilium:                  Ok   1.15.6 (v1.15.6-a09e05e6)
NodeMonitor:             Listening for events on 4 CPUs with 64x4096 of shared memory
Cilium health daemon:    Ok
IPAM:                    IPv4: 15/254 allocated from <IP_ADDRESS>/24,
IPv4 BIG TCP:            Disabled
IPv6 BIG TCP:            Disabled
BandwidthManager:        Disabled
Host Routing:            Legacy
Masquerading:            IPTables [IPv4: Enabled, IPv6: Disabled]
Controller Status:       99/99 healthy
Proxy Status:            OK, ip <IP_ADDRESS>, 0 redirects active on ports 10000-20000, Envoy: embedded
Global Identity Range:   min 256, max 65535
Hubble:                  Ok              Current/Max Flows: 4095/4095 (100.00%), Flows/s: 33.53   Metrics: Disabled
Encryption:              Disabled
Cluster health:          2/2 reachable   (2024-07-10T17:15:13Z)
Modules Health:          Stopped(0) Degraded(0) OK(11)

But now I want to expose my applications outside of the cluster and I'm not been able to do it so. For a while I've started suspecting the issue was some kind related to my wlan0 interface. I've put the cable on my node and set eth0 on cilium configmap:
root@raspberrypi4-5:~/cilium/manifests# cilium config view | grep -i eth0
device                                            eth0

After restart the pods as usual the curl from my machine it started working fine, but it was just for a couple os seconds.
root@raspberrypi4-5:~/cilium/manifests# k get svc -n default | grep -i LoadBalancer
my-nginx-cilium.   Ignoring endpoint" endpointID=1283 error="stat /var/run/cilium/state/1283/ep_config.h: no such file or directory" file-path=/var/run/cilium/state/1283/ep_config.h subsys=endpoint

At this moment my endpoint it's not accessible:
~|⇒ curl -vvvv <IP_ADDRESS>
*   Trying <IP_ADDRESS>:80...
* connect to <IP_ADDRESS> port 80 failed: Operation timed out
* Failed to connect to <IP_ADDRESS> port 80 after 75004 ms: Operation timed out
* Closing connection 0
curl: (28) Failed to connect to <IP_ADDRESS> port 80 after 75004 ms: Operation timed out

My current configuration:
root@raspberrypi4-5:~/cilium# cat manifests/CiliumL2AnnouncementPolicy-IPPool.yaml
---
apiVersion: cilium.io/v2alpha1
kind: CiliumLoadBalancerIPPool
metadata:
  name: default-pool
  namespace: kube-system
spec:
  cidrs:
    - start: <IP_ADDRESS>
      stop: <IP_ADDRESS>
---
apiVersion: cilium.io/v2alpha1
kind: CiliumL2AnnouncementPolicy
metadata:
  name: default-l2-announcement-policy
  namespace: kube-system
spec:
  externalIPs: true
  loadBalancerIPs: true

I've started a thread on slack channel for this topic which with the solution provided by HB worked temporarily:
https://cilium.slack.com/archives/C53TG4J4R/p1720482506442029
IDK where to look at it anymore.
Cilium Version
# cilium version
cilium-cli: v0.16.11 compiled with go1.22.4 on linux/arm64
cilium image (default): v1.15.6
cilium image (stable): v1.15.6
cilium image (running): 1.15.6

Kernel Version
# uname -a
Linux raspberrypi4-5 6.8.0-1006-raspi #6-Ubuntu SMP PREEMPT_DYNAMIC Mon Jun 17 11:30:51 UTC 2024 aarch64 aarch64 aarch64 GNU/Linux

Kubernetes Version
v1.29.6+k3s2

Regression
No response
Sysdump
cilium-sysdump-20240710-140712.zip
Relevant log output
No response
Anything else?
No response
Cilium Users Document

[X] Are you a user of Cilium? Please add yourself to the Users doc

Code of Conduct

[X] I agree to follow this project's Code of Conduct

[UPDATE] - I've set these flags into cilium-config Configmap:
root@raspberrypi4-5:~/cilium# cilium config view | egrep -i "enable-bpf-masquerade|enable-host-legacy-routing"
enable-bpf-masquerade                             true
enable-host-legacy-routing                        false

After rollout the pods:
kubectl -n kube-system rollout restart ds/cilium
kubectl rollout restart deployment -n kube-system cilium-operator

Things started working a little bit better. This time the service type LoadBalancer (<IP_ADDRESS>) worked for more than 20 minutes straight, after that it stopped working for no reason. I don't see anything in the logs.

[UPDATE] - After deleting and recreating the svc it started working again:
root@raspberrypi4-5:~# k get svc -n default | grep -i load
my-nginx-cilium                                     LoadBalancer   <IP_ADDRESS>   <IP_ADDRESS>   80:31851/TCP                 5m16s

root@raspberrypi4-5:~# k delete svc/my-nginx-cilium -n default --grace-period=0 --force
Warning: Immediate deletion does not wait for confirmation that the running resource has been terminated.  Any ideas?

[UPDATE] - I've rebuild the cluster (k3s) and disable wifi adapter (wlan0) and build the cluster with an static IP assigned to eth0. Life it's good again =)
 apt install rfkill
 rfkill block wifi

root@raspberrypi4-5:~# cat /etc/netplan/01-netcfg.yaml
network:
  ethernets:
    eth0:
     dhcp4: false
     addresses:
       - <IP_ADDRESS>/24
     routes:
     - to: default
       via: <IP_ADDRESS>
     nameservers:
       addresses:
         - <IP_ADDRESS>

root@raspberrypi4-5:~# k exec -it -n kube-system ds/cilium -- cilium-dbg status
Defaulted container "cilium-agent" out of: cilium-agent, config (init), mount-cgroup (init), apply-sysctl-overwrites (init), mount-bpf-fs (init), clean-cilium-state (init), install-cni-binaries (init)
KVStore:                 Ok   Disabled
Kubernetes:              Ok   1.29 (v1.29.6+k3s2) [linux/arm64]
Kubernetes APIs:         ["EndpointSliceOrEndpoint", "cilium/v2::CiliumClusterwideEnvoyConfig", "cilium/v2::CiliumClusterwideNetworkPolicy", "cilium/v2::CiliumEndpoint", "cilium/v2::CiliumEnvoyConfig", "cilium/v2::CiliumNetworkPolicy", "cilium/v2::CiliumNode", "cilium/v2alpha1::CiliumCIDRGroup", "core/v1::Namespace", "core/v1::Pods", "core/v1::Secrets", "core/v1::Service", "networking.k8s.io/v1::NetworkPolicy"]
KubeProxyReplacement:    True   [eth0   <IP_ADDRESS> fe80::da3a:ddff:fe57:c42c (Direct Routing)]
Host firewall:           Disabled
SRv6:                    Disabled
CNI Chaining:            none
CNI Config file:         successfully wrote CNI configuration file to /host/etc/cni/net.d/05-cilium.conflist
Cilium:                  Ok   1.15.6 (v1.15.6-a09e05e6)
NodeMonitor:             Listening for events on 4 CPUs with 64x4096 of shared memory
Cilium health daemon:    Ok
IPAM:                    IPv4: 17/254 allocated from <IP_ADDRESS>/24,
IPv4 BIG TCP:            Disabled
IPv6 BIG TCP:            Disabled
BandwidthManager:        Disabled
Host Routing:            Legacy
Masquerading:            BPF   [eth0]   <IP_ADDRESS>/16 [IPv4: Enabled, IPv6: Disabled]
Controller Status:       107/107 healthy
Proxy Status:            OK, ip <IP_ADDRESS>, 0 redirects active on ports 10000-20000, Envoy: embedded
Global Identity Range:   min 256, max 65535
Hubble:                  Ok              Current/Max Flows: 4095/4095 (100.00%), Flows/s: 24.23   Metrics: Disabled
Encryption:              Disabled
Cluster health:          2/2 reachable   (2024-07-15T14:51:07Z)
Modules Health:          Stopped(0) Degraded(0) OK(11)


Transition image to OSG 23 (SOFTWARE-5673)

CI is failing because https://github.com/opensciencegrid/build-container-action/pull/10 isn't merged and we haven't tagged a 0.6.1 yet


SketchCanvas gets cut off in landscape mode in iOS

In iOS, when I change the device orientation to landscape, my drawings are only visible on one half of the canvas.

I know the canvas is taking up the whole area, because I made the background colour of the SketchCanvas red.
It seems like the canvas size isn't being updated on rotation. The weird part is that if I redraw the paths from the landscape canvas onto another canvas, it shows the full drawing.

Release 0.7.0 fixes this issue. You can try it!

That's awesome!!! I just tried it out and it works!
Thanks for making another release by the way, this library is super cool! I work at a company called Dot Health, and I implemented a signature solution using your library! So thanks a lot!!

It is @jeanregisser's contribution! Say thanks to him!


Jenayah: Watch upstrackingit.com

Jenayah requests the watch of the watch_keyword upstrackingit\.com. See the MS search here and the Stack Exchange search here.

upstrackingit\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


IGNITE-17534 Introduce HashIndexStorage interface and test implementation

https://issues.apache.org/jira/browse/IGNITE-17534

Tests have failed. This can be fixed by adding UnlimitedBudgetConfigurationSchema.class to the list of the polymorphic configurations.


STT DEMO KWS -- OPUS audio file is supported in Firefox, however OPUS excluded it from the “Transcribe Audio” section, where WAV and FLAC are mentioned

Hi Leonid,
I uploaded a raw file 001.raw (note: I cannot attach it in this defect because file is not supported, however, I will send it to you, if needed), as per instructions in the “Transcribe Audio” section, however, I received a message that “Only WAV or FLAC or Opus files can be transcribed, please try another file format”.
a) Why isn't the word “browser” used in the message, as it was in a message re: FLAC audios (like this ... “Notice: browers do not support playing FLAC audio, so no audio will accompany the transcription”)?
b) Why is OPUS excluded from the “Transcribe Audio” section, where WAV and FLAC are mentioned?
c) Why isn't “Opus” in the error message in capital letter like, “WAV” and “FLAC” are? Shouldn't it be “OPUS”
d) Why aren't the two complete sentences in the error message separated by a period or semicolon, instead of a comma?
See attached spreadsheet for screen shots and details.
Please let me know if you have any questions or comments.
Alexandra
STT DEMO KWS -- OPUS audio file is supported in Firefox, however OPUS excluded it from Transcribe Audio section, where WAV and FLAC are mentioned.xlsx

Fixed. Please try it out.

Leonid,
Thank you for the fixes. I will close this issue now.
Summary:
a) OK, was not a problem.
b) Fixed, tested, and OK now.
c) Fixed, tested, and OK now.
d) Fixed, tested, and OK now.
Alexandra

Leonid,
One more thing ....
Can you please change this error message by removing the second "or"?
from: Only WAV or FLAC or OPUS files can be transcribed. Please try another file format.
to: Only WAV, FLAC, or OPUS files can be transcribed. Please try another file format.
Thank you,
Alexandra

Leonid,
I have successfully tested your fixes.
I will close this issue now.
Thank you,
Alexandra


docs: migrate to homebrew-core

relates to https://github.com/Homebrew/homebrew-core/pull/84359

related: https://github.com/cue-lang/cue/issues/1125
The same question would be how does this ensure the same binary is produced? I suspect that it cannot be guaranteed due to inability to ensure the same Go version is used.
See how the cue tap downloads the binaries: https://github.com/cue-lang/homebrew-tap/blob/master/cue.rb
There will also be the issue of effort to maintain in CUE's repos vs making PR's against Homebrew's. @myitcv has setup automation for the release process. Note that the tap is actually updated by the Goreleaser tool. https://github.com/cue-lang/cue/blob/master/.goreleaser.yml#L54
It is generally prefered to open an issue or discussion before opening a PR. Having cue unofficially in the homebrew tap while the official version is maintained through the cue tap may be an undesirable situation.

homebrew-core formulae is pretty well maintained. And the contribution to homebrew-core is very easy.
There are also couple of options of automate the version bump process as well, like:

https://github.com/mislav/bump-homebrew-formula-action
https://github.com/dawidd6/action-homebrew-bump-formula

And homebrew-core is also working on automate this process as well.

@chenrui333 It looks like homebrew does not like core formulas downloading binaries. Is that an accurate assessment of the preferences?
Per #1125 concerns, how can the formula ensure that the binaries have the same hash as the official releases when compiling ? My understanding is that the same Go version 1.x.y would need to be used.

how can the formula ensure that the binaries have the same hash as the official releases when compiling ?

Also we are not shipping software without responsibility, meaning, we can always patch the build and update if needed. (that being said, if there is any usage issue, we can troubleshot and patch the artifact)
Does that make sense?

Based on this file: https://github.com/Homebrew/homebrew-core/pull/84359/files
The Go version does not appear to be locked, so cue would be compiled with whichever version the user has on their system at the time of installing cue via homebrew.
cue-v0.4.0 is compiled with Go 1.15.8 (https://github.com/cue-lang/cue/blob/master/.github/workflows/release.yml#L22)
So if a user has the latest Go (v1.17.0 at the time of writing) then they will have a different binary. This seems like it will happen more often than not, considering Go releases patches pretty frequently. i.e. The latest Go version will be different from the one used to compile more days than it matches, and subsequently the homebrew installed version would be different from the official binary more days than not. This is why the official cue tap installs by downloading the binary rather than compiling.
Does this help clarify the same concern raised in #1125 ?
I actually don't think that the "using goreleaser to build would result in the same binary" holds. While it will guarantee the same dep hashes are embedded into the binary for provenance, the Go binary is paired with the standard library, which changes between versions. Thus it seems there will be different code being compiled.


I'm having this problem

python instagram.py
File "instagram.py", line 58
if all([not self.isFound, msg]):
^
IndentationError: unindent does not match any outer indentation level

I too have the same error.

@EminRma @SioVerino @dashacker4732 @importCTF
if you all people get any fix please tell
if there's a new method please help

@gitmnj I've never gotten a fix on that error, therefore I've given up on it. I would suggest you finding other projects about IG brute, but they must be maintained. Good luck!

Okay thanks but If you get any New Update Regarding it please share and help

guys, thats an indentation error. with a few spaces or backspaces you can fix it.
don't rely on others, if something doesn't work you can just fix it yourself :)

guys, thats an indentation error. with a few spaces or backspaces you can fix it.
don't rely on others, if something doesn't work you can just fix it yourself :)


Minimap only hides when side panels are in "side-stones arrangement"

http://prntscr.com/irmlch
This has to be toggle off for the minimap to hide.

I can't seem to convince the minimap to hide by toggling the "Sidepanels can be closed by the hotkeys" option. Looking at the code, it appears as though RESIZABLE_VIEWPORT_OLD_SCHOOL_BOX_GROUP_ID is just wrong. It's currently set to 161, but I'm seeing 164 in the inspector

Whoops. I've just been informed that it's the collapsible sidepanels themselves, not the hotkey toggle, that causes the problem (which, admittedly, makes more sense). The problem is still roughly the same though: the widget group:ID isn't right when sidepanels aren't collapsible. The minimap's widget group is RESIZABLE_VIEWPORT_BOTTOM_LINE_GROUP_ID (164) when collapsible panels are turned on, and RESIZABLE_VIEWPORT_OLD_SCHOOL_BOX_GROUP_ID when the panels aren't collapsible.


Add support for signing with ecdsa keys

Signing with Eliptic Curve DSA sertificates is a lot faster than using RSA.
This PR adds support for ECDSA and also adds benchmarks to show the performance win.
$ go test -bench=. -run=XXX -benchtime=30s
goos: linux
goarch: amd64
pkg: github.com/elazarl/goproxy
BenchmarkRsaTls-4<PHONE_NUMBER>36 ns/op
BenchmarkRsaX509-4<PHONE_NUMBER>72 ns/op
BenchmarkEcdsaTls-4        30000           1701068 ns/op
BenchmarkEcdsaX509-4      200000            340498 ns/op
PASS
ok      github.com/elazarl/goproxy      266.347s

By also adding PR #338, the benchmark numbers are as follows:
$ go test -bench=. -run=XXX -benchtime=30s
goos: linux
goarch: amd64
pkg: github.com/elazarl/goproxy
BenchmarkRsaTls-4           2000          19618000 ns/op
BenchmarkRsaX509-4          5000           9091853 ns/op
BenchmarkEcdsaTls-4        30000           1672054 ns/op
BenchmarkEcdsaX509-4      200000            325934 ns/op
PASS
ok      github.com/elazarl/goproxy      223.161s

As discussed, this is missing the point of having stable certificates for each site, given a stable CA.

I have added back the predictable certificate generation. I also changed the benchmarks to only benchmark the signing method.

Thanks. I need to rethink the stable CA. Not sure if its worth it. It seems like the serial number is wide enough to allow GUID generation without fearing collision. Maybe that's what we should do.


use light sensor instead of characteristic?

hello! i'm using this plugin for getting SOMA Tilt 2s onto Home Assistant. i can see in HomeKit that the light level is being exported (as apple writes it in 'lux' units), but when importing the devices into Home Assistant all i get is the main control of the blinds and the battery level. would it be possible to create an actual sensor "device" so i can use the status for automation? (like the battery is now)

Technically, yes.  In fact, I used to expose a separate Light Sensor service before v0.0.14.  Functionally, that didn’t work out (see also #3):

Home doesn’t like accessories with both an actuator and a sensor.  It shows a combined tile, but the “accessories” cannot be shown as separate tiles, requiring an additional touch before you can control the blinds;
The SOMA device doesn’t provide a lux value, and I don’t know how to convert the raw value reported by the solar panel to lux.  Neither do the otherwise very helpful folks at Wazombi labs;
The sensor actually measures the outside light level, not the room light level, so the sensor should be in a separate “room” as the blind.

So the choice was either to expose the light sensor as a separate accessory, or as an additional characteristic on the Window Covering service.  I chose the latter, so Home doesn’t show it (as it would be reporting an incorrect light level for the “room”), but you can use it for automations in Eve or Home+.  I would still recommend to use on-device triggers though, disabling or re-enabling these from HomeKit automations to handle “Watch TV” scene logic.
I don’t use nor support Home Assistant.  I would never suggest chaining Homebridge and Home Assistant, and would recommend a native integration to Home Assistant.  I won’t be making any changes solely for Home Assistant.

gotcha! perhaps make it configurable? fwiw this is the beta SOMA Tilt controller out there for Home Assistant too. additionally i do want to use the outside light level -- im using it as a way of telling if to close the blinds, but also as a signal for my curtains which dont have a sensor. finally im ok with an inaccurate lux value, so long as it's consistent.
just want to say, thank you a ton for working on this, super helpful! 🙏

As a workaround, you could use the on-device trigger to close the blinds, and an HA automation to close the curtains when the blinds are closed.  I love the idea to “set and forget” on-device automations, that work independently from any server.  The only reason I need integration is the TV scene, otherwise my Venetian blinds handle sunrise, sunset, and straight sunlight onto the window by themselves.


All new profiles for the Camunda Platform Helm chart

The content of this repository is pretty outdated as points to the old Zeebe Helm charts. I suggest to switch out the content for new profiles that we developed together with customers.
Also after merging this PR, the repository should be renamed to camunda-8-helm-profiles.

It would be a bit hard to review this in detail, but in general, I'd suggest making a structure in the repo dirs to make it easy to navigate and work with.

Not sure what I look for. What is now the difference to https://github.com/camunda-community-hub/camunda8-greenfield-installation can we merge both?

Please see my last two comments in: https://github.com/camunda/camunda-platform-helm/pull/373#issuecomment-1216434646

Another way to decide on this pull request is to ask: I anyone even feeling responsible as a maintainer for this repo, maybe @Zelldon? Or is this as abandoned as its commit history indicates and you're happy for @upgradingdave and me giving it a new try?

@falko in my opinion you could just go ahead and do what you want with the repo :sweat_smile: I would prefer one repo instead of the two (speaking of the profiles and the greenfield repo).

I would prefer one repo instead of the two (speaking of the profiles and the greenfield repo).

Let's discuss this in a separate issue so that we can make progress with this PR independently.

@falko do we still need the meeting tomorrow? :thinking:

Let's have a chat about the KIND and OpenShift profiles and the proposed repo merge.

I guess then @npepinpe should be invited as well :man_shrugging:

Merging now since all change requests have been resolved.


Batch size mismatch in the paper and the code

Wonderful work!
But I found  a batch size mismatch problem between paper and code. The batch size is reported as 16 and 16 for labeled and unlabeled data, but the batch size is implemented as 8 and 8 for labeled and unlabeled data in code.

Please follow the script and config files. The batch size in paper might not be changed since we have several updates after the paper and we will fix it.

Got it! Thanks.


File size validation issue

Hi,
I've got a weird bug when using ngf-min-width & ngf-min-height.
I have set the following ngf-min-width="850" ngf-min-height="210" value. When I drag a file (which is 800x533), the file pass the validation and I end up with the following results:
blobUrl: "..."
height: 732
name: "sample.jpg"
size: 94400
type: "image/jpeg"
webkitRelativePath: ""
width: 1099
The files[0] value doesn't return me the correct photo size. Seem to kind of zoom it as it's still the correct file ratio.
Can someone help me on this?

Can you attach your image here.

Yep, here's the file that give me the above result.
Thanks

The size of this file is "width":800,"height":533 so if you change the min-width from 850 to 800 it would be uploaded successfully on the demo page.

Thanks @danialfarid.
Can you also tell me why in my initial post ng-file-upload was returning me a width of 1099px instead of the real 800px width of the photo?
That was mostly my problem.

I don't think that would ever happen, if you can reproduce it let me know.

Ok thanks alot.


updates for block volumes support

This change is

Additional updates for raw block devices @zacharysarah @msau42 @mtanino please review

/assign

Improvement to the matrix can be left to a follow-up PR.


Need to dynamically change the fastselect url parameter

Hello,
I need to display the colleges based on the country. Initially i have select box to list the country and a text box to display the colleges in which the fastselect is enabled. When a user select the country, i need to pass the url with country name in the fastselect url parameter. How it can be implemented?

You can change the url with this code:
$('#fastselect').data('url', 'your_new_url' ).fastselect();
but i havent found the way to reload fastselect since this will only work once.

itis not working


Dashboard: Add Aggregator and Type to Graph struct.

Need to do some testing, I'll be back :)

Hah alright :)
--
Mark Smith
<EMAIL_ADDRESS>On Thu, Aug 11, 2016, at 08:22 AM, Otto Jongerius wrote:

Closed #51[1].
— You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub[2], or mute the
thread[3].

Links:

https://github.com/zorkian/go-datadog-api/pull/51
https://github.com/zorkian/go-datadog-api/pull/51#event-752819500
https://github.com/notifications/unsubscribe-auth/AAWD1MHzT-lOk0cixDULSP5PxceTYOdeks5qez4kgaJpZM4JiCkc


Brighter main icon

@rmcanany all the icons used in the software are in color-flat style; this is a style that I like and have used in the whole UI.
The main icon is in a different style, and I suggest brightening it a bit to match the other icons better.
My suggestion in the attachment

Housekeeper_Bright.zip

Sure, go for it.

Implemented with #143

Closing as complete.  Thank you!


check_port ignores the bind address

When you start Opencast, the start scripts (specifically check_port) first check whether the HTTP and HTTPS ports configured in etc/org.ops4j.pax.web.cfg are already in use. They do this by looking for any open connection with the appropriate port number, but don't look at the address bound in that connection. It should compare this to the address(-es) configured in the same file (after potentially resolving hostnames ...).
This is problematic for example in the following situation: I'm running a certain web service locally for my own personal use. "Naturally" it listens on 8080. Being an Opencast dev, I of course already foresee conflict, and I could just change that port, but I always have trouble coming up with random port numbers, plus they might just collide with something else. 8080 (or http-alt in IANA lingo) is the correct port for this service, as it is for Opencast. But luckily I can also just bind that service to an alternative loopback IP; in my case (for no particular reason) <IP_ADDRESS>.
While admittedly a bit esoteric, this works: You have the entire address space of <IP_ADDRESS>/8 as loopback addresses at your disposal, and each of those can accept connections on the same port at the same time. And considering that there is actually some semantic meaning to a lot of port numbers, this seems to me to be a more reliable way to disambiguate situations like this. :thinking:
Anyway, Opencast doesn't support this use case currently. :sweat_smile: I wanted to change that, but it turned out to be harder than I thought, so for now this issue will have to suffice.

I wonder if this is why I'm seeing conflicts with Chrome of all things...  Lately I've noticed that when starting Opencast it will sometimes say that 8080 is already in use, and will continue to do so until Chrome is closed.  Hrm...

Huh, that's weird. Haven't seen that. :thinking: Maybe an extension is running something on 8080 somewhere?

Probably, 8080 is too common a dev port for this to be missed by everyone else on the planet :)


[ZEPPELIN-5449] Zeppelin currently cannot do the CI under k8s environment

What is this PR for?
Zeppelin uses GitHub Action to complete Continuous integration (CI). Specifically, it has three workflows: core.yaml, frontend.yaml and rat.yaml. However, It is currently not possible to verify the functions of zeppelin in the k8s environment in these workflows.
This PR utilizes Minikube as the k8s environment for CI testing. Specifically, it defines a new GitHub Action workflow: k8s_ci_tests.yaml. It contains following key steps:

start Minikube (i.e., a single-node Kubernetes cluster) on the GitHub-hosted VM.
build Zeppelin and output the distribution tar.gz
download Spark and build image
build zeppelin image from the distribution tar.gz (Dockerfile is in scripts/docker/zeppelin/build-from-distribution/Dockerfile)
mvn testing

mvn test -DskipRat -Dtest=K8sMinikubeTestBasic,K8sMinikubeTestPySpark,K8sMinikubeTestSparkR -pl :launcher-k8s-standard -B
It starts and tests multiple interpreters (shell, python, spark, pyspark, sparkr, sparksql) in the Minikube cluster.

What type of PR is it?
[Improvement]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-5449

How should this be tested?

CI passed and manually tested

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Hi, @zjffdu
Thanks for the comments. I've made the following changes:

Move the test classes into module zeppelin-interpreter-integration
Reuse the org.apache.zeppelin.integration.MiniZeppelin but add a parameter to set the ZeppelinConfiguration.
Reuse the properties file about log4j.

ping

@rickchengx Could you rebase this PR ?

@zjffdu Sorry for the late response, I was busy with my school things. I have rebased my PR.


Error during Android compile

Hi!
today I have installed last version (1.6.2) and follow android installa guide (Without react-native-reanimated)
but when try to run the app returns this error:
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':react-native-mmkv:generateJsonModelDebug'.
> C:\MYPATH\node_modules\react-native-mmkv\android\CMakeLists.txt : C/C++ debug|armeabi-v7a : CMake Error at C:\MYPATH\node_modules\react-native-mmkv\android\CMakeLists.txt:22 (add_library):
    Syntax error in cmake code when parsing string
      C:\MYPATH\node_modules/react-native/ReactCommon/jsi/jsi/jsi.cpp

    Invalid character escape '\U'.

Thanks in advance!

Ok, I 've found the problem, but I don't know how can I fix for all...
in CMakeLists.txt file at line 22:

As you can see at #1 will print my PC path describe in #2
The result is:
C:\Users\USER\Documents\GENERIC_PATH\node_modules/react-native/ReactCommon/jsi/jsi/jsi.cpp
to work need to replace " \ " with " / "
the path is inside ${NODE_MODULES_DIR} var.
for now I have paste my direct path
C:/Users/USER/Documents/GENERIC_PATH/node_modules/react-native/ReactCommon/jsi/jsi/jsi.cpp
and all is work, but is a "FIX" only for my PC....

OK :D
I have write a replace syntax to normalize path for all PCs 👍
from line 15 replace with this:
if(${REACT_NATIVE_VERSION} LESS 66)
        string(REPLACE "\\" "/" NORMALIZE_PATH ${NODE_MODULES_DIR})
        set (
                INCLUDE_JSI_CPP
                "${NORMALIZE_PATH}/react-native/ReactCommon/jsi/jsi/jsi.cpp"
        )
endif()

add_library(reactnativemmkv  # <-- Library name
        SHARED
        src/main/cpp/cpp-adapter.cpp
        src/main/cpp/MmkvHostObject.cpp
        ${INCLUDE_JSI_CPP} # only on older RN versions
)

And I'll attach full file at this post.
CMakeLists.txt
Can you apply this patch at next release?
Thanks a lot!

Yes, all paths in windows are with "" not "/"

I'm facing the same issue while compiling in windows. Can we get this as a fix?. Thanks

Hey! Sorry I cannot reproduce the issue here and it has been inactive for a while.
I am going over issues right now and want to keep the repo clean - so I'm closing this for now, if anyone can post a reproduceable example here I can re-open and take a look :)


feat(externaltable): added support for table_format

This pull request  adds a table_format attribute to the snowflake_external_table resource.
Test Plan
I ran make test-acceptance but I keep running into the following issue :
"Your user account has been temporarily locked due to too many failed attempts"

[x] acceptance tests, for tests related to external tables :
=== RUN   TestExternalTable
--- PASS: TestExternalTable (0.00s)
=== RUN   TestExternalTableCreate
--- PASS: TestExternalTableCreate (0.00s)
=== RUN   TestExternalTableRead
--- PASS: TestExternalTableRead (0.00s)
=== RUN   TestExternalTableDelete
--- PASS: TestExternalTableDelete (0.00s)
=== RUN   TestExternalTablesCreate
=== RUN   TestExternalTablesCreate/basic_options
=== RUN   TestExternalTablesCreate/every_optional_field
=== RUN   TestExternalTablesCreate/invalid_options
--- PASS: TestExternalTablesCreate (0.00s)
--- PASS: TestExternalTablesCreate/basic_options (0.00s)
--- PASS: TestExternalTablesCreate/every_optional_field (0.00s)
--- PASS: TestExternalTablesCreate/invalid_options (0.00s)
=== RUN   TestExternalTablesCreateWithManualPartitioning
=== RUN   TestExternalTablesCreateWithManualPartitioning/valid_options
=== RUN   TestExternalTablesCreateWithManualPartitioning/invalid_options
--- PASS: TestExternalTablesCreateWithManualPartitioning (0.00s)
--- PASS: TestExternalTablesCreateWithManualPartitioning/valid_options (0.00s)
--- PASS: TestExternalTablesCreateWithManualPartitioning/invalid_options (0.00s)
=== RUN   TestExternalTablesCreateDeltaLake
=== RUN   TestExternalTablesCreateDeltaLake/valid_options
=== RUN   TestExternalTablesCreateDeltaLake/invalid_options
--- PASS: TestExternalTablesCreateDeltaLake (0.00s)
--- PASS: TestExternalTablesCreateDeltaLake/valid_options (0.00s)
--- PASS: TestExternalTablesCreateDeltaLake/invalid_options (0.00s)
=== RUN   TestExternalTableUsingTemplateOpts
=== RUN   TestExternalTableUsingTemplateOpts/valid_options
=== RUN   TestExternalTableUsingTemplateOpts/invalid_options
--- PASS: TestExternalTableUsingTemplateOpts (0.00s)
--- PASS: TestExternalTableUsingTemplateOpts/valid_options (0.00s)
--- PASS: TestExternalTableUsingTemplateOpts/invalid_options (0.00s)
=== RUN   TestExternalTablesAlter
=== RUN   TestExternalTablesAlter/refresh_without_path
=== RUN   TestExternalTablesAlter/refresh_with_path
=== RUN   TestExternalTablesAlter/add_files
=== RUN   TestExternalTablesAlter/remove_files
=== RUN   TestExternalTablesAlter/set_auto_refresh
=== RUN   TestExternalTablesAlter/set_tag
=== RUN   TestExternalTablesAlter/unset_tag
=== RUN   TestExternalTablesAlter/invalid_options
--- PASS: TestExternalTablesAlter (0.00s)
--- PASS: TestExternalTablesAlter/refresh_without_path (0.00s)
--- PASS: TestExternalTablesAlter/refresh_with_path (0.00s)
--- PASS: TestExternalTablesAlter/add_files (0.00s)
--- PASS: TestExternalTablesAlter/remove_files (0.00s)
--- PASS: TestExternalTablesAlter/set_auto_refresh (0.00s)
--- PASS: TestExternalTablesAlter/set_tag (0.00s)
--- PASS: TestExternalTablesAlter/unset_tag (0.00s)
--- PASS: TestExternalTablesAlter/invalid_options (0.00s)
=== RUN   TestExternalTablesAlterPartitions
=== RUN   TestExternalTablesAlterPartitions/add_partition
=== RUN   TestExternalTablesAlterPartitions/remove_partition
=== RUN   TestExternalTablesAlterPartitions/invalid_options
--- PASS: TestExternalTablesAlterPartitions (0.00s)
--- PASS: TestExternalTablesAlterPartitions/add_partition (0.00s)
--- PASS: TestExternalTablesAlterPartitions/remove_partition (0.00s)
--- PASS: TestExternalTablesAlterPartitions/invalid_options (0.00s)
=== RUN   TestExternalTablesDrop
=== RUN   TestExternalTablesDrop/restrict
=== RUN   TestExternalTablesDrop/cascade
=== RUN   TestExternalTablesDrop/invalid_options
--- PASS: TestExternalTablesDrop (0.00s)
--- PASS: TestExternalTablesDrop/restrict (0.00s)
--- PASS: TestExternalTablesDrop/cascade (0.00s)
--- PASS: TestExternalTablesDrop/invalid_options (0.00s)
=== RUN   TestExternalTablesShow
=== RUN   TestExternalTablesShow/all_options
=== RUN   TestExternalTablesShow/in_database
=== RUN   TestExternalTablesShow/in_schema
=== RUN   TestExternalTablesShow/invalid_options
--- PASS: TestExternalTablesShow (0.00s)
--- PASS: TestExternalTablesShow/all_options (0.00s)
--- PASS: TestExternalTablesShow/in_database (0.00s)
--- PASS: TestExternalTablesShow/in_schema (0.00s)
--- PASS: TestExternalTablesShow/invalid_options (0.00s)
=== RUN   TestExternalTablesDescribe
=== RUN   TestExternalTablesDescribe/type_columns
=== RUN   TestExternalTablesDescribe/type_stage
--- PASS: TestExternalTablesDescribe (0.00s)
--- PASS: TestExternalTablesDescribe/type_columns (0.00s)
--- PASS: TestExternalTablesDescribe/type_stage (0.00s)

References
Resolves #1564

Hey @Cedric-Magnan. Thanks for the contribution!
Unfortunately, we are currently changing the implementation used in resources as a part of our ongoing SDK rewrite. This means we won't use snowflake/external_table.go anymore.
However, we can add this missing parameter as a part of the process. CC: @sfc-gh-jcieslak.

Hey @Cedric-Magnan. Thanks for the contribution!
Unfortunately, we are currently changing the implementation used in resources as a part of our ongoing SDK rewrite. This means we won't use snowflake/external_table.go anymore.
However, we can add this missing parameter as a part of the process. CC: @sfc-gh-jcieslak.

Thanks for the reply @sfc-gh-asawicki, no problem, refactoring is important.
Do you have any available public roadmap about this SDK rewrite, in order for me to know when I could be able to provision external delta tables with Terraform using this provider ?
I will use https://github.com/aidanmelen/terraform-provider-snowsql in the meantime.
Could you also please close the original issue with a comment that explains why it won't be fixed (so that, if I close this PR, nobody opens a similar PR for the same issue) ?

@Cedric-Magnan, thanks for understanding!
We do not have a community-shared roadmap (yet). We added a table summarizing the SDK rewrite progress. You can check it out here: https://github.com/Snowflake-Labs/terraform-provider-snowflake#sdk-migration-table. As for the external table, it has been already migrated to the SDK. The new implementation has yet to be used, but @sfc-gh-jcieslak has it on his current TODO list so you can expect it soon.
I have answered in #1564.


fix: fixes the issue #609

Developer Checklist

Followed guidelines mentioned in the readme file.
Followed directory structure. (e.g. WeatherApp/rituraj12797/...yourfiles)
Starred ⭐ the Repo

Summary
Added the weather app project
Screenshot

Live Project Link
https://rituraj12797.github.io/weatherApp/

@PBJI
@thinkswell
kindly review this pr
it  fixes the issue #609  which will close this issue

thank you @PBJI  , will keep this in mind from next time.


[BCDB] server should bind on localhost only

BCDB server shouldn't be reachable from outside world

https://github.com/threefoldtech/jumpscaleX_core/blob/d86eb01457956dcaf7145c947e931ccc8bfd833e/JumpscaleCore/data/bcdb/connectors/redis/RedisServer.py#L33

Starting a redis connector to bcdb system with
j.data.bcdb.system.redis_server_start()

We see the server listening on localhost only,
tcp        0      0 <IP_ADDRESS>:6379          <IP_ADDRESS>:*               LISTEN      113        44257      14866/redis-server
tcp        0      0 <IP_ADDRESS>:6380          <IP_ADDRESS>:*               LISTEN      0          99227      22273/python3


segment.io integration

As a Kubefirst user I want to collect metrics during the installation process so that I know better how Im experiencing the installation on a user stand point from start to finish.
Acceptance criteria:
AC1: it uses official Segment.io Golang library[1]
AC2: the implementation is feature parity[2] with Datadog
AC3: the implementation is loosely coupled to allow unit tests
AC4: there are unit tests (at least some)
AC5: there are Segment.io documentation at docs.kubefirst.com describing how we use Segment.io
AC6: segment.io is default, and doesn't require a new flag
AC7: metrics are associated with user domain, and Kubefirst cli version
AC8: We need to be able to privately dashboard this information with varying time granularity (last hour, last 2 months, etc)
[1] https://github.com/segmentio/analytics-go
[2] events captured: init started, init completed, cluster create started, cluster create completed
Details
Today (as of 1.9), we implement our telemetry MVP leveraging a custom metric microservice named telemetry that runs in our mgmt account so we can measure the frequency and success rate of init and create commands. This is implemented with 4 calls in kubefirst/kubefirst that look approximately like this:
https://github.com/kubefirst/kubefirst/blob/70f3d268040ac8af921b5416d53113f8138e9b06/cmd/createUtils.go#L51
Note: We must KEEP the telemetry microservice operational until we're convinced nobody is using the older cli versions that leveraged this microservice. (probably a few months out)

Do we care about protecting the SEGMENT_WRITE_KEY on segment?

Let’s add this to the service we proxy to for Datadog metrics currently to keep it protected for now

I did research regarding the SEGMENT_WRITE_KEY, from Segment website[1]:
The write key is the unique identifier for a source that tells Segment which source data comes from, to which workspace the data belongs, and which destinations should receive the data.

They had a security incident with a customer, and in the incident description it was raised that the write key is considered public:
Two months of recent data about how Segment's own customers interact with different aspects of our product, including customer write keys for Segment (which are considered public)

Other projects also expose[3] the key.
[1] https://segment.com/docs/connections/find-writekey/
[2] https://segment.com/blog/bulletins-incident090519/
[3] https://github.com/search?p=1&q=segment_write_key&type=Code

I believe we have two ways to implement it. Disabling AppRunner, and rely only on Segment.io, that will forward the metric to DataDog via Segment.io destination, will have one last app(app runner) for us to maintain.

yeah i agree with the direction of the 2nd approach as our permanent approach. i do want to mention however that while we're removing the dependence on the telemetry app running in apprunner, old kubefirst clients in the wild will still use it until they upgrade the cli, so let's be sure to not disrupt anything about the cli -> telemetry -> datadog flow of data for the next few months - no deprovisioning operations in apprunner please.

yeah i agree with the direction of the 2nd approach as our permanent approach. i do want to mention however that while we're removing the dependence on the telemetry app running in apprunner, old kubefirst clients in the wild will still use it until they upgrade the cli, so let's be sure to not disrupt anything about the cli -> telemetry -> datadog flow of data for the next few months - no deprovisioning operations in apprunner please.

added a new AC in the ticket description to follow that.
AC3: previous telemetry implementation still works for user running previous Kubefirst installer


NCL 3393

Adding the remaining _raw methods. This should result in fully functional integration tests.

Current problems with the test results:
12 Total failures.
test_buildconfigurationsets_api:

test_add_configuration: add_configuration in the sets_api does not seem to work correctly. No BuildConfiguration is added to the specified set and thus the tests fails, since no BuildConfigurations are associated with the BuildConfigurationSet the test creates.
test_remove_configuration function: same as above.

test_buildrecords_api:

test_get_all_for_project: The API has changed to accept a project name, not an ID. Will submit fix for this in a new PR.
test_get_built_artifacts / test_get_dependency_artifacts / test_query_by_attribute: These tests use a query on the list of BuildRecords from get_all which no longer works. Query is: '(buildConfigurationAudited.nam=like=%cli-test%)', and the failure is because the BuildRecord model doesn't contain an attribute with this name anymore.

test_productmilestones_api:

test_add_distributed_artifact / test_remove_distributed_artifact: same query related failure as test_get_built_artifacts et al.
test_get_distributed_artifacts / test_get_distributed_build: there are no artifacts / builds associated with the newly created milestones and artifacts. Likely could modify tests to add them beforehand.

Looks good to me too


Improve prompt customization.

We should be able to change the prompt to adjust the response to our needs.

Added the PROMPT env variable in new release v1.0.7


feat(git-hook): Add unit test to pre-commit git hook

Motivation
This is an improvement to AEROGEAR-8495.
What
Add automatic unit testing to the pre-commit hook.
Why
To ensure all unit tests pass before pushing to remote.
Verification Steps
If you haven't already, check out the steps in https://github.com/aerogear/mobile-security-service/pull/24 to test the earlier versions of this hook.

To test if go test runs correctly and blocks your commit, temporarily make a change to one of the unit tests to make it fail.
Run git add . && commit --amend
The commit should be prevent and you should see the following output:

...
Unit test(s) failed. Please fix your code and try again.

Checklist:

[x] Code has been tested locally by PR requester
[] Changes have been successfully verified by another team member

Progress

[x] Finished task

Coverage remained the same at 69.403% when pulling 379a3e02683fc46a33b77a0b539a88eea07b622d on craicoverflow:add-unit-test-git-hook into 250f52b345072ee20eb18cd409a06f272ae4359a on aerogear:master.

@craicoverflow it shows fine. However, did you check if the test has been checked in the CI? If not, wdyt about create a Jira for that?


Adds support for GA4 and UA at the same time

During migration from Universal Analytics (UA) to Google Analytics 4 (GA) it is useful to be able to configure both site tags at the same time. This PR adds the ability to specify a site tag list.

Closes #4327
Closes #8549
Closes #10093

(I experimented with multiple solutions and the one propose here seems the least obtrusive and most backwards compatible).
/cc @jmooring @nate-double-u

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add x-LAY3R - Ethereum, Arbitrum, BSC

I have push another commit changing the address, since as you have detected in the monorepo PR we were using the LockBox address instead of the xERC20


Send operation name to the server

Sending the operation name of the query down the server (probably helpful for logging, etc.).
TODO:

[x] Update CHANGELOG.md with your change
[x] Make sure all of the significant new logic is covered by tests
[x] Rebase your changes on master so that they can be merged easily
[x] Make sure all tests and linter rules pass

lgtm, apart from the string vs String thing.

Actually, changelog entry seems to be missing. For some reason you deleted it again.

Fixed.


Is this toilet free = Yes returning a ?

I entered a new toilet selecting all fields to be 'yes'.
However the toilet appears as Free = ?
https://gbptm.now.sh/loos/21139711be6b9ae5d225a31e
Safari Version 13.1.1 (136<IP_ADDRESS>.3)

I can replicate this, it appears the add toilet mutation is not including the "noPayment" field, possibly because it doesn't change from the default


Need to update directory path for LINUX compatibility

https://docs.microsoft.com/en-us/dotnet/api/system.io.path.combine?view=netframework-4.7.2

Completed.


Use latest gosec vetsion

Description
Use latest gosec vetsion
Fixes #479
Type of change

Refactor (refactoring code, removing useless files)

Testing steps
Please describe how the change was tested locally. If, for some reason, the testing was not done or not done fully, please describe what are the testing steps.
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report

Merging #480 (b424e32) into master (b6e30cc) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #480   +/-   ##
=======================================
  Coverage   72.14%   72.14%
=======================================
  Files          12       12
  Lines         718      718
=======================================
  Hits          518      518
  Misses        196      196
  Partials        4        4

Impacted Files
Coverage Δ

commands/profiles.go
57.35% <100.00%> (ø)

restapi/utils.go
94.73% <100.00%> (ø)

Continue to review full report at Codecov.

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


How to style a placeholder?

As per the example I can see that we can specify a string as placeholder in input props. What if I want to style my placeholder where I need provide a different font size, colour or add some icons to it? Any idea? It should look like this http://jsfiddle.net/ADjTF/

hey :)

for the icon. you could use the before pseudo-element of the input field or the label of your input and set the icon as your background image. as soon as you reach the :focus state of your input field you can hide it again. some position magic of the pseudo element and a padding to your input and then you have an icon "inside" your input field thats acting like a placeholder.

the placeholder text itself you can style via a prefixed selector. readon here: https://css-tricks.com/almanac/selectors/p/placeholder/
lovely greetings
Lelith


100's of duplicate media_player components being automatically created.

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.39.2
Python release (python3 --version):
stock docker container
Component/platform:
stock docker container from HA docker repository
Description of problem:
plex.media_player and roku.media_player components keep spawning every 10-15.  Old compomponent hangs and doesn't clear the interface.
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:

discovery compnent is enabled
media players are automatically discovered

Traceback (if applicable):

Additional info:
Log throwing these exceptions
17-03-02 16:47:41 WARNING (MainThread) [homeassistant.components.media_player] Updating roku media_player took longer than the scheduled update interval 0:00:10 17-03-02 16:54:32 WARNING (MainThread) [homeassistant.components.media_player] Updating roku media_player took longer than the scheduled update interval 0:00:10 17-03-02 16:54:42 WARNING (MainThread) [homeassistant.components.media_player] Updating roku media_player took longer than the scheduled update interval 0:00:10 17-03-02 16:54:53 WARNING (MainThread) [homeassistant.components.media_player] Updating roku media_player took longer than the scheduled update interval 0:00:10 17-03-02 16:59:36 WARNING (MainThread) [homeassistant.components.media_player] Updating roku media_player took longer than the scheduled update interval 0:00:10

Duplicate of #5588?

Very similar, I'll close this issue and comment further with the additional issues I have there

Also seeing this in 0.39.2....lot of duplicate media player entries


Mavlink2 test

Built px4fmu-v2_default with generated C headers from mavlink fork here.

@DonLakeFlyer This should allow you to copy the headers and to test against a SITL running v2.0

@DonLakeFlyer Did you get a chance yet to create a QGC branch to test?

Started, but not much progress. Too many other things going on.

I'm closing this and taking a fresh stab at it.


  This
keeps validation layers from crashing or reporting false positives

I forgot to add a test for these - working on it now

Committed to master


Can not get MS Teams Locale while Adding bot to MS Teams

Could you please add a note in below section:
https://docs.microsoft.com/en-us/microsoftteams/platform/concepts/deploy-and-publish/apps-upload#upload-your-app
"You can get the Microsoft Teams Locale while Adding the bot to Microsoft Teams in onInstallationUpdateActivityAsync() method using turnContext.Activity.GetLocale().

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

ID: 54780e5a-ca3a-5277-dc84-37f85d9ea1cb
Version Independent ID: 5f89865b-a9fd-ada5-72d2-6375357bba8c
Content: Upload your custom app - Teams
Content Source: msteams-platform/concepts/deploy-and-publish/apps-upload.md
Product: msteams
GitHub Login: @KirtiPereira
Microsoft Alias: surbhigupta

@surbhigupta12 - Could you please help us here to update the contents?

@surbhigupta12 - Could you please help us here to update the contents?


Fix null collision list case

Bug fix to better handle empty updated collisions list and avoid conflict with the previousCollisions list.
GearVRf-DCO-1.0-Signed-off-by: JulianaFigueira<EMAIL_ADDRESS>

+1, looks good to me.


Invalid data format: Invalid passphrase

I've never set a passphrase for this wallet
JsonRpcError: Invalid data format: Invalid passphrase
    at JsonRpcError.ExtendableBuiltin (file:///C:/Users/roque/AppData/Local/Programs/EmeraldWallet/resources/app.asar/app/index.js:24080:30)
    at new JsonRpcError (file:///C:/Users/roque/AppData/Local/Programs/EmeraldWallet/resources/app.asar/app/index.js:24109:116)
    at file:///C:/Users/roque/AppData/Local/Programs/EmeraldWallet/resources/app.asar/app/index.js:24146:19

How's possible the wallet ask me to put the password and sign before sending, when I've never been asked for encrypting the wallet?


Update README.md

@horis233 Please review and update the readme.
Do you have any information for these sections? If you don't, you can delete these sections.
Quick start guide
Use the following quick start commands for building and testing the operator:
<!--- Include any required quick start commands -->

Debugging guide
Use the following commands to debug the operator:
<!--- Include any required quick start commands -->

@DanielXLee Could you manually upload this CSV to your own testing quay and verify if it works or not?

@chenzhiwei verified in my quay.io, running well.


[Bug]: Constants not identifying its type

Description
const CAPACITY = 100;
const EVICTION_FACTOR = 0.2;

final cache:Cache userInfoCache = new (capacity = CAPACITY, evictionFactor = EVICTION_FACTOR);

This gives the following error, as it doesn't recognize the constant types as int or float:
invalid value: For input string: "CAPACITY"
invalid value: For input string: "EVICTION_FACTOR"
It seems the constants are not being identified with their intended types.
Steps to Reproduce
No response
Affected Version(s)
No response
OS, DB, other environment details and versions
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Related to https://github.com/ballerina-platform/ballerina-library/issues/6036


[fix][broker] add the S3 offload option in the broker.conf comment

Fixes #xyz

Main Issue: #xyz

PIP: #xyz

Motivation

https://github.com/apache/pulsar/blob/master/pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/OffloadPoliciesImpl.java#L69 has the option for S3 offloader and related doc https://pulsar.apache.org/docs/3.0.x/tiered-storage-s3/.
But broker.conf offloader comment is missing the option for S3.
Modifications

Add the S3 offload option in the broker.conf comment.
Verifying this change

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

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[x] doc
[ ] doc-required
[ ] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Merging...
Thank you!


Default model is set to text-davinci-002 not text-davinci-003

The intended default was the latest but instead it uses version 2 instead of version 3.
chatgpt.py:92:             engine = "text-davinci-002"
Should change to
chatgpt.py:92:            engine = "text-davinci-003"

resolved with v1.1.0


Creates wrong archive type

This may be a minor issue, but, this script produces a tgz, which can not be uploaded to GitHub.  So the user has to uncompress and create a zip file.

Hi @treaves sorry for the incredibly slow response.
The script currently produces a .tgz file which GitHub doesn't allow. However, that is shorthand for .tar.gz which is the more proper name and GitHub does allow. Would changing the name to the long version be acceptable?
That said, be cautious uploading the tar file this script creates. While we are careful to avoid logging any sensitive information like passwords, some companies may consider portions of the information confidential.

+1 please change to widely-accepted *.tar.gz.

To reiterate what @jhaynes mentioned, it's imperative that the user does not upload the resulting archive without acknowledging that what they're uploading will be publicly accessible regardless of its contents once its on Github. A very loud warning message should (must?) accompany this change.


Alexnet and googlenet symbol in example/image-classification is wrong

@piiswrong  Symbol for alexnet and googlenet should be switched. Pls check.
https://github.com/dmlc/mxnet/blob/master/example/image-classification/symbols/alexnet.py
https://github.com/dmlc/mxnet/blob/master/example/image-classification/symbols/googlenet.py

Yes could you make a pr?

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Add missing space to make header work

Codecov Report

Merging #443 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #443      +/-   ##
==========================================
+ Coverage   17.54%   17.54%   +<.01%
==========================================
  Files         213      213
  Lines      122889   122889
  Branches    11839    11839
==========================================
+ Hits        21555    21557       +2
+ Misses     100475   100474       -1
+ Partials      859      858       -1

Flag
Coverage Δ

#2017
17.54% <ø> (ø)
:arrow_up:

#Debug
17.54% <ø> (ø)
:arrow_up:

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

Impacted Files
Coverage Δ

src/PerfView/StackViewer/StackWindow.xaml.cs
24.14% <0%> (+0.07%)
:arrow_up:

Continue to review full report at Codecov.

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


Deletion of a pre-v1.3 environment fails

This is a bit challenging in a migration scenario, as I would expect the normal flow is:

Install new ff
Find you need to delete your old dev env
Run ff rm (currently fails)
Re-run ff init

./ff/ff rm -f dev
deleting FireFly stack 'dev'... Error: Error: No such volume: dev_geth

Usage:
  ff remove <stack_name> [flags]

Aliases:
  remove, rm

Flags:
  -f, --force   Remove the stack without prompting for confirmation
  -h, --help    help for remove

Global Flags:
      --ansi string   control when to print ANSI control characters ("never"|"always"|"auto") (default "auto")
  -v, --verbose       verbose log output

Ah - so I've realized the main branch of the CLI also doesn't use the 1.3 RC by default either... so I ended up with a broken environment when I created a new environment as well - guess not strictly related:
FF00184: Database migration failed: FF00184: Database migration failed: unable to open database file: no such file or directory

Hmmmn, well I also can't create a new environment I just created...

➜  firefly-cli git:(main) ./ff/ff init --multiparty=false gw2 1
initializing new FireFly stack...
Stack 'gw2' created!
To start your new stack run:

ff start gw2

Your docker compose file for this stack can be found at: /Users/pbroadhurst/.firefly/stacks/gw2/docker-compose.yml

➜  firefly-cli git:(main) ./ff/ff rm gw2
WARNING: This will completely remove your stack and all of its data. Are you sure this is what you want to do?
completely delete FireFly stack 'gw2' [y/N] y
deleting FireFly stack 'gw2'... Error: Error: No such volume: gw2_geth

Usage:
  ff remove <stack_name> [flags]

Aliases:
  remove, rm

Flags:
  -f, --force   Remove the stack without prompting for confirmation
  -h, --help    help for remove

Global Flags:
      --ansi string   control when to print ANSI control characters ("never"|"always"|"auto") (default "auto")
  -v, --verbose       verbose log output

Error: Error: No such volume: gw2_geth

Ok - with a bit more testing, I thinks the scope of the issue is:

v1.2 environments
v1.3 environments that haven't been intialized

Thanks for opening this! I think I have seen this issue as well. I thought I fixed it, but maybe it was a similar but slightly different issue.
For background we used to swallow a lot of errors, and when we added the linter it wasn't a big fan of that. Making the linter happy ended up with some errors that still should have been swallowed (or at least logged) getting returned all the way up. This was one of those. Should be an easy fix. I'll open a PR for it.

@peterbroadhurst Are you sure you are running the latest code in main? I think this existing check should allow this condition to not return the error:
https://github.com/hyperledger/firefly-cli/blob/3679daf0650b0fa1870b86945603f2b2f71577d6/internal/stacks/stack_manager.go#L707-L709


Add-AzureADGroupMember error "Error occurred while executing AddGroupMember"

We re experiencing a consistent error adding members to a security group
$refObjectId=Get-AzureRmADUser -UserPrincipalName $UserEmail
Add-AzureADGroupMember -ObjectId [securitygroupID] -RefObjectId $refObjectId.Id
script above succeeds about half of the time and fails about half throwing :
Exception: Error occurred while executing AddGroupMember
StatusCode: BadRequest
ErrorCode: Request_BadRequest
Message: One or more added object references already exist for the following modified properties: 'members'.
This group typically has hundreds of users that primarily have a common user name (i.e. Temporary User) but unique UPNs (i.e. user12345@) I am assuming when Add-AzureADGroupMember runs, it is first verifying whether member already exists in the group and is checking username (will have duplicates) rather than objectid (no duplicates)
Any suggestions to force command to use ObjectID for verification?

Hi @terriharmon
Thank you for your issue submission. Unfortunately, this appears to be a problem with the cmdlt and a support issue. Issues in this repo are for documentation issues. You can try one of the following links for additional help.
Microsoft Support: https://support.microsoft.com/
Azure AD forums: https://social.technet.microsoft.com/Forums/en-US/home?forum=windowsazureaditpro
Stackoverflow: https://stackoverflow.com/
Bill

i wanted to add massive owners for a Azure AD application but this is my error output :

        Add-AzureADApplicationOwner -ObjectId $app.ObjectId -RefO ...

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Add-AzureADApplicationOwner], ApiException
FullyQualifiedErrorId : Microsoft.Open.AzureAD16.Client.ApiException,Microsoft.Open.AzureAD16.PowerShell.AddApplicationOwner

Add-AzureADApplicationOwner : Error occurred while executing AddApplicationOwner
Code: Request_BadRequest
Message: One or more added object references already exist for the following modified properties: 'owners'.
RequestId: 3f541a87-1945-4be2-813d-05aa7bf0becc
DateTimeStamp: Wed, 19 Oct 2022 22:29:39 GMT
HttpStatusCode: BadRequest


Setup Next.js App to interact with the smart contract

PR: https://github.com/chrisstef/flash-pay/pull/2


Syntax highlighting not working

Dear All,
I'm trying to use the extension and I cannot find what I'm doing wrong. I've several working stylesheets that I use successfully or transformation but syntax highlighting doesn't work for me in vscode:

What am I doing wrong ?

The screenshot you're showing has got syntax highlighting, different
'tokens' are shown with different colours.
Is it that syntax highlighting is not working in another part of your code?
Your screenshot shows a problem reported with the xsl:stylesheet element,
is that related to this issue? If so, please can you  attach an example
XSLT that reproduces the issue?
Note that for some themes syntax highlighting requires the VSCode semantic
highlighting setting set true, as explained in the Readme file.
On Wed, 27 Jan 2021, 17:42 oblaise<EMAIL_ADDRESS>wrote:

Dear All,
I'm trying to use the extension and I cannot find what I'm doing wrong.
I've several working stylesheets that I use successfully or transformation
but syntax highlighting doesn't work for me in vscode:
[image: image]
https://user-images.githubusercontent.com/16148676/106031253-5b3aa980-60cf-11eb-8462-19125881d623.png
What am I doing wrong ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DeltaXML/vscode-xslt-tokenizer/issues/38, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAU3RGWDMKEXC45XWYNNYHLS4BGBFANCNFSM4WVVE4ZQ
.

The screenshot you're showing has got syntax highlighting, different
'tokens' are shown with different colours.
Is it that syntax highlighting is not working in another part of your code?
Your screenshot shows a problem reported with the xsl:stylesheet element,
is that related to this issue? If so, please can you  attach an example
XSLT that reproduces the issue?
Note that for some themes syntax highlighting requires the VSCode semantic
highlighting setting set true, as explained in the Readme file.
On Wed, 27 Jan 2021, 17:42 oblaise<EMAIL_ADDRESS>wrote:

Dear All,
I'm trying to use the extension and I cannot find what I'm doing wrong.
I've several working stylesheets that I use successfully or transformation
but syntax highlighting doesn't work for me in vscode:
[image: image]
https://user-images.githubusercontent.com/16148676/106031253-5b3aa980-60cf-11eb-8462-19125881d623.png
What am I doing wrong ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DeltaXML/vscode-xslt-tokenizer/issues/38, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAU3RGWDMKEXC45XWYNNYHLS4BGBFANCNFSM4WVVE4ZQ
.

The screenshot you're showing has got syntax highlighting, different 'tokens' are shown with different colours. Is it that syntax highlighting is not working in another part of your code?

No, I'm indeed referring to the issue below

Your screenshot shows a problem reported with the xsl:stylesheet element, is that related to this issue? If so, please can you attach an example XSLT that reproduces the issue?

That's my issue that I mix up with syntax highlighting. I'm attaching a problematic stylesheet.

Note that for some themes syntax highlighting requires the VSCode semantic highlighting setting set true, as explained in the Readme file.

I think I enabled it correctly.
ets_calimero_group_name.zip

The screenshot you're showing has got syntax highlighting, different 'tokens' are shown with different colours. Is it that syntax highlighting is not working in another part of your code?

No, I'm indeed referring to the issue below

Your screenshot shows a problem reported with the xsl:stylesheet element, is that related to this issue? If so, please can you attach an example XSLT that reproduces the issue?

That's my issue that I mix up with syntax highlighting. I'm attaching a problematic stylesheet.

Note that for some themes syntax highlighting requires the VSCode semantic highlighting setting set true, as explained in the Readme file.

I think I enabled it correctly.
ets_calimero_group_name.zip

When I open the file in VSCode with the extension enabled I don't see the same 'Invalid attribute' message that is in your screenshot. The screenshot I've attached at the end shows that no problems are reported on my local MacOS environment.
Could you try disabling all other extensions to see if this resolves the issue? If this still fails, please could you provide more information on:

VS Code Version:
XSLT/XPath Extension Version:
Your Operating System and version

When I open the file in VSCode with the extension enabled I don't see the same 'Invalid attribute' message that is in your screenshot. The screenshot I've attached at the end shows that no problems are reported on my local MacOS environment.
Could you try disabling all other extensions to see if this resolves the issue? If this still fails, please could you provide more information on:

VS Code Version:
XSLT/XPath Extension Version:
Your Operating System and version

I disabled all other extensions and it solved the issue. After more analysis the extension causing the issue is 'XML Tools' (https://github.com/DotJoshJohnson/vscode-xml) which is a widely used extension. It can be a bit annoying that these extensions doesn't coexist because when you work on XSLT docs you also work on XML docs.

VS Code Version:

1.52.1

XSLT/XPath Extension Version:

0.2.0

Your Operating System and version

win10 20H2

I disabled all other extensions and it solved the issue. After more analysis the extension causing the issue is 'XML Tools' (https://github.com/DotJoshJohnson/vscode-xml) which is a widely used extension. It can be a bit annoying that these extensions doesn't coexist because when you work on XSLT docs you also work on XML docs.

VS Code Version:

1.52.1

XSLT/XPath Extension Version:

0.2.0

Your Operating System and version

win10 20H2

I agree this is an annoyance. From this issue it seems that maintainer has been rather busy with other priorities.
I will see if there's anything I can do to fix compatibility this end. There is an underlying problem here because Visual Studio Code has a built-in extension for XSLT that uses the 'xsl' language id. This extension can't uses that language id because of conflicts with Visual Studio Code's built-in XSLT extension so this extension uses the 'xslt' language id instead.
I've asked the Microsoft team through GitHub issues to help with this but unfortunately they are too busy at the moment and closed the issues with no action.

I agree this is an annoyance. From this issue it seems that maintainer has been rather busy with other priorities.
I will see if there's anything I can do to fix compatibility this end. There is an underlying problem here because Visual Studio Code has a built-in extension for XSLT that uses the 'xsl' language id. This extension can't uses that language id because of conflicts with Visual Studio Code's built-in XSLT extension so this extension uses the 'xslt' language id instead.
I've asked the Microsoft team through GitHub issues to help with this but unfortunately they are too busy at the moment and closed the issues with no action.

It would be very helpful if you could let me know the features missing from this extension that you use the DotJoshJohnson XML Tools extension for?
I've tried to support all the basic XML Editing features in this extension so there is a lot of overlap, but I haven't yet added XQuery support.

It would be very helpful if you could let me know the features missing from this extension that you use the DotJoshJohnson XML Tools extension for?
I've tried to support all the basic XML Editing features in this extension so there is a lot of overlap, but I haven't yet added XQuery support.

Hard to say as both extensions were installed in parallel since a few months. I observed the incompatibility after the latest updates even if obviously some functionalities weren't working as expected before.
But now I removed the XML Tools and checked a little further. The 'Get Current XPath' could be a very good candidate to add to your extension specially as you are specifically targeting XSLT/XPath.
PS: I don't use XQuery.

Hard to say as both extensions were installed in parallel since a few months. I observed the incompatibility after the latest updates even if obviously some functionalities weren't working as expected before.
But now I removed the XML Tools and checked a little further. The 'Get Current XPath' could be a very good candidate to add to your extension specially as you are specifically targeting XSLT/XPath.
PS: I don't use XQuery.

Thanks for this feedback. I'll be sure to add this once I start adding the
XPath evaluator. There's an online tool that I developed a few years ago
that I may use as a starting point:
http://www.qutoric.com/xslt/analyser/xpathtool.html
On Fri, 29 Jan 2021, 12:08 oblaise<EMAIL_ADDRESS>wrote:

Hard to say as both extensions were installed in parallel since a few
months. I observed the incompatibility after the latest updates even if
obviously some functionalities weren't working as expected before.
But now I removed the XML Tools and checked a little further. The 'Get
Current XPath' could be a very good candidate to add to your extension
specially as you are specifically targeting XSLT/XPath.
PS: I don't use XQuery.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DeltaXML/vscode-xslt-tokenizer/issues/38#issuecomment-769767166,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAU3RGRC4QZ7ZGA4ANQCANDS4KQKRANCNFSM4WVVE4ZQ
.

Thanks for this feedback. I'll be sure to add this once I start adding the
XPath evaluator. There's an online tool that I developed a few years ago
that I may use as a starting point:
http://www.qutoric.com/xslt/analyser/xpathtool.html
On Fri, 29 Jan 2021, 12:08 oblaise<EMAIL_ADDRESS>wrote:

Hard to say as both extensions were installed in parallel since a few
months. I observed the incompatibility after the latest updates even if
obviously some functionalities weren't working as expected before.
But now I removed the XML Tools and checked a little further. The 'Get
Current XPath' could be a very good candidate to add to your extension
specially as you are specifically targeting XSLT/XPath.
PS: I don't use XQuery.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DeltaXML/vscode-xslt-tokenizer/issues/38#issuecomment-769767166,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAU3RGRC4QZ7ZGA4ANQCANDS4KQKRANCNFSM4WVVE4ZQ
.

I’ve added “Get current XPath” as a new feature request issue

I’ve added “Get current XPath” as a new feature request issue

@oblaise I've added a 'XML: Goto XPath' VSCode command to this extension.
This command can also be used as 'Get current XPath' as it pre-populates the InputBox with the current XPath location, the selected XPath can then be copied easily to the clipboard.


[SPARK-2673][CORE][WEBUI] Enable to attach Debugger to Executors easily

In current implementation, it's difficult to attach debugger to each Executor in the cluster.
There are some reasons as follows.

Multi Executors can run on the same machine so each executor should open individual debug ports.

Even if we can open unique debug port to each Executors running on the same machine, it's a bother to check debug port of each executor.

To solve those problem, I think following 2 improvement is needed.

Enable executor to open unique debug port on a machine by setting debug port to 0.
Expand WebUI to be able to show debug ports opening in each executor.

Following image is how Web UI(ExecutorsPage) shows the debug ports for each Executor.

Test build #29929 has started for   PR 5437 at commit dbf69f3.

Test build #29929 has finished for   PR 5437 at commit dbf69f3.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Test build #29932 has started for   PR 5437 at commit 79ee2f9.

Test build #29933 has started for   PR 5437 at commit e3ba120.

Test build #29932 has finished for   PR 5437 at commit 79ee2f9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Test build #29933 has finished for   PR 5437 at commit e3ba120.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Referring to the JIRA, I do wonder how much this adds over just setting up the agent manually with JVM opts, because it touches a reasonable amount of code. Yes, you have to pick a fixed port, but then problem 2 goes away; you know what the agent port is. Yes, that doesn't work with several executors per machine, but if you're restarting your app anyway to debug, would you not perhaps just run a leaner config with 1 per machine? in the event that you really have to debug a multiple executor per machine job, maybe you can make the port a function of the executor's ID? I think we added some support for that?

What about ApplicationMaster (ExecutorLauncher) and Driver side? As there may be too much executors so we attach debuggers to them for convienience, and there's no same trouble in AM and Driver, am I right?

@sarutak are you still actively working on this? If not, would you mind closing this patch for now? If there's interest in the future we can always reopen it against the latest master branch.

Thanks for the notification. Unfortunately, now I don't have time to improve this patch. So I'll close for now.