When using OVN on IPv4/IPv6 dual stack the MAC address of the IPv4 gateway is not learnt.

Issue description
When using LXD and OVN on a dual stack uplink network the MAC address of the IPv4 gateway is not learnt. This leads to broken northbound connectivity. However we do have a mac binding for the IPv6 gateway,
Looking in the OVN driver code I see there's a workaround that makes the driver issue an ICMPv6 request in order to discover the IPv6 gateway but there is no corresponding IPv4 workaround.
Steps to reproduce

Deploy LXD on a dual stack IPv4/IPv6 network.
Install and run the OVN chassis on all LXD nodes.
Create an uplink network attached to a physical NIC on the IPv4/IPv6 network.
Create an OVN network that uses the uplink as a parent.
Launch a container and attempt to ping a public IP or the next hop.
No response will be received.
Check the SB MAC Binding database and there will be no entry.
Ping the routers external IPv4 address.
A MAC binding will be created and IPv4 internet connectivity will be established.

Information to attach
LXD version:  5.12-c63881f
OVN version: 22.03.0
$ microovn.ovn-nbctl show
switch 0d8be25c-8503-4989-8af0-76f4d408313b (lxd-net2-ls-int)
    port lxd-net2-instance-c0890aa9-a87a-4a4a-b184-6f5d6ec3b04e-eth0
        addresses: ["00:16:3e:b0:55:c4 dynamic"]
    port lxd-net2-ls-int-lsp-router
        type: router
        router-port: lxd-net2-lr-lrp-int
switch 47f2de8f-e5f9-47b4-ba6b-4a4dba63f9ee (lxd-net2-ls-ext)
    port lxd-net2-ls-ext-lsp-router
        type: router
        router-port: lxd-net2-lr-lrp-ext
    port lxd-net2-ls-ext-lsp-provider
        type: localnet
        addresses: ["unknown"]
router 3354b1b6-1ae4-4de5-b2d7-e5c189474137 (lxd-net2-lr)
    port lxd-net2-lr-lrp-int
        mac: "00:16:3e:f8:47:c0"
        networks: ["<IP_ADDRESS>/24", "fd42:3db6:cbae:c768::1/64"]
    port lxd-net2-lr-lrp-ext
        mac: "00:16:3e:f8:47:c0"
        networks: ["<IP_ADDRESS>/24", "fd55:13f7:9dcd:1::100/64"]
    nat 1b5b525d-1e4f-43ef-81e2-f7040b3a75a3
        external ip: "<IP_ADDRESS>"
        logical ip: "<IP_ADDRESS>/24"
        type: "snat"
    nat 6ad0727b-c01c-415a-9208-096abc31ee08
        external ip: "fd55:13f7:9dcd:1::100"
        logical ip: "fd42:3db6:cbae:c768::/64"
        type: "snat"

$ microovn.ovn-sbctl list MAC_Binding
_uuid               : abcf63c5-dee7-4295-ae63-ad7381fa12ef
datapath            : d3217d6b-1e26-43e1-8e08-fec0c44e9e78
ip                  : "::"
logical_port        : lxd-net2-lr-lrp-int
mac                 : "00:00:00:00:00:00"

_uuid               : cb41e19e-b5f0-4110-a82c-73703ae80917
datapath            : d3217d6b-1e26-43e1-8e08-fec0c44e9e78
ip                  : "fe80::f816:3eff:fead:e264"
logical_port        : lxd-net2-lr-lrp-ext
mac                 : "fa:16:3e:ad:e2:64"

_uuid               : 4e5263a7-4491-48ad-8c14-b82b133ae5ab
datapath            : d3217d6b-1e26-43e1-8e08-fec0c44e9e78
ip                  : "fd55:13f7:9dcd:1::1"
logical_port        : lxd-net2-lr-lrp-ext
mac                 : "fa:16:3e:ad:e2:64"

Looking in the OVN driver code I see there's a workaround that makes the driver issue an ICMPv6 request in order to discover the IPv6 gateway but there is no corresponding IPv4 workaround.

We've never needed that before. So unless its a new regression in OVN I suspect its a misconfiguration or feature limitation of OVN.

$ lxc network show uplink
config:
  dns.nameservers: <IP_ADDRESS>,2606:4700::1111
  ipv4.gateway: <IP_ADDRESS>/24
  ipv4.ovn.ranges: <IP_ADDRESS>-<IP_ADDRESS>
  ipv4.routes: <IP_ADDRESS>/24
  ipv6.gateway: fd55:13f7:9dcd:1::1/64
  ipv6.ovn.ranges: fd55:13f7:9dcd:1::100-fd55:13f7:9dcd:1::253
  ipv6.routes: fd55:13f7:9dcd:1::/64
  volatile.last_state.created: "false"
description: ""
name: uplink
type: physical
used_by:
- /1.0/networks/default
managed: true
status: Created
locations:
- lxd-virtual-edge-0
- lxd-virtual-edge-1
- lxd-virtual-edge-2

$ lxc network show default
config:
  bridge.mtu: "1422"
  ipv4.address: <IP_ADDRESS>/24
  ipv4.nat: "true"
  ipv6.address: fd42:3db6:cbae:c768::1/64
  ipv6.nat: "true"
  network: uplink
  volatile.network.ipv4.address: <IP_ADDRESS>
  volatile.network.ipv6.address: fd55:13f7:9dcd:1::100
description: ""
name: default
type: ovn
used_by:
- /1.0/instances/bold-goat
managed: true
status: Created
locations:
- lxd-virtual-edge-0
- lxd-virtual-edge-1
- lxd-virtual-edge-2

lxc info bold-goat
Name: bold-goat
Status: RUNNING
Type: container
Architecture: x86_64
Location: lxd-virtual-edge-0
PID: 4114
Created: 2023/03/20 19:31 UTC
Last Used: 2023/03/20 19:31 UTC

Resources:
  Processes: 57
  Disk usage:
    root: 1.40GiB
  CPU usage:
    CPU usage (in seconds): 138
  Memory usage:
    Memory (current): 1.01GiB
    Memory (peak): 1.25GiB
  Network usage:
    eth0:
      Type: broadcast
      State: UP
      Host interface: veth321964e4
      MAC address: 00:16:3e:b0:55:c4
      MTU: 1422
      Bytes received: 99.31MB
      Bytes sent: 674.90kB
      Packets received: 21391
      Packets sent: 8391
      IP addresses:
        inet:  <IP_ADDRESS>/24 (global)
        inet6: fd42:3db6:cbae:c768:216:3eff:feb0:55c4/64 (global)
        inet6: fe80::216:3eff:feb0:55c4/64 (link)
    lo:
      Type: loopback
      State: UP
      MTU: 65536
      Bytes received: 28.03kB
      Bytes sent: 28.03kB
      Packets received: 317
      Packets sent: 317
      IP addresses:
        inet:  <IP_ADDRESS>/8 (local)
        inet6: ::1/128 (local)

lxc exec bold-goat -- bash
root@bold-goat:~# ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
^C
--- <IP_ADDRESS> ping statistics ---
3 packets transmitted, 0 received, 100% packet loss, time 2048ms

root@bold-goat:~#

Basically OVN learns the IPv6 gateway (at fd55:13f7:9dcd:1::1/64) MAC address, but that may be due to router advertisements.
However OVN never learns the IPv4 gateway (at <IP_ADDRESS>/24) MAC address. Unfortunately we have to use dual stack due a lack of northbound IPv6 connectivity from this environment.

IPv4 discovery of the uplink gateway does work normally.

ipv4.routes: <IP_ADDRESS>/24

That looks suspect. You're indicating that the <IP_ADDRESS>/24 subnet can be used by OVN networks downstream, but this isn't valid because its the same network as the uplink, and includes the uplink gateway.

However running ping <IP_ADDRESS> from the LXD host adds the following MAC binding:
$ microovn.ovn-sbctl list MAC_Binding
_uuid               : bdebead3-8468-4761-90dc-fc853a963e4f
datapath            : d3217d6b-1e26-43e1-8e08-fec0c44e9e78
ip                  : "<IP_ADDRESS>"
logical_port        : lxd-net2-lr-lrp-ext
mac                 : "fa:16:3e:63:a1:f1"

And northbound connectivity is established from the container.

Try taking those out and see if it helps.

Will try, thanks for the heads up

Trying to remember why I set the routes in the first place, I believe this was for loadbalancers.

Did it fix it?
You might need to use multiple smaller route subnets, rather than assigning the whole one.
If it fixes it, we should probably add an overlap check to prevent assigned downstream routes from covering the uplink gateway.

Did it fix it?

Just rebuilding the environment atm, so I can start from a clean state.

Issue remains:
$ lxc network show uplink
config:
  dns.nameservers: <IP_ADDRESS>,2606:4700::1111
  ipv4.gateway: <IP_ADDRESS>/24
  ipv4.ovn.ranges: <IP_ADDRESS>-<IP_ADDRESS>
  ipv6.gateway: fd55:13f7:9dcd:1::1/64
  ipv6.ovn.ranges: fd55:13f7:9dcd:1::100-fd55:13f7:9dcd:1::253
  volatile.last_state.created: "false"
description: ""
name: uplink
type: physical
used_by:
- /1.0/networks/default
managed: true
status: Created
locations:
- lxd-virtual-edge-1
- lxd-virtual-edge-2
- lxd-virtual-edge-0

$ lxc network show default
config:
  bridge.mtu: "1422"
  ipv4.address: <IP_ADDRESS>/24
  ipv4.nat: "true"
  ipv6.address: fd42:5964:71c2:3d43::1/64
  ipv6.nat: "true"
  network: uplink
  volatile.network.ipv4.address: <IP_ADDRESS>
  volatile.network.ipv6.address: fd55:13f7:9dcd:1::100
description: ""
name: default
type: ovn
used_by:
- /1.0/instances/helped-caribou
managed: true
status: Created
locations:
- lxd-virtual-edge-0
- lxd-virtual-edge-1
- lxd-virtual-edge-2

$ lxc exec helped-caribou -- bash
root@helped-caribou:~# ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
^C
--- <IP_ADDRESS> ping statistics ---
3 packets transmitted, 0 received, 100% packet loss, time 2031ms

root@helped-caribou:~#

$ microovn.ovn-nbctl show
switch 3aed55f6-e3c8-4077-bf9a-86fcf4a0d5fa (lxd-net2-ls-int)
    port lxd-net2-ls-int-lsp-router
        type: router
        router-port: lxd-net2-lr-lrp-int
    port lxd-net2-instance-72cdf500-5f1a-44b7-9fbe-be601ee8e82e-eth0
        addresses: ["00:16:3e:a0:6a:f0 dynamic"]
switch 01705e83-2e8c-448b-a0f2-709e76aef3fb (lxd-net2-ls-ext)
    port lxd-net2-ls-ext-lsp-router
        type: router
        router-port: lxd-net2-lr-lrp-ext
    port lxd-net2-ls-ext-lsp-provider
        type: localnet
        addresses: ["unknown"]
router 99806d9c-82eb-4624-8efd-81adc0d07d34 (lxd-net2-lr)
    port lxd-net2-lr-lrp-int
        mac: "00:16:3e:24:25:12"
        networks: ["<IP_ADDRESS>/24", "fd42:5964:71c2:3d43::1/64"]
    port lxd-net2-lr-lrp-ext
        mac: "00:16:3e:24:25:12"
        networks: ["<IP_ADDRESS>/24", "fd55:13f7:9dcd:1::100/64"]
    nat 7a7fc07c-5822-4a42-b557-250bc1002c03
        external ip: "<IP_ADDRESS>"
        logical ip: "<IP_ADDRESS>/24"
        type: "snat"
    nat 7eaf9134-ed77-42e5-8952-f46b55a437de
        external ip: "fd55:13f7:9dcd:1::100"
        logical ip: "fd42:5964:71c2:3d43::/64"
        type: "snat"

$ microovn.ovn-sbctl list MAC_Binding
_uuid               : 879563ff-de61-4857-ab67-7906ed152db8
datapath            : dd678903-afe8-469b-b2e4-6fe422b8c8dd
ip                  : "::"
logical_port        : lxd-net2-lr-lrp-int
mac                 : "00:00:00:00:00:00"

Please can you use tcpdump on the uplink port to check if ovn is sending ARP requests?

Noticed something weird, if I ping the IPv6 gateway address with the empty MAC_Binding table,
$ ping6 fd55:13f7:9dcd:1::1
PING fd55:13f7:9dcd:1::1(fd55:13f7:9dcd:1::1) 56 data bytes
64 bytes from fd55:13f7:9dcd:1::1: icmp_seq=1 ttl=63 time=19.9 ms
^C
--- fd55:13f7:9dcd:1::1 ping statistics ---
1 packets transmitted, 1 received, 0% packet loss, time 0ms
rtt min/avg/max/mdev = 19.935/19.935/19.935/0.000 ms

OVN sends out a neighbour solicitation, and we learn the gateway address.
$ tcpdump -i ens4
tcpdump: verbose output suppressed, use -v or -vv for full protocol decode
listening on ens4, link-type EN10MB (Ethernet), capture size 262144 bytes
09:00:57.036939 IP6 fe80::f816:3eff:fe0a:dae3 > ip6-allnodes: ICMP6, router advertisement, length 56
09:01:03.155484 IP6 fe80::216:3eff:fe24:2512 > ff02::1:ff00:1: ICMP6, neighbor solicitation, who has fd55:13f7:9dcd:1::1, length 32
09:01:03.166733 IP6 fd55:13f7:9dcd:1::1 > ff02::1:ff00:100: ICMP6, neighbor solicitation, who has fd55:13f7:9dcd:1::100, length 32
^C
3 packets captured
3 packets received by filter
0 packets dropped by kernel

But pinging the IPv4 gateway results in failure.
$ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
--- <IP_ADDRESS> ping statistics ---
4 packets transmitted, 0 received, 100% packet loss, time 3047ms

And no ARP:
sudo tcpdump -i ens4
tcpdump: verbose output suppressed, use -v or -vv for full protocol decode
listening on ens4, link-type EN10MB (Ethernet), capture size 262144 bytes
^C
0 packets captured
0 packets received by filter
0 packets dropped by kernel

I experience something similar today on a fresh setup, seems like it may be an OVN bug.

@iherdgoats are you still experiencing this issue?

I've since moved onto other projects, so cannot confirm the fix.

Thanks. I've got something similar that @stgraber is experiencing with microcloud, although again, I am unable to reproduce, so will leave this open until we figure it out.

@fnordahl identified an issue in OVS that should be fixed in the next OVS 2.17 point release:
https://github.com/openvswitch/ovs/commit/182b9cb3524c5e7be20be7ebf061c6730a7f7e26


Fixing an issue with the API end point in the extension package of gemini + a little README typo

Hello again @svilupp,
Following-up to my previous pull request regarding the Gemini issue. If you decided to integrate GoogleGenAI.jl package, there's a minor missing part in end point that is used in GoogleGenAIPromptingToolsExt.jl.
Also, there is a little README typo that is corrected as well (could have made it in a another pull request but thought that this very minor).

Thank you for the PR. Would you mind separating the README typo please? I would accept that fix right away.
As for the GoogleGenAI, that was a light interface over old version of Tyler’s package. I suspect that it won’t be needed with the version he just requested for registering - we need to check what changes are required when the package is registered.
You can have a look if we need this function - some of my requests were captured here: https://github.com/tylerjthomas9/GoogleGenAI.jl/issues/2
There will be a need to have a look at what’s new and if we need to change how we interface with it.

I will wait for the registered version and see what should be done 👍🏼

Similarly to the other PR, I'd suggest closing this as resolved.


'Students work alone' message

On the student side, when looking at the landing page for an assignment, having a message that says 'Students work alone' when students choose not to make a group is not a necessary message. We should get rid of it.

Looking at this with @reidka, seems ok to leave it as is.


System.TypeInitializationException: The type initializer for 'ChineseToPinyinResourceHolder' threw an exception.

ℹ Computer information

PowerToys version: <IP_ADDRESS>
PowerToy Utility:
Running PowerToys as Admin: yes
Windows build number: Windows 10 Home 19041.546

📝 Provide detailed reproduction steps (if any)

… update PowerToys from AM auto-prompt
… restart
… error

(I clean my registry keys frequently with CCleaner and virus scan frequently)
✔ Expected result
Updated
❌ Actual result
Issue "Something went wrong."
📷 Screenshots
...

2020-10-10.txt

System.IndexOutOfRangeException: Index was outside the bounds of the array.
   Source: Pinyin4Net
   TargetAssembly: Pinyin4Net, Version=2<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
   TargetModule: Pinyin4Net.dll
   TargetSite: System.String <initializeResource>b__4_2(System.String[])
   at hyjiacan.util.p4n.ChineseToPinyinResource.<>c.<initializeResource>b__4_2(String[] l)
   at System.Linq.ParallelEnumerable.ToDictionary[TSource,TKey,TElement](ParallelQuery`1 source, Func`2 keySelector, Func`2 elementSelector, IEqualityComparer`1 comparer)

System.AggregateException: One or more errors occurred. (Index was outside the bounds of the array.)
   Source: System.Linq.Parallel
   TargetAssembly: System.Linq.Parallel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a
   TargetModule: System.Linq.Parallel.dll
   TargetSite: System.Collections.Generic.Dictionary`2[TKey,TElement] ToDictionary[TSource,TKey,TElement](System.Linq.ParallelQuery`1[TSource], System.Func`2[TSource,TKey], System.Func`2[TSource,TElement], System.Collections.Generic.IEqualityComparer`1[TKey])
   at System.Linq.ParallelEnumerable.ToDictionary[TSource,TKey,TElement](ParallelQuery`1 source, Func`2 keySelector, Func`2 elementSelector, IEqualityComparer`1 comparer)
   at System.Linq.ParallelEnumerable.ToDictionary[TSource,TKey,TElement](ParallelQuery`1 source, Func`2 keySelector, Func`2 elementSelector)
   at hyjiacan.util.p4n.ChineseToPinyinResource.initializeResource()
   at hyjiacan.util.p4n.ChineseToPinyinResource..ctor()
   at hyjiacan.util.p4n.ChineseToPinyinResource.ChineseToPinyinResourceHolder..cctor()

System.TypeInitializationException: The type initializer for 'ChineseToPinyinResourceHolder' threw an exception.
   Source: Pinyin4Net
   TargetAssembly: Pinyin4Net, Version=2<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
   TargetModule: Pinyin4Net.dll
   TargetSite: hyjiacan.util.p4n.ChineseToPinyinResource getInstance()
   at hyjiacan.util.p4n.ChineseToPinyinResource.getInstance()
   at hyjiacan.util.p4n.PinyinHelper.getUnformattedHanyuPinyinStringArray(Char ch)
   at hyjiacan.util.p4n.PinyinHelper.getFormattedHanyuPinyinStringArray(Char ch, HanyuPinyinOutputFormat outputFormat)
   at hyjiacan.util.p4n.PinyinHelper.toHanyuPinyinStringArray(Char ch, HanyuPinyinOutputFormat outputFormat)
   at Wox.Infrastructure.Alphabet.<InitializePinyinHelpers>b__5_0()
   at Wox.Infrastructure.Stopwatch.Normal(String message, Action action)
   at Wox.Infrastructure.Alphabet.InitializePinyinHelpers()
   at Wox.Infrastructure.Alphabet.Initialize(Settings settings)
   at PowerLauncher.App.<>c__DisplayClass15_0.<OnStartup>b__0()
   at Wox.Infrastructure.Stopwatch.Normal(String message, Action action)
   at PowerLauncher.App.OnStartup(Object sender, StartupEventArgs e)
   at System.Windows.Application.OnStartup(StartupEventArgs e)
   at System.Windows.Application.<.ctor>b__1_0(Object unused)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

@ryanbodrug-microsoft @crutkas The error is present in the pinyin NuGet package and is thrown during initialization. A similar issue has been reported here before: https://github.com/Wox-launcher/Wox/issues/2345. The error seems to be thrown on following line in the
Pinyin4Net library, possibly because of failure loading from a text file containing pinyin mappings. @bao-qian Have you seen this issue before ?

@somil55, we're calling this done?

@crutkas Yes!

Added in 0.25.  https://github.com/microsoft/PowerToys/releases/tag/v0.25.0


Implement runner(Covalent)-initiated data transfer

Runner (Covalent) initiated: Covalent pushes all assets (function, inputs, deps) to the compute backend before submitting the job and downloads the task outputs after the job completes. This is appropriate when only Covalent can initiate connections to the executor backend but not vice versa — such as when Covalent is installed in a personal workstation without a well-known IP address.

Hey team! Please add your planning poker estimate with Zenhub @AlejandroEsquivel @cjao @kessler-frost


Question

Hello, I'm a newcomer, sorry to bother you with these stupid questions.

do we add is_score_enabled again inside setting => [] ? cause that what I see in readme but I think we already have, outside.

I have DesignRequest extends FormRequest

and in rules() method I add
'g-recaptcha-response' => ['required', new GoogleReCaptchaV3ValidationRule('submit_design')]
in controller I injected it.
in view:
<form action=design.store>
// other fields...
{!! GoogleReCaptchaV3::renderField('submit_design_id','submit_design') !!}
</form>

I have the badge appeared
because {!!  GoogleReCaptchaV3::init() !!}
and everything seems fine when I submitted a request.
but  I want to know if I've configured it right so I assign wrong value for my secret key but I got no error.
Any idea?

do we add is_score_enabled again inside setting => [] ? cause that what I see in readme but I think we already have, outside.

You don't need to add it again in the setting => []. Yeah, I might need to re-wording it a bit to clear your confusion.

For the wrong value,

Your config might be cached, you might need to run php artisan config cache or php artisan cache:clear
Let me know if it solves your problems.

Thank you, now it won't submit the request, just like validation error but without any message. How do I get error messages and what would appear if suspicious user/bot submit a request?
Again, Thank You Very Much!

Thank you, now it won't submit the request, just like validation error but without any message. How do I get error messages and what would appear if suspicious user/bot submit a request?
Again, Thank You Very Much!

You have to check the Facade Usage section to see the messages. However, I think the validation class has already returned some error messages. If you see nothing, it might be something wrong or something missing from my package, I will check it later tonight.
Check Facade Usage to see the returned object first. I will get back to you tonight.

in my friend machine, he got a message that the site key isn't right when he didn't assign a value to it, Although I'm not sure why I don't even after config:cache, but I guess it works, which is great!!
Thank you for your attention and fast response

thats good！


Move RefreshSelectorMenuBuilder and RefreshMenuBuilder from jbpm-cons…

…ole-ng to uberfire-extentions to allow reuse across modules

Could someone review this one? @manstis, @ederign, @Rikkola

+1 to merge.

+1.


File icons not showing on WSL

code-server version: 1.604-vsc1.32.0
OS Version: Windows 10 running code-server on WSL (Ubuntu 18.04.2 LT)

Description
The file icons on the File Explorer panel do not show no matter what icon theme I choose.
I think it is related to WSL because if I open the same folder using the docker image the file icons show fine.
Steps to Reproduce

Run code-server on WSL and see the File Explorer panel.

I opened it on an Incognito window and it worked fine. So I clear my cache and reopened it on a normal tab and is fine. Not sure what happened but they are showing fine now.


SVG Adaptive icons

Hi, I'm wondering to know if is there plan to generate the adaptive icons using SVG.
thank you

I think app-icon only works on raster images. If you want to work with SVG images, you can check out https://github.com/aeirola/react-native-svg-app-icon instead. Disclosure, I made it.


Disable Model Occlusion

Would be p.  Nice clean cheat btw.
https://www.unknowncheats.me/forum/counterstrike-global-offensive/330483-disable-model-occulusion.html

Not bad, up for it.

Added.


run/end: check if .acbuild/target is mounted, unmount if so

If acbuild was killed in the middle of an acbuild run command, the
overlayfs it mounts at .acbuild/target would remain mounted. This caused
issues whenever the user attempts to use acbuild end to clean up after
the failed build.
Fixes https://github.com/appc/acbuild/issues/168.

Should be ready for another pass.

lgtm


feat: add _chainIdRpc option

Coverage decreased (-44.04%) to 38.561% when pulling e9c100cb723b496fb49abd47917a9368a1f65d00 on feat/_chainIdRpcOption into b6761e6a9b032fd0bc95748d9511206f3f913423 on develop.

Coverage decreased (-44.04%) to 38.561% when pulling e9c100cb723b496fb49abd47917a9368a1f65d00 on feat/_chainIdRpcOption into b6761e6a9b032fd0bc95748d9511206f3f913423 on develop.


Bug: approvals page "jumps" when approving

when on the approvals page, after approving several reports, some reports will go missing from each page, meaning the page will show "no more results" despite having heaps of approvals left.

https://github.com/TanStack/query/discussions/3360#discussioncomment-4023966


Anchore step fails curling inline scan script

Something is up with the shell script that the Anchore task is calling based on this error:
Starting: Anchore
==============================================================================
Task         : Anchore Security Scan
Description  : Anchore container security scan client
Version      : 0.0.3
Author       : Anchore
Help         :
==============================================================================
ANCHORE_CI_IMAGE:  docker.io/anchore/inline-scan-slim:v
/usr/bin/curl --silent --fail --show-error --output /tmp/inline_scan.sh https://ci-tools.anchore.io/inline_scan-v
curl: (22) The requested URL returned error: 403
curl: (22) The requested URL returned error: 403

undefined
##[error]Curl command failed
Finishing: Anchore

Navigating to the script url directly also produces a 403 Forbidden. This is my first time trying this extension, I do not have it in use anywhere else.

I just noticed the step failed to add in the version number, it is working now :)


New element: Universe

Where does this comes from?
The idea is to tag the universe of an issue within the metadata. Each issue would have 0...* universes associated with it (potential for multiple in issues where they leap between universes). The only difficulty I can see with this is validating results for consistency, however I assume usage is outside the scope of this project.
What is the rationale for adding support for this element?
There are so many universes it is difficult to keep track of what is happening where.
It would be nice to be able to filter within reading software by Universe. For example, the ability to filter to show only comics from the Star Wars universe within a library of Marvel comics. This would also help users find similar content set within the same universe (Image Comics Massive-Verse, Marvel's Ultimate Universe, etc).
Is the element already handled by any application or tool?
I have just raised this as a feature request with Metron, however I'm not aware of any software currently using this. https://github.com/bpepple/metron/issues/173
There is potential for some overlap with the Locations tag, however that seems to be more general in nature (and thus unusable for filtering in software).

How is that different from a tag or the book being part of a collection?

Is there anything that couldn't be a tag or collection? The main benefit to having a specific element is the same as for any other property (Team, Location, etc) currently listed in the schema.
This is a piece of information that will exist for every issue ever released; it's not an optional tag based on user preference in the traditional sense so much as a useful piece of information about every issue which helps provide context to the contents of the issue. Having a tag titled earth-616 or earth-1610 isn't really going to provide much in the way of clarity for end users. It also allows for way too much ambiguity when it comes to validation or consistent approaches to storing the data IMO.

My problem with making a universe tag is that the term is mostly specific to marvel comics. There are lots of different websites that allow categorizing content from different fandoms (for lack of a better word) and none of them have a universe property to sort by. There is also a huge number of comics that don't have an internal concept of universes or alternate realities or anything that would fit this field. In any case I don't believe that it belongs in comicinfo, it's not stated much of anywhere or very well but the intent is to fix the biggest issues with the format and keep it backwards compatible with previous versions not to make it the standard to use also this format is not designed for end-users this format is for existing applications to use. You will get better reception in the https://github.com/anansi-project/rfcs repo where hopefully eventually a new format will eventually be born. You can also look at https://github.com/Metron-Project/metroninfo where metron is testing out it's own metadata format.
Also if we want to make it make sens with DC's multiverse they start using terms like 'omniversal overvoid' and I don't think this standard should be involved with those type of words

The DC Multiverse isn't the only Multiverse; It's a part of an Omniverse filled with other Multiverses, sharing the omniversal overvoid with the Marvel Multiverse, the Wildstorm Multiverse, and others.[61] For a full list of worlds, see Multiverse/List of Universes.

Is it mostly specific to Marvel though? The reason I actually thought of this was Image Comics recent Massive-Verse, which has several concurrent series in the universe. Looking at Wikipedia, there's actually quite a few Image universes (albeit far less than there are in Marvel or DC). DC Comics also has a pretty extensive list.
Sure, a lot of the indie content probably doesn't have this concept embedded, but there's enough for it to be meaningful IMO. Having different series following a set group of characters with different series names could be nicely organised by Universe (here's a list of some of the big ones). I get your point about tags and collections, I just don't think they're specific enough, and they definitely don't allow for any validation or consistency.
I was a little uncertain whether this fit into comicinfo or RFCS aha. I'm happy to move it over there if it's more well suited?


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/541 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit ee3e02c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded

docs/to-be-archive/level1/sql-database-engine.png
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Update proptest

This updates proptest from 0.9.1 to 1.1.0. This helps drop some old, unsupported dependencies like fuchsia-cprng from an old version of rand.
Changelog: https://github.com/proptest-rs/proptest/blob/master/proptest/CHANGELOG.md
I don't see any particularly relevant breaking changes for our use.

cc @Eh2406 If you want to review.

@bors r+
Thank you!

:pushpin: Commit 3e20ffda68557cf944369ab37a675d815d1f6a6f has been approved by Eh2406
It is now in the queue for this repository.

:hourglass: Testing commit 3e20ffda68557cf944369ab37a675d815d1f6a6f with merge 7a598bfc3a58a2398e64c615dc81de290634f31b...

:sunny: Test successful - checks-actions
Approved by: Eh2406
Pushing 7a598bfc3a58a2398e64c615dc81de290634f31b to master...


Turn on Intel and AMD IOMMU support and interrupt remapping

Signed-off-by: Pooja Trivedi<EMAIL_ADDRESS>Description

Why is this needed

Fixes: #
How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

Hey @ptrivedi, would you mind a short description to the PR body and also the "Why this is needed" to the PR description and/or to your commit message, please?


Using the allow-list for the "ComposeM2Api" issue does not work since the resolved name is mangled and does not always match the ones passed into the Allow-list

As I was using the M2ApiDetector, I noticed that on this line, (resolved as? PsiNamedElement)?.name actually returns an odd name.
For example for when I was using androidx.compose.material.Text it returned Text--4IGK_g as the resolved name.
And to my surprise, when I switched over to using the slack lints instead of my custom one, when I wrote my lint.xml file like this
<?xml version="1.0" encoding="UTF-8"?>
<lint>
    <issue id="all" severity="ignore" />
    <issue id="ComposeM2Api" severity="error">
        <option name="allowed-m2-apis" value="Text--4IGK_g" />
    </issue>
</lint>

I did not get the warning on this, so it was passing the in allowList.value check since it seems to consistently resolve to Text--4IGK_g for its name.
When I switched the option to be <option name="allowed-m2-apis" value="Text" /> again, it was not properly getting filtered out through the AllowList.
Copying the rule locally and changing the source code to do this instead
val resolvedName = (resolved as? PsiNamedElement)?.name
val resolvedNameWithoutNameMangling = resolvedName?.substringBefore("-")

did fix this issue, but this feels like a hack if anything.

Filed this https://issuetracker.google.com/issues/297544175 too which if resolved in any way will be relevant to fixing this issue.

it seems to consistently resolve to Text--4IGK_g for its name.

That's because compiler (kotlinc) consistently mangles that declaration (deterministic!). I wonder what you'll get if you change the version of androidx.compose.material. 😉

resolvedName?.substringBefore("-")

I was about to say this is fragile, but maybe quite effective(?) as per compiler's ABI: https://github.com/JetBrains/kotlin/blob/master/compiler/ir/backend.jvm/src/org/jetbrains/kotlin/backend/jvm/InlineClassAbi.kt#L90

Thanks a lot for the entire write-up in the issue tracker, this is very informative!
I will continue with this approach in my project, by making this assumption on the "-" being there, which does seem effective enough for my needs. I do wonder if this is something that this library would also be interested in doing or not though. Not sure if there are any other alternatives atm. That's what I understand from the explanation you wrote in the issue tracker.

@StylianosGakis I'm going to add a flag to enable this conditionally (opt-in) in the next release if you want to try it. I don't know how to make a repro case for this on its own though

Actually I think this is safe enough to do opt-out

I agree with having it as default with an option to opt out, I like it better this way too.


(chore) add coveralls

Add coveralls for all packages.

Coverage increased (+18.4%) to 100.0% when pulling c2818c262ca36749be9afdf400b183423918596c on code-coverage2 into 137326a6e521c41fee767a5e1675c8bbe81e5c73 on master.

Coverage decreased (-1.08%) to 80.513% when pulling f428ba9b80d0335dc95b80bd9ec5e991b8efb497 on code-coverage2 into 137326a6e521c41fee767a5e1675c8bbe81e5c73 on master.


streamline authorization

It would be great if a user only had to think about authorization (e.g. providing a client_secrets file) the first time they install this package and from then on they are all set, regardless of their current working directory.

This was fixed by @siddharthab in c94c48691ffb439d3431549ed4962634a6f4ffc4


Tag new releases with their version number

At the moment I believe when a new version of flagger-k6-webhook is released it just gets tagged with 'latest'.
Would it be possible to also tag the new images with the actual version as well please? For example, k6-loadtester-1.2.0 was released back in October but there are no images available with this tag from  what I can te;ll.
At Aviva, we prefer to use a tagged 'version' of each 3rd party image so that any upgrades of the image on our clusters is a deliberate action rather than being caught out by accidentally downloading a newer version without realising it.

Hi :) We'll try to do that consistently in the future. Yesterday, we released flagger-k6-webhook v0.3.1 and the Helm chart in version v1.3.1.
The image is available under ghcr.io/grafana/flagger-k6-webhook:v0.3.1
I hope this helps 🙂

@zerok Can you elaborate on the versioning scheme? Why the two different versions?


executor: fix out of bounds write in lookup_connect_response_in()

gcc 12.1 reports the following -Werror=array-bounds error:
///
In function 'bool lookup_connect_response_in(...)'
executor/common_usb.h:632:66:
error: array subscript 'usb_qualifier_descriptor[0]' is partly outside array
bounds of 'char [8]' [-Werror=array-bounds]
|
632 |        qual->bNumConfigurations = index->dev->bNumConfigurations;
|        ~^~~~~~~~
In function 'volatile long int syz_usb_connect_impl(...)':
executor/common_usb_linux.h:332:23: note: object 'response_data' of size 8
|
332 |        char* response_data = NULL;
|              ^~~~~~~~~~~~~
...
In function 'bool lookup_connect_response_in(...)',
executor/common_usb.h:633:57:
error: array subscript 'usb_qualifier_descriptor[0]' is partly outside array
bounds of 'char [8]' [-Werror=array-bounds]
|
633 |        qual->bRESERVED = 0;
|        ~~^
In function 'volatile long int syz_usb_connect_impl(...)':
executor/common_usb_linux.h:332:23: note: object 'response_data' of size 8
332 |        char* response_data = NULL;
|              ^
///
Current code in USB_DT_DEVICE_QUALIFIER case treats respose_data as a buffer,
but in reality it is just a pointer, as detailed in the error trace above. In
order to allow passing a usb_qualifier_descriptor struct back to the caller
(via response_data), add a new parameter to lookup_connect_response_in().
Build tested only.
Fixes: 0c00210ff32 ("executor: always provide DEVICE_QUALIFIER USB descriptor")
Signed-off-by: Ovidiu Panait<EMAIL_ADDRESS>
Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

Looks reasonable, thanks.
If I am reading the code correctly, this line you added also seems to be important:
*response_data = (char*)qual;

because later we do:
		if (response_data)
			memcpy(&response.data[0], response_data, response_length);
		else
			memset(&response.data[0], 0, response_length);

So if we don't set response_data, we still send all 0s and filling qual is pointless.


TypeError: Cannot read properties of undefined (reading 'WebSocket')

just followed the instruction.
npm install @opensea/stream-js
install --save-dev @types/phoenix
then try to construct a OpenseaStreamClient and will encounter this issue.
C:\portfolioProgram\nodejs\node_modules\phoenix\priv\static\phoenix.cjs.js:781
this.transport = opts.transport || global.WebSocket || LongPoll;
^
TypeError: Cannot read properties of undefined (reading 'WebSocket')
How should i solve this?

Having same issue

having the same issue, however tried it on a react/browser setting, it worked

worked

is it react/browser setting issue? what did you modify?

worked

is it react/browser setting issue? what did you modify?

Did not modify anything, was trying on nodejs but got the same error, tried it on create-react-app and it worked

having similar issue :(

You need to
npm i ws
then import WebSocket
add
        transport: WebSocket,
    },```
to OpenSeaStreamClient

const {WebSocket} = require('ws')
const wsClient = new OpenSeaStreamClient({
network: Network.MAINNET,
apiUrl: "wss://stream.openseabeta.com/socket",
token: osKey,
onError: console.error,
connectOptions: {
transport: WebSocket,
},
})

We recommend switching to Node.js version 16 to make sure common crypto dependencies work. Our minimum supported version is 16.11.0.
Install this package with npm install @opensea/stream-js
NodeJS only: Install the WebSocket library with npm install ws
Install the required dev-dependency to use the Phoenix client for web socket connections npm install --save-dev @types/phoenix
Closing this issue, if you are still running into blockers please let me know we can reopen this issue.


Cache the latest block

This is part of the definition of done for the checksum backlog issue.

@TrustHenry can you Review & Approve this PR?

The reading of the checksum at start-up was merged in https://github.com/bpfkorea/agora/pull/275, so this PR is now only about caching. I've rebased it.


Automated System Naming, Displaying Region, and distance to nearest trade hub on system nameplate

I couldnt find where to submit suggestions so ill put it here.

Set it so the region for K-space systems is always displayed under the system name.
Integrate an auto naming system like Vippy has.
Display the distance to the nearest trade hub on the System nameplate.
Ability to update who the current inhabitant of a Hole is. (In vippy you can update who lives in a hole and that information will be used univerally with anyone else using vippy thus creating a database of what corps are active in what hole, That information is then displayed on the nameplate for the system)
These four features are the main things that are  keeping my corp from switching to pathfinder.

Here is a screen shot of what i am talking about.  You can see how the Mapper gives a name to the next hole of that type alphabetically. That makes is easier to tell others where you are or where you are going and makes for much easier bookmark tracking in game.
I understand that this information is available with a click or a scroll in pathfinder but we would like it so its something that could be seen at a glance without having to interactive with the map at all(Thus everything displayed on the nameplate). Even if this was something that could be permanently toggled on or off it would be great. Thanks for reading.

Thanks for your suggestions @Meekss :)
@WildStrawberryEVE is right, most of the features are already there. I´m always up for UI improvements and reduce the required clicks for users to show them the information they need.
But, as he said, moving more information into the map view requires good arguments for it. If an information is just 1 click off (e.g. region names) I don´t see the point moving it into the map view and make is always accessible  even if it not used that often.
Also see this Issue for "auto system aliases" #360
Nether the less, there is a feature requests I like:
Save inhabitants to a system would be a nice addition to the "Intel/structure" panel. (Not sure about the "auto share" feature for that information. I´m sure some groups don´t want to share their intel with others)
Please keep in mind, that Issues like this, are very unhandy. Throwing multiple feature requests in one Issue is a bad practice. - So people can discuss on a single topic/feature.  That´s why I have to close this one. Feel free to create new issues with the features you really need.


Update google-cloud-pubsub to 1.114.0

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-pubsub" } ]

labels: library-update, semver-minor

Superseded by #255.


Add support for ng-required

When the multiple-date-picker is used inside a form, support for the ng-required attribute would be great in order to disable the submit button as long as not at least one date is selected.
Maybe support for a mimimal amount of dates that have to be selected would also be interesting - so in order for the submit button to be enabled at least for example 3 dates have to be selected. This could be done for example with min and max attributes.

Hi,
you can manually do it by watching the ng-model, I'll dev it but not right now, I'm back from holidays and have some work to do ;)

just watch the ng-model, I don't want to support ng-required because we don't know what you require (1 day selected at least, only ?)


0/1 Knapsack using Dynamic Programming in JAVA

💥 Proposal
I am looking forward to work on the implementation of 0/1 Knapsack problem using Dynamic Programming in Java.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


New project hook is not executable

I have created a new blank project (ionic start my-app blank)
And now I am trying to build for android (cordova platform add android) I see this error
Running command: .../my-app/hooks/after_prepare/010_add_platform_class.js .../my-app
execvp(): Permission denied

The problem is that the created file /hooks/after_prepare/010_add_platform_class.js is not executable, and cordova expects it to be
If I run chmod +x .../my-app/hooks/after_prepare/010_add_platform_class.js then the problem is fixed.

You should use ionic platform add android instead of cordova. The chmod on the hook file actually happens from the ionic command.

Actually, I don't think that's the issue. When you generate a new project with ionic start (v 1.3.20) the 010_add_platform_class.js file is created with -rw-r--r--    Running ionic platform add after that does not seem to modify those permissions.

I have the same issue like @uooq on osx v.10.11.3


Typescript errors in node_modules

Hey everyone! I'm porting my current monorepo to this awesome setup and I'm facing some new issues that I didn't have before. I have some 3rd party libraries installed in my repo and I'm getting some errors on those libraries' code that I didn't have before.
The thing is that even though I have my node_modules folder excluded from transpiling within my tsconfig.json file (the same way you did), I'm still getting typescript errors from those libraries.
You can replicate this issue just by cloning your repo and installing react-native-linear-gradient or react-native-dialog in the app package, add some usage in the App.tsx file and then start the web package. You will see something like:

I don't know if I have to configure babel in a certain way in order to make this error disappear, but my previous monorepo was also using a CRA scaffold for the web package and I was not getting any of this errors.
Thank you very much,
Eric.

I have just tested this again starting from scratch on your monorepo scaffold and I'm seeing the same issues. For example if you want to install react-native-linear-gradient for the web you need to:

Run yarn add react-native-linear-gradient on the app package
Run yarn add react-native-web-linear-gradient on the app package
Add the webpack's alias section to the web's craco.config.js:

webpack: {
    alias: {
      'react-native-linear-gradient': 'react-native-web-linear-gradient',
    },
    ...
}
`

- Use it anywhere in app's App.tsx:

<LinearGradient colors={['#4c669f', '#3b5998', '#192f6a']} style={styles.linearGradient}>

Sign in with Facebook

- And add the styles

linearGradient: {
flex: 1,
paddingLeft: 15,
paddingRight: 15,
borderRadius: 5
},
buttonText: {
fontSize: 18,
fontFamily: 'Gill Sans',
textAlign: 'center',
margin: 10,
color: '#ffffff',
backgroundColor: 'transparent',
},

- Start the web app and you will get those errors:
<img width="348" alt="image" src="https://user-images.githubusercontent.com/7376892/141474947-79d0990d-aa15-4f66-b1cf-ea49baf1cceb.png">

I'm not sure if this is something that has happened in the past.

Thanks,

Eric.

@ericlifs sorry for being late, I'm busy these days.
Can you try following the same flow we're using with the @react-native-async-storage/async-storage package? It's a native module as well, so I would expect react-native-linear-gradient to work in the same way.


APEXMALHAR-2001 Fixed checkstyle violations in Kafka

@sandeshh @tweise please review and merge as appropriate

Please squash your commits for all PRs


Updated copyright notices (#1477)

Signed-off-by: Dave Lago<EMAIL_ADDRESS>opensearch-security pull request intake form
Please provide as much details as possible to get feedback/acceptance on your PR quickly

Category: (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)
Maintenance

Github Issue # or road-map entry, if available:
Backport of #1477 for 1.2 release branch

Description of changes:

Why these changes are required?

What is the old behavior before changes and new behavior after changes? (Please add any example/logs/screen-shot if available)

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)

Is it backport from main branch? (If yes, please add backport PR # and commits #)

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1495 (88c7582) into 1.2 (8ecb2d7) will not change coverage.
The diff coverage is n/a.

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

@@            Coverage Diff            @@
##                1.2    #1495   +/-   ##
=========================================
  Coverage     64.85%   64.85%
  Complexity     3227     3227
=========================================
  Files           247      247
  Lines         17335    17335
  Branches       3072     3072
=========================================
  Hits          11242    11242
  Misses         4536     4536
  Partials       1557     1557

Impacted Files
Coverage Δ

...ic/auth/http/jwt/AbstractHTTPJwtAuthenticator.java
50.00% <ø> (ø)

...mazon/dlic/auth/http/jwt/HTTPJwtAuthenticator.java
82.07% <ø> (ø)

...t/keybyoidc/AuthenticatorUnavailableException.java
0.00% <ø> (ø)

...th/http/jwt/keybyoidc/BadCredentialsException.java
40.00% <ø> (ø)

...byoidc/HTTPJwtKeyByOpenIdConnectAuthenticator.java
94.11% <ø> (ø)

...azon/dlic/auth/http/jwt/keybyoidc/JwtVerifier.java
85.71% <ø> (ø)

.../dlic/auth/http/jwt/keybyoidc/KeySetRetriever.java
76.82% <ø> (ø)

.../auth/http/jwt/keybyoidc/SelfRefreshingKeySet.java
59.85% <ø> (ø)

...ttp/jwt/oidc/json/OpenIdProviderConfiguration.java
100.00% <ø> (ø)

...ic/auth/http/kerberos/HTTPSpnegoAuthenticator.java
14.21% <ø> (ø)

... and 201 more

Continue to review full report at Codecov.

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


WIP: Start Open Source Documentation

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #273
Type of change

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules

@JumaX @domenicbove  Linking you here for discussion.
So imho there are two things to be done to get a "none-commercial" version of the confluent stack.

Document open source usage and create an example inventory

Do not install any packages that are "commercial". I am a bit lost with all the packages confluent is offering, so maybe you can help me out here :)
If I am correct this is a list of all packages that are possibly installed within this repo.

confluent-common :question:
confluent-rest-utils :question:
confluent-security :question:
confluent-kafka :heavy_check_mark:
confluent-schema-registry  :heavy_check_mark:
confluent-kafka-connect-storage-common :heavy_check_mark:
confluent-kafka-connect-s3 :heavy_check_mark:
confluent-kafka-connect-elasticsearch :heavy_check_mark:
confluent-kafka-connect-jdbc :heavy_check_mark:
confluent-ksqldb :heavy_check_mark:
confluent-kafka-rest :heavy_check_mark:
confluent-server :x:
confluent-control-center-fe :x:
confluent-control-center :x:
confluent-rebalancer :x: (assuming this is the auto data rebalancer)
confluent-metadata-service :question: (not sure here - also #197 )
confluent-kafka-connect-jms :x:
confluent-kafka-connect-replicator :x:

Could you review this list and maybe also amend if there are any further steps I forgot about so far?

@Fobhep When I read your issue on this one, I thought you were asking us to install opensource kafka and configure it, not just install our free bits. I would need to ask some people internally. The big one I know of is confluent-kafka-2.12 vs confluent-server, the later being not free

From talking to various confluent people I always had the understanding that using the "open source and community (aka free) parts" was perfectly fine and also intended :) Maybe I am wrong there?
Also since those parts are partly documented https://docs.confluent.io/current/installation/available_packages.html
We can of course also change this PR completely and use not Confluent packages at all for an open source kafka installation - is this more what you excpected or are there any other possible scenarions you would see in the scope of this repository? :)

Hey! So I do not think we have plans to help with the install of opensource kafka, but I dont see a reason why helping install our community packages shouldnt be an option.
I did try one thing which might answer your question on the packages. If those docs are correct, then running yum install confluent-community-2.12 should get you the community packages and their dependencies. And heres an out put of that command:
Dependencies Resolved

=======================================================================================================================================
 Package                                                Arch                   Version                 Repository                 Size
=======================================================================================================================================
Installing:
 confluent-community-2.12                               noarch                 5.5.0-1                 Confluent                 3.6 k
Installing for dependencies:
 confluent-common                                       noarch                 5.5.0-1                 Confluent                 121 k
 confluent-kafka-2.12                                   noarch                 5.5.0-1                 Confluent                  57 M
 confluent-kafka-connect-elasticsearch                  noarch                 5.5.0-1                 Confluent                 2.2 M
 confluent-kafka-connect-jdbc                           noarch                 5.5.0-1                 Confluent                 8.1 M
 confluent-kafka-connect-s3                             noarch                 5.5.0-1                 Confluent                  22 M
 confluent-kafka-connect-storage-common                 noarch                 5.5.0-1                 Confluent                  40 M
 confluent-kafka-rest                                   noarch                 5.5.0-1                 Confluent                  47 M
 confluent-ksqldb                                       noarch                 5.5.0-1                 Confluent                  96 M
 confluent-rest-utils                                   noarch                 5.5.0-1                 Confluent                  19 M
 confluent-schema-registry                              noarch                 5.5.0-1                 Confluent                  83 M

Transaction Summary
=======================================================================================================================================
Install  1 Package (+10 Dependent packages)

Now, thinking about this PR. I think there will need to be more logic gates around certain properties/configs. Like i know c3 packages are included in some components so they can configure interceptors... ie if you dont install c3 packages then the interceptors code should be skipped.

hey @domenicbove Thanks for the feedback!
And a great catch - the community-package should be exactly what we need.
I see - thanks for the remark. I will try to dig into that and see what I can forge together :)

@domenicbove So I had a first idea how to deal with the package selection inside of roles/confluent.variables_handlers/vars/main.yml.
I am not sure how to best deal with a scenario where a user sets confluent_community: true but also confluent_server_enabled: true
That is why I introduced the "hard rest" in roles/confluent.common/tasks/main.yml.

Can you make this PR into the master branch? we are moving away from adding features to already "released" branches. Master is going to become the 6.0 branch which will be cut later this summer

So another thing I noticed when trying this out, is the conditionals on certain properties need to be updating. For example the metrics reporter bit in the kafka_broker properties I don't think works when community is enabled.... I am not sure of the full list of properties (features) that should be turned off, but its all handled in variables_handlers/vars/main.yml

@domenicbove is correct regarding metrics reporter, this is proprietary to Confluent-Server and thus will not work with pure open source. It's used to supply metrics to Control Center.  This would apply to metrics configs across all components.

@domenicbove is correct regarding metrics reporter, this is proprietary to Confluent-Server and thus will not work with pure open source. It's used to supply metrics to Control Center. This would apply to metrics configs across all components.

ok thx - I will have a look at what I can find

@domenicbove @JumaX would the metrics reporter thing actually be "resolved" with #197 being merged and having a logic that triggers that feature when confluent_community: true

@domenicbove I finally had the chance to continue on this PR. I hope I caught all of the metrics reporters :)

I'm really interested in this PR is there any chance to get it reviewed and merged soon ?
Thanks for all your work.
Best,
Jérome

@jrevillard I will continue this hopefully later today or tomorrow by rebasing it on latest changes and putting in your fixes.
I will afterwards ping the repo maintainers to make sure we get some feedback and can continue from there on

@jrevillard  @JumaX @domenicbove  I updated this PR - please feel free to review.
Sadly I cannot check the failed Jenkins state - if somebody has an idea how I can fix that please let me know.

@Fobhep Now that we have documentations, do you want to proceed with this PR? If yes, then can you please make it a regular PR and resolve the conflicts.

IMO, this is taken care of. We can reopen if needed.

IMO, this is taken care of. We can reopen if needed.


Is there an API for file signing LSA plugins and UEFI firmware?

There is  Hardware dashboard API documentation. However, from examples provided it is not clear if and how the API can be used for LSA/UEFI modules. Please clarify.

Hi @kirillkovalenko,
I'm not really sure how to address your feedback. Do you have a specific task you're trying to accomplish for which you can't find documentation?
~ Mark

Hi @mhopkins-msft ,

Do you have a specific task you're trying to accomplish for which you can't find documentation?

Yes, I have. I want to automate LSA modules signing into our CI pipeline. I don't a documented API to accomplish it and currently I have to execute this task manually. However I see an API for automating very similar task,  i.e. hardware drivers signing. My guess was that LSA modules signing API actually exists but is not documented yet.

Hi @kirillkovalenko,
I checked with the dashboard development team and was told that there is currently no API for LSA signing.
Thanks,
~ Mark

Has the situation changed?


Initial version of ArrayStack

Hi, this PR implements an ArrayStack inspired by the ArrayQueue implementation.

This implementation is wrong. When running pop, after decrementing top from t to t-1, another thread can concurrently push and write at t-1. This races with the popping thread's read of t-1. Even if the accesses are made atomic, it won't be linearizable.


Output in other languages

Hey there,
again, thanks for this great script!
Just want to share what I did to have the output in other language (for me in german)
I just changed 6 locations of "Please provide a detailed summary of the" into "Please provide a detailed summary in german of the" in lines 451, 520 and 584
Perhaps you can build this into the script as a parameter....
Many greetings from germany
Olli

Nice! I was expecting to hear that you had prompted it with something like “Bitte geben Sie eine detaillierte Zusammenfassung der”, but if asking in English for a summary in German works just as well, then we could easily add a command-line language selector flag and include that into the English prompt like you did manually, if present.
You could probably even get ChatGPT to do it for you, if you’d like to PR the change. Labeling this as a good first issue to see if someone wants to pick that up and try it.

Good work, thanks! I also updated the README to indicate how to use the new language selection feature.


docs: Add weekly-report

Fixes: #[Add issue number here]
Changes:

Screenshots of the change:

Please merge this before https://github.com/apache/apisix-website/pull/1612.
If merge https://github.com/apache/apisix-website/pull/1612 before this, it will create merge conflicts, so just don't.


Add People Data Labs to Business Section

Added People Data Lab's API (Self Serve + Free Tiers Available) to the Business Section

Whoops, accidentally submitted this to your fork hah - apologies


wmic deprecation warning

Was debugging printing on Windows and ran across this deprecation warning:

https://learn.microsoft.com/en-us/windows-server/administration/windows-commands/wmic
I don't have much experience with Windows development, but I'm curious if using Get-Printer would be better:
https://learn.microsoft.com/en-us/powershell/module/printmanagement/get-printer?view=windowsserver2022-ps

Fixed in #13 adopting powershell like you proposed in #9


Twitter likes fail

Need a bit of help to understand if the issue is on my end or on bridgy's side. I'm not sure if it's a timing thing or related to my latest blog post, but either way, since this morning all webmentions to my latest post don't come through and show as failed on my listing: https://brid.gy/twitter/islayblog
I had a look at the logs (eg this one) but I'm not sure if the cannot find target link error 400 is on my side or on bridgy's end (unfortunately my coding and log reading skills are basic). The post on my blog does exist, as does the endpoint on my blog.
Any help/guidance appreciated.

Thanks for the report, and sorry for the trouble! I see the problem, it's due to #1037. Should be a straightforward fix.

OK, should be fixed. Thanks again for the heads up!

Yes, coming all through now, thanks for fixing so quickly!


OptionalParameter:deserialize log level changed to DEBUG

OptionalParameter:deserialize logging level changed to DEBUG so we can tune via log4j settings.
it makes too much noise for log4j when delivery report has unknown tag when the level is INFO

i've provided the test method for second commit.


Release: 0.11.0

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.96%. Comparing base (03b4e9d) to head (fa9f79c).

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #154       +/-   ##
===========================================
- Coverage   83.72%   67.96%   -15.76%
===========================================
  Files          15       15
  Lines        1948     1948
  Branches      321      321
===========================================
- Hits         1631     1324      -307
- Misses        214      547      +333
+ Partials      103       77       -26

Flag
Coverage Δ

pytest
67.96% <ø> (-15.71%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 5 files with indirect coverage changes


How to request recursively dependencies?

There are some tree-like datastructures. How deal with this?
For example: The tile-view of motion-list.
Relates to openslides-autoupdate-service#219.

I will close this in favor of openslides-autoupdate-service#219.


Updated Wordpress Fingerprint and Documents

From my testing I got two scenarios where subdomain takeover is possible using Wordpress.com services.
Scenario-1:
If subdomain name is somethingtesttarget.target.com and if it's pointing to WordPress and vulnerable to takeover then visiting the subdomain will take user to https://wordpress.com/typo/?subdomain=somethingtesttarget where error page will look like below which confirms it's vulnerable to takeover

Scenario-2:
If subdomain name is something_test.target.com and if it's pointing to WordPress and vulnerable to takeover then visiting the subdomain will take user to https://wordpress.com/typo/?subdomain=something_test where error page will look like below

Note that it even says The address something_test.wordpress.com cannot be registered. Site names can only contain lowercase letters (a-z) and numbers. but ignore this as you can register a domain via a domain mapping upgrade of Wordpress.com and it will not matter what the underlying .wordpress.com address is.

How to Takeover and create P0C
To takeover a subdomain we need to use Domain Mapping service what is only available for Paid account so you need to buy the Personal package worth 48$ and then

Create a site using your wordpress free account .

In following page past the subdomain name you want to takeover and click on Add button and you will be taken to Checkout webpage. [ You will also asked for upgrade account in these steps in that time select Personal package ]
Now In checkout page pay the pricing and you will get Domain Mapping free with you plan.
As I am already on a upgraded account of mine my checkout page will look like below

Now click on Complete checkout page and the domain will be added in your account and you can see it at https://wordpress.com/domains/manage
Now click on three dots and select  Make Primary Domain to serve your P0C page on that subdomain directly.

Happy Hacking <3

Today I just got another Scenario for wordpress subdomain takeover. I will call this Scenario-3
Scenario-3

If subdomain name is somethingtesttarget.target.com and it's pointing to WordPress CNAME somethingtesttargetdottargetdotcom.wordpress.com
If there is already a site exist on somethingtesttargetdottargetdotcom.wordpress.com
Then you will see a error page saying Warning! Domain mapping upgrade for this domain not found. which is also vulnerable to takeover

To takeover just follow the same steps to add the domain with you account via domain mapping service ;

Hey @codingo
can you take a look here ?
Thanks.

Hey @codingo
can you take a look here ?
Thanks.

Hi @EdOverflow @0xPrial

What if we have this as shown in POC
Thanks

The finger print may change  according to this article
https://sapt.medium.com/wordpress-subdomain-takeover-on-bugcrowd-private-program-f59b5a0d74a7

The finger print may change according to this article
https://sapt.medium.com/wordpress-subdomain-takeover-on-bugcrowd-private-program-f59b5a0d74a7

Hi @cyb3rsalih,
I can confirm wordpress update contents of error page for Scenario-3 I described in #issuecomment-699688260
Thanks for your update <3


IndexOrDocValuesQuery does not support query highlighting

Description
While working with the IndexOrDocValuesQuery in https://github.com/opensearch-project/OpenSearch/issues/7057, I noticed that highlighting was broken. This is potentially caused by the extract function that does not check if the query is indexordocvalue.
https://github.com/apache/lucene/blob/main/lucene/highlighter/src/java/org/apache/lucene/search/highlight/WeightedSpanTermExtractor.java#L128
Version and environment details
Lucene version 9.8.0

Hey @harshavamsi the PR to support the IndexOrDocValuesQuery for query highlighting is now merged.
Thanks.
@getsaurabh02 @msfroh

Thanks @prudhvigodithi!


Rel/1.0.1

Not actually merging this. Creating this branch to build 1.0.3.
cc @pranavkm The ripple effect of Configuration version update is going to be massive...

Updated to use this branch to build 1.0.3.


Source code for eng and src missing

Hello all,
I do not see the source code for the Upgrade assistant project here. Specifically the eng, src as well as the legacy try-convert folders. Was there any change done after 0.5.2? If so, where can I get the earlier versions of the source code - is there any branch where I can take the same? Any changes to this open source project?
Regards,
Ganesh.

The Upgrade Assistant that used to be in this repo has been replaced by closed-source versions. We have repurposed this repo to host 3rd party mappings that can be used by the newer Upgrade Assistants (VS extension or CLI).

Thank you!


fix ReplyKeyboardMarkup serialization error

@RomanErnst thank you for opening the pr. do you need a new version of the library today or can it wait until tomorrow?

Thanks for the quick response. It can wait until tomorrow.

A new version (0.5.1) was released


AB#252 Improvement/separate assets

Describe pull-request

Separate icons into another package
Use CDN for image and fonts (Remove base64 solution)
Use dynamic import for adding the icons via sdds-icons
Icons now available as webfont (can be used from local package or from CDN)
Enable source-mapping in theme-light only on dev environment (on npm start). BUt disable it on build (on npm run build).

Solving issue
Add which issue this pull-request solves by adding # plus the number of the issue
Fixes: #129  AB#252
How to test

Check if icons package work

Go to sdds/theme/core/icons
Run npm run build
Build @scania/components and @scania/theme-light as well.
Test solution in HTML demo (I made a page called icons.html, linking to all 3 packages - icons, theme, component), this page shows the webfont
Check in storybook that all components are okay (including header, footer, navigation)
You can compare to the latest build of storybook in netlify https://deploy-preview-139--sdds-storybook.netlify.app/?path=/story/patterns-footer--with-social-links
Test solution in angular (use npm link in sdds-website, linking to all 3 packages) - Use this branch https://github.com/scania-digital-design-system/sdds-website/tree/improvement/theme_light_optimized and check in Foundation > Pictograms page

Check if theme light are not having issue with new solution (Pay attention to logotypes, symbol, and fonts)

Additional context
** DO NOT MERGE **

Need to update CDN link when the URL is ready (waiting for DNS registration)

Will need to update c-icon to sdds-icon in site, branches created, but PR can be made when package is released
https://github.com/scania-digital-design-system/sdds-website/tree/improvement/theme_light_optimized

Documentation has been added in the CMS

c-cookie, c-footer, and c-modal has dependency to c-icon, it is replaced with sdds-icon now, but application need to import icons package to make it work. (Should we care about old c- components and add the svg code instead?)

There are some issue with the link color for old components like footer and header, slotted once.
Maybe add a override for --sdds-link inside for those components with old values
https://deploy-preview-139--sdds-storybook.netlify.app/?path=/story/patterns-footer--with-social-links

Local branch

[ ] Vue

Components
Web font

[ ] Angular

Components
Web font

[ ] HTML

Components
Web font

[ ] React

Components
Web font

[ ] Checked old components
[ ] Tried linking package into site

@mmexvr thanks for a thorough review, it's a big one 😉
For the footer colour, I got it blue in netlify actually! That's why I thought it was another bug to fix


СontextMenu does not update color on theme switch.

Just checked the last commit  and noticed Just checked out the latest commit and noticed that the context menu doesn't update the color when switching the theme.  🤔

@ParzivalExe
any ideas?

Well, I figured it out in the end and made a fix. The most important one was fixing the BorderBrush. This now properly changes whenever the Theme changes at runtime.
The Demo was a bit more problematic. The general problem was that I used the existing ContextMenu and simply opened it by changing ContextMenu.IsOpen = true every time somebody pushed the Button. The problem with this approach is, that the ContextMenu will not be rebuild after a Theme-Change during Runtime. This is no problem when using ContextMenu with a Right Click because apparently WPF does something different than just what I did which results in ContextMenu actually being rebuild and the colors therefore changed accordingly.
My dirty Fix (since I couldn't get the colors to change anyway else) is to simply create a new ContextMenu in CodeBehind every time the Button is clicked.
This is oke in my opinion because this is simply a demo and the real usage of ContextMenu works without any workaround like this.


question

is it possible to go into the scp driver and recode it to use mouse movement instead of right stick

what?

@rob19841 The action you are requesting is to "Mouse -> XINPUT Controller -> Mouse"? If so why are you converting mouse movements to mouse movements ? (im lost)


cordova-plugin-camera clash with this plugin

If you use
    cordova plugin add cordova-plugin-camera

to install camera plugin . It will clash with this plugin .
The reason is this camera plugin use too old api of android .
You must fix cordova-plugin-camera|
FileProvider.java
Line 21 : "android.support.v4.content.FileProvider" Change To "androidx.core.content.FileProvider"
CameraLauncher.java
Line 42 : "android.support.v4.content.FileProvider" Change To "androidx.core.content.FileProvider"

Hi,
I can't speak for the camera plugin but I do have a note in the readme from years ago that states "If you are installing this plugin along with cordova-plugin-camera you MUST install cordova-plugin-camera first."
That note would have been written before this plugin supported Android X.  Please note that:

For Android X Support please use version >= 2.x.x
For Android Support Library please use version 1.3.x

So, perhaps you also need to use the older version of this plugin prior to Android X support.
Hope that helps

Hi,
Yes, I c .
if you use
cordova plugin add cordova-plugin-camera

to install camera , it will clash with this plugin.
if
cordova plugin add https://github.com/apache/cordova-plugin-camera.git

it's ok . no clash .
it seems cordova initial plugin not update to latest .


sqlalchemy.types has no attribute 'Uuid'

After release of databricks-sql-connector 3.0.0 receiving error sqlalchemy.types has no attribute 'Uuid' when using:
from langchain.sqldatabase import SQLDatabase
db = SQLDatabase.from_databricks()
Downgrading to version 2.9.3 resolves the error. Multiple reports of this issue. Additional report that downgrading sqlalchemy to version 2.0.12 resolved error.

What version of SQLAlchemy do you have installled?

Thank you for going the extra mile Jesse.


Compilation error

Hello,
First thanks for this very interesting software. I'm trying to package it for the great pure package manager nix, but I have a compilation error: after running cabal2nix . > wstunnel.nix, that generated the following file:
{ mkDerivation, async, base, base64-bytestring, binary, bytestring
, classy-prelude, cmdargs, connection, hslogger, mtl, network
, network-conduit-tls, stdenv, streaming-commons, text
, unordered-containers, websockets
}:
mkDerivation {
  pname = "wstunnel";
  version = "<IP_ADDRESS>";
  src = ./.;
  isLibrary = true;
  isExecutable = true;
  libraryHaskellDepends = [
    async base base64-bytestring binary bytestring classy-prelude
    connection hslogger mtl network network-conduit-tls
    streaming-commons text unordered-containers websockets
  ];
  executableHaskellDepends = [
    base bytestring classy-prelude cmdargs hslogger text
  ];
  testHaskellDepends = [ base text ];
  homepage = "https://github.com/githubuser/wstunnel#readme";
  description = "Initial project template from stack";
  license = stdenv.lib.licenses.bsd3;
}

I put in default.nix the code:
{ nixpkgs ? import <nixpkgs> {}, compiler ? "ghc863" }:
nixpkgs.pkgs.haskell.packages.${compiler}.callPackage ./wstunnel.nix { }

and to compile I ran nix-build default.nix. This downloads all the dependencies, however when it comes to compilation, the 3rd file "src/Types.hs" fails to compile:
Preprocessing library for wstunnel-<IP_ADDRESS>..
Building library for wstunnel-<IP_ADDRESS>..
[1 of 7] Compiling Credentials      ( src/Credentials.hs, dist/build/Credentials.o )
[2 of 7] Compiling Logger           ( src/Logger.hs, dist/build/Logger.o )
[3 of 7] Compiling Socks5           ( src/Socks5.hs, dist/build/Socks5.o )
[4 of 7] Compiling Types            ( src/Types.hs, dist/build/Types.o )

src/Types.hs:22:1: error:
    • Can't make a derived instance of ‘Generic PortNumber’:
        The data constructors of ‘PortNumber’ are not all in scope
          so you cannot derive an instance for it
    • In the stand-alone deriving instance for ‘Generic PortNumber’
   |
22 | deriving instance Generic PortNumber
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
builder for '/nix/store/dp76w5kgmypi9n4fj9smlgd7zf5px3z3-wstunnel-<IP_ADDRESS>.drv' failed with exit code 1
error: build of '/nix/store/dp76w5kgmypi9n4fj9smlgd7zf5px3z3-wstunnel-<IP_ADDRESS>.drv' failed

Any idea why? (note: with stack init && stack build it works)

Can you try with GHC 8.0.2 instead of 8.6.3 ?
PortNumber constructor is correctly imported in Types.hs, so I don't know why GHC complains in this case :x

Unfortunately NixOs does not have such old GHC version available... so not sure how to do that. :-\

Ok, it appears that the Network package changed a bit, so it's not possible to derive Generic automatically, but it's not a problem as it's still possible to implement Hashable from the numerous value of PortNumber. See my pull request (tested on both nix and stack) here.

Ha sorry, I can't upgrade to the latest LTS due to this issue #8
I need to make some changes in order for wstunnel to work with the latest tls package.
Sadly I don't have much free time right now, so it may take time before I upgrade it.
Can you try and let me know if you reproduce the issue #8 with your version, or if everything is working fine ?

it works now with the latest version of ghc (8.6.5)


Update requestbinCurl.js

Probably requestBin changed it recently. Only works with https

Fixed with another pull request. Thanks.


Cannot clone repo

When using go get github.com/kubicorn/kubicorn it stops at
Cloning into 'bootstrap'...
fatal: reference is not a tree: a83ebae7bb515e12620b66d7601f697f5406c603
Unable to checkout 'a83ebae7bb515e12620b66d7601f697f5406c603' in submodule path 'bootstrap'
package github.com/kubicorn/kubicorn: exit status 1

Then if I go check whats wrong in home/go/src/git..../kubicorn
git show a83ebae7bb515e12620b66d7601f697f5406c603
fatal: bad object a83ebae7bb515e12620b66d7601f697f5406c603

Thank you so much for reporting the issue!
I think the problem is that this commit doesn't exist anymore, as we had to rebase the pre-release branch to include the latest changes from master.
I've merged #646 which should fix the issue.
If problems persists, let me know and I'll do my best to fix the issue. Thank you once again!

Thanks, everything works fine again :+1:


Task terraformApply does not show standard error when apply fails with errors

Task terraformApply does not show standard error when apply fails with errors. One has to do the following as workaround:

pushd build/terraform/main/runtimeExecution
env TF_DATA_DIR=../data ../../terraform_ apply "/home/farrukh/altran/novartis/mosiac-fv/mosaic-backend/terraform/build/terraform/main/plan.bin"

What would be better is for terraformApply task to show error output by default (why would we want to hide it and make it hard to debug the underlying problem?).

@lamba92 It seems that this you may not have much time to address issues in this plugin.
I have switched to the https://ysb33rorg.gitlab.io/terraform-gradle-plugin/ and all my problems seems to be solved. This new plugin is much more sophisticated and flexible. At first blush it looks much more complicated than this https://github.com/JetBrains/jetbrains-gradle-plugins plugin. However, it only took me about 2 hours to switch and I am finding it to be a much better fit.


V6 rebuild project

Goal
Previous work to combine three separate Xcode projects didn't go far enough and left a folder structure that didn't follow Apple best-practice and, further, wasn't amenable to consistent install instructions.  This work rebuilds the entire project structure to address these issues.
Design

A new Xcode iOS framework project was created.
Framework targets for each platform were added, along with tests and a static target.
Build settings from the previous project merge were copied across
Source files were moved from existing locations to preserve history, with additional directory structure added.  Tests were also re-added.  A single redundant Swift test was elided.

Commits were granular and show the progression towards a stable state.  Pushes were frequent to ensure that CI issues were addressed.  Builds were attempted frequently locally. Popular open source frameworks were consulted to ensure best (common) practice was followed as much as possible.
Changeset

Project file added at top level, source files moved; this generates a large changeset that may be better reviewed as a checkout.  v6 changes were merged in to ensure that the two branches did not diverge too greatly.
Makefile adjusted for new source path

Tests

Manual install, Cocoapods, Carthage both local and remote.

Updated PR to address the review points raised above - I've tested out on an example app using Cocoapods only. @tomlongridge would be good to get another pair of eyes reviewing this.

Configurations/Config.xcconfig does not appear to be necessary - I've deleted this and rebased the PR to resolve conflicts from the deadlock fix merged into v6.


[msal-node-extensions]: Improve general documentation

Improved the extensions general documentation
As part of reviewing this pull request, do review this wiki too.

@sameerag I have moved the wiki to the readme as it seemed like the best way to integrate the FAQs too. We can do the rest of the improvements progressively as we improve the package's documentation.


Delete: tibbo.ioninja version 5.0.4

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 2531
Microsoft Reviewers: Open in CodeFlow

@ItzLevvie Should we remove all the versions?

@ItzLevvie Should we remove all the versions?

@ItzLevvie

Should we remove all the versions?

No, looks like the URL was missing -core at the end which is why it wasn't installing.


WIP: Si 2149 rewards by vehicle

What did you add to the API?
- New rewards query added to vehicles Query
- Rewards pagination

What work does this depend upon?

What was difficult about this?

Handling creation of a SubQuery within the vehicles Query

What major components did we add?

Rewards
Breakdown of Earnings for Device, SyntheticDevice and Streak

_______________________________________________________________________________________

I think this is moving in a good direction. There are two major hurdles:

We have fields in the database that are not required but that we are requiring in the API. I think this is going to lead to problems. Anywhere where you see .Int without a guard I think that's suspicious. The ways out are
a. Don't require this stuff in the API
b. Restrict the query to rows where, e.g., the streak is populated

We still need to implement all the normal connection stuff.

I think it would also help, by the way, to have a PR description that quickly guides the readier through what he’s looking at.

What did you add to the API?
What work does this depend upon?
What was difficult about this?
What major components did we add?

I wonder if some of these are failing on main as well. I'll take a look.

@Allyson-English Take a look?


Update the Azure SDK to 1.15.1

Azure SDK has a bug that it can't deal with app names with capital letter,  we need to update the sdk version to 1.15.1, or plugin will crashed when  appName have upper cases.

duplicate with #352 close it.


Remove hardcoded recent environment

Either this existed for a demo, dev purposes, or something Pinterest Internal related. When clicked, it causes essentially a 404 error to occur.

Thanks for bring this up, that item should be guarded by an if statement. and it needs a change in the view as well, I will create PR for that

https://github.com/pinterest/teletraan/pull/141 should fix this. Thanks again for bringing this up, closing this for now...

Excellent - thanks for the update and I appreciate the fix!


Saving doesn't work even though objects are properly merged

I'm trying to save an entry with the help of 'deep-assign' and Datastore (I'm running Express.js on GCP's functions/api endpoint service) to merge the data I want to save but each time it seems to give me an error (I tried datastore.save() some time ago and it should work, unless there was an update I'm not aware of?). Does anybody see why this goes wrong? This is my code:
// createdAtTimestamp.                                                                                ||
const months = ["01", "02", "03", "04", "05", "06", "07", "08", "09", "10", "11", "12"];
const d = new Date();
const Timestamp = d.getDate() + '-' + months[d.getMonth()] + '-' + d.getFullYear() + ' - ' + d.getHours() + ':' + d.getMinutes() + ':' + d.getSeconds();
const TimestampString = Timestamp.toString();

module.exports = {
    post: async function (KIND, body) {
        const name = body.id; // The name/ID for the new entity.
        const taskKey = datastore.key([KIND, name]); // The Cloud Datastore key for the new entity.
        const setData = {
            created_at: TimestampString,
            updated_at: "-"
        }
        const loadData = deepAssign(setData, body); // Merge setData and body so it can be called as one object
        const task = // Set (save) task with a task key and the merged data
            {
                key: taskKey,
                data: loadData
            };
        await datastore.save(task, (err, entity) => { // Save merged data with appropriate task key
            if (err) {
                result = err;
            } else {
                result = task;
            }
        })
        return result;
    },
}

Hey @RFlintstone, I think the problem is with the result variable.
        await datastore.save(task, (err, entity) => { // Save merged data with appropriate task key
            if (err) {
                result = err;
            } else {
                result = task;
            }
        })
        return result; // **Here**

It is out of scope. That could be the problem. try following way and let me know if it works.
        await datastore.save(task, (err, entity) => { // Save merged data with appropriate task key
            if (err) {
                result = err;
                return result;
            } else {
                result = task;
                return result;
            }
        })

Hey @RFlintstone, I think the problem is with the result variable.
        await datastore.save(task, (err, entity) => { // Save merged data with appropriate task key
            if (err) {
                result = err;
            } else {
                result = task;
            }
        })
        return result; // **Here**

It is out of scope. That could be the problem. try following way and let me know if it works.
        await datastore.save(task, (err, entity) => { // Save merged data with appropriate task key
            if (err) {
                result = err;
                return result;
            } else {
                result = task;
                return result;
            }
        })

Problem solved! It turned out that I tried to parse an object which was already parsed and that made it drop the connection. I removed JSON.parse() and now it works fine. Both the solutions are working like a charm now! Thank you!!


Fix for issue #21 - Python: ApiHost.set_rack_id() fails with Type Error

Fix for #21
This fix renames ApiHost._put() so it doesn't override the base class _put().
This approach changes the external API, but seems to match the original intent based on similar functions.

This has been fixed.


Number of bytes in 1 KB/MB/GB

Does 1 MB contain 10^6 or 2^20 bytes?
Due to a calculation found in the preview docs 100,000 devices * 200 bytes/event * (3/60 event/sec) = 1,000,000 = 1 MBps it seems like 1 MB = 10^6 bytes.

From a throttling perspective, an ingressed data packet that has a packet size of 32 KB is treated as 32 events, each 1 KB in size. The maximum allowed event size is 32 KB. Data packets larger than 32 KB are truncated.

Same as above, is 1 KB = 10^3 or 2^10 bytes?
Do we have to limit the packet size to 32000 or 32768 bytes? (<- this might be a source of bugs since packets larger than "32KB" are truncated.)

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

ID: 750c5eeb-be8d-a927-7c63-1fbce1e92cc0
Version Independent ID: 6e55272d-b6fa-64df-a7b8-243c28c111bb
Content: Plan your GA environment - Azure Time Series Insights
Content Source: articles/time-series-insights/time-series-insights-environment-planning.md
Service: time-series-insights
GitHub Login: @deepakpalled
Microsoft Alias: dpalled

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Hi @Karishma-Tiwari-MSFT how do I link a PR to the issue?

@lyrana Mention https://github.com/MicrosoftDocs/azure-docs/issues/51335 directly on the PR it will link automatically.

#please-close


Hi, Can you please explain what do files starting with link contain?

In Section 5.4 of the paper (Disambiguation difficulty part), entities in subsets correct and errorneous are evaluated separately. These link* files actually record which entities are linked successfully and therefore should be in the correct subset (based on the parameters given in paper_parameter/ folder)
Ignoring these files will not affect type classification itself, but will probably gives you wrong results when you wish to explore these two subsets.

Hi, Thank you for your explanation. Can you please share the disamb file for training dataset part? It would be really helpful for our further analysis and research.

It's in the data folder, disamb_file.gz.

Hi,
Thank you for your reply. As far as I understand ,data/disamb_file.gz only contains the disambiguated entities for testing data, not for training data. And for your paper, you have taken different training and testing entities.

Hi,
I have one more query. Can you please provide the mapping file from freebase types to your types, so that we can use it for our further research?

types.mapping.txt


Questions not being answered

Could we make something that would delay bots by like a few milliseconds allowing them to answer the question?

Duplicate of #141
After merging PR #140 this should be fixed.


Deadlock when using both writeBlocking and Flows

I ran into this behavior when I was trying to write up a sample that showed how to interop between Java and Kotlin. It looks like somehow we end up deadlocking our internals when using writeBlocking and Flow. Not 100% sure how it is happening though.
This JVM unit test demonstrates the behavior:
    @Test
    fun writeBlockingAndFlows() {
        val tmpDir = PlatformUtils.createTempDir()
        val configuration = RealmConfiguration.Builder(setOf(Parent::class, Child::class))
            .directory(tmpDir)
            .build()
        val realmScope = CoroutineScope(CoroutineName("RealmScope") + Dispatchers.Default)
        val realm = Realm.open(configuration)

        // Do initial write
        realm.writeBlocking {
            copyToRealm(Parent())
        }
        assertEquals(1, realm.query<Parent>().count().find())
        println("Wrote 1st data")

        // Start notifications
        val updateLatch = CountDownLatch(1)
        val job = realmScope.launch {
            realm.query<Parent>().asFlow().collect {
                println(it)
                if (it is UpdatedResults) {
                    updateLatch.countDown()
                }
            }
        }
        println("Started observer")

        // Do a 2nd write
        realm.writeBlocking {
            copyToRealm(Parent())
        }
        println("Wrote 2nd data")

        // Wait for the Update notification. This will currently fail, and all Coroutines are in
        // the WAITING state.
        // Note: If we move the notification block to run as the first thing, the writes do
        // not block it.
        if (!updateLatch.await(10, TimeUnit.SECONDS)) {
            fail("Did not receive update notification in time")
        }
        job.cancel()
        realm.close()
    }

The thread dump look like this:

I made a mistake in reasoning about the flow of events. The 2nd write could happen before the listener fired the first event, which meant that no update was ever coming.
Closing


Inital Setp Failed to renew token

Hi everyone,
I'm new to Home Assistant, transitioning from IOBroker, and I'm excited to be here! Most of my devices have integrated smoothly with the panasonic_cc integration. Most of them are of the same type, and except for one device, they are working like a charm.
However, I'm having trouble with one specific device out of the group of same devices.
The integration found the device and created everything like the other devices. However, it doesn't display any values except for the firmware version. Within the system log, I keep encountering the following error:
vbnet
Copy code
This error originated from a custom integration.
Logger: custom_components.panasonic_cc.panasonic
Source: custom_components/panasonic_cc/panasonic.py:79
integration: Panasonic Comfort Cloud (documentation, issues)
First occurred: 12:17:12 PM (18 occurrences)
Last logged: 12:34:13 PM
Failed to renew token for device af0ca7acf02190262001f2ec8a8523d779c6d0084788ea207f3ac8526650a6eb, giving up for now
Has anyone experienced a similar issue or can offer some guidance on how to resolve this? Any help would be greatly appreciated!
Thank you in advance!
Best,
Dennis

Debug logging is enabled, to be honest I'm not sure if it's saved somewhere else. Nothing addition within the log section of HA.

retry the setup after you enabled the debug log, and when you encounter the issue, click on "disable debugging log" - you will get then the detailed log as a download and can upload it here (remove any unrelated entries & redact any sensitive data)

Panasonic_cc_debug_log_extract.txt
Attached you'll find the logfile.
The Device MTZ16WKE is the one which does not show any vaules within HA.
From the Logfile itself, it seems that the integration is capable of receiving data like the temperature.
Any ideas?
Thank you very much for your support.

that is odd - you get successfully data for /deviceStatus/ but at the same time get a Failed to renew token for device error. it looks like the token refresh succeded, but then self._api.get_device(self.id) fails with a masked exception when the response is parsed.

@sockless-coding I think 2 things need to be done:

explicitly check for an invalid / expired token on all api calls. panasonic has dedicated response codes for these cases. this way we avoid refreshing the token when any other unrelated exception is happening
in the case of a non-token based exception in self._api.get_device(), this should be reported in the logger

@DonBlock it's the Left/Right swing mode that is causing the issue. What do you have it set to in the App?

Hi,
I‘m not where to find this Option.
Swing is set Auto in the App.
Can‘t any left right options in the app.
Fan Speed is Auto or Quiet. There are in more additional settings.

@DonBlock v1.0.49 should work and only log a warning about invalid horizontal swing mode

Hi,
Thank so much, it‘s working like a charm.


add corner slide if player jumps, but touches the corner of an overhead object.

When players jump and get blocked by one pixel of an overhead obstacle, they get frustrated. It's best practice to allow them to slide a few pixels over in order to get past the obstruction.

not likely to happen. There are more important bugs to fix with 3 days remaining.


clients/ipfs: Use CIDv1 for all IPFS uploads

It is the recommended approach for CIDs as it's
future-proof (hash alg is embedded in the CID)
and also because it's case insensitive and will
work better on all scenarios (like URLs).
It is weirdly not the default on Piñata yet, but
while it is not we can easily toggle it on our
export requests.
It does not break compatibility on anything that I can
think of.
Reference: https://docs.ipfs.io/how-to/best-practices-for-ipfs-builders/

FTR just minted this to make sure it works fine E2E with the CIDv1
https://testnets.opensea.io/assets/mumbai/0xA4E1d8FE768d471B048F9d73ff90ED8fcCC03643/84


Include images, surveyor seal, extra pages in monument record sheet

Just a reminder issue since I don't think we have one for it, even though we're aware that it needs to happen.

@nathankota The commit above makes the front page complete i believe. Will you play with that?

@steveoh I've made it to the review page a few times, but the MR sheet preview is not loading.

Yep, my streams weren't ending. Check it again.

The preview is loading and the images are included. Other things found during testing will be included in new issues.


Support older ActiveSupport versions

For some old versions of ActiveSupport, ActiveSupport::Logger does not exist.

👍