Perf event ring buffer full, dropped 16 samples

root@bpf2:~# ./pwru --filter-proto icmp  --output-tuple
2022/01/21 04:44:15 Attaching kprobes...
577 / 577 [---------------------------------------------------------------------------------------------------------------------------------] 100.00% 34 p/s
2022/01/21 04:44:32 Attached (ignored 13)
2022/01/21 04:44:32 Listening for events..
               SKB         PROCESS                     FUNC        TIMESTAMP
0xffff96f353734300          [ping]           __ip_local_out<PHONE_NUMBER>6335 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]                ip_output<PHONE_NUMBER>5120 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]             nf_hook_slow<PHONE_NUMBER>8827 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping] __cgroup_bpf_run_filter_skb<PHONE_NUMBER>2013 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]     neigh_resolve_output<PHONE_NUMBER>6902 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]       __neigh_event_send<PHONE_NUMBER>9217 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]               eth_header<PHONE_NUMBER>3094 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]                 skb_push<PHONE_NUMBER>6681 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]           dev_queue_xmit<PHONE_NUMBER>8785 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]      netdev_core_pick_tx<PHONE_NUMBER>1881 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]       netif_skb_features<PHONE_NUMBER>4686 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]  passthru_features_check<PHONE_NUMBER>6700 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]     skb_network_protocol<PHONE_NUMBER>9855 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]       validate_xmit_xfrm<PHONE_NUMBER>3733 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]      dev_hard_start_xmit<PHONE_NUMBER>5646 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]   skb_clone_tx_timestamp<PHONE_NUMBER>9343 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]        __dev_forward_skb<PHONE_NUMBER>1788 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]         skb_scrub_packet<PHONE_NUMBER>4002 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]           eth_type_trans<PHONE_NUMBER>6256 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]                 netif_rx<PHONE_NUMBER>8560 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]     tcf_classify_ingress<PHONE_NUMBER>7115 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]      skb_ensure_writable<PHONE_NUMBER>8730 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]      skb_ensure_writable<PHONE_NUMBER>0414 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]      skb_ensure_writable<PHONE_NUMBER>1676 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]      skb_ensure_writable<PHONE_NUMBER>2948 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]          skb_do_redirect<PHONE_NUMBER>0830 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]                   ip_rcv<PHONE_NUMBER>5740 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]               sock_wfree<PHONE_NUMBER>9246 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]     ip_route_input_noref<PHONE_NUMBER>2592 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]       ip_route_input_rcu<PHONE_NUMBER>4256 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]      fib_validate_source<PHONE_NUMBER>9766 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]         ip_local_deliver<PHONE_NUMBER>3844 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]  ip_protocol_deliver_rcu<PHONE_NUMBER>6338 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734300          [ping]        raw_local_deliver<PHONE_NUMBER>8362 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
2022/01/21 04:44:35 Perf event ring buffer full, dropped 3 samples
0xffff96f353734300          [ping]     fib_compute_spec_dst<PHONE_NUMBER>8761 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
2022/01/21 04:44:35 Perf event ring buffer full, dropped 5 samples
0xffff96f353734b00          [ping]     neigh_resolve_output<PHONE_NUMBER>1024 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
### 2022/01/21 04:44:35 Perf event ring buffer full, dropped 16 samples
0xffff96f353734300       [<empty>]   skb_release_head_state<PHONE_NUMBER>5111 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
2022/01/21 04:44:35 Perf event ring buffer full, dropped 1 samples
0xffff96f353734b00       [<empty>]      skb_ensure_writable<PHONE_NUMBER>9565 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
0xffff96f353734b00       [<empty>]      skb_ensure_writable<PHONE_NUMBER>1188 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
2022/01/21 04:44:35 Perf event ring buffer full, dropped 2 samples
0xffff96f353734b00       [<empty>]          skb_do_redirect<PHONE_NUMBER>8721 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
2022/01/21 04:44:35 Perf event ring buffer full, dropped 13 samples
0xffff96f353734300       [<empty>]       sk_filter_trim_cap<PHONE_NUMBER>3108 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
2022/01/21 04:44:35 Perf event ring buffer full, dropped 9 samples
0xffff96f353734300       [<empty>]        skb_free_datagram<PHONE_NUMBER>1122 <IP_ADDRESS>:0-><IP_ADDRESS>:0(icmp)
^C2022/01/21 04:44:43 Received signal, exiting program..
root@bpf2:~#

root@bpf1:~# kubectl get pods -o wide
NAME        READY   STATUS    RESTARTS   AGE   IP           NODE   NOMINATED NODE   READINESS GATES
cni-bpz2q   1/1     Running   0          14h   <IP_ADDRESS>    bpf2   <none>           <none>
cni-hx8gz   1/1     Running   0          14h   <IP_ADDRESS>   bpf1   <none>           <none>
same        1/1     Running   0          79s   <IP_ADDRESS>    bpf2   <none>           <none>
root@bpf1:~# kubectl exec -it same -- ping -c 1 <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=63 time=0.269 ms

--- <IP_ADDRESS> ping statistics ---
1 packets transmitted, 1 packets received, 0% packet loss
round-trip min/avg/max = 0.269/0.269/0.269 ms

How about the issue when monitor the two pods communicate to each other on the same node.
2022/01/21 04:44:35 Perf event ring buffer full, dropped 3 samples
which is it means that the ring buffer is full？
ENV：
root@bpf2:~# uname -a
Linux bpf2 5.10.0-051000-generic #202012132330 SMP Sun Dec 13 23:33:36 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
root@bpf2:~#

You can try increasing the per cpu buffer size  - http://github.com/brb/pwru/blob/70487475cbc00e66375b724febd223290bc686f9/main.go#L118-L118, depending on the available memory. Please note that this will drive up the memory usage on the host though. Alternatively, you can try tweaking the filters to minimize the number of events.

depending on the available memory. Please note that this will drive up the memory usage on the host though. Alternatively, you can try tweaking the filters to minimize the number of events.

yes. with 8G memory in this node. is it enough?
and when i run the pwru, watch the free -mh , it is not increase so mush.
but i run this test in the vmware workstation env.

yes. with 8G memory in this node. is it enough?

Yes, it should be more than enough.
Have you tried to increase the buffer size as suggested above?

Have you tried to increase the buffer size as suggested above?

Yes.Unfortunately,  the address can't be accessed now.

Could you elaborate more? What error did you get?

Could you elaborate more? What error did you get?

404 now.

404 now.

Em? I'm not following.

Yes, but i really get this 404 error here.

Ah, got you. The actual link supposed to be https://github.com/cilium/pwru/blob/master/main.go#L145.

https://github.com/cilium/pwru/blob/master/main.go#L145

@brb, that line is rd, err := perf.NewReader(events, os.Getpagesize()), you mean increase the page size and how?

never mind, I got what you mean, should have read the code :-)
// NewReader creates a new reader with default options.
//
// array must be a PerfEventArray. perCPUBuffer gives the size of the
// per CPU buffer in bytes. It is rounded up to the nearest multiple
// of the current page size.
func NewReader(array *ebpf.Map, perCPUBuffer int) (*Reader, error) {
        return NewReaderWithOptions(array, perCPUBuffer, ReaderOptions{})
}

Yes.
main.go
        bar.Finish()
        log.Printf("Attached (ignored %d)\n", ignored)
       // modify syscall to customize
        rd, err := perf.NewReader(events, 16384)
        if err != nil {
                log.Fatalf("Creating perf event reader: %s", err)
        }
        defer rd.Close()

@vincentmli @BurlyLuo maybe you could make it configurable via a flag and send a PR?


Smoke Tests Should Run All OSes Against All Clouds

Currently the smoke tests are only configured to run Windows against all clouds with all Java version permutations, Linux and MacOS permutations only run on the Azure Cloud and not China Cloud or US Gov cloud.
Add the following permutations if possible:

[ ] Linux Java 8 on China Cloud
[ ] Linux Java 11 on China Cloud
[ ] Linux Java 8 on Gov Cloud
[ ] Linux Java 11 on Gov Cloud
[ ] MacOS Java 8 on China Cloud
[ ] MacOS Java 11 on China Cloud
[ ] MacOS Java 8 on Gov Cloud
[ ] MacOS Java 11 on Gov Cloud

@weshaggard and I think that this is a good request, but we should use a sparse matrix like we've done in other test pipelines.

Haven't gotten to this in 3+ years.  Not a priority.
(also java 8 is no longer supported anyway :))


Open Folder in File Explorer not working

Issue Type: Bug

Open an empty workspace
Go to File Explorer and then click Open Folder
Nothing happens

cmd+r and then choose a folder works fine.
VS Code version: Code - Insiders 1.51.0-insider (bb16786cff3d231facbdcee90741ac25fcffd3ac, 2020-10-28T05:53:18.953Z)
OS version: Darwin x64 19.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (16 x 2400)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
32.00GB (0.36GB free)

Process Argv
-psn_0_5260548 --enable-proposed-api ms-vscode.vscode-notebook-history --enable-proposed-api vscode.notebook-editing --crash-reporter-id 1a907bbd-fa02-4595-81e0-63e1648afb31

Screen Reader
no

VM
14%

Extensions (41)

Extension
Author (truncated)
Version

comment-tagged-templates
bie
0.3.0

github-markdown-preview
bie
0.0.2

markdown-emoji
bie
0.0.9

markdown-preview-github-styles
bie
0.1.6

gitlens
eam
10.2.2

tsl-problem-matcher
eam
0.3.1

vscode-pull-request-github-insiders
Git
2020.10.34021

vscode-fluent-icons
mig
0.0.1

theme-monokai-pro-vscode
mon
1.1.17

vscode-docker
ms-
1.7.0

vscode-language-pack-zh-hans
MS-
1.51.0

dotnet-interactive-vscode
ms-
1.0.152203

vscode-dotnet-runtime
ms-
0.1.2

python
ms-
2020.9.114305

remote-containers
ms-
0.147.0

remote-ssh-edit-nightly
ms-
2020.10.29700

remote-ssh-nightly
ms-
2020.10.29700

hexeditor
ms-
1.3.0

js-debug-nightly
ms-
2020.10.2617

powershell-preview
ms-
2020.9.0

vscode-github-issue-notebooks
ms-
0.0.56

vscode-markdown-notebook
ms-
0.0.13

vscode-notebook-history
ms-
0.0.1

vsliveshare
ms-
1.1.0-dev

vsonline
ms-
1.0.3073

debugger-for-chrome
msj
4.12.11

toggle
reb
0.0.1

java
red
0.69.0

kuskus-extensions-pack
ros
1.0.1

kuskus-kusto-language-server
ros
1.0.1

kuskus-kusto-syntax-highlighting
ros
1.1.8

rust
rus
0.7.8

sourcekit-lsp
unp
0.0.1

vscode-lldb
vad
1.6.0

vscodeintellicode
Vis
1.2.10

vscode-java-debug
vsc
0.29.0

vscode-java-dependency
vsc
0.14.0

vscode-java-pack
vsc
0.10.0

vscode-java-test
vsc
0.25.0

vscode-maven
vsc
0.25.0

notebook-editing
vsc
1.0.0

(5 theme extensions excluded)

@rebornix you dont see a dialog opening doing this?

right, I'm not seeing the native dialog showing up and I'm only seeing this error
  ERR No application in the Launch Services database matches the input criteria.: Error: No application in the Launch Services database matches the input criteria.

I have no ideas why this happens. Did you try from a fresh user data dir? Do you see any output with --verbose?

@rebornix did you open code from command line ? or via finder , dock ?

I can't reproduce this issue any more, which is really weird. Now I only see long lag sometimes when clicking the open folder button, which made me think that probably it's just being slow. Not sure why though. Closing it for now.

@rebornix next time you see this issue, would be good to check how you launched the app. The error from https://github.com/microsoft/vscode/issues/109612#issuecomment-718135486 definitely sounds like a side effect of https://github.com/microsoft/vscode/issues/102975

I have same problem.
macOS 10.15.7 (19H2)

Workaround so far is to "add folder to workspace".


Is There an event when bot die ?

Is There an event when bot die ?

The Bots reconnect automatically :
const bot = new Bot({         name: process.env.SLITHER_SERVER_NAME || 'www.slither.gg',         reconnect: true, // Here the bots reconnect automatically         skin: skin,         server: server       })

@yario-o thanks.

So. There is an event when bot die?

@yario-o i dont think so if yes then what will it be for?

https://github.com/ClitherProject/Slither.io-Protocol/blob/master/Protocol.md#type_v_detail
Check here

Because the number of bot alive is not perfect. Look :
bot.on('disconnect', function() {
alive--
print(bot.name + ' died')
})
It do  alive-- when bot disconnect so when proxy disconnect is do alive--

@devesh457 Thx

EDIT : I just find the dead fonction now by number of bots alive can't be negative Just replace :
bot.on('disconnect', function() {
alive--
print(bot.name + ' died')
})
To :
bot.on('dead', function() {
alive--
print(bot.name + ' died')
})


Set system proxy by default, if it exists. Solves #60.

So this is enough to make it work? Nice :) Does it work on your machine now?
Shouldn't this also happen for downloading/parsing single tasks (or, ideally, globally when the application starts)?
Please try to keep the spacing style as in the rest of the code. Thanks!

I shifted it to MainJFrame.java, and it works with tasks too now on my PC.
Oh, sorry. I changed the style.
I hope it's working on yours!

It works on my machine, so I will pull the code in. Thanks a lot!


It is possible to add the same event several times to the same input action

Tested versions
v4.3.dev.custom_build [11d376813]
System information
linux
Issue description
in Project > Project Settings > Input Map:

you can add the same key/button several times to the same action.
Steps to reproduce

add the same event twice to the same action

Minimal reproduction project (MRP)
any

Hi @KoBeWi @mieldepoche
I am new to this project and would like to help in resolving this bug, I think we could implement some sort of check to see if an input corresponding to the said action already exists and if it does then we can ignore adding it additionally. Would you guys like suggest anything before I try this?

I'd like to work on this if that's ok - I'll have a look but I presume it should be just checking that the newly input key is not the same as the ones currently bound.


{ properties: { showGridLines: false } } not working

I am trying to not show the grid lines with this code:
var sheet1 = workbook.addWorksheet( 'sheet1', { properties: { showGridLines: false } } )

But it is still showing the grid lines. Am I doing something wrong?
Here is the rest of my code:
const Excel = require('exceljs')
var workbook = new Excel.Workbook()
var sheet1 = workbook.addWorksheet( 'sheet1', { properties: { showGridLines: false } } )
sheet1.columns = [
  { header: 'Mark', key: 'Mark', width: 20 }
  ,{ header: 'QTY', key: 'QTY', width: 10 }
  ,{ header: 'Description', key: 'Description', width: 50 }
  ,{ header: 'Thickness', key: 'Thickness', width: 10 }
  ,{ header: 'Width', key: 'Width', width: 10 }
  ,{ header: 'Depth', key: 'Depth', width: 10 }
  ,{ header: 'Length', key: 'Length', width: 10 }
  ,{ header: 'Remarks', key: 'Remarks', width: 25 }
]
sheet1.getRow(1).alignment = { horizontal: 'center' }

sheet1.addRow({
  Mark: 'D6x12-4x7'
  ,QTY: 2
  ,Description: 'Column Assembly'
  ,Thickness: '1/2'
  ,Width: 10
  ,Depth: 6
  ,Length: 145
  ,Remarks: 'A992'
}).alignment = { horizontal: 'center' }

I had same issue and played around for hours until I worked it out. Do it using views like this instead. The below will create a sheet names "test", colour it blue, remove the grid-lines, freeze the top 5 rows and make the active cell A1:
workbook.addWorksheet('test',{properties:{tabColor:{argb:'FF00FF00'}},views:[
{state: 'frozen', ySplit: 5, activeCell: 'A1', showGridLines:false}
]})

that worked, thanks.

I had same issue and played around for hours until I worked it out. Do it using views like this instead. The below will create a sheet names "test", colour it blue, remove the grid-lines, freeze the top 5 rows and make the active cell A1:
workbook.addWorksheet('test',{properties:{tabColor:{argb:'FF00FF00'}},views:[ {state: 'frozen', ySplit: 5, activeCell: 'A1', showGridLines:false} ]})

Thanyou so much dudde, it worked !!


manager.PackagesDownloadStarted not fired

Hey guys,
Maybe i'm just not good enough with C#, but for me it seems that the event manager.PackagesDownloadStarded is not fired.
My Code:
using nUpdate.Updating;
using System;
using System.Diagnostics;
using System.Globalization;
using System.Threading;
using System.Windows;

namespace KLauncher____Installer {
 /// <summary>
 /// Interaktionslogik für MainWindow.xaml
 /// </summary>
 public partial class MainWindow: Window {
  public MainWindow() {
   InitializeComponent();
  }

  private void button_Click(object sender, RoutedEventArgs e) {
   UpdateManager manager = new UpdateManager(new Uri("ServerURL"), "Key provide by nUpdate Administration", new CultureInfo("de-DE"));
   manager.PackagesDownloadStarted += Manager_PackagesDownloadStarted;
   var updaterUI = new UpdaterUI(manager, SynchronizationContext.Current);
   updaterUI.ShowUserInterface();
   manager.RestartHostApplication = false;

  }
  public void Manager_PackagesDownloadStarted(object sender, EventArgs e) {
   MessageBox.Show("Event fired!");
  }
 }
}

Please tell what i'm doing wrong:)
Have nice easter days.

First: You must set your properties before starting the update process.
The line manager.RestartHostApplication = false; should consequently be placed above the call of updaterUI.ShowUserInterface(). The reason is that the class UpdaterUI does everything automatically in this method (seaching, downloading and installing) and thus, your program will be terminated at the end of this method, not taking care of your property. In any case, the UpdateManager instance must already be fully initialized when you're handling it over as argument to the constructor of UpdaterUI as this one keeps its instance in a backing field not noticing any further changes.
The second problem, that is also responsable for the event not being fired, can be split into two parts:
The UpdaterUI class, as already mentioned, implements the whole update process over the UpdaterManager, so that the user does not have to do anything else than calling ShowUserInterface().
Now, there are two configurations of nUpdate. The one is ProvideTAP (offering the Taskbased asynchronous pattern) which you may be using, I think, and the other one is WithoutTAP (offering the Eventbased asynchronous pattern only). WithoutTAP will be removed in v4 btw.
Now, the UpdaterUI is determining the used configuration and either uses Tasks + async/await, or async methods with events then. But, in any case, even if you use EAP, the class is not designed to fire any events aiming that the user should handle them. The reason is, as already described, that everything is done automatically and there is no need for subscription of events. The events should only be used when you're implementing a custom UI or no UI at all. That would mean that you only use the UpdateManager itself and in this case you may implement the updating process using events.
tl;dr: UpdaterUI is a finished implementation of the UpdateManager for offering the user an integrated user interface and besides that, the user does not have to do anything else. You can also have a look at it on GitHub to see how this is working internally.

Much thanks for your answer!
My root problem is, that nUpdate does not close my application while installing, therefore the user have to do it manually. This is the reason why im trying to catch the event.
May you help me a little bit out.

Ah, I see. You're using WPF, that's the problem. But, in any case, that wouldn't have been the necessary event. ;)
Version 3.x of nUpdate requires a small workaround for WPF as nUpdate, by default, calls Application.Exit which will not close the host application correctly. Create a new class and add this:
using nUpdate.Updating;

public class WPFUpdateManager : UpdateManager
{
    public WPFUpdateManager(Uri updateConfigurationFileUri, string publicKey, CultureInfo languageCulture) : base(updateConfigurationFileUri, publicKey, languageCulture)
    { }
    public override void TerminateApplication()
    {
        Environment.Exit(0);
    }
}

Then use that class instead of `UpdateManager`. This should fix your problem.

This works!
Thanks alot!

See also: #31
This issue is now fixed and thus, a workaround is no longer necessary.


[TEST] Add RWO and RWX filesystem trim tests

What's the test to develop? Please describe

Describe the tasks for the test

Additional context

https://github.com/longhorn/longhorn-tests/pull/1689 should be able to cover RWO and RWX volumes.

Let's track it by https://github.com/longhorn/longhorn/issues/7771. Thank you.


can signing be made faster?

takes about 350ms at the moment...

got it down to 60-80ms. good enough


Make an app icon

We need an icon to use for the application icon.

A little sleepy cat maybe?

Ermagad its little face! I love sleeping cats! The colors here are lovely! This would suit me fine as an icon. 😸  Thank you!
I'm looking through my macOS app icons now and most of them are square or in portrait orientation except for Activity Monitor, which is a bit wider than it is tall. Do you have any concerns about the width:height ratio here?
Does this icon bring any names to mind, re: #57?

This was a first draft to see if we liked the idea - I can tweak him to be more square, and maybe enlarge the face a little. I need to try him at icon-size to make sure he works. Are we planning on using Electron's notifications at all (little red circles)? I will add my tweaks to this issue so we can see how our little cat progresses.

Are we planning on using Electron's notifications at all (little red circles)?

I haven't seen it discussed by thats' a good idea. Are you wondering if the icon will work when there's a red circle in the corner?

Square-r width to height ratio:

I think the Electron notifications are on the top right corner so they should just cover up some of his l'il butt, which is fine.

Oh man, that square ratio looks weird, like kitty is doing yoga. I prefer your original wider image.
One way of getting a portrait orientation would be a cat standing, perhaps stretching forward toward the viewer, and yawning. That would be a completely different image, though, so I understand it'd be more work. We might not even need a portrait or square icon, I wouldn't worry about it until we find we need it.

Eh, I kind of like him as a circle. More of a top-down view. Unfortunately, I'm going to have to open a branch to add in all the electron packaging tools to see a sample icon in use. So I'll get that all set up so we can see how it behaves as an icon.

Relaying feedback from @probablycorey:


As a CNB user on Knative, I can use a PVC to cache builds

Acceptance Criteria
Case 1
When I configure a CNB build template for knative that uses a persistent volume claim as an interbuild cache
And The build complete successfully
Then The cache from that build is available to the next build that uses the same PVC
Case 2
When I configure a CNB build template for knative that uses a persistent volume claim as an interbuild cache
And The build fails or is otherwise interrupted at any point
Then The cache from that build is not available to the next build that uses the same PVC.
NOTES

Update this repo with template and example for now. Make a note to create a PR to https://github.com/knative/build-templates after the next Lifecycle release

Knative template merged: https://github.com/knative/build-templates/pull/119


Update README.md

Signed-off-by: Heather Halter<EMAIL_ADDRESS>Fixed grammar on Points of Contact
Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
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.

Testing DCO check integration


400 bad request error while Sign In

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.22.0
Wrapper Library
MSAL React (@azure/msal-react)
Wrapper Library Version
1.3.0
Public or Confidential Client?
Public
Description
Can't be able to do azure b2c login with two tabs.
Reproduce the error:
Step: 1 Open the login page in two tabs.
Step 2: Now go to the first tab and do login and left the tab as it is.
Step 3: Now go to second tab and try to do login
Step 4: See the network tab of second you will get error of 400 bad request
Error Message

Msal Logs
No response
MSAL Configuration
const msalConfig: Configuration = {

    auth: {
        clientId: xyz....,
        authority: b2cPolicies.authorities.signUpSignIn.authority,
        knownAuthorities: [b2cPolicies.authorityDomain],
        redirectUri: process.env.REACT_APP_AUTH_REDIRECT_URI,
        postLogoutRedirectUri: process.env.REACT_APP_AUTH_REDIRECT_URI,
        navigateToLoginRequestUrl: true
    },
    cache: {
        cacheLocation: BrowserCacheLocation.SessionStorage,
        storeAuthStateInCookie: isIE
    }
}

Relevant Code Snippets
----

Reproduction Steps
Step: 1 Open the login page in two tabs.
Step 2: Now go to the first tab and do login and left the tab as it is.
Step 3: Now go to second tab and try to do login
Step 4: See the network tab of second you will get error of 400 bad request
Expected Behavior
It should not return 400 error instead it should do login as usual.
Identity Provider
Azure B2C Basic Policy
Browsers Affected (Select all that apply)
Chrome, Firefox, Edge, Safari, Internet Explorer, None (Server), Other
Regression
No response
Source
Internal (Microsoft)

@anjalikalsariya Can you share us msal logs? You can enable them as instructed here

@sameerag I have added below msal config
export const msalConfig: Configuration = {
auth: {
    clientId: clientId,
    authority: b2cPolicies.authorities.signUpSignIn.authority,
    knownAuthorities: [b2cPolicies.authorityDomain],
    redirectUri: process.env.REACT_APP_AUTH_REDIRECT_URI,
    postLogoutRedirectUri: process.env.REACT_APP_AUTH_REDIRECT_URI,
    navigateToLoginRequestUrl: true
},
cache: {
    cacheLocation: BrowserCacheLocation.SessionStorage,
    storeAuthStateInCookie: isIE
},
system: {
    loggerOptions: {
        loggerCallback: (level: LogLevel, message: string, containsPii: boolean): void => {
            if (containsPii) {
                return;
            }
            switch (level) {
                case LogLevel.Error:
                    console.error(message);
                    return;
                case LogLevel.Info:
                    console.info(message);
                    return;
                case LogLevel.Verbose:
                    console.debug(message);
                    return;
                case LogLevel.Warning:
                    console.warn(message);
                    return;
            }
        },
        piiLoggingEnabled: true
    },
    windowHashTimeout: 60000,
    iframeHashTimeout: 6000,
    loadFrameTimeout: 0,
    asyncPopups: false
}

}
And not getting any log in inspect.

Please set logLevel: LogLevel.Trace in your config options and collect MSAL JS logs in the Console. An example can be found here.
@bmahal assigning this to you for follow up.

@anjalikalsariya Can you please share verbose logs on email mentioned on my profile, if they are not showing up for some reason, please update.


Recommend Roarr over Pino

The current recommended component for logging is Pino
https://github.com/nodeshift/nodejs-reference-architecture/blob/main/docs/operations/logging.md
Roarr is an actively maintained Node.js JSON logger that has several benefits over Pino and other frameworks:

Does not block the event cycle (=fast).
Does not require initialisation.
Produces structured data.
Decouples transports.
Has a CLI program and web application for reading logs.
Works in Node.js and browser.
Configurable using environment variables.

I have provided an in depth write up about my motivation for creating and maintaining Roarr.
Perhaps the most useful (unique) feature of Roarr is adopt, which allows to inherit log context and describes relationships between logs in async contexts using sequence.

@gajus thanks for the heads up on another option for logging.
One of the key elements of the recommendations is that they are based on the existing experience in our internal deployments and work with our customers. From https://github.com/nodeshift/nodejs-reference-architecture/blob/main/README.md, the key aspect we've documented is Where possible the opinion is based on what we've used internally and in our customer engagements.
We acknowledge that there will be other options which are as good but the net is that we'll end up pointing to ones that we have experience with.
At this point we have a number of internal teams/implementations using pino and have experience with it across the different groups which is one of the reasons it is in the architecture.
I've leave this issue open for another week for visibility across the team, but to set expectations, we'd have to see our internal usage to shift from pino to Roarr before we are likely to update our suggestions.

Closing this out now as it has been open for awareness for over a week.


Javascript actions

Is it possible to render a button inside the card and on click of it, I can perform some options. I used cardDetailedText but didn't work. May be I am missing something. Could you please provide an example if it is possible.

I'm experiencing a similar problem. I used "a tag" for cardDetailedText, but the page movement doesn't work. I think the JavaScript method of choosing a card to guess preoccupies the click event.

@ajayaj724 have you tried the custom rendering option. the custom rendering option allows you to render custom views inside the timeline card. let me know if this is sufficient.

This error appears in vertical mode.
ex)
const data = [
{
title: '25 July 1940',
cardTitle: 'Berlin',
cardSubtitle: RAF Spitfire pilots scramble for their planes,
cardDetailedText: After France’s surrender in June 1940, Churchill told the British people, “Hitler knows that he will have to break us in this island or lose the war”. To mount a successful invasion, the Germans had to gain air superiority. The first phase of the battle began on 10 July with Luftwaffe attacks on shipping in the Channel.   The following month, RAF Fighter Command airfields and aircraft factories came under attack. Under the dynamic direction of Lord Beaverbrook, production of Spitfire and Hurricane fighters increased, and despite its losses in pilots and planes, the RAF was never as seriously weakened as the Germans supposed.,
},
];

I temporarily resolved it by handling event exceptions in the JS file generated after the npm build.
origin  code : function mh(e,t,n,r){var i=re,o=Ar.transition;Ar.transition=null;try{re=1,ps(e,t,n,r)}finally{re=i,Ar.transition=o}}
edit code : function mh(e,t,n,r){var i=re,o=Ar.transition;Ar.transition=null;try{rif(r.target.tagName==="A"){re=i,Ar.transition=o}else{re=1,ps(e,t,n,r)}finally{re=i,Ar.transition=o}}

@zeri-k can you expand on this issue you have mentioned. what is happening with the rest of the page?

I used "a tag" for cardDetailedText, but the page movement doesn't work

@prabhuignoto
There is no problem elsewhere.
When I refer to the file I created and run it, I can see that when I click on the tag a in cardDetailedText, page movement does not occur.
package.json
{
  "name": "react-chrono",
  "private": true,
  "version": "0.0.0",
  "type": "module",
  "scripts": {
    "dev": "vite",
    "build": "tsc && vite build",
    "lint": "eslint . --ext ts,tsx --report-unused-disable-directives --max-warnings 0",
    "preview": "vite preview"
  },
  "dependencies": {
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
    "react-chrono": "2.6.1"
  },
  "devDependencies": {
    "@types/react": "^18.2.79",
    "@types/react-dom": "^18.2.25",
    "@typescript-eslint/eslint-plugin": "^7.2.0",
    "@typescript-eslint/parser": "^7.2.0",
    "@vitejs/plugin-react": "^4.2.1",
    "eslint": "^8.57.0",
    "eslint-plugin-react-hooks": "^4.6.0",
    "eslint-plugin-react-refresh": "^0.4.6",
    "typescript": "^5.4.5",
    "vite": "^5.2.9"
  }
}

App.tsx
import './App.css'
import { Chrono } from "react-chrono"
import data from "./data";

function App() {
  return (

              <Chrono
                  items={data}
                  mode="VERTICAL"
                  cardHeight={100}
                  readMore={true}
                  parseDetailsAsHTML
                  mediaSettings={{ align: 'center', fit: 'contain' }}
              />

  )
}

export default App;

data.ts
const items = [
    {
        title: '25 July 1940',
        cardTitle: 'Berlin',
        cardSubtitle: `RAF Spitfire pilots scramble for their planes`,
        cardDetailedText: `After France’s surrender in June 1940, Churchill told the British people, “Hitler knows that he will have to break us in this island or lose the war”. To mount a successful invasion, the Germans had to gain air superiority. The first phase of the battle began on 10 July with Luftwaffe attacks on shipping in the Channel.
  The following month, RAF Fighter Command airfields and aircraft factories came under attack. Under the dynamic direction of Lord Beaverbrook, production of Spitfire and Hurricane fighters increased, and despite its losses in pilots and planes, the RAF was never as seriously weakened as the Germans supposed.`,
    },
];

@zeri-k just curious why does the enclosing div has height 100vh. does the timeline take up the entire breadth and length of the page. can you try to host the code you are running in a codesandbox.io and share it with me.

@prabhuignoto When there are many timelines, the entire browser scroll becomes active. However, this causes the top control panel to not be fixed, and it becomes invisible when scrolling down.
so I set it to 100vh to show it to match the size of the screen currently running.

In vertical mode, disableInteraction = true solves the issue but then it won't render the timeline points...


docs: pin furo version

Fixes #987.

I subscribed to the issue for making Breathe compatible with Sphinx 7. Once that passes, I'll unpin this and bump the minimum Sphinx version. (That said, I've been seeing colleagues evaluating Quartodoc and I may do that here as well...)


I18N Issues based on 1.0.2

Fix some I18n issues.

Thanks for the PR @alexclassroom! We'll include it in the next release (with some adjustments for future updates of our modules).


Format output of dapr --version as JSON.

Description
modify for the output of dapr --version as JSON format.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #737
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

@fanfanbj pls. fix lint issue too.

@fanfanbj pls. fix CI failures.

note:
There is command: "dapr mtls export -o ./certs”.
"-o json" as global will be conflict with that, so change to "--output-as-json".

/ok-to-test

@fabistb If we are following that style I would still recommend the flag --output-as-json only to be present in the version command. Also the --version flag for dapr command with no options ... do not remove that as that would be a breaking change.

@fabistb If we are following that style I would still recommend the flag --output-as-json only to be present in the version command. Also the --version flag for dapr command with no options ... do not remove that as that would be a breaking change.

@mukundansundar I followed your suggestion, to keep dapr --version, and add dapr version --output-as-json for use too.
But I think this is only for keeping dapr--version without breaking. I am thinking the issue is to add --output-as-json as global for all command.

@fanfanbj would you be able to add a self-hosted e2e test for this command ?

ok. I will add it.

@fanfanbj would you be able to add a self-hosted e2e test for this command ?

ok, I will add it.
And @mukundansundar would you help me take a look why above CI failed. I failed to find the reason. Thanks a lot.

@fanfanbj golangci-lint is giving the error that file is not goimport-ed See https://github.com/dapr/cli/runs/5387099012?check_suite_focus=true#step:4:81

@mukundansundar Yes, some files has wrong goimport sort, I ran the command  goimports -local github.com/dapr/ -w . to change those files, but those files is not related with my check.

@mukundansundar I have fixed CI failure, and added self-hosted e2e test. Please help to review it, thanks a lot.

@mukundansundar I did squash my last 10 commits together, and make rebase and push. Please help  to approval it，thanks a lot.

@fanfanbj will you be able to update the PR to address some of the comments above?

@fanfanbj will you be able to update the PR to address some of the comments above?

ok, no problem.

@mukundansundar ok, done. Help to make review, thanks.

@fanfanbj it seems to have conflicts, can you please resolve them by merging the latest master in?

@shubham1172 The conflict has been fixed. Help to review, thanks.

@fanfanbj I had put in some more comments (I don't have permissions to unresolve them), can you have a look please?

@fanfanbj I had put in some more comments (I don't have permissions to unresolve them), can you have a look please? #878 (comment) https://github.com/dapr/cli/pull/878/files#r838078229

@shubham1172 Fixed. Thanks a lot.

@fanfanbj Can you resolve the conflicts in this PR? thanks ...

@mukundansundar ,
The conflict has been fixed.
Could you merge this PR （not take too long time） for not creating new conflict.

@fanfanbj would you mind to fix the lint issue (https://github.com/dapr/cli/runs/6186809220?check_suite_focus=true)?
  Error: Comment should end in a period (godot)
  Error: Comment should end in a period (godot)
  Error: structtag: struct field cliVersion has json tag but is not exported (govet)
  Error: structtag: struct field runtimeVersion has json tag but is not exported (govet)
  Error: SA9005: struct type 'github.com/dapr/cli/cmd.daprVersion' doesn't have any exported fields, nor custom marshaling (staticcheck)

  Error: issues found

Thank you so much for this contribution @fanfanbj!


Some changes to the way sampling intervals are specified/used

In the current code base, there is code to randomize the sampling interval, but it got disabled by 18dbd6c45ffc9d60490fab6b6d4e727af805cfc2.
This PR adds back the functionality, but, by default, uses a fixed sampling interval, with the option of specifying the interval explicitly via a min/max parameter pair.
Also, when the interval is randomized, it changes it every 200 collected samples rather than on every iteration of the processor loop (resetting the timer too often means the actual intervals are never honored, because the timer is reset on the setitimer() call, which is not syncronized with timer expiration).

Awesome, thanks.


Game Design Idea: Ultimate Tic Tac Toe

https://en.wikipedia.org/wiki/Ultimate_tic-tac-toe

pain

I'm still not sure how i'd code this, but now that i'm reimplementing tictactoe in rust for this project, it may be easier to do this. not sure yet

FWIW, i've found this, and we may be able to tie it into the rust portion of charbot:
https://github.com/evilnose/ultimate-tictactoe


⚠ Warning: SPAM Small Issues is a Big Open-Source Cheat ⚠

What would you like to share?
Please be aware of the following information:

SPAM Small Issues is not a legitimate or ethical open-source practice.
Engaging in SPAM Small Issues can harm the open-source community and its reputation.
SPAM Small Issues refers to the act of creating numerous small, low-value issues or pull requests in open-source repositories without providing meaningful contributions.
It is important to contribute to open-source projects with genuine and valuable contributions, such as bug fixes, feature implementations, documentation improvements, or meaningful discussions.
SPAM Small Issues can disrupt the workflow of maintainers and developers, create unnecessary noise, and waste valuable resources.
Open-source communities thrive on collaboration, respect, and meaningful contributions. Engaging in SPAM Small Issues undermines these principles and can result in negative consequences.
If you encounter SPAM Small Issues in open-source repositories, please report them to the repository maintainers or the respective platform administrators.
Be mindful of your actions and strive to contribute positively to open-source projects, fostering a healthy and supportive environment for developers worldwide.

Remember, open source is built on trust, collaboration, and mutual respect. Let's work together to create meaningful contributions that benefit the entire community.
Additional information
No response
Checklist

[X] I have read the Contributing Guidelines
[X] I have checked the existing issues
[ ] I am willing to work on this issue (optional)
[ ] I am a GSSoC'23 contributor

I completely agree with the information provided @darkterminal. Spamming on small issues is not only unethical but also harmful to the open-source community and its reputation. It is crucial to contribute to open-source projects with genuine and valuable contributions, such as bug fixes, feature implementations, documentation improvements, or meaningful discussions.
SPAM Small Issues only disrupt the workflow of maintainers and developers, creating unnecessary noise and wasting valuable resources. Open-source communities thrive on collaboration, respect, and meaningful contributions, and it's important to uphold these principles.
Let's all strive to contribute positively to open-source projects and foster a healthy and supportive environment for developers worldwide.

SPAM Small Issues: It's hurt me a lot... as an open-source activist and repository maintainer.

I want to take part in this organization.


Expand open Vpn access

Hi,
I have created the s2s vpn between a desktop in my premises and a cloud environment.
From the client server i can ping the Openvpn server (cloud enviroment) but i cant ping the server from a different desktop from the same premises vlan.
I have add static route to my second desktop in my premises but the connection fails.
Do you know how i can fix that?

Please add more technical details


No ARM64 RHUI installation method

Hi there,
Now there are ARM64 RHEL images in the Azure Marketplace. But there's no way to install the ARM64 RHUI packages. Could you please add this part? Thanks!

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

ID: 5d7e0b6a-1b40-f874-4a68-f368761a1ef8
Version Independent ID: cca3c590-2725-6167-2cf0-412b1aae4433
Content: Red Hat Update Infrastructure - Azure Virtual Machines
Content Source: articles/virtual-machines/workloads/redhat/redhat-rhui.md
Service: virtual-machines
Sub-service: redhat
GitHub Login: @ju-shim
Microsoft Alias: jushiman

@yuxisun1217
Thanks for your feedback! We will investigate and update as appropriate.

@yuxisun1217
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

Thanks for your contribution to our documentation
We are moving to a new user feedback system. As part of that effort, we are creating a content request in our backlog to address this gap and are closing this GitHub issue. The updated content will be published shortly.

Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


(PUP-9497) Deprecate fine grained control of file and environment tim…

…eouts

CLA signed by all contributors.

$ be ./bin/puppet config print environment_timeout filetimeout
Warning:             Fine grained control of filetimeouts is deprecated. In future
            releases this value will only determine if file content is cached.

            Valid values will be 0 (never cache) and 'unlimited' (always cache).
            With a default of 'unlimited'.

   (location: /Users/Melissa/puppet/lib/puppet/defaults.rb:613:in `block in <module:Puppet>')
Warning:           Fine grained control of environment timeouts is deprecated,
          please use `0` or `unlimited` to control default caching behavior
          and the environment-cache endpoint in Puppet Server's administrative
          API to expire the cache as needed

   (location: /Users/Melissa/puppet/lib/puppet/defaults.rb:651:in `block in <module:Puppet>')
environment_timeout = 20
filetimeout = 20

and with the default values:
$ be ./bin/puppet config print environment_timeout filetimeout
environment_timeout = 0
filetimeout = 15

@justinstoller @Magisus can we get confirmation that we still want these deprecation warnings to land in 6.y, and the associated removals will happen in puppet 7.y?

Yes.
I doubt we can confirm right now that we will be building the feature for 7.0 that required this change. But everyone I discussed it with considered the current behavior buggy and misleading, and this change valuable regardless.

$ be ./bin/puppet config print environment_timeout filetimeout
Warning: Fine grained control of filetimeouts is deprecated. In future
releases this value will only determine if file content is cached.

Valid values will be 0 (never cache) and 'unlimited' (always cache).
With a default of 'unlimited'.

   (location: /Users/Melissa/puppet/lib/puppet/defaults.rb:613:in `block in <module:Puppet>')
Warning: Fine grained control of environment timeouts is deprecated,
please use `0` or `unlimited` to control default caching behavior
and the environment-cache endpoint in Puppet Server's administrative
API to expire the cache as needed

   (location: /Users/Melissa/puppet/lib/puppet/defaults.rb:651:in `block in <module:Puppet>')
environment_timeout = 20
filetimeout = 20

@joshcooper updated

$ be ./bin/puppet config print environment_timeout filetimeout
Warning: Fine grained control of filetimeouts is deprecated. In future
releases this value will only determine if file content is cached.

Valid values will be 0 (never cache) and 'unlimited' (always cache).
With a default of 'unlimited'.

   (location: /Users/Melissa/puppet/lib/puppet/defaults.rb:613:in `block in <module:Puppet>')
Warning: Fine grained control of environment timeouts is deprecated,
please use `0` or `unlimited` to control default caching behavior
and the environment-cache endpoint in Puppet Server's administrative
API to expire the cache as needed

   (location: /Users/Melissa/puppet/lib/puppet/defaults.rb:651:in `block in <module:Puppet>')
environment_timeout = 20
filetimeout = 20


add libressl compatibility

fixes undefined reference to ASN1_STRING_get0_data on systems with libressl

tested in travis under Alpine Linux

Thanks for the patch!  Can you please add a "Signed-off-by" header to the commit message?
For more information, please see http://docs.openvswitch.org/en/latest/internals/contributing/submitting-patches/

commit message revised with Signed-off-by

Thanks, I applied this to master.


postmark costs december

Current dependencies on/for this PR:

main

PR #78   👈

This stack of pull requests is managed by Graphite.


Add cifdump CLI utility

Given the recent change by the PDB to no longer provide MTZ files for structures (https://www.rcsb.org/news/feature/66ead31f2c751d5c77fbb706) it would be very useful to have a CLI tool equivalent to mtzdump for CIF files. We plan on using this repo at my workplace and this feature would be of great utility to us.
This was brought up in #229 and the changes are relatively straightforward given that rs can already read CIF files. Most of the logic was taken from the existing mtzdump utility.
Testing

cifdump can be accessed from the CLI
cifdump can be used to log the contents of a structure factor CIF file from the PDB
Docs can be built and new CLI utility appears

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

Project coverage is 88.94%. Comparing base (ca2a590) to head (2b7f7b1).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #283      +/-   ##
==========================================
- Coverage   88.98%   88.94%   -0.04%
==========================================
  Files          40       40
  Lines        2841     2841
==========================================
- Hits         2528     2527       -1
- Misses        313      314       +1

Flag
Coverage Δ

unittests
88.94% <ø> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks for the welcome @kmdalton!
I just corrected a final typo. Looking forward to contributing elsewhere.

I triggered the CI workflows, and we can merge this in assuming all passes as expected

Merged in -- thanks for the contribution, @jrobsontull!


No callbacks when AlarmManager is used

Thank you very much for your work. I would like to integrate unifiedPush into my app. In the process, I noticed a critical (for me) error.
It is not possible to use the AlarmManger with the android_alarm_manager_plus plugin and also the unifiedPush Flutter Connector. When adding the AlarmManger to the project, the callback functions e.g. "onNewEndpoint" are not called anymore.
How to reproduce the problem:

Create a new project with working Flutter connector (tested with the sample app).
add the android alarm manager plugin (tested with the version 2.1.2)
initialize the alarm manager
try to use the callback function, e.g. to register a new endpoint

At the moment I have no idea where exactly the problem is and how to fix it.

Do you know if the alarm manager works when UnifiedPush is initialized ?

Following some tests, UnifiedPush does not work as soon as new FlutterEngine(context) (FlutterBackgroundExecutor.java#startBackgroundIsolate) is called

Any update on this? Did you discover the exact problem and how to solve it?


Java 11 Support

I am using OpenJDK V11. Code compiles and runs under eclipse without issue, but when I run on production release, I get:
$ java -version
openjdk version "11.0.4" 2019-07-16
OpenJDK Runtime Environment (build 11.0.4+11-post-Ubuntu-1ubuntu219.04)
OpenJDK 64-Bit Server VM (build 11.0.4+11-post-Ubuntu-1ubuntu219.04, mixed mode, sharing)

$ java -jar /export/share/Seshat.jar
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.nustaq.serialization.FSTClazzInfo (file:/export/share/Seshat.jar) to field java.lang.String.value
WARNING: Please consider reporting this to the maintainers of org.nustaq.serialization.FSTClazzInfo
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

The POM imports:

com.fasterxml.jackson.core
jackson-core
2.10.0.pr1

I cannot fix those warnings as serialization is impossible without using reflection. However 3.0.0 (java 14) replaces most of the "illegal" private API's by official replacements.


Night CI 2023-05-27: Failed

action url: https://github.com/spidernet-io/egressgateway/actions/runs/5100524519

ignore ci issue


[Add] Neetcode in Competitive Programming Category

Site URL
https://www.youtube.com/@NeetCode
Description of the above provided site
This is a youtube channel which help students and professional to become better in Competitive Programming and Tech Interviews
Category
Competitive Programming
Sub Category
tutorials
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am participating in Hacktoberfest 2023

@singodiyashubham87
I have assigned you the issue. Please proceed with it, and don't hesitate to tag me @Anmol-Baranwal or @CBID2 if you require any assistance.


Enable setting instructions when activating bank transfers

As an admin setting up fiscal hosting for the first time, I click start accepting financial contributions and then can select add stripe or add bank account. If I select `add bank account, I see this screen.

I can select my currency and add my bank account details. Fine.
The second paragraph of explainer text says:

Define payment instructions: Include any details contributors will need to send you money, such as: account name and number; IBAN, Swift, or routing codes; bank name and address, etc. The amount and order ID will be automatically included.

But there's no way for me to set the instructions on this page. I have to save the bank account details, then go into my settings to see or edit the instructions.

I prefer we change the copy to match how it's working today. On the short term, it's not likely we will prioritize adding instructions there.

I prefer we change the copy to match how it's working today. On the short term, it's not likely we will prioritize adding instructions there.

OK, I'll update the copy as part of my language changes I'm working on a PR for now. Maybe I can add a note there explaining how to edit the instructions in settings.

OK, I'll update the copy as part of my language changes I'm working on a PR for now. Maybe I can add a note there explaining how to edit the instructions in settings.

I'll pick this up in my 'onboarding processes' review that I'll propose for this Q.


Need a version upgrade of rocksdb

Hi,
The CI build fails due to the incompatible versions between rust-rocksdb and rocksdb.
The dependency of rust-rocksdb has been updated to https://github.com/pingcap/tikv/blob/master/Cargo.lock#L528, which contains the codes for sst file ingestion.
But the rocksdb 4.13.5 version which is used in CI build doesn't have these codes.

Hi @hhkbp2
Can you fix it? Now we can use https://github.com/facebook/rocksdb/releases/tag/v5.0.1 directly.


Slide Steps Removed?

Was the slide step code talked about in #56 removed? A search of the repo for the word "step" comes up with nothing. Adding a .step() method to a slide results in an error.
Was the method renamed?

I found the initial implementation a bit rough around the edges, and the contributor and I kind of never agreed on how to handle the initial step. The functionality is more or less implemented in the steps branch, but has never made it into master. Would be cool to have, though.

I'm starting to get asked about animations in my slides. I'm thinking that having macros with animations would be very interesting. I looked into some of the things JQuery can do out of box and they'd be interesting macros.

Please re-open if still relevant.


CI: Disable failing on code coverage report upload, Codecov has issues

Problem
Code coverage report upload to Codecov started failing intermittently. Example: https://github.com/crate/crate-python/actions/runs/3177244114/jobs/5177463683
[2022-10-03T20:30:53.977Z] ['error'] There was an error running the uploader: Error uploading to [https://codecov.io:](https://codecov.io/)
Error: There was an error fetching the storage URL during POST: 404 - {'detail': ErrorDetail(string='Unable to locate build via Github Actions API. Please upload with the Codecov repository upload token to resolve issue.', code='not_found')}
Error: Codecov: Failed to properly upload: The process '/Users/runner/work/_actions/codecov/codecov-action/v3/dist/codecov' failed with exit code 255

Run codecov/codecov-action@v3
==> macos OS detected
https://uploader.codecov.io/latest/macos/codecov.SHA256SUM
==> SHASUM file signed by key id 806bb28aed779869
==> Uploader SHASUM verified (ccc032e70958ea3eca9bd15c7fdad5bbacc279c3bab22f227417573356569666  codecov)
==> Running version latest
==> Running version v0.3.2
/Users/runner/work/_actions/codecov/codecov-action/v3/dist/codecov -n  -Q github-action-3.1.1 -Z -C 6492325de0ce91500fa435a9f7e6afb27a579b2b
[2022-10-03T20:30:52.747Z] ['info']
     _____          _
    / ____|        | |
   | |     ___   __| | ___  ___ _____   __
   | |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
   | |___| (_) | (_| |  __/ (_| (_) \ V /
    \_____\___/ \__,_|\___|\___\___/ \_/

  Codecov report uploader 0.3.2
[2022-10-03T20:30:52.772Z] ['info'] => Project root located at: /Users/runner/work/crate-python/crate-python
[2022-10-03T20:30:52.778Z] ['info'] -> No token specified or token is empty
[2022-10-03T20:30:52.830Z] ['info'] Searching for coverage files...
[2022-10-03T20:30:53.501Z] ['info'] Warning: Some files located via search were excluded from upload.
[2022-10-03T20:30:53.501Z] ['info'] If Codecov did not locate your files, please review https://docs.codecov.com/docs/supported-report-formats
[2022-10-03T20:30:53.502Z] ['info'] => Found 2 possible coverage files:
  coverage.xml
  .venv/bin/coverage-3.11
[2022-10-03T20:30:53.502Z] ['info'] Processing /Users/runner/work/crate-python/crate-python/coverage.xml...
[2022-10-03T20:30:53.540Z] ['info'] Processing /Users/runner/work/crate-python/crate-python/.venv/bin/coverage-3.11...
[2022-10-03T20:30:53.572Z] ['info'] Detected GitHub Actions as the CI provider.
[2022-10-03T20:30:53.574Z] ['info'] Pinging Codecov: https://codecov.io/upload/v4?package=github-action-3.1.1-uploader-0.3.2&token=*******&branch=amo%2Fpython311&build=3177244114&build_url=https%3A%2F%2Fgithub.com%2Fcrate%2Fcrate-python%2Factions%2Fruns%2F3177244114&commit=6492325de0ce91500fa435a9f7e6afb27a579b2b&job=Tests&pr=450&service=github-actions&slug=crate%2Fcrate-python&name=&tag=&flags=&parent=
[2022-10-03T20:30:53.977Z] ['error'] There was an error running the uploader: Error uploading to [https://codecov.io:](https://codecov.io/) Error: There was an error fetching the storage URL during POST: 404 - {'detail': ErrorDetail(string='Unable to locate build via Github Actions API. Please upload with the Codecov repository upload token to resolve issue.', code='not_found')}
Error: Codecov: Failed to properly upload: The process '/Users/runner/work/_actions/codecov/codecov-action/v3/dist/codecov' failed with exit code 255

References
Others are observing the same issue. Apparently, https://codecov.io occasionally responds with 404 Not Found.

https://github.com/codecov/codecov-action/issues/557
https://github.com/codecov/codecov-action/issues/595
https://github.com/codecov/codecov-action/issues/598
https://github.com/codecov/codecov-action/issues/837
https://community.codecov.com/t/intermittent-unable-to-locate-build-via-github-actions-api/2936
https://community.codecov.com/t/unable-to-locate-build-via-github-actions-api-for-the-public-repository/3894

Debugging
Trying different approaches like 67d63cc3f or da277721dd.
Supposed solution
The patch da277721dd explicitly adds the use of the with.token option, as confirmed by others to improve the behaviour even on public repos.
The token is not required for public repos, but it makes the workflow less likely to randomly fail because of a limit in codecov CI capabilities [1]. However, the secret token does not always save from erratic behaviour [2-4].
[1] https://github.com/codecov/codecov-action/issues/557#issuecomment-1216749652
[2] https://github.com/codecov/codecov-action/issues/557#issuecomment-1223737955
[3] https://github.com/codecov/codecov-action/issues/598#issuecomment-1223307820
[4] https://github.com/orgs/community/discussions/25701
Still croaks
Example: https://github.com/crate/crate-python/actions/runs/3177539843/jobs/5178341299
Solution
Using cb2d904b8 instead now, using fail_ci_if_error: false.  Thank you! When this is resolved, we may revert this patch.
[1] https://community.codecov.com/t/unable-to-locate-build-via-github-actions-api-for-the-public-repository/3894

It looks like adding CODECOV_TOKEN as an environment variable with f5aedc86 works pretty well so far.

Another occurrence on behalf of #458? The error message is different.
[2022-10-18T18:08:10.081Z] ['error'] There was an error running the uploader:
Error uploading to [https://codecov.io:](https://codecov.io/) Error: There was an error fetching the storage URL during POST:
503 - upstream connect error or disconnect/reset before headers. reset reason: connection failure
Error: Codecov: Failed to properly upload: The process '/home/runner/work/_actions/codecov/codecov-action/v3/dist/codecov' failed with exit code 255

-- https://github.com/crate/crate-python/actions/runs/3275823201/jobs/5391136840#step:5:44

[2022-10-18T19:57:03.064Z] ['error'] There was an error running the uploader:
Error uploading to [https://codecov.io:](https://codecov.io/) Error: There was an error fetching the storage URL during POST: 500 - {"error": "Server Error (500)"}
Error: Codecov: Failed to properly upload: The process '/Users/runner/work/_actions/codecov/codecov-action/v3/dist/codecov' failed with exit code 255

-- https://github.com/crate/crate-python/actions/runs/3276395895/jobs/5392400640#step:5:43


Links opening in popup windows

Is your feature request related to a problem? Please describe.
Since I updated to 5.6.0-beta.2 links now open in a popup rather than my main browser. Therefore, to open them in my browser, I have to right click and press "Open link" rather than "Open link in ferdi"
Describe the solution you'd like
A setting to change default behavior from "Open link in ferdi" to "Open link" in the browser
Describe alternatives you've considered
Right clicking every time and pressing "Open Link"

Seems to be the opposite of what https://github.com/getferdi/ferdi/pull/685 was supposed to have fixed, now it's broken.

I'm also having this issue, so it's not an issue with your install.
I'm using the ferdi-git package from Archlinux AUR.

Same thing happening here with 5.6.0-beta.2 on macOS 10.14.6. Didn't have this issue with Ferdi 5.5.0

Same here with 5.6.0-beta.2 on ubuntu. I have to "control click" to open in browser.

in Slack links open in Ferdi, in O364 Outlook they open in browser, it should be the other way round.

On last nightly (.8), rocket.chat and slack links does not open anymore (unless I ctrl+click) in skype it's opening in browser and no more in ferdi popup.

Appears to be a duplicate of #941. Also, CMD + Left Click is a workaround.


[H] Factor out BorderSide creation for dividers

This allows other code to get the same style without having to know exactly it is computed.

cc @abarth

Thanks!

Going to land on red because it looks like the failure is a flake, and I don't have my corp credentials with me to restart the bot. I'll revert if this fails in the same way.


Error in creating WebSocket connection

Hello! I am getting an error in generating a WebSocket connection, specifically:
Uncaught DOMException: Failed to construct 'WebSocket': The URL 'undefined' is invalid.

I've done a little source-diving, and it appears to be a mismatch between the output to SubscriptionClient of subscriptions-transport-ws and the input to WebSocketLink from apollo-link. It seems that SubscriptionClient has a url property, but WebSocketLink is looking for uri:
https://github.com/apollographql/subscriptions-transport-ws/blob/0da748556bc24ec972dd27afc840e6d63b661b2e/src/client.ts#L112
https://github.com/apollographql/apollo-link/blob/f2124e60dc3720226fd4a88402435fa96835221d/packages/apollo-link-ws/src/webSocketLink.ts#L13
https://github.com/apollographql/apollo-link/blob/f2124e60dc3720226fd4a88402435fa96835221d/packages/apollo-link-ws/src/webSocketLink.ts#L42
Does this seem like a bug, or am I missing something else here?

This issue pertains to the following package(s):

[ ] GraphQL Playground - Electron App
[ ] GraphQL Playground HTML
[x] GraphQL Playground
[x] GraphQL Playground Express Middleware
[ ] GraphQL Playground Hapi Middleware
[ ] GraphQL Playground Koa Middleware
[ ] GraphQL Playground Lambda Middleware

What OS and OS version are you experiencing the issue(s) on?
macOS High Sierra
What version of graphql-playground(-electron/-middleware) are you experiencing the issue(s) on?
v1.5.7
What is the expected behavior?
No errors in WebSocket connection.
What is the actual behavior?
Error in creating a WebSocket connection.

What steps may we take to reproduce the behavior?
Unknown yet.

I set a breakpoint right before the creation of WebSocketLink, and injected the following into the console:
subscriptionClient.uri = subscriptionClient.url

And this makes the error go away.

experiencing same problem as @eliperelman , was a fix done ?

Dunno what the hell is happening there, but the instanceof check is failing (even if we pass an instance of SubscriptionClient), hence it enters on the other branch of the if, resulting in an error
because the instance is treated as configuration, so the uri param is obviously undefined there... 😕

Also it's really strange because my code was working before, then I fresh cloned the repo re-installed everything and 💥 ...
🤔

It should be closed once https://github.com/apollographql/apollo-link/pull/832 got merged. Thanks!

No sign of that pull request being merged, is there a workaround ?

I'm not creating the link in my own code though, I'm just using the <Playground/> component.
I seem to have been able to avoid the problem, based on the hint in this comment, I've added to package.json:
  "resolutions": {
    "subscriptions-transport-ws": "0.9.14",
    "apollo-link-ws": "1.0.8",
    "graphql": "14.0.2"
  }

Thanks @adrian-baker !!! It helped me by just yarn add apollo-link-sw

In case anyone else is here in 2020 or beyond, you need to add these three packages to your package.json if you don't have them already:
subscriptions-transport-ws
apollo-link-ws
graphql

The versions don't need to be pinned (at least, not yet) - I'm using the latest of everything.

Still experiencing this error. Tried to install the following packages without pinning, but still getting error
subscriptions-transport-ws
apollo-link-ws
graphql


Editor: Ignore CapsLock when pressed alone

Now: CapsLock will behave like other modifier keys: does nothing when pressed alone.
Fixes #73073

Thanks!


Add adapter for FoxWallet

Thanks for your contribution. We are no longer accepting PRs of adapters for this type of wallet. You can read about this policy here, but more importantly, please check out this guide for wallets to implement the Wallet Standard.
However, most of what you've written for this adapter is portable to the Wallet Standard interface, and implementing this interface will grant your wallet distribution on dapps across Solana. After reading the guide and looking at the reference implementations, if you have questions on how to do this, please open an issue here:
https://github.com/solana-labs/wallet-standard


Implement DoubleEndedIterator (or Iterator::rev) for EnumSetIter

I ran into a situation where I needed to find the greatest item in an EnumSet. Although this can be done straightforwardly with Iterator::last, it would be more efficient to use DoubleEndedIterator::next_back…if EnumSetIter implemented DoubleEndedIterator, which it currently doesn't.

I'll take a look at what I can do. Note that your use case relies on guarantees that [EnumSet<T>::iter] currently doesn't actually have, namely that the iteration returns values in a particular order. This should be safe to add, but it is worth nothing.

Good point. I had assumed that any bit-set would have a predictable iteration order, but you're right that the documentation doesn't actually promise that. I'll use Iterator::max instead for now.
Thanks for making this crate, by the way. It's a pretty major improvement over the bespoke bit-set implementation I was using.

I added the guarantees needed for this and implemented DoubleEndedIterator. I can't think of a good reason why I might want to not do lowest-to-highest bit order unless on some obscure platform, highest-to-lowest iteration is more efficient -- and the predictability of order seems more important.


export const prerender = true not working

Describe the bug
I am trying to render a static route with:
export const prerender = true;
inserted inside page.ts.
Reproduction

Logs
node:internal/event_target:1006
  process.nextTick(() => { throw err; });
                           ^
Error: The following routes were marked as prerenderable, but were not prerendered because they were not found while crawling your app:
  - /terms

See https://kit.svelte.dev/docs/page-options#prerender-troubleshooting for info on how to solve this
    at prerender (file:///C:/Backup/Projects/Crypto/tier10k/webapp/node_modules/@sveltejs/kit/src/core/postbuild/prerender.js:475:9)
    at async MessagePort.<anonymous> (file:///C:/Backup/Projects/Crypto/tier10k/webapp/node_modules/@sveltejs/kit/src/utils/fork.js:22:16)
Emitted 'error' event on Worker instance at:
    at [kOnErrorMessage] (node:internal/worker:290:10)
    at [kOnMessage] (node:internal/worker:301:37)
    at MessagePort.<anonymous> (node:internal/worker:202:57)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:731:20)
    at exports.emitMessage (node:internal/per_context/messageport:23:28)

System Info
System:
    OS: Windows 10 10.0.19045
    CPU: (32) x64 AMD Ryzen 9 5950X 16-Core Processor
    Memory: 35.78 GB / 63.92 GB
  Binaries:
    Node: 18.12.1 - C:\Program Files\nodejs\node.EXE
    npm: 8.19.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1266.0), Chromium (114.0.1823.67)
  npmPackages:
    @sveltejs/adapter-node: ^1.2.4 => 1.2.4
    @sveltejs/kit: ^1.20.4 => 1.21.0
    svelte: ^4.0.0 => 4.0.1
    vite: ^4.3.6 => 4.3.9

Severity
serious, but I can work around it
Additional Information
No response

Did you read the documentation linked to in the errors message?
GitHub issues aren't the right place for support questions like this. Please ask on StackOverflow or in our Discord chat room. And when you do, provide a reproduction, or they won't be able to help you either.


Add 'CaaS' to spellcheck exception wordlist

Add 'CaaS' to spellcheck exception wordlist
to resolve the spellcheck CI error in https://github.com/cncf/glossary/pull/295#issuecomment-1009542172

/cc @seokho-son @sandipanpanda

Sort words ascending
(How to with VS Code: https://thechrisgreen.com/2021/08/vs-code-sort-lines-of-code-in-ascending-or-descending-order/ )

@jihoon-seo Thanks!
LGTM


Update TextStyle and StrutStyle height docs

This docs change reflects the changes to TextStyle.height and StrutStyle.height we are proposing to make.
See https://github.com/flutter/flutter/issues/29479 and https://github.com/flutter/engine/pull/9041
height default behavior is unchanged, however, when provided, the value now produces line height that is truly height * fontSize tall. Before, we were multiplying the font-provided ascent and descent, which did not necessarily add up to the font size.
This should not be landed until the engine-side PR lands and rolls.

Not yet, haven't written it yet. Writing it now!

Breaking change proposal sent to flutter-announce. The migration for this one may be harder than it might initially seem, so if anyone has any problems with doing so please let me know! I'll see if I can write a more comprehensive guide to doing so.

Update: Holding off on committing this until the tree is in a position to take riskier changes. Due to the very breaking nature of this change, the delay will allow for a much more flexible and longer period of time to work with breakages that this causes.

why is this a breaking change? i only saw comment updates in the diff

The implementation is changing in the engine-side patch (flutter/engine#9041)

new diagram: https://github.com/flutter/assets-for-api-docs/pull/90

Engine patch is part of https://github.com/flutter/flutter/pull/35814


:bug: Fix update restart label

Summary

restart the pod directly instead of add label if the certs are changed
correct the cert name so that it can trigger the inventory api restarts
should not use DeepDerivative for typed object due to https://github.com/kubernetes/apimachinery/issues/110. using unstructured.Unstructured

Related issue(s)
Fixes https://github.com/stolostron/multicluster-global-hub/issues/1125
Tests

[x] Unit/function tests have been added and incorporated into make unit-tests.
[ ] Integration tests have been added and incorporated into make integration-test.
[ ] E2E tests have been added and incorporated into make e2e-test-all.
[ ] List other manual tests you have done.

/lgtm


Implement Necessary LibSodium Primitives in Golang

For interoperability with related projects (eg, ACA-Py), we need to use or reimplement certain libsodium primitives (for example, crypto_box_seal), some of which are not available in golang.org/x/crypto (for example, golang.org/x/crypto/nacl/box.Seal is equivalent to NaCl crypto_box_easy, despite the name)

The current primitives we know we need are equivalent to NaCl crypto_box_seal and crypto_box_seal_open. I've implemented and tested them in my fork of the golang libsodium wrapper: https://github.com/Moopli/libsodium-go/blob/master/cryptobox/crypto_box_seal_test.go
How we should regression-test these is an open question - do we introduce a testing-only dependency to GoKillers/libsodium-go? Do we accept that these functions are tested on a fork of another repo, and that they won't be broken someday once we begin using them?

@troyronda assign me

Key conversion was implemented in #292


Cannot read image file (blender render animation)

Maybe it isn't a bug and maybe it's a Blender problem but still. When I open an image rendered in Blender using Render Animation and trying to open that image sequence through DJV I'm getting that error.
ERROR: Cannot read image file "D:\Creative\showcase\026\rendered\img0001.exr". No such file or directory.
Everything works in other image viewers/editors but not in DJV. Also when I'm opening regular render from blender (saved as single frame render) everything works as expected.
syslog.txt

Hi, thanks for the bug report; I'll check this out...

Hi @darbyjohnston. I also prepared a test blend file with a rendered sequence and single frame image to test.
https://1drv.ms/u/s!AgCDDnfGXq0lyyKkSBR7S9ZC9QI1?e=ia5doj

Great, thanks for the test file!

I'm running into the same issue but for a different reason. DJV throws these errors:

However, I don't see where DJv is getting that EXR1 name from, as none of the files have that name:

Maybe it doesn't like hypens as well and the files must follow some very specific naming convention?

I'm having the same issue. It seems to remove the hyphen separating the number from the original file name, for instance: “animation-exr-001” would be interpreted as “animation-exr001” when no such file or directory exists. It's a shame so many bugs are in the new version of DJV, while the old version (1.3.0) runs perfectly fine.

@JTCHE The old version is still available if you would prefer to use it:
https://sourceforge.net/projects/djv/files/djv-stable/1.3.0/

Thanks for leaving it available to download, that's usually what I use ;)
Though you do miss out on certain features (such as importing an OCIO profile) which I do find useful, but I'm sure after many bug fixes the new revamped version will eventually become as good or even better as 1.3

its the - , DJV cannot see it for some reason! all your files also have them, I deleted these in mine and now it works


glade: 3.38.1 -> 3.38.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for glade is: "User interface designer for GTK applications"
meta.homepage for glade is: "https://wiki.gnome.org/Apps/Glade"
meta.changelog for glade is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.38.2 with grep in /nix/store/pvrwc6xvjcnz8qj6a20s79qxf82gcfda-glade-3.38.2

directory tree listing: https://gist.github.com/08b1e0138e832d5aff8a5cb44681be73

du listing: https://gist.github.com/c23a1a2d572ce6c8933d9eb948343e9f

Rebuild report (if merged into master) (click to expand)

125 total rebuild path(s)

43 package rebuild(s)

43 x86_64-linux rebuild(s)
41 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
41 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
adapta-gtk-theme
chrome-gnome-shell
epiphany
glade
gnome-builder
gnome-passwordsafe
gnome-tour
gnome-usage
gnome3.epiphany
gnome3.geary
gnome3.glade
gnome3.gnome-calendar
gnome3.gnome-clocks
gnome3.gnome-contacts
gnome3.gnome-control-center
gnome3.gnome-initial-setup
gnome3.gnome-maps
gnome3.gnome-screenshot
gnome3.gnome-session
gnome3.gnome-shell
gnome3.gnome-sound-recorder
gnome3.gnome-terminal
gnome3.gnome-tweak-tool
gnome3.gnome-tweaks
gnome3.gnome-usage
gnome3.gnome_control_center
gnome3.gnome_session
gnome3.gnome_shell
gnome3.gnome_terminal
gnome3.pomodoro
gnome3.seahorse
gnomeExtensions.gsconnect
gnomeExtensions.night-theme-switcher
gnunet-gtk
libhandy
lollypop
pantheon.elementary-greeter
pantheon.elementary-gsettings-schemas
pantheon.elementary-onboarding
pantheon.elementary-session-settings
pantheon.wingpanel-applications-menu
pantheon.wingpanel-with-indicators
vala-language-server

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pvrwc6xvjcnz8qj6a20s79qxf82gcfda-glade-3.38.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A glade https://github.com/r-ryantm/nixpkgs/archive/edc252668b91568c67b9bb13993d2c796d7b0866.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pvrwc6xvjcnz8qj6a20s79qxf82gcfda-glade-3.38.2
ls -la /nix/store/pvrwc6xvjcnz8qj6a20s79qxf82gcfda-glade-3.38.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package failed to build:

gnome-passwordsafe

34 packages built:

adapta-gtk-theme
chrome-gnome-shell
epiphany
glade
gnome-builder
gnome-tour
gnome-usage
gnome3.geary
gnome3.gnome-calendar
gnome3.gnome-clocks
gnome3.gnome-contacts
gnome3.gnome-control-center
gnome3.gnome-initial-setup
gnome3.gnome-maps
gnome3.gnome-screenshot
gnome3.gnome-session
gnome3.gnome-shell
gnome3.gnome-sound-recorder
gnome3.gnome-terminal
gnome3.gnome-tweak-tool
gnome3.pomodoro
gnome3.seahorse
gnomeExtensions.gsconnect
gnomeExtensions.night-theme-switcher
gnunet-gtk
libhandy
lollypop
pantheon.elementary-greeter
pantheon.elementary-gsettings-schemas
pantheon.elementary-onboarding
pantheon.elementary-session-settings
pantheon.wingpanel-applications-menu
pantheon.wingpanel-with-indicators
vala-language-server

Maintainer pings
cc @hedning @jtojnar @worldofpeace @dasj19 for testing.

Changes look okay:

============
Glade 3.38.2

Fix windows compatibility issues
Fix build against Python 3.9 (Jan Alexander Steffens)
Fix issue #355 "Make notification text selectable"
Fix issue #53 "Can't choose a FileChooserDialog for a FileChooserButton"
Fix issue #371 "Glade user survey not working (TLS error)"


build: Use encoding=UTF-8

sol-flow-node-type-gen fails on systems not using UTF-8 for locale.
The generation step fails while processing freegeoip.json,
which contains a non-ASCII character. This patch changes this generator
to open files using enconding=UTF-8, so it is not affected by
the system locale.
Signed-off-by: Anselmo L. S. Melo<EMAIL_ADDRESS>

+1

Applied, tks.


[Bug]: 404 error at URL: /txns/1GgDN_7UGBLx9YZmVe5JhGf8bYaK9Ns_Z0y7JJId82A

What happened?
A bug happened!
URL
https://explorer.helium.com/txns/1GgDN_7UGBLx9YZmVe5JhGf8bYaK9Ns_Z0y7JJId82A
What browsers are you seeing the problem on?
Chrome
Do you see the issue on Mobile or Desktop?
Mobile
Relevant log output
No response

The API is a bit behind at the moment. Give it a few more hours, up to a day or so.

The API is caught up, and that transaction now loads properly.


added local decorator

for issue: https://github.com/Netflix/metaflow/issues/350

added a new decorator: @local
this will force steps to run locally even when doing --with batch and  --with kubernetes, by preventing attachment of batch and kubernetes decorators to the steps which have @local attached to them already.
to achieve this:

a classmethod should_attach is introduced in StepDecorator class which will always return True
that method is overridden in batch and kubernetes decorator classes to check for presence of local decorator on the step
the logic to check should_attach condition is implemented in _attach_decorators_to_step function inside decorators.py file

@romain-intel  @savingoyal I have updated the PR with the should attach class method approach. Please review once.

@vardhaman-surana, thanks for the PR. I would like to understand the use case better. Currently, we have @resources and @batch/@kubernetes decorators (and more coming). The attributes for @resources (CPU, mem, etc.) are a strict subset of @batch/@kubernetes today and are effectively ignored unless used --with batch/kubernetes. If the intention is that @local simply ignores the @batch/@kubernetes decorator - the step may not execute as expected if @batch/@kubernetes were providing a specific image or fiddling with tempfs settings, for example.

@vardhaman-surana, thanks for the PR. I would like to understand the use case better. Currently, we have @resources and @batch/@kubernetes decorators (and more coming). The attributes for @resources (CPU, mem, etc.) are a strict subset of @batch/@kubernetes today and are effectively ignored unless used --with batch/kubernetes. If the intention is that @local simply ignores the @batch/@kubernetes decorator - the step may not execute as expected if @batch/@kubernetes were providing a specific image or fiddling with tempfs settings, for example.
One possible workaround is to maybe have METAFLOW_DISABLE_DECOSPECS=kubernetes as an environment variable to disable all references of @kubernetes without providing an explicit @Local decorator.

hi @savingoyal the use case as mentioned in the issue is
A possible use-case for this would be to annotate a GPU-heavy step that you want to run locally to save on AWS costs but take advantage of fast S3 access for other steps (from the EC2 machines batch uses).
i tried adding the local decorator based on this requirement mentioned in the issue. I am not sure if the env variable approach which you are suggesting will work for this case or not.


Building Tagged Release from Source Doesn't set Version

Description of the problem or steps to reproduce
The binary produced from downloading the source of a tagged release, then 'make build'ing it doest get a version number or commit hash.
$ make build
fatal: Not a git repository (or any of the parent directories): .git
go get -d ./cmd/micro
go build -ldflags "-s -w -X main.Version=0.0.0-unknown -X main.CommitHash= -X 'main.CompileDate=June 22, 2017' " ./cmd/micro

Specifications
You can use micro -version to get the commit hash.
Commit hash:
OS: void linux
Terminal: xiate

If you are getting the source from Source code (tar.gz) then it does not include the git repo info so you can not get a version or hash from it, with out hard coding version info in the Makefile yourself.
You have to use the micro-1.2.0-src.tar.gz to get version and hash from the git repo. There is a tag fault on v1.2.0 at the minute but hopfully it will be updated soon.

@tommyshem is correct. If you are getting the source from Github's build in Source code tar.gz, then it will be provided without the .git directory so micro will be unable to get the version from the git history.
You should use the micro-version-src.tar.gz instead which will provide the .git directory (as well as all dependencies and their .git directories -- which explains for the huge size).
I stopped providing the micro-version-src.tar.gz for a limited time, and it was also slightly bugged before (as you pointed out in #713). Please see my response their for the new situation.


Remove Numpy 1.6 compatibility

Based on PR #1748.

Looks good.  Will merge once Travis-CI finishes.


Flow doesn't realise that ! operator always returns a boolean

Link: https://flow.org/try/#0C4TwDgpgBAksEFsoF4oG8CGAuKBnYATgJYB2A5gL4BQVAZgK4kDGwRA9iVPPgBRgCMODCRAAaKLQA2GMgCYcAIzZtJEYQEoccRAG0AuuipQoq4BOll+KKAEIBNY6agEIuepOBb4CfdZ1UASCkZKwAyUPRsKAAiAAsiaIpRIyg9B2cIYHoCThc3DypqIA
type Item = {a: string}

function test(p1: any, flag2: boolean): Item[] {
  let flag1 = !p1

  let result: Item[] = [
	flag1 && {a: "hi"},  // ~~
  ]

  return result
}

Flow accepts this code, but if you change the marked line to use flag2 instead, then it will fail, despite both flags being booleans.

Interesting. Even though any is, by design, completely unsafe and
unchecked, this still seems like a reasonable complaint. The operand may
be of type any, but the result of logical negation is always a
boolean. Furthermore, this issue demonstrates that logical negation
behaves inconsistently with the other operators: if x and y are
variables at type any, then:

x R y has type boolean, where R is one of <, >, <=,
>=, ==, ===, !=, or !==;

~x has type number;

x O y has type number, where O is one of |, &, ^, <<,
>>, or >>>;

x O y has type number, where O is one of -, *, /, %,
or **, but not +; and

of course, x ? false : true has type boolean, and is exactly
equivalent to !x.

@fishythefish this is inconsistent with the way x ? false : true is treated if x: any. It also just doesn't make any sense, because ! always returns a boolean, and !! is frequently used to cast to the boolean type, so why would the "unsafety" leak across that boundary?

That's true - I'll reopen and would welcome a PR. This isn't high-priority for the team right now since any doesn't make any guarantees, so fixes like this are nonessential quality-of-life improvements over Flow's default conservatism.


probably typescript support

i know this library is written in typescript but could make it work in typescript :(
see images below

Sorry i was hyped when i found this hidden gem then i could think straight coz been looking for lightweight crypto solution without the use of  SubtleCrypto for days now. Then i finnaly found this hidden gem.
import "uluru-crypto";
const {encrypt,decrypt} = Uluru

I already came up with solution, it works but some people like me may not be get used to code above. They may not figure out and pass on this gem.

Glad to see that someone actually uses this 😄.
The main reason I actually used typescript was for compilation of many files into one (so the same compiled file can also be used in browser) and self-documentation. If you are experienced with typescript, is there something you would change for better support? Thanks!
Note: The latest version of uluru-crypto has a different pbkdf function!

Everything is good, i dont even know you can write bundle with namespaces  as i use rollup some ts plugins to build single file. I dont know much how namespaces works when import i.e. if every modules has been loaded when importing a namespace or it works just like any other modules that you can only choose what to import.

The problem seems to be the esm way of exporting. Unlike commonjs, i cannot export inside an if statement, which is what I need if I want the same file to be usable in both browser and node. The only other way is to build the whole repo with es6 imports (even instead of namespaces, why not at that point) and bundle it up with something like browserify for browser usage, which is what I always wanted to avoid.

I'll give this update a try later on. I use Rollup for bundling my libraries, you can checkout my usual rollup config  here.

Alright! Rollup works wonders, thank you for the suggestion @paradis-A. Feel free to check out the esm branch c81c867 and let me know what do you think! Should I make this the main branch or just merge it with a pull request?
Only a few big commits, restructuring the whole library for esm, and development already feels much more convenient. Now I am looking to improve testing with something like jest.

With  the #3 pull request, the Ulurucrypto module is fully transfering to ESM modules, that closes this issue :D


[apm] add new out.host and dns.hostname new peer_tags

What does this PR do?
Add out.host and dns.hostname to peer_tags.ini.

Motivation
Improve coverage of inferred services for the Inferred Service dependencies beta.

Additional Notes
These attributes are known to sometimes contain raw IP addresses, depending on the tracer implementation. Since raw IP addresses can be high cardinality, and therefore may potentially increase the agent's memory usage by aggregating stats on all of those IP addresses, we will block merging this change on https://github.com/DataDog/datadog-agent/pull/28229 being merged first.

Describe how to test/QA your changes

Enable both apm_config.peer_service_aggregation and apm_config.compute_stats_by_span_kind.
Send span.kind=span.kind spans to the trace agent with dns.hostname:foo.com set in the meta.
Validate that the peer.hostname:foo.com tag is available on the client metrics

We want to make sure we use semantic core when updating any INI file - please refer to this link and copy the exact file here instead of manually updating it.
This way we ensure everything is linked to SC
Once that is done, PR should be good to go to merge

Updated

IIRC the peer_tags.ini file is now generated from semantic core pipeline. Is this change coming from such pipeline and replicated in the backend .ini file counterpart?

Yes, updated just now

/merge

/merge


feat(forge): add fork_url as an alias for eth_rpc_url in foundry.toml

Component
Forge
Describe the feature you would like
Brief
A user is able to setup fork scenarios using the following declaration:
[arbitrum]
eth_rpc_url = "https://rpc.ankr.com/arbitrum"
match_contract = "Saddle"

If we want to continue the convention of letting config set CLI options, then it would make sense to alias fork_url to eth_rpc_url, as shown:
[arbitrum]
fork_url = "https://rpc.ankr.com/arbitrum"
match_contract = "Saddle"

This was prompted after testing the changes in: #1378
Scope

Are eth_rpc_url in foundty.toml and forge test --fork-url already equivalent? Is priority given to the command line flag?
Is it as simple as adding fork_url as an alias for eth_rpc_url, do we want to reverse the relationship and make eth_rpc_url an alias for fork_url?

Additional context
No response

Are eth_rpc_url in foundty.toml and forge test --fork-url already equivalent? Is priority given to the command line flag?

Yes, and yes.

Is it as simple as adding fork_url as an alias for eth_rpc_url, do we want to reverse the relationship and make eth_rpc_url an alias for fork_url?

I think it's easier to alias fork_url since Cast also depends on eth_rpc_url

how would I approach this if i wanted to work on this issue

Actually, I think we might not want to create this alias since that would also run all tests in fork mode if ETH_RPC_URL is set globally. cc @mds1 for thoughts on this, I know this was not expected behavior when I implemented it the last time

I'm not sure I totally follow the suggesting aliasing and how it would affect config options/defaults, but:

Allowing users to specify fork_url in the config as an alias for eth_rpc_url seems reasonable to me.
Automatically forking if a FOUNDRY_FORK_URL env var is found also seems ok.
The main thing we want to prevent is: the presence of an ETH_RPC_URL env var should not lead to tests forking from that URL. Having a globally defined ETH_RPC_URL is common for cast users, so we don't want that to affect tests.

Now that cast supports config files though, I wonder if this alias puts us in a weird spot: I might want tests to not fork, but cast to use e.g. Optimism by specifying an Optimism RPC in the config file's eth_rpc_url. AFAIK that is not currently doable because specifying  eth_rpc_url will result in tests also forking?
So perhaps we want to decouple the two and have something like this?

fork_url used to determine the network for forge
eth_rpc_url to determine the network for cast, which defaults to fork_url if present, and ETH_RPC_URL env var if fork_url isn't present

To second @mds1's comments, the following scenario is currently not supported:

Specify eth_rpc_url in config
Pass in --fork-block-number on command line

In this scenario the following error will be returned:

Will close this as this is sort of supported now - you can add RPC URLs in the [rpc_endpoints] section of foundry.toml and alias one of them to ETH_RPC_URL, e.g.:
[rpc_endpoints]
mainnet = "${ETH_RPC_URL}"

I think that should cover this functionality, but please let me know if it didn't quite cover what you were asking for :slightly_smiling_face:


Adds test for transaction.

Adds tests around transactions
Description
There's 2 tests, 1 for testing a transaction while inserting data into 2 separate collections.
and the second test is doing the same while still on the same connection.
Extra Context
This is happening on MongoDB V5
Motivation and Context
Have been seeing crashes when running back to back transactions and this adds tests around that area. But in adding the tests, this test case breaks because the connection pool can't seem to determine the max wire version.
Checklist:

[ ] If applicable, I have updated the documentation accordingly.
[x] If applicable, I have added tests to cover my changes.

@Joannis Still failing :/


Fix index documentation page for SDK

Signed-off-by: Shubham Sharma<EMAIL_ADDRESS>Description
The index page was merged with actors page due to a bad merge, and this PR reverts it along with a few more changes.
Issue reference
None.
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[X] Extended the documentation

Result: