[Issue] Fail to Update Module

Description of bug
Currently, me and my co-worker are working on migration from v9.1.1 to v9.7.2, and find that the Extension Installation is not working at all : the DLL file in *_install.zip is not updated during Replace Module
Steps to reproduce
List the precise steps to reproduce the bug:

login with admin account
Go to 'Settings/Extensions' (by side bar)
Click on 'Install Extension'
Select an existed extension
continue the 5 steps

Current behavior
Current behavior (v9.7.2)
missing the *.dll file backup process, then the server do not update the bin\*_module_name_*.dll
StartJob Starting Installation

Info Starting Installation - ShoppingCartConfirmation
. . .
Info Starting Installation - Assembly

Info Assembly already registered - bin\ShoppingCartConfirmation.dll

Info Component installed successfully - Assembly

Info Installation committed

Info Installation successful. - ShoppingCartConfirmation

Info Deleted temporary install folder

EndJob Installation successful.

Expected behavior
Provide a clear and concise description of the expected behavior.
in v9.1.1, it run with this logging, then the server update the bin\*_module_name_*.dll
StartJob Starting Installation
. . .
Info Starting Installation - Assembly

Info Assembly already registered - bin\ShoppingCartConfirmation.dll

Info Creating backup of previous version - bin\ShoppingCartConfirmation.dll

Info Created - bin\ShoppingCartConfirmation.dll

Info Component installed successfully - Assembly

Info Installation committed

Info Installation successful. - ShoppingCartConfirmation

Info Deleted temporary install folder

EndJob Installation successful.

Affected version

[ ] 10.00.00 alpha build
[ ] 09.08.00 release candidate
[X] 09.07.02 latest supported release

Did you unblock downloaded upgrade package (in file properties), before installing?

you mean unzipping to install the updated library ?

no, right click downloaded package in Windows Explorer and select "File Properties". At the bottom of first tab, unblock the file.

checked with co-worker, it was unblocked

I am thinking maybe folder permissions or anti-virus blocking ? I have installed a ton of extensions on 9.7.2 and never had this issue.
I would also maybe verify the module package on https://evs.dotnetnuke.com/ to check for manifest errors maybe...

I may find my co-worker and try the verification, and I close this issue first, but I will keep tracking and reporting for your reference
anyway, thanks for your answering ;)

Yep and participants on the issue keep getting notified about comments on it even if it's closed, so feel free to comment if you need any more advice...

How did you solve this problem ?


Rename camtrap-package-profile.json to camtrap-dp-profile.json [REPLACEMENT ISSUE]

The original issue
Id: 66
Title: Rename camtrap-package-profile.json to camtrap-dp-profile.json

could not be created.
This is a dummy issue, replacing the original one. It contains everything but the original issue description. In case the gitlab repository is still existing, visit the following link to show the original issue:
TODO

In GitLab by @kbubnicki on Dec 2, 2020, 12:19
mentioned in commit e9bcb1311a3a856496a4f5b16d15b31816ce4196

In GitLab by @kbubnicki on Dec 2, 2020, 12:22
mentioned in commit 53381b4620970d9a32d38f41168819a89a8e2fc0


Forge Version Bump

Bump Forge version used to build with to fix cryptic errors (Issue #148 + duplicate issues).

Confirmed to fix Issue #152 also.


Testing module fixes

Reorganize interfaces internally to make sure MockNgRedux and NgRedux are structurally compatible (issue #419)
Fix a boundary condition where two instances of MockNgRedux could be created in certain cases.

This reorg allows us have MockNgRedux and NgRedux conform to the same interface, without MockNgRedux having to stub out all the private fields from NgRedux in order to be assignable to it.

👍


All code examples should rethrow the exception it catches

Right now, we do a catch-and-print approach for errors, which causes the test cases to pass even in a broken example, because they expect that a GoogleAdsException will be thrown.

Fixed in https://github.com/googleads/google-ads-dotnet/pull/169

Fixed in v2.9.0.


Hide permissions table unless custom permissions are enabled

I just spent too much time figuring out why my custom permissions were not working. Turned out I didn't enable them.
This PR hides the permissions table unless you enable custom permissions. That should save some time and frustration for users who apparently cannot read (like me :smile:).

Neatly implemented, thanks @timoschwarzer.
Eventually I want to change the permissions system to be sensible so you selectively add roles to permit/deny but that is on the roadmap for the future, as this is a nice UX improvement until then.
Will merge for the next feature release.


Update littlefs module

Hi, this updates the littlefs module to https://github.com/zephyrproject-rtos/littlefs/pull/12, plain merge of v2.5.0, just needed a minor tweak on the LOG macros.

DNM while it has thepull/ ref.


feat: XML/JSON results more human-readable

expected result:

add an parameter to the serializes, that controls indention.
default to "OFF" -- no indention and such
if indention exists, then beautify JSON/XML
one property declaration per line
nested structures are indented

--> in addition, the test data - SNAPSHOTS - should

be marked as "binary" and compared byte-wise, not deep-content-wise.
these files are designed to be human-readable, not human-maintainable.
be generated with an indention of 2 spaces for better human readability and showcasing this feature.

questionable, if this shall be part of the library.
it is able to return the raw normalized data, not only the string/serialized representtion.
if one wanted a different/pretty serialized version, they could just fetch the normalized/non-string version, and serialize themselves. ...


Could not store secondary backup of session

From<EMAIL_ADDRESS>on March 02, 2013 18:31:41
Thank u for u interested with the issuse.
i use mms in this environment：msm-1.6.3,tc7-1.6.3,spymemcached2.7.3 and kyro 1.6.3
When i visit my web site ,i got this log output:
INFO: Could not store secondary backup of session 1B7D569C36DB1D7CE5161B9A49870BA6-n1
java.lang.IllegalArgumentException: No node found for key bak:1B7D569C36DB1D7CE5161B9A49870BA6-n1 (nodeId: null, known nodeIds: [n1, n2])
And here is the useful content in my context.xml:

how to solve? thanks!
Original issue: http://code.google.com/p/memcached-session-manager/issues/detail?id=157

From martin.grotzke on March 02, 2013 13:23:48
This happens because you're using failoverNodes with sticky="false", which does not make sense. FailoverNodes are only useful for sticky sessions. Once you remove the failoverNodes configuration the mentioned log messages should be gone.
Status: Invalid

From bighead99 on August 22, 2013 19:33:15
I'm  not set failoverNodes:

but ,it still happens,so any idea with this problem?
========================logs==============================
2013-08-24 02:12:54.163 WARN net.spy.memcached.MemcachedConnection:  handling node for operation is not set
Aug 24, 2013 2:12:54 AM de.javakaffee.web.msm.LockingStrategy$OnBackupWithoutLoadedSessionTask call
INFO: Could not store secondary backup of session 835ECC64588362E99D3E451C31108870-nd47.jvm4
java.lang.IllegalArgumentException: No node found for key bak:835ECC64588362E99D3E451C31108870-nd47.jvm4 (nodeId: null, known nodeIds: [nd47, nd48])
at de.javakaffee.web.msm.SuffixBasedNodeLocator.getPrimary(SuffixBasedNodeLocator.java:100)

From martin.grotzke on August 23, 2013 00:42:46
Does this happen always, or only sometimes? I looked at the code and verified some test cases, and I'd say this should only happen if nd48 is not available. If it happens in your case even if nd48 is available please provide a test case or a sample app that allows me to reproduce this issue.
Also it would help if you'd provide the full stack trace.
Status: New

From bighead99 on August 23, 2013 01:20:11

My architecture is:
LVS(<IP_ADDRESS> it's ok,when I test from the VIP,its on machine <IP_ADDRESS>)
/
http(<IP_ADDRESS>:80)  http(<IP_ADDRESS>:80)
/           \ /
Tomcat1(ajp://<IP_ADDRESS>:8109:jvm1) Tomcat2(ajp://<IP_ADDRESS>:8109:jvm4)
I use apache mod_proxy as a balancer for tomcat1(jvm1)and tomcat2(jvm4)
here is the configuration:
<Proxy balancer://is-cluster/>
BalancerMember ajp://<IP_ADDRESS>:8109/ loadfactor=1 route=jvm1
BalancerMember ajp://<IP_ADDRESS>:8109/ loadfactor=1 route=jvm4
ProxySet lbmethod=bytraffic

ProxyPass / balancer://is-cluster/ stickysession=JSESSIONID|jsessionid
I use ie explorer to access <IP_ADDRESS>:80 , then apache(I dont know on which one) refer to jvm4(29.48:8109), then I success to login,and then  I reboot the machine(29.48), I got the error,can you unstand what i say ,my english is soso. :)

From bighead99 on August 23, 2013 01:24:16

Now ,I found the error on both Tomcat:
=============================log=========================
WARNING: Found no validity info for session id AEBD1FE6A416C82B304DF13A50CF9349-nd47.jvm1
Aug 24, 2013 8:04:48 AM de.javakaffee.web.msm.LockingStrategy onBackupWithoutLoadedSession
WARNING: Found no validity info for session id AEBD1FE6A416C82B304DF13A50CF9349-nd47.jvm1
Aug 24, 2013 8:04:48 AM de.javakaffee.web.msm.LockingStrategy onBackupWithoutLoadedSession
WARNING: Found no validity info for session id AEBD1FE6A416C82B304DF13A50CF9349-nd47.jvm1
Aug 24, 2013 8:04:48 AM de.javakaffee.web.msm.LockingStrategy onBackupWithoutLoadedSession
WARNING: Found no validity info for session id AEBD1FE6A416C82B304DF13A50CF9349-nd47.jvm1
Aug 24, 2013 8:04:49 AM de.javakaffee.web.msm.LockingStrategy onBackupWithoutLoadedSession
WARNING: Found no validity info for session id AEBD1FE6A416C82B304DF13A50CF9349-nd47.jvm1

From martin.grotzke on August 23, 2013 01:46:52
Not sure what's happening, but an issue might be that non-sticky sessions (sticky=false) are used in combination with a jvmRoute / with stickyness in apache.
Can you run apache without stickyness (and remove jvmRoute from tomcat config) and see if this changes the situation?

From bighead99 on August 23, 2013 02:19:33

When I use sticky in memcached and set failoverNodes,I did not see the error again:
WARNING: Found no validity info for session id AEBD1FE6A416C82B304DF13A50CF9349-nd47.jvm1
but when I do the same things ,I got an apache 503 error.
When I run apache without stickyness (and remove jvmRoute from tomcat config),I login the app ,and click any link ,and I soon be refer to the login page ,and login again!

From martin.grotzke on August 31, 2013 15:19:41
Can you please activate debug logging according to https://code.google.com/p/memcached-session-manager/wiki/SetupAndConfiguration#Configure_logging and send the output? Also post the configuration used for the test.

From martin.grotzke on December 27, 2013 00:52:39
 Issue 191  has been merged into this issue.

closed due to missing feedback


fix: remove middleware to prevent redirects

:tada: This PR is included in version 1.148.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Implement instrument ganza

1 - Ganza must be an instrument.
2 - Has to sound like 'chuki chuki chuki chuki'
3 - Must be louder than a pandeiro
4 - The color must be yellow

Nice! Just one comment, which is more related to the "BDD" aspect of the exercise. The test named thereShouldBeAnIInstrumentInterfaceAndAGanzaClass does not really specify the behaviour of the application. It describes the internal code structure, which is not the concern of the client. I will merge the PR anyways. Thanks!

Ah... one thing that you forgot is to include the "Fixes #43" marker in your PR text. I will close the issue manually.


[Bug] RayJob envtest is flaky

Search before asking

[X] I searched the issues and found no similar issues.

KubeRay Component
ray-operator
What happened + What you expected to happen
https://github.com/ray-project/kuberay/actions/runs/6437188474/job/17481908089
Reproduction script
https://github.com/ray-project/kuberay/actions/runs/6437188474/job/17481908089
Anything else
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

@rueian are you interested in this issue?

Sure. This is the failed test.

Do not use *myRayCluster.Spec.WorkerGroupSpecs[0].Replicas++ at L313, but use *myRayCluster.Spec.WorkerGroupSpecs[0].Replicas = int(*myRayJob.Spec.RayClusterSpec.WorkerGroupSpecs[0].Replicas)+1, which guarantees it will always be 4.

Using an idempotent operation makes sense to me.


Avoid separate table for each base entity

Hi,
I'm trying to subclass IdentityDbContext to support multi-tenancy. I want to implement something similar to JSkimming's solution.
Anyway, generally if I have one Person and one Teacher and I have a DbSet<Person> in my context, two tables containing ALL the properties of both Person and Teacher are being generated in the database.
Is there a way to avoid that?
I have an entity Tenant, here is the code:
public class Tenant : Tenant<string>
{
}

public class Tenant<TKey>
  where TKey : IEquatable<TKey>
{
  public virtual TKey Id { get; set; }
  public virtual string TenantName { get; set; }

  public virtual ICollection<TenantUser<TKey>> Users { get; set; } =
    new HashSet<TenantUser<TKey>>();
}

public DbSet<TTenant> Tenants { get; set; }

protected override void OnModelCreating(ModelBuilder builder)
{
  builder.Entity<TTenant>(b =>
  {
    b.Key(t => t.TenantId);
    b.Index(t => t.TenantName).Unique().IndexName("TenantNameIndex");
    b.ToTable("Tenants");

    b.Collection(t => t.Users).InverseReference().ForeignKey(u => u.TenantId);
  });
}

But when I try to add a migration, I get the following error:

Microsoft.Data.Entity.Metadata.ModelItemNotFoundException: The entity type 'Microsoft.AspNet.Identity.EntityFramework.Tenant.Tenant' requires a key to be defined.

If I add a Key attribute on Tenant<TKey>.Id the following is generated to the migration:
migration.CreateTable(
    name: "Tenants",
    columns: table => new
    {
        TenantId = table.Column(type: "nvarchar(450)", nullable: false),
        TenantName = table.Column(type: "nvarchar(16)", nullable: true)
    },
    constraints: table =>
    {
        table.PrimaryKey("PK_Tenant", x => x.TenantId);
    });
migration.CreateTable(
    name: "Tenant<string>",
    columns: table => new
    {
        TenantId = table.Column(type: "nvarchar(450)", nullable: false),
        TenantName = table.Column(type: "nvarchar(16)", nullable: true)
    },
    constraints: table =>
    {
        table.PrimaryKey("PK_Tenant<string>", x => x.TenantId);
    });

Inheritance support is still being worked on and depending on which build you are using you might get different levels of functionality. In fact we have only recently added convention and configuration support for discriminator columns in Code First.
@AndriySvyryd can comment on what the minimal level of configuration you will have to do to get inheritance working is currently.
Regardless, if as you said in the comment on #247 you don't want the base class (in this case Person) to be mapped in your model, then do not add a DbSet<Person> to your DbContext. As a general rule, EF will not walk up the inheritance hierarchy to find a base type and add it to the model unless you mention it explicitly, e.g. either in a DbSet, with the modelBuilder.Entity<T>() method or in a navigation property. Another thing you may want to try is to make the base type abstract.

@divega
Hi Diego and thanks for your reply.
Unfortunately the DbSet<Person> is already declared in the base DbContext.
Besides, I don't want it NOT to be declared, I just want TPerson of DbContext<TPerson> where TPerson : Person to be initialized as table, all the base class' properties should be part of this one.

I'm sharing here my experiences:
https://gist.github.com/weitzhandler/f9ebca6936ed0489d42d
I can't finish off the multi-tenancy implementation, before the DB is up and running.

I've posted a new repository that tries to implement support for multi-tenancy leveraging the existing Identity API, would thank anyone who can direct me in making it work.
https://github.com/weitzhandler/Tenants

I've posted a new repository that tries to implement support for multi-tenancy leveraging the existing Identity API, would thank anyone who can direct me in making it work by addressing the above issue.
How can I get rid of EF generating two SQL tables for TenantUser and TenantUser<string> both consisting of ALL properties.
This is a blocking issue.
https://github.com/weitzhandler/Tenants

@weitzhandler
You can map derived entity types to the same table as the parent by calling
modelBuilder.Entity<TenantUser>.BaseType<TenantUser<string>>();

However inheritance support is not fully implemented yet, so you might run into errors at runtime.

@AndriySvyryd
Thanks for your help.
With this I managed to go a bit further. Please see my updated context.
However I have a different issue. I have a one-to-many reference between TUser.Tenant and TTenant.Users, as you can see, the the FK is generated twice, thus throwing the following error when trying to generate database:

Cannot create, drop, enable, or disable more than one constraint, column, index, or trigger named 'FK_TenantUser_Tenant_TenantId' in this context. Duplicate names are not allowed.

@AndriySvyryd
Thanks for your help.
With this I managed to go a bit further. Please see my updated context.
However I have a different issue. I have a one-to-many reference between TUser.Tenant and TTenant.Users.
As you can see, the the FK is generated twice, thus throwing the following error when trying to generate database:

Cannot create, drop, enable, or disable more than one constraint, column, index, or trigger named 'FK_Tenant_TenantUser' in this context. Duplicate names are not allowed.

I've posted a question about this on SO.
Well looks like this is under control:
https://github.com/aspnet/EntityFramework/pull/2910


Can't use LineDashedMaterial with Line

We can't build a line using LineDashedMaterial, see sample code
	Geometry lineGeometry = new Geometry();
	List<Vector3> vertArray = lineGeometry.getVertices();
	vertArray.add( new Vector3(-50, -100, 0) );
	vertArray.add(  new Vector3(-50, 100, 0) );
	lineGeometry.computeLineDistances();
	lineDashedMaterial = new LineDashedMaterial();
	lineDashedMaterial.setColor(new Color(0x0000cc));
	lineDashedMaterial.setDashSize(1);
	lineDashedMaterial.setGapSize(3);
	Line line = new Line( lineGeometry, lineDashedMaterial ); // no constructor using dashedMaterial
	getScene().add(line);

Thank you for your quick response. If you have more time please can you see my comment on issue #9 related to maven.


[🐞] A empty index.ts file will introduce bug

Which component is affected?
Qwik City (routing)
Describe the bug
Everything is right until you create a empty index.tx file in the src/routes dir.
if you can't reproduction, you can restart project by ctrl + c and npm start.
Reproduction
no
Steps to reproduce
create a empty index.tx file in the src/routes dir.
error info:
Code(25): Invalid JSXNode type. It must be either a function or a string. Found:
/Users/xxxxxx/WebstormProjects/qwik_demo/node_modules/@builder.io/qwik-city/index.qwik.mjs:16:12.
let cmp = null; 15 |      for (let i = contentsLen - 1; i >= 0; i--) 16 |        cmp = jsx(contents[i].default, {    |              ^ 17 |          children: cmp 18 |        });
System Info
> node 16

Additional Information
No response

Hi! It’s unlikely we will prioritise a issue if there is no a fast way for us to reproduce it, and the system info is not filled


Domains Required for Web

Purpose
Update docs to say that domains is required for web properties.
Changes
Added some additional descriptive text between the forms.

This addresses https://jira.corp.adobe.com/browse/PDCL-4866


[flutter roll] Revert "Add ability for ModalRoutes to ignore pointers during trans…

…itions and do so on Cupertino routes (#95757)"
This reverts commit 4c0b0be2da5d1ee80c3d713e68ddd88d2cf2e72d.

Please briefly add some context why this was reverted for people who want to look at this again in the future and cc the original author for their information that it got reverted.

@XilaiZhang what was the issue here? Anything I can help with?

yes sir i copied the error msg to https://github.com/flutter/flutter/pull/95757#issuecomment-1137492558. I am not sure if there is an open source version of the test you can reference


Update documentation part (flags for traversal and port forwarding)

@etherunit what is status of this pull request? Are you going to adapt suggestions?


🚀 feat: o1

Summary

Added o1 models to the default OpenAI models list, ensuring they are available for selection.
Implemented special handling for o1 models with custom instructions, adjusting how prompts are constructed.
Updated token cost calculations to account for the complexity of o1 models, including separate tracking for reasoning tokens.
Refactored the BaseClient to use new input/output token keys, improving flexibility across different model types.
Updated the MessageEndpointIcon component to differentiate the OpenAI icon color for o1 models, providing visual distinction.
Updated localization to include new error messages related to o1 models and system message support.
Adjusted the getResponseSender function to return 'o1' as the default sender string for o1 models.

Other Changes

Added a new error type for models that don't support system messages, improving error handling and user feedback.
Enhanced the AnthropicClient to use the new input/output token keys and added prompt caching support for claude-3-opus.

Testing:

Verify that o1 models appear in the model selection list.
Test conversations with o1 models, ensuring that custom instructions are handled correctly.
Check that token usage is accurately calculated and displayed for o1 models.
Confirm that the MessageEndpointIcon displays the correct color for o1 models.
Test error scenarios, particularly when attempting to use system messages with unsupported models.
Verify that the correct sender string is displayed for o1 model responses.

@danny-avila I have updated LibreChat (deploy-compose) but there are no o1 models in the model selection list....

@danny-avila I have updated LibreChat (deploy-compose) but there are no o1 models in the model selection list....

You need to be API tier 5:
https://platform.openai.com/settings/organization/limits
And if you do:
https://community.openai.com/t/were-tier-5-usage-but-dont-have-access-to-o1-models/937904/3
"""
OpenAI confirmed that we should have access this afternoon, Pacific time.
"""

Do these changes work with o1-preview on openrouter?

Do these changes work with o1-preview on openrouter?

OpenRouter o1 also works!


feat(mempool/lanes): Allow lanes with equal priorities

Closes #4014 .
The PR changes the proto files in InfoResponse to return a map of LaneID(string) : LanePrio(uint32) instead of an array of priorities and the default lane is identified by its ID (a string).
CheckTx returns a LaneId which is a string defined as :
Before marking it ready for review, we need to make sure that the corner case of default lane and priorities are properly addressed now with these changes.

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

Do we have any test scenario with multiple lanes having the same priority?


blackd appears to fail tests with a slightly longer body

I ran into this while trying to write a PR for #622. If I add this test to tests/test_black.py (I put it at line 1351, but it shouldn't matter), it fails:
    @unittest.skipUnless(has_blackd_deps, "blackd's dependencies are not installed")
    @async_test
    async def test_blackd_long_body(self) -> None:
        app = blackd.make_app()
        async with TestClient(TestServer(app)) as client:
            response = await client.post("/", data=b'print("hello a very long world")\n')
            self.assertEqual(response.status, 204)
            self.assertEqual(await response.read(), b"")

Scrolling way up to the error, here it is:
======================================================================
FAIL: test_self (tests.test_black.BlackTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/miniconda3/envs/black-dev/lib/python3.6/unittest/mock.py", line 1179, in patched
    return func(*args, **keywargs)
  File "/Users/chtu8001/Work/black/tests/test_black.py", line 181, in test_self
    self.assertFormatEqual(expected, actual)
  File "/Users/chtu8001/Work/black/tests/test_black.py", line 156, in assertFormatEqual
    self.assertEqual(expected, actual)
AssertionError: '#!/u[58639 chars]post("/", data=b\'print("hello a very long wor[6606 chars]")\n' != '#!/u[58639 chars]post(\n                "/", data=b\'print("hel[6638 chars]")\n'

This one, which is already in the test suite, passes:
    @unittest.skipUnless(has_blackd_deps, "blackd's dependencies are not installed")
    @async_test
    async def test_blackd_request_no_change(self) -> None:
        app = blackd.make_app()
        async with TestClient(TestServer(app)) as client:
            response = await client.post("/", data=b'print("hello world")\n')
            self.assertEqual(response.status, 204)
            self.assertEqual(await response.read(), b"")

Using data=b'print("hello long world123456")\n' passes, but data=b'print("hello long world1234567")\n' fails. The former is 31 characters plus the newline; the latter is 32.
I can't come up with any reason that this is happening. Any thoughts?
Operating system: MacOS 10.13.6, High Sierra
Python version: 3.6.7
Black (and blackd) version: 18.9b0
Does also happen on master: yes

I think it's because you made the file poorly formatted (the line is too long). Try running black on test_black.py first.

That was it, thank you!


Adding any certificate to the Java Keystore

Hi,
We need to add a certificate to the java keystore to allow Jira (and confluence) to connect to our internal user directory. This is already done for the letsencrypt certificates.
Would it be possible to add variables to allow to do that for certificates of our choice ?
Em

Best way to do this would be to extend the image. Write your extension Dockerfile and just add them simply to the store as I am doing with the letsencryp certificates.

You can see the instructions here:
https://github.com/blacklabelops/jira#extending-this-image

If you do not want to extend the image, you can inject cacerts into the container. What I have done is copy /opt/jira/jre/lib/security/cacerts to the host. Add the certificate to cacerts and mount it to the same path.
docker cp :/opt/jira/jre/lib/security/cacerts cacerts
keytool ...
docker run -v $(pwd)/cacerts:/opt/jira/jre/lib/security/cacerts ...
Remember to do the above again when java is upgraded together with the jira container.

Thanks for the tip. @alktlr

Remember to do the above again when java is upgraded together with the jira container.
I am trying to avoid to have to remember and looking for a programatic way to assure that it is done at each deployement, automatically.

Do you know of another way than extendi the container image, but that is programatic?

Something like this? https://github.com/blacklabelops/bitbucket#ssh-keys
Can be done, though takes a while.

You could script the extract of cacerts, keytool import and docker run. So these commands will be run on every startup.

I think there are two issues involved.

How do you get your certificates into your environment
How do you get Jira to add your certificates.

There are two ways to get your certificates inside the environment.

Copy them into your environment (This includes to curl them from an url or retrieve them from some kind of storage).
Put them inside an image.

In my case, I usually tend to put certificates inside volume-images and build them each time I deploy containers.
Example with docker-compose:
services:
  # Key container
  server_keys:
    build: ./keys
    container_name: keys
    volumes:
      - keys:/server_keys
    command: bash

I know one of my colleagues worked on this... I will ask him what he went for in the end and let you know.


Train Barrier Feature (Finished)

This adds the option of train Barriers for the R179, Changes the spacing from int to float and Widget Checkbox on the main siding screen to enable or disable train Barriers.
The HAS_TRAIN_BARRIER Custom Resource command is not usable

@ExtremeMakerX can you please explain a little bit what your code does? I would like to make this code more generic so they can be applied to the other trains as well.

@ExtremeMakerX can you please explain a little bit what your code does? I would like to make this code more generic so they can be applied to the other trains as well.

Ok hold on. I will tell you soon

@ExtremeMakerX can you please explain a little bit what your code does? I would like to make this code more generic so they can be applied to the other trains as well.

Okay feel free to let me know if I didn't explain enough.
So what my code is the R179_TB in TrainClientRegistry is invisible in the train selection is because in KEY_ORDERS I added in the !trainBarrierOption which basically allows any train with the trainBarrierOption enabled will not show in the train selection. I added trainBarrierlength which of course allows float values on train spacing. In the siding menu when The TrainBarrierWidget is checked. The siding gets updated only if there is a R179 on the siding track then it replace the R179 with the R179_TB and then when unchecked goes back to the R179. Train barrier that is on 2 sides is rendered like a gangway connection but without players going to other cars. Reason the sides, up and down are transparent is because there is transparent connection textures. Also reason the train barrier is in the siding script is so the TB widget if checked can be saved when you open up the siding again. Hopefully I helped you understand what's going on.


tour: Default Selection & time channels

Context: https://tour.golang.org/concurrency/6
First of all thanks for the tour! It is a really perfect introduction to Go! As a new comer, I will suggest that the page "Default Selection" indicates that time.Tick() and time.After() return Go channels. I could provide a PR if you think that could improve the tour.

Thanks for reporting this; in my opinion that is clear enough just by looking at the code. After being defined, they're immediately used with <- which at this point the reader knows is a channel operation. I'd would avoid cluttering the text with too much information. Also it can be productive to leave something out from the text so that the reader can make an effort to read and learn something from the code snippets, too.
Closing here.


Checklist for 2020.08

Changes:

[x] Methods name and usage-name of Parameter now return an empty Str,
when there is no name, instead of Nil. Method default returns Code type object,
when no default was provided instead of Any type object -c11f4b18
[x] Remove redundant (()) from Map.new.raku -491e1def
[x] Routine done-testing of Test module returns Bool now,
indicating if the tests passed or not -337848fe

Additions:

[ ] Add set_parameterizer, parameterize_type, type_parameterized,
type_parameters and type_parameter_at methods for Metamodel::Primitives -2ab94429

the .raku output format isn't fully doc'd, we don't need to put this update in.

None of the metamodel primitive items are spec'd in roast. Closing ticket.


Failure in shellcgo.go's getpwnam_r on illumos

When trying to build chezmoi on illumos, the following error is encountered when downloading go-shell
# github.com/twpayne/go-shell /root/go/pkg/mod/github.com/twpayne/go-shell@v0.3.0/shellcgo.go:30:21: too many arguments in call to _Cfunc_getpwnam_r         have (*_Ctype_char, *_Ctype_struct_passwd, *_Ctype_char, _Ctype_ulong, **_Ctype_struct_passwd)         want (*_Ctype_char, *_Ctype_struct_passwd, *_Ctype_char, _Ctype_int) /root/go/pkg/mod/github.com/twpayne/go-shell@v0.3.0/shellcgo.go:34:3: invalid case 0 in switch on rc (mismatched types int and *_Ctype_struct_passwd) /root/go/pkg/mod/github.com/twpayne/go-shell@v0.3.0/shellcgo.go:36:3: invalid case _Ciconst_ERANGE in switch on rc (mismatched types int and *_Ctype_struct_passwd)
(uname -a results: "SunOS bz1 5.11 illumos-65ecf39910 i86pc i386 i86pc")

Ah, so it is! I guess I'm literally just 12 hours late to the party!
I'll go ahead and close this issue, but could you please update chezmoi to require go-shell v0.3.1?
Also, huge thank-you to you for maintaining these projects! I feel like developers/maintainers don't get enough praise for their work sometimes 😃


Add a languages file

Add a languages file to allow every message to be edited.
A big project.

Added a basic language file


Delta Green V2.0

Added NPC/Monster character sheet
Added Creature attack for monsters

In your last commit, your code was using the default Windows new-line characters (\r\n) when we need it to use the default Unix character (\n). Our website runs on a Unix system, and requires the Unix line-endings rather than windows. To avoid this in the future, you can change the settings in your text editor to use Unix line-endings rather than Windows.
Please change the files to use Unix-style (\n) line endings so I can properly compare and merge your changes.
Thanks,
Jeff

Hi Jeff,
I'm going to close this pull request and submit another with the format changes. Thanks


Update to latest C# TextMate grammar

Provides fixes for https://github.com/dotnet/csharp-tmLanguage/issues/44, https://github.com/dotnet/csharp-tmLanguage/issues/45 and https://github.com/dotnet/csharp-tmLanguage/issues/50.

@DustinCampbell,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Thanks @DustinCampbell !


Prismarine Trader

Is your feature request related to a problem?
Is too hard to obtain a good amount of prismarine blocks at the moment
Describe the solution you'd like.
Create a trader that sells all prismarine related blocks
Describe alternatives you've considered.
Increase droprate/orespawn of prismarine materials
Agreements

[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] I have ensured the feature I'm requesting isn't already in the latest supported Paper build.

Other
No response

Added


bug_5526 PEP8 standard fixes

https://gist.github.com/filipgorczynski/3383a16928f31dfd2489

Looks good :+1:


Distribute as a Unity package to allow easier updating and dependency management

Maybe consider the package folder structure (https://docs.unity3d.com/Manual/cus-layout.html) to enable easier importing and updating of the package just adding the Github URL (https://docs.unity3d.com/Manual/upm-ui-giturl.html)

Great idea, thanks!


Add support for python 3.11 and remove for 3.7

Summary
Adds Python 3.11 support, and removes Python 3.7 support.
Details and comments
The removal of 3.7 corresponds with the removal of 3.7 support from JAX, so it's maybe only a matter of time before 3.7 use of Dynamics + JAX becomes a problem.

Some required 3.7 test jobs won't leave the above list of tests for some reason. Going to remake the PR.


Modifier -> SwiftUI -> Drawing and graphics: colorInvert()

Doc: https://developer.apple.com/documentation/swiftui/view/colorinvert()

[ ] Swift implementation
[ ] Elixir implementation

Contribution needed


Is Ungoogled as safe as Chromium?

While Chromium gets new updates every 15 days or so, ungoogled has not been updated for about 3 month, which is a very long time for a browser release... I am not so savvy when it comes to browsers and thus I do not know the extend of the security issues that the smaller Chromium updates (the ones which Ungoogled does not offer) fix. Does one have to worry about those security issues? To what extend is Ungoogled less safe than Chromium?

I too would like to know the security implications of this.

The modern web browser is much more sophisticated than web browsers were back in the day, so there are many options when it comes to attempting a hack; you have the networking stack, plugins, and JavaScript. However, most of these hacks are pretty sophisticated and/or limited, so it'll be hard to come across in daily browsing.
How safe you are also depends on how Chromium is built. In ungoogled-chromium's case, a lot of newer and/or potentially privacy-invading features are disabled, like NaCl and WebRTC, so there are fewer attack points. Chromium has these features enabled by default.
On a related note, there have been no new tags recently, but all platforms except Windows have support for version 55 in the master branch. I might tag a version before Windows support is added if it takes too long.
If you have more questions, let me know.


Publish the helm chart to GHCR

As a Flux user I want to install the Weave GitOps UI from GitHub Container Registry instead of using Weaveworks Helm Registry. As a cluster admin, I don't want to depend on the weave.works uptime as there is no SLA for that, instead I want to pull the charts from ghcr.io.
PS. Pushing helm charts to GHCR is an easy task, here is how podinfo does it with GH Actions:
      - name: Publish Helm chart to GHCR
        run: |
          helm package charts/podinfo
          helm push podinfo-${{ steps.prep.outputs.VERSION }}.tgz oci://ghcr.io/stefanprodan/charts

I suggest that Weaveworks created a ghcr.io/weaveworks/charts repo to host the weave-gitops chart.

Start with making sure that publishing the chart works - this is easiest done in ghcr.io/weaveworks/weave-gitops.
Before publishing instructions to end users, decide if we want to move it to weaveworks/charts.

After some trials and errors, I found that ghcr.io/weaveworks/charts was a pseudo name, not a real repo name.
The steps I did locally were:

cd charts/gitops-server
helm package .
helm push weave-gitops-2.0.10.tgz oci://ghcr.io/weaveworks/charts
Then I jumped to GitHub, link ghcr.io/weaveworks/charts/weave-gitops:2.0.10 to the weaveworks/weave-gitops repo.
Set the package to be public.

Remaining steps are:

Verification of the package content
Test the published OCI image on a real cluster


Access to the component instance in moduleForComponent should be possible

For some reason, in the component integration test, this.subject() returns an error and there is no other way provided to allow anyone to access the component instance.
This does not allow for many types of tests, like checking the value of a computed property after clicking on a button.

Component Unit tests are still possible if you want to check properties on your components. However, I bet you'll still end up with a better test if you use the integration style.
If the value of that component is completely internal then it should not be tested. If the value of that component has some side effect, then that side effect should be tested. In this case, I bet there is some side effect when you click that button and the CP is just an internal detail.

yep, I tend to agree with @mitchlloyd
closing this issue as things are working as intended


[Enhancement]

Some suggestion for this plugin :

Add a way to indicate the place of the cursor in edit mode and change the cursor place when we click on a place.
Add a way to adjust the timer between change edit/preview mode

Not sure what you mean with the first point, if you are talking about the cursor saving between edit and preview, it already does that. And for the second point, that is a planned feature that I will probably add soon (Though I am leaving for a few weeks soon so I'm not sure when I will get around to adding it)

For the first point : I remark that you can't see your cursor move, so when the switching happened you can't saw where you stop.

Regarding the first point: indeed, you can't see where the cursor is when you first open a note, you'd have to start typing in order to see. At first, I was clicking on the text in preview mode exactly where I wanted to type—which is what we always do in edit mode, but instead it was typing where my cursor was the last time I opened the note rather than where I want it now, if that makes sense. The current workaround would be to type something, delete the text, and then move the cursor in edit mode to where desired.
What was suggested by Mara is for a way to know where the cursor is currently located while in preview mode, and then have a way to change that location before typing by clicking on the desired location in preview mode, similar to what we already do in edit mode.

Alright I get what you mean, I am not sure how exactly I would do that but I will try and figure it out

Second this... Would be nice to be able to automatically switch to edit mode when clicking! Also, when clicking, shifting the cursor to wherever we have clicked. This would be a really nice touch because currently it has some friction in how we can switch to edit mode.

Sure I could make clicking toggle to edit mode, I should probably add a setting for that as not everyone would want that. First I need to actually figure out how I would turn normal click coordinates into line and char.

Simple Clicking can conflict when clicking a link instead something like alt + click would be better

I'll probably leave it as a single click but ill make links still work. I am leaving tomorrow for a few weeks so the update probably won't be out for a while (unless i finish it today)


provide UI/UX prototypes

Either gather them from different sources into one or make the them from scratch or existing resources

This needs to be submitted latest on Sunday, hopefully @brafii can meet up
And we should start looking for prototypes from Figma and anywhere else we can find go ones


Adding category to physical properties

I think it will be useful to start adding some more definitions to PhysicalProperty (until Area D fixes this, I will simply add them as quantities).
One of them which might be useful is categories. This should be a MEnum with shape to define properties belonging to a certain category (electronic, magnetic, optical, thermodynamic, etc.). This might help us later when normalizing and identifying one or another type of property for searches, and it is information which can be extracted from the ttl file from the taxonomy task force.
@JFRudzinski what do you think? @markus1978 I think this is relevant also for you, it just means that the PhysicalProperty(Section) definition class you are doing will now have another attr called categories.

@ndaelman is this relevant to your Area D discussions?


[FR] Cleanup Names

There is a decent number of different names for all the elements on the different pages.
By now, there is no visible strategy to name new elements such that they don't conflict with existing ones, but still have a meaningful name.
This probably goes in hand with some more descriptive page names.
After that, one could consider having ids in the style of "PAGENAME-ELEMENTNAME-ELEMENTTYPE".
E.g. "expressiblity-num-params-input".

This issue should also aim to find more expressive names for the pages

As discussed

remove "Noise" page
rename second page to "Training"
leave third page as is


I found a way that juice write pure Markdown text in blogger HTML mode

I found a way that just write pure Markdown text in blogger HTML mode
https://www.youtube.com/watch?v=UjhAkpdcQ4o
Demo page
https://use-markdown-in.blogspot.com/2024/10/markdown-demo.html

Yes, your point is correct.
But about the script hosting, I consider these,

Github is a big company and has M$ behind, so I think Github's server is strong.
Another way is to use Cloudflare's Pages or Worker. Also strong sever.
Everyone can host the script file by themselves.


[Icon Request] ANTLRv4

Extensions: g4
SVG Logo:
<?xml version="1.0" encoding="utf-8"?>
<svg width="800px" height="800px" viewBox="0 0 32 32" xmlns="http://www.w3.org/2000/svg">
    <path d="M14.177,7.278a2.08,2.08,0,0,1,3.041-.8A3.571,3.571,0,0,1,18.2,8.022c1.356,3.122,2.9,6.165,4.119,9.345.645,1.5,1.429,2.938,1.994,4.468a1.455,1.455,0,0,1-2.258,1.376c-2.8-1.572-5.628-3.094-8.385-4.731,2.009.008,4.018-.008,6.025.013a19.707,19.707,0,0,0-1.288-2.918c-.781-1.858-1.6-3.7-2.358-5.565a9.783,9.783,0,0,0-1.032,2.125c-1.3,3.182-2.87,6.241-4.136,9.435-.281.59-.424,1.344-1.035,1.69a1.447,1.447,0,0,1-2.094-.738c-.241-.61.151-1.2.382-1.743.779-1.725,1.645-3.413,2.283-5.2C11.65,12.8,12.916,10.041,14.177,7.278Z" style="fill:#fefefe"/>
    <path d="M13.817,2.2A13.923,13.923,0,0,1,29.526,12.549a13.733,13.733,0,0,1-2.082,11.519A14.074,14.074,0,0,1,7.738,27.293a13.852,13.852,0,0,1-5.615-9.483A14.152,14.152,0,0,1,3.451,9.85,13.961,13.961,0,0,1,13.817,2.2m.359,5.08c-1.261,2.762-2.526,5.525-3.762,8.3-.638,1.786-1.5,3.473-2.283,5.2-.231.542-.623,1.133-.382,1.743a1.447,1.447,0,0,0,2.094.738c.61-.347.753-1.1,1.035-1.69,1.266-3.194,2.833-6.253,4.136-9.435a9.783,9.783,0,0,1,1.032-2.125c.756,1.868,1.577,3.707,2.358,5.565a19.707,19.707,0,0,1,1.288,2.918c-2.007-.02-4.016-.005-6.025-.013,2.757,1.637,5.588,3.159,8.385,4.731a1.455,1.455,0,0,0,2.258-1.376c-.565-1.529-1.349-2.971-1.994-4.468-1.22-3.179-2.762-6.223-4.119-9.345a3.571,3.571,0,0,0-.982-1.544A2.08,2.08,0,0,0,14.177,7.278Z" style="fill:#e44a32"/>
</svg>

The color probably needs to be toned down to match the pastel palette.
Also: the red circle could be simpler, I guess (<circle .../>)

thanks for giving the logo, i'll make one that fits the (current) guidelines (that is 100x100 canvas and 5px strokes)

see #66


Update MoltenVK to <IP_ADDRESS>

Philip, why do we need both libvulkan.1.dylib and libMoltenVK.dylib? They don't depend on each other, I'm unclear which one does what.


[BUG] Radiance HDR files aren't loadable with an ioproxy

Describe the bug
Trying to load a .hdr file with an in-memory ioproxy seems to fail.
To Reproduce
Steps to reproduce the behavior:
  Filesystem::IOMemReader memreader((void *)mem, size);
  auto in = ImageInput::open("mem", nullptr, &memreader);
  if (!in) {
    fprintf(stderr, "BAD: %s\n", OIIO::geterror().c_str());
    return nullptr;
  }

Expected behavior
The code itself seems to work for .bmp, .png, .tga, .tif, and .jpg so generally things seem ok. Additionally, the .hdr file is valid:
.\iinfo.exe T:\test.hdr
T:\test.hdr : 1024 x 1024, 3 channel, float hdr

Evidence
It's like the ioproxy is skipped and a fallback using the filename occurs:
BAD: Image "mem" does not exist. Also, it is not the name of an image format that OpenImageIO recognizes.
Platform information:

OIIO branch/version: 2.4.3 beta
OS: Windows 10
C++ compiler: MSVC 2022

Some further information
This doesn't work:
  Filesystem::IOMemReader memreader((void *)mem, size);
  auto in = ImageInput::open("mem", nullptr, &memreader);
  if (!in) {
    // we get sadness... and the following debug message from OIIO
    // OIIO DEBUG: ImageInput::create: "mem" did not open using format "hdr" hdr.
  }

This works:
  Filesystem::IOMemReader memreader((void *)mem, size);
  ImageSpec config;
  ImageSpec tmp;
  auto in = ImageInput::open("hdr", nullptr, nullptr); // explicitly get a HdrInput
  in->set_ioproxy(&memreader); // explicitly set the ioproxy
  bool ok = in->open("", tmp, config);
  // ok is true and reading works correctly

This no longer seems to be the case. So I'll close my own issue here.


api.orders.getOrders() does not return any output

I have run the exact code in main.py, and above function only returns None.
The setup and authorization works fine, and following codes runs perfect:
accountNumbers = api.accounts.accountNumbers()
universe.credentials.encryptedId = accountNumbers[0].get('hashValue')
I got correct accountNumbers and encryptedId.
so, what could go wrong here?

Noted. Looking into it.

I have tried to use following code to change the string format into: yyyy-MM-dd'T'HH:mm:ss.SSSZ:
fromEnteredTime.strftime('%Y-%m-%d'T'%H:%M:%S.%f')[:-3] + 'Z'
But still got None as output.

actually, the code should be (escape for 'T'):
fromEnteredTime.strftime('%Y-%m-%d'T'%H:%M:%S.%f')[:-3] + 'Z'

just print out 'response' from: _ResponseHandler(response)
it turns out <Response [401]>: An error message indicating either authorization token is invalid or there are no accounts the caller is allowed to view or use for trading that are registered with the provided third party application
I assume authorization token and accountNumber is correct, as it is retrieved sucessfully with following code with out any problem:
accountNumbers = api.accounts.accountNumbers()
universe.credentials.encryptedId = accountNumbers[0].get('hashValue')

Finally, found the solution:

change the time string format to following: (this should apply to all the api functions that need this format - might need to revise _TimeConvert())
toEnteredTime.strftime('%Y-%m-%dT%H:%M:%S.%f')[:-3] + 'Z'   -- (assume toEnteredTime assigned by datetime.now() format)
pass the hashvalue of account number directly to getOrders():
current way of pass value of encryptedId seems not working, getOrders() does not pice up accountNumber through "accountNumber=universe.credentials.encryptedId", guess this is due to different scopes. following code seems fix this problem:

accountNumbers = api.accounts.accountNumbers()
acct_num = accountNumbers[0].get('hashValue')
orders = api.orders.getOrders(3000, datetime.now()-timedelta(days=7), datetime.now(), accountNumber=acct_num)

Finally, found the solution:

Change the time string format, using toEnteredTime as example: (this should apply to all the api functions that need this format - might need to revise _TimeConvert())

current_time = datetime.now()
toEnteredTime = current_time.strftime('%Y-%m-%dT%H:%M:%S.%f')[:-3] + 'Z'

pass the hashvalue of account number directly to getOrders():
current way of passing value of encryptedId seems not working, getOrders() does not pick up the hash value of accountNumber through "accountNumber=universe.credentials.encryptedId", guess this is due to different scopes. following code seems fixed this problem:
accountNumbers = api.accounts.accountNumbers()
acct_num = accountNumbers[0].get('hashValue')
orders = api.orders.getOrders(3000, datetime.now()-timedelta(days=7), datetime.now(), accountNumber=acct_num)

Fixed in latest update with a variation of your proposed fix.


Help Wanted: Add setup guides for specific IDEs

There should be documentation on how to set up specific IDEs with:

Linters
Autocomplete
Executing tests within the IDE
...

Possible IDEs:

VS Code
vim
...

@hmhealey - Does it make sense to tackle this ticket as part of monthly developer documentation maintenance updates? I'm also curious to hear your thoughts on the impact/value of this ticket for staff and contributors. Is this something that belongs in the Mattermost Developer Documentation? Or potentially elsewhere?


Development

Omega Trade sample application
Added Readme with environment setup informations for frontend , backend and cloud spanner Emulator.

@googlebot I signed it!


Calls to spanner are hung from java docker

We are having issues in Spring Boot microservice, when app tries to get data from spanner, there is no error in the logs but it just hangs, all other service call which is not connecting to Spanner works fine
This issue is similar to #969 & #1637
The app works fine for a week and then it stops responding for DB calls, we are noticing this in all of our apps which has Spanner inetractions
We are using Hoxton.M2 and Greenwich.SR2 in 2 services respectively and behavior is reproducible in both

I wonder if this could potentially be related to this issue in the client library: https://github.com/googleapis/google-cloud-java/issues/6164.
Have you tried increasing spring.cloud.gcp.spanner.maxSessions to see if the app can go longer without hanging? Or, alternatively decreasing the number of sessions to see if the app fails sooner?
Another thing you can try in debugging this, is to set the RPC timeout as described here. You would need to provide your own Spanner bean to overwrite the auto-configured one.
Once we have a better understanding of the root cause, we can work on a permanent fix in the library.
Thanks!

cc/ @olavloite in case you have some thoughts on this.

This sounds very much like a session leak that builds up over time and in the end will exhaust the session pool. The SessionPoolOptions contains the option setFailIfPoolExhausted. If you set this for the session pool, a session leak will be easier to detect as you will get an RESOURCE_EXHAUSTED error instead of a process that waits indefinitely. Closing the session pool after such an error will also log a stack trace of all the sessions that are still checked out of the pool, giving you a hint about what method is leaking the session.
@meltsufin Is it possible to configure setFailIfPoolExhausted in the spring-cloud-gcp-spanner config options?

Thanks @olavloite!
Currently it's not exposed, but one can configure it by providing their own SessionPoolOptions bean in the Spring application context.
@srivraokp Can you try this please and report back?

@meltsufin
Please find below code I did:
@Bean
public SpannerTransactionManager transactionManager() {
SessionPoolOptions sessionPoolOptions = SessionPoolOptions.newBuilder()
.setFailIfPoolExhausted()
.setMinSessions(1)
.setMaxSessions(2)
.build();
SpannerOptions options = SpannerOptions.newBuilder()
.setSessionPoolOption(sessionPoolOptions)
.build();
Spanner spanner = options.getService();
DatabaseId db = DatabaseId.of(options.getProjectId(), spannerInstance, spannerdatabase);
DatabaseClient dbClient = spanner.getDatabaseClient(db);
return new SpannerTransactionManager(() -> dbClient);
}
I have uploaded the Exception,
Exception.txt

@srivraokp Could you share the code that is using the SpannerTransactionManager? Could it be that the transaction that your method is returning is never committed or rolled back in your code? That would (as far as I can tell) cause a session leak.

@olavloite
Please find below code, this is a readonlyTransaction call to get all entities based on a filter
I have attached Service, DAO and repository
Code.txt

@olavloite You're right about why we get the read/write transaction in doGetTransaction. Unfortunately Spring doesn't give the detail about if it's read-only until later on in doBegin.
I'm not sure if Spring calls our doCommit if the method was annotated as readonly, but I would be really surprised if they didn't (after all how else do we know when to clean up! )

@srivraokp Thanks, that's clearing up a couple of things.
@meltsufin I noticed in the SpannerTransactionManager#doGetTransaction() method that you are calling this.databaseClientProvider.get().transactionManager(); on line 82 when a transaction is needed, regardless of the type of transaction (read-only or read/write). In doBegin() the SpannerTransactionManager checks whether it is is a read-only transaction, and if so, returns a wrapper around the read/write transaction to prevent it from ever writing anything to the database. I assume that you do it like that because it is unknown to the doGetTransaction() what kind of transaction the user wants. The this.databaseClientProvider.get().transactionManager(); will always return a Spanner read/write transaction, and that is inherently a lot less efficient than a read-only transaction. But the most important question in this case is: Do you know if spring-data will explicitly call commit or rollback on a read-only transaction when the method with the @Transactional(readOnly=true) annotation has finished? Because if the Spanner TransactionManager is not explicitly committed, rolled back or closed, it will leak a session.
@srivraokp Could you try to execute the same example with a read/write transaction instead of a read-only transaction? (I.e. temporarily remove the readOnly=true from the @Transactional annotation)

@olavloite
just using 	@Transactional is working it is closing the session

@meltsufin
Can you please let us know in which release will this fix be available

@meltsufin hmmm this might mean we need to disable read-only access. There is no read-only transaction manager.

From what I can see, doTransaction() is only called from getTransaction(TransactionDefinition definition), where the read-only status is already known. It's just that it's not passed into the doTransaction() method. We can just abandon using AbstractPlatformTransactionManager and implement PlatformTransactionManager directly to get around the limitation.

@meltsufin There's still the challenge of not having a transaction-context that auto-closes without a call to doCommit. When we wrote this (and i think still the case), the only transaction manager the client lib has is read-write.

@ChengyuanZhao Actually, I don't think doGetTransaction() should even be creating new transactions. That's the job of doBegin().

@meltsufin I was under the impression that Spring calls doGetTransaction on our behalf to provide the TransactionObject to doBegin and that there is no way in doBegin to create a transaction and somehow feed it back to the other way for Spring to keep track of.

Looking a bit more into this, for read-only transactions, we create a new transaction that is disconnected from the current transaction context. So, in the doCommit() we forget to call close() on it.

@ChengyuanZhao @meltsufin @olavloite  we are currently using pubsub, spanner and spring cloud gcp spanner data libraries, our application is using the latest spring boot version 2.1.3.RELEASE and Greenwich.SR1 version for spring cloud release train. We also facing similar issues as above to be specific, spanner
com.google.cloud.spanner.SpannerException: CANCELLED: io.grpc.StatusRuntimeException: CANCELLED: Failed to read message.
at io.grpc.PartialForwardingClientCallListener.onClose (PartialForwardingClientCallListener.java:39)
at io.grpc.ForwardingClientCallListener.onClose (ForwardingClientCallListener.java:23)
at io.grpc.ForwardingClientCallListener$SimpleForwardingClientCallListener.onClose (ForwardingClientCallListener.java:40)
at com.google.cloud.spanner.spi.v1.SpannerErrorInterceptor$1$1.onClose (SpannerErrorInterceptor.java:100)
at io.grpc.PartialForwardingClientCallListener.onClose (PartialForwardingClientCallListener.java:39)
at io.grpc.ForwardingClientCallListener.onClose (ForwardingClientCallListener.java:23)
at io.grpc.ForwardingClientCallListener$SimpleForwardingClientCallListener.onClose (ForwardingClientCallListener.java:40)
at io.grpc.internal.CensusStatsModule$StatsClientInterceptor$1$1.onClose (CensusStatsModule.java:678)
at io.grpc.PartialForwardingClientCallListener.onClose (PartialForwardingClientCallListener.java:39)
at io.grpc.ForwardingClientCallListener.onClose (ForwardingClientCallListener.java:23)
at io.grpc.ForwardingClientCallListener$SimpleForwardingClientCallListener.onClose (ForwardingClientCallListener.java:40)
at io.grpc.internal.CensusTracingModule$TracingClientInterceptor$1$1.onClose (CensusTracingModule.java:397)
at io.grpc.internal.ClientCallImpl.closeObserver (ClientCallImpl.java:459)
at io.grpc.internal.ClientCallImpl.access$300 (ClientCallImpl.java:63)
at io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl.close (ClientCallImpl.java:546)
at io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl.access$600 (ClientCallImpl.java:467)
at io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl$1MessagesAvailable.runInContext (ClientCallImpl.java:531)
at io.grpc.internal.ContextRunnable.run (ContextRunnable.java:37)
at io.grpc.internal.SerializingExecutor.run (SerializingExecutor.java:123)
Show all
I could open another ticket, but while searching for solutions to my problem I ended up here

@naivegeek This is a different problem and not related to this issue.
Based on the above stack trace, it looks like the connection to Cloud Spanner has been closed (i.e. your instance of com.google.cloud.Spanner has been closed). Could it be that some other part of your application has already closed it?

@olavloite  sorry, we only started experiencing all this after we upgraded to spring boot version 2.1.3.RELEASE and Greenwich.SR1 version for spring cloud release train.
Thina where much more stable before.

@naivegeek Greenwich.SR3 is about to be released. Please try Greenwich.SR2 first, and if you're still having problems, please create a separate issue from this one. Thanks!

@meltsufin and @olavloite I will try upgrading to Greenwich.SR2 and see if we have any issues. Thanks for looking into it


add git versions of AUR packages

Looks good to me!
The caja package naming is different, but can change in the future - I decided to put the -py2 in there since it looks like caja extensions support will soon move to py3 on arch. But that's a problem for the future.

Yeah, I'll worry about a separate version for python2-caja later.


for postgresql identifier uescape

Fixes #ISSUSE_ID.
Changes proposed in this pull request:

Pull Request Test Coverage Report for Build 8061

0 of 0   changed or added relevant lines in 0 files are covered.
8 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.02%) to 60.952%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/sql/segment/common/TableSegment.java
8
0.0%

Totals

Change from base Build 8060:
0.02%

Covered Lines:
8693

Relevant Lines:
14262

💛  - Coveralls


node pagination not working

Hello brocaar,
I have an issue with the pagination functionnality of application nodes.
I am unable to paginate over the differents pages of my nodes.
I am using chrome and firefox.
When I click, a new _k parameter is generated, but no web call...
Thanks

Thanks for reporting! A regression occurred when introducing organizations. I will release a fix soon.


Don't auto-suggest grid area names that already exist

Created area0, area1, area2, and then deleted area1. The next area I created, SUE suggested area2, which was already taken.

I've bounced off this one for a while. Do you think it should suggest area1, area3, or something completely different in this situation? (I've never really liked the area[1-9]+ pattern.)


Change the data structure of the form argument of encode function

Codecov Report

Merging #4 (f15c488) into master (bd3cdb2) will increase coverage by 0.02%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   96.00%   96.02%   +0.02%
==========================================
  Files           1        1
  Lines         175      176       +1
==========================================
+ Hits          168      169       +1
  Misses          7        7

Flag
Coverage Δ

unittests
96.02% <90.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

lib/urlencoded.lua
96.02% <90.00%> (+0.02%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.