Fix spring app name startup exception

What does this PR do?
Deploying https://github.com/spring-petclinic/spring-framework-petclinic application on a Tomcat server 9.0.x produces the following exception on startup:
java.lang.UnsupportedOperationException

java.lang.UnsupportedOperationException: Section 4.4 of the Servlet 3.0 specification does not permit this method to be called from a ServletContextListener that was not defined in web.xml, a web-fragment.xml file
 nor annotated with @WebListener
        at org.apache.catalina.core.StandardContext$NoPluggabilityServletContext.getClassLoader(StandardContext.java:6675)
        at co.elastic.apm.agent.springwebmvc.SpringServiceNameInstrumentation$SpringServiceNameAdvice.afterInitPropertySources(SpringServiceNameInstrumentation.java:75)
        at org.springframework.web.context.support.AbstractRefreshableWebApplicationContext.initPropertySources(AbstractRefreshableWebApplicationContext.java:214)
        at org.springframework.context.support.AbstractApplicationContext.prepareRefresh(AbstractApplicationContext.java:600)
        at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:519)
        at org.springframework.web.context.ContextLoader.configureAndRefreshWebApplicationContext(ContextLoader.java:401)
        at org.springframework.web.context.ContextLoader.initWebApplicationContext(ContextLoader.java:292)
        at org.springframework.web.context.ContextLoaderListener.contextInitialized(ContextLoaderListener.java:103)
        at org.apache.catalina.core.StandardContext.listenerStart(StandardContext.java:4676)
        at org.apache.catalina.core.StandardContext.startInternal(StandardContext.java:5139)
        at org.apache.catalina.util.LifecycleBase.start(LifecycleBase.java:183)
        at org.apache.catalina.core.ContainerBase.addChildInternal(ContainerBase.java:717)
        at org.apache.catalina.core.ContainerBase.addChild(ContainerBase.java:690)
        at org.apache.catalina.core.StandardHost.addChild(StandardHost.java:706)
        at org.apache.catalina.startup.HostConfig.deployWAR(HostConfig.java:978)
        at org.apache.catalina.startup.HostConfig$DeployWar.run(HostConfig.java:1848)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at org.apache.tomcat.util.threads.InlineExecutorService.execute(InlineExecutorService.java:75)
        at java.util.concurrent.AbstractExecutorService.submit(AbstractExecutorService.java:112)                                                                                at org.apache.catalina.startup.HostConfig.deployWARs(HostConfig.java:773)
        at org.apache.catalina.startup.HostConfig.deployApps(HostConfig.java:427)
        at org.apache.catalina.startup.HostConfig.start(HostConfig.java:1576)
        at org.apache.catalina.startup.HostConfig.lifecycleEvent(HostConfig.java:309)
        at org.apache.catalina.util.LifecycleBase.fireLifecycleEvent(LifecycleBase.java:123)
        at org.apache.catalina.util.LifecycleBase.setStateInternal(LifecycleBase.java:423)
        at org.apache.catalina.util.LifecycleBase.setState(LifecycleBase.java:366)
        at org.apache.catalina.core.ContainerBase.startInternal(ContainerBase.java:936)
        at org.apache.catalina.core.StandardHost.startInternal(StandardHost.java:843)
        at org.apache.catalina.util.LifecycleBase.start(LifecycleBase.java:183)
        at org.apache.catalina.core.ContainerBase$StartChild.call(ContainerBase.java:1384)
        at org.apache.catalina.core.ContainerBase$StartChild.call(ContainerBase.java:1374)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at org.apache.tomcat.util.threads.InlineExecutorService.execute(InlineExecutorService.java:75)

On top of that, we can already initialize the application name with getApplicationName() method on the application context.
Checklist

[x] This is a bugfix

[ ] I have updated CHANGELOG.asciidoc
[ ] I have added tests that would fail without this fix

Codecov Report

Merging #1464 into master will increase coverage by 0.00%.
The diff coverage is 25.00%.

@@            Coverage Diff            @@
##             master    #1464   +/-   ##
=========================================
  Coverage     59.73%   59.74%
  Complexity       91       91
=========================================
  Files           391      391
  Lines         17528    17532    +4
  Branches       2408     2409    +1
=========================================
+ Hits          10470    10474    +4
- Misses         6340     6341    +1
+ Partials        718      717    -1

Impacted Files
Coverage Δ
Complexity Δ

...springwebmvc/SpringServiceNameInstrumentation.java
42.85% <0.00%> (-17.15%)
0.00 <0.00> (ø)

...va/co/elastic/apm/agent/impl/ElasticApmTracer.java
66.03% <100.00%> (ø)
0.00 <0.00> (ø)

...va/co/elastic/apm/agent/impl/transaction/Span.java
82.14% <100.00%> (ø)
0.00 <0.00> (ø)

...ic/apm/agent/profiler/collections/LongHashSet.java
17.46% <0.00%> (+1.58%)
0.00% <0.00%> (ø%)

Continue to review full report at Codecov.

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

:broken_heart: Tests Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: [Pull request #1464 opened]

Start Time: 2020-10-30T16:48:15.466+0000

Duration: 44 min 47 sec

Test stats :test_tube:

Test
Results

Failed
1

Passed
1601

Skipped
12

Total
1614

Test errors
Expand to view the tests failures

Name: Tests / Smoke Tests 02 / greetingShouldReturnDefaultMessage – co.elastic.apm.spring.boot.SpringBootTomcatIT

Age: 1
Duration: 0.454
Error Details:
Expecting:

to be equal to:
<"spring-boot-test">
but was not.

Steps errors
Expand to view the steps failures

Name: Shell Script

Description: ./scripts/jenkins/smoketests-02.sh

Duration: 26 min 59 sec

Start Time: 2020-10-30T17:05:49.564+0000

log

Log output
Expand to view the last 100 lines of log output

[2020-10-30T17:31:53.098Z] [websphere] | ConnectionInstrumentation                          |       3,035,271 |               0 |
[2020-10-30T17:31:53.098Z] [websphere] | AbstractSpanInstrumentation$EndWithTimestampInstrumentation |       2,808,779 |         213,937 |
[2020-10-30T17:31:53.098Z] [websphere] | ElasticApmApiInstrumentation$CurrentSpanInstrumentation |       2,710,113 |         276,549 |
[2020-10-30T17:31:53.098Z] [websphere] | AbstractSpanInstrumentation$GetIdInstrumentation   |       2,801,302 |         178,406 |
[2020-10-30T17:31:53.098Z] [websphere] | ApmSpanInstrumentation$LogInstrumentation          |       2,953,533 |               0 |
[2020-10-30T17:31:53.098Z] [websphere] | ScheduledTransactionNameInstrumentation            |       2,949,300 |               0 |
[2020-10-30T17:31:53.098Z] [websphere] | LegacySpanInstrumentation$DoCreateSpanInstrumentation |       2,943,992 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | AbstractSpanInstrumentation$DoCreateSpanInstrumentation |       2,632,446 |         297,363 |
[2020-10-30T17:31:53.099Z] [websphere] | LegacySpanInstrumentation$EndInstrumentation       |       2,925,005 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | AbstractSpanInstrumentation$CaptureExceptionInstrumentation |       2,679,035 |         230,214 |
[2020-10-30T17:31:53.099Z] [websphere] | ServerCallHandlerInstrumentation                   |       2,906,801 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ElasticApmApiInstrumentation$CaptureExceptionInstrumentation |       2,689,831 |         196,771 |
[2020-10-30T17:31:53.099Z] [websphere] | AbstractSpanInstrumentation$SetTypesInstrumentation |       2,705,289 |         179,440 |
[2020-10-30T17:31:53.099Z] [websphere] | ApmSpanInstrumentation$GetTraceContextInstrumentation |       2,882,691 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | AbstractSpanInstrumentation$IsSampledInstrumentation |       2,713,096 |         162,354 |
[2020-10-30T17:31:53.099Z] [websphere] | ElasticApmTracerInstrumentation                    |       2,844,153 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | AbstractSpanInstrumentation$SetTypeInstrumentation |       2,610,892 |         230,283 |
[2020-10-30T17:31:53.099Z] [websphere] | ApmSpanBuilderInstrumentation$CreateSpanInstrumentation |       2,827,905 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | AbstractSpanInstrumentation$AddNumberLabelInstrumentation |       2,621,844 |         201,548 |
[2020-10-30T17:31:53.099Z] [websphere] | ScopeManagerInstrumentation$CurrentTraceContextInstrumentation |       2,801,137 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ScopeManagerInstrumentation$ActivateInstrumentation |       2,774,737 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ServerCallListenerInstrumentation$FinalMethodCall  |       2,742,512 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | AbstractSpanInstrumentation$InjectTraceHeadersInstrumentation |       2,564,011 |         178,248 |
[2020-10-30T17:31:53.099Z] [websphere] | ExternalSpanContextInstrumentation$ToSpanIdInstrumentation |       2,733,851 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | Lettuce5StopSpanInstrumentation$OnCompleteExceptionally |       2,724,446 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ElasticsearchClientAsyncInstrumentation            |       2,723,307 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | AbstractSpanInstrumentation$EndInstrumentation     |       2,528,223 |         174,248 |
[2020-10-30T17:31:53.099Z] [websphere] | ScopeManagerInstrumentation$CurrentSpanInstrumentation |       2,700,437 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | FutureInstrumentation$RunInstrumentation           |       2,699,164 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | AbstractSpanInstrumentation$GetTraceIdInstrumentation |       2,497,823 |         187,520 |
[2020-10-30T17:31:53.099Z] [websphere] | OkHttpClientAsyncInstrumentation                   |       2,659,414 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ChannelInstrumentation                             |       2,659,580 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ServerCallListenerInstrumentation$NonFinalMethodCall |       2,644,351 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | Lettuce5StopSpanInstrumentation$OnCancel           |       2,635,830 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ApmSpanInstrumentation$SetOperationName            |       2,628,139 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ClientCallImplInstrumentation$Start                |       2,626,489 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | LegacySpanInstrumentation$ActivateInstrumentation  |       2,613,306 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ApmSpanInstrumentation$FinishInstrumentation       |       2,604,783 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | TimerTaskInstrumentation                           |       2,588,306 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | SpanContextInstrumentation$BaggageItemsInstrumentation |       2,535,671 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | LegacySpanInstrumentation$IsSampledInstrumentation |       2,534,683 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ApiScopeInstrumentation                            |       2,525,786 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ServerCallInstrumentation                          |       2,484,785 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ApmSpanInstrumentation$TagInstrumentation          |       2,439,961 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | SpanContextInstrumentation$ToTraceIdInstrumentation |       2,422,676 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | ApmScopeInstrumentation                            |       2,379,667 |               0 |
[2020-10-30T17:31:53.099Z] [websphere] | OkHttp3ClientAsyncInstrumentation                  |       2,267,698 |               0 |
[2020-10-30T17:31:53.099Z] [websphere]
[2020-10-30T17:31:53.099Z] [websphere] 2020-10-30 17:31:52,811 [elastic-apm-circuit-breaker] INFO  co.elastic.apm.agent.impl.circuitbreaker.CircuitBreaker - Stopping the Circuit Breaker thread.
[2020-10-30T17:31:53.099Z] [websphere] [AUDIT   ] CWWKE1100I: Waiting for up to 30 seconds for the server to quiesce.
[2020-10-30T17:31:53.099Z] [websphere] [AUDIT   ] CWWKT0017I: Web application removed (default_host): https://60de82f11025:9443/simple-webapp/
[2020-10-30T17:31:53.099Z] [websphere] [AUDIT   ] CWWKT0017I: Web application removed (default_host): https://60de82f11025:9443/cdi-app/
[2020-10-30T17:31:53.099Z] [websphere] [AUDIT   ] CWWKT0017I: Web application removed (default_host): https://60de82f11025:9443/jsf-http-get/
[2020-10-30T17:31:53.099Z] [websphere] [AUDIT   ] CWWKZ0009I: The application simple-webapp has stopped successfully.
[2020-10-30T17:31:53.099Z] [websphere] [AUDIT   ] CWWKZ0009I: The application jsf-http-get has stopped successfully.
[2020-10-30T17:31:53.099Z] [websphere] [AUDIT   ] CWWKZ0009I: The application cdi-app has stopped successfully.
[2020-10-30T17:31:54.048Z] [websphere] [AUDIT   ] CWWKE0036I: The server defaultServer stopped after 19.386 seconds.
[2020-10-30T17:31:54.315Z] 2020-10-30 17:31:54,270 [main] INFO  co.elastic.apm.servlet.AbstractServletContainerIntegrationTest - Check file /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-1464/src/github.com/elastic/apm-agent-java/integration-tests/application-server-integration-tests/../simple-webapp/target/simple-webapp.war
[2020-10-30T17:31:54.315Z] 2020-10-30 17:31:54,270 [main] INFO  co.elastic.apm.servlet.AbstractServletContainerIntegrationTest - Check file /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-1464/src/github.com/elastic/apm-agent-java/integration-tests/application-server-integration-tests/../jsf-app/jsf-app-dependent/target/jsf-http-get.war
[2020-10-30T17:31:54.315Z] 2020-10-30 17:31:54,270 [main] INFO  co.elastic.apm.servlet.AbstractServletContainerIntegrationTest - Check file /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-1464/src/github.com/elastic/apm-agent-java/integration-tests/application-server-integration-tests/../cdi-app/cdi-app-dependent/target/cdi-app.war
[2020-10-30T17:31:55.304Z] script returned exit code 143
[2020-10-30T17:31:56.425Z] Post stage
[2020-10-30T17:31:56.450Z] Recording test results
[2020-10-30T17:31:58.187Z] Running in /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-1464/src/github.com/elastic/apm-agent-java
[2020-10-30T17:31:58.225Z] [INFO] Codecov: Getting branch ref...
[2020-10-30T17:31:58.264Z] Masking supported pattern matches of $GITHUB_TOKEN
[2020-10-30T17:31:58.450Z] [INFO] Codecov: Sending data...
[2020-10-30T17:31:58.945Z] + curl -sSLo codecov.sh https://codecov.io/bash
[2020-10-30T17:31:59.348Z] + bash codecov.sh
[2020-10-30T17:31:59.348Z]
[2020-10-30T17:31:59.348Z]   _____          _
[2020-10-30T17:31:59.348Z]  / ____|        | |
[2020-10-30T17:31:59.348Z] | |     ___   __| | ___  ___ _____   __
[2020-10-30T17:31:59.348Z] | |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
[2020-10-30T17:31:59.348Z] | |___| (_) | (_| |  __/ (_| (_) \ V /
[2020-10-30T17:31:59.348Z]  \_____\___/ \__,_|\___|\___\___/ \_/
[2020-10-30T17:31:59.348Z]                               Bash-20201009-048fee3
[2020-10-30T17:31:59.348Z]
[2020-10-30T17:31:59.348Z]
[2020-10-30T17:31:59.348Z] ==> Jenkins CI detected.
[2020-10-30T17:31:59.348Z]     project root: .
[2020-10-30T17:31:59.348Z] --> token set from env
[2020-10-30T17:31:59.348Z]     Yaml not found, that's ok! Learn more at http://docs.codecov.io/docs/codecov-yaml
[2020-10-30T17:31:59.349Z] ==> Running gcov in . (disable via -X gcov)
[2020-10-30T17:31:59.613Z] ==> Python coveragepy not found
[2020-10-30T17:31:59.613Z] ==> Searching for coverage reports in:
[2020-10-30T17:31:59.613Z]     + .
[2020-10-30T17:32:00.562Z] --> No coverage report found.
[2020-10-30T17:32:00.562Z]     Please visit http://docs.codecov.io/docs/supported-languages
[2020-10-30T17:32:00.898Z] Failed in branch Smoke Tests 01
[2020-10-30T17:32:01.172Z] Stage "Integration Tests" skipped due to earlier failure(s)
[2020-10-30T17:32:01.261Z] Stage "AfterRelease" skipped due to earlier failure(s)
[2020-10-30T17:32:01.299Z] Stage "AfterRelease" skipped due to earlier failure(s)
[2020-10-30T17:32:01.649Z] Running on Jenkins in /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-1464
[2020-10-30T17:32:01.754Z] [INFO] getVaultSecret: Getting secrets
[2020-10-30T17:32:01.851Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID
[2020-10-30T17:32:02.760Z] + chmod 755 generate-build-data.sh
[2020-10-30T17:32:02.760Z] + ./generate-build-data.sh https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1464/ https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1464/runs/1 FAILURE 2627026
[2020-10-30T17:32:03.010Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1464/runs/1/steps/?limit=10000 -o steps-info.json
[2020-10-30T17:32:03.561Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1464/runs/1/tests/?status=FAILED -o tests-errors.json


DISP-S1 Integration Test Pipeline

Ticket to track setup of Integration Test pipeline for DISP-S1 PGE. This includes formation and staging of the expected input/output datasets in S3. Note for DISP-S1, the integration test should utilize the "minimal" set of expected inputs and outputs referenced in #283 .

Resolved by #308


Add KeyCloak Profile Link

It would be nice, if a logged in user could click on his name in the upper right corner and be connected to the KeyCloak profile login, e.g., https://keycloak.dukecon.org/auth/realms/dukecon-testing/account/.
@srose / @ahus1 : As far as I can see navigating to the link is sufficient when logged in (re-use of tokens), right?

Yes, this should be fine. Keycloak creates an "Back to application" link. This should work fine (but please verify it)

To make this work without having to change variables, I'll have to somehow extract the realm from the url. Having a look at the options now.

I don't see a "Back to Application" link on the profile page.
Does this need to be added? If that's not possible, perhaps the profile should be opened in a new tab/window?

The realm can be retrieved from /rest/keycloak.json. If desired we could also add the full URL of the KC user settings URL (or at least the generic part)?

I got the realm figured out :)


Makefile parameter "-j"

Hello, I use a CPU with 2 cores and I was wondering if I were to use the "-j8/16" parameter in the Makefile, would that make the computer demand too much of my CPU?
Sorry if my English is bad, I'm using google translator =)

If you have a CPU with only two cores, you probably have to use -j2:
https://www.gnu.org/software/make/manual/html_node/Parallel.html

Ok, thankyou =)


Chat - Original message appears after editing in offline mode and relaunch the app

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.2-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Pre-requisite: user must be logged in

Go to any chat and send a message
Disable internet connection
Edit the recently sent message
Edit it again to something new
Force-kill and relaunch the app
Enable internet connection
Open New Expensify app again and go to chat

Expected Result:
The message should be displayed as last edit
Actual Result:
Original message appears
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/6522cd66-4f17-4d50-83be-1a3c116bb628
View all open jobs on GitHub

Production:
https://github.com/Expensify/App/assets/78819774/ac35351a-84f3-4aff-b44d-d8da86e709c3

I have a feeling that this was caused by https://github.com/Expensify/App/pull/30425 which was reverted and CPed

Testing in staging now

yea, can't repro in latests staging anymore. Gonna close this issue.


Filter the logs

Filtering the loggers to log only warnings and errors.
In both the server and client side.

How should this be acomplished?

The most simple way is by adding a logging filter
builder.Logging.AddFilter("Microsoft.EntityFrameworkCore.Database.Command", LogLevel.Warning);

, but I don't think it's the best practice.

I see, so this is related only to EF Core?
Perhaps some there should be some global error logger that picks up errors?

I found another way by adding the LogLevel in appsettings.json.
https://learn.microsoft.com/en-us/ef/core/logging-events-diagnostics/simple-logging
At the moment I see the most interaction with the database is when we retrieving or sending data to the database. We can just filter this option by adding "Microsoft.EntityFrameworkCore.Database": "Warning" as a property to the LogLevel.

That sounds good as we do not really need to know when a database operation was performed.
Only errors and warnings are of interest.


Constraint should describe if axis is inverted

Simple addition, lots of fiddly things to update:

unit tests
notebooks
documentation

Have to undo these changes.  Axis inverted is applied in code, not as a state that can be saved and restored.  It's not a constraint parameter.


New version of Jenkins (2.107.1) is available for download

As of 2018-03-23

Try reproducing issue on nemo:

OS: Ubuntu 16.04.4 LTS 64-bit with latest updates
Docker version 18.03.0-ce, build 0520e24
docker-compose version 1.19.0, build 9e633ef
easy-jenkins master (8af0182be6b549c2321975f9ac13782eb57ea6ae)

Performed a scratch build of the Docker image
docker-compose build --pull

then reinstalled containers
docker-compose down -v
./runme.sh

Result: TODO

Closing as the main subject of this issue is no longer reproducible.
Tracking the remaining two warnings in the following issues:

[ ] https://github.com/gmacario/easy-jenkins/issues/258
[ ] https://github.com/gmacario/easy-jenkins/issues/256


#26 Thai translation

Thanks for this! Please replace the link in credits_content6 with your information. You did the translation so you get to be in the app credits.

Thank you! I'll have this available in the next release.


mdSidenav method "show" should be "open" according to the docs

According to the docs, mdSidenav should have an open method but when looking into the source code, it appears it is called show.
See: https://github.com/marcosmoura/vue-material/blob/master/src/components/mdSidenav/mdSidenav.vue#L29

Steps to reproduce
Which browser?

What is expected?
What is actually happening?

Reproduction Link

<!-- You could start with this template: http://codepen.io/vue-material/pen/WGavBE.

Oh. Yeah. You're right. I will fix this ASAP.


Fix bad output from nc-config

I was getting "yes" output from some of the options which were not enabled.  This was due to two reasons:

If CMake set the variable to "OFF", then the -z $var test would fail and $var would be set to "yes" even though it should be "no"
If CMake set the variable to empty string (has_var=""), then the -z $has_var test would also fail and set it to yes instead of no.

With these changes, it looks like I am getting the correct output from nc-config consistent with my configuration options.

Thanks; would it be possible to re-issue this pull request against the 4.5.0-release-branch branch? If not I will merge it in manually.

I will try but may not be until tomorrow or Sunday.


/public vs /src/assets behavior

OS?  Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)
Mac OSX El Capitan
Versions. Please run ng --version. If there's nothing outputted, please run
in a Terminal: node --version and paste the result here:
angular-cli: 1.0.0-beta.11-webpack.8
node: 5.10.1
os: darwin x64

The latest version of the cli replaces the public directory with the new src/assets directory which works great for copying static files into dist but I'm not sure if the behavior is quite right. Many files are expected to be on the root of a hosted site such as robots.txt and ico files. When the cli copies files over from assets into the dist they are placed at dist\assets. Is there any way to be able to copy files over to the root level of the dist?

ico/faviocon files can be anywhere, the default location is usually the root, but you can change the locations in html. I consider robots and favicons web specific, not directly under the app scope. I would place them in the dist folder and commit those under git.
If you must change this behavior, edit the angular-cli.json assets key, changing it to a . should work.

@robertbaker I've tried this. Turns out it doesn't work. But I think that this is a legitimate question of @splintercode. It should definetly be possible to add files like a robots.txt without hacking. assets should by default be mapped the way public was mapped. If you want a assets folder in your dist, you can always create a assets/assets folder.

This new behavior also affects our projects here. We have had to work around in the obvious hacky way, by building a build process around the CLI-provided build process which adds the files we need at the "root" of the output.
I think most real projects will eventually have at least one, and often many, files that needs to be located somewhere in the "root" of the output - files that CLI doesn't know about and can't be responsible for. Until .webpack.3..8, this was supported and worked very well. I request the feature be put back and kept in the future also.

I would be fine if the contents of assets just where emitted to dist/ rather than dist/assets. The public folder bugged me anyway, because everything in dist is public, too. Those are assets, therefore the name of the folder should be assets. Maybe resources, but please not public. Coming from a PHP background, this confused me a lot. I was thinking that the build process didn't work, because I was expecting the websever to serve from public.

Summary:

Rename of public to assets: good!
Restriction from being able to locate assets at dist root: bad!

@kylecordes exactly, thank you, except:

Restriction from not being able to locate assets at dist root: bad!

If angular-cli.json can configure "assets" property value as an array, that will be good.
For example:
"assets": ["assets", "robots.txt"],

I really don't want a solution like @doggy8088 outlined, in which I need to... tell... CLI... about... each... and... every... file... I... want... to... include.....

@doggy8088 I think thats unnecessary, because if the contents of assets would be emitted directly into dist, you wouldn't need that. If you want a assets folder in dist, you could just create another assets folder. Like so:
assets
├ assets
   ├ images
   ⋮
   └ .htaccess
├ robots.txt
└ favicon.ico

would be mapped to:
dist
├ assets
   ├ images
   ⋮
   └ .htaccess
├ bundle.js
⋮
├ robots.txt
└ favicon.ico

@christiandreher Actually, I like the original pubilc folder design better than current one.  For the current 1.0.0-beta.11-webpack.8 design, the src/assets will copy to under dist/ so there is nothing can copy to the root of the 'dist/' folder, only the assets folder does.

That's exactly what I am saying. Like I already said, I just didn't like the name public, because in the end, everything is public

/public was fine, in my opinion (while also adding the possibility to customize copying other files too, like they've added /src/assets). But the /public folder behavior needs to be brought back (under the same or under a different folder name or even better: give us to possibility to set it up or not, our choosing). Problem with /public was: the cli was not copying .dot files over (like .htaccess) - see #1758. And when they fixed copying .dot files, they've also removed the /public folder, and closed my ".htaccess not getting copied" bug :)) nice :|
Anyway, adding the /src/assets folder is not bad, perhaps. But we'd also definitely need to have files copied to the root of the /dist folder, and certainly not have to point the cli to each and every single file.

This is preventing our team from upgrading to the latest CLI version as we have to have files in the root during development time. ng serve and ng build --watch wipe out the directory when building at development time. Has anyone come up with a workaround for this?

It shouldn't have anything to do with the "mode" we're in (development or production). Custom files need to be copied to dist root no matter if you're in development mode or building for production

@MrCroft correct, we have a workaround for a ng build command, just have a script to copy the files over to the dist but as for dev time ng serve or ng build --watch there doesnt seem to be a temporary workaround for this. I'm looking around to see if I can submit a PR but unfortunately my webpack skills are not up to par so might take a bit to figure how everything works.

@splintercode oh, yeah...
Well, the solution needs to come from the CLI of course, because it should work both when building and serving, with the prod flag or without it, like it was intended before the public folder was removed (just that it had some bugs).
I wouldn't know how to accomplish that either, I've barely started to look at webpack recently.
And I don't know what they're planning here, but I suppose we will have the option to set up something similar to the public folder, because sooner or later everybody needs to have a bunch of files copied over to dist root. Whether we're serving or building, for production or not... the only difference is that in cases like yours (serve / build --watch) the dist folder isn't created physically in the project folder, but everything should work the same (wherever the dist/ files may reside - project folder, temp folder or memory).

Looks like this but was introduced with this changeset https://github.com/angular/angular-cli/commit/3dcd49bc625db36dd9f539cf9ce2492107e0258c

+1 on bringing back public. It setup nicely for my bower dependencies (polymer) and a couple of others. Not sure what the official word on how to handle bower dependencies.

@jjarrell50 for now, with webpack.8 at least, you can add styles (either css or even scss, if the app was initialized with --style=sass) and scripts by using the styles=[] and scripts: [] arrays in angular-cli.json (I've tried by giving paths relative to the angular-cli.json file). So you can have your bower components folder anywhere (or even get your packages through npm instead of bower) and tell the cli what styles/scripts you need from there (--prod flag also minifies/bundles those).
More than that, the problem with public being dropped is that there is no way to tell the cli to simply copy other/any types of files to dist root, as is, not just js/css (.dot files and other types of files that need to be directly in the dist root and nowhere else)

What if I want to copy some assets from node_modules (maybe svg files or something else)
with  @doggy8088 solution it can be possible.
What do you think ?

Yes, like I've already said (and others as well, including @doggy8088), we definitely need the public functionality back (a possibility to have any kind of files copied over anywhere we'd need them, up to the root of dist, not just in a sub-folder).

Per default, I would just copy the contents of assets/ into dist/ (So that assets/example.txt maps to dist/example.txt)
For assets that are located in other directories, like node_modules/ as @serhiisol mentioned, we could have an dedicated option in angular-cli.json to take care of that. This is mostly what @doggy8088 described, except for the fact that you don't have to explicitely mention every file in assets/ which should directly be copied to dist/. This would be way to verbose. I hope that the difference is clear

Anyone looking for a temporary fix, especially for those needing .htaccess - in your angular-cli.json file set:
assets:""
And this will copy over everything from src/
Definitely not a solution but will let you continue development until the public folder behavior is sorted.

To me it looks like  @CodeJason's inbetween solution will break the compile on the second cycle of ng build --watch or ng serve. Works the first time, but as soon as you make changes, the webpage remains white and no JS is loaded. Can anyone verify this?

Sorry, I should have mentioned I had only tried it with ng build.
-----Original Message-----
From: sqwk<EMAIL_ADDRESS>To: angular/angular-cli<EMAIL_ADDRESS>Cc: CodeJason<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Sent: Wed, 14 Sep 2016 5:58 pm
Subject: Re: [angular/angular-cli] /public vs /src/assets behavior (#1942)
To me it looks like  @CodeJason's inbetween solution will break the compile on the second cycle of ng build --watch or ng serve. Works the first time, but as soon as you make changes, the webpage remains white and no JS is loaded. Can anyone verify this?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/angular/angular-cli/issues/1942#issuecomment-246941907

Should I expect .scss files in ./src/assets to be compiled? In
angular-cli: 1.0.0-beta.11-webpack.8
node: 6.5.0
os: linux x64

they are not, even though my component-level .scss files are. If not, where should cross-component .scss files go?

@JanStureNielsen It's for static assets only.
But this should give you what you want.
http://stackoverflow.com/a/39497573/263457

If you are using some kind of CI / build chain simply just copy the .htaccess.
Here an example if u are using FTP
lftp -e "put src/.htaccess; bye" -u $USERNAME,$PASSWORD $HOST

Potentially similar issue with the React Create App project
https://github.com/facebookincubator/create-react-app/issues/667

Hi everyone! Lengthy discussion.
So what is the current workaround, folks? Where do we put favicon/robots/some other data files?

Unfortunately, my pull request was closed, as there is a more elaborate fix in the works. For those wanting the old functionality back now, you can use npm link to link to a local git repo. There you can change one line in packages/angular-cli/models/webpack-build-common.ts to copy files to the dist root instead. (See pull request above.)
However, do know that this is only a temporary fix and will change again in the near future.

Why would you npm linkthe angular-cli project? Just copy the files needed after ng build --prod into your dist folder - automated or manually.

@cschroeter that doesn't work with ng s, like multiple times mentioned above

@christiandreher You don't have a deploy script or a build chain? I'm hosting my source code on GitLab and in my .gitlab-ci.yml  I simply do:
deploy_stage:
  image: ubuntu:16.04
  stage: deploy
  dependencies:
    - build_stage
  script:
    - apt-get update -qy
    - apt-get install -y lftp
    - lftp -e "mirror --reverse --delete --use-cache --verbose $LOCAL_DIR $REMOTE_DIR ; bye" -u $USERNAME,$PASSWORD $HOST
    # Workaround as long as angular-cli don't support public folders
    # https://github.com/angular/angular-cli/issues/1942
    - lftp -e "put src/.htaccess; bye" -u $USERNAME,$PASSWORD $HOST
    - lftp -e "put src/robots.txt; bye" -u $USERNAME,$PASSWORD $HOST
  only:
    - master

If this is to advanced simply install npm install --save-dev cpx and add to your package.json
  "scripts": {
    ...
    "build": "ng build --prod && cpx 'src/.htaccess' dist"
  }

@cschroeter Well, you can always go nuts and have dozens of other tools (chained or not)...
But I think the point is not to bring another tool for a simple task, especially one which the CLI could/should easily do.

I am also using gitlab, but I don't want to commit + push + run all builds + deploy to finally view what I've changed. I just want to make a change, head to my browser, and see if it works.

I'm not saying this is the preferred solution but adding one devDependency and extending the build command by a few characters isn't that big of a deal
.. considering the fact that this will be the 37th comment in this thread ^^

Once again, this does not work for ng s! It needs to be changed in the webpack config, like @sqwk said, theres no other possibilty. That's all I said the whole time

My proposal is the same as most people here, is to forge the assets behaviour complete, and return the public folder behaviour obviously. Keep it outside of src, and call it resources or static or whatever instead of public, whatever makes sense as property name in angular-cli.json.
P.S.
Thanks for everyone who helped create this awesome free open source CLI.

I agree with @Meligy.  I also am using IIS and require web.config file (btw, thanks for rewrite rules example @Meligy).  I think these static files should be outside of src and should be called static which get written to root of dist folder.  If there are subfolders in 'static', then they become subfolders of dist.
Ideally, I'd like dist to contain subfolders for each environment as opposed to wiping out dist and starting over.  If I do 'ng build' and 'ng build --prod', then dist would be:
dist
|-- dev
|-- prod
|-- [whatever other environment you have]...
this allows me to compare different environments.
Thanks for your time.

@tomschreck just use the --output-path parameter. ng b --dev --output-path dist-dev and ng b --prod --output-path dist-prod. There is no need to have environments in dist, as this functionality already exists. If you don't want to type that command, just make some npm scripts.

At least lets have the root public/assets/resources/static folder back for the shown scenarios please.

@Meligy To get around the IIS issue (I am in the same situation)  build your project into dist/build
source control dist except dist/build. Now you can put your web config at the root but ignore generated code. Then to build you have to run ng build --watch -o dist/build. Update your web config rewrite rule to point to the index in dist/build.

@christiandreher - thanks for the tip.  good to know

@splintercode I was planning on something like that until I realised: in this way I need a rewrite for the routes, and a rewrite for the script files and other assets. That won't work very well.
The script tags in the HTML will be root (dist) relative because the browser thinks the HTML is served from the root when the rewrite happens, the script tags now will match nothing because there's nothing in root (dist), so, I'll need to have them rewritten not to dist/build/ (default document being index.html), but to dist/build/{path}.
Which sounds OK, until you think about client routes, those should not go to dist/build/{path}. They should go to dist/build/ only (effectively dist/build/index.html as default document)
It then gets very complex, for something that I already had solved when the public folder was supported, because I didn't need to worry about relation to scripts in that scenario, they are matched as existing physical files, and my rewrites only worry about client routes.

We realize that the change from public/ to src/assets meant that now there's no way to copy files to the root of the dist directory, and we're looking at alternatives for that.
The main motivations for src/assets were:

better support of upcoming multi-app setups by having a local folder instead of a global one
maintain reference integrity for editor tooling and just general making sense (e.g. referencing ./assets/img.png from ./src/styles.css exists, and the same if you want to reference something from index.html)

Point two presents a problem for any setup that copies stuff from a certain folder to the dist root.
There's a number of solutions that we're considering, but I don't have any to share at the moment since we haven't decided what avenue to take.

@filipesilva also referenced this: https://github.com/angular/angular-cli/issues/2232

I was hosting bower references (polymer) into the public/bower_components
and referencing it from index.html with something like this

  <!--Polymer-->
  <script src="bower_components/webcomponentsjs/webcomponents.js"></script>

  <script>
    window.Polymer = window.Polymer || {};
    window.Polymer.dom = 'shadow';
  </script>

  <!--Polymer-->
  <link rel="import" href="bower_components/polymer/polymer.html">
  <link rel="import" href="bower_components/paper-styles/color.html">

So i tried to redirect the reference to assets/bower_components. it didn't
work, 404s. I'm thinking there is more to it than a directory rename and
would appreciate it if someone could confirm that. Perhaps, webpack has a
subtle change from it's former public behavior.
Can anybody confirm, that the move to assets is more than just a rename of
the folder?

Why are you using bower, there is no need to use it, especially for Polymer.
On Wed, Sep 21, 2016, 10:50 PM jeff<EMAIL_ADDRESS>wrote:

I was hosting bower references (polymer) into the public/bower_components
and referencing it from index.html with something like this

<!--Polymer-->
<script src="bower_components/webcomponentsjs/webcomponents.js"></script>

<script>
window.Polymer = window.Polymer || {};
window.Polymer.dom = 'shadow';
</script>

<!--Polymer-->
<link rel="import" href="bower_components/polymer/polymer.html">
<link rel="import" href="bower_components/paper-styles/color.html">

So i tried to redirect the reference to assets/bower_components. it didn't
work, 404s. I'm thinking there is more to it than a directory rename and
would appreciate it if someone could confirm that. Perhaps, webpack has a
subtle change from it's former public behavior.
Can anybody confirm, that the move to assets is more than just a rename of
the folder?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/angular/angular-cli/issues/1942#issuecomment-248818252,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3YJcUyFR9F3wznbdWa88oJU6E1Qtnyks5qshc4gaJpZM4Jy29p
.

Why are you using bower? You shouldn't need it anymore. Use 'npm i --save polymer'

@robertbaker - you got me encouraged for a moment, but the NPM polymer is not the polymer I'm referring to. The one I'm referring https://www.polymer-project.org/1.0/ which happens to be a google project.  The docs all say it's bower.  It's much farther along, today than Angular2 Material Design. At this point, I'm looking any solution, I'm thinking about serving these dependencies CDN style, at least until I can figure a way to include them in my build.
I'd generally expect support in angular-cli for polymer. They both come out of the googleverse after all.

@filipesilva would it be too bold to ask what are the major downsides of the following approach?

Bring back public folder into the root, same level as src
Add a key copyToOutput, pointing at "public"
Copy the folder to output with some Webpack copy plugin

I can think of a few questions

It might complex / confusing people
You can avoid generating folder and key by default, and leave it as documented feature only
People might be greedy and ask for an array
Shouldn't be a big difference though to implement or ignore
What if people want it in src, or even apps
Since it's relative to root, people can still do that

Are there other concerns with this approach compared to other approaches? It's the closest one to the old approach, which is proven valuable by many comments here.
P.S.
Thanks a lot for all your own voluntary efforts in this free tool

Another workaround for those who need to include web.config into the prod build, here is another workaround that works. I simply added an entry for it in the assets section of angular-cli.json
"assets": [
"assets",
"favicon.ico",
"Web.config"
]
Webpack will then copy it from src/app to the root of dist.

Thats no workaroound, @cheresier, thats how it is intended to work after the update. The issue is old, back then it was not possible to do that

Oh, cool. Sorry, Looks like I got the wrong idea looking through the thread that we are looking for a solution equivalent to the /public folder.

Is there really no other way than typing every file into apps[0].assets?
Eg. when using http://realfavicongenerator.net to generate the favicon files you get ~10 files - thats a lot of clutter to have in the root folder. (Not including .htaccess or web.config and robots.txt)

@Nikkelmann There is nothing stopping you from putting your favicons into assets/favicons and adding the favicon/browser meta tags in your index for it. That's what I do and works fine.

@robertbaker Sure, but the recommended way is to put the files in the root folder.
http://realfavicongenerator.net/faq

Why do you recommend to put all files at the root of the web site?
When generating a favicon with RealFaviconGenerator, the instructions ask you to place all files at the root of your web site. You may want to place them in a sub-directory, for example in http://mywebsite.com/icons/, just to make things clearer. However, there are three drawbacks with this approach:

Internet Explorer looks for favicon.ico at the root of the web site. Granted: this is because we ask you to not declare favicon.ico.
iOS devices look for files such as apple-touch-icon-144x144.png at the root of the web site, as described by Apple. This issue can be mitigated by declaring the icons in the HTML code (this is necessary for Android anyway), but following Apple conventions is probably the best move.
By default, Internet Explorer 11 looks for browserconfig.xml at the root of the web site.
Several services, such as Yandex, look for favicon.ico in the root directory.

Why is favicon.ico not declared in the HTML code?
Looking for some sample favicon code, you've probably encountered:
<link rel="shortcut icon" type="image/x-icon" href="favicon.ico">
When your favicon is in the root directory, RealFaviconGenerator does not generate this code. This is because it somehow confuses some other browsers like Chrome. And since IE looks for a file named favicon.ico anyway, the best solution is to not even talk about it. This is described by Jonathan T. Neal (http://www.jonathantneal.com/blog/understand-the-favicon/) and Mathias Bynens (http://mathiasbynens.be/notes/rel-shortcut-icon).

Really? But W3C says this is a bad thing!
Yes, W3C discourages the usage of a predefined URI. If you follow this recommandation, you will get into troubles with browsers which wrongly choose favicon.ico instead of the nice PNG versions gently prepared just for them. It is way better and pragmatic to trick them with this little hack.


Stop logging confirmation count for older transactions when already in a state where these don't matter anymore

For instance, at the Info level, we don't need to see the Lock confirmation count once AccLock has been published - that just pollutes the interface for normal users.

Fixed in 2854f104a88c62e8be70db49ff8c2e56833f7cc7


InsufficientFreeAddressesInSubnet

Version

Karpenter Version: v0.23.0

Kubernetes Version: v1.23.14
Expected Behavior
Successfully auto scale up EKS EC2 g5.12xlarge instances.
Successfully deploy application.
Actual Behavior
Successfully auto scaled up EKS EC2 g5.12xlarge instances.
Failed to deploy the application.
Ran out of IPs in the subnets, subnet-03e6c02fabb432142. EKS installed in 6 subnets. One subnet run out of IPs. Other 5 subnets the available IPs are all 250. Karpenter should auto use other five subnets that have available IPs.
Errors:
 ERROR   controller.provisioner  launching machine, creating cloud provider instance, creating instance, with fleet error(s), InsufficientFreeAddressesInSubnet: There are not enough free addresses in subnet 'subnet-0783288fba3453a60' to satisfy the requested number of instances.; launching machine, creating cloud provider instance, creating instance, with fleet error(s), InsufficientFreeAddressesInSubnet: There are not enough free addresses in subnet 'subnet-0783288fba3453a60' to satisfy the requested number of instances.        {"commit": "5a7faa0-dirty"}

Steps to Reproduce the Problem
Provisioner:
apiVersion: karpenter.sh/v1alpha5
kind: Provisioner
metadata:
  name: provisioner-virginia
spec:
  requirements:
    - key: karpenter.sh/capacity-type
      operator: In
      values: ["spot", "on-demand"]
    - key: node.kubernetes.io/instance-type
      operator: In
      values: ["g5.12xlarge"]
  limits:
    resources:
      cpu: 10000
      memory: 10000Gi
  taints:
  - effect: NoSchedule
    key: "nvidia.com/gpu"
    value: "true"
  providerRef:
    name: provisioner-virginia
  ttlSecondsAfterEmpty: 300 # 5 minutes, comment out for indefinite
  ttlSecondsUntilExpired: 604800 # 7 days, commented out for indefinite
---
apiVersion: karpenter.k8s.aws/v1alpha1
kind: AWSNodeTemplate
metadata:
  name: provisioner-virginia
spec:
  subnetSelector:
    karpenter.sh/discovery/eks-virginia: eks-virginia
  securityGroupSelector:
    karpenter.sh/discovery/eks-virginia: eks-virginia

Deployment:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: inflate-gpu
spec:
  replicas: 10
  selector:
    matchLabels:
      app: inflate-gpu
  template:
    metadata:
      labels:
        app: inflate-gpu
    spec:
      terminationGracePeriodSeconds: 0
      containers:
        - name: inflate-gpu
          #image: public.ecr.aws/eks-distro/kubernetes/pause:3.2
          image:<PHONE_NUMBER>.dkr.ecr.us-west-2.amazonaws.com/test:1.5.9999999
          resources:
            limits:
              cpu: "12"
              ephemeral-storage: 10Gi
              memory: 48Gi
              nvidia.com/gpu: "1"
            requests:
              cpu: "4"
              ephemeral-storage: 10Gi
              memory: 4Gi
              nvidia.com/gpu: "1"
      tolerations:
      - key: "nvidia.com/gpu"
        operator: "Equal"
        value: "true"
        effect: "NoSchedule"
      nodeSelector:
        node.kubernetes.io/instance-type: g5.12xlarge

Resource Specs and Logs
 ERROR   controller.provisioner  launching machine, creating cloud provider instance, creating instance, with fleet error(s), InsufficientFreeAddressesInSubnet: There are not enough free addresses in subnet 'subnet-0783288fba3453a60' to satisfy the requested number of instances.; launching machine, creating cloud provider instance, creating instance, with fleet error(s), InsufficientFreeAddressesInSubnet: There are not enough free addresses in subnet 'subnet-0783288fba3453a60' to satisfy the requested number of instances.; launching machine, creating cloud provider instance, creating instance, with fleet error(s), InsufficientFreeAddressesInSubnet: There are not enough free addresses in subnet 'subnet-0783288fba3453a60' to satisfy the requested number of instances.    {"commit": "5a7faa0-dirty"}

 ERROR   webhook Failed the resource specific validation {"commit": "5a7faa0-dirty", "knative.dev/kind": "karpenter.sh/v1alpha5, Kind=Provisioner", "knative.dev/namespace": "", "knative.dev/name": "gpu", "knative.dev/operation": "CREATE", "knative.dev/resource": "karpenter.sh/v1alpha5, Resource=provisioners", "knative.dev/subresource": "", "knative.dev/userinfo": "kubernetes-admin"}

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Do the 6 subnets belong to the same AZ?
If your subnets belong to multiple subnets, I think trying topologySpreadConstraints might alleviate the problem. If your subnet is a single AZ, I think it's the same problem as my #3364 issue.

#3364 may not help here. That issue is more around multiple subnets in one AZ but not using the second one in the AZ when doing a lot of launches into the first one with more IPs.
It looks like these 6 subnets are all in different AZs.  I would agree with @zensai3805 that using a topology spread to make these a little more even should help here. That may not be desirable especially from a spot perspective when using GPU types.
If the subnet completely runs out of IPs, then EC2 Fleet should return back an error and not launch the node in the subnet with insufficient IPs. I suspect that the IPs are actually just very low in the subnet, Karpenter launches the node and then the CNI isn't able to allocate IPs to the pods on the node.  We could check that there are enough IPs in the subnet for the max-pods +1 (for the node IP) before we utilize the subnet and that may help here.

Sorry for asking so long after issue is closed, but does that applies only for single zone case? @bwagner5


Version String in Release 2023-12-19

Release 2023-12-19 has:
version = '2023-11-19'  # ISO 8601 (YYYY-MM-DD)
Should that have been:
version = '2023-12-19'  # ISO 8601 (YYYY-MM-DD)
Does it matter?

Thanks for pointing this out. It's not a problem (except in that it could cause confusion), but it's easy enough to overlook, so I made a pre-commit hook to prevent it in future.


fix: Consider existing capacity for scheduling

Fixes #N/A
Description
How was this change tested?
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 17 (100.0%)  changed or added relevant lines in 3 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 81.42%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/provisioning/scheduling/topology.go
2
86.49%

pkg/controllers/provisioning/scheduling/topologygroup.go
2
96.75%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7051

Relevant Lines:
8660

💛  - Coveralls


Action header wrapping

Summary
Changed the word-break to break the entire word so the text does not get cut off.
Closes #1726
Thanks for contributing to Terra.
@cerner/terra
Before:

After:

Deployment Link
https://terra-core-deployed-pr-1966.herokuapp.com/#/components/terra-action-header/action-header/action-header

Functional verification completed. The action header examples display and function correctly with the wrapping changes.


ops.index_add double precision performance on GPUs

It appears that double precision performance of jax.ops.index_add is far slower (maybe around 400x times) than single precision when running on gpu. I used the following code to benchmark:
# Precision to use. Configure: config.update("jax_enable_x64", True)
T=jnp.float64
# Compare to:
T=jnp.float32

# Setup the data
i=jax.device_put(numpy.random.randint(0,12, size=10_000_000))
v=jax.device_put(jnp.array(numpy.random.normal(size=i.shape), dtype=T))
g=jnp.zeros(12, dtype=T)

# Actual timing
%timeit jax.ops.index_add(g, i, v).block_until_ready()

On Google Colab I find double precision is 400 times slower, even when running on an P100 instance.
To speculate: maybe this is due to lack of atomic double precision add on earlier compute capabilities? However I think this was added from Pascal onward.

I can reproduce (it's more than 1000x slower for me). Filed an internal bug for the XLA:GPU team.

The XLA:GPU team confirmed your diagnosis: "The bug is that we don't use atomic add for f64 (available since sm_50)." I think they're working on a fix.

Fixed in https://github.com/tensorflow/tensorflow/commit/e0769309642b8547d9b720b5d42b35db87031dcd. You'll need to either rebuild from source or wait until we update jaxlib binaries.

@jekbradbury I'm going to mark you as "assigned" since you already followed up on this issue, but IMO there's no additional action to take here.
@bnwebcode thanks for finding this, leading to a fix! Even though we haven't updated the jaxlib wheels yet, I'm going to close this issue as fixed (since otherwise it's tricky to remember which issues to close when we update jaxlib). I suspect we'll push new jaxlib wheels in the next week.

Excellent, many thanks! Good to hear the diagnosis was helpful.


feat: animation component and scene list

Owner

[x] Nickname(Tuya): 奥里
[x] Intra-group review（组内审阅人员）：萧瑟,罗丘

Description
feat: animation component and scene list
Package

[x] tuya-panel-lamp-sdk

Type of change

[x] New feature (non-breaking change which adds functionality)

Self CheckList

[x] Intra-group review
[x] ESLint lint
[x] Example Run
[x] Jest Run
[x] No Grammer Error
[x] No Spell Error
[x] No Merge Conflict

@Doraemon91 处理下冲突


Using YAML for similar device

I have a similar device, but it only lets be choose simple switch. How do I force the yaml file for YYM-805SW Aroma diffuser for my device

Not a bug.  You cannot force an incompatible devices config to be used.


Possible move to voxpupuli

Mike,
I opened the following to help out if you are okay with moving. Didn't seem like you wanted to be doing module development anymore:
https://github.com/voxpupuli/plumbing/pull/198
https://github.com/voxpupuli/modulesync_config/pull/551
Let me know if you'd like to do this or not, I've opened #11 if not. (only failure is Ruby 1.8.7 which probably should be removed)
-Chris

Yes.  I am sorting out the migration steps.

@cdenneen It should be complete.

Let me know if razorsedge/puppet-vmwaretools should go over as well.

Yes please
On Mon, Apr 8, 2019 at 10:34 PM Mike Arnold<EMAIL_ADDRESS>wrote:

Let me know if razorsedge/puppet-vmwaretools should go over as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/voxpupuli/puppet-openvmtools/issues/12#issuecomment-481078283,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAr84Qym57k5SIB_HkoKHPMprOb0IPJyks5ve_w4gaJpZM4bZUQi
.

Done.

Fixed by #14 and #15


cert manager not issuing certs for one ingress (in work queue no longer exists)

Describe the bug:
I am rebuilding my cluster using the same argocd config that has worked before and realized that one of the ingress is without SSL certs.
Upon checking, it seems like cert manager is working correctly if I deploy a new app with a different domain, cert is requested and issued.
But for the ingress openldap/openldap-phpldapadmin, cert is not issued, upon checking the log of manager, I only see this one line related to the ingress without the ticket issued
E0502 17:18:45.684718 1 controller.go:98] ingress 'openldap/openldap-phpldapadmin' in work queue no longer exists

Expected behaviour:
Expecting the cert to be issued
Anything else we need to know?:
Ingress including the following annotation
        cert-manager.io/cluster-issuer: letsencrypt-prod-issuer
        cert-manager.io/common-name: ldap.xxx.xx

The configuration for the prod issuer is check to be working by deploying a new app.
Environment details::

Kubernetes version: K3S v1.29.1+k3s2
Cloud-provider/provisioner: Homelab
cert-manager version: helm chart version v1.14.4
Install method: helm

Any help with this is greatly appreciated, thanks.
/kind bug

Fixed the issue adding missing tls config to ingress, not sure why it was missing before


Video feed not working in Agora retail

Is your issue related to a Jumpstart scenario, ArcBox, HCIBox, or Agora?
Agora retail
Describe the issue or the bug
Video feed not showing
To Reproduce

Have you looked at the Troubleshooting and Logs section?

Screenshots

Additional context

Pin openVino version to 2023.3.0 in requirements.txt


Elasticsearch support

Implemented AWS::Elasticsearch resource creation.
Fixes #411

Hey @edubxb, thanks for this.  Can you provide an example in examples and the output in tests/examples_output?  That way this will be tested, and it'll give folks an idea of how to use it.
A good example would probably be this: http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-elasticsearch-domain.html#d0e51584
Thanks!

Yeah, working on it!
Also, I'm improving the code a bit, with more input values validations.
Sorry for the late response.

I'd recommend allowing for a bit of flexibility with the validations. Currently the EBSOptions IOPS validation prevents using Cloudformation Condition functions to configure the IOPS volume type:
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/intrinsic-function-reference-conditions.html
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/pseudo-parameter-reference.html
e.g. The validation currently prevents something along the lines of:
par_iops = Parameter(
    'IOPS',
    Type='Number',
    Description='Dedicated IOPS for the volumes (zero for disabled)',
)

...
con_iops = ['con_iops',
            Not(Equals(`
            Ref(par_iops), "0"))]

...
EBSOptions(
    "EBSOptions",
    EBSEnabled=True,
    Iops=If('con_iops', Ref(par_iops), Ref("AWS::NoValue")),
    VolumeSize=Ref(par_volume_size),
    VolumeType=If('con_iops', 'io1', 'gp2')
)

@ink-adavison can you provide the exception output?  If should be a AWSHelperFn, which should be allowed (and un-validated) by troposphere for any field because of this exact issue.

Sure, here's an example to try:
https://gist.github.com/ink-adavison/20562a3fb898babb93cc
I got the same error when using simple Ref functions instead of Ifs too.
The exception is:
Traceback (most recent call last):
  File "examples/Elasticsearch.py", line 69, in <module>
    main()
  File "examples/Elasticsearch.py", line 66, in main
    print(template.to_json())
  File "build/bdist.linux-x86_64/egg/troposphere/__init__.py", line 470, in to_json
  File "/usr/lib/python2.7/json/__init__.py", line 250, in dumps
    sort_keys=sort_keys, **kw).encode(obj)
  File "/usr/lib/python2.7/json/encoder.py", line 209, in encode
    chunks = list(chunks)
  File "/usr/lib/python2.7/json/encoder.py", line 434, in _iterencode
    for chunk in _iterencode_dict(o, _current_indent_level):
  File "/usr/lib/python2.7/json/encoder.py", line 408, in _iterencode_dict
    for chunk in chunks:
  File "/usr/lib/python2.7/json/encoder.py", line 408, in _iterencode_dict
    for chunk in chunks:
  File "/usr/lib/python2.7/json/encoder.py", line 443, in _iterencode
    for chunk in _iterencode(o, _current_indent_level):
  File "/usr/lib/python2.7/json/encoder.py", line 434, in _iterencode
    for chunk in _iterencode_dict(o, _current_indent_level):
  File "/usr/lib/python2.7/json/encoder.py", line 408, in _iterencode_dict
    for chunk in chunks:
  File "/usr/lib/python2.7/json/encoder.py", line 408, in _iterencode_dict
    for chunk in chunks:
  File "/usr/lib/python2.7/json/encoder.py", line 442, in _iterencode
    o = _default(o)
  File "build/bdist.linux-x86_64/egg/troposphere/__init__.py", line 367, in default
  File "build/bdist.linux-x86_64/egg/troposphere/__init__.py", line 176, in JSONrepr
  File "build/bdist.linux-x86_64/egg/troposphere/elasticsearch.py", line 46, in validate
ValueError: Cannot specify Iops if VolumeType is not 'io1'.

Ahh, right - I forgot that this PR overrides validate() to provide it's own validation on top of the standard stuff.  Thanks @ink-adavison !

Fixed the issue when using Conditionals.

@phobologic is this good? I'm actually anxiously waiting for this change to be available. Thanks @edubxb !

I've done some checks, but any additional feedback will be wellcome!

This is great - thanks for all the hard work on this @edubxb - just the one comment, then I'll merge it.  Sorry for the delay - work got crazy!

I also removed the unnecessary Tag Class that I "borrowed" from the ec2 module.

Awesome!  Thanks @edubxb !


[IBM-MQ] Add tests for IBM MQ client

add ibm mq test resource + tests with ibm mq client
changed messaging tests to use "predictable" destination names (named after the test method), as IBM MQ requires destinations to exist before one can use them

Very nice, thanks a lot, @avano!
Besides the comments inline, I wonder why we need the separate camel-quarkus-integration-tests-support-ibm-mq module? It would make sense when it would be re-used at least twice, but there is no re-use here. Maybe there are other reasons for this structure?

Very nice, thanks a lot, @avano!
Besides the comments inline, I wonder why we need the separate camel-quarkus-integration-tests-support-ibm-mq module? It would make sense when it would be re-used at least twice, but there is no re-use here. Maybe there are other reasons for this structure?

you're right, currently a separate module is not needed. I though for future-proofing it would be easier to just add a dependency to a new module, rather than refactoring it, but maybe there isn't such case where another module would need it. So if you prefer I can squash it together

if you prefer I can squash it together

Yes, please :)

Thanks @avano and it is really great work! The failing test with netty seems unrelated. I will re-run the task to see if CI can pass.

Thanks a lot @avano!


Refactor methods that have too many parameters

Breaking change

Proposed change

Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[ ] New feature (which adds functionality to an existing integration)
[ ] Deprecation (breaking change to happen in the future)
[ ] Breaking change (fix/feature causing existing functionality to break)
[x] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[ ] The code change is tested and works locally.
[ ] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the development checklist
[ ] I have followed the perfect PR recommendations
[ ] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[ ] For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.
[ ] Untested files have been added to .coveragerc.

To help with the load of incoming pull requests:

[ ] I have reviewed two other open pull requests in this repository.

To solve the functions with many parameters, which currently block readability and maintainability, we use dataclasses to group the parameters. By grouping related parameters into data classes, we enhance function clarity, facilitate easier use, and simplify future modifications.

Changes look ok, but too many in one PR.


Allow a select component to be disabled

Do you think we should add a unit test for that or would you argue that it is to simple to break anyway?

Thought about that as well. Actually the new option is part of the contract, so it would make sense to test the behavior. Currently we don't test all the optional options, but I'll add one for this.

LGTM :+1:


ImportError: cannot import name 'LocalSource' from 'jschon'

@marksparkza Thank you for this wonderful package.
I am using jschon-0.7.3
I am trying to use this example as a template - https://github.com/marksparkza/jschon/blob/main/examples/load_from_files_2.py but I am seeing import errors.
I can see the LocalSource class in https://github.com/marksparkza/jschon/blob/main/jschon/catalog/init.py but the class doesn't exist in locally installed package on my machine.
I guess the pip package isn't updated to the latest code. Are there any plans to push the latest code to PyPI?

Hi @anantmittal,
Yes, a new release is in the works! Unfortunately I've had limited time available to work on jschon lately, so I can't say how soon that will be. In the meantime, there are a couple of options you can try:

take a look at the 'stable' documentation examples which work for v0.7.3: https://jschon.readthedocs.io/en/stable/examples/file_based_schemas.html
install the latest jschon code directly from GitHub with pip install -e git+https://github.com/marksparkza/jschon.git#egg=jschon

Thank you @marksparkza


Update SSRS Documentation Extensibility Page to encompass adding custom data sets

Microsoft's own documentation on the extensibility of SSRS reports lays out 4 scenarios where someone might want to extend an SSRS report.
https://docs.microsoft.com/en-us/dynamics365/fin-ops-core/dev-itpro/analytics/customize-app-suite-reports-with-extensions?toc=/dynamics365/commerce/toc.json
*Expand an existing data set attached to a current SSRS report
*Extend report menu items to redirect users to new reports
*Customize the design of an existing SSRS report
*Add new, custom data sets to an existing SSRS report
Microsoft documented the first three scenarios. The final scenario (bold above), which is arguably the most complex, because it potentially involves extending both the data provider class and the data contract classes, was left completely unexplained. This stub of text is the complete documentation of that scenario by Microsoft:
Composing custom datasets – Add more data to application reports by extending an existing RDP class to return a custom dataset.
The page in question can be found here
This is totally unacceptable from a documentation perspective. This area of D365 is, in my opinion, already overly complex with regard to pulling in additional data, designs and menus. It does not help when Microsoft does not document their own stuff. Someone on the D365 team, please get someone on this.
Thank you,
Jonathan

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

ID: 96d7d2cb-d661-2153-51a2-70193d80e7aa
Version Independent ID: f48e2cf4-e3fd-4a6e-7080-2e468a8d68bd
Content: Customize App Suite reports by using extensions - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/analytics/customize-app-suite-reports-with-extensions.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @TJVass
Microsoft Alias: tjvass

@TJVass - Can you take a look? Thanks.

Ok I gotta ask. @TJVass - are you related: https://www.seattletimes.com/seattle-news/tj-vassar-pioneering-african-american-educator/?
From: Robin Reynolds-Haertle @.>
Sent: Thursday, March 25, 2021 2:49 PM
To: MicrosoftDocs/dynamics-365-unified-operations-public @.>
Cc: Jonathan Rosen @.>; Author @.>
Subject: Re: [MicrosoftDocs/dynamics-365-unified-operations-public] Update SSRS Documentation Extensibility Page to encompass adding custom data sets (#3242)
@TJVasshttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FTJVass&data=04|01|jonathanr%40alascop.com|a81cf5e830bd4007aba308d8efd7da78|e6c09d0aaa884974a7edaaa28f327bae|1|0|637523057675489133|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=OCALB0vYkcE42G96KUwbx4WU6VJB3ghLebn2%2FmsAhtk%3D&reserved=0 - Can you take a look? Thanks.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-unified-operations-public%2Fissues%2F3242%23issuecomment-807558451&data=04|01|jonathanr%40alascop.com|a81cf5e830bd4007aba308d8efd7da78|e6c09d0aaa884974a7edaaa28f327bae|1|0|637523057675499125|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=YHLV613nAtoj64DpNrhJok19NgCO27B6eYuMae%2F7MZ4%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJHSDJIV2UFX7V7VOMD6BZTTFOVWHANCNFSM4ZYDHCTA&data=04|01|jonathanr%40alascop.com|a81cf5e830bd4007aba308d8efd7da78|e6c09d0aaa884974a7edaaa28f327bae|1|0|637523057675499125|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Kt9mz3jLL9B7NybCy%2F1rRqbDUBGpu8q3gfAqDMgAsLU%3D&reserved=0.

@RichdiMSFT - Can you take a look? Thanks.

@alaskancode Thanks for pointing out the issue. We will review the article and see if we can add the requested topic. This may take some time.

Created bug 572044 to track this issue.


only hide font panel if it was for this picker

Previously, if you had two FontPickers, and clicked one, then the other, the font panel would disappear again, rather than show the panel tied to the new button.

Thanks for PR!


Floating reward and contributions nodes

Hi!
How to be a non-constant reward and deductions master nodes? For example Darkcoin.
On good, the value of awards need not specify the configuration file, and to receive from wallet.
Masternode uptrend payments https://darkcointalk.org/threads/schedule-for-masternode-uptrend-payments.3700/

I am not I understand correctly but if you are asking what about payouts of blocks changing their value - either one time each X blocks or constantly, MPOS has an option to set payouts to the actual block value or fixed. Default should be dynamic payouts based on block values.
Other than that, try asking around other MPOS pool owners running Dark Coin. This repository is not actively maintained anymore and any changes that may be required will have to be done via Pull Requests by the community.


Jonmv/require parent geq compile version

@gjoranv please review.
Only throw if the parent project is a hosted Vespa parent.
Use parentArtifact, which is defined even when parent project cannot be generated.
Fallback to the version of the plugin itself for what the "vespaversion" is, for those without a vespa parent.

The failing unit test is probably among the usual suspects
MessageBusDocumentApiTestCase>AbstractDocumentApiTestCase.requireThatAsyncSessionWorks:123 » NullPointer

Definitely is.

👍


开发进度 - NuyoaH-L

https://blog.nuyoah-l.tk/progress/
Write an awesome description for your new site here. You can edit this line in _config.yml. It will appear in your document head meta (for Google search results) and in your feed.xml site description.

Hello world

#include <stdio.h>
int main()
{
    printf("Hello world!");
    return 0;
}


MM-19600 Align Sidebar separator line between Teams and Channel view

Ticket Link
https://mattermost.atlassian.net/browse/MM-19600
Screenshots

Thank you @enahum
Can you please help by building an app for testing?

@jgilliam17 is not really enough with the screenshot ? the change in the app was only the height of the element

@jgilliam17 is not really enough with the screenshot ? the change in the app was only the height of the element

Yes, that is a good point, the screenshot should be good enough for this change. QA will verify again after merge.
Thanks again.


no backend :)

hello!
i am Nick, full stack dev (mostly backend)
i am really interested in developing this project.
it would be nice to have ability to save and share diagrams or draw with friends.
i can integrate backend if you would like (I use Django btw)
contact me if you agree<EMAIL_ADDRESS>

Sorry for the late reply.
The original plan of this project is for a low-cost local base db scheme design tools, which will not run the same as other online existing cloud base tools. Hence, there will be no plan for implementing any server for data storage/keeping or other OATH features.
Yet, the current web supports an export and import scheme that may help you to share diagrams with others (In top nav -> File -> Export JSON / Import JSON).
Once again, thank you for the ideas.


Accessibility and Mutable Highlights

Note:  This is a tracking issue for accessibility, but there is no action to take at this time.
I spoke with @dpoeschl about accessibility requirements for default colorization in the Blue Theme for VS.  This is the theme which has requirements in this area.  Colorization for mutables does not fall within the required contrast ratio.  Same with other colors we use.  That said, Roslyn also uses default colors which fall short of these requirements.  We'll be hearing back about this in the future, at which point I'll take one of the following actions:

Close this issue and forget about it.  That means it won't be a problem.
Add this issue to the VS Updates milestone.  We'll have to pick a default color (or colors) which meet accessibility requirements for the contrast ratio in the Blue Theme for VS.

I don't even understand what it's all about.

@vasily-kirichenko Compliance.  A 4.5:1 (or higher) contrast ratio is the bar for accessibility requirements on colors for VS Blue Theme.  Mutables in Blue Theme are at 3:1.  We're below that for other colors as well.  So is Roslyn, though, and because we share some colors there may not end up being any action we have to table.

We are still in the process of deciding whether classification colors in particular need to satisfy this 4.5:1 requirement.

Currently we don't have separate theme for Blue vs Light. Both use the same set of colors. Personally I'm all for making mutables red again :)

@majocha red makes errors hard to see, same for dark orange.

Looks like we'll have to update the color here.  What we choose will also have to work under light/dark themes and Windows High Contrast White/Black modes.  I'll get a designer to give us a set of colors to choose from.


Plan for web compatibility ?

Hi there
Thank's for this package, great job !
Do you have plan to adapt this to Flutter web ?

Hi @poka-IT

Thank's for this package, great job !
Thanks!

No, I don't have any plan to adapt it for web, in fact I won't develop new features in the short term. But I will accept, review and test new contributions if you're interested in implement new features.


Failed to initialize NVML

Ubuntu 24.10
$ ./target/release/nvidia-tuner --index 0 --pairs 30:100
thread 'main' panicked at src/main.rs:23:86:
called Result::unwrap() on an Err value: "Failed to initialize NVML: a libloading error occurred: libnvidia-ml.so: cannot open shared object file: No such file or directory"
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
This solved the issue for me:
sudo ln -s /usr/lib/x86_64-linux-gnu/libnvidia-ml.so.1 /usr/lib/x86_64-linux-gnu/libnvidia-ml.so

Did you try to reinstall your driver? Although it seems to work now, it sounds more like a workaround.

This is a fresh install of Ubuntu Server 24.10 with apt instal nvidia-utils-560 nvidia-driver-560.

Well, I am using Endeavour OS and those links exist by default. I did some google search and found people having issues with this file missing when using Nvidias own nvidia-smi, so this must be something unrelated to my tool. Also, since you made it work, I close this topic.


[FR] FR-24 The preview box feature displays the query configuration in real time

Search before asking

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

DeepFlow Component
Grafana Plugin
What you expected to happen
The preview box feature displays the query configuration in real time
How to reproduce
The preview box feature will only work properly if the query is correct

When create a deepflow panel，if you configured a wrong query , sql preview box will not display
When modify a deepflow panel，if you configured a wrong query , sql preview  box  will not change

DeepFlow version
Name: deepflow-server community edition
Branch: v6.1.7
CommitID: ec882eea9310bd13495e086694a42f5caa04d7bd
RevCount: 7195
Compiler: go version go1.18.8 linux/amd64
CompileTime: 2022-11-29 02:48:14

Defaulted container "deepflow-agent" out of: deepflow-agent, configure-sysctl (init)
7195-ec882eea9310bd13495e086694a42f5caa04d7bd
Name: deepflow-agent community edition
Branch: v6.1.7
CommitId: ec882eea9310bd13495e086694a42f5caa04d7bd
RevCount: 7195
Compiler: rustc 1.65.0 (897e37553 2022-11-02)
CompileTime: 2022-11-29 02:48:30

DeepFlow agent list
No response
Kubernetes CNI
No response
Operation-System/Kernel version
No response
Anything else
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

feature merged.


Error locking PEP508 VCS dependency

Issue description
I have a setuptools-based module with a direct URL requirement to a Git repository; installation does work, but not locking.
Expected result
Locking the package works as intended.
Actual result
When possible, provide the verbose output (--verbose), especially for locking and dependencies resolving issues.
Steps to replicate
Sources
Pipfile
[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

[packages]
broken-stuff = {editable = true,path = "."}

[dev-packages]

[requires]
python_version = "3.6"

setup.py
from setuptools import setup, find_packages

setup(
    name='broken_stuff',
    version='0.0.1',
    description='A package that can\'t be installed.',
    long_description='A package that can\'t be installed.',
    long_description_content_type='text/markdown',
    author='Giovanni Santini',
<EMAIL_ADDRESS>    classifiers=[
        'Development Status :: 3 - Alpha',
        'Intended Audience :: Developers',
        'Programming Language :: Python :: 3.6',
    ],
    keywords='broken package',
    packages=find_packages(),
    python_requires='>=3, <3.7',
    install_requires=[
        'keras',
        'keras_contrib @ git+https://github.com/keras-team/keras-contrib.git@master',
		'tensorflow',
    ],
)

broken_stuff/init.py
"""
I am just a broken package.

Let me be.
"""

if __name__ == '__main__':
    print('Leave me alone!!!')

Commands
> pipenv install
Creating a virtualenv for this project…
Pipfile: E:\Progetti\Pull Requests\broken_pipenv\Pipfile
Using C:/Program Files/Python36/python.exe (3.6.8) to create virtualenv…
[    ] Creating virtual environment...Using base prefix 'C:\\Program Files\\Python36'
New python executable in C:\Users\Giovanni\.virtualenvs\broken_pipenv-OZETfJy1\Scripts\python.exe
Installing setuptools, pip, wheel...
done.
Running virtualenv with interpreter C:/Program Files/Python36/python.exe

Successfully created virtual environment!
Virtualenv location: C:\Users\Giovanni\.virtualenvs\broken_pipenv-OZETfJy1
Pipfile.lock not found, creating…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
Locking Failed!
[pipenv.exceptions.ResolutionFailure]:   File "c:/program files/python37/lib/site-packages/pipenv/resolver.py", line 69, in resolve
[pipenv.exceptions.ResolutionFailure]:       req_dir=requirements_dir
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 726, in resolve_deps
[pipenv.exceptions.ResolutionFailure]:       req_dir=req_dir,
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 480, in actually_resolve_deps
[pipenv.exceptions.ResolutionFailure]:       resolved_tree = resolver.resolve()
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 395, in resolve
[pipenv.exceptions.ResolutionFailure]:       raise ResolutionFailure(message=str(e))
[pipenv.exceptions.ResolutionFailure]:       pipenv.exceptions.ResolutionFailure: ERROR: ERROR: Could not find a version that matches keras_contrib@ git+https://github.com/keras-team/keras-contrib.git@master from git+https://github.com/keras-team/keras-contrib.git@master
[pipenv.exceptions.ResolutionFailure]:       No versions found
[pipenv.exceptions.ResolutionFailure]: Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.
  First try clearing your dependency cache with $ pipenv lock --clear, then try the original command again.
 Alternatively, you can use $ pipenv install --skip-lock to bypass this mechanism, then run $ pipenv graph to inspect the situation.
  Hint: try $ pipenv lock --pre if it is a pre-release dependency.
ERROR: ERROR: Could not find a version that matches keras_contrib@ git+https://github.com/keras-team/keras-contrib.git@master from git+https://github.com/keras-team/keras-contrib.git@master
No versions found
Was https://pypi.org/simple reachable?
[pipenv.exceptions.ResolutionFailure]:       req_dir=requirements_dir
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 726, in resolve_deps
[pipenv.exceptions.ResolutionFailure]:       req_dir=req_dir,
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 480, in actually_resolve_deps
[pipenv.exceptions.ResolutionFailure]:       resolved_tree = resolver.resolve()
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 395, in resolve
[pipenv.exceptions.ResolutionFailure]:       raise ResolutionFailure(message=str(e))
[pipenv.exceptions.ResolutionFailure]:       pipenv.exceptions.ResolutionFailure: ERROR: ERROR: Could not find a version that matches keras_contrib@ git+https://github.com/keras-team/keras-contrib.git@master from git+https://github.com/keras-team/keras-contrib.git@master
[pipenv.exceptions.ResolutionFailure]:       No versions found
[pipenv.exceptions.ResolutionFailure]: Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.
  First try clearing your dependency cache with $ pipenv lock --clear, then try the original command again.
 Alternatively, you can use $ pipenv install --skip-lock to bypass this mechanism, then run $ pipenv graph to inspect the situation.
  Hint: try $ pipenv lock --pre if it is a pre-release dependency.
ERROR: ERROR: Could not find a version that matches keras_contrib@ git+https://github.com/keras-team/keras-contrib.git@master from git+https://github.com/keras-team/keras-contrib.git@master
No versions found
Was https://pypi.org/simple reachable?

> pipenv lock --clear
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
Locking Failed!
[pipenv.exceptions.ResolutionFailure]:   File "c:/program files/python37/lib/site-packages/pipenv/resolver.py", line 69, in resolve
[pipenv.exceptions.ResolutionFailure]:       req_dir=requirements_dir
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 726, in resolve_deps
[pipenv.exceptions.ResolutionFailure]:       req_dir=req_dir,
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 480, in actually_resolve_deps
[pipenv.exceptions.ResolutionFailure]:       resolved_tree = resolver.resolve()
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 395, in resolve
[pipenv.exceptions.ResolutionFailure]:       raise ResolutionFailure(message=str(e))
[pipenv.exceptions.ResolutionFailure]:       pipenv.exceptions.ResolutionFailure: ERROR: ERROR: Could not find a version that matches keras_contrib@ git+https://github.com/keras-team/keras-contrib.git@master from git+https://github.com/keras-team/keras-contrib.git@master
[pipenv.exceptions.ResolutionFailure]:       No versions found
[pipenv.exceptions.ResolutionFailure]: Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.
  First try clearing your dependency cache with $ pipenv lock --clear, then try the original command again.
 Alternatively, you can use $ pipenv install --skip-lock to bypass this mechanism, then run $ pipenv graph to inspect the situation.
  Hint: try $ pipenv lock --pre if it is a pre-release dependency.
ERROR: ERROR: Could not find a version that matches keras_contrib@ git+https://github.com/keras-team/keras-contrib.git@master from git+https://github.com/keras-team/keras-contrib.git@master
No versions found
Was https://pypi.org/simple reachable?
[pipenv.exceptions.ResolutionFailure]:       req_dir=requirements_dir
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 726, in resolve_deps
[pipenv.exceptions.ResolutionFailure]:       req_dir=req_dir,
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 480, in actually_resolve_deps
[pipenv.exceptions.ResolutionFailure]:       resolved_tree = resolver.resolve()
[pipenv.exceptions.ResolutionFailure]:   File "c:\program files\python37\lib\site-packages\pipenv\utils.py", line 395, in resolve
[pipenv.exceptions.ResolutionFailure]:       raise ResolutionFailure(message=str(e))
[pipenv.exceptions.ResolutionFailure]:       pipenv.exceptions.ResolutionFailure: ERROR: ERROR: Could not find a version that matches keras_contrib@ git+https://github.com/keras-team/keras-contrib.git@master from git+https://github.com/keras-team/keras-contrib.git@master
[pipenv.exceptions.ResolutionFailure]:       No versions found
[pipenv.exceptions.ResolutionFailure]: Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.
  First try clearing your dependency cache with $ pipenv lock --clear, then try the original command again.
 Alternatively, you can use $ pipenv install --skip-lock to bypass this mechanism, then run $ pipenv graph to inspect the situation.
  Hint: try $ pipenv lock --pre if it is a pre-release dependency.
ERROR: ERROR: Could not find a version that matches keras_contrib@ git+https://github.com/keras-team/keras-contrib.git@master from git+https://github.com/keras-team/keras-contrib.git@master
No versions found
Was https://pypi.org/simple reachable?

Workarounds
Skipping the lock file creation seems to do the trick.

$ pipenv --support
Pipenv version: '2018.11.26'
Pipenv location: 'c:\\program files\\python37\\lib\\site-packages\\pipenv'
Python location: 'c:\\program files\\python37\\python.exe'
Python installations found:

3.7.3: C:\Program Files\Python37\python.exe
3.6.8: C:\Program Files\Python36\python.exe
2.7: C:\Python27\python.exe

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.7.3',
 'os_name': 'nt',
 'platform_machine': 'AMD64',
 'platform_python_implementation': 'CPython',
 'platform_release': '10',
 'platform_system': 'Windows',
 'platform_version': '10.0.17763',
 'python_full_version': '3.7.3',
 'python_version': '3.7',
 'sys_platform': 'win32'}

System environment variables:

ALLUSERSPROFILE
ANDROID_SDK_HOME
APPDATA
ASL.LOG
COMMONPROGRAMFILES
COMMONPROGRAMFILES(X86)
COMMONPROGRAMW6432
COMPOSE_CONVERT_WINDOWS_PATHS
COMPUTERNAME
COMSPEC
D2_API_KEY
DOCKER_CERT_PATH
DOCKER_HOST
DOCKER_MACHINE_NAME
DOCKER_TLS_VERIFY
DOCKER_TOOLBOX_INSTALL_PATH
DRIVERDATA
ERLANG_HOME
FP_NO_HOST_CHECK
GDAL_DATA
GECODEDIR
GIT_SSH
GRADLE_OPTS
HADOOP_HOME
HOMEDRIVE
HOMEPATH
JAVA_HOME
JEKYLL_GITHUB_TOKEN
LOCALAPPDATA
LOGONSERVER
MSYS2_PATH_TYPE
NO_PROXY
NUMBER_OF_PROCESSORS
NVM_HOME
NVM_SYMLINK
ONEDRIVE
ONEDRIVECONSUMER
OS
PATH
PATHEXT
POSTGIS_ENABLE_OUTDB_RASTERS
POSTGIS_GDAL_ENABLED_DRIVERS
PROCESSOR_ARCHITECTURE
PROCESSOR_IDENTIFIER
PROCESSOR_LEVEL
PROCESSOR_REVISION
PROGRAMDATA
PROGRAMFILES
PROGRAMFILES(X86)
PROGRAMW6432
PSMODULEPATH
PUBLIC
PYTHONPATH
SPARK_HOME
SSH_AUTH_SOCK
SVN_SSH
SYSTEMDRIVE
SYSTEMROOT
TEMP
TMP
UD_INSTALL_DIR
USERDOMAIN
USERDOMAIN_ROAMINGPROFILE
USERNAME
USERPROFILE
VBOX_MSI_INSTALL_PATH
VS140COMNTOOLS
WINDIR
PIP_DISABLE_PIP_VERSION_CHECK
PYTHONDONTWRITEBYTECODE
PIP_SHIMS_BASE_MODULE
PIP_PYTHON_PATH
PYTHONFINDER_IGNORE_UNSUPPORTED

Pipenv?specific environment variables:
Debug?specific environment variables:

PATH: c:\program files\python37\scripts\;c:\program files\python37\;e:\programmi\perl\site\bin;e:\programmi\perl\bin;c:\windows\system32;c:\windows\syswow64;c:\windows;c:\windows\system32\wbem;c:\windows\system32\windowspowershell\v1.0\;e:\programmi\miktex\miktex\bin\x64;e:\programmi\git\cmd;e:\programmi\nvm;e:\programmi\winscp\;e:\programmi\gecode\bin\;c:\android;c:\program files (x86)\gnupg\bin;e:\programmi\nodejs;c:\windows\system32\openssh\;c:\program files (x86)\windows kits\10\windows performance toolkit\;e:\programmi\microsoft vs code\bin;e:\programmi\cloud foundry;C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common;E:\Programmi\nvm;E:\Programmi\nodejs;E:\Programmi\Microsoft VS Code\bin;C:\Program Files\NVIDIA Corporation\NVIDIA NvDLISR;C:\Android;C:\Windows\System32;E:\Programmi\PuTTY\;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0\;C:\WINDOWS\System32\OpenSSH\;E:\Programmi\Git\cmd;E:\Programmi\Ruby-x64\bin;C:\Program Files\Python36\;E:\Programmi\Java\jdk\bin;C:\Program Files\Notepad++;E:\Programmi\youtube-dl;E:\Programmi\youtube-dl\ffmpeg\bin;E:\Programmi\Oracle\VirtualBox;C:\Users\Giovanni\AppData\Local\Microsoft\WindowsApps;E:\Programmi\Z3\bin;E:\Programmi\PostgreSQL\9.6\bin;E:\WinPath;E:\Programmi\Oracle\Database\app\oracle\product\11.2.0\server\bin;E:\Programmi\Gradle\bin;E:\Programmi\MiKTeX\miktex\bin\x64;E:\Programmi\Docker;C:\Program Files (x86)\Microsoft Visual C++ Build Tools;E:\Programmi\CMake\bin;C:\Users\Giovanni\AppData\Local\Pandoc\;E:\Progetti\Apps\SDK_Win\tools;E:\Progetti\Apps\SDK_Win\platform-tools;E:\Programmi\Spark\bin;E:\Programmi\Erlang\bin;C:\Program Files (x86)\GnuPG\bin;E:\Programmi\NMap;E:\WinPath\synctex;E:\Programmi\MongoDB\Server\4.0\bin;E:\Programmi\Cloud Foundry;E:\Programmi\NSIS

Contents of Pipfile ('E:\Progetti\Pull Requests\broken_pipenv\Pipfile'):
[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

[packages]
broken-stuff = {editable = true,path = "."}

[dev-packages]

[requires]
python_version = "3.6"

This is fixed on the master branch, which will be released shortly.  Sorry for the issue, closing this out as it's already been addressed and is just awaiting release
thanks for reporting!


render empty while created canvas node not in visible area

Please make sure you are testing with the latest release of html2canvas.
Old versions are not supported and issues reported for them will be closed.
Please follow the general troubleshooting steps first:

[x] You are using the latest version
[x] You are testing using the non-minified version of html2canvas and checked any potential issues reported in the console

Bug reports:
Render empty while DOM node not in visible area. While render canvas list full page,  u scroll result page to the bottom and wait a minute, scroll page to the top, u can see page render emtpy canvas.
jsfiddle.
Specifications:

html2canvas version tested with:  1.0.0-rc.3
Browser & version: chrome
Operating system: windows

Seems to be an issue introduced in RC2. RC1 doesn't seem to be effected by this issue.


Add a configuration engine

Since the WebUI will be used by several rucio communities, its UI needs to be customizable based on environment variables and/or configuration files provided during its deployment.
At present, the  rucio.cfg and environment variables are used to configure the old UI.
In most Kubernetes-based deployments, the configuration is supplied as environment variables and then a rucio.cfg is generated at a later stage.
For the new UI, since it is independent of rucio server and rucio core, we would not require a section in rucio.cfg, and can just use environment variables directly to configure the app.
To begin with, the oidc config variables specified in rucio.cfg must be made available to the app via environment variables.

@maany As per my research, there is a dotenv npm package that we could leverage for this use case, and all the variables we need. Then we can add a helper utility function say env('key') which returns the value, accessible with process.env['key'].

Yup dotenv is well known and we should definitely leverage it. Please proceed with the helper utility around it :)


We want to install GUIX Studio locally

Hi!
We have used GUIX Studio on Windows10 at company.
We can't download from the Store app due to company IT policy.
Previously, Windows installer was prepared, could you prepare a package that can be installed locally?

@yuxin-azrtos any comments on this issue? thanks.

@CYG01571 please contact<EMAIL_ADDRESS>and the support team can help you.

Thank you, @jdeere5220 .
The latest version of GUIX Studio is V<IP_ADDRESS> at Microsoft Store.
Renesas provides GUIX Studio software at their site, the version is V<IP_ADDRESS>.
And they declare that they don't provide the latest version software any more, download at Store...
(https://www.renesas.com/jp/en/software-tool/guix-studio)
So I have questioned at this site...
Best Regards.

@CYG01571 thanks for the feedback. To clarify, the latest version we have posted to the Microsoft App Store is version <IP_ADDRESS>. This is a minor patch/bug fix to correct a reported error. We have provided the standalone installer link to Renesas for every version up to <IP_ADDRESS>.  It is my understanding that we want to encourage developers to use the App Store version for security reasons. We are able to "push" updates via the Store channel in the event that some security threat is discovered. We can't do that with the standalone installer. In any case, you can (for now) download the standalone installer using this link: https://expresslogic.sharefile.com/d-s9055403587414096bfe3e70b4ce815e8
Let me now if you have any issues.

Thank you for your effort, @jdeere5220 !
OK. I'll try it.
I try to send e-mail to microsoft.
Best Regard