Connectors redesign

This PR is a complete redesign of how sources and sinks work in Arroyo. Currently, details about each source or sink (which we'll generically call connectors, since they typically connect Arroyo to an external system) are hard-coded in many parts of our system, including:

The web console
API
Database
SQL layer
Compiler
Pipeline representation
Worker code

This makes the process of adding a new connector very difficult and tedious, and makes it very had to change common details about how connectors interact with the rest of the system. For example, we previously added DDL support for creating connectors in SQL but it only worked for Kafka.
This PR attempts to solve that problem by genericizing all of the interactions between connectors and the rest of the system—in other words, none of those layers will know about kafka or server-sent events, they just need to know about "connectors" as a generic concept.
All of the connector-specific logic is now centralized into a single struct in the new arroyo-connectors crate, which implements the Connector trait.
The metadata for that connector is sent to the web ui to build the configuration interfaces and determines how the connector is constructed via SQL DDL statements:

I've also reworked the console to reduce the emphasis on the connection/source/sink distinction, which was confusing for users. Now there is a single flow to create a source or sink that starts with the type of connection the user would like to create (like Kafka) without requiring the user to juggle multiple types of data.

This is still a WIP. Some known issues:

~Nexmark sources can't be created via SQL yet, as we need to figure out how to handle SQL schemas for connectors with fixed schemas~ this is going to stay out of scope for this PR, as it requires better datafusion struct support (or possibly using CREATE TABLE EXTERNAL)
~Impulse source hasn't been migrated yet~
~RawJson schemas aren't properly inferred~
There's no way to delete connections or ~view their configuration~
Update tables are not well tested and likely have issues
~Lots of tests need to be updated~
~Connector icons aren't available yet in the UI~

I tried creating an eventsource connection with Mastodon and am getting an error

Received error from server: UnexpectedResponse(404)

What do you think of adding a short README when we add a new crate? Just one or two sentences would be helpful for new developers.

I tried creating an eventsource connection with Mastodon and am getting an error

Received error from server: UnexpectedResponse(404)

@jbeisen can you try with the configuration in the updated tutorial:
https://github.com/ArroyoSystems/arroyo-docs/blob/04f3aa6595809fa6d0357b17921aaabcbac3cae8/tutorial/mastodon.mdx


Use shorthand for FieldPanel / InlinePanel in docs

Pertinent section of the Wagtail docs
https://docs.wagtail.org/en/stable/getting_started/tutorial.html, https://docs.wagtail.org/en/stable/reference/pages/panels.html and elsewhere
Details
#12557 has introduced the ability to use plain strings in panel definitions, in place of FieldPanel and InlinePanel. We want to encourage users to adopt this new more compact approach, and thus need to update the documentation to cover it.
The tutorial would start with content_panels being defined as a list of strings, and would introduce the concept of panel objects at the point that we bring in MultiFieldPanel and FieldPanel('authors', widget=forms.CheckboxSelectMultiple) in the Authors section. To avoid introducing too many concepts at once, I'd suggest splitting this into two steps, where we first use
        MultiFieldPanel(['date', 'authors'], heading="Blog information"),

and then change 'authors' to a FieldPanel to give it a nicer widget.
The panels reference page should be updated to say that a plain string in a panel definition is equivalent to a FieldPanel or InlinePanel with no arguments (depending on whether the string corresponds to a field or a one-to-many relation). All other parts of the documentation that just use panel definitions incidentally in code snippets can probably be updated to use plain strings with no further clarification, although this is lower priority.
Working on this
Anyone can contribute to this. View our contributing guidelines, add a comment to the issue once you’re ready to start.

I wanted to work on this and decided to try plain strings in panel definitions as shown here
class BlogIndexPage(Page):
    intro = RichTextField(blank=True)
    content_panels = Page.content_panels + ["intro"]

instead of
class BlogIndexPage(Page):
    intro = RichTextField(blank=True)
    content_panels = Page.content_panels + [
        FieldPanel("intro")
    ]

And it is giving the error
    self.children = [child.bind_to_model(self.model) for child in self.children]
                     ^^^^^^^^^^^^^^^^^^^
AttributeError: 'str' object has no attribute 'bind_to_model'

I don't know should I still proceed to updating the documentation.

@unyimeudemy are you using the latest in-development version? #12557 was merged only recently, it's not yet released. You can follow the guide at https://docs.wagtail.org/en/stable/contributing/developing.html to install Wagtail from the latest main branch.

Thank you @laymonage , it's working now. I later pulled the latest changes and then tested on the BlogIndexPage's model of the bakery site as shown below
    # content_panels = Page.content_panels + [
    #     FieldPanel("introduction"),
    #     FieldPanel("image"),
    # ]

    content_panels = Page.content_panels + ["introduction", "image"]

Now, I will proceed to updating the documentation


Class Issues

I noticed that in the scripts used we have some classes to identify, for example.
List of Product categories
Login Forms
Etc...
As each WordPress site can be built with different tools, it would be interesting to have documentation teaching which classes should be changed.
But it would be even more interesting if there was a copy of the website you used to do the tests, so we could import it onto our servers for testing purposes.
Could you please tell me which theme you used and plugins you had in your environment to replicate the tests?

Could you please tell me which theme you used and plugins you had in your environment to replicate the tests?

https://woocommerce.com/products/storefront/


Add the used Kubernetes Version to kudo version output

Signed-off-by: Andreas Neumann<EMAIL_ADDRESS>What this PR does / why we need it:
This PR shows the used Kubernetes API in the version output. Helps users to understand which version of kubernetes is supported, see #1642

This is great stuff and a welcomed addition... my issue with it is the naming of things (including the title of the PR). I'm against the use of the word kubernetesapi it is not. It is a kubernetes version which very indirectly supports an api. I see that you are getting the version of the API from k8s.io/api so it seems like it defines the API... but this is a client side dependency. The real API is at the server.
The version of k8s.io/api is selected when we bump the client-go library and we keep those in-sync. I wonder if it would be better to expose our client-go version or this "k8s api version" as the client-go version... then people will have somewhere to go to understand the mapping to their server version.
If we don't change to client-go version... then I do feel like we need to express this NOT as a kubernetes API version... but as a client kubernetes API version (which could be confusing... although accurate and seemingly necessary)

I have renamed the field to KubernetesVersion and retrieve it from the client-go library now. You're correct that this is the top-level library that we should use to determine which version we use.
I don't think it makes sense to expose this version as the ClientGo version. It doesn't help the user, and making the jump from "ClientGo" to "KubernetesVersion" might be a bit much.
Thankfully, all Kubernetes components are released together, so the Kubernetes 0.18.6 release contains the ClientGo 0.18.6.
I think KubernetesVersion makes the most sense and helps users to understand, wdyt?

if you are trying to stay away from "ClientGo"... that makes sense!   Could we call it "ClientKubernetesVersion"  or "KubernetesClientGoVersion"... leaving the word kubernetes out wouldn't be good.   ah... naming is hard and this is better than the previous options.   Whatever you think is best.

Thanks. I see your point. I've renamed it to KubernetesClientVersion, I think that fits pretty good.


Start enforcing python formatting & linting in CI

The comment doesn’t say this but presumably it’s just for Fud? Or should we do this for all python scripts?

The comment doesn’t say this but presumably it’s just for Fud? Or should we do this for all python scripts?

I think we should start with fud and maybe add it to other things later. I have to figure out a standard set of tools to use first.

I think we should start with fud and maybe add it to other things later. I have to figure out a standard set of tools to use first.

flake8 is the boring but effective choice! Here's one Actions setup, FWIW:
https://github.com/beetbox/mediafile/blob/d7bea5efbe00c54fbe9ca169a3ffd6c827302e7d/.github/workflows/build.yml#L58-L63

flake8 is the boring but effective choice! Here's one Actions setup, FWIW:
https://github.com/beetbox/mediafile/blob/d7bea5efbe00c54fbe9ca169a3ffd6c827302e7d/.github/workflows/build.yml#L58-L63

88 columns?! Kids these days and their wide screens. 😄

88 columns?! Kids these days and their wide screens. 😄


fix detox 0.72 error duplicated libfbjni.so for Android builds

Description
Android builds were failing for detox 0.72 because of duplicated libfbjni.so.
Very similar to PR https://github.com/ammarahm-ed/react-native-mmkv-storage/pull/336

@ammarahm-ed can you please merge this?
I am facing the same issue.

Thanks for the fix!


REFPLTB-1629: After SSID update in WebUI, client could not connect to…

Signed-off-by: Manigandan Gopalakrishnan<EMAIL_ADDRESS>

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/9041/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/14187/ started

https://jenkins.code.rdkcentral.com/job/build-component-rdk-next/47875/ started.

build-tur-ext Console Log uploaded to Gist URL: https://gist.github.com/3182837e0f2694484428fbaac7f01b50

build-rpi4-extender Console Log uploaded to Gist URL: https://gist.github.com/86933f68e5d35e2e5be368301461d4cc

build-tur-bb Console Log uploaded to Gist URL: https://gist.github.com/5e638025a0dec056963bc87c29a5b654

https://jenkins.code.rdkcentral.com/job/github-triggered/14188/ started

https://jenkins.code.rdkcentral.com/job/build-component-rdk-next/47877/ started.

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/9042/ started

Smoke tests for ['build-tur-bb'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/111751/

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/9077/ started

https://jenkins.code.rdkcentral.com/job/build-component-rdk-next/47958/ started.

Smoke tests for ['build-rpi-broadband'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/111909/

Smoke tests for ['build-tur-bb'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/111910/

Raised another PR.https://github.com/rdkcentral/meta-turris/pull/408


Upgrade to webpack 2

What's changed:

Upgrade dependancies to the latest versions
Update README.md to be webpack 2 compatible
Update the example to work with webpack 2.x.x
Minor README.md cleanup (mainly line lengths limits)

Thanks for the feeback @eeue56. Those points have now been addressed 😃

Published as 4.4.0! Thanks!


Unable to launch cellpose 2.0 GUI on Mac OS 10.14.6 (Mojave)

I was able to install Cellpose 2.0 following the recommended terminal commands below:

conda create --name cellpose python=3.8
conda activate cellpose
python -m pip install cellpose[gui]

However, when I attempt to launch the gui I get error below. Any thoughts how to troubleshoot this?
best,
Justin
======
(cellpose) elstrotj-6CJGH7:~ elstrotj$ python -m cellpose
Traceback (most recent call last):
File "/Users/elstrotj/opt/anaconda3/envs/cellpose/lib/python3.8/runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/Users/elstrotj/opt/anaconda3/envs/cellpose/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/Users/elstrotj/opt/anaconda3/envs/cellpose/lib/python3.8/site-packages/cellpose/main.py", line 7, in
from cellpose import utils, models, io, core
File "/Users/elstrotj/opt/anaconda3/envs/cellpose/lib/python3.8/site-packages/cellpose/utils.py", line 5, in
import cv2
File "/Users/elstrotj/opt/anaconda3/envs/cellpose/lib/python3.8/site-packages/cv2/init.py", line 181, in
bootstrap()
File "/Users/elstrotj/opt/anaconda3/envs/cellpose/lib/python3.8/site-packages/cv2/init.py", line 153, in bootstrap
native_module = importlib.import_module("cv2")
File "/Users/elstrotj/opt/anaconda3/envs/cellpose/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ImportError: dlopen(/Users/elstrotj/opt/anaconda3/envs/cellpose/lib/python3.8/site-packages/cv2/cv2.abi3.so, 2): Symbol not found: ___darwin_check_fd_set_overflow
Referenced from: /Users/elstrotj/opt/anaconda3/envs/cellpose/lib/python3.8/site-packages/cv2/.dylibs/libX11.6.dylib (which was built for Mac OS X 11.0)
Expected in: /usr/lib/libSystem.B.dylib
in /Users/elstrotj/opt/anaconda3/envs/cellpose/lib/python3.8/site-packages/cv2/.dylibs/libX11.6.dylib

hmm it looks like python 3.9 or 3.10, might work, please try that? https://opencv.org/opencv-python-for-apples-m1-chip-a-detective-story-with-a-happy-ending/
if it works let us know and we'll add the info


fix: pacman ghost valid action calculations result in NaNs

In the PacMan environment, when trying to calculate all the valid actions a ghost could take (in check_ghost_wall_collisions in pac_man/utils.py) the invert_mask * jnp.inf call was producing an array full of NaN's where invert_mask == 1. This lead to all actions being valid for ghosts.
Instead, what this line should be doing is a jnp.where call, that conditionally replaces all 1's in invert_mask with jnp.inf.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi, thank you for spotting this bug!
If this is true and that this fixes it, we then need to bump the version of PacMan from PacMan-v0 to PacMan-v1. Could you please make the following changes to the registry and documentation?

I've updated my commit to bump the version of PacMan to v1. Let me know if I've missed anything!

That's perfect, thanks!
I'm struggling to reproduce the issue as I am not finding NaNs in different tests I've done. Would you have a small reproduction of the NaNs that you would be able to share?
Thank you!

I did a bit of digging, and essentially if the function is jited, then jnp.inf * False returns 0 (which works with PacMan), whereas it should return NaN:
https://github.com/google/jax/issues/12233#issuecomment-1238401228
The NaN's don't show up in the action_mask, but essentially zeros out the action_mask for ghosts. I put a breakpoint at the return statement of check_ghost_wall_collisions to see the NaNs in invert_mask. This only shows up if you set the environment variable JAX_DISABLE_JIT=1, which turns jit off.

Here is the script that I run, with the environment variable JAX_DISABLE_JIT=1:
import jax
from jumanji.environments.routing.pac_man import PacMan

if __name__ == "__main__":
    jax.disable_jit(True)

    seed = 2024
    key = jax.random.PRNGKey(seed)
    reset_key, key = jax.random.split(key)

    env = PacMan()

    state, tstep = env.reset(reset_key)

    next_state, tstep = env.step(state, 1)
    ```

When I train with and without the fix, I get the exact same learning curves (same loss at every step), hinting that the behavior has not changed. I wonder if the NaN behavior depends on the version of JAX?
I'm happy to merge this change as it is a cleaner implementation but if the environment behavior has not changed, then we should probably not bump the version. Would you have a way to show that the environment produces NaNs before the fix and not after this change?

Yes, the behavior would be the same, since according to this thread, XLA returns 0 instead of NaN when things are JIT'ed. The issue more comes when debugging: when JAX_DISABLE_JIT=1, you have weird issues with ghosts going through walls. Here's an animated example:

If this doesn't warrant a version bump, then I'm more than happy to change the version back to v0.

Oh that makes complete sense. Since the behavior of the non-jitted environment changes, let's then bump the version.
Thank you for your contribution!


[SPARK-8246] [SQL] Implement get_json_object

This is based on #7485 , thanks to @NathanHowell
Tests were copied from Hive, but do not seem to be super comprehensive. I've generally replicated Hive's unusual behavior rather than following a JSONPath reference, except for one case (as noted in the comments). I don't know if there is a way of fully replicating Hive's behavior without a slower TreeNode implementation, so I've erred on the side of performance instead.

Merged build triggered.

Merged build started.

Test build #39578 has started for   PR 7901 at commit 940c060.

Test build #39578 has finished for   PR 7901 at commit 940c060.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class MesosExternalShuffleClient extends ExternalShuffleClient
public class RegisterDriver extends BlockTransferMessage
case class GetJsonObject(json: Expression, path: Expression)

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #39586 has started for   PR 7901 at commit 25eebef.

Test build #39586 has finished for   PR 7901 at commit 25eebef.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class MesosExternalShuffleClient extends ExternalShuffleClient
public class RegisterDriver extends BlockTransferMessage
case class GetJsonObject(json: Expression, path: Expression)

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #39596 has started for   PR 7901 at commit 18ce26b.

Test build #39596 has finished for   PR 7901 at commit 18ce26b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class GetJsonObject(json: Expression, path: Expression)

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #39671 has started for   PR 7901 at commit c818519.

Test build #39671 has finished for   PR 7901 at commit c818519.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class GetJsonObject(json: Expression, path: Expression)

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #39686 has started for   PR 7901 at commit a7dc6d0.

Test build #39686 has finished for   PR 7901 at commit a7dc6d0.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class GetJsonObject(json: Expression, path: Expression)

Merged build finished. Test PASSed.

Merged build triggered.

Merged into master and 1.5 branch!

Merged build started.

Test build #39726 has started for   PR 7901 at commit 3ace9b9.

Test build #39726 has finished for   PR 7901 at commit 3ace9b9.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class GetJsonObject(json: Expression, path: Expression)
case class FindInSet(left: Expression, right: Expression) extends BinaryExpression

Merged build finished. Test FAILed.


Monitoring multiple resources using metric alerts - Review

I thought this was already set to include hybrid workers (on-prem)? Thanks

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

ID: 37e35140-5a03-b8a1-d17c-b06b7cbb03db
Version Independent ID: 4c16ba87-0e6e-f60e-8271-ff6d9492d4ae
Content: Understand how metric alerts work in Azure Monitor.
Content Source: articles/monitoring-and-diagnostics/alert-metric-overview.md
Service: azure-monitor
GitHub Login: @snehithm
Microsoft Alias: snmuvva

@cconrado, need additional clarification on your comment.
Are you asking if Azure monitor metric alerts work on Hybrid Worker Roles or are saying you tested it on HRW and it didn't work as expected?

This is the same experience when running it from Azure Monitor as well.
The activation alert shows all the objects defined in the dimension, but the deactivation alert shows the object that triggered the alert.

@CraigBoydell shouldnt it be in the activation alert not in the deactivation one. It's not very useful being told after it was resolved. My focus is to make one alert that covers all the on-prem machines.

I'm thinking its a bug at this point... the fact the deactivation does have the triggering object suggests to me that something has been overlooked.
Its not much use to me until this is resolved as we end up with an alert and no specifics on what triggered it.

@CraigBoydell how do we go about resolving this?

@cconrado nothing that is in our control.
To work round this I have created custom queries based on the Perf Log Analytics and used the metric rules where possible.
The JSON payload for these Alert Rules sent to the webhooks and the Email notifications contain the detail of what machines triggered the alert.
I'm currently setting up an extra webhook in the middle to further refine the detail of the alerts before they are sent out as notifications.
The downside is that the metrics don't auto deactivate, but that Azure Functions or maybe some Azure automation can be setup to close out alerts that are no longer current.
Also, in testing I have found that the Alerts can take longer to trigger for the custom queries when compared to the metrics but the added benefit of actually knowing what fired the alert outweighs the delay at this stage.

@CraigBoydell , @cconrado I would like to understand the issues you are seeing. I sent out an email to @cconrado. @CraigBoydell could you reach out to me at snmuvva AT Microsoft DOT com

Closing this issue for now as it has been redirected to email.
@snehithm will greatly appreciate if you can update the thread with the final outcome.


[RESEARCH] HAT ID Autodetect LCD and other RPi HATs

Maybe there is a way to detect LCD hat and others when connected by "HAT ID"
_The Raspberry Pi HAT (Hardware Attached on Top) ID is a mechanism used by the Raspberry Pi to automatically identify a connected HAT and configure the GPIOs and drivers for the HAT, without the need for manual setup. This is part of the HAT specification, which includes a small EEPROM (Electrically Erasable Programmable Read-Only Memory) chip on the HAT itself.
This EEPROM contains information such as:

Manufacturer Information: Who made the HAT.
Product Information: What the HAT is or does.
GPIO Setup: Information about how the HAT uses the GPIO pins.
Device Tree Fragment: A piece of code that helps the Raspberry Pi to configure its hardware to work with the HAT.

When a HAT is attached to a Raspberry Pi, the Pi reads this EEPROM during boot-up to automatically configure the system to work with the HAT, making the process of using add-on boards much simpler for the end user. This feature is particularly useful for non-technical users or in educational settings where ease of use is a priority._
The following shows entries of the LCD Hat:
dtc -I fs /proc/device-tree | grep waveshare
needs to be checked if this is just because we ionstalled the drivers or if this also is a available on a fresh system. Especially during setup to know if a LCD HAT is available is good to know and drivers could just be installed when needed.
But epsecially M2 Hats for RP5 like this support the HAT ID:
https://pineberrypi.com/products/hat-top-2230-2240-for-rpi5
If HAT ID is not working with LCD, try to check with this.

Looks like we are not going with a PCIe HAT that is connected to the GPIO pins - so this is not relevant for the v1.11.0 release ... but in general could be interesting for other HAT modules in the future. Pushing to "Future Release" milestone.


Translate english to korean

This project was very useful to me.
So I want to translate this project into Korean.
Could you allow me to translate this project?
But I don't have much time, so I'm going to do it like a toy project, is that okay?

@trimstray How are you operate translate branch?


Add Dockerfile

Resolves https://github.com/MeetKai/functionary/issues/276

Any progress here? Would be nice to have a docker deployment option for vllm @jeffreymeetkai
I mean if the dockerfiles need some user defined attributes, the user can change it themselves when cloning this repository. And otherwise just having a Dockerfile here even if it is per default not super customizable would be very wild.

It's been ready for another review from @jeffreymeetkai, though it's possible it's gone stale since it's been a little while


When is the app going to be available on F-Droid?

See #477

Closed as duplicate.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team


Rewrite SpriteText to use DrawNodes + baked flow logic

This is basically an entire rewrite, sorry for the large PR but there's no other way.
There were two goals for this change:

Use a single DrawNode to draw all the text - remove all intermediate Drawables.
Bake in the FillFlowContainer logic - there's no need for logic such as children anchors, bounding boxes, etc.

In the process, the following has been achieved:

SpriteText is now a Drawable (previously a container you could add to).
AutoSizeAxes does not exist anymore.

Width/Height still work, but the size provided remains for the entire duration of the SpriteText, and it can no longer be autosized beyond that point.
RelativeSizeAxes still work.

Size is now accurate when DrawWidth/DrawHeight/DrawSize is queried, so it should avoid any update ordering issues.
FPS boost from ~300 -> ~1500 with fullscreen chat open in main menu, significant gains everywhere else.

A slight drawback is you can no longer specify CreateFallbackDrawable which on osu! was used to apply a rotation effect, but CreateFallbackTexture is provided.

Comparison vs the old spritetext (visually) can be found in https://gist.github.com/smoogipoo/40ce17b5097b62dbe78fe336ca5992d3


send recsys powered-by

Had to change urisByQuery to resultsByQuery so that I could store the recsys id with the query since it changes by load balancer.


BGP ASN not getting set/picked

After upgrading from Controller 6.2 (and TF 0.12) to Controller 6.3 (and TF 0.13), the same code breaks with this message:
module.transit-peering.aviatrix_transit_gateway_peering.peering["ace-gcp-us-east1-transit1:ace-azure-east-us-transit1"]: Creating...
module.transit-peering.aviatrix_transit_gateway_peering.peering["ace-gcp-us-east1-transit1:ace-aws-eu-west-1-transit1"]: Creating...
module.transit-peering.aviatrix_transit_gateway_peering.peering["ace-azure-east-us-transit1:ace-aws-eu-west-1-transit1"]: Creating...

Error: failed to create Aviatrix Transit Gateway peering: rest API create_inter_transit_gateway_peering Post failed: [AVXERR-TRANSIT-0168] Transit gateway ace-gcp-us-east1-transit1 and ace-aws-eu-west-1-transit1 has the same ASN number None. This is not allowed in Activemesh 2.0 in transit peering.

Error: failed to create Aviatrix Transit Gateway peering: rest API create_inter_transit_gateway_peering Post failed: [AVXERR-TRANSIT-0168] Transit gateway ace-gcp-us-east1-transit1 and ace-azure-east-us-transit1 has the same ASN number None. This is not allowed in Activemesh 2.0 in transit peering.

Error: failed to create Aviatrix Transit Gateway peering: rest API create_inter_transit_gateway_peering Post failed: [AVXERR-TRANSIT-0168] Transit gateway ace-azure-east-us-transit1 and ace-aws-eu-west-1-transit1 has the same ASN number None. This is not allowed in Activemesh 2.0 in transit peering.

I don't trust these messages. This worked fine before the upgrade. The code is here. It just uses modules aws-transit-firenet, gcp-transit, and azure-transit to build the transits, and then peers them with this module mc-transit-peering. Either the transit modules aren't setting BGP ASN or the mc-transit-peering module isn't reading it properly.

The issue is on the transit GW side of things, not on the peering resource. As such this needs to be addressed by the transit modules. As this is new behavior of 6.3, it will be integrated in newer transit module versions.


noissue: allow admins to call /queues endpoint

how about stg?

Updating the main file updates for both staging and prod


Only fractional scale values are supported

The title says it all. Only decimal fractions work as scaling values. E.g. scale 2 doesn't work.

Fixed in https://github.com/emersion/kanshi/pull/29


[LP#1949913] update delimieter to use a colon rather than a hyphen

change the unit-name delimiter from - to :

Closing because changing this from a hyphen to a colon results in an invalid URL path in vault charm


Title GF freezing, No Music Plays

Describe your problem here.
Tried adding custom title music (replacing freaky or whatever the song Is called) nothing plays and GF becomes frozen
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Yes, Replaced the music file

i think you removed the title music because i did that and the title screen froze

put a youtube video in the background while you run the game, believe me it happens to me too and that solves it xd

i think you removed the title music because i did that and the title screen froze. otherwise idk

well all I did was try replacing the title music..idk why it would freeze


Removed leading tabs and trailing spaces from ui/scripts/*.js

@karuturi @DaanHoogland  @remibergsma As discussed on Slack August 3rd.

LGTM

We've used the following scripts to remove the trailing white spaces and leading tabs:
Tabs to 4 white spaces:
find . -iname '*.js' -type f -exec sed -i '' 's/\t/    /' {} +
Remove trailing white spaces:
find . -iname '*.js' -type f -exec sed -i '' 's/[[:space:]]{1,}$//' {} +

I've removed the above mentions of commented code. Should be all set.

LGTM, i will have a go at merging when I have Miguels new version of the merge script and the origin/apache triangle ready on my new (post schuberg philis) workstation


Bump windows-rs dependency

Updates windows-rs to version 0.40.0 on the accesskit_windows and accesskit_winit crates.
Also updates winit to version 0.27.3 on the accesskit_windows crate (I forgot it previously...).

 ;)

@DataTriny I just noticed one thing: In platforms/windows/src/tests/mod.rs, you didn't update the code for registering the window class as you did for the example. The tests still pass, but we should probably be consistent here and get rid of the old way of setting the window class name.

@mwcampbell you're right. Fixed.