docs: kubectl commands don't work unless the user sets a namespace

Once the Deis charts have been loaded, we need to tell users how to set up their kubectl to use the deis namespace.
Option 1: Show kubectl commands with the --namespace=deis option.
Option 2: Instruct them to modify the namepsace with kuebctl config [...]

I should have put this in there:
⇒  k get po
NAME      READY     REASON    RESTARTS   AGE

⇒  k get --namespace=deis po
NAME                        READY     REASON    RESTARTS   AGE
deis-builder-9bvxb          1/1       Running   1          17m
deis-database-qftow         1/1       Running   0          17m
deis-etcd-1-3rvvf           1/1       Running   0          17m
deis-etcd-1-5le12           1/1       Running   0          17m
deis-etcd-1-vnapt           1/1       Running   1          17m
deis-etcd-discovery-4kvtw   1/1       Running   0          17m
deis-minio-r47jo            1/1       Running   0          17m
deis-registry-mcoht         1/1       Running   0          17m
deis-router-k30v5           1/1       Running   0          17m
deis-workflow-gplaw         1/1       Running   0          17m

And you can check which namespace your kubectl is in using kubectl config view.

Ooo... I misread the instructions. @bacongobbler does say to put --namespace=deis on the kubectl get pods command.


move setup data into cfg, fix licence info

Hi !
I improved your packaging, so it should work out of the box. Just make sure the good names and infos are set, and use twine for the upload on pip. Plus, i fixed the licence info to MIT, as put in your repository.
xstate is an available package name, you should register it as soon as possible.
Have a good day !

Thank you!

Thank you!


Bicep Deployment Error - WebhookInvalidParameterValue (Private Azure Container Registry)

[x] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
Failure to pull container image from private azure container registry when deploying azure container app via bicep.
"WebhookInvalidParameterValue\\\",\\r\\n  \\\"message\\\": \\\"The following field(s) are either invalid or missing.
Invalid value: \\\\\\\"cosmossdkidentitydemoacr.azurecr.io/apps/books-api:18a49dc9e045714edaafad60a3f2f9ae67b542e3\\\\\\\":
GET https:?scope=repository%3Aapps%2Fbooks-api%3Apull&service=cosmossdkidentitydemoacr.azurecr.io:
UNAUTHORIZED: authentication required, visit https://aka.ms/acr/authorization for more information.: template.containers.books-api.image.\\\"\\r\\n***\"\r\n          ***\r\n        ]\r\n      ***\r\n    ]\r\n  ***\r\n***"***]***

Steps to reproduct

Provision azure container apps environment.
Provision azure container registry (store admin password in KV).
Build & Push docker image to ACR tagged with GitHub sha.
Deploy azure container apps passing in the image name with tag as the image to deploy (get admin pass from KV) via bicep.

Actual behavior
An error was thrown as shown above, the example error can also be seen in this build here:
https://github.com/mumby0168/cosmos-repository-sdk-identity/runs/6628955590?check_suite_focus=true#step:5:20
Screenshots
n/a
Additional context
There is a full working example of the issue available on this GitHub repository:
https://github.com/mumby0168/cosmos-repository-sdk-identity
The GitHub Actions file can be found here:
https://github.com/mumby0168/cosmos-repository-sdk-identity/blob/main/.github/workflows/deploy.yml

I have recently updated this to use managed identities to pull the image and I get the exact same error see below

@mumby0168 I think the problem here is that the provided registry server does not match the registry server in the image name. This is the transpiled ARM request from the bipec deployment:
  "registries": [
  {
    "server": "cosmossdkidentitydemoacr",
    ...
  }
  ...
  "template": {
      "containers": [
        {
          "image": "cosmossdkidentitydemoacr.azurecr.io/apps/books-api:49feb3713bccacaf8832268bb7691e71dac77290",
          ...
        }
     ]
  }

The registry server needs to include the full domain: cosmossdkidentitydemoacr.azurecr.io. Right now what's happening is that we look for a registry named cosmossdkidentitydemoacr.azurecr.io based on the image, which is not found in the provided registry list, so we try to pull the image using anonymous pull (which is a valid configuration, as long as the registry has anonymous pull enabled. In your case it doesn't, so you get an unauthorized response).

@mumby0168 I think the problem here is that the provided registry server does not match the registry server in the image name. This is the transpiled ARM request from the bipec deployment:

  "registries": [

  {

    "server": "cosmossdkidentitydemoacr",

    ...

  }

  ...

  "template": {

      "containers": [

        {

          "image": "cosmossdkidentitydemoacr.azurecr.io/apps/books-api:49feb3713bccacaf8832268bb7691e71dac77290",

          ...

        }

     ]

  }

The registry server needs to include the full domain: cosmossdkidentitydemoacr.azurecr.io. Right now what's happening is that we look for a registry named cosmossdkidentitydemoacr.azurecr.io based on the image, which is not found in the provided registry list, so we try to pull the image using anonymous pull (which is a valid configuration, as long as the registry has anonymous pull enabled. In your case it doesn't, so you get an unauthorized response).

Brilliant, thanks for that! I will retry this tomorrow. Then I'll update here and close the issue.
Many thanks

Hi @vturecek I tried this just now and I set the ACR name to have the azure.io suffix, you can see this in the below inputs to the ace module:

The error is the same:
{
    "status": "Failed",
    "error": {
        "code": "WebhookInvalidParameterValue",
        "message": "The following field(s) are either invalid or missing. Invalid value: \"cosmossdkidentitydemoacr.azurecr.io/apps/books-api:228380a04acb0fc091ec117c18cda53d7f3cda5b\": GET https:?scope=repository%3Aapps%2Fbooks-api%3Apull&service=cosmossdkidentitydemoacr.azurecr.io: UNAUTHORIZED: authentication required, visit https://aka.ms/acr/authorization for more information.: template.containers.books-api.image."
    }
}

Azure correlation ID: a0e48484-d61e-4b00-80fe-5c6fce72c823 and operation ID 0BC59E506D6F9B51 Any ideas?
Thanks,
Billy

@vturecek any updates on this ? :)

I have the same issue. Could still not find any solution.

Hi I am still stuck on this also.

@mumby0168
it probably should be azurecr.io, not azure.io
However I am stuck with the same error

I have found a solution by accessing azure container registry with username and password:
Bicep module:
param location string
param containerAppsEnvironmentId string
param imageVersion string
param containerRegistryUsername string
@secure()
param containerRegistryPassword string

var environmentConfig = [
  {
    name: 'ASPNETCORE_ENVIRONMENT'
    value: 'Development'
  }
  {
    name: 'ASPNETCORE_URLS'
    value: 'http://<IP_ADDRESS>:80'
  }
]

resource containerApp 'Microsoft.App/containerApps@2022-03-01' = {
  name: 'shopping-api'
  location: location
  properties: {
    managedEnvironmentId: containerAppsEnvironmentId
    template: {
      containers: [
        {
          name: 'shopping-api'
          image: 'andaha.azurecr.io/andaha/services/shopping:${imageVersion}'
          env: environmentConfig
          probes: [
            {
              httpGet: {
                port: 80
                path: '/hc'
              }
              type: 'Readiness'
            }
            {
              httpGet: {
                port: 80
                path: '/liveness'
              }
              type: 'Liveness'
            }
          ]
        }
      ]
      scale: {
        minReplicas: 0
        maxReplicas: 2
      }
    }
    configuration: {
      activeRevisionsMode: 'single'
      dapr: {
        enabled: true
        appId: 'shopping-api'
        appPort: 80
      }
      ingress: {
        external: true
        targetPort: 80
        allowInsecure: true
      }
      registries: [
        {
          server: 'andaha.azurecr.io'
          username: containerRegistryUsername
          passwordSecretRef: 'container-registry-password'
        }
      ]
      secrets: [
        {
          name: 'container-registry-password'
          value: containerRegistryPassword
        }
      ]
    }
  }
}

Bicep main module:
param location string = resourceGroup().location
param version string
param containerRegistryUsername string
param containerRegistryPassword string

module coreInfrastructure 'core-infrastructure.bicep' = {
  name: 'andaha-core-infrastructure'
  params: {
    location: location
  }
}

module shoppingService 'service.bicep' = {
  name: 'andaha-shopping-service'
  params: {
    location: location
    containerAppsEnvironmentId: coreInfrastructure.outputs.containerAppEnvironmentId
    imageVersion: version
    containerRegistryUsername: containerRegistryUsername
    containerRegistryPassword: containerRegistryPassword
  }
}

From the pipeline i am reading the username and password from azure container registry and pass it to the main bicep file:
- task: AzureCLI@2
      displayName: 'Deploy to dev'
      inputs:
        azureSubscription: $(azureServiceConnection)
        scriptType: bash
        scriptLocation: inlineScript
        inlineScript: |
          acrUser=$(az acr credential show --name andaha --resource-group andaha --query username -o tsv)
          acrPassword=$(az acr credential show --name andaha --resource-group andaha --query "passwords[0].value" -o tsv)
          az deployment group create --resource-group andaha-dev --template-file $(bicepTemplateFile) --parameters containerRegistryUsername="$acrUser" containerRegistryPassword="$acrPassword" version="$(Build.SourceBranchName)"

Also blocked because of this.
This leads to using ACR premium with dedicated scope maps and tokens in the mean time 😞

I am also experiencing this error. I have registry configuration set properly:
            "secrets": [
                {
                    "name": "container-registry-password"
                }
            ],
            "registries": [
                {
                    "server": "docker.io",
                    "username": "...",
                    "passwordSecretRef": "container-registry-password",
                    "identity": ""
                }
            ]

when trying to apply the image from the private Docker registry
{
    "properties": {
        "template": {
            "containers": [
                {
                    "image": "docker.io/....:v1",
                    "name": "api",
                    "resources": {
                        "cpu": 0.5,
                        "memory": "1Gi",
                        "ephemeralStorage": ""
                    }
                }
            ]
        }
    }
}

it ends up with an error:
{
    "code": "WebhookInvalidParameterValue",
    "message": "The following field(s) are either invalid or missing. Invalid value: \"docker.io/...:v1\": GET https:: UNAUTHORIZED: authentication required; [map[Action:pull Class: Name:... Type:repository]]: template.containers.api.image."
}

@CezaryKlus I have tested my solution only for azure container registry. Not sure if this is working on docker registry aswell. Maybe you can try another registry url, some examples are here: https://stackoverflow.com/questions/34198392/docker-official-registry-docker-hub-url
Have you forgot to set the secret value for 'container-registry-password'?

Is there a resolution to this? because i suppose it is a very basic requirement to be able to pull an image from a private azure container registry while we deploy an instance of Container apps via templates...

This has become an issue for me.

@zbuchheit if you have a correlation ID from the failed attempt to create/update a Container App, I can take a look for you.

@vturecek, here is a correlation ID that you can look at: a319c1ec-fcee-467f-aaa0-16dbb9f3645d

We are no longer experiencing this issue. Thanks for the help @vturecek.
For anyone interested, we found that azureADAuthenticationAsArmPolicy was 'disabled'. This was causing the error: "WebhookInvalidParameterValue". Setting azureADAuthenticationAsArmPolicy to 'enabled', resolved this issue.

In case anyone else runs into this, you can use the following command to check if ARM tokens are allowed to access your ACR:
az acr config authentication-as-arm show -r <registry>

If ARM tokens are disallowed, you can allow them with the following command:
az acr config authentication-as-arm update -r <registry> --status [enabled/disabled]


Define filter function semantic

It's not the same in restorer and archiver

What exactly do you mean by "semantic" here? Do you mean they should have the same function signature (some parameter types)? And what's the advantage of unifying that even? Or are you saying the return values are interpreted differently?

I have the impression that the return values for the filter functions mean different things. I'd like to specify the meaning of that explicitely for both functions. We'll need filter functions in other places in the future, so this should be done right so we can reuse it.


Parameterize appsettings.json file location

The emulator currently makes assumptions about where to find appsettings.json file.
Let's parameterize it instead. That way we can be more explicit about where to find the file.
I'm thinking something like:
MBBSEmu -S /foo/bar/appsettings.json
I'm not married to the flag being -S. Whatever floats your boat.

Ok, if I'm reading this correctly, I think what needs to be done is:

move the ILogger _logger = ServiceResolver.GetService line down to get the logger out after the command line options are parsed
Move the config = ServiceResolver.GetService to also be below the command line parsing
add a case statement for pulling out a -S option for the appsettings.json
somehow pass the above into the ServiceResolver when everything gets built so that it can honor the appsettings.json  file location

I /think/ that's what has the least impact here

Notes from Discord conversation with @enusbaum

you can define the two variables with .GetService() calls below the command line switches
just define ILogger where it is and assign it after the command line parsing
you'll probably need to change the pattern in ServiceResolver though
thats where it'll get hairy
But you could just leave the static constructor empty
and make a new Initalize() method
and call that in program.cs to setup the Service Collection

Should be all taken care of by https://github.com/enusbaum/MBBSEmu/pull/13


typings update

Improvements to the TypeScript typing file (index.d.ts).

Coverage remained the same at 71.739% when pulling 0464e9850b24867dbe1019e7ae38a5e9bd553ac1 on nrakic90:master into d8bbc9b961cb29f3147972b5797cb7a254bca6b7 on LeonardoGentile:master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


question about the hello world example

In the hello world example, using a fixed constant for the srand() function doesn't show the same results across multiple runs. Is this expected ?
run 1:
0: mean activity = 0.439031
1: mean activity = 0.501078
2: mean activity = 0.317878
3: mean activity = 0.729056
4: mean activity = 0.502201
5: mean activity = 0.625352
6: mean activity = 0.563613
7: mean activity = 0.688112
8: mean activity = 0.438253
9: mean activity = 0.688300
run 2:
0: mean activity = 0.500670
1: mean activity = 0.563959
2: mean activity = 0.568377
3: mean activity = 0.504105
4: mean activity = 0.440226
5: mean activity = 0.625791
6: mean activity = 0.688198
7: mean activity = 0.562702
8: mean activity = 0.688510
9: mean activity = 0.563258
Thanks

Sorry, I don't think anyone is maintaining this anymore and you asked your question so long ago that you probably don't care about the answer anymore. :)
It is expected because of the parallel execution on the GPU. Order of execution might be slightly different on each run leading to slightly different output


⚠ Bofrost.fr has degraded performance

In 4d5f6e4, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2176 ms

Resolved: Bofrost.fr performance has improved in 3f6f107.


DSL free functions -> types

Replace free functions such as oneOrMore with types such as OneOrMore. This hides overloads of free functions as initializers within those types. It also makes the DSL consistent with SwiftUI.

oneOrMore -> OneOrMore
zeroOrMore -> ZeroOrMore
optionally -> Optionally
repeating -> Repeat
choiceOf -> ChoiceOf
capture -> Capture
tryCapture -> TryCapture

Note: The reason we didn't realize this was possible (e.g. in #126) was because we were narrowly focused on including the subpattern type in the quantifier/combinator's generic parameter, i.e. OneOrMore<Component: RegexComponent>, which made it impossible to deduce each type's typealias Match from Component. Now we have an unconstrained generic parameter (e.g. OneOrMore<Match>) which gives us the full flexibility to hide Match deduction rules in initializers' type signatures.
Result: All non-operator top-level functions have been eliminated.

@swift-ci please test

Just when I was starting to appreciate the functions...

I had the same inertia myself…

@swift-ci please test


WIP: Fix Docker Deprecated Warnings

Fixes #878

Codecov Report

Merging #879 (afb8d9c) into develop (cee75c2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #879   +/-   ##
========================================
  Coverage    94.74%   94.74%
========================================
  Files          152      152
  Lines        10690    10690
========================================
  Hits         10128    10128
  Misses         562      562

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


logs processor is not emitting event.dataset some UIs do not work properly

Bug Report
Which version of the demo you are using? (please provide either a specific
https://github.com/elastic/opentelemetry-demo/tree/1.11.5
Symptom
logs processor is not emitting event.dataset some UI do not work properly
What is the expected behavior?
logs processor should emit event.dataset
What is the actual behavior?
event.dataset is missing
Reproduce
Run the otel demo add the metrics and logs daemonset.
Fix
This line should have attributes/dataset as part of the list of processors
https://github.com/elastic/opentelemetry-demo/blob/27b4923ba9acd316d3726a29aad1f7e32299bc8c/kubernetes/elastic-helm/daemonset.yaml#L314
Is
        processors: [batch, k8sattributes, resourcedetection/system, resourcedetection/eks, resourcedetection/gcp, resource/demo, resource/k8s, resource/cloud, attributes/k8s_logs_dataset]
Should be
        processors: [batch, k8sattributes, resourcedetection/system, resourcedetection/eks, resourcedetection/gcp, resource/demo, resource/k8s, resource/cloud, attributes/k8s_logs_dataset, attributes/dataset]

@rogercoll Just pinging on this I am getting more interest and this bug is creeping in causing issues, do you want me to create a PR etc?

@rogercoll Just pinging on this I am getting more interest and this bug is creeping in causing issues, do you want me to create a PR etc?

Sorry for my late response, a PR would be great, but I think this attribute decoration is not needed anymore with the otel native data model. I am about to merge this PR that changes the Otel demo deployment within Elastic completely to be Otel native: https://github.com/elastic/opentelemetry-demo/pull/120 (so this decoration won't be needed)


[feature] Remove Blank Profile Objects

Fixes: #2412 and #2384
A resubmission of  #3515
Correct Internal Server Error when Clicking Submit in Profile when there is a blank School or Job object.  Also add exception handling.

After fix.

Blank objects will be ignored when "Submit" is clicked.
The Add Institution button will be disabled if there is a  blank object.  It becomes enabled as soon as any field is filled in.
The "Remove" link will delete the job or school if there is more then one.  If there is only one, then all fields will be cleared and a save is allowed.
If any field is filled in and "Institution" is not, the user will be alerted that Institution is needed. If no fields are filled in, then the save is allowed the blank object is ignored.

The Add Institution button will be disabled if there is a blank object. It becomes enabled as soon as any field is filled in.

We decided a while back that this is an anti-pattern because it doesn't make clear to the user what is wrong with their input. To follow the current pattern, the user should be allowed to click "Add Institution" and receive feedback about what was wrong with their input, if appropriate.

Pass finished. The main things to improve about this approach:

Only valid data must be allowed to reach the server.  This means that the same validation should happen on the server and client side. Partial data (e.g. an "Education" item without the institution specified should not even be sent to the server).
Simplify how validation happens. Notice that you have overridden submit in two subclasses; this is a code "smell"--a hint of bad design (see the open/closed principle). See if you can use built-in browser validation (HTML required) or KO-validation so that isValid() returns false when the data are invalid rather than checking the values in submit.

It seems that the "Institution" field has been made required since this PR has been sent, so I haven't been able to reproduce https://github.com/CenterForOpenScience/osf.io/issues/2412 .

I updated the behavior of the add button so it will give user input on how to move forward.
I also simplified the validation, using knockouts internal validation for institutions using required: onlyIf
Finally, blank objects are not saved on the server.

Pass finished.

I still see a 500 error when submitting an empty form in Employement or Education.

Please do another functional review and address the above bugs. Once that is addressed, I will do another pass.

CR complete - one potential bug identified.

What I thought might have been a bug was not a bug. Good to go :)
:star:

I'll make the final minor changes when I merge.


Explicitly set allValues for hub variable

Currently, on latest grafana, the dashboards will currently show 'no data' because the $hub variable is not set correctly (see https://github.com/2i2c-org/infrastructure/issues/3237 for details). The longer term solution to this is:

Move away from the deprecated graphPanel to timeseries objects for panel
https://github.com/jupyterhub/grafana-dashboards/issues/89

Both of these are being undertaken in
https://github.com/jupyterhub/grafana-dashboards/pull/90, but in the meantime, this PR implements
this
workaround so end users can at least see data.

Is this to be considered a bugfix perhaps?


handle dsps

Handles the IS_x_INVERTED properties by inverting the nets again back to their original state. Let me know what you think @reillymck If this solution isn't viable, then we need to adjust some fasm2bels code instead of our code, because this is the only solution using our code that I've thought of. Also, if you think the solution is fine, but you'd rather I'd format it differently, I can do that as well. I gave dsp's their own function, but it could just be integrated into the main mapping function.

I need to fix this, it's not working right now

This is working now

I'm not sure if we can do this since the property isn't showing up for one of the designs. For all we know fasm2bels is right about the property and vivado is hiding it. Let's hold off on putting this in main for now. But, let's keep using it for now so that these pins pass and we see if there are other things to work on or if this is the only thing holding us back from passing.


Proposal: Allow router to accept query_data array for what data to include in $post

OK so 99% of the time whenever I need a custom filter, it is to one of these scenarios:

Add pages to related
Include "featured" pages/posts on the homepage (or a shop landing page etc)
Add some detail to the site data object (like News page name, URL etc).
Manipulate a post meta data value in someway (like format a date, or add in gallery images or related posts out of a meta field)

Now it seems that some of that (3) would be best as a filter, but (1) and (2) are using the rez_gather_related filter and thus the most dangerous and confusing things to do.
Using rez_gather_related It's super easy to get an infinite loop, or make an insane query that slows down your server. The filters end up looking like a mess of if() statements testing against the dev_ids to see when to load all siblings etc...
So could we make an alternative to the rez_gather_related filter? And it seems to me we could do this in a WordPress way by defining an array of settings on the route (trying to match the wp_query syntax).
Now I'm not exactly sure how these settings get used behind the scenes, like do they go on the loop posts? Perhaps anytime a post with the matching dev_id is queried it factors in the provided settings?
This method would also be backwards compatible, which is nice. You could also opt out of data this way too, making things even faster!
I was thinking something like this in router.php:
    $story_args = array(
        'title'         => true,
        'post_content'  => true,
        'post_excerpt'  => true,
        'relative_path' => true,
        'meta'          => array('all', 'meta_key_1', 'meta_key_2'),
        'attachments'   => array('all', 'image/jpeg', 'image/gif'),
        'related'       => array('parent', 'siblings', 'children', 'next', 'previous'),
        'taxonomies'    => array('all', 'category', 'custom_taxonomy_name')
    );

    $programmed_routes = array(

        path_from_dev_id('stories')     => array(
    		'component' => 'StoriesGrid',
    		'name'      => 'StoriesGridPage',
    		'query_data'      => $story_args
    	)

    );

I think there is also an opportunity to include more things in the $post object by default (all attachments, all ACF content, all meta etc). We have some of this all ready, but it's good to look at it again now that we are down the road a little with Vuepress. Also change the name! LOL!
Then most themes would only need to use the rez_serialize_post or the rez_build_site_data filters, which are way easier to understand and less dangerous.

These are awesome thoughts! Going to go through them some here:
rez_gather_related is definitely a dangerous area for infinite loops, so the more we can avoid it (or the easier we can make it), the better!
I like the query idea you’re suggesting - it looks a lot like a GraphQL query like @lwaldner mentioned in Slack. I don’t think router.php would be the place for a server-side query, though - $programmed_routes maps directly to the routes object here, which is used to build the router client-side, so any extra data you’d be requesting would end up being fetched client-side.
If we were okay with that slight delay that comes with fetching data client-side, we could use route meta fields to do something like this:
$story_args = array(
        	'title'         => true,
	// …etc
);

$programmed_routes = array(

	path_from_dev_id('stories')     => array(
		‘component' => 'StoriesGrid',
		'name'      => 'StoriesGridPage',
		‘meta’		=> array(
			'query_data'      => $story_args
		)
	)

);

and write a handler that fetches the data specified in query_data on load.
That doesn’t actually solve the issue, though, since Rest-Easy filters prepare content server-side instead of putting the work on the client.
Maybe it’d be more robust to add a widget to posts that lets you check additional data you’d want attached to the post. We could then add a filter in Rest-Easy core that checks for the values you’ve selected and adds them whenever serializing that particular post.
We should definitely make sure we’re getting everything we want from the default $post object - I’ll raise a new issue where we can make and complete a list!


Tracking Issue: Reduce RSS for development environments

https://github.com/tigerbeetle/tigerbeetle/pull/1882
https://github.com/tigerbeetle/tigerbeetle/pull/1880
https://github.com/tigerbeetle/tigerbeetle/pull/1883

https://github.com/tigerbeetle/tigerbeetle/pull/1981

https://github.com/tigerbeetle/tigerbeetle/pull/2009

https://github.com/tigerbeetle/tigerbeetle/pull/2010

Closing this for now -- the RAM of a development replica is now down to ~660MiB (for a single-node cluster).
All of the above improvements will be part of the next release -- v0.15.4.

Awesome work @sentientwaffle ! You truly live up to the motto that "not only TigerBeetle is very fast, it is also small"!


[tool_crash] FileSystemException: Cannot create link, OS Error: Operation not permitted, errno = 1

Command
flutter pub add path_provider

Logs
FileSystemException: Cannot create link, OS Error: Operation not permitted, errno = 1
#0      _Link.throwIfError (dart:io/link.dart:254:7)
#1      _Link.createSync (dart:io/link.dart:182:5)
#2      ForwardingLink.createSync (package:file/src/forwarding/forwarding_link.dart:20:16)
#3      ForwardingLink.createSync (package:file/src/forwarding/forwarding_link.dart:20:16)
#4      _createPlatformPluginSymlinks (package:flutter_tools/src/flutter_plugins.dart:1055:12)
#5      createPluginSymlinks (package:flutter_tools/src/flutter_plugins.dart:985:5)
#6      refreshPluginsList (package:flutter_tools/src/flutter_plugins.dart:1087:5)
<asynchronous suspension>
#7      FlutterProject.ensureReadyForPlatformSpecificTooling (package:flutter_tools/src/project.dart:355:5)
<asynchronous suspension>
#8      PackagesGetCommand.runCommand (package:flutter_tools/src/commands/packages.dart:326:7)
<asynchronous suspension>
#9      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1336:27)
<asynchronous suspension>
#10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#11     CommandRunner.runCommand (package:args/command_runner.dart:212:13)
<asynchronous suspension>
#12     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:296:9)
<asynchronous suspension>
#13     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#14     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:244:5)
<asynchronous suspension>
#15     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:91:9)

[✓] Flutter (Channel stable, 3.10.5, on Ubuntu 22.04.2 LTS 5.15.0-75-generic, locale en_US.UTF-8)
    • Flutter version 3.10.5 on channel stable at /home/st/snap/flutter/common/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 796c8ef792 (6 days ago), 2023-06-13 15:51:02 -0700
    • Engine revision 45f6e00911
    • Dart version 3.0.5
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /home/st/Android/Sdk
    • Platform android-33, build-tools 33.0.0
    • ANDROID_SDK_ROOT = /home/st/Android/Sdk
    • Java binary at: /home/st/Android/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 4.1)
    • Android Studio at /home/st/work/mdevelop/android-studio
    • Flutter plugin version 57.0.1
    • Dart plugin version 201.9380
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] Android Studio (version 2022.2)
    • Android Studio at /home/st/Android/android-studio
    • Flutter plugin version 73.0.2
    • Dart plugin version 222.4560
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code
    • VS Code at /snap/code/current
    • Flutter extension version 3.66.0

[✓] Connected device (3 available)
    • M2101K7BNY (mobile) • QC9HJRTK6PEU55PV • android-arm64  • Android 13 (API 33)
    • Linux (desktop)     • linux            • linux-x64      • Ubuntu 22.04.2 LTS 5.15.0-75-generic
    • Chrome (web)        • chrome           • web-javascript • Google Chrome 98.0.4758.102

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 9cd3d0d9ff05768afa249e036acc66e8abe93bff
Plugins
path_provider-2.0.15
path_provider_android-2.0.27
path_provider_foundation-2.2.3
path_provider_linux-2.1.11
path_provider_windows-2.1.7

Hi @dam-ease. The project was on an external disk with the file system exfat. I moved the project to my linux home directory and the problem disappeared. Probably, i have problems with the disk. I'm sorry, i didn't see it myself.
Thanks for your reply.


feat/807

jambonz/jambonz-feature-server#807
Introducing minConfidence property for gather verb

I'm not sure I agree with the need for this feature.  Why can't the client filter transcripts as desired based on confidence level?  How does this add any significant value to the product?

in Microsoft in single session its send low confidence to bot  confidence level below 0.1 , hence threshold value can be set to receive the confidence level


Coral-Spark: Migrate 'CAST(ROW: RECORD_TYPE)' and 'extract_union' transformations from RelNode to SqlNode layer

Similar to #351, this PR migrates CAST(ROW: RECORD_TYPE) and extract_union transformations from RelNode layer in IRRelToSparkRelTransformer to SqlNode layer.
In order to avoid creating a PR that is too big, I will open several more PRs to finish migrating all the transformations in IRRelToSparkRelTransformer to SqlNode layer and remove IRRelToSparkRelTransformer.
Tests:

Existing unit tests, which already cover the changes
Regression test
Querying several complex production views on gateway, no regression

@aastha25 Thanks for your review! Modified these 2 comments based on your suggestions. PTAL.


UI issue, running notification shifting top-bottom

ISSUE TYPE

Bug Report

COMPONENT NAME

UI

CLOUDSTACK VERSION

4.16

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

When having a in-progress task in the notification drop down, the tasks is shifting top to bottom with any other items in the menu.
Checkout the gif:

Correct @DaanHoogland , not sure about the origin though..

@borisstoyanov this may be fixed by #7609

@borisstoyanov this may be fixed by #7609

will test it out, thanks!

fixed by #7609


Binary can't be used in Apache 2.2.29 on Windows

I have an old system running Apache 2.2.29 (Win64) on Windows Server 2012. The pre-compiled module will cause error while starting Apache. The message reads:
httpd: Syntax error on line 508 of D:/Apache/conf/httpd.conf: API module structure 'cspnonce_module' in file D:/Apache/modules/mod_cspnonce.so is garbled - expected signature 41503232 but saw 41503234 - perhaps this is not an Apache module DSO, or was compiled for a different Apache version?
What should I do? If I have to compile it by myself, is there any options I need to change?
Regards,
Gary

https://github.com/wyday/mod_cspnonce/pull/4#issuecomment-905826015


[jdbc][jdbc parameter] Muilt Table Sync Use same jdbc connection properties

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
The database connection information in the JDBC source table & sink table needs to be written repeatedly, and cannot be configured only once
suggest Solution: extract it into env module for unified definition or external parameter reference
Usage Scenario
When synchronizing multiple tables, avoid repeatedly writing database connection information, and configure it only once
eg :
env {
spark.app.name = "sync_base_tables"
spark.executor.instances = 1
spark.executor.cores = 1
spark.executor.memory = "1g"
}
jdbcconfig {
driver = "com.mysql.jdbc.Driver"
url = "jdbc:mysql://<IP_ADDRESS>:3306/testdb"
user = "root"
password = "123456"
}
xxxconfig{
......
}
source {
jdbc {
 ${jdbcconfig}
table = "location"
result_table_name = "location"
jdbc.fetchsize = 500
jdbc.batchsize = 500
}
jdbc {
${jdbcconfig}
table = "subsystem"
result_table_name = "subsystem"
jdbc.fetchsize = 500
jdbc.batchsize = 500
}
jdbc {
${jdbcconfig}
table = "me_alarm_type"
result_table_name = "tablec"
jdbc.fetchsize = 500
jdbc.batchsize = 500
}
}
.......
Related issues
there is not another issue associated with this?
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This function is very similar to Hocon's Substitutions. You can refer to SeaTunnel's existing Config library to see if it can be implemented directly.


Travis CI: switch to dist: trusty

Test .travis.yml:

[ ] https://github.com/ruslo/polly
[ ] https://github.com/forexample/hunter-simple
[ ] https://github.com/hunter-packages/gate
[ ] https://github.com/travis-ci-tester/toolchain-table

Create/find docker image:

[x] https://github.com/hunter-packages/travis-linux-docker
[ ] Update section: https://github.com/hunter-packages/travis-linux-docker#testing

Test some packages:

[ ] GTest (simple, all toolchains)
[ ] OpenCV (CMake-based, big package with dependencies)
[ ] Qt (upload on Travis doesn't work because of timeouts, testing of Docker image)

@chfast For your info I got problems with Python on new Trusty image:
> easy_install3 --user requests
> python3 "${POLLY_SOURCE_DIR}/bin/install-ci-dependencies.py"
...
ImportError: No module named 'requests'

Example:

https://travis-ci.org/ruslo/polly/builds/214181910

Trusty should have pip. I've seen you used easy_install because of the lack of pip.

Trusty should have pip

Yep, did the fix, thanks.
Current problem is that sanitize-thread toolchain is not working:

https://travis-ci.org/ruslo/polly/builds/214269616

Executable produce error:
FATAL: ThreadSanitizer can not mmap the shadow memory (something is mapped at 0x5572fc6cd000 < 0x7cf000000000)

Suggested fix from documentation:
> echo 2 >/proc/sys/kernel/randomize_va_space

Bug we have to run it with sudo which is disabled :(

Current problem is that sanitize-thread toolchain is not working

To clarify, the error is runtime. So we build/link fine but we can't run tests. It's bad if we lose toolchain by moving from 12.04 to 14.04 but I like the fact that we don't have to install a lot of stuff. This reduce the build time from ~3 minutes to ~40 seconds:

https://travis-ci.org/ruslo/polly/builds/212960859
https://travis-ci.org/ruslo/polly/builds/214269616

The issue with thread sanitizer is probably doe to Ubuntu upgrade. I have do the same on my local machine. Maybe asking Travis to enable that kernel option by default in their non-sudo images?

Maybe asking Travis to enable that kernel option by default in their non-sudo images?

I guess as a last resort. Will spend some time looking for solution, may be we can do it without sudo.

Trying to build docker image for finding solution to current problems:

https://github.com/travis-ci/travis-ci/issues/7518

Trying to build docker image for finding solution to current problems

Image is ready, instructions updated:

https://github.com/hunter-packages/travis-linux-docker

The issue with thread sanitizer is probably doe to Ubuntu upgrade. I have do the same on my local machine. Maybe asking Travis to enable that kernel option by default in their non-sudo images?

It is set to 2 in docker container but still doesn't work...
Related links:

https://bugzilla.kernel.org/show_bug.cgi?id=66721
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59410
https://github.com/google/sanitizers/issues/503

So there are kernel fixes and compiler fixes. On my local Ubuntu 16.04 machine I got similar error, I can "fixed" it by using Clang 3.8 without PIE, but Clang 3.5 (which is used on Travis CI) doesn't support ThreadSanitizer without PIE.
So I'm planning to give up ThreadSanitizer toolchain for now. Don't want to make the .travis.yml too complex by adding instructions for installing Clang 3.8+ just for ThreadSanitizer. Using custom compilers should be manageable by Hunter itself in future and discussed here:

https://github.com/ruslo/hunter/issues/495

Polly updated:

https://github.com/ruslo/polly/pull/131

Almost ready. Hit this one now:

https://github.com/travis-ci/travis-ci/issues/7558

We need it for Qt+Android:

https://travis-ci.org/ingenue/hunter/builds/217670290
http://superuser.com/a/360398/252568

Almost ready. Hit this one now: travis-ci/travis-ci#7558

Removing APK creating from this test for now, C++ build still remains.
Everything ready, closing. Most of the packages still use Ubuntu 12.04 in .travis.yml, will be updated one-by-one on demand.


update development db and migrations

Update development database and migrations for most recent pull to develop

@hyi @pkdash @ironfroggy - a quick +1 would be appreciated to bring in the updated db and migrations file for the dev environment

@mjstealey the updated db is too large to be viewed with change differences on github. Is the difference mainly on bringing in extra hs_core component permissions to HydroShare Author group? If so,I don't see an issue with it.

@hyi - yep, updated db to include newly generated migration and bring hs_core components into Hydroshare Author group.
Won't get flagged that there are missing migrations that require a makemigrations call on deploy or have mezzanine complain due to missing group permissions.


I'm just demoing how to do a PR

Write a useful description here.

No need to sound so energetic @pbitutsky

Hi


Netflix and Kodi crash a few seconds after having started a video

Bug report
Netflix and Kodi crash a few seconds after having started a video
Your Environment

Netflix add-on version: 0.16.4
Operating system version/name: LibreELEC 9.2.0 (Kodi 18.5)
Device model:  Raspberry Pi 3B

Used Operating system:

[ ] Android
[ ] iOS
[ ] Linux
[ ] OSX
[X] Raspberry-Pi
[ ] Windows

Describe the bug
When starting a video, it plays for a few seconds and then Netflix crashes along with Kodi. Kodi restarts.
Expected behavior
The video should play.
Actual behavior
Netflix crashes and Kodi restarts.
Steps to reproduce the behavior
After installing using the method in https://github.com/CastagnaIT/plugin.video.netflix/wiki/How-install-the-addon,

Login on Netflix add-on
Play any video with subs enabled (for me it was Star Trek: Discovery, Season 2 episode 1)

Possible fix
Debug log
Link for debug log:
https://paste.ubuntu.com/p/xdr7wjzzTg/
Installation

[X] I used CastagnaIT repo to install Netflix.
[ ] I'm using other Netflix Repo
[ ] I'm using a different source

Other information
I have tried to uninstall and reinstall  Netflix, Inputstream and Inputstream Helper add-ons, and also to reset CDM / Widevine. Also, because default subtitle language is Greek for my installation, I tried reverting this to the default language and disabling subs altogether. None of these actions corrected the problem. I read the topic https://github.com/CastagnaIT/plugin.video.netflix/issues/8 but it does not seem to bee the case.

In the log there are no unexpected errors
this seems more like a problem of your Kodi installation or with the device

Well, I removed 0.16.4 and installed version 1.0.0 and I also de-activated "Ask to skip Intro/recap" and now it works.
Sometimes I get "AddonSignals call timed out" though...!

@akirikos
Turn on "Enable IPC over HTTP" in expert settings and check... it helped me.

it's particularly strange what you say
if that was the real problem of "Ask to skip Intro/recap", it would happen to everyone
but that part of code has never been changed

Yeah, well, the whole thing was strange... I will try to switch on "skip intro/recap" again and post here again. The "AddonSignals call timed out" has disappeared without changing anything.

UPDATE: I can confirm that enabling "ask to skip intro/recap" in Playback settings causes Kodi to crash. When it is disabled, the plugin works fine.

If it is Kodi to be crashed it means it's something wrong in Kodi core
i can't do anything about it
if instead a verifiable error happens in the log, then we can figure out what to do
your case is unique and difficult to understand because the code "ask to skip intro/recap" doesn't do anything in particular
you can try to do last two things:
-update libreelec to latest version
-download and install netflix addon Daily Build (link in readme)

closed due to inactivity


Converted type is None according to Parquet Tools then utilizing logical types

Describe the bug

This regards the output written by the parquet crate. Declaring a column to containt a timestamp of microseconds using a LogicalType causes the written file to not have a converted type. At least according to parquet-tools.
To Reproduce

Write a file tmp.par with a single column with type Timestamp of Microseconds, using a logical type.

use std::sync::Arc;

use parquet::{
    basic::{LogicalType, Repetition, Type},
    data_type::Int64Type,
    file::{properties::WriterProperties, writer::SerializedFileWriter},
    format::{MicroSeconds, TimeUnit},
    schema::types,
};

fn main() {
    let mut data = Vec::with_capacity(1024);
    let logical_type = LogicalType::Timestamp {
        is_adjusted_to_u_t_c: false,
        unit: TimeUnit::MICROS(MicroSeconds {}),
    };
    let field = Arc::new(
        types::Type::primitive_type_builder("col1", Type::INT64)
            .with_logical_type(Some(logical_type))
            .with_repetition(Repetition::REQUIRED)
            .build()
            .unwrap(),
    );
    let schema = Arc::new(
        types::Type::group_type_builder("schema")
            .with_fields(&mut vec![field])
            .build()
            .unwrap(),
    );

    // Write data
    let props = Arc::new(WriterProperties::builder().build());
    let mut writer = SerializedFileWriter::new(&mut data, schema, props).unwrap();
    let mut row_group_writer = writer.next_row_group().unwrap();
    let mut column_writer = row_group_writer.next_column().unwrap().unwrap();
    column_writer
        .typed::<Int64Type>()
        .write_batch(&[1, 2, 3, 4], None, None)
        .unwrap();
    column_writer.close().unwrap();
    row_group_writer.close().unwrap();
    writer.close().unwrap();

    // Write file for inspection with parqute tools
    std::fs::write("tmp.par", data).unwrap();
}

Install parquet-tools in a virtual environment and inspect the file

pip install parquet-tools==0.2.11
parquet-tools inspect tmp.par

The resulting output indicates no Converted type
############ file meta data ############
created_by: parquet-rs version 26.0.0
num_columns: 1
num_rows: 4
num_row_groups: 1
format_version: 1.0
serialized_size: 143

############ Columns ############
col1

############ Column(col1) ############
name: col1
path: col1
max_definition_level: 0
max_repetition_level: 0
physical_type: INT64
logical_type: Timestamp(isAdjustedToUTC=false, timeUnit=microseconds, is_from_converted_type=false, force_set_converted_type=false)
converted_type (legacy): NONE
compression: UNCOMPRESSED (space_saved: 0%)

Expected behavior

I would have expected the converted type to show up in the Metainformation emitted by parquet-tools.
Additional context

Triggered by upstream odbc2parquet issue https://github.com/pacman82/odbc2parquet/issues/284. Azure can not seem to handle the output since migration to LogicalType.
Previously misdiagnosed this to not set the converted type correctly in the schema information, this however does happen. See: #2984.
Thanks any help is appreciated!

I've narrowed this down to pyarrow not being able to read the converted type correctly.
>>> import pyarrow.parquet as pq
>>> pq.ParquetFile('tmp.par').schema.column(0).converted_type
'NONE'

However, fastparquet is able to read the converted type, as it is correctly encoded in the thrift definition
>>> ParquetFile('tmp.par').schema.schema_element('col1').converted_type
10

I also tried https://github.com/xitongsys/parquet-go/tree/master/tool/parquet-tools, which resulted in
./parquet-tools -cmd schema -file /home/raphael/repos/external/arrow-rs/parquet/tmp.par
{
  "Tag": "name=Schema, repetitiontype=REQUIRED",
  "Fields": [
    {
      "Tag": "name=Col1, type=INT64, convertedtype=TIMESTAMP_MICROS, repetitiontype=REQUIRED"
    }
  ]
}

This leads me to think that this is actually a bug in the pyarrow, and therefore the C++ arrow implementation. Perhaps you might like to raise a bug there?

Hi thanks for investigating this, so in context of the steps to reproduce you think the bug lies with parquet-tools ? I mean could be. However here is the output parquet-tools (the one installed via pip) gives me if I set the converted type instead of the logical type in code:
created_by: parquet-rs version 26.0.0
num_columns: 1
num_rows: 4
num_row_groups: 1
format_version: 1.0
serialized_size: 134

############ Columns ############
col1

############ Column(col1) ############
name: col1
path: col1
max_definition_level: 0
max_repetition_level: 0
physical_type: INT64
logical_type: Timestamp(isAdjustedToUTC=true, timeUnit=microseconds, is_from_converted_type=true, force_set_converted_type=false)
converted_type (legacy): TIMESTAMP_MICROS
compression: UNCOMPRESSED (space_saved: 0%)

Hey, I found a (for me at least) new thing: If I set is_adjusted_to_u_t_c to true it works with the logical type. So the converted type is not written if is_adjusted_to_u_t_c is false, but if it is true it is. Is this intentional?

So the converted type is not written if is_adjusted_to_u_t_c is false, but if it is true it is. Is this intentional?

I can state categorically the converted type is always being written to the thrift metadata payload in the parquet file. As shown above readers not based on arrow C++ correctly find and read it, and as shown previously manually decoding the the metadata shows it is there.  You will need to take this up with them.

Hello @tustvold ,
thanks for investigating. I will close the ticket. And open a ticken in apache/arrow. Thanks for everything.
Cheers, Markus


balancer gateway connector

Before submitting this PR, please make sure:

[ X] Your code builds clean without any errors or warnings
[ X] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:

Add Balancer gateway connector and manual curl tests
Add schema for Balancerconfig file
Add dependency on @balancer-labs/sdk

Tests performed by the developer:
Tips for QA testing:

@williamstarkro are you still interested in working on the Balancer connector bounty? If so, please let me know ASAP.
You would need to create the connector in the new standalone Gateway repo https://github.com/hummingbot/gateway

I'll take this


Load scenes rather than robot files for MJCF descriptions

When loading a robot description from MuJoCo Menagerie, it seens it is more common to load the scene.xml files for each description rather than the individual robot files (that have, notably, no ground).
The proposal is then to make all MJCF descriptions loaded via robot_descriptions go for scene.xml by default, with optionally a variant https://github.com/robot-descriptions/robot_descriptions.py/issues/62 for bare robots.

Thank you both for your feedback! Then it seems the best way forward is to allow it but not make it the default.


Merge release 1.7 into master

Creating this PR but it's not ready to merge yet.

The doc bot isn't building this. Does it need to be retargeted into main? Did we accidentally mix up a bunch of PR commits that should have gone into main?


List of eks failing tests

See https://github.com/cilium/cilium/pull/9682 for how to run these
See https://github.com/cilium/cilium/pull/9675 for fixups and hacks

[ ] [Fail] K8sHealthTest [BeforeEach] checks cilium-health status between nodes
[TODO] Skip on eks because endpoint-endpoint probe won't work with chaining

[ ] [Fail] K8sFQDNTest [BeforeEach] Restart Cilium validate that FQDN is still working
[TODO] Hardcoded DNS bind service IP is wrong for this cluster. Need to switch to a template.

[ ] [Fail] K8sFQDNTest [BeforeEach] Validate that multiple specs are working correctly
[TODO] Hardcoded DNS bind service IP is wrong for this cluster. Need to switch to a template.

[ ] [Fail] K8sUpdates [It] Tests upgrade and downgrade from a Cilium stable image to master
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/Updates.go:194

[ ] [Fail] K8sKafkaPolicyTest Kafka Policy Tests [It] KafkaPolicies
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/KafkaPolicies.go:237

[ ] [Fail] K8sIstioTest [BeforeEach] Istio Bookinfo Demo Tests bookinfo inter-service connectivity
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/istio.go:109

[ ] [Fail] K8sServicesTest Checks ClusterIP Connectivity [It] Checks service on same node
fix w/ GetNodeNames

[ ] [Fail] K8sServicesTest Checks service across nodes [It] Tests NodePort (kube-proxy)
[TODO] Hardcoded DNS bind service IP is wrong for this cluster. Need to switch to a template.
Passes with fixed DNS IP

[ ] [Fail] K8sServicesTest Checks service across nodes with L7 policy [It] Tests NodePort with L7 Policy
Fails on curls to cilium host internal IP and remote IP.  test/k8sT/services.go:286

[ ] [Fail] K8sServicesTest Bookinfo Demo [It] Tests bookinfo demo
503 Service Unavailable probably because envoy isn’t able to upstream

[x] [Fail] K8sDatapathConfig MonitorAggregation [It] Checks that monitor aggregation restricts notifications
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/DatapathConfiguration.go:127

[ ] [Fail] K8sDatapathConfig Encapsulation [It] Check connectivity with sockops and VXLAN encapsulation
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/DatapathConfiguration.go:168

[ ] [Fail] K8sDatapathConfig Encapsulation [It] Check connectivity with VXLAN encapsulation
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/DatapathConfiguration.go:168

[ ] [Fail] K8sDatapathConfig Encapsulation [It] Check connectivity with Geneve encapsulation
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/DatapathConfiguration.go:168

[ ] [Fail] K8sDatapathConfig Transparent encryption DirectRouting [It] Check connectivity with transparent encryption and direct routing
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/assertionHelpers.go:135

[ ] [Fail] K8sDatapathConfig IPv4Only [It] Check connectivity with IPv6 disabled
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/DatapathConfiguration.go:262

[x] [Fail] K8sDatapathConfig ManagedEtcd [It] Check connectivity with managed etcd
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/assertionHelpers.go:135

[ ] [Fail] K8sPolicyTest Basic Test [It] checks all kind of Kubernetes policies
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/Policies.go:269
[TODO] ingress proxy - skip

[ ] [Fail] K8sPolicyTest Basic Test [It] CNP test MatchExpressions key
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/Policies.go:355
[TODO] ingress proxy - skip

[ ] [Fail] K8sPolicyTest Basic Test Validate CNP update [It] Enforces connectivity correctly when the same L3/L4 CNP is updated
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/Policies.go:674
[TODO] Hardcoded DNS bind service IP is wrong for this cluster. Need to switch to a template.
Passes with fixed DNS IP

[ ] [Fail] K8sPolicyTest Basic Test Redirects traffic to proxy when no policy is applied with proxy-visibility annotation [BeforeEach] Tests HTTP proxy visibility without policy
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/Policies.go:786
[TODO] Hardcoded DNS bind service IP is wrong for this cluster. Need to switch to a template.
Passes with fixed DNS IP

[ ] [Fail] K8sPolicyTest Basic Test Redirects traffic to proxy when no policy is applied with proxy-visibility annotation [BeforeEach] Tests DNS proxy visibility without policy
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:539
[TODO] Hardcoded DNS bind service IP is wrong for this cluster. Need to switch to a template.
Passes with fixed DNS IP

[ ] [Fail] K8sPolicyTest Basic Test Redirects traffic to proxy when no policy is applied with proxy-visibility annotation [BeforeEach] Tests proxy visibility interactions with policy lifecycle operations
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:539
[TODO] Hardcoded DNS bind service IP is wrong for this cluster. Need to switch to a template.
Passes with fixed DNS IP

[x] [Fail] K8sPolicyTest GuestBook Examples [It] checks policy example
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/Policies.go:1019

[x] [Fail] K8sPolicyTest Namespaces policies [BeforeEach] Tests the same Policy in different namespaces
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/Policies.go:1113

[x] [Fail] K8sPolicyTest Namespaces policies [BeforeEach] Kubernetes Network Policy by namespace selector
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:539

[ ] [Fail] K8sPolicyTest Namespaces policies [BeforeEach] Cilium Network policy using namespace label and L7
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:539
[TODO] ingress proxy - skip

[x] [Fail] K8sPolicyTest Clusterwide policies [BeforeEach] Test clusterwide connectivity with policies
/Users/ray/covalent/gopath/src/github.com/cilium/cilium/test/k8sT/Policies.go:1324

I took a new stab at this and the following tests are the ones left failing:

K8sConformance

Check connectivity-check compliance with portmap chaining
Check one node connectivity-check compliance with portmap chaining

K8sDatapathConfig

Checks that monitor aggregation restricts notifications
Check connectivity with transparent encryption and direct routing
Check connectivity with transparent encryption and direct routing with bpf_host

K8sFQDNTest

Validate that multiple specs are working correctly

K8sHubbleTest -- I could not test this because the hubble image was defaulting to the local k8s registry that gets set up in Jenkins.

Test L3/L4 Flow
Test TLS certificate
Test L3/L4 Flow with hubble-relay
Test L7 Flow
Test L7 Flow with hubble-relay

K8sPolicyTest

Denies traffic with cnp default-deny ingress policy
Allows traffic with k8s default-allow egress policy
Validate toEntities All
Validate toEntities World
Tests DNS proxy visibility without policy

K8sUpdates -- also failed because some weird docker image formatting.

Tests upgrade and downgrade from a Cilium stable image to master

Tagging @joestringer since we talked about this on Slack a few weeks ago.

I took a new stab at this and the following tests are the ones left failing (at commit 9c0939a):
* `K8sDatapathConfig`

  * `Check connectivity with transparent encryption and direct routing`
  * `Check connectivity with transparent encryption and direct routing with bpf_host`

Could they be failing for the same reason they were failing on GKE? See https://github.com/cilium/cilium/commit/e0fba7473bc4d03071ddc9e857f5e2bb91fbb3a3.

I took a new stab at this and the following tests are the ones left failing (at commit 9c0939a):
* `K8sDatapathConfig`

  * `Check connectivity with transparent encryption and direct routing`
  * `Check connectivity with transparent encryption and direct routing with bpf_host`

Could they be failing for the same reason they were failing on GKE? See https://github.com/cilium/cilium/commit/e0fba7473bc4d03071ddc9e857f5e2bb91fbb3a3.

I tried the tests with direct routing disabled so we test just the encryption functionality and the thing I found out is that the connectivity check is failing due to Cilium on the server side is detecting the packets as being received from the identity remote-node rather than the client side pod. After that I disabled masquerading because that it's enabled by default in the ENI Helm integration but I had no success (same failure mode).

I tried the tests with direct routing disabled so we test just the encryption functionality and the thing I found out is that the connectivity check is failing due to Cilium on the server side is detecting the packets as being received from the identity remote-node rather than the client side pod. After that I disabled masquerading because that it's enabled by default in the ENI Helm integration but I had no success (same failure mode).

Is <IP_ADDRESS> actually a remote node IP or it is the IP of the client pod? If it's the IP of the remote node, that shouldn't happen because masquerading is disabled, right? Happy to help debug this via Slack if that helps.

Is <IP_ADDRESS> actually a remote node IP or it is the IP of the client pod? If it's the IP of the remote node, that shouldn't happen because masquerading is disabled, right? Happy to help debug this via Slack if that helps.


How to find first token latency on TensorRT LLM for llama model?

Hi, I have a use case where I want to report the first token latency on Jetson ORIN for different set of context length and generation for benchmarking. I am working on the release/0.5.0  branch for TRT-LLM with GPTQ, and I wanted to know how to report first token latency. I am using two ways-

In runtime/generation.py   measuring the latency of the step self.runtime._run(context, stream)  which I assume would be the GPU compute time
In runtime/generation.py   measuring the latency of self.handle_per_step  measuring the latency per step.

Questions-

I see the numbers for 2 is greater than 1. Which would be the correct to report?
Apart from the above two steps, the decoder.decode has steps like setting up buffer, the KV cache, setting up generation sequence. Should these also be included in the first token latency?
Also on ORIN in TRT-LLM with GPTQ, I see setting up the decoder in benchmarks/python/gpt_benchmark.py  in decoder=tensorrt_llm.runtime.GenerationSession  with prebuilt engine takes around 20-25 mins which I did not see in TRT-LLM weight only quantization. Is this expected?

Also interested in this.

You can refer to https://github.com/NVIDIA/TensorRT-LLM/blob/main/docs/source/performance.md#first-token-latency-benchmark-1

Thanks for the pointer. I had a doubt. My understanding is that the latency would be the whole end to end generating steps time taken, which would include TRT engine inference + decoding + cache setup. So would not the time taken for TRT engine inference or the cache setup be dependent on the generation length sequence? i.e. it would be different for inlen, outlen 20481 vs 20482048?

cache setup is only related to memory allocation, which would be covered in warmup and it does not affect generation. So, the output len will not affect the perf. In short, setting output len as 1 is a correct setting.


Unable to use flashtext to remove keywords

kwp = KeywordProcessor()
kwp.add_keyword('
', '')
kwp.replace('one
two')

As a result I get an unchanged string. Is there a way to use flashtext to remove keywords from the text?

in the doc you can find something like this:

Characters that will determine if the word is continuing.
Defaults to set([A-Za-z0-9_])

special character is not supported at the moment

you can also use the add_non_boundary method to add you special chars like this. also make sur to give a not empty clean name. instead of kwp.add_keyword('
', '') give somthing else as clean name like kwp.add_keyword('
', 'test') this work for me see the example below
kwp = KeywordProcessor()
kwp.add_non_word_boundary('<')
kwp.add_non_word_boundary('>')
kwp.add_keyword('
', 'test')
kwp.replace('one
two')

results: 'one test two'

also make sur to give a not empty clean name
But that's not what I want. I want to have result 'onetwo', not 'one something two'. Or at least 'one two'.


add-tr-language

@mikropsoft thanks for adding support for turkish! Will be included within the next release 😄


fix: neo button text conflicts

ref #8852
Screenshot 📸

[x] My fix has changed UI

@shashkovdanil seems there are some conflicts between neobutton --k-accent and --rounded
https://github.com/kodadot/nft-gallery/pull/8838#issuecomment-1875331761

thanks!
pay 20 usd
yolo


MFA confirmation with Authenticator App is broken

When I try to call sign in for a user that has MFA set on the Authenticator App, the call returns an AuthError, citing its an "UnSupported challenge response softwareTokenMfa"

Broken on the new v2.1.0 but works fine on the older v1.28.3.

Hey guys, is there an ETA on this bug?

@rpatel8del
Can you share the code snippet that you are using for softwareTokenMfa. It seems that signing in software token MFA is not supported in 1.x and 2.x. If you are able to successfully confirm sign in with softwareTokenMfa in 1.28.3, I would like to understand how you are parsing the sign in response. Because ATM sign in only returns one of the following as next steps.
public enum AuthSignInStep {

    /// Auth step is SMS multi factor authentication.
    ///
    /// Confirmation code for the MFA will be send to the provided SMS.
    case confirmSignInWithSMSMFACode(AuthCodeDeliveryDetails, AdditionalInfo?)

    /// Auth step is in a custom challenge depending on the plugin.
    ///
    case confirmSignInWithCustomChallenge(AdditionalInfo?)

    /// Auth step required the user to give a new password.
    ///
    case confirmSignInWithNewPassword(AdditionalInfo?)

    /// Auth step required the user to change their password.
    ///
    case resetPassword(AdditionalInfo?)

    /// Auth step that required the user to be confirmed
    ///
    case confirmSignUp(AdditionalInfo?)

    /// There is no next step and the signIn flow is complete
    ///
    case done
}

Thank you for responding. :)
In 1.x case confirmSignInWithSMSMFACode(AuthCodeDeliveryDetails, AdditionalInfo?) handles SMS and softwareToken whereas in 2.x it returns an AuthError.

I am still unsure how 1.x can handle the signIn response, because the code path doesn't have softwareTokenMfa as an implemented case.
extension SignInResult {

    func toAmplifyAuthSignInStep() throws -> AuthSignInStep {
        switch signInState {
        case .signedIn:
            return .done
        case .smsMFA:
            let deliveryDetails = AuthCodeDeliveryDetails(destination: .sms(codeDetails?.destination))
            return .confirmSignInWithSMSMFACode(deliveryDetails, parameters)
        case .customChallenge:
            return .confirmSignInWithCustomChallenge(parameters)
        case .newPasswordRequired:
            return .confirmSignInWithNewPassword(parameters)
        default:
            throw (AuthError.unknown("AWSMobileClient auth state is not handled"))
        }
    }
}

Can you share your implementation details? So that I can try to reproduce on my end.

Can you enable verbose logging and share the request and response for sign in redacted any sensitive information.
            AWSDDLog.sharedInstance.add(AWSDDTTYLogger())
            AWSDDLog.sharedInstance.logLevel = .verbose

and you would need to import AWSCore
import AWSCore

I would like to understand the response in more detail.

Can I email you the response? Don't want to accidentally post anything here that I should've redacted and missed. :)

Can you post this on our discord channel and we can have follow up conversations there to further investigate into this matter?

Yup, that works.

When will this fix be released?

We are actively working on providing TOTP support for Amplify v2, closing this ticket as duplicate. Please follow https://github.com/aws-amplify/amplify-swift/issues/938 for future updates.


Send telemetry when creating container image registry service accounts and subscription activation keys

There should be telemetry send after tha API calls to create:

container registry service account
subscription activation key

Telemetry data should indicate if the call was successful or failed
Related to #35.

this is unclear.

@slemeur What telemetry should be sent from extension then?

Things we want to know:

Installation of the extension
Login and logout
Errors in the flow (e.g., when service-account creation fails or sub-man fails or logging into the registry, etc.).  Helps us understand where things break and identify those more easily.


Fixing the output method & add examples

[x] All the spiders are working well.

Please someone accept the PR, I urgently need the change to start scraping some newspapers notes 🙏

@RayverAimar The PR will be merged soon, but here's a pro tip for you: you can git checkout into the branch, and you can use it without the need to wait for it to be merged 👍🏻


MonitorPlugins fires a change event every frame

Aside from the event itself being rendered useless, this also greatly spams the verbose logs

should be fixed now!


fix: remove deprecated functions from the jest object

Summary
Been deprecated for a long time, let's remove them

Test plan
Green CI

Let's do it in 27. I'd prefer 26 not to require any heavy codmodding, but I am fine doing it right after.


Enhancement: Add support for uploading packages to Amazon S3

I'm a big fan of using Amazon S3 as the primary distribution point in my JSS, and not having JSSImporter work with it natively is a real bummer.

Is that a CDP? I'm guessing no, since CDP's work. The CDP I tested with used Amazon, so help me understand how this could happen!

It is a CDP... I was under the impression that it wouldn't work since there's no configuration options for it in AutoPkgr.

Ah, I see what you're getting at. Autopkgr hasn't added the settings in.
If you set up a JDS in autopkgr, you're 90% there. Then go in and edit the prefs so that instead of type: JDS it's CDP.
Don't forget to defaults read afterwards!

would you provide more details on changing the type to CDP? Not sure where/how to find those preferences in order to change them...thanks!

@tim-soulen
The preferences file for AutoPkg is at /Library/Preferences/com.github.autopkg.plist, and the relevant info is here: https://github.com/sheagcraig/JSSImporter#cdp-and-jds-cloud-distribution-point-and-jamf-distribution-servers


fingerprint 2019 vw jetta sel

What’s the route this fingerprint came from?

Hello
186c80a9e18ac816|2021-12-22--22-42-56
That's the route name
II am new to this if not providing correct information
Please tell me where to get the route
Thanks
Sent from my T-Mobile 5G Device
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Chris McCammon @.***>
Sent: Thursday, December 23, 2021, 10:00 AM
To: commaai/openpilot
Cc: cyanet01; Author
Subject: Re: [commaai/openpilot] fingerprint 2019 vw jetta sel (PR #23299)
What’s the route this fingerprint came from?
—
Reply to this email directly, view it on GitHubhttps://github.com/commaai/openpilot/pull/23299#issuecomment-1000453181, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AW3ICREM5YTF6LSRV3F2NWTUSNPTPANCNFSM5KUNDAWQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>

Hello
I don’t believe there is any duplicate lines
Richard
From: Jason Young @.>
Sent: Thursday, December 23, 2021 8:52 PM
To: commaai/openpilot @.>
Cc: cyanet01 @.>; Author @.>
Subject: Re: [commaai/openpilot] fingerprint 2019 vw jetta sel (PR #23299)
@jyoung8607 commented on this pull request.

In selfdrive/car/volkswagen/values.pyhttps://github.com/commaai/openpilot/pull/23299#discussion_r774871616:

@@ -319,10 +321,12 @@ class CAR:

   b'\xf1\x875QM909144C \xf1\x891082\xf1\x82\00521A00642A1',

   b'\xf1\x875QM909144C \xf1\x891082\xf1\x82\00521A10A01A1',

   b'\xf1\x875QN909144B \xf1\x895082\xf1\x82\00571A10A11A1',

 b'\xf1\x875QM909144B \xf1\x891081\xf1\x82\x0521A10A01A1',

Remove; duplicate
—
Reply to this email directly, view it on GitHubhttps://github.com/commaai/openpilot/pull/23299#pullrequestreview-839735197, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AW3ICRHXIBZE75U3PAEPEBDUSP36LANCNFSM5KUNDAWQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Hello I don’t believe there is any duplicate lines

Your EPS firmware was already on that list, second position. Automated CI checks later in the process will kick back duplicates, they're just not running for you yet as a new contributor.
I took the liberty of filing #23306 with your commit preserved along with the sorting/dedupe changes so we can go ahead and get you supported. Thank you for the data!


Updating DB first Entity Model Digram Doesn't Generate the files

From VS Feedback:
I'm using entity framework 6.2 with database first. I've tried this on webforms as well as class library projects.
When I'm updating the .edmx file, the new tables or columns in existing tables appear in the digram and the model browser but the code is missing. Whole new .cs files or new C# lines are not created.

EF Team Triage: This issue is lacking enough information for us to be able to effectively triage it. In particular, it is missing the following information requested in the new issue template. Can you please provide this information?

Steps to reproduce

Ideally include a complete code listing that we can run to reproduce the issue.
Alternatively, you can provide a project/solution that we can run.
BTW we're not just doing this to be mean :smile:... we get a lot traffic on this project and it takes time to attempt to reproduce an issue based on fragments of information. In addition, our attempt is often unsuccessful as the exact conditions required to hit the issue are often not explicitly included in the code provided. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we ask that folks give us a self-contained way to reproduce an issue.
For a guide on submitting good bug reports, read Painless Bug Tracking.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.

I had same issue while using VS2019 preview. After updating from database, the DbContext and corresponding model class were not generated.

I am having a similar issue too. When I Update model from Database, I am adding an existing table, so I find my table in the db list check it and click add, but nothing happens, no error or message that I can see.
I believe this is happening because my table does not have a primary key. That is the only difference I see between tables I have already add and this one. Is it a requirement for tables to have a primary key? If so is there a workaround?
My table in question is an audit table. I don't care for it to have an Id I only want to dump some data in it.

Duplicate of #705


Incorrect parsing of multiline strings within lists

Source:
list = [
    "first",
    """multi\
    line""",
    "last",
]

Expected:
{"list": ["first", "multiline", "last"]}

Actual:
{"list": ["first", "mu"]}

The same behavior.
And more examples even for one line. Comma is a problem in the string.
Those are OK.
toml.loads('''a=[' "a","b" ']''')
{'a': [' "a","b" ']}

toml.loads('''a=[""" "a" "b" """]''')
{'a': [' "a" "b" ']}

toml.loads('''a=[""" 'a' 'b' , """]''')
{'a': [" 'a' 'b' , "]}

toml.loads('''a=[""" 'a','b' """]''')
{'a': [" 'a','b' "]}

This is not OK.
toml.loads('''a=[""" "a" "b" , """]''')
{'a': [' "a" ', '']}

toml.loads('''a=[""" "a","b" """]''')
*** toml.decoder.TomlDecodeError: Found tokens after a closed string. Invalid TOML. (line 1 column 1 char 0)
Traceback (most recent call last):
  File "/path/.venv/lib/python3.7/site-packages/toml/decoder.py", line 512, in loads
    multibackslash)
  File "/path/.venv/lib/python3.7/site-packages/toml/decoder.py", line 778, in load_line
    value, vtype = self.load_value(pair[1], strictly_valid)
  File "/path/.venv/lib/python3.7/site-packages/toml/decoder.py", line 880, in load_value
    return (self.load_array(v), "array")
  File "/path/.venv/lib/python3.7/site-packages/toml/decoder.py", line 1026, in load_array
    nval, ntype = self.load_value(a[i])
  File "/path/.venv/lib/python3.7/site-packages/toml/decoder.py", line 849, in load_value
    raise ValueError("Found tokens after a closed " +
ValueError: Found tokens after a closed string. Invalid TOML.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/path/.venv/lib/python3.7/site-packages/toml/decoder.py", line 514, in loads
    raise TomlDecodeError(str(err), original, pos)

Hi, just to add another example:
>>> t1 = "foo = [\n\t'a',\n\t'''\\\n\tb\\\n\t''',\n]"
>>> t2 = "foo = [\n\t'''\\\n\ta\\\n\t''',\n\t'''\\\n\tb\\\n\t''',\n]"
>>> t3 = "foo = [\n\t'''\\\n\ta\\\n\t''',\n\t'b',\n\t'''\\\n\tc\\\n\t'''\n]"
>>> print(t1)  # One-line + multi-line
foo = [
        'a',
        '''\
        b\
        ''',
]
>>> print(t2)  # Multi-line only
foo = [
        '''\
        a\
        ''',
        '''\
        b\
        ''',
]
>>> print(t3)  # Multi-line + mixed
foo = [
        '''\
        a\
        ''',
        'b',
        '''\
        c\
        '''
]
>>> toml.loads(t1)  # Wrong
{'foo': ['a', '']}
>>> toml.loads(t2)  # OK
{'foo': ['a', 'b']}
>>> toml.loads(t3)  # OK
{'foo': ['a', 'b', 'c']}

Seems that string arrays which contain multiline strings but start with one-liners trip the parser.


Function to re-calculate the position of portaled popovers

🚀 Feature request
Add a function to re-calculate the position of portaled popovers and tooltips.
🧱 Problem Statement / Justification
I have an issue where elements get server-side rendered (Nuxt) and the position of all tooltips, selects and popovers is in the upper left corner. I can fix the issue by triggering the element in question to re-render on the client side. Ideally, I'd like to be able to call a function on the underlying machine to do this one bit of recalculation when I know the UI is ready instead of refreshing the whole component.
✅ Proposed solution or API
For example api.reposition()
Many thanks for your work on this project!

The machines provide an api.setPositioning(...)  function you can call to reposition the component.

The select machine misses this API but I'll add it shortly.


[BEAM-12164]: Parses change streams fields as json / strings

The backend is migrating from returning strings for certain fields to json. We need to change the parsing logic to accommodate for both until the migration is completed.

retest this please

Presubmit failures are unrelated to this PR

R: @pabloem

Run Java PreCommit

this LGTM. Thanks @thiagotnunes

Run Java PreCommit

Run Java PostCommit


Increase specificity of flex-padding-* classes to apply to flex-col and flex-size-* items only.

Completed.


tls-etcd command and alternative names from deploy_utils exceeds maximum of 64 for certificates

The values used for the tls-etcd common_name and alternative_names exceed the limit of 64 if your network +deployment name values exceed ~40 characters.
Task 32 | 11:21:03 | Preparing deployment: Preparing deployment (00:00:02)
                   L Error: Config Server failed to generate value for '/p-bosh/service-instance_e4d95d07-58ab-4159-8399-094ded22def0/tls-etcd' with type 'certificate'. HTTP Code '400', Error: 'The request could not be completed because the common name is too long. The max length for common name is 64 characters.'

Unfortunately this is a limitation of the X.509 standard. We recommend adjusting your deployment name and network name to avoid having a DNS name which exceeds this limit.

@cppforlife @tinygrasshopper FYI, this will likely be a problem for ODB services in general. Any recommendations or workarounds?

@st3v I think you can use the bosh use_short_dns_addresses feature for this. I can send you a doc about an explore the mysql team did for a similar issue.

Thanks @tinygrasshopper. use_short_dns_addresses looks interesting. You mentioned the way those  short names is generated is non-deterministic and might change in-between deployments. However, the MySQL doc just mentioned that those names are not predictable which reads different to me.
For etcd, non-deterministic would be problematic, while not predictable isn't really a problem.
@cppforlife Could you clarify? Or is there some documentation around this feature somewhere?
@jfmyers9 you might have some input too as you worked on this release and are now in the mids of BOSH.


Search examples are broken in version 4.24

They run fine in version 4.21 but don't work in 4.24

Looks like 4.22 changed locator to url in the source.  After changing this it works.
Before:
sources: [
  {
    name: "what3words",
    placeholder: "Please enter a what3words address",
    singleLineFieldName: "SingleLine",
    locator: locatorUrl
  }
]

After:
sources: [
  {
    name: "what3words",
    placeholder: "Please enter a what3words address",
    singleLineFieldName: "SingleLine",
    url: locatorUrl
  }
]


Update Installation Docs according to latest create-mock-backend app

Currently the Documentation Site doesn't contain the latest installation guide. Should contain details upon how to install with create-mock-backend

Fixed!


New data resource - Origin CA certificate chain

Current Terraform and Cloudflare provider version
Terraform v0.14.11

provider registry.terraform.io/cloudflare/cloudflare v2.21.0

Description
Add new data resource cloudflare_origin_ca_certificate_chain
Use cases
When applying certificates generated using the resource cloudflare_origin_ca_certificate one often needs access to the Cloudflare Origin CA Certificate Chain too. Until now I have been reading this from a link I found via the Cloudflare support site. Unfortunately the link I was using - https://support.cloudflare.com/hc/article_attachments/360037885371/origin_ca_rsa_root.pem - now returns a 404. The replacement does not like programmatic access; trying to access URL this via Terraform/curl returns a 403 with a Cloudflare Captcha page. So, I'm left searching for alternatives... It would be great if this provider handled that complexity for me.
Potential Terraform configuration
data "cloudflare_origin_ca_certificate_chain" "cloudflare" {}

References
No response

I ran into a similar issue due to the URL returning a 404. The error message from Terraform I got just said Error: HTTP request error. Response code: 404 with no information about the request that caused it. I had to use TF_LOG=trace to find the issue, which took quite a while since this particular workspace is very large and takes a long time to run a plan.
Please add this data resource.

Unfortunately, adding a data source isn't going to fix the issues outlined here as the Terraform resource will still need to call out to the endpoints to get this which if you can't do programmatically, won't be any better with Terraform.
I'm getting a 200 for that endpoint however I can see it does have bot management enabled which will be using various approaches to determine if the incoming request is coming from a machine or human.
$ curl -so /dev/null https://developers.cloudflare.com/ssl/e2b9968022bf23b071d95229b5678452/origin_ca_rsa_root.pem --write-out '%{http_code}'
200

Have you raised with this Cloudflare Support? If not, I can raise a ticket internally to see if we have a preferred endpoint for programmatic access.

Unfortunately, adding a data source isn't going to fix the issues outlined here as the Terraform resource will still need to call out to the endpoints to get this which if you can't do programmatically, won't be

I sort of realised this although I hoped that given that this was a cloudflare repo that if there was a data resource påut in place then maintenance of this resources (and the link from which it pulls the cert) would thus become a "Cloudflare" problem... I.e. in some way an "official" solution rather than me scraping links from support pages.

Have you raised with this Cloudflare Support? If not, I can raise a ticket internally to see if we have a preferred endpoint for programmatic access.

I haven't. If you could do that that would be fantastic!

The bot management on this endpoint has been tuned so these paths should now be accessible. We can still look at doing a data source in the future but this will unblock you for now.

The bot management on this endpoint has been tuned so these paths should now be accessible. We can still look at doing a data source in the future but this will unblock you for now.

Thank so much for following up! I can confirm that I no longer hit bot detection when trying to use a http data source now. But sadly this still isn't working. The endpoint for the (RSA) cert sets the content type returned when accessing the URL to application/x-x509-ca-cert whereas the http data source

can only retrieve data from URLs that respond with text/* or application/json content types, and expects the result to be UTF-8 encoded regardless of the returned content type header

I attempted to explicitly set the accept header via the http data source but this did not work either so it would seem like there is no content type negotiation in place on the resource at present.
Not sure on how best to handle this? Options I have come up with include:

prioritise adding a custom data source as this could handle this implementation detail without any trouble
attempt to get Cloudflare to change the content-type/implement content type negotiation on the endpoint
try to get application/x-x509-ca-cert added as an acceptable content type on the http data source (it is just text after all)

As I can't comment on the likelihood of 2 ever happening, I'd love to hear your thoughts @jacobbednarz?

Your second option seems to be trying to solve the wrong thing. Cloudflare is sending the correct content type however the tool you are using isn't compatible; that isn't a good reason to break the correct content type being sent. Option three is also viable if the maintainers are open to it but 🤷♂
I'd recommend taking a pass at implementing this as a datasource if you want this now as that is the most maintainable way forward. There are plenty of past examples to get you going, and you can simplify this even more by adding it to cloudflare-go first and have the HTTP client in there handle this for you. IPs are a good example of this:
cloudflare-go: https://github.com/cloudflare/cloudflare-go/blob/e625e78fb682d9f46bfe5ffde938ed19a1eff3e2/ips.go
terraform: https://github.com/cloudflare/terraform-provider-cloudflare/blob/master/cloudflare/data_source_ip_ranges.go


Rename MemoryIO to IO::Memory

Similar to #3401. MemoryIO is still available but produces a deprecation warning at compile-time, to ease migration.

My PR also did a s/a MemoryIO/an IO::Memory/g in the docs. This also happens over a line ending in IO::Buffered, so make sure you catch that.

@RX14 Done the a/an replacement, good catch :-)

Specs spec/std/io/io_spec.cr and spec/std/io/memory_io_spec.cr are still using old class names.

I spent quite a bit of time on the old PR, it's not as easy a PR as you would think at first.

@RX14 Do you want to rebase your PR instead? It will probably be easier :-)

(well, rebase or do it from scratch, whatever is easier)

Well you've done a lot with this PR too so why not go ahead with this.

@asterite there are MemoryIO leftovers left in spec/std/http/request_spec.cr.

@Sija Thanks. It seems they got added after I merged an old PR

@asterite yep, #3075 to be exact :)


English "ago" and "on" as postpositions

For feature GB075: Are there postpositions?, English is listed type 0 = no postpositions. However, ago as in two years ago and on as in two years on sure seem like postpositions, exactly parallel to in (in two years) or after (after two years). Grambank cites Huddleston & Pullum pgs 602, 603 as its source but on pg 632, H & P write (boldface added):

Etymologically, ago derives from a past participle form agone (the prefix a- + the past participle of go), so that ten years ago has its origin in a subject-predicate construction comparable to that in [17ii] except that the past participle is active rather than passive. Ago is of course no longer construed as a past participle, but how the construction should now be analysed is unclear: it is syntactically highly exceptional. [...] Traditional grammar classifies ago as an adverb, but on the basis that it takes a complement we analyse it as a preposition, in accordance with the criteria given in §2.4. It is even more exceptional than notwithstanding, etc., in that it always follows its complement. On, in the sense it has in [18b], namely “later”, likewise cannot occur without the accompanying phrase [...]. We again take the dependent to be a complement, therefore, and treat on in this sense like ago, as a preposition that follows its complement.

H & P also mention a some more marginal cases (e.g., all the world over). I would think this would qualify Standard English, and maybe some of the other English varieties, as type 1 = has postpositions languages. If it doesn't, it might be helpful to add a comment about why it doesn't to the wiki. That would make it easier for future users to understand the coding criteria.

Hi @rmalouf, thank you for your comment!
I'm the 'patron' for this feature, so responsible for its interpretation. I thought about this for a bit after you posted it and had a look at the data. I agree that English should qualify as 1 for this feature - the fact that these items should be analyzed as taking complements means that they should be analyzed as adpositions.
Changes are made and will be implemented in the next version of Grambank.
Thank you again for the correction!
Jakob


"Cannot find a Resource with the Name/Key" when ResourceDictionary contains one resource

Describe the bug
A runtime UnhandledException occurs when a StaticResource references a resource defined in a ResourceDictionary B that only contains a single resource, which itself has a StaticResource to another ResourceDictionary A. Both ResourceDictionary A and B are included in App.xaml as MergedDictionaries.
Steps to reproduce the bug

Create a new project using the blank UWP template
Add a folder to the project "Resources" and inside add new ResourceDictionary Colors.xaml and Brushes.xaml
Add a single <Color x:Key="MyColor">#FF0000</Color> to the Colors.xaml ResourceDictionary
Add a single <SolidColorBrush x:Key="MyBrush" Color="{StaticResource MyColor}"/> to Brushes.xaml
Add the Colors.xaml and Brushes.xaml to App.xaml:

     <Application.Resources>
        <ResourceDictionary>
            <ResourceDictionary.MergedDictionaries>
                <ResourceDictionary Source="/Resources/Colors.xaml"/>
                <ResourceDictionary Source="/Resources/Brushes.xaml"/>
            </ResourceDictionary.MergedDictionaries>
        </ResourceDictionary>
    </Application.Resources>

Add <Rectangle Fill="{StaticResource MyBrush}" Width="100" Height="100"/> to the MainPage.xaml

Expected behavior
The red Rectangle is shown on the MainPage when the app runs.
Screenshots
N/A
Version Info

NuGet package version:
Seeing this with Windows.UI.Xaml but was unable to find a proper location to file the issue, and was redirected here from the doc page for Windows.UI.Xaml.ResourceDictionary.

Windows app type:

UWP
Win32

Yes
N/A

Windows 10 version
Saw the problem?

May 2019 Update (18362)
Yes

Device form factor
Saw the problem?

Desktop
Yes

Additional context
Adding another Brush resource to Brushes.xaml will prevent the UnhandledException.

@bartekk8 and @MikeHillberg weird bug.  Maybe there is some optimization for RD with a single element?

Adding another Brush resource to Brushes.xaml will prevent the UnhandledException.

Well, good news is that you have a workaround. But yeah... that definitely sounds like a bug. Perhaps some sort of optimization/confusion with the difference between a collection and a single item.

Still presents in WinAppSDK 1.4 (WinUI3)

Can still repro this in WinAppSDK 1.5.


ci: run tests before building binaries

The main reason to do this is to cache the test dependencies and results. Github Actions caches are immutable, so we want to run tests first to ensure their results are cached.
We also run tests on main before dev builds for the same reason.

Holding for now, this might not be the best way to achieve better caching in the CI.

Closing in favor of #1131

Closing in favor of #1131


More on JSON serialization

I've added in all the StringEnumConverter attributes, so everything serializes properly.
Also, I've flipped IThing and IListItem in the Values<> of ListItem, since with this new behavior it should deserialize as the most specific type (right to left).
I had to fix the ListItem test for the new semantics where it won't deserialize duplicate objects.  All the other unit tests still worked fine, so I think this will be OK.
I refactored the default JSON serializer settings into TestDefaults class just to simplify things a bit.

We use a tool (Schema.NET.Tool) to generate all Schema.NET types from the schema.org schema file that we download. Changing the enums manually and re-ordering the types (i.e. IThing) is going to get overwritten.
Take a look at Enumeration.cs and Property.cs to change the way the enum and property code is generated.

@RehanSaeed Actually, I'd already fixed up Enumeration.cs to generate the enums like that (maybe a week ago or so?) but just needed to unit test the changes, so did them manually for now, knowing they'll get overwritten when you next run the tool.
I see how ItemListElement matched schema.org, and maybe that's just a glitch with the order they've defined things.  In most cases, the most "specific" interface match is listed last, but in this case, it's first.
I've reverted it back to the original (schema.org) order, and fixed up the related unit tests, to look for IThing not IListItem.    So, now if you stick a IListItem in ItemListElement, it'll come back out as an IThing - not a IListItem.  Not optimal, but it seems like that is the only odd case.
Currently we try and assign the best match from right-to-left (or last-to-first, in schema.org terms).
Another approach may be to keep a "weight" on each type, and sort the Values<> types by priority, and try to match in that order.  So any primitive types are tried last, and any interface at the top of the type hierarchy is tried first.  Not sure if this overcomplicates things though?

I made some changes which introduced a merge conflict. I used the new GitHub feature to resolve it which was cool. I saw your comment in https://github.com/RehanSaeed/Schema.NET/issues/75. Are you planning on more changes?

Nope, no more changes. All checked in.
On Fri, Jun 28, 2019 at 3:30 AM Muhammad Rehan Saeed <
<EMAIL_ADDRESS>wrote:

I made some changes which introduced a merge conflict. I used the new
GitHub feature to resolve it which was cool. I saw your comment in #75
https://github.com/RehanSaeed/Schema.NET/issues/75. Are you planning on
more changes?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RehanSaeed/Schema.NET/pull/71?email_source=notifications&email_token=ADHW7NUXL2DETEJWMMVYUDLP4XR4RA5CNFSM4H3EQ5M2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYZW4WA#issuecomment-506687064,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADHW7NQAD4242N5EKQF4XLTP4XR4RANCNFSM4H3EQ5MQ
.

--
KIRK MARPLE | Candela Software Technologies, LLC
http://candela.software | Direct:<PHONE_NUMBER> | Skype: kirkmarple |
Email<EMAIL_ADDRESS>

Noticed a build failure with previous checkin, so fixed that.  Now, no more changes planned.
Would love to get a formal release of this soon.

Merged. Thanks for your hard work!


feat(prisma-pg-worker): add to test suite

This PR complements and adds @prisma/pg-worker to the functional integration test suite in prisma/prisma.

Closing via this comment by @millsp.


vastTag Recall

Hello I need help how to call vast tag if it fail first time to call it for example 5 times and check am I get response?

Hi @bajro17 ,
Do you have any sample code to reproduce this issue?
We recommend continuing with content playback if your ad request fails. Recalling a VAST tag multiple times can appear as spammy behavior to the ads server.

Hi @Kiro705,
With other side of content playback, it will lead to lower Ad fill-rate, right? And if spammy, what will happen to it?
Thank you.

Hello @lphnam ,
I am not sure about your first questions, about ad fill-rates. Spammy behavior may lead to consequences imposed from the Ad Manager side.


Problem with  VS for python

I'm trying to make a simply math in VS with python.
a = 2
b = 3
c = a + b
print (c)
but it gave me this error:

print (c)
Traceback (most recent call last):
File "", line 1, in
NameError: name 'c' is not defined

Copying and pasting this code exactly works for me. Something is wrong with your setup.

is there anyway to know how can I set up this correctly ?

What version of python are you using? Maybe record a gif from https://gifcap.dev/

For python i'm using 3.11 (64 bit)
The weird is that on the python works perfectly, just in  Visual studio doesn't work.

The file you're showing is not saved. Does saving it before running fix it?

How can I save it ? can you show me an example pls ? I think I did, just want to be sure.

Just press CTRL + S

Yes, it's save it and the error still there.