text:
img[usemap] is color:#00f in Firefox's default stylesheets

Would like to see something like img[usemap]{color:inherit;} in normalize.

My suggestions:

FF applies the same styling to object[usemap] so if this is also unique to FF it may make sense to style that as well. Unless we consider there is no real need to do that since this is now obsolete (object element usemap should be deprecated).
I would not qualify the selector—drop img to lower the specificity (go with [usemap] instead of img[usemap])
What about using the currentColor keyword instead of inherit?

Thanks @thierryk, I've updated my PR to include your suggestions.

I would ask that you add a test, preferably to test.html but optionally in its own CodePen. Once I’ve validated it against all the browsers, I’ll be happy to merge. Thanks!

Actually, on second thought, since anything with [usemap] is technically a link -- isn't inheriting the browser's default text link color actually the correct behavior?

If there’s a cross browser inconsistency, share a CodePen showing it off, and I’ll run the tests and verify if the suggested fix works.

@jsit, you're right, it is technically a link. And Firefox's default color for usemaps, #00f, is actually not far off from their default link color, #00e. But normalizing link colors is something that this project has intentionally avoided doing.

normalizing link colors is something that this project has intentionally avoided doing.

@mattbrundage Sounds right to me. Closing.

source: gharchive/issue

authors: jonathantneal jsit mattbrundage thierryk

license: mit

license_type: permissive

license_source: bigquery


text:
Allow passing --source-pkg to always be treated as a package

Currently coverage.py treats a package --source differently than a file path based --source. In our case, we need the behavior where it treats the --source as a package, however the os.path.exists('foo') returns True here, so coverage.py will treat it as a file system.
Instead of trying to get increasingly smarter, this simply adds a --source-pkg option which is always treated as a source package and is never looked at on the filesystem.

Current coverage is 90.67%

Merging #3 into master will decrease coverage by -0.02% as of 0d10f21

@@            master      #3   diff @@
======================================
  Files           83      83
  Stmts         9464    9468     +4
  Branches       999    1000     +1
  Methods          0       0
======================================
+ Hit           8583    8585     +2
- Partial        171     172     +1
- Missed         710     711     +1

Review entire Coverage Diff as of 0d10f21

Powered by Codecov. Updated on successful CI builds.

+1

source: gharchive/pull-request

authors: Julian codecov-io dstufft

repo: nedbat/coveragepy


text:
Redo the locate control

Use the method of creating a control as I was doing for the custom markers. Should allow moving all the stupid state preservation stuff into the control instead of being in AccessControlMap, and I'll be able to simply have a single "autostart" property.

Another huge amount of wasted time :(

Not going to touch it.

repo: neilenns/react-access-code-map

license: MIT

license_source: github-api


text:
sq inspect metadata doesn't populate the "schema" field for MySQL, SQLite, or SQL Server.

sq inspect returns schema for Postgres, but  is empty/omitted for other drivers.
$ sq inspect @sakila_pg12 -j
{
  "handle": "@sakila_pg12",
  "location": "postgres://sakila:xxxxx@192.168.50.132/sakila",
  "name": "sakila",
  "name_fq": "sakila.public",
  "schema": "public",
  "driver": "postgres",

Fixed.

repo: neilotoole/sq


text:
Mainnet - no amount input on transfer/invoice views

There is no amount for mainnet sending form:

I'm also unable to init transaction for mainnet with message:

Is this intentional?

@cryptoBeliever is it reproducible here? http://desktop-build.symboldev.com

@cryptoBeliever blocking transactions is intentional, but you should be able to see the amount field displayed as normal

authors: bassemmagdy cryptoBeliever

repo: nemgrouplimited/symbol-desktop-wallet


text:
Added f2 changes

Closes #45

What's the plan with the catbuffer reserved values? Should they be included in the open api and sdk transaction objects or the user pretty much don't care about them? It's just a private/internal feature/implementation of catbuffer?

In my opinion private / internal. It does not provide any value to the end developer to know that there are reserved values at this level.

The API is currently filtering what is redundant. We are already doing this when returning arrays : the serialization schema requires the array size to reserve the space in bytes, but the API  returns just the array.
Returning reserved1 : "" looks more confusing than helpful, since it does not indicate the number of reserved bytes.
The json response fields are not always ordered as they appear in the schema, if I'm not mistaken. So one can't trust the response directly to get the transaction in serialized form without the help of the catbuffer builders.

I agree with you David. The same points apply at SDK level. I don't see any
value and it's worse if we do it half way. I would not add those fields at
SDK model level. SDK will just send empty when serialising to catbuffer
On Tue, 5 Nov 2019, 19:44 David Garcia, notifications@github.com wrote:

In my opinion private / internal. It does not provide any value to the end
developer to know that there are reserved values at this level.

The API is currently filtering what is redundant. We are already
doing this when returning arrays : the serialization schema requires the
array size to reserve the space in bytes, but the API returns just the
array.
Returning reserved1 : "" looks more confusing than helpful, since
it does not indicate the number of reserved bytes.
The json response fields are not always ordered as they appear in
the schema, if I'm not mistaken. So one can't trust the response directly
to get the transaction in serialized form without the help of the catbuffer
builders.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/nemtech/nem2-openapi/pull/46?email_source=notifications&email_token=ABJEBXT53FIAQJKHSHRFORDQSG5IFA5CNFSM4JJFH6O2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDD4G3A#issuecomment-549962604,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJEBXUE2D3ALSLMXY76GR3QSG5IFANCNFSM4JJFH6OQ
.

@fboucquez @rg911 Just added one missing change we missed reported by @martinayora  (restrictionType-> restrictionFlags)
Diff: https://github.com/nemtech/nem2-openapi/compare/a9e66ee1453bb7caf04c7b5c20ff8843cd4a07d9...45-f2


text:
Failed to deploy a smart contract to NEO testnet blockchain

Hello,
After I created a smart contract and compiled it into avm file, I failed to deploy it using NEO-GUI. I didn't find a documentation about how to deploy a smart contract to blockchain using NEO-GUI. Could anyone help with it please? Thanks!
Following are what I did:
I used NEO-GUI and got testnet blocks synced. I have applied enough test NEO and NEO GAS.
I created a smart contract with below content if I load it in NEO-GUI:
60c56b6c766b00527ac4616168164e656f2e53746f726167652e476574436f6e7465787410616d6f756e745f646576656c6f706572617c680f4e656f2e53746f726167652e4765746c766b51527ac46168164e656f2e53746f726167652e476574436f6e7465787412616d6f756e745f6469737472696275746f72617c680f4e656f2e53746f726167652e4765746c766b52527ac46168164e656f2e53746f726167652e476574436f6e746578740f616d6f756e745f706c6174666f726d617c680f4e656f2e53746f726167652e4765746c766b53527ac46168164e656f2e52756e74696d652e47657454726967676572009c6c766b54527ac46c766b54c3644f026161654e046c766b55527ac4611404d86df4cb9331db3975d7dc5728b4503c8ba9866168184e656f2e52756e74696d652e436865636b5769746e6573736c766b56527ac46c766b56c3648f00616c766b55c36c766b52c3a0009c6c766b57527ac46c766b57c3646700616c766b52c36c766b55c3946c766b52527ac46168164e656f2e53746f726167652e476574436f6e7465787412616d6f756e745f6469737472696275746f726c766b52c3615272680f4e656f2e53746f726167652e50757461516c766b58527ac4620302006c766b58527ac462f801616168184e656f2e52756e74696d652e436865636b5769746e6573736c766b59527ac46c766b59c3648c00616c766b55c36c766b53c3a0009c6c766b5a527ac46c766b5ac3646400616c766b53c36c766b55c3946c766b53527ac46168164e656f2e53746f726167652e476574436f6e746578740f616d6f756e745f706c6174666f726d6c766b53c3615272680f4e656f2e53746f726167652e50757461516c766b58527ac4624f01006c766b58527ac4624401616168184e656f2e52756e74696d652e436865636b5769746e6573736c766b5b527ac46c766b5bc3648d00616c766b55c36c766b51c3a0009c6c766b5c527ac46c766b5cc3646500616c766b52c36c766b55c3946c766b51527ac46168164e656f2e53746f726167652e476574436f6e7465787410616d6f756e745f646576656c6f7065726c766b51c3615272680f4e656f2e53746f726167652e50757461516c766b58527ac4629a00006c766b58527ac4628f00006c766b58527ac46284006168164e656f2e52756e74696d652e47657454726967676572609c6c766b5d527ac46c766b5dc3644f00616c766b00c300c36c766b5e527ac46c766b5ec306627579617070876c766b5f527ac46c766b5fc3642300616c766b51c36c766b52c36c766b53c3615272653a006c766b58527ac4620f0061006c766b58527ac46203006c766b58c3616c756651c56b61516c766b00527ac46203006c766b00c3616c756658c56b6c766b00527ac46c766b51527ac46c766b52527ac461616530016c766b53527ac46c766b53c361950164966c766b54527ac46c766b53c361950164966c766b55527ac46c766b53c361950164966c766b56527ac46168164e656f2e53746f726167652e476574436f6e7465787410616d6f756e745f646576656c6f7065726c766b00c36c766b56c393615272680f4e656f2e53746f726167652e507574616168164e656f2e53746f726167652e476574436f6e7465787412616d6f756e745f6469737472696275746f726c766b51c36c766b55c393615272680f4e656f2e53746f726167652e507574616168164e656f2e53746f726167652e476574436f6e746578740f616d6f756e745f706c6174666f726d6c766b52c36c766b54c393615272680f4e656f2e53746f726167652e50757461516c766b57527ac46203006c766b57c3616c756651c56b61006c766b00527ac46203006c766b00c3616c756651c56b61006c766b00527ac46203006c766b00c3616c756657c56b6161682953797374656d2e457865637574696f6e456e67696e652e476574536372697074436f6e7461696e65726c766b00527ac46c766b00c361681d4e656f2e5472616e73616374696f6e2e4765745265666572656e6365736c766b51527ac4616c766b51c36c766b52527ac4006c766b53527ac4629e006c766b52c36c766b53c3c36c766b54527ac4616c766b54c36168154e656f2e4f75747075742e4765744173736574496461209b7cffdaa674beae0f930ebe6085af9093e5fe56b34a5c220ccdcf6efc336fc59c6c766b55527ac46c766b55c3642d006c766b54c36168184e656f2e4f75747075742e476574536372697074486173686c766b56527ac4622c00616c766b53c351936c766b53527ac46c766b53c36c766b52c3c09f6359ff006c766b56527ac46203006c766b56c3616c756651c56b6161682d53797374656d2e457865637574696f6e456e67696e652e476574457865637574696e67536372697074486173686c766b00527ac46203006c766b00c3616c7566
I tried to deploy it to NEO Test blockchain. I input name, version, etc. I loaded the avm file which generated above code. I checked need created storage and need dynamic call. Then, I click deploy.
It lead me to another dialog with some content of customized code ( It looks like it's invoking a contract rather than deploy it ). I click button to try running it. It gave me some messages like below ones:
VM State: HALT, BREAK
Gas Consumed: 490
Evaluation Stack: [{"type":"InteropInterface"}]
Whatever, I click button to invoke it, it gave me the transaction number as : 0xaeff5231c4dda16f4371a12ec7bf2b7ac3667eaa9a4cd277aa5b3512d830f936. While, I cannot find such transaction from neo testnet after around 20 mintues (more than 70 confirms of blocks).
https://testnet.neotracker.io/tx/aeff5231c4dda16f4371a12ec7bf2b7ac3667eaa9a4cd277aa5b3512d830f936. I got message that : Sorry, that page was not found.
And my test neo gas got deducted by 490.
Did I do anything wrong? What should I do to deploy a smart contract please? Thanks!
BTW: I did above operation from testnet address: AGDVd3yUb6Nywo8pKb3gdh1y6jWHBkenjr.

steps here not helpful?
http://docs.neo.org/en-us/sc/tutorial/Lock2.html

@jrsikors This is the way of deploying a local contract rather than deploying an application contract to blockchain.
I did found a link of doc of deploying contract to blockchain following your link:
http://docs.neo.org/en-us/sc/tutorial/deploy-invoke.html
While, I think I used the same steps as documented, but met the problem I mentioned in this issue.
Thanks!

Sorry I'm out of my element for deploying contracts.  Took a shot in the dark for ya.  Hopefully someone else who has knowledge of that topic swoops in.  Good luck & sorry !

@jrsikors Thank you anyway! :-)

repo: neo-project/neo-gui


text:
Add converter from Driver values to literals.

Cc @loveleif

Would be useful for Cypher Shell https://github.com/neo-technology/neo4j/blob/a2228408c89d2c8c3738777ee2ea0a231fc835be/public/community/cypher-shell/cypher-shell/src/main/java/org/neo4j/shell/parameter/ParameterPrettyRenderer.java#L62. For that use case we always want sorted maps though, so preferably make that configurable.

authors: loveleif michael-simons


text:
Dumping a summarized result may error

Describe the bug
Apparently, references don't play nicely with certain libraries. When an unconsumed summarized result gets dumped, it may crash.
To Reproduce
I run a query and then want to dump the result to see it, but an error occurs
Steps:

Create client
Run a query then dump the results

$result = $client->run("MATCH (n:Node) SET n.testing = 'hello' RETURN n");
dump($result);

This error is produced
In VarCloner.php line 108:

  Type error: Cannot assign Symfony\Component\VarDumper\Cloner\Stub to reference held by property Laudis\Neo4j\Databags\SummarizedResult::$summary of type ?Laudis\Neo4j\Databags\ResultSummary

This happens for both transactions and direct runs.
If I dump dump($result->getSummary()); it works and prints the summary, is this an intended behavior? I know this is not a critical issue, but I thought I would mention it :sweat_smile:
Note: something weird happens with the run, if a result is expected to be return (there is a return in the query), just the run itself doesn't run the query, but getSummary() or toArray() do, so this
$result = $client->run("MATCH (n:Node) WITH * CALL apoc.util.sleep(2000000) RETURN n");

finishes instantly, indicating that the query is not being ran or something.
But this
$result = $client->run("MATCH (n:Node) WITH * CALL apoc.util.sleep(2000000) RETURN n");
$summary = $result->getSummary();

or this
$result = $client->run("MATCH (n:Address) WITH * CALL apoc.util.sleep(2000000) RETURN n");
$array = $result->toArray();

Actually run the query and perform the sleep properly, is this behavior intended?
Expected behavior
The dump should work and return either null or the summary in place of the reference.
Additional context
Taken from #124

Hey @transistive :wave:
Unfortunately, I'm still running into the same exact issue
$result = $client->run("MATCH (n:Node) SET n.testing = 'hello' RETURN n");
dump($result);

Still gives the same error
In VarCloner.php line 108:

  Type error: Cannot assign Symfony\Component\VarDumper\Cloner\Stub to reference held by property Laudis\Neo4j\Databags\SummarizedResult::$summary of type ?Laudis\Neo4j\Databags\ResultSummary

is there something I'm missing? :thinking:
Client version: 2.6.1
Neo4j version: 4.4.3

authors: noureldin-khaled transistive


text:
Fix skipped integration tests for transactions

Describe the bug
Some tests in TransactionIntegrationTest.php fail due to a ConnectionTimeoutException and Write errors. We have decided to skip these tests for the time being
See the failed pipeline

Perfect. Can you create a new branch where you enabled the test again so we can see what is going on? We can dig a little bit deeper from there.


text:
Making the intergration test fixture os independed

I have added a Process based Neo4jInstaller that does not use PS or Windows Services to install/uninstall/start/stop the Neo4j services.
This new ProcessBasedNeo4jInstall can be opted in by using the compile symbol PROCESS_TEST_BAESED. If this symbol is not specified, the code run as before.
The motivation for the process based is that it is less error prone due to missing administrator rights and missing PS (Miscrosoft.Automation) assemblies. And it will also work on non-Windows systems, like Linux or OSX using mono.

Ill update the code to match 1.1 to make the PR easier :)

I am a bit surprised that this process installer could run on other os with the server in the windows zip. I actually would suggest instead of using the direct java command to run it, we could download the server in unix tar file and then use the neo4j start command normally.

Yes, that seems like a better solution. I will first update this PR to work with 1.1 and then I can start working on the Mono track and look into the unix tar. I actually started working on getting it to compile in Mono on OSX (including the test projects), but I could not run the intergration tests and didn't want to make any changes without the tests working :)
Please let me know if you think this process installer is a dead end and you have no plans to merge it :)

Do you have a link for the unix tar file?

All the latest downloads could be found here http://alpha.neohq.net/
I would say pls do the changes to get your code working so that you will not be blocked by failed ITs. The ITs are not write for any other platform other than windows at the beginning. And pls issue the Pr against 1.0 as we would like to support multi platforms for the first version too.
But instead of using the pure java command, I would suggest you to do something similar to this
https://github.com/neo4j/neo4j-java-driver/blob/1.1/driver/src/test/java/org/neo4j/driver/v1/util/Neo4jInstaller.java
https://github.com/neo4j/neo4j-java-driver/blob/1.1/driver/src/test/java/org/neo4j/driver/v1/util/Neo4jDefaultInstaller.java (which is a Unix installer, where you got all the commands for running the server)
I got some problem to find a good untar lib so I have not get a PR for your problem yet.
BTW, would you mind to sign CLA so that we could add your name in our build, the build is red is mainly because the build is not recognizing you now. http://neo4j.com/developer/cla/
Pls let me know when you sign the CLA then I will add your name on the white list.
I am recently ill and I will take a few days off. I will be back on Next Tuesday. During this time I might be really slow to help you.

I have refactured the code to use the neo4j.bat instead of running the java commend :)

I opened a new PR targeting at 1.0. There is no need to also keep a PR for 1.1 as I will do forward merge once the code to previous branches merged in :)
I actually thinking that the process-installer might not be needed. Could you try the new service installer in #33 to see if it works for you? I also change to use neo4j.bat, though we still need some code for other os. Once #33 is green, I will merge it in and hopefully you will not be blocked by ITs.

Yes, Ill try #33 and see if it works without the process stuff :)
I have already made code and changes so the ITs works on OSX using Xamarin and Mono and it seems to work great. Ill make a PR for this after #33 has been integrated :)

authors: IngvarKofoed zhenlineo


text:
v1.9 - appTemp graph

appTemp is displaying correctly on the combined Wind Chill/Heat Index/appTemp plot; thank you.  However, it also generates an empty appTemp graph.  Is it possible to remove that empty graph?

The empty card will be hidden in the next release 👍

authors: neoground swirlingair

repo: neoground/neowx-material


text:
Bottom sections are not pinned in KitKat

According to Material design guidelines > Settings and support, bottom sections are not pinned at bottom.
You can see above, screenshots from Moto X genymotion emulators.

Thanks for your work, an awesome Android library.

Ehm, I see that sections are not pinned to the bottom, what is the problem?

I think the problem is that I have 2 bottom sections, and the list of content in the navigation drawer is very long, I can scroll down, so these 2 bottom sections must be pinned to the bottom of the navigation drawer.
This same layout in Lollipop or old versions like 4.2 stay pinned at bottom. Why? This is not a problem?

Material guidelines tells that bottom sections should be pinned to the bottom ONLY IF there is enough space. If not, the section are not pinned to the bottom but it is inserted as normal section at the end of the list.

Look this carefully:

If the list of content in the navigation drawer is very long, the two options can be pinned to the bottom of the navigation drawer on a surface with a higher elevation. This surface is present only while at the top of the list; any other scroll position will immediately result in dismissing the surface and placing the options at the end of the list, in-line with the rest of the list content. The navigation drawer retains its scroll position when closed and reopened.

In my example, the Drawer is in the top of the list, so the settings sections must be pinned. Believe me is a bug.

authors: filipebezerra neokree

repo: neokree/MaterialNavigationDrawer


text:
Index corruption after compute crash and restart

Compute crashed while restoring a large schema (somewhere around 1000-2000 tables). After restart:
main=# set enable_sort=off;
SET
main=# select * from pg_depend order by refclassid, refobjid, refobjsubid;
ERROR:  index "pg_depend_reference_index" contains unexpected zero page at block 3
HINT:  Please REINDEX it.
main=#

This happened on the production env

I just try to load  backup on the latest sources and got the multiple errors likes this:
pg_restore: error: could not execute query: ERROR:  right sibling's left-link doesn't match: block 2 links to 42 instead of expected 38 in index "pg_constraint_conrelid_contypid_conname_index"

I do not using some noticeable memory consumption while pg_restore and later when compute is restarting and querying data. Maximal size of postgres backend doesn't exceed  170Mb (most of them are shared memory with shared_buffer_size=128MB). It it possible to assume that even i this case we can get OOM if total memory size of VM is limited by 256Mb and multiple parallel workers are started (I see 6 running parallel workers). But s far as I understand t even 1Gb of VM memory is not enough and only after 5Gb og_restore is able to complete. It is really strange.
Actually Ketteq schema is not so large:
postgres=# select count(*) from pg_class;
 count
-------
 49952
(1 row)

postgres=# select pg_relation_size('pg_class');
 pg_relation_size
------------------
         13697024
(1 row)

postgres=# select pg_relation_size('pg_attribute');
 pg_relation_size
------------------
         33857536
(1 row)

authors: hlinnaka knizhnik

repo: neondatabase/neon


text:
Instability in  test_create_churn_during_restart

https://neon-github-public-dev.s3.amazonaws.com/reports/pr-9715/11793083611/index.html#/testresult/18965ea660d34c01
fixtures.pageserver.http.PageserverApiException: wait for timeline initial uploads to complete: queue is in state Stopped

Failing since Wednesday 6th

The test environment is initialized with:
    env = neon_env_builder.init_configs()
    env.start()
    tenant_id: TenantId = env.initial_tenant
    timeline_id = env.initial_timeline

That doesn't create the initial tenant/timeline, it merely generates random IDs for them. So one of the 'tenant_create' and 'timeline_create' calls in the main loop will succeed to create the tenant and timeline.

Hmm, I see two initdb calls in the pageserver log though, for the first iteration of the loop. One of them is apparently cancelled,  (I added debug print here and it was printed), and the second one succeeds and creates the timeline.  Weird...

what's the expected behavior of the test before the restart? looking at the log, it seems that the initial tenant/timeline is not created successfully,
2024-11-12T08:52:57.245720Z  INFO attach{tenant_id=1cf8f19e855474304dcc9fbb62ce17e4 shard_id=0000 gen=00000001}: Attaching tenant attach_mode=Single
2024-11-12T08:52:57.245800Z  INFO attach{tenant_id=1cf8f19e855474304dcc9fbb62ce17e4 shard_id=0000 gen=00000001}:preload: listing remote timelines
2024-11-12T08:52:57.248838Z  INFO request{method=POST path=/v1/tenant/1cf8f19e855474304dcc9fbb62ce17e4/timeline request_id=2cdcc662-0089-4598-8c88-c68b5ce2097b}: Handling request
2024-11-12T08:52:57.250055Z  INFO attach{tenant_id=1cf8f19e855474304dcc9fbb62ce17e4 shard_id=0000 gen=00000001}:preload: found 0 timelines, and no manifest
2024-11-12T08:52:57.250201Z  INFO attach{tenant_id=1cf8f19e855474304dcc9fbb62ce17e4 shard_id=0000 gen=00000001}: Done

2024-11-12T08:52:58.741720Z  INFO attach{tenant_id=1cf8f19e855474304dcc9fbb62ce17e4 shard_id=0000 gen=00000002}: Attaching tenant attach_mode=Single
2024-11-12T08:52:58.741783Z  INFO attach{tenant_id=1cf8f19e855474304dcc9fbb62ce17e4 shard_id=0000 gen=00000002}:preload: listing remote timelines
2024-11-12T08:52:58.742722Z  INFO attach{tenant_id=1cf8f19e855474304dcc9fbb62ce17e4 shard_id=0000 gen=00000002}:preload: found 0 timelines, and no manifest

I think we should fix the test by:

either, we want the timeline to be fully set up before restarting, there should be no more issues
or, we want to test the case "what happens if the tenant/timeline is not fully set up" during restarts

it seems that the initial tenant/timeline is not created successfully,

not created successfully b/c the pageserver receives a shutdown request immediately after the request is being handled.

After further looking into the test case, I think it's valid for it to test "what happens if timeline creation is not complete but the pageserver shuts down". Therefore, I added more comments to the test case and correctly processed that "queue is stopped" error as a shutdown error.

authors: hlinnaka jcsp skyzh


text:
proxy: connect redis with AWS IAM

Problem
Support of IAM Roles for Service Accounts for authentication.
Summary of changes

Obtain aws 15m-long credentials
Retrieve redis password from credentials
Update every 1h to keep connection for more than 12h
For now allow to have different endpoints for pubsub/stream redis.

TODOs:

PubSub doesn't support credentials refresh, consider using stream instead.
We need an AWS role for proxy to be able to connect to both: S3 and elasticache.

Credentials obtaining and connection refresh was tested on xenon preview.
https://github.com/neondatabase/cloud/issues/10365
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Code is rather complex but it seems ok. I don't see anything inherently wrong. How do we plan to roll this out safely?

Code is rather complex but it seems ok. I don't see anything inherently wrong. How do we plan to roll this out safely?

Right now we don't have regional redis anywhere except the preview (and even with the preview it's a bit tricky).
For now I suggest the following plan:

Merge
Add helm-values
Check with the preview in full proxy and cplane setup.

authors: conradludgate khanova


text:
Documentation - Clarify the IntegerDecoder.int function

📖 Documentation
It would be nice to add to the "IntegerDecoder.int" function's documentation that there is no autocast from a string.
Maybe it makes sense to reference the "fromString" function in the same documentation, since there is an autocast in others library.

Hello,
Concerning this issue, we have multiple solutions:

No autocast + add the documentation to avoid such an issue
Add autocast to the Number / Int Decoder

What would be the best solution here? What would be the inconveniences of each solution?

repo: neoxia/apoyo


text:
Towards 4.0

A PR for following the progress towards 4.0.
Planned :

Remove legacy code
Better IDE compliance
Statement object
Pipeline
AutoFlushableTransaction
Bolt Support
GraphAware Response Formatter will replace the current one
More events

Moved to GraphAware repository


text:
Adding new requests, tests and examples.

This is the pull request for issue #8
( @pietbrauer Sorry about deleting the old one and opening this one, had to do some house cleaning on the branches.)
For anyone else, the idea of this pull request will be to basically extend TanukiKit to cover basically the entire GitLab API. We'll start by adding the GET requests, then the POST, then the DELETE and finally the PUT requests.
@pietbrauer , I was thinking of creating a standard structure for each file:

No typical Xcode header on files.
// MARK: Router the router segment.
The file name must start with capital and if it as a:

GET, just use the name of what you're getting, for example: Builds.swift or Commits.swift.
POST, start with Send and then the name of what you're posting, for example: SendKeys.swift.
DELETE, start with Remove and then the name of what you're deleting, for example: RemoveKeys.swift.
PUT, start with Update and then the name of what you're updating, for example: UpdateKeys.swift.

The main class for the file should be of style: api + name capitalised and on singular form + Class, for example: apiBuildClass, not apiBuildsClass, or for a POST, apiSendKeyClass.
Xcode document all main functions.
The main function name must be the file name camelised.
All functions must have tests with the Nocilla framework.

(Open to suggestions)

Hey Tiago, first of all thank you for your Pull Request!
There are a couple of things I would like you to clean up before I start to review:

You changed the permissions of almost every file from 0644 to 0755 afaik this was unnecessary, please revert that
file names are the resource name
Class names begin with a capital letter
Class names do not need to say that they have something to do with API or Class, the first one is implicit as this is a Swift API client for GitLab. The second one is just Swift convention, e.g. you don't see NSStringClass or AnyObjectClass as the user of the library already knows it is a class.
There is a lot of noise going on in the .pbxproj and I don't know why this was necessary. The only change should be the added file
Why was Repositories renamed to Projects? This is a breaking API change and I would have to release a new major version for this change, which could have been prevented. I would also like to have one change at a time and the change for this particular pull request is to add build support
Tests are missing and you broke the other tests

There is also an outstanding architectural changes in #6, just a heads up but nothing to worry about yet.

Thanks for the heads up @pietbrauer:

File permissions make sense and will be fixed, did that by mistake!
Agreed, will update that on the initial message.
Also agreed, forgot by mistake, will be fixed on next commit.
Right, this one is a bit hard to explain, but the main thing is, I bet most people using this framework will reference these classes in other places, for example to save the data to an array of objects:

class User {
    var login: String = ""
    var bio: String? = nil
    var name: String = ""
    var email: String = ""
    var privateToken: String = ""
    var avatarURL: String? = nil
    var webURL: String = ""
    var websiteURL: String? = nil
    var skype: String? = nil
    var linkedin: String? = nil
    var twitter: String? = nil
    var isAdmin = false
    var projectsLimit = 0
    var canCreateProject = false
    var canCreateGroup = false

    func addUserData(someAPIUser: APIUserClass) {
        name = someAPIUser.name!
        login = someAPIUser.login!
        bio = someAPIUser.bio
        email = someAPIUser.email!
        privateToken = someAPIUser.privateToken!
        avatarURL = someAPIUser.avatarURL
        webURL = someAPIUser.webURL!
        websiteURL = someAPIUser.websiteURL
        skype = someAPIUser.skype
        linkedin = someAPIUser.linkedin
        twitter = someAPIUser.twitter
        isAdmin = someAPIUser.isAdmin!
        projectsLimit = someAPIUser.projectsLimit!
        canCreateGroup = someAPIUser.canCreateGroup!
        canCreateProject = someAPIUser.canCreateProject!
        listOfUserData.append(User())
    }
}

As you can see this is quite a popular scenario, and the user would prefer to be able to call their class User instead of something else just because TanukiKit already uses that name, it would also cause confusion later, I do agree though with the removing the Class part, but I just think we should have a prefix on the class name to indicate it is the TanukiKit one, please tell which way you prefer.
5. I also noticed the noise, although I haven't touched that file in any way! Will check that.
6. I do realise this would mean changing the functions on the programs, but it is extremely necessary! Why? Firstly, because on GitLab they are called Projects, not repositories, also the API clearly treats them as projects, (https://gitlab.com/api/v3/projects) but that's acceptable, what really is a concern is that on GitLab repos are actually the files managed by git itself, the bare propose of the project let's say, and that's the way gitlab references them, so later, when I added the repository request, it would cause a big bowl of spaghetti mess.
7. One change at the time makes complete sense, this will be the builds one.
8. Firstly, sh**, I didn't even touch the tests yet, don;t know how I broke them, will take a look at that, secondly, haven't done the test for the builds yet because I realised that you only seem to have tests for the POST requests, like the keys, do you also want tests for the GETs?
Finally, I also usually work with Swift-Clean on Xcode, an amazing extension that works as build phase that mars as warnings stuff like variables that start with a capital letter and shouldn't etc. are you interested in keeping this or should I only use it locally?
Tiago
PS: Was thinking of doing a CONTRIBUTING.md with these guides later, approved?

@pietbrauer, commit 11cc33e (Fixed points 1, 2 and 3. (pull request #10)), fixes the permissions, updated the class names to point 3 until I wait for you to evaluate my request on naming.
I don't think I actually broke the tests, firstly because I didn't change the code, I literally just fixed the indentation because it was a bit messed up. I did find however that travis seems to be failing because of a Could not find jwt-1.5.3 in any of the sources error. What that is I don't know, but it seems to me like a dependency is missing, doesn't seem connected to the code itself.
The noise on the project file is because of A) the name change to TanukiKit instead of GitLabKit, you seem to have forgotten to change it after you found out GitLabKit already existed and B) to link the Pods required to develop the framework (if you handle the dependencies in any other way please feel free to say how and I'll change it).
Tiago

I use Carthage simply install using brew install carthage and then carthage bootstrap. This will download RequestKit and the tests should just run.
Regarding the error just cherry-pick https://github.com/nerdishbynature/TanukiKit/commit/c822abbba0b64fd847f97b1fc8b705f2cba83e6d into your branch.
I understand your point about the repositories/projects change, can you please do it in a separate PullRequest? And please do a separate PullRequest for the TanukiKit/GitlabKit change.
Regarding the class naming: Swift has modules and names have to be exclusive in a module, e.g. TanukiKit. If your app defines a class User what the swift compiler sees is MyApp.User (ModuleName.Class) and TanukiKit.User. Objective-C did not have this functionality and therefore you had to use class prefixes. This is not necessary with Swift anymore.

Yup got it, will start by removing cocoa pods and use carthage, then I will do a pull request with the TanukiKit change, then once that gets approved, I'll open another with the Projects change, then add the commits and runner first, as they are necessary later for the builds.
Got it, will use the normal class names then.
Tiago

Alright, @pietbrauer, this will be the GitLabKit -->  TanukiKit pull request, figured out what was wrong with the tests and as you can see everything is working now, also applied already the changes from RequestKit 1.0
Tiago

Hm, why exactly did you update to RequestKit 1.0? There is a pull request which was just waiting for this here to finish. Have a look at #11. That's multiple changes in one again. Also please don't change the file order. Please open a new pull request and change the name from GitLabKit to TanukiKit.

Got it, sorry, third time's the charm right?

authors: pietbrauer tiferrei

repo: nerdishbynature/TanukiKit


text:
The xiaomi_viomi integration does not support YAML setup ?

After latest updates to HAOS:
Logger: homeassistant.setup
Source: setup.py:233
First occurred: 20:06:15 (1 occurrences)
Last logged: 20:06:15
The xiaomi_viomi integration does not support YAML setup, please remove it from your configuration
What should be done ?

Ok well, i deleted everything under the card yaml and it generated excactly same config back and now it works ..

Ok well, i deleted everything under the card yaml and it generated excactly same config back and now it works ..

Do you mean deleted under the custom_components folder, or deleted the vacuum card from the lovelace gui?
Because I have the same issue.

Hi All, any instructions with the above issue as I have the same problem.

I've deleted the viomi folder under custom_components and redownloaded it through HACS. It's now working fine again.

Ok well, i deleted everything under the card yaml and it generated excactly same config back and now it works ..

Do you mean deleted under the custom_components folder, or deleted the vacuum card from the lovelace gui? Because I have the same issue.

No, i opened lovelace card and deleted all yaml code Under it.

I'm getting the same notification even though I've already deleted everything related to the component from configuration.yaml
Searching for xiaomi_viomi shows nothing except for files in the custom component itself. Does anyone else have the same problem and know how to stop the notification?

I'm getting the same notification even though I've already deleted everything related to the component from configuration.yaml Searching for xiaomi_viomi shows nothing except for files in the custom component itself. Does anyone else have the same problem and know how to stop the notification?

Yes, i have this error too again. nothing in configuration.yaml, but i have xiaomi-vacuum-map-card where i have 👍
entity: vacuum.viomi vacuum_platform: marotoweb/viomise
But i cannot remove those, as card will surely not work without :o

I have the same issue. Does anyone know any solution?

I just ignored it.

I just ignored it.

It makes me anxious to see that error every time I go into the settings. 🤣🤣
Also I updated the manifest
"python-miio>=0.5.12"

any new solutions?

Any news on this? I have the same problem and is annoying..

authors: 2137home Smoetzak Stefos13 SunilKhusal Tontze herberthobregon salvadornico

repo: nergal/homeassistant-vacuum-viomi


text:
Azure Devops Repo Gets Relative/Undefined URL

- Do you want to request a feature or report a bug?
bug
- What is the current behavior?
When using Azure Devops Repos, netlify-cli logs the repository url as "https://undefined.com/Team/Project/_git/repo. This ends up becoming a local URL in Netlify and prevents the site from being deployed.
- If the current behavior is a bug, please provide the steps to reproduce.
Create a repository in azure devops.
Run netlify init --manual
Check repository url
- What is the expected behavior?
Should get the full repository url
- Please mention your node.js, and operating system version.
Node: 10.15.1
OS: Windows 10

Thanks for the report.
Can you share anything else about your build spec?
What are the CLI commands being run etc
Thanks!

So I'm just running netlify init --manual.
It was for a standard Vue CLI 3 app with an Azure Devops git repo.
And followed the steps (added the SSH key, added the webhook, etc).
The initial build log looks like this:
7:47:56 PM: Build ready to start
7:47:57 PM: build-image version: 9e0f207a27642d0115b1ca97cd5e8cebbe492f63
7:47:57 PM: build-image tag: v3.3.2
7:47:57 PM: buildbot version: 75cd99f62ada9e21edea53208e8baf0eab85a045
7:47:58 PM: Fetching cached dependencies
7:47:58 PM: Failed to fetch cache, continuing with build
7:47:58 PM: Starting to prepare the repo for build
7:47:58 PM: git ref refs/heads/master does not exist or you do not have permission
7:47:58 PM: Failing build: Failed to prepare repo
7:47:58 PM: failed during stage 'preparing repo': git ref refs/heads/master does not exist

Here's a screenshot of the repository setting in the netlify dashboard.

When I ran netlify unlink and ran netlify init --manual , I would get the following options:
? What would you like to do? ⇄  Link this directory to an existing site

netlify link will connect a site in app.netlify.com to this folder

? How do you want to link this folder to a site? (Use arrow keys)
> Use current git remote url https://undefined.com/MOBY235/sharktank-collaboration-presentation
  Site Name
  Site ID

The actual repository url: https://MOBY235@dev.azure.com/MOBY235/sharktank-collaboration-presentation/_git/sharktank-collaboration-presentation
I thought maybe the @ symbol was causing some problems. URL encoding the "@" symbol broke the repository, but I found I could remove the "MOBY235@" part of the url and still be able to pull/push from the remote repository. However, it still had the same undefined.com url.

In the CI process you want to avoid netlify link and netlify init
Instead you can use the site, auth with the netlify deploy command
This is the "whole flow"
npm install
npm install netlify-cli -g
netlify deploy --prod --dir yourbuilddir --message "my deploy message" --auth xyz --site 123

The auth token is a personal access token you can grab from your account settings page and the site ID is inside the site settings

It seems like our dep https://github.com/netlify/cli/blob/master/src/utils/get-repo-data.js#L5 isn't picking up the git remote in your setup for whatever reason. Perhaps the git remote isn't listed as origin?
Anywho, If this is for an automated CI build, the netlify deploy with args should work for ya

I wonder if this is related to #258 since azure devops repos are private by default.

@mdarrik did you try the following commands?
npm install
npm install netlify-cli -g
netlify deploy --prod --dir yourbuilddir --message "my deploy message" --auth xyz --site 123

That worked for manual deploys.

I want to add that this issue isn't related to CI purposes. I was using Azure Dev Ops as my git repository so I didn't have to link to my personal Github from my work PC.

Interesting! TIL
It looks like the issue might be here then https://github.com/netlify/cli/blob/master/src/utils/get-repo-data.js#L40-L49 which means it will show undefined here: https://github.com/netlify/cli/blob/61285ee88792a1e81dd46d7e80147cdc0506df26/src/utils/link/link-by-prompt.js#L24
What is the git remote -v origin of your repo? We can add that to the supported list of hosts

So it's a little weird with Azure. When you grab it from Azure, it sets it at {username}@dev.azure.com/{username}/{repository}.
It also works with https://dev.azure.com/{username}/{repository} which probably fits your existing schema better.

Hey @mdarrik, can you please try out raees/fix-repo-url branch and let me know if it works correctly.

I am having this same issue. Within Netlify the repo is a relative path, which is then not found.
stored as:
USERNAME/PROJECT/_git/REPO

as opposed to the correct repo URL:
https://USER@dev.azure.com/USER/PROJECT/_git/REPO

What can be done? Is there a manual way to set the repo url?

authors: DavidWells andwilr mdarrik raeesbhatti

repo: netlify/cli


text:
chore: bump @netlify/build to 29.21.1

🎉 Thanks for submitting a pull request! 🎉
Summary
Bump because renovate sad

For us to review and ship your PR efficiently, please perform the following steps:

[ ] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved. If you`re fixing a typo or something that`s on fire 🔥 (e.g. incident related), you can skip this step.
[ ] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[ ] Update or add tests (if any source code was changed or added) 🧪
[ ] Update or add documentation (if features were changed or added) 📝
[ ] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

I am a renovate newbie so I really appreciate that - I was triggering it a billion times via the dashboard but every time coming up that it was skipping. Had a google and found nothing related to that, really appreciate the pointer!


text:
Allow markdown code blocks in the editor

- Do you want to request a feature or report a bug?

feature (possibly bug?)

- What is the current behavior?

trying to include markdown code blocks (```) in an editor markdown widget field "by hand" does not write them to the output .md file

- If the current behavior is a bug, please provide the steps to reproduce.

not sure it's a bug or not implemented funtionality

- What is the expected behavior?

the editor will write to the .md file (i.e. on github) WITHOUT escaping/removing the back ticks the following (preferably maintaining the "code language" i.e. the "js" in the example below, so it can be used by syntax highlighters):

```js
// some sample code
const f = () => {
console.log('foo');
}
```

more about this here: https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet#code

- Please mention your node.js, and operating system version.

This appears to be a bug. When I look at the PR it creates, the diff shows it's adding 4 spaces to the beginning of the code block:
+    ```js
+    function () {
+    }
+    ```

Which renders as:
    ```js
    function () {
    }
    ```

Removing the spaces at the beginning correctly renders as:
function () {
}

I think it's because the component is reading the html and adding both <pre> and <code> blocks, but I'm not 💯 sure of the solution.

Closing in favor of #246

authors: WebStackDev erquhart tcbyrd

repo: netlify/netlify-cms


text:
Gatsby - NetlifyCMS build fails in remote, while builds in local

Describe the bug
My project build using Gatsby + Netlify CMS Starter, doesn't build in netlify environment. While the build is successful in local Windows 10 environment.
From the error assuming it is something related building css and js bundles.
Since build is successful in local, guessing the issue could be with netlify.
Terminal
12:11:36 PM: Waiting for other deploys from your team to complete
12:15:26 PM: Build ready to start
12:15:28 PM: build-image version: 9cade8af58c2cf3a17a1e9433d2e979149488837
12:15:28 PM: build-image tag: v3.3.5
12:15:28 PM: buildbot version: 6fd888c8d66fd09bb18f09e1728e11fbc555b868
12:15:28 PM: Fetching cached dependencies
12:15:28 PM: Failed to fetch cache, continuing with build
12:15:28 PM: Starting to prepare the repo for build
12:15:29 PM: No cached dependencies found. Cloning fresh repo
12:15:29 PM: git clone git@gitlab.com:shaheen-timesworld/timesworld-website
12:15:32 PM: Preparing Git Reference refs/heads/master
12:15:33 PM: Found Netlify configuration file netlify.toml in site root
12:15:33 PM: Found Netlify configuration file(s). Overriding site configuration
12:15:33 PM: Different functions path detected, going to use the one specified in the Netlify configuration file: 'lambda' versus '' in the Netlify UI
12:15:33 PM: Creating functions prep folder
12:15:33 PM: Starting build script
12:15:33 PM: Installing dependencies
12:15:36 PM: Downloading and installing node v10.19.0...
12:15:36 PM: Downloading https://nodejs.org/dist/v10.19.0/node-v10.19.0-linux-x64.tar.xz...
12:15:37 PM:
########################                                                  33.7%
12:15:37 PM:
#################################################
12:15:37 PM: ####################### 100.0%
12:15:37 PM: Computing checksum with sha256sum
12:15:37 PM: Checksums matched!
12:15:39 PM: Now using node v10.19.0 (npm v6.13.4)
12:15:39 PM: Attempting ruby version 2.6.2, read from environment
12:15:41 PM: Using ruby version 2.6.2
12:15:41 PM: Using PHP version 5.6
12:15:41 PM: Started restoring cached node modules
12:15:41 PM: Finished restoring cached node modules
12:15:41 PM: Started restoring cached yarn cache
12:15:41 PM: Finished restoring cached yarn cache
12:15:41 PM: Installing yarn at version 1.9.4
12:15:41 PM: Installing Yarn!
12:15:41 PM: > Downloading tarball...
12:15:41 PM: [1/2]: https://yarnpkg.com/downloads
12:15:41 PM: /1.9.4/yarn-v1.9.4.tar.gz --> /tmp/yarn.tar.gz.gSL2H43cN
12:15:41 PM: v
12:15:41 PM:   % Total    % Received % Xfe
12:15:41 PM: rd  Average Speed   Time    Time     Tim
12:15:41 PM: e  Current
12:15:41 PM:               Dload  Upload   Tot
12:15:41 PM: al   Spent    Left  Speed
12:15:41 PM:
0     0    0     0    0
12:15:41 PM:  0      0      0 --:--:-- --:--:-- --:--:--     0
12:15:41 PM:
0    78    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--
12:15:41 PM:     0
100    78  100    78    0     0    259      0 --:--:-- --:--:-- --:--:--   259
12:15:41 PM:
100    91  100    91    0     0    146      0 --:--:-- --:--:-- --:--:--   146
12:15:42 PM:
100   608  100   608    0     0    563      0  0:00:01  0:00:01 --:--:--   563
12:15:42 PM:
0     0    0     0    0     0      0
12:15:42 PM:  0 --:--:--  0:00:01 --:--:--     0
12:15:42 PM:
100  915k  100  915k    0     0   618k      0  0:00:01  0:00:01 --:--:-- 4975k
12:15:42 PM: [2/2]: https://yarnpkg
12:15:42 PM: .com/downloads/1.9.4/yarn-v1.9.4.tar.gz.asc --> /tmp/yarn.tar.gz.gSL2H43cNv.asc
12:15:42 PM:
100    82  100    82    0     0   2945      0 --:--:-- --:--:
12:15:42 PM: -- --:--:--  2945
12:15:42 PM:
100    95  100    95    0     0   1715      0 --:--:-- --:--:-- --:--:--  1715
12:15:43 PM:
100   612  100   612    0     0   2859      0 --:--:-- --:-
12:15:43 PM: -:-- --:--:--  2859
12:15:43 PM:
100   832  100   832    0     0   3052      0
12:15:43 PM:  --:--:-- --:--:-- --:--:--  3052
12:15:43 PM: > Verifying integrity...
12:15:43 PM: gpg: Signature made Fri 03 Aug 2018 07:53:36 PM UTC using RSA key ID B6FF4DE3
12:15:43 PM: gpg: Good signature from "Yarn Packaging yarn@dan.cx"
12:15:43 PM: gpg: Note: This key has expired!
12:15:43 PM: Primary key fingerprint: 72EC F46A 56B4 AD39 C907  BBB7 1646 B01B 86E5 0310
12:15:43 PM:      Subkey fingerprint: E219 30C4 D0A4 AA46 1858  1F7A E074 D16E B6FF 4DE3
12:15:43 PM: > GPG signature looks good
12:15:43 PM: > Extracting to ~/.yarn...
12:15:43 PM: > Adding to $PATH...
12:15:43 PM: > Successfully installed Yarn 1.9.4! Please open another terminal where the yarn command will now be available.
12:15:44 PM: Installing NPM modules using Yarn version 1.9.4
12:15:44 PM: yarn install v1.9.4
12:15:44 PM: [1/4] Resolving packages...
12:15:45 PM: [2/4] Fetching packages...
12:15:46 PM: (node:1376) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
12:16:12 PM: info fsevents@2.1.2: The platform "linux" is incompatible with this module.
12:16:12 PM: info "fsevents@2.1.2" is an optional dependency and failed compatibility check. Excluding it from installation.
12:16:12 PM: info fsevents@1.2.11: The platform "linux" is incompatible with this module.
12:16:12 PM: info "fsevents@1.2.11" is an optional dependency and failed compatibility check. Excluding it from installation.
12:16:12 PM: [3/4] Linking dependencies...
12:16:12 PM: warning "gatsby > react-hot-loader@4.12.18" has unmet peer dependency "@types/react@^15.0.0 || ^16.0.0".
12:16:12 PM: warning "gatsby > @typescript-eslint/eslint-plugin > tsutils@3.17.1" has unmet peer dependency "typescript@>=2.8.0 || >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.6.0-beta || >= 3.7.0-dev || >= 3.7.0-beta".
12:16:12 PM: warning "gatsby > gatsby-cli > ink@2.6.0" has unmet peer dependency "@types/react@>=16.8.0".
12:16:12 PM: warning "gatsby > gatsby-cli > ink > auto-bind@3.0.0" has unmet peer dependency "@types/react@>=16.8.0".
12:16:12 PM: warning "gatsby > socket.io > engine.io > ws@7.2.1" has unmet peer dependency "bufferutil@^4.0.1".
12:16:12 PM: warning "gatsby > socket.io > engine.io > ws@7.2.1" has unmet peer dependency "utf-8-validate@^5.0.2".
12:16:12 PM: warning "gatsby-plugin-netlify > webpack-assets-manifest@3.1.1" has unmet peer dependency "webpack@>=4.4.0".
12:16:12 PM: warning "gatsby-plugin-sass > sass-loader@7.3.1" has unmet peer dependency "webpack@^3.0.0 || ^4.0.0".
12:16:12 PM: warning " > lodash-webpack-plugin@0.11.5" has unmet peer dependency "webpack@^2.0.0 || ^3.0.0 || ^4.0.0".
12:16:12 PM: warning "netlify-cms-app > netlify-cms-widget-code@1.1.2" has unmet peer dependency "codemirror@^5.46.0".
12:16:12 PM: warning "netlify-cms-app > netlify-cms-widget-code > react-codemirror2@6.0.0" has unmet peer dependency "codemirror@5.x".
12:16:12 PM: warning "netlify-cms-app > netlify-cms-core > redux-notifications > react-redux@4.4.10" has incorrect peer dependency "redux@^2.0.0 || ^3.0.0".
12:16:12 PM: warning " > netlify-cms-media-library-cloudinary@1.3.4" has unmet peer dependency "netlify-cms-lib-util@^2.1.3-beta.0".
12:16:12 PM: warning " > slick-carousel@1.8.1" has unmet peer dependency "jquery@>=1.8.0".
12:16:12 PM: warning " > styled-components@5.0.0" has unmet peer dependency "react-is@>= 16.8.0".
12:16:24 PM: [4/4] Building fresh packages...
12:16:31 PM: Done in 46.28s.
12:16:31 PM: NPM modules installed using Yarn
12:16:31 PM: Started restoring cached go cache
12:16:31 PM: Finished restoring cached go cache
12:16:31 PM: unset GOOS;
12:16:31 PM: unset GOARCH;
12:16:31 PM: export GOROOT='/opt/buildhome/.gimme/versions/go1.12.linux.amd64';
12:16:31 PM: export PATH="/opt/buildhome/.gimme/versions/go1.12.linux.amd64/bin:${PATH}";
12:16:31 PM: go version >&2;
12:16:31 PM: export GIMME_ENV='/opt/buildhome/.gimme/env/go1.12.linux.amd64.env';
12:16:31 PM: go version go1.12 linux/amd64
12:16:31 PM: Installing missing commands
12:16:31 PM: Verify run directory
12:16:31 PM: Executing user command: npm run build
12:16:31 PM: > timesworld-website@0.0.1 build /opt/build/repo
12:16:31 PM: > npm run clean && gatsby build
12:16:32 PM: > timesworld-website@0.0.1 clean /opt/build/repo
12:16:32 PM: > gatsby clean
12:16:32 PM: info Deleting .cache, public
12:16:32 PM: info Successfully deleted directories
12:16:35 PM: success open and validate gatsby-configs - 0.036s
12:16:36 PM: success load plugins - 1.046s
12:16:36 PM: success onPreInit - 0.008s
12:16:36 PM: success delete html and css files from previous builds - 0.013s
12:16:36 PM: success initialize cache - 0.009s
12:16:36 PM: success copy gatsby files - 0.036s
12:16:36 PM: success onPreBootstrap - 0.004s
12:16:36 PM: success createSchemaCustomization - 0.013s
12:16:36 PM: success source and transform nodes - 0.553s
12:16:37 PM: success building schema - 0.406s
12:16:37 PM: success createPages - 0.082s
12:16:37 PM: success createPagesStatefully - 0.106s
12:16:37 PM: success onPreExtractQueries - 0.000s
12:16:37 PM: success update schema - 0.028s
12:16:37 PM: success extract queries from components - 0.559s
12:16:37 PM: success write out requires - 0.004s
12:16:37 PM: success write out redirect data - 0.001s
12:16:37 PM: success onPostBootstrap - 0.000s
12:16:37 PM:
12:16:37 PM: info bootstrap finished - 5.128 s
12:16:37 PM:
12:16:39 PM: success Generating image thumbnails - 1.717s - 37/37 21.55/s
12:16:55 PM: failed Building production JavaScript and CSS bundles - 16.996s
12:16:55 PM: error Generating JavaScript bundles failed
12:16:55 PM: Cannot read property 'match' of undefined
12:16:55 PM: not finished run queries - 17.205s
12:16:55 PM: npm
12:16:55 PM:  ERR! code ELIFECYCLE
12:16:55 PM: npm
12:16:55 PM:  ERR!
12:16:55 PM: errno 1
12:16:55 PM: npm
12:16:55 PM: ERR! timesworld-website@0.0.1 build: npm run clean && gatsby build
12:16:55 PM: npm ERR!
12:16:55 PM:  Exit status 1
12:16:55 PM: npm
12:16:55 PM: ERR!
12:16:55 PM: npm
12:16:55 PM: ERR! Failed at the timesworld-website@0.0.1 build script.
12:16:55 PM: npm ERR!
12:16:55 PM:  This is probably not a problem with npm. There is likely additional logging output above.
12:16:55 PM: npm ERR!
12:16:55 PM:  A complete log of this run can be found in:
12:16:55 PM: npm ERR!     /opt/buildhome/.npm/_logs/2020-02-24T06_46_55_313Z-debug.log
12:16:55 PM: Function Dir: /opt/build/repo/lambda
12:16:55 PM: TempDir: /tmp/zisi-5e537020b301e20007c76af8
12:16:59 PM: Prepping functions with zip-it-and-ship-it 0.3.1
12:16:59 PM: [ { path: '/tmp/zisi-5e537020b301e20007c76af8/hello.zip',
12:16:59 PM:     runtime: 'js' } ]
12:16:59 PM: Prepping functions complete
12:16:59 PM: Caching artifacts
12:16:59 PM: Started saving node modules
12:16:59 PM: Finished saving node modules
12:16:59 PM: Started saving yarn cache
12:16:59 PM: Finished saving yarn cache
12:16:59 PM: Started saving pip cache
12:16:59 PM: Finished saving pip cache
12:16:59 PM: Started saving emacs cask dependencies
12:17:00 PM: Finished saving emacs cask dependencies
12:17:00 PM: Started saving maven dependencies
12:17:00 PM: Finished saving maven dependencies
12:17:00 PM: Started saving boot dependencies
12:17:00 PM: Finished saving boot dependencies
12:17:00 PM: Started saving go dependencies
12:17:00 PM: Finished saving go dependencies
12:17:02 PM: Error running command: Build script returned non-zero exit code: 1
12:17:02 PM: failed during stage 'building site': Build script returned non-zero exit code: 1
12:17:02 PM: Failing build: Failed to build site
12:17:03 PM: Shutting down logging, 1 messages pending
Package.json
{
"name": "timesworld-website",
"description": "Timesworld Website with Netlify-CMS",
"version": "0.0.1",
"author": "Shaheen Umar & Sreedarsh CP",
"dependencies": {
"babel-plugin-styled-components": "^1.10.6",
"bulma": "^0.8.0",
"gatsby": "^2.13.31",
"gatsby-image": "^2.0.23",
"gatsby-plugin-netlify": "^2.0.6",
"gatsby-plugin-netlify-cms": "^4.1.6",
"gatsby-plugin-purgecss": "^4.0.0",
"gatsby-plugin-react-helmet": "^3.0.4",
"gatsby-plugin-sass": "^2.0.7",
"gatsby-plugin-sharp": "^2.2.9",
"gatsby-plugin-styled-components": "^3.1.17",
"gatsby-remark-copy-linked-files": "^2.0.7",
"gatsby-remark-images": "^3.1.6",
"gatsby-remark-relative-images": "^0.2.1",
"gatsby-source-filesystem": "^2.0.26",
"gatsby-transformer-remark": "^2.6.9",
"gatsby-transformer-sharp": "^2.1.9",
"lodash": "^4.17.15",
"lodash-webpack-plugin": "^0.11.4",
"netlify-cms-app": "^2.9.6",
"netlify-cms-media-library-cloudinary": "^1.3.2",
"netlify-cms-media-library-uploadcare": "^0.5.2",
"node-sass": "^4.11.0",
"prop-types": "^15.6.0",
"react": "^16.8.4",
"react-dom": "^16.8.4",
"react-helmet": "^5.2.0",
"react-slick": "^0.25.2",
"slick-carousel": "^1.8.1",
"styled-components": "^5.0.0",
"uuid": "^3.2.1"
},
"keywords": [
"gatsby"
],
"license": "MIT",
"main": "n/a",
"scripts": {
"clean": "gatsby clean",
"start": "npm run develop",
"build": "npm run clean && gatsby build",
"develop": "npm run clean && gatsby develop",
"format": "prettier --trailing-comma es5 --no-semi --single-quote --write "{gatsby-.js,src/**/.js}"",
"test": "echo "Error: no test specified" && exit 1"
},
"devDependencies": {
"prettier": "^1.15.3"
}
}

Somehow issue is fixed, wasn't able to know root cause of earlier issue.

Maybe a caching issue, thanks for following up here!

authors: erezrokah iamshaheenumar


text:
Preview mode not working

I'm unable to get preview mode working, I've followed this guide to set up previews and this guide to get it working on netlify.  It works when I run it locally but it doesn't work on netlify, it sends me to a 404 instead of a preview. Is there a different way I should set up preview mode? There's only one guide available and I followed it.

@theelk801 hello! could you please provide a deployable repo that reproduces your issue? that would help us quickly diagnose why you're running into issues :).

repo: netlify/netlify-plugin-nextjs


text:
Successful PUT to /deploys/{deploy_id}/files/{path} returns capitalized keys

https://community.netlify.com/t/api-endpoint-deploys-deploy-id-files-path-randomly-gives-strange-results/22091/
The OP noted that a successful PUT request returns:
  {
    ID: '/static/2555efdf6712bc438581a141651552c2eb423ab6/photo-1598454242110-0744560ec77e.jpg',
    Path: '/static/2555efdf6712bc438581a141651552c2eb423ab6/photo-1598454242110-0744560ec77e.jpg',
    SHA: '2555efdf6712bc438581a141651552c2eb423ab6',
    MimeType: 'image/jpeg',
    Size: 170443
  }

whereas the docs (https://open-api.netlify.com/#operation/uploadDeployFile) show the returned object as:
{
    "id": "string",
    "path": "string",
    "sha": "string",
    "mime_type": "string",
    "size": 0

}

I think this is the server code for the response: https://github.com/netlify/netlify-server/blob/dee71a22aa145d66897c0c1ba62ebc3269bae6e3/server/request_file_upload.go#L45-L51
We should either lowercase the keys or update the open-api docs to show the capitalized keys.

Also note that other endpoints adhere to the spec, so this is not a general problem and only pertains to the uploadDeployFile operation.

This should be fixed now, the response now adheres to the docs.
Thanks for reporting!

Perfect! Is working fine now, thanks for fixing!

authors: kaganjd kitop styxlab

repo: netlify/open-api


text:
NETOBSERV-1554 Add CLI to Krew index

Description
This is a preparatory work for krew index creation:

[X] make command unique since krew is accepting a single bin entry
[X] add make release target to generate krew package and prepare github release
[x] detect when CLI is running from krew to force either kubectl or oc

Here is a krew index sample:
https://gist.github.com/jpinsonneau/a34785c85a20c3899ec91f266ff33823
You can test it using:
kubectl krew install --manifest=netobserv-cli-index.yaml

The krew index will be automatically generated by github release job and appened to body release notes as draft.
Dependencies
n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[X] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[ ] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[X] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

/lgtm

repo: netobserv/network-observability-cli


text:
No way to choose error code for RST_STREAM when closing Http2MultiplexHandlerStreamChannel

Expected behavior
If a handler in an Http2MultiplexHandlerStreamChannel pipeline needs to shut down the channel and send a RST_STREAM, I should be able to easily choose the error code sent with the RST_STREAM.
Actual behavior
Closing an Http2MultiplexHandlerStreamChannel results in a default Http2Error code of CANCEL being sent in a RST_STREAM before shutting down, with no way to choose this.
Getting around this involves one of:

Manually creating a RST_STREAM frame with the right error code, sending it, then closing the chan. This lacks the knowledge of the stream state that netty has, unfortunately (e.g., if a RST_STREAM has already been sent)
Triggering .fireExceptionCaught() on the Http2FrameCodec in the parent connection channel. Will have to hold a reference to the parent channel/context/pipeline/etc, have to craft the right exception, invoke it, etc. Hacky but doable.
Pass down (or look up) a reference to the Http2FrameCodec, so the Http2LifecycleManager methods (like resetStream, onError, goAway, etc) can be called on it with the local Http2MultiplexHandlerStreamChannel's handler's context. This feels less awkward then pushing errors back up into the parent pipeline. Plus, onError is very handy for exception handling.
Ditch Http2MultiplexHandler entirely and go back to the older style of handlers extending Http2ConnectionHandler. A lot of work for this.

Steps to reproduce
n/a
Relevant code
https://github.com/netty/netty/blob/57d536c59e434e88c0d8374d6b03671edb02885f/codec-http2/src/main/java/io/netty/handler/codec/http2/AbstractHttp2StreamChannel.java#L684-L688
Note the necessary .close function is non-public.
Netty version
4.x, not sure about 5
JVM version (e.g. java -version)
n/a
OS version (e.g. uname -a)
n/a
Potential Changes

Make AbstractHttp2StreamChannel#closeWithError public
Extract the Http2LifecycleManager implementations from Http2ConnectionHandler into something more suitable for reuse.
Make Http2MultiplexHandlerStreamChannel implement Http2LifecycleManager?
Something else???

Original Discord thread
KingMob — Today at 12:32 PM
What's the best way to close an HTTP2 multiplexed frame stream to get Netty to send a RST_STREAM for me with a particular error code? (Closing by default seems to send a RST_STREAM with the CANCEL error code, which isn't necessarily what I want.)
AbstractHttp2StreamChannel.closeWithError isn't public, or I'd call it directly.
Http2MultiplexHandler has the right logic. Should I hold a reference to the context of the handler before it and call .fireExceptionCaught() on it, to trigger the desired processing? That seems convoluted.
(I'm really unclear why AbstractHttp2StreamChannel doesn't seem to have much exception-handling itself.)
I see Http2LifecycleManager.onError() is public; should I be calling that? Or better still, maybe Http2LifecycleManager.resetStream()? I could hold a reference to the Http2FrameCodec and its context in the children channels, and make use of the Http2LifecycleManager methods that way.
Any guidance appreciated. 🙏
yawkat — Today at 1:16 PM
hm it seems to me like there should be a Http2Channel.close with an error code parameter. but not sure how that should interact with pipeline close.
Rogan — Today at 2:00 PM
close(error) does writeAndFlush(error).addListener(CLOSE);
?
Rogan — Today at 2:08 PM
How does SSLChannel.close() work? That surely also has to write something to the other end, receive confirmation, and only then close the channel, I think.
KingMob — Today at 4:55 PM
@yawkat The pipeline close (and indeed, most .close fns), are all quite generic, so it'd require making an http2-specific pipeline. AbstractHttp2StreamChannel.closeWithError is exactly what I want, but it's not public. There's also close(final ChannelPromise promise, Http2Error error), which isn't public either.
@Rogan Where are you looking at? I don't see an SSLChannel class in either netty 4 or 5.
Rogan — Today at 5:21 PM
Was without reference to code, so more conceptual than anything else. Referring to "the SslHandler" rather than SSLChannel, actually.
violetagg — Today at 5:29 PM
@KingMob well it depends what you want to do. If you have Http2MultiplexHandler, when this handler receives an error event it closes the stream with the proper error.
Do you need somethin like this? Do you need it for a code outside of Netty code base? Isn't fireExceptionCaught a solution for you?
Rogan — Today at 5:31 PM
From his comment above:

Http2MultiplexHandler has the right logic. Should I hold a reference to the context of the handler before it and call .fireExceptionCaught() on it, to trigger the desired processing? That seems convoluted.

It means you need to know what Exception maps to which error, requires construction of an Exception instance, etc
If you know the error you want to return, just call closeWithError(error), makes more sense to me
KingMob — Today at 5:47 PM
@violetagg I'm interested in calling this from the multiplexed child streams (Http2MultiplexHandlerStreamChannel), which would be after Http2MultiplexHandler. If I just close the channels, netty uses the CANCEL code, but what if I want a different code?
@Rogan closeWithError isn't public.
Rogan — Today at 5:51 PM
Yes, I understand it is not public.
KingMob — Today at 5:52 PM

Do you need somethin like this? Do you need it for a code outside of Netty code base?

Yes, this is for Aleph, which is built on Netty. I can live with CANCEL, I'm just surprised it's so hard to do.
Rogan — Today at 5:52 PM
Sounds like a simple solution would be to make it public, but Netty is fairly strongly on the side of not exposing things if they are not required.
It obviously does make changing internals easier
KingMob — Today at 5:55 PM
It just seems like an exception handled locally in a stream channel should be able to choose the error code if it needs to send a RST_STREAM and close
Should I open an issue for this? It feels like there's no non-hacky way to do this with http2 multiplexed child channels
yawkat — Today at 5:58 PM
yes i agree there should be a way to do this on Http2StreamChannel. i'm just not sure on the api given that pipeline close cant have attached information atm. you should make an  issue.
KingMob — Today at 5:59 PM
Thank you for your help everyone

I wonder why you would use any other code... Can you give some details ?

also I think we could just allow people to write the HttpResetFrame and just close the channel in this case

@normanmaurer Thanks for taking the time to read this.
I learned a bit more a couple days ago. I should probably have updated this immediately.
Earlier this week, I explored writing reset frames directly (option 1 in my first comment), and that seems to work. I thought it would cause a problem with stream state, but it appears when the Http2MultiplexCodec gets it, it actually performs all the necessary checks instead of blindly writing out what I send it.

I wonder why you would use any other code

Well, generally, because I might want to send a different H2 error than CANCEL. If anything should be a default, the RFCs suggest it should be PROTOCOL_ERROR, not CANCEL. CANCEL is fine for just closing a stream channel, I guess, but since it suggests there was no error, it means we can't use .close for error conditions on stream channels. (Though there's a bigger discussion about whether 4xx/5xx responses should be used instead of, or in addition to).
More specifically, I needed to send a RST_STREAM because Netty doesn't properly check yet for missing mandatory pseudo-headers (Netty checks the values, but not if a request is missing :method entirely). RFC 9113 § 8.1.1 says that malformed messages like that are PROTOCOL_ERROR, so this all started because I was trying to send that.
FWIW, I still think making .closeWithError public is a good solution. That logic could even be extended to conn channels to send GOAWAYs.
I'm comfortable closing this, since I have a working solution. I don't know if it could be handled with better documentation or not. LMK

@KingMob WDYT about https://github.com/netty/netty/pull/13651 ?

So, whenever we want to send a RST_STREAM, we throw a Http2FrameStreamException with the right Http2Error, and let it reach the end of the pipeline?
And then, if I want/need to send a response first, I can write it out, then throw after? (See https://lists.w3.org/Archives/Public/ietf-http-wg/2023JulSep/0183.html for the discussion I triggered about this. Generally, a lot of people think HTTP responses offer more info and better compatibility than a pure RST_STREAM, so I'm trying to do that when possible.)
If so, it looks good to me. Thanks!
My only question is just semi-related, so feel free to ignore: When I want to (1) send a response, and then (2) send a RST_STREAM, there's a race condition with stream state. The flow control code buffers up the HEADERS/DATA, and if I don't wait for the write future, the RST_STREAM can close the stream before the flow controller flushes. This is fine according to the RFC, but is there a way to force the FlowController to flush so there's no delay, and the RST_STREAM immediately follow the ES DATA frame?
(Side note: why is Http2FrameStreamException so separate from Http2Exception and Http2Exception.StreamException? I can kinda see that Http2FrameStreamException is used for definitely-existing streams, and Http2Exceptions might entail full GOAWAYs/shutdowns, but fwiw, it seems like they should be a bit closer than they are.)

authors: KingMob normanmaurer

repo: netty/netty


text:
Deadlock when starting server

Expected behavior
Server starts
Actual behavior
Sometimes server hangs
Steps to reproduce
This happens in the Infinispan test suite that starts and stops server dozens of times. There isn't a
reproducer at the moment
Minimal yet complete reproducer code (or URL to code)
Netty version
4.1.5.Final
JVM version (e.g. java -version)
openjdk version "1.8.0_121"
OS version (e.g. uname -a)
Linux gfernandes 4.9.10-100.fc24.x86_64 #1 SMP Wed Feb 15 18:35:50 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
Details about the issue:
We observed a WARN during one of our netty based servers startup:
12:13:08,259 WARNING [io.netty.channel.epoll.EpollEventLoop] (MemcachedServerMaster-1-1) Unexpected exception in the selector loop.: java.lang.NullPointerException
	at io.netty.util.internal.shaded.org.jctools.queues.MpscChunkedArrayQueue.poll(MpscChunkedArrayQueue.java:264)
	at io.netty.util.concurrent.SingleThreadEventExecutor.pollTaskFrom(SingleThreadEventExecutor.java:223)
	at io.netty.util.concurrent.SingleThreadEventExecutor.pollTask(SingleThreadEventExecutor.java:218)
	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:408)
	at io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:306)
	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:873)
	at io.netty.util.concurrent.DefaultThreadFactory$DefaultRunnableDecorator.run(DefaultThreadFactory.java:144)
	at java.lang.Thread.run(Thread.java:745)

Trimmed thread dump:
2017-02-23 13:43:18
Full thread dump OpenJDK 64-Bit Server VM (25.121-b14 mixed mode):

"Attach Listener" #184 daemon prio=9 os_prio=0 tid=0x00007f5fcc003800 nid=0x3749 waiting on condition [0x0000000000000000]
   java.lang.Thread.State: RUNNABLE

"MemcachedServerMaster-1-1" #123 prio=5 os_prio=0 tid=0x00007f5f9c7ba800 nid=0x2dfd runnable [0x00007f5f102e3000]
   java.lang.Thread.State: RUNNABLE
	at io.netty.util.internal.shaded.org.jctools.queues.MpscChunkedArrayQueue.poll(MpscChunkedArrayQueue.java:252)
	at io.netty.util.concurrent.SingleThreadEventExecutor.pollTaskFrom(SingleThreadEventExecutor.java:223)
	at io.netty.util.concurrent.SingleThreadEventExecutor.pollTask(SingleThreadEventExecutor.java:218)
	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:408)
	at io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:306)
	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:873)
	at io.netty.util.concurrent.DefaultThreadFactory$DefaultRunnableDecorator.run(DefaultThreadFactory.java:144)
	at java.lang.Thread.run(Thread.java:745)

"MSC service thread 1-4" #17 prio=5 os_prio=0 tid=0x00007f5fa4001800 nid=0x2d85 in Object.wait() [0x00007f5fe5fb7000]
   java.lang.Thread.State: WAITING (on object monitor)
	at java.lang.Object.wait(Native Method)
	at java.lang.Object.wait(Object.java:502)
	at io.netty.util.concurrent.DefaultPromise.await(DefaultPromise.java:239)
	- locked <0x00000000f82f4920> (a io.netty.bootstrap.AbstractBootstrap$PendingRegistrationPromise)
	at io.netty.channel.DefaultChannelPromise.await(DefaultChannelPromise.java:129)
	at io.netty.channel.DefaultChannelPromise.await(DefaultChannelPromise.java:28)
	at io.netty.util.concurrent.DefaultPromise.sync(DefaultPromise.java:340)
	at io.netty.channel.DefaultChannelPromise.sync(DefaultChannelPromise.java:117)
	at io.netty.channel.DefaultChannelPromise.sync(DefaultChannelPromise.java:28)
	at org.infinispan.server.core.transport.NettyTransport.start(NettyTransport.java:136)
	at org.infinispan.server.core.AbstractProtocolServer.startTransport(AbstractProtocolServer.java:95)
	at org.infinispan.server.core.AbstractProtocolServer.startInternal(AbstractProtocolServer.java:73)
	at org.infinispan.server.memcached.MemcachedServer.startInternal(MemcachedServer.java:52)
	at org.infinispan.server.memcached.MemcachedServer.startInternal(MemcachedServer.java:30)
	at org.infinispan.server.core.AbstractProtocolServer.start(AbstractProtocolServer.java:80)
	at org.infinispan.server.endpoint.subsystem.SecurityActions$6.run(SecurityActions.java:136)
	at org.infinispan.server.endpoint.subsystem.SecurityActions$6.run(SecurityActions.java:133)
	at org.infinispan.security.Security.doPrivileged(Security.java:76)
	at org.infinispan.server.endpoint.subsystem.SecurityActions.doPrivileged(SecurityActions.java:42)
	at org.infinispan.server.endpoint.subsystem.SecurityActions.startProtocolServer(SecurityActions.java:140)
	at org.infinispan.server.endpoint.subsystem.ProtocolServerService.startProtocolServer(ProtocolServerService.java:190)
	at org.infinispan.server.endpoint.subsystem.ProtocolServerService.start(ProtocolServerService.java:148)
	- locked <0x00000000e1b23f88> (a org.infinispan.server.endpoint.subsystem.ProtocolServerService)
	at org.jboss.msc.service.ServiceControllerImpl$StartTask.startService(ServiceControllerImpl.java:1948)
	at org.jboss.msc.service.ServiceControllerImpl$StartTask.run(ServiceControllerImpl.java:1881)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:745)

At this point the process is totally hang.

@gustavonalle can you try upgrade to netty 4.1.8.Final ? We updated the jctools version there.
Also /cc @nitsanw

Will do, thanks @normanmaurer

Please report back

Let me close this for now and please re-open if you still think there is an issue

@gustavonalle @normanmaurer The NPE is very strange:
12:13:08,259 WARNING [io.netty.channel.epoll.EpollEventLoop] (MemcachedServerMaster-1-1) Unexpected exception in the selector loop.: java.lang.NullPointerException
	at io.netty.util.internal.shaded.org.jctools.queues.MpscChunkedArrayQueue.poll(MpscChunkedArrayQueue.java:264)
	at io.netty.util.concurrent.SingleThreadEventExecutor.pollTaskFrom(SingleThreadEventExecutor.java:223)
	at io.netty.util.concurrent.SingleThreadEventExecutor.pollTask(SingleThreadEventExecutor.java:218)
	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:408)
	at io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:306)

Looking at the relevant code for v1.2.1:
https://github.com/JCTools/JCTools/blob/v1.2.1/jctools-core/src/main/java/org/jctools/queues/MpscChunkedArrayQueue.java#L264
        soElement(buffer, offset, null);
        soConsumerIndex(index + 2);
        return (E) e; // <--- 264
    }

Does the shading somehow mangle line numbers?

@nitsanw I thought the same thing when I checked from the jar..

@johnou yep... at this point I don't see a way to make sense of the error. There was an MPSC related issue fixed in the 2.0 release which may be related though:
https://github.com/JCTools/JCTools/issues/135
So worth updating and seeing if it helps.

@gustavonalle you'll want to pull in Netty 4.1.9 when released for the MPSC issue @nitsanw mentioned above, see https://github.com/netty/netty/pull/6399

authors: gustavonalle johnou nitsanw normanmaurer


text:
SelfSignedCertificate configurable valid dates

Motivation:
Users may want to control the valid dates for SelfSignedCertificate.
Modifications:

Allow NOT_BEFORE and NOT_AFTER to be controlled via java system properties.

Result:
Fixes https://github.com/netty/netty/issues/3978

@normanmaurer  @ganskef - FYI.

How about renaming notBefore/After to defaultNotBefore/After and allow a user to specify an alternative notBefore/After value via constructor?

@trustin - Good idea. Done.

Cherry-picked 4.0 (8bc3964) 4.1 (c116c35) master (db0fdfe)

@trustin Any thoughts on this test failure? I've seen it a few times now.

@Scottmitch Some DNS servers were returning NoError for non-existent domains. I removed them from the public DNS server list. Will keep my eyes on the failure..

@trustin @Scottmitch I think it makes sense to use a "stub" DnsServer for our tests. I will work on this and so make it more robust.

@normanmaurer +1. It would be nice to have isolation in our unit tests that automatically run. It would be nice for dev to switch to the "real" DNS servers too though.

authors: Scottmitch normanmaurer trustin


text:
Feat/#41 exception handling

related RFC document: https://github.com/networknt/light-rfcs/commit/f440c47e7b2931995ec7263f6496063106d76416
related issue: https://github.com/networknt/light-router/issues/41
before change: in client module whenever there's an error, throw exceptions, then ExceptionHandler will catch it and respond to the client (lost detail information)
after change: return a com.networknt.monad.Result, the user can handle based on the "Result" is success or failure. "Result" will keep failure information if fails.

now whenever needs to pass auth server's response to the client, now it will escape based on the type that the server returns. @NicholasAzar
https://github.com/networknt/light-4j/pull/380/commits/4b354196cd5b77cf2315e3343088f6e0d776943c

@BalloonWen
{"statusCode":401,"code":"ERR10052","message":"GET_TOKEN_ERROR","description":"Cannot get valid token: "{\r\n  \"error\":\"invalid_client\",\r\n  \"error_description\":\"The given client credentials were not valid\"\r\n}".","severity":"ERROR"}
Json format error is:
"Expected ',' instead of '{'"

Looks better! :
{
"statusCode": 401,
"code": "ERR10052",
"message": "GET_TOKEN_ERROR",
"description": "Cannot get valid token: {\r\n  "error":"invalid_client",\r\n  "error_description":"The given client credentials were not valid"\r\n}.",
"severity": "ERROR"
}

authors: BalloonWen NicholasAzar


text:
Add short circuit paths

Resolves networkservicemesh/cmd-forwarder-vpp#66

@edwarnicke Could you have a look at this patch

Closes networkservicemesh/cmd-forwarder-vpp#63

authors: Tiberivs denis-tingaikin

repo: networkservicemesh/sdk-vpp


text:
Exploratory network data science notebooks.

Notebooks which uses different network science techniques and algorithms to explore and analyse real world datasets.

http://snap.stanford.edu/data/index.html

I would be very interested to contribute to the particular issue and project in general. However, it is my first time contributing to any github project so I would appreciate indulgence, support and feedback. In detail, I have worked with the networkx library for a graph analysis assignment so I already have some experience. To my understanding: you would appreciate to have a graph analysis on a real world dataset (of the above link) containing graph theory algorithms and techniques like degree measures, topological properties, clustering effects, bridges, communitites etc. I hope I can be of some assistance to you.

Hello, I think I will make a notebook to explore and analyse Social circles from Facebook (http://snap.stanford.edu/data/ego-Facebook.html)

repo: networkx/notebooks


text:
Running Adaptive learning on Jetson TX2 and Jetson Nano

I was wonder if I can run the adaptive learning framework on edge devices such as Jetson TX2 and Jetson Nano. Since the teacher model is a really large model and also the student model require training I think it is a really challenging task to be run on low resource edge devices.
First of all I will try to run it on a more powerful device like Jetson TX2 and if the attempt was successful I will try to run it on Jetson Nano.
I will share my journey under this issue.

First I tried to run a large teacher model on Jetson TX2. In my first attempt I chose the TensorFlow NAS Faster RCNN model as my teacher model. To this end first I installed TensorFlow on Jetson TX2 with JetPack 4.3 base on this instruction by Nvidia. then I tried to download and run the NAS RCNN model which I couldn't and the processed have been killed every time. I determined that the model requires more RAM than the Jetson TX2 has.
In my next try I will try to convert the NAS RCNN model to TensorRT engine with TF-TRT tool.

I followed the Nvidia's instuction to convert a TensorFlow Saved Model to a TensorRT optimized version. However I still had RAM problem and the script have been killed. Then I tried to optimize the model on a GPU tower with lots of hardware resources. I sucessfully converted the model but again I had RAM problem when running the converted model on TX2. I think the optimization should be execute on the final target machine.
I think know I will give up in running this NAS RCNN model as the teacher model and try to use IterDet as my teacher model.

So I tried to install and run IterDet on Jetson TX2. First I tried to install Pytorch 1.5.0 which is the backend of the IterDet model. Since the installed JetPack on my device was 4.3 I couldn't use Nvidia's container for running Pytorch on Jetson, also the Pytorch 1.5.0 wheels for Jetson is only available for JetPack 4.4.
So I was forced to use an lower version of IterDet (V1 version) which is compatible with PyTorch 1.3.0.
I started with the Smart Social Distancing docker image for the Jetson which has some necessary libraries and dependencies such as opencv. I followed this article and sucessfully installed PyTorch 1.3.0 and TorchVision 0.4.1 on the Jetson TX2.
Now I needed to install IterDet V1 to run its model. I followed this instruction to install the IterDet. In the middle of installation process I found out that I should install some of the dependencies from the source since the PyPi wheels were incompatible with Jetson devices.
I cloned mmcv (version 0.6.0 was compatible with IterDet V1) and brambox repositories and build these libraries from the source.
At this stage I successfully installed the IterDet dependencies and everything was ready to run the model and getting inference.

For running the IterDet model I wrote an script inspired by the Adaptive Learning Teacher class.
I observed that when I force the model to run on GPU I get this error:
RuntimeError: cuda runtime error (7) : too many resources requested for launch at mmdet/ops/roi_align/src/roi_align_kernel.cu:139
It seems this error is a built-in mmdetection error (mmdetection is the backend library of IterDet). see this issue for more information about this error.
On the other hand this IterDet version that I was using had not CPU support. so inspired by the newer version which has the CPU support I changed some lines in mmcv an mmdetection source code and create a CPU compatible version of the IterDet.
Finally I successfully get an inference from the CrowdHuman checkpoint of the model.
The model runs with just 1GB of RAM and there is no RAM issues with this model. However it is really really slow and for a image with 704x576 resolution it takes around 5:30 minutes to get the inference.
you can check one of the results frames from the SoftBio dataset.

In the next step I should find a way to run this model faster.
stay tuned :)

Before performing some optimizations to the IterDet model I decided to make sure that the TensorFlow Object Detection API which is the training API for the student model can run properly on Jetson TX2, So I installed TensorFlow 1.15.2 with the help of this instruction. Then I followed the installation guide of TensorFlow Object Detection API and installed it without a problem.
Then I ran a mock training to profile the performance of TX2. With batch size equal to 2 it requires 3GB of RAM and each training step takes around 2.5 seconds for a SSD MobileNet V2 model. I am not sure if the model is running on GPU or CPU. I will update this comment after I monitored the GPU.
So I concluded the training is not a problem in Jetson TX2 and the only think I should take care of is running teacher model with more reasonable inference time.

repo: neuralet/neuralet


text:
Add: Sparsification + Quantization Recipes for YOLACT

yolact.pruned.md
yolact.pruned.perf.md
yolact.quantized.md

the YAML recipe section should be surrounded by dashes --- instead of underscores ___
where is the module. coming from in the recipes? is this part of the core yolact model or the DDP wrapper?

Good catch @bfineran; Corrected!

@bfineran can you work with @rahul-tuli to paramaterize these recipes where if a user enters a different number of epochs it will shift the pruning and quantization start and ends appropriately?

@bfineran can you work with @rahul-tuli to paramaterize these recipes where if a user enters a different number of epochs it will shift the pruning and quantization start and ends appropriately?

Done!

authors: markurtz rahul-tuli

repo: neuralmagic/sparseml


text:
Create a template .env file

Something analogous to https://github.com/neurobagel/api/blob/18d5d95ecf8ae6c2ee4b56cbe7a279ef684a8498/.template-env
so I can reference it in the docs and to describe deployment

:rocket: Issue was released in v0.1.0 :rocket:

repo: neurobagel/federation-api


text:
xrdp-sesman becomes  and cannot login.

// Version
OS: Ubuntu 18.04
Version: 0.9.5-2 (Provided by Ubuntu.)
// Playback procedure

login to server.
logoff from server.
login to server.

// Correct behavior
Login succeeded.
// Actual behavior
cannot login.
// Possible cause
Trying to reconnect to the deleted display.

Sounds like https://github.com/neutrinolabs/xrdp/pull/1487 fix in https://github.com/neutrinolabs/xrdp/releases/tag/v0.9.13

Looks like already fixed issue, closing.

authors: derekschrock metalefty tanakakz-alpha

repo: neutrinolabs/xrdp


text:
Xvnc faster (less laggy) than x11rdp/Xorg drivers

Hi,
I have been testing xrdp and it is a great piece of software. In order to make experience better, I built x11rdp binary as well as Xorg drivers (all using RH-Matic).
The problem is that when using Xvnc dekstop reacts almost instantly - there is virtually no lag.
However, when using both x11rdp and Xorg driver the experience is slightly worse. The difference is tiny (we are talking about something like 50ms vs 100ms latency) but you can clearly feel it.
This is quite surprising, as Xorg drivers (or x11rdp - which actually is even slower, but it i based on older source, so I suppose it is understandable) are supposed to be 'native' and provide superior experience, witn no VNC overhead, etc.
Is this normal or am I doing something wrong?
OS: RHEL 7.2, xrdp with drivers/x11rdp build from RH-Matic (tried both master and devel), Xvnc TigerVNC 1.3.1.

When you feel slow, can you see the log like this in Xorg.*.log?
[903647.103] rdpDeferredUpdateCallback: reschedual rect_id 154481 rect_id_ack 154480
[903647.145] rdpDeferredUpdateCallback: reschedual rect_id 154481 rect_id_ack 154480

xorgxrdp and x11rdp are sometimes slow and sometimes fast.  It's slow during above logs are dumped.
I think this may a bug of xorgxrdp/x11rdp.

Correct, this is what I am observing @metalefty
When this slight lag occurs (well, pretty much all the time), I am seeing a log of messages like the following one on the console:
rdpDeferredUpdateCallback: reschedual rect_id 318 rect_id_ack 317
The message is not there when using Xvnc (although I am not sure if this would even be visible in the log, due to different back-end).
Very quick test to replicate it (at least with my setup) is to click 'start' button in KDE 4 (vanilla RHEL 7.2, with "yum install @kde") - the menu appears virtually instantly with Xvnc, but with Xorg there is 200ms or so lag (with "rdpDeferredUpdateCallback" message in the log every time: usually it is just a single entry, but sometimes more - which introduces even more lag).
I would love to use xorgxrdp (better scaling, etc), but currently Xvnc delivers visibly better user experience with my setup. Any idea what the problem could be? I am happy to test this further.

In addition, you see high CPU consumption of xrdp process and high network bandwidth consumption during the slight lag, right?  At least I see that.
I haven't clarified what the actual problem is however increasing log level may help us.  I added the option --xorgxrdpdebug to RH-Matic in develop branch.  Rerun RH-Matic with --xorgxrdpdebug and let's see  ~/.local/share/xorg/Xorg.*.log

Thank you, attaching the log from a short session that has this problem.
Not entirely sure about network/CPU - it generally hoovers around 10-15% at periods of intense activity.
Xorg.17.txt

@metalefty were you able to track this down? We are in the same situation: we would like to use Xorg but cant because users find it unacceptable due to the lag.

Ping for update.  I moved from xrdp 0.6.1-2 + Xvnc to 0.9.1-7 with x11RDP on Ubuntu 16.04 and there's significantly more latency.

Also, I just tested setting /etc/xrdp/xrdp.ini:
autorun=Xvnc
(force Xvnc instead of Xorg, which is the default first option).  Performace is better / back to normal.

How do you turn on Xorg debug logging to Xorg.*.log?

u can use alternative rdp client in windows, like mRemoteNG and set 256 color mode, in this case performace will better

Any update on this? I'm getting poor performance on xorg (though I haven't tried anything else; definitely way worse than native rdp on windows)

I also confirm problem using build-in Windows RPD client. Lag was about 2 seconds. mRemoteNG with 256 color mode ( Appearance bar ) works like a charm. No lags anymore. Thanks a lot @coolemza! #protip

I also confirm problem using build-in Windows 10 RDP clients

is there any workaround that does not require user to recompile from source?
I asked also in neutrinolabs/xorgxrdp but "neutrinolabs locked and limited conversation to collaborators Oct 25, 2018"

video in action (240 fps slow motion)
https://youtu.be/2UzpAEa_UUM

config https://pastebin.com/A0hJ7GdH

using remmina with default settings (no channels enabled on client, connection type "fast") gives instant mouse highlight movement over login dropdown menu, but once logged in is laggy again.
so I think there are two distinct issues, one with first connection screen, second with xrdp.
note that on login window there is no cpu spike, but on session afterwards I get high cpu from both xrdp and Xorg processes.
video https://youtu.be/zFfrmsBUjbQ

In my case on Ubuntu 18.04.04 is very fast (90% form console performance) on xorg X11 and laggy on VNC (tigervnc-standalone-server).
I'm using xrdp-0.9.5-2 x64

In case of Ubuntu Remmina connecting to Ubuntu xrdp use RemoteFX in Colordepth field in properties. In case of GFX RFX, it is laggy probably because its implementation in xrdp (0.9.5) is not final. It is similar in Windows Remote Desktop Client connecting to xrdp, but here I don't know how to force it. But if I use the highest bandwidth in the menu, the performance is OK so it is probably not using the GFX RFX variant. Just to mention, if I am not doing it this way, than the clients are always taking 2Mbit/s of network traffic even if they are minimized.

In case of Ubuntu Remmina connecting to Ubuntu xrdp use RemoteFX in Colordepth field in properties. In case of GFX RFX, it is laggy probably because its implementation in xrdp (0.9.5) is not final. It is similar in Windows Remote Desktop Client connecting to xrdp, but here I don't know how to force it. But if I use the highest bandwidth in the menu, the performance is OK so it is probably not using the GFX RFX variant. Just to mention, if I am not doing it this way, than the clients are always taking 2Mbit/s of network traffic even if they are minimized.

Also, I just tested setting /etc/xrdp/xrdp.ini:
autorun=Xvnc
(force Xvnc instead of Xorg, which is the default first option). Performace is better.

Truly a hidden gem

Note: using Xvnc is not a silver bullet, Xvnc is not always faster than Xorg driver. Sometimes Xorg driver is faster, sometimes Xvnc is faster.  It depends on the graphic type to show. The only fact is Xvnc is faster FOR YOU. Xorg driver might be faster for someone else.

Let's turn back to the original topic, "Xvnc faster (less laggy) than x11rdp/Xorg drivers"It depends on the use case but some causes that Xorg driver is slower are already fied, for example in neutrinolabs/xorgxrdp#94.
Any further reports should be posted with using the latest version. Not with outdated versions.

Xvnc (tigervnc-server) has an annoying bug - it does not support Input Language change in GNOME.
It ignores the selection of the input language and keyboard always prints in English.
Affected distros are RHEL, Alma, Fedora, CentOS, Rocky. For some reason they have [Xorg] session type commented out in xrdp.ini.
Ubuntu 22.04 and Debian 11 work fine - they have both [Xorg] and [Xvnc] sessions types enabled in xrdp.ini and [Xorg] is the default
Related Links and The Solution to the issue:

[XRDP] Keyboard always types in English regardless of selected Input Language
CentOS7 | xRDP on Xorg: VNC not supporting language switching

authors: Bartolomejka apivovarov bolkedebruin coolemza crazy-blazer johnarnold kaysond mailinglists35 metalefty myset nktl overpod wsdookadr


text:
[Imp] REFL #8 Task #65: Implement dialog box to confirm summing runs

This is the implementation of https://code.ornl.gov/sns-hfir-scse/reflectometry/reflectometry/-/issues/65.

TDD implementation for a customized dialog box such that user can choose to accept or reject.
Travis setup is updated for pytest-qt

The current branch has been merged back to feature branch MR8_Merge_Files.  And thus this PR is closed without merge.