Kubernetes deployment example for zqd

I’m investigating cloud deployment for some of the zq functionality. Deploying to a Kubernetes cluster on AWS, and connecting Brim to a zqd pod in the cluster, will be our starting point. In this context, zqd will only process data residing in S3, so this depends on issue “zqd support for S3 datadir #1051”.
I will create a branch called “k8s” for the on-going work, and create a PR when the branch has useful features and is stable. Part of it will be describing the procedures for creating a cluster and deploying zqd pods.
My current plan is:
(1) Create a “dev” cluster using AWS EKS. My EKS cluster will be visible to Brim employees, but other developers can use the instructions in the k8s/README to create their own cluster.
(2) Provide a Dockerfile to build an image for zqd.
(3) Provide Helm 3 templates to deploy the zqd image.
(4) Provide shell scripts that use the AWS CLI to expose a zqd endpoint using Route53 so we can easily connect to it with Brim. Initially we will try a NLB (AWS Network Load Balancer) to terminate TLS and expose the endpoint.

I will add comments here as work progresses in the k8s branch.
The README now has instructions for creating a local Kind cluster for testing. Because the primary use case for zqd in K8s is to search logs is S3, the local k8s cluster will not be the preferred way to use it, but it is helpful for development and simple testing of deployments.
The code in the branch right now is not yet useful, since its use case will depend on  #1052.


MacOS: VSCode won't update

VSCode Version: Code 1.12.1 (f6868fce3eeb16663840eb82123369dec6077a9b, 2017-05-04T21:18:32.269Z)
OS Version: Darwin x64 15.6.0
Extensions:

Extension
Author
Version

vscode-markdownlint
DavidAnson
0.8.0

vscode-devskim
MS-DevSkim
0.1.2

material-icon-theme
PKief
2.0.6

vscode-caniuse
akamud
0.5.3

vscode-svgviewer
cssho
1.4.0

phpunit
emallin
1.0.0

json-tools
eriklynd
1.0.2

php-debug
felixfbecker
1.10.0

php-intellisense
felixfbecker
1.2.2

php-symbols
linyang95
2.1.0

vscode-todo-parser
minhthai
1.8.10

debugger-for-chrome
msjsdiag
3.1.1

Ruby
rebornix
0.12.1

vsc-nwjs
ruakr
1.0.7

Align
steve8708
0.2.0

html-preview-vscode
tht13
0.1.1

Steps to Reproduce:

Wait for notification about update or check for updates.
Click on "Update Now".
VSCode restarts and it is the same version as before the update.

I'm running VSCode as a restricted user, maybe some permission errors... Console log contains some errors right after clicking Update Now button.
VSCode Update Log.log.zip

Unfortunately, the update never worked before...

Thanks for including the error messages. Looks like you don't have permission to update it.

Because it never asks for administrator rights. It would be great to do that in case of regular/restricted users, like many other applications do.

#7426


support validity period of Product Price

the fields validFrom and validTo of Product Price can neither be exported nor imported through this tool.
This information currently get's lost; even worse it leads to invalid prices since you'll get two or more prices with same currency and country for the same variant in the exported CSV file.
Currently each price has following pattern:
<country>-<currenyCode> <centAmount>|<discountedCentAmount> <customerGroupName>#<channelKey> that needs to be extended to accommodate additional validity period.

@stmeissner @hisabimbola @PhilippSpo - Please check my idea and add your comments before I start working on this.
Here is an export price mapping function:
https://github.com/sphereio/sphere-node-product-csv-sync/blob/master/src/coffee/exportmapping.coffee#L143
And here is a parsing regexp:
https://github.com/sphereio/sphere-node-product-csv-sync/blob/master/src/coffee/constants.coffee#L42
We can add another two special characters which will separate validFrom and validTo dates (saved in ISO 8601 format - eg. 2001-09-11T14:00:00.000Z).
What about this format?
<country>-<currenyCode> <centAmount>|<discountedCentAmount> <customerGroupName>#<channelKey>$<validFrom>~<validTo>

I think this is good...

Fixed by #151


Save as:: label in SaveDialog

Describe the bug
There is a double :: character in "Save as" label in SaveDialog control.
To Reproduce
Steps to reproduce the behavior:
Create a SaveDialog and SaveAs.
Expected behavior
it's working good, this is a minor quick change for a typo error.
Screenshots

Desktop (please complete the following information):
MacOS (Ventura 13.2.1)
NuGet version 1.10.0
Smartphone (please complete the following information):
Not mobile
Additional context
Add any other context about the problem here.

Thank you @tig, will do it, I would like to start learning more about the v2, where do you suggest me to start?

https://github.com/gui-cs/Terminal.Gui/discussions/1940


HDDS-9317. Provide an option for displaying Ozone snapshot diff output as JSON

What changes were proposed in this pull request?
As per HDDS-6637 and discussion, Ozone CLI is supposed to support JSON output format. And currently, SnapshotDiff CLI doesn't support that.
This change is to add the support of JSON format along with the existing format so that issues like HDDS-10834 can be avoided.
What is the link to the Apache JIRA
HDDS-9317
How was this patch tested?
Locally on docker.

Thanks for the patch @hemantk-12. Changes LGTM overall but could we add a robot test in snapshot-sh.robot using jq for better testing?

@Tejaskriya I've added acceptance tests as suggested. Please take another look.

@hemantk-12 Thanks for adding the tests! LGTM +1


Need  explanation about Output files

Thanks about the code, I need some information about the output files. For example for the chair class  of pcn there are 5 folders as pcd0 to pcd3 and pcdc with many files in each of them. First what are they and why are they in 5 different files and second, what kind of tools do you suggest for visualizing and considering them?
I found that all of the  output .h5 files  have the same data values!

Hi, pcdc, pcd0, pcd1, pcd2, pcd3 are Pc, P0, P1, P2, P3 in the paper.  You can change the core/inference_*.py file according to your need.
We use open3 for visualizing point cloud.

I found that all of the output .h5 files have the same data values!

Do you mean that .h5 files all have the same key 'data'? you can access the point cloud by
f = h5py.File('path_to_pcd.h5', 'r');
pcd = f['data'][:]

Hi, I tried to access the point clouds with the code you mentioned, but some files have no key! Or maybe libraries are needed to import or install. I ll be appreciate to let me know the solution.

Hi, I tried to access the point clouds with the code you mentioned, but some files have no key! Or maybe libraries are needed to import or install. I ll be appreciate to let me know the solution.

I'm sorry I never met problem like this, did you change the inference file? are all .h5 files have no key or just some of them?  my suggestion is to debug the inference file and check if the saving function (utils.io.IO.put) is working.

Hi, I tried to access the point clouds with the code you mentioned, but some files have no key! Or maybe libraries are needed to import or install. I ll be appreciate to let me know the solution.

hi,could you share me the dataset with me?my conmputer can"t download.Thank you very much!!!


Deep link Module with CR

Add a deep link for each module of Kyma to the corresponding CR.
The information is in ModuleTemplate that is mentioned in the status of the modules
Prepare it for the details page and injection widget

No longer valid, we have separate view for modules.


$this->Model->find('list'); fails on models with permissionable attached

When trying to do a $this->Model->find('list'); on a model that has the permissionable beavior attached it fails with the following SQL error: (this is for a model called Location
Warning (512): SQL Error: 1051: Unknown table 'LocationPermissionBit' [CORE/cake/libs/model/datasources/dbo_source.php, line 684]
Query: SELECT `Location`.`id`, `Location`.`name`, `LocationPermissionBit`.* FROM `locations` AS `Location`   WHERE 1 = 1

I'm getting the same thing.  I set the params variable to array('permissionable'=>false) to get it to work properly.

I'm getting the same issue on mysql,
SELECT Store., StorePermissionBit. FROM stores AS Store WHERE 1 = 1 ORDER BY number asc LIMIT 20
1051: Unknown table 'StorePermissionBit'
Any workaround that doesn't disable the permissionable behavior like above?
Thanks

Hi guys, as a workaround you can put the following in your app model, credit to @darxmac for tracking down recursive=>0 being the problem:
       switch ($type) {
           case 'list':
   		      $options['recursive'] = 0;
   	          return parent::find('list', $options);
           break;
           default:
               return parent::find($type, $options);
           break;
       }
   }```


Add packagePort and remotePort options

Fixes #29.

@chrisdp @triwav @georgejecook @lvcabral, how do you feel about the following property names?

packagePort - anything that relates to current Roku port 80
remotePort - anything related to sending remote commands (port 8060).


Add bioconda recipe to bring the package to bioconda

generate meta.yaml file using grayskull
open PR to bioconda recipes
add badge to readme file

Resolved in https://github.com/bioconda/bioconda-recipes/pull/49514


Using '*' and '/' together in the Validator attribute path is not handled correctly

Laravel Version
10.24.0
PHP Version
8.2.10
Database Driver & Version
No response
Description
If you pass a path using * and / such as *.email/id to the second argument of \Illuminate\Validation\Validator::make, an error will occur at the following location.
https://github.com/laravel/framework/blob/v10.24.0/src/Illuminate/Validation/ValidationRuleParser.php#L152
> Validator::make([['email/id' => '1']], ['*.email/id' => 'required|string'])->validate()

   WARNING  preg_match(): Unknown modifier 'd' in vendor/laravel/framework/src/Illuminate/Validation/ValidationRuleParser.php on line 152.

= []

> Validator::make([['email|id' => '1']], ['*.email|id' => 'required|string'])->validate()
= [
    [
      "email|id" => "1",
    ],
  ]

> Validator::make(['email/id' => '1'], ['email/id' => 'required|string'])->validate()
= [
    "email/id" => "1",
  ]

In the following places, preg_quote is used to escape special characters, but / is not escaped.
https://github.com/laravel/framework/blob/v10.24.0/src/Illuminate/Validation/ValidationRuleParser.php#L147
https://www.php.net/manual/en/function.preg-quote.php

Note that / is not a special regular expression character.

So I think '/' needs to be escaped individually.
preg_match('/^'.$pattern.'\z/', $key)

preg_match('/^'.preg_match('/^'.str_replace('/', '\/', $pattern).'\z/', $key)'\z/', $key)

Steps To Reproduce
Run the following
 Validator::make([['email/id' => '1']], ['*.email/id' => 'required|string'])->validate()

This is not supported sorry. Please refrain from using characters like / in your validation keys.


Screenshots Tools

Description of the new feature / enhancement
Could please provide screenshot tool which better than PrtSc button?
Scenario when this would be used?
I think this is a very useful tool in daily life.
Supporting information
Look like snipaste.

You can use Snipping tool on Windows 11. Or download it for Windows 10 from the store.

Any update with this feature?
Really need screenshot tools with crop, resize, annotation, blur, etc.

Any update with this feature?
Really need screenshot tools with crop, resize, annotation, blur, etc.

@siunus As I mentioned snipping tool has some of these features. For feedback for snipping tool please use the Feedback Hub (Link only opens on Windows)

I think this is a /dup #11530 (opened on May 31, 2021), and the answer is the same. PowerToys is not intended to replace the standard Windows tools. If these standard tools do not work for you, request an improvement from Microsoft via the Microsoft Feedback Hub.


startup loop in the k8s scheduler could loop forever

The loop is in #172 here. As @helgi comments, the loop should not loop forever according to k8s pod phases, but the loop should be made more resilient anyway, for defensive purposes.

closing in favor of #406 which addresses this issue


microk8s dbctl backup panics: etcdwrapper: key not found

When running microk8s.dbctl backup on a 3 node HA cluster (v1.20.7), I get the following error:
root@node13:~# microk8s.dbctl backup
Backing up the datastore
INFO[0000] mode: backup-dqlite, endpoint: unix:///var/snap/microk8s/current/var/kubernetes/backend/kine.sock, dir: /tmp/tmpyfpmjp9c/backup-2021-06-15-10-25-47
panic: etcdwrapper: key not found

goroutine 1 [running]:
main.check(...)
	/build/microk8s/parts/migrator/build/main.go:75
main.backup_dqlite(0xc36b28, 0x42, 0x7ffcdfc55396, 0x2b, 0x0, 0x0)
	/build/microk8s/parts/migrator/build/main.go:182 +0x887
main.run(0xc0000d0dc0)
	/build/microk8s/parts/migrator/build/main.go:283 +0x606
github.com/urfave/cli.HandleAction(0xb22fa0, 0xc441e8, 0xc0000d0dc0, 0xc0000d0dc0, 0x0)
	/build/microk8s/parts/migrator/go/pkg/mod/github.com/urfave/cli@v1.22.4/app.go:528 +0x58
github.com/urfave/cli.(*App).Run(0xc0000d6380, 0xc0000320a0, 0x5, 0x5, 0x0, 0x0)
	/build/microk8s/parts/migrator/go/pkg/mod/github.com/urfave/cli@v1.22.4/app.go:288 +0x655
main.main()
	/build/microk8s/parts/migrator/build/main.go:68 +0x4f6
Backup process failed. 2

Here is the inspection report of my cluster:
inspection-report-20210615_102327.tar.gz
Thank you

same issue found in v1.23.0

Glad I found this issue, as I was about to create the same. I had been running a nightly microk8s dbctl backup on a non-HA cluster for probably 6 months or so. I recently got my cluster up to 3 nodes and it went into HA mode. Within a few days the backup begin failing with the etcdwrapper: key not found error.
One bit of additional info is that the key that's not found corresponds to the pod for a frequently running CronJob CRD. So perhaps a race condition somewhere creating the inconsistency. I've confirmed that Pod for the missing key isn't something that's visible in the cluster currently.
The captured output corresponding to the issue:
time="2023-05-01T00:02:00-07:00" level=fatal msg="couldn't get key /registry/pods/<namespace>/<cj>-cron-28048730-pksls: etcdwrapper: key not found"

Hi @jdewald thanks for raising this, looks like it was missed with the stale bot.
Indeed, there is an issue with the dbctl command when a key is deleted most likely while the backup operation is on the way, causing the error message that you have observed.
The correct way to handle this would be to improve the dbctl command to take the database revision into account while backing up the cluster. Reopening so that we deal with it.

...
Indeed, there is an issue with the dbctl command when a key is deleted most likely while the backup operation is on the way, causing the error message that you have observed.

Based on the above, I just confirmed that the actual key that's missing varies on each run (I hadn't been paying attention and assumed it was always the same one since it was the same frequently-running CronJob).
So I do at least know how to work around the issue a bit if needed until fix is available!


Set language for verifyPhoneNumber verification code template

Is it possible to have dynamic language for verification code template?

Feature request

Please post a link to the native API documentation which provides this functionality

Closed due to no response


Add new callout to warming-up-an-ip-address.md

Description of the change:
Add new callout to warming-up-an-ip-address.md
Reason for the change:
New callout!
Link to original source:
https://sendgrid.com/docs/ui/sending-email/warming-up-an-ip-address/

Closes #

Hello @music-mind,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Full tag claims@winterrowd

This didn't have some changes merged in. It was easier to just create #139 than fix this one. This PR is abandoned.


Add some common objc nits to the style guide

@nicklockwood @a2 @vjeux Want to add something else? (or remove something)

@tadeuzagallo - to JS, we could add something like:

Leave a trailing comma after the list item in object literals
Use object destructuring assignment where possible

For objc, do you have a preference on using [x setVar:y] vs x.var = y? Might be worth picking one to avoid any future bike shedding.

It's difficult to know what level of detail to go into with something like this. We do have very specific style guidelines for Objective-C code, but they'd probably run to a dozen pages if we mentioned everything!
The [x setVar:y] vs x.var = y is a good one (we prefer the latter, but only for @properties, not any method with no arguments).
Besides that, may as well merge this as is and we can expand on it later as and when issues arise.


Canvas Mode: RenderTexture "fill" method does not set "globalCompositeOperation"

Version

Phaser Version: 3.55, 3.60.0-beta
Operating system: Windows, macOS
Browser: Chrome, Firefox
Phaser Rendering Mode: CANVAS

Description
Using fill method in RenderTexture does not set CanvasRenderingContext2D.globalCompositeOperation, resulting in reusing value set by last drawn gameObject.
rt.fill(0x0000ff);
rt.erase(...);
rt.draw(...);
console.log(rt.context.globalCompositeOperation); // "source-over"
rt.fill(0x0000ff);

acts as expected, resulting in filling RenderTexture with color.

Reversing order of draw end erase methods, causes fill to remove pixels from whole RenderTexture.
rt.fill(0x0000ff);
rt.draw(...);
rt.erase(...);
console.log(rt.context.globalCompositeOperation); // "destination-out"
rt.fill(0x0000ff);

Example Test Code
https://codepen.io/mkchomik/pen/QWQaxbz
Additional Information
Digging in the codebase, I found that value of globalCompositeOperation is set in SetTransform function:
https://github.com/photonstorm/phaser/blob/master/src/renderer/canvas/utils/SetTransform.js#L47
I was thinking if doing ctx.save() right before setting blendMode and globalAlpha would help, however I am not sure if it doesn't break other parts of code.

wow you see this

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


Article missing explanation of one scenario

This article is very well done.  But I feel like there's one solution missing.  If you look at the 2nd solution, it talks about having an on-premises network use a DNS Forwarder in Azure using server level forwarding.  But in that same on-premises scenario, it then also talks about using a conditional forwarder if you already have a custom on-premises DNS solution.
The last scenario talks about a hybrid approach using a Server level forwarder that exists inside Azure.  However, many hybrid environments have Domain Controllers or a custom solution that synchronizes DNS in both on-premises and Azure.  Therefore, I think there should still be a 2nd scenario for the last hybrid approach in which you still leverage conditional forwarding even with an on-premises network, a hub vnet, and peered vnets.

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

ID: a88af182-32aa-a8f3-359d-b92ae1cfb8c7
Version Independent ID: 01ce07fc-4bc2-def5-9ecd-a80330ad9488
Content: Azure Private Endpoint DNS configuration
Content Source: articles/private-link/private-endpoint-dns.md
Service: private-link
GitHub Login: @asudbring
Microsoft Alias: allensu

@ElanShudnow Thank you for sharing your query! We are currently investigating and will get back to you shortly on this.

Sending to PM for review
#reassign:@ivapplyr


raw-loader

For some use cases it is valuable to have raw file contents included in the build process.  For one such example, lets say we have a  component which also has a show source view.
<Example>
  <LivePreview />
  <ShowSource fileContents={livePreviewContents} />
</Editor>

If as part of the build process one could require raw files, it would allow such use cases.

Similar in ask to the proposal over in create-react-app. https://github.com/facebook/create-react-app/issues/3722


Vulnerability roundup 98: flatbuffers-1.12.0: 1 advisory [7.5]

search, files

[ ] CVE-2020-35864 CVSSv3=7.5 (nixos-20.09, nixos-unstable)

Scanned versions: nixos-20.09: aa5b9cd16b9; nixos-unstable: 257cbbcd3ab.
Cc @teh

false positive; the CVE is about the Rust binding

false positive; the CVE is about the Rust binding


Fix broken card in demo

New flat tool style for the ListBox broke a card in the demo. This is a tiny fix for that.

Damn, it's broken more than just that card :/

Gonna revert; and fix underlying issue.


[ARM] policy definition: example using parameters

Needs an example showing how to create a policy definition that uses parameters.

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

ID: 436bf02d-dfe3-ef6e-2be8-1bd7b1f39623
Version Independent ID: e8b92df3-9e3d-c4c7-1965-0b6b3f7d00ea
Content: az policy definition | Microsoft Docs
Content Source: src/command_modules/azure-cli-resource/azure/cli/command_modules/resource/_help.py
Service: azure-resource-manager
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Discovered that the "Create a policy parameter definition with the following example" is an incorrect example. Parameter JSON cannot be passed to --rules parameter.
az policy definition create -n allowedLocations --rules '{
                            "allowedLocations": {
                                "type": "array",
                                "metadata": {
                                    "description": "The list of locations that can be specified when deploying resources",
                                    "strongType": "location",
                                    "displayName": "Allowed locations"
                                }
                            }
                        }'

Fixed by #7470


DST creation code duplication

As seen in https://github.com/hermitcore/libhermit-rs/pull/709, the code for creating DSTs for FUSE is duplicated a lot.
We should add some kind of common generic method for creating those.

Not sure if this is a good first issue or not. Considering that this might not be straight forward and might require touching and reworking more than meets the eye, this might not be a very good first issue. Still, the scope of this issue is more limited than that of others.


Extend dask.array no-chunk solution

Recently in https://github.com/dask/dask/pull/1838 we allowed dask.arrays to exist with unknown chunk sizes.  There is still some work to do here:

Support more operations like the following:

reshape
slicing dask.arrays with other dask.arrays (an often requested feature)
...

Document the feature and its limitations

It appears that .values with dask DataFrames doesn't work as I would expect with addition / subtraction of numpy arrays.  Example:
A = dd.from_array(np.array([[0,0], [1,1]]))
B = da.from_array(np.array([[0,0], [1,1]]), chunks=2)
A.columns = ['x', 'y']

B[:,0] - np.zeros(2) ## works fine, and as expected
A.y.values.compute() - np.zeros(2) ## works fine, as expected
A[['y']].values - np.zeros(2) ## works, broadcasts to create (2,2) array
A.y.values - np.zeros(2) ## throws a ValueError

ValueError: operands could not be broadcast together with shapes (nan,) (2,)

In the A[['y']].values case we know the shape to be (nan, 2) (we know the number of columns, but not rows) so subtracting a (2,) array works because of broadcasting.  In the A.y.values case we don't know the number of rows, so I think it's proper to err here.
In this particular case we do know the number of rows, and so could set chunks explicitly
v = A.y.values
v._chunks = ((2,),)

But this is a special case and not something that we can guarantee generally.  Going from dask.arrays to dask.dataframes and back will lose information.  Dask.array reserves the right to err rather than hope that things are correct in some cases.

I've implemented slicing.  It would be nice to also implement reshape (this comes up decently often.)

Not sure if this should be a new issue or if this is an ok place to add this idea, but it would be nice to have stack and concatenate work when dimensions are unknown. Admittedly this is not easy to solve in the general case as one doesn't know the length of the unknown dimensions. So one can't verify that they are the same. However if there was some way to mark the unknown length dimensions as the belong to the same group of unknown lengths, then it might be possible to validate stack and concatenate in the general case. Though there may be even simpler solutions for specific cases.

It would not be unreasonable to just assume that they matched and err at
runtime if they don't.  We do similar things in dask.bag
On Fri, Jul 28, 2017 at 11:44 AM, jakirkham<EMAIL_ADDRESS>wrote:

Not sure if this should be a new issue or if this is an ok place to add
this idea, but it would be nice to have stack and concatenate work when
dimensions are unknown. Admittedly this is not easy to solve in the general
case as one doesn't know the length of the unknown dimensions. So one can't
verify that they are the same. However if there was some way to mark the
unknown length dimensions as the belong to the same group of unknown
lengths, then it might be possible to validate stack and concatenate in
the general case. Though there may be even simpler solutions for specific
cases.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/1839#issuecomment-318688633, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszBmGONtWheaDRA9jTKdoyXqElkAxks5sSgHpgaJpZM4LJztO
.

Sure that could be reasonable too and probably easier to implement. If we go that way, it's probably worth raising some kind of warning when concatenating or stacking any arrays with unknown dimensions.

Historically we have handled this by raising an exception if, while
computing the graph, we notice that the chunks don't line up.  This could
be changed to something else though.
On Fri, Jul 28, 2017 at 12:27 PM, jakirkham<EMAIL_ADDRESS>wrote:

Sure that could be reasonable too and probably easier to implement. If we
go that way, it's probably worth raising some kind of warning when
concatenating or stacking any arrays with unknown dimensions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/1839#issuecomment-318700132, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszL0ptE854xFnQEJLQJxAeK-UtJ8Sks5sSgwLgaJpZM4LJztO
.

Much of this has been implemented.  It's a bit open ended.  Closing.


When analyzing from a certain date, all files from before that date are missing

To reproduce, do the following via REST API:

create a project
add an analyzer
add a file set
set an AnalyzingStrategy with a start date, meaning to analyze from this date forward

Actual Result:
Only files that were changed after that date are being analyzed. In the JSON returned from the /deltaTree REST endpoint, this results in the following:

commit1Metrics are null for all files
all files from before the start date are missing

Expected Result:

commit1Metrics should only be null for files that were added with the second commit passed into the /deltaTree endpoint
all files that existed before the start date should be listed with the metric values from the commit previous to the first commit passed into the /deltaTree endpoint).

to fix this, you have to reset your AnalyzingStrategy to have all commits analyzed again


[DNM] Running autopep8 and autoflake against whole project

Updating tox.ini with flake8 styleguide
68aae83e
Updating tox.ini with flake8 styleguide
fce07b6e
Running 'autopep8 -i tests//.py'
96805c9a
Ignoring PEP E731 (lambdas to defs)
e9587fb1
Running autopep against whole codebase
96a3b442
Removing unused imports
c4bbf0f6
Removing unused variables
e1b6370c
Removing all unused imports

Coverage increased (+0.006%) to 92.138% when pulling e1b6370c8da4f1865f5d20041ee7f1f29ceb3356 on JackDanger:jack/autopep-project into 101210e4d4cb14d83eceb6ba29f543db7eddb733 on spulec:master.


Contains the content of the V-IF, and the order causes a rendering exception

Version
2.6.13
Reproduction link
https://codepen.io/hasasa/pen/zYZWRRE
Steps to reproduce
the firs button has "v-if";
the second button is normal
What is expected?
Content without a judgment condition is rendered normally in each column
What is actually happening?
Normally, button 2 should be in every column. In fact, when there is the content of the judgment condition. Cannot render properly without the normal content of the judgment condition.
When the content without a judgment condition is placed in the first position, the rendering is normal

#12104

+1

Always search existing issues first


loopback_ipv4_address under node: not resolve the variable input

Issue Summary
The dict key "loopback_ipv4_address" not resolve the inputs as variable, while other keys do, like "mgmt_ip" and "bgp_as".
the AVD inputs:
l3leaf:
defaults:
platform: cEOS
loopback_ipv4_pool: "{{ NETv4_LPBK_ULAY }}"
loopback_ipv4_offset: 0
...
nodes:
- name: cleaf01
id: 1
loopback_ipv4_address: "{{ NETv4_LPBK_ULAY | ansible.utils.ipaddr(2) | ansible.utils.ipaddr(address) }}"
mgmt_ip: "{{NETv4_MGMTA | ansible.utils.ipaddr(seq_id) }}"
bgp_as: "{{ bgp_asn_base + dc_id*100 + seq_id }}"
Generate CFG
interface Loopback0
description EVPN_Overlay_Peering
no shutdown
ip address {{ NETv4_LPBK_ULAY | ansible.utils.ipaddr(2) | ansible.utils.ipaddr(address) }}/32
!
Which component(s) of AVD impacted
eos_cli_config_gen
How do you run AVD ?
Ansible CLI (with virtual-env or native python)
Steps to reproduce
# L3 Leaf switch group
l3leaf:
  defaults:
    platform: cEOS
    loopback_ipv4_pool: "{{ NETv4_LPBK_ULAY }}"
    loopback_ipv4_offset: 0
    vtep_loopback_ipv4_pool: "{{ NETv4_LPBK_OLAY }}"
    uplink_switches: ['spine01', 'spine02']
    uplink_ipv4_pool: "{{ NETv4_PTP }}"
    virtual_router_mac_address: 00:1c:73:{{'%02d' % rack_id}}:{{'%02d' % rack_id}}:99
    spanning_tree_priority: 4096
    spanning_tree_mode: mstp

  nodes:
    - name: cleaf01
      id: 1
      loopback_ipv4_address: "{{ NETv4_LPBK_ULAY | ansible.utils.ipaddr(2) | ansible.utils.ipaddr(address) }}"
      mgmt_ip: "{{NETv4_MGMTA | ansible.utils.ipaddr(seq_id) }}"
      bgp_as: "{{ bgp_asn_base + dc_id*100 + seq_id }}"
      uplink_switch_interfaces:
        - Ethernet1
        - Ethernet1

Relevant log output
interface Loopback0
   description EVPN_Overlay_Peering
   no shutdown
   ip address {{ NETv4_LPBK_ULAY | ansible.utils.ipaddr(2) | ansible.utils.ipaddr(address) }}/32
!

Contributing Guide

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

My guess is that address needs to be enclosed in quotes to be picked up as a string instead of an undefined variable.
Unfortunately Ansible stops templating of vars (silently) if any errors are hit.

Recreating the issue:

Working example:

@jixh99 closing this stale issue. Update here if it is still a problem.


Wrap sd.cpp

We should wrap sd.cpp api into Alpaca Core's Model, Instance objects as we did for whisper and llama. This will produce ac-sd inference library for examples/tests.

Basic wrap is done, we have text to image conversion. Will implement image to image and passing all of the parameters. The following implementation will be tracked in separate issues.


We can't use this library with the microsoft AddDbContextCheck

When the dbcontext configuration is set to scoped then you can't have both working as same dbcontext is used and so as healthcheck are execute in parallel task it's  throwing errors.
The fix is to create a scope when creating the context.
There is an issue on AspNetCore repository https://github.com/dotnet/aspnetcore/issues/14453

Thanks !

Thanks !


expose palleteclick event

Any chance of exposing the paletteElementClick event similar to how you did the dragend and dragstart?

Nevermind, I realized this is exposed via the change event


Skin support final implementation

Hey @raysan5 as stated in https://github.com/raysan5/raygui/pull/98 here is another pull request adding support for skinning. Compared to my previous attempt this time i made sure to fix most of the bugs and also included a fully functional skin editor in /examples/ to make creating/editing skins a bit easier.
Here are some features:

supports almost all the raygui controls except GuiLine(), GuiGroupBox(), GuiLabel(), GuiGrid(), GuiLabelButton() and GuiDummyRec()
some controls have multiple sub components (ex GuiListViewEx() ) those can be skinned too
a skin file can hold multiple skin styles, this way one can have different skins for the menu GUI, ingame GUI etc. in the same file
support for saving loading skins to/from a text file, the syntax resembles .rgs as much as possible
if useColor is enabled for a skin it can use the currently active color style

Using it in your code is quite easy i think here's an example:
// Loading from a file
GuiSkin skins[4] = {0}; // create an empty array to hold our skins
int skinCount = GuiLoadSkin("test.skin", skins, 4); //load at most 4 skins from file
if(skinCount == 0) TraceLog(LOG_ERROR, "Failed to load skins!"); // handle failure

GuiSetSkin(&skin[0]); // set the first skin as the global skin
GuiButton(bounds, "Hello"); // render a button using the first skin
GuiSetSkin(&skin[1]);
GuiButton(bounds, "World"); // render a button using the second skin
// ....

// `GuiLoadSkin()` will automatically load a texture file but you can also use your own
GuiSetSkinTexture(mytexture); // now mytexture is the global gui texture.

// Saving a skin to file
GuiSaveSkin("test.skin", "texture.png", skins,  4); // Save 4 skins styles to file

Please try the editor to see more of what the current implementation can do!
I think this feature will benefit raygui greatly, so what do you say Ray :+1: or :-1:
PS: i don't know why d321a0b and 37f20a4 are in there :(

@Demizdor Wow! Really impressive! Congrats! This is a huge change for the library, I need some time to review it and decide...
In any case, thanks for this great feature and tool!

Sure thing take your time, the update is huge and you'll need to maintain in the future so i fully understand.

Hey @raysan5 , i will close this!
The new raylib changes broke this code.
Some things are easy to fix like getting rid of the #ifdef __unix__ hacks adding the dot when calling GetExtension() in editor.c and correcting DrawTextureNPatch()  to use NPATCH_NINE_PATCH in raygui.h but others are harder like the newly patched GuiScrollPanel() that made drawing the edges not behave correctly.
I'm sure if i had time everything would work once again but i don't have time to fix things, so i believe this is the best way to handle all of this, sorry!

Hi @Demizdor, ok, no sorry, it's ok. It was a big change and I haven't put much time into raygui lately. I also admit I felt a bit scary about maintaining it... No worries, the PR is here for future reference.


modulus login results in "TypeError: Cannot read property 'api_uri' of undefined"

I just downloaded the latest modulus code from Github, and when I ran
"modulus login", I get an error that:
TypeError: Cannot read property 'api_uri' of undefined

Here's full stack returned:
$ modulus login

/usr/lib/node_modules/modulus/lib/common/api.js:8
if (typeof userConfig.data.api_uri !== 'undefined') {
                          ^
TypeError: Cannot read property 'api_uri' of undefined
    at Object.<anonymous> (/usr/lib/node_modules/modulus/lib/common/api.js:8:27)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/usr/lib/node_modules/modulus/lib/modulus.js:2:18)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)

It's not clear what I'm supposed to do from here to continue logging in.

Please help me out, i am getting following error if i try to login in modulus.
TypeError: Cannot read property 'user' of undefined
at module.exports.server.get.res.render.meta.title (/opt/modulus/aperture/build/server/lib/routes.js:5308:18)
at callbacks (/opt/modulus/aperture/build/server/node_modules/express/lib/router/index.js:161:37)
at param (/opt/modulus/aperture/build/server/node_modules/express/lib/router/index.js:135:11)
at pass (/opt/modulus/aperture/build/server/node_modules/express/lib/router/index.js:142:5)
at nextRoute (/opt/modulus/aperture/build/server/node_modules/express/lib/router/index.js:100:7)
at callbacks (/opt/modulus/aperture/build/server/node_modules/express/lib/router/index.js:164:11)
at module.exports.checkRedis (/opt/modulus/aperture/build/server/lib/routes.js:5228:5)
at callbacks (/opt/modulus/aperture/build/server/node_modules/express/lib/router/index.js:161:37)
at param (/opt/modulus/aperture/build/server/node_modules/express/lib/router/index.js:135:11)
at pass (/opt/modulus/aperture/build/server/node_modules/express/lib/router/index.js:142:5)

Sorry about that, the API server was having issues that caused this. It's all fixed up now.

Thanks @fiveisprime for quick response. It's working now.


Version 4.2.2

Fixing #141 default_box doesn't first look for safe attributes before falling back to default (thanks to Pymancer)
Changing from TravisCI to Github Actions
Removing pep8speaks in favor of flake8 in Github Actions

Hello @cdgriffith! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file test/test_converters.py:

Line 84:114: W292 no newline at end of file

Coverage increased (+0.0006%) to 99.875% when pulling 20dbdb72486888ae532322270aae4c3127fb05e7 on development into ba48dc79d913771813ea10d20c1b658572e5915b on master.

Coverage decreased (-0.6%) to 99.247% when pulling 20dbdb72486888ae532322270aae4c3127fb05e7 on development into ba48dc79d913771813ea10d20c1b658572e5915b on master.


Bugfix for Manager timing out on high intial url/request count

This is a bugfix where the Manager crashes due to timing out on the init callback, especially when there is a high number of start requests/urls.
This PR implements a split strategy for storing urls/requests using both sync and async methods, by storing the first 1000 requests and firing off a linked task that adds the remaining requests.

Thanks for the PR @Ziinc. Indeed the case with bug number of start URLs is currently incompatible with Crawly.
However, as soon as we're going to fix it anyway I want to suggest the following:

We should simplify init. Long term operations in init may cause crashes depending on supervision strategies. So I was thinking about using handle_continue (or init_ack + enter_loop) for URLs initialization;
I think at this point we should also discuss a possibility to support incoming streams of data. As having 400_000 URLs list in memory does not sound quite right
Regarding the async Tasks - I would try to avoid having them in the Manager if possible, however, it might be a topic for further investigation.

OK. for (1), will refactor to utilize handle_continue.
For (3), since the start urls/requests are one-off insertions, I think placing them in the manager is fine. We add async bulk request insertion to the RequestStorage, if that is what you mean.
Will open up separate issues for (2) once this is merged.


The Overall duration get totally wrong in some containers (TS and Matroska)

These files are not truncated, at least not showed as it by MediaInfo:
darkgraysubs.m2ts.txt
MKV-H264-AACHE-2ch (2).mkv.txt
Flash.flv.txt
General
Unique ID                                :<PHONE_NUMBER>55175913603379713516689528352 (0x86D32D04B9CC2541A2050CCC06DD9620)
Complete name                            : F:\!MEDIA_TEST_FILES\UMS Video Testing Suite (20150121)\TESTS\INTERNAL SUBTITLES\MKV-H264-AACHE-2ch (2).mkv
Format                                   : Matroska
Format version                           : Version 4 / Version 2
File size                                : 4.38 MiB
Duration                                 : 13 h 15 min
Overall bit rate                         : 770 b/s
Encoded date                             : UTC 2016-11-27 16:35:52
Writing application                      : mkvmerge v9.5.0 ('Quiet Fire') 32bit
Writing library                          : libebml v1.3.4 + libmatroska v1.4.5

Video
ID                                       : 1
Format                                   : AVC
Format/Info                              : Advanced Video Codec
Format profile                           : High@L4
Format settings                          : CABAC / 5 Ref Frames
Format settings, CABAC                   : Yes
Format settings, RefFrames               : 5 frames
Codec ID                                 : V_MPEG4/ISO/AVC
Duration                                 : 11 s 428 ms

As i don't want open issues if not necessary, i prefer add these durations issues here.
This one is weird with a 12s duration reported by the players, perhaps some truncation not detected ?
MediaInfo show
LastManStanding.dvr-ms.txt
This one have only the general overall duration, not the audio or video ones: (truncated)
Francis en la Marina (1955)_cut.mkv.txt
This one miss the video duration, but have the general and audio ones (truncated file):
audio-issue-sample-file.mkv.txt
This one miss only the audio duration (truncated file):
j-b-avc-dts-x-ums-sample-file_Truncated.mkv.txt

Regarding MKV-H264-AACHE-2ch (2).mkv: The file is strange: First you have normal video+audio+subtitles for a bit more than 11s; then you sporadically have subtitle packets:
I frame, track 2, timestamp 00:00:11.209000000, size 1033, adler 0x0fb302e1
I frame, track 2, timestamp 00:00:11.209000000, size 1059, adler 0x65090a83
I frame, track 2, timestamp 00:00:11.209000000, size 136, adler 0xafc33cb6
P frame, track 1, timestamp 00:00:11.220000000, size 21052, adler 0x064e01fb
B frame, track 1, timestamp 00:00:11.136000000, size 13482, adler 0x86288c07
B frame, track 1, timestamp 00:00:11.094000000, size 10149, adler 0x8bab9b2b
P frame, track 1, timestamp 00:00:11.178000000, size 12079, adler 0x11ad950c
P frame, track 1, timestamp 00:00:11.386000000, size 18233, adler 0xe0888a39
B frame, track 1, timestamp 00:00:11.303000000, size 14303, adler 0xae25c587
B frame, track 1, timestamp 00:00:11.261000000, size 12441, adler 0x664e4da4
P frame, track 1, timestamp 00:00:11.345000000, size 10733, adler 0x1210dd03
I frame, track 4, timestamp 00:00:12.050000000, size 36, adler 0x03510d34
I frame, track 4, timestamp 00:00:12.315000000, size 71, adler 0xdf211af0
I frame, track 4, timestamp 00:00:13.770000000, size 36, adler 0x0d0d0cde
I frame, track 4, timestamp 00:00:14.035000000, size 71, adler 0xdddd1a5f
I frame, track 4, timestamp 00:00:18.530000000, size 36, adler 0x0dbd0d0c
I frame, track 4, timestamp 00:00:18.795000000, size 71, adler 0xdee41aca
I frame, track 4, timestamp 00:00:21.250000000, size 36, adler 0x0e250d37
I frame, track 4, timestamp 00:00:22.275000000, size 71, adler 0xdda01a39
I frame, track 4, timestamp 00:00:24.930000000, size 36, adler 0x0ed50d65
I frame, track 4, timestamp 00:00:25.195000000, size 71, adler 0xad7119d2
I frame, track 4, timestamp 00:00:28.930000000, size 36, adler 0xfaae0cbe
I frame, track 4, timestamp 00:00:29.195000000, size 71, adler 0xaf291a40
I frame, track 4, timestamp 00:00:30.890000000, size 36, adler 0xfb5e0cec
I frame, track 4, timestamp 00:00:31.155000000, size 71, adler 0xad3419ac
I frame, track 4, timestamp 00:00:32.450000000, size 36, adler 0xf8ca0c18
I frame, track 4, timestamp 00:00:32.715000000, size 71, adler 0xaeec1a1a
I frame, track 4, timestamp 00:00:36.215000000, size 36, adler 0xf97a0c46
I frame, track 5, timestamp 13:15:21.814000000, size 71, adler 0x8f621932
I frame, track 5, timestamp 00:00:09.955000000, size 36, adler 0xef260c54
I frame, track 5, timestamp 00:00:29.955000000, size 71, adler 0x8d6d189e
I frame, track 5, timestamp 00:00:39.955000000, size 36, adler 0xec920b80
I frame, track 5, timestamp 00:00:59.955000000, size 71, adler 0x8f25190c
I frame, track 5, timestamp 00:01:09.955000000, size 36, adler 0xed420bae
I frame, track 5, timestamp 00:01:29.955000000, size 71, adler 0xadb619f5
I frame, track 5, timestamp 00:01:39.955000000, size 36, adler 0xf9fa0c57
I frame, track 5, timestamp 00:01:59.955000000, size 71, adler 0xac721964
I frame, track 5, timestamp 00:02:09.955000000, size 36, adler 0xfaaa0c85
I frame, track 5, timestamp 00:02:29.955000000, size 71, adler 0xad7919cf
I frame, track 5, timestamp 00:02:39.955000000, size 36, adler 0xfb120cb0
I frame, track 5, timestamp 00:02:59.955000000, size 71, adler 0xac35193e
I frame, track 5, timestamp 00:03:09.955000000, size 36, adler 0xfbc20cde
I frame, track 5, timestamp 00:03:29.955000000, size 71, adler 0x7c0618d7
I frame, track 5, timestamp 00:03:39.955000000, size 36, adler 0xe4ae0b38
I frame, track 5, timestamp 00:03:59.955000000, size 71, adler 0x7dbe1945
I frame, track 5, timestamp 00:04:09.955000000, size 36, adler 0xe55e0b66
I frame, track 5, timestamp 00:04:29.955000000, size 71, adler 0x7bc918b1
I frame, track 5, timestamp 00:04:39.955000000, size 36, adler 0xe5c60b91

As you can see track 5 has a packet at 13:15:21.814 which explains why the duration in the header (which MediaInfo seems to rely on) is so big.
Btw: 13:15:21 is very close to half the wraparound time for timestamps in a transport stream. This is in all likelihood a result of a bug in mkvmerge.


avoid unnecessary COUNT queries when exporting

Problem
Fix for #1714
Solution
Described at #1714
Acceptance Criteria
I didn't yet write tests and documentation. I would like to get some feedback first.

coverage: 100.0%. remained the same
when pulling 00e7b9c2dea1ec82ea39a3d957c0ed3142595c63 on PetrDlouhy:remove_count_queries
into b63ddd6148c786573838fbfc0e30131dfdef3a71 on django-import-export:main.

It sounds interesting, thanks for picking it up.
Does the FakePaginator break pagination when we have larger datasets?

@matthewhegarty I didn't know about that functionality, but according to my tests it seems the chunking works OK. If you look at the code (https://github.com/django-import-export/django-import-export/blob/b63ddd6148c786573838fbfc0e30131dfdef3a71/import_export/resources.py#L1087) it seems that django-import-export creates it's own Paginator which is independent on the Paginator created by ModelAdmin.
Also I hope the chunking is already covered by unit tests.
Here is comparison of the called queries after this change:

And before this change:

In both cases there is query for each chunk, but in the first case the COUNT queries are missing.
BTW: I was confused for a while how the chunking works until I figured out from the code that it is used only if the query contains prefetch_related, which is something I don't see mentioned in the docs: https://django-import-export.readthedocs.io/en/stable/installation.html#import-export-chunk-size

@matthewhegarty I refactored the code, added some comments, changelog mention and test for number of queries.
Is there anything more that might be needed?

BTW: I was confused for a while how the chunking works until I figured out from the code that it is used only if the query contains prefetch_related, which is something I don't see mentioned in the docs

If you wanted to update the docs for this to make it clearer, it would be much appreciated.

@matthewhegarty I think this change would be beneficial to all users of django-import-export and should be transparent.
I would rather address all concerns about side effects than hiding this behind some configuration.
I have looked at it once more and I found better approach to this. It is no longer modifying the ModelAdmin, but I rather overriding the ChangeList.get_results(). I hope that is even less intrusive than the original approach.
Now the only scenario I can think of this would affect users is if they were relying on the ChangeList.get_results() being called. I think that there is only very slight chance that somebody uses such thing. I think it would be enough to mention this in the changelog.
In my production environment I use django-cachalot, so I didn't notice this before, but Django already calls the get_queryset() method during initialization of the ChangeList. This means that get_queryset() is called twice and the SQL queries too.
I have fixed this and even further reduced number of queries called.
I will try to improve the testing and also look at #974 if there is some connection to this.

@matthewhegarty Now I have improved the testing, now it ensures number of calls to the paginator and get_queryset.
I have left the assertNumQueries checks, because I think it would be good to ensure the number of queries will not bloat spontaneously. I can remove that, if you think contrary.
I have also written some more detailed information about the change to the changelog.

Thanks - much appreciated.


bootstrap: remove deprecated fields

Right now we are using quite a few deprecated fields in the bootstrap. This PR attempts to replace these with the modern equivalents. In theory, there should be no behavioral change here, but I would strongly encourage reviewers to closely review the golden files, as this is a fairly error prone PR. Existing tests give pretty good coverage of this, but the customer tracers in particular may warrant a second look.

TODO trusted CA is wrong

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-distroless-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-distroless-k8s-tests_istio

unit-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-distroless-k8s-tests_istio

unit-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test unit-tests_istio

integ-local-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-local-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-distroless-k8s-tests_istio

unit-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test unit-tests_istio

integ-local-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-local-tests_istio

integ-telemetry-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-distroless-k8s-tests_istio

unit-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test unit-tests_istio

integ-local-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-local-tests_istio

integ-telemetry-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-telemetry-k8s-tests_istio

integ-pilot-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-distroless-k8s-tests_istio

unit-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test unit-tests_istio

integ-local-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-local-tests_istio

integ-telemetry-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-telemetry-k8s-tests_istio

integ-pilot-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-pilot-k8s-tests_istio

integ-security-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-distroless-k8s-tests_istio

integ-local-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-local-tests_istio

integ-telemetry-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-telemetry-k8s-tests_istio

integ-pilot-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-pilot-k8s-tests_istio

integ-security-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-security-k8s-tests_istio

unit-tests_istio
24c81500e481cc20ac138ef31a6a619dbc0889c1
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest
flake, I filed an issue

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-telemetry-k8s-tests_istio

integ-pilot-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-pilot-k8s-tests_istio

integ-security-k8s-tests_istio
9d63afb57e7247a1dc17068322f375b6e886b95b
link
/test integ-security-k8s-tests_istio

unit-tests_istio
24c81500e481cc20ac138ef31a6a619dbc0889c1
link
/test unit-tests_istio

integ-local-tests_istio
24c81500e481cc20ac138ef31a6a619dbc0889c1
link
/test integ-local-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

local test fails because its trying to do istiod stuff now

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
24c81500e481cc20ac138ef31a6a619dbc0889c1
link
/test integ-local-tests_istio

unit-tests_istio
4fce5c895ac045f259787e8a6ee76119e65e9e3b
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@howardjohn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-distroless-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-distroless-k8s-tests_istio

unit-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-distroless-k8s-tests_istio

unit-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test unit-tests_istio

integ-telemetry-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-distroless-k8s-tests_istio

unit-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test unit-tests_istio

integ-telemetry-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-telemetry-k8s-tests_istio

integ-local-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-local-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-distroless-k8s-tests_istio

unit-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test unit-tests_istio

integ-telemetry-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-telemetry-k8s-tests_istio

integ-local-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-local-tests_istio

integ-pilot-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-distroless-k8s-tests_istio

unit-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test unit-tests_istio

integ-telemetry-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-telemetry-k8s-tests_istio

integ-local-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-local-tests_istio

integ-pilot-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-pilot-k8s-tests_istio

integ-security-k8s-tests_istio
1ccb1935f8333332233e48cec76e8ebf0fdc97ec
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram @hzxuzhonghu @nmittler


Google - use offline access token

Currently the Google provider uses an online token which expires in 60 minutes. This means that the token is not valid when passed to upstreams for some duration of time, and that it's not possible to refresh the cookie for a new duration (because the token expires and can't be revalidated).
The good news, refreshing a token does ensure that the token is valid. The bad news, with the google provider sessions fail when a refresh is attempted >1hr after cookie set.

referencing https://developers.google.com/identity/protocols/OAuth2WebServer#offline

Did that change recently? Coulda swore when I first started using the google_auth_proxy that the Google auth page said the app wanted offline access. The API docs don't seem to describe a way to set the expiration manually, but according to this, it seems like it should take six months, not one hour:
https://developers.google.com/identity/protocols/OAuth2#expiration
Still, I think customizing the Google provider shouldn't take too much effort. From the point of view of OauthProxy.ServeHTTP(), the contents of the access_token are opaque; we could encode/decode a refresh_token to/from the same value within the Google provider without rippling changes through most of the rest of the code. We'd have to override GetLoginURL(), and probably add a RefreshToken() method (or update the interface to ValidateToken() to return the validated token string). Only catch is that clients using --pass-access-token will need to know about the concatenated format. Happy to take a stab at it, unless you get to it first.


how to use android-exoplayer in my RN project,I can't find the usage.

I feel MediaPlayer was slowly when I paly video.
I thought exoplayer use okhttp, I think it will be faster.
But I don't know ,how to use it.

@cloudsafe last post in here:
https://github.com/react-native-community/react-native-video/issues/668#issuecomment-312383674

@frankiewiczkamil THX


People snippet broken?

first_name and last_name are unfortunately not editable in the backend. So it's not possible to add another person, as well.
I set up a fresh bakerydemo Docker installation and a fresh bakerydemo virtualenv installation. Same thing. Everything else works fine.
Any help greatly appreciated.

Thanks for the report @gabrielwolf! Have fixed this in #210.

Thank you @gasman. Works here as expected.


[Feature Request]: Set state to "undefined" if price is 0

The feature
I live rural and in my area there are not many people who report the prices to GasBuddy so I see very often the price drop to 0. I think it would be better to have the state turn to undefined in that case. If it would be "undefined" it would not look wrong on a graph and it would make it easier to make an automation where you're trying to detect a price drop.
Additional information
No response

I'll see what I can do.

This should be working in 1.0.15-b0 please give it a test.

It looks good. It now shows "undefined" instead of 0.00

Can you confirm your sensor shows it as "unavailable" in HA?

Yes, they do.

I just installed the beta version, works for me too!

Thanks everyone.


(Re)move statics in Message class to an utility class

Describe the issue
Static methods in the class Message are abused to cast data to other types, message should only be used as pipe(line) input/result, and not for intermediate type casting of data.

[ ] Documentation in Message class.
[ ] Type safe utility methods (in utility classes), meaning not use an Object as parameter.

Reporter
Core Team

Hi @nielsm5
I'm having a bit of trouble understanding the issue description. Could you please provide some more details or clarify a few points for me?

In Message class there are static methods. One method (asMessage) takes  Object as parameter. So we need to pass Message in place of Object here?

What is the expectation for below like do we have to add documentation for this
Documentation in Message class.

Thanks in advance for your help!

Right now we have an Message.asString(Object ...) method, which is 'misused' by many implementations in order to turn an Object into a Message, just in case it's of type Message. The same is happing for Message.asByteArray(Object ...)
These methods should firstly be moved to MessageUtils, and via tests should ensure that they do not close the underlaying message.
The Message.asMessage(Object ...) should also be typed, so it no longer takes any type of Object.
The Message class needs to be documented to reflect these changes. Any asSomething method should convert the data but not close the Message. This should only be done when close has explicitly been called.

Hi @nielsm5
I am not able to push feature branch to this repo. could you give me write access

Hi @akshitk20, we don't hand out access. Pull-requests can be opened via a fork 😃 !

I've created #7078 for the more extensive / places to be researched in the codebase.

asByteArray and asString are done for now, but asMessage still needs to be refactored.


getAriaValueText on Slider not working

The getAriaValueText prop doesn't work for Slider even in the examples of the docs.

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔

When function is provided on getAriaValueText  prop for Slider, it should display the return string value in the label.
Current Behavior 😯

Right now it is showing value in the label which shouldn't be the case.
Steps to Reproduce 🕹

Link:

https://codesandbox.io/s/5r8pi

Context 🔦

I am trying to show the selected value as label on slider.
Your Environment 🌎

Tech
Version

Material-UI
v4.3.0

React
16.8

Browser
Chrome

TypeScript
x

etc.

This is not a support. This is bug. The docs, API and example given does not match. How can a mentioned functionality that is not working can be marked as a support?

@junaidatique getAriaValueText affects the aria-valuetext attribute (https://github.com/mui-org/material-ui/blob/ede30a5a0af518e780414bd853b93bb275863f48/packages/material-ui/src/Slider/Slider.js#L700) maybe you’re looking for the valueLabelFormat prop?

same issue, your demo page also displays the same behavior in the temperature sliders demo.
https://material-ui.com/components/slider/
function valuetext(value) {
  return `${value}°C`;
}

expects the returned string to be a value appended with °C to be displayed in the pop up label and it's just not happening. This is indeed a bug or the documentation is lacking.  I very much would like to use the label feature but cannot because of this issue

If anyone else comes across this one.. the variable you pass to this prop should be a function, not a sting!

Anyone know how to fix this issue, I do exactly like the docs and it still not showing the text the I have modify with getAriaValueText function

Anyone know how to fix this issue, I do exactly like the docs and it still not showing the text the I have modify with getAriaValueText function

use 'valueLabelFormat' instead, its work as expected

Thanks, It work perfectly


any instruction to build src?

I tried.
./autogen.sh
make

./autogen.sh && ./configure && make && make install should work.


[Bug]: The app crashes when clicking on the album tab

What Happened?
When clicking on the album tab, the album cover will be displayed correctly if less music is imported, but if a lot of music is imported (I imported 1,500 albums), it will crash and exit directly.
Wora Version
0.3.6
Operating System
Windows 11
Steps to Reproduce
The general steps to reproduce are to import more music and then click on the Albums tab
Environment Details
Windows 11 Home
16 GB RAM
x64 Intel CPU
Severity
Major
Screenshots/Logs
No response
Discord Username
No response
Code of Conduct

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

Right, pagination is a planned feature and this will be fixed in the next update. Please let me know if the same happens with search?
Tip: You can use Cmd/Ctrl + F

Cool. I'm looking forward to it.

Please check this in our beta build, I have added infinite scroll to the albums page.
https://github.com/hiaaryan/wora/releases/tag/v0.4.0-beta0

Since, there is no response on this issue, I'll assumed this is fixed and close this for the moment. If you have any feedback, please feel free to comment and open this issue 😄


sfdx force:org:display throws error when defaultusername is set to an expired org

Summary
I ran sfdx force:org:display with a defaultorg that was selected as defaultusername for my vscode project and I got this error that it is not treated.

Then I ran sfdx force:org:list to see if the org was still valid, and it wasn't.

I concluded that this error happens if you have an expired org selected as your defaultusername when running sfdx force:org:display.
Basically this happened because yesterday I created an SO that I discovered it was expired today when I opened my vscode. When I opeed VSCode this org was still selected as default (defaultusername) on sfdx.  So there are two small enhancements we can do, one is to treat the error message on sfdx to show to the user that the org is already expired, and the other is to check if the org status before showing the button on the VSCode Status Bar (the one that shows the local defaultusername), or if the org was expired change the state of the button as shown below by running sfdx force:org:config defaultusername=, because that button is listening to changes on the file .sfdx/sfdx_config.json, so If I change sfdx defaultusername then the button changes too. We could also make sfdx set defaultusername to null automatically when the org becomes expired.

Steps To Reproduce:

Create a SO that is expired and it can still be seen on sfdx list command
Set this expired SO as the defaultusername, you can use sfdx config command for that, since the vscode extension does not show expired ones for you.
Run sfdx force:org:display using that defaultusername and you will see the non treated error message.

OBS: This will most likely happen when the user open vscode in the next day and the SO is already expired. The user will only notice it after he run display command or list command.
Expected result
sfdx force:org:display should display an error message saying that the current selected org is expired.
Actual result
sfdx force:org:display is showing a non treated error message when it runs with a expired scratch org set as defaultusername
Additional information
Feel free to attach a screenshot.
SFDX CLI Version(to find the version of the CLI engine run sfdx --version):

OS and version: Windows 10

This should be fixed in the new open-source version.  You can try it early, here: https://github.com/forcedotcom/cli/issues/835

Now I can see a better error message.


Feat flexovl elster flex 1541

FLEX-1541

SonarQube analysis reported 4 issues:

 3 major
 1 minor

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 The Cyclomatic Complexity of this method "findDevices" is 11 which is greater than 10 authorized.
 The Cyclomatic Complexity of this method "applyFilter" is 15 which is greater than 10 authorized.
 Remove the unused method parameter(s) "request".

All is well.

SonarQube analysis reported 3 issues:

 3 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 The Cyclomatic Complexity of this method "applyFilter" is 15 which is greater than 10 authorized.
 The Cyclomatic Complexity of this method "findDevices" is 11 which is greater than 10 authorized.
 Remove the unused method parameter(s) "request".

All is well.

SonarQube analysis reported 3 issues:

 3 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 The Cyclomatic Complexity of this method "applyFilter" is 15 which is greater than 10 authorized.
 The Cyclomatic Complexity of this method "findDevices" is 11 which is greater than 10 authorized.
 Remove the unused method parameter(s) "request".

All is well.

This pull request makes the build fail.

SonarQube analysis reported 3 issues:

 3 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 The Cyclomatic Complexity of this method "findDevices" is 11 which is greater than 10 authorized.
 The Cyclomatic Complexity of this method "applyFilter" is 15 which is greater than 10 authorized.
 Remove the unused method parameter(s) "request".

All is well.

All is well.

Since these changes are also present in the pr for FLEX-1540, I close this pr.


"Couldn't cleanly terminate all actors in 10 seconds" error when exiting console or starting tests

no special configuration. just
gem 'wisper-celluloid' in your Gemfile
Rails 4.2.6
Ruby 2.2.0
enter the rails console
$./bin/rails c
Running via Spring preloader in process 20405
Loading development environment (Rails 4.2.6)
[1] pry(main)> exit
E, [2016-08-04T17:43:55.040738 #20405] ERROR -- : Couldn't cleanly terminate all actors in 10 seconds!
$

that's it. every time you exit the console, it tries to terminate "actors" for 10 seconds and finally throws an error.
No need to use the gem anywhere in the app. Just install it.
When testing, the same delay and error message,
$ rspec
Running via Spring preloader in process 20450
E, [2016-08-04T17:46:01.335538 #20450] ERROR -- : Couldn't cleanly terminate all actors in 10 seconds!
..................................

Related issue: https://github.com/celluloid/celluloid/issues/696
Need to wait for celluloid v0.17.4 for the fix: https://github.com/celluloid/celluloid/issues/728

Looks pretty stale: https://github.com/celluloid/celluloid/commit/8d8f1d460d7916598da875291b8e5ea68f948d3a and https://github.com/celluloid/celluloid/commit/3444c6c9a8832caf3203146e6e9447bd06f92526
Or am I missing something?

Latest stable is 0.17.3, I also see 0.18 prerelease, maybe it is fixed there.

For 0.18.0.pre I get Celluloid is not yet started; use Celluloid.boot when running the specs, so it looks like some work will be needed.


test: test postcss hmr

#635  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hi-ogawa and the rest of your teammates on  Graphite


Update dav4-dasv4-series.md

Changing 96vCPU NW BW back to 32Mbps

@bhavanibigamudra1 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Republishing using PR link


New sample datatool

Introducing a new addition for datatools, sample. Used for sampling k old, k new, and k random records from a dataset to detect concept drift.
If you could test the code for bugs and look at my section in the readme file that would be of great help!

Seems to work well, and this is a nice addition!

Two more things:

Could you apply formatting (see the ASReview development instructions
A test would be nice. Just something that takes a test dataset, runs your sample tool and check that the output is in the format that you expect. You can see some examples of tests in the tests directory. If you need any help with this, just let me know!

Thanks for implementing this and ironing out the last details!


Fails if out_data directory does not exist

The call to sb_get results in an error if the directory out_data/ does not already exist.

Closed by #7


Add configuration for multiple pooler services and integration with nspooler

Here I've modified vmfloaty to add the following:

configuration of multiple pooler services
an API client for @McdonaldSeanp's nonstandard pooler service

I wanted to make this an unobtrusive addition, but the scale of the changes ended up being much larger than I'd hoped; If you'd rather I keep this as a fork instead of merging it into vmfloaty, that is completely okay with me!
Here's a longer description of what I've done:
Multiple service configuration
This keeps the old config file format intact for existing users, but adds another key, services, which can be filled out with a map of different pooler service url/user/token combinations.
Example:
user: 'casey.williams'
url: 'http://vmpooler.default.mycompany.net'
token: 'nzed8n6v00fc7pnsqquoapeigmt5qlcs'
services:
  alternate:
    url: 'http://vmpooler.alternate.mycompany.net'
    token: 'g32fi2mc5a3x8mxy3l48fu9vel6az7cu'

With this configuration:

floaty list would still list only the OSes on the default vmpooler instance.
floaty list --service alternate, would list only the OSes on the alternate instance. Note that the alternate instance here also has no configured username - if this is the case and the user doesn't specify a username at the command line, the default username will be used instead.

I did not include a way to interact with multiple services at once (for example, to get a list of OSes available from all services at once or show VMs running on all services at once), although that would be easy to add.
Nonstandard Pooler API
The nonstandard pooler's API is very similar to that of vmpooler, but not quite the same. I added a Service entity which gathers the right url, token, and/or username from the CLI options and the config file, then calls api methods on either the existing Pooler class or a new NonstandardPooler class. My goal with this approach was to minimize the scale of the changes I was making to Pooler, but I did end up having to change it.
To use a nonstandard pooler service, the config file above can be adjusted like this:
user: 'casey.williams'
url: 'http://vmpooler.default.mycompany.net'
token: 'nzed8n6v00fc7pnsqquoapeigmt5qlcs'
services:
  alternate:
    url: 'http://vmpooler.alternate.mycompany.net'
    token: 'g32fi2mc5a3x8mxy3l48fu9vel6az7cu'
  nonstandard:
    url: 'http://nspooler.mycompany.net'
    token: '847a4699ce6ea41206c5b220'
    type: 'nonstandard'

Including type: nonstandard in the nonstandard pooler service config tells it to use the NonstandardPooler instead of Pooler (which is the default).

Used this a bit today, seems to work for me.

Woo @caseywilliams ! This pull request looks awesome...lots of refactoring that I had been meaning to do a long time ago! Thanks for that :) Gimmie a bit to digest all the changes and provide any feedback (if needed at all) and I'll be happy to merge and cut a new release. Also thanks @MikaelSmith for testing. If there's anything that doesn't work with the nspooler just let me know since I can't test that at the moment. :)

@briancain I think I've finally got this updated now - Thank you so much for your review!

@caseywilliams yay! thanks again. I should have some time this weekend to do another once over on the changes to make sure it's all good (just some simple smoke tests to make sure nothing is broken) but I'm hoping to get this merged and released by next week! :)

Ok @caseywilliams - this should now be released in version 0.8.0!

Woohoo! Thanks so much @briancain! 😄  🙌


[bug]: force close for unclear reason

Background
Channel with lnmarkets was force closed, then cascaded to the downstream channel. For the sake of clarity I'll call these nodes like this: A (my peer) — B (me) — C (lnmarkets). There was a stuck HTLC of 10 sats from A to C going through B, it timed out. For some reason I can't understand it wasn't failed off-chain but instead went on-chain, I can confirm the lnmarkets node (C) was online all the time and I don't see it disconnecting before the FC. The message says:
Nov 18 06:42:53 fullnode lnd[1564518]: 2022-11-18 06:42:53.617 [INF] CNCT: ChannelArbitrator(c0ff658a2a02b5c04efa06e374d12a90c88a7cc6a8be8c87fadd739360fda679:1): go to chain for outgoing htlc 1a0ab6ca1880592e4665372caef469c78983d650f5a093f82b33abc64b8ce134: timeout=763657, blocks_until_expiry=0, broadcast_delta=0

However, due to mempool conditions the fee in that tx (10 sat/vB) wasn't enough to be confirmed. I'm not sure if I'm correct but this caused the channel this HTLC came from (A—B) to also be force closed by peer A in this case (who was also online, we were tracking this issue in real time).  What's even more weird is that my peer (A) reported seeing that FC (it was also unconfirmed due to the same reason) while to me (B) the channel was active though unusable, I tried to rebalance through it and it failed at hop 0. I tried to restart lnd and also manually reconnect the peer, the channel was still shown as active.
Is that true that until the force close tx is confirmed on chain the corresponding HTLC can't be failed off-chain? If that's the case it can easily cause a chain of FCs when the minimum fee is above 10 (the default maximum for anchor-type channels) and operators don't babysit their channels all the time to manually bump the fee through anchors and CPFP. If it's not then there's a bug that prevented lnd from cancelling the incoming HTLC when the outgoing channel FC is still not confirmed. I suppose the same might have happened to C, the outgoing channel for that HTLC on their node was offline, it timed out and was FCed but couldn't confirm in 40 blocks (our hop timeout) so my node B couldn't cancel it off-chain and had to FC as well. My peer A, however, said that he doesn't see that 10-sat HTLC anywhere among his channels so it must've been cancelled off-chain.
There are no messages in the logs regarding HTLC failure errors, at least not with the default INFO-level settings. Maybe there should be.
Your environment

version of lnd 0.15.4
which operating system (uname -a on *Nix) Raspbian arm64
version of btcd, bitcoind, or other backend bitcoind 23.99

This states the close reason:
go to chain for outgoing htlc 1a0ab6ca1880592e4665372caef469c78983d650f5a093f82b33abc64b8ce134: timeout=763657, blocks_until_expiry=0, broadcast_delta=0

The HTLC was about to timeout. If you send an HTLC and the other peer never resolves or times out, then we need to go on chain to sweep it. If we attempted to fail it off chain, but the peer never responded (or the connection died, or the tor connection stalled, etc, etc), then we have no option but to go on chain to resolve the HTLC.

See https://github.com/lightningnetwork/lnd/issues/1226. This proposes that we start to take into account the expected gain when we decide to go on chain or not.

Yeah, I just don't understand this flow clearly enough. Who should cancel the HTLC off chain (sender or receiver) and when? What happens if the first attempt failed for some reason, like you said if the connection stalled or was dropped? Do we retry that? And most importantly, was that caused by high fees that prevented an upstream force close to confirm in time and caused this domino effect? If that's the case, should the node operator watch their channels and manually bump the fee? I think lnd can do it automatically if there's a risk of losing another channel.

What I often see is dead tor connection and stuck htlcs on it.

Yeah there's an old idea that was never fully implemented to send a ping over a connection before we send an HTLC. If we get a pong back (we should immediately), then we'd actually use the channel. If not, we'd treat it like the channel was actually offline. This would ensure we never try to use a stale connection (due to tor, mobile roaming, etc, etc). This implementation here is pretty simple, so I think we should brush this off again.

It seems LND should be able to re-establish these connections if that is all it takes (automatically, without my manual intervention) to save the channel.

lnd would re-establish the connection as soon as it gets some sort of user space signal that the TCP connection has actually died (you'd see an unable to read: EOF in the logs or something like that). Without something like that, we need a user space ping system which is described above.

Who should cancel the HTLC off chain (sender or receiver) and when?

If you have an incoming HTLC, then you should be the one that cancels it. However if that has a corresponding outgoing HTLC (it was a forward), then the remote party needs to cancel it. If the remote party doesn't cancel it (stale connection that wasn't detected, or peer offline), then you (lnd) needs to actually go on chain to cancel it. In your case, we went on chain, but things didn't confirm in time (default 40 block CLTV delta, can be raised on the command line) since the mempool was jam packed. As a result, the incoming time lock also expired, the peer that sent us the HTLC needed to go on chain. This peer will then cancel back off chain once it resolves the HTLC.
So in summary, everything worked as expected, but things took too long to confirm. We have some basic deadline awareness, but it'll only initially target with a higher confirmation target. The missing link here is to dynamically fee bump as it gets closer to the deadline. We have a lot of research and design for stuff like this, but it hasn't all been implemented yet.
One thing that can prevent this in the future, is for a user to manually increase their CLTV delta when the mempool gets "full". In the future, we'll also start to do this automatically.

If that's the case, should the node operator watch their channels and manually bump the fee? I think lnd can do it automatically if there's a risk of losing another channel.

Yes an operator can do that, and yeah ideally lnd should do it automatically. Our goal is to implement the automated bumping for lnd v0.17: https://github.com/lightningnetwork/lnd/issues/4215.
Feel free to close this issue if the above statement answers your lingering questions @rkfg.

Yeah, I guess that all explains it. For now we need to be extra cautious during high fee times, hopefully these ideas will be implemented soon! Thank you.


[MaterialUI v4] ExpansionPanelSummary default styling tweaks

The :last-child styling for the summary children means that if there are button icons in the summary then the last one's hover is skewed out of whack due to using padding-right.  If this was changing to margin-right instead then the hovering still works properly and the spacing reserved for the expansion icon on the right is preserved.
Also the ExpansionSummary is display flex, but could benefit from also adding align-items: center to the content CSS class.

The :last-child styling for the summary children means that if there are button icons in the summary then the last one's hover is skewed out of whack due to using padding-right. If this was changing to margin-right instead then the hovering still works properly and the spacing reserved for the expansion icon on the right is preserved.

@TroySchmidt What do you think of this change?
--- a/packages/material-ui/src/ExpansionPanelSummary/ExpansionPanelSummary.js
+++ b/packages/material-ui/src/ExpansionPanelSummary/ExpansionPanelSummary.js
@@ -45,19 +45,13 @@ export const styles = theme => {
       flexGrow: 1,
       transition: theme.transitions.create(['margin'], transition),
       margin: '12px 0',
-      '& > :last-child': {
-        paddingRight: 32,
-      },
       '&$expanded': {
         margin: '20px 0',
       },
     },
     /* Styles applied to the `IconButton` component when `expandIcon` is supplied. */
     expandIcon: {
-      position: 'absolute',
-      top: '50%',
-      right: 8,
-      transform: 'translateY(-50%) rotate(0deg)',
+      transform: 'rotate(0deg)',
       transition: theme.transitions.create('transform', transition),
       '&:hover': {
         // Disable the hover effect for the IconButton,
@@ -66,7 +60,7 @@ export const styles = theme => {
         backgroundColor: 'transparent',
       },
       '&$expanded': {
-        transform: 'translateY(-50%) rotate(180deg)',
+        transform: 'rotate(180deg)',
       },
     },
   };

Do you want to submit a pull request? :)

Also the ExpansionSummary is display flex, but could benefit from also adding align-items: center to the content CSS class.

That was made on purpose. I'm not sure that we should change the alignment. We have one demos that relies on the current value.


update auth0 setup-scripts

Affected Components

[ ] Content & Marketing
[ ] Pricing
[ ] Test
[x] Docs
[ ] Learn
[ ] Other

Pre-Requisites

[ ] Code is linted (npm run lint)

Notes for the Reviewer

Updated old auth0 setup scripts for client-credentials and password grant
New Dependency Submission

Screenshots

LGTM


Running infer on linux kernel current and staging

I was working on running infer on linux kernel, current and staging. I think infer only compiles with Clang, I was able to compile linux kernel current(not the llvm branch) with quite a few modules disabled with clang(6.0) on Ubuntu 17.04, it didn't work on 16.04 because of asm-goto even with gcc-8.1 not sure why.
When I run infer on a file and not a complete module, it seems to miss the header files searches. Let's say I would run something like infer -- clang -c fs/file_table.c or infer -- gcc -c fs/file_table.c (this will still parse with Clang). Basically I'm thinking there needs to be a Makefile to build the file(in this case file_table.c) only then infer would be able to successfully search the header files? Is there any work done on building kernel with infer? or if there are any instructions available that I have missed, any leads would be great.

Hello,
If additional include folders are needed to compile a .c file, simply use the -I compiler flag.
infer -- clang -c fs/file_table.c -I <INCLUDE FOLDER>.
As for running Infer on the Linux kernel modules, theoretically its possible. The problem you are facing (I assume) is getting the 'capture' phase to run correctly. This is when infer extracts all compiler flag information from the build-system (assuming Make in this case) and runs it through Infer's embedded clang compiler. I urge you to read the first couple of chapters of the user guide to get a better understanding of how this works. http://fbinfer.com/docs/infer-workflow.html
Please note that -- clang capture mode above is different from the -- make capture mode you are more likely to use for linux kernel modules.

Thanks for writing the info, do you mean that all .c file in kernel can be compiled using -I <INCLUDE FOLDER> compiler flag and all .c file in kernel would be having a <INCLUDE FOLDER> and having Makefile output to run with make to compile .c with infer is not needed? which I think would also mean that any .c file in kernel can be compiled independently with some <INCLUDE FOLDER> available without the need of the Makefile; leaving a kernel module or a non kernel c file here.

Thanks for writing the info, do you mean that all .c file in kernel can be compiled using -I <INCLUDE FOLDER> compiler flag and all .c file in kernel would be having a <INCLUDE FOLDER> and having Makefile output to run with make to compile .c is not needed? which I think would also mean that any .c file in kernel can be compiled independently with some <INCLUDE FOLDER> available without the need of the Makefile; leaving a kernel module or a non kernel c file here.

I think the issue here is that clang -c fs/file_table.c doesn't work on its own either, is that right? Infer needs a working compilation command to run.
Normally the kernel's Makefiles would include the right compilation flags. Are you able to run infer on a make target instead with infer -- make ...? It should also be possible to run infer on a single file by asking make to build only that file, for instance rm fs/file_table.o; infer -- make fs/file_table.o (if fs/file_table.c compiles to fs/file_table.o; if not you need to adapt this command).

Thanks for writing the info, do you mean that all .c file in kernel can be compiled using -I <INCLUDE FOLDER> compiler flag with infer and all .c file in kernel would be having a <INCLUDE FOLDER> and having Makefile output to run with make to compile .c is not needed? which I think would also mean that any .c file in kernel can be compiled independently with some <INCLUDE FOLDER> available without the need of the Makefile; leaving a kernel module or a non kernel c file here.

Thanks for the pointers, ya I think infer -- clang wouldn't work for kernel files even with -I compilation flag. I should be able to make file.o in kernel which is not working for fs/, could be my local settings. I think infer needs the project to be compiled with Clang so I need to make sure I run infer -- make file.o as I have a Clang compiled kernel. I'm not sure if I have to infer -- make CC=clang file.o or I think infer will always parse with Clang and rm fs/file_table.o; infer -- make fs/file_table.o would be same as a Clang compiled kernel file and running a infer -- make file_table.o on it.

Thanks for writing the info, ya I think infer -- clang wouldn't work for kernel files even with -I compilation flag. I should be able to make file.o in kernel which is not working for fs/, could be my local settings. I think infer needs the project to be compiled with Clang so I need to make sure I run infer -- make file.o as I have a Clang compiled kernel. I'm not sure if I have to infer -- make CC=clang file.o or I think infer will always parse with Clang and rm fs/file_table.o; infer -- make fs/file_table.o would be same as a Clang compiled kernel file and running a infer -- make file_table.o on it.

Hey @tapasweni-pathak, let us know how it went, whether it worked as intended and whether you find something interesting :)

yes sure, was thinking of updating the ticket with the commands, the .inferconfig and/or any infer models after a run on complete kernel.
Infer required to skip directories like arch/, mm/, fs/, kernel/bpf, drivers/acpi, net/mac80211, latest release. I think running infer on complete kernel would also require writing infer models in infer, surely will update about the findings after complete kernel analysis.

yes sure, was thinking of updating the ticket with the commands, the .inferconfig and/or any infer models after a run on complete kernel.
Infer required to skip directories like arch/, mm/, fs/, kernel/bpf, drivers/acpi, net/mac80211, latest release. I think running infer on complete kernel may also require writing infer models in infer, surely will update about the findings after complete kernel analysis.

Thanks for checking, was thinking of updating the ticket with the commands, the .inferconfig and/or any infer models after a run on complete kernel.
Infer required to skip directories like arch/, mm/, fs/, kernel/bpf, drivers/acpi, net/mac80211, latest release. I think running infer on complete kernel may also require writing infer models in infer, surely will update about the findings after complete kernel analysis.


Experience Crystal always appears to have at least one full tank (2 megabuckets) of XP

Minecraft version
1.20.1
LibX version
1.20.1-5.0.12
UtilitiX version
1.20.1-0.8.21
Forge version
47.3.0
The latest.log file
N/A
Issue description
Experience Crystals from this mod always appear to have at least one full tank (2.14 × 106 buckets) of XP, even when empty or there's not enough levels stored in the crystal to reach that much experience.
Jade screenshot:

AE2 screenshot (the other 5 resources shown are unrelated):

Steps to reproduce
Install UtilitiX and at least one of Jade or Applied Energistics 2, then place down an Experience Crystal from UtilitiX.
If Jade is installed, simply look at the crystal to notice it starts with 0 levels but already has 2.14 × 106 buckets of XP.
If Applied Energistics 2 is installed,

Create a network consisting of an ME Terminal, a full or Creative energy cell, and a storage bus.
Place an Experience Crystal on the storage bus.
Open the terminal and notice there's 2.14 × 106 buckets of XP available.

Other information
This issue was originally reproduced in Project Architect 2 modpack, version 2.8.
This issue may be caused by https://github.com/ChaoticTrials/UtilitiX/commit/750b84c0ea14a2a7ce09b3fd69eb19e25522d5a6.
This creates an exploit whereby one can extract infinite XP from an Experience Crystal, even if it is empty.

Thank you for reporting this so quickly! Fixed it, and fix should be up now.


[NO-ISSUE] update cockroach-operator version to v1.2.0-pre for testing

Summary of changes

update cockroach-operator version to v1.2.0-pre for testing. It contains the node-config in the initContainer of the pso-db pods.

does that mean k8s version is also jumping from 6.1.0 to 6.2.0?

does that mean k8s version is also jumping from 6.1.0 to 6.2.0?

Yes, we can do it later before we release 6.2.0.


handling kubectl emitting usage in response to an error

e.g. kubectl get pods --bogus emits a usage model (ascii art) and a non-zero exit code; right now, we don't pretty-print the usage model

desired: