Traefik doesn't request a certificate when a new SAN is added

Do you want to request a feature or report a bug?
bug
What did you do?
I wanted to configure traefik in such a way that when I add a new [[acme.domains]] section OR a new sans value then traefik would request a new certificate from letsencrypt(without traefik restart)
What did you expect to see?
I add new section / new SAN and I get a certificate from letsencrypt.
What did you see instead?
instead, I see this in a log when I add a new SAN into traefik.toml:
time="2017-05-13T16:26:44Z" level=debug msg="Challenge GetCertificate _mynewsanhere.domain.com_"
2017/05/13 16:26:44 h2_bundle.go:3223: http2: server: error reading preface from client <IP_ADDRESS>:9809: remote error: tls: unknown certificate authority

and it's repeated each time I try to open mynewsanhere.domain.com in browser.
where <IP_ADDRESS> is my PC's IP (NOT traefik server's IP)
However, if I restart traefik then certificate is requested and applied properly, but that's definitely a showstopper for a production use.
Output of traefik version: (What version of Traefik are you using?)
v1.2.3 in docker

Please tell me how can I configure traefik to work as I want?
I have onDemand = true, and there's really nothing else to configure, at least not what I know about.

a little bit different log message in 1.3 -
time="2017-05-15T21:16:07Z" level=debug msg="Challenge GetCertificate mynewsanhere.domain.com"
2017/05/15 21:16:08 server.go:2753: http: TLS handshake error from <IP_ADDRESS>:12326: EOF

did one more test with 1.3, now same message as in 1.2.3 -
2017/05/15 21:19:15 h2_bundle.go:3354: http2: server: error reading preface from client <IP_ADDRESS>:28325: remote error: tls: unknown certificate authority

Did one more test, this time with Consul instead of file backend. Got exactly same result.

Any change for updates on this? I still have this behavior with 1.3rc2.
Can you at least confirm that please?
Only solution I see in my case for now is to restart traefik instances one by one after I add a new domain or san in ACME section.

Close in favor of  #1623  - Feel free to subscribe there for updates.

I re-open the issue which has not to be linked to the #1623.
For me is not a bug but a feature.
Indeed, if  LE certificates are generated by using information in the TOML file (with the tag [[acme.domains]]), these data are done in a static way has all TLS configurations linked to the entryPoints.
The PR #2233 created the internal mechanism which allows managing TLS certificates dynamically.
This mechanism can be used to extend the behavior to the LE certificates.

I find that #4216 was closed on the grounds that you can manually delete the old certificate, at least, strange. The whole point of any automation is that we, as humans, should intervene with the process as fewer as possible. If I'm practicing GitOps, I should expect to change the configuration commit it, and watch how the new certificate is getting fetched. But currently, I must remember that some changes are automatically applied and some, like caServier or SANs, are not.
Would it be possible to automate it and build some sort of a hash of the parameters affecting certificates, keep them alongside the certificate, and check them when the configuration changes?


HDDS-10252. [hsync] Revisit configuration keys for incremental chunk list after HDDS-9884

What changes were proposed in this pull request?
Enable ozone.fs.hsync.enabled, ozone.chunk.list.incremental, ozone.client.incremental.chunk.list by default.
Please describe your PR in detail:

Let's enable these features in HDDS-7593 branch.
If DataNode has not completed upgrade, fall back to full chunk list mode.

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10252
How was this patch tested?
Unit test

CC: @ashishkumar50


Read .ruby-version if present

for convenience's sake, it would be nice if Rubocop would prefer to use the version specified by .ruby-version, if present, instead of TargetRubyVersion. this way, those of us using .ruby-version will have one less thing to worry about when upgrading ruby.

duplicate of https://github.com/bbatsov/rubocop/issues/2956
TL;DR: waiting for someone interested in making the PR.

fixes https://github.com/bbatsov/rubocop/issues/2956

thanks for the heads up @mikegee !

Prefix the commit message with [Fix #...] as we customary do. Probably this should be mentioned in the manual as well.

For what its worth this breaks when using the system keyword in .ruby-version. This setting is helpful for forcing use of the OS default ruby.
This change effectively makes rubocop the arbiter of valid .ruby-version values, instead of the ruby version management tool one happens to be using.


Added fullWidth prop on button component

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
Description
Updated button button component, added new fullWidth prop which when set to true button occupies 100% width of its parent and fit for content otherwise,
Related Issue
Fixes #352

📦 Published PR as canary version: 20.39.4--canary.874.b19d050cdac0a853451515694420f0767312bba4.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

@akashyeole
The new property works well, however, you have removed the display: inline-flex from the :host, thereby re-opening a previously solved issue.
Every component needs to have its :host element be display inline-block/flex or block in order for margins and similar attributes to work.
Relevant issue: https://github.com/Infineon/infineon-design-system-stencil/issues/558
Note:
If there's a property, value, style or method placed somewhere, then there is a reason for it. Please do not just remove things without first checking why they are there, and making sure your new changes do not re-open or cause previously solved issues.

@tishoyanchev
Changed the display property of host element to inline-flex as before.

Hi @akashyeole. We only use the labels major, minor and patch so that the version gets updated accordingly after a merge. Can you please specify if this is a patch or a minor version update?


Error handling for expression evaluation throws with the wrong class

Repro:
main.dart:
import 'ext.dart' as ext;

void main() {
  final i = 0;
  print(i.foo());
}

ext.dart
extension E on int {
  int foo() => this;
}

Place a breakpoint under main.dart at any line and add to watch:

i.foo()

The error shown in the expression evaluation is:
The method 'foo' isn't defined for the class '_Smi'.

Even when the class for i is an int.
Related: https://github.com/dart-lang/sdk/issues/56554

CC: @DanTup

//cc @derekxu16

My point in making two separate issues was so that this one focuses on the error message and the other one focuses on solving the error. But if you believe that they should be handled together thats alright by me.

_Smi is technically the type it has. While it might be unfortunate in this case because _Smi might not be something one is used to I generally see it as a good thing that the expression compilation knows about the actual types.

@jensjoha out of interest, what is _Smi and what's the relationship to int? If I evaluate i.runtimeType I get back something that includes int but doesn't mention Smi:
{
	"jsonrpc": "2.0",
	"result": {
		"type": "@Instance",
		"_vmType": "Type",
		"class": {
			"type": "@Class",
			"fixedId": true,
			"id": "classes/48",
			"name": "_Type",
			"_vmName": "_Type@0150898",
			"location": {
				"type": "SourceLocation",
				"script": {
					"type": "@Script",
					"fixedId": true,
					"id"<EMAIL_ADDRESS>					"uri": "dart:core-patch/type_patch.dart",
					"_kind": "kernel"
				},
				"tokenPos": 817,
				"endTokenPos": 1112,
				"line": 24,
				"column": 1
			},
			"library": {
				"type": "@Library",
				"fixedId": true,
				"id": "libraries/@0150898",
				"name": "dart.core",
				"uri": "dart:core"
			}
		},
		"identityHashCode":<PHONE_NUMBER>,
		"kind": "Type",
		"fixedId": true,
		"id": "classes/184/types/0",
		"typeClass": {
			"type": "@Class",
			"fixedId": true,
			"id": "classes/184",
			"name": "int",
			"location": {
				"type": "SourceLocation",
				"script": {
					"type": "@Script",
					"fixedId": true,
					"id"<EMAIL_ADDRESS>					"uri": "dart:core/int.dart",
					"_kind": "kernel"
				},
				"tokenPos": 1265,
				"endTokenPos": 16970,
				"line": 28,
				"column": 1
			},
			"library": {
				"type": "@Library",
				"fixedId": true,
				"id": "libraries/@0150898",
				"name": "dart.core",
				"uri": "dart:core"
			}
		},
		"name": "int"
	},
	"id": "22"
}

However, if I just evaluate i, it does return an instance with a class of _Smi. It's not obvious to me why those things are different (and I do think it's unfortunate that this message uses the one the user might have no knowledge of because it's an internal class).

We're getting into very technical territory here I think.
I'm not a VM engineer so this attempt at a short explanation might be wrong --- but here goes:
In Dart everything is an object, but someone at some point found that having ints not actually be objects when they live in memory is a good thing for at least memory consumption (but probably also, if nothing else because of less memory consumption, for speed).
To do this a tagging scheme is used where the least-significant bit is used for the tag: 0 is a Smi (by shifting the int up) and (pointers to) objects have a 1 (which I think is made a 0 when actually addressing stuff which is safe to do because of alignment).
Because we shift the int up, though, we suddenly have less bits for storing the number. So Ints that use all bits can't be Smis but are Mints instead ("medium ints" I think --- once upon a time Dart had infinite precision ints which was probably called something else) - which are then actual objects.
From the outside it should still look like an object though (because everything is an object) so it's a _Smi. Why .runtimeType still says int I don't know --- but we can all overwrite what it says though, e.g.
void main() {
  int i = 42;
  print("i is ${i.runtimeType}");
  Map<String, String> m = {};
  print("m is ${m.runtimeType}");
  Foo foo = new Foo();
  print("foo is ${foo.runtimeType}");
  foo.myType.name = "int";
  print("but now foo is ${foo.runtimeType}");
  foo.myType.name = "String";
  print("and now foo is ${foo.runtimeType}");
}

class Foo {
  MyType myType = new MyType();
  Type get runtimeType => myType;
}

class MyType implements Type {
  String name = "not set yet";
  toString() => name;
}

will print
i is int
m is _Map<String, String>
foo is not set yet
but now foo is int
and now foo is String

which leads me to my next point: int vs _Smi might be weird but creating a Map but actually getting a _Map is sort of the same thing.
Additionally If you have an Object o but can see it's some other type (or it's a Foo but you can see it's a actually a FooImpl) you'd probably want to be able to run expression compilations on it as if it was that type. At least I do.
Anyway, _Smi is defined here:
https://github.com/dart-lang/sdk/blob/c73677e606f78b15efa25971699af1fdc5824fd6/sdk/lib/_internal/vm/lib/integers.dart#L535
with
https://github.com/dart-lang/sdk/blob/c73677e606f78b15efa25971699af1fdc5824fd6/sdk/lib/_internal/vm/lib/integers.dart#L8
where on top of Smi we also have Mint:
https://github.com/dart-lang/sdk/blob/c73677e606f78b15efa25971699af1fdc5824fd6/sdk/lib/_internal/vm/lib/integers.dart#L738
(with the dartdoc comment "Represents integers that cannot be represented by Smi but fit into 64bits." which at least doesn't contradict my explanation above)

@jensjoha thanks, that's really useful! I've seen something like a "OneByteString"(?) class show up in some of these places too, and I guess there's some similar optimisation going on there.
I think I agree, if a class like _Smi might be overriding runtimeType I think seeing the real type makes most sense - it's just a shame these names aren't something a user is more used to (I suspect if it had been called _SmallInteger or something, it would've been slightly less confusing/cryptic).


Trigger CI.

Context:
Need to test CI.
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues:

Nice PR.


Docs: Add a book preprocessor

This PR adds a preprocessor for the book that removes Sphinx-related sections.
This was previously done by docs/build_book.py script, which this PR removes.
There are some advantages of new approach:

No unnecessary file writing. Book content is passed to preprocessor by stdin. This is both faster and more friendly to disk.
No way to accidentally build a book without preprocessing. Also now developer doesn't need to know about the script or Makefile target that uses it.
New script is simpler as it doesn't need to handle files anymore. As a contributor I can certify that it's possible to not know about the script or Makefile target and be frustrated about Sphinx sections.

There is one disadvantage: book must be built either from main directory, or docs directory. If it's built from different location, build will succeed buy Sphinx sections won't be stripped.
This is because of mdbook limitation - PWD of build process is the folder the command is executed in, not the folder in which book.toml is located.
This shouldn't be a problem - I suspect book build is never / very rarely executed from other dir.
If it becomes a problem, it's possible to fix it. The script in bash.sh will need to be modified:

if executed with supports argument - just exit.
Otherwise, read stdin using jq. From first element extract root element, cd to it's value.
Execute python ./sphinx_preprocessor.py with previosuly-read stdin.

Pre-review checklist

[ ] I have split my patch into logically separate commits.
[ ] All commit messages clearly explain what they change and why.
[ ] I added relevant tests for new features and bug fixes.
[ ] All commits compile, pass static checks and pass test.
[ ] PR description sums up the changes and reasons why they should be introduced.
[ ] I have provided docstrings for the public items that I want to introduce.
[ ] I have adjusted the documentation in ./docs/source/.
[ ] I added appropriate Fixes: annotations to PR description.

Submitted an issue to mdbook: https://github.com/rust-lang/mdBook/issues/2289

mdbook issue was closed in favor of https://github.com/rust-lang/mdBook/issues/1424 which is open and not touches for 3 years now, so we shouldn't hope for mdbook fix. Imo the script I wrote should be enough, if it isn't I can write more complicated one that would always work.

v2

Changed contributing.md: added a note about limitations
Removed unnecessary in the script coment.


Assets Upload Endpoint

Ability to upload an file in the API from URL or direct file stream.

Done here https://github.com/joselfonseca/laravel-api/commit/248c6f2dd04eabcb0d8934866e79d34707d8a14d


run command functionality

Disclaimer: probably only works as long as we stick to uji, no guarantees
if the file is manually edited.
I was tired of copying the output result of the commands everytime.
So implement the 'r' key, which checks if we are on a command and runs it:

whenever a command is run, append a new line - result: 1234 with the actually returned exit code (so we manually need to toggle SUCCESS or FAIL in case of the exitcode type)
if the type is attach, take the ouput and attach it as the given file
if the type is single, append/replace the output in the markdown file
if the type is multi, same than above but interactively
if the file is human, not sure what to do so it's equivalent to exitcode right now, i.e. just writes the returned code

Force pushed with the changes. Hopefully I didn't miss any request.
I am still having an issue with libinput debug-events. I do not get any output from the command if I start it with sudo :(

I am still having an issue with libinput debug-events. I do not get any output from the command if I start it with sudo :(

See my Popen.communicate() comment above, I think this is because unlike all other commands, it never exits until you kill it and proc.stdout.readlines() blocks until the command exits.

I am still having an issue with libinput debug-events. I do not get any output from the command if I start it with sudo :(

See my Popen.communicate() comment above, I think this is because unlike all other commands, it never exits until you kill it and proc.stdout.readlines() blocks until the command exits.

It was simply because libinput debug-events never flush stdout, and so we are not getting notified when there is a new line. I am not sure why I don't get the output after the ctrl-c, but adding flush(stdout) or setbuf(stdout, NULL); in main is enough to make the job working as expected.

It was simply because libinput debug-events never flush stdout

Fixed in https://gitlab.freedesktop.org/libinput/libinput/-/merge_requests/845


Fixed WAL corruption on partial writes

This week we got a couple of incidents in our Cortex cluster caused by WAL corruption. Specifically, we got this error:
reload blocks: head truncate failed: create checkpoint: read segments: corruption in segment /<redacted>/wal/00028082 at 92762769: unexpected checksum fb8694d0, expected 89d4e21d

We noticed that in both incidents, some time before TSDB logged a train of these errors for a short period:
write to WAL: log samples: write /<redacted>/wal/00028082: cannot allocate memory

The cannot allocate memory comes from WAL.flushPage(), specifically the call to w.segment.Write(). My theory is that the WAL corruption happens when a partial write happens, which is when the os.File.Write() returns error but some bytes have been written.
Following this theory, I've reproduced the corruption in a unit test and in this PR I'm proposing a fix. Unfortunately, the fix doesn't solve WAL corruption in all scenarios: I can't find an easy way to solve the case the partial write happens while writing the initial part of a record which spans over 2 pages. I left a couple of TODO comments to show it. Any idea how to solve it?

We could tackle the case of record spanning multiple pages in separate PR when we figure out a solution for it. For now, we can get this PR in.


Update schema service to no longer have required label for orcid

To test, load up the UI, login as nih-user and start a new submission using a DOI like 10.1039/c7an01256j to make things slightly easier. Progress through the workflow. When you get to the metadata forms in Step 5, verify that the author orcid input does not have (required) in its label
See https://github.com/OA-PASS/pass-ember/issues/958

Closing in favor of a newer PR


Semicolon automatically added at the end of one import statement

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

enable

        "editor.codeActionsOnSave": {
            "source.organizeImports": true
        },

enable  no Semicolon rule in ESLint
save file
If there is just one  import ,  vscode always add Semicolon at the end of line.
if many import,  everything will be ok.

I don't want it.
Is there some config to remove it ?

Do you have javascript.format.semicolons configured to remove semicolons?

Does this reproduce in the latest VS Code insiders build with all extensions disabled?


using for_each vs Count

If wanting to build multiple bigips with the external interface having a static ip in azure this module doesnt seem to be setup to work like that.  Is there a reason it uses count vs for_each.  Is there a way to use for_each and still use static ips on the external nic

any update on this. was curious as to why the module used count vs for_each

Hi @letmetechyou,
Can you share the Terraform files of what you're trying to do?

module bigip {   for_each = local.f5_vm_names   source                      = "F5Networks/bigip-module/azure"   vm_name                     = each.value   f5_image_name               = var.image_name   f5_instance_type            = var.instance_type   f5_product_name             = var.product   f5_version                  = var.bigip_version   enable_ssh_key	          = true   prefix                      = each.value   f5_ssh_publickey            = data.azurerm_key_vault_key.main.public_key_openssh   resource_group_name         = var.existing_resource_group_name   mgmt_subnet_ids             = [{"subnet_id" = data.azurerm_subnet.subnet-mgmt.id, "public_ip" = false, "private_ip_primary" = "" }]   mgmt_securitygroup_ids      = [module.mgmt-network-security-group.network_security_group_id]    external_subnet_ids         = [{"subnet_id" =  data.azurerm_subnet.subnet-ext.id, "public_ip" = true, "private_ip_primary" = "" , private_ip_secondary = ""}]   external_securitygroup_ids  = [module.external-network-security-group.network_security_group_id]   availability_zone           = "1" }
I'm trying to utilize this module but I was wondering if a for_each if possible since in the code it uses count.  We set static ips on our external subnet ip's and noticed the documentation said for_each wouldnt be possible and was wondering if there was a specific reason the module used count vs for_each.  I also need to be able to reference the interface id's for another resources but the outputs dont provide that.  Would these be things i would need to pull the module down and make modifications myself.

any update

Hi,
Thanks for reporting. Added to the backlog and internal tracking ID for this request is: INFRAANO-1362.

Hi @letmetechyou,
Are you not able to use count on module ? you should be use count on module and pass the required values based on count.index

Hi @letmetechyou,
We didn't get any feedback from our last message so we are closing this issue.
Please reopen a new one if you are still facing the issue.


ci: consistent quotation marks in codeql-analysis action

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#393
#392  👈
#391
#390
#389
next

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

Join @AliSajid and the rest of your teammates on  Graphite

Merge activity

Oct 11, 2:11 PM EDT: A user started a stack merge that includes this pull request via Graphite.


Moving cloudstack registry mirror regex checks

Description of changes:
We are setting the general registry mirror env var but using values from Tinkerbell i.e. the colo lab.
Hence moving the regex check around to set the right variables.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve


[Parent PR] Pyo3 bindings for omf-rust

Using this PR to collect together different streams of work that will form the pyo3 bindings for omf-rust.

Closing in favour of #31.


Add PrivacyInfo.xcprivacy

Addresses #324.
See Describing use of required reason API for more information.

Required Reason API Used
Used in File

NSFileModificationDate
PINCache/Source/PINDiskCache.m

NSFileModificationDate
PINCache/Tests/PINCacheTests.m

NSURLContentModificationDateKey
PINCache/Source/PINDiskCache.m

NSURLCreationDateKey
PINCache/Source/PINDiskCache.m

The PrivacyInfo.xcprivacy file also needs to be added to the Package.swift file as a resource to by copied/processed.

When would you like to merge this PR?

Hi @EFFYLYX. Thank you for replying. It would be great if this PR can merge now. Are you able to merge it?

@tinder-owenthomas any chance of merging this PR soon? we're approaching the deadline and it's good to give teams lead-time on updating their frameworks'.

This change looks reasonable to me, so I'll help move things along by merging this. That will also give time for folks to adopt this change and provide feedback on anything that might need to be adjusted.

Hey @jparise,
I added a new PR related to this issue for including the .xcprivacy file as resource when installing PINCache via Cocoapods.


Incorrect username

Hello, i have a trouble.
I try to setup integration. I use the same login details as in the app (airCloud Home). But i've got this massage:
Failed to login API: An error occurred when signing into AWS Cognito Service: NotAuthorizedException Incorrect username or password.
PC with x86 cpu
Home Assistant 2023.3.6
Supervisor 2023.03.2
Operating System 9.5

Can you provide a link pointing to the app you used in App Store/Google Play?

Sure. App from Google Play:
https://play.google.com/store/apps/details?id=com.jch_hitachi.aircloudglobal&hl=en&gl=US&pli=1

The app for which this integration is designed is
https://play.google.com/store/apps/details?id=com.hitachi.TaiSEIA.smarthome&hl=zh_TW&gl=US
So I’m afraid that the integration does not work for you.
On Tue, Mar 28, 2023 at 4:50 PM 3FoH9l @.***> wrote:

Sure. App from Google Play:
https://play.google.com/store/apps/details?id=com.jch_hitachi.aircloudglobal&hl=en&gl=US&pli=1
—
Reply to this email directly, view it on GitHub
https://github.com/qqaatw/JciHitachiHA/issues/41#issuecomment-1486462217,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF5PKNTQ4LIY3LQTREBK5ETW6KQ3XANCNFSM6AAAAAAWJUWXGY
.
 There are several applications from hitachi. It was unexpected.
Thank you for your time and good luck with the development.

Yeah indeed. Hitachi partners with local companies around the world and creates apps for the needs in those areas. So it can be confusing. The app I mentioned above is for Taiwan's Hitachi devices only.


Not Chaning the Dots

In the responsive demo the owl carousel dots are not changing. Mainly the last dot when active is not changed. You can checkout the demo at http://owlcarousel.owlgraphic.com/demos/responsive.html.

+1
It's only the last dot. Only if you don't change the whole group of photos.

Hi!
This issue is fixed -- or I suppose, to be more accurate, replaced -- in the new version of our documentation website. That's coming Soon (TM).


python metadata test for libCI

Tracking on [LRS-684]

All test_alive tests passed.

Last small comment.
The new log looks like
Starting a test "Frame timestamp".

Starting a test "Frame timestamp".

Starting a test "Frame timestamp".

Starting a test "Frame timestamp".

Lets merge this line with the profile print line and call the test with the profile name (no need for sensor if it's too long)
test.start("Verifing increasing time for profile: ..", profle).`


Shouldn't filebe accepted too?

Doc says it's possible to use single file:
https://github.com/netlify/netlify-cms/blob/master/website/site/content/docs/collection-types.md#single-file
https://github.com/netlify/netlify-cms/blob/cfbf31b130172d0f2c44fcfa0bcc4d96bd84ef5f/src/reducers/collections.js#L31

Yeah, sorry, the docs were wrong! I've updated them to explain that files should be nested inside a files collection.

Thanks!


[DOC] drivers: pinctrl: Improvement for the pinctrl documentation

Is your enhancement proposal related to a problem? Please describe.
Needless to say, pinctrl is one of the awesome features of Zephyr RTOS. I want to mention a minor problem with the documentation. I just brought up some parts of the code for better understanding. As the description mentions, 'group1' name is arbitrary, so you can set any name you want. I actually use those names in my program.
https://github.com/zephyrproject-rtos/zephyr/blob/10130d8e6de8c3889f42d498d3de6115db8f2d35/dts/bindings/pinctrl/nordic%2Cnrf-pinctrl.yaml#L23-L39
The problem is that the pinctl document doesn't mention that 'group1' name is arbitrary.
https://docs.zephyrproject.org/latest/hardware/pinctrl/index.html
Describe the solution you'd like
I wish there was an explanation for that.
Describe alternatives you've considered
none
Additional context
none
Thanks in advance.

It's arbitrary because it doesn't mean anything, that is, no code is expecting the node to be named exactly groupN.


removal of other cpcall mentions

no point in defining if not used anywhere

This was purposesfully left there as backwards compatibility for userplugins

didn't think of that, good point


feat: Unify and update the setup- go and checkout in the action operation to the latest version

Description

feat: Unify and update the setup- go and checkout in the action operation to the latest version
Related Issue

#14
Checklist

check with "x", if applied to your change

[ ] All methods associated with structs has func (<first character of struct name> *struct) method() {} name style.
[ ] Wrote unit tests for new/changed features.
[ ] Updated docs/doc.go
[ ] Updated pkg/example_test.go
[ ] Updated README.md
[ ] New public methods/structs/interfaces has comments upside them explaining they responsibilities
[ ] Executed make dod with none issues pointed out by golangci-lint and goreportcard-cli

Sorry for the delay, I didn't received a notification for this.


How to turn off debug mode in headscale serve

headscale serve command runs in debug mode by default.
Is there any way to run serve as a background process so the new connections can be established while i am not logged in to the server. (Running the serve without a terminal).
Thanks in advance.

You can use systemd (or any other init system) to serve as a background process (service), here's an example systemd unit file:
[Unit]
Description=headscale
After=network.target

[Service]
User=headscale
Group=headscale
WorkingDirectory=/etc/headscale
ExecStart=/usr/bin/headscale serve
AmbientCapabilities=CAP_NET_BIND_SERVICE
CapabilityBoundingSet=CAP_NET_BIND_SERVICE
SecureBits=keep-caps
NoNewPrivileges=yes

[Install]
WantedBy=multi-user.target

This unitfile assumes that you have a user and group named headscale, that your configs are in /etc/headscale (and that the user has permission to access them), and allows you to run it on privileged ports via the CAP_NET_BIND_SERVICE (80, 443 as prime examples) [note that you might have to run sudo setcap CAP_NET_BIND_SERVICE=+eip /usr/bin/headscale to actually let the binary also have that capability).

As for the debug mode, it actually says in the message- You can set the environment variable GIN_MODE to release, which you can also do in the systemd unitfile by adding Environment=GIN_MODE=release under [Service]. ^-^

You can use systemd (or any other init system) to serve as a background process (service), here's an example systemd unit file:
[Unit]
Description=headscale
After=network.target

[Service]
User=headscale
Group=headscale
WorkingDirectory=/etc/headscale
ExecStart=/usr/bin/headscale serve
AmbientCapabilities=CAP_NET_BIND_SERVICE
CapabilityBoundingSet=CAP_NET_BIND_SERVICE
SecureBits=keep-caps
NoNewPrivileges=yes

[Install]
WantedBy=multi-user.target

This unitfile assumes that you have a user and group named headscale, that your configs are in /etc/headscale (and that the user has permission to access them), and allows you to run it on privileged ports via the CAP_NET_BIND_SERVICE (80, 443 as prime examples) [note that you might have to run sudo setcap CAP_NET_BIND_SERVICE=+eip /usr/bin/headscale to actually let the binary also have that capability).
As for the debug mode, it actually says in the message- You can set the environment variable GIN_MODE to release, which you can also do in the systemd unitfile by adding Environment=GIN_MODE=release under [Service]. ^-^

Can you share how to run this systemd unit file.
And I am not sure whether i need to create user and group named headscale, if yes how should I do it.
I am new to linux. Testing this in my Ubuntu 18.04 machine.

Sure, but please do take some time to give the documentation a read for yourself, there's a lot more than what I can cover here, haha!
First, copy/move the headscale binary to somewhere like /usr/bin/, so it's in the default path. That'll make it much easier to control headscale, as you can run it from anywhere.
Proceed to creating a [system] user, setting its home directory to /etc/headscale, and denying it access to a shell with sudo useradd --system --home /etc/headscale --shell /bin/false headscale. We'll run headscale as this [low-privileged] user.
Now, ensure that the headscale binary is executable (sudo chmod +x /usr/bin/headscale), and that the headscale user & group can access /etc/headscale (sudo chown -R headscale:headscale /etc/headscale).
To set up the Systemd unit, go ahead and make a file in/called /etc/systemd/system/headscale.service, with the contents mentioned above. Run sudo systemctl daemon-reload to reload systemd's services, and let it know about headscale.service.
If your configs are not already in /etc/headscale, go ahead and move them there along with your WireGuard private key file and the derp.yaml file [if you're using SQLite, also move the database over]. Once done, all that's left to do is sudo systemctl enable --now headscale, which will enable the service to start on boot, as well as start it immediately (--now).

Thanks for spending time in this @gtsatsis
It worked 🥳🥳🥳.

Hi @gtsatsis
Trying to logging in my peer using
sudo tailscale up -login-server YOUR_HEADSCALE_URL --authkey YOURAUTHKEY
It seems it doesn't responding
In peer checking status of tailscale it shows Logged out
But in Headscale it shows the peer name by executing
headscale -n namespace nodes list

@bharathmsd7 are you running headscale behind a proxy?

Yep behind an Nginx proxy

Then I am going to close this issue. We are tracking the issues with nginx here https://github.com/juanfont/headscale/issues/56


[question] how can one use sqlite3_create_function() or sqlite3_create_window_function()

👋
I'd like to try defining a custom aggregate / window function in a NIF and I wonder if there is some easy way to do that. Like
Exqlite.Sqlite3NIF.with_conn_resource(fn ref -> MyNif.create_func(ref) end)

The NIF would be in my app that depends on :ecto_sqlite3, so I don't have direct access to :exqlite connections. Maybe some kind of callback like on_open would be provided to :ecto_sqlite3 adapter to be called with db reference each time connection opens.

Seems like it's already possible to define custom functions via extensions.
https://hexdocs.pm/exqlite/readme.html#using-sqlite3-native-extensions

@ruslandoga you may want to check out @mindreframer's stuff.

https://github.com/mindreframer/sqlite_init
https://github.com/mindreframer/ex_sqlean

Another issue
https://github.com/elixir-sqlite/exqlite/issues/174


Creating a new hook to be used for complex reducers

This creates a new hook to be used on projects with existing stores that contain multiple reducers.

Thanks for the PR @martinrojas.
I’ve been thinking about your comment on the issue and this PR. I think it might be better to modify the existing useHookedOnState with a function signature like:
useHookedOnState(path, defaultValue, options)
Where options is an object with various config options such as namespace and top level path. This would keep the function signature simple while allowing for further non-breaking expansion of the api.
I’ll probably take on this change myself in the next few days.

Closing in favor of #9


PowerToys Run - Random Crash - Desktop Composition/DWM

Microsoft PowerToys version
0.77.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
General, PowerToys Run
Steps to reproduce
Attempt to launch Microsoft Onedrive desktop application. - See below logs/error message:
Error Log: 2024-01-12.txt
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22635.0
IntPtr Length: 8
x64: True
Date: 1/12/2024 4:11:26 PM
Exception:
System.Runtime.InteropServices.COMException (0x80263001): {Desktop composition is disabled} The operation could not be completed because desktop composition is disabled. (0x80263001)
at Standard.NativeMethods.DwmExtendFrameIntoClientArea(IntPtr hwnd, MARGINS& pMarInset)
at System.Windows.Shell.WindowChromeWorker._ExtendGlassFrame()
at System.Windows.Shell.WindowChromeWorker._UpdateFrameState(Boolean force)
at System.Windows.Shell.WindowChromeWorker._HandleDwmCompositionChanged(WM uMsg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Shell.WindowChromeWorker._WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Interop.HwndSource.PublicHooksFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
✔ Expected Behavior
Application launch and PowerToys Run continue running as usual.
❌ Actual Behavior
Attempt to launch Microsoft Onedrive desktop app using Alt+Space shortcut results in application launching as expected and then PowerToys Run crashing with error message:
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22635.0
IntPtr Length: 8
x64: True
Date: 1/12/2024 4:11:26 PM
Exception:
System.Runtime.InteropServices.COMException (0x80263001): {Desktop composition is disabled} The operation could not be completed because desktop composition is disabled. (0x80263001)
at Standard.NativeMethods.DwmExtendFrameIntoClientArea(IntPtr hwnd, MARGINS& pMarInset)
at System.Windows.Shell.WindowChromeWorker._ExtendGlassFrame()
at System.Windows.Shell.WindowChromeWorker._UpdateFrameState(Boolean force)
at System.Windows.Shell.WindowChromeWorker._HandleDwmCompositionChanged(WM uMsg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Shell.WindowChromeWorker._WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Interop.HwndSource.PublicHooksFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
Error Log:2024-01-12.txt
Other Software
OneDrive device ID: bca6c188-d1bf-d97a-3da0-fb6ac07c3bf3
OneDrive version: Build 23.250.1203.0001 (64-bit)

/dup #30507

ah, ok sorry...I should've checked first.


RTL support

As one of the two big reasons for this fork is the broken cursor mechanic on mobile browsers (which will be really fun to debug, i'm sure), we should probably remember to take into account that there could be RTL (right to left) users, so we'll need to make sure that we support going the opposite way from normal.

.... numbers are not reversed in RTL, i'm a moron :D


add get_text_generation_parameters methods

grpc client: add get_text_generation_parameters method
http client: add get_text_generation_parameters method
README: add get_text_generation_parameters method

closes #61 and #62

Mocked tests are failing, due to some the test hanging for mocked endpoints. Merging this as is since these failures will be fixed by #64


[libkeyfinder] update to 2.2.6

Describe the pull request

What does your PR fix?

This fixes a few issues with the pkgconfig and CMake package
config files.

Which triplets are supported/not supported? Have you updated the CI baseline?
all

Does your PR follow the maintainer guide?
yes

If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?
yes

Thanks!


Controllable type

Abstract graphically listed lists to ControllableObj trait and give them differing implementations for functionality needed.
Implement also paging to work with ControllableObj implementing structs.
Major changes:
New trait ControllableObj and two implementations for it.
No view specific scroll areas anymore. All handled with controllable_list.rs

Is the text_sudoku.txt included by accident? And can you please put the “Show trail vs show conflict literals” switch button on its own row, because it stretches the controls past the GUI midway into  sudoku grid’s space? This can be done either by inserting ui.end_row(); before it or by making a new function to be called from fn controls in gui/controls.rs and putting end_row there.

Many thanks for your prompt review and comments, Heljä! 1) Removed data/text_sudoku.txt 2) Show trails... button has been fixed in another branch, pls. see PR Fix trail gui, thanks :-)


PeerAuthentication tls mode not work in ambient

Bug Description
I'm following the tutorial https://istio.io/latest/docs/tasks/security/authentication/authn-policy/#globally-enabling-istio-mutual-tls-in-strict-mod.
I find that I can access the pod without sidecar from ambent pod or sidecar pod when I've set the mtls mode to STRICT globally or in namespace level or per-pod.
From the envoy config_dump of caller side, I see the actual behaviour is PERMISSIVE mode no matter what I set.
      "cluster_name": "outbound|8000||httpbin.default-sidecar.svc.cluster.local",
      "endpoints": [
       {
        "locality": {},
        "lb_endpoints": [
         {
          "endpoint": {
           "address": {
            "envoy_internal_address": {
             "server_listener_name": "tunnel",
             "endpoint_id": "<IP_ADDRESS>:80"  // pod with sidecar
            }
           },
           "health_check_config": {}
          },
          "health_status": "HEALTHY",
          "metadata": {
           "filter_metadata": {
            "tunnel": {
             "destination": "<IP_ADDRESS>:80",
             "address": "<IP_ADDRESS>:15008"
            },
            "envoy.transport_socket_match": {
             "tunnel": "h2"
            },
            "istio": {
             "workload": "httpbinv1;default-sidecar;httpbin;v1;Kubernetes"
            }
           }
          },
          "load_balancing_weight": 1
         },
         {
          "endpoint": {
           "address": {
            "socket_address": {
             "address": "<IP_ADDRESS>",  // pod without sidecar
             "port_value": 80
            }
           },
           "health_check_config": {}
          },
          "health_status": "HEALTHY",
          "metadata": {
           "filter_metadata": {
            "istio": {
             "workload": "httpbinv2;default-sidecar;httpbin;v2;Kubernetes"
            }
           }
          },
          "load_balancing_weight": 1
         }
        ],
        "load_balancing_weight": 2
       }
      ],
      "policy": {
       "overprovisioning_factor": 140
      }
     }
    },

Version
./bin/istioctl version
client version: 0.0.0-ambient.191fe680b52c1754ee72a06b3e0d3f9d116f2e82
control plane version: 0.0.0
data plane version: 0.0.0-ambient.191fe680b52c1754ee72a06b3e0d3f9d116f2e82 (11 proxies)

Client Version: v1.25.2
Kustomize Version: v4.5.7
Server Version: v1.25.2

Additional Information
No response

This is working as implemented; there is currently open discussion if we will support this API in ambient

This is working as implemented; there is currently open discussion if we will support this API in ambient

Can I see similar discussions somewhere, I'm more concerned about the completeness of ambient's capabilities

Doc is in https://docs.google.com/document/d/1Lk1I1qB-XwVWFP0sVcIN5JMJZHjuF3j8lGWtUeDK6pY/edit#heading=h.nluvz4y67j4c. Note its very much under discussion.
I can assure anyone reading this though, some equivalent to "only allow mTLS" will be available. That is 100% critical

not stale, we still need discussion on if peer authentication should be supported and if so when.

Update: After discussion in the Ambient WG, we decided to move forward with supporting PeerAuthentication for now. I'll take care of the ambient implementation


Uncaught Exception (cannot start program on Ubuntu)

VSCode Version: 1.12.0-1491200824
OS Version: Ubuntu 16.10

Steps to Reproduce:

Cannot run the program

Uncaught Exception:
SyntaxError: Unexpected identifier
at /usr/share/code-insiders/resources/app/out/vs/code/electron-main/main.js:4:39806
at Object. (/usr/share/code-insiders/resources/app/out/vs/code/electron-main/main.js:4:57360)
at Object. (/usr/share/code-insiders/resources/app/out/vs/code/electron-main/main.js:4:310456)
at t._loadAndEvalScript (/usr/share/code-insiders/resources/app/out/vs/loader.js:4:12007)
at /usr/share/code-insiders/resources/app/out/vs/loader.js:4:11252
at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:445:3)

Same problem here on Linux Mint 18 with the latest insiders build:

There shouldn't be any 1.12 build published. How did you get this?
@remojansen Can you also paste your version?

The given location is at WinJS of t("WinJS/Core/_Global".
@remojansen @byroniac if you can attach the file main.js from /usr/share/code-insiders/resources/app/out/vs/code/electron-main

@aeschli My file is https://gist.github.com/remojansen/36691151b266064ecb1fb2f380090651

Seeing the same thing with latest update to insiders, v1.11.0 on OSX 10.11.6
Uncaught Exception:
SyntaxError: Unexpected identifier
    at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/code/electron-main/main.js:4:39806
    at Object.<anonymous> (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/code/electron-main/main.js:4:57360)
    at Object.<anonymous> (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/code/electron-main/main.js:4:310456)
    at t._loadAndEvalScript (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/loader.js:4:12007)
    at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/loader.js:4:11252
    at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:445:3)

Gist with main.js is up at: https://gist.github.com/michalzaptk/051acd89817bf8f1a3067be3a72edf77

Issue looks like an un-escaped "-" in a regex. Until Microsoft releases an official patch, I made a fix and have a working copy here: https://gist.github.com/michalzaptk/232bc91395c4d3852dc9092d4eecef5d

@michalzaptk Can you point me to the problematic RegEx? Is it that one?

@aeschli Yep, thats the little bugger

It's strange as that regex has been there for the last 9 month and according to http://regexr.com/ isn't wrong.
Also, the same main.js that caused the problem works with me, so I don't think there's a real syntax error there.
Could it be that VSCode was still running while updating? My theory would be that this changed the underlying bits and a reload resulted in that strange bug.

Hmm, if that's the case then there's most likely an issue with the update functionality. I for one, updated by clicking on the update button that pops up near the top of the window. That seems to close the program, install the update, and then start itself back up again.
I'm guessing that's how most of the others applied the update as well, @byroniac @remojansen ?
I'm not familiar with this codebase, so I'm not exactly sure how to go about testing this theory.

I actually went to the website to download the binary. I don't update clicking on the update icon because it fails (due to lack of root access). Instead of trying to find how to give it root access I just download it from the website because I don't upgrade everyday....

I completely uninstalled vscode-insiders, then removed the ppa from my system, then reinstalled the insiders binary from the main website, and i get the same error. As far as I can tell, it's not the update process, but the binary itself (at least on Ubuntu). The stable VSCode works fine.

I've attached the zipped main.js file that I have.
I think I followed these original instructions for the PPA:
https://code.visualstudio.com/docs/setup/linux

For reference, I went back and installed version 1.11.0-1491197413 of Insiders from the web site, same error. It installs the PPA on my system after I completely removed /etc/apt/sources.list.d/vscode.list:
deb [arch=amd64] http://packages.microsoft.com/repos/vscode stable main
Then when I do an apt-get update, I get the 1.12.0-1491200824 version of the initial request above.

Getting same thing on Windows 10.

Code-Insiders doesn't update on Ubuntu the same way it does on Windows. I can check for updates, and if it finds any, it sends me to the website to download the .deb file. However, the .deb file now automatically seems to install a PPA for updating the system with "apt-get update". I do have several custom PPAs so it's not impossible that something is conflicting, but I hope it's unlikely.
BTW, when I open up the main.js and go to line 4, character 39806, there is a string constant of "WinJS/Core/_Global". Is this where the error is? Is there something wrong with WinJS?

/cc @jrieken

After debugging this a while with @jrieken we think this is a combination of things that went wrong from the Friday build to the Monday build:

our minifier changed in a way that the output of the build is slightly different even though there have been no code changes on the main side (caused by https://github.com/mishoo/UglifyJS2/pull/1744)
we leverage the cachedData option of VM (https://nodejs.org/docs/v6.5.0/api/vm.html#vm_class_vm_script) to speed up startup time and even though there has been a change in the script due to the minifier change, we are not getting the error that should be thrown when the script files on disk changed, making us recache the script

I pushed a workaround to have an actual change in the main script to see if that helps. The long term fix would probably be to change our cached data files to use the commit hash (from product.json) as identifier so that we never trust the script engine but always drop the cache when a version from a new build is loaded.

A new insiders build is out and I can update from cc46bbf5b0412e464f89280e828d602fbc06a0f4 to 	4076fec539a2bac23ae3ac8fc5c0d2c6f5bb0a1d so it looks like the workaround worked.
People who are stuck on this error, here is how you can get out of it: Delete the CachedData directory before starting:

Windows: %APPDATA%\Code - Insiders\CachedData
macOS: ~/Library/Application Support/Code - Insiders\CachedData
Linux: ~/.config/Code - Insiders\CachedData

Thank you! Have a virtual cookie (or virtual drink) on me. :)

@bpasero @jrieken I cannot reproduce the original problem. How can I verify the fix?

@chrmarti verify you can update from Fridays insider build from last week to todays

Verified on Windows 10.

Working on Mint 18 👍


network: support for abstract namespace unix sockets

title: network: Support for abstract namespace unix sockets
Description:
Following up on https://github.com/envoyproxy/envoy/issues/2085.
Risk Level: Low
Testing:
I modified an existing test that was failing on abstract socket names. Existing tests on pipes seem pretty limited. I would appreciate some guidance on how to test this better.
Docs Changes:
Release Notes:

Also, the hot restart code is currently using an abstract unix socket, but doesn't use any of the address code.  Can you see if you can make hot restart use your new address code?  See envoy/source/server/hote_restart_impl.cc.

Hi Greg, thanks for the review. I would prefer keeping the url encoding as a separate effort to keep this PR simple and unblock simple uses cases. I looked at hot_restart_impl.cc and the refactoring doesn't seem very obvious as the code deals with raw addresses versus Instances like in address_impl.cc. As a result I would prefer keeping that out of this PR too.
Where is a good place for the integration test? Is there an existing one for "regular" sockets I can look at?

I think that's reasonable to keep both of those other changes as separate PRs.
For a test, I would take a look at envoy/test/integration/uds_integration_test.*, and either extend that, or clone it and create a similar test for this new type of address.

Can you merge master to pickup the latest Bazel changes? I think we might need that to pass on OS X.

What is the proper way of testing if we're running on linux? I have
#ifndef linux
 throw ...
#endif

But it seems my tests are failing that way due to triggering the linux-only exception (and my machine is linux).
Also how can I make sure the tests will only be ran on linux? #ifndef linux as well?

I think you can do this in the address code:
#if !defined(__linux__)
throw ...
#endif

And in the test code:
#if defined(__linux__)
// normal test expectations here
#else
EXPECT_THROW(......);
#endif

Actually, this should probably have release notes and potentially doc updates? Can we add some?

I added release notes. For the docs it seems I should update https://github.com/envoyproxy/data-plane-api/blob/master/envoy/api/v2/core/address.proto (Pipe.path). Should I create a separate PR for that?

Yes, please open a separate doc PR.  That looks like the correct location (Pipe.path)

Done: https://github.com/envoyproxy/data-plane-api/pull/462


[BUG] Markdown and Html parse insertion left behind

The text

🆕 Kyrgyzstani som and new payment methods in 💠 P2P Market.
Kyrgyzstani som and the following payment methods for this currency are already available in P2P Market: MBank, Doscredobank, Bakai Bank, FINCA Bank, Optima Bank, DemirBank, MegaPay, О!Money.
To buy or sell coins using this new currency, go to P2P Market, tap “Payment and Currencies” and choose “KGS” in the list.
We've also added new payment methods for the existing currencies:
• UralSib, UniCredit Bank, Levoberezhny Bank, GenBank, Ak Bars Bank, LAVA for RUB;
• Forte Bank and Jusan for KZT.
To configure your payment methods, go to P2P → “Payment and Currencies” → “Payment Methods”.
throws an error

https://github.com/Lonami/grammers/commit/0108ec3976fd4b8e45c83fdb32bd20bc2e6b390a


Analyzer 5 support

Is your feature request related to a problem? Please describe.
Other dependencies are beginning to to require analyzer: ^5.0.0, such as freezed: 2.1.1. I'm not sure if this is a hard requirement, I'm not aware of what's changing in analyzer 5. But I didn't see another issue tracking this so here it is!
Version

Platform: -
Flutter version: 3
Hive version: 1.1.3

auto_route_generator: ^5.0.3 requires analyzer: ^6.0.0 already.
So it might be a good idea to request 6?

This seems resolved in the 2.0.0 version.


no load my language...

[16:09:12] [Server thread/INFO]: [Quests] Enabling Quests v3.5.1-b${env.BUILD_NUMBER}
[16:09:12] [Server thread/ERROR]: [Quests] Failed loading lang files for pt-BR because they were not found. Using default en-US
[16:09:12] [Server thread/INFO]: [Quests] If the plugin has not generated language files, ensure Quests has write permissions
[16:09:12] [Server thread/INFO]: [Quests] For help, visit https://github.com/FlyingPikachu/Quests/wiki/Translations#troubleshooting
[16:09:12] [Server thread/INFO]: [Quests] Loaded language en-US. Translations via Crowdin

Follow the Pro-tip at this link - https://github.com/FlyingPikachu/Quests/wiki/Casual-‐-Translations
Sent with GitHawk

Updated URL in e223dc29215ed64af6bd7b32854cca8ed73a7ea2


added blockhash input to claimtrie RPC methods plus tests

I'm marking this as WIP as I'm unsure on the approach and looking for feedback. You'll notice that getclaimsforname is done differently (more robustly) than the others. The others could be done like that one but not vice-versa, but I'm less sure on the consequences of that approach and the completeness of what's there.

Apparently I need to add those minconf parameters to the client.cpp file for automatic CLI conversion to int.

I plan to standardized JSON objects names, between function they have different names which is ugly and hard to parse in a script.
claimId
txId
txn
value
presentInTrie (in claim trie)
effectiveAmount
validAtHeight
and so on, first word starts with lower letter, second word, if it presents, starts with upper. Most help does not match returned JSON object, Any other suggestions?

I plan to standardized JSON objects names...

We need to do that as a separate pull request. That's a breaking change, whereas adding additional outputs and taking new optional inputs are not breaking changes. I agree that the current mismatches between methods are silly. Do you know where each RPC method is used in the LBRY stack? We should locate each call and write up an epic story that includes updating each method that is necessary.

We need to do that as a separate pull request. That's a breaking change, whereas adding additional outputs and taking new optional inputs are not breaking changes. I agree that the current mismatches between methods are silly.

I understand, you're right.

Do you know where each RPC method is used in the LBRY stack? We should locate each call and write up an epic story that includes updating each method that is necessary.

lbry.go, i think, but @shyba knows better.

Rebase to be ready for merge.

I added a commit for some unit tests for the new CClaimtrieCache functions in claimtriecache_tests.cpp, by no means extensive but I think it'd be good to have some coverage on those.
https://github.com/lbryio/lbrycrd/pull/189/commits/0159a57765154ca2a09df4fc3845546eb3908a00

Fixed formatting on the test I added.
Also noting that this will affect calls in https://github.com/lbryio/lbryumx for lbryumx/session.py , this should be the only place where this PR will affect. I will open issue there

How will libryumx be affected? The intent was that all new parameters here were optional.

Update help strings.

I was just doing some testing by hand and seeing a problem:

start up a clean regtest daemon (after removing ~/.lbrycrd/regtest)
generate 200 blocks
claim a new name
generate 1 block
get the block hash for block 190
run getclaimsintrie passing in the hash for 190
the results should be empty but they aren't

Noting here that RPC command getclaimbyid does not have a blockhash argument in this PR .
I think it might be a bit involved to implement .. since we'll have to redo the getClaimById function in CClaimTrie ( it relies on leveldb and a cache, but the cache I think does not support decrementing multiple blocks at a time, just one block at at time) . We could probably open a new PR for this instead of  doing this here.


360 Videos freeze after 20 seconds

the videos created with this tool start out normal, but freeze after 20 seconds.. The sound continues to play back, but the video stops..

quite the same behaviour for me

by the way, editing the script and setting to 'no' the after complete cleanup, i could process the few frames in error and clone some empty frames with a copy of the previous one. Than i merged all frames togheter with:
ffmpeg -f image2 -r 30 -i ./image%05d_pano.jpg -vcodec copy ../out.mk
and then i dropped the video (and the extracted audio) to kdnelive for the final editing :-)


ɵPendingTasks is no longer exported by @angular/core (angular 19)

Please provide the environment you discovered this bug in.
"dependencies": {     "@analogjs/content": "1.10.1",     "@analogjs/router": "1.10.1",     "@analogjs/trpc": "0.2.47",     "@angular/animations": "^19.0.0",     "@angular/cdk": "^19.0.0",     "@angular/common": "^19.0.0",     "@angular/compiler": "^19.0.0",     "@angular/core": "^19.0.0",     "@angular/forms": "^19.0.0",     "@angular/material": "^19.0.0",     "@angular/platform-browser": "^19.0.0",     "@angular/platform-browser-dynamic": "^19.0.0",     "@angular/platform-server": "^19.0.0",     "@angular/router": "^19.0.0", ... }
Which area/package is the issue in?
content
Description
After updating my application to angular ^19.0.0, with "@analogjs/content": "1.10.1",
I get the following error:
Please provide the exception or error you saw
Here is the stack trace:

Prerendering static pages...
Building Server...

The '@analogjs/platform' server has been successfully built.
x Build failed in 40.29s

 NX<EMAIL_ADDRESS>(2:58): "ɵPendingTasks" is not exported by<EMAIL_ADDRESS>imported by<EMAIL_ADDRESS>
file: /Users/louiiuol/apps/optee/node_modules/@analogjs/content/fesm2022/analogjs-content.mjs:2:58

1: import * as i0 from "@angular/core";
2: import { inject, Directive, HostListener, InjectionToken, ɵPendingTasks as _PendingTasks, Injectable, TransferState, ...
                                                             ^
3: import { DOCUMENT, Location, AsyncPipe, isPlatformBrowser } from "@angular/common";
4: import { Router, ActivatedRoute } from "@angular/router";

Other information
Locally I've "fixed this error" by updating the import located in<EMAIL_ADDRESS>I've replaced ɵPendingTasks as _PendingTasks by ɵNoopNgZone as _PendingTasks and my build completes.
Not sure if this a good fix though..
Thanks for your help ! 🙏
I would be willing to submit a PR to fix this issue

[ ] Yes
[X] No

Thanks @louiiuol. Its fixed in 1.10.2-beta.8


Move terratest tests to the acceptance test workflow

The terratest tests are end-to-end tests; the Go test workflow is not set up for real API interactions, so including terratest tests in that workflow means it will always fail.
As a first pass, this attempts to get the Go test workflow passing by excluding terratest tests in that workflow.  The acceptance test workflow has also been updated to run the terratest tests, but they still fail (although they fail differently, which counts for something).

The terratest tests now fail due to 401 Unauthorized errors (previously they were failing to request a token b/c the Go test workflow doesn't include any of the env vars needed for API authentication).
Did our EQUINIX_API_TOKEN lose some access it used to have, or did these tests never work in CI?  If they never worked, what is the benefit of running them in CI?

Did our EQUINIX_API_TOKEN lose some access it used to have, or did these tests never work in CI? If they never worked, what is the benefit of running them in CI at all?

GH isn't configured with EQUINIX_API_TOKEN (or CLIENT_ID / CLIENT_SECRET, which is what we would probably do here).
The pattern that is used in E2E tests could be made to match only the metal tests where AUTH_TOKEN (metal) is defined.  I thought this was in place (@ocobleseqx). The Fabric PR may have changed that.
Persistent EQUINIX_API_TOKEN values are only used in the sandbox API, which is not made for e2e tests (endpoints return a valid result but there is no state or way to simulate alternate responses). https://developer.equinix.com/environment/sandbox
EQUINIX_CLIENT_ID and EQUINIX_CLIENT_SECRET would need to run full E2E on Fabric and NE resources. We would also need an org and project for Terraform to create and destroy the related resources.

The pattern that is used in E2E tests could be made to match only the metal tests where AUTH_TOKEN (metal) is defined. I thought this was in place (@ocobleseqx). The Fabric PR may have changed that.

It looks like the TF acceptance tests are filtered to Metal.
I see what you mean now. The terratest action will always fail. It could be moved to a workflow that is only triggered manually with CLIENT_ID and CLIENT_SECRET and other required params as inputs to GH. (or the action could be commented out).

For now I think we should exclude the terratest tests from actions entirely, as we do for non-Metal acceptance tests, and add a separate issue to get all of those tests added to & working in CI.


Bottleneck for MQTT publish when using Qos > 0

Is this a bug or a feature request?
Limitation / Bug
What happened?
When using QoS 1 or 2, the uplink rate is limited to a few packets per second (around 10).
For a Gateway with a high concentration of nodes, or if the Gateway Bridge is installed in the cloud and handles messages for several gateways, this is an issue.
The bottle-necking is due to the following code in backend.go:
if token := b.conn.Publish(topic.String(), b.qos, false, bytes); token.Wait() && token.Error() != nil {
	return token.Error()
}

The token.Wait() call is blocking. With QoS 0, it's returning without any delay, but with QoS 1 it's blocking until the PUBACK is received for that message, which can take up to a few hundred milliseconds depending on network latency.
I don't know if token.Wait() should be called at all since no provisions are made to retry publishing the message if it fails anyway. From what I understood reading the code, If the MQTT connection drops, the Paho client will re-publish the messages upon reconnection if they were queued with QoS 1 or 2. Maybe this is enough and should remove the need for the token.Wait().
Or message publishing needs to be parallelized so that one message is not blocking the others while in the token.Wait(). And in that case maybe token.WaitTimeout() should be used instead.
I'd be happy working on a PR to fix this if needed, but I am not familiar enough with Go yet to code anything complex.
What version are your using?
Latest (2.6.1)
How can your issue be reproduced?
Have a lot of uplink packets going through the bridge (a few dozens).
Note: the UDP packets are queued in the network layer at the socket level so they are not always dropped, but they are not sent up to the Network Server as fast as they should, and will be dropped eventually, once the socket queue is full.

I think the changes are minimal, see the runV2 and runV3 functions in root_run.go. Example:
	go func() {
		for rxPacket := range gw.RXPacketChan() {
			if err := pubsub.PublishGatewayRX(rxPacket.RXInfo.MAC, rxPacket); err != nil {
				log.WithError(err).Error("publish uplink message error")
			}
		}
	}()

I think this needs to be changed to (untested):
	go func() {
		for rxPacket := range gw.RXPacketChan() {
                    func(rxPacket gw.RXPacketByte) {
                 	if err := pubsub.PublishGatewayRX(rxPacket.RXInfo.MAC, rxPacket); err != nil {
				log.WithError(err).Error("publish uplink message error")
			}
                    }(rxPacket)
		}
	}()

Would you be able to update these for loops (there are a couple in runV2 and runV3) and do some testing?

I will have a look in the next few days. Yes I'll do throughput tests to make sure it's a workable solution. I'll keep you posted.


Font Used In Screenshots

Hey, I was just wonder what font was used in the screenshots. Its nowhere on the page and I was really curious what its called/if I could have a link to it.

Hey, thanks! The font is actually a font I designed called Anomaly Mono, available here: https://github.com/benbusby/anomaly-mono.

Thank you! I really like the look of it, great work!


49: move to position from analysis

Fixes #49
Note: branched from https://github.com/DiamondLightSource/python-artemis/pull/110 so merge that first

I noticed the grid position -> motor position method we have doesn't account for omega at all, is that already taken care of by epics/analysis or should we be adjusting the motor positions depending on our scan's omega angle? (as an example, we'll get different centring results taking scans at omega=0-90 vs omega=90-180 as the y/z axes will be flipped, but we should find the same x, y, z motor positions in each case)

I think I do need to think about Ben's comment here. I will need to:

Ensure the grid scan is done from omega = 0
Move to omega = 0 before moving the other axes

After discussing with @ndg63276 and @npat113 it's ok to assume omega = 0 for now. We will fix this in https://github.com/DiamondLightSource/python-artemis/issues/154

As seen on beamline the assert for the omega needs to be less severe and just check that they are close


Rewards are not included in history when "up to block height" block value is changed

Actual behavior:
When I try to export the transaction history with option "include rewards" activated, the export also includes the rewards.
But as the export fails when trying to export more than 2100000 transactions, I tried to export with value "up to block height" set to 1300000. Even if "include rewards" is set and no matter how I tried to set the values in the dialog ("up to" first, than number of transactions or vice versa or activating the "include rewards" at the beginning or end), the export does not inlcude the rewards.
Expected behavior:
Either, it is possible to export more than 2100000 transactions, which may also be a problem of too less memory (export sometimes needs more than 12 GB of RAM)
or
the exported transaction history covers the rewards too, even if exported from a block somewhere in between.
How to reproduce it (as minimally and precisely as possible):
Only the not working way is described here:

Open transaction, wait until loaded
Activate "include rewards" option, wait until loaded
Open "Export wallet data" dialog
Set "up to block height" value to, e.g. 1250000 (or one of the last block height values of the last log file startet from max. block height)
Set number of transactions, wait until export button gets active
Activate option "include rewards" and wait until export button gets activate again (NOTE: this doesn't by far take nearly as long, as if you try to export 2000000 transactions incl. rewards from max. block height)
Click export and save data -> only transcation without rewards are shown

What are your environment parameters?

DeFiCh/app Version: 2.7.3
Operating System (OS): Win 10
– RAM (GB): 16 GB
– CPU (Intel Core i5/i7/etc.): Core i7
Current free storage capacity: 117 GB

Anything else we need to know?
It already worked in the past, I think with version 2.5.x or so
ab3d_logs.zip

Hi @ab3dat , could you transfer your comment to Export Button is clickable too early, we can further discuss/follow up on the issue there. We would like to keep same issues in one thread. The team is aware of this issue.Thank you.


return incorrect values

http://google.com    Ping time was 2 ms
https://google.com  Ping time was 250 ms
google.com              Ping time was 359 ms
and from shell
ping google.com
64 bytes from <IP_ADDRESS>: seq=0 ttl=57 time=56.154 ms

This project is not a true ping. It's more like a load tester, probably issue from this SO question.
👉 https://github.com/jdfreder/pingjs#caveats
How do those values compare to the ones from your Developer Tools network requests?

Because calculation is wrong. It does not take into consideration tcp handshake, http keep-alive, slow start and tcp windows


Fix PowerShell linking for AMD64 builds

Trying to do my own VMWare build on Windows, but it always links the arm script.
I've copied the logic from link-to-bento.sh, but I've only kept the find/replaces that were already present.
I have updated the values to mirror the .sh ones though, memory 2084 to 2048, disk_size 131072 to 524288
Also used Set-Content -NoNewLine to stop adding new lines to end of the output.
13.x branch will need similar but it looks like it also uses a boot_wait replacement.

I've changed the disk_size back, although I'm unclear why link-to-bento.ps1 and link-to-bento.sh would set different values.

It was time to make a 13.0.0 vmware build so rebased.
Two commits, first just copies as homestead.sh and homestead-arm.sh and adds arm changes.
Second adds -NoNewLine to each Set-Content so it doesn't add newlines to the end of the file.


Add ThreadLocals middleware to documentation.

Add documentation for the additional middleware required to use the Twilio gateway for voice.

Thank you!  This just helped me big time!

Thank you!


Long audio Transformer performance and loading older models

Question
A previous issue #265 and  my personal experience with the models in the  shows that the models' performance degrades as the size of the audio increases.

Have you identified the source of the performance degradation ?
What is the ideal audio file length to get best performance ?
If transformers still suffer from the above issue, do you have pointers on how to load other models  with the latest flashlight ?

Additional Context
Flashlight was run from the docker image flml/flashlight:cuda-latest (mage id: 37b70a3e7b65)

Hmm, I noticed performance degradation on the 28M conformer on a long (1.5min clip), it just devolved into nonsense halfway through. Based on this it seems expected: https://github.com/facebookresearch/flashlight/issues/397#issuecomment-756643698
I would probably solve this at inference time by converting long audio into roughly 32s batched chunks with a small overlap during the forward pass, and either throwing away the leading overlap or blending the edges (e.g. take an average of the overlap).
So 90s of audio turns into three chunks during forward pass: (1s pad + 0-31s), (29-61s), (59-90s + 1s pad)
Then you can throw away the leading and trailing padding, average together the 29-31s overlap, and flatten back out to a single emission.

As for loading old models, I have an answer for this - I don't use flashlight serialization for my work, I made up my own format called b2l that can round trip convert between flashlight models and back, and I wrote converters for both old and new flashlight.
This means you can convert an old wav2letter model to b2l with old flashlight, then back to a new flashlight model. I'm considering publishing precompiled binaries for this soon (as it's a huge pain to compile the tool against several versions of flashlight).

Thanks ! I would be quite interested in your tool for model conversion.
On the long audio side, I observed performance degradation with much shorter audio files:

For am_conformer_ctc_stride3_letters_25Mparams_distill model: as low as 20s
For am_transformer_ctc_stride3_letters_300Mparams model: as low as 15s
For am_conformer_ctc_stride3_letters_300Mparams: as low as 25s
For am_conformer_ctc_stride3_letters_87Mparams: as low as 20s

So the chunking approach would probably work better with audio clips in the 10s-15s range


House of the Dead III - Lisa's hair detaches from her head

Software information
The House of the Dead III (PC)
System information

GPU: NVIDIA Geforce RTX 3060 Ti
Driver: NVIDIA 545.29.02
Wine version: Wine-GE-Proton8-23
DXVK version: 1.0

Apitrace file(s)
hod3pc.5.trace.tar.xz
Log files
n/a
The first stage that the character appears (which is stage 2... which is the first real stage, but semantics aside), her hair detaches from her head - almost like it's frozen where it first appears in the shot.
Examples for visual aid, both from Stage 2:

NOTE: Though the apitrace is attached with WineD3D as instructed, some facial models are broken under it and I'm not sure if the resulting trace is reproduced perfectly:
(subtitles glitch here happens on D8VK too)

These issues aren't reproduced in dgvoodoo2-through-DXVK.

@SeongGino Thank you for the very detailed report and the trace. Can confirm I'm seeing the same problem with missing/detached hair.

NOTE: Though the apitrace is attached with WineD3D as instructed, some facial models are broken under it and I'm not sure if the resulting trace is reproduced perfectly

The trace is good as far as I call tell, it's WineD3D that is even more broken than d8vk in this case:

I've looked at the scene and haven't noticed anything off in terms of calls. At least not too odd... the game queries device capabilities and supported texture formats multiple times on each frame, which is certainly... an interesting design.
There's no ProcessVertices involved at least, which would have been my first guess. I guess this needs the RenderDoc treatment. Could be some clamping/bounding issues around DrawIndexedPrimitive/DrawPrimitiveUP that we're not getting exactly right vs native.

In my case resolve heavy stutter and now is more playable

Has this problem been addressed? I haven't seen any notification on about a commit directly addressing this particular issue.

In my case appear this:

Has this problem been addressed? I haven't seen any notification on about a commit directly addressing this particular issue.

Lisa's hair is still broken, so no, the rebase and merge with dxvk didn't improve the situation here. But no worries, we'll get to it eventually.

Has this problem been addressed? I haven't seen any notification on about a commit directly addressing this particular issue.

Lisa's hair is still broken, so no, the rebase and merge with dxvk didn't improve the situation here. But no worries, we'll get to it eventually.

Which is exactly what I was referring to, thank you. I will continue to pray for the recovery of Lisa's hair in silence in the meantime. :)

Has this problem been addressed? I haven't seen any notification on about a commit directly addressing this particular issue.

Lisa's hair is still broken, so no, the rebase and merge with dxvk didn't improve the situation here. But no worries, we'll get to it eventually.

well in my case appear this:

We don't have the game, so only checked the trace. Probably worth rechecking in-game also at this point with latest dxvk master to see if it's fixed, as it appears to be based on @mrdeathjr28's screenshots.

Yea apparently the missing hair is baked in to the trace and shows also on native Windows. Wasn't checked before now

Checked with the artifact from doitsujin/dxvk 4b0c9c6: no change, still yikes material.

That said, if it's merged, should we transfer this issue there or keep it to this repo? And not sure if that would necessitate another trace capture or not.

That said, if it's merged, should we transfer this issue there or keep it to this repo? And not sure if that would necessitate another trace capture or not.

If you have time, please move it over and close this one, yes. You don't need a new trace, attaching the existing one is fine.

Fair enough. Self-closing in favor of doitsujin/dxvk#4130


Carthage Support

I don't believe there is a framework configured, or a shared scheme to build the library using Carthage.

Confirmed. I'll fix it soon


Update README with new docs links

The documentation has moved, this updates the links in the readme as well as fixing some dead links in there.
Signed-off-by: Maartje Eyskens<EMAIL_ADDRESS>
What this PR does / why we need it:
Fixing links in the README.md
Release note:

NONE

🙌
/lgtm
/approve


miners are hidden

Looks like it can't contact the web panel, make sure that you can reach the endpoint.php normally in your browser (from outside your host) and also that the database is writeable.

yes reachable and writable

Are you able to edit a configuration in the "Configurations"? Does the changed configuration remain if you reload the page?

No 3.4.0 has no reported problems.

yes i used hhtps

And you have a valid SSL certificate?

yes

The "Remote Configuration" checks on start and then every 100 minutes after that (to avoid getting your "Remote Configuration" banned).

You can.

You can.

how is specyfy in build time
ex: like sterionp.com/config.txt
(it's never end RAW)


how defined symbol class property?

TypeScript Version:  2.3.4
Code

const key = Symbol.for("key");

class C{
     [key]:number = 1;  // error!  A computed property name in a class property declaration must directly refer to a built-in symbol.
}

See #15473 for progress on making this potentially work.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


Malloc Error

Hi Steve and Vijitha
Steve the improvements to the Primary Generator Action are neat and elegant.
I have had an issue when attempting to
AFRODITE(10405,0x10d594000) malloc: *** error for object 0x7fff7b1cf570: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug
I'm sorry but I just commented things out temporarily just to test the histogramming. Please change the primary generator action back as you see fit… I will try getting it working when you commit that.
Kind regards
Kevin

I will take a look.
Sent via my BlackBerry from Vodacom - let your email find you!

From: Kevin Ching Wei Li<EMAIL_ADDRESS>Date: Thu, 5 Feb 2015 08:07:53 -0800
To<EMAIL_ADDRESS>ReplyTo: KevinCWLi/AFRODITE<EMAIL_ADDRESS>Subject: [AFRODITE] Malloc Error (#1)
Hi Steve and Vijitha
Steve the improvements to the Primary Generator Action are neat and elegant.
I have had an issue when attempting to
AFRODITE(10405,0x10d594000) malloc: *** error for object 0x7fff7b1cf570: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug
I'm sorry but I just commented things out temporarily just to test the histogramming. Please change the primary generator action back as you see fit… I will try getting it working when you commit that.
Kind regards
Kevin
—
Reply to this email directly or view it on GitHubhttps://github.com/KevinCWLi/AFRODITE/issues/1.

UNIVERSITY OF CAPE TOWN
This e-mail is subject to the UCT ICT policies and e-mail disclaimer published on our website at http://www.uct.ac.za/about/policies/emaildisclaimer/ or obtainable from +27 21 650 9111. This e-mail is intended only for the person(s) to whom it is addressed. If the e-mail has reached you in error, please notify the author. If you are not the intended recipient of the e-mail you may not use, disclose, copy, redirect or print the content. If this e-mail is not related to the business of UCT it is sent by the sender in the sender's individual capacity.


Make sure every transaction is complete

Actions creating a transaction might run into funny stuff with the DB. Those actions should be wrapped in DB transactions.

Taken care of in #34


Initiate 3 dummy pages under governance directory

Is your feature request related to a problem?

According to the site skeleton there should be the following pages.

Administration Structure

Government Ordinance

OSB Congregation

Describe the solution you'd like

Create 3 dummy pages for each and link them with the navigation bar.

Reference Links
https://docs.google.com/spreadsheets/d/1l8N5ry_ZHOwsR8cD_WkzQGUOvwzqvYA5NYpnaxnodRE/edit#gid=0

can you assign this to me?


Adding classes to provide different sizes

Hi there,
I like how this stuff made hamburger icons easy to handle! Thank you for your effort.
It would be great if there are classes to be used to control size, for example like adding class large, medium or small to give different sizes look. Or some another method to control hamburger icons size.

@MoradHamdy Hey buddy, do the following:

Open dist/hamburgers.css
Find line with .hamburger-inner, .hamburger-inner::before, .hamburger-inner::after{}  classes
Change height:40px; value to needed size.

+1, this would be nice to have for responsive layouts

As @aleksandrmaksimoviclt suggested, I would advise for customizing the sizing in the CSS file or ideally if you import the SCSS version you could change the variables for the sizing, which is in the documentation.


update guava

https://github.com/google/guava/wiki/CVE-2018-10237

CI looks red: Could not find com.google.guava:guava:24.1.1.
Looking at maven central it looks like the version is missing '-jre'. i.e. It looks like it should be compile group: 'com.google.guava', name: 'guava', version: '24.1.1-jre


Ambassador fails to start after adding TLS

Describe the bug
I have added the full chain and private key in Kube secrets as ambassador-certs.
My service is configured as follows :
getambassador.io/config: |
     ---
     apiVersion: ambassador/v1
     kind:  Mapping
     name:  mapping
     grpc: True
     headers:
       lang: french
     add_request_headers:
       userid: voice
     prefix: /Listener/
     rewrite: /Listener/
     service: service:9080
     timeout_ms: 60000
     tls: ambassador-certs
     # ---
     apiVersion: ambassador/v1
     kind: Module
     name: tls
     config:
     server:
     enabled: True
     secret: ambassador-certs
     ---

When I restart ambassador I encounterd this issue :
File "/usr/lib/python3.6/site-packages/ambassador-0.0.0.dev0-py3.6.egg/ambassador/ir/irresource.py", line 63, in __init__    self.set_active(self.setup(ir, aconf))  File "/usr/lib/python3.6/site-packages/ambassador-0.0.0.dev0-py3.6.egg/ambassador/ir/irambassador.py", line 145, in setup    cert=legacy_ctx, termination=False, validation_ca=None) value = cert.get(key, None) AttributeError: 'str' object has no attribute 'get'
Expected behavior
Ambassador has to read the certificate from kube secret and serve the request
Versions (please complete the following information):

Ambassador: 0.52
Kubernetes environment : gcp cluster
Version : gcp master : 1.11.8-gke.6

The first thing here is that your configuration looks incorrect. I think you want:
getambassador.io/config: |
      ---
      apiVersion: ambassador/v1
      kind:  Mapping
      name:  mapping
      grpc: True
      headers:
        lang: french
      add_request_headers:
        userid: voice
      prefix: /Listener/
      rewrite: /Listener/
      service: service:9080
      timeout_ms: 60000
      tls: server
      ---
      apiVersion: ambassador/v1
      kind: Module
      name: tls
      config:
        server:
          enabled: True
          secret: ambassador-certs

and that may prevent the crash. But. The crash is a separate issue that shouldn't happen.

Also, tls in the Mapping configures the context to use for originating TLS to the upstream service. If you just want Ambassador to serve HTTPS, all you need is the tls Module defining a context named server.
And if you want to originate HTTPS and you don't care about which certificate it uses, just do
service: https://service:9080
in your Mapping and get rid of the tls attribute.


pod install warning error: "No podspec file was found" and running error.

[!] use_native_modules! skipped the react-native dependency 'rn-admob'. No podspec file was found.

then run error:

Unhandled JS Exception: Requiring module "node_modules/rn-admob/RNAdMobRewarded.js", which threw an exception: Invariant Violation: Native module cannot be null.
ExceptionsManager.js:94
Unhandled JS Exception: TypeError: TypeError: Cannot read property 'default' of undefined

rn: 0.60.3
"rn-admob": "^1.0.3"

Can you show here your Podfile?! :/

autolinking, so I didn't edit the pod file.
platform :ios, '9.0'
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'
target 'yi2' do
Pods for yi2
pod 'React', :path => '../node_modules/react-native/'
pod 'React-Core', :path => '../node_modules/react-native/React'
pod 'React-DevSupport', :path => '../node_modules/react-native/React'
pod 'React-fishhook', :path => '../node_modules/react-native/Libraries/fishhook'
pod 'React-RCTActionSheet', :path => '../node_modules/react-native/Libraries/ActionSheetIOS'
pod 'React-RCTAnimation', :path => '../node_modules/react-native/Libraries/NativeAnimation'
pod 'React-RCTBlob', :path => '../node_modules/react-native/Libraries/Blob'
pod 'React-RCTImage', :path => '../node_modules/react-native/Libraries/Image'
pod 'React-RCTLinking', :path => '../node_modules/react-native/Libraries/LinkingIOS'
pod 'React-RCTNetwork', :path => '../node_modules/react-native/Libraries/Network'
pod 'React-RCTSettings', :path => '../node_modules/react-native/Libraries/Settings'
pod 'React-RCTText', :path => '../node_modules/react-native/Libraries/Text'
pod 'React-RCTVibration', :path => '../node_modules/react-native/Libraries/Vibration'
pod 'React-RCTWebSocket', :path => '../node_modules/react-native/Libraries/WebSocket'
pod 'React-cxxreact', :path => '../node_modules/react-native/ReactCommon/cxxreact'
pod 'React-jsi', :path => '../node_modules/react-native/ReactCommon/jsi'
pod 'React-jsiexecutor', :path => '../node_modules/react-native/ReactCommon/jsiexecutor'
pod 'React-jsinspector', :path => '../node_modules/react-native/ReactCommon/jsinspector'
pod 'yoga', :path => '../node_modules/react-native/ReactCommon/yoga'
pod 'DoubleConversion', :podspec => '../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec'
pod 'glog', :podspec => '../node_modules/react-native/third-party-podspecs/glog.podspec'
pod 'Folly', :podspec => '../node_modules/react-native/third-party-podspecs/Folly.podspec'
pod 'RNLocalize', :path => '../node_modules/react-native-localize'
target 'yi2Tests' do
inherit! :search_paths
# Pods for testing
end
use_native_modules!
end
target 'yi2-tvOS' do
Pods for yi2-tvOS
target 'yi2-tvOSTests' do
inherit! :search_paths
# Pods for testing
end
end

Add pod 'RNAdMobManager', :path => '../node_modules/rn-admob' to your Podfile and run pod install

[!] No podspec found for RNAdMobManager in ../node_modules/rn-admob

Do you find RNAdMobManager.podspec in node_modules/rn-admob??

Do you find RNAdMobManager.podspec in node_modules/rn-admob??

no

I removed the package and yarn add rn-admob again, the issue remains.

You can try the following step:

Remove Podfile.lock and Pods folder in ios
Reinstall all lib by the following commands:

rm -rf node_modules
rm yarn.lock
yarn cache clean
yarn install
react-native link rn-admob

cd ios and run pod install

you suggest I link the lib? but the rn version > 0.60

Just do it man! ;) I've tried it before.
Autolinking is not support some libraries and i think this lib is one of them :v

I did all above, after linked the lib, podfile not changed.
and when I debug, the app crashes with errors below:

无法进行调试。执行命令 react-native run-ios --simulator --no-packager --verbose 时出错: 执行命令 react-native run-ios --simulator --no-packager --verbose 时出错 (error code 101)

@Long2k1708 thank u for the patience, I'd like to init a new project and try this lib again.

@RohanSensei what are the results? Is it working for you?


Fix: Disabling while in 'serve' mode after first run

This fixes a case where rebuilding the documentation when running in serve mode will ignore the enabled_on_serve.
When running exclude-unused-files in conjunction with mkdocs-material (9.5.25), I noticed that runs after the first run in serve mode were ignoring the enabled_on_serve flag. First run would be 1.3s, second would be 5.0 seconds.
After some log debugging, it turns out that self.config is actually reset between build runs in serve mode. And therefore setting self.config.enabled = False was not sticking.
To account for that, I stored an alternative variable disabled_by_serve that's tied to the self variable, outside of the config. This way, it should not be reset between runs.

Please Note: I've never really programmed in Python before, so debugging this has been a unique experience. :sweat_smile:

Hi, first of all; thanks for catching this bug and sorry for the review delay from my side! Great find :)
Second:
Maybe instead of introducing a second flag called self.disabled_by_serve we could just refactor and use one unified flag like self.is_enabled which we initially set to the config value self.config.enabled and then when checking for the self.config.enabled_on_serve instead of setting self.config.enabled = False like we did before (which is wrong), we could just set self.is_enabled to False.
I hope I didn't explain this too complicated 😅, but the gist is that maybe it's better to keep one final source of truth that decides if the plugin's functionality should be enabled instead of checking for multiple variables (if not self.config.enabled or self.disabled_by_serve) so that we don't forget to check for both variables when editing the code in the future and only have to check one source-of-truth variable.
What do you think about that? @Sewer56
You did really well even without being a Python dev :D
Greetings from Austria

That's a good idea actually, didn't think of unifying the value, would keep things simpler.
I'll get this sorted later today.

Oops, I forgot about this for a second somehow.
Anyway, the latest commit should get this sorted, pretty trivial patch.

Hi, thanks for the addition @Sewer56
Looks good to me :)
I just quickly added a commit running the formatter (poetry run pre-commit run --all-files) to fix 2 whitespace issues in the PR.
I will release a new version including this change later today!
Greetings from Austria and I wish you a relaxing sunday

No worries.
And yeah, it does remind me, there's no decent universal ways to set things up like pre-commit hooks to automatically kick in locally. Every user has to manually init.
I do know it's possible to set up bots to format all incoming PRs provided user enables 'allow edits by maintainer' when submitting, but I've never looked into it much.

Yeah, tbh I agree, pre-commit hooks are not that optimal
Running linting in CI Pipelines is probably still the best choice, but I also haven't looked into auto-formatting bots that add commits proactively yet


Add a photo viewing web UI

We can walk through this on Monday.
It's about 90% old code, 50% of which has been modified to work with the new noms SDK and index structure.

Current coverage is 87.56% (diff: 100%)

Merging #2662 into master will decrease coverage by 0.01%

@@             master      #2662   diff @@
==========================================
  Files           242        242
  Lines         18110      18110
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          15860      15858     -2
- Misses         1985       1987     +2
  Partials        265        265

Powered by Codecov. Last update 40b28f9...6f99db4

PTAL I removed the flow files

what is holding this up?

SLGTM ?

ptal :-)

Should there be a README.md explaining how to use this?


Patterns["ISIS"] is not usable

Env: MSFS SU11+ Fenix A320 v<IP_ADDRESS>
Patterns["ISIS"] I always got false and 1013. so BARO data are not usable. Thank you

Always can't be true - I've tested with SU11/1.0.6 initially and just retested it: The ISIS Barometer was working and could also be switchted to STD.
Maybe there are now variations in the Location which I don't have yet included, but it should be working sometimes at least.
Are you sure the Binary is working properly? Do other Values work? (Only the second Battery does not work at all currently)
Or maybe your Loading Times are very long so you don't press "ready to fly" in time. Try to stop and start the Binary manually when the Session is completely loaded (sitting at the Gate). Does it work then?

Thanks.
I have  tested to start it very later, I have confirmed FenixSystem is ready. I still get false and 1013 fixed values of Patterns["ISIS"]
FCU SPD
FCU HDG
FCU HDG FMA
FCU ALT
FCU VS
FCU VS FMA
these values are Good,

Very later? No need to wait extra long or anything like that, it is just important that the initial scan is done when the session is really running (otherwise some locations are not found).
Why would you need to confirm that FenixSystem is ready as an average user? And how do you confirm that?
And when you say "still get false" - you mean the Offset is always zero?
Could you explain how you're are using the binary exactly? I somehow have the suspicion you're using it in a different use-case than it is designed for ...

Because I am debugging your source code to adapt my environment.
ElementManager: Offset for  is at 0x542C:6:s
MemoryScanner: Pattern Search took 0.192883s
Program: Pattern  is at Address 0x5CC683C (97282108)
looks like location is found

Well, maybe it would have been wiser to start with that ;)
It normally does not make a difference when a debugger is attached - but did you try it without debugging?
In your initial Description you've said either it is false or 1013? Maybe it is have something to do with the way you are reading that Offset? It can be 6 characters long, but sometime it is shorter.
When you're debugging it - do you see the Value change in the Code?

I am debugging isis.MemoryOffsets["isisBaro"].GetValue() again. the result is fixed whenever I change BARO value in Fenix
double hex = 0x408FA80000000000 = 1.013

Aha - okay.
You know, when I'm being brutally honest: I can't help you like that. I'll put more effort in asking questions and trying to understand the situation than you answering them or explaining what exactly you're trying to do.
I can't help you, if you don't help my to understand the situation better.
All I can say: the Binary is working, when used in the intended way - especially the ISIS Baro.
What is the Reason you can't use the resulting String which is created by the Binary?
Did you have a look at the other open Issue? Would an Option/Feature that outputs the Raw-Values directly (instead of a formatted string) also help with your Use-Case?

Fenix A320 v<IP_ADDRESS> has a different ISIS pattern layout. the issue is solved,


Repurpose network_config_override variable

This change repurposes the edpm_network_config_override variable to accept a Jinja2 formatted string that will be rendered. This change will enable us to eventually replace the edpm_network_config_template variable and remove this override one entirely.

recheck

recheck

recheck

recheck

Looks like the deployment itself succeeded, the OpenStackDataPlaneNodeSet deployed fine, but we appear to have Vexxhost API errors:
HttpException: 500: Server Error for url: https://compute.rdo.mtl2.vexxhost.net/v2.1/servers/658d8e2d-57c0-4dbc-976f-a687d37f1262, Unexpected API Error. Please report this at http://bugs.launchpad.net/nova/ and attach the Nova API log if possible.
<class 'sqlalchemy.exc.TimeoutError'>

recheck

Successful build reported here:
https://logserver.rdoproject.org/41/341/15fd40bffc2f15bed77396d58925836c2bded6d1/github-check/podified-multinode-edpm-deployment-crc/2c60970/controller/ci-framework-data/logs/crc/crs/openstackdataplanenodesets.yaml
Will check CI again later once infra issues are sorted.