SLO workload database/sql

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
Pull request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
Issue Number: #568
What is the new behavior?
SLO workload for database/sql driver

Codecov Report
Patch coverage has no change and project coverage change: -18.76 :warning:

Comparison is base (0f3012c) 65.05% compared to head (c765707) 46.30%.

:exclamation: Current head c765707 differs from pull request most recent head ee68029. Consider uploading reports for the commit ee68029 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #702       +/-   ##
===========================================
- Coverage   65.05%   46.30%   -18.76%
===========================================
  Files         228      154       -74
  Lines       22271    16091     -6180
===========================================
- Hits        14489     7451     -7038
- Misses       7019     8308     +1289
+ Partials      763      332      -431

Flag
Coverage Δ

?

1.17.x
46.05% <ø> (-18.75%)
:arrow_down:

1.19.x
46.37% <ø> (-18.30%)
:arrow_down:

1.20.x
?

integration
?

macOS
?

secure
?

ubuntu
46.30% <ø> (ø)

unit
46.30% <ø> (-0.10%)
:arrow_down:

windows
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 174 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


WIP: Customized ALB actions

Addresses #487. Provide a method for defining custom ALB rule actions via annotations.

Still need to implement support on default backends, but everything else should be in order.


Need the ability to modify key in another extension such as Base Application

We need a way of disabling/enabling a existing index which was created in another extension such as base extension using a table extension.  This will allow developers to not only create new key in their extension but also disable unused key due to customization.  Please see my suggested syntax below for this.  In this example Key18 and Key17 are altered and a new key Tune6 is created.
tableextension 50001 "Cust Legger Ext" extends "Cust. Ledger Entry"
{
keys
{
    key(Tune6; "Document No.", "Posting Date")
    {
        //Enabled = false;
        SumIndexFields = "Sales (LCY)";
        IncludedFields = "Adjusted Currency Factor";
        //MaintainSiftIndex = false;
        //MaintainSqlIndex = false;
    }

    modify(Key18)
    {
        MaintainSiftIndex = false;
        MaintainSqlIndex = false;
        Enabled = false;
    }

    modify(Key17)
    {
        MaintainSiftIndex = true;
        MaintainSqlIndex = true;
    Enabled = true;
    }

}

}

Thanks for the suggestion. Since this is an improvement and not a bug, you can post this to our Ideas forum at https://aka.ms/BusinessCentralideas, or vote up the idea if its already there. We're constantly monitoring top Ideas and will consider them for a future release.


gfx: Add mirrored tile check when generating tilemap

To help users save on VRAM, rgbgfx can detect and skip duplicate tiles. The Gameboy also allows for easy X- and Y-mirroring of sprites (DMG and CGB) and background tiles (CGB only), so it would be nice if rgbgfx could detect these mirrored-duplicates as well.
This patch adds an -m flag to perform this check, an -a option to specify an attribute map file to write the X- and Y-mirroring flags, and an -A flag to automatically generate the name of the attribute map. Like -u, if the tilemap or attrmap is omitted, the duplicate detection is still performed.
Passing -m implies -u as the mirror-tile check is an additional step on top of the uniqueness check.

[X] checkpatch is mostly satisfied (a couple lines exceed 80 characters)
[x] usage and manpage have been updated
[X] wwwman has been regenerated
[X] no existing behavior has been altered

I think it would be nice if, instead of generating this custom format where the XY flip byte follows the tile index byte, you'd generate a proper CGB attrmap with the correct bits set as a separate file. This will make it easier to compress the files to load them in a linear manner into memory, or concatenate them if the user wishes to, instead of having to disentangle the bytes during runtine or with a separate program.

Good idea @mid-kid, I'll do that.

@mid-kid Done!

Generating wwwman added whitespace errors to make develop which have now been fixed manually.

Could you create another PR for the patch that fixes the CHECKPATCH variable problem?

Well, to be fair, I think that you should squash all the commits into one, except for the one that updates wwwman. You are modifying the same code you have introduced in the first commit, and that way there is less to review.

@AntonioND Okay, I split the Makefile CHECKPATCH issue into #352. I've also squashed all the remaining commits here, except for wwwman.

Thanks @AntonioND!


Web: simple file dialog acts up in Firefox

Steps to Reproduce:

run web from Firefox
File > Open...
play around with typing paths and hit enter, the picker does not like that. The OK button does not seem to work either

Looks like a Firefox bug: https://stackoverflow.com/questions/33480562/firefox-document-execcommandinserttext
However, I can have better fallback behavior.

Added better fallback, but it does mean that the undo buffer for the simple file dialog will be broken in Firefox and anywhere else that insertText doesn't work.

Thanks for this! :clap:


Release notes for 4.0.1

General

The program run script is changed to addax.sh to remove the dependency on Python (#253)
Strip part of the methods of the StorageReaderUtil class as a separate class, which has reduced the dependency of other plugins on plugin-storage-util  (#227)
addax.py remove unused options, remove --mode, --jobid, --reader, --writer
Bump commons-compress from 1.20 to 1.21 (#232)
Change project structure (#234)
Bump commons-io from 2.10.0 to 2.11.0 (#236)
Extract the configuration items and default values used by each plugin into a common class to reduce redundant code (#239)
Bump logback.version from 1.2.3 to 1.2.4 (#243)
Bump slf4j-api from 1.7.31 to 1.7.32 (#246)
Fixed some typos (#250)

Reader plugins
dbf file reader

Fixed missing 3rd dependency jars ( #225)
Fixed null pointer exception when reading null value ( #229 )
Add support for wildcard to column item (#235)

hbase11xsql reader

Fix the problem that connecting hbase fails when kerberos is enabled (#241)

mongodb reader

Bump mongo-java-driver from 3.12.8 to 3.12.9 (#238)

redis reader

Bump redis-replicator from 3.5.4 to 3.5.5 (#221)
Bump jedis from 3.6.2 to 3.6.3 (#247)

stream reader

Add support for self-increment of time data type (#237)

textfile reader

Add support for get column by name to txtfilereader plugin (#251,  #254)

Writer plugins
hbase11x writer

Add support for login with kerberos (#245)

hbase11xsql writer

Improve the performance of writing (#242)

hbase20xsql writer

Add support for kerberos authentication and phoenix thick connection mode (#248)

mongodb writer

Bump mongo-java-driver from 3.12.8 to 3.12.9 (#238)

redis writer

Bump jedis from 3.6.2 to 3.6.3 (#247)

version  4.0.1 has released!


Feature/Enable tooltips on Mouse hover in flutter web/desktop

Partially closes #275

Implemented MouseRegion only for line chart, bar chart and pie chart (Scatter chart left untouched)
Tested only in flutter web: Chrome 87.0.4280.141 (Official Build) (x86_64) on Mac OS Big Sur 11.1
Also added flutter web execution to example app
Fix SVG in pie chart badge example not loading in web
Added Next/Previous FABs to example app for web and disabled scrolling in PageView for web

Hi,
Thanks for your great merge request.
I forgot to write in CONTRIBUTIONS.md that you must set your merge request target to the dev branch.
I changed the target to dev and now there are a lot of conflicts (we made some dartfmt changes on the dev branch)
BTW I apologize for that. I am reviewing your changes.
Please fix the conflicts.
Thanks!

Hi,
Thanks for your great merge request.
I forgot to write in CONTRIBUTIONS.md that you must set your merge request target to the dev branch.
I changed the target to dev and now there are a lot of conflicts (we made some dartfmt changes on the dev branch)
BTW I apologize for that. I am reviewing your changes.
Please fix the conflicts.
Thanks!

I reviewed all your changes smoothly.
I have to say that you are awesome.
Everything handled very well. I like that. Hope to see more from you :))
Thanks!

I reviewed all your changes smoothly.
I have to say that you are awesome.
Everything handled very well. I like that. Hope to see more from you :))
Thanks!

Done, thanks!

Done, thanks!

by the way, i just noticed line_chart.dart and bar_chart.dart import cupertino lib, and not material lib, is that by mistake or intended?

by the way, i just noticed line_chart.dart and bar_chart.dart import cupertino lib, and not material lib, is that by mistake or intended?

It is by mistake.

It is by mistake.

@imaNNeoFighT done

@imaNNeoFighT done

Thanks!

Thanks!


Hunger never depletes naturally

Installed Mods:
JEI
Valkyrien Skies
Valkyrien Control
Valkyrien World
How to reproduce:
Download the above mods and open a world on survival. Hunger never goes down naturally unless you take damage. Difficulty is set to normal.

In vanilla Minecraft, you loose hunger from running, jumping or harvesting a block. With Valkyrien skies installed, the only source of hunger loss is health regeneration.

In vanilla Minecraft, you loose hunger from running, jumping or harvesting a block. With Valkyrien skies installed, the only source of hunger loss is health regeneration.


flip connected flag on connected

I've spotted that connected is set to false on disconnect but is not restored to true on connect. Is that on purpose? If yes, plz explain, what's the trick, if it's really is a mistake, you can merge this.

I think waitDdpConnected runs every time it connects, so it's redundant. Make sense?


BLD: Bump the build number to pick up latest arrow

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)

Closes #42

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

We are still getting warnings about not finding libc, libgcc_s, libstd++, boost, arrow, and libm:
WARNING (parquet-cpp,lib/libparquet.so.1.4.0): did not find - or even know where to look for: /lib64/libc.so.6
WARNING (parquet-cpp,lib/libparquet.so.1.4.0): Needed DSO lib/libboost_regex.so.1.67.0 found in ['boost-cpp']
WARNING (parquet-cpp,lib/libparquet.so.1.4.0): .. but ['boost-cpp'] not in reqs/run, i.e. it is overlinked (likely) or a missing dependency (less likely)
WARNING (parquet-cpp,lib/libparquet.so.1.4.0): lib/libgcc_s.so.1 not found in any packages
WARNING (parquet-cpp,lib/libparquet.so.1.4.0): lib/libstdc++.so.6 not found in any packages
WARNING (parquet-cpp,lib/libparquet.so.1.4.0): lib/libarrow.so.0 not found in any packages
WARNING (parquet-cpp,lib/libparquet.so.1.4.0): did not find - or even know where to look for: /lib64/libm.so.6

@xhochy Shouldn't boost-cpp be a run dependency (in addition to a build dependency) because we're linking dynamically to boost here?

@cpcloud yes, it should be. We forgot that when we switched to dynamic linking.

Ok, I'll add.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Selectors are suggested to take a <two spaces>#<one space>[<expression>] form. See lines [40, 41]

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

@xhochy I'm not sure why conda is complaining that it can't find the SOs, even though they are clearly available.
Moreover, when I build this locally I only get warnings about libc and libm (which makes sense since those are not controlled by conda).

Another thing, do we actually need a C compiler in the build deps? Is parquet-cpp transitively compiling any C code?

@xhochy Thanks for merging!


get rid of deprecation warnings

we have updated the event infrastructure in Brackets and I got some warnings that didn't look good in my console window. ;-)

Thanks!


[WIP] data dir removal, hardlinks, various fixes

I have a few important and useful modifications to do and then will merge.


okta - unable to renew the session. The session store may not support this feature

Describe the bug
I use v<IP_ADDRESS>, and for sso I use okta. I get bad gateway error, it only works if I cleanup the cookie or open datahub in incognito.
To Reproduce
Steps to reproduce the behavior:

Go to 'www.mydatahub.com'
Get the following error

And in the datahub-frontend pod, I see the following error
'Unable to renew the session. The session store may not support this feature'

have the same

it seems something wrong with this pr https://github.com/datahub-project/datahub/pull/7011

After cookies clearing works fine.

Cookie clear makes it work, but it reappears once the session expired.

Same here, It also reappears when I logout from GUI.

Could it be fixed by #7088?? Can anyone try upgrading to v0.10.0 version?

Same here, but I'm on v0.10.0. After the session expires, or if I click on logout, I'll get a 502 if I try to open Datahub again.

It is not fixed. As @caiopavanelli mentioned if I logout, I get the bad gateway.

One thing I noticed is that once I get the 502 when trying to access the frontend, the debug log in the frontend service prints the following line. If I keep refreshing the page these lines repeat as well.
2023-03-01 12:20:45,348 [application-akka.actor.default-dispatcher-18] DEBUG o.p.o.r.OidcRedirectionActionBuilder - Authentication request url: https://************.okta.com/oauth2/v1/authorize?scope=openid+profile+email+groups&response_type=code&redirect_uri=https%3A%2F%2F************%2Fcallback%2Foidc&state=************&code_challenge_method=S256&client_id=************&code_challenge=************
2023-03-01 12:20:45,348 [application-akka.actor.default-dispatcher-18] DEBUG o.p.play.http.PlayHttpActionAdapter - requires HTTP action: 302

If I navigate to that link in the browser, I get an error from my OIDC Provider (Okta). However, the following lines are printed to the debug log.
2023-03-01 12:20:57,331 [application-akka.actor.default-dispatcher-13] DEBUG o.p.core.engine.DefaultCallbackLogic - === CALLBACK ===
2023-03-01 12:20:57,331 [application-akka.actor.default-dispatcher-13] DEBUG o.p.c.c.f.DefaultCallbackClientFinder - result: [oidc]
2023-03-01 12:20:57,331 [application-akka.actor.default-dispatcher-13] DEBUG o.p.core.engine.DefaultCallbackLogic - foundClient: #CustomOidcClient# | name: oidc | callbackUrl: https://************/callback | callbackUrlResolver: org.pac4j.core.http.callback.PathParameterCallbackUrlResolver@68737b76 | ajaxRequestResolver: org.pac4j.core.http.ajax.DefaultAjaxRequestResolver@394cd712 | redirectionActionBuilder: org.pac4j.oidc.redirect.OidcRedirectionActionBuilder@51e6ded | credentialsExtractor: org.pac4j.oidc.credentials.extractor.OidcExtractor@40beb6dd | authenticator: auth.sso.oidc.custom.CustomOidcAuthenticator@3365cd77 | profileCreator: org.pac4j.oidc.profile.creator.OidcProfileCreator@6cce684e | logoutActionBuilder: org.pac4j.oidc.logout.OidcLogoutActionBuilder@1ac9013f | authorizationGenerators: [auth.sso.oidc.OidcAuthorizationGenerator@2ff444e5] | configuration: #OidcConfiguration# | clientId: ************ | secret: [protected] | discoveryURI: https://************.okta.com/.well-known/openid-configuration | scope: openid profile email groups | customParams: {} | clientAuthenticationMethod: client_secret_basic | useNonce: false | preferredJwsAlgorithm: null | maxAge: null | maxClockSkew: 30 | connectTimeout: 500 | readTimeout: 5000 | resourceRetriever: com.nimbusds.jose.util.DefaultResourceRetriever@32672a19 | responseType: code | responseMode: null | logoutUrl: null | withState: true | stateGenerator: org.pac4j.core.util.generator.RandomValueGenerator@2d74b803 | logoutHandler: #DefaultLogoutHandler# | store: #GuavaStore# | size: 10000 | timeout: 30 | timeUnit: MINUTES | | destroySession: false | | tokenValidator: org.pac4j.oidc.profile.creator.TokenValidator@43644653 | allowUnsignedIdTokens: false | |
2023-03-01 12:20:57,332 [application-akka.actor.default-dispatcher-13] DEBUG o.p.o.c.extractor.OidcExtractor - Authentication response successful

After that, the PLAY_SESSION cookie gets cleared and I can access Datahub frontend again. Since I'm not logged out of the OIDC provider, It authenticates directly for me, which is a behavior I actually prefer.

Is there any update on this. Will the fix make a release?

If I navigate to that link in the browser, I get an error from my OIDC Provider (Okta). However, the following lines are printed to the debug log.

What error are you seeing exactly when you navigate to Okta? Trying to understand what could be happening here.
Also, I see that the screenshots above are showing 502 from NGINX.. Is everyone using some reverse proxy in front of datahub-frontend?

My guess is that this has to do with how this redirect is interacting with Nginx reverse proxy sitting in front of it. Because without that, we've not seen any issues here

Unsure whether something like this may be related -. https://serverfault.com/questions/1019510/nginx-is-not-redirecting-302-redirect-response-to-https

when the session cookie is expired, the UI tries to hit /sso which fails on that 502 page.
The debug log only shows that line
2023-03-01 12:20:45,348 [application-akka.actor.default-dispatcher-18] DEBUG o.p.play.http.PlayHttpActionAdapter - requires HTTP action: 302

If I enable TRACE logs for controller, then I get a stacktrace for the FoundAction at exactly that line.
I'll take a look on the link above though.

Still having this issue with v0.10.0

I have the same issue with OIDC keycloak auth with 502 Bad gateway and
2023-04-17T15:38:05+05:00 2023-04-17 10:38:05,679 [application-akka.actor.default-dispatcher-15] ERROR o.p.core.engine.DefaultCallbackLogic - Unable to renew the session. The session store may not support this feature

We also had the same issue in our installation of datahub. The 502 is returned by the ingress in kubernetes (GCP) when the response header size is large due to the FoundAction being passed in the returned_url of the PLAY_SESSION cookie.
The recent implementations of PAC4J is accepting String in RETURNED_URL but not v4.5.7. So, for getting around the issue, added below annotations in ingress to allow large cookie size. This fixed the 502 error.
metadata:
  annotations:
    kubernetes.io/ingress.class: nginx
    nginx.ingress.kubernetes.io/client-body-buffer-size: 64k
    nginx.ingress.kubernetes.io/client-header-buffer-size: 100k
    nginx.ingress.kubernetes.io/http2-max-header-size: 96k
    nginx.ingress.kubernetes.io/large-client-header-buffers: 4 100k
    nginx.ingress.kubernetes.io/proxy-body-size: 150m
    nginx.ingress.kubernetes.io/proxy-buffer-size: 96k
    nginx.ingress.kubernetes.io/server-snippet: |
      client_header_buffer_size 100k;
      large_client_header_buffers 4 100k;

We also had the same issue in our installation of datahub. The 502 is returned by the ingress in kubernetes (GCP) when the response header size is large due to the FoundAction being passed in the returned_url of the PLAY_SESSION cookie. The recent implementations of PAC4J is accepting String in RETURNED_URL but not v4.5.7. So, for getting around the issue, added below annotations in ingress to allow large cookie size. This fixed the 502 error.
metadata:
  annotations:
    kubernetes.io/ingress.class: nginx
    nginx.ingress.kubernetes.io/client-body-buffer-size: 64k
    nginx.ingress.kubernetes.io/client-header-buffer-size: 100k
    nginx.ingress.kubernetes.io/http2-max-header-size: 96k
    nginx.ingress.kubernetes.io/large-client-header-buffers: 4 100k
    nginx.ingress.kubernetes.io/proxy-body-size: 150m
    nginx.ingress.kubernetes.io/proxy-buffer-size: 96k
    nginx.ingress.kubernetes.io/server-snippet: |
      client_header_buffer_size 100k;
      large_client_header_buffers 4 100k;

This fixed my issue as well! Thank you!

Awesome, thanks you a lot @mkamalas !
Added this line for increase cookie size nginx.ingress.kubernetes.io/proxy-buffer-size: 96k


[BUG] calcFitHeight() throws error when called when pane has not been shown

Describe the bug
The calcFitHeight() throws an error when called before the pane has been shown.
To Reproduce
Steps to reproduce the behavior:

Initialise pane.
Call calcFitHeight()
Error is shown in logs:

ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'style' of undefined
TypeError: Cannot read property 'style' of undefined
    at Breakpoints.<anonymous> (cupertino-pane.esm.bundle.js:882)

Line #882 is:
this.instance.wrapperEl.style.visibility = 'hidden';
Expected behavior
The calcFitHeight() function to recalculate the height without throwing an error.
Desktop (please complete the following information):

OS: MacOS
Browser: Chrome
Version: 88.0.4324.96

Additional context
https://github.com/roman-rr/cupertino-pane/issues/123

@sjregan Thank you for your effort!
Fixed in master branch.
Anyway you must call method after present:
this.pane = new CupertinoPane('.pane', settings);
this.pane.present({animate: true});
this.pane.calcFitHeight();

And better with some timeout
this.pane = new CupertinoPane('.pane', settings);
this.pane.present({animate: true});
setTimeout(() => this.pane.calcFitHeight());

@sjregan Thank you for your effort!
Fixed in master branch.
Anyway you must call method after present:
this.pane = new CupertinoPane('.pane', settings);
this.pane.present({animate: true});
this.pane.calcFitHeight();

And better with some timeout
this.pane = new CupertinoPane('.pane', settings);
this.pane.present({animate: true});
setTimeout(() => this.pane.calcFitHeight());


better docker-host pruning

I'll follow 2 ideas:

Prune should run with -a -f flags
Making sure all exited pods are removed (they seem sometimes to prevent images from being pruned)

I wonder if we should do the -a -f pruning once a week
and a daily prune that also takes care of the exited pods.
or even making sure we only prune away images that are older than a week

The docker image prune with the filter works great. This way we could leverage regular pruning while ensuring caches within a week stay fresh and builds fast.


OCC-223: Upgrade to Orchard Core 1.8

OCC-223
Fixes #411

@sarahelsaig Could you please checkout this ZAP report and determine how valid of these errors are? They were not present before now with .NET 8 and OC 1.8? maybe they came to live.
2024-02-26-ZAP-Report-localhost.json

Okay now I resolved that. The following error still persist though and I also think that this is another close to nonsense error:
2024-02-26 18:17:49.0025 - An exception has occurred while interacting with the page https://localhost:9027/ (My Shop - Orchard Setup). 2024-02-26 18:17:49.0045 - The test failed with the following exception: Shouldly.ShouldAssertException: Shouldly uses your source code to generate its great error messages, build your test project with full debug information to get better error messages The provided expression     should be empty but had 1     item and was ["2024-02-26 19:17:27.7620|Default|00-d1384c43efd3c6f22046919e903789ca-d537c6bb36bf1168-00||Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware|ERROR|An unhandled exception has occurred while executing the request. System.IO.IOException: A fájlnév, a könyvtárnév vagy a kötetcímke szintaxisa nem megfelelő. : 'C:\Lombiq\OrchardCore.Commerce\test\OrchardCore.Commerce.Tests.UI\bin\Debug\net8.0\Temp\9db19990-aafe-4978-8627-a44ef46e07cd\App\App_Data\Sites\Default\Media\ProductImages\sample-product-image.png\'"]    at Lombiq.Tests.UI.Extensions.WebApplicationInstanceExtensions.LogsShouldBeEmptyAsync(IWebApplicationInstance webApplicationInstance, Boolean canContainWarnings, ICollection 1 permittedErrorLines, CancellationToken cancellationToken)    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task, ConfigureAwaitOptions options)    at Lombiq.Tests.UI.Extensions.VerificationUITestContextExtensions.AssertLogsAsync(UITestContext context)    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at Lombiq.Tests.UI.Extensions.VerificationUITestContextExtensions.AssertLogsAsync(UITestContext context)    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task, ConfigureAwaitOptions options)    at Lombiq.Tests.UI.Services.UITestExecutionSession.ExecuteAsync(Int32 retryCount, String dumpRootPath) 2024-02-26 18:17:49.2918  INFO > Take page snapshot #01 - FailureDumpPageSnapshot 2024-02-26 18:17:51.3444  WARN - Failed to take CDP snapshot. PageSource snapshot will be taken. OpenQA.Selenium.WebDriverException: Unexpected error creating WebSocket DevTools session.  ---> OpenQA.Selenium.WebDriverException: DevTools version is not in the supported range. Desired version=121, range=5. Supported versions: 116, 115, 114, 85    at OpenQA.Selenium.DevTools.DevToolsDomains.MatchDomainsVersion(Int32 desiredVersion, Int32 versionRange)    at OpenQA.Selenium.DevTools.DevToolsDomains.InitializeDomains(Int32 protocolVersion, DevToolsSession session, Int32 versionRange)    at OpenQA.Selenium.DevTools.DevToolsDomains.InitializeDomains(Int32 protocolVersion, DevToolsSession session)    at OpenQA.Selenium.DevTools.DevToolsSession.StartSession(Int32 requestedProtocolVersion)    at OpenQA.Selenium.Chromium.ChromiumDriver.<>c__DisplayClass34_1.<<GetDevToolsSession>b__0>d.MoveNext() --- End of stack trace from previous location ---    at OpenQA.Selenium.Chromium.ChromiumDriver.GetDevToolsSession(Int32 devToolsProtocolVersion)    --- End of inner exception stack trace ---    at OpenQA.Selenium.Chromium.ChromiumDriver.GetDevToolsSession(Int32 devToolsProtocolVersion)    at OpenQA.Selenium.Chromium.ChromiumDriver.GetDevToolsSession()    at Atata.CdpPageSnapshotStrategy.TakeSnapshot(AtataContext context)    at Atata.CdpOrPageSourcePageSnapshotStrategy.TakeSnapshot(AtataContext context) 2024-02-26 18:17:51.3485  INFO < Take page snapshot #01 - FailureDumpPageSnapshot (2.056s) >> "01 - FailureDumpPageSnapshot.html" 2024-02-26 18:17:51.3537 - Attempting to stop the Orchard Core instance. 2024-02-26 18:17:51.3584 - The Orchard Core instance was stopped.
So this by the URI that it whines about seems to be fine. I don't have that temp file that it cries about but under the SQLite snapshots this path seems to be fine and image is there.
This one actually gets caught on CI aswell. https://github.com/OrchardCMS/OrchardCore.Commerce/actions/runs/8006384127?pr=412
Can you spot something maybe that I can't? On CI there is a double OrchardCore.Commerce/OrchardCore.Commerce/ in the path that seems fishy but that is not the case locally set it is failing.

Okay so the previous error is coming from a missing Temp folder. Which I have no idea about why is happening. Also it doesn't break anything.

The following error still persist though and I also think that this is another close to nonsense error

That's a known error that I was already working on in #410 before the OC 1.8 upgrade complicated things. It's ZAP probing for path traversal exploits, which correctly results in an error log and UITT should permit this. I ended up disabling path traversal tests altogether, because as long as you only use ASP.NET/OC for resources it's not a real security concern and it generates a ton of error logs and weird false positives. Like this case for example. wtf am I supposed to do with this?

To straighten things out, I've merged this branch into issue/OCC-218 and made its PR target your branch. Sadly this make OCC-218 a blocker for this issue, because you need its changes to fix SecurityScanningTests.

This makes much more sense now. Well it's non sense but it makes sense to me at least :D I will halt the progress here in this case. Please ping me if there are any news on OCC-218. I started chewing my hair due to these Security error logs.

You know times are bad when Roland starts spitting hairballs... 😿

After doing OC 1.6 - 1.7 - 1.8 upgrades I am hypersensitive to retardedly failing UI tests. At least now I now whener I see a ?= on the internet I should deploy sql injection attac

@Psichorex it's good now. Did you need to do anything else or is this ready for review?
It's up for review. Thanks for saving my hair.

Wait, never mind. Please address this first.

Thats already pushed. CI is running.


A "Limit All Cancel-Jobs" policy makes "lpstat -p" unsuccessful with 3 minutes hangup

It happens on latest CUPS 2.3b4 and also on CUPS 2.2 down to CUPS 1.7.5.
I seems it is basically still the same as https://github.com/apple/cups/issues/4659
When I have in /etc/cups/cupsd.conf

<Policy allowallforanybody>
  JobPrivateAccess all
  JobPrivateValues none
  SubscriptionPrivateAccess all
  SubscriptionPrivateValues none
  &ltLimit All Cancel-Jobs>
    Order deny,allow
    Allow from all
  </Limit>
</Policy>
DefaultPolicy allowallforanybody

I get

# time lpstat -p
lpstat: Success

real    3m0.404s
user    0m0.025s
sys     0m0.025s

Actually lpstat is not successful because it does not show my local queues.
With 'LogLevel debug2' I see in /var/log/cups/error_log
(during that "lpstat -p") three times a one minute timeout (excerpts):

d [17/Apr/2018:15:49:45 +0200] select_timeout: JobHistoryUpdate=0
d [17/Apr/2018:15:49:45 +0200] select_timeout(0): 300 seconds to timeout a client connection
d [17/Apr/2018:15:50:44 +0200] [Client 1] cupsdReadClient: error=0, used=0, state=HTTP_STATE_POST_SEND, data_encoding=HTTP_ENCODING_LENGTH, data_remaining=0, request=0x559939dbeb40(IPP_STATE_DATA), file=-1
--
d [17/Apr/2018:15:50:45 +0200] select_timeout: JobHistoryUpdate=0
d [17/Apr/2018:15:50:45 +0200] select_timeout(0): 300 seconds to timeout a client connection
d [17/Apr/2018:15:51:45 +0200] [Client 2] cupsdReadClient: error=0, used=0, state=HTTP_STATE_POST_SEND, data_encoding=HTTP_ENCODING_LENGTH, data_remaining=0, request=0x559939dce3b0(IPP_STATE_DATA), file=-1
--
d [17/Apr/2018:15:51:46 +0200] select_timeout: JobHistoryUpdate=0
d [17/Apr/2018:15:51:46 +0200] select_timeout(0): 300 seconds to timeout a client connection
d [17/Apr/2018:15:52:45 +0200] [Client 3] cupsdReadClient: error=0, used=0, state=HTTP_STATE_POST_SEND, data_encoding=HTTP_ENCODING_LENGTH, data_remaining=0, request=0x559939dd9950(IPP_STATE_DATA), file=-1

In contrast when "Limit All" is on its own
clients like "lpstat -p" do no longer hang up:

<Policy allowallforanybody>
  JobPrivateAccess all
  JobPrivateValues none
  SubscriptionPrivateAccess all
  SubscriptionPrivateValues none
  &ltLimit All>
    Order deny,allow
    Allow from all
  </Limit>
  &ltLimit Cancel-Jobs>
    Order deny,allow
    Allow from all
  </Limit>
</Policy>
DefaultPolicy allowallforanybody

I get

# time lpstat -p
printer postscript is idle.  enabled since ...
printer pstest is idle.  enabled since ...

real    0m0.383s
user    0m0.034s
sys     0m0.026s

I know that the extra "Limit Cancel-Jobs" is no longer needed
according to https://github.com/apple/cups/issues/4659
so that with a simple

<Policy allowallforanybody>
  JobPrivateAccess all
  JobPrivateValues none
  SubscriptionPrivateAccess all
  SubscriptionPrivateValues none
  &ltLimit All>
    Order deny,allow
    Allow from all
  </Limit>
</Policy>
DefaultPolicy allowallforanybody

things work.
Nevertheless I like to report this issue because it may indicate
a more general problem in the IPP communication
(i.e. why are there those timeouts/hangups).

Nope, looks like the "All" value didn't actually get added to the list... Fix coming up in a sec...

[master 0873f681e] Fix policy limits using All (Issue #5296)
[branch-2.2 417d0e163] Fix policy limits using All (Issue #5296)

Wow - that was fast!
Many thanks for the fix.
I tested it with CUPS 2.3b4 and it works for me.
FYI:
For my patch I used only the actually relevant code change:

--- scheduler/conf.c.orig       2018-03-23 21:39:17.000000000 +0100
+++ scheduler/conf.c    2018-04-18 13:38:45.000000000 +0200
@@ -3853,7 +3853,7 @@ read_policy(cups_file_t *fp,              /* I - Con
         if (num_ops < (int)(sizeof(ops) / sizeof(ops[0])))
        {
          if (!_cups_strcasecmp(value, "All"))
-           ops[num_ops] = IPP_ANY_OPERATION;
+           ops[num_ops ++] = IPP_ANY_OPERATION;
          else if ((ops[num_ops] = ippOpValue(value)) == IPP_BAD_OPERATION)
            cupsdLogMessage(CUPSD_LOG_ERROR,
                            "Bad IPP operation name \"%s\" on line %d of %s.",


Bug: Error Convert Type

Stopped at ., 3/23:\n[XPTY0004] Cannot convert xs:string to xs:integer: "xx".

Thanks for this.  I have never used typed parameters though this API.
There seems to be no reason not to pass the type if has been set.
I plan add a test for the binding behavoir.
Do you have a simple example where this is needed?

Thank.
below is an example to add in the test-commands.js file.
describe(
		'create query and bind with type ',
		function() {
			var reply, err;
			before(function(done) {
				var input = "declare variable $max external := 20; for $i in 1 to $max return element Number { $i }";
				var query = session.query(input);

				// bind variable with type
				query.bind("max", 50, "xs:integer");

				// print results
				query.execute(function(e, r) {
					reply = r;
					err = e;
					done();
				});
			});

			it('It should not error', function() {
				should.not.exist(err);
			});
			it('It should return a string', function() {
				reply.result.should.be.a.String
			});
		});


Cannot start-video-annotator

Describe the bug
Hello, after downloading everything, I cannot start the program on the terminal. When entering the command start-video-annotator to run the program, I get this error: ('Unable to load OpenGL library', 'dlopen(OpenGL, 10): image not found', 'OpenGL', None). It seems Big Sur does not support OpenGL. I was wondering what are some suggestions as to runnin this program?

Hi,
After some investigation regarding the issue, it seems that there were some changes on how some libraries are handled in Big Sur.
This is a known issue and there is already a merged PR for python 3.9 here: https://github.com/python/cpython/pull/22855
I assume that it should be available in the next point release of python 3.9 since that merge is after the release date of 3.9.2 (the current latest).
There might be additional unknown issues by using Python Video Annotator on Python 3.9.
I will leave this issue open and try to find an alternative workaround in the meantime.

If you are at ease to change already installed python library files, there is an ugly workaround that works.
If you go to the folder mentioned in the error you get and edit the ctypesloader.py file (should be something similar to: /Users/<your_user>/opt/anaconda3/envs/videoannotator/lib/python3.9/site-packages/OpenGL/platform/ctypesloader.py and search for the following line: fullName = util.find_library( name ) (should be line 79).
Replace that line with:
fullName = "/System/Library/Frameworks/{}.framework/{}".format(name,name)
After that, save the file and try to run Python Video Annotator again. It should open without issues.
That solution was found on StackOverflow here: https://stackoverflow.com/questions/63475461/unable-to-import-opengl-gl-in-python-on-macos
I will try to find a way to properly fix this issue instead of this workaround.
Please let me know if you were able to do these steps.


⚠ Bofrost.at has degraded performance

In f2168e1, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1313 ms

Resolved: Bofrost.at performance has improved in d71d67e after 6 minutes.


Error while running http server

Hi when i try to run python manage.py runserver <IP_ADDRESS>:8000 i get the below error.
Unhandled exception in thread started by <function check_errors.<locals>.wrapper at 0x7fdcbd9b8c80>
Traceback (most recent call last):
  File "/home/hdpuser/anaconda3/lib/python3.6/site-packages/channels/routing.py", line 32, in get_default_application
    value = getattr(module, name)
AttributeError: module 'SpeechServer.asgi' has no attribute 'application'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/hdpuser/anaconda3/lib/python3.6/site-packages/django/utils/autoreload.py", line 225, in wrapper
    fn(*args, **kwargs)
  File "/home/hdpuser/anaconda3/lib/python3.6/site-packages/channels/management/commands/runserver.py", line 81, in inner_run
    application=self.get_application(options),
  File "/home/hdpuser/anaconda3/lib/python3.6/site-packages/channels/management/commands/runserver.py", line 107, in get_application
    return StaticFilesWrapper(get_default_application())
  File "/home/hdpuser/anaconda3/lib/python3.6/site-packages/channels/routing.py", line 28, in get_default_application
    module = importlib.import_module(path)
  File "/home/hdpuser/anaconda3/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/hdpuser/django-deepspeech-server-master/SpeechServer/asgi.py", line 12, in <module>
    application = get_default_application()
  File "/home/hdpuser/anaconda3/lib/python3.6/site-packages/channels/routing.py", line 34, in get_default_application
    raise ImproperlyConfigured("Cannot find %r in ASGI_APPLICATION module %s" % (name, path))
django.core.exceptions.ImproperlyConfigured: Cannot find 'application' in ASGI_APPLICATION module SpeechServer.asgi

Can you please provide channels version.

Most probable reason might be channels version greater than 2.0.0. For now can you please remove channels package from your installation and install channels==1.1.8 and check. Mean while I'll update my code for channels 2.0.0 and above.

Thanks it worked :)


routing is express like your server.js

I'm building a vuejs app on express like your project.
https://github.com/jackhutu/jackblog-vue/blob/master/server.js
and here i have a problem when I change the route from '/' to '/*'
I opened a question on stackoverflow , do you have an idea why it is working for you and not for me?
http://stackoverflow.com/questions/37609116/express-routing-all-routes-allowed-not-working/37609257?noredirect=1#comment62703419_37609257
The problem comes probably from my app.js script directly intagrated in my application.html template. You have not... because you use vueify
By the way thank you very much for your shared blog, it helped me a lot

Sorry, the solution was here http://stackoverflow.com/questions/21764155/serve-assets-before-other-routes


Langchain-Weave gives Validation Error

Thank you for the package. Love it.
I use Langchain's ChatOpenAI function with stream=True. When I add Weave to trace the chains by simply `weave.init()', it throws the following error:

([llm/error] [chain:RunnableWithMessageHistory > chain:check_sync_or_async > chain:RunnableSequence > chain:ChatOpenAI > llm:ChatOpenAI] [2.11s] LLM run errored with error:
"1 validation error for ChatCompletion\nchoices.0.message.role\n  Field required [type=missing, input_value={'content': " The text ap...2.", 'tool_calls': None}, input_type=dict]\n    For further information visit https://errors.pydantic.dev/2.9/v/missingTraceback (most recent call last):\n\n\n  File "/opt/conda/envs/testweave/lib/python3.10/site-packages/langchain_core/language_models/chat_models.py", line 932, in _agenerate_with_cache\n    result = await self._agenerate(\n\n\n  File "/opt/conda/envs/testweave/lib/python3.10/site-packages/langchain_openai/chat_models/base.py", line 809, in _agenerate\n    return await agenerate_from_stream(stream_iter)\n\n\n  File "/opt/conda/envs/testweave/lib/python3.10/site-packages/langchain_core/language_models/chat_models.py", line 118, in agenerate_from_stream\n    chunks = [chunk async for chunk in stream]\n\n\n  File "/opt/conda/envs/testweave/lib/python3.10/site-packages/langchain_core/language_models/chat_models.py", line 118, in \n    chunks = [chunk async for chunk in stream]\n\n\n  File "/opt/conda/envs/testweave/lib/python3.10/site-packages/langchain_openai/chat_models/base.py", line 1989, in _astream\n    async for chunk in super()._astream(*args, **kwargs):\n\n\n  File "/opt/conda/envs/testweave/lib/python3.10/site-packages/langchain_openai/chat_models/base.py", line 779, in _astream\n    async for chunk in response:\n\n\n  File "/opt/conda/envs/testweave/lib/python3.10/site-packages/weave/legacy/monitoring/openai/openai.py", line 79, in aiter\n    self._on_finish()\n\n\n  File "/opt/conda/envs/testweave/lib/python3.10/site-packages/weave/legacy/monitoring/openai/openai.py", line 87, in _on_finish\n    result = wrapped_stream.final_response()\n\n\n  File "/opt/conda/envs/testweave/lib/python3.10/site-packages/weave/flow/chat_util.py", line 104, in final_response\n    return ChatCompletion(\n\n\n  File "/opt/conda/envs/testweave/lib/python3.10/site-packages/pydantic/main.py", line 209, in init\n    validated_self = self.pydantic_validator.validate_python(data, self_instance=self)\n\n\npydantic_core._pydantic_core.ValidationError: 1 validation error for ChatCompletion\nchoices.0.message.role\n  Field required [type=missing, input_value={'content': " The text ap...2.", 'tool_calls': None}, input_type=dict]\n    For further information visit https://errors.pydantic.dev/2.9/v/missing")

Package info:
langchain = "0.3.0"
langchain-openai = "0.2.0"
weave = "0.50.7"

Hey @AIApprentice101 Thanks for writing in! Your weave package looks quite a bit out of date, do you mind updating and trying again? Happy to help if the issue persists :)

Thank you. I just tried version 0.51.8 which I think is the most recent release. Unfortunately, it still throws the same validation error.

Ticketing for internal tracking, I will keep you posted!

Do you mind providing a code sample so that I can reproduce the issue? @AIApprentice101

Thank you for your reply. I think I figure out where the issue is from. I'm using the Mistral-7B model which doesn't have role "system". I used a custom chat template that basically converts "system" message to "user". Without weave, this works fine. With weave enabled, it throws a Pydantic validation error. This also only happens with streaming=True.
from langchain_openai import ChatOpenAI
import weave
weave.init('langchain-demo')

OPENAI_API_KEY="EMPTY"
OPENAI_API_BASE="http://localhost:8000/v1"
LLM_MODEL_NAME = "TheBloke/Mistral-7B-Instruct-v0.2-AWQ"
llm = ChatOpenAI(
    openai_api_key=OPENAI_API_KEY,
    openai_api_base=OPENAI_API_BASE,
    model_name=LLM_MODEL_NAME,
	streaming=True
)
# validation error
response1 = llm.invoke("Hi, my name is Alice.")
# validation error
for chunk in llm.stream("Hi, my name is Alice."):
    print(chunk.content, end="", flush=True)

If you think this is not a general case, please feel free to close this issue. Any help would be very appreciated. Thank you.


localizable menu multimedia

cross with https://github.com/dimagi/commcare/pull/71
@ctsims

minor comments only, waiting on the commcare PR before merging this one

@orangejenny should be gtg


trouble with installing

hello
I am trying to install mininet on my Ubuntu 24.04 noble amd64, but there is a problem.
just to let you know I wanna explain the issues I deal with them till this step of installation.

after I ran command '~/mininet/util/install.sh -a' I got the error :

E: Package 'pep8' has no installation candidate

for this error I opend the install.sh file and replaced all pep8 with pycodestyle .

after trying again I got error: externally-managed-environment

error: externally-managed-environment
× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
python3-xyz, where xyz is the package you are trying to
install.
If you wish to install a non-Debian-packaged Python package,
create a virtual environment using python3 -m venv path/to/venv.
Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
sure you have python3-full installed.
If you wish to install a non-Debian packaged Python application,
it may be easiest to use pipx install xyz, which will manage a
virtual environment for you. Make sure you have pipx installed.
See /usr/share/doc/python3.12/README.venv for more information.

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.'
for this problem I create and activated python env and tried again.

the third and my unsoled problem when I run '~/mininet/util/install.sh -a', I get this error:

: warning: missing terminating " character
mnexec.c: In function ‘main’:
: error: missing terminating " character
mnexec.c:208:28: note: in expansion of macro ‘VERSION’
208 |             printf("%s\n", VERSION);
|                            ^~~~~~~
mnexec.c:208:35: error: expected expression before ‘)’ token
208 |             printf("%s\n", VERSION);
|                                   ^
make: *** [Makefile:50: mnexec] Error 1'

if anyone know what should I do, please help me to resolve the problem or install the mininet from begining.
thank you.

hello，do you solve your problem?I met the same problem

hello
Fortunately yes.
the solution that worked for me is in the below:
1.install mininet-wifi first.
2.cd to mininet-wifi/util
3.using "gedit install.sh" find and replace all "pep8" with "pycodestyle". save and exit. you can do it with gedit or enything you know.
4.run "sudo install.sh -Wlnfv" and install it.
at this point I think you can you mininet and "mn" command.
I will glad to help you if the problem didn't solve.

exactly the same situation.
cc -Wall -Wextra  \
-DVERSION=\"`PYTHONPATH=. python3 -B bin/mn --version 2>&1`\" mnexec.c -o mnexec
<command-line>: warning: missing terminating " character
mnexec.c: In function ‘main’:
<command-line>: error: missing terminating " character
mnexec.c:208:28: note: in expansion of macro ‘VERSION’
  208 |             printf("%s\n", VERSION);
      |                            ^~~~~~~
mnexec.c:208:35: error: expected expression before ‘)’ token
  208 |             printf("%s\n", VERSION);
      |                                   ^
make: *** [Makefile:50：mnexec] error 1


[GSSOC'23] Creation of Facial Expression to Emoji Converter

Is your feature request related to a problem? Please describe.
In this project, I would like to explore Deep Learning with Python in addition to GUI creation with Tkinter
Collecting the appropriate Dataset for emotion detection , adding the concepts of neural networks to ease out the process.
Dataset:- https://www.kaggle.com/datasets/msambare/fer2013
Describe the solution you'd like
A clear and concise description of any alternative solutions or features you've considered.
1.Importing the necessary python libraries
2.Processing the Emotion Dataset
3. Using the concepts of Deep Learning (- Neural Networks)
4. Avatar Creation
5. Tkinter GUI Creation
6. Output - Creation of a Facial Expression to Emoji Avatar Converter
Describe alternatives you've considered
No response
Additional context
Kindly assign me this issue under GSSOC'23 .
Code of Conduct

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

@khusheekapoor, Kindly assign me this issue


runtime: fatal error: runtime: address space conflict

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

What did you do?

Running latest version (master, v0.13.0-beta.0-147-g31362df) of Syncthing. Used to run stable on Solaris.

What did you expect to see?

No unexpected crashes.

What did you see instead?

runtime: address space conflict: map(0xc85b130000) = 0xb
fatal error: runtime: address space conflict

runtime stack:
runtime.throw(0xc9e8a0, 0x1f)
        /usr/local/go/src/runtime/panic.go:530 +0x99 fp=0xfffffd7ffb5ffc58 sp=0xfffffd7ffb5ffc40
runtime.sysMap(0xc85b130000, 0x100000, 0xfffffd7fff28d500, 0x1152d18)
        /usr/local/go/src/runtime/mem_bsd.go:84 +0x139 fp=0xfffffd7ffb5ffc98 sp=0xfffffd7ffb5ffc58
runtime.(*mheap).sysAlloc(0x1138760, 0x100000, 0xfffffd7ffb5ffda0)
        /usr/local/go/src/runtime/malloc.go:429 +0x191 fp=0xfffffd7ffb5ffd20 sp=0xfffffd7ffb5ffc98
runtime.(*mheap).grow(0x1138760, 0x8, 0x0)
        /usr/local/go/src/runtime/mheap.go:651 +0x63 fp=0xfffffd7ffb5ffd78 sp=0xfffffd7ffb5ffd20
runtime.(*mheap).allocSpanLocked(0x1138760, 0x4, 0xfffffd7ffe5b5268)
        /usr/local/go/src/runtime/mheap.go:553 +0x4e3 fp=0xfffffd7ffb5ffdd0 sp=0xfffffd7ffb5ffd78
runtime.(*mheap).alloc_m(0x1138760, 0x4, 0x10000000042, 0xfffffd7ffe5b5268)
        /usr/local/go/src/runtime/mheap.go:437 +0x122 fp=0xfffffd7ffb5ffe00 sp=0xfffffd7ffb5ffdd0
runtime.(*mheap).alloc.func1()
        /usr/local/go/src/runtime/mheap.go:502 +0x41 fp=0xfffffd7ffb5ffe30 sp=0xfffffd7ffb5ffe00
runtime.systemstack(0xfffffd7ffb5ffe50)
        /usr/local/go/src/runtime/asm_amd64.s:307 +0xc1 fp=0xfffffd7ffb5ffe38 sp=0xfffffd7ffb5ffe30
runtime.(*mheap).alloc(0x1138760, 0x4, 0x10000000042, 0x452818)
        /usr/local/go/src/runtime/mheap.go:503 +0x63 fp=0xfffffd7ffb5ffe80 sp=0xfffffd7ffb5ffe38
runtime.(*mcentral).grow(0x113b710, 0x0)
        /usr/local/go/src/runtime/mcentral.go:209 +0x93 fp=0xfffffd7ffb5ffee8 sp=0xfffffd7ffb5ffe80
runtime.(*mcentral).cacheSpan(0x113b710, 0xfffffd7ffe5b5268)
        /usr/local/go/src/runtime/mcentral.go:89 +0x47d fp=0xfffffd7ffb5fff28 sp=0xfffffd7ffb5ffee8
runtime.(*mcache).refill(0xfffffd7fff10f4b0, 0x42, 0xfffffd7ffe5b5268)
        /usr/local/go/src/runtime/mcache.go:119 +0xd5 fp=0xfffffd7ffb5fff60 sp=0xfffffd7ffb5fff28
runtime.mallocgc.func2()
        /usr/local/go/src/runtime/malloc.go:642 +0x2b fp=0xfffffd7ffb5fff80 sp=0xfffffd7ffb5fff60
runtime.systemstack(0xc820020000)
        /usr/local/go/src/runtime/asm_amd64.s:291 +0x86 fp=0xfffffd7ffb5fff88 sp=0xfffffd7ffb5fff80
runtime.mstart()
        /usr/local/go/src/runtime/proc.go:1048 fp=0xfffffd7ffb5fff90 sp=0xfffffd7ffb5fff88

Does this happen consistently?  Can you tell us how to recreate the problem?
The call to mmap seems to have failed with EAGAIN.  That suggests that you  may be out of swap space.

It could indeed be that it's running out of memory. I don't have a small reproducer, but I can allocate more memory to the system and see if that's the case. If it is, we can close this, otherwise I'll try to narrow it down.
(If it's "just" a case of running out of memory, could the error possibly be worded more clearly? I would not have filed an issue for fatal error: runtime: out of memory...)

We do print a better error message for ENOMEM, which is what is returned on GNU/Linux in that case.  I had to look at the Solaris man page to see what EAGAIN means on Solaris.  If the problem is indeed being out of memory, we can handle EAGAIN like ENOMEM.

:+1: The memory handling is a bit different on Solaris as every allocation is indeed a swap allocation and there is no memory overcommit. :)

I'm not seeing this crash after raising the amount of available memory, so I think that's it, yes.

Thanks for following up.  Leaving the issue open to change the error message when mmap returns EAGAIN on Solaris.

so I am trying to get this to work (using syncthing 0.14.7) on sol10
out.txt
My error is similar to above (see attached):
Please tell me that I do not need go installed?
The system is a sol10 server with like 260G of ram

@choman I don't think that what you are seeing has anything to do with this issue.  Please use a forum (http://golang.org/wiki/Questions) or open a new issue.


Extension is not working properly.

I tried to upload the code to Github, but it's not working then I refresh my Leetcode and Github website and the push button from the leetcode was gone and the extension is not able to fetch my details also from the username.

The extension button displayed when you submit a solution and it's correct
the username of your LeetCode account is Harshhk not Harshh18, please recheck it

You're right, but after re checking everything I try to push the code but it is not getting committed to my Github repo. I am pushing the code for problem no. 58 in my repo.
Actually, my token was expired and I regenerated the token again and after adding the new token it is not pushing my code in the repo.
Repo link

Hey, I'm still unable to figure out what's going wrong with my side. I have even generated a new token for the same but still the extension is not able to commit changes into my repository. It'd be great if someone hops in to help me with this.

can you provide me with your GitHub repo link?

https://github.com/Harshh18/Daily-DSA
Here it is


update affinity assistant creation implementation

Changes
Before this commit, the affinity assistant was created in the beginning of the pipelineRun. And the same affinity assistant was used for the entire lifecycle of a PR. Now, there could be a case when the node on which affinity assistant pod is created goes down. In this case, the rest of the pipelineRun is stuck and cannot run to the completition since the affinity assistant (StatefulSet) tries to schedule rest of the pods (taskRuns) on the same node but that node is cordoned or not scheduling anything new.
This commit always makes an attempt to create Affinity Assistant (StatefulSet) in case it does not exist. If it exist, the controller checks if the node on which Affinity Assistant pod is created is healthy to schedule subsequent pods. If not, the controller deletes Affinity Assistant pod so that StatefulSet can upscale the replicas (set to 1) on other node in the cluster.
Closes https://github.com/tektoncd/pipeline/issues/6586.
Testing is pending, will update the PR after testing locally.
These changes are now tested locally on a cluster with two nodes and works as expected. The affinity assistant pod along with remaining taskRun pods are created on a healthy node when the existing node is cordoned.
/kind bug
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Resilient Affinity Assistant - make sure the Affinity Assistant pod is always on a healthy node during the entire life cycle of the pipelineRun

@lbernick @skaegi
This is an alternate solution to fix the pipelineRun with affinity assistant where the node could go down while the pipelineRun is still running.

I was originally concerned about a race condition that could lead to deadlock with this PR. I was worried that the affinity assistant pod could schedule successfully, but then the node could become unschedulable before the taskrun pod got scheduled on it, leading to deadlock.
However, @pritidesai and I experimented with this scenario today. We introduced a delay locally into runNextSchedulableTask, and cordoned the node w/ the affinity assistant pod before the taskrun could be scheduled. We observed that the pod creation triggered another reconcile loop for the TaskRun, which triggered another reconcile loop for the PipelineRun, which led to the affinity assistant getting deleted and recreated on a new node and the TaskRun pod successfully scheduling. So I think this is the right approach! Thanks Priti!
The only downside is this will be hard to write integration tests for, but that's also true of the alternative solution (https://github.com/tektoncd/pipeline/pull/6584).

thanks @lbernick 🙏
@tektoncd/core-maintainers This PR is ready for review, please take a look and let me know if you find anything concerning with these changes. Thanks!
I demoed this PR in the API WG this morning if anyone would like to watch it.

Thanks for this @pritidesai!
The concern I have with this change is that now we're doing 3 API calls on each reconcile cycle, as long as affinity assistant is enabled, which may impact scalability.
I think the change that tries to create the affinity assistant all the time is something that we should do anyways - I would like to learn more about why we need to delete the pod on Tekton side.

The concern I have with this change is that now we're doing 3 API calls on each reconcile cycle, as long as affinity assistant is enabled, which may impact scalability.

Yup, that is correct. We are adding 3 API calls in each reconcile cycle:

Pods.List(LabelSelector)
Nodes.Get(pod.Spec.NodeName)
Pods.Delete(pod)

I tried to look for an alternate more efficient solution but haven't been able to get the better alternative to "delete the StatefulSet pod if the node it was scheduled on becomes unscheduable" or "support node repair".

I think the change that tries to create the affinity assistant all the time is something that we should do anyways

Right, the StatefulSet creation was happening only before the first taskRun was created. This commit changes it to get created anytime during the life cycle of a pipelineRun. I am not sure what would be the additional driver to change the existing behavior.

I would like to learn more about why we need to delete the pod on Tekton side.

The StatefulSet is created with the required number of replicas set to 1. It's the replication controller's responsibility to make sure the desired number of replicas exist all the time. Deleting the pod triggers the replication controller to recreate the pod with the same specifications on any available node.

The concern I have with this change is that now we're doing 3 API calls on each reconcile cycle, as long as affinity assistant is enabled, which may impact scalability.

Yup, that is correct. We are adding 3 API calls in each reconcile cycle:

Pods.List(LabelSelector)
Nodes.Get(pod.Spec.NodeName)
Pods.Delete(pod)

@afrittoli, @lbernick I have updated API calls a bit. Instead of listing with a label selector, I have Pods.Get(podName) since podName can be derived from the name of the StatefulSet. This does not reduce the number of API calls. But to be clear, tekton controller is adding 2 API calls (Pods.Get and Nodes.Get) in general and  in case of cordoned node, its adding 3 API calls (Pods.Get, Nodes.Get, and Pods.Delete).

/test pull-tekton-pipeline-integration-tests

hey @afrittoli and @lbernick, I have updated this PR to implement an alternate proposal based on the offline conversation with @afrittoli:
Node.List(unschedulable=true)
for each unschedulableNode {
  Pods.Get()
  if spec.nodeName == unschedulableNode {
    Pods.Delete()
  }
  break
}

PTAL 🙏

hey @afrittoli and @lbernick, I have updated this PR to implement an alternate proposal based on the offline conversation with @afrittoli:
Node.List(unschedulable=true)
for each unschedulableNode {
  Pods.Get()
  if spec.nodeName == unschedulableNode {
    Pods.Delete()
  }
  break
}

PTAL 🙏

I'm curious what the reason for this change was? If we want to go that approach, would it make sense to reopen https://github.com/tektoncd/pipeline/pull/6584?

I'm curious what the reason for this change was? If we want to go that approach, would it make sense to reopen https://github.com/tektoncd/pipeline/pull/6584?

The biggest motivation is to reduce the number of API calls. So now, unless a node is cordoned in a cluster, there is no API call to get the affinity assistant pods. My concerns going with https://github.com/tektoncd/pipeline/pull/6584:

Creating an additional controller for watching nodes instead of only querying node status when affinity assistant is enabled.
Isolation of responsibilities: Deleting the affinity assistant pods by a separate controller outside of the controller which created them. https://github.com/tektoncd/pipeline/pull/6584#discussion_r1178487015
Deadlock: As described in https://github.com/tektoncd/pipeline/pull/6584#discussion_r1181654274

I'm curious what the reason for this change was? If we want to go that approach, would it make sense to reopen #6584?

The biggest motivation is to reduce the number of API calls. So now, unless a node is cordoned in a cluster, there is no API call to get the affinity assistant pods. My concerns going with #6584:

Creating an additional controller for watching nodes instead of only querying node status when affinity assistant is enabled.
Isolation of responsibilities: Deleting the affinity assistant pods by a separate controller outside of the controller which created them. RFC: Handle affinity assistant deadlock for node maintenance #6584 (comment)
Deadlock: As described in RFC: Handle affinity assistant deadlock for node maintenance #6584 (comment)

@pritidesai and I chatted offline, I think it's totally reasonable to reduce the number of API calls we're doing by only getting the AA pod if there are unschedulable nodes.
Just want to clarify re: my deadlock comment, it was not meant to describe a concern with the approach in #6584; it was meant to discuss the concern with this PR which Priti and I resolved in https://github.com/tektoncd/pipeline/pull/6596#issuecomment-1538932774

Thanks, @pritidesai ! Looking forward to this improvement!
I probably missed some previous discussions. It looks like this PR and the alternative: https://github.com/tektoncd/pipeline/pull/6584 (which is clsoed) attempt to solve the same problem, so I was wondering how do we decide to adopt this solution?

Thanks, @pritidesai ! Looking forward to this improvement!
I probably missed some previous discussions. It looks like this PR and the alternative: #6584 (which is clsoed) attempt to solve the same problem, so I was wondering how do we decide to adopt this solution?

thanks @QuanZhang-William, you can check the other PR for more details. But @lbernick and I both came up with possible solutions and decided to go this route instead of creating a separate controller to watch nodes and delete them outside of a pipelineRun reconciler. This implementation is built in the reconciler and only deletes the affinity assistant pod which it owns whenever necessary i.e. does not orchestrate neighboring pipelineRuns.

/retest

thanks a bunch @lbernick 🙏
@QuanZhang-William please let me know if you have any feedback on these changes, thanks!

thanks a bunch @lbernick 🙏
@QuanZhang-William please let me know if you have any feedback on these changes, thanks!

Thanks @pritidesai ! LGMT!

Just to add further voice to this as we would really like to see this in 0.48. This PR is important to us as we currently have a problem when performing maintenance on nodes in our cluster. This PR let's us run more of a lights out operation.

We would like to upgrade to the latest LTS release which is 0.47. These changes along with the PR https://github.com/tektoncd/pipeline/pull/6860 are required for us to manage the clusters. I would like to request cherry picking these changes into 0.47. Thanks!
/cherry-pick release-v0.47.x


Suggestion: VS Code Web - Save As the file extension of the programming language

When testing the VS Code web text editor, we specify the language of the code on a new file.
Then upon exporting the test script using File > Save As.
The exported file is saved as Untited-1 without any file extension.
Please use the file extension of the defined programming language
In the example below, the editor recognises C# as the specified language

When the file is output it has no extension appended

I have to manually rename the file to open it in another editor like Visual Studio 2022

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette).
Happy coding!

Additional info:

When user attempts to save a new code file using File > Save As or using shortcut CTRL + S the Save As dialog is not displayed and the file is downloaded directly as Untitled-1 without a file extension.
This issue affects Firefox 102.0.1 (64 bit)
When the File > Save As option is used in Chromium based Edge, it works.

Issue steps

Browse: https://vscode.dev/ in current version of Firefox
Click: New File... under start
Click Text File built-in
Click "Select a Language"
Click C# from drop down list
Type in any C Sharp code

Save by expanding Hamburger menu
7.1. File > Save / Save As
7.2. A Save As Dialog is not displayed and the code is automatically downloaded as Untitled-1
7.3. The same occurs when using CTRL+S shortcut to save
Untitled-1

Note: When used in Microsoft Edge the dialog displays ok.

This was previously done in https://github.com/microsoft/vscode/issues/130035 but is probably using an API that Firefox does not support.

Duplicate of https://github.com/microsoft/vscode/issues/130035. If Firefox doesn't support the API then we will not do the right thing in Firefox.


Fix weather updates

wttr.in redirects to https, causing the if-clause to not match.
enable follow_redirects to end up with a 200 status code at some point.
Also

add command to set the weather's update interval
fix some indentations

thanks


regression in 0.5.3

Hello, i noticed that coordinates translation example, that worked in rev previous to 0.5.3, now is not working.

Hi @violent-boomerang,
Thank you very much for your report!!
Fix the event.offsetX/Y values were not calculated correctly. (v0.5.4)
Kind Regards,

Great! fixed, though it seems now there is another issue:
Double click/tap on mobile on node or edge is not working, it is on view though. I am not 100% percent sure if it was working before 0.5.3
Many thank for your work

No, just checked it did not work in 0.5.2, this is then not related, I'll open a dedicated issue and close this one


Support JSON output.

This add support for JSON output. It does not add support for JSON output in TableWriter. If you are interested I can take a look at included this.

I think this could be a little confusing for people without the JSON output of the table.
I do totally agree it would be great to have all commands outputting JSON.
I want to hide the table writer, and all the console output things behind an interface, with a couple of implementations.

Sure, that makes sense, let's see if I can refactor this to your liking.

It's been a month and don't have time nor direct need for this at the moment. Closing.

Going to merge this now then.
Cheers for your work so far.


docs(APIRef.DetoxCLI) fix: missing - for options

Add missing - for --configuration and --gpu

[x] This is a small change
[ ] This change has been discussed in issue #<?> and the solution has been agreed upon with maintainers.

Description:
Long option should be -- instead of - or else it will be error on parameter when run command

Reference: https://travis-ci.org/guhungry/react-native-photo-manipulator/jobs/520852289

Thanks!


navbar toggleBreakpoint 'xs' broken for rc.4

Bootstrap 4 beta 2 changed the navbar-expand classes a bit. In order to never collapse the navbar content, the class navbar-expand must be added instead of a navbar-expand-* class. There currently seems to be no supported way to do this or to fully turn off toggleBreakpoint functionality.

Output from ember bootstrap:info (available since 1.0.0-alpha.9):
Npm packages:
ember-bootstrap: ^1.0.0-rc.4 -> 1.0.0-rc.4
ember-cli: ~2.16.2 -> 2.16.2
bootstrap: ^4.0.0-beta.2 -> 4.0.0-beta.2
bootstrap-sass: n/a
ember-cli-sass: ^7.0.0 -> 7.0.0
ember-cli-less: n/a
Bower packages:
bootstrap: n/a
bootstrap-sass: n/a
ember-bootstrap configuration:
bootstrapVersion: 4
importBootstrapFont: false
importBootstrapCSS: false
whitelist: undefined

@nickschot The current BS4 implementation of the breakpoint class would set navbar-expand if toggleBreakpoint is empty (null or ""). See https://github.com/kaliber5/ember-bootstrap/blob/master/addon/components/bs4/bs-navbar.js#L45-L53
Wouldn't that work?
Probably needs some better documentation though!

That should work indeed! I tried "false" but isBlank won't cover that of course. I'll test it today and report back if it doesn't work for some reason. Small addition to the docs would "fix" this then.

Updated the docs, will close this now. Feel free to reopen if required!


Change alignment of the label on y axis #99

Add textAlignment property to ChartLabelSettings which is used to properly align text in the y axis label

To merge this in the library it would need to satisfy following:

The alignment should have an effect in the reserved space. If the labels are e.g. aligned right in the left y-axis, the chart should not reserve space on the left.

The title axis is positioned correctly, which would be next to the divider. The title to labels space setting in this case can be reused as title to line space.

The space should be correct when using multiple axes. If the labels are e.g. aligned right in the left y-axis, and after it there's another axis left y-axis, this should start after the line and not the space which would have been reserved by the labels on the left side. You also have to take into account that this second axis can have their labels aligned to the left or right. If it has the labels on the right side then it has to start with a distance from the previous axis determined by it's own max labels width.

From what I can see in the implementation I think it's not meeting at least some of these points.

Hi Ivan,
Thanks for your comments - I found a bug in the small piece of logic that I've submitted in this pull request :) With a new commit, now it behaves and renders the labels as it was planned.
And please let me explain what comes with this small request - It's not about changing the side of label axis on which labels are displayed - it' only about aligning the label text to left or right within the same reserved space (based on labelsMaxWidth).
Are you fine with this change now?
Cheers,
Marcin

Ah! Sorry. Was thinking about the side indeed. The PR is fine except 2 things - 1. For the implementation to be consistent, it would be good if you can implement this for the right y axes also. 2. I assume you added the "Default" enum value to simplify the settings for the different axes. It's convenient but semantically not quite right as alignment can be only left or right. Default is given by where we use this enum. So if you can find a cleaner way to model this it would be great. Not a must though. Thanks!

It's implemented for ChartAxisYLowLayerDefault and ChartAxisYHighLayerDefault (which I believe is the right y axis) :) - please let me know if I'm wrong.
For the 2nd point I will look into it today.

Right, I tested with multiaxes example and set the alignment on the axis titles which has no effect. Sorry for being so distracted, a bit in a hurry lately ;) Then the only point remaining is 2, as I said, if you find this too complicated to implement we can leave it like this.

Can I merge this?

Sorry for being not responsive but I was traveling. For the 2nd point I don't see a solution yet how the configuration should be passed which should define when left and when right alignment to choose in current scenarios.
If you could merge the current state of this pull request it would be great since I might not have a time now to come up with a solution :)

Hi Marcin,
no problem! About the default values, actually UIKit also uses this quite a lot so I guess it's at least consistent :) Thanks for your contribution!

Thank a lot! :)


Add DWR JavaEE CDI

No we don\t need this fiucking shit...
Just refere to my repo where it was implemented: https://github.com/daggerok/java-ee-dwr-example


Tuya baby monitor camera not working in HA

I have purchased a baby monitor camera from tuya, this camera has a monitor, and it allows to connect through the tuya app
This is the camera
https://a.aliexpress.com/_EyT2Rrr
This camera also has a open local RTSP stream, that is available in the rtsp:CAMERA_IP:554/stream_0, which i can see though VLC Player, but it's not working in Home assistant with the webRTC as my other cameras
I have found very little info about this camera, (the only place where I found the correct RTSP path is https://www.ispyconnect.com/camera/tuya) and I don't know what kind of codes is using
What can I do to find how to make it work?
Thanks!

You can check stream info in the go2rtc Web UI when stream active