[stable/prometheus-pushgateway] Add persistence support

Signed-off-by: Patrick Kolze<EMAIL_ADDRESS>What this PR does / why we need it:
It allow the usage of persistent storage of the prometheus-pushgateway
Special notes for your reviewer:
Hey @gianrubio I hope this PR works for you - I tested it on GCP Kubernetes 1.13.6-gke.13 - on the bottom is the values file I used.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

My test values.yaml
# Default values for prometheus-pushgateway.
# This is a YAML-formatted file.
# Declare variables to be passed into your templates.
image:
  repository: prom/pushgateway
  tag: v0.9.0
  pullPolicy: IfNotPresent

service:
  type: ClusterIP
  port: 9091
  targetPort: 9091

# Optional pod annotations
podAnnotations: {}

# Optional pod labels
podLabels: {}

# Optional service labels
serviceLabels: {}

# Optional serviceAccount labels
serviceAccountLabels: {}

## Additional pushgateway container arguments
##
## for example: persistence.file: /data/pushgateway.data
extraArgs:
  persistence.file: /data/pushgateway.data
  persistence.interval: 1m
# Optional additional environment variables
extraVars: []

resources: {}
  # We usually recommend not to specify default resources and to leave this as a conscious
  # choice for the user. This also increases chances charts run on environments with little
  # resources, such as Minikube. If you do want to specify resources, uncomment the following
  # lines, adjust them as necessary, and remove the curly braces after 'resources:'.
  # limits:
  #   cpu: 200m
  #    memory: 50Mi
  # requests:
  #   cpu: 100m
  #   memory: 30Mi

serviceAccount:
  # Specifies whether a ServiceAccount should be created
  create: true
  # The name of the ServiceAccount to use.
  # If not set and create is true, a name is generated using the fullname template
  name:

## Configure ingress resource that allow you to access the
## pushgateway installation. Set up the URL
## ref: http://kubernetes.io/docs/user-guide/ingress/
##
ingress:
  ## Enable Ingress.
  ##
  enabled: false

    ## Annotations.
    ##
    # annotations:
    #   kubernetes.io/ingress.class: nginx
    #   kubernetes.io/tls-acme: 'true'

    ## Hostnames.
    ## Must be provided if Ingress is enabled.
    ##
    # hosts:
    #   - pushgateway.domain.com

    ## TLS configuration.
    ## Secrets must be manually created in the namespace.
    ##
    # tls:
    #   - secretName: pushgateway-tls
    #     hosts:
    #       - pushgateway.domain.com

tolerations: {}
  # - effect: NoSchedule
  #   operator: Exists

## Node labels for pushgateway pod assignment
## Ref: https://kubernetes.io/docs/user-guide/node-selection/
##
nodeSelector: {}

replicaCount: 1

## Security context to be added to push-gateway pods
##
securityContext:
  fsGroup: 65534
  runAsUser: 65534
  runAsNonRoot: true

## Affinity for pod assignment
## Ref: https://kubernetes.io/docs/concepts/configuration/assign-pod-node/#affinity-and-anti-affinity
affinity: {}

# Enable this if you're using https://github.com/coreos/prometheus-operator
serviceMonitor:
  enabled: true
  namespace: prometheus-pushgateway
  # fallback to the prometheus default unless specified
  interval: 10s
  ## Defaults to what's used if you follow CoreOS [Prometheus Install Instructions](https://github.com/helm/charts/tree/master/stable/prometheus-operator#tldr)
  ## [Prometheus Selector Label](https://github.com/helm/charts/tree/master/stable/prometheus-operator#prometheus-operator-1)
  ## [Kube Prometheus Selector Label](https://github.com/helm/charts/tree/master/stable/prometheus-operator#exporters)
  selector:
    prometheus: kube-prometheus
  # Retain the job and instance labels of the metrics pushed to the Pushgateway
  # [Scraping Pushgateway](https://github.com/prometheus/pushgateway#configure-the-pushgateway-as-a-target-to-scrape)
  honorLabels: true

# The values to set in the PodDisruptionBudget spec (minAvailable/maxUnavailable)
# If not set then a PodDisruptionBudget will not be created
podDisruptionBudget:

persistentVolume:
    ## If true, pushgateway will create/use a Persistent Volume Claim
    ## If false, use emptyDir
    ##
    enabled: true

    ## pushgateway data Persistent Volume access modes
    ## Must match those of existing PV or dynamic provisioner
    ## Ref: http://kubernetes.io/docs/user-guide/persistent-volumes/
    ##
    accessModes:
      - ReadWriteOnce

    ## pushgateway data Persistent Volume Claim annotations
    ##
    annotations: {}

    ## pushgateway data Persistent Volume existing claim name
    ## Requires pushgateway.persistentVolume.enabled: true
    ## If defined, PVC must be created manually before volume will be bound
    existingClaim: ""

    ## pushgateway data Persistent Volume mount root path
    ##
    mountPath: /data

    ## pushgateway data Persistent Volume size
    ##
    size: 2Gi

    ## pushgateway data Persistent Volume Storage Class
    ## If defined, storageClassName: <storageClass>
    ## If set to "-", storageClassName: "", which disables dynamic provisioning
    ## If undefined (the default) or set to null, no storageClassName spec is
    ##   set, choosing the default provisioner.  (gp2 on AWS, standard on
    ##   GKE, AWS & OpenStack)
    ##
    # storageClass: "-"

    ## Subdirectory of pushgateway data Persistent Volume to mount
    ## Useful if the volume's root directory is not empty
    ##
    subPath: ""

/assign @gianrubio

I hope the PR can be merged please remove stale flag

Hey @gianrubio / @unguiculus,
could you please have a look into this PR. Please let me know if I should adapt something.

/ok-to-test

When this PR will be merged make sense to redo the same also here: https://github.com/helm/charts/tree/master/stable/prometheus

@zanhsieh @gianrubio @unguiculus could you please give your "ok-to-test" approval on this PR?

/retest

@pierluigilenoci
It will take a long time to get this lgtm. According to the community member reported, @gianrubio , this chart owner, disappeared at least 17 days. You might find some helm member to overwrite it, otherwise only the owner could merge. Sorry I am not the owner for this chart.

@gianrubio are you the owner?

@pierluigilenoci
Unfortunately some users report @gianrubio disappears for more than 17 days. You should ask some Helm member to overwrite the approval, and might self-promote the PR contributor to Helm member as this chart OWNERS after that.

Done https://kubernetes.slack.com/archives/C0NH30761/p1571131197156800

/ok-to-test

Also pls rename you PR according to the PR template. https://github.com/helm/charts/blob/master/.github/PULL_REQUEST_TEMPLATE.md

/assign

Still valid. Please remove stale flag.

@paprickar

Due to no action from @paprickar staling this is the right way to go in my eyes.

@cstaud I will push @paprickar phisically to fix this 😝

However this is done: Pls rename the PR as advised above and resolve the current conflicts. Maybe it is easier to do this if you do a fresh checkout of the master and ship your changes within a new PR, but thats up to you :-)

I was a bit more busy then expected lets see if I got it right now : >

/retest

@cstaud the test was successfull. 🚀

@cstaud could you have a look into it again? (it seems there was an update for the chart again so I updated it again - would be great to get this PR closed - if everything is fine)

@cstaud @gianrubio could you please take a look?

@pierluigilenoci @paprickar sry for the late reply there was some christmas/ new year stuff, you know ;-)

/lgtm

@paprickar could you please bump again the chart version?

/lgtm

Thank you @cstaud and @davidkarlsen for your help!


build separated commands into a single command line builder

this combines separated modes (tool, luncher) commands into a single consolidated command line builder,
allowing automatic dynamic runtime mode determination and code command reusability

@bagdxk kindly review and give feedback if any, thanks

closed as currently merged pr has all changes


Tasklist API returns user task with null taskState property

I have the method GetTask which calls Tasklist API to get state of user task by id:
public async Task<UserTask> GetTask(string id)
{
    var request = new GraphQLRequest
    {
        Query = @"
query GetTask($taskId: String!) {
  task(id: $taskId) {
    id
    name
    taskState
    taskDefinitionId
    processName
    creationTime
    completionTime
    assignee
    processDefinitionId
  }
}",
        Variables = new
        {
            taskId = id
        }
    };

    var response = await _graphQlClient.SendQueryAsync<UserTaskResponse>(request);

    if (response.Errors != null)
    {
        var error = string.Join(Environment.NewLine, response.Errors.Select(obj => obj.Message));
        throw new Exception(error);
    }

    return response.Data.Task;
}

I call this method in JobHandler to fill list of user tasks:
private async Task UserTaskJobHandler(IJobClient client, IJob activatedJob)
{
    if (activatedJob.ProcessInstanceKey != _processInstanceKey)
    {
        return;
    }

    TraceJob(activatedJob);

    var userTask = await _zeebeTasklistApi.GetTask(activatedJob.Key.ToString());

    _workflowWatcher.StartUserTask(activatedJob.ProcessInstanceKey, activatedJob.Key, activatedJob.Variables, userTask);
}

Problem is that this call gives GraphQL error:
{
    "errors": [
        {
            "message": "The field at path '/task/taskState' was declared as a non null type, but the code involved in retrieving data has wrongly returned a null value.  The graphql specification requires that the parent field be set to null, or if that is non nullable that it bubble up null to its parent and so on. The non-nullable type is 'TaskState' within parent type 'Task'",
            "path": [
                "task",
                "taskState"
            ],
            "extensions": {
                "classification": "NullValueInNonNullableField"
            }
        }
    ],
    "data": null
}

Also Tasklist GUI is not showing this task in browser.

Hey @Ar4ics sorry but this has nothing to do with the C# client. I will close this, please post such questions or concerns in the forum https://forum.camunda.io/ or via slack https://zeebe-slack-invite.herokuapp.com/


Display aggregated task/service logs from service/stack page, like docker-compose logs does

Is your feature request related to a problem? Please describe.
I have several stacks with several services each and I often want to see at a glance what the whole stack is doing.
For example, while deploying a new version of our application code the first task for a particular image that happens to start will do some database migration work and other tasks will be blocked until that work is done.
I won't know which task is doing this work, but I'll want to check on its progress while it is happening in case there's an issue and I need to act to fix the deployment.
Currently, I have to click from a stack into ~4-5 different services (that use the same image) and then from there either view service logs in 4-5 different browsers, and some of those might have multiple tasks which makes the log output a bit unclear as each line is not prefixed with the task name/slot.
Describe the solution you'd like
Exactly what docker-compose logs -f does. Aggregate all the logs from running containers in the stack, prefixing each log entry with {{ .Service.Name }}-{{ .Task.Slot }} and colour coding the prefix.
With one click from a stack, I could see which task is currently executing the startup migration, and what the other stacks have done or are doing.
A bonus would be a multi-select widget where I can choose easily check and uncheck individual services or tasks to narrow focus.
Describe alternatives you've considered
I could rely on external log aggregation (e.g. DataDog or LogEntries), but then I have to go to another site, open up the right logs and then filter them. That's probably harder than just opening the 4-5 service logs from Portainer now.

This is something I wanted to see as well. I might have commented this inside an existing issue but we'll track this evolution here.

Maybe you are thinking of #335?

This will be addressed in #3749


Polygon contains method throwing error

(new Polygon([
  new Point(0,0),
  new Point(100,0),
  new Point(100,100),
  new Point(0,100),
])).contains(new Point(50,50));

Throws "Uncaught TypeError: e.p1.equals is not a function" in Polygon.contains

You don't need an array that wraps around the points of a Polygon, e.g.
(new Polygon(
  new Point(0,0),
  new Point(100,0),
  new Point(100,100),
  new Point(0,100),
)).contains(new Point(50,50));

Thank you. Webpack allows me to write new Polygon(...points) if I have an array. It works now.


[FEATURE REQUEST] Syntax highlighting

Hey 👋🏾
I was wondering if you would consider adding syntax highlighting for code blocks (perhaps on an opt-in basis?) to this library, using something like https://github.com/wingio/syntakts perhaps.
Thanks for your time.

Thank you for the suggestion. I think you should be able to already be able to do that, by adding a custom handling for the Code block.
Might be interesting to add an example for that, but I don't think that I plan to have it as a core part of the project at this time 🤔

https://github.com/mikepenz/multiplatform-markdown-renderer?tab=readme-ov-file#syntax-highlighting


Status bar popup hides when trying to hover it.

Describe the bug
After opening a status bar menu, when trying to move the mouse over it, it closes itself.
When the mouse hovers the tiny gap, the focus must get lost bcause the popup closes.

I would expect to be able to move the mouse over the popup.
Workaround :
Move the mouse quickly enough so no mouse move event will be triggered on the tiny gap.
To Reproduce
No response
Expected Behavior
No response
Screenshots
No response
Desktop (please complete the applicable information)
No response
Additional context
No response

Closed, Marked as working has intended.
https://github.com/iTwin/iTwinUI/issues/2284#issuecomment-2400309195


Support for H7037 - Outdoor String Lights v2

Govee Device SKU
H7037
Govee2MQTT Version
2024.07.13-82ddc6e9
Describe the issue
Wondering if support could be added for the H7037 Outdoor String Lights 2.

It gets detected by the addon, but no controls are available.
Startup Diagnostics
[2024-08-26T13:33:34 INFO  govee::commands::serve] H7037_6F80 (0B:86:C9:F0:85:C6:6F:80 H7037)
[2024-08-26T13:33:34 INFO  govee::commands::serve]   Undoc: room=None supports_iot=true ble_only=false
[2024-08-26T13:33:34 WARN  govee::commands::serve]   Unknown device type. Cannot map to Home Assistant.
[2024-08-26T13:33:34 INFO  govee::commands::serve]

Additional Logs
No response
Home Assistant Logs
No response
Anything else?
No response

I have the H7037 outdoor string lights and they work with LAN API just fine using the HACS version. They appear as a single light though, so you can turn them on and off and control the colour, but not scenes etc.
[2024-08-26T03:23:31 TRACE govee::commands::serve] UndocDeviceInfo {
...
        support_scene: false,
[2024-08-26T02:46:06 TRACE govee::service::state] Platform API unavailable: Don't know how to list scenes for H7037_1985 (39:DB:C9:F0:85:C6:19:85 H7037)
I tried editing qwirks to add them as a string light, but I still get the same response 'don't know how to list scenes'.  I can poke more into the code I just haven't seen this codebase before so not sure if there's some guide on how to add support for new LAN enabled devices.

[2024-08-26T13:33:34 INFO  govee::commands::serve] H7037_6F80 (0B:86:C9:F0:85:C6:6F:80 H7037)
[2024-08-26T13:33:34 INFO  govee::commands::serve]   Undoc: room=None supports_iot=true ble_only=false
[2024-08-26T13:33:34 WARN  govee::commands::serve]   Unknown device type. Cannot map to Home Assistant.
[2024-08-26T13:33:34 INFO  govee::commands::serve]

You don't have a LAN_API line... do you have local access enabled in the app for the lights?  Here's my log
[2024-08-26T03:28:52 ^[[32mINFO ^[[0m govee::commands::serve]
[2024-08-26T03:28:52 ^[[32mINFO ^[[0m govee::commands::serve] H7037_1985 (39:DB:C9:F0:85:C6:19:85 H7037)
[2024-08-26T03:28:52 ^[[32mINFO ^[[0m govee::commands::serve]   LAN API: ip=<IP_ADDRESS>
[2024-08-26T03:28:52 ^[[32mINFO ^[[0m govee::commands::serve]   Undoc: room=None supports_iot=true ble_only=false
[2024-08-26T03:28:52 ^[[36mTRACE^[[0m govee::commands::serve] UndocDeviceInfo {

I have the H7037 outdoor string lights and they work with LAN API just fine using the HACS version 2024.07.13-82ddc6e9. They appear as a single light though, so you can turn them on and off and control the colour, but not scenes etc.

Using the exact same version and they don't appear as a single light with on/off for me.

You don't have a LAN_API line... do you have local access enabled in the app for the lights? Here's my log

You're correct, I do not have a LAN_API line. And yes the LAN Control toggle is ON in the govee app.
[2024-08-26T21:00:18 INFO  govee::commands::serve] H7037_6F80 (0B:86:C9:F0:85:C6:6F:80 H7037)
[2024-08-26T21:00:18 INFO  govee::commands::serve]   Undoc: room=None supports_iot=true ble_only=false
[2024-08-26T21:00:18 WARN  govee::commands::serve]   Unknown device type. Cannot map to Home Assistant.

A bit more investigation:
--lan-control works for on/off and colour, but not scene.
curl "https://app2.govee.com/appsku/v1/light-effect-libraries?sku=H7037" -H 'AppVersion: 5.6.01' -s
returns a list of plausible looking scene IDs for the lights.   Twilight for example is 11008 (0x2b00) but sending that scene directly doesn't make any change to the lights.
$target/debug/govee ... lan-control --ip <IP_ADDRESS> scene Twilight
...
[2024-08-26T23:54:40 TRACE govee::cache] cache hit for scenes-H7037
Computed MwUEACsAAAAAAAAAAAAAAAAAABk=
[2024-08-26T23:54:40 TRACE govee::lan_api] LanDevice::send_request to <IP_ADDRESS> PtReal { command: ["MwUEACsAAAAAAAAAAAAAAAAAABk="] }
And that base64 string looks okay too:
33 05 04 00 2b 00 00 00 00 00 00 00 00 00 00 00 00 00 19 00
which matches what I'd expect from https://github.com/bwp91/homebridge-govee/issues/694
So maybe the govee api is sending the wrong scene id codes for this device?

lan-control works for on/off and colour, but not scene.

I did get the on/off control by manually specifying the IP. It doesn't pick up just on discovery. Can confirm no scene control.

Well I figured out how to change scene by sniffing bt, but it's odd, and I've not found any write up of the 0xa3 command to dig further.
So sending an extended packet thing before selecting the scene works and you can change scenes.... this minimal example changes to twlight.
debug/govee lan-control --ip=<IP_ADDRESS> command 0xa3 0x00 0x01 0x03 0x02 0x01 0x20 0x00 0x00 0x00 0x01 0x00 0x01 0xff 0x4c 0x03 0xca 0x08 0x05
debug/govee lan-control --ip=<IP_ADDRESS> command 0xa3 0x01 0x00 0xcb 0x05 0x03 0xff 0xbf 0x07 0xff 0x7f 0x00 0xff 0x60 0x07 0x00 0x00 0xff
debug/govee lan-control --ip=<IP_ADDRESS> command 0xa3 0xff 0x00 0xed 0x01
debug/govee lan-control --ip=<IP_ADDRESS> command 0x33 0x05 0x04 0x00 0x2b
but without those 0xa3's it doesn't change mode.  And it seems to then ignore the scene code as changing 0x00 0x2b (twilight) to some other valid scene and you still get twilight.

lan-control works for on/off and colour, but not scene.

I did get the on/off control by manually specifying the IP. It doesn't pick up just on discovery. Can confirm no scene control.

How were you able to manually specify the IP?  I'm having a similar issue with the H7053 (Outdoor Ground Lights 2) where they pull in but there is no on/off control at all.  Looks similar to your screen shot in the OP.

How were you able to manually specify the IP?

In the config you can use scan: 192.X.X.X to specify the IP addresses manually. Once i did that, I gained on/off control.

Thanks!  My apologies for my ignorance on this, but where in the configuration can I specify this particular IP?  I added to one spot thinking it was correct, but the then the Add-on wouldn't start.  I removed it and it started right back up.

Got it!  Thank you!  That's where I was putting it, but it wasn't working.
I finally realized it wasn't working because I also had the "Govee Lights Local" integration running as well.  As soon as I disabled that, it worked immediately.  Thanks again!


Fatal Error...

Not sure what else to say... I just installed it and opened it up

I also have this issue, have you found a solution yet?

is there any way to macke it work?

I have this issue to.  Solution?

Duplicate of #6

Duplicate of #6
Unfortunately, I have reinstalled the program but it has not resolved anything

Try to find wherever a "settings.file" file is, and delete it. That's the only way to fix this issue. If you uninstalled it, that file might have been left over, so make sure to search your computer for the file.

Are you guys updating from before 3.3.6? I don't understand how there are still so many people with this issue...


Make the Sluice Key work properly on Shrine door

Currently, shrine door leading to megaliths memory game is locked to the condition of opening the chest containing Sluice Key in the vanilla game. This needs to be changed to actually check if Sluice Key is owned.

Fixed in b7ef17a


Backport:Monitoring app get redeployed when have system-default-registry configured

https://github.com/rancher/rancher/issues/20186

Close in favor of https://github.com/rancher/rancher/issues/20202


Users are getting 401's when using acquireTokenSilent

Please follow the issue template below. Failure to do so will result in a delay in answering your question.

Library

[X]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Framework
React 16.8.6
Description
After upgrading from msal 1.2.0 to msal 1.2.1, users keep getting 401's if their token is expired, acquireTokenSilent isn't refreshing the token as it's supposed to. Also, noticed a bunch of in progress states for the login in local storage, which might be the cause of the issue (even though the user is already logged in). Below is the code we are using:
private _getToken(): SyncTasks.Promise<void> {
        if (this._msal.getLoginInProgress()) {
            return SyncTasks.Defer<void>().promise();
        }
        const request: Msal.AuthenticationParameters = this._formMsalAuthParams();
        return SyncTasks.fromThenable(this._msal.acquireTokenSilent(request).then(response => {
            if (this._user && this._user.accessToken !== response.accessToken) {
                // update access token info for the user.
                this._tokenReceivedCallback(response);
                this._resetBackoff();
            }
        }, error => {
            Instrumentation.log(LogTraceArea.Network, 'MSAL AcquireTokenSilent Error: ' + JSON.stringify(error));
            // Track in telemetry so we can see if problems are brewing
            if (!this._msal.getLoginInProgress()) {
                request.forceRefresh = true;
                this._msal.acquireTokenRedirect(request);
            }
        }));
    }

Security
Is this issue security related?
Regression
Did this behavior work before?
Configuration
Please provide your MSAL configuration options.
const msalConfig: Msal.Configuration = {
            auth: {
                clientId: AAD_APP_ID,
                authority: 'https://login.microsoftonline.com/common',
                validateAuthority: false,
                redirectUri: window.location.origin + RouteUtils.routeForSignIn(),

                navigateToLoginRequestUrl: true,
            },
            cache: {
                cacheLocation: 'localStorage',
            },
        };

// Provide configuration values here.
// For Azure B2C issues, please include your policies.

Reproduction steps

Expected behavior
A clear and concise description of what you expected to happen.
Browsers
Safari, Edge, Firefox, Chrome.

I think this might be related to the error I am also face: Token is not in cache for scope:openid profile user.read then it seems to not go past this step: Renew token for scope: openid profile user.read is in progress. Registering callback

Having same issue with multiple inProgress values in localStorage.

@ahqardah Is this still an issue with<EMAIL_ADDRESS>

This may have been fixed with<EMAIL_ADDRESS>Please let us know if this is still an issue


Remove docker compose files + add GH action to compile examples during CI

@swift-server-bot test this

@fabianfett Here is a first version of a Matrix that launch integration tests in parallel.
Each example is built on swift-6.0-nigthly and swift-main-nightly
I'm not super happy with the place where the list of example is defined (currently in .github/workflows/examples_matrix.yml) because I don't think we can pass an array as parameter between workflow.
(Ideally, the list of examples should be defined in .github/workflows/pull_request.yml)

All green !  Except Swift 6 language mode

Examples are built on only one version of Swift Linux (the latest one)
I kept a separate file to define the matrix job


[BUG] Trace is not using OTEL_EXPORTER_OTLP_TRACES_ENDPOINT env variable even though it set

Affected project(s)

[ ] documentation
[ ] examples
[X] wasmCloud host
[ ] wasmCloud CLI (wash)
[ ] wasmCloud dashboard UI (washboard)
[ ] capability providers
[ ] provider bindgen
[ ] control interface client
[ ] other / not sure

Describe the bug
I am setting the environment variable OTEL_EXPORTER_OTLP_TRACES_ENDPOINT to  send traces to my custom endpoint but it seems wasmcloud is still trying to send to default http://localhost:4318/v1/traces) .
Followed this doc : https://wasmcloud.com/docs/deployment/observability/tracing
Steps to reproduce
I have used the following environment variables to setup traces using otel and export it to my end point (instead of default http://localhost:4318/v1/traces) .
export WASMCLOUD_TRACES_ENABLED="true"
export OTEL_EXPORTER_OTLP_TRACES_ENDPOINT=https://myendpoint.com/ot/v1/traces
wash up

I have a wasm component with wadm file as below
spec:
  components:
    - name: http-component
      type: component
      properties:
        image: file://./build/http_client_tinygo_s.wasm
      traits:
        # Govern the spread/scheduling of the component
        - type: spreadscaler
          properties:
            replicas: 1
        - type: link
          properties:
            target: httpclient
            namespace: wasi
            package: http
            interfaces: [outgoing-handler]

    # Add a capability provider that enables HTTP access
    - name: httpserver
      type: capability
      properties:
        image: myhost.com/ghcr-io-proxy/ghcr.io/wasmcloud/http-server:0.21.0
      traits:
        # Link the httpserver to the component, and configure the HTTP server
        # to listen on port 8080 for incoming requests
        - type: link
          properties:
            target: http-component
            namespace: wasi
            package: http
            interfaces: [incoming-handler]
            source_config:
              - name: default-http
                properties:
                  address: <IP_ADDRESS>:8080

    - name: httpclient
      type: capability
      properties:
        image: myhost.com/ghcr-io-proxy/ghcr.io/wasmcloud/http-client:0.11.0

I did wash build and wash app deploy wadm.yaml
The app is started and I am able hit the curl http://localhost:8080 and see the result.
I expect traces to be sent to my endpoint, but in the terminal I see below error message where it is still trying to send to localhost
OpenTelemetry trace error occurred. error sending request for url (http://localhost:4318/v1/traces): error trying to connect: tcp connect error: Connection refused (os error 61)

Other Options tried:
I saw this code and thought I can send wash up --override-traces-endpoint=<myendpoint>, but seems wash up cli command doesn't support that.
https://github.com/wasmCloud/wasmCloud/blob/d92af18f183ba36e3794d4983905a7437ff2efce/src/main.rs#L259-L261
Expected behavior
I expect traces to be sent to my endpoint, but in the terminal I see below error message where it is still trying to send to localhost
OpenTelemetry trace error occurred. error sending request for url (http://localhost:4318/v1/traces): error trying to connect: tcp connect error: Connection refused (os error 61)

Environment

OS: MacOS
Shell: zsh
wash version: 0.29.2

Screenshots / Logs / Additional context
No response

Hey @saikiran1043 I just wanted to check and see if you were able to reproduce this on wash 0.31.0, perhaps we fixed this in the release of 0.30.0 (as far as I know, no new fixes went into 0.31.0`
➜ export WASMCLOUD_TRACES_ENABLED="true"
export OTEL_EXPORTER_OTLP_TRACES_ENDPOINT=https://myendpoint.com/ot/v1/traces
wash up
>>> ⢃⠀  Starting wadm ...
   Found wadm version on the disk: wadm-cli 0.14.0
✅ Using wadm version [v0.14.0]
🏃 Running in interactive mode.
🎛  To start the dashboard, run `wash ui`
🚪 Press `CTRL+c` at any time to exit
2024-08-26T18:24:06.623798Z  INFO async_nats_wrpc: event: connected
2024-08-26T18:24:06.623884Z  INFO async_nats_wrpc: event: connected
2024-08-26T18:24:06.670263Z  INFO wasmcloud_host::wasmbus: created bucket with 1 replica bucket=LATTICEDATA_default
2024-08-26T18:24:06.671596Z  INFO wasmcloud_host::wasmbus: created bucket with 1 replica bucket=CONFIGDATA_default
2024-08-26T18:24:06.676140Z  INFO wasmcloud_host::wasmbus: wasmCloud host started host_id="NC2EZ7FC2V6CFJLCXGE42XMOMDIZMEHW3BXOAEYSNYEGJRW65JD7KGOL"
OpenTelemetry trace error occurred. HTTP status client error (404 Not Found) for url (https://myendpoint.com/ot/v1/traces)

sure @brooksmtownsend , I will update my wash version and try this.

Hi @brooksmtownsend , I have updated wash version and I see it is initially trying to send to correct endpoint, but seems it has some kind of a fallback mechanism to send to localhost if that fails
OpenTelemetry trace error occurred. HTTP status client error (400 Bad Request) for url (https://myendpoint.com/ot/v1/traces)
OpenTelemetry trace error occurred. error sending request for url (http://localhost:4318/v1/traces): error trying to connect: tcp connect error: Connection refused (os error 61)
OpenTelemetry trace error occurred. error sending request for url (http://localhost:4318/v1/traces): error trying to connect: tcp connect error: Connection refused (os error 61)

Hey @saikiran1043 thanks for adding more information about this -- I think those trace errors you're seeing might actually be from providers -- here's what I think might be happening:

The host and two providers start up (httpserver and httpclient in this case)
The host reads the relevant OTEL_* env variables and coordinates appropriately
After some time, the host does error as it can't access the endpoint (you can replicate this with only wash up)
The providers do not inherit environment from the host (this would be a security issue, we changed this a while back)
Both hosts start their own reporting machinery (they are separate child processes), and are individually picking default values.

i.e. if you stop at wash up and never deploy more providers, you'll never see those 2 additional OTEL failures.
The idea that providers need to be able to inherit more env from the host is definitely not new (it's become an issue from time to time, ex. when we were adding flamegraph support), but it looks like this is due to that discrepancy.
Would you mind trying just wash up and seeing if you get that second set of errors w/ the default values?

Thank you @vados-cosmonic  for the detailed info.

I just ran wash up and initially I saw all three error statements (one with value configured in OTEL env and two default values).
I went back to my app, ran wash app delete <mycomponent>
Killed the wash up
Ran the wash up again, this time I see only error statement with OTEL env variable
Ran the wash app deploy <mycomponent>, then I saw the two error statements with default values.

So, just like you said, those two default values seems to be coming from the providers.
Now can you help with these questions?

How can I pass the env (or) OTEL env values to the provider?
How do I see env values in provider individually?
Do we have any option in the wadm.yaml to pass the env variables when specifying the capabilities?

Thanks for checking! It seems we might have figured out what was wrong!

How can I pass the env (or) OTEL env values to the provider?

So unfortunately, right now you can't -- we don't yet have a great solution for this, unfortunately.

How do I see env values in provider individually?

Since the provider is a subprocess its a bit hard to see the ENV values provided to the provider from the outside, if you're running on Linux could do:
cat /sys/<pid>/environ

That said, since we have the code I can tell you that the only env var passed through is RUST_LOG.

Do we have any option in the wadm.yaml to pass the env variables when specifying the capabilities?

Yes!
Providers can be configured (here's an example) -- the config that you pass in there (either just named, or defined inline) can be used during provider setup to perform the configuration you'd like.
What we need to do is make that possible/easy to do from the initialize_observability macro that providers should call.
I'll get working on this and get a change in to provider-sdk

Thanks for the info.
If I set a source config with OTEL end point, will it be available as a env variable (or) I need to make use of the source config and set required env in my provider?

Providers can be configured (here's an example) -- the config that you pass in there (either just named, or defined inline) can be used during provider setup to perform the configuration you'd like.

Hey @saikiran1043 we've just released wasmcloud-provider-sdk v0.9.1 which has a fix to this feature, but what you need to do is use provider configuration to set the OTEL variables.
Long story short, the choice to do it this way makes sense for two reasons:

Provider config is language/platform agnostic
Some OTEL clients ignore whatever value you give it, and always use the values available in ENV. This means that if we pass values from the host, it will be later impossible to use distinct OTEL backends for host and launched providers.

So, if you're building a new provider, you can use the new wasmcloud-provider-sdk -- and if you're not, we'll have new versions of all the providers released very soon!

Thank you @vados-cosmonic , if I understand correctly

In wadm.yml, for the capability type, we can now add config and set the required otel variables there.

Once I pass it, do I need to add logic in my provider to use that config and set as env variables, so that otel can honor it?

Nope! You shouldn't have to do anything in your provider :)
The whole suite of official providers are also being released right now, so the latest versions will also take those config. The change was actually made in provider-sdk (which they all use), so as long as you have a newer version (or are using provider-sdk 0.9.1 in your own provider) the provider will pick up those new vars without any other changes!
Of course, make sure you use/call the initialize_observability macro!

ok, once new versions are released, I will pass the config in wadm.yml and test it. Thank you.

Hey @saikiran1043 the versions were released, and the examples in the repo have been updated! You should be able to use the config (see the wadm.yaml files for reference) now, from any latest version of the providers you were using.

Closing this for now! We can re-open if you find an issue with the config :)

@vados-cosmonic , Can you provide an example configuration on how to set tracing endpoint for http client and http server provider?

Hey @ritesh089 we have these in the codebase now, under the wash new-able examples though it's a little hard to find:
https://github.com/wasmCloud/wasmCloud/blob/main/examples/rust/providers/custom-template/wadm.yaml#L14


Profile for vividvilla

I'd like to have my profile on ghuser.io, thanks!

Me too, if its not too soon yet 👍

Same :+1:

here you go :)

https://ghuser.io/vividvilla
https://ghuser.io/wopian
https://ghuser.io/mkg20001

😳

Same :+1:

@maple3142 https://ghuser.io/maple3142 :)


Same as 328 - missing image update to0.10.6

Describe the bug
repeat of https://github.com/davidusb-geek/emhass/issues/328 for Armv7 (RPi x86)
To Reproduce
Attempt to update in HA
Expected behavior
Updates normally
Screenshots

Home Assistant installation type

Home Assistant OS
Home Assistant Supervised
Home Assistant Core

Your hardware

OS: HA OS
Architecture: armv7

EMHASS installation type

Add-on

Additional context
This has happened before, when there was a blocker on (if I recall) a lib on Armv7 that didn't move forward. That cleared up, and periodically I might see a delay when the build happens, but this has been a while now. I saw the previous bug report and posted there but that didn't reopen so here's a new report.

Will look into some more later. This is interesting tho (on armhf/armv7):
#11 27.88 ERROR: Ignored the following versions that require a different python version: 0.2.14 Requires-Python >=3.8, <3.10
#11 27.88 ERROR: Could not find a version that satisfies the requirement emhass==0.10.6 (from versions: 0.1.0, 0.1.1, 0.1.2, 0.1.3, 0.1.4, 0.1.5, 0.2.0, 0.2.1, 0.6.0, 0.6.1, 0.6.2, 0.7.0, 0.7.1, 0.7.2, 0.7.3, 0.7.4, 0.7.5, 0.7.6, 0.7.7, 0.7.8, 0.8.0, 0.8.1, 0.8.2, 0.8.3, 0.8.4, 0.8.5, 0.8.6, 0.9.0, 0.9.1, 0.10.0, 0.10.1, 0.10.3)
#11 27.88 ERROR: No matching distribution found for emhass==0.10.6

Exactly, that I don't understand, the pypi is published and there. The action ran without issue: https://github.com/davidusb-geek/emhass/actions/runs/9923583868

@davidusb-geek maybe rerun the release? I'm interested if it was maybe  they PyPI database lagginh behind on its arm repositories.
I'm thinking of transferring from requirements.txt to poetry.lock. I'm not sure if it will help, but im wondering if a tools such as this is a better method of assuring the correct packages are installed on the correct CPU architectures.

I've just done that. The release launch fails because pypi won't accept you to overwrite the same version. But anyway the wheels are there available on pypi, so that's ok. I've just relaunched the add-on release, let's see how it goes

Just saw the same error on the re-run action... defiantly need to look deeper into the issue.

Worst case scenario, we could input the EMHASS into EMHASS-add-on as a sub modul. then build the EMHASS package like we do on the standalone docker file. (pypassing PyPI)
https://github.blog/open-source/git/working-with-submodules/

I like that idea.
I confirm that it is better IMO to keep separate repos for the core code and the add-on.

I like that idea.
I confirm that it is better IMO to keep separate repos for the core code and the add-on.

Alright. When I have some time I'll test to see how doable it is.

This is deemed solved with the current version, all platform images were compiled correctly.
Closing as completed


metatable support + raw functions

coroutine.wrap requires this feature

support for __mode for now, creating a separate ticket

Hex literals not implemented yet, fixing...


providesTags does not work with callback when defining return type via TypeScript generic

Prior Issues
Can't seem to find any
What is the current behavior?
With RTK Query, if using a providesTags callback on a simple query while dictating return types/arguments with a generic, you receive a complicated TypeScript error:
Type '(result: PostId[] | undefined, error: FetchBaseQueryError | undefined, arg: void) => ("Post" | { type: string; id: string; })[]' is not assignable to type 'ResultDescription<"Post", PostId[], void, FetchBaseQueryError, FetchBaseQueryMeta | undefined> | undefined'.
  Type '(result: PostId[] | undefined, error: FetchBaseQueryError | undefined, arg: void) => ("Post" | { type: string; id: string; })[]' is not assignable to type 'GetResultDescriptionFn<"Post", PostId[], void, FetchBaseQueryError, FetchBaseQueryMeta | undefined>'.
    Type '("Post" | { type: string; id: string; })[]' is not assignable to type 'readonly TagDescription<"Post">[]'.
      Type '"Post" | { type: string; id: string; }' is not assignable to type 'TagDescription<"Post">'.
        Type '{ type: string; id: string; }' is not assignable to type 'TagDescription<"Post">'.
          Type '{ type: string; id: string; }' is not assignable to type 'FullTagDescription<"Post">'.
            Types of property 'type' are incompatible.
              Type 'string' is not assignable to type '"Post"'.ts(2322)
endpointDefinitions.d.ts(223, 5): The expected type comes from property 'providesTags' which is declared here on type 'OmitFromUnion<QueryDefinition<void, BaseQueryFn<string | FetchArgs, unknown, FetchBaseQueryError, {}, FetchBaseQueryMeta>, "Post", PostId[], "api">, "type">'

Steps to Reproduce
Try and define return type and params as generic for builder.query<any, any> and TypeScript complains about your advanced providesTags usage:
type Post = {
  id: string
  title: string
  content: string
}

export const apiSlice = createApi({
  reducerPath: 'api',
  baseQuery: fetchBaseQuery({ baseUrl: '/fakeApi' }),
  tagTypes: ['Post'],
  endpoints: builder => ({
    getPosts: builder.query<Post[], void>({
      query: () => '/posts',
      providesTags: (result = [], error, arg) => [
        'Post',
        ...result.map(({ id }) => ({ type: 'Post', id }))
      ]
    }),
  })
})

The above example is practically identical to the tutorial describing advanced RTK Query patterns (have shaved some endpoints to focus on the issue) and differs only that in my second code example I'm attempting to use a type and a generic to dictate the return type and arguments for the GET request.

[!NOTE]
If I @ts-ignore, things seem to work as expected.

What is the expected behavior?
There should be no TypeScript errors.
Environment Details
Version of Redux is v8.1.3. Not sure if this worked before.

try adding as const to the type, i.e. result.map(({ id }) => ({ type: 'Post' as const, id }))

@EskiMojo14 Thanks. This appears to work, but can you explain why?

Sure - basically inside your map callback Typescript isn't aware that type can only be a specific set of strings, it just infers that it's a string. Adding as const means it becomes the literal type 'Post' which matches the type requirements for tags.

Thanks for the explanation.


added blog section (issue #12 and #62)

Title:
Added Blog Section
Issue No.:
#12
Code Stack:

React.js
CSS

Close #12
Video (mandatory)

https://github.com/user-attachments/assets/2426e829-4ac0-46ec-9787-2908228dd0b3
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have created a helpful and easy-to-understand README.md if it's a new page/tech stack.
[x] I have gone through the contributing.md file before contributing.

Additional context (Mandatory)
Are you contributing under any Open-source programme?

[✔] I'm a GSSOC-EXT contributor
[✔] I'm a HACKTOBERFEST contributor

@jeevan10017 check it, please.

@SCR01 resolve merge conflicts. Kindly pull before you merge
And also don't create more than 2 issues at a time.
also merge #62 in this.

@jeevan10017 Yeah, my bad. I will add the comment and like button in the blog post, then I will merge it.

@jeevan10017 Hey I got issue #62 in this also pulled it before committing and still check has failed but running fine locally on my PC.  and Branch has no conflicts with the base one.

@jeevan10017 Yeah, it's done!

@jeevan10017 Should I close this PR and redo it?

@SCR01 yup

@SCR01 update?

@jeevan10017 yeah it will be done before morning!


[QUESTION] What does Size and Actual size really mean?

Question What does Size and Actual size really mean? I have 7Gi volume that have 20Gi actual size
Environment:

Longhorn version: 1.2.0

You can refer to this document https://longhorn.io/docs/1.2.0/volumes-and-nodes/volume-size/

Close it since the question is answered.


Spider mode does not recognize existing spider.txt

Every time wraith runs in spider mode, it regenerates a new spider.txt. I can replicate this on 2 servers and the wraith docker image as well.
custom.yaml file
spider_file: spider.txt
spider_days: 10

Each time running wraith:
wraith capture configs/custom.yaml
Config validated. No serious issues found.
no paths defined in config, crawling from site root
creating new spider file

bug is on https://github.com/BBC-News/wraith/blob/master/lib/wraith/spider.rb line 66
it's expecting spider_days to be an array.
so, as a workaround, change spider_days: 10 to
spider_days:
  - 10

Spidering has been overhauled in Wraith v4. Please re-open if this is still an issue.


New release

It has been almost a year since the last release of embedded alloc. There have been multiple fixes and interesting features like tlsf added. Could we please cut a new release?
Thanks :)

Done in v0.6.0!


Update index.yml

Docs Build status updates of commit 70e9a23:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/index.yml
:white_check_mark:Succeeded

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

Docs Build status updates of commit 05c96c3:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch saraclay-patch-1 has been deleted or has been force pushed: fatal: Couldn't find remote ref saraclay-patch-1

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


HBASE-28978 StoreFileInfo.isMobRefFile() would return true if the file is mob file not a reference mob file

Details see: HBASE-28978
This issue was discovered by me while solving another PR problem #6290  .
Since the two are not considered the same problem, I opened a new PR for this issues.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 42s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m  0s

master passed

+1 :green_heart:
compile
2m 55s

master passed

+1 :green_heart:
checkstyle
0m 39s

master passed

+1 :green_heart:
spotbugs
1m 33s

master passed

+1 :green_heart:
spotless
0m 45s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 51s

the patch passed

+1 :green_heart:
compile
2m 56s

the patch passed

+1 :green_heart:
javac
2m 56s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 39s

the patch passed

+1 :green_heart:
spotbugs
1m 45s

the patch passed

+1 :green_heart:
hadoopcheck
10m 26s

Patch does not cause any errors with Hadoop 3.3.6 3.4.0.

+1 :green_heart:
spotless
0m 43s

patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 12s

The patch does not generate ASF License warnings.

35m 22s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless

uname
Linux 9e3d8cc041cb 5.4.0-195-generic #215-Ubuntu SMP Fri Aug 2 18:28:05 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 0891d16a910932380e0c34f2810be9d077eec365

Default Java
Eclipse Adoptium-17.0.11+9

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

modules
C: hbase-server U: hbase-server

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

versions
git=2.34.1 maven=3.9.8 spotbugs=4.7.3

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

This message was automatically generated.


Syntax enhancement to PUT JSON data to a MAP property (embedded or link)

Both of the following syntax options work (the first completely replaces the content of the Samples EMBEDDEDMAP property with the supplied JSON data, while the second adds a new item):
update #23:0 set content {"Samples":{"00":{"@type":"d", "@class":"VSample", "Count": 1, "Scale": 0, "Sum16": 12345}}}
update #23:0 put Samples = "00", {"@type":"d", "@class":"VSample", "Count": 1, "Scale": 0, "Sum16": 12345}
The following syntax fails (the intent is to add a new item to the Samples EMBEDDEDMAP property):
update #23:0 put content {"Samples":{"00":{"@type":"d", "@class":"VSample", "Count": 1, "Scale": 0, "Sum16": 12345}}}
Would it not make sense to support the failing syntax? It seems much more consistent with the corresponding set (for complete property replacement).
Also, using the working PUT (second example above), how would you add more than one new item at a time? How to do this using the failing syntax would be obvious. In fact, it is supported as expected by the SET syntax.
Runninng Mode: Embedded, using PLOCAL access mode, executing SQL in Studio
OrientDB Version: 2.2.0 on Linux (CentOS)
Java Version:
openjdk version "1.8.0_91"
OpenJDK Runtime Environment (build 1.8.0_91-b14)
OpenJDK 64-Bit Server VM (build 25.91-b14, mixed mode)

Strange things happening... Yesterday, I was able to execute this statement (in Studio):
update #23:0 set content {"Samples":{"00":{"@type":"d", "@class":"VSample", "Count": 1, "Scale": 0, "Sum16": 12345}}}
But today, it fails (the parser complains about not expecting "update in column 1"). I can still execute the "put" version just fine. Is there something wrong with this syntax?
The only difference I'm aware of between yesterday and today is that I downloaded the GeoSpatial add-on jar file and restarted the server (which appears to have worked as expected). Could that somehow affect this?

@Eric24 Hello.  would the following query work for you:
update #23:0 set content.Samples = {"00":{"@type":"d", "@class":"VSample", "Count": 1, "Scale": 0, "Sum16": 12345}}
Or something to that effect?  This should add a new entry to the map?  Full disclosure, I haven't tried this exact query, but we do something like it quite often.

So yes, this works ('content' not needed here), and it works with one or multiple items:
update #23:0 set Samples = {"01":{"@type":"d", "@class":"VSample", "Count": 1, "Scale": 0, "Sum16": 12345}}
But this does not (I originally thought it did, but I can not make any version of it work again, so I must have been mistaken):
update #23:0 set content {"Samples":{"01":{"@type":"d", "@class":"VSample", "Count": 1, "Scale": 0, "Sum16": 12345}}}
The idea of the latter is that it allows several individual properties can be set at once as part of a JSON object, rather than using itemized property lists.
But more importantly, this does not work:
update #23:0 put Samples = {"01":{"@type":"d", "@class":"VSample", "Count": 1, "Scale": 0, "Sum16": 12345}}
Nor does this:
update #23:0 put content {"Samples":{"01":{"@type":"d", "@class":"VSample", "Count": 1, "Scale": 0, "Sum16": 12345}}}
This is the only syntax I can find that works with put:
update #23:0 put Samples = "00", {"@type":"d", "@class":"VSample", "Count": 1, "Scale": 0, "Sum16": 12345}
And it doesn't allow multiple items to be put at once.


PageClone: Cloning stalls with "Detach instances" enabled + error in log

It does not seem to be possible to clone a page and simultaneously detach all instances. An error is shown in the log. Cloning works without any issues when the aforementioned setting is turned off.
Reproduction Steps:

Have a page with some component instances
Trigger PageClone and make sure "Detach instances" is enabled

Plugin window spins forever, error following is shown in the log:
    at s (PLUGIN_13_SOURCE)
    at <anonymous> (PLUGIN_13_SOURCE)
    at forEach (native)
    at <anonymous> (PLUGIN_13_SOURCE)
    at forEach (native)
    at <anonymous> (PLUGIN_13_SOURCE)

(anonymous) @ figma_app.bce3587c4bfd9cbd0b9b7144e399a3ab.min.js.br:39
callFunction @ figma_app.bce3587c4bfd9cbd0b9b7144e399a3ab.min.js.br:838
iframeMessageHandler @ figma_app.bce3587c4bfd9cbd0b9b7144e399a3ab.min.js.br:1509
windowMessageEventListener @ figma_app.bce3587c4bfd9cbd0b9b7144e399a3ab.min.js.br:1468
p @ figma_app.bce3587c4bfd9cbd0b9b7144e399a3ab.min.js.br:71
postMessage (async)
messageHandler @ plugin-sandbox:16
postMessage (async)
(anonymous) @ VM945:1
setTimeout (async)
document.getElementById.onclick @ VM945:1

Thanks for looking into it 👍

Hello @marioaltvater I'll try to look into it this weekend okay!

Hey Stevanus, awesome. Thank you.

Usually this is because of an unhandled frame or figma file structure. Would you mind sharing with me a sample of where this occurs for me to reproduce? Can share to<EMAIL_ADDRESS>:) @marioaltvater


Add sugestions based on Wikidata

Since some tags like names ... can translate 1:1 into tagging it would seem usefull to suggest those as changes as well

Can you elaborate? I'm not really sure what you're asking for, sorry..

If I may, I had similar need as to manually copying data from Wikidata to OSM. We were mapping this historical downtown which comprises ~250 heritage buildings: https://www.openstreetmap.org/relation/9963374
For instance, say I click on this building (https://www.openstreetmap.org/way/720714962) and add the wikidata Q65936743 (https://www.wikidata.org/wiki/Q65936743), the editor iD could "query" the wikidata page and suggest a few useful properties, such as name, name:, image, inception (i.e. start_date), etc. The mapper then must review and confirm the data.

ok yeah #4382 and #5566 have some thoughts on this.
I'm going to close this in favor of #5566.
It would be nice to support a UI that lets people better sync up the data between OSM tags and Wikidata claims.


cargo build failed

Hi
When I try run cargo build I got this errors

How can I fix it?

Hi @kak-smko
I guess you tried on Windows. The APIs set_reuse_port in tokio works on UNIX-like systems (c.f., https://docs.rs/tokio/1.36.0/tokio/net/struct.TcpSocket.html#method.set_reuseport ).
If you are on Windows, I think it works by deleting L18 and L36. These are required to re-bind ports after reloading the configuration without restarting the process. So it works without these options, and please restart the process when you update the configuration.

thanks.

Hi @kak-smko
I guess you tried on Windows. The APIs set_reuse_port in tokio works on UNIX-like systems (c.f., https://docs.rs/tokio/1.36.0/tokio/net/struct.TcpSocket.html#method.set_reuseport ).
If you are on Windows, I think it works by deleting L18 and L36. These are required to re-bind ports after reloading the configuration without restarting the process. So it works without these options, and please restart the process when you update the configuration.

Thank you


[Bug] Role icons do not display if Discord is set to “Compact” mode

Describe the Bug
When Discord is set to “Compact” mode in “App Settings” → “Appearance” → “Message Display”, role icons are swallowed by BetterDiscord.
To Reproduce
Set “Message Display” to “Compact”.

Expected Behavior
Role icons should still display next to people’s names.
Screenshots

(Compact)
vs.

(Cosy)
Discord Version

For the record, same thing on my Linux machines.
Additional Context
This is by no means a new bug, it has existed since these icons were introduced. I was just wondering why it was still a thing, and decided to test some things. That’s when I realized that it only happens in Compact mode <.<
I also made sure to test w/o any additional plugins installed in BetterDiscord.

Go tell the plugin dev about it. This isn't a repo to report issues for plugins.
https://github.com/mwittrien/BetterDiscordAddons/issues if you need the link for ShowBadgesInChat

Does this happen without BD installed?

@SmolAlli I believe you have mistaken this for something else. The user is referring to Discord's role icons from boosting.

Does this happen without BD installed?

No, vanilla Discord displays the icons just fine in both modes.

Edit: Actually … WTF. I just went and double-checked. Same bug. Even though I know for a fact that it did display the icons during the last period when BD hadn’t been updated for latest Discord.

This one is really throwing me through a loop now, because at that point (BD didn’t work, role icons appeared) I was sure it was a bug in BD.

Yeah I thought as much, BD by default doesn't affect anything in that area so it would very strange for BD to cause that.

Sorry for wasting your time and thanks for the immediate response :+1:


Dotfiles fails: /bin/sh: 2: Bad substitution

The dotfiles modules fails on my side:

I added the module like this:
module "dotfiles" {
  source = "https://registry.coder.com/modules/dotfiles"
  agent_id = coder_agent.workspace.id
}

and specified this repository. Is there anything else I need to set up?

Hi @jaulz, this module uses coder dotfiles to setup/install the dotfiles, which requires the presence of a script. It looks like there is no script in your dotfiles repo.
Please see this for more details: https://coder.com/docs/v2/latest/dotfiles#setup-script-support

@matifali thanks for the feedback! In fact, I checked the documentation but it sounded to me that the script is optional is and if it doesn't exist the configuration files are copied only.

@jaulz can you manually run coder dotfiles <your-dotfiles-repo> from within your workspace and see if it works? This will help us identify if the issue is with the module or the coder dotfiles command.

@matifali it's also not really working:

Maybe because there is already other stuff in the folder (e.g. code-server from other module)?


cmake format

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
NONE
What this PR does / why we need it:
Adds a .cmake-format file.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: leodido
If they are not already assigned, you can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mstemm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mstemm]

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

LGTM label has been added.  Git tree hash: 697df4ee02b4509fbf29f31f48eb1bcf06b52e01

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, mstemm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz,mstemm]

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

LGTM label has been added.  Git tree hash: ec0b1ca338daa0ce0acaba4b10ada503fffa2aaa


Low Throughput Issue with unbound DNS over TLS on Ubuntu 22.04

I'd like to report a rather peculiar issue that I encountered approximately a year ago. It seems that the reviewers during the evaluation of my paper artifact also stumbled upon this problem. Therefore, I'm curious to understand the root cause of this unusual issue.
I noticed that when testing unbound DNS over TLS (DoT) throughput on Ubuntu 20.04, the throughput was approximately 1000 connections/second. However, when testing the DoT throughput on Ubuntu 22.04, the throughput was unusually low, averaging only around 20 connections/second.
I utilized the configuration file available at: config_file. Within the same directory, you can find the relevant certificate files, ed25519.key and ed25519.pem. In the configuration file, I enabled data statistics using "statistics-interval: 60" to monitor throughput-related data.
The client I used is available at: DoT Timer. Its behavior is straightforward: within a timing loop, it establishes a TLS connection, queries a domain name, and then closes the connection.
Both the client and server communicate within an internal network, and my client implementation is very simple. Thus, I suspect the issue does not stem from the client.
I obtained the latest code directly from the master branch of unbound.
I apologize for not being able to provide a readily available machine to help replicate the issue, as this was a problem I encountered a year ago.
Another clue I can offer is that I suspect the issue is not caused by the computational overhead of the public key cryptography algorithms within OpenSSL. I experimented with using different OpenSSL Engines to accelerate these algorithms and found that regardless of the ENGINE configuration, the throughput remained strangely low.
My current workaround is to use Ubuntu 20.04 instead of 22.04.
I understand that the information provided may not be sufficient to replicate the problem. If you need further details, please let me know, and I'd be happy to assist in reproducing the issue.

If I understand correctly you are using the exact same version of Unbound built from source on two different systems with different results. Could you share your configure command? If you have a running Unbound at the moment, unbound -V would also include it.

I've documented nearly all commands and their corresponding outputs when replicating this issue. Please take a look at the debug_unbound. It contains ample information.
If the above details are still insufficient, please let me know what additional information you need.

Thanks for this! I can already spot something that my coworker also thought would be an issue:
Linked libs: mini-event internal (it uses select), OpenSSL 1.1.1q  5 Jul 2022

Is mini-event also used on a 20.04 system?

Yes, all configurations are same on Ubuntu 20.04.

I wasn't clear enough or your answer is not clear to me :)
I meant to ask if you get the same output for the unbound -V command on a 20.04 system.

Sorry, I don't have a ubuntu20.04 system to run unbound, I just remember that the operation flow including all the commands on the ubuntu20.04 system is exactly the same

No worries, I'll try to replicate on 20.04 and come back to this.

Now that it's late at night in China, I can use the Libevent library to link and rerun unbound on my 22.04 system tomorrow

Has there been any further progress regarding this issue?

I verified that mini-event is also used with a default 20.04 system but I can look further on reproducing next week.

I can reproduce the exact same behavior but not sure what is happening. I am comparing 20.04 and 24.04 btw; 24.04 is slow.
The last thing I noticed that is different with tcpdump is that your client seems to be sending RST packets in 24.04. Haven't investigated further though yet.


[bug] order of the env variables breaks parsing

Describe the bug
If dynamic env variables are in the wrong order, and depend on each other (eg. in the .env file) the config will not parse.
This is problematic in docker environment because the order of env variables might be random.
The problem didn't occur when I deleted config-defaults.yaml from the example or @format was used in place of @jinja.
To Reproduce
Steps to reproduce the behavior:

Having the following folder structure

 Project structure

.
├── Dockerfile
├── config-defaults.yaml
├── config.py
└── requirements.txt

1 directory, 4 files

Having the following config files:

 Config files
.env
APP_PATH_ROOT=@jinja {{this.PATH_ROOT_REL | abspath}}
APP_PATH_DATA=@format {this.PATH_ROOT}/some_data
APP_PATH_ROOT_REL=..
APP_PATH_MODEL=@format {this.PATH_ROOT}/some_model

and
config-defaults.yaml
PATH_ROOT: ../..
PATH_DATA: "@jinja {{this.PATH_ROOT | abspath}}/data"
PATH_MODEL: "@jinja {{this.PATH_ROOT | abspath}}/models"
PATH_CONFIG: "@jinja {{this.PATH_ROOT | abspath}}/config"

Having the following app code:

 Code
/config.py
from pathlib import Path

from dynaconf import Dynaconf
from pprint import pprint

src_dir = Path(__file__).parent.resolve()

settings = Dynaconf(
    envvar_prefix="APP",
    settings_files=["config.yaml", src_dir / "config-defaults.yaml"],
    load_dotenv=True,
    merge_enabled=True,
)

pprint(settings.as_dict())

Executing under the following environment

 Execution
$ python config.py
Traceback (most recent call last):
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/config.py", line 15, in <module>
    pprint(settings.as_dict())
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/base.py", line 115, in __getattr__
    self._setup()
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/base.py", line 174, in _setup
    self._wrapped = Settings(
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/base.py", line 253, in __init__
    self.execute_loaders()
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/base.py", line 1035, in execute_loaders
    core_loader.load(self, env, silent=silent, key=key)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/loaders/env_loader.py", line 24, in load
    load_from_env(obj, global_prefix, key, silent, IDENTIFIER + "_global")
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/loaders/env_loader.py", line 82, in load_from_env
    obj.update(data, loader_identifier=identifier)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/base.py", line 952, in update
    self.set(
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/base.py", line 869, in set
    existing = getattr(self, key, None)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/base.py", line 298, in __getattribute__
    return self._store[name]
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/utils/boxing.py", line 26, in evaluate
    return recursively_evaluate_lazy_format(value, settings)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/utils/__init__.py", line 413, in recursively_evaluate_lazy_format
    value = value(settings)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/utils/parse_conf.py", line 188, in __call__
    result = self.formatter(self.value, **self.context)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/utils/parse_conf.py", line 146, in __call__
    return self.function(value, **context)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/utils/parse_conf.py", line 157, in _jinja_formatter
    return jinja_env.from_string(value).render(**context)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/jinja2/environment.py", line 1301, in render
    self.environment.handle_exception()
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/jinja2/environment.py", line 936, in handle_exception
    raise rewrite_traceback_stack(source=source)
  File "<template>", line 1, in top-level template code
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/jinja2/environment.py", line 485, in getattr
    return getattr(obj, attribute)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/base.py", line 298, in __getattribute__
    return self._store[name]
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/utils/boxing.py", line 26, in evaluate
    return recursively_evaluate_lazy_format(value, settings)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/utils/__init__.py", line 413, in recursively_evaluate_lazy_format
    value = value(settings)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/utils/parse_conf.py", line 188, in __call__
    result = self.formatter(self.value, **self.context)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/utils/parse_conf.py", line 146, in __call__
    return self.function(value, **context)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/dynaconf/utils/parse_conf.py", line 157, in _jinja_formatter
    return jinja_env.from_string(value).render(**context)
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/jinja2/environment.py", line 1301, in render
    self.environment.handle_exception()
  File "/Users/amadeusz.lisiecki/workspace/dynaconf-test/.venv/lib/python3.9/site-packages/jinja2/environment.py", line 936, in handle_exception
    raise rewrite_traceback_stack(source=source)
  File "<template>", line 1, in top-level template code
  File "/Users/amadeusz.lisiecki/.pyenv/versions/3.9.14/lib/python3.9/posixpath.py", line 375, in abspath
    path = os.fspath(path)
TypeError: expected str, bytes or os.PathLike object, not Undefined

Expected behavior
Environment variables can come in random order and freely depend on each other.
Environment (please complete the following information):

OS: macOS Ventura 13.1
Dynaconf Version 3.1.11
Jinja2 Version 3.1.2

Additional context
I faced this problem mostly in a cloud environment where I was running docerized jobs - because environment variables where passed by the job controller (sagemaker in this case). I reproduced this docker container and then locally in virtual environment. Because of this I wasn't able to use dynamic variables in my jobs.

@pedro-psb do you think we can close this issue as resolved?

@rochacbruno I was wanting to review this first but was busy with other things. I'll try to do it this week

Fix worked for me in multiple, previously bugged, cases - thanks for the great work!

I've reviewed that case and we can close it now!
I'll summarize some more in-depth understanding of the original problem (as a note for the future, as it may be relevant for related issues).
Problem breakdown
Given these dependencies loaded in this order: (b'-b and a'-a are the same keys, but from different sources)
# first source A
b' -> a' # reads: b' depends on a'
a'

# second source B
a -> c
b -> a (this will break)
c

Internally something like this happens:
# internall calls
setting.set(b') # ok: store lazy (b')
setting.set(a') # ok: store (a')

setting.set(a) # ok: replaces lazy (a')  with lazy (a)
setting.set(b) # raises
	# check if (b) already exist (for merging purposes)
	existing = self.get(b, None)
	# it exists: lazy (b'), from source A

	# this then triggers recursive evaluation, because all Box access triggers this
	recursive_evaluation:
		resolve (b'), but
		(b') dependends on (a) # previously a'
		(a) depends on (c)
		(c) isn't set yet

And somehow (a) ends up in jinja template resolver and filter processing as undefined(when trying to resolve (b')), which raises the original exception. This is odd, as I would expect another kind of error, but this is not the critical point.
Conclusions
My current workaround may cause some edge merging cases involving Lazy object to fail, because it just ignores the existing setting if it is of Lazy type. While it works well I think we can focus on other priorities, but if this become a problem I have the following hints:
Improve Box acessors
A more broad and complete solution would involve allowing a more flexible acessor to Box objects, which allow internal use to properly skip recursive evaluations. Unintentionally calling recursive evalution internally is wastefull and may cause bugs. Also, it would be nice to separate public and private get and set methods for Settings, as requirement for both can be very different.
I did a workaround for bypassing evaluation before, while working on other issues (setting box._conf_box["_bypass_evaluation"]). It kinda worked but it is not robust at all. Recently it presented some problems, due to it's kind-of-global-variable nature.
Improve dependency resolving
Still, it maybe be necessary to evaluate a Lazy object to check if it is mergeable (in this hypothetical merge-with-lazy edge case), so if this happens to be a real problem, I'd go for implementing a dependency analysis with graphlib (similar to a draft I did for configx), which would allow for isolating the dependent setting from a bulk loading process and, after that, set only those dependent keys in the proper order.


Migrate to angular 6.0.1 and Rxjs 6.1.0

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Migration to angular 6.0.1 and Rxjs 6.1.0

What is the current behavior? (You can also link to an open issue here)

https://github.com/ObaidUrRehman/ng-drag-drop/issues/132

What is the new behavior (if this is a feature change)?

Travis build failed due to a strange timeout while fetching some dependencies.
Nice PR 👍 I was about to do the same as removing rxjs-compat is a top priority for a lot of people right now (including me).

@Supamiu Thanks! Should I do anything concerning the Travis build or should I wait for anyone to restart it if possible ?

I think @ObaidUrRehman should restart the build, or test the build by him/herself, just to make sure everything is going fine.

@arnaudleclerc, Thanks for this PR.
I remember fixing the Travis issue in master. I forgot to merge it into develop. Anyway, I will take care of it in a few hours.

@ObaidUrRehman I managed to start the demo app without the angular-cli. I'll take a look at it and let you know. Thanks for the merge!


[Docs] RE-structure the TSV to be more reader friendly (unimportant columns to the end)

First attempt: https://github.com/NFDI4Chem/VibrationalSpectroscopyOntology/blob/sort-logically-columns/src/templates/vibso_classes.tsv

I'm totally fine with your proposal. :)


Transform CSGShape collision when necessary

Fixes #26827.
Fixes #27364.
Fixes #40683.

This PR will be superseded by #40814 if merged.

Thanks!

Cherry-picked for 3.2.4.


300% benchmarks regression between Dec 13 and 14

There is a huge regression in many engine benchmarks and some stdlib ones between Dec 13 and Dec 14:

[ ] Investigate
[ ] Fix

The regression has happened between following commits e9b0ba95b087210ceb4b988473d6ec219ebb939c and e9b0ba95b087210ceb4b988473d6ec219ebb939c which leaves us with only one suspect:

which identified the only culprit to be #11850

git log e9b0ba95b087210ceb4b988473d6ec219ebb939c...63ed62921012f57b7a5cf169351e8872c2fde59a
commit 63ed62921012f57b7a5cf169351e8872c2fde59a (tag: 2024.5.1-nightly.2024.12.16, tag: 2024.5.1-nightly.2024.12.15, tag: 2024.5.1-nightly.2024.12.14)
Author: James Dunkerley<EMAIL_ADDRESS>Date:   Fri Dec 13 21:01:16 2024 +0000

    Excel Reading (XLSX) using SAX Default Handler (#10877)

    - Implement Excel reading as a SAXMLParser.

commit e6bcd5e485a82abe9b0fd5364c8a1b98c926171f
Author: Radosław Waśko<EMAIL_ADDRESS>Date:   Fri Dec 13 20:28:03 2024 +0100

    `aggregate ..Sum` of integer column remains integer and handles overflow (#11860)

    - This was mentioned in #7192 but didn't get a proper ticket.
    - Ensuring that summing integers gives an integer and not a float.
    - Only in-memory, as in Database the result type is database-dependent and we want it to be like that.
    - Also allowing the integer sum to overflow and become a `BigInteger`, in that case the resulting column will become `Decimal`.

commit be6eb1e49f8256fd0bab49094b6c1f3b84764c52
Author: Paweł Buchowski<EMAIL_ADDRESS>Date:   Fri Dec 13 20:14:21 2024 +0100

    Fix sending same refreshToken as accessToken in open project request (#11862)

    In `open_project` request body we send `cognitoCredentials` for the project to be able to communicate with Cloud API. Currently the `refreshToken` has the same value as `accessToken` making that communication buggy.

commit 7998ed52181258996f530ecc7b8301532ca1b986
Author: Jaroslav Tulach<EMAIL_ADDRESS>Date:   Fri Dec 13 16:02:51 2024 +0100

    case of and intersection types (#11850)

    Also: Prefer `getURI` path when gathering location of a source


Allow alternative location for BioFormatsCache

Ideally there would be a property along the lines of omero.bioformatscache.dir. Since there isn't this new option omero_server_datadir_bioformatscache creates a symlink under omero.data.dir
Tag: 2.0.0-m3

TASK [ansible-role-omero-server : omero server | create omero BioFormatsCache] *
fatal: [omero-server-newdep]: FAILED! => {"changed": false, "failed": true, "msg": "/OMERO/BioFormatsCache already exists as a link", "path": "/OMERO/BioFormatsCache", "state": "absent"}
fatal: [omero-server-olddep]: FAILED! => {"changed": false, "failed": true, "msg": "/OMERO/BioFormatsCache already exists as a link", "path": "/OMERO/BioFormatsCache", "state": "absent"}

I think Travis is failing due to the issue fixed by https://github.com/openmicroscopy/ansible-role-omero-common/pull/3

Travis passes after https://github.com/openmicroscopy/ansible-role-omero-common/pull/3 was merged

Follow-up card: https://trello.com/c/yzXcSYfe/223-bioformatscache-should-be-configurable

Perfect. Cheers.


Upload Gallery placeholder images

Refs #3664

[ ] Upload the Gallery placeholder images to staging and production.  Be sure to remember to set the file as permanent, and also keep track of each File fid you are adding.

[ ] Set the Gallery Defaults variable via admin/config/dosomething/dosomething_campaign.

@kerigoff uploaded these to staging. once we review how they look in the slots we'll add them to prod

blocked on prod by #3664

this is now up on prod


chore(spider): remove optional params for data_get method

Wait

ready to merge

spider.spider_types is the correct one


Discussions - Separator lines standard margin

Jira: https://openedx.atlassian.net/browse/MA-2019

@aleffert Please review

@aleffert Please review

@aleffert Travis build is failing for some old code.
❌  /Users/travis/build/edx/edx-app-ios/Source/CourseCatalogDetailView.swift:103:25: value of type 'UIScrollView' has no member 'oex_addObserver'

Left a comment on the ticket, but I believe this was intentional behavior

That build breakage appears to be a compiler bug introduced in Xcode 7.3.1. I pushed a fix to master that works around it.

Ticket closed with no action needed so closing this PR.


Bug: Localization is not setting setlocale function from php.net

Direction
Hello i would like to translate months and days of week trough localization settings
Describe the bug
setLocale() method is not setting setlocale from php.net
as in example
     setlocale(LC_ALL, 'fr_FR.UTF-8');

and when i echo this line
     <?=strftime('%A')?>

i would get the name of the day in that language if is set correctly.
CodeIgniter 4 version
Latest Version

This is not a bug but rather not implemented. If you want this functionality, please send a PR.


python2.7 dist-packages urllib3 python error please help

File "/usr/lib/python2.7/dist-packages/urllib3/request.py", line 66, in request
**urlopen_kw)
File "/usr/lib/python2.7/dist-packages/urllib3/request.py", line 87, in request_encode_url
return self.urlopen(method, url, **extra_kw)
File "/usr/lib/python2.7/dist-packages/urllib3/poolmanager.py", line 244, in urlopen
response = conn.urlopen(method, u.request_uri, **kw)
File "/usr/lib/python2.7/dist-packages/urllib3/connectionpool.py", line 671, in urlopen
release_conn=release_conn, **response_kw)
File "/usr/lib/python2.7/dist-packages/urllib3/connectionpool.py", line 671, in urlopen
release_conn=release_conn, **response_kw)
File "/usr/lib/python2.7/dist-packages/urllib3/connectionpool.py", line 671, in urlopen
release_conn=release_conn, **response_kw)
File "/usr/lib/python2.7/dist-packages/urllib3/connectionpool.py", line 643, in urlopen
_stacktrace=sys.exc_info()[2])
File "/usr/lib/python2.7/dist-packages/urllib3/util/retry.py", line 363, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))

You haven't posted the actual error message OR any part of the traceback that's in ElastAlert's code, so this is not super useful.
That being said, I'm pretty sure MaxRetryError just means that it couldn't connect to Elasticsearch. Checking your es_host and es_port settings and make sure the cluster is reachable and responding.

I always get this error. sample msf and other python extension applications. I'm sorry my English is bad

Can you curl Elasticsearch?
$ curl <IP_ADDRESS>:9200

where <IP_ADDRESS> is your es_host and 9200 is your es_port?

Failed connect


Update MD file on new branch 2017-11-17_15-24-47

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

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


Add note to testing docs about rendered_content helper

When writing tests, it's helpful to be able to see the full HTML for a component to debug problems. Add a note about the existing rendered_content helper to make it easier for folks to find out how to do this.
I had to source dive to find out how to do this, which is a barrier to people new to the library.

@joelhawksley ty -- not sure if you want a Changelog entry or if you can force merge. Doesn't seem to need one to me!


some error with proxy

Log:
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--prefer-insecure', u'--verbose', u'--proxy', u'<IP_ADDRESS>:1080', u'-ciw', u'-f', u'bestaudio', u'--no-mtime', u'--download-archive', u'.archive.txt', u'--playlist-start', u'1', u'https://www.youtube.com/watch?v=Nf4N93QgLh4&index=4&list=PLM0YCMPhuD8FNznXBOBF2p0lQ2sNQLQ9e']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.10 - CYGWIN_NT-6.1-WOW-2.2.0-0.289-5-3-i686-32bit
[debug] exe versions: ffmpeg N-72346-ga838b22, ffprobe N-72346-ga838b22
[debug] Proxy map: {u'http': u'<IP_ADDRESS>:1080', u'https': u'<IP_ADDRESS>:1080'}
[youtube:playlist] Downloading playlist PLM0YCMPhuD8FNznXBOBF2p0lQ2sNQLQ9e - add --no-playlist to just download video Nf4N93QgLh4
[youtube:playlist] PLM0YCMPhuD8FNznXBOBF2p0lQ2sNQLQ9e: Downloading webpage
ERROR: Unable to download webpage: '' (caused by BadStatusLine("''",)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 326, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1860, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 431, in open
response = self._open(req, data)
File "/usr/lib/python2.7/urllib2.py", line 449, in _open
'_open', req)
File "/usr/lib/python2.7/urllib2.py", line 409, in _call_chain
result = func(*args)
File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 749, in https_open
req, **kwargs)
File "/usr/lib/python2.7/urllib2.py", line 1194, in do_open
h.request(req.get_method(), req.get_selector(), req.data, headers)
File "/usr/lib/python2.7/httplib.py", line 1053, in request
self._send_request(method, url, body, headers)
File "/usr/lib/python2.7/httplib.py", line 1093, in _send_request
self.endheaders(body)
File "/usr/lib/python2.7/httplib.py", line 1049, in endheaders
self._send_output(message_body)
File "/usr/lib/python2.7/httplib.py", line 893, in _send_output
self.send(msg)
File "/usr/lib/python2.7/httplib.py", line 855, in send
self.connect()
File "/usr/lib/python2.7/httplib.py", line 1266, in connect
HTTPConnection.connect(self)
File "/usr/lib/python2.7/httplib.py", line 835, in connect
self._tunnel()
File "/usr/lib/python2.7/httplib.py", line 812, in _tunnel
(version, code, message) = response._read_status()
File "/usr/lib/python2.7/httplib.py", line 417, in _read_status
raise BadStatusLine(line)

SOCKS proxies are not supported.


play course not work 404 error

when i press play course it give me 404 error

This is a common error caused by a mismatch in settings. Look in Appsetting.json at these two values, and adjust them if necessary:
"UploadFolder": "SCORMPackages/",
  "CourseFolder": "SCORMCourses/",

Make sure the paths for UploadFolder and CourseFolder actually match the way your system is installed. UploadFolder and CourseFolder might actually be in a different location or have different names.


Yuxin Wang collaborate with Yuxuan Li, Xingqi Pan

##Extra Credit

A working PIO sequencer (Part 7) before 11:59PM Wednesday Nov 16
working ADPS9960 communication (Part 8) before 11:59PM Wednesday Nov 16


[PointToSegmentPen] outputImpliedClosingLine=False still outputs implied closing lines

I'm running into an issue with this code:
https://github.com/fonttools/fonttools/blob/647abe70a9780515f32b744ffcf1511667744452/Lib/fontTools/pens/pointPen.py#L232-L248
Related issues:
https://github.com/googlefonts/fontmake/issues/572
https://github.com/fonttools/fonttools/pull/1720
https://github.com/googlefonts/cu2qu/pull/185
I have point-based data, and need segments to produce CFF2 CharsStrings. I very much do not want to output implied closing lines, as that would be redundant and wasteful.
However, #1720 added a case where if the last two points are the same, it will output the implied closing line even when outputImpliedClosingLine is set to False. This can create non-interpolatable output from perfectly interpolatable input. Example: I have a variable glyph containing a rectangle. In one of the sources, the width of this rectangle is zero, causing the above heuristic to output an additional lineto, breaking compatibility. (Ironically, this sounds exactly like the problem described in googlefonts/fontmake/issues/572.)
Looking at the code, and reading the issue that lead up to the current situation, I'm confused about what the real initial problem was. It also looks like googlefonts/cu2qu/pull/185 was done to fix the problem caused by #1720.
There are two things here

I desparately need the opposite of outputImpliedClosingLine, say, doNotOutputImpliedClosingLine, or any other way to get the PointToSegmentPen behavior of before #1720. I can prepare a PR for this.
Perhaps reconsider the solution of #1720 altogether. In retrospect, I think it's asking for trouble to change the topology of the output based on the input coordinates.

As to number 2, given that googlefonts/cu2qu/pull/185 was done, isn't the general solution to set outputImpliedClosingLine to True if you need to reliably round-trip between segments and points? That is, the fix to the original problem is to set outputImpliedClosingLine to True in relevant places, and not apply heuristics based on the coordinates?

Strike "desparately": I managed to work around the problem in my own code:
https://github.com/googlefonts/fontra-compile/blob/c9da4b0a5d0ff399fb31217ccb4d59c260b283e0/src/fontra_compile/builder.py#L951-L960
TBH, I'm not really sure how to proceed here.


Code Placeholders that break the whole code

Things like:
//this code remains unchanged
That app tries to input, and i have a simple solution to it, if it's possible to implement into plugin.
Make the Claude ai to write out every function that it needs to change, not the whole code.
This way it would write out new functions and the changed ones, the plugin has to find this functions by search and then, change them.
This way we would achieve 2 things:

Less tokens for every code change(Since the app doesn't need to write the whole code every time)
More accuracy, since it specifically changes the functions.

It might be made into an option in the ClaudeDev, so the user would be able to choose what fits him.
Could the dev make it true? I Believe that would help the app greatly!

This is likely due to Claude trying to edit a large file. I recommend using the Anthropic provider option if you don't already, since it has a max output of 8192 tokens whereas OpenRouter doesn't handle large outputs well and AWS Bedrock is capped at 4096 tokens. You can also try breaking your file down into smaller files.
I also just released v1.1.13 which updates the system prompt to be stricter against lazy coding. Please give it a try and let me know if you still keep running into this issue.


Senegal iban validation

Example IBAN for test was taken from: https://www.iban.com/structure
The structure taken from: https://en.wikipedia.org/wiki/International_Bank_Account_Number

@mdomke , would you have time in the nearest future to review this PR? Any estimates? :)

Thanks!


TAF and other aspects that should be mentioned in the adjoint tutorials

@Shreyas911 , I feel that in https://github.com/ECCO-Hackweek/ecco-2024/blob/main/book/tutorials/AD_tuts_index.md it would be useful to add :

a presentation of TAF -- the tool generally used today for getting the MITgcm adjoint in ECCO and beyond.
pointers to the relevant MITgcm docs
links to the EMU part that is based on TAF

During the session on these adjoint tutorials, maybe @owang01 could demo TAF as an explanation for how EMU was built?

I agree @gaelforget, it's just that I have never used TAF, so I don't know much about it. Someone more experienced with TAF could take this on if possible.


feature/CE-440v2

Description
Please provide a summary of the change and the issue fixed. Please include relevant context. List dependency changes.
Fixes # (issue)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Further comments

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend available
Frontend available
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

@marqueone-ps Can you address the 4 issues reported by SonarCloud? (the TODO can be closed as accepted)

@marqueone-ps - Something has broken here, and I think it's on this side.   I'm trying to pinpoint the issue, but I think what is happening is that when you are displaying all actions instead of only the active actions.
Steps to reproduce:

Go to a HWCR
Select only 1 Assessment (or Prevention and Education)
Save
All values show up


Use Pipelinerun templates in pac generate command

Use builtin pipelinerun templates as shipped to the one for the console.
Don't use templating since we want to give it as raw to the console so
use string substitions.
Make java template use maven and not deploy
Closes #737
Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>
Changes

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

/retest

/retest

great cluster reinstall
seems stuck

/retest

Codecov Report

Merging #738 (04da04f) into main (ed8daee) will increase coverage by 0.05%.
The diff coverage is 80.95%.

@@            Coverage Diff             @@
##             main     #738      +/-   ##
==========================================
+ Coverage   65.32%   65.37%   +0.05%
==========================================
  Files          70       70
  Lines        4193     4168      -25
==========================================
- Hits         2739     2725      -14
+ Misses       1183     1175       -8
+ Partials      271      268       -3

Impacted Files
Coverage Δ

pkg/cmd/tknpac/generate/template.go
77.77% <80.95%> (+4.09%)
:arrow_up:

pkg/kubeinteraction/secrets.go
60.86% <0.00%> (-5.80%)
:arrow_down:

pkg/acl/acl.go
100.00% <0.00%> (ø)

pkg/provider/provider.go
72.72% <0.00%> (+2.13%)
:arrow_up:

Continue to review full report at Codecov.

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


GH Action – Comment built image names and tags to a PR workflow failing

Context
I have updated the requirements.txt file in the Handbook Authoring image in PR https://github.com/2i2c-org/community-showcase/pull/49.
The workflows for generating the build matrix, as well as building and pushing image to Quay.io, appear to be working fine. However, the subsequent workflow appears to be failing with the error message:
The template is not valid. .github/workflows/comment-built-images-to-pr.yaml (Line: 25, Col: 25): A mapping was not expected

Proposal
No response
Updates and actions
No response

Tagging @sgibson91 ✋

I think this line just needs to be a string:
https://github.com/2i2c-org/community-showcase/blob/2e42fc9136b160b03413f9c68521bc3e7c183069/.github/workflows/comment-built-images-to-pr.yaml#L25

I have made progress. I have solved the issue in the GitHub workflow file, and now have a new error of:
Traceback (most recent call last):
  File "/home/runner/work/community-showcase/community-showcase/scripts/comment-image-name-to-pr.py", line 92, in <module>
    with open(f"name.txt") as f:
FileNotFoundError: [Errno 2] No such file or directory: 'name.txt'
Error: Process completed with exit code 1.


docs(agendash): add documentation for agendash package

Information

Type
Breaking change

Doc
No

Added documentation for Agendash.

:tada: This PR is included in version 6.102.8-rc.1 :tada:
The release is available on:

v6.102.8-rc.1
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 6.102.8 :tada:
The release is available on:

v6.102.8
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.0.0-beta.4 :tada:
The release is available on:

v7.0.0-beta.4
GitHub release

Your semantic-release bot :package::rocket:


Almost not possible to load the parent project in IDEA

There are too many sub-modules, almost impossible to load parent project (Even though the sub-modules are grouped by profile).
Not sure if I'm loading it correctly with IDEA.
Is there anyone facing the same issue with me?

This repository isn't really meant to be loaded all at once.
Our articles typically just span a single module, Just load the module that you need at the time.


Add controller-branch

Add controller-branch
Modify service-branch

Add controller-branch
Modify service-branch


Does the CLI support passing directories?

Yes, for these two cases

Input from a directory https://github.com/ga4gh/dockstore/blob/develop/dockstore-client/src/test/resources/dir6.cwl
Output to a directory https://github.com/ga4gh/dockstore/blob/develop/dockstore-client/src/test/resources/splitBlob.cwl https://github.com/ga4gh/dockstore/blob/develop/dockstore-client/src/test/resources/splitBlob.json

Let's keep an eye out for more cases


[Help] Indesirable white background container while dragging

Hello,
I'm facing an issue while styling my dragging element: when element fire the drag event a weird white background is set to the dragged element.
First, I tried to set the background color to transparant, or add my own background, but whatever, the dragging element still keep this corned white background.
I think, this background is owned by a dragging container where the dragged element is placed as it appear, not on choosen or ghost element. I tried to use event callback (onStart callback) an change any item style, but nothing change.
Then i check the demo, and it seems that this white background exist already.
You can check the problem in the demo here: http://sortablejs.github.io/Sortable/
Or in the codesanbox template here: https://codesandbox.io/s/sortablejs-javascript-jy3tl?file=/src/index.js
Just be aware of corner while hover other list element.
It's more visible in my project as my element container have empty space.
Someone does have, found a hack to remove this behavior ?
Thanks for help !

I don't see it.
Here's a screenshot of your codesandbox in Firefox v92
Looks pretty good to me.

Hello,
Thanks for reply.
I tested again but in differents browser and the issue seem to be on chrome (I tested firefox, and brave)

Probably a browser issue with native drag & drop. Set the settings forceFallback: true to avoid native drag & drop.

forceFallback: true

That really helps me. Thanks!!!

Probably a browser issue with native drag & drop. Set the settings forceFallback: true to avoid native drag & drop.

That really helps me out !!!

Yes, white left border is easy to reproduce.