Error when building to a folder not owned by the Hugo process

What version of Hugo are you using (hugo version)?
$ hugo version
Hugo Static Site Generator v0.71.0-06150C87/extended linux/amd64 BuildDate: 2020-05-18T16:15:29Z

Does this issue reproduce with the latest release?
Yes it does.
The error occures when the Hugo process have write access to the destination folder but does not own it.
To reproduce, create a public folder owned by root in your Hugo site project (requires sudo access):
sudo sh -c "mkdir public && chmod a+w public"

Run Hugo:
hugo

Log:
Building sites … WARN 2020/05/21 14:09:25 found no layout file for "HTML" for kind "section": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
Total in 111 ms
Error: Error copying static files: chtimes [..]/public/: operation not permitted

Can you please check the file rights on the folder? ls -al foldername should do the trick on a linux system. Nobody should be able to use a folder with only root rights, so that is not a bug. The issue is that you want to give write access to group and all. Worst case something around 0777. chmod 0777 public will solve the issue. The question is if that is safe enough for your environment and setup. If you need to run the creation of the folder with root then something is wrong with your users and setup, not Hugo. Hugo is doing the right thing.

@davidsneighbour Thank you for having a look at my issue report.
I'm aware of the safety aspects of this, I simply constructed an example that should make it easy to reproduce what I reported.
Let me make it step by step, basically as described in my initial post.
Listing before manually creating public folder:
hugo:/workspaces/site-klakegg/src$ ls -la
total 32
drwxr-xr-x 7 hugo hugo 4096 May 21 15:13 .
drwxr-xr-x 7 hugo hugo 4096 May 21 06:34 ..
drwxr-xr-x 3 hugo hugo 4096 May  1 20:20 assets
-rw-r--r-- 1 hugo hugo  282 May  1 20:09 config.yml
drwxr-xr-x 3 hugo hugo 4096 May  1 20:18 content
drwxr-xr-x 4 hugo hugo 4096 May  1 20:18 layouts
drwxr-xr-x 3 hugo hugo 4096 May  1 19:19 resources
drwxr-xr-x 2 hugo hugo 4096 May  1 20:17 static

Creating the public folder, and setting rights accoring to your suggestion:
hugo:$ sudo mkdir public
hugo:$ sudo chmod 0777 public

Here is the listing with the public folder created:
hugo:$ ls -la
total 36
drwxr-xr-x 8 hugo hugo 4096 May 21 14:27 .
drwxr-xr-x 7 hugo hugo 4096 May 21 06:34 ..
drwxr-xr-x 3 hugo hugo 4096 May  1 20:20 assets
-rw-r--r-- 1 hugo hugo  282 May  1 20:09 config.yml
drwxr-xr-x 3 hugo hugo 4096 May  1 20:18 content
drwxr-xr-x 4 hugo hugo 4096 May  1 20:18 layouts
drwxrwxrwx 4 root root 4096 May 21 14:27 public
drwxr-xr-x 3 hugo hugo 4096 May  1 19:19 resources
drwxr-xr-x 2 hugo hugo 4096 May  1 20:17 static

Triggering build:
hugo:$ hugo
Building sites … WARN 2020/05/21 15:10:36 found no layout file for "HTML" for kind "section": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
Total in 103 ms
Error: Error copying static files: chtimes [..]/public/: operation not permitted

Clearly chmod 0777 public did not solve the issue.
It is not clear to me why it is important to trigger chtimes on the folder containing the produced content.

That is weird. With those file rights I would expect everyone being able to change things in the folder. I googled around and found, that chtimes is a GO function, not a shell thing - assumed the first. It might still be file rights related, see the following issue unrelated:
https://github.com/rclone/rclone/issues/738#issuecomment-248603775

Why do you need to create the folder public as user root, not user hugo? Can you try creating it as user hugo (should somehow work via sudo -u hugo or the like)?
Is the file location somehow mounted and are the mount rights either set restrictive or without rights as the issue above discussed?

What's the operating system you are using her?

I've create an example using the root user as I guess that user is accessible to most non-Windows users following this repo.
The folder is not mounted, however everything runs inside a Docker container without any mounting magic.

I've verified it using both Debian and Ubuntu.

I can confirm that it's not docker related. I tried on my own repo locally with a root user created public folder (process got frozen) that was set to 0777 (process ended up with a chtimes error.
So it's probably an issue in how chtimes is called in Go or Hugo.
sudo hugo (the script kiddie solution) by the way ends up in Hugo not finding the configuration.
This all does not solve the mystery why you would create a folder owned by root instead of creating a folder owned by a more public user (the one running hugo, or a completely public one).
Some ideas that probably won't work:

add root to the usergroup hugo
add hugo to the usergroup root/sudo (unsafe)
and still, why is root creating a public directory?

This is where my knowledge about the system ends and @bep or another Go-God might have a look at it.

It is not related to the root user. Try making the folder this way (1337 or other uid not used):
$ mkdir public
$ chmod 0777 public
$ sudo chown 1337 public

Please notice the group ownership is not changed in the above commands.
This will result in the same error when triggering Hugo. The root ownership is simply not relevant, the fact that the folder is owned by any other entity than id -u is the important part.

Error: Error copying static files: chtimes [..]/public/: operation not permitted

Does it work if you do
hugo --noChmod

Or maybe
hugo --noChmod --noTimes

Using --noTimes makes it work, both with and without --noChmod.

Using --noTimes

OK, so this is what's giving a permission error https://golang.org/pkg/os/#Chtimes

TLDR: This is an OS restriction. I recommend closing this issue.
According to POSIX.1-2017:

Only a process with the effective user ID equal to the user ID of the file, or with write access to the file, or with appropriate privileges may use futimens() or utimensat() with a null pointer as the times argument or with both tv_nsec fields set to the special value UTIME_NOW. Only a process with the effective user ID equal to the user ID of the file or with appropriate privileges may use futimens() or utimensat() with a non-null times argument that does not have both tv_nsec fields set to UTIME_NOW and does not have both tv_nsec fields set to UTIME_OMIT.

What does this mean?

If you have write permissions to the file, you can update both access and modify times with touch foo.txt
You must be the file owner, not just a member of the group, to:

Change only the access time with touch -a foo.txt
Change only the modify time with touch -m foo.txt

touch foo.txt
chmod 666 foo.txt
sudo chown root foo.txt #  The behavior is not specific to root; could be any user
stat foo.txt

  File: foo.txt
  Size: 0         	Blocks: 0          IO Block: 4096   regular empty file
Device: 802h/2050d	Inode: 6685210     Links: 1
Access: (0666/-rw-rw-rw-)  Uid: (    0/    root)   Gid: ( 1000/jmooring)
Access: 2021-11-27 06:34:11.052026188 -0800
Modify: 2021-11-27 06:34:11.052026188 -0800
Change: 2021-11-27 06:38:54.189880908 -0800
 Birth: -

touch foo.txt  # No problems, all times are changed
stat foo.txt

  File: foo.txt
  Size: 0         	Blocks: 0          IO Block: 4096   regular empty file
Device: 802h/2050d	Inode: 6685210     Links: 1
Access: (0666/-rw-rw-rw-)  Uid: (    0/    root)   Gid: ( 1000/jmooring)
Access: 2021-11-27 06:41:43.049344443 -0800
Modify: 2021-11-27 06:41:43.049344443 -0800
Change: 2021-11-27 06:41:43.049344443 -0800
 Birth: -

touch -a foo.txt  # Operation not permitted
touch -m foo.txt  # Operation not permitted

I recommend closing this issue:

It is an OS restriction
The setup (a public folder owned by somebody else) is not common
There is a CLI option to overcome the restriction when needed (--noTimes)

If this isn't going to be fixed, could the error message instead be improved to catch this case and recommend the --noTimes solution?

A simple solution would be to change the ownership of the directory to the ID with which you execute hugo. no further cli parameters required.

Not always possible in a CI environment, unfortunately.

I also ran into this, I want a user to be able to run this without owning the files in question (group access).
I guess chtimes is a user-level permission?
I made a uhugo shortcut that aliases to hugo --noTimes - maybe this shouldn't be a default option (or detect if you have the permissions and warn instead of fail).

@smaudet You can set noTimes = true in the site configuration instead of the command line flag. That might be a bit easier.


Add Arabic, Chinese and Korean fallback font

None of fonts from asses support Arabic, Chinese and Korean properly. So I made a PR to add these fallback fonts.
Then in MaterialUI, OZONE and XMB menu driver, let RetroArch use fallback fonts when these languages are set. See https://github.com/libretro/RetroArch/pull/10699
Result:
Korean fallback font: Sunflower

Chinese fallback font: Droid Sans Fallback

Arabic fallback font: DejaVu Sans

Hi there, what is the status of this PR? Can this be merged?

I saw @RobLoach approved this PR. Maybe @twinaphex you can merge this PR. And I will make a PR for RetroArch to support CJK fallback fonts.

There are still three problems, and solutions for fix:
Android version 1.9.12 Google Play
Devices: Honor V9
Theme: glui

according to my research:

changing assets/pkg/chinese-fallback-font.ttf will fix title bar and title “口 · 口”。

changing assets/ozone/bold.ttf will fix OSD toast character display issue.

changing assets/ozone/regular.ttf will fix game loading OSD title display issue.

Hope it will help.
Currently the font defaults:
assets/pkg/chinese-fallback-font.ttf -> Droid Sans Fallback
assets/ozone/bold -> Inter UI Bold
assets/ozone/regular.ttf -> Inter UI

The font I'm using: Open Harmony Sans Sc.


gateway does not work well with path,e.g. ( "/v1/{name=apps/}"  or "/v1/{app.name=apps/}" ) when keeping the google.api.http principle

it returns wrong from grpc server when matching with path like this ( /v1/{name=apps/*})
(⎈|test-ctx:vault)➜  ~ curl http://<IP_ADDRESS>:15020/v1/apps/test-app
404 page not found

using adjustHttpPath function is wrong

@kevwan pls  fix this ASAP

@kevwan will U fix this issue in next release of v1.5.3?

Would you please give an example code on this issue? And how to reproduce?
We can discuss how to support the feature.

@kevwan
proto like
service AppStore {
  // Gets the specified app.
  rpc GetApp(GetAppRequest) returns (App) {
    option (google.api.http) = {
      get: "/v1/{name=apps/*}"
    };
    option (google.api.method_signature) = "name";
  }

the issue is the Wildcard.

@kevwan
example proto
service AppStore {
  // Gets the specified app.
  rpc GetApp(GetAppRequest) returns (App) {
    option (google.api.http) = {
      get: "/v1/{name=apps/*}"
    };
    option (google.api.method_signature) = "name";
  }

it will register route /v1/name=apps/* ,but should be  /v1/apps/* .

@kevwan  pls pay attention to it


Tests fail when ran in a different timezone

I'm based in the Czech Republic (UTC+2 now in the summer) and I have Engine tests failing because the date formatters are using specific timestamps, which translate to different dates here. Specifically those are:
Failing because of timezone:
SMTPDateTests.testSMTPDate()

Because of getting different response to google.com:
SockStreamTests.testTCPInternetSocket()
SockStreamTests.testFoundationStream()

I get: HTTP/1.0 302 Found\r\nCache-Control: private\r\nContent-Type: text/html; charset=UTF-8\r\nLocation: http://www.google.cz/?gfe_rd=cr&ei=WYG8V_CHBtDR8geml7zYAg\r\nContent-Length: 258\r\nDate: Tue, 23 Aug 2016 17:01:13 GMT\r\n\r\n<meta http-equiv=\"content-type\" content=\"text/html;charset=utf-8\">\n<TITLE>302 Moved</TITLE>\n
302 Moved
\nThe document has moved\nhere.\r\n\r\n
We just need to find a way to ensure the timezone tests work around the world (probably by setting the timezone in tests to a set one), and pick a different URL than google.com which responds the same way to all locations.

Sounds like moving to http://httpbin.org/ will fix, let's integrate this before official release for international contributors.

Yeah, that'll fix the live requests. We also need to fix the timezone failure, which could be achieved by setting a specific time zone on the formatter.

We've mostly moved to HTTP bin now.


style change of component when dynamic changes

Version
2.4.2
Reproduction link
https://jsfiddle.net/oricou/qmnvd7xj/23/
Steps to reproduce
Click on one of the cloud except the last one.
What is expected?
The cloud you click on should change to red and disappear.
What is actually happening?
It turns red and disappear but then the cloud on the next row, takes it place but also turns red. It should not since it has not the right data-test attribute.

Use a key attribute https://vuejs.org/v2/guide/list.html#key

Thanks and sorry for the trouble. I start Javascript, Vue, Django and I must say I have fun but it takes time to understand all details.

No worries, I recommend you to check the Discord server, you can get fast feedback 🙂


ImportError: No module named request

for some reason it won't let me run the example. even if I try to pip install that module it tells me it does not exist.

Hi, it sounds like you might need "requests" with an s?
pip install requests
If that's not the case, feel free to reopen this Issue.


Analyse outputting empty ts/tv file

Hello
As far as I could tell Haystac does not accept fasta files as sample inputs, only fastq. I have some fasta files I'd like to analyse, so I used BBMap reformat.sh and seqtk seq to convert the fasta files to fastqs with dummy quality score values of 40. The sequence headers are of the form @NC_XXXX-seqN/1, where N is the read number and XXXX are numbers belonging to an NCBI taxon. I have successfully built samples with these fastqs, but analysing using --mode reads results in the errors in the attached log files. It appears that the ts_tv count files are all empty. Do you have any guidance as to how to troubleshoot?
Thank you
ts_tv_log_2.txt
ts_tv_log.txt

Also, it appears that bowtie is run with the --ignore-quals flag. Is the read quality information used by the pipeline?

@antonisdim I have resolved this issue. No reads were aligning to taxa other than Dark Matter causing empty ts/tv files to be generated. I am working with fasta files; does Haystac use quality scores for anything other than the bowtie alignment? I am wondering if it is possible to modify the code minimally to support fasta.

Hello Peter,
I hope you are doing great and apologies for the delayed response !
Indeed we only currently support fastq files. I do not think it would be hard to integrate some support for fasta files in a future version of haystac.  Of course I'll keep you updated.
Thank you for your patience !
Best,
Antony

Hi Antony,
Thank you for the response. Does Haystac use the quality scores for abundance calculation or dirichlet read assignment. Based on the paper and code I am leaning towards no, but would like to confirm with you.
Thanks again!

Hello Peter,
No after the first filtering alignment with bowtie2 base quality scores are not considered. So the individual metagenomic alignments (with bowtie2) and the dirichlet read assignment do not use the base quality info, but instead they focus on the edit distance of the reads.
Hope this helps and please let me know if you have any other questions !
Best,
Antony


ParentSpanId is lost on subsequent webclient-calls

As discussed on Gitter there is a problem where the ParentSpanId is lost on subsequent webclient calls using spring-webflux.
I have created a minor sample application where the issue is reproduced together with an investigation.
What we found out from the investigation was that it started to loose the ParentSpanId after commit f6a0e38c7195b9bd9c418195d0c5e22db4e46de6
And only when both spring-boot-starter-web and spring-boot-starter-webflux were included as dependencies. Some more details about the issue can be found in the sample application's README
The reason why we need both spring-boot-starter-web and spring-boot-starter-webflux is due to Springfox lack of spring 5 support

Congrats @goober on finding this :D I'll do my best to fix this ASAP!

Can you double check that the snapshots work fine, please?


[Release-1.26] Confusing log when deploying dualStack

Backport: https://github.com/k3s-io/k3s/issues/8797

Validated on release-1.26 branch with commit d2d63ce3
Environment Details

Infrastructure

[ ] Cloud
[ ] Hosted

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

Linux 5.19.0-1025-aws #26~22.04.1-Ubuntu SMP Mon Apr 24 01:58:15 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
PRETTY_NAME="Ubuntu 22.04.2 LTS"
VERSION_ID="22.04"

Cluster Configuration:

1 server

Config.yaml:

write-kubeconfig-mode: "0644"
tls-san:
  - fake.fqdn.value
cluster-init: true
node-name: ip-xxx-xxx-xx-244.us-west-1.compute.internal
cluster-cidr: 2001:cafe:42:0::/56,<IP_ADDRESS>/16
service-cidr: 2001:cafe:43:0::/112,<IP_ADDRESS>/16
token: <token>
node-ip: <IPv6>,<PRIVATE IPv4>

Testing Steps

Copy config.yaml

$ sudo mkdir -p /etc/rancher/k3s && sudo cp config.yaml /etc/rancher/k3s

Install k3s
Check logs for the info
Should show both ipv4 and ipv6 IPs in the log in case of dual-stack

Replication Results:

k3s version used for replication:

k3s version v1.26.10+k3s2 (cb5cb555)
go version go1.20.10

$ sudo journalctl -u k3s | grep -i interface
Nov 21 18:18:17 ip-xxx-xxx-xx-244 k3s[28415]: time="2023-11-21T18:18:17Z" level=info msg="The interface ens5 with ipv4 address <PRIVATE IPv4> will be used by flannel"

Validation Results:

k3s version used for validation:

k3s version v1.26.10+k3s-d2d63ce3 (d2d63ce3)
go version go1.20.10

$ sudo journalctl -u k3s | grep -i interface
Nov 21 19:07:06 ip-xxx-xxx-xx-244 k3s[34317]: time="2023-11-21T19:07:06Z" level=info msg="Using dual-stack mode. The interface ens5 with ipv4 address <PRIVATE IPv4> and ipv6 address <IPv6> will be used by flannel"


Suggestion: Line wrap .method(... the same as operators like +

I have a suggestion - not sure if this is the best place for it, but here goes:
Consider the following input file:
def foo():
    return my_first_really_long_variable_name + my_other_really_long_variable_name_extra_long
def bar():
    return "foo bar qux alp this is a lot of {} in a long {}".format(something, otherthing)

Currently, black -l79 makes the following choices:
def foo():
    return (
        my_first_really_long_variable_name
        + my_other_really_long_variable_name_extra_long
    )

def bar():
    return "foo bar qux alp this is a lot of {} in a long {}".format(
        something, otherthing
    )

I propose that it instead do the following:
def bar():
    return (
        "foo bar qux alp this is a lot of {} in a long {}"
        .format(something, otherthing)
    )

I think that this has several benefits, and I think all the arguments in favor of line breaking in front of an operator like + instead of after apply here.
When I'm looking over code, I find that the former is more readable because it puts the relationship between the things clearer keeps my eyes from having to do as long of a sweep right followed by the eye-movement equivalent of a carriage return.
(This is only a subtly annoying movement when we're used to tuning it out, but it is less ergonomic for the eyes in general, and is also more acutely felt by people with myopia - the most common cause of myopia is an elongation of the eyeball/eyesocket in one directly, which can be physically felt the more severe it gets, as a slight pressure/discomfort sensation that "sweeps" across somewhere inside one's head the further one's eyes swivel in their eye sockets.)
In general I find that this leads to less lines being "jaggedly" outside of the comfortable circle of visual field focus/detail that it is most ergonomic for the eye to stay within for each movement while scanning.
This is especially prominent when going over code quickly - skimming/scanning to refresh my memory or find something - because in those cases I don't have to do a rigorous, meticulous, character-by-character parse.
TL;DR: Having the method call at the start of the line puts the operation, rather than the operands, at the beginning of the visual line, which I've found is the most useful piece of information to have first, and in my experience leads to more visually ergonomic code.

To be clear, I am happy to spend the time to make and then polish up a pull request to implement this, if that is desired.

I realize that this is contrary to previously stated reasoning, but at least in my experience, this trade-off isn't actually as bad as it might initially seem. Some years ago, I felt the same way, but gradually optimizing for readability led me here.
Nowadays, given the need to manually line-break and the choice of how, this is how I do it:
my_thing.some_other_thing
.some_operation(arg1, arg2)

and I think it looks weird because we're not used to it, but conceptually/logically/functionally, how is that different from this?
    my_thing.some_other_thing
    + some_expression

Sorry for the late reply here. Currently your suggested change would make too many changes in projects that already adopted Black and we will not be able to justify this. This is especially true given the discussion on #67. I guess, you can think about it like this: having a single method call detached from the object looks unnatural. While Black is doing its best to keep your line length as you specify it, it does not consider line length trumping all the other rules of formatting it applies.


[Improvement] Remove “frames” title from the profile

before
after

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


JAVA: Introduction to Auto type promotion when overloading

JAVA: Introduction to Auto type promotion when overloading
JAVA: Introduction to Auto type promotion when overloading
Proposed article introduction
Type promotion is done while doing method overloading in Java. When the datatype is not the same then we promote one datatype to another datatype. We cannot de-promote one datatype to another datatype. when a class has multiple methods with the same name but have different parameters, it is known as method overloading and is a very common scene in java programming.
In this article I will be discussing a brief introduction to auto type promotion and method overloading and their relationships, reasons or circumstances under which you may be required to apply automatic type promotion and the different types or ways of implementing type promotion. Finally I will go ahead and outline come illustratory code examples just to clearly explain the concept in question.
Key takeaways

The reader should be able to understand what Auto  type promotion is with respect to java programming.
The reader should be able to understand what method overloading is in java programming.
The reader should be able to understand the circumstances under which auto type promotion is imminent and how it id implemented.
The reader should be able to know the different types of Auto type promotions that can be implemented in java programming.
The reader will be able to understand  the rules required to achieve auto type promotion in java programming.

Article quality
This article will provide an in-depth tutorial on how automatic type promotion works and how to implement it in java. Unlike other articles, this article will show the rules required for auto type promotion to be achieved. In addition, it will show illustratory code examples clearly implementing the concept in question.
References
Please list links to any published content/research that you intend to use to support/guide this article.
Conclusion
Finally, remove the Pre-Submission advice section and all our blockquoted notes as you fill in the form before you submit.
We look forwarding to reviewing your topic suggestion.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Sound like a good topic @root5086. Please make sure your article is unique and adds to what is available online.


24H2

Description
hello
tnx for this great tool
if it possible we need new update for windows 11 24h2
Reason
because it's one of the best update for windows in this year
tnx  a lot
Examples
No response

Yes, please update


don't propagate escape infos when using field to get a Integer

This PR make the following changes:

getfield will not propagate escape information to its argument (field source) when the field it is getting is of integer type, even if itself is escaped. The reason is that getting an integer field should not affect the escape state of the source, in some sense it is pretty similar to sizeof.
Furthermore, for tuple and also other calls (the else branch), we don't propagate the current statement's escape info to the arg if the arg itself is of integer type. The reason is that using a computed integer shouldn't create escape connections between statements. Integer typed statement should be an endpoint for an escape chain.

Notice this PR wouldn't solve #9 unless we handle foreigncall.
This method can theoretically be applied to any primitive type including integer, float, and char.

IR diff of the eltype case:
diff --git a/getsize.txt b/newgetsize.txt
index e7ae0a9..c101e32 100644
--- a/getsize.txt
+++ b/newgetsize.txt
@@ -1,4 +1,4 @@
- typeof(getsize)(_2::Vector{Int64} →, _3::Union{Nothing, Tuple{Int64, Int64}} ↓, _4::Vector{Int64} ↓, _5::Int64 ↓)
+typeof(getsize)(_2::Vector{Int64} →, _3::Union{Nothing, Tuple{Int64, Int64}} ↓, _4::Vector{Int64} ↓, _5::Int64 ↓)
  1 → 1 ── %1  = $(Expr(:foreigncall, :(:jl_alloc_array_1d), Vector{Int64}, svec(Any, Int64), 0, :(:ccall), Vector{Int64}, 0, 0))::Vector{Int64}
  2 ◌ │    %2  = Base.bitcast(UInt64, 1)::UInt64
  3 ◌ │    %3  = Base.sub_int(%2, 0x0000000000000001)::UInt64
@@ -8,19 +8,19 @@
  7 ◌ │    %7  = Base.ult_int(%3, %6)::Bool
  8 ◌ │    %8  = Base.and_int(%5, %7)::Bool
  9 ◌ └───       goto #3 if not %8
-10 → 2 ── %10 = Base.arrayref(false, _2, 1)::Int64
-11 → │    %11 = Base.add_int(1, 1)::Int64
-12 → │    %12 = Core.tuple(%10, %11)::Tuple{Int64, Int64}
+10 ↓ 2 ── %10 = Base.arrayref(false, _2, 1)::Int64
+11 ↓ │    %11 = Base.add_int(1, 1)::Int64
+12 ◌ │    %12 = Core.tuple(%10, %11)::Tuple{Int64, Int64}
 13 ◌ └───       goto #4
-14 → 3 ── %14 = Base.nothing::Nothing
+14 ◌ 3 ── %14 = Base.nothing::Nothing
 15 ◌ └───       goto #4
-16 → 4 ┄─ %16 = φ (#2 => %12, #3 => %14)::Union{Nothing, Tuple{Int64, Int64}}
+16 ◌ 4 ┄─ %16 = φ (#2 => %12, #3 => %14)::Union{Nothing, Tuple{Int64, Int64}}
 17 ◌ └───       goto #5
 18 ◌ 5 ── %18 = (%16 === nothing)::Bool
 19 ◌ │    %19 = Base.not_int(%18)::Bool
 20 ◌ └───       goto #20 if not %19
-21 → 6 ┄─ %21 = φ (#5 => %16, #19 => %58)::Union{Nothing, Tuple{Int64, Int64}}
-22 → │    %22 = π (%21, Tuple{Int64, Int64})
+21 ◌ 6 ┄─ %21 = φ (#5 => %16, #19 => %58)::Union{Nothing, Tuple{Int64, Int64}}
+22 ◌ │    %22 = π (%21, Tuple{Int64, Int64})
 23 → │    %23 = Core.getfield(%22, 1)::Int64
 24 → │    %24 = Core.getfield(%22, 2)::Int64
 25 ◌ │    %25 = Core.lshr_int(1, 63)::Int64
@@ -37,9 +37,9 @@
 36 ◌ 12 ─       goto #13
 37 ◌ 13 ─       $(Expr(:foreigncall, :(:jl_array_grow_end), Nothing, svec(Any, UInt64), 0, :(:ccall), :(%1), :(%32), :(%32)))::Nothing
 38 ◌ └───       goto #14
-39 → 14 ─ %39 = Base.arraysize(%1, 1)::Int64
+39 ◌ 14 ─ %39 = Base.arraysize(%1, 1)::Int64
 40 ◌ │    %40 = Base.slt_int(%39, 0)::Bool
-41 → │    %41 = Base.ifelse(%40, 0, %39)::Int64
+41 ◌ │    %41 = Base.ifelse(%40, 0, %39)::Int64
 42 ◌ │          Base.arrayset(false, %1, %23, %41)::Vector{Int64}
 43 ◌ └───       goto #15
 44 ◌ 15 ─ %44 = Base.bitcast(UInt64, %24)::UInt64
@@ -50,13 +50,13 @@
 49 ◌ │    %49 = Base.ult_int(%45, %48)::Bool
 50 ◌ │    %50 = Base.and_int(%47, %49)::Bool
 51 ◌ └───       goto #17 if not %50
-52 → 16 ─ %52 = Base.arrayref(false, _2, %24)::Int64
-53 → │    %53 = Base.add_int(%24, 1)::Int64
-54 → │    %54 = Core.tuple(%52, %53)::Tuple{Int64, Int64}
+52 ↓ 16 ─ %52 = Base.arrayref(false, _2, %24)::Int64
+53 ↓ │    %53 = Base.add_int(%24, 1)::Int64
+54 ◌ │    %54 = Core.tuple(%52, %53)::Tuple{Int64, Int64}
 55 ◌ └───       goto #18
-56 → 17 ─ %56 = Base.nothing::Core.Const(nothing)
+56 ◌ 17 ─ %56 = Base.nothing::Core.Const(nothing)
 57 ◌ └───       goto #18
-58 → 18 ┄ %58 = φ (#16 => %54, #17 => %56)::Union{Nothing, Tuple{Int64, Int64}}
+58 ◌ 18 ┄ %58 = φ (#16 => %54, #17 => %56)::Union{Nothing, Tuple{Int64, Int64}}
 59 ◌ │    %59 = (%58 === nothing)::Bool
 60 ◌ │    %60 = Base.not_int(%59)::Bool
 61 ◌ └───       goto #20 if not %60

oh, conflicts... Let me resolve that after meeting, if necessary.

Can you add appropriate tests that are supposed to be improved by these changes ?
Braking up a problem into as small pieces as possible and have unit tests against them are the general strategy to develop and maintain a software.
(And please note that improvements on escape information on intermediate calls are not our interest -- we're interested in escape information on arguments and allocations).
I come up with the one example below that can be improved by this PR:
julia> mutable struct A
           a::Int
       end

julia> analyze_escapes((Int,)) do a
           o = A(a) # no need to escape
           f = getfield(o, :a)
           return f
       end
var"#5#6"(_2::Int64 ↑, _3::Int64 ↓, _4::A ↓)
1 ↑ 1 ─ %1 = %new(Main.A, _2)::A
2 ↑ │   %2 = Main.getfield(%1, :a)::Int64
3 ◌ └──      return %2

But the problem here is that getfield doesn't need to escape the first argument.
The reason why getfield propagate escape information of the first argument right now is that we don't have alias analysis yet, e.g.: we need to escape r0 in the example below
julia> mutable struct MyRef{T}
           x::T
       end

julia> analyze_escapes() do
           r0 = MyRef(0)
           r = MyRef(r0)
           return r.x
       end
var"#7#8"(_2::MyRef{MyRef{Int64}} ↓, _3::MyRef{Int64} ↓)
1 ↑ 1 ─ %1 = %new(MyRef{Int64}, 0)::MyRef{Int64}
2 ↑ │   %2 = %new(MyRef{MyRef{Int64}}, %1)::MyRef{MyRef{Int64}}
3 ↑ │   %3 = Base.getfield(%2, :x)::MyRef{Int64}
4 ◌ └──      return %3

For that example, it sounds quite similar to our standard lattice PartialStruct element (can probably even use the existing type?). It is correct to say "non-escaping", since the new call (and getfield call) there is semantically copying for isbitstype objects. The test T<:Integer would also include mutable struct BigInt, which does escape through there, so you want to look at the struct properties (isimmutabletype), not subtyping.

For that example, it sounds quite similar to our standard lattice PartialStruct element (can probably even use the existing type?). It is correct to say "non-escaping", since the new call (and getfield call) there is semantically copying for isbitstype objects. The test T<:Integer would also include mutable struct BigInt, which does escape through there, so you want to look at the struct properties (isimmutabletype), not subtyping.

That's what I intended to implement, by primitive type I want to include these immutable types.

Yes, however, each of those terms has a specific meaning already in Julia (with corresponding predicate):
isa Integer : declared subtype of Integer <: Real <: Number <: Any, could have any structure or be abstract
isprimitivetype : declared with primitive type T nbits end, a sized object without fields (complement of isstructtype)
isbitstype : composed only of primitive types (recursively) and not recursively defined, akin to pointerfree
Base.datatype_pointerfree : similar to isbitstype, but also includes any mutable object whose fields are all isbitstype, and therefore also less general

Thanks for the detailed comment and the testcase! I thought the eltype case could be a testcase for this but now I think it's not that obvious. The second example let me understand why we treat getfield in this way, and I don't think it's conflict with this PR, as long as r.x is an immutable type.

(And please note that improvements on escape information on intermediate calls are not our interest -- we're interested in escape information on arguments and allocations).
I agree, but I think this change will improve how we analyze the intermediate calls and values, the more precise intermediate states are, the more precise allocations and arguments will be. In my diff above, if this PR isn't applied, escapes will appear very frequently and therefore make it hard to see it's really the foreigncall leads to marking a as an escape.

The failing two test_nodispatch tests are:

using widenconst isbitstype and update_non_bitstype_changes! in escape_buitlin!(::typeof(getfield))
using update_non_bitstype_changes in escaped_builtin!(::typeof(tuple))

Code looks good to me.
I'm wondering it might be enough to add this barrier where we introduce "new" escape information (e.g. uninferred callsite, return site), and leave other places untouched where we just propagate information introduced elsewhere.
So we don't change the current implementation of getfield. In other word, I think the test case I showed above should be improved by incoming alias analysis, not by this PR.

should we revert the change to getfield then?

should we revert the change to getfield then?

Or just leave a comment saying that we can remove that part once we have implemented field analysis ?

should we revert the change to getfield then?

Or just leave a comment saying that we can remove that part once we have implemented field analysis ?

Sure!

I added some changes in 7a8e3f980210992e4d3d6f6f9343b216ea5d8b72, and I'd like to merge this if you're okay with it.

I added some changes in 366cbb9, and I'd like to merge this if you're okay with it.

I took a look and thanks for fixing the other two cases. Let's merge this.


IOptions doesn't just magically work as written in /fundamentals/configuration.rst

Please give the fully qualified name for IOptions or mention adding the using statement in /fundamentals/configuration.rst.
using Microsoft.Framework.OptionsModel;

Missing name spaces in documentation and tutorials is a general beef of mine.

We should fix this as part of the update to RC1. Should just need to change this:
.. literalinclude:: configuration/sample/src/UsingOptions/Controllers/HomeController.cs
:linenos:
:language: c#
:lines: 11-24
:dedent: 4
to
.. literalinclude:: configuration/sample/src/UsingOptions/Controllers/HomeController.cs
:linenos:
:language: c#
:lines: 1-24
:dedent: 4
and perhaps emphasize the using statement.

We don't want to add using statements for all code snippets. The code snippets come from sample apps that are included in the docs repo that you can look at. We could look into making it easier though to jump to the code from the snippet.


How to set\pass #id_token received from custom B2C policy and setup MSAL angular from it?

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[X] Other... Please describe:

This is more of a question: I already get and #id_token from my custom B2C Sign Up policy. However this policy is called from a link in an e-mail, not from the Angular SPA app.
With this in mind, the Angular JS app doesn't have a callback to process this #id_token.
Is there any way to set the #id_token in the msalService in Angular JS so that the user is logged in automatically?
This id_token from the sign up policy already has all the user info I need to log them in into the application. I wouldn't like to redirect them to the login page. This is an unnecessary step.
I'd like for something like this function:
UserAgentApplication.prototype.processCallBack = function (hash) {

or
UserAgentApplication.prototype.handleAuthenticationResponse = function (hash) {

@leniel  correct me if I am wrong, but are you saying that the user is already logged in at this point, you just want a way to make sure msal has context of it by passing in the id token?

Exactly that Daryl... after the sign up plociy finished, that is, the user
clicked the Create button (the user) is added to Azure AD and so the policy
returns the #id_token with the claims like email, user name, policy name
(acr), etc.
I just need a way of making msal aware of this #id_token and create it's
context based on this #id_token.
Is this possible? If yes, this would be awesome in our use case because
that means the user won't need to be redirected to the login page. We
already know the use successfully registered (with his selected password)
given the #id_token at hand.
On Thu, 25 Jul 2019 at 20:15 Daryl Thayil<EMAIL_ADDRESS>wrote:

@leniel https://github.com/leniel correct me if I am wrong, but are you
saying that the user is already logged in at this point, you just want a
way to make sure msal has context of it by passing in the id token?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/866?email_source=notifications&email_token=AAF2XNVU6ZD6QV7EF36ZGHDQBHNTXA5CNFSM4IGNJP6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD22EOOI#issuecomment-515131193,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAF2XNVDB7BM24OU724QSETQBHNTXANCNFSM4IGNJP6A
.

--
Leniel Maccaferri
+55 24 98826-3578 https://wa.me/5524988263578 mobile\WhatsApp
Skype: lenielmacaferi
Blog:
http://leniel.net
Software Engineer
http://stackoverflow.com/users/114029/leniel-macaferi

User can pass in a login_hint in the request and msal js can build that context. Will that work for you @leniel? Please share the msal js version you are currently working with, I can reach out with the specifics.

Hi Sameera... I'm using the latest version v1.1.1.
I'm also using msal-angularjs wrapper (published 1 year ago):
https://www.npmjs.com/package/@azure/msal-angularjs
If you can point me in the right direction that would be awesome.
On Thu, 25 Jul 2019 at 20:59 Sameera Gajjarapu<EMAIL_ADDRESS>wrote:

User can pass in a login_hint in the request and msal js can build that
context. Will that work for you @leniel https://github.com/leniel?
Please share the msal js version you are currently working with, I can
reach out with the specifics.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/866?email_source=notifications&email_token=AAF2XNSETZJEWWFHDPUJ2ILQBHSYVA5CNFSM4IGNJP6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD22JB3Y#issuecomment-515150063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAF2XNV7C75KYBDLXXUE37TQBHSYVANCNFSM4IGNJP6A
.

--
Leniel Maccaferri
+55 24 98826-3578 https://wa.me/5524988263578 mobile\WhatsApp
Skype: lenielmacaferi
Blog:
http://leniel.net
Software Engineer
http://stackoverflow.com/users/114029/leniel-macaferi

Hey guys, can you provide any update on this? I'm not sure how to pass this login_hint that @sameerag mentioned above.
The user finished the sign up B2C policy journey and then control is sent back to my application.
Right now I have a Web API controller with this method signature:
/// <summary>
/// Signs Up the User to the application after they successfully fill out the Sign Up Invitation form on Azure.
/// </summary>
/// <param name="id_token">The #id_token sent back from Azure policy.</param>
[Route("signup", Name = "SignUp")]
[HttpGet]
public HttpResponseMessage SignUp(string id_token)
.
.
.

This method is hit because I have defined it as a Reply URL in Azure Portal. It's the redirect URI for the SignUp policy.
See that I have the #id_token returned from the policy as a parameter. From this SignUp method I'm redirecting back to my AngularJS application.
Is there any way to create the context (in the C# code) and pass it to MSAL so that when I redirect from this Web API method the user is already logged in?
Right now I'm just redirecting to the a custom route I've setup in AngularJS like this:
// Redirect User to the signup route in the AngularJS app...
Uri redirectLocation = new Uri(string.Format("/#/signup?id_token={0}", Uri.EscapeDataString(id_token)), UriKind.Relative);

var response = Request.CreateResponse(HttpStatusCode.Moved);
response.Headers.Location = redirectLocation;

return response;

so the id token has some claims in it, SID (session id) is one of them but there are a few others.
https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/dev/lib/msal-core/src/AuthenticationParameters.ts
I think in the loginHint you can also use the users email, is that correct @sameerag

Not really...
If it's not asking too much :| could you please somehow show us a sample of
this working?
Right now I'm redirecting to the login page but it's not the ideal solution
since the user has the id_token already.
On Wed, 28 Aug 2019 at 14:26 Daryl Thayil<EMAIL_ADDRESS>wrote:

@leniel https://github.com/leniel were you able to try this
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/866?email_source=notifications&email_token=AAF2XNRNKT5FUO36NIKZT63QG2YNFA5CNFSM4IGNJP6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5L3RRQ#issuecomment-525842630,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAF2XNSXZUD3QX3RFDQ6GY3QG2YNFANCNFSM4IGNJP6A
.

--
Leniel Maccaferri
+55 24 98826-3578 https://wa.me/5524988263578 mobile\WhatsApp
Skype: lenielmacaferi
Blog:
http://leniel.net
Software Engineer
http://stackoverflow.com/users/114029/leniel-macaferi

@negoe  can you share an example of passing in login_hint?

@leniel One thing that confused me: msal 1.1.1 is not compatible with Angular wrapper we have. Did you guys make any local modifications for the same?
Regarding how to use msal 1.1.1 and send a login hint, please set it in the request parameters, source code linked to the msal 1.1.1 version here
const tokenRequest = {
scopes: ["Mail.Read"],
loginHint<EMAIL_ADDRESS>};
acquireTokenSilent(tokenRequest);

@sameerag I'm using 1.1.2 right now. It's working for what we need. However I did not try your sample code.
I'm using msal-angular from here: https://www.npmjs.com/package/@azure/msal-angular
Thanks for taking the time to provide the sample code.

@sameerag
Can you please help on how to achieve the same for msal-angular. We need to have roles in the jwt token that is attached, when we make api calls mentioned in protectedResourceMap.
The sample works well when using msal library for js

@maheshkumagali Our angular wrapper is currently in progress of being upgraded to be compatible with our msal-core library. @jasonnutter is working on this and will be able to provide the details.
@jasonnutter This could be a use case we should address with our angular work.


Modified emv yaml with correct xarray version.

Fixed xarray to 2022.3.0

I don't want to accept this pull request since xarray is a dependency of RASCIL (see https://github.com/i4Ds/rascil-build/blob/main/conda/meta.yaml) and it should be made there. Therefore I suggest to reject this request.


fix(mps-legacy-sync-plugin): use legacy plugin parts

This ensures that persisted binding configuration is loaded when a project is opened.
Also, minor bugs where fixed that were discovered in tests during initialization and dispose of project plugin parts.

:tada: This PR is included in version 0.5.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add debian based

Adds installation for:

ubuntu/trusty
ubuntu/yakkety
ubuntu/xenial
debian/wheezy
debian/jessie
debian/stretch

cc/ @andrewhsu


Can't browser the cluster from web portal after upgrading to v2.6

Rancher Server Setup

Rancher version: v2.6.0
Installation option (Docker install/Helm Chart):

Helm: "v3.4.2"
Kubernetes: v1.16.2 (RKE1)

Proxy/Cert Details: Let's Encrypted certificate

Describe the bug
After upgrading Rancher from v2.5.5 to v2.6.0, I'm no longer access my custom cluster from the new Rancher portal.
I got the following screen when I click the name of my existing clusters.

I have tried the following ways but none of them works

Clean the cache
Use incognito windows

The worst part is the local cluster is gone after the upgrade, I can see three clusters in my previous v2.5.9 version, one for the local cluster(RKE for Rancher), two for my custom clusters, but the local cluster is disappeared from the portal.

To Reproduce
Prepare Rancher v2.5.5 cluster and then use the Helm (v3.4.2) to upgrade it to v2.6.0

Result
Expected Result

Can successfully browser existing cluster with new Rancher UI.
Screenshots

Additional context
Log message from Rancher Pod
2021/09/02 16:57:22 [ERROR] failed to call leader func: failed to add management data: namespaces "c-8b6tz" not found
2021/09/02 16:57:22 [ERROR] error syncing 'local': handler kubernetes-provider: Operation cannot be fulfilled on clusters.management.cattle.io "local": StorageError: invalid object, Code: 4, Key: /registry/management.cattle.io/clus
ters/local, ResourceVersion: 0, AdditionalErrorMsg: Precondition failed: UID in precondition: 5c9e250b-ec68-4911-9a46-3ecabc644c4d, UID in object meta: , handler cluster-watch: clusterregistrationtokens.management.cattle.io "defaul
t-token" is forbidden: unable to create new content in namespace local because it is being terminated, handler mgmt-cluster-remove: Operation cannot be fulfilled on clusters.management.cattle.io "local": StorageError: invalid objec
t, Code: 4, Key: /registry/management.cattle.io/clusters/local, ResourceVersion: 0, AdditionalErrorMsg: Precondition failed: UID in precondition: 5c9e250b-ec68-4911-9a46-3ecabc644c4d, UID in object meta: , requeuing
2021/09/02 16:57:31 [ERROR] failed to call leader func: failed to add management data: namespaces "c-8b6tz" not found
2021/09/02 16:57:37 [ERROR] failed to call leader func: failed to add management data: namespaces "c-8b6tz" not found
2021/09/02 16:57:44 [ERROR] failed to call leader func: failed to add management data: namespaces "c-8b6tz" not found
2021/09/02 16:57:49 [ERROR] failed to call leader func: failed to add management data: namespaces "c-8b6tz" not found
2021/09/02 16:57:50 [ERROR] error syncing 'fleet-local/local': handler workspace-backport: fleetworkspaces.management.cattle.io "fleet-local" not found, requeuing
2021/09/02 16:57:55 [ERROR] failed to call leader func: failed to add management data: namespaces "c-8b6tz" not found
2021/09/02 16:57:59 [ERROR] Failed to handling tunnel request from <IP_ADDRESS>:56624: response 400: cluster not found
2021/09/02 16:57:59 [INFO] Active TLS secret serving-cert (ver=310839050) (count 23): map[listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:10.42.17
1.81 listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>
0:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn
-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-
<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/cn-localhost:localhost listener.cattle.io/cn-rancher.cattle-system:rancher.cattle-system listener.cattle.io/cn-testing.aether.onlab.us:testing.aet
her.onlab.us listener.cattle.io/fingerprint:SHA1=2DC3E71D4977C36BFB417E0AC4648C6385865D9B listener.cattle.io/hash:a5639694794df315a4c33e4bafc20a2c1e613111e84b586fa75eb2a8f3a4cf4d]
2021/09/02 16:57:59 [INFO] Active TLS secret tls-rancher-internal (ver=310839051) (count 1): map[listener.cattle.io/cn-<IP_ADDRESS>:<IP_ADDRESS> listener.cattle.io/fingerprint:SHA1=6124651E78FCBFBF00304CFE0EC22301A08BE7F9]
2021/09/02 16:58:01 [ERROR] failed to call leader func: failed to add management data: namespaces "c-8b6tz" not found
2021/09/02 16:58:04 [INFO] error in remotedialer server [400]: read tcp <IP_ADDRESS>:443-><IP_ADDRESS>:55172: use of closed network connection
W0902 16:58:04.612419      32 reflector.go:437] pkg/mod/github.com/rancher/client-go@v0.21.0-rancher.1/tools/cache/reflector.go:168: watch of *v1.RoleBinding ended with: an error on the server ("unable to decode an event from the w
atch stream: tunnel disconnect") has prevented the request from succeeding
W0902 16:58:04.612462      32 reflector.go:437] pkg/mod/github.com/rancher/client-go@v0.21.0-rancher.1/tools/cache/reflector.go:168: watch of *v1.Pod ended with: an error on the server ("unable to decode an event from the watch str
eam: tunnel disconnect") has prevented the request from succeeding
W0902 16:58:04.612463      32 reflector.go:437] pkg/mod/github.com/rancher/client-go@v0.21.0-rancher.1/tools/cache/reflector.go:168: watch of *v1.ClusterRole ended with: an error on the server ("unable to decode an event from the w
atch stream: tunnel disconnect") has prevented the request from succeeding
W0902 16:58:04.612482      32 reflector.go:437] pkg/mod/github.com/rancher/client-go@v0.21.0-rancher.1/tools/cache/reflector.go:168: watch of *v1.Endpoints ended with: an error on the server ("unable to decode an event from the wat
ch stream: tunnel disconnect") has prevented the request from succeeding
W0902 16:58:04.612493      32 reflector.go:437] pkg/mod/github.com/rancher/client-go@v0.21.0-rancher.1/tools/cache/reflector.go:168: watch of *v1.ClusterRoleBinding ended with: an error on the server ("unable to decode an event fro
m the watch stream: tunnel disconnect") has prevented the request from succeeding
W0902 16:58:04.612501      32 reflector.go:437] pkg/mod/github.com/rancher/client-go@v0.21.0-rancher.1/tools/cache/reflector.go:168: watch of *v1.Service ended with: an error on the server ("unable to decode an event from the watch
 stream: tunnel disconnect") has prevented the request from succeeding
W0902 16:58:04.612552      32 reflector.go:437] pkg/mod/github.com/rancher/client-go@v0.21.0-rancher.1/tools/cache/reflector.go:168: watch of *v1.Secret ended with: an error on the server ("unable to decode an event from the watch
stream: tunnel disconnect") has prevented the request from succeeding
W0902 16:58:04.612559      32 reflector.go:437] pkg/mod/github.com/rancher/client-go@v0.21.0-rancher.1/tools/cache/reflector.go:168: watch of *v1.Role ended with: an error on the server ("unable to decode an event from the watch st
ream: tunnel disconnect") has prevented the request from succeeding
W0902 16:58:04.612593      32 reflector.go:437] pkg/mod/github.com/rancher/client-go@v0.21.0-rancher.1/tools/cache/reflector.go:168: watch of *v1.Namespace ended with: an error on the server ("unable to decode an event from the wat
ch stream: tunnel disconnect") has prevented the request from succeeding
W0902 16:58:04.612599      32 reflector.go:437] pkg/mod/github.com/rancher/client-go@v0.21.0-rancher.1/tools/cache/reflector.go:168: watch of *v1.APIService ended with: an error on the server ("unable to decode an event from the wa
tch stream: tunnel disconnect") has prevented the request from succeeding
W0902 16:58:04.612653      32 reflector.go:437] pkg/mod/github.com/rancher/client-go@v0.21.0-rancher.1/tools/cache/reflector.go:168: watch of *v1.ServiceAccount ended with: an error on the server ("unable to decode an event from th
e watch stream: tunnel disconnect") has prevented the request from succeeding
2021/09/02 16:58:07 [ERROR] failed to call leader func: failed to add management data: namespaces "c-8b6tz" not found
2021/09/02 16:58:09 [INFO] Handling backend connection request [<IP_ADDRESS>]
2021/09/02 16:58:09 [ERROR] Failed to handling tunnel request from <IP_ADDRESS>:56730: response 400: cluster not found
2021/09/02 16:58:12 [ERROR] failed to call leader func: failed to add management data: namespaces "c-8b6tz" not found
2021/09/02 16:58:14 [ERROR] Failed to handling tunnel request from <IP_ADDRESS>:44528: response 400: cluster not found

Experimenting a similar issue, imported a custom cluster but cannot open it in the explorer (it appears with a gray logo instead of blue at the navbar). Looking at the logs I have several errors like this one:
[ERROR] error syncing 'cattle-system': handler resourceQuotaSyncController: projects.management.cattle.io "c-m-6tf7kztp/p-jnxlr" not found, handler resourceQuotaUsedLimitController: projects.management.cattle.io "c-m-6tf7kztp/p-jnxlr" not found, requeuing

Referring to cattle-system, kube-system...

I have the same problem. I see all imported clusters, as Available (EKS 1.21), but none are clickable to get to the resources.
If i go to the Cluster Manager, i can click on the downstream cluster and can see the nodes but that's it.
It also shows the Registration tab with the kubectl apply -f xxx command just when to register the downstream cluster when it's not registered yet. I tried re-running the command from the downstream cluster, but it's not fixing the problem.


Support for MIME headers when using NewEmailFromReader

This PR resolves #139 - Missing MIME headers with NewEmailFromReader.
The issue once again:
In the latest master version (943e75fe5223047b345337ea74d5951d19d229ed) when e-mails are created with NewEmailFromReader and they contain attachments, the attachments will have empty MIME headers. This is due to commit b84218f6af87ca49ed7465d40545587bd94b379f on Dec 30, 2020 which resolved a different issue. The MIME headers were never covered by a Unit test, so this was not picked up.
The old removed implementation however was also buggy as it set the Content-Id header of the attachment always to the file name. This is incorrect behavior as Content-Id can have a different value and should be read directly from the MIME header.
This PR:
Resolves the issue by attaching the original MIME headers also to the Attachment object. The parsed values will thus be available separately as struct fields but the raw values will be available as well.
In order to maintain API compatibility a new variation of the Attach() function was also created named AttachWithHeaders() which takes an extra argument. It would have been possible to add the headers directly in the NewEmailFromReader() function and skip this new function but I thought it would be better to make it explicit in the API as well that the original function does not add headers. If this new function is too much clutter, I will be happy to amend the PR and add the headers directly in the NewEmailFromReader() function.
Tests have been created to cover the Content-Id header of attachments

Hello @tarmo-randma
Having support for Content-ID would be awesome.
However, I am not sure requiring attachment to have both inline AND filename set is correct (https://github.com/jordan-wright/email/pull/140/commits/a5c5e9b85911c0010cd7b3804beb50e4fc15ac9b).
Some emails just have Content-Disposition: inline + Content-Id. Here is an example found in the wild:
Content-Disposition: inline
Content-Id: <12562.2245409347447>
Content-Transfer-Encoding: base64
Content-Type: image/gif

R0lGODlhFAAUAIAAAAAAAAAAACH5BAEAAAAALAAAAAAUABQAAAIRhI+py+0Po5y02ouz3rz7rxUA
Ow==```

What do you think ?

@leucos I agree, this could go both ways: the library could actually skip adding filename when at.HTMLRelated is set, just Content-ID should be sufficient.

I will try to make another pass at the PR. Have been busy lately. I am interested of handling the ContentID correctly for my own project as well.

The build is failing but I think this is due to the missing go.mod file not my changes? Could you please take look?
As for the changes I did in relation to content-id:

In NewEmailFromReader inline attachments are now created if content-id is also defined. File name is no longer checked. The content-id check for inline attachments had to be kept due to the test TestMultipartNoContentType which wants to treat inline attachments as bodies. I read through some specs I thought applied but found no corresponding requirements. Still.. just to be sure this functionality will remain unaffected
Tried to also follow @bosim advice and made sure that filename is not set in setDefaultHeaders when at.HTMLRelated is set.
Updated TestAttachmentEmailFromReader and TestEmailWithHTMLAttachments to test these changes

I believe the build error is due to using Go 1.16 instead of 1.15? The change seems to have been really recent - this PR from just 2 days ago still uses 1.15:
https://github.com/jordan-wright/email/pull/143/checks?check_run_id=2106134138
See Go 1.16 release notes: https://golang.org/doc/go1.16. I think the first part about modules applies:

Module-aware mode is enabled by default, regardless of whether a go.mod file is present in the current working directory or a parent directory. More precisely, the GO111MODULE environment variable now defaults to on. To switch to the previous behavior, set GO111MODULE to auto.

Might be something else but for me this looks like the prime suspect.

@tarmo-randma yes you're probably right. PR #143 passes but the author added a go.mod for this it seems:
https://github.com/jordan-wright/email/pull/143/files#diff-33ef32bf6c23acb95f5902d7097b7a1d5128ca061167ec0716715b0b9eeaa5f6

How would you like to handle this? Are you going to merge that go.mod? Proper module support would be great but I am not sure how adding it would impact compatibility for everyone. Maybe it should include a major version bump?
I am ofcourse OK with either solution - go-mod or a build flag to change the new default behavior

I think this works now. Should also allow accepting other PR-s


Add ForcesSCFNTGenerator

This tool allows you to drag and drop any TTF or OTF and have it be converted into a SCFNT compatible with Sonic Forces.

Some commits are still going through, so don't merge yet.


getEntries function needs a filter for visibility

Right now all blog posts will be show, even if they are in visibility=draft. I think the getEntries function needs a switch to only return items where visibility=visible.

this is may resolve this issue
{'where': {'key': 'visibility', 'expr': '=', 'value': 'published'}

fixed


[Feature request] Typescript support

Somebody should open this issue :)
If you will inject support on the project start - it would be easily to support it during development.

Hey man, sorry about this one. I'm going to have to say no for now I'm not interested in having typescript in this project


Add transactionhash to receipt logs

Signed-off-by: Nana-EC<EMAIL_ADDRESS>Description:
go-eth client seems to be stricter than other clients and requires the transaction hash in logs

Add transactionHash in logs
Updated eth.spec.ts

Related issue(s):
Fixes #226
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report

Merging #235 (d46a2e2) into main (75625e1) will increase coverage by 0.05%.
The diff coverage is 57.14%.

:exclamation: Current head d46a2e2 differs from pull request most recent head 30d12f6. Consider uploading reports for the commit 30d12f6 to get more accurate results

@@            Coverage Diff             @@
##             main     #235      +/-   ##
==========================================
+ Coverage   66.08%   66.13%   +0.05%
==========================================
  Files           7        7
  Lines         625      626       +1
  Branches       90       92       +2
==========================================
+ Hits          413      414       +1
  Misses        184      184
  Partials       28       28

Impacted Files
Coverage Δ

packages/relay/src/lib/eth.ts
55.72% <57.14%> (+0.13%)
:arrow_up:

Continue to review full report at Codecov.

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


Automated cherry pick of #2479: Improve NetworkPolicy batch installation

Cherry pick of #2479 on release-1.0.
#2479: Improve NetworkPolicy batch installation
For details on the cherry pick process, see the cherry pick requests page.

Codecov Report

Merging #2508 (0950b80) into release-1.0 (529d6f8) will decrease coverage by 19.18%.
The diff coverage is 70.27%.

@@               Coverage Diff                @@
##           release-1.0    #2508       +/-   ##
================================================
- Coverage        60.73%   41.54%   -19.19%
================================================
  Files              272      131      -141
  Lines            20375    16473     -3902
================================================
- Hits             12374     6844     -5530
- Misses            6711     9044     +2333
+ Partials          1290      585      -705

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
41.54% <70.27%> (+0.20%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/ovs/openflow/ofctrl_flow.go
36.92% <0.00%> (-8.70%)
:arrow_down:

pkg/util/ip/ip.go
72.85% <0.00%> (-7.45%)
:arrow_down:

pkg/agent/openflow/pipeline.go
25.67% <66.66%> (-41.12%)
:arrow_down:

pkg/agent/openflow/network_policy.go
67.23% <86.79%> (-8.59%)
:arrow_down:

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/ovs/openflow/default.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/helper.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/cniserver/pod_configuration_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/v1beta1/register.go
0.00% <0.00%> (-92.86%)
:arrow_down:

... and 232 more

/test-all

@tnqn The Go tidiness check is failing

/test-all

/test-e2e
/test-conformance


Feature/goss sampler

Thanks @gravesee , are we OK to merge this in now you think? This all looks great to me!

I'm working on adding the parameters to the GradientBooster but should be good after that


Back button added

Description

This PR does the following:

Adds the back button to all the pages
Fixes ...
Updates ...

Related Issues
Link any related issues using the format Fixes #issue_number.
This helps to automatically close related issues when the PR is merged.

Placeholder: "Fixes #429 "

Changes
List the detailed changes made in this PR.

Added a new feature to ...
Refactored the ...
Fixed a bug in pages where a back button is missing

Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Verify that ...

Screenshots (if applicable)
Add any screenshots that help explain or visualize the changes.

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I am working on this issue under GSSOC

fix these errors.


Dec 04 16:24:09 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]: waiting until apiserver starts
Dec 04 16:24:19 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]: waiting until apiserver starts
Dec 04 16:24:29 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]: waiting until apiserver starts
Dec 04 16:24:39 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]: waiting until apiserver starts
Dec 04 16:24:49 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]: {
Dec 04 16:24:49 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]:   "major": "1",
Dec 04 16:24:49 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]:   "minor": "8+",
Dec 04 16:24:49 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]:   "gitVersion": "v1.8.4+coreos.0",
Dec 04 16:24:49 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]:   "gitCommit": "4292f9682595afddbb4f8b1483673449c74f9619",
Dec 04 16:24:49 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]:   "gitTreeState": "clean",
Dec 04 16:24:49 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]:   "buildDate": "2017-11-21T17:22:25Z",
Dec 04 16:24:49 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]:   "goVersion": "go1.8.3",
Dec 04 16:24:49 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]:   "compiler": "gc",
Dec 04 16:24:49 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]:   "platform": "linux/amd64"
Dec 04 16:24:49 ip-10-0-0-156.ap-northeast-1.compute.internal bash[1701]: }
Dec 04 16:24:56 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: NAME          STATUS    AGE
Dec 04 16:24:56 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: kube-system   Active    11m
Dec 04 16:25:04 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: NAME                     AGE
Dec 04 16:25:04 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: v1beta1.metrics.k8s.io   11m
Dec 04 16:25:05 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: apiserver is up but kube-proxy isn't up. We have likely encountered #1039.
Dec 04 16:25:05 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: Temporary deleting the v1beta1.metrics.k8s.io apiservice as a work-around for #1039
Dec 04 16:25:12 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: apiservice "v1beta1.metrics.k8s.io" deleted
Dec 04 16:25:12 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: Waiting until controller-manager stabilizes and it creates a kube-proxy pod.
Dec 04 16:25:12 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: Sleeping 3 seconds.
Dec 04 16:25:15 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: Sleeping 3 seconds.
Dec 04 16:25:18 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: Sleeping 3 seconds.
Dec 04 16:25:21 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: Sleeping 3 seconds.
Dec 04 16:25:24 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: Sleeping 3 seconds.
Dec 04 16:25:27 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: Sleeping 3 seconds.
Dec 04 16:25:30 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: Sleeping 3 seconds.
Dec 04 16:25:33 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: Sleeping 3 seconds.
Dec 04 16:25:36 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]:  2429 ?        Ssl    0:00 /hyperkube proxy --config=/etc/kubernetes/kube-proxy/kube-proxy-config.yaml
Dec 04 16:25:36 ip-10-0-0-156.ap-northeast-1.compute.internal retry[2069]: kube-proxy stared. apiserver should be responsive again.

I wish this won't be a long-term solution 😉

Should the apiservice v1beta1.metrics.k8s.io  somehow get recreated after applying the workaround?

@mumoshu  you have deleted the apiservice v1beta1.metrics.k8s.io but how did you recreated after cluster bootstrap?

@tsuna @sohel2020 Nope. The commands in the {{ if .Addons.MetricsServer.Enabled -}} block in install-kube-system of cloud-config-controller should recreate it, along with serviceaccount, daemonset, svc, and so on.


Pushing down the AGORA_VERSION to DockerHub build

Why don't you just run the script and then read the resulting file ?

Why don't you just run the script and then read the resulting file ?

Why don't you just run the script and then read the resulting file ?

I was thinking about that, but then I guess I have to install rdmd before every build...

Why don't you just run the script and then read the resulting file ?

I was thinking about that, but then I guess I have to install rdmd before every build...

Good point. Current method is better then.

Good point. Current method is better then.

Looks good. Why is it still a draft ?

Looks good. Why is it still a draft ?

I think this broke the dockercloud build, because it does a shallow clone:
Executing build hook...
+ set -o pipefail
+ AGORA_VERSION=HEAD
++ git describe
fatal: No names found, cannot describe anything.
+ GIT_DESCRIBE_OUT=
build hook failed! (128)

I think this broke the dockercloud build, because it does a shallow clone:
Executing build hook...
+ set -o pipefail
+ AGORA_VERSION=HEAD
++ git describe
fatal: No names found, cannot describe anything.
+ GIT_DESCRIBE_OUT=
build hook failed! (128)

I think this broke the dockercloud build, because it does a shallow clone:
Executing build hook...
+ set -o pipefail
+ AGORA_VERSION=HEAD
++ git describe
fatal: No names found, cannot describe anything.
+ GIT_DESCRIBE_OUT=
build hook failed! (128)

my dockerhub build initially failed, but after I pushed smoe tag back to the remote repository I no longer got that error message

I think this broke the dockercloud build, because it does a shallow clone:
Executing build hook...
+ set -o pipefail
+ AGORA_VERSION=HEAD
++ git describe
fatal: No names found, cannot describe anything.
+ GIT_DESCRIBE_OUT=
build hook failed! (128)

my dockerhub build initially failed, but after I pushed smoe tag back to the remote repository I no longer got that error message

You mean you pushed them to your fork ?

You mean you pushed them to your fork ?

and it strangely worked after that...

and it strangely worked after that...

Yeah, docker hub isn't great and tends to pull from the fork...

Yeah, docker hub isn't great and tends to pull from the fork...


artifactory upgrade

We need the ivy pattern for sbt and that isn't available in oss until artifactory 4.6. We need that pattern to consolidate all repos needed for community build.

(the motivation here is to reduce the time the community build takes, currently 10+ hours, much of it spent resolving. @InTheNow reports running Artifactory locally and having the community build finish in less than 3 hours; we ought to be able to get in that same ballpark)

Hi,
From the GUI - "You are running Artifactory version 4.0.2",  so it might be that just consolidating your repositories on artifactory will suffice.

thanks -- I've upgraded artifactory in #171, will look into that virtual repo tomorrow, now that we have the right repo layouts available to us

@InTheNow, would you have the consolidated repo config you're using readily available for sharing?

would you have the consolidated repo config you're using readily available for sharing?

I hope to have a docker image ready for the whole thing today.... let's see

Thanks for the pointers! I went ahead and implemented your suggestions. I had to upgrade our artifactory, which triggered a whole avalanche of upgrades, but we're all on Java 8 and using caching.


Config environment variable not replaced by merge

Hi,
I would like to do be able to build multiple variations of the same app.
Very basic example:
webpack.config.js
ConfigEnvironment.INSTANCE.setAll({
    env: () => process.env.WEBPACK_ENV || process.env.NODE_ENV
    theme: () => argv.THEME || 'default'
});

export default new Config().extend('./webpack.base.config.js');

webpack.base.config.js
export default new Config().merge({
    entry: 'path/[theme]/App.js'
    ...
});

[theme] is not replaced when used with merge.
.extend('./webpack.[theme].config.js') would work.
Is there a reason why [theme] is not replaced by merge?
Thank you

@cdreve It is not supported now, webpack-config replaces only params of .extend('path/[theme]/App.js') method.
I have some plans to do that and I will add merge/default strategies which would support that behaviour.

@mdreizin Cool, thank you. Until then I am making the replace manually. It will look a lot better after we have this behaviour :)

@cdreve I have just added that feature, now all String values are preprocessed by ConfigOptionsResolver.js.
Please update to the latest version (6.1.0) and close that issues if everything is ok.

@cdreve Also if you interested how it was implemented please see ConfigMergeCommand.js

@cdreve I have forgot to tell you about some breaking changes in new release. Please see more at changelog.

@cdreve Please update to 6.1.2.

hello, any updates?


Deployment Issue related to VTAP service

We have identified a deployment issue in OC1 Region related to the VTAP feature as it has been temporarily disabled.
Error: 404-NotAuthorizedOrNotFound, NotAuthorizedOrNotFound
The following resources are not available:
https://registry.terraform.io/providers/oracle/oci/latest/docs/resources/core_capture_filter
https://docs.oracle.com/iaas/api/#/en/iaas/20160918/CaptureFilter/CreateCaptureFilter
This is due to the fact that all VTAP service management operations (Create, Update, List, Get & Delete) are currently disabled.   Please reach out to Oracle Customer Support if you should have any questions related to VTAP service issues.

We had new variable to enable or disable VTAP service deployment.
Variable : is_vtap_enabled(boolean)
Default Value : true (i.e. VTAP service will be deployed)
If the end users decided not to deploy the VTAP service , they can set the variable to false on the tfvars file(CLI Deployment)
is_vtap_enabled        = false
Fix is there on the integration branch*.
https://github.com/oracle-quickstart/oci-scca-landingzone/tree/integration
*PS : integration branch is not end to end tested.


feat: post preview

Only works on already existing posts. This is because new post do not yet have an id. Which needs to be implemented.
After image hosting is done we should add the cover image to the preview.
Closes #41

Hey @sidemt,
What Browser are you using? I do not see any scrollbars.

First, it’s OK to show only the saved content in the preview. So, you can get the content from the API instead of passing it in as a query parameter.

I kind of like showing the unsaved changes to the user. This way they do not need to save the draft every time they want to preview a change they made.

I asked to open the preview in a new tab because it was opening it in the same tab, replacing the editor that had unsaved changes, without warning. So, when I first tried opening the preview, I forgot to save the changes, and they got erased. I didn’t want that to happen.

I understand. Does it open in a new tab for you now?

Secondly, you changed the warning about the preview mode to a dismissible toast. Are there any reasons behind? I’m afraid it could be a little annoying for authors when they refresh or reopen the preview frequently.

Yes there is a reason behind this. Currently when using an Alert at the top of the page, then scroll down you'll see that the text from the Alert and the text inside the preview (editor) will start z-fighting and the text becomes unreadable.
One option for the dismissible toast would be to make the duration finite and close it automatically.
Another option is to not make the Alert sticky at the top and let it scroll with the page itself. But in my opinion that looks strange. Let me know what you think.


Updating spiffe-helper to support the new version of Workload API

The spiffe-helper has been updated to work fetching the SVID from the new Workload API using streaming over gRPC. It uses the workload.X509Client to obtain the SVIDs and then stores the certificate, bundle and key to the configured certs folder.
Removed a test file that didn't follow the go testing conventions and added new tests for spiffe_helper and config. Moved test fixtures to a better named folder structure.

👍


Updated the Arm Template to use the windows Os Info in place of the is sysprepped bool

Changed the isSysprepped bool, to use the windowsOsState property

Can one of the admins verify this patch?

Aside from the comments I left inline, this looks good to me.


[Request template]: deno deploy example

Description?
Add https://github.com/Loskir/grammy-deno-deploy-example or https://github.com/grammyjs/examples/blob/main/deno-deploy to the list
Platform
Deno

Easy to add these to templates now with the Go version. I think we should add all examples from grammyjs/examples to templates.json.
Also, as a continuation of the discussion we had in #17, I think we should split the templates.json to each platforms to avoid fetching templates for another platforms.


Group icons aren't displayed anymore

Description
For a few days group icons aren't displayed anymore - no matter the page.

Instead of
cdn.edgecast.steamstatic.com/steamcommunity/public/images/avatars/*
the URLs should be
steamcdn-a.akamaihd.net/steamcommunity/public/images/avatars/*

I have no idea if that's possible.


autocorrection ability

we need autocorrecting, for example haml-lint app/views -a like rubocop

Hey @stereodenis, thanks for suggesting ways we can improve haml-lint.
Adding auto-correcting functionality simply isn't practical. RuboCop enjoyed the benefit of using the powerful parser gem, which makes it trivial to rewrite code by modifying the abstract syntax tree.
The haml gem's parser comes nowhere close to this level of flexibility, and the situation is further complicated by the fact that there are two languages here: Ruby and HAML, so we'd have to leverage RuboCop's autocorrecting as well build our own mutable syntax tree representation for HAML.
It's certainly possible but would be a huge undertaking in itself, so unless someone comes along and implements it you won't be seeing this feature in haml-lint.


Feature/deploy staging from develop

PR to deploy from develop branch instead of staging. Also added some scripts and documentation

Nice PR! I do appreciate it.


Issue69

UT:
1, Run function vlan_status from report_rc.sh on test machine 121 to get valn status list;
2, Parse the above list in Go function the same as VlanStatus;
3, Remove old_vlan_list from test machine 121 to trigger vlan status report. Add log in vlan_status.go and check log from /var/log/message, the vlan list was passed to vlan_status.go and was parsed correctly.

close this pr and resubmit


[Feature #158] [v10] Dashboard: Vertical (Battle Mode)

resolved #158
Single Fleet Interface

Combined Fleet Interface
Due to the lack of space, I created four buttons for users to see corresponding elements of the fleet. They are:
Lv: Level & Experience
Mr: Morale
Eq: Equipment
Fu: Fuel & Ammo

Note 1: I also add more color for map nodes to show the type of the node (e.g. battle, resource, maestrom and battle avoided).
Note 2: The interface can auto-detect the type of fleet but I still add a button to switch between single and combined fleet for other users to play around with it while waiting for the event xD

I just decided to decrease the size of the number next to the equipment a bit. It will look a little smaller than what is shown in the image above.

look forward for today release :+1:

:+1: excited to test this out


Add ‘Measuring the value of the GOV.UK Design System’ talk to the blog posts, videos and podcasts page

I've requested a review from @timpaul just in case it was a conscious decision not to add this to the page for a reason that I'm not aware of…


Generate Permutations

Write code in C,JS or Python programming language to generate the permutations for the set of numbers entered .

I want to work on this issue

Can I implement this in Python?

I'd want to work on this issue. May I be assigned this?

I'd want to work on this issue. May I be assigned this?

In JS or python ?

I'll implement in python

I'd want to work on this issue. May I be assigned this?

In JS or python ?

I thought it was in C/C++. I'd prefer these languages.

I'd want to work on this issue. May I be assigned this?

In JS or python ?

I thought it was in C/C++. I'd prefer these languages.

Its already done . You can look at other issues

I'll implement in python. Assign it to me.

Please assign this to me. I'll work in python

I want to work on this in java, please assign it to me.

Hi @ankitapuri
Please assign it to me in JS.


Increase separation between menu elements

Signed-off-by: Vicente J. Jiménez Miras<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area videos

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #670
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: vjjmiras
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval by writing /assign @leogr in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 0d751134991a765fbe748497408ca75e6cdbc395


Avoid leaking HTTPParser from a ClientRequest error path

Description

This avoids a memory leak where a pair of HTTPIncomingMessage and HTTPParser objects are left with a strong reference cycle when a ClientRequest is unsuccessful (curl returns an error, or unable to parse the response).
Motivation and Context

Partial fix for Kitura #548 (Memory leak when executing HTTP.request)

there is still a leak that is somehow related to the headers passed to CCurl, which will be addressed in a future PR.

How Has This Been Tested?

I ran the Kitura-net tests on Linux and Mac, and a simple testcase for demonstrating the leak.
Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

After reviewing this with David, we've decided to take a different approach


DWI MRIQC report were not generated

On Oscar's machine, /data/derivatives/hcph-derivatives/mriqc_23.2.0 does not contain the individual report for the dwi. I thus suspect the MRIQC dwi workflow was not successful.

Can you attach debugging information?
On Mon, Jan 15, 2024, 12:59 celprov @.***> wrote:

Assigned #418 https://github.com/TheAxonLab/hcph-sops/issues/418 to
@oesteban https://github.com/oesteban.
—
Reply to this email directly, view it on GitHub
https://github.com/TheAxonLab/hcph-sops/issues/418#event-11487362552,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAESDRU5473IQLIK5CZDIVLYOUKZJAVCNFSM6AAAAABB3E3PCWVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGQ4DOMZWGI2TKMQ
.
You are receiving this because you were assigned.Message ID:
@.***>

I do not have it right now, I just observed that the reports are missing.


_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 43s
master passed

+1 :green_heart:
compile
2m 24s
master passed

+1 :green_heart:
checkstyle
0m 31s
master passed

+1 :green_heart:
spotless
0m 41s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
1m 25s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 21s
the patch passed

+1 :green_heart:
compile
2m 21s
the patch passed

+1 :green_heart:
javac
2m 21s
the patch passed

-0 :warning:
checkstyle
0m 33s
hbase-server: The patch generated 4 new + 0 unchanged - 0 fixed = 4 total (was 0)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
13m 11s
Patch does not cause any errors with Hadoop 3.2.4 3.3.4.

-1 :x:
spotless
0m 32s
patch has 63 errors when running spotless:check, run spotless:apply to fix.

+1 :green_heart:
spotbugs
1m 29s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m  9s
The patch does not generate ASF License warnings.

38m 44s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5170/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5170

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux fc8651408de6 5.4.0-1094-aws #102~18.04.1-Ubuntu SMP Tue Jan 10 21:07:03 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / e4b4cef80e

Default Java
Eclipse Adoptium-11.0.17+8

checkstyle
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5170/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

spotless
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5170/1/artifact/yetus-general-check/output/patch-spotless.txt

Max. process+thread count
83 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5170/1/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


TypeScript runtime implementation

We need a TypeScript runtime implementation and a compact JSON serialization format for the domain model. In Blaze-Expression we will use this for editor autocompletion.

Done ed44875a083d049125e86a99b2455e47bcb8390a


make sure the quickstart guide targets host only and move all docker and 3sdk building to another document

make sure the quickstart guide targets host only and move all docker and 3sdk building to another document

https://github.com/threefoldtech/js-sdk/commit/ffe970eba7c602c16d331e5b1fe22ad9b91c8fb8

verified


Fix to #12358 - The use of the ToListAsync method leads to missing data in version 2.1.0 of Microsoft.AspNetCore.All

Problem was in the implementation of TaskLiftingExpressionVisitor and how it interacts with correlated collection optimization. When we perform task lift we do so recursively if we haven't encountered a CancellationToken parameter yet (essentially fishing for it, so that we know what to use).
This becomes a problem if correlated collection optimization async is inside another async method (like Task.FromResult). We ended up recursively lifting the task twice, which resulted in the same correlate call executed twice, which is a problem because both calls share reference to the same reader.
Fix is to fetch CancelationToken parameter as a separate step and don't recursively lift the tasks. It can still deadlock potentially, but will now return correct results.

we already have tests: Correlated_collections_nested_inner_subquery_references_outer_qsre_one_level_up and ToArray_on_nav_subquery_in_projection_nested

merged 3d2ba391cc1d8d6cc7b0a67ced4539067ad07a64


gqrx not opening after install

$ gqrx   (gqrx-sdr_20231130-1_amd64.deb)
gqrx: error while loading shared libraries: libgnuradio-digital.so.3.10.4: cannot open shared object file: No such file or directory
I can not find libgnuradio-digital.so.3.10.4:   I have looked at every search option I came across, but only find earlier or later versions.
I found some information suggesting there may be a typo from the GQRX  calling for it.
Any suggestions how I may resolve this?
(gqrx-sdr_20231130-1_amd64.deb)

These packages were only meant for DragonOS FocalX. They will not work on anything else. May I ask what you were trying to install it on?

DragonOS FocalX, update/upgrade to jammy.    Worked perfectly for quite some time.  I tried updating GQRX. Had problems, tried going back to original focalx_ppa.   Can't seem to do anything with any builds from source either.  Seems to install, but errors as mentioned no matter what.
I can not figure out where that ( libgnuradio-digital.so.3.10.4)  is being called from.
Maybe something isn't purging preventing a clean install?   All the other apps still work fine --Just problem with GQRX

I think I am making progress downgrading.  Looks like I kept getting upgrade installs until now.  Hopefully this will work (gnuradio 3.08)

It’s only for GNU Radio 3.10, currently this PPA is mostly for 3.10.4 specifically as built in R35 DragonOS. In DragonOS R36 it is all rebuilt manually around 3.10.10 and that requires a reinstall of the latest DragonOS FocalX ISO. By the way, the name is confusing as FocalX is actually Jammy already.
Anyways, I’m slowly getting all packages on the PPA setup to only apply to those running R36 or higher.

libgnuradio-digital.so.3.10.4 Is part of gnuradio. The gnuradio, if messed with, will also mess up gqrx as it uses it under the hood. If things get to far out of whack for you, id grab the latest DragonOS FocalX ISO which has now 3.10.10 Gnuradio at its core. PPA updates will then starts to flow again as I put new ones up.

okay - I appreciate the information.
I started trying to restore SRDA..... to get the older version with the DMR demod dsd plugin.
Will we see that again?   I have had no  success with implementing dsdccx in Dragon.
I'm gonna go ahead with Dragon_F36.   If you have any suggestions about getting the digital demod working , I would like to follow up on that as well.  Thanks again for all of your feedback.

I’ll have to go look, but SDRAngel should still have the DSD portion but maybe I missed it being removed?
I also include DSDFME in DragonOS that’s located under /usr/src. It works really well when feed audio and setup properly.

Thank you.  So far 36 looks good.
When in the demod the graphic of the signal remains a large dot which seems
unusual, but it appears the rest seems normal.  Will know better in time.
Thank you for the suggestion.  I was wasting too much time wrestling with
all that stuff.
On Wed, Jul 31, 2024, 19:32 alphafox02 @.***> wrote:

I’ll have to go look, but SDRAngel should still have the DSD portion but
maybe I missed it being removed?
I also include DSDFME in DragonOS that’s located under /usr/src. It works
really well when feed audio and setup properly.
—
Reply to this email directly, view it on GitHub
https://github.com/alphafox02/focalx_ppa/issues/3#issuecomment-2261833027,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASHSJR72EKRZRM25OS7IOPLZPGM2NAVCNFSM6AAAAABLY2TR36VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRRHAZTGMBSG4
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


fix: allow for execution of timelocked proposal when overruled in closed state #ntrn-225

checks for fix: close the overrule proposal and check that you can still execute timelocked proposal
Task: https://hadronlabs.atlassian.net/browse/NTRN-225
Related PR's:

https://github.com/neutron-org/neutron-dao/pull/100
https://github.com/neutron-org/dao-ops/pull/27
https://github.com/neutron-org/neutron-integration-tests/pull/268

https://github.com/neutron-org/neutron-tests/actions/runs/8235688238


Error of XMLHttpRequest

When I launch an old html built with jspsych with chrome I am running into the following error, even though the script has worked fine before
“jspsych.js:2198 Access to XMLHttpRequest at 'file:///Users/Seara/Desktop/prototype_experiment/mp3s/CelineDion1.mp3' from origin 'null' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, https.
load_audio_file_webaudio @ jspsych.js:2198”
I have never encountered this error before, and the bug seem to be coming from jsPsych. Do you have any idea as to why is this happening?

This is a security feature of web browsers. If you run the experiment on a web server the problem will be solved. You can also disable the security for local testing.