Ability to provide a default value for TimeoutPolicy via configMap

Currently TimeoutPolicy is hardcoded to infinity. As an operator I should be able to specify a  default value which is not infinity in a configMap.

/assign


Add ability to specify a policy id at creation time

This PR changes the following:

Add ability to specify the policy id as a string at creation time; if not specified a uuid v4 is used
Start refactoring of tests so that they create their own data at the beginning and then clean up; some still need to be changed and maybe we can even group in some helper the creation of test data
Move policies tests to end-to-end tests

Ref. issue #251

@paolochiodi rebased now :)

Looks good except the migration update. Also, what's the practice for releases ?
As these are kind of breaking changes, do external teams point to our master or a specific tag ?

@nherment I think they will have only master as long as we do not tag a "stable" release.
About db breaking changes: That's something we need a strategy for. I think that it will also depend on who is going to use this (we may not be able to provide an automatic script for everyone)


[feat] On Windows, allow to customize the name of window class used v1.x

Describe the problem
Currently, on Windows, all app windows created by tauri have the same class named "Window Class", it would be neet to allow for customization of the Windows' window class name.
One use case: some windows custom hotkey managers leverage window class names to select windows...
Describe the solution you'd like
I would like to add a WindowConfig option called window_classname. Default should still be "Window Class" : it would be a non-breaking change :)
Alternatives considered
No alternative, unless simply dumping those tools and find one using other characteristics of windows...
Additional context
I have patched it on my side and want to contribute it to the upstream project.
It does depend on a new feature on Tao / winit side. Here are the correponding PRs:

winit : https://github.com/rust-windowing/winit/pull/2978
Status : All 55 checks passed, waiting for merge
tao : https://github.com/tauri-apps/tao/pull/770
Status : waiting for merge

This issue is a mirror of #7498 but dedicate to the version 1 of Tauri.

No need to open multiple issues, if the feature lands in 1.x, it will be merged to 2.0 as well


Avoid node reuse during desktop-hosted builds

If msbuild is invoked with the (default) /nodeReuse:true argument,
worker nodes hang around for a period of time after the build is complete
to see if more work will be incoming. This is normally a nice
startup-time-reducing perf optimization. But in our own build, it means
that many newly-created MSBuild assemblies are opened and uneditable,
failing builds and causing cleans to fail.
Changing the Windows build script to just always pass /nodeReuse:false.

LGTM

D'oh, forgot that we dropped that switch entirely on core, not just made it a no-op.

Is this because there is no node reuse on core?  Or you can't turn it off on core?

@jeffkl You can't turn it on. The way it's implemented on Windows uses a named pipe named after the PID, so you can enumerate all MSBuild processes and ask if they're available for your use. But that's a potential elevation-of-privilege attack if you can talk to an elevated process (or another user) while yourself running in low-priv mode. Windows provides APIs to check that both sides of the pipe are the same user + elevation level, but other OSes don't, so we just cut the feature for Core. See 86f213b and #506.

Interesting, well then I'm sure you know what to do, conditionally add the argument...

Can you branch whether to use the switch or not in the caller of build.cmd? I think that one knows what the targeting configuration is.

Turned out to be easier to branch inside build.cmd but after checking for a host.

:shipit:


Time zone setting

RainLoop version, browser, OS:
All versions, all browsers, all OS
Expected behavior and actual behavior:
Steps to reproduce the problem:
In the next releases, could you add a time zone setting for any email account?
My mail is used by users from different countries and I would like them to see the time of receiving or sending a letter in their local time, and not the server time.
Logs or screenshots:

That's up to the OS in use.
If your PC is set to New York, all times are that area.


[bsc#1087447] validate pillars

In the past Rails projects I've worked on we always used https://github.com/rmm5t/strip_attributes to solve this issue. But I'm not sure if it would be fine for Velum since is another gem.

I don't see this gem in obs://devel:languages:ruby:extensions. If you want to use that gem, we need to package that one.
I don't see any issue using that, but this will get blocked until we have the rpm.

I don't see any issue using that, but this will get blocked until we have the rpm.

probably there's no issue, but i think there is also no benefit in functionality. if we use this validator for other fields as well, we might reconsider, but i'd say let's do the minimal now, and save us some gem maintenance


Not able to deploy a repo subfolder to Azure app services

Context:

nodejs project stored in Azure DevOps repository
The code and package.json are in a subfolder called server
Code should be deployed to Azure App Service, using continuous deployment from the Azure DevOps repository
For deployment to work, Kudu/oryx need to know to look into the server  folder and not in the root of the repository.

According to documentation this can be done by the PROJECT configuration:
https://github.com/projectkudu/kudu/wiki/Customizing-deployments#deploying-a-specific-folder-for-a-node-php-or-aspnet-site
On the list of ORYX configurations, PROJECT is listed as a only working for .NET:
https://github.com/microsoft/Oryx/blob/main/doc/configuration.md
Repro steps
Add application setting in Azure: PROJECT = server
Commit changes to repo to trigger Kudu
The log/error given by the failure.
Detecting platforms...
Error: Could not find the .NET Core project file.
Is this a documentation error, or is the PROJECT command expected to work as described?

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Replaced WebL10n with i18next in Chart activity

Fixes #1378
Locales work, no errors in console.

Nice. Thanks


Fix getgrpgid

The Getgrgid function does not work since the error check is wrong.

Thx! These packages severely lacks unit tests... :)


assign should keep unique values

Description
assign should keep unique properties of both object.
issue reproduction example
Checklist

[x] Changes are covered by tests if behavior has been changed or added
[x] Tests have 100% coverage
[ ] If code changes were made, the version in package.json has been bumped (matching semver)
[ ] If code changes were made, the yarn build command has been run and to update the cdn directory
[ ] If code changes were made, the documentation (in the /docs directory) has been updated

Resolves
Resolves #247

Any news on this? I have a really ugly temp implementation in my codebase at the moment, waiting until this gets merged in so I can remove it, would be a shame if this PR went stale.

@tsa96 and @apstanisic I'm merging this. Appreciate the contribution and the patience 🙏 it will be available later today when v11 is published.


Clean unwanted folder, files from the res folder

Hi,
Attached contained few changes that should clean or remove some unwanted images from our res folder.

Coverage remained the same at 27.987% when pulling ecaadd28d12fca91c3d9d2076c2fc0cc61e99e57 on nuruddeensalihu:nuru/clean_splash into 0123786d3bd317db25273fd80ca189dcc4e1f4de on binary-com:master.

Coverage remained the same at 27.987% when pulling 0349b6948bf0bd17a66c2605518be62230522a12 on nuruddeensalihu:nuru/clean_splash into 0123786d3bd317db25273fd80ca189dcc4e1f4de on binary-com:master.


More helpful error from entrypoint_waiter

Test looks good to me - but it just looks like you're testing a test helper, which is not expected in test_service. Does this relate to framework code? Do we need to test test helpers? Apologies if i've missed the point here!

The code in nameko/testing is providing test helpers for other users of the library. So if you build services in nameko you can use these helpers when writing tests for them. We should make sure they work as expected :)


console.log(parsed.groups);

This debug message has been left in the code.

It was removed in the last batch of commits I just added


[AUTOPATCHER] Patches CVE-2021-3968 for vim

        Patches CVE-2021-3968 for vim.
        This patch and PR was automatically generated and needs human verification before merging.

Not required per #1679

Not required per #1679


[WIP] Common error message helper.

Overview.
This is the 3rd attempt.
Depends upon:

[ ] #229

Goals

Provide more verbose error messages.
Introduce more pedantic error checking.
Move more error checking to initialize time.

Changes
Parsing

Ensure required fields are defined (see note).
Reject fields that are the wrong type.
Reject unknown fields.

Next processing

Ensure Next/Default are valid.
Add Next validation to ChoiceRule and Catcher.
Reject a invalid Next field in a child ChoiceRule.
Remove State access of Workflow#payload["States"] for validations (punted).

Reduction of scope
I did the following changes to reduce the scope of this

No longer type checking/requiring on every field (booleans, timestamps, strings).
No longer rejecting unknown fields.
Fixes to Choice have been extracted to #189

I can easily add the first 2 items back but I was trying to keep this down.
Before / After
- Missing field "States"
+ Workflow requires Hash field "States"
- Missing field "StartAt"
+ Workflow requires String field "StartAt"
- "StartAt" not in the "States" field
+ Workflow requires field "StartAt" to be in "States" list but got [Foo]
- Choice state must have "Choices"
+ States.Choice1 requires Array field "Choices"
- "Choices" must be a non-empty array
+ States.Choice1 requires Array field "Choices"
- "Default" not in "States"
+ States.Choice1 requires field "Default" to be in "States" list but got [MissingState]
- Invalid state type: [Invalid]
+ States.FirstState requires String field "Type" but got invalid value [Invalid]
- Missing "Type" field in state [FirstState]
+ States.FirstState requires String field "Type"
- (ruby runtime error)
+ States.FirstMatchState.Choices.0.Data requires Path field "Variable"
- Path [wrong] must start with "$"
+ States.FirstMatchState.Choices.0.Data requires Path field "Variable" Path [wrong] must start with "$"
- (ignored)
+ States.FirstMatchState.Choices.0.Not.0.Data does not recognize field "Next"

Changelog History

Pass a Validator instead of workflow.
Pass an Enhance payload with getter methods.
State#include Validation. <==

Have another branch that:

Adds booleans, timestamps, and strings back.
Rejects unknown fields
uses a dsl

update:

introduced common validation helper methods

update:

rubocops

update:

trying to reduce complexity

update:

extra use of validate_field
remove validation methods with a single line.

update:

Introduce a validator (so we won't be passing around the workflow everywhere)
NonTerminal now runs own validation and sets next/end

update:

implemented common error validation for other PRs
separate validator from workflow (will pass around)

update:

rubocop fixes

update:

rubocop fixes

update:

no longer able to pass state into validator
validator state_for no longer modifies selt.

update:

remove no longer valid validator comments
string interpolation

update:

rebase (to help other PRs resolve conflicts)

update:

rebase
no longer moving End and Next parsing in parent NonTerminalMixin
validating Catch#Next is a valid ref, and Catch#ErrorEquals is a list
validating ChoiceRule Next (first and second levels)
validating Retrier#ErrorEquals is a list
Fixed up Choice texts that were not valid

update:

dropped key? to make code climate happy

Goal is to simplify this PR
update:

rebase
build upon name_path
dropped PayloadValidator that wrapped a payload, and instead have a mixin.
don't handle strings, booleans, timestamps, nor detect missing fields.

update:

trying to get the complexity of ChoiceRule#validate down

update:

added reject_field to reduce complexity from 8 => 5

update:

reducing changed line count +232/-110 => +218/-78

update:

rebased onto newest name_path
rearrange a few methods in validation mixin methods
now passing workspace to all objects that need validation
fixed 2 specs that had Next and End

WIP: This keeps starting small and growing large. Conflating too many ideas. Going to move the path code into the full name change.
This may become multiple PRs:

unify error messages
common helpers to make validation easier
validate all things

I'm merging the dsl work into here and I squashed the commits down to the various forms of this code (did not keep the payload validator.)

handle paths and complicated parsing
handle every field (with larger datatype coverage)
dsl to "simplify" some of the data concepts (having the object know the fields does makes validation and parsing easier)

This has run its course with a good chunk of the error message changes being promoted to #229


Investigate Forgotten Exceptions in Inspector

As mentioned here, the Object Inspector apparently catches Exceptions without logging them. This would be not very helpful and should be changed. Investigate.

Can't reproduce so far, but improved error handling of Property getters and setters throwing Exceptions.


Match types are unable to correctly handle opaque types.

Compiler version
3.5.0
Minimized code
object A:
    opaque type Str = String

object B:
    type F[X] = X match
        case A.Str => String
        case Int => Boolean
        case Boolean => Int

    val x1: F[A.Str] = ""
    val x2: F[Int] = true
    val x3: F[Boolean] = 1

Output
compile error.
Expectation
This issue hinders the ability to match Tuple and NamedTuple within Match Types.

As expected. There are countless issues already closed as "as designed" on the topic.

As designed. There are countless issues already closed as "as designed" on the topic.

How to calculate opaque type?


[lint] Pin rubocop to 0.56.x & fix issues

Having rubocop pinned to any 0.x version was causing inconsistent behavior, including the travis build to fail.
Rubocop 0.58.x is latest but drops support for ruby 2.1. I don't have a use case for ruby 2.1 or care about dropping support for it, but that's not my call.
Rubocop 0.57 has a bug (with an unreleased fix, https://github.com/rubocop-hq/rubocop/pull/6003) that impacts this project.
So pinned to 0.56.x for now.

@jimmyngo @igor47 I'd start with this PR since it fixes the travis build.
We've had a fork for over a year with many enhancements and fixes.  We'd like to be as least-forked as possible. Ideally not at all. Many of the PRs @JKHiggins and I have opened this week we've been running in production for months.

I have a bunch of these fixes locally that I haven't pushed to the public repo. I'll try to work through a bunch of them throughout the week and close out duplicates. I did see some that would be useful and will revisit after.

Excellent, thank you.

Closing in favor of: https://github.com/airbnb/interferon/pull/51


python-netaddr issue when installing zabbix-agent when not running as sudo

Describe the bug
python-netaddr fails to install zabbix-agent when not running under sudo
Installation method/version
I normally add the role as a git submodule inside the project using that latest commits.

Github / latest

Ansible Version
ansible 2.7.7
  config file = /home/willhall/DevOps/zabbix/ansible/ansible.cfg
  configured module search path = ['/home/willhall/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /home/willhall/.local/lib/python3.6/site-packages/ansible
  executable location = /home/willhall/.local/bin/ansible
  python version = 3.6.7 (default, Oct 22 2018, 11:32:17) [GCC 8.2.0]

Targetted hosts
Concerns the following OS(es):

Ubuntu

Expected behavior
I would expect that the playbook completes, however, it seems to have inconsistent results. I have also tried using docker to isolate my environment from the playbook, however, that appears to have no better results.
When installing zabbix agent (3.4) I seem to bounce between two errors:
TASK [dj-wasabi.zabbix-agent : Install local python-netaddr package] ***************************************
FAILED - RETRYING: Install local python-netaddr package (3 retries left).
FAILED - RETRYING: Install local python-netaddr package (2 retries left).
FAILED - RETRYING: Install local python-netaddr package (1 retries left).
fatal: [example.com -> localhost]: FAILED! => {"attempts": 3, "changed": false, "module_stderr": "sudo: a password is required\n", "module_stdout": "", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 1}

and
TASK [dj-wasabi.zabbix-agent : Install local python-netaddr package] ***************************************
changed: [example.com -> localhost]

TASK [dj-wasabi.zabbix-agent : Set default ip address for zabbix_agent_ip] *********************************
ok: [example.com]

TASK [dj-wasabi.zabbix-agent : Get Total Private IP Addresses] *********************************************
fatal: [example.com]: FAILED! => {"msg": "The ipaddr filter requires python-netaddr be installed on the ansible controller"}

It only appears to be resolved by using sudo ansible-playbook -K to be able to enter the sudo password to successfully interact with your local install.
Additional context
Pip shows netaddr as installed:
pip install netaddr
Requirement already satisfied: netaddr in /usr/local/lib/python3.6/dist-packages (0.7.19)

I am not sure if this is intended behaviour, but I've never seen this on any other ansible roles I am using.

Sorry put this on the wrong project.


python3-watchdog_0.8.5: fix PYPI_SRC_URI

Python3-watchdog that is a dependency of qtivi was renamed in pypi
from watchdog3 to watchdog. This stops any of the automotive builds.
Fixes: AUTOSUITE-1504
Fixed in upstream thud branch but not for QtAS branch. Therefore,
this fix is cherry-picked here.

I do not think you need to mask and copy the recipe. All you need to do is override PYPI_SRC_URI in a *.bbappend.

I do not think you need to mask and copy the recipe. All you need to do is override PYPI_SRC_URI in a *.bbappend.

Thanks, I will change it.

Build has been failed:
[2020-03-23T10:23:45.760Z] ERROR: python3-watchdog-native-0.8.5-r0 do_fetch: Fetcher failure for URL: 'http://files.pythonhosted.org/packages/36/48/743fa68f043bf8cab48e1fa70503a49c18ade92c543f7517af2ab3fdfaff/watchdog3-0.8.5.tar.gz'. Unable to fetch URL from any source.
[2020-03-23T10:23:45.760Z] ERROR: python3-watchdog-native-0.8.5-r0 do_fetch: Function failed: base_do_fetch
[2020-03-23T10:23:45.760Z] ERROR: Logfile of failure stored in: /workspace/pelux_yocto/build/tmp/work/x86_64-linux/python3-watchdog-native/0.8.5-r0/temp/log.do_fetch.21604

Seems like hostname resolves to different ip:s depending on which node we build on. Some servers requres "https://" while others work with "http://".
I can wget "https://" on both stockholm and reykjavik. But reykjavik build still failed in Docker... ?

I changed the URI to another option I got from @martin-ejdestig. IP is hardcoded.

Looks like temporary solution

I believe you should keep using tags in titles, e.g. python3-watchdog: fix PYPI_SRC_URI

I believe you should keep using tags in titles, e.g. python3-watchdog: fix PYPI_SRC_URI

Commit message updated.

There is a fix in boot2qt for this now: https://codereview.qt-project.org/c/yocto/meta-boot2qt/+/292454/2

There is a fix in boot2qt for this now: https://codereview.qt-project.org/c/yocto/meta-boot2qt/+/292454/2

I know, but not on the 5.13.1 QtAS branch which PELUX manifest is based on, i.e. it cannot be found here: https://code.qt.io/cgit/yocto/meta-boot2qt.git/log/?h=5.13.1_QtAS
I have requested TCQ to merge this fix to this branch as well but no response yet.


fluffychat: init at v0.20.0 [major wip]

Motivation for this change
Fixes #101111. Major WIP. I don't think there's any Flutter packages in tree, I think due to SDK freedom issues. (Another case where getting some things from Robotnix into nixpkgs might be useful? Or a new "flake-flutter" or "flake-android" with these tools available?)
status: probably not even close to building, probably needs someone familiar with Flutter to help fix this.
Things done

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

[ ] 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"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I'll try to get this packaged as part of #147107

Superseeded by https://github.com/NixOS/nixpkgs/pull/147107


Nothing happened After clicking and redirecting MetaMask

Describe the bug
It's an issue about MetaMask doesn't work, including no popup Connecting interface or anything else after clicking and redirecting MetaMask app. (iOS 16.1. did not test in Android yet, but there is an another discussion about this problem in Android)
To Reproduce
Steps:
Use the sample demo to build the app and run on iPhone.

choose MetaMask
Unity app will redirect and launch MetaMask app successfully.
Nothing happened after MetaMask launched, no popup showed, no tips in MetaMask even after several minutes(It should show me the popup to confirm to connect, but not)
If I return the Unity app, the WC waiting for connection UI closes immediately, as if it were shut down. And stayed in network list UI.

Environment

OS: [iOS 16.1]
Unity version [2021.3.29f1c1]
Package name and version [latest]

Wallets used for testing
Metamask iOS
Additional context
Thanks for investigating. Anything If needs, contact me at anytime.

Hey @defchen,
Thanks for opening the issue! Does the error persist after changing relay url to relay.walletconnect.org?

I test the web3modal.com, open it in safari or any other browser on mobile, MetaMask is also broken. I think it's a common problem in v3/v4.

I think it's a common problem in v3/v4.

Are you referencing to the version of Web3Modal?

@defchen, could you check if any of these fields in the WalletConnectProjectConfig scriptable asset are empty?

Id
Name
Description
Url
Icon (at least one item should be present)

I think it's a common problem in v3/v4.

Are you referencing to the version of Web3Modal?

I'm referencing to the version of www.web3modal.com. The web version is also broken with MetaMask. Not only the unity version.

@defchen, could you check if any of these fields in the WalletConnectProjectConfig scriptable asset are empty?

Id
Name
Description
Url
Icon (at least one item should be present)

Id: not empty 60de9604fdb9bdc7931c....
Name : not empty
Description: empty
Url: empty
Icon: empty


detect main = main case #39

Fixes #39.

Coverage decreased (-1.9%) to 65.421% when pulling 0421d9d36dba88756c9d6ba81bd2bdf7e4657fea on main-main into beda9334b4d98c38682662ca2c6c0e449d4dada4 on main.


Metadata validation provides more complete report

Closes #216 .
This PR modifies the behaviour of the function utils.validate_against.schema so that a report of all metadata schema errors is provided, rather than just the first issue found.
The function still raises jsonschema.ValidationError, but the error message is now formatted as follows:
01 | <field name or (missing) > : <reason for failure>
02 | <field name or (missing) > : <reason for failure>
...

For example, the error message for a metadata file that has a bad value for name, and no value at all for experiment_uuid, would be:
01 | name : None is not of type 'string'
02 | (missing) : 'experiment_uuid' is a required property

This behaviour then flows through to the metadata-validate command-line script. The returns of this script have also been refined to:

More clearly show a success message;
Print out the new report in the case of a validation error;
To provide stack trace output in the event of any other form of error/exception.

Tests for the new error handling have also been included. These tests are exhaustive, but not complete (I didn't think it was necessary to provide every single possible permutation).

I've checked out to this branch & run the tests & all the additional tests seem to be failing on my machine - I'd just like to do a tad more prodding to work out why before I approve the changes. Bit strange given they're working on CI...

@charles-turner-1 what was the environment issue you ran into?

@marc-white just a pytest extension (either watcher or sugar, not sure which - I rebuilt the environment without either) capturing the exception and assuming it was wrapped with exception.value, rather than a bare exception - caused errors to get turned into TypeErrors.


dep does not import kubernetes/client-go correctly

What version of dep are you using (dep version)?

0.3.1
What dep command did you run?
Dep does not seem to import kubernetes/client-go code correctly. After stubbing my toe a bit on this with more complex examples, I decided to simplify and see if I could get it working with the very basic example code at https://github.com/kubernetes/client-go/blob/release-4.0/examples/in-cluster-client-configuration/main.go
I used the following Gopkg.toml file:
[[constraint]]
  name = "k8s.io/client-go"
  version = "4.0.0"

After getting the code in place and running dep ensure, I get the following errors when trying to compile the code:
# github.com/drud/site-manager/vendor/k8s.io/client-go/tools/clientcmd/api
vendor/k8s.io/client-go/tools/clientcmd/api/register.go:35:11: cannot use Config literal (type *Config) as type runtime.Object in argument to scheme.AddKnownTypes:
	*Config does not implement runtime.Object (missing DeepCopyObject method)
# github.com/drud/site-manager/vendor/k8s.io/client-go/pkg/apis/authentication
vendor/k8s.io/client-go/pkg/apis/authentication/register.go:47:16: cannot use TokenReview literal (type *TokenReview) as type runtime.Object in argument to scheme.AddKnownTypes:
	*TokenReview does not implement runtime.Object (missing DeepCopyObject method)

<a few hundred similar lines>

I also tried with this additional constraint, which I've seen listed as required in a few different places. I had similar (nonworking) results.
[[override]]
  name = "k8s.io/apimachinery"
  branch = "version-1.7"

Originally I was thinking client-go or one of it's dependencies was just broken, so I decided to give it a try with glide. I was able to get the exammple working with the following glide.yaml:
package: github.com/some/project
import:
- package: k8s.io/client-go
  version: v4.0.0

And finally, I'm not sure if they are useful here, but this is the result of dep hash-inputs:
-CONSTRAINTS-
k8s.io/client-go
svc-^4.0.0
-IMPORTS/REQS-
k8s.io/apimachinery/pkg/api/errors
k8s.io/apimachinery/pkg/apis/meta/v1
k8s.io/client-go/kubernetes
k8s.io/client-go/rest
-IGNORES-
-OVERRIDES-
-ANALYZER-
dep
1

I'm happy to provide the .lock files generated by both dep and glide if that would be helpful. This seems like it may be related to https://github.com/golang/dep/issues/1124.
Thanks for all the effort going into making dep awesome!

hi! and ugh, sorry for slow response - i'd flipped a switch in my mind that said "this is subsumed by #1124, and doesn't need to be addressed separately." and, while i expect that a satisfactory solution to #1124 would result in this not being a problem, this should still be able to be addressed today.
it's quite likely that your issue is with some dependency of k8s.io/client-go - indeed, possibly k8s.io/apimachinery. the [[constraint]] you added on it won't have any effect unless it's a direct dependency of your own project. but, hard to be sure, so yeah - if you could provide the Gopkg.lock (and contrast with the glide.lock that did work), that'd be super helpful 😄

Hi, I have this same issue. I tried adding the constraint for apimachinnery in the toml file, and update the lock file to the right branch/revision, but it did not work either. I've attached the original files generated by dep.
Gopkg.lock.txt
Gopkg.toml.txt

As commented in https://github.com/kubernetes/client-go/issues/298 ,
[[override]]
  name = "github.com/ugorji/go"
  revision = "8c0409fcbb70099c748d71f714529204975f6c3f"

in Gopkg.toml makes dep ensure pull the right files.  I had this same problem and that solution worked for me.

Thanks, that worked for me.

Hi all - I've created a little tool 'godep-to-dep' to try and make this easier/fix it upstream https://github.com/munnerz/godep-to-dep
If we can integrate this tool with our release process, we should be able to ship Gopkg.toml files out of the box.

v6.0.0 appears to be working via dep for me now, not sure if it's by design or luck.
[[constraint]]
  name = "k8s.io/client-go"
  version = "v6.0.0"

It's because they swapped out ugorji for a different library. 😀

What is the current status of this issue?
I am currently getting this error trying to use dep to install k8s.io/client-go/tools/clientcmd
See below:
dep ensure -add k8s.io/client-go/tools/clientcmd
Fetching sources...

Failed to add the dependencies:

  ✗ could not infer project root from dependency path: k8s.io/client-go/tools/clientcmd: unable to deduce repository and source type for "k8s.io/client-go/tools/clientcmd": unable to read metadata: unable to fetch raw metadata: failed HTTP request to URL "http://k8s.io/client-go/tools/clientcmd?go-get=1": Get https://k8s.io/client-go/tools/clientcmd?go-get=1: net/http: TLS handshake timeout

adding dependencies failed

The above workarounds don't seem to work these days because of #1306:
Solving failure: No versions of k8s.io/client-go met constraints:
	v7.0.0: Could not introduce<EMAIL_ADDRESS>as its subpackage k8s.io/client-go/pkg/api/v1 is missing. (Package is required by (root).)
[...]

Very frustrating.

It appears your own code is not up to date with recent versions of the
codebase. You'll need to update your code to compile against client-go
7.0.0 (or at least, all your code should reference valid packages @ that
dependency version).
TL;DR: replace all k8s.io/pkg/api with k8s.io/api imports (their new home).
Btw - easiest way to start this process would be to remove your vendor dir
and check out the versions of k8s.io/* you want to use on your gopath.
Future upgrades won't be as bad - but because your own code is relying on
such an old v of k8s.Io/*, more significant work is required!
On Sat, 9 Jun 2018 at 02:21, Alexander Staubo<EMAIL_ADDRESS>wrote:

The above workarounds don't seem to work these days because of #1306
https://github.com/golang/dep/issues/1306:
Solving failure: No versions of k8s.io/client-go met constraints:
v7.0.0: Could not introduce<EMAIL_ADDRESS>as its subpackage k8s.io/client-go/pkg/api/v1 is missing. (Package is required by (root).)
[...]
Very frustrating.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/golang/dep/issues/1207#issuecomment-395929229, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAMbP2pUvJAZjM2MHLh8ShG1ZNogJNapks5t6yMAgaJpZM4PhPIg
.

@munnerz Thanks, you're right. I edited my comment after I realized that I was referencing packages that had been moved. (I've never liked Dep's behaviour, where it reads your imports to determine your dependencies, as opposed to requiring explicit declarations in Gopkg.toml. Because it should at least be able to give me an understandable error message, and tell me which files tried to import the non-existent package.)

I struggled importing "k8s.io/kubernetes/pkg/kubectl/cmd" and wanted to share my solution as a reference. I ran dep ensure -v and manually resolved the dependencies for Kubernetes 1.13.4 until I could import, build, and run kubectl.
https://gist.github.com/alexeldeib/f2406b4e0f12f5cd44a82f544b47d312


doc update

Specify in step 6 that you must have a custom attribute display created for the widget to pick up the address field.
https://docdev.arcgis.com/en/web-appbuilder/create-apps/widget-public-notification.htm

wrong repo


fix: increase MinDeposit from 1K to 10K TIA

Closes #2629

Codecov Report

Merging #2631 (135891d) into main (44be82a) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2631   +/-   ##
=======================================
  Coverage   20.63%   20.63%
=======================================
  Files         133      133
  Lines       15346    15346
=======================================
  Hits         3166     3166
  Misses      11877    11877
  Partials      303      303

Files
Coverage Δ

app/default_overrides.go
15.71% <100.00%> (ø)


Nil inhibit rule in alertmanager configuration causing alertmanager pods to crash because of NPE

I was running Cortex and I came across an issue that caused my alertmanager pods to crash unexpectedly. I was trying to use the following alertmanager configuration:
global:
  resolve_timeout: 5s
receivers:
  - name: example-email
    email_configs:
      - to<EMAIL_ADDRESS>inhibit_rules:
  -

I forgot to add my inhibition rules but didn't realize it would cause a NPE. The error from the crash was:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x10a83b1]

goroutine 261 [running]:
golang.a2z.com/AWSPrometheusCortex/vendor/github.com/prometheus/alertmanager/inhibit.NewInhibitRule(0x0)

The error is coming from https://github.com/prometheus/alertmanager/blob/main/inhibit/inhibit.go#L53-L56:
	for _, cr := range rs {
		r := NewInhibitRule(cr)
		ih.rules = append(ih.rules, r)
	}

Are nil list elements, in this case inhibition rules, processed even if they are nil on purpose? If I add  if cr == nil continue to the code, it doesn't cause the alertmanager pods to crash since it skips the nil inhibition rule.

Thanks for the report! Not sure why it wasn't noticed before but it should be fixed.
Another null-pointer panic would also happen with this snippet:
receivers:
-
- name: foo

I think that we should change the slices of pointer items to slices of value items.
https://github.com/prometheus/alertmanager/blob/9aa5974c0821631b18ec33a0d538b217521f7e78/config/config.go#L300-L301
cc @gotjosh

Thanks for the comment! I'll work on the PR to address this.


Remove BeOS code

Since Haiku support is available, it makes sense to get rid of old BeOS
code. Documentation will be handled later, in order to keep necessary
information.

I suppose this code has not been maintained for a while, might not even build.
If someone thinks it would be good to keep anyway, please speak up!

Before this is done we should probably see if anyone still running BeOS gets a chance to check on this.
As a side note, would it be advisable to use python 3.7 over python 2.7 at this stage?

Well, I haven't heard anything for more than a month.  I also have no idea if the current code even builds.
Python 2 has been deprecated, maintenance is stopped, thus Python 3 should be used if possible.

We've checked the build with python 3.7 that seems to be working OK: https://ibb.co/7pmx65Y

FWIW, @bitigchi is looking into bumping our vim version at: https://github.com/haikuports/haikuports/pull/4900
You could check progress there too :)


Add long_short flag for performance.alpha_beta and modify tearsheet

I recommend adding a long_short flag to the performance.alpha_beta() function along with associated functionality in the function code. My rationale is that when running tears.create_full_tear_sheet() with the long_short flag set to False, the "Returns Analysis" table still shows the alpha and more importantly the beta being calculated by regressing the factor-weighted long-short portfolio vs. the universe mean return, as opposed to the factor-weighted long-only portfolio. vs the universe mean return.
I can work on this and submit a pull request, but I would appreciate it if someone could look over my rationale and comment as to whether you think this makes sense or not before I make the changes.

oh, that's right. Thank you for reporting this @MichaelJMath . If you submit a PR that would be greatly appreciated.


2.7.5 - GUI improvements

@sprout2000
Please check screenshots.

About window

"and other contributors" in trademark for me could be remove.
Also in the instalelr could remove it
For me the copyright could be "(c) 2020-2023 sprout2000"
"LeafView" on top left could be renamed as "About LeafView" and add to the language file ("Info su LeafView" in Italian)
In version field "(27.0.4)" could be removed to avoid misunderstanding about version numbering.
Version is 27.0.5

Installer Exe file properties

For me the copyright could be "(c) 2020-2023 sprout2000"

@bovirus
Thank you for the report with screen shots.
Fixes will be included in the next release.

@sprout2000
You released 10 hours ago a new release (2.7.7).
But without the fix reported here.
May I check also the Inno Setup installer script (.,iss file) for possible impovements?
Thanks.

@bovirus
Since you seem to be using the Windows version, I tried v2.7.7 myself. And the fixes appear to be properly applied.

Could you please upload your screenshots?

@bovirus
Oh... You mean I could remove "and othe contributors". Sorry.
Uh, do you think that's reasonable?

@sprout2000
The copyright message generally contains info about copyright year (start/end) and company orauthor.
You can add below copyright message a line like "Thanks to all contributors for your important contribution" (add it to the language file to enable relative translation"
But is not relative to copyright.
Same think about copyright info in the installer (just startcopyright year-Endcopyrightyerar company or author).
About  Eecroun version use a strings like
v. 2.7.7 (Electron v. 27.0.4)
Please take care about space after "v." and adding of "Electon".
More clear.
Then you can change the string in about window like
LeafView v. 2.7.7 (Electron v. 27.0.4)
(c) 2020-2023 sprout2000
Thanks to all contributors for your important contribution
In "About" window change "LeafView" with "About LeafView" and add "About" is not already included in language file.

@bovirus
OK. I undarstand.
Although there are limitations due to the Electron API specification, I will try to do the best I can.

See electron/electron.

@sprout2000
Thanks for changes.
My proposal

Please take care about new lines in About window between text strings and Caption text on top left

@bovirus
Thank you for your suggestions.
Unfortunately, there is no way to change that part due to limitations imposed by the Electron API: app.setAboutPanelOptions(options).

@sprout2000
Also for Caption on top left?

@bovirus
Unfortunately, that is also out of the developer's control.


Adding instrumentation for github.com/tidwall/buntdb (WIP)

I would recommend closing this in favor of https://github.com/signalfx/splunk-otel-go/pull/228

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@38757b5). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   76.64%
=======================================
  Files           ?       45
  Lines           ?     2308
  Branches        ?        0
=======================================
  Hits            ?     1769
  Misses          ?      489
  Partials        ?       50

Flag
Coverage Δ

Linux
76.64% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out 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 38757b5...76d865e. Read the comment docs.


Adding additional scrape config for a target outside the kubernets cluster

I deployed kube-prometheus in a cluster. Would it be possible to monitor, say an IoT device which is running 'fluent-bit' with node-exporter plugin (which should be scrap-able by prometheus) ?
This IoT device will be running inside the same LAN as the kuberentes cluster where kube-prometheus is deployed.
I know that we can configure prometheus-operator with additional scrape configs as shown here: https://github.com/prometheus-operator/prometheus-operator/blob/master/Documentation/additional-scrape-config.md
But how can we do that with kube-prometheus?

@mac-chaffee I am a bit surprised that this works :D Personally I used a Service object without selectors and created Endpoints object for it manually (like here). But it seems that your solution would be much easier to manage! Kudos to you.
It seems to me that with your tip, we should rewrite how we handle adding windows hosts as Service with ExternalName should be easier to implement than what we have right now.


Raspberry Pi Zero W blue-alsa input

Curious to see if anyone has been able to get blue-alsa to provide bluetooth input over something like a bluetooth headset, on the raspberry pi zero W. I have been trying pretty vigurously for the past couple of days to no avail. After I have followed all the steps on issue#22 and issue#7 and still receive a no device found error when I try to arecord. Any input would be appreciated!

Hi,
Firstly, you have to find out where the logs for bluealsa are stored (if anywhere) if you using Raspbian Stretch (I'm not a raspberry user, so I do not know where such logs are stored). If you're running bluealsa by yourself, post logs here (make sure that bluez-alsa was compiled with --enable-debug).

Try to do it with Bluetooth-to-USB dongle, using Raspberry Pi Bluetooth doesn't give any success.
http://youness.net/raspberry-pi/bluetooth-headset-raspberry-pi-3-ad2p-hsp

Although this is an old thread, I though the following info might be of interest regarding the Raspberry Pi Zero using Stretch:
Up until mid August 2018, "sudo apt-get install bluealsa" installed: "0.7 100  100 /var/lib/dpkg/status"
Together with "bluez v5.50", this populated "sudo bluetoothctl show" with 11 "UUID" profiles, including the Audio Sink profile, enabling me to connect my mobile telephone to my PiZero fine using "Media Audio"
However, the mid-August "sudo apt-get upgrade" updated bluealsa to: "0.9 500 500 http://archive.raspberrypi.org/debian stretch/main armhf"
This update has removed/disappeared the following 3 UUID profiles from "sudo bluetoothctl show":
UUID: Audio Sink
UUID: Handsfree
UUID: Headset
(and also caused the "sudo systemctl status bluetooth" error: "bluetoothd: Failed to set privacy: Rejected (0x0b)"
Predictably, my mobile telephone will no longer connect to the Raspberry Pi Zero.
The only solution I have found is to re-install an earlier backup version of Stretch with bluealsa 0.7, and then issue a "sudo apt-mark hold bluealsa" command so that "sudo apt-get upgrade" will NOT update/change the bluealsa version number.
(Or use the full version of Stretch 27/6/18 which has the 0.7 version of bluealsa, instead of the Stretch-lite version.)

@Rob58329, see issue #137

Sorry about posting here instead of there, but I'm getting 404 on that issue and hovering over it says it's private.

I think that the problem is cause by recent bluealsa changes (according to this comment). By default not all BT profiles are enabled. In order to enable A2DP Sink, one has to specify this profile on the command line. So, it seems that the startup script has to be changed from bluealsa to bluealsa -p a2dp-source -p a2dp-sink. See bluealsa --help for details.

Would there be any negative effects from using bluealsa -p a2dp-source -p a2dp-sink by default?

Alright, thanks. I'll leave it as is for now then.

re. No "UUID: Audio Sink" after mid-Aug18 "sudo apt-get update" of bluealsa to v0.90 (Raspberry Pi Zero W)
@Arkq, as @XECDesign says, issue #137 is marked as "Issue title is private" so I can't view it.
@XECDesign, thanks for repeating the info which I am guessing is in issue #137.
I can confirm that for my Raspberry Pi Zero W, with full "sudo apt-get upgrade" to 31Aug18:

If I edit "/lib/systemd/system/bluealsa.service" and change the relevant line to "ExecStart=/usr/bin/bluealsa --profile=a2dp-sink"
Then after reboot the "UUID: Audio Sink" appears again under "sudo bluetoothctl show" (and I can connect my mobile phone again as "Media Audio"!)  Yay!

Many thanks for this info - very much appreciated!! :-)

I tried your suggestion, Arkq and it worked. I finally dares to update bluealsa, and I had to change the /lib/systemd/system/bluealsa.service to activate all the profiles using the -p flags and it all works just like it used to. Thank you

According to the last 2 comments, this issue was resolved some time ago.


Register “trevin”

@tmchow your page.json seems invalid

@tmchow your page.json seems invalid

Any idea what is wrong? The json is valid: https://github.com/tmchow/my-links/blob/main/page.json

I don't know what was the problem but CI passed now @tmchow, merging it

I don't know what was the problem but CI passed now @tmchow, merging it

The biggest change is I moved away from an empty string as a description value. You may want to check that out in case that’s a problem?


Hybridize

Doing the good ol <d2l-button><button is="d2l-button"></button></d2l-button>

The main change is the primary property. It is now just an attribute on the <button> element

@dlockhart suggested going with <d2l-button>Button!</d2l-button> instead, in anticipation of extending native elements

I'm going to close this PR for now so that it doesn't get merged -- we're going to switch this to just use <d2l-button> straight-up (instead of wrapping a native button) to line us up better with the future ability to extend native elements. I (or someone else) will hopefully get to making those changes shortly.

I'll keep the branch around for reference


Create Question model #15

A Question class model thought for the purpose to solve the issue 15# and minimize issue 14#

ok..I will..


Castblock not working since last update. Response_Status_Code: 500

Hello,
Since the last update castblock isn't working anymore. I attached the output of castblock below.
2021-10-23T16:23:22.853673Z   INFO - castblock.chromecast: Starting the go-chromecast server.
2021-10-23T16:23:22.857282Z  DEBUG - http.client: Performing request -- method: "GET", host: "<IP_ADDRESS>", port: 8011, resource: "/"
2021-10-23T16:23:23.365420Z  DEBUG - http.client: Performing request -- method: "GET", host: "<IP_ADDRESS>", port: 8011, resource: "/"
2021-10-23T16:23:23.369742Z   INFO - castblock.chromecast: The go-chromecast server is up
2021-10-23T16:23:23.864829Z  DEBUG - castblock.blocker: Checking for new devices
2021-10-23T16:23:23.864930Z  DEBUG - http.client: Performing request -- method: "GET", host: "<IP_ADDRESS>", port: 8011, resource: "/devices"
2021-10-23T16:23:26.870112Z   INFO - castblock.blocker: New device found -- name: "Wohnzimmer", uuid: "02829eba8aac03f6cf28c08ba8624129"
2021-10-23T16:23:26.870118Z   INFO - castblock.blocker: Connect to device -- name: "Wohnzimmer", uuid: "02829eba8aac03f6cf28c08ba8624129"
2021-10-23T16:23:26.870143Z  DEBUG - http.client: Performing request -- method: "POST", host: "<IP_ADDRESS>", port: 8011, resource: "/connect?uuid=02829eba8aac03f6cf28c08ba8624129"
2021-10-23T16:23:29.883713Z  DEBUG - castblock.chromecast: Error while connecting to device -- name: "Wohnzimmer", uuid: "02829eba8aac03f6cf28c08ba8624129", status_code: 500, error: "unable to start application: unable to connect to chromecast at '<IP_ADDRESS>:8009': EOF"
2021-10-23T16:23:29.883820Z  ERROR - castblock.blocker: Error while connecting -- name: "Wohnzimmer", uuid: "02829eba8aac03f6cf28c08ba8624129"
2021-10-23T16:23:59.877425Z  DEBUG - castblock.blocker: Checking for new devices
2021-10-23T16:23:59.877445Z  DEBUG - http.client: Performing request -- method: "GET", host: "<IP_ADDRESS>", port: 8011, resource: "/devices"
2021-10-23T16:24:02.880936Z   INFO - castblock.blocker: New device found -- name: "Wohnzimmer", uuid: "02829eba8aac03f6cf28c08ba8624129"
2021-10-23T16:24:02.880942Z   INFO - castblock.blocker: Connect to device -- name: "Wohnzimmer", uuid: "02829eba8aac03f6cf28c08ba8624129"
2021-10-23T16:24:02.880966Z  DEBUG - http.client: Performing request -- method: "POST", host: "<IP_ADDRESS>", port: 8011, resource: "/connect?uuid=02829eba8aac03f6cf28c08ba8624129"
2021-10-23T16:24:05.894206Z  DEBUG - castblock.chromecast: Error while connecting to device -- name: "Wohnzimmer", uuid: "02829eba8aac03f6cf28c08ba8624129", status_code: 500, error: "unable to start application: unable to connect to chromecast at '<IP_ADDRESS>:8009': EOF"
2021-10-23T16:24:05.894363Z  ERROR - castblock.blocker: Error while connecting -- name: "Wohnzimmer", uuid: "02829eba8aac03f6cf28c08ba8624129"
2021-10-23T16:24:35.894695Z  DEBUG - castblock.blocker: Checking for new devices
2021-10-23T16:24:35.894713Z  DEBUG - http.client: Performing request -- method: "GET", host: "<IP_ADDRESS>", port: 8011, resource: "/devices"
2021-10-23T16:24:38.898432Z   INFO - castblock.blocker: New device found -- name: "Wohnzimmer", uuid: "02829eba8aac03f6cf28c08ba8624129"
2021-10-23T16:24:38.898435Z   INFO - castblock.blocker: Connect to device -- name: "Wohnzimmer", uuid: "02829eba8aac03f6cf28c08ba8624129"```

It is already with debug enabled


Token permissions only for resource pool and not for global path

Hello,
my API token only has the necessary permissions for a specific resource pool. The provider only checks the permissions for the global path. That is why the terraform command fails. Error: permissions for user/token <userid> are not sufficient, ...
Example:
API Token has Administration permissions for pool/XYZ
The provider checks the permissions for /api2/json/access/permissions?userid=<userid>&path=/
Is it possible to change the permission route to /api2/json/access/permissions?userid=<userid>&path=/pool/XYZ
Thank you

It used to work with version 2.9.11. At least the permission checks were not there in that version.

I'd suggest to have an option to skip permissions check at all.
Out technical user account in Proxmox has permissions to only one Pool. It fails now on 3.0.1-rc1

To get working  3.0.1-rc2 with user/token with privilege separation and pool's fine-grained permissions here is an example how it could be achieved  - https://github.com/Telmate/terraform-provider-proxmox/issues/784#issuecomment-2135266013


[Toggl Track] Menu bar item shows weird day total hours sometimes -479744:-27:-5

Extension
https://www.raycast.com/franzwilhelm/toggl-track
Raycast Version
1.83.1
macOS Version
15.0
Description
Sometimes the menu bar shows weird total day hours
this is the chrome extension values:

and this is the raycast menu bar

Steps To Reproduce
just open multiple time entries, doesn't always happen but a lot of times it does
Current Behaviour
shows a weird total duration number
Expected Behaviour
should show the sum of all the entries of the day

@RaedGhazal
Could you provide the following information?
Open up Raycast and enter "toggl"

Select Start/Stop time entry
At the bottom, there is Total time:

Could you double check if it also shows incorrect values, as does the menu bar?

@lukebars yes

Okay, so this seems like a global issue with the total time, not only the menu bar.

yeah sorry, a global issue in the raycast extension, just don't use it other than the menu bar so didn't really think of checking there

@bkeys818 seems like a really busy week for me, so it would be great if someone would pick this up. Also - I haven't noticed this behavior even though I am using the extension every day.

oh I might now know how to reproduce, if you edit any time entry from the website/chrome extension, this happens instantly


Add TMDB images quality (tmdb_images_quality) option.

Description

UI changes for new "TMDB images quality" tmdb_images_quality option.
depends on https://github.com/elgatito/elementum/pull/111
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Translation

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

as stated in https://github.com/elgatito/plugin.video.elementum/issues/998 - this works good and it is ready.

@antonsoroko

as stated in #998 - this works good and it is ready.

You mean as stated in https://github.com/elgatito/plugin.video.elementum/issues/945
We are still hoping for this :-) https://github.com/elgatito/plugin.video.elementum/issues/998


🐛 [bug] - Extra ` in copy for exercise 1

📝 Description
Looks like this - (look after labs.com)
echo export CLUSTER_DOMAIN="apps.tl500-2.dev-1.rht-labs.com`" | tee -a ~/.bashrc -a ~/.zshrc

🚶 Steps to reproduce
https://rht-labs.com/tech-exercise/#/1-the-manual-menace/1-the-basics Step 5. Copy and paste into codeready
🧙♀ Suggested solution
remove `
echo export CLUSTER_DOMAIN="apps.tl500-2.dev-1.rht-labs.com" | tee -a ~/.bashrc -a ~/.zshrc
``

hey @mcanoy you sure ? i just did this OK.

what's in your browser local storage where the docs are ?

you're right. I added the tick


Storage/feature/upload no overwrite default

Added overwrite flags to blob creation methods.
This does NOT add overwrite flags to the following operations:

Put block
Append block
Put pages

Addresses #5881

@jaschrep-msft Could you please take a look at this PR and let us know if this is still relevant?

It looks like these overloads ended up being added in a different PR. Closing.


Update allcourses.json

do not make so many pr at once , they review every pr  make sure you have only  2,3 pr in one repo


Long Playlist overlaps with related videos

With YTC enabled longer playlists in video detail view don't scroll but get shown in full length and overlap with the related videos.
Here's what it looks like http://note.io/1I13eI9
I found that adding height: 400px to #watch-appbar-playlist.watch-playlist .playlist-videos-list does the trick. The overflow is already set to auto so it scrolls.
After fixing it looks like this: http://note.io/1FHqLOU

never mind - haven't used the dev version. seems to be fixed there already.


[Improvement] Possible thread issue in MySQLConnectorAdapter.java

What would you like to be improved?
Static variable version is being updated in non-static method buildInternalConnectorConfig.
Correctly updating a static field from a non-static method is tricky to get right and could easily lead to bugs if there are multiple class instances and/or multiple threads in play. Ideally, static fields are only updated from synchronized static methods.
How should we improve?
Make method static?

Fixed by @coolderli in #2139

@coolderli
Can you leave a comment here, I'm handling the release work, as this is your credit, but due to GitHub restrictions, I can't assign it to you.

Closed by https://github.com/datastrato/gravitino/pull/2139


Integrate extractor template with TravisCI

Task to do
After tests are built, integrate with TravisCI
Reason
Able to automatically test changes to extractor template
Result
Full integration with TravisCI and ability to make Pull Requests dependent upon successful CI runs

Integrate test cases remains

Completed .yml.travis file
https://github.com/AgPipeline/base-docker-support/blob/test-development/.travis.yml
and completed a build run

There are some test failures that need to be worked out but basically the system is working as intended. I'll need to make another ticket for fixing whatever's broken.


Document tutorial missing one line of code ?

The tutorial.md line 1044-1049
jumpTo(step) {
    this.setState({
      stepNumber: step,
      xIsNext: (step % 2) === 0,
    });
  }

should add history : this.state.history.slice(0, step + 1)
to make Doms rerender the history.

Jeu Denny. Thanks for the suggestion!
This change would erase part of the history though (so that stepping back would discard all later steps). I don't think that's the intent. 😄  It's meant to be more of a "time travel" feature, where you can step backward and forward.

Jeu vaughn . Thank you for your answer!
I finally know the true intentions about that. 😄
But I think you change the stepNumber that also make the history changed, because the code const history = this.state.history.slice(0, this.state.stepNumber + 1); in the handleClick function . 😄

You're right, you could override history if you stepped back and then clicked a different move. Maybe the tutorial could be updated to disallow that


Attribute information improvements

Existing implementation
The existing form places the URL after the name requiring user to manually enter all fields
UX recommendations

Place URL as the first field
If the URL can be validated it can be used to populate the remaining fields.

Hi @Misjohns and @vishnoianil I would like to work on this issue as my first contribution to the UI side of instructlab

@ediazgallego awesome, please feel free to assign the issue to your self. If you need any help with the dev environment setup, please feel free to ping us on the UI slack channel in instructlab workspace. More details about the weekly community meeting and slack channel here - https://github.com/instructlab/ui?tab=readme-ov-file#community-meeting

@vishnoianil thanks, I probably do not have enough permissions yet to assign the issue to myself, anything I should request or do to get that enabled?

@vishnoianil thanks, I probably do not have enough permissions yet to assign the issue to myself, anything I should request or do to get that enabled?

@ediazgallego This should be fixed now.

Hi @Misjohns I'm currently looking into this issue, I've made the change so the URL field is the first in the section but I'm not 100% sure how to test the second bullet point you mention If the URL can be validated it can be used to populate the remaining fields. , can you explain a little bit more the expectations? Is there some kind of logic that will use the URL value to gather information for the subsequent fields and fill them automatically?

@ediazgallego Knowledge submission can be sourced from various targets. A simple example would be wikipedia. If user adds the URL to a wikipedia page, we should automatically populate other fields (title, revision, license, author) from wikipedia page.
At this point we can target wikipedia, because upstream taxonomy repo only accepting knowledge contribution based on the wikipedia. In future we will add support for more sources for knowledge contribution, and the extraction process for attribution information can be very specific to each target as well.
You can follow this scripts https://github.com/mairin/instructlab-knowledge-utils?tab=readme-ov-file#1-%EF%B8%8F-wikipedia-attribution-genpy  to determine how we can populate this information from wikipedia. Big thanks to @mairin for writing these utilities.


JGRP-2486 FD Monitor gets stuck on TransferQueueBundler

The FD Monitor was split into a heartbeat sender task and a timeout check task, similar to FD_ALL and FD_ALL2. This prevents the timeout check in the FD protocol from being block by the transport.
Ticket: https://issues.redhat.com/browse/JGRP-2486.

Thanks! Applied


Strict tests

This PR turns warnings into errors in tests.
Even if we decide not to use this, we should do something with the warning that it currently fails on. It's from bpe.ini and it says The rnn_size argument is ignored in this type of projection.. Apparently, it comes from the decoder. But there is no rnn_size for the decoder in the ini file. So it's not clear to me (as a user) what should I do to get rid of this warning.

The thing was, that the decoder computed its rnn_size prior calling the encoder projection, which does not need rnn_size. So instead warning, I added an assert inside the projection method to ensure it uses the correct rnn_size.

OK. Next we have small.ini inferring vocabulary from lazy dataset.

I saved vocabularies from vocab.ini and reused them in small.ini so the dataset can remain lazy

in future, for the vocabulary creation, we should use something that could be called "fixtures"..

Why do we actually have a warning while inferring vocabulary from lazy datasets? Because it's potentially slow?

Because you would generally prefer to load the vocabulary from a file and I believe that the fact that you are inferring it from lazy dataset should be emphasized, so I chose to do the emphasis by a warning.


ffe-radio-button-react: Docs use wrong name for RadioButtonGroup

Which package is this issue related to?
ffe-radio-button-react
Describe your issue
The Komponenter page lists the RadioButtonGroup as RadioButtonInputGroup, and the import string is wrong.
What is the expected behavior?
The docs should reflect the exported name RadioButtonGroup
Steps to reproduce
https://design.sparebank1.no/styleguidist/index.html#radiobuttoninputgroup

Aaand I was looking at an internal wrapper... 😨


Speed up docs build

Pull Request Description
Following the process described in this document
I think this is about a 45% reduction in total run time, so it's about twice as fast now!
It's hard to look at the internals of sphinx process but I think there are two main buckets:

Running our notebooks and converting them to html
Creating our api ref from our source code by importing our modules

The sphinx build time is about 8 minutes total. It takes about 5 minutes to run all of the notebooks. So the other 3 minutes are sphinx creating our api reference according to our templates.
I scaled the notebook computations as much as possible while still being reasonable, so that 5 minutes looks hard to beat (feel free to make suggestions!). I'm not exactly sure how to speed up the api reference beyond setting collapse_navigation to True. It's not clear how we can profile that to see where the bottleneck is (if it exists). One thing that comes to mind is using the -j parameter to build the api ref in parallel, but RTD does not let us control that as far as I know (once again, correct me if I'm wrong!).
So I think this 8 minute build time is about as low as we can go if we still want to have dynamically generated docs, which I think we do!

After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

An observation @freddyaboulton:

Looks like for the fraud demo - both pipelines score evenly on fraud. AUC is actually higher on the fraud optimized pipeline as well. Not sure if this is a change in behavior but could make this section a little confusing.


Support for Angular 7.x.x

Angular 7 is currently in beta. I didn't see a branch for 7.0.0 development so I wanted to make sure this was brought to your attention. Might be as simple as a dependency version bump. So far I haven't seen any obvious issues while using codelyzer with<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS><7.0.0 || >6.0.0-beta <7.0.0".
warning " ><EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS><7.0.0 || >6.0.0-beta <7.0.0".

"dependencies": {
    "@angular/animations": "~7.0.0-beta.5",
    "@angular/cdk": "7.0.0-beta.0",
    "@angular/common": "~7.0.0-beta.5",
    "@angular/compiler": "~7.0.0-beta.5",
    "@angular/core": "~7.0.0-beta.5",
    "@angular/forms": "~7.0.0-beta.5",
    "@angular/http": "~7.0.0-beta.5",
    "@angular/material": "7.0.0-beta.0",
    "@angular/platform-browser": "~7.0.0-beta.5",
    "@angular/platform-browser-dynamic": "~7.0.0-beta.5",
    "@angular/router": "~7.0.0-beta.5",
    "core-js": "^2.5.4",
    "rxjs": "~6.3.2",
    "zone.js": "~0.8.26"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "~0.9.0-beta.1",
    "@angular/cli": "~7.0.0-beta.3",
    "@angular/compiler-cli": "~7.0.0-beta.5",
    "@angular/language-service": "~7.0.0-beta.5",
    "@types/jasmine": "~2.8.8",
    "@types/jasminewd2": "~2.0.3",
    "@types/node": "~8.9.4",
    "codelyzer": "~4.3.0",
    "jasmine-core": "~2.99.1",
    "jasmine-spec-reporter": "~4.2.1",
    "karma": "~3.0.0",
    "karma-chrome-launcher": "~2.2.0",
    "karma-coverage-istanbul-reporter": "~2.0.1",
    "karma-jasmine": "~1.1.2",
    "karma-jasmine-html-reporter": "^0.2.2",
    "protractor": "~5.4.0",
    "ts-node": "~7.0.0",
    "tslint": "~5.11.0",
    "typescript": "~3.0.1"
  }

You can check https://github.com/mgechev/codelyzer/issues/709. Once we coordinate with the Angular CLI team, we'll release the next version.

Thanks for the quick response.  Sorry about wasting your time by not searching the closed issues 😄

No worries.


Preprocessing the dialog when OK button clicked.

I show the Dialog to let user input some data, but I cannot check the input and show error to user when they click OK button.
So I want show handle the OK Command before close the dialog.
Use the custom dialog will not work as I cannot close the dialog in my viewmodel when the check is passed.
so add a event options like HandlingOKFunc will be very helpfull, which can return a Handled value to prevent dialog closing.
So the cancel/yes/no button's command should has a same event?
What is the Ursa team's idea?

Why, custom dialog should work, just call RequestClose.Invoke(your result) after your validation succeeded.

I found IDialogContext interface finally. It should work for me. Thanks.


Unknown IVS WEB SDK Error Message

I'm using IVS Web Sdk v1.21.0 in Next.JS / React.JS framework.
I'm getting various error code & error messages from ivs instance.
const onErrorCallback = (err) => {
/.../
const { code, type, message } = err
/.../
}

ivsPlayerRef.addEventListener(IVSPlayer.PlayerEventType.ERROR, onErrorCallback)

I'm getting various different code/message which I'm unable to replicate or understand.
like: code -1, -3 & 4
message: 'No playable format', 'Failed to parse HLS master playlist invalid content type: application/octet-stream' & various others.
Can someone tell me where to find error code or error types

"No playable formats" means that the browser does not support any of qualities listed in the HLS playlist. We're delivering AVC/h264 streams which have wide support, so on a normal computer/browser you shouldn't see this. However, Chromium browsers without proprietary software installed will not work because there is no h264 decoder. If you're using Chromium, you can turn that on with a flag: https://groups.google.com/a/chromium.org/g/chromium-dev/c/9EqnUb2fDvg.
If you're not using Chromium, what it sounds like to me is that one of your packages is intercepting and rewriting the HLS playlist. Failed to parse HLS master playlist invalid content type: application/octet-stream is weird because the HLS playlist should have the application/vnd.apple.mpegurl mime type. If you're not using an IVS HLSstream, unfortunately I can't help you with that.

Hi @johnBartos
Thanks for your support. It really help me to decode the issues.
There are a lot of more error message I'm getting from IVS Player..
I would appreciate, if you spare your time & help me to understand these..
According to me, These all are browser specific errors... As on Chrome browser I'm getting few errors not all.
1. Failed to execute 'addSourceBuffer' on 'MediaSource': The type provided ('video/mp4;codecs="avc1.42001e"') is unsupported.
2. Failed to execute 'addSourceBuffer' on 'MediaSource': The type provided ('video/mp4;codecs="mp4a.40.2"') is unsupported.
3. TypeError: t is undefined
4. DECODER_ERROR_NOT_SUPPORTED: audio decoder initialization failed
5. Unsupported source type
6. The MediaSource was closed upon opening
7. MediaSource.addSourceBuffer: Can't play type
8. t.headers is undefined
9. No playable format
10. PIPELINE_ERROR_DECODE
11. PIPELINE_ERROR_DECODE: video decoder reinitialization failed
12. PIPELINE_ERROR_NETWORK
13. AUDIO_RENDERER_ERROR: audio render error
14. Source create failed
15. TypeError: can't access property "indexOf", t is undefined
16. NS_ERROR_FAILURE (0x80004005)


0.5.1 release

Hi. Any plans on releasing 0.5.1? Quite a lot changes since 0.5.0, would be nice to make use of them.

Certainly! But time isn't my friend at the moment... I will really make some efforts in getting a new release somewhere in May.

Got it. Thanks!


solves issue #57

Update edit_handlers.py to use form['field_name'].value() because I believe this to be correct in Django 2.2.8
Please make sure to check properly because I do not know why the former implementation had been choosen as it was.

Thanks! It was done this way to support Draftail. See https://github.com/wagtail/wagtail-condensedinlinepanel/pull/56


Would like non-async sender on postage::watch

postage::watch::Sender offers, really, just the interface of Sink.  But in fact its implementation never blocks (never returns Pending).
It would be nice to make this a formal part of the API and to have a method which wasn't async.  tokio::sync::watch::Sender calls that method send.

This would be a convenience method, equivalent roughly to *self.borrow_mut() = value.


Update teams-recording-policy.md

list certified partners in alphabetical order
add Insightful Technology to list of partners in progress

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Having 2 cols looking at the same field

This is to enable:
@Fields.number({ dbName: 'siteId' })
siteId: number
@Field(() => Site, { dbName: 'siteId' })
site: Site

(⚠ Has to be merged after the jycouet:fix/schema-builder one as it's based on it)

This should run with tests on all databases - see tests at:
https://github.com/noam-honig/cfp-list/blob/master/src/server/postgres-with-schema.ts#L41
(Yes I need house keeping)
to run these tests:
npm run server-test

I'll have a look 👍

I'll close this one as you can get the id with
id = remult.repo(Task).getEntityRef(item).fields.category.getId();

And userland, you can reduce it with
idOf(Task, item).category.getId()

const idOf = <T>(entity: ClassType<T>, item: T) => {
  return remult.repo(entity).getEntityRef(item).fields;
};


Show .png response as image

Originally reported on Google Code with ID 308
When testing WMS services (http://en.wikipedia.org/wiki/Web_Map_Service), it is very
common to receive an image (.png, .jpg, .gif) as response. It would be nice to see
the image itself in the response pane instead of the bytes. I have attached a screenshot
to clarify this.

When I paste the same url in a new browser tab, the image is shown correctly.

An example of such a (GET) request: http://geo.api.agiv.be/geodiensten/raadpleegdiensten/GRB-basiskaart/wms?SERVICE=WMS&REQUEST=GetMap&FORMAT=image/png&TRANSPARENT=TRUE&STYLES=&VERSION=1.3.0&LAYERS=GRB_BASISKAART&WIDTH=1680&HEIGHT=674&CRS=EPSG:3857&BBOX=468241.9483493314,6638450.68553925,476267.8363194879,6641670.595355849

Reported by ward.delabastita on 2015-04-29 13:00:01

 * *Attachment: 2015-04-29 14_54_34-Advanced Rest Client Application.png![2015-04-29 14_54_34-Advanced Rest Client Application.png](https://storage.googleapis.com/google-code-attachments/chrome-rest-client/issue-308/comment-0/2015-04-29 14_54_34-Advanced Rest Client Application.png)*

It may require change in core request class and replace XHR2 with fetch API which is more flexible for getting a binary data.

+1 on this.  Also support for .svg :)

But svg id a XML data. I'm not sure if the app should display it as an image or as a source.

How about both? One tab with the image and one tab with the xml?

It may work. Right now, however, I'm working on an app upgrade to Chrome Packaged apps. It also brings new UI for the app. It may be good opportunity to do so. But I'll can handle it in couple of weeks since I have a lot of work with upgrade itself.


Development: Add documentation for modeling exercises

Checklist
General

[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Description

Add documentation of Modeling Exercises.
Steps for Testing

Look at the new documentation about Modeling Exercises: https://artemis-platform--5369.org.readthedocs.build/en/5369/user/exercises/modeling/
Review Progress

[x] Review 1
[x] Review 2

Thank you all for your suggestions and feedbacks. I have made changes as per your suggestions.
Now if you all are satisfied with the changes, could you all please approve the PR.  Thanks. (:


After update not all sensors loaded

Describe the bug
I make an upgrade of HA (from 2023.07 to 2023.11) and after that a upgrade PyLoxone (from 0.3.13 to 0.4.10).
After this update, binary sensor dissappear (PIR sensor, magnetic contacts, etc).
Try to uninstall PyLoxone from Devices&Services, remoce Pyloxone from HACS integration, restart HA. No Looxne files left in HACS folder. Also remove this binary sensor from "core.entity_registry" file (was in section Deleted_entities). Still no binary_sensors...
Try to rename one binary sensor in Loxone, but also didn show in Home Assistant...
Firmware of your Miniserver
miniserver 1gen, fw. <IP_ADDRESS>
HomeAssistant install method
intel NUC, virtual box (linux)
Version of HomeAssistant
2023.11.2
Version of Pyloxone
0.4.10
Update pyloxone
yes
Log
https://codefile.io/f/vEFtcBxl7F

@Dohnalradek do you use some smoke detectors? if so can you paste the sections from your structure file where I can see what states your device has? only the part with the smoke detector. you can delete some digits from the uuids if you want.

@Dohnalradek I know what is wrong. You must update your miniserver.
The integration is only valid for newer miniserver firmwares
The problem causes a attribute which was introduced in 10.3
This is from the Doku:
areAlarmSignalsOff
○ State if all alarm signals are disabled (by confirming alarm)
■ available since 10.3

I read about speed problem with newer version of fw on miniserver gen. 1 (esp. with modbus extension) - thats the reason why I stay with older fw (9.3).
But it works with older pyloxone (0.3.13). So is it possible to install this older pyloxone version and it will works again?

@Dohnalradek I think it will not work with newer home assistant installation.

Wow, I erese block fire and water leak (in loxone) - replace by mail notofication. Restart HA and 50 new binary sensors appears! Thats perfect, thank you very much to point where is the problem.
So Loxony 9.3, HA 2023.11 and Pyloxone 0.4.10 can work together!


Dispose the source disposable

When disposing a debounce stream we also need to dispose the source disposable

Summary
I was trying to use debounce in the following scenario
combine(...streams$)
    .debounce(1000)
    .chain(chainStream)
    .subscribe(observer);

But I soon realized that when I tried to dispose the above stream, it would ignore disposing the stream (source stream) that comes before the debounce.
Todo

[ ] Unit tests for new or changed APIs and/or functionality
[ ] Documentation, including examples

It'd be great to add a regression test for this as well.  There's a FakeDisposeSource in test/helper that is helpful in writing tests to verify disposal, and here's an example of using it (there are other examples in the test folder as well).

Looking good @ppoliani.  Thanks for adding the test.  I was porting your fix to @most/core in mostjs/core#107, and I noticed that there's actually another bug in the DebounceSink constructor.  It's not necessary to use dispose.all (or disposeBoth in @most/core), and the source disposable can just be assigned directly to this.disposable.
Take a look at mostjs/core#107 and see what you think.  If that seems right, then you can do the same here.

@briancavalier The change makes sense 😄
I'll include it here, as well.

Awesome, thank you, @ppoliani.  We'll get this out in the next release soon.


tests: Retry pushing docker image

We are quite often running into net/http: TLS handshake timeout errors (e.g. build # 240).
To prevent this, we are now trying 5 times to push the docker image.

lgtm, thanks!


Step 7 is not showing up

When I click on "Diagnose and solve problems" I see:

In your docu it looks like this:

So I can't select the Config Server detector.
I'm using Standard tier, so it's the same as in the docu.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 43a08caa-dcc0-fddd-fafd-2ea447979dc8
Version Independent ID: 3686bdb5-4762-8b37-07cb-2175230b376c
Content: Quickstart - Set up Azure Spring Cloud Config Server
Content Source: articles/spring-cloud/quickstart-setup-config-server.md
Service: spring-cloud
GitHub Login: @KarlErickson
Microsoft Alias: karler

Also the docu is very thin on the config server. How can I manage several config files for several spring boot services in one config server? How do you use additional config urls etc.

And can you explain name & patterns more? How can I make sure that my configs for Service A are only used by Service A, if I have multiple config files in one repo?

And is there a full tutorial for the PetClinic example??

Hi @grgkro , it's only UI change. You need to click Available and Performance tile first to see all detectors listed in the category.

#assign:@v-kevinmic3

#reassign:karlerickson


SA1201 code fix does not respect conditional compilation statements

Consider the following piece of code:
namespace ClassLibrary1
{
    public class Class1
    {
        public void Method1()
        {
        }

#if DEBUG
        public string Property1 { get; }
#endif
    }
}

This will trigger:
Warning	SA1201	A property should not follow a method.	ClassLibrary1

Choosing Fix element order for Property1 will result in the following code:
namespace ClassLibrary1
{
    public class Class1
    {
#if DEBUG
        public string Property1 { get; }

        public void Method1()
        {
        }
#endif
    }
}

i.e., Method1 has been moved into the #if DEBUG code block - changing the semantics of the code.

Classic does report a warning in this instance. I'm thinking we should offer no code fix if the move encounters a directive.

@Noryoko Seems like a reasonable start.

Grabbing this.


[debug][tao bridge] dump cluster input proto in debug mode

Maybe we can add a unit test in the future.


fix(auditLog): Correct nickname type

Please describe the changes this PR makes and why it should be merged:
This should be a string!
Reference Discord API Docs PRs or commits:
https://discord.com/developers/docs/resources/audit-log#audit-log-change-object-audit-log-change-key

Can you run the git hooks, please?


FIX: Generating with mixed adapter batches and with beam search enabled

See #2283
Right now, using mixed adapter batches (introduced in #1558) with beam search generations does not work. This is because users need to pass the adapter names associated with each sample, i.e. the number of adapter names should be identical to the number of samples in the input.
When applying beam search, transformers internally repeats the samples once per beam (or so it looks like). Therefore, we have more samples during generation than samples in the input. Consequently, the adapter names have to be extended accordingly. This is now taken care of.
For encoder-decoder models, we need to be careful. I seems like only the decoder needs to be extended, whereas the encoder receives the original number of inputs. Therefore, when an encoder-decoder model is identified, the extension is only applied to the decoder part.

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


There're no output of cluster info in etcdctl v3

There're output in v2
...
	fmt.Printf("Added member named %s with ID %s to cluster\n", newName, newID)
...
	fmt.Print("\n")
	fmt.Printf("ETCD_NAME=%q\n", newName)
	fmt.Printf("ETCD_INITIAL_CLUSTER=%q\n", strings.Join(conf, ","))
	fmt.Printf("ETCD_INITIAL_CLUSTER_STATE=\"existing\"\n")
}

But they are gone in v3
...
	fmt.Printf("Member %16x updated in cluster %16x\n", id, resp.Header.ClusterId)
}

Finished in PR: https://github.com/coreos/etcd/pull/7050

@fanminshi The CI can't pass. Could you help re-trigger it?

@armstrongli sorry for the delay, let me check.

Usage of the v2 output is shown here, https://github.com/coreos/etcd/blob/master/Documentation/op-guide/runtime-configuration.md#add-a-new-member
The v2 add member output generates necessary env variables to help user to start the newly added member.
The v3 is missing that.


400 errors do not return enough useful information to debug.

This needs to ensure that the full error message from the API is returned,
Otherwise it is very difficult to debug 400 errors.
class ErrAtlasBadRequest(ErrAtlasGeneric):
    """Atlas : Bad Request
    Constructor
    Args:
        c (int): HTTP code
        details (dict): Response payload
    """

    def __init__(self, c, details):

        if details.get('errorCode', None) == 'DUPLICATE_CLUSTER_NAME':
            raise (ErrAtlasDuplicateClusterName(c, details))
        if details.get('errorCode', None) == 'RESOURCE_NOT_FOUND_FOR_JOB':
            raise (ErrAtlasJobError(c, details))
        if details.get('errorCode', None) == 'CANNOT_CANCEL_AUTOMATED_RESTORE':
            raise (ErrAtlasBackupError(c,details))
        if details.get('errorCode', None) in ['ATLAS_MAINTENANCE_ALREADY_SCHEDULED',
                                              'ATLAS_NUM_MAINTENANCE_DEFERRALS_EXCEEDED']:
            raise (ErrMaintenanceError(c,details))
        else:
            logger.critical(f"A generic error was raised")
            logger.critical(details)

        super().__init__("Something was wrong with the client request.", c, details)

Hmmm, in testing I do get back details...
Test:
 def test_00_handle_400s(self):
        """
        Force a 400 error so we can test details being passed through.
        """

        cluster = self.a.Clusters.get_single_cluster(cluster=self.TEST_CLUSTER_NAME)
        update = self.a.Clusters.modify_cluster(cluster=self.TEST_CLUSTER_NAME,cluster_config=dict(backupEnabled="whatevs"))
        pprint(update)

Error_Handler: CRITICAL: A generic error was raised
Error_Handler: CRITICAL: {'detail': 'An invalid enumeration value whatevs was specified.', 'error': 400, 'errorCode': 'INVALID_ENUM_VALUE', 'parameters': ['whatevs'], 'reason': 'Bad Request'}

So this should show up in logs....
But perhaps should pass the details all the way up?

The getAtlasResponse method of the error object will return the full error information from the API.
That said, I will go ahead and submit a PR to include more of the origin error information in the base error.


[3.x] Make theme item overrides more obvious in the Inspector

A 3.x version of https://github.com/godotengine/godot/pull/51119 but without property renames, just Inspector reorganization.

Now, this would obviously render some tutorials obsolete, but I think we can live with that if we get better and more clear organization in return. If not, feel free to close unmerged. 🙃

Thanks!


Custom telemetry source using '--telemetry-source' argument

Why are we doing this? What's the use case?
Doing this makes things harder on us on the back end - queries in our logs get harder since there's now an unbounded set of telemetry sources instead of just a couple. And that help text for the switch is completely inadequate - it'll be displayed in every command help screen and doesn't explain a thing about what this is actually FOR.
I really don't want to do this unless there's a really compelling reason I'm not seeing right now.

Hello @christav , sorry, should've written a detailed description, to clarify, why we need this change.
This argument will allow other applications to use appcenter-cli as a mediator between itself and App Center portal.
More concretely, right now we are working on a App Center extension for Visual Studio Code:
https://github.com/Microsoft/vscode-appcenter
We mostly use autogenerated REST API client to issue commands to App Center, so we can set correct telemetry source ourself in those places:
https://github.com/Microsoft/vscode-appcenter/pull/108/files
But in one place, to launch UI tests, we use appcenter-cli:
https://github.com/Microsoft/vscode-appcenter/blob/master/src/tests/appCenterUITestRunner.ts#L112
Right now in a telemetry dashboard it will look like user launched UI tests from a appcenter-cli, but in reality he did it from a VS Code extension
If we start tests using appcenter-cli while passing it our telemetry source we will be able to easily see how our extension are used, by filtering telemetry by our telemetry source
I hope that I was able to show the reasoning behind this changes and I will be happy to hear your thoughts about how we can improve this solution

Since this is completely internal, rather than hacking up the command line parameter parsing I'd rather see this in an environment variable instead. Plus that way you don't need to set it on every command, but instead just set it in the environment for the shell you're running the CLI in. Much less intrusive, doesn't show up in the help, and gives you what you want.

@christav Thanks for a great idea, we are going use it and implement passing of telemetry source using environment variable, so I am going to close this PR and open another one, when we get everything ready


Addition of app release or app preview for the project

What's wrong with the existing documentation
There should be an app release or preview version of the project available, allowing users to quickly download and test the app's functionality on Android devices. This version will enable developers, testers, and stakeholders to easily access the app, review its features, and identify any issues or areas for improvement. Providing a preview or release version ensures efficient feedback and helps in refining the app before the final release.This helps in testing the app a lot easier.
Either there should be an app release file or a link to download the app.
The app download link should be given in README file.
Add ScreenShots

or

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@a-m-a-nkumar Currently, this app is not fully ready. Once all the programs and features have been added, we will release it.


KAKFA-9612 CLI Dynamic Configuration with file input

Suggested change for KIP-574 (still under discussion)
Add an option to kafka-configs.sh --add-config-file that adds the configs from a properties file.
Testing: Added new tests to ConfigCommandTest.scala
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@rajinisivaram, @cmccabe, @gwenshap would you mind taking a look at this PR?

Thanks for your review, @omkreddy! I've rebased and incorporated your feedback.

ok to test

JDK 8, 11 tests passed. JDK 14 test failures are not related. Merging to trunk.


Channel closed due to 404 when ExchangeDeclarePassive

I understand that the documentation clearly states that a Channel is closed when an error is returned. Seems a little excessive in the case of a DeclarePassive in that one is trying to test for existence. In my use case I am waiting for a server side process to create an exchange and am looping and testing and creating a new channel as a result of testing for existence doesn't make sense.

@mikechack this is how the server works. It closes the channel on any channel exception. Use a one-off channel.

Yup, there is not much that this client can do as it's expected behavior.


New component: Keynote

I am thinking of adding HTML presentation component for JSF.
I'm not sure if we should use the name "keynote" or the original name "revealjs".
reveal.js is an open source HTML presentation framework. It's a tool that enables anyone with a web browser to create fully-featured and beautiful presentations. It can be accessed from https://revealjs.com.
<pe:keynote>

Slide 1

Slide 2

Slide 3

</pe:keynote>

Renders

Slide 1

Slide 2

Slide 3

OK this is pretty slick.  I would vote for Keynote or SlideDeck.   KeyNote reminds people of Apple's product and PowerPoint is a lame name :)

I will go on with Keynote then.

Why should this be a component instead of plain JavaScript? See also the discussion at #124

@jepsar thanks, i read the discussion. Do you think it is not necessary or does it need improvement like database interaction?
Actually i thought this to use for one of my projects. Instead of providing a guide or an user manual provided by customer (uploading/downloading power points), they can prepare an HTML presentation.

@aripddev I appreciate your work, but I don't really see the point if it being a component. Unless maybe there are events that need to be decoded. I think you can achieve the same by simply render the needed HTML and initialize Reveal. Or did you run into issues while doing that?

By the way, the whole "database" thing is a non arg in the other issue.

@jepsar I didn't think about it but you gave me a good idea about "event". In my case, i can use an "event" to check if user complete to read the slides. After reading slides is complete, the form or button can be submitted. What do you think?
Currently I use "disabled" and "rendered" attributes with bean properties.

Yeah, Reveal has some events that can be included.
Also, I would have done:
<pe:keynote>
  <pe:keynoteSlide>Slide 1</pe:keynoteSlide>
</pe:keynote>

or
<pe:keynote>
  <h:panelGroup>Slide 1</h:panelGroup>
</pe:keynote>

instead of
<pe:keynote>

Slide 1

</pe:keynote>

so you can use the rendered attribute on the slides.

@melloware why did you use the 12.0.0 milestone? It can be added to 11.0.x, right?

@jepsar :) I already did it as pe:keynoteItem

I didn't think this component would be complete and tested in time.  PrimeTek was supposed to release 11.0.0 on Monday but did not so maybe today.

Okay, but we can release it with 11.0.1 or 2

oh yeah of course.  I just marked it for 12.0.0 right now so it wasn't in 11.0.0

I just moved it to 11.0.1.

Nice component...
If I see that right, you could use it also for a gallery (only show pictures) also ?
For me it would be also important, that everything can be loaded from database.
So you can have than two Entities:
Keynote

Setting1
Setting2

KeynoteItem

HtmlContent
Setting1
Setting2
....

And those can be than use to fill the properties from Primefaces:
<pe:keynote setting1="valueFromDatabase">
But this should not be the problem. In you bean you have than:
private Keynote keynote;
private List keynoteItemList
and than something like this:
<pe:keynote var="#{mybean.keynoteItemList}" setting1=#{myBean.keynote.setting1}>
Also events (as already mentioned) would be nice:

So you can track if a user completed the slides

@rider87 wait until it is merged and simply try c:forEach and events has been taken care of.

Looks like we have some Sonar issues to clean up.


[Feature]: Changing the flag to US flag in currency widget's default option

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
The default option when using the currency widget is a lesser known flag corresponding to USD. It should probably be US flag.  This is on Appsmith cloud.

Why should this be worked on?
US $ is a reasonable default to have in the currency input. However it makes sense to put the US flag there, instead of the current option (I don't know what country that is) since for most cases where people will use US $, they're probably based in USA.
I'd also imagine it might be a relatively minor fix.

@Rishabhkaul congrats on opening your first issue :)
@dilippitchika this should be a pretty minor fix right?

We have multiple conflict around this, specially the US - types (even comes up with country codes), here is a similar issue #18136
@dilippitchika did document the way forward here

@dilippitchika can you frame what the final option needs to look like, we will modify our object accordingly!

@somangshu https://www.loom.com/share/a50c531a48fb475bab890d7b500148c4

@dilippitchika makes sense, so for the currency thing we will do currency symbol - currency code > ₹ - INR
While we are thinking about this, Should we also rectify the problem with country code in phone input widget? #18136

We can try it out in phone input but there we need to write the countries because +1 or +2 on it's own is not clear to users. So we have to mention all countries either in short code or full name in the field. But that's a ux issue.

Hey! Is anyone working on this issue? Or is anyone open to collaborating?

Thanks for showing interest on the issue @Vishrut19, We would like to manage this issue internally.
Can you please go though other good first issue and see is something is interesting to you


Checking sessions between servers using ApiGility

Hi -
I am using ApiGility to manage SSO between my servers and it works great. My question however is this - is it possible to use ApiGility to check sessions?
Here is an example:
I am logged into server A, and access server B. At this point I need to login with my server A credentials, I enter them into a login form, ApiGility calls server A, authenticates, and I am logged into server B.
Ideally, I would like server B to call server A and check if I have a pre-existing session before offering the login page. If I am logged into Server A - then to simply grab the credentials and log me into server B.
Is this possible?

Created quite an elegant solution for this called: SessionMagnets which uses ApiGility to test sessions between applications. Inbox me if anyone is interested.


moved order of defined constants to take care of COCKPIT_BASE_URL for api request

If COCKPIT_BASE_URL was defined in defines.php for using Cockpit in a subfolder, the backend worked and the api calls redirected to login.
That happened, because COCKPIT_API_REQUEST was never set to '1' after comparing the auto-guessed $COCKPIT_BASE_URL instead of defined COCKPIT_BASE_URL.
Tested by me and @PilsFour an a shared host on strato.
fixes: #873
references: https://discourse.getcockpit.com/t/cant-access-api/221

Thanks!


jest moduleNameMapper issue for file aliases

hi, in use case like below
"paths": {
      "@app-utils": ["src/utility.js"]
 }

such a config occurs;
{
  '^@app-utils/(.*)$': '<rootDir>/src/utility.js/$1'
}

this config is not working correctly for jest. it has to be like this for it to work correctly
{
  '^@app-utils$': '<rootDir>/src/utility.js'
}

https://github.com/oklas/react-app-alias/pull/77


Toggled checked Inputs become latched.

If you drag a root element with a checkbox input child. The childs checked state effectively gets latched.
e.g.
Drag with checked false => stays false
change state to true and drag => stays true
change back to false => stays true.
this is because it is not being cleared from the savedInputChecked array when saveInputCheckedState is being re-run.
Basic solution would be to re-initialise the savedInputChecked array e.g.
function _saveInputCheckedState(root) {
    //Re-Init saved input checked array to stop latching
    savedInputChecked = []

    var inputs = root.getElementsByTagName('input');
    var idx = inputs.length;

    while (idx--) {
        var el = inputs[idx];
        el.checked && savedInputChecked.push(el);
    }
}

Another solution may be to check then remove the reference when checked is false

This may also be affecting bootstrap-switch elements inside a Sortable.  Still trying to get to the bottom of my issue.

It's taken me forever to track down the cause of this issue, but I'm also experiencing it. Like you said,if you check and then uncheck the checkbox and then do a drag, the checkbox will revert to the :checked state.

Looks like this is a duplicate of https://github.com/RubaXa/Sortable/issues/1052

Hi !
This is to fix the problem:

Add 1 line of code:
            savedInputChecked = []; // clear the array

This has been solved in the latest.


Export types from root

Describe the desired behavior
All component types should be available to import from the root/barrel, e.g.: import { InfoListItemProps } from '@pxblue/react-native-components'
Describe the current behavior
Types are exported from the component file, but they are not exported from the root index file.
Is this request related to a current issue?
Related to an issue with Snack (unable to do deep imports from a dependency node_module)

Just checked this and all of the types are exported from the root currently in the dev branch.


Automated cherry pick of #3010: lbagents: 禁止在公有云、托管物理机上部署

Cherry pick of #3010 on release/2.10.0.
#3010: lbagents: 禁止在公有云、托管物理机上部署

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@fb7ede5). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #3016   +/-   ##
================================================
  Coverage                  ?   7.46%
================================================
  Files                     ?     414
  Lines                     ?   66091
  Branches                  ?       0
================================================
  Hits                      ?    4931
  Misses                    ?   60737
  Partials                  ?     423

Flag
Coverage Δ

#aFlag
7.46% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/loadbalanceragents_deploy.go
0.34% <0%> (ø)

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 fb7ede5...2999930. Read the comment docs.

/lgtm

/approve


Update download link

The download link was broken, as reported in #179. I searched for the plugin in the Extension Warehouse and found the updated URL, and updated the README to match.

Thank you very much for the fix! 👍👍

Thank you very much for the fix! 👍👍