No L2 Construct for EKS

Right now in order to create a workable EKS Cluster with worker nodes, one has to fall back to L1 cloudformation construct. This makes it difficult to use CDK for purposes of setting up any type of managed kubernetes cluster.
The user must:

Use VpcNetwork to create the network
Use cloudformation construct to deploy the cluster
Manually create autoscaling groups
Properly tag autoscaling group instances
Create user data to match the bootstrap script
Create a role
Attach managed policies to the role
Apply securityGroup rules by following the Getting Started Guide

As the nature of EKS clusters and more so worker nodes are probably added and removed from existence quite often, especially during dev and testing.
Design Case
The user should be able to create a cluster, by passing in some properties. The only required properties in the cloudformation construct are the VPC and the VpcPlacement, such as.
const cluster = new eks.Cluster(clusterStack, "Cluster", {
  vpc: clusterVpc,
  vpcPlacement: {
    subnetsToUse: ec2.SubnetType.Public
  }
});

Which would then create

A security group to attach to the cluster
The correct role for the cluster

with provided managed policies by AWS

Make certain properties public for access during node creation

the cluster name
the cluster endpoint
others for future reference

Be able to import and export available for future multi-stack deployments

The user can then add worker nodes on demand of any of the supported classes and sizes. The only required inputs would be the VPC to and which Cluster (control plane) to use for the nodes. Optional properties can be added to scale the autoscaling group, provide ssh keypair if SSH access to the nodes is required.
These options can be added in future releases to make on-par with autoscaling group options as well, but provide an abstraction to those.
The nodes should automatically

Tag the instances with required EKS tags
Create the node role and instance profile

with correct policies

Automatically add the userdata for startup script and cluster discovery
Add recommended security group rules to cluster and nodes based on the getting started documentation
Track an array of node autoscaling groups created

Create Worker Nodes (default values)
const grp1 = new eks.Nodes(clusterStack, "NodeGroup1", {
  vpc: clusterVpc,
  cluster
});

Create Worker Nodes (custom values)
const grp2 = new eks.Nodes(clusterStack, "NodeGroup2", {
  vpc: clusterVpc,
  cluster,
  nodeClass: ec2.InstanceClass.T2,
  nodeSize: ec2.InstanceSize.Medium,
  nodeType: eks.NodeType.Normal,
  minNodes: 2,
  maxNodes: 4,
  sshKeyName: "aws-dev-key"
});

I'll attempt to create a PR for this, feedback is welcome, will be my first contribution here if accepted.
Otherwise, I'll learn from the review :)

I’d be happy to contribute / help review this

Some quick feedback. This structure is missing info on T3 insances, which leads to max being undefined in the instance user-data. Seems to prevent nodes from registering.
export const maxPods = Object.freeze(

#!/bin/bash
set -o xtrace
/etc/eks/bootstrap.sh stargatedemoclusterClusterA-oChms63INSG8 --use-max-pods undefined

Probably need some typed checking on the hash lookup to prevent misses like this from sneaking through.

I also ended up adding a securityGroups option to IClusterProps so we could use some imported security groups.
  securityGroups?: [ec2.SecurityGroupRef]

    const securityGroupIds = new Array(this.securityGroupId)
    if (props.securityGroups) {
      for (var sg of props.securityGroups) {
        securityGroupIds.push(sg.securityGroupId)
      }
    }

We released an EKS library in 0.25.0


OAuth Error: Invalid client_id parameter

I followed this tutorial https://github.com/howdyai/botkit/blob/master/docs/provisioning/slack-events-api.md#3-configure-oauth and created a Botkit slack bot as well as a bot user on slack. After following all the steps, while accessing https://bot_url/, on clicking the 'Add to Slack' button, I am getting an invalid client_id parameter error. Any solutions?

I am having the same error. How did you resolve it?

So, I fixed the problem by following the documentation and watching the video by @benbrown on this page:
https://botkit.ai/docs/provisioning/slack-events-api.html#3-configure-oauth


Small update

added pandas as a required import
small bug fixes
fixed logo

Codecov Report

Merging #16 (d685dca) into main (baf8205) will decrease coverage by 0.18%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
- Coverage   14.93%   14.74%   -0.19%
==========================================
  Files           8        8
  Lines        1018     1017       -1
==========================================
- Hits          152      150       -2
- Misses        866      867       +1

Impacted Files
Coverage Δ

bigplanet/bp_extract.py
36.48% <0.00%> (-0.74%)
:arrow_down:

bigplanet/bp_get.py
6.35% <0.00%> (ø)

Continue to review full report at Codecov.

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


example: build fail

This PR is not labeled with ok-to-test and author of PR is not collaborator or owner of repository. You need to wait untill collaborator or owner of repository will label this PR with ok-to-test.


linux: add missing netfilter definitions

Bring libc up to parity with Linux v5.18 headers wrt. to Netfilter.
libc-test fails due to unrelated issues on my Fedora box but the
changes should be inocuous enough IMO.
Note there’s a warning in the code indicating potential trouble due to this
issue. I’ll sort this out if
the CI run fails.

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

The “Docker Linux Tier1” CI run fails like so:
cargo:warning=/checkout/target/i686-unknown-linux-gnu/debug/build/libc-test-4eae72712e4eb4b5/out/main.c:30472:66: error: 'NFNL_SUBSYS_HOOK' undeclared here (not in a function); did you mean 'NFNL_SUBSYS_ULOG'?

So the caveat about musl relying on pre-historic kernel headers
still seems to apply. Could this change be made a compile-time
conditional? To it sounds rather suboptimal to let one specific
target hold back the crate like this.
Let’s see about those other changes.

You can add exceptions to the tests in libc-test/build.rs. You will see many other cases where exceptions are added due to old kernel headers in CI. Make sure to mark the kernel version the constants were introduced in so we can remove the exceptions when upgrading the CI images.

2022-08-16 (Tuesday), @.*** (Amanieu):

You can add exceptions to the tests in libc-test/build.rs.
You will see many other cases where exceptions are added due to
old kernel headers in CI. Make sure to mark the kernel version
the constants were introduced in so we can remove the exceptions
when upgrading the CI images.

Thanks for the pointer, I’ll give it a shot and resubmit!

@bors r+

:pushpin: Commit 8c194e6174f4c52953d2c682c3eeaca4782d8872 has been approved by Amanieu
It is now in the queue for this repository.

:hourglass: Testing commit 8c194e6174f4c52953d2c682c3eeaca4782d8872 with merge a7e6facd378e9a571bb7f1aefcd4775719a3c220...

:broken_heart: Test failed - checks-actions

2022-08-20 (Saturday), @.*** (bors):

:broken_heart: Test failed - checks-actions

Sigh. Those were introduced with v5.4. Confusingly only
mips-unknown-linux-gnu seems to be affected by this but not
other platforms with older kernels.
Exceptions for those NFULA_* constants added.

S-waiting-on-author

@JohnTitor could you clarify what you’re waiting for? I updated the pr to address
the issues that became apparent during the bors run.

Oh, missed you pushed a commit after the failure, @bors r+

:pushpin: Commit 12f3b38b104e3bd2b49df7d5aea9bc29d6f7d2a7 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 12f3b38b104e3bd2b49df7d5aea9bc29d6f7d2a7 with merge af5de5015907643c8ddba62dc33ec561e583ae40...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing af5de5015907643c8ddba62dc33ec561e583ae40 to master...

Oh, missed you pushed a commit after the failure, @bors r+

No worries, thank you!

@@ -2101,15 +2102,31 @@ pub const NFNL_SUBSYS_CTNETLINK_TIMEOUT: ::c_int = 8;
pub const NFNL_SUBSYS_CTHELPER: ::c_int = 9;
pub const NFNL_SUBSYS_NFTABLES: ::c_int = 10;
pub const NFNL_SUBSYS_NFT_COMPAT: ::c_int = 11;
-pub const NFNL_SUBSYS_COUNT: ::c_int = 12;
+cfg_if! {

if #[cfg(target_env = "musl")] {

   // FIXME: CI kernel headers lag bit behind;

   // NFNL_SUBSYS_HOOK was added with v5.14.

   pub const NFNL_SUBSYS_COUNT: ::c_int = 12;

} else {

   pub const NFNL_SUBSYS_HOOK: ::c_int = 12;

   pub const NFNL_SUBSYS_COUNT: ::c_int = 13;

}
+}

The exception in build.rs should apply to both glibc and musl, so you shouldn't need to add this cfg_if.

I tried that initially but without the cfg_if the
x86_64-unknown-linux-gnu run fails because of the value
mismatch for NFNL_SUBSYS_COUNT.


A service with Topo_RD projection the error 404 appears.

Following the documentation from Esri: https://developers.arcgis.com/openlayers/layers/add-a-vector-tile-layer. When you make a service with Topo_RD projection the error 404 appears, when trying to add the vector tile service into OpenLayers.
Please check this documentation https://openlayers.org/en/latest/examples/vector-layer.html.
Kindly check why this error appears.

Hello,
A 404 error usually appears when a network service is not available. This is nothing which can be fixed on OpenLayers side and you need to check the network services that you request.
On the other hand we sadly can not guarantee that HowTo from esri will work and you would have to reach out to esri if there is something else wrong with it. But for this problem you need to check if the service is available in your network.

Topo_RD is an EPSG:28992 raster tile source, not a vector tile source.  You could use its WMTSCapabilities https://services.arcgisonline.nl/arcgis/rest/services/Basiskaarten/Topo/MapServer/WMTS/1.0.0/WMTSCapabilities.xml to set it up in the same way as the ESRI hosted EPSG:27700 source used in https://openlayers.org/en/latest/examples/print-to-scale.html and https://openlayers.org/en/latest/examples/reprojection.html


scannet datasets

Can you share the following scannet datasets?

Sorry that we can not share it. Users should sign the ScanNet Terms of Use before getting access to the dataset. Please download the data from the official website.


Nice library

This is the best c++ implementation of JSON I have seen. Good job! Very easy to use and very intuitive. Thanks!

Thanks! If you like it, you may also like https://github.com/sponsors/nlohmann ;-)


Method put is not working.

method put is not working.
It can't get data from serverside.

Ideally:  PUT is for putting files on the server.  GET is for getting data from the server.


Why columns in WHERE clause are not included in desiredColumns?

Hello, my presto version is 0.202 and the picture below is the relative code (ConnectorMetadata::getTableLayouts).

For example, the sql is "select sum(a) from metric where b > 1 group by c order by sum(d)".
Column a、c、d are in the desiredColumns .
But why the column b are not in the desiredColumns？

why the column b are not in the desiredColumns？

It's because it need not be returned to the engine.
Consider a case when connector has some means of identified which pieces of data satisfy constraint without materializing b values.
For example, Hive connector can identify Hive partitions which match the predicate. But it doesn't need to return the partitioning column, since it's not going to be used later on.
(I am assuming this answers your question, so let me close the issue.)
BTW table layouts are deprecated since Presto 306 and we're migrating existing connectors not to use them.
We now use methods like ConnectorMetadata#applyFilter to pushdown constraint into a connector.
Please join the #dev channel on our slack https://prestosql.io/slack.html.
See you there!

@findepi  Thanks for your reply！But i still have some problems.
I think desiredColumns are the columns that presto need to get from datasource, is this right?
If it is right, the unenforcedConstraints may contain some columns that are not in desiredColumns, so the filtered result may not be right？

(continued on slack @ https://prestosql.slack.com/archives/CFLB9AMBN/p1575343012109600)


Bump dcos-diagnostics

High Level Description
This updates dcos-diagnostics to the latest version.
Related Issues

DCOS_OSS-1345 "dcos-diagnostics check --list" uses wrong key name for check command

Checklist for all PR's

[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: tests in component repo
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): changelog
[x] Test Results: link to CI job test results for component
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label ready-for-review

@darkonie @amitaekbote Ready for review.

@mesosphere-mergebot label shipit


Doc description is not updated

Hi,  I have updated my extension doc description on my Quarkus extension YAML file, but I don´t see any effect on the website.
I would expect the following sites updated, according to these changes:

https://quarkus.io/extensions/io.quarkiverse.hivemqclient/quarkus-hivemq-client/
https://quarkus.io/extensions/?search-regex=hivemq

Instead of Quarkus - Hivemq Client Implementation of MQQ Messaging I would expect Transform your business with the most trusted MQTT platform.
The changes were made following this documentation: https://quarkus.io/guides/extension-metadata#quarkus-extension-yaml

Description that can be displayed to users. In this case, the description was copied from the pom.xml of the extension module but it could also be provided in the template file.

So ok... looks like could be also on the pom description field but, based on the docs looks like is also possible to have this extension description on the quarkus-extension-yaml file
WDYT ?

It looks like the description is now showing as intended on https://quarkus.io/extensions/io.quarkiverse.hivemqclient/quarkus-hivemq-client/:

I suspect the original delay happened because we show the description of the latest release, rather than reading it from the GitHub source code. (Some other things we do read dynamically, like issue counts, but the metadata, including description, is intended to be specific to a release.)


Support More Complex Data Types

Support for ROW, MAP, ÀRRAY and MULTISET

I am working on a solution that supports ARRAY, ROW and MAP data types. For arrays and maps may be generated with a query specifying size and expression types (i.e. superhero name). For a row, the row fields can be specified in the query. The current prototype still needs a few fixes, but I will finish it soon.

I am working on a solution that supports ARRAY, ROW and MAP data types. I will open a PR soon.


Improvements to arguments and finetuning

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


feat adds beforeLabel and afterLabel props

What kind of change does this PR introduce?
Feature - resolves https://github.com/supabase/ui/issues/16
What is the current behavior?
None
What is the new behavior?
beforeLabel and afterLabel props added to all inputs

🔥  nice!

:tada: This PR is included in version 0.27.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fmsynth: init at 2015-02-07

Motivation for this change
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@FRidh Done.


external CA roles based on SPIFFE SVID

There's one set of default roles added to all auto-enrolled identities from an external CA. This means I need a separate CA for each distinct set of identity roles.
A hypothetical feature of Ziti external CAs is interpreting an SVID as identity role(s).
A simplistic implementation could treat a particular position in the SVID as a Ziti identity role. A more sophisticated implementation could introduce a new type of identity role which is an SVID match pattern.

Found this relevant doc: https://docs.openziti.io/docs/learn/core-concepts/security/authentication/third-party-cas/#external-id--x509-claims

Similar requests have come up for roles based on OIDC claims property that we will implement, will revisit this if future external demand


Improve modeling token nodes

With @ahoppen we identified potential confusion and overcomplication in code relating to modeling token nodes. Specifically, we're currently trying to collapse multiple token kinds into a single one without indicating this in our variable names.

The only non-trivial place that needs to be updated is Child.type in SyntaxBuildableChild and, IMHO, the confusion stems from the fact that we try to collapse multiple token kinds into a single one without giving any indication of doing so in the variable names.
I think the way to go about fixing this is to change the associated value of SyntaxOrTokenNodeKind.token to [TokenChoice]. Then we won’t need to call Child.tokenKind in Child.type anymore and could shift the computation that’s currently done in this property down to uses where there’s a lot more context. Potentially, the computation might also just become a lot simpler once we see that context.

Originally posted by @ahoppen in https://github.com/apple/swift-syntax/pull/2006#discussion_r1285082312
In relation to issue #1978, changes were made in PR #2006 to the Child.tokenKind implementation. However, this solution still appears to be incorrect and needs further attention, which this issue aims to address.

Tracked in Apple’s issue tracker as rdar://113567645


Add dark theme setting for settings screen

Kind (Required)

Improvement

Overview (Required)

The night mode design has already been implemented.
Last year's app may be helpful for implementing switching.
The design of the setting is similar to last year's design.

Links

https://github.com/DroidKaigi/conference-app-2020/pull/526/files

🙋♂

@ocuto Thank you, give it a shot!


Could not initialize NNPACK! Reason: Unsupported hardware

‘’‘
INFO:opentts:Synthesizing with coqui-tts:zh_baker (3 char(s))...

Using model: tacotron2
Model's reduction rate r is set to: 2
Vocoder Model: fullband_melgan
Generator Model: fullband_melgan_generator
Discriminator Model: melgan_multiscale_discriminator
INFO:opentts:Synthesizing with coqui-tts:zh_baker (9 char(s))...
Text splitted to sentences.
Text splitted to sentences.
['门开了.']
['开锁失败请再试一次.']
Building prefix dict from the default dictionary ...
DEBUG:jieba:Building prefix dict from the default dictionary ...
Loading model from cache /tmp/jieba.cache
DEBUG:jieba:Loading model from cache /tmp/jieba.cache
Loading model cost 0.891 seconds.
DEBUG:jieba:Loading model cost 0.891 seconds.
Prefix dict has been built successfully.
DEBUG:jieba:Prefix dict has been built successfully.
[W NNPACK.cpp:80] Could not initialize NNPACK! Reason: Unsupported hardware.
ERROR:opentts:Sizes of tensors must match except in dimension 1. Got 15 and 39 in dimension 2 (The offending index is 1)
Traceback (most recent call last):
File "/home/opentts/app/.venv/lib/python3.9/site-packages/quart/app.py", line 1490, in full_dispatch_request
result = await self.dispatch_request(request_context)
File "/home/opentts/app/.venv/lib/python3.9/site-packages/quart/app.py", line 1536, in dispatch_request
return await self.ensure_async(handler)(**request_.view_args)
File "/home/opentts/app/app.py", line 718, in app_say
wav_bytes = await text_to_wav(
File "/home/opentts/app/app.py", line 368, in text_to_wav
wavs = [result async for result in wavs_gen]
File "/home/opentts/app/app.py", line 368, in
wavs = [result async for result in wavs_gen]
File "/home/opentts/app/app.py", line 469, in text_to_wavs
line_wav_bytes = await tts.say(line, voice_id, **say_args)
File "/home/opentts/app/tts.py", line 1716, in say
audio = await loop.run_in_executor(
File "/usr/lib/python3.9/concurrent/futures/thread.py", line 52, in run
result = self.fn(*self.args, **self.kwargs)
File "/home/opentts/app/TTS/utils/synthesizer.py", line 303, in tts
outputs = synthesis(
File "/home/opentts/app/TTS/tts/utils/synthesis.py", line 271, in synthesis
outputs = run_model_torch(
File "/home/opentts/app/TTS/tts/utils/synthesis.py", line 100, in run_model_torch
outputs = _func(
File "/home/opentts/app/.venv/lib/python3.9/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "/home/opentts/app/TTS/tts/models/tacotron2.py", line 229, in inference
decoder_outputs, alignments, stop_tokens = self.decoder.inference(
File "/home/opentts/app/TTS/tts/layers/tacotron/tacotron2.py", line 397, in inference
decoder_output, alignment, stop_token = self.decode(memory)
File "/home/opentts/app/TTS/tts/layers/tacotron/tacotron2.py", line 314, in decode
self.context = self.attention(
File "/home/opentts/app/.venv/lib/python3.9/site-packages/torch/nn/modules/module.py", line 889, in call_impl
result = self.forward(*input, **kwargs)
File "/home/opentts/app/TTS/tts/layers/tacotron/attentions.py", line 322, in forward
attention, _ = self.get_location_attention(query, processed_inputs)
File "/home/opentts/app/TTS/tts/layers/tacotron/attentions.py", line 252, in get_location_attention
attention_cat = torch.cat(
RuntimeError: Sizes of tensors must match except in dimension 1. Got 15 and 39 in dimension 2 (The offending index is 1)
ERROR:opentts:Sizes of tensors must match except in dimension 1. Got 15 and 39 in dimension 2 (The offending index is 1)
Traceback (most recent call last):
File "/home/opentts/app/.venv/lib/python3.9/site-packages/quart/app.py", line 1490, in full_dispatch_request
result = await self.dispatch_request(request_context)
File "/home/opentts/app/.venv/lib/python3.9/site-packages/quart/app.py", line 1536, in dispatch_request
return await self.ensure_async(handler)(**request.view_args)
File "/home/opentts/app/app.py", line 718, in app_say
wav_bytes = await text_to_wav(
File "/home/opentts/app/app.py", line 368, in text_to_wav
wavs = [result async for result in wavs_gen]
File "/home/opentts/app/app.py", line 368, in
wavs = [result async for result in wavs_gen]
File "/home/opentts/app/app.py", line 469, in text_to_wavs
line_wav_bytes = await tts.say(line, voice_id, **say_args)
File "/home/opentts/app/tts.py", line 1716, in say
audio = await loop.run_in_executor(
File "/usr/lib/python3.9/concurrent/futures/thread.py", line 52, in run
result = self.fn(*self.args, **self.kwargs)
File "/home/opentts/app/TTS/utils/synthesizer.py", line 303, in tts
outputs = synthesis(
File "/home/opentts/app/TTS/tts/utils/synthesis.py", line 271, in synthesis
outputs = run_model_torch(
File "/home/opentts/app/TTS/tts/utils/synthesis.py", line 100, in run_model_torch
outputs = _func(
File "/home/opentts/app/.venv/lib/python3.9/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "/home/opentts/app/TTS/tts/models/tacotron2.py", line 229, in inference
decoder_outputs, alignments, stop_tokens = self.decoder.inference(
File "/home/opentts/app/TTS/tts/layers/tacotron/tacotron2.py", line 397, in inference
decoder_output, alignment, stop_token = self.decode(memory)
File "/home/opentts/app/TTS/tts/layers/tacotron/tacotron2.py", line 314, in decode
self.context = self.attention(
File "/home/opentts/app/.venv/lib/python3.9/site-packages/torch/nn/modules/module.py", line 889, in _call_impl
result = self.forward(*input, **kwargs)
File "/home/opentts/app/TTS/tts/layers/tacotron/attentions.py", line 322, in forward
attention, _ = self.get_location_attention(query, processed_inputs)
File "/home/opentts/app/TTS/tts/layers/tacotron/attentions.py", line 252, in get_location_attention
attention_cat = torch.cat(
RuntimeError: Sizes of tensors must match except in dimension 1. Got 15 and 39 in dimension 2 (The offending index is 1)
’‘’

Are you trying to run this on an arm64 system?


chef_gem_compile_time's nil is the same as true

according to lib/chef/resource/chef_gem.rb#after_created, true and nil are the same.
if compile_time || compile_time.nil?
  Array(action).each do |action|
    self.run_action(action)
  end
  Gem.clear_paths
end

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@cl-lab-k

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

:+1: this is just a typo in a comment
According to https://www.chef.io/blog/2015/02/17/chef-12-1-0-chef_gem-resource-warnings/

nil: this is the current default and results in all the spammy warnings
false: this will be the future default, it removes ALL warnings, but will flip the behavior of chef_gem and some recipes may fail
true: this maintains the current default behavior, suppresses the individual warnings, but WARNs once per chef run that this is set to true

:+1: merging


v6.0.0 has ContainerInsights log analytics solution being removed, which results in tables needed by current Container Insights deployments to be deleted

Community Note

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

Versions

terraform:
1.8.5
azure provider:
3.111.0
module:
6.0.0
Description
Describe the bug
With v.6.0.0, a slew of log analytics solutions are marked for removal.   Some of these are understood to be deprecated and that's fine, but the 'containerinsights' one in particular is concerning us.  This seems to indicate that, while prometheus managed service is now an option that could replace it, ContainerInsights is still a valid and supported option: https://learn.microsoft.com/en-us/azure/azure-monitor/containers/container-insights-overview
We followed Microsoft's guidance to transition from the old "Container Monitoring Solution" to Container Insights, and now we are concerned that this recent change in the CAF terraform might break our AKS container insights setup if we allow it to remove the "ContainerInsights" LA solution (which results in removal of a bunch of required tables)
I'm hoping someone can clarify this for us :)
Steps to Reproduce
n/a
Screenshots
n/a
Additional context

Btw, I do have a 'Plan B' here in case the behaviour here is deemed correct by design.  I could simply remove those resources from the Terraform back end state and then they would be ignored going forward.
But clarification would still be helpful I think, not just for myself but potentially for others as well who are wondering the same thing.

Update:  We have the same question for the 'VMInsights' solution

We plan to re-instate VM and container insights in the next release. Apologies!

@matt-FFFFFF No problem , thanks for the information and all your efforts :)


Duplicate Flow Logs

The notification listeners are being registered multiple times in some scenarios and then creating duplicate flow start/end logs.

@adesjardin Just asking but do we have any more information on how we can duplicate this error?

I don't think we know the specific way to reproduce it, but I think I can show you a few examples in running apps where we are seeing it and we can take a look at the code and see if we can quickly identify a possible root cause

@adesjardin Okay that would be great. Can you point me to those and/or provide the raw logs to include in this?


fix: 🐛 Use unique portfolios while creating BtreeSet

Description
When an instruction had two or more legs with sender portfolio as same, the creation of BtreeSet of portfolioIds was failing because of duplicate items in the list. This fixes the issue by only using unique items in the list.
Breaking Changes
NA
JIRA Link
DA-1392
Checklist

[ ] Updated the Readme.md (if required) ?

/fast-forward

:tada: This PR is included in version 26.2.1-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 27.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat!: implement adapters

Please describe the changes this PR makes and why it should be merged:
Resolves #64 by implementing an adapter pattern. This removes @discordjs/voice's dependency on Discord.js - it can now be used anywhere (that supports JavaScript). All you need is a connection to the Discord gateway and some "glue" (discussed below).
For example, this enables running @discordjs/voice:

With alternative Node.js Discord API libraries (e.g. Cordis or Eris)
In a distributed architecture (making it scalable!)

How the adapter works
When joining a voice channel, you must also specify an adapter:
await joinVoiceChannel({
  channelId,
  guildId,
  adapter
});

adapter is a function that allows the library and adapter implementer to share methods with each other:
const adapter = ({ onVoiceServerUpdate, onVoiceStateUpdate }) => {
  // Adapter lifecycle begins
  return {
    sendPayload(payload) {},
    destroy() {
      // Adapter lifecycle ends
    }
  }
});

An overview of the methods here:

onVoiceServerUpdate - the implementer must call this whenever a VOICE_SERVER_UPDATE packet is received that is relevant to the adapter (i.e. for the guild that the adapter was created for).
onVoiceStateUpdate - the implementer must call this whenever a VOICE_STATE_UPDATE packet is received that is relevant to the adapter.
sendPayload - the library will call this with a payload that it wants forwarding to the gateway connection of the adapter.
destroy - the library will call this when the adapter can no longer be used since its voice connection has been destroyed.

Some pitfalls

The implementer should only do setup logic (e.g. adding event listeners to a gateway, acknowledging that the adapter is valid by adding it to a list) once the adapter function is called (where the lifecycle begins). There is one case where the adapter function is not called - when you are trying to establish a voice connection to a guild that already has one. To prevent memory leaks and needless event listeners, make sure that you only call your setup logic within the function itself.
The destroy method is technically optional since it doesn't need to exist for all use-cases. However, most use-cases will require it. For example, if you're tracking your adapters in a list somewhere, you could remove the adapter from the list when this method is called.

Provided Discord.js adapter example
This PR includes an example for a Discord.js adapter. The plan is to have Discord.js provide this adapter itself. However, it does not currently do this, so as a stopgap, I've included one here as an example.
Status and versioning classification:

Code changes have been tested against the Discord API, or there are no code changes
This PR changes the library's interface (methods or parameters added)
This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

Values representing what would be a Snowflake should use Snowflake from discord-api-types (eg: guildId, channelId)

IMO this is out of the scope of this PR


Minor typo fix

Thanks for felte, just getting started, but it looks awesome so far. 🧡

Codecov Report

Merging #222 (5a0e994) into main (ae7e7cb) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main     #222   +/-   ##
=======================================
  Coverage   99.57%   99.57%
=======================================
  Files          97       97
  Lines        6794     6794
  Branches     1318     1318
=======================================
  Hits         6765     6765
  Misses         28       28
  Partials        1        1

Continue to review full report at Codecov.

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


Hi,
Im using NodeMailer with NodeEmailTemplates to send emails. I used EJS for templates.
When i update to latest version, my "body" css style (background-color) defined in style.ejs not working. When i check email source, there is no "style" & "body" tag inline in the email.
Is there any configuration i can set to enable the inline?

Thanks for the report. Can you send some sample templates so I can include this case on the test suite?

On feb 26, 2015, at 8:58 PM, chongwang87<EMAIL_ADDRESS>wrote:
Hi,
Im using NodeMailer with NodeEmailTemplates to send emails. I used EJS for templates.
When i update to latest version, my
css style (background-color) defined in style.ejs not working. When i check email source, there is no &  tag inline in the email. Is there any configuration i can set to enable the inline?
—
Reply to this email directly or view it on GitHub.

@chongwang87 See his comment above, please respond

example i have an email template folder name account-new
here are my html.ejs
<% include ../email-header %>

Hi <%- message.user.firstname + ' ' + message.user.lastname %>,

In order to get started, you need to confirm your email address by click on the button below.

<a href="http://www.keepital.com/account/verify/<%- message.token %>" class="btn btn-primary">Confirm My Email

If above button not working, please paste below link into your browser.
http://www.keepital.com/account/verify/<%- message.token %>

We may need to send you critical information about our service and it is important that we have an accurate email address.

If you did not request this, please ignore this email and your password will remain unchanged.

<% include ../email-footer %>

and below are my style.ejs
<% include ../styles/style-master %>

inside style-master.ejs
//normal stuff like
@font-face { },
body { },
.classes { },
#ids { }

when email sent, the styles inline directly to the html element but CSS styles from style-master.ejs doesn't include in the email. eg. body styles
below are the raw source from email
Return-path<EMAIL_ADDRESS>Envelope-to<EMAIL_ADDRESS>Delivery-date: Tue, 10 Mar 2015 08:48:22 +0800
Received: from [<IP_ADDRESS>] (port=60229 helo=[<IP_ADDRESS>])
	by yiling.noc40.com with esmtp (Exim 4.85)
	(envelope-from<EMAIL_ADDRESS>	id 1YV8LO-0006kq-40
	for<EMAIL_ADDRESS>Tue, 10 Mar 2015 08:48:22 +0800
Received: from localhost (<IP_ADDRESS>) by keepital with SMTP; Tue Mar 10 2015
 08:48:53 GMT+0800 (SGT)
Content-Type: multipart/alternative;
 boundary="----sinikael-?=_1-14259485334240.01832712348550558"
From: KEEPITAL Admin<EMAIL_ADDRESS>To: Koo Chong Wang<EMAIL_ADDRESS>Subject: KEEPITAL Password Reset
X-Mailer: nodemailer (1.3.1; +http://www.nodemailer.com; SMTP
 (direct)/1.0.1[client:1.1.0])
Date: Tue, 10 Mar 2015 00:48:53 +0000
Message-Id<EMAIL_ADDRESS>MIME-Version: 1.0

------sinikael-?=_1-14259485334240.01832712348550558
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

http://www.keepital.comYou are receiving this because you (or someone else)=
 have requested the reset of
the password for your account. Please click on=
 the following button to complete
the process:

Reset Password

If above button not working, please paste below link into your =
browser.

If you did not request this, please ignore this email and =
your password will
remain unchanged.

Sent with =E2=99=A5 from KEEPITAL =
[http://www.keepital.com]

=C2=A92014 Keep Pte Ltd, 20 Shaw Rd, #02-08A =
Ching Shine Ind Bldg, Singapore 367956

Please note that this message was =
intended for Koo Chong Wang<EMAIL_ADDRESS>------sinikael-?=_1-14259485334240.01832712348550558
Content-Type: text/html; charset=utf-8
Content-Transfer-Encoding: quoted-printable

You are receiving this because you (or someone =
else) have requested the reset of the password for your account. Please =
click on the following button to complete the process:

<a href=3D"http://www.keepital=
.com/account/reset/42902e19d7995859b65b3691821d9281b6a7ae36" class=3D"btn =
btn-primary" style=3D"background-color: #04b9ed; background-image: none; =
border: 0; border-color: #04a5d4; box-sizing: border-box; color: #ffffff; =
cursor: pointer; display: block; font-family: 'Roboto', 'Helvetica Neue', =
Helvetica, Arial , sans-serif; font-size: 1em; font-weight: 500; =
line-height: 1.3; margin: 0; margin-bottom: 0; padding: 10px; text-align: =
center; text-decoration: none; vertical-align: middle; white-space: =
nowrap;">Reset Password

If you did not request this, please ignore this email and =
your password will remain unchanged.

Sent with ♥ from KEEPITAL

=C2=A92014 Keep Pte Ltd, 20 Shaw Rd, #02-08A Ching Shine=
 Ind Bldg, Singapore 367956

Please note that this message was intended for <a =
<EMAIL_ADDRESS>style=3D"box-sizing: border-box; =
color: #333; font-family: 'Roboto', 'Helvetica Neue', Helvetica, Arial , =
sans-serif; font-size: 11px; margin: 0; padding: 0; text-decoration: =
none;">Koo Chong Wang

------sinikael-?=_1-14259485334240.01832712348550558--

What does your email-header look like?
Note I have been having issues with @font-face and @media rules due to juice stripping them out, maybe this is some side-effect of the same issue.

it just normal html code
email-header.ejs

email-footer.ejs

Sent with ♥ from KEEPITAL

©2014 Keep Pte Ltd, 20 Shaw Rd, #02-08A Ching Shine Ind Bldg, Singapore 367956

Please note that this message was intended for <a href="mailto:<%- receiver.email %>"><%- ( typeof receiver.name !== 'undefined' && receiver.name !== null ) ? receiver.name : receiver.firstname + ' ' + receiver.lastname %>

Ok, so as far as I can see this totally makes sense: Your html.ejs first includes the header, which starts with
. Therefore, your result also starts with
. You need to add the doctype, html and body tags to either of those files yourself for them to show up in the result.

If not resolved, you can try passing a juiceOptions object https://github.com/automattic/juice#options. Please reopen if you still have problems.

added  &  tag solve the problem


RFC: Eval into JunoMain by default

This allows us to implement something akin to workspace() for the JunoMain module (namely the clearall() function here and a Reset Workspace in Atom).
I think these PRs are fairly comprehensive, but maybe there's some general problem with this approach that I don't know of -- have only tested this for a bit.
Let me know what you think!

Not sure I understand the aim here; does this give us something that just using Main and workspace() doesn't?
I guess this would ensure that Juno APIs are loaded at any rate.

Hm, for some reason I thought there'd been problems when calling workspace() from inside Juno, but obviously I didn't check that very carefully. At least I can't find anything going wrong right now.
I'll leave this open for now, maybe I (or someone else) remember(s) why I thought this'd be a good idea :D
If not, a reset-workspace command would be kinda nice anyways -- then just implemented as
clearall() = workspace(); isactive() && eval(Main, :(using Juno)); nothing

Current coverage is 12.06% (diff: 0.00%)

Merging #24 into master will decrease coverage by 0.43%

@@             master        #24   diff @@
==========================================
  Files             8          8
  Lines           112        116     +4
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits             14         14
- Misses           98        102     +4
  Partials          0          0

Powered by Codecov. Last update b967b69...8845515

Using workspace() seems to somewhat break the rendering of stuff:

I think the reason I thought this might be a good idea is that it kinda makes me uncomfortable to muck around with Main and therefore with Main.Atom and Main.Juno -- can we guarantee that none of the @asnyc stuff we do for message handling doesn't break from that?

can we guarantee that none of the @asnyc stuff we do for message handling doesn't break from that?

No part of Juno should be dependent on the user's Main module so I think if things are breaking we can consider that a bug. At one point we may have done conditional stuff with Main.Atom etc but I think there are better ways than that.


Jshell cannot find symbol

is there anyone who encountered the same problem before? My code worked fine when I was doing the first two levels.

There's likely some compilation error. Try do javac Num.java in the Terminal, outside of Jshell.

Oh it works now! thank you so much!


enyo.Scroller programmatic touch scrolling does not adjust position

I'm using this code to implement an enyo.Scroller (currently using the latest bleeding edge; thanks for fixing the entire-page-scroll problem before I even had a chance to post about it Steve Orvell!):
Javascript
components: [
    {name: "listScroller", kind: "enyo.Scroller", nofit: true, classes: "recent-items-scroller", components: [
        // A couple of controls here: empty message (which gets hidden if there's content),
        // and a generic compoment that is populated via DOM methods
    ]}
]

CSS
.recent-items-scroller {
    position: absolute;
    top: 94px;
    left: 24px;
    right: 24px;
    bottom: 108px;
    padding: 24px 2px 0;
    overflow: hidden;
    -webkit-overflow-scrolling: touch;
}

.enyo-touch-scroller {
    overflow: hidden;
}

This works great in iOS 5 and desktop browsers (because scrolling behavior falls back on the native scrolling, and -webkit-overflow-scrolling: touch ensures that iOS 5 provides things like flicking). However, on iOS 4.3 in the simulator (since I don't have a device handy), I am able to tap and drag to move the content around, but when I release the content animates back to the top scroll position. I have not been able to locate where exactly in the Enyo source code this problem is being generated, but I assume it's a shortcoming of TouchScrollStrategy. It is also possible that I'm misunderstanding what I need to do in my code to get styling running, though, so if you are able to get scrolling to work on iOS 4.3 and other places where Enyo needs to provide its own touch-based scrolling solution, let me know what JS and styling you're using and I'll test it out in my own app.

Closing out - issue quite old and not a priority to retest and assess current behavior.


BUG: select query parameter no longer works

Category

[ ] Question
[ ] Documentation issue
[x] Bug

Expected or Desired Behavior
The URI https://graph.microsoft.com/v1.0/me/drive?select=quota should return a quota element (as per the documentation in https://docs.microsoft.com/en-us/onedrive/developer/rest-api/concepts/optional-query-parameters?view=odsp-graph-online).
Observed Behavior
The URI https://graph.microsoft.com/v1.0/me/drive?select=quota no longer returns a quota element. This appears to be a regression as this used to work a couple of months ago.
Steps to Reproduce

Open Graph Explorer at https://developer.microsoft.com/en-us/graph/graph-explorer and sign in with a Personal OneDrive account.

Issue a GET https://graph.microsoft.com/v1.0/me/drive. The quota element is returned:
{
<EMAIL_ADDRESS>"https://graph.microsoft.com/v1.0/$metadata#drives/$entity",
    "id": "38f255bb65bdd594",
    "driveType": "personal",
    "owner": {
        "user": {
            "displayName": "",
            "id": "38f255bb65bdd594"
        }
    },
    "quota": {
        "deleted": 0,
        "remaining":<PHONE_NUMBER>,
        "state": "normal",
        "total":<PHONE_NUMBER>,
        "used": 577055
    }
}

Issue a GET https://graph.microsoft.com/v1.0/me/drive?select=quota. Observe that the quota element is missing:
{
<EMAIL_ADDRESS>"https://graph.microsoft.com/v1.0/$metadata#drives/$entity"
}

I believe this was resolved quite some time ago.


Enable Crawlera on demand

It would be great to enable Crawlera on demand once we detect bans from a website.
We could add some logic to the middleware to enable Crawlera once it detects the first 403 from the website, as opposed to be always on or off.
This would serve two purposes:

save a lot of troubles (and costs) by not forcing the user to decide beforehand if a particular spider should run with crawlera enabled or not. It'd be a matter of setting CRAWLERA_ENABLED = "auto" and let it enable itself when needed
make this middleware (and the scrapy+crawlera integration, in general) more useful and convenient

We could also support extending the rule for triggering the activation so as to not only be triggered by 403's.

403s will only capture some of the bans, and will fail to work correctly in many cases. Replicating the crawlera ban detection logic, and/or exporting the rules from crawlera, opens them up publicly. This is not something we want to do, but is unavoidable if this approach is to work.
Another drawback is how does this get enabled? Presumably a user must experience a ban somewhere first, then think to search for a solution or contact support? We can't preemptively do it since it will incur a cost and may delay or break some existing code. If we want users to enable it, we need to ask them and it may be irrelevant to them, or add a barrier somewhere we don't want one (e.g. upon signup).
I'd like to propose that we do not add this functionality here.
How about instead we send the user a notification when a job experiences a ban? Perhaps via intercom? We could provide instructions on how to sign up to crawlera, if needed, or just a link to enable for that spider and reschedule if desired. The ban detection could be implemented using the requests stored in hubstorage and the crawlera rules. It won't work for the case where the body of the request is needed, but this is very rare and there are work arounds.
The advantage of this approach is that we're telling the user about Crawlera at the precise time that it is useful to them. We can reuse our Crawlera rules and not worry about them leaking externally.

This is not to offer Crawlera to those who don't know it, this is to make it more convenient for Crawlera users to, well, use Crawlera with their spiders.

+1, the issue is that with this middleware, the decision to use is made at spider level -- either a spider uses crawlera or not.
If you want more fine grained control, you have to manually set dont_proxy in request.meta.
It would be nice if it had the option to enable crawlera automatically if a certain number of requests are failing.

We could also support extending the rule for triggering the activation so as to not only be triggered by 403's.

Why did you choose 403 instead of 503 or 429? is it the most common ban code? I think it won't take long until someone propose to extend the http status list, or inspect the body looking for captcha boxes, or "Sorry, no results for you" text in 200 http responses. Users surely need different rules for different websites, so let's add that too.
I'm with @shaneaevans on this, we are going to recreate ban detection logic and expose what took years to collect and build as Crawlera business assets.

This is not to offer Crawlera to those who don't know it, this is to make it more convenient for Crawlera users to, well, use Crawlera with their spiders.

So the idea is not to help users find out about Crawlera as a solution to their (yet unknown?) banning or throttling problems, but to optionally route requests through Crawlera on first ban after they already activated Crawlera to help avoid bans and/or increase crawling rate.
If users already activated Crawlera because of bans, why would they want to use this enable-on-first-ban feature? Once enabled it won't persist across multiple job runs so each job will go through the delay of triggering the ban to enable the middleware.
We enable AutoThrottle extension by default because we want crawlers running in our platform to be polite, but this extension is not compatible with Crawlera, so there are 2 options:

Run with AT enabled from the start and turn it off once Crawlera is turned off after first ban.
Ignore our politeness policy and run without AT, crawling as fast as possible until it trigger the ban that enables Crawlera MW (and cross fingers it doesnt take website down)

I don't think option 2 is something we want
Option 1 is a problem too because AT is very conservative about throttling, so it rarely triggers a ban but doesn't provide good request rate either. This is another topic where Crawlera knowledge base about websites excels, Crawlera knows what is the req/s limit a website can sustain. Do we want to maintain a list of throttling limits per website on the extension? no please.
In general I prefer @shaneaevans idea to detect bans by mining information stored in Hubstorage and suggest our users to enable/disable Crawlera from the start per spider.

If users already activated Crawlera because of bans, why would they want to use this enable-on-first-ban feature?

Well, my understanding of the motivation is that users want to simply enable crawlera for ALL of their spiders (even ones that are currently working without bans), and the middleware would use crawlera for them automatically if needed (no need to enable it) to avoid using too much resources.
Yes, it would involve some sort of guessing on what is "needed", I was imagining it would be something a lot simpler than what's in Crawlera.

The point is easing the trouble of having to enable (or disable) on a spider-by-spider or job-by-job basis. It's not perfect, Crawlera isn't perfect either (and will never be). It's not about replicating all the Crawlera ban rules in the middleware either. I'll try to find some time to elaborate this more.

From my time in crawlera I am really with @dangra and @shaneaevans, it's very much dependent on the website to decide when to use or not crawlera.
The only use case I have seen, and it popped up this week on slack, was on broadcrawls, you don't want crawlera beforehand, but when some specific code pops up, you might want to enable crawlera for that request.
If scrapy-crawlera is not supposed to be any smart about it and only the user configure the rules for enabling crawlera I guess it could be ok.


[Feature] Allow fd to be used instead of find

This is fd, a faster find that allows regexp as well. It seems to be much much faster. Would you make it so that if fd is found, it will automatically use that instead of find, for those who have set external command to find?
https://github.com/sharkdp/fd

thanks!


how to run this program?

I'm a fresh man to learn this program, can you help me tutorial. thank you

Well first start by learning what issues are:
'Issues are a great way to keep track of tasks, enhancements, and bugs for projects.'
Then about asking for help: https://stackoverflow.com/help/how-to-ask
But anyway, more to the point, you should look into managing javascript packages with node:
https://stackoverflow.com/questions/2353818/how-do-i-get-started-with-node-js
Then on how to serve local files: https://stackoverflow.com/questions/tagged/webserver
Good luck


chore: add exception for refs to immutable data rule

Jira
Overview of change

Add an exception for refs to the functional/immutable-data rule

After merging this PR we can upgrade ESlint in all repositories we use it.
centra.com - https://gitlab.com/centrahq/centracom/-/merge_requests/372
showroom - https://github.com/centrahq/showroom/pull/2899

LGTM 👌 What do you think about doing the release in the end of sprint? So we can update the version in repos with one PR

Good point, let's do it at the end of the sprint


Linux: (>= GlibC 2.36 ?) static declaration of ‘arc4random_buf’ follows non-static declaration

I received this compilation error:
[ 33%] Compiling:archive_write_add_filter_gzip.c
[ 33%] Compiling:archive_write_add_filter_program.c
[ 33%] Compiling:archive_write_add_filter_zstd.c
/home/scb/Projects/BlitzMaxNG.downloads/BlitzMax/mod/archive.mod/core.mod/libarchive/libarchive/archive_random.c:54:13: error: static declaration of ‘arc4random_buf’ follows non-static declaration
   54 | static void arc4random_buf(void *, size_t);
      |             ^~~~~~~~~~~~~~
In file included from /home/scb/Projects/BlitzMaxNG.downloads/BlitzMax/mod/archive.mod/core.mod/libarchive/libarchive/archive_random.c:30:
/usr/include/stdlib.h:542:13: note: previous declaration of ‘arc4random_buf’ with type ‘void(void *, size_t)’ {aka ‘void(void *, long unsigned int)’}
  542 | extern void arc4random_buf (void *__buf, size_t __size)
      |             ^~~~~~~~~~~~~~

It indicates that libarchive uses a different function definition than the stdlib.h of that linux user's distribution. Happens as with GLibC >= 2.36 (so most probably rolling-release-distros) as it has it's own ARC4RANDOM_BUF already.
Googling for that issue brings up various similar problems.
Yet I assume HAVE_ARC4RANDOM_BUF might have to be configured differently there.
And:
https://github.com/vmchale/libarchive/issues/33

Hmm ... we could maybe simply identify the glibc version on that machine:
#include <gnu/libc-version.h>
#include <stdio.h>
int main(int argc, char *argv[]) {
    printf("%s %s\n",gnu_get_libc_version(),gnu_get_libc_release());
    printf("glibc v%i %i.%i\n",__GNU_LIBRARY__,__GLIBC__,__GLIBC_MINOR__);
    return 0;
}

source: https://unix.stackexchange.com/questions/120380/what-c-library-version-does-my-system-use
So maybe simply branch between <2.36 and >=2.36 ?


Login request is sent without port

What kind of request is this (question/bug/enhancement/feature request): Bug
Steps to reproduce (least amount of steps as possible):

Deploy Rancher 2.3.5 Helm Chart with Rancher-generated certificates with the Ingress exposing ports different than 80/443 for respectively HTTP and HTTPS (in my case that was 8080 and 8443)
After opening the Rancher panel site for the first time, configure the hostname to the same one as from Helm Chart values, with added port (in my case https://rancher.rancher:8443)
Wait for the local cluster to be provisioned
Upgrade the Rancher Chart from version 2.3.5 to 2.4.1
Try to login

Result:
The login request made after clicking "login" button on the Rancher's panel login site doesn't include the port on which Rancher is actually deployed which makes it impossible to login. After rollbacking Rancher Chart to version 2.3.5 the login button works properly.
Other details that may be helpful:
As you can see in the screenshot from Chrome DevTools which I attached below, the request made after clicking "login" button doesn't include the port, which is present in the browser address field:

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.4.1
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Deployed with RKE with Rancher installed from Helm Chart (3 control plane machines, 3 workers)
Machine type (cloud/VM/metal) and specifications (CPU/memory): VM (2 CPU, 4 GB RAM for control plane, 4 CPU, 8 GB RAM for workers)
Kubernetes version (use kubectl version): 1.17.2

Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.0", GitCommit:"70132b0f130acc0bed193d9ba59dd186f0e634cf", GitTreeState:"clean", BuildDate:"2019-12-07T21:20:10Z", GoVersion:"go1.13.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:22:30Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client: Docker Engine - Community
 Version:           19.03.5
 API version:       1.40
 Go version:        go1.12.12
 Git commit:        633a0ea
 Built:             Wed Nov 13 07:25:41 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.5
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.12
  Git commit:       633a0ea
  Built:            Wed Nov 13 07:24:18 2019
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.2.10
  GitCommit:        b34a5c8af56e510852c35414db4c1f4fa6172339
 runc:
  Version:          1.0.0-rc8+dev
  GitCommit:        3e425f80a8c931f88e6d94a8c831b9d5aa481657
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

I'm confused about your setup.  I think I know the fix, but regarding your setup I don't understand it.  Can you share the specific helm values you use to install?  I don't understand how ingress can be :8443 but you access the site through :443.  I would think ingress will drop the request if you don't configure the host as :443.

I believe we are facing a similar issue. We also upgraded from 2.3.5 to 2.4.1 and can not login. We are using external tls termination though. When I inspect our headers the requested url is adding port 80.

@toddexp how did you do your initial Rancher installation w/ RKE and helm?

@bmdepesa Yes. This is a Rancher HA install with RKE and helm.

@toddexp can you elaborate on your Helm set values and any ingress options you supplied in the RKE yaml?

@bmdepesa
RKE ingress section: ingress:   provider: nginx   options:     use-forwarded-headers: 'true'
helm: helm install rancher rancher-latest/rancher --namespace cattle-system --set hostname=example.com,rancherImageTag=v2.4.1,tls=external

Tested in v2.4.2-rc2, Worked fine when upgraded from v2.3.5 to v2.4.2-rc2
Steps:

I had a setup v2.4.1 HA set up with ingress pointing to 8080/8443 (3 nodes all roles)
Downgraded HA setup to v2.3.5
Upgraded it to v2.4.2-rc2
Login worked.

I'll try with a setup initially with v2.3.5 and update the result.

@toddexp I am not able to reproduce the same behavior.

v1.17.2 local cluster (RKE v1.04) (w/ forward headers)
Rancher v2.3.5 HA w/ external TLS
TLS offloaded to nginx lb

helm install rancher rancher-stable/rancher --version 2.3.5 --namespace cattle-system --set hostname=host.com --set tls=external

helm upgrade rancher rancher-latest/rancher --version 2.4.1 --namespace cattle-system --set hostname=host.com --set tls=external

After upgrading Rancher, I am able to login (including in new browser / session). I do not see :80 appended to the hostname in the request.

@toddexp Could you please log a separate issue to track the issue you are seeing ? Issue that is noted in this bug originally is fixed v2.4.2-rc2 and validated - https://github.com/rancher/rancher/issues/26422#issuecomment-607479384

We have reverted the PR in the v2.4 line so that this login works as expected. An official fix will need to be created in v2.4 which is being tracked #26434.

This 2.4.2 release fixed my issue as well. I am now able to login. Thank you all.


YM2149F envelope mode emulation bug

When turning on the hardware envelope on a channel, and setting that channel's tone and noise mode off, the channel stays silent and does not reflect the envelope value.
See issue https://github.com/tildearrow/furnace/issues/77 and potential fix.

Thanks for the report and pointer to a potential fix. I'll look into it.

This should be fixed with dc253c91dd600655bc6583ec16880ceee6901374


Columns are overlapping when readmore options included

Bug Report
I'm using the react-virtualized for a while. it's really cool, but I'm facing some overlapping issues sometimes while included the Readmore option if the column contains large text or it contains more than 350 characters.

What is the current behavior?
it start overlapping when performing some actions inside the column, like Readmore or edit
What is the expected behavior?
it should not overlap when I perform some action inside each column
Which versions of React and react-virtualized, and which browser / OS are affected by this issue? Did this work in previous versions of react-virtualized?

Browser
chrome

OS
mac

React
16.12.0

React DOM
16.12.0

react-virtualized
9.21.2

You will need to call recomputeRowHeights(rowIndex) whenever you change the height of the cell dynamically.


Back camera not saved

This is wonderful example thank you. But  back camera not saved images and videos. Front camera is working.  What must i do . Thank you..

sorry it was about  Imagesize it is saved .


Fix: enable loading video file as data URI (fixes #4830)

This PR

Fixes a bug

Describe the changes below:

Detect videoType from data URI and return proper VideoURL object

👍


Day 5: advancing or 'skipping' an iterator

So I was playing with peekable iterators today. I noticed that I wanted to be able to advance an iterator to skip one char inside a while let, which I suspect isn't possible. Is there an idiomatic equivalent? This just seems ugly:
fn reaction (polymer: &Vec<char>) -> Vec<char> {
    let mut stable: Vec<char> = vec![];

    let mut c = polymer.iter().peekable();
    let mut skip_char = false;

    while let Some(first) = c.next() {
        if skip_char {
            skip_char = false;
            continue;
        }

        match c.peek() {
            Some(second) => {
                if will_react(first, second) {
                    skip_char = true;
                } else {
                    stable.push(*first);
                }
            },
            None => stable.push(*first),
        }
    }

    stable
}

https://github.com/richchurcher/advent-of-code-2018/blob/master/src/day5.rs

That looks fine to me. What do you think is ugly about it?

Mmm, maybe you're right. I guess I was hoping I could somehow call c.skip() from within the loop to artificially advance it without caring about the value, and without having to rely on a variable external to the loop. But this works too :smile:


Match Azure and Local responses. Add host and port on local provider

NOTE: Depends on https://github.com/boostercloud/booster/pull/1182.
- `package.json` needs to be updated after Booster deploy this version

NOTE: Once https://github.com/boostercloud/booster/pull/1182 is merged, this PR needs to be updated to match new Booster versions
Changes:

Azure and Local providers return same responses
Local providers return host:port when needed

Converted to draft until we update it to new Booster version and fix the package.json files


fix(fetchers): Fix provider override

Description of change

Pull-Request Checklist

[ ] Code is up-to-date with the main branch
[ ] This pull request links relevant issues as Fixes #0000
[ ] There are new or updated unit tests validating the change
[ ] Documentation has been updated to reflect this change
[ ] The new commits follow conventions outlined in the conventional commit spec

init is not called when setting the provider https://github.com/morpho-org/morpho-aave-v3-sdk/blob/fix/provider-override/src/fetchers/Chain/ChainFetcher.ts#L23C16-L23C27

Yes but it is called at every fetch (maybe we should change this)

But if poolDataProvider is set, there is no reinitialization. So we have to force reinit when we change the provider


[BEAM-745] Support display data for transforms with nested components of the same instance

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

/CC @bjchambers @pabloem Just showing a failing test case so far.

R: @bjchambers
The RunnableOnService tests for primitive display data still need work, but I wanted to get some feedback on new APIs.

I've addressed all outstanding feedback. Travis failures are flakes. PTAL @bjchambers

This generally LGTM.

LGTM

Update description? Right now this whole PR is described as:
"Test case for failing display data for nested component structure"
and/or
"Add new include API with path"

Updated PR title, and squashed commits with more descriptive commit message.


Docs for Observer should mention what happens if you use a bundle in the Trigger

I wanted to make a trigger that triggers when either one of two components is removed, but it was not obvious to me that Trigger<OnRemove, (ComponentA, ComponentB)> would do this. In query filters a tuple like this expresses an AND, but here it is an OR.
I think this should be stated in the docs somewhere, and/or in an example. I couldn't find anything in the docs or examples that used an observer with more than one component in the filter. I'd say there should be at least a sentence in the Observer docs explaining this, and maybe one in Trigger too, since that's where you would usually specify the parameter.

Related to https://github.com/bevyengine/bevy/issues/14726


regex-dna to regex-redux because string sub used instead of regex

fwiw The regex-dna programs were re-written as regex-redux because several contributed programs were using string substitution instead of pattern matching.

in this benchmark is there different programs? may be i ll change to it.

Yes, the regex-dna programs were re-written to do regex-redux
regexdna		obsolete,removed
regexredux		ACTIVE


Typescript support

I'd love to see typescript support or at least type declaration file.
thanks!

This is the only drawback I have from using it + the number of download/months in npm. but I hope that will change with growing popularity

Good suggestion, thanks!

I just evaluated a number of similar libraries (including JSONCrush, lz-string's compressToEncodedURIComponent()), this library had both the best compression and best readability.  lz-string of course is not readable... and is more general purpose than just JSON.  So I am choosing  it.  I needed the typescript defs too.. so just added #165.

I just evaluated a number of similar libraries (including JSONCrush, lz-string's compressToEncodedURIComponent()) and this library had both the best compression and best readability for my data.

Wow, that's a pleasant surprise. Thanks for the info! What sorts of input did you try it with?

I am mentioning lz-string because it wasn't in https://www.jsonurl.org/#relatedwork, and it seems to be very popular according to npm. Of course lz-string is not readable... but I was surprised that it didn't do nearly as well as this library for my data. It makes sense in retrospect though... because lz-string is more general purpose and is probably better for larger data sizes than my use cases.

Thanks for the heads up. I'll take a look at lz-string and add a paragraph to related work section.

Yes - it was nice to see.   In my case, the JSON I am compressing has short key/values (1 char each). And I am using chars that don't need escaping. So this helps json-url.
I believe lz-string is using base64 encoding of the lz compressed result.  It just uses a different character set https://github.com/pieroxy/lz-string/blob/master/libs/lz-string.js#L14-L15.  If I am correct, then it makes sense it doesn't do as well because of the overhead of base64.  Perhaps with larger input sizes it can overcome this overhead.
JSONCrush had better compression than lz-string too... but not as good as json-url.
Some example inputs I used:
[{"n":8,"i":6},{"n":7,"i":7}]
[{"n":10,"i":1},{"n":2,"i":2},{"n":3,"i":3},{"n":4,"i":4,"s":0},{"n":5,"i":5,"w":21},{"n":6,"i":6,"w":30},{"n":7,"i":8},{"n":9,"i":9}]

I am only listing a couple of examples... In these examples there are definitely patterns with i and n.  But this is not always the case.   The important thing is that for different types of JSON input, the results of which library is the best at compression varies.
json-url will do well if the developer experiments with different schemas/interfaces for their JSON object.  Whereas, if the payload is large and/or is arbitrary (not just some JSON), then maybe lz-string is the best choice.
The bonus, is that I think you did well aesthetically with choosing how to re-encode the JSON for the URL.  Your choices make the result easy for a human to read.  This is valuable to me... and a nice bonus when it beats other libraries in terms of size.


Official Chia Pool Support (as of July 7th) need -p to become -c

Per the new Chia release that allows official support for pooling, chia plot create cannot use the -p flag anymore, it needs to be replaced with the -c flag (see Step 4 in https://github.com/Chia-Network/chia-blockchain/wiki/Pooling-User-Guide).
Any plan to update plotman to allow that?
Thanks for the good work and
Happy ploting!

It's not per se a replacement, but yes, it is already done and released.
https://github.com/ericaltendorf/plotman/discussions/835#discussioncomment-980425


Merge Uat ke PRD 12 April 2017 - validasi qty kanban

Coverage increased (+0.3%) to 64.701% when pulling d659c45c83edd2341836d83a463edcce49026051 on uat into 70cf77e1e60bbc9886531d8333964f6ce3555b87 on master.


added security context to template if using persisted volumes

What this PR does / why we need it:
This chart is broken when using a PVC with updated versions of Grafana.
Grafana now runs as a non privileged user, with uid 472.
This PR ensures that if we're using a persistent volume, that it has the right access and perms to write to disk.
Which issue this PR fixes
fixes grafana-docker #167
https://github.com/grafana/grafana-docker/issues/167#issuecomment-401774994
Special notes for your reviewer:

Question Why do you use 472 as  fsGroup ?

472 is the uid of the grafana user the upstream docker image creates and uses to run grafana in the container.
Without this, grafana doesn't have access to persistent volumes, and will failed due to insufficient permissions.

Ok I'm not an expert in rbac.psp so may I do a mistake but  have you think about people who use rbac.pspEnable: false ? in this case it may not work isn't it ?

I'm not very familiar with pod security policies, but this PR addresses the linux systems permissions issue in the issue linked to above specifically that stems from the grafana team updating their image.
If this won't work for those disabling psp, I'd love to know about it, and what a possible work around might be.

@RyanHartje I am getting this issue when trying to create Grafana with a persistence turned on in the values.yaml.

@antodoms I'm sorry, I don't quite understand. Do you mean that you're also getting issues when enabling persistence with the current Grafana charts, or do you mean that you receive an error with my changes in the Grafana chart when enabling persistent volumes?
Thank you for your help.

I've been getting a lot of github notifications for this issue where I filed it in Grafana.
Here's a walkthrough of the problem this PR seeks to solve (copied from an issue filed in Grafana).
The helm chart has broken linux permissions when mounting a volume for persistence.
Within the docker image for grafana, we create the user grafana, with a uid of 472:
https://github.com/grafana/grafana/blob/master/Dockerfile#L40
https://github.com/grafana/grafana/blob/master/Dockerfile#L61
We then give this user parts of the filesystem ownership here:
https://github.com/grafana/grafana/blob/master/Dockerfile#L69
The issue being reported is when helm mounts a volume over /var/lib/grafana, the ownership is changed to the default (root:root).
Here's a walk through of issue replication and identification.
Steps:
Start Minikube
▶ minikube start
Starting local Kubernetes v1.10.0 cluster...
Starting VM...
Downloading Minikube ISO
 160.27 MB / 160.27 MB [==================================] 100.00% 0s
Getting VM IP address...
Moving files into cluster...
Setting up certs...
Connecting to cluster...
Setting up kubeconfig...
Starting cluster components...
Kubectl is now configured to use the cluster.
Loading cached images from config file.

Init Helm
▶ helm init
$HELM_HOME has been configured at /Users/ryan/.helm.

Tiller (the Helm server-side component) has been installed into your Kubernetes Cluster.

Please note: by default, Tiller is deployed with an insecure 'allow unauthenticated users' policy.
For more information on securing your installation see: https://docs.helm.sh/using_helm/#securing-your-helm-installation
Happy Helming!

Modify Grafana values to persist data
Copy default values file (https://github.com/helm/charts/blob/master/stable/grafana/values.yaml)
Modify:
 85 persistence:
 86   enabled: false
 87   # storageClassName: default
 88   # accessModes:
 89   #   - ReadWriteOnce
 90   # size: 10Gi
 91   # annotations: {}
 92   # subPath: ""
 93   # existingClaim:

to
 85 persistence:
 86   enabled: true
 87   accessModes:
 88     - ReadWriteOnce
 89   size: 10Gi

Run helm install
▶ helm install stable/grafana -f ./values.yml
NAME:   linting-umbrellabird
LAST DEPLOYED: Fri Sep  7 09:18:15 2018
NAMESPACE: default
STATUS: DEPLOYED
[truncated]

Inspect persisted volume permissions
Here's the deployment template for Grafana in it's helm chart:
https://github.com/helm/charts/blob/master/stable/grafana/templates/deployment.yaml#L256 - enables the PVC
https://github.com/helm/charts/blob/master/stable/grafana/templates/deployment.yaml#L147 -
Since we see it will be mounted at /var/lib/grafana, let's check out perms in there:
▶ kubectl exec linting-umbrellabird-grafana-fccdbc468-9wjc6 -- stat /var/lib/grafana
  File: /var/lib/grafana
  Size: 4096      	Blocks: 8          IO Block: 4096   directory
Device: 801h/2049d	Inode: 1572866     Links: 3
Access: (0777/drwxrwxrwx)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2018-09-07 13:18:15.591558589 +0000
Modify: 2018-09-07 13:18:32.858762207 +0000
Change: 2018-09-07 13:18:32.858762207 +0000
 Birth: -

▶ kubectl exec linting-umbrellabird-grafana-fccdbc468-9wjc6 -- ls -ahl /var/lib/grafana
total 400K
drwxrwxrwx 3 root    root    4.0K Sep  7 13:18 .
drwxr-xr-x 1 root    root    4.0K May 16 18:25 ..
-rw-r--r-- 1 grafana grafana 384K Sep  7 13:18 grafana.db
drwxr-xr-x 2 grafana grafana 4.0K Sep  7 13:18 plugins

What does this look like by default without persistence?
We can helm install the chart with no values file specified to get default behavior.
▶ helm install stable/grafana
NAME:   eponymous-ladybug
LAST DEPLOYED: Fri Sep  7 09:29:38 2018
NAMESPACE: default
STATUS: DEPLOYED

▶ kubectl exec eponymous-ladybug-grafana-79cf48b9db-5xfcs -- stat /var/lib/grafana
  File: /var/lib/grafana
  Size: 4096      	Blocks: 8          IO Block: 4096   directory
Device: 801h/2049d	Inode: 1057153     Links: 3
Access: (2777/drwxrwsrwx)  Uid: (    0/    root)   Gid: (  472/ grafana)
Access: 2018-09-07 13:30:09.909008138 +0000
Modify: 2018-09-07 13:29:40.667179653 +0000
Change: 2018-09-07 13:29:40.667179653 +0000
 Birth: -

▶ kubectl exec eponymous-ladybug-grafana-79cf48b9db-5xfcs -- ls -ahl /var/lib/grafana
total 400K
drwxrwsrwx 3 root    grafana 4.0K Sep  7 13:29 .
drwxr-xr-x 1 root    root    4.0K May 16 18:25 ..
-rw-r--r-- 1 grafana grafana 384K Sep  7 13:29 grafana.db
drwxr-sr-x 2 grafana grafana 4.0K Sep  7 13:29 plugins

This causes data to be unreadable by the grafana user, making a chart deployed with a persistent volume now useable since grafana 5.1.2 (guesstimate).

/ok-to-test

Looks like this is already resolved, closing.


Make Print API work on Windows.

The Print API(window.print()) on Windows will hang on printing spool(see #865). The reason is that Chromium doesn't generate the printed file content and keep waiting in a loop.
In the implementation, Chromium seperates a prcoess called Utility to do the printing work, so we need to custom the Utility process for Electron. On Windows, the print API relies on the pdf component which using to convert to EMF
(see https://code.google.com/p/chromium/issues/detail?id=170859). So we need to port pdf component in libchromiumcontent.
But there are something need to be considered. Introducing pdf component will make the release binary size larger. Is this reaonable? Or we can make the pdf as a seperated shared library(dll), and if developers don't need the print API, they can safely remove it in saving the binary size.
As a side effect, with pdf component, we may easyly support print to pdf and view pdf features.
This patch will rely on atom/libchromiumcontent#104, and fix #865.
Any comments and advice are welcomed.
##TODO

[x] API window.print() works. The printer will print the document correctly.
[ ] Pass component build(Dev build)
[x] Pass non-component build(Release build)
[ ] Don't effect the OS X and Linux Platforms

Here are some output.
Print through PDFCreator:

Print through real printer

We would like to see printing / PDF support show up in Electron for sure

@zcbenz
This PR is ready for review now. Before get merged, atom/brightray#102 should be required to merge for component build.

Awesome!
v0.25.3 is released with PDF libraries, and the binary size is increased by about 14mb, I agree we should make it a DLL. We can probably patch libchromiumcontent to compile pdf target into a shared_library to make things easier.


Fix the Tests

This branch attempts to fix some of the test failures. However, it was based off of the 0.1.9 release. These changes should be pulled into master if they'd be helpful.

Closing this because I don't want it assigned to me forever.


Disabling midnightlib's hats seems to prevent the game from loading worlds

Stacktrace:
at eu.midnightdust.hats.witch.WitchHatFeatureRenderer.render(WitchHatFeatureRenderer.java:60)
at eu.midnightdust.hats.witch.WitchHatFeatureRenderer.method_4199(WitchHatFeatureRenderer.java:28)
Edit: The embedded version of MidnightLib.

The embedded version of MidnightLib is pretty old, please put the newest version of MidnightLib in your mod folder alongside Blur, it should be fixed there. https://modrinth.com/mod/midnightlib/version/kU3LjFF9

The newest version of MidnightLib is now shipped with the latest release of Blur :)


How to compile examples

I am trying to compile the examples, and I am unsuccessful. I am mainly interested in compiling the lwt client example. I can deduce it requires a development version of httpaf, so I pinned httpaf with --dev-repo. However, it still fails with unbound value Reqd.response_with_upgrade. Can you tell me what version of httpaf I need to pin?
https://github.com/anmonteiro/ocaml-h2/blob/a0e3f4506528bf31e0f32f272c688662436af366/examples/lwt/lwt_h2c.ml#L115
I would also recommend adding this information to the opam file as a dev dependency.

Hey @reynir, you should pin the packages in https://github.com/anmonteiro/httpaf to get upgrade support and compile the examples successfully.
As for opam: I don't use it, so I'll defer to your judgement on the best way to track that in the opam files.

A note in a README file would do, too.

This should be fixed now that httpun has been released to opam and h2 depends on httpun-types.


add namespace in configure-installer-tests-cluster-s390x.sh

What type of PR is this:
/kind bug

What does this PR do / why we need it:
For Z, we need to add namespace "nodejs-14" in configure-installer-tests-cluster-s390x.sh refer to configure-installer-tests-cluster.sh. this namespace is used in test cases, so I create this pull request to add it.
Which issue(s) this PR fixes:

Fixes #
PR acceptance criteria:

[x] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

/area system-PZ


Optimize fixedNavbar height detection

Like Bootstrap navbar, using jQuery height() could miss the outer height of the navbar, which this could use outerHeight() to exactly detect the navbar height instead for deviation.

Fixed during the latest release, thanks!


Update to conrod 0.57, cpal 0.6, glium 0.19, image 0.18. Publish 0.2.

Version 0.2 includes almost all changes on master since this repo started! See merged PRs for more details.

Travis is failing due to a bug in rustdoc nightly rust-lang/rust#46675 - going to merge and publish anyways as stable and beta work fine.


Container size

Hi, are there any plans to work on smaller container size? Currently it's almost 500mb while node containers are ofter ~80mb and go even 5mb.
Maybe there should be used alpine images?

Today I see it has almost 600MB. How is this possible for such small service?


SDL_Init with SDL_INIT_JOYSTICK or SDL_INIT_GAMECONTROLLER stopped working

Hyprland Version
 System/Version info
Hyprland, built from branch main at commit 3d64b0e9f08151786d354c0ca45c5c6016a16836 dirty (flake.lock: update).
Date: Sat Apr 6 18:09:37 2024
Tag: v0.38.0-54-g3d64b0e9, commits: 4387

flags: (if any)

System Information:
System name: Linux
Node name: zjeffer
Release: 6.8.2-arch2-1
Version: #1 SMP PREEMPT_DYNAMIC Thu, 28 Mar 2024 17:06:35 +0000

GPU information:
01:00.0 VGA compatible controller [0300]: NVIDIA Corporation GA104M [GeForce RTX 3070 Mobile / Max-Q] [10de:24dd] (rev a1) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:
  split-monitor-workspaces by Duckonaut ver 1.1.0

Bug or Regression?
Bug
Description
Since a couple of days, Jellyfin Media Player (a Qt C++ application that uses SDL) has stopped working on Hyprland. Someone else is having the same problem: https://github.com/jellyfin/jellyfin-media-player/issues/589
As I found here, SDL_Init is stuck when calling SDL_Init(SDL_INIT_JOYSTICK). I'm not sure how to debug this further, but I managed to reproduce it with the C++ code below.
How to reproduce
#include <SDL2/SDL.h>

int main(int argc, char *argv[]) {
  SDL_Log("Initializing SDL");
  if (SDL_Init(SDL_INIT_JOYSTICK) != 0) {
    SDL_Log("Failed to initialize SDL: %s", SDL_GetError());
    return 1;
  }
  SDL_Log("SDL initialized successfully");

  SDL_Quit();
  return 0;
}

cmake_minimum_required(VERSION 3.12)
project(sdl-test)
set(CMAKE_CXX_STANDARD 23)

find_package(SDL2 REQUIRED)

set(SOURCES src/main.cpp)
add_executable(sdl-test ${SOURCES})
target_link_libraries(sdl-test SDL2)

I found that the SDL_Init function freezes the application completely when calling it with SDL_INIT_JOYSTICK or SDL_INIT_GAMECONTROLLER. The other flags work fine.
Crash reports, logs, images, videos
Backtrace after interrupting the process when it freezes:
(gdb) bt
#0  0x00007ffff79196bc in __GI___libc_read (nbytes=19, buf=0x7fffffffcbe0, fd=11) at ../sysdeps/unix/sysv/linux/read.c:26
#1  __GI___libc_read (fd=fd@entry=11, buf=buf@entry=0x7fffffffcbe0, nbytes=nbytes@entry=19) at ../sysdeps/unix/sysv/linux/read.c:24
#2  0x00007ffff742fe32 in read (__nbytes=<optimized out>, __buf=<optimized out>, __fd=<optimized out>, __fd=<optimized out>,
    __buf=<optimized out>, __nbytes=<optimized out>) at /usr/include/bits/unistd.h:28
#3  read_sysfs_attr (ctx=0x55555558ecc0, sysfs_dir=<optimized out>, attr=attr@entry=0x7ffff743c280 "bConfigurationValue",
    max_value=max_value@entry=255, value_p=value_p@entry=0x7fffffffcc44) at os/linux_usbfs.c:511
#4  0x00007ffff743008c in sysfs_get_active_config (config=0x7fffffffcc44, dev=0x5555555d2450) at os/linux_usbfs.c:584
#5  op_get_active_config_descriptor (dev=dev@entry=0x5555555d2450, buffer=buffer@entry=0x7fffffffcc9e, len=len@entry=9) at os/linux_usbfs.c:806
#6  0x00007ffff742f14f in get_active_config_descriptor (dev=dev@entry=0x5555555d2450, buffer=buffer@entry=0x7fffffffcc9e "", size=size@entry=9)
    at /usr/src/debug/libusb/libusb-1.0.27/libusb/descriptor.c:483
#7  0x00007ffff7433e33 in libusb_get_active_config_descriptor (dev=0x5555555d2450, config=0x7fffffffcd68)
    at /usr/src/debug/libusb/libusb-1.0.27/libusb/descriptor.c:563
#8  0x00007ffff7de6939 in LIBUSB_hid_enumerate (vendor_id=vendor_id@entry=0, product_id=product_id@entry=0)
    at /usr/src/debug/sdl2/SDL2-2.30.2/src/hidapi/libusb/hid.c:799
#9  0x00007ffff7de6e9b in SDL_hid_enumerate_REAL (vendor_id=0, product_id=0) at /usr/src/debug/sdl2/SDL2-2.30.2/src/hidapi/SDL_hidapi.c:1236
#10 0x00007ffff7ed77a3 in HIDAPI_UpdateDeviceList () at /usr/src/debug/sdl2/SDL2-2.30.2/src/joystick/hidapi/SDL_hidapijoystick.c:1108
#11 0x00007ffff7ed7daf in HIDAPI_JoystickInit () at /usr/src/debug/sdl2/SDL2-2.30.2/src/joystick/hidapi/SDL_hidapijoystick.c:620
#12 0x00007ffff7dbf708 in SDL_JoystickInit () at /usr/src/debug/sdl2/SDL2-2.30.2/src/joystick/SDL_joystick.c:632
#13 SDL_InitSubSystem_REAL (flags=512) at /usr/src/debug/sdl2/SDL2-2.30.2/src/SDL.c:300
#14 0x000055555555519e in main (argc=1, argv=0x7fffffffd148) at /home/zjeffer/git/sdl-test/my-sdl-project/src/main.cpp:5

bisect please

Can't reproduce after bisecting: the current commit was bad, I rebooted into an older commit that was good, did that a couple of times, then rebooted back into the commit that was supposedly bad and it works fine now.
This is really weird as I had already rebooted once and that didn't fix it.
I actually had the same exact issue a couple of days ago with Jellyfin Media Player not starting, and it was also resolved after a (single) reboot. Seems to be an intermittent issue? Not sure if it's even Hyprland related.
I'll reopen once I get the problem again and/or have more info about if it's Hyprland related or not.


Prove AES256Equivalence Axiom key_expand_equiv

I noticed we don't have this, while trying to port proofs over to know sequential circuit setup.
Are there any other axioms we need to finalize phase I?

Phase 1 intentionally excludes key expansion (see the description on the project: "minus key expansion"); the axiom is expected to remain. We decided at the start of phase 1 to narrow the scope as much as possible, and since key expansion is relatively separate from the rest of the cipher (and also an "optional" part of a hardware AES block --  see discussion here under "Datapath Architecture and Operation"). It may seem counter-intuitive to leave out key expansion, but the goal here isn't to produce a production-ready system; we just want a case study for our methodology.


UI system and guidlines

Summarize your proposed improvement
Formation of a UI "taskforce" to work out a modular, simple and future proof UI system. This should make sure that everything added to the project fits together nicely. It needs to look appealing but also consist of a simple but effective, modular structure so we can expand and use it for everything. Some basics are: defined colors, a modular interaction menu, uniform sizes of images/icons/textboxes/etc.
Why is this needed?
 Creating a solid UI system as a base that we can use for all future updates to build on makes the most sense in the early stages of development. Changing such a system later is going to require quite some work, so why don't do it the right way from the start.
Describe alternatives you've considered
No response
Add any other context or screenshots about the request here.
No response

I agree mostly with that, it defiantly has a good structure but I do feel it is missing (or at least stuff has no documentation on how to do it yet) some things. There are a few cases I have run into that make it seem like custom Widgets are needed.

I would suggest one thing.
It has to be compatible with the console so it's still Cross-platform.
We have all the tools for it, so we should use them correctly.
I know that HTML UI looks probably better or is more functional.
But then its just not Cross-platform

It has to be compatible with the console so it's still Cross-platform.
Exactly my point. We need to think of players that use controllers. Especially with things like selecting amounts of an item etc.


Can't login to share page

I just composed acs-community-deployment using docker-compose, but when trying to open http://$HOST:8080/share/page, and logging in using username=admin & password=admin credentials, it keep returns Your authentication details haven't been recognized or Alfresco Content Services may not be available at this time.

HI, can you please check if alfresco container is running? You can do that by running docker ps.

@justinushermawan  I had similar issue running compose. Alfresco container was shutting down after some time which lead to inability to login through /share/. The root cause was small amount of allocated memory for docker (on macOS). Significant increase to ~3-4Gb allowed to start.

I'm going to close this issue as there was no activity for some time. Please reopen with a comment or create a new one if there are still problems unresolved. Thanks


Providing String as a argument causes crash

I've tried multiple ways to pipe options to faker but cannot succeeded, when I do things like {{ lorem.lines | fake: 1 }} where argument is privitive things ten to work - but only when argument is integer; as soon as I will add string there is a crash.
Example: ${{ phone.number | fake: "+48#########" }}
To be checked on https://stepci.com/ playground.

Playground was outdated. Feel free to reopen the issue if you're experiencing this on local

Hi @mishushakov ,
how can we pass objects to faker, as in faker.color.cmyk({ format: 'css' })
Thanks, Hannes

Like this?
{{color.cmyk | fake: {"format": "css"}}}
Keep in mind this is probably not possible to do in YAML

I would be interesting in knowing real example how I can pass the object to faker for example, like @HKandulla mentioned, faker.color.cmyk({ format: 'css' }) inside of the .yaml test file that is run in stepci.

Playground was outdated. Feel free to reopen the issue if you're experiencing this on local

Problem is still persistent in playground and on my local machine, it's a real issue with stepci and how faker or liquidless is handled.

I have since added JSON5 support, you can now use single quotes for JSON objects:
version: "1.1"
name: Fake Data
tests:
  example:
    steps:
      - name: GET request
        http:
          url: "https://en.wikipedia.org/wiki/${{color.cmyk | fake: {'format': 'css'}}}"
          method: GET
          check:
            status: /^20/

Okay, that seems to work - however when I try to reference phone number format it does not :/
version: "1.1"
name: Fake Data
tests:
  example:
    steps:
      - name: GET request
        http:
          url: "https://en.wikipedia.org/wiki/${{ phone.number | fake: '+48#########' }}"
          method: GET
          check:
            status: /^20/

At this point I have no clue it's issue with faker (where function works in faker library used in way it should be parsed), liquidless (it seems to pass things correctly) or liquidless-faker (where something on the road is wrong)

Currently I've found workaround with exposing a endpoint on server that generates a random phone number that is later used in tests, but it would be good to more advanced fake data on the fly

Hi @mishushakov ,
thanks for looking into this!
Adding JSON5 supports makes it possible to pass objects but faker still ignores the value.
Passing not format
version: "1.1"
    name: Fake Data
    tests:
      example:
        steps:
          - name: POST request
            http:
              url: "https://httpbin.org/post"
              method: POST
              json:
                name: default
                color: "${{color.cmyk | fake}}"
              check:
                status: /^20/

Request
POST https://httpbin.org/post HTTP/1.1
Content-Type: application/json

{"name":"default","color":"0.5,0.56,0.52,0.96"}

Passing format: css
version: "1.1"
name: Fake Data
tests:
  example:
    steps:
      - name: POST request
        http:
          url: "https://httpbin.org/post"
          method: POST
          json:
            name: test_css
            color: "${{color.cmyk | fake: {'format': 'css'}}}"
          check:
            status: /^20/

Request
POST https://httpbin.org/post HTTP/1.1
Content-Type: application/json

{"name":"test_css","color":"0.37,0.28,0.83,0.03}"}

Passing format: css should resolve to:
faker.color.cmyk({ format: 'css' }) // cmyk(100%, 0%, 0%, 0%)

Have you properly re-installed stepci?

Hi @mishushakov ,
sorry for my late reply..
Yes, I have re-installed stepci and also tried the following example using the playground on https://stepci.com/
Example:
version: "1.1"
name: Fake Data with objects as parameter
tests:
  example:
    steps:
      - name: POST request
        http:
          url: "https://httpbin.org/post"
          method: POST
          json:
            name: test_css
            color: "${{color.cmyk | fake: {'format': 'css'}}}"
          check:
            status: /^20/

Request is:
POST https://httpbin.org/post HTTP/1.1
Content-Type: application/json

{"name":"test_css","color":"0.02,0.92,0.41,0.9}"}

Request should be:
POST https://httpbin.org/post HTTP/1.1
Content-Type: application/json

{"name":"test_css","color":"cmyk(100%, 0%, 0%, 0%)}"}

See https://fakerjs.dev/api/color.html#cmyk


Is this working under iTerm 3?

I'm planning to update my iTerm to the major release on this weekend.
Does anyone already did that? Is this repo fully functional?
Version 3.0.0 of iTerm2

This is a major new version of iTerm2. It contains hundreds of bug fixes and
many new features. All the details are at http://iterm2.com/version3.html

As far as I can tell, everything seems to work as expected

@emaiax Did you try it in iTerm 3? I guess we can say yes by now and just close this issue, what do you think?

Yes, it's working smoothly on iTerm 3, we can close this issue.
Thank!
—
Eduardo Maia
On Fri, 23 Sep 2016 at 06:55 Luiz Gonzaga dos Santos Filho
<
mailto:Luiz Gonzaga dos Santos Filho<EMAIL_ADDRESS>
wrote:

a, pre, code, a:link, body { word-wrap: break-word !important; }
https://github.com/emaiax
Did you try it in iTerm 3? I guess we can say yes by now and just close this issue, what do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly,
https://github.com/skwp/dotfiles/issues/712#issuecomment-249150938
, or
https://github.com/notifications/unsubscribe-auth/ABOtvXgO-AenT5tnoqYraIf_xeDhm-Ktks5qs6H3gaJpZM4IxYps
.

I can't do it by myself. Either you or @skwp must do it.

@lfilho closing now, thanks, bro!


[Suggestion] Add EOF to lexer

It is useful to be able to match the end of the string when lexing. Consider the following grammar in ebnf:
text_line ::= [a-zA-Z]* line_end
line_end ::= "\u000D\u000A" | "\u000A" | EOF

The grammar above will take any alphabetic characters until either a newline or the end of the file. As a result, a blank newline at the end of a line is optional.
This grammar can not accurately be represented in nimly because of the EOF. nimly can be expanded by adding the $ symbol to mean the end of input, similar to regex:
niml fluentLexer[MyToken]:
  "[a..zA..Z]":
    MyAlphaToken(token.token)
  "[\u000D\u000A|\u000A|$]":
    MyLineEndToken()

Thank you for your suggestion, @wyattjsmith1.
It sounds like a good idea that lexer produces a token which means EOF. But, it is not good that lexers recognize EOF as the same as other characters.
I suggest you wrapping lexIter (https://github.com/loloicci/nimly/blob/fa3a01ed8d51d0381a447a20c84459b664a13d1c/src/nimly/lexer.nim#L116) as it produces the token for EOF (in this example, MyLineEndToken()) after the original lexIter stops iteration.
Does it solve your problem?

Ah, ok. That should work. Thanks for the advice!


GAME-5286 - Celery Broker Supports Task Delay

Issue Link
https://forteio.atlassian.net/browse/GAME-5286
Acceptance Criteria

[ ] We have the ability to limit the rate at which celery processes tasks.

Testing Instructions
See: https://github.com/fortelabsinc/platform-sandbox/pull/2220
Potential Improvements
Contributer & Reviewer Quality Checklist
See RUST_BEST_PRACTICES.md for Rust-specific coding guidelines.

[ ] Branches follow the gitflow naming convention with Jira smart branching and in lower case (i.e. “feature/plt-122-brief-description-of-feature”). Note: feature can be replaced with hotfix/bugfix/release.
[ ] All Acceptance Criteria are completed.
[ ] Verify that submodules are up to date with latest changes from develop and not going backwards.
[ ] All tests are passing.
[ ] The most current version of develop is merged into the PR’s branch, and anytime before requesting re-reviews.
[ ] Check for merge conflicts.
[ ] Developer Comments have been added to ticket.

Post Submission

[ ] Change the label to Need Review.
[ ] Notify team that your PR is ready to review.

This is no longer needed.


Closing main window no longer minimizes to tray

I just updated to the latest master, and closing the main window no longer closes the player to tray.
I'll try to find what commit caused it (but it can also be caused by an Electron update).

aha, got it. That was easy.
It started after the very latest commit: 3597d506005bbefedc751c8befad665432b9b110
Got to revert that for my fork ....

Ahh, sorry, this issue is currently fixed in my local changes.  I'll push it tomorrow morning (about to sleep now)