Distinguish between events from files with the same name

Describe the issue
Hi!
While using this API I found myself unable to distinguish between events that arrive from files with the same name yet in different paths. The problem can be formalised as below.

Given two file watchers with the same file name but in different paths:
file watch 1: /tmp/file_a
file watch 2: /tmp/another_dir/file_a
And randomly triggering an event on one of the files. For which file was the event triggered?

Proposed changes
This PR proposes a getter method for the WatchDescriptor, allowing one the ability to store the id for a given file watch after an .add_watch() and using this to compare the Event struct coming from an event stream.

As a side note, I am still learning Rust so if I used something where perhaps something else was better let me know and I am happy to make the changes! Also apologies for the auto-formatting on the files, I can revert them. Let me know! Thanks

Are you in a hurry to get this merged? We could wait for a while, to get feedback on #194, or we could do the conservative MSRV upgrade right now, to get things going again.

Thanks for reviewing so quickly. Not in a hurry, this can wait :smile:

This is fine. In fact, it would be better to auto-format the whole code base and enforce the formatting on CI. But please put formatting changes in separate commits in the future. Having them mixed in with actual changes makes those changes needlessly hard to review.

I went ahead and turned off auto formatting and removed the linting stuff. It makes the PR clearer. Sorry again! I can make a separate PR for the linting stuff! Happy to help where I can.

Thanks for reviewing so quickly. Not in a hurry, this can wait :smile:

Great :+1:
If no feedback on the issue comes for a while, feel free to just submit a PR with an MSRV upgrade to the latest version.

I went ahead and turned off auto formatting and removed the linting stuff. It makes the PR clearer. Sorry again! I can make a separate PR for the linting stuff! Happy to help where I can.

No problem, it wasn't a big deal! If you're willing to sort out the formatting, that would definitely be welcome! It would prevent issues like this in the future.
What basically needs to happen, is to run cargo fmt (which should auto-format the whole codebase), commit that, then add a step to the CI configuration that runs cargo fmt --check (that's from memory; could be slightly different).

Hey @initard! The MSRV was raised in #196. I've rebased this pull request and re-run the CI build, but it still fails, although for a different reason. Looks like some test code fails to compile. Could you have a look?

Hi @hannobraun! Apologies for the wait. I updated the tests — it was an issue with the feature flags for the "stream" feature.
Can you please try again?
As you can see from the linked issue above (https://github.com/thin-edge/thin-edge.io/issues/1453), our team ran into this exact issue recently and it would be nice to have this merged.

PS. I noticed there was also an API change (.add_watcher and .event_stream were changed), so I hope this PR is still relevant?


Issue

[01:26:44] [Server thread/CRITICAL]: TypeError: "Argument 1 passed to pocketmine\lang\BaseLang::translate() must be an instance of pocketmine\lang\TextContainer, string given, called in phar:///home/container/plugins/1vs1.phar/src/vixikhd/onevsone/arena/Arena.php on line 387" (EXCEPTION) in "src/pocketmine/lang/BaseLang" at line 135
This occurs in the middle of a duel in our Second arena. I think I was either close to killing my opponent, or I had just killed them when they were kicked with an Internal Server Error, and that error showed in console.

In order to fix it, I had to delete both arenas, delete and re-add the plugin, and then re-add 1 arena.

This is pocketmine error


Minor updates to Snowflake changefeed tutorial

Fixes DOC-2129
Updates to tutorial:

Relevant additions to Snowflake doc links
GCS is now covered in auto-ingest by snowflake (updated note)
Removed resolved option from the create changefeed statement to avoid creation of too many objects in the bucket.
(the experimental was removed from the s3 URI in another PR, which is mentioned in this ticket)

This change is

1 nit on docs URL. LGTM!

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @kathancox and @taroface)

cockroachcloud/stream-changefeed-to-snowflake-aws.md, line 124 at r1 (raw file):

~~~

Be sure to replace the placeholders with your AWS access key ID and AWS secret access key. See [Use Cloud Storage for Bulk Operations — Authentication](https://www.cockroachlabs.com/docs/stable/use-cloud-storage-for-bulk-operations.html#amazon-s3) for more detail on authenticating to Amazon S3.

Use a relative link here
Code quote:
https://www.cockroachlabs.com/docs/stable/use-cloud-storage-for-bulk-operations.html#amazon-s3

cockroachcloud/stream-changefeed-to-snowflake-aws.md, line 179 at r1 (raw file):

    Be sure to replace the placeholders with your AWS access key ID and AWS secret access key.

6. In the Worksheet, create a Snowpipe called `cdc-pipe`, which tells Snowflake to auto-ingest data:

🏂
Code quote:
Snowpipe

Whooops, that was a quick copy/paste mistake with the URL! Thanks for the catch and review @taroface!


I cant make use of the Entity.FrameworkCore

Type of issue
Typo
Description
[Enter feedback here]
Install-Package : NU1202: Package Microsoft.EntityFrameworkCore 8.0.4 is not compatible with net7.0 (.NETCoreApp,Version=v7.0). Package
Microsoft.EntityFrameworkCore 8.0.4 supports: net8.0 (.NETCoreApp,Version=v8.0)
At line:1 char:1

Install-Package Microsoft.EntityFrameworkCore

  + CategoryInfo          : NotSpecified: (:) [Install-Package], Exception
  + FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PackageManagement.PowerShellCmdlets.InstallPackageCommand

Install-Package : Package restore failed. Rolling back package changes for 'IAP-CW'.
At line:1 char:1

Install-Package Microsoft.EntityFrameworkCore

  + CategoryInfo          : NotSpecified: (:) [Install-Package], Exception

Page URL
https://learn.microsoft.com/en-gb/ef/core/
Content source URL
https://github.com/dotnet/EntityFramework.Docs/blob/main/entity-framework/core/index.md
Document Version Independent Id
f4681bc6-57be-6e69-cad9-776ce64d93bd
Article author
@ajcvickers

As the error message says, you seem to be trying to use EF Core 8.0 from a project targeting .NET 7.0 - this is not supported. Update your .NET project to target .NET 8.0.


Add -A/--argumentfile support in Testdoc

Using testdoc seems the best choice to me when trying to have a preview of tests selected by robot on a certain situation. There is no runtime dependency that we must have to run it, and at the moment I can't trust the tests to have no consequences when importing their test libraries to use the dry run mode.
We are heavily using the --argumentfile option to select tests using --test and --suite. The problem is that testdoc does not support that option. I need to manually build the command line based on the argument files to call testdoc at this point.
It would also be cool to accept or ignore any robot/rebot options, so we can use the exact same files with testdoc. Do you think this is reasonable?
I can try looking into implementing this if you can provide some guidelines.
This is somewhat related to #1876.

First of all, sorry for ignoring this issue for so long. It was buried under a pile of other issues during summer time when I didn't have much time to follow this tracker.
Good news is that adding argument file support to Testdoc is is trivial. Just need to add --argumentfile into Testdoc's usage similarly as in Robot's and Rebot's usages and the custom argument parsing logic takes care of the rest. Well, should add a test for it too and perhaps mention it also in the User Guide.
This is so trivial that I can do it for 3.0.2 that I plan to release this week. Let me know if you'd like to help, though!

Ignoring Robot's option isn't, in my opinion, a good idea. It can easily mask errors and could cause problems if some of those options were later added to Testdoc.

That was really fast, thanks! Sorry I did not had the time to look into it.

Well, it took 6 months before I noticed this issue. =) Figured I could as well implement it since it was so trivial. The biggest problem was trying to understand why --ArgumentFile didn't work which lead to #2535.


Error: [$injector:unpr] Unknown provider: $translateStaticFilesLoaderProvider issue

Hello,
I'm trying to use the $translateStaticFilesLoaderProvider but I can't. I have an issue at my application loading (maybe linked to asynchronous loading ?)
Error: [$injector:unpr] Unknown provider: $translateStaticFilesLoaderProvider <- $translateStaticFilesLoader
http://errors.angularjs.org/1.3.15/$injector/unpr?p0=%24translateStaticFilesLoaderProvider%20%3C-%20%24translateStaticFilesLoader
   at REGEX_STRING_REGEXP (angular.js:63)
   at angular.js:4015
   at Object.getService [as get] (angular.js:4162)
   at angular.js:4020
   at Object.getService [as get] (angular.js:4162)
   at $get.loadAsync (angular-translate.js:1040)
   at Function.$get.$translate.use (angular-translate.js:1572)
   at angular-translate.js:38
   at Object.invoke (angular.js:4203)
   at angular.js:4025

I have installed the $translateStaticFilesLoaderProvider using bower install angular-translate-loader-url to install and I have the js files added in my project.
I have imported them in my index.html
...
<script src="bower_components/angular-translate/angular-translate.js"></script>
<script src="bower_components/angular-translate-loader-url/angular-translate-loader-url.js"></script>
...

I'm currently using AngularJS 1.3.15. Could it be the cause of the issue ?
Do you have any other ideas of what could cause this issue ?

Did you include the angular-translate-loader-static-files file also?

Ok that was the issue actually :)
I did not read the documentation well enough.
Thanks a lot for this very quick answer.


Added SeeedStudio Grove Hat for RPI & RPI Zero support

The two Seeedstudio RPI hats have onboard analog to digital convertors (A2D) with associated connectors for Grove analog sensors.
The RPI 2/3 hat has 8 analog ports
The RPI Zero hat works on an RP2/3 but has only 4 x analog ports
The library supports both RPI hats via a conditional compilation setting.

Docs Build status updates of commit 8157d92:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/learn-about-hardware/HardwareCompatList.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Getting public from private key

Hi Jonathan,
I am writing a program that has to compute the STARK public key based on the private key.
I have to private key in the form of a hex encoded string. I then decoded it into a byte slice. However, I see that the get_public_key function accepts a fixed size byte slice of &[u8; 32]. Having looked online, I couldn't find any way of converting &[u8] into &[u8; 32]. So I created a helper function which takes in the input string and attempts to convert it:
fn hex_to_32byte(inp: &str) -> Result<[u8; 32], AllErr> {
    let decoded = hex::decode(inp)?.as_slice();
    let out: [u8; 32];

    for i in 0..32 {
        out[i] = decoded[i];
    }

    Ok(out)
}

Is there a more efficient way?

Basically, I'm looking to get the public key and the public key y coordinate. Does the function accomplish this?

The library only exposes a function that accepts a byte array of 32, since the library itself is supposed to be a very thin FFI layer which directly exposes the underlying C signature.
You function can be improved by using copy_from_slice instead of a for loop.

For the second question regarding public key y coordinate, I think you're basically trying to get the uncompressed form of the public key.
Again, this library is a thin FFI layer which refrains from adding additional functionalities other than those offered by the underlying library itself. So no, this library does not allow you to do that, unfortunately.

Does it take the private key and put it on an ec point in the stark curve?

That said, there is indeed another library which contains the logic you're after (though it does not expose the said logic). The code which converts a compressed public key into a full key can be found here.

Actually at this point it's recommended to use the starknet-crypto crate offered by the starknet-rs project, since there are a lot of known issues with the underlying crypto-cpp library, which is not actively maintained.

Does it take the private key and put it on an ec point in the stark curve?

No you put the compressed public key on the curve, not the private key. It's demonstrated by the line I quoted above.

The starknet-crypto crate is a much better alternative written in pure Rust. It's much more performant as well.


ServiceBus Admin Client - receives 401 responses after migration from standard to premium tier

Bug Report
When using the ServiceBus Admin client it does not automatically connect to the Premium namespace when a Standard Tier namespace is migrated to premium as documented here:
https://learn.microsoft.com/en-us/azure/service-bus-messaging/service-bus-migrate-standard-premium#what-happens-when-the-migration-is-committed
Instead it always runs into an unauthenticated error. Restarting the application does fix the issue, but according to the documentation it should not be necessary.

GET https://XXX.servicebus.windows.net/TOPIC/Subscriptions/SUBSCRIPTION-------------------------------------------------------------------------------- RESPONSE 401: 401 Unauthorized ERROR CODE: 401 -------------------------------------------------------------------------------- Error Code 401  Detail Connection rejected after GeoDR failover. TrackingId:e6768e23-213f-476c-a654-f7bab349c5af_G4, SystemTracker:XXX.servicebus.windows.net:TOPIC/Subscriptions/SUBSCRIPTION, Timestamp:2023-05-16T13:24:18   --------------------------------------------------------------------------------

The problem was noticed with KEDA: kedacore/keda#4678.
The problem can be easily reproduced with running the following program:
package main

import (
	"context"
	"fmt"
	"time"

	"github.com/Azure/azure-sdk-for-go/sdk/messaging/azservicebus/admin"
)

func GetAdminClient() *admin.Client {
	connectionString := "<connection string>"
	client, err := admin.NewClientFromConnectionString(connectionString, nil)
	if err != nil {
		panic(err)
	}

	return client

}

func PrintMessageCountOrError(ctx context.Context, client *admin.Client) {
	properties, err := client.GetSubscriptionRuntimeProperties(ctx, "mytopic", "mysubscription", nil)
	if err != nil {
		fmt.Println(err)
	} else {
		fmt.Println(properties.TotalMessageCount)
	}
}

func main() {
	ctx := context.TODO()
	client := GetAdminClient()

	for {
		time.Sleep(1)
		PrintMessageCountOrError(ctx, client)
	}
}

Furthermore the Standard Tier ServiceBus namespace needs to have a topic: mytopic and a subscription: mysubscription. Then during the runtime of the program the namespace can be migrated to a Premium namespace through the Azure Portal. After the migration is completed (a few seconds/minutes later) the application will start to log, that it is not able to connect anymore.
As far as I was able to debug the problem already, it occurs because the connection is after the migration still to the standard namespace. Therefore restarting the application causes it to connect to the new premium namespace. Also doing a long pause in the debugger, so that the connection gets closed, causes it to work again.

SDK versions:

github.com/Azure/azure-sdk-for-go/sdk/azcore v1.6.0 h1:8kDqDngH+DmVBiCtIjCFTGa7MBnsIOkF9IccInFEbjk=
github.com/Azure/azure-sdk-for-go/sdk/azcore v1.6.0/go.mod h1:bjGvMhVMb+EEm3VRNQawDMUyMMjo+S5ewNjflkep/0Q=
github.com/Azure/azure-sdk-for-go/sdk/azidentity v1.3.0 h1:vcYCAze6p19qBW7MhZybIsqD8sMV8js0NyQM8JDnVtg=
github.com/Azure/azure-sdk-for-go/sdk/azidentity v1.3.0/go.mod h1:OQeznEEkTZ9OrhHJoDD8ZDq51FHgXjqtP9z6bEwBq9U=
github.com/Azure/azure-sdk-for-go/sdk/internal v1.3.0 h1:sXr+ck84g/ZlZUOZiNELInmMgOsuGwdjjVkEIde0OtY=
github.com/Azure/azure-sdk-for-go/sdk/internal v1.3.0/go.mod h1:okt5dMMTOFjX/aovMlrjvvXoPMBVSPzk9185BT0+eZM=
github.com/Azure/azure-sdk-for-go/sdk/messaging/azservicebus v1.4.0 h1:MxbPJrYY81a8xnMml4qICSq1z2WusPw3jSfdIMupnYM=
github.com/Azure/azure-sdk-for-go/sdk/messaging/azservicebus v1.4.0/go.mod h1:pXDkeh10bAqElvd+S5Ppncj+DCKvJGXNa8rRT2R7rIw=
github.com/Azure/go-amqp v1.0.0 h1:QfCugi1M+4F2JDTRgVnRw7PYXLXZ9hmqk3+9+oJh3OA=
github.com/Azure/go-amqp v1.0.0/go.mod h1:+bg0x3ce5+Q3ahCEXnCsGG3ETpDQe3MEVnOuT2ywPwc=
github.com/AzureAD/microsoft-authentication-library-for-go v1.0.0 h1:OBhqkivkhkMqLPymWEppkm7vgPQY2XsHoEkaMQ0AdZY=
github.com/AzureAD/microsoft-authentication-library-for-go v1.0.0/go.mod h1:kgDmCTgBzIEPFElEF+FK0SdjAor06dRq2Go927dnQ6o=
github.com/golang-jwt/jwt v3.2.1+incompatible/go.mod h1:8pz2t5EyA70fFQQSrl6XZXzqecmYZeUEB8OUGHkxJ+I=
github.com/golang-jwt/jwt/v4 v4.5.0 h1:7cYmW1XlMY7h7ii7UhUyChSgS5wUJEnm9uZVTGqOWzg=
github.com/golang-jwt/jwt/v4 v4.5.0/go.mod h1:m21LjoU+eqJr34lmDMbreY2eSTRJ1cv77w39/MY0Ch0=
github.com/google/uuid v1.3.0 h1:t6JiXgmwXMjEs8VusXIJk2BXHsn+wx8BZdTaoZ5fu7I=
github.com/google/uuid v1.3.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo=
github.com/kylelemons/godebug v1.1.0 h1:RPNrshWIDI6G2gRW9EHilWtl7Z6Sb1BR0xunSBf0SNc=
github.com/kylelemons/godebug v1.1.0/go.mod h1:9/0rRGxNHcop5bhtWyNeEfOS8JIWk580+fNqagV/RAw=
github.com/pkg/browser v0.0.0-20210911075715-681adbf594b8 h1:KoWmjvw+nsYOo29YJK9vDA65RGE3NrOnUtO7a+RF9HU=
github.com/pkg/browser v0.0.0-20210911075715-681adbf594b8/go.mod h1:HKlIX3XHQyzLZPlr7++PzdhaXEj94dEiJgZDTsxEqUI=
golang.org/x/crypto v0.7.0 h1:AvwMYaRytfdeVt3u6mLaxYtErKYjxA2OXjJ1HHq6t3A=
golang.org/x/crypto v0.7.0/go.mod h1:pYwdfH91IfpZVANVyUOhSIPZaFoJGxTFbZhFTx+dXZU=
golang.org/x/net v0.8.0 h1:Zrh2ngAOFYneWTAIAPethzeaQLuHwhuBkuV6ZiRnUaQ=
golang.org/x/net v0.8.0/go.mod h1:QVkue5JL9kW//ek3r6jTKnTFis1tRmNAW2P1shuFdJc=
golang.org/x/sys v0.0.0-20210616045830-e2b7044e8c71/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
golang.org/x/sys v0.6.0 h1:MVltZSvRTcU2ljQOhs94SXPftV6DCNnZViHeQps87pQ=
golang.org/x/sys v0.6.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
golang.org/x/text v0.8.0 h1:57P1ETyNKtuIjB4SRd15iJxuhj8Gc416Y78H3qgMh68=
golang.org/x/text v0.8.0/go.mod h1:e1OnstbJyHTd6l/uOt8jFFHp6TRDWZR/bV3emEE/zU8=

output of go version: go version go1.20.5 linux/amd64

Sorry about taking so long on this, a lot going on!
I'm talking with the service team. I think the ideal behavior is for the service to cut off old HTTP connections as well. I don't think we want to do string matching on 401s.
I'll ping back when I have something to report.

We have brought this item in our current planning. We don't have a specific date when development will start for this, once we have more information around this, we will update this thread.


Normalize API module names for outgoing requests

This pull request allows special characters inside Api module names by url-encoding the Api name before sending it out as part of a URL parameter. See zfcampus/zf-apigility-documentation#16 for more info.
Specifically, it is designed to allow the backslash character () to be present, which is a prerequisite for supporting namespaced Api names (e.g. "Acme\Account") in the Apigility admin. Without this, the UI would insert "Acme\Account" directly as part of the URL for requests to get info from that api. This would cause issues because:

The backslash character is not a valid character in URL's (see RFC1738)
And therefore some browsers (like Chrome) convert it to a forward slash, causing a route mismatch when querying the Admin endpoint for that API.

The Apigility Admin API project tried to solve this by converting backslashes into periods. However that may change in the future. This PR doesn't care about that: it will just url-encode whatever the Admin API reports as the Api name before sending it as part of a URL, while still showing it correctly in the user-facing UI elements.
Specific changes this PR introduces:

Requests to the admin API will be used with the url-encoded version of the Api name. So if its "Acme\Account" the requested URL will use "Acme%5CAccount" instead.
The above will also be true for the routes (AngularUI Router states) in the address bar.
The user-facing UI elements will correctly show the backslash.

It would seem to me that all of the changes are backwards-compatible, but it would probably be a good idea to get that reviewed JIC.

Note: a couple more commits still pending (e.g. tests right now don't pass), just wanted to see if this is interesting enough to merge. Its also part of other PRs I'll hopefully be making across some of the Apigility projects in order to provide proper support for namespaced API names, so I'll reference those other PRs as well to provide a better picture of how everything could make sense as a whole.

Ok, couldn't fix tests and actually realised the issue with that is also happening on master with me as well. I suspect it might be related to the fact that with me bower can't resolve which version of Angular to use, so I manually opted for version 1.4.8 - but this might be material for a different ticket.

Updated the commit to take a more transparent approach (all changes were moved to api.service.js)

I would have named the variable moduleName instead of moduleParam ;-) but beside that, I like this PR.
The expected "Module Naming convention" is very strict and tends to cause troubles with the Admin UI if you edit the module config manually.
The whole idea behind these combined PRs would give us more freedom in
a) structuring and naming of Modules and Namespaces
b) switching back and forth between using the Admin UI and editing the config files directly

Haha funny: I was also considering moduleName (its indeed the most natural choice), but settled for moduleParam cause it also reflects the fact that its being used as a param - e.g. urlEncodedModuleName would have that same effect (but then: its way too long for my taste). But honestly I'm not against changing back to moduleName if that's better for everyone else.
Anyway, thanks for reviewing this PR! I had precisely those issues you mentioned about editing the module config manually :)

Merged to develop for release with 1.3.0; thanks, @gsomoza !


Adds an option to build k8s 1.22 kind cluster

What this PR does / why we need it:
This PR:

adds an option to create an e2e test kind cluster with Kubernetes v1.22 (using eu.gcr.io/jetstack-build-infra-images/kind:v1.22.0-beta.2 kind image)
This option would be triggered by the optional Prow presubmit pull-cert-manager-e2e-v1-22 that's going to be added in testing#536.

Bumps NGINX Ingress versions for e2e tests: with Kubernetes v1.16 -  v1.21 it now installs ingress controller image v0.48.1, Helm chart v3.34.0.  With Kubernetes v1.22 ingress controller image v1.0.0-alpha.2, Helm chart v3.34.0

Bumps version of sample external issuer used in e2e tests v0.1.0 -> v0.1.1

Release note:

NONE

/kind cleanup
Signed-off-by: irbekrm<EMAIL_ADDRESS>

Note that there are a few changes to the addons to work under 1.22, I just tried running on 1.22 and I saw a couple of errors, like with sample-webhook:
Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "APIService" in version "apiregistration.k8s.io/v1beta1"

and bind:
The Service "bind" is invalid: spec.clusterIPs: Invalid value: []string{"<IP_ADDRESS>"}: failed to allocate IP <IP_ADDRESS>: provided IP is not in the valid range. The range of valid IPs is <IP_ADDRESS>/16

Note that there are a few changes to the addons to work under 1.22, I just tried running on 1.22 and I saw a couple of errors, like with sample-webhook

Good catch! I've only tried installing cert-manager and creating a self-signed issuer and a cert (succesfully).
I assume that this beta version already has the beta networking API removed from looking at cert-manager logs:

ess: the server could not find the requested resource
E0722 15:21:13.688268       1 reflector.go:138] external/io_k8s_client_go/tools/cache/reflector.go:167: Failed to watch *v1beta1.Ingress: failed to list *v1beta1.Ingress: the server could not find the requested resource
E0722 15:22:03.744348       1 reflector.go:138] external/io_k8s_client_go/tools/cache/reflector.go:167: Failed to watch *v1beta1.Ingress: failed to list *v1beta1.Ingress: the server could not find the requested resource

In fact, this may not be doable yet- I see that there is no ingress Helm chart yet that would use v1 networking API and thus work on Kubernetes v1.22.
There is an Ingress alpha release where v1 networking API is used (https://github.com/kubernetes/ingress-nginx/releases/tag/v1.0.0-alpha.1), but no Helm chart for it yet.

I just tried using the ingress-nginx v1.0.0-alpha.2 and it seems to work with the current helm chart:
diff --git a/devel/addon/ingressnginx/install.sh b/devel/addon/ingressnginx/install.sh
index 896cf84ec..d8b32e510 100755
--- a/devel/addon/ingressnginx/install.sh
+++ b/devel/addon/ingressnginx/install.sh
@@ -53,8 +53,10 @@ helm upgrade \
     --wait \
     --version 3.31.0 \
     --namespace "${NAMESPACE}" \
-    --set controller.image.digest="" \
-    --set controller.image.pullPolicy=Never \
+    --set controller.image.registry=k8s.gcr.io \
+    --set controller.image.image=ingress-nginx/controller \
+    --set controller.image.tag=v1.0.0-alpha.2 \
+    --set controller.image.pullPolicy=IfNotPresent \
     --set "controller.service.clusterIP=${SERVICE_IP_PREFIX}.15"\
     --set controller.service.type=ClusterIP \
     --set controller.config.no-tls-redirect-locations="" \

I just tried using the ingress-nginx v1.0.0-alpha.2 and it seems to work with the current helm chart

On Kubernetes v1.22? For me that only seemed to work on v1.21

/test pull-cert-manager-e2e-v1-16

Great work on working around the fact that ingress NGINX does not handle both v1 and v1beta1 👍👍
I just found out that sample-externam-issuer can't install on 1.22, I opened https://github.com/cert-manager/sample-external-issuer/pull/21 to fix that.

@maelvls I just accidentally force pushed and overwrote your commit 😭

Sure, sorry for pushing here, I should have asked 😅😅

Sure, sorry for pushing here, I should have asked

Not at all, thank you for bumping the external issuer 🙏🏼
It should now work locally with:
K8S_VERSION=1.22 ./devel/cluster/create.sh
./devel/setup-e2e-deps.sh  (without needing to specify K8S version again).

/test pull-cert-manager-e2e-v1-16

/test pull-cert-manager-e2e-v1-16

I ran ./devel/run-e2e.sh. The only failures I saw are related to the missing v1beta1 Ingress, which is expected (I looked at the cert-manager logs and the end-to-end ginkgo output).
Thank you Irbe!
/lgtm


Enable kebab-case component names

As of now, components can only be loaded in PascalCase. To enable kebab-case component names some tweaks need to be implemented.
Works today
<NuxtDynamic name="SomeComponent" />

Will work when feature is implemented
<NuxtDynamic name="some-component" />

This feature is implemented with v.1.0.4. Issue closed


Promethus: Unable to monitor kube-scheduler, kube-proxy and kube-contoller-manager components

Issue Description
kube-prometheus-stack  helm default installation unable to discover targets of some of the kubernetes components:

kube-scheduler
kube-proxy
kube-controller-manager
kube-etcd

The solution implies:

Change K3S configuration to make kube-scheduler, proxy and controller processes to be bound to the IP address of the master node, instead using the default: <IP_ADDRESS>.
Modify Kube-prometheus-stack chart configuration to establish the IP address of the master node in the ServiceMonitoring section.

See how to solve this issue here:
https://github.com/k3s-io/k3s/issues/3619

Worker node outage after applying changes specified in k3s-io/k3s#3619
Changes applied:

Additional k3s server arguments to make the components bind to node IP address
 --kube-controller-manager-arg 'bind-address=<IP_ADDRESS>'
 --kube-controller-manager-arg 'address=<IP_ADDRESS>'
 --kube-proxy-arg 'metrics-bind-address=<IP_ADDRESS>'
 --kube-scheduler-arg 'bind-address=<IP_ADDRESS>'
 --kube-scheduler-arg 'address=<IP_ADDRESS>'

Adding end-points to Prometheus ServiceMonitoring configuration of the kube components.  Only etc metrics scrape is disabled (K3S is using embedded sqllite database)
kubeApiServer:
     enabled: true
   kubeControllerManager:
     enabled: true
     endpoints:
     - <IP_ADDRESS>
   kubeScheduler:
     enabled: true
     endpoints:
     - <IP_ADDRESS>
   kubeProxy:
     enabled: true
     endpoints:
     - <IP_ADDRESS>
   kubeEtcd:
     enabled: false

When the new targets are discovered Prometheus memory consumption increases a lot eventually producing a node outage for lack of memory.
There is a known issue in K3S: https://github.com/k3s-io/k3s/issues/2262, where duplicated metrics are emitted by the three components (kube-proxy, kube-scheduler and kube-controller-manager).
This behaviour can be checked scraping the metrics using curl command
10249 - kube-proxy
10251 - kube-scheduler
10252 - kube-controller-manager
curl localhost:10249/metrics
Duplicated metrics might cause higher memory consumption in Prometheus. As the proposed solution in https://github.com/k3s-io/k3s/issues/2262 for Rancher Monitoring is to activate only the service monitoring of one of the components, this way all metrics will be collected (no duplicates).

Changing helm chart configuration to just enabling one of the services makes prometheus not suffer any memory increase but the resulting configuration is not complete (only grafana dashboards of the enabled component is created)
Applying for example:
      kubeControllerManager:
        enabled: false
      kubeScheduler:
        enabled: false
      kubeProxy:
        enabled: true

Prometheus-kube-stack helm chart create all the required resources (headless services , servicemonitor, and grafana dashboards) for monitoring kube-proxy but not the resources for monitoring the rest of components (headless services, service monitoring).
Solution: Keep the configuration for monitoring these kube componentes out of kube-prometheus-stack control, disable the monitoring of these components within Chart configuration, and create the required resources (headless service, service monitor and grafana dashboards)  for monitoring them: One single headless service (k3s metrics endpoint), one single serviceMonitor,  and 3 configMaps containing grafana dashboards (one per component)


CORE-6554 clean up validation error messages

Looks ok generally. It looks like a couple of tests are still failing.

These were fixed, just tests that verified specific exception types were used which I'd changed.

Also #2030 just been merged by @benmcmahon100

Looks ok generally. It looks like a couple of tests are still failing.

Resolved.


Avoid the need for @Schema decorator

While doing the decorators code refactory I noticed that the GraphQL schema cold be defined on the queries/mutation/subscription directly (similar to how routing-controllers project works) by adding the @Query decorator directly into query fields.
This approach goes against the current approach where everything is a @Field or  @ObjectType and some are queries defined in the schema elsewhere, making it harder to visualize what the schema actually is and IMO being less declarative than it could possibly be.
A proposed approach would be to get rid of the @Schema decorator in favor of loading all classes with decorators from a specific folder (routing-controllers example) or passing an array of classes (routing-controllers example)
An example schema definition would be

// TypeScript definitions

// Output type
@ObjectType()
export class Foo {
    bars: string;
}

@ObjectType()
export class MyQuery {
    @Query({ type: Foo } )
    AQuery() : Promise<Foo> {
     // return Foo
    }

    @Mutation({ type: Foo } )
    AMutation(
          @Arg() aValue: string,
    ): Promise<Foo> {
     // return Foo
    }
}

// Resulting GraphQL Schema

type Foo {
  bars: String
}

type Query {
  AQuery(): Foo
}

type Mutation {
  AMutation(aValue: String): Foo
}

schema {
  query: Query
  mutation: Mutation
}

This would be a breaking change but would avoid having to create a file with the @Schema manually referencing  all queries and mutations.
So my question is if this a reasonable (breaking) change for a 1.0.0 release?

I prefer this proposed approach. Though we will have a huge breaking change, it simplify things a lot.

I agree. 99 out of 100 times I updated some schema, I did forget to update the correspondent @Schema class.

@babbarankit Good point! I had on this on y mind but forgot to add to the issue 👍


mapping changes for ignite JS releases

Docs Build status updates of commit 5f9b7cd:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/speech-service.md
:warning:Warning
View (azure-node-latest) View (azure-node-preview)
Details

docs-ref-mapping/reference.yml
:white_check_mark:Succeeded

preview-packages/docs-ref-mapping/reference.yml
:white_check_mark:Succeeded

docs-ref-services/speech-service.md

[Warning] Uid(microsoft-cognitiveservices-speech-sdk) has already been defined in docs-ref-services/speech-service.md.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Log improvements

[ ] don't strip the Lambda logs (as they contain timeouts etc)

How do I show logs from "Client Errors" in production stage? That has stumped me.

Oops, I was omitting the stage field from the log output (f870ca2). They all go to the same CloudWatch group for now (later might be per-stage).
For now filtering with something like:
$ up logs -s 50h 'stage = "development" status >= 400'


Weight-Agnostic-Neural-Networks (in Python)

https://github.com/yuedajiong/super-brain-weight-agnostic-neural-networks

Hi!
If you are implementing WANNs yourself, please leave all issues in the correct repository -- this is meant for questions on the paper or template.


Prettier Configuration Clashes with the one on my project

Hello Guys,
I am attempting to implement svgr with Node.js. I am getting the following error:
TypeError: Super expression must either be null or a function
    at _inherits (/Users/fbuenano/Documents/GitHub/aviator-components/node_modules/prettier/index.js:1945:11)

The reason why this is happening is that my prettier configuration is clashing with the one used by svgr. Is there any way I could prevent svgr from running (or including) prettier?
This is how my current svgr configuration looks:
const { default: svgr } = require('@svgr/core');
...

const rawFile  // svg string.
const componentName = 'myComponent';
svgr(rawFile,
        { icon: true },
        { componentName }))
      .then((jsCode) => writeToFile(jsCode, fileName))
...

Thanks!

Hello again guys,
I changed my configuration to this:
svgr(rawFile,
        { icon: true },
        { prettier: false, componentName }))
      .then((jsCode) => writeToFile(jsCode, name))

I believe that is all I needed. I am going to test a bit before I close this issue.
Thanks.

Hello Guys, I spoke too soon.
The reason why I was able to bypass the error, was because I had commented out a couple of lines in my dependency directory (node_modules/@svgr/core) to be exact.
These are the lines I commented out in order to bypass the error:
https://github.com/smooth-code/svgr/blob/master/packages/core/src/convert.js#L3
https://github.com/smooth-code/svgr/blob/master/packages/core/src/convert.js#L17

@fesebuv can you provide a reproducible environment?


Help

As shown here, I write the token and chat id of the bot I created with botfather, but when I test it, I get this error, can you tell me how to do it?

Can you type this in cmd curl https://api.telegram.org > nul and then try again.

Also I think your token is not correct. A typical bot token is like [numbers]:[letters]. Are you sure you copied it from botfather and didn't make any changes to it?

I am doing it in accordance with the format here, botfather gives a result like below

Can you type this in cmd curl https://api.telegram.org > nul and then try again.

it didn't work

 I am doing it in accordance with the format here, botfather gives a result like below

You have to paste full token there. Including the number at the beginning.

shouldn't I do this?
000000000$g22923GASKDNMJKNJ23JNNJ2NJDSNDSD
When I do it this way and press the test button, I get the error unable to connect. Maybe I'm doing something wrong, can you send a message showing how to do it as an example?

Example:<PHONE_NUMBER>:AAHbnjC4IuyyG-97w8riT3gjIbvFUqgGe58$-934369079
It should be like this, the one you put there is missing the ID at the beginning. Make sure you paste the exact token there as the one botfather sent you.


Expose Descriptor mappings

Would it be possible to expose mapping descriptors in 1.x and 2.x. In our case I want to extend the fluent mappings to add support for aliases (local alias map) but also get access to analyzer info so we can pass a map to our query builder to optimize (term vs query string builders). Right now all of this is internal.

Can you provide an example?  I'm all for opening the client up for extension where possible but need to get a better understanding of what you're proposing here.

I want to do something like this .String(f => f.Name(e => e.Name).IndexName(Fields.Name).Index(FieldIndexOption.Analyzed).AddAlias("Blah", _alias))
Which takes a look at the index name and allows me to create an alias map. Alternatively if I could get access to the raw mapping, I could create a visitor and create these much easier. I'd also like to create a map of all the analyzed fields.

Was thinking it would be cool if you could get access to the mapping and also have like a bag on the mapping itself (that wasn't sent to elastic, but allowed me to add meta data to the mapping and I could then view it later (via a visitor). Then I could just do .String(f => f.Name(e => e.Name).IndexName(Fields.Name).Index(FieldIndexOption.Analyzed).AddAlias("custom_name").AlwaysUseMatchQuery()).

@gmarz exposing the mapping configuration is useful in a lot of ways. Probably the biggest way would be to allow someone to build a proper LINQ provider without having to duplicate the mapping configuration like this project had to do: https://github.com/CenturyLinkCloud/ElasticLINQ

Discussed in slack with @ejsmith and @niemyjski our MappingVisitor especially in its 2.x incarnation suffices here.


Redact Kubernetes Secret data

This PR changes the Kubernetes data gatherer so that when Secret resources are gathered their data is removed.
Closes #129

This looks perfect
/lgtm


There is no get_videos method in conversation.py

It seems that the video conversation not implemented yet? In gradio_web_server.py there are several calling for state.get_videos(), but there is none in class Conversation

Same question

Thanks for this report. We add this function in Conversation. Please pull the code and try it again.

Please open the issue again if you have further questions.


Server crash on startup

is this TeaStory not working on a server??
https://pastebin.com/nxCKU0FU
that was TeaStory-<IP_ADDRESS>9-1.12.2.jar
rolled back to TeaStory-<IP_ADDRESS>1-1.12.2.jar and all is good again ...

There is a bug I confirmed just now. It is to be fixed next version.

The version <IP_ADDRESS>0-1.12.2 is available.


How can I disconnect the Wifi?

After I see the code, I see there is only support for scan, status, connect, and kill. So how can i disconnect the wifi or I need to modify the code myself?

@soxipi you are welcome to fork and make a version that suits your requirements.


ctrl-o in Ubuntu always opens "Recently Used" in file chooser

Using keyboard shortcut ctrl-o in Ubuntu launches open file dialog with "Recently used" as default folder. The default in basically all other apps (including Sublime Text) is to open the currently open file's folder as default.
Yes, the tree view is there to quickly open files from current directory, but breaking convention is bad.

How do you go about debugging this?  I wasn't able to get console.log or window.console.log to show up in the developer console (has to do with nslog?).  Also I couldn't get any errors to show up when I threw them explicitly.

After trial and error, I realized my problem was projectPath not being set.  This seems to be an issue with AtomWindow.projectPath being removed which I assume was part of the #770 fix (support multiple open projects).  Still, I remain stuck due to not having an efficient way to debug.

I would appreciate any debugging tips - I would really like to borrow this code in creating a fix.

Opening the Recently Used dialog via ctrl-o on Ubuntu 14.04 and 12.04 turns out to be a serious drawback for me (Atom 1.0.10). I open find myself navigating to the folder of the currently open file again. I am not sure if I follow the argument above, but gedit also opens the folder of the currently open file, not Recently Used. In case kevinsawicki post above from 25-09-2014 should indicate that the behavior should now be 'open current file directory', then maybe this issue needs to be reopened?
If it was a design choice to stick to platform standards, maybe there should be a preferences setting for this?

FYI, I reopened this issue in May as #6762


refactor: remove webpack-command from CLIs

What kind of change does this PR introduce?
refactoring
Did you add tests for your changes?
no required
Does this PR introduce a breaking change?
no
What needs to be documented once your changes are merged?
no
After merge need patch release and we archived webpack-command

A lot of other code could be refactored

@montogeek why?

@evilebottnawi Ok, sounds good

@edmorley long term is webpack-cli, webpack-command is will be proof of concept and many features already implemented in webpack-cli, some in WIP


Closes #1409 - NTR choroidal melancoyte

Added new class 'choroidal melancoyte'

I believe that the SOP is to represent requestors via their ORCID as a definition dbxref (annotation of the definition), so, separately from whoever created the term. Sometimes they'll be the same person.
Re. creation date, I find it useful when archeological digging of terms is called for - the ID range and/or created_by etc. can tell you who created the term, but sometimes it's really helpful to see at a glance which of two terms is older, e.g. when considering merges.


Check network tutorial

https://forums.ghielectronics.com/t/is-tutorial-networking-interfaces-wifi-still-up-to-date-for-tinyclr-v2-1-0-preview-4/23711

no issue found

however we can change the string to use the "pins" NuGet as this is the proper use

Updated in PR #537


Lua script non-atomicity needs to be clearly documented

From redis documentation:
Atomicity of scripts
Redis uses the same Lua interpreter to run all the commands. Also Redis guarantees that a script is executed in an atomic way: no other script or Redis command will be executed while a script is being executed. This semantics is very similar to the one of MULTI / EXEC. From the point of view of all the other clients the effects of a script are either still not visible or already completed.
However this also means that executing slow scripts is not a good idea. It is not hard to create fast scripts, as the script overhead is very low, but if you are going to use slow scripts you should be aware that while the script is running no other client can execute commands since the server is busy.

If ardb does not properly implement this critical feature of lua scripting (see the response here: https://github.com/yinqiwen/ardb/issues/154), then that limitation needs to be clearly documented. It's not reasonable to say
Full redis-protocol compatibility

without further documenting the limitations of lua scripting.
This spot seems like a reasonable location to add the documentation:
https://github.com/yinqiwen/ardb/wiki/ARDB-Commands.
I'm guessing the same issue that exists for multi/exec. If that's true, that should be documented as well.

There is a global lock for exec, so no such issue  for multi/exec.
https://github.com/yinqiwen/ardb/blob/master/src/command/transaction.cpp#L74

I'll add a global lock for lua script too.

Added global lock for lua interceptor.


pandoc cvLanguage command

pandoc can't convert /cvlanguage to html

UPD: let's say we need a script/command to install packages from CTAN.
then we need to install moderncv  packge from CTAN.


Multiple sort update v4

sortRecord fix
[中文版模板 / Chinese template]
🤔 This is a ...

[ ] New feature
[x] Bug fix
[ ] TypeScript definition update
[ ] Bundle size optimization
[ ] Performance optimization
[ ] Enhancement feature
[ ] Refactoring
[ ] Update dependency
[ ] Code style optimization
[ ] Test Case
[ ] Branch merge
[ ] Site / documentation update
[ ] Demo update
[ ] Workflow
[ ] Chore
[ ] Release
[ ] Other (about what?)

🔗 Related issue link
https://github.com/VisActor/VTable/pull/2316
💡 Background and solution
📝 Changelog

Language
Changelog

🇺🇸 English

🇨🇳 Chinese

☑ Self-Check before Merge
⚠ Please check all items below before requesting a reviewing. ⚠

[ ] Doc is updated/provided or not needed
[ ] Demo is updated/provided or not needed
[ ] TypeScript definition is updated/provided or not needed
[ ] Changelog is provided or not needed

🚀 Summary
copilot:summary
🔍 Walkthrough
copilot:walkthrough

@AntonPolyakin still has problems，check message leave in this pr.  https://github.com/VisActor/VTable/pull/2316


Permissiom problem

I was using permission, but I face this error: Call to a member function isRole() on null.
You Now how to solve this?

How do you use this？show your code?

Hello z-song, tanks for replying me. I Give up the first way I was using. Now, I'm using the middleware in route and this works perfect for my project.

It seems I also miss something and I get the same error.
My admin_users table:
+----+----------+--------------------------------------------------------------+---------------+--------+--------------------------------------------------------------+---------------------+---------------------+
| id | username | password                                                     | name          | avatar | remember_token                                               | created_at          | updated_at          |
+----+----------+--------------------------------------------------------------+---------------+--------+--------------------------------------------------------------+---------------------+---------------------+
| 1  | admin    | $2y$10$yuRxufCpbCRrEOVB89QmyeE/Dhh.adDABB2wl1OCb51fY5cThSg/C | Administrator |        | 6R4D3TQVDbud73D6mzqAG7uQ9R5HlIBo06jrhTi1cEHQgLHKsNjOLj4ErI1x | 2017-02-24 21:40:37 | 2017-02-24 21:40:37 |
| 2  | manager  | $2y$10$WSWU5l6mFJk9Ihj39fdcDuoBP6iwu9sMEv.PUWCRjLThWZ0dZmCJ. | Manager       |        |                                                              | 2017-03-02 15:10:35 | 2017-03-02 15:11:11 |
| 3  | user     | $2y$10$z2G/DHCnD5QfPPj.oqxVN.GB56fHbqYiZ6XKpkeSQRs.PVbC884UO | User          |        |                                                              | 2017-03-02 15:11:32 | 2017-03-02 15:11:32 |
+----+----------+--------------------------------------------------------------+---------------+--------+--------------------------------------------------------------+---------------------+---------------------+

My admin_roles table:
+----+---------------+---------------+---------------------+---------------------+
| id | name          | slug          | created_at          | updated_at          |
+----+---------------+---------------+---------------------+---------------------+
| 1  | Administrator | administrator | 2017-02-24 21:40:37 | 2017-02-24 21:40:37 |
| 2  | Manager       | manager       | 2017-03-02 15:08:29 | 2017-03-02 15:08:29 |
| 3  | Contributor   | contributor   | 2017-03-02 15:09:31 | 2017-03-02 15:09:31 |
+----+---------------+---------------+---------------------+---------------------+

My admin_role_users table:
+---------+---------+------------+------------+
| role_id | user_id | created_at | updated_at |
+---------+---------+------------+------------+
| 1       | 1       |            |            |
| 2       | 2       |            |            |
| 3       | 3       |            |            |
+---------+---------+------------+------------+
And I want deny access for 'contributor' group in my controller. So according to doc, my controller looks like:
namespace App\Admin\Controllers;

use App\User;

use Encore\Admin\Form;
use Encore\Admin\Grid;
use Encore\Admin\Facades\Admin;
use Encore\Admin\Layout\Content;
use App\Http\Controllers\Controller;
use Encore\Admin\Controllers\ModelForm;
use Encore\Admin\Auth\Permission;

class MyTestController extends Controller
{
    use ModelForm;

    public function __construct()
    {
        Permission::deny('contributor');
    }
(...)
# other functions like index(), edit(), create()
(...)
}

The menu item visibility set under admin/auth/menu works as expected, however if I type url which should show index page handled by MyTestController (or should not for contributor group) I get error 500 with message: Call to a member function isRole() on null in MYAPP/vendor/encore/laravel-admin/src/Auth/Permission.php:90.
Any ideas what I missed?

Ref https://laravel-news.com/controller-construct-session-changes-in-laravel-5-3
Use:
    public function __construct()
    {
        $this->middleware(function () {
            Permission::deny('contributor');
        });
    }

After applying those changes now I get:
ErrorException in VerifyCsrfToken.php line 156:
Trying to get property of non-object

However, if I use http://z-song.github.io/laravel-admin/#/en/permission.md?id=permission-middleware then I get:
FatalThrowableError in Permission.php line 90:
Call to a member function isRole() on null

According to https://laravel.com/docs/5.4/controllers I replaced constructor with:
$this->middleware(function ($request, $next) {
    Permission::deny('contributor');
    return $next($request);
});

And voilà!

I'll test this solution. Thanks for share this

厉害了，郁闷了好长时间，原来答案在这儿


Class "Symfony\Component\Serializer\Normalizer\ObjectNormalizer" not found

Before you submit this issue, you has been search all existed issues and search the documentation

[yes] I've been search all existed issues
[yes] I've been read all documentation

Describe your question
在tangwei\dto\src\Serializer\ObjectDtoNormalizer.php里面找不到Symfony的ObjectNormalizer这个类

安装进来啊


In IE11, all the animations disappear

In IE11, all the animations disappear:
the page height is 2000, the canvas is at the top of the page and the height is 1000.
When the page moves to the bottom for 10 seconds and then moves back to the canvas,
all the animations in Pixi 4.5.1 disappear.
Pixi4.6.0 PIXI.sprite animation exists, PIXI.mesh.Rope still disappeared.

Hi @CrackRen , sorry it's taken so long to get back to you. Gonna close this issue as it's so old and I'd have hoped you'd have solved it by now! If not, please do open a new issue with a jsfiddle or something that we can use to try and help you with


Update FG and Gradle

Using anatawa12's ForgeGradle-2.3 fork and update Gradle to 6.

where rfg

If possible, updating to use RetroFuturaGradle would be better, template here if you need any guidance!

Can I use GTCEu's Buildscripts?

If you're comfortable with it, go ahead!

I created PR https://github.com/CleanroomMC/BetterQuesting/pull/90

#90 now supersedes this. Closing.


Potential gvm integration problem with vscode?

I installed go1.13 via:
gvm install go1.13 -B
I've installed 3 versions in total:
$ gvm list

gvm gos (installed)

   go1.10
   go1.12
=> go1.13

Then I changed the gopath and goroot in settings.json:

  "go.gopath": "/Users/xuzhiqiang/.gvm/pkgsets/go1.13/global",
  "go.goroot": "/Users/xuzhiqiang/.gvm/gos/go1.13",

The "run test" works, but "debug test" fails:
# runtime/internal/atomic
compile: version "go1.13" does not match go tool version "go1.12"
# github.com/google/go-cmp/cmp/internal/diff
compile: version "go1.13" does not match go tool version "go1.12"
# math/bits
compile: version "go1.13" does not match go tool version "go1.12"
# unicode/utf8
compile: version "go1.13" does not match go tool version "go1.12"

Not sure if it's a bug of gvm, or vscode?

gvm use go1.13 --default should allow vscode to find the correct version installed with GVM.


build failure (main, zig 0.11.0)

First of all, thanks for this project!
I just cloned and tried to build this project, but it failed. Exactly the same output as the latest pipeline:
https://github.com/ZigEmbeddedGroup/microchip-atmega/actions/runs/5985749765/job/16238164264
> zig version
0.11.0

> zig build -Doptimize=ReleaseSmall
[...]
 (I am using Linux)
Do you know which flags I could set to get more info regarding the error? (zig build --verbose does not show anything more)
Error looks similar to https://github.com/ZigEmbeddedGroup/microzig/issues/119 (reported long ago)

It does not compile, it's an LLVM bug we need to work around. I made everything else up-to-date, but we have to re-engineer compiler_rt to compile for AVR.
I need to write some words on how we get better AVR support in zig. There is a plan, but it needs tk be documented

Ok, thank you for the quick reply.
Shall I send a PR to update the README with a warning that it currently does not compile and a link to this issue?

Or better, linking to https://github.com/ZigEmbeddedGroup/microzig/issues/42


Unreachable code reached in server with invalid Content-Length header

This test doesn't pass on 0.10.x:
diff --git a/src/server/request.rs b/src/server/request.rs
index c632d27..2c05e72 100644
--- a/src/server/request.rs
+++ b/src/server/request.rs
@@ -304,4 +304,22 @@ mod tests {
         assert_eq!(read_to_string(req).unwrap(), "1".to_owned());
     }

+    /// Tests that when an invalid Content-Length header doesn't crash the
+    /// server.
+    #[test]
+    fn test_invalid_content_length() {
+        let mut mock = MockStream::with_input(b"\
+            POST / HTTP/1.1\r\n\
+            Host: example.domain\r\n\
+            Content-Length: x\r\n\
+            \r\n\
+            I'm a bad request.\r\n\
+        ");
+
+        // FIXME: Use Type ascription
+        let mock: &mut NetworkStream = &mut mock;
+        let mut stream = BufReader::new(mock);
+
+        let _ = Request::new(&mut stream, sock("<IP_ADDRESS>:80"));
+    }
 }

Apparently, older versions of cURL do this.

That's unfortunate, I don't remember why at all.
What server is running this? hyper 0.10 ended 2 years ago, I'm sure there's other bugs floating in there too ;_;

hyper 0.10 ended 2 years ago

What do you mean by this? There was a release in April.

Ah, so there was! I meant 0.11 came out 2 years ago (and 0.12 a year ago). I don't have any time to maintain 0.10 myself. I can publish small bug fixes that are sent.


[ENHANCEMENT] Grafana migration: use new CUE evaluator to improve memory usage

Description
This PR bumps the CUE dep to v0.11.0-alpha3 to allow enabling the new "EvalV3" evaluator from CUE, with the latest improvements brought to it.
By doing so we were hoping to solve the memory leak issue we encounter when migrating dashboards (https://github.com/perses/perses/issues/2039). Unfortunately it doesn't, the memory usage still increase after each migration & doesnt drop before several minutes.
Still, with evalv3 the memory used is much less than before (you also see the difference in response time in the UI of course):

When migrating a big dashboard (23k lines-long json): ~15GB with old evaluator vs ~3GB with the new one.
When migrating the blackbox exporter dashboard attached here: ~3.7GB with the old evaluator vs ~175MB with the new one.
When migrating the haproxy dashboard attached here: ~7.2GB with the old evaluator vs ~150MB with the new one.

[!WARNING]
Side effect of this change: the new evaluator is still under development & got some bugs. Regarding our usage of CUE for grafana migration, using it unfortunately breaks some of the logic we had for the Table panel migration. Therefore I disabled this part of the migration logic to still get the benefits of the memory usage reduction - at the cost of losing the table's overrides previously supported.

Checklist

[x] Pull request has a descriptive title and context useful to a reviewer.
[x] Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the
following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, DOC,IGNORE.
[x] All commits have DCO signoffs.

maybe we should keep the dashboard that contains the table to use them to test it back the migration when it will be fixed for the table ?

maybe we should keep the dashboard that contains the table to use them to test it back the migration when it will be fixed for the table ?

yeah ok I can try to bring the old logic that was revamped with https://github.com/perses/perses/pull/2273


[T5A1][T16-A1]Lai Zhiyuan

Ready for review

reviewed in class :-)
quite an extensive list of user stories and NFRs. the use case needs a bit of fixing, please recollect the discussion during the tutorial.
good job!


Appstudio update test-component-pac-kpon

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-kpon-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
2 minutes
clone-repository

✅ Succeeded
46 seconds
build-container

✅ Succeeded
19 seconds
deprecated-base-image-check

✅ Succeeded
20 seconds
sanity-inspect-image

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
35 seconds
clamav-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
8 seconds
sanity-optional-label-check

✅ Succeeded
7 seconds
sanity-label-check

✅ Succeeded
5 seconds
show-summary


[hi] Footer "होम" now reads "प्रलेखन"

Resolves Hindi localization part of Issue #43068

/retitle [hi] Footer "Home" now reads "Documentation"

The footer is already showing the required content per another PR. Screencap attached below.

Closing the PR since it is no longer required.
/close


cu_tiedtke.f90 divide by zero

I'm tracking down whether this is a problem with the input data or the code itself. ICAR was built with MODE=debugslow on Intel compilers. About 30% of the way through the first time step an error forrtl: error(73): floating divide by zero occurs.  The domain is situated over Idaho.
Physics Options:
pbl = 2 ! Simple
lsm = 3 ! Noah
water = 2 ! simple
mp = 2 ! simple
rad = 2 ! simple
conv = 1 ! Tiedke Scheme
adv = 1 ! upwind
wind = 1 ! linear theory

The traceback provides the following (edited) information:
Routine             Line               Source
module_cu_tiedtke    2923    cu_tiedtke.f90
module_cu_tiedtke    1211    cu_tiedtke.f90
module_cu_tiedtke     648     cu_tiedtke.f90
module_cu_tiedtke     443     cu_tiedtke.f90
convection_mp_con    153     cu_driver.f90
time_step_mp_step     477     time_step.f90
MAIN__                         90     driver.f90

Hi Nick,
I believe that generally when I have seen errors in the core physics from WRF, it is often a problem with the inputs.  One thing that jumps to mind is the order of the vertical dimension.  Do you have 1000hPa (or 0m ASL or AGL) as the first level in the vertical dimension?  If it is the last level the code will break.
I would recommend that while you are still testing, you may want to turn off most of the physics parameterizations to make it as simple as possible.  You could even set wind=0 to turn off the linear theory perturbation.
pbl = 0 ! Simple
lsm = 0 ! Noah
water = 0 ! simple
mp = 2 ! simple
rad = 0 ! simple
conv = 0 ! Tiedke Scheme
adv = 1 ! upwind
wind = 1 ! linear theory

On Jul 17, 2017, at 12:15 PM, Nick Dawson<EMAIL_ADDRESS>wrote:
I'm tracking down whether this is a problem with the input data or the code itself. ICAR was built with MODE=debugslow on Intel compilers. About 30% of the way through the first time step an error forrtl: error(73): floating divide by zero occurs. The domain is situated over Idaho.
Physics Options:
pbl = 2 ! Simple
lsm = 3 ! Noah
water = 2 ! simple
mp = 2 ! simple
rad = 2 ! simple
conv = 1 ! Tiedke Scheme
adv = 1 ! upwind
wind = 1 ! linear theory
The traceback provides the following (edited) information:
Routine             Line               Source
module_cu_tiedtke    2923    cu_tiedtke.f90
module_cu_tiedtke    1211    cu_tiedtke.f90
module_cu_tiedtke     648     cu_tiedtke.f90
module_cu_tiedtke     443     cu_tiedtke.f90
convection_mp_con    153     cu_driver.f90
time_step_mp_step     477     time_step.f90
MAIN__                         90     driver.f90
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/NCAR/icar/issues/34, or mute the thread https://github.com/notifications/unsubscribe-auth/ADOgXtDak_AGxGrqKwzE-0PlDJJTomONks5sO4Y3gaJpZM4OaJx5.

Using the simplified options (along with wind=0), the model ran. Wind values are provided for all time steps but after the first time step the ta2m output is blank (missing values).
The problem must be the input files. Input files are created from CFSv2 data in WPS. Two runs of ungrib are needed to provide the 3D data and the surface data. I've added variables to the ungrib/metgrid tables to provide the necessary inputs for icar. Looking at the "Z" field, the first metgrid level ranges between 600-1300 gpm. But the next level starts at ~100 gpm and increases monotonically up to ~45,000 gpm thereafter.
ICAR does provide a warning, "Forcing Z levels are below the terrain provided. For pressure level forcing data this is expected, but not well tested."

Hi Nick, have you made any more progress on this?  Does it work with other input files so we can close this?

I wasn't able to get things working and moved on to testing MPAS. I will give it another shot tomorrow and see if I can get ICAR running with CFSv2 inputs.

OK, if you have a minimal forcing dataset you can post that reproduces this error, let me know and I'll take a look at it.

What is the minimal forcing needed in order run ICAR with nearly all parameterizations turned off? Is it simply a wind field, lat(u,v)/lon(u,v), Z, and heights? The current CFSv2 inputs are too large (217MB) to share via github. I assume the &var_list can also be trimmed down with the following &physics options?
Here's what I mean when I say turn off parameterizations
&physics
pbl = 0
lsm = 0
water = 0
mp = 2
rad = 0
conv = 0
adv = 1
wind = 0
https://github.com/NCAR/icar/issues

You also need (potential) temperature, water vapor and pressure. 217MB isn’t very large though, I can set up an FTP site for you to upload to, I’ll email directly with an ftp site.
ethan

On Aug 1, 2017, at 10:10 AM, Nick Dawson<EMAIL_ADDRESS>wrote:
What is the minimal forcing needed in order run ICAR with nearly all parameterizations turned off? Is it simply a wind field, lat(u,v)/lon(u,v), Z, and heights? The current CFSv2 inputs are too large (217MB) to share via github. I assume the &var_list can also be trimmed down with the following &physics options?
Here's what I mean when I say turn off parameterizations
&physics
pbl = 0
lsm = 0
water = 0
mp = 2
rad = 0
conv = 0
adv = 1
wind = 0
https://github.com/NCAR/icar/issues https://github.com/NCAR/icar/issues
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/NCAR/icar/issues/34#issuecomment-319418451, or mute the thread https://github.com/notifications/unsubscribe-auth/ADOgXkt33kqcmypsXCbrzvVoQiTxdXYoks5sT04KgaJpZM4OaJx5.

I decreased the grid spacing of the forcing files so they are much smaller now, but I will still need to send via FTP.
Nick Dawson, Ph.D.
ATMOSPHERIC SCIENTIST
Idaho Power | Power Supply
Work<PHONE_NUMBER>
Mobile<PHONE_NUMBER>
From: Ethan Gutmann<EMAIL_ADDRESS>Sent: Tuesday, August 01, 2017 10:34 AM
To: NCAR/icar<EMAIL_ADDRESS>Cc: Dawson, Nick<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [NCAR/icar] cu_tiedtke.f90 divide by zero (#34)
You also need (potential) temperature, water vapor and pressure. 217MB isn’t very large though, I can set up an FTP site for you to upload to, I’ll email directly with an ftp site.
ethan

On Aug 1, 2017, at 10:10 AM, Nick Dawson <notifications<EMAIL_ADDRESS>wrote:
What is the minimal forcing needed in order run ICAR with nearly all parameterizations turned off? Is it simply a wind field, lat(u,v)/lon(u,v), Z, and heights? The current CFSv2 inputs are too large (217MB) to share via github. I assume the &var_list can also be trimmed down with the following &physics options?
Here's what I mean when I say turn off parameterizations
&physics
pbl = 0
lsm = 0
water = 0
mp = 2
rad = 0
conv = 0
adv = 1
wind = 0
https://github.com/NCAR/icar/issues https://github.com/NCAR/icar/issues
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/NCAR/icar/issues/34#issuecomment-319418451, or mute the thread https://github.com/notifications/unsubscribe-auth/ADOgXkt33kqcmypsXCbrzvVoQiTxdXYoks5sT04KgaJpZM4OaJx5.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/NCAR/icar/issues/34#issuecomment-319425250, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AcBLKtV9Fr0bt5komeJELBZjUFwlYhZIks5sT1OAgaJpZM4OaJx5.

This transmission may contain information that is privileged, confidential and/or exempt from disclosure under
applicable law.  If you are not the intended recipient, you are hereby notified that any disclosure, copying,
distribution, or use of the information contained herein (including any reliance thereon) is STRICTLY PROHIBITED. If
you received this transmission in error, please immediately contact the sender and destroy the material in its
entirety, whether in electronic or hard copy format.  Thank you.

Given how many other things have changed since this was opened, I'm going to close this for now.  We can always come back to this with a test case in the future.


🌐 Add Turkish translation for docs/tr/docs/deployment/index.md

🌐 Add Turkish translation for docs/tr/docs/deployment/index.md
Original File
Discussion: #9193

📝 Docs preview for commit 16ed179f3aac9e479b626ff5902e27470ee8ae40 at: https://1d19be67.fastapitiangolo.pages.dev

📝 Docs preview for commit 40363ae45a79b71410f64a2db2b21fd270966ce0 at: https://9ae3708f.fastapitiangolo.pages.dev

I had some suggestions about the general structure of the translation. Of course, it isn't necessary to change it according to my suggestions. Besides those suggestions, the translation looks nice and clean. Thanks for the hard work!

Thank you for the review 🙏, I have committed your change requests and I think it looks far better now 🤓.

📝 Docs preview for commit 675fcb99662c3018e1364fd2635fc5148864b647 at: https://bcb4d7b0.fastapitiangolo.pages.dev

📝 Docs preview for commit 2423be69d47aeb960ad8bdf3b6a61529707b2657 at: https://5a3a34f5.fastapitiangolo.pages.dev

Thank you! nice job @hasansezertasan :partying_face:
Thanks for the review @alperiox :v:


Question: 集群环境 增加master失败

当前集群环境 , 2 master 1 node
root@master1:~/.sealos/default# kubectl get pod -A
NAMESPACE         NAME                                       READY   STATUS    RESTARTS       AGE
calico-system     calico-kube-controllers-6b44b54755-b4fvq   1/1     Running   14 (18m ago)   18h
calico-system     calico-node-g9gjl                          1/1     Running   1 (33m ago)    18h
calico-system     calico-node-nw92w                          1/1     Running   2 (32m ago)    18h
calico-system     calico-node-v9rcc                          1/1     Running   2 (31m ago)    18h
calico-system     calico-typha-86c4d6d567-qblhd              1/1     Running   2 (32m ago)    18h
calico-system     calico-typha-86c4d6d567-sxrd4              1/1     Running   1 (33m ago)    18h
kube-system       coredns-6d4b75cb6d-6kpz7                   1/1     Running   2 (32m ago)    18h
kube-system       coredns-6d4b75cb6d-ssfrm                   1/1     Running   2 (32m ago)    18h
kube-system       etcd-master1                               1/1     Running   4 (26m ago)    18h
kube-system       etcd-master2                               1/1     Running   1 (33m ago)    18h
kube-system       kube-apiserver-master1                     1/1     Running   11 (25m ago)   18h
kube-system       kube-apiserver-master2                     1/1     Running   1 (33m ago)    18h
kube-system       kube-controller-manager-master1            1/1     Running   3 (31m ago)    18h
kube-system       kube-controller-manager-master2            1/1     Running   3 (19m ago)    18h
kube-system       kube-proxy-4pq8v                           1/1     Running   1 (30m ago)    18h
kube-system       kube-proxy-4zbpx                           1/1     Running   1 (33m ago)    18h
kube-system       kube-proxy-qdz2r                           1/1     Running   2 (32m ago)    18h
kube-system       kube-scheduler-master1                     1/1     Running   3 (30m ago)    18h
kube-system       kube-scheduler-master2                     1/1     Running   3 (19m ago)    18h
kube-system       kube-sealos-lvscare-node1                  1/1     Running   2 (32m ago)    18h
tigera-operator   tigera-operator-d7957f5cc-qmj4n            1/1     Running   9 (19m ago)    18h

需要增加一个master，执行 sealos add --masters <IP_ADDRESS>
报错如下
<IP_ADDRESS>:22: [download-certs] Downloading the certificates in Secret "kubeadm-certs" in the "kube-system" Namespace
<IP_ADDRESS>:22: error execution phase control-plane-prepare/download-certs: error downloading certs: error downloading the secret: Secret "kubeadm-certs" was not found in the "kube-system" Namespace. This Secret might have expired. Please, run `kubeadm init phase upload-certs --upload-certs` on a control plane to generate a new one
<IP_ADDRESS>:22: To see the stack trace of this error execute with --v=5 or higher
2022-09-22T02:24:13 error Applied to cluster error: exec kubeadm join in <IP_ADDRESS>:22 failed failed to execute command(kubeadm join --config=/root/.sealos/default/etc/kubeadm-join-master.yaml -v 0 --ignore-preflight-errors=SystemVerification) on host(<IP_ADDRESS>:22): output(W0922 02:24:12.357475   29704 initconfiguration.go:120] Usage of CRI endpoints without URL scheme is deprecated and can cause kubelet errors in the future. Automatically prepending scheme "unix" to the "criSocket" with value "/run/containerd/containerd.sock". Please update your configuration!
[preflight] Running pre-flight checks
        [WARNING FileExisting-socat]: socat not found in system path
[preflight] Reading configuration from the cluster...
[preflight] FYI: You can look at this config file with 'kubectl -n kube-system get cm kubeadm-config -o yaml'
[preflight] Running pre-flight checks before initializing the new control plane instance
[preflight] Pulling images required for setting up a Kubernetes cluster
[preflight] This might take a minute or two, depending on the speed of your internet connection
[preflight] You can also perform this action in beforehand using 'kubeadm config images pull'
[download-certs] Downloading the certificates in Secret "kubeadm-certs" in the "kube-system" Namespace
error execution phase control-plane-prepare/download-certs: error downloading certs: error downloading the secret: Secret "kubeadm-certs" was not found in the "kube-system" Namespace. This Secret might have expired. Please, run `kubeadm init phase upload-certs --upload-certs` on a control plane to generate a new one
To see the stack trace of this error execute with --v=5 or higher), error(Process exited with status 1)
2022-09-22T02:24:13 info

不清楚问题在哪里

像是因为没获取到证书

系统信息，sealos 版本 和镜像版本都发一下哈，我看下如何复现

kubeadm-certs has an expiration binding token, there has two method solve
1、re upload kubeadm-certs  agagin on first master bykubeadm init phase upload-certs --upload-certs, then join others node as soon as possible.
2、set kubeadm-certs never expire when first master insalled just in time.
more info see https://github.com/dyrnq/kubeadm-vagrant/issues/11,hope it can help.

@dyrnq sealos add will generate a new token

kubeadm init phase upload-certs

如果token过期是会重新上传证书

For the first time, I immediately added master2, and it was successful. After waiting for an hour to add it, the problem was reproduced.
<IP_ADDRESS>:22: error execution phase control-plane-prepare/download-certs: error downloading certs: error downloading the secret: Secret "kubeadm-certs" was not found in the "kube-system" Namespace. This Secret might have expired. Please, run `kubeadm init phase upload-certs --upload-certs` on a control plane to generate a new one
<IP_ADDRESS>:22: To see the stack trace of this error execute with --v=5 or higher
2022-10-18T11:03:01 error Applied to cluster error: exec kubeadm join in <IP_ADDRESS>:22 failed failed to execute command(kubeadm join --config=/root/.sealos/default/etc/kubeadm-join-master.yaml -v 0 --ignore-preflight-errors=SystemVerification) on host(<IP_ADDRESS>:22): output(W1018 11:02:20.407213   13176 initconfiguration.go:119] Usage of CRI endpoints without URL scheme is deprecated and can cause kubelet errors in the future. Automatically prepending scheme "unix" to the "criSocket" with value "/run/containerd/containerd.sock". Please update your configuration!
[preflight] Running pre-flight checks
        [WARNING FileExisting-socat]: socat not found in system path
        [WARNING SystemVerification]: missing optional cgroups: blkio
[preflight] Reading configuration from the cluster...
[preflight] FYI: You can look at this config file with 'kubectl -n kube-system get cm kubeadm-config -o yaml'
[preflight] Running pre-flight checks before initializing the new control plane instance
[preflight] Pulling images required for setting up a Kubernetes cluster
[preflight] This might take a minute or two, depending on the speed of your internet connection
[preflight] You can also perform this action in beforehand using 'kubeadm config images pull'
[download-certs] Downloading the certificates in Secret "kubeadm-certs" in the "kube-system" Namespace
error execution phase control-plane-prepare/download-certs: error downloading certs: error downloading the secret: Secret "kubeadm-certs" was not found in the "kube-system" Namespace. This Secret might have expired. Please, run `kubeadm init phase upload-certs --upload-certs` on a control plane to generate a new one
To see the stack trace of this error execute with --v=5 or higher), error(Process exited with status 1)

kubeadm init phase upload-certs --upload-certs
I see this is a command of sealctl,looks like execsealctl token will create  certs,how to install sealctl?

@xiao-jay seems we can't reproduce this bug?

For the first time, I immediately added master2, and it was successful. After waiting for an hour to add it, the problem was reproduced.
<IP_ADDRESS>:22: error execution phase control-plane-prepare/download-certs: error downloading certs: error downloading the secret: Secret "kubeadm-certs" was not found in the "kube-system" Namespace. This Secret might have expired. Please, run `kubeadm init phase upload-certs --upload-certs` on a control plane to generate a new one
<IP_ADDRESS>:22: To see the stack trace of this error execute with --v=5 or higher
2022-10-18T11:03:01 error Applied to cluster error: exec kubeadm join in <IP_ADDRESS>:22 failed failed to execute command(kubeadm join --config=/root/.sealos/default/etc/kubeadm-join-master.yaml -v 0 --ignore-preflight-errors=SystemVerification) on host(<IP_ADDRESS>:22): output(W1018 11:02:20.407213   13176 initconfiguration.go:119] Usage of CRI endpoints without URL scheme is deprecated and can cause kubelet errors in the future. Automatically prepending scheme "unix" to the "criSocket" with value "/run/containerd/containerd.sock". Please update your configuration!
[preflight] Running pre-flight checks
        [WARNING FileExisting-socat]: socat not found in system path
        [WARNING SystemVerification]: missing optional cgroups: blkio
[preflight] Reading configuration from the cluster...
[preflight] FYI: You can look at this config file with 'kubectl -n kube-system get cm kubeadm-config -o yaml'
[preflight] Running pre-flight checks before initializing the new control plane instance
[preflight] Pulling images required for setting up a Kubernetes cluster
[preflight] This might take a minute or two, depending on the speed of your internet connection
[preflight] You can also perform this action in beforehand using 'kubeadm config images pull'
[download-certs] Downloading the certificates in Secret "kubeadm-certs" in the "kube-system" Namespace
error execution phase control-plane-prepare/download-certs: error downloading certs: error downloading the secret: Secret "kubeadm-certs" was not found in the "kube-system" Namespace. This Secret might have expired. Please, run `kubeadm init phase upload-certs --upload-certs` on a control plane to generate a new one
To see the stack trace of this error execute with --v=5 or higher), error(Process exited with status 1)

1 hours after , exec kubeadm token list   and cat  ~/.sealos/default/etc/kubeadm-token.yaml

Proxy for managing TTL for the kubeadm-certs secret token is 1h . It seems that this cannot be modified from parameters.


Issues with Table Extraction Accuracy in Sample

Problem Description
When using the sample provided by the llmware project, I've encountered issues with the accuracy of table extractions. Specifically, not all tables are being extracted correctly.
As an example given in the sample, Annual_Report_2003.pdf.
Steps to Reproduce
Run the sample extraction process as in examples. Set query parameter to be empty string.
Review the output and compare it to the expected tables within the documents. Only one table, to be exact, part of the table spreading from page 44-46, got correctly extracted.
Expected Outcome
All tables within the sample documents should be identified and extracted accurately. In this file, tables that are supposed to be extracted are given below as screenshot,

Actual Outcome
Only one csv outputs.
Please refer to the outcome table_0.csv.
All other table contents, though extracted in json file, is labeled as text.
Potential Impact
This issue may lead to incomplete or inaccurate data capture, which can affect the integrity of data analysis and further processing steps.
Request for Assistance
I would appreciate any guidance on how to resolve this issue or any suggested workarounds. Additionally, if there are any plans to improve the table extraction feature in the near future, information on that would also be helpful.

Also, I just dive into the repo for further investigation. It seems like you hide the implementation of method add_pdf_main_llmware. May I get more details about the layout detection model you integrated for your algorithm? Any information about the design choices, data structures, or algorithms used would be very helpful.

Hi @li-rongzhi
Have you fixed the issue? I'm still not getting any tables either.


SQLITE_CORRUPT after syncing

I am currently building a benchmark (benchmark-postgres-vs-libsql) to test the performance of Postgres compared to libSQL (hybrid). The example I'm working on is similar to your own embedded-replica-examples, with the additional feature that the table creation and database seeding also take place within the test. However, I encountered an error while running the current version of @libsql/client (0.3.6). The error occurs after syncing the changes from the remote database:
About to measure the performance with 250 queries ...
280 |     } else {
281 |       rows = statementRowsSync.call(this.stmt, bindParameters.flat());
282 |     }
283 |     const iter = {
284 |       next() {
285 |         const row = rowsNext.call(rows);
                        ^
error: database disk image is malformed
 code: "SQLITE_CORRUPT"

I tested the code with Bun 1.0.7 and Node.js v20.9.0 under Ubuntu 22.04 and macOS 14.0.

@akoenig when was the last time you upgraded the db with turso db update? We had an issue a while ago that got solved with https://github.com/tursodatabase/libsql/pull/467

@psarna Thanks for your answer. I created the database two hours ago.

The link from the description 404's for me, is it up-to-date? Could share the code and steps how to run the project and get to this failure? I'll try with my own db

(and when sharing steps, assume I'm a javascript noob 😇)

this may also be fixed by #505 and #477

If you update to turso db update and also update your clients to the latest released pre version I believe these should be fixed. Feel free to reopen if not.


Cannot bind PasswordBox's Password property

Describe the bug
I cannot bind the PasswordBox's Password property to a static string (for example from Properties.Settings.Default). Error is :
Cannot define 'Binding' of 'Password' property of type 'PasswordBox'. You can only define a binding on an DependencyObject 's DependencyProperty
To Reproduce
Steps to reproduce the behavior:

Add a hc:PasswordBox
Try to bind a value to the Password property

Expected behavior
Value binds
Environment (please complete the following information):

.net: 4.5
IDE Visual Studio 2019
Version 16.2.2

Additional context
XAML Code :
xmlns:properties="clr-namespace:Watchguard.Report.Properties"

<hc:PasswordBox x:Name="TxtPassword"
Password="{Binding Source={x:Static properties:Settings.Default},
Path=Password, Mode=TwoWay}"  />

I don't think this is a bug.
Take a look here
https://www.wpftutorial.net/PasswordBox.html

@ghost1372 Hello, thanks for the answer. It would be nice a to have a 'ActualPassword' feature which will act like Text and not SecureString. Btw, why is there no PasswordChanged event ? Or TextChanged ?

@TheKeytrap  there is, Use it like this
        <hc:PasswordBox PasswordBox.PasswordChanged="PasswordBox_PasswordChanged"/>

@ghost1372 Thanks !


Unresolved dependency: not found

Hi,
When adding:
resolvers += "bintray-spark-packages" at "https://dl.bintray.com/spark-packages/maven/"
addSbtPlugin("org.spark-packages" % "sbt-spark-package" % "0.2.6")

To my project/plugins.sbt file, I get this error:
[error] (update) sbt.librarymanagement.ResolveException: unresolved dependency: org.spark-packages#sbt-spark-package;0.2.6: not found
I've found previous issues mentionning this, but they appeared to be solved by adding a resolver.
In my case, it didn't work.
Any idea on how to fix this ?
When looking at https://dl.bintray.com/spark-packages/maven/org/spark-packages/, there only appear to be the 0.1.1 version in the sbt-spark-package folder, but the 0.2.6 is in the other one. I've tried replacing sbt-spark-package by sbt-spark-package_2.10_0.13but it didn't work either.
Thanks for your help!

same issue

According to https://spark.apache.org/news/new-repository-service.html , we should not use dl.bintray anymore. I replaced the following
resolvers += "bintray-spark-packages" at "https://dl.bintray.com/spark-packages/maven/"

with this
resolvers += "spark-packages" at "https://repos.spark-packages.org/"

and it seems to work now.


Add documentation for .bicepparam files usage without providing a template

#11703 allows .bicepparam files to reference registry, template spec and JSON files directly.
This also required changes to Azure CLI (https://github.com/Azure/azure-cli/pull/27311) & Azure PowerShell (https://github.com/Azure/azure-powershell/pull/22728).
We should document this, including:

AzCLI & AzPwsh usage
Version compatibility (this feature will require users to upgrade both Bicep & AzCLI/AzPwsh to a recent version).

Azure CLI: 2.53.0 or higher
Azure PowerShell: 10.4.0 or higher

Will we make it possible to deploy a .bicepparam file directly from vscode?

Will we make it possible to deploy a .bicepparam file directly from vscode?

This works today in the deploy pane: https://youtu.be/kPCpNGYu4xA?si=tOwAlyGfFwBbBikW&t=690 (thanks @mattallford for the video 😄 )

PowerShell deployment failed.
PS D:\Bicep\storage> new-azresourcegroupdeployment -resourcegroupname jgaotest -templateParameterfile storage.bicepparam
WARNING: WARNING: The following experimental Bicep features have been enabled: Compile-time imports. Experimental features should be enabled for testing purposes only. Do not enable these settings for any production usage, or you may be unexpectedly broken at any time!
New-AzResourceGroupDeployment: 11:04:37 AM - Error: Code=InvalidTemplate; Message=Deployment template validation failed: 'The provided value for the template parameter 'projectName' is not valid. Expected a value of type 'String, Uri', but received a value of type 'Object'. Please see https://aka.ms/arm-create-parameter-file for usage details.'.
New-AzResourceGroupDeployment: The deployment validation failed
PS D:\Bicep\storage> more storage.bicepparam
using 'storage.bicep'
param projectName = 'jgao'
PS D:\Bicep\storage>

@mumian this is unfortunately a pretty major bug in the latest AzPwsh release: https://github.com/Azure/azure-powershell/pull/22885. AzCLI will work.

@anthony-c-martin - do you mean not covering the PowerShell option for the time being?

@anthony-c-martin - do you mean not covering the PowerShell option for the time being?

Yes, that's what I meant.

@anthony-c-martin - can you review the PR - https://github.com/MicrosoftDocs/azure-docs-pr/pull/254345 ?  Thanks.

@anthony-c-martin - Azure/azure-powershell#22885 has been merged. Do you know when the fix will be released to the production environment? So I can update the content?

I'll let you know - should be fairly soon.

Unlike Azure CLI, Azure PowerShell lets me to provide both the Bicep file and the Bicep parameters file.  See -
PS C:\bicep\storage> New-AzResourceGroupDeployment -Name bicepparam -ResourceGroupName jgaotest -TemplateFile c:\bicep\storage\storage.bicep -TemplateParameterFile "c:\bicep\storage\storage.bicepparam"
DeploymentName          : bicepparam
ResourceGroupName       : jgaotest
ProvisioningState       : Succeeded
Timestamp               : 10/20/2023 7:32:42 PM
Mode                    : Incremental
TemplateLink            :
Parameters              :
Name             Type                       Value
===============  =========================  ==========
prefix           String                     "jgao"
location         String                     "centralus"
Outputs                 :
DeploymentDebugLogLevel :


Can't open Local Disk D (<IP_ADDRESS>)

Hello!
When I try to access Local Disk D I got this

What can it be?
Thanks!

Sorry, I'm stoopid, I didn't grant permission to my storage.


Ignore result when error is not nil

The Future.Get() will ignore the value if the err is not nil. This is also true for workflow. When workflow returns non-nil error, the worker only report the error to server and ignore the result. We should keep the test framework the same behavior to ignore the value when error is not nil.

Coverage increased (+0.01%) to 68.849% when pulling 3a342b144f90188d1b5c65431e1ae9169d91675f on yiminc:fix_test_GetWorkflowResult into fd0b3dabb22f30f67c830d04e7b6389bf5789b02 on uber-go:master.


Can't create additional users in CRC!!

I am learning OpenShift. I deployed CRC 1.12. on Ubuntu 18.04 LTS (KVM). I can login to the cluster using kube:admin / developer both from cui & gui. I can see, form gui, that htpasswd is setup as identity provider.
My question is how can I add additional users to my cluster? With htpasswd, I need local access to the crc vm to be able to add user, change password. But local access to the crc vm over ssh is not permitted I guess.
I did not find any option to add user in the GUI either although it shows the existing developer user there.

@nilanu Check https://github.com/code-ready/crc/wiki/Add-another-user-to-cluster one but this is very specific around how auth works in the Openshift, not something crc specific.

https://docs.openshift.com/container-platform/4.2/authentication/identity_providers/configuring-htpasswd-identity-provider.html#add-identity-provider_configuring-htpasswd-identity-provider

closing this issue.


Count on Oci8 adapter

Hello,
I can't get DB-Connected resources fetch single item from Oracle. I always get a "Item not found" message, but I've noticed that it only occurs when I call $resultSet->count(), because it returns null.
$resultSet = $this->table->select([ $this->identifierName => $id ]);

        if (0 === $resultSet->count()) {
            throw new \Exception('Item not found', 404);
        }

        return $resultSet->current();

Is there any restriction to the Oci8 adapter?
I can get it to work if I don't check count() function, but to do so, I have to overwrite every DB-Connected resource class.

I have exactly the same problem on Oci8. Removing $resultSet->count() in DbConnectedResource#fetch($id) fixes the problem with single items.

Yes, I tried to persuade my boss to use Apigility to rewrite a legacy application, but this problem was an immense wall to that.
Furthermore, as we can note, I got no answer.

You three are the vested interest.
Why should someone go and fix a bug you're having an issue with?
Please submit a pull request because you are experiencing the problem you're likely best suited to fix it.  Rest assured if this results in a patch to Apigility or ZF2 it will get review.

Sorry man, but I don't want someone to fix a bug for me, I just wanted to know if it's working the way it supposed to be rather than being a bug.
Unfortunately I've moved to another job, so I don't have access to a Oracle database neither access to the modified code.

This repository has been closed and moved to laminas-api-tools/api-tools; a new issue has been opened at https://github.com/laminas-api-tools/api-tools/issues/43.


Fixed issue with gutter width initialization

Reproduce the original issue (in Chrome):

Directly go to http://intljusticemission.github.io/react-big-calendar/examples/index.html#selectable
Notice that you cannot see the day headers because the width of the gutter is set way too big

By wrapping it in the requestAnimationFrame it ensures that it is set after the initial paint is finished (which does the css etc).

Hi gents,
Thanks @janb87.
@jquense when are you going to release this fix?


Fix legacy peer dependency workaround

Summary of the bug
Fix legacy peer dependency workaround. Exploration work required first.
🪜 How to reproduce
Tell us the steps to reproduce the problem:

Delete all package-locks
Try to npm install without --legacy-peer-dep flag
It will fail

🧐 Expected behaviour
Npm install to work without the --legacy-peer-dep flag

Currently ic-design-system has a conflicting peer dependency relating to Gatsby. We use Gatsby 4.2.0, which is suitable as a peer dependency for most of the plugins we use, apart from 1 (https://github.com/jonearley/gatsby-plugin-stencil). There is a pull request in review to update the peer dependency to Gatsby version 4. However until this is merged, would need to either leave the flag in place, or downgrade the guidance site to Gatsby 2, like bringing in many other issues and requiring significant development capacity.


Is there any way around this?

This is the same issue as this: https://github.com/kimmobrunfeldt/progressbar.js/issues/65
The workarounds for this issue are:

Setting shape-rendering="crispEdges" to SVG hides the issue, but lines are not smooth anymore.
Make the yellow progress bar a bit larger then its container so the black won't be seen behind the yellow.

Enlarging the progress bar works in Firefox, but in Chrome the gray arc itself also cast a shadow like that, so no dice. I guess both arcs need to be clipped to really solve this.

If you end up clipping the arc, I'm interested of the solution if you don't mind sharing it.
This works for me in firefox and chrome: http://jsfiddle.net/wsyb65au/
The markup and css is a bit more complex and some things would be handy to have in variables, but it works.
The idea is to have trailWidth a bit smaller then strokeWidth. The background needs to be adjusted according to the trailWidth so that there won't be any white leaking from the background.
The code:
JS
var circle = new ProgressBar.Circle('.progress-container', {
    color: '#FCB03C',
    trailColor: '#888',
    strokeWidth: 5,
    trailWidth: 3
});

circle.animate(1.0);  // Number from 0.0 to 1.0

HTML

CSS
.container {
    width: 200px;
    height: 200px;
    position: relative;
}

.container > div {
    width: 100%;
    height: 100%;
    position: absolute;
    top: 0;
    left: 0;
}

.container .bg-container {
    background: #333;
    border-radius: 50%;
    top: 3%;
    left: 3%;
    width: 94%;
    height: 94%;
}

.progress-container {
    z-index: 9999;
}

svg {
    width: 100%;
    display: block;
}

Result:

Closing because of inactivity.


Add albumin-free serum

Closes #493. Adds "albumin-free serum."
The diff on this is weirdly big, and I'm not sure why. The ROBOT diff is clean. We can discuss in next OBI call—if there's a reason why this one turned out so messy, I'll re-do this PR.
Editing to add: James said the messy diff is probably OWL API issue, but it looked benign. To discuss on call: the logical axiom "blood serum specimen and (has part only (not albumin))".

I dunno how you ended up with genids, maybe from CLO. If you don't get rid of them they wll continue to cause churn
(of course robot diff by design shows no change as there is no change at the OWL axiom level)

It looks like OBI is interested in modeling media mixtures as might be found in e.g sigma aldritch catalog
Is there a document with the design patterns documented? I think you will encounter issues with OWL quantification further down the line. I have a general interest in this area and in ontological representation of media mixes for microbial culturing, and am planning to use chemrof:ChemicalMixture as DPs for hierarchical quantification of both precise and imprecise mixtures. You can ignore my comment for the purposes of this PR though, just an expression of interest. cc @realmarcin

I did some experimentation with this new class in

https://github.com/obi-ontology/obi/issues/1767

I never accomplished what I had hoped for, but that doesn't mean I have any objection to this PR

@cmungall I just put together a pull request (#1769) to remove the genids. They were related to OBI:0100061, which had an editor note attached to one axiom; switching that to a general editor note annotation got rid of the genids.
Thanks for the interest in the issue of modeling mixtures. That's a topic that @ddooley and I will likely be working on in our material processing ontology project, though that's still in the early stages. Agreed that OWL quantification will be challenging.


[ISSUE #8804] clean offset when remove group offset

Which Issue(s) This PR Fixes

Fixes #8804
Brief Description

implement the clean offset,then we invoke deleteSubscriptionGroup ,we can clean
How Did You Test This Change?

Codecov Report
Attention: Patch coverage is 0% with 15 lines in your changes missing coverage. Please review.

Project coverage is 47.41%. Comparing base (95b1c51) to head (d9c44ff).

Files with missing lines
Patch %
Lines

...cketmq/broker/offset/LmqConsumerOffsetManager.java
0.00%
15 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8805      +/-   ##
=============================================
- Coverage      47.47%   47.41%   -0.06%
+ Complexity     11591    11576      -15
=============================================
  Files           1283     1283
  Lines          89888    89903      +15
  Branches       11568    11571       +3
=============================================
- Hits           42670    42629      -41
- Misses         41981    42027      +46
- Partials        5237     5247      +10

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

LGTM～


feat: generate events on pods when apply Kong configurations

What this PR does / why we need it:

Generates an event attached to KIC pod on KIC applying configuration to Kong.
When KIC successfully applied configurations, attach a Normal event;
When applying fails, generate a Warming event with error message.
Which issue this PR fixes:

fixes #4003
Special notes for your reviewer:

PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[x] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

Can we add a changelog entry pray ?

Added CHANGELOG item.


feature: allow user to disable alerts

I've got rangy working in a browserified chrome extension. The extension is always on, and every now and then, rangy does not work in the environment. This happens every time I open a new tab from a Google search link, for instance. In such cases, it throws an alert, although I'm not using rangy at that moment and it is just alerting from the Chrome extension.
Is there a way I can require rangy without enabling alerts? I'd rather the user never see them, or just console.log them.

You can disable alerts by setting rangy.config.alertOnFail = false. The failure message will then go to console.log instead.

Thanks. Works.

I may have spoken too soon. Even with this:
var rangy = require('rangy')
rangy.config.alertOnFail = false

I still can see alerts. Is there a way we can pass the config directly into the module?

I've set the default for that option to false.

Thanks @timdown. Can you bump the version for npm?

I'm planning to do another beta release at the weekend once I've fixed a few bugs. Can you wait until then?

Not a problem. Thanks.
—
Sent from Mailbox
On Wed, Mar 18, 2015 at 5:20 PM, Tim Down<EMAIL_ADDRESS>wrote:

I'm planning to do another beta release at the weekend once I've fixed a few bugs. Can you wait until then?
Reply to this email directly or view it on GitHub:
https://github.com/timdown/rangy/issues/280#issuecomment-83232432


[WIP] Last error not recorded

fixing a regression where SNR doesn't record the last error in case node isn't found.

/test 4.14-openshift-e2e

Giving a chance to get more reviews, feel free to unhold
/hold

/unhold