V1: Distinct repos for frontend and backend

Hey guys, JAWS is looking really great can't wait to try out v1. I've been following the doc and am curious if you would consider separating the backend and frontend into their own distinct repos?
I'm not too familiar yet with JAWS or lambda so I don't know if this is feasible but I find separating the client code from the server code into separate repos much easier to maintain with a larger app, especially when the backend is strictly an API.
This might also allow for front-end guys (like myself) to contribute easier by extending the core frontend repo. I may also already have the client built and just want JAWS to serve as my backend API or I just want to quickly deploy a static site with no backend using JAWS' cli tools.
Like I said, I don't really know what implications separating them has but just some food for thought!

Hi @jschr, there won't be no front and no back end. jaw-cli is a client able to grab separately a front module and some back end modules. Take a look at https://github.com/jaws-stack/JAWS/tree/v1.0.

Ah ok, after rereading the v1 spec doc I think I misinterpreted the scafolding section.
So am I correct that the use case I described above I would create 2 JAWS apps (one for back and one for front) and simply install the applicable modules?

@jschr Glad you arrived at the party :)  Yes, JAWS V1 is nothing but some empty folders, a powerful CLI and a Module system.
We think the best organizational tiers for the jaws-modules are:

A single lambda function can be a module
A directory with a group of lambda functions and some other folders can also be a module
The entire front folder can also be a module.  This is the least developed idea.  Essentially, you will be able to run jaws install https://github.com/jaws-stack/JAWS/jaws-react-front to replace the front folder with the module at the specified url.  What this means is that we need some front-end modules for JAWS.  Especially, a React one.  It should consist of a simple React boilerplate app and integrate handy AWS-SDK for the browser functions.

@jschr, the basic steps to write a JAWS app will be jaws new, then grab some jaws modules for the back end (they will be downloaded in \back) and grab a front module (downloaded in \front).

@ac360 thanks for the clarification, this is awesome! Really excited for the v1 release.
Do you have any front-end module examples you can share?
I realize you've got a lot on your plate with v1 so no worries if not and I will anxiously await for the release!

Closing since not an issue :)

No front-end modules yet.  Although, any front-end boilerplate works.


Textures with huge amount of allocating memory in Unity

Hello there,
i discovered somethin very weird in the textures. So i importet a scene in WebGL at runtime. The textures that i am going to import are .jpg. They are 4096x4096 and 24 bit in depth. On Harddisk they have memory about 4-6 MB.
I used the memory profiler to track the memory and realized each of those texture are allocating 128MB of ram.

Is this a normal behavior ? Or where the huge ammount from the textures are come from ? Did i miss something on the realtime import when i import the file at runtime ?
Cheers
Marcus

hey, you should set on the importer the KeepCPUCopyOfTexture to false. Otherwise the textures will be also exist on the RAM.
When it's not solving your problem, please come back :)

Hello there,
Yes it did the job. And of course the compression i was looking at. Thank you :)

In a future release there will be also this option on the asset importer, i already has made a PR for this :)
So your issue was helpful to take another look to that :)


Apply DPD CFR does not get applied

The settings are there but the source\Methods\RFmxDPD\Measurements\Configure Memory Polynomial.vi
does not configure the related properties

wrong configuration


AMPI collectives on COMM_SELF using derived datatypes are broken

Original issue: https://charm.cs.illinois.edu/redmine/issues/1412

For collectives on MPI_COMM_SELF, we just copy the message data from the send buffer to the recv buffer, but currently we do that based only on the send type/count.
We need copyDatatype() to also use the recv type/count when unpacking.

Original date: 2017-02-13 19:29:22

~~https://charm.cs.illinois.edu/gerrit/#/c/2237/~~ https://github.com/UIUC-PPL/charm/commit/838c706dbf0803f61a5ca5ab4b3e3b6717142f28
Also realized that MPI_Scan and MPI_Exscan are the only collectives that don't explicitly check for MPI_COMM_SELF, so I added those checks here: ~~https://charm.cs.illinois.edu/gerrit/#/c/2238/~~ https://github.com/UIUC-PPL/charm/commit/7f73aa1a89f2e39cd719034e103a08eb8965c515


const names like ROUTER_DIRECTIVES according Dart style guide

Do you plan to fix these names to align them with the Dart style guide (routerDirectives)?
In TS they are gone at all (now RouterModule)

I'm closing this to be realistic it's not a priority and nobody plans on working on it as it's a potentially breaking change with very little real-world benefit. Maybe we can revisit in the future when we have some headroom in features.


Truncated Shapes calculator

I would like to add a calculator that calculates the, area, volume, Circumsphere radius, Midsphere radius of the following Truncated shapes, Please assign these to me. I will create separate PR for each of them.

Truncated tetrahedron
Truncated cube
Truncated cone
Truncated Dodecahedron
Truncated octahedron
Truncated icosahedron

@sairish2001 Please don't close this issue as I will make more shape calculator PR referencing this issue


Youtube DL Python Youtube Music

Good afternoon,
I have encountered a little problem;
When I try to download all the videos of an artist on youtube music (url such as: https://music.youtube.com/channel/{id}),
youtube dl starts looping without ending;
it downloads all the videos and then download it again... infinitely...
Any suggestions ?
It has not been fixed during the last 3 release.
However, Youtube Music is not in the list of supported websites so it is may be why it does not work.
I am at your disposal for more information.
Thanks for reading me.
I am using youtube dl 2020.12.22

What is missing ?

@dstftw,tanks for having read me I am sorry to mention you but I just want to know what is missing; if you try with a youtube music channel url it will not work that 's why i did not put any debugging informations.

Read new issue template.

Ok, thanks for your answer, i thought i had read it carefully...


XRDP Keyboard evdev support?

Is it possible to switch XRDP to evdev keyboard rules or at least make it compatible?
All ubuntu 20.04 flavors (gnome, xfce, ...) seem to use 'evdev' keyboards:

setxkbmap -query -v
Trying to build keymap using the following components:
keycodes:   evdev+aliases(qwertz)
types:      complete
compat:     complete
symbols:    pc+de+inet(evdev)
geometry:   pc(pc105)
rules:      evdev
model:      pc105
layout:     de

However when i connect to this same machines via XRDP i get a 'base' rule with a 'xfree86' setting:

setxkbmap -query -v
Trying to build keymap using the following components:
keycodes:   xfree86+aliases(qwertz)
types:      complete
compat:     complete
symbols:    pc+de
geometry:   pc(pc104)
rules:      base
model:      pc104
layout:     de

As far as i understand this might be intended to make it compatible for Xvnc sessions. However i don't use Xvnc sessions as i connect to Xorg sessions via xorgxrdp extensions.
So, how can i switch XRDP to evdev to make it behave exactly like it does on the X console?
My problem arises with certain applications (mostly remote consoles) that do work on the machine's X console as intended but having keyboard issues when i access my machine via XRDP. They all suffer the same problem of ignoring alt-gr keys (german keyboard @, , ...) and actually no cursor keys - only cursor-down seems to be enter.
The same effect appears if i switch keyboard to 'evdev' in running XRDP session via setxkbmap -rules evdev
Then all applications have this same problem. So it seems to me that these applications won't play with 'base' mode but need 'evdev' rules?
So question is, what can be done to make XRDP play with 'evdev' rules. I tried this on ubuntu 20.04, xubuntu20.04 with xrdp 0.9.12 (ubuntu repos) and 0.9.18 built from source - all the same. Any ideas?
Is this a bug or a limitation of the current implementation?
https://github.com/neutrinolabs/xrdp/issues/179 seems to imply that there is a way to switch to evdev rules (breaking Xvnc - but that would be ok for me) but it does not tell what needs to be done to achieve this. As this is quite old i'm not sure if it still applies.

I'm going to move this to the xorgxrdp repository, as I think it's more relevant there.


When the sidecar injector has been explicitly closed, pilot-discovery still returns  "warn: sidecar injector is not ready", ReadyProbe failed

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
        - name: INJECT_ENABLED
          value: "false"

pilot-discovery log
2024-01-03T05:22:37.595126Z     info    Skipping sidecar injector, injection path is missing or disabled.
2024-01-03T05:22:37.883901Z     info    sidecars.networking.istio.io is now ready, building client
2024-01-03T05:22:40.712055Z     warn    sidecar injector is not ready

It seems that in the implementation, whether the sidecar inject is turned on is not considered.
in pilot/pkg/bootstrap/server.go
func (s *Server) initReadinessProbes() {
	probes := map[string]readinessProbe{
		"discovery": func() bool {
			return s.XDSServer.IsServerReady()
		},
		"sidecar injector": func() bool {
			return s.readinessFlags.sidecarInjectorReady.Load()
		},
		"config validation": func() bool {
			return s.readinessFlags.configValidationReady.Load()
		},
	}
	for name, probe := range probes {
		s.addReadinessProbe(name, probe)
	}
}

This will cause /ready on port 8080 to always return 503
in pilot/pkg/bootstrap/server.go
func (s *Server) istiodReadyHandler(w http.ResponseWriter, _ *http.Request) {
	for name, fn := range s.readinessProbes {
		if ready := fn(); !ready {
			log.Warnf("%s is not ready", name)
			w.WriteHeader(http.StatusServiceUnavailable)
			return
		}
	}
	w.WriteHeader(http.StatusOK)
}

Version
~/kubectl version
Client Version: v1.29.0
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.18.6-580-dirty

Additional Information
No response

What is your istio version? This is fixed by https://github.com/istio/istio/pull/46003

@hanxiaop Tks a lot
The version information was lost when i was building, resulting in my wrong version.


Move code to CircleCI for automated builds

I can see that TravisCI is currently being used, but it might behoove us to move this project to CircleCI.
CircleCI 2.0 gives us a lot of flexibility, and allows us to run things in containers with versions of code that we desire.  It also means we don't have to maintain our own build cluster.
Just something to think about.  I can take this on if you want; shouldn't take longer than a few hours.

I did some research before choosing travis and the main reason for choosing it is that it has first-class support for Rust, including parallel builds for different toolchains (stable, nightly, etc) on multiple operating systems and therefore has become the de-facto standard for Rust projects.
Seems to be working great so far so I don't think it's worth switching to be honest.


Bug for S/4 readiness check psuedo comments

We are running S/4 readiness checks, and have some cases where psuedo comments are applied with the note number indicating usage is ok. The "align parameters and components" rule however is moving this comment from the correct location to incorrect location, like below:
CALL FUNCTION 'STANDARD_FUNCTION' "#EC CI_USAGE_OK[1234567]
    EXPORTING
      param1 = val1

CALL FUNCTION 'STANDARD_FUNCTION'
    "#EC CI_USAGE_OK[1234567]
    EXPORTING
      param1 = val1

Hi Josh,
oh, that's no good! I think this cleanup rule does not distinguish pseudo-comments from "normal" comments at this point, but it definitely should – thanks for reporting this!
Kind regards,
Jörg-Michael

Hi Josh,
thanks again – this should be fixed in the next release!
Kind regards,
Jörg-Michael

Hi Josh,
version 1.4.1 was just released, fixing this!
Kind regards,
Jörg-Michael


Fixed method signature for custom messages.

uint -> ulong

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


Automatic update of Shouldly to 3.0.2

NuKeeper has generated a patch update of Shouldly to 3.0.2 from 3.0.1
Shouldly 3.0.2 was published at 2018-11-06T21:58:19Z, 21 days ago
2 project updates:
Updated tests/HttpClientInterception.Tests/JustEat.HttpClientInterception.Tests.csproj to Shouldly 3.0.2 from 3.0.1
Updated samples/SampleApp.Tests/SampleApp.Tests.csproj to Shouldly 3.0.2 from 3.0.1
Shouldly 3.0.2 on NuGet.org
This is an automated update. Merge only if it passes tests
NuKeeper: https://github.com/NuKeeperDotNet/NuKeeper

Codecov Report

Merging #59 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #59   +/-   ##
=======================================
  Coverage   99.75%   99.75%
=======================================
  Files           8        8
  Lines         804      804
  Branches      107      107
=======================================
  Hits          802      802
  Partials        2        2

Continue to review full report at Codecov.

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

Change already pending in #44.


[8.x] Add ability to customize json options on JsonResource response

Currently, it is possible to customize the options with JsonResource::withResponse($request, $response) doing something like
public function withResponse($request, $response)
{
    $response->setEncodingOptions(JSON_PRETTY_PRINT);
}

The behaviour is that the original json is decoded then re-encoded with the given options https://github.com/laravel/framework/blob/676ba54b7d62eb76f0d1b025173b8f7134ed8b78/src/Illuminate/Http/JsonResponse.php#L119-L124
https://github.com/laravel/framework/blob/676ba54b7d62eb76f0d1b025173b8f7134ed8b78/src/Illuminate/Http/JsonResponse.php#L63-L66
https://github.com/laravel/framework/blob/676ba54b7d62eb76f0d1b025173b8f7134ed8b78/src/Illuminate/Http/JsonResponse.php#L73-L92
Unfortunately, json_encode with no option is destructive :
json_decode(json_encode(['float' => 3.0]), true) gives ['float' => 3].
With this, $response->setEncodingOptions(JSON_PRESERVE_ZERO_FRACTION); returns a json like
{"data":{"float":3}}

That is, there is currently no way to encode JsonResources with JSON_PRESERVE_ZERO_FRACTION.
This PR adds the ability to customize json serialization options for responses from JsonResource at the time that the JsonResponse is created, providing the ability to preserve the zero fraction of floats.

Why do you need JSON_PRESERVE_ZERO_FRACTION btw ?
Imagine a Order model with a quantity property as float and you want to test some simple api call like
public function it_returns_the_order()
{
    $order = Order::factory()->createOne();

    $this->getJson("api/orders/{$order->id}")
        ->assertOk()
        ->assertJsonPath('data', [
            'id' => $order->id,
            'quantity' => $order->quantity,
        ]);
}

Pretty easy no ?
Well, if your factory is defined with
public function definition()
{
    return [
        'quantity' => $this->faker->randomFloat(1, 0, 10), // random float with 1 digit max between 0 and 10
    ];
}

and faker generates a float with 1 digit, your test will pass. But if faker generates 3.0 for example, the test will fail with
Failed asserting that two arrays are identical.
--- Expected
+++ Actual
@@ @@
-  'quantity' => 0.0
+  'quantity' => 0

Quite annoying to have randomly failing tests...

What about resource collections? Does anything need to be done for them?

No, no change needed for ResourceCollection.
But you make me remember that the same change must be done in PaginatedResourceResponse
https://github.com/laravel/framework/blob/8.x/src/Illuminate/Http/Resources/Json/PaginatedResourceResponse.php#L15
Currently afk, I will make the change tomorrow.

I will reopen the PR once the changes are pushed.

Well, you were right, some changes were needed for ResourceCollections... Let me know if something looks wrong

Really good work @bastien-phi

Thanks !

This has introduced an error for me:
namespace App\Http\Resources;

use App\Models\User;
use Illuminate\Http\Resources\Json\ResourceCollection;

class UserResourceCollection extends ResourceCollection
{
    public $collects = User::class;

    /**
     * Transform the resource collection into an array.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return array
     */
    public function toArray($request)
    {
        return $this->collection->map(function ($user) use ($request) {
            // ...
        });
    }

    // ...
}

Using this ResourceCollection throws BadMethodCallException with message 'Call to undefined method App\Models\User::jsonOptions()'.
Should I not use model classes in $collects property anymore? In this resource collection I want it NOT to make each row into a UserResource instance.
I've been following this guide to make it work the way I want.

@bastien-phi can you fix the issue from above? Otherwise it'll be best to revert this PR.

@driesvints It looks like an unexpected usage of ResourceCollection. As documentation states, $collects should be a class-string of JsonResource

As it worked with that, I will open a PR to fix this.

@bastien-phi you're correct. @ninjami-matti it seems you're using a model here but $collects is intended for resources only. That's also explained in the guide you're linking to.

Fair enough 👍
I would still argue that it is slightly confusing that you can still make ResourceCollection classes without a corresponding JsonResource class existing, which then makes the ResourceCollection collect the actual Model class as the resource.
I set the $collects property with a Model class based on the assumption that it should work since using the Model does work when not explicitly defined. And it did work before this addition.
But indeed I see now that setting a Model class to the $collects property is the wrong way go about this. Thanks for the help!

@ninjami-matti If you want to bypass auto-discovery of underlying resource, you can just rename your resource collection in order to make it not ending with Collection and remove $collects
class UserCollectionResource extends ResourceCollection
{
    /**
     * Transform the resource collection into an array.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return array
     */
    public function toArray($request)
    {
        return $this->collection->map(function (\App\Models\User $user) use ($request) {
            // ...
        });
    }

    // ...
}

will work

That's true! However, I'm just too much of a stickler on following "correct" naming conventions 😄 So I went ahead and changed my collections and resources to reflect the correct usage and the problem is now gone.
Thanks for the tip!


Auto update

Hi Jonas,
Thanks again for your awesome work, we have been using your images to run our small FTB servers for years.
In all that time I admit I have never really gotten any good at programming in Docker and use your images because they just tend to work out of the box! With this release you have provided update instructions but I do not know where to begin I'm afraid. In previous images you have set it up to always use the latest stable release and update automatically. Mau I ask is there a reason you have not implemented that in this instance? It is very helpful. We are planning to start a new Stoneblock server and are wishing to use 1.6.1.
Kind Regards

Hi @aXGAZXa,
Thank you for reaching out to me, and I am glad to hear you enjoy the containers.
I simply forgot to setup my usual monitoring of new release for this mod-pack so I totally missed this new stable release.
Now corrected and setup for any future releases, and the new version is being built as I am writing on Docker Hub.


Allow raw request body via hub api --input <FILE>

This is to enable nested JSON arrays/hashes.
Closes #2029, fixes #2028
/cc @mlk @fsouza

@mislav I actually take my approval back lol Why not build on the standard of @filename? It works like that for graphql requests (I don't imagine anyone will ever write the gql query in the command line, but they technically could).
So one could do --input '{"field1": "value1", "arr": [1, 2, 3]}' or --input @body or --input @-. What do you think?


Introduce nixf-tidy

https://github.com/nix-community/nixd/issues/497
https://discourse.nixos.org/t/list-of-nix-linters/19279/9
https://github.com/NixOS/nixpkgs/pull/330066
https://github.com/NixOS/nixpkgs/issues/332695
https://github.com/inclyc/flakes/blob/168b2fb0c66c9946037afaf1b4bb884e3b58b465/.github/workflows/nixf-tidy.yml#L25C13-L25C36

💭 Without this, zed and the default nixd integration displays many warnings

Draft
git ls-files '*.nix' | xargs -I{} bash <<'BASH'
echo -n '{}: '
git diff --exit-code --quiet <(printf '[]\n') <(<{} nixf-tidy --pretty-print --variable-lookup)
echo "$?"
BASH

git ls-files '*.nix' | xargs -I{} bash <(<<'BASH'
echo -n "$1: "
BASH
) {}

git ls-files '*.nix' | xargs -I{} bash -c '
echo -n "{}: "
git diff --exit-code --quiet <(printf "[]\n") <(<{} nixf-tidy --pretty-print --variable-lookup)
echo "$?"
'

When refactoring in https://github.com/kachick/dotfiles/pull/943, it throws much parse-redundant-paren
https://github.com/nix-community/nixd/blob/0f3c0c730e5cb9414f3b269dc9b36062a0e5236e/libnixf/src/Basic/diagnostic.py#L49-L51
I'm not positive for these, because of I wish nixfmt-rfc-style should cover these if need.
And nixf-tidy does not take CLI options to modify behavior. And the returned JSON does not include severity. Might be dropped :thinking:


Pagination should be available when querying players

Issue
VaingloryAPI::Client#players should allow a developer to paginate results as per the Vainglory API "Pagination" documentation.
TODO

[x] Implement fix
[ ] Update specs
[x] Update YARD

Successfully fixing this should bring us to v0.1.0

Not needed.


simplify acc logic

I think the PR makes unintended changes to the computation in Dispatcher mode. In dispatcher mode and if strings.Compare(dispatchedAgentId, agentId) == 0 we should add the aggregated AW request to preemptable if we want to preserve the existing behavior.
if strings.Compare(dispatchedAgentId, agentId) != 0 {
					klog.V(10).Infof("[getAggAvaiResPri] %s: Skipping adjustments for %s since it is in cluster %s which is not in the same cluster under evaluation: %s.",
						time.Now().String(), value.Name, dispatchedAgentId, agentId)
					continue
} else {
    for _, resctrl := range qjm.qjobResControls {
        qjv := resctrl.GetAggregatedResources(value)
          preemptable = preemptable.Add(qjv)
     }
    for _, genericItem := range value.Spec.AggrResources.GenericItems {
        qjv, _ := genericresource.GetResources(&genericItem)
        preemptable = preemptable.Add(qjv)
    }
    continue
}

No changes were done for dispatcher mode and I would avoid doing it as there are no test cases yet AFAIK

No changes were done for dispatcher mode and I would avoid doing it as there are no test cases yet AFAIK

OK. You are right, the problematic changes were in fact made in PR #415. I was looking at the combination of the two PRs. We'll address these changes separately.


Problem with overriding some default TLS options

Do you want to request a feature or report a bug?

Bug

What did you do?
I apologize in advance if this issue has already been answered, but I could not find anything about it.
I have 2 routes safe.example.com and example.com each pointing to a different service: the first one is supposed to require a client certificate while the other does not.
I also want to use let'encrypt to obtain a certificate valid for both routes: CN: example.com, SAN: example.com, safe.example.com.
Traefik has two entrypoint: web (http) and websecure (https). The web entrypoint just redirects all the requests to websecure.

What did you expect to see?
I would expect that the safe route to require a client certificate (safe TLS option) without compromising the certificate resolver (default TLS option): when I set the TLS option field at the router level, from what I read in the documentation https://docs.traefik.io/v2.2/routing/entrypoints/#tls the default configuration (applied at the entrypoint level) will not be applied at all. Am I right?
What did you see instead?
I did not want to mess up my configuration so before trying I wanted to be extra sure by asking here. Thanks in advance for your awesome work.
Output of traefik version: (What version of Traefik are you using?)

2.2.1

What is your environment & configuration (arguments, toml, provider, platform, ...)?
static configuration:
[providers]

[providers.file]
directory = "/etc/traefik/dynamic/"
watch = true

[entryPoints]

[entryPoints.web]
address = ":80"
[entryPoints.web.http.redirections]
[entryPoints.web.http.redirections.entryPoint]
to = "websecure"

[entryPoints.websecure]
address = ":443"
[entryPoints.websecure.http.tls]
options = "default"
certResolver = "letsenc"
[[entryPoints.websecure.http.tls.domains]]
main = "example.com"
sans = ["safe.example.com"]

[api]
insecure = false
dashboard = true
debug = false

[certificatesResolvers]

[certificatesResolvers.letsenc]
[certificatesResolvers.letsenc.acme]
email =<EMAIL_ADDRESS>storage = "/etc/traefik/acme.json"
[certificatesResolvers.letsenc.acme.httpChallenge]
entryPoint = "web"

[tls.options]

[tls.options.default]
minVersion = "VersionTLS12"
sniStrict = true
cipherSuites = [
    "TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384",
    "TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384",
    "TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256",
    "TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256",
    "TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305",
    "TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305"
]

[tls.options.safe]
minVersion = "VersionTLS12"
sniStrict = true
cipherSuites = [
    "TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384",
    "TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384",
    "TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256",
    "TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256",
    "TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305",
    "TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305"
]

[tls.options.safe.clientAuth]
caFiles = ["/etc/traefik/certs/ca.cert.pem"]
clientAuthType = "RequireAndVerifyClientCert"

dynamic configuration:
[http.routers]

[http.routers.app]
rule = "Host(`example.com`)"
service = "app"
entryPoints = ["websecure"]

[http.routers.safe]
rule = "Host(`safe.example.com`)"
service = "safe"
entryPoints = ["websecure"]
[http.routers.safe.tls]
options = "safe"

[http.services]

[http.services.app.loadBalancer]
[[http.services.app.loadBalancer.servers]]
url = "http://app:3000"

[http.services.safe.loadBalancer]
[[http.services.safe.loadBalancer.servers]]
url = "http://safe:3000"

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Traefik!
We dedicate the issue tracker to bug reports and feature requests only. My advanced AI has spotted that your issue might be a configuration problem or relates to something that doesn't look like a bug.
To confirm this, please join our Community Forum and reach out to us on the Traefik section.
In case I'm wrong (well, that would be embarrassing :sweat_smile:), my developers will re-open the issue and fix me!
In the meantime, you can double check Traefik's documentation.


DOC-1496: README re-write.

cc: @jbondpdx

:+1:  to merge. We need to do another editorial pass, but we'd really love to get this merged in to avoid conflicts. We'll do a second edit on it once it's been merged.


unable to configure Authorization policies at other than AddAuthorization

I would to configure Authorization policies at the Configuration stage of Startup rather than ConfigureServices stage.  However, when I have the AuthorizationService (IAuthorizationService passed) to Configure(), there is not an exposed method or access to options list  add to it.  Not clear to me why this is not allowed since web pages are not running at this stage and so Authorization is not at risk.
Second, I do not understand how to use the services.Configure<> to load the desired polices, or if that is even possible, since ConfigureAuthorization was removed.

Why do you want to do it inside Configuration()? Authorization is a service, so by design it's configured inside ConfigureServices()

Let's start with that I like the new Authorization capability and the power it provides. Kudos to open sourcing the code which is helpful.
But let me try to answer the why:

Present architecture (as I see via the playing and reading through the code) requires that I do the configuration as part of the AddAuthorization() at the time the service is created.  This okay but requires a fixed architecture at time of compile.  Okay during early phases, but requires IIS server or application restart to implement changes, losing any existing Session and in memory information.  As noted, okay at debug time but long term not best for new applications.

I have about 12 policies that I add.  Short term, I would like to create them in separate method where I return the list of policies to be added.  While I could just pass IServiceCollection, do the work and return IServiceCollection (aka the extensions), would prefer to isolate so just returns the policies to be added.  Not clear to me how this can be accomplished to add the policies at the AddAuthorization() as it does not accept a list of policies. Or am I missing something simple.

Long term, I would like to pull the policy requirements information from a configuration file or even pull from a database.  Could still be done in ConfigureServices, but doesn't feel right to me.

And lastly, why would you care whether I configure the Services in ConfigureServices() or Configure()? From my view, ConfigureServices() is meant to add and initialize the services needed for the application and should only have configuration information requied at startup time.  Any other configuration information (such as adding the policies or adding new ones) in my opinion should be in Configure(), or at least should have the option to make changes there. But that is my opinion.  I can live with a requirement for service configuration to be in the service addition, but as noted above, not clear as to how I can create a list of policies and have them passed to AddAuthorization().

I will admit that I am just digging into this area.  I could modify the Authorization code to my liking, but trying to live within the present and hopefully future structure for Authorization.

I don't see what's wrong with pulling policies from the database inside configure services, but I'd like to know what you want inside the database. Remember you can inject into your handler.
I'm guessing that your database may contain roles or permissions that change at runtime? If so, I'd inject the permissions repo into the handler itself, and pull them out at evaluation time.

 I understand that I should be able to.  However, it is not clear how to.
So I create a method to build the desired policies:
public AuthorizationPolicies BuildPolicies()
{
        AuthorizationOptions options = new AuthorizationOptions();

        options.AddPolicy("Administrators", policy =>

        {

            policy.RequireRole("Administrator");

        });

        options.AddPolicy("Bidder", policy =>

        {

            policy.RequireRole("Tester");

        });

        Return options;

}
Now, when I add
            Services.AddAuthorization requires a delegate, which the example lambda expressions meet.  So how do I now change the BuildPolicies so that it is delegate that will meet the structure requirements for the AddAuthorization.  I know it is probably simple, but just not seeing yet.

From: Barry Dorrans<EMAIL_ADDRESS>Sent: Wednesday, February 24, 2016 5:57 PM
To: aspnet/Security<EMAIL_ADDRESS>Cc: Tgueth<EMAIL_ADDRESS>Subject: Re: [Security] unable to configure Authorization policies at other than AddAuthorization (#712)
I don't see what's wrong with pulling policies from the database inside configure services, but I'd like to know what you want inside the database. Remember you can inject into your handler.
I'm guessing that your database may contain roles or permissions that change at runtime? If so, I'd inject the permissions repo into the handler itself, and pull them out at evaluation time.
—
Reply to this email directly or view it on GitHub https://github.com/aspnet/Security/issues/712#issuecomment-188497915 .  https://github.com/notifications/beacon/ADkfz14AUBrp05sj8y2E0YB6tFpi05BBks5pniy7gaJpZM4Hh-c-.gif

I wouldn't do it with roles, I'd do it with code based requirements and handlers. That way you can access your repos inside the handler.
I realise this is a huge change from what we had before. I have a workshop you can work through which will help you move away from roles :) https://github.com/blowdart/AspNetAuthorizationWorkshop

This is all doable in rc2 by implementing a policy provider. The current model is push only at startup time, overriding the policy provider gives you full control over the logic that manifests policies at runtime.

I appreciate that I could change away from Roles, and may down the road.  But it doesn’t matter which approach I take as basically the AddAuthorization requires the setup at when it is invoked (aka must be passed as a parameter).    I am not using [Authorize] as I move.  But ASP.NET5/MVC6 requires so many changes that I simply don’t need something else to rewrite.
Tom
From: Barry Dorrans<EMAIL_ADDRESS>Sent: Wednesday, February 24, 2016 10:58 PM
To: aspnet/Security<EMAIL_ADDRESS>Cc: Tgueth<EMAIL_ADDRESS>Subject: Re: [Security] unable to configure Authorization policies at other than AddAuthorization (#712)
I wouldn't do it with roles, I'd do it with code based requirements and handlers. That way you can access your repos inside the handler.
I realise this is a huge change from what we had before. I have a workshop you can work through which will help you move away from roles :) https://github.com/blowdart/AspNetAuthorizationWorkshop
—
Reply to this email directly or view it on GitHub https://github.com/aspnet/Security/issues/712#issuecomment-188596238 .  https://github.com/notifications/beacon/ADkfzwAAzrq1UEYkuxAxdm_ItuGP7M2dks5pnnNCgaJpZM4Hh-c-.gif

Okay, so any idea when RC2 will be available?
Tom
From: David Fowler<EMAIL_ADDRESS>Sent: Wednesday, February 24, 2016 11:20 PM
To: aspnet/Security<EMAIL_ADDRESS>Cc: Tgueth<EMAIL_ADDRESS>Subject: Re: [Security] unable to configure Authorization policies at other than AddAuthorization (#712)
This is all doable in rc2 by implementing a policy provider. The current model is push only at startup time, overriding the policy provider gives you full control over the logic that manifests policies at runtime.
—
Reply to this email directly or view it on GitHub https://github.com/aspnet/Security/issues/712#issuecomment-188607273 .  https://github.com/notifications/beacon/ADkfz4EbUQe5AM4Hou9puspT45w3yfbXks5pnnhfgaJpZM4Hh-c-.gif

Okay, I figured it out.  I can put the method in the lamda expression as such:
services.AddAuthoriization( options => { BuildAuthorizationOptions(options);});

Barry, first of all, thanks for your videos.  Good stuff.  Helping me out a lot.  My scenario is similar, except they want to store the names of the policies in the DB and have the roles/claims stored in the db as well so when the app loads they are pulled dynamically, not hard coded.  I'm not using Repos(bad developer, I know)...can I just use an dbContext somehow to pull them and then load them?  Thanks, keep up the great work!


[Whois] ...

Extension
https://www.raycast.com/zavbala/whois
Description
I'm not sure if this is even possible but is there a way for the extension to automatically get the URL of the active tab?

Who will benefit from this feature?
No response
Anything else?
No response

Hey 👋 @kyellareddy It would be a nice feat, let me research


Create generic _show function for simple content types

shows/html/map.js contains the most generic content item templating so far. That's likely reusable (with some tweaks) for contact's and possibly many others.

This is really about dealing with item preview. The item menu needs to preview the item. However, the template used is page conditional--though there should be one basic one for all types.
Ideally, there should be a default template available with the type~{name} document, but that's not currently how it is...they're only in the template~{name} documents.
Next steps:

[ ] provide default Handlebars templates in the type~{name} docs
[ ] only use template~{name} type templates for overrides
[ ] store the type of the item in page_items (so we can include them in the map/reduce)

Know troubles:

what if the type of the item changes?...we don't actually allow that currently...maybe we never should...


Move the var-builder in a central place.

Merge the different codes together, the llama bit is to be rewritten.
This is pending some rewrite of the var-builder so as to untangle it a bit from safetensors, allow different storage and random initialization (based on hints given by the model, e.g. fan-in, fan-out, glorot, etc).

Can we please have 2 different methods ?
Model::load(vb)
and
Model::init() ?
Both need to go through the model, but the first one is tied to safetensors (or whatever storage and actual files).
While the other isn't, and for which passing something with a Rng might be better suited. (And it doesn't make sense to pass a Rng when loading something existing).
I feel strongly about this.

This first PR just re-groups the implemenation so that it's easier to refactor, for the init vs load this will be in a further PR.


generate_joint_distribution() function not found

Issue: generate_joint_distribution() function not found
Description:
Hi, I encountered an issue while running the scripts from the project. I have successfully installed all the dependencies as outlined in the dependencies.R script, but I'm receiving an error that the function generate_joint_distribution() cannot be found.
Here is the error message:
Error in generate_joint_distribution(n, p, model = model, df = df, distr = distr):
  could not find function "generate_joint_distribution"

Steps to Reproduce:

Ran the dependencies.R script to install all required packages.
Tried running the following code:train_dat <- generate_joint_distribution(n, p, model = model, df = df, distr = distr)

The function generate_joint_distribution() is not recognized, leading to the above error.

What I've tried:

I searched through the project files but couldn't find the function generate_joint_distribution() defined anywhere.
I've confirmed that all dependencies are properly installed and loaded.

Could you please confirm whether this function is missing, or if I need to source a specific file that contains this function? Any guidance on how to resolve this issue would be much appreciated!
System Information:

R version: [4.4.1]
Operating system: [Windows]

Thank you in advance for your help!

Thank you!


Recompute header limits for Shelley

Shelley will change signatures, what affects header and block sizes.  This
means, we need to recompute ingress queue limits ('MiniProtocolLimits') for the
chain-sync  and block-fetch protocols.

Related to:

https://github.com/input-output-hk/cardano-base/pull/115
https://github.com/input-output-hk/cardano-ledger-specs/pull/1433


Make the 3D View tool work with <dialog> elements

The new <dialog> elements allow displaying web content in a top layer.
You can see a nice demo here: https://cdpn.io/pen/debug/ExQBMgq
And docs here: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/dialog
The 3D View tool does not seem to support this notion of top layer. A top layer always appears above everything else in the DOM (and multiple top layers can co-exist and be stacked on top of each other).
The 3D View tool probably needs some special handling for this.
This was requested here: https://twitter.com/_baxuz/status/1540095041041059840
And, worth noting, the Chromium devtools is gaining support for top layer in the Elements panel too. See https://twitter.com/bramus/status/1540029969660887043

@joselea for info.


Add Dependabot or Renovate

What you would like to be added?
There should be a dependabot or renovate to help keep dependencies up to date.
Why is this needed?
This helps keep dependencies update and does not require manually upkeep to update them.
Love this feature?
Give it a 👍 We prioritize the features with most 👍

This is a good point. TBH, I want to introduce the dependabot, but our CI testing is too flaky in building images and unit/integration/e2e testing.
So, I would like to postpone the dependabot until the v1 API is removed.

I think, we already have it, isn't ?
Here is the PR example: https://github.com/kubeflow/training-operator/pull/2056

/remove-lifecycle stale

/assign


Build Status updated on Drone Dashboard but not on GitHub

I was checking out drone.  and set it upon AWS using docker-compose. The build status is updated on Drone Dashboard but the same status is not reflected on guthub.

please use discourse.drone.io for support questions


ERROR (MainThread) [homeassistant.components.device_tracker]

2017-12-25 14:58:42 ERROR (MainThread) [homeassistant.components.device_tracker] Unable to load /home/homeassistant/.homeassistant/known_devices.yaml: Config file not found: /home/homeassistant/.homeassistant/known_devices.yaml
After looking at the log，I created an empty file of known_devices.yaml under the home assistan folder, and the result log is still reporting this error
please help me , thanks~!

Maybe you are out if disk space or don't have write permissions?
In any case this is unrelated to customui.


[Duplication]

Description
hey @Shariar-Hasan
While I was scrolling through the Quotes.js file, I found duplicate quotes uploaded by a lot of users. I can clean up the Quotes.js for the project so that there is a single instance of each quote.
Thank you!
Screenshots
No response
Additional information
No response
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the contribution guideline

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

yes of course,
fir duplicate quotes, keep the first one and delete others.
thanks for noticing.


updated docs

NO.


[improve][broker] PIP-293: Delete config disableBrokerInterceptors

PIP: #293
Documentation

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

Codecov Report

Merging #20995 (50711e3) into master (63d9eaf) will increase coverage by 35.43%.
Report is 15 commits behind head on master.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             master   #20995       +/-   ##
=============================================
+ Coverage     36.92%   72.36%   +35.43%
- Complexity    12187    32297    +20110
=============================================
  Files          1698     1863      +165
  Lines        129846   141466    +11620
  Branches      14163    15960     +1797
=============================================
+ Hits          47947   102365    +54418
+ Misses        75570    30910    -44660
- Partials       6329     8191     +1862

Flag
Coverage Δ

inttests
24.16% <18.75%> (+0.01%)
:arrow_up:

systests
?

unittests
72.09% <100.00%> (+40.01%)
:arrow_up:

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

Files Changed
Coverage Δ

...org/apache/pulsar/broker/ServiceConfiguration.java
99.38% <ø> (+1.32%)
:arrow_up:

.../java/org/apache/pulsar/broker/web/WebService.java
94.14% <100.00%> (+18.48%)
:arrow_up:

...g/apache/pulsar/client/impl/RawBatchConverter.java
93.50% <100.00%> (+19.82%)
:arrow_up:

...rg/apache/pulsar/compaction/TwoPhaseCompactor.java
74.88% <100.00%> (+4.70%)
:arrow_up:

... and 1465 files with indirect coverage changes


not deleted same row get added again

https://github.com/x3n0r/BIMStore/blob/55478d7637fd77db352aadab692bc3d23b26ada2/BIMStore/DAL/DeaCustDAL.cs#L122
this is not a delete it is an add ....
Should be
db.tbl_(deacust?).Remove(erg.FirstOrDefault())

solved with c1e49715714aff5e868580cff5ccdaeee1dbb4bb


Merging changes from #229 and #231 - Two Solutions for Quarkus/Native Compilation

HI, We were discussing with @csviri how to merge changes introduced by #229 and #231 because merging #229 will require removing #231 almost completely. But on the other side we would like to avoid merging alpha version of kubernetes-client.
We came up with this kind of roadmap:

Put #229 on hold till release candidate of kubernetes-clinet (we agreed that this is minimal version that we can put in master)
Merge #231
Introduce quarkus-starter package that will add proper annotation for generated Doneable classes

But it's not made in stone. I've check and there is no public roadmap for kubernetes-client  that would help us a lot to know when we can expect RC version. @metacosm if it will be ready soon, let's say in week or two then we could freeze both to finally merge only #229 because code generation won't be needed anymore and quarkus integration will be moved to dedicated package (if it will be needed).
I would love to hear your opinion about this @metacosm @psycho-ir @adam-sandor @csviri.

Doesn't #231 need to remove quarkus dependency first, though?

Indeed I've we have to remove java 8 parts and quarkus dependency

The problem, though, is that #229 makes things a lot simpler and maintaining both branches will be a pain, especially considering that there are things that will need to be handled that will just go away if we were to use 5.0 because basically 5.0 was developed for the sole purpose of making things simpler for operator use cases.

@metacosm Absolutelly, the final goal is to have it with client version 5.0, that would be the cleanest solution. The only concert here is the timeline. If the stable version of fabricate client will be released within few weeks, we should probably use that branch, and merge it as soon as the client is at least in RC version.
If you see that it might take longer, we could also go with an intermediary solution, by generating Doneable classes, until we have the client ready.
On the other hand  if we merge now the client which is in alpha version, we will not be able to release further versions until the client is stable.
@metacosm What do you think, how long will it take a to release the new fabric8 client?

Maybe the best solution is to leave master in a stable state indeed and create a branch where all feature work would be performed with the expectation of using client 5.0 when it's ready? master would then only receive minimal changes in a stable fashion.
The problem with the fabric8 client 5.0 is that it's also experimental so we can indeed expect more changes that would be targeted at improving the operator-related experience. As such, we can't really release it until we get more feedback, feedback coming, for example, from being used in this project! So we're kind of in a deadlock situation where the sdk waits on the 5.0 release, while client 5.0 waits on feedback from project actually using it… 😄 My perspective on this is that both need to grow at the same time but that implies having a working sdk that uses 5.0 that people can actually use with the understanding that things are still evolving and bugs are expected.

One thing to note is that we also need to test the new version is compatible with [quarkus-openshift-client](https://github.com/java-operator-sdk/basic-native-operator-example/blob/init/pom.xml#L37.

@metacosm Absolutelly, the final goal is to have it with client version 5.0, that would be the cleanest solution. The only concert here is the timeline. If the stable version of fabricate client will be released within few weeks, we should probably use that branch, and merge it as soon as the client is at least in RC version. (And discard the other one)
If you see that it might take longer, we could also go with an intermediary solution, by generating Doneable classes, until we have the client ready.
On the other hand if we merge now the client which is in alpha version, we will not be able to release further versions until the client is stable.
@metacosm What do you think, how long will it take a to release the new fabric8 client?
@psycho-ir What do you think about it?

I think it makes sense to not rely on alpha dependencies in the mainstream but trying to simplify the APIs as much as possible (like the way we are going). IMO the developers will be very excited about the new features and the simplicity that the new version of the SDK has, Simpler annotations, Quarkus support, the universal eventing model to name a few.
I doubt depending their release to the release of  Kubernetes-client RC is crucial!

I agree with @psycho-ir , what we want to avoid is getting blocked to do releases.
We can still maintain the Fabric client alpha version support on a branch, test / play with it.

I agree with the conclusions here. Merging #231 will enable native compilation until fabric8 5 is finalized, I think that's a good thing to have even temporarily. At the same time moving to alpha release of fabric8 is too risky and we might end up with bug reports caused by the underlying client. I think we can still provide very good feedback to the fabric8 team by implementing things on the branch of #229 and seeing if the new APIs work as intended.

Because situation is clear I'm going to close this issue and create dedicated branch for implementing kubernetes-client 5.0 se we can merge and close #299 for now. This will allow us to release test versions from this new branch if needed.


Add retry policy for remaining GCP acc tests

Description
This PR adds a retry policy for some GCP tests that failed recently without retrying.
References

https://app.circleci.com/pipelines/github/snyk/driftctl/4195/workflows/5e665d3d-db9e-485b-bc29-1b106fd9846a/jobs/9071

Codecov Report

Merging #1347 (6ed6a14) into main (d256d98) will increase coverage by 8.87%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1347      +/-   ##
==========================================
+ Coverage   73.15%   82.02%   +8.87%
==========================================
  Files         482      410      -72
  Lines       18092    14948    -3144
==========================================
- Hits        13235    12261     -974
+ Misses       4537     2387    -2150
+ Partials      320      300      -20

Impacted Files
Coverage Δ

pkg/terraform/plugin_client.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/remote/aws/client/s3_client_factory.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/remote/aws/init.go
0.00% <0.00%> (-95.74%)
:arrow_down:

pkg/remote/azurerm/init.go
0.00% <0.00%> (-84.22%)
:arrow_down:

pkg/remote/google/init.go
0.00% <0.00%> (-77.78%)
:arrow_down:

pkg/remote/google/repository/storage.go
0.00% <0.00%> (-76.93%)
:arrow_down:

...g/remote/google/repository/cloudresourcemanager.go
0.00% <0.00%> (-76.93%)
:arrow_down:

pkg/remote/github/init.go
0.00% <0.00%> (-76.32%)
:arrow_down:

pkg/remote/terraform/provider.go
6.08% <0.00%> (-68.25%)
:arrow_down:

pkg/terraform/provider_factory.go
0.00% <0.00%> (-62.50%)
:arrow_down:

... and 129 more


Allow configurable batch size for RD_RecurringDonations_BATCH

We receive errors ("Too many query rows: 50001") when the daily Recurring Donations batch (RD_RecurringDonations_BATCH) process runs to create/refresh opportunities in Recurring Donations. The batch size for RD_RecurringDonations_BATCH is set to 50 and cannot be changed. We've reduced the Opportunity forecast months and it partially alleviates the problem, but a configurable batch size would be great.
Can we allow the RD_RecurringDonations_BATCH batch size to be configurable, similar to how the Donation Rollup processes are in the NPSP Settings tab? (like RLLP_OppContactRollup_BATCH, RLLP_OppAccRollup_BATCH etc).

Thanks for the suggestion @stopthesignal - I see the size is indeed set to 50 here. Not sure if this is something that can be controlled via setting the same way the other batch is, but I'll push this forward for future consideration.

**lurch: add

Tracking W-019043

Included in beta release 3.120 (Beta 6)

Included in production release rel/3.120


While keycloak.login() In App browser is Opened And When Application is Minimized Then In App browser is Automatically Closed. It Should be Open

Describe the bug

To Reproduce

Expected behavior

Any Update on This ? And Anyone have solution for This ?


Annotation and weight links are no longer available

Thank you for sharing your outstanding work, the link is now failed, would it be possible to share it again

thanks！


Update embeddable's value via entity.assign won't try to fix types

Describe the bug
Update embeddable's value via entity.assign won't trying to fix data type, it is expected to works like entity.create
Example:
@Embeddable()
class Nested {
  @Property({ type: Date })
  date: Date | string;
}

@Entity()
export class Test {
  @PrimaryKey()
  _id: ObjectId;

  @Embedded(() => Nested, { object: true })
  nested: Nested;
}

---
const entity = await this.testRepo.findOne({ _id: ObjectId('6357a02753179128e7f92789') });

this.testRepo.assign(entity, { nested: { date: '2022-10-25T09:19:49.861Z' } });
await this.testRepo.persistAndFlush(entity);

Current behvaior
Generated query
db.getCollection('test').updateMany({ _id: ObjectId('6357a02753179128e7f92789') }, { '$set': { nested: { date: '2022-10-25T09:19:49.861Z' } } }, {})

Expected behavior
Query should be
db.getCollection('test').updateMany({ _id: ObjectId('6357a02753179128e7f92789') }, { '$set': { nested: { date: ISODate('2022-10-25T09:19:49.861Z') } } }, {})

Versions

Dependency
Version

node
16.14.0

typescript
4.7.4

mikro-orm
5.5.0

mongodb
5.5.0

Update embeddable's value via entity.assign won't trying to fix data type, it is expected to works like entity.create

Nope, it is not expected to fix your types. If your definition was valid (date: Date | string should be just date: Date), this woudn't even type check,


fix: remove captures from context

As long as this does not make the performance worse, I'm liking this a lot. It would only slow things down if reusing Captures across cost models from different indexers would speed things up. But I would assume that cost models of different indexers are sufficiently different that their captures have little overlap and I'm not even sure if the existing code saves any work if Captures is carried over from indexer to indexer, even if there is a lot of overlap.
tl;dr: I'm cool with this change.


Missing ACT event id 41

Describe the bug
ACT ID 41 isn't recognized by nari. Here's some sample line(s) from an ACT log file:
41|2022-02-09T20:12:47.4280000-06:00|8003759E|267B|00|00|E0000000|fe464ebc1b31b6d4
41|2022-02-09T20:13:51.3740000-06:00|8003759E|267C|3DCDF00|20|19946D90|2d73dd43f17c571b
41|2022-02-09T20:18:05.6180000-06:00|8003759E|267C|3D62000|7FCF|3D63048|25b3eb6dde254335

To Reproduce
from nari.io.reader.actlog import ActLogReader
from sys import argv

for event in ActLogReader(argv[1], raise_on_checksum_failure=True, raise_on_invalid_id=True):
    pass

Where argv[1] points to a valid act log fle.
Expected behaviour
No raised exception.
Screenshots or Stack Traces
Traceback (most recent call last):
  File "/Users/nono/Projects/nari/basic_parsing.py", line 4, in <module>
    for event in ActLogReader(argv[1], raise_on_checksum_failure=True, raise_on_invalid_id=True):
  File "/Users/nono/Projects/nari/nari/io/reader/__init__.py", line 15, in __next__
    event: Optional[Event] = self.read_next()
  File "/Users/nono/Projects/nari/nari/io/reader/actlog.py", line 61, in read_next
    if (event := self.handle_line(line)) is not None:
  File "/Users/nono/Projects/nari/nari/io/reader/actlog.py", line 30, in handle_line
    raise EventNotFound(f"ACT id: {id_}")
nari.util.exceptions.EventNotFound: ACT id: 41

System (please complete the following information):
head of event-rewrite branch
Additional context
More events to parse I guess.

This is what ACT calls SystemLogMessage, it has an ID and 3 params. The ID is a direct reference to the LogMessage EXD, and the params tell the client how to format it. Looking at the contents of the messages, I don't see why ACT needs to know these, we can no-op it.


Disable semantic highlighting

Hi! I'm wondering if there's a way I can turn off semantic highlighting.
I appreciate how much work went into this feature, but sadly I find the colors distracting.
I don't see an existing way to turn off highlighting. Perhaps a setting could be added?
Thanks for your hard work!

You can control semantic highlighting via the following setting: "editor.semanticHighlighting.enabled"

If you want to disable it only for F# you can use:
    "[fsharp]": {
        "editor.semanticHighlighting.enabled": false
    }


Release plan

Purpose

As a customer
I want / need an overview of the product capabilities
So that I can adapt to the product use

Involved team members

PO

Touched or interesting artifacs

planning document

Expected result

releases planned

Planning documents used instead of git


delete vertex with limit throws errors

Syntax
delete vertex XXXXXX  limit 10000
Does NOT work. Error --
com.orientechnologies.orient.core.sql.OCommandSQLParsingException: Error on parsing command at position #0: Class 'LIMIT was not found
and same table ...................
delete vertex XXXXXX where 1=1  limit 10000
WORKS with a dummy where clause. 10000 rows deleted.

fixed in develop branch


CLOUDSTACK-9323: Fix cancel host maintenance can…

Fix cancel host maintenance so that if maintenance is cancelled the host come back to normal state gracefully.
Added marvin tests for host maintennace.

@abhinandanprateek I don't see the Marvin test case in the PR.  Have you pushed the latest commit?
Also, most of the changes seem to be formatting changes in non-related parts of the class which hides the actual fix.  Would it be possible to reverse these formatting changes to reduce the size of the patch to only the change in doCancelMaintenance?

@jsb added the marvin file and reverted to pre-commit formatted code.

Marvin test output:
root@ccp:~/cloudstack(host-maint)# ./host_maint.sh
++ date

echo Mon Mar 28 11:47:45 IST 2016
Mon Mar 28 11:47:45 IST 2016
TMP=/tmp
CLOUDDIR=/root/cloudstack
mkdir -p /tmp/simulator/smoke/misc
nosetests --with-xunit --xunit-file=/tmp/quagga/test_quagga.xml --with-marvin --marvin-config=/root/cloudstack/advanced.cfg /root/cloudstack/test/integration/component/test_host_maintenance.py -s -a tags=advanced,required_hardware=false --zone=Bootcamp --hypervisor=XenServer

==== Marvin Init Started ====
=== Marvin Parse Config Successful ===
=== Marvin Setting TestData Successful===
==== Log Folder Path: /tmp//MarvinLogs//Mar_28_2016_11_47_46_LB5B4I. All logs will be available here ====
=== Marvin Init Logging Successful===
==== Marvin Init Successful ====
test_01_cancel_host_maintenace (integration.component.test_host_maintenance.TestHostMaintenance)
Hypervisor = 34777316-62ae-4590-868a-71daa23dad3d
Hypervisor = 2e872579-efb9-4bb2-ae00-dfb21a994f08
Create VMs as there are not enough vms to check host maintenance
Creating vms = 5
Using template 7e374f74-b471-46bd-bbbe-f457d5376acc
Using service offering 94589b7c-ef32-484b-8ef7-59ad1ba6b69b
Using template 7e374f74-b471-46bd-bbbe-f457d5376acc
Using service offering 9c95b1bf-0ca3-4bba-a00b-1fe61eb078be
VM create = a0901dc0-a568-4d78-8d46-ddf976f6af95
VM create = ebba3c2b-d950-4969-ae34-2cc785023ef9
VM create = 0a9635be-5599-4259-a89b-ad9017e2d4e7
VM create = 9a885588-db3f-4aca-9885-c931fbd6ea71
Host with id 34777316-62ae-4590-868a-71daa23dad3d is in prepareHostForMaintenance
Host with id 34777316-62ae-4590-868a-71daa23dad3d is in cancelHostMaintenance
Host with id 2e872579-efb9-4bb2-ae00-dfb21a994f08 is in prepareHostForMaintenance
Host with id 2e872579-efb9-4bb2-ae00-dfb21a994f08 is in cancelHostMaintenance
=== TestName: test_01_cancel_host_maintenace | Status : SUCCESS ===
===final results are now copied to: /tmp//MarvinLogs/test_host_maintenance_CXM1EL===
++ date

echo Mon Mar 28 11:58:36 IST 2016
Mon Mar 28 11:58:36 IST 2016

Output from marvin test:
TMP=/tmp
CLOUDDIR=/root/cloudstack-apple
mkdir -p /tmp/simulator/smoke/misc
nosetests --with-xunit --xunit-file=/tmp/test_quagga.xml --with-marvin --marvin-config=/root/cloudstack-apple/advanced.cfg /root/cloudstack-apple/test/integration/component/test_host_maintenance.py -s -a tags=advanced,required_hardware=true --zone=Bootcamp --hypervisor=KVM
==== Marvin Init Started ====
=== Marvin Parse Config Successful ===
=== Marvin Setting TestData Successful===
==== Log Folder Path: /tmp//MarvinLogs//Mar_30_2016_10_02_53_F4MC43. All logs will be available here ====
=== Marvin Init Logging Successful===
==== Marvin Init Successful ====
1 Hypervisor = 3e270f2d-f054-4cbe-85e1-0fbc30e8a414
1 Hypervisor = 6a34ca0a-ac2f-44db-a8f4-96b7de01dcd4
Host with id 3e270f2d-f054-4cbe-85e1-0fbc30e8a414 is in prepareHostForMaintenance
Host with id 3e270f2d-f054-4cbe-85e1-0fbc30e8a414 is in cancelHostMaintenance
Host with id 6a34ca0a-ac2f-44db-a8f4-96b7de01dcd4 is in prepareHostForMaintenance
Host with id 6a34ca0a-ac2f-44db-a8f4-96b7de01dcd4 is in cancelHostMaintenance
=== TestName: test_01_cancel_host_maintenace_with_no_migration_jobs | Status : SUCCESS ===
2 Hypervisor = 3e270f2d-f054-4cbe-85e1-0fbc30e8a414
2 Hypervisor = 3e270f2d-f054-4cbe-85e1-0fbc30e8a414
2 Hypervisor = 6a34ca0a-ac2f-44db-a8f4-96b7de01dcd4
2 Hypervisor = 6a34ca0a-ac2f-44db-a8f4-96b7de01dcd4
Create VMs as there are not enough vms to check host maintenance
Create VMs as there are not enough vms to check host maintenance
Creating vms = 5
Creating vms = 5
Using template a18cc46d-d4e8-476f-bdf6-9f3e5818a200
Using template a18cc46d-d4e8-476f-bdf6-9f3e5818a200
Using service offering 5bee807e-602a-4203-b9c3-ed72797b1d93
Using service offering 5bee807e-602a-4203-b9c3-ed72797b1d93
VM create = b67468ff-8120-42d7-87fe-2c7361c4afd3
VM create = b67468ff-8120-42d7-87fe-2c7361c4afd3
VM create = 8309716e-03ac-442a-ab18-a80748d5988c
VM create = 8309716e-03ac-442a-ab18-a80748d5988c
VM create = af8d8991-0ee2-44ca-bee4-d706f883dc0a
VM create = af8d8991-0ee2-44ca-bee4-d706f883dc0a
VM create = 1bacebb1-9be7-40a1-99f3-7b7e9deb45b7
VM create = 1bacebb1-9be7-40a1-99f3-7b7e9deb45b7
VM create = eac8b28d-0bea-4912-b6e4-3402ac07e802
VM create = eac8b28d-0bea-4912-b6e4-3402ac07e802
Host with id 3e270f2d-f054-4cbe-85e1-0fbc30e8a414 is in prepareHostForMaintenance
Host with id 3e270f2d-f054-4cbe-85e1-0fbc30e8a414 is in prepareHostForMaintenance
Vms found = 2
Vms found = 2
VirtualMachine on Hyp id = 8309716e-03ac-442a-ab18-a80748d5988c is in Migrating
VirtualMachine on Hyp id = 8309716e-03ac-442a-ab18-a80748d5988c is in Migrating
Host with id 3e270f2d-f054-4cbe-85e1-0fbc30e8a414 is in cancelHostMaintenance
Host with id 3e270f2d-f054-4cbe-85e1-0fbc30e8a414 is in cancelHostMaintenance
Host with id 6a34ca0a-ac2f-44db-a8f4-96b7de01dcd4 is in prepareHostForMaintenance
Host with id 6a34ca0a-ac2f-44db-a8f4-96b7de01dcd4 is in prepareHostForMaintenance
Vms found = 4
Vms found = 4
VirtualMachine on Hyp id = 8309716e-03ac-442a-ab18-a80748d5988c is in Migrating
VirtualMachine on Hyp id = 8309716e-03ac-442a-ab18-a80748d5988c is in Migrating
Host with id 6a34ca0a-ac2f-44db-a8f4-96b7de01dcd4 is in cancelHostMaintenance
Host with id 6a34ca0a-ac2f-44db-a8f4-96b7de01dcd4 is in cancelHostMaintenance
=== TestName: test_02_cancel_host_maintenace_with_migration_jobs | Status : SUCCESS ===
===final results are now copied to: /tmp//MarvinLogs/test_host_maintenance_RF5DFR===
++ date
echo Wed Mar 30 10:06:16 IST 2016 Wed Mar 30 10:06:16 IST 2016

@abhinandanprateek Looks like jenkins found some problems with your index. I'd suggest rebasing against the current master making all the needed merges and pushing again.

@abhinandanprateek would you mind rebasing to current master so I can test this in my CI?  Thanks...

@cristofolini @swill rebased the PR, up for CI now, Thank you.

CI RESULTS
HAS FAILURES, NEEDS WORK!
Please accress the following issue.
======================================================================
ERROR: test_02_cancel_host_maintenace_with_migration_jobs (integration.component.test_host_maintenance.TestHostMaintenance)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/data/git/cs1/cloudstack/test/integration/component/test_host_maintenance.py", line 277, in test_02_cancel_host_maintenace_with_migration_jobs
    self.vmlist = self.createVMs(listHost[0].id, no_vm_req)
  File "/data/git/cs1/cloudstack/test/integration/component/test_host_maintenance.py", line 108, in createVMs
    self.logger.debug("Using template %s " % self.template.id)
AttributeError: 'str' object has no attribute 'id'

Associated Uploads
test_host_maintenance_P9AKTO:

results.txt

Uploads will be available until 2016-06-04 00:00:00 +0000 GMT
Comment created by upr comment.

@swill In my environment I am using macchinina templates for testing. These are not there by default but are pretty handy in testing due to their small size.  Adding these to CI environment will speed up many such tests. If it is lot of work I will revert to using standard CentOS templates, comments ?
                     "template_name_xen" : "macchinina-xen",
                     "template_name_kvm" : "macchinina-kvm",

cc @jburwell

On comparing test execution times with macchinina Vs Centos builtin template I did not find much of a difference. So modifying the test to go with the standard builtin templates. @swill

Updated marvin test to use the builtin template. @swill

@abhinandanprateek: Thank you.  I will run the whole set of tests against it again tonight.  I have to let the current tests finish and run this on its own because I have to change the tests run specifically for this PR.  I should have results in the morning.

@abhinandanprateek slight delay.  I am going to blow away my CI setup and reinstall it now to get it running on SSD drives so I can better parallelize my tests.  This will delay this test till at least tomorrow evening.

@swill if it is possible to run integration/component/test_host_maintenance.py first than please do that as it is a new test added for this fix, followed by full marvin suite.

CI RESULTS
84/85 TESTS PASSED
The test that failed is a test that commonly fails in my environment and has been verified to be an environment issue.
Associated Uploads
test_host_maintenance_X027L5:

results.txt

test_vpc_routers_093RH7:

results.txt

Uploads will be available until 2016-06-07 02:00:00 +0200 CEST
Comment created by upr comment.

@abhinandanprateek tests have passed, thanks for the updates.  Can you do a push -f to kick off a Jenkins run so we can try to get this PR all green.
@jburwell Does this pass your code review now?
I am trying to get two independent LGTM code reviews before merging whenever possible, so if you have reviewed the code, please let me know.

I have gone through the code, it LGTM

@swill just reviewed.
@abhinandanprateek I have a few more comments to be addressed on the test cases.

@abhinandanprateek, I believe you are away this week, but can your address @jburwell's comments when you have a chance.  Thanks.

@swill @jburwell the concerns noted above are taken care off.

@jburwell can I get your LGTM?  I will run CI on this again today because some code has changed since the last run.

LGTM for code

Sorry for the delay on this one guys.  I think this one is ready now...


Support Windows

I think a good approach at this would be using AutoHotkey.

Are you working on this? I think @enko ´s idea is very good since no additional programming would be required and AHK is known to work good. Also, it could be of interest looking at WSL (Windows subsystem for linux) as some people claim it is faster than native version of emacs. Although more configuration would be required probably (x server and clipboard sync).

Hi @jaimehrubiks !
I've done zero research into implementing this on Windows. You and @enko are way ahead of me on this! 👍
There are a few big tickets to get out of the way before we implement this one but I'll ping both of you when we're ready.
Thanks for scouting this out @jaimehrubiks and @enko !

This freeware tool solves the Windows use-case pretty well (and will even bring what is already in the text area into emacs)
https://www.listary.com/text-editor-anywhere


Resistance (bounce) when reaching max/min values

I started work on this, but realized that we'll have to animate from the "bounce" state back to normal. This is fine, but we'd have to follow the state of the squares during animation to update audio. I'll return to this some other time, maybe after release. Or someone else can do it.

Alternative solution that's easier: don't notify of values that fall outside the accepted boundaries, ever. That way we can animate to the "acceptable" position without any audio issues.

Implemented in https://github.com/superjohan/contrast/commit/80fd502cb8afd37c12ffc33a6cb5b5f360a88ff5


rivaling http and tcp router with wrong priority

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
Backstory: I'm running traefik in a docker setup behind a nat enabled router. A wildcard certificate is in place and functional. A freshly installed kubernetes setup (with its own traefik) was added to the primary traefik as fallback (via tcp router) for everything that's not explicitly defined. Both traefiks will share the same domain.
Issues became apparent when response code and content of a testing service were different in my browser and with curl. Digging deeper a http/2 feature called connection coalescing showed up causing the issue: The browser considers requests to different subdomains belonging to the same origin as long as the CNs on the certificate and the resolved ips for the subdomains are identical, and subsequently reuses an existing http/2 connection. Since this is no new connection, the tcp router has no chance to pick up the request causing 404 errors.
To solve this issue, I tried adding another http fallback router returning 421 (misdirected request) causing browsers to try again with a fresh connection. Surprisingly that http router took precedence over the tcp router contrary to your documentation stating

If both HTTP routers and TCP routers listen to the same entry points, the TCP routers will apply before the HTTP routers. If no matching route is found for the TCP routers, then the HTTP routers will take over.

Currently, curl is always returning the 421, without even reusing a connection.
What did you see instead?
Browser and curl are now showing the 421 for every catched request.
What version of Traefik are you using?

The primary traefik deployment:
version: "3"

services:
  traefik:
    image: traefik:v2.10
    restart: always
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - $PWD/traefik/traefik.yaml:/etc/traefik/traefik.yaml
      - $PWD/traefik/acme:/etc/traefik/acme
      - $PWD/traefik/dynamic:/etc/traefik/dynamic
    networks:
      public:
        ipv4_address: <IP_ADDRESS>
      shared:
    labels:
      - "traefik.enable=true"
      - "traefik.http.routers.dashboard.rule=Host(`traefik.domain.com`)"
      - "traefik.http.routers.dashboard.service=api@internal"
      - "traefik.http.routers.dashboard.entrypoints=secure"
      - "traefik.http.services.dashboard.loadbalancer.server.port=80"
      - "traefik.http.routers.dashboard.tls=true"
      - "traefik.http.routers.dashboard.tls.certresolver=default"
      - "traefik.http.routers.dashboard.tls.domains[0].main=domain.com"
      - "traefik.http.routers.dashboard.tls.domains[0].sans=*.domain.com"

Dynamic content: The tcp router as catchall, then the http 421 workaround for open http/2 connections.
tcp:
  routers:
    catchall:
      entryPoints:
        - secure
      service: catchall
      rule: HostSNI(`*`)
      priority: 2
      tls:
        passthrough: true

  services:
    catchall:
      loadBalancer:
        servers:
          - address: <IP_ADDRESS>:443 # the k8s ingress

http:
  routers:
    catchallfix:
      entryPoints:
        - secure
      rule: PathPrefix(`/`)
      priority: 1
      middlewares:
        - killswitch
      service: catchallfix
      tls: true

  services:
    catchallfix:
      loadBalancer:
        servers:
          - url: https://<IP_ADDRESS>:9999 # nothing running here, just to satisfy traefik

  middlewares:
    killswitch:
      plugin:
        noop:
          responseCode: 421 # see https://plugins.traefik.io/plugins/634ed49499550ba2e3b43d5c/noop

If applicable, please paste the log output in DEBUG level
(can be provided, but needs a lot of filtering and trimming...)

Hello @max06,
Thanks for reporting this!
This seems to be the same report as https://github.com/traefik/traefik/issues/10186.
Please take a look at https://github.com/traefik/traefik/issues/10186#issuecomment-1781200577 for more explanations.
I'm closing this issue as a duplicate, but don't hesitate to reopen it if you think this is not the case.

Hello @rtribotte , thank you for your reply.
Yes, that issue matches mine. The documentation misses any mention of a different handling of tcp and tls-tcp, similar to http. $someone should find the chance to update this, since I've found multiple similar issues in your own community forum regarding the same confusion.
And while we're on it: Is there any preferred solution to my original question? I'm not a big fan of re-encrypting the traffic multiple times if there's no need for that. And somewhere on that way I'm loosing the original client ip... different issue.

Hello @max06,
To make the catchAll TCP-TLS router take precedence over the HTTPS catchAll router, you can redefine the TCP-TLS router rule to prevent Traefik from considering it as a "catchAll" rule, while being a rule matching any host.
tcp:
  routers:
    catchall:
      entryPoints:
        - secure
      service: catchall
      # keep the catchAll behavior in regards to matching, but prevent Traefik from detecting this as a "catchAll" router.
      rule: !HostSNI(`foobar`)
      priority: 1
      tls:
        passthrough: true

This is only an example, the rule HostSNI(`*`) || HostSNI(`*`) would have worked too.
Here is the explanation of why:
https://github.com/traefik/traefik/blob/40de310927507c8c2e24b0ac6d32091e3c167e9b/pkg/muxer/tcp/mux.go#L158-L161

Oh no.... I was so close 😂
Thank you, I'll give it a try this evening!


Support .git-less remotes

I tend to have remotes without .git:
$ git remote -v
origin	https://github.com/hugovk/tinytext (fetch)
origin	https://github.com/hugovk/tinytext (push)
$ git runs
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.11/bin/watch_gha_runs", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/watchgha/watch_runs.py", line 82, in main
    url = f"https://api.github.com/repos/{m[1]}/actions/runs?per_page=40&branch={branch_name}"
                                          ~^^^
TypeError: 'NoneType' object is not subscriptable

And sometimes with a trailing /:
$ git remote -v
origin	https://github.com/hugovk/norwegianblue/ (fetch)
origin	https://github.com/hugovk/norwegianblue/ (push)
$ git runs
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.11/bin/watch_gha_runs", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/watchgha/watch_runs.py", line 82, in main
    url = f"https://api.github.com/repos/{m[1]}/actions/runs?per_page=40&branch={branch_name}"
                                          ~^^^
TypeError: 'NoneType' object is not subscriptable

Thanks!

Oops, had to fix it for .git URLs :)  https://github.com/nedbat/watchgha/commit/1babcc681eb0f44343cb10f8fdeb91a5e9c9765f


File gives hints on every line

Issue Type: Bug
In a fresh, empty folder, create a file called features.js:
module.exports = {
    accessibilityHelp: {
      entry: 'vs/editor/standalone/browser/accessibilityHelp/accessibilityHelp',
      worker: undefined,
    },
    bracketMatching: {
      entry: 'vs/editor/contrib/bracketMatching/bracketMatching',
      worker: undefined,
    },
    caretOperations: {
      entry: 'vs/editor/contrib/caretOperations/caretOperations',
      worker: undefined,
    },
    clipboard: {
      entry: 'vs/editor/contrib/clipboard/clipboard',
      worker: undefined,
    },
    codelens: {
      entry: 'vs/editor/contrib/codelens/codelensController',
      worker: undefined,
    },
    colorDetector: {
      entry: 'vs/editor/contrib/colorPicker/colorDetector',
      worker: undefined,
    },
    comment: {
      entry: 'vs/editor/contrib/comment/comment',
      worker: undefined,
    },
    contextmenu: {
      entry: 'vs/editor/contrib/contextmenu/contextmenu',
      worker: undefined,
    },
    coreCommands: {
      entry: 'vs/editor/browser/controller/coreCommands',
      worker: undefined,
    },
    cursorUndo: {
      entry: 'vs/editor/contrib/cursorUndo/cursorUndo',
      worker: undefined,
    },
    dnd: {
      entry: 'vs/editor/contrib/dnd/dnd',
      worker: undefined,
    },
    find: {
      entry: 'vs/editor/contrib/find/findController',
      worker: undefined,
    },
    folding: {
      entry: 'vs/editor/contrib/folding/folding',
      worker: undefined,
    },
    format: {
      entry: 'vs/editor/contrib/format/formatActions',
      worker: undefined,
    },
    gotoDeclarationCommands: {
      entry: 'vs/editor/contrib/goToDeclaration/goToDeclarationCommands',
      worker: undefined,
    },
    gotoDeclarationMouse: {
      entry: 'vs/editor/contrib/goToDeclaration/goToDeclarationMouse',
      worker: undefined,
    },
    gotoError: {
      entry: 'vs/editor/contrib/gotoError/gotoError',
      worker: undefined,
    },
    gotoLine: {
      entry: 'vs/editor/standalone/browser/quickOpen/gotoLine',
      worker: undefined,
    },
    hover: {
      entry: 'vs/editor/contrib/hover/hover',
      worker: undefined,
    },
    inPlaceReplace: {
      entry: 'vs/editor/contrib/inPlaceReplace/inPlaceReplace',
      worker: undefined,
    },
    inspectTokens: {
      entry: 'vs/editor/standalone/browser/inspectTokens/inspectTokens',
      worker: undefined,
    },
    iPadShowKeyboard: {
      entry: 'vs/editor/standalone/browser/iPadShowKeyboard/iPadShowKeyboard',
      worker: undefined,
    },
    linesOperations: {
      entry: 'vs/editor/contrib/linesOperations/linesOperations',
      worker: undefined,
    },
    links: {
      entry: 'vs/editor/contrib/links/links',
      worker: undefined,
    },
    multicursor: {
      entry: 'vs/editor/contrib/multicursor/multicursor',
      worker: undefined,
    },
    parameterHints: {
      entry: 'vs/editor/contrib/parameterHints/parameterHints',
      worker: undefined,
    },
    quickCommand: {
      entry: 'vs/editor/standalone/browser/quickOpen/quickCommand',
      worker: undefined,
    },
    quickFixCommands: {
      entry: 'vs/editor/contrib/quickFix/quickFixCommands',
      worker: undefined,
    },
    quickOutline: {
      entry: 'vs/editor/standalone/browser/quickOpen/quickOutline',
      worker: undefined,
    },
    referenceSearch: {
      entry: 'vs/editor/contrib/referenceSearch/referenceSearch',
      worker: undefined,
    },
    rename: {
      entry: 'vs/editor/contrib/rename/rename',
      worker: undefined,
    },
    smartSelect: {
      entry: 'vs/editor/contrib/smartSelect/smartSelect',
      worker: undefined,
    },
    snippets: {
      entry: 'vs/editor/contrib/snippet/snippetController2',
      worker: undefined,
    },
    suggest: {
      entry: 'vs/editor/contrib/suggest/suggestController',
      worker: undefined,
    },
    toggleHighContrast: {
      entry: 'vs/editor/standalone/browser/toggleHighContrast/toggleHighContrast',
      worker: undefined,
    },
    toggleTabFocusMode: {
      entry: 'vs/editor/contrib/toggleTabFocusMode/toggleTabFocusMode',
      worker: undefined,
    },
    transpose: {
      entry: 'vs/editor/contrib/caretOperations/transpose',
      worker: undefined,
    },
    wordHighlighter: {
      entry: 'vs/editor/contrib/wordHighlighter/wordHighlighter',
      worker: undefined,
    },
    wordOperations: {
      entry: 'vs/editor/contrib/wordOperations/wordOperations',
      worker: undefined,
    },
  };

Observe: each line has a refactoring hint and the lightbulb is permanently visible

VS Code version: Code 1.22.1 (950b8b0d37a9b7061b6f0d291837ccc4015f5ecd, 2018-04-06T00:21:13.607Z)
OS version: Darwin x64 17.5.0

Is this just one decoration spawning many lines or actually on per line?

Duplicate of #47346. It's a single hint, the rendering is just off. Probably should look into this for the recovery build


Revert bundle changes

Reverts these changes because they might be causing an issue in staging
@orta seems like being reasonably sure isn't enough 😅

merge on green


versions

Sensitive to versions. need to document requirements:
R version 3.3.1 (2016-06-21)
Platform: x86_64-redhat-linux-gnu (64-bit)
Running under: Fedora 23 (Twenty Three)

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8
 [4] LC_COLLATE=en_US.UTF-8     LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8
 [7] LC_PAPER=en_US.UTF-8       LC_NAME=C                  LC_ADDRESS=C
[10] LC_TELEPHONE=C             LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] DT_0.1               visNetwork_1.0.2     shinydashboard_0.5.1 dplyr_0.5.0
[5] data.table_1.9.6     synapseClient_1.13-4 shiny_0.14           devtools_1.8.0

loaded via a namespace (and not attached):
 [1] Rcpp_0.12.7     xml2_0.1.1      magrittr_1.5    xtable_1.8-2    rjson_0.2.15
 [6] R6_2.1.3        tools_3.3.1     DBI_0.4-1       git2r_0.11.0    htmltools_0.3.5
[11] rversions_1.0.2 yaml_2.1.13     lazyeval_0.2.0  digest_0.6.10   assertthat_0.1
[16] tibble_1.0      tidyr_0.5.1     htmlwidgets_0.7 bitops_1.0-6    rsconnect_0.4.3
[21] RCurl_1.95-4.8  curl_0.9.3      memoise_0.2.1   mime_0.5        jsonlite_1.1
[26] httpuv_1.3.3    chron_2.3-47

kenny figured it out


dolfinx requires cffi

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Note page(0) and page(1) return the same results

I didn't realize that pagination starts at 1 rather than 0, which led to some time debugging this afternoon. If you're amenable to a small change to the readme, noting that point might be useful for others.
Thanks for the great gem!

Thank you!

Thank you!


access_logs.s3.bucket empty errors

Failure configuring ALB attributes: ValidationError: The value of 'access_logs.s3.bucket' cannot be empty 	status code: 400, request id: cd69fceb-bb12-11e7-a276-19494eb0aecb
I get the above message when I do not set any value for log_bucket. Currently I can see that 'enabled' for access_logs is based on setting the log_bucket:
 access_logs {     bucket  = "${var.log_bucket}"     prefix  = "${var.log_prefix}"     enabled = "${var.log_bucket != ""}"   }
I think it would be better to have this as a separate value (enable_access_logs ?) so it is possible to pass the log_bucket but not enable the access_logs. What do you think about this?
Also, the job failed when setting the log_bucket to a chosen one when it has just been created - I think s3 didn't converge fully before the bucket assignement to ALB - maybe a backoff timer or something. Don't know if terraform actually has something to fix this

Agree, PR is always welcome

Hey @mbolek . Thanks for the issue submission. Can you show both how the module was called and the output of terraform when this issue happens? Terraform version info would also be helpful.

Sure, here's the module usage:
module "alb" {
  source = "terraform-aws-modules/alb/aws"

  alb_is_internal     = "false"
  alb_name            = "${var.alb_name}"
  alb_protocols       = ["${var.alb_protocols}"]
  alb_security_groups = "${var.alb_security_groups}"
  region              = "${var.aws_region}"

  log_bucket = "${var.log_bucket}"
  log_prefix = "${var.log_prefix}"

  backend_port      = "${var.backend_port}"
  backend_protocol  = "${var.backend_protocol}"
  certificate_arn   = "${var.certificate_arn}"
  cookie_duration   = "${var.cookie_duration}"
  health_check_path = "${var.health_check_path}"
  subnets           = ["${var.subnets}"]
  vpc_id            = "${var.vpc_id}"
  tags              = "${var.tags}"
}

log_prefix has some value, but log_bucket is empty:
log_bucket = ""
log_prefix="${var.alb_name}"

And here's the output, same as above:
Error: Error applying plan:

1 error(s) occurred:

* module.alb.module.alb.aws_alb.main: 1 error(s) occurred:

* aws_alb.main: Failure configuring ALB attributes: ValidationError: The value of 'access_logs.s3.bucket' cannot be empty
	status code: 400, request id: cd69fceb-bb12-11e7-a276-19494eb0aecb

I'm using terraform v0.10.8 and terragrunt 0.13.14
Looking at the TF docs regarding alb. It seems that bucket_name is a required parameter so it probably shouldn't be empty if the entire 'access_logs' block has been added. I'd like to be able to specify all the params but still keep it disabled which is currently impossible since setting log_bucket also sets enabled to true.
I just tested the code from the example dir (small issue with the source of module - it uses '../../../' and I think it's one dir too far) - it works ok, but it has log_bucket and log_prefix set. If I delete both (from examples/test_fixtures/main.tf), I get:
* module.alb.aws_alb.main: 1 error(s) occurred:

* aws_alb.main: Failure configuring ALB attributes: ValidationError: The value of 'access_logs.s3.bucket' cannot be empty
        status code: 400, request id: 7580a60b-bc8f-11e7-8a04-e7ddc10ac001

I think defaults come into play then, and log_prefix is set to "my-alb-logs". I tried with removing the default (examples/test_fixtures/variables.tf) and still get the error:
* module.alb.aws_alb.main: 1 error(s) occurred:

* aws_alb.main: Failure configuring ALB attributes: ValidationError: The value of 'access_logs.s3.bucket' cannot be empty
        status code: 400, request id: 1bf1992d-bc90-11e7-9665-d7f7044996df

Generally, I think that my use case may just be weird - settings vars but disabling access_logs. However, should you consider changing the behaviour (adding variable e.g. enable_access_logs), I can try and create a PR :)

Thanks on the details. If I understand you here, there's some value to you in filling out each of those options for logging but still keeping logging disabled. Not a need I would have anticipated but if that's your use-case, let's work with it.
Re: terragrunt - I haven't used terragrunt so I really don't know if this module is compatible there. This module makes no guarantees to work with that tool but given that terragrunt is just a thin wrapper on terraform, I can't imagine that's the source of any issues.
Re: ../../../ : agreed that it it appears to be one dir too far but when running tests via test kitchen, this arrangement is what works (the proper directory location doesn't). Perhaps we have a bug in kitchen-terraform or test-kitchen itself? While validating the test fixture via terraform validate and fmt in CI, I adjust the path using sed in place to what you suggested to get those commands working. So which to chose? I think there's more benefit to keeping test runs more accessible by leaving the path as is. If someone can think of a better way that resolves both, I'm all ears, but interpolations/variables aren't allowed in a source definition so options are significantly limited.
Let me know if you need help on the PR. It's a relatively small change but we're here to help. :dancer:

I have a similar problem. I have two ALBs which I want to write logs into the same bucket. Now I can't make it, because "bucket already exists" after first ALB has been created.

Interesting. Yea thinking about the case of several ALBs each wanting to talk to the same bucket but likely under different prefixes, it would make more sense to control that bucket outside of this module and in a separate terraform module/set of templates. That likely adds another variable to the mix to determine count of bucket and policy creation. Sound reasonable?

Yes, I prefer to make a creation of buckets outside of modules where it is used because then I can set up more granular bucket policies and control lifecycle more precisely. Will you be able to fix it during next week? (I will be using this module in 8 days) :)

@antonbabenko Yep, I was waiting to give @mbolek a fair shot at a PR but I can patch it up this over the next day or so. Expect to see a review request early next week at latest.

sorry... took some vay-kay. Regarding terragrunt I don't think it should have an impact on the code. And of course, with "../../" issue, tests are more important.
Besides, since in your main.tf, there is an 'access_log' block already, all the params need to be there or it won't work. I'll work on the PR, unless you'll be faster:)


Post install iotedge permissions needed

Description: Iot edge lacks permissions to read it's own config and /var/lib directory post install.
To Reproduce Steps to reproduce the behavior:
Add meta-iotedge layer to yocto build
Build image
Flash image to sd card
Boot from sd card, then provide connection string to /etc/iotedge/config.yaml
Expected behavior: iotedge should simply start working and pulling images
Logs If applicable, add logs to help explain your problem.
Build System (please complete the following information):
OS: Linux x64
Version 3.12.49-11
Target System (please complete the following information):
OS: yocto rocko
Version rocko
Running on variscite VAR-SOM-6UL (imx6ul-var-dart)
To fix this I had to run the following commands:
$ chown -R iotedge /etc/iotedge
$ chown -R iotedge /var/lib/iotedge
There's also a problem with the hostname property in the config, it's not set properly and you have to set it manually. I feel that all these should be taken care of in post install scripts.

Seemingly stale issue.  Closing.


Test delays with “Timed out waiting for the exit barrier block”

Describe the bug
Running snapshot tests with existing screenshots results in test delays and “Timed out waiting for the exit barrier block” messages on console.
To Reproduce
Have this test class.
import SwiftUI
import SnapshotTesting
import XCTest

final class SettingsViewTests: XCTestCase {
  func test_simple_view() {
    let sut = Text("Hello settings")
      .frame(width: 100, height: 100)
    SnapshotTesting.assertSnapshot(matching: sut, as: .image)
  }
}

When it is run without a snapshot being present, it quickly and correctly records the snapshot and reports a failure.
Test Suite 'Selected tests' started at 2023-07-30 14:35:52.852
Test Suite 'SettingsTests.xctest' started at 2023-07-30 14:35:52.853
Test Suite 'SettingsViewTests' started at 2023-07-30 14:35:52.853
Test Case '-[SettingsTests.SettingsViewTests test_simple_view]' started.
/Users/jaanus/Developer/Tact/Tact/Library/Targets/Settings/Tests/SettingsViewTests.swift:30: error: -[SettingsTests.SettingsViewTests test_simple_view] : failed - No reference was found on disk. Automatically recorded snapshot: …

open "/Users/jaanus/Developer/Tact/Tact/Library/Targets/Settings/Tests/__Snapshots__/SettingsViewTests/test_simple_view.1.png"

Re-run "test_simple_view" to test against the newly-recorded snapshot.
Test Case '-[SettingsTests.SettingsViewTests test_simple_view]' failed (0.069 seconds).
Test Suite 'SettingsViewTests' failed at 2023-07-30 14:35:52.922.
	 Executed 1 test, with 1 failure (0 unexpected) in 0.069 (0.069) seconds
Test Suite 'SettingsTests.xctest' failed at 2023-07-30 14:35:52.922.
	 Executed 1 test, with 1 failure (0 unexpected) in 0.069 (0.070) seconds
Test Suite 'Selected tests' failed at 2023-07-30 14:35:52.923.
	 Executed 1 test, with 1 failure (0 unexpected) in 0.069 (0.071) seconds
Program ended with exit code: 1

When the snapshot is present, the test works correctly (passes or fails as expected). However, the process has several seconds of delay in the end.
Test Suite 'Selected tests' started at 2023-07-30 14:36:29.432
Test Suite 'SettingsTests.xctest' started at 2023-07-30 14:36:29.432
Test Suite 'SettingsViewTests' started at 2023-07-30 14:36:29.432
Test Case '-[SettingsTests.SettingsViewTests test_simple_view]' started.
Test Case '-[SettingsTests.SettingsViewTests test_simple_view]' passed (0.071 seconds).
Test Suite 'SettingsViewTests' passed at 2023-07-30 14:36:29.503.
	 Executed 1 test, with 0 failures (0 unexpected) in 0.071 (0.071) seconds
Test Suite 'SettingsTests.xctest' passed at 2023-07-30 14:36:29.504.
	 Executed 1 test, with 0 failures (0 unexpected) in 0.071 (0.072) seconds
Test Suite 'Selected tests' passed at 2023-07-30 14:36:29.504.
	 Executed 1 test, with 0 failures (0 unexpected) in 0.071 (0.072) seconds
2023-07-30 14:36:32.011572+0300 xctest[18827:731647] [client] Timed out waiting for the exit barrier block. activeSendTransactions=0
Program ended with exit code: 0

Expected behavior
When snapshots are present, the test should complete without any delay.
Environment

swift-snapshot-testing version 1.11.1
Xcode Version 14.3.1 (14E300c)
Swift 5.8.1
OS: macOS Ventura 13.5 (22G74)
iOS Simulator: iPhone 14 Pro, iOS 16.4

+1 Experiencing this issue as well.

I just created a new project without this library as a dependency and experienced the same issue. So this would indicate that it is not an issue of swift-snapshot-testing, and could be an Apple issue. Please test a project without swift-snapshot-testing and report back if you experience the same thing. Thank you.

I just created a new project without this library as a dependency and experienced the same issue. So this would indicate that it is not an issue of swift-snapshot-testing, and could be an Apple issue. Please test a project without swift-snapshot-testing and report back whether you experience the same thing. Thank you.

I just created a new project without this library as a dependency and experienced the same issue. So this would indicate that it is not an issue of swift-snapshot-testing and could therefore be an Apple issue. Please test a project without swift-snapshot-testing and report back whether you experience the same thing. Thank you.

This is an obsolete bug and I have not seen it lately, so, closing.
The slowness I observed was actually this: https://github.com/pointfreeco/swift-snapshot-testing/pull/664


Update remote_host_management.md

Adding Windows Server 2016 and Nano Server to the list of Supported Hyper-V host combinations with Hyper-V Manager in Windows 10. Thanks!

Thank you :)

Thank you :)


Create daewon-h.txt

@repryty Thank you for your patience while waiting for a review and processing your request. Pull request merged.