[BUG] Does not stay signed in

Describe the bug
After a relaunch, I need to sign in again. This started happening on 2.4.1.
To Reproduce
Steps to reproduce the behavior:

Sign in
Close app
Launch app

Expected behavior
Session retained across relaunch.
Desktop (please complete the following information):

OS: Windows 10 Pro
Application Version: 2.4.1

Log File
Censoring any unique looking strings.
Session storage:
*®Ë#           version1
namespace-ÓR®Ïf           next-map-id1Gnamespace-xxxxxxxx_xxxx_xxxx_xxxx_xxxxxxxxxxxx-https://music.apple.com/xxxxx           next-map-id2Knamespace-xxxxxxxx_xxxx_xxxx_xxxx_xxxxxxxxxxxx-https://mediaauth.apple.com/1

Local Storage:
ð{Š(          VERSION1META:https://music.apple.com¶Ô°™ÊÉç#R_https://music.apple.com mtClientConfig_cachedSource_xp_amp_music_cs_unidentifiedá{"blacklistedFields":[],"compoundSeparator":"_","fieldsMap":{"custom":{"impressions":["id","adamId","link.type","station-hash"],"location":["id","adamId","dataSetId","name","fcKind","kindIds","type","link.type","station-hash","core-seed-name"]},"multi":{},"single":{"targetId":["id","adamId","contentId","type","link.type","fcId","userPreference","label","station-hash","linkIdentifier"]}},"ignoreClientIdCookie":true,"metricsUrl":"https://xp.apple.com/report","postFrequency":60000,"postFrequencyLowLatency":5000,"tokenSeparator":"|","treatmentProfiles":{"AMPFunnel":{"treatments":[{"filters":"any","eventActions":{"blacklistedFields":["mtClientId","mtRequestId","pliIds","userAgent","screenHeight","screenWidth","windowInnerHeight","windowInnerWidth","windowOuterHeight","windowOuterWidth"]},"fieldActions":{"eventTime":{"treatmentType":"timeDeres","precision":86400000},"clientId":{"treatmentType":"idGenerator","storageKeyPrefix":"mtId_clientId_unidentified","lifespan":15552000000}}}]}}}G_https://music.apple.com mtClientConfig_cachedSource_xp_amp_music_perf°
{"blacklistedFields":["capacitySystem","capacitySystemAvailable","capacityDisk","capacityData","capacityDataAvailable"],"compoundSeparator":"_","fieldsMap":{"cookies":["itcCt","itscc"],"custom":{"impressions":["id","adamId","link.type","station-hash"],"location":["id","adamId","dataSetId","name","fcKind","kindIds","type","link.type","station-hash","core-seed-name"]},"loadUrl":{"timestampFields":{"fetchStartTime":["fetchStart"],"domainLookupStartTime":["domainLookupStart"],"domainLookupEndTime":["domainLookupEnd"],"connectionStartTime":["connectStart"],"connectionEndTime":["connectEnd"],"secureConnectionStartTime":["secureConnectionStart"],"requestStartTime":["requestStart"],"responseStartTime":["responseStart"],"responseEndTime":["responseEnd"]},"otherFields":{"requestUrl":["name"],"transferSize":["transferSize"]}},"multi":{},"single":{"targetId":["id","adamId","contentId","type","link.type","fcId","userPreference","label","station-hash","linkIdentifier"]}},"impressions":{"viewableThreshold":1000,"viewablePercentage":0.5},"metricsUrl":"https://xp.apple.com/report","performance":{"samplingPercentage":0.25,"samplingPercentageUsers":0.25,"sessionDuration":60000,"samplingPercentageUsersPageRender":0.25,"sessionDurationPageRender":60000},"postFrequency":60000,"postFrequencyLowLatency":5000,"tokenSeparator":"|"}G_https://music.apple.com mtClientConfig_cachedSource_xp_its_music_mainã{"blacklistedFields":[],"constraints":{"profiles":{"AMPWeb":{"precedenceOrderedRules":[{"filters":"any","fieldConstraints":{"clientId":{"generateValue":true,"namespace":"AMPWeb_isSignedOut","expirationPeriod":86400000}}},{"filters":{"valueMatches":{"isSignedIn":[true]}},"fieldConstraints":{"clientId":{"generateValue":true,"namespace":"AMPWeb_isSignedIn","expirationPeriod":15552000000}}}]}}},"treatmentProfiles":{"AMPWeb":{"treatments":[{"filters":"any","fieldActions":{"clientId":{"treatmentType":"idGenerator","storageKeyPrefix":"mtClientId","namespace":"AMPWeb_isSignedOut","lifespan":86400000,"persistIdForSession":true},"analyticsClientId":{"treatmentType":"idGenerator","namespace":"AMPWeb_isSignedOut","lifespan":86400000,"persistIdForSession":true}}},{"filters":{"isSignedIn":{"valueMatches":[true]}},"fieldActions":{"clientId":{"treatmentType":"idGenerator","storageKeyPrefix":"mtClientId","namespace":"AMPWeb_isSignedIn","lifespan":15552000000,"persistIdForSession":true},"analyticsClientId":{"treatmentType":"idGenerator","namespace":"AMPWeb_isSignedIn","lifespan":15552000000,"persistIdForSession":true}}}]}},"compoundSeparator":"_","fieldsMap":{"cookies":["itcCt","itscc"],"custom":{"impressions":["id","adamId","link.type","station-hash"],"location":["id","adamId","dataSetId","name","fcKind","kindIds","type","link.type","station-hash","core-seed-name"]},"multi":{},"single":{"targetId":["id","adamId","contentId","type","link.type","fcId","userPreference","label","station-hash","linkIdentifier"]}},"metricsUrl":"https://xp.apple.com/report","postFrequency":60000,"postFrequencyLowLatency":5000,"tokenSeparator":"|"}7_https://music.apple.com mtClientId_AMPWeb_isSignedOutP{"value":"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx","expirationTime":1627068551618}C_https://music.apple.com mtId_analyticsClientId_AMPWeb_isSignedOutQ{"value":"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx","expirationTime":1627068551618}4_https://music.apple.com mtId_clientId_unidentifiedR{"value":"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx","expirationTime":1642534151620}/_https://music.apple.com mtTimestamp_eventTime{"exp":1626998400000,"sn":2}7_https://music.apple.com music-app-pref:hasSeenWelcometrueG_https://music.apple.com music-app-pref:volume-pref-for-user-undefined0.5UY”Õ           META:https://music.apple.comÕÓ¡Ê™ÊÉš$=_https://music.apple.com dismissed-recommended-geo-prompt-us1626982161159/_https://music.apple.com mtTimestamp_eventTime{"exp":1626998400000,"sn":11}¯îÝ+”            META:https://mediaauth.apple.comÖþåä™ÊÉ:I_https://mediaauth.apple.com mk-auth-bridge-storage-event-cookie-updated1626982262497ÎBð5•          META:https://music.apple.comØéïæ™ÊÉµ+6_https://music.apple.com mtClientId_AMPWeb_isSignedInR{"value":"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx","expirationTime":1642534262098}B_https://music.apple.com mtId_analyticsClientId_AMPWeb_isSignedInR{"value":"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx","expirationTime":1642534262098}/_https://music.apple.com mtTimestamp_eventTime{"exp":1626998400000,"sn":19}ª_https://music.apple.com music-app-library:tab-state-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/xxxxxxxxxx+xxxxx+R/xxxx/xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/xxxx==…{"library.recently-added":true,"library.artists":true,"library.albums":true,"library.songs":true,"library.made-for-you.index":false}²_https://music.apple.com music-app-pref:volume-pref-for-user-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/xxxxxxxxxxxxxxxxxx/xxxx/xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/xxxx==1üåÍ$”            META:https://mediaauth.apple.comxxxxxx:I_https://mediaauth.apple.com mk-auth-bridge-storage-event-cookie-updatedxxxxxxxxxxxxxxxxxxxx”            META:https://mediaauth.apple.comxxxxxxxx:I_https://mediaauth.apple.com mk-auth-bridge-storage-event-cookie-updatedxxxxxxxxxxxxxxxxx           META:https://music.apple.comõõáú™ÊÉµ+/_https://music.apple.com mtTimestamp_eventTime{"exp":1626998400000,"sn":21}

Looks like Apple is having server issues again. This should be resolved soon.


Create a script to deploy WMAgent docker image

Impact of the new feature
WMAgent
Is your feature request related to a problem? Please describe.
We need to have a set of scripts to deploy and manage the agent packaged in a docker image.
Describe the solution you'd like
A few scripts need to be created for deploying and managing the agent, such as:

deploy script as already available in CMSKubernetes (or WMCore)
script to bootstrap the agent (tweak configuration based on secrets) and to populate initial databases
script to manage the agent (usual stop/start/restart/status actions)

NOTE: it might require further discussion and whenever possible, reuse code already available.
Describe alternatives you've considered
None
Additional context
Depends on: https://github.com/dmwm/WMCore/issues/8797
Part of the following meta issue: https://github.com/dmwm/WMCore/issues/11314

This has been addressed with: https://github.com/dmwm/CMSKubernetes/pull/1393
Thanks Todor! Closing this one out.


Allow to link against upstream espeak-ng

This PR allows to link piper-phonemize with the unmodified upstream version of espeak-ng.
Tested and working on glibc and musl.

Following up on a related discussion over here, this PR was changed to a more robust flow, the original implementation can be reviewed on this branch.

Unless I'm missing something, this seems to not preserve punctuation (which is the reason for the espeak-ng fork). The problem is that espeak_TextToPhonemes does not return punctuation with phonemes. This is important for Piper, because voices use punctuation to correctly pause and for correct intonation.
The previous solution looked like it would do the right thing, despite being less robust.

Almost :) Both solutions do preserve punctionation support, only that the new solution does not outsource punctuation from espeak-ng anymore, thus providing a hack-free solution at the cost of being marginally less flexible.
However, I did check the espeak-ng code for punctuation and for the cases handled by piper-phonemize the chosen manual implementation should not be signifficantly less complete.

My tests fail though when using this PR and espeak-ng 1.51. Have they changed things in a later release?

Sorry for the late reply, work came along!
Indeed, the new implementation was not handling the case where inputTextPointer was already NULL – that didn't happen during my tests because I tested through stdin, thus always had a \n behind my test phrases.
Should be fixed with c46ba6e, test.cpp is now passing on my side.

Great, I'll check again. Thanks!

@synesthesiam FYI, PR was rebased on the latest master branch :)


Toggle Type in React

New to this library, how to I get they chart type to change? Here is my code.
<div>
  <ChartistGraph
     className="ct-chart"
     data={chartData}
     type={isLine ? "Line" : "Bar"}
     options={options}
  />
  <Switch
     checked={isLine}
     label="Uncontrolled"
     onClick={() => setIsLine(!isLine)}
   />
</div>

Hello @jaredswenson 👋
Chartist has recently been updated to version 1.0 with some new exciting changes like ESM and TypeScript support. The migration guide can be found right here.
Please let me know if the issue is still relevant to you so we can further investigate it!


Bug in opt -full

module ltbug (input [3:0] a, output res);
    assign res = a < 6'b100000;
endmodule

Above code crashes yosys with opt -full command.
The crash happens in opt_expr.cc on this line :
					RTLIL::SigSpec a_prime(RTLIL::State::S0, width - bit_set);
as width - bit_set becomes negative.

I should have a fix out by tomorrow (I wrote that part).

Fixed by #308.


Updated Helm Installation Instructions

added helm Prometheus setup for users new to K8s who are not familiar with Prometheus Node Exporter which needs to be installed for Kepler to work. I recommend the installation of the kube-prometheus-stack.
clarified the usage of the serviceMonitor for the Kepler charts: The value serviceMonitor.enabled needs to be set to true and the value serviceMonitor.labels.release needs to be set to the release name of the kube-prometheus-stack installtion.
added some convenient post install instructions, also aimed at newer users.
added chinese version of this docs, translated by gpt-4. This is probably very wrong and should only serve as a draft for someone who speaks the language ^^

@marvin-steinke thanks! can you sign DCO
Ensure you have a local copy of your branch by [checking out the pull request locally via command line](https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/checking-out-pull-requests-locally).
In your local branch, run: git rebase HEAD~2 --signoff
Force push your changes to overwrite the branch: git push --force-with-lease origin main

Can we try to have a link to prometheus install steps? Honestly I am not sure if we should cover prometheus steps in our document.... which means for any update on prometheus we should have an update ...

I'm afraid I don't know how to install the Kepler helm package for a non-helm prometheus installation regarding the release label that has to be set. Because of this I included specific instructions for a full helm install. The only way installation instructions for the kube-prometheus-stack are going to change is if the repository stops to exists in which case we would have to link another helm chart anyways, right?

Can we try to have a link to prometheus install steps? Honestly I am not sure if we should cover prometheus steps in our document.... which means for any update on prometheus we should have an update ...

I'm afraid I don't know how to install the Kepler helm package for a non-helm prometheus installation regarding the release label that has to be set. Because of this I included specific instructions for a full helm install. The only way installation instructions for the kube-prometheus-stack are going to change is if the repository stops to exists in which case we would have to link another helm chart anyways, right?

ok, please go ahead to complete this PR and notify me after it's ready for review :-)


Add fingerprint into CSV file

Since the Powershell script uses the fingerprint to get transactions we should put that into the CSV file also. At least in verbose mode, and I think normal also.

Added in v0.1.1


EloquentUserProvider does not seem to honor setting for config/auth User model

I've change the location of my User model, updated the namespace accordingly and updated config/auth. Still Laravel 5.1 fails with a:
FatalErrorException in EloquentUserProvider.php line 127:
Class '\App\User' not found

If I change the method createModel there to return a fixed string (my User class name), instead of the model paramter passed in, everything works. It's as if it's completey ignoring the auth/config setting, and return a default one. If i leave the default User class, it also works as expected.
If I do Auth::user() the default class is used. Since this is a AJAX application and the correct user is fetched through an API key stored with the user id in a table, and Auth is only used for inital access, I do not access data through Auth::user() so it does not pose a problem for me -- still ugly though.
Custom User class:
namespace  App\Todo\Users;

use App\Todo\IIdentifiable;
use Illuminate\Auth\Authenticatable;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Contracts\Auth\Authenticatable as AuthenticatableContract;
use Illuminate\Support\Facades\Hash;
use Nicolaslopezj\Searchable\SearchableTrait;

class User extends Model implements AuthenticatableContract, IIdentifiable {

	use Authenticatable, SearchableTrait;

       /* relations and custom methods */
}

config\auth.php:
'model' => '\App\Todo\Users\User',

Does anyone know what is going on here? Is there something missing in relation to AuthenticatableContract?

    /**
     * Create an instance of the Eloquent user provider.
     *
     * @return \Illuminate\Auth\EloquentUserProvider
     */
    protected function createEloquentProvider()
    {
        $model = $this->app['config']['auth.model'];

        return new EloquentUserProvider($this->app['hash'], $model);
    }

Looking at the code, this definetely works.

Are you trying to change that value at runtime, You'll need to clear the resolved eloquent user provider instance if that's the case.

No, not changeing anything at runtime.
When I do dd($this->model) in createEloquentProvider() I still returns 'App\User'. Is there any caching of the config files I am stuck with?
This is really wired, it does not appear to have any trouble reading any of my other config files entries.

Is there any default  values provided for the $app['config'] getter?

more digging reveales that changes in my auth config isn't reflected, though where it gets the default values I have no idea.

Is there any caching of the config files I am stuck with?

yes there is. You need to clear the config cache. There's an artisan command for that.

Thanks man!
Can't believe it was that easy! (Clearing the config worked)


Feature: callback when the webcontainer is finished booting

Hi there,
I'm working on integrating stackblitz into an app, and I'd like to show my own loading screen up until the web container finishes running the startScript. Would it be possible to allow me to provide a callback function when the start script completes or encounters an error?

Hi there,
I'm working on integrating stackblitz into an app, and I'd like to show my own loading screen up until the web container finishes running the startScript. Would it be possible to allow me to provide a callback function when the start script completes or encounters an error?

Hi team,
What are your thoughts on this? Would it be too optimistic to expect something about this?
Dear @atrakh , have you somehow managed to do this?


[bug]:  "Open Jellyfin"-link in tray bar leads to a fixed port number

Please describe your bug
The "open Jellyfin"-link in the tray bar on windows always leads to the port 8096. I have changed my port and that link doesn't adapt to it. Not sure if it is fixable at all or if it is just a static link that can't adapt to port changes

leads to

when actual port is

Jellyfin Version
10.8.0
if other:
No response
Environment
- OS: win10
- Virtualization:
- Clients: Server
- Browser:
- FFmpeg Version:
- Playback Method:
- Hardware Acceleration:
- Plugins:
- Reverse Proxy:
- Base URL:
- Networking:
- Storage:

Jellyfin logs
No response
FFmpeg logs
No response
Please attach any browser or client logs here
No response
Please attach any screenshots here
No response
Code of Conduct

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

Thanks for the catch. In 10.8 we moved the port from the system.xml file to the network.xml file. We had the tray app all set up for 10.7 still.
Have to move these lines:
https://github.com/jellyfin/jellyfin-server-windows/blob/48e9c9bf4fc90665a465ba132f28aab3a155bada/Jellyfin.Windows.Tray/TrayApplicationContext.cs#L158-L162
To the the space in L169:
https://github.com/jellyfin/jellyfin-server-windows/blob/48e9c9bf4fc90665a465ba132f28aab3a155bada/Jellyfin.Windows.Tray/TrayApplicationContext.cs#L165-L171
We plan on releasing a 10.8.1 very, very soon, so I'll make this fix a part of that.


NPM for this node-lame?

I have found the TooTallNate/node-lame is working better than the jankarres/node-lame, but when I do a "npm install node-lame" I get the jankarres version.
Anyone know how to npm this version?  Help appreciated!

npm install lame


Cant access kubeflow dashboard after using "kubectl port-forward svc/istio-ingressgateway -n istio-system --address <IP_ADDRESS> 8085:80"

I tried install kubeflow on aws with s3 storage by following tutorial from https://awslabs.github.io/kubeflow-manifests/docs/deployment/
Everything works well except the last step to access kubeflow dashboard: kubectl port-forward svc/istio-ingressgateway -n istio-system --address <IP_ADDRESS> 8085:80
After using port-fowarding, I cant access http://localhost:8080/
This page gave me 403 error: You don't have authorization to view this page!
How can I fix  this?

@TranThanh96 Can you make sure your command is the same as the doc https://awslabs.github.io/kubeflow-manifests/docs/deployment/vanilla/guide/#port-forward ?
kubectl port-forward svc/istio-ingressgateway -n istio-system 8080:80

Hey @TranThanh96, I responded to you on slack, can you additionally specify which deployment option you ran, was it the rds-s3?

@TranThanh96 Can you make sure your command is the same as the doc https://awslabs.github.io/kubeflow-manifests/docs/deployment/vanilla/guide/#port-forward ?
kubectl port-forward svc/istio-ingressgateway -n istio-system 8080:80

ya, I tried both with and without  --address <IP_ADDRESS>
still cant access from browser

Hey @TranThanh96, I responded to you on slack, can you additionally specify which deployment option you ran, was it the rds-s3?

I use s3 only

after re-installing everything, I can reach the login page now

sounds good, verify you are able to login and run any samples you wish.

sounds good, verify you are able to login and run any samples you wish.

@ryansteakley I cant see any example pipelines in dashboard

and I cant create a new notebook server, error: 0/1 nodes are available : 1 too many pods

looks like you have several pods in crashloop backoff. Is your instance the same size or similar to the one described in https://awslabs.github.io/kubeflow-manifests/docs/deployment/prerequisites/ Did you follow the auto-setup python script?

run kubectl describe pod  -n  and similarily kubectl logs  -n  on the pods in failure state. and share anything you find there as well

@ryansteakley
I follow guide install with s3 only.
I am using 2 node: t3.xlarge
now I have 3 pod that keep CrashLoopBackOff:

and this is log for each pod
metadata-grpc-deployment-f8d68f687-pdzcx_describe.txt
metadata-grpc-deployment-f8d68f687-pdzcx_log..txt
metadata-writer-d7ff8d4bc-qqtjz_describe.txt
metadata-writer-d7ff8d4bc-qqtjz_log.txt
ml-pipeline-777648985d-jhkvl_describe.txt
ml-pipeline-777648985d-jhkvl_log.txt

Warning  Failed     34m (x5 over 34m)      kubelet            Error: secret "mlpipeline-minio-artifact" not found in ml-pipeline logs. Can you check to see if this secret exists. Run kubectl get secrets -n kubeflow

secrets_kf_log.txt
seem like it exist

Can you verify that you are using v3.2.0 of kustomize? Run kubectl delete pods -n kubeflow --all and see if the pods come up normally.

yes, I am using kustomize v3.2.0

Tried kubectl delete pods -n kubeflow --all but the pod metadata-grpc-deployment-f8d68f687-mqs82  keep crashloopbackoff

What do you see when you login? Are any other pods still failing?

everything is good except those 3 pods keep crashloopbackoff

and I get some errors on Pipelines and Runs, any suggestions please?

errors on Runs:

and these pods:

Can you verify that the s3-secret you created is following this requirement.  Configure a Secret (e.g. s3-secret) with your AWS credentials. These need to be long-term credentials from an IAM user and not temporary.

yes, I can confirm that

No way, to prove. Can you one more time describe the ml-pipeline pod. I would suggest restarting from a fresh cluster, and follow the cluster pre-req listed above.

Yes, this is 3rd times I re-install kubeflow on aws eks from a fresh cluster. and this error keep occurring

Sorry you are running into these problems, if you can please share the logs from the latest crashloopbackoff mlpipeline. Which version of AWS kubeflow are you running? I will try to reproduce your issue on my end and see if there is some underlying issue.

Sorry you are running into these problems, if you can please share the logs from the latest crashloopbackoff mlpipeline. Which version of AWS kubeflow are you running? I will try to reproduce your issue on my end and see if there is some underlying issue.

how can I get these log? I can provide it to you.
I am using this version:
KUBEFLOW_RELEASE_VERSION=v1.5.1
AWS_RELEASE_VERSION=v1.5.1-aws-b1.0.1

kubectl logs <ml-pipeline-pod> -n kubeflow i see you are running 2 node: t3.xlarge, we reccomend a minimum of 5 nodes and m5.xlarge. Stated here https://awslabs.github.io/kubeflow-manifests/docs/deployment/prerequisites/ if you have time try to re-create following the suggested cluster create command

kubectl logs  -n kubeflow

This is log from ml-pipeline

@ryansteakley @TranThanh96 I think this is because of a bug related to missing mysql deployment in S3 only deployment option. It was fixed in main branch recently but not backported to release branch https://github.com/awslabs/kubeflow-manifests/pull/310
@TranThanh96 Can you comment out this like - disable-mysql-pv-claim.yaml in awsconfigs/apps/pipeline/s3/kustomization.yaml and run
kustomize build awsconfigs/apps/pipeline/s3 | kubectl apply -f -
Please delete the ml-pipeline and grpc pod after doing this so that a new pod gets created

@ryansteakley @TranThanh96 I think this is because of a bug related to missing mysql deployment in S3 only deployment option. It was fixed in main branch recently but not backported to release branch #310
@TranThanh96 Can you comment out this like - disable-mysql-pv-claim.yaml in awsconfigs/apps/pipeline/s3/kustomization.yaml and run
kustomize build awsconfigs/apps/pipeline/s3 | kubectl apply -f -
Please delete the pods which are in crashloopbackoff after doing this so that a new pod gets created

yes, I try with rds + s3 deployment. everything works. So the problem is related to  mysql

Thanks for reporting this issue. We have released a patch version (v1.5.1-aws-b1.0.2) to fix this issue


Could not execute query - unable to open database file: out of memory (14)

Describe the bug
I'm able to successfully import a 2MB+ sqlite3 db file into grafana. But when I try to run a basic query, I get this error:

logger=plugin.frser-sqlite-datasource t=2022-11-16T06:34:44.612548514Z level=error msg="Could not execute query" err="unable to open database file: out of memory (14)" query="select * from apipayments;"

To Reproduce
Steps to reproduce the behavior:

run grafana with docker, bind a local file to the docker with -v
install this plugin
setup a panel using the sqlite3 file
See error

Screenshots

Versions (please complete the following information):

OS: debian 10
Run via docker? yes
Browser chrome
Grafana Version 9.3.0-beta1
Plugin Version 3.1.0
Plugin installed via grafana-cli or manually? from GUI

Additional context
VM has 64GB RAM so there's plenty of memory available.

Interesting behavior indeed. Do you see the same for older versions of Grafana?
And do you have a full debug level log to check?


Target debug_android_application failed: FileSystemException: Cannot copy file to...

Steps to Reproduce

Create a default flutter app
Add an emulator
Click on 'Run'

Logs

Launching lib/main.dart on AOSP on IA Emulator in debug mode...
Running Gradle task 'assembleDebug'...
Target debug_android_application failed: FileSystemException: Cannot copy file to '/Users/huangzheng/AndroidStudioProjects/flutter_app/build/app/intermediates/flutter/debug/flutter_assets/vm_snapshot_data', path = '/Users/huangzheng/Desktop/Projects/FlutterFolder/flutter/bin/cache/artifacts/engine/darwin-x64/vm_isolate_snapshot.bin' (OS Error: No such file or directory, errno = 2)
build failed.

FAILURE: Build failed with an exception.

* Where:
Script '/Users/huangzheng/Desktop/Projects/FlutterFolder/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 780

* What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.
> Process 'command '/Users/huangzheng/Desktop/Projects/FlutterFolder/flutter/bin/flutter'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 9s
Finished with error: Gradle task assembleDebug failed with exit code 1

[✓] Flutter (Channel stable, v1.12.13+hotfix.9, on Mac OS X 10.15 19A603, locale en-CA)
    • Flutter version 1.12.13+hotfix.9 at /Users/huangzheng/Desktop/Projects/FlutterFolder/flutter
    • Framework revision f139b11009 (3 weeks ago), 2020-03-30 13:57:30 -0700
    • Engine revision af51afceb8
    • Dart version 2.7.2

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/huangzheng/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✗] Xcode - develop for iOS and macOS
    ✗ Xcode installation is incomplete; a full installation is necessary for iOS development.
      Download at: https://developer.apple.com/xcode/download/
      Or install Xcode via the App Store.
      Once installed, run:
        sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
        sudo xcodebuild -runFirstLaunch
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install:
        sudo gem install cocoapods

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] Connected device (1 available)
    • AOSP on IA Emulator • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

! Doctor found issues in 1 category.
Process finished with exit code 0

The reason might be in the $PATH. I uninstalled Flutter SDK and Andriod Studio, then ran over the whole process and eventually the problem got solved.


Add side panels for commit pipeline visualization

Fixes
Fixes HAC-2578
Description
Add support for node click in the commit visualization. Upon a click a side panel is shown withe the appropriate information for the selected node.
Type of change

[x] Feature

Screen shots / Gifs for design review
Commit:

Build:

Integration Test:

Environment:

Release:

Managed Environment:

/cc @MariaLeonova @karthikjeeyar @christianvogt

Need to add tests still

@jeff-phillips-18 Vulnerability scan results UI has been updated by @christianvogt in this PR and should be the same here.

Sidebar heading for component test empty state is should not be a link (right now it takes you to 404 page).

@MariaLeonova We are not showing the release and managed environment nodes in lifecycle visualization today, we have hidden this under the MVP feature flag. If we need to show this in commits page, then we must show in lifecycle graph too or else it would be confusing to the user.  Do you know if we are bringing back in lifecycle graph? or shall we hide this release/managed env until we show in lifecycle graph?

@MariaLeonova We are not showing the release and managed environment nodes in lifecycle visualization today, we have hidden this under the MVP feature flag. If we need to show this in commits page, then we must show in lifecycle graph too or else it would be confusing to the user. Do you know if we are bringing back in lifecycle graph? or shall we hide this release/managed env until we show in lifecycle graph?

It is hidden behind the MVP flag here as well, I just turned it on to show in the screenshot.

/approve

/retest

/lgtm


demos: do not generate twice the HelpersDispatchTable

The HelpersDispatchTable.* are already generated by the encoder lib.

TBH I am hoping once Vulkan Video APIs stabilize in the Vulkan SDK, that I can skip using the dispatch table + dll hookups in lieu of direct vulkan-1.lib C function calls.
I suppose that may not happen in a while given how much this is still being actively developed, but long-term it would be nice.


Add new manager

Please add omf :D

Sorry it took so long. I overlooked your request and only noticed it today when searching for an old email.
I added support for omf in v0.4. Don't forget to set the package manager, as fisher still is the default.

thanks :D


Shorten context

I didn't make it all the way through. The property list is shortened but not the classes. @dr0i, please use this version for indexing anyway.

I now also managed to shorten the class list. have a nice weekend.

thx u2 :)

Looking at the full index, I see that there are some properties not compacted. Will adjust the context accordingly.

I guess it is ok for now. We have to re-index it then. I don't intend to do any more work on this  (a few errors will be good for demonstration anyway) so this will be the last indexing round.


Chart to Image

Hi, i was wondering if it's possible to add an option to export all charts to an image format.
I'm planning to use this awesome library to automatize an exorbitant amount of data sources into one application. My final report must be in a plain format, so i'll be needing this feature.
Thanks!

@brogine, i think this can be done with the RenderTargetBitmap directly
a Sample can be found here:
http://stackoverflow.com/questions/2522380/get-a-bitmap-image-from-a-control-view
i used this for other usercontrol but must be work for every wpf controls.

Yeah I agree with @eduard-schwenk

Thank guys. I've ended up using this function:
public static Bitmap ControlToImage(Visual target, double dpiX, double dpiY)
        {
            if (target == null)
            {
                return null;
            }
            // render control content
            Rect bounds = VisualTreeHelper.GetDescendantBounds(target);
            RenderTargetBitmap rtb = new RenderTargetBitmap((int)(bounds.Width * dpiX / 96.0),
                                                            (int)(bounds.Height * dpiY / 96.0),
                                                            dpiX,
                                                            dpiY,
                                                            PixelFormats.Pbgra32);
            DrawingVisual dv = new DrawingVisual();
            using (DrawingContext ctx = dv.RenderOpen())
            {
                VisualBrush vb = new VisualBrush(target);
                ctx.DrawRectangle(vb, null, new Rect(new System.Windows.Point(), bounds.Size));
            }
            rtb.Render(dv);

            //convert image format
            MemoryStream stream = new MemoryStream();
            BitmapEncoder encoder = new BmpBitmapEncoder();
            encoder.Frames.Add(BitmapFrame.Create(rtb));
            encoder.Save(stream);

            return new Bitmap(stream);
        }

Hi everyone,
Above is a solution for WPF.  What about winforms?

What about solution which is not using visual components ? Like generate chart pictures directly?
Thanks

@MartinBodocky @ceds for both cases you can try this sample:
https://github.com/beto-rodriguez/Live-Charts/blob/master/Examples/Wpf/CartesianChart/Chart to Image/ChartToImageSample.xaml.cs

Hi,
I have followed the above mentioned link to generate an image of live graph in my application and it is working fine locally but when I deploy it on azure it is not doing anything. I am getting an empty image that does not contains graph in it further I have checked in logs no error or exception is being thrown. It seems that it is not working on Azure.
Can anybody help me please?

I can reproduce the issue on my local environment now by just commenting the following line of code:
myChart.Update(true, true); //force chart redraw
viewbox.UpdateLayout();
It seems that control cannot be updated on Azure. I have also tried to update the controls through "Dispatcher" but still getting the same issue on Azure.
Please help me to resolve this issue.


Work out how we handle triplicate simulations

The OpenFE default is to run all triplicates in one go. This may be ok for standard transformations, but for net charge transformations this could lead to very large walltime needs.
Should we split the simulation in 3 sets of 1 repeat instead?

@mikemhenry @RiesBen @hannahbaumann as discussed in today's call, please put together your ideas on how you would like this to go and/or how you have been doing this until now.
This includes, but is not limited to, any scripts you have been using (e.g. Mike's 1.0 QA script).

I had forgotten that Mike actually already did what I think might be best:
results_p38_0/edge_AB_complex
results_p38_0/edge_AB_complex.json
results_p38_0/edge_AB_solvent
results_p38_0/edge_AB_solvent.json
results_p38_0/edge_CD_complex

results_p38_1/edge_AB_complex
results_p38_1/edge_AB_complex.json
results_p38_1/edge_AB_solvent
results_p38_1/edge_AB_solvent.json
results_p38_1/edge_CD_complex

results_p38_2/edge_AB_complex
results_p38_2/edge_AB_complex.json
results_p38_2/edge_AB_solvent
results_p38_2/edge_AB_solvent.json
results_p38_2/edge_CD_complex

Are there other options?


Add spec for #8428

I've added not_nil! that was missing for the correct type. I think there will still be some small error in terms of checking for the actual Proc type. If that is so, please instruct how to fix and I will make the final commit.

Since this PR adds the spec for an existing known issue and there is no fix for it I am adding the pr:on-hold-do-not-merge label. Otherwise, we will break the CI :-P

@bcardiff actually it's been fixed, that's why I tried to add the spec, as @jhass suggested in https://github.com/crystal-lang/crystal/issues/8428 .
Result in earlier Crystal versions was a bug: https://play.crystal-lang.org/#/r/7y27
Result in recent versions was fixed: https://carc.in/#/r/990k
I just needed some pointers in actually making the spec work.

Oh, my bad. From the issue I didn't figure it is something already fixed.
Either raise, or if that does not work either you can do
lib LibC
  fun exit(Int32) : NoReturn
end

def create_proc(b : String)
  if true
    begin
      ->{ "" }
    rescue
      return ->{}
    end
  else
    LibC.exit(0)
  end
end

@docelic Do you wish to finish this PR?

The fix is simply to add Nil to the expected types. That does not affect the test's effectiveness.
Adding an else branch with NoReturn breaks the test case. This code even compiles without the fix.
I can't push directly to this PR (probably disabled maintainer access). But this is the diff that makes this work (the first to changes are just format):
--- a/spec/compiler/semantic/proc_spec.cr
+++ b/spec/compiler/semantic/proc_spec.cr
@@ -1215,15 +1215,15 @@ describe "Semantic: proc" do
       def foo
         if true
           begin
-             ->(){""}
+            ->{""}
           rescue
-            return ->(){}
+            return ->{}
           end
         end
       end

-      foo.not_nil!
-    )) { proc_of string, nil_type }
+      foo
+    )) { union_of proc_of(string), proc_of(nil_type), nil_type }
   end
 end

@Sija certainly, I made the changes as suggested by @straight-shoota . So just waiting for the 2nd reviewer I guess.

Can this be merged if we mark it as pending?
(Aside, this would be a good use case for the RSpec feature where when you mark a test as pending it will still execute the test with a reversed test condition.  I tried looking for an open issue for this but I don't see it.  Closest I could find was my comment here: https://github.com/crystal-lang/crystal/issues/10095#issuecomment-762929055 )

@beta-ziliani Why not 1.2.2?

@Sija At some point we need to stabilize the 1.2.2 milestone, and this is clearly not an important topic to rush it to 1.2.2.

@beta-ziliani It's just one passing spec, no possibility of regression, thus I've thought it'll manage to get there :)

It adds a spec. That's important for further development, but completely irrelevant for a release.


/app/run.sh: line 42: /root/go/bin/tesla-keygen: No such file or directory

s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
[22:49:19] INFO: Current Nginx configuration
{"domain":"xaser.duckdns.org","hsts":"max-age=31536000; includeSubDomains","certfile":"fullchain.pem","keyfile":"privkey.pem","cloudflare":false,"customize":{"active":true,"default":"nginx_proxy_default*.conf","servers":"nginx_proxy/*.conf"}}
[22:49:19] INFO: Adding custom config to /share/nginx_proxy/nginx_tesla.conf
server {
server_name XXXXXXXXXXX;
ssl_certificate /ssl/fullchain.pem;
ssl_certificate_key /ssl/privkey.pem;
# dhparams file
ssl_dhparam /data/dhparams.pem;
listen 443 ssl http2;
add_header Strict-Transport-Security "max-age=31536000; includeSubDomains" always;
resolver <IP_ADDRESS>;
set $target c03d64a7-tesla-http-proxy;
    # temporary Flask app for initial auth
    location / {
        proxy_pass http://$target:8099;
    }
    # static public key for Tesla
    location /.well-known/appspecific/com.tesla.3p.public-key.pem {
        root /share/tesla;
        try_files /com.tesla.3p.public-key.pem =404;
    }
}

[22:49:19] INFO: Generating self-signed SSL certificate
[22:49:19] INFO: Generating keypair
/app/run.sh: line 42: /root/go/bin/tesla-keygen: No such file or directory
s6-rc: info: service legacy-services: stopping
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped

Oops, sorry about that! I broke it in the last update. I will fix it tonight.

Fixed in v1.3.3


Roll Flutter Engine from 6a14798358bc to 2f195db9c57c (1 revision)

https://github.com/flutter/engine/compare/6a14798358bc...2f195db9c57c
2022-06-11<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from 6pQd3rR6f... to n7uROVH2O... (flutter/engine#33978)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from 6pQd3rR6fnnP to n7uROVH2OT1H
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


unthemed bits: popups and embed subtitles

^ the channel name here is white, noticeable on light themes

found a few more small things!

spoilers seem to be unthemed
settings tabs (eg. in profiles, the "user profile" and "server profiles" tabs) have a default blurple underline
"1 new" etc. on forum channels are blurple
some muted text things (eg. "enter your pronouns") are the default color
the shadow in grid-view forums are also the default color

i'll see if any of these things i can try to fix sometime

trying to fiddle around with this— side note, discord seem to have some weird overrides in light mode. kind of annoying and i'm not sure if there's a fix for that

Some other examples of popups:

Channel description

Notifications settings

Privacy settings

Invite to server (names devtooled out)

Add status

Add friend nickname

Create/join server

Invite screen

Gift

App launcher

Add app

Remove friend

Failed friend request

Block user

Relax with Wumpus (Ctrl + Shift + Alt + W)

100 server limit (probably)

Some other small things:

Add status bubble in own profile

Pfp hover shadow should be sqaure

Mic test button text cuts off

Download desktop app popup uses wrong accent color

App directory category text cuts off

more issues from the server:

"input" title shows up when making forums
"open in browser" isn't very visible


Authentication

We require a mechanism to authenticate the PS module to the Zendesk API.
I think something like AWS or Azure where we initialise credentials using a cmdlet which are then stored on the machine for use.

Moved to project board.


Dependencies

I think scala-libraryadded by mistake.
        <dependency>
            <groupId>org.scala-lang</groupId>
            <artifactId>scala-library</artifactId>
            <version>${scala.version}</version>
        </dependency>

And I believe it is better to make rocksdb optional for project since the binary is around 20 mb.

rocksdb is now optional


Fix responsive on small screens

With ♥ to Oscar Barajas Jr.

It's not needed to remove the cart image, you could just add an overflow: auto property to div. App and it will fix the background behind 😊

Although it looks good, haha. Decision is up to @gndx

It's not needed to remove the cart image, you could just add an overflow: auto property to div. App and it will fix the background behind 😊

Thanks!! It was more a visual decision, I personally think it makes no sense to have two figures on small screens and scroll just to see the second one.

It's not needed to remove the cart image, you could just add an overflow: auto property to div. App and it will fix the background behind 😊

Thanks!! It was more a visual decision, I personally think it makes no sense to have two figures on small screens and scroll just to see the second one.

Yeah, I actually like it!


Add environment variables to pass additional flags to llvm-cov/llvm-profdata

This adds two environment variables:

CARGO_LLVM_COV_FLAGS to pass additional flags to llvm-cov. (value: space-separated list)
CARGO_LLVM_PROFDATA_FLAGS to pass additional flags to llvm-profdata. (value: space-separated list)

CI failure is due to https://github.com/bitflags/bitflags/issues/251

bors r+


Optimize syscall lookups by sharing the bytes vector

This PR addresses the concern I raised in https://github.com/o1-labs/proof-systems/pull/2276 about reducing the number of witness columns and lookups required to constrain that the bytes read from the oracle are indeed 8-bits long.
After having tested the behavior of the oracle using our demo, I realized that it always reads the 8 leading bytes corresponding to the preimage length in two instructions reading 4 bytes in each row. This means, it never shares both length bytes with preimage bytes in the same row.
This simplifies the approach in the previous PR, because now we checked that the vector of bytes can contain length bytes (always 4) or preimage bytes (at most 4). The rest remains unchanged, meaning has_n_bytes still refers to preimage bytes. Keeping that meaning results in no changes required for the constraints. And more importantly, this allows us to bring down the number of extra lookups from 8 to 4 per SyscallReadPreimage instruction.

As part of locally running the syscall instruction as a unit test I found out that this is not necessarily true. Specially, if the addr passed to request_preimage_write is not aligned. In that case, one can start by reading a non-4 amount of bytes of the length, and then the third (?) call could contain both length bytes and real preimage bytes.

https://github.com/o1-labs/proof-systems/issues/2312


Fix pytest 8 issues

[ ] Closes #xxxx
[ ] Tests added / passed
[ ] Passes pre-commit run --all-files

pinning pytest for now....

I pinged @rjzamora on slack for the gpu ci

@I pinged @rjzamora on slack for the gpu ci
@phofl - Just to clarify. This PR no longer pins the pytest version, correct?

Yep sorry for not updating you, turns out Moto caused the remaining failures and pinning it fixed our issues


Sub IF evaluate even if in evaluate false side

Hello,
I meet an issue with EvalEx
Let us look at a concrete example:
IF ( a = 0, 1, IF (2/a > 1, 2, 3))

When a=0, the 2nd IF condition is evaluated and it crash with a division by 0, as the 2nd IF, in this case, is in the false result of the 1rst IF.

This might be related to #194.

Yes it's seems to be same problem.
I have a look at https://github.com/gnembon/carpetmod/blob/5ec533ff6a158015bc8cdc3da028238ea6aa1ba4/src/main/java/carpet/script/Expression.java#L1353 but there's lot of changes and some breaking change with existing public method signature.
For instance, in EvalEx:
public Expression setVariable(String variable, BigDecimal value) {...}
public Expression setVariable(String variable, LazyNumber value) {...}
public Expression setVariable(String variable, String value) {...}

And in gnembon version:
public Expression setVariable(String variable, Value value) {}
public Expression setVariable(String variable, LazyValue value) {}

What do you suggest ?

If you can come up with a solution, please do so. The best course of action would be to look through all evaluations and swap everything that isn't already using LazyNumbers to that, but that's quite a piece of work. More unit tests for that would also be nice.
I'd love to do that, but I'm very short on time currently, that won't change for the next few months. So if you can come up with something, please do so and I'll happily review and test it.

I met same problem


ARXIVNG-308 user may optionally search by author full name (text match, not keyword)

There is now a dijunct full-name field on the search form....

The full_name field has this mapping:
            "full_name": {
              "type": "text",
              "analyzer": "folding"
            },

with a new analyzer:
        "folding": {
          "type": "custom",
          "tokenizer": "standard",
          "filter": [
            "icu_folding",
            "lowercase"
          ]
        },

Also changes the way that they query is displayed in the search results:

Coverage decreased (-0.6%) to 66.7% when pulling cc5f94881e627ad07d0707201aa667efaa21341b on task/ARXIVNG-309 into a1fcfbeb757e8c29163afe00a5cabec10ef3f52a on develop.

This fixes part of #38


Cloning objects

I'd love to be able to easily clone an object, through serializable, but cannot...
  static Foo clone(Foo foo) {
    return Foo.fromJson(foo.toJson());
  }

Here is an example from your own test code of why it fails:
https://github.com/dart-lang/json_serializable/blob/master/json_serializable/test/integration/json_test_example.g.dart#L21
https://github.com/dart-lang/json_serializable/blob/master/json_serializable/test/integration/json_test_example.g.dart#L42
The fromJson fails because order is attempted to be cast to a Map<String, dynamic>.
One solution would be in the PersonToJson method, call order: instance.order.toJson(). Another would simply be a guard around the cast that if it fails, attempt to call the .toJson() method.
I've seen prior art in #57 and #105, but I don't see why this cannot be fixed in the generated code.

@lookfirst If you still looking for that approach,
this library https://github.com/k-paxian/dart-json-mapper supports .clone method.

@k-paxian This looks fantastic! Thanks.


CORS error is missing in the preflight call

hello! we re getting a CORS error is missing, even though we added the nedded headers. after investigating we saw that in the preflight call the headers look like this:
OPTIONS /url...  Any idea if this has happened before or what we can do?
Thanks a lot!

@karakostasi Can you share your code snippet on how you use SSE and configure the headers you're trying to send?
The sse.js library doesn't do any transformation on the headers, it just passes them on to the underlying XMLHttpRequest:
    for (let header in this.headers) {
      this.xhr.setRequestHeader(header, this.headers[header]);
    }


Project not maintained

Description of the bug:

We value this project. Please provide updates to the project as there are lots of bug. Its a great library which needs to be maintained and adds a lot of value when it comes to deploying resources into kubernetes cluster. Please provide updates to this project and fix its bugs.
Reproduction Steps:

Error Log:

Environment:

Framework Version:
OS:

Other:

This is :bug: Bug Report

@cloudbow I think it's also up to the users to fix the bugs, do you have some that are critical for you ?
I'm not part of the dev team


⬆ UPGRADE: Autoupdate pre-commit config template

Base PullRequest
default branch (https://github.com/s-weigand/cookiecutter-pypackage/tree/main)
Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/build/src/bin

pip install pre-commit
Collecting pre-commit
  Downloading pre_commit-2.20.0-py2.py3-none-any.whl (199 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 199.5/199.5 kB 12.9 MB/s eta 0:00:00
Collecting virtualenv>=20.0.8
  Downloading virtualenv-20.16.5-py3-none-any.whl (8.8 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 8.8/8.8 MB 119.5 MB/s eta 0:00:00
Collecting nodeenv>=0.11.1
  Downloading nodeenv-1.7.0-py2.py3-none-any.whl (21 kB)
Collecting identify>=1.0.0
  Downloading identify-2.5.5-py2.py3-none-any.whl (98 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 98.7/98.7 kB 37.2 MB/s eta 0:00:00
Collecting toml
  Downloading toml-0.10.2-py2.py3-none-any.whl (16 kB)
Collecting pyyaml>=5.1
  Downloading PyYAML-6.0-cp310-cp310-manylinux_2_5_x86_64.manylinux1_x86_64.manylinux_2_12_x86_64.manylinux2010_x86_64.whl (682 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 682.2/682.2 kB 103.3 MB/s eta 0:00:00
Collecting cfgv>=2.0.0
  Downloading cfgv-3.3.1-py2.py3-none-any.whl (7.3 kB)
Requirement already satisfied: setuptools in /opt/hostedtoolcache/Python/3.10.7/x64/lib/python3.10/site-packages (from nodeenv>=0.11.1->pre-commit) (63.2.0)
Collecting platformdirs<3,>=2.4
  Downloading platformdirs-2.5.2-py3-none-any.whl (14 kB)
Collecting filelock<4,>=3.4.1
  Downloading filelock-3.8.0-py3-none-any.whl (10 kB)
Collecting distlib<1,>=0.3.5
  Downloading distlib-0.3.6-py2.py3-none-any.whl (468 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 468.5/468.5 kB 101.7 MB/s eta 0:00:00
Installing collected packages: distlib, toml, pyyaml, platformdirs, nodeenv, identify, filelock, cfgv, virtualenv, pre-commit
Successfully installed cfgv-3.3.1 distlib-0.3.6 filelock-3.8.0 identify-2.5.5 nodeenv-1.7.0 platformdirs-2.5.2 pre-commit-2.20.0 pyyaml-6.0 toml-0.10.2 virtualenv-20.16.5

pre-commit autoupdate || (exit 0);
Updating https://github.com/pre-commit/pre-commit-hooks ... [INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks.
already up to date.
Updating https://github.com/python/black ... [INFO] Initializing environment for https://github.com/python/black.
already up to date.
Updating https://github.com/pre-commit/mirrors-prettier ... [INFO] Initializing environment for https://github.com/pre-commit/mirrors-prettier.
already up to date.
Updating https://gitlab.com/pycqa/flake8 ... [INFO] Initializing environment for https://gitlab.com/pycqa/flake8.
already up to date.
Updating https://github.com/PyCQA/isort ... [INFO] Initializing environment for https://github.com/PyCQA/isort.
already up to date.

pre-commit autoupdate -c \{\{cookiecutter.project_slug\}\}/.pre-commit-config.yaml || (exit 0);
Updating https://github.com/pre-commit/pre-commit-hooks ... already up to date.
Updating https://github.com/asottile/pyupgrade ... [INFO] Initializing environment for https://github.com/asottile/pyupgrade.
already up to date.
Updating https://github.com/MarcoGorelli/absolufy-imports ... [INFO] Initializing environment for https://github.com/MarcoGorelli/absolufy-imports.
already up to date.
Updating https://github.com/PyCQA/isort ... already up to date.
Updating https://github.com/python/black ... already up to date.
Updating https://github.com/pre-commit/mirrors-prettier ... already up to date.
Updating https://github.com/asottile/setup-cfg-fmt ... [INFO] Initializing environment for https://github.com/asottile/setup-cfg-fmt.
already up to date.
Updating https://github.com/pycqa/flake8 ... [INFO] Initializing environment for https://github.com/pycqa/flake8.
already up to date.
Updating https://github.com/asottile/yesqa ... [INFO] Initializing environment for https://github.com/asottile/yesqa.
already up to date.
Updating https://github.com/pre-commit/mirrors-mypy ... [INFO] Initializing environment for https://github.com/pre-commit/mirrors-mypy.
updating v0.971 -> v0.981.
Updating https://github.com/PyCQA/pydocstyle ... [INFO] Initializing environment for https://github.com/PyCQA/pydocstyle.
already up to date.
Updating https://github.com/terrencepreilly/darglint ... [INFO] Initializing environment for https://github.com/terrencepreilly/darglint.
already up to date.
Updating https://github.com/econchick/interrogate ... [INFO] Initializing environment for https://github.com/econchick/interrogate.
already up to date.
Updating https://github.com/myint/rstcheck ... [INFO] Initializing environment for https://github.com/myint/rstcheck.
already up to date.
Updating https://github.com/pre-commit/pygrep-hooks ... [INFO] Initializing environment for https://github.com/pre-commit/pygrep-hooks.
already up to date.
Updating https://github.com/codespell-project/codespell ... [INFO] Initializing environment for https://github.com/codespell-project/codespell.
already up to date.
Updating https://github.com/econchick/interrogate ... already up to date.

Changed files

Changed file:

{{cookiecutter.project_slug}}/.pre-commit-config.yaml

:octocat: Repo | :memo: Issues | :department_store: Marketplace

Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/build/src/bin

pip install pre-commit
Collecting pre-commit
  Downloading pre_commit-2.20.0-py2.py3-none-any.whl (199 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 199.5/199.5 kB 11.7 MB/s eta 0:00:00
Collecting pyyaml>=5.1
  Downloading PyYAML-6.0-cp310-cp310-manylinux_2_5_x86_64.manylinux1_x86_64.manylinux_2_12_x86_64.manylinux2010_x86_64.whl (682 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 682.2/682.2 kB 62.3 MB/s eta 0:00:00
Collecting cfgv>=2.0.0
  Downloading cfgv-3.3.1-py2.py3-none-any.whl (7.3 kB)
Collecting toml
  Downloading toml-0.10.2-py2.py3-none-any.whl (16 kB)
Collecting virtualenv>=20.0.8
  Downloading virtualenv-20.16.5-py3-none-any.whl (8.8 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 8.8/8.8 MB 115.1 MB/s eta 0:00:00
Collecting identify>=1.0.0
  Downloading identify-2.5.6-py2.py3-none-any.whl (98 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 98.7/98.7 kB 32.2 MB/s eta 0:00:00
Collecting nodeenv>=0.11.1
  Downloading nodeenv-1.7.0-py2.py3-none-any.whl (21 kB)
Requirement already satisfied: setuptools in /opt/hostedtoolcache/Python/3.10.7/x64/lib/python3.10/site-packages (from nodeenv>=0.11.1->pre-commit) (63.2.0)
Collecting filelock<4,>=3.4.1
  Downloading filelock-3.8.0-py3-none-any.whl (10 kB)
Collecting platformdirs<3,>=2.4
  Downloading platformdirs-2.5.2-py3-none-any.whl (14 kB)
Collecting distlib<1,>=0.3.5
  Downloading distlib-0.3.6-py2.py3-none-any.whl (468 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 468.5/468.5 kB 86.7 MB/s eta 0:00:00
Installing collected packages: distlib, toml, pyyaml, platformdirs, nodeenv, identify, filelock, cfgv, virtualenv, pre-commit
Successfully installed cfgv-3.3.1 distlib-0.3.6 filelock-3.8.0 identify-2.5.6 nodeenv-1.7.0 platformdirs-2.5.2 pre-commit-2.20.0 pyyaml-6.0 toml-0.10.2 virtualenv-20.16.5

pre-commit autoupdate || (exit 0);
Updating https://github.com/pre-commit/pre-commit-hooks ... [INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks.
already up to date.
Updating https://github.com/python/black ... [INFO] Initializing environment for https://github.com/python/black.
updating 22.8.0 -> 22.10.0.
Updating https://github.com/pre-commit/mirrors-prettier ... [INFO] Initializing environment for https://github.com/pre-commit/mirrors-prettier.
updating v3.0.0-alpha.0 -> v3.0.0-alpha.1.
Updating https://gitlab.com/pycqa/flake8 ... [INFO] Initializing environment for https://gitlab.com/pycqa/flake8.
already up to date.
Updating https://github.com/PyCQA/isort ... [INFO] Initializing environment for https://github.com/PyCQA/isort.
already up to date.

pre-commit autoupdate -c \{\{cookiecutter.project_slug\}\}/.pre-commit-config.yaml || (exit 0);
Updating https://github.com/pre-commit/pre-commit-hooks ... already up to date.
Updating https://github.com/asottile/pyupgrade ... [INFO] Initializing environment for https://github.com/asottile/pyupgrade.
updating v2.38.2 -> v3.0.0.
Updating https://github.com/MarcoGorelli/absolufy-imports ... [INFO] Initializing environment for https://github.com/MarcoGorelli/absolufy-imports.
already up to date.
Updating https://github.com/PyCQA/isort ... already up to date.
Updating https://github.com/python/black ... updating 22.8.0 -> 22.10.0.
Updating https://github.com/pre-commit/mirrors-prettier ... updating v3.0.0-alpha.0 -> v3.0.0-alpha.1.
Updating https://github.com/asottile/setup-cfg-fmt ... [INFO] Initializing environment for https://github.com/asottile/setup-cfg-fmt.
already up to date.
Updating https://github.com/pycqa/flake8 ... [INFO] Initializing environment for https://github.com/pycqa/flake8.
already up to date.
Updating https://github.com/asottile/yesqa ... [INFO] Initializing environment for https://github.com/asottile/yesqa.
already up to date.
Updating https://github.com/pre-commit/mirrors-mypy ... [INFO] Initializing environment for https://github.com/pre-commit/mirrors-mypy.
updating v0.981 -> v0.982.
Updating https://github.com/PyCQA/pydocstyle ... [INFO] Initializing environment for https://github.com/PyCQA/pydocstyle.
already up to date.
Updating https://github.com/terrencepreilly/darglint ... [INFO] Initializing environment for https://github.com/terrencepreilly/darglint.
already up to date.
Updating https://github.com/econchick/interrogate ... [INFO] Initializing environment for https://github.com/econchick/interrogate.
already up to date.
Updating https://github.com/myint/rstcheck ... [INFO] Initializing environment for https://github.com/myint/rstcheck.
already up to date.
Updating https://github.com/pre-commit/pygrep-hooks ... [INFO] Initializing environment for https://github.com/pre-commit/pygrep-hooks.
already up to date.
Updating https://github.com/codespell-project/codespell ... [INFO] Initializing environment for https://github.com/codespell-project/codespell.
already up to date.
Updating https://github.com/econchick/interrogate ... already up to date.

Changed files

Changed 2 files:

.pre-commit-config.yaml
{{cookiecutter.project_slug}}/.pre-commit-config.yaml

:octocat: Repo | :memo: Issues | :department_store: Marketplace


Fields loaded are more than in the Model definition

Hi.
Great job but I have encountered maybe a trouble:
If I use the 'find' method of my little example in that I defined only two fields (username and email), if there are more fields in the database (firstname, lastname, username, email, password, ...), Typegoose load all the field. If I use the mongoose schema, mongoose load only the two field. It seems like when you define an empty schema.
Can you explain me that?
Below I attached my source files: server, the model and a route page.
user.model.ts
IndexRoute.ts
server.ts
Thank you! Typegoose is really THE typescript solution for mongoose.
EDIT (06/oct/2017):
I found the solution using the 'projection' parameter of the 'find' method.
I don't know if there is a typegoose method or a better idea to compose the projection string:
let proj = Object.keys(UserModel.schema.obj).join(' ');
public index(req: Request, res: Response, next: NextFunction) {
       let proj = Object.keys(UserModel.schema.obj).join(' ');

        UserModel.find({}, proj, (err, data) => {
            if(err) {
                res.status(200).json({msg: 'err', "err": err});
                return next(err);
            }
            res.status(200).json({msg: 'ok', "data": data});
           // prints:
           // {"msg":"ok","data"<EMAIL_ADDRESS>        });

    }

Hello. That is an interesting finding. Actually Typegoose under the hood composes a Mongoose schema using the props defined by your class, no more. It wont scan the db for pre existing fields, so it is really interesting.
Anyway I will try to reproduce the problem when I will have the time and will get back to you with the results.

Hi @pgsfredda
Based on your edit in your issue I close this issue.
I paste your solution so if someone from google ends up here, will find it faster.
Cheers,
Daniel
EDIT (06/oct/2017):
I found the solution using the 'projection' parameter of the 'find' method.
I don't know if there is a typegoose method or a better idea to compose the projection string:


Fix issue#385 rewriting the handle method

This is another alternative to solve the issue#385 without any drawbacks.
I rewrite the handle method to execute the tenancy commands adding website_id and realpath params.

Will pick this up asap, unless @Thijmen does.

@luceos I think this needs attention. 😄


Would it be possible to add a settings do disable auto-scroll to the bottom?

Since it is quite annoying when one wants to read already outputted content from one bot, while the other one is still outputting, since it keeps scolling to the bottom erratically.
I would like it best if this would be configurable and if the page is scrolled to the bottom once the message is sent, so the bot outputs are in view.

Just scroll up a little, the auto-scroll to bottom will stop

I just saw this issue and have already implemented it locally 😂


Silencing ImportErrors raised in modules referenced by operationId

To reproduce simply create an incorrect import that will raise an ImportError in a module that is referenced by operationId of an endpoint specified in a swagger API.
project/swagger/api.yaml
---------------------------------
...
paths:
    /scms:
         get:
             operationId:
                  api.scms.test
...

project/src/api/scms.py
---------------------------------
#broken import
import something_broken

def test():
    return {'test': 'ok'}, 200

When URL /api/scms is requested the module raises an ImportError and the test method is not reachable, but connexion proceeds and it says:
{"status": 404, "type": "about:blank", "detail": "The requested URL was not found on the server.  If you entered the URL manually please check your spelling and try again.", "title": "Not Found"}
without re-raising the exception so it is not clear what failed.

:+1: good point, better error messages would be helpful here --- maybe just log an error?

I would rather re-raise the ImportError. Logging the error would have the following meaning for me:
Some of your views could not be imported, but the applications has been started anyway.
I'd rather prevent application from starting when something could not be imported and show why and where it failed. To me that seems to be the most reasonable choice.

I agree with @jkozlowicz even more because it is a route that will be unavailable and your specification is saying that it should be available. So it should fail badly and not start the server in that case.

That is actually the problem: https://github.com/zalando/connexion/pull/28
It was intentionally added to keep the application going, there is even a test for it. Connexion keeps going even if there is a problem adding a operation. @hjacobs @jmcs Should we change that?
I still agree that this kind of problem should not be silenced. But maybe you have any use case where it is useful?


Fix anchor element selector to support usage of React.useId()

Closes #836.

Thanks for the PR @gabrieljablonski!


[bazel] targets fail to build if $ANDROID_NDK_HOME is not set

This is an underlying Bazel issue where android_ndk_repository is fetched eagerly even when there are no Android targets to build. See bazelbuild/bazel#14260 for more information.
To workaround:

If you're not planning to build Android targets, you can simply comment out the android_ndk_repository rule from WORKSPACE.bazel as follows:# android_ndk_repository(
#     name = "androidndk",
# )
# register_toolchains("@androidndk//:all")

If you're targeting Android, you must set the ANDROID_NDK_HOME environment variable to your Android NDK root directory. Alternatively, you can set the path and api-level fields explicitly as documented here: https://bazel.build/docs/android-ndk
Please also note that there is yet another Bazel issue that prevents building with Android NDK versions later than R21.x. See bazelbuild/bazel#12889 for more information.

dropped bazel support completely due to the lack of official support -- in favor of fully migrating to cmake via #134


A Basic Expense Tracker web app with React

I want to add a Expense Tracker web app  project built with React js.

Can you please give some awesome brief descriptions about this project?

Its a basic webApp for storing expenses by a particular date.

There is nothing awesome about this. its just a basic project which stores expenses by particular dates. it was project 0 on a course.

Hey @SxxAq we are not accepting React Projects


Function localrun quits when using partitioned topic.

First i write a demo package by gradle
plugins {
    id 'java'
    id "com.github.johnrengelman.shadow" version "2.0.4"
}

package com.codelipenghui.pulsar.function;

import org.apache.pulsar.functions.api.Context;
import org.apache.pulsar.functions.api.Function;

public class WordCountFunction implements Function<String, Void> {

    @Override
    public Void process(String input, Context context) throws Exception {
        context.incrCounter("test-case-total", 1);
        return null;
    }
}

I got info :
[root@qa-5-171 apache-pulsar-2.2.0-incubating-SNAPSHOT]# bin/pulsar-admin functions localrun --jar target/pulsar-functions-0.1.0-SNAPSHOT-jar-with-dependencies.jar --className pulsarfunctions.starter.sdk.WordCountFunction --tenant public --namespace default --name word-count-test --inputs persistent://public/default/plat.correctness.verification --output persistent://public/default/word-count-test-output
19:58:34.946 [main] INFO  org.apache.pulsar.functions.runtime.ProcessRuntimeFactory - Java instance jar location is not defined, using the location defined in system environment : /opt/apache-pulsar-2.2.0-incubating-SNAPSHOT/instances/java-instance.jar
19:58:34.954 [main] INFO  org.apache.pulsar.functions.runtime.ProcessRuntimeFactory - Python instance file location is not defined using the location defined in system environment : /opt/apache-pulsar-2.2.0-incubating-SNAPSHOT/instances/python-instance/python_instance_main.py
19:58:34.974 [main] INFO  org.apache.pulsar.functions.runtime.RuntimeSpawner - RuntimeSpawner starting function word-count-test - 0
19:58:34.987 [main] INFO  org.apache.pulsar.functions.runtime.ProcessRuntime - ProcessBuilder starting the process with args java -cp /opt/apache-pulsar-2.2.0-incubating-SNAPSHOT/instances/java-instance.jar -Dpulsar.functions.java.instance.jar=/opt/apache-pulsar-2.2.0-incubating-SNAPSHOT/instances/java-instance.jar -Dlog4j.configurationFile=java_instance_log4j2.yml -Dpulsar.log.dir=/opt/apache-pulsar-2.2.0-incubating-SNAPSHOT/logs/functions -Dpulsar.log.file=word-count-test org.apache.pulsar.functions.runtime.JavaInstanceMain --jar target/pulsar-functions-0.1.0-SNAPSHOT-jar-with-dependencies.jar --instance_id 0 --function_id ac8f5862-34cd-4f0e-869e-39f5717a99a2 --function_version 1498f7f5-5483-42c2-b2ac-c07c5c1626ce --tenant public --namespace default --name word-count-test --function_classname pulsarfunctions.starter.sdk.WordCountFunction --auto_ack true --processing_guarantees ATLEAST_ONCE --pulsar_serviceurl http://localhost:8080/ --use_tls false --tls_allow_insecure false --hostname_verification_enabled false --max_buffered_tuples 1024 --port 16821 --source_type_classname "java.lang.String" --source_subscription_type SHARED --source_topics_serde_classname {"persistent://public/default/plat.correctness.verification":""} --sink_type_classname "java.lang.Void" --sink_topic persistent://public/default/word-count-test-output
19:58:34.995 [main] INFO  org.apache.pulsar.functions.runtime.ProcessRuntime - Started process successfully
19:58:41.297 [main] INFO  org.apache.pulsar.admin.cli.CmdFunctions - RuntimeSpawner quit because of
19:58:41.299 [Thread-3] INFO  org.apache.pulsar.admin.cli.CmdFunctions - Shutting down the localrun runtimeSpawner ...

And then i use https://github.com/streamlio/pulsar-functions-java-starter.git
I got the same.

Hi,
Just tried it from pulsar master(your logs indicated that you were on 2.2.0-incubating) and was able to start the function. Which exact pulsar branch/head that you saw this error?

master branch on monday.
ok i will retry in master branch today.

I think the problem is state storage is not enabled in cluster mode by default. I am updating the documentation in #2335 . I will verify local run and update here.

Topic of plat.correctness.verification is a partitioned topic with six partitions.
When i set input to plat.correctness.verification local run will quit.
When i set input to plat.correctness.verification-partition-0 local run can process success.
# Local run success
bin/pulsar-admin functions localrun --jar target/function-starter-1.0.0-all.jar --className com.codelipenghui.pulsar.function.WordCountFunction --inputs persistent://public/default/plat.correctness.verification-partition-0 --output persistent://public/default/plat.correctness.verification.wordcount.output --name plat-correctness-verification-wordcount

# Local run quit
bin/pulsar-admin functions localrun --jar target/function-starter-1.0.0-all.jar --className com.codelipenghui.pulsar.function.WordCountFunction --inputs persistent://public/default/plat.correctness.verification --output persistent://public/default/plat.correctness.verification.wordcount.output --name plat-correctness-verification-wordcount

I had a discussion with @codelipenghui .
the original question was around stateful function. stateful function was shipped as a preview feature in 2.1. however it is not fully integrated in cluster mode. so I will be working on that piece in #2335 .
However there is a bigger problem around partitioned-topic. It seems there is some regression in partitioned topic in 2.1, which pulsar functions can't run with partitioned-topic. because we switched to using multi-topic subscription in 2.1, so it will return a partition name instead of topic name, which cause function runtime confused on finding a suitable serde to deserialize the message and it throws RuntimeException and quits.
I marked this issue as bug and updated the issue title to be more specific. this fix should also be cherry-picked to 2.1.1 release as well.


build: restrict building bst for only linux platforms

Currently bst cannot build or run on platforms other than linux. Headers like
prctl.h, linux/capability.h etc are only available on Linux and not on other
platforms. Hence, the meson build process should check the target platform (the
platform for which the binary is being built) and bail out for non-supported
platforms. For example, when trying to build bst on my Mac's darwin platform, I
get the following descriptive error with this change:
Running test binary command: /Users/ani/workspace/bst/build/meson-private/sanitycheckc.exe
C compiler for the build machine: cc (clang 12.0.5 "Apple clang version 12.0.5 (clang-1<IP_ADDRESS>)")
C linker for the build machine: cc ld64 650.9
Build machine cpu family: x86_64
Build machine cpu: x86_64
Host machine cpu family: x86_64
Host machine cpu: x86_64
Target machine cpu family: x86_64
Target machine cpu: x86_64
Program sh found: YES (/bin/sh)
Program git found: YES (/usr/bin/git)
Program scdoc found: NO
meson.build:19:1: ERROR: Problem encountered: bst is not supported on darwin platform.
Signed-off-by: Ani Sinha<EMAIL_ADDRESS>

Ping

LGTM, thanks for the change!


Question: Why is MemoryFileUpload not released by DefaultHttpDataFactory?

Expected behavior
I expect any MemoryFileUploads to be released when any of the cleanAll methods are called
Actual behavior
Mixed and Disk uploads are cleaned, but memory isn't. This seems intentional as the same logic is applied to attributes.
The upload/attributes are never added to the list that gets iterated over when cleaning
https://github.com/netty/netty/blob/4.1/codec-http/src/main/java/io/netty/handler/codec/http/multipart/DefaultHttpDataFactory.java#L245
Netty version
4.1.30
JVM version (e.g. java -version)
Java 8
OS version (e.g. uname -a)
macOS

@normanmaurer Sure thing

@normanmaurer Done https://github.com/netty/netty/pull/9672

@jameskleeh Indeed, that was intentional. Maybe wrongly, since you feel it should be included...
Your PR seems OK to change the behavior, but could you also fix the comments on the related "clean" methods, such that it does not explicitely stands for "disk" only (or mixed so mixed too) HttpData ?

@fredericBregier Sure thing

@fredericBregier I don't see what comments you are referring to. Can you point me in the right direction?

Sure:
https://github.com/netty/netty/blob/c70254e3af242c6134063bbb3d6f4ade8775a4ff/codec-http/src/main/java/io/netty/handler/codec/http/multipart/HttpDataFactory.java#L75
https://github.com/netty/netty/blob/c70254e3af242c6134063bbb3d6f4ade8775a4ff/codec-http/src/main/java/io/netty/handler/codec/http/multipart/HttpDataFactory.java#L80
https://github.com/netty/netty/blob/c70254e3af242c6134063bbb3d6f4ade8775a4ff/codec-http/src/main/java/io/netty/handler/codec/http/multipart/HttpData.java#L121
(even if this one was already wrong, since in AbstractMemoryHttpData, the delete method indeed call release on bytebuf)
I don't go everywhere, but I think this are the main ones.
Of course, we might also add a new method clean duplicate of :
https://github.com/netty/netty/blob/c70254e3af242c6134063bbb3d6f4ade8775a4ff/codec-http/src/main/java/io/netty/handler/codec/http/multipart/HttpPostStandardRequestDecoder.java#L660
since it now will delete all, even not files, in order to be clean (even if cleanFile will delete all). Or just changing the associated comment to ensure the new behavior is understood.
And same on https://github.com/netty/netty/blob/c70254e3af242c6134063bbb3d6f4ade8775a4ff/codec-http/src/main/java/io/netty/handler/codec/http/multipart/HttpPostMultipartRequestDecoder.java#L949


attend to uncaught promise error in configuration.ts

When I have the console open, I often see something like this when launching the app
Uncaught (in promise) Error: Error: your settings are undefined. Try reloading the page. If this error persists, try the troubleshooting guide: https://github.com/glacambre/firenvim/blob/master/TROUBLESHOOTING.md
    at getConfForUrl (configuration.ts:163:15)
    at getConf (configuration.ts:149:12)
    at content.ts:299:51

How did you launch the app? I ran it on my development server, and I don't see this error on the console. I also ran the actual webapp and checked the console, but this is not the error that shows up. Could you please provide the steps to reproduce this issue?

I'm getting this in Chrome today when I load the attached csv.

MathStat-2022-23-current.csv


Setup GitHub Actions

Is your feature request related to a problem? Please describe.

Client wants to have GitHub actions available to deployment and CI

Describe the solution you'd like

Configure GitHub actions.

Master branch for the deployment.
Automatic deploy when PR is accepted in master branch.
Pass test before deploy.
The branch for make the PR to master will be a QA branch
QA branch receive code from Dev branch and run test before PR accepting

Done first approach to Actions


Added checking for sandbox_mode, w backwards compatibility

Coverage decreased (-2.6%) to 90.592% when pulling 28cb8ef46e7aca0388966214a0ce681e1a27063e on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.592% when pulling f1b9eab3ab3fe2db92389d8274e35cb277a0b735 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.592% when pulling 74360ab1775701b18c004d120b5583a6737f3deb on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.592% when pulling ff9469c8644c7ea0fc371c1ab874e52ebf5150c7 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.592% when pulling 492f2e82e3aaf56f2adcf5a23626621759e5bff7 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.592% when pulling d411eb21eaf1f933c8fa0b00057c75477d064e4b on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.592% when pulling ffa321dbb9d6291c153a38a9f73eaf59860e41b0 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.592% when pulling 462d99fbe9af42782f9353a7050dbf9496caefb3 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.592% when pulling 388d109c2f0940e97fbd0e47401983c4596962f2 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.625% when pulling 6e33161b2f2a82481f1cdefb7d6ddc331ce614db on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.625% when pulling fd92326b963d1b5fc6236c4a91ab2d410f3e291e on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.625% when pulling f498c294f306802452799f1f5f13f114601ff991 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.625% when pulling ac71b60f313da1b1724022583add4dd96bf9dcf3 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage increased (+0.2%) to 93.403% when pulling a76517505a85e3e83f938262a5505b10c216e5a2 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage increased (+0.2%) to 93.403% when pulling e42549607906b92a8d25699c71ba66516179b99a on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.625% when pulling 4bc9e84dc478130661a78d89ffe99f25e862c1e7 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.625% when pulling 9ec7589a89202e3a153b3ed5615c9582783bed30 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage decreased (-2.6%) to 90.625% when pulling 43da63738c833e9b5edff2ebaab8e4e8960a451b on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage increased (+0.2%) to 93.403% when pulling cbeff97d9491f4af1e6a02d7696c67c53b77f32b on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Coverage increased (+0.2%) to 93.403% when pulling 8f17ed701b931f557427f604b440c0091d1a01f7 on dev_sandbox_filter into 122fc1e5e16050ba90a371050f737dfbfe41cf01 on sandbox_filter.

Changes Unknown when pulling b71b931a2b92e59b2d0c8b2c00d762d04fe02546 on dev_sandbox_filter into ** on sandbox_filter**.

Changes Unknown when pulling b71b931a2b92e59b2d0c8b2c00d762d04fe02546 on dev_sandbox_filter into ** on sandbox_filter**.

Coverage increased (+0.2%) to 93.559% when pulling 30edc7957115f654b8de80d22269381653944971 on dev_sandbox_filter into d996beee691eba4681b9ea1e7bca4728fb450f15 on sandbox_filter.

Coverage decreased (-4.3%) to 89.153% when pulling dc784850131f7788bd90c4d8693bb707f61e66b7 on dev_sandbox_filter into d996beee691eba4681b9ea1e7bca4728fb450f15 on sandbox_filter.

Coverage increased (+0.2%) to 93.559% when pulling 3bd67b907186ad7fa14a8a439dd6eaf3a287416a on dev_sandbox_filter into d996beee691eba4681b9ea1e7bca4728fb450f15 on sandbox_filter.

Coverage increased (+0.1%) to 93.537% when pulling 5d09913eabbff9086bc648f8eaf12780480a0c7f on dev_sandbox_filter into d996beee691eba4681b9ea1e7bca4728fb450f15 on sandbox_filter.

Coverage increased (+0.1%) to 93.537% when pulling 8343250aa0fd95b0f48c572cf2ae49393246a9bd on dev_sandbox_filter into d996beee691eba4681b9ea1e7bca4728fb450f15 on sandbox_filter.

Coverage increased (+0.1%) to 93.537% when pulling 8343250aa0fd95b0f48c572cf2ae49393246a9bd on dev_sandbox_filter into d996beee691eba4681b9ea1e7bca4728fb450f15 on sandbox_filter.

Coverage decreased (-5.5%) to 87.879% when pulling ac8c38888a2d06e38e0bdefe459d152a9a7b9ed4 on dev_sandbox_filter into d996beee691eba4681b9ea1e7bca4728fb450f15 on sandbox_filter.

Coverage increased (+0.2%) to 93.603% when pulling e6e8cddd213f7c22901397cd9459733d55cdbc8d on dev_sandbox_filter into d996beee691eba4681b9ea1e7bca4728fb450f15 on sandbox_filter.

Coverage decreased (-5.5%) to 87.879% when pulling 44e327a940ccfec311ff67ca8da2d8e9b49fe142 on dev_sandbox_filter into d996beee691eba4681b9ea1e7bca4728fb450f15 on sandbox_filter.

Coverage increased (+0.2%) to 93.603% when pulling 819910f9cdbea9f9e7df958241bcad79b191556b on dev_sandbox_filter into d996beee691eba4681b9ea1e7bca4728fb450f15 on sandbox_filter.


codec/common/inc/asmdefs_mmi.h: fix mips32 build

Fix the following build failure on mips32 which is raised since version 2.0.0 and https://github.com/cisco/openh264/commit/b13e5bceb18ebb93d0313b46aab4af6f480ca933:
codec/common/mips/copy_mb_mmi.c: In function 'WelsCopy16x16_mmi':
./codec/common/inc/asmdefs_mmi.h:293:21: error: '_ABI64' undeclared (first use in this function)
  293 |    if (_MIPS_SIM == _ABI64)                                    \
      |                     ^~~~~~

Fixes:

http://autobuild.buildroot.org/results/cba3e9d0fd061cc3a92cb732bcdc2c7b66dbf6cb

Signed-off-by: Fabrice Fontaine<EMAIL_ADDRESS>

LGTM


Feature request: Make the CloudFront Distribution available as a property

Use case
I would like to use the CloudFront Distribution domain to configure a Congito User Pool Client.
Solution/User Experience
I would like to be able to do:
const { distribution } = new Hosting(this, 'Hosting', {...});

Alternative solutions
Currently, I have used `tryFindChild` on the construct node itself to drill down to what I need.

The CloudFront Distribution is now exposed as a public readonly property on the Hosting construct.
You can now access the CloudFront Distribution like this:
const { distribution } = new Hosting(this, 'Hosting', {...});


Validation fails when all controls on the form are disabled and all have values

Description
Validation fails when all controls on the form are disabled and all have values
Minimal Reproduction

Open official example: https://formly.dev/docs/examples/validation/toggle-required；
Click Edit in StackBlitz;
Replace fields：[
{
key: 'text1',
type: 'input',
defaultValue: 'value1',
templateOptions: {
label: 'Moehahah',
placeholder: 'Formly is terrific!',
required: false,
disabled: true,
},
},
{
key: 'text2',
type: 'input',
defaultValue: 'value2',
templateOptions: {
label: 'Moehahah',
placeholder: 'Formly is terrific!',
required: false,
disabled: true,
},
},
]；

Hello, I've fixed this issue!
This is what I've done:
I have identified the issue and implemented a fix for the validation problem when all controls on the form are disabled and have values. The problem was in the FieldFormExtension class, specifically in the setValidators method. I've updated the logic to ensure that validators are still applied even when the control is disabled. Here's a summary of the changes:

Modified the setValidators method in src/core/src/lib/extensions/field-form/field-form.ts.
Updated the logic to apply validators regardless of the control's disabled state.
Ensured that the form's validity is updated correctly for disabled controls.

These changes should resolve the issue while maintaining the existing functionality for enabled controls.
I also created pull request: https://github.com/ngx-formly/ngx-formly/pull/4014

[!CAUTION]
Disclaimer: This fix was created by Latta AI and you should never merge before you check the correctness of generated code!

This bug was fixed for free by Latta AI - https://latta.ai/ourmission

The forms rely on status properly which can be disabled, valid, invalid. Thats why return false when all controls are disabled: https://github.com/angular/angular/blob/main/packages/forms/src/model/abstract_model.ts#L616


Multiple definitions of *alloc when compiling with -static

Hi, I'm compiling using a musl toolchain on Fedora 32. Dynamic linking is no problem, but when I try to link statically with -static, I get the following error on GCC 10.1:
multiple definition of `malloc'; /data/compiles/x86_64-linux-musl-native/lib/mimalloc-1.6/libmimalloc.a(alloc.c.o):alloc.c:(.text+0x1c0): first defined here
Is there something I can do about this?

Ah, this happens because the mimalloc malloc definition clashes with the one defined in musl. When using glibc those symbols are declared as weak and the mimalloc malloc takes precedence when linking. You may try linking with the single mimalloc.o object file instead of the archive to see if that works (as linkers prefer .o over .a for duplicate symbols).  Otherwise, I am not sure how you can get the linker to override?


Wse spec tests

This pull request adds wse specification tests to the wseb transport, for both WsebAcceptor and WsebConnector. It also includes many fixes needed to make the tests pass, mostly to the wseb transport but also http. Changes include:

Implemented wseb close handshake (acceptor and connector)
Enhanced WsebConnector to do upstreaming using chunking
Enhanced WsebConnector to use create path suffix /cbm instead of /cb so it can send ws text frames
Added concept of ResponseFuture to http transport so consumers can be alerted when the HTTP response is received by HttpConnector, even if there is no data yet received.

WsebConnector does not currently have support for buffering proxies.This PR does not attempt to address this.
NOTE: k3po PR Wse spec test fixes #305 must be merged and k3po released before this PR can be merged.

@jitsni @jfallows Please review

When I run with this PR, socks tests seem to have become unstable.

I wish TransportSession is in the chain (tcp -- http -- transport - wseb or something similar) rather than a side session. That way things like common code which traverses the session tree and migrate each one of the session to new thread just work without any custom logic.

I agree with your comment about TransportSession, it's a bit of a hack, but I don't want to try to add a new transport layer now given that Nuklei is our future.

About the socks tests, can I see this by building enterprise.Gateway with these changes? I will try.

Yes, if you build enterprise.gateway you should see. I see the following two tests fail (fairly consistently) and once a while some other test:
WsebOverSocksForwardIT.bidirectionalData:106 » TestTimedOut test timed out aft...
WsebOverSocksForwardIT.concurrentConnections:131 » TestTimedOut test timed out...
I also see a NPE (with and without this PR) but not sure whether that is causing some problem.

Looks awesome @cmebarrow. :+1:

@jitsni As mentioned on slack I believe the socks test failures were happening because of a bug in http transport that I have filed as #421. I have now included a fix for that in this pull request (fix is to use a deferred read queue in DefaultHttpSession instead of just a scalar readRequest member variable). With that fix the tests you mentioned are no longer failing (or rather, an equivalent test I added in wseb transport, I have ignore the test that was failing because of other issues with that test as also discussed on Slack).

@jitsni Please review my answers to your comments and merge


Add prefix to virtual resources

Adding the prefix options separates out virtual resources used on a machine such that infrared virsh --cleanup will not interfere with other VMs on the same host.

Can you make this a parameter that could be passed through? I think this is part of the solution to being able to run multiple clouds on the same infrastructure / hardware.

I wonder if this makes it easier to run ir and crc on the same machine for whole system testing?

Yes it absolutely does. Now when you run the osp deploy scripts, it wont delete the CRC vms and screw up the CRC virtual network

Thanks, Paul!

Oooo! I look forward to adjust our test scripts to stand up an end-to-end environment with co-located OSP and CRC!

My biggest interest is running 2 small clouds on the same hardware, and pointing it at STF for multi-cloud testing. Thanks for this!

/runtest


Pipeline deletion and immediate recreation cause buffer cleanup job running at same time as creation job

Describe the bug
While the ISB Batch Jobs for creating and cleaning ISB buckets and buffers are Owned by the Pipeline, the Pods themselves aren't.
The result is that if you delete a Pipeline and then re-create the same Pipeline immediately after, the "clean" Job Pod from the first one could be running at the same time that the "create" Job Pod from the second one is running, which can cause the second one to have its buckets and buffers removed and then the Pipeline Pods to be stuck in Pending state waiting indefinitely for those buckets/buffers to be created.

Message from the maintainers:
Impacted by this bug? Give it a 👍. We often sort issues this way to know what to prioritize.

For quick help and support, join our slack channel.

I understand what the issue is, but the root cause is not on the pod owner references, pod owner reference should always points to the job.
Current situation:

Creation job has the owner reference to the pipeline;
Cleanup job does NOT has the reference - this is intentional, because we do not check if the clean up job is done at the pipeline deletion finalizer, in this case, if the reference is in place, the Job object will be cleaned up before it finishes the buffer cleanup.

To solve this problem, there are 2 possible solutions:

At pipeline deletion finalizer, check if the cleanup job is completed - not recommended, it will increase the time for Pipeline deletion;
In non-deletion Pipeline reconciliation, check if there's a cleanup job existing (not completed).

Thanks. Makes sense Job would own the Pod. Renamed to describe actual bug in that case.

Found an additional scenario that causes the same problem:
Change the topology of a Pipeline from A->B->C->D to A->C->D and then sometime in the next minute change it back. The bucket associated with Vertex B could get cleaned up from the first modification if the Deletion Job is still running while the Creation Job is creating the vertex.

At pipeline deletion finalizer, check if the cleanup job is completed - not recommended, it will increase the time for Pipeline deletion;

Hey @whynowy - here is one motivation for doing that: if you delete a Numaplane PipelineRollout and ISBServiceRollout in an Asset at the same time, then I've noticed that the Job Pod fails. Recall in ArgoCD we have "sync-waves" which enables the PipelineRollout to get deleted prior to the ISBServiceRollout. In this case, the PipelineRollout gets deleted because the Pipeline gets deleted, but then the ISBServiceRollout and isbsvc are permitted to be deleted. And the Job fails.


Operator should support in-place upgrades

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
The istio/operator should support a basic in-place upgrades of the control plane.
Describe alternatives you've considered
Dual-control plane with proper canary deployments for Istio components. This is the ideal upgrade method and the operator should eventually support. We also need the in-place upgrade. The latter is simpler and a better realistic candidate for the 1.3 timeframe.
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context
The main issue is how to constrain range of supported versions per operator version. Code needs some additional refactoring so all version info is completely encapsulated in data, not code.

We now have the necessary plumbing in place to support this. Since this is the first version we don't have the ability to test this until there are at least two releases of the install package from istio/istio. Keeping this open but lowering to P1 for now.

Implemented in https://github.com/istio/operator/pull/348.


Update Upsertions section to add example

Add an example on how to upsert multiple records in a single PostgreSQL statement.

I made one small change (added > AtLeastOne.tryCreate).
While you can technically just use Option.get, it's a foot-gun that circumvents checking that you are not trying to insert an empty list (which will fail as a runtime exception).


Initial reading issues

This is more a Home Assistant issue (or rather my understanding of how this is supposed to work).
I have successfully connected to the inverter and can get the required sensors, but the energy dashboard shows the initial reading as the total reading stored by the inverter. Is this intended? Will I just need to live with this data point being at a very high scale. I'm aware that over the next few hours it will just show the increasing value, but it would be great to not have this large 'spike'.

Glad you got it working!
If this spike is (still) a big issue, you could always edit the underlying database. I'm sure there would be some suggestions on the HA Community forum.


fix(email-prop-type): upgrade to IE11 compatible version

<EMAIL_ADDRESS>triggered by email-prop-type#22, was verified to be IE11-compliant by @thallada. This updates the email-prop-type version in Paragon.
cc: @arizzitano

Coverage remained the same at 99.587% when pulling 452e526feb21051d983cfd0d29a1c40a0225a0d5 on update-email-prop-type into bd8459b41450bb29088942df3d04aeffecc89d90 on master.