additional bash scripts

There's a few items to add to this repo either by extracting from other repos, or reproducing entirely here.
TODO

[ ] https://github.com/mattcantstop/gitline
[ ] https://github.com/nathos/ghinfo
[ ] https://github.com/IQAndreas/github-issues-import
[ ] https://github.com/WtfJoke/rtc2git

cc @matthewmccullough @mattcantstop @jordanmccullough @nathos @PatrickBM @crichID @loranallensmith @jaw6.
If any of you have ideas on how to do this or preferences how some of those get over here let me know. Otherwise i'll come up with something!

Amazing timing, @brntbeer! This just came up in conversation regarding bash script examples we'd like to refer to during and after our Implementation engagements.
To ensure the community and GitHub customers benefit from a single location, we might want to get a final :+1: from the github/platform-samples curators and place the example scripts over there.
Thoughts?

My preference is for our team to own the repository that we share with customers and the community at large. We can bring in resources from other places but I prefer we have our own repository.

If any of you have ideas on how to do this

It's kind of "stupid git tricks" (I had to confirm it would work, even) but... you could add those repos as remotes and just merge them in. (There'll be lots of readme conflicts, I'm sure.) It's almost certainly going to be faster/better organized to copy individual files over, but I guess you'd lose the history.
I guess my question would be: do you intend this to be a collection of useful scripts or demo scripts? Like, "here are things you might want to try before writing your own" or "here are things you could read and get ideas for what's possible" ?
I'm guessing it's more the first one...? I guess what I'm getting at is if the purpose is to be generally useful (like, do something) then I'd just copy them over as-is; if the purpose is to be instructional (like, look and then do something else) it might be more valuable to adapt those scripts to fit in with the others already in here? (Or, if showing a diversity of approaches is its own thing, maybe folders to group these into similar categories or something?)
I'm ... not sure if I'm actually helping?

Thanks everyone for the inputs and ideas. We're definitely all on the same page regarding a one-repo resource for ease of reference for ourselves as much as our customers. :sparkles:
I've reached out to the Platform folks and have the conversation started for us to begin using github/platform-examples. This provides us with a place where engineering can help us curate and caretake focused examples.

This just came up in conversation regarding bash script examples we'd like to refer to during and after our Implementation engagements.

Thanks @jordanmccullough ! This was actually why i made this in the first place =) I commented over on that thread too.
Glad to see these being in the community spot of platform-examples tho!

@brntbeer, Appreciate the response and opening this thread which coincided so well with conversations.
@brntbeer, @mattcantstop, @jaw6, I'm thrilled the Implementations and Training teams continue to play a crucial role in enabling customers and provide a springboard for extending the platform.
Aligning this goal with the established platform-examples/ repository and capitalizing on existing engineering resources garners all the more strength and support for our efforts. The Docs Team's interest in aligning with API examples and Developer Guides is all the more exciting, too.
If there are any other details to sort out, decisions which examples to submit first, or more what best next-steps look like, the team is just an @github/services-implementationspecialists mention away.


Fix #836, Replace CFE_MISSION_SPACECRAFT_ID with CFE_PSP_GetSpacecraftId()

Describe the contribution
Fix #836 - replaced CFE_MISSION_SPACECRAFT_ID use with CFE_PSP_GetSpacecraftId() and updated unit test
Testing performed
Built version 2 with unit tests, executed and passed
Expected behavior changes
No longer uses soon to be deprecated CFE_MISSION_SPACECRAFT_ID
System(s) tested on

Hardware: cFS Dev Server
OS: Ubuntu 18.04
Versions: bundle integration candidate + this change

Additional context
Depends on changes in integration candidate
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

CCB 2020-08-26 - APPROVED


WIP: Benchmarking 193

Fixes #193
I have moved a limited amout of text on benchmarking in README.md to the added BENCHMARKING.md file.
If you prefer to move more of it, either let me know or push a commit later?
Since adding markdown files and linking from README.md, I have also:

Moved contributing info into a CONTRIBUTING.md file and linked from main
Added a link-check step at the end of CI, so broken links between docs won't make it to main branch

I will now focus adding docs as per chat comments and maybe some minimal automation of some of that.

@davidlattimore I will add some questions I have in a code review, so beside the text in question

The code changes are due to me creating this branch from main, with my initial changes for forking, which was a mistake (didn't expect to have multiple branches/PRs open so soon...).
They will definitely be removed one way or other before being ready to merge.

Removed code changes via rebase on main.
Ready for review.
I have other scratch pad notes that I will continue working on to extend discussion about benchmarking - but I think these docs are correct as they stand with the current code, and so can be merged after review.

Thanks!


chore: ✨ introduce v1.0.0 (#8)

Codecov Report
Base: 90.86% // Head: 90.86% // No change to project coverage :thumbsup:

Coverage data is based on head (b47dd92) compared to base (d0385d8).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop       #9   +/-   ##
========================================
  Coverage    90.86%   90.86%
========================================
  Files           27       27
  Lines          372      372
  Branches        50       50
========================================
  Hits           338      338
  Misses          31       31
  Partials         3        3

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


When running wasm compiled by go, I encountered an error. Please help me.

When running wasm compiled by go, I encountered an error. Please help me.
The go code(test.go) to compile into wasm is as follows.
package main

import "fmt"

func main() {
	rs := Dododo(54, 21)
	fmt.Println(fmt.Sprintf("rs: %d", rs))
}

func Dododo(x int, y int) int {
	return x - y
}

Then compile to wasm.
$ GOOS=js GOARCH=wasm go build -o test.wasm test.go

The code to run wasm is as follows.
package main

import (
	"fmt"
	wasm "github.com/wasmerio/go-ext-wasm/wasmer"
)

func main() {
	testTest()
}

func testTest() {
	bytes, err := wasm.ReadBytes("test.wasm")
	if err != nil {
		fmt.Printf(fmt.Sprintf("Error wasm.ReadBytes, err: %s", err))
		return
	}

	instance, err := wasm.NewInstance(bytes)
	if err != nil {
		fmt.Printf(fmt.Sprintf("Error wasm.NewInstance, err: %s", err))
		return
	}
	defer instance.Close()

	Dododo := instance.Exports["Dododo"]

	result, err := Dododo(86, 37)
	if err != nil {
		fmt.Printf(fmt.Sprintf("Error Dododo, err: %s", err))
		return
	}

	fmt.Println(result)
}

Then the error message is as follows.
Error wasm.NewInstance, err: Failed to instantiate the module:
    19 link errors: (1 of 19) Import not found, namespace: go, name: debug (2 of 19) Import not found, namespace: go, name: runtime.wasmExit (3 of 19) Import not found, namespace: go, name: runtime.wasmWrite (4 of 19) Import not found, namespace: go, name: runtime.nanotime (5 of 19) Import not found, namespace: go, name: runtime.walltime (6 of 19) Import not found, namespace: go, name: runtime.scheduleTimeoutEvent (7 of 19) Import not found, namespace: go, name: runtime.clearTimeoutEvent (8 of 19) Import not found, namespace: go, name: runtime.getRandomData (9 of 19) Import not found, namespace: go, name: syscall/js.stringVal (10 of 19) Import not found, namespace: go, name: syscall/js.valueGet (11 of 19) Import not found, namespace: go, name: syscall/js.valueSet (12 of 19) Import not found, namespace: go, name: syscall/js.valueIndex (13 of 19) Import not found, namespace: go, name: syscall/js.valueSetIndex (14 of 19) Import not found, namespace: go, name: syscall/js.valueCall (15 of 19) Import not found, namespace: go, name: syscall/js.valueNew (16 of 19) Import not found, namespace: go, name: syscall/js.valueLength (17 of 19) Import not found, namespace: go, name: syscall/js.valuePrepareString (18 of 19) Import not found, namespace: go, name: syscall/js.valueLoadString (19 of 19) Import not found, namespace: go, name: syscall/js.copyBytesToJS

When running wasm compiled by go, I encountered an error. Please help me.
The go code(test.go) to compile into wasm is as follows.
package main

import "fmt"

func main() {
	rs := Dododo(54, 21)
	fmt.Println(fmt.Sprintf("rs: %d", rs))
}

func Dododo(x int, y int) int {
	return x - y
}

Then compile to wasm.
$ GOOS=js GOARCH=wasm go build -o test.wasm test.go

The code to run wasm is as follows.
package main

import (
	"fmt"
	wasm "github.com/wasmerio/go-ext-wasm/wasmer"
)

func main() {
	testTest()
}

func testTest() {
	bytes, err := wasm.ReadBytes("test.wasm")
	if err != nil {
		fmt.Printf(fmt.Sprintf("Error wasm.ReadBytes, err: %s", err))
		return
	}

	instance, err := wasm.NewInstance(bytes)
	if err != nil {
		fmt.Printf(fmt.Sprintf("Error wasm.NewInstance, err: %s", err))
		return
	}
	defer instance.Close()

	Dododo := instance.Exports["Dododo"]

	result, err := Dododo(86, 37)
	if err != nil {
		fmt.Printf(fmt.Sprintf("Error Dododo, err: %s", err))
		return
	}

	fmt.Println(result)
}

Then the error message is as follows.
Error wasm.NewInstance, err: Failed to instantiate the module:
    19 link errors: (1 of 19) Import not found, namespace: go, name: debug (2 of 19) Import not found, namespace: go, name: runtime.wasmExit (3 of 19) Import not found, namespace: go, name: runtime.wasmWrite (4 of 19) Import not found, namespace: go, name: runtime.nanotime (5 of 19) Import not found, namespace: go, name: runtime.walltime (6 of 19) Import not found, namespace: go, name: runtime.scheduleTimeoutEvent (7 of 19) Import not found, namespace: go, name: runtime.clearTimeoutEvent (8 of 19) Import not found, namespace: go, name: runtime.getRandomData (9 of 19) Import not found, namespace: go, name: syscall/js.stringVal (10 of 19) Import not found, namespace: go, name: syscall/js.valueGet (11 of 19) Import not found, namespace: go, name: syscall/js.valueSet (12 of 19) Import not found, namespace: go, name: syscall/js.valueIndex (13 of 19) Import not found, namespace: go, name: syscall/js.valueSetIndex (14 of 19) Import not found, namespace: go, name: syscall/js.valueCall (15 of 19) Import not found, namespace: go, name: syscall/js.valueNew (16 of 19) Import not found, namespace: go, name: syscall/js.valueLength (17 of 19) Import not found, namespace: go, name: syscall/js.valuePrepareString (18 of 19) Import not found, namespace: go, name: syscall/js.valueLoadString (19 of 19) Import not found, namespace: go, name: syscall/js.copyBytesToJS

Could you give me an example of go code to compile into wasm and run on this project? Thanks!

Could you please give me an example of go code to compile into wasm and run on this project? Thanks!

Check #95 to learn more. Go compiler generates Wasm module tailored for JS only for the moment. Closing this issue, as it is duplicate of others, but #95 is the central discussion.


BBE Ballerina Test example fixes

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #18922 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #18922   +/-   ##
=======================================
  Coverage   15.73%   15.73%
=======================================
  Files          47       47
  Lines        1258     1258
  Branches      195      195
=======================================
  Hits          198      198
  Misses       1047     1047
  Partials       13       13

Continue to review full report at Codecov.

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


Added About.jsx

Resolved issue #522
Kindly review it.I have added new About.jsx

Resolved issue #522 Kindly review it.I have added new About.jsx

Kindly follow the template for creating issue. @DinkyRajpoot56

@DinkyRajpoot56 create a new PR following the template which is automatically generated. Otherwise we won't be able to merge your changes.


Add version env var support

Description

This tweaks the behavior of pulumi-java-gen for the --version flag based on feedback from @stack72.
Problem: the basic issue is that we do not want the generated code to depend on the version since that creates spurious source code diffs any time version changes, and that can be every commit.
Solution: --version becomes optional in pulumi-java-gen, recommended not to set. The generated build code will default to 0.0.1 but let the user set the version by env var PROJECT_VERSION or else the Gradle property gradle -Pversion=1.2.3 build.
Crucially, Gradle build code and not pulumi-java-gen will now generate version.txt and plugin.json resources as part of the build, so that the released Java code is aware of its own version. This is more foolproof. The situation where Gradle built an artifact with version X but version.txt contains version Y because pulumi-java-gen was run with a different --version is prevented here.
Test script:
#!/usr/bin/env bash

set -euo pipefail

rm -rf bin/pulumi-java-gen

make bin/pulumi-java-gen

./bin/pulumi-java-gen \
    generate \
    --sdk 0.4.0 \
    --out gen \
    --schema https://raw.githubusercontent.com/pulumi/pulumi-random/v4.6.0/provider/cmd/pulumi-resource-random/schema.json

echo "Found resources: "
(find gen | grep resources) || echo "none"
echo "--"

echo "Building"
(cd gen && PROJECT_VERSION=0.2.1-alpha gradle build)
echo "Found resources: "
find gen | grep resources
echo "--"
echo "Version: "
cat $(find gen | grep resources | grep version.txt)
echo ""
echo "--"

Fixes # (issue)
Checklist

[ ] I have added tests that prove my fix is effective or that my feature works

[ ] I have updated the CHANGELOG-PENDING file with my change

[ ] Yes, there are changes in this PR that warrants bumping the Pulumi Service API version

Broke version.txt generation on the legacy path (in-repo) it seems, fixing.


ComboBox ignores first character typed after clearing

https://www.loom.com/share/b3f5b0a750d24e399289598be37c2b18
The first character typed just opens the dropdown, it doesn't enter the character into input. See above video, I attempted to type 'qwerty' both times but the second time chops off the 'q'.

Strange. It doesn't work in Chrome 107 (and Brave), but works fine in Firefox 107.

It's related to afterUpdate handler.


Specify FHIR resources for using in clinical documents

Several options, we don't have strict guidance yet on which one is best.

Prescription contains Medication as normal, and that Medication has extensions for the attributes
1.1. Prescription contains additional Medication besides the original, and that new Medication has extensions for the attributes
Prescription contains Medication which has the link to a contained MedicationKnowledge, and that MedicationKnowledge has the attributes
2.1. Prescription contains additional MedicationKnowledge (same as above but Medication stays as is, the extension is on Prescription)
We use the medication definitional resources
...

The version here is also 4.3.0?
Because for MedicationKnowledge, the difference between R4B and R5 is very big.
My gut feeling says Medication is more IDMP-compatible than MedicationKnowledge. But actual analysis would be welcome (and please include combination packages like Canifug Cremolum and Qlaira).
MedicationDefinition module resources would probably not look good on a prescription, but again, it would be correct to have a reasoning to go with the decision.

Beyond UNICOM scope.


admin page of user services - simplifing the filter list

On the page http://localhost:8000/admin/django_th/userservice/, the content list of the filter, at the right of the page, should to be reduced.
Actually :
For each UserService, there are as far as user than userservice entries
Expected Result:

For 'user' we just need to display the auth_user model (i suppose) to filter to current userservice list for a given user

@foxmask Can i work on this?

yes sure.
if you want to participate to the hacktoberfest, dont forget to register :)

@foxmask I tried to install locally, but i am not abe to run :(

what's the problem ?

you need to do :

clone the repo for you
python3.6 -m venv foxmask (for example)
git clone your repo
cd to the django-th
source ../bin/activate
pip install -r requirements.txt
./manage.py migrate
./manage.py runserver

http://localhost:8000/th/ is the main page of the project
http://localhost:8000/admin/ for the admin page

@foxmask Small issue before i fix this 😄

set DEBUG to True :)

@foxmask This behaviour i have seen for first time, when debug=true everything working smoothely but debug=false everything breaks

Yes I know. I thought I fixed it earlier. I'm going to check if a CSS is not missing.

it's ok when debug is set to False and the HTTP config point on the static folder correctly
if the result of collectstatic is in /home/foxmask/DjangoVirtualEnv/django-trigger-happy36/django-th/static, then, for example for nginx i'll do
    # ROOT website
    root  /home/foxmask/DjangoVirtualEnv/django-trigger-happy36/django-th/;
    location /static/ {
            root  /home/foxmask/DjangoVirtualEnv/django-trigger-happy36/django-th/;
    }

does the trick

I think, I will set DEBUG to False and add to the doc how to manage when set DEBUG to True and set NGINX  config

@foxmask 👍   I was going throught the codebase and also can you please exlain what i have to do as i am not able to understand 😅

Actually, that page http://localhost:8000/admin/django_th/userservice/ display the list of the service that one user has decided to enable for his own usage.
If there are 5 users that have enabled 10 services I have a list of 50 users in the filter list.
I would like to reduced this list in the filter list (at the right of the page), by only displaying one user and when I select one user, this should filter and only display the service of that user.
nota : as the users are in the auth_user model, I dont know if such a thing is possible to do in the admin on the UserService page ?

Is anyone still working on this? Can I grab this one?

Hi I think that if vaibhavsingh97 does not reply  tomorrow, then you have my agreement to work on it

@foxmask I am trying my best to fix this issue, Gimme some time 😄

No pb :)

@vaibhavsingh97 can I accept the @scomert PR or you are still on that issue ?

@foxmask Yes, Please...i had exams so i was not able to work.

thank you
and good luck for exams


Add extracting command for iTunes downloaded from Microsoft Store

The last time I tried to verify this on Windows, iTunes wasn't recognizing my iPhone. I might try again on a different Windows machine later.

Hi @KnugiHK , I tested it on same day (September 29th) and it worked like a charm! (also, thanks for the tool, very useful.)


Showing sub-models as part of enum?

I have a model that has a discriminator, "_type". That discriminator needs to be one of the names of the subTypes of the current model. I currently have it listed in the swagger JSON as an enum which appears as an array of [string] and the model JSON for the subTypes isn't shown.
So, is there a way to show a specific list of values for the discriminator and have those be all of the subTypes? If so, will it show all of the models for the subTypes?
Rule {
rule (RuleDetail): The details of the rule itself.,
data_sources (array[string]): The ids of the Data Sources that will be covered by this rule.
}
RuleDetail {
description (string): The general description for users.,
condition (RuleCondition, optional): The rule condition that will be evaluated.,
then_actions (array[RuleAction], optional): The rule actions that will be executed if the rule condition evaluates to true.,
else_actions (array[RuleAction], optional): The rule actions that will be executed if the rule condition evaluates to false.,
downstream_rule_id (string, optional): The _id of the rule that, if evaluated to true after this rule evaluates to true will trigger the delayed actions.,
delayed_actions (array[RuleAction], optional): The rule actions that will be executed if the rule condition evaluates to true and the next evaluation of the downstream rule evaluates to true.,
active (boolean, optional): Indicates if the rule is active.
}
RuleCondition {
parent_or_sibling_key (string, optional): The payload key that will indicate if we are in the correct part of the structure for the given property being evaluated.,
parent_or_sibling_value (string, optional): The value of the parent_or_sibling_key that will indicate we're int he correct part of the payload structure.,
_type (string) = ['EqualRuleCondition' or 'NotEqualRuleCondition' or 'GreaterThanRuleCondition' or 'GreaterThanEqualRuleCondition' or 'LessThanRuleCondition' or 'LessThanEqualRuleCondition' or 'MaxTimeBetweenReportsRuleCondition' or 'NearRuleCondition' or 'NotNearRuleCondition' or 'InRuleCondition' or 'JoinCondition']: The actual type of rule condition.
}

I imagine you're referring to the swagger-ui project, in which it is not yet supported. Please follow this ticket - https://github.com/swagger-api/swagger-ui/issues/300.

Right. Of course. Thank you for the swift response. Will follow.


Ytviewer not working on Mac?

Issue Label:

[x] bug
[ ] feature
[ ] enhancement

Description of bug or feature enhancement:
So, @alexanderepstein. You say that ytviewer isn't working on mac? What is the issue? Is it not playing the video? The only problem I get when running it on a friend's mac is that there is no video player to play it with, but I'm sure if you had vlc installed it would work? Please detail the issues.
OS and OS version:

[x] Mac
[ ] Linux 32 Bit
[ ] Linux 64 Bit
[ ] Windows 32 Bit
[ ] Windows 64 Bit

OS Version:

I misspoke it does work on Mac now I solved the main issue in one of the commits to the fork but forgot to retest it.
The only issue is on OSX it cannot be called as vlc it must be called as /Applications/VLC.app/Contents/MacOS/VLC which will be okay since I will automate finding the video player so the user never has to enter it.

Alright, thanks!


Shopify handle in view model

We are using umbraco to feed some content to a shopfiy hydrogen front end, it would be awesome if the product's handle could be returned to the view model too.
Hyrdogen uses the product handle in the url, which is passed though the the graphql query that fetches the page data.
We would like to run an asyncrophs request to get futher rich content for based on the same handle, but there is not a good way to match the products at the moment.
If we can't do this - I think we can get the raw value from the property, and run the query ourselfs.
As a further note, I would like to create a PR for this issue myself as it would be trivial, but I need a little bit of help figuring out how exaclty to pull the repo down and test the package locally, are there any resources about how to get started?
Thanks so much for these integrations :)

Hi @james-whittington1 ,
You can fork the repository, add the changes and then submit a PR that I'll review afterwards.
Thank you,
Adrian

I want to test that the change actually works when the integration is installed though (mostly to save you time!)
If I just code it blindly there is a fairly high chance it might not work quite as I expect when it's actually being used?

If it helps, you could send me the code snippet and I would try to test and implement.

Hi @james-whittington1 , is this still actual?
Thanks,
Adrian

Hi @james-whittington1 ,
I am closing this now due to long inactivity. We have released a new version of the integration with support for pagination.
Regards,
Adrian


Permission denied (publickey)

after following these commands in azure:
az group create --name myResourceGroup --location westeurope
az vm create \
  --resource-group myResourceGroup \
  --name myVM \
  --image UbuntuLTS \
  --admin-username azureuser \
  --generate-ssh-keys

ssh azureuser@publicIpAddress
I get this error
Permission denied (publickey)
and i am not able to connect to VM. any idea where is the problem?

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

ID: cf395b11-607d-1421-3b71-4dc442535a1c
Version Independent ID: c5f1f9fc-6b74-5cd4-b411-6676863e6d1a
Content: Quickstart - Create a Linux VM with the Azure CLI
Content Source: articles/virtual-machines/linux/quick-create-cli.md
Service: virtual-machines-linux
GitHub Login: @cynthn
Microsoft Alias: cynthn

Hi @soheilade I'm sorry to hear that you are facing this issue. Could you please link the URL of the documentation you were following? That way, we can pass your feedback to the appropriate content author.

Hi @soheilade just a friendly reminder to share the link to the doc this feedback is for. If this is not related to any doc I will suggest you to please search and or post on the developer forums like StackOverflow where unlike here there is a community that can help.

https://docs.microsoft.com/en-us/azure/virtual-machines/linux/quick-create-cli?toc=%2Fazure%2Fvirtual-machines%2Flinux%2Ftoc.json

@soheilade in the cli command to create vm the flag --generate-ssh-keys is given.
Do it would have picked the public key in your home directory (~/.ssh/id_rsa.pub) and uploaded  it to the Azure virtual machine.
Then you can use the private key (~/.ssh/id_rsa) to login .
Make sure you have that file in your machine.
Please try with the below command which explicitly specifies which key to use.
ssh -i ~/.ssh/id_rsa azureuser@yourpublicip
Also the permissions of the id_rsa should be 400
-rw------- 1 user user 1679 Feb 18  2019 /home/user/.ssh/id_rsa
you can also go to the password option instead of ssh keys.
for specifying password us the flag --admin-password

@jakaruna-MSFT , thanks, but how do I move the public key to a VM that I cannot access to? The serial console of azure is not working either.

@soheilade
You need to enable boot diagnostics for the serial console.
That public key would have been placed in the VM automatically by the CLI.
You can find that on the ~/.ssh/authorized_keys file in the Azure virtual machine
If you are looking for a way to login to that VM,  You can easily reset the password for your username or create a new user with username and password.
For resetting the password, Please go to the VM blade and search for "reset password"
Then provide the required details ans submit.

Try this out and let me know.

@jakaruna-MSFT thanks, i needed to create a user. problem solved 👍

hey @jakaruna-MSFT, what do you mean by creating a user?

hey, @soheilade what do you mean by creating a user?

@dbshawt In order to use the serial console you have to have a user account on the machine that is secured with a password (not an SSH key). I think that might be what they mean by creating an account - the original VM used an SSH key, so they needed to create a new acocunt with a password to use serial console.
But, someone can correct me if I am wrong :)


Colegio San Gabriel Arcangel

Colegio San Gabriel Arcangel, the domains seems only to be used with Google Workspace for education due to pandemic, not sure

@tcrms Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@tcrms Please provide us with:
* the school official website URL

* the school street address, including city and country

* an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course

* a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

thats the school official url, tho not website (as i mentioned, they use Google Workspace)
Please contact me on Discord or email<EMAIL_ADDRESS>to provide this "privately"
(check the first one)
Sure, give me a email address and I'll make sure to try prove as u ask (sending a email with it, showing screenshot of classroom.google.com or something else)

@tcrms Thank you!
Please send a message to<EMAIL_ADDRESS>with a subject: ATTN Philip Torchinsky.
You can share the school street address, including city and country there, and provide a screenshot or another proof.
Thank you!

@tcrms You might miss my previous comment. Please react.

@tcrms  Thank you for the clarifications. Pull request merged.


org settings: Set message_retention_days to NULL for retain forever case.

Rather than saving Realm.RETAIN_MESSAGE_FOREVER to the database when we
want message retention forever, we will save it as NULL (None) in the
database as NULL implies "no realm-level message retention policy set".
In frontend discard_property_element_changes() couldn't handle the case
when the updated value of the setting is null, so fixed that too.

@mateuszmandera what are your thoughts on this?  It feels to me like maybe we should just change Realm.message_retention_days to use -1 as "Retain forever" rather than None?

But then we still have the  Stream issue to contend with - how to do None there. Alternative idea: 0 seems like a rather useless, if not nonsensical, value for "retention days", if not downright dangerous. Should we make these field non-nullable and make 0 be a special value meaning what None means right now, for both Stream and Realm?

I think for the stream case, if you send the same value the realm has, it changes you to None on the backend.  That's how the stream notification settings work.

@pragatiagrawal31 Do you still have time/availability to work on this PR?  If so, can you try to rebase this PR to the latest version of master and give a status update?
(I am a little out of the loop here on who else may be working in this code, but I am trying to keep track of settings-related commits for other reasons.)

(I'll finish this tomorrow morning as the priority)
On Sat, Jun 6, 2020 at 7:43 PM Steve Howell<EMAIL_ADDRESS>wrote:

@pragatiagrawal31 https://github.com/pragatiagrawal31 Do you still have
time/availability to work on this PR? If so, can you try to rebase this PR
to the latest version of master and give a status update?
(I am a little out of the loop here on who else may be working in this
code, but I am trying to keep track of settings-related commits for other
reasons.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/pull/14938#issuecomment-640067298, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJTWQKBS2FZEVSWWSAA6TGDRVJFHXANCNFSM4M56VMOQ
.

@mateuszmandera does this logic make sense to you?

Yeah makes sense to me. I think RETAIN_MESSAGE_FOREVER should be called something more like RETAIN_MESSAGES_FOREVER_API_VALUE perhaps? Unless we want to change the meaning of the message_retention_days values according to one of our suggestions above.

Yeah, to be clear, we'll still be using RETAIN_MESSAGE_FOREVER in the database; it'll just be represented as "forever", not -1, in the API interface.

Yup, that's how i understood it (for stream, for realms we only have "no policy" concept)

@sahil839 can you coordinate with @pragatiagrawal31 to make sure this gets finished in the next few days?

I've made some changes, please review it, I wish to get this merged before next release.

I still think that calling it "no_policy" rather than "forever" in the Realm case would be less confusing and more representative of the nature of this value, but not sure what @timabbott 's opinion is. This could be done while keeping the user-facing text in the org settings the way it is "Retain forever", because from user perspective that's what it is and they don't need to dive into the subtleties of the difference between "no policy" vs "forever".

Hmm, I think I disagree; forever feels better just for consistency with what we use in the per-stream APIs, and that feels more important to being slightly more clear that the "no policy" default state is to retain forever.

So I looked at changing how this PR works to use None for "not retention policy", at the realm level and it's just awkward.  @mateuszmandera what do you think about just changing the convention to use -1 for "forever" at the realm level similar to how it works at the stream level?  That would let us have these two APIs work similarly, and remove the need to try managing None in contexts where it could mean "no value passed".
I think if we made that change in zerver/lib/retention.py /  did a migration, then this PR would be a correct implementation of what we want here.  @mateuszmandera would you be up for trying that out and pushing the extra commit(s) we need to this PR?

Yeah, after making this change I actually agree, this will be a much cleaner approach. #15545

Great, merged, thanks @pragatiagrawal31 and @mateuszmandera!


There is no update notification when running '.\StorageExplorer.exe --add-update-group=CTI' to open Storage Explorer

Storage Explorer Version: 1.34.0 (99)
Build Number: 20240510.2
Branch: rel/1.34.0
Platform/OS: Windows 10/Windows 11/Linux Ubuntu 22.04/MacOS Sonoma 14.4.1(Apple M1 Pro)
Architecture: x64/arm64/x64/arm64
How Found: From running test cases
Regression From: Previous release (1.33.1)
Steps to Reproduce

Close Storage Explorer if already open.
Open a Terminal -> Use cd to navigate to where the Storage Explorer executable is located.
Run '.\StorageExplorer.exe --add-update-group=CTI' to open Storage Explorer.
Check whether there is an update notification asking you to update to 1.1.0.

Expected Experience
There is an update notification.
Actual Experience
There is no update notification.

Verified this issue on the rel build 20240512.1. Fixed.


Anticheese stops flatbed script from working

Describe the issue
Anticheat causing error message when lowering flatbed bed from this script: (https://github.com/TinusNL/tinus_flatbed)
Attempted Solutions
Disable all Anticheese options under: -- these MIGHT trigger in legitimate circumstances, beware before enabling
Screenshots/Logs

Server Infos (please complete the following information):
OS: Windows Server 2019
FXServer Version 5949
EasyAdmin Version 6.91
Additional context
Disabling Anticheese allows the script to work correctly, do you know if it is possible to make the flatbed script work with Anticheese?
Thank you.

8839120d is FIND_NEXT_VEHICLE, do you have server.blockClientEntities enabled? not sure what else would cause this.

I don't have that setting enabled, thanks for coming back to me.
I have removed the flatbed for now if it is not possible to work with anticheese.

i doubt this is an issue even related to anticheese, nothing ac does would block FindNextVehicle as far as i can tell.

It works fine without anticheese, no issues at all. #weird


fix(express): listener leak

Which problem is this PR solving?
Fixes https://github.com/open-telemetry/opentelemetry-js-contrib/issues/258
Short description of the changes
When a middleware is calling the callback synchronous inside the middleware the span was already ended and the added finish listener below was never removed.

Fixed the merge conflicts, anything else required before getting this merged?

Fixed the merge conflicts, anything else required before getting this merged?

@johanneswuerbach I see there are open comments from @obecny. If they are resolved please mark them as resolved. If not, please make the suggested fixes or explain your reasoning not to. We typically do not merge any PRs that have open conversations so we can ensure all concerns are addressed before merging.

@johanneswuerbach I see there are open comments from @obecny. If they are resolved please mark them as resolved. If not, please make the suggested fixes or explain your reasoning not to. We typically do not merge any PRs that have open conversations so we can ensure all concerns are addressed before merging.

@dyladan all four comments are about the extracting the fn used in the loop or keeping it inside, which is being discussed here https://github.com/open-telemetry/opentelemetry-js-contrib/pull/278#discussion_r552698032.
As an outside contributor I'm not really sure whether your agreement to my argument and the one +1 is resolving the issue (and the other 3) or not and as it feels to me like a decision all involved maintainers need to agree upon. As already said I'm happy yo change the code, it just feels wrong, but I'm also not the person who is maintaining the code base so it isn't my call to make :-).

@dyladan all four comments are about the extracting the fn used in the loop or keeping it inside, which is being discussed here https://github.com/open-telemetry/opentelemetry-js-contrib/pull/278#discussion_r552698032.
As an outside contributor I'm not really sure whether your agreement to my argument and the one +1 is resolving the issue (and the other 3) or not and as it feels to me like a decision all involved maintainers need to agree upon. As already said I'm happy yo change the code, it just feels wrong, but I'm also not the person who is maintaining the code base so it isn't my call to make :-).

@dyladan all four comments are about the extracting the fn used in the loop or keeping it inside, which is being discussed here #278 (comment).
As an outside contributor I'm not really sure whether your agreement to my argument and the one +1 is resolving the issue (and the other 3) or not and as it feels to me like a decision all involved maintainers need to agree upon. As already said I'm happy yo change the code, it just feels wrong, but I'm also not the person who is maintaining the code base so it isn't my call to make :-).

eslint has a rule no-loop-func which we don't have yet unfortunately (we should imho), once we enable it this code will need to be changed - someone will have to do it. Declaring a new function in a loop is a bad practice as it creates a new function with each iteration. So this isn't even about "micro-optimizing " this is just basic principle of "good code" which is even supported by the mentioned eslint rule - I'm surprised we don't have it enable by default as it should be recommended default and we would not have this discussion as the build would fail then. Having said I think that fixing this issue should be a 1 minute job as it is pretty straight forward and it will make sure the code quality is fine - even in test :).

@dyladan all four comments are about the extracting the fn used in the loop or keeping it inside, which is being discussed here #278 (comment).
As an outside contributor I'm not really sure whether your agreement to my argument and the one +1 is resolving the issue (and the other 3) or not and as it feels to me like a decision all involved maintainers need to agree upon. As already said I'm happy yo change the code, it just feels wrong, but I'm also not the person who is maintaining the code base so it isn't my call to make :-).

eslint has a rule no-loop-func which we don't have yet unfortunately (we should imho), once we enable it this code will need to be changed - someone will have to do it. Declaring a new function in a loop is a bad practice as it creates a new function with each iteration. So this isn't even about "micro-optimizing " this is just basic principle of "good code" which is even supported by the mentioned eslint rule - I'm surprised we don't have it enable by default as it should be recommended default and we would not have this discussion as the build would fail then. Having said I think that fixing this issue should be a 1 minute job as it is pretty straight forward and it will make sure the code quality is fine - even in test :).

👍  updated.

👍  updated.


Remove ase and dataprovider

Removed dataproviders (not needed since QMLearn)
Removed ASE from requirements.txt
Moved Compound class back to root as in the current pip version of QML.
Updated tests to reflect changes

Noice!


[arte.tv extractor] Subtitles never found

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log
youtube-dl --list-subs https://www.arte.tv/fr/videos/102958-001-A/en-therapie-saison-2-1-35/  -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--list-subs', 'https://www.arte.tv/fr/videos/102958-001-A/en-therapie-saison-2-1-35/', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: 33709297c
[debug] Python version 3.10.2 (CPython) - macOS-12.2.1-arm64-arm-64bit
[debug] exe versions: ffmpeg 4.4.1, ffprobe 4.4.1, phantomjs 83104, rtmpdump 2.4
[debug] Proxy map: {}
[ArteTV] 102958-001-A: Downloading JSON metadata
[ArteTV] 102958-001-A: Downloading m3u8 information
[ArteTV] 102958-001-A: Downloading m3u8 information
[ArteTV] 102958-001-A: Downloading m3u8 information
102958-001-A has no subtitles

Description
The arte.tv extractor never finds subtitles, even though reading through its code, subtitles should be handled. For example for the video link above (https://www.arte.tv/fr/videos/102958-001-A/en-therapie-saison-2-1-35/), reading through the browser logs I can find subtitles for the subtitle file at https://arte-cmafhls.akamaized.net/am/cmaf/102000/102900/102958-001-B/220330204635/medias/102958-001-B_st_VF-MAL.vtt, yet running youtube-dl never finds subtitles.

The extractor doesn't support separate subtitles. Instead, you have to select the formats labelled VOx-STMx, so for FR these ones:
HTTPS_MQ_2     mp4        384x216    VOF-STMF, Français (sourds et malentendants)  300k
HTTPS_HQ_2     mp4        640x360    VOF-STMF, Français (sourds et malentendants)  800k
HTTPS_EQ_2     mp4        720x406    VOF-STMF, Français (sourds et malentendants) 1500k
HTTPS_SQ_2     mp4        1280x720   VOF-STMF, Français (sourds et malentendants) 2200k

The subtitles are built into the video stream, I guess. They were certainly visible in HTTPS_MQ_2 when I viewed.
There doesn't seem to be any reference to subtitle URLs in the site JSON returned. Possibly they're hidden inside a m3u8 manifest. The routine that extracts formats from m3u8 has been enhanced in the yt-dlp fork to extract subtitles as well, but the yt-dlp extractor doesn't yet do that.

There is a newer API endpoint for Arte, which neither youtube-dl nor yt-dlp has been updated to use. This newer endpoint contains links to m3u8 manifests that do contain links to subtitle streams. The newer endpoint also enforces geoblocking, but XFF spoofing bypasses that easily.
Just change /v1 to /v2 in _API_BASE and see. (The data structure is different, obviously.)

Above PR maybe back-ported when practical to resolve this issue.


How to use an actual HTTPclient [no mock up], during testing of flutter app ?

Hi All,
I have created a data replication client. For testing it end to end, I need to create data locally on user mobile and then do replication , and then check it on server if all data has reached there and is intact.
For this I absoluetly need to use HTTPclient to automate my testing.
My test will automitcally check on remote server if it has same data as on my local machine.
If I search internet, all solution I get is to mock HTTP client, but for cases like replication, one need to check if data is same at local mobile and remote server.
How do I use an actual HTTPClient instead of a mocked client ?

Hi @anuragvohraec
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Arguments provided in argument array in custom configuration should not contain shell escaping

This is related to an issue repro'ed in cpptools, here: https://github.com/microsoft/vscode-cpptools/issues/8962
From that issue:

The second issue is a CMake Tools issue (if it's breaking up the args into an array based on parsing a command line from CMake. It would instead be a CMake issue if it's providing it already in the form of an array of args). It's providing cpptools with the following compiler args:

"\"/opt/azurespheresdk/Sysroots/12/tools/sysroots/x86_64-pokysdk-linux/usr/bin/arm-poky-linux-musleabi\"",

"--sysroot=\"/opt/azurespheresdk/Sysroots/12\"",

Because these are being provided in an arg array and not a shell command line, they should not contain shell escaping. The quotes in both of these args are instances of shell escaping. Quotes are used in shell command lines to ensure a space does not get misinterpreted as delimiting multiple arguments. But when args are passed via an API as an array of individual arguments, they should not contain shell escaping. The quotes are instead interpreted literally, as if \" were specified via command line).

I believe this was addressed when switching to v6 of the cpptools-api.  https://github.com/microsoft/vscode-cmake-tools/issues/2508


dep: bump nghttp2 => 1.55.1

Commit Message: dep: bump nghttp2 => 1.55.1
Additional Description: The upstream released notes mention that they have resolved the memory leak, so that patch has now been removed. Fix can be seen here
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] #28417
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Looks like this needs a main merge. cc @yanavlasov
/wait

apologies @RiverPhillips ive landed this already in #28348

No worries @phlax, I will close


Fix: Do not extend BaseTestCase if unit test

This PR

[x] has a unit test extend from PHPUnit\Framework\TestCase instead of OpenCFP\Test\BaseTestCase

Follows https://github.com/opencfp/opencfp/pull/629#issuecomment-346014095.
💁♂ Next up, moving BaseTestCase into the OpenCFP\Test\Integration namespace.

Thank you, @chartjes!


Honor user C(XX)FLAGS

Could users have CFLAGS set to invalid values for zopfli? Is this behaviour normally desired? What do typical linux projects do?
Thank you.

Sure they could set values that have bad effects, but why would they? Anyway it's desired and standard behavior for project build systems to honor user-specified CFLAGS/CXXFLAGS from the environment. In this particular case, users (or distro maintainers) might not be interested in seeing certain types of warnings that the zopfli build emits by default, or might want to use some other optimization level than -O2. All this can obviously already be done, but currently requires modification of Makefile which is unusual and unnecessary.

See #57

@lvandeve One popular thing to do is for some users who compile an application for specifically their system to pass -march=native into the CFLAGS. If the build is to be packaged as a binary for multiple systems of different nature, that flag is undesirable.
Arch Linux has a makepkg.conf option to control custom make flags, but as it states, whether these are respected depends on the build system of the software that is being compiled. Currently, zopfli's makefile does not honour these settings.
Ignoring the question whether using any performance-specific CFLAGS makes a difference, it would be nice of the Makefile to follow the usual practice of allowing this.

Zopfli depends only in the C standard. The main goal of the makefile was so that you don't need to recompile everything during development. The compile command for zopfli is now given in the readme. Is it an option for package managers and platforms to depend on the pure C code rather than on makefiles? C looks like to be more portable than makefiles.

It starts to get very messy when there are several executables, libraries etc to take care of. Better to have a working build system in place. But yeah, instead of managing Makefiles by hand and worrying about portability, I suggest something portable that generates them, such as cmake (or autotools).

But anyway, I still do suggest applying this change, cmake or something like it can be done in the future.

@scop it has already been done but not merged, because @lvandeve is not fond of the idea of having build systems besides GNU Make.

Is it an option for package managers and platforms to depend on the pure C code rather than on makefiles? C looks like to be more portable than makefiles.

No. Not having a build system is ludicrous. If people need to tweak a Makefile for their platform, it's still better than not having a Makefile at all. And if a sophisticated enough build system such as autotools or cmake was brought into the mix, platform independence would be a non-issue.
Have a target for a zopfli binary, have a target for a zopfli library, have a target for the zopflipng binary, have a target for the zopflipng library, have a phony target to build all of them, and finally have an install target which installs headers for the libraries and the libraries into well defined locations so people can #include <zopfli.h> or whatever and create a proper symlink from zopfli.so to the version-numbered sonames.

Looks like this was taken care of in 4a35d304787ffc02551b9bda7213a6a3d50ff87e


CNS Debug API to expose PodIpIdByOrchrestratorContext

feat: This API exposes one of the Httpservice struct field to
display map of OrchestratorContext->PodIpID (UUID) and
includes debug command getPodContexts for CLI use-case.

Looks good to me, left 3 format comments.


timeframe seems to be ignored in the rule file

i have a rule file (a basic frequency rule copied from the example) which has a timeframe of 10 days. When i run test-rule on it, it generates no hits however if I run test-rule with --days 10 it then generates what is expected:
timeframe:
  days: 10

This is an example of where the timeframe seems to be ignored:
D:\app\elastalert> elastalert-test-rule --config config.yaml  rules\frequency.yaml
...
elastalert_status - {'hits': 0, 'matches': 0, '@timestamp': datetime.datetime(2017, 4, 24, 19, 35, 59, 368000, tzinfo=tz
utc()), 'rule_name': 'SampleRule1', 'starttime': datetime.datetime(2017, 4, 23, 19, 35, 58, 290000, tzinfo=tzutc()), 'en
dtime': datetime.datetime(2017, 4, 24, 19, 35, 58, 290000, tzinfo=tzutc()), 'time_taken': 1.0469999313354492}

compare the above to the time I specify the days on the command line:
D:\app\elastalert>elastalert-test-rule --config config.yaml --days 10 rules\frequency.yaml
...
elastalert_status - {'hits': 56932, 'matches': 56932, '@timestamp': datetime.datetime(2017, 4, 24, 19, 42, 37, 761000, t
zinfo=tzutc()), 'rule_name': 'SampleRule1', 'starttime': datetime.datetime(2017, 4, 14, 19, 40, 52, 586000, tzinfo=tzutc
()), 'endtime': datetime.datetime(2017, 4, 24, 19, 40, 52, 586000, tzinfo=tzutc()), 'time_taken':<PHONE_NUMBER>1498718}

The timeframe is not related to the time each query covers. The timeframe is a sliding window of events that grows as time progresses. Elastalert will query at the current time and continue querying every few minutes.
You can query it over the last 10 days of data using the --days flag, or with --start, as you've discovered, in order to test how it would have behaved had it been running that whole time.
With a timeframe of 10 days, you would leave elastalert running and when it hit num_events it would send an alert, if you passed 10 days, the old events would be removed and new ones added. It always tracks 10 days of the most recent events.


tokenizer not working properly with strings containing "..."

currently the tokenizer creates en empty token if a string contains "...". this is probably not intended...
f.eg.:
aTokenizer.tokenizeString("hello example ...") =>{"hello", "example", ""}

Should already be fixed in upcoming patch

the tokenizer also creates tokens 1 character long. Is this fixed as well in the next patch?

Yes, as mentioned in Issue #4. The new tokenizer should eliminate Strings shorter than 2 and should replace all separators within a given String e.g.: "..." or "!!!" etc.
Should be ready in a few hours.

New Tokenizer on master branch


edgetpu-compiler: init at 15.0

Motivation for this change
Adds the Coral Edge TPU model compiler to nixpkgs.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 127909 at 3982f5a9 run on x86_64-linux 1

1 package built successfully:

edgetpu-compiler

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/libraries/science/robotics/edgetpu-compiler/default.nix:43:3:
   |
43 |   installPhase = ''
   |   ^


Remove the step about sourcing istio.VERSION

I pushed a change last night that fixed how a lot of the PRE blocks were indented so they wouldn't render in HTML with blanks on their left and would have the right indent relative to their containing bullet point. I think some of this change ends up undoing some of these fixes.

Andra, I suggest you close this PR and create a new one to just remove the instructions about istio.version


OCS operator install on OCP on ppc64le failed

Prepared a OCP on ppc64le and tried to install OCS on it with this yaml:
apiVersion: operators.coreos.com/v1alpha1
kind: CatalogSource
metadata:
  name: ocs-catalogsource
  namespace: openshift-marketplace
spec:
  displayName: OpenShift Container Storage
  icon:
    base64data: PHN2ZyBpZD0iTGF5ZXJfMSIgZGF0YS1uYW1lPSJMYXllciAxIiB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHZpZXdCb3g9IjAgMCAxOTIgMTQ1Ij48ZGVmcz48c3R5bGU+LmNscy0xe2ZpbGw6I2UwMDt9PC9zdHlsZT48L2RlZnM+PHRpdGxlPlJlZEhhdC1Mb2dvLUhhdC1Db2xvcjwvdGl0bGU+PHBhdGggZD0iTTE1Ny43Nyw2Mi42MWExNCwxNCwwLDAsMSwuMzEsMy40MmMwLDE0Ljg4LTE4LjEsMTcuNDYtMzAuNjEsMTcuNDZDNzguODMsODMuNDksNDIuNTMsNTMuMjYsNDIuNTMsNDRhNi40Myw2LjQzLDAsMCwxLC4yMi0xLjk0bC0zLjY2LDkuMDZhMTguNDUsMTguNDUsMCwwLDAtMS41MSw3LjMzYzAsMTguMTEsNDEsNDUuNDgsODcuNzQsNDUuNDgsMjAuNjksMCwzNi40My03Ljc2LDM2LjQzLTIxLjc3LDAtMS4wOCwwLTEuOTQtMS43My0xMC4xM1oiLz48cGF0aCBjbGFzcz0iY2xzLTEiIGQ9Ik0xMjcuNDcsODMuNDljMTIuNTEsMCwzMC42MS0yLjU4LDMwLjYxLTE3LjQ2YTE0LDE0LDAsMCwwLS4zMS0zLjQybC03LjQ1LTMyLjM2Yy0xLjcyLTcuMTItMy4yMy0xMC4zNS0xNS43My0xNi42QzEyNC44OSw4LjY5LDEwMy43Ni41LDk3LjUxLjUsOTEuNjkuNSw5MCw4LDgzLjA2LDhjLTYuNjgsMC0xMS42NC01LjYtMTcuODktNS42LTYsMC05LjkxLDQuMDktMTIuOTMsMTIuNSwwLDAtOC40MSwyMy43Mi05LjQ5LDI3LjE2QTYuNDMsNi40MywwLDAsMCw0Mi41Myw0NGMwLDkuMjIsMzYuMywzOS40NSw4NC45NCwzOS40NU0xNjAsNzIuMDdjMS43Myw4LjE5LDEuNzMsOS4wNSwxLjczLDEwLjEzLDAsMTQtMTUuNzQsMjEuNzctMzYuNDMsMjEuNzdDNzguNTQsMTA0LDM3LjU4LDc2LjYsMzcuNTgsNTguNDlhMTguNDUsMTguNDUsMCwwLDEsMS41MS03LjMzQzIyLjI3LDUyLC41LDU1LC41LDc0LjIyYzAsMzEuNDgsNzQuNTksNzAuMjgsMTMzLjY1LDcwLjI4LDQ1LjI4LDAsNTYuNy0yMC40OCw1Ni43LTM2LjY1LDAtMTIuNzItMTEtMjcuMTYtMzAuODMtMzUuNzgiLz48L3N2Zz4=
    mediatype: image/svg+xml
  image: quay.io/rhceph-dev/ocs-olm-operator:latest-4.6
  publisher: Red Hat
  sourceType: grpc
---
apiVersion: operators.coreos.com/v1alpha1
kind: Subscription
metadata:
  name: ocs-subscription
  namespace: openshift-storage
spec:
  channel: "stable-4.6"
  config:
    resources: {}
  name: ocs-operator
  source: ocs-catalogsource
  sourceNamespace: openshift-marketplace

Tried the same last week and csv got installed as 4.6.0-531.ci and everything was fine. Today, csv got intalled as 4.6.0-532.ci and the following three pods are not coming up in openshift-storage namespace:
rook-ceph-crashcollector-worker-0-556778b7c4-sxd85   0/1     Init:0/2   0          6h23m
rook-ceph-crashcollector-worker-1-5dcf994fc9-8mmh4   0/1     Init:0/2   0          6h46m
rook-ceph-crashcollector-worker-2-76dfddf584-4q74d   0/1     Init:0/2   0          6h44m
saw these in the oc describe of these pod:
Events:
  Type     Reason       Age                      From               Message
  ----     ------       ----                     ----               -------
  Warning  FailedMount  175m (x18 over 6h15m)    kubelet, worker-0  Unable to attach or mount volumes: unmounted volumes=[rook-ceph-crash-collector-keyring], unattached volumes=[rook-ceph-crash-collector-keyring rook-config-override rook-ceph-log rook-ceph-crash default-token-r6x4s]: timed out waiting for the condition
  Warning  FailedMount  46m (x24 over 6h6m)      kubelet, worker-0  Unable to attach or mount volumes: unmounted volumes=[rook-ceph-crash-collector-keyring], unattached volumes=[default-token-r6x4s rook-ceph-crash-collector-keyring rook-config-override rook-ceph-log rook-ceph-crash]: timed out waiting for the condition
  Warning  FailedMount  26m (x51 over 6h13m)     kubelet, worker-0  Unable to attach or mount volumes: unmounted volumes=[rook-ceph-crash-collector-keyring], unattached volumes=[rook-config-override rook-ceph-log rook-ceph-crash default-token-r6x4s rook-ceph-crash-collector-keyring]: timed out waiting for the condition
  Warning  FailedMount  21m (x28 over 5h57m)     kubelet, worker-0  Unable to attach or mount volumes: unmounted volumes=[rook-ceph-crash-collector-keyring], unattached volumes=[rook-ceph-log rook-ceph-crash default-token-r6x4s rook-ceph-crash-collector-keyring rook-config-override]: timed out waiting for the condition
  Warning  FailedMount  6m57s (x190 over 6h17m)  kubelet, worker-0  MountVolume.SetUp failed for volume "rook-ceph-crash-collector-keyring" : secret "rook-ceph-crash-collector-keyring" not found
  Warning  FailedMount  76s (x34 over 5h54m)     kubelet, worker-0  Unable to attach or mount volumes: unmounted volumes=[rook-ceph-crash-collector-keyring], unattached volumes=[rook-ceph-crash default-token-r6x4s rook-ceph-crash-collector-keyring rook-config-override rook-ceph-log]: timed out waiting for the condition

One of the warning above is due to missing rook-ceph-crash-collector-keyring secret and it is missing. In the previous install with 4.6.0-531.ci, the secret is found but not now with this 4.6.0-532.ci
May be a regression between 531 and 532.

Do not open issues against downstream builds on GitHub.


eni: Don't restore the connmark when Cilium is carrying the identity

Please see the commit message for more details. This fixes this issue (https://github.com/cilium/cilium/issues/20797), but only for the setup which is using Cilium with ENI mode and "not" using AWS VPC CNI (e.g. self-managed k8s on EC2).
Partially fix the issue that ENI changes the security identity carried with mark when Cilium uses ClusterID 128-255 for the setup uses ENI mode without AWS VPC CNI.

Don't merge this. The second iptables rule bypasses KubeProxy's LB logic. Needs fix.

@YutaroHayakawa Thank you for addressing the issue.
For the second part of the issue, the poststart-eni.bash script already does similar cleanup. https://github.com/cilium/cilium/blob/d6483e1d26052b7f210a746ff85e919b76c9430f/install/kubernetes/cilium/files/nodeinit/poststart-eni.bash#L7-L15
Can we extend that to clean up for this case as well ?

@hemanthmalla Yeah that could be a good idea. I did some research around the second rule we were unaware of in the past, and it looks like it is used for the "external SNAT" that uses the external VPC NAT Gateway to go outside of the VPC instead of SNAT'ing on the node. This is enabled by default in the eksctl setup and if we let AWS VPC CNI to run before Cilium, the problematic iptables rule will be inserted.
https://github.com/aws/amazon-vpc-cni-k8s/blob/master/pkg/networkutils/network.go#L607
However, in your environment, you don't have any moment that installs the AWS VPN CNI, right? That means just merging the first fix resolves the problem at least in your environment IIUC.
-A CILIUM_PRE_mangle -i lxc+ -m comment --comment "cilium: primary ENI" -m mark ! --mark 0x0F00/0x0F00 -j CONNMARK --restore-mark --nfmask 0x80 --ctmask 0x80

I'm still not sure how we can resolve the second problem. So, I think we can separate the problem and resolve the first issue in this PR so that the problem resolves, at least in your environment. How do you think?

@YutaroHayakawa Yes, it makes sense to split up the PR. That would definitely fix the bug for using ENI mode outside EKS clusters. Thank you.

Let me close this PR and open another one.


.: adding travis file for future CI

https://github.com/opencontainers/specs/issues/90
Signed-off-by: Vincent Batts<EMAIL_ADDRESS>

LGTM

LGTM


fix: git-absorb getting stuck

This fixes neogit hanging forever when using git absorb as reported in the issue #1564
Closes #1564

Awesome - why does this work? 😅

@CKolkey from what I gathered, the --and-rebase option in git-absorb does start interactive rebase, which in turn starts default git editor (confirmed by looking at the list of processes when this gets stuck). Unfortunately neogit does not display that editor for me. In the case of git-absorb I don't want to edit the sequence anyway, so I think it's okay to just allow it to run non-interactively by setting GIT_SEQUENCE_EDITOR env variable, just as neogit does for rebase.
full disclosure - I didn't really have a lot of time to read through the neogit's source code, so if you have any suggestions how to solve this better I'd be very grateful

Ah, ok, makes perfect sense then. Didn't think about why it was stuck.. waiting on an editor.
Thanks for having a look at this :)


Standalone install

Add scripts/guides on how to install Spoolman without using Docker.

I would really like to have this direct installation (for e.g. CT/LXC in Proxmox) without Docker as another layer in between...


add perms to use flex postgres on div projects

Already there - https://github.com/hmcts/cnp-jenkins-config/blob/master/terraform-infra-approvals/global.json#L55

Already there - https://github.com/hmcts/cnp-jenkins-config/blob/master/terraform-infra-approvals/global.json#L55

We're having issues trying to get v14 onto our demo env. We are getting errors with the whitelist as seen here: https://build.platform.hmcts.net/blue/organizations/jenkins/HMCTS_d_to_i%2Fdiv-case-orchestration-service/detail/demo/53/pipeline/
Can you please advise?


[QUERY] [Search] Missing NativeBlobSoftDeleteDeletionDetectionPolicy after update

Library name and version
Azure.Search.Documents v.11.6.0
Query/Question
After updating the package Azure.Search.Documents from version 11.6.0-beta4 I can't find the class NativeBlobSoftDeleteDeletionDetectionPolicy anymore

Which class can be used to achieve the same behavior?
Environment
No response

Currently, the search service supports the NativeBlobSoftDeleteDeletionDetectionPolicy type only in preview releases. This type is not included in the stable release because it is part of a preview feature supported only in the preview version of the Azure Search service. A stable client library cannot rely on a preview version of the service. If the feature becomes stable and is supported in the Search service, the client library will include support for it.
This feature request is specifically for the service team. The SDK can only incorporate features that are supported by the service. If you have any support requests or questions related to the search service, please raise an issue at Azure Search · Community. For any SDK-related questions, feel free to create issues on the Azure SDK for .NET GitHub.


Tween created in _ready does not work if used later

Godot version
4.0.dev 6e87d62
System information
Windows 10, NVidia, Vulkan
Issue description
I have an autoload manager which creates a Tween and will use it to fade the screen in and out.
This code worked in Godot 3, but after making a best effort attempt to port the Tween code to the new API in Godot 4, it seems to give confusing errors about Tweeners and fail unless I recreate the object every time. I have also triple-checked and Autoloaded class are definitely in the scene tree. It shows the same error if I use with get_tree() or without.
Here is an excerpt of the output if I attempt to run the attached project. The "is_valid" and "fade_manager" prints are mine. The others come from the C++ Tween class itself.
Created a new Tween: is_valid=False is_running=True
ERROR: Tween started, but has no Tweeners.
   at: (scene\animation\tween.cpp:221)
Tween: is_valid=True is_running=True
ERROR: Tween was created outside the scene tree, can't use Tweeners.
   at: (scene\animation\tween.cpp:56)
ERROR: fade_manager.gd: Failed to create a tween!
   at: VariantUtilityFunctions::push_error (core\variant\variant_utility.cpp:607)
ERROR: Tween invalid, can't play.
   at: (scene\animation\tween.cpp:120)

A random guess is maybe tweens are garbage collected after some delay, but I can't find any mention of this in the updated docs in doc/classes/Tween.xml. It's possible I misread it.
The thing is if I start the fade immediately inside of _ready, it actually seems to work if I do it there, so there's something about timing.
Steps to reproduce

Play the attached project.
Wait a second for the simulated 1 second loading time is finished.
It should fade to black, but prints an error and hits fallback code to instantly set the screen to black.
Edit fade_manager.gd and uncomment the call to "_recreate_tween" on line 29
Try again. The Tween will now work.

Minimal reproduction project
TweenAutoload.zip

The stop bug you mentioned (it was actually a bug) was fixed in #57353
The is_valid() bug was fixed in #51237
I still don't know why would you want to re-use Tweens, but it's now fully possible. You just need to stop them right after creating and right after finishing.

The stop bug you mentioned (it was actually a bug) was fixed in #57353 The is_valid() bug was fixed in #51237
I still don't know why would you want to re-use Tweens, but it's now fully possible. You just need to stop them right after creating and right after finishing.

Consider this:

var tween: Tween = null

func jump_to(target : Vector2):
	if tween != null and tween.is_valid():
		return

	tween = create_tween()
	# do tween stuff

vs.

@onready var tween: Tween = create_tween()

func jump_to(target : Vector2):
	tween.stop()
	# do tween stuff

I'm settling with:

@onready var tween: Tween = create_tween()

func _ready() -> void:
	# have to stop it manually after creation otherwise Godot complains
	tween.kill()

func jump_to(target : Vector2):
	tween.kill()
	tween = create_tween()
	# do tween stuff

Even though I think 2. would be the developer experience.


fix(userspace/libscap): fix memleak in scap_linux_get_fdlist

What type of PR is this?
/kind bug

Any specific area of the project related to this PR?
/area libscap

Does this PR require a change in the driver versions?
No

What this PR does / why we need it:
This PR fixes a memory leak problem.
I built sinsp-example from master (2d118d79e59576b2f1341798a56032b0bdf3b9cb) and tested it by following command line:
valgrind --leak-check=yes ./sinsp-example -b probe.o -f "evt.type = execve and evt.dir = < and container.id != host and proc.name = bash"

The output of valgrind indicated that there were some memory leaked:
==428618==
==428618== 346,048 (1,120 direct, 344,928 indirect) bytes in 1 blocks are definitely lost in loss record 175 of 177
==428618==    at 0x4C3BE4B: calloc (vg_replace_malloc.c:1328)
==428618==    by 0x6A61E0: scap_fd_read_netlink_sockets_from_proc_fs (scap_fds.c:598)
==428618==    by 0x6A7725: scap_fd_read_sockets (scap_fds.c:1176)
==428618==    by 0x6A7DF8: scap_fd_handle_socket (scap_fds.c:396)
==428618==    by 0x6A8462: scap_fd_scan_fd_dir (scap_fds.c:1352)
==428618==    by 0x6A4C74: scap_linux_get_fdlist (scap_procs.c:1391)
==428618==    by 0x58736F: parse_rw_exit (parsers.cpp:4949)
==428618==    by 0x58736F: sinsp_parser::parse_rw_exit(sinsp_evt*) (parsers.cpp:4780)
==428618==    by 0x58529A: sinsp_parser::process_event(sinsp_evt*) (parsers.cpp:306)
==428618==    by 0x4A6E1F: sinsp::next(sinsp_evt**) (sinsp.cpp:1469)
==428618==    by 0x47C68F: get_event(sinsp&, std::function<void (std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)>) (test.cpp:457)
==428618==    by 0x46B0F1: main (test.cpp:425)
==428618==
==428618== 1,032,448 (3,360 direct, 1,029,088 indirect) bytes in 3 blocks are definitely lost in loss record 176 of 177
==428618==    at 0x4C37135: malloc (vg_replace_malloc.c:381)
==428618==    by 0x6A5BAB: scap_fd_read_unix_sockets_from_proc_fs (scap_fds.c:465)
==428618==    by 0x6A76EB: scap_fd_read_sockets (scap_fds.c:1168)
==428618==    by 0x6A7DF8: scap_fd_handle_socket (scap_fds.c:396)
==428618==    by 0x6A8462: scap_fd_scan_fd_dir (scap_fds.c:1352)
==428618==    by 0x6A4C74: scap_linux_get_fdlist (scap_procs.c:1391)
==428618==    by 0x58736F: parse_rw_exit (parsers.cpp:4949)
==428618==    by 0x58736F: sinsp_parser::parse_rw_exit(sinsp_evt*) (parsers.cpp:4780)
==428618==    by 0x58529A: sinsp_parser::process_event(sinsp_evt*) (parsers.cpp:306)
==428618==    by 0x4A6E1F: sinsp::next(sinsp_evt**) (sinsp.cpp:1469)
==428618==    by 0x47C68F: get_event(sinsp&, std::function<void (std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)>) (test.cpp:457)
==428618==    by 0x46B0F1: main (test.cpp:425)
==428618==
==428618== LEAK SUMMARY:
==428618==    definitely lost: 10,568 bytes in 52 blocks
==428618==    indirectly lost: 1,724,672 bytes in 1,672 blocks
==428618==      possibly lost: 350,720 bytes in 316 blocks
==428618==    still reachable: 52,402 bytes in 617 blocks
==428618==         suppressed: 0 bytes in 0 blocks
==428618== Reachable blocks (those to which a pointer was found) are not shown.
==428618== To see them, rerun with: --leak-check=full --show-leak-kinds=all
==428618==
==428618== Use --track-origins=yes to see where uninitialised values come from
==428618== For lists of detected and suppressed errors, rerun with: -s
==428618== ERROR SUMMARY: 3145 errors from 100 contexts (suppressed: 0 from 0)

By reading releated code thoroughly, I believed the function scap_linux_get_fdlist has to call scap_fd_free_ns_sockets_list to free its local variable sockets_by_ns, just like what scap_proc_read_thread does.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
No.

NONE

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

223f59c fix(userspace/libscap): fix memleak in scap_linux_get_fdlist

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ZhiyuanXin
Once this PR has been reviewed and has the lgtm label, please assign incertum for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @ZhiyuanXin! It looks like this is your first PR to falcosecurity/libs 🎉

Great catch!
/cc @loresuso

/milestone 0.14.0

LGTM label has been added.  Git tree hash: 480fb63a27d885407b8d0e5be6c2f9db013620bb

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, loresuso, ZhiyuanXin
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


1.17???

Is this mod also 1.17????
I hope so 😇😇

Yeah it is already

yes it's on curseforge. Can also be downloaded from here https://github.com/qouteall/ImmersivePortalsMod/releases


Replace logo image if needed it

Aiven will support and celebrate pride month for the next 11 months as you can fin here: https://aiven.io/blog/celebrating-our-colorful-cast
The images were replaced in this PR: https://github.com/aiven/devportal/pull/909
This issue should be revisited in a year (June 2023) to check if the images should be replaced back or not.

The logo for the tab will not be changed for the Aiven websites including devprotal

Want to mention here, that we might also need to change the site-preview.png file once we go back to the original image.
I'm now merging https://github.com/aiven/devportal/pull/922 with the updated logo for preview


MSU Information Missing? (Track name, etc)

This 100% could be user error, so please feel free to tell me if this is the case.
I am trying to get the "Current Song" area to work, and though it does show the MSU based on the folder name, it almost always only shows
"MSU:
Song: Track 3"
or on rare occasions, it'll mention an artist.  However, I thought it was supposed to pull the full track name.  Is there something I need to configure, a way the msu packs needed to have been generated, or is there something I'm just missing in general?

For it to be able to show the track information, it has to be able to get the track and artist information from somewhere (it's not included in the MSU or PCM files themselves). This means it needs to have that information accessible either in a tracks.json file or YAML file with the MSU information. Unfortunately a lot of older MSUs don't have those provided (and even many newer ones won't depending on the community), so it can't get that info.
You can create your own YAML files to go alongside the MSU with all that information if you want, but obviously you'd need to know about all the songs in the MSU.

I guess to clarify though, does it sometimes show up blank for the actual MSU name? Or was that just more and example of the generic track number info?

That was just an example.
But thanks for the info on the JSON.  Do you have a sample of the proper format the Json has to be in for the tracker to work?  I can then go ahead and try to manually fill out the ones that are missing.
Though on that note, I do have the JSON file for some of them, in the msu pack directory named the same base as the rest of the MSU, but it is still seeming to not pick it up?  (though that might explain why it was getting the artist name for some of them).  I assume I'm missing a property in the tracks array (since they tend to have the following:
{
"track_number": 2,
"title": "Light World Overworld",
"file": "Tracks\A Hat in Time OST - 58 Alpine Skyline.mp3",
"trim_end": 28809855,
"loop": 14405483
},
But even then, it shows just as "Track 2"


Is it necessary to explicitly set incrementing to false...?

The docs suggest that "[t]o get your model to work with the encoded UUID (i.e. to use uuid as a primary key), you must let your model use the Spatie\BinaryUuid\HasBinaryUuid" but "[i]f [you] don't like the primary key named uuid you can leave off the HasUuidPrimaryKey trait and manually specify $primaryKey. Don't forget set $incrementing to false."
However, Spatie\BinaryUuid\HasBinaryUuid has the following method:
    public function getIncrementing()
    {
        return false;
    }

So...shouldn't it be enough to just use Spatie\BinaryUuid\HasBinaryUuid and not explicitly set $incrementing to false...?

Yeah it is... Sorry about the mix up with respect to the dos

@isocroft feel free to also improve this in your README PR 👍


[SPARK-20197][SPARKR][BRANCH-2.1] CRAN check fail with package installation

What changes were proposed in this pull request?
Test failed because SPARK_HOME is not set before Spark is installed.
Also current directory is not == SPARK_HOME when tests are run with R CMD check, unlike in Jenkins, so disable that test for now. (that would also disable the test in Jenkins - so this change should not be ported to master as-is.)
How was this patch tested?
Manual run R CMD check

Test build #75468 has started for PR 17515 at commit 5d8897e.

Test build #75468 has finished for PR 17515 at commit 5d8897e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75468/
Test PASSed.

AppVeyor doesn't run for branch-*, so the red mark is expected.
This should be good to go to branch-2.1.

LGTM. I think this fix is fine for 2.1 branch (though its a shame as this was one of the main changes we were trying to get in 2.1 ?)

while we are skipping that one test here, the test actual will pass when run - it's just that when running as R CMD check, SPARK_HOME = ~/.cache/spark/spark* whereas spark-warehouse and metastore_db are actually created under R/inst/tests/testthat (where the actual test .R files are)
I suspect that is by design for R CMD check to change the wd that way and ultimately we should consider changing Jenkins to always build the R source package and then run R CMD check --as-cran for the "real test".

merged to branch-2.1


Node integration

Kind of a big PR.
Introduces node integration to subspace desktop. Turning it into a standalone app.
Also solves #79, #48 and #61.
Question: should we bump the version into 0.2.0 with this PR (I can do a quick rebase), or should that be another PR?
I don't know if there is a best practice for that.

We actually discussed that in the meeting. We will be making the switch pretty soon, that's why we decided to make this change and be ready for it. But I'm fine with both strategies :)
I believe @1devNdogs and @ImmaZoni are more knowledgeable on this issue than me

Well, it will be broken before the switch though, meaning we can't test the final builds. It also means that when we do the switch next time, we'll break the previously produced builds instead of giving them some more time to switch. I think setting more specific endpoint is actually beneficial because of that.

Well, it will be broken before the switch though, meaning we can't test the final builds.

Oh, I hadn't considered that. I have updated values to target the inactive network on aries-farm-rpc-b

It would be nice to add advanced settings in the future where we can customize bootstrap nodes and other settings.

So I guess we will wait for the switch to happen, then update the version and bootnodes in the code, and have a release.
In the meantime, we can merge this PR.
I have squashed the last tweak commit, and enabled auto-merge.


Cleaned up the syntax page

The syntax page now uses the new panels stuff (the content is still loaded server side, just reformatted locally)
Criteria (e.g. x:... for text) are ordered alphabetically based on the property they sort
The panel search bar now ignores html tags

Also a note for prosperity: I accidentally pushed a slightly earlier version of this direct to main instead of my own fork, so that's why there's some similar commits before this (now reverted)

Looks like the line endings changed. Maybe we should change the .gitattributes to enforce consistency.

Yeah, once the line endings are fixed, this looks great.

I double checked and it seems the file originally had the wrong line endings and my new settings fixed it.


feat: add bias and drive mode options to switches

Hello,
Yesterday I struggled big time to power a relay from a raspi 4. Until I figured via the CLI that I needed no bias, and open drain drive mode.
Those options werent present so, here we are !
Let me know if you need anything else. It's barely tested right now, gonna deploy it today and get back here tomorrow with feedback !

nice! please share test results. Any idea why these settings are required? Will look into the settings, but will be somewhere this weekend. Switching a relay differs in switching a led? using RPI4 myself, but only tested with leds, understanding the difference might improve the update, dunno

added to main, please test.


Tokens::ensureWhitespaceAtIndex - clean up comment check, add check for T_OPEN

The current logic assumes comment tokens can have a trailing line break, however the transformers make sure this is never the case anymore.
Second change is that an T_OPEN tag can still have a trailing line break (i.e. not transformed to own whitespace token).
I theory this is also true for close tags, but inserting an white space token after a close tag is not valid in the first place.

Thank you @SpacePossum.


⚠ SceneTime has degraded performance

In c005e07, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1356 ms

Resolved: SceneTime performance has improved in 5b189ee after 6 minutes.


HACS Information link points to wrong integration.

A small matter: HACS Information selection points to 'Welland Canal Bridge Status Integration'

Thanks for highlighting. Fixed.


updated changelog to reflect release 0.56.1

@seut please

Current coverage is 80.96% (diff: 100%)

Merging #4223 into master will increase coverage by <.01%

@@             master      #4223   diff @@
==========================================
  Files          1300       1300
  Lines         44236      44236
  Methods           0          0
  Messages          0          0
  Branches       7057       7057
==========================================
+ Hits          35813      35815     +2
+ Misses         5771       5770     -1
+ Partials       2652       2651     -1

Powered by Codecov. Last update e37b278...4364153


feat: Replace combobox command menu with cmdk-solid

This PR replaces the usage of Kobalte's combobox with cmdk-solid for the command menu component

Thanks for the work!


Probe account type via userAccountControl instead of prefiltering wit…

…h sAMAccountType

After testing code needs to be replicated to other Tomcat modules as well.


Lowercasing not working in 1.4.4

The ABL: Convert to lowercase command does not do anything in v1.4.4.
Select a program (cls or p) with some uppercase code. CTRL-P and select the convert to lowercase command. Nothing happens. This worked in 1.4.3.
The following error is seen in the ABL Language Server log.
Jan 12, 2023 2:31:00 PM org.eclipse.lsp4j.jsonrpc.RemoteEndpoint fallbackResponseError
SEVERE: Internal error: argument type mismatch
java.lang.IllegalArgumentException: argument type mismatch
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
	at org.eclipse.lsp4j.jsonrpc.services.GenericEndpoint.lambda$null$0(GenericEndpoint.java:65)
	at org.eclipse.lsp4j.jsonrpc.services.GenericEndpoint.request(GenericEndpoint.java:120)
	at org.eclipse.lsp4j.jsonrpc.RemoteEndpoint.handleRequest(RemoteEndpoint.java:261)
	at org.eclipse.lsp4j.jsonrpc.RemoteEndpoint.consume(RemoteEndpoint.java:190)
	at org.eclipse.lsp4j.jsonrpc.json.StreamMessageProducer.handleMessage(StreamMessageProducer.java:194)
	at org.eclipse.lsp4j.jsonrpc.json.StreamMessageProducer.listen(StreamMessageProducer.java:94)
	at org.eclipse.lsp4j.jsonrpc.json.ConcurrentMessageProcessor.run(ConcurrentMessageProcessor.java:113)
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
	at java.base/java.lang.Thread.run(Thread.java:833)

What does the first line of the language server output say ? It should be the version number.

[2023-01-12T16:39:58.561-0500] [INFO] LanguageServerInfo: 1.5.0-SNAPSHOT-391f7c0 -- ClientInfo: Visual Studio Code 1.74.3
[2023-01-12T16:39:58.561-0500] [DEBUG] Client Capabilities: WS Config true -- WS Folder true -- Diagnotics true -- Symbol Hierarchy true -- Document Highlight true

Thx, will check that tomorrow.

Confirmed, all commands going to the language server seem to have a problem. I'm having a look.

And I thought breaking the lowercasing was a step towards a code beautifier ;-)

Fixed, sorry for the regression.
Version 1.4.5 is currently being uploaded to the marketplace (and open-vsx.org).

Thanks for the fast turnaround. Looks to be working now.
I am seeing a different error now, on save of a file.
[2023-01-13T08:24:58.814-0500] [INFO] Command line arguments:
[2023-01-13T08:24:59.728-0500] [INFO] LanguageServerInfo: 1.5.0-SNAPSHOT-a7a171e -- ClientInfo: Visual Studio Code 1.74.3
[2023-01-13T08:24:59.728-0500] [DEBUG] Client Capabilities: WS Config true -- WS Folder true -- Diagnotics true -- Symbol Hierarchy true -- Document Highlight true
[2023-01-13T08:24:59.736-0500] [DEBUG] Found CABL License
[2023-01-13T08:24:59.745-0500] [INFO] Register CABL License for 'Consultingwerk' - Valid until 2022-12-31T00:00:00

<snip>

[2023-01-13T08:26:33.086-0500] [DEBUG] Lint rules execution for Consultingwerk\Web2\WebHandler\FileUploadWebHandler.cls
[2023-01-13T08:26:33.107-0500] [ERROR] Rule exception:
org.sonar.plugins.openedge.api.InvalidLicenseException: Invalid license - Signature OK - Expiration KO
	at org.sonar.plugins.openedge.AbstractCommercialRule.setContext(SourceFile:116)
	at eu.rssw.openedge.ls.a.k.a(SourceFile:61)
	at eu.rssw.openedge.ls.a.k.apply(SourceFile:37)
	at java.base/java.util.concurrent.CompletableFuture.uniApplyNow(CompletableFuture.java:684)
	at java.base/java.util.concurrent.CompletableFuture.uniApplyStage(CompletableFuture.java:662)
	at java.base/java.util.concurrent.CompletableFuture.thenApply(CompletableFuture.java:2168)
	at eu.rssw.openedge.ls.a.u.run(SourceFile:141)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
	at java.base/java.lang.Thread.run(Thread.java:833)

org.sonar.plugins.openedge.api.InvalidLicenseException: Invalid license - Signature OK - Expiration KO
Did you buy your license ? :-)

Did you buy your license ? :-)

[2023-01-13T08:24:59.745-0500] [INFO] Register CABL License for 'Consultingwerk' - Valid until 2022-12-31T00:00:00

Apparently my brain thinks that it's stil 2022. My bad, sorry.


Windows users may hit MAX_PATH problems when installing Managed Cloud SDK

A user on StackOverflow seems to be hitting an installation problem of the Managed Cloud SDK on Windows:
#============================================================#
#= Installing: gcloud app Java Extensions                   =#
#============================================================#
ERROR: gcloud crashed (IOError): [Errno 2] No such file or directory:  u'C:\\Users\\reddingBobulus\\AppData\\Local\\google-cloud-tools-java\\managed-cloud-sdk\\LATEST\\google-cloud-sdk.staging\\platform\\google_appengine\\google\\appengine\\tools\\java\\jetty93\\jetty-distribution\\lib\\apache-jsp\\org.eclipse.jetty.apache-jsp-9.3.18.v20170406-nolog.jar'

That path is 265 characters (including the terminating nul), which exceeds Windows' maximum path length MAX_PATH (260).  This maximum path length can be removed in Windows 10, but is opt-in (!) by setting a registry key.
That same MSDN article notes that there are many functions that have Unicode versions that permit an extended path length of 32767 characters.  Such extended paths can be specified with a \\?\ prefix (e.g., \\?\C:\Users\Foo\Bar).  We should investigate whether this extended path prefix works.

Gosh. Given the path, looks like this will happen quite frequently.

We should probably check for this in the managed cloud SDK installer and throw a relevant exception if it's a problem. What's the best way on Windows to check for the maximum path length? Is there anything more reasonable than creating a temporary file with a long path and seeing if it succeeds?

I can confirm that I can create a file with path length > 260 from cmd.exe using the \\?\ prefix.

I fear that gcloud may be the one to test and use \\?\.
I like the idea to test creating a temp file with a long name and if it fails, display explanation and instructions to enable long path names.

I suppose there's the same possibility that someone has put an eclipse.exe.manifest to enable long-path names for Eclipse

I think the manifest approach (i.e., placing python.exe.manifest next to python.exe) won't work. It requires modifying the Windows registry to prefer the external manifest file: https://appuals.com/disable-display-scaling-on-high-dpi-devices-in-windows-10/
My GCE Windows did not set the PreferExternalManifest registry key by default. I verified the external manifest file is picked up only after defining the key.

Does that suggest that the python.exe has an embedded manifest?

#605 will reduce the path length.
After the fix rolls out, we could eventually clean up SDKs in the deprecated locations (if any), after sufficient lead time. A working implementation for such cleanup is at https://github.com/GoogleCloudPlatform/appengine-plugins-core/compare/i595-reduce-path-len?expand=1.

close as not planned


Remove / hide printf messages in Simple MCC library

Move printf to user app or put them under debug flag

This should be now resolved. If not - please reopen.


Async Broker Routing

Add QueryRouter class to route the query with single connection between the broker and each server (count OFFLINE and REALTIME as different servers)
Use single connection with multiplexing to gather responses from servers
Add SingleConnectionBrokerRequestHandler which works on QueryRouter and handle request in async manner
This PR is not complete yet, will add more tests and benchmark

Codecov Report

Merging #3143 into master will increase coverage by 0.05%.
The diff coverage is 79.51%.

@@             Coverage Diff              @@
##             master    #3143      +/-   ##
============================================
+ Coverage     70.15%   70.21%   +0.05%
  Complexity        4        4
============================================
  Files           934      941       +7
  Lines         43477    43722     +245
  Branches       5894     5916      +22
============================================
+ Hits          30503    30698     +195
- Misses        10937    10972      +35
- Partials       2037     2052      +15

Impacted Files
Coverage Δ
Complexity Δ

...com/linkedin/pinot/common/metrics/BrokerMeter.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

...va/com/linkedin/pinot/common/utils/ServerType.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

...kedin/pinot/broker/broker/BrokerServerBuilder.java
91.52% <100%> (+0.78%)
0 <0> (ø)
:arrow_down:

.../pinot/core/common/datatable/DataTableFactory.java
42.85% <100%> (ø)
0 <0> (ø)
:arrow_down:

...java/com/linkedin/pinot/core/transport/Server.java
57.57% <57.57%> (ø)
0 <0> (?)

.../linkedin/pinot/core/transport/ResponseFuture.java
75% <75%> (ø)
0 <0> (?)

...core/transport/netty/handler/DataTableHandler.java
75% <75%> (ø)
0 <0> (?)

...com/linkedin/pinot/core/transport/QueryRouter.java
77.96% <77.96%> (ø)
0 <0> (?)

.../linkedin/pinot/core/transport/ServerResponse.java
80.95% <80.95%> (ø)
0 <0> (?)

...thandler/SingleConnectionBrokerRequestHandler.java
88.37% <88.37%> (ø)
0 <0> (?)

... and 29 more

Continue to review full report at Codecov.

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


bug: Suspected to be incompatible with viem 2.0.0, error TypeError: Cannot read properties of undefined (reading 'http')

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
1.4.13
Current Behavior
Configure WagmiProvider configureChains
Expected Behavior
No response
Steps To Reproduce
`"use client";
import { WagmiConfig, configureChains, createConfig } from "wagmi";
import { mainnet, localhost } from "wagmi/chains";
import { publicProvider } from "wagmi/providers/public";
import { CoinbaseWalletConnector } from "wagmi/connectors/coinbaseWallet";
import { InjectedConnector } from "wagmi/connectors/injected";
import { MetaMaskConnector } from "wagmi/connectors/metaMask";
import { WalletConnectConnector } from "wagmi/connectors/walletConnect";
const projectId = "XXXXXXXXXXXXXXXXXXXXX";
const { chains, publicClient, webSocketPublicClient } = configureChains(
[mainnet, localhost],
[publicProvider()]
);
const config = createConfig({
autoConnect: false,
connectors: [
new MetaMaskConnector({ chains }),
new CoinbaseWalletConnector({
chains,
options: {
appName: "Starfab.io",
},
}),
new WalletConnectConnector({
chains,
options: {
projectId: "XXXXXXXXXXXXXXXXXXXXX",
},
}),
new InjectedConnector({
chains,
options: {
name: "Browser wallet",
shimDisconnect: true,
},
}),
],
publicClient,
webSocketPublicClient,
});
export function WagmiProvider({ children }: { children: React.ReactNode }) {
return {children};
}
`

Link to Minimal Reproducible Example (StackBlitz, CodeSandbox, GitHub repo etc.)
No response
Anything else?
It works fine with the old version viem 1.21.4.

Hi, viem v2 is still in beta, when its out of beta, I am sure that wagmi will release new version with its support.

just use the <2 version until Wagmi v2 is stable 😄

Hi, viem v2 is still in beta, when its out of beta, I am sure that wagmi will release new version with its support.

@imaksp viem v2 is now stable as of yesterday! And wagmi v2 stable seems to be releasing this week.

Correct, Viem v2 and Wagmi v2 work together.


Allow showing optional message with the qr code helper in chatflows

qrcode_show and qrcode_show_data don't allow that, the goal is we can provide (copy/paste) usage for farmer addresses for payments
to allow:

title ( default ) scan with your application
qr code image
text (default empty), but can show specific text along with the image

commit: https://github.com/threefoldtech/jumpscaleX_core/commit/adfa5ffad790d4e07903a6f3456d0528d173eaaa


The herokuapp of the demo app is down

Hello,
The endpoint for demo is down. I am trying the example with this url https://always-read-the-plaque.herokuapp.com/plaque/red-rocks-amphitheatre and I do not get any response.
Could it be possible to restart it?
Thanks!

Hu. Sorry about that. Since heroku no longer supports free dynos, I stopped this deploymend.
Got any alternative suggestions?

If the app is built using one of these frameworks, I think there is a generous free tier plan on Vercel.
If it's built with something else, I don't have any easy solution, but you could make the backend code available on github, to let people run it locally. This way we could make the example pointing at localhost e.g. https://always-read-the-plaque.herokuapp.com/plaque/red-rocks-amphitheatre to http://localhost:3000/plaque/red-rocks-amphitheatre.
A possible improvement on the later solution would be to offer a way to specify just the host, so that all examples automatically point at the right place. But this would be more of a "nice-to-have": I am totally, and I think most people are, comfortable just editing the examples manually.

Great!!! I'll go through your documentation when available.
Thanks for your work!


[CODE IMPROVEMENT] Allow NAN for parent_id

Currently, when using parent_id it is required to use parquet to properly load None as parent_id.
Let's also allow CSV why allowing np.nan. Check infinite loop behavior.
Have also fallback if it cant be found.

was a bug in the data, code is fine


I have added a new Notie style

Notie can now just display a message with no input/action from the user

@ballerabdude Could you rebase your repo to upstream and squash your commits?